How to set correct phonema for Chinese?

It seems that wei4 and wei&#2e7 are both invalid ph value in the phoneme tag with alphabet=ipa for "为". Can any provide an example for any chinese character? Thanks a lot.

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

ID: aae85d9f-b4f4-fc4d-cb0f-0fc4a1d4e7c9
Version Independent ID: dbd55a99-dbde-a3f3-47a5-ded8309e2748
Content: Speech Synthesis Markup Language (SSML) - Speech Services - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/speech-synthesis-markup.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @erhopf
Microsoft Alias: erhopf

@xulxqqqq Thanks for the feedback. We are investigating into the issue and will update you shortly.

@Yueying-Liu Could you please help to provide examples for chinese characters?

Here is a sample of setting phoneme for Chinese.
**为**了保持良好的公众秩序，请遵守此项规定。

@Yueying-Liu Thanks for the update. @xulxqqqq Could you please check if this helps?

Solved. Thank you.


Creating the project The installation has failed.

Inimist-one@Inimist-one-PC MINGW64 ~/Desktop
$ djangocms myproject
Creating the project
Please wait while I install dependencies
Dependencies installed
Creating the project
The installation has failed.

Check documentation at https://djangocms-installer.readthedocs.io

Traceback (most recent call last):
File "c:\python27\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "c:\python27\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\python27\Scripts\djangocms.exe_main.py", line 9, in
File "c:\python27\lib\site-packages\djangocms_installer\main.py", line 41, in execute
django.setup_database(config_data)
File "c:\python27\lib\site-packages\djangocms_installer\django_init.py", line 407, in setup_database
command, env=env, stderr=subprocess.STDOUT
File "c:\python27\lib\subprocess.py", line 376, in check_output
**kwargs).stdout
File "c:\python27\lib\subprocess.py", line 468, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['c:\python27\python.exe', '-W', 'ignore', 'manage.py', 'migrate']' returned non-zero exit status 1.

Python version:  3.7.0
Django version: (1, 11, 0, 'final', 1)
django CMS version: 3.5.2

I recently had the same problem. With the --verbose mode I found out that this is a syntax error in the lib\site-packages\django\contrib\admin\widgets.py file.
See solution at github #28814 https://github.com/django/django/commit/931c60c5216bd71bc11f489e00e063331cf21f40
As far as I can see you just have to remove the comma at the end of the brackets in line 165 of widgets.py:
related_url += '?' + '&'.join('%s=%s' % (k, v) for k, v in params.items(),)  <-remove comma
related_url += '?' + '&'.join('%s=%s' % (k, v) for k, v in params.items())

nevertheless I was able to install and work with django-cms 3.5.2, django1.11 and python 3.7 after removing the comma.
additionally I also installed django-cms 3.5.2, django1.11 with python 3.6.7 and it works perfectly without removing the comma so I guess we should follow the official instructions.
thx for the advice jrief

@MohitPrakashSharma please close this issue


Ore Refiner compatibility

Lead ore from Embers Rekindled can't be refined in the machine

Check 2 things:

Is the ore correctly oredicted as oreLead?
Is there Lead Dust in your instance?

That is what the machines are looking for. If the ore isn't oredicted properly then it's a Embers issue.
You can test that everything is working by adding something that adds Lead Dust like Base Metals or Thermal Foundation, making an new world and checking if the recipe is there, then removing that mod.

Ok will do as soon as I can, thanks for responding so fast!
Il gio 18 lug 2019, 09:38 Lykrast<EMAIL_ADDRESS>ha scritto:

Check 2 things:

Is the ore correctly oredicted as oreLead?
Is there Lead Dust in your instance?

That is what the machines are looking for. If the ore isn't oredicted
properly then it's a Embers issue.
You can test that everything is working by adding something that adds Lead
Dust like Base Metals or Thermal Foundation, making an new world and
checking if the recipe is there, then removing that mod.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Lykrast/ProdigyTech/issues/10?email_source=notifications&email_token=AGUZ3SYWNQY663DPLA3SUATQAAMWJA5CNFSM4IEVH2GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2HTORA#issuecomment-512702276,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGUZ3S7KCK5CUNN4ESRDAU3QAAMWJANCNFSM4IEVH2GA
.

Ok i made some test and turns out that the issue was what you suggested, so thanks for the help!


March Summary Post

Add donation information, Freeloaderz project, and Captive Audience

https://www.captiveaudienceptrt.com/
https://www.captiveaudienceptrt.com/support-ukraine-ocv


Error appeared using the latest commit

neovim version: 0.9.5
os: Manjaro Linux x86_64 6.1.71-1-MANJARO
The plugin worked well in the past, but something went wrong with the latest commit https://github.com/rockyzhang24/arctic.nvim/commit/7297a66075f9b27ba73d5ee4ba8a831cf5de34d8 on both the main and v2 branches.
" mini.vim
set runtimepath^=/home/hexh/Desktop/arctic.nvim/arctic.nvim
set runtimepath^=/home/hexh/Desktop/arctic.nvim/lush.nvim

colorscheme arctic

nvim --clean -u ./mini.vim ./mini.vim

IDK is it related to treesitter. I don't use treesitter.

Should fixed. Sorry for my carelessness.

@rockyzhang24 Ty for fixing. It helps for me and issue closed.


Warning on loading extension on chrome

A cookie associated with a cross-site resource at http://www.toonin.ml/ was set without the SameSite attribute. A future release of Chrome will only deliver cookies with cross-site requests if they are set with SameSite=None and Secure. You can review cookies in developer tools under Application>Storage>Cookies and see more details at https://www.chromestatus.com/feature/5088147346030592 and https://www.chromestatus.com/feature/5633521622188032.

Archiving this issue since we're moving away from the extension.


Make OkHttpClient and Call final.

This will break mocks. We have a much simpler solution now, with
interceptors. Instead of mocking the HTTP client, use interceptors
to change its behavior.
Fix a bug where interceptor lists aren't independent after cloning
the OkHttpClient.

checkstyle is quite unhappy. otherwise LGTM especially substituting mocks with an alternative.

@adriancole checkstyle is not only one, javac complains about unchecked or unsafe operations in Util.java too.

LGTM. Grab WebSocketCall too?


Question: Can I use Ollama for function calling through magentic

Hey folks!
First, would like to express my appreciation for maintaining and developing this cool tool!
I have a quick question when using function calling through Ollama, does it currently support function calling? I'm using mistral model.
Thank you!

Hi @DevAseel , thanks for the kind words! There's an open issue here for using structured outputs and function calls with ollama https://github.com/jackmpcollins/magentic/issues/194  so please follow that.

Resolution from the other issue: https://github.com/jackmpcollins/magentic/issues/194#issuecomment-2097471824

See also: https://github.com/jackmpcollins/magentic/issues/207#issuecomment-2507359293


Seed breaks when trying to add roles to admin

Updated seed to create dummy admin before adding roles to activeadmin.roles

PR submitted


When I only turn on network event tracking, process creation / destruction events will still be captured

When I only turn on network event tracking, process creation / destruction events will still be captured.
tm.SetKernelEventTypes({ KernelEventTypes::Network});

Is this a bug? The code is based on the ProcMonC project.

Yes, process events are always captured. This is by design.

Well, thank you. The work has brought me a lot of help. Thank you for your answer and your contribution to the community.


License?

What license covers this work?

Using Apache License 2 as storm does. License file added.


Disable automatic breadcrumb logging except in production

Meant to address #156
This change defaults the automatic breadcrumbs for console logging to be disabled unless the release stage is set to production.
Unfortunately this means that the breadcrumb will be disabled if you don't end up setting the release stage at all. I think this is preferable though to the alternative of having the console log line numbers messed up and not knowing why.

This is going to require more thought. There is a problem with the setup of console log tracking the way it is currently architected. If it is disabled during initial setup, it cannot be re-enabled after the fact.
We might need to change how the breadcrumb settings work so that we have setter functions so that when they get set we can create the wrappers for native browser functions.

Abandoning in favor of #199


Update Numpy

Update numpy==1.24.3 >> 2.0.0
wetterdienst 0.88.0 depends on numpy<1.27 and >=1.22
pandas 2.2.2 depends on numpy>=1.22.4

Current version of wetterdienst do not support latest numpy. Try to update wetterdienst first.
Solution
update Wetterdienst
update Numpy
Workflow checklist

[ ] I am aware of the workflow in CONTRIBUTING.md
[ ] I have created a branch for my feature

The latest version of wetterdienst is 0.95.1 while current using version is 0.88.0
After testing in jupyter notebook, the wetterdienst== 0.95.1 is working well but the API parameters have been changed a little:

previous parameter of datasets in wetterdienst==0.88.0 is

"TEMPERATURE_AIR_MEAN_200"

the available parameters in new version of wetterdienst==0.95.1 are

TEMPERATURE_AIR_2M
TEMPERATURE_AIR_MEAN_0_05M
TEMPERATURE_AIR_MEAN_2M
TEMPERATURE_AIR_MEAN_2M_LAST_24H

Which one should i use? Please give me some advice. Thanks.
@c0nb4

TEMPERATURE_AIR_2M would be my suggestion.


missing ingress-nginx on this repository

Is there an existing issue for this?

[X] I have searched the existing issues

What did you expect to happen?
I am using minikube (1.33.0) to enable nginx-ingress but it is failing due to nginx ingress is not available (registry.k8s.io/ingress-nginx/)
it looks like nginx ingress does not exist on this repository anymore
Debugging Information
Failed to pull image "registry.k8s.io/ingress-nginx/kube-webhook-certgen:v1.4.1@sha256:36d05b4077fb8e3d13663702fa337f156585ba8667cbd949c03a8e8ea6fa4366": Error response from daemon: Get "https://registry.k8s.io/v2/": tls: failed to verify certificate: x509: certificate signed by unknown authority
Anything else?
N/A
Code of Conduct

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

https://explore.ggcr.dev/?repo=registry.k8s.io/ingress-nginx/kube-webhook-certgen ( repo is correct )
https://explore.ggcr.dev/?image=registry.k8s.io%2Fingress-nginx%2Fkube-webhook-certgen:v1.4.1 ( version exists )
https://explore.ggcr.dev/?image=registry.k8s.io/ingress-nginx/kube-webhook-certgen@sha256:36d05b4077fb8e3d13663702fa337f124675ba8667cbd949c03a8e8ea6fa4366&mt=application%2Fvnd.docker.distribution.manifest.list.v2%2Bjson&size=1272 ( SHA is correct )
docker pull registry.k8s.io/ingress-nginx/kube-webhook-certgen:v1.4.1 works

