Getting "HTTP ERROR 400" while sharing with Twitter

The other networks work great.
Any idea?
Thanks!

Hello @abeltodev,
I can't help you with so little information on your bug.
Feel free to re-open the issue with more details if you are still experiencing it.

Hi @nicolasbeauvais,
Thank you for your response. I'm not sure which information could help you. My code:
                       <social-sharing inline-template                                         :url="seo.meta['og:url'].content"                                         :title="seo.title"                                         :description="seo.meta.description.content"                                         :hashtags="seo.meta.keywords.content"                                         twitter-user="##">                             <div class="q-mt-md">                                 <network network="twitter">                                     <q-btn icon="fab fa-twitter" size="sm" class="q-mr-sm">                                         <q-tooltip>Share with Twitter</q-tooltip>                                     </q-btn>                                 </network>                             </div>                         </social-sharing>
Version: "vue-social-sharing": "^2.3.3",


Remove usage of platform-intrinsics where unadjusted works

As suggested here!

Unfortunately not because of the aggregate return, which needs to be a specific LLVM type that otherwise doesn't manifest as such (since tuples aren't ABI-safe)


Add files via upload

What does the program do ?
This program help you to understanding "for" looping in java.
In what programming language it is written?
This program is written in java.

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


possible memory leak, in an certain length in a each block

Describe the bug
In my current project, I'm looping through each block x amount of time that is determined by how many items are in my globalStore. I noticed my memory usage through this went from around 460mb up to over 1000.
Reproduction
https://github.com/Bradley-H/Invoice-App
npm run dev
go to any invoice on your screen, and click on the edit button (desktop view) and should bring out a modal, thats when the memory usages go up.
I deduced it has something to do in my invoiceModal.svelte in the each block at the Array(length) when I remove it, there is no memory leak

Logs
No response
System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (12) x64 Intel(R) Core(TM) i5-10500H CPU @ 2.50GHz
    Memory: 8.48 GB / 15.78 GB
  Binaries:
    Node: 16.13.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.1.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (96.0.1054.62)
    Internet Explorer: 11.0.19041.1202
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.4
    @sveltejs/adapter-netlify: ^1.0.0-next.36 => 1.0.0-next.36
    @sveltejs/kit: ^1.0.0-next.201 => 1.0.0-next.202
    node-sass: ^7.0.0 => 7.0.0
    sass: ^1.45.0 => 1.45.1
    svelte: ^3.44.0 => 3.44.3
    svelte-check: ^2.2.6 => 2.2.11
    svelte-preprocess: ^4.9.4 => 4.10.1
    tslib: ^2.3.1 => 2.3.1
    typescript: ^4.4.3 => 4.5.4

Severity
blocking all usage of SvelteKit
Additional Information
No response

We would need some additional information indicating a bug in SvelteKit before we could look at this. If there is a bug here, it seems more likely to be a bug in Svelte rather than SvelteKit. But even still, there's no clear indication there's a bug here rather than just some sort of user error


What does the initial-scratch-message mean?

Hello.
Further, I don't understand why the word "Hacking" is used.
Because I am a non-English speaking person. I am sorry if I asked a silly question.
Thank you very much.

"Hacking", to programmers, means "programming clever solutions". Non-programmers use it to mean "breaking into computer systems", but I think the users of this config are programmers, so I use this phrase to help them feel welcome.

Thank you very much！


README.md: add go-hfuzz

Hi Robert,
recently, I created go-hfuzz for a project of mine.
Here's a PR to add it to the readme.
Like my python-hfuzz, this does not instrument any code.
It's just glue code to use libhfuzz.a from go.
Ofc, feel free to close the PR if you don't feel like merging it ;)
Cheers!
P.S.: Also did s/python-fuzz/python-hfuzz.

Nice! thanks for letting us know :)


Use liqwid-nix; Bump plutarch

CI not passing, this will have to be on staging. I'll update it once Staging get merged into main


adding all ledger device targets to build.rs

Adds missing targets for ledger device types. These devices don't have access to 64-bit pointers/ atomics

For some information, Ledger is a Secure Element chip device that you can read more about here, https://shop.ledger.com/products/ledger-nano-x


Rename downloadable directories to avoid confusion

When downloading an IPFS installation folder from the website, a folder called ipfs has a file called ipfs in it. This can cause mild confusion for users who move the entire folder to their bin, instead of just the ipfs executable. Renaming the top-level folders to ipfs-install, ipfs-download, or something similar would make sure this is never a friction point. See https://github.com/ipfs/ipfs/issues/88#issuecomment-139282006.
This may be a small idea, but in case of mass-adoption it might be a good move just to make this clear.

