Can't add text to labels when SetOptIn is true

Hi, I am using the geoman library in a situation where other layers have been drawn before that I don't want to edit, so I've been using the SetoptIn option.
When I tried to add the ability of using labels in the map via the enableDraw('Text') I noticed that the textarea was not editable, and the SetOptIn option appears to be the culprit, how can I solve this while using the option still?
I make a jsfiddle with the issue, if the option is false, then the text layer works as intended https://jsfiddle.net/y2r1hpLc/1/

did not realized this is a copy of #1227, however I still want to ask, this does not appear to be fixed, is the work around in the issue linked the only option?

You need to do following:
map.on("pm:create", (e) => {
  e.layer.options.pmIgnore = false;
  L.PM.reInitLayer(e.layer);
  if(e.shape === "Text"){
    e.layer.pm.enable();
    e.layer.pm.focus();
  }
});

Having OptIn enabled means to do some things manually, like explicitly saying that you want to enable editing of this text layer.


Terminal leaves cursor artifacts

VSCode Version: 1.49.2
OS Version: Windows 10 64 bit

I am getting weird terminal cursor blocks from time to time, here's an example. Sometimes they even occur in the middle of the text too.

Steps to Reproduce:

Write some commands in the terminal, I think the possibility rises when you make the overall text size of the editor smaller

Does this issue occur when all extensions are disabled?: Yes

Another way to reproduce is to do the following:

have the termianl window focused.
move focus to another window, the editor will do, the cursor in the terminal should now be hollow.
move focus back into the terminal and write any character or change line.

It seems the issue is the outer frame of the cursor when the window is not focussed is 1 pixel too big so it is not cleared when focus returns.
if you follow my steps above a couple of times, pressing characters (not breaking the line) and then delete the characters with backspace, the top and bottom part of the artifact remains.

Here's much more crazy example by just running node script

Have the same problem:

/duplicate https://github.com/microsoft/vscode/issues/106981


AWS deployment failure from Mac osx

AWS fails to run, cause _regex.cpython-36m-x86_64-linux-gnu.so is required for the linux.

I’m guessing this is related to question https://stackoverflow.com/questions/59438843/how-can-i-import-regex-on-aws-lambda (with no useful answer at the moment).
So, if regex can be installed on AWS Lambda it’s out of the scope of Dateparser. If you would like to get Dateparser to work without regex, that’s covered by https://github.com/scrapinghub/dateparser/issues/446


Custom date formats always use english locale

Description
When using placeholder for DateTimes with custom formatting, flutter always uses the custom format from english.
Steps to Reproduce

Setup Localization as per official docs
Fill localization with custom date formats:

// app_en.arb
{
	"date": "{date}",
	"@date": {
		"placeholders": {
			"date": {
				"type": "DateTime",
				"format": "MM/dd/yyyy",
				"isCustomDateFormat": "true"
			}
		}
	}
}

// app_de.arb
{
	"date": "{date}",
	"@date": {
		"placeholders": {
			"date": {
				"type": "DateTime",
				"format": "dd.MM.yyyy",
				"isCustomDateFormat": "true"
			}
		}
	}
}

run flutter gen-l10n
Look into the generated code for de_locale: .dart_tool/flutter_gen/gen_l10n/app_localization_de.dart

Expected results:
I expect the generated format to be dd.MM.yyyy
  String date(DateTime date) {
    final intl.DateFormat dateDateFormat = intl.DateFormat('dd.MM.yyyy', localeName);
    final String dateString = dateDateFormat.format(date);

    return '$dateString';
  }

Actual results:
But the generated format is MM/dd/yyyy
  String date(DateTime date) {
    final intl.DateFormat dateDateFormat = intl.DateFormat('MM/dd/yyyy', localeName);
    final String dateString = dateDateFormat.format(date);

    return '$dateString';
  }

Triage report
I can reproduce this issue on Master (3.7.0-4.0.pre.38). I followed the official docs, expect deleted the l10n.yaml as flutter gen-l10n was giving me the following error:
Because l10n.yaml exists, the options defined there will be used instead.
To use the command line arguments, delete the l10n.yaml file in the Flutter project.

@thkim1011 can you take a look at this? is this supposed to be supported?

https://github.com/lsaudon/l10nization/pull/228#issuecomment-1597823315

Any solutions? I have the same problem

Any solutions? I have the same problem
You can do this.

{
  "@@locale": "en",
  "dateFormat": "dd MMMM yyyy",
  "textWithDate": "It's {date}",
  "@version": {
    "placeholders": {
      "date": {
        "type": "String"
      }
    }
  }
}

And
context.l10n.textWithDate(DateFormat(context.l10n.dateFormat).parse(DateTime.now()));

Hi, we also face this problem and I see this issue hasnt been resolved in 2 years. Is there a plan to do so?


[Impeller] Implement mipmap generation for backdrop filters for opengles

This makes blurs look nicer.  It was disabled out of caution to get the feature landed for Metal only after vulkan didn't work.  It might work for opengles though.
See TODO in impeller/entity/entity_pass.cc

@gaaclarke was there a problem observed in OpenGL?

With out tests in CI we can verify it doesn't crash at least.  It would be nice to get visual confirmation that it looks good too with some manual testing.

Since we're not using mipmaps for backdrop filters anymore, I'm going to close this issue.


ENH: shapely.plotting module with basic matplotlib-based functionality (+ updated docs to use this)

Closes #1285
As explained in https://github.com/shapely/shapely/issues/1285, this adds basic functionality to visualize single geometries, useful for debugging/reporting issues, and needed for our own docs as well.
This PR adds a shapely.plotting module (currently with plot_polygon, plot_line, and plot_points functions), and updates our doc examples to use this public module (instead of descartes + bunch of private helpers).
You can see the built docs using this branch at https://shapely.readthedocs.io/en/plotting/manual.html, and you can compare the resulting figures with the ones on the stable docs: https://shapely.readthedocs.io/en/stable/manual.html
The default plot_polygon without any options gives (with the current version of the PR):

But for the doc examples we do some additional customizations (eg no grid, changing some colors, not always showing the points, etc).
Some general notes / questions:

~The current methods (plot_polygon, plot_line) don't yet handle their multi counterpart (so you have to loop through .geoms manually), but for convenience we should probably add that capability~
What would be the best name for plot_points? plot_coords instead? (this function takes any kind of geometry, extract all coordinates, and plots each coordinate pair (the vertices) as points)
~In the current examples, when plotting a line or polygon, we typically plot the points (if they are plotted) always in grey, while in my current code those points get plotted in the same color as the line/polygon. Any preference here?~ -> kept original look of grey points in the docs (but the default is to use the same color)
Some examples showing a line (eg https://shapely.readthedocs.io/en/stable/manual.html#LineString) highlight the start and end points of the line (black instead of grey). Do we want to keep that?

And if so, is that useful enough to add this to the public function, or do we keep that as a customization in the doc example code?
-> added this manually to the code snippets (so not built-in)

~TODO: clean-up the public module (and improve docs) + add tests (for now I didn't add any explicit tests, except for running our own docs with it)~

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 49 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.6%) to 82.662%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shapely/plotting.py
0
49
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.6%

Covered Lines:
2074

Relevant Lines:
2509

💛  - Coveralls

I updated this PR:

Support for directly plotting a MultiLineString or MultiPolygon with the plot_line and plot_polygon functions
Tweaked the figures code a bit more to ensure we now have more or less exactly the same figures as before (making points grey, start/end points black (where the figure caption in manual.rst mentioned this), etc)
Expanded the docstrings and added a note that this is experimental

I am planning to merge this shortly since this PR is needed to get the figures in the doc build working again. It does add public API (so review would still be very welcome!), but so I added a disclaimer that this is experimental  / mostly meant for explorative use (so we can still further update the exact interface if needed)

This will be a really useful feature! Answer to the question: plot_points is good, since it has closer alliance with matplotlib point plots with markers than the geometry type. I think other features, like point start/end colors etc. could be changed later.
One question is with the add_points option. The defaults are add_points=True, but there are 89 instances in the code with add_points=False. Should this keyword be removed? As demonstrated in the code, if a user wants to add points to a line or polygon, they can simply add plot_points(a, ax=ax, ...).

One question is with the add_points option. The defaults are add_points=True, but there are 89 instances in the code with add_points=False. Should this keyword be removed? As demonstrated in the code, if a user wants to add points to a line or polygon, they can simply add plot_points(a, ax=ax, ...).

Yeah, I have certainly been wondering that myself as well (or at least whether the default should be add_points=True if most of the time in our code snippets I turn it off).
The reason that I kept it for now (and with a default of True), is that I think it is the best default for interactive / debugging use cases. For example, see the plot of the polygon in the top post, which is what you get from a plain plot_polygon(poly), and I personally quite like this for a quick introspection of a polygon.
For making more fine-tuned figures for illustrations in the documentation, it appears that most of the time I didn't actually use this default. But in those code files I also always manually set up the matplotlib figure and axes, changed the default colors (regularly changed the default alpha), etc. So for those cases it is fine to manually do plot_points in addition to plot_line/plot_polygon to have greater control (in one of the later commits I added a lot of manual add_points to give them a different color as the line/polygon, just to preserve the look&feel of the figures we currently have)

@mwtoews I am going to merge this, because I want to further get the docs fully ready for the 2.0 (beta) release.
But I certainly would like to further discuss the default / the need for the add_points keyword. I opened https://github.com/shapely/shapely/issues/1550 for this.

Is this intentionally left out of the API reference? It seems to be public API.


Filter by namespace

I would like to run the analyze for a specific namespace instead of the entire cluster. --filter by namespace or just a --namespace flag would be ideal.

Great idea, we'll put this into an upcoming release

I think we need not only namespace, but exact target. like:
$: k8sgpt analyze --namespace $NAMESPACE --%resource_type% $RESOURCE_NAME

in case if you want to analyze pod/service/deployment/statefulset and etc.

@yurymkomarov: Thank you for your suggestion. We already have the ability to filter for resource types (--filter). I'm curious if we should add naming patterns in there.
@AlexsJones: What do you think about this?

I think filtering namespace is an easy thing to implement and we will do it. Exact resource names could be doable.. GVR ( group version resource) but it has a maintenance over head so I am a bit reluctant unless there is an issue it solves


AttributeError: 'float' object has no attribute 'split'

Hello....
When I run the below code,
python longQC.py sampleqc -x pb-rs2 -o prueba14 /home/jforero/mis_datos/tutoriales/anaconda3/prueba10.fasta -p 72 -d --fast -m 2 -i 10 --trim_output trimmedsequences
It gives me an error saying that there is attribute error: 'float' object has no attribute 'split' .
I would like to know why this error comes about.
lq_coverage:2020-08-03 10:38:59,423:122:INFO:Estimation of coverage distribution finished.
Traceback (most recent call last):
  File "longQC.py", line 932, in <module>
    main(args)
  File "longQC.py", line 62, in main
    args.handler(args)
  File "longQC.py", line 592, in command_sample
    adp3_pos=np.mean(adp_pos3) if args.adp3 and adp_pos3 and np.mean(adp_pos3) > 0 else None)
  File "/datos/datosjforero/tutoriales/anaconda3/LongQC/lq_coverage.py", line 373, in plot_unmapped_frac_terminal
    t5l, t3l, il = self.__region_analysis(3, 1)
  File "/datos/datosjforero/tutoriales/anaconda3/LongQC/lq_coverage.py", line 596, in __region_analysis
    regs = [(int(reg.split('-')[0]), int(reg.split('-')[1])) for reg in str.split(',')]
