Encounter install error when install vllm package.

System Info
Target: x86_64-unknown-linux-gnu
Cargo version: 1.75.0
Commit sha: N/A
Docker label: N/A
nvidia-smi:
  +---------------------------------------------------------------------------------------+
   | NVIDIA-SMI 535.171.04             Driver Version: 535.171.04   CUDA Version: 12.2     |
   |-----------------------------------------+----------------------+----------------------+
   | GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
   | Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
   |                                         |                      |               MIG M. |
   |=========================================+======================+======================|
   |   0  NVIDIA RTX A6000               Off | 00000000:01:00.0 Off |                  Off |
   | 52%   77C    P2             291W / 300W |  47471MiB / 49140MiB |     96%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
   |   1  NVIDIA RTX A6000               Off | 00000000:25:00.0 Off |                  Off |
   | 38%   67C    P2             266W / 300W |  47527MiB / 49140MiB |     97%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
   |   2  NVIDIA RTX A6000               Off | 00000000:41:00.0 Off |                  Off |
   | 48%   75C    P2             285W / 300W |  47483MiB / 49140MiB |     95%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
   |   3  NVIDIA RTX A6000               Off | 00000000:61:00.0 Off |                  Off |
   | 41%   69C    P2             285W / 300W |  47454MiB / 49140MiB |     98%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
   |   4  NVIDIA RTX A6000               Off | 00000000:81:00.0 Off |                  Off |
   | 49%   75C    P2             297W / 300W |  41417MiB / 49140MiB |    100%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
   |   5  NVIDIA RTX A6000               Off | 00000000:A1:00.0 Off |                  Off |
   | 39%   66C    P2             295W / 300W |  39745MiB / 49140MiB |    100%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
|   6  NVIDIA RTX A6000               Off | 00000000:C1:00.0 Off |                  Off |
   | 47%   73C    P2             293W / 300W |  23995MiB / 49140MiB |    100%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+
   |   7  NVIDIA RTX A6000               Off | 00000000:E1:00.0 Off |                  Off |
   | 39%   66C    P2             293W / 300W |  35729MiB / 49140MiB |    100%      Default |
   |                                         |                      |                  N/A |
   +-----------------------------------------+----------------------+----------------------+

Information

[ ] Docker
[X] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
TGI version: 2.0.2
create an new conda environment, running:

1.BUILD_EXTENSIONS=True make install

2.cd server

3.make install-causal-conv1d

4.make install-flash-attention

5.make install-flash-attention-v2-cuda

6.make install-vllm-cuda

The commands 1-5 are executed normally. But command 6 ends with below error message, in which vllm is installed successfully but vllm-nccl-cu12 failed:
...
creating 'dist/vllm-0.4.1+cu122-py3.11-linux-x86_64.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing vllm-0.4.1+cu122-py3.11-linux-x86_64.egg
removing '/server9/cbj/programming/anaconda3/envs/tgi_server/lib/python3.11/site-packages/vllm-0.4.1+cu122-py3.11-linux-x86_64.egg' (and everything under it)
creating /server9/cbj/programming/anaconda3/envs/tgi_server/lib/python3.11/site-packages/vllm-0.4.1+cu122-py3.11-linux-x86_64.egg
Extracting vllm-0.4.1+cu122-py3.11-linux-x86_64.egg to /server9/cbj/programming/anaconda3/envs/tgi_server/lib/python3.11/site-packages
Adding vllm 0.4.1+cu122 to easy-install.pth file

Installed /server9/cbj/programming/anaconda3/envs/tgi_server/lib/python3.11/site-packages/vllm-0.4.1+cu122-py3.11-linux-x86_64.egg
Processing dependencies for vllm==0.4.1+cu122
Searching for vllm-nccl-cu12<2.19,>=2.18
Reading https://pypi.org/simple/vllm-nccl-cu12/
Downloading https://files.pythonhosted.org/packages/41/07/c1be8f4ffdc257646dda26470b803487150c732aa5c9f532dd789f186a54/vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz#sha256
=d56535da1b893ac49c1f40be9245f999e543c3fc95b4839642b70dd1d72760c0
Best match: vllm-nccl-cu12 <IP_ADDRESS>.4.0
Processing vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz
Writing /tmp/easy_install-c0u46qco/vllm_nccl_cu12-<IP_ADDRESS>.4.0/setup.cfg
Running vllm_nccl_cu12-<IP_ADDRESS>.4.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-c0u46qco/vllm_nccl_cu12-<IP_ADDRESS>.4.0/egg-dist-tmp-r7fzlcup
error: SandboxViolation: mkdir('/server9/cbj/.config/vllm/nccl/cu12', 511) {}

The package setup script has attempted to modify files on your system
that are not within the EasyInstall build area, and has been aborted.

This package cannot be safely installed by EasyInstall, and may not
support alternate installation locations even if you run its setup
script by hand.  Please inform the package's author and the EasyInstall
maintainers to find out if a fix or workaround is available.

After checking, I found the error command is python setup.py install. But I don't understand why vllm_nccl_cu12 will install to /server9/cbj/.config dir instead of my conda environment.
Note, I can successfully execute commands 1-6 in TGI 2.0.1 following Faster-LLM-Survey, installing flash-attention, vllm, ...
Expected behavior
vllm-nccl-cu12 could be installed normally

you need to install vllm in other way!
like pip install vllm

you need to install vllm by other way!
like pip install vllm

----------------------报错：
Installed /home/tgi/miniforge3/envs/tgi/lib/python3.10/site-packages/vllm-0.4.1-py3.10-linux-x86_64.egg
Processing dependencies for vllm==0.4.1
Searching for vllm-nccl-cu12<2.19,>=2.18
Reading https://pypi.org/simple/vllm-nccl-cu12/
Downloading https://files.pythonhosted.org/packages/41/07/c1be8f4ffdc257646dda26470b803487150c732aa5c9f       532dd789f186a54/vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz#sha256=d56535da1b893ac49c1f40be9245f999e543c3fc95b48       39642b70dd1d72760c0
Best match: vllm-nccl-cu12 <IP_ADDRESS>.4.0
Processing vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz
Writing /tmp/easy_install-sr28_yp0/vllm_nccl_cu12-<IP_ADDRESS>.4.0/setup.cfg
Running vllm_nccl_cu12-<IP_ADDRESS>.4.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-sr28_yp0/vllm_nc       cl_cu12-<IP_ADDRESS>.4.0/egg-dist-tmp-rsid1j5o
error: SandboxViolation: mkdir('/home/tgi/.config/vllm/nccl/cu12', 511) {}
The package setup script has attempted to modify files on your system
that are not within the EasyInstall build area, and has been aborted.
This package cannot be safely installed by EasyInstall, and may not
support alternate installation locations even if you run its setup
script by hand.  Please inform the package's author and the EasyInstall
maintainers to find out if a fix or workaround is available.
解决方法：
本地下载安装依赖 vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz：https://files.pythonhosted.org/packages/41/07/c1be8f4ffdc257646dda26470b803487150c732aa5c9f532dd789f186a54/vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz#sha256=d56535da1b893ac49c1f40be9245f999e543c3fc95b4839642b70dd1d72760c0
解压
tar -xzvf vllm_nccl_cu12-<IP_ADDRESS>.4.0.tar.gz
cd vllm_nccl_cu12-<IP_ADDRESS>.4.0
python setup.py install
----------------------------继续报错：
Downloading nccl package from https://github.com/vllm-project/vllm-nccl/releases/download/v0.1.0/cu12-libnccl.so.2.18.1
Failed to download nccl package from https://github.com/vllm-project/vllm-nccl/releases/download/v0.1.0 /cu12-libnccl.so.2.18.1
<urlopen error [Errno 110] Connection timed out>
Traceback (most recent call last):
File "/home/tgi/csh/workspaces/cogvlm2-cogvlm2/server/vllm_nccl_cu12-<IP_ADDRESS>.4.0/setup.py", line 83,  in
if get_md5_hash(destination) != file_hash:
File "/home/tgi/csh/workspaces/cogvlm2-cogvlm2/server/vllm_nccl_cu12-<IP_ADDRESS>.4.0/setup.py", line 43,  in get_md5_hash
with open(file_path, "rb") as f:  # Open file in binary read mode
FileNotFoundError: [Errno 2] No such file or directory: '/home/tgi/.config/vllm/nccl/cu12/libnccl.so.2. 18.1'
解决方法：
本地下载依赖:
https://github.com/vllm-project/vllm-nccl/releases/download/v0.1.0 /cu12-libnccl.so.2.18.1
手动改名放到对应路径下：
mv cu12-libnccl.so.2.18.1 libnccl.so.2. 18.1
mv libnccl.so.2. 18.1 /home/tgi/.config/vllm/nccl/cu12/
---------------------报错：
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
vllm 0.4.1+cu122 requires fastapi, which is not installed.
vllm 0.4.1+cu122 requires lm-format-enforcer==0.9.8, which is not installed.
vllm 0.4.1+cu122 requires nvidia-ml-py, which is not installed.
vllm 0.4.1+cu122 requires openai, which is not installed.
vllm 0.4.1+cu122 requires prometheus-fastapi-instrumentator>=7.0.0, which is not installed.
vllm 0.4.1+cu122 requires ray>=2.9, which is not installed.
vllm 0.4.1+cu122 requires tiktoken==0.6.0, which is not installed.
vllm 0.4.1+cu122 requires uvicorn[standard], which is not installed.
vllm 0.4.1+cu122 requires vllm-nccl-cu12<2.19,>=2.18, which is not installed.
vllm 0.4.1+cu122 requires outlines==0.0.34, but you have outlines 0.0.36 which is incompatible.
解决方法：
python依赖包手动安装：
conda config --add channels https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/main/
conda config --add channels https://mirrors.tuna.tsinghua.edu.cn/anaconda/pkgs/free/
conda config --set show_channel_urls yes
conda install watchfiles==0.23.0 openai uvicorn[standard]
pip config set global.index-url Simple Index
pip install fastapi lm-format-enforcer==0.9.8  prometheus-fastapi-instrumentator>=7.0.0 tiktoken==0.6.0  outlines==0.0.34


Inline file insertion in HTML Editor

File uploads but link is not created.

Resolved in https://github.com/SaintJosephCatholicChurch/saint-joseph-catholic-church-site/commit/d73bbaba0b92ff601cd0de148d6d6cc1b7976c59


Support async storage libraries like localforage

Hi, I'll try to see what I can do for this :)