Problem is probably in your environment. Please see the debugging tips:
https://github.com/kubernetes/registry.k8s.io/blob/main/docs/debugging.md

Get "https://registry.k8s.io/v2/": tls: failed to verify certificate: x509: certificate signed by unknown authority

This is an issue with your local environment, I would guess you have a MITM proxy and minikube needs to have the certs from your host or be allowed to bypass the proxy.
Our certs are definitely signed by a standard known certificate root.


FileNotFoundError: [WinError 2]

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.
Check the diagnose documentation for common issues before posting! We may close your issue if it is very similar to one of them. Please be considerate, or be on your way.
Make sure to mention your debugging experience if the documented solution failed.
Issue description
The pipenv shell command doesn't work on my system
Expected result
The command should create a virtual env.
Actual result
Launching subshell in virtual environment…
Traceback (most recent call last):
File "c:\users\zhang\appdata\local\programs\python\python36\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\Zhang\AppData\Local\Programs\Python\Python36\Scripts\pipenv.exe_main.py", line 9, in
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\core.py", line 829, in call
return self.main(*args, **kwargs)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\core.py", line 782, in main
rv = self.invoke(ctx)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\core.py", line 1259, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\core.py", line 610, in invoke
return callback(*args, **kwargs)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\decorators.py", line 73, in new_func
return ctx.invoke(f, obj, *args, **kwargs)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\vendor\click\core.py", line 610, in invoke
return callback(*args, **kwargs)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\cli\command.py", line 433, in shell
pypi_mirror=state.pypi_mirror,
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\core.py", line 2390, in do_shell
shell.fork(*fork_args)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\shells.py", line 97, in fork
_handover(self.cmd, self.args + list(args))
File "c:\users\zhang\appdata\local\programs\python\python36\lib\site-packages\pipenv\shells.py", line 62, in _handover
sys.exit(subprocess.call(args, shell=True, universal_newlines=True))
File "c:\users\zhang\appdata\local\programs\python\python36\lib\subprocess.py", line 287, in call
with Popen(*popenargs, **kwargs) as p:
File "c:\users\zhang\appdata\local\programs\python\python36\lib\subprocess.py", line 729, in init
restore_signals, start_new_session)
File "c:\users\zhang\appdata\local\programs\python\python36\lib\subprocess.py", line 1017, in _execute_child
startupinfo)
FileNotFoundError: [WinError 2]
Steps to replicate
OS: Windows.10.0.17763

Open the terminal of Pycharm
run pipenv install
run pipenv shell

P.S. I've uninstalled and installed different versions of Python, and reinstalled pipenv everytime. So it shouldn't be python's interpreter problem.

@zzpsy Is this still an issue on pipenv==2022.8.19?


Disable fixed field and equivalent type spec code from DISABLE_JIT builds

Put equivalent type spec and property guard code under ENABLE_NATIVE_CODEGEN
Also fixed ability to disable SUPPORT_FIXED_FIELDS_ON_PATH_TYPES
Disable fixed field for when JIT is disabled.
Reduced binary size of the lite build by ~41K on x86 and ~65K on x64.

Rubber-stamp LGTM if CI is green


fix(transparent) avoid redefinition of kernel ipv6 definitions

The following errors occur when building on Debian 7:
docker: In file included from src/os/unix/ngx_linux_config.h:121:0,
    docker:                  from src/core/ngx_config.h:26,
    docker:                  from src/core/ngx_log.c:8:
    docker: /usr/include/linux/in6.h:30:8: error: redefinition of 'struct in6_addr'
    docker: In file included from src/os/unix/ngx_linux_config.h:45:0,
    docker:                  from src/core/ngx_config.h:26,
    docker:                  from src/core/ngx_log.c:8:
    docker: /usr/include/netinet/in.h:198:8: note: originally defined here
    docker: In file included from src/os/unix/ngx_linux_config.h:121:0,
    docker:                  from src/core/ngx_config.h:26,
    docker:                  from src/core/ngx_log.c:8:
    docker: /usr/include/linux/in6.h:46:8: error: redefinition of 'struct sockaddr_in6'
    docker: In file included from src/os/unix/ngx_linux_config.h:44:0,
    docker:                  from src/core/ngx_config.h:26,
    docker:                  from src/core/ngx_log.c:8:
    docker: /usr/include/x86_64-linux-gnu/sys/socket.h:92:17: note: originally defined here
    docker: In file included from src/os/unix/ngx_linux_config.h:121:0,
    docker:                  from src/core/ngx_config.h:26,

We must include linux/in6.h before netinet/in.h, since the latter
maintains definition guards around the IPv6 kernel structures:
https://code.woboq.org/userspace/glibc/inet/netinet/in.h.html#209

Do not merge for now.


use the Read + Seek trait for the GDS reader. allows parsing the GDS …

…from borrowed data

Love it! Thanks, and glad you're finding this helpful.


Lots of small changes

Fixes WADL infinite loop issue when using "OPTIONS" http verb on rest
services, due to self-referencing sub-resources.
Rename to remove "api" sub-package.
Move "/rest/xml" API to "/rest"
Separate view servlet from rest servlet
Remove unnecessary constraint mapping to prevent "TRACE" http verb
(already disabled in ResourceConfig and DefaultServlet)
Remove broken /rest/master/tservers_info path, which makes Jackson think
thrift files are POJOs which need serializing
Remove redundant synchronizing in monitor
Rename some packages to make them consistent with their root resource
name (e.g. /rest/zk -> org.apache.accumulo.monitor.rest.zk)

Thanks for the help. Merging in.


Multiple forms

Hello,
I am trying to get multiple forms to use different view templates. However because each of my form inherits from MailForm::Base which in turn calls MailForm::Notifier.contact() when the deliver! meh, when rails comes to guess the template, it uses the contact template.
I am probably missing something, however because of the abstraction, I cannot see a way to use multiple view templates for the emails.
Thank you in advance for any help you can provide.

I found the solution, by setting a header within the form. In case anyone else has this issue, by setting a header hash as shown here: https://github.com/rails/rails/blob/0fe76197d2622674e1796a9a000995a7a1f6622b/actionmailer/lib/action_mailer/base.rb#L914
`class MyForm < MailForm::Base
def headers
{
subject: "My Subject",
template_name: "my_template", # Unless otherwise specified, this will be app/views/mail_form
}
end
end`


Prevent deletion of part which is used in an assembly

Adds protection against deleting a part which is a component in an assembly.
@matmair @wolflu05 thoughts here? Should this behaviour be behind a configurable setting?

Sorry that I cannot add some production feedback here, I have not really used assemblies yet. But just from thinking about it,  if we delete an part, the part for the assembly is missing, that would mean, we either delete the assembly too, or remove the part from the assembly or deactivate the part somehow that its not shown in search anymore.

Sorry that I cannot add some production feedback here, I have not really used assemblies yet. But just from thinking about it, if we delete an part, the part for the assembly is missing, that would mean, we either delete the assembly too, or remove the part from the assembly or deactivate the part somehow that its not shown in search anymore.

I think that this is not quite right. The change I am implementing here prevents the sub_part from a BOM being deleted - which would in practice mean that a BOM item disappears (without warning) from the BOM of an existing assembly. This change prevents the part from being deleted while it is still in the BOM for a different assembly.


New odoc rules

Recreating after the accidental close of the previous PR.
The odoc rules, as implemented in this PR, work with the git version of odoc and follow the guidelines listed here.
Currently, for a given library, .mld files are found in the same directory as normal source files (i.e. in the same directory as the jbuild).
Things that need to happen before merging this

[x] odoc should be released on opam
[ ] we need a way to put .mld files in an arbitrary directory and tell jbuilder where to find them. The way I currently have in mind is an addition to the jbuild type of libraries to allow writing:(library
  ((name foo)
    ...
    (odoc <something>)
    ))

where something should probably follow this syntax
[ ] we need to install all the .mld files in $DOCDIR/$PKG/odoc-pages (at least that's what I understand from this comment, but @dbuenzli will correct me if I'm wrong).

@dbuenzli will correct me if I'm wrong

According to https://github.com/dbuenzli/odig/issues/31 and https://github.com/dbuenzli/odig/issues/18 it's rather $DOCDIR/$PKG/odoc-pages.

Currently, for a given library, .mld files are found in the same directory as normal source files (i.e. in the same directory as the jbuild).

Do you then namespace the .mld files ?

Do you then namespace the .mld files ?

Yes, in the sense that every .odoc file contain information about which package it belongs too (which you specify when calling odoc compile).
Unfortunately there is currently no way to tell odoc (when it resolves references) "I want the page Foo of package P", but we will definitely add that (rather soon I hope).
In any case, this is purely an odoc issue and doesn't impact jbuilder (or odig) or the work being discussed here.

In any case, this is purely an odoc issue and doesn't impact jbuilder (or odig) or the work being discussed here.

I think it does. If two libraries define p.mld those are going to overwrite when you copy them to odoc-pages. An easy way to get out would be to support only this:

we need a way to put .mld files in an arbitrary directory and tell jbuilder where to find them.

and only once per package.

Curious as to why we need to allow jbuilder to specify .mld files in different dirs? The way we solved a similar problem for source files is using copy_files. Why can't we rely on copying to work for this as well? Not suggesting that this is the way to go, but would like to understand the use case first.

I think it does. If two libraries define p.mld those are going to overwrite when you copy them to odoc-pages

Right, because library and package are two distinct things.
From what I've seen, currently jbuilder will use the "public library name" as package name when invoking odoc; I assumed it would also use that as package name when installing files but I'm probably wrong there.

Curious as to why we need to allow jbuilder to specify .mld files in different dirs? The way we solved a similar problem for source files is using copy_files. Why can't we rely on copying to work for this as well? Not suggesting that this is the way to go, but would like to understand the use case first.

I've personally always found that distasteful.
I understand that you want to discourage users spreading their sources in several directories if this isn't reflected later in the language by proper namespacing. Here however I don't think it is bad if users put their .mld files in a doc/ or manual/ subdirectory.
It is not too important but I think it'd be nicer.

Ping

Is this a ping to me? I'm fine with you allowing mld files in other directories. Though I'd like to hear a second opinion from another maintainer (like @bobot for example).
Seems like this PR is still WIP though? Or is there something you wanted us to work on?
Btw, this needs a rebase on top of master.

For both of you actually.
I don't feel like I can properly answer @dbuenzli's concerns, I think a discussion needs to happen. And that point needs to get resolved before progress can be made on this feature. So let's also ping @lpw25 and @aantron .
As for the second question yes, the ping was for "jbuilder's dev" in the general sense yes.
We could decide that the second item in the todo list (i.e. putting .mld in arbitrary directories) is not that important and delay the decision. But it's probably better to just make a decision now, or at least to make sure we're not going to force people to change their codebase in a few months because we've changed our mind (i.e. make sure that in the future we're still going to support the choice we make in this PR).