sure. we cannot rename the directory (it's handled by something else) but we should be able to tell tar to rename the output?

This process is part of dist.ipfs.io. @RichardLitt if this is still an issue for you please open an issue there.

Looks like this was solved by renaming to go-ipfs. Thanks for closing, @dignifiedquire!


Please don't abuse tag helpers

Tag helpers are really neat and useful for generating or modifying HTML but I'm afraid they are being used more and more for things they're not well suited for.
As an example, I've seen this shown at Build 2017:

(I don't know if this actually shipped in the final product, I wasn't able to find it.)
This to me seems really like an if statement and I see no reason to hide it inside a tag. I'm afraid tag helpers could be overused just because they're new and look nice (if you have a hammer, everything looks like a nail). I hope there is some thought put into decisions about making these before non-html-related tag helpers are introduced as part of the product.
A good example is the environment tag helper that was introduced from the very beginning:
<environment names="Development">
</environment>
<environment names="Staging,Production">
</environment>

This seemed fine as a replacement for an if-statement but it was lacking one major feature - an else clause. As a result, it was later modified to support include/exclude but I feel like the underlying problem here is that we're trying to replace C# with tags.
I'm happy to have tag helpers for actual tags or html generation like validation, as a replacement for DisplayFor or even for including a partial view - after all partial views are just another big chunk of other html that gets generated. But tag helpers just aren't suited for things like conditional or any kind of logic.
What I really hope for is that there won't ever be something like a for-loop tag helper in the future. I'm sure such a thing could even work for most cases but never for all of them, only leading to inconsistencies. Please don't blindly start using tag helpers for everything until we later realize that they can't actually be used for everything. There will always be need to use C# for certain logic - and Razor syntax already makes that really convenient. Let C# do what it's best at and let's use tag helpers for what they are good at - simplifying HTML. In my view, all tag helpers should be declarative and never used as a replacement for imperative code.

asp-repeat-for="enumerable"? 🧐

Ok fine, we'll cancel our plans for this new syntax:
<asp-var name="count" type="System.Int32" />

    <asp-for start="0" var="count" max="10">
        <asp-html>

        </asp-html>
    </asp-for>

:trollface:

cc @DamianEdwards @NTaylorMullen
Ok in all seriousness, I don't think there's much to worry about. Of course we don't plan to reimplement C# in tag helpers, but there will always be some overlap: there's nothing a tag helper can do that you can't write enough C# code to do the same thing. But, if we think there's sufficient value, and tag helpers add enough valuable semantic meaning, we'll continue to write them.
The <environment> tag helper is a great example. It's true that the aspect of it that is just an if(foo) or if(!foo) is quite trivial. But try to write the actual code for it in C# (without the tag helper) and it's a bit trickier because you have to get the IHostingEnvironment from the service container first. Now the "1 line of C# code" is a directive, variable declaration, explicitly involves DI, plus the C# code.

I do realize it's a little more complicated to get the environment, but since it's so fundamental and already a first-class concept, maybe it could have been a property on the view itself. Then it would be as simply as this:
@if (Environment.Name == "Development")
{

}
else
{
    <otherHtml />
}

There will always be more flexibility doing this in C#, because you just might want to do something more interesting like if (Environment.Name.StartsWith())... etc. A valid use-case might be to extract the logic of which environment you want to include - maybe into an extension method on the environment itself, if you have more than 3 environments let's say. It's true that you could always write your own tag helpers for that, though it would be a little more involved.
That said, I probably wouldn't have introduced the environment tag helper but I'm ok with it as it is now since you can do include & exclude - that's probably good enough for the majority of scenarios. But I was getting a little concerned after seeing the asp-if stuff.

The actual intent of the asp-if example was as a visibility predicate, not control flow. I just preferred the terseness of asp-if rather than asp-visible or asp-show or asp-when or asp-predicate, etc. It just means "render this element if this expression equates to true".

But what's the difference between this and wrapping the entire thing inside @if (condition) { }? I agree it's nice and short but you might soon realize you not only want to show something based on a condition, but also show something else otherwise. If you had the actual if-statement, you would just put an else block after that but with this tag helper you would have to replicate the same condition but make it negative. That just doesn't seem right.

My opinion, is that it depends on the circumstance.  One of the reason's that angular was so popular was that it allowed you to add simple logic to html without having to worry about the underlying javascript code.  It's much easier to reason about tags without the clutter of curly braces (or in the case of one-off tags i.e. a img tag).  But there are definite reasons to also stick to c# syntax for all cases.
But it does make life easier when you are working with designers or when you are prototyping something quickly where this can be useful.


Review current rialto logs and find logs which have too high level

Summary: Review current rialto logs and find logs which have too high level
Type: Error Fix
Test Plan: Unit Tests & Full Stack
Jira: RIALTO-193

https://jenkins.code.rdkcentral.com/job/github-triggered/33434/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/1954/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4118/ started.

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/9826/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/9827/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/9841/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2006/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4198/ started.

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/9967/

https://jenkins.code.rdkcentral.com/job/github-triggered/33688/ started

https://jenkins.code.rdkcentral.com/job/build-component/71257/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/33692/ started

https://jenkins.code.rdkcentral.com/job/build-component/71260/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2084/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2086/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4354/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4355/ started.

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10289/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10290/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10293/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10294/

https://jenkins.code.rdkcentral.com/job/github-triggered/33797/ started

https://jenkins.code.rdkcentral.com/job/build-component/71429/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2140/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4518/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/8a2101e5cfa11a0e1ca7c7f7b062ede6

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10592/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10593/

https://jenkins.code.rdkcentral.com/job/github-triggered/33827/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/33829/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2156/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2157/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4666/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/33938/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2214/ started

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10906/

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10905/

https://jenkins.code.rdkcentral.com/job/github-triggered/33977/ started

https://jenkins.code.rdkcentral.com/job/build-component/71662/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/33979/ started

https://jenkins.code.rdkcentral.com/job/build-component/71663/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2235/ started

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10973/

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10972/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10975/

https://jenkins.code.rdkcentral.com/job/github-triggered/33985/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2238/ started

https://jenkins.code.rdkcentral.com/job/build-component/71665/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/33991/ started

https://jenkins.code.rdkcentral.com/job/build-component/71667/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/2241/ started

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10981/

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10980/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/4728/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10984/

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10988/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10989/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/10990/


Log cost/price per epoch

Would it be possible to log the price per epoch and per training, along the equivalent number of kms in car?
It would be quite informative.
Thanks

Duplicate of #38.

Sorry about that, didn't see it was created in the first place.


Add feature to update internal pull requests with cover letters

@peter-fogg, I saw your hackathon project, and thought that it could be easily folded into our existing webhooks bot. Does this look right to you? (I added some automated tests, as well.)

@singingwolfboy Sorry I haven't gotten to this earlier -- it's been a few days of vacation/illness. Thanks for doing this work!
I really like the idea of consolidating all our webhooks into once place, so I think this will be the right approach. That being said, I'm not sure if I want to merge this just yet. The cover letter template I used for the hackathon project was just lifted from the TNL PR template, and I'm not yet totally satisifed that it's right for all teams. I'd also like to make this an opt-in system for at least a little while -- I'd discussed with @sarina about having the template applied only when a certain label is applied to the PR. That way we can socialize the standard version and get feedback before flipping it on for all PRs.
FYI @dan-f

Nice @singingwolfboy. This looks right to me, or at least it does what we did in the hackathon.
That being said, I think we should come to some consensus on default cover letters. We currently have many templates/guidelines:

https://edx.readthedocs.org/projects/edx-developer-guide/en/latest/process/cover-letter.html
https://openedx.atlassian.net/wiki/pages/viewpage.action?pageId=25559771
https://openedx.atlassian.net/wiki/display/TNL/TNL+Pull+Request+Template

Based on the comment threads on some of those openedx wiki pages, it looks like @sarina has seen a lot of attempts to standardize on a cover letter, so I'd love to see her thoughts.
I personally would like to see the default cover letter template be friendly to the open-source community. I'd also like to see a feature by which applying a certain label gets a customized cover letter template that's more pertinent to the actual PR. That'd require a web interface in the webhook where users could log in and associate templates with labels. Does that idea interest anyone else?

@singingwolfboy I spoke at length with @peter-fogg and now is not the right time to be including this in our bot. I will continue to work with Peter to first roll this out to all internal teams, and get buy-in on a general basis from all teams on a standard template. As Dan notes, we need to get consensus from everyone; rolling this out right now I think will only cause more confusion.
Once we have consensus, we can open this up to all contributors, including updating our existing documentation. I'm eager to get this rolled out by the end of the quarter, as I agree this will be great for both internal and external developers.

Sounds good to me. I'll leave the pull request open for now, but if we decide later on that it would be better to close it, we can do that.

This is a checkbox test

[] Test

FYI, this feature is now documented as something we want this project to do. Maybe it will happen, even after I leave! http://openedx-webhooks.readthedocs.org/en/latest/future.html#add-a-review-checklist-to-pull-requests

@singingwolfboy: I fixed it!


vite.middleware but for WinterCG Request/Response

Description
https://vitejs.dev/guide/ssr#setting-up-the-dev-server shows how to setup a dev server for ssr using express. with more and more server frameworks moving away from the Node.js Connect style API to be more Request/Response, I wonder if maybe Vite should have middlewares for those applications too?
Suggested solution
import { Hono } from 'hono'
import { createServer as createViteServer } from 'vite'

const app = new Hono()
const vite = await createViteServer({
  server: { middlewareMode: "winter-cg" },
  appType: 'custom'
})

app.use("*", (c, next) => {
  // run the vite middlewares
  await vite.middlewares(c.req, c.res, next)
});

or similar to make it compatible with req/res frameworks?
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Hi @juliusmarminge
I'm a creator of Hono. We are working on a similar project for Hono. Below is a dev server Vite plugin for Hono and fetch-based applications.
https://github.com/honojs/vite-plugins/tree/main/packages/dev-server
Whether or not Vite itself will implement these things is a matter of discussion, but for now, try this.

Hi @yusukebe , I think the dev-server plugin for Hono has a limitation with websockets currently, specifically for Node and Bun.
I'm wondering if it's possible to expose a createMiddleware function as an optional function from the package instead of a whole Vite plugin?
The limitation I encountered is with Websockets and mainly because of its forced-way of writing the Entry Point in Dev.
It forces you to always do this at the entry point:
export default {
  fetch: app.fetch,
  port: 3000,
};

as well as making sure your pnpm dev script calls vite

In Node
The dealbreaker with Node is that I can't use serve() during dev. Which I think is the only option for me to run websockets in Node: https://github.com/honojs/vite-plugins/issues/148
In Bun
Since I can't use Bun.serve in dev either, websockets don't seem to behave correctly. Even if you can attach websocket on the entry point: https://github.com/honojs/vite-plugins/issues/140
export default {
   fetch: app.fetch,
   port: 3000,
   websocket: websocket
}

If there was a createViteToHonoMiddleware function, maybe we could have the option to do something like this at the handler instead:
import { resolve } from 'node:path';

if (privateConfig.NODE_ENV !== 'production') {
  const root = resolve(__dirname, '../..');

  // Instantiate Vite's development server and integrate its middleware to our server.
  // ⚠ We should instantiate it *only* in development. (It isn't needed in production
  // and would unnecessarily bloat our server in production.)
  const vite = await import('vite');
  const viteDevMiddleware = (
    await vite.createServer({
      root,
      server: {
        middlewareMode: true,
        hmr: { port: privateConfig.HMR_PORT },
      },
    })
  ).middlewares;

  // @ts-ignore
  app.use(createViteToHonoMiddleware(viteDevMiddleware)); // 👈  createViteToHonoMiddleware
}

And then be able to keep pnpm dev as tsx server.ts instead of vite.

I am basing this on the implementation here: https://github.com/blankeos/express-vike-websockets wherein both HMR and websockets work in a custom express server with Vite.

Hi @Blankeos
createViteToHonoMiddleware sounds good.
This is not only for the Vite matter, but if we can use the Connect in the Hono app, we can use vite.middlewares.

@yusukebe Let me know if you want to move this discussion elsewhere, but after researching some more, I found that it's actually overcome the limitation I mentioned. (But only on Node).
I have recreated the express-vike-websockets example above but in Hono:
https://github.com/blankeos/hono-vike-websockets
I think a createViteToHonoMiddleware that works on both Bun and Node would be cool. But I don't know if it's possible because Bun does not expose an http.Incoming for the Bun.serve like Node does with @hono/node-server serve(..) (but maybe I'm just missing something).

Credits to @phonzammi's solution on Discord for making Hono + Vite's createServer work.

Great Hono has a solution however a WinterCG middleware is still needed for other modern servers like Elysia, H3 (v2), etc..

https://github.com/vikejs/vike-node/blob/main/packages/vike-node/src/runtime/adapters/connectToWeb.ts
Supports streaming also
const webHandler = connectToWeb(vite.middlewares)
const response = await webHandler(request)
return response


NIFI-7505 Add InvokeHTTPRecord processor

Summary
NIFI-7505 - Add a record-based processor that executes an HTTP request for each record in a flowfile. Many of the options are similar to the InvokeHTTP processor's, but adapted for record-based operation.
Request attributes like the URL, headers and request body can vary per record, using RecordPath queries to extract data from each input record. The response body and various metadata (headers, status code, etc) can be written to each record.
The processor can be configured to route the entire input flowfile to a failure relationship if an error is encountered, or failing records can be split off into a separate flowfile. HTTP requests can be retried a configurable number of times.
The response body can optionally be parsed using a record reader service. As well as allowed the parsed body to be included in the output record, in this mode the response body can also be used to determine whether the request is treated as having succeed or failed. This can be useful for APIs that always return a 200 status code, with error information in the response body.
This PR isn't complete yet (it still needs tests), but I've created it in case anybody has any feedback on the approach I'm taking.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks for the feedback!
I did base InvokeHTTPRecord quite heavily on InvokeHTTP, so if that's not a good example to follow then I can understand that changes are required. I can definitely see the value in using WebClientServiceProvider - that would remove a number of config properties from InvokeHTTPRecord, as well as making the code easier to test.
Having the retry handling and record splitting all inside the one processor does feel a bit like it's going against Nifi's default philosophy (break tasks up into multiple simple processors). My first instinct when building a flow was to split up the input records into individual FlowFiles and then handle HTTP requests individually. Synchronising and reassembling everything at the end got pretty painful, though, especially handling retries.
The current InvokeHTTPRecord approach was largely copied from processors like LookupRecord (i.e. do the same operation for each record in the FlowFile). I could remove the record splitting logic from InvokeHTTPRecord and let splitting be handled later by a separate SplitRecord processor, if that'd be cleaner.
I can write up an outline of how I see the processor working on the Jira issue, if that's the best way to move forward.

Thanks for the reply @pspiller!
One other thing to note about HTTP communication in relation to this Processor, often REST services will have separate endpoints for batch processing. This is much more efficient in terms of network communication and transaction handling, but of course it depends on the service in question.
Another point to consider is that while InvokeHTTP supports all of the standard HTTP verbs, a record-oriented Processor only seems to make sense in the context of sending a request body, such as with POST, PUT, or PATCH.
Given these points, I think it would be best to close the current PR and scope out the options on the Jira issue.
With the goal being a generic reusable Processor, it would also be very helpful to highlight some well-known REST services where this proposed Processor would make sense. That would help highlight whether it is a good fit for something generic, or whether it ends up being too complicated for general use.

One could consider the PutElasticsearchRecord processor as a service/application-specific instance of this - operations are executed per-Record based upon processor properties and/or fields set within Records
For example, the Index Operation (index, update,  delete) can be set based upon a Record field. Similar options are available for other parameters in the Elasticsearch _bulk API call. These settings dictate the body of the request passed to Elasticsearch
A similar approach could be considered here, i.e. allow a Record Path property to set the request Verb based on the value of a field within each Record. In not, however, all that used to using InvokeHTTP, so such an approach may not be so relevant at such a generic level

Thanks for the additional input @ChrisSamo632. The Elasticsearch components are a good example of a more specific use case for record-oriented HTTP communication. As you mentioned, it is more difficult to implement a more generic solution given the unbounded options available for HTTP request and response strategies.
With this background, thanks again for proposing this component @pspiller! As recommended earlier, I think continuing the discussion on the Jira issue would be the best way forward, so I will close the pull request for now, pending further discussion.


tests: accelerate docker load

Signed-off-by: Long<EMAIL_ADDRESS>Description

Issue reference

Please reference the issue this PR will close: #3580
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@wcs1only can you take a quick look? I remember we got burned in the past because we didn't have the Always policy in tests?

@wcs1only could you help look?

@wcs1only could you help look?

/ok-to-test

The default

@wcs1only can you take a quick look? I remember we got burned in the past because we didn't have the Always policy in tests?

Default is still Always, it allows override of the behavior. I am OK with this change as long as the OK to test passes.

@artursouza @yaron2 all tests pass!


Update details on how example files are filtered.

bringing instructions in line with how the x-api and configlet work
(see exercism/configlet@27424c1e081d07544eb1009e3ecefbf5a7e62cba
and exercism/xjava#50 for details).

Thanks, good catch!

Pull a weed, plant a seed... :)


WebScreenshot: Add ability to create authorized Imgur screenshots

Workflow name: WebScreenshot
Workflow version: 1.5.10
Alfred version: 4.0.2
macOS version: 10.14.6

Summary
Currently all screenshots made are made anonymously. I would like for there to be a setting so I can authorize all screenshots made and have them be uploaded from my account.
Thank you.

This has been discussed before.


auto-close app after inactivity

may share code with Kifu/#70

finish keremkoseoglu/incubus#1 first


Feature/core #201

Codecov Report

Merging #202 (a69a09d) into master (01c3eda) will decrease coverage by 2.05%.
The diff coverage is 75.37%.

@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
- Coverage   97.56%   95.50%   -2.06%
==========================================
  Files          66       72       +6
  Lines        2668     2916     +248
==========================================
+ Hits         2603     2785     +182
- Misses         65      131      +66

Impacted Files
Coverage Δ

glasses/models/AutoConfig.py
93.33% <ø> (ø)

glasses/models/AutoModel.py
94.73% <ø> (ø)

glasses/models/classification/deit/__init__.py
0.00% <0.00%> (ø)

tmp.py
0.00% <0.00%> (ø)

glasses/nn/regularization.py
95.00% <95.00%> (ø)

glasses/models/classification/resnest/__init__.py
96.36% <96.36%> (ø)

glasses/models/classification/__init__.py
100.00% <100.00%> (ø)

glasses/models/classification/resnetxt/__init__.py
92.85% <100.00%> (ø)

glasses/models/classification/vit/__init__.py
96.42% <100.00%> (+0.55%)
:arrow_up:

glasses/nn/blocks/__init__.py
100.00% <100.00%> (ø)

... and 11 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #202 (a69a09d) into master (01c3eda) will decrease coverage by 2.05%.
The diff coverage is 75.37%.

@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
- Coverage   97.56%   95.50%   -2.06%
==========================================
  Files          66       72       +6
  Lines        2668     2916     +248
==========================================
+ Hits         2603     2785     +182
- Misses         65      131      +66

Impacted Files
Coverage Δ

glasses/models/AutoConfig.py
93.33% <ø> (ø)

glasses/models/AutoModel.py
94.73% <ø> (ø)

glasses/models/classification/deit/__init__.py
0.00% <0.00%> (ø)

tmp.py
0.00% <0.00%> (ø)

glasses/nn/regularization.py
95.00% <95.00%> (ø)

glasses/models/classification/resnest/__init__.py
96.36% <96.36%> (ø)

glasses/models/classification/__init__.py
100.00% <100.00%> (ø)

glasses/models/classification/resnetxt/__init__.py
92.85% <100.00%> (ø)

glasses/models/classification/vit/__init__.py
96.42% <100.00%> (+0.55%)
:arrow_up:

glasses/nn/blocks/__init__.py
100.00% <100.00%> (ø)

... and 11 more

Continue to review full report at Codecov.

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


License?

Thank you for posting this project, which looks very useful!  I also appreciate the slides (https://www12.cs.fau.de/edu/map/talks/bgf_tesla.pdf).  What is the license for this code?  I cannot find a license file, or any license or copyright statements in the source files.  Thank you!
Sincerely,
Chris White

I just updated the license!
As this was my (and @gerry3309's) first ever CUDA project, the implementation is not the best it can be. Use at your own risk!

:) Much appreciated!


How to determine rule collection group size

The document states: "Rule collection groups have a maximum size of 2 Mb. If you need more than 2 Mb, you can split the rules into multiple rule collection groups."
Suggest to add the following information:

How can the size of a rule collection be determined?
Or are any values known to get an estimation, e.g. x rule collections or x lines of json when exported to arm-template

That would be very helpful to get a feeling, when thresholds will be reached and would allow for proper upfront planning.
Many thanks in advance!

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

ID: 08324cd3-7f61-81f6-a7e8-3f2111304d20
Version Independent ID: 348ef905-d8d2-3dc0-08d9-961b1ba5f7ed
Content: Azure Firewall policy rule sets
Content Source: articles/firewall/policy-rule-sets.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

Thank you for your feedback! We will review and update as appropriate.

Hello @nbachm,
Unfortunately, we currently don't have a quick solution for measuring RuleCollectionGroups size when stored in our system.
You can estimate this value by saving to disk the "ruleCollections" section content.
In order to get the most accurate estimation, make sure the following is true:

Each line of the JSON ends with a single \r\n.

JSON starts in indentation level 0

Each indentation level contains 2 space characters only.

Additionally to the above, we add a 1024 byte overhead to the above for required metadata. This should also be taken into account for the estimation.
Hope this helps! I am creating a feedback item internally to address this issue. We will now proceed to close this issue out. Please tag me in your response if you have any additional questions. Thank you!

Is there any update on this one? @ChaitanyaNaykodi-MSFT


Announcement:  'Reality Forward' highlight Cubemos Skeleton Tracking on RealSense D435i  (UPDATE: Cubemos and the new FRAMOS D415e)

Hi everyone,
The YouTube channel Reality Forward has highlighted use of the RealSense D435i camera with the new Cubemos Skeleton Tracking SDK and a Raspberry Pi 4.
https://www.youtube.com/watch?v=q-JhE5MLDB4
You can learn more about skeleton tracking at a recent article on the subject at the official RealSense blog.
https://www.intelrealsense.com/skeletal-tracking/
UPDATE:
After the vision technology company FRAMOS released the D435e industrial ethernet camera in 2019, they are following it up with an industrial variant of the D415 called the FRAMOS D415e.

At the Embedded World 2020 conference recently, FRAMOS demonstrated the D415e being used with the new Cubemos Skeleton Tracking SDK.
https://www.youtube.com/watch?v=Ao34n_9XrW4

For what it's worth, the Cubemos SDK cannot be installed on a Raspberry Pi (AMD vs ARM architecture), so the original news posting is misleading.


SAK-31948 Don't order anonymous assignments by sortname.

When displaying lists of submitters involving anonymous assignments we have to make sure we sort by the anonymous ID and not the user's sortname so we don't leak information about who has what result.
Ideally we would clean up all the comparators here, but that's lots of work and won't be easy to merge back to 11.x

LGTM


Flaky-test: ResourceGroupUsageAggregationTest.testProduceConsumeUsageOnRG

Search before asking

[X] I searched in the issues and found nothing similar.

Example failure
https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234
Exception stacktrace
Error:  Tests run: 6, Failures: 1, Errors: 0, Skipped: 5, Time elapsed: 8.132 s <<< FAILURE! - in org.apache.pulsar.broker.resourcegroup.ResourceGroupUsageAggregationTest
[630](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:631)
  Error:  testProduceConsumeUsageOnRG(org.apache.pulsar.broker.resourcegroup.ResourceGroupUsageAggregationTest)  Time elapsed: 0.185 s  <<< FAILURE!
[631](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:632)
  org.apache.pulsar.client.admin.PulsarAdminException: Resource group does not exist: runProduceConsume
[632](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:633)
  	at org.apache.pulsar.broker.resourcegroup.ResourceGroupService.checkResourceGroupExists(ResourceGroupService.java:417)
[633](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:634)
  	at org.apache.pulsar.broker.resourcegroup.ResourceGroupService.registerTenant(ResourceGroupService.java:196)
[634](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:635)
  	at org.apache.pulsar.broker.resourcegroup.ResourceGroupUsageAggregationTest.testProduceConsumeUsageOnRG(ResourceGroupUsageAggregationTest.java:140)
[635](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:636)
  	at org.apache.pulsar.broker.resourcegroup.ResourceGroupUsageAggregationTest.testProduceConsumeUsageOnRG(ResourceGroupUsageAggregationTest.java:85)
[636](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:637)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[637](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:638)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
[638](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:639)
  	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[639](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:640)
  	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
[640](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:641)
  	at org.testng.internal.invokers.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:139)
[641](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:642)
  	at org.testng.internal.invokers.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:47)
[642](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:643)
  	at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:76)
