Incorrect documentation (or bug?)

Summary:
The documentation says near the bottom of the README:

By default the modal is closed when clicking outside of it (the overlay area). If you want to prevent this behavior you can pass the 'shouldCloseOnOverlayClick' prop with 'false' value.

However that's not exactly true. It appears to not close when clicking on the overlay unless onRequestClose={this.handleCloseModal} is passed into <Modal />
Steps to reproduce:

it works here
it doesn't work here

Expected behavior:
modal closes when clicking on the overlay by default
Additional notes:
maybe this is a bug and not an issue with the documentation

@shonin It is indeed a problem with the documentation.  Because the open/close state of the modal is handled external to the modal itself, it needs the consumer to provide details on how to actually close the modal.  We'd be happy to accept a PR fixing the documentation.

#352 closes this issue.


Speed not updated when actor speed is updated

The speed on the token isn't updated when I add a speed in the character sheet.
Tested on pf2e system.

I was gonna report the same for PC Actors. When the speed is changed by an effect (i.e. Monk Mountain Stance -5 speed) the speed icon doesn't change. It stays on the PC's base movement (doesn't show speed minus penalty even though the character sheet reflects the proper speed (including the penalty)


Document config changes for new version

When updating the CHANGELOG for the next version, we should include steps to migrate old configs to the new version. So explicitly say which new imports were added in the default configs and potentially also other breaking changes that might have to be taken into account.

Not sure if we would need all of these and how detailed they would have to be, but these are the breaking changes for 2.0.0 as of writing:

BasicRequestParser now needs a conditionsParser argument.
IdentityProviderHttpHandler and InteractionRoute arguments have changed drastically.
WebAclAuthorizer now needs an accessChecker, these are being imported from config/ldp/authorization/authorizers/access-checkers/

No changes yet to the imports yet.


Shift Tab Doesn't Cycle

Is this intended? You can test this on the live demo page.  http://myplanet.github.io/angular-tab-trap/
You can tab forward and have the focus loop around the page the form.
However, you can only shift tab to the first element and it does not cycle back to the end.  Is this intended?

That's certainly a bug!

I fixed so Shift-Tab cycles by adding an id to the first template button:
var tmpl = ''
+ '<button type="button" id="tab-trap-start"></button>'
+ '<div ng-transclude></div>'
+ '<button type="button"></button>';

Passed the event into the focus:
children.on('focus', function (e) { ... }

Then modified the part within that which calls .focus() to this:
if (tabbables[tabbables.length - 1] && focusLeader[0].id == e.currentTarget.id) {
	tabbables[tabbables.length - 1].focus();
} else if (tabbables[0]) {
	tabbables[0].focus();
}


Support Cross-DB Stored Procedure Call

This commit uses ANTLR to confirm if the query is cross-database.
It will then set a flag in the statement's execution object and
check that flag during the executor stage. If true, the current user
will be temporarily set to the Login to enable permissions for the
statement execution. Otherwise, permission checks will prevent the
cross-database statement from executing.
Task: BABEL-3275
Signed-off-by: Shalini Lohia<EMAIL_ADDRESS>

Please validate that there are no new test failures with this change

I have verified, there is no new failure due to my change.


pre-release

description not really possible, this merges over 6 months of work.

Git reset works but you will have to force push the changes
git checkout develop
git pull origin develop
git checkout main
git reset --hard origin/develop
git push origin main --force

@.***
From: johndanenRV @.>
Date: Tuesday, August 20, 2024 at 3:27 PM
To: rossvideo/Catena @.>
Cc: John Naylor @.>, Author @.>
Subject: [EXTERNAL] Re: [rossvideo/Catena] pre-release (PR #244)
Git reset works but you will have to force push the changes
git checkout develop
git pull origin develop
git checkout main
git reset --hard origin/develop
git push origin main --force
—
Reply to this email directly, view it on GitHubhttps://github.com/rossvideo/Catena/pull/244#issuecomment-2299705778, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6R5EEPY5VPV7EE6PFWPSHDZSORDLAVCNFSM6AAAAABM2TNEA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJZG4YDKNZXHA.
You are receiving this because you authored the thread.Message ID: @.***>

This e-mail and any attachments may contain information that is confidential to Ross Video.
If you are not the intended recipient, please notify me immediately by replying to this message. Please also delete all copies. Thank you.

I resolved the issues and pushed merge fixers to main. It builds but I’ve not run anything yet
From: John Naylor @.>
Date: Tuesday, August 20, 2024 at 4:16 PM
To: rossvideo/Catena @.>
Subject: Re: [EXTERNAL] Re: [rossvideo/Catena] pre-release (PR #244)
@.***
From: johndanenRV @.>
Date: Tuesday, August 20, 2024 at 3:27 PM
To: rossvideo/Catena @.>
Cc: John Naylor @.>, Author @.>
Subject: [EXTERNAL] Re: [rossvideo/Catena] pre-release (PR #244)
Git reset works but you will have to force push the changes
git checkout develop
git pull origin develop
git checkout main
git reset --hard origin/develop
git push origin main --force
—
Reply to this email directly, view it on GitHubhttps://github.com/rossvideo/Catena/pull/244#issuecomment-2299705778, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A6R5EEPY5VPV7EE6PFWPSHDZSORDLAVCNFSM6AAAAABM2TNEA6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJZG4YDKNZXHA.
You are receiving this because you authored the thread.Message ID: @.***>

This e-mail and any attachments may contain information that is confidential to Ross Video.
If you are not the intended recipient, please notify me immediately by replying to this message. Please also delete all copies. Thank you.


Crash observed in KMAKeychainItemWrapper writeToKeychain

I'm starting to receive crash reports with [KMAKeychainItemWrapper writeToKeychain] being the root cause.
Using KISSmetrics-iOS-SDK (2.3.1)
The crashes appear to have started when I introduced a Background Fetch entitlement (as a part of UIBackgroundModes) to retrieve content on a periodic basis.
Stack trace is attached. Any help would be greatly appreciated.
Thanks
Kunal.
KMCrashReport_v2.txt

This issue can be closed.


Question: Why doesn't cargo expand this derive?

I might be missing something, but I wanted to see the implementation of this derived trait. But cargo expand gives nothing.
#[derive(utoipa::ToSchema)]
pub enum CompoundingFrequency{
    Continuous
}

Cargo.toml:
utoipa ="3.3.0"
Am I missing something?

Seems to work.
$ cargo expand

#![feature(prelude_import)]
#[prelude_import]
use std::prelude::rust_2018::*;
#[macro_use]
extern crate std;
pub enum CompoundingFrequency {
    Continuous,
}
impl<'__s> utoipa::ToSchema<'__s> for CompoundingFrequency {
    fn schema() -> (&'__s str, utoipa::openapi::RefOr<utoipa::openapi::schema::Schema>) {
        (
            "CompoundingFrequency",
            utoipa::openapi::ObjectBuilder::new()
                .schema_type(utoipa::openapi::SchemaType::String)
                .enum_values::<[&str; 1usize], &str>(Some(["Continuous"]))
                .into(),
        )
    }
}

Many thanks. Interesting, must be some issue locally then. Just for the reference, I am on Windows Stable, rustc 1.69.0 (84c898d65 2023-04-16).
Will close for now.


Exclude dependencies of ReDoc webjar by default

In the wiki for the plugin there is an exclusion defined, however on the article I was referencing when trying to use this plugin there was no exclusion, however this exclusion is required for this plugin to function correctly due to a webjar issue.
When trying to build the project without the exclusion, you encounter the issue ZipException opening "node-fetch-3.0.0-beta.9.jar": ZIP file can't be opened as a file system because entry<EMAIL_ADDRESS>has a '.' or '..' element in its name. Simple fix for this issue will be to implement this exclusion directly into the plugin.
<dependency>
    <groupId>org.webjars.npm</groupId>
    <artifactId>redoc</artifactId>
    <version>2.0.0-rc.70</version>
    <exclusions>
        <exclusion>
            <groupId>*</groupId>
            <artifactId>*</artifactId>
        </exclusion>
    </exclusions>
</dependency>

This exclusion was intentionally mentioned in wiki, not sure about the article you're referencing to, but it wasn't written be me for sure. I guess we can exclude it in the plugin, but this issue will occur for people trying to change ReDoc UI version, so I just wanted to make it clear in docs.

This exclusion was intentionally mentioned in wiki, not sure about the article you're referencing to, but it wasn't written be me for sure. I guess we can exclude it in the plugin, but this issue will occur for people trying to change ReDoc UI version, so I just wanted to make it clear in docs.

https://javalin.io/tutorials/openapi-example


ts-web: mysterious ts-jest re-unification

unexplained in #1119 why it got split into specific packages' node_modules. bringing it back

tests pass, I'll sneak this into #1139

or no, #1146 would be better


Improve Slides

Known issues:

[x] Pager is an attribute - probably want to use the swiper version - options
[x] Multiple sliders on one page use the same pagination
[x] Improve documentation behind options and methods
[ ] Remove timeout around create
[ ] Don't emit slide events when runCallbacks is false

I'm hoping there is (to be documented), or will be (added), a way to access the Swiper API's method: mySwiper.slideTo(index, speed, runCallbacks). This is particularly useful for using slides on first-run and allowing user to skip to end if they want.

@steveharbick There's actually been a PR submitted for this it's just something we need to look into merging. :smile: https://github.com/driftyco/ionic/pull/5491

Related: https://github.com/driftyco/ionic/issues/6198

So some of the improvements were made. I improved the docs to add a lot more information and document the methods better, these can be seen here: http://ionicframework.com/docs/v2/api/components/slides/Slides/
Please check the changelog for the breaking changes to slides in beta.5: https://github.com/driftyco/ionic/blob/2.0/CHANGELOG.md#200-beta5-2016-04-20
Let me know if you have any feedback on the docs! Going to move this issue to beta.6 to continue work on it.

slideTo marked time is an optional parameter, but can not omitted in typescript
    /**
     * Transition to the specified slide.
     *
     * @param {number} index  The index number of the slide.
     * @param {number} speed  Transition duration (in ms). Optional.
     * @param {boolean} runCallbacks  Whether or not to emit the `willChange`/`didChange` events. Optional. Default true.
     */
    slideTo(index: number, speed: number, runCallbacks: boolean): void;

Thanks for pointing that out @davyzhang, I fixed it in the source so it should be in the next release! Sorry about that. :smile:

Fantastic, Thanks for the great work.

Looping doesn't seem to work when the slides are loaded dynamically. For example:
<ion-slides [options]="optsWithLoopEnabled">
  <ion-slide>

  </ion-slide>
  <ion-slide>

  </ion-slide>
</ion-slides>

works great and does the behind the scenes duplication so that you can swipe from last to first or vice versa. However:
<ion-slides [options]="optsWithLoopEnabled">
  <ion-slide *ngFor="let slide of mySlides">>

  </ion-slide>
</ion-slides>

does not work as the duplication doesn't happen, so swiping last to first flickers to the first and then skips one. Looped autoplay is broken in the same way.

Related: #7422

Closing this since slides have been refactored.

Any update on having multiple sliders on one page?

Any update on having multiple sliders on one page?

Indeed it would be great! I was amazed about this component but it is too bad we can't use multiple sliders on one page.


Create surface object with special tag when using Spatial Mapping

As we all know, there are losts of surface objects which will be ctreated after we use Spatial Mapping. Maybe it will be better if Spatial Mapping create those surface objects with a special tag? Because we always use gestures or gaze, a special tag will help us select the right object

Does using a layer and layermasks not work better for the same objective?

Looking through SpatialMappingSource where SurfaceObjects are added in AddSurfaceObject they are added to the layer specified by SpatialMappingManager.Instance.PhysicsLayer. This looks to be controllable through the editor.
Are these the surface objects you're referring to? If so, can you filter by that physics layer for your use case?

As @jwittner suggests, you can use the collision layer to detect if an object is part of the spatial mapping mesh. For my applications, I like to create an empty game object called 'Spatial Mapping' in my scene, that I then parent all of the spatial mapping meshes to (the 'SpatialMapping' prefab in HoloToolkit will already do this). Then, if I want to get the meshes, I just need to look for that parent object to find them.
Tags are very convenient when working in your own application (and you can certainly modify the SpatialMapping code in your own application to add a tag whenever a new mesh is created). However, tags do not export to other projects. So, even if we were to create a 'SpatialMapping' tag in the 'HoloToolkit', all users would have to manually add that tag to their project to take advantage of it. Because of this limitation, we avoid using tags in prefabs and code that is meant to be shared across multiple projects.

Totally agree with @jwittner and @angelaHillier. Tags would break the scenario of create a unity package of the toolkit and import into another project. Or just copy code that depends on tags.

Thank you, guys. Layer is actually better than tag. Happy Mid-autumn Festival


Improving code documentation for functions in geospatial_mods.py

This links to improving code documentation in https://github.com/ONSdigital/SDG_11.2.1/issues/158 and can also be achieved alongside issue https://github.com/ONSdigital/SDG_11.2.1/issues/196.
The function buffer_points needs:

[ ]  an explanation of the arguments involved
[ ]  a returns section (explaining the dataframe outputted)

Reviewed with changes pushed to #197


bug(*) missing service info

Summary
In https://github.com/kumahq/kuma/issues/2728 it was reported that external services are not displayed properly.
The reason behind it was the part of refactoring which was done in https://github.com/kumahq/kuma-gui/pull/193 where the typo made it call the wrong endpoint. This PR fixes that issue.

Signed-off-by: Tomasz Wylężek<EMAIL_ADDRESS>

We use fix(*) in kumahq/kuma


Themes doesn't apply or save

Latest Themes like Vacationzr or Bitbucket Dark have no changes after pressing checkbox

Solved in f3a11560a176b3dab75e4632c3a8ee64821573d7 bafc310cd1e2405d73c01dde6bee369705f0ea9d bdc514c974d2547eae0175679e704bbd89696363


Completed Shell Assignment

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding answers to the rest of the questions.
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

manually marked


Your request to provision "Microsoft 365 learning pathways"  template failed!

SharePoint Provisioning Service
Your request to provision "Microsoft 365 learning pathways" template failed!
Please report the issue here providing the following provisioning action correlation ID: 4fb026a8-b2ae-4d96-86db-fb502079a54d !
Please provide use feedback around your used template or the service in general by using our feedback form . Thank you for your input advance.
Exception: Object reference not set to an instance of an object.
Provisioning action correlation ID: 4fb026a8-b2ae-4d96-86db-fb502079a54d

Hi,
Thanks for your patience. We've just updated the provisioning service with several fixes. Hopefully, this one is now solved or at least mitigated. Please try again and let us know in case of any further issues. Thanks for your cooperation.

Thank you very much and have a wonderful weekend. We will try it again today. 🙂
Regards,
Johnny Crockett Lopez
Microsoft MVPhttps://mvp.microsoft.com/en-us/PublicProfile/5004098
about.me/JCLopez81https://about.me/JCLopez81?promo=email_sig&utm_source=product&utm_medium=email_sig&utm_campaign=edit_panel&utm_content=thumb
Cell:<PHONE_NUMBER>
@.***
[cid:8098bc35-4ce5-47ff-9642-28818468f30b]  [cid:f1c97265-5586-4836-b6cc-03f3f779ac64] [cid:26e36e10-8d14-436d-a82d-9dd9d348f075] [cid:a46305e8-40c3-4672-9695-9dc103e10f1f]

From: Paolo Pialorsi @.>
Sent: Friday, May 21, 2021 3:25 AM
To: SharePoint/sp-provisioning-service @.>
Cc: Johnny Lopez @.>; Author @.>
Subject: Re: [SharePoint/sp-provisioning-service] Your request to provision "Microsoft 365 learning pathways" template failed! (#521)
Closed #521https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-provisioning-service%2Fissues%2F521&data=04|01||61684ffa9e004e0104be08d91c31fc92|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637571823273299982|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=K0ALeQvmQBSdZcGd89sY3SYKeDzXTG%2BE1Uxb6fIifS0%3D&reserved=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-provisioning-service%2Fissues%2F521%23event-4777926330&data=04|01||61684ffa9e004e0104be08d91c31fc92|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637571823273309939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1qcAV4g2O8WQpWnJh6wR1RKUNW3RLCFKFizp%2Bif3acc%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQ3SRW7GIAGX6QGRL6FHC7DTOYKHLANCNFSM45DHSDSQ&data=04|01||61684ffa9e004e0104be08d91c31fc92|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637571823273309939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=mrZsY9tgVaXpvsztgpPStVn4ucsex0QI4xhFFUJGxbg%3D&reserved=0.


Investigate environment detection in our overall environment UX to see if there are scenarios where discovery is unnecessary

Related: https://github.com/microsoft/vscode-python/issues/17498 https://github.com/microsoft/vscode-python/issues/18647 https://github.com/microsoft/vscode-python/issues/18671
Users can use the API or the refresh icon for triggering it if needed:

Check https://github.com/microsoft/vscode-python/issues/19102#issuecomment-1129355391 for investigation results.

Triggering discovery is probably not needed. Users can use the API or the refresh icon if triggering it is still needed. Only a couple places need it:

When auto-selecting interpreters for a new workspace.
Triggering it for every scope exactly once (across sessions).

Decided that it's okay to do it for the first session for now as:

Jupyter is going to do that anyways via the API to fetch kernels.
Internally it is assumed discovery is triggered atleast once. For example, Auto-selection (which triggers discovery) is always triggered even if an interpreter is explicitly selected, so that user has an auto-selected interpreter to fallback on if the selected interpreter is cleared.
Select Interpreter section in the getting started section already has environments populated.

We also need to make sure legacy jupyter integration APIs still work.

Proposed approach, only auto-trigger discovery if:

A user attempts to choose a different interpreter
When a particular scope (workspace or global) is opened for the first time
When jupyter queries legacy discovery APIs for envs

Verification steps:
If Jupyter is not installed or disabled, verify we auto-trigger discovery if:

A user attempts to choose a different interpreter
When a particular scope (workspace or global) is opened for the first time
Environment cache is empty
If Jupyter is installed, discovery is always triggered

Certain components (like diagnostics) call for hasInterpreters() API, which requires that discovery is triggered atleast once for it to work. See if we can avoid calling it and make sure information returned by it is reliable enough.

/verified?
It does trigger discovery just when opening 'Select interpreters'? Is this expected?
Otherwise behaves as you outlined.

A user attempts to choose a different interpreter when quickpick list is opened the first time

Yep, that's what the above point means in https://github.com/microsoft/vscode-python/issues/19102#issuecomment-1130394556, only happens the first time though.


Inconsistent async output when used with JavaScript kernel (tslab)

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
I've been using Jupyter Notebooks in the past to teach students async programming in JavaScript using the tslab kernel. This worked fine in earlier versions of the vscode-jupyter extension but more recently the async output became inconsistent between different runs. For instance:
console.log('starting');

setTimeout(() => {
  console.log('timeout 1');
}, 0);

setTimeout(() => {
  console.log('timeout 2');
}, 0);

console.log('ending');

Run 1:
starting
ending
timeout 1
timeout 2

Run 2:
starting
ending
timeout 1 timeout 1 timeout 2

Run 3:
starting
ending
timeout 1timeout 2

The same JavaScript code when used with jupyter-lab consistently produces correct output 1.
VS Code Version
Version: 1.73.0-insider (user setup) Commit: a8108049ab61b970f2ec1839dfb753054e07395e Date: 2022-10-13T05:19:31.002Z Electron: 19.0.17 Chromium: 102.0.5005.167 Node.js: 16.14.2 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.22623 Sandboxed: Yes
Jupyter Extension Version
v2022.10.1102881143
Jupyter logs
Visual Studio Code - Insiders (1.73.0-insider, undefined, desktop)
Jupyter Extension Version: 2022.10.1102881143.
Python Extension Version: 2022.16.0.
Workspace folder c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda
info 16:10:15.381: ZMQ install verified.
User belongs to experiment group 'jupyterTest'
User belongs to experiment group 'jupyterEnhancedDataViewer'
info 16:10:15.757: LSP Notebooks experiment is enabled
info 16:10:16.870: Preferred Remote kernel for c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb is undefined
info 16:10:16.870: Find preferred kernel for c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb with metadata {"kernelspec":{"display_name":"JavaScript","language":"javascript","name":"jslab"},"language_info":{"file_extension":".js","mimetype":"text/javascript","name":"javascript","version":""},"orig_nbformat":4} & preferred interpreter
info 16:10:16.924: Preferred Remote kernel for c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb is undefined
info 16:10:17.175: Preferred Remote kernel for c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb is undefined
info 16:10:17.175: Preferred kernel .jslab.tslab.cmd.\.tslab.cmd#kernel#--config-path#{connection_file}#--js is exact match or top match for non python kernels, (false, undefined, true, true)
info 16:10:17.179: PreferredConnection: .jslab.tslab.cmd.\.tslab.cmd#kernel#--config-path#{connection_file}#--js found for NotebookDocument: c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb
info 16:10:18.210: Got empty env vars with python c:\Users\jimcr\.pyenv\pyenv-win\versions\3.10.8\python.exe in 2320ms
info 16:10:18.251: Process Execution: > ~\.pyenv\pyenv-win\versions\3.10.8\python.exe -m pip list
> ~\.pyenv\pyenv-win\versions\3.10.8\python.exe -m pip list
info 16:10:24.284: Starting Jupyter Session startUsingLocalKernelSpec, .jslab.tslab.cmd.\.tslab.cmd#kernel#--config-path#{connection_file}#--js for 'c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb' (disableUI=true)
info 16:10:24.285: Computing working directory for resource 'c:\Users\jimcr\dev\hackyourfuture\usingapis-qanda\week1\eventloop.ipynb'
info 16:10:25.845: Launching Raw Kernel & not daemon JavaScript # tslab.cmd
info 16:10:25.846: No custom variables nor do we have a conda environment
info 16:10:25.867: Process Execution: > tslab.cmd kernel --config-path ~\AppData\Roaming\jupyter\runtime\kernel-v2-17076lvCMIvGHQYLb.json --js
> tslab.cmd kernel --config-path ~\AppData\Roaming\jupyter\runtime\kernel-v2-17076lvCMIvGHQYLb.json --js
info 16:10:25.867: Process Execution: cwd: ~\dev\hackyourfuture\usingapis-qanda\week1
cwd: ~\dev\hackyourfuture\usingapis-qanda\week1
info 16:10:27.954: Started Kernel JavaScript (pid: 25064)

Coding Language and Runtime Version
JavaScript
Language Extension Version (if applicable)
tslab 1.015
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

The problem lies with the rendering of the outputs in VS Code core.
Seems to be another issue related to https://github.com/microsoft/vscode-jupyter/issues/11443
I.e. its a problem with the CSS, and seems to be a regression.
See below for the HTML content and the Notebook Outputs

duplicate of https://github.com/microsoft/vscode/issues/160745


Out of place hydration using render and eval

Commit 1:

Add --output(-o) flag to render and eval commands, to support out of place hydration and writes rendered resources to stdout(stdout wrapped)|unwrap(stdout unwrapped)|OUT_DIR_PATH(to another directory). (https://github.com/GoogleContainerTools/kpt/issues/1412)
Deprecate --dry-run flag from eval command as it is supported by --output flag.

Commit 2
Add e2e tests for oop hydration
Commit 3
Reference docs.
Supported example
kpt fn render [HERE] -o stdout \
| kpt fn eval --image gcr.io/kpt-fn/set-annotations:v0.1.3 -- foo=bar \
| kpt fn eval --image gcr.io/kpt-fn/set-labels:v0.1.3 -o [stdout|unwrap|OUT_DIR]-- tier=backend

Note: render can only read from directory and not STDIN. eval can read from both stdin and directory.

This PR is doing too many things, and combines multiple UX changes each deserving their own consideration e.g (Using dry-run in render for output) and e2e testing.

@phanimarupaka Thanks for working on this! 💯 From a cursory look it seems this would indeed unblock the hydration functionality needed for LZ.

Source using -output consistently.
Outputting meta resources (cross-cutting).


Inclusion of null values in code generation

          > I have went ahead and added this as another test. That is a pretty comprehensive spec that should be supported out of the box, IMO. It actually did uncover some minor issues that I resolved and have an impact on this issue's feature request. So that's good.

Pretty good progress!

The model enum is reflected as a doc string at the moment and type hint denotes String - just as defined in the spec.

This is already a good improvement.

I didn't have any time to test against the OpenAI API, but the code looks good.

I've played a bit with the completions endpoint and these are the findings:
Server URL
Currently, the generator only takes the schema and host from the defined server. It ignores any defined path.
So for:
servers:
  - url: https://api.openai.com/v1

The generator generates:
final r = await _request(
  host: 'https://api.openai.com',
  path: '/completions',
  ...
);

(the v1 is not included)
Maybe we should rename host to baseUrl while fixing it.
Update: I've issued a fix in https://github.com/tazatechnology/openapi_spec/pull/15
CreateCompletionRequest
The generated class looks good. The only issue that I've encountered is that nulls are also serialized and OpenAI doesn't seem to like it.
Eg.
Calling:
final res = await client.createCompletion(
  request: CreateCompletionRequest(
    model: 'gpt-3.5-turbo-instruct',
    prompt: 'Repeat: This is a test',
  ),
);

Generates:
{
  "model": "gpt-3.5-turbo-instruct",
  "prompt": "Repeat: This is a test",
  "best_of": 1,
  "echo": false,
  "frequency_penalty": 0.0,
  "logit_bias": null,
  "logprobs": null,
  "max_tokens": 16,
  "n": 1,
  "presence_penalty": 0.0,
  "stop": null,
  "stream": false,
  "suffix": null,
  "temperature": 1.0,
  "top_p": 1.0,
  "user": null
}

But OpenAI replies with the following error:
OpenAIClientException({
  "uri": "https://api.openai.com/v1/completions",
  "method": "POST",
  "code": 400,
  "message": "Unsuccessful response",
  "body": {
    "error": {
      "message": "None is not of type 'object' - 'logit_bias'",
      "type": "invalid_request_error",
      "param": null,
      "code": null
    }
  }
})

I'm not sure what's the best default behaviour for the generator, should it include nulls or not?
I've checked dart-dio generator and it doesn't include them (it generates: @JsonKey(name: r'logit_bias', required: false, includeIfNull: false)).
Anyway, it's not a blocking issue as you can always configure include_if_null: false in the build.yaml file.
CreateCompletionResponse
The choices array is generated as List<Map<String, dynamic>> instead of a List of the inner object.
dart-dio generator creates a final List<CreateCompletionResponseChoicesInner> choices by default. So they concatenate the schema name (CreateCompletionResponse), plus the field (choices) plus Inner to create the class name of the inner object.
Originally posted by @davidmigloz in https://github.com/tazatechnology/openapi_spec/issues/13#issuecomment-1770441157

Anyway, it's not a blocking issue as you can always configure include_if_null: false in the build.yaml file.

@davidmigloz moving to a new issue for this specific item. Using build.yaml is how I typically resolve this issue (as opposed to adding more verbose generated code).

I'm not sure what's the best default behaviour for the generator, should it include nulls or not?

This is the real question. If we believe the default should be to not include it, then we should add includeIfNull: false to the JsonKey for all nullable variables. I need to do a little bit of research on a best practice, but my initial hunch is that they should be removed by default.
If we really want we can even add it as a generator option:
spec.generate(
  schemaOptions: SchemaGeneratorOptions(
    enabled: true,
    includeIfNull: false // Default is false
  ),
);

Let me know your thoughts.

@davidmigloz I thought about this a bit more after working a PR, and it is a bit hairy when you dig into the details. There is not enough information in the OpenAPI spec to know if include_if_null should be true/false and in turn it should be left to the user. The default json_serializable build configuration is linked below. You will see the default is include_if_null: true.
https://pub.dev/packages/json_serializable#build-configuration
Overriding it at the schema code generation will create unexpected conflicts and essentially hard code the include_if_null setting in the schema generation JsonKey as opposed to allow the user to control it in the build.yaml code.
The purpose of JsonKey in the schema generator is primarily used to ensure that names are handled, one of the more annoying aspects of creating these schemas by hand. All other build options are left to the user to control via the build.yaml configuration as they see fit.
Not to mention, it adds a ton of code generation changes in each file when it is likely best handled by a single include_if_null: false line in the build.yaml file.

TLDR: I think we should keep things as is and allow the user to control this at build.yaml. I have the code to satisfy this issue ready to go via SchemaGeneratorOptions, but after writing it, am not so sure that is the best solution due to the comments above. Feedback welcome - else, will close this issue.

Thanks for all the details!
One option could be to add includeIfNull: false to fields that are nullable: true and required: false.
So if you still want to serialize the null value you just need to make that field required. Would that work?

One option could be to add includeIfNull: false to fields that are nullable: true and required: false.

Wouldnt that still conflict with this primary issue, the fact that it is now hardcoded in the generated schema:

Overriding it at the schema code generation will ... hard code the include_if_null setting in the schema generation JsonKey as opposed to allow the user to control it in the build.yaml code.

Wouldnt that still conflict with this primary issue, the fact that it is now hardcoded in the generated schema
Isn't that a desired feature in a spec?

There is not enough information in the OpenAPI spec to know if include_if_null should be true/false and in turn it should be left to the user

In which cases we don't have enough information?

If the server expects all the fields to be serialized (even the null ones), then all fields should be marked as required in the spec (including the nullable ones)
If the server doesn't expect nullable fields to be serialized, then the nullable fields should not be marked as required

Isn't that a desired feature in a spec?

Yea I wont argue with that. Regarding your example above, the issue is "should be marked" or "should not be marked"... although they should (good practice), that is not always the case.

One option could be to add includeIfNull: false to fields that are nullable: true and required: false.
So if you still want to serialize the null value you just need to make that field required

In any case, I think this is a reasonable middle ground and I can get that in! Thanks for the discussion @davidmigloz


Not working properly In-App Dark Mode

it's working fine with os configuration but it's not working properly when i use In-App Dark Mode

What platform you are having issues with?


Improve error message when file path isnt a file

This change just tries to provide a friendlier message when there is a failure reading from the provided file
Before
$ inlyne missing_file.md
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/main.rs:890:60
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

After
$ inlyne missing_file.md
Error: Error reading file at "missing_file.md"

Caused by:
    No such file or directory (os error 2)

@LovecraftianHorror Definitely looks a lot clearer and less of an information overload than just an unwrap. Thanks 👍.
One tiny thing to change is that instead of stating error twice, could the message be changed to Error: Could not read file at "{location}"

Done and done!


Add support for specifying JRE or JDK

When building using the spring boot plugin it would be very useful to be able to pick JRE or JDK for the final build. I would be very grateful if this could be incorporated as an option.

Thank you! 🙏😊


Allow use reserved keywords as variable, argument function and etc.

For example rustc code contains many krate variables just because crate is reserved keyword. I like the way it is implemented in C# with @.
    let @crate = cache.fold_crate(@crate);

The goal is eliminate intentional typos.

You can already do this using _.

You are right. I don't think about it.

AFAIK adding _ changes their name, it does not allow you to use them as identifiers.
The following library would not export a function named fn; it would export a function named _fn.
#![crate_type="lib"]

#[no_mangle]
pub extern fn _fn() -> usize {
    42
}

Note that the main use I know for this kind of features is interop and code generation (example: I have a schema and need to generate structs to represent them; ideally I want to keep the name matching the original ones). It becomes even more interesting when reflection is allowed, as the (type, member, ...) names are usually also observable as strings.

@ranma42 First of all. The _ should be placed after the ident to avoid silencing the unused warnings. Also, @KalitaAlexey propose an @ operator, which is no different from the _, we already got.

@Ticki I tried to move the _ as you suggested: http://is.gd/w77SqT
In the LLVM IR and in the assembly output I still get the wrong name for the function (fn_ instead of fn). How should I call the function so that its name is fn?
In C# @ can be used to "escape" the identifier, but it does not become part of it. I cannot get the same behaviour with _ (but I might be misusing it... a code example would be very useful).

@ranma42 _ will be part of name. It is ok for me.

Yeah, ranma42, I must have misunderstood you, there are currently no way to escape the names, but (unless you're using #[no_mangle]) the names are mangled anyways. A possibility is to add an attribute for setting the name, you want.


Remove dotnet/standard entry

Looks like dotnet/standard is now using Azure DevOps for ci.
@wtgodbe, please confirm it's ok to cleanup the Jenkins master job for Standard

/cc @danmosemsft

@chcosta that's correct, it's safe to remove standard


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_8987a02d-dcc8-4726-8a41-6a5716a1372d/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_8987a02d-dcc8-4726-8a41-6a5716a1372d.git

Docs Build status updates of commit 0eba2e3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-07-08-08-37-21 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[BUG] login always takes you to the last selected tenant

What is the bug?
url query parameter security_tenant is ignored when you log in
How can one reproduce the bug?
Steps to reproduce the behavior:

create a dashboard in a tenant (e.g. tenant-1)
select share > permalink with short url
copy dashboard short link
select Overview
select another tenant from profile > swich tenants (e.g private)
logout
paste dashboard short link and log in
see error dashboards does not exists
check current tenant from profile, you are in the wrong tenant (private)

What is the expected behavior?
After login you should be in tenant specified in security_tenant query parameter
What is your host/environment?
opensearch 2.8.0
opensearch-dashboards 2.8.0
Do you have any screenshots?

Thanks for the detailed report, @agottoli. Closing as a duplicate of #1203. Good news is, there is a PR out to fix this and we're targeting our next release to get this change in.


repo news bloated

When i go to a repo, in the news I get news about who started watching that repo or who forked it.
Who is interested in this kind of news?
Also it is really annoying in popular repos such as twitter/bootstrap for example.
The comments on commits and other issues are gettin burried by these non-news.
is this by design or is it a bug?

Filtering sounds like the best option, continued in #515


Unhandled exception

The Flutter environment：
[✓] Flutter (Channel master, v1.9.8-pre.45, on Mac OS X 10.13.6 17G4015, locale zh-Hans-CN)
• Flutter version 1.9.8-pre.45 at /Users/***/Flutter
• Framework revision 2b7e59b9c0 (3 hours ago), 2019-09-02 04:20:42 -0400
• Engine revision 2baba33c82
• Dart version 2.5.0 (build 2.5.0-dev.4.0 36985859e4)
In flutter_tools ，git apply --3way 0001-aspectd.patch
My project add aspectd，build it，get app.dill
then
dart /Dart/sdk/pkg/vm/bin/dump_kernel.dart app.dill app.dill.txt
Exception：
Unhandled exception:
RangeError (index): Index out of range: no indices are valid: 0
#0      Uint8List.[] (dart:typed_data-patch/typed_data_patch.dart:2207:7)
#1      BinaryBuilder.readByte (package:kernel/binary/ast_from_binary.dart:104:27)
#2      BinaryBuilder.readUint32 (package:kernel/binary/ast_from_binary.dart:124:13)
#3      BinaryBuilder.readComponent (package:kernel/binary/ast_from_binary.dart:444:17)
#4      main (file:///Dart/sdk/pkg/vm/bin/dump_kernel.dart:47:40)

#5      _startIsolate. (dart:isolate-patch/isolate_patch.dart:303:32)
#6      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

@donald99
First you have to get a dart-sdk exactly matching your flutter repo.
Its sha is specified in flutter/bin/cache/dart-sdk/revision.
After that, try this command.
If it remains, you'd better file a issue under https://github.com/dart-lang/sdk/issues
This issue also occurs sometimes in my side.

Close this issue as believe not related to aspectd.


Parquet UTF-8 max statistics are overly pessimistic

Describe the bug
A recent comment regarding incrementing UTF-8 encoded characters got me wondering about the implementation in parquet-rs. It turns out that the increment_utf8 function in the parquet::column::writer module isn't optimal.
Take for example the Unicode character U+00FF ('ÿ'), which is encoded as 0xC3BF or b11000111 b10111111. Incrementing this code point by one should yield U+0100 (0xC480 or 'Ā'). increment_utf8, however, will yield instead U+013F ('Ŀ'), due to how overflow is handled.
https://github.com/apache/arrow-rs/blob/08ac58761cb9a80b6ffd651727aa957db0c32bd7/parquet/src/column/writer/mod.rs#L1444-L1451
What happens in this case is the lower byte 0xBF is incremented to 0xC0 which is not a valid UTF-8 continuation character. The byte is then left alone and the next higher byte 0xC3 is incremented to 0xC4, yielding a final UTF-8 encoded char of 0xC4BF which is Unicode U+013F. This is analogous to incrementing the high nybble of an ASCII character, for instance 'O' (0x4F) becoming '_' (0x5F) rather than 'P' (0x50).
The end result is still a valid upper bound, but it is not as good as it could be and could result in some row groups or pages being read when they don't actually need to be.
To Reproduce
Add the following to test_increment_utf8 and it will pass.
        let s = "\u{ff}\u{ff}";
        let s1 = "\u{ff}\u{100}";
        let s2 = "\u{ff}\u{13f}";
        let v = increment_utf8(s.as_bytes().to_vec()).unwrap();
        assert_eq!(v, s2.as_bytes());
        assert_ne!(v, s1.as_bytes());

Expected behavior
The above test should fail, i.e. the incremented string should be "ÿĀ" rather than "ÿĿ".
Additional context
One possible fix would be to do something more akin to what is being done in https://github.com/apache/datafusion/pull/12978, where the string data is first converted to chars, incremented, and then converted back to UTF-8. This can result in some characters increasing in size, which may not be desired here considering we're trying to truncate to less than a certain number of bytes. Another alternative is to set overflowed continuation bytes to 0x80 (the minimum valid continuation byte) when moving on to the next higher byte. If the character is at a byte width boundary, the bytes for that character can be set to 0 and the next higher character can be incremented. This would not increase the byte count, but can result in less optimal upper bounds (but still better than the current ones).

cc @alamb, more fallout from dinner 🤣

Changing to enhancement as bound is technically still valid.
If memory serves this logic was based on another implementation, but I can't remember precisely

https://github.com/apache/parquet-java/blob/4aeba6cb7348a8fb57d8585058d27f53ce592b48/parquet-column/src/main/java/org/apache/parquet/internal/column/columnindex/BinaryTruncator.java#L155
I guess

One possible fix would be to ...

It would be amazing if we figured out how to do this truncation / incrementing / decrementing correctly in once place (the parquet crate) and then just reused the same logic in datafusion
Thank you @etseidl @tustvold and @mapleFU  for your insights

It would be amazing if we figured out how to do this truncation / incrementing / decrementing correctly in once place (the parquet crate) and then just reused the same logic in datafusion

They are slightly different use cases, though. Here we're taking an N-character M-byte string and truncating it to no larger than T bytes, but it may be smaller due to character boundaries, and then incrementing the final character if possible. Since we're constrained by the size of the vector of bytes we're operating over, we can't promote a 2-byte character to a 3-byte, and so get a less ideal bound. What @adriangb et al are doing in datafusion is a bit different. There they have a prefix that they want to increment, but they're not constrained by size, so are free to switch to wider characters if necessary. We could do the same in parquet-rs if we were willing to have a truncated max statistic that's 1 byte larger than requested (which seems ok to me as long as it's communicated that the truncation is a best effort, just like with page and row group sizes).
I've submitted #6870 which continues with the do-no-overshoot approach. If we want to relax the bounds a bit, then we could adopt what's being proposed in https://github.com/apache/datafusion/pull/12978. I'd also like to do some testing to see if there are performance impacts (although I'd expect these to be minimal given the truncation happens at most once per page and column chunk).

label_issue.py automatically added labels {'parquet'} from #6870


Allow templates to include other templates

This feature may require breaking compatibility, so it should either be implemented or rejected before the release of version 1.0.0.

Exposing ::tclssg::templating::* to the interpreter would get you almost all of the way there but then because of how prepare-content and apply-template work you would have to spawn another safe interpreter for every level of nesting. That's requires a potentially big overhaul and may negatively impact the performance. If you don't spawn separate interpreters the best you can implement is the equivalent of a tail call for templates (discard the current interpreter, start a new one transferring select variables to it). The latter option would be considerably weird and not very practical.
However, just exposing ::tclssg::templating::parse and introducing a way to read template files like you can read data files (except with directory overrides as always for templates) would allow templates to read and eval other templates. Sounds like a possible "90% solution".


Version 2.0

Major revision to make pipeline accept nanopore data. Create separate workflows for Illumina and nanopore sequence data.
Identify best tools for QC, assembly, polishing of nanopore data.

For Polishing Medaka/PEPPER/Homopolish.


Prepare shell app for IVY

There is a visual issue with quantity when IVY is enabled. No other issues found. As it's not a priority, I pause this task before 1.0.


add project_root_callback for tsserver

This PR simply adds in a project_root_callback for tsserver to make tsserver work as intended in projects that have nested tsconfig.json files.
I tested these changes in a repo I was having issues with and these changes resolved my problem.
Let me know if you need anything further.

@w0rp Look those over when you get time and let me know if I did that correctly.
Thanks

Cheers! :beers:


Is virtual network (VNet) data gateway in Preview mode?

This title confuses. The feature is not in Preview mode, however this article title tells that this is Preview. I haven't found any confirmation that VNet data gateways are in preview.
Thanks.

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

ID: d47e61c8-a5b8-0a8b-06dd-c242a0acd06c
Version Independent ID: e80385c1-b254-525d-b5ec-2b653e7763d8
Content: What is a virtual network (VNet) data gateway (Preview)
Content Source: data-integration/vnet/overview.md
Service: virtual-network-data-gateway
GitHub Login: @nikkiwaghani
Microsoft Alias: nikkiwaghani

Hi @zakharRealogic
VNet data gateways are currently in public preview.


Example Notebook for testing

Hi
Could you provide a simple notebook to test the output of the model on personal websites?
Thanks

This looks like the same issue as #3. I will add this feature in the near future.

Hi Bruno, long time no see, I updated the deployment code for neuscraper. Soon I will write some tutorials about it.


Support in-cluster private registries

A common way to do local development is to deploy an in-cluster registry. e.g., you run a registry in-cluster at localhost:32000, expose it on the local network, and push/pull to it directly.
The Microk8s team wrote up some great documentation on how people do this here:
https://github.com/ubuntu/microk8s/blob/04144c0466e6a079035ca66bac7f23522a4d81d0/docs/working.md#working-with-a-private-registry
Working with a private registry has a lot of operational complexity. You need to set up the registry in-cluster, port-forward the registry to localhost, and tag images with the new registry name. Tilt should be able to interoperate with this workflow.
Some of this work might involve feature work, or Tiltfile API hooks, or just documentation on how to do this.

I created a sample repo that demonstrates how to do this
https://github.com/windmilleng/kind-local
going to send to the Kind team to get feedback on it, maybe it's something we can get more cluster management tools to support natively

Here are guides on how to do this for

KIND - https://kind.sigs.k8s.io/docs/user/local-registry/
K3D - https://github.com/rancher/k3d/blob/master/docs/examples.md#connect-with-a-local-insecure-registry

We mainly need a better protocol for detecting these registries when they exist. You can connect to them manually with the Tiltfile default_registry() function (https://docs.tilt.dev/api.html#api.default_registry)

Tilt now can read the local registry from Node annotations. We have a blog post on how to do it here: https://blog.tilt.dev/2020/02/11/delete-clusters-faster-with-kind.html


Spike: What work has been completed on the Character Check Consent Form?

It looks like the great work that Enor has completed on the Character Check Consent Form has been implemented on staging. However, we need to undertand whether all workflows related to this feature have been completed and where this functionality has been released.
It looks like the great work Enor completed on the character consent form has been put on stagging. We need to understand if all of the elements of this work have been completed and the extent of its implementation.
Please confirm the follwing:

[ ] Whether the consent form has been realeased on Production, in anyway eg. realeased but hidden.
[ ] We can see the form on staging, but has the required work been completed on Admin? we need to be able to view the information and for it to be included in the automated JO handover.
[ ] Does the current consent form include questions for all of the data we need to collect A) for character check consent and B) the JO handover? (Kate and Joy are more responsible for this task)

There is a consent form present but hidden in the project on the Apply side (as per the Apply prototype). We need to check whether information requested is still relevant and correct and update links if required. (Title field need to be changed as per gov.design guidelines and changes to first/last names as per recent changes in #615/#616).

On Admin side some of the collected information is displayed in Applications->Candidate->Agency report. We need to update to reflect title, first/last names changes +/or do we need another tab Consent Check to display all the info we collect? We don't have the "Send email" button functionality which would send consent check form to candidates using Notify as per Admin prototype.

Hello @mbrookeswebdev, you're right that we can't send emails from the platform yet, but can we set it up the same way IAs are currently set up? assessors recieve a link and then log-in to upload the assessment. If we can replicated that or set it up so that candidates can receive a notification and then log in and complete the form, or could we generate a link that the team then sends via notify?
A link for notify isn't the best way forward, but it would do for an MVP, get what we need for the JO transfer and still be a massive imporvement on the way teams currently collect this information.

@KateMJAC I believe IAs are set up using the Notify API. I've also found some unused code in the digital-platform dealing with sending consent checks email. I'll need to discuss with Warren whether this can be updated if needed and used to send consent checks requests.

This work is now being done via


Replaced "functionThreshold/constructorThreshold" properties of LongParameterList rule.

Replaced "functionThreshold" and "constructorThreshold" properties of LongParameterList rule by "allowedFunctionParameters" and "allowedConstructorParameters" (#3679)
The rule reported an issue when the amount of parameters of a method or constructor is greater or equal the value defined for the  "XThreshold" properties.
It is intended that the value defined in the rule is the maximum allowed parameter amount. So the properties are renamed to
"allowedXParameters" and the check for reporting an issue is changed to only check for greater, no longer for equal.
The origin issue is https://github.com/detekt/detekt/issues/3679

Warnings

:warning:
This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

  Generated by :no_entry_sign: dangerJS against 65ece68afbb85173765e6e714501bdcade73545e

What went wrong:
Execution failed for task ':detekt-generator:verifyGeneratorOutput'.
At least one generated configuration file is not up-to-date. You can execute the generateDocumentation Gradle task to update generated files then commit the changes.


Apply fails when running in Alpine Linux (musl libc) environment

I'm trying to run a migration via terraform within a docker environment based on the docker:27.0.2 image, which itself is built on Alpine Linux.  I receive the following error:
╷
│ Error: Failed to read migration status
│
│   with module.redacted_db.data.atlas_migration.redacted_db,
│   on modules/redacted/migrations.tf line 18, in data "atlas_migration" "redacted_db":
│   18: data "atlas_migration" "redacted_db" {
│
│ fork/exec
│ /builds/user/redacted/devops/.terraform/providers/registry.terraform.io/ariga/atlas/0.4.7/linux_amd64/atlas:
│ no such file or directory
╵

I've found that this is occurring because Alpine uses musl libc rather than glibc, and the atlas binary packaged with the terraform provider was compiled against glibc.  The binary is definitely present at /builds/user/redacted/devops/.terraform/providers/registry.terraform.io/ariga/atlas/0.4.7/linux_amd64/atlas, but the no such file or directory likely indicates that a shared object can't be found:
$ ldd /builds/user/redacted/devops/.terraform/providers/registry.terraform.io/ariga/atlas/0.4.7/linux_amd64/atlas
	/lib64/ld-linux-x86-64.so.2 (0x7fe35f568000)
	libdl.so.2 => /lib64/ld-linux-x86-64.so.2 (0x7fe35f568000)
	libpthread.so.0 => /lib64/ld-linux-x86-64.so.2 (0x7fe35f568000)
	libc.so.6 => /lib64/ld-linux-x86-64.so.2 (0x7fe35f568000)
Error relocating /builds/user/redacted/devops/.terraform/providers/registry.terraform.io/ariga/atlas/0.4.7/linux_amd64/atlas: __memcpy_chk: symbol not found
Error relocating /builds/user/redacted/devops/.terraform/providers/registry.terraform.io/ariga/atlas/0.4.7/linux_amd64/atlas: __memset_chk: symbol not found
Error relocating /builds/user/redacted/devops/.terraform/providers/registry.terraform.io/ariga/atlas/0.4.7/linux_amd64/atlas: fcntl64: symbol not found

Eek, I created this issue and then found this: https://github.com/ariga/terraform-provider-atlas/issues/72
I will upgrade my version and close this ticket if it succeeds.

Success with version 0.5.9.  Sorry for the duplicate!


Spatial-LDA Summary and Future Functionality

Here is a brief summary of the current spatial-LDA integration into the ark pipeline and some things users should be aware of:
Notebooks:
There are currently two notebooks for spatial-LDA.  One is for processing and formatting data in the form of a cell table into a form compatible with the spatial_lda library from Calico.  This notebook also contains some functions for calculating and plotting exploratory metrics related to parameter tuning.  The other notebook runs the actual training and inference of the spatial-LDA algorithm and also supports some visualizations for inspecting the results.
Parameter Values:
The two main parameters which the user needs to specify are (1) the size of the neighborhood radius in the featurization step, and then (2) the number of topics in the training step.  There are functions in the processing notebook to help the user explore potential reasonable values for these parameters.  Users can feel free to adjust the difference penalty as well, but this has not been shown to make too great a difference in the end result.
Parallel Processing and Improving Runtime:
The other important value users can specify is the number of parallel processes to use locally which will make a difference in runtime.  Users have the option to also change the primal_tol and threshold arguments during training in order to cut down on erroneous looping, but users are encouraged to test this out on a subset of data before making any final decisions regarding the models trained on their particular samples.
Some thoughts about further runtime improvements are to use a C-based solver from the CVXPY library to possibly replace the admm.py module.  Another thought is to cython-ize admm.py.
Additional Visualizations and Metrics:
A useful plot which is not currently implemented would be a scatter plot of the cell locations in an FOV (similar to that of plot_fovs_with_topics()), but shades the color of the points based on the probability of that cell belonging to a given topic.  It may also be worth adapting the current code for the gap statistic in processing.py to work with a trained or complete spatial-LDA model since this metric can be applied to almost any clustering technique.  (While spatial-LDA is not a clustering technique in the traditional sense, these hierarchical probability models can be generalized as mixture models similar to the way k-means can be.)
Next Steps:
The most up-to-date branch in the spatial-LDA pipeline is LDA_Training_and_Inference.  As it is now, this branch could likely be merged into the spatial_LDA branch and then merged into master unless people wish to implement any of the above mentioned (or other) changes.
Both notebooks are functional and have been tested using real data.  The notebooks in the LDA_Training_and_Inference  branch have been tested on a subset of 8 FOVs which contain a total of 10,740 cells.  Using a 2020 MacBook Air with 8 available CPUs, 16GB RAM, and 250GB disk space, I was able to train a spatial-LDA model on 75% of the data (8,055 cells) with 4 CPUs in about 15 minutes.  Running inference on all 10,740 cells took about 7 minutes with 4 CPUs.

Closed by #437


Use URL.createObjectURL for reading local files

Somehow I overlooked this method of extracting data from a local file. Now that
we have a real URL for a local file that does not break Cross Origin Request
rules, we can go back to using an <audio> tag for playing our media which
greatly simplifies much of the media functionality.
The main advantage here is that compressed audio files can be decoded as they play, instead of having to be decoded and put into memory before they could start playing which was terrible.
I still need to do some testing, so I'm opening this as a pull request so myself and others can give it a once-over.
One known issue, is the default media file hosted on RawGit does not seem to be including the content-length header which causes the duration to be reported as Infinity if the media file takes too long to completely load.
Test Here

Chanel count does not seem to work.

Looking for help with the channel question here: http://stackoverflow.com/questions/32835149/how-can-i-detect-the-number-of-audio-channels-in-an-mp3-in-an-audio-tag

cdn.rawgit.com does not return the content-length header which breaks seeking. Using non-cdn version  while I wait to hear why: https://github.com/rgrove/rawgit/issues/81


CTFE: 6984 7143 7144

6984 is a big diagnostic improvement, it allows CTFE to continue in most cases. The hard work was done in my earlier CTFE pull request.
7144 is a simple wrong-code bug. 7143 is a straightforward missing case.
6984 CTFE generates a torrent of spurious errors, if there was a previous error
7143 [CTFE] cannot compare class references with "is"
7144 [CTFE] base class does not call overridden members
http://d.puremagic.com/issues/show_bug.cgi?id=6984
http://d.puremagic.com/issues/show_bug.cgi?id=7143
http://d.puremagic.com/issues/show_bug.cgi?id=7144

This causes testconst.d to seg fault with the 64 bit build of dmd under Linux.


remove fte from the list

Thanks, Maira! I'll look into why Rob was included on the report and tweak the code accordingly.


Feature : Adding Slider to Top Artist and enhancing it

Describe the feature
We need to add a slider to the "Top Artist" component to enhance its functionality and user experience. The slider will allow users to smoothly navigate through a list of top artists, making the component more interactive and user-friendly.
Add ScreenShots

Record

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

@richab246 you can work on it. But after @The-Phoenics completes his part.


[SPARK-11774][SPARKR] Implement struct(), encode(), decode() functions in SparkR.

Test build #46207 has started for PR 9804 at commit a199d79.

Test build #46207 has finished for PR 9804 at commit a199d79.

This patch fails R style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46207/
Test FAILed.

Merged build finished. Test FAILed.

rebased to master

Test build #46863 has started for PR 9804 at commit b4b0a61.

Test build #46863 has finished for PR 9804 at commit b4b0a61.

This patch fails R style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class Count(children: Seq[Expression]) extends DeclarativeAggregate \n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46863/
Test FAILed.

Test build #46869 has started for PR 9804 at commit f5add6e.

Test build #46869 has finished for PR 9804 at commit f5add6e.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46869/
Test FAILed.

Merged build finished. Test FAILed.

Test build #46877 has started for PR 9804 at commit b253205.

Test build #46877 has finished for PR 9804 at commit b253205.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

LGTM. Thanks @sun-rui -- Sorry for the delay in reviewing this. Merging this to master and branch-1.6


Changes made on the default parameters of Text Vectorizer

The parameter in Text Vectorizer, which is pad_to_max_tokens has been changes from True to False.
From The Video:

From the Docs

I don't know if it will make my model perform worse or not. I haven't trained my model yet. But I did notice something which is, even if I set pad_to_max_tokens= False which is by default then also it is padding the integer tensor with zeroes at the end

So I am a bit confused now. Let me train my model and check the accuracy score then I think the confusion will go away.
My Text Vectorizer Config:

I don't think it any or little effect on the outcome of the project. All worked out perfectly for me. I don't know I am still a bit confused about it.


Update Selenium to 3.13.0

Update Selenium to latest version.
Update Guava to match version used by Selenium.
Related to zaproxy/zaproxy#4776 - Update Selenium

This is now updating to 3.13.0.


Fix/types in tests

I merged everythinf from here except the events library dependency, which we will discuss further in https://github.com/GevatterGaul/epubts/pull/3


Introduce a pulsar log4j2 appender

Motivation
when developing pulsar-functions, it would be good to log the log messages generated by functions to pulsar, so the user can easily tail the log messages topic to get all the messages per functions.
Modifications
This PR is to add a pulsar based appender.
Result
We are able to use pulsar log4j2 appender to log messages.

NOTE:
Ideally I would like to contribute this module as part of log4j2 - https://github.com/apache/logging-log4j2 . However we would like to use this module for pulsar-functions in #1314 , so make the initial contribution here. At the same time, I would send a PR to log4j2 as well.

filed a jira issue to log4j2 to contribute this appender back to log4j2 : https://issues.apache.org/jira/browse/LOG4J2-2280

retest this please

retest this please

retest this please

@merlimat I have addressed all your comments.


Sort population by age-group before plotting

This PR aims to fix #1506
I have sorted the dataframe by pre-defined age categories available via make_age_grp_types.categories and checked that population pyramid is now being plotted correctly.
Incidentally, I wonder if this line contains a typo? Since we are looping over sex,I expected data[_sex][_source].
https://github.com/UCL/TLOmodel/blob/58256ce671ba316ba55534357af5c516ac5faaa5/src/scripts/calibration_analyses/analysis_scripts/analysis_demography_calibrations.py#L201

Incidentally, I wonder if this line contains a typo? Since we are looping over sex,I expected data[_sex][_source].

This is because both sexes have the same sources - it doesn't matter which you use.


TagLib::MPEG slower than TagLib::FileRef for wma files

Using Taglib::MPEG to parse the audio metadata of .wma files is several (10x) times slower than TagLib::Fileref
testcase
require 'optparse'
require 'stackprof'
require 'taglib'

mpeg = false
OptionParser.new do |opt|
	opt.on('--mpeg') {|result| mpeg = result}
end.parse!

StackProf.run(mode: :cpu, out: 'stackprof.dump') do
	Dir.glob(ENV["HOME"]+ "/Music/**/*.wma") { |f|
		if mpeg
			TagLib::MPEG::File.open(f, false) do |file|
				# do something with file
			end
		else
			TagLib::FileRef.open(f) do |file|
				# do something with file
			end
		end
	}
end

stackprof results

(23:07) In [134]: time !ruby test.rb
CPU times: user 0 ns, sys: 10 ms, total: 10 ms
Wall time: 1.38 s

(23:08) In [135]: !stackprof stackprof.dump
==================================
  Mode: cpu(1000)
  Samples: 142 (0.00% miss rate)
  GC: 0 (0.00%)
==================================
     TOTAL    (pct)     SAMPLES    (pct)     FRAME
       142 (100.0%)         142 (100.0%)     TagLib::FileOpenable#open
       142 (100.0%)           0   (0.0%)     block (2 levels) in <main>
       142 (100.0%)           0   (0.0%)     block in <main>
       142 (100.0%)           0   (0.0%)     <main>
       142 (100.0%)           0   (0.0%)     <main>

(23:08) In [136]: time !ruby test.rb --mpeg
CPU times: user 3.33 ms, sys: 0 ns, total: 3.33 ms
Wall time: 20.7 s

(23:08) In [137]: !stackprof stackprof.dump
==================================
  Mode: cpu(1000)
  Samples: 1053 (70.45% miss rate)
  GC: 0 (0.00%)
==================================
     TOTAL    (pct)     SAMPLES    (pct)     FRAME
      1053 (100.0%)        1053 (100.0%)     TagLib::FileOpenable#open
      1053 (100.0%)           0   (0.0%)     block (2 levels) in <main>
      1053 (100.0%)           0   (0.0%)     block in <main>
      1053 (100.0%)           0   (0.0%)     <main>
1053 (100.0%) 0 (0.0%) <main>

Does TagLib::MPEG even find any tags in .wma files? Because .wma files don't use an MPEG container, so that would surprise me a bit. So maybe it scans the whole file just to find nothing.
FileRef on the other hand detects the type and uses the appropriate parser.

Sorry, FileLib::MPEG does not. I was extracting audio tags from .mp3 and .wma files together and did not notice. Is there a correct way to obtain audio tags from .wma files that is not in TagLib::FileRef?

Is there a correct way to obtain audio tags from .wma files that is not in TagLib::FileRef?

Nope, that is currently the only way (and it's unlikely that we'll add specific API for that). Is there anything wrong with FileRef or do you need more information than what it returns?

There is nothing wrong with it, it is just missing albumArtist.
Thank you for your help.


how can I optimize the highlight performance

I need some help for highlight performance. Could you give me some advice? Thank you!  ^_^
when I highlight a json. the application use about 10 seconds. the StyleSpans have 17396 span count in one paragraph. how can I optimize the program to make the application run faster?
demo application : https://github.com/FrogIf/FrogJson/tree/highlight-performance, branch: highlight-performance.

I have use token count limit to avoid.


debug

Enhancement:
Reason:
Result:
Issue Tracker Tickets (Jira or BZ if any):

[test]

[test]

[test]

[test]

[test]

[test]

[test]


Check if it's possible to send the plot images trough the Github API

@mendesfabio That's an API that converts an HTML file (exported by panel) to an image and sends back only the link to the image on the cloud: https://htmlcsstoimage.com/. Would be great to use it, because then we do not need to worry about sending an image file through the GitHub API, just the image url.

We're not using the GitHub API by now. Since using the output scenario table as an image, the 2k char limit of the URL will be enough.


IRI connection retry attempts before error / quit

Working on getting Nelson working alongside IRI using Docker as Docker seemed to be the preferred route to take for deployments during early discussions in Slack.
I am currently experimenting with a Docker Compose file  for setting up the entire stack with a single command  however, Nelson throws and error and quits as there is a slight race condition and it tries to connect to IRI before the IRI container is ready.  I can workaround this by simply running docker-compose start nelson slightly after the initial docker-compose up but this isn't very user friendly.  It would be great if Nelson could simply try again a couple of times before erroring out.  Is that a possibility?
If you would prefer not to implement this there are a few possible workarounds:
Option 1
Add a healthcheck to the IRI service, in Docker Compose you can do something like:
healthcheck:
  test: ["CMD", "curl", "-f", "http://localhost"]
  interval: 1m30s
  timeout: 10s
  retries: 3

The 'official' IRI Docker images does not have curl or anything else suitable to carry out a check so we'd need to create a custom image which ideally I'd like to avoid.  Also, this is a Docker Compose thing and would not help improve matters if you were deploying containers via some other method.
Option 2
Amend the Nelson Dockerfile to change it to start via a small shell script that wraps the whole thing in a loop, something like:
while ! nc -z iri 14265; do sleep 3; done

I think option 2 is probably the better of the 2 here if you do not wish to make the code handle this.  I'd love to hear your thoughts.  This sort of thing is a common issue with Docker deployments, some further info and ideas can be seen here:  https://docs.docker.com/compose/startup-order/

Apologies, I completely forgot to post an example error about the issue I spoke about here.
nelson_1      | Unhandled Rejection at: Promise Promise {
nelson_1      |   <rejected> Error: Invalid Response: Error: connect ECONNREFUSED <IP_ADDRESS>:14265
nelson_1      |     at Object.exports._errnoException (util.js:1022:11)
nelson_1      |     at exports._exceptionWithHostPort (util.js:1045:20)
nelson_1      |     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1087:14)
nelson_1      |     at Object.invalidResponse (/usr/local/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/errors/requestErrors.js:5:12)
nelson_1      |     at makeRequest.prepareResult (/usr/local/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:160:24)
nelson_1      |     at request.onreadystatechange (/usr/local/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:62:25)
nelson_1      |     at dispatchEvent (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:591:25)
nelson_1      |     at setState (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:610:14)
nelson_1      |     at handleError (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:532:5)
nelson_1      |     at ClientRequest.errorHandler (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:459:14)
nelson_1      |     at emitOne (events.js:96:13)
nelson_1      |     at ClientRequest.emit (events.js:188:7)
nelson_1      |     at Socket.socketErrorListener (_http_client.js:310:9) } reason: Error: Invalid Response: Error: connect ECONNREFUSED <IP_ADDRESS>:14265
nelson_1      |     at Object.exports._errnoException (util.js:1022:11)
nelson_1      |     at exports._exceptionWithHostPort (util.js:1045:20)
nelson_1      |     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1087:14)
nelson_1      |     at Object.invalidResponse (/usr/local/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/errors/requestErrors.js:5:12)
nelson_1      |     at makeRequest.prepareResult (/usr/local/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:160:24)
nelson_1      |     at request.onreadystatechange (/usr/local/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:62:25)
nelson_1      |     at dispatchEvent (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:591:25)
nelson_1      |     at setState (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:610:14)
nelson_1      |     at handleError (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:532:5)
nelson_1      |     at ClientRequest.errorHandler (/usr/local/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:459:14)
nelson_1      |     at emitOne (events.js:96:13)
nelson_1      |     at ClientRequest.emit (events.js:188:7)
nelson_1      |     at Socket.socketErrorListener (_http_client.js:310:9)
nelsoncli_nelson_1 exited with code 0

Option 2 looks like a "hack" to me.
in my opinion the better way is to not error out nelson. It should just try to connect to the api, when it doesnt work, wait a few seconds and try again?

@H3npi Both options are pretty hacky, they are just necessary workarounds if the connection retry is not going to be implemented within Nelson itself.

And what about a ECONNRESET? After install I only get this with a locally installed IRI <IP_ADDRESS> up and running and no firewall:
Unhandled Rejection at: Promise Promise {
  <rejected> Error: Invalid Response: Error: read ECONNRESET

An intermediate solution can be found here: https://github.com/jinnerbichler/iota-node-docker
I am using a separate service to wait for the IRI node to start up.

I ran it by hand after IRI was full loaded.
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   597  100   572  100    25    572     25  0:00:01  0:00:01 --:--:--   548
{
    "appName": "IRI",
    "appVersion": "<IP_ADDRESS>",
    "jreAvailableProcessors": 4,
    "jreFreeMemory": 408453744,
    "jreVersion": "1.8.0_151",
    "jreMaxMemory":<PHONE_NUMBER>,
    "jreTotalMemory":<PHONE_NUMBER>,
    "latestMilestone": "9NUUINVENCYBQXDIUTDDOKMFVDLKWMNJLUNBTKIQRKVXPH9CROKOKIPAHIQIDNARNSWZUMQDOTXTA9999",
    "latestMilestoneIndex": 305511,
    "latestSolidSubtangleMilestone": "QYKENTHDNINZUPUM99FCDYNREQSMPPRLDYJEBWZXWQFD9UHPTJPFTPRSKKKCJOZUKGXKUVCQYAQK99999",
    "latestSolidSubtangleMilestoneIndex": 305408,
    "neighbors": 5,
    "packetsQueueSize": 0,
    "time":<PHONE_NUMBER>402,
    "tips": 3297,
    "transactionsToRequest": 3,
    "duration": 12
}

root@...:~# iptables -L
Chain INPUT (policy ACCEPT)
target     prot opt source               destination

Chain FORWARD (policy ACCEPT)
target     prot opt source               destination

Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination

root@....:~# ./nelson.sh
Unhandled Rejection at: Promise Promise {
  <rejected> Error: Invalid Response: Error: read ECONNRESET
    at _errnoException (util.js:999:13)
    at TCP.onread (net.js:629:25)
    at Object.invalidResponse (/usr/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/errors/requestErrors.js:5:12)
    at makeRequest.prepareResult (/usr/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:160:24)
    at exports.XMLHttpRequest.request.onreadystatechange (/usr/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:62:25)
    at exports.XMLHttpRequest.dispatchEvent (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:591:25)
    at setState (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:610:14)
    at exports.XMLHttpRequest.handleError (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:532:5)
    at ClientRequest.errorHandler (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:459:14)
    at ClientRequest.emit (events.js:159:13)
    at Socket.socketErrorListener (_http_client.js:389:9)
    at Socket.emit (events.js:159:13) } reason: Error: Invalid Response: Error: read ECONNRESET
    at _errnoException (util.js:999:13)
    at TCP.onread (net.js:629:25)
    at Object.invalidResponse (/usr/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/errors/requestErrors.js:5:12)
    at makeRequest.prepareResult (/usr/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:160:24)
    at exports.XMLHttpRequest.request.onreadystatechange (/usr/lib/node_modules/nelson.cli/node_modules/iota.lib.js/lib/utils/makeRequest.js:62:25)
    at exports.XMLHttpRequest.dispatchEvent (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:591:25)
    at setState (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:610:14)
    at exports.XMLHttpRequest.handleError (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:532:5)
    at ClientRequest.errorHandler (/usr/lib/node_modules/nelson.cli/node_modules/xmlhttprequest/lib/XMLHttpRequest.js:459:14)
    at ClientRequest.emit (events.js:159:13)
    at Socket.socketErrorListener (_http_client.js:389:9)
    at Socket.emit (events.js:159:13)

Maybe my exec is wrong (without --gui):
nelson --neighbors "mainnet.deviota.com/16600 mainnet2.deviota.com/16600 mainnet3.deviota.com/16600 iotairi.tt-tec.net/16600"

If I try with gui it just pops up for less than a second.
Why should I use docker? I have Ubuntu 17, nodeJS, Java and IRI on it. What else does it need to run?

@ortreum please hit me up in slack (@H3n) and or create a new issue, since this is not the right place (or do you run docker?)

@ortreum , make sure your IRI has API listening locally on the default port: 14265. Otherwise you will need to provide Nelson with a correct port.

@rosstimson , I will fix that. Makes sense!

Fixed with 0.2.1


As an unorganised SoC student, I can sort the courses by alphabetical order so that I can

I can find a specific course easily

Updated: As an unorganised SoC student, I can sort the contacts according to the number of courses taken so that I can find a specific course easily.


Add "blob-container-named" service plan to Storage

It would be useful to have a service plan for Storage which allowed users to pass a container name as parameter. This would be similar to the currently available blob-container plan, but it would accept as parameter the container name (or better a list of container names) to be created.

Hi @norshtein, in my specific case I need to be able to create a few containers inside the storage. To replicate the environment I am testing these containers would ideally have the same names as in the original environment. At the moment I do all this manually after OSBA has created the storage space. This works but I suspect it could be a common enough use case to be considered for an enhancement of OSBA.

It makes sense. I'll add this enhancement.

Hi @luigibk , this functionality has been added in recent refactoring in storage module, feel free to re-open the issue if you have any question.

@norshtein would it be possible to create a list of blob containers and not just one?

Hi @timja , currently not. That's because creating a list of blob container is not an "atomic" operation. Almost in every place, for example, Azure Portal, you can only create a single blob container, and we think OSBA should observe this rule too.

Is there any work around using OSBA to create more than one?

You may create an azure-storage-blob-storage-account instance and then create several azure-storage-blob-container instances upon it.


TEM2015 Test 02: URL in document links for VCP and Lodgment Planning are not working

Noticed that the url in the document links are not working, as in when you click on the url it takes you to a new page, with a 404 error.  Using the dropdown below the thumbnail to "open" also returns a 404.
https://[testserver05]/arcgis/home/item.html?id=18578392acfb40a5b55fabfee4bb65a6  -- vcp
https://[testserver05]/arcgis/home/item.html?id=5edf7000d0c14f8695eca14386f35ab2   -- lodgment
don't know how widespread this problem is, i did not go through checking other document links, i actually only mistakenly discovered this while testing something else.... recommended all document links be checked

@dfoll The "Downloads" folder is still copying (it takes hours) to the test server so not all links we work right now. Only EmergencyManagment and Intel are copied right now. In addition I believe @lfunkhouser still needs to rename folders and/or files in the build "Downloads" folder.

check.
@elinz feel free to close this then if you are confident that it will be fine after the downloads folder is done copying

Assigned to @dfoll for retesting. Also, I do not consider this a showstopper for the release so even if it doesn't get fixed we will proceed with the release.

these two are working now, however i am seeing this behavior with the visibility and range document link... will open a new issue for that


fix opsrecipe links

This PR:

fixes ops-recipe names so we generate working links

Checklist

[x] Update changelog in CHANGELOG.md.
[ ] Request review from oncall area, as well as team (e.g: oncall-kaas-cloud GitHub group).
[ ] Alerting rules must have a comment documenting why it needs to exist.
[ ] Consider creating a dashboard (if it does not exist already) to help oncallers monitor the status of the issue.

We need a validator of sorts :D


⚠ Bofrost.lu has degraded performance

In 935c9af, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 959 ms

Resolved: Bofrost.lu performance has improved in c56e5da.


Add new plugin to export data to ElasticSearch

Current plugin architecture is more geared towards writing into a local file. As part of this enhancement I would like to:

Tweak the plugin APIs so that update takes in the config and not just the filename.
Introduce a parameter called local data exporter versus remote data exporter so that create function is not called every time based on filename check.
Create a new plugin to export data to Elastic.
I can work on raising a PR to implement this, but will need some discussion since some of the plugin APIs will need to change to make them more generic.

Hi, thanks for the feedback / request

Current plugin architecture is more geared towards writing into a local file

The plugin base is already generic enough to allow implementation of an ElasticSearch plugin, i think you might be too focused on how the whisper plugin is set up :)
The whisper plugin extends the rrd plugin which was specifically created with create and update functions because it makes sense in that context.
We have other plugins like the zeromq plugin that initiate a connection once and then write to that connection - which sounds like an approach closer to what you are looking for and not at all based on local files.
Have a look at plugins/init.py and plugins/zeromq.py

Tweak the plugin APIs so that update takes in the config and not just the filename.

The entire config dict for a plugin is available on self.pluginmgr_config, no need to expose it to the update function :)

Introduce a parameter called local data exporter versus remote data exporter so that create function is not called every time based on filename check.

Not sure i understand this correctly, but if the requirement is to only do something once per plugin instance init you can already do that by overriding the plugin's init function.  If i misunderstood, please elaborate the use case :)

Hi,
I was looking at the TimeSeriesDb class and trying to inherit from it since Elastic is a time series db. But you are right, I can inherit directly from EmitBase and get the right implementation similar to ZeroMQ.
I will dig more into ZeroMQ.
Thanks,
Ramkumar

On Oct 22, 2019, at 5:27 PM, Stefan Pratter<EMAIL_ADDRESS>wrote:
Hi, thanks for the feedback / request
Current plugin architecture is more geared towards writing into a local file
The plugin base is already generic enough to allow implementation of an ElasticSearch plugin, i think you might be too focused on how the whisper plugin is set up :)
The whisper plugin extends the rrd plugin which was specifically created with create and update functions because it makes sense in that context.
We have other plugins like the zeromq plugin that initiate a connection once and then write to that connection - which sounds like an approach closer to what you are looking for and not at all based on local files.
Have a look at plugins/init.py and plugins/zeromq.py
Tweak the plugin APIs so that update takes in the config and not just the filename.
The entire config dict for a plugin is available on self.pluginmgr_config, no need to expose it to the update function :)
Introduce a parameter called local data exporter versus remote data exporter so that create function is not called every time based on filename check.
Not sure i understand this correctly, but if the requirement is to only do something once per plugin instance init you can already do that by overriding the plugin's init function. If i misunderstood, please elaborate the use case :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@vegu There is no CLA to be signed for contributing to this project right?

@rgowrishankar correct on the CLA


feat: Add default Cache-Control headers to upload functions

Added default Cache-Control headers to Upload, UploadFile, and UploadMultipart methods.
Updated "Serve"to use the new default Cache-Control directives constant.
Declared a constant for the default Cache-Control directives to ensure consistency across methods.
Updated tests to verify that the Cache-Control header is correctly set to "max-age=2592000, stale-while-revalidate=86400".

Related Issues
Closes #4878

Please check https://github.com/pocketbase/pocketbase/discussions/4355 and the note in the readme.
The filesystem interface also has slightly changed with the refactoring.
I'll have a look at it after the refactoring release.


Integrate CSGHub with dstack for apps, training, and inference

CSGHub integrates with Kubernetes, Ray, and Knative to deploy apps, train models, and run model inference.
dstack is a lightweight alternative to Kubernetes, Ray, and Knative that could be a much better fit.
Here are its key advantages:

dstack supports leading cloud GPU providers and makes it easy to add native support for new providers.
dstack simplifies the use of on-prem GPU machines compared to Kubernetes.
dstack addresses many challenges related to deploying apps, managing endpoints, and scheduling complex training tasks.

By integrating with dstack, CSGHub users could experience up to a 10x simplification when connecting to cloud providers or on-prem clusters.

If required, our team is happy to assist with the integration.

Thank you for reporting feature requirement, I just looked at it, dstack is a very light framework for LLM deployment. we will start investigation in next a few weeks.

Guys, @peterschmidt85 @ganisback can you guys create one PR for the further integration?


Add Craft asset_path helpers

@mzlock or @tommymarshall created these on another project, and they'd be useful to share with the community. One of you want to grab this?
Not sure where these should go... maybe an extras/craft folder, with the helper file and a README? I plan on doing the same for Rails.

Nice, what does it do?

Production filenames get hashed: filename.js becomes filename-fdha92379fb2cx84.js. A rev-manifest.json file is generated mapping original filenames to the hashed ones. To reference assets from Twig templates, you need to use a helper to resolve the paths in production.

might output something like


Facebook Graph API v2 notice

From FB

Flow (dev) has been making recent API calls to Graph API v2.0, which will reach the end of the 2-year deprecation window on Monday, August 8, 2016. Please migrate all calls to v2.1 or higher in order to avoid potential broken experiences. We recommend using our new Graph API Upgrade Tool to see which of your calls are affected by this change as well as any replacement calls in newer versions. You can also use our changelog to see the full list of changes.

Need to make sure nothing will break.

This was fixed, but it likely needs it again.


feat: Support for datetime2 data type

Defines a date that is combined with a time of day that is based on 24-hour clock. datetime2 can be considered as an extension of the existing datetime type that has a larger date range, a larger default fractional precision, and optional user-specified precision.
https://learn.microsoft.com/en-us/sql/t-sql/data-types/datetime2-transact-sql?view=sql-server-ver16
https://docs.sqlalchemy.org/en/14/dialects/mssql.html#sqlalchemy.dialects.mssql.DATETIME2

The definition of DATETIME2 in JSON Schema format can be achieved with the following
{
	"type": ["string"],
	"pattern": "^[0-9]{4}-[0-1][0-9]-[0-3][0-9]T[0-2][0-3]:[0-5][0-9]:[0-5][0-9]\\.[0-9]{1,7}\\+00:00$",
	"format": "date-time"
}

The pattern: will validate via REGEX that the string conforms to the pendulum ISO format string a tap outputs for a DATETIME2 column.  We use format: to get the target to place the string into a datetime variable so SQLAlchemy can add only the sections is needs for DATETIME2.
I just need to get pattern: added to the SDK SCHEMA data class.


Render charts inline of instance component

always render chart, no clicks
render chart below hero metric and metadata
show top y-axis again

Having the charts right under the instances is a great improvement, nice work @davkal


CIP-0057 | Add 'compiler' field

:round_pushpin: Add 'compiler' field to the blueprint.
This is a purely additive, and optional, change. So it shouldn't pose much problem to add (existing implementation that supports blueprints would keep working). Yet it documents a field that we've started using in Aiken and that has turned to be incredibly useful.

:round_pushpin: Document few more projects in connection with blueprints.

@scarmuega @nielstron @Unisay as recent contributors to the CIP please also post here what you think of the change and/or come to the next meeting if there is anything still debatable.

looks good to me

LGTM

🙏

merging based on unanimous approval from all quarters & taking off agenda 😎


Db/chore/issue templates

Release notes

In this release, we:

Did this and that

Summary

Breaking Changes

Checklist

[ ] I added — tests to prove my changes
[ ] I updated — all the necessary docs
[ ] I reviewed — the entire PR myself, using the GitHub UI
[ ] I described — all breaking changes and the Migration Guide

Superceded by #2738


JMAP support

Given that we all hope WebDAV dies, I'd like to add JMAP to the table as an alternative.
JMAP does several things, email, contacts and calendars. We'd only care about the last two.
My schedules in the near future don't allow me to pick this up, so I'll just wishlist it. I'll get back when/if I have time to work on it.
Please let me know if there's something that makes JMAP unacceptable.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26688989-jmap-support?utm_campaign=plugin&utm_content=tracker%2F574345&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F574345&utm_medium=issues&utm_source=github).

It doesn't use VCard or iCalendar, so a JMAP-storage-type would have to include a translation layer for the datastructures it uses.

Apparently there have been some efforts to make Thunderbird support JMAP: https://www.google-melange.com/gsoc/project/details/google/gsoc2015/sshagarwal/5733935958982656

In principle I'd accept a PR adding a JMAP backend, but I don't want to keep an issue lingering around if nobody is going to work on it.

Nevermind, I'll reopen. I'm unsure on how to handle items tagged with wishlist.


CB-10510: Retry starting the Android emulator if it hangs on boot

This PR relies on https://github.com/apache/cordova-android/pull/258 being merged.
Adds logic to retry starting the Android emulator if it hangs on boot. Also has the side effects of changing medic-kill so that it can be required and so that it does not kill ADB when the killing Android tasks. Killing ADB causes issues with restarting the emulator and I don't think there was any reason to kill it in the first place.
@dblotsky @nikhilkh please review. This was split off of #76

Merged in 90d06a39cbaf9313edae0fa3cc63a63003b65553