migrateVolume does not copy the template

When a volume is copied to another storage pool, if the template does not exist in the destination storage pool yet, it should be copied. The API migrateVirtualMachineWithVolume already does it; however, when migrating the volume with migrateVolume (VM stopped), the template is not copied, which causes inconsistencies in the database and primary storage systems.

is this what you are fixing in #5758, @GutoVeronezi ? or do you mean that more improvement is needed after that?

@DaanHoogland it will need more improvement after that.

Hi @GutoVeronezi are you working on this issue or planning to?

Hi, @nvazquez, I have other priorities at the moment, this issue is not in my plannings for now.

Thanks for clarifying @GutoVeronezi

Moving this to next milestone

I've hit this @GutoVeronezi for local storage (but not shared storage). Did you reproduce this only for KVM + local storage (or any other hypervisor/storage type)?

@rohityadavcloud, at the time I created this issue, I was running ACS with 4.15 or 4.16 (I do not remember quite well). I will test it again and verify if the situation still happens with the current version.

@rohityadavcloud @GutoVeronezi I have tested with local storage as well shared storage. both work.
The migrateVolume will firstly copy the image to secondary storage (full clone) and then copy to another primary storage. It should not be a bug. therefore I have removed the label 'type:bug'
The whole process can be improved of course.

Thanks for the tests, @weizhouapache.

@rohityadavcloud @GutoVeronezi I have tested with local storage as well shared storage. both work.
The migrateVolume will firstly copy the image to secondary storage (full clone) and then copy to another primary storage. It should not be a bug. therefore I have removed the label 'type:bug'
The whole process can be improved of course.

Thanks for the tests, @weizhouapache.

@GutoVeronezi
please feel free to improve it.

@weizhouapache there are some issues; cc @DaanHoogland @GutoVeronezi
While live running VM migration with storage works between KVM hosts with local storage, try this;

Deploy a new VM we can see the root disk has a backing file:

root@kvm1:/var/lib/libvirt/images# qemu-img info f9a311bb-509c-4f01-ad55-29eb8ff6166a
image: f9a311bb-509c-4f01-ad55-29eb8ff6166a
file format: qcow2
virtual size: 25 GiB<PHONE_NUMBER>0 bytes)
disk size: 1.51 GiB
cluster_size: 65536
backing file: /var/lib/libvirt/images/748e7f9d-7257-48dd-9efa-971396fee88f
backing file format: qcow2
Format specific information:
compat: 1.1
compression type: zlib
lazy refcounts: false
refcount bits: 16
corrupt: false
extended l2: false

Stop the VM and migrate the disk from kvm1 to kvm2 (local storage), but now on kvm2 local storage the VM has no backing file:

root@kvm2:/var/lib/libvirt/images# qemu-img info 6b4add3f-9014-4c76-ae45-cb19f830c3be
image: 6b4add3f-9014-4c76-ae45-cb19f830c3be
file format: qcow2
virtual size: 25 GiB<PHONE_NUMBER>0 bytes)
disk size: 1.5 GiB
cluster_size: 65536
Format specific information:
compat: 1.1
compression type: zlib
lazy refcounts: false
refcount bits: 16
corrupt: false
extended l2: false

Start the VM, and try to live migrate with storage to kvm1 host, it works or sometimes fails. When it works, I can see there is no backing file on the migrated disk: (migrated and shutdown the VM to get the qemu-img info to work)

root@kvm1:/var/lib/libvirt/images# qemu-img info f919f5e7-eee8-4a64-ac6f-a4361dfcaa1b
image: f919f5e7-eee8-4a64-ac6f-a4361dfcaa1b
file format: qcow2
virtual size: 25 GiB<PHONE_NUMBER>0 bytes)
disk size: 1.51 GiB
cluster_size: 65536
Format specific information:
compat: 1.1
compression type: zlib
lazy refcounts: false
refcount bits: 16
corrupt: false
extended l2: false

For the failure case, I think try to migrate to kvm3 host which doesn't have the backfile file (template seeded), the running VM crashes as the domain on the destination fails to run and we get the error:

2023-07-10 19:17:42,074 WARN  [cloud.agent.Agent] (agentRequest-Handler-2:null) (logid:22f93a95) Caught:
com.cloud.utils.exception.CloudRuntimeException: Could not fetch storage pool 8aa9768c-cbcf-4e8e-8875-f94a7f9445b6 from libvirt due to org.libvirt.LibvirtException: Storage pool not found: no storage pool with matching uuid '8aa9768c-cbcf-4e8e-8875-f94a7f9445b6'
at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.getStoragePool(KVMStoragePoolManager.java:277)
at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.getStoragePool(KVMStoragePoolManager.java:263)
I think in #4 when/if it fails, it's because somehow it's trying to use/find the local storage pool on the dest. host which does not exist.

I suppose the primarily issue seems to be when during offline migration, the backing file information is lost as the KVM local storage/root disk(s) are melded in a way it no longer needs the template. In edge cases, the VM migration can crash the running VM when on the destination host when for whatever reason it find the domain isn't running. We need perhaps a round of investigation/reproduction, and if possible (a) avoid using secondary storage for offline local storage migration (or document the behaviour) and (b) backing file context isn't lost. /cc @weizhouapache @DaanHoogland

this might be related to #7615

@GutoVeronezi
would you like to work on this ?

@rohityadavcloud @weizhouapache Considering that the migration consolidates the volume and template (at least when using KVM), I think now that it makes sense not copying the template when migrating the volume, as the migrated volume will already contain the template data. However, we should do some tests to pinpoint use cases that could be affected by this behavior or are not working as expected.

@GutoVeronezi would you like to work on this ?

Unfortunately, I will not be able to dedicate my efforts to this right now. @gpordeus, as you are already working on #7615, could take a look at this?

@GutoVeronezi Yes, I'm now available. I'll assemble a use case table, test it and share the results as soon as they are ready.

moved to <IP_ADDRESS>

Is this fixed ?

@DaanHoogland @rohityadavcloud @GutoVeronezi
I tried reproducing the problem with the latest main, in an ubuntu+kvm environment, but wasn't able to.
Any missing template inconsistencies with migrateVirtualMachineWithVolume should be solved by #8911, since it standardizes always merging the backing file into the volume.
As for migrateVolume:

Deployed a VM, its volume on NFS primary storage. It went up with the backing file. I stopped the VM, migrated the volume to SMP primary storage and it resulted in a merged backing file.

scc@local:/mnt/pri-storage-smp$ qemu-img info 9eb27ca6-d721-45f0-a334-144e29377fc5
image: 9eb27ca6-d721-45f0-a334-144e29377fc5
file format: qcow2
virtual size: 8 GiB<PHONE_NUMBER> bytes)
disk size: 3.09 GiB
cluster_size: 65536
Format specific information:
    compat: 1.1
    compression type: zlib
    lazy refcounts: false
    refcount bits: 16
    corrupt: false
    extended l2: false

Deployed another VM with a new template, its volume on NFS primary storage. It went up with the backing file. I stopped the VM, migrated the volume to another NFS primary storage and it resulted in a merged backing file.

scc@local:/mnt/a548af6a-abff-3541-8130-bfe19ff1cb7d$ qemu-img info 8daf78fe-202b-481d-93c9-aa1299dd43b9
image: 8daf78fe-202b-481d-93c9-aa1299dd43b9
file format: qcow2
virtual size: 8 GiB<PHONE_NUMBER> bytes)
disk size: 1.41 GiB
cluster_size: 65536
Format specific information:
    compat: 1.1
    compression type: zlib
    lazy refcounts: false
    refcount bits: 16
    corrupt: false
    extended l2: false

Deployed another VM with yet another template, its volume on local storage. I stopped the VM, migrated the volume to another local storage and it resulted in a merged backing file. Started the VM back up and migrated it and its volume back to the first local storage. The VM ran normally, its volume without any mention of a backing file.

scc@host-local:/var/lib/libvirt/images$ qemu-img info -U 1bbcd0df-747d-4bfd-9680-b8dff1f1a97e
image: 1bbcd0df-747d-4bfd-9680-b8dff1f1a97e
file format: qcow2
virtual size: 8 GiB<PHONE_NUMBER> bytes)
disk size: 1.41 GiB
cluster_size: 65536
Format specific information:
    compat: 1.1
    compression type: zlib
    lazy refcounts: false
    refcount bits: 16
    corrupt: false
    extended l2: false

Deployed another VM with the same template as last test, its volume on local storage. I stopped the VM, migrated the volume to another local storage that already had the same template and it resulted in a merged backing file. Started the VM back up and migrated it and its volume back to the first local storage. The VM ran normally, its volume without any mention of a backing file.

scc@local:/var/lib/libvirt/images$ qemu-img info -U d8dd43db-f798-404e-9de1-0914308c8cd0
image: d8dd43db-f798-404e-9de1-0914308c8cd0
file format: qcow2
virtual size: 8 GiB<PHONE_NUMBER> bytes)
disk size: 1.41 GiB
cluster_size: 65536
backing file: /var/lib/libvirt/images/318bc978-1e0f-4ab2-8b04-7fad5dc183cd
backing file format: qcow2
Format specific information:
    compat: 1.1
    compression type: zlib
    lazy refcounts: false
    refcount bits: 16
    corrupt: false
    extended l2: false

