Suggestion: run emulator in web worker

a good idea would be to run v86.wasm in a web worker process. this would isolate the emulator process from the browser and could provide faster runtimes.

You can see this example:
https://github.com/copy/v86/blob/master/examples/worker.html

It was implement a long time ago, but I would indeed like to revisit it. PRs welcome.
Currently has some known issues: https://github.com/copy/v86/issues/219


phone is not displayed

phone is not displayed but  contact:phone is.
Examples:

https://healthsites.io/map#!/locality/node/7211390276 phone=+33 3 84 25 40 92
https://healthsites.io/map#!/locality/node/2140012373 contact:phone=+33 4 50 38 09 51

See also #1292

could you check here please @markherringer @timlinux ?

Thank You @Irwan Fathurrahman<EMAIL_ADDRESS>I can confirm that
https://healthsites.io/map#!/locality/node/2140012373
contact:phone=+33
4 50 38 09 51
..is displayed
[image: healthsites-number.png]
ᐧ
On Tue, 31 Mar 2020 at 08:00, Irwan Fathurrahman<EMAIL_ADDRESS>wrote:

could you check here please @markherringer
https://github.com/markherringer @timlinux https://github.com/timlinux
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/healthsites/healthsites/issues/1368#issuecomment-606417859,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAJPNSN4PX2KWCGZSKZOJTRKGBHJANCNFSM4LQ7LKVA
.

--
Kind regards
Mark Herringer
www.healthsites.io
https://medium.com/healthsites-io
@sharehealthdata https://twitter.com/sharehealthdata

I can confirm that phone=+33 3 84 25 40 92 is not (yet) displayed

https://healthsites.io/map#!/locality/node/7211390276

I think that you could map several osm tags to one postgresql field.


Wrong words - Application setup area

Step 5
On the Register an application page, under Supported account types, Select Accounts in any organizational directly (Any Azure AD directory - Multitenant).
after organizational .... the word directly needs to be updated.
Step 13
In the Request API permissions windows, select APIs my organization users, and search for office 365 exchange online, select it.
after the word organization > users should be uses.

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

ID: c61c960c-f5c3-76f7-ac18-dfeaf30230e4
Version Independent ID: 3dc607b2-869e-976f-80da-d73cfb1210e5
Content: Cross-tenant mailbox migration - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/cross-tenant-mailbox-migration.md
Product: microsoft-365-enterprise
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@Karr Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/7472
Thanks
Sri


Automatically parse request object

Be able to send the request object so body and urlParams can be directly parsed instead of explicitly having to set them.

This is in working progress status. In this branch https://github.com/kevinccbsg/config-req/tree/req-options 👍
It needs more documentation and better testing.

That's great!!! Will it be merged to master or will it live as an independent branch?

It will be merged with the master branch.

This was merged @inigomarquinez. I'd recommend checking this https://github.com/kevinccbsg/config-req#use-with-req-complete-object to use in your projects until V2 will be released.


Add 'utf-8' support

Hello,
Sometimes, we might have chinese character in the ldap config, like CN=xiaohan.shi, OU=用户
So, Maybe you should do some changes to the code to support chinese character.
# encoding= utf8
import sys
reload(sys)
sys.setdefaultencoding("utf-8")

@xiaohanshi is it a theoretical problem or do you have a stacktrace to show?

As you can see, my config.py of LDAP contains chinese character

When i am trying to login in via LDAP, i meet the problem below

So, i add some codes to views.py and i solves the problem

@xrmx
Another error we have found:

http://stackoverflow.com/questions/3828723/why-should-we-not-use-sys-setdefaultencodingutf-8-in-a-py-script
export PYTHONIOENCODING=UTF-8


[Feature] jvm-package of LightGBM

So far LightGBM does not offer a JVM integration like https://github.com/dmlc/xgboost/tree/master/jvm-packages What is the current state? Is anyone working on a JVM integration?
Maybe https://github.com/java-native-access/jna starting out with https://code.google.com/archive/p/jnaerator/ would be an interesting start.
https://github.com/Microsoft/LightGBM/blob/master/python-package/lightgbm/basic.py seems to define all the C interactions which are outlined in https://github.com/Microsoft/LightGBM/blob/master/include/LightGBM/c_api.h and could be used as input for jnaerator.

Here https://github.com/Microsoft/LightGBM/wiki/Features they say Spark support is planned. A XGBoost4J alternative would certainly be nice.

Just out of curiosity, why was the issue closed even though it was labelled as call-for-contribution? Is JVM support no longer in the roadmap?

@superbobry refer to https://github.com/Microsoft/LightGBM/issues/909

@superbobry I've checked in the code to generate SWIG Java wrappers to LightGBM
To build, you just need to run:
mkdir build ; cd build
cmake -DUSE_SWIG=ON ..
make -j4
The jar file is also available in maven central:
https://repo.maven.apache.org/maven2/com/microsoft/ml/lightgbm/lightgbmlib/
You can import it with sbt via:
"com.microsoft.ml.lightgbm" % "lightgbmlib" % "2.0.120"
If you have any suggestions for how to improve the SWIG wrappers please let me know.

@imatiach-msft catch an exception like java.lang.ClassNotFoundException: com.microsoft.ml.lightgbm.SWIGTYPE_p_void
spark-shell --jars mmlspark-0.12.jar,lightgbmlib-2.0.120.jar

traning model
import com.microsoft.ml.spark.LightGBMClassifier
val lgbm = new LightGBMClassifier().setLabelCol("dep_delayed_15min").setFeaturesCol("newColumn").setRawPredictionCol("rawPrediction").setNumLeaves(5).setNumIterations(10)
val lgbmModel = lgbm.fit(tmpData)


Conditional reference while processing transaction bundle

Is your feature request related to a problem? Please describe.
When processing transactions, servers SHALL:
-check all references for search URIs
-For search URIs, use the search to locate matching resources
-if there are no matches, or multiple matches, the transaction fails, and an error is returned to the user
-if there is a single match, the server replaces the search URI with a reference to the matching resource
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
See "Conditional references" under https://www.hl7.org/fhir/http.html#trules

This is an enhancement request, I've moved the text to the formatted template.

should benefit from #1353

I just discovered that after switching to FHIR 4.7.1 all of our Synthea-generated data no longer loads.  They are all being flagged as invalid because they use conditional references for the Practitioner field.  I have disabled validation for our use, but it would be appreciated if this were not necessary by supporting conditional references.

Hi @sridhar-shivraj The enhancement will be in version 4.8.0  Please let us know if you have any feedback.  We appreciate it.


Add documentation for project outside react or javascript

Subject
using chakra-ui outside typescript/react project
Description
So I have upcoming project using rust's webassembly. I found that compiling chakra-ui using rollup really uncomfortable, so I might asking if it is possible to add the documentation on how to use chakra-ui? There is some request to add it into cdn like in here: https://github.com/chakra-ui/chakra-ui/issues/341 so I can just include in my index.html
This is some example on my wasm project: https://github.com/Rizary/dominator-site

We haven't done any testing on Chakra UI in Web Assembly, and it's not a priority right now, so I'm closing this issue.


Need a clear example to use streamlit for tasks, chain with example prompt

What piece of documentation is affected?
Need an example each for task, chat, chain etc. to work on streamlit.
What part(s) of the article would you like to see updated?
examples in streamlit (samples)
Additional Information
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

Also, with the new git clone today, in streamlit the main is not working

What piece of documentation is affected?
Need an example each for task, chat, chain etc. to work on streamlit.
What part(s) of the article would you like to see updated?
examples in streamlit (samples)
Additional Information
No response
Acknowledgements

[x]  My issue title is concise, descriptive, and in title casing.
[x]  I have searched the existing issues to make sure this feature has not been requested yet.
[x]  I have provided enough information for the maintainers to understand and evaluate this request.

The documentation that we do have can be found here: https://josh-xt.github.io/AGiXT/
I would recommend giving the core concepts a good read over to get a better idea of what these features do and their purposes. I've also added another chain for writing a poem and saving it to a file in the latest version to help give an example.
Top priority has been working through the issues as they come up and making high value improvements to the software.  Anyone is welcome to contribute better documentation or examples.

Anyone is welcome to contribute better documentation or examples.

Nobody can contribute if nobody can figure out how to make this work at all. There having absolutely 0 youtube videos showing/mentioning/clickbaiting AGiXT in ANY way should be a huge red flag on the state of accessibility.

Anyone is welcome to contribute better documentation or examples.

Nobody can contribute if nobody can figure out how to make this work at all. There having absolutely 0 youtube videos showing/mentioning/clickbaiting AGiXT in ANY way should be a huge red flag on the state of accessibility.

People have been contributing, some people are able to go read our documentation to understand the program, if that is insufficient, understand that this is a one man show where I get the help from a few people here and there. Sponsorships to increase the amount of time I'm able to spend on the project would help here. I'm a software engineer, not a video creator or documentation expert. I suggested contributions because that is not my wheel house.

Anyone is welcome to contribute better documentation or examples.

Nobody can contribute if nobody can figure out how to make this work at all. There having absolutely 0 youtube videos showing/mentioning/clickbaiting AGiXT in ANY way should be a huge red flag on the state of accessibility.

Also, no one is forcing anyone's hand. If you see it as a red flag and can't go read the code, then it may just not be for you.


Make clear the search is using google

So people don't get surprised when they are suddenly on a different webpage when searching for something.

I was very surprised!


Post release automated changes for devcenter releases

Post release automated changes for azure-rest-developer-devcenter

API change check
API changes are not detected in this pull request.


Skip retries for known failures

When making a network request, Net::HTTP::NotFound is never going to spontaneously start working on retry. Likewise TooManyRedirects, Forbidden, and probably others I'm not thinking of. We should not retry those responses.

