Terrain toolbox crashes in Release mode

The Terrain Toolbox crashes in Release mode but works perfectly fine in Debug.
Unfortunately the logs are not helpful. I have added few logs to figure out where the problem might be happening
oapiWriteLog("[TerrainToolbox] Initialize");
g_pCore = pCore = gcGetCoreInterface();
oapiWriteLogV("[TerrainToolbox] after gcGetCoreInterface %p", g_pCore);

The few last lines are as follows
000012.612: Loading module LuaInline
000012.909: Module ShuttleA.dll .......... [Build 221209, API 221209]
000013.042: Module ShuttlePB.dll ......... [Build 221209, API 221209]
000013.120: Finished initialising status
000013.120: Finished initialising camera
000013.129: Finished setting up render state
000013.149: D3D9: [Scene Initialized]
000013.187: Finished initialising panels
000013.433: D3D9: NewShader [DG\deltaglider_ns]=4
000021.735: [TerrainToolbox] OpenToolsClbk
000021.736: [TerrainToolbox] Initialize

As we can see it fails while gcGetCoreInterface(), specifically while initializing gcCore2 if (pBindCoreMethod) return (pCoreInterface = new gcCore2());
Does anyone know what might be happening?

Thanks for the report. This issue should be fixed now. The exact reason for failing in release mode remains unclear. Correct value was assigned to a global variable but it was reading as NULL in release mode causing a crash. Could it be due to code optimizations somehow.
If everything works alright then the issue can be closed.

Thank you for fixing it this quickly. I am also thinking the same that it might be caused due to optimizations in Release mode. Its working now so this issue can be closed.


Template categories + search

Replace options, including templates can now be optionally classified by category:

Also, an additional search meta-data allows to specify search terms not displayed through name, label or description.

Try it out via
npx @bpmn-io/sr bpmn-io/bpmn-js-connectors-extension#template-categories-search -c 'npm start'

Enable create/append anything modeling UX via a feature toggle.

@currandwyer I'd love to make basic grouping available to our users. Please have a look at this change and highlight and feedback potential improvements.

@nikku I think this looks great, particularly in this append anything context. I also expect that append anything is the direction of travel for our modeling UX, so this will be a nice feature to have. When this does happen, I suggest changing the title to "Append specialized element" or something like this, to be specific and precise about the purpose of the component.
I realize it's out of scope, but I think that adding the ability to filter would be nice to have alongside this.
I don't think it makes sense to launch categories until we have a higher number of specialized tasks to categorize.  Rather than categorizing them, we should focus on shipping auth so that the auth configuration exists within the properties panel, as it should. Then we only have two connectors, not enough to categorize. In any case, the categories will work differently (for more higher order distinctions), and we mustn't confuse these different mental models so early on.
I think it is very important not to refer to connectors as templates. We must align on one word (whether it is connectors or integrations or whatever) to refer to these things. The element template is a part of the connector, but it's not the main thing. Introducing this additional technical language, which requires users to develop an understanding of our technical architecture, could be unnecessarily confusing.

Thanks for your comments, both of you.

I realize it's out of scope, but I think that adding the ability to filter would be nice to have alongside this.

Agreed @currandwyer.

Updated to not show the category, unless provided in connectors mode:

Also updated the branch so it can be merged independently.


Data via mobile network

Is it possible to transfer data using GSM2 click module on a discovery board using the HAL libraries? I was not able to find any library associated with GSM/GPRS to do the same.

it is possible.
You need USART library to interface your GSM module.
I don't have similar library like my esp8266at.com but I'm planning to make similar like that for SIM800 or SIM900 devices.

So does the USART library work for GSM 800/900 ?

I don't know what knowledge you have in microcontrollers or similar, but:

USART is only for communication. Sending/Receiving data.
You have to know by yourself what you will send and what you wanna receive back.


Create TypeScript Interface for the Transporter model.

🚀 Feature Request
Add a new file, or add to Handler.ts, with the Transporter model TypeScript interface definition in the /client/src/types directory.
context
The Transporter extends the Handler model with two fields, order (integer) and manifest (integer, also a foreign key to Manifest model). The model definition for transporters on the server can be found at /server/apps/trak/models/transporter.py.

@dpgraham4401 , assign this to me.

🚀 Feature Request
Add a new file, or add to Handler.ts, with the Transporter model TypeScript interface definition in the /client/src/types directory.
context