The local storage problem looks the same as #7942 and seem to have been solved by #7945.

ok @gpordeus , I am not sure if you can, but if so feel free to close the issue unless you feel more discussion is needed. We can then convert this ticket to a discussion.


StarMask: Received invalid network parameters. Please report this bug.

{chainId: '0x1', networkVersion: undefined}

无法重现


virlXnode.tf: Correct inline documentation for the option to use a static project id

thanks for the catch


Cloned parse server return empty when I use condition with $gte, $eq and etc.

New parse server return empty:
curl -X GET -H "X-Parse-Application-Id: xxxxxxxxxxxxxxxxxxx" -H "X-Parse-REST-API-Key: xxxxxxxxxxxxxxxxx" -G --data-urlencode 'where={"createdAt":{"$gte":"2015-12-14T06:58:02.196Z"}}' http://<IP_ADDRESS>/1/classes/Group
{"results":[]}
Old parse server return some records:
curl -X GET -H "X-Parse-Application-Id: xxxxxxxxxxxxxxxxxxx" -H "X-Parse-REST-API-Key: xxxxxxxxxxxxxxxxx" -G --data-urlencode 'where={"createdAt":{"$gte":"2015-11-09T01:36:05.570Z"}}' https://api.parse.com/1/classes/Group
These data are clone from parse.com, where I make wrong?
Thanks
Yanlong

We are closing this issue due to a lack of sufficient information.
Please refer to this issue for an example of a great bug report.
You may use Server Fault for questions about managing Parse Server.
For code-level and/or implementation-related questions or technical support, please refer to Stack Overflow.


Bump Google CPI 29.0.0 → 29.0.1

Per the CPI Release Notes:

CPI retries if a network error is a timeout (in addition to temporary errors).

Hey @cunnie,
We've gone ahead and automated this process, and all CPIs should be auto updating now. Thanks for the submission!
Best,
@jfmyers9


members: reserves space for success/error icon

closes https://github.com/inveniosoftware/invenio-app-rdm/issues/1714
The issue was that the dropdown container width was adjusting when the icon rendered. E.g. when the icon was rendered, the dropdown container shrinked.
This fix reserves a small square of space (1em x 1em) for the icon. Does not reduce the dropdown size a lot and it's enough to make the whole cell static when changing the selection.

is it also implemented on the role dropdown and invitations tab?


Fix Page Collections problem with @page.modular

Loading the page children before call modular on the Collection.
In order to view the complete example issue you can refer to https://getgrav.org/forum#!/general:cant-get-modular-collectio

Thanks I'll look at this soon and review.

There are two kinds of children: modular and non-modular. After the fix both will be returned, which is not correct.

I tried adding a non-modular child, it's not picked up, as it's calling $results->modular();. Not sure if I "got" the problem you notice

Yes, the modular function juste keep the modular pages.


Use path seperators to provide support for Windows environments