In that case same directory is the safest.

So I gather you will then namespace the mld files the jbuilder does with ml files, right ?
My impression is that manuals are going to be per package rather than per library this is why I would rather advocate a doc directory (which would also eschew the namespacing problem).

In that case same directory is the safest.

Sounds good to me!

My impression is that manuals are going to be per package rather than per library this is why I would rather advocate a doc directory (which would also eschew the namespacing problem).

The can be added after the fact then, with a (documentation ...) stanza (i.e. it'd be its own thing, rather than part of the (library ...) stanza).
It would also probably interact nicely with https://github.com/ocaml-doc/odoc/issues/94
I do think that pages inside a library, i.e. not part of a "standalone" manual, still make sense though. Don't they?
And I still don't know where these should be installed :/

I think the discussion is getting a little bit confusing here.
I don't see how https://github.com/ocaml-doc/odoc/issues/94 would interact nicely with my proposal this is a global namespace.  I'm not sure exactly what @bobot says with "currently two libraries can't be part of the same exact package". As far as documentation is generated by odoc as driven by odig this doesn't make sense: a package corresponds to an opam package and treated as such with the --pkg option of odoc.
Here are a few points:

AFAIK for now .mld processing in odoc is per package. Each .mld name has to be unique in the package like toplevel modules of libraries need to be.
Whether you want to have the .mld in the directory of libraries or in one or more separate directories, the problem of name clashes subsists both at generation time and at install time regardless of where the files are installed (since usually libraries are installed in a flat directory).
Given 1. and 2. if you centralize the .mlds for a given package to a single directory you solve the problem of name clashes.
Regarding where the .mld files need to be installed a proposal was made here.

I haven't been following this too closely, but I basically agree with @dbuenzli on this. SInce odoc manages documentation for opam packages rather than libraries it is packages that jbuilder should associate the .mld files with. I don't know the details, but it seems like libraries are associated with packages by just associating any (library ...) stanzas with the surrounding package. So I would assume that having a (documentation ...) stanza which associates the listed documentation files with the surrounding package would be most reasonable approach. Just as (I assume) jbuilder checks libraries in a package have unique names it should check that documentation in a package has a unique name.

it seems like libraries are associated with packages by just associating any (library ...) stanzas with the surrounding package.

I don't think that's quite right. I think the public name of the library has to be of the form package-name[.whatever].
Anyway, thanks for the precisions / clarifications.
It seems like have the .mld files live among the .ml(i) files is just a source of confusion, leading people to think that that the doc granularity is the library, not the package. (Note that this confusion is already present in the way jbuilder generate documentation before this PR, by pretending that libraries are packages)
I'll try to update this PR sometime in the next couple of weeks to clean up all of that.

It seems like have the .mld files live among the .ml(i) files is just a source of confusion

Just to clarify, I don't think it is the mixing of .mld files with .ml files that is the problem, so much as having the odoc construct appear within the library construct. I think that is what makes it seem like the documentation is associated with the library rather than the package.

There's a test failure with one of my recently introduced tests:
        cram alias test/blackbox-tests/runtest (exit 1)
(cd _build/default/test/blackbox-tests/test-cases/meta-gen && ../../cram.exe run.t)
------ /Users/rgrinberg/reps/jsc/jbuilder/test/blackbox-tests/test-cases/meta-gen/run.t
++++++ /Users/rgrinberg/reps/jsc/jbuilder/test/blackbox-tests/test-cases/meta-gen/run.t.corrected
File "/Users/rgrinberg/reps/jsc/jbuilder/test/blackbox-tests/test-cases/meta-gen/run.t", line 2, characters 0-1:
 |  $ $JBUILDER runtest --force -j1 --root .
-|  description = "contains \"quotes\""
-|  requires = "bytes"
-|  archive(byte) = "foobar.cma"
-|  archive(native) = "foobar.cmxa"
-|  plugin(byte) = "foobar.cma"
-|  plugin(native) = "foobar.cmxs"
-|  package "baz" (
-|    directory = "baz"
-|    description = "sub library with modes set to byte"
-|    requires = "bytes"
-|    archive(byte) = "foobar_baz.cma"
-|    archive(native) = "foobar_baz.cmxa"
-|    plugin(byte) = "foobar_baz.cma"
-|    plugin(native) = "foobar_baz.cmxs"
-|  )
-|  package "rewriter" (
-|    directory = "rewriter"
-|    description = "ppx rewriter"
-|    requires(ppx_driver) = "bytes foobar"
-|    archive(ppx_driver,byte) = "foobar_rewriter.cma"
-|    archive(ppx_driver,native) = "foobar_rewriter.cmxa"
-|    plugin(ppx_driver,byte) = "foobar_rewriter.cma"
-|    plugin(ppx_driver,native) = "foobar_rewriter.cmxs"
-|    # This is what jbuilder uses to find out the runtime dependencies of
-|    # a preprocessor
-|    ppx_runtime_deps = "bytes foobar.baz"
-|    # This line makes things transparent for people mixing preprocessors
-|    # and normal dependencies
-|    requires(-ppx_driver) = "bytes foobar.baz"
-|    ppx(-ppx_driver,-custom_ppx) = "./ppx.exe --as-ppx"
-|  )
+|  Multiple rules generated for _build/default/page-index.odoc
+|  [1]
make: *** [test] Error 1

I've seen an error like this pop up before as well in ppx_tools_versioned or omp - can't remember. You should be able to reproduce by rebasing and running $ make test.

I've added some real odoc tests now as well. So this PR should be rebased on top, and those be updated if necessary.

I've resolved the conflicts in this branch with master. @trefis please have a look.
The odoc test suite is still failing because it doesn't yet support generating an index.html for multiple libraries defined in the same directory. In this situation we have to be a bit more clever and make sure that the index.html is generated for all libraries. It shouldn't be too hard to work around this but I wonder if it's best to just make this unsupported. jbuilder really assumes that there's only library per directory.

cc @aantron as well on this issue

Implementation continues at #570


Keybinding use Shift+Command+E not working

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

VS Code Version:
Version: 1.60.0 (Universal)
Commit: e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff
Date: 2021-09-01T10:54:53.442Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 19.6.0
OS Version:
MacOS Catalina
10.15.7 (19H1217)

Steps to Reproduce:

open KeyboardShortcuts
click KeyBinding
press Shift+Command+E , not recogonize

(the shortcut in karabiner-elements viewer, third one is the shift+command+e)

Let's first try to find out what command executes on your machine, with your settings and your keybindings when pressing cmd+shift+E. Could you please try the following:

Run the command Developer: Toggle Keyboard Shortcuts Troubleshooting
This will activate logging of dispatched keyboard shortcuts and will open an output panel with the corresponding log file.
Press exactly the keybinding you desire.
check what VS Code detects and what command is invoked.

If you are having trouble understanding the output, please feel free to paste here and I can try to help.

Let's first try to find out what command executes on your machine, with your settings and your keybindings when pressing cmd+shift+E. Could you please try the following:

Run the command Developer: Toggle Keyboard Shortcuts Troubleshooting
This will activate logging of dispatched keyboard shortcuts and will open an output panel with the corresponding log file.
Press exactly the keybinding you desire.
check what VS Code detects and what command is invoked.

If you are having trouble understanding the output, please feel free to paste here and I can try to help.

[2021-09-08 14:37:47.000] [renderer3] [info] [KeybindingService]: / Received  keydown event - modifiers: [meta], code: MetaLeft, keyCode: 91, key: Meta
[2021-09-08 14:37:47.001] [renderer3] [info] [KeybindingService]: | Converted keydown event - modifiers: [meta], code: MetaLeft, keyCode: 57 ('Meta')
[2021-09-08 14:37:47.001] [renderer3] [info] [KeybindingService]: \ Keyboard event cannot be dispatched in keydown phase.
[2021-09-08 14:37:47.008] [renderer3] [info] [KeybindingService]: / Received  keydown event - modifiers: [shift,meta], code: ShiftLeft, keyCode: 16, key: Shift
[2021-09-08 14:37:47.008] [renderer3] [info] [KeybindingService]: | Converted keydown event - modifiers: [shift,meta], code: ShiftLeft, keyCode: 4 ('Shift')
[2021-09-08 14:37:47.009] [renderer3] [info] [KeybindingService]: \ Keyboard event cannot be dispatched in keydown phase.
[2021-09-08 14:37:47.388] [renderer3] [info] [KeybindingService]: + Storing single modifier for possible chord shift.
[2021-09-08 14:37:47.689] [renderer3] [info] [KeybindingService]: + Clearing single modifier due to 300ms elapsed.
hi this is my result

Thanks @eret9616 . This indicates that VS Code does not receive this keybinding. Most likely, the OS or some other software is consuming it and preventing it from reaching VS Code. There is nothing we can do on our side in this case.


Could not find facility which can set grid system as per device

When user make four columns for example & want to set col-lg-4 col-md-4 col-sm-6 col-xs-12 for those all, could not. By default it is taking col-lg-4 col-md-4 col-sm-4 col-xs-4.
Yes we can do from source code. But if you can bring that handy, appreciable

I agree that this is a problem. Normally I only set col-md-4 if I want 3 equal columns. And then when displayed on a mobile device by standard the columns are shown on top of each other which is fine.
But when using this grid-editor col-sm-4 and col-xs-4 are automatically added for every column which means it shows the 3 columns also on a extra small screen and this does not look good.
Then I have to manully edit all code in code-view mode. And if you dont know anything about bootstrap this is a problem. (e.g. my customers..)

I don't get this issue. You don't have to edit code, you can select tablet or phone at the top right, and any change you make will only reflect that screen-size.

Ahh okay. I didn't know you could actually use that and make changes for the different screen sizes this way. Great. Then there is no issue anymore:)


Update "migrate from" code to match any updates to training data formats

There have been a few changes/updates to some of (or maybe all of) the platforms we offer migrations from:

[ ] Dialogflow
[ ] wit.ai
[ ] LUIS
[x] IBM Watson (already ticked because we just have a docs page for this, and not an actual converter)

For Rasa 2.0 these should be updated both to convert to the new yaml training data (unsure if that's already done), and also to read any new training data formats from these platforms.
Things that need to be done:

[ ] investigate which platforms have made changes to their platform and/or training data format (i'm pretty sure both wit.ai + LUIS at least have)
[ ] implement updates to our converters
[ ] update the documentation

@akelad The @RasaHQ/production-engineers were asking if your team has bandwidth to take this on. Maybe it could be a joint effort? For example, someone on your team could handle investigate the changes to to other platforms and update the documentation, while someone on the production squad updates the converters

@RasaHQ/cseteam thoughts? I'm not so sure about bandwidth, there's a lot of docs stuff going on still as well.

I'm not so sure about bandwidth either tbh

@TyDunn I think this is a @RasaHQ/loop-squad topic as it's related to the training data.

@wochinge The loop squad has a lot going on right now to make Rasa X compatible. If the production squad has bandwidth, then I think we should pick it up

We moved this to 2.1 as no squad has the bandwidth for it. Do you have strong objections @akelad ? I know it's not ideal but the only alternative would be to push the release.

@wochinge We agreed to this on Slack

Ok, let's discuss these things on issues 👍

@RasaHQ/cseteam @RasaHQ/production-engineers @m-vdb  I split the work between squads. CSE will investigate what we need to do update the convertors and docs, so that production squad can make the necessary updates to the convertors. Before we ship, we will need one of the two teams to update the docs

@TyDunn should this issue be closed then since we have other issues?

@akelad Let's keep this as an umbrella ticket that we close once all three sub-issues are complete


How to deploy a project in laradock ？

Q
A

Issue Type
Question

Deployer Version
N/A

Local Machine OS
mac

Remote Machine OS
ubuntu (laradock)

Description
I want to deploy a laravel project in laradock. I can update code files to server.But how to run php  command  in laradock ？Is there a configuration to run php  command  in laradock ？Thank you!

I never worked with laradock, can't tell.

@antonmedv
Can I run some commands after ssh host?
I need run command "docker-compose exec workspace bash". Then I can run command other tasks in docker container.
I try to add a task:
task('deploy:intodocker', function () {
    run("cd /home/www/laradock && docker-compose exec workspace bash");
})
before('deploy:prepare', 'deploy:intodocker');

But I failed.

You need to connect to laradock by ssh.

@kisexu @antonmedv I have the same problem. I am new to Laradock and Deployer (both look so cool!), and I installed them yesterday, and where I'm stuck is a couple places:

I keep getting this warning To speed up composer installation setup "unzip" command with PHP zip extension https://goo.gl/sxzFcD even though I'm pretty sure I've installed the zip extension in the appropriate containers.
When Deployer reaches the vendors.php script, I get this error:

In Client.php line 99:

  [Deployer\Exception\RuntimeException (1)]
  The command "command -v 'php'" failed.

  Exit Code: 1 (General error)

So then I tried overriding with:
task('deploy:vendors', function () {
    run('cd ~/laradock && docker-compose exec workspace bash');
    if (!commandExist('unzip')) {
        //writeln('<comment>To speed up composer installation setup "unzip" command with PHP zip extension https://goo.gl/sxzFcD</comment>');
    }
    //run('cd {{release_path}} && {{bin/composer}} {{composer_options}}');
    run('cd /var/www/myproject/release && {{bin/composer}} {{composer_options}}');
});

(sort of like @kisexu ) but now I'm stuck with this error:
[Deployer\Exception\RuntimeException (255)]
  The command "cd ~/laradock && docker-compose exec workspace bash" failed.

  Exit Code: 255 (Unknown error)

I'm curious if there is a way to get Deployer working with Docker.

The unzip extension you can add to your PHP worker. As for the other errrors, I would check logs in workspace.

@kisexu Did you ever get Deployer to work with Docker?
I posted a question here: https://stackoverflow.com/questions/51274489/how-to-use-deployer-with-docker-laradock
@jasperf Yes, I think the zip extension is already installed in my php-fpm Docker container, but the problem is that Deployer won't switch into that container (as you can see in my recent StackOverflow question).
I'm super confused about why run('cd /root/laradock && (docker exec -i $(docker-compose ps -q php-fpm) bash;)'); would be completely ignored by Deployer. It's as if nothing happens. I've used Xdebug to step through the code and see that it successfully runs proc_open at https://github.com/symfony/process/blob/v4.1.0/Process.php#L304, but in reality, Deployer stays outside of the Docker container that it would have entered if docker exec -i $(docker-compose ps -q php-fpm) bash actually had been run.
I'd appreciate any ideas, @antonmedv . Your tool is so impressive! I hope I can get it to work for me. This is all I've been working on for almost 3 days now.
More details about what I've tried are at: https://stackoverflow.com/questions/51274489/how-to-use-deployer-with-docker-laradock
Thanks!

@ryancwalsh laradock created only for local development?

@antonmedv I'm almost positive that it doesn't matter how the Docker containers were created (Laradock or otherwise).
(If I had created Docker containers from scratch without Laradock, I'd face the same problem right now.)
So this question could be renamed to "How to use Deployer with Docker containers?"

@antonmedv Thanks for your response! I look forward to v7.

@antonmedv I'm ready to help))
Here is my mail
<EMAIL_ADDRESS>Please, write your telegram phone number (or suggest any other option)

https://github.com/deployphp/deployer/pull/1382
This is a good start.

same problem “How to use Deployer with Docker containers?”
look forward to v7 too.....

I've sent the docker setup 1-2 months ago to the maintainer :)

I sent the docker setup 1-2 months ago to the maintainer :)

Would you care to share a repo on this? There is a pull request by @blizzke already by the way.

To use laravel deployer you should connect via ssh directly to the workspace container.
You can expose the container's ssh port: https://laradock.io/documentation/#access-workspace-via-ssh
Let's say you've forwarded container ssh port 22 to vm port 2222. In that case you need configure your deployer to use the port 2222.
Also remember to set proper secure SSH keys, not the default ones.

@Yudikubota Forwarding workspace ssh port 22 to VPS port 2222 is an interesting idea. This so port 2222 would give direct access to the workspace via SSH instead of the VM SSH server.

@kisexu Did you ever get Deployer to work with Docker?
I posted a question here: https://stackoverflow.com/questions/51274489/how-to-use-deployer-with-docker-laradock
@jasperf Yes, I think the zip extension is already installed in my php-fpm Docker container, but the problem is that Deployer won't switch into that container (as you can see in my recent StackOverflow question).
I'm super confused about why run('cd /root/laradock && (docker exec -i $(docker-compose ps -q php-fpm) bash;)'); would be completely ignored by Deployer. It's as if nothing happens. I've used Xdebug to step through the code and see that it successfully runs proc_open at https://github.com/symfony/process/blob/v4.1.0/Process.php#L304, but in reality, Deployer stays outside of the Docker container that it would have entered if docker exec -i $(docker-compose ps -q php-fpm) bash actually had been run.
I'd appreciate any ideas, @antonmedv . Your tool is so impressive! I hope I can get it to work for me. This is all I've been working on for almost 3 days now.
More details about what I've tried are at: https://stackoverflow.com/questions/51274489/how-to-use-deployer-with-docker-laradock
Thanks!

Hi.Have you ever solved this problem?I have the same problem.How to enter the php container in deployer.The command like php or composer it need to run in php container.In the outside of the container i have no install any php or composer,so it absolutly deploy failed.
My English is not good,hope you would understand what i say.Thank you.

Couldn't you use port 2222 for your Workspace or something as suggested here by https://github.com/kOld
host('SERVER-IP')
    ->set('deploy_path', '/path/to/deployer')
    ->set('bin/php', 'cd /path/to/laradock/ && docker-compose exec -T workspace php')
    ->set('bin/composer', 'cd /path/to/laradock/ && docker-compose exec -T workspace composer -d={{release_path}}');

and to reload this:
set('php_fpm_service', 'php7.2-fpm');
set('php_fpm_command', 'echo "" | sudo -S /usr/sbin/service {{php_fpm_service}} reload'); // for setups like laravel forge
set('php_fpm_command', "kill -USR2 "$(ps -ef | grep '[p]hp-fpm: master' | awk '{print $2}')""); // for docker based setups with laradock or similar
task('fpm:reload', function () {
run('{{php_fpm_command}}');
})->desc('Reload the php-fpm service');

Is there a no problem way for deploy using laradock now?


Replace files modal iframe with simple HTML

Also fixes file_links initialization for newly uploaded files.
Fixes #807

Awesome!


Body Menu

Im using a Template for Slider but i t stays on the Menu when i use it. its the Idu and Brakka mods it just does not want to move to adjusting the other body parts when i use the template in slider i cant post the templates in here i guess first time here

What version?
Also can you upload the target config.ini?
Additionally, the console should output some logs, if you could provide those.
My educated guess from looking at the mod https://www.nexusmods.com/dragonsdogma2/mods/158 is that it uses the old config.ini style.
I would update it, but I think the user expressed wanting to have the file on nexus and not included by default in the tool.

Can confirm this is the issue:
[Character]
; FEMALE PRESET - ONLY WORKS FOR FEMALES
; Body
height = 215
torso = 53
muscle_style = 8
muscle_strength = 100
traps = 51
upper_mass = 89
lower_mass = 90
upper_size = 0
lower_size = 33

; Head
head_size = -36
head_length = 10
head_width = -2
head_depth = -42
face_height = 82
neck_length = 52
neck_thickness = 31

Is the old format. Needs to be updated.

I've pinged the mod author:
https://github.com/Mx772/dragons-dogma-2-auto-character-creator/issues/9

They note it's fixed in their latest patch.


Use at instead of recursion for Searchable Iterables

For Iterables, Searchable could use at instead of recursive iteration whenever that's more efficient, which is probably always the case anyway.

Turns out that we can't do that, because that would require knowing the length of the sequence. Since Iterables are allowed to be infinite (i.e. they don't have to be Foldable), we can't peek at their length. What could be done, however, is to have overloads for Iterables that are also Foldable, in which case we could use index-based iteration. However, this is probably not that useful, since Tuple already optimizes the Searchable minimal complete definition, and everything finite is essentially based on Tuple. Closing this.


Improve rest test flakyness

Checking the traces of https://app.circleci.com/pipelines/github/mui/mui-toolpad/9320/workflows/5ef0f164-077b-4de9-ba2f-1ebaf2ab5480/jobs/40994/artifacts

It looks like the test app gets stuck on old urls embedded in the project files. At the moment we rely on the project reloading correctly when we update the files, but there seem to be some race conditions during startup of Toolpad.
This PR just replaces the file contents before starting Toolpad. Which was always the intent of this test.

First time since long I see aCI test run without any flaky tests


Show progress when reconstructing blocks log with --hard-replay

When running "/opt/eosio/bin/nodeos --hard-replay --replay-blockchain" it would be nice to see some progress output to the log.  Once per minute or so.  On mainnet more than 10 minutes go by with no logs on a fast computer
Dec 27 10:53:16 server1 nodeos[15900]: info  2018-12-27T15:53:16.585 thread-0  http_plugin.cpp:422           plugin_initialize    ] configured http to listen on <IP_ADDRESS>:8888
Dec 27 10:53:16 server1 nodeos[15900]: info  2018-12-27T15:53:16.585 thread-0  chain_plugin.cpp:335          plugin_initialize    ] initializing chain plugin
Dec 27 10:53:16 server1 nodeos[15900]: info  2018-12-27T15:53:16.585 thread-0  chain_plugin.cpp:494          plugin_initialize    ] Hard replay requested: deleting state database
Dec 27 10:53:16 server1 nodeos[15900]: info  2018-12-27T15:53:16.585 thread-0  block_log.cpp:368             repair_log           ] Recovering Block Log...
Dec 27 10:53:16 server1 nodeos[15900]: info  2018-12-27T15:53:16.588 thread-0  block_log.cpp:388             repair_log           ] Moved existing blocks directory to backup location: '/root/.local/share/eosio/nodeos/data/blocks-2018-12-27T15:53:16.585'
Dec 27 10:53:16 server1 nodeos[15900]: info  2018-12-27T15:53:16.588 thread-0  block_log.cpp:393             repair_log           ] Reconstructing '/root/.local/share/eosio/nodeos/data/blocks/blocks.log' from backed up block log
Dec 27 11:05:47 server1 nodeos[15900]: info  2018-12-27T16:05:47.225 thread-0  block_log.cpp:522             repair_log           ] Existing block log was undamaged. Recovered all irreversible blocks up to block number 34049752.
Dec 27 11:05:47 server1 nodeos[15900]: info  2018-12-27T16:05:47.245 thread-0  chain_plugin.cpp:504          plugin_initialize    ] Reversible blocks database was not corrupted. Copying from backup to blocks directory.
Dec 27 11:05:52 server1 nodeos[15900]: info  2018-12-27T16:05:52.627 thread-0  block_log.cpp:134             open                 ] Log is nonempty
Dec 27 11:05:52 server1 nodeos[15900]: info  2018-12-27T16:05:52.628 thread-0  block_log.cpp:178             open                 ] Index is empty
Dec 27 11:05:52 server1 nodeos[15900]: info  2018-12-27T16:05:52.628 thread-0  block_log.cpp:330             construct_index      ] Reconstructing Block Log Index...
...