Any other exception classes to not retry on? There are a few Error types that seem applicable that aren't currently on the list, but they were added in relatively recent versions of the Ruby core lib so adding them to the list throws errors in the Travis builds for the earlier rubies that wouldn't have those Error classes in core lib yet.

@RochesterinNYC we can always do ERRORS << NewExceptionClass if defined?(NewExceptionClass)

Actually I ran into this today, I think due to the Rubygems move to Fastly.
For a period of time today gem downloads were sporadically failing with 404s. E.g.:
Not Found 404 (https://rubygems.global.ssl.fastly.net/gems/devise-3.5.3.gem)

Despite my retry setting, Bundler did not retry, due to the recently merged #4155.
I think this is evidence that "once a 404, always a 404" is not necessarily the case. Also, maybe Bundler should retry on 502 as well?

Actually I ran into this today, I think due to the Rubygems move to Fastly.
For a period of time today gem downloads were sporadically failing with 404s. E.g.:
Not Found 404 (https://rubygems.global.ssl.fastly.net/gems/devise-3.5.3.gem)

Despite my retry setting, Bundler did not retry, due to the recently merged #4155.
I think this is evidence that "once a 404, always a 404" is not necessarily the case. Also, maybe Bundler should retry on 502 as well?

@mattbrictson if Fastly returns a 404 due to a system error, there is no reason to think that retrying will succeed, though. You'll have to wait until the system issue is resolved before it'll work, and mechanically trying over and over doesn't seem like a good way to check for that...

@mattbrictson if Fastly returns a 404 due to a system error, there is no reason to think that retrying will succeed, though. You'll have to wait until the system issue is resolved before it'll work, and mechanically trying over and over doesn't seem like a good way to check for that...

if Fastly returns a 404 due to a system error, there is no reason to think that retrying will succeed

@indirect Fair enough. But isn't that true of all HTTP errors other than perhaps a 503 with an accompanying Retry-After header?
I'm not intending to be pedantic; I am just curious to know what use case --retry is intended to solve. Is it mostly for network issues (socket timeout, connection dropped, etc.) and not for HTTP errors?

@mattbrictson I believe that is the main intention, yes.

@mattbrictson yes, it's primarily for network errors and server 500 errors that might be transient or limited to a single backend.

OK, makes sense. Thanks.

closing this as fixed by @RochesterinNYC's recent work


kpt doesn't work with symlinks

Expected behavior
When working in a symlinked directory, kpt should behave normally and have commands process properly.
Actual behavior
kpt commands fail unpredictably when working in symlinked directories.
Information
v0.38.1
Steps to reproduce the behavior

Get a package: kpt pkg get https://github.com/GoogleContainerTools/kpt/package-examples/helloworld-set hello2
Create a symlink to the package: ln -s hello2/ hello3
Try various kpt commands and see they return no output. (Ex. kpt cfg tree hello3/)

We will see what it takes to get this to work but setting the priority to p2 for now since I have not seen this widely reported and I don't know if customers have mentioned this.

I am seeing this issue as well, and it has become a blocker for us when trying to use kpt with an existing monorepo.
kpt pkg get<EMAIL_ADDRESS>Package "v0.1":
Fetching /Users/me/src/myproj@master
From /Users/me/src/myproj
 * branch            master     -> FETCH_HEAD
 + e2c7f6c...23411f2 master     -> origin/master  (forced update)
Error: error copying package: read /Users/me/.kpt/repos/f5axgwlsomxxg4dfoz2s693smmxw64dt2qryzwcpacbbj3mabgmozbncpy======/bbb/xyz: is a directory

Addressing this TODO would fix the issue in our particular case, as the directory containing the symlink is external to where the kpt package is located.
https://github.com/GoogleContainerTools/kpt/blob/cbd342d350b88677e522bf0d9faa0675edb8bbc1/internal/util/fetch/fetch.go#L195-L196

@mikebz Can we increase priority on this. Another customer has run into it.

So I think this feature requires more than just making sure kpt follows symlinks. Some questions we should answer:

Are there any constraints on following symlinks and what are the security implications? Symlinks can point to anywhere on the users filesystem, so it is not clear to me if we should blindly follow all symlinks. In particular since users might fetch a package with symlinks from someone else and might not be aware that there are symlinks pointing outside the package.
How should we handle broken symlinks?
What does this mean for support on Windows and other filesystems that doesn't support symlinks?
Git treatment of symlinks is actually dependent on a configuration option: https://git-scm.com/docs/git-config#Documentation/git-config.txt-coresymlinks. Is this something kpt should check if encountering symlinks?

I am writing up a doc to discuss the scope and possible solutions for this problem.

@phanimarupaka as @mortent has already mentioned the reasons that makes this problem lot harder.
Resolving the arg for the top level pkg dir (if its a symlink) in kpt commands is the best we can do at this point.
something related: in future, we want to support other location targets for packages such as OCI artifacts, GCS storage buckets and use of symlinks in the package makes it less portable.

This feature will be supported in beta.5 release of kpt.

@phanimarupaka Are you sure this is supported now? I'm on kpt 1.0.0-beta.14 and symlinks still appear to be ignored. Here's an example:
caleb@work-vmware> kpt pkg tree
Package "test1"
└── [Kptfile]  Kptfile kptfile
caleb@work-vmware> ln -s ../../base/ ./base
caleb@work-vmware> kpt pkg tree
Package "test1"
└── [Kptfile]  Kptfile kptfile
caleb@work-vmware> rm base
caleb@work-vmware> cp -r ../../base/ base
caleb@work-vmware> kpt pkg tree
Package "test1"
├── [Kptfile]  Kptfile kptfile
└── Package "base"
    ├── [Kptfile]  Kptfile kptfile
    ├── [cluster-autoscaler.yaml]  ClusterRole cluster-autoscaler
    ├── [cluster-autoscaler.yaml]  ClusterRoleBinding cluster-autoscaler
    ├── [cluster-autoscaler.yaml]  Deployment kube-system/cluster-autoscaler
    ├── [cluster-autoscaler.yaml]  Role kube-system/cluster-autoscaler
    ├── [cluster-autoscaler.yaml]  RoleBinding kube-system/cluster-autoscaler
    ├── [kpt-annotations.yaml]  ConfigMap kpt-annotations
    └── [kpt-labels.yaml]  ConfigMap kpt-labels
caleb@work-vmware> kpt version
1.0.0-beta.14

cc @droot

@phanimarupaka @droot any update on support for this? The lack of symlink support makes it difficult to have different Kptfiles that rely on a common base.
If you do not plan to support symlinks, how do you advise setting up environment-specific Kptfiles (e.g. separate Kptfiles for dev/stg/prd) that each rely on a common base configuration?


Fix grammar in Container Tools Guide

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>Thanks to @fatherlinux 's eagle eye at spotting them, I've squashed some grammar issues in the Container Tool Guide.

LGTM
@rh-atomic-bot r+


Correct broken link to quick start guide

Similarly to https://github.com/notaryproject/notaryproject.dev/issues/360,
I attempted to follow the link in the README.md for the quick start guide and discovered the broken link. Linking to the official page.
URL: https://notaryproject.dev/docs/quickstart/
Expected page: https://notaryproject.dev/docs/quickstart-guides/quickstart-sign-image-artifact/

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

Comparison is base (f8aab15) 64.05% compared to head (4a59209) 64.05%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #831   +/-   ##
=======================================
  Coverage   64.05%   64.05%
=======================================
  Files          40       40
  Lines        2323     2323
=======================================
  Hits         1488     1488
  Misses        709      709
  Partials      126      126

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

@rcrozean can you please sign your commit?
https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

Commit signed!


GPU device not found on OSX

Hello,
I'm trying to install TensorFlow with GPU support for a CUDA Capability 3.0 device. I have followed the instructions found here: https://gist.github.com/Mistobaan/dd32287eeb6859c6668d, and everything compiles without error, but TensorFlow doesn't recognize my GPU device. The steps that I'm taking are below, as is the error output.
I've modified bazel to use gcc-4.8, so these are the commands I'm running:
bazel build -c opt --config=cuda //tensorflow/cc:tutorials_example_trainer ---crosstool_top=//tools/cpp:toolchain
bazel build -c opt --config=cuda //tensorflow/tools/pip_package:build_pip_package --crosstool_top=//tools/cpp:toolchain

Running the example with:
bazel-bin/tensorflow/cc/tutorials_example_trainer --use_gpu

produces the following output:
F tensorflow/cc/tutorials/example_trainer.cc:118] Check failed: ::tensorflow::Status::OK() == (session->Run({{"x", x}}, {"y:0", "y_normalized:0"}, {}, &outputs)) (OK vs. Invalid argument: Cannot assign a device to node 'Const/_2': Could not satisfy explicit device specification '/gpu:0'
     [[Node: Const/_2 = Const[dtype=DT_INT32, value=Tensor<type: int32 shape: [] values: 0>, _device="/gpu:0"]()]])
F tensorflow/cc/tutorials/example_trainer.cc:118] Check failed: ::tensorflow::Status::OK() == (session->Run({{"x", x}}, {"y:0", "y_normalized:0"}, {}, &outputs)) (OK vs. Invalid argument: Cannot assign a device to node 'Const/_2': Could not satisfy explicit device specification '/gpu:0'
     [[Node: Const/_2 = Const[dtype=DT_INT32, value=Tensor<type: in

I can also confirm that after installing the pip package and running a test script, no devices are found.
A few things to note:

Running deviceQuery confirms that a CUDA device (Device 0: GeForce GT 650M) is found.
I've made sure to run bazel clean before compiling, and have tried compiling using sudo just to see if this was a permissions issue.

Thanks in advance for your help.

It turns out that I was actually compiling with gcc-5, not gcc-4.8. From what I can tell CUDA doesn't support version 5, so this might be the problem.
I'm trying now to compile with 4.8, but I'm getting this error:
/var/tmp//cccfsYPu.s:2774:expecting string instruction after `rep'

De-duping with #491


Consistency in environment variables to reflect reality

Summary
Documentation references both SYNC_WITH_REMOTE and SYNCH_WITH_REMOTE. Code references only SYNC_WITH_REMOTE. Fix documentation to be in line with what is actually used in code.
Checks

[x] In case of new feature, add short overview in docs/<corresponding file>
[x] Tested changes

@deepseth Thanks, that must have slipped through.


Changed default "endpoint" servers order

And added "DEPRECATED" note to a "peers" option (relates to https://github.com/coreos/etcd/commit/d685135832677a8ebf97ea88fdd7ac7fe7c2aa2f)
@yichengq @yifan-gu

LGTM. Defer to @xiang90  as well.
Thanks,

@kayrus Lgtm. But please update the commit message to etcdctl: Changee default "endpoint" servers order

+1. Thank you @kayrus
(Related to docs-wide https://github.com/coreos/etcd/issues/3480)

@kayrus kindly ping.

Closed in favor of #4359 which changes commit message only.


Feature/get matrix data

Hi @hdbeukel can you please check why travis is failing? It seems to be killed, but I am not sure why?

I reran the checks but they were killed on the same test so I will try to find out why.

Strange, the checks run fine on my laptop...

It seems to be a Travis issue which you may be able to solve by increasing the memory of the VM used by the surefire plugin. See

http://stackoverflow.com/questions/23260057/the-forked-vm-terminated-without-saying-properly-goodbye-vm-crash-or-system-exi
https://issues.jboss.org/browse/RF-13996

It also compile with no issues on my laptop. I will look into the memory issues.

It all seems to work now, merged in to develop.


fix(payload-handler): fixed udnef set payload bug

Codecov Report

Merging #104 (21bcfb5) into main (61bfb11) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   82.91%   82.95%   +0.04%
==========================================
  Files          18       18
  Lines         749      751       +2
  Branches      147      148       +1
==========================================
+ Hits          621      623       +2
  Misses        128      128

Impacted Files
Coverage Δ

src/instrumentations/utils/PayloadHandler.ts
92.59% <100.00%> (+0.59%)
:arrow_up:


fix(ag-grid-styles): fix issue with hovering on custom tooltips

closes: #1148
Component Development

Type

[ ] Feature
[x] Fix
[ ] Hotfix (should release ASAP)
[ ] Maintenance (deps only)

Reference to assignment

Description of assignment

Description of Proposed Changes

Checklist
Create User Story(DevOps) or Issue(Github)

[ ] Describe functional requirements
[ ] Design plan/suggestion (e.g. images, link to eds design or relevant figma links, mention of eds tokens to use)
[ ] Communication with UX (evaluate if needed)

Development

[ ] Describe the component
[ ] Code funtional requirements
[ ] Create stories
[ ] Create snapshots (evaluate if needed)
[ ] Remove unused imports and commented code
[ ] Lint code with EsLint (and follow best practises for code quality

Create Pull Request

[ ] Assign relevant reviewers in Github and Cromatic
[ ] Inform colleagues of pending reviews. We use Teams for this purpose, under fusion-frontend team please tag involved reviewers in the Code review channel

Review of Pull Request

[ ] Follow up on comments from code review and implement necessary changes (evaluate if needed)
[ ] Review approved in github (and chromatic)

Publish Code

[ ] Increment version number
[ ] Merge PR
[ ] Consider letting fusion developers know this this shiny new thing exists

https://github.com/equinor/fusion-react-components/issues/1148#issuecomment-1886416783


official unc0ver link & cydia impactor method

Thanks for the pull request. I've decided to stick with guide links for unc0ver for now. I think the github page might be a bit overwhelming for first time users.


tundra.syntax.install broken

When using the install syntax extension I get an error message (it seems like it was not updated when an API breaking change was done before.). I am trying to use this feature for copying DLL's of third party dependencies into the output directory. Does anyone have a workaround for this?
...ed\Documents\projects\tundra\scripts\tundra\boot.lua:23: Sources: source_list must provide ExtensionKey
stack traceback:
        [C]: in function 'error'
        ...ed\Documents\projects\tundra\scripts\tundra\boot.lua:23: in function 'errorf'
        ...Documents\projects\tundra\scripts\tundra\nodegen.lua:686: in function 'add_evaluator'
        ...ts\projects\tundra\scripts\tundra\syntax\install.lua:40: in main chunk
        [C]: in function 'require'
        [string "tundra.lua"]:2: in main chunk
        [C]: in function 'xpcall'
        ...cuments\projects\tundra\scripts\tundra\buildfile.lua:242: in function 'run'
        ...ed\Documents\projects\tundra\scripts\tundra\boot.lua:93: in function 'generate_dag_data'
        ...ers\Ted\Documents\projects\tundra\scripts\tundra.lua:8: in function 'action'
        ...ers\Ted\Documents\projects\tundra\scripts\tundra.lua:38: in function <...ers\Ted\Documents\projects\tundra\scripts\tundra.lua:22>
        (tail call): ?

You're right this is broken. The problem the extension key stuff is solving is that of filtering out unwanted input files by only picking up certain extensions. For example the C++ unit types use this to only pick up .cpp, .h and similar files and not every random file produced by subtargets like .pdb. In this case maybe the copy builder does want everything. I'm not sure how to best convey that, but it sounds like it will need a code patch. A hack might be to create a env variable EVERY_FILE = { "" } and then specify ExtensionKey="EVERY_FILE" in the install rule. That could work if the rest of the code accepts an empty string as matching every "extension".

I was looking at this again today and I made a patch that removes the need for ExtensionKey ( accepts all suffixes when ExtensionKey is nil ).
teddemunnik/tundra@f19b52f512c25a34f24a228e7a376ce415087161
However it looks like there are some API changes that the install syntax was not updated for, which I currently do not have enough understanding of to attempt to fix:

There is no way to give the Install node a name, but the code expects a name parameter. This possibly also causes a duplicate guid error on building.
files.copy_file no longer exists.

Is there another way to copy DLL's into the target directory that I'm missing, which supports separate paths for 32 and 64 bit builds?

OK; that's going to require a bit more fixup.
If the DLLs are built with Tundra you can override Target on them to redirect where they go. If not, you'll need something like CopyFile - you might have more luck with that than Install, the latter probably hasn't been used since after the API changes which is why it's broken. CopyFile has an example (I just tested it) so that seems workable: examples/filecopy


Fixed bags(Sending empty message, Text output for messages)

Fixed a bug with the text going beyond the message, sending an empty message

CR Done


Add a contribution guide to the php-runtimes refactor

This PR will add a contributing.md file to the refactor branch of php-runtimes detailing how to contribute to the repository

@PineappleIOnic  didn't we have another PR for this which was already reviewed and merged ?

@PineappleIOnic  is this still relevant after we migrated the contributing guide over to open-runtimes?
https://github.com/open-runtimes/open-runtimes/blob/main/docs/add-runtime.md


Follow Me/Followers: add inherit mode for dynamic templating

We received this piece of feedback from our friends at Woo:

A site owner should be able to configure a Widget so that the Widget tries to extract the correct user from the page it is on. This way it could be used in Post-Footers or Author-Page-Headers

This accomplishes that goal.
Proposed changes:

Allow setting an inherit mode called Dynamic User on the Follow Me and Followers blocks
It shows the associated user's ActivityPub profile or followers on 1) user pages and 2) individual posts (the author's profile/followers are shown)

Other information:
It looks like this:

I haven't thoroughly tested it yet, particularly with users that exist and have posts but are disabled.

https://github.com/Automattic/wordpress-activitypub/pull/892/commits/4ade1f5863bea440798dbdde1d0a15305f8809cd tweaked the behaviour for disabled users in the Followers block, or for any other case where a user_id produces a WP_Error. The Follow Me block was already bailing on errors, so now we're consistent there too.


remove trailing / on Database Normalization url

fixes 404
thanks for the list!

Thanks for the fix!


General pagination fixes and cleanup - Fixes #29

Some is new and some is from past Sponge commits - Fixes #29

Still need to fix specific page command and miscount of line amount.

@Meronat Status?

@Cybermaxke I will get back to this soon.

Now that my API PR got in I will finish this when I wake up. I still have to fix one thing relating to counting lines, but I should be able to figure it out.

So this seems to be good in a sense of porting features over, but I posted in the #development channel on Discord about my issues with proper amount of lines. I spent quite a while on it but couldn't figure it out, but other than that this PR supports my latest API changes and ports other various SpongeCommon changes.


Name should be required field in Add Host

Rancher Version: master 11/11
Steps to Reproduce:

Add Host without Name

Results: Able to go through steps, but it will fail because it needs it.
Expected: Make Name required

Version - master 11/15 - This is "fixed" through the UI.  But we should make it required through the API as well.

I don't know the cattle api well. @cjellick, could you help with the change?

note: It's the hostname field that should be required on create; name is for the user and optional.

Name is required because docker-machine requires a name while creating the machine.

@ibuildthecloud , When "Add Host" is done form UI , in the add host API only hostname is set (name is null ). name is not a requited field in for add host API. the hostname is used when sending request to docker machine.


function_calls example fails with default max_tokens configuration

👋🏻 Thanks for this great crate. Just a heads up, the function_calls example fails with:
 Error("data did not match any variant of untagged enum ServerResponse", line: 0, column: 0) })

I traced it down to this line https://github.com/Maxuss/chatgpt_rs/blob/master/src/client.rs#L418. The debugged response was:
{
  "id": "chatcmpl-...",
  "object": "chat.completion",
  "created":<PHONE_NUMBER>,
  "model": "gpt-3.5-turbo-0613",
  "choices": [
    {
      "index": 0,
      "message": {
        "role": "assistant",
        "content": null,
        "function_call": {
          "name": "send_message",
          "arguments": "{\n  \"message\": \"This is a test"
        }
      },
      "finish_reason": "length"
    }
  ],
  "usage": {
    "prompt_tokens": 108,
    "completion_tokens": 16,
    "total_tokens": 124
  }
}

Just needed to increase my max_tokens from 16 in the default ModelConfiguration.

Latest version no longer has a default max token cap, so this issue should no longer arise by itself


Update howlongtobeat extension

Description

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Many thanks for fixing this! I had no time for it, and I'm happy someone fixed it! 🙏

Is this ready to be merged?

Is this ready to be merged?

i'm done and using the changes myself for the extension.
Maybe @Cronos87 wants to do a code review and change things ?

I can do the code review in around 2 hours. Thank you @JoeriTheGreat!

Few comments, and it's ready to merge for me. Thank you again!

@JoeriTheGreat Great! Thanks!
Could you add a changelog entry, forgot to mention that earlier 🙂

@JoeriTheGreat Great! Thanks!
Could you add a changelog entry, forgot to mention that earlier 🙂

Sure, no problem!

Thank you for the awesome work!


Barrier() Feature?

Hi, I was curious if it is possible to add a barrier function like MPI_Barrier(). Currently, it can be done by articulating a Broadcast() call.

I think doing a Broadcast is equally as cheap, given the data size is small

Sure :_D
#ifdef RABIT_RABIT_H_
namespace rabit {
  inline void Barrier(){ static float a = 1; rabit::Broadcast(&a, sizeof(float), 0); }
}
#endif

contribution is actually welcomed!


Any chance for IPv6 networking?

Microsoft Windows [Version 10.0.18999.1]
$ ping6 www.google.com
    ping6: connect: Network is unreachable

Upps..
$ ip addr show | grep inet6
    inet6 ::1/128 scope host
    inet6 fe80::215:5dff:fe40:655a/64 scope link

I have a server that's IPv6 only and I cannot access it from WSL 2

I use WSL 1 all the time with IPv6. If WSL2 does not support it, I wont be upgrading.

I've started with WSL 2 without using WSL 1 much. So maybe I'm missing something important about network configuration? But everything else is working fine.
$ uname -a
Linux SPHOME 4.19.72-microsoft-standard #1 SMP Sun Sep 15 10:03:24 UTC 2019 x86_64 GNU/Linux

IPV6 only sites and addresses are not yet supported by the Host Networking service which powers networking in WSL 2. This is a feature request we've relayed to the team, and we are tracking it on this thread: https://github.com/microsoft/WSL/issues/4518. I'll mark this thread as a duplicate and close it out, please feel free to watch and/or comment on the linked thread for further discussion. Thanks! :)


rocm 5.7: hipcc crashes if ranges is included nd g++-12 is installed

This issue is a duplicate of

ROCm-Developer-Tools/HIPCC/issues/137

that I created yesterday. But it seems this repository gets more activity.
I think this is a critical bug by the way, since it is impossible to compile a decent c++20 simple code with hipcc provided by ROCm 5.7.
If this is not the right place for this kind of issue, I'd be glad to create it elsewhere!
Thanks.

@maartenarnst will be interested in this.

Tagging @yxsamliu who can correct me if I am wrong here.
I think this is a clang issue, not a hipcc or hip runtime issue.
clang, in this case shipped with ROCm 5.7 is not compatible with libstdc++ shipped with g++-12.
If you want to use C++20 features, maybe using libc++ shipped with clang (instead of using libstdc++ shipped with gcc/g++) makes sense.
I tried building and running it with libc++ and it runs fine.
➜  build git:(amd-stg-open) cat t.cc
#include <ranges>

int main() { return 0; }
➜  build git:(amd-stg-open) hipcc t.cc -std=c++20 -stdlib=libc++ -L ./lib -o ranges
➜  build git:(amd-stg-open) ./ranges
➜  build git:(amd-stg-open)

Hi @cjatin @yxsamliu !
What's the plan for fixing this? I guess aligning the clang version shipped with hipcc to the latest available official release of clang from LLVM could do the trick.
I also have 2 very practical question:

When do you plan the next release of ROCm that would incorporate the fix?
How could we ensure this never happens again? (e.g. by adding some test somewhere)

The issue was caused by https://reviews.llvm.org/D146178. It has been reverted in the trunk but not in ROCm 5.7. I will try reverting it in ROCm 6.0. The author has added lit tests to cover the regressions caused by the patch, which should prevent them from happening again.

Please re-open the issue, if this is still seen on the latest ROCm.
Closing this, since the fix has been identified.


Add transform for System.Uri (#8)

Please have a look and let me know if I should change anything. Thanks!
I also realized that there is not a lot of type safety when it comes to these attributes - maybe it would be nice if tranforms could specify a base type they can be applied to?

Thank you very much for contribution!
Could you please do one thing: could you please specify in the description of this PR that you contribute this on Apache License 2.0 terms. This is needed to maintain license clean from different claims.
I will merge it then and run release tonight.

Sorry @danyx23, I missed your comment about type-safety of Transform.
The problem there is that Attribute (JsonField) members can't be parametrized based on type of member they are applied to (Uri). This means that there will be always possibility to wrongly apply to field incompatible by target type transform. You will get run-time exception in this case. But I do not think it's possible to provide compile-time type safety. Let me know if have any ideas there - I would be glad to experiment...

Yes you are unfortunately right - I hadn't looked into Attributes in a while and only vaguely remembered that you could restrict them but that unfortunately only by a fixed enum that describes various kinds of targets and is thus not flexible enough for further type checking. FxCop or something similar would work but that is another story. Thanks for your comments!


Fixed Service.php when I pulled from master crashed my system due to …

…the passing from Model_ServiceHosting to Model_ServiceHostingServer

@timothygwebb it appears you introduced all those changes in your previous commits that is  causing the issue you have mentioned. The Service.php file didn't have any issues and worked alright prior to that.
Check the History:
https://github.com/boxbilling/boxbilling/commits/df141caed467330cf76cf84a669ab12931f05692/src/bb-modules/Servicehosting/Service.php
I suggest you revert the file to the last stable commit https://github.com/boxbilling/boxbilling/commit/e64244ff6bf0ef8fc345d36449fb24bfc490cdcb
That way it will be easier to identify and fix any issues that were introduced.

@fordnox @timothygwebb this should not be merged yet.
I have compared the current state of the file with the last working commit https://github.com/boxbilling/boxbilling/commit/e64244ff6bf0ef8fc345d36449fb24bfc490cdcb and the file still has a lot of changes which I suspect will not make it work properly.
It should be reverted to last working state and any new features that depend on it should be rewritten to work with the file as it will break a lot of things.

@fordnox  Below is the unit test result for the ServiceHosting module for this pull request

@user /e/GitHub/PHP/boxbilling-master (Service)
$ src/bb-vendor/bin/phpunit ./tests/bb-modules/Servicehosting/
PHPUnit 9.4.2 by Sebastian Bergmann and contributors.

.............EEEE..............................................E. 65 / 88 ( 73%)
.....EEEFFE.......EEE..                                           88 / 88 (100%)

Time: 00:01.427, Memory: 12.00 MB

There were 12 errors:

1) Box\Mod\Servicehosting\Api\AdminTest::testserver_get
TypeError: Argument 1 passed to Mock_Service_7f1f218a::toHostingServerApiArray() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php on line 228

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php:228
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\Api\AdminTest.php:325

2) Box\Mod\Servicehosting\Api\AdminTest::testserver_delete
TypeError: Argument 1 passed to Mock_Service_7f1f218a::deleteServer() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php on line 246

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php:246
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\Api\AdminTest.php:356

3) Box\Mod\Servicehosting\Api\AdminTest::testserver_update
TypeError: Argument 1 passed to Mock_Service_7f1f218a::updateServer() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php on line 278

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php:278
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\Api\AdminTest.php:386

4) Box\Mod\Servicehosting\Api\AdminTest::testserver_test_connection
TypeError: Argument 1 passed to Mock_Service_7f1f218a::testConnection() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php on line 298

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Api\Admin.php:298
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\Api\AdminTest.php:416