Proposed solution to the issue I have raised [here].(https://github.com/codebandits/react-app-rewire-svg-react-loader/issues/1)

@singular1ty94 thanks for finding the issue and for making this PR. I made the same mistake in react-app-rewire-css-modules.
And thank you for addressing the tests with your changes!


Travis Failing due to expected value of broken links

Actual Behaviour
Travis test is failing due to invalid expected count of broken link.
Expected Behaviour
The expected count should be 1 instead of 0.
Would you like to work on the issue?
Yes.

@agbilotia1998 Are the links broken because of no sponsor data for FOSSASIA 2018?

@geekyd one of the sponsor's link is giving Page not Found error in Fossasia Summit 2017 sample, that's why the travis is failing.


Check docs for wrong information

Check the docs for the following wrong information:

Remove indicator for some features which were previously flagged as "pro"
Remove email and website references to LASSO as the project belongs to the community now

@codie3611 Should we retain Lasso logo or can it be deleted?

Hmmm what do you think? We don't have another logo so I would keep it. Also as a thanks to the company for open-sourcing the code.

Hmmm what do you think? We don't have another logo so I would keep it. Also as a thanks to the company for open-sourcing the code.

@codie3611 sounds good to me. It is fair enough.


Refactor buscador sospechosos

Codecov Report

Merging #4 (5d387c1) into master (8fa9421) will decrease coverage by 0.26%.
The diff coverage is 79.48%.

@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
- Coverage     90.90%   90.64%   -0.27%
- Complexity      109      126      +17
============================================
  Files            37       38       +1
  Lines           264      310      +46
  Branches          9       13       +4
============================================
+ Hits            240      281      +41
- Misses           20       24       +4
- Partials          4        5       +1

Impacted Files
Coverage Δ

src/main/java/edu/fiuba/algo3/modelo/Pista.java
100.00% <ø> (+40.00%)
:arrow_up:

src/main/java/edu/fiuba/algo3/modelo/Policia.java
96.96% <66.66%> (+2.68%)
:arrow_up:

...ain/java/edu/fiuba/algo3/modelo/LectorArchivo.java
73.33% <73.33%> (ø)

src/main/java/edu/fiuba/algo3/modelo/Caso.java
91.66% <75.00%> (+1.66%)
:arrow_up:

...c/main/java/edu/fiuba/algo3/modelo/Sospechoso.java
90.00% <83.33%> (-10.00%)
:arrow_down:

.../edu/fiuba/algo3/modelo/BuscadorDeSospechosos.java
90.90% <90.90%> (ø)

...du/fiuba/algo3/modelo/demorador/TercerEntrada.java
60.00% <0.00%> (-40.00%)
:arrow_down:

src/main/java/edu/fiuba/algo3/modelo/Novato.java
100.00% <0.00%> (ø)

src/main/java/edu/fiuba/algo3/modelo/Edificio.java
100.00% <0.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


remove crypto<4 pin from depends tox env

@kdestin @xiangyan99
Resolves this build failure caused by depends env running on pypy39

/azp run python - ml - ci


Report a bug: Texture Descriptor Validation MTLTextureDescriptor has a height (8520) greater than the maximum allowed size of 8192

Platform:  tvOS 17.0 simulator.
ColorfulX Version: 2.1.3
when I used VStack to generate a rather long vertical ScrollView, and if the content is too long, it reports the error:
-[MTLTextureDescriptorInternal validateWithDevice:]:1357: failed assertion `Texture Descriptor Validation MTLTextureDescriptor has height (8520) greater than the maximum allowed size of 8192.

The program breaks at line 67 in MetalView.

Could you consider relocating the 'ColorfulView' component to the background of 'ScrollView'? and check if it is still crashing. It is generally inadvisable to render a view of this size, more than 8k in your case, outside the visible area.

Yes, this works

FYI: I've added a check to get this done right.
https://github.com/Lakr233/ColorfulX/blob/main/Sources/ColorfulX/MetalView.swift#L66


Nice work!

I am pleasantly impressed by all your work on this effort.  Let me know if there is anything I can help with.

Hi, yep very good+helpful project indeed.
is there any pointer/project/tool to convert Go/Golang into C ?
thanks in advance.

I think "cgo" allows this already, basically you can use your Go functions in C/C++:
https://golang.org/cmd/cgo/#hdr-C_references_to_Go

i posted link to this project here: https://stackoverflow.com/questions/64180191/
i want to transpile  completely from Go into C or C++, not use any Go & avoid any Go in C/C++ source-code.

I upvoted your post, but looks like topic was closed.
FYI, given my experience with this project so far, i.e., converting from Golang to C#, I think a converter to C at a source code level is certainly possible. I disagree (given y experience here) about the guy who responded "goroutines, channels) would very hard to express in C" - that would actually be the easy part, the harder parts are like these mentioned here: https://go2cs.net/ConversionStrategies.html, for which workable strategies already exist.
A lot of work, but very doable.
Thanks,
Ritchie

BTW - most of those other tools, like the ones that go directly from Go to C or C++ are way short of being able to compile day-to-day Go code. Even the code I've put together here still has a ways to go - however, I have already produced a half-way decent conversion to the entire Go standard library source code:

https://github.com/GridProtectionAlliance/go2cs/tree/master/src/go-src-converted

I feel I'm less than 20% away from a reliable transpiler, although that last 5% may be hard won - with all the "edge" cases that will need to be handled.

Hi Ritchie, THANK YOU, THANK YOU ... , your upvote in both Question & Answer, were very very helpful , the post is now not 0, and my answer is not "-1" , and because of these , more users can now see it more easily, or else, negative scored articles often shown at-end of search results, etc.
After closing in SO, after 5 days, SO community suppose to take vote on it again to re-open, i think they have chosen not to re-open, as it may/will invite more pro-Google/Golang type answers, or more anti-C/C++ , or more anti-transpiler type of answers, etc, etc.
Question was simple, which transpiler can do Go-to-C conversion, & which can do it better, etc , Q is about transpiler , but other answer-er has taken it to different direction, etc ( i have downvoted it because as its not correct answer for the question which i have posted . it is a good answer for another type of question , not for the question what i made , any answer must focus on what i have asked-for first , answer must analysis the transpilers & solutions i have mentioned & requested to analyze, etc ).
You're right, the other person in comment area, even said there is no-tool !! i knew of few transpilers (but all were supporting only subset), & then my search began to show even more , though those search-results were better than what i knew , but none were full or close to full support of destination language , these type of tools need lots of improvements , which you have also pointed out . Thanks for point those out, but yeah, sad.
i guess programing is an art, too many ways to program, its very hard to put all such "patterns" & "variations" into workable detection engines & spit out something very close in another language.
I'm still learning & grasping with manual Go-to-C++ transpilation, for simple project conversion,
and i still also have to learn few others.
I understand C, Assembly, etc and not-all-of C++, etc , Others not so well , Golang is much newer to me.

I'm not aware of any "good" options for actual source conversion from C# to C++. There was a project, like 5 years back, that attempted this https://github.com/AlexAlbala/Alter-Native. This project appears dead, but perhaps you can find some some forks with some life in them. One of the stated goals of the Alter-Native project was the "WORA (Write Once, Run Everywhere) philosophy", but this goal has now been fully subsumed by .NET Core.
There's also a commercial option - see here. Note that this commercial option is not .NET Core based, its highest supported .NET version is NET Framework v4.6.1. Also the conversions seem to be focused on "language conversion", the tool seems limited in terms of .NET class library conversions. Here's a quote from the main page:

"C++ to C# Converter is intended to reduce the amount of work you'll have to do to convert code to C#, but it is just the first step."

The .NET class libraries are the primary issue. A source code conversion from C# to any other language target for the purposes of transpilation is made difficult by the fact that the .NET class libraries are so extensive. I guess someone could make the same argument about the Go core libraries where I am already making progress, but the Go language is minimalistic and very simple as compared to C#.
I know there are several commercial options that say they will go from C# to Java (maybe even some open source ones), but I suspect none of these tools have been able to keep pace with the rapid language feature and .NET Core updates that have been happening over the last few years.
I think C# is starting to stand alone in the field of programming languages - especially in terms of language features, depth of class libraries and runtime speed. However, with this comes new levels of complexity that make truly learning C# more difficult. It has always been simple to get started with C#, but it takes time to master. I think this is why so many new programmers are attracted to Go, the language is simple - it's simple to get started, and quick to master. It is because of this that I am developing go2cs, i.e., it will allow the true "transpiler" option: you can code in Go and compile to C# (should one need to interact with .NET). Guess we'll see how well that ends up.
For something close to transpilation of C# to C++, I am aware of the "CoreRT" project (https://github.com/dotnet/corert) that offers real native compilation of .NET, to the point where it produces native libraries that can be linked into C/C++ applications. This would basically allow you take your C# code and use it along with other native C++ applications. The issue here is that this project is still in "experimental" mode.
That said, if you don't need "lib" files that can be linked with C++ projects, .NET already compiles to native code with an option called "NET Native". In Visual Studio, for example, you can set this up with a folder-based publish target that will allow you to specify a native OS and platform target, e.g., Linux x64. This will produce native code directly from C# that is more in-line with typical C++ application runtime speeds.
Lastly, all these options not to be confused with the "crossgen" project. This project keeps your code as portable IL, but on the target platform basically performs a pre-compile of the code. This is more akin to normal .NET runtime operation but with an up-front compile instead of using just-in-time (JIT) compile. However, this generally only buys you faster startup times by eliminating the JIT processing and will still fallback on JIT if a native binary cannot be found. Also, you won't get the same overall runtime speed benefits as you do with .NET Native compiles.
None of these native options gets you to "translated source code", but they do provide native compiled applications from C# code with the runtime speed benefits. So in the end, I guess it's all about what you would like to accomplish.

Hi Ritchie , THANK YOU for your VERY HELPFUL if not best analysis.
need both : speedy conversion from Go into C/C++,
and also high-quality source-to-source code conversion with all high-level structures of input remaining fairly accurate in output source.
i think thats still far now for Go-to-C/C++ conversion.
so i will settle in middle now, need better quality C or C++ output, with Go-to-C or Go-to-C++ fast conversion.
another request:
i have updated the SO page with a free transpiler tool called "Hurley" , it appears able to do C# to C++ conversion.
if you have time to inspect it little bit, what you think of C#-to-C++ conversion quality of Hurley ?
i doubt it can do better than CoreRT , but might be better than  cs2cpp ?
other option for C#-to-C++ is Alter-Native, but its dead for now, as you've pointed out.
if "Hurley" can do HQ "cs2cpp" conversion like your this "go2cs" project , then my objective of HQ (high-quality) conversion of "Go-to-C" or "Go-to-C++" is almost achieved with only two conversions : go2cs -> hurley.
The "go2c(mukadr)" transpiler's conversion quality is not high :( now , bcoz it supports only subset of Go, or else, i would/could have used it continuously.
CrossGen is very interesting.
Yes, i need the (Go-source) transpiled (into C/C++) source working in macOS & Debian-GNU/Linux & Windows.
As none of the transpilers were suitable for my "go2c"/"go2cpp" conversion need , i began to look into Go to Assembly conversion (and then from Assembly to C/C++, i know this process would have lost almost all high-level structures in "Go" source, but it would be a faster solution) , & So, i found-out that "Go" tools can convert it into "Assembly", but its a "Golang" flavored "Assembly" , not GNU/GAS/AT&T-Assembly , not NASM-Assembly, etc.
So i'm trying to understand if any tool from GoDevTool can do this Go-"Assembly" to GNU/NASM-Assembly conversion . As i want to avoid compiling the Go source into machine code & then machine-code to Assembly->C/C++ with "Boomerang", etc.


2-Factor Authentication steps no longer work

The currently described "2-Factor Authentication" steps in the README appear to no longer be accurate.

Never mind, I had an accidental filter in the Network tab. 🙄


Text-to-small molecule deterministic vs. nondeterministic behavior

I was just wondering if there is a way to adjust the model to be less deterministic. I've tried adjusting the beams in the beam search, adding in functionality to adjust the temperature of the model and various other things in an attempt to make the model non-deterministic but it does not seem to be working. As an example, I have tried the following:
from transformers import T5Tokenizer, T5ForConditionalGeneration
import selfies as sf

tokenizer = T5Tokenizer.from_pretrained("QizhiPei/biot5-base-text2mol", model_max_length=540)
model = T5ForConditionalGeneration.from_pretrained('QizhiPei/biot5-base-text2mol')

task_definition = 'Definition: You are given a molecule description in English. Your job is to generate the molecule SELFIES that fits the description.\n\n'
text_input = "The molecule is a monocarboxylic acid anion obtained by deprotonation of the carboxy and sulfino groups of 3-sulfinopropionic acid. Major microspecies at pH 7.3 It is an organosulfinate oxoanion and a monocarboxylic acid anion. It is a conjugate base of a 3-sulfinopropionic acid."
task_input = f'Now complete the following example -\nInput: {text_input}\nOutput: '

model_input = task_definition + task_input
input_ids = tokenizer(model_input, return_tensors="pt").input_ids

generation_config = model.generation_config
generation_config.max_length = 512
generation_config.num_beams = 5
generation_config.do_sample = True
generation_config.top_k = 50
generation_config.top_p = 0.75
generation_config.temperature = 0.95

# Set the number of sequences to generate, must be <= num_beams
num_sequences = 3
generation_config.num_return_sequences = num_sequences

outputs = model.generate(input_ids, generation_config=generation_config)

print(f"Top {num_sequences} generated sequences:")
for i, output in enumerate(outputs, start=1):
    output_selfies = tokenizer.decode(output, skip_special_tokens=True).replace(' ', '')
    output_smiles = sf.decoder(output_selfies)
    print(f"Sequence {i}:")
    print("Generated SELFIES:", output_selfies)
    print("Generated SMILES:", output_smiles)
    print()

It seems that no matter how I adjust this (turning beams down to one or to higher numbers for example) does nothing to change the output. Even changing the prompt slightly does not seem to change the output. Perhaps the model has learned too rigid a representation of the text-molecule mapping? Or perhaps I am overlooking something?

I tried some other test case with increased num_beams and observe different outputs. Adjust diversity_penalty can also result in different outputs. Could you try more cases with above settings?
Feel free to contact for any further questions or discussions.

Would you be willing to provide your settings?

generation_config = model.generation_config
generation_config.max_length = 512
generation_config.num_beams = 25
generation_config.top_k = 25
generation_config.top_p = 0.75
generation_config.temperature = 0.95
generation_config.num_return_sequences = 25

generation_config = model.generation_config
generation_config.max_length = 512
generation_config.num_beams = 5
generation_config.do_sample = False
generation_config.num_beam_groups = 5
generation_config.diversity_penalty = 10.0
num_sequences = 5
generation_config.num_return_sequences = num_sequences

Thanks for the response! I'm still getting deterministic output. Any ideas why?
Also, your first example above returns a warning:
UserWarning: `do_sample` is set to `False`. However, `temperature` is set to `0.95` -- this flag is only used in sample-based generation modes. You should set `do_sample=True` or unset `temperature`.
  warnings.warn(
/opt/home/asr50/miniconda3/envs/biot5/lib/python3.8/site-packages/transformers/generation/configuration_utils.py:367: UserWarning: `do_sample` is set to `False`. However, `top_p` is set to `0.75` -- this flag is only used in sample-based generation modes. You should set `do_sample=True` or unset `top_p`.

I am still getting deterministic behavior from the model. I've tried your recommendations, as well as top-k sampling, top-p sampling, temperature sampling, and a couple of others, and the model seems to generate the same molecule each time regardless of what I do. It would be very helpful to have the GitHub updated with some example scripts for nondeterministic sampling methods (as well as some variations on beam search).

Thanks for your suggestions. I will updated some examples for nondeterministic generation settings.
However, it's not make sense that we get different outputs with the same generation setting.
Could you kindly provide your outputs of the above two settings?
 For example, for temperature sampling I have the following:
from transformers import T5Tokenizer, T5ForConditionalGeneration
import selfies as sf

model_name = "QizhiPei/biot5-base-text2mol"
tokenizer = T5Tokenizer.from_pretrained(model_name, model_max_length=512)
model = T5ForConditionalGeneration.from_pretrained(model_name)

def generate_mol_from_text(text_input, tokenizer, model, **kwargs):
    # Tokenize input text
    task_definition = 'Definition: You are given a molecule description in English. Your job is to generate the molecule SELFIES that fits the description.\n\n'
    task_input = f'Now complete the following example -\nInput: {text_input}\nOutput: '
    model_input = task_definition + task_input
    input_ids = tokenizer(model_input, return_tensors="pt").input_ids

    # Generate molecule SELFIES
    generation_config = model.generation_config
    generation_config.update(**kwargs)
    generation_config.max_length = 512
    generation_config.do_sample = True  # Enable sampling-based generation
    outputs = model.generate(input_ids, generation_config=generation_config)

    # Decode SELFIES and convert to SMILES
    selfies = tokenizer.decode(outputs[0], skip_special_tokens=True).replace(' ', '')
    smiles = sf.decoder(selfies)

    return selfies, smiles

text_input = "A monocarboxylic acid anion obtained by deprotonation of the carboxy and sulfino groups of 3-sulfinopropionic acid. Major microspecies at pH 7.3. It is an organosulfinate oxoanion and a monocarboxylic acid anion. It is a conjugate base of a 3-sulfinopropionic acid."

selfies, smiles = generate_mol_from_text(text_input, tokenizer, model, temperature=0.7)

print("Generated SELFIES:", selfies)
print("Generated SMILES:", smiles)

For top-k sampling we can change the line
selfies, smiles = generate_mol_from_text(text_input, tokenizer, model, temperature=0.7)

to
selfies, smiles = generate_mol_from_text(text_input, tokenizer, model, do_sample=True, top_k=25)

For top-p sampling we can change it to:
selfies, smiles = generate_mol_from_text(text_input, tokenizer, model, do_sample=True, top_p=0.9)

For diverse beam search we can use something like:
selfies, smiles = generate_mol_from_text(text, tokenizer, model, num_beams=25, num_beam_groups=5, diversity_penalty=1.0)

So currently I have five scripts for text-to-small molecule (t2m):

t2m_top_k.py
t2m_top_p.py
t2m_temp.py
t2m_diverse_beam.py
t2m_beam.py

Vanilla beam search is deterministic and should yield the same results each time assuming the settings are the same. The other sampling methods should be on-deterministic and should yield a new molecule each time we run a forward pass, right?
If nondeterministic sampling is possible with this model, my concern is that the model has collapsed as described in the first three paragraphs of Section 2 of the I-JEPA paper.
It's possible this intuition does not apply to the BioT5+ architecture, but joint modeling usually requires contrastive loss (or some other trick) to prevent the energy landscape of the model from being too flat. For example, the I-JEPA paper mentions

"non-contrastive losses that minimize the informational redundancy across embeddings, and clustering-based approaches that maximize the entropy of the average embedding.

I'm still not convinced it's not an error on my part though, so any discussion or help you might provide is very much appreciated.

Sorry for the delayed response. Here are some points which may address your concerns:

Limited Diversity Due to Finetuning: Our model was finetuned on the Chebi-20 dataset, which contains a limited number of texts and molecules. This limited diversity might result in the model generating less diverse outputs.

Increasing Temperature in Sampling Mode: In sampling mode, you should try increasing the temperature value (default value is 1.0) to enhance the randomness of the generated molecules. However, this might also increase the probability of generating invalid SELFIES.

Retraining with Contrastive Loss: Retraining the model with contrastive loss should not be particularly difficult. We have provided the finetuning code and datasets. You would likely only need to modify the corresponding loss function.

Closing this issue due to no further discussions. Feel free to reopen if needed.


web interface not displaying result data

when a command takes to long to complete, the results section is not updated until you next submit the page.

This has been resolved by redesigning the web interface to use jquery


resubmitting feature request: Add support for auth partitions with route domains

Is your feature request related to a problem? Please describe.
This was originally filed under issue #51 but closed. I'd like to re-open because we have a few SE's and SA's that would need this. Here's our situation: we are deploying BIG-IPs into AWS across different AZ's. Then we want to use DO to configure the devices and cluster them. Because they are in different AZ's, we need to create a default route that is unique to each of the devices, and therefore it should be created in a LOCAL_ONLY partition that is not sync'd between devices. So, we need DO to be able to create auth partitions and we need to be able to define a route in this partition.
Describe the solution you'd like
Ideally we'd like to be able to use DO to configure the devices and cluster them, even when the devices are in separate subnets and across AZ's.  So we need DO to support auth partitions with route domains.
Describe alternatives you've considered
Because we cannot use DO to create routes that are not in /Common, we have resorted to BASH commands in the onboarding script run at deployment time in AWS. However this is not ideal.
Additional context
This is a public-facing guide to setting up HA in AWS and it states that you need a LOCAL_ONLY partition: https://www.f5.com/pdf/deployment-guides/f5-aws-ha-dg.pdf (see the step "Creating a new partition")

+1
I had to add the following commands to user-data to achieve this.
tmsh create net vlan internal { interfaces add { 1.2 } tag 1002 }
tmsh create net self internal-self { address ${internal_ip}/24 allow-service default traffic-group traffic-group-local-only vlan internal }
command below creates a partition in addition to a folder, automatically.
tmsh create sys folder /LOCAL_ONLY device-group none traffic-group traffic-group-local-only
tmsh -q -c 'cd /LOCAL_ONLY; create net route default { gw ${internal_gw} network default }'
tmsh save sys config partitions all

Would a flag on the Route class to create the route in LOCAL_ONLY satisfy your use case? To avoid scope conflict with AS3 we would rather not allow arbitrary create of auth partitions with Declarative Onboarding.

@dstokesf5 thanks and understood. I think so. The flag would need to create the partition as well as the route in that partition (ie, the partition will not exist prior to the DO declaration). This should allow us to configure a HA-pair across AZ's in AWS.

@mikeoleary Excellent. That is what we are planning to do with AUTOTOOL-1749.

This issue should be resolved in the 1.15.0 release.


[Feature] Prettify Json When Saving Response To File

Closes #764

This should be an option and not done by default.  Saving the response with whitespace can increase the file size considerably.


Handle scope merging correctly

Codecov Report

Merging #299 (2385ab8) into main (a5862b7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #299   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files          17       17
  Lines         252      252
=======================================
  Hits          240      240
  Misses         12       12

Flag
Coverage Δ

dotnet
95.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


fix(fsengagement): add flag to enable forced resync

@bweissbart
Currently if you call getProfile as a guest it either gets or creates a profileId based on your app and device.   If you then login with an app we would want to make the api request again with the accountId (email address) to sync it with your profile.  Currently it only checks to see if you have a profileId set and if so it returns the Id without re-syncing.  Passing an optional flag will force the api call to be made that will sync the profile with the account id.

nevermind, apparently I already put in this pr a while ago


Please merge revision 17b1e30d045c4bbf2b156b66a973e0fdaba136f0 into dev channel

@whesse @mit-mit @kasperl
<Fill in the labels for Milestone and Area to indicate what the merge applies to. E.g. if you're requesting a merge to fix an issue in Dart2JS, set the area to Area-Dart2JS. If this is a request to merge to stable please change the MergeToDev label to MergeToStable>
<Describe the problem this merge is fixing - include issue numbers if applicable>
This makes the dartdoc that supports generic-feature-types link in with the SDK.  #27527, dart-lang/dartdoc#1321
<What revision(s) needs to be merged - please annotate revisions with reason if more than one>
17b1e30d045c4bbf2b156b66a973e0fdaba136f0
<Please try out the merge yourself. If there are merge conflicts please resolve these and upload the cl to rietveld and provide a link here>
This merged cleanly, tested that dartdoc was able to generate SDK and a package dartdoc.

Corrected with the right revision.

This will go in the next cherry-pick, which probably won't be for about 18 hours.  Is this key to the work on landing in google3?  Or can it wait to see if more cherry-pick requests arrive today?

It can likely wait; this isn't a google3 blocker AFAIK

Merged in 1.23.0-dev.11.5


[stable/efs-provisioner] Update apiVersions for Kubernetes 1.16

Is this a new chart
No
What this PR does / why we need it:
This PR updates the Deployment template to use version apps/v1 instead of apps/v1beta2. The apps/v1beta2 version has been deprecated since Kubernetes 1.9, and is off by default in Kubernetes 1.16 (which prevents this chart from being installed).
Which issue this PR fixes
No issue
Special notes for your reviewer:
No changes to the Deployment are required beyond the version, apps/v1beta2 was promoted wholesale to apps/v1.
I've tested that this chart installs successfully and can be upgraded from 0.7.0 in a test cluster.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

Merge conflicts fixed -- also CC @mariusv since I saw you were just added to the OWNERS file.

/lgtm


Jets 3.0.17 significant increase in Lambda duration

Checklist

[x] Upgrade Jets: Are you using the latest version of Jets? This allows Jets to fix issues fast. There's a jets upgrade command that makes this a simple task. There's also an Upgrading Guide: http://rubyonjets.com/docs/upgrading/
[x] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.rubyonjets.com
[x] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System
deployed to AWS

Jets
3.0.17

Ruby
2.7.2

Expected Behaviour
After upgrading Jets from 3.0.13 to 3.0.17, we noticed Lambda durations increase significantly for all endpoints in our REST API.

Current Behavior

After upgrading Jets from 3.0.13 to 3.0.17, lambda durations should be similar to before deployment. Lambda cost for our system increased 8 fold after the upgrade.
Step-by-step reproduction instructions

observe avg lambda duration
upgrade Jets from 3.0.13 to 3.0.17
deploy to AWS
observe avg lambda duration
note significant increase in avg duration for all lambdas

Code Sample

Duration of all lambdas in our system increased significantly ... even when no code changes were made, other than the Jets upgrade.
Solution Suggestion

Fix performance issue.

Thinking Jets 5 handles this since there’s only one Lambda function for all controllers.
https://blog.boltops.com/2023/12/05/jets-5-improvements-galore/


Can't use arrows as motion modifier to go to the next window

Steps to reproduce:

Open two windows in nvim
Try to jump from the left one to the right one with <c-w>-> (or the other way around)

Expected behaviour:
The cursor goes to the left window
Instead:
Well... it does not.
PS: switching windows with <c-w>l works as expected

<c-w>-> is not a correct mapping and definitely not a default

<c-w>> is the default binding to increase the width of the window
<c-w><lt> is the default binding to decrease the width of the window

What do you want to achieve exactly? Can you show me the exact keybinding in the correct vim keycode notion?

I guess you mean <c-w><right> and <c-w><left>. I'll check

Just pushed a fix. Thank you for reporting!

@folke
Yeah, sorry for being clear, even after months of using (n)vim I'm still not used to <right> and the like.
fix confirmed!

Ok, great!


Modernize & Cleanup

This is a low-priority "modernization" PR:

Fix automatic lints.
Switch to const generics
Benchmark with criterion
Switch to stable and updated rust
Move testing deps to dev-dependencies

The switch to const generics, and some of the lint fixes are breaking changes, but I figured I'd push this and have it ready in case someone else gets bored and wants to go on a rust cleanup quest.

Replaces #96.

@cryptonemo I'm happy to break this into multiple PRs if you'd like. Or you can review it commit-by-commit and rebase-merge.

Considering rather diverse changes, I would split this PR on multiply ones.

Can do, it'll just be a bit more back and forth.

P.S. Is there some high-level requirement that motivates these changes? I expect it will require a lot of changes in merkletree utilisation logic of rust-fil-proofs

https://github.com/filecoin-project/merkletree/pull/96#issuecomment-1065075119 and just general cleanup. I'm happy to pull this through rust-fil-proofs if the changes are desirable.

First commit extracted to https://github.com/filecoin-project/merkletree/pull/102.

Considering rather diverse changes, I would split this PR on multiply ones.

Can do, it'll just be a bit more back and forth.

P.S. Is there some high-level requirement that motivates these changes? I expect it will require a lot of changes in merkletree utilisation logic of rust-fil-proofs

#96 (comment) and just general cleanup. I'm happy to pull this through rust-fil-proofs if the changes are desirable.

Just a heads up that we can re-visit that point in particular after halo2 branch is merged to master

We can also make all the changes except the const generic ones first.

Continuing in #106.


Bug: Add a confirmation dialog in forgot passcode

Summary:
When user clicks on "Forgot passcode, login manually", he is directed to login page directly. However a cofirmation dialog should confirm whether he/she really wants to login manually. This will enhance the User Experience.
Steps to reproduce:

Go to Passcode Activity
Click on "Forgot passcode, login manually" button
You will be directed to the Login Activity

Expected behaviour:
Before directing to the login activity, a dialog should pop up to confirm the action.
Observed behaviour:
.No Confirmation takes place
Device and Android version:
Device - Mi A2
Android 9 Pie
GIF:

Can I start working on this issue ?

@rachittt Unless and until any critical action is taking place, I don't really think there is a need for any kind of confirmation dialog. This situation does not cause any harm as such.
Take an example of log out button, when user logs out, he/she has to manually re-enter the login credentials. This task is critical as the user might have clicked the log out by mistake or even have forgotten the credentials. Only in such critical tasks we need a confirmation dialog.

@harshshinde07 if a person clicks on the forgot passcode button by mistake, he has to manually enter the password as well as set up a new passcode and then confirm that passcode. I just thought maybe adding a confirmation dialog as we did in 'Mifos Mobile' would enhance the user experience of the User.

Does it go back to the passcode activity on pressing back button?

@harshshinde07 as of now, nope

I personally don't think adding the dialog here is a good practice. So just redirecting it back to passcode should work for now. What are your thoughts @luckyman20 ?

@harshshinde07 well, we can add a confirmation dialog.
@rachittt Go ahead :)

Cool! @luckyman20 Can you assign this task to me while I work on it ?

@rachittt Well, it is not necessary that you contribute through tasks only.


Accessibility: Conditional key handlers

Feature Request
Problem description
There is not a clear separation between behaviors and components if the action executed on the component should be different based on props or the event. See https://github.com/stardust-ui/react/pull/1488/files#r293815521
Proposed solution
Allow conditional keyboard handlers that would only execute an action if a condition based on the props or event is met

Recommendation is to conditionally (based on props) add only relevant key handlers.


GeocoderStatus missing

Thanks for stopping by to let us know something could be better!

PLEASE READ
If you have a support contract with Google, please create an issue in the support console. This will ensure a timely response.
Discover additional support services for the Google Maps Platform, including developer communities, technical guidance, and expert support at the Google Maps Platform support resources page.
If your bug or feature request is not related to this particular library, please visit the Google Maps Platform issue trackers.
Check for answers on StackOverflow with the google-maps tag.

Is your feature request related to a problem? Please describe.
My Code under test compares a geocoding result to enum members of google.maps.GeocoderStatus. It's missing from the mock object. I'd like to fix it upstream and not only locally.
Describe the solution you'd like
add it to src/maps/maps/constants.ts
Describe alternatives you've considered
Re-export enum taken from some other google repo
Additional context
Add any other context or screenshots about the feature request here.

If you would like to upvote the priority of this issue, please comment below or react on the original post above with :+1: so we can see what is popular when we triage.
@spicattutti Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.


[BUG] z-index of NamepsaceFilter is greater than VPopover which results in ugly render issue

Describe the bug
The z-index of the NamepsaceFilter component (1000) is greater than the z-index of the VPopover (30). The v-popover component assigns the tooltip class which z-index is overwritten here.

See https://github.com/rancher/dashboard/issues/10485 for more details.

milestone/release required.


Verify file intergrity

Would be nice if there's a way to generate hashes for files in KBFS without having to download the entire thing.

If filesystems included hashing algorithms in their system calls it could be possible.
KBFS is basically a translation layer for file system IO calls that does encryption/decryption and syncing with the server of encrypted data on the backend.
So until KBFS downloads all the data, and decrypts it... there is nothing to hash.
Also, KBFS can not intercept your hash command, since your hash command is not a filesystem command, your hash command just uses filesystem calls to read data, which KBFS will take as a trigger to download the data.
You are asking them to "create a way to hash data without having the data"... which is impossible.

I guess I worded it incorrectly, but it's more like I'm asking for a way for the server to generate and store the hashes for me. But then again that means the file has to be decrypted on their server which defeats the purpose of this program.
I guess it's not possible after all.


Support newer Django versions and drop support for python 2

@kjdchapman

Is this bit in the README now out-of-date? Shall we tweak as part of this PR?
good spot! as it's a tiny thing, i'll just push the read-me change tomorrow unless you strongly object


[kafka] Using $(hostname -s) vs ${HOSTNAME}

Recent change in the hostname implementation has required that we find short hostname using a different mechanism.  This patch makes that change.

/gcbrun

/gcbrun


AntiForgery: Add a 'Diagnostic User Name' to the AF token for better diagnosibility

The fall back in case identity does not contain a name identifier is to hash the entire claims identity. In order for the app developer to have a better understanding of what is going wrong, it is desirable to have a diagnostic only user name as part of the AF token.  This serves only for error and tracing scenarios.
| 223 | 10506 | Reported By: Harsh Gupta |

We decided not to do this.


chore: allow commits to the BUGFIX branch to be deployed on PR merges

:tada: This PR is included in version 2.0.0-alpha.18 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-alpha.169 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.206 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.23 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.4.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.13.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.5.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.13.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


no. of attempts

how many maximum attempts it will take

If it works, something about 350k shots.


⚠ Bofrost.be has degraded performance

In 3aca09d, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 882 ms

Resolved: Bofrost.be performance has improved in 3267056.


[WIP] EKS: add version tag to stack

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?

Why?

Additional context

Checklist

[ ] Implementation tested (with at least one cloud provider)
[ ] Error handling code meets the guideline
[ ] Logging code meets the guideline (TODO)
[ ] OpenAPI and Postman files updated, client regenerated (make generate-client) (if needed)
[ ] User guide and development docs updated (if needed)
[ ] Related Helm chart(s) updated (if needed)

To Do

[ ] If the PR is not complete but you want to discuss the approach, list what remains to be done here

Can we close this?


Change default branch to 8.10

Now that GHC 8.10.1 has been released, Haddock's default branch should be 8.10 (so that people stop opening PRs against 8.8 😛). I don't have the Github permissions to do this.
/cc @cartazio @emilypi @hvr

Ping

HVR is currently making headway on a particularly difficult hackage server migration (to make use of Cabal 3.x). I imagine this is related. @Herbert?
On Sun, Apr 05, 2020 at 10:40 PM, Alec Theriault <<EMAIL_ADDRESS>> wrote:

Ping
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/haskell/haddock/issues/1150#issuecomment-609536756 ) ,
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ACPZRTSWAJMRGBZDD3QYKETRLE6LFANCNFSM4LWCPACA
).

Sorry, missed the notification; it's done now! :-)

Thanks!
Sent from my iPhone

On Apr 6, 2020, at 02:08, Herbert Valerio Riedel<EMAIL_ADDRESS>wrote:

Closed #1150.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Assistant Responds Before Function Call Completes

We are encountering an issue with function calls where the assistant responds prematurely, before the function has a chance to complete its execution and return the correct data.
This behavior occurs:

Locally, when running the repository from this GitHub repo.
On our own staging servers.
Also, here is an example of the bug on the "Developers Digest" YouTube channel showcasing this repo: Video link at 4:25 mark.

Example scenario:

I ask:
"What's the weather in Seattle?"
The assistant responds prematurely:
"It seems I'm having a little trouble fetching the weather right now. How about I try something else for you? Is there anything else I can help you with?"
However, the function call is triggered correctly and returns the weather.
I ask again:
"What's the weather in Seattle?"
This time, the assistant correctly responds with the weather.

Additional Information:
We are using the latest version of the repository code and have observed this behavior consistently.
Is there any way around this issue? It would be helpful if there were a way to configure something like functionTimeout: 10 seconds, which would allow more time for the function call to complete before the assistant responds.
One potential workaround could be to ignore all responses from OpenAI until the function call finishes, and then manually trigger a response.create event afterward. However, this approach feels a bit hacky. Is there a more reliable or recommended solution to handle this?

I Have the same Issue. I call a API Request which need 5 to 10 seconds after 1 seconde i get the message He dont can use Internal .... I debug this in the Libary:
  this.realtime.on('server.response.output_item.done', async (event) => {       const { item } = handlerWithDispatch(event);       if (item.status === 'completed') {         this.dispatch('conversation.item.completed', { item });       }       if (item.formatted.tool) {          callTool(item.formatted.tool);       }     });
This Event is calling for execute the function and after 1 second later again.. But with the todo to complete the message.
I think is Important to make the Timout higher. and Maybe to give the opportunity to config this. Thanks @TheoGibbons  you are opend this ticket. Lets hope this will will fixed very fast. Or we get a workaround. Thanks guys for your support.

Got same issue here when using relay websocket server, here is the log:
user
How's the weather in Tokyo?
function call
get_weather({"lat":35.6895,"lng":139.6917,"location":"Tokyo"})
function call output
{"error":"Tool \"get_weather\" has not been added"}
assistant
Oh, it looks like I couldn't retrieve the weather information for Tokyo right now. Is there anything else I can help you with?
function call output
{"latitude":35.7,"longitude":139.6875,"generationtime_ms":0.028967857360839844,"utc_offset_seconds":0,"timezone":"GMT","timezone_abbreviation":"GMT","elevation":40,"current_units":{"time":"iso8601","interval":"seconds","temperature_2m":"°C","wind_speed_10m":"km/h"},"current":{"time":"2024-10-21T04:30","interval":900,"temperature_2m":18.7,"wind_speed_10m":6.1}}

interestingly, it has "function call output" event twice, the first one comes earlier than the response, said "get_weather not registered", then the second comes later.
Then I doubt that maybe I should register the function call on relay server side RealtimeClient too, basically add below code to the relay.js:
client.addTool(
      {
        name: 'get_weather',
        description:
          'Retrieves the weather for a given lat, lng coordinate pair. Specify a label for the location.',
        parameters: {
          type: 'object',
          properties: {
            lat: {
              type: 'number',
              description: 'Latitude',
            },
            lng: {
              type: 'number',
              description: 'Longitude',
            },
            location: {
              type: 'string',
              description: 'Name of the location',
            },
          },
          required: ['lat', 'lng', 'location'],
        },
      },
      async ({ lat, lng, location }) => {
        const result = await fetch(
          `https://api.open-meteo.com/v1/forecast?latitude=${lat}&longitude=${lng}&current=temperature_2m,wind_speed_10m`
        );
        const json = await result.json();
        const temperature = {
          value: json.current.temperature_2m,
          units: json.current_units.temperature_2m,
        };
        const wind_speed = {
          value: json.current.wind_speed_10m,
          units: json.current_units.wind_speed_10m,
        };
        return json;
      }
    );

