Is my code wrong or I don't konw the scatter's mechanism?

❓ Questions & Help

I want to use scatter to generate a matrix.
My code is:
x, (edge_index, edge_wight), batch = data.x, add_remaining_self_loops(data.edge_index), data.batch

xi, xj = x[edge_index[0]], x[edge_index[1]]
x_neig = xi-xj

#compute max(xi-xj)
x_max = scatter(x_neig, edge_index[0], dim=0, reduce='max')

I think x_neig's dimension is(num_nodes,num_feature)
But it reported error:
The above operation failed in interpreter.
Traceback (most recent call last):
  File "\lib\site-packages\torch_scatter\scatter.py", line 31
                out: Optional[torch.Tensor] = None,
                dim_size: Optional[int] = None) -> torch.Tensor:
  File "\lib\site-packages\torch_scatter\scatter.py", line 12, in scatter_sum
                out: Optional[torch.Tensor] = None,
                dim_size: Optional[int] = None) -> torch.Tensor:
    index = broadcast(index, src, dim)
            ~~~~~~~~~ <--- HERE
    if out is None:
        size = src.size()
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_scatter\utils.py", line 13, in broadcast
    for _ in range(src.dim(), other.dim()):
        src = src.unsqueeze(-1)
    src = src.expand_as(other)
          ~~~~~~~~~~~~~ <--- HERE
    return src

And I ues torchsnooper.snoop() to tracked tensor:
tarting var:.. x = tensor<(4021, 128), float32, cuda:0, grad>
Starting var:.. edge_index = tensor<(2, 11652), int64, cuda:0>
Starting var:.. edge_attr = None
Starting var:.. batch = tensor<(4021,), int64, cuda:0>
22:14:17.365985 call        19     def forward(self, x, edge_index, edge_attr=None, batch=None):
22:14:17.367979 line        20         if batch is None:
22:14:17.371970 line        23         x = x.unsqueeze(-1) if x.dim() == 1 else x
22:14:17.373963 line        24         xi, xj = x[edge_index[0]], x[edge_index[1]]
New var:....... xi = tensor<(11652, 128), float32, cuda:0, grad>
New var:....... xj = tensor<(11652, 128), float32, cuda:0, grad>
22:14:17.374960 line        25         x_neig = xi-xj
New var:....... x_neig = tensor<(11652, 128), float32, cuda:0, grad>
22:14:17.377953 line        28         x_neig_max = scatter(x_neig, edge_index[0], dim=0, reduce='max')
New var:....... x_neig_max = tensor<(4014, 128), float32, cuda:0, grad>

Can you tell me why?
Thank you!

Seems like a TorchScript issue. Seems weird, as I cannot reproduce and it does only occur on that dataset.
You can try to remove the @torch.jit.script annotations in D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_scatter\utils.py and D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_scatter\scatter.py to see if that fixes the issue.

If I comment all @torch.jit.script,it report error:
Traceback (most recent call last):
  File "main.py", line 87, in <module>
    out = model(data)
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch\nn\modules\module.py", line 532, in __call__
    result = self.forward(*input, **kwargs)
  File "D:\muqian\code\SAGPool-master333333333333333333\SAGPool-master\networks.py", line 41, in forward
    x = F.relu(self.conv1(x, edge_index))
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch\nn\modules\module.py", line 532, in __call__
    result = self.forward(*input, **kwargs)
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_geometric\nn\conv\gcn_conv.py", line 102, in forward
    self.node_dim), edge_weight, self.improved, x.dtype)
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_geometric\nn\conv\gcn_conv.py", line 80, in norm
    deg = scatter_add(edge_weight, row, dim=0, dim_size=num_nodes)
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_scatter\scatter.py", line 31, in scatter_add
    return scatter_sum(src, index, dim, out, dim_size)
  File "D:\anna\Anaconda\envs\torch1.4.0\lib\site-packages\torch_scatter\scatter.py", line 16, in scatter_sum
    size[dim] = dim_size
TypeError: 'torch.Size' object does not support item assignment

If I only comment utils and scatter.scatter_max it stall get the same error of dimension.
Do you have another methods?It bothered me for a few days.
Thank you very match!

You can try replacing line 14 in scatter.py with size = list(src.size()). This might be a hacky fix, but I am absolutely unsure what may cause this.

You can try replacing line 14 in scatter.py with size = list(src.size()). This might be a hacky fix, but I am absolutely unsure what may cause this.

I'm sorry.It's still get an same error of dimension.
Is line14 used to be size = src.size() in scatter_sum?
I try to print the size(after replace) and I find the size from 11659 become 11652(I use ---------- to note).
------ [11659]
------ [11659, 128]
Starting var:.. self = SAGPool(  (score_layer): GCNConv(128, 1))
Starting var:.. x = tensor<(4021, 128), float32, cuda:0, grad>
Starting var:.. edge_index = tensor<(2, 11652), int64, cuda:0>
Starting var:.. edge_attr = None
Starting var:.. batch = tensor<(4021,), int64, cuda:0>
18:44:46.323857 call        19     def forward(self, x, edge_index, edge_attr=None, batch=None):
18:44:46.325852 line        20         if batch is None:
18:44:46.326848 line        23         x = x.unsqueeze(-1) if x.dim() == 1 else x
18:44:46.327847 line        24         xi, xj = x[edge_index[0]], x[edge_index[1]]
New var:....... xi = tensor<(11652, 128), float32, cuda:0, grad>
New var:....... xj = tensor<(11652, 128), float32, cuda:0, grad>
18:44:46.328843 line        25         x_neig = xi-xj
New var:....... x_neig = tensor<(11652, 128), float32, cuda:0, grad>
18:44:46.331836 line        28         x_neig_max = scatter(x_neig, edge_index[0], dim=0, reduce='max')
New var:....... x_max = tensor<(4014, 128), float32, cuda:0, grad>
18:44:46.335856 line        31         if x.size(0) == 1:
18:44:46.343804 line        49             score = selx.score_layer(x_max,edge_index).squeeze()
------ [11652]
------ [11652, 1]
New var:....... score = tensor<(4014,), float32, cuda:0, grad>
18:44:46.352780 line        52             perm = topk(score, self.ratio, batch)
18:44:46.357766 exception   52             perm = topk(score, self.ratio, batch)

I really don't know why.
Could you figure it out?

Ah, I see. You are talking about a wrong output shape, so the computation runs through by now. You should pass the dim_size attribute to the scatter call, otherwise it guesses the output shape based on the highest value it finds in edge_index:
x_max = scatter(x_neig, edge_index[0], dim=0, dim_size=x.size(0), reduce='max')

x_max = scatter(x_neig, edge_index[0], dim=0, dim_size=x.size(0), reduce='max')

You are right!Grateful to you!
Let me think about the principle again.
I use func add_remain_self_loop.So I think the highest value of edgae_index is equal to highest value of num_nodes.But after pooling, there maybe remove the nodes that highest value of index.So we need use dim_size.
Am I right?

I see. You should also pass num_nodes to add_remaining_self_loops.

I see. You should also pass num_nodes to add_remaining_self_loops.

add_remaining_self_loop(data.num_nodes),is this your mean?

add_remaining_self_loops(data.edge_index, num_nodes=data.num_nodes)

add_remaining_self_loops(data.edge_index, num_nodes=data.num_nodes)

If like this, I don't need use dim_size?
Anyway, thank you so much.

You should still use dim_size for faster computation, but it is not mandatory.

yes!Thank you for your precious answer!
I wish you have a happy life!


Beacon tweak: removed redundant header for browse_signup.html

What Changed
Header text for the subscribe form for the opportunities listing page, and individual opportunities pages was embedded in browse_signup.html, which made it impossible to title the form appropriately for its context. I simply took it out.
What's Needed
N/A
Screenshots