New logging added.  There are two new progress messages, one for reconstruction of the index file and one for reconstruction of the block log file.  The formats are, respectively:
Block log index reconstructed for block ####

and
Recovered block ####

where '####' is a block number.  The log message appears every thousand blocks.


Copy ID on popup menu of note always copy nevent1xxx instead of note1xxx.

Describe the bug
Copy ID on popup menu of note always copy nevent1xxx instead of note1xxx.
To Reproduce
Click button ︙ and click "Copy ID"
Screenshots

OS: Linux/Android/Windows...
Browser Google Chrome
Version latest

Yes, but nevent can't be used for quoted post like below.
I know it

@note1xxx

I see, so this is an issue with quoting events

Fixed recently


Deleting posts deletes md column in database

When a post is deleted, the md is cleared in the database (publications table, md column)
This doesn't currently appear to be a visible bug, because md is never read from the database. But it will be, if md is used in the future.
The issue is that save-pub calls db-save-publication, which deletes and re-creates the publications row, but doesn't restore the markdown, which is no longer part of the pub structure.
The solution is to divide save-pub into two functions, one of which inserts, and one updates, to avoid deleting
Alternatively, we could make db-save-publication query the md field, delete and insert, and re-insert the md. But that would be slow. Better to not delete. Especially since we'll be removing more fields in the future.

