Support for 3D Conv-Net

Hi all,
Thank you for developing such a nice repo. I've been using it in many of my projects for network explainability, and it has been incredibly convenient!
Recently, I've been working with medical datasets using 3D-UNet. However, I noticed that 3D convolution is not yet supported in this library, and there are also some issues like #351 requesting for the feature. Therefore, I made several changes on GradCAM and BaseCAM to extend the functionality of GradCAM to support 3D images.
Please let me know if you have any questions or suggestions regarding the changes I've implemented. I'm excited to contribute to this project and look forward to your feedback!

Hey, sorry for the late reply.
Thanks a lot for this functionality, this will be great to merge.
Is there a way to share an example use case for this: maybe some model and and input image example,
or an image example for the readme?

@jacobgil  Thanks for your reply!

Is there a way to share an example use case for this: maybe some model and and input image example, or an image example for the readme?

I added an animation of gradcam-visualized CT scans in the readme.
Hope this can make it clearer.

@kevinkevin556 Thanks for providing the code for applying Grad-Cam on 3D CNN!
I have used your code to get the grad-cam outputs, my input 3D image size is (1, 1, 24, 224, 224) as (batch, channel, depth, height, width).
Then I got the output size is (1, 224, 224, 7).
I don't know why it is 7, shouldn't it be 1?
Looking forward to your replying, thanks!

@kevinkevin556 Thanks for providing the code for applying Grad-Cam on 3D CNN!
I have used your code to get the grad-cam outputs, my input 3D image tensor size is (1, 1, 24, 224, 224) representing (batch, channel, depth, height, width).
Then I got the grayscale_cam outputs size is (1, 24, 224, 224).
I'm curious to know if I take one of the outputs, for example, depth=11, the output will be "outputs[ 0, : ][ 11, : , : ] (depth, height, width)", will it corresponds to "input image[ : , 11 , : , : ]  (channel, depth, height, width)" ?
Since I found that every depth of the output heatmap looked same.
Looking forward to your replying, thanks!

I have used your code to get the grad-cam outputs, my input 3D image tensor size is (1, 1, 24, 224, 224) representing (batch, channel, depth, height, width). Then I got the grayscale_cam outputs size is (1, 24, 224, 224). I'm curious to know if I take one of the outputs, for example, depth=11, the output will be "outputs[ 0, : ][ 11, : , : ] (depth, height, width)", will it corresponds to "input image[ : , 11 , : , : ] (channel, depth, height, width)" ? Since I found that every depth of the output heatmap looked same.

@Syax19 Sorry for the late reply. I'm glad to hear that someone is using it 😄
Although I followed MONAI's convention to assign each dimension in the order of (height, width, depth), the output dimensions should still correspond with your input tensor, as there is no dimension swap when calculating Grad-CAM.
Therefore, the grayscale_cam of size (1, 24, 224, 224) represents dimensions (batch, depth, height, width) in your case.

@jacobgil
Any update on this feature?

This is incredible functionality, thank you so much for contributing this, and sorry for being so late with my reply.
I really want to merge this.
The .gif file weights 24 mb which is a bit much, will look into resizing it.

@kevinkevin556 merged!! better late than never. Thank you so much for this contribution!


add logic to detect public repo with private GHE, issue #2135

There is a case that I don't handle:
if a public GHE with a public repo POST to kudu
kudu will try to use https, however it is possible for the public GHE to have a self signed certificate (which is the case for my test personal GHE) the default behavior of git is

fatal: unable to access https://<IP_ADDRESS>/GHErepo/ SSL certificate problem: self signed certificate

you can disable certificate check with a git command, but I don't know if we should commit to that

Yeah, let's not worry about the self-signed cert case. I think in a real deployment companies would never use that as it's bad security practice. If it ever comes up, we'll tackle.

tested with github, gitlab and codebase.


Python library loss after update python version.

After update python version, the original library installed by pip is located in the old version folder. It can not be used directly through command line.

just reinstall them by pip install ?

One option is to make use of some virtualenv/venv

I don't understand why the Scoop mechanism for persisting data is not used. Is the package maintainer aware of this mechanism?

I don't understand why the Scoop mechanism for persisting data is not used. Is the package maintainer aware of this mechanism?

After upgrading python, there is no guarantee that the previously installed libraries are still compatible. It is best to reinstall them.

A minor upgrade - say from 3.8.0 to 3.8.1 - is always compatible. Only a major upgrade - say from 3.7 to 3.8 - will affect libraries which have a binary component (pyd files).
Not persisting site-packages at all is the worst option. If I reinstall Python (scoop update --force) or uninstall the old Python (scoop cleanup python) ALL packages will be gone! How do I reinstall packages that have been deleted without a trace?

A minor upgrade - say from 3.8.0 to 3.8.1 - is always compatible. Only a major upgrade - say from 3.7 to 3.8 - will affect libraries which have a binary component (pyd files).
Not persisting site-packages at all is the worst option. If I reinstall Python (scoop update --force) or uninstall the old Python (scoop cleanup python) ALL packages will be gone! How do I reinstall packages that have been deleted without a trace?

It is not always compatible, even if a minor upgrade, some libraries is not pure python.

maybe persisting a site-packages folder with python version is a good solution, like: persist/python3.9/site-packages

A minor upgrade - say from 3.8.0 to 3.8.1 - is always compatible. Only a major upgrade - say from 3.7 to 3.8 - will affect libraries which have a binary component (pyd files).
Not persisting site-packages at all is the worst option. If I reinstall Python (scoop update --force) or uninstall the old Python (scoop cleanup python) ALL packages will be gone! How do I reinstall packages that have been deleted without a trace?

You should install packages in user site: pip install --user ...


Clarification: Immutable to json in react

I am using the immutable in state but I was wondering if the react is used only to render using the state and since there are no mutations, we can easily get the json from the state at the top level after the reducers and pass so that we dont have to deal with the immutable getter methods. I will still use immutable in reducers but not on react.
Are there any side affects by doing this?

You will lose main benefit of Immutable, cheap shouldComponentUpdate check.
To avoid getters situation in Components you can use Immutable.Record.
Currently there are some quircks, but Immutable.js 2016 Roadmap includes improvements for Records.
Main difference is that instead of using fromJS you need to write your own rehydration/deserialization logic. Example from Este.

if im using the PureRenderMixin this is sorted out i guess. Or is it not?
+1 for the Records. Will check that

This would destroy the perf benefit of Immutable. toJS() is expensive and doing it upfront means too many allocations. Just don't use Immutable if you don't want to deal with it inside the components.


Cookie handling kills hand-rolled cookie headers

Before, we were setting the cookies manually via the header:
request.get({
  url: 'http://someurl.com',
  headers: {
    'Cookie': sessionCookie
  }
},

but since the introduction of the built-in cookie handling in request 2.2+, this has broken.  I now have to do:
var request = require('request').defaults({jar: false})

…in every file that used the headers mechanism.
Perhaps the cookie system shouldn't override the Cookie header if it already exists?

yeah, we shouldn't set the cookie header if it's already set.
should be a simple fix.

this was fixed.

Really? I ran into this issue again (forgetting .defaults({jar: false}) ) just two days ago.

a check for headers.cookie was merged in to master a while back, might not be in a release yet. can you test with master?


Multiple FIPS endpoints msising from botocore/data/endpoints.json

Currently botocore does not list any FIPS validated endpoints for multiple AWS services, including EC2, Route53, and S3. Per the list here, those services (and others) do indeed support access via FIPS validated endpoints.
This breaks some functionality in aws-cli as well when any the following regions are used:

us-east-1-fips
us-east-2-fips
us-west-1-fips
us-west-2-fips

I see from the commit logs that modifications to endpoints.json are committed from an automated system. Does this mean that any pull request I might submit would interfere with that?
Thanks,
@broosa

The endpoints are added via an automated system. We pull our endpoints.json from an internal upstream source so we would need to get the upstream source updated in order to pick these up. So any pull request sent to fix the endpoint.json file would be clobbered by our automated system. I will look into getting this updated from our side.

Also seems applicable to FIPS regions for us-gov (i.e. fips-us-gov-west-1) ref: https://docs.aws.amazon.com/govcloud-us/latest/ug-west/using-govcloud-endpoints.html

Confirmed that FIPS endpoints for these services are now in the endpoints file.


Add history feature

Description of change
Use the dialoguer history feature to enable command history for the account command interface.
Links to any relevant issues
#98
Type of change

Enhancement (a non-breaking change which adds functionality)

How the change has been tested
Manual testing
Change checklist
Tick the boxes that are relevant to your changes, and delete any items that are not.

[ ] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that new and existing unit tests pass locally with my changes

Can we merge this one? Or are there still open questions?

Hello @hauju, we have migrated the CLI wallet to https://github.com/iotaledger/wallet.rs and will archive this repository. Would you mind re-opening this PR there so we can finally get it merged? Sorry for the delay.


feat: add msquic wrapper handles

Pull Request Checklist
Please check if your pull request fulfills the following requirements:

[x] The commit message follows our guidelines: https://gitlab.playpod.ir/alpha/backend/server-services-next/-/blob/main/CONTRIBUTING.md#commit-message-format
[x] Tests for the changes have been added / updated (for feat / fix / refactor)
[x] Docs for the changes have been added / updated (for feat / fix / refactor)

Pull Request Type
What kind of change does this pull request introduce?

[ ] Build: Changes that affect the build system or external dependencies
[ ] CD: Changes to our CD configuration files and scripts
[ ] CI: Changes to our CI configuration files and scripts
[ ] Docs: Documentation only changes
[x] Feat: A new feature
[ ] Fix: A bug fix
[ ] Perf: A code change that improves performance
[ ] Refactor: A code change that neither fixes a bug nor adds a feature
[ ] Test: Adding missing tests or correcting existing tests

What is the current behavior?

Issue #476
What is the new behavior?

added handle wrappers for registration, configuration, listener, connection, and stream.
some minor fixes and refactors

Does this Pull Request introduce a breaking change?

[ ] Yes
[x] No

Other information
Closes #476

@cthulhu-irl thank you


Better implementation of zoom/panning in the next version (0.6.0)

Describe the bug
Hello everyone,
we have recently published the version 0.5.0 of 'beautiful-react-diagrams', this version introduces the new pan/zoom functionality.
Unfortunately, we also introduced a bug that causes to miscalculate the position of nodes with ports and links between them, some nodes are thrown out of the visibile screen and in some other cases it makes the Diagram rather unusable.
We apologise for this inconvenience.
Today we will try to release a new version (presumably the 0.5.1) that will roll back to the previous one, so that the bug will not be on NPM and possibly on other projects
This will also give us more time to work on the 0.6.0.
The 0.6.0 will have a better implementation of the zoom/panning provided by a new brand new component, called Canvas, to wrap the Diagram(s).
This next version will also have a useCanvasState hook to help with the Canvas component states and a component for the zoom controls called CanvasController.
The new API will look like this:
import Diagram, { Canvas, createSchema, useSchema, useCanvasState, CanvasControls } from 'beautiful-react-diagrams';
// the diagram model
const initialSchema = createSchema({
  nodes: [
    { id: 'node-1', content: 'Hey Jude', coordinates: [312, 27], },
    { id: 'node-2', content: 'Don\'t', coordinates: [330, 90], },
    { id: 'node-3', content: 'be afraid', coordinates: [100, 320], },
    { id: 'node-4', content: 'let me down', coordinates: [306, 332], },
    { id: 'node-5', content: 'make it bad', coordinates: [515, 330], },
  ],
  links: [
    { input: 'node-1',  output: 'node-2' },
    { input: 'node-2',  output: 'node-3' },
    { input: 'node-2',  output: 'node-4' },
    { input: 'node-2',  output: 'node-5' },
  ]
});
const DiagramExample = () => {
  const [canvasState, handlers] = useCanvasState(); // creates canvas state
  const [schema, { onChange }] = useSchema(initialSchema); // creates diagrams schema
  return (
    <div style={{ height: '30rem' }}>
      <Canvas {...canvasState} {...handlers}>
        <Diagram schema={schema} onChange={onChange} />
        <CanvasControls />
      </Canvas>
    </div>
  );
};
<DiagramExample />

Furthermore, the next version will solve the following bug:
#69

A pan-zoom was exactly what I needed for the project I'm using this with! So excited to hear it's already been being worked on, and look forward to the release :) . Thank you for the hard work. Thought I might have to figure out how to do it myself hahaha.