5) Box\Mod\Servicehosting\ServiceTest::testtoApiArray
TypeError: Argument 1 passed to Box\Mod\Servicehosting\Service::toHostingServerApiArray() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php on line 449

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:465
E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:449
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:811

6) Box\Mod\Servicehosting\ServiceTest::testdeleteServer
TypeError: Argument 1 passed to Box\Mod\Servicehosting\Service::deleteServer() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 942

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:655
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:942

7) Box\Mod\Servicehosting\ServiceTest::testupdateServer
TypeError: Argument 1 passed to Box\Mod\Servicehosting\Service::updateServer() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 982

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:663
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:982

8) Box\Mod\Servicehosting\ServiceTest::testgetServerManager
TypeError: Argument 1 passed to Box\Mod\Servicehosting\Service::getServerManager() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 1000

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:699
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1000

9) Box\Mod\Servicehosting\ServiceTest::testtestConnection
TypeError: Argument 1 passed to Box\Mod\Servicehosting\Service::testConnection() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 1048

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:723
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1048

10) Box\Mod\Servicehosting\ServiceTest::testgetServerManagerWithLog
TypeError: Argument 1 passed to Box\Mod\Servicehosting\Service::getServerManagerWithLog() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 1220

E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:874
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1220

11) Box\Mod\Servicehosting\ServiceTest::testgetMangerUrls
Error: Call to undefined method Mock_Service_22a04a24::getMangerUrls()