[643](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:644)
  	at org.testng.internal.invokers.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
[644](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:645)
  	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[645](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:646)
  	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[646](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:647)
  	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[647](https://github.com/apache/pulsar/actions/runs/3779970439/jobs/6425838234#step:10:648)
  	at java.base/java.lang.Thread.run(Thread.java:833)

Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

another one: https://github.com/apache/pulsar/actions/runs/4370505227/jobs/7651296557


[VL] Daily Update Velox Version (2024_03_31)

Upstream Velox's New Commits:
dc682d6a9 by xiaoxmeng, Spill test improvement and some cleanup (9283)

Closing this pr, as not required. cc @marin-ma


Run RTW prototype daily with local KGO

The recipe test workflow works with a small set of recipes on Jasmin (using container) Met Office (with 2.9 environment). To flush out any remaining problems, it should be run daily on Jasmin and Met Office using local (separate) KGO on each machine. Central management of KGO, using e.g. zenodo, will come later.
Would you be able to help out?
Yes, me and @ehogan will do this

Closed by #3556


Stitching with 3D tiles

The current DaskTileStitcher works only with 2D tiles. I have a use-case where the raw data is stored as 3D tiles and I must load the whole stack from the filesystem to access a single z plane. This leads to a lot of data loading overhead.
It would be nice if the current code could be modified to accept 3D tiles. The transformation/shifting should still only be in YX, but instead of moving only a single plane we could process a stack.

Done with #82.


Adaptive Card "Something Went Wrong" with Toggle.Visibility action

I am getting this issue when using an adaptive card with Toggle.Visibility action with Bot framework in MS Teams Channel. The toggle is working and data is being updated as expected but MS Teams keep giving "Something Went Wrong" whenever I use the toggle.
This is the payload of the adaptive card -
AdaptiveCardPayload.txt
I am using botbuilder version 4.18.0 and Adaptive card target version 1.4.

Hi @anirban-celebal - We have already raised a bug for this issue and engineering team has fixed it internally. We currently don't have an ETA when the fix will be available publicly. It might take couple of weeks to be GA.
Please follow this thread for further updates.
Thanks!

Engineering team has mentioned that as per latest updates the fix is expected to reach public rings by end of January.

THis was fixed very recently.

We see this issue happening again since 2023-02-13. We are using Power Automate to send adaptive cards on behalf of Power Virtual Agents bot in Teams. Teams client version is <IP_ADDRESS>1 (64-bit). Card target version 1.4.

@Toleikis - We tried again to repro this issue by sending card with Toggle.Visibility action via Power Automate. However, its working fine at our end both in MS Teams desktop (Microsoft Teams Version <IP_ADDRESS>78) and web client.

Card JSON:
{
        "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
        "type": "AdaptiveCard",
        "version": "1.4",
        "body": [
          {
            "type": "TextBlock",
            "text": "**Action.ToggleVisibility example**: click the button to show or hide a welcome message"
          },
          {
            "type": "TextBlock",
            "id": "helloWorld",
            "isVisible": false,
            "text": "**Hello World!**",
            "size": "ExtraLarge"
          }
        ],
        "actions": [
          {
            "type": "Action.ToggleVisibility",
            "title": "Click me!",
            "targetElements": [
              "helloWorld"
            ]
          }
        ]
      }

My test result is: Adaptive Card Toggle.Visibility action does not work anymore.   No "Something went wrong" message is shown, but just Toggle.Visibility does not work.

@michaelslz23 - Your issue is being tracked here. Please follow that thread for further updates.
Thanks!


ListLike-4.7.3

'ListLike >= 4.7.4' is a better constraint than 'ListLike < 4.7.2', and it
works too.

4.7.2 did not build against bytestring-<IP_ADDRESS>: it introduced instances which also where introduced in bytestring.
It seems 4.7.3 works as well, I'll relax the constraint.

4.7.2 did not build against bytestring-<IP_ADDRESS>: it introduced instances which also where introduced in bytestring.
It seems 4.7.3 works as well, I'll relax the constraint.

bors merge

bors merge


chore: external merge request from Contributor

Description
Issue: https://github.com/appsmithorg/appsmith/issues/10278
Original PR: #36865
EE PR: https://github.com/appsmithorg/appsmith-ee/pull/5373

[!WARNING]
If no issue exists, please create an issue first, and check with the maintainers if the issue is valid.

Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!CAUTION]
If you modify the content in this section, you are likely to disrupt the CI result for your PR.

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