AttributeError: 'float' object has no attribute 'split'

I was misplacing the input file

Hi,
I got the same problem as you but I don't understand what you meant by "putting the input file wrong" ?


make mattermost resizeable on Android DeX #3427

Summary
This makes Mattermost resizeable on Android Desktops like Samsung DeX
Ticket Link
https://github.com/mattermost/mattermost-mobile/issues/3427
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates

Device Information
This PR was tested on: Samsung S9

Hello @antifarben,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.
Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.
This is a standard procedure for many open source projects.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

/check-cla

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/11618
https://circleci.com/gh/mattermost/mattermost-mobile/11617

Artifact links:
https://11618-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.apk
https://11617-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.ipa

Ticket for reference: https://mattermost.atlassian.net/browse/MM-30210


Gazetteer Data sets restricted by roles

Problem
Currently in a workspace you cannot restrict a gazetters dataset per user role. You have to make two separate locales and restrict them to a role to achieve this.
Requested Solution
Could we add the role object to a gazetteers dataset!
    "gazetteer": {
        "provider": "GOOGLE",
        "country": "GB",
        "placeholder": "eg. London",
        "label": true,
        "optional": true,
        "datasets": [
          {
            "layer": "LAYER",
            "table": "TABLE",
            "label": "LABEL",
            "roles": {
              "role_1": null
            }
          },
          {
            "layer": "LAYER_2",
            "table": "TABLE_2",
            "label": "LABEL_2",
            "roles": {
              "role_2": null
            }
          }]
      },

The gazetteer module will get a location from the layer in the dataset configuration.
Roles should therefore be taken from the layer and not the dataset configuration in the gazetteer.
This is now enabled. https://github.com/GEOLYTIX/xyz/commit/6e9a1eba27ce4b6e0fd59fde0715cce62af6ec12


Ability to add tags

if possible please add a feature where we can attach tags to link and search and sort via tags .
this would make sorting and searhing very easy if there is a large no. of links

Linkora has folders that already organize links, and you can sort and search through them as needed. Adding tags wouldn't make much sense with folders in place 🤔. Searching within folders will be added in upcoming releases.

ok bro

So are you saying a link can be in multiple folders at the same time?
Tagging really is much better than folders. But I do use both with Raindrop at least, it does make perfect sense.


Cross validation in Elastic Net