E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1245

12) Box\Mod\Servicehosting\ServiceTest::testgetMangerUrlsException
Error: Call to undefined method Mock_Service_22a04a24::getMangerUrls()

E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1264

--

There were 2 failures:

1) Box\Mod\Servicehosting\ServiceTest::testgetServerManagerManagerNotDefined
Failed asserting that exception of type "TypeError" matches expected exception "Box_Exception". Message was: "Argument 1 passed to Box\Mod\Servicehosting\Service::getServerManager() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 1012" at
E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:699
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1012
.

2) Box\Mod\Servicehosting\ServiceTest::testgetServerManagerServerManagerInvalid
Failed asserting that exception of type "TypeError" matches expected exception "Box_Exception". Message was: "Argument 1 passed to Box\Mod\Servicehosting\Service::getServerManager() must be an instance of Model_ServiceHosting, instance of Model_ServiceHostingServer given, called in E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php on line 1029" at
E:\GitHub\PHP\boxbilling-master\src\bb-modules\Servicehosting\Service.php:699
E:\GitHub\PHP\boxbilling-master\tests\bb-modules\Servicehosting\ServiceTest.php:1029
.

ERRORS!
Tests: 88, Assertions: 242, Errors: 12, Failures: 2.

@samapraku This was my final unit test.
webmaster@webbhostingservices:~/httpdocs/boxbilling$ src/bb-vendor/bin/phpunit ./tests/bb-modules/Servicehosting/
PHPUnit 9.4.2 by Sebastian Bergmann and contributors.
................................................................. 65 / 88 ( 73%)
.......................                                           88 / 88 (100%)
Time: 00:00.105, Memory: 6.00 MB
OK (88 tests, 288 assertions)