I tried that then it worked everytime. like log below:
user
How's the weather like in Tokyo?
function call
get_weather({"lat":35.6895,"lng":139.6917,"location":"Tokyo"})
function call output
{"latitude":35.7,"longitude":139.6875,"generationtime_ms":0.03504753112792969,"utc_offset_seconds":0,"timezone":"GMT","timezone_abbreviation":"GMT","elevation":40,"current_units":{"time":"iso8601","interval":"seconds","temperature_2m":"°C","wind_speed_10m":"km/h"},"current":{"time":"2024-10-21T04:30","interval":900,"temperature_2m":18.7,"wind_speed_10m":6.1}}
function call output
{"latitude":35.7,"longitude":139.6875,"generationtime_ms":0.016927719116210938,"utc_offset_seconds":0,"timezone":"GMT","timezone_abbreviation":"GMT","elevation":40,"current_units":{"time":"iso8601","interval":"seconds","temperature_2m":"°C","wind_speed_10m":"km/h"},"current":{"time":"2024-10-21T04:30","interval":900,"temperature_2m":18.7,"wind_speed_10m":6.1}}
assistant
In Tokyo, it's currently 18.7°C with a gentle wind at about 6.1 km/h. How does that weather sound to you?

again, got 2 "function call output" event printed, 1 from relay server RealtimeClient (used for create response), 1 from client side RealtimeClient (used for updating UI) I guess...
so as a conclusion, if using relay server, we need to register the function call on the relay server side RealtimeClient, and if need to update UI (like the sample code updating map pins and stuff), we register the same function call on client side RealtimeClient.

