DDoS incorrectly removed from Landing Zone tab when 'No' selected in Deploy network topology tab during Deploy to Azure

When deploying Contoso Deploy to Azure, if you select 'No' on Deploy network topology tab later on this removes the DDoS option from the Landing Zones tab. See screenshots below:

Then navigate to Landing Zones tab

Notice there is no DDoS option
If you enable vWAN on the Deploy network topology

Then the DDoS option appears in Landing Zones tab as shown

These options should not be linked and DDoS should remain in both scenario's.

If there’s no DDoS deployment in the connectivity hub, then there’s nothing to associate with in the landing zones, so this is per design.

Would it not make sense to still have a DDoS deployment in connectivity even if there isn't any hybrid connectivity option selected so online LZ's can associate with that deployment (I may be missing something in how DDoS works)?

It is a chicken and egg problem. If 'No' is selected under the network topology tab, we won't require a connectivity subscription since no central networking resources will be deployed, so we have nowhere to deploy the DDoS plan.
Outside of the Portal RI experience, customers can of course choose to deploy a DDoS plan in any subscription of their choice and link their LZ VNets (Corp/Online).

Closing this per Johan's comment. If no connectivity subscription is provided, we do not have a dedicated 'home' for DDoS during the setup.


fixed font color and weight in setting search bar

Description
Fixed #658

i can barely see it this way

Could you please elaborate me a little bit? The preview is working fine.

Could you please elaborate me a little bit? The preview is working fine.

yeah try typing anything in the search box in light mode.

@rwiteshbera once you sign the CLA we can merge this :)

@rwiteshbera once you sign the CLA we can merge this :)

How to sign it?

I have read the CLA Document and I hereby sign the CLA

just copy paste the above.

I have read the CLA Document and I hereby sign the CLA


Do not show pre-release versions by default

E.g. going to https://docs.rs/crate/rand/ currently shows the doc for 0.5.0-pre.1, which is a pre-release (unstable and not widely used). It would be more helpful to show the doc for the last stable release by default; currently this is https://docs.rs/crate/rand/0.4.2

Duplicate of https://github.com/rust-lang/docs.rs/issues/223


sleeper agent fake announcement

About the PR

this PR adds a CanFalseActivate field to the GameRuleComponent, nuke the evil AlertLevelInterceptionRule  by moving it to a nicer place and adds CanFalseActivate = true to the sleeping agents event
Why / Balance

fixes part of #28016
I'm tired of two agents in every round🤏.
Technical details

I tried to implement CanFalseActivate at the StationEvent level, but failed.
CanFalseActivate == a simple skip of executing game rules.
Seeing the AlertLevelInterceptionRule made me want to cry at the amount of code for 2 separate files and hardcore “green”... and it interfered with the implementation of CanFalseActivate, alas.
The AlertLevelInterceptionRule was nicely shoved into StationEvent :trollface:
the sleeping agents event collapsed due to the lack of at least 1 rule there :trollface: and I gave it MassHallucinationsRule :trollface:, which basically shouldn't work because of duration: 0, but in any case, aggravating paracusia due to enemy communications sounds logical to me
at this moment, the event, if CanFalseActivate = true, throws the switch function, which with a 50% probability writes in the logs that the event triggered falsely, enrolling occurrences and skipping everything except StationEvent comp.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Sleeper Agents can cause false announcements

thank u

Is this not already a thing? I thought the sleeper agent could roll on 0 people... (if it can't... there's an idea)

yaml linter is angry but in game and console everything is fine 🤏(I don't understand what the problem is)

I finished this.
Unfortunately, I had to resort to a system that literally just calls start for StationEventSystem.
but, uh, it works.
and I fixed the hardcoded bool.

"fuck you"
unconfirms your station threat

If you are coming back to this, leave a comment and I will re-open the PR.


Fix tests after GHA runner Java11 migration

Follow up of #31677, mitigate #32189
There is a hard blocker migrating all tests running on Java11 such that hive-exec (used by HCatalogIO test) not compatible with Java9+ due to the incompatible use of JDK internal class loader, see #32189 for detail

Still run HCatalogIO tests on Java8

Fix Python Flink validates runner test java container

Fix PostCommit Java Example Dataflow missing task named "java8PostCommit"

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

PostCommit Java Hadoop Versions fixed: https://github.com/apache/beam/actions/runs/10405574018

R: @damccorm

R: @tvalentyn


Implement JWT Authentication for regular users

This pull request primarily introduces JSON Web Token (JWT) authentication to secure image handling functionalities within the FastAPI application. Here's a breakdown of the changes:

JWT Authentication for Operations: Authentication mechanisms have been implemented to manage gathering/updating  account/shop and image uploading. User privileges are enforced, ensuring authorized access and enhancing system security.

Code Efficiency Refinements: The codebase has been optimized for efficiency by removing redundancies and streamlining processes. This includes the elimination of unnecessary functions and improved logic for ownership validation, enhancing the code's maintainability.

Miscellaneous Enhancements: Enhanced error handling and documentation updates, including clearer error messages aligned with HTTP status code conventions. The README file now provides improved instructions for setting up environment variables, aiding new contributors in the project setup.

The primary focus of this PR is the implementation of JWT authentication for account/shop/image operations, coupled with code efficiency refinements. These changes bolster security, functionality, and maintainability within the FastAPI application.

While this PR shares some similarities with #34, it also includes JWT implementations which will greatly help in addressing issues like those seen in #37.

Shit, i forgot to do auth-things to product


SUMO-218223 Add the details about AWS regions info

Purpose of this pull request
This pull request (PR) adds a table that lists the deployment and the aws region where Sumo's Privatelink service is set up. By this table, the user can reference it to know when the vpc peering is needed for them.
Issue number:
SUMO-218223

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

Thank you, @yuting-liu ! May I push this to production?

I just merged it. @kimsauce Thank you for approval.

@kimsauce Yea. sure. This should be good to be pushed to production


Links within docs page can't be navigated back

Godot version
v4.2.dev5.official [e3e2528ba] (or any 4.x)
System information
Windows 10
Issue description
Using links that points to another anchor within a docs page doesn't count as navigation ie. pressing back button takes you out of the page instead of the point where you clicked.

Steps to reproduce

Open docs page in script editor.
Press any link that navigates within the same page (for example property name).
Press back.

Minimal reproduction project
N/A

can you assign me this issue?

Godot engine does not practice assigning issues. Just create a PR.

can you assign me this issue?

Thanks for looking into it! If you figure it out, you might be interested in #82292 too, which is quite similiar problem.

Can't reproduce in latest build, it seems that this got fixed somewhere around 4.3 beta phase (altough I couldn't find the exact pr).


Refactor GPIOQueue.fill()

squash the two while loops down into one.
The unit-tests still all pass, and IMHO this is a little bit neater.  Last update f607a27...ac37ec2

Looks good to me; my only reason for using two loops originally is that filling the queue is a one-off operation so this permitted the body of the second loop (which will form the vast majority of the runtime) to be a bit simpler. But that's almost certainly premature optimization. Might be worth seeing just how long it takes to check an Event to see whether it's set - it might be an expensive operation (if it involves thread locks). But if not, this looks good to merge.

Yup, it's fast: no threading lock for checking whether it's set, and timings are about 1.3µs-1.5µs on a Pi3 under both Py2 and Py3.


[WIP] Fixes #1244 (When selecting category, show sample images)

Description (required)
Fixes #1244 (When selecting category, show sample images)

Updated Design of categorization page
Linked categorization with CategoryDetails page

Tests performed (required)
ManuallyTested on MOTO G5S+, with ProdDebug variant,
Screenshots showing what changed (optional)

Rotating the screen uploads the picture again, it seems.

@ujjwalagrawal17 Am curious if this page is the same screen that is used while uploading images?

Thanks @maskaravivek . We will try to complete #66 also. It would be even more nice then.

It works great, but there is still a bug when rotating the screen.
Could you have a look? Thanks!

@nicolas-raoul can we have a separate fix for this issue as this PR focusses on changing UI and adding support to view category details?

@ujjwalagrawal17 Yes, a separate fix branch (from current 2.8-release branch) is needed.
Thanks! :-)

Maybe this could be merged with the upload-overhaul, I checked the code on this commit and porting this to the new UI doesn't look too complicated. On the other hand, would merging this with the upload overhaul make everything harder to test?

Personally, I like the font size @ujjwalagrawal17 has used. It would be nice to have more opinions about it.

The screenshot in the first post looks similar to the existing size if I'm not wrong? I guess if we are wanting users to tap on it, keeping the larger size would make it easier for them.

Yup I haven't changed the font size. Also I am liking bigger font size.

I would like to merge this, but rotating the screen still tries to re-upload the image.

What is the status of this PR, @ujjwalagrawal17 ?

Umm, actually I am quite busy these days due to preparations for Campus placements. I will be completing it in free time. work is left on this PR. Can we please have it on hold for sometime. Thanks

Cool, thanks for the update. :)

Up @ujjwalagrawal17 :) We are excited for this PR.

@neslihan yup this will be a nice feature. I will try to find time to complete it soon.

If I remember correctly this issue was about this PR: https://github.com/commons-app/apps-android-commons/issues/1867

@albendz , do you still interested with this issue?

Like the #1712 PR, this one also requires some work. But in general, nicely implemented. I decided to note that since currently we have several beginner contributors who are looking for some tasks. IMO completing partially done tasks can be an option for beginners.

But in general, nicely implemented

Thanks :)

IMO completing partially done tasks can be an option for beginners.

I agree! That's how I started as well. By the way, I had added support for
removing non-exif metadata but as far as I remember, it is completely
broken. It could be better to remove that part but there may be metadata
that is not removed even if we remove all exif.
On Mar 18, 2019 3:14 PM, "neslihanturan"<EMAIL_ADDRESS>wrote:
Like the #1712
https://github.com/commons-app/apps-android-commons/pull/1712 PR, this
one also requires some work. But in general, nicely implemented. I decided
to note that since currently we have several beginner contributors who are
looking for some tasks. IMO completing partially done tasks can be an
option for beginners.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/pull/1784#issuecomment-473885470,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHRLo64iaXIJ6ehqX6MZz9T8Axy4xuJOks5vX4MRgaJpZM4VsJEW
.

Is rotating the screen still tries to re-upload the image. If yes can i put it in my to do list . Am currently trying to close of an issue an i would like to try this as a next thing.

Is rotating the screen still tries to re-upload the image. If yes can i put it in my to do list . Am currently trying to close of an issue an i would like to try this as a next thing.