Note: Currently this is rebased on top of the grouping PR (#69) and has the commits from that PR. When that PR is merged, this should be corrected to only the CV commits.

@njayaram2
Apologies for the major history change. Rebasing to master required squashing commits (to ease the rebase). Your comment related to independent variables naming should be addressed now.
Please continue with your review.

+1 LGTM


Need a SO_BINDTODEVICE to complete my fork.

the Issue #37 is closed.
I try to use 2 UDP socket, but that not enough. I did not find valid solution to bypass this lack of feature.
Should I use raw-socket or ffi patch ?
any point of view ?

added an issue on node  for the same problem https://github.com/nodejs/node/issues/35769

Thx for this issue.

Thx for this issue.


Update rebar to work with pc 1.0; lock port compiler dep

An update to the port compiler on hex caused an incompatibility with the
version of rebar we had vendored. Here we update the vendored rebar3 to
work with the latest pc plugin, and lock the pc plugin to avoid future
unexpected breakage.

LGTM, hope travis likes it. :+1:

:+1:


No error on application key denial

When an application key has either been deleted or revoked, there is no indication from the connector. It continues as normal and displays the same 'null' result as it would on success.

The current best option appears to be an initial query to Thingworx at connector startup using a different Accept header in the HTTP request. If we use the application/xml value, we may receive a more verbose response with an indication of app key failure which we can interpret on the Flexy side.


new h5 links and bugfixes

This implements the h5 link writing and fixes a bug with the datatype of the samples

Agreed. Is this now in a new pip release or will that come later?

On Aug 30, 2021, at 7:05 AM, Jeremy Magland @.***> wrote:
@magland commented on this pull request.
In src/python/sortingview/extractors/labboxephysrecordingextractor.py https://github.com/magland/sortingview/pull/133#discussion_r698515907:

def store_recording_link_h5(recording: se.RecordingExtractor, save_path:str, format='h5_v1'):

   #print('Creating h5 recording')

   H5RecordingExtractorV1.write_recording(recording=recording, h5_path=save_path)

   print('Creating efficient recording: storing link in kachery...')

   h5_uri = kc.link_file(save_path)

   object = {

       'recording_format': 'h5_v1',

       'data': {

           'h5_uri': h5_uri

       }

   }

   print('Done creating efficient recording.')

   return LabboxEphysRecordingExtractor(object)

I think the format argument can be removed
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/magland/sortingview/pull/133#pullrequestreview-741686319, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSIHN4D4QBGWFKQ33KTT7OF2FANCNFSM5DBDKSNQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This will be released in sortingview 0.6.x, which I think should be ready once we've agreed on the steps we're discussing on slack right now.


Update eth-keyring-controller

eth-keyring-controller: 5.3.0 -> 5.6.0

Codecov Report

Merging #216 into develop will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop      #216   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           22        22
  Lines         1488      1488
  Branches       206       206
=========================================
  Hits          1488      1488

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 6dd8e89...c5d1f73. Read the comment docs.


Using TModel in ViewImports causes compile errors at design time

We generate directive helper statements to provide intellisense for @inject IHtmlHelper<TModel> that look like:
((System.Action)(() => {
    global::Microsoft.AspNetCore.Mvc.Rendering.IHtmlHelper<TModel> __typeHelper = null;
}
))();

The problem is TModel isn't a valid C# term. @pranavkm got around this in the past by essentially doing a using TModel = dynamic statement at the top of the file.

0b5113c76ee3391bd2e46f2f62dad7a5081d8c14


ikhal: Slow when saving events

From #160
ikhal seems to re-scan the whole calendar (or do some other inappropriate task) when saving an event. This freezes the whole UI.

Another technically unrelated problem I've found is that creating an event with infinite daily recurrence takes an extremely long time.

I cannot actually reproduce this, how long a freeze are we talking about here?


Dellet my work waht same look for thma sami waht we can do    Revert "exposed loop defer function from capi (#1579)"

This reverts commit 09d2795852bf3217e944ceaea8f3fddf538ada40.

Mr. porno, I can't understand what you're saying


Cannot install K-9 Mail v5.109 (alpha) apk

Expected behavior
I should be able to install K-9 Mail v5.109 (alpha) apk.
Actual behavior
I get a generic error: App not installed.
Steps to reproduce

Download apk.
Attempt to install it.
Have it fail… download it a few more times just in case it was corrupted; it still fails.

Environment
K-9 Mail version: I currently have v5.010 installed, via F-droid.
Android version: 5.1.1 (Cyanogenmod 12.1-20151117-SNAPSHOT-YOG7DAO1KI-d802)
FWIW I'm attempted to upgrade to use PGP/MIME support. The current issues are a bit vague, but there's some suggestion that it's been fixed. However, the more-recent release, 5.010 still doesn't support PGP/MIME. Does the alpha support it?
Further, my current install of K-9 was via F-droid. Do I need to uninstall this version first? I'm worried that my email and settings will be deleted if I do so.

You can't upgrade installations from F-Droid with our APKs due to different signatures.
In the future please use the mailing list for support requests.

Thanks for the information. Apologies, I will post there in the future.


feat(vertical nav): adds simple vertical nav list

Closes #27

Deploy preview for pf-next ready!
Built with commit 63baeb4ebeee4923469ce1b834d394b346b72d0f
https://deploy-preview-311--pf-next.netlify.com

Hi Matt, I still need to get your branch and review it, but to kick it off we need to remove the demo. is that just to preview it as we review?

@andresgalante Removed demo and updated border decoration!

@andresgalante @jgiardino I've addressed all comments in this review.
Some notable changes:

Padding has been moved from the button to the button text. This allows devs to omit a presentational icon and have the active border positioned correctly.
Added a second conceptual layout for including a link AND button that opens a submenu.
Added aria labels throughout


Karma 0.13.0 - "Can not resolve circular dependency!"

After upgrading to Karma 0.13.0, I'm hitting the following issue:
16 07 2015 08:23:32.494:WARN [preprocess]: Can not load "webpack"!
  Error: Can not resolve circular dependency! (Resolving: preprocessor:webpack -> webpackPlugin -> preprocessor:webpack)
    at error (project/node_modules/karma/node_modules/di/lib/injector.js:22:68)
    at get (project/node_modules/karma/node_modules/di/lib/injector.js:44:15)
    at instantiatePreprocessor (project/node_modules/karma/lib/preprocessor.js:56:37)
    at Array.forEach (native)
    at null.<anonymous> (project/node_modules/karma/lib/preprocessor.js:77:31)
    at tryCatcher (project/node_modules/karma/node_modules/bluebird/js/main/util.js:26:23)
    at ret [as _preprocess] (eval at <anonymous> (project/node_modules/karma/node_modules/bluebird/js/main/promisify.js:163:12), <anonymous>:13:39)
    at project/node_modules/karma/lib/file-list.js:182:19
    at Array.map (native)
    atproject/node_modules/karma/lib/file-list.js:167:30
    at Array.map (native)
    at List._refresh (/project/node_modules/karma/lib/file-list.js:150:37)
    at List.refresh (/project/node_modules/karma/lib/file-list.js:249:27)
    at Plugin.notifyKarmaAboutChanges (/project/karma-webpack/index.js:109:16)
    at Plugin.<anonymous> (/project/node_modules/karma-webpack/index.js:73:9)
    at Tapable.applyPlugins (/project/node_modules/webpack/node_modules/tapable/lib/Tapable.js:26:37)

I'll keep investigating on my own, but I thought I'd report it in any case.

It looks like similar reports are coming through on Karma's end:
https://github.com/karma-runner/karma/issues/1497

Have the same error....

+1

+1

bump

+1
Oh hey @nhunzaker

:+1:

Cool thanks, after downgrading to 0.12.37 it also works.

+1

+1!!!!1111

+1

+1

+1000^42

+1

I second.

+1

Let's merge PR #64 :-)

Or #63 though

+1

hooray :smile:


chore(main): release 1.3.2

:robot: I have created a release beep boop
1.3.2 (2024-02-03)
Miscellaneous Chores

deps-dev: bump symfony/http-client from 7.0.2 to 7.0.3 (#63) (1e01420)
deps: bump symfony/property-access from 7.0.0 to 7.0.3 (#62) (3de6497)
deps: bump symfony/serializer from 7.0.2 to 7.0.3 (#64) (1778b53)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/mattermost-php/releases/tag/v1.3.2 :sunflower:


Fix: Wrong credential alert position

Issue Template
Issue type

Bug

Description
When the User enters wrong credentials, the alert displayed is not properly aligned.
Expected behavior

Properly align and show the wrong credential alert to the user OR enhance the way of showing the alert.

Screenshots or videos

Hello, I am very interested in contributing to MealMatters and would like to tackle this one. Hope you have a great day.

Hello, I am very interested in contributing to MealMatters and would like to tackle this one. Hope you have a great day.

@mjs001
You can start working on this.

Thank you, I will start on this today.

OK, before I submit a pull request I wanted to send you a screenshot of my changes and see if they match your needs. Happy to alter the alerts position.


Support 'max_rows' parameter in DataFrame.to_markdown()

https://github.com/databricks/koalas/blob/master/databricks/koalas/frame.py (line: 1767)

According to the code above, the to_markdown function in Koalas DataFrame seems to support the max_rows parameter. However, to_markdown, which can be supported by pandas version from 1.0.0, did not support 'max_rows' from the beginning. (https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.to_markdown.html)
Therefore, the following error occurs.
>>> kdf = ks.DataFrame(data={"animal_1": ["elk", "pig"], "animal_2": ["dog", "quetzal"]})
>>> kdf.to_markdown(max_rows=1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/hwpark/Desktop/git_koalas/koalas/databricks/koalas/frame.py", line 1814, in to_markdown
    kdf._to_internal_pandas(), self.to_markdown, pd.DataFrame.to_markdown, args
  File "/Users/hwpark/Desktop/git_koalas/koalas/databricks/koalas/utils.py", line 413, in validate_arguments_and_invoke_function
    % (pd.__version__, param.name, pandas_func.__name__)
TypeError: The pandas version [1.0.3] available does not support parameter 'max_rows' for function 'to_markdown'.

In general, I'm curious whether to remove max_rows parameter support to match pandas in this case, or bypass validate_arguments_and_invoke_function to only use Koalas.

I think we can just remove the max_rows parameter with related note in docstring and code since pandas doesn't support it.

Okay I will just remove the max_rows parameter.

Yup, go for it :)


NWPS-1107: Removal of conditional check on GTM inclusion

This ticket will need an additional change to the JavaScrtipt to manage the name change for the cookie. (Currently in the JS it is called analyticsCookie but it needs to change in the JS to be called cookie_consent, to match the checking done in this ticket).
See the video in the JIRA ticket to see the way in which the dataLayer array is used in conjunction with the GTM calls

@FranciscoRuizHEE @smanivasag . Please review this request.

Moved all JS from static assets to repository data/webfiles locations
Updated all references in freemarker to reflect on new location
Renamed analyticsCookie to cookie_consent and updated JS, java & freemarker files
Update readme files
Created GitHub action to push changes to test  environment to ensure how it is reflecting on live data


Connect submitted form data with database

Create queries to transfer requests and users into the database.
This is a point where we should validate inputs and whitelist certain types of inputs:
https://www.owasp.org/index.php/Testing_for_Input_Validation
https://www.owasp.org/index.php/Input_Validation_Cheat_Sheet
http://codereview.stackexchange.com/questions/30237/form-validation-security-and-input-specification

[ ] - Store users (Yoti data)
[ ] - Store requests (form data)


fix(resource): RHOAIENG-1150, fix typo in tutorial

Closes: #RHOAIENG-1150
Description
One of the tutorial in the Dashboard > Resources page has a typo in the title: Buildng a binary classification model

/ok-to-test


Add CI to sig-win-dev-tools

We have no github actions! we need ... something?

can we make windows VMs in github actions ?
or should we use a vagrant cloud provider or something ?

Hi @jayunit100,
Unfortunately Github Actions doesn't support VT-x flag:
==> controlplane: Clearing any previously set network interfaces...
==> controlplane: Preparing network interfaces based on configuration...
    controlplane: Adapter 1: nat
    controlplane: Adapter 2: hostonly
==> controlplane: Forwarding ports...
    controlplane: 22 (guest) => 2222 (host) (adapter 1)
==> controlplane: Running 'pre-boot' VM customizations...
==> controlplane: Booting VM...
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["startvm", "af6d8f40-2a53-411a-9c6a-cb49ded533a2", "--type", "headless"]

Stderr: VBoxManage: error: VT-x is not available (VERR_VMX_NO_VMX)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component ConsoleWrap, interface IConsole
make: *** [Makefile:42: 2-vagrant-up] Error 1
Error: Process completed with exit code 2.

A possible solution: Allocate some machines 24/7 with VTx and add these machines as Self-host workers (Github runner)

Another point to consider, adjust the network classes, GitHub Actions currectly complains about 10.20.30.x already set in the scripts. It's trivial change as showed here:
diff --git a/sync/linux/controlplane.sh b/sync/linux/controlplane.sh
index 070db0b..f88a05f 100644
--- a/sync/linux/controlplane.sh
+++ b/sync/linux/controlplane.sh
@@ -21,7 +21,7 @@ if [[ "$1" == "" || "$2" == "" ]]; then
   cat << EOF
     Missing args.
     You need to send kubernetes_version, k8s_kubelet_nodeip i.e.
-    ./controlplane.sh 1.21 <IP_ADDRESS>
+    ./controlplane.sh 1.21 <IP_ADDRESS>
     Normally these are in your variables.yml, and piped in by Vagrant.
     So, check that you didn't break the Vagrantfile :)
     BTW the only reason this error message is fancy is because friedrich said we should be curteous to people who want to
diff --git a/sync/shared/variables.yaml b/sync/shared/variables.yaml
index e409610..f63fae5 100644
--- a/sync/shared/variables.yaml
+++ b/sync/shared/variables.yaml
@@ -5,12 +5,12 @@ kubernetes_version: "1.25"
 build_from_source: "false"

 ## Linux settings
-k8s_linux_kubelet_nodeip: "<IP_ADDRESS>"
+k8s_linux_kubelet_nodeip: "<IP_ADDRESS>"
 linux_ram: 4096
 linux_cpus: 4

 ## Windows settings
-windows_node_ip: "<IP_ADDRESS>"
+windows_node_ip: "<IP_ADDRESS>"
 windows_ram: 6048
 windows_cpus: 4

diff --git a/sync/windows/1-antrea.ps1 b/sync/windows/1-antrea.ps1
index 47e06a1..004d3ab 100644
--- a/sync/windows/1-antrea.ps1
+++ b/sync/windows/1-antrea.ps1
@@ -15,7 +15,7 @@ limitations under the License.
 #>
 Param(
     [parameter(HelpMessage="Windows Node IP")]
-    [string] $windowsNodeIP = "<IP_ADDRESS>"
+    [string] $windowsNodeIP = "<IP_ADDRESS>"
 )
 $ErrorActionPreference = 'Stop'

Here a demo GitHub Actions for the project:
sig-windows-dev-tools> mkdir -p  .github/workflows/initialtest.yml
name: Run Tests

on:
  workflow_dispatch:
  push:
    branches: [ main ]
  pull_request:
    branches:
      - main
jobs:
  initialtest:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - name: Run tests
        run: |
          sudo curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo apt-key add -
          sudo apt-add-repository "deb [arch=amd64] https://apt.releases.hashicorp.com $(lsb_release -cs) main"
          sudo apt-get update
          sudo apt install build-essential vagrant virtualbox -y
          sudo vagrant plugin install vagrant-reload vagrant-vbguest winrm winrm-elevated vagrant-ssh
          make all

As mentioned above, even with the network class change, GH Actions still doesn't support VT-x. As soon it supports, it should work out of box.

@jayunit100 please let me know your thoughts.

ah great find thanks doug

ah looking https://github.blog/changelog/2022-09-01-github-actions-larger-runners-are-now-in-public-beta/ enterprises can do this?

ah looking https://github.blog/changelog/2022-09-01-github-actions-larger-runners-are-now-in-public-beta/ enterprises can do this?

From my understanding, this is only increasing capacity, not CPU flags in their environment. :(


Vendor ID is missing, causing an out-of-index crash

Heyo! I wanted to contribute data for my board (ASRock B450M Pro4), but I get a crash:
Traceback (most recent call last):
  File "/root/iommu_info_generate/generate_data.py", line 174, in <module>
    main()
  File "/root/iommu_info_generate/generate_data.py", line 142, in main
    vendorid = json.loads(r.text)[0].get("vendorid")
IndexError: list index out of range

(r.text is just [])
I guess the ASRock vendor ID isn't registered? In any case, the client should handle that gracefully :)

Thanks for reporting the issue. I found some other vendors that don't have registered id's.
I'll put together a patch to the script with a DEBUG mode. Hopefully it will expose the vendor id that i can add to the db.

Can you send me the output of
cat /sys/devices/virtual/dmi/id/board_vendor

This will handle the failed lookup - d7fbe43b92782b18e638e17a02d5ab3564b7bc8f - but there is still some work to support your setup.
If you can paste me the output of these commands, I can track down the error.
cat /sys/devices/virtual/dmi/id/board_vendor
cat /sys/devices/virtual/dmi/id/bios_vendor

@mkoreneff


2.x Fix bug when Timber\Theme encoding breaks Timber caching

Resolves:

#2429
#2646

Issue
In #2264, a change was added that adds a recursion for the Timber\Theme object. When serializing the object with json_encode(), this will cause the serialization to fail. Because of this, Timber can’t create a caching key and returns the same for different posts (see #2429 and #2646).
Solution
This pull request:

Implements the JsonSerializable interface for the Timber\Theme class so that it doesn’t break json_encode().
Adds a test for the serialization.
Adds a check whether json_encode() was successfull and skips the cache if no cache key can be generated.
Fixes testing some other theme functions (in https://github.com/timber/timber/commit/11f601e391a703edc4a4a844cb68c239db31e9b6, a change was added that caused a lot of theme tests to be skipped).

Impact
None.
Usage Changes
None.
Considerations
I chose to go with JsonSerializable because this is a pattern that we already advise to use in cases like these: https://timber.github.io/docs/v2/guides/posts/#serialization.
This change may also be helpful for people who need theme data in a JSON-friendly version.
Testing
Yes.

Tests only fail when running with coverage, but I couldn’t see why yet. I would merge this in and then check whether tests still fail on the 2.x branch.

I fixed the tests. Test only fail on PHP 8.2, which we ignore for now. I’m merging this.


mtools: 4.0.18 -> 4.0.19

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/mtools/versions.
meta.description for mtools is: '"Utilities to access MS-DOS disks"'.

Checks done (click to expand)

built on NixOS
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mattrib passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcat passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcd passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mclasserase passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcopy passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mdel passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mdeltree passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mdir passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mdu passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mformat passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/minfo passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mlabel passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mmd passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mmount passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mmove passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mpartition passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mrd passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mren passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mtype passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mtoolstest passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mshortname passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mshowfat passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mbadblocks passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mzip passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/lz passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mtools passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mkmanifest passed the binary check.
Warning: no invocation of /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcheck had a zero exit code or showed the expected version
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mxtar passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/uz passed the binary check.
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/tgz passed the binary check.
Warning: no invocation of /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcomp had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/amuFormat.sh had a zero exit code or showed the expected version
30 of 33 passed binary check by having a zero exit code.
0 of 33 passed binary check by having the new version present in output.
found 4.0.19 with grep in /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19
directory tree listing: https://gist.github.com/3335bbe7e0d135e290e366fccd7c66c1
du listing: https://gist.github.com/ec18825cfbda7f99fb0b9c7dd5660772

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/aee3689349c3ba13422c4c63de82dd6902abbb1f.tar.gz -p mtools
ls -la /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19

Success on aarch64-linux (full log)
Attempted: mtools
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin
patching script interpreter paths in /nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin/amuFormat.sh: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin/mcomp: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin/tgz: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin/uz: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin/mxtar: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19/bin/mcheck: interpreter directive changed from "/bin/sh" to "/nix/store/017xymp5i03qzigafnfx7sgf6602gmzq-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19...
/nix/store/gyi9rhdv2n0rlxj88rwmx8d64v1p8zka-mtools-4.0.19

Success on x86_64-darwin (full log)
Attempted: mtools
Partial log (click to expand)
strip is /nix/store/9xjkb4xz0b5lmizij9ppxy7lkxdxhx6b-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin
patching script interpreter paths in /nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin/mcheck: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin/mcomp: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin/uz: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin/tgz: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin/mxtar: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19/bin/amuFormat.sh: interpreter directive changed from "/bin/sh" to "/nix/store/f2lp21azk4w88f17zgf4mji64agb18ga-bash-4.4-p23/bin/sh"
/nix/store/vhzjywn2jh6qk5jhay1z053yw0sqq5s8-mtools-4.0.19

Success on x86_64-linux (full log)
Attempted: mtools
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin
patching script interpreter paths in /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mxtar: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcheck: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/mcomp: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/uz: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/amuFormat.sh: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19/bin/tgz: interpreter directive changed from "/bin/sh" to "/nix/store/nii7pk6pv4x4as7vsxbvwyzjn67vax6r-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19...
/nix/store/gavcdh0xvgimjss4bgjig6dpnzbfsyyb-mtools-4.0.19


Highlight npx as a command that runs other commands

npx is a command from Nodejs that executes a command either from the path or from the installed Node packages: https://www.npmjs.com/package/npx
This means that when I run something like npx webpack, the command that "really" gets run is webpack, not npx.
I know that when I run time or sudo, that gets highlighted differently from the command that "really" gets run. Is it possible to do something similar for npx? I expect that one issue is that the following command may or may not be in the PATH, which makes it tough to highlight it correctly.
I'm not deep enough in zsh to know if this is something that requires a code change to this package or some way of tagging the command within zsh, but any advice or thoughts would be welcome!

Since npx in effect uses its own $PATH (e.g. npx echo foo fails but npx tsc runs even though tsc is not a command zsh recognizes), npx doesn't fit with the currently supported precommands. Supporting npx properly would probably take adding a way to specify a precommand specific PATH (which could also be used by sudo cf. #107).


Removed unecessary data-validation attributes from passwords

Closing, just noticed that @jollopre did this in his latest PR


400 Bad Request (error code: 50109): The request body contains invalid JSON.

Summary
Failed to send a message, while it was working before.
Reproduction Steps

Bot definition:

bot = commands.Bot(
    self_bot=False,
    command_prefix='r! ',
    captcha_handler=HCaptchaHandler()
)

Send code:

channel = await author.create_dm()
await channel.send(some_message, file=discord.File(file_path))

Note: I tried with a slightly older version of the master branch and it works fine. It was the master branch from a week before or so, which is installed on my local computer.
Code
No response
Expected Results
Message sent
Actual Results
Error:
ERROR:asyncio:Task exception was never retrieved
future: <Task finished name='discord.py: on_message' coro=<Client._run_event() done, defined at /usr/local/lib/python3.9/dist-packages/discord/client.py:488> exception=AttributeError("'str' object has no attribute 'traceback'")>
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/discord/client.py", line 496, in _run_event
await coro(*args, **kwargs)
File "", line 620, in on_message
await channel.send(fine_tune(random.choice(starter_phrases)), file=discord.File(recruit_banner))
File "/usr/local/lib/python3.9/dist-packages/discord/abc.py", line 1654, in send
data = await state.http.send_message(channel.id, params=params)
File "/usr/local/lib/python3.9/dist-packages/discord/http.py", line 843, in request
raise HTTPException(response, data)
discord.errors.HTTPException: 400 Bad Request (error code: 50109): The request body contains invalid JSON.
System Information
Using the latest master branch. (Just installed)
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

Should be fixed! Not sure captcha handling with form body payloads ever worked though; if it did, Discord changed something.


UI for adding or editing API proxy

For the Settings view, add a form for adding and editing proxy. (In first phase, only API Umbrella) . The code should be reusable for the Settings wizard, if possible.
Definition of done

[ ] Settings page contains form for adding and editing proxy
[ ] The form is autopopulated from the Proxies schema
[ ] The form contains show/hide fields based on proxy type

Need to learn what is already included before I start wireframing this.

@Nazarah we had an idea to place the add proxy modal window on the /settings page.

Wireframes added.
Added a message notifying users about the consequence of adding/not adding an API proxy.
Please review and give feedback

@Nazarah The message is a good idea, but since it is administrators who adds the proxy settings to the platform (while API owners only select a proxy for their own API in API profile), I would like to suggest some changes to the wording:
"With no proxy defined, you will only have access to the Catalog feature. Defining a proxy will give you access to proxy features for APIs and to Analytics."

Closed by PR #1399


Add ability to send a tiny test file to the check/sftpcheck Azure Function

CheckFunction is designed to help us diagnose problems with the system.   Right now it only checks one thing:  sftp connections too states.
It is meant to be run manually, with postman or curl, ad-hoc, as we on-board states, and as we detect problems.  So, its a diagnostic tool.
This task is DONE when:
[ ] the sftp check has a new optional way to send a file.  this should be implemented as an http parameter, eg &sendfile.   The file should be sent as the content body of the sftp check call, just like we do for reports.   For example the payload could be specified like this to a. curl call --data-binary<EMAIL_ADDRESS>[ ] if the file is over a certain size, fail.  Only allow small files (???)
[ ] The current way of running sftp check work exactly as it does not, if the new file option is not included.

Would you setup the rate limiting in Azure as part of some WAF controls? Or were you thinking of implementing rate limiting in the application code?

@jimduff-usds @MauriceReeves-usds
I have some thoughts / questions on this ticket. The current end point is a GET end point. I'll need to make it a POST as well to allow for an uploaded file on the SFTP check. Thinking through options here's what I have in mind:

Allow for the new optional sendfile parameter and honor GET/POST. If it is a GET request then use a zero byte helloworld.txt file as @MauriceReeves-usds mentioned in the comment above.
Create a new POST end point and move the common code into supporting functions in the class containing two distinct end point functions

Keeping as one might make it a bit easier and using the GET with the generic helloworld.txt for simple quick checks. And the POST could be used to test a specific file for troubleshooting.

I'd keep this really simple - its a small endpoint.  Why not only have the POST, and get rid of the GET?

I'm running all the tests and submitting a PR right now. I ended up keeping it in the same endpoint and conditionally upload the file based on how it is called.

@jimduff-usds we merged in #1115. We can probably put this into Review/QA if not done. Agreed?

Had forgotten to assign this to myself. Since it has been merged, do I close the ticket or will someone else move it along to Review/QA?

Can we close this or is further review needed.


[GSSOC '23]Performing Data Analytics on Uber Dataset.

I would like to perform data analysis on the Uber dataset and try to implement a machine-learning model. I would also like to point out that I would be using different types of charts to perform data analysis.

@karthikbhandary2 - we already have another issue on the same (#72). Hence, we won't be able to allot you this issue.

Can I at least make a machine learning model which runs on this dataset?


Release 4.7.12-0-1.3.1 (CI TEST)

CI for windows being problematic as usual.... Running in KVM of windows on local machine just fine though.

CI Passed for 4.7.12-0-1.3.1 on all platforms


Fixes warframe.market cookie status confusion between login / available

The recent JWT cookie length check for data sending prevented a login from completing which means the check will always fail as the full login cookie could never be set in a catch 22.
Considerations

I 100% just make up the number 300 out of looking at what the cookie length was when I loaded the page after deleing the cookie to sign myself out.  A total guess if this length is reasonable for all users or if the original not null check should still be used for "available" while 500 length should be used for "logged in"

The only places isJwtAvailable is called at is on starting search it and on setting WFM status. The problem appears to happen in both cases, so I don't really understand why you made search it still use the old and broken value of 500.
Can confirm 300 works for me, my JWT length was 389 for reference

It must've changed then, when I was testing my token was 600+ characters. Merging


Update markdown of auth package

Purpose
$subject
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Merging without travis since this is a documentation change.


ZOSVPC-3055- Give the option of select the image for a z/OS VSI.

Updated files to customize the image for z/OS VSI for both Quickstart & Standard Variations.
Also, worked on indentation of JSON & .tf files.

closing the pR


EfficientAD

Description
Hey,
this is the work in progress for implementing EfficientAD based on the implementations https://github.com/nelson1425/EfficientAD/blob/main/efficientad.py and https://github.com/rximg/EfficientAD/tree/main
Right now the performance doesn't meet the paper results but I'm on it :)

Fixes # (issue)

Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which refactors the code base)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] This change requires a documentation update

Checklist

[ ] My code follows the pre-commit style and check guidelines of this project.
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing tests pass locally with my changes
[ ] I have added a summary of my changes to the CHANGELOG (not for minor changes, docs and tests).

Wow @alexriedel1, this is great contribution! Thanks a lot!

Right now I'm searching for a way to iterate through the validation dataloader at the very last epoch and save the model via the PL Lightning ModelCheckpointer after that. Does anyone have a hint on how to do this?

Regarding your question, the quickest way to do this would be something like the following. (Haven't checked if it works, though. You might want to double check)
class EfficientAD(AnomalyModule):
    """PL Lightning Module for the EfficientAD algorithm."""
    ...

    def on_validation_epoch_end(self) -> None:
        if self.current_epoch == self.trainer.max_epochs:   # or `self.trainer.max_epochs - 1`? Need to check
            for i, data in enumerate(self.trainer.datamodule.val_dataloader()):
                print("You could save the model here.")

Right now I'm searching for a way to iterate through the validation dataloader at the very last epoch and save the model via the PL Lightning ModelCheckpointer after that. Does anyone have a hint on how to do this?

Regarding your question, the quickest way to do this would be something like the following. (Haven't checked if it works, though. You might want to double check)
class EfficientAD(AnomalyModule):
    """PL Lightning Module for the EfficientAD algorithm."""
    ...

    def on_validation_epoch_end(self) -> None:
        if self.current_epoch == self.trainer.max_epochs:   # or `self.trainer.max_epochs - 1`? Need to check
            for i, data in enumerate(self.trainer.datamodule.val_dataloader()):
                print("You could save the model here.")

Thanks! Yes I now went for on validation start, to have the quantiles calculated for the actual final validation
https://github.com/openvinotoolkit/anomalib/blob/2eb33bf7c33cf8439e12d630304f7f4318926f67/src/anomalib/models/efficientad/lightning_model.py#L165-L172

Yeah, this looks good!

Hi @alexriedel1, I have run @nelson1425's version and noticed that the Lhard loss in my version is decreasing more slowly than in nelson1425's. I have checked for differences between the two versions, but have not found any obvious differences. I suspect that the problem may be occurring during the process of distilling the student network.

Hi @alexriedel1, I have run @nelson1425's version and noticed that the Lhard loss in my version is decreasing more slowly than in nelson1425's. I have checked for differences between the two versions, but have not found any obvious differences. I suspect that the problem may be occurring during the process of distilling the student network.

In this pull request I have added both models and yes, the difference is caused by the different pre trained students

Hi @alexriedel1, I have fixed some bugs in gradient updating and model evaluation status, and updated the results of MVTec AD which closely match the official ones.https://github.com/rximg/EfficientAD/commit/03ecb3ef100b5186594b2ee4ce5c684ce9457291#diff-7a26d02d1b7f4bc9d8fee30e821584c68b76e21f0f0adec99b69c82f009f5c82

Hi @alexriedel1, is this ready for review? Cheers!

Hi @alexriedel1, is this ready for review? Cheers!
I'm doing some last performance tests and will tell you when it's done :)

One thing that bothers me is that EfficientAD needs to be trained with ImageNet images.  Thats why anomalib needs gdown as a package.
Do you know any way to upload this Imagenet subset to a safer and more consistent place than my personla Goole Drive?

Do you know any way to upload this Imagenet subset to a safer and more consistent place than my personal Google Drive? It's roughly 3GB

One possibility would be to publish it in the Releases similar to this. There could be some problems though

File size limit is 2GB (link)
License could also be an issue.

Do you know any way to upload this Imagenet subset to a safer and more consistent place than my personal Google Drive? It's roughly 3GB

One possibility would be to publish it in the Releases similar to this. There could be some problems though

File size limit is 2GB (link)
ImageNet License could also be an issue.

OK I see. I guess reducing the imagenet suibset to 2GB would be possible, about the license I'm not sure how to deal with it.
Anyway, the commit can be reviewed now.
I didn't run benchmarks for all MVTec AD categories though.
On the bottle class, the following results are obtained using the pre-trained model by @rximg
https://wandb.ai/alexried/mvtec/reports/image_AUROC-image_F1Score-pixel_AUROC-pixel_F1Score-23-05-12-17-12-27---Vmlldzo0MzQ0MzI0
The authors state that padding doesn't affect the performance but only speeds up training. However I realized that using a padded pre-trained model and removing padding in the actual EfficientAD training causes a drop in pixel F1 score.
As you can see in the training log, the model converges relatively fast and shows good performance. Maybe training for 70,000 epochs (which is 325 epochs on MVTec bottle) isn't even necessary.
The imagenet-pretrained teacher model is crucial for the performance. Using @rximg pre-trained model gave the best performance compared to some models trained by me using albumentation instead of torchvision transforms..

Would it be an idea to use fastai's Imagenette dataset, which is a subsample of the original ImageNet dataset.
https://github.com/fastai/imagenette
There is a direct link in the readme to download the dataset.

Contributor

Thanks @samet-akcay, imagenette works just as well :)
https://api.wandb.ai/links/alexried/v5k5ws5o
So, please have a look over the code :)

@alexriedel1, this is great contribution! I only have some minor comments here and there and a question:
Do you think this model would be ONNX friendly since the model graph might need to contain some image transforms using albumentations?

Hi! I tried to generate an ONNX model from this implementation of EfficientAD in anomalib and, actually, the answer is no. I don't know if it's an opset type issue, but the inference process (with CUDA provider) requires 0.5 seconds to process an image (clearly, something is wrong, the model should be very light and fast). Looking at the model's graph I can't see anything strange, the model is "simple" (teacher + student + autoencoder) and operations too (convolutions, subs, resizes).

@alexriedel1, this is great contribution! I only have some minor comments here and there and a question:
Do you think this model would be ONNX friendly since the model graph might need to contain some image transforms using albumentations?

Hi! I tried to generate an ONNX model from this implementation of EfficientAD in anomalib and, actually, the answer is no. I don't know if it's an opset type issue, but the inference process (with CUDA provider) requires 0.5 seconds to process an image (clearly, something is wrong, the model should be very light and fast). Looking at the model's graph I can't see anything strange, the model is "simple" (teacher + student + autoencoder) and operations too (convolutions, subs, resizes).

What timing do you expect on your machine? @samet-akcay the model graph doesn't actually need any transforms different from the other models pre-processing pipelines at inference time.

One thing that would be really valuable would be if someone is able to run all mvtec classes for a model benchmark.

@alexriedel1, this is great contribution! I only have some minor comments here and there and a question:
Do you think this model would be ONNX friendly since the model graph might need to contain some image transforms using albumentations?

Hi! I tried to generate an ONNX model from this implementation of EfficientAD in anomalib and, actually, the answer is no. I don't know if it's an opset type issue, but the inference process (with CUDA provider) requires 0.5 seconds to process an image (clearly, something is wrong, the model should be very light and fast). Looking at the model's graph I can't see anything strange, the model is "simple" (teacher + student + autoencoder) and operations too (convolutions, subs, resizes).

What timing do you expect on your machine? @samet-akcay the model graph doesn't actually need any transforms different from the other models pre-processing pipelines at inference time.

Solved, there was a problem with the kind of algorithm for the convolution's operation in the CUDAProvider for OnnxRuntime, setting "cudnn_conv_algo_search" to "DEFAULT" in the provider options solved the timing problem. Using a GTX 1060 I got 88ms of inference time for an image, which is a reasonable timing. It's not as the 5.3ms for the A100 as the paper but it's still a good result. I think the onnx model works well!

@alexriedel1, this is great contribution! I only have some minor comments here and there and a question:
Do you think this model would be ONNX friendly since the model graph might need to contain some image transforms using albumentations?

Hi! I tried to generate an ONNX model from this implementation of EfficientAD in anomalib and, actually, the answer is no. I don't know if it's an opset type issue, but the inference process (with CUDA provider) requires 0.5 seconds to process an image (clearly, something is wrong, the model should be very light and fast). Looking at the model's graph I can't see anything strange, the model is "simple" (teacher + student + autoencoder) and operations too (convolutions, subs, resizes).

What timing do you expect on your machine? @samet-akcay the model graph doesn't actually need any transforms different from the other models pre-processing pipelines at inference time.

Solved, there was a problem with the kind of algorithm for the convolution's operation in the CUDAProvider for OnnxRuntime, setting "cudnn_conv_algo_search" to "DEFAULT" in the provider options solved the timing problem. Using a GTX 1060 I got 88ms of inference time for an image, which is a reasonable timing. It's not as the 5.3ms for the A100 as the paper but it's still a good result. I think the onnx model works well!

Great! Do you also get reasonable anomaly detection Performance results?

@alexriedel1, this is great contribution! I only have some minor comments here and there and a question:
Do you think this model would be ONNX friendly since the model graph might need to contain some image transforms using albumentations?

Hi! I tried to generate an ONNX model from this implementation of EfficientAD in anomalib and, actually, the answer is no. I don't know if it's an opset type issue, but the inference process (with CUDA provider) requires 0.5 seconds to process an image (clearly, something is wrong, the model should be very light and fast). Looking at the model's graph I can't see anything strange, the model is "simple" (teacher + student + autoencoder) and operations too (convolutions, subs, resizes).

What timing do you expect on your machine? @samet-akcay the model graph doesn't actually need any transforms different from the other models pre-processing pipelines at inference time.

Solved, there was a problem with the kind of algorithm for the convolution's operation in the CUDAProvider for OnnxRuntime, setting "cudnn_conv_algo_search" to "DEFAULT" in the provider options solved the timing problem. Using a GTX 1060 I got 88ms of inference time for an image, which is a reasonable timing. It's not as the 5.3ms for the A100 as the paper but it's still a good result. I think the onnx model works well!

Great! Do you also get reasonable anomaly detection Performance results?

Unfortunately I used a custom dataset, therefore I haven't a global reference of the anomaly detection's performance for the model. Compared to other models I got good results with my dataset in anomaly detection, I will test the MVTec's dataset.

@alexriedel1, this is great contribution! I only have some minor comments here and there and a question:
Do you think this model would be ONNX friendly since the model graph might need to contain some image transforms using albumentations?

Hi! I tried to generate an ONNX model from this implementation of EfficientAD in anomalib and, actually, the answer is no. I don't know if it's an opset type issue, but the inference process (with CUDA provider) requires 0.5 seconds to process an image (clearly, something is wrong, the model should be very light and fast). Looking at the model's graph I can't see anything strange, the model is "simple" (teacher + student + autoencoder) and operations too (convolutions, subs, resizes).

What timing do you expect on your machine? @samet-akcay the model graph doesn't actually need any transforms different from the other models pre-processing pipelines at inference time.

Solved, there was a problem with the kind of algorithm for the convolution's operation in the CUDAProvider for OnnxRuntime, setting "cudnn_conv_algo_search" to "DEFAULT" in the provider options solved the timing problem. Using a GTX 1060 I got 88ms of inference time for an image, which is a reasonable timing. It's not as the 5.3ms for the A100 as the paper but it's still a good result. I think the onnx model works well!

Great! Do you also get reasonable anomaly detection Performance results?

Unfortunately I used a custom dataset, therefore I haven't a global reference of the anomaly detection's performance for the model. Compared to other models I got good results with my dataset in anomaly detection, I will test the MVTec's dataset.

Ok great, just wanted to make sure that nothing is lost during onnx conversion and the models works in principle

I have tried this using Notebooks section, I got an error with 001_getting_started notebook.

`---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[14], line 5
2 config.optimization.export_mode = "openvino"
4 # Get the model and callbacks
----> 5 model = get_model(config)
6 callbacks = get_callbacks(config)
File ~\Intel\tests\anomalib_effad_env\lib\site-packages\anomalib\models_init_.py:106, in get_model(config)
103     model = getattr(module, f"{_snake_to_pascal_case(config.model.name)}Lightning")(config)
105 else:
--> 106     raise ValueError(f"Unknown model {config.model.name}!")
108 if "init_weights" in config.keys() and config.init_weights:
109     model.load_state_dict(load(os.path.join(config.project.path, config.init_weights))["state_dict"], strict=False)
ValueError: Unknown model efficientad!`

I have tried this using Notebooks section and also OpenVINO model. I got an error with 001_getting_started notebook.
`--------------------------------------------------------------------------- ValueError Traceback (most recent call last) Cell In[14], line 5 2 config.optimization.export_mode = "openvino" 4 # Get the model and callbacks ----> 5 model = get_model(config) 6 callbacks = get_callbacks(config)
File ~\Intel\tests\anomalib_effad_env\lib\site-packages\anomalib\models__init__.py:106, in get_model(config) 103 model = getattr(module, f"{_snake_to_pascal_case(config.model.name)}Lightning")(config) 105 else: --> 106 raise ValueError(f"Unknown model {config.model.name}!") 108 if "init_weights" in config.keys() and config.init_weights: 109 model.load_state_dict(load(os.path.join(config.project.path, config.init_weights))["state_dict"], strict=False)
ValueError: Unknown model efficientad!`

Guess you didn't pull from this PR? I have no issues loading the model

I have tried this using Notebooks section and also OpenVINO model. I got an error with 001_getting_started notebook.
--------------------------------------------------------------------------- ValueError Traceback (most recent call last) Cell In[14], line 5 2 config.optimization.export_mode = "openvino" 4 # Get the model and callbacks ----> 5 model = get_model(config) 6 callbacks = get_callbacks(config) File ~\Intel\tests\anomalib_effad_env\lib\site-packages\anomalib\models__init__.py:106, in get_model(config) 103 model = getattr(module, f"{_snake_to_pascal_case(config.model.name)}Lightning")(config) 105 else: --> 106 raise ValueError(f"Unknown model {config.model.name}!") 108 if "init_weights" in config.keys() and config.init_weights: 109 model.load_state_dict(load(os.path.join(config.project.path, config.init_weights))["state_dict"], strict=False) ValueError: Unknown model efficientad!

Guess you didn't pull the right branch? I have no issues loading the model

Yes, It is working installing from your source files. Thanks

I have trained a teacher model using albumentation transform and it's giving even better results than the previous model. I added the model weights and removed the old ones. Everything should be in place now and ready for merging. What about adding the model to tests etc.?

@alexriedel1 very well done thank you for your work! Can you enter the Mvtec AD results in the tables in https://github.com/openvinotoolkit/anomalib/blob/main/README.md ?

@alexriedel1 very well done thank you for your work! Can you enter the Mvtec AD results in the tables in https://github.com/openvinotoolkit/anomalib/blob/main/README.md ?

Hey, I dont have the ressources to run the anomalib implementation on all mvtec AD classes so I cannot add the results for this implementation of EfficientAD. If there is someone here who could do a benchmark (https://github.com/openvinotoolkit/anomalib/tree/main/tools/benchmarking) with EfficientAD that would be great

@alexriedel1 very well done thank you for your work! Can you enter the Mvtec AD results in the tables in https://github.com/openvinotoolkit/anomalib/blob/main/README.md ?

Hey, I dont have the ressources to run the anomalib implementation on all mvtec AD classes so I cannot add the results for this implementation of EfficientAD. If there is someone here who could do a benchmark (https://github.com/openvinotoolkit/anomalib/tree/main/tools/benchmarking) with EfficientAD that would be great

I'll try to run a benchmark this week

Im having a hard time getting the precommit hooks prettier and markdownlint to run on my windows machine ☹ I will give it another try though today

Im having a hard time getting the precommit hooks prettier and markdownlint to run on my windows machine ☹ I will give it another try though today

We had this feedback from someone else too. Wasn't aware that pre-commit wasn't working well on a windows machine. Or maybe our configuration is problematic. We might need to investigate.
Meanwhile I could run the checks on your PR

@alexriedel1 @samet-akcay
I saw that you used the https://github.com/nelson1425/EfficientAD code to improve it
What I want to know is:

Did you use pretraining.py in nelson1425 to train the teacher weights?
Then the training code in anomalib is equivalent to the efficientad.py training code in nelson1425, right?
If I use the teacher weights trained by pretraining.py in nelson1425, can they be used in anomalib?
I only used 200 positive samples of my own data to train efficientad in anomalib. The classification effect was image_AUROC 70% and image_F1Score 60%.
If segmentation training is used, pixel_AUROC is 70% and pixel_F1Score is 30%. Is there any way to improve the above?
Also, when the above classification model is tested, the defect intensity of the heat map of the abnormal data is not obvious or the performance is not obvious. Is there any way to improve it?
Although the segmentation model can reflect the heat map defects of abnormal data, it is often segmented incorrectly?


Backlight module icon missing

I have the latest waybar running on Arch. I have otf-font-awesome installed. When I look at the default config file, I see all icons except the ones for backlight module. Icon next to backlight percentage is missing in the bar. Any suggestions as to what if any other fonts I'm missing?

@J0nnyMak0 do you have ttf-font-awesome installed too? How does your waybar config look like? (the backlight part)

@J0nnyMak0 do you have ttf-font-awesome installed too? How does your waybar config look like? (the backlight part)

Yes, I have ttf-font awesome installed. I'm using the default config file. In there, brightness icons display as a bunch of empty boxes like so:
 "format-icons": ["", "", "", "", "", "", "", "", ""]

@J0nnyMak0 I don't know if these icons use font-awesome....I personally just use two sun emoji's like this:
"format-icons": ["🔅", "🔆"]
If you want to choose one from font-awesome there isn't much to choose from....
paid version: https://fontawesome.com/search?q=brightness&o=r
free version: https://fontawesome.com/search?q=brightness&o=r&m=free

I see. Ok, thanks. I'll use those then.

I was able to get those icons with ttf-meslo-nerd-font-powerlevel10k, available in the AUR or from here: https://github.com/romkatv/powerlevel10k-media
However, I just ended up reverting the brightness icons to those from this commit: https://github.com/Alexays/Waybar/pull/873/commits/2273ade3590edb572ba234e8c2d6bf3e0da3bb9b
Perhaps a note about compatible fonts should be added to the wiki? I also still have some issues with certain glyphs, like "hands-free": "", despite having the complete set of Noto fonts, font-awesome, etc. installed. I think it might be worth considering changing some of these very obscure glyphs which aren't included in most fonts.

What package for ubuntu?
I installed fonts-font-awesome but no brightness icon

The font packages you need are Font Awesome and Meslo LG.
Using arch you can install ttf-meslo-nerd and ttf-font-awesome from the official repo and everything works.
@mlazzarotto I don't know why this font is not packaged for ubuntu but you can follow this to install them: https://github.com/romkatv/powerlevel10k#meslo-nerd-font-patched-for-powerlevel10k


No direct method (Lcom/google/android/exoplayer2/upstream/DefaultAllocator;IIJJ)V in class

I am facing issue regarding exoplayer, App is crashing on start
Versions
react-native: "0.49.0"
react-native-track-player: devBranch
java.lang.NoSuchMethodError: No direct method <init>(Lcom/google/android/exoplayer2/upstream/DefaultAllocator;IIJJ)V in class Lcom/google/android/exoplayer2/DefaultLoadControl; or its super classes (declaration of 'com.google.android.exoplayer2.DefaultLoadControl' appears in /data/app/com.test.myapp-OHj1RziNXZi6KAifrwaU0w==/base.apk)
        at guichaguri.trackplayer.player.players.ExoPlayback.<init>(ExoPlayback.java:62)
        at guichaguri.trackplayer.logic.MediaManager.createLocalPlayback(MediaManager.java:65)
        at guichaguri.trackplayer.logic.MediaManager.setupPlayer(MediaManager.java:78)
        at guichaguri.trackplayer.logic.components.MediaWrapper.setupPlayer(MediaWrapper.java:36)
        at guichaguri.trackplayer.TrackModule$1.run(TrackModule.java:143)
        at android.os.Handler.handleCallback(Handler.java:790)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:164)
        at android.app.ActivityThread.main(ActivityThread.java:6494)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:438)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:807)

My Project dependency
dependencies {
    compile project(':react-native-brightcove-player')
    compile project(':react-native-image-picker')
    compile project(':react-native-track-player')
    compile project(':react-native-text-input-mask')
    compile project(':react-native-share')
    compile project(':react-native-sound')
    compile project(':react-native-pdf')
    compile project(':react-native-onesignal')
    compile project(':appcenter-crashes')
    compile project(':appcenter-analytics')
    compile project(':appcenter')
    compile project(':react-native-fetch-blob')
    compile project(':tipsi-stripe')
    compile project(':react-native-payments')
    compile project(':react-native-fs')
    compile project(':react-native-blur')
    compile project(':react-native-picker')
    compile project(':react-native-device-info')
    compile project(':react-native-extra-dimensions-android')
    compile project(':react-native-immersive')
    compile project(':react-native-orientation')
    compile project(':react-native-splash-screen')
    compile project(':react-native-config')
    compile project(':react-native-code-push')
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile 'com.android.support:appcompat-v7:26.1.0'
    compile 'com.android.support:support-v4:26.1.0'
    compile "com.facebook.react:react-native:+"  // From node_modules
    compile project(':react-native-linear-gradient')
    compile project(':react-native-webview-bridge')
}

It might be a conflict with react-native-sound. Do you really need both modules?

@Guichaguri
Thanks for response,
I had removed react-native-sound and tried again but it has same issue, i think react-native-brightcove-player is conflicting because i had found exo player dependency in it's build.gradle
here is 'react-native-brightcove-player build.gradle'
// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:2.2.3'
    }
}

allprojects {
    repositories {
        jcenter()
        maven {
            url "$projectDir/../node_modules/react-native/android"
        }
        maven {
            url 'http://repo.brightcove.com/releases'
        }
    }
}

apply plugin: 'com.android.library'

android {
    compileSdkVersion 23
    buildToolsVersion "23.0.1"

    defaultConfig {
        minSdkVersion 16
        targetSdkVersion 22
        versionCode 1
        versionName "1.0"
        ndk {
            abiFilters "armeabi-v7a", "x86"
        }
    }
    lintOptions {
        warning 'InvalidPackage'
    }
}

dependencies {
    compile 'com.facebook.react:react-native:+'
    compile 'com.brightcove.player:exoplayer2:6.+'
}

After implementing react-native-brightcove-player i had face this issue and it is start giving exo error in react-native-track-player. Can you provide suggestion how can i fix that.

Facing the same issue here, I am using react-native-video "^3.1.0" which depends on 'com.google.android.exoplayer:exoplayer:2.7.3'.

You can remove the conflict by unifying the ExoPlayer version, read this: https://github.com/react-native-kit/react-native-track-player/pull/165#issuecomment-407914722

Indeed. That's how I solved the issue, just a heads up there are some deprecation warnings and a few changes you'll have to make to ExoPlayer.java to use an updated version of Exoplayer (change a few method signatures). They don't seem to be altering the player but I am still unsure.


Simplifying ApplicationLifetime.ExecuteHandlers()?

Unless I'm missing something, it seems that the allocation of List<Exception> exceptions is unnecessary (that code cannot generate more than one exception).
The whole method can be simplified as:
private void ExecuteHandlers(CancellationTokenSource cancel)
{
    // Noop if this is already cancelled
    if (cancel.IsCancellationRequested)
        return;
    // Run the cancellation token callbacks
    cancel.Cancel(false);
}

@KPixel yeah I believe you are right. Feel free to make a pull request if you would like!

Yes, this is a remnant of some old refactoring by @davidfowl where IHostedServices were embedded in ApplicationLifetime.


[Cleanup] Client-side

Mandatory user stories:

[X] Should have mocks
[ ] Client should adhere to mocks
[ ] Should use Flexbox

Other things:
Bill's initial mocks
Open questions:

things to do:

login page styling (buttons, background, logo),
homescreen (buttons, background),
navigation bar (i.e. do we want a navbar like instagram where you can navigate to pages from the bottom?, assigning a “title” at the top of each scene in the navbar, etc.)


The new messageID generated with #74 is a bit too long by default. Th…

…is patch reduces the length

Codecov Report

Merging #78 (d327ca7) into main (dd0e3a9) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
- Coverage   83.85%   83.63%   -0.22%
==========================================
  Files          13       13
  Lines        1381     1381
==========================================
- Hits         1158     1155       -3
- Misses        153      155       +2
- Partials       70       71       +1

Impacted Files
Coverage Δ

msg.go
86.52% <100.00%> (ø)

client.go
73.58% <0.00%> (-0.81%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Problem with scikit 0.24

There is a problem with the  version 0.24 of scikit: No module named 'sklearn.metrics.scorer'

There is a problem with the version 0.24 of scikit: No module named 'sklearn.metrics.scorer'

I‘m having the same issue. But I tried to uninstall sklearn 0.24 and install an older version, still get the same error’

The issue arises because the check_scoring function has moved from sklearn.metrics.scorer.check_scoring to sklearn.metrics.check_scoring in the most recent version of scikit learn (ref).
I fixed the issue manually by going into the library's cv.py file which can be accessed via the following path:
C:\_<project directory>_\venv\Lib\site-packages\evolutionary_search\cv.py
Note: If you use an Anaconda environment or your base system environment, the path will be different.
Then in that file, change line 13 from:
from sklearn.metrics.scorer import check_scoring
to
from sklearn.metrics import check_scoring
Then that should fix the issue 👍

Hey @anselmo-parnada thanks for checking this! I will commit a fix when I have some time

Issue is solved in sklearn-deap==0.3.0


Being asked to re-authenticate when using an alias

Redacting

@amondnet please delete this issue if you can


documentation is missing permission for search_projects

There was an internal bug filed (b/246937743) to show that search_projects is missing permissions for folks to be able to run the code as needed. The documentation seems to hint at requiring "both" permissions, but it currently only lists resourcemanager.projects.get. Could you take a look to see where this is missing?

Filed https://issuetracker.google.com/issues/278353656 in the public issue tracker for the Resource Manager API.


ACM-2859 Cluster actions refactor

https://issues.redhat.com/browse/ACM-2859
Before, hibernating clusters are eligible for bulk detach

After, hibernating clusters are not eligible for bulk detach

/cc @jeswanke @Randy424


Improvement: Add method setUserAvatar to UserApi

This works for me
	public User setUserAvatar(User user, File avatarFile)
			throws GitLabApiException
	{
		Response response = putUpload(Response.Status.OK, "avatar", avatarFile,
				"users", user.getId());
		return (response.readEntity(User.class));
	}

@vpreclik
This has been added and is available in release 4.9.18, please verify and close when you have a chance.

@vpreclik
I'm closing this since the functionality has been added and released.  Feel free to re-open if you find any issues with the implementation.


Barcode/QR scanner integration

research available packages to hook/intercept barcode scanner results

So throughout my research, I learned two things. One, the barcode scanner acts like a keyboard which is just inputs. Two is that it is possible to intercept barcode scans if it's connected to the Ipad and needs information to the raspberry pi.
So first, if information needed to be scanned from the barcode to the IPAD then the result of the barcode is just a string of keyboard inputs. So I was thinking just having the results of the barcode given with a string is just needed. Then having code correlate the string  with the database and returning what is needed on a specific page. Then once having checked out it would have code again and return changes to the database on what needed to be removed.
There is one youtube video that uses python to be able to do this and could follow through how they did it. Then also below is the github code that we could also go by off and learn.
https://www.youtube.com/watch?v=zU7YVKnwTdo
https://github.com/vpatron/barcode_scanner_python
I do think that most of the code would be once scanning the barcode. Then it would most likely send a request to the database and return the name of the product. Which then displays what the product is. Then once they check out, it will show how many items were scanned and see how many need to be removed from inventory. Or if they want to add stuff into inventory with the barcode, they could use the barcode scanner just to scan the string and store it.
Other than that I do not think a library is needed for it but there are youtube videos explaining more about it. And basic functionalities of it with code as well I believe. It also tests with their own database in which we could possibly just adjust the code to work with ours.
https://www.youtube.com/watch?v=ywakseuir-o
https://piddlerintheroot.com/barcode-scanner/#google_vignett
Another thing I realized was that the barcode works with bluetooth as well as a cord. However, In the case they do want to work with the bluetooth on the tablets, and if needed access from the raspberry pi. This would work on intercepting bluetooth signals.
https://www.youtube.com/watch?v=dsZN0dqh81k
https://ubertooth.sourceforge.net/hardware/one/
https://www.kismetwireless.net/


Making the README.md a little more accurate

Hi, I noticed you say the provided urls are reset_password/, /reset_password/confirm/ and /reset_password/validate_token but I believe the reset_password bit is not necessary and can be a little confusing, so tried to clean up a little bit the README

That is indeed more acurate. Thanks :+1:


Mac arm64 release (and possibly also x64 release) not notarized and cannot be opened

Describe the issue
After downloading the latest NWB-GUIDE-arm64.dmg file, running it, dragging the NWB GUIDE icon to Applications, and then running the NWB GUIDE app from Applications, I get this error:

The latest Mac release build passed but appears to have had an error during notarization.
SyntaxError: Unexpected token E in JSON at position 0
    at JSON.parse (<anonymous>)
    at /Users/runner/work/nwb-guide/nwb-guide/node_modules/@electron/notarize/src/notarytool.ts:79:25
    at Generator.next (<anonymous>)
    at fulfilled (/Users/runner/work/nwb-guide/nwb-guide/node_modules/@electron/notarize/lib/notarytool.js:28:58)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

I will investigate now but could use some help
Steps to Reproduce
See above.
Operating System
Mac M1
GUIDE Version
0.0.14
Code of Conduct
Yes
Did you confirm this issue was not already reported?
Yes

My best guess is that I had not agreed to the latest Apple Developer Program License Agreement, and so when the CI tried notarize NWB GUIDE 0.0.14, the message ERROR: You must first sign the relevant contracts online. (1048) was returned (ref). This string does not parse as JSON - the error message starts with E. As a result, notarization fails.
I have since signed the latest Apple agreement, and so now I am re-running that release workflow to see if that fixed the issue. We'll see in an hour.

Re-running the release workflow failed to upload/publish the new files to the existing 0.0.14 release because it can only publish to draft releases, so I converted the release to draft mode (which deleted that release and created a new one), deleted the assets, and re-ran both the windows and mac release workflows from the 0.0.14 release (so the release does not contain any commits after 0.0.14). The workflows added new assets to the draft release, and I published a new 0.0.14 release. Installing the NWB-GUIDE-arm64.dmg file from this release worked!


profile: reorder custom profile fields

Fixed Issue #28511
profile: rename custom profile field types
Before:

After:

@haroon10725 please don't open multiple PRs for the same issue -- GitHub supports updating an existing PR.
Check out our GitHub guide and commit guidelines for more details.
Closing in favor of https://github.com/zulip/zulip/pull/28558 for this issue.


Add old-omz-completion plugin

Provides a simple way to use the old omz completion
https://github.com/robbyrussell/oh-my-zsh/issues/5560

Has no interest.


Is there a way to use this as an external tool in pycharm

It would be so much easier to use this as an external tool for pycharm

I have no plans for that right now, but if anyone want to go and write a PyCharm plugin, I will gladly accept pull requests.


Raise NoMethodError in case of absence of ActionMailer.

Steps to reproduce

Comment in config/application.rb require "action_mailer/railtie".
Run rails generate activejob:schedule

Actual results
/home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/railtie/configuration.rb:95:in `method_missing': undefined method `action_mailer' for #<Rails::Application::Configuration:0x00000003f1b460> (NoMethodError)
	from /home/zekefast/development/project/job/brightgrove/icon_peak/dashboard/backend/config/environments/development.rb:17:in `block in <top (required)>'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/railtie.rb:210:in `instance_eval'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/railtie.rb:210:in `configure'
	from /home/zekefast/development/project/job/brightgrove/icon_peak/dashboard/backend/config/environments/development.rb:1:in `<top (required)>'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:274:in `require'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:274:in `block in require'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:240:in `load_dependency'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/activesupport-4.2.1/lib/active_support/dependencies.rb:274:in `require'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/engine.rb:598:in `block (2 levels) in <class:Engine>'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/engine.rb:597:in `each'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/engine.rb:597:in `block in <class:Engine>'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/initializable.rb:30:in `instance_exec'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/initializable.rb:30:in `run'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/initializable.rb:55:in `block in run_initializers'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:226:in `block in tsort_each'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:348:in `block (2 levels) in each_strongly_connected_component'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:420:in `block (2 levels) in each_strongly_connected_component_from'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:429:in `each_strongly_connected_component_from'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:419:in `block in each_strongly_connected_component_from'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/initializable.rb:44:in `each'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/initializable.rb:44:in `tsort_each_child'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:413:in `call'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:413:in `each_strongly_connected_component_from'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:347:in `block in each_strongly_connected_component'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:345:in `each'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:345:in `call'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:345:in `each_strongly_connected_component'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:224:in `tsort_each'
	from /home/zekefast/.rvm/rubies/ruby-2.2.2/lib/ruby/2.2.0/tsort.rb:203:in `tsort_each'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/initializable.rb:54:in `run_initializers'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/application.rb:352:in `initialize!'
	from /home/zekefast/development/project/job/brightgrove/icon_peak/dashboard/backend/config/environment.rb:5:in `<top (required)>'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/application.rb:328:in `require'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/application.rb:328:in `require_environment!'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/commands/commands_tasks.rb:142:in `require_application_and_environment!'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/commands/commands_tasks.rb:128:in `generate_or_destroy'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/commands/commands_tasks.rb:50:in `generate'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/commands/commands_tasks.rb:39:in `run_command!'
	from /home/zekefast/.rvm/gems/ruby-2.2.2@icon_peak_-_dashboard/gems/railties-4.2.1/lib/rails/commands.rb:17:in `<top (required)>'
	from bin/rails:4:in `require'
	from bin/rails:4:in `<main>'

Expected results

Degrade gracefully in case of absence of Rails components.

ActiveJob::Scheduler doesn't use ActionMailer at all, this has to do with your Rails configuration in config/development.rb. When you exclude components of Rails, you have to go through your configuration and comment out or remove the settings pertaining to those files. Same thing if you removed Sprockets or ActionController, except you'd hit a different line for the same error.

Thanks.

Sorry it took so long! :)

T

On Mon, May 18, 2015 at 5:38 PM, Zeke Fast<EMAIL_ADDRESS>wrote:

Thanks.
Reply to this email directly or view it on GitHub:
https://github.com/tubbo/activejob-scheduler/issues/3#issuecomment-103220379