[!IMPORTANT]
🟣 🟣 🟣 Your tests are running.
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/11456348609
Commit: dd9b0b4fc9a11cafae9164d6cb588ec8cb90db46
Workflow: PR Automation test suite
Tags: @tag.All
Spec: ``
Tue, 22 Oct 2024 08:33:40 UTC

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

/build-deploy-preview skip-tests=true

Closing as original PR is merged.


Language server randomly freezes when editing

This bug occurred very randomly, I haven't pinned it down yet but this usually happens when you try to edit command that has already been typed out to the end.
Re-doing the same action doesn't reproduce the bug as well so this is going to take some time to figure out.
Also, a side-effect of this is that I can no longer press [Spacebar] to add whitespace I have to press Shift + [Spacebar].

Please tell me if you're still encountering this in 3.0.0

The bug is unpredictable so I'll keep using it for a week first and then report back.

No problem so far, this should be fine now.

Nope, the bug is still there.

Closing as I do not have enough information to reproduce this issue and the language server is under complete rewrite.


use the AVMetadataObjectTypeDataMatrixCode only if the system support it...

use the AVMetadataObjectTypeDataMatrixCode only if the system support this API.
and then we can use the react-native-camera in iOS7.
otherwise it will compile error if the system version less than iOS8.

@yinuoba Thank you. I'm preparing a bug fix release that should be out in a day or two if I can find some time. :)


Add 'revisions' action to 'more'.

Replaces #2410

Merged in bedde59aaa0e0cf012cfaa15fb6563633bd557f9 - thanks!

:+1:


Add support for $in and $nin operators in query

This PR adds the $in and $nin operator support to Query.  Additionally, there was some code clean up and  refactoring done to the SQL Translator logic to aid in future maintenance, readability and some SQL optimization.

Okay, I think we're good here. The changes in 5b5040e need to be split up and either exist as separate commits, if useful, or squashed into the appropriate commits from earlier. E.g., simplifying code due to being able to make the assumption that things are certain types would probably be separate commits, as would QueryConstants stuff.

+1 once Travis is happy.


Mixed results with feeding content to Colorbox modal

When a modal is opening, we want to execute some custom logic that results in the actual content being displayed.  Our first instinct would be to do something like...
var $content = "Hi Bob."
$.colorbox($content)

From trying a few things it looks like this is not possible so we're trying something like...

			$(document).ready(function(){

				$(".modal").colorbox({
					inline: true,
					onComplete: function(ev) {
						$('#cboxContent').html("Hi Bob")
					},
					width: '100%',
					height: 500
				});

			});

But we're getting mixed results, (may have something to do with using different versions of colorbox). I figured I'd put this out there to see if other folks have seen similar issues, perhaps there is a better way.

Nvm, I'm now doing ...
$.colorbox({html: "Hi Bob."})


Readme code samples print the address instead of a value

As title says. For example this code:
import (
    "fmt"
    "gorm.io/gorm"
    "github.com/Thor-x86/nullable"
)

func main() {
    // Create new
    myBasicString := "Hello blяoW!"
    myNullableString := nullable.NewString(&myBasicString)

    // Get and print to command console
    fmt.Println(myNullableString.Get()) // Output: Hello blяoW!
}

The code will not output the string content but the address. Hence, the print needs dereference on the call to Get. Or am I doing it wrong?

Pardon me, I didn't notice there is an open issue.
Seems like there is a behavioral change after several Go compiler update. Let me fix this problem. Thanks!

Could you tell me which version of Golang you are currently using?
Travis-CI is currently down, so I tested it on Ubuntu machine with Go v1.15 and it's working as documented. However, when I test it on Arch Linux with Go v1.17 I got the exact same problem. Probably fmt.Println won't resolve pointer's value anymore.
I'm going to update the new docs and fixes after current test executed

I'm using 1.17


virtual box images does not provide the password for roscon user

virtual box images does not provide the password for roscon user

expired

Same question. No password found


Update Root.js

Changed to captial R as is the case with the directory "Routes"

Nice, thank you.
Maybe it's a better idea to rename the directory to lower-case to match the rest of code. What do you think?
The "Template" directory can also be renamed.

Either way it will achieve the purpose. I will leave the choice open to you.


refactor(AlgebraicGeometry): Make Scheme.Hom a structure.

We make LocallyRingedSpace.Hom (resp. Scheme.Hom) into a structure
instead of a subtype (resp. type-synonym of the former) to make clearer the
abstraction boundaries between them and presheafed spaces, which allow
us to remove some erws.
Plus, this enables us to write f.base and f.c instead of f.val.base and f.val.c.

!bench

!bench

Ping me when you merged master, I'll have a closer look at all the changes then.

!bench

Thanks for the script!
I think I fixed most occasions. Some toHoms are actually toPshHom instead of toShHom so I added the latter definition.

@erdOne can you add a PR summary explaining to someone with limited information the justification for this change.
bors delegate+

Thanks for the reviews!
bors merge

bors merge


Failed to build module 'Test' from its module interface