@alimirakim
Yes a way better pan and zoom functionality will be release in the next version, the 0.6.0.
Unfortunately we don't have a release date yet but it's on its way.
If you need to have a look on how it look so far, be please pull the code from the feature/canvas-ext branch and feel free to contribute.
About the hard work, I appreciate your comment, thank you... I must say, we worked on this as a company but some important  improvements come from the community and the help of splendid contributors like @9renpoto, @genbs and @joancc

@Lindsay89
Thanks to you and the team for this awesome resource, question.  When would ETA for Zoom be possibly released?
I am approaching a project start and would like to know if I can use this library

Just started playing with the lib for an hour and i just want to say thank you for your work. The DX is 💯 and it's super easy to hit the ground running building Diagrams.
Looking forward to the zoom/pan feature as I believe this is a key feature missing at the moment.
Thanks you for your work and I hope for a successful release 🙏

Thank you!
Really looking forward to the new zoom functionality.

Could you also add handling of diagram node scrolling? E.g. diagram root element has non-zero scrollLeft, scrollTop because of overflow.
At the moment this causes visual glitches (because scrollLeft, scrollTop does not taken into account in getEntityCoordinates.ts.


[Menu] Menu containing links

I am trying to use the Menu component to create a dropdown where some of the items are links (e.g. "My profile", "Log out", etc). The documentation (demo and api) does not mention this use case . I have so far found it impossible to achieve the following:

valid html (no  or
 directly in a
)

expected behavior for links when using a mouse (hover to see the link target, right click to open in new tab, etc.)

expected behavior when using the keyboard (moving around the menu and activating links with the keyboard only)

[x] The issue is present in the latest release.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
The obvious solution (<MenuItem component={Link}>) behaves as expected, but generates invalid html.
Other solutions are broken in different ways.
Expected Behavior 🤔
Have a clear way to include links in a Menu, that behaves as expected with both keyboard and mouse, and generates valid html.
Steps to Reproduce 🕹
Steps:

See code sandbox: https://codesandbox.io/s/basicmenu-material-demo-forked-6w0jp?file=/demo.js

Context 🔦
In the topbar, when the user clicks on their avatar, a menu should pop up containing some links (their profile, settings, logout) as well as buttons. The result should be valid html and behave as expected with both keyboard and mouse.
Your Environment 🌎
See code sandbox.
Tested in Firefox.

The obvious solution () behaves as expected, but generates invalid html.

What is the expected HTML structure that you are looking for? and please you provide the resource for that reference.

Hi,
I would like to be able to achieve what is described here: https://www.w3.org/TR/wai-aria-practices/examples/menu-button/menu-button-links.html.

        W3C Home Page

        W3C Web Accessibility Initiative

The last example in the codesandbox shows that this HTML structure can be achieved, however the keyboard behavior is broken:

when navigating the items, the link is not focused (the browser does not show the url)
pressing Enter does not activate the link

Additionally, I would expect to be able to mix links and button in the same menu (for example, some menu items navigate somewhere, others trigger a dialog).

(For Material-UI v4)
To make keyboard focus with the valid HTML structure, I programmed as below:
<Menu
  MenuListProps={{
    // @ts-expect-error https://github.com/mui-org/material-ui/issues/17579
    component: "nav"
  }}
>
  <MenuItem component="div">
    Plain text
  </MenuItem>

  <MenuItem component={Link}>
    Link
  </MenuItem>
</Menu>

This example code is based on https://v4.mui.com/components/lists/#simple-list


Release 0.1.0

Release checklist

[x] Check outstanding issues on Github.
[x] Create a release branch.

[x] Set version to a stable number.
[x] Change current development version in CHANGELOG.rst to stable version.

[x] Merge the release branch into main.
[x] Create a test pypi package from the main branch. (Instructions.)
[x] Install the packages from the test pypi repository to see if they work.
[x] Created an annotated tag with the stable version number. Include changes
from history.rst.
[x] Push tag to remote.
[x] Push tested packages to pypi.
[x] merge main branch back into develop.
[x] Add updated version number to develop.
[x] Create a new release on github.
[ ] Update the package on conda-forge.

@rhpvorderman this is looking awesome, are we at a good point to publicize this? (link from miniwdl homepage, tweet, send around to HPC folks, etc.)
anything we should coordinate on timing of that? defer to you of course

Hi @mlin, sure go ahead! As long as it is clear in the communication that this is the first beta (still on 0.1.0) I see no harm in spreading the word around. I already have notified some of my colleagues in another institute in the Netherlands that I know that currently use Cromwell on SLURM.
I think a link from the miniwdl homepage would be great. Next to an AWS link. I do think that the byline for miniwdl:

Workflow Description Language local runner & developer toolkit for Python 3.6+

will need some changes to the "local" part once these extensions mature :wink:.


Using breadcrumbs with route caching is not possible

It works fine without route caching. But nothing happens after caching it.

Hi @happy-soft-solutions
I have been working on a new version to fix that.
Tomorrow I'll release the v2 with a new API and the ability to cache routes.
Thanks for reporting


Update Data.js

Alphabetically organized the data and removed duplicate entries.

@Spyware007 Kindly give level rating tag and merge the PR if the proposal seems to be fine.

I already did this a while ago!

Okay someone has messed up with it!


getIdentityClaims() returns null while I'm using fetchTokenUsingPasswordFlowAndLoadUserProfile

I'm using an OAuth 2.0 server for user authorization and authentication matters
I want to get user profile information before the page is loaded (cause I want to know what theme is she\he using!?) So I tried using fetchTokenUsingPasswordFlowAndLoadUserProfile method to get user's profile, right after getting token. The problem is that, when I call getIdentityClaims method, It returns null while there's a log in the back-end, showing that get profile API has been called and it was successful! (with 200 status code!)
So, it's how I've coded all of these:
AppComponent:
@Component({
  selector: "app-root",
  template: "<router-outlet></router-outlet>",
})
export class AppComponent {
  constructor(private oauthService: OAuthService) {
    this.configure();
  }

  private configure() {
    this.oauthService.redirectUri = window.location.origin;

    this.oauthService.logoutUrl = window.location.origin;

    // ApiUrl is Server base URL
    this.oauthService.tokenEndpoint = AppSettings.ApiUrl + "auth/token";

    // It's the URL which returns user profile
    this.oauthService.userinfoEndpoint = AppSettings.ApiUrl + "user-profile/";

    this.oauthService.clientId = AppSettings.oauthClientId;

    this.oauthService.scope = "read write";

    this.oauthService.setStorage(localStorage);
  }
}

sign in service:
@Injectable()
export class SignInService extends ServiceBase {
  constructor(private _http: HttpClient, private oauthService: OAuthService) {
    super(_http);
  }

  protected handleError(error: any): Observable<any> {
    return error.error;
  }

  getToken(data: SignInModel) {
    debugger;
    return this.oauthService
      .fetchTokenUsingPasswordFlowAndLoadUserProfile(
        data.userName,
        data.password
      )
      .catch(this.handleError);
  }
}

And in the login component, there's a function which will be ran when the user hit the login button:
userLogin() {
    if (this.signInModel.userName && this.signInModel.password) {
      if (this.form.invalid == false) {
        this.showSpinner = true;

        this.signInService.getToken(this.signInModel)
        .then(() => {
          let claims = this.oauthService.getIdentityClaims(); // <-- this is where I want to get profile information, but it's always null!

          this.showSpinner = false;
          this.router.navigate(['/']);
        });
      }
    }
  }

Any thoughts?

have you tried setting this.oauthService.showDebugInformation: true and see what's happening?

In addition to the above suggestion, please also try adding this as soon as possible:
this.authService.events.subscribe(e => (e instanceof OAuthErrorEvent) ? console.error(e) : console.warn(e));

and tell us what's going on in your app.
Otherwise, try to open the devtools debugger, find the library's source, and step through the loadUserProfile logic in your instance.

have you tried setting this.oauthService.showDebugInformation: true and see what's happening?

It doesn't show more information!

In addition to the above suggestion, please also try adding this as soon as possible:
this.authService.events.subscribe(e => (e instanceof OAuthErrorEvent) ? console.error(e) : console.warn(e));

and tell us what's going on in your app.
Otherwise, try to open the devtools debugger, find the library's source, and step through the loadUserProfile logic in your instance.

This is what I get in browser console:
> OAuthSuccessEvent {type: "token_received", info: null}

And this is its log: console.log
In addition: I checked my browser's network and it showed me that user-profile/ request was successful and it returns all the user information I want.

any solution for this?
I am facing the same situation with angular -v 9 and  angular-oauth2-oidc -v 10.
My user is logged in (session is open in keycloak) but getting null token from  oauthService.getAccessToken()

Don't think many people use (this lib with) Password flow, so I reckon the community will rely on someone running into this actual issue to debug the problem in the library (or app) code....


WIP: test pr to see if it fixes the failures

testing this before filing upstream

/retest

/retest


Change Map Marker Based on Zoom Level

hey!
This is more of a question than a problem - just wondered if it's currently possible.
Is there current functionality to get the zoom level so we can change the map marker style when you zoom on the map?
Our use case is that we want to change the scale of the .Marker when you zoom out, instead of clustering.
Any help welcome!
/blam

It called 'Image Overlay' or 'Ground Overlay' on Google Maps which react-native-maps currently doesn't have. But one guy did it for iOS and I used with AirGoogleMaps on iOS, it works successfully. Now I'm researching to do same on Android, if you find, call us here please.
I'm putting it here for iOS (Don't forget to use on AirGoogleMaps, Mapkit doesn't work correctly);
#1359

Hello!
In our application, we have solved a similar problem by render different markers depending on the latitudeDelta of the map region.
function renderCustomMarker(point, region) {
  if (region.latitudeDelta > MIN_DELTA_FOR_CIRCLE_MARKER) {
    renderCircle(point);
  } else {
    renderPinMarker(point);
  }
}

This should be quite easy to modify to set the height and width on a marker depending on the delta...

Aha! I forgot about the delta! That will do! Thanks @pellepersson

Hi @benjdlambert  , How to you fix the zoom level in different region the map marker?
How to set the zoom level in multiple map markers and different regions ? Help @benjdlambert and @pellepersson

@Sathiyamm - I am afraid that I do not understand the question? The solution I posted above shows how to call different render methods for different latitude deltas on the map. The latitude delta (distance between to different latitude values) is one way of expressing zoom level". Is this what you are asking for or do you want something else?

@pellepersson -I do not understand the your function. I have multiple location marker in different region. So how to i set the zoom level?

@Sathiyamm  - still don't understand what you are after. Do you want to change the looks of your markers depending on the zoom level of your map or is it something else you want to do?

@pellepersson -yes i want to change the looks of my markers depending on the zoom level of my map.

@pellepersson -Are you help me ?
I'm set the region value
this.setState({
region:{
latitude: responseData[0].originLatitude,
longitude: responseData[0].originLongitude,
latitudeDelta: 0.0922,
longitudeDelta: 0.0421
}
});
and map view

                            }

                        > </MapView.Marker></MapView>