@noahwalugembe The first step would be to make a branch on your side (on latest master), pull from this PR to the latest master, solve the conflicts, test a bit and send a new PR towards the ujjwalagrawal17:feat_issues_66 branch mentioning the issues you found. Thanks! :-)

@noahwalugembe The first step would be to make a branch on your side (on latest master), pull from this PR to the latest master, solve the conflicts, test a bit and send a new PR towards the ujjwalagrawal17:feat_issues_66 branch mentioning the issues you found. Thanks! :-)

Okay am going to work on this now

dear @nicolas-raoul  its like am not able to reproduce this error (Rotating the screen uploads the picture again, it seems.) but instead am getting th

is

Closing the PR as it is very old and needs more work to resolve so many conflicts than to open a new one. Also, the design of the already existing category details page while uploading seems similar.


Validation issue with +18004563456

Please read the "guidelines for contributing" (linked above) and fill in the template below.
Country/region affected (e.g., "US"):
Example number(s) affected ("+1<PHONE_NUMBER>"): +18004563456
The phone number range(s) to which the issue applies ("+1 555 555-XXXX"):
The type of the number(s) ("fixed-line", "mobile", "short code", etc.):
The cost, if applicable ("toll-free", "premium rate", "shared cost"):
Supporting evidence (for example, national numbering plan, announcement from mobile carrier, news article): IMPORTANT - anything posted here is made public. Read the guidelines first!
link to demo

dari mohamed

@ki1023lou210,
Thank you for using libphonenumber!
This is actually working as intended; +18004563456 is already marked by the demo as a valid TOLL FREE number.
I'll close this as it's not an issue with the library. Please consider joining the discussion group if you'd like to discuss this further.


How do I avoid Heroku Free Tier's 1hr "Sleep Mode"?

Hey-
Wondering if anyone has any experience hosting on heroku using just 1 dyno (free tier). It is working great for me except when using only 1 dyno it goes to sleep about an hour of "inactivity". I am curious how one goes about avoiding this. Is it as simple as setting the bot to run a command every 59 minutes, or does it require user input (ie: someone giving him a command) to avoid this sleeping issue.
Thanks!

what @raviteja548  said!
There's no option to stop it right now. And no contact info.

@raviteja548 @chesterl
For now I can remove them manually from the DB. See https://github.com/romainbutteaud/Kaffeine/issues/3 and https://github.com/romainbutteaud/Kaffeine/issues/4

If you are into Java EE another option would be to create a seperate app with embedded TomEE. Create a Scheduler via @Schedule and 'ping' the apps you dont want to go to sleep.
Important: Start the app on an Worker-Dyno, they arent reachable from outside, but they dont idle..
Just a thought....

@romainbutteaud kaffeine is very impressive and does a nice job of keeping the dynos awake. Much appreciation.

@romainbutteaud - Awesome little tool, kudos to you, my friend !

Is there any use case out there where pinging our Heroku app every hour for a site that gets little traffic resulting in dyno hours being drastically diminished to the point where the developer had to stop scheduling an hourly ping?


Playback not starting on next branch with either HttpRequest or YoutubeDL on M series ARM SoCs

Songbird version: next branch
Rust version (rustc -V): 1.69.0
Serenity/Twilight version: next
Description:
I'm trying to play a track with HttpRequest and it doesn't seem to be playing I was also able to recreate this using the voice example on the next branch. I don't get any errors or anything it just doesn't play.
...
Steps to reproduce:

Copy serenity/voice example from the next branch
Attempt to play youtube video using bot

It sounds like you're trying to play a track from youtube using an HttpRequest. You need to use YoutubeDl (and ensure you have yt-dlp installed) -- HttpRequest lets you download from an URL which points directly to an audio source (e.g., "https://test.com/song.mp3"), but youtube effectively requires that you use a tool like yt-dlp to retrieve the URL of the audio/video data from a CDN,

Sorry I wasn't clear is what i meant was I am trying to download a youtube video with YoutubeDL and it's not working and i am also having the same problem downloading and streaming standard files with HttpRequest

I found the issue. Rubato does not seem to work on MacOS with apple silicon I was able to get it working by running it in a linux VM on AWS.

Ouch, that's unfortunate. Perhaps you could raise an issue with the Rubato devs? From what I can see they claim SIMD support on AArch64 as a feature, so it's not as if the library is x86_64 only by design.

Yeah I’ll create an issue over there.

Interesting; unfortunately, I don't have a personal mac to test with. My best guess is endianness is biting somewhere between songbird and symphonia, unless there's some weird interaction with the chunk sizes we use?

I might be able to look into this when i have some more time.

Update: I tried to recreate this on an AWS Graviton instance and I was not able to recreate it. So this seems to be some sort of obscure issue with the M1 series specific architecture.  And not an issue with ARM architecture in general.

@maxall41 Sorry to necropost, but after #193 was raised I've had the opportunity to test on an M-series Mac and I'm able to playback and resample MP3 sources via HttpRequest in a call. The issue there was that Darwin errors out when setting the UDP receive buffer length to zero -- let me know if you still have any issues.


required params in dynamic routes does not work in all cases

As per the documentation of dynamic routes to make a route param :slug required, we need to create an index.vue file in _slug directory and this behaviour is working only when _slug is at the first level directory under pages directory. But when _slug is deeply nested in directory tree, the param becomes optional.
To reproduce this issue create pages with following structure
pages
├── README.md
├── _slug
│   ├── comments.vue
│   └── index.vue
├── index.vue
└── users
    └── _id
        └── index.vue

The routes definition in generated .nuxt/router.js will be

const _617e8672 = () => import('../pages/index.vue' /* webpackChunkName: "pages/index" */).then(m => m.default || m)
const _1b1466ab = () => import('../pages/users/_id/index.vue' /* webpackChunkName: "pages/users-id" */).then(m => m.default || m)
const _cf64727c = () => import('../pages/_slug/index.vue' /* webpackChunkName: "pages/slug" */).then(m => m.default || m)
const _3ac95998 = () => import('../pages/_slug/comments.vue' /* webpackChunkName: "pages/slug-comments" */).then(m => m.default || m)

export function createRouter () {
  return new Router({
    mode: 'history',
    base: '/',
    linkActiveClass: 'nuxt-link-active',
    linkExactActiveClass: 'nuxt-link-exact-active',
    scrollBehavior,
    routes: [
    {
      path: "/",
      component: _617e8672,
      name: "index"
    },
    {
      path: "/users/:id?", // THIS SHOULD BE /users/:id as per documentation
      component: _1b1466ab,
      name: "users-id"
    },
    {
      path: "/:slug", // This is working as expected, as _slug is at first level under pages directory **
      component: _cf64727c,
      name: "slug"
    },
    {
      path: "/:slug/comments",
      component: _3ac95998,
      name: "slug-comments"
    }
    ],
    fallback: false
  })
}

In above use case, path for users-id route should be /users/:id, not /users/:id? as generated.
This is reproducible with 1.0.0-rc3 version of nuxt as I tested today.

Hi @mnishihan,
Sorry, the documentation needs more details and we are working on it.
This following structure should resolve your issue:
pages
├── README.md
├── _slug
│   ├── comments.vue
│   └── index.vue
├── index.vue
└── users
    └── _id.vue
    └── index.vue // ID param is empty

Then, you have 3 options in with the pages/users/index.vue component:

Use validate to return 404 error page if params.id is undefined.
Use fetch to redirect to a specific URL
Display a content for /users route.


Fiber::try not working as expected

In the docs, it says
[Y]ou can run a fiber using the try method. If a runtime error occurs in the called fiber, the error is captured and the try method returns the error message as a string.
However, if I do something like
var fiber = Fiber.new {
    return "Hello"
}

IO.print(fiber.try())

shouldn't I see null, since no error occurred? Currently, I see "Hello" instead.
I can check for fiber.error, which is great, but seeing that return value from the try seemed odd. Let me know if I'm out of line :smile:

Yeah, the way the error is reported is kind of hacky here since the return value is used for both the error and the normal return value. I can think of a few options:

Keep it as is and just document it better.
Just return null on an error and require the user to check fiber.error.
Pass a function to try that gets invoked if an error occurs. Like:

var fiber = Fiber.new {
    "string".doesNotHaveThisMethod()
}

fiber.try {|error|
  IO.print("caught ", error)
}

If we do that, it may make more sense to call it catch than try. What do you think?

I probably don't have much insight since I've only written Wren for like the past 48 hours, and I haven't done much with Fiber. I ran into this confusion while updating Please.
That being said, I think it makes sense to only return an explicit return value. If the fiber aborts, just return a null.
I don't really understand the differences between Fiber::try and Fiber::call, as they currently exist. However, having something like
fiber.catch {|error|
  IO.print("caught ", error)
}
// null if fiber aborts
var returnValue = fiber.call()

makes sense to me. Are there any Wren programs making ample use of Fiber?

Fiber.try catch the error if there is one (same as try/catch in C++ or try/except in python), while Fiber.call just abort itself the let the error propagates to the calling fiber.

Are there any Wren programs making ample use of Fiber?

I don't think there are many Wren programs in general yet, so probably no. I've written a few toy programs to play with fibers, but it's definitely unexplored territory.

FWIW, I used fibers quite a bit for the testing framework I've been hacking on in Wren. I haven't found the current API to be too much of an issue; as long as the return type from a Fiber either through try or call is just a simple object we'll have this issue one way or another. There's no "state" for a Fiber as it is right now.

I used fibers quite a bit for the testing framework I've been hacking on in Wren. I haven't found the current API to be too much of an issue

That's a good sign. Real-world user feedback is always the best signal.

There's no "state" for a Fiber as it is right now.

It does have a bit, actually. You can call .isDone on a fiber to see if it has returned from its initial function yet. Also .error returns the runtime error if the fiber aborted. (This means the return value of try is redundant, but I put it there mostly for convenience.)

I like how Rust does this, with a Result<Type, Error> valued enum that can be pattern-matched to decompose to the appropriate value like this:
match thing_that_fails() {
  Error(error) {
    // Do something with error
  }
  Result(result) { result }
}

I think it'd make sense for try() to produce a similar Result type, which can be checked for an error or value. Could be a complicated pattern-matching feature, or it could be as simple as:
var result = fiber.try()
if (result.error) {
  // do error stuff
}
if (result.value) {
  // do value stuff
}

I feel like having try() maybe return a value or maybe return an error is just a bad idea and will likely only make it harder to support things like try/finally.

