Dart isolate group create callback called with incorrect signature.

Caught by undefined behavior sanitizer in the shell_unittests harness:
../../third_party/dart/runtime/lib/isolate.cc:164:44: runtime error: call to function flutter::DartIsolate::DartIsolateGroupCreateCallback(char const*, char const*, char const*, char const*, Dart_IsolateFlags*, std::__1::shared_ptr<flutter::DartIsolate>*, char**) through pointer to incorrect function type '_Dart_Isolate *(*)(const char *, const char *, const char *, const char *, Dart_IsolateFlags *, void *, char **)'
dart_isolate.cc:647: note: flutter::DartIsolate::DartIsolateGroupCreateCallback(char const*, char const*, char const*, char const*, Dart_IsolateFlags*, std::__1::shared_ptr<flutter::DartIsolate>*, char**) defined here

Is this still happening?


Add a "manualControl" boolean field to policy rules.

When a rule cannot (or shouldn't) be verified by Magpie, we need to set a top-level value in it called manualControl and set the value to true.  This field should default to false if missing or explicitly set to false.
For any rule with manualControl enabled, the PolicyAnalyzerService must not attempt to execute SQL or Python code.  Instead log a statement that the rule will not be checked and move on to the next rule.
Set this value on all existing rules that are unable to be verified with Magpie.

Already Done.


Create and publish contracts on NPM package

To automate the compilation of contracts on the subgraph rpo, it is necessary to publish a npm package at each modification of the contracts, containing the last version of them.

@posix4e

@Charlie85270 agree!

Closed. See the #333 for further work


Italics are a little light

I love this theme - simple, without a lot of crazy, flashy features, but softened a little to be easier on the eyes than the default. But the font color for italics is a little hard for me to read -- not enough contrast with the background. It might be nice for that color to be a little darker.

Try out 1.1.0. This meets contrast checking up to AAA text. If this still isn't good, I might play with making the font a little thicker for italics, since I don't want the color too different from the other ranges of colors.


fix get job api slow response issue

Coverage decreased (-16.5%) to 17.536% when pulling b9603c80a1fc2d5f23042d188003f3ff61ddd38b on suiguoxin:rest-server-speed-fix into 5467091c462ad51b46133b9e866922e44601f5a4 on microsoft:master.

@Binyang2014  @debuggy who ever touched the code to review

@suiguoxin Can you provide some performance test results after this fix?

@suiguoxin Can you provide some performance test results after this fix?

I test with the same job. The time needed for convertTaskDetail function is decreased from 4~5s to 100-200ms, which is the same as in previous version.

The performance test result is recorded in https://github.com/microsoft/pai/issues/5012. @Binyang2014 @yqwang-ms


DC ChargingStation Settings

Hello together,
the server and the dashboard are working. I have connected a DC charging station with 350KW. The plug type is CCS type 2. 900 volts and 500 amps are the maximum possible.
Is there a possibility to change the configuration in the attached picture?

the server and the dashboard are working. I have connected a DC charging station with 350KW. The plug type is CCS type 2. 900 volts and 500 amps are the maximum possible.

As the DC Voltage/Amp are changing depending on the type of the battery (400V, 600V...) we cannot enter fixed DC values here.
For Smart Charging reason, we use here AC Power (Amps, Voltage) delivered by the grid to the charging station before the transformation to DC.
So if the voltage is 230, for your 350kW charging station, enter 507 A per phase (1521 A for 3 phases x 230 V = 349 830 W.)
This should do the trick.
Let me know.

have made the changes and the first loading tests look promising
Many thanks for the support

have made the changes and the first loading tests look promising Many thanks for the support

You welcome ;-)


[Feature] Checking the hand value at showdown

A function (or maybe a seperate package) to determine the winner of the round at the end.

implemented in commit 560dc25987bd9035701d4f189a9014697902f4de


Download Thunderbird header button is not really downloading anything

The Download Button in the header area redirects to the website, which is misleading and not correct.
Also, we currently don't have any easy indication on "how to go to the main website" from the blog.
I'd suggest changing that button to "Get Thunderbird". More generic, and a somewhat good entry point to go to the website.
@ryanleesipes, thoughts?

Agreed. I'll change it.

@Alecaddd Does this still need to be changed?


teller cashier issues #334 #432 #433 #434 #435

This is merged. Due to sync issue it is not reflecting here.


Autoform update with nested fields in Schema