Fixed in changeset 8e3bea.


[chore] discussion

Hi, can you enable the beta discussion feature please

This will add a new tab where we can chat

done


Support additional styling in stylesContributions

VSCode Version: all
OS Version: all

Hello,
I'd like to know if it could be possible to add support for other CSS properties in the styleContributions code. Namely:

text-transform
text-decoration

(and why not all the ones mentionned in http://www.w3schools.com/css/css_text.asp , but that might be overkill).
For example, I'd like to have support for capitalization via style in order to develop a syntax style that looks like the PICO-8 UI:

(note that in the usual code, all text is actually typed as lowercase and displays as uppercase.)
However the actual idea lies in the idea of supporting a slightly more extended set of css properties that could be customized via styles. This breaks the backward compatibility, though.
I can provide a PR, if required.
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/themes/electron-browser/stylesContributions.ts#L120

I think you could have to do that with the font at the moment.

What about editing? How would the user get to enter the correct case for a case-sensitive language like JavaScript?

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
If you disagree and feel that this issue is crucial: We're happy to listen and to reconsider.
Maybe you want to give it a try yourself and provide the feature through an extension or a PR: We're happy to give advise and get you started.
Thanks for your understanding. Thanks for helping and happy coding!


Migrating Migrations from .NET Framework to NET Core

Describe what is not working as expected.
 The current documents suggest to retrieve the model from the database.  What I try to have is get all the migrations migrated as well, so that it is nicely in sync with the current state the application is in and sync with the application. See stackoverflow question: https://stackoverflow.com/questions/50157585/migrate-ef-migrations-to-efcore
So, I am trying to re-generate every migration by retrieving old changes of the application. The first one already got me into trouble.
Model:
    public class AllowedProductCombination
    {
        public int allowedProductCombinationId { get; set; }
        public int product1Id { get; set; }
        public int product2Id { get; set; }
        public int? productVariant1Id { get; set; }
        public int? productVariant2Id { get; set; }
        public bool product1Priority { get; set; }

        public Product Product1 { get; set; }
        public Product Product2 { get; set; }
        public ProductVariant Product1Variant { get; set; }
        public ProductVariant Product2Variant { get; set; }

        public override string ToString()
        {
            return "allowedProductCombiId:" + allowedProductCombinationId + " product1Id:" + product1Id + " productVariant1Id:" + productVariant1Id
                + " - product2Id:" + product2Id + " productVariant2Id:" + productVariant2Id;
        }
    }

EF6 working model relation binding:
modelBuilder.Entity<AllowedProductCombination>().HasRequired(p1 => p1.Product1).WithMany(p => p.AllowedProductCombinations1).HasForeignKey(p1 => p1.product1Id);
modelBuilder.Entity<AllowedProductCombination>().HasRequired(p2 => p2.Product2).WithMany(p => p.AllowedProductCombinations2).HasForeignKey(p2 => p2.product2Id);
modelBuilder.Entity<AllowedProductCombination>().HasOptional(pv1 => pv1.Product1Variant).WithMany(pv => pv.AllowedProductCombinations1).HasForeignKey(pv1 => pv1.productVariant1Id);
modelBuilder.Entity<AllowedProductCombination>().HasOptional(pv2 => pv2.Product2Variant).WithMany(pv => pv.AllowedProductCombinations2).HasForeignKey(pv2 => pv2.productVariant2Id);

Converted to EFCore
 modelBuilder.Entity<AllowedProductCombination>().HasOne(pv1 => pv1.Product1Variant).WithMany(pv => pv.AllowedProductCombinations1).HasForeignKey(pv1 => pv1.productVariant1Id).OnDelete(DeleteBehavior.ClientSetNull);
modelBuilder.Entity<AllowedProductCombination>().HasOne(pv2 => pv2.Product2Variant).WithMany(pv => pv.AllowedProductCombinations2).HasForeignKey(pv2 => pv2.productVariant2Id).OnDelete(DeleteBehavior.ClientSetNull);
modelBuilder.Entity<AllowedProductCombination>().HasOne(p1 => p1.Product1).WithMany(p => p.AllowedProductCombinations1).HasForeignKey(p1 => p1.product1Id).IsRequired().OnDelete(DeleteBehavior.Cascade);
modelBuilder.Entity<AllowedProductCombination>().HasOne(p2 => p2.Product2).WithMany(p => p.AllowedProductCombinations2).HasForeignKey(p2 => p2.product2Id).IsRequired().OnDelete(DeleteBehavior.Cascade);

When executing the command in PowerShell:
dotnet ef migrations add initital

It generates this in the file<PHONE_NUMBER>2409_initital.cs
//rest not important for this issue
                    table.ForeignKey(
                        name: "FK_AllowedProductCombinations_Products_product1Id",
                        column: x => x.product1Id,
                        principalTable: "Products",
                        principalColumn: "productId",
                        onDelete: ReferentialAction.Cascade);
                    table.ForeignKey(
                        name: "FK_AllowedProductCombinations_Products_product2Id",
                        column: x => x.product2Id,
                        principalTable: "Products",
                        principalColumn: "productId",
                        onDelete: ReferentialAction.Cascade);
                    table.ForeignKey(
                        name: "FK_AllowedProductCombinations_ProductVariants_productVariant1Id",
                        column: x => x.productVariant1Id,
                        principalTable: "ProductVariants",
                        principalColumn: "productVariantId",
                        onDelete: ReferentialAction.Restrict);//note this
                    table.ForeignKey(
                        name: "FK_AllowedProductCombinations_ProductVariants_productVariant2Id",
                        column: x => x.productVariant2Id,
                        principalTable: "ProductVariants",
                        principalColumn: "productVariantId",
                        onDelete: ReferentialAction.Restrict);//note this

So it does not do what I have told the OnModelCreating, the designer, which is for some reason next to it and not "within it":

Generates this:
                    b.HasOne("Babywatcher.Core.Data.Entities.ProductVariant", "Product1Variant")
                        .WithMany("AllowedProductCombinations1")
                        .HasForeignKey("productVariant1Id");

                    b.HasOne("Babywatcher.Core.Data.Entities.ProductVariant", "Product2Variant")
                        .WithMany("AllowedProductCombinations2")
                        .HasForeignKey("productVariant2Id");

So it is missing the .OnDelete()
when I execute dotnet ef database update it throws an error, even after inserting the OnDelete myself like so: .OnDelete(DeleteBehavior.ClientSetNull); it throws the same error.
If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message: Failed executing DbCommand (20ms) [Parameters=[], CommandType='Text', CommandTimeout='60']
CREATE TABLE [AllowedProductCombinations] (
    [allowedProductCombinationId] int NOT NULL IDENTITY,
    [product1Id] int NOT NULL,
    [product1Priority] bit NOT NULL,
    [product2Id] int NOT NULL,
    [productVariant1Id] int NULL,
    [productVariant2Id] int NULL,
    CONSTRAINT [PK_AllowedProductCombinations] PRIMARY KEY ([allowedProductCombinationId]),
    CONSTRAINT [FK_AllowedProductCombinations_Products_product1Id] FOREIGN KEY ([product1Id]) REFERENCES [Products] ([productId]) ON DELETE CASCADE,
    CONSTRAINT [FK_AllowedProductCombinations_Products_product2Id] FOREIGN KEY ([product2Id]) REFERENCES [Products] ([productId]) ON DELETE CASCADE,
    CONSTRAINT [FK_AllowedProductCombinations_ProductVariants_productVariant1Id] FOREIGN KEY ([productVariant1Id]) REFERENCES [ProductVariants] ([productVariantId]) ON DELETE SET NULL,
    CONSTRAINT [FK_AllowedProductCombinations_ProductVariants_productVariant2Id] FOREIGN KEY ([productVariant2Id]) REFERENCES [ProductVariants] ([productVariantId]) ON DELETE SET NULL
);
Stack trace:
System.Data.SqlClient.SqlException (0x80131904): Introducing FOREIGN KEY constraint 'FK_AllowedProductCombinations_Products_product2Id' on table 'AllowedProductCombinations' may cause cycles or multiple cascade paths. Specify ON DELETE NO ACTION or ON UPDATE NO ACTION, or modify other FOREIGN KEY constraints.
Could not create constraint or index. See previous errors.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean async, Int32 timeout, Boolean asyncWrite)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean asyncWrite, String methodName)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.ExecuteNonQuery(IRelationalConnection connection, IReadOnlyDictionary`2 parameterValues)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.MigrationCommandExecutor.ExecuteNonQuery(IEnumerable`1 migrationCommands, IRelationalConnection connection)
   at Microsoft.EntityFrameworkCore.Migrations.Internal.Migrator.Migrate(String targetMigration)
   at Microsoft.EntityFrameworkCore.Design.Internal.MigrationsOperations.UpdateDatabase(String targetMigration, String contextType)
   at Microsoft.EntityFrameworkCore.Design.OperationExecutor.OperationBase.Execute(Action action)