I don't think this can be considered for now. The plugin hooks onto pinia's $subscribe that watches mutations and not actions, so whatever we do should also be synchronous. If you're looking to integrate async operations withing the store, you can consider subscribing to actions (more in the pinia docs) and do so in a more "manual" way.
I'll be closing this for now, feel free to reopen if you have a proper solution to implement. Thanks!


Suggestion for new way to trigger bot

To help with accidental triggering of the bot.
I suggest switching from  to  .tachi mysearch

That's how usually bots work, but with tachibot we wanted to be able to seamlessly tag manga inside a message. An example would be /r/anime's anime bot.

could it be more strict then like <.tachi my search>

I don't really like this type of syntax. Do you have examples as to when its accidentally triggering? We could find a more elegant solution.
Worth noting that in channels where the <> syntax is used a lot, you can just deny tachibot reading or sending permissions and the bot just wont work on said channel.

https://discordapp.com/channels/349436576037732353/349466829212876813/664377119362646027

That issue was fixed with #3

triggered just today

Alright, will fix the regex

Commit 887ce12f184925cf525ddb762846507aab13449c should fix this.

Still happening this time on emoji
https://i.imgur.com/hkKFhGd.png

And others
https://i.imgur.com/NKK9I8b.png

Maybe trigger it with <: my manga :>

Its been solved, just waiting on the deployment of the latest fix.

Let me know when it's out there so bot can be reenabled

Not much we can do if they use the syntax... this is an edge case that wont normally happen. This trigger was caused by

why is it parsing urls at all shouldn't it exclude urls from search?

It excludes URLs encased in <>. Matching parts of a URL is a bit less reliable, since you cant really know when it ends. In the first place, it works by matching the <>. Locating where the exact trigger is would just screw up actual attempts.


to stix slect stix field pop up - search doesnt work properly

In the to stix, when you select the select field pop up that allows you to select a field from the stix terminology, if you search something and close the window, and then open the window again - it is still filtered according to the prior search, although the search text box is empty.
steps to reproduce:
open to stix screen. create an object. click the "select field" button.
enter a search term. the options get filtered by the search term.
either select a field or click the close X button.
click the select field again - the options are still filtered as per the prior search term.
should be - all options show up.

fixed & deployed to demo, clear cache if you don't see the updated version


Adding the filled in template also when I add the link [[R: citekey]] manually. It is possible?

Hey,
I have written some scripts for the Zotero Plugin "Actions and Scripts" that generate plaintext links like so:
[[R: Noppel2023Disguising]] [[item](zotero://select/library/items/KNWTWU8Y)] [[pdf](zotero://open-pdf/library/items/NRRCFQYV)]

This enables me to work in Zotero and copy references into LogSeq via Drag and Drop or via Shortcuts. However the respectice page R: Noppel2023Disguising does not contain the template then (and if it would, the template would not be filled out). Can you estimate how much effort it would be to hook your plugin into the creation of a new page and paste the template into it if it starts with R: ? What do you think?
Kind regards,
Max

Sorry I'm not an advanced Zotero user so I really am not sure what you mean. What does hooking the plugin into the creation of a new page mean?

Ah sorry for my fuzzy expressions. I mean hooking plugin in LogSeq, such that whenever an empty page with a name starting with R: is opened, it is execute and fills in the Zotero Template. That way I could manually create link like [[R: Noppel2023Disguising]] and as soon as I click on them, the plugin would do its magic: adding the template and filling it out. As far as I understand it, you only add the template and the fill it out when the /insert zotero full item command is excuted, right?

Ah I see what you mean. Yes, the "magic" happens only when you click on the item that you want to insert and not before that.

Hmm to me it looks different. If I run /Zotero: Insert full item the plugin will add [[R: <key>]]. If I click on that link then, I get to the filled out page, according to the template. However, if I add [[R: <key>]] manually, e.g., by typing, for a new citekey that is not in logseq yet, and if I then click that one, I got to an empty page. Does the plugin store a list of created page internally or is it just triggered by R: ?

Here is what happens when you run '/Zotero: Insert full item' and select the item you want to add:

Creates the page with the title format defined in the plugin settings
Populates the page according to the template
Creates a reference to the page created in Step 1, depending on where you trigger the slash command

Which is exactly what you described. There is no 'hook' in place actually.

Hey,
okay. Thanks for the clarification. I will try to find out if it is even possible to do it the way I desire, and come back here then.
Max


sending menu + other initialisation on /start

Greetings,
If bot.start() is invoked, the  menu.command('start') is not executed.
How can I manage to initialise a bot context and then push the menu to the user on receiving the start command ?
Thank you.

Just like with other middlewares in Telegraf: you pass it to the next middleware if you want to.
bot.command('start', async (ctx, next) => {
  // Initialise stuff

  // go to the next middleware for the same telegram update (if there is any)
  return next();
});

menu.setCommand('start');
bot.use(menu.init());

This way first your bot.command is run. It passes through (via next) to the next available middleware. This will be the menu.
This library currently does not support passing through on setCommand but the way above works fine.
Hope it helps :)

It works. Thank you !


Custom indicator

Hi,
for larger number of slides (>20) I think it would be better to change indicator from dot or rect to number format: "actual / total" e.g. 1/20. Is there any way to achieve this with your library?
Something like using onPageSelected() from OnPageChangeListener to get current position. Total slide count is not a problem, I know how many images are in slider. And finally display this instead of default indicator...
Thanks in advance

Try this out: https://github.com/marten83/CustomIndicators


Consolr updates

In order to support more tools than ipmitool in consolr, I'd like to modularize it a bit. As you can see here, I've moved all the ipmitool-related code in to a class called Consolr::Runners::Ipmitool. The command loads one or more runners dynamically from the config, and chooses the first one that reports it can run on a given node (meaning the runner's can_run? method returns true).
An example use case could be an environment with mixed Openstack VMs and bare metal servers. An openstack runner would then handle all the VM assets and the ipmitool runner all the bare metal assets. This way, you can easily customize consolr to new tools by writing a Consolr::Runners::Yourtool class and then installing it in whichever way you prefer.
In order to maintain backwards compatibility I've made the ipmitool runner the default if no runners are specified in the config.
Finally, I'm adding an option (-S, or --status) which just returns the power status of the assets because it seemed like something that would be useful :)
@tumblr/collins

LGTM

IMO there should be some way to force using a particular runner.
Do you want to add missing ipmitool functions in the PR (e.g. soft reboot vs hard reboot) or is that too much out of scope.
I like the approach though.

Very nice. :+1:

Added support for --soft-reboot and --soft-off.

This LGTM
But, I think we should add a barebones example gem that would show people how to add their own, internal runners with custom logic.

Can you also add unit tests before merging?


can we use this lib for dynamics 365 online ? question

hi there. thank you for your great project. i dont know if it is a good place for this or this is a proper question to ask but can we use pyodata for dynamics365 F&O ?
because we also have odata source and able to use it with powerbi and excel. also from SSIS. but i tried to use pyodata and it gave an error which i have no information :/ but i guess it comes from authentication problem most possible.
thats all. i will be happy if you can answer my question. thank you for your contributions.

Hi @ibrahimhalici - sorry, but there is no real action item on our side from your issue description.
If dynamics365  has odata V2 (not V3/V4) API, then yes you can use pyodata.
Read the docs for this project, especially about how to enable logging and go from there.
Study the issue #119 and #126 as an example how to create issue which can be investigated, ideally reproduce locally by developers.


Fix #3604 - missing enter after stopping exit event

The same event object is being used for enter and exit in fireEnterAndExit(). If the user decides to stop() the exit event, then enter will be stopped as well.
Fix: fire enter with a fresh event object.

Good find, thanks! I imagine it was fun to find. ;)


fix(versions): update Activiti/activiti-cloud-runtime-bundle-service versions into develop

UpdateBot pushed maven dependency: org.activiti.dependencies:activiti-dependencies to: 7.1.99

UpdateBot commands:
updatebot push-version --kind maven org.activiti.dependencies:activiti-dependencies 7.1.99 org.activiti.api:activiti-api-dependencies 7.1.50 org.activiti.core.common:activiti-core-common-dependencies 7.1.57 org.activiti.build:activiti-parent 7.1.28


TODO: Analyze the results

TODO
Dice rolls results analysis.
It must check the combinations of dices and find the number of points associated to the selected 'combo'
It may also check for all the possible combinations (this would be useful as an enhancement to allow the player to directly chose the best option based on the provided analysis).

This method should work as a complement to the method that will check the possible choices for each user.
Don't know where to implement this method though...

I will probably create a Player class to implement this method into.
This class may also hold multiple game methods that will allow the developers to code more easily (i.e. by using only a Player instance for each player)


Type alias spec in oci package

This allows Go to build third party packages correctly without vendoring
issues what want to create their own SpecOpts.
Fixes #2289
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

@williammartin can you take a look at test this solution?

Codecov Report

Merging #2336 into master will not change coverage.
The diff coverage is 44.18%.

@@           Coverage Diff           @@
##           master    #2336   +/-   ##
=======================================
  Coverage   44.98%   44.98%
=======================================
  Files          92       92
  Lines        9340     9340
=======================================
  Hits         4202     4202
  Misses       4459     4459
  Partials      679      679

Flag
Coverage Δ

#linux
49.28% <50%> (ø)
:arrow_up:

#windows
41.24% <23.07%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

oci/spec_windows.go
0% <0%> (ø)
:arrow_up:

oci/spec_opts_windows.go
0% <0%> (ø)
:arrow_up:

oci/spec.go
40% <100%> (ø)
:arrow_up:

oci/spec_unix.go
98.37% <100%> (ø)
:arrow_up:

oci/spec_opts_unix.go
21.77% <42.85%> (ø)
:arrow_up:

oci/spec_opts.go
64.61% <57.14%> (ø)
:arrow_up:

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 a15e7a0...b83f569. Read the comment docs.

Have asked the team to take a look, thanks.

Any word on this fixing the issues you are seeing?

LGTM

Thanks for chasing and sorry for the delay.
The place where we get this error is actually on https://github.com/containerd/containerd/blob/129167132c5e0dbd1b031badae201a432d1bd681/container_opts.go#L199 because we are still building the spec ourselves as a smaller change (https://github.com/cloudfoundry/guardian/blob/master/rundmc/runcontainerd/nerd/nerd.go#L27)
I did try to play the same trick by adding:
type Spec = specs.Spec

and changing the function signature to:
func WithSpec(s *Spec, opts ...oci.SpecOpts) NewContainerOpts

But it didn't resolve anything (not sure if the type is leaking somewhere else):
../../rundmc/runcontainerd/nerd/nerd.go:27:85: cannot use spec (type *"github.com/opencontainers/runtime-spec/specs-go".Spec) as type *"github.com/containerd/containerd/vendor/github.com/opencontainers/runtime-spec/specs-go".Spec in argument to containerd.WithSpec

Can you point me to where you found that type alias would resolve vendor conflicts? I can’t seem to find anything about it and in a simplified case (https://github.com/williammartin/alias) type aliasing doesn’t seem to resolve conflicts (but it’s totally possible I’m doing something stupid). Maybe you can fill me in on what I’m missing.
Finally, because of this, and a bunch of other reasons that have built up over time, we are in the process of switching all our vendoring to a more suitable scheme (dep or vndr) which as discussed previously, should flatten the vendor hierarchy and avoid this whole thing altogether. As such, I suspect we won't really need anything from containerd. Can update you as we progress with that, since it's halfway through in a weird state right now. Thanks.

@williammartin you actually have to change your implementations also to use oci.Spec instead of specs.Spec

@crosbymichael Ah you're right, when I did that in my test example it seemed to work. I can give this a go in Garden when I have a spare moment (infrequently I'm afraid!).

LGTM


Implement LAG construct

Check the LAG construct implementation and documentation in NEURON and add related unit test in NMODL.

@alkino @pramodk this we said is deprecated, right?


anim, doming

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,d]