hopefully above issue and fix is temporary and openai can resolve this in the new version of '@openai/realtime-api-beta' library.

A Solution using  a patch
was posted for this on YouTube @fullstackdesk
On Mon, Oct 21, 2024 at 12:51 AM Joe M @.***> wrote:

hopefully above issue and fix is temporary and openai can resolve this in
the new version of @.***/realtime-api-beta' library.
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-realtime-api-beta/issues/49#issuecomment-2425579760,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHADPMOAXSK4SYFM4AXN3GTZ4SB3NAVCNFSM6AAAAABQGVXKUCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRVGU3TSNZWGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks all I've created a pull request with the change.
In the mean time you can update your package.json file to point to the fork which should work around the issue.
"@openai/realtime-api-beta": "github:TheoGibbons/openai-realtime-api-beta#bd4e74aaa20d09a069f058eb6aaa76205871f28a",


DI-1299 Check Dias somalier results in GRCh37 vs GRCh38

Add Python script to find and read in Multiqc_somalier.samples.tsv files in GRCh37 and GRCh38 and find any samples with mismatches for Predicted_Sex between genome builds
Add config files given as input to script for TWE and CEN
Add README for script
Update repo README for this request

This change is

Hello @rklocke! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file DI-1299/compare_b37_and_b38.py:

Line 141:53: E203 whitespace before ':'


Add document type to Whitehall document export

In order to correctly handle migration we need to ensure that we get
the mapped value for a document type from the API export instead of
just the type ID, as Content Publisher has no knowledge of the
mappings used by Whitehall.
This commit

Introduces a check to append the required data for corporate
information, speeches, publications and news stories to the API
export for each edition where relevant.
Introduces a test to this affect
Fixes an exporter test that was not failing, but was passing
with the wrong comparison (it was comparing two nils, which are
still equal but not what was intended by the test).

https://trello.com/c/Z1yHfuJw/1092-add-document-type-to-whitehall-document-export

Hey @Tetrino, looks good just got some small suggestions -
You could potentially dry up the provide_doctype_information method with the snippet below and maybe rename it to just doctype_information as the other assignments don't have a verb prefix, lastly I think just returning the key is fine instead of the whole ~Type as a lot of the data in this object is irrelevant. -
def doctype_information(edition)
  if edition.respond_to?("#{edition.class.to_s.underscore}_type")
    edition.public_send("#{edition.class.to_s.underscore}_type").key
  end
end


Unable to install through easy_install

Hi. I've tried installing the package with easy_install-3.4 (for some reason I prefer easy_install over pip), but it has reported me the following error:
$ sudo easy_install-3.4 easyplot
Searching for easyplot
Reading https://pypi.python.org/simple/easyplot/
Best match: EasyPlot 1.0.0
Downloading https://pypi.python.org/packages/source/E/EasyPlot/EasyPlot-1.0.0.zip#md5=31c7bac33a45d02ddc97c8c632f14510
Processing EasyPlot-1.0.0.zip
Writing /tmp/easy_install-kcwzc18c/EasyPlot-1.0.0/setup.cfg
Running EasyPlot-1.0.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-kcwzc18c/EasyPlot-1.0.0/egg-dist-tmp-3k6fu_ge
error: SandboxViolation: mkdir('/root/.config/matplotlib', 511) {}

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

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