Hi there! Thank you for such a good job. I'm having a hard time trying to use this tool with the simplest example. I'm not sure if I misunderstood something or it's a real issue. Basically, I'm trying to build xcframework from the Test package which has some dependencies, for example Kingfisher. The command itself finishes successfully but manually imported xcframework shows:
Failed to build module 'Test' from its module interface; it may have been damaged or it may have triggered a bug in the Swift compiler when it was produced
swift-create-xcframework version: 1.3
Swift version: Apple Swift version 5.3 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.6.0
Environment: Xcode 12.1, Catalina 10.15.7
Checklist

[x] If possible, I've reproduced the issue using the master branch of this package
[x] I've searched for existing GitHub issues

Steps to Reproduce

Create package, with Kingfisher dependency

// swift-tools-version:5.3
// The swift-tools-version declares the minimum version of Swift required to build this package.

import PackageDescription

let package = Package(
    name: "Test",
    platforms: [.iOS(.v11)],
    products: [
        // Products define the executables and libraries a package produces, and make them visible to other packages.
        .library(
            name: "Test", type: .dynamic,
            targets: ["Test"]),
    ],
    dependencies: [
        .package(name: "Kingfisher", url: "https://github.com/onevcat/Kingfisher.git", from: "5.15.7"),
        // Dependencies declare other packages that this package depends on.
        // .package(url: /* package url */, from: "1.0.0"),
    ],
    targets: [
        // Targets are the basic building blocks of a package. A target can define a module or a test suite.
        // Targets can depend on other targets in this package, and on products in packages this package depends on.
        .target(
            name: "Test",
            dependencies: ["Kingfisher"]),
        .testTarget(
            name: "TestTests",
            dependencies: ["Test"]),
    ]
)

Import and use Kingfisher somewhere in the package.
Build it with swift create-xcframework command
Import a generated xcframework manually to any project (target -> general -> Frameworks, Libraries...)

Expected behavior
Project should compile successfully
Actual behavior
Failed to build module 'Test' from its module interface; it may have been damaged or it may have triggered a bug in the Swift compiler when it was produced in the importing Test file and No such module Kingfisher in the arm64-apple-ios.swiftinterface

Hi @SzymonWojcikCrustlab, apologies for the delayed reply, but are you producing Test.xcframework as well as Kingfisher.xcframework? You will need to ensure that any dependencies you need are also supplied either as XCFramework files or by some other means. You can build both from your Swift Package by naming them both, eg. swift create-xcframework Test Kingfisher


Update maven-surefire-plugin to allow excluding tests

Some tests take a while and it would be nice to be able to be able to exclude them for faster turnaround.  maven-surefire-plugin 2.19 or later supports a bang notation:
$ mvn -Dtest='!ManagedCloudSdkTest' verify

The two I've found useful:

ManagedCloudSdkTest: if no Cloud SDK is available (e.g., running tests from a scratch docker instance), this test downloads and installs the Cloud SDK
LibrariesTest: does a whole series of network calls

$ mvn -Dtest='!ManagedCloudSdkTest' verify

Out of curiosity, no need to give fully qualified names to -Dtest?

From the doc:

Each pattern you specify here will be used to create an include pattern formatted like **/${test}.java, so you can just type -Dtest=MyTest to run a single test called "foo/MyTest.java".

2.21.0 supports ANSI colour. Whee!

These errors are funnily enough caused by ManagedCloudSdkTest:
testManagedCloudSdk_latest(com.google.cloud.tools.managedcloudsdk.ManagedCloudSdkTest)
Welcome to the Google Cloud SDK!
Your current Cloud SDK version is: 205.0.0
Installing components from version: 204.0.0
+-------------------------------------------------+
|        These components will be updated.        |
[…]
ERROR: (gcloud.components.update) The component [bq] failed to download.
<urlopen error [Errno 185090050] _ssl.c:344: error:0B084002:x509 certificate routines:X509_load_cert_crl_file:system lib>
This will install all the core command line tools necessary for working with
the Google Cloud Platform.
[ERROR] Tests run: 8, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 28.489 s <<< FAILURE! - in com.google.cloud.tools.managedcloudsdk.ManagedCloudSdkTest
[ERROR] testManagedCloudSdk_latest(com.google.cloud.tools.managedcloudsdk.ManagedCloudSdkTest)  Time elapsed: 3.389 s  <<< ERROR!
com.google.cloud.tools.managedcloudsdk.command.CommandExitException: Process failed with exit code: 1
	at com.google.cloud.tools.managedcloudsdk.ManagedCloudSdkTest.testManagedCloudSdk_latest(ManagedCloudSdkTest.java:117)

The errors won't go away until #667 is merged.


Debug message to figure error

Description
Changes proposed in this pull request.
Test plan
Outline the test plan used to test this change before merging it.

Fixes #.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "trireme-lib",
    "pr-id": "766",
    "commit-sha": "faadcb3324727d1e9f7934b750e209e07d28950e"
  },
  {
    "project": "",
    "component": "enforcerd",
    "pr-id": "1105",
    "commit-sha": "671bc46c173057073c743152c25bb8ab74827add"
  }
]


TE-3.3: Get NI and Intf data from telemetry

This PR changes the test to log NI and Interface state data rather than conf as it the case for all other tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1320

Relevant Lines:
2753

💛  - Coveralls

Pull Request Functional Test Report for #1654 / bbb9f99b91515941325f04177ab24a91de1afb48
No tests identified for validation.
Help

@
kjahed could you please resolve the conflicts

Looks like this has already been taken care - https://github.com/openconfig/featureprofiles/blob/main/feature/gribi/otg_tests/hierarchical_weight_resolution_test/hierarchical_weight_resolution_test.go#L296
Can we close this PR?


[system] container queries support in breakpoints api and sx props

Added container queries support with newly introduced shorthands in sx props
Added container queries support in the breakpoints api
Documented everything to the best of my knowledge in the docs

Feel free to change the sx props shorthand that I chose for container queries. I had some discussions with my teammates and we came to the conclusion that a cq prefix for container queries would make most sense. I'm not too opinionated on this though. Happy to discuss this further. :)
In short: This is my suggestion to implement the desired feature #36670. If you decide to go a different path, that's of course up to you, but I'd really appreciate a handy sx support for container queries.

[x] I have followed (at least) the PR section of the contributing guide.

@oliviertassinari any chance a maintainer will have a look at this? :) I really need this feature 🙃

Interesting PR! appreciate your effort. However, I might have to discuss the API before merging this PR.

I really need this feature

I will try to create a workaround so that it unblocks you.

Glad someone is taking a look :)
As mentioned I'm completely fine if you decide to name things differently or even find a nicer approach to solve the same problem. As long as there's a way to do this, I'm happy.

@siriwatknp Is there an update on what this PR is trying to solve? Do we have a native was in MUI for container queries? The API is supported by pretty much every browser now. I too really need this feature and would appreciate a resolution.
https://caniuse.com/css-container-queries

@saksham-sharma What do you think of this 👉 https://github.com/mui/material-ui/issues/36670#issuecomment-1588886160

Based on https://github.com/mui/material-ui/issues/36670#issuecomment-1588886160, I will take over this.


Camera exception during morph

In b29 and the camera-terrain branch.

Run Viewer.
Switch to 2D.
Use the Geocoder to zoom to "Exton."
Switch to Columbus Viewer.
Part way through morph:

Not 1.0.

Another way to reproduce this from #2825:

Run http://localhost:8080/Apps/Sandcastle/index.html?src=Camera.html&label=Showcases
Switch to 2D
Fly to rectangle
Then switch to CV.

I'm pretty sure this is because the morph started before the flight finished.

Cannot reproduce.  Also, this looks like a duplicate of #1596


[docs] Fix wording in backdrop.md

Update inconsistency between use of the word Overlay and Backdrop

[ ] I have followed (at least) the PR section of the contributing guide.

@matt-savvy Thanks


Switch deriving "subtype" for Datasets from data_types to dataset_type

Switch from deriving the "subtype" for Datasets using the data_types field to the new dataset_type field.
See also search-api#653 to use organ codes from UBKG

Also link to the related issue: https://github.com/hubmapconsortium/search-api/issues/653

Already addressed via #737

Dupe of #737, closing


home dir is hardcoded in class ONNXMiniLM_L6_V2(EmbeddingFunction[Documents]): [Bug]:

What happened?
i'm trying to add documents to a collection, using an embedding model.
as far as i understand, the collection first creates an instance of this model and tries to save it in a cache directory.
instead of having a way to control the cache directory- it simply uses pathlib Path.home()
in my case, the home directory is full and that causes the code to crash.
the only way i can run the code is by modifying the home directory before running the script.
bug location:
site-packages\chromadb\utils\embedding_functions.py
class ONNXMiniLM_L6_V2(EmbeddingFunction[Documents]):
MODEL_NAME = "all-MiniLM-L6-v2"
DOWNLOAD_PATH = Path.home() / ".cache" / "chroma" / "onnx_models" / MODEL_NAME
EXTRACTED_FOLDER_NAME = "onnx"
ARCHIVE_FILENAME = "onnx.tar.gz"
MODEL_DOWNLOAD_URL = (
"https://chroma-onnx-models.s3.amazonaws.com/all-MiniLM-L6-v2/onnx.tar.gz"
)
_MODEL_SHA256 = "913d7300ceae3b2dbc2c50d1de4baacab4be7b9380491c27fab7418616a16ec3"
Versions
chromadb==0.4.24
Relevant log output
No response

We can prioritize this for our next release. @tazarov would you mind TAL at the PR in question?

Any progress on this?


idea: should lettre convert String Body linebreaks to be DOS-style?