/trunk merge


Cardfix

Fixed incorrect card code, updates hash format, formated with rubocop specs.

I think it would be better to separate style fixes from bug fixes.
Maybe create another pull request with the styles fixes, and include the rubocop file so people can run it themselves? It could also be setup to run on the CI (via HoundCI).
You should squash the commits.

@louim Thank You .. I am currently studying how to best revert / fix that failing commit .. I have a question in to @alexmamonchik on this problem, and what I think is a best solution.  I share it here:
git stash # saves my current work in my stripe-1.24.0 branch
git checkout cc0f8cf # checks out the breaking commit
git checkout -b fixcc0f8cf # create a new branch from it, make changes in new branch
git commit -m 'fixes ccof8cf' # commits changes in the new branch
git push origin fixcc0f8cf # Issue a PR on this branch, this should fix that broken commit
git checkout stripe-1.24.0
git stash apply

I am still studying this, however ..
Any thoughts you have on how to best fix that failing commit, I am all ears.
I like your idea about a separate PR for rubocop changes, and including it so folks can run it. ( it is entirely overwhelming the first time you run it on the entire gem. you have to run it file by file ) I have done that in my fork's stripe-1.24.0 branch, which is where I am working to bring our stripe-ruby-mock gem up to stripe-1.25.0.  When I have the 1.24.0 branch ready to be called stripe-1.25.0, i will do a squash commit into a new branch named stripe-1.25.0 .  It will have any and all files I changed in any way structurally/code wise, to mirror stripe itself, also changed to the new hash form, and changed to conform as close as possible to rubocop.  I am ending up with much cleaner, clearer and more readable code from these changes.  From this new master, I don't know what will be the best way to bring all those changes into the gem.  Right now, it affects a lot of files, but nowhere near all.
I also just included the simplecov gem, which gives a great picture of the code test coverage.


Change of Secrets in Key Vault are not reflected in Azure Container Apps after the active revision is restarted

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Key Vault is integrated into the Azure Container Apps and the secrets from Key Vault are used as environment variables in the app. When the secrets in the Key Vault are changed and the active revision in Azure Container App is restarted through Azure Portal, the values of environment variables that reference key vault secrets are not updated.
Steps to reproduce

Key Vault is integrated into Azure Container App;
Secrets of latest version are referenced with Key Vault url in Azure Container Apps and are used as envrionment variables in the containers;
A new version of a secret is created in Key Vault;
The active revision of the Azure Container Apps is restarted through Azure Portal (Container App -> Revisions -> Click the active revision -> Restart)

Then the old version is used in the app instead of the latest version of the secret. The app needs to be redeployed through the pipeline to get the updated version.
Expected behavior [What you expected to happen.]
The latest version of secret is used since the Key Vault url does not contain the version and it references the latest version.
Actual behavior [What actually happened.]
The old version is used.

Hi, @pengxo, do you mean old revision is not with the latest secrets?
If yes, can I know is this is a single revision mode app?
Can I also know if the old revision is in inactive status?
If the old revision was in inactive status, we would not poll secrets for it.
If the revision was in active status and restarted without getting latest secret, can I have your app resource Id for further investigation? (please send information to<EMAIL_ADDRESS>

After the support from<EMAIL_ADDRESS>the polling of secrets is container app level operation. The restart of the active revision won't trigger the polling. Only the container app level changes would trigger the polling. More flexibilty of polling will be provided in future. So this is not an issue and I will close it.
Thanks for your support @xwang971

the polling of secrets is container app level operation. The restart of the active revision won't trigger the polling. Only the container app level changes would trigger the polling.

I am curious about this. What is a container app-level change? Do you mean one has to rollout a new revision of the container app so that it gets the latest secret from KV again?

@praneetloke for now yes, any behaviour that trigger a new revision generation would force that revision to have latest secrets. For old revision, it would take at most 30 minutes to get latest secrets. In future this gap could be shorten by customized polling interval(i.e. you can set it to poll secrets every 30seconds)

Stopping and starting the container app in the portal does refresh the referenced kv secrets. I'm not sure how to do this via the az cli yet though..

Well, this works, but beware calls to these endpoints return before the operation is complete so you may need a wait loop if you're stopping and starting:
        az rest -m POST -u "/subscriptions/$SUBSCRIPTION/resourceGroups/$RG/providers/Microsoft.App/containerapps/$APP/start?api-version=2023-04-01-preview"
    az rest -m POST -u "/subscriptions/$SUBSCRIPTION/resourceGroups/$RG/providers/Microsoft.App/containerapps/$APP/stop?api-version=2023-04-01-preview"


Cannot use Play Every Frame and (Playback ping-pong or Playback once) at the same time

When using Play Every Frame, the playback is looped even if Ping-pong or Playback once is selected. It happens whether Memory Cache is enabled or not.

Hi, sorry for the delay, i just checked in a change that should fix this. It works OK in my test cases but if you have some time to try it out that would be great.
One caveat is that ping-pong playback is still broken on movie files until this issue is fixed:
https://github.com/darbyjohnston/DJV/issues/152

Hi, no problem. It’s working on the sequences I previously tested with, thanks!
It seems that you are out of issues for the 2.0.6 milestone… ;)

Great, thanks for testing it out!
Yes, it's nice to finally see 2.0.6 at 100% :) I'll try building the distribution packages tomorrow and if everything goes well, I'll make the release.

OK, I just made the 2.0.6 release, the distribution packages are uploaded and the source code is tagged.


Error: connect ECONNREFUSED <IP_ADDRESS>:55432

🐛 Bug Report
Im have this error when i try to run docker compose up --build :
Error: connect ECONNREFUSED <IP_ADDRESS>:55432
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1555:16)
🤷♀ What did you do
i have used the tool to generate the dockerfile and docker-compose.yaml file .. then i tried to run the app using docker compose up --build , defined hosts for db as localhost and port as 55432

my docker compose file :
version: '3'
services:
  xtech:
    container_name: xtech
    build: .
    image: xtech:latest
    restart: unless-stopped
    env_file: .env
    environment:
      DATABASE_CLIENT: ${DATABASE_CLIENT}
      DATABASE_HOST: xtechDB
      DATABASE_NAME: ${DATABASE_NAME}
      DATABASE_USERNAME: ${DATABASE_USERNAME}
      DATABASE_PORT: ${DATABASE_PORT}
      JWT_SECRET: ${JWT_SECRET}
      ADMIN_JWT_SECRET: ${ADMIN_JWT_SECRET}
      DATABASE_PASSWORD: ${DATABASE_PASSWORD}
      NODE_ENV: ${NODE_ENV}
    volumes:
      - ./config:/opt/app/config
      - ./src:/opt/app/src
      - ./package.json:/opt/package.json
      - ./package-lock.json:/opt/package-lock.json

      - ./.env:/opt/app/.env
      - ./public/uploads:/opt/app/public/uploads
    ports:
      - '1337:1337'
    networks:
      - xtech
    depends_on:
      - xtechDB

  xtechDB:
    container_name: xtechDB
    platform: linux/amd64 #for platform error on Apple M1 chips
    restart: unless-stopped
    env_file: .env
    image: postgres:14.5-alpine
    environment:
      POSTGRES_USER: ${DATABASE_USERNAME}
      POSTGRES_PASSWORD: ${DATABASE_PASSWORD}
      POSTGRES_DB: ${DATABASE_NAME}
    volumes:
      - xtech-data:/var/lib/postgresql/data/ #using a volume
      #- ./data:/var/lib/postgresql/data/ # if you want to use a bind folder

    ports:
      - '55432:55432'
    networks:
      - xtech

  xtechAdminer:
    container_name: xtechAdminer
    image: adminer
    restart: unless-stopped
    ports:
      - '9090:8080'
    environment:
      - ADMINER_DEFAULT_SERVER=xtechDB
    networks:
      - xtech
    depends_on:
      - xtechDB

volumes:
  xtech-data:

networks:
  xtech:
    name: Xtech
    driver: bridge

⛔ Error log
Error: connect ECONNREFUSED <IP_ADDRESS>:55432
at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1555:16)
🕵♀ Stack trace
><EMAIL_ADDRESS>develop
> strapi develop

- Building build context

[INFO] Including the following ENV variables as part of the JS bundle:
    - ADMIN_PATH
    - STRAPI_ADMIN_BACKEND_URL
    - STRAPI_TELEMETRY_DISABLED
✔ Building build context (69ms)
- Creating admin

><EMAIL_ADDRESS>develop
> strapi develop

- Building build context

[INFO] Including the following ENV variables as part of the JS bundle:
    - ADMIN_PATH
    - STRAPI_ADMIN_BACKEND_URL
    - STRAPI_TELEMETRY_DISABLED
✔ Building build context (64ms)
- Creating admin
✔ Creating admin (18349ms)
- Loading Strapi
[ERROR]  There seems to be an unexpected error, try again with --debug for more information

┌───────────────────────────────────────────────────────────────────────────┐
│                                                                           │
│   Error: connect ECONNREFUSED <IP_ADDRESS>:55432                            │
│       at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1555:16)   │
│                                                                           │
└───────────────────────────────────────────────────────────────────────────┘
 *  Terminal will be reused by tasks, press any key to close it.

## 🙇♀ Expected behavior/code

Show have start the app normally and connect to database

## 👩💻 Environment

- 📦 Node version: 18 (based on the docker compose file)

Default port for poatgres is 5432 not 55432

