UX Research - Report Out for Prioritization Workshop

Title of ticket: UX Research - Report Out for Prioritization Workshop
Description
Summarize issue
Dependencies
Are there any dependencies?
DOD

[ ] List the items that need to be complete for this ticket to be considered done
[ ]
[ ]
[ ]
[ ]

Conducted at Demo on July 18th


Updated the content of line # 655 and 657

Changed the content of two paragraphs in Our Best CARDIOLOGY Services

Approved, but next time change only 1 thing in 1 PR


Expo SDK 50 Build Error

Description
I am getting this error on build:

  1 | #import "KlaviyoReactNativeSdk.h"
> 2 | #import <klaviyo_react_native_sdk-Swift.h>
    |         ^ 'klaviyo_react_native_sdk-Swift.h' file not found
  3 |
  4 | @implementation KlaviyoReactNativeSdk
  5 | RCT_EXPORT_MODULE()

Checklist

[ ] I have determined whether this bug is also reproducible in a vanilla project
[ ] If possible, I've reproduced the issue using the master branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or pull request.

Expected behavior
No response
Actual behavior
No response
Steps to reproduce
No response
The Klaviyo React Native SDK version information
No response
Environment Description
No response

Hi, while we have not tested against expo explicitly, we've have heard from a few users who saw this error and resolved it by cleaning their build directories, deleting derived data, and running a clean pod install. Based on the information you've provided, this is really all I can guess.

Guys this still isn't fixed, It seems to be IOS only. Can you please reopen this and re address it? Ive been trying all the suggestions and my environment is as recceomnded.

The header import issue was patched in Version 0.2.1.


Usage on non-NixOS Platforms?

Hi,
I guess krops can also be used to deploy non-nixos systems (e.g. ubuntu with nix installed) somehow? With writeCommand maybe somehow?
Would be nice if there was an example for that! :smile:

I never used nix without NixOS. Isn't nix just a package manager? So there is no module system, just pkgs. I don't understand what you need to provision in a situation like that? You can't manipulate systemd or anything in /etc using just the nix package manager.
Or am I wrong?

Yes, the idea was to just deploy package sets to a non-nixos host. Like user environments and so on, nothing in the actual system!

can this be done with writeCommand?

Yes this can be done with writeCommand. But you deployment depends heavily on what you want. For example I use home-manager and flakes on my work machine. Another person would use plain nix.


Add CLI

fixes #15
Start on a CLI. I didn't get too fancy with the setup, there's a lot more we can do, first of which would be including a watch mode.

Fixing conflict and adding better handling


English version : review

I'm checking the translation work on #641 to launch a good enough english version as soon as posible. Reporting the work in progress here to avoid polluting this already big PR @EmileRolley

This is a first check before another check by a native speaker.

[x] landing
[x] tutorial
[x] test questions, answers and descriptions

[x] transport
[x] food
[x] house
[x] others

[x] ending pages
[x] action screen
[ ] changer "alert banner" : pour l'anglais, ce n'est plus une version automatique mais une beta
[ ] bad link in "This category does not depend on your answers, read more", question services publics

After a first release :
What I haven't checked :

all action pages and action plus pages

What's not translated yet :

this markdown page http://localhost:8080/pétrole-et-gaz

[ ] #711

The action screens may be optional for a first launch.

bad link in "This category does not depend on your answers, read more", question services publics

It's working on my machine..


ObstacleAvoidance: refactor check on z mission progression

Refactors the mission progession check on the z component. Right now the check excludes takeoff points because the condition of being inside the xy acceptance radius but more than z acceptance away is always true. However this condition happens not only at takeoff. The refactor uses an hysteresis on the z distance to the goal to judge is the mission is progressing.

I put this commit together with #12349 to avoid conficts


institution_id registration of WACCEM-PNNL

institution_id = WACCEM-PNNL
institution = PNNL (Pacific Northwest National Laboratory, Richland, WA 99352, USA)

@beharrop just curious if just PNNL is what you want, I understand this is the  WACCEM project which is hosted at PNNL, but not sure it's in the right order. It also means that if PNNL (non-WACCEM) wants to contribute any other simulations to CMIP6, doesn't give you much leeway.
Take a peek at the other registrations and their format at CMIP6_institution_id.json.
Just a question here, you can register what you like
Following other examples the registration would look like:
"institution_id":"institution"
"PNNL":"Pacific Northwest National Laboratory, Richland, WA 99352, USA",

@durack1 Thanks for the suggestions.  We discussed this, and have decided to keep in line with the other DOE projects in the registrations that include the name of the project instead of just the institution.  We will go ahead and switch the order of PNNL and WACCEM in the institution_id.  Please use
institution_id = PNNL-WACCEM
institution = PNNL (Pacific Northwest National Laboratory, Richland, WA 99352, USA)

@beharrop no problem, in addition to PNNL being expanded, did you want to expand WACCEM in the institution description field?

@durack1 I don't think it is necessary.  E3SM and RUBISCO don't expand their acronyms.

No problem, it’s up to you. The institution_id field is used in directory structure, the long description remains in our CV registration, with no active use within file or directory names

@beharrop all done, please take a peek at CMIP6_institution_id.html and let me know of any further tweaks required

Looks great, thanks @durack1 !


[import/order] Regression upgrading from 2.26.0 to 2.27.1+

I have the following setup and code experiencing a linting issue with 2.27.1 but not 2.26.0 for the import/order rule and what I think is the root issue.
.eslintrc fragment:
    'import/order': [
        2,
        {
            alphabetize: {
                order: 'asc',
                caseInsensitive: true
            },
            'newlines-between': 'always'
        }
    ],

tsconfig.json:
{
    "compilerOptions": {
        "module": "commonjs",
        "esModuleInterop": true,
        "declaration": true,
        "removeComments": true,
        "emitDecoratorMetadata": true,
        "experimentalDecorators": true,
        "allowSyntheticDefaultImports": true,
        "strictBindCallApply": true,
        "strictNullChecks": true,
        "target": "es2022",
        "sourceMap": true,
        "outDir": "./dist",
        "baseUrl": "./",
        "incremental": true,
        "skipLibCheck": true,
        "paths": {
            "@core/*": ["core/src/*"],
            "@schedule/*": ["schedule/src/*"],
        }
    }
}

I have a TS file that has the following structure:
import CoreA from '@core/A';
import CoreB from '@core/B';
import CoreC from '@core/C';

import PackageA from '@schedule/A';
import PackageB from '@schedule/B';
import PackageC from '@schedule/C';
import PackageD from '@schedule/D';
import PackageE from '@schedule/E';
import PackageF from '@schedule/F';
import PackageG from '@schedule/G';

export default () => {};