@Sathiyamm - alright. I think that what you need to do is to pass your own marker as a child to <MapView.Marker>:
<MapView.Marker coordinate={marker.latlng}>
  <CustomMarker {...marker} />
</MapView.Marker>

and then based on a condition on your region.latitudeDelta in your CustomMarker render the different layouts for the marker... A smaller latitudeDelta means that your map is more zoomed in... Hope this helps...

Hi @benjdlambert
I have faced the same requirement as you. I found a solution like:
<MapView ref={ref => { this.storeMap = ref; }}
intialRegion={this.state.region}
zoomControlEnabled={true}
onRegionChangeComplete={(region)=>this.onRegionChangeComplete(region)}>
onRegionChangeComplete=(region)=>{
const northBoundary = region.latitude + region.latitudeDelta/2
const eastBoundary = region.longitude + region.longitudeDelta/2
const southBoundary = region.latitude - region.latitudeDelta/2
const westBoundary = region.longitude - region.longitudeDelta/2
const filteredStore = filter(this.state.storeLatLong, (store) => {
return inRange(store.latitude, southBoundary, northBoundary) && inRange(store.longitude, eastBoundary, westBoundary)
})
console.log(filteredStore)
}

filter and inRange are functions of lodash library.
this.state.storeLatLong is my location array


FR - Node tracking

First of all, thank you for your great work. I'm a thankful every day user of the map.
Back in the day's I've looked at APRS. Some of the bigger APRS sites (https://aprs.fi) do offer tracking of a station automatically. This way you can backtrack where the node has been. It will plot its route by dots and colored lines.
Perhaps this could be a useful option to have on your map?
I do have some requests:

For privacy reasons only look back for 1, 2 or 3 hours?
Add info to the plotted dots who heard that node on that location. It will help to identify coverage issues.

Thanks for the feature request and for the feedback :)
I merged in #54 that implemented this feature the other week, and it caused quite the controversy in the Meshtastic discord...
Anyway, I'll close this out as being completed.


Add the ability to specify an alternate PyPi repository URL

Resolves #9.
Unit test coverage is not expanded, but unit tests do continue to pass as-is.

Nice one James - this is a great new feature!

You're welcome; thanks @AlexIoannides!


agent: get content of a file in sandbox

Description of problem
In https://github.com/kata-containers/runtime/pull/624, we have such problems:

Hot add memory should be aligned to memory section size, but the size of a memory section is architecture dependent. For example, power uses 16MB, ia64 uses 1GB, x86_64 uses 128MB, ppc64le uses 256MB.
Each memory block is described under /sys/devices/system/memory as /sys/devices/system/memory/memoryXXX (XXX is the memory block id.), which is the unit of memory online/offline. In sparse memory model, memory block size is a multiple of memory section size, reference. We can online a memory block only if we hotplug a memory-block-sized dimm, see pages_correctly_reserved in reference.

So we should get the size of guestos memory block by reading /sys/devices/system/memoryblock_size_bytes in sandbox, and make hot add memory to be aligned to memory block size of sandbox.
I think it's needed to add a ReadSandboxFile rpc to read file above in a common way.

As mentioned on https://github.com/kata-containers/runtime/pull/624, the ideal would be that the agent handles all the memory boundary logic rather than having to add a new gRPC call so the runtime can do the calculations.
However, if we really need an agent call, how about we abuse the CheckRequest() call?

https://github.com/kata-containers/agent/blob/master/protocols/grpc/health.proto

The runtime currently makes that call before doing anything else:

https://github.com/kata-containers/runtime/blob/master/virtcontainers/kata_agent.go#L530
https://github.com/kata-containers/runtime/blob/master/virtcontainers/kata_agent.go#L1270

... so we could update the HealthCheckResponse to also include the block_size_bytes value? It's a bit of a hack maybe but it:

avoids adding a new gRPC call.
avoids allowing arbitrary files to be accessed inside the sandbox (I'm not convinced we need that).

@jodh-intel Yep, I will be busy in the following few days, @linzichang will handle this issue instead of me.


Early data problem with s_client

Hi,
I've been testing early data with s_server/s_client. On the server side I see:
ACCEPT
Early data received:
test

End of early data
-----BEGIN SSL SESSION PARAMETERS-----
MGMCAQECAgMEBAITAgQABDCPd6mZMgfXaVPk970I4uJX/8Th9EHKvsAg7LfudQIz
WNXdqxZ+FmAO7iUK4OD6XFKhBgIEWMwnnqIEAgIcIKQGBAQBAAAArgYCBAQDAgGv
BAICQAA=
-----END SSL SESSION PARAMETERS-----
Supported Elliptic Groups: X25519:P-256:P-521:P-384
Shared Elliptic groups: X25519:P-256:P-521:P-384
CIPHER is TLS13-AES-256-GCM-SHA384
Reused session-id
Secure Renegotiation IS supported

On the client side I see:
New, TLSv1.3, Cipher is TLS13-AES-256-GCM-SHA384
Server public key is 2048 bit
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
Early data was rejected
SSL-Session:
    Protocol  : TLSv1.3
    Cipher    : TLS13-AES-256-GCM-SHA384
    Session-ID: F551DD7821EBED38730B3953FC22E6C74C71186D52E8337DFFFE4D408FA4B33E
    Session-ID-ctx:
    Master-Key: BA06DBD3A9E188670335F14A851228EB81777EEA92BA2AFB04C11ADA5A7FD1F200E2417477157C7045C5EC2E8E92793F
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    TLS session ticket lifetime hint: 7200 (seconds)
    TLS session ticket:
    0000 - 01 02 03 04 05 06 07 08-09 0a 0b 0c 0d 0e 0f 10   ................
    0010 - 01 02 03 04 05 06 07 08-09 0a 0b 0c 0d 0e 0f 10   ................
    0020 - 2f af f9 73 2b f0 cb 67-b9 94 c8 22 83 0f 56 09   /..s+..g..."..V.
    0030 - e3 59 49 df e0 af dc b5-5b 2c b2 49 6a c4 42 27   .YI.....[,.Ij.B'
    0040 - 8a 3b 29 e4 c0 02 4b d2-dc a4 ec 79 fb a8 d8 74   .;)...K....y...t
    0050 - ba fe 18 86 f6 33 f6 d9-dc cf 7c 90 d4 c6 19 13   .....3....|.....
    0060 - 84 f2 91 6b c7 de 52 dc-df b6 d2 c3 a2 a0 8b 07   ...k..R.........
    0070 - 8c 46 9f 1e 6b af d5 17-04 0c fa 3b d4 12 3a a7   .F..k......;..:.
    0080 - cc 33 02 51 22 7a 83 e4-66 ce 47 27 d5 d4 96 ca   .3.Q"z..f.G'....
    0090 - 27 30 36 7e 60 c3 1d 91-d2 e6 dd 53 50 68 54 be   '06~`......SPhT.
    00a0 - 79 2f d4 70 f8 bc 38 44-d9 30 f5 bf 9f 39 e5 65   y/.p..8D.0...9.e

    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 21 (unable to verify the first certificate)
    Extended master secret: no

I'm confused that the server shows it received the early data while the client says that it was rejected.

I am seeing something similar; testing against https://tls13.crypto.mozilla.org/ the server is clearly receiving my early data (which contains a basic GET), since my regular data is blank and it returns a page to me, which it does not do when I just feed it blank data.  But I still see the "Early data was rejected" message from s_client.  So, maybe it's just a reporting issue in s_client...

Well, I guess the behavior I observe would also be consistent with s_client silently replaying the 0-RTT data as 1-RTT data after the server rejects early data, but I really hope we're not doing that...

Yeah, this is just a reporting issue.  Running in gdb, we print out "Early data was rejected" before s_client even gets to tls_parse_stoc_early_data() (which sets it to SSL_EARLY_DATA_ACCEPTED), so the print statement is just printing the value it was initialized with when we sent the early data.

Yes, I spotted this the other day...and then forgot all about it again! Sorry. Its an s_client bug. Basically s_client gets a bit confused and thinks its finished connecting, and prints out the connection summary information too early (before we know that the early_data has been accepted).

This got fixed by commit 12557a344.


Failure to install Visual Studio workloads with VSComponents step

Brief description of your issue
Cannot install Visual Studio 2022 Community workloads using the Microsoft.VisualStudio.DSC/VSComponents resource, either via winget configure or DevHome. Tested on two machines plus Windows Sandbox.
Fails with:
Apply :: VSComponents
  The configuration unit was not in the module as expected.

Tried installing the modules separately with Install-Module -Name Microsoft.VisualStudio.DSC -AllowPrerelease but it didn't help.
Steps to reproduce
Using the following configuration file with winget configure .\configuration.dsc.yaml --verbose-logs :
# yaml-language-server: $schema=https://aka.ms/configuration-dsc-schema/0.2
# Reference: https://github.com/microsoft/terminal#prerequisites
properties:
  assertions:
    - resource: Microsoft.Windows.Developer/OsVersion
      directives:
        description: Verify min OS version requirement
        allowPrerelease: true
      settings:
        MinVersion: '10.0.22000'
  resources:
    - resource: Microsoft.Windows.Developer/DeveloperMode
      id: DeveloperMode
      directives:
        description: Enable Developer Mode
        allowPrerelease: true
      settings:
        Ensure: Present
    - resource: Microsoft.WinGet.DSC/WinGetPackage
      id: vsPackage
      directives:
        description: Install Visual Studio 2022 (any edition is OK)
        allowPrerelease: true
      settings:
        id: Microsoft.VisualStudio.2022.Community
        source: winget
    - resource: Microsoft.VisualStudio.DSC/VSComponents
      dependsOn:
        - vsPackage
      directives:
        description: Install required VS workloads
        allowPrerelease: true
      settings:
        productId: Microsoft.VisualStudio.Product.Community
        channelId: VisualStudio.17.Release
        components:
          - Microsoft.VisualStudio.Workload.ManagedDesktop
          - Microsoft.VisualStudio.Workload.Universal
  configurationVersion: 0.2.0

This results in:
Assert :: OsVersion
  Configuration successfully applied.
Apply :: DeveloperMode [DeveloperMode]
  Configuration successfully applied.
Apply :: WinGetPackage [vsPackage]
  Configuration successfully applied.
Apply :: VSComponents
  The configuration unit was not in the module as expected.
Some of the configuration was not applied successfully.

Expected behavior
Visual Studio to install the required components. It also doesn't work with specifying a .vsconfig file.
Actual behavior
The Visual Studio installer installs, but none of the components install.
Environment
Windows Package Manager v1.6.3133
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.1702
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.3133.0

Winget Directories
-----------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diag…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\sett…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Was the configuration run in user mode or administrator? I believe the resource requires administrator.

Was the configuration run in user mode or administrator? I believe the resource requires administrator.

I ran it from both an Administrator Powershell and with the run-as-admin option from Dev Home

Does (Get-DscResource VSComponents).Properties respond with the properties in PowerShell 7?

Does (Get-DscResource VSComponents).Properties respond with the properties in PowerShell 7?

PS C:\Users\WDAGUtilityAccount\Desktop\.configurations> (Get-DscResource VSComponents).Properties
Get-DscResource: The 'Get-DscResource' command was found in the module 'PSDesiredStateConfiguration', but the module could not be loaded. For more information, run 'Import-Module PSDesiredStateConfiguration'.
PS C:\Users\WDAGUtilityAccount\Desktop\.configurations> Import-Module PSDesiredStateConfiguration
Import-Module: Failure from remote command: Import-Module -Name 'C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psd1': Could not load file or assembly 'file:///C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\Microsoft.Windows.DSC.CoreConfProviders' or one of its dependencies. The system cannot find the file specified.

I think the latest version published on the Powershell Gallery (https://www.powershellgallery.com/packages/Microsoft.VisualStudio.DSC/1.0.18-beta) may be broken. I just did a Install-Module -Name Microsoft.VisualStudio.DSC -RequiredVersion 1.0.15-beta -AllowPrerelease and then we get:
PS C:\Users\darob> (Get-DscResource VSComponents).Properties

Name                 PropertyType   IsMandatory Values
----                 ------------   ----------- ------
channelId            [string]              True {}
productId            [string]              True {}
components           [string[]]           False {}
DependsOn            [string[]]           False {}
includeOptional      [bool]               False {}
includeRecommended   [bool]               False {}
PsDscRunAsCredential [PSCredential]       False {}
vsConfigFile         [string]             False {}

Did you file a bug for them at VisualStudioDSC

Brief description of your issue
Cannot install Visual Studio 2022 Community workloads using the Microsoft.VisualStudio.DSC/VSComponents resource, either via winget configure or DevHome. Tested on two machines plus Windows Sandbox.
Fails with:
Apply :: VSComponents
  The configuration unit was not in the module as expected.

Copy And Save

Share

Ask Copilot
Tried installing the modules separately with Install-Module -Name Microsoft.VisualStudio.DSC -AllowPrerelease but it didn't help.
Steps to reproduce
Using the following configuration file with winget configure .\configuration.dsc.yaml --verbose-logs :
yaml-language-server: $schema=https://aka.ms/configuration-dsc-schema/0.2
Reference: https://github.com/microsoft/terminal#prerequisites
properties:
assertions:
- resource: Microsoft.Windows.Developer/OsVersion
directives:
description: Verify min OS version requirement
allowPrerelease: true
settings:
MinVersion: '10.0.22000'
resources:
- resource: Microsoft.Windows.Developer/DeveloperMode
id: DeveloperMode
directives:
description: Enable Developer Mode
allowPrerelease: true
settings:
Ensure: Present
- resource: Microsoft.WinGet.DSC/WinGetPackage
id: vsPackage
directives:
description: Install Visual Studio 2022 (any edition is OK)
allowPrerelease: true
settings:
id: Microsoft.VisualStudio.2022.Community
source: winget
- resource: Microsoft.VisualStudio.DSC/VSComponents
dependsOn:
- vsPackage
directives:
description: Install required VS workloads
allowPrerelease: true
settings:
productId: Microsoft.VisualStudio.Product.Community
channelId: VisualStudio.17.Release
components:
- Microsoft.VisualStudio.Workload.ManagedDesktop
- Microsoft.VisualStudio.Workload.Universal
configurationVersion: 0.2.0

Copy And Save

Share

Ask Copilot
This results in:
Assert :: OsVersion
  Configuration successfully applied.
Apply :: DeveloperMode [DeveloperMode]
  Configuration successfully applied.
Apply :: WinGetPackage [vsPackage]
  Configuration successfully applied.
Apply :: VSComponents
  The configuration unit was not in the module as expected.
Some of the configuration was not applied successfully.

Copy And Save

Share

Ask Copilot
Expected behavior
Visual Studio to install the required components. It also doesn't work with specifying a .vsconfig file.
Actual behavior
The Visual Studio installer installs, but none of the components install.
Environment
Windows Package Manager v1.6.3133
Copyright (c) Microsoft Corporation. All rights reserved.
Windows: Windows.Desktop v10.0.22621.1702
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.3133.0
Winget Directories
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diag…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\sett…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads
Links
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale
Admin Setting                             State
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

https://github.com/microsoft/VisualStudioDSC/issues/17


Fix typo in converter list

Fixed what I suspect is a typo in readme

Yep, it was! The perils of find & replace.


What use does this have?

https://github.com/expressjs/express/blob/c087a45b9cc3eb69c777e260ee880758b6e03a40/lib/middleware/init.js#L30
any reason as to why someone would want to enable this header?

https://github.com/expressjs/express/issues/3110#issuecomment-256400928


Chaining actions with 'then' bug

I might be missing something, but according to this, you should be able to chain actions like this:
const action = () => // Thunk action creator
       dispatch => {
         return dispatch({
           type: 'TYPE',
           payload: { promise: Promise.resolve('foo') } // Dummy promise
         }).then(({ value, action }) => {
             // On resolve
         });
      };

However, I get a dispatch(...).then is not a function error.

It looks like you aren't properly initializing the promise middleware when you create the store.
const createStoreWithMiddleware = applyMiddleware(
  thunkMiddleware,
  promiseMiddleware() // call function
)(createStore);

Does that fix your issue?


Contrasts information button on RNA-Seq service missing any information

You click on it and it says "Help text missing"

The help text was added a while ago.
I have updated text for contrast in the user guide and it will show up in the contrast tool top from the submission form.
"Contrasts specify which pairs of conditions to compare in differential expression analysis. The RockHopper strategy performs all vs all contrats by default, hence, no cotrasts need to be specified."


Milestone: Create conference page

In this PR, I have created a conference page. The changes made are on the milestone PR numbers below.
Changes
#1
#2
#3
#4

@alexander16108
All the code that I am showing in the video, is code that I am actually proud of.


Which size  images  gives the best performance?

hi Vincent,
im used many input size images ( images > 150 *150   ,  it you told in orther issuse) ! but i dont know Which size gives the best performance?

hi Vincent,
hi guy,
im used many input size images ( images > 150 *150 , it you told in orther issuse) ! but i dont now Which size gives the best performance?

The face-recognition net has an input size of 150x150, all other classification nets use an input size of 112x112, the size of the face after face extraction should be around the same size. The smaller they are the more blurrier they will be after upsampling and thus give worse results.
So it depends on how large the face appear in your input image.

oh thanks for your anwser ^^


The rendering of labels in the horizontal bar depends on if the labels are showing in the value axis

Bug report
Codepen
https://codepen.io/aavezel-the-styleful/pen/oNZZRaq

Explanation

What is the behavior you expect?
Title "The United Kingdom of Great Britain and Northern Ireland" must be сropped with an ellipsis in both cases

What is happening instead?
If labels in the value axis not shown, title of category is not cropped, but shifted to the right edge

this bug is still there


Bug fix for host parameter issue with lists

As explained in #1772 there is an issue when using a list as an argument for the host parameter in the Blueprint.route() decorator. I've traced the issue back to this line, and the if conditional should ensure that the name attribute isn't accessed when route is None.

@ahopkins I added some unit tests for using lists with the host parameter. There were already some for testing strings with the host parameter.

@ahopkins can you resolve your change request if you think that we've met your threshhold?

Just following up to see if there is anything else I can or need to do in order to get this merged?

This seems like an okay change. But, it might be nice to have a test with host routing with a string and a list of strings.
Thanks for the PR.

#1775 was closed (stale) and this now has tests. Can we resolve this change request and get this merged?


Update domains_allowlist.txt

I am sorry, but somehow I must have missed the notification informing me of this pull request.
Thank you


core.py unable to import some Python Standard Library modules

On Ubuntu 18, with PREP <IP_ADDRESS> installed and configured:
Create an /etc/irods/core.py with one line:
import csv

Any client command such as ils will then cause  a crash in the agent, and an error in the /var/log/irods/irods.log to the effect that  _Py_NoneStruct cannot be resolved.

The following core.py line also created problems in recent run-throughs of Compute To Date (by @korydraughn)
import ssl


ERROR: failed to solve: process "/bin/sh -c dpkg-buildpackage -b -uc -tc" did not complete successfully: exit code: 3

Install with this tutorial crashed on my assus zenbook. Any help will be appreciate.
#12 [builder  6/18] RUN dpkg-buildpackage -b -uc -tc
#12 0.425 dpkg-buildpackage: info: source package vhba-module
#12 0.425 dpkg-buildpackage: info: source version 20240202-1
#12 0.425 dpkg-buildpackage: info: source distribution debian
#12 0.425 dpkg-buildpackage: info: source changed by Henrik Stokseth<EMAIL_ADDRESS>#12 0.470  dpkg-source --before-build .
#12 0.470 dpkg-buildpackage: info: host architecture amd64
#12 0.653 dpkg-checkbuilddeps: error: Unmet build dependencies: dh-dkms
#12 0.655 dpkg-buildpackage: warning: build dependencies/conflicts unsatisfied; aborting
#12 0.655 dpkg-buildpackage: warning: (Use -d flag to override.)
#12 ERROR: process "/bin/sh -c dpkg-buildpackage -b -uc -tc" did not complete successfully: exit code: 3

[builder  6/18] RUN dpkg-buildpackage -b -uc -tc:
0.425 dpkg-buildpackage: info: source package vhba-module
0.425 dpkg-buildpackage: info: source version 20240202-1
0.425 dpkg-buildpackage: info: source distribution debian
0.425 dpkg-buildpackage: info: source changed by Henrik Stokseth<EMAIL_ADDRESS>0.470  dpkg-source --before-build .
0.470 dpkg-buildpackage: info: host architecture amd64
0.653 dpkg-checkbuilddeps: error: Unmet build dependencies: dh-dkms
0.655 dpkg-buildpackage: warning: build dependencies/conflicts unsatisfied; aborting
0.655 dpkg-buildpackage: warning: (Use -d flag to override.)

Dockerfile:31
29 |
30 |     WORKDIR /cdemu/vhba-module
31 | >>> RUN dpkg-buildpackage -b -uc -tc
32 |
33 |     WORKDIR /cdemu/libmirage
ERROR: failed to solve: process "/bin/sh -c dpkg-buildpackage -b -uc -tc" did not complete successfully: exit code: 3

Hello, please provide output of:
lsb_release -cs

on your machine.

As a temp fix for bullseye - use:
git clone --branch vhba-module-20211218 https://github.com/cdemu/cdemu.git

on step 1. Updated README.
TODO: above will also work for bookworm and newer releases, but installing a dh-dkms dependency should be discovered.


Separate packages for deployment and testing

We currently only really use the requirements.txt, but this adds a lot of useless packages in the running machine and useless security risks. So we should reduce such issues as much as possible.

Done in f21a027


Empty documentation

Hi,
After I ran php artisan api:generate --routePrefix="api/v1/*", the 'public/docs' folder was created, but the inside the doc is empty.
I have followed through the instruction but still can't get it right.
I'm using: Laravel 5.4, Passport to authenticate user
Anyone has this same problem? or did I miss something?
Here is the screenshot of the generated doc:

Hi. When I used --routePrefix="" incorrectly, I also created emtpy docs. Try playing around with using the parameter correctly in order to get the docs you're looking for.

I'm having the same problem and I tried setting routePrefix to * with no luck.

Hi, I had the same problem and fixed it by changing the app/Http/Request.php to

I'm using Laravel 5.5


darwin.apple_sdk_11_0: add Security dependency on xpc

Description of changes
recreated https://github.com/NixOS/nixpkgs/pull/207104 - changing target branch from master to staging exploded there because it can't be done atomically with rebase in github UI and picks up lots of master commits, adds lots of codeowners etc.
Some packages like bazel-watcher seems to have broken after 9dc3b1485936943f6a432dbb32a9f7c576cdab45
Where xpc was removed from IOSurface dependencies. CoreServices were pulling xpc via IOSurface and so Security didn't break. Now explicit dependency on xpc is needed to avoid errors like
In file included from __main__/external/com_github_fsnotify_fsevents/go_1_10_after.go:6:
In file included from /nix/store/2k3mdkl9jvwwzpbfaqhchfiqjq64046b-apple-framework-CoreServices-11.0.0/Library/Frameworks/CoreServices.framework/Headers/CoreServices.h:39:
In file included from /nix/store/2k3mdkl9jvwwzpbfaqhchfiqjq64046b-apple-framework-CoreServices-11.0.0/Library/Frameworks/CoreServices.framework/Frameworks/LaunchServices.framework/Headers/LaunchServices.h:23:
In file included from /nix/store/2k3mdkl9jvwwzpbfaqhchfiqjq64046b-apple-framework-CoreServices-11.0.0/Library/Frameworks/CoreServices.framework/Frameworks/LaunchServices.framework/Headers/IconsCore.h:23:
In file included from /nix/store/2k3mdkl9jvwwzpbfaqhchfiqjq64046b-apple-framework-CoreServices-11.0.0/Library/Frameworks/CoreServices.framework/Frameworks/OSServices.framework/Headers/OSServices.h:29:
In file included from /nix/store/2k3mdkl9jvwwzpbfaqhchfiqjq64046b-apple-framework-CoreServices-11.0.0/Library/Frameworks/CoreServices.framework/Frameworks/OSServices.framework/Headers/CSIdentity.h:43:
In file included from /nix/store/dg51rm1bapffbqvn46bh43km4dhcsy9p-apple-framework-Security-11.0.0/Library/Frameworks/Security.framework/Headers/Security.h:87:
/nix/store/dg51rm1bapffbqvn46bh43km4dhcsy9p-apple-framework-Security-11.0.0/Library/Frameworks/Security.framework/Headers/SecCode.h:35:10: fatal error: 'xpc/xpc.h' file not found
         ^~~~~~~~~~~

Should help with https://github.com/NixOS/nixpkgs/pull/203519
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Tbh, workflow is a little bit of a mystery to me too. 🙂
Hydra doesn't build staging, so testing is manual and something you have to arrange for yourself, I believe? We recently had a community M1 Mac Mini set up which I've been using for large testing builds. You can ask Winter in #macos:nixos.org for access to that, I think yay or nay is reputation-based. (I believe there are also Linux community boxes, but don't use them or know who manages those.)
Staging-next is tested by Hydra. There is always a PR open for staging-next while it's active, and you'll likely be pinged for issues related to your change, but it's also recommended you try monitor these builds yourself. I'm not sure how averse we are to reverting changes. (Ie. if it's okay to land small changes on staging without manual testing, then see how they fare on Hydra in staging-next and revert if necessary.)
As for this PR, I think it looks good on the surface. Security does have a dependency on XPC, so this makes sense. You can verify this with something like:
swiftc -scan-dependencies - <<< "import Security"

This spits out a structure with a modules list. The entries of this are pairs, where the first is an identifier ({ clang: "foo" } or { swift: "foo"} to distinguish Clang / Swift modules.) and the second metadata. Security is a Clang module only (no Swift overload), and has XPC listed in directDependencies.

I'd like to bring this up again. In the macOS 11 SDK, the xpc package contains only headers, and those same headers are already part of libsystem. So I believe this shouldn't change anything?
I tried builds of bazel-watcher and bazel_6 on aarch64-darwin, using current staging with this change reverted, and both work.
The issue I have is that this is introducing two sets of xpc headers in include paths, and that's an error for Swift modules.

Swift support for Darwin was recently merged into staging (#189977), and I'm fixing merge issues. The Swift compiler itself doesn't build after the merge because of the two xpc modules in the include path.


Hygieia Audit API

Hi - Please help to understand the purpose of Audit API.
I am normally starting API and Dashboard.
If i start Audit API, i need to stop API since both are running on same port.
Is there any UI for Audit API?
Regards
Venkat

@venkiradkris  run audit-api on different port. That can be changed in apiaudit properties file. I use 8090 for audit-api and 8080 for api

Hi - Fine. If Audit API runs in 8090, how to connect Audit API with dashboard? current dashboard is automatically connects with API only. Thanks.

Hi - Can anyone help? Thanks.

Hi - Any update/help? Thanks.

@venkiradkris so there is a collector named nfrr that calls api-audit endpoints and aggregates the JSON responses into mongoDB.
I looked in the UI repo but I did not see a widget for audit. @nireeshT is audit widget on the roadmap?


[ISSUE] databricks bundle init

Description
Calling databricks bundles init returns the following error:
Error: failed to compute file content for {{.project_name}}/databricks.yml.tmpl. template: :35:31: executing "" at <user_name>: error calling user_name: unexpected error handling request: json: cannot unmarshal number into Go struct field APIErrorBody.error_code of type string. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues.
Reproduction
Running databricks bundles init:
databricks bundle init

Welcome to the default Python template for Databricks Asset Bundles!
Please provide the following details to tailor the template to your preferences.

Unique name for this project [my_project]:
Include a stub (sample) notebook in 'my_project/src': yes
Include a stub (sample) Delta Live Tables pipeline in 'my_project/src': no
Include a stub (sample) Python package in 'my_project/src': no

Expected behavior
Creation of databricks asset bundle
Is it a regression?
using databricks cli v0.212.3
Debug Logs
Request log:
GET /api/2.0/preview/scim/v2/Me
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * User-Agent: cli/0.212.3 databricks-sdk-go/0.30.1 go/1.21.6 os/darwin cmd/bundle_init auth/pat
< HTTP/2.0 403 Forbidden
< * Content-Length: 52
< * Content-Type: application/json; charset=utf-8
< * Date: Thu, 01 Feb 2024 07:11:53 GMT
< * Server: databricks
< * Vary: Accept-Encoding
< * X-Databricks-Reason-Phrase: Invalid access token.
< {
<   "error_code": 403,
<   "message": "Invalid access token."
< }

Other Information

OS: macOS
Version: 14.3

Additional context
Add any other context about the problem here.

Hey @mzwiesler , I solved it for me. Do you have multiple user profiles in your .databrickscfg file?

Hi @christopher-danz
Yes I have a couple of them and a default one. Tried the command with profile and without but nothing worked so far. Did you solve it?

I feel like I tried everything there is to troubleshoot.. empty cache, create new .databrickscfg file, generate new token, ... the whole thing and I am not entirely sure what made the differnce in the end..
After ensuring you authenticate correctly, try to run:
databricks jobs list
... it should list all the jobs for your DEFAULT profile.
then run:
databricks jobs list --profile=YOUR_DESIRED_PROFILE_TO_USE_DAB
and it should list the jobs for the other host, right?
I then ran:
databricks bundle init  (specified the unique name and the additional stuff)
and it worked then.
I hope this will solve your issue, if not let me know. Maybe I can further assist you.

Hi @christopher-danz
True it seems it really was a token issue. It now works with a new token.
Still not 100% sure what happened. Thanks for your support.

I'm glad it worked out for both of us. 😊

Similar issue here, I'm using an asset bundle generated with databricks bundle init mlops-stacks, the CI/CD is set to run databricks bundle validate with a DATABRICKS_TOKEN env var, but it's failing with that "cannot unmarshal" error

Error: unexpected error handling request: json: cannot unmarshal number into Go struct field APIErrorBody.error_code of type string. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:

GET /api/2.0/preview/scim/v2/Me
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * Traceparent: 00-fdc3a9a[7](https://github.com/KardFinancial/merchant-identification/actions/runs/9898272483/job/27344733076#step:4:8)5520e3039777b7e44a23e6[8](https://github.com/KardFinancial/merchant-identification/actions/runs/9898272483/job/27344733076#step:4:9)f-8af3b60ecbd5fbc2-01
> * User-Agent: cli/0.221.0 databricks-sdk-go/0.42.0 go/1.21.10 os/linux cmd/bundle_validate auth/pat cicd/github
< HTTP/2.0 401 Unauthorized
< * Content-Length: [9](https://github.com/KardFinancial/merchant-identification/actions/runs/9898272483/job/27344733076#step:4:10)8
< * Content-Type: application/json; charset=utf-8
< * Date: Thu, [11](https://github.com/KardFinancial/merchant-identification/actions/runs/9898272483/job/27344733076#step:4:12) Jul 2024 20:48:14 GMT
< * Server: databricks
< * Vary: Accept-Encoding
< * Www-Authenticate: ***"DatabricksRealm"
< * X-Databricks-Reason-Phrase: Credential was not sent or was of an unsupported type for this API.
< {
<   "error_code": 401,
<   "message": "Credential was not sent or was of an unsupported type for this API."
< }

I am running into the same issue as joe using a token to deploy DABs.

I've updated the databricks CLI version my DAB's using from v0.221.0 to v0.229.0. The github action is still failing to run databricks bundle validate, now with a more vague message Process completed with exit code 1.
databricks bundle validate -t stage > ../validate_output.txt
  shell: /usr/bin/bash -e {0}
  env:
    STAGING_WORKSPACE_TOKEN: ***
    PROD_WORKSPACE_TOKEN: ***
    DATABRICKS_TOKEN: ***
Error: Process completed with exit code 1.

What worked for me was, instead of setting a DATABRICKS_TOKEN, setting a DATABRICKS_CLIENT_ID and DATABRICKS_CLIENT_SECRET generated for the service principal. E.g.
env:
  DATABRICKS_TOKEN: ${{ secrets.STAGING_WORKSPACE_TOKEN }}

becomes
env:
  DATABRICKS_CLIENT_SECRET: ${{ secrets.DATABRICKS_CLIENT_SECRET }}
  DATABRICKS_CLIENT_ID: ${{ secrets.DATABRICKS_CLIENT_ID }}

Using<EMAIL_ADDRESS>


Allowing for plain suspend to reschedule thread right away

This reschedules a thread waiting (being suspended) on a spinlock right away without having to wait to get through the whole thread queue.

This is something we've been needing since day 1.

I'd review this, but it has a lot of unrelated commits in it, so I think I'll steer clear.

The last commit is the one specific to this PR, the rest is from the annotated_function branch (PR #2476) which is unrelated indeed. Sorry for that - this just simplified my workflow significantly. Once #2476 is merged, this PR should look more reasonable.

@biddisco I have cleaned up this PR, it should contain only the relevant commit now.


Ensure Project level actions can run as part of another namespace besides Project

Description
Currently a project level action needs to be run as such:
{
"namespace": "Project",
"Assembly": "Project",
"Reference": "Project",
"Type": "Project",
"Actions": [
{
"Name": "ArchiveFiles",
"Type": "Project",
"Value": "",
"Description": "Add NetTopologySuite.Geometries namespace"
}
]
}
This can cause issue if we need to perform a project level action based on another trigger such as the presence of a specific class.
We need to be able to run this action as part of any other rule.
Solution idea(s)
The solution for this is modifying the AnalysisRun.cs class and including a section for project level actions as well.
Additional context
Add any other context or screenshots about the feature request here.

Closing. Re-open if this is rule type is still needed


Eigen include fix

Not sure why this works on OS X. But on Linux this reports that the header can't be found
In file included from /home/jacco/tudat-matlab/tudatBundle/tudat/Tudat/External/JsonInterface/Support/deserialization.cpp:16:0:
/home/jacco/tudat-matlab/tudatBundle/tudat/Tudat/External/JsonInterface/Support/utilities.h:24:10: fatal error: eigen/Eigen: No such file or directory
 #include <eigen/Eigen>

          ^~~~~~~~~~~~~
compilation terminated.

Thanks! I added the fix manually because I changed the structure of the folders and files and that is causing a lot of conflicts...

Hmm, there goes my attribution ;-).


Eval

evaluate() takes 4 arguments, be explicit when setting debug to avoid clashing with num_beams.

Thanks for your contribution, Tom!


[Doc] Download page must not use the archive server

Search before asking

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

What issue do you find in Pulsar docs?
https://pulsar.apache.org/docs/4.0.x/getting-started-standalone/ says to download from the site:
https://archive.apache.org/dist/pulsar/...
This is not permitted; the archive site is ONLY for historic releases.
What is your suggestion?
Follow the instructions at:
https://infra.apache.org/release-download-pages.html#links
Any reference?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Is there anyone working on this? If not, please assign this to me. @lhotari

Is there anyone working on this? If not, please assign this to me. @lhotari

@Shawyeok sure, please go ahead

Is there anyone working on this? If not, please assign this to me. @lhotari

@Shawyeok sure, please go ahead

Please feel free to take a look.
https://github.com/apache/pulsar-site/pull/978

@Shawyeok Another detail missing from the pulsar.apache.org/download page is about "Remind users to check sums and signatures", explained in https://infra.apache.org/release-download-pages.html#best_practice . Would you like to add that information too? The column "Crypto files" on the download page doesn't make much sense since there is no common concept as "Crypto files" for the checksums and signatures.

Another detail missing from the pulsar.apache.org/download page is about "Remind users to check sums and signatures", explained in https://infra.apache.org/release-download-pages.html#best_practice . Would you like to add that information too?

There is a section about verify release integrity on the download page : https://pulsar.apache.org/download/#release-integrity

The column "Crypto files" on the download page doesn't make much sense since there is no common concept as "Crypto files" for the checksums and signatures.

This could be a one line change, how about Checksum & Signature?


Exclude self from unseen parties list in SyncPoller

Hi Tessera team,
In our Tessera 0.8 environments, the nodes are attempting to connect back to themselves to perform party discovery.
I suspect this is not an issue in most environments, as the HTTPS connection would normally succeed. However, if there is NAT involved the connection fails or times out - resulting in repeated entries in the log as the SyncPoller retries the connection. We don't see a functional issue, just confusing log entries.
We haven't observed this behaviour previously with Constellation.
The attached code change simply filters out the local address from the unseen parties list before it attempts to connect. It removes the connection attempts / log entries in our environments.

Codecov Report

Merging #688 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #688      +/-   ##
============================================
+ Coverage     99.49%   99.49%   +<.01%
- Complexity     1859     1861       +2
============================================
  Files           285      285
  Lines          5545     5552       +7
  Branches        278      278
============================================
+ Hits           5517     5524       +7
  Misses            5        5
  Partials         23       23

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/com/quorum/tessera/sync/SyncPoller.java
100% <100%> (ø)
11 <4> (+2)
:arrow_up:

Continue to review full report at Codecov.

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

Hey Peter. This changes look good, thanks for contributing!
Could you fill out our CLA please? Once it's complete we can merge this change.
https://github.com/jpmorganchase/cla/blob/master/cla-2016-09-01.pdf

Hey Peter
Thanks for the contribution!
We require all contributors to fill out the CLA, as found here:
https://github.com/jpmorganchase/cla

Thanks @melowe @prd-fox - sorry for the delay getting you the CLA. I've got it signed, and will slack it to you on the Quorum slack

Linked issue: https://github.com/jpmorganchase/tessera/issues/725


Mol construction from bytes crashes inside PostgreSQL plpython3 function, but works outside

Hello all,
according to the documentation, it is possible to exchange RDKit molecules in binary format between PostgreSQL and Python using
bytes = m.ToBinary()
and
mol = Mol(bytes)
on the Python side of things, and
bytea = mol_to_pkl(mol)
and
mol = mol_from_pkl(bytea)
in PostgreSQL.
This works for me as described, UNLESS I try to pass a bytea Molecule from mol_to_pkl into a Python function based on plpython3:
Example:
CREATE or replace FUNCTION chemreg.tautomers (molbytes bytea)
  RETURNS bytea[]
AS $$
from rdkit.Chem import AllChem
from rdkit.Chem.MolStandardize import rdMolStandardize

enumerator = rdMolStandardize.TautomerEnumerator()

m = AllChem.Mol(molbytes)

tautomers = enumerator.Enumerate(m)

return [t.ToBinary() for t in tautomers]
$$ LANGUAGE plpython3u;

Called:
select chemreg.tautomers(mol_to_pkl('Oc1c(cccc3)c3nc2ccncc12'::mol));
Crashes the server with ntstatus 0xC0000005, which means STATUS_ACCESS_VIOLATION
(https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-erref)
However, passing binary molecules OUT of plpython3 works fine, when the molecule is passed in e.g. as SMILES:
CREATE OR REPLACE FUNCTION chemreg.tautomers(smiles text)
 RETURNS bytea[]
 LANGUAGE plpython3u
AS $function$
from rdkit.Chem import AllChem
from rdkit.Chem.MolStandardize import rdMolStandardize

enumerator = rdMolStandardize.TautomerEnumerator()

m = AllChem.MolFromSmiles(smiles)

tautomers = enumerator.Enumerate(m)

return [t.ToBinary() for t in tautomers]
$function$;

Called:
select chemreg.tautomers('Oc1c(cccc3)c3nc2ccncc12'::text);
But this obviously increases roundtrip time.
When using those functions in plain Python and PostgreSQL, e.g. via psycopg2, everything works. The binary strings are identical, I have checked them with a HEX-Editor for identity.
Configuration:

RDKit version: 2023.03.2
OS: Windows 10 x64
Python version 3.10.7 x64
PostgreSQL 15.3 x64
No Conda
RDKit compiled from Source Release 2023_03_2 as described here: https://theplateisbad.blogspot.com/2023/05/how-to-build-rdkit-on-windows-wo-conda.html

best regards,
Ernst-Georg

I cannot reproduce this on a different machine with identical setup (except Windows 11 instead of 10).


OpenModal6 function is duplication

OpenModal6 function is duplication.
So not working this example.

Thanks @osungjin, this has been fixed in the last version


Load from and dump to streams

Have pytomlpp.load to parse TOML from a text stream, and pytomlpp.dump to serialise TOML to a text stream.
import io, pytomlpp
stream = io.StringIO()
pytomlpp.dump({"spam": 42}, stream)
stream.seek(0)
print(pytomlpp.load(stream))

This will then match the functionality of the standard-library JSON module. This could be implemented in either Python or PyBind11, simply calling the existing pytomlpp.loads and pytomlpp.dumps for now. Further iterations I imagine could use C++ streams.

I want to deal with all possible IO with pure python... I will think about this and see if I can implement it.

Hey @EpicWink  do you know how to combine pure python code with the compiled module in one project? I imagine we need to change the setup.[py|cfg] a bit to do it but not sure how. Do you have any guidance on that?

Choose one:

For packages (directories of Python code, with init.py's) at the top level:

In the [options] section, specify packages = find:
In the [options.packages.find] section, specify exclude = tests/ (etc)

For packages inside a subdirectory (eg "src/") (preferred):

In the [options] section, specify packages = find:, and specify
package_dir =
  = src

In the [options.packages.find] section, specify where = src (etc)

For modules (individual Python files) at the top level

In the [options] section, specify py_modules = <module name>

For modules inside a subdirectory (preferred):

In the [options] section, specify py_modules = <module name>, and specify
package_dir =
  = src

See the documentation


Send confirmation code and the flow confirmation code separately in user account recovery API v2

Describe the issue:
In the user account recover API v2, WSO2 IS is sending the confirmation code and flow confirmation code appended with a dot.
With the improvement done in https://github.com/wso2/product-is/issues/14808, users can use an OTP instead of UUID in the recovery flows and utilize it in the API based implementation such as mobile applications that will utilize REST API to confirm.
When the flow confirmation code and the confirmation code are appended, it is not to show only the confirmation code without string manipulation.
It is required to send the confirmation code without appending the confirmation code to the user in the recovery flow.
How to reproduce:

Enable OTP support for recovery flows https://github.com/wso2/product-is/issues/14808
Initiate password recovery flow with API v2 [1]
In the recovery flow, the confirmation code will return with flow confirmation code appended.

{
    "code": "PWR-02001",
    "message": "Password recovery information sent via user preferred notification channel.",
    "flowConfirmationCode": "cc8136de-ea44-440b-8338-f5c625a8954c",
    "notificationChannel": "EXTERNAL",
    "confirmationCode": "cc8136de-ea44-440b-8338-f5c625a8954c.wR5hT4",
    "resendCode": "b87de6d1-abb2-4aee-b079-cc2934b8ef69",
    "links": [
        {
            "rel": "next",
            "href": "/api/users/v2/recovery/password/confirm",
            "type": "POST"
        },
        {
            "rel": "resend",
            "href": "/api/users/v2/recovery/password/resend",
            "type": "POST"
        }
    ]
}

Expected behavior:
Only the confirmation code should send in the email. Flow confirmation code should be send in the API response to use in the next API call.
Environment information:

Product Version: IS 7.0.0

[1] - https://is.docs.wso2.com/en/latest/apis/user-account-recovery-v2-rest-api/

This fix is added to the master branch but will be disabled by default as this is a behavioural change in the recovery V2 API. This fix will be tracked by https://github.com/wso2/product-is/issues/21160 so that when the V3 API is planned for release, this fix can be added with that.


⚠ Spryr has degraded performance

In 8bac926, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 8560 ms

Resolved: Spryr performance has improved in 707e8de after 7 minutes.


Comparing signatures with == is bad security practice due to byte-wise short-circuit being a potential source of subtle timing attack.

If you compare user supplied signature to calculated signature using ==, Ruby will helpfully abort the comparison early if you have a mismatch between the signatures. There's at least a few places in Signet that do this, with the OAuth 1 server code that @mechazoidal contributed being the most exposed to the issue. In a nutshell, you probably want a non-short-circuited equality check for that.
Realistically, I doubt anyone could actually build an exploit for this in large part due to Ruby's wildly varying runtimes for web requests, but I don't really want to find out the hard way, so this should probably get eliminated as even a possibility, just in case.

In terms of non-short-circuited equality operations you could use, assuming a signature of 'pretendsignature', you can safely abort if the supplied signature is the wrong length (invalid signature, no information revealed), and zip on the pair of 'pretendsignature'.unpack('C*') and supplied_signature.unpack('C*'), doing a bitwise | reduce on the output of a ^ b, and indicating a match if the final output is zero.

The other, possibly easier and more readable option is to do a double HMAC – of both the calculated value and then the attacker supplied value.
If:
HMAC(m, k) == their_signature
Then the following is also true:
HMAC(HMAC(m, k), k) == HMAC(their_signature, k)
But in the second scenario, the attacker no longer has any control whatsoever over the timing of the thing being compared and should be unable to take advantage of the short circuit, while also maintaining readability of the code.

def safe_compare a, b
check = a.bytesize ^ b.bytesize
a.bytes.zip(b.bytes) { |x, y| check |= x ^ y.to_i }
check == 0
end

Yup, that looks right to me.


🌟Enhance: Default View

Added the views for non-registed users.

Done in the following commit https://github.com/JuanRamos15/ProyectoDWPII/commit/8bb64bd97674a7d1f41e8151d75a45dda7fe9343


Move Viper and VTOL to collapsed section

This IMO might be a controversial change given that I wrote FlightCore but there has a been a common sentiment over the last few months that out of FlightCore, VTOL, and Viper, FlightCore is the installer/mod-manager that causes the least issues with end-users.
It's not without reason that #48 exists. Further showing 4 different installation options confuses end-usersbased on feedback on Discord and it makes sense to highlight a mod-manager that is developed by someone closely associated with Northstar development.
At the same time I'm well aware of the improvements @0neGal made to Viper in the last few weeks and that VTOL also still sees development whenever @BigSpice is able to work on it.
So I struggle whether to merge this change or not and would highly appreciate any feedback on this <3
Rendered:
https://github.com/user-attachments/assets/47632442-dfdf-48a6-ba7f-1eb5c47ffc6b
Supersedes #48
Sorta supersedes #64

Understood honestly.
I've been very mia , and can't excuse that.
Agree with the changes.
Im back starting in 2 weeks and will be sure to bring VTOL back to current release standard.
❤🫡.

I honestly think any opinion I would give on a PR like this would be biased, so I would honestly just let it be up to you, even if it may also be biased for you. I do however think adding support links directly on the website alongside with possibly just redirecting users automatically in tickets may help a lot, or well, move the issues from the NS Helpers over to the actual mod manager developer(s), which I wouldn't particularly mind...

Contributor

no regular user wants to make a github issue, even if u give them a link, tickets are already done by mods/helpers/contributors/modders alike. there is nothing to "move"

I do however think adding support links directly on the website [...]

I like that idea. Feel free to make a PR to this repo to add ^^

[...] alongside with possibly just redirecting users automatically in tickets may help a lot, or well, move the issues from the NS Helpers over to the actual mod manager developer(s), which I wouldn't particularly mind...

We could maybe have Spectre (the Discord bot @itscynxx wrote) check for keywords like "Viper" in tickets and post a message with a link to the Viper GitHub repo. Just keep in mind that the majority of players have no clue how GitHub works or have an account, hence me making majority of recent Viper GitHub issues in their place.

Aight, Imma just say we give this a shot, see if it has a positive impact in terms of tickets and reducing player confusion and if it doesn't we can always revert.
Also like, feel free to PR any changes if dislike this decision or want to adjust the description or want to add your own mod-manager etc ^^
I cannot stress enough how important open communication and discussion is and want to encourage everyone to reach out ASAP if anything is unclear or feels unfair. We always want to find the solution that fits everyone and that only works if everyone participates in the discussion and reaches out <3


Evolution asks to verify mail server's cert every time it's launched

Evolution does not trust internal CA certs.  This results in the user being given the option to accept temporarily, or permanently the cert from the mail server.
Also means that if the certs are updated, the user is prompted again.


Add real wrf

This PR adds initial, head-node serial-only capabilities to run real.exe and WRF tasks.
Expected Differences

[ ] Do these changes modify the system output in any way? [Yes or No]
If yes, please describe:

Pull Request Testing

[ ] Describe testing already performed for these changes:

[ ] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

[ ] Run on Cheyenne (instructions in comment below)
[ ] Run on AWS if possible (currently in my tests metgrid fails, see #68)

[ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes or No]

[ ] Documentation for test case available in comment below

[ ] Do these changes include sufficient testing updates? [Yes or No]

[ ] Will this PR result in changes to the test suite? [Yes or No]
If yes, describe the new output and/or changes to the existing output:

[ ] Please complete this pull request review by [Fill in date].

Pull Request Checklist

[ ] Review the source issue metadata (labels, project, and milestone).
[ ] Complete the PR definition above.
[ ] Ensure the PR title matches the feature or bugfix branch name.
[ ] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Project
Select: Milestone as the version that will include these changes
Select: Development to link to the original development issue.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

Note: #67 should be merged before this PR so I have not yet assigned it a priority. But this is obviously an important one.
Testing details for Cheyenne
Clone and set up code
git clone git@github.com:NCAR/wrfcloud -b add_real_wrf
module load conda
conda activate /glade/p/ral/jntp/CCproject/wrfcloud/conda/wrfcloud
export PYTHONPATH=`pwd`/wrfcloud/python/src
cd wrfcloud/python/src/wrfcloud/runtime/
cp /glade/p/ral/jntp/CCproject/wrfcloud/wpsprd/geo_em.d01.nc configurations/test/

Modify test.yml for Cheyenne values
    wpscodedir: /glade/p/ral/jntp/CCproject/wrfcloud/WPSV4.4_intel_serial_large-file/
    wrfcodedir: /glade/p/ral/jntp/CCproject/wrfcloud/WRFV4.4_intel_dmpar/
...
    workdir: /glade/scratch/your_name_here
...
    local_data: /glade/p/ral/jntp/CCproject/wrfcloud/input_data/gfs/2022060100/gfs.t00z.pgrb2.0p25.f*

Run the test case, should be successful through real.exe
./run.py

WRF may fail with segfault, suspected due to out-of-memory condition on login node

Follow-up note: Decided to test on a compute node on Cheyenne to ensure that WRF was writing forecast files, and it did indeed work, so the failures on the login node must be memory-related.


CDI BeanManager fix for Quarkus

This is a fix for an issue in Quarkus (https://github.com/quarkusio/quarkus/pull/7639) that is currently
temporarily fixed inside the Quarkus Extension (https://github.com/quarkusio/quarkus/pull/7639)
Signed-off-by:Phillip Kruger<EMAIL_ADDRESS>

@asoldano @stuartwdouglas please have a look (I can not assign reviewers :) )

thanks @phillip-kruger ; I believe @ronsigal is the one who should really review this, as he provided the original implementation. Thanks!

Hi @phillip-kruger,
I'm re-running the failed tests and also running the testsuite on my laptop.
I have a question and a comment:

What are you seeing that suggests moving the acquisition of the BeanManager out of observeBeforeBeanDiscovery()? Was it getting a null BeanManager?

There are a few places in RestClientExtension.construct() where the manager variable is assumed to have a non-null value. With your changes, manager will get assigned in isCDIActive(). Now the only place where construct() is called is in ClientHeaderProviders.getCustomHeadersFactory():

if (RestClientExtension.isCDIActive()) {
    Object factory = RestClientExtension.construct(factoryClass);

so it's safe, at least for now. But what if the code changes in the future? Maybe there should be a test at the beginning of construct() that verifies that manager != null or acquires a value.
-Ron

Hi @ronsigal - thanks for getting back to me.

Yes,
public void observeBeforeBeanDiscovery(@Observes BeforeBeanDiscovery event, BeanManager beanManager) does not work in Quarkus. So even if CDI is active, the manager is never set.

Agree. This was an issue even before my change. I did not want to do a null check, as some other code might rely on a NullPointerException, but if you agree that we should then I am happy to add that. Let me know.

Hey @phillip-kruger,
Yeah, I think it's reasonable to do the null check.
The same test (SSETest in resteasy-vertx) is still failing in the jdk 11 runs. I'm running the testsuite with jdk 11 on my laptop to see what happens. It doesn't seem like your change should affect an SSE test, but I'll have to look into it. If you have any insights, they would be welcome. ;-)

Hi @ronsigal . Ok I am adding a null check to this PR. Thanks

@ronsigal - please check again. Now with CDI checking in the RestClientExtension.construct() method

re: "I am adding a null check to this PR."
Thanks, @phillip-kruger.
It looks like the intermittent test failures come from a recent unrelated change. Still looking.

Thanks @ronsigal - Please let me know if there is anything I can help with.

Thanks @ronsigal for the checks and @phillip-kruger for the discussion. The intermittent failure is unrelated, so I'm merging this.


ARROW-717: [C++] Implement IPC zero-copy round trip for tensors

This patch provides:
WriteTensor(tensor, file, &metadata_length, &body_length));
std::shared_ptr<Tensor> result;
ReadTensor(offset, file, &result));

Also implemented Tensor::Equals and did some refactoring / code simplification in compare.cc

green build: https://travis-ci.org/wesm/arrow/builds/216839036

rebased

rebased. merging on green build

Here's the appveyor build: https://ci.appveyor.com/project/wesm/arrow/build/1.0.32


post: v23.4.0 announcement (#17)

Reopened from https://github.com/nodejs/bluesky/pull/17 because I discovered that we need to disable https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-the-automatic-deletion-of-branches to stop GitHub from automatic deletion of the PR branch.

Adding an edit to remove the redundant \n at the end. Let's see if squashing and merging this time is okay.

We should explicitly write with version of node is impacted because BSK trunk the link


Update guidance on non-UK components

Trello card
https://trello.com/c/kfPvMtUp/6310-add-content-guidance-re-non-uk-inset-and-expander-components
Context
We've recently introduced new non-UK instet and expander components, so need to update our guidance on how and when to use them.
Changes proposed in this pull request
Guidance to review

@jenhadfield-dfe this should be all ready for you :)


[Enhancement] A button with content layout

Summary
Xamarin forms Button control is limited to text and icon. Usually we used to create components with Frame or Grid if we require a to define a Button with complex template.
API Changes
e.g.
<views:ContentButton Padding="2" BorderColor="Red" BackgroundColor="Gray" CornerRadius="12" IsClippedToBounds="True"  WidthRequest="200">
                <views:ContentButton.Content>
                        <StackLayout Orientation="Vertical" HorizontalOptions="FillAndExpand" VerticalOptions="FillAndExpand">
                            <Label Text="Test Button" TextColor="Black" HorizontalOptions="Center" VerticalOptions="Center" FontSize="Medium" />
                             <Label Text="Test subheading" TextColor="Black" HorizontalOptions="Center" VerticalOptions="Center" FontSize="Small" />
                        </StackLayout>
                </views:ContentButton.Content>
            </views:ContentButton>

Who Will Do The Work?

[x] I am willing to take this on myself

@levelfivecoder i've done something similar here - https://github.com/IeuanWalker/Xamarin.Forms.FlexButton
When I have time I was going to create a new control that would allow people to anything to the content.
You also would need to extend different controls within XF to allow the user to set different colours for different states i.e. pressed unpressed changes background and text color

@levelfivecoder i've done something similar here - https://github.com/IeuanWalker/Xamarin.Forms.FlexButton
When I have time I was going to create a new control that would allow people to anything to the content.
You also would need to extend different controls within XF to allow the user to set different colours for different states i.e. pressed unpressed changes background and text color

Nice to hear that community has the requirements for the button with content

What about if I put button in button? Which button event will fire?

@levelfivecoder @AndreiMisiukevich @pictos
I know there is already a PR for this in progress (#386), just wanted to share my solution -
https://github.com/IeuanWalker/Xamarin.Forms.StateButton
Feel free to copy stuff if you need to for the existing PR.

Here an example of what the control can do.
This shows the StateButton as the wrapper for a card. When the card is pressed or clicked then the title is set to bold and the border goes darker.
The card also shakes when clicked, this shows that it works with the animation behaviour stuff from this NuGet.
It also shows that it works with nested TapGestureRecognizer, XF native button and nested StateButton -

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


Need help using the library

Hi,
Thanks for the amazing library! It has helped me get started with my personal project at link.
However, I have been facing a couple of issues with it:

The webapp currently doesn't scale for mobile devices.
I.e. it scales fine on the desktop but when I access the site in a mobile browser.. I see the same page.
I am unable to use the full-page Spinner component as intended using <Spinner size='lg' />
Right now it appears in the top-left corner.
I am currently using Row and Col components to implement the grid in my app.
The way I am centring the content right now is

<Row>
  <Col md='1/4'><Col/>
  <Col md='2/4'><Col/>
    <ContentToRender />
  <Col md='1/4'><Col/>
<Row/>

Not using any CSS atm.
Is this the correct way? Can the Container component be of use?
Not sure if this is the best place to ask. Would be glad to have any sort of help here.
Thanks!

@purezen This sounds like it is not an issue with the library. This is a feature that a specific meta tag adds.Here is the tag. Place this in the head of your index file <meta name="viewport" content="width=device-width, initial-scale=1">
Hope this helps!

@Garretttaco Hey, thanks there. I used the meta tag in my app and now it does indeed scale for mobile devices!
Thanks there 👍
Right now the Spinner still appears at the top-left corner of the page when I use it. I would like to get it in the middle of the app while being compatible on mobile devices. What would be the recommended way to achieve that?


Fix in determining root of some extensions.

Also tiny fix in test of extensions import

Superseded by #989


if i want to use "Learnable Ophthalmology SAM" model, how to edit the json file.

If i want to use train "Learnable Ophthalmology SAM" model on ciiyscape dataset, how to edit the json file?
{
"init_lr" : 0.01,
"momentum" : 0.9,
"weight_decay": 5e-4,
"root" : "D:\workspace\datasets\segmentation\CamVid\preprocessed",
"epochs": 300,
"lr_sche": "poly",
"image_size": [360, 480],
"super_reso": true,
"fusion": true,
"num_classes": 32,
"gpu_index": "0",
"model_name": "deeplabv3plus",
"backbone": "resnet101",
"channel": 3,
"upscale_rate": 2,
"num_workers": 4,
"ckpt_dir": "./ckpt",
"dataset": "camvid",
"batch_size": 2
}

Thank you for your attention. You need to modify the training script to add the model. We suggest you to modify the scripts/cityscape_train.py.
The following is the script to import the model to the code.
from imed_vision.models.segmentation.learnable_sam import PromptSAM

Then, you need to add the following codes after line 119.
    elif model_name == "promptsam":
         model = PromptSAM(num_classes=num_classes, model_name=backbone, checkpoint=config['pretrained'])

Note: You need to add the parameter pretrained to the JSON config file.


Make ReadMe File

Configure the ReadMe file and add all user requirements and stories

Aaditya has mad the readme file


Make the Counter page Increment by 3 instead of 1

Currently, the Counter page increments by 1 when you click the button. I want it to increment by 3.

Fixed


Docs enhancement 1.6.0

Modulus Pull Request
Description
Update docs for 1.6.0 release.
Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[ ] An issue is linked to this pull request.

Dependencies

/blossom-ci

Duplicate commits seen due to pushes made to RC. Will use "Squash and merge"

/blossom-ci

/blossom-ci


Undefined is not an object (evaluating '_reactNatuve.UIManager.RNVideoProcessing.Constants')

react-native-cli: 2.0.1
react-native: 0.47.2

I did npm i --save react-native-video-processing, then followed the Manual Installation tutorial (no react-native link).
However, I'm getting this error:

Please help @shahen94 as I've tried re-installing this multiple times, but it seems UIManager isn't seeing RNVideoProcessing for some weird reason.

Hi  @iMagdy !
The library written in Swift - that's why it's not easy to integrate in react-native project
Please revert all your changes and watch this Demo video.
https://www.youtube.com/watch?v=HRjgeT6NQJM&feature=youtu.be
This should help.

@shahen94 Thanks for your prompt response, I've already followed this video (mentioned in my original comment above).
I'll give it another go.
However, I'd appreciate if you give me any idea on the following questions:

How to make sure that I've really completely reverted everything?
Is dropping the entire dir of ios/RNVideoProcessing into the project root is as same as manually creating groups one by one?

Thanks so much

@iMagdy

just what you did for manual linking - revert all that changes(remove added files, undo included libraries)

NO - it's not the same - you should manually create groups one by one.

Hi @shahen94, I'm having the same issue, I'm following you video on youtube and noticed that the same compile sources as you do. Should I add them manually? I also notice that I don't have the same imports on ProjectName-Bridging-Header.h. Can be that the causes of the errors?

Definitely yes!
You should manually create groups and add imports correctly

@shahen94 After reverting all changes, and creating groups manually, it finally works fine now!
Thanks a lot, I figured out the issue was dropping the entire directory instead of manually creating each group.

@shahen94 Hello, this library can still support the current react-native?

@shahen94 In my xcode project, my project name file does not contain AppDelegate. Swift and header.h，I did it through react-native init my project

@shahen94  I am trying to use your library in the react native expo app (version 0.71), the demo video is little confusing, I am using windows machine and testing mostly in android device, do you have any steps to fix the problem for me.
the error I am getting is TypeError: Cannot read property 'Constants' of undefined, js engine: hermes