it worked after changing mapping to default port ,, thank you
but now it is not creating the user as expected . i get
xtechDB       | 2024-02-06 13:55:51.451 UTC [35] FATAL:  password authentication failed for user "xtechdbuser"
xtechDB       | 2024-02-06 13:55:51.451 UTC [35] DETAIL:  Role "xtechdbuser" does not exist.```

Ahmed, Try with the default settings first.
For example:

run #1: Use all default settings.  Run with docker-compose up
run #2: set your own env vars, in the .env file, as shown below.  Run with docker-compose up
run #3: maybe for this example, start trying to use the Dockerfile.prod file by specifying it in the docker-compose.yml file

You'll need to set these items as env vars, in the .env file:
DATABASE_CLIENT=postgres
DATABASE_HOST=localhost
DATABASE_NAME=strapi
DATABASE_USERNAME=strapi
DATABASE_PASSWORD=someGoodPassword
DATABASE_PORT=5432
JWT_SECRET=someKeyFromRandomKeyGenerator
ADMIN_JWT_SECRET=someKeyFromRandomKeyGenerator
NODE_ENV=development

# has to be <IP_ADDRESS> to be reached by laptop running locally
HOST=<IP_ADDRESS>
# HOST=localhost
PORT=1337
APP_KEYS=someKeyFromRandomKeyGenerator1,someKeyFromRandomKeyGenerator2,someKeyFromRandomKeyGenerator3,someKeyFromRandomKeyGenerator4
API_TOKEN_SALT=someKeyFromRandomKeyGenerator
TRANSFER_TOKEN_SALT=someKeyFromRandomKeyGenerator


[MRG] Keep order of variables in LabelEncoder

The problem
The current behavior of the LabelEncoder is to sort the variables when the mapping is performed. This happens because of the use of np.unique which returns a sorted array of unique values: See https://github.com/Elementa-Engineering/scikit-optimize/blob/master/skopt/space/transformers.py#L175-L177 and https://numpy.org/doc/stable/reference/generated/numpy.unique.html
For example:
>>> from skopt.space.space import Categorical

>>> c = Categorical(("c", "b", "a"), transform="label")
>>> c.transform(["a", "b", "c"])
[0, 1, 2]

Note that the returned labels are 0, 1 and 2 (equivalent to ("a", "b", "c") even if the specified order was ("c", "b", "a")). This can be counter-intuitive, especially when the order of the variable "means" something for the user.
Implemented Fix
This PR, implements a simple fix, which retains the order of the categorical dimensions. The expected behavior then becomes:
>>> from skopt.space.space import Categorical

>>> c = Categorical(("c", "b", "a"), transform="label")
>>> c.transform(["a", "b", "c"])
[2, 1, 0]

The order is conserved.
Same goes for numerical numbers:
from skopt.space.space import Categorical

c = Categorical((10, 30, 20), transform="label")
c.transform([10, 20, 30])
[0, 2, 1]

@kernc, not sure why CI didn't fire up here, but this is ready for a review. :)

Try to push a commit again to trigger the CI, perhaps ?

Try to push a commit again to trigger the CI, perhaps ?

Still not working! Weird!

Well, I'm at a loss here... Have you run the tests locally using pytest ? Maybe the CI is straigh crashing on this PR, hence no info ?


game instance tree state lost when indexing up ancestors using .Parent

when trying to require anything from script.Parent, I get messages like this:
src/Array/concat.lua(3,25): TypeError: Key 'isArray' not found in class 'Instance'

it looks like in the globalTypes.d.lua, it declares script: LuaSourceContainer. I can change that to any and  it no longer emits this error as a workaround. It would be nice to not have to trade one for the other, though.

note that it doesn't actually affect require, but just indexing the tree.
require(script.Parent.Child) will resolve correctly and return typed code but the script.Parent.Child indexing part will currently error as we lose the game tree state and don't know that Child is a descendant of script.Parent.

https://github.com/JohnnyMorganz/luau-analyze-rojo/commit/80f68a924f1003c570e1067dfe8e8a9b0290bbc5 temporarily marks script as any, which will hopefully silence most of these spurious type errors for now


The reset of useSandpackConsole does not work

Bug report
Packages affected

[ ] sandpack-client
[x] sandpack-react

Description of the problem
What were you doing when the problem occurred?
I have a custom CleanIcon requirement for console, and when I used reset of useSandpackConsole to clear the log, nothing happened.
What steps can we take to reproduce the problem?
CleanIcon looks like this：
const CleanIcon = () => {
  const { reset } = useSandpackConsole();

Clean

}

Here's how I use it
<SandpackProvider template="react" files={files}>
  <CleanIcon />
  <SandpackCodeEditor />
  <SandpackPreview />
  <SandpackConsole />
</SandpackProvider>

Link to sandbox: link (optional)
Your Environment

Software
Name/Version

Sandpack-client version

Sandpack-react version
2.0.1

Browser

Operating System

Hey, @wyy0512!
Actually, useSandpackConsole is working correctly. Each CleanIcon and SandpackConsole contain its own console context, and when you try to reset the consoles from CleanIcon it won't trigger the same action on other's context. Does it make sense?
So, I can see two options here to manage your issue:

You could build your own Console component and render the console.logs, which would give you full control of resetting it;
SandpackConsole implements a new API to clean the logs imperatively by React.ref, which would looks something like this:

// proposal solution, not implement yet
const consoleRef = useRef();

const onCleanConsole = () => {
  consoleRef.current?.reset();
}

return <SandpackConsole ref={consoleRef} />

@danilowoz
Thank you very much for taking the time to answer my questions. The second solution looks great and fits my needs better, and I'm looking forward to the new api for SandpackConsole


[Good First Issue]: Verify youri-7b-chat with GenAI text_generation

Context
This task regards enabling tests for youri-7b-chat. You can find more details under openvino_notebooks LLM chatbot README.md.
Please ask general questions in the main issue at https://github.com/openvinotoolkit/openvino.genai/issues/259
What needs to be done?
Described in the main Discussion issue at: https://github.com/openvinotoolkit/openvino.genai/issues/259
Example Pull Requests
Described in the main Discussion issue at: https://github.com/openvinotoolkit/openvino.genai/issues/259
Resources

Contribution guide - start here!
Intel DevHub Discord channel - engage in discussions, ask questions and talk to OpenVINO developers

Contact points
Described in the main Discussion issue at: https://github.com/openvinotoolkit/openvino.genai/issues/259
Ticket
No response

.take

Hello! @p-wysocki I am these errors while downloading and convert the model and tokenizers, during the process, warnings related to TracerWarning appear. These warnings indicate potential issues with converting tensors to Python boolean values, which might cause inaccuracies in the tracing process.
The script tries to send some events, presumably for logging or monitoring purposes. However, it encounters errors related to SSL connections and HTTP errors. These errors seem to be related to network connectivity or server-side issues. I made sure that all dependencies are up to date. Is there anything I am missing maybe ? Please guide me and help me regarding this. Thank you!

cc @pavel-esir

I never encountered this. But it feels it's your SSL set up issue. If searching doesn't help I'd propose to try a different machine. Maybe it's your firewall or something.

.take

Hi@p-wysocki, I was looking at the #259 and I am trying to understand what's needed to be done here. The gist is that I have to verify this LLM and add it to a sort of list in text_generation/causal_lm/cpp/README.md? But I can't find any text_generation directories in the openvino.genai repo

The samples were regrouped. They are here now:

beam_search_causal_lm
benchmark_genai
chat_sample
greedy_causal_lm
multinomial_causal_lm
prompt_lookup_decoding_lm
speculative_decoding_lm

.take

@mentors i am new to open source, and this is my first one so please bear with me if my doubt is really silly
so i built openvino successfully and when i tried to install dependencies from here
i get this error
and i have boiled down the cause of this error, it is thransformers-steam-generator, i tried to search up a little on this and tried to use chatgpt here, chatgpt suggestied me to install jstyleson, which also gives this error, im not sure what is causing this error and or how to fix this, please mods if you could help that would be really great
im on macos (sequoia 15.2)
some posts also told me that it might be because im using an unsupported python version so my python version is 3.11.2
and yeah idts i have seen anything else related to this on the internet
error:

yep i have the requirements satisfied

I don't have a good solution.

Find who's dependency is thransformers-steam-generator and jstyleson and try removing it. Maybe they are not required for your task. You can also contact the authors.

Try downgrading python.

i tried to install dependencies from here

From where?

Is the original error message the same as in your screenshot?

i'm sorry i might come off as annoying right now but because im stuck at an issue and i dont really know how to describe my issue i might have to type out every step that i have followed
as you suggested, i tried downgrading my python version and endedup installing 3 different python versions on my mac, so it created a whole lot of other issues so i decided to delete everything and start over again,
i will list all steps that i followed so that if in case i have made any mistakes you guys can correct me
i started off by following this link
brew install coreutils scons
xcode-select --install
and then i tried to take a look at @rk119's work on :
[Good First Issue]: Verify chatglm3-6b with GenAI text_generation
this step in particular
and so after that i tried doing the same thing on my mac
git clone --recursive https://github.com/openvinotoolkit/openvino.genai.git cd openvino.genai git submodule update --remote --init cd ../ curl --output ov.tgz https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.0.0-17646-691385e7dde/m_openvino_toolkit_macos_12_6_2025.0.0.dev20241218_arm64.tgz tar -xzf ov.tgz mv m_openvino_toolkit_macos_12_6_2025.0.0.dev20241218_arm64 openvino.genai/ov cd openvino.genai source ov/setupvars.sh
then proceeded with
cmake -DCMAKE_BUILD_TYPE=Release -S ./ -B ./build/
cmake --install ./build/ --config Release --prefix ov
cmake --build ./build/ --config Release --target package -- -j10
(at this point im assuming my thing is built)
then i went to the readme file
and follow the steps given
`pip install --upgrade-strategy eager -r ../../export-requirements.txt
optimum-cli export openvino --trust-remote-code --model TinyLlama/TinyLlama-1.1B-Chat-v1.0 TinyLlama-1.1B-Chat-v1.0`
afnd then i follow the steps given here to build the sample
cd /Users/shubhamsachdeva/openvino.genai/ov/ source setupvars.sh build_samples.sh mkdir build cd build cmake -DCMAKE_BUILD_TYPE=Release /Users/shubhamsachdeva/openvino.genai/ov/samples/cpp

You've built the samples two times and the third time failed. My guess is that you switched terminals between source setupvars.sh and cmake -DCMAKE_BUILD_TYPE=Release /Users/shubhamsachdeva/openvino.genai/ov/samples/cpp which resulted in error. But you can use samples from two previous successful builds.

The first one was build as part of GenAI compilation: cmake -DCMAKE_BUILD_TYPE=Release -S ./ -B ./build/
 cmake --install ./build/ --config Release --prefix ov
 cmake --build ./build/ --config Release --target package -- -j10

You should be able to find the built samples in /Users/shubhamsachdeva/openvino.genai/build/samples/cpp/. For example greedy_causal_lm should be here /Users/shubhamsachdeva/openvino.genai/build/samples/cpp/greedy_causal_lm/greedy_causal_lm
The second samples build was triggered by build_samples.sh. But let's stick to the artefacts produced by the previous point.

.take

hey i tried doing it and it worked for tinyllama but then when i try to do it for youri it for some reason terminates the task


Cannot run a singe .bal file by giving the path to the file

Description:
When you try to run a .bal file from outside the directory in which the file resides in, you get the following error:
~/wso2/ballerina/examples (release-stage)$ ballerina run records/records.bal
error: no ballerina source files found in module 'records.bal'

Steps to reproduce:
Try to run a BBE from the root directory of the BBEs (<BALLERINA_REPO>/examples)

The same issue is reported here as well: https://github.com/ballerina-platform/ballerina-lang/issues/20454


Incompatible Swift version - framework was built with 3.0.2 and the local version is 3.1.

What did you do?
updated to swift 3.1
remove Carthage folder
carthage update --platform IOS
What did you expect to happen?
Should download and install dependency
What actually happened instead?
*** Downloading SwiftyJSON.framework binary at "3.1.4"
*** Skipped installing SwiftyJSON.framework binary due to the error:
        "Incompatible Swift version - framework was built with 3.0.2 and the local version is 3.1."
*** Checking out SwiftyJSON at "3.1.4"
*** xcodebuild output can be found in /var/folders/1n/98nykp5d1597hwvr40kn93xw0000gn/T/carthage-xcodebuild.GjfyaR.log

Environment
List the software versions you're using:

SwiftyJSON: 3.1.4
Xcode Version: 8.3.1 (????) (Open Xcode; In menubar: Xcode > About Xcode)
Swift Version: 3.1 (Open Xcode Preferences; Components > Toolchains. If none, use Xcode Default.)

Please also mention which package manager you used and its version. Delete the
other package managers in this list:

Carthage: 0.20.1 (Use carthage version in Terminal)

When running Carthage update, you need to specify an additional switch:
carthage update --platform IOS --no-use-binaries

I think we can close this. It looks like this solution worked for someone else as well as myself. Additionally, it looks like 5 people upvoted the issue that haven't upvoted the solution, which means they may not know it exists. Check this out: Tell Me When It Closes.

@pachun @spurdow  Unfortunately, it is unsafe to publish a pre-built framework until Swift 4.0 ABI get stable.
See https://github.com/Quick/Quick/issues/700#issuecomment-287955781

In case anyone has the same problem,
What i did was  executing this one carthage update --platform iOS --no-use-binaries
unfortunately it will not work yet most of the times. but after i removed Carthage folder and re run again carthage update --platform iOS --no-use-binaries.
and also restart the xcode,  it should work.
Maybe i was wrong because some dependencies I have also got error. So maybe executing carthage update --platform iOS --no-use-binaries should suffice. But if you have the same dependency list in your Cartfile then I think try what I did.
github "Alamofire/Alamofire" == 4.4.0
github "SwiftyJSON/SwiftyJSON" == 3.1.4
github "tristanhimmelman/AlamofireObjectMapper" == 4.0.2
github "Hearst-DD/ObjectMapper" == 2.2.3
github "realm/realm-cocoa" == 2.5.0
github "kaandedeoglu/KDCircularProgress"
github "robbiehanson/CocoaAsyncSocket" "master"
github "mindsnacks/MSWeakTimer" "master"
github "emqtt/CocoaMQTT" "master"
github "cesarferreira/SwiftEventBus" == 2.2.0
github "dekatotoro/SlideMenuControllerSwift" "master"
github "kean/Nuke" ~> 5.0

Thanks for the following up! Glad you work fine. @spurdow

I'm experiencing the same issue, however the --no-use-binaries update has no effect, nor does any sequence of restarts, clearing of derived data, or specifying == 3.1.4 in the cartfile. I'm at a loss as to what to try next.
exact message is:
Module compiled with Swift 3.0.1 cannot be imported in Swift 3.1
Are there any other things I could try?

@agibsonuk have you tried removing Carthage folder and re run again carthage update --platform iOS --no-use-binaries?

Yes I had tried that with no affect, but I found the cause eventually.
Using xcrun swift -version in terminal I identified that Xcode was using 3.0.2, which I think was linked to the fact I had an older version of Xcode Beta on my machine as well. This was causing SwiftyJSON to be built with 3.0.2 with --no-use-binaries
I removed the beta version which prompted Xcode to run Swift 3.1, so all good now.
Thanks!


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
94280fca7f94811c8efcfcd2f2cba7168f50f3a8
link
/test e2e-suite-rbac-no_auth

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

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
94280fca7f94811c8efcfcd2f2cba7168f50f3a8
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-rbac-auth.sh
94280fca7f94811c8efcfcd2f2cba7168f50f3a8
link
/test e2e-suite-rbac-auth

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

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-suite-rbac-no_auth.sh
94280fca7f94811c8efcfcd2f2cba7168f50f3a8
link
/test e2e-suite-rbac-no_auth

prow/e2e-suite-rbac-auth.sh
94280fca7f94811c8efcfcd2f2cba7168f50f3a8
link
/test e2e-suite-rbac-auth

prow/e2e-cluster_wide-auth.sh
94280fca7f94811c8efcfcd2f2cba7168f50f3a8
link
/test e2e-cluster_wide-auth

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


TEP-0135 isolating pipelineruns and multitenancy

This feature uses podAffinityTerm - only running one PipelineRun per worker node using a label selector. However if there are multiple namespaces/tenants using Tekton, if there are PipelineRuns of one tenant already running a node, it'll prevent a PipelineRuns from another tenant from scheduling there.
Is this the desired behaviour? Or should the label selector include a namespace, making it one PipelineRun per tenant/namespace can run on each worker node?

Hi @jimmyjones2 👋 . Thanks for your questions.
I think the current behavior is expected and a namespace is not involved. This is an alpha feature and we are looking for feedbacks for it: https://github.com/tektoncd/pipeline/issues/6990.
Please feel free to start a FR with your use case. We are happy to review and improve!

Thanks @jimmyjones2 and @QuanZhang-William; this sounds like a reasonable feature to support! I think we can use this issue for tracking the FR; @jimmyjones2 would you mind giving a bit more detail on your use case?

Hey @lbernick - not sure if there is too much more detail to give! My use case would be shared clusters - I think in the general case you wouldn't want the activity of one tenant to affect another


New Async API Spec does not clarify how an "empty" Link should be reported

Something I just stumbled on, when navigating the Async compliant ssh provider to an empty directory on a filesystem, it returns nothing except complete. No data is returned (which is compliant with the API as the API doesn't keep track of if data has been returned, just that the data that is returned is valid), and thus the consumer has absolutely no idea what happened and it is told the read was completed.
We should probably update the spec a bit to require the TYPE key in the data field with every data return (even empty data tables).

Closed with #166


[Fine tuning] Pass hyper-parameters to SFTTraining in a flexible/generic way

test passed

test passed

/test rhoai-psap fine_tuning

test passed, merging


Scenario.combine(phases=["1st", "3rd"]) does not delete 2nd phase

Summary
Scenario.combine(phases=["1st", "3rd"]) does not delete 2nd phase and raises ValueError.
(Optional) Related classes

covsirphy.Scenario
covsirphy.PhaseUnit

Codes and outputs:
import covsirphy as cs
# Dataset preparation
data_loader = cs.DataLoader("input")
jhu_data = data_loader.jhu()
population_data = data_loader.population()
# Scenario analysis
snl = cs.Scenario(jhu_data, population_data, "Japan")
snl.add(end_date="31Mar2020")
snl.add(end_date="30Apr2020")
snl.add(end_date="31May2020")
snl.add(end_date="30Jun2020")
snl.combine(phases=["1st", "3rd"])
snl.summary()

This code should returns a summary dataframe with 0th (-31Mar2020) and 1st (01Apr2020 -30Jun2020), but raises KeyError: '3rd phase is not registered.'
Environment

CovsirPhy version: 2.8.2
Python version; 3.8
Installation: pipenv
OS: WSL (Ubuntu)

This should returns

Type
Start
End
Population

0th
Past
06Feb2020
31Mar2020
126529100

1st
Past
01Apr2020
30Jun2020
126529100


Rename type to configuration type(declarative config)

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #6172

PR #6184   👈

This comment was auto-generated by Graphite.

@ivan-degtiarenko a proper PR description would be nice 😸


Make test executor a statically-linked executable

https://github.com/scalapb/scalapb-validate relies on PGV's test executor to drive its tests. For better local development experience and easier CI testing, I'd like to have a pre-built statically linked executable for the executor so it can be used on different Linux distros.
Some more background: up until recently, a simple bazel build tests/harness/executor was sufficient to get the executor built on both OS X and Linux, locally and on Github Actions. I am not sure what had changed (maybe the ubuntu image on Github?) but I've been running into different build errors involving bazel, go and protoc. In interest of saving time debugging this, my preference is to use the Dockerfile provided with PGV for a reproducible build of the executor, and possibly extract it out of the image for local usage. To make sure it works across distros it should be statically linked.

It looks like go_binary supports this natively? https://github.com/bazelbuild/rules_go/blob/master/go/modes.rst#building-static-binaries


Add filtering to published events API endpoint

Filtered values:

administrator organization
host

Both take IDs.
Additionally a new value is being returned by the API: event's
eventbrite ID.
This fixes #529.

On Sat, Nov 21, 2015 at 08:14:21AM -0800, Piotr Banaszkiewicz wrote:

Filtered values:

administrator organization
host

Looks good to me.  I'd recommend documenting the API.  At work, we use
Swagger 1, development looks like 2, and the rendered UI looks
like 3.  But having docs/api.md with a list of Markdown descriptions
would work too.  Anything that gives folks who are consuming AMY's API
(but not actually working on AMY's code) an overview of whats
available and details on how the endpoints work.

@wking
We have a browsable API and I extended it to show query parameters too:

This metadata is available after you click "OPTIONS" in http://<IP_ADDRESS>:8000/api/v1/events/published/ or https://amy.software-carpentry.org/api/v1/events/published/.


Range error

Steps to Reproduce:

Open import categories for CSV stuff
try to import a csv file that is TRASH - aka make everything out of range
Import. It will get mad at u and be lke no ty.
3.Back button on browser to go back to the page to reupload
Import another trash csv file
Press import again

Expected behavior:
It says ok no thank u u suck. Do it again and right, this time.
Actual behavior:
It says you are wrong and something went wrong and is also trash.
SS:

2017-02-07T02:47:03.797924+00:00 app[web.1]: Completed 500 Internal Server Error in 30ms (ActiveRecord: 9.9ms)
2017-02-07T02:47:03.799597+00:00 app[web.1]:
2017-02-07T02:47:03.799599+00:00 app[web.1]: RangeError<PHONE_NUMBER>98237498239849283423 is out of range for ActiveRecord::ConnectionAdapters::PostgreSQL::OID::Integer with limit 4):
2017-02-07T02:47:03.799600+00:00 app[web.1]:   lib/equipment_import.rb:25:in block in import_cats' 2017-02-07T02:47:03.799601+00:00 app[web.1]:   lib/equipment_import.rb:14:in each'
2017-02-07T02:47:03.799601+00:00 app[web.1]:   lib/equipment_import.rb:14:in import_cats' 2017-02-07T02:47:03.799602+00:00 app[web.1]:   app/controllers/import_equipment_controller.rb:28:in import'
2017-02-07T02:47:03.799602+00:00 app[web.1]:
2017-02-07T02:47:03.799603+00:00 app[web.1]:
(Log from heroku)

see #1549. This is a duplicate of that bug

Closing as duplicate.


uucp, autojump: fix cross compilation

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[x] riscv64-linux (cross)
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Autojump uses python at runtime, so moving it to nativeBuildInputs breaks it if there is no python in the PATH (which is the case on my system).

You are right. It also needs to be in buildInputs.

see https://github.com/NixOS/nixpkgs/pull/202363


Running a game with a RigidBody using the "_integrate_forces(state)" function on a built-in script lags my PC

Godot version:
Godot 3.2.1
OS/device including version:
Macbook pro OS: Catalina 10.15.4
Issue description:
https://youtu.be/JnHyyacYnOA
Steps to reproduce:
-prepare a RigidBody
-attach a built-in script
-add the function: "func _integrate_forces(state)"
-run, at least try...
Minimal reproduction project:
testing.zip

Duplicate of #38454.

shouldn't this be it's own thing since it happens for another reason?

Duplicate of #38454.
It lags whenever I enter my password, using _integrate_forces on a built-in script when running triggers the prompt to do so. The other post talks about it just lagging whenever the prompt gets my password.

From the description of both issues it sounded like the same thing.

From the description of both issues it sounded like the same thing.

the other one's the real problem I agree, but wouldn't it be safe to point out that this specific function triggers a prompt that requires a password? I mean, I imagine that this doesn't really happen in other systems...

I think I solved it... It's stupid but, I just forgot to set contact reported and turn on custom integrator and contact monitor.


refactor: rewrite proxy/provider plugin mechanism

What this PR does / why we need it:
Add your description
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @whalecold

Release note:

NONE

/lgtm

/cherrypick release-0.5


missing lib/libcurl.4.dylib in mac os x distribution

I am attempting to use navcoin-cli provided by navcoin-4.1.2-osx64.tar.gz for testing purposes - and I get the following error when attempting to execute it:
dyld: Library not loaded: /home/ubuntu/build/navcoin-core/depends/x86_64-apple-darwin11/lib/libcurl.4.dylib
  Referenced from: /Users/brian/Downloads/navcoin-4.1.2/bin/./navcoin-cli
  Reason: Incompatible library version: navcoin-cli requires version 9.0.0 or later, but libcurl.4.dylib provides version 7.0.0
Abort trap: 6

Any help would be appreciated.
Thank you :)

Elegantt data. What's this?

Have exactly the same issue.
My commandline curl is up to date, I am not sure where it's trying to reference / look for curl.

Solved, pending pr


Do you have a JavaScript client that can connect to an instance of this running on the server?

Do you have a JavaScript client that can connect to an instance of this running on the server?

Found https://github.com/permitio/fastapi_websocket_pubsub/discussions/18

Found #18

As I noted in #18 - If you want to pickup that challenge we'd be happy to provide guidance.


Added Metadata for SageMath.SageMath v9.2

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

PS C:\Users\Esco> winget install SageMath.SageMath
Found SageMath [SageMath.SageMath]
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/sagemath/sage-windows/releases/download/0.6.2-9.2/SageMath-9.2-Installer-v0.6.2.exe
  ██████████████████████████████   815 MB /  815 MB
Successfully verified installer hash
Starting package install...
Successfully installed

Microsoft Reviewers: Open in CodeFlow

Hey @OfficialEsco ,
Sorry for the delays, I had overlooked this PR. I had issues installing this on an x86 host machine. Can you try setting the architecture to x64? That worked for me.

Lets try again


show unity on Flutter

I wanted to show Unity on Flutter, so I customized The Platformview in the Android project; The flutter side I take two ways to receive the SurfaceView;|

one：

flutter code

 ` AndroidView(
    viewType: 'unity_view',
    onPlatformViewCreated: onPlatformViewCreated,
  );`

android code

 ` FlutterUnityView(PluginUnityPlugin plugin, Context context, int id) {
    this.context = context;
    PluginUnityPlugin.views.add(this);
    this.plugin = plugin;
    this.id = id;
    view = new FrameLayout(context);
    view.setBackgroundColor(Color.TRANSPARENT);
    surfaceView = new UnitySurfaceView(context);       //it‘s UnityPlayer
    view.addView(surfaceView);
    }
    @Override
   public View getView() {
    return view;
     }