Interestingly enough this error only occurs once there is 10+ imports in the file, no `import/order' error occurs otherwise.
Digging into the rule's code, it seems the issue comes from a default .sort() that implicitly cast numbers into strings for ordering and will order 10 before 2 (in my case).
This results into this ranking assignment:
{
  '@schedule/A|value': 10,
  '@schedule/B|value': 11,
  '@schedule/C|value': 12,
  '@schedule/D|value': 13,
  '@schedule/E|value': 14,
  '@schedule/F|value': 15,
  '@schedule/G|value': 16,
  '@core/A|value': 9, // smaller than previous item, causing error reporting
  '@core/B|value': 10,
  '@core/C|value': 11
}

Changing it to .sort((lhs, rhs) => lhs - rhs) seems to resolve the issue although ranking has gaps (not sure this could cause issues elsewhere).
This results in this ranking:
{
  '@core/A|value': 2,
  '@core/B|value': 3,
  '@core/C|value': 4,
  '@schedule/A|value': 13,
  '@schedule/B|value': 14,
  '@schedule/C|value': 15,
  '@schedule/D|value': 16,
  '@schedule/E|value': 17,
  '@schedule/F|value': 18,
  '@schedule/G|value': 19
}

Let me know if you know more info/config to repro 😄

Rather than create a new issue, I am also getting a regression but for me it is to do with the ordering of imports with extensions.
My config is:
    'import/order': [
      'warn',
      {
        'newlines-between': 'always',
        alphabetize: {
          order: 'asc',
          caseInsensitive: false,
        },
        pathGroupsExcludedImportTypes: ['builtin'],
        warnOnUnassignedImports: true,
        pathGroups: [
          {
            pattern: '**/*.svg',
            patternOptions: { matchBase: true },
            group: 'type',
            position: 'after',
          },
        ],
      },
    ],

and I am now getting the error:

@shooftie can you share the code itself being warned on, in text form? (all the imports)

There are loads. I will put together a smaller example.

Full config
    'import/order': [
      'warn',
      {
        'newlines-between': 'always',
        alphabetize: {
          order: 'asc',
          caseInsensitive: false,
        },
        groups: ['builtin', 'external', 'internal', 'parent', 'sibling', 'index', 'object', 'type'],
        pathGroupsExcludedImportTypes: ['builtin'],
        warnOnUnassignedImports: true,
        pathGroups: [
          {
            pattern: '**/*.raw.js',
            group: 'type',
            position: 'after',
          },
          {
            pattern: '**/*.svg',
            group: 'type',
            position: 'after',
          },
          {
            pattern: '@app/config/**',
            group: 'internal',
            position: 'after',
          },
          {
            pattern: '@app/constants/cookies',
            group: 'internal',
            position: 'after',
          },
          {
            pattern: '@app/constants/**',
            group: 'internal',
            position: 'after',
          },
          {
            pattern: '@app/services/**',
            group: 'internal',
            position: 'after',
          },
          {
            pattern: '@app/util/**',
            group: 'internal',
            position: 'after',
          },
          {
            pattern: '@app/component/**',
            group: 'internal',
            position: 'after',
          },
          {
            pattern: './constants',
            group: 'sibling',
            position: 'after',
          },
          {
            pattern: './helper',
            group: 'sibling',
            position: 'after',
          },
        ],
      },
    ],

imports
import PropTypes from 'prop-types';
import React from 'react';
import styled from 'styled-components';

import { device } from '@app/config/theme/Styles';

import { COOKIE_FOO } from '@app/constants/cookies';

import { CLASS_BIG_LETTER, MIN_PARAGRAPH_LENGTH } from '@app/constants/article';
import { ID_CONTENT } from '@app/constants/ids';

import { isFeatureEnabled } from '@app/services/featureFlag';

import { addStrAfterPosition } from '@app/util/addStrAfterPosition';
import transformProduct from '@app/util/transformProduct';

import MPU from '@app/component/library/MPU';
import Product from '@app/component/library/Product';

import Newsletter from '../Newsletter';
import Quote from '../Quote';

import Readmore from './Readmore';
import SocialShare from './SocialShare';
import Table from './Table';
import Topics from './Topics';

import { readmoreOnce, removeEmptySnippet } from './helper';

import LogoStamp from<EMAIL_ADDRESS>
2.27.0, error

2.26.0, no error (other than no-unused-vars)

hm, i can't reproduce this in our tests, although the cause you suggest seems highly likely to be accurate.

@ljharb maybe the missing detail is that this came up in a repo using yarn workspaces and the example code would exist in the schedule workspace, effectively causing all @core/* entries to be treated as externals vs. @schedule/* internals (core workspace) for the code provided above.

Hello
Same issue over here after the release of v 2.27

same problem here, all of my imports where changed after the update to 2.27.4, I downgrade to 2.26.0 and all the imports are now in the correct order:
here is my import order
		"import/order": [ 			"error", 			{ 				"groups": [ 					"builtin", // Built-in imports (come from NodeJS native) go first 					"external", // <- External imports 					"internal", // <- Absolute imports 					["sibling", "parent"], // <- Relative imports, the sibling and parent types they can be mingled together 					"index", // <- index imports 					"unknown" // <- unknown 				], 				"newlines-between": "always", 				"alphabetize": { 					/* sort in ascending order. Options: ["ignore", "asc", "desc"] */ 					"order": "asc", 					/* ignore case. Options: [true, false] */ 					"caseInsensitive": true 				} 			} 		],

Likely fixed by https://github.com/import-js/eslint-plugin-import/pull/2674

Please try v2.27.5, and we can reopen if it's not fixed.

@ljharb Looks all good on my end. Thanks!


Voice over reads Radio Group Legend Twice

Bug Report
What is the issue?
While doing the exploratory testing for MAC VO on the NHS App, it was observed that the heading (NHS Radio Group Legend) was being read twice. This occurs across all uses of the NHS Radio Legend.
Note: It is being read twice while navigating like a real user from prev page to current page. If you are on the page and try to read element after element you may not see the issue.
The radio legend should only be read once.
What steps are required to reproduce the issue?
We see this issue on the GP messaging page of the NHS App. I have attached a video of this here but it can be seen here when you open the radio group in a tab on Firefox. (We have seen this on other browsers but this specific on only does it on firefox and is the best example. This is a video of that page on firefox.
What was the environment where this issue occurred?

Device: Macbook
Operating System: MacOS 14.6.1
Browser: Firefox
Browser version: 130.0.1
NHS.UK frontend package version: N/A
Node version: N/A
npm version: N/A

Is there anything else you think would be useful in recreating the issue?
I have added screen recordings and references in the above.

Do you observe the same if you try one of the examples in the design system?
@tjhayward is your code available anywhere I could view it? The spacing on the radios in the video looks different, so I'd like to check the markup. Is it deployed anywhere I could try?


Use toolkit in JMeter

Are there some plugins available to use toolkit in JMeter to sign AuthNRequests?

I think that question should be made to the JMeter community.
I'm open to solve any doubt that Jmeter's developer can have when using the java-saml toolkit.


Softmax use mask

Do unmasked load/stores in loop iteration except the last one. This results in performance improvement on ROCm

Wrong branch


Add unit tests to verify release of framework objects

I had to make significant revisions to resolve memory consumption issues caused by unmanaged retention of framework object references. To ensure that I don't introduce regressions in future revisions, I need to add unit tests to verify that all collected references are released at key points of execution for all supported scenarios:

Standard happy-path execution
Pass on automatic retry
Fail on automatic retry
Method-specific timeout
Rule-based timeout
Parameterized runner
JUnitParams runner
Theories runner
PowerMock runner

Checks for reference retention must be performed at relevant points in the flow of execution. This can probably be accomplished with a properly crafted watcher/listener class.

SCOPE: Atomic Test / Test Class Instance

ArtifactCollector.WATCHER_MAP
CreateTest.HASHCODE_TO_TARGET
CreateTest.TARGET_TO_METHOD
EachTestNotifierInit.DESCRIPTION_TO_ATOMICTEST
EachTestNotifierInit.TARGET_TO_DESCRIPTION
EachTestNotifierInit.DESCRIPTION_TO_TARGET
EachTestNotifierInit.HASHCODE_TO_DESCRIPTION
MethodBlock.RUNNER_TO_STATEMENT (Theories runner)
RunReflectiveCall.DESCRIPTION_TO_CALLABLE

SCOPE: Class Runner

Run.START_NOTIFIED
Run.CHILD_TO_PARENT
Run.RUNNER_TO_NOTIFIER
RunChild.DID_NOTIFY

SCOPE: Test Run

Run.RUNNER_STACK

Unmanaged

Run.NOTIFIERS


login to dev ECR via OIDC

Dev account counterpart of https://github.com/neondatabase/neon/pull/9146

https://github.com/neondatabase/neon/pull/9178 is the successor.


give user a total quota, let themselves decide how to use quota

when creating a container, give them a choice and a default config, they can choose to create one strong container, or several weak containers. strong container will cost more resources (we will use bean to measure the cost), while weak container cost less.

Done


Changed Speed Units From km/h To m/s

Changed According to the Android documentation for the androix.health.services.client.data.DataType class.

Thanks

Released in 0.6.1


ChoiceField doesn't load previously saved value

Hey Jpic it is me again.
Thanks for the help yesterday and I got it working just fine! However there is a new bit of problem so I guess it would be better to start a new issue.
The field:
manufacturer = forms.ModelChoiceField(
        queryset=Manufacturer.objects.all(),
        widget=autocomplete.ModelSelect2(url='calbase:manufacturer')
    )

After I successfully saved a value in a ChoiceField like this

If I tried to update this model and use an update form, the field appears to be blank

while other non-autocomplete field load saved value fine.
I did a bit of research online and found somebody's solution which is something like this:
def __init__(self, *args, **kwargs):
    	super(EquipmentForm, self).__init__(*args, **kwargs)
    	if self.initial.get('manufacturer', None):
        	manufacturerPK = self.initial['manufacturer']
        	manufacturer = Manufacturer.objects.get(pk=manufacturerPK)
        	self.fields['manufacturer'].widget=autocomplete_light.ChoiceWidget('ManufacturerAutocomplete', extra_context={'values':[manufacturerPK], 'choices':[manufacturer]})

But this doesn't help. I guess it might be for a older version of DAL. Anyway, could you please help me figure how I could load the field with saved value to prevent user the trouble of inputting this every time he or she would like to make any changes?

Sorry, but nothing comes to my mind.
Is there any demonstration of this in your project source code or in a fork of this repo inside test_project ?

@jpic
Sorry for replying so late. the current project is like this: https://github.com/lhsdaniel/EquipmentCalibration.
Also as you can see here:

those field with auto complete rendered to be None, while the template of this is simply:
                  {{ equipments.serial_number }}
                  {{ equipments.manufacturer }}
                  {{ equipments.model }}
                  {{ equipments.description }}
                  {{ equipments.timestamp }}
                  {{ equipments.department }}

(of which manufacturer, description and department are autocomplete fields)
So I have a feeling that those field are not really registered..

Oh I think I get closer to the actual problem. I was using
instance = form.save(commit=False)
        instance.save()

before and apparently this doesn't save many-to-many fields. I need to either use something like form.save_m2m(), or just instance = form.save().
However, with that being done, I got his error now:

TypeError at /calbase/equipment/4/update/
'Manufacturer' object is not iterable

and the following traceback:
Environment:

Request Method: POST
Request URL: http://<IP_ADDRESS>:8000/calbase/equipment/4/update/

Django Version: 1.10.1
Python Version: 3.5.2
Installed Applications:
['calbase.apps.CalbaseConfig',
 'dal',
 'dal_select2',
 'django.contrib.auth',
 'django.contrib.contenttypes',
 'django.contrib.sessions',
 'django.contrib.messages',
 'django.contrib.staticfiles',
 'haystack',
 'whoosh',
 'carton',
 'adminplus',
 'django.contrib.admin.apps.SimpleAdminConfig',
 'crispy_forms',
 'auditlog',
 'popup_forms']
Installed Middleware:
['django.middleware.security.SecurityMiddleware',
 'django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'auditlog.middleware.AuditlogMiddleware']

Traceback:

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\core\handlers\exception.py" in inner
  39.             response = get_response(request)

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\core\handlers\base.py" in _get_response
  187.                 response = self.process_exception_by_middleware(e, request)

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\core\handlers\base.py" in _get_response
  185.                 response = wrapped_callback(request, *callback_args, **callback_kwargs)

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\contrib\auth\decorators.py" in _wrapped_view
  23.                 return view_func(request, *args, **kwargs)

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\contrib\auth\decorators.py" in _wrapped_view
  23.                 return view_func(request, *args, **kwargs)

File "C:\Users\hansong.li\Documents\GitHub\EquipmentCalibration\calbase\views.py" in post_update
  120.         instance = form.save()

File "C:\Users\hansong.li\Documents\GitHub\EquipmentCalibration\calbase\models.py" in save
  223.         instance = super(EquipmentForm, self).save(commit)

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\forms\models.py" in save
  454.             self._save_m2m()

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\forms\models.py" in _save_m2m
  436.                 f.save_form_data(self.instance, cleaned_data[f.name])

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\db\models\fields\related.py" in save_form_data
  1573.         getattr(instance, self.attname).set(data)

File "C:\Users\hansong.li\AppData\Local\Programs\Python\Python35-32\lib\site-packages\django\db\models\fields\related_descriptors.py" in set
  928.             objs = tuple(objs)

Exception Type: TypeError at /calbase/equipment/4/update/
Exception Value: 'Manufacturer' object is not iterable

I don't have access to your repo !
It seems like it's trying to iterate over a list/qs of Manufacturer, but it crashes because it's given a single Manufacturer instance.
Perhaps try ModelMultipleSelect2 ? In this case you should be using a ModelMultipleChoiceField too.
The form's save() function should call save_m2m if commit=True, what django version are you on ?
https://github.com/django/django/blob/master/django/forms/models.py#L454

@jpic
Hey Jpic! I was using django 1.10.
And sorry about the repo being private.
https://github.com/lhsdaniel/test
this should work.
And also, I tried to make manufacturer field which was ModelChoiceField to ModelMultipleChoiceField, and that did solve the problem. Now it saves correctly and I could display by using a for loop like this, thanks:
{% for manuf in equipment.manufacturer.all %}{{ manuf }}{% endfor %}
However, I still would like manufacturer and description to be Foreign key field, and it seems that if I make them foreign key, the sweet "Creation of new choices in the autocomplete form" function is not working. Does this only work for many to many and one to one?

Could you reproduce this with a minimalist project please ?

@lhsdaniel , I think you should add to_field_name when you want to create an object of ModelChoiceField class


Create a Share Target

I find pwa's as I browse the web, I'd love to share it straight into gulliver so it can get indexed or return the data.

This is a great idea, we should definitely implement it.

I think we can do this as part of rebuilding the PWA submission process. I've added an endpoint to allow us to do it asynchronously on #501 , and this could also be used to handle the share target.

@fstanis, FYI


AnnDataSchema validate_anndata is over-zealous for obs categoricals

Here is where the problem occurs:
https://github.com/cellarium-ai/cellarium-ml/blob/220ba90b47378c99d4c08b9d91c5c31b796cb3ca/cellarium/ml/data/schema.py#L62
Here's a simplified version of my problematic example (which occurred during scvi testing with multiple datasets and a "batch" variable as a categorical column in adata.obs):
import pandas as pd

df = pd.DataFrame(data={'col1': ['a', 'b', 'c'], 'col2': ['b', 'a', 'c']})
df['col1'] = df['col1'].astype('category')
df['col2'] = df['col2'].astype('category')

df[['col1']].dtypes.equals(df[['col2']].dtypes)

This, shockingly, returns False.
I propose to change line 62

I think this is happening because the .dtypes returns a Series where index is the original DataFrame's columns. So in one case the index is col1 and in the second it is col2. Thus, for the same columns it is checking that their dtype match. If you want to validate only a subset of columns you can specify obs_columns_to_validate argument in DistributedAnnDataCollection.

I should change my example.  I think this also happens when the names match, but let me check.

Have you checked that the categories for the batch column match between anndata files? It's possible that when the extracts where created they were not synchronized.

Hmmm it looks like you're right.  Maybe it is not the fault of
ref_value.dtypes.equals(value.dtypes)

Yeah I guess I was wrong about this
import pandas as pd

df1 = pd.DataFrame(data={'col1': ['a', 'b', 'c']})
df2 = pd.DataFrame(data={'col1': ['a', 'b', 'c', 'b']})
df1['col1'] = df1.astype('category')
df2['col1'] = df2.astype('category')
df1.dtypes.equals(df2.dtypes)

True

I'll close the linked PR


[ACS][Chat] Added Few Information About Inline Images

@jpeng-ms, the warning in this PR has to be resolved before it can be merged. Thanks.

#hold-off

Hi @jpeng-ms,
This pull request was opened in the public repo. Microsoft FTEs should work in the private repo, per the Microsoft Docs contributor guide.
We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


modules/visuals: update indent-blankline to v3

Updates indent-blankline to v3 and removes the workaround requiring vim.wo.colorcolumn = "99999" - reported to be fixed upstream in neovim.
I've renamed some of the module options, and I'd really like to use lib.mkRemovedOptionModule and lib.mkRenamedOptionModule, however, both of those cause the build system to assert the files they are defined in (wherever they are) are not actually modules.

Blocked by #178

I think scope.enabled might be a better name for scope.showCurrContext and helps users that are translating examples from README.md. For same reasons, description might be better defined as Current scope of variables and functions with treesitter. Other than that it looks good to me 👍


Add pagination

Added pagination to the next branch, in the form of methods on the Page and CursorPage structs.
This meant changing a bit of how the internal Endpoint trait behaves and now it has a method that returns the URL of a certain endpoint. It's currently only implemented that way for a couple of endpoints, becuase now it's required for CursorPage to function.
However, over time I will transition the endpoint URLs to this endpoint_url() trait method for consistency's sake.

I forgot to merge this I think oops


a few questions about felogin

Hello,
i would like to redirect to a page outside from the typo3. is this possible? I have an external PHP application. This application may only be called after a successful login. Is it possible to bind the login data from felogin to a session that I can use in my PHP application? I need the login information in my PHP application. I would like to use the felogin logout mechanism in my external PHP application. Is that possible?
Regards
MS-Tech

This place is for reporting issues about the rendering / theme of the official TYPO3 docs.
For help and support, please see: https://typo3.org/help/


[FOR DISCUSSION] Add 'sandboxed' option. If set to false, iframe is not sandboxed.

This is useful for cases where you want oasis services but don't care about the oasis security model (i.e. you control code inside and outside the iframe).
Thoughts?

Supplanted by #85


v10 - Ssh tunnel

From #94

We should avoid setting connection parameter while running component. The whole settings should be immutable during run.
Better check for creating Ssh Tunnel

1st part was solved right in #94 (https://github.com/keboola/db-extractor-common/pull/94/commits/33c9d8de31a04eac09cbddc0f92f66764a6f5104, https://github.com/keboola/db-extractor-common/pull/94/commits/1d965f213b0d09f49ac0570e5ec256bbcdd8bdba)


Does asp.net core authentication support reference token?

Hi,
For cookies authentication, it seems that it doesn't support Client(WPF and  so on)/Server model well,
and for jwt authentication, it also has some problems, hard to refresh the token and can't clear the token(for logout the user when the account is logined in two places)
So I want to seek a sollution to generate referece token(can store the token in Redis, Database, session), could you let me know
if asp.net core authenticaion suppor it and could you give me any guide?
Thanks

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

ID: 393b3379-6061-50a4-0975-b3d1d82b660b
Version Independent ID: 48e4e204-ea22-c225-0cbc-0d811bdb90ed
Content: Overview of ASP.NET Core Authentication
Content Source: aspnetcore/security/authentication/index.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @mjrousos
Microsoft Alias: riande

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

Potential bugs in the ASP.NET Core framework and product feedback can be reported at dotnet/aspnetcore issues (include full repro steps and cross-link to a repro project in GH when appropriate). For Visual Studio, use the Report a Problem gesture or Suggest a Feature gesture within VS, which open internal issues for the VS team. For Visual Studio Code, you can open an issue at microsoft/vscode issues.


Do not show "other login options" when SP requires MFA

This issue is imported from pivotal
When a service requests MFA from eduID, the login page shows the option "use other login method". And after choosing "other login method" the warning "are you sure" is shown while this is not correct.

(Peter Clijsters - Jun 16, 2023)


Update pricing-guidance.md

Customer riased case to query if there was a difference for SQL Server 2019. As this is not the csae, I have just updated the line to include SQL Server 2019.

@PhilB-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off thanks @PhilB-MSFT !


Sharing weights across layers in keras 3 [feature request]

It seems that sharing weights is not possible anymore afterwards in keras 3. We should instead share layers as explained here.
But I have a usecase where I need to share a weight

after init/build
without sharing a layer

In my usecase, I transform a model by splitting activations out of each layer, that means a Dense(3, activation="relu") is transformed in  a Dense(3) + Activation layer. But I need

to let the original model unchanged (so i cannot just remove the activation from the original layer)
to share the original weights so that further training of the original model will impact also the splitted layers and thus the converted model
preferably the resulting layers to still be simple keras layers (like Dense and not a custom SplittedDense new class)

For now I have a solution but that use private attribute since by design this is currently not possible in keras 3.
Here is an example that works for sharing kernel (I actually will use something more generic to share any weight, but this is simpler to look at):
from keras.layers import Input, Dense

def share_kernel_and_build(layer1, layer2):
    # Check the layer1 is built and the layer2 is not built
    if not layer1.built:
        raise ValueError("The first layer must already be built for sharing its kernel.")
    if layer2.built:
        raise ValueError("The second layer must not be built to get the kernel of another layer")
    # Check that input exists really (ie that the layer has already been called on a symbolic KerasTensor
    input = layer1.input  # will raise a ValueError if not existing

    # store the kernel as a layer2 variable before build (ie before the lock of layer2's weights)
    layer2.kernel = layer1.kernel
    # build the layer
    layer2(input)
    # overwrite the newly generated kernel
    kernel_to_drop = layer2.kernel
    layer2.kernel = layer1.kernel
    # untrack the not used anymore kernel  (oops: using a private attribute!)
    layer2._tracker.untrack(kernel_to_drop)

layer1 = Dense(3)
input = Input((1,))
output = layer1(input)
layer2 = Dense(3)

share_kernel_and_build(layer1, layer2)

assert layer2.kernel is layer1.kernel
assert len(layer2.weights) == 2

Notes:

doing layer2.kernel = layer1.kernel after build will raise an error because of the lock.
doing afterwards allows to modify it again after the build because the variable being already tracked, this does not go into add_to_store
not untracking the unused kernel will result in an additional weight tracked by the layer

A simpler solution to your problem would be:

Instantiate the new Dense layer, e.g. dense = Dense.from_config(...) layer. (It doesn't have weights at that time)
Set dense.kernel = old_layer.kernel, dense.bias = old_layer.bias, dense.built = True
Just use the layer -- no new weights will be created since the layer is already built

Nice!
But are we sure that the build() method does only create the weights? Perhaps i will miss something else by skipping build() ?
I would like a solution that works with any layer. By setting self.built = True, I skip the build() and thus do not overwrite the weights, but is there anything else that could be important not to bypass so that the call() works ?

A simpler solution to your problem would be:

Instantiate the new Dense layer, e.g. dense = Dense.from_config(...). (It doesn't have weights at that time)
Set dense.kernel = old_layer.kernel, dense.bias = old_layer.bias, dense.built = True
Just use the layer -- no new weights will be created since the layer is already built

It does not work anymore from keras 3.0.3 since Dense.kernel is now a property not settable...

We'll add a setter for the kernel.

The setter thing turned out to be problematic. What I would recommend is just direct setting but use ._kernel instead of .kernel.


[MM-26812] Add bot-mention and bot leave/join subscriptions

Summary
This PR adds subscriptions for:

bot_mentioned
bot_joined_channel
bot_left_channel
bot_joined_team
bot_left_team

For the bot_mention subscription, bot channel membership is checked before sending the notification.
Also added tests for the subscriptions in general
Ticket Link
Fixes https://mattermost.atlassian.net/browse/MM-36812
Fixes https://mattermost.atlassian.net/browse/MM-36813

Codecov Report

Merging #220 (de833de) into master (cd10c76) will increase coverage by 1.44%.
The diff coverage is 52.33%.

:exclamation: Current head de833de differs from pull request most recent head f7371e6. Consider uploading reports for the commit f7371e6 to get more accurate results

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
+ Coverage   22.62%   24.06%   +1.44%
==========================================
  Files          51       52       +1
  Lines        2953     3009      +56
==========================================
+ Hits          668      724      +56
+ Misses       2228     2215      -13
- Partials       57       70      +13

Impacted Files
Coverage Δ

apps/subscription.go
0.00% <ø> (ø)

server/proxy/proxy.go
37.69% <ø> (+10.62%)
:arrow_up:

server/proxy/service.go
0.00% <ø> (ø)

server/proxy/notify.go
52.33% <52.33%> (ø)

Continue to review full report at Codecov.

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

I've added a GET subscriptions route to be able to examine the saved subscriptions. Need to follow up with https://mattermost.atlassian.net/browse/MM-37447 Apps subscriptions should only be createable using the App's bot token

I removed myself in favor of @hanzei

@DHaussermann Please see testing notes below and leave feedback. Thank you 🙂
Scope
Bot mention notification is sent to app when:

the app is subscribed to the bot_mention subscription
the bot is mentioned in a channel the bot is a member of

Testing notes

When app has subscribed to the bot mention event:

app will receive call when:

bot must be a member of channel
bot is mentioned in post
app only receives one call when the bot is mentioned more than once
when the bot and another app's bot is mentioned

app will not receive call when:

the bot is not mentioned in post
when another bot is mentioned, but this bot is not mentioned

When app has not subscribed to the bot mention event:

app should never receive a call in any scenarios above

Follow-up concerns

Authentication for subscriptions API is currently just that the token being used must be from a sysadmin. This will change once we have OAuth scopes for the bots.
Bot must be a member of the channel to receive the bot mention. This may not be obvious to users.

Screenshots

/test subscriptions bot_mention subscribe and /test subscriptions bot_mention unsubscribe commands added to the test app here: https://github.com/larkox/mattermost-app-test/pull/7

@DHaussermann I've added some commands to test the join/leave functionality here https://github.com/larkox/mattermost-app-test/pull/8

Tested and passed

All subscription events can be delivered
All subscription events can be successfully enabled and disabled
No issues found

LGTM!

@mickmister Is there a docs PR for this change?

@mickmister Is there a docs PR for this change?

@hanzei I'm hesitant to make docs for it because the way the App subscribes may change, but we don't have any docs on subscriptions so I'll make a docs page. Existing ticket for this here https://mattermost.atlassian.net/browse/MM-36970

@mickmister Got it. Just wanted to check that we have a ticket.


docs: updated docs to add quotes to the URL

Description
The following errors were being experienced by some grantees
> [1] 60342zsh: no matches found: https://mainnet.fuel.network/v1/graphql?username=blah
[1]  + exit 1     forc deploy --node-url https://mainnet.fuel.network/v1/graphql?username=blah

Adding quotes resolves this issue, so I have added documentation for this.

Looks like this is outdated now that you can just use --mainnet


cannot display on firefox

https://codepen.io/gionkunz/pen/zHBtn
It can display chart on chrome, safari but cannot on firefox? What Should I do?

Please help me. Thank you very much

This is an extremely old version of Chartist. Please refer to the documentation site for examples: http://gionkunz.github.io/chartist-js/
There's a more recent CSS animation and 3d transform example here: http://jsbin.com/wupuhu/edit?css,js,output


OAK-9310 Filter restriction only performs for the same selector.

This commit solve the problem that I described in OAK-9310. The problem depends on the query plan of execution and in some situations it makes wrong results.

Hi community,
I want to know why no one checks my PR? Is there any guidance that I should create PR based on it for approval?
@anchela @rishabhdaim @joerghoh

@mohammadMahdavi can you please add an Apache license header to your test class FullTextSearchImplTest? It currently fails the build. It would also be good if the PR does not contain whitespace changes.
@thomasmueller, WDYT about the test and proposed change?

@mreutegg I did the changes.

@mreutegg  I apologize again for posting a message in an unrelated place. Please check OAK-9842 whenever you have a chance.


NIFI-3362 update FlowConfiguration.xsd to allow all current time period units

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

reviewing

Was able to verify updated config.  Thanks for the tests.  Will get this merged in.

Hey @mosermw.  I had noticed you listed affected versions for both master and 0.x.  I was able to apply this onto master, but seems as though it is not a clean cherry-pick onto 0.x.  Did you intend for those changes there as well?  If so, happy to review a separate PR for that branch.


Document dotnet support

😆 true. I've created the PR on my phone and trust me, editing readmes on such a small screen isn't easy. I know this is a bad excuse for the wrong content. Will fix that tomorrow morning.

Oh, that's no problem! Let me see if I can push the changes to the branch. 😄

Nice collaboration 😊

Cheers!
P.S. Github recently made this possible across forks as well, as long as the forker agrees: https://github.com/blog/2247-improving-collaboration-with-forks


[APIM-3.2.0]-Publisher Portal Access Issues with the assigned user roles

Description:
When user is assigned with Internal/analytics, Internal/subscriber and Internal/everyone role, the publisher portal get loaded partially and freeze on the same screen.
Steps to reproduce:
Create a user from the management console of APIM
Assign roles Internal/analytics, Internal/subscriber and Internal/everyone role to the created user.
Try to access the publisher portal
Affected Product Version:
wso2am-3.2.0

Public fix is merged, hence we can close this issue.


Multiple Script Errors when copying from a storage account

Missing closing } starting at line 559
Encryption piece errors out on parameters.  It appears the latest version of New-AzureRMStorageAccount doesn't even take an encryption parameter.  Commenting out this section has the script function until it gets to the final VM creation.
New-AzureRmVM : Required parameter 'networkProfile' is missing (null).
ErrorCode: InvalidParameter
ErrorMessage: Required parameter 'networkProfile' is missing (null).
StatusCode: 400
Add-AzureRmVMDataDisk : Cannot validate argument on parameter 'VhdUri'. The argument is null or empty. Provide an argument that is
not null or empty, and then try the command again.
Copy-AzureRMresourceGroupv2.ps1:1581 char:21

... $newNICs += Get-AzureRmNetworkInterface -Name $NICName -ResourceGroup ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : CloseError: (:) [Get-AzureRmNetworkInterface], NetworkCloudException
FullyQualifiedErrorId : Microsoft.Azure.Commands.Network.GetAzureNetworkInterfaceCommand

For reference I am copying a vm attached to a storage account between subscriptions.  It is a single linux vm with 1 extra data disk.  I have run the Update-Module AzureRM command and am using the  latest version of PowerShell.  Any help would be appreciated.

Hello,
With respect to point 1, I believe that the corresponding missing closing key } must go on line 1017.
Regarding the other points, I can not give much help.
I attach a file with that small correction.
Regards.
Copy-AzureRMresourceGroup.zip

@atindula  Thank for finding this.   @soyanede  You are correct and that fix will be added to the others.   Seems we have problems with the newer modules as it pertains to managed disks.  It appears the VHD property is no longer returned, and I was using that. Fixes are in the works.

@atindula.  The latest check in should fix the reported issues.  It wasn't the vhd property after all. I had handled that properly in past updates, but rather Set-AzureRmVMOSDisk now requires -StorageAccountType when using managed disk

Successfully tested the Copy and Clone scripts using AzureRM 6.7


Getting Started For Contributors

Motivation
Our documentation contains lots of useful information about contributing but since we use couple different repositories, we gathered required information in one place, till we get them ported to AeroGear website over time.
Introduction
AeroGear is set of the mobile focused services that help developers to write their mobile and backend software. For more general information about AeroGear please check our website:
http://aerogear.org
Voyager (this repository) is one of the services that are currently under active development. Voyager is a set of GraphQL extensions that help mobile developers to adapt GraphQL technology.
Early preview of documentation is available here:
https://deploy-preview-613--aerogear-docs-preview.netlify.com/aerogear/latest/data-sync/
Please mind that this document is still incomplete and individual repositories should have much more information.
How I can start with  GraphQL/Voyager
GraphQL is a data query standard that is gaining lots of attention recently over classical REST API approach.
There are numerous tutorials related to GraphQL and Apollo GraphQL.
Best way to start is to learn the basics of GraphQL
https://graphql.org/graphql-js/
Next step is to get yourself familiar with Apollo GraphQL technology.
I personally found robinwieruch.com tutorials one of the best and most up to date resources:
For the client:
https://www.robinwieruch.de/graphql-apollo-client-tutorial/
For the server:
https://www.robinwieruch.de/graphql-apollo-server-tutorial/
Voyager Getting Started
Voyager is currently maintained in 3 different repositories:
Our Example app that contains server and client integrations for voyager and other AeroGear extensions:
https://github.com/aerogear/ionic-showcase
Server:
https://github.com/aerogear/voyager-server/
Client:
https://github.com/aerogear/aerogear-js-sdk/tree/master/packages/sync
Please check individual README.md files and CONTRIBUTING.md files for more information.
Working on the tasks
To start working on one of the ideas please leave a a comment on one issue:
https://github.com/aerogear/voyager-server/issues?q=is%3Aissue+is%3Aopen+label%3Aideas
Mentors will reach out to you and explain the details of the tasks and possible steps.
Ask us anything
If you found any information unclear feel free to ask question below.
ping @aerogear/voyager team.

Hello, I am Pratik currently in the final year masters program. I am skilled in backend development using Node and GraphQL. I am excited about GSoC 2020 and would like to know if there are any projects/issues which involve GraphQL and API development. I want to start contributing and would love to hear back from you.

Hello, I am Pratik currently in the final year masters program. I am skilled in backend development using Node and GraphQL. I am excited about GSoC 2020 and would like to know if there are any projects/issues which involve GraphQL and API development. I want to start contributing and would love to hear back from you. I can also be reached at<EMAIL_ADDRESS>(preferable).

Hi
We help with mentoring and can help to with finding the issue that will be the best for you.
Please join our community chat that where we have GSOC channel to speak about the issues:
https://discord.gg/mJ7j84m

Closing as outdated


38 new2 organize level 8+ sub topics

Issue #38

comments from meeting with @efucile

keep space-based-obs and surface-based-obs
don't include 'fields'
this approach allows for meaningless THs to be build


add files field into package.json

resolve #537

Does this mean that README.md and example don't come with npm install? Because that would make me sad 😢

@emilbayes
README is always included!
https://docs.npmjs.com/files/package.json#files
I'm just curious that how come do you want example via npm install?

This looks good to me.

v6.0.0-4


Suggestion: Round Segments Toward 50% Point

Thanks again for adding the segments feature. If the damage dealt initially is kinda small, then the 'ceil' calculation found here means that small amount of damage doesn't really show up. My cleric had to keep asking if folks were damaged so his bells could jingle all the way.
Might work better to round toward the 50% mark? Doing a ceiling below and a floor above the mid-point.
Here's an untested version of what I mean:
export const calculatePercentage = async (data: HpTrackerMetadata) => {
    const metadata = (await OBR.scene.getMetadata()) as Metadata;
    const sceneData = metadata[sceneMetadata] as SceneMetadata;
    const segments = sceneData.hpBarSegments ?? 0;

    const percentage = data.maxHp === 0 || data.hp === 0 || data.hp < 0 ? 0 : data.hp / data.maxHp;

    if (segments > 0 && percentage > 0.01 && percentage < 0.99) {
        const filledSegmentsExact = segments * percentage;
        const filledSegmentsRounded = percentage > 0.5 ? Math.floor(filledSegmentsExact) : Math.ceil(filledSegmentsExact);
        return filledSegmentsRounded / segments;
    }
    return percentage;
}

If you don't like that idea, that's fine. I can always run a local copy with these tweaks, once I learn a bit more on owlbear plugins.

Hi, thank you for your suggestion.
Before I make a decision I want to explain how i designed the features to work.

HP Text: Is a detailed information about current and max HP with two use cases:

GM uses it to get detailed information about creature HP without having to consult a table
If the GM allows access to this information it can be shared with the players typically for Player Character Tokens

HP Bar: Is a HP approximation to give Players an idea of the current state of a creature to prevent the question "how is XY looking right now"

With this in mind the easiest solution for your usecase would be adding the HP Text to the map and enableing the Player Visibility. This would also allow them to change their own HP either via the Action Window or the popup. This way the Cleric would see the exact HP and wouldn't need to rely upon a intentionally vague bar.
But to take a look at the HP Bar a little closer, I want to explain my concept there a little better:
This is the bar:
 _ _ _ _ _ _ _ _
|_ _ _ _ _ _ _ _|

When the segments settings is set to 0 it's esentially dvided into as many segments as you have max HP
When you have it set to 1 segment, there are only 2 different states, full HP, 0 HP.
When you have it set to 2 segments, there are exactly two segments in the bar, looking like this:
 _ _ _ _ _ _ _ _
|_ _ _ _|_ _ _ _|

If you have it set to 4 segments, there are exactly 4 segments in the bar, looking like this:
 _ _ _ _ _ _ _ _
|_ _|_ _|_ _|_ _|

Let's say you have set segments to 8 and have a maxHP of 24, then each segment represents 3 HP, for a creature with 240 maxHP a segment would represent 30 HP. So for the 24HP creature losing less than 3 HP would mean the HP Bar is still full, for the 240 HP creature the bar would still be full after losing 29 HP. But is this a case where the cleric would already heal someone? Probably not, so I kind of feel this works as intended, because if the HP Bar is full, the cleric should not consider healing.
Even with 4 Segments, would the cleric really heal someone with more then 3/4 of health?
So then let's consider you approach: The first segment of the HP bar always represents only 1 HP. This is kind of fine for the 24 HP creature because there isn't a big difference between 3HP and 1 HP. But makes a big difference for the 240 HP creature. With 8 Segments, hitting it for just 1 Damage results in a visible loss of 1/8 of it's health basically indicating it has 8 HP and giving the Players a wrong sense of strength. Even worse when you're playing with 4 segments.
So I hope you understand that for now I won't change it, but I'll add it to the feature wishlist and might add a setting for it at some point.

Sorry, I think you misunderstood my reference. Clerics get "Toll the Dead," which does more damage if the target is missing any health at all.
Currently, if an enemy is missing 1hp, the bar shows full HP still. That's why I requested the rounding, for enemies, not healing PCs.
(Also, also, I'm the guy that initially requested the chunks feature, just as a reminder.)

There's a few other abilities that require knowledge of even minor damage. No worries, though, I'll just fork it.

Please when forking change the ID here https://github.com/kamejosh/owlbear-hp-tracker/blob/master/src/helper/variables.ts

Please when forking change the ID here https://github.com/kamejosh/owlbear-hp-tracker/blob/master/src/helper/variables.ts

I doubt I'll ever publish my fork anywhere, but if I do, Ill be sure to change that.


Configure Airpod device does nothing

When I try to setup Airpod connector with airp config nothing happens. The WF just shows "Please select your AirPods...".
I tried to debug get_devices.py: The devicesRaw filled from os.popen is empty.
I tried to exec the popen command from shell /bin/bash -c /usr/local/bin/BluetoothConnector and receive the following output:
Error:
MAC Address missing. Get the MAC address from the list below (if your device is missing, pair it with your computer first):
7c-04-d0-8b-f4-05 - AirPods ...

Or do I miss something ?

Found the issue: BluetoothConnector without parameter writes the AirPod List into stderror instead of stdout. The script does not cover stderror.

#3 should do it

Hey, please try to update to the latest version of the workflow by following the install instructions in the readme. In case this doesn't solve the problem for you, feel free to reopen issue.

I'm facing the exact same issue, even on the latest version of the workflow. Any idea why that could be happening?


chore: remove accidentally commited tests

Remove accidentally committed tests
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2984 into master will increase coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2984      +/-   ##
==========================================
+ Coverage   61.42%   61.72%   +0.29%
==========================================
  Files         223      223
  Lines       11413    11359      -54
  Branches     2359     2193     -166
==========================================
  Hits         7011     7011
+ Misses       4086     4040      -46
+ Partials      316      308       -8

Impacted Files
Coverage Δ

packages/amplify-cli/src/context-extensions.ts
25.69% <0%> (ø)
:arrow_up:

...amplify-appsync-simulator/src/server/operations.ts
0% <0%> (ø)
:arrow_up:

...lify-graphql-types-generator/src/compiler/index.ts
90.32% <0%> (ø)
:arrow_up:

...y-graphql-types-generator/src/compiler/legacyIR.ts
96.72% <0%> (ø)
:arrow_up:

...ges/amplify-graphql-types-generator/src/loading.ts
74.5% <0%> (ø)
:arrow_up:

...ages/amplify-appsync-simulator/src/server/index.ts

...c-simulator/src/schema/directives/aws-subscribe.ts

...s/amplify-appsync-simulator/src/type-definition.ts

...ync-simulator/src/mqtt-server/abstract-listener.ts

...plify-appsync-simulator/src/server/subscription.ts

... and 7 more

Continue to review full report at Codecov.

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


[Python][Packaging] Strip unnecessary symbols from libarrow.so to reduce wheel package size

Describe the enhancement requested
There has been some effort in order to reduce pyarrow size and there are some issues opened in order to split pyarrow wheels in order to have pyarrow-core, pyarrow-all, etcetera.
There seems to be the possibility to strip unnecessary symbols via strip --discard-all. The libarrow.so file seems to be reduced from 61MB to 45 MB.
Another example:
-rwxrwxr-x 1 user group  49M Feb 22 18:55 libarrow.so.1600.0.0

$ strip --strip-unneeded libarrow.so.1600.0.0

-rwxrwxr-x 1 user group  33M Mar 12 15:27 libarrow.so.1600.0.0

This issue is to investigate the possibility of using strip and the possible disadvantages.
edited to fix typo
Component(s)
Packaging, Python

Can we check whether backtrace on crash is still available with strip --discard-all/--strip-unneeded? (We may need to use gdb for it.)

Potentially related issue with some useful content (but specifically for the wheels, so maybe mostly relevant for our cython code?): https://github.com/pypa/cibuildwheel/issues/331

I've tried on the reproducer in https://github.com/apache/arrow/issues/38770 and only --strip-debug preserves the full backtrace including non-public functions:
[...]
#7  0x00007ffff310b277 in std::terminate() () from /lib/x86_64-linux-gnu/libstdc++.so.6
#8  0x00007ffff310b4d8 in __cxa_throw () from /lib/x86_64-linux-gnu/libstdc++.so.6
#9  0x00007ffff373737b in std::__throw_bad_variant_access(char const*) () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
#10 0x00007ffff3737399 in std::__throw_bad_variant_access(bool) () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
#11 0x00007ffff37960e7 in arrow::compute::internal::(anonymous namespace)::FilterMetaFunction::ExecuteImpl(std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::compute::FunctionOptions const*, arrow::compute::ExecContext*) const [clone .cold] ()
   from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
#12 0x00007ffff42cc3e0 in arrow::compute::MetaFunction::Execute(std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::compute::FunctionOptions const*, arrow::compute::ExecContext*) const () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
[...]

More powerful strip options make the backtrace less useful because the symbols of non-public functions are removed:
[...]
#7  0x00007ffff310b277 in std::terminate() () from /lib/x86_64-linux-gnu/libstdc++.so.6
#8  0x00007ffff310b4d8 in __cxa_throw () from /lib/x86_64-linux-gnu/libstdc++.so.6
#9  0x00007ffff373737b in ?? () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
#10 0x00007ffff3737399 in ?? () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
#11 0x00007ffff37960e7 in ?? () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
#12 0x00007ffff42cc3e0 in arrow::compute::MetaFunction::Execute(std::vector<arrow::Datum, std::allocator<arrow::Datum> > const&, arrow::compute::FunctionOptions const*, arrow::compute::ExecContext*) const () from /home/antoine/t/venv-3.10/lib/python3.10/site-packages/pyarrow/libarrow.so.1500
[...]

This is on the PyArrow 15.0.2 wheels. The savings are still significant: from 61MB (original) to 54MB (stripped) for libarrow.so.

A user-reported stack trace would be possible with --strip-debug.

For macOS on pandas it seems that -g0 was used:

https://github.com/pandas-dev/pandas/issues/51900

And the fix

https://github.com/pandas-dev/pandas/pull/51971

Issue resolved by pull request 42028
https://github.com/apache/arrow/pull/42028


add a timestamp to trace events

@marten-seemann : what does the resulting trace line look like?  Are we using a IS08601 timestamp?

Yes, but the better name for the time format is RFC 3339 (https://ijmacd.github.io/rfc3339-iso8601/).


ikev2 ipsec roadwarrior with virtual subnet configuration issue

Hi there,
I am using opnsense as an IPSec VPN access point i.e. opnsense is configured as a responder only, with a roadwarrior/mobile client ikev2 configuration, as shown in https://docs.opnsense.org/manual/how-tos/ipsec-road.html . After authentication, each client gets an ip address from the virtual pool.
This set-up works fine with a single client. As soon as a second session is brought up (by a different user), traffic on the first client stops. Both clients authenticate just fine, security associations are set up (and stay up), but packets are only passed to the last client to have connected.
Logging in through SSH and poking at configuration files, I tracked down the issue to be an ipsec.conf problem:
root@opnsense:~ # cat /usr/local/etc/ipsec.conf
# This file is automatically generated. Do not edit
config setup
  uniqueids = yes
  charondebug=""

conn con2-000
  aggressive = no
  fragmentation = yes
  keyexchange = ikev2
  mobike = yes
  reauth = yes
  rekey = yes
  forceencaps = no
  installpolicy = yes
  type = tunnel
  dpdaction = clear
  dpddelay = 30s
  dpdtimeout = 330s
  left = <globally routable ipv4 address>
  right = %any
  leftid = <FQDN of the firewall>
  ikelifetime = 86400s
  lifetime = 3600s
  rightsourceip = <IP_ADDRESS>/24
  ike = aes256-sha256-ecp384!
  leftauth = pubkey
  rightauth = eap-mschapv2
  eap_identity = %any
  leftcert = /usr/local/etc/ipsec.d/certs/cert-2.crt
  leftsendcert = always

conn con2
  rightsubnet = <IP_ADDRESS>/24
  leftsubnet = <IP_ADDRESS>/0
  esp = aes256gcm16-sha256-modp4096!
  also = con2-000
  auto = add

From https://wiki.strongswan.org/projects/strongswan/wiki/VirtualIP :
[...] the remote traffic selector in swanctl.conf and ipsec.conf (remote_ts and rightsubnet, respectively) defaults to the value %dynamic. This value gets dynamically replaced by the virtual IP assigned to a client. Therefore, no remote traffic selector must be configured on the server when using virtual IPs. This ensures the client's traffic selector is correctly narrowed to the assigned virtual IP (otherwise, every client would get the same traffic selector assigned, resulting in conflicting policies).

Removing "rightsubnet = <IP_ADDRESS>/24" and restarting strongswan solved the issue: multiple roadwarriors can now simultaneously connect and their traffic is correctly routed.
Could that be an ipsec.conf generator problem, or did I miss a setting somewhere?
This is on the latest release (18.1.5, i386/openssl), on an alix2d3 board (not the fastest but it gets the job done).
When creating bug reports, please make sure you provide the following:
    The current OPNsense version where the bug first appeared

I am not sure unfortunately. I started using opnsense around last december and I believe the bug has always been present (VPN flakiness has always been reported but I haven't had time to dig in until now)
    The last OPNsense version where the bug did not exist

Like i said above, I believe the bug was already present when I started using opnsense.
    The exact URL of the GUI page involved (if any)

That would be the IPSec configuration pages. VPN > IPSec > Mobile client/Tunnel configuration.
    A list of steps to replicate the bug

Configure opnsense 18.5.1 as in https://docs.opnsense.org/manual/how-tos/ipsec-road.html
add two VPN users with a corresponding PSK (I don't think the auth type matters much, but I used EAP for mschapv2).
apply changes (which restarts strongswan)
connect the first client, notice that it gets an ip from the virtual pool and that its traffic is correctly routed through the tunnel,
connect the second client, notice that the first client, while still connected, stops receiving tunneled traffic.
log in through SSH, edit /usr/local/etc/ipsec.conf, remove or comment the rightsubnet line, type

ipsec reload

to restart strongswan.
7) disconnect and reconnect both clients, notice that they both get inbound traffic.
Optionally before step 6, type
ipsec statusall
setkey -DP

to inspect ipsec policies and see that with the rightsubnet statement, the entire virtual pool is routed to every client, thus explaining the routing issue i've been seeing.

oops, missed a spot apparently. https://github.com/opnsense/core/commit/eaf19276aefa3f3c229766d9f403844c448baf48 solves the empty rightsubnet when tunnel isolation is enabled.


fix JetBrains.gitignore template

Update

[x] Template - Update existing .gitignore template

Details
Fix JetBrains.gitignore adding "idea/" at top of file. This ensures that all files in this directory are ignored correctly

We will close this PR, as our template follows the recommendations outlined here, and ignoring the whole folder, would rather complicate than simplify things.


Enabling search on site

This PR addresses https://github.com/ml5js/ml5js.github.io/issues/4
Search is now enabled!

awesome! thanks @ashleytqy
the keys don't need to be registered with anything specific?

nope! they don't according to the docs – https://community.algolia.com/docsearch/ !

great! thanks

Thank you @ashleytqy! Just curious, how did you discover the project? (Feel free to drop me a note at the e-mail on this page.)


Allowing Besu to parse a Geth-style genesis like other clients

Description
As a node operator, I would like Besu to be able to parse configs in the same way as other clients, so interoperability testing and multi-client network operation is simpler.

@non-fungible-nelson are there testing vectors for this? What doesn't work right now?

https://discord.com/channels/595666850260713488/1252403393461489704/1252606839787622492
not much context to go on -- we should ask @shemnon or barnabas when is back from OOO

https://github.com/ethpandaops/pectra-devnets/tree/master/network-configs/devnet-2/metadata/genesis.json
https://github.com/ethpandaops/pectra-devnets/blob/master/network-configs/devnet-2/metadata/besu.json
the above shows that two separate genesis files are needed for devnet-2, in this case. one for Besu, the other for the remaining clients.

Reference: Ephemery https://github.com/ephemery-testnet/ephemery-resources

@non-fungible-nelson in case the extra parsing, needed to detect in which format the genesis file is, will impact the in case the extra parsing needed to detect in which format the genesis file is, will impact the performance in a significant way, could be an option to introduce a new dedicated option (for example --standard-genesis-file) when the user want to use the Geth-style genesis, and keeping the current --genesis-file as the optimized path?


Linkvertise: TXT files are blank

Domain
linkvertise.com
Link
https://linkvertise.com/414421/ccrm/1
Version
0.2025
What browsers are you seeing the problem on?
Firefox
What OS are you seeing the problem on?
Windows
Anything else?
Some linkvertise links are showing a text file instead of redirecting to a link. This text file is blank when FastForward is enabled. Disabling it shows the text file's contents.
FF enabled, blank text:

me too

just checked and this is still an issue with FastForward 0.2039 and Firefox 104.0b7.

fixed in later versions. if this is still an issue, please update.


CI broken @ master

https://dev.azure.com/PowerShell-FeatureFlags/PowerShell-FeatureFlags/_build/results?buildId=53&view=logs&j=068a9b68-2bbf-5452-0348-bd699ac0d931&t=ba3706fa-f190-5c4e-3095-1ff5f32c7e56
The error seems to be related to NJsonSchema:
Starting discovery in 1 files.
Microsoft.PowerShell.Commands.WriteErrorException: Error loading JSON schema
at <ScriptBlock>, /home/vsts/work/1/s/FeatureFlags.psm1: line 50
at <ScriptBlock>, /home/vsts/work/1/s/test/FeatureFlags.Tests.ps1: line 9
at <ScriptBlock>, /home/vsts/.local/share/powershell/Modules/Pester/5.0.2/Pester.psm1: line 2724
at Invoke-File, /home/vsts/.local/share/powershell/Modules/Pester/5.0.2/Pester.psm1: line 2733
at Invoke-BlockContainer, /home/vsts/.local/share/powershell/Modules/Pester/5.0.2/Pester.psm1: line 2662
at Discover-Test, /home/vsts/.local/share/powershell/Modules/Pester/5.0.2/Pester.psm1: line 1289
at Invoke-Test, /home/vsts/.local/share/powershell/Modules/Pester/5.0.2/Pester.psm1: line 2195
at Invoke-Pester<End>, /home/vsts/.local/share/powershell/Modules/Pester/5.0.2/Pester.psm1: line 4201
at <ScriptBlock>, /home/vsts/work/1/s/tools/run-tests.ps1: line 10
at <ScriptBlock>, /home/vsts/work/_temp/27c03e40-fcbc-4216-95ae-11ea3e7a5256.ps1: line 2
at <ScriptBlock>, <No file>: line 1

That was fixed in the last few commits.


not getting email after facebook success oauth.

$cordovaOauth.facebook("my_app_id", ["email,public_profile"]).then(function(result) {
$localStorage.accessToken = result.access_token;
console.log(result);
}
i using above code but not getting email in result only access token is coming.

The point of oauth is to get tokens that you then use towards provider APIs to get more information.
Unless it says in the documentation that the email should be returned with the access token, it is working correctly.  Feel free to share the documentation if its there.
Best,

 $http.get("https://graph.facebook.com/v2.2/me", {params: {access_token: result.access_token, fields: "name,email", format: "json" }}).success(function (data, status, headers, config) {         fbemail=data.email;         console.log(data);     }).error(function (data, status, headers, config) {         alert("Error: " + error);     });
by this we can get email


IE8, class reserved word

    //#########################
// Class Definition Table #
//#########################
var ClassRangeRecord = new r.Struct({
  start: r.uint16,
  end: r.uint16,
  class: r.uint16
});

This code doesn't work in IE8 because you called the property class.
Please rename it and all instances for the next version :)

Separate issue, Not wanting to spam you, but the .delete is also not allowed in IE8.
I thought this was supposed to be IE8 compatible. Did I download the wrong version?

ill report this in pdfmake

Thanks for posting back. I was confused initially as I didn't recognise the code!


Manifest 'autohotkey-installer' is removed from the Extra bucket

Bug Report
autohotkey-installer
Current Behaviour
The manifest of Version <IP_ADDRESS> is removed for the Extra bucket.
Expected Behaviour
Hope it could be back soon. I could not find any reason to remove it while the not-install version is still in the bucket.
System details
Windows version: Windows 10 Pro 21H2 19044.1766
OS architecture: 64bit
PowerShell version: PowerShell 7.2.5
Scoop Configuration
{
  "lastUpdate": "2022-07-30T12:59:08.6594548+08:00",
  "proxy": "localhost:10809",
  "SCOOP_REPO": "https://github.com/ScoopInstaller/Scoop",
  "SCOOP_BRANCH": "master"
}

It has been moved to Nonportable bucket as autohotkey-np.
related discussions: https://github.com/ScoopInstaller/Extras/discussions/8936


Should be able to restart container processes

Heroku has ps:restart and Deis needs something similar to bounce databases or other processes that may become stuck.

I'm going to take a look at this, how does this interface work for you guys?
$ heroku help restart
Alias: restart redirects to ps:restart
Usage: heroku ps:restart [DYNO]

 restart an app dyno

 if DYNO is not specified, restarts all dynos on the app

Examples:

 $ heroku ps:restart web.1
 Restarting web.1 dyno... done

 $ heroku ps:restart web
 Restarting web dyno... done

 $ heroku ps:restart
 Restarting dynos... done

I'm going to take a look at this, how does this interface work for you guys?

Could you open a proposal like #2911? I think that's the vehicle we're using for planning features with the community moving forward.

sure @carmstrong  proposal is here #3110


Cloning system labels doesn't work. Need to clone the system labels that make sense to clone.

io.rancher.container.pull_image is a system label, but it would be nice if it cloned along with user labels, as it's added by users rather than the system.

Thinking more about this, this is a bug where those type of labels should be copied.
This should not be limited to only io.rancher.container.pull_image but the other list of labels that we allow users to set that don't have a specific UI area to input.
http://docs.rancher.com/rancher/labels/

But some make no sense to copy (like sidekicks), so it's going to have to be a whitelist/blacklist.

@vincent99 I trust your judgment on which ones should be copied. :)

When cloning a service that was created from"ExternalDns" catalog , cloned service does not start successfully because of the following 2 labels missing in the cloned service:
"io.rancher.container.agent.role"
"io.rancher.container.create_agent"

This is breaking in cloning and upgrading.

Start-once is not copied over for cloning/upgrading, if selected in original service in UI.
Requested IP is not copied over for cloning/upgrading. If you clone or upgrade a stopped service, they might want to keep the same IP. Should we add it back in during clone/upgrade?
Still need to test DNS as the UI enhancement part wasn't working when I tested.

Fixed in https://github.com/rancher/ui/pull/382

Verified fixed - v0.50.2-rc2
Start-once and requested IP are updating for clone/upgrade.


Media Fusion

Addon Manifest URL
https://mediafusion.fun/manifest.json
Addon Description
Watch Tamil, Hindi, Malayalam, Kannada, English, and dubbed movies & series with the Media Fusion add-on. Source: https://github.com/mhdzumair/MediaFusion
Language of Content
Multilingual (Default)
Choose Labels

[ ] anime
[X] asian drama
[X] bollywood
[ ] cartoons
[X] debrid support
[X] http streams
[X] live tv
[X] metadata
[ ] misc
[X] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[X] torrents
[X] tv shows
[ ] video games

not getting content. Installed mutilple time but no content

not getting content. Installed mutilple time but no content

If you need any help or suggestion, Open the issues in https://github.com/mhdzumair/MediaFusion/issues.
Btw, I have introduced rate limiting as below.

General API limit: 50 / 1 min
Catalog endpoint: 150 / 5 min
Stream endpoint: 10 / 1 hour
Streaming provider endpoint and poster endpoints are excluded from rate limitation since its a real user's usage endpoints.

Maybe you're affected from this?


dev-state not showing up in 38.1

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.38.1
Python release (python3 --version):
Component/platform:
Description of problem:
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

Hi, could you provide a little more information? Thanks!

I have the same problem with 0.38.1 - nothing in dev-state, just the spinning wheel. I've searched the log for something relevant, but haven't found anything suspicious  Is

Could you clear the cache for the page? (Hold shift and hit refresh)

clearing the cache solved for me

How is that done, apart from rebooting?

@johanolde in the browser 😄

Haha! Thanks - and it solved it for me as well!

Brilliant!


Protected indices are marked for deprecation--they should no longer be marked this way.

I see a comment in ConfigConstants about the deprecation of protected indices.
// Protected indices settings. Marked for deprecation, after all config indices move to System indices.

protected indices, different from system indices, are indices that are given system index protection from the security plugin but are not formally system indices. I have not seen any use of protected indices in the demo configuration that is provided and it appears from the comment that it may have been used historically but has since been supplanted by system indices.
Should the security plugin continue to support protected indices or should these be removed in a future major version?
Task: Remove the message.

[Triage] Hi @cwperks, thank you for filing this issue. For the time being it seems like removing the logging output is the actionable task forward for this item.

@cwperks @scrawfor99 Interested in taking on this issue. What logging output are you referring to specifically? The comment or something else. Also what is this issue asking for, removing the protected indices from the codebase, or the logging that states that they are going to be deprecated since they are no longer marked? Thanks!

Hi @prabhask5, thanks for checking in :). This issue is just about removing the message for now. Since we follow Semver, there are a few things we need to do before we can completely remove the code. For now, you could open a PR to main removing the use since we have released at least one minor version with the deprecation notice. You just won't be able to backport that change since we cannot make breaking changes in minor releases.
The other thing you could do is make sure that there is a clear message logged when a used uses any of the protected index logic that way no one is caught by surprise in the eventual removal.

Hi @cwperks, I think we can remove this functionality starting version 3.x.
I'm not sure that it uses broadly in any case. Since there is no documentation about it.

@willyborankin based on this comment from @shikharj05 : https://github.com/opensearch-project/security/pull/4471#discussion_r1655816275, these might still be needed

@willyborankin based on this comment from @shikharj05 : #4471 (comment), these might still be needed

Hm, interesting. We don't provide documentation for it. The old configuration doesn't seem to have it either: https://opendistro.github.io/for-elasticsearch-docs/docs/security/configuration/system-indices/


[FormLabel] [FormHelperText] classes keys

Even though [FormControl] provides context filled/focused/error/required, not all these keys are available in [FormLabel] and [FormHelperText] classes.
I believe that filled/focused/required keys should be available as well. It is valid use case to change Component styling based on it's internal state.
I know that it is possible to achieve similar functionality now (hijack MuiFormControl context or control Component state externally), but it's bit overkill since it's already been done once.

@Mangatt Would having #7633 solve your problem and remove the need for adding these extra classes?

Probably. But I still think that it should be available directly through classes prop, there's no reason to omit those keys.

@Mangatt I agree


Change the way NDArrayIter handle the last batch

Description
Update some behaviors of NDArrayIter

Change the functionality of class NDArrayIter last_batch_handle = roll_over.
Change the timing of shuffling. Previously, it shuffles only during the initialization, which didn't meet training needs.

The above change is tackled by #12285. This PR add the following thing, please refer to concat, edge case for end variable, unit test for different kinds of input of labels and NDArrayIter_csr

Fix index out of range error (#12526 #12539 #12538) by handling the missing labels case.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

shuffle when calling the reset.
when last_batch_handle is roll_over, the last batch should be rolled over to next epoch
add the unit tests for different situations (shuffle equals to True/False, data source is mx.NDArray/ np.ndarray, 3 last_batch handle parameters and pass the normal labels/ None/ empty list)

Comments
@sandeep-krishnamurthy @zhreshold

Thanks for your contribution @stu1130
@mxnet-label-bot[pr-awaiting-review]

@sandeep-krishnamurthy  Please change the label to pr-work-in-progress

@stu1130 any update on the requested changes? Requesting to look into resolving this

@zhreshold - Any other feedback here? Thanks.

add (5000, 1, 28, 28) data source test case
https://github.com/apache/incubator-mxnet/blob/4a7c9018b392c5f8bace62ff92032c32e803cf50/tests/python/unittest/test_io.py#L112-L113


High resolutions (4K and beyond) result in a black screen

Environment:

M1 Pro 14" MBP
external 4K display
UTM 4.0.9
latest Spice Guest Tools from here
macOS Ventura host
Windows 11 ARM64 guest (I checked on both 21H2 and 22H2; I have no idea whether the bug happens on x86_64 as well or not)
any virtio display device configured for the guest (either 3D accelerated or not, it doesn't matter)
Retina Mode enabled in GUI might make it easier to reproduce

When I resize the Windows guest's display to any resolution up to 3840x2048 (which is exactly 7.5*1024*1024 pixels - not sure if that's relevant), I have a clear display, but if I try to go any bigger (which unfortunately includes standard 4K, i.e. 3840x2160), the whole VM window goes black and only the mouse cursor is rendered properly.
Note that a Linux (Ubuntu 22.04 to be precise) guest with identical VM config works fine at 4K and beyond (I tried 6720x3780 and it's fine), so I'm pretty sure this is indeed an issue with the Windows driver.
This is still better than the upstream virtio driver which seems to be limited to 16MiB framebuffer, which would cap it around ~1500p, but not being able to run Windows full screen on a 4K display at a native resolution feels somewhat limiting. Even worse, this bug is also triggered when trying to maximize the Windows VM on the MBP's internal display in Retina mode, when the display is set to "More space" - the framebuffer is then, excluding the notch area, 3600x2250, which is still above that apparent 7.5 mebipixel limit.

same issue, need help.
@Keenuts  @osy @vrozenfe @ubenmackin

I've got this issue as well. Were you ever able to resolve it?

Here's what I see happens when you try 4K:

So the cmd type is not handled here and the return value is STATUS_INVALID_TOKEN:

I'm not familiar with the code but next step would be to add more logging, recompile, and use a patched driver to get more information.


Slow download speed

Slow download speed is very annoying

Carefully read new issue template and provide all requested information.


 : Adding random for Integer.

 : feature/test
Checklist

[x] New Extension
[x] New Test
[ ] Changed more than one extension, but all changes are related
[ ] Trivial change (doesn't require changelog)

          2 Messages

:book:
Executed 174 tests, with 0 failures (0 unexpected) in 4.133 (4.363) seconds

:book:
Executed 160 tests, with 0 failures (0 unexpected) in 3.922 (4.107) seconds

  Generated by :no_entry_sign: danger

Current coverage is 38.87% (diff: 100%)

Merging #360 into master will increase coverage by 0.08%

@@             master       #360   diff @@
==========================================
  Files            46         46
  Lines          2155       2158     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            836        839     +3
  Misses         1319       1319
  Partials          0          0

Powered by Codecov. Last update e6a420b...4e68e4f

Heres a whole repo with these stuff
https://github.com/thellimist/SwiftRandom

@goktugyil Fair enough, but as a stdlib this should be a part of Int.

Can another approver approve this or post a review comment if something needs to be changed ? @piv199 @goktugyil


Implement logging for slacs

As per the title, we need more logging.

Ideally, this is restricted to ERROR level and above - we can enable the 'lower tiers' - i.e TRACE, DEBUG and INFO - via environmental variables.
External plugins should have support for the logging APIs.

Logging is essential, and I think the idea of opting-in to more verbose logging is a good one, otherwise the logs would either need to be cleaned out or rotated more frequently or it would be difficult to get necessary debugging information.
Perhaps take inspiration from Emacs with an editor buffer to display the logs and change the logging verbosity, as that would make it easier to debug the editor or its plugins without having to restart it.

Samadi van Koten<EMAIL_ADDRESS>writes:

Logging is essential, and I think the idea of opting-in to more
verbose logging is a good one, otherwise the logs would either need to
be cleaned out or rotated more frequently or it would be difficult to
get necessary debugging information.
Perhaps take inspiration from Emacs with an editor buffer to display
the logs and change the logging verbosity, as that would make it
easier to debug the editor or its plugins without having to restart
it.

You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub:
https://github.com/shymega/slacs/issues/1#issuecomment-234917261

Yes, agreed.
Yes, a 'special' buffer as per Emacs's buffers design would be
beneficial.
I can add that 'type' to the Buffer struct.
Slacs does tend to take a lot of inspiration from Emacs, I've
noticed..
--
Best regards,
Dom Rodriguez

Logging has been partially implemented in the 'logging' module.
This module implements a logging instance return function that can be called.

I've implemented TRACE and INFO level logging in the binary crate for Slacs.

For the record, it looks like the log4rs crate has been chosen.
It doesn't have support for outputting to a buffer as discussed above,
but hopefully that can be implemented.


Install instructions are for vim-plug

Clarify that the install instructions are for vim-plug.

Thanks! I original wrote it like Plug[in] so that it signals both vim-plug's Plug and Vundle.vim's Plugin, but I agree that might look confusing, and also it doesn't cover Pathogen.vim. And I think that there is no reason to ever use another plugin manager than vim-plug, so I like this change.


Unable to run build executable on different machine

I am trying to run the dmriexplorer executable on a machine where I did not build it from src. As I understand from going through the CMake files the libs are either compiled into the executable or build statically. So dependency wise we should be fine.
However, when copying the executable and zlib DLL to another machine (which has the exact same specs than mine), I only see the window popping up for a split second. Have you any idea why this happens? It runs just fine on the machine I build it with.

Hi @LorenzE
In my experience, this kind of behaviour is often a consequence of a failed shader compilation. Because they are compiled at program execution, the shader files would need to be moved altogether with the executable and DLLs. However, because the path to the shader files is hard-coded when the program is compiled and corresponds to the CMake build directory, there might be a little bit of tweaking required in order to get around this.
See line below:
https://github.com/scilus/dmri-explorer/blob/eab6662f4719f6f265d11f88f0caaa271e81d7dc/Engine/CMakeLists.txt#L13
Let me know if this helps!