Error: Building package icu:x64-windows-static

Host Environment

OS: Windows 10
Compiler: VS2019 Enterprise 16.11.0

To Reproduce
Steps to reproduce the behavior:
git clone https://github.com/microsoft/vcpkg
cd vcpkg
git pull
bootstrap-vcpkg.bat -disableMetrics
vcpkg integrate install
vcpkg install icu:x64-windows-static

Failure logs
E:\dev\projects\build\vcpkg>vcpkg.exe install icu:x64-windows-static
Computing installation plan...
The following packages will be built and installed:
    icu[core]:x64-windows-static -> 69.1#11
Detecting compiler hash for triplet x64-windows-static...
Could not locate cached archive: C:\Users\sartory\AppData\Local\vcpkg\archives\8c\8cadd1d010a78285f1ea7a98910eabaeb590118a626ab543c5e73833688413dd.zip
Starting package 1/1: icu:x64-windows-static
Building package icu[core]:x64-windows-static...
-- Using E:/dev/projects/build/vcpkg/downloads/icu4c-69_1-src.tgz
-- Cleaning sources at E:/dev/projects/build/vcpkg/buildtrees/icu/src/c-69_1-src-78c1b28057.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source E:/dev/projects/build/vcpkg/downloads/icu4c-69_1-src.tgz
-- Applying patch E:/dev/projects/build/vcpkg/ports/icu/disable-escapestr-tool.patch
-- Applying patch E:/dev/projects/build/vcpkg/ports/icu/remove-MD-from-configure.patch
-- Applying patch E:/dev/projects/build/vcpkg/ports/icu/fix_parallel_build_on_windows.patch
-- Applying patch E:/dev/projects/build/vcpkg/ports/icu/fix-extra.patch
-- Using source at E:/dev/projects/build/vcpkg/buildtrees/icu/src/c-69_1-src-78c1b28057.clean
-- Found external ninja('1.10.2').
-- Getting CMake variables for x64-windows-static
-- Using msys root at E:/dev/projects/build/vcpkg/downloads/tools/msys2/703e5aa15d402568
-- Using msys root at E:/dev/projects/build/vcpkg/downloads/tools/msys2/aa5af7b2aa7e90e8
-- Configuring x64-windows-static-dbg
-- Configuring x64-windows-static-rel
-- Building x64-windows-static-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:155 (message):
    Command failed: E:/dev/projects/build/vcpkg/downloads/tools/msys2/703e5aa15d402568/usr/bin/make.exe -j 5 --trace -f Makefile all
    Working Directory: E:/dev/projects/build/vcpkg/buildtrees/icu/x64-windows-static-dbg
    See logs for more information:
      E:\dev\projects\build\vcpkg\buildtrees\icu\build-x64-windows-static-dbg-out.log
      E:\dev\projects\build\vcpkg\buildtrees\icu\build-x64-windows-static-dbg-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_make.cmake:190 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_make.cmake:26 (vcpkg_build_make)
  ports/icu/portfile.cmake:137 (vcpkg_install_make)
  scripts/ports.cmake:141 (include)

Error: Building package icu:x64-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: icu:x64-windows-static
  Vcpkg version: 2021-08-03-1a6d67b8150e94b7f473c6ab1c0cfdd794316a64

Additionally, attach any relevant sections from the log files above.

build-x64-windows-static-dbg-err.log
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9025 : overriding '/W3' with '/W4'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
cl : Command line warning D9002 : ignoring unknown option '-Xcompiler'
'bash' is not recognized as an internal or external command,
operable program or batch file.
-- return status = 1
Failed to compile oma.c
'LIB.exe' is not recognized as an internal or external command,
operable program or batch file.
-- return status = 1
Error creating Windows DLL library. Failed command: LIB.exe /nologo /out:"../lib/sicudtd.lib" "./out/tmp/icudt69l_dat.obj"
Error generating package data.
make[1]: *** [Makefile:176: packagedata] Error 1
make: *** [Makefile:153: all-recursive] Error 2

Duplicated to #18086.


Disable shift click event

What platform were you using when you found the bug?

React Flow / Svelte Flow version: 11.10.3
Browser and version:
OS and version:

Live code example
https://codesandbox.io/p/sandbox/reactflowv11-cj7cx8?file=%2Fsrc%2Fcomponent%2FReactFlow1.tsx%3A344%2C10
Describe the Bug
Click the node or vertical line with shift+click to select it. Also, if you move left and right with the node selected with shift+click, the screen moves. I want to disable all the functions of shift, is there an option?
Steps to reproduce the bug or issue

Select a node from shift+click status
Move left and right mouse while holding the shift+click

Expected behavior
Options that are being processed as false do not work normally
Screenshots or Videos
https://github.com/xyflow/xyflow/assets/130032473/044f62d2-ef2c-4cc1-bbf0-5a15af8b444d
Additional context
No response

Disable shift click event

What is this supposed to mean?
Your "bug" report has no bug being reported and it's more than vague about what your problem is and what you want to resolve.
If you want to disable the pane being moved on node drag use the autoPanOnNodeDrag prop and set it to false

I mean, when I press the shift key and click, I wanted to disable all the features that I support. When I migrated from the previous version 10 to version 11, I treated the click function as false, but it didn't work. If I press shift and click the edge as shown in the picture, the edge container box will be displayed, and I'm trying to prevent this


More updates to README

This PR removes the scaling of an image in the README which seemed to cause issues publishing on PyPI, as well as removes the links to the readthedocs page which currently isn't working.
I also removed a note in the contributing guide that I just noticed and thought was unnecessary.

@hansen508 this is what the image looks like at 500px: https://github.com/miso-sound/misosoupy/tree/f1a26630b77d22f353928b4126e402bf2417e6d5?tab=readme-ov-file#examples

I think I will just remove the image width specifier for now to make it consistent with the others but feel free to update.

@danibene Yup, totally fine to just remove width specifier, especially if it's causing issues with PyPI
And weird about the readthedocs link not working, but I'm not sure I know enough to understand why. So I'm down to remove that too.
Feel free to merge when you're ready!