ClientConnectionId:6788fabb-a7b9-4ea5-a962-777d811e82df
Error Number:1785,State:0,Class:16
Introducing FOREIGN KEY constraint 'FK_AllowedProductCombinations_Products_product2Id' on table 'AllowedProductCombinations' may cause cycles or multiple cascade paths. Specify ON DELETE NO ACTION or ON UPDATE NO ACTION, or modify other FOREIGN KEY constraints.
Could not create constraint or index. See previous errors.

Steps to reproduce
Create a .NET Standard project.
Make it compatible for NETCore:
  <PropertyGroup>
    <TargetFrameworks>netcoreapp2.0;netstandard2.0</TargetFrameworks>
  </PropertyGroup>

Installed packages:
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Identity.EntityFrameworkCore" Version="2.0.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="2.0.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="2.0.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.0.2" />
  </ItemGroup>

CLI added:
  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.DotNet" Version="2.0.2" />
  </ItemGroup>

Then just insert model, the DatabaseContext that uses IdentityDbContext from Microsoft.AspNetCore.Identity.EntityFrameworkCore
Console.WriteLine("Hello World!");

Further technical details
EF Core version: 2.0.2
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows 10, Visual Studio 2017 Enterprise, MSSQLLocalDB v13
connection string, if important, in DatabaseContext:
  protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            optionsBuilder
                .UseSqlServer(@"Data Source=(localdb)\MSSQLLocalDB;Initial Catalog=newDatabase;Integrated Security=True;Connect Timeout=30;Encrypt=False;TrustServerCertificate=True;ApplicationIntent=ReadWrite;MultiSubnetFailover=False",
                    providerOptions => providerOptions.CommandTimeout(60))
                .UseQueryTrackingBehavior(QueryTrackingBehavior.NoTracking);
            base.OnConfiguring(optionsBuilder);
        }

IDE: (e.g. Visual Studio 2017 15.4)

@Ruud-cb A few questions:

Why is it that you want to recreate all the migrations?
When you say, "So it does not do what I have told the OnModelCreating" do you mean because the onDelete behavior is set to Restrict? If so, this is correct. The "client" part of ClientSetNull has no meaning on the database, so they get the normal behavior for optional relationships in the database, which is Restrict.
When you sat, "So it is missing the .OnDelete()" it looks like you are referring to the model snapshot. It's possible that these should be there - @bricelam may be able to confirm. However, I don't see this actually causing any issues.
With regard to the cascade cycles, this is an issue with SQL Server and was the same in EF6. What does the EF6 database look like with respect to these constraints?

We could generate .OnDelete(DeleteBehavior.ClientSetNull). 😏 We could also generate .ValueGeneratedNever() on every property...
lol, not sure if we should but it shouldn't be causing any functional issues since it's the default (without conventions)

Hi @ajcvickers

Today I will be testing if 1 single migration will work as well, but in some migrations manual adjustments have been made (like insert default values). So it would be the most ideal solution to automatically refactor the current migrations, which seems to be not possible.
I would expect that the settings I've set in the OnModelCreating would also be included in the migrations, but apparently Restrict is a good representation of what I have put in OnModelCreating.

Well, eventually this seems to much work, looks like the most easiest solution is to generate a new database and migrate all the data to the new database. Something not really convenient, but to much work to do it in any other way... -1's for starting out with an existing application/database in EFCore.


Update autoscaling example name

Minor correction for example name.

Codecov Report
Patch coverage has no change and project coverage change: -0.03% :warning:

Comparison is base (7c92254) 93.39% compared to head (bd0335f) 93.36%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2997      +/-   ##
===========================================
- Coverage    93.39%   93.36%   -0.03%
===========================================
  Files           66       66
  Lines        13963    13961       -2
===========================================
- Hits         13041    13035       -6
- Misses         922      926       +4

see 5 files with indirect coverage changes

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


Translation ending in error

When trying to translate "net/minecraft/client/player/LocalPlayer.displayClientMessage" I recieve an error and a notification to report it.

That is a linkie issue, not a Geckolib issue.

Sorry! I meant to post this on Linkies github and must have gotten confused.
On Sat, Apr 2, 2022, 11:27 AM AzureDoom @.***> wrote:

That is a linkie issue, not a Geckolib issue.
—
Reply to this email directly, view it on GitHub
https://github.com/bernie-g/geckolib/issues/236#issuecomment-1086697001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR2XK7S7NSH2YCU6XCAY2YDVDCGRFANCNFSM5SLGGHVA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update redis to 2.8.21

http://benmmurphy.github.io/blog/2015/06/04/redis-eval-lua-sandbox-escape/

Merged in https://github.com/Homebrew/homebrew-versions/commit/50d7869f84be4136fb0c0fa0564aeb90cb28fe91. Thanks for your contribution to Homebrew, we appreciate it! :smiley_cat:


Rancher CLI should provide an appropriate error message for invalid/unsupported -e option

Rancher versions:
rancher/server:v1.6.3-rc1
Rancher CLI version
v0.6.2-rc1
**Operating system and kernel: (cat /etc/os-release, uname -r preferred)**Ubuntu
**Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)**GCE
**Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)**Single node
**Environment Template: (Cattle/Kubernetes/Swarm/Mesos)**Cattle
Steps to Reproduce:

Set up rancher CLI
Start a container with -e "=test" which is not supported

Results
CLI just prints the container id.  The container fails to start and errors out in the setup
CLI should provide a valid error message that the option -e"=test" is not supported or it is invalid similar to docker
soumyas-MBP:rancher-v0.6.2-rc1 soumya$ ./rancher run -e "=test" -i --name testcontainer1 ubuntu
1s95

Available in CLI v0.6.2-rc2

Tested with CLI v0.6.2-rc2 using rancher v1.6.3-rc1.  Starting a container with -e "=foo1" option gives an appropriate error message. The container does not error out in the setup.
soumyas-MBP:rancher-v0.6.2-rc2 soumya$ ./rancher run -e "=foo1" --name test1 -i ubuntu
invalid argument "=foo1" for e: invalid environment variable: =foo1
See 'rancher run --help'.