Hello,
I searched the web for this issue but couldn't solve the problem. I have the following schema:
Schemas.Pages = new SimpleSchema({
name: {
type: String,
label: "Name"
},
url: {
type: String,
label: "Url"
},
model: {
type: String,
label: "Model"
},
brandId: {
type: String,
label: "Brand"
},
userId: {
type: String,
label: "User ID"
},
header: {
type: Object,
label: "Header"
},
'header.title': {
type: String,
label: "Title"
},
'header.headline': {
type: String,
label: "Headline"
},
'header.subheadline': {
type: String,
label: "Sub Headline"
},
'header.disclaimer': {
type: String,
label: "Disclaimer"
});
And the following autoform:
{{#autoForm collection="Pages" doc=this id="updatePageForm" type="update" autosave=true}}

{{> afQuickField name='name'}}
{{> afQuickField name='url'}}
{{> afQuickField name='brandId' type='select' options=brandOptions}}
{{> afQuickField name='header'}}

{{/autoForm}}
When I just have non-nested fields (like name) it works fine, but when I also try to update the header it doesn't save anything anymore. How can I solve that? Thanks!

+1 I am too facing this.


Update flow.py

The Switch class where conditions with identical repr outputs could lead to key collisions, causing an incorrect mapping between conditions and their corresponding branches. To resolve this, we've enhanced the uniqueness of the condition keys by appending the id of the condition object to its repr string.
Changes:

Modified the key generation logic in the Switch class to include both repr and id: python items = {f"{repr(k)}-{id(k)}": v for k, v in zip(self.conds, items)}

This ensures that each key in the items dictionary is unique, even if the repr of different conditions is the same.
Example:
from enum import Enum
from pydantic import BaseModel
import lazyllm
class ServiceType(str, Enum):
    order_query = " order_query"
    web_search = "web_search"
    text_summary = "text_summary"
    chat = "chat"

    @classmethod
    def _generate_is_methods(cls):
        for member in cls:
            method_name = f'is_{member.name}'
            setattr(cls, method_name, classmethod(lambda cls, value, member=member: value == member))

ServiceType._generate_is_methods()

switch=lazyllm.switch(ServiceType.is_order_query, lambda x: 2 * x,
               ServiceType.is_web_search, lambda x : x,
               ServiceType.is_text_summary, lambda x : '000',
               'default',lambda x : '00',
               judge_on_full_input=True)
switch("web_search")

Right Answer:
web_search

solved in https://github.com/LazyAGI/LazyLLM/pull/152


Allow user to customize the certificate of external S3 endpoint

When user is configuring external S3 endpoint as registry's storage, we need to make sure there're attributes in the manifest for configuring the cert or CA so that Harbor can use the storage service when it is serving HTTPs with self-signed certificate.

PRing: https://github.com/goharbor/harbor-operator/pull/373

PRing: https://github.com/goharbor/harbor-operator/pull/373

I think we need to enhance the imagechartsorage.S3 spec to support customized certs

I think we need to enhance the imagechartsorage.S3 spec to support customized certs

Supported now


Correction for raw affiliation Institut Curie, Université PSL, Sorbonne Université, CNRS UMR3244, Dynamics of Genetic Information, Paris, France

Correction needed for raw affiliation Institut Curie, Université PSL, Sorbonne Université, CNRS UMR3244, Dynamics of Genetic Information, Paris, France
raw_affiliation_name: Institut Curie, Université PSL, Sorbonne Université, CNRS UMR3244, Dynamics of Genetic Information, Paris, France
new_rors: 02feahw73;02en5vm52;04t0gwh46;013cjyk83;02q6fa122
previous_rors: 02feahw73;02en5vm52;04t0gwh46;013cjyk83
works_examples: W3122463677
contact: 894df03c8751c68f904dc7136076884c:227fb2f8b510bc99871e8f5078 @ sorbonne-universite.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Add unittest

I decide using shunit2 for unit test.
Test codes are commited into tests directory.

I decide that using kcov to code coverage of unit test. Commit 7bc84da3091204a8121b1eded29bd0d48e8885e5

coverage: 40.4%

lib/Package unittest is very difficult. It using filesystem access many time. And shunit2 cannot test to AWK script.
It needs more refactoring.


qr.test failed, phenix-1.13-2998, CentOS 6

Hello, I installed qrefine using the command
phenix.python modules/cctbx_project/libtbx/auto_build/bootstrap.py --builder=qrefine --nproc=8
in the existing phenix-1.13-2998 directory.  However when I ran qr.test the system wrote a series of errors (see file attached). The system is CentOS 6.6.
qrefine_test.log
Has anybody a clue how to proceed with this ? Thanks.

Hi. Please try running the qr.test command in an empty directory.

Yes, I created an empty directory, ran qr.test therein, and I got that result.

Are you redirecting the output to a file called log?
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Fri, Sep 7, 2018 at 4:44 AM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Yes, I created an empty directory, ran qr.test therein, and I got that
result.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-419414340, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPEUIqKj9oJhMlHknUwRCoBvvQUQ0ks5uYlwWgaJpZM4Wep7i
.

Yes, I did it on purpose for providing a log file here. I noticed that I ran qr.test the twice in order to get the log file so it complains the existing files. Here attached please find the new log file I copied from the standard output.
log.txt

Great!
So some of the errors are due to 1.13 so update to Phenix 1.14. Can you try
that and we'll see what is left to correct.
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Fri, Sep 7, 2018 at 10:14 AM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Yes, I did it on purpose for providing a log file here. I noticed that I
ran qr.test the twice in order to get the log file so it complains the
existing files. Here attached please find the new log file I copied from
the standard output.
log.txt https://github.com/qrefine/qrefine/files/2361899/log.txt
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-419506818, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPCsbxv2laoSn-3nrh6cPNTtElO_Wks5uYqmFgaJpZM4Wep7i
.

Hi,
thanks for the tip. I installed phenix-1.14.3247 with qrefine on CentOS6 and the log file of qr.test is attached.
qr.log
This time not all, but there are few steps failed.
WCK

OK. So this error
tst_07 FAILED java.lang.RuntimeException: Class org.wallerlab.yoink.Yoink
not found
can mean you don't have a current version of Java. I had to install a
recent version (1.8 or so) and set it as the current version
setenv JAVA_HOME /usr/libexec/java_home -v 1.8.0_144, x86_64
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Sat, Sep 8, 2018 at 10:57 AM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Hi,
thanks for the tip. I installed phenix-1.14.3247 with qrefine on CentOS6
and the log file of qr.test is attached.
qr.log https://github.com/qrefine/qrefine/files/2363571/qr.log
This time not all, but there are few steps failed.
WCK
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-419661321, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPBcZXd4rgg3lDi6SR1pWkJzLi5HJks5uZATxgaJpZM4Wep7i
.

Good, I updated java via yum install java-1.8.0-openjdk-devel and ran qr.test again. Indeed updating java resolved the test_07 FAILED and related issues. Now test 10 and test 13 are skipped (see qr.test.log).
I consider that qr.test is probably fine, so I proceeded to run qr.finalise --example, and it failed. See part 1 of qr.finilise.log. I am not sure if this is related to the test 10 and test 13 issue. If I just run qr.finalise then the program could at least be started (see part 2 of the log above) but of course nothing could be proceeded.
WCK

So, --example seems broken but you should be able to move forward. Do you
have a model you wish to refine?
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Sat, Sep 8, 2018 at 11:41 AM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Good, I updated java via yum install java-1.8.0-openjdk-devel and ran
qr.test again. Indeed updating java resolved the test_07 FAILED and
related issues. Now test 10 and test 13 are skipped (see qr.test.log
https://github.com/qrefine/qrefine/files/2363607/qr.test.log).
I consider that qr.test is probably fine, so I proceeded to run qr.finalise
--example, and it failed. See part 1 of qr.finilise.log
https://github.com/qrefine/qrefine/files/2363613/qr.finilise.log. I am
not sure if this is related to the test 10 and test 13 issue. If I just run
qr.finalise then the program could at least be started (see part 2 of the
log above) but of course nothing could be proceeded.
WCK
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-419664458, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPMf0CftqbJHtJeHpmNzkTCXhuHWdks5uZA9ngaJpZM4Wep7i
.

Yes, so I started qr.refine --help and the program printed the phenix eff file but was somehow interrupted (see qr.refine.log. I truncated the first few lines till the parallel section).
If I just continue with qr.refine input.mtz input.pdb, then the program said "MOPAC command not specified" and it couldn't continue (sorry I didn't try to find/ read further documentations but just wanted to see if something could happen by providing a data and a model to it like phenix.refine).
Another issue is, the aforementioned CentOS6 machine is my test workstation. Now I wanted to install qrefine on my CentOS7 server which is equipped with more cores (is qrefine fully parallised?). Unfortunately the standard installation couldn't proceed (qr.install.log).
WCK

It is parallel and queue able but you are best to use a normal installer
not the bootstrap for the main Phenix installation.
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Sat, Sep 8, 2018 at 12:48 PM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Yes, so I started qr.refine --help and the program printed the phenix eff
file but was somehow interrupted (see qr.refine.log
https://github.com/qrefine/qrefine/files/2363656/qr.refine.log. I
truncated the first few lines till the parallel section).
If I just continue with qr.refine input.mtz input.pdb, then the program
said "MOPAC command not specified" and it couldn't continue (sorry I didn't
try to find/ read further documentations but just wanted to see if
something could happen by providing a data and a model to it like
phenix.refine).
Another issue is, the aforementioned CentOS6 machine is my test
workstation. Now I wanted to install qrefine on my CentOS7 server which is
equipped with more cores (is qrefine fully parallised?). Unfortunately the
standard installation couldn't proceed (qr.install.log
https://github.com/qrefine/qrefine/files/2363666/qr.install.log).
WCK
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-419668368, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPNQ3yjTQsQj_NoQ6WGhopbIVluEtks5uZB78gaJpZM4Wep7i
.

That sounds interesting so it can work with openmp and sge. Now I am confused: I thought the way of qrefine installation are either bootstrap or git-clone. Where can I find more information about the qrefine installer? Thanks.
WCK

I recommend the normal installer for Phenix and the bootstrap install for
QRefine. Simpler and easier.
BTW, it uses Python multiprocessor not openmp.
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Sat, Sep 8, 2018 at 2:04 PM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

That sounds interesting so it can work with openmp and sge. Now I am
confused: I thought the way of qrefine installation are either bootstrap or
git-clone. Where can I find more information about the qrefine installer?
Thanks.
WCK
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-419672714, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPKn-5yk2JTw2gnCBxJBqDop60FwQks5uZDDwgaJpZM4Wep7i
.

Hi, I have updated phenix to 1.14-3260, and I can reproduce all the previous tests discussed in this thread. I am trying to run a qr.refine with a protein coordinate and x-ray data, and I got the following message (only the last few lines I think which is relevant).  Does that mean I must have one of the quantum engine installed so that I can use one of them?
Thanks
WCK

Initial r_work=0.1881 r_free=0.1864
Setting QM defaults
Setting QM method to PM7

restart info will be stored in /home/wekao/data5/171013_P13/x1_13keV/phenix_cml/pdb/input.rst.pickle

Start:
0 Rw: 0.1881 Rf: 0.1864 Rf-Rw: -0.0017 rmsd(b):  0.0159 rws:  1.000 n_fev: 0
Optimal weight search:
MOPAC command not specified
Sorry: Failed to get weight

Daer WCK,
Correct, you need to to have MOPAC installed. Then you can just create an environment variable called MOPAC_COMMAND to point to the location of your MOPAC installation.
Please send us an update when that next step works.
Cheers,
Mark

Dear Mark,
thanks for the hint. I installed MOPAC2016 for CentOS6 (the non-GPU version) and I could execute MOPAC with the supplied example dataset with success. However when I re-ran qr.refine, it did not complain the MOPAC command not specified but uttered
Optimal weight search:
[Errno 13] Permission denied
Sorry: Failed to get weight
I installed MOPAC in my personal directory (not the default /opt/mopac) but all the environmental variables have been adjusted accordingly (including LD_LIBRARY_PATH, MOPAC_LICENSE and MOPAC_COMMAND), so I think the permission denied issue is perhaps not relevant to MOPAC?
Best,
WCK

Set this env. var. to 1 (for example in tcsh)
setenv LIBTBX_DISABLE_TRACEBACKLIMIT 1
and rerun.
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Mon, Oct 8, 2018 at 8:43 AM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Dear Mark,
thanks for the hint. I installed MOPAC2016 for CentOS6 (the non-GPU
version) and I could execute MOPAC with the supplied example dataset with
success. However when I re-ran qr.refine, it did not complain the MOPAC
command not specified but uttered
Optimal weight search:
[Errno 13] Permission denied
Sorry: Failed to get weight
I installed MOPAC in my personal directory (not the default /opt/mopac)
but all the environmental variables have been adjusted accordingly
(including LD_LIBRARY_PATH, MOPAC_LICENSE and MOPAC_COMMAND), so I think
the permission denied issue is perhaps not relevant to MOPAC?
Best,
WCK
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-427883526, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPLcdPrqrQ62K_lo9GS7j1aUmJGeEks5ui3KrgaJpZM4Wep7i
.

Hello,
I updated by .bashrc (export LIBTBX_DISABLE_TRACEBACKLIMIT=1) and I got the following result
Optimal weight search:
[Errno 13] Permission denied
Traceback (most recent call last):
File "/home/programs/phenix-1.14-3260/build/../modules/qrefine/command_line/refine.py", line 123, in
run(args=sys.argv[1:], log=log)
File "/home/programs/phenix-1.14-3260/build/../modules/qrefine/command_line/refine.py", line 118, in run
log      = log)
File "/home/programs/phenix-1.14-3260/modules/qrefine/qr.py", line 438, in run
results               = results_manager)
File "/home/programs/phenix-1.14-3260/modules/qrefine/driver.py", line 305, in refine
raise Sorry("Failed to get weight")
Sorry: Failed to get weight

OK, I would remove it now. It looks like you are trying to write into a
directory that does not have the correct writing permission.
Cheers
Nigel

Nigel W. Moriarty
Building 33R0349, Molecular Biophysics and Integrated Bioimaging
Lawrence Berkeley National Laboratory
Berkeley, CA 94720-8235
Phone :<PHONE_NUMBER>     Email<EMAIL_ADDRESS>Fax   :<PHONE_NUMBER>       Web  : CCI.LBL.gov
On Mon, Oct 8, 2018 at 9:34 AM Wei-Chun Kao<EMAIL_ADDRESS>wrote:

Hello,
I updated by .bashrc (export LIBTBX_DISABLE_TRACEBACKLIMIT=1) and I got
the following result
Optimal weight search:
[Errno 13] Permission denied
Traceback (most recent call last):
File
"/home/programs/phenix-1.14-3260/build/../modules/qrefine/command_line/refine.py",
line 123, in
run(args=sys.argv[1:], log=log)
File
"/home/programs/phenix-1.14-3260/build/../modules/qrefine/command_line/refine.py",
line 118, in run
log = log)
File "/home/programs/phenix-1.14-3260/modules/qrefine/qr.py", line 438, in
run
results = results_manager)
File "/home/programs/phenix-1.14-3260/modules/qrefine/driver.py", line
305, in refine
raise Sorry("Failed to get weight")
Sorry: Failed to get weight
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/qrefine/qrefine/issues/18#issuecomment-427899989, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-IPKwyYy_-0BeD0qYDhEke9-Ry-x0Gks5ui36GgaJpZM4Wep7i
.

to remove (?), please advise how to proceed (reinstall qr refine after you have fixed something or?)
Best,
WCK


fix: worker rewrite api

#20.

I had the same problem too. I edited the config the way it's written in this PR and it worked instantly. Massive shoutout!
@stephenou you probably should merge this. Thanks


integrations: Add live kwarg to callbacks

Instead of assuming a new Live instance in callbacks, should it allow for DvcLiveCallback(live=myliveinstance)? This could better support the workflow mentioned in https://github.com/iterative/dvc.org/issues/2773.
For example:
from dvclive import Live
from dvclive.keras import DvcLiveCallback

live = Live("custom_path")
model.fit(callbacks=[DvcLiveCallback(live=live)])

This could better support the workflow mentioned in iterative/dvc.org#2773.

Not sure what workflow do you refer to 😅
What would be the benefits of that approach? or What are the limitations of the current one?

It's not high priority, and it's not intending to replace the current approach.
In iterative/dvc.org#2773, it was unclear whether to instantiate the Live instance before using the callback.
For example:
live = Live("custom_path")
model.fit(callbacks=[DvcLiveCallback()])