`
This way on huawei phones, the second start will appear white screen
https://user-images.githubusercontent.com/22164576/103138201-1fe13f80-470b-11eb-9203-75882d8f9056.mp4

two

flutter code

 ` return PlatformViewLink(
        viewType: viewType,
        surfaceFactory: (BuildContext context,
            PlatformViewController controller) {
            controller.clearFocus();
          return AndroidViewSurface(
            controller: controller,
            gestureRecognizers: const <
                Factory<OneSequenceGestureRecognizer>>{
            },
            hitTestBehavior: PlatformViewHitTestBehavior.opaque,
          );
        },
        onCreatePlatformView: (PlatformViewCreationParams params) {
          return PlatformViewsService.initSurfaceAndroidView(
              id: params.id,
              viewType: viewType,
              layoutDirection: TextDirection.ltr,
              creationParamsCodec: StandardMessageCodec(),
              onFocus: () {
                print("unity onFocus");
              }
          )
            ..addOnPlatformViewCreatedListener(params
                .onPlatformViewCreated)..addOnPlatformViewCreatedListener(
                onPlatformViewCreated)
            ..create();
        }
    )`

android code

 ` FlutterUnityView(PluginUnityPlugin plugin, Context context, int id) {
    this.context = context;
    PluginUnityPlugin.views.add(this);
    this.plugin = plugin;
    this.id = id;
    view = new FrameLayout(context);
    view.setBackgroundColor(Color.TRANSPARENT);
    surfaceView = new UnitySurfaceView(context);       //it‘s UnityPlayer
    view.addView(surfaceView);
    }
    @Override
   public View getView() {
    return view;
     }