While looking around for uses of lettre I realized that some projects convert strings to have CRLF line breaks (for example: https://github.com/dani-garcia/bitwarden_rs/blob/16eb0a56f9767b33256ab6c9d66b5dd49af22e6e/src/mail.rs#L102).
It this something that lettre should implement instead?
https://tools.ietf.org/html/rfc5322#section-2.3
I don't think we'd want to release this in a patch version, so this should probably either go in 0.10.0 or in the next minor release.
Also tagging @BlackDex since he's the one who implemented it for the above mentioned project.

Hello,
Yea i added that because i wanted msglint to pass.
And the RFC indicates that all line endings need to be \r\n.
I never thought to bring this upstream since our templates are saved with unix line endings and thus kept it locally.


Add entry for FastClick breakage

To summarize what we know and what we tried.

https://bugzilla.mozilla.org/show_bug.cgi?id=1448747


Speed Up augmentModels

Add in lodash’s quicker _.each function.

This jsPerf indicates that _.each is actually slightly slower than a native for..in loop. I'm getting an even larger difference when running that same benchmark in Node:
_.each x 100,933 ops/sec ±3.47% (80 runs sampled)
for in x 142,130 ops/sec ±4.07% (83 runs sampled)

I had might doubts about performance as well, but there's a better reason to implement this, which is that _.each is just safer than for...in.  This is code that runs infrequently anyway.  I'll merge pending testing.


Update layer's loaded state before vispy and thumbnail

Description
This moves the update to Layer.loaded to be before rather than after the updates to the vispy layer and qt thumbnail. I made that change for a few reasons.

One of the async slicing tests flaked because it was waiting on the vispy update and assumed that loaded would also be updated.
Consistent with the previous definition of Layer.loaded with regards to the old async.
The vispy layers still read the layer's slice state (after Layer.events.set_data) has been emitted, and in at least one case that includes Layer.loaded.

References
Related to #4795

I get the practical reasons but... this just feels wrong? Isn't the whole point of Layer.loaded to signal that the canvas is in fact showing what you asked for? Should we instead consider having another event (say: ready) that does what you're asking (mark a layer as "just about to be rendered")?

I don't have strong opinions about Layer.loaded, so I'm open to ideas here. I'd also be appreciative of any code examples using it to help clarify what it's actually for, especially most other slice/render state is not public.
But Layer.loaded has been around for a while (i.e. before the new async slicing), and it was not indicating state described above, instead the immediate slice state in the layer as in the docstring: https://github.com/napari/napari/blob/acddda2000ef6498731c4687cfb5ac60e378a68a/napari/layers/base/base.py#L535
It's also strange that Layer would have state related to the corresponding vispy layer because the ownership model is the other way round - i.e. the vispy layer owns the layer.
My suspicion is that we really want something like Viewer.wait_until_layers_rendered, which would effectively wait for all the layers to be loaded and then wait for any other Qt events to be processed.

My suspicion is that we really want something like Viewer.wait_until_layers_rendered(), which would effectively wait for all the layers to be loaded and then wait for any other Qt events to be processed.

Yup, 100% agree on this! Not sure what that means for this PR. I guess you're saying that either way (pre/post PR) my expectations about layer.loaded are wrong (i.e: no guarantee of the vispy canvas to be up to date)?

I guess you're saying that either way (pre/post PR) my expectations about layer.loaded are wrong (i.e: no guarantee of the vispy canvas to be up to date)?

Yes, mostly because that's how it was defined before by the docstring and the implementation.
We might be able to change its definition to ensure that it represents the rendered canvas state, but that depends on exactly how the vispy Qt backend is implemented. My assumption is that updating the vispy visual causes an event to be posted to the Qt main event loop, so you'd need to the extra call to processEvents (and maybe sendPostedEvents) to ensure that all pending events (like draws) are actually executed. Would be great to clarify if vispy makes any guarantees about that stuff like that.
I'm also not certain if this PR will fix the test flake, as there's another possible cause for that, which is that changing current_step causes the corresponding event to be fired twice. Will file a separate bug for that.

I'm also not certain if this PR will fix the test flake, as there's another possible cause for that, which is that changing current_step causes the corresponding event to be fired twice. Will file a separate bug for that.

Filed this bug: https://github.com/napari/napari/issues/6032


Determine schedule for episodes

Preliminary thinking:

Nov 18
Nov 25
Dec 2
Dec 9
Dec 16
[Holiday break]
Jan 6
Jan 13
Jan 20
Jan 27
Feb 3

Schedule is set — I've added a rough outline for the first 5 episodes, and I have plans for all 10, but since there's a two week break in the middle, my plan is to potentially adjust the schedule/topics for the last five episodes in case some of the first ones require a little more attention (or in case of any technical difficulties... when you're live, it can always be interesting!).


📝 Update includes in docs/de/docs/advanced/custom-response.md

📝 Update includes in docs/de/docs/advanced/custom-response.md

Thanks for the effort! :cake:
This was handled in another PR, so I'll close this one now. :coffee:


set black with 120 line length

Why are these changes needed?
as Flake8 is set already to 127 line length Black has default 88 so it is natural to have both aligned, see flake8 setting: https://github.com/microsoft/FLAML/blob/f404b66a4c20d421c9cee465a6796ca76a0a8992/.flake8#L3
Related issue number

Checks

[x] I've used pre-commit to lint the changes in this PR, or I've made sure lint with flake8 output is two 0s.
[ ] I've included any doc changes needed for https://microsoft.github.io/FLAML/. See https://microsoft.github.io/FLAML/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Not sure, but the two failing test does not seem to be related to the formating update...

@Borda Thanks for your PR! Can you figure out how to address the conflicts?

@Borda Thanks for your PR! Can you figure out how to address the conflicts?

Sure, it shall be trivial as the main contribution of this PR is update config, the rest is just bot work, so I will accept all changes from main branch and re-run the bot 🐰

@skzhang1 shall be fixed now :chipmunk:


Login with existing account–After login via link user redirect on "Hmm... it is not here" page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.3.89-6
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Open https://staging.new.expensify.com/.
Log out
Navigate to this link - https://staging.new.expensify.com/concierge
Log in with<EMAIL_ADDRESS>
Expected Result:
Navigated to the concierge chat after login
Actual Result:
"Hmm... it is not here" page opens when navigate via deeplink https://staging.new.expensify.com/concierge
and log in with<EMAIL_ADDRESS>account
"Hmm... it is not here"
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] Windows: Chrome
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

Windows: Chrome

https://github.com/Expensify/App/assets/78819774/c6f5d9bb-b902-4e75-812d-c6283412c252
logs2.txt

MacOS: Desktop

View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

I could not reproduce this problem with my account:

@zanyrenney - do you want to see if this happens for you?

I can't repro either. Let's demote.

I don't know why this was assigned to me at 00.45 😓 @tylerkaraszewski

I have tried now and I also can't reproduce, should we make this daily instead?

I'm gonna close it since nobody can make it happen.


Configure logger to trace

Just suggesting some edits to the readme to help the next person who comes along.
fixes #316

:+1:


Add point mass gravity fields in spherical coordinates

Description of the desired feature:

For larger scale applications, it would be great to have versions of the point mass code in spherical coordinates. The input coordinates would be in longitude, spherical latitude, and radius and the output vector components would be in a local Cartesian system (east, north, up).
The equations for this can be found in this paper: https://doi.org/10.1007/s00190-008-0219-8
The format for the functions would be pretty much the same as those in choclo/point/_forward.py. The new functions implemented in 2 places (not sure which would be best):

In a module called choclo/point_spherical/_forward.py and they would have the same name as their cartesian counterparts.
In choclo/point/_forward_spherical.py and they would be called gravity_spherical_e, and so on.

I'm leaning towards option 2.
Are you willing to help implement and maintain this feature?

Maybe but it will be a while before I have to do it, so please be my guest!

Sounds a great idea @leouieda!
I do have one design question. The spherical forward modelling functions make use of the cosine and sine of the latitudes of the observation point and the source. When having multiple sources and observation points we would need to compute those cosine and sine functions for each one of them. We've seen (see Harmonica forward models for points in spherical coordinates) that those cosine and sine values can be precomputed for each observation point and source, saving significant computational time in comparison to computing them one time for each combination of observation point and source.
So, should these new functions take the longitude, latitude_spherical and radius as arguments, or it's better if they ask the longitude, cos_latitude_spherical, sin_latitude_spherical and radius?


feat: START add prooperty disableAutoClose

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bug in split view

in the split view I don't see the close button! Also, when there are too many tabs, I can't slide to close the last one. It became really buggy since the new update!

please wait...:confused:

Please Update and Report...

Thank you very much DH3ALEJANDRO! The new update fixed all the issues! Thanks a million matey! :+1:


Name conflict between models.Response and types.Response

I ran into an OpenAPI spec which included a Response type. That caused a name conflict in the generated python code:
from ...models.response import Response
from ...types import Response

This was fixed easily enough using a class_overrides config, but since the classes in types.py are pretty generic I think it would be better if the classes were just fully qualified to avoid this kind of conflict.

@drwilly  could you add the class_overrides stanza you used here?


BL crashes when I wear a custom armor.

Mod: http://www.mediafire.com/?833ja0cj1809j55
Zenfone 5 Android 6.0.1

@Rodrigo294X I can't reproduce this on my Nexus 6P; this issue might be specific to Intel devices.

@zhuowei The new function Item.defineThrowable also cause crashes . I think that may have some relation .

Maybe this will help you:
Custom Armor: http://i.imgur.com/W3AiyQ5.png
Item Throwable: http://i.imgur.com/EoJnosd.png


Adapter configuration

Change adapter to read vito.xml from vcontrold configuration (/etc/vcontrold/vito.xml) as the adapter presumes a running vcontrold (really not necessary to have the vito.xml twice).
In addition, add function to reload configuration while instance is already present (it's really awful if you have to change something within the vito.xml and have to delete the instance first and reconfigure everything afterwords!)

First part is done.
Second, it is not so easy but i try it the next weeks.


[RTC-32] Fetch actual bitrate of a variant

TODO:

fix integrations tests (js sdk from this pr required)
update media event docs

I think this PR is ready for review except for the thing I mentioned above, also I was wondering whether I should add some more test (webrtc_endpoint.ex specifically, even tho there's no test for this file at the moment).


Fix for expiration of records in DynamoDB not found in 6.2.5

Hello,
according to the current release notes, the problem with the incorrect entries for expiring entries should be fixed in the current release. But I couldn't find the fix from the pull request #47 is that correct?

That's correct - this was missed. Thanks for catching that. It will be included in the next release.

This should be fixed in v6.2.6


Node.js Foundation Community Committee (CommComm) Meeting 2017-08-03

Node.js Foundation Community Committee (CommComm) Meeting 2017-08-03
Time
UTC Thu 03-Aug-2017 20:30 (08:30 PM):

Timezone
Date/Time

US / Pacific
Thu 03-Aug-2017 13:30 (01:30 PM)

US / Mountain
Thu 03-Aug-2017 14:30 (02:30 PM)

US / Central
Thu 03-Aug-2017 15:30 (03:30 PM)

US / Eastern
Thu 03-Aug-2017 16:30 (04:30 PM)

Amsterdam
Thu 03-Aug-2017 22:30 (10:30 PM)

Moscow
Thu 03-Aug-2017 23:30 (11:30 PM)

Chennai
Fri 04-Aug-2017 02:00 (02:00 AM)

Hangzhou
Fri 04-Aug-2017 04:30 (04:30 AM)

Tokyo
Fri 04-Aug-2017 05:30 (05:30 AM)

Sydney
Fri 04-Aug-2017 06:30 (06:30 AM)

Or in your local time:

http://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Community Committee (CommComm)+Meeting+2017-08-03&iso=20170803T20
or http://www.wolframalpha.com/input/?i=8pm+UTC%2C+Aug+03%2C+2017+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1Vvl8VMehZXlgXt3iFf_xQYcXWkaTOfBgVwT9FWTKk1U
Previous Minutes Google Doc: https://docs.google.com/document/d/1utEp06Pyk0uwJn77TWdxIpwuoiFRjHvKinSdbY1BwNk

Agenda
Note: we also have something we need to discuss during the private session.
Extracted from cc-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
nodejs/community-committee

Invite Interested Website WG members to Next CommComm meeting #95
Create an email alias for CommComm? #89
Move Node Together Repo as an org under CommComm #63

Invited

Ashley Williams @ashleygwilliams (CommComm)
Bryan Hughes @nebrius (CommComm)
Emily Rose @emilyrose (CommComm)
Gregor Martynus @gr2m (CommComm)
Jenn Turner @renrutnnej (CommComm)
Joe Sepi @joesepi (observer)
JP Wesselink @jpwesselink (CommComm)
Michael Dawson @mhdawson (CommComm)
Michelle Garrett @msmichellegar (CommComm)
Myles Borins @MylesBorins (CommComm)
Olivia Hugger @pup (CommComm)
Rachel White @rachelnicole (CommComm)
Richard Littauer @RichardLitt (CommComm)
Rod Vagg @rvagg (CommComm)
Tierney Cyren @bnb (CommComm)
Tracy Hinds @hackygolucky (CommComm)
William Kapke @williamkapke (CommComm)
Mike MacCana @mikemaccana (observer)
Sota Yamashtia @sotayamashita (observer)
Frederic Hemberger @fhemberger (observer)
Oz Haven @therebelrobot (observer)

Notes
The agenda comes from issues labelled with cc-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting
Google Hangouts; link will be posted here shortly before the meeting starts.
A standing invitation exists for all @nodejs/TSC members who are not also CommComm members to attend in observer capacity.
Public participation
We stream our conference call straight to YouTube so anyone can listen to it live, it should start playing at https://www.youtube.com/c/nodejs+foundation/live when we turn it on. There's usually a short cat-herding time at the start of the meeting and then occasionally we have some quick private business to attend to before we can start recording & streaming. So be patient and it should show up.

@bnb Will you be able to take notes for this meeting? I just discovered I have been double booked :/

I'm probably going to miss most of this meeting if not all.  I'll be picking up my Mom from the hospital after a visit with my Dad around that time.  I'm also on vacation the next week so I'll see you all at the meeting the following week.

@renrutnnej Yes, I can take notes for the meeting 👍

how are we going to accomodate this many observers? hangouts has a maximum capacity

Can we suggest that observers watch over YouTube?

Can we suggest that observers watch over YouTube?

🤔  I suspect we'll only have roughly half of the CommComm members join the call, but that still gets us close to the limit.
Do we need the observers present to discuss the website work described in https://github.com/nodejs/community-committee/issues/95?

I can skip this meeting if it is helpful. 🤔

I don't think that would be helpful, @joesepi. :) Let's all just jump on the Hangout; if it maxes out, we can request that observers or non-participants jump onto the YouTube stream, and we can monitor the comments closer than normal.

moving to meet.google.com would increase max participants fwiw... but I don't think it supports on air

My Zoom account will accommodate 100 folks, but I dont know if that is a solution we want to use.

moving to meet.google.com would increase max participants fwiw... but I don't think it supports on air

That's unfortunate about not supporting on air. Do you know/are able to comment on if meet will support on air?

My Zoom account will accommodate 100 folks, but I dont know if that is a solution we want to use.

Let's consider it for the next meeting. I'm a little wary of switching to it for this meeting though since we don't have much time before it starts.

I like @RichardLitt's suggestion. Let's all hop on the hangout, and we can figure out if observers need to switch to the live stream based on how many folks show up.

I'd like to point out that youtube streaming as per https://github.com/nodejs/TSC/tree/master/Streaming requires upfront setup but nothing in particular per meeting other than someone running it.
Also, I'm not entirely certain if #95 invited me as a Website-y person, but if it did I can certainly do the streaming honors.
(Or even if I'm not really attending the meeting, I can easily stream from a second PC.)

@Fishrock123 yeah, you're more than welcome to join! Maybe it would be a good idea to switch to uberconference and stream that way, since you're already set up to do that. Also, thank you!
@nodejs/community-committee and observers, if you do not already have the uberconference link, please email me at bryan at nebri dot us or DM me on Twitter at nebrius (my DMs are open) and I'll send it to you privately.

That also goes for @nodejs/website members who are interested in attending!

Livestream will be at http://www.youtube.com/c/nodejs-foundation/live

minutes are merged!


replace foo/bar example with username/repository

If username/repository doesn't work, we could go with a simplified version:

user/repo
username/repo

what do you think? Thanks!

Great catch, @pjcdawkins, thank you  - just updated it to owner/repository which kinda covers the 3 cases better, wdyt?


[SPARK-47447][SQL] Allow reading Parquet TimestampLTZ as TimestampNTZ

What changes were proposed in this pull request?

Currently, Parquet TimestampNTZ type columns can be read as TimestampLTZ, while reading TimestampLTZ as TimestampNTZ will cause errors. This makes it impossible to read parquet files containing both TimestampLTZ and TimestampNTZ as TimestampNTZ.
To make the data type system on Parquet simpler, this PR allows reading TimestampLTZ as TimestampNTZ in the Parquet data source.
Why are the changes needed?

Make it possible  to read parquet files containing both TimestampLTZ and TimestampNTZ as TimestampNTZ
Make the data type system on Parquet simpler

Does this PR introduce any user-facing change?

Yes, Parquet TimestampLTZ type column are now allowed to be read as TimestampNTZ
How was this patch tested?

UT
Was this patch authored or co-authored using generative AI tooling?

No

cc @sadikovi @srielau too

Could you explain why we make a difference in the way Parquet and JDBC handle LTZ? As for JDBC LTZ, this kind of conversion is forbidden if we know we are reading an LTZ value

due to schema evolution, it's very likely to have mixed data types in parquet files. e.g. turn on NTZ, write some files, turn it off (maybe hit some issues), write some files, and turn it on again (issue fixed), read some files.

@yaooqinn For parquet files, it's possible that a file directory contains two different kinds of timestamp columns. Before this change, only timestamp type can be used to read such a file directory. While for JDBC, a table type is certain.
But you make a good point here. I am also considering allowing reading INT96 as TimestampNTZ, since there is no clear definition of this Parquet physical type. We can still fobid reading Int64 timestamps with isAdjustedToUTC=true as TimestampNTZ.

"Understood. Thank you for explaining that, @gengliangwang."

@yaooqinn @cloud-fan @beliefer Thanks for the review. I am merging this one to master.


The minimum node version should be listed somewhere in the readme.md

The minimum node version should be listed somewhere in the readme.md
Which node version is required?

https://github.com/hypercore-protocol/hyperdrive-daemon#installation now mentions 12. Can this issue be closed?

Yep, thanks @millette


Potential clarification of repo setup instructions

The current setup instructions state:

The tests rely on a checked out version of the Kotlin source repository since it uses the test data there to build a corpus to test against. The path to the base of the repo needs to be set as the KOTLIN_REPO environment variable.

However, just checking out the Kotlin repo (on master) and pointing this repo to that isn't sufficient by itself. All the Kastree tests failed until I went and set up the Kotlin repo properly. I think it would be helpful to clarify that after checking out the Kotlin repo, the user should follow their build setup guidelines. I'm not entirely sure it's necessary but I ran gradle in the Kotlin project root directory to perform the default build setup tasks (and also gradle compilerTests but I quit that 30 minutes in). Perhaps something like:

The tests rely on a checked out version of the Kotlin source repository since it uses the test data there to build a corpus to test against. The path to the base of the repo needs to be set as the KOTLIN_REPO environment variable, and you should ensure that the Kotlin source repository is properly set up itself by following the build setup instructions in its README and invoking $ gradle in its root directory. Once this is done, ...

Hrmm, I will have to check. I didn't require that in the past, but the repo may have changed in the meantime.

@TAGC - After merging in #6, the tests seem to work just fine. Simple having the latest Kotlin source repo checked out from master, having KOTLIN_REPO set to that path, and running the tests seems to work just fine. No steps on the Kotlin repo are required. Can you verify?

I deleted the Kotlin source repo, re-cloned it and tried running gradle test again in the kastree repo (with the latest commit from master checked out), and yeah it seems to work fine.


VS Code is pointing an error in a no longer existing file.

VSCode Version: 1.15.1
OS Version: Windows 8.1 Enterprise

I had changed the file name, or its folder's name before fixing bug.
I had fixed the bug.
The bug is continuing to exists, but in the no longer existing file, clicking on the error, and the popup panel shows that: ,,Unable to open 'Authorization.security.php'; File not found (...)."

So I think this is a VS Code bug.

Are you using a PHP extension?


Quick Tip for USFL Draftkings Showdown

Sorry, figured the only way to reach most is through an issue.
I used the Draftkings Showdown for NFL on USFL data and it worked without issues.  It's a new sport so I think using pydfs will give us a bit of an edge with good projections.  It's worth the try.
Eric

Should be easy to add in
On Mon, Apr 18, 2022, 4:23 PM Eric Colon @.***> wrote:

Sorry, figured the only way to reach most is through an issue.
I used the Draftkings Showdown for NFL on USFL data and it worked without
issues. It's a new sport so I think using pydfs will give us a bit of an
edge with good projections. It's worth the try.
Eric
—
Reply to this email directly, view it on GitHub
https://github.com/DimaKudosh/pydfs-lineup-optimizer/issues/360, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANBWO7TJDH5ZW2YLMB4S5C3VFXADDANCNFSM5TWYSKJA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Adjust Lerna to version components independently

Currently Lerna is bumping versions of all the components, even if some components have had no code changes.
This is undesirable because of 1) the way semver works and 2) because these are design components.
Minor version bumps in PFE are going to include design changes, and as hard as we try to make designs work with everything, invariably there are going to be edge cases we're not thinking about and solving for. So downstream projects should opt into minor versions and test things before pushing to prod, to avoid inadvertent updates that might change the look of something unintentionally.
The general recommendation then, is that they should probably lock the major & minor versions and opt into patch updates only, which fix bugs but don't drastically change any designs.
        "@patternfly/pfe-card": "1.1.x",
        "@patternfly/pfe-cta": "1.1.x",

To continue this hypothetical scenario, let's say in the PFE project,  the pfe-card gets several updates. So its legitimately at 1.4, whereas the pfe-cta is not really changing, but it keeps getting minor version bumps too, so it's now at 1.4.  Since the downstream project is only interested in the card changes, and they don't have time to test the CTA, they update their dependencies to:
        "@patternfly/pfe-card": "1.4.x",
        "@patternfly/pfe-cta": "1.1.x",

Then the day comes where a patch release is created for the pfe-cta. The downstream project wants that fix. But instead of it being a 1.1.1 patch, it is now a 1.4.1 patch, which they will never receive unless they manually update the package.json again, which differs from their expectations.

Implementation suggestions:

Use the "Independent mode" setting in Lerna

In this mode, each version of packages is separated and allow developer to choose version to release when publishing. To enable this mode, you are supposed to assign version property as independent in lerna.json file or add--independent argument when running lerna init command.
https://medium.com/shopback-engineering/lerna-tutorial-series-brief-f77f40c5777f

I just noticed that the package-lock.json files seem to be correct in the sense that the repo has had up to prerelease-17, and yet the pfe-cta component lock file is still at 13:

Fixed in #532


AttributeError: 'NoneType' object has no attribute 'create_execution_context'

Hello I wanted to test best.engine model on AGX Orin but I've got this error:
[02/27/2024-12:24:03] [TRT] [I] Loaded engine size: 2 MiB
[02/27/2024-12:24:03] [TRT] [E] 1: [runtime.cpp::parsePlan::314] Error Code 1: Serialization (Serialization assertion plan->header.magicTag == rt::kPLAN_MAGIC_TAG failed.)
Traceback (most recent call last):
File "/home/bjku/TwinLiteNet/test_trt.py", line 68, in
trt=TRT('best.engine')
File "/home/bjku/TwinLiteNet/test_trt.py", line 45, in init
self.context = model.create_execution_context()
AttributeError: 'NoneType' object has no attribute 'create_execution_context'
Sombody could help me ?

Can you verify whether you using different versions of TRT for constructing the engine compared to deserializing it?  are you using  different devices for building and deserializing the engine?


chore: fixed the linters configuration

Typescript files were not linted by the pre-commit hook.
Two major changes:

VS Code is going to use the top-level .prettierrc.json so it may affect the web and subgraph packages.
The precommit hook does not run eslint anymore for .js files, only prettier.

What do you think? @alcercu @AndreiMVP

hey @jaybuidl!
Just checked and vim is getting the prettier config from the web directory, so no problem on that side. I don't know about VSCode though.
Regarding the pre-commit hook for typescript files, if it's also intended for the web package, we should have the file extension tsx in the rules too.


Fix symbol conversion into qiskit

Description
Fix the conversion of symbols from pytket circuits to qiskit so that round-trip equality checks pass.
This needs qiskit >= 1.2.0 to work, so bump the minimum qiskit version.
Related issues
Fixes #376 .
Checklist

[x] I have performed a self-review of my code.
[ ] I have commented hard-to-understand parts of my code.
[ ] I have made corresponding changes to the public API documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have updated the changelog with any user-facing changes.

We should bump up the pytket version, too. Do you want to do that in this PR?

I was thinking to do that in a separate PR.


X-Plane 12 cannot find resource and stuck at "Reading new scenery files"

Describe the bug
X-Plane 12 hangs on loading screen "Reading new scenery files" and auto-ortho has many errors and other log entries.
To Reproduce
Steps to reproduce the behavior:

Open auto-ortho
Click Run
Open XP-12
Start a flight

Expected behavior
Sim should load the scenery files and open
(please complete the following information):

OS: Windows 11 64-bit
XPlane Version: XP12 latest version
AutoOrtho Version 0.4.4
Filesystem type NTFS

Additional context
The logs can be found below.  FYI the Error, could not locate image file for terrain error at the bottom of the X-Plane log file is because I closed Auto-ortho exe.
@kubilus1
autoortho.log
Log.txt

cptalpdeniz:
I would think that you are from the group of unfortunate ones that version 0.5.0-2 does not work for us (like me...!) that we have Windows 11 with an Intel processor.
Install -with confidence- version 0.4.4 which works great with XP12, Windows 11 and Intel processor.
Regards
carlos

Install -with confidence- version 0.4.4 which works great with XP12, Windows 11 and Intel processor.

Hello Carlos, as you can see from my initial post that I tried the latest version, 0.5.0-2, 0.5.0-1 and 0.4.4.
UPDATE: The issue was fixed when I enabled VPN. Any idea why? @kubilus1

You could be blocked or throttled by ip address or if outside the US, geo
location.
On Wed, Jul 12, 2023, 4:21 PM Alp Deniz Şenyurt @.***>
wrote:

Install -with confidence- version 0.4.4 which works great with XP12,
Windows 11 and Intel processor.
Hello Carlos, as you can see from my initial post that I tried the latest
version, 0.5.0-2, 0.5.0-1 and 0.4.4.
UPDATE: The issue was fixed when I enabled VPN. Any idea why? @kubilus1
https://github.com/kubilus1
—
Reply to this email directly, view it on GitHub
https://github.com/kubilus1/autoortho/issues/262#issuecomment-1633154539,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVEDLGLRDC2UNMR4MGOZADXP4BL3ANCNFSM6AAAAAA2G7SQOY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Blocked by ISP or the map provider service? Can you elaborate more?

Basically the system relays requests to map providers such as servers for
Bing maps.  Bing may decide to limit the requests, or block requests
completely for whatever reason they choose.  I know some folks have had
luck using a VPN to get around that, but I can't speak personally on how
well that works.
On Wed, Jul 12, 2023 at 7:24 PM Alp Deniz Şenyurt @.***>
wrote:

Blocked by ISP or the map provider service? Can you elaborate more?
—
Reply to this email directly, view it on GitHub
https://github.com/kubilus1/autoortho/issues/262#issuecomment-1633328292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVEDLB25DSMIODQ6EKMK7LXP4WZRANCNFSM6AAAAAA2G7SQOY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have the same trouble with version 0.6.1. Windows 11, Intel CPU. Any solution?

UPD: Tried to switch to other map provider (Firefly), but still stuck


[blocked] Add a dedicated cst::Node::Invalid for incomplete/invalid CST nodes

Closes #835
Closes #507
Closes #700 (although we still should explore not having to recursively match validity of a node when we have perf)
This also renames TokenKind::SKIPPED to TokenKind::INVALID; it's only possible to emit by the lexer in case there's an error matching any token but will not end up in the tree as the parser will skip over it and embed it in the newly introduced variants here.

Converting to draft until we reach a design consensus in #835. If anyone wants to tackle this at any point, feel free to take it over!

Closing since the consensus in #835 is to not introduce a third CST node variant and use terminal kinds to mark invalid or missing syntax.


Searching "JWT" should return the "JSON Web Token" tool

Description
I realize the search is very basic here, but it is very idiomatic to call "JSON Web Tokens" JWTs, so it would be good if this tool showed up when a user types in that search term.
Example: Search on "Token". Finds the tool.

Example: Search on "jwt". Does not find the tool.

Thanks for this, will create some improvement for the search!


Add '?' to ruby functions that return booleans

Done here https://github.com/JanEricNitschke/TicTacToe/commit/53da5ad367e9d59020fcf50e3b970a7874c987ea