The outputs will be in dvclive instead of 'custom_path`, which might be unexpected. If they want to get the expected behavior, they will have to rewrite their code like:
model.fit(callbacks=[DvcLiveCallback(path="custom_path")])

An alternative could be to also allow:
live = Live("custom_path")
model.fit(callbacks=[DvcLiveCallback(live=live)])

If both live and other kwargs like path are provided, an error could be thrown. It just gives users another way to fix the problem in case it's not obvious to drop the instantiation.
I will close it for now. If anyone encounters confusion, we can revisit.


MML Mardown

Hi,
has anyone ever reconstructed the graph of the MML or a proof of a theorem in markdown to make it visible, for example, with Obsidian?
It seems to me, if I'm not mistaken, that when I use https://em1.cs.shinshu-u.ac.jp/emwiki/release, I encode in markdown ?
If this is the case, wouldn't it be possible to encode directly in markdown in the source code of the *.miz file, for example ?
Would it be possible, for example, to put an explanation in markdown for a line of a particular demonstration that the author would like to explain.
Moreover this line of explanation (but starting with a "::" comment of mizar), could be displayed or not (for example with a level 0: displayed in Formalized Mathematics, level 1: technical mathematical information, level 2: comment of the author, etc.)

Dear Mr. Kazuhisa Nakasho,
Thank you very much for your reply,
for your work and efforts in developing the new features.


Disabled state added with custom attr

Sometimes it's vital to have the button in the disabled state (enabled="false")

In this case the FAB should probably simple be hidden

agree with @almozavr

Merged! Thanks a lot.
P.S. I changed the default color for the disabled state the darker gray.


combining injob queue and model loading class

hey is there any way we can set up a ML web app API utilizing the injob queue and model loading instance such that that when a API call comes a job id is returned , this job id  initializes a model class in which a ML model is loaded into memory and utilized, now if another API call comes within this moment it utilizes the same initiated model rather than loading another model.

Hey @haiderasad could this question be redirected to our community Slack? I think it's better to have ideas discussion there and leave these Github Issues for bugs or other problems 🙂


Runs tests based on capabilities of the image

Description
Instead of running the tests based on labels provided in a PR, we can inspect the built image and check which architectures it supports and test based on that information. It effectively decouples the tests from the builds, and makes it more flexible / robust.
Checklist

[ ] Investigated and inspected CI test results
[ ] Updated documentation accordingly

Automated testing

[ ] Added unit tests
[ ] Added integration tests
[ ] Added regression tests

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why you did not do so. (Valid reasons include "CI is sufficient" or "No testable changes")
In addition to reviewing your code, reviewers must also review your testing instructions, and make sure they are sufficient.
For more details, ref the Confluence page about this section.

Why are P and Z not using identify-image-platforms.outputs.supported_archs instead of the multiarch label ?

A strange error occurred https://github.com/stackrox/collector/actions/runs/7613242274/job/20732912611?pr=1483#step:15:41
But once this is solved, I suggest to remove the run-multiarch-build to see whether the tests are skipped.


refactor(core): renamed BufferEncoder to TypedArrayEncoder

BufferEncoder is incorrectly named as it accepts Uint8Arrays (and buffers). The name TypedArrayEncoder is more fitting as it fits both versions.

The reason why it's called BufferEncoder is because the return is always Buffer (if decoding).

Is there actually something specific we need from the buffer? Most likely there is, but if not we might just want to work with uint8arrays.

Yeah okay, that makes sense. Thanks

Can you resolve conflicts @blu3beri?

Codecov Report

Merging #675 (886fd0c) into main (88ad790) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #675   +/-   ##
=======================================
  Coverage   85.94%   85.94%
=======================================
  Files         342      342
  Lines        7366     7366
  Branches     1139     1139
=======================================
  Hits         6331     6331
  Misses       1034     1034
  Partials        1        1

Impacted Files
Coverage Δ

packages/core/src/utils/JWE.ts
100.00% <ø> (ø)

packages/core/src/crypto/JwsService.ts
90.90% <100.00%> (ø)

...rc/decorators/signature/SignatureDecoratorUtils.ts
100.00% <100.00%> (ø)

packages/core/src/modules/dids/domain/Key.ts
100.00% <100.00%> (ø)

...e/src/modules/dids/methods/indy/IndyDidResolver.ts
100.00% <100.00%> (ø)

packages/core/src/utils/TypedArrayEncoder.ts
100.00% <100.00%> (ø)

packages/core/src/utils/attachment.ts
68.75% <100.00%> (ø)

packages/core/src/utils/did.ts
94.87% <100.00%> (ø)

packages/core/src/utils/index.ts
100.00% <100.00%> (ø)

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 88ad790...886fd0c. Read the comment docs.


Use treasure on bosses when triggering metal detector

If the boss is below 30% health, attempt to trigger Treasure in addition to Metal Detector.
Since treasure has a short cooldown (5s), it may trigger it multiple times on the same boss.
I also attempted to estimate the time till the boss will be defeated, to do a better triggering condition for bosses that may take longer than the 20 second effect time, but just calculating health / dps was usually not very correct.

The problem with bosses is that the number steadily rise as more players switch to the lane

Perhaps if you checked the dps in each lane combined (minus what the abilities contribute), then assume something like ~75-80% of that will be coming to the boss lane?

The boss's health meter was stalling frequently, so I don't know if network or server load were affecting it, or maybe it's just that the DPS really isn't high enough to kill bosses quickly without items, but I was getting extremely large numbers even when the boss levels were completed within a minute, and after plenty of time for people to switch lanes.
From some recent logs of boss levels:
g_Minigame.m_CurrentScene.m_rgLaneData[ g_Minigame.m_CurrentScene.m_rgPlayerData.current_lane ].friendly_dps / g_Minigame.m_CurrentScene.GetEnemy(g_Minigame.m_CurrentScene.m_rgPlayerData.current_lane, g_Minigame.m_CurrentScene.m_rgPlayerData.target).m_flDisplayedHP

Run 1
Run 2 (1:23)
Run 3 (0:33)

2.80
1.29
214939

6.33
176514
198838

5.49
176513
208256

10.30
199698
298684

0.83
150598
99427

433744
203636

0.37


Add some sane utilities for environment variable handling

env_var.?pp files are from the nnp-transformer

@diyessi Yes it could maintain a cache but that is beyond the scope of the PR. Here I am only trying to normalize access to environment variables which were previously read and interpreted in a very haphazard way. As an example some environment variables only checked for the variables existence to be true, even if the user set that variable to false. This leads to user confusion.
In the NNP transformer we have another file which creates a static cache of each environment variable used so the first time it is accessed it is read and all subsequent reads use the cached value, but as I said above, that is beyond the scope of this PR. I want to keep things small and focused. This PR is no worse in caching efficiency than the existing code it replaces.

Added [MLIR] to title to enable MLIR CI.


Creating a new work item board view doesn't show up

http://www.screencast.com/t/rMBJ3lChHqr

They do show up on refresh


Amplitude SessionId Support

Hi
Are we have any method to set Amplitude SessionId just like: https://segment.com/docs/connections/destinations/catalog/amplitude/#sessionid

Hi @wudaming1  thank you for your report, we start looking into this.

I have the same issue. The value is -1. Do you have any recommendations ?

Any update on this? I'd like to be able to track sessions in amplitude with segment and for that we need session_id as well.


Add update_with_instant method to App

Objective

Adds update_with_instant method from #6146

Solution

Allow App access to bevy_time crate by creating a new trait inside of bevy_time::Time with update method
Insert UpdateTime resource with desired value (only accessible in bevy_time crate)
Option<UpdateTime> is used in time_system, allowing users to set Time value before time.update() is called

Changelog

Add UpdateTime(Instant) resource
Update bevy_time::time_system to use optional UpdateTime resource
Add TimeUpdater trait with update_with_instant method
Update App to implement the new trait

Is it OK that the TimeReceiver channel interaction has precedence over the manual time setting?
If the answer is yes, then this all seems fine to me.

I think so. I wasn't 100% sure how important TimeReceiver was. I even thought about combining it with DesiredTime since they almost do the same thing, but I wasn't too sure.

Is it OK that the TimeReceiver channel interaction has precedence over the manual time setting?

Ping @hymm; do you have opinions on this?

I even thought about combining it with DesiredTime since they almost do the same thing, but I wasn't too sure.

Overall it feels weird to only update time once instead of taking over the machinery completely. Are there use cases for only updating the time once and then going back to using the Instant::now()? Seems like something that would be problematic with frames reporting zero time for multiple frames or an extra long frame time once.
I feel like a better approach would be to somehow not use the time_system and just do everything (including updating the Res) in the app.update_with_instant function.

Are there use cases for only updating the time once and then going back to using the Instant::now()? Seems like something that would be problematic with frames reporting zero time for multiple frames or an extra long frame time once.

When reproducing tricky bugs, you may want to immediately "fast forward" time to the recorded problem state, and then play through as normal. I don't think it's common though.

I feel like a better approach would be to somehow not use the time_system and just do everything (including updating the Res) in the app.update_with_instant function.

Agreed, this feels like the more coherent strategy overall.

I feel like a better approach would be to somehow not use the time_system and just do everything (including updating the Res) in the app.update_with_instant function.

I agree as well. I would've preferred to not do it the way I did but I didn't think it was possible. Given the information in the issue, time should only be changed after time_system but before everything else otherwise the desired time isn't going to be accurate. The only other thing I can think of is having an option to prevent the system from running.

Can we have a config resource that controls whether time is updated? This resource could even supply the "next time", and then get reset after that Instant is consumed.

Can we have a config resource that controls whether time is updated? This resource could even supply the "next time", and then get reset after that Instant is consumed.

Something like this?
pub struct TimeConfig {
    pub should_update: bool,
    pub next_time: Instant,
}

If so, would it be best to set next_time to Instant::now() or should it be wrapped in an Option? It might also be better if next_time holds a Duration instead assuming users are constantly adding that to Instant::now() to get the next time.

Yep, that's the basic strategy I had in mind :) should_update probably needs a clearer name, but I think this is a good path forward.

would it be best to set next_time to Instant::now()  or should it be wrapped in an Option?

I think just setting the next_time field directly should work fine.

It might also be better if next_time holds a Duration instead assuming users are constantly adding that to Instant::now() to get the next time