`
In this way, on the vivo phone, the PlatformViewLink will always be floating on the page. Switching the page, I call unityplayer.pause()
https://user-images.githubusercontent.com/22164576/103138297-042a6900-470c-11eb-9f93-d4ee858974f5.mp4

Page structure

Scaffold () + PageView
please help me！

@JuneLeo
this doesn't seem a bug with flutter or a feature request, but rather assistance for your code
I recommend to follow the link at I want help writing my application in the new issue selection

@JuneLeo
this doesn't seem a bug with flutter or a feature request, but rather assistance for your code
I recommend to follow the link at I want help writing my application in the new issue selection

I think it was a bug with flutter, because there was no problem with most phones, only some phones had this problem!

@JuneLeo
I am guessing you are using https://pub.dev/packages/flutter_unity_widget. I'd suggest you to open this issue and follow-up in plugin owner's repo.
Closing this. If you disagree, write in comments and I'll reopen.

@JuneLeo
about unity support you may follow up here https://github.com/flutter/flutter/issues/18509

@JuneLeo
about unity support you may follow up here https://github.com/flutter/flutter/issues/18509

I solved the problem by using TextureView replace SurfaceView;

I solved the problem by using TextureView replace SurfaceView;


DirectX 12 not working

So i tried this with DirectX 12 64 bit game and nothing shows up in the game when i press insert, game doesn't crash either. Here's what the console window says: https://prnt.sc/zsRN7VXGFsEV. It would be great to get this working.

whats the game name?

test it whenever you have time, im waiting for a reply. I hooked a lot more thing and pressing home should work for dx12 right now. you wont have mouse visible but thats just cuz i dont want to get in that area. input handling is left to the user

First of all thank you so much for making it work for this game, if you don't mind me asking what you had to change in code for it to work i would like to know. Also i can continue from here, i think the hardest part is already done now when its actually showing up ingame. im gonna fix the mouse and everything else. Thank you.

happy to see it worked. To make it "work" i hooked 6 more functions, the first 2 are Present1 and ResizeBuffers1, didn't know there are 2 present functions that can be called and i hooked ResizeBuffers1 to handle window resizing BUT the game doesn't call that when resizing? I noticed that when resizing the swapChain pointer changes. I hooked all 4 functions that create a swapchain so i can safely release the rendertarget (with CleanupRenderTarget() function) without the game crashing.
If you want the DLL to inject to the game as quick as possible you must create your own exe "loader". It should be fairly simple since you create an exe that starts KINGDOM HEARTS HD 1.5+2.5 ReMIX.exe and then get its PID. I ll give you a pseudocode.
int main() {
    DWORD pID = 0;
    StartGame();

    while (pID == 0) {
        GetGamePID(); // You should search on google "CreateToolhelp32Snapshot" or "EnumProcesses"
// i prefer EnumProcesses. compare if a process name matches your process name and break, there you go you have your process pid.
    }

    InjectDLLIntoProcess(); // many ways to do it i recommend loadlibrary because of its simplicity

/* THIS IS SOME OLD CODE FROM 2020 MAYBE
auto* const h_proc = OpenProcess(PROCESS_ALL_ACCESS, 0, pID);

	if (h_proc && h_proc != INVALID_HANDLE_VALUE)
	{
		auto* loc = VirtualAllocEx(h_proc, nullptr, MAX_PATH, MEM_COMMIT | MEM_RESERVE, PAGE_READWRITE);
		WriteProcessMemory(h_proc, loc, dll_path, strlen(dll_path) + 1, nullptr);
		auto* const h_thread = CreateRemoteThread(h_proc, nullptr, 0, reinterpret_cast<LPTHREAD_START_ROUTINE>(LoadLibraryA), loc, 0, nullptr);

		if (h_thread) CloseHandle(h_thread);
	}
*/
} // make sure u build the loader with the same architecture as the game

EnumProcesses

Thanks for that pseudocode, I think the game uses directinput8 and I would have to hook to it or somehow block mouse input from it but I have never done that so that's going to be a challenge.

i think you should look into GetDeviceData for that


Open Interactive python console per file

I am migrating from PyDev to VSCode.
Many times I have to work with multiple python files.
But not able to figure out how to run each file in its own Interactive python console (just like multiple Jupyter notebooks )
Can someone please guide me ?

Sorry the current implementation is that the files all share the interactive window. It's more like a terminal than a jupyter notebook.

I don't know if per-file is the right requirement, but much like you said @rchiodo - where the interactive window is like a terminal - I think there is definite justification for allowing more than one Interactive Window to be open. Specifically, this comes up a lot in data science where one Interactive Window is bound to the local machine, and another to some cloud/HPC/gpu-powered remote machine. Having both open in the IDE is invaluable. Would you consider this? Should I open a different issue?
Thanks

@syagev 	I think this issue should be sufficient to ask for multiple windows. We'd need to figure out how the user specifies the jupyter kernel to connect to (assuming we support local/remote at the same time).


Add an example of  "wallet-worker.html"

README provides this example:
{
  "icons": [
    {
      "sizes": "48x48 64x64",
      "src": "demo-wallet.png",
      "type": "image/png"
    }
  ],
  "name": "Demo Wallet",
  "short_name": "Demo Wallet",
  "credential_handler": {
    "url": "/wallet-worker.html",
    "enabledTypes": ["VerifiablePresentation"]
  }
}

But no html for wallet-worker is available in repo.

I'm trying to work through a nextjs / typescript wallet example, I will share the react / server side rendering issues here... but so far so good.

Any progress or examples?

The README has been updated to link to chapi.io which gives a much better introduction to CHAPI along with demos, examples, and a playground. Those examples include an example worker html file. I'm going to close this on that basis.


Provide network/storage/accelerator power attribution

what will be add in addition to our existing model (e.g the network card, the SAN or the GPU/DPU ) etc??

There are two approaches that we can take to tackle this issue.
First, we can conduct a fine-grain estimation of the power consumption of each process for a given device (i.e., CPU, DRAM, and GPU). Alternatively, we can estimate the total power consumption of the node using multiple metrics, leading to a coarse-grain estimation (i.e., Platform power).
The Kepler code already has the coarse-grain (i.e., Platform power) and fine-grain (i.e., CPU, DRAM, and GPU) estimations.
Therefore, If we can collect network and storage power consumption data, we can develop a power model to estimate the power consumption per process.
Furthermore, we can improve the current Platform power model by including more metrics that better reflect the utilization of other resources. However, we must ensure that the collected Platform power includes the power consumption of the target resource.

network and storage power consumption data

I assume the network = NIC /DPU, yes, this seems ok, so either coarse-grain or fine-grain should be able to provide such data
but storage usually SAN/SDS will be used which is distributed, so maybe not that easy..

Due to accelerator power attribution topic, I think there are two kinds of definition of the word "accelerator":

Stand-alone HW accelerator, such as GPU. DPU also can be regarded as such accelerator since it offloads specific computing from CPU.
CPU built-in accelerator, such as Intel Xeon SP's bunch of in-die/in-package accelerators.
The work mechanism should be greatly different between the two. So let's think about how to organize the current code hierarchy of pkg/power and define appropriate framework to support different kinds of accelerators.

Sounds good.
But note that CPU built-in accelerator on Intel Processors is part of the RAPL UNCORE power.

But note that CPU built-in accelerator on Intel Processors is part of the RAPL UNCORE power.
I am sorry to say that the "uncore" type RAPL only belongs to client CPUs, such as Intel Core, Atom, etc. But for server CPUs such as Xeon SP, there is no "uncore" definition under RAPL sysfs... instead, there are only pkg+dram

05/08 meeting:

Intel investigates accelerators (in-die and out-of-die). How to measure in-die accelerator: this can be done in kepler through RAPL pkg domain. The usage of the accelerator is cpu usage based. Out-of-die is measured separately.

IBM Z also has a set of CPU built-in accelerator  like zip/crypto etc purpose so we can joint build the model with x86 world

May 30 meeting: using perf interface to collect stats from accelerator AMX, this will be made generic so that Kepler can use it for other type of accelerators. The overhead of continuous monitoring needs to be addressed, either by trading off overhead vs accuracy, or based on test results. Sampling can help reduce the monitoring the overhead.


track etcd database version ID

This tracks the data model version ID in addition to the hash value in the etcd database. The model version ID is a monotonically increasing integer, and can be used to determine if an upgrade has been upgrade or not, regardless of whether the current software is still tracking the upgrade associated with it.
The intent here is to be able to tag objects with a db model version ID to guarantee that upgrade functions are idempotent. So far upgrade functions guarantee idempotency by looking at the current data in the object. This does not work for new non-nil fields that need to be set on upgrade, but may be cleared by the user later on. Consider the "CrmOnEdge" bool. On upgrade we need to set it "true" because all existing cloudlets use Crms on Edge. However, new cloudlets may not use crm on edge, so may set it to false. Therefore there is no way to tell from a false CrmOnEdge value if the cloudlet needs to be upgraded or not. As an aside, the CrmOnEdge upgrade function guarantees idempotency by looking at the "ObjId" field which does not have the same issue, and is part of the same upgrade.
The intent here then is to add a "dbModelVersion int32" field to an object, like Cloudlet, and compare that to the ID for the running upgrade function to tell if the object needs to be upgraded or not. This will be used for the Zone upgrade feature which adds a string field to the Cloudlet.

Thanks Lev 👍


Add initial version of Python bindings

A first implementation of Python bindings for the rust library. I started out with objects that seemed to make most sense to expose, but more can be added. Additionally, for some objects, more fields could be exposed in the future.
I thought it would be good to keep all code related to the Python bindings separate from the main rust package itself. That does mean that all object have to be wrapped with a "Python" version, resulting in a lot of boilerplate code.
The Python package is built automatically on GitHub Actions for each PR or push to release. On a tag (or GitHub release), the package is uploaded to PyPI. For this, trusted publishing should still be set up.
In terms of version management, I thought it would be good to keep in sync with the rust library.
Documentation is built automatically with Sphinx on readthedocs.org. A version is already online from my branch: https://rustyms.readthedocs.io.

I am sorry I did a bit of restructuring again. I did not like the idea of having to set up 6 crates whenever I want to publish a new release. The API will not have changed though except for two minor things:

LinearPeptide::generate_theoretical_fragments now does not take the peptide_index anymore
Species::from_str now returns a full error with names that might be what you intended and it takes any captialisation
All in all this 'should' not result in any major work for you in this PR.

Current to do's in the module:

Element.most_abundant_mass(): Waiting on upstream fix, or could be removed
MolecularFormula: Can this be constructed from a string?
MolecularFormula.with_global_isotope_modifications(): Could not get the mapping to Python types working so far.
MolecularFormula.most_abundant_mass(): Idem as above
MolecularCharge: Are you planning to make this public? If not, I cannot expose it in Python. Currently, peptide charge can be passed through, but not the adduct information (I think).
FragmentType: Not yet mapped to Python, so returned as strings.
RawSpectrum: (re)implement constructor

Here are come comments on your todo's

The most_abundant_mass is indeed broken upstream, for now keep this private in the Python api. I will have to dive into isotope maths to fix this at some point, which is something I want to do but does not have a very high priority.
Molecular formulas can be constructed from strings, using the ProForma notation and the Unimod notation. I am not sure though if these are public members. I will check and add them if necessary.
The with_global_isotope it looks like a vector should be able to work but I have never worked with PyO3 before.
MolecularCharge should have been public, oops.

I added a charge method to MultiFormula (the trait), publicised MolecularCharge, and added two methods fro constructing a MolecularFormula from string (Pro Forma and PSI-MOD, Unimod has a more advanced systems that also incorporates glycans). Additionally for the spectrum generation, I would go and use RawSpectrum::default() followed by RawSpectrum::extend() to create a new spectrum.
So pretty much done! Very nice work, is there something you still want to implement in the PR or do you want it merged as soon as possible?

All remaining things I wanted to implement are done, so I'm good for merging.
For the publishing of the Python package, the workflow should run on any tagged release. Not sure how you handle releases currently?

Also note that to implement bindings to MolecularCharge, I had to make charge_carriers a public field. This can be reverted if you do not want this.

Release right now are handled by locally running cargo publish and then once that is done tagging the correct commit in github. There should be some ways to automate this and make it do this stuff automatically. But those things are a bit of pain to set up and I did not bother yet. Also I have been fighting with many of the releases as the included databases can go over the 10 MB max file upload for crates.io very easily.

Ready to merge.
If everything goes well, the Python package should be build on GitHub Actions and pushed to PyPI whenever a tag is pushed to GitHub.

That sounds splendid. Is there any more setup to be done from my side? (Any secrets I should add or something?)

We can try to push out 0.8.2 when everything has merged, this would contain the small additions I made for this PR and could be a very good test case to see if everything is running.


Interface 'Illuminate\Translation\LoaderInterface' not found

When migrating from laravel 5.4 to 5.5 I got this error. Base from their spec doc it says
"The Illuminate\Translation\LoaderInterface interface has been moved to  Illuminate\Contracts\Translation\Loader."

Because where is no support for Laravel 5.5 yet

@xakzona  when can we expect for the support of Laravel 5.5?

Support has been added on branch 2.3


Update PR links for the tasks

@developer-diganta I have updated some PR links for the GSoC tasks. Please have a look :smile:


Fix creating empty files at log closing

There is a possibility that empty log files will be created at the closing log fabric.
The fix allows to suppress such empty log files creating.

@Alessar Could you please change this PR to the beta branch?


Azure DevOps Deployment Error Encountered

I tried to follow the Azure DevOps deployment doc (https://github.com/AzureIoTGBB/iot-edge-offline-dashboarding/blob/master/documentation/deployment-devops.md) but the Azure Pipeline fails. I am attaching few screenshots related to the failure.
Screenshot 1:

Screenshot 2:

There's a couple common things this could be.  Usually this is some kind of failing in the build step, specifically on the "Publish" IoT Edge Step.  Either the credential isn't getting added to the final deployment manifest, or the containers aren't successfully pushing to azure.  Would you be able to post the "Build" step logs as well as the final config/deployment.amd64/json (obviously strip the credentials, but let me know if they are present and correct).

@jlorich Thank you for the quick response. Please see attached for my logs (logs_208.zip) and deployment.amd64.json (sanitize-deployment.amd64.json.txt). For reference, I was able to download the deployment.amd64.json and manually create the IoT Edge deployment through Az CLI.
az iot edge deployment create -d {deployment-name} -n {iot-hub-name} --content deployment.amd64.json --target-condition "tags.dashboard=true" --priority 10

However, the Azure pipeline still fails for some reason.


error when run docker compose up ??

I'm beginner use command with this work flow
pip install cookiecutter
cookiecutter https://github.com/tiangolo/full-stack-fastapi-postgresql

# cd to project and run with docker compose up
docker compose up

error-docker-compose

As I am also working through this/experimenting - changing the URLs in:
celeryworker.dockerfile
backend.dockerfile
from
https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py
to
https://install.python-poetry.org/
Should work.

In backend/app/backend.dockerfile and celeryworker.dockerfile change poetry installation section to:
RUN curl -sSL https://install.python-poetry.org | python3 -  && \     cd /usr/local/bin && \     ln -s /root/.local/bin/poetry && \     poetry config virtualenvs.create false

As I am also working through this/experimenting - changing the URLs in: celeryworker.dockerfile backend.dockerfile
from https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py to https://install.python-poetry.org/
Should work.

This did the job. More precisely, your end result should be:
# Install Poetry
RUN curl -sSL https://install.python-poetry.org/ | POETRY_HOME=/opt/poetry python && \
    cd /usr/local/bin && \
    ln -s /opt/poetry/bin/poetry && \
    poetry config virtualenvs.create false```