The Transporter extends the Handler model with two fields, order (integer) and manifest (integer, also a foreign key to Manifest model). The model definition
for transporters on the server can be found at [/server/apps/trak/models/transporter.py].
(https://github.com/USEPA/haztrak/blob/main/server/apps/trak/models/transporter.py).

Does manifest contain a populated object or just manifest_id (i.e foreign key) ?

Does manifest contain a populated object or just manifest_id (i.e foreign key) ?

the manifest field just contains a foreign key, not a complete manifest object.

closed with #163


add RRIB data

RRIB CSV files included:
Pokemon stats/typing
Move properties
Learnsets
Shown Pokemon
Legal Pokemon

.


Cordova platform installation failed for Android

Error message
Adding android project...
Error: Error validating package name. Package name must look like: com.company.Name
Wrong playStoreId - Validation to be implemented.

Hello there, I understand npm run android is forced launching an emulator previously set in AVD.
I have no emulators because lack of RAM (in my system 4Gb isn't enough for studio+emu), is there any chance to run adb via usb on a phisical device?
I'm following documentation and is told to deploy to phone by npm run studio but it's trying to deploy to PlayStore and an ID is needed. I'm not yet registered (they ask 25$ also).
npm run android
BUILD SUCCESSFUL

Total time: 19.318 secs
Built the following apk(s):
	/home/tassoman/devel/appframework/node_modules/cordova/bin/platforms/android/build/outputs/apk/android-debug.apk
ANDROID_HOME=/home/tassoman/Android/Sdk
JAVA_HOME=/usr/lib/jvm/java-8-oracle
Error: No emulator images (avds) found.
1. Download desired System Image by running: /home/tassoman/Android/Sdk/tools/android sdk
2. Create an AVD by running: /home/tassoman/Android/Sdk/tools/android avd
HINT: For a faster emulator, use an Intel System Image and install the HAXM device driver

npm run studio

Error: Cordova platform installation failed for Android.

Can we have a `npm run android-adb' command? Or something similar?


Changes to sdl field of _service object to make it compliant with the…

… federation specification

if the Query type is present, it should have the @extends directive, since it extends the query type
the default schema definition should not be includeded in the sdl. This causes problems for example with the @apollo/gateway, because if the default schema definition is present, no queries are picked up from the federated schema.

if the Query type is present, it should have the @extends directive, since it extends the query type

if all Query types are extended then which service defines the base?

the default schema definition should not be included in the sdl. This causes problems for example with the @apollo/gateway, because if the default schema definition is present, no queries are picked up from the federated schema.

Sounds like a bug in @apollo/gateway - @pcarrier can you take a look?

Documenting for completeness

if the Query type is present, it should have the @extends directive, since it extends the query type
if all Query types are extended then which service defines the base?

Apollo gateway defines the base Query type.  Each service that is part of the federated schema should be extending the Query type to signal that its queries are part of a larger federated schema.

the default schema definition should not be included in the sdl. This causes problems for example with the @apollo/gateway, because if the default schema definition is present, no queries are picked up from the federated schema.

Sounds like a bug in @apollo/gateway - @pcarrier can you take a look?

This still does sound like a bug but indeed there is no need to include those defaults.

Thanks for the fix!


Fix issue #11260: Added (PDF). CC BY-SA. to all links

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
This PR adds "(PDF). CC BY-SA." to all relevant book links, ensuring proper attribution and clarity for file format and license.
Why is this valuable (or not)?
The added information provides users with licensing clarity and a consistent structure for identifying PDF resources. It's important for legal transparency and user experience.
How do we know it's really free?
All the resources linked have already been vetted for being free. This PR only updates the format and license info.
For book lists, is it a book? For course lists, is it a course? etc.
These are all books and resources listed in the markdown files.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Merge it 😄

merge it. all test cases have passed

you didn't check the resources for licenses

Ok I will look into it again


Remove global  style

The global style applied to p tags causes a few issues.

News that contains a p style looks incorrect
The line-height is far too big for genuine paragraph content
It is being used to give elements size but divs or similar should be used instead

This work is to remove the global p style from the App component and replace all instances of p in anywhere that isn't a true paragraph (buttons,  dropdowns, titles, etc.)
p tags should never be used to give style/height to an element. Only to apply default paragraph styles to paragraphs of text.
Blocks #6 because news cannot be rendered

This still needs to be done but no longer blocks #6


General: styling of all icon/label links (buttons?) should match

Styling:

label should always be positioned to the right of the icon
use same color for the icons and labels
"view and download" link on the subsetting tab should have the same styling as the “download”, “+ new analysis”, "my analyses" on the study header (font, text size, color)
maybe make them look like buttons?

Hi @danicahelb May I ask your opinion about the last item, make them look like buttons ? I have tried to change the top links (Download, New analysis, and My analyses) to be button style as follows. Certainly, the Download is the current style (without change), the New analysis is a button style with a color (color may be changed), and the My analyses is applied to have outline so that it looks like a button. Do you have any preference among the three ? It would be very helpful for me.
Also I just want to make sure that the view and download link you mentioned points out the top right table-like icon at the second screenshot ?

button styles

view and download link ?

thanks for the mockups @moontrip! I've checked in with the ClinEpi team, and we prefer option (1) Download (so no need to make them look like buttons.
Yes, the "View and download" link is the top right table icon. The styling for that link is discussed in this ticket: https://github.com/VEuPathDB/web-eda/issues/449. Could you implement these changes while you are working on this ticket?
Use the table overlaid with the download icon, and label it: “View and download”
include mouseover text with more information: “View and download current subset of Communities”
Ann created this icon for "View and download" (https://github.com/VEuPathDB/web-eda/issues/415)

Oh! and we want to use a new icon Ann created for My analyses

@danicahelb Thank you for your comments and suggestions 👍 I looked through several issues and corresponding PRs via your info, and I found that actually there is an unresolved issue for the table icon itself, which Ann and Dave are currently discussing in a PR (here). So, I will wait for its progress before working on View and download icon/label/tooltip etc.

Just a note - @moontrip the My Analyses icon swap in the workspace heading and the table icon for View and Download will be resolved with this PR (same PR you linked). Should be done shortly!
But wanted to let you know to prevent duplicating work :)

@moontrip we're just finalizing a few things on the above mentioned pr but you should at least have access to the icons as long as you use web-common >v0.1.8 (includes this PR).  Due to the webpack issue Dave mentioned at last week's meeting, we can't import them like normal in local dev. Instead, you'll need to yarn link a local updated version of web-common when developing with the icons.
Also in case you run into a similar issue with the mui buttons, in web-eda PR #488 I had a lot of trouble getting the icons to be aligned with the text, but finally found a solution which is implemented in the latest commits. Hope this is helpful!

@asizemore I think that the task that should be done in this issue is only to change the view and download icon to be your icon with label. So, I think you can take this issue as it is closely related to your work, #488. It is quite simple if you change this to my suggestion below adopting your solution (adding class and change startIcon) at Subsetting.tsx
from
          <button
            type="button"
            className="link"
            onClick={() => alert('Coming soon')}
          >
            <i className="fa fa-table" />
          </button>

to
          <Button
            variant="text"
            color="primary"
            //DK classes and startIcon are taken from your implementation #488
            classes={{ startIcon: iconClasses.ebrcStartIcon }}
            startIcon={<Icon className="ebrc-icon-table" />}
            component={Link}
            onClick={() => alert('Coming soon')}
          >
            View and download
          </Button>

certainly this needs to import Button and Icon components like
import { Tooltip, Button, Icon } from '@material-ui/core';
What do you think ?

@moontrip works for me! I'd be happy to take the issue, feel free to reassign it to me. Thank you also for all of the details! I'll likely reach out to you with any questions if you don't mind :)

@asizemore Thank you for your kind consideration 👍 Yep please feel free to contact me if you have any questions. I will reassign this issue to you 😃

@danicahelb please see screenshot below. Does all look good? Is there anything else that I've missed?


🚀Pro Manifest Improvements

Found that basic configurations are needed when deploying to non local cluster.
This PR adds the following

cleanupInfrastructure.ps1
a script to quickly delete installed infrastructure components,
Add zookeeper-zoo-configmap.yaml
a basic zoo.cfg file for single instance operation
init containers to configure kafka and zookeeper
setting security context to user 1001 to match bitnami containers
set default resource requests and limits for all pods.  These are arbitrary and should be set to the users specific needs.

Hello @wss-rbrennan, when I test the code, I see this error, it cannot start the kafka pod, it may be related to this change:
setting security context to user 1001 to match bitnami containers

LGTM, thank you


Also run zypper ar for local repos

While preparing the admin node, if $localrepodir_target is set, the
directory is mounted from the host, but the repositories are never
added to zypper. Refactor the function so that they are always added.

We could also make use of the fourth argument to add_mount() which would add the repositories to zypper as well, but this change seemed the least error-prone.

@s-t-e-v-e-n-k please rebase

+1

+1


Golang: add support for the gcflags attribute

This is required to disable optimizations and produce binaries that can be debugged with delve, for example:
pkgs.caddy.overrideAttrs (final: prev: {
  # upstream ldflags have "-s" "-w" and we don't want that:
  ldflags = [ "-X "github.com/caddyserver/caddy/v2.CustomVersion=${prev.version}" ];
  gcflags = [ "all=-N -l" ];
  dontStrip = true;
  dontFixup = true;
});

"-all=-N -l" disables optimization and inlining see go tool compile -help.
This is a follow up to #274503.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I've already rebased the PR but for future reference: CONTRIBUTING.md#rebasing-between-branches-ie-from-master-to-staging

Thank you for doing that @zowok, sorry I missed the part about mass rebuilds, would love to have your input on #274503#issuecomment-1872322299.

Could GOFLAGS be used instead here?
GOFLAGS = [
    "-gcflags='all=-N -l'"
];

should have the same effect, shouldn't it?

On the general matter: I'm a fan of using the Go "-s -w" flags for stripping, but I agree that this shouldn't be done on the package level. Maybe we could move it into the builder, set it by default and remove it when dontStrip = true is set? I've recently added some filtering to ldflags (https://github.com/NixOS/nixpkgs/pull/276685), I think we could work more into this direction. I can also help with this change if wanted. :)

Hello @katexochen, thanks for the idea,  it works though you have to use "'-gcflags=all=-N -l'" (i.e. put the whole thing inside 's), to work around escaping issues.

caddy = pkgs.caddy.overrideAttrs (final: prev: {
    # upstream ldflags have "-s" "-w" and we don't want that:
    ldflags = [ "-X ${importPath}.CustomVersion=${prev.version}" ];
    GOFLAGS = prev.GOFLAGS ++ [ "'-gcflags=all=-N -l'" ];
    dontStrip = true;
    dontFixup = true;
});

I would be happy if dontStrip was honored, and toggled the use of -s -w.
I saved some (horrible) code to remove the "-s" and "-w" ldflags from almost all Golang packages in Nixpkgs, so if you wish I could take on that part.

@lopter can this PR be closed, as it is already covered by GOFLAGS?

Sure, otoh I am still interested in not setting "-s" and "-w" at the package level to make debugging easier. You were saying that we could add them from pkgs/build-support/go/package.nix depending on dontStrip instead?

I've opened https://github.com/NixOS/nixpkgs/issues/346380 to discuss the stripping refactoring.


help_docs: Update documentation about notifications.

Updates 2 help articles related to notifications in Zulip:

desktop-notifications : Adds documentation for unread badge count to help article on desktop notifications.
pm-mention-alert-notifications : Generally, cleans up instructions and article structure with a focus on the Notifications triggers table in the settings modal.

Replaced by https://github.com/zulip/zulip/pull/21518.


⚠ Bofrost.de has degraded performance

In a319ab3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1228 ms

Resolved: Bofrost.de performance has improved in d6a7c59 after 13 minutes.


M3: Illumio Playbook support

Required items, please complete
Change(s):

Added playbook

Reason for Change(s):

Addition of playbook support

Version Updated:
- Not yet
Testing Completed:

Not yet

Checked that the validations are passing and have addressed any issues that are present:

Not yet

A few questions:

I have a playbook that needs to use an azure function app, how to create an arm template and createUidefn for the same?
What should be the folder structure for the function app?

Please check the commits and let me know if the folder structure can be used for deploying function app.

https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/AtlassianJiraAudit/Playbooks/Sync-CommentsFunctionApp
This is Playbook which uses Function App. You can refer this

@v-prasadboke
I have modified as per the reference file. Can you please check if you can deploy this?


Code for evaluation on Cityscape Foggy and NuScenes-Night.

Hello,
Thanks for your great work. May you release the code for evaluating on Cityscape Foggy and NuScenes-Night?

Hello, thank you for your interest in our work. The code to evaluate on both datasets is now available.


Azure b2c - Msal js - SPA - Edit profile redirects to sign in

Did anyone try 'profile edit' with MSAL with spa? If so can you please point me to some demo samples? Thanks in advance.
I was able to integrate 'SignIn', 'SignUp' and 'PasswordReset' but when integrating profile edit it did the route to 'profile edit' policy but the issue is, the application is asking to sign in again to edit the user profile.
Below is the code,
private clientApplication: Msal.UserAgentApplication;
constructor(private router: Router, private token: tokenDataService) {
this.clientApplication =
  new Msal.UserAgentApplication(
    environment.clientID,
    this.authority,
    this.authCallback.bind(this),
    {
      redirectUri: window.location.origin
    });

}
public login(): void {
this.clientApplication.loginRedirect(environment.b2cScopes);
}
public editProfile() {
let clientApp = window.msal as Msal.UserAgentApplication;
clientApp.authority = this.profileEditAuthority;
clientApp.loginRedirect(environment.b2cScopes);
}
public logout(): void {
this.clientApplication.logout();
}
public firstLogin(): string {
return this.clientApplication.getUser().idToken['newUser'];
}
public isOnline(): boolean {
return this.clientApplication.getUser() != null;
}
public getUser(): User {
return this.clientApplication.getUser();

}
public getAuthenticationToken(): Promise {
return this.clientApplication.acquireTokenSilent(environment.b2cScopes)
  .then(token => {
    this.token.set(token);
    console.log('hi')
    console.log(token);

    return token;
  }).catch(error => {
    return this.clientApplication.acquireTokenPopup(environment.b2cScopes)
      .then(token => {
        return Promise.resolve(token);
      }).catch(innererror => {
        console.error('Could not retrieve token from popup.', innererror);
        return Promise.resolve('');
      });
  });

}
private authCallback(errorDesc: any, token: any, error: any, tokenType: any) {
this.router.navigate(['/pages/home']);
if (error) {
  if (errorDesc.indexOf("AADB2C90118") > -1) {

    let clientApp = window.msal as Msal.UserAgentApplication;
    clientApp.authority = this.resetpasswordAuthority;

    clientApp.loginRedirect(environment.b2cScopes);
  }
  console.error(`${error} ${errorDesc}`);
}

}

@venkateshwj Please provide us with the link to the documentation you are referring to so we can better assist.

Hi @venkateshwj please either open this issue on a specific document or open the issue in the Azure Active Directory forum on MSDN. https://social.msdn.microsoft.com/Forums/en-US/home?forum=WindowsAzureAD
Currently there is no sample or document specifically for this. This document is the closest tutorial available. https://docs.microsoft.com/en-us/azure/active-directory-b2c/active-directory-b2c-tutorials-spa
This thread looks like a pretty similar issue that got resolved. This may lead you to the right answer.

Closing this issue as it needs more information and is better suited to MSDN, but I'm happy to continue the discussion here and keep the thread updated.


Add a 'prefix' feature along with documentation and tests for this fe…

django-sqids builds off of django-hashids, but one of the nice features of its sister package, django-hashid-field was the ability to provide a per-field prefix that was prepended to the resulting Hashids for that model field.
This pull request adds such a feature to django-sqids - an optional prefix that can be provided as an argument in the model field's implementation. For instance:
class TestModel(Model):
    sqid = SqidsField(real_field_name="id", min_length=7, prefix="P-")
    other_sqid = SqidsField(real_field_name="id", min_length=9, prefix="user_")

will result in model instances that have sqid values like "P-J3TAC2V" or "P-M2A87BX", and other_sqid values like "user_3N9CAW12V" and "user_J90MGN4RD".
Included in the pull request are:

Changes to SqidsField to implement prefixes
Tests (and test_app models) for the prefix feature (and for DRF serialization with or without a prefix)
Updates to Readme to document the new feature
A new Exception IncorrectPrefixError to help identify when the prefix is used incorrectly

Please let me know if you find this useful. I hope you will consider merging it.

Thank you for this PR and contribution, Jack!
I had a look at your proposed changes and would like to merge this after some points for discussion:

I see a conflict in the way invalid prefixes are handled. Let's take this example:
# urls.py
urlpatterns = [path("<str:sqid>/", views.detail)]
# views.py
def detail(req, sqid):
  obj = get_object_or_404(queryset, sqid=sqid)

If the given model does not use a prefix, trying to access this view with an invalid sqid results in a 404. This is the same behavior you get when using e.g. <int:pk> because of no match in the patterns. This is the result of the upstream Sqids library returning an empty list when given an invalid string as input.
If the given model however does use a prefix, trying to access the same view with an invalid one will result in a server error. I do not think this is desired behavior.
The url pattern needs to use str, because we don't know which patterns make a valid Sqid (or maybe slug or path).
An alternative approach would be to add the prefix to your patterns themselves:
urlpatterns = [
  path("api/detail/H-<sqid>/", views.house_detail),  # "H" as a sort of prefix illusion
  path(("api/detail/G-<sqid>/", views.garden_detail),  # Looks like the same route from outside
]

I previously worked on a large project before that used django-hashid-field with prefixes and I personally saw no benefit of having them, other than how they looked in URLs. Do you have a more specific use case in mind that would require more than just the prefixes in URL patterns?

This is a rather big change and the DRF tests + dependency update would be better suited in a separate commit. Would you be willing to split your commit and submit a separate PR? I also recommend you update Poetry (The newly committed lockfile was generated with 1.6.1)

Side note: this PR sparked an interesting thought in my head: It would be possible to use regular Sqids for all possible object instances while maintaining full separation: by also encoding the content type ID as the first number in the sqid! I can see this as a possible addition to this project.
def every_detail(r, sqid):
  content_type, object_id = sqids_instance.decode(sqid)
  ...

urlpatterns = path("detail/<sqid>/", every_detail)

@julianwachholz thanks for your input. I just submitted a separate PR for the DRF portion.
I will rework this PR as requested, and implement some tests for urls once I resolve that issue.

As for use-case, I really like the prefixes as a simple identifier. The main application I work on provides a suite of tools for utilities districts. Each service request starts with "SR-" Each Invoice starts with "I-", Each Serial Number starts with "S-".
The prefix just provides some quick context about what the identified we're looking at belongs to.
I'm not sure if anyone else will find it useful, but I really like being able to provide a short pseudo-random value with a bit of added context for each record my customers see.

That content-type idea is pretty cool!

@julianwachholz
Please let me know if this meets the intent.
I added a set of tests to ensure urls function correctly. Here's the current behavior - let me know if it still needs rework:

-
Model without prefix
Model with prefix

prefix not provided
✅
IncorrectPrefixError is raised

wrong prefix provided
N/A (404)
IncorrectPrefixError is raised

prefix provided
404
✅

Note: I did leave the DRF dev requirement in place, since the tests in this PR check that prefixes serialize correctly, but I moved all DRF-only tests to #2.

I believe option 1 to be the least surprising and most in line with how Django's path converters work.
Imagine registering a custom converter for every SqidsField, the expected behavior as documented here would be to raise a ValueError, which will in turn be treated as a 404.

That makes sense. Updated the PR to behave more in-line with the expected results, as requested.

-
Model without prefix
Model with prefix

prefix not provided
✅
404

wrong prefix provided
404
404

prefix provided
404
✅

That makes sense. Updated the PR to behave more in-line with the expected results, as requested.

-
Model without prefix
Model with prefix

prefix not provided
✅
404

wrong prefix provided
404
404

prefix provided
404
✅


Fixed md format

Based on suggestions by Docs Build

Docs Build status updates of commit 388d49c:
:clock10: Full build: the average full build time is 11 min(s) 56 sec(s), this is based on the last 787 build(s) for this repository.

Docs Build status updates of commit 388d49c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/docs-conceptual/sharepoint-online/sharepoint-online-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/docs-conceptual/sharepoint-server/sharepoint-server-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

spmt/docs-conceptual/index.md
:white_check_mark:Succeeded
View (spmt-ps)

spmt/docs-conceptual/intro.md
:white_check_mark:Succeeded
View (spmt-ps)

spmt/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (spmt-ps)

staffhub/docs-conceptual/index.md
:white_check_mark:Succeeded
View (staffhub-ps)

staffhub/docs-conceptual/intro.md
:white_check_mark:Succeeded
View (staffhub-ps)

staffhub/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (staffhub-ps)

teams/docs-conceptual/index.md
:white_check_mark:Succeeded
View (teams-ps)

teams/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ValueError in exp_T2D/predict_colnet.py

Hi,
Thanks for your work.
When I run the AAAI19/exp_T2D/predict_colnet.py script, I run into error message:
Traceback (most recent call last):
File "predict_colnet.py", line 122, in
p = predict(X, classifier)
File "predict_colnet.py", line 78, in predict
test_p = sess.run(probabilities, {input_x: test_x, dropout_keep_prob: 1.0})
ValueError: Cannot feed value of shape (930, 60, 300, 1) for Tensor u'input_x:0', which has shape '(?, 15, 300, 1)'
Could you please resolve this issue?
Thanks.

Hi,
Maybe you can check whether the size of synthetic column is consistent in training and prediction.
Thanks.


Feature request: Add extension support to makeMarkdown

Since the package is a bidirectional converter to/from Markdown and HTML, extension support should be available for both makeHtml and makeMarkdown.
As a user eagers to customize the behavior of makeMarkdown, now I have to accomplish this in a dirty method.

That is indeed planned for the next version, 3.0


Increase virt-launcher memory overhead

What this PR does / why we need it:
Recent tests show that 75Mb is not enough.
virt-launcher memory overhead may vary during execution
depending on many factors, such as when the garbage collector runs.
Increase the virt-launcher overhead to avoid OOM kill.
Here is a report of some collected data

Here is the document where we are collecting data from the current running test:
https://docs.google.com/spreadsheets/d/1Dd_bRffT6hr1fw_rlh7hqOFRFbVQWaCOyU7IZVwGDVo/edit?usp=sharing
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Increase virt-launcher memory overhead

/cc

/retest-required

/lgtm
Thanks a lot @fossedihelm!

/approve

/retest

/cherry-pick release-0.53

Interesting to see the drop in the case of 2 VMs. It might be beneficial to understand where the memory went.
FYI @rthallisey @marceloamaral  This might be interesting for SIG-scale

virt-launcher memory overhead may vary during execution
depending on many factors, such as when the garbage collector runs.

Can you explain the test further?  How many VMIs, how quickly you create/delete, and how many nodes?

/retest-required

virt-launcher memory overhead may vary during execution
depending on many factors, such as when the garbage collector runs.

Can you explain the test further?  How many VMIs, how quickly you create/delete, and how many nodes?

@rthallisey The cluster has 3 nodes, in which there are 6 VMs (2 per nodes). The VMs were never deleted, always running from their creation. I can also tell you that the 2 VMs with similar drop live in the same node.

The cluster has 3 nodes, in which there are 6 VMs (2 per nodes). The VMs were never deleted, always running from their creation. I can also tell you that the 2 VMs with similar drop live in the same node.

This sounds like a bug?  Or are you saying the mem overhead change wasn't made previously wasn't enough?

The cluster has 3 nodes, in which there are 6 VMs (2 per nodes). The VMs were never deleted, always running from their creation. I can also tell you that the 2 VMs with similar drop live in the same node.

This sounds like a bug? Or are you saying the mem overhead change wasn't made previously wasn't enough?

The last attempt to capture our overhead was done based on looking on RSS without running the VM for some time or trying to understand what features might be adding and increasing the overhead.  We had observations indicating we might be leaking memory or just underestimating the overhead. This is the reason why we started this experiment and are looking to better capture the overhead.

Sounds good.  Ya I agree @xpivarc this would be a good topic for SIG-Scale, please raise it in the next meeting.  Ideally, we should try to find something we can put into CI to catch unexpected mem increases.


[BUG][Main] Fix flaky test cases for bundled OpenSearch Dashboards plugins

What is the bug?
We have noticed flaky test when running the test cases for Dashboards plugins. To collect enough data points for analysis, we have set up multiple Github workflows against different browsers every three hours. By analyzing the past 10 failed runs, below is a summary of the frequency of failure.

Test File
Plugin Owner
Failed Times (out of 10)

plugins/anomaly-detection-dashboards-plugin/create_detector_spec.js
anomaly-detection-dashboards
8

plugins/anomaly-detection-dashboards-plugin/detector_list_spec.js
anomaly-detection-dashboards
2

plugins/index-management-dashboards-plugin/policies_spec.js
index-management-dashboards
6

plugins/observability-dashboards/trace_analytics_traces.spec.js
observability-dashboards
1

plugins/anomaly-detection-dashboards-plugin/overview_spec.js
anomaly-detection-dashboards
3

plugins/anomaly-detection-dashboards-plugin/detector_configuration_spec.js
anomaly-detection-dashboards
1

plugins/observability-dashboards/panels.spec.js
observability-dashboards
3

plugins/observability-dashboards/app_analytics.spec.js
observability-dashboards
2

plugins/reports-dashboards/03-details.spec.js
reports-dashboards
1

How can one reproduce the bug?
https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions?query=event%3Aschedule
What is the expected behavior?
We shall see all test cases pass.
What is your host/environment?

OS: Github hosted Ubuntu latest

Do you have any screenshots?
Every workflow will upload the video. Use this as an example https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/1987477734
Do you have any additional context?
Note that when running from Jenkins Docker env, we are seeing more flaky test cases due to the less empowered execution env. We will consolidate and share the results later. E.g a recent Jenkins run has the following result (You may need to save the file locally or use curl to view it)
https://ci.opensearch.org/ci/dbc/integ-test-opensearch-dashboards/1.3.0/1511/linux/x64/test-results/d550538e3386451fbcccea7c0ecf3d46/integ-test/functionalTestDashboards/with-security/test-results/stdout.txt
https://ci.opensearch.org/ci/dbc/integ-test-opensearch-dashboards/1.3.0/1511/linux/x64/test-results/d550538e3386451fbcccea7c0ecf3d46/integ-test/functionalTestDashboards/without-security/test-results/stdout.txt
https://ci.opensearch.org/ci/dbc/integ-test-opensearch-dashboards/1.3.0/1511/linux/arm64/test-results/f83b7e7b4ef14a4e9b5cc80a00745347/integ-test/functionalTestDashboards/with-security/test-results/stdout.txt
https://ci.opensearch.org/ci/dbc/integ-test-opensearch-dashboards/1.3.0/1511/linux/arm64/test-results/f83b7e7b4ef14a4e9b5cc80a00745347/integ-test/functionalTestDashboards/without-security/test-results/stdout.txt

cc @dblock @peterzhuamazon @seraphjiang

@ohltyler for AD
@bowenlan-amzn for ISM
@joshuali925 for observability
@davidcui1225 for reports

@joshuali925 shared a tip to handle the interaction with dropdown component https://github.com/opensearch-project/opensearch-dashboards-functional-test/blob/c76fdc95db7d864b5616d947469589d7ef1c637d/cypress/integration/plugins/observability-dashboards/notebooks.spec.js#L272

AD flakiness was cleaned up in #144 besides one test that may be related to the Cypress upgrade (link)

All tests marked as skip in https://github.com/opensearch-project/opensearch-dashboards-functional-test/pull/131/files#diff-e24d33d1246206d9a3c5dc03e4bd5ce9e6ce6e8ed3a93d5bf5b3c5b134fa9707 are no longer skipped. Seems have been fixed in later release. This ticket can be closed.

@tianleh We've run all tests based 2.11 branch for above plugins. After 5 attempts, no failed cases found.
Here is the job details:

Without security enabled
AD: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7244651329/job/20564369091
Reports: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7244651322/job/20566660833
Observability: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7244651334/job/20567098056
Index Management: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7244651364/job/20566657988
With security enabled
AD: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7554212666/job/20599961908?pr=1000
Reports: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7554212687/job/20599957422?pr=1000
Observability: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7554212668/job/20599959519?pr=1000
Index Management: https://github.com/opensearch-project/opensearch-dashboards-functional-test/actions/runs/7554212684/job/20599956092?pr=1000

The test cases failed in the every 3 hours release sign off workflow may caused by share the same OpenSearch or OpenSearch Dashboards. Some tests may forgot to reset the operated data. There are two ways to make sure each functional tests of specific plugin run in isolate env:

Modify the workflow, run new OpenSearch and OpenSearch Dashboards before plugin tests start.
Inspired by this remote functional tests RFC, trigger functional tests in their own workflows.
These two ways can provide an isolate OpenSearch and OpenSearch Dashboards for functional tests. It may reduce the differences between the every 3 hours workflow and the PR workflow.

@tianleh Since there are no failed cases in these plugins, we will close this issue first. Will create new issue to track flaky tests case by case.


QA: test Rancher admin gets notified when a new operator update is available

It was easy to test it manually, however, I tried to automate and it's not obvious... because it takes a random time to view the update pop-up in the Rancher UI. Even a reload of the page does not make it pop up. It will add flakiness to our test for sure...

Manual test done


Validate endpoint names

Follow up to https://github.com/dotnet/aspire/pull/1691
Is anything else that has a name that should be validated? env vars are connection strings can both map to real world configuration so I left them out.
Microsoft Reviewers: Open in CodeFlow

I'm so worried about this change. It's going to break so many existing applications. Did we validate that new projects will be fine for any project name?

I'm so worried about this change. It's going to break so many existing applications. Did we validate that new projects will be fine for any project name?

What are some common examples of apps breaking?
The default name for endpoints is the scheme (http/https) so they're not impacted. And specified names will mostly be alpha numeric. The most common cause of failure would be underscores but they can be replaced with hyphens. Changing names to be valid should be pretty simple.

@JamesNK every app that uses dots in the resource names. After this change, we should update eshop and all of our samples (do we have a preview3 branch yet?).

eshop uses dashes - https://github.com/dotnet/eShop/blob/3b49f61a888656b038b5f044b0d4c5096fc9f073/src/eShop.AppHost/Program.cs
And not a problem in the samples repo - https://github.com/search?q=repo%3Adotnet%2Faspire-samples AddProject&type=code

Lots of people are using . as separators, so we'll tell them to use - hopefully in the error message 😄

The error message is very clear about what you're doing wrong and need to follow to be right:
https://github.com/dotnet/aspire/blob/7794997f667368b4895bd9fbf1d0bd2d19f35a7f/src/Aspire.Hosting/ApplicationModel/Resource.cs#L40-L79


python311Packages.pex: 2.3.2 -> 2.4.0

Changelog: https://github.com/pantsbuild/pex/releases/tag/v2.4.0
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 319905 run on x86_64-linux 1

4 packages built:

python311Packages.pex
python311Packages.pex.dist
python312Packages.pex
python312Packages.pex.dist


Create logos/ logo theme for tutorials

Taken from retreat pop-up meeting:
Create a theme / CI for the tutorial logos and create logos for the existing ones.
https://substrate.dev/en/tutorials
// cc Zsofia

cc @meheszsofi

Closing this because comms has done a lot with branding since the retreat. cc @thisisvilla for context.


Update certificate parsing for certificate chains with multiple certificates

Fix for https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/issues/49.
Update the certificate parsing so that it works when certificate chains have multiple certificates within them.
@carlnordenfelt Can you review & merge please?

Hi,
I'll take a look at the PR as soon as I get back to civilization which is
on thursday.
On Mon, Jul 18, 2016, 16:47 Craig Blaszczyk<EMAIL_ADDRESS>wrote:

Fix for #49
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/issues/49
.
Update the certificate parsing so that it works when certificate chains
have multiple certificates within them.
@carlnordenfelt https://github.com/carlnordenfelt Can you review &
merge please?
You can view, comment on, or merge this pull request online at:
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50
Commit Summary

Update certificate parsing for certificate chains with multiple
certificates

File Changes

M lib/service/util/certificate-parser.js
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50/files#diff-0
(18)
M tests/unit/service/util/certificate-parser-unit-test.js
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50/files#diff-1
(55)

Patch Links:

https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50.patch
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEzRoZTI7ZdQ-MuZxQjHSAjhozAmSW1kks5qW5HUgaJpZM4JOyi_
.

I've merged but won't be able to  relate a new package until tomorrow. In the meantime you can create your own package and deploy if you want.

2.0.1 has been released with the fix.

Cheers!
Em qui, 21 de jul de 2016 17:21, Carl Nordenfelt<EMAIL_ADDRESS>escreveu:

2.0.1 has been released with the fix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/carlnordenfelt/aws-api-gateway-for-cloudformation/pull/50#issuecomment-234306395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0qSmR4MoOvgWhPumX8zbfPgzFZcQJFks5qX5yigaJpZM4JOyi_
.


(NewFeature) Request for stitching/merging height map tiles.

Hello,
Is there a way to add support for stitching/merging height maps that where already tiled, that also happen to overlap by any desired pixel number.
Lets say i have a section of a map in tiles that is formed out of 8x8 tiles, each tile is a 257x257 pixels, meaning they must overlap by one pixel, once merged/stitched you get a 2049x2049 final map.
First column would be from bottom left at section_00-00.raw going up vertically up to section_00-07.raw
Followed by section_01-00.raw -> section_01-07.raw to the right, overlapping as well by one pixel, loading vertically as well, etc.
The format of the height map tiles can be RAW 8-bit, 16-bit or 32-bit float in my case, including PNG, TIFF, etc.
The ability to change direction of the tiles to load, be it vertically up/down, bottom left or bottom right corner, horizontally left/right, bottom left or or bottom right corner.
To change the pixel number to overlap.
To select for import what format the map tiles are for RAW.
As well the ability to import a large map to get it ready to tile back to a desired pixel resolution, if required to be overlapped or not, if yes by how many pixels is required and save it to desired/needed format, be it RAW 8, 16, 32-bit, PNG, TIFF, etc.
I can provide a sample if need be.

Interesting. Are these height maps that you are working with, or are they
just pictures that you are trying to stitch together?
On Fri, Jun 24, 2022 at 4:09 PM Unknown @.***> wrote:

Hello,
Is there a way to add support for stitching/merging height maps that where
already tiled, that also happen to overlap by any desired pixel number.
Lets say i have a section of a map in tiles that is formed out of 8x8
tiles, each tile is a 257x257 pixels, meaning they must overlap by one
pixel, once merged/stitched you get a 2048x2048 final map.
First column would be from bottom left at section_00-00.raw going up
vertically up to section_00-07.raw
Followed by section_01-00.raw -> section_01-07.raw to the right,
overlapping as well by one pixel, loading vertically as well, etc.
The format of the height map tiles can be RAW 8-bit, 16-bit or 32-bit
float in my case, including PNG, TIFF, etc.
The ability to change direction of the tiles to load, be it vertically
up/down, bottom left or bottom right corner, horizontally left/right,
bottom left or or bottom right corner.
To change the pixel number to overlap.
To select for import what format the map tiles are for RAW.
As well the ability to import a large map to get it ready to tile back to
a desired pixel resolution, if required to be overlapped or not, if yes by
how many pixels is required and save it to desired/needed format, be it RAW
8, 16, 32-bit, PNG, TIFF, etc.
I can provide a sample if need be.
—
Reply to this email directly, view it on GitHub
https://github.com/gordon-vart/opensource/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIO5EOF3UWCCYEW7GI3D3TVQYPYXANCNFSM5ZY76WQQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes they are actual terrain height maps.
Each and every one of them is a actual full 32-bit floating point RAW height map.
I only meant in my case they are float 32-bit RAW, and would be extremely helpful to support other bit depth as RAW or even PNG, TIFF, etc
To have support for various needs not just Unreal Engine specs. which is if i am not mistaken only PNG at 16-bit?
I have searched high and low on the www, came up empty handed until i found your GitHub, also the only other interesting links i found that could potentially be useful are the following, they include code example:
https://stackoverflow.com/questions/16433122/how-can-you-stitch-multiple-heightmaps-together-to-remove-seams
https://thegroundtruth.blog/2021/02/28/creating-maps-by-stitching-map-tiles/
https://www.rastertek.com/dx11ter08.html

Just curious, are you doing game development? Or are you doing something
else with these height Maps? if you send me an example set of files, I
think I know what you're looking for. I will play around with it. I assume
you are trying to stitch together the edges of the height map so that you
don't have gaps in between them, correct?
On Fri, Jun 24, 2022 at 10:32 PM Unknown @.***> wrote:

Yes they are actual terrain height maps.
Each and every one of them is a actual full 32-bit floating point RAW
height map.
I only meant in my case they are float 32-bit RAW, and would be
extremely helpful to support other bit depth as RAW or even PNG, TIFF, etc
To have support for various needs not just Unreal Engine specs. which is
if i am not mistaken only PNG at 16-bit?
I have searched high and low on the www, came up empty handed until i
found your GitHub, also the only other interesting links i found that could
potentially be useful are the following, they include code example:
https://stackoverflow.com/questions/16433122/how-can-you-stitch-multiple-heightmaps-together-to-remove-seams
https://thegroundtruth.blog/2021/02/28/creating-maps-by-stitching-map-tiles/
https://www.rastertek.com/dx11ter08.html
—
Reply to this email directly, view it on GitHub
https://github.com/gordon-vart/opensource/issues/4#issuecomment-1166182274,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIO5EO7H7U2HUHPSPWD5QTVQZ4UTANCNFSM5ZY76WQQ
.
You are receiving this because you commented.Message ID:
@.***>

I guess i can say i am a 3D enthusiast, over all, anything that is related to CGI including 3D art, currently trying to learn how to deal with various types of terrain maps, RAW maps seem to be the best option in terms of lossless quality, of course other 3D editors might be limited to lets say PNG 16-bit, which the preview sample is in, attached.
The RAW tiles might require to be inverted on the Y axis (flipped vertically), for example on the tiles i provided the last float value of the first tile will match the first value of the adjacent tile, and once overlapped in this case by one pixel they match keeping the values accurate and proper scale across entire height map, with out any type of seams or errors as well.
I am basically looking for ways to automate the loading/importing and generating/exporting of thousands of tiles, or whole height map/albedo/normal/etc, old or new map projects that can have features described above in initial post, so far found zero such tools to aid in such tedious and time consuming tasks.
https://mega.nz/file/TUlzSTIb#s550HtsKd1qty5LGjzu1WmW8zPHDa58G1dnMnuCm7QI
I also want to learn how to import such maps in Unreal Engine, too, you seem to have some very interesting videos, and that should help in testing, as i want to see how that would work for my projects.

I did some looking last night and this is not a trivial task. It's going to
take some time, and to be honest I've got too many other things going on
right now. I may look at this in the future, but I think you're going to
have to take on this task yourself for the time being and thanks for the
interest.
On Tue, Jun 28, 2022 at 5:49 PM Gordon Vart @.***> wrote:

understood. The files that you sent me are 32-bit Raw, right? Anyway, I'm
not exactly sure why I'm doing this because I don't have the time but I
am intrigued. LOL
I'm not making any promises oh, but let me see if I can come up with
something. You would make a pretty decent product manager.
On Sun, Jun 26, 2022 at 2:41 PM Unknown @.***> wrote:

I guess i can say i am a 3D enthusiast, over all, anything that is
related to CGI including 3D art, currently trying to learn how to deal with
various types of terrain maps, RAW maps seem to be the best option in terms
of lossless quality, of course other 3D editors might be limited to lets
say PNG 16-bit, which the preview sample is in, attached.
The RAW tiles might require to be inverted on the Y axis (flipped
vertically), for example on the tiles i provided the last float value
of the first tile will match the first value of the adjacent tile, and once
overlapped in this case by one pixel they match keeping the values accurate
and proper scale across entire height map, with out any type of seams or
errors as well.
I am basically looking for ways to automate the loading/importing and
generating/exporting of thousands of tiles, or whole height
map/albedo/normal/etc, old or new map projects that can have features
described above in initial post, so far found zero such tools to aid in
such tedious and time consuming tasks.
https://mega.nz/file/TUlzSTIb#s550HtsKd1qty5LGjzu1WmW8zPHDa58G1dnMnuCm7QI
I also want to learn how to import such maps in Unreal Engine, too, you
seem to have some very interesting videos, and that should help in testing,
as i want to see how that would work for my projects.
—
Reply to this email directly, view it on GitHub
https://github.com/gordon-vart/opensource/issues/4#issuecomment-1166626488,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIO5EPO224GB6YHGGPPRPTVRCXADANCNFSM5ZY76WQQ
.
You are receiving this because you commented.Message ID:
@.***>

Thanks for taking a look at it.

understood. The files that you sent me are 32-bit Raw, right? Anyway, I'm
not exactly sure why I'm doing this because I don't have the time but I am
intrigued. LOL
I'm not making any promises oh, but let me see if I can come up with
something. You would make a pretty decent product manager.
On Sun, Jun 26, 2022 at 2:41 PM Unknown @.***> wrote:

I guess i can say i am a 3D enthusiast, over all, anything that is related
to CGI including 3D art, currently trying to learn how to deal with various
types of terrain maps, RAW maps seem to be the best option in terms of
lossless quality, of course other 3D editors might be limited to lets say
PNG 16-bit, which the preview sample is in, attached.
The RAW tiles might require to be inverted on the Y axis (flipped
vertically), for example on the tiles i provided the last float value
of the first tile will match the first value of the adjacent tile, and once
overlapped in this case by one pixel they match keeping the values accurate
and proper scale across entire height map, with out any type of seams or
errors as well.
I am basically looking for ways to automate the loading/importing and
generating/exporting of thousands of tiles, or whole height
map/albedo/normal/etc, old or new map projects that can have features
described above in initial post, so far found zero such tools to aid in
such tedious and time consuming tasks.
https://mega.nz/file/TUlzSTIb#s550HtsKd1qty5LGjzu1WmW8zPHDa58G1dnMnuCm7QI
I also want to learn how to import such maps in Unreal Engine, too, you
seem to have some very interesting videos, and that should help in testing,
as i want to see how that would work for my projects.
—
Reply to this email directly, view it on GitHub
https://github.com/gordon-vart/opensource/issues/4#issuecomment-1166626488,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIO5EPO224GB6YHGGPPRPTVRCXADANCNFSM5ZY76WQQ
.
You are receiving this because you commented.Message ID:
@.***>

Hello,
First and foremost, thank you for getting back to me with an intrigued interest to at least discover what can be truly done about my initial feature request, very much appreciated.
I know it may seem like a trivial task as previously described, yet if supported the mentioned features, a time saver with out any doubt, as the technology progresses at a rapid phase, this will come handy for many 3D artists, new and old alike that want to learn, or currently working in the filed that would require such specific dedicated software.
To answer your question, yes they are indeed 32-bit RAW height maps.
Regardless of what you come up with, i am grateful you took a genuine interest in it, for that alone, thank you once again.
I suppose I too am intrigued, by the following: "You would make a pretty decent product manager."

Hello, happy 2024, hope all is well.
Was wondering if you happen to have had a chance at all in looking in to the options i had mentioned previously by chance?

Hello and happy 2024 to you as well. I have not looked into that. I have
been so busy with work and I kind of stopped doing unreal development so
this fell far down on the priority list. I will take a look and see if
there is something quick I can do. What is your use case?
On Wed, Jan 31, 2024 at 7:05 PM Unknown @.***> wrote:

Hello, happy 2024, hope all is well.
Was wondering if you happen to have had a chance at all in looking in to
the options i had mentioned previously by chance?
—
Reply to this email directly, view it on GitHub
https://github.com/gordon-vart/opensource/issues/4#issuecomment-1920282790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIO5EOMKIOL6LHQTM2FWTLYRLS4PAVCNFSM5ZY76WQ2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJSGAZDQMRXHEYA
.
You are receiving this because you commented.Message ID:
@.***>

Thank you for your response.
My use case is to use old projects where there are no more full size height maps, and use already made tiles that are in either RAW 32-bit, 16-bit, or EXR, PNG, TIFF, to be able to stitch them back to that full size they have, overlap them by 1 pixel or more if needed, and to also support their diffuse, normal, mask, alpha, etc maps to stitch them as well back to their original full size.
Later on to export them in to UE5.x supported formats/sizes, would be current/main example.
So far i have yet to find a solution out there free or paid that can do batch the way your tool does but to export already made full height maps to tiles ones such as hundreds, thousands or even tens of thousands of height maps, to stitch back in to one, all based on options i described in OP.

Sorry if i come across as a broken record by now, has anything at all been considered for support of the mentioned features from before?

I appreciate the interest, but I do not have the time to invest in this.
The code was made open source so that others could take and improve because
I do not have the time to invest in making things better. Good luck.
On Mon, May 6, 2024 at 8:41 PM Unknown @.***> wrote:

Sorry if i come across as a broken record by now, has anything at all been
considered for support of the mentioned features from before?
—
Reply to this email directly, view it on GitHub
https://github.com/gordon-vart/opensource/issues/4#issuecomment-2097213778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIO5EOE2HZWFV2FF5RT75DZBAWMVAVCNFSM5ZY76WQ2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBZG4ZDCMZXG44A
.
You are receiving this because you commented.Message ID:
@.***>


WW-28612 - new config host_url for EC

adding new config for EC to allow ec to load assets and requests from absolute URL instead of relative

@asurin this is the change to Evercookie to allow us to set a host url so our analytics scripts look at a central server for these evercookies.  We'll be overriding the host_url in ww-ng


0.2 Add methodology discussion

Answer the following questions in the report:
Do the authors clearly describe and justify the methodology used in the data analysis? Do the authors communicate any assumptions or limitations of their methodologies?

https://github.com/miniatureseal/dsci-310-group-10-GCC/pull/187

Done with https://github.com/miniatureseal/dsci-310-group-10-GCC/pull/187


CopyFilesOverSSH cleanTargetFolder does not delete hidden files

Feature?
Type: CopyFilesOverSSH cleanTargetFolder does not delete hidden files
Enter Task Name: CopyFilesOverSSHV0
list here (V# not needed):
https://github.com/microsoft/azure-pipelines-tasks/tree/master/Tasks/CopyFilesOverSSHV0
Environment

Server
Agent Private:

Issue Description
When using the CopyFilesOverSSH task with the cleanTargetFolder, the hidden files in the target folder are not clean on linux.
On linux the task performs the command 'rm -rf *' which does delete regular files but not hidden ones starting with '.'.
In my opinion or the file pattern must be adapted to include hidden files, or this is done on purpose and must be documented.
Br.

Hi @boubou191911, we can not start working on this issue right now, but we will be back as soon as we can.


File corrupt error when run create_data.py in nuScenes dataset

Since the nuscnes dataset is so hudge. It cost me 3 days to untar it and cost me 3 TB storage space.
When I run the create_data.py as the mm3d official tutorial : python tools/create_data.py nuscenes --root-path ./data/nuscenes --out-dir ./data/nuscenes --extra-tag nuscenes
It threw an error like below:
------------------------error slash-----------------------------
'NoneType' object has no attribute 'shape'
File "/home/benzlife/mmdetection3d/tools/data_converter/nuscenes_converter.py", line 373, in export_2d_annotation
(height, width, _) = mmcv.imread(cam_info['data_path']).shape
File "/home/benzlife/mmdetection3d/tools/create_data.py", line 85, in nuscenes_data_prep
nuscenes_converter.export_2d_annotation(
File "/home/benzlife/mmdetection3d/tools/create_data.py", line 250, in
nuscenes_data_prep(
AttributeError: 'NoneType' object has no attribute 'shape
------------------------error slash-------------------------------
Then I find the specific cam_info['data_path'], it's './data/nuscenes/samples/CAM_BACK/n008-2018-07-26-12-13-50-0400__CAM_BACK__1532622015687558.jpg', and found that this jpg file is accually corrupted.
So question is : I'm not sure if this file corruption is caused by my unproperly untar or something else. If my untar process has something wrong, how to find the unproperly untared file becuase I don't want to untar whole dataset again.
Or someone may have some other advices?
The whole create_data.py run log is below
-----------------------log slash----------------------------------
Loading NuScenes tables for version v1.0-trainval...
23 category,
8 attribute,
4 visibility,
64386 instance,
12 sensor,
10200 calibrated_sensor,
2631083 ego_pose,
68 log,
850 scene,
34149 sample,
2631083 sample_data,
1166187 sample_annotation,
4 map,
Done loading in 28.431 seconds.
Reverse indexing ...
Done reverse indexing in 6.3 seconds.
total scene num: 850
exist scene num: 850
train scene: 700, val scene: 150
[>>>>>>>>>>>>>>>>>>>>>>>] 34149/34149, 20.3 task/s, elapsed: 1678s, ETA:     0s
train sample: 28130, val sample: 6019
Loading NuScenes tables for version v1.0-trainval...
23 category,
8 attribute,
4 visibility,
64386 instance,
12 sensor,
10200 calibrated_sensor,
2631083 ego_pose,
68 log,
850 scene,
34149 sample,
2631083 sample_data,
1166187 sample_annotation,
4 map,
Done loading in 41.962 seconds.
Reverse indexing ...
Done reverse indexing in 6.4 seconds.
[>>>>                      ] 5278/28130, 7.0 task/s, elapsed: 758s, ETA:  3281sTraceback (most recent call last):
File "tools/create_data.py", line 250, in
nuscenes_data_prep(
File "tools/create_data.py", line 85, in nuscenes_data_prep
nuscenes_converter.export_2d_annotation(
File "/home/benzlife/mmdetection3d/tools/data_converter/nuscenes_converter.py", line 373, in export_2d_annotation
(height, width, _) = mmcv.imread(cam_info['data_path']).shape
AttributeError: 'NoneType' object has no attribute 'shape'
-----------------------log slash----------------------------------
Thank you.

@benz725 Thanks for your feedback.

The cause of  'NoneType' object has no attribute 'shape' may be the path of the image is wrong. We'll appreciate it if you could create a PR to fix this bug?
We'll upload the annotation files generated offline in the main branch today and you could just download the annotation file.

@benz725 Thanks for your feedback.

The cause of  'NoneType' object has no attribute 'shape' may be the path of the image is wrong. We'll appreciate it if you could create a PR to fix this bug?
We'll upload the annotation files of all 3d datasets generated offline in the main branch today and you could just download the annotation file.

Thank you.

There is a img file in that path, but the img is corrupted, so the return of the mmcv.imread().shape is None.
I don't understand what "PR" is as you just said. But I modified the code to record the corrupted file paths as below:
---------------------------code-----------------------------------
try:
(height, width, _) = mmcv.imread(cam_info['data_path']).shape
except:
with open('corrupt_files.txt', 'a') as f1:
f1.write(cam_info['data_path']+'\n')
print(cam_info['data_path'] + ' corrupt error.')
continue
----------------------------code----------------------------------
I'll appreciate if you remind me when I can download the offline files.


Return  RECONSUME_LATER   when I consume the message，but the trackType of the msg is  CONSUMED

The issue tracker is ONLY used for bug report(feature request need to follow RIP process). Keep in mind, please check whether there is an existing same report before your raise a new one.
Alternately (especially if your communication is not a bug report), you can send mail to our mailing lists. We welcome any friendly suggestions, bug fixes, collaboration and other improvements.
Please ensure that your bug report is clear and that it is complete. Otherwise, we may be unable to understand it or to reproduce it, either of which would prevent us from fixing the bug. We strongly recommend the report(bug report or feature request) could include some hints as the following:
BUG REPORT

Please describe the issue you observed:

What did you do (The steps to reproduce)?

Return  RECONSUME_LATER   when I consume the message，but the trackType of the msg is  CONSUMED

What did you expect to see?

I think if  the comsumer system Return  RECONSUME_LATER   after consumed  the message，
the trackType of the msg should be NOT_CONSUME_YET

What did you see instead?

Please tell us about your environment:

Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):

FEATURE REQUEST

Please describe the feature you are requesting.

Provide any additional detail on your proposed use case for this feature.

Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

If there are some sub-tasks using -[] for each subtask and create a corresponding issue to map to the sub task:

sub-task1-issue-number: sub-task1 description here,
sub-task2-issue-number: sub-task2 description here,
...

Generally, return CONSUME_LATER means there is no new messages or to decrease consume speed. Message is CONSUMED meaning is apparent. It seems these two statuses have no relations.


Add ml-common into 2.3 manifests

Signed-off-by: Xun Zhang<EMAIL_ADDRESS>Description
Add ML-common into 2.3 manifests
Issues Resolved
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2580 (0e0ff65) into main (80eb0bf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2580   +/-   ##
=========================================
  Coverage     94.41%   94.41%
  Complexity       28       28
=========================================
  Files           219      219
  Lines          4462     4462
  Branches         29       29
=========================================
  Hits           4213     4213
  Misses          243      243
  Partials          6        6

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Says on the iceConnectionState that is connected but does not run the onaddstream function

Hey,
onaddstream not being fired.
I have a code like that that runs the ice connection state change and says that the this.pc.iceConnectionState is completed but the function this.pc.onaddstream never runs
I've tried refactoring the code and spliting in two components but i got the same behaviour
this.pc.oniceconnectionstatechange = () => {
  console.log('ICE STATE', this.pc.iceConnectionState);
  if (
    this.pc.iceConnectionState === 'failed' ||
    this.pc.iceConnectionState === 'disconnected' ||
    this.pc.iceConnectionState === 'closed'
  ) {
    console.log('Failed');
  } else if (this.pc.iceConnectionState === 'checking') {
    console.log('Checking');
  } else {
    this.setState({
      success: true,
    });
    console.log('Success');
  }
};

this.pc.onaddstream = e => {
  console.log('Stream event', e);
}

The expected result is to log the stream event when the event is completed.

Well, are you exchanging SDPs? Are you calling set{Local,Remote}Description? When? You example above should not trigger onaddstream.

@saghul Here is how am i doing the Video component
Heres how im doing the component
OBS: I'm using pubnub to send and receive messages and redux saga to handle the messaging
1 - I the component mounts and receives by route params a calling: boolean, callingTo: string and when its calling he gets the description and send a calling signal for the user
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L166
2 - When the user answers a call he receives a calling: false and the caller username on the callingTo: string route param and he gets the offer sdp sets the remote description that he received from pubnub, and after he creates a answer using the createAnswer method and seed to the other peer using pubnub.
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L200
3 - The caller receives the pubnub message and sets the description on componentDidUpdate and also sets the remote answer and send a message to the user set the iceCandidate that has been received from the onicecandidate
Receiving the description
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L116
Setting the description
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L255
onicecandidate method
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L228
4 - The other peer receives the signal to set the ice candidate and add the ice candidate
Receiving the ice candidate
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L108
Adding the ice candidate
https://gist.github.com/LFSCamargo/3e03c53c538206ae42d01b6f208791be#file-video-js-L276
I think thats all the flux that i've made with pubnub and react-native-webrtc
Here's how i get the pubnub messages
https://gist.github.com/LFSCamargo/ed4a6481135540c7a76c0e71eb340962#file-presencemanager-js-L98

Correct me if I'm wrong, but onaddstream should only fire when remote peer calls peerConnection.addStream on its end. SDP offer/answer exchange is not enough to trigger this callback.

Both need to happen. When remote adds the stream on their end, onnegotiationneeded will fire, so SDP O/A needs to happen. Once that happens, the other user will get onaddstream.

Should anyone hit this problem, I went through it before. Please make sure: did you addStream BEFORE you setRemoteDescription? If you didn't, onaddstream will never be triggered. OnAddStreamEvent is triggered after setRemoteDescription and it will check if the sdp in the remoteDescription contains a stream.

@saghul
i am not much idea about what i am doing wrong but in my cash i am getting always iceConnectionState first checking and then after some time its return failed. can you please guide me to solve the issues


[BUG]

Describe the bug
App Crashed when run AssetEntity.file or AssetEntity.loadFile
To Reproduce
Steps to reproduce the behavior:

获取相册内的图片，然后获取到图片的AssetEntity
然后通过AssetEntity获取file
File file = await assetEntity.loadFile(isOrigin: false).catchError((error){
print(error);
});
App crashed

Expected behavior
如果全都能够获取到图片名称，类似于image_1234.HEIC，这种就不会崩溃
但是如果获取titleAsync返回的值是一长串id的，就无法获取到file,如果选择的图片较多app就会崩溃
Screenshots
If applicable, add screenshots to help explain your problem.
Flutter version

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Log
If applicable, add logs to help explain your problem.
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start
get title start

thread #101, queue = 'com.apple.root.user-initiated-qos', stop reason = EXC_RESOURCE RESOURCE_TYPE_MEMORY (limit=2098 MB, unused=0x0)
frame #0: 0x00000001a5bebf84 vImagevConvert_420Yp8_CbCr8ToARGB8888_vec + 724 vImagevConvert_420Yp8_CbCr8ToARGB8888_vec:
->  0x1a5bebf84 <+724>: stp    q22, q19, [x22, #0x20]
0x1a5bebf88 <+728>: zip2.16b v18, v18, v25
0x1a5bebf8c <+732>: zip1.16b v19, v20, v23
0x1a5bebf90 <+736>: zip2.16b v20, v20, v23
Target 0: (Runner) stopped.

pm version? id example?

I just upgraded to this version, and then I saw printing. Maybe I got too many files at one time, which led to exceeding the memory limit

I seem to have solved this problem, nice


[1542] add mcb apiv1 course find command

###Why are we doing this?
To allow admins to view what a course looks like as it's being sent to UCAS.
###What are we adding or changing?
A command tomcb to find a course in the output generated by the api v1.
###Background
Useful for investigating issues with the system.
Checklist

[x] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Allow mcb to find a course using api v1


[REQUEST]: allow non-conda installation of dependencies

Description
I have been trying to get phg to work on my system as a singularity image, but I have been facing an issue with that: the code assumes that the user always has a conda environment which contains the dependencies. Since a singularity image (or docker container for that matter) can have all dependencies inside without the need for a conda environment that is needed, is it possible to have a --no-conda option or something similar which assumes all dependencies are available on the ${PATH}?
For testing, I have written this singularity .def file (for version 2.3):
Bootstrap: docker
From: mambaorg/micromamba:1.5.8

%post
   apt-get update && apt-get install -y wget

%post
   micromamba install -y -n base -c conda-forge -c bioconda -c tiledb python=3.8.15 tiledb-py=0.22.3 tiledbvcf-py=0.25.3 anchorwave=1.2.2 bcftools=1.16 samtools=1.16.1 agc=3.0 openjdk=17.0.10
   micromamba clean --all --yes

%post
   mkdir -p /opt
   cd /opt
   wget https://github.com/maize-genetics/phg_v2/releases/download/<IP_ADDRESS>/PHGv2-v2.3.tar
   tar xvf PHGv2-v2.3.tar
   rm PHGv2-v2.3.tar

%environment
   export PATH=/opt/phg/bin:$PATH
   export JAVA_OPTS="-Xmx50g"

Alternatives
No response
Additional Context
No response

This is something we can consider.  Is there a reason you prefer docker vs conda?

I tried to implement it myself in a PR but I cannot get the tests to run successfully (also not without any changes, there seems to be the assumption that I have a full TileDB available at $HOME/temp/phgv2Tests/tempDir/testTileDBURI/, which I don't.
If you prefer to implement it yourself, no worries! Just thought I could give it a go!

After playing around with your suggestion and some other ideas, I have created a working version. It basically creates a script called conda which checks if phg is wanting to run something in your default environment name and if so, it removes the conda run -n phgv2-conda from the command.
The singularity .def file (works with singularity v3.9; haven't tested other versions):
Bootstrap: docker
From: mambaorg/micromamba:1.5.8

%post
   apt-get update && apt-get install -y wget

%post
   mkdir -p /opt
   cd /opt
   wget https://github.com/maize-genetics/phg_v2/releases/download/<IP_ADDRESS>/PHGv2-v2.3.tar
   tar xvf PHGv2-v2.3.tar
   rm PHGv2-v2.3.tar

%post
   micromamba install -y -n base -c conda-forge -c bioconda -c tiledb python=3.8.15 tiledb-py=0.22.3 tiledbvcf-py=0.25.3 anchorwave=1.2.2 bcftools=1.16 samtools=1.16.1 agc=3.0 openjdk=17.0.10
   micromamba clean --all --yes

%post
    cat << 'EOF' > /usr/local/bin/conda
#!/bin/bash
if [[ "$1" == "run" && ("$2" == "-n" || $2 == "--name") && "$3" == "phgv2-conda" ]]; then
    shift 3
    exec micromamba run --name base "$@"
else
    echo "conda is not installed; use micromamba instead"
    exit 1
fi
EOF
    chmod +x /usr/local/bin/conda

%environment
    export PATH=/usr/local/bin:/opt/phg/bin:/opt/conda/bin:$PATH
    export JAVA_OPTS="-Xmx50g"

%runscript
    echo "Running: $*"
    exec "$@"

You may close this issue as this solves it for me and you indicated such a workaround is preferred for now.

I'm glad you found a solution that works for you.  Keep in mind you need to be sure the phgv2 required programs you load from conda must have tags that match the release of the phgv2 version you are pulling.  Otherwise there will be errors in execution.

Yes I will! That's also why I have the version of phgv2 hardcoded. But as far as I'm aware the YAML file is not part of the github release files? If it is, that would make it easier to write it for another version but for now I'll check the dependencies per version :)

Correct, the yml file is not part of the release as an individual file.  To access the contents of it you would need to do this programmatically with a getResource("phg_environment.yml") command against the java class.  If you think this would be useful, we could consider putting phg_environment.yml in the phg/resources/main folder with the application.conf file.

For me it is not needed since I know where to look, but should you decide to add a docker and/or singularity definition file to your repo it would definitely make it more future- and fool-proof I think.

Closing this issue as user has workaround in place.


Consider using startup code directly from real-time-cpp

One stumbling block is that this project actually can be built optionally with either GCC or KEIL-ARM-CC. And i don't have the full bare-metal startup for KEIL-ARM-CC at hand at the moment.
This issue might need to be put on hold. I'm not quite ready to toss KEIL-ARM-CC for GCC-only, as I like that added compiler support.


Create a contributing guide

This project should have a contributing guide (named CONTRIBUTING.adoc). It should explain how to get setup for development (pulling in content from the README), how to run tests, preferred coding style and goals for the project.
We can draw generic content and messaging from other contributing guides in the ecosystem (See https://github.com/asciidoctor/atom-language-asciidoc/blob/master/CONTRIBUTING.adoc and https://github.com/asciidoctor/asciidoctor.js/blob/master/CONTRIBUTING.adoc). What should be unique about this contributing guide is a clear list of goals for the project. Here's a draft of those goals:

Avoid unnecessary whitespace in the generated AsciiDoc

Unnecessary whitespace can be a real nuisance for users.
If we handle whitespace well, it's a chance to stand out against pandoc.
Tests should verify whitespace accuracy).

The code should not generate any Ruby warnings
Follow contribution workflow

Every change should be submitted as a pull request (and ideally associated with an issue)
Unless it's a very trivial change, pull requests should remain open for ~24 hours to provide a chance for comment and review
Pull requests should get a thumbs up from at least one other person in the project. Don't hesitate to mention other members in a comment to prod them.
If you want to avoid a merge commit, you can use the "squash and merge" feature 0of the merge button
Each change should have one or more tests that can verify it.

We should generate clean, consistent AsciiDoc, following the recommendations in the Asciidoctor User Manual. (see http://discuss.asciidoctor.org/Joining-the-Docbookrx-project-tp4695p4709.html)

The document doesn't have to be perfect to start. What's most important is that we get it started.

The following section from the Jekyll AsciiDoc plugin describes the coding style I'd like to adopt for this project:
https://github.com/asciidoctor/jekyll-asciidoc#coding-style


Remove explicit dependency on window

For server-side-rendering to work we need to remove the explicit
dependency on window object because it won't be available on the server.
window-or-global package will give us the global object depending on the
current environment (window on the browser and global on node). This way
we won't get an undefined error during SSR phase.
Closes #64

@ramonrf thanks so much! I will take a look at it this weekend and will release then. This is great!

Closing this since #64 was closed using a better approach


IoT APIs

Hi @namkedia, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Namrata Kedia). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Thanks Namrata! I assigned this to @rachitb777 since he knows the IoT APIs the best and also dropped some general comments in about some of the APIs where I couldn't tell what the API was supposed to do.

Awesome, tons of new APIS! :) I made a few comments mostly about StyleCop issues.

Looks good.  :+1:

Hey!
So the string is essentially the date time information. The format is “9/14/2016 8:39:23 PM”.
Hope this clarifies it.
Thank you,
Namrata
From: Hirsch Singhal<EMAIL_ADDRESS>Sent: Wednesday, September 14, 2016 3:45 PM
To: Microsoft/WindowsDevicePortalWrapper<EMAIL_ADDRESS>Cc: Namrata Kedia<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/WindowsDevicePortalWrapper] IoT APIs (#172)
@hpsin commented on this pull request.

In WindowsDevicePortalWrapper/WindowsDevicePortalWrapper.Shared/IoT/WindowsUpdate.cshttps://github.com/Microsoft/WindowsDevicePortalWrapper/pull/172#pullrequestreview-64517:

         /// <summary>

         /// Gets the time when a reboot is scheduled.

         /// </summary>

         [DataMember(Name = "rebootscheduledtime")]

       public int rebootscheduledtime { get; private set; }

       public string RebootScheduledTime { get; private set; }

What are the contents of the string? e.g. "8pm 1/12/2017"? Or "2874653287421"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/WindowsDevicePortalWrapper/pull/172#pullrequestreview-64517, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATdsCGsz0yvc-ymksQS5PjGWCGZpKtw8ks5qqHjsgaJpZM4J2eXY.

Hm, can that be formatted into something usable? I believe DateTime time = DateTime.Parse(timeString); will give you a DateTime object.  It may just work if you replace string with DateTime.


Performer Profiles

Scrape performer data from the official site. Import it into twitarr. Create API endpoints and web views for accessing the data.
If we get access to an improved event feed (#225), ensure the schedule has links to the performer profiles.
Bonus feature: Allow content management/editing once on board (typo fixes, removing last-minute-cancellations, adding last-minute-additions, etc).

A bit of background: performer photos and blurbs are available on jococruise.com each year, and that site is whitelisted so cruisers can see it while aboard. The value of us copying it into Twitarr is in our ability to link performer profiles with other content. It's not worthwhile doing this just to copy stuff cruisers already have access to.
Showing a list of all the events a performer is participating in on that performer's page would be useful, particularly since we may be able to make that list update to match schedule changes. Linking to performer pages from event and forum pages would also be useful.

This is now in, and we have access to an improved event feed, so we can bulk import links. Mods and above can edit performer profiles on boat. We can't (yet) bulk import Performers themselves; instead there's a web form for creating/editing official performers.
This feature also lets shadow cruise organizers create and edit performer profiles and link them to their events. They can only do this during the pre-registration period (not on boat).


test(cdc): fix canal integration

What problem does this PR solve?

Issue Number: ref #2246
What is changed and how it works?

add -k to curl

the cert is expired:
curl -o /etc/yum.repos.d/CentOS-Base.repo https://www.xmpan.com/Centos-6-Vault-Aliyun.repo
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (60) SSL certificate problem: certificate has expired
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/run-all-tests

/merge

/run-all-tests tidb=f5487e3f2596cdd63ae9840606c7fff3fb539e03

/run-verify

/run-verify

/merge


pills: Add flex-wrap: wrap to pill container for all pills.

Fixes #10059.
In 66df4e3,
display: inline-flex was added to .pill-container but
flex-wrap: wrap was missing which forced overflow pills to be on
one line and made the pill text overflow vertically. This was not
observed in composebox pills as .pm_recipient .pill-container
already had a flex-wrap: wrap rule which has been removed in this
commit to avoid duplication.
Before:

After:

Nice.  Might be worth a quick check for whether any of the other CSS in the pm-recipient .pill_container block belongs in the generic one as well...

@timabbott I don't think any of the other CSS in the pm-recipient .pill_container block belongs in the generic one.


Mount cacerts via CSI for Helm chart

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
Currently, by default we can mount cacerts only via k8s secret in a Helm.
It is beneficial to extend this possibility and add an option to use CSI, it will be great security enhancement for Helm setup.
Describe alternatives you've considered
Yeah, as alternative can be deployed via IstioOperator, which does patching - but it is not our case, as we are using Helm
`
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
 components:
   pilot:
     enabled: true
     k8s:
       overlays:
         - kind: Deployment
           name: istiod # https://github.com/istio/istio/issues/31076
           patches:
             - path: spec.template.spec.volumes[name:cacerts]
               value:
                 name: cacerts
                 csi:
                   driver: secrets-store.csi.k8s.io
                   readOnly: true
                   volumeAttributes:
                     secretProviderClass: "istio-tls"

`
Affected product area (please put an X in all that apply)
[ ] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[X] Security
[ ] Test and Release
[ ] User Experience
[X] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

It can be done by helm as well https://preliminary.istio.io/latest/docs/setup/additional-setup/customize-installation-helm/

@howardjohn  thanks for help.
But still would be nice to have CSI in Istio Helm.

Hi @howardjohn, could you please help?)
I will be so grateful if you can share an example of automating customize istio installation (link above) via Terraform

An simple demo can be found here, which I used to debug https://github.com/istio/istio/issues/38831.

Short update,
I have tried to use this solution for putching Istiod deployment, but it coudln't putch field which already presented in defenition
Error
│ Error: release istiod failed, and has been rolled back due to atomic being set: cannot patch "istiod" with kind Deployment: Deployment.apps "istiod" is invalid: [spec.template.spec.volumes[0].csi: Forbidden: may not specify more than 1 volume type, spec.template.spec.containers[0].volumeMounts[2].name: Not found: "cacerts"]
Customization

apiVersion: apps/v1
kind: Deployment
metadata:
name: istiod
namespace: istio-system
spec:
template:
spec:
volumes:
- name: cacerts
csi:
driver: secrets-store.csi.k8s.io
readOnly: true
volumeAttributes:
secretProviderClass: "istio-mtls"

I tested the approach suggested in https://github.com/istio/istio/issues/40490#issuecomment-1216627201 and I believe it does not work because of the patchesStrategicMerge approach.
I created a file csipatch.yaml
% cat csipatch.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: istiod
  namespace: istio-system
spec:
  template:
    spec:
      volumes:
      - name: cacerts
        csi:
          driver: secrets-store.csi.k8s.io
          readOnly: true
          volumeAttributes:
            secretProviderClass: "istio-mtls"

I created the kustomize.sh file as described here:
cat > kustomize.sh <<EOF
#!/bin/sh
cat > base.yaml
exec kubectl kustomize # you can also use "kustomize build ." if you have it installed.
EOF
chmod +x ./kustomize.sh

I created my kustomization.yaml file:
% cat kustomization.yaml
resources:
- base.yaml
patchesStrategicMerge:
- csipatch.yaml

Now I can test if the template renders correctly:
helm template istiod istio/istiod -n istio-system --post-renderer ./kustomize.sh

I see this:
      volumes:
      - csi:
          driver: secrets-store.csi.k8s.io
          readOnly: true
          volumeAttributes:
            secretProviderClass: istio-mtls
        name: cacerts
        secret:
          optional: true
          secretName: cacerts

this is not working because the secret block should be dropped.
Following instructions here:
https://kubernetes.io/docs/tasks/manage-kubernetes-objects/kustomization/#customizing
You need to use a patchesJson6902 patch.
Do the following, change the kustomization.yaml:
% cat kustomization.yaml
resources:
- base.yaml
patchesJson6902:
- target:
    kind: Deployment
    name: istiod
  path: patch.yaml

create a patch.yaml file:
% cat patch.yaml
- op: replace
  path: /spec/template/spec/volumes[name:cacerts]
  value:
    name: cacerts
    csi:
      driver: secrets-store.csi.k8s.io
      readOnly: true
      volumeAttributes:
        secretProviderClass: "istio-tls"

Now it renders correctly for me with:
helm template istiod istio/istiod -n istio-system --post-renderer ./kustomize.sh

In comment https://github.com/istio/istio/issues/40490#issuecomment-1239365163 I just explain how to patch the helm template, I did not provide an example how to integrate this in Terraform.
In comment https://github.com/istio/istio/issues/40490#issuecomment-1223637685 it was suggested to look at istiod.tf but in this example it does not use the --post-renderer feature.
I believe it is necessary to use the postrender block documented in the Terraform resource:
https://registry.terraform.io/providers/hashicorp/helm/latest/docs/resources/release#postrender
@ElkRom if you manage to come up with the full example can you please post it here ? thanks

@zioproto  thanks for help, appreciated
What is working in my case:
patch.yaml
- op: replace
  path: /spec/template/spec/volumes/2
  value:
    name: cacerts
    csi:
      driver: secrets-store.csi.k8s.io
      readOnly: true
      volumeAttributes:
        secretProviderClass: "istio-tls"

When compering with @zioproto  example, the path a bit different
kustomization.yaml is the same:
% cat kustomization.yaml
resources:
- base.yaml
patchesJson6902:
- target:
    kind: Deployment
    name: istiod
  path: patch.yaml

For kustomiz.sh I have added one input argument, which accepts Terraform ${path.module} value
% cat kustomiz.sh
#!/bin/sh
cd $1
cat > base.yaml
exec kubectl kustomize # you can also use "kustomize build ." if you have it installed.
EOF

Terraform file for deploying Istio via Helm
resource "helm_release" "istiod" {
  name              = "istiod"
  namespace         = var.istio_ns_name
  dependency_update = true
  repository        = local.istio_chart_repo
  chart             = "istiod"
  version           = var.istio_version
  atomic            = true
  values            = local.helm_values_istio
  lint              = true

  #For automating csi putch
  postrender {
    binary_path = "${path.module}/kustomize.sh"
    args = ["${path.module}"]
     }
  depends_on = []
   }

@ElkRom you are right, this patch file does not render correctly:
% cat patch.yaml
- op: replace
  path: /spec/template/spec/volumes[name:cacerts]
  value:
    name: cacerts
    csi:
      driver: secrets-store.csi.k8s.io
      readOnly: true
      volumeAttributes:
        secretProviderClass: "istio-tls"

the correct solution is the one you found, changing the path to:
  path: /spec/template/spec/volumes/2

it seems this is a limit of JSON:
https://stackoverflow.com/questions/27439986/what-is-the-json-patch-format-to-remove-an-element-from-an-array

@ElkRom could you confirm if we can close this GitHub issue ? thanks