unit test shiuld be run on each pull request with travis-ci or github actions

Yes, I normally do unit testing with github actions. But it seems that the
project location of the composer.json and lock files are giving me errors
on my php.yml file. The object is wanting to have it in the root folder not
in the src folder. I am working on a way around this currently but that is
not a high priority for me right now. I have been using the travis ci and
working with @samapraku https://github.com/samapraku doing unit testing
on my pull request because we were having different results using travis
ci. What is in the pull request has no errors in travis ci.
On Tue, Oct 27, 2020 at 12:42 PM Andrius Putna<EMAIL_ADDRESS>wrote:

unit test should be run on each pull request with travis-ci or github
actions
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/boxbilling/boxbilling/pull/689#issuecomment-717373304,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD24K7IDMQK26HT6T5PUTYLSM3Z5XANCNFSM4S65SOZQ
.

@samapraku If you are good now with these changes I will merge this edit. @fordnox has approved the changes.

@timothygwebb please merge this pull request as it is causing my pull request https://github.com/boxbilling/boxbilling/pull/695 for the unit tests to fail.

@samapraku  I have merged the request.


trims the last characters from the bottom if the caret is at the very bottom of the screen

trims the last characters from the bottom if the caret is at the very bottom of the screen
(if you print underscores, they are not displayed at all)

duplicate of #118


Persisting column config issue

I'm looking to persist the column config by using localstorage but the problem is if I have the selection checkbox within the columns, it keeps on adding the checkbox whenever I refresh the page. Is there a way I can remove the checkbox from the column config?
Thanks

I had sort of the same problem, and since the checkboxes column has a precise name i find it and skip it. I hope that can help you.

How do you skip it? The column name for the checkbox column is a tick mark.

If with column config you meant column definitions, this is what i do
gridApi.grid.columns.filter(function noCheckboxes(col) {
      return col.name !== 'selectionRowHeaderCol'
}).map(function copyCol(col) {
     var colDef = col.colDef
     // ...
})

Thanks for your help. I was able to implement it in ng-grid. Now I'm moving to ui-grid. How can I do this w/o asking the user to save and restore states?
Like the user moved columns. When comes back to this page, he should see the changed order of columns in the grid. This needs to be done for everything, ie, grouping, column show/hide etc

Actually, there's a better and straightforward way to get those columns which is getOnlyDataColumns.
To answer your question, you could simply set listeners for any event of interest. E.g. groupingChanged event.


Fix missing k8s manifests from release artifact

For legacy releases, we'll still refer to _site/vX.Y/manifests
Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit a0c31cb68d0192f42c549a2e18a4f4fa14bb9cde
https://deploy-preview-3235--calico.netlify.com

Deploy preview for calico ready!
Built with commit 8d9d8cbc9dc0c0d0e2d727772f34d1f041f1b385
https://deploy-preview-3235--calico.netlify.com


Handle escape sequence for tmux/screen window name

It is possible to set window name in screen/tmux with ESC k name ESC \. See http://web.mit.edu/gnu/doc/html/screen_9.html. Currently, asciinema doesn't handle those escape sequences.

asciinema recorder has been rewritten from scratch in Go since you reported this issue so it's possible that this problem doesn't happen anymore. If you will experience this again feel free to reopen this issue.


Remove "Quick Fix..." link when no code actions are available

Remove "Quick Fix..." link when no code actions are available. It's a minor inconvenience to click it, see that there are no fixes, and have to hover for intellisense again.

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


feat(table): add data on cdk row

closes #14807
I guess this change would introduce a breaking change of the public api since it changes the CdkRow to a generic class

Closing this since I'm not super clear on what the goal was. WRT TreeControl, I would like to move away from the TreeControl concept in the future for cdk tree, but that will need some more thought.


Merge generic Deployment and Service reconcilers into a truly generic reconciler

Merges GenericDeploymentReconciler and GenericServiceReconciler into GenericAdapterReconciler[T] (uses Go generics).
This removes all the code duplicated in the previous two implementations.
Marking as draft, depends on #772.
Also, I'm not sure I'm happy with the result. Although this change does remove code duplication, some parts of it feel convoluted and this fully generic approach may prevent future additions to one implementation or the other. For example, if we ever want to reconcile a Kubernetes Service as part of the reconciliation of a Deployment, we will need to split the logic again.

Based on this attempt of implementation, it seems clear to me that we shouldn't be doing this, even if we technically can.
A little repetition is better than the wrong abstraction.
I will keep exploring how we can leverage generic functions instead of genetic types in other to reduce code duplication inside pkg/reconciler.


Allow task and fleet updates over ROS 2

What's new
Sets up ROS 2 subscriptions for fleet and task related updates, that were solely on websockets _internal route in the past.
The motivation is related to https://github.com/open-rmf/rmf-web/pull/899, where non-reproducible issues with websockets occur unpredictably in production environments, causing various systems to fail.
This allows the option to mitigate issues with the upstream websocketpp library, and just rely on ROS 2 to publish updates instead.
Works with https://github.com/open-rmf/rmf_ros2/pull/383
Self-checks

[ ] I have prototyped this new feature (if necessary) on Figma
[ ] I'm familiar with and follow this Typescript guideline
[ ] I added unit-tests for new components
[ ] I tried testing edge cases
[ ] I tested the behavior of the components that interact with the backend, with an e2e test

I fully agree that focusing on a single transport mode would be the cleanest way to handle it.
For updating dependencies, we could be limited to half of the equation only, we could update rmf-web as we go, but as the cpp stack uses https://github.com/zaphoyd/websocketpp, which is generally stable but won't be having any updates any time soon.

There is also a chance that this doesn't fix it as we have seen ros2 messages being significantly delayed before as well.

It does not resolve the root cause of the issue, but that could actually be infrastructure for all we know and therefore there is nothing we can do to properly diagnose and provide a solution for. However allowing the option to use ROS 2 will at least circumvent the symptoms of stalling all the components that use websockets (API server, fleet adapter, task dispatcher), and require manual restarting of these components in a production environment.

That said, I'm not opposed to using ros2, but I think we should only receive updates from a single source, there should be an option to choose which source to use.

I agree with this too, however we might have situations where there are fleet adapters running on the same system as the API server (also encountering websocket connection issues) and could benefit from rmw shared memory transport by using ROS 2, whereas there are fleet adapters running in the cloud somewhere that would like to use websockets instead.
The fleet adapters will only have 1 mode of transport to choose from either websockets or ROS 2, but not both, and we can plaster as many warnings or deprecation notices as we want on the ROS 2 endpoints

The fleet adapters will only have 1 mode of transport to choose from either websockets or ROS 2, but not both, and we can plaster as many warnings or deprecation notices as we want on the ROS 2 endpoints

I see that there is multiple way this can play out.

If the fleet adapters offer both transport simultaneously, it will cause issues if we also listen on both transport. But if we only listen on 1 transport, then there would be no issue.
If the fleet adapters only offer one transport at once, then listening on both ws and ros2 gives the advantage of supporting a mix of ws and ros2 adapters.

I think there is value in supporting both transport simultaneously so I prefer going with 1.

With the discussion to allow publishing over ROS 2 regardless of the availability of websockets, https://github.com/open-rmf/rmf_ros2/pull/383, we will inevitably run into issues where the API server will be receiving duplicated information if a locally running fleet adapter is provided websockets access.
Apologies for the back-and-forth, we can retain the current behavior of using websockets only on rmf-web for now, until there is a more tangible requirement for supporting both websockets and ROS 2.
For now, with the fleet adapters and task dispatcher publishing over ROS 2 by default, since the original motivation is to provide a workaround for some obscure issue, users/collaborators can reference this PR to implement the changes they require for their deployments.
Closing this PR


Bug: Deleting a table row causes a crash

Description
Deleting the last table row causes the following crash inside $deleteTableRow_EXPERIMENTAL:
main.788dd689.js:21 Uncaught Error: Expected GridNode childAtIndex(5) to be RowNode
    at jj.$deleteTableRow__EXPERIMENTAL (main.788dd689.js:21:426261)
    at main.788dd689.js:21:1208697
    at eh (main.788dd689.js:21:189961)
    at th (main.788dd689.js:21:192102)
    at Dh.update (main.788dd689.js:21:213346)
    at main.788dd689.js:21:1208684
    at onClick (main.788dd689.js:21:1211094)
    at Object.Ct (main.788dd689.js:21:16658)
    at Ot (main.788dd689.js:21:16812)
    at main.788dd689.js:21:36721