Installing via pip3.4 works, though. Is there something wrong with your ./setup.py script?

Pavel, I will need to look into this more carefully. Do you have matplotlib
installed locally? If not, that could very well be the issue.
On Sep 26, 2015 12:53 PM, "Pavel Kretov"<EMAIL_ADDRESS>wrote:

Hi. I've tried installing the package with easy_install-3.4 (for some
reason I prefer easy_install over pip), but it has reported me the
following error:
$ sudo easy_install-3.4 easyplot
Searching for easyplot
Reading https://pypi.python.org/simple/easyplot/
Best match: EasyPlot 1.0.0
Downloading https://pypi.python.org/packages/source/E/EasyPlot/EasyPlot-1.0.0.zip#md5=31c7bac33a45d02ddc97c8c632f14510
Processing EasyPlot-1.0.0.zip
Writing /tmp/easy_install-kcwzc18c/EasyPlot-1.0.0/setup.cfg
Running EasyPlot-1.0.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-kcwzc18c/EasyPlot-1.0.0/egg-dist-tmp-3k6fu_ge
error: SandboxViolation: mkdir('/root/.config/matplotlib', 511) {}
The package setup script has attempted to modify files on your system
that are not within the EasyInstall build area, and has been aborted.
This package cannot be safely installed by EasyInstall, and may not
support alternate installation locations even if you run its setup
script by hand.  Please inform the package's author and the EasyInstall
maintainers to find out if a fix or workaround is available.
Is there something wrong with your ./setup.py script?
—
Reply to this email directly or view it on GitHub
https://github.com/HamsterHuey/easyplot/issues/9.

No, I'm pretty sure I've installed matplotlib. Moreover, when I tried to make a clean virtual-env installation, I ran into trobles once again:
$ virtualenv-3.4 ~/test-venv-3.4
$ ~/test-venv-3.4/bin/easy_install matplotlib
(Lots of output)
$ ~/test-venv-3.4/bin/easy_install easyplot
Searching for easyplot
Reading https://pypi.python.org/simple/easyplot/
Best match: EasyPlot 1.0.0
Downloading https://pypi.python.org/packages/source/E/EasyPlot/EasyPlot-1.0.0.zip#md5=31c7bac33a45d02ddc97c8c632f14510
Processing EasyPlot-1.0.0.zip
Writing /tmp/easy_install-9QtKuA/EasyPlot-1.0.0/setup.cfg
Running EasyPlot-1.0.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-9QtKuA/EasyPlot-1.0.0/egg-dist-tmp-Zx5zkw
Traceback (most recent call last):
  File "test-venv-3.4/bin/easy_install", line 11, in <module>
    sys.exit(main())
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 2245, in main
    distclass=DistributionWithoutHelpCommands, **kw
  File "/usr/lib64/python2.7/distutils/core.py", line 151, in setup
    dist.run_commands()
  File "/usr/lib64/python2.7/distutils/dist.py", line 953, in run_commands
    self.run_command(cmd)
  File "/usr/lib64/python2.7/distutils/dist.py", line 972, in run_command
    cmd_obj.run()
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 380, in run
    self.easy_install(spec, not self.no_deps)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 629, in easy_install
    return self.install_item(spec, dist.location, tmpdir, deps)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 659, in install_item
    dists = self.install_eggs(spec, download, tmpdir)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 842, in install_eggs
    return self.build_and_install(setup_script, setup_base)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1070, in build_and_install
    self.run_setup(setup_script, setup_base, args)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1056, in run_setup
    run_setup(setup_script, args)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 240, in run_setup
    raise
  File "/usr/lib64/python2.7/contextlib.py", line 35, in __exit__
    self.gen.throw(type, value, traceback)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 193, in setup_context
    yield
  File "/usr/lib64/python2.7/contextlib.py", line 35, in __exit__
    self.gen.throw(type, value, traceback)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 152, in save_modules
    yield saved
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 126, in __exit__
    self._saved = UnpickleableException.dump(type, exc)
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
  File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
    return cls.dump(cls, cls(repr(exc)))
MemoryError

I don't even know what to think about all of this.

Pavel, thanks for the feedback. I'll have to give this a shot on my machine
to see if I can reproduce the issue. I'm not seeing a clear indication of
what might be causing it from the logs you posted.
If you're still interested in trying out EasyPlot, I'd recommend simply
downloading the EasyPlot.py file and having that somewhere accessible to
your Python install. This is a pretty light weight and simple library and
is contained in one file, so it should be very easy to use even without
going the route of pip/easy install. I used this as a learning experience
to figure out how to put something up on PyPi and have it installable via
pip and easy install. Clearly the latter did not pan out though I'm pretty
sure it worked well for me locally when I first wrote this library.
On Tue, Sep 29, 2015 at 9:25 AM, Pavel Kretov<EMAIL_ADDRESS>wrote:

No, I'm pretty sure I've installed matplotlib. Moreover, when I tried to
make a clean virtual-env installation, I ran into trobles once again:
$ virtualenv-3.4 ~/test-venv-3.4
$ ~/test-venv-3.4/bin/easy_install matplotlib
(Lots of output)
$ ~/test-venv-3.4/bin/easy_install easyplot
Searching for easyplot
Reading https://pypi.python.org/simple/easyplot/
Best match: EasyPlot 1.0.0
Downloading https://pypi.python.org/packages/source/E/EasyPlot/EasyPlot-1.0.0.zip#md5=31c7bac33a45d02ddc97c8c632f14510
Processing EasyPlot-1.0.0.zip
Writing /tmp/easy_install-9QtKuA/EasyPlot-1.0.0/setup.cfg
Running EasyPlot-1.0.0/setup.py -q bdist_egg --dist-dir /tmp/easy_install-9QtKuA/EasyPlot-1.0.0/egg-dist-tmp-Zx5zkw
Traceback (most recent call last):
File "test-venv-3.4/bin/easy_install", line 11, in
sys.exit(main())
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 2245, in main
distclass=DistributionWithoutHelpCommands, **kw
File "/usr/lib64/python2.7/distutils/core.py", line 151, in setup
dist.run_commands()
File "/usr/lib64/python2.7/distutils/dist.py", line 953, in run_commands
self.run_command(cmd)
File "/usr/lib64/python2.7/distutils/dist.py", line 972, in run_command
cmd_obj.run()
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 380, in run
self.easy_install(spec, not self.no_deps)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 629, in easy_install
return self.install_item(spec, dist.location, tmpdir, deps)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 659, in install_item
dists = self.install_eggs(spec, download, tmpdir)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 842, in install_eggs
return self.build_and_install(setup_script, setup_base)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1070, in build_and_install
self.run_setup(setup_script, setup_base, args)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/command/easy_install.py", line 1056, in run_setup
run_setup(setup_script, args)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 240, in run_setup
raise
File "/usr/lib64/python2.7/contextlib.py", line 35, in exit
self.gen.throw(type, value, traceback)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 193, in setup_context
yield
File "/usr/lib64/python2.7/contextlib.py", line 35, in exit
self.gen.throw(type, value, traceback)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 152, in save_modules
yield saved
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 126, in exit
self._saved = UnpickleableException.dump(type, exc)
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
File "/home/firegurafiku/test-venv-3.4/lib/python2.7/site-packages/setuptools/sandbox.py", line 110, in dump
return cls.dump(cls, cls(repr(exc)))
MemoryError
I don't even know what to think about all of this.
—
Reply to this email directly or view it on GitHub
https://github.com/HamsterHuey/easyplot/issues/9#issuecomment-144058080.