I feel like having try() maybe return a value or maybe return an error is just a bad idea and will likely only make it harder to support things like try/finally.

Yeah, I think you're probably right. Having a result type feels like overkill to me in a dynamically typed language, but it's not a bad idea. Another one is to pass a block that gets invoked only when a runtime error occurs, like:
var result = fiber.catch {|error|
  System.print("%(error) occurred!")
}

This would let you tell when an error occurred or not without having to guess from the type of the result. What do you think?

That handles the catch half, but what about finally?
Also, what would the behaviour be after running the catch handler? Return null? You'd basically have to handle the error branch twice...

Also, what would the behaviour be after running the catch handler? Return null?

I haven't put much thought into it, but I'm thinking .catch(_) would return the result of calling the fiber on success or the result of the error block on failure. But, yeah, without nonlocal returns, it would be a little tedious to propagate the error outwards from the block.


Test should run in a standalone database, not the default database.

The tests that make GET requests to the Django server currently run in the default database (which is specified here).
This is bad for development, as the tests may fail if data is present and then erase some but not all of the data.
It's also a risk for those deploying the container, as the tests are part of the container.

One solution is to have the tests run in a separate database when a database is needed.
I have made a start on this branch.


Keyboard is not dismissed if the user navigates to LHN when editing a comment - Reported by: @parasharrajat

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

Action Performed:

Edit a message in any chat on mobile.
Click the edit box to show the keyboard.
From the UI back arrow, go back to LHN.
Keyboard is not closed.

Minimize the app from Home button android.
Maximize the app from the recent apps, LHN is shown but keyboard is active.

Expected Result:
Keyboard should be dismissed when navigating to LHN.
Actual Result:
Keyboard is not dismissed when navigating to LHN.
Workaround:
User can dismiss the keyboard manually.
Platform:

Where is this issue occurring?

iOS
Android

Version Number: 1.1.12-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/44479856/139751815-db4dcb2b-4f84-497b-88da-f2d73c938167.mp4

Expensify/Expensify Issue URL:
Issue reported by: @parasharrajat
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1635751023286900
View all open jobs on GitHub