In my use case, the pattern is actually to set the exact time that was previously observed :) However when testing, you'll often want to increment it by a Duration. Maybe a
#[derive(Resource, Default)
pub enum TimeUpdateStrategy {
   #[default]
   Automatic,
   ManualInstant(Instant),
   ManualDuration(Duration),
}

If it's going to be a config resource, is there still a reason to have the update_with_instant method? It seems kind of redundant if you're just going to insert the desired instant/duration and then manually change the values anyways. I suppose it would be pretty annoying to constantly do:
if let Some(TimeUpdateStrategy::ManualInstant(mut instant)) = app.world.get_resource::<TimeUpdateStrategy>() {
    instant = prev_instant;
    app.update();
}

// or
app.insert_resource(TimeUpdateStrategy::ManualInstant(prev_instant));
app.update();

Maybe TimeUpdateStrategy should only have Automatic & Manual with no instant/duration values provided. Manual implies it's up to the user to grab the time resource and call time.update_with_instant(desired_time). app.update_with_instant could still be used to set a specific time and have update() called. If the UpdateStrategy isn't set to Manual, calling that method could either not do anything or would set the strategy to manual for a single frame.

If it's going to be a config resource, is there still a reason to have the update_with_instant method? It seems kind of redundant if you're just going to insert the desired instant/duration and then manually change the values anyways

I think we can cut it; the insert_resource strategy is very clear and idiomatic. Then we can point to the TimeUpdateStrategy resource (with three variants) from the App::update docs.

Can you update the PR description and title to reflect the new changes?

Might be worth adding the breaking change label to this PR as it changes the behavior of time.delta on startup.
Before it would be [0, some big number, correct] and this PR changes it to be [0, "approximately the time between the time_system and present_frame", correct].


PermissionError when trying to write to voltage MSRs

This great little program doesn't work in its current version on Ubuntu 18.04 and T480. It seems the script is unable to write voltage configuration.
I'm not interested in undervolt so I commented out the call to undervolt, so the rest of the script works fine.

I see this is mentioned in #6 , but perhaps some extra checks would avoid a crash.

I've just add some more checks as suggested.


proxy/router: fix panic & gofmt

Fixes #106
Take the same action as in server.go.
Took advantage to run gofmt in the project too.

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

LGTM, Thanks @jsign .


Noninteractive iOS debugger session

Description
Remove debugger detach capabilities, make noninteractive.
Related Issues
https://github.com/flutter/flutter/pull/66399
https://github.com/flutter/flutter/issues/68119
Reverts part of https://github.com/flutter/flutter/pull/68046

Let's see if this does anything. Merging before tests are complete YOLO


Run check on automatic transitions on state change

Currently  automatic transitions will be evaluated whenever you enter a state, send an event or set an expression property. However we also have a guard that checks if a certain state is active and people are regularly confused  about  why the guard checking an expression property will be evaluated whenever we change an expression property, but the guard checking for a state being active is not checked when a state change happens.
The state charts "specification" is somewhat vague on when exactly automatic transitions are checked:

If a machine is in a state with a guarded automatic transition, then that guard is checked as often as possible.  Being event driven, the guards are effectively only checked whenever an event has been processed, but also after other automatic transitions have fired, or other internal events (such as raised events) are fired

On one hand we have "as often as possible" which indicates we don't even need a trigger and should check them basically every frame,  on the other hand there are specific occasions given.
If we lean more towards the former, then any change to the state of a state chart (actual states and expression properties) and the sending of any event would trigger the evaluation of automatic transitions.

This change will not be backwards compatible, so this needs to have some section in the manual and changelog. I also need some examples of flows that are broken by this change.

This has proven to be extremely tricky. There are tons of edge cases where you end up in endless loops because any transition can trigger even more transitions. A particularly nasty example is:

Because A stays active all the time, it trigger a state enter/exit repeatedly which will again re-evaluate the transitions. A way to fix this would be to do it like this:

This way the DECIDE state would be left immediately after it has decided, thus breaking the loop because both transitions are no longer in play. But I really don't want the library to freeze if someone builds the problematic thing, especially since this thing is probably the first thing someone will come up with..


Optimized route images are no different and confusing

The two images shown in the optimizing route sample are identical and don't differentiate how the optimized route actually is displaying any different: https://docs.microsoft.com/en-us/azure/azure-maps/how-to-use-best-practices-for-routing#sample-query-6

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

ID: 5235f066-7e95-bca9-73b5-384cfb0b5437
Version Independent ID: 2c74c74b-ab15-a7ed-15d7-258c33729c85
Content: Best practices for Azure Maps Route Service
Content Source: articles/azure-maps/how-to-use-best-practices-for-routing.md
Service: azure-maps
GitHub Login: @anastasia-ms
Microsoft Alias: v-stharr

Updating with the correct image.

This is fixed and should be closed

#reassign stevemunk


Feature Request: Multiple Channels

Hello!
When I utilize 'DaVinci' in a different channel, the 'other bot' generates the content and only displays it in the designated channel specified in the Globals.py file. However, I'm curious if it's possible to expand the bot's functionality by enabling it to show the content in multiple permitted channels. Specifically, it would be ideal if the bot could display the generated content in the same channel where the /mj_imagine command was initiated. Would it be possible to implement this feature?

It was implemented in my last commit 286b26dcbe6d9cff48e4dc905e7f6b4239d7dea8. To enable this feature you will have to set USE_MESSAGED_CHANNEL Globals.py to True.
Note that upscaling images should be done in the same channel. If you use the bot with the variable set on true and then switch to false, you can't upscale images published in channels different from the one you chose. But this should not be a problem if you plan to always use the feature.
Make me know, after testing, if everything is ok.

Just wanted to let you know that everything has been running smoothly for the past couple of days. Thank you for your work on this - it's much appreciated!

 To enable this feature you will have to set USE_MESSAGED_CHANNEL Globals.py to True. Note that upscaling images should be done in the same channel. If you use the bot with the variable set on true and then switch to false, you can't upscale images published in channels different from the one you chose. But this should not be a problem if you plan to always use the feature. Make me know, after testing, if everything is ok.

Just wanted to let you know that everything has been running smoothly for the past couple of days. Thank you for your work on this - it's much appreciated!


Mocha is a required devDependency

As a peerDep of karma-mocha, npm2 used to install mocha automatically.
But that is no longer the case with npm3. peerDeps must be installed
manually. In this case, we need to specify mocha explicitly as a devDep.

Thanks for the contribution.


As a keptn user, I want support for projects in a number of GitHub organizations

Right now only one GitHub organization is supported. This should be extended to an arbitrary number of GitHub projects that hold an arbitrary number of projects that hold an arbitrary number of services. The keptn CLI needs to support switching between GitHub organizations.


Provide an easier way to spawn a single process server

Would be cool to have an easier way to spawn a single process server. ReactPHP makes something like that pretty easy. Could for example be useful if you use it only to accept webhooks instead of a full-fledged production HTTP server, such as in a simple chat bot.

Isn't this already possible by http://amphp.org/aerys/classes/bootstrapper?

@bwoebi Could you add an example into ./examples to listen for HTTP requests in a process without using ./bin?

This has been solved with initServer.


linger.ms equivalent

Is there a linger.ms and batch.size equivalent, that currently exists in Java, to handle batches?

There is no linger functionality. You can implement that using sendBatch, however. Wrap send in a function that aggregates the payloads over time, and when you've hit the limit, send them off using sendBatch.


Readme added

For someone who worked with boost for a while this may seam trivial but for new kids in town this is essential. It took me couple of days of hair pooling to accidentally stumble upon that wiki page.
Long story short: if this info was available on the repository it would've made my life so much easier.

Thinking about it, this should not be required at all. Complete rebuild should check if includes are populated and rebuild all headers automatically. I am moving it to issues.


Add metadata for GitHub Actions

https://help.github.com/en/actions/automating-your-workflow-with-github-actions/development-tools-for-github-actions#logging-commands

Implemented in v1.61: https://github.com/vlsi/vlsi-release-plugins/blob/master/plugins/gradle-extensions-plugin/README.md#test-output-formatting

Do you have a sample repository?
plugins {
  id("com.github.vlsi.gradle-extensions")
}

Should be enough


TC - Verify Channel Checkstop Through OS With Auto Reboot fails due to Error accessing sysfs scom directory

TC - Verify Channel Checkstop Through OS With Auto Reboot fails due to Error accessing sysfs scom directory while injecting checkstop through OS.
Full Name: | Test Ras.Verify Channel Checkstop Through OS With Auto Reboot
-- | --
Verify Channel Checkstop (MBS FIR REG INT PROTOCOL ERROR) through OS With Auto Reboot settings enabled.
Verify_Channel_Checkstop_Through_OS_With_Auto_Reboot
20170828 02:04:43.607 / 20170828 02:08:37.731 / 00:03:54.124
FAIL (critical)
'Error accessing sysfs scom directory: No such file or directory' should be empty.

@spotthur : Move this test case out of witherspoon as it is NA - https://github.com/openbmc/openbmc-test-automation/issues/911

Code in Review
https://gerrit.openbmc-project.xyz/#/c/6336


Add completion option to control insertion of parentheses

Environment

Elixir & Erlang versions (elixir --version):Erlang/OTP 25 [erts-13.1.5] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [jit:ns] [dtrace]
Elixir 1.14.3 (compiled with Erlang/OTP 25)

Elixir Language Server version:Started ElixirLS v0.13.0
ElixirLS built with elixir "1.12.3" on OTP "22"
Running on elixir "1.14.3 (compiled with Erlang/OTP 25)" on OTP "25"

Operating system: MacOS 11.7.3
Editor or IDE name (e.g. Emacs/VSCode): Emacs
Editor Plugin/LSP Client name and version: lsp-mode 20230221.1558