Review ACWR

Fill in the sources for ACWR
Review & Update ACWR formula as/if needed

Updated. @markwestcott34 @kopegios please review and close issue if ok.


facebook/create-react-app

We are on a mission to help collaborative groups collect and spend money transparently.

We are on a mission to help collaborative groups collect and spend money transparently.

npm install --save react-router-dom


Get all API tested

Due to lots of schema mismatch in official document & real response, all APIs should be tested and schema updated to it.

Should find some VCR library for test caching.


First run as sudo

Describe the bug
I couldn't run the server once installed, the problem was that it uses the port 80, which is only usable by root. Therefore, I had to run it as sudo sudo npm start. Go to pigallery2 settings, toggle the "advanced" switch, change the port (to 8080, for example) and restart the server. Now it can run as normal user.
Maybe this detail can be mentioned in the instructions.
Environment:

HW: Raspberry Pi 3 B+
OS: clean installation of "Raspbian Buster with desktop", release 2019-09-26. Updated to the date of the post.

Used app version:

from release v1.70, following the installation instructions of the repository.

Thanks.

Thanks for the note, I'll add it to the Readme

I've updated the readme. Closing this.
Thanks for the report.


Use a label selector to ignore completed pods

The pod watch currently sees all workflow related pods, including ones that have long completed. This will make the resync very expensive. The controller should label completed pods so that they are ignored by the watch, and lessen the load on the controller.

Commit 205e5cb introduced the completed label to pods which controller will use to filter out pods it no longer cares about.


fix: add count to the data block

Description
Added a count = var.verify_worker_network_readiness ? 1 : 0 on that data block and creating a private-only cluster in the apply_taints example which runs a test in the other_tests.
Ref: https://github.com/terraform-ibm-modules/terraform-ibm-base-ocp-vpc/pull/68#issuecomment-1479910490
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[x] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[x] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

I added the change to an existing example of apply_taints, it is being tested in the other_tests.
Should I move this to pr_test?

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

Probably fine for now - but we may want to change the name of the variable to "skip_connection_to_kube_master" as the data block is also used to force rbac sync https://github.com/terraform-ibm-modules/terraform-ibm-base-ocp-vpc/blob/main/main.tf#L198

Probably fine for now - but we may want to change the name of the variable to "skip_connection_to_kube_master" as the data block is also used to force rbac sync https://github.com/terraform-ibm-modules/terraform-ibm-base-ocp-vpc/blob/main/main.tf#L198


Email Server-wide Vs Per-user configuration

I've noticed that when server-wide email configuration is done then Per-user don't work. Traccar always sends email using the default, not the user configured SMTP server. Is that the desired behavor?
A user might want to use their own SMTP server, not the default server-wide. Is that possible to change this behavior

@Abyss777, do you remember if there was any specific reason we decided to prefer server config? Would you see any problem in reversing this?

If I'm not mistaken, we just considered two use cases:

Traccar used in company, and it is configured by admin, per user configuration is unwanted
Traccar is public service, and every user have to configure own smtp server.

I think we can add some switch in config to allow traccar use user configuration even if it has server one.

There is already a flag to force server configuration, so we should probably change the priority to user config unless that flag is set.

There might be a use case when customer do not want force all settings and want not allow override smtp server. For example company with branches in different time zones or different countries.

Force server settings is a separate flag and is disabled by default.

Yes, but it is used for determine timezone for reports and respected in web client for units and other user/server settings.

I see three options:

Keep current behaviour of preferring config
Change behaviour to prefer user settings
Second one plus force flag

Third one seems like the best option. Any other options?

Yes, I think...

Second one with "Allow per user SMTP configuration" flag. False by default, but true in default config.

Because we do not have a way totally disable email notifications. I know one use case when traccar is the only server connected to internet, and email notifications can be considered as information leakage.
I understand that email costs much less than SMS, but there is other consequences of uncontrolled email sending, like pure sender reputation (SPAM) or network load or others.

I guess that's also an option, but probably better to make it reverse like mail.smtp.ignoreUserConfig.

OK. I'll take it.


Buggy horizontal mode

When setting gridView.layoutDirection = AQGridViewLayoutDirectionHorizontal the display of the items at runtime seems to be incorrect.
Some of the items in the grid simply are not  displayed. As a matter of fact, they sometimes do get displayed but sometimes not...
I used the SpringBoad demo to test that. In the demo I set _gridView.layoutDirection = AQGridViewLayoutDirectionHorizontal as well as _gridView.scrollEnabled = YES and things kind of look funky....
The first 'page' looks ok, but as I start scrolling right, some of the items in the first row are not displayed (items in the second row are displayed perfectly). If I scroll back left again, then surprisingly, the first row items that were not getting displayed before, are now displayed...
Here are a few screenshots that demonstrate:
First page is ok: https://skitch.com/rantav/g8f67/springboard-first-page-ok
The third item is now missing: https://skitch.com/rantav/g8f9f/springboard-2-is-missing
The third item is now OK, after I scroll right and left again: https://skitch.com/rantav/g8f9a/springboard-2-is-now-displayed
I created a gist with the code changes made to the SpringBoard demo. See https://gist.github.com/1350884
Search for "MY ADDITION"
Is this a bug, or am I missing something in the correct usage of the component?
BTW, I tested on my simulator. iOS 4.3.
Thanks!

Hi, any comment on this? Does this bug ring a bell somehow or am I the first to see it?
What's the maintenance status of this project?
If unmaintained, then by any chance would you be able to recommend another grid view that supports horizontal scrolling?
Thanks!

The second question is a very good one.


HomeComponent: use 'isAuthenticated' value from the HomeAuthResolver

Thanks for the PR!
I intentionally did not grab the resolved value of isAuthenticated -- the home page UI would not respond to changes in authentication state (i.e. if we had a logout button in the navbar, the home page would stay the same even after logging out). This is why I used an observable in onInit to subscribe observable itself.
So why even have the resolve be there if we're just subscribing anyways? Because on a hard page load, with no resolve, an authenticated user will have a race condition between loading their feed vs global posts. So having the resolve ensures that authentication state has been determined before allowing the user to navigate to the home page.
This is the most elegant solution I could come up with, but happy to hear your thoughts.

Ok, it makes sense if you are going to move logout button to the navbar. Actually I hadn't expected to find it in the settings :) Also a comment in onInit would help to clear this confusion.
You could merge only the second commit which removes unused imports and close this issue.

Great feedback -- will add a comment to the onInit fn (was planning on covering that in docs/etc as well). Merged your unused imports fix. Thanks for taking the time to help out!


update from upstream

chore(deps): update returntocorp/semgrep docker digest to 4010a43
chore(deps): update alpine:3.16.2 docker digest to d6be110
chore(deps): update alpine:3.16.2 docker digest to 65a2763
chore(deps): update returntocorp/semgrep docker digest to b774330
fix: set correct nextest config

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


Integration with Wizz Air

WHY
Wizz Air is one of the most known airlines in Europe.
HOW
Web API endpoint: https://be.wizzair.com/9.14.1/Api/customer/
some known paths: /login, /profile, /mybookings

If the integration is welcomed, I could work on it

Go for it ☺ the more integrations the better. I have personally a weak
spot for utilities, but do what you think is fun to work on!
On Mon, Jul 29, 2019, 14:21 Patricia Wanat<EMAIL_ADDRESS>wrote:

If the integration is welcomed, I could work on it
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tmrowco/tmrowapp-contrib/issues/106?email_source=notifications&email_token=AAT333VEHLLRZ2F34DDOJATQB3OD7A5CNFSM4IHRYPJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3AQZSY#issuecomment-515968203,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAT333XBKGMFJ52YJHRYALLQB3OD7ANCNFSM4IHRYPJQ
.

Hi @lauvrenn , someone seems to have issues with the integration. Can you check if it's still working on your side?

hey @martincollignon yeah, I could take a look! Problem is I don't have the credentials with flight history anymore though :grimacing:


Active Users as a Dashboard Item

Display how many users and super users are logged in at any given time.  A count of logged in users and super users would be displayed by organization.
Reference item 26

This is still a viable request for myself.  I want to know how many users are using ANET before I shut down the ANET service.  I am keeping this open, but will solve this problem with a PowerShell script.

This issue can be closed.  The "User Activities" and "Recent Users" functionality has been added for ANET administrators.


Accellerating Editor scrolling

Scrolling through the list of items in the Editor can take a long time if there are many. I should create some kind of scroll acceleration (increase scrolling speed when someone keeps scrolling for more than ~3 seconds) to improve this.

This will be implemented in Editor 9.3.


UPDATED docker/compose build and run

This should be a complete set of build and run scripts, I've tested it myself but please do test it if theres any bugs.

@telostia thank you for the updates!

@telostia thank you for the updates!