See PRs https://github.com/tiangolo/full-stack-fastapi-postgresql/pull/524 and https://github.com/tiangolo/full-stack-fastapi-postgresql/pull/529 for the corrected files if you want to take a look.

Thanks! This was solved in https://github.com/tiangolo/full-stack-fastapi-postgresql/pull/480 🎉
And thanks @willingc! 🍨


minor edits and link corrections

these are all but one of the edits -- the next/last one is to change a link target from "acom..." to "azure.microsoft.com..."

#sign-off with new images


add factor country to the variants in lineage table

@rgudimel your request to take into account the country for the table of variants in a lineage.


infer does not find any Java issues with OpenJDK-11 (jdk-11.0.1)

I'm trying a simple Java case from https://fbinfer.com/docs/hello-world.html#hello-world-java.  infer finds the expected NULL_DEREFERENCE issue with the JDK being Java10 (or 8), but as soon as I switch to JDK11, I'm getting "No issues found."  Is it a known problem with jdk11?  I can imagine that the internal intermediate representation has changed with Java11.

I have the same problem trying to execute infer over the java examples

Hi,
If you get the latest version of Infer, you should be able to get Infer to report on the Java examples with Java 11. I have added support for it in https://github.com/facebook/infer/commit/ce190547.

The new version of infer does support java 11, but the current binary release has been built against java 8 so for now you'll need to build infer from source to get java 11 support.