Lexical version: Latest
Steps To Reproduce

Visit the Lexical Playground app
Insert a 5x5 table(not the experimental)
Try to delete the first cell of the last row

I am not able to reproduce the issue
https://user-images.githubusercontent.com/2174170/230070779-efe130fd-62fa-4a73-9df7-d3393466e5f6.mov

I am not able to reproduce the issue

The issue is with the non-experimental table


Introduce Y059: Generic[] should always come last in a bases tuple

Fixes #426. I don't expect there to be many typeshed_primer hits here, since nearly all violations of this rule in typeshed were already fixed in https://github.com/python/typeshed/pull/10610.

⚠ Flake8 diff showing the effect of this PR on typeshed:
> ./stubs/tensorflow/tensorflow/keras/layers.pyi:41:13: Y059 "Generic[]" should always be the last base class

Looks like a true positive to me: https://github.com/python/typeshed/blob/b9640005eb586afdbe0a57bac2b88a7a12465069/stubs/tensorflow/tensorflow/keras/layers.pyi#L41


fix:CXL traffic generator read/write bandwidth and input arguments bou…

…ndary check
fix#1
For the correct value of loop count, the following condition must be satisfied. max(READ,WRITE) * loop_count <= 268,435,456
fix #2
cannot have READs only (without any previous WRITEs)
fix #3
READs cannot exceed #WRITEs
Fix #4
Read HDM size from CSR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 26 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 67.551%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

samples/cxl_mem_tg/cxl_mem_tg.h
0
2
0.0%

samples/cxl_mem_tg/cxl_tg_test.h
0
24
0.0%

Files with Coverage Reduction
New Missed Lines
%

samples/cxl_mem_tg/cxl_tg_test.h
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
15755

Relevant Lines:
23323

💛  - Coveralls


createDir doesn't work for file target and NLog.config

Hi! Thanks for reporting this feature/bug/question!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
For .NET Core users, please check the platform support: https://github.com/NLog/NLog/wiki/platform-support
Bug
NLog version: 4.4.4
Platform:  .Net 4.6.2
Current NLog config (xml or C#, if relevant)

<targets>
	<target xsi:type="File" name="mainLogFile" fileName="c:\Temp\Logs\123XYZ\info.log" />
</targets>

<rules>
	<logger name="*" minlevel="Trace" writeTo="mainLogFile" />
</rules>

In case of a BUG:

What is the current result?
Exception: System.IO.DirectoryNotFoundException: Could not find a part of the path 'c:\Temp\Logs\123XYZ\info.log'.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)

What is the expected result?
'C:\Temp\123XYZ\Logs folder will be created, file info.log will be created, log record will be added to the log file.

Did you checked the Internal log?
Yes.

Please post full exception details (message, stacktrace, inner exceptions)
No exceptions found in the NLog internal log file. Only Trace, Info, Debug messages. Last 2 lines:

2017-03-20 13:39:19.3917 Debug Creating file appender: c:\Temp\Logs\123XYZ\info.log
2017-03-20 13:39:19.3997 Trace Opening c:\Temp\Logs\123XYZ\info.log with allowFileSharedWriting=False

Are there any work arrounds?
Yes. Create a log folder explicitly in code like:
Directory.CreateDirectory(logFolderPath);

Is there a version in which it did worked?
Do not know.

Can you help us by writing an unit test?

If I uncomment the config code in Unit Test - createDir works. If I comment it and NLog uses the NLog.config file - createDir doesn't work.
[TestMethod]
public void ShouldCreateDirectory()
{
string logDir = @"c:\Temp\Logs\123XYZ";
string logFilePath = logDir + "info.log";
if (Directory.Exists(logDir))
{
	Directory.Delete(logDir, true);
}

////var config = new LoggingConfiguration();
////LogManager.ThrowExceptions = true;

////var target =
////		new FileTarget
////		{
////			FileName = logFilePath
////		};

////config.AddTarget("logfile", target);
////var rule = new LoggingRule("*", LogLevel.Trace, target);
////config.LoggingRules.Add(rule);

////LogManager.Configuration = config;

ILogger logger = LogManager.GetCurrentClassLogger();
logger.Trace($"Test {DateTime.Now}");

if (File.Exists(logFilePath) == false)
{
	throw new FileNotFoundException(logFilePath);
}

}

Found some my errors.


assets.go causes index out of range if worker image has no slash

What happened?:
panic: runtime error: index out of range

goroutine 206 [running]:
github.com/pachyderm/pachyderm/src/server/pkg/deploy/assets.AddRegistry(0x0, 0x0, 0xc00005800d, 0x17, 0x0, 0x0)
	src/github.com/pachyderm/pachyderm/src/server/pkg/deploy/assets/assets.go:1577 +0x349