Triggered auto assignment to @tylerkaraszewski (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

There are two causes for this issue.

When we load a new report and it has an old edited message. then they will be autofocused and the keyboard will open.
When we edit a message on any report and use the UI back button to go back. The keyboard does not close.

Proposal

Point 1 will be solved in https://github.com/Expensify/App/pull/6138.
For point 2, we can use Keyboard.dismiss to close the keyboard .

We previously disabled this behavior which comes by default with React navigation for a reason. https://github.com/Expensify/App/blob/6299f48f645135224ba9af97d410bce6a17af156/src/libs/Navigation/AppNavigator/AuthScreens.js#L260
I think we should be revert that change instead I have a generic approach to close all the keyboards in this case.  When users navigate back to LHN, we call https://github.com/Expensify/App/blob/6299f48f645135224ba9af97d410bce6a17af156/src/libs/Navigation/Navigation.js#L41
so we can add Keyboard.dismiss here so that it works well without making it complicated for other components to manage back behavior.

Seems like an obvious external issue.

Triggered auto assignment to @kadiealexander (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Upwork job is live: https://www.upwork.com/jobs/~01cc79eda5fa4843d7

Hey everyone ! I am new here. I just saw this on Upwork and as per the requirement first I looked into the issue. So here are my findings,
Seems like this UI is designed with Tablets in mind as well, so on the mobile it is implemented as the separate screens but those are actually not.
HeaderView.js file has the following code,
{props.isSmallScreenWidth && (
<Tooltip text={props.translate('common.back')}>

)}
{props.re
This file shows back button only for the small screens. On pressing the back button it opens the LHN (I think it is left hand navigation) but the Edit Comment text field still remains in focus. If you type something on keyboard even after coming to LHN and then open the same thread again, you will see what you typed there.
So the text field never looses the focus even on pressing the back button, therefore the keyboard remains visible.
A quick solution just to fix this issue is call Keyboard.dismiss() in ReportScreen.js. Like this,
<HeaderView
reportID={reportID}
onNavigationMenuButtonClicked={() => {
Keyboard.dismiss()
Navigation.navigate(ROUTES.HOME)
}}
/>
This will forcefully close the keyboard, but note that the text field still remains in focus even if we do this. We can work more on this together. :)

Triggered auto assignment to @mountiny (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@mountiny do you have any thoughts on @thecodingeek's proposal above? :)

Hello @thecodingeek! Great to have you onboard. Thank you very much for your proposal. I must admit I like @parasharrajat more general solution a little bit more 😬  Essentially, as you proposed to add the Keyboard.dismiss() into the callback, I believe it would be better to place the keyboard dismissal into the openDrawer() function in Navigation.
We would want to close the keyboard anytime we go back to the Drawer so it make sense to put it into that place. Therefore, I am sorry, but I will pick @parasharrajatfor this job as he has reported this issue and proposed solid solution before you.
@thecodingeek Please, have a look around existing issues with Help Wanted label. I am sure you will be able to solve some quickly! 😊
@parasharrajat Could you please apply for the job on Upwork and let @kadiealexander once you do so 😊

Hello @mountiny, hmm if the keyboard is supposed to be closed anytime user moves to Drawer, then it is better to put the same in openDrawer() method.
I will check some other issue too :)

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.14-4 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/6205

If no regressions arise, payment will be issued on 2021-11-22. :confetti_ball:

@parasharrajat I've hired you in Upwork, could you please let me know once you've accepted so I can get payment out quickly for you? Thanks!

@kadiealexander Done

Paid $500 for reporting and fixing, thanks @parasharrajat!!


Move to Undo position before undoing (or redoing)

It's really hard to know what's being undone if it's happening off-screen...
I personally really, really appreciate editors that make sure you are viewing the undo location before performing it, otherwise it's hard to see what's been undone (you just see the after, not the before...)  This would follow an algorithm  something like:
auto lastEdit = undoStack.pop();
if (lastEdit.isVisible())
    lastEdit.undo();
else
{
    lastEdit.positionTo();
    undoStack.push(lastEdit);
}

Doesn't hurt to ask, right?  ;-)

There is a counter-argument to this: sometimes the thing you do at address A, such as defining a label or code hint, can have an effect at address B.  If you want to hit undo/redo while watching address B, you can right now.  This is fundamentally different from a source code editor, where changes are always local.
I'm wondering if an explicit "jump to address of last thing you changed" would be better.

This leads to a kind of diff between two versions of the project. Maybe a second tool which takes two 6502bench project files to compare them?

Moved to https://github.com/fadden/6502bench/wiki/TO-DO-List


Updates to the LED Preferences section of the Settings page

reorganized the Settings page slightly (by flattening out the headings and moving the WLED version declarations to the individual sections)
Started the process of updating the LED Preferences section to 0.14.1 (everything in the UI above the buttons section so far)

Thank you.
Looks ok but I'd like to ask you if you could also update other aspects of this page (while you are at it) as all of them require updating to be in sync with 0.14
Especially the added options in Sync, Security and effect transitions in LED settings.
If you are unwilling/unable to do that I'll understand, still it would be great if you could.

Thank you. Looks ok but I'd like to ask you if you could also update other aspects of this page (while you are at it) as all of them require updating to be in sync with 0.14 Especially the added options in Sync, Security and effect transitions in LED settings.

In a very technical sense I don't think I'll be able to. My actual WLED usage is quite basic, and as it stands I have no experience with (or need for) any of those more advanced features, so documenting them would require either a lot of time spent experimenting or a lot of time spent digging through the source code (and writing documentation takes me long enough even when I know what I'm talking about!); moreover, even in this PR I have some doubts about my work (e.g. my description of the Global LED Buffer setting). Mostly this was simply a case of me looking at the Settings UI, asking "what the heck does that do?", coming up with an answer, and then wanting to contribute that answer back to the community, and if I end up repeating that process in the future with more settings I'll be sure to create another PR.
FWIW, I'm not opposed to doing a bit of editing or cleaning up of existing descriptions, but writing new documentation for new features that I have zero experience with is beyond what I can contribute right now. Sorry!


DOC:  Add Pull Request Template

What's wrong with the existing documentation
Describe the issue
Without a proper pull request (PR) template, contributors sometimes submit PRs without providing adequate descriptions of the changes they have made. This lack of information makes it challenging for reviewers to understand the purpose and scope of the changes, verify them effectively, and ensure that the code meets the project's standards. Consequently, reviewers often need to spend extra time reviewing the entire codebase, which is inefficient and time-consuming.
Steps To Reproduce

A contributor raises a PR without using a PR template.
The PR lacks a clear description of the changes made.
Reviewers have to manually inspect all the code changes to understand their purpose.
The review process becomes lengthy and cumbersome.

Expected Behavior
When a PR template is used:

Contributors provide a clear and concise description of the changes made.
The template helps outline what areas of the code were modified and why.
Reviewers can quickly understand the context of the changes.
The review process becomes more efficient and less time-consuming.

Additional Information
Implementing a PR template can significantly improve the efficiency of the code review process. A well-defined template should prompt contributors to provide:

A summary of the changes made.
The reasons for the changes.
Any relevant issue numbers or references.
Instructions for testing the changes.
Any additional notes or context that might help the reviewers.

By using a PR template, we can ensure that all necessary information is provided upfront, allowing reviewers to focus on the quality and correctness of the code rather than spending excessive time deciphering what changes have been made and why. This will not only save time for reviewers but also improve the overall quality of the codebase.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I want to work on this project.Please assign this issue to me.under gssoc24

@arghadipmanna101 assign to me .

Hey, I already have the template ready. Can you please assign me this issue under gssoc'24?

@arghadipmanna101 I would like to work on this issue . so please assign this task to me under the label gssoc24 and level1,2,or 3

Duplicate issue. Already worked on #615


Cancell submission

PoH = When I put cancel submission it remains processing (5 days ago)

Can you please provide a link to your profile?

Hello!
https://app.proofofhumanity.id/profile/0xC18a0cb531697212183c2D13b84885B630577b1A
[https://app.proofofhumanity.id/images/open-graph-image.png]https://app.proofofhumanity.id/profile/0xC18a0cb531697212183c2D13b84885B630577b1A
Proof of Humanity, a sybil-proof list of humans.https://app.proofofhumanity.id/profile/0xC18a0cb531697212183c2D13b84885B630577b1A
Proof of Humanity, a system combining webs of trust, with reverse Turing tests, and dispute resolution to create a sybil-proof list of humans.
app.proofofhumanity.id
Thanks!!!

De: Henrique Barcelos @.>
Enviado: martes, 27 de abril de 2021 10:58
Para: Proof-Of-Humanity/proof-of-humanity-web @.>
Cc: Estebang188 @.>; Author @.>
Asunto: Re: [Proof-Of-Humanity/proof-of-humanity-web] Cancell submission (#199)
Can you please provide a link to your profile?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FProof-Of-Humanity%2Fproof-of-humanity-web%2Fissues%2F199%23issuecomment-827628478&data=04|01||6170f2211c7442a153fb08d9098497c6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637551287348123532|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=yxw77vDSGWEYtIKFbTXILx%2FXs6bPt4kyCBStVW8ZCOc%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATTKCT2QG3L56DUHQJBP3HTTK27J3ANCNFSM42VCIK5A&data=04|01||6170f2211c7442a153fb08d9098497c6|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637551287348123532|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Mil4ZhcRlzJ6Cg3olRqh9kf072ZxdazMtUuNYrgJmQE%3D&reserved=0.

This was likely a caching issue on the browser. Since then cache can no longer block the button.


DT Trees do not work with Roots Ritual

Describe the bug
When using the Ritual of the Spreading Forest with Roots nothing happens. This is because the DT leaves are returning either "Air" or null as their dropped itemstack, instead of the proper sapling itemstack.
Mods List
Minecraft: 1.12.2
Forge: <IP_ADDRESS>47
DynamicTrees-1.12.2-0.9.6.jar
dynamicTreesRoots-0.2.1.jar
mysticalworld-1.12.2-1.7.0.jar
mysticallib-1.12.2-1.5.0.jar
Roots-1.12.2-3.0.22.jar

Not currently resolve-able.


_get_kriging_matrix is very long for large datasets

Hi again,
I just saw that the function '_get_kriging_matrix' is very very long (about 5-10 secs) when the training points size is about 10,000 or 15,000.
Do you know if a speed up would be possible ? For instance, I was thinking that as I specified in my model 'n_closed_points' = 100, computing the kriging matrix over the 100 nearest points would be sufficient ?

@alexisrosuel The bottleneck must be in the cdist and variogram calculations in _get_kriging_matrix .
Would this issue be related to a local variogram model discussed in https://github.com/bsmurphy/PyKrige/issues/41?

Yes indeed, I was not sure yesterday, but it is clear now that this is the same issue

@rth Yes, very much so. A local variogram model could help in this situation.
I am still looking to get some funding for that work.

This should be discussed in #143. Closing for now. Feel free to re-open or go on with discussion in the linked issue.


Package unavailable at jspm

I wanted to share this in case it relates to any real issues with the library. Feel free to close it with "not our problem" though!
I am using Tiptap via Rhino and Rails. Specifically I am loading libraries with Import Maps. I couldn't pin this extension in my usual way:
> bin/importmap pin "@aarkue/tiptap-math-extension"
Couldn't find any packages in ["@aarkue/tiptap-math-extension"] on jspm

Nor did jsdelivr work
> bin/importmap pin "@aarkue/tiptap-math-extension" --from jsdelivr
Couldn't find any packages in ["@aarkue/tiptap-math-extension"] on jsdelivr

unpkg did work:
> bin/importmap pin "@aarkue/tiptap-math-extension" --from unpkg
Pinning "@aarkue/tiptap-math-extension" to https://unpkg.com/<EMAIL_ADDRESS>
Leading to this in my importmap.rb file:
pin "@aarkue/tiptap-math-extension", to: "https://unpkg.com/<EMAIL_ADDRESS>pin "evaluatex/dist/evaluatex", to<EMAIL_ADDRESS>pin "katex", to<EMAIL_ADDRESS>
I visited https://generator.jspm.io to see if I could cobble together a jspm entry manually. I searched for "@aarkue/tiptap-math-extension" and here's what I saw:

Error: No './dist/evaluatex' exports subpath defined in<EMAIL_ADDRESS>resolving evaluatex/dist/evaluatex imported from https://ga.jspm.io/npm<EMAIL_ADDRESS>

In case it's helpful, this seems a bit similar to my challenges importing MathExtension when loading up Rhino/Tiptap. Snippet from my rhino_controller.js file:
import MathExtension from "@aarkue/tiptap-math-extension"
...
rhino.addExtensions([MathExtension])

In the browser:
Failed to register controller: rhino (controllers/rhino_controller) SyntaxError: The requested module 'evaluatex/dist/evaluatex' does not provide an export named 'default'

Thanks for reporting this!
I am not familiar with jspm, but I will try to look into it when I find the time.


Reduce host permissions

Currently Spegel requires access to the Containerd sock to function properly. This mean that it is basically root on the node. In reality Spegel only needs read only access to all of the layers and tags. The issue is that the containerd client is required to do tag resolution.
One option would be to switch over to using the CRI API. One drawback currently is that it does not have an event subscription service which we currently use with the containerd client.
If  tags could be listed without the use of the containerd client the blobs directory could be mounted read only to the container and new layers could be detected with the help of a file watcher.

Created github.com/containerd/containerd/issues/8085 to see if there is an immediate solution that I have not seen yet.


Invalid Exception during multiple minting

Description
on mainnet, we have got strange error during minting as below.

exception: transaction<EMAIL_ADDRESS>failed precheck with status FAIL_INVALID
but hashscan says that minting tx has been success
(https://hashscan.io/mainnet/transaction/1674796082.966612026?tid=0.0.1112163-1674796073-250093810)

this exception has been occured during trying to minting 2500 NFTs.
from the client side, minting to 950th is suceess
and after that, we got that error.
but actually minting to 960th is success as shown in hashscan
we have designed the logic so that we can continue minting from the fail position when occurring exception.
but because of this kind of situation, we can't continue minting
(our db maintains the minted value up to 950, but in reality it is minted up to 960.)
Steps to reproduce
we have executed below logic on mainnet for minting 2500 NFTs

as a reference, our logic looks like as below.
async function asyncTokenMintTransaction(operator, bcTokenId, metadataBufferList) {
const transaction = new TokenMintTransaction();
transaction.setTokenId(bcTokenId);
transaction.setMetadata(metadataBufferList);
transaction.freezeWith(operator.client);
const transactionSigned = await  transaction.sign(PrivateKey.fromString(operator.private_key));
const response = await transactionSigned.execute(operator.client);
const transactionReceipt = await response.getReceipt(operator.client);
return transactionReceipt;
}
async function mintingNFT( ... ) {
while (!(no more nft data to mint)) {
// prepare metadata(separateSetMetadatas) for this time minting
// separateSetMetadatas is holding the metadata for 10 nfts
    // mint, we request to minting 10 nft at one time.
    const receipt = await hedera.asyncTokenMintTransaction(operator, bcTokenId, separateSetMetadatas);

    // do something
}
return ;

}
Additional context
No response
Hedera network
mainnet
Version
v2.18.1
Operating system
Linux

Hello @sehyoung-james
I tried the same setup: minting 2500 NFTs in batches of 10, so in total 250 iterations
The result was that all of them were successfully minted.
Also, just to point out that status FAIL_INVALID states a system error and the transaction failed because of invalid request parameters. This basically suggests that something unusual happened on the services side. Might also have something to do with the state of the network at the given time. Given the fact that so many previous iterations were successful, I think that the state of the network was the issue (perhaps it might have something to do with the throttling at that time or something else)
I would suggest to try handle the eventual appearance of this error on your side: maybe by checking if the NFTs were in fact minted if the error occurs

@petreze
Thank you for your test and review comments.
I agree with your comments about FAIL_INVALID .
However, the problem seems to be that the same minting request was processed as success on one side and registered as success on hashscan, and the other side generated a FAIL_INVALID error.
Because of this phenomenon, it is impossible to handle exceptions for errors in dapp.
I would like to ask if there is any room for improvement in the part where the same minting request gives different results.

I have noticed similar issues, in a node scripts but also in the browser, through HashPack, where you would get an error but the transaction would be successful as could be seen on HashScan.
This has happened with different types of transactions (Smart Contract transactions mostly I believe) from what I can recall and also I believe has happened the other way around, where you would get a success status back but the transaction would have failed.
Here is an example :
This is the transaction that errored out and please notice the timestamp:
PrecheckStatusError: transaction<EMAIL_ADDRESS>failed precheck with status INSUFFICIENT_TX_FEE
Here is the confirmed transaction, where the tid timestamp is before the timestamp in the error and the consensus timestamp is after:
https://hashscan.io/mainnet/transaction/1673888924.859310014?tid=0.0.1041055-1673888912-639808678
This transaction has been run from a script and there wasn't any batch of transactions this being the only one.

It is in fact a strange behaviour, will dig into it deeper and investigate it. Note that this might as well has something to do with services part of Hedera, it can take some time to discover the issue while we are syncing with the other team

Hi @victorholo, @sehyoung-james after digging deeper and testing, we are unable to reproduce this issue.
There will be a new testnet release this Thursday - can you try again then and if you are still facing this issue - please send us a code snippet so that we can properly investigate based on your use-case.

Hi @victorholo, @sehyoung-james, did you get the chance to test the new version of the SDK together with the new testnet release?


New readme Issue #1565 resolved.

Fixes Issue #1565
Closes #1565

Checklist

[ x] You've linked this PR to the correct issue.
[ x] You have checked that the code is working correctly.
[ x] You ⭐ the repository!

Note to reviewers

@arcVaishali We are extremely Sorry For The Delay But because of some personal Reasons @Spyware007 is not available. This is an important PR, So we have to wait for him before merging it.
Thanks for your patience.

@arcVaishali We are extremely Sorry For The Delay But because of some personal Reasons @Spyware007 is not available. This is an important PR, So we have to wait for him before merging it.
Thanks for your patience.

I didn't know that was the case. I get it now, I will wait.


Add Support for Adafruit QTPY ESP32-S2

Adding support for Adafruit QT Py ESP32-S2 WiFi Dev Board with STEMMA QT
Product URL: https://www.adafruit.com/product/5325

Matching board definition - https://github.com/adafruit/Wippersnapper_Boards/pull/31

Successfully running and connected to WipperSnapper, merging when passing.


JDK folder name does not match

First off, thank you for this template, it has been really helpful.
As for the issue I found... The JDK folder in the most recent version of Defold is labeled "jdk11.0.15+10". The defold.sh file in this example project has the JDK folder listed as "jdk11.0.1-p1". This is causing a build error that prevents the project from being opened from VS Code.

Oh, thanks! Is it enough to just edit the path?


Bump Elasticsearch 6.8.14 -> 6.8.15 / 7.9.1 -> 7.9.3

A large number of bug fixes in both of these updates, but also
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-22137
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/2619#0feb0988-357c-4735-8381-fa4f99101777


instancetypes: Address functional test TODOs and extend libdv

/sig code-quality
What this PR does
This PR introduces WithDefault{Instancetype,Preference and WithDataVolumeSourceRef to libdv to address some basic TODOs within the instancetype functional tests.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/approve


dbatools is loading an assembly reference of the wrong version

Before submitting a bug report:

[X] Ensure you are able to reproduce it on the latest released version (we release often)
[X] Verified this bug is not already reported in an issue
[X] Verified errors are not related to permissions
[X] Can reproduce in a clean PowerShell session (clean = powershell -NoProfile)

Steps to Reproduce
<#
    Provide exact (or sanitized) code to reproduce the error
#>
Import-module dbatools

add-type -AssemblyName "Microsoft.SqlServer.Management.IntegrationServices, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=89845dcd8080cc91"

$ConnectionString = "Data Source = SQLServer;Initial Catalog=master;Integrated Security=SSPI"
$conn = new-object System.Data.SqlClient.SqlConnection $ConnectionString
$SSISServer = New-Object Microsoft.SQLServer.Management.IntegrationServices.IntegrationServices $conn

Expected Behavior
I expect to get an SSIS Object I can then manipulate directly, instead I get an exception referencing the wrong version of SQL Server.

Actual Behavior
I get the following exception error (referencing Version <IP_ADDRESS>, but I'm on SQL 2014, so the highest versions I have are <IP_ADDRESS>)
New-Object : Exception calling ".ctor" with "1" argument(s): "Could not load file or assembly 'Microsoft.SqlServer.Diagnostics.STrace, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=89845dcd8080cc91' or one
of its dependencies. The system cannot find the file specified."
If I don't do anything to bring in dbatools (import-module, get-module, use commands from dbatools, etc...) it works fine.

Environmental data

PowerShell:

Name                           Value

PSVersion                      5.1.14409.1012
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1012
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

SQL Server:
Microsoft SQL Server 2014 (SP2-CU8) (KB4037356) - 12.0.5557.0 (X64)
Oct  3 2017 14:56:10
Copyright (c) Microsoft Corporation
Enterprise Edition: Core-based Licensing (64-bit) on Windows NT 6.3  (Build 9600: ) (Hypervisor)

Thank you for the report. I don't know enough about the way DLLs load, @FriedrichWeinmann could you provide some insight?

Well, dbatools brings its own copy of the SIS library in version 14. You can also load the older version by explicitly calling the full name as you do, but it appears to default to using the newer version.
If you forgo your Add-Type call, will it work or does it produce the same error?

if I comment out the Add-Type, I still get the error so long as dbatools has any sort of reference before getting to the Integration services assignment.  Of course if I comment out both the dba tools and the add-type I get an error of no such class type found.

I've discovered that so long as I add all of the SQL 2014 assemblies/import SQLPS before I reference anything with dbatools, PoSH grabs things correctly, and I have no errors.  So that's at least a functional workaround.

From here:

The type will be loaded from the first assembly which you load.

So I guess this is behavior by default.
PS. SSIS is a bastard kind of dll, which only works with respective version of SSIS Server without any backward compatibility. I really dislike that decision, which not only complicates life when working with Powershell, but also prevents you from connecting to an older SSIS server from the latest SSMS. Mildlyinfuriating.

having just the right executable installed on every server just to deploy packages without auto-upgrading them does not qualify as mildly..... ;-)


model.fit hangs if validation_steps is zero

System information

I written custom code (as opposed to using a stock example script provided in TensorFlow)
OS Platform and Distribution: Kaggle Kernel, 'Linux-4.19.112+-x86_64-with-debian-buster-sid'
TensorFlow installed from (source or binary): installed from pip
TensorFlow version (use command below): 2.1
Python version: 3.7.6
CUDA/cuDNN version: V10.1.243
GPU model and memory: Tesla P100-PCIE-16GB

Current Behaviour
model.fit suspends  execution or hangs after training first epoch runs if value passed for validation_steps=len(valid_inputs)//BATCH_SIZE is equal to zero.
Moreover, validate for x steps is not printed before training starts.
I believe a ValueError could be raised if validation_steps is 0 and not None. (or execution could proceed further with next epoch over training set)
Standalone code to reproduce the issue
VALIDATION_STEPS=len(valid_noisy)//BATCH_SIZE # Results in 0
model.fit(
    train_dataset,
    steps_per_epoch=len(train_noisy)//BATCH_SIZE,
    epochs=200,
    validation_data=valid_dataset,
    validation_steps=VALIDATION_STEPS
)

Complete Notebook
If feasible, can I contribute via a PR if the above issue is valid and can be addressed by not a core contributor?
Thanks a bunch!

@vasudev13
I have tried in colab with TF version 2.2 .Please, find the gist here.Is this the expected behavior? Thanks!

Close enough, in
train_noisy,valid_noisy,train_clean,valid_clean=train_test_split(noisy_paths,clean_paths,test_size=0.1,random_state=SEED)

VALIDATION_STEPS=1 # len(valid_noisy)//BATCH_SIZE = 0

change the following:
VALIDATION_STEPS=len(valid_noisy)//BATCH_SIZE  and print the value, its probably going to be 0.
If fit is called with that argument (i.e. value 0) for parameter validatiton_steps value, it just hangs after first training iteration as opposed to some message or error being raised.
Thanks for quick response!

Can you please test with tf-nightly version? Thanks!

@ymodak I tried with nightly and was getting an altogether different error, fit gives error on training itself. However, with 2.2 it does raise an error on validation (training epoch does occur) but not exactly to this issue in particular.
I apologise for the late response. Should I attach detail error logs?

Yes please attach new error logs.
I created a toy model that uses validation_steps = 0 and happens to execute successfully.
Please take a look at the gist

@ymodak , checked with version 2.3, the execution is successful
Thanks for helping out to you and @ravikyram as well, you can close this issue!

One peculiar thing, I noticed when changing validation_steps argument from 0 to say 1 (or any non-zero integer) in the following snippet, validation step is still not performed (see pic 1), as well as the vice-versa i.e. when changing from 1 to 0, validation is performed (see pic 2) as opposed to not being performed. This occurs only when rerunning the same code cell in the notebook without restarting.
model.fit(
    train_dataset,
    steps_per_epoch=len(train_noisy)//BATCH_SIZE,
    epochs=200,
    validation_data=valid_dataset,
    validation_steps=1
)

You can look into this if you get the time. Thanks for your patience and help!


add versat-io submodule; add versat-io units; separate DATA_W, used for the CPU, from DATAPATH_W, used for datapath

@JDLopes I would suggest to put define nIO (nVI+`nVO) in xdefs.vh instead of xversat.vh.
xversat.vh should only contain hardcoded constants that can be changed using the mkvhdr.py script


[TRIVIAL] Document secure_gateway for gRPC

It is recommended to add a secure_gateway section to [port_grpc] when the server is an ETL source for a reporting mode server. This commit adds a few lines to the example config to explain such. There are no code changes.

This change is

I think the section is port_grpc, not grpc_port.
I suggested removing a few extra words, but it's not a big deal either way :)

Good catch, thank you @intelliot . I applied the rest of your suggestions too.

How can i unsubscribe???????


Add overlord

PR for overlord part 1

I didn't see any specs for BombHelpers.

This is looking like a good start.  It looks like you basically tried to get Rails in Sinatra.  Lot of work!
Work on moving logic to the correct classes/locations.  Take a look at our comments and refactor.

So there are definite fixes to make to the backend, but as a testing assignment, the biggest thing you should do is flesh out the integration tests. Right now they cover very little of the app.
Please take another stab.


MEDT -> NONE

This change is

Coverage remained the same at 96.873% when pulling 4476ffcd69e2fb9b7e0e22ea0acbe029573dac8b on VolVoz:master into fdb07e3f7ce74b55932836b6f2fa4f0a0fede191 on openprocurement:master.


JsonSerializer do Not call OnDeserializedAtribute function

HI， I don't know why the JsonSerializer doesn't call the OnDeserialized but JsonConvert does
Thanks  for your  attention!
namespace ConsoleApp1
{
    class Program
    {
        private static JsonSerializerSettings defaultJsonSerializerSettings =
        new JsonSerializerSettings
        {
            NullValueHandling = NullValueHandling.Ignore,
            TypeNameHandling = TypeNameHandling.Auto,
            TypeNameAssemblyFormatHandling = TypeNameAssemblyFormatHandling.Simple,
            PreserveReferencesHandling = PreserveReferencesHandling.Objects,
            Formatting = Formatting.Indented
        };

        static void Main(string[] args)
        {
            School school = new School();
            Student a = new Student { Name = "Apple" };
            Student b = new Student { Name = "Banana" };
            Student c = new Student { Name = "cup" };
            Student d = new Student { Name = "dean" };
            school.Students.Add(a);
            school.Students.Add(b);
            school.Students.Add(c);
            school.Students.Add(d);

            school.StudentCouncil.Add(d);
            school.StudentCouncil.Add(a);

            Console.WriteLine("-------------------------------");
            Console.WriteLine("JsonConvert Begin");
            var res = JsonConvert.SerializeObject(school, defaultJsonSerializerSettings);
            var s = JsonConvert.DeserializeObject<School>(res, defaultJsonSerializerSettings);
            Console.WriteLine("JsonConvert End");
            Console.WriteLine("-------------------------------");
            Console.WriteLine("JsonSerializer Begin");
            using (MemoryStream ms = new MemoryStream())
            using (StreamWriter writer = new StreamWriter(ms))
            using (JsonWriter jsonWriter = new JsonTextWriter(writer))
            {
                var serializer = JsonSerializer.Create(defaultJsonSerializerSettings);
                serializer.Serialize(jsonWriter, school);
            }

            using (MemoryStream ms = new MemoryStream())
            using (StreamReader reader = new StreamReader(ms))
            using (JsonReader jsonReader = new JsonTextReader(reader))
            {
                var serializer = JsonSerializer.Create(defaultJsonSerializerSettings);
                var school2 = serializer.Deserialize<School>(jsonReader);
            }
            Console.WriteLine("JsonSerializer End");
            Console.WriteLine("-------------------------------");

        }
    }
    public class Student
    {
        public string Name { get; set; }
        [OnDeserialized]
        private void OnDeserialized(StreamingContext context)
        {
            Console.WriteLine("Student OnDeserialized was called");
        }
    }
    public class School
    {
        public ObservableCollection<Student> Students { get; } = new ObservableCollection<Student>();
        public ObservableCollection<Student> StudentCouncil { get; } = new ObservableCollection<Student>();
        [OnDeserialized]
        private void OnDeserialized(StreamingContext context)
        {
            Console.WriteLine("SCHOOL OnDeserialized was called");
        }
    }
}

results
-------------------------------
JsonConvert Begin
Student OnDeserialized was called
Student OnDeserialized was called
Student OnDeserialized was called
Student OnDeserialized was called
SCHOOL OnDeserialized was called
JsonConvert End
-------------------------------
JsonSerializer Begin
JsonSerializer End
-------------------------------

Because there is no content in the JsonTextReader the serializer is using.
Use stackoverflow for questions.

You are right, I made a mistake .
Thank you for your so quickly resonse
            Console.WriteLine("-------------------------------");
            Console.WriteLine("JsonConvert Begin");
            var res = JsonConvert.SerializeObject(school, defaultJsonSerializerSettings);
            var s = JsonConvert.DeserializeObject<School>(res, defaultJsonSerializerSettings);
            Console.WriteLine("JsonConvert End");
            Console.WriteLine("-------------------------------");
            Console.WriteLine("JsonSerializer Begin");
            using (MemoryStream ms = new MemoryStream())
            {
                StreamWriter writer = new StreamWriter(ms);
                JsonWriter jsonWriter = new JsonTextWriter(writer);
                {
                    var serializer = JsonSerializer.Create(defaultJsonSerializerSettings);
                    serializer.Serialize(jsonWriter, school);
                    writer.Flush();
               }
                ms.Position = 0;
                StreamReader reader = new StreamReader(ms);
                JsonReader jsonReader = new JsonTextReader(reader);
                {
                    var serializer = JsonSerializer.Create(defaultJsonSerializerSettings);
                    var school2 = serializer.Deserialize<School>(jsonReader);
                }
            }
            Console.WriteLine("JsonSerializer End");
            Console.WriteLine("-------------------------------");
-------------------------------
JsonConvert Begin
Student OnDeserialized was called
Student OnDeserialized was called
Student OnDeserialized was called
Student OnDeserialized was called
SCHOOL OnDeserialized was called
JsonConvert End
-------------------------------
JsonSerializer Begin
Student OnDeserialized was called
Student OnDeserialized was called
Student OnDeserialized was called
Student OnDeserialized was called
SCHOOL OnDeserialized was called
JsonSerializer End
-------------------------------


add license file

adding a file for the license in addition to the readme indication, should make it computer readable

Thank you!


Ticket: #24863 - Copy properties of QuerySet too when using "QuerySet.as_manager()"

Resolves this: https://code.djangoproject.com/ticket/24863
Example:
Let's say we have a queryset like this:
class MyQuerySet(QuerySet):
    @property
    def active(self):
        return self.filter(is_active=True)

    def filter_visibles(self):
        return self.filter(is_visible=True)

If we use MyManager = MyQuerySet.as_manager(), django doesn't include active property to MyManager currently. With the help of this PR, django also copies active property and makes it usable.

@KaratasFurkan Thanks for your efforts :+1: We decided to close the ticket as "wontfix".


HttpClientBuilder is no longer supported on Android starting SDK version 23.

It isn't possible to instantiate RedditClient with HttpClientBuilder anymore, if an Android project is using the latest SDK. Maybe it's time to start migrating to OkHttpClient?
RedditClient client = new RedditHttpClient(REDDIT_USER_AGENT, HttpClientBuilder.create().build());
is no longer supported.

You'll need to add
android {
    compileSdkVersion 23
    buildToolsVersion "23.0.0"
//The following line
    useLibrary 'org.apache.http.legacy'
    ...
}

As shown in this by user Jinu
If you are using target sdk as 23 add below code in your build.gradle

android{
    compileSdkVersion 23
    buildToolsVersion '23.0.1'
    useLibrary  'org.apache.http.legacy'
}

and change your buildscript to
classpath 'com.android.tools.build:gradle:1.3.0'

For more info follow this link


Nixify the project.

Add a nix developer environment. WIP!

Fyi Here is the current j2do list
GPB Jonah To Dos's
TODO:
https://annas-archive.org/
https://github.com/aigis-llm for Jonah AI
Using port 8069-8092
https://localai.io/advanced
frm bills phone
nix-flake-update
git hub actions
Nix-collect-garbage
Nix home manager
Catppuccin
https://nix.dev/tutorials/nix-language.html
nixos-config
atuin for nushell
atuin sync as well
carapace and starship as well
update all should have for nix:

cd /etc/nixos
git pull
sudo nixos-rebuild switch

TODOs for GPB and Jonah:
rpi nix install (zoom)
kupd nix (you have this at the bottom of your notes)
nix-on-droid setup (zoom)
other fun nix things (zoom?)
collaborating on aigis (github pull requests from branches on aigis and probably some zoom meetings)
WSL2 with arch and gui  for 4H. Yes I have fractional scaling.
Remote access to the vm via rdp:
VBoxManage modifyvm VM-name --vrde on
For kryz:
Step 0: Use KDE Plasma 6. Done.
Step 1: Set up home-manager on it so we can use Floorp and Nushell. We need to zoom for this.
Step 2: Switch home-manager to use their Floorp config options to install extensions and stuff.
Step 3: Probably more nix.
Block Chain
look at ethereum and ipfs, they are important. learn solidity and truffle
development framework and ganache mock blockchain
use geth not meatamask
use light sync
Nix Remote Access ???:
From Perplexity:
To remotely access a newly installed NixOS desktop, there are several options you can consider, each with its own setup process and features. Here are some of the best methods:
XRDP:
XRDP is supported natively on NixOS and allows remote desktop access using the RDP protocol, which is compatible with Windows clients.
To set up XRDP, add the following to your configuration.nix:
text
services.xserver.enable = true;
services.xserver.displayManager.sddm.enable = true; # or another display manager
services.xserver.desktopManager.plasma5.enable = true; # or another desktop manager
services.xrdp.enable = true;
services.xrdp.defaultWindowManager = "startplasma-x11"; # or another window manager
services.xrdp.openFirewall = true;
Rebuild your system with sudo nixos-rebuild switch to apply the changes
Done:
Bill Kamp
IAGP
(651) 308-3703
@.***
From: bitbloxhub @.>
Sent: Monday, October 14, 2024 12:18 PM
To: aigis-llm/aigis @.>
Cc: Bill Kamp @.>; Assign @.>
Subject: Re: [aigis-llm/aigis] Nixify the project. (PR #1)
@bitbloxhub commented on this pull request.
Right now, I am running into an issue similar to this https://discourse.nixos.org/t/python-importerror-libstdc-so-6-cannot-open-shared-object-file-no-such-file-or-directory/36988 , where a python package is trying to get libstdc++.so.6 and cannot find it. If i set LD_LIBRARY_PATH as reccomended in here https://discourse.nixos.org/t/python-importerror-libstdc-so-6-cannot-open-shared-object-file-no-such-file-or-directory/36988/4 , then i run into an issue similar to this https://discourse.nixos.org/t/new-install-nix-wants-glibc-2-38-debian-12-has-2-36/36109 , where it appears to be trying to use the system glibc.
—
Reply to this email directly, view it on GitHub https://github.com/aigis-llm/aigis/pull/1#pullrequestreview-2367229291 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAERY3WPXF6HTO724VDD5JLZ3P4GHAVCNFSM6AAAAABP5NOSRKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRXGIZDSMRZGE .
You are receiving this because you were assigned.  https://github.com/notifications/beacon/AAERY3QXQZVD5GUQYPYXCJ3Z3P4GHA5CNFSM6AAAAABP5NOSRKWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTUNDEGWW.gif Message ID: @.*** @.***> >

11/14/24: When I run git pull in /etc/nixos, I get:
error: insufficient permission for adding an object to repository database .git/objects
fatal: failed to write object
fatal: unpack-objects failed
Bill Kamp
IAGP
(651) 308-3703
@.***
From: bitbloxhub @.>
Sent: Monday, October 14, 2024 12:18 PM
To: aigis-llm/aigis @.>
Cc: Bill Kamp @.>; Assign @.>
Subject: Re: [aigis-llm/aigis] Nixify the project. (PR #1)
@bitbloxhub commented on this pull request.
Right now, I am running into an issue similar to this https://discourse.nixos.org/t/python-importerror-libstdc-so-6-cannot-open-shared-object-file-no-such-file-or-directory/36988 , where a python package is trying to get libstdc++.so.6 and cannot find it. If i set LD_LIBRARY_PATH as reccomended in here https://discourse.nixos.org/t/python-importerror-libstdc-so-6-cannot-open-shared-object-file-no-such-file-or-directory/36988/4 , then i run into an issue similar to this https://discourse.nixos.org/t/new-install-nix-wants-glibc-2-38-debian-12-has-2-36/36109 , where it appears to be trying to use the system glibc.
—
Reply to this email directly, view it on GitHub https://github.com/aigis-llm/aigis/pull/1#pullrequestreview-2367229291 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAERY3WPXF6HTO724VDD5JLZ3P4GHAVCNFSM6AAAAABP5NOSRKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRXGIZDSMRZGE .
You are receiving this because you were assigned.  https://github.com/notifications/beacon/AAERY3QXQZVD5GUQYPYXCJ3Z3P4GHA5CNFSM6AAAAABP5NOSRKWGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTUNDEGWW.gif Message ID: @.*** @.***> >

i forgot to add --bun to bunx, so it was trying to run node and not bun 🤦🏻. also i wasted some time misspelling library when setting LD_LIBRARY_PATH.


[ASCII-1531] Split tagger component into interface and implementation

What does this PR do?

Split the tagger component into interface package and implementation package
Motivation

As part of the Q2 OKR the ASC is trying to reduce the serverless binary overhead. As part of that effort we are removing certain components for serverless binary.
We are creating noop implementation of those components. In the case of the tagger we first needed to split the component into interface and implementation.
The great things about this is that not only the serverless binary benefits from this split as we can see in the import differences comment https://github.com/DataDog/datadog-agent/pull/24922#issuecomment-2069879151
Additional Notes

Some of the changes I made are:

Move types from the implementation package comp/core/taggerimpl/... into the interface package comp/core/tagger
Update the Tagger interface. Unfortunately, that means I had to add the Start and Stop functions which is an anti-pattern for the component framework. I will try removing those in a future PR.
Start using the tagger component interface rather than concrete struct types for some of the component interface functions.
Update all references in the code.

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

There is no behavior change in the PR as it is mostly refactoring/moving code around

For the record I computed some stats for the init duration of the serverless agent on this branch:
base:   min=478.33 ms, max=511.23 ms, avg=493.6 ms, median=490.38 ms
branch: min=460.34 ms, max=495.8 ms, avg=476.4 ms, median=477.56 ms
So splitting the tagger component removes around ~15ms of init time duration

/merge

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/merge


DCA: test deployment with updated REST API and docker containers

Once changes from #399, #400,  https://github.com/Sage-Bionetworks/schematic/issues/899 and https://github.com/Sage-Bionetworks/schematic/issues/900,  are incorporated in DCA and schematic respectively, test the deployment in our EC2 development environment.
AC: docker container(s), reducing persistency (i.e. no use of local file system in DCA), and schematic's REST API updates are confirmed working.

This issue is relevant for dashboard functionality

Successfully have the schematic REST API running from a container and DCA running on a container on the same EC2 instance. DCA can access all of the validation and submission endpoints. The dashboard functionality will be implemented after some updates to schematic.

Also, DCA is set configured with envrionmental variables, not config files. Though the workflow is still dependent on some files on the server.

@afwillia it sounds like at this point we have #400 resolved and also the app works with the REST API. Once #399 is done, are we at a point where we'll no longer need config files and manifests stored locally, so that we can pass for testing in the Fargate architecture (at least for core features, perhaps not the dashboard functionality)? Or are there other outstanding items?

@milen-sage we need to figure out what to do with www/config.yml. Maybe we can create it during the docker image build.

Yes, since we have @rrchai script that auto generates www/config.yml, we should be able to do that now?

Yeah. I want to wait until the JSON updates for manifest/validate and manifest/submit are both merged into the schematic develop branch. Then once I confirm the DCA non-dashboard code works with that, I'll add the config.yml generation to the dockerfile. I think this can be done next sprint.

All of the relevant schematic endpoints have been updated to accept/return JSON in the develop branch. The DCA + schematic images have been rebuilt.


Feature request: Support nacos v2 sdk

Is your feature request related to a problem? Please describe.

The nacos v2 sdk is not currently supported.

Describe the solution you'd like

Support nacos v2 sdk

Describe alternatives you've considered

None

Additional context

Add a new naocs-v2 directory, managed using submodule
sdk repo: https://github.com/nacos-group/nacos-sdk-go

can I have a try?


Add an option to inject additional measurements to load config

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR allows to inject additional measurements via CL2_ADDITIONAL_MEASURMENT_MODULES to load/config.yaml.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

/assign @mborsz

/retest

It looks like tests failures are unrelated to my change as clusterloader is not able to ssh on to  the master:
error getting SSH client to prow@<IP_ADDRESS>:22: 'ssh: handshake failed: ssh: unable to authenticate, attempted methods [none publickey], no supported methods remain

/lgtm
/approve

Tests are failing with:
ERROR: (gcloud.compute.instances.create) Could not fetch resource:
 - Quota 'N2_CPUS' exceeded.  Limit: 24.0 in region us-east1.
	metric name = compute.googleapis.com/n2_cpus
	limit name = N2-CPUS-per-project-region
	limit = 24.0
	dimensions = region: us-east1

https://github.com/kubernetes/test-infra/pull/31546
I will raise this in sig-testing slack channel.

/retest

FYI https://kubernetes.slack.com/archives/C09QZ4DQB/p1704722468347639

/retest

Thanks @Argh4k for looking into the test failures.
I have the same cl2 test failures in my PR https://github.com/kubernetes/perf-tests/pull/2494.
I see that the quota increase for N2 machines is in progress.
Is it tracked somewhere?

/retest

@dlapcevic unfortunately all the information that I have are from the slack thread that I posted here :(

@mborsz can we push it through? Clusterloader2 test is failing with N2 quota issue and clusterloader2-kubemark is failing with:
ERROR: (gcloud.compute.instances.create) Could not fetch resource:
 - Setting minimum CPU platform is not supported for the selected machine type e2-standard-8.

They do not seem to be connected to my CL.
For testing I've set up a simple test with measurments in file one directory above and run the clusterloader with that. I'm attaching files and generatedConfig for that (they are formatted as txt but in reality those are yamls as github does not allow to attach yamls in the comment)
generatedConfig_test.txt
measurements.txt
config.txt

FYI https://github.com/kubernetes/k8s.io/issues/6271


Fix link to design document

We shouldn't point to PRs but to the merged document /cc @glennc

@terrajobst I think we should just remove this link completely. What goal is a user going to accomplish by clicking on that link? Are they going to read that entire design spec? It seems a bit out of place.


Detect different memory size grouping

With WmiObject classes Win32_PhysicalMemory and Win32_PhysicalMemoryLocation we should be able to detect the grouping per CPU socket. This is critical for determining NUMA boundaries as you can run into performance issues if you have a large difference.

In previous testing of my home computer, this didn't seem to line up. Don't see the benefit at this time to look into this.

In previous testing of my home computer, this didn't seem to line up. Don't see the benefit at this time to look into this.


Garbage collector stacktrace in log in v3.45 and later

From times to time, we have this stack trace inside error log of java server.
Exception in thread "async-dispatch-121" org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: com.sixsq.slipstream.persistence.Run.runtimeParameters, could not initialize proxy - no Session
	at org.hibernate.collection.internal.AbstractPersistentCollection.throwLazyInitializationException(AbstractPersistentCollection.java:575)
	at org.hibernate.collection.internal.AbstractPersistentCollection.withTemporarySessionIfNeeded(AbstractPersistentCollection.java:214)
	at org.hibernate.collection.internal.AbstractPersistentCollection.readElementByIndex(AbstractPersistentCollection.java:376)
	at org.hibernate.collection.internal.PersistentMap.get(PersistentMap.java:164)
	at com.sixsq.slipstream.statemachine.StateMachine.createNodeExtrinsicState(StateMachine.java:71)
	at com.sixsq.slipstream.statemachine.StateMachine.recreateStateFromRun(StateMachine.java:128)
	at com.sixsq.slipstream.statemachine.StateMachine.<init>(StateMachine.java:64)
	at com.sixsq.slipstream.statemachine.StateMachine.getStateMachine(StateMachine.java:120)
	at com.sixsq.slipstream.statemachine.StateMachine.createStateMachine(StateMachine.java:106)
	at com.sixsq.slipstream.util.Terminator.terminateInsideTransaction(Terminator.java:123)
	at com.sixsq.slipstream.util.Terminator.purgeRun(Terminator.java:109)
	at com.sixsq.slipstream.util.Terminator.purge(Terminator.java:74)
	at slipstream.async.garbage_collector$purge.invokeStatic(garbage_collector.clj:19)
	at slipstream.async.garbage_collector$purge.invoke(garbage_collector.clj:19)
	at slipstream.async.garbage_collector$collect_BANG_$fn__240$state_machine__7658__auto____243$fn__245.invoke(garbage_collector.clj:36)
	at slipstream.async.garbage_collector$collect_BANG_$fn__240$state_machine__7658__auto____243.invoke(garbage_collector.clj:36)
	at clojure.core.async.impl.ioc_macros$run_state_machine.invokeStatic(ioc_macros.clj:973)
	at clojure.core.async.impl.ioc_macros$run_state_machine.invoke(ioc_macros.clj:972)
	at clojure.core.async.impl.ioc_macros$run_state_machine_wrapped.invokeStatic(ioc_macros.clj:977)
	at clojure.core.async.impl.ioc_macros$run_state_machine_wrapped.invoke(ioc_macros.clj:975)
	at

slipstream.async.garbage_collector$collect_BANG_$fn__240.invoke(garbage_collector.clj:36)
	at clojure.lang.AFn.run(AFn.java:22)
	at

java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

In reviewing the code, the place where the exception is thrown is logically within a JPA transaction (and hence a hibernate Session).  The EntityManager (and Session) is stored within a TheadLocal variable where it can be retrieved by any JPA code accessing the database. This code, however, is started via core.async, which uses a fixed thread pool and shares tasks (even parts of tasks) between those threads. In this case, what is likely happening is that core.async is starting a task on one thread (where the ThreadLocal variables will be set) and completing the task on another.  If the task was completed on a thread where the EntityManager/Session had not been set, then a stack trace like the one given would be the result. Similarly, it would be possible to use the wrong EntityManager/Session to complete a task, although this would not result in a stack trace.
The first task is to verify this explanation by adding logging to provide the thread identities when the task is started, ended, and at the point of the error.
If this is confirmed, then possible solutions to the problem are:

Managing our own processes/threads in clojure either via futures or threads to ensure that the garbage collection tasks always run on a single thread. (That is, remove the use of core.async for tasks involving access to HSQLDB via JPA.)
Moving these processes to the job framework, although this might bring additional problems with multiple clients accessing HSQLDB.


Resolver integration

Integrate resolver with the API service
This PR integrates the name resolution to the API service. All endpoints that receive an {addr} parameter will expect either a bzz address or a name. During the execution of the request, if a name has been passed, the api service will try to resolve it into a hash. If the resolution is successful, execution proceeds as normal.
Acceptance criteria
Start the bee node with the following resolver:
bee start -resolver-options "eth:https://cloudflare-eth.com" --verbosity debug

Log should output:
...
DEBU[...] name resolver: resolver for ".eth": connecting to endpoint https://cloudflare-eth.com
INFO[...] name resolver: resolver for ".eth" domain: connected to https://cloudflare-eth.com
...

Resolving
localhost:8080\files\latest.bookofswarm.eth

Should yield the following messages in the log:
INFO[...] name resolve: resolved name latest.bookofswarm.eth to a983eae4225c19068f6b795ffaa3fe7b23170842027d908a7b11c6fc36247f01

It would be good to have a default chain of resolvers as a fallback, similar as we have in swarm. So that for example --resolver-options "https://cloudflare-eth.com" would work. Currently it is not possible to have such resolver configuration.

@janos, @zelig, @acud - thank you for your comments.
I've opened separate issues to resolve some of your comments outside the scope of the release: #637 #638

This should fix #638


Code indents not displaying properly

Gitea version (or commit ref): 1.13.0
Git version: 2.20.1
Operating system: Debian
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No

Description
I am running gitea on a Raspberry Pi 4 with Debian via a binary. Everything seems to work perfectly fine, however when viewing code in a repository, any sort of indentation seems to be broken. The first layer of indentation always appears as 1 space, and additional levels do not appear at all. When viewing the code in raw form however, all indents display properly.
I am having this issue with both tabs and spaces, as well as both mirrors and non-mirrors. I have also tried adding a .editorconfig in the repositories, however that did not work.
Screenshots
Code: https://i.imgur.com/xPq7Hg1.png
Raw code: https://i.imgur.com/m6SRYaz.png

Edit: I solved the issue, just had to turn off minifying HTML with cloudflare


GitHub Actions when forked - Quick test before sleep

PR o'clock
Description
Quick testing of some GitHub actions before sleep. I really want to see if this works and I apologize for the spam.
Checklist

[ ] terraform fmt and terraform validate both work from the root and examples/* directories
[ ] CI tests are passing
[ ] I've added my change to CHANGELOG.md and highlighted any breaking changes
[ ] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

😞 of course forked repos behave differently. This was working in non-forked repos.
I'll create some repos and test this without spamming you all. Apologies.


Add line tools for edition and visualization on Jupyter

This is not clear what is the best Jupyter plugin we should use yet but here is an example of one that we could start working with: https://github.com/InsightSoftwareConsortium/itk-jupyter-widgets/issues/89

Ideally, I want something to the ROIManager in ImageJ.


Add a tutorial to the Constellation activity

Add a tutorial to the Constellation activity following tutorial 9 in Sugarizer development tutorial: https://github.com/llaske/sugarizer/blob/dev/docs/tutorial_step9.md
The tutorial:

[ ] Should display a welcome message to explain the activity (could reuse the help message describing the activity in Sugarizer activities List view)
[ ] Should explain usage of each button in the toolbar
[ ] Should be localized
[ ] Should respect the Sugarizer UI, as explained in tutorial

Fixed in https://github.com/llaske/sugarizer/pull/496


TimesNet  for imputation ERROR Exception: The size of tensor a (5461) must match the size of tensor b (1000) at non-singleton dimension 1

Issue description
The TimesNet method was used to recover the missing value of long time series observation meteorological data (11 variables).
The error is as follows:
2024-06-07 12:58:25 [INFO]: No given device, using default device: cpu
2024-06-07 12:58:25 [INFO]: Model files will be saved to imputation/timesnet\20240607_T125825
2024-06-07 12:58:25 [INFO]: Tensorboard file will be saved to imputation/timesnet\20240607_T125825\tensorboard
2024-06-07 12:58:25 [INFO]: TimesNet initialized with the given hyperparameters, the number of trainable parameters: 5,414,155
2024-06-07 12:58:25 [ERROR]: ❌ Exception: The size of tensor a (5461) must match the size of tensor b (1000) at non-singleton dimension 1
The length of the dataset is 5461 rows of records.

This issue is caused by the overlong length that 5461 is larger than 1000, which is the default sample max length in TimesNet. I will make a PR to make the max length adaptive.


LanguageServerProtocolJSONRPC: make Windows path work

This adjusts the use of Dispatch to build on Windows.  Windows does not
provide stdout_fileno and stderr_fileno.  However, it is possible to
use fileno to get the associated fileno from the descriptor.
Dispatch on Windows does not deal with fd's but rather with handles.
Convert the file descriptor to a handle and pass that off to dispatch.
The handle is a non-owning reference, and should not be closed.
Fortunately, dispatch does not close the handle when the DispatchIO is
closed.

@benlangmuir - hmm, is CDispatch not available on Darwin and Dispatch just imports the C interfaces as well?

I believe so - you could wrap with hasImport(CDispatch)

@swift-ci please test

@compnerd it looks like dispatch_fd_t is explicitly marked unavailable in Swift on Darwin.  Using Int32 on non-Windows should be fine.

Thanks @benlangmuir!  I'll update this.

@swift-ci please test

@benlangmuir - would you mind merging the change?


Cannot locate SRV record in docker container

I'm trying to authorize httpclient using following code:
var kerbCred = new<EMAIL_ADDRESS>$"{config.Password}");
await client.Authenticate(kerbCred);
var ticket = await client.GetServiceTicket($"http/{config.Host}");
var header = Convert.ToBase64String(ticket.EncodeGssApi().ToArray());
httpclient.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Negotiate", header);

On windows works perfect, but in docker container on same machine it doesn't work:
System.AggregateException: One or more errors occurred. (Cannot locate SRV record for DOMAIN) (KDC KRB_ERR_RESPONSE_TOO_BIG: Response too big for UDP; retry with TCP) (Cannot locate a KDC Proxy endpoint for DOMAIN)
 ---> Kerberos.NET.Transport.KerberosTransportException: Cannot locate SRV record for DOMAIN
   at Kerberos.NET.Transport.KerberosTransportBase.SelectedPreferredInstance(String domain, String servicePrefix, IEnumerable1 results, Int32 defaultPort) in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportBase.cs:line 187
   at Kerberos.NET.Transport.KerberosTransportBase.LocatePreferredKdc(String domain, String servicePrefix) in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportBase.cs:line 168
   at Kerberos.NET.Transport.TcpKerberosTransport.GetClient(Func1 locatePreferredServer) in D:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 99
   at Kerberos.NET.Transport.TcpKerberosTransport.SendMessageTCP(String domain, ReadOnlyMemory1 encoded, CancellationToken cancellation, Func1 locatePreferredServer) in D:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 75
   at Kerberos.NET.Transport.TcpKerberosTransport.SendMessage(String domain, ReadOnlyMemory1 encoded, CancellationToken cancellation) in D:\a\1\s\Kerberos.NET\Client\Transport\TcpKerberosTransport.cs:line 54
   at Kerberos.NET.Transport.KerberosTransportSelector.<>c__DisplayClass7_0.<<SendMessage>b__0>d.MoveNext() in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportSelector.cs:line 76
--- End of stack trace from previous location ---
   at Kerberos.NET.Transport.KerberosTransportSelector.SendMessageOnTransport(String domain, Func2 cbSend) in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportSelector.cs:line 126
   --- End of inner exception stack trace ---
   at Kerberos.NET.Transport.KerberosTransportSelector.SendMessageOnTransport(String domain, Func2 cbSend) in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportSelector.cs:line 145
   at Kerberos.NET.Transport.KerberosTransportSelector.SendMessage(String domain, ReadOnlyMemory1 encoded, CancellationToken cancellation) in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportSelector.cs:line 72
   at Kerberos.NET.Transport.KerberosTransportBase.SendMessage[T](String domain, ReadOnlyMemory1 req, CancellationToken cancellation) in D:\a\1\s\Kerberos.NET\Client\Transport\KerberosTransportBase.cs:line 112
   at Kerberos.NET.Client.KerberosClient.RequestTgt(KerberosCredential credential, KrbPrincipalName tgtServicePrincipal) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 1325
   at Kerberos.NET.Client.KerberosClient.AuthenticateCredential(KerberosCredential credential, String tgtServiceName) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 384
   at Kerberos.NET.Client.KerberosClient.AuthenticateCore(KerberosCredential credential, String tgtServiceName) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 365
   at Kerberos.NET.Client.KerberosClient.Authenticate(KerberosCredential credential) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 322

I tried nslookup on both environments, windows:
╤хЁтхЁ:  server.internal
Address:  10.129.0.X

_kerberos._udp.DOMAIN  SRV service location:
          priority       = 0
          weight         = 100
          port           = 88
          svr hostname   = adhost.DOMAIN
adhost.DOMAIN internet address = 10.129.0.X

on docker container:
nslookup -type=SRV _kerberos._udp.DOMAIN
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53

Non-authoritative answer:
_kerberos._udp.DOMAIN  service = 0 100 88 adhost.DOMAIN.

So seems that dns works fine.
Expected behavior
All works fine
What am I doing wrong?

Weird thing. If I nslookup domain only I will get different answers from docker and windows. Windows:
`nslookup -type=SRV DOMAIN
╤хЁтхЁ: server.internal
Address:  10.129.0.X
DOMAIN
primary name server = adhost.DOMAIN
responsible mail addr = hostmaster.DOMAIN
serial  = 2564
refresh = 900 (15 mins)
retry   = 600 (10 mins)
expire  = 86400 (1 day)
default TTL = 3600 (1 hour)But docker seems can't find it:nslookup -type=SRV DOMAIN
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53
Non-authoritative answer:
*** Can't find DOMAIN: No answer
Authoritative answers can be found from:
DOMIAN
origin = adhost.DOMAIN
mail addr = hostmaster.DOMAIN
serial = 2564
refresh = 900
retry = 600
expire = 86400
minimum = 3600`
May be it's relevant

Hi!
This is a duplicate of https://github.com/dotnet/Kerberos.NET/issues/185
The solution is to use DnsQuerry in your code.

Wow! Thank you so much it worked!
May be it could be a part of main library?

Wow! Thank you so much it worked! May be it could be a part of main library?

You're welcome.
I think the nugget author was working on it, I don't really know if it's been completed.

The goal is to keep 3rd party dependencies to a minimum. There aren't many Linux users of this library so there hasn't been a big demand, though that is changing. Another contributer started work on native DNS support but it hasn't gone anywhere recently. I suspect it was a bigger job than they thought it would be. https://github.com/dotnet/Kerberos.NET/pull/334

The goal is to keep 3rd party dependencies to a minimum. There aren't many Linux users of this library so there hasn't been a big demand, though that is changing. Another contributer started work on native DNS support but it hasn't gone anywhere recently. I suspect it was a bigger job than they thought it would be. #334

I thought there are many users of docker. Sound reasonable about 3d party. Thanks.

I'm just basing that on cases I've debugged. Could be higher. There's no telemetry to say one way or another. Submissions to finish the PR are of course always welcome.


[HybridApp] Recognizing which app is being run

Implement the "Recognizing which app is being run" section of the design doc
cc @staszekscp @mczernek @WoLewicki - Please comment if you are going to work on this one and I will assign you 😄
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0176f29a1c850e2630
Upwork Job ID:<PHONE_NUMBER>896237056
Last Price Increase: 2023-11-17

Working on this soon ™

The NewDot PR is already on the repo

@AndrewGable now that the above PR is on prod (as of 5 days ago) is anything left here?

C+ review payment is pending

@dylanexpensify - Can you pay out @situchan the standard here? Thanks

Yes! @situchan sent invite to apply!

Bump @situchan!

@situchan bump 🙇♂

@situchan sure can!

@situchan here you go!

Applied job

offer sent @situchan!

Not received yet.
@AndrewGable can you please reopen?

reviewing

Paid!