Closing since this is fixed in master.


Pushing changes to new branch on origin.

Created a pester case for New-PlatyPsCab, and it's not working. Looking to review with @vors

This change is

Review status: 0 of 3 files reviewed at latest revision, 3 unresolved discussions, some commit checks broke.

test/Pester/PlatyPs.Tests.ps1, line 15 [r3] (raw file):
You don't use this variable anywhere. This call also fails the test in the CI:
https://ci.appveyor.com/project/PowerShell/markdown-maml/build/<IP_ADDRESS>#L249

test/Pester/PlatyPs.Tests.ps1, line 31 [r3] (raw file):
You don't need this two lines, do you?

test/Pester/PlatyPs.Tests.ps1, line 32 [r3] (raw file):
Add new line at the end of the file to make Git happy

Comments from Reviewable


Make target browsers configurable

Proof of concept for https://github.com/facebookincubator/create-react-app/issues/892.
Doesn't work with ejecting yet.
Both autoprefixer and Babel are driven by a config like this in package.json:
  "browsers": {
    "development": [
      "last 1 chrome version"
    ],
    "production": [
      ">1%",
      "last 4 versions",
      "Firefox ESR",
      "not ie < 9"
    ]
  }

Currently I throw if it doesn't exist since I could generate a reasonable config by default (e.g. "last two versions" for development) in new projects and ask old projects to include it during migration. In a way this would be "configuration" but actually sensible one. I don't have a strong opinion here though.

TODO: this likely requires switching to Babili. https://github.com/babel/babili
Because otherwise targeting too modern browsers in prod would crash Uglify.

Similarly had an issue with webpack 1 since it's using an older version of acorn for async - forgot about that (https://github.com/babel/babel-preset-env/issues/66)

Note, that I am planning to add package.json support to Browserslist in next month. But I will use browserslist key.

My only concern with browserslist is that we still need a way to separate development from production configuration. If it will support this, I’m happy to use it.

@hzoo
Alternatively we could add uglify and webpack targets to document their minimally supported environments.

Yeah totally true, didn't even think of that! Could just alias it to node 0.10/0.12 although diff for webpack 2

Another todo: make sure builds are reproducible
https://twitter.com/brian_d_vaughn/status/808143454739959808

development/production environment are planned too with same ETA 😊

Sounds great. This is a bit of a longer term project so I’m in no rush. It’ll take a while to work out most issues and get this to a stable state.

Please enlighten me! Why different browsers in development and production? Is it because of easier-to-read compiled code? What about when I need to fix those nasty IE 11 or Safari 9 bugs (which require certain vendor prefixes, for instance)?

@lydell I asked for feedback in Autoprefixer twitter. Maybe some user have good use cases for environments (maybe not only development/production).

I think it's up to developers to decide for themselves. Personally I'd use a lighter setting in development for better compiled output and errors, especially in code heavy with async/await or generators. But I’m the kind of person to test features with production settings before merging. It’s not safe if you don’t do this kind of testing so I’d make them match by default and document it somewhere that it is possible to loosen it up if you know what you’re doing.

It would be nice if this config also loaded required core-js poly and ideally inlined them via babel-runtime

It would be nice if this config also loaded required core-js poly and ideally inlined them via babel-runtime

👍 Yeah, that's the plan.

@gaearon browserslist master got env and package.json support. @hzoo do you need node queries in package.json on this step? Or we could release new Browserslist?

Browserslist 1.5 was released with production/development and package.json support.
@hzoo I really need your feedback. Please do not ignore my messages :).

Autoprefixer 6.6 is out with package.json support https://github.com/postcss/autoprefixer/releases/tag/6.6.0

Thoughts on this -> https://philipwalton.com/articles/deploying-es2015-code-in-production-today/
cc @gaearon @Timer


Update linkedin-learning.md

Removed double use of free access

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@eoverfield and @JohanFreelancer9 thanks for your help on this article! Unfortunately, this article has since been retired.


Create a cross chain test harness

It would be nice if we could write tests that actually function cross-chain. We should write a test that can connect to two chain instances and perform various cross-chain tests.

Moved to https://github.com/interwallet/Bridge-Wallet/issues/3


Update Readme

The readme is also used for grafana.net and for that you don't really need the clone instructions. But more importantly it would be great if you could also add more info in the readme about how this panel works, what data sources it supports etc.
A changelog would also be good, mention that this panel is very early alpha state etc. The black y/x axis in the screenshot on grafana.net makes it looks like it's only a test and not ready for other to test so might be good to mention if this is readme

I'm sorry. I should check carefully, and prepare instruction for user.
I'll update the README, and add CHANGELOG in this weekend.

Awesome! Sorry, it was our mistake to publish it on grafana.net before it was ready :)

@torkelo I update README.
The README becomes very brief...
Because I omit the explanation of Epoch library options, just refer the official document.
I add CHANGELOG.md too.
I worry about these are enough, would you check these documents?

better, changelog could be in readme.md so it will show on Grafana.net


Add tests to increase coverage

I'd appreciate some help from contributors!

I'd be interested in helping write some tests. I'm going to go ahead and take a stab at writing some and will submit PR for review once I feel I have some good progress.

Alright so after playing around with this for a bit. I wanted to try and understand more about what exactly we should be using and doing for writing these tests.
I initially tried using React's test renderer and Jest's snapshots to test the GoogleMap component but ran into this issue https://github.com/facebook/jest/issues/1353. The recommended solve there was trying a number of different things but ultimately I had to install enzyme, enzyme-to-json, and react-addons-test-utils.
I was successfully then able to go ahead and create the snapshots and do a basic render test with the GoogleMap.
import React from "react";
import { withGoogleMap, GoogleMap } from "../index";
import { shallow } from 'enzyme';
import { shallowToJson } from 'enzyme-to-json';
import _ from "lodash";

const SimpleGoogleMap = withGoogleMap(props => (
  <GoogleMap
    ref={props.onMapLoad}
    defaultZoom={3}
    defaultCenter={{ lat: -25.363882, lng: 131.044922 }}
    onClick={props.onMapClick}
  />
));

describe(`GoogleMap`, () => {
  it(`should render`, () => {
    const googleMap = shallow(
      <SimpleGoogleMap
        containerElement={
          <div style={{ height: `100%` }} />
        }
        mapElement={
          <div style={{ height: `100%` }} />
        }
        onMapLoad={_.noop}
        onMapClick={_.noop}
      />
    );
    let tree = shallowToJson(googleMap);
    expect(tree).toMatchSnapshot();
  });
});

I'm hoping some of the other contributors can chime in here and let me know if this is the direction I should be moving or if this is not really the test setup you are looking for.

When you were running tests, what command did you use? It took me forever to find the correct one (you can see it in the .travis.yml file), but the command you should be using is npm test -- --coverage. This will not only run your tests but also give you a coverage report.
Also on a side note, I like how you went about the tests. I looked at testing a little bit ago, and as I was writing them I had to do a whole lot of import statements as well, which was weird because the test files currently have no import statements in them (that reference node_modules). Because of that it's hard to know what the eventual vision is of Tom & Co is, but you have a very nice start.

Yea so I've been running npm test for now without the code coverage because I was just interested in getting the tests up and running. I feel like once I get some good direction from Tom & Co about what I have above and if I am going down the right path for how they want the project tests structured then I will begin to generate the test coverage report. Also, @danielehrlich  thank you! I'm glad I'm going about it the right away (or so we think). 😄

Anyone interested?

I'll be picking this one up @tomchentw I'll start a feature branch and progressively start increasing coverage.

@oshalygin great!

It seems that the render function SimpleGoogleMap wasn't called, I debug with console.log and it didn't move there.