Current behavior
When inserting a completion candidate for a function, an opening parenthesis gets inserted after the function name.
Example: Typing IO.p gets completed to IO.puts( (note the open parenthesis at the end).
This is undesirable for me, since I use the smartparens package in Emacs, and I have smartparens-strict-mode turned on for all programming languages. smartparens-strict-mode means it is impossible for the user to create unbalanced parentheses, quotes, etc. by typing or deleting text.
Ironically, it means that in the case of the above example (IO.puts(), I am not able to insert the closing ), since smartparens detects that this would lead to unbalanced parentheses. Same holds for deleting the orphaned (.
Thus, with smartparens-strict-mode, the described completion behaviour leaves the code buffer in a deadlock state (not possible to insert or delete any further parentheses).
Expected behavior
It would be great if there were a server option to set from the client, which, when turned on, would make sure that any paired delimiters (parentheses, quotes, etc.) would only ever be inserted in matching pairs when completing code.
This new option could be off by default, so as to preserve the current behaviour, and make it a backwards compatible change.
The documentation for the new option might even choose to make a reference to the smartparens package for Emacs, and its smartparens-strict-mode.

Please send a PR

Closing this. Will reopen if there's a PR implementing that


Leverage cache in an action that only listens to deployment_status events

I see that deployment_status events dont have a ref so we can't install to the cache from this job.
How would I make it so I install dependencies from the push that triggered the deployment_status and then leverage those cached dependencies for the action relying on the deployment_status event?
For reference, I'm essentially trying to do this workflow:
https://glebbahmutov.com/blog/develop-preview-test / https://www.philgiese.com/post/e2e-with-vercel

Basically, how would I cache dependencies in this workflow:
https://github.com/kylemh/cypress-preview-url-issue

+1 I have the same need

https://github.com/cypress-io/github-action/issues/385

Having the same issue.

Why was this closed? You said yourself that it's a problem and will only be fixed if others want it fixed.

@kylemh this is by design currently. Hence closed the issue.

@aparna-ravindra I think that the issue isn't the closing of the thread but on how the situation was handled.
We can revisit if there is more ask from the community. -> close feels somewhat mute to the issue. I think that it is usually better to transform the issue into a discussion, given that is a design/feature decision that will need votes and ask from the community.
I checked and there is one already, https://github.com/actions/cache/discussions/516, which I "linked" to this issue. For the rest of the participants just go there and discuss/vote on the issue so we can get this prioritized.

In my experience, closing an issue means the issue will NEVER be resolved, the issue IS resolved, or the issue is no longer relevant.
Even if this specific practical use is dismissed, conceptually I don't understand why deployment_status would not have a scope? It can change as commits are added to a branch. It should have access to the ref.

Reopening the issue to continue the conversation.

I don't understand why deployment_status would not have a scope

deployment_scope can have a ref (scope) if the deployment was created on a branch or tag and the action will work if it does. It depends on how the deployment event was created. If the deployment event was created with commit SHA (and not a branch or tag), then there is no ref associated and the action will not support it.
Since ref (scope) is a mandatory property of a cache object as per current design, we will not be able to support events not associated with a ref. Hence this issue should not be considered as a bug. However, we could treat it as a feature request to support events with no ref.

I checked and there is one already, https://github.com/actions/cache/discussions/516

Thank you for following up on this discussion. Let's use the discussion to understand more and hear from the community. The issue is tagged in the discussion. We can use it to unify all discussions.
From the perspective of this issue, it might not be possible to take up this design change at the current time. With this understanding, I suggest to close this issue and continue in the discussion thread as a feature request.

However, we could treat it as a feature request to support events with no ref.

Let's do that! Now, is this issue in the right repo?

Now, is this issue in the right repo?

Yes 👍

Actually, my issue is that I think deployment_status should have a ref for every event. Vercel, Netlify, Render.io... many services will trigger deployment_status events from commits. If commit events for deployment_status had refs, there'd be no issue.

Hey @kylemh ,
The action is only aware of the event that triggered the job. Hence, looking at the event for deployment_status, it is not possible to deduce the source branch/tag.
The deployment_status event can have a ref if the event was created with it. This is controlled by the system that is triggering the event. Hence this is not in the scope of this action and there is little that can be done without the information in the event.

Right. Where do I post an issue requesting that this be changed. Even if it requires some sort of rearchitecting.

There is a similar discussion in this issue. If you find it similar to your usecase, you are welcome to add your views.

Ah. This looks basically like a dupe then.


Misalignment of rear-side printing using double-sided mode on Samsung ProXpress M3820ND

Set-up:
Linux Mint 19.1 Cinnamon 64-bit (also occurred when testing Linux Mint 21.1 Cinnamon 64-bit)
CUPS versions since May 2019 or earlier (currently CUPS 2.2.7)
Samsung ProXpress M3820ND
A4 paper, double-sided printing
Problem scenario:

When printing to A4 sheets in double-sided mode using CUPS without using the Samsung printer driver, the print output on the rear side of the sheet is displaced to the right by 10mm compared to that on the front side.
When printing the same way from the same platform after installing the Samsung Print Driver for Linux (V1.00.39_01:17, Sep 1, 2017) and configuring a print queue with this driver, the print output on the front and rear sides is correctly aligned.
This issue does not occur when printing to the same printer from Windows, Android and Chromebook clients -- so it appears to be CUPS-specific.

Further analysis:

I originally described this issue in this posting in 2019 in the Linux Mint forums.  The posting contains detailed information including the outputs from various diagnostic commands.  At that time, I worked around the issue (hence shown as "[SOLVED]") by installing the Samsung driver and using a CUPS print queue associated with this driver.
I re-visited this scenario in January 2023 whilst testing out the latest Linux Mint installation (Linux Mint 21.1).  The same issue occurred when using "driverless" printing for this printer under CUPS.  Once again, I could work around the issue by installing the Samsung printer driver and creating a print queue that adopted it.
My concern now is that CUPS is intending to drop support for printer drivers, so the workaround using the Samsung driver may become unusable in the near future.

Images:
Below is the front side of a print using CUPS without the Samsung printer driver.  The left margin measures 5mm on the A4 sheet:

Below is the rear side of a print using CUPS without the Samsung printer driver.  The left margin measures 15mm on the A4 sheet (i.e. output shifted right by 10mm):

Below is the front side of a print using a CUPS print queue associated with the installed Samsung printer driver.  The left margin measures 5mm on the A4 sheet as before:

Below is the rear side of a print using a CUPS print queue associated with the installed Samsung printer driver.  The left margin correctly measures 5mm on the A4 sheet (i.e. aligned with the front side):

@Chapmip is the problem printer specific means only for Samsung ProXpress M3820ND printer ?

@sourabh1952 I'm unable to answer that question as I only have access to this one model of Samsung printer.
I imagine that some other Samsung printers may use the same print engine and would experience the same problem, but I simply don't know.
I therefore raised the issue for this specific Samsung model, in the hope that the issue may be readily reproducible on this and perhaps other Samsung models.  The Samsung driver that solves the problem (see link above) appears to be generic (i.e. not targeted at this specific model of printer).

@Chapmip can you please share your error log in debug mode and share your ppd file in .txt format.

I'm not familiar with CUPS, so I'm hoping that I've taken the right steps (see attachments) to secure an debug error log for a double-sided A4 print with the Samsung printer.  I carried these out using Linux Mint 21.1 Cinnamon (64-bit) edition.   I've attached both my terminal command input and the error log output here as separate text files.
When you say "your ppd file", I assume that you mean the PPD file from the Samsung Print Driver for Linux (V1.00.39_01:17, Sep 1, 2017) download -- this is the one that produces correctly aligned output when installed.  I've attached that file here as well.
Debug Error Log:
error_log.txt
terminal_commands.txt
PPD File from Samsung Driver for Linux:
Samsung_M332x_382x_402x_Series.ppd.txt
I hope that's helpful!

Moving this over to the cups-filters project since CUPS itself isn't responsible for generating the raster data for these printers.


replace transport status path with cloudevents

Signed-off-by: myan<EMAIL_ADDRESS>We can set the annotation value of MGH CR mgh-cloudevents to decide whether to use the cloudevents or not, the default value is "true" which means enabled the cloudevents.

/test test-unit

I would like to expose transport_format in API. the values can be message and cloudevents. the default is cloudevents. what do you think?

I would like to expose transport_format in API. the values can be message and cloudevents. the default is cloudevents. what do you think?

That is a grea

mgh-cloudevents

Good idea! we can expose this option to the customer. I will update the API like this:
apiVersion: operator.open-cluster-management.io/v1alpha2
kind: MulticlusterGlobalHub
metadata:
  annotations:
    mgh-image-overrides-cm: mgh-images-config
  name: multiclusterglobalhub
spec:
  dataLayer:
    type: largeScale
    largeScale:
      kafka:
        name: ${TRANSPORT_SECRET_NAME}
        cloudevents: "true" # or false
      postgres:
        name: ${STORAGE_SECRET_NAME}

/cc @clyang82 @morvencao

I think @clyang82 is suggesting to change API to something like this:
apiVersion: operator.open-cluster-management.io/v1alpha2
kind: MulticlusterGlobalHub
metadata:
  annotations:
    mgh-image-overrides-cm: mgh-images-config
  name: multiclusterglobalhub
spec:
  dataLayer:
    type: largeScale
    largeScale:
      kafka:
        name: ${TRANSPORT_SECRET_NAME}
        messageType: kafkaMessage # or cloudEvents
      postgres:
        name: ${STORAGE_SECRET_NAME}

If we have messageType field, it is easy to add a new message type. If we use cloudevents(true or false) knob, the API change will be hard in the future.

@morvencao I agree with you that the messageType(string) field might be flexible to add another value, but for the Kafka transport, it seems more readable to add a boolean value to let the customer decide to use cloudevent or not. Because in terms of Kafka scenarios, I don't think there are other message types to consider yet. Anyway, thanks for the suggestion, let's discuss tomorrow which way is more suitable for our scenario.

I think @clyang82 is suggesting to change API to something like this:
apiVersion: operator.open-cluster-management.io/v1alpha2
kind: MulticlusterGlobalHub
metadata:
  annotations:
    mgh-image-overrides-cm: mgh-images-config
  name: multiclusterglobalhub
spec:
  dataLayer:
    type: largeScale
    largeScale:
      kafka:
        name: ${TRANSPORT_SECRET_NAME}
        messageType: kafkaMessage # or cloudEvents
      postgres:
        name: ${STORAGE_SECRET_NAME}

If we have messageType field, it is easy to add a new message type. If we use cloudevents(true or false) knob, the API change will be hard in the future.

@clyang82 @morvencao Just as we discussed. I updated the MGH API to this:
apiVersion: operator.open-cluster-management.io/v1alpha2
kind: MulticlusterGlobalHub
metadata:
  annotations:
    mgh-image-overrides-cm: mgh-images-config
  name: multiclusterglobalhub
spec:
  dataLayer:
    type: largeScale
    largeScale:
      kafka:
        name: ${TRANSPORT_SECRET_NAME}
        format: kafkaMessage # or cloudEvents
      postgres:
        name: ${STORAGE_SECRET_NAME}

apiVersion: operator.open-cluster-management.io/v1alpha2
kind: MulticlusterGlobalHub
metadata:
  annotations:
    mgh-image-overrides-cm: mgh-images-config
  name: multiclusterglobalhub
spec:
  dataLayer:
    type: largeScale
    largeScale:
      kafka:
        name: ${TRANSPORT_SECRET_NAME}
        transportFormat: message # or cloudEvents
      postgres:
        name: ${STORAGE_SECRET_NAME}

apiVersion: operator.open-cluster-management.io/v1alpha2
kind: MulticlusterGlobalHub
metadata:
  annotations:
    mgh-image-overrides-cm: mgh-images-config
  name: multiclusterglobalhub
spec:
  dataLayer:
    type: largeScale
    largeScale:
      kafka:
        name: ${TRANSPORT_SECRET_NAME}
        transportFormat: message # or cloudEvents
      postgres:
        name: ${STORAGE_SECRET_NAME}

@clyang82 Done!


What can and can't I do with Bluetooth?

Hi,
I successfully integrated Xiaomi Gateway 3 (ZNDMWG03LM) with Home Assistant, updated firmware, ncp Zigbee and ncp Bluetooth.
ZHA works and my Zigbee devices are recognized and functional.
But what about my Bluetooth devices?
I have 4 Mijia thermometers (LYWSD03MMC) which were associated with the Xiaomi gateway. I updated the firmware of one of the thermometers with pvvx/ATC_MiThermometer. The thermometer works, but I can no longer recognize it from the gateway with "Xiaomi Home". It is seen from time to time in Home Assistant, but the sensor values do not appear.
What should I do to make this thermometer work with pvvx firmware in Home Assistant via Xiaomi gateway?
By the way, I have a Bluetti EB3A power station with Bluetooth connectivity. Is it possible for Home Assistant to communicate with this device using the Xiaomi gateway? It also does not appear in the list of Bluetooth devices...
Thank you very much for your work

@RedFox-Fr this gateway only supports xiaomi-approved ble devices. If you flashed your thermometers with atc fw they start to advertise (by default) using another protocol, also flashing usually changes device name.
If you want these to work with gateway you have to retain their original names (not ATC_XXX etc), mi-like protocol and original ble key.

OK.
I was unable to attach the thermometer to the gateway. But after reflashing the thermometer with original firmware, it works.
Anyway, I need to find a good Bluetooth solution if I want to talk to any devices...
Thanks.


A small problem

How to solve this problem please? Thank you.

Disable your anti-virus, or try moving the injector and the R3nzSkin.dll to a different folder.


Fix boost version in cmake pkg config and fix multi config check

The boost version in the cmake package config template is updated to 1.71.
GENERATOR_IS_MULTI_CONFIG isn't a variable it's a global property, fixed the if condition to use it correctly.
Boost::headers is replaced with Boost::boost. Because the headers target was added in a newer CMake version and it doesn't exists in 3.10.
find_dependency macro in the package config file automatically infers the REQUIRED and QUIET keywords from the calling find_package command. So, the REQUIRED parameter is removed from the config.cmake.in file.

Fixes #752. Well the issue said not to repeat the version but on second thought, it's not something that would change that frequently and I guess an amount of repetition between the main CMakeLists.txt and the config.cmake.in is inevitable. We have to be careful when we update our package dependencies and keep the two files in sync.

Windows test FAILed.

Windows test FAILed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Windows test FAILed.

Windows test FAILed.

Linux test FAILed.

Linux test FAILed.

Windows test FAILed.

Windows test FAILed.

verify-windows

verify-windows

Linux test PASSed.

Linux test PASSed.

Windows test PASSed.

Windows test PASSed.


⚠ Fortnite has degraded performance

In 9526785, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1600 ms

Resolved: Fortnite performance has improved in 07cc9dc.


Allow Models-config to change (configReload) while deployments are in progress

Added ModelConfig model, DB table, etc to keep track of old configs.

Added checksum calculations to keep track of each configReload
iteration. This calculation reads all the directories and files
in a deterministic order and computes checksum, which should remain
same for given config on any platform.

The checksum is stored in the DeployDB app object as well Flow object
in order to associate them with the given iteration of config.

On the subsequent Flow/Deployment transitions, if checksum from the
Flow matches with that in the App object, then in-memory (i.e. registry)
Model objects are used. Otherwise Model objects are read from the
ModelConfig objects.

In loadConfigModels, moved code repetitions such as loading from
string, inserting into registry, checksum calculations, etc.
to common function

Added cucumber and spock tests to validate the code.

References #126

@GiriDandu Please review.

Hi @mbbx6spp, could you please review the schema changes in this commit

Looks fine to me but needs to be synced with master looks like

:+1:


refactor(display): various minor refactoring

use if let syntax
replace basic matches! macro with ==
use format! macro to create a String

TODO

[x] Use cargo fmt
[ ] Add necessary tests
[ ] Add changelog entry
[ ] Update default config/theme in README (if applicable)
[ ] Update man page at lsd/doc/lsd.md (if applicable)

Codecov Report

Merging #683 (b80299a) into master (997dd9f) will increase coverage by 0.06%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master     #683      +/-   ##
==========================================
+ Coverage   88.35%   88.41%   +0.06%
==========================================
  Files          41       41
  Lines        4602     4592      -10
==========================================
- Hits         4066     4060       -6
+ Misses        536      532       -4

Impacted Files
Coverage Δ

src/display.rs
92.03% <77.77%> (+0.98%)
:arrow_up:

Continue to review full report at Codecov.

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

Woops, accidentally closed the PR. Merged!


How should we minimize/eliminate negative impacts to projects that opt out?

Splitting this out from #9 to make sure we don't lose it:

One thing that I already wrote on Twitter: it's perfectly fine for Microsoft to come up with a set of requirements that they give to the community and say: "look those are the things you need to do so that we can put your product into our stack". That would be a great thing if done by MS.
Putting the ladder into the foundation on the other hand makes it a thing that will spread to the whole community and will put pressure on projects that will never need to go into MS stack. People in companies will just blindly adopt it and now projects are called "untrustworthy" just because they don't align with a set of requirements that MS needs internally. That's not good

The readme does say we have a general goal of improving overall ecosystem quality, and we did anticipate some positive network effects by allowing L4 projects to take long term, trusted dependencies on other L4 projects.
However, we want this to be opt-in and to avoid or absolutely minimize any negative impacts to projects for which this is not a good feedback, for whatever reason: project unique technical constraints, wrong fit for how the project team works, or whatever - maybe it just feels like a poor tradeoff of extra process that's not worth it.
Do you have suggestions on how we should alter policies or program communications balance this better? We'd like to keep the positive ecosystem impacts for projects that opt in, and eliminate or drastically minimize the impacts for projects that don't want to or can't participate.

A related aspect is that it's not really opt-in vs. opt-out, it's that every rung on the ladder needs to be considered a valid end point for a project with no pressure. Nobody should be shamed for being "stuck".

@terrajobst Then the names of the levels really need to change, eh?


Make Docker supervisord not run programs as root so it plays nice with Kubernetes

Users trying to make the UmpleOnline docker image run with Kubernetes have been experiencing problems because umpleonline/docker_config/supervisord.conf specifies user=root
It needs to run as a user other than root.
The solution might be to simply delete that line, or change to user=www. It might also be necessary to change to group=www too
Additional changes may be needed to umpleonline/docker_config/nginx.conf and/or php-fpm.conf
A change to Dockerfile-base might be needed to add a user (possibly)
Some documentation to help with this can be found at the following sites:
Supervisord configuration: http://supervisord.org/configuration.html
Suggestion to use www or possibly webuser or possibly www-date: https://serverfault.com/questions/433265/how-do-i-change-the-nginx-user
Current supervisord log says:

2020-07-28 16:38:23,642 CRIT Can't drop privilege as nonroot user

2020-07-28 16:38:23,644 INFO supervisord started with pid 8

2020-07-28 16:38:24,648 INFO spawned: 'nginx' with pid 11

2020-07-28 16:38:24,652 INFO spawned: 'php-fpm' with pid 12

2020-07-28 16:38:24,660 DEBG 'nginx' stderr output:

2020/07/28 16:38:24 [warn] 11#11: the "user" directive makes sense only if the master process runs with super-user privileges, ignored in /etc/nginx/nginx.conf:2

nginx: [warn] the "user" directive makes sense only if the master process runs with super-user privileges, ignored in /etc/nginx/nginx.conf:2

2020-07-28 16:38:24,662 DEBG fd 9 closed, stopped monitoring <POutputDispatcher at<PHONE_NUMBER>79528 for <Subprocess at<PHONE_NUMBER>78880 with name php-fpm in state STARTING> (stdout)>

2020-07-28 16:38:24,662 DEBG 'nginx' stderr output:

2020/07/28 16:38:24 [emerg] 11#11: bind() to <IP_ADDRESS>:80 failed (13: Permission denied)

nginx: [emerg] bind() to <IP_ADDRESS>:80 failed (13: Permission denied)

2020-07-28 16:38:24,662 DEBG fd 8 closed, stopped monitoring <POutputDispatcher at<PHONE_NUMBER>78952 for <Subprocess at<PHONE_NUMBER>78736 with name nginx in state STARTING> (stderr)>

2020-07-28 16:38:24,662 DEBG fd 6 closed, stopped monitoring <POutputDispatcher at<PHONE_NUMBER>18880 for <Subprocess at<PHONE_NUMBER>78736 with name nginx in state STARTING> (stdout)>

2020-07-28 16:38:24,663 INFO exited: nginx (exit status 1; not expected)

2020-07-28 16:38:24,663 DEBG received SIGCHLD indicating a child quit

2020-07-28 16:38:24,687 DEBG 'php-fpm' stderr output:

[28-Jul-2020 16:38:24] NOTICE: [pool umple] 'user' directive is ignored when FPM is not running as root

2020-07-28 16:38:24,688 DEBG 'php-fpm' stderr output:

[28-Jul-2020 16:38:24] NOTICE: [pool www] 'user' directive is ignored when FPM is not running as root

[28-Jul-2020 16:38:24] NOTICE: [pool www] 'group' directive is ignored when FPM is not running as root

2020-07-28 16:38:24,688 DEBG 'php-fpm' stderr output:

[28-Jul-2020 16:38:24] ERROR: unable to bind listening socket for address '/var/run/php-fpm.sock': Permission denied (13)

[28-Jul-2020 16:38:24] ERROR: FPM initialization failed

2020-07-28 16:38:24,688 DEBG fd 14 closed, stopped monitoring <POutputDispatcher at<PHONE_NUMBER>79240 for <Subprocess at<PHONE_NUMBER>78880 with name php-fpm in state STARTING> (stderr)>

2020-07-28 16:38:24,688 INFO exited: php-fpm (exit status 78; not expected)

2020-07-28 16:38:24,689 DEBG received SIGCHLD indicating a child quit
...

This may require some experimentation. A new Docker image will be created for the branch that is created for this change, so test that image.

Closed in pull request #1616


fix: updat source attribute color

:tada: This PR is included in version 1.2.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove bundledDependencies

We've had too many issues related to how the buggy npm client treats bundledDependencies.
Since LTS Node is now shipped with npm@3, we should remove bundledDependencies, bump npm version in engines field, and mention that we are npm@3+ in README.

Is there anyone working on this? If not, I think I can do. Is this similar with this one https://github.com/kitze/create-react-app/commit/cdaa8923f181b7203b5f332fa5992c560884c57a

@dvkndn 👍

Ok I'll make a PR for this one. All credit to @kitze

@dvkndn finally. I'm not sure if that was causing this issue though, some people still have problems on specific verisons of npm 😥

actually @fson worked on this already :D https://github.com/facebookincubator/create-react-app/pull/1068

Oh yeah, sorry for not commenting on this issue. I needed this for fixing the ESLint integration and forgot that there was a separate issue about this.

Don't worry, I haven't started yet 😄

Fixed in #1068.


Corrected bug in Python code, wrong variable being used.

Corrected variable usage.

Can one of the admins verify this patch?

I signed it!

LGTM @vrv

@tensorflow-jenkins: test this please

Merged. Thanks!


Session object is nil while testing with ganache.

I'm testing the Foo contract with Ganache -cli. For some reason, the session object is always nil.

Thank you for your report!
Sadly, Perigord is currently un-maintained. Closing this ticket so as to avoid cluttering various Projects.
SHOULD WE RESUME MAINTENANCE, RE-OPEN THIS TICKET.


[PIP 95] Smart Listener Selection with Multiple Bind Addresses

Status: Under Discussion
Author: Eron Wright
Pull Request:
Mailing List discussion:
Release:

Motivation
The Pulsar broker has the concept of advertised listeners representing broker endpoints that are discoverable by, and accessible to, Pulsar clients.  In addition, one listener may be designated as the internal listener, to be used for broker-to-broker communication.  Listeners may be understood as named routes to the broker.
Each broker stores its listener data into ZooKeeper to support topic lookup requests.  A lookup request may contain a listener name parameter to obtain a specific endpoint.  For example, a lookup request for the listener named external may return an endpoint of pulsar+ssl://broker-1.cluster.example.dev:6651 (an external address). If a listener name is not specified, the response contains the endpoint for the internal listener.
This PIP seeks to improve the experience when a listener name is not specified, to select an appropriate endpoint automatically without requiring explicit configuration on the client.  The proposed approach is to use information that the client naturally has, that is the Pulsar service endpoint.  The broker shall select a listener based on which service endpoint was used to make the lookup request.  For example, a client that makes a lookup request via an ingress gateway would use that same gateway for the subsequent broker connection.
A secondary goal is to improve the interoperability of KOP by implementing smart endpoint selection for ordinary Kafka clients.  This shall be based on the semantics of KIP-103 and is consistent with this proposal.  Similarly for other protocol handlers.
Public Interfaces
Configuration: Bind Addresses
It is proposed that a new configuration property be defined to specify a map of listener name to server bind address.  The broker shall bind a server socket for each address, and use the associated listener name (by default) for any lookup request that comes through that address.  The property shall be named bindAddresses and have the form:
<listener_name>:<scheme>://<host>:<port>,[...]

Where listener_name is a configured listener via the advertisedListeners configuration property.
Where scheme selects a protocol handler, e.g. pulsar or pulsar+ssl for the Pulsar protocol.
Where host has a local interface address to bind to, such as <IP_ADDRESS> to bind to all interfaces.
Where port is a unique server port number, such as 6682.
See the compatibility section for further information.
Admin API: Lookup Request: Listener Name Header
It is proposed that a new header-based parameter be added to the Topic Lookup operation on the Admin API (/v2/topic/{topic-domain}/{tenant}/{namespace}/{topic}).  The header name shall be X-Pulsar-ListenerName and the value shall correspond to the name of a configured listener.
The operation shall return 400 Bad Request if the listener name doesn't match the configuration.
Proposed Changes
The main technical challenge is to know which advertised listener a lookup request came from, and the proposed solution is two-fold:

for the Pulsar binary protocol, use a unique bind address for each listener.
for the Pulsar HTTP-based admin protocol, use an HTTP header to specify a listener name.

Multiple Bind Addresses
The broker shall be configurable to open a server socket on numerous bind addresses, and to associate each socket with a listener name.  When a lookup request arrives on a particular socket, the broker shall use the associated listener by default.  Note that an explicit listener name in the request parameters would take precedence.
For example, port 6682 may be associated with the external listener to establish the default response for a lookup request received on that port.
The scheme information shall be used to determine whether to terminate TLS on the server socket.  Note that all TLS sockets would use the same TLS configuration (a per-address configuration is out-of-scope).
Listener Name Header
The broker shall look for the X-Pulsar-ListenerName header as an additional parameter to the Topic Lookup operation.  The value shall be used when a listener name is not supplied via the similar listenerName query parameter.
The rationale for supporting a header-based alternative to the query parameter is to allow an HTTP-aware gateway to inject the header value as a hint for requests that comes thru the gateway.  Istio, for example, is able to inject headers but is not able to set query parameters (docs).
Relax Validation of Listener Addresses
The validation logic for advertised listeners shall be relaxed to allow certain configurations that are currently rejected.  The root issue is that the broker erroneously assumes that TLS is always terminated at the broker, as opposed to at the gateway.  This precludes a configuration where a gateway terminates TLS and then forwards the request as plaintext.
TLS-Only Configurations
The validation and lookup logic shall be updated to allow for pure TLS configurations, that is where an advertised listener consists only of a TLS endpoint (no plaintext endpoint).  Currently, the broker insists that all listeners have a plaintext endpoint.  This makes little sense, e.g. for an Internet-facing gateway endpoint where TLS is a hard requirement.
The behavior of the Pulsar client is to use the same scheme for the service endpoint and for the broker endpoint.  For example, if a client uses pulsar+ssl to perform a lookup request, it then uses pulsar+ssl to connect to the broker to produce messages.  No change is anticipated.
Compatibility, Deprecation, and Migration Plan
For compatibility, the primary bind addresses shall not be associated with any particular listener.  The behavior of lookup requests to those addresses shall be unchanged.  The primary bind addresses are configured using bindAddress, brokerServicePort, and brokerServicePortTls as normal.
It is assumed that any bind addresses that are configured with bindAddresses are in addition to the primary bind addresses and do not conflict.  If one wishes to purely use bindAddresses, one must unset the brokerServicePort property (note that it has a default value of 6650).
Test Plan
Unit Tests
New unit tests will test the parsing and validation logic for the bindAddresses configuration property.
Adhoc Tests
Pulsar will be tested on an adhoc basis in various combinations, primarily using Pulsar standalone mode.  Specific cases:

migration scenarios involving bindAddress, brokerServicePort, brokerServicePortTls, webServicePort, and webServicePortTls.
TLS and/or non-TLS channels
end-to-end testing with multiple listeners

Rejected Alternatives
None.

A secondary goal is to improve the interoperability of KOP by implementing smart endpoint selection for ordinary Kafka clients. This shall be based on the semantics of KIP-103 and is consistent with this proposal.

If we only add multiple bind addresses of pulsar binary port in broker, I think we could not make smart endpoint selection for ordinary Kafka clients, we need to add multiple bind addresses of Kafka port in KoP.
I already have a desgin to add multiple bind addresses in KoP and working on it. https://github.com/streamnative/kop/issues/669

@wangjialing218 thanks for sharing, I believe our efforts are complimentary.  It would be reasonable to use the new bindAddresses property to define KOP ports; each protocol handler would filter on the scheme component.  For example, the Pulsar broker service would act on the bindings for pulsar and pulsar+ssl.

@EronWright can you please start a discussion on<EMAIL_ADDRESS>mailing list ?
this is required in order to accept this PIP

btw the PIP looks good to me.
I have no comments to make.

Please take a look at the linked PR. My expectation is that the KOP handler obtain its server ports using:
List<BindAddress> bindAddresses = BindAddressValidator.validateBindAddresses(serviceConfig,
                Arrays.asList("kafka", "kafka+ssl"));

@EronWright
Let me know if I understand correctly, The scope of this PIP is only for Pulsar bind address.
KOP bind kafka address through KafkaChannelInitializer when ProtocolHandler.newChannelInitializers() is called.


Assume in loops and array type

The following code seems to work fine:
alg = HMC(1000, 0.2, 4)

@model vdemo() = begin
  x = Vector{Real}(undef, N)
  for i = 1:N
    x[i] ~ Normal(0, sqrt.(4))
  end
end

t_loop = @elapsed res = sample(vdemo(), alg)

However, changing Real to Float64 seems to make it break.
alg = HMC(1000, 0.2, 4)

@model vdemo() = begin
  x = Vector{Float64}(undef, N)
  for i = 1:N
    x[i] ~ Normal(0, sqrt.(4))
  end
end

t_loop = @elapsed res = sample(vdemo(), alg)

ERROR: MethodError: no method matching Float64(::Flux.Tracker.TrackedReal{Float64})
There seems to be an easy fix:
Float64(f::Flux.Tracker.TrackedReal{Float64}) = f.data

A few thoughts/questions:

Is there a reason we use an abstract type Real instead of, e.g., Float64?
Are we interested in adding the above shim?
Why the broadcast in the sqrt call?

Is there a reason we use an abstract type Real instead of, e.g., Float64?

This is required to make AD work. Flux.Tracked is a subtype of Real.

Are we interested in adding the above shim?

I'm not sure cause it might cause incorrect gradients.

@xukai92 might be able to elaborate on this.

I agree with @yebai on the first two answers. + I think doing Float64(f::Flux.Tracker.TrackedReal{Float64}) = f.data is against type piracy (https://docs.julialang.org/en/v1.0.0/manual/style-guide/#Avoid-type-piracy-1)?

Why the broadcast in the sqrt call?

I don't think it's necessary to use broadcast here.

Adding
Float64(f::Flux.Tracker.TrackedReal{Float64}) = f.data

will almost certainly cause AD to fail silently, so we should definitely not do this.

If the above sqrt broadcasting is something that is in one of our examples, then @hessammehr it would be good if you could make a PR to fix it. Otherwise we should close this issue.

@willtebbutt Will do.
@willtebbutt @xukai92 If you will indulge me, I was hoping to ask why the AD seems to handle Real just fine but chokes on Float64/Float32. Also wondering whether using an abstract type here has any performance implications.

Ah, of course, happy to explain. It results from the way AD is currently implemented in Julia generally, as opposed to being something fundamental to AD, and has strong negative performance implications.
Say we're computing the gradient of a function f(x), then (reverse-mode) AD fundamentally needs to keep track of every operation that happens during programme execution that involves x. The way people have typically gone about this in Julia is to add extra types that contain the metadata necessary to do tracking. For example, Flux's Tracker.TrackedReal is a subtype of Real and contains a value, which is itself a concrete subtype of Real, and some extra information needed to perform tracking. This design is chosen in anticipation of f having been written in a generic enough manner that any subtype of Real can pass through it. When you create the concretely-typed Vector{Float64}, you can't place any arbitrary Real in it, so Flux is unable to propagate the information that it needs to through it. (The story is similar for ForwardDiff). When we get a Cassette-based AD, this should change, but this is the situation for now unfortunately.
You're also correct that it has significant performance implications, because a Vector{Real} isn't type stable. There's not really another way to write this code at the minute unfortunately using Turing.

@willtebbutt Thank you very much for the explanation. I think I understand the situation a lot better now. The sqrt issue has been fixed, so I am going to close this issue.


No visible @interface for 'UIView' declares the selector 'stringByEvaluatingJavaScriptFromString:'

I'm getting an error when trying to build an iOS Project with this plugin.
AppPreferences.m:24:16: No visible @interface for 'UIView' declares the selector 'stringByEvaluatingJavaScriptFromString:'
cordova iOS 4.1.0

Seems like issue with new cordova iOS and WKWebView. I'll try to fix it on this weekend

Cool. Take a look at these other relates issues too: similar issue

Temporary solution found here helped me:
cordova platform remove ios
cordova platform add<EMAIL_ADDRESS>

I m not seeing any ui when I select preferences on iOS.. 9.3 same case?

fixed

cordova platform remove ios
cordova platform add<EMAIL_ADDRESS>


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1259 ms

Resolved: Bofrost.at performance has improved in bee4d73 after 26 minutes.


Can it run in Jetson Nano

We only tested this code on PCs. If you can run PyTorch with CUDA support on Jetson Nano, I believe it can run on it too.

Thanks, I tried on Jetson Nano,well done

@ZhaoZeng101 Have you successfully run it on jetson nano? I also tried but when I run a demo of bag.avi, after selecting ROI, the video just freeze and the whole system just stuck and I have to restart the nano.

I have run demo.py in Jetson Nano with siamrpn_alex_dwxcorr_otb, at 3fps while showing opencv window, or 5fps while hiding window.
IMX219-77 and 720p video file tested.

@mengqlTHU
The jetson nano has only 4G of memory, which is a bit of a headache when running large images. It is recommended to run large images, you need to compress the image before running.

@ZhaoZeng101 Thank you. I have found two things are important: First you have to use a power supply with larger output current.https://www.jetsonhacks.com/2019/04/10/jetson-nano-use-more-power/. Second you have to set up a swap file to use more memory. https://www.jetsonhacks.com/2019/04/14/jetson-nano-use-more-memory/

I am sorry to write in the closed topics, but have anybody tried converting the model to TensorRT? Does doing this give any significant acceleration to the inference process?


Add Ramda package to this repository

Create a .js file with name same as module name and write a commented code .
Create a documentaion in  Readme.md file and give brief information about the module .
Link to the package : https://www.npmjs.com/package/ramda

Done @Jai-Shree-Krishna


[CIRCLE-7415] Add permission to start and stop instances

This is required in order to use the reusable docker machine.

I committed a comma right after I merged this. We need this fix deployed.


implement three levels of logs

fix #63
Three levels of logs:
- verbose == logging.DEBUG: show all logs
- verbose == logging.INFO: show find and scored pipelines
- verbose == logging.WARNING: show no logs

Ede, please, merge it, thanks!


Update lab 3

I see the plot is gone, I think we should have the plot atleast there, what do you think?


SetupDiag link fixed

Removed .md from the link

@greg-lindsay @TrudyHa - can you please assist to check, approve and merge the content for this PR based on issue #2631?
Thank you.

@Malind19 If the link isn't working, the problem is something other than the file suffix. When we link within a project (as opposed to a URL), we must include the .md suffix. In this case, I think the issue is with the relative path. Rather than change it in your pull request, I'm going to fix the link on our side (quicker to publish that way). Thanks for pointing out that the link doesn't work.

@jdeckerms Ok thanks. When I removed the .md suffx it worked. Same happened with another issue.

@Malind19 : The problem seems to be that the migration process copying the Github version of the document (containing the link with the .md suffix) to the docs.microsoft.com site did not remove the .md suffix, even though it is supposed to do so. This error needs to be fixed by Microsoft internal technicians with access to the public pages, because the Github pages need the .md suffix.


Constructor not entirely suitable as created hook (has wrong this)

"vue": "^2.6.11",
"vue-class-store": "^2.0.5",

Experimenting with vue-class-store as a replacement for Vuex and loving it so far. However...
The following example does not work as expected. Setting local properties in the constructor does work, but calling methods that reference this will have a bad reference.
@VueStore
class Store {
  counter = 0
  constructor() {
    this.counter = 256 // this works
    this.startCounting() // will not work (method will reference wrong `this`)
  }

  private startCounting() {
    setInterval(() => this.counter++, 1000) // does not work if called from constructor (wrong `this.counter`)
  }
}

Workaround (somewhat awkward):
@VueStore
class Store {

  counter = 0

  // fired immediately by Vue with correct `this`
  private "on:any_random_name" = {
    immediate: true,
    handler: this.startCounting
  }

  private startCounting() {
    setInterval(() => this.counter++, 1000)
  }
}

Decorator workaround (still awkward):
// decorator function
function Creatable(target: Function) {
  const uuid = createUUID() // for example
  target.prototype[`on:${uuid}`] = {
    immediate: true,
    handler: "created"
  }
}

@VueStore
@Creatable // must be AFTER @VueStore -- awkward!
class Store {
  counter = 0
  private created() { // pseudo-created hook
    setInterval(() => this.counter++, 1000) // has correct `this`
  }
}

Maybe this can be fixed in a more elegant way? Any advice is most welcome. Thanks for your work on this project!

Hi, thanks for your reply. Still experimenting with your lib and really liking its direction. I feel that this issue #26 & issue #14 (object replacement reactivity) are stumbling blocks to adopting this more widely, because they break from the patterns of typical Vue components, and would likely introduce mistakes in use. The workarounds are awkward at best.
Fully understand your time constraints. If I learn a bit more about Vue's internals, I may be able to offer a PR (but not very soon).
Do you think the if (typeof vue['created'] === 'function') vue.created() patch is the right approach, or would it be cleaner integration to (somehow) augment makeOptions() such that Vue's own initialization calls the created() hook as part of the normal Vue lifecycle? ... if that's even the way to approach this.
Thanks for any advice!

I can confirm that PR #27 fixes this issue, as well as issue #14.


Empty repository in HA 2022.11.4

I have HA at version 2022.11.4 and when I add repository https://github.com/thomasmauerer/hassio-addons I did not see any available package.
It looks like that Samba Backup package is not compatible with the latest version of HA.
Could you help me?

Yeah solved, it is needed to completly reload page in internet browser by pressing F5.


Add support of Ukrainian (or Cyrillic in general) language for Lilita One and  Comic Shanns fonts

Please, add Ukrainian language for following fonts:

Lilita One
Comic Shanns

@vasylkorchynskyi It seems that these fonts don't support Ukrainian language, so I suggest to close it with a "won't fix"

Yeah, unfortunately, none of the fonts supports cyrillic, so we would likely have to find some fallback font which looks similar.

@Mrazator What do you think about these font alternatives ?


[FLOC-4339] Run all tests on Ubuntu 16.04

Fixes: https://clusterhq.atlassian.net/browse/FLOC-4339

Blocked by https://github.com/ClusterHQ/flocker/pull/2853

@moshez reviewed this as part of #2872
Merging.


sage: don't test the downgraded R

Motivation for this change
Fix hydra build (details in #51191).
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: sage
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: sage
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Timed out, unknown build status on x86_64-linux (full log)
Attempted: sage
Partial log (click to expand)
build/cythonized/sage/matrix/matrix0.c:6243:7: note: '__pyx_v_row' was declared here
   int __pyx_v_row;
       ^~~~~~~~~~~
building of '/nix/store/2xjfh4mw19g0fqh7zhc92w5iq537pq4b-python2.7-sagelib-8.3.drv' timed out after 3600 seconds
cannot build derivation '/nix/store/frcxa7mb98dc4y38a0r6vfnd61hbyryf-python-2.7.15-env.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/55314msh7jryxbs7s2l9kacbv5yprxwd-sage-env.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/skm746xq0vxzwvk22qndgnih4c8ds2bn-sage-with-env-8.3.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/jpm2v3lwackzs6r20592dkcamn7llycd-sage-tests-8.3.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/2hqf6n11dkn2yzby8mbfq293f9nx6bp0-sage-8.3.drv': 1 dependencies couldn't be built
error: build of '/nix/store/2hqf6n11dkn2yzby8mbfq293f9nx6bp0-sage-8.3.drv' failed


Proteoform annotation

New features:

peptide outlier removal based on maximum correlation criterion, including iterative filtering option
determine genes/proteins with multiple proteoforms based on a minimum correlation criterion
fit gamma distribution to estimate p-values
cluster peptides according to their pairwise correlations either per condition or across conditions

Example script for running the new functions:
example_annotateProteoforms.R.zip

latest commit fix: Added the option to do also perform prefiltering on combined traces in order to get better clustering results. Example script:
example_annotateProteoforms_multi.R.zip

In estimateProteoformPval.trace, changing split_value from default to custom did not work for me.
Three-dots construct may be included into fitGammaDist within estimateProteoformPval.trace?


Metadata schema parsing fails when display view is used via freshmint.yaml

Closed by https://github.com/packagelabs/freshmint/commit/14c76a04dfa637d570aa5c7b78d5e9ff20ed78b0