[WIP] Converter for DataValidationErrors

What does the pull request do?
Adds a converter attached property to control how DataValidation messages are shown
What is the current behavior?
No easy way to modiy the displayed messages.
What is the updated/expected behavior with this PR?
Easy to use converter attached
How was the solution implemented (if it's not obvious)?
See new sample page
Checklist

[ ] Added unit tests (if possible)?                                          :TODO:
[ ] Added XML documentation to any related classes?     :TODO:
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation                             :TODO:

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0034392-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

For reference https://github.com/AvaloniaUI/Avalonia/pull/11234#issuecomment-1535426553

You can test this PR using the following package version. 11.0.999-cibuild0034549-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0037161-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0038250-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0039060-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@maxkatz6 I tried your changes and I think these are a useful addition. Now I need to write some docs before 11.1 is released.
Thx ❤

@maxkatz6 we have a failing test here: https://github.com/AvaloniaUI/Avalonia/blob/e40d5d624cb6e3eb54743b4046d69ebe69e56b5a/tests/Avalonia.Controls.UnitTests/TextBoxTests_DataValidation.cs#L43-L65
Do you know how to solve it?


Location for Schema files

For some data provision options, we have to create additional schemas (e.g. extending the Coverage data models as required, here we need to provide XSD files). We need a place to provide them.
Based on discussions, we'll set up GitHub pages for this purpose, based off the main FAIRiCUBE Repo

Text for base page:
For some data provision options, additional schema files are required (e.g. when extending the Coverage data models as required, XSD files are required to document the extensions). These are provided here
Please provide a subfolder for Coverage XSDs, then we can create other subfolders as required

Hi @KathiSchleidt,
I have just created the repository Schemas available at the link https://github.com/FAIRiCUBE/Schemas. The GitHub pages are enabled and available at the link https://fairicube.github.io/Schemas/ .

@cozzolinoac11 Thanks! Works!
MD-LinkSchema.xsd

@Mohinem @bangph exciting news as you can see: we have a schema repo - pls have a look and then mail me directly so that we discuss next steps.

@KathiSchleidt Likely the metadata reference listed in the repo schema is still under work, and this below will be replaced:
<documentation>-- Definition -- Monitored environmental medium.</documentation>

Thanks for finding that cut&paste error, fixed.

@cozzolinoac11 Something seems to have gone wrong with the GitHub pages, https://fairicube.github.io/Schemas/ no longer resolves
Could you please take a look at this?

Hi @KathiSchleidt, thanks for finding the error, fixed.

@cozzolinoac11 Thanks!!!

looks nice now, can we close it?

@pebau if you look closely, you will see that this issue was closed a while back (May 9th). The schema for the metadata link I provided is available there.
Still missing schemas for the proprietary aspects of rasdaman metadata, when can we expect this?


Find channel in Netty 4.0.32

How to get  target channel from ChannelGroup in Netty 4.0.32? I want to write message to target channel.
I have to implement ConcurrentHashMap to store Channel with generate uuid key? Have some good solution?

@demonhunterg please use stack overflow for questions. This is for bugs only.


Allow disabling colors in log output

In Kibana ansi color codes are not handled so they introduce random
characters to the logging output. With this flag we can turn colors of
in staging but keep them when running locally.
Test Plan
cargo run --bin orderbook has colors
cargo run --bin orderbook-- --colored-output false` does not

Thanks for linking atty. I think I actually like removing the option and always doing it automatically more so that we have less command line arguments.


Pine(A)64 Debian MusicBot Image (16GB or higher cards ONLY)

I made a Debian image for the Pine(A)64 with MusicBot 1.9.5_2, SSH, and FTP support already installed.
This will help for people that are too lazy or scared to compile everything.
Windows Installation: Use WinRAR or 7zip to extract the RAR file, then use Win32DiskImager write to the microSD card.
Mac/Linux Installation: Use tar (or something else) and use dd to write the image onto the microSD card.
Google Drive RAR Download

We've written our guides, including the Debian one, to be more friendly. We strongly encourage compiling Python and required dependencies manually.


feat: add staking migration

Add staking migration in prepare for opengov release.
This migration is clearing existing staking votes if there are any left.

no longer relevant


Create message_zh.yml

add zh_cn translate

Thank you for this translation! I will work on implementing multi-language support into the plugin 🙂

Thank you for this translation! I will work on implementing multi-language support into the plugin 🙂

nevermind😆


Fixed #28377 -- Retain media asset order while merging.

Merge two media file lists while keeping the order of the elements.
Previously elements form the second list where append if they are not
already in the first list. As the result the order of assets could
be inverted.
This patch fixed this issue and ensure a consistent order or assets
unless there are two lists in a reversed order. Those cases will now
raise a warning.
Ref https://code.djangoproject.com/ticket/14370
Ref #6385
This changes is required to avoid disabling jQuery.noConflict in
future Django versions, see https://github.com/django/django/pull/6385#issuecomment-306452627

@apollo13 I need this fix to continue my work on #6385
I figured it's better to do it in a separate PR.
It will allow me to use the official release of Select2 and at the same time keep noConflict(true).
@timgraham let me know if you think this deserves release notes :)
thx guys!

I think this is pretty clever, though it seems a little magical. Maybe it would make sense to document your merging behavior here with an example?
https://docs.djangoproject.com/en/dev/topics/forms/media/#combining-media-objects
I think it should have a release note. Django didn't really define what order scripts would get added, but I would assume that newer scripts would be added to the end, so it's a little unintuitive behavior change, and it could cause problems on existing websites.

Absolutely, docs would certainly help.

but I would assume that newer scripts would be added to the end, so it's a little unintuitive behavior change, and it could cause problems on existing websites.

Newer scripts generally get added to the end, at least as long as the do not have the same name as existing ones…

@apollo13 thanks, yes you are absolutely right. It still is amending unless an element is preexisting. Than the curser moves to in front of that point.
I added some documentation. Already big thanks for all your feedback!

Looks good to me, @timgraham any thoughts on this one? (aside from the merge conflicts ;))

@felixxm thx for the review, very detailed, very helpful! Thanks mate 👍
@apollo13 rebased :)
@timgraham any final words? 😉

@timgraham thanks a lot for the review. I tried to resolve all you comments and added a new ticket djangoproject#28377.

Looks good to me so far, will merge after my latest nitpicks are addressed (sorry for the delay).

I can make those edits when I do a detailed docs review.

👍 -- the only downside I can find with the patch is that the runtime (literally "time") behavior changes, but I do not think that we include so many media files that this should be an issue.

@apollo13 yes, time could change depending how in and index are implemented as well as append and insert but big O should remain O(n, m) = m. I think we can live with that ;)

@timgraham so we're ready for checkin? woooooohooooo 🎉  😃
makes my weekend!

@timgraham thx! Looks good to me :)
I would leave the copy in, but it's an unrelated change. I can open a separate ticket for that. What do you think?

If there's a use case where copy is needed, sure, a separate ticket seems fine. Could you debug the selenium failures at your convenience?

@timgraham sure, where do I find the selenium failures? All the tests I see have passed 🤔

https://docs.djangoproject.com/en/dev/internals/contributing/writing-code/unit-tests/#running-the-selenium-tests
Use Chrome as Firefox isn't reliable since they introduced geckodriver.

Thanks @timgraham, I found the issue. Both AdminDateWidget and AdminTimeWidget use DateTimeShortcuts.js. This script does require jQuery, but it was missing from the media-js list.
It's fixed now, all selenium tests are passing.
BTW, it might be good to have chrome headless running on CI.

@codingjoe: The last change looks weird, the previous code would work without explicitly including jquery -- where did it come from before? And why is it needed now? This would be a backwards incompatible change for custom admin widgets.

@apollo13 it only worked because the ModelAdmin includes jQuery in it's media class as well, see https://github.com/django/django/blob/master/django/contrib/admin/options.py#L583-L584
Let me explain why it broke only now, so you can make up your mind, if that is bad or right:
It did use to work, because the previous implementation did not care about the order and simply amended new asset paths. So we where lucky it worked.
The widgets did not work outside of the admin, this was certainly a bug.
But, the question regarding custom fields remains valid. If we have cases where the assets are not proper defined, others might two and they don't have the tests that cover the behavior. Luckily the tests didn't fail because of JavaScrip errors but because of the warning I added. I believe this will help other people to safely update too.
@apollo13: I can more information to the release notes, but I still believe this patch is valid. What do you think could help to alert people further about the changes?

@codingjoe I'll have to test this, but from a naive (ie mine) or any enduser standpoint I'd expect that (even with the ordering in fact) that first media of admin classes is included and then the one of the forms.

Just tested it and with your latest patch applied the user admin page in the admin fails because SelectFilter2.js cannot find $ either (probably requires the same fix). I'll see if we can get around without redefining jquery.js

Just for reference, are:
======================================================================
ERROR: test_populate_existing_object (admin_views.tests.SeleniumTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/florian/sources/django.git/tests/admin_views/tests.py", line 4122, in test_populate_existing_object
    self.selenium.find_element_by_id('id_name').send_keys(' the best')
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/webdriver.py", line 289, in find_element_by_id
    return self.find_element(by=By.ID, value=id_)
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/webdriver.py", line 791, in find_element
    'value': value})['value']
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/webdriver.py", line 256, in execute
    self.error_handler.check_response(response)
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/errorhandler.py", line 194, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"id","selector":"id_name"}
  (Session info: chrome=59.0.3071.104)
  (Driver info: chromedriver=2.30.477691 (6ee44a7247c639c0703f291d320bdf05c1531b57),platform=Linux 4.11.0-1-amd64 x86_64)

======================================================================
ERROR: test_prepopulated_fields (admin_views.tests.SeleniumTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/florian/sources/django.git/tests/admin_views/tests.py", line 4001, in test_prepopulated_fields
    self.selenium.find_element_by_id('id_pubdate').send_keys('2012-02-18')
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/webdriver.py", line 289, in find_element_by_id
    return self.find_element(by=By.ID, value=id_)
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/webdriver.py", line 791, in find_element
    'value': value})['value']
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/webdriver.py", line 256, in execute
    self.error_handler.check_response(response)
  File "/home/florian/.local/lib/python3.5/site-packages/selenium/webdriver/remote/errorhandler.py", line 194, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"id","selector":"id_pubdate"}
  (Session info: chrome=59.0.3071.104)
  (Driver info: chromedriver=2.30.477691 (6ee44a7247c639c0703f291d320bdf05c1531b57),platform=Linux 4.11.0-1-amd64 x86_64)

the failing selenium tests?

Yes, runtime warnings are converted to exceptions so the 500 exception page is displayed.

Ok, I think I understand the issue now. The whole issue occurs because the inline admin also includes jquery.js even though it isn't exactly usable without the normal admin anyways. I think the fix is okayish and the fallout is not too much.
But I'd prefer:
diff --git a/django/contrib/admin/widgets.py b/django/contrib/admin/widgets.py
index 4b932f2c01..823f2973f7 100644
--- a/django/contrib/admin/widgets.py
+++ b/django/contrib/admin/widgets.py
@@ -22,7 +22,7 @@ class FilteredSelectMultiple(forms.SelectMultiple):
     """
     @property
     def media(self):
-        js = ["core.js", "SelectBox.js", "SelectFilter2.js"]
+        js = ["core.js", "jquery.init.js", "SelectBox.js", "SelectFilter2.js"]
         return forms.Media(js=["admin/js/%s" % path for path in js])

     def __init__(self, verbose_name, is_stacked, attrs=None, choices=()):
@@ -43,7 +43,7 @@ class FilteredSelectMultiple(forms.SelectMultiple):
 class AdminDateWidget(forms.DateInput):
     @property
     def media(self):
-        js = ["calendar.js", "admin/DateTimeShortcuts.js"]
+        js = ["jquery.init.js", "calendar.js", "admin/DateTimeShortcuts.js"]
         return forms.Media(js=["admin/js/%s" % path for path in js])

     def __init__(self, attrs=None, format=None):
@@ -56,7 +56,7 @@ class AdminDateWidget(forms.DateInput):
 class AdminTimeWidget(forms.TimeInput):
     @property
     def media(self):
-        js = ["calendar.js", "admin/DateTimeShortcuts.js"]
+        js = ["jquery.init.js", "calendar.js", "admin/DateTimeShortcuts.js"]
         return forms.Media(js=["admin/js/%s" % path for path in js])

     def __init__(self, attrs=None, format=None):

over the last commit (note that I also have a fix for SelectBox in there which was missing). So instead of specifying indirect dependencies like jquery I just specify direct dependencies core and jquery.init (which provides django.jquery) and let the admin take care of the rest.

@codingjoe I'll have to test this, but from a naive (ie mine) or any enduser standpoint I'd expect that (even with the ordering in effect) that first media of admin classes is included and then the one of the forms.
I agree. I also fixed that for form sets. The behavior seems to be as you described for the admin, I also did not touch it, see https://github.com/django/django/blob/master/django/contrib/admin/options.py#L1475-L1479

The reason why this failed, is because the inline admin for those two tests included widgets, that had jQuery in the assets and some didn't.
Haha, while writing this, it becomes more and more clear to me how unreliable the old implementation was. Honestly I can't not fully tell you things where ordered prior to this patch, but at least now I can now 😆
Now it retains the order and if there is a conflict you get a warning and the "super" class wins.

I realize that this is slightly backwards incompatible, but contrib has a relaxed policy anyways and the fix is easy enough…

The reason why this failed, is because the inline admin for those two tests included widgets, that had jQuery in the assets and some didn't.

I do not think that was the case (widgets afaik never had jquery in there), but the inline admin itself has it…

@apollo13 I agree :)

@codingjoe If you could apply my last patch instead of yours and selenium works then for you we can ship this unless Tim has further objections. :shipit:

@apollo13 thx, I missed the FilteredSelectMultiple widget. I do not agree with adding jquery.init only tho. It implicitly relies on jQuery be loaded by some other entity and that this entity also has jquery.init in it and the right order.
I believe widgets should be atomic and not know or rely on any form, admin or inline admin for that matter. I think SoC is important here, since people might have they own admin implementations.

Meh, I am mostly fine with either approach, in the end the widgets are probably only usable in the admin anyways (I think). Let's leave it as you have it now then…

I'm having a error with the python manage.py startapp command. It creates
the folder and everything fine but it is missing the apps.py file.
Any ideas?
On Sun, Jul 16, 2017 at 1:24 PM, Florian Apolloner<EMAIL_ADDRESS>
wrote:

Meh, I am mostly fine with either approach, in the end the widgets are
probably only usable in the admin anyways (I think). Let's leave it as you
have it now then…
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/8667#issuecomment-315623621, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ab0QRkOfvLKi-RVcu_B-E7b3ZOKcZbXbks5sOkcqgaJpZM4OA9EJ
.

@alewish2192 works for me:
$ django-admin startapp mytest
$ ls -lah mytest/
total 40
drwxr-xr-x   9 johannes  staff   306B Jul 16 19:45 .
drwxr-xr-x  31 johannes  staff   1.0K Jul 16 19:45 ..
-rw-r--r--   1 johannes  staff     0B Jul 16 19:45 __init__.py
-rw-r--r--   1 johannes  staff    63B Jul 16 19:45 admin.py
-rw-r--r--   1 johannes  staff    87B Jul 16 19:45 apps.py
drwxr-xr-x   3 johannes  staff   102B Jul 16 19:45 migrations
-rw-r--r--   1 johannes  staff    57B Jul 16 19:45 models.py
-rw-r--r--   1 johannes  staff    60B Jul 16 19:45 tests.py
-rw-r--r--   1 johannes  staff    63B Jul 16 19:45 views.py

drew@drew-Inspiron-3541:~/mysite$ python manage.py startapp exampletest
drew@drew-Inspiron-3541:~/mysite$ ls -lah exampletest/
total 28K
drwxrwxr-x 3 drew drew 4.0K Jul 16 13:17 .
drwxrwxr-x 6 drew drew 4.0K Jul 16 13:17 ..
-rw-r--r-- 1 drew drew   63 Jul 16 13:17 admin.py
-rw-r--r-- 1 drew drew    0 Jul 16 13:17 init.py
drwxrwxr-x 2 drew drew 4.0K Jul 16 13:17 migrations
-rw-r--r-- 1 drew drew   57 Jul 16 13:17 models.py
-rw-r--r-- 1 drew drew   60 Jul 16 13:17 tests.py
-rw-r--r-- 1 drew drew   63 Jul 16 13:17 views.py
whats the code inside apps.py? I can just manual enter it for now until I
get it figured out
On Sun, Jul 16, 2017 at 1:47 PM, Johannes Hoppe<EMAIL_ADDRESS>wrote:

@alewish2192 works for me:
$ django-admin startapp mytest
$ ls -lah mytest/
total 40
drwxr-xr-x   9 johannes  staff   306B Jul 16 19:45 .
drwxr-xr-x  31 johannes  staff   1.0K Jul 16 19:45 ..
-rw-r--r--   1 johannes  staff     0B Jul 16 19:45 init.py
-rw-r--r--   1 johannes  staff    63B Jul 16 19:45 admin.py
-rw-r--r--   1 johannes  staff    87B Jul 16 19:45 apps.py
drwxr-xr-x   3 johannes  staff   102B Jul 16 19:45 migrations
-rw-r--r--   1 johannes  staff    57B Jul 16 19:45 models.py
-rw-r--r--   1 johannes  staff    60B Jul 16 19:45 tests.py
-rw-r--r--   1 johannes  staff    63B Jul 16 19:45 views.py
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/8667#issuecomment-315624910, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ab0QRtadj_fQodgaGK160u18s2XBd-noks5sOky6gaJpZM4OA9EJ
.

@alewish2196 This seems to be completely off-topic for this issue?!

@alewish2196 I think you might be responding to the wrong thread.
This one in fact: https://github.com/django/django/pull/8667
Please seek help in our IRC channel and make sure to read our getting stated guide: https://www.djangoproject.com/start/

The last commit introduced selenium test failures. (The reason we don't run the selenium tests on all PRs is because they don't run reliably on Jenkins and they are unnecessary for most patches.)

@timgraham thanks, it was a missing trailing comma ;)
I also made the script order consistent thought the admin. I have no more objections 👍

buildbot, test on selenium

Buildbot, test on selenium.

Wohoo 🎉 🍍 🥇 thanks everyone 👍

I've found this to be problematic if the admin has ['jquery.js'], and the form has ['use-jquery-from-anywhere.js', 'jquery.js', 'calendar.js'], because this will currently result in ['use-jquery-from-anywhere.js', 'jquery.js', 'calendar.js'].
See #9743 for a possible fix (keeping media from admin and forms separate).


initial commit of Parse-DbaCsvMetadata

This was previously called  Get-CsvMetadata
Started synopsis, (still not complete...)
advanced command BEGIN, PROCESS, END added
pscustomobject is now in use
added bits and bobs... little tidy up
Verbose output sort of completed.
Few debug checkpoints available.
Includes some level of error handling.
E.g:

returns PS object with vital metadata which then can be parsed.
E.g.:

You're trying to add a file to functions and the root of the module. Remove the file for your cmdlet from the root of the module then commit to the fr3ky/dbatools/development branch again, which will update this PR.

Hey @barticsoka,
Are there any plans to make the required changes within the next month or two? If not, I'll close this PR and you can resubmit.

Till next time!


Multiple area generation

On Location creation, allow the creator to specify multiple areas to be generated.
This would enable a creator to have the players tped to different areas far apart from each other without having to generate all of the terrain in between.

This is low priority


Stack and CI coverage for some modules

Do you feel like adding Stack configurations to the project, or is this extra maintenance burden?  This PR provides Stack configurations for

codeworld-account
codeworld-api
codeworld-auth
codeworld-compiler
codeworld-error-sanitizer
codeworld-game-api
codeworld-game-server
codeworld-prediction
codeworld-server
a root project which combines these

stack build works for all of these; stack test works for all but codeworld-compiler and the root project.  The passing tests don't seem to depend on ghcjs being available on the path (although it is).
I've had no success with compiling codeworld-base and funblocks-client with Stack.  Maybe if this is resolved: https://github.com/ghcjs/ghcjs/issues/745
stack exec codeworld-server seems to work, but I've only looked at the home page.  Again, I don't know if this depends on ghcjs on the path.
Intero seems to work with these Stack configurations.
It also has a YAML for Travis-CI.  Tests for codeworld-compiler and the root project are disabled.  If google/codeworld has Travis configured, it may be necessary to disable this https://github.com/peterbecich/codeworld/blob/13338d676e84eab26a2311891558364b95205a5c/.travis.yml#L55 for one CI build to allow stack build enough time to complete and cache without exceeding 50 minutes.
I'm expecting these to pass: https://travis-ci.org/peterbecich/codeworld/builds/598007572

An easy improvement I could make to this is a common file for extra-deps, to avoid duplication.

It would be good to add a note to the README clarifying that the stack.yaml file is there to support use of intero and travis, and that it cannot be used to build and run CodeWorld.

Sorry for my late response.
As for the failure of the codeworld-compiler tests on Stack, I'm not sure if this is the error you are describing

because you're getting errors that the modules aren't exporting program?

missingEqualSign: [Failed]
missingEqualSign
expected: "program.hs:16:1-9: error:\n    Parse error: An equation was expected here.\n    Are you missing the equal sign?"
 but got: ": Loaded package environment from /tmp/stack19221/test-ghc-env\n<command line>: cannot satisfy -hide-package codeworld-api\n    (use -v for more information) (no loc)"

The other failures are all similar to this one.
         Test Cases   Total
 Passed  11           11
 Failed  33           33

If we can get stack test codeworld-compiler to succeed locally I'm sure it could work on Travis in the future, assuming the 50 minute timeout is enough to set up GHCJS.

This looks valuable as-is.  Thanks!  I'll go ahead and merge it, and any further work on getting more of the project to build can happen in new PRs.


Artifact upload fails on what appears to be an extra step

Expected Behavior
When using S3/GCS bucket storage to persist output artifacts in a task, it should execute in the correct order and return successfully.
Actual Behavior
The injected steps to facilitate the upload do not seem to be waiting for the previous step(s) to complete. Specifically, the problem is with what appears to be an extra "upload" container/step. I'm not certain if that step is even necessary, given it doesn't get the BOTO_CONFIG env var added, but it's failing before the previous steps finish.
Steps to Reproduce the Problem

Create a PipelineRun using the following manifest:

---
apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:
  generateName: steps-
  namespace: tekton-pipelines
spec:
  resources:
    - name: os-info
      resourceSpec:
        type: storage
        params:
          - name: location
            value: s3://some-bucket/archive.zip
          - name: type
            value: gcs
  pipelineSpec:
    resources:
    - name: os-info
      type: storage
    tasks:
    - name: step-1
      taskSpec:
        resources:
          outputs:
          - name: os-info
            type: storage
        steps:
        - name: get-os-info
          image: alpine:3.13.1
          script: |
            #!/bin/sh
            echo "hello world"
            cat /etc/os-release
            cat /etc/os-release > /workspace/output/os-info/content
            sleep 60
      resources:
        outputs:
        - name: os-info
          resource: os-info
    - name: step-2
      resources:
        inputs:
        - name: os-info
          resource: os-info
          from: [ step-1 ]
      taskSpec:
        resources:
          inputs:
          - name: os-info
            type: storage
        steps:
        - name: print-os-info
          image: alpine:3.13.1
          script: |
            #!/bin/sh
            echo "hello world"
            sleep 60

kubectl -n tekton-pipelines create -f <above-yaml-file>
Watching the pipeline run in the Tekton Dashboard shows the last step of the first task hits an error. Additionally, I'm not certain what the difference is between the "artifact-copy-to-" and "upload-" steps. See following screenshots and pod outputs:

...
- args:
    - -wait_file
    - /tekton/tools/1
    - -post_file
    - /tekton/tools/2
    - -termination_path
    - /tekton/termination
    - -entrypoint
    - gsutil
    - --
    - cp
    - -P
    - -r
    - /workspace/output/os-info
    - s3://some-bucket/steps-h8rl7-tekton-pipelines-bucket/step-1/os-info
    command:
    - /tekton/tools/entrypoint
    env:
    - name: BOTO_CONFIG
      value: /var/bucketsecret/tekton-storage/boto-config
    image: gcr.io/google.com/cloudsdktool/cloud-sdk@sha256:27b2c22bf259d9bc1a291e99c63791ba0c27a04d2db0a43241ba0f1f20f4067f
    imagePullPolicy: IfNotPresent
    name: step-artifact-copy-to-os-info-c5kd8
    resources:
      requests:
        cpu: "0"
        ephemeral-storage: "0"
        memory: "0"
    terminationMessagePath: /tekton/termination
    terminationMessagePolicy: File
    volumeMounts:
    - mountPath: /var/bucketsecret/tekton-storage
      name: volume-bucket-tekton-storage
    - mountPath: /tekton/tools
      name: tekton-internal-tools
    - mountPath: /tekton/creds
      name: tekton-creds-init-home-v86cj
    - mountPath: /workspace
      name: tekton-internal-workspace
    - mountPath: /tekton/home
      name: tekton-internal-home
    - mountPath: /tekton/results
      name: tekton-internal-results
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-2sgwz
      readOnly: true
    workingDir: /workspace
  - args:
    - -wait_file
    - /tekton/tools/2
    - -post_file
    - /tekton/tools/3
    - -termination_path
    - /tekton/termination
    - -entrypoint
    - gsutil
    - --
    - cp
    - /workspace/output/os-info/*
    - s3://some-bucket/archive.zip
    command:
    - /tekton/tools/entrypoint
    image: gcr.io/google.com/cloudsdktool/cloud-sdk@sha256:27b2c22bf259d9bc1a291e99c63791ba0c27a04d2db0a43241ba0f1f20f4067f
    imagePullPolicy: IfNotPresent
    name: step-upload-os-info-hbh2r
    resources:
      requests:
        cpu: "0"
        ephemeral-storage: "0"
        memory: "0"
    terminationMessagePath: /tekton/termination
    terminationMessagePolicy: File
    volumeMounts:
    - mountPath: /tekton/tools
      name: tekton-internal-tools
    - mountPath: /tekton/creds
      name: tekton-creds-init-home-ck4fw
    - mountPath: /workspace
      name: tekton-internal-workspace
    - mountPath: /tekton/home
      name: tekton-internal-home
    - mountPath: /tekton/results
      name: tekton-internal-results
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-2sgwz
      readOnly: true
    workingDir: /workspace

Additional Info

Kubernetes version:

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.3", GitCommit:"1e11e4a2108024935ecfcb2912226cedeafd99df", GitTreeState:"clean", BuildDate:"2020-10-14T12:50:19Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"17+", GitVersion:"v1.17.12-eks-7684af", GitCommit:"7684af4ac41370dd109ac13817023cb8063e3d45", GitTreeState:"clean", BuildDate:"2020-10-20T22:57:40Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}

Tekton Pipeline version:

$ kubectl get pods -n tekton-pipelines -l app=tekton-pipelines-controller -o=jsonpath='{.items[0].metadata.labels.version}'
v0.20.1

@nwang92 did you manage to resolve this issue or does it remain a problem for you?

One alternative here would be to use catalog tasks + workspaces instead of relying on pipeline resources.

I haven't had a chance to look at it recently or use any of the newer releases to see if it's mitigated. I think using disk storage/workspaces works but I couldn't get object store to work for S3. Since there are other options here, I'm fine with closing this unless there's others hitting the same issue.


Port to Pandas 2.0

Describe the bug
when running gandlf_run, I am encountering:
ERROR: 'DataFrame' object has no attribute 'append'
To Reproduce
Train a model using gandlf_run.
I trained using 2d_rad_segmentation data from https://upenn.box.com/shared/static/y8162xkq1zz5555ye3pwadry2m2e39bs.zip and the config file from samples in the repo config_classification.yaml
Additional context

check the changelog of pandas here
The training runs successfully when downgrading pandas to 1.5.3

Yes, if nothing goes wrong the PR goes tomorrow


setState() or markNeedsBuild() called during build.

So a little bit of history before deep into this error.
I'm introducing the use of CustomScrollView and Slivers due to a previous error. That error was weird because was throwing inside a SingleChildScrollView when scrolling down the nth child (15 +-). That child had a rounder border with this Widget extension method which works okay:
Widget rounded({
    double radius = 50,
    BoxBorder? border,
    bool borderActive = true,
    bool active = true,
    Clip clipBehavior = Clip.antiAlias,
    BorderRadius? borderRadius,
  }) =>
      active
          ? Container(
              clipBehavior: clipBehavior,
              decoration: BoxDecoration(
                borderRadius: borderRadius ?? BorderRadius.circular(radius),
                border: borderActive ? border : null,
              ),
              child: this,
            )
          : this;

... so I stopped using it and the error gone out. I marked it as follows in the specific widget:
[...]
// Weird assertion bug
// https://stackoverflow.com/questions/64109444/error-assertion-failed-debugduringdeviceupdate-is-not-true-flutter
// https://github.com/flutter/flutter/issues/84241#issuecomment-1236344314
.rounded(
  radius: 15,
  active: false,
)

So, I decided to try with slivers and seems to work okay with 30 children, even using my rounded corners... Everything is okay until I put them to be autodisposed when no visible on the scroll area, by putting addAutomaticKeepAlives to false:
@override
  Widget buildContent(
    BuildContext context,
    WidgetHandle use,
    BoxConstraints parentConstraints,
  ) {
    final lang = use(languageCapsule);
    final posts = use(postsWrapperCapsule);

    final scrollController = use.disposable(
      ScrollController.new,
      (controller) => controller.dispose(),
    );

    // Trigger posts loading
    use.callonceAsync(posts.load);

    return CustomScrollView(
      controller: scrollController,
      slivers: [
        // 1 (Main)
        SliverList.list(
          children: [
             // Fixed widgets
             [...]
          ],
        ),

        // 2 (Posts)
        SliverList.list(
          addAutomaticKeepAlives: false,
          children: [
            // Post title
            _PostsTitle(
              lang: lang,
            ).marginBottom(20),

            // Posts
            ...posts.posts.map(
              (post) => post.build().withMargin(
                    const EdgeInsets.symmetric(horizontal: 40, vertical: 30),
                  ),
            ),

            // Posts loading indicator
            if (posts.loading())
              const CircularProgressIndicator().marginBottom(20),
          ],
        ),
      ],
    );
  }

Then this error appears when (I suppose) it tries to dispose the first post widget when scrolling down and it is not visible:
Post 0 -- Init
Post 1 -- Init
Post 2 -- Init
Post 3 -- Init
Post 4 -- Init
Post 5 -- Init
══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY
╞═════════════════════════════════════════════════════════
The following assertion was thrown during performLayout():
setState() or markNeedsBuild() called during build.
This TextPostWidget widget cannot be marked as needing to
build because the framework is already in
the process of building widgets. A widget can be marked as
needing to be built during the build
phase only if one of its ancestors is currently building.
This exception is allowed because the
framework builds parent widgets before children, which means
a dirty descendant will always be
built. Otherwise, the framework might not visit this widget
during this build phase.
The widget on which setState() or markNeedsBuild() was called
was:
  TextPostWidget-[<'0'>]
The widget which was currently being built when the offending
call was made was:
  SliverList

As you can see, post widgets are built lazily. I suspect this error is related to a patch you did here
PostWidget
mixin PostWidget on RearchConsumer, Named, DefaultMsg {
  @override
  String get named => post.named;

  Post get post;

  Alignment get contentAlignment => Alignment.topLeft;

  @override
  Widget build(BuildContext context, WidgetHandle use) {
    final lang = use(languageCapsule);

    use.callonce(() => print('$named -- Init'));
    use.onDispose(() => print('$named -- Dispose'));

    use.automaticKeepAlive();

    return LayoutBuilder(
      builder: (context, constraints) {
        final maxHeight = constraints.maxHeight * 0.85;
        final double minHeight = min(maxHeight, 200);

        return Builder(
          builder: (context) {
            return RearchBuilder(
              builder: (context, use) {
                return Column(
                  children: [
                    // Title
                    _buildTitle(lang),

                    // Divider
                    const Divider(
                      height: 10,
                      thickness: 2,
                      color: Color.fromARGB(255, 117, 117, 117),
                    ),

                    // Content
                    Align(
                      alignment: contentAlignment,
                      child: buildContent(context, use, constraints),
                    ),
                  ],
                )
                    .padding(
                      const EdgeInsets.symmetric(
                        horizontal: 20,
                        vertical: 10,
                      ),
                    )
                    .bgColor(
                      const Color.fromARGB(255, 189, 171, 171),
                    )
                    .withSizeConstraints(
                      BoxConstraints(
                        minWidth: min(
                          700,
                          constraints.maxWidth * 0.85,
                        ),
                        minHeight: minHeight,
                        maxHeight: maxHeight,
                      ),
                    )

                    // Weird assertion bug
                    // https://stackoverflow.com/questions/64109444/error-assertion-failed-debugduringdeviceupdate-is-not-true-flutter
                    // https://github.com/flutter/flutter/issues/84241#issuecomment-1236344314
                    .roundedCrownBorder(
                      color: const Color.fromARGB(255, 117, 117, 117),
                      radius: 15,
                      active: true,
                    )
                    .rounded(
                      radius: 15,
                      active: false,
                    );
              },
            );
          },
        );
      },
    );
  }

  Widget buildContent(
    BuildContext context,
    WidgetHandle use,
    BoxConstraints parentConstraints,
  );

  Widget _buildTitle(Language lang) {
    final post = this.post;
    final createdAt = post.createdAt;

    return Column(
      mainAxisAlignment: MainAxisAlignment.center,
      mainAxisSize: MainAxisSize.min,
      children: [
        // Title
        Label(
          post.title,
          color: Colors.black,
          fontSize: 20,
        ),

        // Creation datetime
        (Label(
          createdAt.getAsHumanReadable(
            hideSeconds: true,
            separator: ' - ',
          ),
          color: Colors.black,
          fontSize: 12,
        ) as Widget)
            .tooltip(
          createdAt.timeZoneName,
          waitDuration: const Duration(milliseconds: 350),
        ),
      ],
    );
  }
}

I'm sorry. Seems my error. I forgot to remove this on PostWidget:
use.automaticKeepAlive();


Add collate estimates to README + add download link.

Looks like the collated estimates for the UK are missing from the README. I have been looking at getting the persistent download link to share with users but so far been unable to understand the API sufficently to make this happen. Any ideas @joeHickson - users were still on the latest_summary.csv that is no longer being updated.

Hi @seabbs
When you open a file in dataverse the file ID is in the url

You then pop the id in the url the others use:

that should give you a download link to the latest version of that file id in the original upload  file format.


Azure CLI:  –data-disk-delete-option : Data disk is not deleted automatically after the VM is deleted

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
Run the below in Azure Cloud Shell to create a VM
az vm create -n Delete_CLI1 -g Delete_CLI --image RedHat:RHEL:7-RAW:7.4.2018010506 -l northeurope --size Standard_E8as_v4 --generate-ssh-keys --public-ip-address "" --os-disk-size-gb 64 --data-disk-sizes-gb 200 --data-disk-delete-option
As per https://docs.microsoft.com/en-us/cli/azure/vm?view=azure-cli-latest when using --data-disk-delete-option, data disk should be deleted or detached upon VM deletion.
After I go and delete VM, the data disk is not deleted automatically
To Reproduce

Open Azure Cloud Shell
Type command to create VM using the --data-disk-delete-option
VM gets created
When deleting VM, attached data disk is not deleted automatically
Expected behavior
Data Disk should be deleted automatically

Compute


Update missing txt file for TestRunCompareTestScript

Description
Update missing txt file for TestRunCompareTestScript.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I assume this was missing in the backports. Thanks


Illegal instruction on Android Virtual Device running on Github Action Runner

Hello,
im working on an issue in the Android/Termux CI from uutils project.
The issue: It's currently only reproducable in our CI.
On my private PC I can't reproduce the issue.
After adding more logs and several CI test-runs, I managed to trace it down to the half-rs crate:
#[test]
fn test_f16c_f16_to_f32_direct() {
    let bo = ::od::byteorder_io::ByteOrder::Little;
    let bits = bo.read_u16(&[0x00, 0x3c]);

    let result_f16 = half::f16::from_bits(bits);
    let result = f64::from(result_f16); // crashes here
    assert_eq!(1.0, result);
}

The function in half-rs responsible for converting f16 to f64 is this:
#[inline]
pub(crate) fn f16_to_f64(i: u16) -> f64 {
    convert_fn! {
        if x86_feature("f16c") {
            unsafe { x86::f16_to_f32_x86_f16c(i) as f64 }   // crashes here
        } else if aarch64_feature("fp16") {
            unsafe { aarch64::f16_to_f64_fp16(i) }
        } else {
            f16_to_f64_fallback(i)
        }
    }
}

I tried to trace it down even further, by extracting relevant code  into a own test.
This test fails with SIG 4. Which is "illegal instruction".
#[test]
fn test_f16c_direct() {

    #[cfg(target_arch = "x86_64")]
    use std::arch::x86_64::{_mm_cvtph_ps, __m128i, __m128};
    #[cfg(target_arch = "x86")]
    use std::arch::x86::{_mm_cvtph_ps, __m128i, __m128};

    let i = 0u16;

    let result: f32 = unsafe {
        let mut vec = std::mem::MaybeUninit::<__m128i>::zeroed();
        vec.as_mut_ptr().cast::<u16>().write(i);
        let retval = _mm_cvtph_ps(vec.assume_init());
        *(&retval as *const __m128).cast()
    };

    assert_eq!(0.0, result);
}

#[test]
fn test_if_f16c_is_detected() {

    let detected = std::is_x86_feature_detected!("f16c");
    if detected {
        println!("f16c detected!");
    } else {
        println!("f16c not detected!");
    }

    assert_eq!(true, detected);
}

According to the last test (test_if_f16c_is_detected) f16c is supportted.
Also the /proc/cpuinfo states that f16c is supported.
I can't explain why its crashing then.
I have the feeling that its really a bug in the android emulator.
It states that the f16c is supported, but actually fails to execute it.
Do you have experience with such kind of error?
What can I do ro reproduce it locally?
Is it related to the actual CPU? The GitHub action runner has a AMD EPYC 7763 64-Core Processor.
My machine has Intel I7.
Is the github runner running in a own VM and can the setup of VM in VM cause this issue?
We are using the reactivecircus/android-emulator-runner for creating the android virtual device.
Any hints or clues?
Is there an other way to test if the f16c instruction set is supported?
Such that we can create an error report for google?

This definitely seems like either an android runner VM or an LLVM issue. The code here is doing f16c instructions, so either the VM is falsely reporting support, or LLVM is falsely detecting support. I don't know enough about the android emulator to know how to detect which is the problem. I'm not sure if you can use linux command line tools like lscpu to find it out or not. As far as I can tell, the processors you listed should have f16c support.

Thanks for your response. I will add the lscpu command as you suggested to see. But I don't expect that it shows something much differnt from the cat /proc/cpuinfo that I tried already.
Do you know if there is some tool available that is capable to not only show the reported features of the CPU but also is able to test if the features are really supported?
Maybe this way we could prove if this is only an issue with rust/llvm, or really an issue with the emulated CPU.

lscpu on android virtual device on the runner. the flags contain the term "f16c".
[2024-02-27 16:20:34] information about AVD:
[2024-02-27 16:20:34] lscpu:
[2024-02-27 16:20:34] Architecture:             x86_64
[2024-02-27 16:20:34] CPU op-mode(s):           32-bit, 64-bit
[2024-02-27 16:20:34] Address sizes:            40 bits physical, 48 bits virtual
[2024-02-27 16:20:34] Byte Order:               Little Endian
[2024-02-27 16:20:34] CPU(s):                   4
[2024-02-27 16:20:34] On-line CPU(s) list:      0-3
[2024-02-27 16:20:34] Vendor ID:                AuthenticAMD
[2024-02-27 16:20:34] Model name:               Android virtual processor
[2024-02-27 16:20:34] CPU family:               6
[2024-02-27 16:20:34] Model:                    6
[2024-02-27 16:20:34] Thread(s) per core:       1
[2024-02-27 16:20:34] Core(s) per socket:       4
[2024-02-27 16:20:34] Socket(s):                1
[2024-02-27 16:20:34] Stepping:                 3
[2024-02-27 16:20:34] BogoMIPS:                 4890.79
[2024-02-27 16:20:34] Flags:                    fpu de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx lm nopl pni pclmulqdq ssse3 cx16 sse4_1 sse4_2 x2apic popcnt tsc_deadline_timer aes avx f16c hypervisor lahf_lm cmp_legacy abm 3dnowprefetch rsb_ctxsw retpoline retpoline_amd vmmcall
[2024-02-27 16:20:34] Hypervisor vendor:        KVM
[2024-02-27 16:20:34] Virtualization type:      full
[2024-02-27 16:20:34] L1d cache:                256 KiB (4 instances)
[2024-02-27 16:20:34] L1i cache:                256 KiB (4 instances)
[2024-02-27 16:20:34] L2 cache:                 2 MiB (4 instances)
[2024-02-27 16:20:34] L3 cache:                 16 MiB (1 instance)
[2024-02-27 16:20:34] Vulnerability Meltdown:   Not affected
[2024-02-27 16:20:34] Vulnerability Spectre v1: Mitigation; __user pointer sanitization
[2024-02-27 16:20:34] Vulnerability Spectre v2: Vulnerable: Minimal AMD ASM retpoline

I found a solution by using a newer AVD system issue.
Image with the issue:

"default", API 28 (Linux localhost 4.4.124+ #1 SMP PREEMPT Mon Jun 18 17:10:07 UTC 2018 x86_64 Android)

Next image w/o the issue:

"google_apis_playstore", API 28 (Linux localhost 4.4.124+ #1 SMP PREEMPT Wed Jan 30 07:13:09 UTC 2019 i686 Android)


Node join ip replace regex error

Hello
I get this error when trying to run the playbook
fatal: [192.168.1.xxx]: FAILED! => {"changed": false, "module_stderr": "Shared connection to 192.168.1.xxx closed.
", "module_stdout": "
Traceback (most recent call last):
  File \"/home/anatol/.ansible/tmp/ansible-tmp-1606683277.6747334-186173704336525/AnsiballZ_replace.py\", line 102, in <module>
    _ansiballz_main()
  File \"/home/anatol/.ansible/tmp/ansible-tmp-1606683277.6747334-186173704336525/AnsiballZ_replace.py\", line 94, in _ansiballz_main
    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)
  File \"/home/anatol/.ansible/tmp/ansible-tmp-1606683277.6747334-186173704336525/AnsiballZ_replace.py\", line 40, in invoke_module
    runpy.run_module(mod_name='ansible.modules.files.replace', init_globals=None, run_name='__main__', alter_sys=True)
  File \"/usr/lib/python3.8/runpy.py\", line 207, in run_module
    return _run_module_code(code, init_globals, run_name, mod_spec)
  File \"/usr/lib/python3.8/runpy.py\", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File \"/usr/lib/python3.8/runpy.py\", line 87, in _run_code
    exec(code, run_globals)
  File \"/tmp/ansible_replace_payload_mh52eo44/ansible_replace_payload.zip/ansible/modules/files/replace.py\", line 302, in <module>
  File \"/tmp/ansible_replace_payload_mh52eo44/ansible_replace_payload.zip/ansible/modules/files/replace.py\", line 272, in main
  File \"/usr/lib/python3.8/re.py\", line 221, in subn
    return _compile(pattern, flags).subn(repl, string, count)
  File \"/usr/lib/python3.8/re.py\", line 327, in _subx
    template = _compile_repl(template, pattern)
  File \"/usr/lib/python3.8/re.py\", line 318, in _compile_repl
    return sre_parse.parse_template(repl, pattern)
  File \"/usr/lib/python3.8/sre_parse.py\", line 1036, in parse_template
    addgroup(int(this[1:]), len(this) - 1)
  File \"/usr/lib/python3.8/sre_parse.py\", line 980, in addgroup
    raise s.error(\"invalid group reference %d\" % index, pos)
re.error: invalid group reference 21 at position 3
", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

System,software and tools versions:
Ubuntu: 20.04 Focal
Python: 3.8.5
Ansible: 2.9.6 (installed with pip)
Kubeadm-playbook: 1.9.4

Python code throwing the error (/usr/lib/python3.8/sre_parse.py\", line 980)
def parse_template(source, state):
    # parse 're' replacement string into list of literals and
    # group references
    s = Tokenizer(source)
    sget = s.get
    groups = []
    literals = []
    literal = []
    lappend = literal.append
    def addgroup(index, pos):
        if index > state.groups:
            raise s.error("invalid group reference %d" % index, pos)
        if literal:
            literals.append(''.join(literal))
            del literal[:]
        groups.append((len(literals), index))
        literals.append(None)
    groupindex = state.groupindex
...

Playbook task and regex code
- name: replace master api server address to {{ master_name }} in the /etc/kubernetes/kubelet.conf
  replace:
    dest: /etc/kubernetes/kubelet.conf
    regexp: '(\s+)(server: https:\/\/)[A-Za-z0-9\-\.]+:'
    replace: '\1\2{{ master_name }}:'
    #backup: yes
  #when: proxy_env is defined and master is defined with fqdn in the inventory file (e.g. master.example.com)
  tags:
  - init
  notify:
  - Restart kubelet

Unfortunately my python regex skills aren't good enough to fix it in the short time I have right now. But I will update with a solution if I find one,
Thank you for your work and for making it open source!

Strange...
Indeed most of my deployments are on older versions of ansible&python.
A quick fix is to simply comment out this replace task if you don't use a proxy (aka you are not in a corporate env which needs proxy to reach the internet).

It actually does replace the server ip value, or at least it is the correct one before the step errors out.I did replace the task with a simple Ansible pause, change/verify it manually, and then proceed further with the process.
The rest of the playbook worked without any errors, except some in the addons.yml.
I had to comment out all, except the calico cni and ingress-nginx  and install the rest manually (), but that is mostly because of the continuous updates and the constantly changing ecosystem of both helm, and the subsequent charts.
Anyway, I got it to work with minor effort, and it works great.
Thank you very much , it helped me a lot (also maybe consider having a donate/buy a coffee/donate cryptocurrency section, some people might use it ;) )
If you have any questions or clarifications please write, otherwise I will close this issue.

(for completness)
Solution (temporary fix)

Comment out the task actions

 replace:
    dest: /etc/kubernetes/kubelet.conf
    regexp: '(\s+)(server: https:\/\/)[A-Za-z0-9\-\.]+:'
    replace: '\1\2{{ master_name }}:'

Add new task to pause the playbook,

pause:
    prompt: Please confirm ...!  Press enter/return to continue. Press Ctrl+c and then "a" to abort

3 . From different terminal do the task manually (change the server ip in /etc/kubernetes/kubelet.conf)
4. Press Enter to resume the playbook

It actually does replace the server ip value, or at least it is the correct one before the step errors out.I did replace the task with a simple Ansible pause, change/verify it manually, and then proceed further with the process.
This is very strange... I guess is something related to ansible&python versions.
The rest of the playbook worked without any errors, except some in the addons.yml.
I had to comment out all, except the calico cni and ingress-nginx  and install the rest manually (), but that is mostly because of the continuous updates and the constantly changing ecosystem of both helm, and the subsequent charts.
Did you take the latest version? I verified and updated all the charts and repos.
If you have the logs, let me know which ones. Again -> please confirm it was latest version (master or release 1.19)
Anyway, I got it to work with minor effort, and it works great.
Thank you very much , it helped me a lot (also maybe consider having a donate/buy a coffee/donate cryptocurrency section, some people might use it ;) )
If you have any questions or clarifications please write, otherwise I will close this issue.

JFYI it happens when you have defined [primary-master] section in hosts file as ip address, not fqdn.
Maybe there is need to add some validation on it.


Experiment with migrating e2e tests from webdriverio to Puppeteer

Puppeteer is an api on top of headless Chrome that suppose to have excellent performance characteristics and stability.
We can use it together with jest https://github.com/smooth-code/jest-puppeteer

Tests were migrated to TestCaffe


fix moveto doc for dls

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Transfer out, change to use uuid instead of putting personnel informa…

@BuchiNy check it out

@BuchiNy check it out

alright


Holes and Defects on the side of the Print

Hello Sielu-Rooster,
I have a AnyCubic I3 Mega and for the most part it prints pretty flawlessly.
I am currently trying to print this SteamDeck Guard and having issues with holes and defects on the side of the print.
It always happens at the same place (to the left of the indented hexagon) and it gets to a certain level and then it's completely fine and goes back to being flawless. Here is what I've tried so far:
• Levelled the print bed.
• Cleaned the print bed.
• Changed the filament type (eSUN PLA+ , Kexcelled PLASilk).
• Printed on the floor for stability.
• Changed the print temperature (200 - 215 degrees).
• Changed the print speed (50% - 100%).
• Changed the infill denstity (25% - 80%).
I have never experienced this issue before and everything else seems completely fine. My next steps are to change the retraction settings and reorient the print to see if this works.
Any assistance or troubleshooting will be really appreciated.

It's been a while since I've worked with FDM printers, and I have seen this before on other prints but I don't remember how to fix it.... so take any advice here with a big ol' grain of salt.
Is the print wobbling or print head wobbling when the head passes over this area?  Did speeding up the print make it worse?  I suspect something is happening when the head goes around the sharp angles of the indent, either in the print itself or in the head.  Slowing down should improve it, but I'm not sure.  Are you able to print the one-piece version with supports?  Do you see the same thing on that STL in the same area?

Before you start a new print, can you post a section of the G-Code outline during this segment?  I'd like to see what path the print head is taking over this feature.

Hey Sielu-Rooster,
Thanks for your prompt reply and help.
I've not seen any such wobble but I'll do another test and pay close attention.
I believe the speed being slower helps the print, I'll give 20mm/s a try and see how this looks.
I'll send you a copy of the G-Code shortly.
Thank you!

Hey Sielu-Rooster,
I hope this is what you need with regards to the G-code, please see attached.
G-code.txt

That looks like a log from the printer... I think I can extract the code from it but it's tedious, can you send me the raw g-code file from the slicer?  Did slowing down the print do anything for you?

Sorry about that, see the text file attached.
DeckGuard_Left.txt
Slowing it it down definitely helped. I set the print speed to 20mm/s and printed a test, see the side by side comparison attached.

I've spent a lot of time investigating this now, changing a lot of basic settings and I believe this an issue to do with the infill settings specifically with Ultimaker Cura slicer.
I successfully printed the DeckGuard_Left piece using Kexcelled Blue PLASilk with the following profile settings:
Profile Reference: Normal - 0.2mm
Support: Yes
Minimum Extrusion Distance Window: 0.0mm
Printing Temperature: 200° C
Print Speed: 60.0mm/s
Infill Density: 0%

Every print kept having the same artifacts regardless of what settings I changed until I dropped the infill density to 0%.
I believe what is happening is that the walls are so thin that when the printer attempts to infill, it moves so quickly between each side of the wall that it overshoots and adds artifacts to the outside of the wall. Coupled with the indented hexagon shape who's complexity only exacerbates the issue.
I had a friend attempt the same print with a Original Prusa i3 printer using PrusaSlicer and this printed without any artifacts. He then copied my Ultimaker Cura slicer profile and sliced the model with Ultimaker Cura slicer and was able to replicate the issue on a Original Prusa i3.

This is a comparison of two pieces he printed on a Original Prusa i3. One was sliced in PrusaSlicer which printed perfectly and one was sliced in Ultimaker Cura slicer which printed with artifacts.
I am currently printing another test on my Anycubic I3 Mega printer using PrusaSlicer to see if this prints as desired. The other test would be to change the infill pattern to something with less edges.

Just a final comment on this, using Prusa Slicer gave me a much nicer finish for this specific print and didn't experience the issue I  was having with Cura.
So, if anyone is having this issue - use Prusa Slicer.


SASS optimizations

[ ] Remove redundancy by using SASS mixins
[ ] Adding repeated values like padding, margin, colors etc to config.scss
[ ] Removing inline CSS from handlebars and adding it to SASS.

@aayusharora can I work on this

Sure, @utkarshnath  will be great if you can do so :)

@aayusharora  , I can also work with @utkarshnath  on this issue

Sure @SparshBansal Please coordinate with @utkarshnath

"Adding repeated values like padding, margin, colors etc to config.scss "
@aayusharora do you want me to add all padding,margins used to config.scss

@aayusharora do you want to add all values to config.scss or only frequently used values

@utkarshnath My idea is to write mixins and call them whenever needed for padding, margin etc.

@aayusharora @Princu7  The inline CSS is not removed completely from all the pages. This is partially solved. please reopen it.

@aayusharora  Can i work on this issue?

Sure @varun560
Go ahead :)


Dir 'book' was copied and pasted under a new name called 'docs'.

I had to do this because GitHub pages points to an 'index.html' file that is either inside root dir or a 'docs' dir. I figured this renaming trick might just work, let's find out!

Hmm, I hope this repo doesn't get bloated. I would suggest you choose to delete either 'book' and 'docs'. It wouldn't really matter to you if you're cloning this repo just for the sake of having it locally on your machine.


Adding TestCase to Stacke_APIs

Adding TestCase to Stacke_APIs
Type of issue:

[ ] Bug / Error
[x] Idea / Feature
[x] Improvement detail

Adding Unit TestCases to test the basic endpoints of stackle Application
Using Mocha.JS and ChaiJS testing of basic endpoints of APIs can be carried out

#138

@agentmilindu I would like to add some test cases


Create process for removing accounts from AWS and add PIV authentication

🐛 Summary
Create a process for removing the IAM accounts that are set up to reach the Accessor EC2 instance. Also, require PIV authentication when creating the ssh tunnel.

On hold to work on #341 first.


Vue3, ts, babel: it won't compile on import GlitchedWriter from 'vue-glitched-writer'

ERROR  Failed to compile with 1 error                                                                                17:16:10
``
error  in ./node_modules/vue-glitched-writer/dist/vue-glitched-writer.esm.js
Module parse failed: Unexpected token (1429:37)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|           element = ref(null),
|           preset = computed( // @ts-ignore

() => cjs.presets[props.preset] ?? {}),

|           writeOnAppear = attrs.appear !== undefined || typeof props.text === 'object' && props.text.length > 0,
|           queueInterval = computed(() => {
@ ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader-v16/dist??ref--0-1!./src/views/Earlyaccess.vue?vue&type=script&lang=js 1:0-49
@ ./src/views/Earlyaccess.vue?vue&type=script&lang=js
@ ./src/views/Earlyaccess.vue
@ ./src/router/index.ts
@ ./src/main.ts
@ multi (webpack)-dev-server/client?http://<IP_ADDRESS>:8080&sockPath=/sockjs-node (webpack)/hot/dev-server.js ./src/main.ts
No issues found.
``

I've tried to do something with that, but I had no luck yet. It has to be an issue with my build tools since the component itself is working fine. I will try some more tomorrow, but for now, if you really want to use this component, you can just grab the component's code from the repo and use it in your project.
Here is the component's source code, just change 'vue-demi' to 'vue', and install glitched-writer via npm i glitched-writer (although you probably already have it installed).
And this is the "escapeHtml" function that is imported in the component:
function escapeHtml(html: string) {
	var text = document.createTextNode(html)
	var p = document.createElement('p')
	p.appendChild(text)
	return p.innerHTML
}

With all that, the component should be up and running. Sorry for the inconvenience, but I relly on some template to build this component and have little knowledge about bundlers.

Hi thetarnav,
First, thanks for your amazing work.
I think I am experimenting the same problem :
 error  in ./node_modules/vue-glitched-writer/dist/vue-glitched-writer.esm.js

Module parse failed: Unexpected token (1429:37)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|           element = ref(null),
|           preset = computed( // @ts-ignore
>     () => cjs.presets[props.preset] ?? {}),
|           writeOnAppear = attrs.appear !== undefined || typeof props.text === 'object' && props.text.length > 0,
|           queueInterval = computed(() => {

Did you already have the opportunity to fix this issue ?
Have a nice day,

Thanks for reminding me to take care of this :)
I tried a different building solution, basically scrapping the whole SFC idea since it requires knowing more about build tools to be able to solve issues. Instead, I just relied on existing tools for building typescript.
The new version is published under the beta tag: vue-glitched-writer@beta
Let me know if it helps!


Bugfix 3.4.1:  Clean-up edge cases of Scheduler shutdown

This PR addresses edge cases relating to shutting down the Scheduler:

there was the potential for other features to submit tasks to the Scheduler as it was attempting to shutdown.  This could lead to crashes and/or stuck shutdown in other features (note:  no cases of this have been seen recently)

beginShutdown() and shutdown() were only clearing tasks that were in the working or queued state.  Tasks sitting in the work FIFOs were not addressed.   FIFO tasks could be added to the Scheduler after stop() was called on the io_context and race condition allowed adding after io_context destroyed.  (io_context worker threads that were shutting down first called drain() which attempted to post work from the FIFOs).

  I do not want to start on devel version until this one is in happy shape.


Fix djgpp in open ssl 1 1 1 stable

Checklist

[x] documentation is added or updated
[x] tests are added or updated

If the commit is identical (no changes to resolve merge conflicts) to the PR against master, it is not needed.


Improve error messages

try to address some of the concern raised here: https://github.com/istio/istio/issues/16858

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
49b284cea05fba3d3bdb809c7114aa1354a9c61b
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
97c2fa0aa938c2d0ad95eb100224da09b510c4f7
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
8b3156136dac3911faae021f9f5f882d80cf6626
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Selecting fonts from an open SVG file...

I used File / Open SVG... and when I try to select text object I can  not edit text until I change the font from Null to a font listed... Then I can not see a cursor... Just a green translucent block over the text letters.   Then I am able to change text.  When Selecting a text object you can not edit by clicking the text or double clicking the block... it is treated like a text block object ... when hitting the Text Tool... it creates another text box instead... of editing the text box selected.  Can I alter this in some way?

I can't know what's going on unless you provide a sample file. Unfortunately text editing is very rudimentary and it is unlikely that I can do anything about it :(


feat: implement lookup-table base comparison and less-than assertion

Is your feature request related to a problem? Please describe.
The current approach for comparison and less-than assertion is costly, requiring several constraints per bit of the values being compared. It is slightly more efficient when using std/math/cmp gadget.
Describe the solution you'd like
Define comparison and less-than assertion using lookup tables. There are possibly several approaches - one approach is to have a lookup table consisting of values x | y | x<y and then we can use the inequality. The benefit of this approach is that we can partition the values into groups of 8 bits, but on the other hand requires big initial overhead of constructing the lookup table.
@ThomasPiellard also described that the current approach is Cmp could be extended to larger values than bits, but before efficient range checks using lookup tables wasn't worth it.
Describe alternatives you've considered
None
Additional context
None

gh pr checkout 835


Revert "Introduce an 'svc' segment for DNS search"

Reverts GoogleCloudPlatform/kubernetes#8089 - too many search paths - see #8512

LGTM.
@quinton-hoole

LGTM


weird behaviour for piped commands

Intuitively, I wanted to check if cukinia supports piped commands.
This sequence
as grep cukinia_cmd echo toto | grep toto
as cat  cukinia_cmd echo toto | cat
as awk  cukinia_cmd echo toto | awk '/toto/'
as wc   cukinia_cmd echo toto | wc
as sed  cukinia_cmd echo toto | sed 's/o/i/g'
cukinia_log "$cukinia_tests"

gives:
[PASS] cat
        1         2        21
[PASS] sed
0

It looks like something, probablyecho toto, is executed because of PASS but the result is ignored (cukinia_test==0). So, the output is confusing because cat and sed look fine. And it's not clear why some commands ouput something while other don't.
Quoting the expression ("cmd1|cmd2") helps cukinia_test incrementing but always returns FAIL regardless of cmd1 and cmd2.
The use of "|" in the syntax should be either forbidden by documentation either fixed somehow.

The cukinia config file is just a shell script, so when you do:
as "my test" cukinia_cmd echo toto | grep toto
The following happens:

as "my test" cukinia_cmd echo toto is executed
this command produces the output [PASS] my test
this output is piped into grep toto
toto is not present on the line, so nothing is displayed.

The 1 2 21 output is normal piping the same type of string into wc.
For more complex tests like that, you should use
# complex test scenario
cat /myfile | sed -e 'some magic' | grep -q 'some result'
as "this complex scenario" cukinia_test $? -eq 0


Fix  "featured speakers" and "featured conferences" display on home page

Looks like they weren't fully implemented in the redesign. Thanks for discovering it @MarcyLina!

Steps:

Get the designs from Nohemi
Probably only scope it down to ___ (not the full list of featured!)
Implement Design, and, if Nohemi expected it, JavaScript


Error while loading Page

Hi, I am getting this Error:
Failed to load resource: the server responded with a status of 500 (Internal Server Error)
Link to my page
I don't know why. Everything is correctly set up.
Do you have a guess what Chrome can't load?

Hey there @xXSchrandXx,
Please answer the following two questions so we can get a slightly deeper scope into the error.
Do you have php errors enabled?
Are your database information correct?

Sry, I am still waiting for the owner to correct the log path. But I can say that the database is correct.

Make sure your database it set up correctly. If your webserver is hosted on the same machine as your MySQL and server use localhost. If your MySQL and server are on one machine but your webserver is on a different machine make sure your MySQL has remote access (contact provider if need be). Another issue could be not having the proper php version or your site is messed up. I suggest a reinstall if need be.

Error:
[Mon Dec 03 20:08:58.548355 2018] [:error] [pid ####] [client ***] PHP Catchable fatal error:  Argument 1 passed to AdvancedBan::initialize() must be an instance of string, string given, called in /var/www/ban/index.php on line 23 and defined in /var/www/ban/AdvancedBan/AdvancedBan.class.php on line 30
SQL is localand host is local

I suggest a reinstall if need be.

Could you remove the old installation in the wiki?

I suggest a reinstall if need be.

Could you remove the old installation in the wiki?

What?

In the readme.md and in your wiki are 2 different installation configurations.
Only the one in the readme.md is useful for the last version.
Anything you can help me to fix that error?

I deleted the web-addon, installed it with the ZIP from spigot, deleted the databasetables from AdvancedBan, set the database host, user, password and table and restarted the BungeeCord Server so they generate a new table.
And set the rights of the webfiles to 777.
Result:
[Tue Dec 04 18:55:57.430550 2018] [:error] [pid 898] [client *] PHP Catchable fatal error:  Argument 1 passed to AdvancedBan::initialize() must be an instance of string, string given, called in /var/www/ban/index.php on line 23 and defined in /var/www/ban/AdvancedBan/AdvancedBan.class.php on line 30
[Tue Dec 04 18:55:57.978626 2018] [:error] [pid 969] [client *] PHP Catchable fatal error:  Argument 1 passed to AdvancedBan::initialize() must be an instance of string, string given, called in /var/www/ban/index.php on line 23 and defined in /var/www/ban/AdvancedBan/AdvancedBan.class.php on line 30

Are you using PHP 7.1 (recommended)?

7.2.12

use the spigot Ver
On Dec 4, 2018 2:11 PM, "xXSchrandXx"<EMAIL_ADDRESS>wrote:
7.2.12
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mathhulk/advancedban-panel/issues/55#issuecomment-444200386,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJhLI8RQb7paZ9YTezqBsaKQMZffINKlks5u1rrTgaJpZM4Yz7Ov
.

?

Sorry, sent that as an email. Hmm must be an issue with the panel.

Update your php to the latest version

I am currently on the last save build, or don't it does update with sudo apt-get update?

PHPInfo

well, php -v does say I have 7.2.12 and phpinfo() does say I have 5.6.38-0...
fk me

my bad, php7 now endabled
$ sudo a2dismod php5
Module php5 disabled.
To activate the new configuration, you need to run:
  service apache2 restart
$ sudo a2enmod php7.2
Considering dependency mpm_prefork for php7.2:
Considering conflict mpm_event for mpm_prefork:
Considering conflict mpm_worker for mpm_prefork:
Module mpm_prefork already enabled
Considering conflict php5 for php7.2:
Enabling module php7.2.
To activate the new configuration, you need to run:
  service apache2 restart
$ sudo service apache2 restart

Solved it...

Is it all working?


Editor settings not persisting on local version

I have a local version of JsBin running and everything works as expected except the editor settings aren't persisting correctly. I have inspected the sqlite database and the changes are being saved there, but they aren't being applied to the editor and have also reverted back to the defaults in the settings window.
To replicate:

Login
Open Account > Editor settings
Change something (reports that settings are saved)
Back to JSBin
Open Account > Editor settings
Changes aren't persisted

If you have a local version do you want to debug and send a PR?
On Wed, 24 Feb 2016 00:12 Julian Fell<EMAIL_ADDRESS>wrote:

I have a local version of JsBin running and everything works as expected
except the editor settings aren't persisting correctly. I have inspected
the sqlite database and the changes are being saved there, but they aren't
being applied to the editor and have also reverted back to the defaults in
the settings window.
To replicate:

Login
Open Account > Editor settings
Change something (reports that settings are saved)
Back to JSBin
Open Account > Editor settings
Changes aren't persisted

—
Reply to this email directly or view it on GitHub
https://github.com/jsbin/jsbin/issues/2710.

I've had a deep look into what is wrong, but can't isolate the issue. It seems like the user.js file does not contain the settings element, but that is not the only problem. Even if I hardcode it to add one in, the editor doesn't interpret it anyway. Do you have any intuition into what (or where) the problem might be?

So, IIRC, local storage is used to maintain "local" state and preferences.
Then, if the user changes then via their account prefs, then that should
override (and appear in the user.js object).
Then finally if the user has a panel name on the query string then that
rules over all.
I think it happens in editors.js (in public) and fair warning, the code is
a bit of a mess!
I think it's worth dropping the local storage support since this was a
stopgap solution whilst there was no concept of a user, and I think that's
messing with the logic.
On Thu, 25 Feb 2016 00:50 Julian Fell<EMAIL_ADDRESS>wrote:

I've had a deep look into what is wrong, but can't isolate the issue. It
seems like the user.js file does not contain the settings element, but that
is not the only problem. Even if I hardcode it to add one in, the editor
doesn't interpret it anyway. Do you have any intuition into what (or where)
the problem might be?
—
Reply to this email directly or view it on GitHub
https://github.com/jsbin/jsbin/issues/2710#issuecomment-188538545.

Ok thanks for that. The thing that really mystifies me is that it works fine on jsbin.com, but the same code doesn't work the same for me. Is there any specific server config that's needed?

Ah, no, it doesn't work fine on jsbin.com! There's some reports of exactly this issue on the live site.

Wish local version could persist editor settings...


Detect and label restatements of the headword

Not sure how sophisticated this needs to be; maybe not very. We've discussed using a new span label HEAD for this.

We've decided against this since it's hard for the model to predict.


Classpath override

Nicely done. unless I'm missing it, would be good to have override capability like
`Properties p = new Properties();
InputStream in = this.getClass().getResourceAsStream("c.properties");
p.load(in);
String externalFileName = System.getProperty("app.properties");
InputStream fin = new FileInputStream(new File(externalFileName));
p.load(fin);`
Your program would be invoked similar to this:
java -jar app.jar -Dapp.properties="/path/to/custom/app.properties"

Move Fakes to test package?

Wow. Build is nice in gradle. Can I ask what the inspiration was for the project?

I get it now. I just override by calling it twice. Closing. Oops. Anyway. Nice job.

With the library as it stands, you could implement the code you have above by using the code in the property package here: https://github.com/michaelruocco/file-loader/tree/master/src/main/java/uk/co/mruoc/file/property and doing the following:
Properties p = new Properties();

PropertyLoader classpathLoader = new ClasspathPropertyLoader();
p.addAll(classpathLoader.load("c.properties"))

PropertyLoader fileSystemLoader = new FileSystemPropertyLoader();
p.addAll(fileSystemLoader.load(System.getProperty("app.properties")));

There is also a static helper to have having to create instances of the loader, but for some reason I hadn't added the properties methods to it. So I have raised this PR which would allow the above to be shortened to:
Properties p = new Properties();
p.addAll(FileLoader.loadPropertiesFromClasspath("c.properties"));
p.addAll(FileLoader.loadPropertiesFromFileSystem(System.getProperty("app.properties")));

It also moves the fakes into the test fixtures package too, and renames and moves the top level file loader class. So I would intend to increment the major version number when merging this. Is this the sort of thing you had in mind?

Yes it is. Thanks. What are your thoughts on handling nullpointerexception? When no system property "app.properties" I have a stacktrace that looks like
Exception in thread "main" java.lang.NullPointerException 	at sun.misc.MetaIndex.mayContain(MetaIndex.java:242) 	at sun.misc.URLClassPath$JarLoader.getResource(URLClassPath.java:1058) 	at sun.misc.URLClassPath.getResource(URLClassPath.java:250) 	at sun.misc.URLClassPath.getResource(URLClassPath.java:303) 	at java.lang.ClassLoader.getBootstrapResource(ClassLoader.java:1257) 	at java.lang.ClassLoader.getResource(ClassLoader.java:1086) 	at java.lang.ClassLoader.getResource(ClassLoader.java:1084) 	at java.net.URLClassLoader.getResourceAsStream(URLClassLoader.java:233) 	at uk.co.mruoc.file.ClasspathInputStreamLoader.load(ClasspathInputStreamLoader.java:14) 	at uk.co.mruoc.file.property.DefaultPropertyLoader.load(DefaultPropertyLoader.java:19) 	at uk.co.mruoc.file.property.ClasspathPropertyLoader.load(ClasspathPropertyLoader.java:17)

The library would expect a path to be passed to load the properties, if you don't have a path configured I would handle that in your calling code personally. It depends on whether code expects that property to be set or not.
If you code expects the property to be set then I would handle it by throwing an error e.g.
        String appPropertiesPath = Optional.ofNullable(System.getProperty("app.properties"))
                .orElseThrow(() -> new IllegalArgumentException("app.properties must be configured"));
        Properties p = new Properties();
        p.putAll(FileLoader.loadPropertiesFromClasspath("c.properties"));
        p.putAll(FileLoader.loadPropertiesFromFileSystem(appPropertiesPath));

Of if you just want to load the other properties and ignore app.properties if not set, probably using some private methods to break up the complexity, so something like this:
        private static void main(String[] args) {
            Properties p = new Properties();
            p.putAll(FileLoader.loadPropertiesFromClasspath("c.properties"));
            getAppProperties().ifPresent(appProperties -> p.putAll(appProperties));
       }

       private static Optional<Properties> getAppProperties() {
           return getAppPropertiesPath().map(path -> FileLoader.loadPropertiesFromFileSystem(path));
       }

       private static Optional<String> getAppPropertiesPath() {
           return Optional.ofNullable(System.getProperty("app.properties"));
       }

I wouldn't want to update the existing behaviour directly to return empty if not present as that might be misleading to other clients. In the case of the path not being supplied at all that seems like an edge case that should be handled in the client code.

Possibly
 @Override
    public Properties load(String path) {
        try (InputStream stream = inputStreamLoader.load(path)) {
            return converter.toProperties(stream);
        } catch (IOException | UncheckedIOException e) {
            System.out.println("exp: " + e.getMessage());
            throw new PropertiesNotFoundException(path, e);
        } catch (NullPointerException e) {
            System.out.println("exp: " + e.getMessage());
            throw new PropertiesNotFoundException("file path to load is null", e);
        }

    }

What's your opinion on making exceptions part of the api?

oh. these are unchecked. ok. makes sense. does it?
Just thinking that if app.properties isn't set will be null path. I suppose null check makes the most sense.

Actually, don't worry about the PR, I noticed a few small issues with the existing error handling, so I have cleaned that up and have added the null / empty string validation check at the same time. Ill push the updates shortly.

Changes pushed and will be available in maven central shortly on version 7.0.2


Game phase

Not too hard to implement Fruit-style.

Should; pending investigation on whether it's easy enough to switch to Must.

Fruit implements it like this:
N/B phase value = 1
R phase value = 2
Q phase value = 4

You start with a counter - phase - initialised to 2*Q+4*R+4*B+4*N (call this number StartPhase or whatever).
You scan through each piece, and subtract its phase value from phase, and update an opening/endgame value as necessary.

At the end of the eval, you crossfade between opening and endgame using phase as the interpolation value, roughly like this:
score = ((opening * phase) + (endgame * (StartPhase - phase)) / StartPhase

(I hope I got the math right...)

Is blocking #10

Implemented in 1afecdda642fcfec1473a8a8c5f92575662c2168
@ZirconiumX to test

Testing blocked on #39.

@ZirconiumX - ready to close?


[BUG] Azure resources created by MONA not showing up on web control panel

Describe the bug
We're doing some tests to deploy a solution on Azure Marketplace and followed the instructions for setting up MONA SaaS.
We checked via the command line interface that the MONA resources (resource groups, app plans, app service, storage account etc.) were deployed correctly and the solution is working. However, those same resources do not show up when you access the Azure control panel via web.
To Reproduce
Steps to reproduce the behavior:

Follow the MONA setup instructions on https://github.com/microsoft/mona-saas
Check via Azure CLI that resources were deployed
Access the Azure subscription control panel via web
Deployed resources do not appear on the control panel

Expected behavior
Resources deployed by MONA should appear on the web control panel.
Desktop (please complete the following information):

OS: Windows 11 (build 22621.1848)
Browser:  Chrome (114.0.5735.198)
Azure CLI: 2.41.0

I'm just relieved it was something this simple! I was about to dive in and start diagnosing. @pealmeid, please continue to bring your thoughts and concerns here when they come up. We're all learning at the same time and your note about including this in the FAQ is a great idea. Perhaps we should make it easier to get to the resource group from the admin center. @jomarvel, want to think about this one?


READMEs have wrong link to sys_log as part of the Installation instructions

Plugin Name
Most if not all
Issue Type
Documentation issue.
The Installation sections in the READMEs in the Azure and AWS plugins and possibly others link to where to find sys_log but the link is not correct.
Plugin version
Current
Scenario:
Just tryin' to follow the instructions.
Steps to Reproduce:
Go to a plugin README and click on the sys_log link in the Installation steps.
Expected Result:
Direct me to the where to find the sys_log package file.
Actual Result:
404

Tracked internally as PSASSET-797

Fixed in #134


Disabling views - returning only JSON responses

Is it possible to disable the views parts after login, logout, etc. and just return a JSON response and let the web client handle the views itself?
Thanks

It's not actually possible in Authboss's current state. It's an unfortunate oversight. We really built it with the "Devise" solution in mind which is the views are done for you, etc.
Both myself and @kris-runzer agree that Authboss needs a refactoring to be able to pull out the various pieces: auth itself, token generation, views - into separate pieces. This would allow us to help out those with SPA-style pages as well as the server side rendering crowd. A much more take-what-you-need approach.
More unfortunate than having not made it that way in the first place, is that we don't have enough time to recreate Authboss in that image currently. That means; it's planned - but don't hold your breath :(

@aarondl is this still true today in v2?

@ernsheong Nope. Lots of the reason behind v2 was to absract the HTTP stack and let you make it behave as you want it to. Now it renders and responds how you tell it to in v2. There are interfaces in the config, namely:
authboss.Config.Core.ErrorHandler
authboss.Config.Core.Responder
authboss.Config.Core.Redirector
authboss.Config.Core.ViewRenderer

These control what gets back to the user's browser.
The defaults package implements many things that are JSON-aware including:

Renderer
Responder & Redirector

@aarondl thanks for the response!


Taiko Early Creator Access

Describe the bug
Cant seem to find my minted nft on mintpad after creating nft. Theres no transaction history to search on for thats why if you click on mint button it always says youre not allowed to mint.
Steps to reproduce
At least make a transaction history bar or history page.
Additional context
Additional context here.

This should be raised on the mintpad discord or via their email<EMAIL_ADDRESS>


Are there any plans to support CUDA 11.7?

Motivation
Hi, currently there seems to only be support for CUDA 11.8 and 12. Currently our system only supports CUDA 11.7 which creates issues with the torch install. Does that mean that we currently cannot use lmdeploy?
Related resources
No response
Additional context
No response

Already did. The lmdeploy+cu118 package can be installed in the cuda 11.7 platform

That is true, however, since it is compiled on Cuda 11.8, the torch version that it installs must also be compiled on Cuda 11.8 and the default installation is torch 2.2.2. However, we have to use a torch installation that is compiled with Cuda 11.7 to be able to run the models. It might be too specific of an error, but perhaps there are already existing workarounds? What do you think?

Can you build lmdeploy from the source?

If your nvidia driver only support cuda 11.7, you may install this way:
export LMDEPLOY_VERSION=0.3.0
export PYTHON_VERSION=38
pip install https://github.com/InternLM/lmdeploy/releases/download/v${LMDEPLOY_VERSION}/lmdeploy-${LMDEPLOY_VERSION}+cu118-cp${PYTHON_VERSION}-cp${PYTHON_VERSION}-manylinux2014_x86_64.whl --extra-index-url https://download.pytorch.org/whl/cu117

# there exists torch-2.0.1+cu117 whl package which meets lmdeploy requirement.

After installation, you should check the version of  'nvidia-nccl-cu11', 'nvidia-cuda-runtime-cu11', 'nvidia-cublas-cu11', 'nvidia-curand-cu11' python packages and make sure the version is lower than your nvidia driver support.

你好，问一下：是不是最新版本也可以支持在11.7的cuda上安装？按照上面的安装流程

yes


Error after switching to main branch

There was a typo in the last commit, so I force pushed. Doesn't seem related though.
Did it happen just after attaching?

Yup that fixed it


Regenerate some GUIDs to prevent HTK -> MRTK collisions

Overview
A few files from the HoloToolkit were migrated to MRTK v2 post RC2.1 without their GUIDs being regenerated. This PR regenerates them and replaces their references.
This would be a breaking change for anybody actively consuming the dev branch, but does not break anybody using RC2.1
Changes

Fixes: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/5122#issuecomment-514980114

/azp run

/azp run


how to detect anomaly

how to modify the code to detect anomaly? can anyone give a demo or  some suggestions?
thx!

This might be helpful:
https://github.com/awslabs/gluon-ts/blob/master/examples/anomaly_detection.py

Hi @vincentff7 ,
Thank you for using GluonTS.
Currently, we don't have more anomaly detection capabilities then the example @dai-ichiro pointed out.
Feel free to contribute if you happen to work on this!


impl(bigquery): Added DebugString() member functions for response logging

This PR adds DebugString() member functions to Dataset response and dataset objects so they can use used  in the dataset logging decorator which would be sent in a future PR.

This change is

Bradley, I'll go ahead and merge this PR when checks are done. If there are additional changes that needs to be done , please comment on this PR and I can follow that up when I do the logging decorator changes. Thanks!


Feature Request: Customize Artifacts Volume

Summary
Presently, the init container uses a typical { "emptyDir": {} } volume to store artifacts. While this a sane default for most cases, it is limiting or outright prohibitive in some cases.
Unfortunately, this is hard-coded and argo provides no mechanism to customize it.
This can be addressed by adding a new field (e.g. "artifactsVolumeSource") to the Template of type VolumeSource. This value would be used, if it is not the zero value, otherwise, the current hard-coded value would be used as a default.
I would be happy to submit a PR if this idea is accepted.
Use Cases

Consider a workflow that wishes to load an artifact into a tmpfs mount (e.g. { "emptyDir": { "medium": "Memory" } }), and then process it in memory, (perhaps it is a legacy workload that requires a filesystem interface). While this is possible, init currently mounts over the tmpfs mount with the normal mount, removing any benefit to using tmpfs, and requiring I/O hairpins. This feature would allow setting the artifacts volume as tmpfs as well, keeping all data in memory.
On the other end of the spectrum, consider a workflow that needs to load so many artifacts that the space required exceeds that provided by any node's ephemeral storage. Like with tmpfs, init mounts over any other PVC that is mounted, making this impossible. With raw Kubernetes, this could also be solved with an ephermal-type volume, or even something old-school like nfs. This feature would allow using remote storage to handle these large artifact sets.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

You might be able to workaround this with podSpecPatch. Not ideal though ofc.

is a particularly strong argument as using up the node can cause unexpected disruptions in multi-tenant environments

Thanks for the tip, I was indeed able to get around this with the podSpecPatch, only to discover that there seems to be a whole zoo of hard-coded emptyDir volumes, which also have to be patched, which, to me, only increases the value of such a field, as it could potentially be used by all of these, if needed.


chore(main): release jekyll 1.1.0

:robot: I have created a release beep boop
1.1.0 (2023-03-21)
Features

jekyll: Add graphiz dot engine (b386c50)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/plantuml/docker/releases/tag/jekyll-v1.1.0 :sunflower:


Not so self-explanatory

Some of these enumeration values are not so obvious from the name.  FastPoint and FastLine could use some explanation.  Or a link to a page that does explain them.

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

ID: 9a22dae5-fb28-2090-b6a7-48750605f2ca
Version Independent ID: 9301e06a-aa6c-20d7-914f-8e1aef1c5b08
Content: SeriesChartType Enum (System.Web.UI.DataVisualization.Charting)
Content Source: xml/System.Web.UI.DataVisualization.Charting/SeriesChartType.xml
Service: unspecified
Product: .net
GitHub Login: @dotnet-bot
Microsoft Alias: mblythe

Thanks @KentRollins for the feedback. We could probably link to the chart types articles under https://docs.microsoft.com/en-us/previous-versions/visualstudio/visual-studio-2012/dd489233(v%3Dvs.110) for more info for each chart type.


Allow DocumentLinkProvider define to which line in file we want navigate

Right now DocumentLinkProvider defines it's target as a Uri. Unfortunately this doesn't allow us to define to which line we want to navigate. I wonder if we could extend API a bit to support navigation to specific line.

It does using a L4,9-fragment for instance file:///Users/foo/test.file#L12 (line) or file:///Users/foo/test.file#L12,99 (line,column)


Chirp razor testbranch

Major changes
We have now evolved to version 2.0.0, in here we have functioning tests and releases when tags are added. Furthermore all razor page features should be involved and work.

Fourth time is the charm. Everything checks out and looks good.


Keyspace setup falls Transport to infinite loop

Hello :-)
I tried to use TcpSessionBuilder with .with_keyspace, but received high CPU and network usage.
When i started digging i found that start_reading and start_writing methods are in infinite loop.
It looks like because in start_reading method cassandra always return response with SetKeyspace result:
[cdrs-tokio-6.2.0/cdrs-tokio/src/transport.rs:320] &body = Result(
    SetKeyspace(
        BodyResResultSetKeyspace {
            body: "my_keyspace",
        },
    ),
)

When i removed .with_keyspace everything worked good.

Thanks a lot))


Unable to find syntax errors from error logs

Description:
It is not possible to find where exactly a syntax error occurs on vscode when there is a compilation error. The pattern of the error log is not identified by the vscode. We have to edit that output everytime.
Example:
Say I get the following error while compiling:
ERROR [semver.bal:(17:1,17:23)] cannot resolve module 'ballerina/math'

I then copy and do a command+p to paste semver.bal:(17:1,17:23). But no results are shown. Search results will be shown if I edit it to be like semver.bal:17:1.

Env:
MAC, slbeta2, plugin v2.1.0

Moved to https://github.com/wso2/ballerina-plugin-vscode/issues/112


Update module name to knative.dev/kn-plugin-event

github.com/cardil/kn-event --> knative.dev/kn-plugin-event
/assign @rhuss

Thank @navidshaikh ! The linter croaks, @cardil would it be ok to update the code base to make the linter happy ?

@rhuss it's these PRs changes that make the linter croak. The imports are no longer sorted correctly.

thanks !
/lgtm
/approve


[#154329640] paas-billing v0.9.0 add pricing estimation endpoint

#154329640 Pricing Calculator: Implement price estimation via billing API
What?
Bump version of paas-billing to v0.9.0
We want to deploy the pricing simulation feature implemented in alphagov/paas-billing#11
How to review?

Review and merge alphagov/paas-billing#11 first
Once our CI tagged a new version (expected to be v0.8.0), remove the WIP commit in this branch.

Who?
Anyone but @dcarley or @keymon

Thanks, placeholders look good.
I think the changes to the routes in server.go have broken oauth login by altering the base Path() that the middleware matches the /oauth/authorize and /oauth/callback paths.

rebased to master and using version v0.9.0

I think the changes to the routes in server.go have broken oauth login.

That has been fixed in the service https://github.com/alphagov/paas-billing/commit/5e37b82d7d130bd4b840bb69defa4a85790d91f7


Add acontext query param

When navigating through Facebook event pages, almost every url has an acontext query param with a bunch of JSON appended to it. I'm not sure if this only happens in context of Facebook events, but it might. I can't really find any documentation about this query param though.
Example: visit https://www.facebook.com/events/, every event teaser has a link with the acontext query param.

Hey @DieterHolvoet  - Thanks for the PR. Is this parameter something that would be found in traffic / requests to non-Facebook websites? For example, a website owner decides to advertise on Facebook, and then traffic coming from Facebook reaches their website with ?acontext in the URL? The goal of this project is the track query parameters that are used like that. If this is just used internally by Facebook while browsing their website, it's not really something I'm looking to document here.

No, you're right, it's not added to external url's for tracking purposes. Thanks for clearing that up!