github.com/pachyderm/pachyderm/src/server/pps/server.(*apiServer).workerPodSpec(0xc0002af680, 0xc0001c3800, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	src/github.com/pachyderm/pachyderm/src/server/pps/server/worker_rc.go:168 +0x2807
github.com/pachyderm/pachyderm/src/server/pps/server.(*apiServer).createWorkerRc(0xc0002af680, 0xc0001c3800, 0x7, 0x2)
	src/github.com/pachyderm/pachyderm/src/server/pps/server/worker_rc.go:429 +0x7e
github.com/pachyderm/pachyderm/src/server/pps/server.(*apiServer).upsertWorkersForPipeline.func1(0xc000520cc0, 0xc0008c3670)
	src/github.com/pachyderm/pachyderm/src/server/pps/server/master.go:346 +0x3de
github.com/pachyderm/pachyderm/src/server/pkg/backoff.RetryNotify(0xc0007464a0, 0x22904c0, 0xc000520cc0, 0xc0007464e0, 0x1, 0x1)
	src/github.com/pachyderm/pachyderm/src/server/pkg/backoff/retry.go:35 +0x4a
github.com/pachyderm/pachyderm/src/server/pps/server.(*apiServer).upsertWorkersForPipeline(0xc0002af680, 0xc0003f0b60, 0xc0008c3950, 0x1)
	src/github.com/pachyderm/pachyderm/src/server/pps/server/master.go:293 +0xff
github.com/pachyderm/pachyderm/src/server/pps/server.(*apiServer).master.func1(0x0, 0x0)
	src/github.com/pachyderm/pachyderm/src/server/pps/server/master.go:175 +0xb0b
github.com/pachyderm/pachyderm/src/server/pkg/backoff.RetryNotify(0xc000586be0, 0x22904c0, 0xc0003ca240, 0xc000fc5fb8, 0x2a, 0xc00063a300)
	src/github.com/pachyderm/pachyderm/src/server/pkg/backoff/retry.go:35 +0x4a
github.com/pachyderm/pachyderm/src/server/pps/server.(*apiServer).master(0xc0002af680)
	src/github.com/pachyderm/pachyderm/src/server/pps/server/master.go:49 +0x1af
created by github.com/pachyderm/pachyderm/src/server/pps/server.NewAPIServer
	src/github.com/pachyderm/pachyderm/src/server/pps/server/server.go:76 +0x5e0

What you expected to happen?:
Continue to use the specified worker image unmolested
How to reproduce it (as minimally and precisely as possible)?:
env:
- name: WORKER_IMAGE
  value: pachyderm_worker:latest

Anything else we need to know?:
Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI version (use pachctl version):
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others: I first noticed this in 3d1c3dcd but that just happened to be master at the time I pulled it down, it is apparently still an issue

If I might point out, there's no reason for that method to do any string manipulation if registry is empty, so:
func AddRegistry(registry string, imageName string) string {
    if registry == "" {
        return imageName
    }
}

although reasonable people can differ about whether calling AddRegistry with a blank registry has any meaning, and thus it's the caller who is in error


[Bug]: docker 启动vllm 报错

Your current environment
The output of `python collect_env.py`

🐛 Describe the bug
我拉取的最新的镜像：docker  pull vllm/vllm-openai:latest
但是编排镜像启动就报错:
version: '3.9'
services:
vllm:
image: vllm/vllm-openai:latest
container_name: qwen1.5
ulimits:
stack: 67108864
memlock: -1
environment:
- TZ=Asia/Shanghai
- CUDA_VISIBLE_DEVICES=0
restart: always
shm_size: '10.24gb'
command:
--model /Qwen1.5-14B-Chat-GPTQ-Int4
--tensor-parallel-size 4
--host <IP_ADDRESS>
--port 8007
--dtype float16
--quantization gptq
--served-model-name qwen-4bit
--gpu-memory-utilization=0.9
--trust-remote-code
--max-model-len 10240
--enforce-eager
volumes:
#- /data/models/OrionStarAI/Orion-14B-Chat-Int4:/Orion-14B-Chat-Int4
- /data/models/Qwen1.5-14B-Chat-GPTQ-Int4:/Qwen1.5-14B-Chat-GPTQ-Int4
ports:
- "8007:8007"
deploy:
resources:
reservations:
devices:
- driver: nvidia
device_ids: ['0']
capabilities: [gpu]
启动后会报下面错误。
[+] Building 0.0s (0/0)
[+] Running 1/1
✔ Container qwen1.5  Created                                                             0.3s
Attaching to qwen1.5
qwen1.5  | INFO 03-15 00:33:27 api_server.py:228] args: Namespace(host='<IP_ADDRESS>', port=8007, allow_credentials=False, allowed_origins=[''], allowed_methods=[''], allowed_headers=['*'], api_key=None, served_model_name='qwen-4bit', lora_modules=None, chat_template=None, response_role='assistant', ssl_keyfile=None, ssl_certfile=None, root_path=None, middleware=[], model='/Qwen1.5-14B-Chat-GPTQ-Int4', tokenizer=None, revision=None, code_revision=None, tokenizer_revision=None, tokenizer_mode='auto', trust_remote_code=True, download_dir=None, load_format='auto', dtype='float16', kv_cache_dtype='auto', max_model_len=10240, worker_use_ray=False, pipeline_parallel_size=1, tensor_parallel_size=4, max_parallel_loading_workers=None, block_size=16, seed=0, swap_space=4, gpu_memory_utilization=0.9, max_num_batched_tokens=None, max_num_seqs=256, max_paddings=256, disable_log_stats=False, quantization='gptq', enforce_eager=True, max_context_len_to_capture=8192, disable_custom_all_reduce=False, enable_lora=False, max_loras=1, max_lora_rank=16, lora_extra_vocab_size=256, lora_dtype='auto', max_cpu_loras=None, device='auto', engine_use_ray=False, disable_log_requests=False, max_log_len=None)
qwen1.5  | WARNING 03-15 00:33:27 config.py:193] gptq quantization is not fully optimized yet. The speed can be slower than non-quantized models.
qwen1.5  | INFO 03-15 00:33:27 config.py:433] Custom all-reduce kernels are temporarily disabled due to stability issues. We will re-enable them once the issues are resolved.
qwen1.5  | 2024-03-15 00:33:29,864      INFO worker.py:1724 -- Started a local Ray instance.
qwen1.5  | Traceback (most recent call last):
qwen1.5  |   File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
qwen1.5  |     return _run_code(code, main_globals, None,
qwen1.5  |   File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
qwen1.5  |     exec(code, run_globals)
qwen1.5  |   File "/workspace/vllm/entrypoints/openai/api_server.py", line 236, in
qwen1.5  |     engine = AsyncLLMEngine.from_engine_args(engine_args)
qwen1.5  |   File "/workspace/vllm/engine/async_llm_engine.py", line 625, in from_engine_args
qwen1.5  |     placement_group = initialize_cluster(parallel_config,
qwen1.5  |   File "/workspace/vllm/engine/ray_utils.py", line 129, in initialize_cluster
qwen1.5  |     raise ValueError(
qwen1.5  | ValueError: The number of required GPUs exceeds the total number of available GPUs in the cluster.
qwen1.5 exited with code 0
直接conda 安装vllm 0.30 ,python -m 启动 是正常的，不知道这个是啥原因导致的报错？

Hi, there seems to be an error message: ValueError: The number of required GPUs exceeds the total number of available GPUs in the cluster. Maybe you should check with the way you use GPUs?

Please attach the output of python collect_env.py in the host machine, so that we can help find the cause.

python collect_env.pyCollecting environment information...
PyTorch version: 2.1.2+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A
OS: Ubuntu 20.04.5 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.31
Python version: 3.10.9 (main, Mar  1 2023, 18:23:06) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-5.15.0-89-generic-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: 12.3.107
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA GeForce RTX 3090
GPU 1: NVIDIA GeForce RTX 3090
Nvidia driver version: 550.54.14
cuDNN version: Probably one of the following:
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn.so.8.9.2
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn_adv_infer.so.8.9.2
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn_adv_train.so.8.9.2
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn_cnn_infer.so.8.9.2
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn_cnn_train.so.8.9.2
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn_ops_infer.so.8.9.2
/usr/local/cuda-11.7/targets/x86_64-linux/lib/libcudnn_ops_train.so.8.9.2
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
架构：                              x86_64
CPU 运行模式：                      32-bit, 64-bit
字节序：                            Little Endian
Address sizes:                      46 bits physical, 48 bits virtual
CPU:                                40
在线 CPU 列表：                     0-39
每个核的线程数：                    2
每个座的核数：                      20
座：                                1
NUMA 节点：                         1
厂商 ID：                           GenuineIntel
CPU 系列：                          6
型号：                              85
型号名称：                          Intel(R) Xeon(R) Gold 6133 CPU @ 2.50GHz
步进：                              4
CPU MHz：                           1100.000
CPU 最大 MHz：                      3000.0000
CPU 最小 MHz：                      1000.0000
BogoMIPS：                          5000.00
虚拟化：                            VT-x
L1d 缓存：                          640 KiB
L1i 缓存：                          640 KiB
L2 缓存：                           20 MiB
L3 缓存：                           27.5 MiB
NUMA 节点0 CPU：                    0-39
Vulnerability Gather data sampling: Mitigation; Microcode
Vulnerability Itlb multihit:        KVM: Mitigation: VMX disabled
Vulnerability L1tf:                 Mitigation; PTE Inversion; VMX conditional cache flushes, SMT vulnerable
Vulnerability Mds:                  Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Meltdown:             Mitigation; PTI
Vulnerability Mmio stale data:      Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Retbleed:             Mitigation; IBRS
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; IBRS, IBPB conditional, STIBP conditional, RSB filling, PBRSB-eIBRS Not affected
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Mitigation; Clear CPU buffers; SMT vulnerable
标记：                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 cdp_l3 invpcid_single pti intel_ppin ssbd mba ibrs ibpb stibp tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm cqm mpx rdt_a avx512f avx512dq rdseed adx smap clflushopt clwb intel_pt avx512cd avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local dtherm ida arat pln pts pku ospke md_clear flush_l1d arch_capabilities
Versions of relevant libraries:
[pip3] flake8==6.0.0
[pip3] mypy-extensions==0.4.3
[pip3] numpy==1.23.5
[pip3] numpydoc==1.5.0
[pip3] torch==2.1.2
[pip3] triton==2.1.0
[conda] blas                      1.0                         mkl    defaults
[conda] mkl                       2021.4.0           h06a4308_640    defaults
[conda] mkl-service               2.4.0           py310h7f8727e_0    defaults
[conda] mkl_fft                   1.3.1           py310hd6ae3a3_0    defaults
[conda] mkl_random                1.2.2           py310h00e6091_0    defaults
[conda] numpy                     1.23.5          py310hd5efca6_0    defaults
[conda] numpy-base                1.23.5          py310h8e6c178_0    defaults
[conda] numpydoc                  1.5.0           py310h06a4308_0    defaults
[conda] torch                     2.1.2                    pypi_0    pypi
[conda] triton                    2.1.0                    pypi_0    pypiROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.3.0
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    GPU1    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      NV4     0-39    0               N/A
GPU1    NV4      X      0-39    0               N/A
Legend:
X    = Self
SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
PIX  = Connection traversing at most a single PCIe bridge
NV#  = Connection traversing a bonded set of # NVLinks

Hi, there seems to be an error message: ValueError: The number of required GPUs exceeds the total number of available GPUs in the cluster. Maybe you should check with the way you use GPUs?

我加了一个配置 --tensor-parallel-size 1 ，但是启动还是报错，如下：
Attaching to qwen1.5
qwen1.5  | Traceback (most recent call last):
qwen1.5  |   File "/workspace/vllm/utils.py", line 176, in get_ip
qwen1.5  |     s.connect(("dns.google", 80))  # Doesn't need to be reachable
qwen1.5  | socket.gaierror: [Errno -3] Temporary failure in name resolution
qwen1.5  |
qwen1.5  | During handling of the above exception, another exception occurred:
qwen1.5  |
qwen1.5  | Traceback (most recent call last):
qwen1.5  |   File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
qwen1.5  |     return _run_code(code, main_globals, None,
qwen1.5  |   File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
qwen1.5  |     exec(code, run_globals)
qwen1.5  |   File "/workspace/vllm/entrypoints/openai/api_server.py", line 236, in
qwen1.5  |     engine = AsyncLLMEngine.from_engine_args(engine_args)
qwen1.5  |   File "/workspace/vllm/engine/async_llm_engine.py", line 628, in from_engine_args
qwen1.5  |     engine = cls(parallel_config.worker_use_ray,
qwen1.5  |   File "/workspace/vllm/engine/async_llm_engine.py", line 321, in init
qwen1.5  |     self.engine = self._init_engine(*args, **kwargs)
qwen1.5  |   File "/workspace/vllm/engine/async_llm_engine.py", line 369, in _init_engine
qwen1.5  |     return engine_class(*args, **kwargs)
qwen1.5  |   File "/workspace/vllm/engine/llm_engine.py", line 128, in init
qwen1.5  |     self._init_workers()
qwen1.5  |   File "/workspace/vllm/engine/llm_engine.py", line 167, in _init_workers
qwen1.5  |     get_ip(), get_open_port())
qwen1.5  |   File "/workspace/vllm/utils.py", line 181, in get_ip
qwen1.5  |     s.connect(("dns.google", 80))
qwen1.5  | socket.gaierror: [Errno -3] Temporary failure in name resolution
qwen1.5 exited with code 0

Yes --tensor-parallel-size 1 should fix your previous problem.
For the next problem, it seems you have a network connection problem. Please check your internet access first, i.e. if your server can access dns.google .

Yes --tensor-parallel-size 1 should fix your previous problem.
For the next problem, it seems you have a network connection problem. Please check your internet access first, i.e. if your server can access dns.google .

我无法访问 dns.google，那需要怎么处理这个问题呢？因为是离线部署启动调试，无法连接外网

If you know the ip of the host, just modify the get_ip function to directly return that ip, please.

FYI: you might use hostname -I | awk '{print $1}' in the host machine to find the ip.

FYI: you might use hostname -I | awk '{print $1}' in the host machine to find the ip.
我知道服务本机的ip,但是你说的 get_ip function 这个找到，因为是直接拉去镜像的服务去启动的，是不是要涉及重新打镜像了啊？这个是不是不能在 docker compose 启动中加上 环境变量的参数去配置啊

I think environment variable might be a good way. Stay tuned about the update.

I think environment variable might be a good way. Stay tuned about the update.

thanks, waiting for update ~

Should be resolved after https://github.com/vllm-project/vllm/pull/3419 .


[Event Request] Codeunit 5522 Order Planning Mgt. - OnBeforeGetOrdersToPlan

Hi, would it be possible to get an event OnBeforeGetOrdersToPlan codeunit 5522 , procedure GetOrdersToPlan, like this :

    procedure GetOrdersToPlan(var ReqLine: Record "Requisition Line")
    var
        IsHandled: Boolean;
    begin
        PrepareRequisitionRecord(ReqLine);
        IsHandled := false;
        OnBeforeGetOrdersToPlan(TempUnplannedDemand, IsHandled);
        if not IsHandled then
            CODEUNIT.Run(CODEUNIT::"Get Unplanned Demand", TempUnplannedDemand);
        TransformUnplannedDemandToRequisitionLines(ReqLine);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeGetOrdersToPlan(var UnplannedDemand: Record "Unplanned Demand"; var IsHandled: Boolean)
    begin
    end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


clean/remove doesn't do a pre check for the device in use

if the snapshot has a filesystem mounted, clean fails in a confusing way and it is possible to get partial results
add test to suite and fix the problem.

also add tests for revert and extend while device is in use

a check as been added for "in use" but still need to add the tests


TCP: Too many connections

In a specific test case I have been able to connection-spam my way to having too many connections, which makes the instance run out of memory and die. This is expected, however it made me think about what would happen on slower computers/servers. We have no way of restricting connections other than using a magic number in the server application, and stop accepting connections after a certain number of connections are active. HOWEVER, this includes the total active connection count in the TCP stack, not just the connection count on the server application, as the connections take a long time to time out properly.  In our case we always run with limited memory due to the nature of the project.
Thoughts?

We have decided that connection count is something that is controlled by the application developer, as it depends on both hardware and the type of service run.


ad creation error fix

This patch forces the library to return an error when there is an ad viewcontroller creation error.

Coverage decreased (-0.06%) to 54.89% when pulling 46d6421da83a971c4c3e437222f6e306e9409bf8 on development_creationErrorFix into 33465f1ed7157d7db5ea188c8ec804ae795a3068 on development.


Add form to tag Education content to beta taxonomy

For documents belonging to DfE and SFA, show a button to tag to the
Finding Things education taxonomy. This allows stakeholders to keep
tagging content to the taxonomy during beta.
Look & Feel

Trello:
https://trello.com/c/miTSiYfk/409-create-tagging-interface-for-new-taxonomy-in-whitehall
Paired with @MatMoore and @klssmith from @alphagov/team-gov-uk-finding-things

This looks really good!

I moved the mixin from Editions to the various models that include the Edition::Organisations concern, because not every Edition has the organisations association.
As far as I can tell, the only exception is CorporateInformationPage, which isn't relevant to the beta.

This is missing a feature flag as discussed offline

This can only be deployed after implementation of feature flag and the deploy of puppet with the new environment variable (feature flag) https://github.com/alphagov/govuk-puppet/pull/5361


Add version attribute as per Python convention

Just a simple PR so that bitsandbytes has a version attribute, which is common in Python scientific computing libraries (PyTorch, NumPy, scikit-learn, pandas, ...)
So, this PR allows the following now:
>>> import bitsandbytes as bnb
>>> print(bnb.__version__)
"0.41.0"

@rasbt , it is not critical, but a very convenient feature. I'd like to see it merged. Could you check the reasons for the merge conflict, please?

@Titus-von-Koeller , would it be possible to merge this PR somehow? -- it would be helpful in situations when we test multiple bnb versions like 0.41.3.post1.

@rasbt , it is not critical, but a very convenient feature. I'd like to see it merged. Could you check the reasons for the merge conflict, please?

It was a minor issue due to to the latest BnB update overwriting a line in this PR. Should be fixed.

Thank you, this is a good idea! Sorry for taking so long.


Checkbox settings appear empty on load

Description
When first loading the extension, all settings that use a checkbox appear empty (none of the options are checked).
Browser
Both Chrome and Firefox
Cause
In the updateHTMLValues function, there is only a case for if the setting is of type range. This means that radio style settings are not being set.

#104 Fixes this issue.


Add support for property defined version update in the goal use-dep-version

I noticed that currently use-dep-version works only when the version is defined inline, as in the following works:
mvn versions:use-dep-version  -DdepVersion=2.3.1 '-Dincludes=javax.xml.bind:jaxb-api'

--- a/example/pom.xml
+++ b/example/pom.xml
@@ -80,7 +80,7 @@
         <dependency>
             <groupId>javax.xml.bind</groupId>
             <artifactId>jaxb-api</artifactId>
-            <version>2.3.0</version>
+            <version>2.3.1</version>
         </dependency>

But the following property defined version doesn't get updated:
    <properties>
      <jaxb.version>2.3.0</jaxb.version>
    </properties>

        <dependency>
            <groupId>javax.xml.bind</groupId>
            <artifactId>jaxb-api</artifactId>
            <version>${jaxb.version}</version>
        </dependency>

This is a feature request to make use-dep-version work for property defined dependencies too.
Use case: we have many internal repositories which depends on few internal common libraries, sometimes we would want to update all of those repos to the latest version of the common libraries for bugs/security issues etc. But the downstream maven projects sometimes would have the version defined as a property or sometimes as inline version. If one command could handle both cases we could easily update all the repos to the target version.

I am still interested in this feature/use case.

So, the problem is that all properties are resolved (interpolated) when processed by the mojo. Whilst the is matched by the mojo, PomHelper is unable to replace the dependency version because it's unable to retrieve the sought string (containing the resolved property value) in the POM, so the POM is not getting updated.
Using a raw POM does help here; checking what the impact of changing that would be.

Proposed new feature.
/**
 *
Will augment normal processing by, if a dependency value is set using a property, trying to update
 * the value of the property.

 *
If the property value is specified directly, will process it normally (as with {@code processProperties} being
 * {@code false}. If the property being updated is redefined in the reactor tree, will only change the property
 * value which lies the closest to the dependency being updated. Finally, if the same property is also used to set
 * the value of another dependency, will not update that property value, and log a warning instead.

 *
Default is {@code false}.

 */

What do you think, @slawekjaranowski @pzygielo?

Thank you for supporting this use case!

This will be fixed in the next release. Please use -DprocessDependencies=false


Make application icon for Mac OS X

Right now we have default Mac OS X icon for our tray application. But we need nice one.
So we need use this manual :
http://doc.qt.io/qt-4.8/appicon.html .

Use iconutil -c icns *.iconset  .
Iconset folder should contains list of files below :
icon_16x16.png
<EMAIL_ADDRESS>...
icon_512x512.png
<EMAIL_ADDRESS>

Waiting for icons from designers team

Implemented


256k disk has no recipe

the 256k disk doesn't have a recipe is this a problem with the mod or my mod pack

On it, will be fixed today

Updated and posted to Curseforge.
https://www.curseforge.com/minecraft/mc-mods/ae2-things-updated/files/5556533
Thanks for the report!

i downloaded the update but the portable disk is the only one that got added but they'res still not 256k disk recipes in src/main/resources/data/ae2things/recipes/cells/disk

My bad, working on it

no youre fine the only reason i asked is i couldn't figure out how to fix it my self so i got no problem waiting for some one who actually knows how to fix it to help

There we go, I gave it a test and it looks to be working now
https://www.curseforge.com/minecraft/mc-mods/ae2-things-updated/files/5556617
I'll leave the issue open if there are any other problems


Consider spec. of eventual consistency expectations

This sounds like something which is going down too detailed a path IMO. What exactly are you thinking of here?

Sorry, I created these issues quickly as notes during the C&W call so I wouldn't forget about them.
The question here is what (if any) guarantees a TES client has from the server while creating tasks. In particular, if a client creates a task and immediately starts a polling loop, is the server guaranteed to return a state for the created task (i.e. HTTP 200 not 404)?
In a large scale system, it could be more difficult to implement that guarantee (e.g. if your distributed database is eventually consistent).

Gotcha. Yeah this is one of those things where I see why it's valuable as part of a spec but also starts bumping into my general sentiment that the more fiddly a spec like this, the less likely people are to implement it.

Agreed. The main goal here is to document "we thought of this and made a decision", even if that decision is "TES makes no guarantees in this aspect".

Closing this because I don't have a good action item for this. The spec shouldn't bother to document all the quirks of distributed systems, and it shouldn't require that implementations be strongly consistent either. I suspect this is not going to be a problem anyway. If it is an issue after people have experience with real TES services for a couple years, we can revisit.


Let's bring TNND #1.1, 1.2 over to TNDD

Let's bring TNND #1.1, 1.2 over to TNDD

1.1 > 84.1
1.2 > 84.2


please

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I guess something went wrong while creating this feature request!

Closed as possible unsuccessful predecessor of #11724


Investigate nightly build failure with AstroSDK

ref - https://astronomer.slack.com/archives/C059004990C/p1717074355350809

We build nightlies from the OSS main branch and there was a change done on OSS airflow to update the version from 2.9.0.dev0 to 2.10.0.dev0 in PR .
With the latest 1.8.0 of AstroSDK, we have this version of file astro/sql/operators/raw_sql.py -
import airflow

if airflow.__version__ >= "2.3":
    from sqlalchemy.engine.row import LegacyRow as SQLAlcRow
else:
    from sqlalchemy.engine.result import RowProxy as SQLAlcRow

In the above code snippet, version comparison is not correct, which leads to the ImportError: cannot import name 'RowProxy' from 'sqlalchemy.engine.result' and explains the recent failures:

Since we no longer have a incorrect version check, which was removed in PR. We can do a release for AstroSDK and it should be fixed.