feat:注销登录(close #2240)

Reviewed, transaction id: 9161
PR
CheckList（PR Assigners）
确认已完成以下操作：

[ ] 已将分支与当前合入分支同步
[ ] 已在关联 Issue 中补充「功能自测」部分内容
[ ] 已更新关联 Issue 的状态 backlog -> todo -> for test -> tested（需通过测试同学测试） -> done
[ ] 将 PR 添加上 pr/reviewable

CheckList（PR Reviewers）
确认已完成以下操作：

[ ] 检查关联 Issue 「功能自测」部分内容
[ ] 必要时打回 pr/reviewable，要求重新修改
[ ] 及时 Approve

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.23%. Comparing base (3c866ca) to head (dec1552).
Report is 45 commits behind head on v2.4.5-dev.

Additional details and impacted files
@@              Coverage Diff               @@
##           v2.4.5-dev    #2265      +/-   ##
==============================================
+ Coverage       74.74%   75.23%   +0.48%
==============================================
  Files             404      404
  Lines           28314    28671     +357
==============================================
+ Hits            21164    21571     +407
+ Misses           7150     7100      -50

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


No Types available

Hi! Thanks for doing this, apparently there are no types for this library, is there an alternative or plan for that?

@contigen
Yes we are using jsdoc, types may be added by February.


Howto use in 404.html and forward to page.url on different domain?

Hi!
I'd love to use your plugin to forward requests to another URL, since my blog moved from WordPress to Jekyll: https://blog.marco.betschart.name - and my homepage is a GitHub Pages static site too: https://marco.betschart.name
Now, every path on the homepage which can't be found, should be redirected to the blog:
url: "https://marco.betschart.name/my-awesome-title"
redirectTo: "https://blog.marco.betschart.name/my-awesome-title"`

For this, I created a 404.html page on GitHub Pages:
---
title: Redirecting...
layout: redirect
redirect_to:
  - http://blog.marco.betschart.name
---

And now wanted to create a custom layout for the redirect page according to your docs using a combination of {{ page.redirect.to }} and {{ page.redirect.from }}:
<!DOCTYPE html>

  <meta charset="utf-8">
  <title>Redirecting...</title>
  <link rel="canonical" href="{{ page.redirect.to }}{{ page.redirect.from }}">

  <meta http-equiv="refresh" content="0; url={{ page.redirect.to }}{{ page.redirect.from }}">
  <meta name="robots" content="noindex">

Redirecting...

  Click here if you are not redirected.

Unfortunately this redirects to http://blog.marco.betschart.name/404.html - which is not the expected behaviour. How to preserve the initial page.url the user really wanted to visit in the first place and how to pass this to the redirect template?

@marbetschar you might want to take a look at Netlify redirections: https://www.netlify.com/docs/redirects/


Show only selected area with always on top

Description of the new feature / enhancement
It would be a cool feature if we can click drag select an area of an app and it hides everything that isn't selected.
Scenario when this would be used?
let say i want internet explorer to be on top of my main app, but i want to view only selected area of the internet explorer and hide area that isn't selected.  such as title bar/address bar etc etc.
Supporting information
No response

I could have used this a few days ago when i needed an e-mail on top to quickly copy and paste parts of a list to another programm.
Whith the e-mail window reduced to 1/9th of my monitor all the borders, scrollbars, ribbons, etc. took up so much space that there was almost nothing left of the actual content of the mail...
Another feature request asked for transparency.

https://github.com/microsoft/PowerToys/tree/dev/feature/cropandlock branch

This issue has been worked on and released during the 0.73 Release project. Please update PowerToys to the latest. 😉


Wrong matching number with custom rule's "search_in" property in "conditions"

Describe the bug
Matching number of custom rule is higher than expected, when using "search_in" property in "conditions".
To Reproduce
target.c file
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <string.h>

#define BUFSIZER1 512
#define BUFSIZER2 ((BUFSIZER1 / 2) - 8)

int main(int argc, char **argv)
{
    char *buf1R1;
    char *buf2R1;
    buf1R1 = (char *)malloc(BUFSIZER1);
    buf2R1 = (char *)malloc(BUFSIZER1);
    free(buf2R1);
    strncpy(buf2R1, argv[1], BUFSIZER1 - 1);
    free(buf1R1);
}

myRule.json file
[
    {
        "id": "SA000005",
        "name": "Testing.Rules.MallocAndFree",
        "tags": [
            "Testing.Rules.MallocAndFree"
        ],
        "severity": "Critical",
        "description": "this rule aims to find malloc() AND free() in 1 line range",
        "patterns": [
            {
                "pattern": "malloc\\(",
                "type": "regex",
                "confidence": "High",
                "modifiers": [
                    "i"
                ],
                "scopes": [
                    "code"
                ]
            }
        ],
        "conditions": [
            {
                "pattern": {
                    "pattern": "free\\(",
                    "type": "regex",
                    "scopes": [
                        "code"
                    ],
                    "modifiers": [
                        "i"
                    ]
                },
                "search_in": "finding-region(0,1)",
                "negate_finding": false
            }
        ],
        "_comment": ""
    }
]

terminal command
appinspector analyze -i -s target.c -r myRule.json -f json -o output.json
output
output.json file
"detailedMatchList": [
      {
        "ruleId": "SA000005",
        "ruleName": "Testing.Rules.MallocAndFree",
        "ruleDescription": "this rule aims to find malloc() AND free() in 1 line range",
        "tags": [
          "Testing.Rules.MallocAndFree"
        ],
        "severity": "Critical",
        "pattern": "malloc\\(",
        "confidence": "High",
        "type": "Regex",
        "language": "c",
        "fileName": "target.c",
        "sample": "malloc(",
        "excerpt": "    char *buf1R1;\n    char *buf2R1;\n    buf1R1 = (char *)malloc(BUFSIZER1);\n    buf2R1 = (char *)malloc(BUFSIZER1);\n    free(buf2R1);\n    strncpy(buf2R1, argv[1], BUFSIZER1 - 1);\n    free(buf1R1);\n",
        "startLocationLine": 14,
        "startLocationColumn": 21,
        "endLocationLine": 14,
        "endLocationColumn": 28
      },
      {
        "ruleId": "SA000005",
        "ruleName": "Testing.Rules.MallocAndFree",
        "ruleDescription": "this rule aims to find malloc() AND free() in 1 line range",
        "tags": [
          "Testing.Rules.MallocAndFree"
        ],
        "severity": "Critical",
        "pattern": "malloc\\(",
        "confidence": "High",
        "type": "Regex",
        "language": "c",
        "fileName": "target.c",
        "sample": "malloc(",
        "excerpt": "{\n    char *buf1R1;\n    char *buf2R1;\n    buf1R1 = (char *)malloc(BUFSIZER1);\n    buf2R1 = (char *)malloc(BUFSIZER1);\n    free(buf2R1);\n    strncpy(buf2R1, argv[1], BUFSIZER1 - 1);\n",
        "startLocationLine": 13,
        "startLocationColumn": 21,
        "endLocationLine": 13,
        "endLocationColumn": 28
      }
    ],

My intention was to find malloc() AND free() that are in 1 line range.
The expected result was 1 pattern match, which is malloc() at line 14.
Instead, it gave result of 2 matches, malloc() at line 14 and line 13.
However, malloc() at line 13 should not be a match, since there is no free() in next line.("search_in": "finding-region(0,1)").
Expected behavior
1 Match found.
Desktop (please complete the following information):

OS: Ubuntu 18.04
ApplicationInpector Version: ApplicationInspector.CLI 1.4.22+ccdc2c44b5

Thanks for the well written report. I'll get a fix out soon.

This should be fixed in 1.4.23 (building in pipeline now, should release within an hour). Thanks again for your reports.


[Feature] Hide HUD when menu paused

Hi, can you add a function  local isPaused =  to hide HUD when menu paused like this https://github.com/indilo53/fivem-es_extended/commit/96272567f01b4c02cc0dd568b0d896a844ff8af1 please ?

https://github.com/ImagicTheCat/vRP/commit/91511dd8410491e1c66eb82f0cb8ca630cfb306b

Love this Thank you !!!!


Phase 1 known issues checklist

Phase 1 beacon chain

[ ] 1. Shard chain slashing
[ ] 2. Exit fee #1521
[ ] 3. Maximum reversions: Enshrine dynamic weak subjectivity reversion period into fork choice rule ("second finality mechanism"): #1520 and #1053
[ ] 4. Rewards on shards #1280 and #1053
[ ] 5. Implement get_start_shard logic

Phase 1 shard chain

[ ] 1. [shard chain fork choice rule] move get_winning_proposal logic to it. Maintain canonical shard chain upon (i) receiving new shard block (ii) new beacon chain block (ii) time tick.
[ ] 2. [shard chain transition] Add shard_transition_digest wrapper. Currently we have hash(hash_tree_root(shard_state) + beacon_parent_root + shard_body_root)
[ ] 3. [data availability] Inefficiencies in data availability proofs #1194 (outdated???)
[ ] 4. [data availability] Generating data availability roots from shard block roots #1438

Not sure if this categorize as a beacon chain known issue, I'm curious to know if we should add Shard as a first class citizen in AttestationData, whether to replace CommitteeIndex or be explicit about both

Add an item: move to Dankrad's 0.001 bit proof of custody?


Storing large files in action results appears to be limited by MongoDB's 16 MB document limit

I am evaluating StackStorm as a replacement for an internal tool we use. Our primary motivation is being able to distribute actions across multiple machines by running several action runners. Our use case requires that certain actions be able to access the data that other actions generated, so I hoped that we could store the action data directly in the action result that StackStorm stores in MongoDB; however, it looks like actions are restricted by MongoDB's 16 MB limit on documents. e.g., if you run the core.http module that delivers a large file, the action will fail:
root@b0024465368b:/# st2 execution get 59691e1266941d00f54a7283
id: 59691e1266941d00f54a7283
status: failed
parameters:
  url: http://foo.com/some_large_file
result:
  error: command document too large
  traceback: "  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2actions/worker.py", line 132, in _run_action
    result = self.container.dispatch(liveaction_db)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2actions/container/base.py", line 68, in dispatch
    action_db=action_db, liveaction_db=liveaction_db)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2actions/container/base.py", line 131, in _do_run
    raise e
"

Since our use case also involves running actions on any of several machines, I'm not sure if it would work to just store the file somewhere on the action runner's file system, since other action runners would need access to it.
Some workaround ideas include:

Using an NFS that is mounted on all action runners to store these payloads in.
Having one action that downloads these files and stores them in S3, then puts a reference to that file in the action results to be used by downstream actions that process this data.

Both of these workarounds require actions to make assumptions about where and how shared data gets stored, and require additional actions that delete old data when action executions age out.
It would be great if StackStorm could handle large action payloads transparently. MongoDB's GridFS is a feature that was introduced to handle files larger than 16 MB, so maybe that is a viable option.

I've hit the same limitation. Here is a traceback from my log files:
'result': '{"traceback": "  File \\"/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/worker.py\\", line 159, in _run_action\
    result = self.container.dispatch(liveaction_db)\
  File \\"/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/container/base.py\\", line 88, in dispatch\
    liveaction_db=liveaction_db\
  File \\"/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/container/base.py\\", line 144, in _do_run\
    liveaction_db = self._update_status(liveaction_db.id, status, result, context)\
  File \\"/opt/stackstorm/st2/lib/python2.7/site-packages/st2actions/container/base.py\\", line 299, in _update_status\
    raise e\
", "error": "command document too large"

This is a very annoying limitation, we have long-running commands which produce lots of output, and we want to have those stored in stackstorm history...

@bigmstone I talked about it a bit in the issue description, but the rationale for handling this is that for any use case that involves data larger than 16 MB (which could be as simple as downloading a log text file from a server) requires the user to work around the limit with strategies like external indexing, which would pollute every action in your workflow and just add a lot of cruft. Without handling this case, ST2 is simply not a good fit for any use case that involves a non-trivial amount of data.

It's also interesting that previous MongoDB hard limit was 4MB: https://jira.mongodb.org/browse/SERVER-431 before lifting to 16MB.
More background about the limitation: https://docs.mongodb.com/manual/reference/limits/#BSON-Document-Size and one of the possible solutions with utilizing gridfs which can use 16MB+: https://docs.mongodb.com/manual/core/gridfs/#when-to-use-gridfs. Another way is to split the document ourselves and store it in several chunks if it exceed the size. Obviously this needs a research and understanding how that change will affect st2 and if it's acceptable or not.
I think it's possible to consider implementing the workaround for this as part of our Enterprise promise (prioritize bug-fixes) if someone will request us to do so.
We're however open to Opensource contributions for "corner case" issues like this, while working on fixing other critical bugs and adding new StackStorm features per our roadmap.


headline and copyright shold be inside the title image

Is it possible to place the title and the copyright inside the title image?

see e.g. here: http://www.gg-webdesign-weinheim.de/htmlcss-bild-mit-textbeschriftung-ueberlagern/

done via css with commit 7d1d9cff034baf2934caf87d515d6a143efee6ee


Adds utcDate field to graphql

Adds utcDate field to blockchain state in graphql. This field adds back the CODA_TIME_OFFSET that gets subtracted from timestamps. This is required to get an accurate picture of the true system time of events in the network.
Also turning time-offsets on dev (as this is we'll need this on for our releases, so we should be dev-ing with it on too)

It works!
{
  "data": {
    "version": "506598fdc54677dca977a85615f78d5a02703628",
    "syncStatus": "SYNCED",
    "blocks": {
      "nodes": [
        {
          "protocolState": {
            "blockchainState": {
              "date": "1587168063160",
              "utcDate": "1587252626160"
            }
          }
        }
      ]
    }
  }
}


Support Path objects for import / export functions

Python's builtin pathlib normalizes path functions with an object-oriented Path object.
I could be very practical if all import and export functions could support Path objects for each argument involving a path.
I can make a PR for this if you agree.

Duplicate of #685. Seems that there was no objection when I brought it up earlier; a PR would be fantastic, I'm sure :)

@roipoussiere I support this idea, PR would be very welcome

+1


add documentation for machine fields

@ibuildthecloud @cjellick

Shouldn't we be able to add some tests to validate this?

This is machine-generated @will-chan. I'm not sure what kind of tests we could add.

Maybe @will-chan is suggesting a test that reads the schema and verifies the description is set?
Since this documentation feature is already part of the framework, I don't think that's necessary.
LGTM.

@wlan0 tests failed because of these changes. Please take a look.


out of memory error when logarithmic Y-Axis scale includes the range from 7e-11 to 9.999e-11

I'm getting an "out of memory" error when trying to plot data on a logarithmic scale, but only when the data is within a certain range. Please look at the following:
https://jsfiddle.net/vp72e7j9/
If the data is above or below the range, the chart renders fine, however, if the data enters this range, the memory spikes and you get the error. (Watch the browser memory in Task Manager to see it spike). I'm using Firefox but it crashes in Chrome as well.

interesting. Thanks for including a test case. I'm guessing the loop that adds tick marks runs forever


CentOS 7: read_text(encoding='utf-8')

Change Summary
With CentOS 7 Python 3.6, running install from source with pip failed:
sudo podman run -ti --rm centos:7
yum -y update
yum -y install epel-release
yum -y install git python3 python3-devel python3-pip python3-setuptools python3-wheel
git clone https://github.com/samuelcolvin/pydantic.git
cd pydantic
pip3 install .

With following error message:
[root@c99d0585636c pydantic]# pip3 install .
Processing /pydantic
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-91v_ixvz-build/setup.py", line 62, in <module>
        history = (THIS_DIR / 'HISTORY.md').read_text()
      File "/usr/lib64/python3.6/pathlib.py", line 1197, in read_text
        return f.read()
      File "/usr/lib64/python3.6/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 14648: ordinal not in range(128)

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-91v_ixvz-build/

This PR add the required read_text(encoding='utf-8') for setup.py.
Signed-off-by: Wong Hoi Sing Edison<EMAIL_ADDRESS>Related issue number

Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

Fine by me, but support for python 3.6 had just been dropped, see #3605

Good idea for dropping Python 3.6 due to EOL, but I also found some else python lib facing similar issue with read_text(encoding='utf-8') as here, which they also apply similar fix for it, e.g.:

https://github.com/docker/docker-py/blob/master/setup.py#L44-L45
https://github.com/arrow-py/arrow/blob/master/setup.py#L6-L7
https://github.com/pyparsing/pyparsing/blob/master/setup.py#L16-L19

Adding this for preventing some edge cases should still be a good idea ;-)

Agreed, please add a change description.

Change description added :-)

thank you.


[Settings UI] Use lock symbol in GPO info bars

Description of the new feature / enhancement

Scenario when this would be used?

Makes the layout of locked settings finally consistent. (Dashboard and Flyout already use the lock icon.)
Allows to distinguish between information and policy locked setting.
The icon fits better for the point of message.

Supporting information
No response

@jaimecbernardo
Do you like it?
@niels9001
Can we create a predefined custom control without much work?

@htcfreek Yes, I think that'd make sense.. all of these have the same titles/descriptions right? Because then I'd just create a custom user control named something like "GPOInfoBar" or whatever.

@htcfreek Yes, I think that'd make sense.. all of these have the same titles/descriptions right? Because then I'd just create a custom user control named something like "GPOInfoBar" or whatever.

@niels9001
No. Two different descriptions: "These setting is managed." and "One of these settings is managed." (You cann see this as we have two special resource strings.)
There will be a special version that will contain some content and has a special title. But we can implement this in a custom way. (This special bar is WIP.)

@niels9001
Can I assign you too? I don't know how to create a custom control.

Sounds pretty good 😄  Thank you for the contribution!

@htcfreek Actually, we don't need a custom control. You can set the IconSource of the InfoBar like this:
<InfoBar IconSource="{ui:FontIconSource Glyph=}"/>

@htcfreek Actually, we don't need a custom control. You can set the IconSource of the InfoBar like this:
<InfoBar IconSource="{ui:FontIconSource Glyph=}"/>

I know.
Thought a standardized control would make sense. But now I saw that we have 5 different title/content version. Maybe a custom control makes no sense. 🤔


Cannot reproducibly save a model to disk

(Migrated from https://github.com/tensorflow/tensorflow/issues/54067)
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes, below
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): OpenSUSE Leap 15.2
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: no
TensorFlow installed from (source or binary): from pip
TensorFlow version (use command below): 2.8.0-rc1
Python version: 3.9.10

Describe the current behavior
When training and saving the same model twice, I get different outputs stored to disk, even without any training: when saving the exact same model twice in the same Python process (same "run"), saved_model.pb files differ while all other files are identical. Interestingly, when saving the model in two separate runes, saved_model.pb files are identical.
Describe the expected behavior
Save the exact same files four times (two in each of two runs).
Standalone code to reproduce the issue
"""Demonstrate bug."""
import filecmp
import sys

import tensorflow as tf

RUN = 1 if len(sys.argv) > 1 else 0

tf.keras.utils.set_random_seed(0)
tf.config.experimental.enable_op_determinism()
model = tf.keras.applications.vgg16.VGG16(weights=None, classes=1)
model.compile(optimizer=tf.keras.optimizers.Adam(), loss="categorical_crossentropy")
model.save(f"tmp_save0_run{RUN}.tf")
model.save(f"tmp_save1_run{RUN}.tf")

if not filecmp.cmp(
    f"tmp_save0_run{RUN}.tf/saved_model.pb",
    f"tmp_save1_run{RUN}.tf/saved_model.pb",
):
    print("Files are different in the same run!")

if (RUN == 1) and filecmp.cmp(
    "tmp_save0_run0.tf/saved_model.pb",
    "tmp_save0_run1.tf/saved_model.pb",
):
    print("Files are identical across runs!")

To compare between runs, run twice like this:
python bug.py && python bug.py X

Then compare:
# compare between two saves within the same run:
diff -r tmp_save0_run0.tf tmp_save1_run0.tf
# compare between two saves of different runs:
diff -r tmp_save0_run0.tf tmp_save0_run1.tf

Output:
Files are different in the same run!
Files are different in the same run!
Files are identical across runs!
Binary files tmp_save0_run0.tf/saved_model.pb and tmp_save1_run0.tf/saved_model.pb differ

Looking at the .pb files in a text editor, the main differences that I notice are counters appended to functions:

I am surprised that this happens as a result of repeated model.save, and even more I wonder how I can reset these. tf.keras.backend.clear_session() does not seem to do it.

It is not guaranteed that all internal attributes in the saved files will be identical when saving the same model twice: they're implementation details. They're not part of the model itself. To get the same file at the byte level, you need to run the same code twice, in two separate and entirely identical Python processes, in the exact same environment.
Or in this case you can simply switch to saving to the h5 format to fix this. It doesn't use autogenerated function suffixes like this.

Thanks, @fchollet!


Slime Split from Spawners removes them not paying.

*Detailed description of the issue:
For Magma Cubes and Slimes when they split they pay even if they were originally spawned from a spawner. ((Also Zombies transforming to Drowned))
From what I've seen it is possible to fix this for slimes anyways, as a plugin I use removes AI from Spawner spawned Mobs and it continues to work after the slimes have split.
I am making this, because the last Bug Report I had commented on was closed. However, you did state to them it wasn't possible, but from my knowledge I've seen several other plugin do something like what was stated above.

Jobs version: 4.17.2
Server Type (Spigot/Paper/etc): PurPur

ExploitProtections.Spawner.PreventSlimeSplit option in generalConfig already do this.

Doesn't seem to work, already have that enabled.

Is the slimes naturally generated or from spawners?

From a Spawner

Can you show your entire generalConfig file?

Sent through discord dms :)


Clean up this town

This repo needs a thorough review of all the open issues and pull requests:

[x] Merge good PRs, or do a review on anything that is not ready yet
[ ] close the issues that are already completely resolved
[ ] add a set of priority tags from P0 (must do before anything else) to P4 (frivolous fun stuff). FontBakery repo is an example of this.
[ ] add a milestone for "Someday, Maybe" and put all the stuff that won't get done in 2019 in there
[ ] add a set of milestones for the new few point releases and segment the issues into reasonable sets

Do you plan to execute on this? - if not please close

Would love to help, Dave!

Reassigning to @tomasdev

I think you're a better candidate to set the right priorities and milestones. I went ahead and closed resolved issues, and added some tags.

enhancements: require confirmation, as to whether we'll do something about them and when (priority)
bugs: require confirmation, as to whether they're on our side, or they should be forwarded to another team or organization


C API bindings for libthreescalers.{so,a}

This sets up cargo-c to generate a dynamic/static library and a C header file.
Includes infrastructure to expose Rust functions and modules, and so far includes:

[x] encoding
[x] mapping_rules

Codecov Report

Merging #100 (42faa66) into master (f813a7a) will decrease coverage by 6.26%.
The diff coverage is 1.33%.

:exclamation: Current head 42faa66 differs from pull request most recent head 86e8ec5. Consider uploading reports for the commit 86e8ec5 to get more accurate results

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
- Coverage   66.82%   60.55%   -6.27%
==========================================
  Files          31       36       +5
  Lines        2146     2368     +222
==========================================
  Hits         1434     1434
- Misses        712      934     +222

Impacted Files
Coverage Δ

src/capi/c_slice.rs
0.00% <0.00%> (ø)

src/capi/encoding.rs
0.00% <0.00%> (ø)

src/capi/ffi_cow.rs
0.00% <0.00%> (ø)

src/capi/mapping_rule.rs
0.00% <0.00%> (ø)

src/capi/version.rs
0.00% <0.00%> (ø)

src/lib.rs
100.00% <ø> (ø)

src/encoding.rs
100.00% <100.00%> (ø)

src/extensions/extension.rs
100.00% <100.00%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


org.apache.dubbo.rpc.filter.ExceptionFilter should't change the RuntimeException again

suggest ExceptionFilter should't change the RuntimeException again
// directly throw if it's checked exception
if (!(exception instanceof RuntimeException) && (exception instanceof Exception)) {
return;
}
//ellipsis some code
// otherwise, wrap with RuntimeException and throw back to the client
appResponse.setException(new RuntimeException(StringUtils.toString(exception)));
on this code it change the Exception
I don't know what purpose of change the Exception,especially when this Exception instanceof RuntimeException,It just wrap it again
If you want to send some message by Exception,a instanceof  of RuntimeException is a good choice.
It don't need to declare on every method,but catch on the global exception handler
For example,if you use java proxy,it auto generate a class like that if you has no exception appears in the signature:
try {
super.h.invoke(this, m3, (Object[])null);
} catch (RuntimeException | Error var2) {
throw var2;
} catch (Throwable var3) {
throw new UndeclaredThrowableException(var3);
}

我也碰到了同样的问题，试了自定义一个exception filter，在onResponse里隐式传递参数，结果到了全局异常处理器里还是获取不到数据。

https://github.com/apache/dubbo/issues/9342
看到这个了，试了一下，中文乱码依旧有，放弃triple协议了。还是老老实实用dubbo协议了，等坑填完了再说。dubbo协议全局异常处理器还是可以用的


Example for predicting the destruction of jet-engines

In your blog post you have a nice visualization and good predictive results for Predicting the destruction of jet-engines.
Could you attach python code or jupyter notebook to examples directory to reproduce visualizations and prediction results?

I'm working on releasing code for the turbofan dataset using Keras. I've added the plots/heatmaps  with 7fea00eb4da5fe54d39b72e39c673f69b4e56f00
Try
    from wtte.plots.plot_heatmap import plot_heatmap

    a = predicted[batch_indx,:,0]
    b = predicted[batch_indx,:,1]
    this_x_train = x_train[batch_indx,:,:].mean(axis=1)
    this_x_test =  x_test[batch_indx,:,:].mean(axis=1)

    this_tte_train = y_train[batch_indx,:,0]
    this_tte_test =  y_test[batch_indx,:,0]

    t= np.array(xrange(n_timesteps))
    fig, ax = plt.subplots(1)

    plot_heatmap(
        fig,ax,
        t,
        a,
        b,
        max_horizon = every_nth,
        time_to_event=this_tte_train,
        true_time_to_event=this_tte_test,
        censoring_indicator = None,
        title='predicted Weibull pmf $p(t,s)$',
        lw=1.0,
        is_discrete=True,
        resolution=None,
        xax_nbins=10,
        yax_nbins=4
    )
    plt.show()

I never got around to release turbofan, but @gm-spacagna did it so much better than the original experiment anyway. Check out https://github.com/gm-spacagna/deep-ttf


[CELEBORN-1700] Flink supports fallback to vanilla Flink built-in shuffle implementation

What changes were proposed in this pull request?
Flink supports fallback to vanilla Flink built-in shuffle implementation.
Why are the changes needed?
When quota is unenough or workers are unavailable, RemoteShuffleMaster does not support fallback to NettyShuffleMaster, and RemoteShuffleEnvironment does not support fallback to NettyShuffleEnvironment at present. Flink should support fallback to vanilla Flink built-in shuffle implementation for unenough quota and unavailable workers.

Does this PR introduce any user-facing change?

Introduce ShuffleFallbackPolicy interface to determine whether fallback to vanilla Flink built-in shuffle implementation.

/**
 * The shuffle fallback policy determines whether fallback to vanilla Flink built-in shuffle
 * implementation.
 */
public interface ShuffleFallbackPolicy {

  /**
   * Returns whether fallback to vanilla flink built-in shuffle implementation.
   *
   * @param shuffleContext The job shuffle context of Flink.
   * @param celebornConf The configuration of Celeborn.
   * @param lifecycleManager The {@link LifecycleManager} of Celeborn.
   * @return Whether fallback to vanilla flink built-in shuffle implementation.
   */
  boolean needFallback(
      JobShuffleContext shuffleContext,
      CelebornConf celebornConf,
      LifecycleManager lifecycleManager);
}

Introduce celeborn.client.flink.shuffle.fallback.policy config to support shuffle fallback policy configuration.

How was this patch tested?

RemoteShuffleMasterSuiteJ#testRegisterJobWithForceFallbackPolicy
WordCountTestBase#celeborn flink integration test with fallback - word count

Ping @reswqa, @RexXiong.

Ping @RexXiong, @FMX.

Thanks, merge to main(v0.6.0)


TensorFlow 2.0

Excuse me, first of all congratulate him on his work which is really interesting. The problem I have is because I want to run the code but when analyzing the ".sh" that you provide, you used TF 1.8. Is there a way to compile it with TF 2.0 ?, some modification or maybe do the compilation "manually"

Excuse me, first of all congratulate him on his work which is really interesting. The problem I have is because I want to run the code but when analyzing the ".sh" that you provide, you used TF 1.8. Is there a way to compile it with TF 2.0 ?, some modification or maybe do the compilation "manually"

hi，what's mean this below functions？ thanks
` def m2full(self, m):
return m * self.std_m_tf + self.mean_m_tf
def shape2full(self, shape):
    return shape * self.std_shape_tf + self.mean_shape_tf`

Please see this link
https://www.tensorflow.org/guide/create_op


Alt text button obscures image in feed view

Describe the bug
Alt text button obscures image
Screenshots

Would like the option to remove the alt icon in the settings if this can’t be relocated to a better position. Alt text serves a great purpose that I have no need for, and it does obscure images in the feed.

I checked and twitter does the same thing except the opacity is lighter — which could be a nice tweak to start with:

That being said, I'm not loving that either and I'd also +1 the ability to remove that overlay in the options.


Misc improvements

keyp gets a timeout => no more sleeping => better input lag (doesn't work on windows)
expunge horrid ungetc hack in favour of poll
portability: should now work on freebsd and macos at least
no more putc; puts does the same thing
rkey result unboxed and scalar (literally every user previously unboxed and scalarised it...)

N.B. not tested on linux or windows

hey, @moon-chilled.  I appreciate the PR!
I can't remember why I didn't use poll() the first time around. Thanks for getting it working.
I probably should have said somewhere that the branch with the latest dev version is work, not main... I put it on a separate branch because I didn't want to break things for people who download a github release from j.
I'm not sure how to merge this into the work branch, but it's probably time for a new release anyway -- work is 124 commits ahead of main, and it's been ages since I touched either of them.
I'll try to merge this and prepare a new release soon.


Puzzling results during training

Describe the bug
Hi, I have encountered an issue similar to what is described in issue #212. When using the model for training on a tile (which I unfortunately cannot share), I obtain ridiculously low precision and recall (in the order of 0.001). I used this same exact code last week, with the same exact tif file and annotation file, and obtained very good precision and recall (above 0.75 for both). When I open the pred_result folder, I see something very similar to what is displayed in issue #212, with all the boxes squeezed in a line at the top.
If I simply predict on the same tile, without going through training, I obtain a reasonable result. So there's something weird happening when I train. I'm particularly puzzled because I have used this exact code with the same data for a couple of weeks and had consistently good results, so what has changed all of a sudden?
To Reproduce
Steps to reproduce the behavior:
This is a link to my Colab, which follows quite closely your Colab demo: https://colab.research.google.com/drive/1UKv5lWqqNpGvlLmDb3pLtcHx9-x6W7SQ?usp=sharing
I only changed the file names to something like "yourTrainingFile", as I cannot share the tile (however I don't believe that is the problem here)
Environment (please complete the following information):

OS:
Python version and environment :
I'm using Google Colab as training does not work on my Windows machine

Screenshots

Additional context
Add any other context about the problem here.
Thank you for your help!

so sorry, not sure that link works :D in case it doesn't, here's one that should:
https://colab.research.google.com/drive/1UKv5lWqqNpGvlLmDb3pLtcHx9-x6W7SQ?usp=sharing

looking into this now since more than one person has reported it. I could not reproduce in the other case and without sharing and image it is very difficult.  Can you please try with the standard package data
from deepforest import get_data
image_path = get_data("OSBS_029.png")
csv_file = get_data("OSBS_029.csv")

and confirm you get the same odd result?

can you confirm when you last ran? Looking here
https://github.com/weecology/DeepForest/commits/main
There has been no updates since June 26th
https://pypi.org/project/deepforest/#history

if you really can't share the image (i'm happy to sign whatever), run that colab and print images of

predict before training to show it works.
Visualize the a training crop to see if preprocess.split_raster works correctly.
Predictions after train
wrap it into a pdf and attach it here.

I can confirm the same thing happens when I use the standard package data:

I'll ask tomorrow at work what is required to share my tile and my annotations, and hopefully we can come to an agreement :) Otherwise I'll try what you suggest in the last comment. However, the fact that it happens also with the standard package makes it seem like the problem does not depend on my data. I'll be in touch soon!

definitely. can you link the colab you did for that last image just so we are on the exact same page. I don't need your data if we can recreate it with package data. Thanks for trying that out. Something strange going on since I cannot recreate locally? We need to get the exact commands down
from deepforest import main
from deepforest import get_data
from matplotlib import pyplot as plt
import os
m = main.deepforest()
m.use_release()
old = m.predict_image(path=get_data("OSBS_029.png"), return_plot=True)
m.config["train"]["csv_file"] = get_data("OSBS_029.csv")
m.config["train"]["root_dir"] = os.path.dirname(get_data("OSBS_029.csv"))
m.config["epochs"] = 1
m.create_trainer()
m.trainer.fit(m)
new = m.predict_image(path=get_data("OSBS_029.png"), return_plot=True)
fig, axes = plt.subplots(nrows=1,ncols=2, figsize=(5,5))
axes[0].imshow(old[:,:,::-1])
axes[1].imshow(new[:,:,::-1])

can you confirm it isn't just colab + matplotlib figsize? Also just do 1 epoch for sanity sake.

Hi, I deeply apologise, I think yesterday I wasn't quite awake, and so tested your proposition not using the data you recommended above, but rather the 2019_YELL_2_528000_4978000_image_crop2.png . I went into autopilot because that's what I had used to follow your tutorial.
Today I used the OSBS_029.png, and obtained the same result as you, and tested using your same code.
However, a problem persists with the YELL data. Here's a link to the Colab I used: https://colab.research.google.com/drive/1JtF0oyP0WmkGy_4cDOw0qQZ3JLjMyKOR?usp=sharing
And even more strangely, today I obtain a slight variation on what was happening before, where it says that no predictions are made after the training process. However this is the same code as yesterday, so I don't understand what changed.
Either there's something off in the way the package is pip installed on Colab (the other person in issue #212 is also using Colab and installing it the same way as me), or perhaps something happens when going through one of the step that are not done for OSBS_029.png. Perhaps when using the xml_to_annotations utility? or one of the subsequent steps. I'm stabbing in the dark here.
So sorry again for creating confusion! However there's still clearly something weird happening, even when using the package data, so hopefully this is useful.

And I get the same behaviour when using my own data. 0.0 for both precision and recall, and no more of the line of squeezed boxes at the top, but rather just no predictions at all. Below a screenshot of the content of my pred_result folder:

Same code as yesterday, so it just seems like perhaps this is a variation of the same behaviour as I was getting before?

Follow up here. Can you show the results of the above code. I'm trying to break down if this is a package contents issue, a interaction with google colab, or specific to the dataset. I need full code, not just output images. I'm concerned you are using the full image annotations instead of the crop annotations, but that's just a guess without full code. Let's stick to package data to make this reproducible and then test it on your own data.

Hi, thank you for the follow-up. Sorry if this is daft, but I'm unclear on what you mean by needing the full code, is the link to colab not enough? That's all the code I ran. It feels to me like there's an issue when the package interacts with colab, but, like I said, I'm stabbing in the dark a little :D

Hi
Today I ran the same code as before and it worked as expected after training

Now because I don't want to lose these results I would like to save this trained model
How should I do that?
I already tried :
model.save_model(path)
but it returns this:

I've run into this same problem on colab. Best to just save as normal torch
torch.save(model.model.state_dict(),model_path)

and restore
model = main.deepforest()
model.model.load_state_dict(torch.load(model_path))


Fixing a typo

In ToC: Ojective -> Objective

Coverage remained the same at 92.861% when pulling fc676989b3d42ab9fb8d0aba98557e452b422fc0 on pozhidaevak:patch-1 into 66cc47f64f1f077af6ca3132712a965bfce46695 on mulesoft:master.


[DNM] CMake: Add -target-min-inlining-version abi to the Swift standard library build

This will prevent issues with incorrect or missing availability checking in inlinable function bodies.
Resolves rdar://89991736

@swift-ci please test

@swift-ci please test Linux

Rebased on top of https://github.com/apple/swift/pull/41813 to use min instead of abi.

@swift-ci please test

@swift-ci please test Linux

@swift-ci please test macOS

@swift-ci please test macOS

Now rebased on main again. A full CI run completed successfully so I'm just going to smoketest the rebase before merging.

@swift-ci please smoke test and merge


Animate Accordion

Describe the solution you'd like
Ideally there would be a toggleable animation on all panel which stretches the viewport  so it creates a smooth transition instead of just appearing
Describe alternatives you've considered
N/A
Additional context
When you open an Accordion the content just appears, whereas my solution would be to animate that event.

Duplicate of https://github.com/radzenhq/radzen-blazor/issues/269


Feature request: compile build.rs scripts in release mode even when doing a debug build

Right now, cargo build compiles build.rs in debug mode, while cargo build --release compiles build.rs in release mode.  However build.rs scripts can potentially be very computationally intensive, and I would like the option to run my build.rs script in release mode even when doing a debug compile of the rest of my crate.
As a motivating example, nikomatsakis/lalrpop is a parser generator that does source generation during the custom build step.  Just the build.rs step takes 6m30s when compiling in debug, and 33s when compiling in release for a complex grammar.

cc @nikomatsakis
I've certainly wanted this in the past as well. The tricky parts I can think of are:

We need some sort of "nice syntax" for configuring this. We may not want to use [profile] as this may want to be a per-package instead of a per-application setting.
It's not just the build script that needs to be optimized here, but also the dependencies. Sometimes the dependencies can be re-used with the original application, so we need to rationalize that as well.

On Tue, Mar 01, 2016 at 10:46:06AM -0800, Alex Crichton wrote:

cc @nikomatsakis
I've certainly wanted this in the past as well. The tricky parts I can think of are:

We need some sort of "nice syntax" for configuring this. We may not want to use [profile] as this may want to be a per-package instead of a per-application setting.
It's not just the build script that needs to be optimized here, but also the dependencies. Sometimes the dependencies can be re-used with the original application, so we need to rationalize that as well.

Yeah, I think the main reason I'd not yet opened an issue requesting
this was not having an answer to these questions. It's also unclear to
me what the defaults ought to be: I'd EXPECT that build.rs typically
wants to be "release by default", but I don't know.
I could also imagine this not being something that one configures, but
rather part of the cargo build command. e.g., perhaps build.rs
defaults always to a release build, but you can run cargo build --preprocessor debug or something when you actually want to debug
your build.rs script.

The story for lalrpop is kind of interesting here. It's fairly expensive to compile, particularly in release mode (~minutes). For small grammars, total build script time is dominated by compiling lalrpop-snap, in which case you'd probably want a release build, but for larger grammars, the situation inverts and you really need an optimized version of lalrpop.

cargo build --preprocessor debug

I'd rather not have this.  In the scenario where I have
user application depends on my library which uses lalrpop
I'd like for my library to use a release preprocessor always because I know that the trade off is worth it for my library.  I don't want to have to tell people "oh, and if you depend on my library, you should really run cargo build --preprocessor=release.

@TyOverby

I don't want to have to tell people "oh, and if you depend on my library, you should really run cargo build --preprocessor=release.

My proposal was that release would be the default. You would use debug if you knew you wanted to debug the build.rs script (which would presumably not be the common case). But the point is this is a decision that only the end-user is really in a position to make, not any particular dependency.

@sfackler

The story for lalrpop is kind of interesting here. It's fairly expensive to compile, particularly in release mode (~minutes).

I'd much rather always have release mode. Sure, the first compile is slow, but after that it doesn't change, so who cares? In general I feel like the first time I compile anything in cargo (or virtually any build system) it is slow, because it has to run off and grab a bunch of deps.

@nikomatsakis Ok yeah, I agree with this.  Running build scripts in release sounds like a good default.

There's a few concerns I'd have about having a default release for build dependencies:

If we still have the option to switch to debug mode, we still have the problem of inventing a syntax for such an operation. Arguably the same syntax at that point could be used for "this should be optimized"
The problem of dependencies is sort of two-fold. A crate can be used both for a build script and for the final artifact generation. In this case do we optimize it or do we not? Unfortunately either choice we make can be the wrong choice depending on how you look at it.

Considering cross-compilation with cargo, isn't already a thing that build dependencies are compiled in a very different way than crate ones?
Having this scenario in mind, the fact that build script are compiled in release mode with the triplet target of the host is a default that make more sense than being similar to the crate parameter.
In case a crate is used for both dependencies, for a cross-compilation use case you would have to compile it twice anyway so it doesn't seems crazy.

On Wed, Mar 09, 2016 at 04:57:18PM -0800, Alex Crichton wrote:

If we still have the option to switch to debug mode, we still have the problem of inventing a syntax for such an operation. Arguably the same syntax at that point could be used for "this should be optimized"

I contend that this is the wrong default -- I think the number of
people writing custom logic in their build.rs that they plan to debug
pales in comparison to the numbe of people injecting logic written by
others.
That said, it may be that the right fix here is to stop using
build.rs altogether. That is, maybe what we want is a way to declare a
"preprocess-dependency" that compiles and executes the crate as a
preprocessing step automatically, without the need for a user to write
their own build.rs script. We could then make the default for that
kind of dependency be release, and keep build.rs as it is today.
This also makes using LALRPOP, syntex, and other similar preprocessors
more ergonomic: you only have to add one line to Cargo.toml, instead
of following several distinct steps.

The problem of dependencies is sort of two-fold. A crate can be used both for a build script and for the final artifact generation. In this case do we optimize it or do we not? Unfortunately either choice we make can be the wrong choice depending on how you look at it.

My assumption was that the "build-dependencies" are distinct from the
"main" dependencies. That is, build.rs is a distinct target, so it's
not clear why it has (or should) to use the same build of the crate?
(Feels a lot like workspaces. :)

Yeah this may indeed be a different class of dependencies (preprocessors), but those also sound a lot like plugins...
When you're compiling for the host target (e.g. you didn't pass --target) then if the application depends on a a crate which is also depended on by a build script, we'll only compile that crate once (to avoid unnecessary compilations). I guess we could just decide to compile it twice though

Yeah this may indeed be a different class of dependencies (preprocessors), but those also sound a lot like plugins...

Is this surprising? I guess you're saying that adding something more "first class" than build.rs doesn't feel right to you, given that we will (eventually) have some kind of plugin system? I could understand that. I think it may well be that we continue to want "preprocessors" of the current form, which generate "all new" source rather than being triggered by some macro use embedded in a .rs file, but I'm not entirely sure about that. It's not clear why that would necessarily be better than some kind of #![invoke_preprocessor] attribute (or maybe even something that triggers automatically when given an extern crate).

Hm yeah I guess they're different enough to warrant different systems. Seems plausible as becoming a thing!

Is the idea of a host profile for build dependencies ridiculously naive? The defaults could be same as the release profile, but you could change it however you like:
[profile.host]
opt-level = 1
debug = true

I would love to not rebuild all the host build dependencies when using other profiles.

@jdub that's plausible, yeah!  The only question would be what to do for a crate which needs to be compiled by the host profile and the target profile. My guess would be the target takes precedent, but in general doesn't matter too too much.

I should like to offer another example: the hyphenation crate normalizes a corpus of UTF-8 text at build time, an operation that takes an order of magnitude longer in debug mode. This was cause of some perplexity upon first encounter.
Dependencies notwithstanding, for small libraries such as hyphenation the problem is easily remedied, but it would be nice to have a dedicated option.

For another example, see https://github.com/sfackler/rust-phf/issues/104 , which provides an example where PHF's perfect hash generation takes 685 seconds when compiled for debug, or ~16.8 seconds when compiled for release.

FWIW we don't really need any more examples of where this is helpful, it's clear that there's a well-motivated use case.
What we need now is a solution!

There are also cases where build.rs depends on crates that take minutes to compile in release mode.
Some way to turn the optimization level down a bit would greatly speed up the build. I've also found that opt-level=1 can sometimes offer the best of both worlds.


Specify -Wc,lp64 -Wl,lp64 in native compilation flags for tck on z/OS

Specify -Wc,lp64 -Wl,lp64 in native compilation flags for tck on z/OS.
Resolves : backlog/issues/595
Signed-off-by: Mesbah-Alam<EMAIL_ADDRESS>

Grinder: Grinder_JCK/217


Mouse 3 messes up booster

Booster is mapped to tab, but if I hit mouse 3 while mouse control is off, the booster fires. Then I can't use my normal mapping to boost until I open the esc->g->c menu and then hit 'accept', which remaps the booster to tab.
Imported from trac ticket #155,  created by TurkeyXIII on 07-19-2010 at 07:22, last modified: 07-19-2010 at 07:28

mouse 3 you must mean mouse 2 (wheel click) ?
remap your mouse click button to something else or offer a suggestion on what button to remap it, if any, by default.  when i originally implemented Wheel and XButton support for the mouse, i defined what i thought would be handy defaults.
      m_wheelupTK(TK_ThrottleUp), //Imago 8/15/09
        m_wheeldownTK(TK_ThrottleDown),
        m_wheelclickTK(TK_FireBooster),
        m_xbutton1TK(TK_TargetFriendlyBase),
        m_xbutton2TK(TK_ToggleAutoPilot),
        m_xbutton3TK(TK_TargetCenter),
        m_xbutton4TK(TK_TargetNearest),
        m_xbutton5TK(TK_Ripcord) // --^

Trac comment by Imago on 07-19-2010 at 07:24

Oops I just noticed this already has a more general ticket. [https://github.com/kgersen/AllegianceTest/issues/54\]
Trac comment by TurkeyXIII on 07-19-2010 at 07:28


[Feature]: SSH into the host or the containers from inside the UI

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
A new button/section in the Coolify Dashboard UI to allow to open a terminal into any of the deployed containers and/or the host running Coolify.
Why should this be worked on?
Because it will make debugging, troubleshooting issues, and monitoring a lot easier, as these can be done from the same interface, rather than opening a separate terminal in the workstation and keeping creds etc...

This is a really cool idea! I would love to work on this! Can anyone assign this issue to me?

This will be added to v4.
I'm closing this issue, because tracking features will be moved to GitHub discussions. Github issues will be only for bugs.
Version 4 is in full focus. v3 related issues will be closed, if it is not a critical bug.
Thanks for your understanding.

Thanks. Is it now possible to "migrate" a current v3 instance to v4?


Validate Identity Provider Certificate

How is "Clear the Validate Identity Provider Certificate check box" in step 4 C acceptable practice? Without this function how does the PA know the SAML assertion is valid?

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

ID: 1e69aa64-39b1-c520-b803-cdf72d374e2f
Version Independent ID: bca80503-0cb4-427b-5789-97dfe3dc6b59
Content: Tutorial: Azure Active Directory integration with Palo Alto Networks - Admin UI
Content Source: articles/active-directory/saas-apps/paloaltoadmin-tutorial.md
Service: azure-active-directory
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@Bievens
Thanks for your feedback! We will investigate and update as appropriate.

@Bievens Thanks for reporting this. I have assigned this to the author to validate this and provide more details.

We were able to confirm this is not required for the PA to validate the SAML signature. This option is to check revocation with CRL or OCSP.
Example with option off

2019-03-07 15:48:48.169 -0700 Warning:  _handle_signature(pan_authd_saml_internal.c:1419): SAML message has signature from IdP "", but validate-idp certificate is NOT enabled, irrespectively validate the signature in SAML message<PHONE_NUMBER> INFO OpenSAML.Utility.SAMLSign : successful signature verification
2019-03-07 15:48:48.181 -0700 debug: _has_valid_signature(pan_authd_saml_internal.c:1373): Succeed to verify signature against certificate of IdP ""
2019-03-07 15:48:48.182 -0700 SAML Assertion: signature is validated against IdP certificate (subject 'crt.Azure_AD_IDP.shared') for user '

Example with option on

2019-03-07 15:28:53.260 -0700 pan_mp_check_cert_status: cert is valid
2019-03-07 15:28:53.260 -0700 debug: _extract_n_verify_idp_cert(pan_authd_saml_internal.c:394): IdP certificate "crt.Azure_AD_IDP.shared" of entity Id "" is not revoked<PHONE_NUMBER> INFO OpenSAML.Utility.SAMLSign : successful signature verification
2019-03-07 15:28:53.272 -0700 debug: _has_valid_signature(pan_authd_saml_internal.c:1373): Succeed to verify signature against certificate of IdP ""
2019-03-07 15:28:53.272 -0700 SAML Assertion: signature is validated against IdP certificate (subject 'crt.Azure_AD_IDP.shared') for user

Azure cert will not import as a CA cert. If you want to trick the PA into setting it as one so you can test this function I found a hack here https://bitbodyguard.com/articles/palo-alto-networks/google-cloud-identity-as-saml-idp/
You may close this issue.


Native notebooks widget priority is lower than text/plain

Make sure widgets work in the interactive window
Run a notebook with a widget
Try to display it.
Text/plain will text precedence.

I believe this is an error in VS Code.
They made changes last week, and its possible they haven't fixed this issue yet.

Sorry I should have tried updating to the latest. It works in the latest bits.


FluentValidation within MediatR pipeline using .NET Core's default DI container

I would greatly appreciate it if someone could help me to integrate FluentValidation into the MediatR pipeline using .NET Core's default DI container.
I have the following classes which represent my request, my handler, and my FluentValidation Validator:
`
public class SimpleRequestRequest : IRequest<string>
{
	public int Id { get; set; }
}

public class SimpleRequestRequestValidator : AbstractValidator<SimpleRequestRequest>
{

	public SimpleRequestRequestValidator(SimpleRequestRequest request)
	{

		RuleFor(m => m.Id).MustAsync(BeValidIdAsync).WithMessage("Id must be greater than 0, it is {PropertyValue}");
	}

	private Task<bool> BeValidIdAsync(int id, CancellationToken cancellationToken)
	{
		return Task.FromResult( id > 0);
	}
}

public class SimpleRequestHandler : IRequestHandler<SimpleRequestRequest, string>
{
	public Task<string> Handle(SimpleRequestRequest request, CancellationToken cancellationToken)
	{
		return Task.FromResult($"Request was for {request.Id}");
	}
}

`
In order to inject FluentValidation, I have created the following behavior class:
`
public class ValidationBehavior<TRequest, TResponse> : IPipelineBehavior<TRequest, TResponse> where TRequest : IRequest<TResponse>
{
	private readonly IEnumerable<IValidator<TRequest>> _validators;

	public ValidationBehavior(IEnumerable<IValidator<TRequest>> validators)
	{
		_validators = validators;
	}

	public Task<TResponse> Handle(TRequest request, CancellationToken cancellationToken, RequestHandlerDelegate<TResponse> next)
	{
		var context = new ValidationContext(request);
		var failures = _validators
			.Select(v => v.Validate(context))
			.SelectMany(result => result.Errors)
			.Where(f => f != null)
			.ToList();

		if (failures.Count != 0)
		{
			throw new ValidationException(failures);
		}

		return next();
	}
}

`
Within my ASP.NET Core (2.1) WebApi, I have the following controller:
`
[Route("api/[controller]")]
[ApiController]
public class TestController : ControllerBase
{
    private readonly IMediator _mediator;

    public TestController(IMediator mediator)
    {
	    _mediator = mediator;
    }

	// GET api/values/5
	[HttpGet("{id}")]
    public Task<string> Get(int id)
	{
		return _mediator.Send(new SimpleRequestRequest {Id = id});
    }

}

`
What I would like to have happen is when the controller calls _mediator.Send, I would like the Validation to run.  I have this working in a non .NET core solution using the IRequestPreProcessor interface.  I am just trying to update things so that I can do the same type of thing within .NET Core.
I believe my struggle is in registering the services.  This is in the Startup.cs file.
`
	public void ConfigureServices(IServiceCollection services)
	{
		services.AddMvc()
			.SetCompatibilityVersion(CompatibilityVersion.Version_2_1)
			.AddFluentValidation(fv=> fv.RegisterValidatorsFromAssemblyContaining(typeof(SimpleRequestRequestValidator)));

		services.AddMediatR(typeof(SimpleRequestHandler).Assembly);
		services.AddTransient(typeof(IPipelineBehavior<,>), typeof(ValidationBehavior<,>));
		AssemblyScanner.FindValidatorsInAssembly(typeof(SimpleRequestHandler).Assembly)
			.ForEach(result =>
				{
					Console.WriteLine("Here we are");
					//services.AddScoped(result.InterfaceType, result.ValidatorType);
					services.AddTransient(result.InterfaceType, result.ValidatorType);
				}
				);
	}

`
Within this, I don't think I need to call AddFluentValidation on the AddMvc builder.  I really am not trying to overwrite the validation on the Web-API controllers.  I just want it on the MediatR portion.
The issue I am running into, however, is when I call the WebAPI controller.  It gets to _mediator.Send, and it throws the following error:
Unable to resolve service for type 'MediatrSample.DLL.BLL.Requests.SimpleRequestRequest' while attempting to activate 'MediatrSample.DLL.BLL.Requests.SimpleRequestRequestValidator'.
It is like the DI container for .NET Core doesn't know how to create the SimpleRequestRequest to pass into the Validator.
Any thoughts or ideas?  I have found a number of articles that use things like AutoFac for the DI Container, but I am trying to use the default DI in .NET Core.
Ideally, I don't want to have to register all validation and request classes.  I would rather it discovers it automatically.  I thought that is what I was doing with the AssemblyScanner, but apparently not.
Thank you!

FluentValidation supports MS Extensions DI with a separate package: https://fluentvalidation.net/aspnet#asp-net-core

It also could be the collection part - have you tried to resolve the validators directly from the container?

Jimmy, thank you for the response.  I looked at that, but I couldn't get it to inject into the Mediatr pipeline, it seemed to only inject into the ASP.NET pipeline, if that makes sense.
I was able to get it to work using StructureMap using the following code:
`
var container = new Container();
		//Configure our DI Container
		container.Configure(config =>
		{

			// Fluent Validation
			config.Scan(scanner =>
			{
				//Register everything in our BLL
				scanner.AssemblyContainingType<SimpleRequestHandler>();
				scanner.AddAllTypesOf<IValidator>();
				scanner.ConnectImplementationsToTypesClosing(typeof(IValidator<>));
			});

			// MediatR
			config.Scan(scanner =>
			{
				//Register everything in our BLL
				scanner.AssemblyContainingType<SimpleRequestHandler>();
				scanner.ConnectImplementationsToTypesClosing(typeof(IRequestHandler<,>));
				scanner.ConnectImplementationsToTypesClosing(typeof(IRequestHandler<>));
				scanner.ConnectImplementationsToTypesClosing(typeof(INotificationHandler<>));
			});

			//Pipeline
			config.For(typeof(IPipelineBehavior<,>)).Add(typeof(RequestPreProcessorBehavior<,>));
			config.For(typeof(IPipelineBehavior<,>)).Add(typeof(RequestPostProcessorBehavior<,>));
			//Add our Validation Behavior, to run Fluent Validation, to our Mediatr pipeline...
			config.For(typeof(IPipelineBehavior<,>)).Add(typeof(ValidationBehavior<,>));

			config.For<IMediator>().LifecycleIs<TransientLifecycle>().Use<Mediator>();
			config.For<ServiceFactory>().Use<ServiceFactory>(ctx => ctx.GetInstance);
			config.Populate(services);
		});

`
I agree, I think it may have something to do with resolving the validators in the DI container, but can't figure out how to do with with the default DI container in .NET Core.  It may have to do with .NET Core not being able to scan assemblies and inject automatically.  I think there was an Scanner add in that was able to do that.  Maybe I need to go that route.
Thank you again for the feedback.  For what it is worth, I really appreciate the work you put into this tool set.  Looking back at some of my older code, I had been attempting CQRS but in a much less organized way.  Using MediatR allows me to not only have an organized way of doing things, it also eases developer adoption and understanding.  Thank you for all of your work on this.

Although this post is already closed and there is a solution/workaround using StructureMap I wanted to mention that removing the parameter from the SimpleRequestRequestValidator constructor fixed it for me.
This is the working SimpleRequestRequestValidator class:
public class SimpleRequestRequestValidator : AbstractValidator<SimpleRequestRequest>
{
	public SimpleRequestRequestValidator()
	{
		RuleFor(m => m.Id).MustAsync(BeValidIdAsync).WithMessage("Id must be greater than 0, it is {PropertyValue}");
	}

	private Task<bool> BeValidIdAsync(int id, CancellationToken cancellationToken)
	{
		return Task.FromResult( id > 0);
	}
}

I know it is late to answer but u can use:
           services.AddValidatorsFromAssemblyContaining(typeof(yourCommandValidator));

maybe it will be useful for other developers who will face the issue like me.
So my issue was in collection type for IValidator validators, because I was using IList and was getting the error something like - Unable to resolve service for type 'System.Collections.Generic.IList1[FluentValidation.IValidator...
Finally, after replacing the IList type on IEnumerable, this error was disappeared. It means that the container can be set up to resolve only for IEnumerable collection

Thanks a ton...
This issue was freaking me out...
I was almost on the verge of ditching fluent assertion and implement custom validators on my own..
Your suggestion saved my day...

public class ValidationBehavior<TRequest, TResponse> : IPipelineBehavior<TRequest, TResponse> where TRequest : IRequest
{
private readonly IEnumerable<IValidator> _validators;
public ValidationBehavior(IEnumerable<IValidator<TRequest>> validators)
{
	_validators = validators;
}

public Task<TResponse> Handle(TRequest request, CancellationToken cancellationToken, RequestHandlerDelegate<TResponse> next)
{
	var context = new ValidationContext(request);
	var failures = _validators
		.Select(v => v.Validate(context))
		.SelectMany(result => result.Errors)
		.Where(f => f != null)
		.ToList();

	if (failures.Count != 0)
	{
		throw new ValidationException(failures);
	}

	return next();
}

}
in this code there in an error occur in the below line
var context = new ValidationContext(request);
to correct that it should be
var context = new ValidationContext(request);
this in couse due to cs 0305 error

public class ValidationBehavior<TRequest, TResponse> : IPipelineBehavior<TRequest, TResponse> where TRequest : IRequest
{
private readonly IEnumerable _validators;
public ValidationBehavior(IEnumerable<IValidator<TRequest>> validators)
{
	_validators = validators;
}

public Task<TResponse> Handle(TRequest request, CancellationToken cancellationToken, RequestHandlerDelegate<TResponse> next)
{
	var context = new ValidationContext(request);
	var failures = _validators
		.Select(v => v.Validate(context))
		.SelectMany(result => result.Errors)
		.Where(f => f != null)
		.ToList();

	if (failures.Count != 0)
	{
		throw new ValidationException(failures);
	}

	return next();
}

}
in this code there in an error occur in the below line
var context = new ValidationContext(request);
to correct that it should be
var context = new ValidationContext(request);
this in couse due to cs 0305 error

In your solution both line is same , By the way  i am also facing an issue on var context = new ValidationContext(request);
Please help me out on this issue

public class ValidationBehavior<TRequest, TResponse> : IPipelineBehavior<TRequest, TResponse> where TRequest : IRequest
{
private readonly IEnumerable _validators;
public ValidationBehavior(IEnumerable<IValidator<TRequest>> validators)
{
	_validators = validators;
}

public Task<TResponse> Handle(TRequest request, CancellationToken cancellationToken, RequestHandlerDelegate<TResponse> next)
{
	var context = new ValidationContext(request);
	var failures = _validators
		.Select(v => v.Validate(context))
		.SelectMany(result => result.Errors)
		.Where(f => f != null)
		.ToList();

	if (failures.Count != 0)
	{
		throw new ValidationException(failures);
	}

	return next();
}

}
in this code there in an error occur in the below line
var context = new ValidationContext(request);
to correct that it should be
var context = new ValidationContext(request);
this in couse due to cs 0305 error

Please use this:  var context = new ValidationContext(request)

yes thats right sory for my mistake

You can also use this package to have it done for you:
https://github.com/GetoXs/MediatR.Extensions.FluentValidation.AspNetCore

Now what if we want to use cache along with FluentValidation?
I proceeded according to the following instruction.
https://codewithmukesh.com/blog/caching-with-mediatr-in-aspnet-core/
Now I want to use FluentValidation before caching.
Is what I mean logical at all?


Whisper: use quick merge in prepost to support large models

Describe your changes
Based on https://github.com/microsoft/onnxruntime-extensions/pull/496, this PR uses quick_merge to merge the pre, core and post models instead of onnx.compose.merge_models which fails when the models being merged are large (>2GB).
Also perfom model check on the saved file path instead of the loaded model to avoid the same issue in Olive.
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

The original PR from onnxruntime-extension also updated the whisper example. Can you double check if we can simplify our whisper example also?

The original PR from onnxruntime-extension also updated the whisper example. Can you double check if we can simplify our whisper example also?

The other changes requires the latest code from onnxruntime-extensions.
i think we should make the other changes when onnxruntime-extensions has a new pypi release with this changes.


Unable to use the image in the docker hub private repository

I encounter a problem when deploying isvc through a custom image. When my image is stored in the public repository of docker hub, I can successfully deploy isvc. However, when my image is stored in the private repository of docker hub, kfserving will not be able to find my image. I have logged in docker hub as my own account.
The error message is as follows，
Status:
  Components:
    Predictor:
      Latest Created Revision:  custom-cifar10-beta-gpu-v1-predictor-default-lmnxd
  Conditions:
    Last Transition Time:  2021-07-28T02:47:13Z
    Message:               Configuration "custom-cifar10-beta-gpu-v1-predictor-default" does not have any ready Revision.
    Reason:                RevisionMissing
    Severity:              Info
    Status:                False
    Type:                  PredictorConfigurationReady
    Last Transition Time:  2021-07-28T02:47:13Z
    Message:               Configuration "custom-cifar10-beta-gpu-v1-predictor-default" does not have any ready Revision.
    Reason:                RevisionMissing
    Status:                False
    Type:                  PredictorReady
    Last Transition Time:  2021-07-28T02:47:13Z
    Message:               Revision "custom-cifar10-beta-gpu-v1-predictor-default-lmnxd" failed with message: Unable to fetch image "seibertron/custom-model-gpu:v1": failed to resolve image to digest: failed to fetch image information: GET https://index.docker.io/v2/seibertron/custom-model-gpu/manifests/v1: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:seibertron/custom-model-gpu Type:repository]].
    Reason:                RevisionFailed
    Severity:              Info
    Status:                False
    Type:                  PredictorRouteReady
    Last Transition Time:  2021-07-28T02:47:13Z
    Message:               Configuration "custom-cifar10-beta-gpu-v1-predictor-default" does not have any ready Revision.
    Reason:                RevisionMissing
    Status:                False
    Type:                  Ready
Events:                    <none>

Can you share the YAML?
Did you include imagePullSecrets ?

Can you share the YAML?
Did you include imagePullSecrets ?

I don't use the imagePullSecrets.
I use pythonSDK to deploy isvc. My code is as follows.
import os
import time
import logging

import kubernetes.client as k8s_client
from kubernetes.client import V1Container
from kubernetes.client import V1ResourceRequirements

from kfserving import constants
from kfserving import KFServingClient
from kfserving import V1beta1PredictorSpec
from kfserving import V1beta1TorchServeSpec
from kfserving import V1beta1InferenceService
from kfserving import V1beta1InferenceServiceSpec

KFSERVING_TEST_NAMESPACE = 'kfserving'

logging.basicConfig(level=logging.INFO)

KFServing = KFServingClient(
    config_file=os.environ.get("KUBECONFIG", "~/.kube/config"))

def test_pytorch(model_name: str, init_args: dict='{}', forward_args: dict='{}'):
    predictor = V1beta1PredictorSpec(
        max_replicas=5,
        min_replicas=1,
        timeout=0,
        pytorch=V1beta1TorchServeSpec(
            args=['--init_args', init_args, '--forward_args', forward_args],
            storage_uri='pvc://kfserving-pytorch-kubeflow/cifar10',
            model_class_name="Net",
            image='seibertron/custom-model-gpu:v1',
            image_pull_policy='IfNotPresent',
            resources=V1ResourceRequirements(
                requests={'cpu': '1', 'memory': '5Gi', 'nvidia.com/gpu': 1},
                limits={'cpu': '1', 'memory': '5Gi', 'nvidia.com/gpu': 1}
            )
        )
    )

    isvc = V1beta1InferenceService(api_version=constants.KFSERVING_V1BETA1,
                                   kind=constants.KFSERVING_KIND,
                                   metadata=k8s_client.V1ObjectMeta(
                                       name=model_name, namespace=KFSERVING_TEST_NAMESPACE),
                                   spec=V1beta1InferenceServiceSpec(predictor=predictor))

    KFServing.create(isvc)

    try:
        KFServing.wait_isvc_ready(
            model_name, namespace=KFSERVING_TEST_NAMESPACE, watch=True)

    except RuntimeError as e:
        print(KFServing.api_instance.get_namespaced_custom_object("serving.knative.dev", "v1", KFSERVING_TEST_NAMESPACE,
                                                                  "services", model_name + "-predictor-default"))
        pods = KFServing.core_api.list_namespaced_pod(KFSERVING_TEST_NAMESPACE,
                                                      label_selector='serving.kubeflow.org/inferenceservice={}'.
                                                      format(model_name))
        for pod in pods.items:
            print(pod)
        raise e

if __name__ == "__main__":
    start_time = time.time()

    test_pytorch(model_name='custom-cifar10-beta-gpu-v1-2', init_args='{}', forward_args='{}')

    end_time = time.time()

    run_time = end_time - start_time

    logging.info("run_time: %f", run_time)

I'm not familiar with the python sdk.
You should try adding image_pull_secrets like this
pytorch=V1beta1TorchServeSpec(
            args=['--init_args', init_args, '--forward_args', forward_args],
            storage_uri='pvc://kfserving-pytorch-kubeflow/cifar10',
            model_class_name="Net",
            image='seibertron/custom-model-gpu:v1',
            image_pull_policy='IfNotPresent',
            image_pull_secrets=[k8s_client.V1LocalObjectReference("secretName")],
            resources=V1ResourceRequirements(
                requests={'cpu': '1', 'memory': '5Gi', 'nvidia.com/gpu': 1},
                limits={'cpu': '1', 'memory': '5Gi', 'nvidia.com/gpu': 1}
            )
        )

@markwinter Are you interested in adding a doc for how to use image pull secret for private register? Alternatively I think you can attach the secret to the default service account if you do not want to specify that for each service see https://knative.dev/docs/developer/serving/deploying-from-private-registry.

@yuzisun Sure I can add that

@markwinter Do you know how to configure a Harbor's imagePullSecrets? My private registry is based on Harbor server.


IIIF Viewer

@mjhea0 I'd love to get this merged in.  Can you resolved the conflicts when you have some time?

@paltman merge conflicts are resolved

@paltman reviewed and merged https://github.com/eldarion/scaife-skeleton/pull/40


[Feature][Project] Change repository name from incubator-seatunnel to seatunnel

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Since 2023.05.18, SeaTunnel has graduated and becomes the top level projcet. So the repository name removed the incubator prefix, so we should update the repository name in whole project files.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

let me realize it

Hello! I would like to take up this issue.

Has this issue been completed? @alibabaMapengfei

Has this issue been completed? @alibabaMapengfei

yeah,i have been completed

Has this issue been completed? @alibabaMapengfei

yeah,i have been completed

OK I will close this issue.


Cannot click next when coming back to previous step

Which version of kubefirst are you using?
2.3.7
Which cloud provider?
None specific
Which DNS?
None specific
Which installation type?
Marketplace, UI (Console app)
Which distributed Git provider?
None specific
Did you use a fork of gitops-template?
No
Which Operating System?
None specific
What is the issue?
I don't know if it's like that on all steps, but when I filled step 2, go to 3, but go back to 2 for whatever reason, even if everything was filled correctly, and still is, I cannot click next to go move forward, I have to retype something in textboxes or like I did in this recording, select the GH org again so it become available. It should be available since all information is correct.
https://github.com/kubefirst/kubefirst/assets/1170989/2c47695c-7c78-474e-869a-978afb4180e9
Code of Conduct

[X] I agree to follow this project's Code of Conduct

confirmed in this PR that field validation on navigation back and forth is working as expected. PR


Vector instructions weight

Hi,
is there a way to set the weight of the vector instructions in the generator ?
Thanks.

You mean increase the weight of overall vector instructions compared with other instructions, or fine grained weight control of each vector instruction.  It's not supported yet, but I could add some knob to control the weight of each ISA extension group.

I was meaning the first thing you mentioned, a knob to control the weight of each ISA extension group.
Thanks


drop python 3.6 support?

Reasons in favour of dropping Python 3.6 before our first release:

Python 3.6 is in maintenace mode and will reach "end of life" this year, with only security fixes being considered for the final patch release scheduled for December 2021 (https://www.python.org/dev/peps/pep-0494/).
We'll drop support in the future anyhow.
Python 3.7 supports dataclasses, which can replace attrs, thereby reducing the number of dependencies. This will facilitate some convergence between the Demes reference implementation in the spec repo and the officially supported demes-python implementation.

Reasons against dropping Python 3.6:

This enforces a minimum version on any users of Demes. In particular, this may make developers reluctant to introduce a demes-python dependency.
attrs works, and there's no immediate reason to convert to dataclasses (development effort would be better spent elsewhere).

So, on balance, I'm in favour of keeping python 3.6 support for our first release(s).

Agreed, let's keep py36

Agreed, let's keep py36

Also, data classes don't really replace attrs, right?  The latter gives validators, easy conversion to dict, etc..

Also, data classes don't really replace attrs, right?  The latter gives validators, easy conversion to dict, etc..

Yeah, there's some stuff that attrs does quite nicely that dataclasses doesn't have. But probably nothing really essential for us (we could implement things slightly differently).

Yeah, there's some stuff that attrs does quite nicely that dataclasses doesn't have. But probably nothing really essential for us (we could implement things slightly differently).

We have multiple downstream applications that want Python 3.6 support, so we'll likely keep this for the forseeable future.


⚠ Bofrost.at has degraded performance

In ae211c1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2157 ms

Resolved: Bofrost.at performance has improved in 6c5e826 after 6 minutes.


Fix typecheck

TypeScript doesn't like dynamic object extension, but doesn't complain about Object.assign().


Multi target for 8, 9 and 10

We are using this package in one of our solutions and we are updating to 9. Is there a 9 version available or would you like us to multi target this one?

Hi @hifi-phil - I'm afraid we haven't yet got around to updating this one to support Umbraco 9, but as you may have seen, we've adopted the multi-targeting approach for the other small integration projects we've built over the past months.  So yes, we'd suggest and would expect to do the same for this one.
As I say though, we haven't got to it yet, so if you had a need for this on Umbraco 9 and were minded to do it and submit it as a PR, we'd certainly appreciate it and arrange an updated release.  Thanks very much if so. // @acoumb.

Hi @AndyButland, should be able to take a look this week. Is there a style you would like me to follow, maybe point me to another repo?

There's an example in this repo, if you see the Zapier integration found here.
Would suggest it's also worth a read of this blog post, and the video embedded within, to get an idea of how we've used the multi-targeting approach to support Umbraco 8 and 9 in the same project.

Hi @hifi-phil , we have also prepared new versions of Hubspot and Zapier , you can check those also for reference.
I will close this now, but feel free to reach out if you have other questions.


Sprint plan 3 for goal Image Management, Q3 2023

Where are we now in relation to our goal?
We were able to deploy interventions to facilitate the usage of custom images in selected communities but the results of interventions usage surfaced bugs located in upstream projects.
What is the biggest uncertainty or obstacle to reaching our goal?
The bigger obstacle is being able to see the results of our interventions in the very same sprint.
Additionally, we are usually overambitious about the things we plan to do.
What should we do in the next sprint to learn and make progress?
Make sure the intervention we did actually works (fix upstream issues) and deploy it to start getting feedback.
Measurement of usage of custom images (Grafana graph to measure it already exists).
Create an actual flowchart to use existing and/or build your own image.
Carryovers from the previous sprints.
What do we expect to happen (hypothesis)?

Sprint plan for this sprint
- [ ] https://github.com/jupyterhub/kubespawner/issues/758
- [ ] https://github.com/2i2c-org/infrastructure/issues/2887
- [ ] https://github.com/2i2c-org/infrastructure/issues/2869
- [ ] https://github.com/2i2c-org/infrastructure/issues/2875
- [ ] https://github.com/2i2c-org/infrastructure/issues/2873
- [ ] https://github.com/2i2c-org/infrastructure/issues/2884
- [ ] https://github.com/2i2c-org/infrastructure/issues/2793
- [ ] https://github.com/2i2c-org/infrastructure/issues/2963

What actually happened, and what did we learn? Did we confirm our hypothesis?

@consideRatio, can you do a first push on https://github.com/jupyterhub/kubespawner/issues/758 before your away time?
Then, I will ask @GeorgianaElena and @yuvipanda to follow up with this one next week.

#2887, #2869 and #2875 depends on the upstream fix referenced above, so I am not yet assigning some of issues until we have more clarity on the upstream resolution.

@jmunroe, I think you opened some new issues some hours ago (I have not read them yet) that might capture better little improvements we might want to do on the community side. If you think we should add some of them to the sprint, let me know and we can discuss it. I want to make sure we are not at overcapacity so open to removing something from the actual plan if we needed to move forward with other pieces.


Create LCU Students and Faculty

The university official website URL:    lcu.edu.ng
URL of a page on the official website where a long-term (>1 year) IT related course is offered by the university :  https://lcu.edu.ng/index.php/all-courses

The university official website URL: lcu.edu.ng
URL of a page on the official website where a long-term (>1 year) IT related course is offered by the university : https://lcu.edu.ng/index.php/all-courses


fixing multi-cache testing of ttl wrap function

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[ ] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
fixing multi-cache testing of ttl wrap function

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2f6d831) 100.00% compared to head (b8de5e6) 100.00%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master      #636   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          320       320
  Branches        79        79
=========================================
  Hits           320       320

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


add font, change font/styling

bro unacheki aje hii font?
alafu hii about itabidi imeredoiwa...working on it..instead of the whole textblock being a , tunaweza weka zikue

Bro, am working on those conflicts.

LGTM now.


Plan for InfoPath Forms Services

Hi,
I'm just curious to know if there is any plan to add a resource to be able to configure InfoPath Forms Services.
Thanks
Dave Tremblay

@wickedave Great question - so as far as I'm aware we already have the required resources to enable the forms to be published (SPStateServiceApp will provision the state service app that is needed, and then SPFeature to enable the "Enterprise Site Collection Features" feature which should enable you to publish the forms to that site collection.
In terms of a resource that will actually publish a specific form to SharePoint, we won't be including this as it falls more in to the "content" category rather than "configuration", and we want to avoid scenarios where DSC gets too involved in that side of things. Now that said it might be possible to write a resource to do this, it's just not something that we would look to include in SharePointDsc
Just while we're here though - @ykuijs @ronx - should we capture a page in our wiki to document how to configure a SharePoint farm to use InfoPath using the resources? This is the first ask for it I've had at all so it hasn't crossed my mind until this point.

@BrianFarnhill Perhaps I wasn't clear enough in my question. I'm talking about some settings that are available in the Central Admin through the "General Application Settings" section. There's a link there called "Configure InfoPath Forms Services". We are changing some settings there and I wanted to know if it will be managed by DSC one day.

@wickedave Ahhh ok - so at this point we have no plans to include those resources, but I also don't see any reason that we can't put this on the backlog and see if we can get someone to pick it up.


Support for unsigned_long Type in Elasticsearch Mapping

While creating migrations, we often utilize the long type for numeric fields. However, it seems there is no unsigned_long type option available in the mapping configurations. This feature would be highly beneficial, especially for cases where we only deal with positive numeric values and need to maximize the range of values stored. Thanks.
https://www.elastic.co/guide/en/elasticsearch/reference/current/number.html

Hey @brfuat9, thanks for the feature request!
I've added support in elastic-adapter v4.1.0. Please update composer dependencies to apply the changes, and feel free to reopen this issue in case something doesn't work 🙂


Fix description of RTM_RETIRED.ABORTED_EVENTS

https://github.com/intel/perfmon/blob/1424ebf3be6f8f3c515fe4296ed35b3c3b19ad39/SPR/events/sapphirerapids_core.json#L6693-L6697
The description refer to "previous 4 categories" while there are only 3 listed.
@andikleen
@edwarddavidbaker

Description update will be added to RTM_RETIRED.ABORTED_EVENTS


UriHelper cannot be used to navigate to non-Blazor URIs in server-side

Repro
Create a new server-side Blazor app (with auth).
Try to use IUriHelper.NavigateTo("Identity/Account/Login") (make a button).
It will go to the Not Found page instead of doing an actual navigation. However, making an <a href> with the same URL will navigate to the login page.
Using preview 6

Does the Router have a NotFoundContent set? We specifically choose to show you the NotFoundContent if the navigation was started programatically (via NavigateTo) and the route that could not be addressed.
The pivot here is that NavigateTo is meant for navigating within the component route-space. You could use NavigateTo("...", force) to force a browser redirect without the router being involved.

Ah sorry. I'm a blazor noob. Thanks for the help.


[DOC] Document the archival blockchain data backup sources

Is your feature request related to a problem? Please describe.
There are backups provided by various providers and we need to make them discoverable:

Allnodes: https://www.publicnode.com/snapshots#near
Pagoda: https://near-nodes.io/intro/node-data-snapshots

Describe the solution you'd like
I think RPC and "Data on Blockchain" pages can be extended with this information

Add a section to RPC with note on archival nodes and link to this backup @garikbesson


Increase slurm_mem allocation for MPI AMIP FINE test

Purpose
An initial solution to #192
Content
The slurm_mem allocation for the MPI AMIP FINE test is increased to 80GB. This is probably not the best long-term solution, but will be helpful for now to fix the coupler CI.
Update: also ensures NCDatasets are always closed after being opened, to prevent memory leaks.

[x] I have read and checked the items on the review checklist.

bors r+


Fix version checker in installer_funcs.bash

The version checker in installer_funcs.bash isn't working with 3.x.x versions:
Checking for CMake >=  2.8.11
DEBUG: check_version_with $1='cmake'
min_version = [2.8.11]
max_version = [3.3.2]
DEBUG: check_version_helper $1='3.3.2' $2='2.8.11' $3='3.3.2'
DEBUG: check_version_between $1='3.3.2' $2='2.8.11' $3='3.3.2'
The return value from the call to helper function check_version_helper is 1
The detected version of cmake [3.3.2] is not between [2.8.11] and [3.3.2]
*******************************
Setting up CMake 3.2.3
*******************************

@doutriaux1 a candidate for closing?

yes it is actually already fixed and now deprecated anyway...


cargo-web 0.6.26  rustc 1.60  [Release flag is ignored]

cargo web build --release
warning: the --release flag should not be specified with the --profile flag
The --release flag will be ignored.

I'm facing the same issue.