Base mime on file extension not on contents (chmln#70)

Copy of https://github.com/chmln/handlr/issues/70

Right now, the crate uses the contents of the file to determine mime, similar to the command file. However, it should base it on solely the file extension, like xdg-*.
Right now, if you make an empty file, eg. touch test.txt, then try handlr open test.txt, it will fail with no handlers found for 'application/x-zerosize'.

I can replicate this on my end.
I disagree that handlr should solely use the file extension to determine mimetype, but I agree that the file extension should take precedence.  Using file contents should still be a fallback for when the file extension is ambiguous.
If you think it would be worthwhile, I could try to implement a config option that completely ignores file contents.
Also, while I'm working on a fix for this, you might want to consider using the regex handler feature as a temporary workaround if you aren't already.
Just in case you aren't, this snippet might help. Just replace EDITOR with whatever program you use for .txt files and delete terminal = true if it's a GUI program.
[[handlers]]
exec = "EDITOR %f"
terminal = true
regexes = ['.*\.txt']

I think I fixed it. Would you mind downloading and running the fix-file-extension-not-contents branch to confirm it works on your system?

Yes this works for me as well! 👍🏽

Great! Thanks for reporting this!


Added titus-network mode support & v6, v4, and TSA awareness

This new network-mode is designed to be a "source of truth"
for what kind of networking to setup on a container.
Specifically I want some way to know for sure if TSA and VPCservice
are supposed to setup a ipv4 transition mechanism.
OR, in the future case where we know we want ipv6 only, vpcservice
needs to know that and forgo certain things.
Likewise, titus-inject-metadata will probably have to behave
differently. And also I even expect user applications to take advantage
of the TITUS_NETWORK_MODE=IPV6_ONLY style hints, so that they
don't have to scrape ifconfig or other weirdness, and can adjust
their behavior appropriately.
This just plumbs the setting through, no additional code is in place
yet to actually act on it.

@sargun redone. EffectiveNetworkMode() is now the source of truth for what kind of network mode we want, and that is the only thing vpctool gets.
EffectiveNetworkMode() is computed based off of the input network mode and the attributes for backwards compat.


Refactor Storm platform to introduce Edges

In order to introduce grouped leftJoin I would like to make tuples we send between nodes to be more precise. In particular I want to send (K, V) tuples as tuples with key and value fields.
In this PR I did a small refactoring to introduce Edge trait which correspond to edges in Storms topology DAG - it contains a way how to serialize/deserialize data into Fields and how to group data over this edge.

Currently when we send tuples over the wire we send them only in two formats: as an item (then it's just one value Field) or as something partially aggregated (aggKey and aggValue with (Int, Map[K, V]) as a content).
This breaks if we want to implement grouped join because we want to have fields we are able to group by (therefore first format is out) but also we don't want to have things partially aggregated (therefore second is out).
Also right now we can runtime issues in topologies where we do flatMap and sumByKey in parallel branches from same flatMap node (see https://github.com/twitter/summingbird/issues/725). This happens because of the same reason - flatMap node should emit aggregated and non aggregated values at the same time.
I can see three ways to fix this issues:

simplest - send all key value pairs in aggregated format, even if it's not suppose to be aggregated.
Pros: all bolts have simple contract with one input and output format, simplest implementation. Cons: on each (K, V) pair we will have some size overhead (shardId Int, two tuple objects) and some performance overhead (but I think it's not really substantial)
medium - all bolts send only one type of tuples (regardless to downstream), but on receive side you can distinguish based on input component. We need second part for next use case: imagine you have flatMap which emits to both sumByKey and another flatMap. In this case the only way to make topology correct is to emit single element aggregated (K, V) on source flatMap, with treating them as (K, V) on flatMap downstream node and as aggregated (K, V) on sumByKey downstream node.
Pros: contract is more or less simple, no size runtime overhead for cases which works today
Cons: still overhead in some cases, not that simple to implement (I chase this one, that's why I have Map[String, Edge[]] as an input type for Bolt)
most complex and precise - implement different strategies which depends on both sending and receiving sides.
Pros: smallest possible runtime overhead (especially if we will implement customizable OperationContainer which will be able to do and not to do partial aggregation at the same time on the same node based on downstream)
Cons: involves biggest change, especially in BaseBolt to support customized emits based on target componentId

Which one is the best, what do you think @johnynek @pankajroark ?

Codecov Report

Merging #728 into develop will increase coverage by 0.04%.
The diff coverage is 93.75%.

@@             Coverage Diff             @@
##           develop     #728      +/-   ##
===========================================
+ Coverage    71.38%   71.43%   +0.04%
===========================================
  Files          141      142       +1
  Lines         3491     3504      +13
  Branches       195      197       +2
===========================================
+ Hits          2492     2503      +11
- Misses         999     1001       +2

Impacted Files
Coverage Δ

...cala/com/twitter/summingbird/storm/Constants.scala
100% <ø> (ø)
:arrow_up:

.../com/twitter/summingbird/storm/StormPlatform.scala
82.9% <100%> (+0.55%)
:arrow_up:

...ain/scala/com/twitter/summingbird/storm/Edge.scala
100% <100%> (ø)

...witter/summingbird/storm/spout/KeyValueSpout.scala
78.94% <100%> (+1.16%)
:arrow_up:

...a/com/twitter/summingbird/storm/EdgeGrouping.scala
66.66% <66.66%> (ø)

...scala/com/twitter/summingbird/storm/BaseBolt.scala
77.01% <75%> (-0.1%)
:arrow_down:

...witter/summingbird/storm/FlatMapBoltProvider.scala
98.55% <92.3%> (-1.45%)
:arrow_down:

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 731da7f...2d02285. Read the comment docs.

On strategies: the second strategy, the one you're pursuing, does seem like the right one to me. I'm still reviewing the code.

lgtm

@johnynek are you good with this change? I'm preparing next review where I did this outputEdge thing unnecessary.


Merge crates (pack-index, cmsis-update, pdsc, utils)

Tentative PR for Unifying the crates
Fixes #130

Working around the PyYAML wheel issue. It seems there is work in progress to have PyYAML wheels for Linux and Mac (including 'arm64') not just Windows, see: https://github.com/yaml/pyyaml/pull/407


Issues with layers/sampler.py and the no_repeat_ngram_size parameter

I'm adding no_repeat_ngram_size parameter and I have some problem with sampler.py.
in  transformers/generation/utils.py , my understanding order is model infer, (no_repeat processing: optional), temperature processing, topk processing, softmax. But in vllm, I found it seems like model infer,  temperature processing, softmax, topk processing.
Can you tell me why the order is different here or give me some advice to add no_repeat_ngram_size?
transformers
          outputs = self(
                **model_inputs,
                return_dict=True,
                output_attentions=output_attentions,
                output_hidden_states=output_hidden_states,
            )
            if synced_gpus and this_peer_finished:
                continue  # don't waste resources running the code we don't need

            next_token_logits = outputs.logits[:, -1, :]

            # pre-process distribution
            next_token_scores = logits_processor(input_ids, next_token_logits)
            next_token_scores = logits_warper(input_ids, next_token_scores)

            probs = nn.functional.softmax(next_token_scores, dim=-1)
            next_tokens = torch.multinomial(probs, num_samples=1).squeeze(1)

vllm
         hidden_states = self.model(......)
        hidden_states = _prune_hidden_states(hidden_states, input_metadata)

        # Get the logits for the next tokens.
        logits = torch.matmul(hidden_states, embedding.t())
        if embedding_bias is not None:
            logits += embedding_bias
        logits = gather_from_tensor_model_parallel_region(logits)
        # Remove paddings in vocab (if any).
        logits = logits[:, :self.vocab_size]

        # Apply presence and frequency penalties.
        output_tokens = _get_output_tokens(input_metadata)
        assert len(output_tokens) == logits.shape[0]
        presence_penalties, frequency_penalties = _get_penalties(
            input_metadata)
        assert len(presence_penalties) == logits.shape[0]
        assert len(frequency_penalties) == logits.shape[0]
        logits = _apply_penalties(logits, output_tokens, presence_penalties,
                                  frequency_penalties, self.vocab_size)

        # Apply temperature scaling.
        temperatures = _get_temperatures(input_metadata)
        assert len(temperatures) == logits.shape[0]
        if any(t != 1.0 for t in temperatures):
            t = torch.tensor(temperatures,
                             dtype=logits.dtype,
                             device=logits.device)
            # Use in-place division to avoid creating a new tensor.
            logits.div_(t.unsqueeze(dim=1))

        # We use float32 for probabilities and log probabilities.
        # Compute the probabilities.
        probs = torch.softmax(logits, dim=-1, dtype=torch.float)
        # Compute the log probabilities (before applying top-p and top-k).
        logprobs = torch.log(probs)

        # Apply top-p and top-k truncation.
        top_ps, top_ks = _get_top_p_top_k(input_metadata, self.vocab_size)
        assert len(top_ps) == len(top_ks) == probs.shape[0]
        do_top_p = any(p < 1.0 - _SAMPLING_EPS for p in top_ps)
        do_top_k = any(k != self.vocab_size for k in top_ks)
        if do_top_p or do_top_k:
            probs = _apply_top_p_top_k(probs, top_ps, top_ks)tates = self.model(......)
        hidden_states = _prune_hidden_states(hidden_states, input_metadata)

        # Get the logits for the next tokens.
        logits = torch.matmul(hidden_states, embedding.t())
        if embedding_bias is not None:
            logits += embedding_bias
        logits = gather_from_tensor_model_parallel_region(logits)
        # Remove paddings in vocab (if any).
        logits = logits[:, :self.vocab_size]

        # Apply presence and frequency penalties.
        output_tokens = _get_output_tokens(input_metadata)
        assert len(output_tokens) == logits.shape[0]
        presence_penalties, frequency_penalties = _get_penalties(
            input_metadata)
        assert len(presence_penalties) == logits.shape[0]
        assert len(frequency_penalties) == logits.shape[0]
        logits = _apply_penalties(logits, output_tokens, presence_penalties,
                                  frequency_penalties, self.vocab_size)

        # Apply temperature scaling.
        temperatures = _get_temperatures(input_metadata)
        assert len(temperatures) == logits.shape[0]
        if any(t != 1.0 for t in temperatures):
            t = torch.tensor(temperatures,
                             dtype=logits.dtype,
                             device=logits.device)
            # Use in-place division to avoid creating a new tensor.
            logits.div_(t.unsqueeze(dim=1))

        # We use float32 for probabilities and log probabilities.
        # Compute the probabilities.
        probs = torch.softmax(logits, dim=-1, dtype=torch.float)
        # Compute the log probabilities (before applying top-p and top-k).
        logprobs = torch.log(probs)

        # Apply top-p and top-k truncation.
        top_ps, top_ks = _get_top_p_top_k(input_metadata, self.vocab_size)
        assert len(top_ps) == len(top_ks) == probs.shape[0]
        do_top_p = any(p < 1.0 - _SAMPLING_EPS for p in top_ps)
        do_top_k = any(k != self.vocab_size for k in top_ks)
        if do_top_p or do_top_k:
            probs = _apply_top_p_top_k(probs, top_ps, top_ks)

fwiw it now looks like top_p/top_k is done before softmax https://github.com/vllm-project/vllm/blob/main/vllm/model_executor/layers/sampler.py

Hello, I noticed that you previously raised this question in the vllm library regarding the implementation of the no_repeat_ngram_size parameter . I found that this feature is still not included in the current version of the vllm library. I would like to discuss with you how this feature can be added, as I currently have a strong need for this parameter (https://github.com/vllm-project/vllm/issues/7842).


Update install-frog.bash

Added git to the packages installed

Getting this all into a single clean patch for ya.


Fix bug in memory tracking test

When no empty slot is found, tracking_index is an unitialized. Even if it happens to be zero (which we cannot assume), this is incorrect behavior because it would silently overwrite the allocation tracked at index zero rather than asserting.

Nevermind, I see now that tracking_index is also the loop iterator, so it will be equal to the length of the array if no empty slot is found. Disregard this PR.

Thanks for the code review regardless! -- cheers


removeduplicates! deletes all but a single row

A recent change somewhere in the pipeline is causing removeduplicates! to act strangely.  A simple example to reproduce is
using SQLite
db = SQLite.DB()
ints = Int64[1,1,2,2,3]
strs = UTF8String["A", "A", "B", "C", "C"]
nvInts = NullableArrays.NullableArray(ints)
nvStrs = NullableArrays.NullableArray(strs)
schema = DataStreams.Data.Schema(["ints", "strs"], [Int64, UTF8String],5)
d = NullableArrays.NullableVector[nvInts, nvStrs]
dt = DataStreams.Data.Table(schema, d,0)
SQLite.drop!(db, "temp", ifexists=true)
sink = SQLite.Sink(dt, db, "temp")
Data.stream!(dt, sink)
SQLite.query(db, "Select * from temp").data

2-element Array{NullableArrays.NullableArray{T,1},1}:
 NullableArrays.NullableArray{Int64,1}[1,1,2,2,3]
 NullableArrays.NullableArray{UTF8String,1}["A","A","B","C","C"]

This is a two-column table with a single duplicate row out of five rows.  Running removeduplicates!:
SQLite.removeduplicates!(db, "temp", "ints, strs")
SQLite.query(db, "Select * from temp").data

2-element Array{NullableArrays.NullableArray{T,1},1}:
 NullableArrays.NullableArray{Int64,1}[3]
 NullableArrays.NullableArray{UTF8String,1}["C"]

Removes all but the last row from the table for some reason.
Note that if you set up the table the same way and then do the same thing with the normal query function things work as expected:
SQLite.query(db, "DELETE FROM temp WHERE _ROWID_ NOT IN (SELECT max(_ROWID_) from temp GROUP BY ints, strs);")
SQLite.query(db, "Select * from temp").data

2-element Array{NullableArrays.NullableArray{T,1},1}:
 NullableArrays.NullableArray{Int64,1}[1,2,2,3]
 NullableArrays.NullableArray{UTF8String,1}["A","B","C","C"]

I'm using the SQLite master and this version of DataStreams which I think should be equivalent to to the master in this case.   One additional hint is that removing duplicates appears to work fine for tables with a single integer column.
I'm still trying to figure out if this is due to some recent commit in SQLite or DataStreams, or if this is something that was broken by 0.4.2 or some other package.

Ok, so it looks like the issue is with the internal function SQLite.esc_id which is used to escape identifiers.  It can accept a single string or a vector of strings, but removeduplicates! only allows a single string for the list of columns.  This means that when we call removeduplicates!(db, "temp", "ints, strs") the query that is actually executed is
DELETE FROM "temp" WHERE _ROWID_ NOT IN (SELECT max(_ROWID_) from "temp" GROUP BY "ints, strs");

which breaks things since there is not a single column called "ints, strs".  This was probably introduced in  https://github.com/JuliaDB/SQLite.jl/commit/b367878bcc0e8624653944e48a0dec86b98f8ddb.  I think this can be fixed by adding an additional replace that adds quotes on either side of commas, and will give it a shot now.

Fixing this is a bit more complicated than I thought.  The problem is that this method of adding escapes is not easily extended since you can have column names with spaces in them.  For example if someone set cols to "ints, strs" there's no way of knowing if the second column name is _strs or strs (where the underscore is a space), since either is a legal column name.
It might be better to make cols be either a single column name or an array of column names rather than always a single string, although that would be a breaking change.  Any thoughts?

yeah, I think we should require the columns argument to be an array of column name strings. For compatibility, we should probably make a single string argument an error for now, so peopel can transition.


Status line at the top

I see hardcoded numbers for the status line. Any plans to refactor the code to make it easy to recompile with status line at the top? Or even having a runtime option to set status line to the top instead of bottom?
Thanks!

Correct me if I'm wrong but I believe you'd just need to amend this line: https://github.com/tauraamui/lilly/blob/4bc952573223795190a2e9ca19aea29e9bbf0129/src/status_line.v#L37
from being: y := ctx.window_height() - 1 to y := 1 as the rest of the positioning is done relative to that. I haven't read all of that part of the code for a while, but I don't see why it can't be a configuration option at some point.


Error when adding a feed

Describe the bug
When I try to add any rss feed i get the error msg "CLEARTEXT communication not permitted by network security policy"
Screenshots

Device Information

Device: OnePlus 8 pro
OS: Android 13
App Version 1.4.0

App only supports links with https and not http


column pinning have some styling bugs

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here

Current behavior
Showing your example for column pinning here http://swimlane.github.io/ngx-datatable/# in my opinion there is a style problem. The divider between City and State Header is not horizontal aligned with the divider in the rows.

Expected behavior
I think the divider should be aligned.
Reproduction of the problem
See the example in our demo page

Any progress on this, you can still see the error in the live pinning demo

There also seems to be an issue with pinned columns overriding the theme of the datatable. For example, I fetch my columns and rows from server and then I say, if this is a pinned column, create as pinned column in html, and for the rest of the columns, create in for loop.

Some news about this bug???

This is because the scrollbar is shown. I want to remove native scrollbars and use a solution like Facebook did here - https://facebook.github.io/fixed-data-table/

Any news on this one?

When there isn't the vertical scroll bar, you can see the below content, as shown in this image

Yes, even there is no scroll bar, it is also not aligned. See #1078.

I found a best way or stupid way to solve the issue temporarily, when the view is changed, find all the element with the class: datatable-row-right datatable-row-group and add margin-left: 15px attributes.
public ngAfterViewChecked() {
    const elements: HTMLElement[] =
        this.elRef.nativeElement.getElementsByClassName('datatable-row-right datatable-row-group');
    if (elements && elements.length > 0) {
      for (const obj of elements) {
        obj.style.setProperty('margin-left', '15px');
      }
    }
  }

Similar issue I am seeing is that when using bootstrap or dark theme, pinned columns don't hide scrolled columns beneath like material theme does. You can see the issue by changing the theme on the pinning demo (http://swimlane.github.io/ngx-datatable/#pinning) using the dev console. If you change it to bootstrap, the middle columns scroll horizontally and you can still see them under the frozen columns

Any updates on this?  Any workarounds to allow striping of the rows, yet let the unpinned columns slide under the pinned ones like material?

The above issue can be fixed by giving margin-left: -9px and use :host /deep/ to overwrite css on dataTable for the class

:host /deep/ .ngx-datatable.material .datatable-header .datatable-row-right{
margin-left: -8px;
}

I created a pr for this bug #1524

Any progress on this issue?

i have the same issue yet not solved
i am using the version  ^16.0.2

Similar issue I am seeing is that when using bootstrap or dark theme, pinned columns don't hide scrolled columns beneath like material theme does. You can see the issue by changing the theme on the pinning demo (http://swimlane.github.io/ngx-datatable/#pinning) using the dev console. If you change it to bootstrap, the middle columns scroll horizontally and you can still see them under the frozen columns

It seems issue still occurs. Is there any solution based on CSS?


Handling proxy request errors

There is an example in docs how to make a proxy request
req.pipe(request('http://mysite.com/doodle.png')).pipe(resp)
The problem is if request error occurs it throws exception and node working process stops. How to handle request errors?

@whitecolor my solution is to add an error handler:
req.pipe(request(url).on('error', e => res.sendStatus(503))).pipe(res)


generate SSH config

So, this fails because we need Ansible 2.3 for named block feature. I made the changes for the test container but due to Gitlab's API timeout I can't get the Quay.io image rebuilt at the moment.

So, can you describe a bit more what is the problem to be fixed ?

The previous SSH config file on the builders were not managed via Ansible but created manually.
Also the way the role is written seem to indicate handling multiple builds was intended, so I made sure we're matching the right key even if we deploy a sub-part of the website on the same machine (we just need another account).

That's fine by me, but what about the ci run, should we merge it nonetheless ?

That's a problem we're going to have more and more unfortunately and we're certainly not going to stick to an old Ansible version if we need the features.
I tried again to use Quay+GL and it still timeout.t has been moths now and noone cares so I guess we should find some time (not sure when) to continue work on this.
In the meanwhile the only thing I can suggest for this PR (not sure the other ones would not have similar problems) is to comment the name on the block.

Well, I am ok to merge the PR, but the interface do not let me. So we can disable the requirement of "ci must pass" for now ?


Windows embedding unit test

We currently only have unit test set up for windows client wrapper. We should also set up the embedding unit test.

@stuartmorgan


Add [BRAVE TEAM ACCOUNT bravepay.net] #custodian GC64SV…PVNZOP

Address: GC64SVY3XSYEE7MYADTEQNO3ACCWJ6NLWNNJLPO2FGS4I2PCNBPVNZOP
Tags: custodian
Title: BRAVE TEAM ACCOUNT  Domain: bravepay.net
Requested by @batcode80

combined with
https://github.com/stellar-expert/public-directory/issues/325

done


Lag in Kourend with Quest Helper on

Navigating through Kourend doing Depth of Despair and the game is almost unplayable lagging every 3 seconds or so. This has so far been in Hosidius and Arceuus

Hey @aichele66 could you please share your runelite logs so I can see if there is an error being thrown? Thanks.

How do I share my Runelite Logs? If I delete the plugin, then reinstall the
issue goes away for a day or two. Almost seems like a cache issue
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Fri, Jan 14, 2022 at 6:16 AM Zoinkwiz @.***> wrote:

Hey @aichele66 https://github.com/aichele66 could you please share your
runelite logs so I can see if there is an error being thrown? Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/Zoinkwiz/quest-helper/issues/668#issuecomment-1013108182,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMY4QQYTSKXZDCI2ECFQ2ALUWAO4FANCNFSM5LUXKQTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm noticing a similar issue, though not quite the same circumstances. When I have the quest helper for Rag and Bone Man II open, I notice a slight delay at the start of each game tick, regardless of where I am ingame--this can be confirmed by rotating the camera and seeing a stutter once a tick. Making it not the active quest helper (without disabling the plugin) stops this behavior.
I checked my logs and did not see any errors.

Following up on this, this same stutter seems to be present for other quests as well. I noticed it while doing A Kingdom Divided as well.

Follow up on the follow up, I did the quest Rag and Bone Man II yesterday. Whilst using the quest helper for it, I had constant frame drops, usually I run at stable 50(capped) w/o any issues what so ever, however, when I started the quest helper for the quest, it dropped down to 46-47 constantly jumping back n forth, also lower for a millisecond which was very noticeable. After a while doing it this way, it was almost like my PC got a bluescreen but not an actual bluescreen, I was in discord with couple of friends and it cut off mid-sentence when they were talking and hung up on that almost like an earrape constantly. Even when I turned off my PC I could hear it, now I know this might have been hardware issue but I got the feeling it was caused from the constant frame drops that somehow bricked my PC for a minute. The issue was fixed when I restarted my PC however, I went on the quest by using the website wiki instead cause it had me worried for a second.

Same here for Rag and Bone Man 2


Firefox/chrome kiosk allows restricted keyboard shortcuts

When running Firefox/chrome in a typical Linux de (tested on sway). Kiosk mode in these browsers do not allow shortcuts such as ctrl+t, ctrl+w, etc. On cage kiosk mode starts and the taskbar is hidden, however I"m able to start new tabs, and close chrome/Firefox via a shortcut
I even tried starting a chrome PWA which should be restricted even further. This pops up another full screen window over the existing one with the address bar visible

Hi @jordankoehn ,
How do you start Firefox in kiosk mode for Cage?

/use/bin/Firefox --kiosk https://google.com

I tried again and sway and you're right. Firefox/chrome/chromium all allow keyboard shortcuts when in kiosk mode. I found some related issues on chromium's GitHub. What's interesting is when running from windows all keyboard shortcuts are disabled in kiosk mode of these browsers.
Any suggestions on disabling these shortcuts in cage/wlroots? Xdg keymaps only allow changing individual keys


added dark & blue body style

Added bg color and font color to the body tag style

@itssamna thnx for contributing.