(Contract shown is test data from existing RFP: http://apps.pittsburghpa.gov/omb/RFP_Final.pdf)

I'm not really sure what you changed here. The purpose of the browse_signup file is to entirely contain that sign-up form, including the title. Can you maybe do a before/after screenshot?

Before screenshot ^^ the title for the signup form doesn't work in the context of a single opportunity.

Ah ok, that makes sense. Can you rename the file to something a bit more descriptive then? Maybe just like opportunity_signup_form or something like that. You'll also have to update the include references.

:+1: thanks @shellicious


"Bujoreni" is listed in the drop-down menu

Description:
"Bujoreni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Bujoreni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Bujoreni, Argeș (Buzoești)".


Improve Container API

Hi,
First off, great work. It looks nice.
I was wondering when looking at the Container API, is there any reason the subscriptions  field on the createContainer method doesn't return an object which gets mapped to this.props ?
The subscriptions would still be auto-fired/auto-stopped, but you could also unsubscribe from them at any time using this.props.subscriptionName.dispose().
You would also be able to inspect any field you would have returned in the sendSubscription method of the NetworkLayer, e.g. the underlying socket.io request providing useful information like the connection status.
To illustrate :

const MyComponent = ({ postCommentsSubscription }) => { /* Do something */ }

const MyContainer = RelaySubscriptions.createContainer(MyComponent, {
    fragments: {
        post: () => /* ... */
    },

    subscriptions: (props) => ({
        postCommentsSubscription: new PostCommentsSubscription({ postID: props.post.id }),
    }),
})

Aside from this, do you have any information on if and when this repo will be merged into relay official repo ? It would make sense !

Hi and thanks for the proposal.
I think that this is already doable with the current imperative API.
The declarative container API is not intended to solve all use-cases only the most common. If you want more advanced control you should use the imperative API instead which gives you more fine grained control of the subscription life-cycle.
cc @taion

Aside from this, do you have any information on if and when this repo will be merged into relay official repo ? It would make sense !

This repo will not be merged into Relay Core. You can read more about why in relay#1298 and relay#541.

I think disposing a managed subscription would also probably be a bad idea – the container wouldn't have any idea whether that subscription got manually disposed or not, and the associated use case is enough of an edge case that I don't think it's worth explicitly supporting.

Ok, sounds good to me.
I will reopen when I find a good example use-case.
Thanks for the replies 👍


Issue while uploading the file

Hi Team,
I always get the message "Upload failed File type must be application/octet-stream". When i try to upload the files. Can i have the detailed steps to use this?

@vanimesh We are working on updating documentation for the latest release. Are you using REST API or Data Source requests? Where do you see this error?
For REST API, you can choose Content-Type application/json to send the file as Base64 encoded.
If you choose multipart, it's sent as a binary file. Seems to be your case.

Hi @mikhail-vl , the message pops up right after I upload the file

@vanimesh Please answer all questions.

We are using API. I am sorry i am beginner on this things. My goal is to upload the csv file and when clicked it has to call a restAPI which inturn calls the python script.

@vanimesh seems to be the API to where you try to send the csv file doesn't accept the real MIME Type of file (text/csv). Please adjust your API to accept text/csv or if you use any existing API/service please let us know and we will consider to add supporting it

Hi @asimonok , i tried to check and script seems to be fine. But seems like the plugin itself is not sending any data . Its just calling the api. When i tried to print the post data from the script its empty everytime.


updated package.json

updated package merge from 1.1.3 to 1.2.0
Npm link :  https://www.npmjs.com/package/merge
Repository link : https://github.com/yeikos/js.merge/releases

Thank you

Thank you


NullPointerException when email doesn't contain a proper date

When i try it from the commandline whit my Spamfolder its looks valid but after some time its stop whit a NullPointerException.

win7
java version "1.8.0_60"
Java(TM) SE Runtime Environment (build 1.8.0_60-b27)
Java HotSpot(TM) 64-Bit Server VM (build 25.60-b23, mixed mode)

...
2015-09-12 05:39:54 INFO  IMAPCrypt:327 -   Did not match all the filters
2015-09-12 05:39:55 INFO  IMAPCrypt:315 - Message #172 on Sat Sep 05 16:22:42 CEST 2015: VIDEO anschauen: Software verdi
ent f³r Sie 439 Euro pro Tag
2015-09-12 05:39:55 INFO  IMAPCrypt:327 -   Did not match all the filters
2015-09-12 05:39:55 INFO  IMAPCrypt:315 - Message #173 on Sat Sep 05 17:14:35 CEST 2015: Vom Bettler zum Multimillionõr
in 2 Jahren! VIDEO zum NACHMACHEN
2015-09-12 05:39:55 INFO  IMAPCrypt:327 -   Did not match all the filters
2015-09-12 05:39:56 INFO  IMAPCrypt:315 - Message #174 on null: Be young, have fun, taste Love
java.lang.NullPointerException
        at org.dhs.chrislee.DateBasedMessageEvaluationCallback.isEncryptableMessage(DateBasedMessageEvaluationCallback.java:60)
        at org.dhs.chrislee.IMAPCrypt.encrypt(IMAPCrypt.java:325)
        at org.dhs.chrislee.IMAPCrypt.main(IMAPCrypt.java:589)

If i try the gui at win7
C:\Users\root\Downloads>java -jar imapcrypt-2.0.2.jar
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
        at org.dhs.chrislee.imapgui.KeySelectorPanel.<init>(KeySelectorPanel.java:52)
        at org.dhs.chrislee.imapgui.KeySelectorPanel.<init>(KeySelectorPanel.java:38)
        at org.dhs.chrislee.imapgui.groups.RecipientPanel.<init>(RecipientPanel.java:36)
        at org.dhs.chrislee.imapgui.IMAPGui.<init>(IMAPGui.java:77)
        at org.dhs.chrislee.imapgui.IMAPGui$4.run(IMAPGui.java:326)
        at java.awt.event.InvocationEvent.dispatch(Unknown Source)
        at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
        at java.awt.EventQueue.access$500(Unknown Source)
        at java.awt.EventQueue$3.run(Unknown Source)
        at java.awt.EventQueue$3.run(Unknown Source)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
        at java.awt.EventQueue.dispatchEvent(Unknown Source)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
        at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
        at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
        at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
        at java.awt.EventDispatchThread.run(Unknown Source)

I'll take a look at this tomorrow. Thanks for letting me know.

Hideio, both of these are excellent bug finds.  The first has to do when the message doesn't present a valid date and the second is caused by not being able to find a secret key ring.  It will take time to fix both of these because I have to create a testing environment that will trigger both of these problems.
I think that I will split this issue into two issues.  I'll work on the GUI-launch issue (which is really a GPG secret key issue) first.

I couldn't reproduce this bug exactly, but I did find interesting ways to make my imap server crash...  I added a "likely" patch to try to work around the missing date issue.  Look to version 2.0.3.

Command Line works now.

An some second small thing, i need umlauts because Drafts=Entwürfe in german.
javax.mail.FolderNotFoundException: Entw?rfe not found
        at com.sun.mail.imap.IMAPFolder.checkExists(IMAPFolder.java:430)
        at com.sun.mail.imap.IMAPFolder.open(IMAPFolder.java:1018)
        at com.sun.mail.imap.IMAPFolder.open(IMAPFolder.java:951)
        at org.dhs.chrislee.IMAPCrypt.encrypt(IMAPCrypt.java:307)
        at org.dhs.chrislee.IMAPCrypt.main(IMAPCrypt.java:590)

Weird!  I thought for sure that I had full UTF-8 support.  This should be easy to test however.  Thanks for finding the bug.

This is strange.  I created a folder called INBOX.imapcrypt.tëstüñç and it seemed to work just fine.
2015-09-15 21:46:42 INFO  IMAPCrypt:353 - Appending the new, encrypted version of the message into the folder, INBOX.imapcrypt.tëstüñç
...
2015-09-15 21:46:44 INFO  IMAPCrypt:316 - Message #14 on Thu Sep 10 16:33:44 JST 2015: Хотите что-то поменять в своем доме?

In thunderbird the imap address folder "Entwürfe" looks like that "imap.gmx.net/Entw%26APw-rfe" maybe thats matter?

So I used Thunderbird to create a subfolder named "Entw%26APw-rfe" and it showed up in my folder list as Entwürfe.  Then I ran 2.0.3 on the folder and it encrypted just fine.  I'm starting to suspect the IMAP server software on imap.gmx.net (my sleuthing indicates Cyrus).  I'm using dovecot for my test server.
It could be how the java code indexes folders.  Perhaps I'm not putting special folders in their proper place.  I couldn't figure out how to register an account on gmx.net...  I'll try again tomorrow.
2015-09-16 21:49:42 INFO  IMAPCrypt:353 - Appending the new, encrypted version of the message into the folder, INBOX.imapcrypt.Entwürfe

GMX have a English portal for *.com emails too https://www.gmx.com/ . But im not 100% sure they use the exact same setups for they servers in europe/america.


Setup.py to install the package with setuptools

@BBillot  added setup.py to read requirement depending on the python version, and added resultant files (after python setup develop) to be added in .gitignore

Hi and thanks for the PR !
very good idea to include a setup.py file. Unfortunately I had to do many changes in the code to make this work, so I didn't use your PR in the end. But thank you very much for the idea :)
I hope you like SynthSeg!
Best
Benjamin


use lowercase imageRef since docker does not support capital letters

Per issue reported in sigstore/cosign-installer, docker push doesn't support references with uppercase characters.
bcallaway@bcallaway01:~/git/sigstore/cosign$ COSIGN_EXPERIMENTAL=1 ./cosign sign ghcr.io/bobcallaway/Custom-Error-Page@sha256:0dd6830f13edeb8a272586a5fe39a15dc3a0a8308be467c9117e7355c5ad9a68
Generating ephemeral keys...
Retrieving signed certificate...
Your browser will now be opened to:
https://oauth2.sigstore.dev/auth/auth?access_type=online&client_id=sigstore&code_challenge=L7k6KnuoCh8MizCXno9R7TJvO7fiZiSYVfG0abWRY_8&code_challenge_method=S256&nonce=24klxgjpuRfR63Y4TzvOoAQN7Hw&redirect_uri=http%3A%2F%2Flocalhost%3A35411%2Fauth%2Fcallback&response_type=code&scope=openid+email&state=24klxjEVeLvfgKtoxNzfF6sMHDD
Successfully verified SCT...
Error: signing [ghcr.io/bobcallaway/Custom-Error-Page@sha256:0dd6830f13edeb8a272586a5fe39a15dc3a0a8308be467c9117e7355c5ad9a68]: parsing reference: could not parse reference: ghcr.io/bobcallaway/Custom-Error-Page@sha256:0dd6830f13edeb8a272586a5fe39a15dc3a0a8308be467c9117e7355c5ad9a68
main.go:46: error during command execution: signing [ghcr.io/bobcallaway/Custom-Error-Page@sha256:0dd6830f13edeb8a272586a5fe39a15dc3a0a8308be467c9117e7355c5ad9a68]: parsing reference: could not parse reference: ghcr.io/bobcallaway/Custom-Error-Page@sha256:0dd6830f13edeb8a272586a5fe39a15dc3a0a8308be467c9117e7355c5ad9a68

There isn't really a useful error message to assist in debugging.
We can either check for uppercase characters and fail with a helpful error message, or we could transform the image ref to be in all lowercase characters before pushing the signature.

cc @mattmoor @jonjohnsonjr I thought we saw this back in December and fixed it in ggcr somewhere...

We didn’t because the starter action uses some metadata action to normalize instead

Ah whoops!


find_packge(OpenVDB) failed after brew install openvdb

So I 'm using openvdb as a module of my project. Everything goes fine until I add afind_package(OpenVDB REQUIRED) command in my CMakeLists.txt. The error message is:
CMake Error at xxx/xxx/CMakeLists.txt:8 (find_package):
  By not providing "FindOpenVDB.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "OpenVDB", but
  CMake did not find one.

  Could not find a package configuration file provided by "OpenVDB" with any
  of the following names:

    OpenVDBConfig.cmake
    openvdb-config.cmake

  Add the installation prefix of "OpenVDB" to CMAKE_PREFIX_PATH or set
  "OpenVDB_DIR" to a directory containing one of the above files.  If
  "OpenVDB" provides a separate development package or SDK, be sure it has
  been installed.

Other modules, like GLEW can be found as expected after installed by homebrew

@Idclip - are you able to shed any light on this question?

IIRC, homebrew uses the makefile to install OpenVDB v6.0.0 and there's no make or CMake support for actually installing the modules themselves. This has been added in v6.1 but the homebrew formula now needs to be updated. You might be able to point CMAKE_MODULE_PATH to homebrew's Cellar/openvdb directory but I'm unable to test this at the moment and can't remember exactly how homebrew configures its installs. I'll try and verify this soon

I solved this temporally by manually adding

/usr/local/lib to target_link_libraries
/usr/local/include to target_include_directories

and linking as  -lOpenVDB.
Looking forward to the update.

Glad to hear! I've created a ticket to update homebrew itself here https://jira.aswf.io/browse/OVDB-92

I am experiencing the same issue with the MinGW openvdb package. I think this is a problem with the CMake files provided by OpenVDB (i.e. this repo) and not with the packages. Someone in #cmake on Freenode IRC also said:

fwiw conceptually a package shouldn't ever be installing a find module for itself either

So it seems the CMake files provided by this repo are a bit broken.

Have you tried pointing your project to the location of your OpenVDB install through MinGW by providing the location of the installed find modules with CMAKE_MODULE_PATH?

fwiw conceptually a package shouldn't ever be installing a find module for itself either
So it seems the CMake files provided by this repo are a bit broken.

These find modules were written at a time when some of the required dependencies didn't have find modules themselves. Yes, I believe we should now be transitioning to a config module for OpenVDB and deprecating the dependency find modules (FindTBB/FindOpenEXR etctec). This work would massively benefit from external contribution from users who are more familiar with "correct" CMake.

Have you tried pointing your project to the location of your OpenVDB install through MinGW by providing the location of the installed find modules with CMAKE_MODULE_PATH?

Yes, that didn't help. CMake searches for the wrong filenames (as per the first post here) since it tries to look for configs, not modules -- if I understand CMake correctly.

This work would massively benefit from external contribution from users who are more familiar with "correct" CMake.

I agree! I know to little CMake to cook up a correct solution, though.

Yes, that didn't help. CMake searches for the wrong filenames (as per the first post here) since it tries to look for configs, not modules -- if I understand CMake correctly.

Hmm If I'm looking at the MinGW port correctly it should be installing VDB 7.0.0 - the original poster ran into this issue due to brew (at the time) using an older version of VDB (6.0.0).
This should work in 7.0.0:
set(CMAKE_MODULE_PATH "C:\path\to\cmake") # Path to the folder containing FindOpenVDB.cmake, located in the cmake folder of the OpenVDB installation
find_package(OpenVDB REQUIRED)

If not I'll try and see if there's an easy patch solution.

I agree! I know to little CMake to cook up a correct solution, though.

No problem, just highlighting for any CMake gurus out there!

Your proposal at least let me compile my files. However, at link time it complained about undefined references to Boost even though I link to it. I am using Boost in the project I am compiling myself. Does OpenVDB also use Boost? Somehow those undefined references sound like incompatible Boost versions.

I have also the same problem.
Environment

osx Catalina 10.15.4
clang <IP_ADDRESS>30032
cmake 3.17.0
brew 2.2.15

CMakeLists.txt
find_package(OpenVDB CONFIG REQUIRED)

Error
CMake Error at CMakeLists.txt:36 (find_package):
  Could not find a package configuration file provided by "OpenVDB" with any
  of the following names:

    OpenVDBConfig.cmake
    openvdb-config.cmake

  Add the installation prefix of "OpenVDB" to CMAKE_PREFIX_PATH or set
  "OpenVDB_DIR" to a directory containing one of the above files.  If
  "OpenVDB" provides a separate development package or SDK, be sure it has
  been installed.

There are no config files at /usr/local/lib/cmake related to openvdb as follows.
@ /usr/local/lib/cmake/OpenVDB

FindBlosc.cmake
FindCppUnit.cmake
FindIlmBase.cmake
FindJemalloc.cmake
FindLog4cplus.cmake
FindOpenEXR.cmake
FindOpenVDB.cmake
FindTBB.cmake
OpenVDBGLFW3Setup.cmake
OpenVDBHoudiniSetup.cmake
OpenVDBMayaSetup.cmake
OpenVDBUtils.cmake

If you have the newer solution, could you tell me it?

Can anyone post a CMakeLists.txt to run OpenVDB samples

@DragonLiu1
Hi, I created tiny demo https://github.com/j20232/vdb_sample
# tbb
find_library(TBB_LIBRARY tbb HINTS /usr/local/lib)

# openexr
find_library(HALF_LIB half HINTS /usr/local/lib)

# openvdb
list(APPEND SAMPLE_INCLUDE_DIR /usr/local/include)
find_library(OpenVDB_LIBRARY openvdb HINTS /usr/local/lib)

## include headers
include_directories(${SAMPLE_INCLUDE_DIR})
add_executable(sample ${SAMPLE_SOURCES})
target_link_libraries(
    sample
    ${TBB_LIBRARY}
    ${HALF_LIB}
    ${OpenVDB_LIBRARY}
)

You need to load tbb and openexr in my example.
There may be other solution to use CMakeLists.txt, so please share us if you know other samples.

Thank you for your kind reply, and the path of HALF_LIB is "/usr/lib/x86_64-linux-gnu" in my computer, however, it doesn't work when I add this path to CMakeLists.txt. Fortunately it works when I manually add the path in CMake GUI, which really confused me.
###CMakeLists.txt
cmake_minimum_required(VERSION 3.10)
project(view)
set(CMAKE_MODULE_PATH "/usr/local/lib/cmake/OpenVDB")
find_library(OpenVDB_LIBRARY openvdb HINTS /usr/local/lib)
find_library(HALF_LIB half  HINTS "/usr/lib/x86_64-linux-gnu")
find_library(TBB_LIBRARY tbb HINTS /usr/local/lib)
add_executable(view view.cpp)
target_link_libraries(view ${TBB_LIBRARY}
    ${HALF_LIB}
    ${OpenVDB_LIBRARY}
)
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年9月2日(星期三) 凌晨1:10
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [AcademySoftwareFoundation/openvdb] find_packge(OpenVDB) failed after brew install openvdb (#412)
@DragonLiu1
Hi, I created tiny demo https://github.com/j20232/vdb_sample
tbb find_library(TBB_LIBRARY tbb HINTS /usr/local/lib) # openexr find_library(HALF_LIB half HINTS /usr/local/lib) # openvdb list(APPEND SAMPLE_INCLUDE_DIR /usr/local/include) find_library(OpenVDB_LIBRARY openvdb HINTS /usr/local/lib) ## include headers include_directories(${SAMPLE_INCLUDE_DIR}) add_executable(sample ${SAMPLE_SOURCES}) target_link_libraries(     sample     ${TBB_LIBRARY}     ${HALF_LIB}     ${OpenVDB_LIBRARY} )
You need to load tbb and openexr in my example.
There may be other solution to use CMakeLists.txt, so please share us if you know other samples.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I am not the cmake expert needed here. However, I might have a useful connection to offer.
We had to use the above workaround on linux / macos to integrate openvdb. However on windows we use vcpkg to take care of our dependencies management, and that project has patched things to vendor the correct cmake files.
https://github.com/microsoft/vcpkg/blame/master/ports/openvdb/portfile.cmake
I think this may have been the PR that fixed the issue in question, hard to say for sure though.
https://github.com/microsoft/vcpkg/pull/14651


OIS 1.3 Skipping API (operation) call use case

I think we still need an short explanation (maybe a brief use case) as to why one would want to skip the API call.

This is a duplicate of #1118


h2: propagate stream close without read pending, avoid SOOE if !autoRead

Motivation:
AbstractHttp2StreamChannel requires that there is a channel.read()
pending in order to notify of channel inactive events. For use cases
that don't use auto read the application could have read the frame
corresponding to end of stream but not get the channel inactive.
If read is called from within channelReadComplete this creates
a reentrant loop into AbstractHttp2StreamChannel#doBeginRead that
may cause a StackOverflowException.
Modifications:

propagate stream/channel closed event if possible (e.g. no pending
data) without invoking read().
adjust readPending state to prevent StackOverflowException and
let the read loop continue.

@Scottmitch great catch!


[wip]🌱 Increasing wait-control-plane timeout to fix CI ephemeral test

What this PR does / why we need it:
Increasing wait-control-plane timeout to fix CI ephemeral test
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/test ?

/test metal3-centos-e2e-feature-test-main-features

/test metal3-centos-e2e-feature-test-main-features

running: When testing MachineDeployment remediation [healthcheck] [remediation] [features] [It] Should replace unhealthy machines [healthcheck, remediation, features]
/test metal3-centos-e2e-feature-test-main-features

failed because timeout was set to 3h now fixed it
/test metal3-centos-e2e-feature-test-main-features

/test metal3-centos-e2e-feature-test-main-features

/test metal3-centos-e2e-feature-test-main-features

/test metal3-centos-e2e-feature-test-main-features

/test metal3-centos-e2e-feature-test-main-features

/test metal3-centos-e2e-feature-test-main-features

it was for testing


3T roadmap

[ ] Implement MeanAtomicMass and MeanAtomicNumber
[ ] Implement Z-split modifier

z-split at least needs to take ionization fraction as an input in the lambda
z-split requires mean atomic mass/number of parent, unionized EOS

[ ] Implement ability to load sub-tables of sesame into EOSPAC/spiner
[ ] Ability to compute ionization level/fraction has to be hot-swappable not monolithic in the EOS

Biggest unknown here.
Potentially implement these calculations in downstream first? But would like to move that optionality into something singularity-like long term.
Could go in new directory of singularity-eos or in singularity-eos-extras maybe?

As I understand, there are at least two ways to do the Z-split. One would be to use one EOS and just scale the electron contribution by ionization state, but you can also imagine a separate EOS for the electrons and the ions. Would this approach be able to do either?

As I understand, there are at least two ways to do the Z-split. One would be to use one EOS and just scale the electron contribution by ionization state, but you can also imagine a separate EOS for the electrons and the ions. Would this approach be able to do either?

The version I was imagining was the latter. I was thinking of creating two modifiers, ZSplitElectrons and ZSplitIons that both modify a single original EOS (say Gruneisen).
That said, if we're interested in supporting the former case too, we could discuss. I'm not sure how to do that generically.

The version I was imagining was the latter. I was thinking of creating two modifiers, ZSplitElectrons and ZSplitIons that both modify a single original EOS (say Gruneisen).
That said, if we're interested in supporting the former case too, we could discuss. I'm not sure how to do that generically.

I'm a little confused here since this sounds like the one EOS case I mentioned. I was thinking that another approach would be to model the electron contribution directly as an ideal gas an pair that with a more appropriate EOS for the ions.

Ah ok. After clarifying on Mattermost:

The implicit assumption is we always hand the host code two EOS's, one electron and one ion
In zsplit mode, we split an underlying EOS into an electron EOS and an ion EOS
Alternatively, we may pass two completely separate EOS's to the host code, one for electrons and one for ions

To support this, we need ways to load the relevant subtables from Sesame
We also need to be able to rescale, say, an ideal gas, by ionization state so the number density is right.

I think we need to support both models.


[Enhancement] Filter by file type

Is your feature request related to a problem? Please describe.
I sometimes have issues trying to find a specific media file.
Describe the solution you'd like
A file type filters, i.e. videos, photos, gifs.
Describe alternatives you've considered
NA
Additional context
Make it possible to apply the filter in the regular view and search results view.

Media type is already available in the search bar (eg: "video", "gif", "svg", "png")


coprocessor/endpoint: log table_id in slow log

@hicqu @breeswish @coocood PTAL

LGTM

/run-integration-tests


Enable to reset card swipe

With a public reset function, we could reset the card swipe, for example on an api error.

Hi there. In order to consider this I would need to see a working example of how to use this as it needs to be documented.
Can you create a working demo of this?

Closing due to inactivity

Literally, this allows you to reset the state offset of your animated item. It can be useful when you have a reactive UI that changes the content of a card when you swipe. And need to show the card at the starting position with the new content.


RFC: consider removing Flatpak: rebuild command

Consider removing flatpak-vscode.rebuild and move its functionality to flatpak-vscode.run.
The rationale is it would be less clutter and confusion with flatpak-vscode.run. Additionally, it helps avoid accidentally running without rebuilding.

The reason I kept the two steps separate is we don't have a way to track the sources files if there were any changes and you would probably end up waiting a couple of seconds before the app is launched which could be not great :/

there's probably a vscode api to track modified file

The issue is to figure out if those files are part of the source code. That is something normally the build system can know

Please don't get rid of the rebuild / run separation. Sometimes its just convenient to rerun the app to check something even when some files are already modified.

I see, that makes sense.
Since the "run" button used in #55 is mapped to rebuild command, it would make sense to rename it to just run and rename run command to run-without-rebuild or something. Mainly for coherency, and to convey semantic oposite of stop

I think we can drop rebuild and only keep build here. We know whether the app was built once or not, so we can just make "build" triggers a "rebuild" instead. Make the run button triggers both build & run, and the run command from the palette only triggers run.

Make the run button triggers both build & run, and the run command from the palette only triggers run.

I think we want to have the run button to have the same behavior as the command palette one (for coherence), or to atleast have that behavior as an entry in the command palette


Use a 3rd party RESTful API and associated specs

Write a job that uses a well-known API (e.g. Stripe, GitHub, ?) and write specs for it that stub requests with VCR.

Consider using one of these gems for the API calls:

spyke https://github.com/balvig/spyke
her https://github.com/remiprev/her


LUCENE-9976: Fix WANDScorer assertion error

Description
Fix WANDScorer assertion error
Solution
Update the assertion as condition (tailMaxScore >= minCompetitiveScore) && (tailSize < minShouldMatch) are valid now
Tests
Passed gradlew test -Ptests.iters=10 --tests TestExpressionSorts.testQueries -Dtests.seed=FF571CE915A0955:537BBD158B33BCFB -Dtests.multiplier=2 -Dtests.nightly=true -Dtests.slow=true -Dtests.asserts=true -p lucene/expressions/
Passed ./gradlew check -Pvalidation.git.failOnModified=false -Dtests.nightly=true
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[ ] I have added tests for my changes.

Thank you!

Thanks Adrien for the review and approval!


Update Lifx from beta to V1

Fixes #7

Previously deprecated methods have had their deprecation notices removed and endpoint calls, tests, updated
Added validateColor and getScenes endpoints

@kz This should resolve the upgrade to V1.
We may be able to bump the version in the package out of dev and into it's first release version with the inclusion of this one.

Thanks for your contributions! I have gone ahead and released 1.0.0 for this package.

If you're interested in being added as a collaborator to this repository, please send me an email. It would be great to have more people keeping this package up to date.


add preconditions for control plane migration

How to categorize this PR?

/area control-plane-migration
/kind enhancement
What this PR does / why we need it:
Adds a brief precondition section to indicate what needs to be done so control plane migration works.

Thank you @christianhuening for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.


Latest DCS Bios library "servo.h" library incompatible with ESP32 boards

I updated the DCS Bios library and now get the following error
f:\Users\LES\Documents\Arduino\libraries\Servo\src/Servo.h:77:2: error: #error "This library only supports boards with an AVR, SAM, SAMD, NRF52 or STM32F4 processor."
#error "This library only supports boards with an AVR, SAM, SAMD, NRF52 or STM32F4 processor."
All ESP32 boards no cannot be used until this is corrected

Have you tried using the official Servo library? Not sure if that helps but it was recently updated.

0.3.9 will allow #define DCSBIOS_DISABLE_SERVO which will disable the servo library.


[Bug] ValueError in generate_lfp for one-dimensional input

Describe the bug
Running the function generate_lfp with one-dimensional input results in a ValueError.
To Reproduce

Define an array xs to x coordinate values with a length of 2304
Run generate_lfp(gauss_1d_dipole, xs)
See the resulting error. (Traceback shown below)

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [22], line 1
----> 1 lfp = generate_lfp(gauss_1d_dipole, xs)

File ~\AppData\Local\Programs\Python\Python39\lib\site-packages\elephant\utils.py:80, in deprecated_alias.<locals>.deco.<locals>.wrapper(*args, **kwargs)
     77 @wraps(func)
     78 def wrapper(*args, **kwargs):
     79     _rename_kwargs(func.__name__, kwargs, aliases)
---> 80     return func(*args, **kwargs)

File ~\AppData\Local\Programs\Python\Python39\lib\site-packages\elephant\current_source_density.py:302, in generate_lfp(csd_profile, x_positions, y_positions, z_positions, x_limits, y_limits, z_limits, resolution)
    300 chrg_x = x
    301 csd = csd_profile(chrg_x)
--> 302 pots = integrate_1D(x_positions, chrg_x, csd, h)
    303 pots /= 2. * sigma  # eq.: 26 from Potworowski et al
    304 ele_pos = x_positions

File ~\AppData\Local\Programs\Python\Python39\lib\site-packages\elephant\current_source_density.py:263, in generate_lfp.<locals>.integrate_1D(x0, csd_x, csd, h)
    262 def integrate_1D(x0, csd_x, csd, h):
--> 263     m = np.sqrt((csd_x - x0) ** 2 + h ** 2) - abs(csd_x - x0)
    264     y = csd * m
    265     I = simps(y, csd_x)

ValueError: operands could not be broadcast together with shapes (50,) (2304,)

Expected behavior I expected the function to return an array of LFP values.
Environment

Windows 10:
Installed elephant with pip install elephant:
Python version: 3.9.10
neo==0.12.0
numpy==1.21.5

Thanks for reporting this issue!
I was able to reproduce it using your minimal example and to track down the error. As an immediate workaround, you can reshape the input xs to (2304, 1) using numpy.reshape.
I will start working on a fix to make this workaround unnecessary in the future.


[VL] Daily Update Velox Version (2024_05_04)

Upstream Velox's New Commits:
e38079f8b by rui-mo, Add arg generator for decimal type (9634)
38abde9a6 by duanmeng, Add fuzzer for RowNumber operator (9524)
f54787b73 by Bikramjeet Vig, Fix data race when resetting lazy vector's consistency check (9701)
5bc681866 by Jialiang Tan, Add global access utility to PeriodicStatsReporter (9696)
238830b31 by Jialiang Tan, Remove deprecated Task::create API (9697)
9fbc54426 by Deepak Majeti, Add monthly update for February 2024 (9650)

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_5596_time.csv
log/native_master_05_03_2024_90f000cae_time.csv
difference
percentage

q1
36.89
35.63
-1.256
96.60%

q2
23.94
28.51
4.569
119.08%

q3
36.25
36.54
0.293
100.81%

q4
39.47
40.29
0.812
102.06%

q5
70.39
70.20
-0.189
99.73%

q6
7.50
7.26
-0.237
96.84%

q7
84.79
83.75
-1.044
98.77%

q8
84.43
82.97
-1.460
98.27%

q9
123.94
124.07
0.120
100.10%

q10
45.11
44.44
-0.677
98.50%

q11
20.35
19.68
-0.677
96.67%

q12
27.35
26.57
-0.780
97.15%

q13
55.38
56.49
1.113
102.01%

q14
19.01
19.61
0.599
103.15%

q15
31.05
30.01
-1.037
96.66%

q16
12.31
12.87
0.559
104.54%

q17
104.18
102.43
-1.755
98.32%

q18
145.93
146.02
0.091
100.06%

q19
14.71
13.46
-1.256
91.46%

q20
29.14
31.45
2.310
107.93%

q21
285.83
289.87
4.043
101.41%

q22
14.82
14.43
-0.396
97.33%

total
1312.81
1316.55
3.746
100.29%


Implement MSE based optimization

In order for a perceptron to "learn", it will need to update its weights proportional its contribution to the error. Mean-Square Error (MSE) is a very good place to start, but we should be able to change the error function in the future.


make debugger expand relative paths to absolute

I have encountered issues with setting breakpoints in modules loaded using require, which uses absolute paths.
Using relative paths brings problems in conjunction with the vscode extension, which might find the same file in multiple sourcesRoot directories. Absolute paths would solve this.
debugger class now has a public method set_working_dir, which accepts char*.
If the method is not called, the old behaviour is preserved - relative paths are not expanded.
server class gets cwd in main and handles it to the debugger.

Coverage decreased (-6.1%) to 77.448% when pulling 1e110ba0df6f628102f4cd21c59a8cf3521c9030 on trogper:master into 68378984d689e7ca36b56a7c625bb5a6e3d318ba on satoren:master.


Incompatibility with Sodium

I added the Crossbow to AutoVanillaHands, and when I open the game using Sodium, when I hold the Crossbow I am transformed into an Among Us

since Minecraft doesn't refresh a model while you're viewing the model, you need to exit F5 and come back
the arms of the skin disappear and do not return even after I am no longer holding the Crossbow

and this is only fixed by restarting the game

im only using this mods:

Fabric 1.20.1
First-Person Model 2.3.4
Sodium 0.5.8

firstperson.json: https://gist.github.com/peow/9169bba9a8ad7f3250924d55d6d2add8
latest.log: https://gist.github.com/peow/2cfa48d82fb53c3ef3b3681422663da5
if is possible to fix this pls make an update to Fabric 1.20.1, thanks

Fabric 1.20.6
First-Person Model 2.4.1
Sodium 0.5.8
Same problem, no vanilla hands. In dynamic mode they show, but not when set to always show and dynamic off.

Same problem, no vanilla hands. In dynamic mode they show, but not when set to always show and dynamic off.
Edit: My issue is not related to the crossbow, but with any item, even bare hands. Vanilla hands never show.

Soooo its an entirely different issue/question?

Same problem, no vanilla hands. In dynamic mode they show, but not when set to always show and dynamic off.
Edit: My issue is not related to the crossbow, but with any item, even bare hands. Vanilla hands never show.

Soooo its an entirely different issue/question?

Im not sure, thats why I only commented.
Seems related to me.
In discord i created an issue in the “mod-support”, since i didnt found any related there, “[1.20.6] No hands with Sodium”.
I can also create the issue here, if thats better.


HTML5 Audio Slow / Stuttering on iOS

Godot version: 3.2 (also seeing the same issue with Godot 3.1, and 3.1.1)
OS/device including version: iPad 5th generation running iOS 13.3.1
Issue description: I'm seeing audio playing REALLY slow and stuttering (sounding robotic) on iPad HTML5 builds using .wav and ogg files.
Steps to reproduce:  Create a blank project,, add an AudioStreamPlayer and load a .ogg or .wav file in the AudioStreamPlayer and set it to play that audio file on a loop. Export to HTML5 and play on an ipad and the audio will play very glitchy, it will sound slow and robotic.
Minimal reproduction project: https://drive.google.com/file/d/14XzsamzGlBn_vx2y2OXpdhtkwTxjWEco/view?usp=sharing

@davidCentervention Please upload a minimal reproduction project to make this easier to troubleshoot.

Minimal reproduction project: https://drive.google.com/file/d/14XzsamzGlBn_vx2y2OXpdhtkwTxjWEco/view?usp=sharing

I have this problem as well, but on an iPhone 11 with iOS 13.3.1.
Happens with all sounds, looped or not.

Same problem here, on an iPad Air 2 with iOS 12.4.
I spent a few hours trying to work around it, but to no avail. It seemed suspicious that the default sampleRate of the audioContext was 48000 on my laptop and Android phone, which play my HTML5 sounds just fine, yet 44100 on the iPad. However, all my sounds are actually mixed with a sample rate of 44100, and my laptop still plays them right when setting the audioContext's sampleRate to 44100. I cannot set the audioContext's sample rate on the iPad, but resampling the sounds to 48000 didn't change anything. Also, the sounds are played back about twice as slow, instead of the 48000 / 44100 = 1.09x you would expect. Doubling their pitch_scale in Godot made the durations approximately right, but they still sounded robotic.
However, the iPad could play the same sounds just fine when using the Web Audio API directly. Hence I assume the problem can be solved in either Godot or WebAssembly/Emscripten, unless it's a performance/thread problem.

Same here. For me it looks like a buffering as the sounds distort in the beginning / after loading very much but suddenly it might clear up if you play the game further. Another interesting thing is that sounds get better if you minimize the browser. And the sounds continue playing if you close the browser.
Godot version 3.2 Custom (with asm.js support)
iPad mini 2, iOS 12.4 (as well as all other iPads and iPhones we have tried)

Having the same exact issues. So weird that when you minimize the browser, it plays the sound clear for a couple of seconds. Must be a loading issue of some sort.
Godot version 3.2 - GLES2 build
iphone 11, iOS13,

Has anyone found a way to make this work yet? A work around?

@Tinkersedge Not that I know of. See also #26554.

@Tinkersedge For me the work-around was to handle audio entirely outside of Godot for HTML5 for the time being. I call Javascript from Godot to load mp3s from the web into an  tag on my HTML page and play audio from there, completely bypassing the audio engine in Godot to use the audio played by the browser.
My assumption is the issue in Godot is likely due to the auto-play audio changes that went in most browsers in the last few years that require audio to be initiated by an interaction (click) by the end user: https://webkit.org/blog/7734/auto-play-policy-changes-for-macos/

@davidCentervention any chance you could share a super simple version of how you did that? Is that method able to handle background audio as well as sound effects? Thank you so much for telling me about the work around. Hopefully I can still make this game happen on tablets.

I tried a lot of things, for instance different sample rates and creating a new audio context. Nothing worked for me. I guess the work-around by @davidCentervention is the only way for now.

@Tinkersedge @davidCentervention @ChronoDK We finally actually managed to solve this: my colleague modified JS audio driver source code and recompiled the Engine's export template. I'll ask him to drop the solution here as well.

@caizu Amazing, I hope the fix can be added to a release soon then.

@ChronoDK I hope it too and I asked my colleague to do that.

I'm experiencing this problem with IPads and IPhones in a simple html5 game made with Godot Engine 3.2

my colleague modified JS audio driver source code and recompiled the Engine's export template
I was hoping issue #38215 for v4 might help this case.   I managed to hack the AudioServer::Init function and compiled the WebAssembly file with version 3.2.2 source.   But am still experiencing the issue.
This is my first foray into debugging source code and I am feeling really happy having achieved this much.  But I am still having this issue with Audio on IOS devices new and old, ipad and iphone, with updated IOS.

Hello people, the great problem is the buffer_size!
Godot audio driver use createScriptProcessor  with the first value zero!
So the device, set by default the buffer_size! I dont know why IOS system is so shit, to set 256 on the buffer_size!

I put this trative, when the buffer_size is lower 2048 they set 2048.
Below is the template export with that trative, so just past on folder C:\Users\Probrain_A\AppData\Roaming\Godot\templates\3.2.1.stable
3.2.1.stable.zip
I hope helped sameone!!

Hello people, the great problem is the buffer_size!
Godot audio driver use createScriptProcessor  with the first value zero!
So the device, set by default the buffer_size! I dont know why IOS system is so shit, to set 256 on the buffer_size!

I put this trative, when the buffer_size is lower 2048 they set 2048.
Below is the template export with that trative, just extract into folder \AppData\Roaming\Godot\templates\3.2.1.stable
3.2.1.stable.zip
I hope helped sameone!!

This fixed my issue. Thanks.
I hope they put this on the officials templates

I hope they put this on the officials templates

Feel free to open a pull request for this :slightly_smiling_face:

Hello people, the great problem is the buffer_size!
Godot audio driver use createScriptProcessor  with the first value zero!
So the device, set by default the buffer_size! I dont know why IOS system is so shit, to set 256 on the buffer_size!

I put this trative, when the buffer_size is lower 2048 they set 2048.
Below is the template export with that trative, just extract into folder \AppData\Roaming\Godot\templates\3.2.1.stable
3.2.1.stable.zip
I hope helped sameone!!

For me, it fixed the ipad issue, but my iphone is back to not loading like it was before the last update. I will try again in case I did something wrong.

Hello people, the great problem is the buffer_size!
Godot audio driver use createScriptProcessor  with the first value zero!
So the device, set by default the buffer_size! I dont know why IOS system is so shit, to set 256 on the buffer_size!

I put this trative, when the buffer_size is lower 2048 they set 2048.
Below is the template export with that trative, just extract into folder \AppData\Roaming\Godot\templates\3.2.1.stable
3.2.1.stable.zip
I hope helped sameone!!

For me, it fixed the ipad issue, but my iphone is back to not loading like it was before the last update. I will try again in case I did something wrong.

Maybe is the buffer lower, the Godot Team put 4096, i set 2048! If dont work to you, i upload with that value to 4096!

I have tried the template with no luck, so I pushed my butt to learn Java and html 5, and found a solution to play my video through Java:

So basically you use the JavaScript.eval to evaluate your java script to directly add a video player html 5 element to the body of your webpage. That Java identifier is a preload script.

I have found audio to be working just fine on iOS with the latest beta and release candidates, so that javascript workaround should not be needed anymore.

The issue was connected with size of buffering in platform/javascript/audio_driver_javascript.cpp
there was the part of code:
`buffer_length = EM_ASM_INT({
var CHANNEL_COUNT = $0;
            var channelCount = _audioDriver_audioContext.destination.channelCount;
            try {
                    // Try letting the browser recommend a buffer length.
                    _audioDriver_scriptNode = _audioDriver_audioContext.createScriptProcessor(0, 2, 1);
            } catch (e) {
                    // ...otherwise, default to 4096.
                    _audioDriver_scriptNode = _audioDriver_audioContext.createScriptProcessor(4096, 2, 1);
            }
            _audioDriver_scriptNode.connect(_audioDriver_audioContext.destination);

            return _audioDriver_scriptNode.bufferSize;
    }, channel_count);

`
and in next string by "// Try letting the browser recommend a buffer length." need to be replaced zero to i.g. 16384. That's it. Safari can not determine the size and it falls to 0 causing crackles.

@IvanF Indeed, that was fixed in https://github.com/godotengine/godot/pull/38816/commits/245c179bd31d65c6b31d156c4f6b08647df0e2fe and #39037. The fix is available in Godot 3.2.2.


BUG: Complete str accessor methods

Add remaining methods
closes #155

[X] Closes #155 (Replace xxxx with the Github issue number)
[X] Tests added: Please use assert_type() to assert the type of any return value

@bashtage let me know when I should look at this again

@Dr-Irv It is ready for review

@Dr-Irv changes in, should be ready.

Can you resolve conflicts?  Probably due to #151


Unpredictable Crash on App Start Due to Test Observer Registration Outside Main Thread

Description
The TCA based application is unpredictably crashing upon start during the configuration load. The crash occurs on the line let config = try? await remoteConfigService.fetchRemoteConfig() from the AppConfig module, producing the error message: 'Test observers can only be registered and unregistered on the main thread.'
On further investigation, it was found that the crash likely happens during the testing environment check in DependencyValues.swift of the Dependencies framework, specifically at else if _XCTIsTesting. The crash seems to continue at the initialization of XCTCurrentTestCase where the test observers are launched.
Additional Notes:

The bug does not occur all the time but is more prevalent on slower machines compared to more powerful ones.
It is suggested that the issue lies within the XCTCurrentTestCase where test observers are being initialized.

Temporary workaround that helped:
        case .loadConfig:
            return .run { @MainActor send in
                let config = try? await remoteConfigService.fetchRemoteConfig()
                send(.configLoaded(config))
            }

Implementation of AppFeature, AppConfigFeature and RemoteConfigService:
struct AppFeature: ReducerProtocol {

    // MARK: - State
    struct State: Equatable {
        var appConfig: AppConfigFeature.State
    }

    // MARK: - Action
    enum Action: Equatable {
        case appDidFinishLaunching

        case appConfig(AppConfigFeature.Action)
    }

    // MARK: - Dependencies
    @Dependency(\.datadogActivator)
    var datadogActivator

    // MARK: - Reducer
    var body: some ReducerProtocol<State, Action> {
        Scope(state: \.appConfig, action: /Action.appConfig, child: AppConfigFeature.init)
        Reduce { state, action in
            switch action {
            case .appDidFinishLaunching:
                return .run { send in await send(Action.appConfig(.loadConfig)) }
            case .appConfig(.configLoaded):
                guard let info = state.appConfig.config?.dataDog else {
                    assertionFailure()
                    return .none
                }
                return .run { _ in await datadogActivator.activate(info: info) }
            case .appConfig:
                return .none
            }
        }
    }
}

public struct AppConfigFeature: ReducerProtocol, Sendable {

    // MARK: - State
    public struct State: Equatable {
        public var config: RemoteConfigValue?
    }

    // MARK: - Action
    public enum Action: Equatable, Sendable {
        case configLoaded(RemoteConfigValue?)
        case loadConfig
    }

    // MARK: - Dependencies
    @Dependency(\.remoteConfigService)
    var remoteConfigService

    // MARK: - Reducer
    public func reduce(into state: inout State, action: Action) -> EffectTask<Action> {
        switch action {
        case .configLoaded(let config):
            state.config = config
            return .none
        case .loadConfig:
            return .run { send in
                let config = try? await remoteConfigService.fetchRemoteConfig()
                await send(.configLoaded(config))
            }
        }
    }
}

import Dependencies
import FirebaseRemoteConfig

struct RemoteConfigService: Sendable {
    var fetchRemoteConfig: @Sendable () async throws -> RemoteConfigValue
}

// MARK: - DependencyKey
extension RemoteConfigService: DependencyKey {
    static let liveValue: RemoteConfigService = {
        @Dependency(\.appClient.isRunningUITests)
        var isRunningUITests
        return isRunningUITests()
        ? .uiTestStub
        : .live
    }()
    static let testValue = RemoteConfigService.failing
}
extension DependencyValues {
    var remoteConfigService: RemoteConfigService {
        get { self[RemoteConfigService.self] }
        set { self[RemoteConfigService.self] = newValue }
    }
}

// MARK: - Implementations
private extension RemoteConfigService {

    static let live = Self {

        let firebaseConfig = RemoteConfig.remoteConfig()
        firebaseConfig.setDefaults(fromPlist: "RemoteConfigDefaults")

        return try await withCheckedThrowingContinuation { continuation in
            firebaseConfig.fetchAndActivate { _, error in

                if let error {
                    continuation.resume(throwing: error)
                    return
                }

                do {
                    continuation.resume(returning: try RemoteConfigValue(from: firebaseConfig))
                } catch {
                    continuation.resume(throwing: error)
                }

            }
        }
    }

    static let failing = Self(
        fetchRemoteConfig: unimplemented("\(Self.self).fetchRemoteConfig")
    )

    static let uiTestStub = Self(
        fetchRemoteConfig: { RemoteConfigValue.mock() }
    )
}

Checklist

[ ] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[ ] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
The application should start and load the app config successfully.
Actual behavior
The application crashes on the line let config = try? await remoteConfigService.fetchRemoteConfig() with the error:
libc++abi: terminating with uncaught exception of type NSException
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Test observers can only be registered and unregistered on the main thread.'
terminating with uncaught exception of type NSException
CoreSimulator 857.14 - Device: iPhone 14 Pro (12DDE56D-B64A-4155-85C5-9D807F586EB2) - Runtime: iOS 16.2 (20C52) - DeviceType: iPhone 14 Pro

Steps to reproduce

Start the application
AppDelegate.didFinishLaunchingWithOptions sends viewStore.send(.appDidFinishLaunching) to the AppFeature.
In the app reducer, the app config is loaded from a dedicated module via

case .appDidFinishLaunching:
return .run { send in await send(Action.appConfig(.loadConfig)) }

Inside AppConfig module we run

case .loadConfig:
return .run { send in
    let config = try? await remoteConfigService.fetchRemoteConfig()
    await send(.configLoaded(config))
}

Dependencies version information
0.5.1
Destination operating system
iOS 16.2 Simulator
Xcode version information
14.2.0
Swift Compiler version information
swift-driver version: 1.62.15 Apple Swift version 5.7.2 (swiftlang-<IP_ADDRESS>.5 clang-14<IP_ADDRESS>)
Target: x86_64-apple-macosx13.0

@Atimca I think we may need more info to diagnose and reproduce, but this sounds like it might not be a bug with the library.
One question that doesn't seem to be covered in the issue: do you encounter this crash during test runs, or are you experiencing a crash when running the app normally? I would guess you're talking about test runs since the code path causing the crash shouldn't run during the course of a normal application.
Assuming this is the case, it sounds like your tests may be running in a host application, which means the host application is exercising live dependencies while XCTest prepares its test run. Running tests in a host application is a gotcha mentioned in the documentation here:
https://pointfreeco.github.io/swift-dependencies/main/documentation/dependencies/testing#Testing-gotchas
We advise ensuring that your tests do not run your application code and attempt to interact with live dependencies.
If I got any of the above wrong, please let us know with more information, and ideally a fully-working repro of the issue.
The only thing actionable in the library, I think, would be for us to avoid some of these XCTest background thread crashes by synchronously sending them to the main queue instead, but ideally we would have a repro to work with to do that work.

Thanks for the update @Atimca! I'll close this for now, but please do re-open if you uncover something that we can improve in the library.


Added Trans model and schema

Codecov Report

Merging #8 into master will increase coverage by 0.31%.
The diff coverage is 97.16%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   97.44%   97.76%   +0.31%
==========================================
  Files          22       26       +4
  Lines         744      849     +105
==========================================
+ Hits          725      830     +105
  Misses         19       19

Impacted Files
Coverage Δ

tckdb/backend/app/models/author.py
100.00% <ø> (ø)

tckdb/backend/app/models/bot.py
100.00% <ø> (ø)

tckdb/backend/app/models/literature.py
100.00% <ø> (ø)

tckdb/backend/app/models/trans.py
81.81% <81.81%> (ø)

tckdb/backend/app/schemas/trans.py
97.82% <97.82%> (ø)

tckdb/backend/app/models/freq.py
100.00% <100.00%> (+7.40%)
:arrow_up:

tckdb/backend/app/tests/models/test_freq.py
100.00% <100.00%> (ø)

tckdb/backend/app/tests/models/test_trans.py
100.00% <100.00%> (ø)

tckdb/backend/app/tests/schemas/test_common.py
100.00% <100.00%> (ø)

tckdb/backend/app/tests/schemas/test_trans.py
100.00% <100.00%> (ø)

... and 6 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 c39deea...db21c27. Read the comment docs.


External references

Hi,
My current scenario is, I have 3 files one of them is the main file, this one has references to the other 2 files (schemas, parameters). Is there some way to configure OpenAPI.Net to load those external files and get only one final file with all external references in this one?
Thank you so much.

It is almost working.  It should be resolved before we release 1.3.0

External references of complete OpenAPI Documents now works in 1.3.  External references to fragments does not.


Adding client quota customization

Adding client quota customization

---
apiVersion: v1
kind: Namespace
metadata:
  name: myNamespace
  cluster: cluster1
spec:
  kafkaUser: kafkaServiceAccount
  kafkaUserQuota:
    producer_byte_rate: 204800
    consumer_byte_rate: 409600

Fixing update user quota filtering. Update has to be performed if namespace user quota definition is different from the broker user quota
Adding integration tests for both custom quota and quota updates

Note: at the beginning I was thinking of specifying the user quota like this
---
apiVersion: v1
kind: Namespace
metadata:
  name: myNamespace
  cluster: cluster1
spec:
  kafkaUser:
    name: kafkaServiceAccount
    producer_byte_rate: 204800
    consumer_byte_rate: 409600

But this format would have broken the namespace unmarshalling with existing namespace definition.
I was also thinking of creating a apiVersion 2 for namespace definition, Namespace v2 model, etc. but kept a simplier approach by adding a new configuration key.
Tell me what you prefer... !
When it will be merged I'll created a PR oin kafkactl to update to Readme accordingly

Hi @loicgreffier
You're right these properties should be in the ResourceQuota resource. My solution was straightforward (no need to do an extra call to the ResourceQuotaRepository because we had everything we need in the namespace definition) but functionally your proposal is better.
I'm finishing some tests and will update the PR

@loicgreffier can you check again the PR please ?

Hi @AlexisSouquiere, in addition, could these new quotas be returned by list (link) and get (link) in quota controller, so we can then display them in kafkactl ?

I did the changes to return these 2 new quotas in the list/get endpoints. If it's ok I'll create the PR for the same on kafkactl (https://github.com/AlexisSouquiere/kafkactl/commit/b83c764c6330ee142f132fc56682b9e11e329009)

@AlexisSouquiere, OK for merge, you can open a PR on Kafkactl


fix #179 error while deleting a gateway

Description
What's new?

#179

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Tests
[ ] Other

Checklist before merge
Developer's responsibilities

[x] This pull request merges to main
[x] Approved by at least one developer
[x] Pull request build has passed
[x] Staging app has been deployed successfully

Product Owner's responsibilities

[ ] No issues have been found on staging environment
[ ] Product Owner has approved the release

Codecov Report

Merging #203 (a9b6679) into main (48cdf31) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@          Coverage Diff          @@
##            main    #203   +/-   ##
=====================================
  Coverage   0.66%   0.66%
=====================================
  Files         71      71
  Lines       2414    2413    -1
=====================================
  Hits          16      16
+ Misses      2398    2397    -1

Impacted Files
Coverage Δ

...Portal/Client/Pages/Devices/CreateDevicePage.razor
0.00% <ø> (ø)

...al/Client/Pages/Gateways/CreateGatewayDialog.razor
0.00% <ø> (ø)

...ub.Portal/Server/Controllers/GatewaysController.cs
0.00% <0.00%> (ø)

...erver/Managers/DeviceProvisioningServiceManager.cs
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


change sql formatter to sql-formatter

The default sql formatter sqlformat doesn't work. For example:
CREATE TABLE student (
sid INT PRIMARY KEY, name VARCHAR(16), login VARCHAR(32) UNIQUE, age SMALLINT, gpa FLOAT );

format this piece of sql with
sqlformat tmp.sql  --reindent --indent_width 4 --keywords upper --use_space_around_operators

gets:
CREATE TABLE student (sid INT PRIMARY KEY,
                                      name VARCHAR(16),
                                           login VARCHAR(32) UNIQUE,
                                                             age SMALLINT, gpa FLOAT);

Without any options, sqlformat tmp.sql just prints the input.
The comment in the first answer about sqlformat points out the same thing.
While sql-formatter works great without any options. So I change sqlformat support to sql-formatter.

We should keep supporting sqlformat. But I agree it's confusing that it does not do anything without any options.
Please edit the PR so that it:

preserves sqlformat
adds sql-formatter
changes the default from sqlformat to sql-formatter


Add custom storage policy and set defs

Proposed changes:

Added custom policy definitions for storage:
policy_definition_enforce_storage_account_container_delete_retention_policy.json
policy_definition_enforce_storage_account_copy_scope_limitation.json
policy_definition_enforce_storage_account_cors_rules_restriction.json
policy_definition_enforce_storage_account_encrypty_with_customer_managed_key.json
policy_definition_enforce_storage_account_local_user_restriction.json
policy_definition_enforce_storage_account_network_acl_bypass_restriction.json
policy_definition_enforce_storage_account_public_blob_access_restriction.json
policy_definition_enforce_storage_account_resource_access_rule_resourceid_restriction.json
policy_definition_enforce_storage_account_resource_access_rule_tenant_restriction.json
policy_definition_enforce_storage_account_storage_services_encryption.json
policy_definition_enforce_storage_account_virtual_network_rules_restriction.json
-Added policy set definition for compliant storage
-Added policy set assignment for compliant storage

@marvinbuss policy_definition_enforce_storage_account_resource_access_rule_resourceid_restriction.json requires help in split

@marvinbuss policy_assignment_storage.json requires help managed identitiy parametrisation

Thanks for this. Have to look into this myself. Also asked Kristian why we don't rely on SystemAssigned identities. Would be simpler.


Without docker

Is your feature request related to a problem? Please describe.
How can i deploy this without docker?
Please add instructions on how to build it locally.
What I tried until now,
composer install
created database in local mariab server.
populated .env file.
php artisan key:generate
php artisan migrate
Nginx config something like this
location /myapp {
alias /path/speedtest-tracker/public;
try_files $uri $uri/ @myapp;
    location ~ \.php$ {
        include snippets/fastcgi-php.conf;
        fastcgi_param SCRIPT_FILENAME $request_filename;
        fastcgi_pass unix:/run/php/php8.2-fpm.sock;
    }

}

location @myapp {
rewrite /myapp/(.*)$ /myapp/index.php?/$1 last;
}
Working until the login page. cannot login

At this time I'm not officially supporting a "non-docker" install. If you know how to run a Laravel app on a VM you can follow those docs and the only other requirement is the speedtest cli.


Add yaml and events tab for ObjectBucketClaim page

This commit fixes a regression where the YAML and Events tabs are not
migrated to the ObjectBucketClaim page
Before:

After:

Signed-off-by: Vineet Badrinath<EMAIL_ADDRESS>

/override ci/prow/odf-console-e2e-aws

/retest odf-console-e2e-aws

/hold cancel


[API, Client] Refactor all data storage to use pouch/IDB

The level db is no doubt great for binary data, but is too much trouble for application JSON docs.
https://github.com/IBM/ipfs-social-proof/pull/11
and https://github.com/IBM/ipfs-social-proof/blob/following-peers-persistence/src/db.js

[[x] contacts DB
[x] proofsDB
[ ] accountDB


Add readiness probe to SD

Saw some strange issues, probe might help.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
f00e7d5c7ceb4e3ea7249a08f21045089de0ec5a
link
true
/test integ-pilot-multicluster_istio

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


Preview components in admin UI

Closes https://ethyca.atlassian.net/browse/PROD-1596, https://ethyca.atlassian.net/browse/PROD-1460,
https://ethyca.atlassian.net/browse/PROD-1803
Description Of Changes
Adds a live preview to the experience config form.
Code Changes
Steps to Confirm
With admin-ui and privacy-center both running, navigate to /consent/privacy-experience.
Edit an existing privacy experience config:

[ ] Preview should show with existing privacy notices
[ ] For privacy centers, banner should show; for modal, modal should show; for banner and modal, both should show;
[ ] When privacy notices are added or removed, preview should update accordingly
[ ] Edit English translation
[ ] When text is changed, preview should update accordingly

Create a new experience config:

[ ] Before adding a privacy notice, preview area should show alert telling you to add a privacy notice to see the preview
[ ] Appropriate component preview should show once component is selected and privacy notices are added
[ ] Before text is added, preview should show with default text

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

@jpople looking good! Just want to confirm this is a product decision to remove interactivity and to have banner and modal show statically like this?

Additionally, it looks like that bug I originally saw but couldn't reproduce is showing up again with mobile preview mode cutting off the last button:

One more thing I noticed is that upon editing a translation, the save button still says "Add translation" which may be a bit misleading as we're only editing the current translation:

Just want to confirm this is a product decision to remove interactivity and to have banner and modal show statically like this?

@eastandwestwind Yeah, talked this over with @NevilleS-- this approach let us sidestep both the bug you were having before where the banner would flicker in and out sometimes and also a new bug that popped up with Neville's i18n changes which made it pretty much impossible to view the modal anyway when interaction was working.


[macOS] Remove unnecessary debug prints.

Fixes https://github.com/godotengine/godot/issues/74906

Thanks!

Cherry-picked for 4.0.1.


Move openstack security group to a variable

Provide the ability for installer to specify their security group for Openstack in the configuration file instead of directly within the playbook.

@kenjones-cisco thanks for this PR!
@ChrisAubuchon I agree this should be per dc.

Great catch, I totally missed that. Change has been made.

@kenjones-cisco we're only merging fixes for 0.2.0 release candidates for the next day or so. As soon as we tag a 0.2.0 release I'll merge this in.

Actually, as part of the openstack doc updates for 0.2 for bug#154, we need to instruct the users to create a security group and document how to use them. Let me test this and we can merge it.


Bug 2034527: Add IP_OPTIONS environment variable

The IP_OPTIONS we need to use for the provisioned node (which has to
pull its Ignition from the external network) are not necessarily the
same as the ones used for starting IPA (which has to talk to ironic via
the provisioning network if it exists).
Add an environment variable that can be passed down to the custom deploy
step to set the IP options for the installed system via
coreos-installer (in https://github.com/openshift/ironic-agent-image/pull/31).
https://bugzilla.redhat.com/show_bug.cgi?id=2034527

/cc @dtantsur

/lgtm

/hold
There might be some dependency on other PRs for this fix.

There shouldn't be any dependency, this is just setting an environment variable.
/hold cancel

/test e2e-metal-ipi-serial-ipv4


LITE-18056 Added CQRS retry docs

Codecov Report

Merging #31 (37e9886) into master (e214de6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   95.63%   95.63%
=======================================
  Files          30       30
  Lines        1559     1559
=======================================
  Hits         1491     1491
  Misses         68       68

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 e214de6...37e9886. Read the comment docs.


There should be a feature where  the user can select the text generated.

Currently, all the words generated by the word cloud are only in upper cases, there should be a feature where the user can decide what types of words can be generated

all uppercase
all lowercases
the first character is uppercase

This would require more select boxes to be added on the front end too.

Hi, I'd like to work on this. Could you assign it to me?

Sure. 🥂


list.with does not work

I have this example of code:
+alias org.eolang.collections.list
+alias org.eolang.txt.sprintf
+alias org.eolang.io.stdout

[] > app
  list > list1
    * 1 2
  list1.with 3
  seq > @
    []
      stdout > @
        sprintf
          "%d\n"
          list1.at 2

The result of a program should be 3, but there is an error, that list1 has only two elements:
EOorg.EOeolang.EOstringν68="Can't at() the 2th element of the array, there are just 2 of them"SF

When I tried to print 0 and 1st elements of list1 it was OK and I got numbers 1 and 2 respectively.

@Graur @mximp pls, have a look.

@ivan-egorov42 list.with creating a new array and do not modifying an old one

@Graur OK, thanks! I didn't understand description. Now it really works)


Add Centos/Fedora/RHEL Install Notes

Looks fine to me, :+1: .
Maybe you could condensate the yum commands into one as well, just like the Debian/Ubuntu variant? It's a detail, but it makes the instructions be smaller.

@maxpoulin64
Done, can't consolidate it more than this hence we need epel before asking for nodejs/npm.

@Rosiak Sorry to be strict, can you also please squash those commits together to keep our commit history as clean as possible? (git rebase -i HEAD^^ and set the second commit to squash, then force push the branch).

maybe also include a note that newer fedora users should use dnf instead of yum?

Let me know if this is ok.

@MaxLeiter
Changed.

Looks good to me :+1:


[PATCH] README clarity, explicit port config, and npm audit fix

Improved the readme to be more clear as to which menus you are looking for (steam version has menus at the top, but only very limited remote API controls from there) and how to debug some issues

Added the port config explicitly to the viteburner config file, to make for easier debugging, and changed the suggested port to one that is less likely to collide with other apps

Ran npm audit fix to update dependencies and remove vulnerability warnings

Hiding port option from default config is by the design of vite. The port 12525 is chosen because the [bitburner-filesync](https://github.com/bitburner-official/bitburner-filesync) package use the same port so no problem will occur while switching. What application does the port 12525 collide with on your environment? Is there any reason that 31337 is chosen?
The deps are old now, I should definitely upgrade them.

I was unable to find exactly what was colliding on port 12345 on my local, but picking any other port allowed me to successfully get a connection.
Even if 12345 has good reason to be kept, it would have taken me a lot less time debugging to discover where that number came from if it was in the project anywhere, so I think it is a good idea to have it on-hand (even if it is commented out as a default in the typescript tradition e.g. https://gist.github.com/ryanatkn/f1793abbe020e08b785cb56d4a712af1 )

( I chose 31337 just to fit the Snow Crash / h4x0r theme of Bitburner, clearly its an elite/31337 game XD)

The default port used by viteburner is written in README and the current one will shown on console so I don't think it will be difficult to find out which port viteburner is using. It may be a bit difficult to find where to change it though. I'll consider to add port option to the default config.
To detect using port by applications, you can use resmon on Windows or netstat on both Windows and Linux. I don't think it is worth changing the default port as it is not used by some commonly used software and it will bring unnecessary migration cost to the end users.


Fixed #29352 -- Allowed specifying a Feed language.

https://code.djangoproject.com/ticket/29352
This PR follows up on the abandoned one at #9879.
I am aware that there are still tests missing that should be added to tests/syndication_tests, but I could use some guidance on which tests to add and where.

Thanks to @carltongibson I added tests, too.

Not sure where the build failure against MariaDB came from, doesn't look related to me.

Thank you for your review, I've added the changes you requested.

@rixx I noticed that we have Language section in feed's documentation so I moved versionchanged there, I also made some cosmetic changes in docs and release notes. Can you take a look?


pkg_rpm: Don't have source_date_epoch apply by default; test modularity cleanup

The logic for pkg_rpm's source_date_epoch attribute should apply only when
set.  This change updates the logic so that a negative value for
source_date_epoch results in no --source_date_epoch being passed to
make_rpm.py.  None does not seem to be a valid option.
The default is now a negative number, thus disabling this feature unless
explicitly requested.
A regression test was also provided.  While writing it, I noticed a few quirks
in rpm that impacted test execution.  These were also addressed, although
perhaps not in the best way.
For more context, see #486.
NOTE: this does not address the similar issue impacting the legacy pkg_rpm
rule.  This is mentioned in #488.
Fixes #486

Thanks for the review!


GIF parses out redundant images

in release/v7, use this GIF，Caused extra images to be parsed, please help to see what caused the problem?
In addition, is there a useful tool for parsing GIF data?

Is it the same with v8 where gif is already part of lvgl?

Is it the same with v8 where gif is already part of lvgl?

yes, my lvgl version is v7,  latest v7 version of my lv_lib_gif

I mean can ou try out in a simulator with v8 too?

yes, you can try it!

I mean can ou try out in a simulator with v8 too?

how about?

I mean can ou try out in a simulator with v8 too?
hi，do you try it? how about it?

I meant you can try it out if the issue still persists.  Anyway I've just checked it an v8 it looked fine.
Can you try it out on your end too with v8?

Also, I'm developing with lgvl v7 now

I think it's just working accidentally.
I tried a lot optimization things and if I recall correctly the final solution is only included in v8.
I suggest backporting the gif decoder from v8 to v7.

I have compared the difference between v8 and release/v7 versions of gif parsers and they are no different in my opinion

I think it's just working accidentally. I tried a lot optimization things and if I recall correctly the final solution is only included in v8.
I suggest backporting the gif decoder from v8 to v7.

but,i can try it

I think it's just working accidentally. I tried a lot optimization things and if I recall correctly the final solution is only included in v8.
I suggest backporting the gif decoder from v8 to v7.

In addition, why does it cause this problem when gif->bgindex = 0xff?

It seems the problem is related to the transparent background. Wikipedia says:

Each frame can designate one index as a "transparent background color": any pixel assigned this index takes on the color of the pixel in the same position from the background, which may have been determined by a previous frame of animation.

I remember that tested this but there might be some issues.
I suspect that the offending line is this: https://github.com/lvgl/lvgl/blob/master/src/extra/libs/gif/gifdec.c#L486

It seems the problem is related to the transparent background. Wikipedia says:

Each frame can designate one index as a "transparent background color": any pixel assigned this index takes on the color of the pixel in the same position from the background, which may have been determined by a previous frame of animation.

I remember that tested this but there might be some issues.
I suspect that the offending line is this: https://github.com/lvgl/lvgl/blob/master/src/extra/libs/gif/gifdec.c#L486
My current solution is： if (gif->bgindex == 0xff) gif->bgindex = 0`，Do you think this solution is reasonable？I need your help, thanks

My current solution is： if (gif->bgindex == 0xff) gif->bgindex = 0`，Do you think this solution is reasonable？I need your help, thanks

gif->gce.tindex is the index of the transparent color. So if it's incorrect then it should be tracked why is not set correctly.
By definition, gif->bgindex = 0 doesn't have any special meaning.

By definition, gif->bgindex = 0 doesn't have any special meaning

yes,i know gif->gce.tindex  is the index of the transparent color, it is included in every frame of the picture, but i use cmd gifsicle -I  this.gif， i found every frame index of the transparent is 255=0xff, it doesn't look like any difference

hi，I know what is causing the above problem，The above gif image is very special, and it has several characteristics that cause the above problems。
When I use gifsicle --cinfo this.gif, I find transparent 255 is #000000 of the global colormap and local colormap of this image, and this.gif  gif->gce.disposal is always 3，It will cause the following problems：
Step1：in gif_open function ， it won't go into the following case：https://github.com/lvgl/lv_lib_gif/blob/27a74507ee73ae462daf8f4d4c2f65b366fc9cfd/gifdec.c#L138-L155
Step2：because gif->gce.disposal = 3，it's restore to previous, don't update canvas
Step3：because every frame Local Color Table gif->gce.tindex= 255, and this data is #000000
So I think the above situation leads to why the background color has an extra white line.
Also, I think white background color is not supported in release/v7，I think the code should be changed like this：
delete this line code： if (bgcolor[0] || bgcolor[1] || bgcolor [2])

in this issues，https://github.com/lvgl/lv_lib_gif/issues/15 his problem is similar to mine,  But I don't think your explanation is accurate：https://github.com/lvgl/lv_lib_gif/issues/15#issuecomment-768136218
in his gif https://github.com/lvgl/lv_lib_gif/issues/15#issuecomment-765151040 and https://github.com/lvgl/lv_lib_gif/issues/15#issuecomment-765171277
I found a similar question，use cmd： gifsicle --cinfo xx.gif，this global color table[63] = #000000， and this background = 63，so when parsing these two gifs, the code of step1 above cannot be entered.
But luckily，this two gif gif->gce.disposal = 2，so, it can  restore to background color，so，this why a white line flashes on the background.
I think there are two problems in release/v7 lv_lib_gif,
1：when background data = #000000, it's caused some problem.
2：in v7 lvgl it's not set white background

I think I mostly understand :slightly_smiling_face:
But isn't #000000 black?

I think I mostly understand 🙂 But isn't #000000 black?

but，

I think I mostly understand 🙂 But isn't #000000 black?

In actual operation, it does cause the problem

I think I mostly understand 🙂 But isn't #000000 black?

when i delete if (bgcolor[0] || bgcolor[1] || bgcolor [2])
All gifs are back to normal
and i think this if (bgcolor[0] || bgcolor[1] || bgcolor [2])  looks useless

I think I mostly understand 🙂 But isn't #000000 black?

yes, in RGB #0000000 is black，but, the actual running scenario does
and when i delete this code if (bgcolor[0] || bgcolor[1] || bgcolor [2])  every gif run is back to normal
this code if (bgcolor[0] || bgcolor[1] || bgcolor [2]) llooks useless

To be honest I also don't know what is the point of if (bgcolor[0] || bgcolor[1] || bgcolor [2]).
So can you confirm that removing it all works well?

yes，i can
---Original---
From: "Gabor @.>
Date: Thu, Feb 17, 2022 21:38 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [lvgl/lv_lib_gif]  GIF parses out redundant images (Issue #38)
To be honest I also don't know what is the point of if (bgcolor[0] || bgcolor[1] || bgcolor [2]).
So can you confirm that removing it all works well?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

yes, I can
---Original---
From: "Gabor @.>
Date: Thu, Feb 17, 2022 21:38 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [lvgl/lv_lib_gif]  GIF parses out redundant images (Issue #38)
To be honest I also don't know what is the point of if (bgcolor[0] || bgcolor[1] || bgcolor [2]).
So can you confirm that removing it all works well?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Great!
I fixed it in lvgl/master here lvgl/lvgl@b1e2c06

Thanks，i will colse this issue

Thanks for the debugging and finding the root of this issue! :1st_place_medal:


Server looping on start - loops at "Downloading Bedrock server version <IP_ADDRESS>"

Hi,
As of just today (7th oct 2022), whenever my docker container starts, it gets stuck at "Downloading Bedrock server version <IP_ADDRESS>" and then restarts.
I've captured the logs (below), and I've also tested that I can access and download the URL for the file  (https://minecraft.azureedge.net/bin-linux/bedrock-server-<IP_ADDRESS>.zip) from my own computer and the box that's running the docker container (using wget).
Logs:
minecraftbds_logs.txt
Any help would be amazing!

It works fine for me.
Something restarted your container right after:
2022-10-07T01:41:48.511061000Z + rm -rf bedrock_server 'bedrock_server-*' '*.so' release-notes.txt bedrock_server_how_to.html valid_known_packs.json premium_cache

Maybe you have set the healthcheck too short?

I have the same issue.
I use NFS as my persistent storage, and it takes a really long time with both the backup and unpacking the new version. (~20 min in total)
This is due to poor performance on (my) NFS when writing many (small) files.
Using tar (or similar) for the backup instead of just moving the folders would help for the backup part, but the performance is still a problem when unpacking the new version.
Adjusting the healthchecks would work, but a better solution might be to add a startupProbe.
That way, you can have sane liveness and readiness probes for normal restarts, and still give the pod the time it needs to start up when performing a slow backup/upgrade.
https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-startup-probes/#define-startup-probes

So I managed to figure out what it was. I had my BP and RP folders mapping as volumes to other locations outside of the hosts mapped /data volume. As soon as i moved them back into the native /data volume, it started working.
I did check permissions and everything on them, and the folders all had identical permissions so I'm not sure why it was failing, but that's how I fixed it.

Issue seems to be persiting
**2023-02-18 22:14:02 Looking up latest version...
2023-02-18 22:14:03 Downloading Bedrock server version <IP_ADDRESS> ...**

@tkacprow it has to be some kind of networking issue, but unfortunately hard to say what. FWIW, I updated the latest image to ensure a 1 minute timeout on the version retrieval so that it at least doesn't hang forever. I'm not sure if that will really make a difference.

Huh running into the same problem. Has anyone found a solution yet? I can't launch the server - all I ever get is:
Looking up latest version...
Downloading Bedrock server version <IP_ADDRESS> ...
Backing up behavior_packs into backup-pre-<IP_ADDRESS>
time="2023-06-03T13:20:36Z" level=debug msg="Forwarding signal" signal=terminated
time="2023-06-03T13:20:36Z" level=debug msg="Sending message on stdin due to SIGTERM" message=stop


DashoGames.PiozaGameLauncher version 0.4.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree company="DashoGames"

Manual Validation ended with:

2024-07-15 09:10:30.315 [FAIL] Installer failed security check. Url: https://github.com/Shieldowskyy/PiozaGameLauncher/releases/download/v.0.4.0/PiozaGL.exe Result: 0x80004005

(Automated response - build 886.)

/azp run

/azp run

Verify with above dependencies:

https://github.com/microsoft/winget-pkgs/issues/107639
https://github.com/microsoft/winget-pkgs/issues/12906


Log-1376: Add service monitor for ocp prometheus to scrape metrics

This PR:

Adds a service monitor as part of reconcilation of loki cluster.
Add unit test cases wherever applicable for the same.

/cc @periklis
Links:
Jira - https://issues.redhat.com/browse/LOG-1376

Will rework and create a new pull request.


SVG support

It would be nice to support SVG sprites as well. glue would take SVG files as input and output both PNG and SVG sprites, as well as the needed CSS to get compatibility with older versions of IE or Android Browser.
Here is some information for a nice cross-browser fallback:
http://tobias.is/geeky/webperf/cross-browser-css-technique-for-svg-sprites-with-png-fallback/
Android 2.x support could be added by the way of Modernizr (with .no-svg).

This would be nice. Support could be added by switching to Wand and dumping Pillow/PIL

For reference, node has svg-sprite, it supports SVG optimization via SVGO. Python's equivalent to it is probably Scour


nwg-panel: 0.9.38 -> 0.9.39

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for nwg-panel is: GTK3-based panel for Sway window manager
meta.homepage for nwg-panel is: https://github.com/nwg-piotr/nwg-panel
meta.changelog for nwg-panel is: https://github.com/nwg-piotr/nwg-panel/releases/tag/v0.9.39
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.39 with grep in /nix/store/x5wh6vsfjq3bh0i0ijbfw50pghx1hhhq-nwg-panel-0.9.39
found 0.9.39 in filename of file in /nix/store/x5wh6vsfjq3bh0i0ijbfw50pghx1hhhq-nwg-panel-0.9.39

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

nwg-panel

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x5wh6vsfjq3bh0i0ijbfw50pghx1hhhq-nwg-panel-0.9.39 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nwg-panel https://github.com/r-ryantm/nixpkgs/archive/c6ecc28531aa4b37655c60982e2c68c5ce8b313d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c6ecc28531aa4b37655c60982e2c68c5ce8b313d#nwg-panel

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x5wh6vsfjq3bh0i0ijbfw50pghx1hhhq-nwg-panel-0.9.39
ls -la /nix/store/x5wh6vsfjq3bh0i0ijbfw50pghx1hhhq-nwg-panel-0.9.39/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

nwg-panel
nwg-panel.dist

Maintainer pings
cc @ludovicopiero for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


Fix "BEGIN TRANSACTION" being serialized as "START TRANSACTION"

"START TRANSACTION" is not valid in SQLite, which means that valid
SQLite SQL code, when parsed and re-serialized by sqlparser, would become invalid.
Fixes #919

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 11 (63.64%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 87.14%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ast/mod.rs
5
7
71.43%

src/parser.rs
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
15497

Relevant Lines:
17784

💛  - Coveralls

@alamb , is that okay for you? I need this for SQLPage.

LGTM @lovasoa Thanks

thanks @alamb . I implemented your suggestions.

Thanks @lovasoa


Is there any supported way to encode a message as a url-safe base64 string?

By "supported" I mean without having to implement a custom BufMut. Maybe it would make sense to have this built in since in google's design patterns they recommend page_tokens be represented this way.
"Page token contents should be a url-safe base64 encoded protocol buffer."

From what I understand, page_token itself should be a protocol buffer. Encoded as a string. The whole response is still a normal protobuf message, encoded in whatever way gRPC wants.

Right so I don't think it makes sense to have an added layer to do the base64 and instead you should just do it ontop of the bufmut buffer.

Ok, I guess I'll implement this and post my solution here and then I'll close this.
For anyone that might have the same use case.