'NoneType' object is not subscriptable

I got an error that the 'NoneType' object is not subscriptable. I created an AsyncGraphDatabase.driver. Got response from neo4j db with the initial request. Later after one hr approx I got the error as below.
Neo4j Version: 4 Professional
Neo4j Mode: Aura DB
Driver version: Python driver 5.0.0a1
Operating System: Windows 11
Steps to reproduce

Run a query with the AsyncGraphDatabase.driver.
Receives response
keep idle for 1hr
Run the same query

Expected behavior
The second query shall run successfully
Actual behavior
File "C:\Users\aksha\Downloads\api\.\utils\AsyncNeo4jClient.py", line 53, in get_data
    data = await db.read_transaction(work)
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\work\session.py", line 407, in read_transaction
    return await self._run_transaction(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\work\session.py", line 327, in _run_transaction
    await self._open_transaction(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\work\session.py", line 257, in _open_transaction
    await self._connect(access_mode=access_mode)
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\work\session.py", line 109, in _connect
    await super()._connect(access_mode)
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\work\workspace.py", line 80, in _connect
    self._connection = await self._pool.acquire(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 657, in acquire
    await self.ensure_routing_table_is_fresh(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 602, in ensure_routing_table_is_fresh
    await self.update_routing_table(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 552, in update_routing_table
    if await self._update_routing_table_from(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 495, in _update_routing_table_from
    new_routing_table = await self.fetch_routing_table(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 436, in fetch_routing_table
    new_routing_info = await self.fetch_routing_info(
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 406, in fetch_routing_info
    cx = await self._acquire(address, timeout)
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_pool.py", line 116, in _acquire
    await connection.close()
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_bolt4.py", line 330, in close
    log.debug("[#%04X]  C: GOODBYE", self.local_port)
  File "c:\users\aksha\appdata\local\programs\python\python39\lib\site-packages\neo4j\_async\io\_bolt4.py", line 100, in local_port
    return self.socket.getsockname()[1]
TypeError: 'NoneType' object is not subscriptable

Thanks for the report.
Unfortunately, I was not able to reproduce it (at least not without waiting 1h, which is a little too long for debugging). But I pretty sure, that https://github.com/neo4j/neo4j-python-driver/pull/724 (for 5.0) and https://github.com/neo4j/neo4j-python-driver/pull/726 (for 4.4) will fix this.
I am not quite sure what makes socket.getsockname() return None after a connection was established. I'm guessing it might be an OS specific thing, since you reported to run on Windows.

@ccloes Could you look at using https://neo4j.com/docs/api/python-driver/current/api.html#liveness-check-timeout-ref it should prevent this exception being thrown in the application layer, and the driver should continue functioning.
Could you also provide any further context on how this error occurred? thanks.

We are using that already.  This is how we are trying to utilize it in the context of our NLB.
    # According to this:
    # https://medium.com/expedia-group-tech/lesson-learned-from-a-nlb-connection-timeout-2c672e59e197
    # NLB has a 350 second timeout. We set the timeout to 300 seconds to be safe.
    max_connection_lifetime: 300
    # As of driver version 5.15, the `liveness_check_timeout` option is available.
    # https://neo4j.com/docs/api/python-driver/current/api.html#liveness-check-timeout-ref
    # We'll set that to half the `max_connection_lifetime` in order to ensure that
    # the driver is able to detect a dead connection before the NLB does.
    liveness_check_timeout: 150
    ```

    Do you have any experience with NLB timeouts and how best to mitigate these connection issues?

While I do not, but it is a good place to start looking(in that we do not see many reporting this issue and load balancers are very capable of doing annoying network behaviours). For a little more context, are you attempting to connect to a neo4j instance through an NLB or is your application behind a NLB and it is reaching out to a neo4j instance i.e. AuraDB?

Thanks for your input @ccloes. I've moved your comment into a separate issue to track it separately as it's very similar but not quite the same situation as the original post.

@ccloes can you please take a look at the linked issue and provide the additional information I've requested so that I can continue investigating the error? This would be a tremendous help for us.


⚙ Build: Subquery manifest

The subquery manifest need to be reviewed to make sure subquery is set up correctly. See official documentation

Pretty sure it's already addressed by the lint-subql-project job.

@ccamel Sorry the description might be a bit vague this issue is to review the manifest configurations like version handling, how to handle the startblock etc.

Oh, sure. I see your point. I believe this should also be considered in the context of releases and deployments.

Implemented with https://github.com/okp4/subql-okp4/pull/152.


Garbage in the end of the page

I mean the following two lines:
"�
�"

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

ID: ddd8d0d4-08cf-039d-25cd-5f32cd45797f
Version Independent ID: 9d8e6ed9-ce8b-2e70-fac2-1b56017a34a4
Content: ACPI system description tables
Content Source: windows-driver-docs-pr/bringup/acpi-system-description-tables.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

@sahikon thanks for catching this issue. The non-printing characters have been removed and the changes should be live soon.


⚠ ClamAV has degraded performance

In 186b25f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 44 ms

Resolved: ClamAV performance has improved in 8bdf29d.


shader: Array initialization doesn't work for OpenGL and Metal

foo := [10]float{
        -0.08, -0.05, -0.03, -0.02, -0.01, 0.01, 0.02, 0.03, 0.05, 0.08,
}

cannot be compiled.
OpenGL:
array constructor supported in GLSL ES 3.00 and above only
Metal:
Simply this is a syntax error.

In OpenGL, assigning an array to an array is forbidden!
shader compile failed: ERROR: 0:116: '=' disallowed on array types

For assigning an array, let's create another issue.


Add multiverso version fb.resnet.torch as submodule.

This is to add a multiverso version fb.resnet.torch. Actually I am not sure whether we should make this as a submodule. A submodule will make it easy to update the code base when the upstream has new commits. But users will need extra effort to get the code base. And it's a little strange to have a submodule from my account. Maybe we should use Microsoft account to fork original fb.resnet.torch?
@chivee @feiga @you-n-g , any suggestions?

I think writing some guidance and link in the document will be enough.

Change the url to repo under official Microsoft account, I think it's ready for review now.


Wrong folder reference generated with yo jasmine

Following these instructions, I used Yeoman to install Jasmine with yo jasmine.  The problem: tests/index.html was requiring files in a non-existent directory: /bower_components/jasmine/, whereas the proper directory name is /bower_components/jasmine-core.
This should be an easy fix, and I'm happy to try making a pull request sometime soon if someone else doesn't get to it first.

@jayqui Pr welcomed :)
I think jasmine change the structure of their project in a release and it just haven't been copied over here.

@SBoudrias Clicking around a little more, it looks like there's been a PR ready to be merged the whole time: https://github.com/yeoman/generator-jasmine/pull/14 .  I'm unsure who has merge permissions.

Hum, merged!

As you can see, this generator is not actively maintained anymore. We're looking for new maintainers if you're interested.


conda-build may raise false alarm on Windows?

During the testing of CUDA 12 + Windows (https://github.com/conda-forge/cupy-feedstock/pull/228), we see this warning
WARNING (cupy,Lib/site-packages/cupy_backends/cuda/api/runtime.cp310-win_amd64.pyd): Needed DSO Library/bin/cudart64_12.dll found in ['cuda-cudart_win-64']
WARNING (cupy,Lib/site-packages/cupy_backends/cuda/api/runtime.cp310-win_amd64.pyd): .. but ['cuda-cudart_win-64'] not in reqs/run, (i.e. it is overlinking) (likely) or a missing dependency (less likely)

which seems to be saying we missed a run_exports somewhere in packages generated from this feedstock, but it's not the case. cuda-cudart_win-64 is installed in both build and test stages just fine. Also, there's no problem on Linux, and the export relation (cuda-cudart-dev exporting cuda-cudart that depends on cuda-cudart_{{ platform }}) is the same on both OS. Seems like a false alarm that can be safely ignored.

In the Linux case, we don't see issues as libraries link to the symlinks in cuda-cudart instead
If we look at the Linux case, we see something like the following
INFO (cupy,lib/python3.9/site-packages/cupy/cuda/thrust.cpython-39-x86_64-linux-gnu.so): Needed DSO lib/libcudart.so.12 found in conda-forge::cuda-cudart-12.0.107-h59595ed_6

It doesn't warn here as the the library links to the SO symlinks, which ship with cuda-cudart
Interestingly conda-build doesn't warn that the symlink points to a file in a different package (cuda-cudart_linux_64). Maybe it should?
https://github.com/conda-forge/cuda-cudart-feedstock/blob/d6fd09777c2765aff562fbab42427c8637465df8/recipe/meta.yaml#L68-L70

One option would be to modify cuda-cudart-dev's run_exports
https://github.com/conda-forge/cuda-cudart-feedstock/blob/d6fd09777c2765aff562fbab42427c8637465df8/recipe/meta.yaml#L104-L106
...to include...
     - {{ pin_subpackage("cuda-cudart_" + target_platform, max_pin="x") }}

This is a bit redundant
Also not sure offhand if this would have other consequences. It could be restricted to Windows
Maybe there are better solutions

What would happen to cuda-cudart_win-64 then?

I am seeing this same error on the libmagma-feedstock. Except on the libmagma feedstock, the warning is promoted to an error.

What would happen to cuda-cudart_win-64 then?

It's not needed as we don't do cross-compilation for windows.

Thought we wanted to avoid special casing for Windows though ( https://github.com/conda-forge/staged-recipes/pull/21723#discussion_r1095048318 )
Currently am seeing the following:

cuda-nvcc_{{ target_platform }} depends on cuda-cudart-dev_{{ target_platform }}
cuda-cudart-dev_{{ target_platform }} depends on cuda-cudart_{{ target_platform }}

If we start dropping the various *_{{ target_platform }} packages on Windows, we would need to special case Windows in multiple cases
Am wondering whether this is worth it to avoid a warning
What other options might we have that would involve fewer touch points?

We could try to get symlinks working for the Windows package like they do for Linux. According to this blog post from 2016, you no longer need admin priviledge to create symlinks, but you may need to enable "Developer Mode". I'm not sure whether "Developer Mode" is enabled on the Azure workers.

I prototyped using relative Symlinks on Windows to mirror how the Linux packages work. Unfortunately, it seems that conda does not handle installting symlinks on Windows correctly.
ERROR conda.core.link:_execute(740): An error occurred while installing package 'local::cuda-cudart-12.0.107-h63175ca_6'.
Rolling back transaction: ...working... done
Traceback (most recent call last):
  File "C:\Users\dchin\miniconda3\Scripts\conda-build-script.py", line 10, in <module>
    sys.exit(main())
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda_build\cli\main_build.py", line 495, in main
    execute(sys.argv[1:])
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda_build\cli\main_build.py", line 475, in execute
    outputs = api.build(
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda_build\api.py", line 180, in build
    return build_tree(
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda_build\build.py", line 3089, in build_tree
    test(pkg, config=metadata.config.copy(), stats=stats)
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda_build\build.py", line 2879, in test
    environ.create_env(metadata.config.test_prefix, actions, config=metadata.config,
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda_build\environ.py", line 1058, in create_env
    execute_actions(actions, index)
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda\common\io.py", line 84, in decorated
    return f(*args, **kwds)
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda\plan.py", line 322, in execute_actions
    execute_instructions(plan, index, verbose)
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda\plan.py", line 532, in execute_instructions
    cmd(state, arg)
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda\instructions.py", line 73, in UNLINKLINKTRANSACTION_CMD
    unlink_link_transaction.execute()
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda\core\link.py", line 283, in execute
    self._execute(tuple(chain(*chain(*zip(*self.prefix_action_groups.values())))))
  File "C:\Users\dchin\miniconda3\lib\site-packages\conda\core\link.py", line 756, in _execute
    raise CondaMultiError(
conda.CondaMultiError: [Errno 2] No such file or directory: 'C:\\Users\\dchin\\miniconda3\\pkgs\\cuda-cudart-12.0.107-h63175ca_6\\Library\\bin\\cudart64_12.dll'
()

It's weird because if I unpack the archives manually, the symlinks seem to be intact, so I'm not sure what's up. There are some other open issues related to links on Windows, though.
https://github.com/conda/conda/issues/12465
https://github.com/conda/conda/issues/10521

Sorry, but I think I'm not quite following the proposal, Daniel
How would symlinks be used here (assuming we could get them to work)?

I think @carterbox might be proposing to match the behavior on Linux. That is, symlink from the targets folder (provided by the cuda-cudart_win-* package(s)) to %PREFIX%/Library/bin.
Unfortunately, symlinks on Windows are still a dumpster fire that we can't rely on. Beyond that, there's myriad code built around the assumption that symlinks don't work, so it would be a major effort to support them. Requiring users to change to "developer mode" or whatever is just too much to ask, unfortunately.
I agree that we need to avoid special casing on Windows. Even if we decide to throw cross-compiling out the window on all OSes, NVIDIA still wants to keep the targets/... folder structure, because that matches CUDA packages distributed in other places. Breaking the targets structure is more disruptive to NVCC and friends.
My inclination is to follow @jakirkham suggestion at https://github.com/conda-forge/cuda-cudart-feedstock/issues/13#issuecomment-1805041487, assuming that it works to fix the issue. If this affected only one consumer package, I'd just ignore the warning/error with a Conda-build option. Since this seems like it will affect any package using CUDA, we need a less ignorant approach.

I think @carterbox might be proposing to match the behavior on Linux. That is, symlink from the targets folder (provided by the cuda-cudart_win-* package(s)) to %PREFIX%/Library/bin.

Yes, exactly.

Requiring users to change to "developer mode" or whatever is just too much to ask, unfortunately.

For clarity, my understanding is the "Developer Mode" is only required for creating new links, not using them once they exist, so I don't think it would require every Conda user to enable "Developer Mode" only creators of packages that contain symlinks. No comment on the usability/reliability of symlinks once they are created.
Why don't we just create copies instead of symlinks? That way the library is in the right place for runtime and also is available for cross-compiling if/when Windows ARM gains support for CUDA. With copy instead of link, we might also trim some dependencies, so that end users don't download two copies of the same file.

Why don't we just create copies instead of symlinks?

Copies are probably a good option here given this should only be required for cudart. Would we need to duplicate .lib files as well? Either way it would only increase package size by <5MB, so may not even be worth trimming dependencies to help keep things less disruptive.

Whether symlinks or copies, am not sure this makes sense in the Windows case as we don't have a targets directory (as on Linux), which means this would create files that exist at the same location in both packages lead
At least atm, am leaning towards the run_exports solution above. It seems like the smallest and least disruptive change we have come up with

Whether symlinks or copies, am not sure this makes sense in the Windows case as we don't have a targets directory (as on Linux), which means this would create files that exist at the same location in both packages leading to clobbering issues

The upstream Windows package doesn't have "targets", but it does have a "lib\x64" folder. I assumed that the corresponding hypothetical arm package would have an "lib\arm64" folder. It's just that in this conda recipe the contents of "lib/x64" is unpacked directly into "lib". So I guess my proposal is to have two copies of whatever files. One in "lib" and the others in "lib\x64" and/or "lib\arm64".
I also just realized that the Windows artifacts in this recipe are probably incorrectly split because of differences between how Unix and Windows artifacts are used:
UNIX:
needed at compile/link time: headers and .SO/.DYLIB (dynamic) or .A (static)
needed at run time: .SO (dynamic) files
WINDOWS:
needed at compile/link time: headers and .LIB (static or dynamic) files
needed at run time: .DLL (dynamic) files
Thus to support the cross-compiling for Windows, the artifact split should be a bit different from unix since DLLs are not needed for cross-compile, but the LIB files are? Thus, the DLL should only appear in the native package, while the LIB file should be copied/linked and appear in cross-compile packages, so something like this:
  # This package is runtime deps for native, we only need DLL at runtime
  - name: cuda-cudart
    files:
      - lib/libcu*.so.*                            # [linux]
      - Library\bin\cudart*.dll                    # [win]
  # This package is compile/link deps for native; we need LIB and header files, let's dump they directly into lib\
  - name: cuda-cudart-dev
    build:
      run_exports:
        - {{ pin_subpackage("cuda-cudart", max_pin="x") }}
    files:
      - lib/libcu*.so  # [linux]
      - lib/pkgconfig  # [linux]
      - Library\lib\cuda*.lib  # [win]
  # This package is compile/link deps for cross-compile; we need LIB and header files keep the LIB files in their target subdirectory
  - name: cuda-cudart-dev_{{ target_platform }}
    build:
      noarch: generic
      run_exports:
        - {{ pin_subpackage("cuda-cudart", max_pin="x") }}
    files:
      - targets/{{ target_name }}/include                               # [linux]
      - targets/{{ target_name }}/lib/*.so                              # [linux]
      - Library\include                                                 # [win]
      - Library\lib\{{ target_name }}\cudart*.lib                       # [win]
  # This package is compile/link deps for cross-compile; unused by Windows because you wouldn't use DLLs when target != host
  - name: cuda-cudart_{{ target_platform }}
    build:
      noarch: generic
    files:
      - targets/{{ target_name }}/lib/libcu*.so.*  # [linux]

The only thing that confuses me is whether headers are target specific as they seem to be fore unix.


Update init containers documentation with 1.6 syntax for YAML files

addressing https://github.com/kubernetes/kubernetes.github.io/issues/3261

This change is

/assign @jberkus @foxish

/assign chenopis for docs review.

/assign chenopis

@foxish So is that a Tech LGTM?

/lgtm
@chenopis Yes.

@chenopis Thanks for the help in rephrasing the description! :)

@crimsonfaith91 Was just implementing the suggestion from @jberkus =)


XDR Read Error: Unknown OperationType member for value 24

I'm using the Typescript client and facing an issue on React side when invoking the write function of the contract.
Error:
Error: XDR Read Error: Unknown OperationType member for value 24
    at h (contentScript.min.js:1:13535)
(anonymous) @ contentScript.min.js:1

Code:
 let data = await exampleProject.setName(
       {
         value:  "John",
       }
     );

Closing as a duplicate of #122.


Copy over IOPS of guest root during image update

If the guest root volume type is PIOPS, copy over the IOPS settings during an image update.
This was verified locally by updating AMIs with and without PIOPS volumes.

LGTM


Add Support for Export to Image

Give the option the user to export the circuit as an image file. It gives the ability to show the circuits in presentation and other programs.

With configurable size of the images


feat: env oracle

This PR introduces the EnvOracle contract that has only one field, namely minimumBeeVersion.
minimumBeeVersion holds the minimum Bee client version required to play in the Redistribution game.
This value must be checked before participating in a round and forbid to send any transaction to the Redistribution contract if the used Bee version is lower than minimumBeeVersion.
This practice ensures that the Bee client is aligned with the breaking changes of the Redistribution contract.

test comment for PR

Have a few thoughts on this one.
The code looks nice, concerns are conceptual, code is basically most about checking proper semver, which could be offloaded to offchain check in deployment code. Usually, that is good practice as onchain computing cost money.
Also I am wondering what would be the integration, do we want to integrate this code into redisitribution.sol or is this going to be just checked from bee node and if it doesn't pass the check they dont call other redis methods, which should be also fine. Maybe to simplify APIs and export for bee node we could use Interface to this contract and just add a call to version in redistribution contract so its just another method there. Would be simpler for them then mapping out new contract. Also if we would just check offchain semver it could just be actually one variable extra in the redistribution contract which would really simplify it all.
Also if there is some idea that we do more dependent integration on SC level, and each call of redis phase is checked on version, then this whole contract should be written as an abstract, and check should be a modifier.
So we could go with any of these choices IMO, its mostly dependent on what concept we want to push out and why.
Wanted to raise this discussion before we finalize it.

code is basically most about checking proper semver, which could be offloaded to offchain check in deployment code. Usually, that is good practice as onchain computing cost money.

IMO it is safer to handle this on-chain, since the chain can be seen as the backend-side of this architecture.
On web2, admin methods also can be implemented on client side in many cases but that is still not a great practice.
If Bee clients integrated EnvOracle and reacted on its events then a little mistake can cause failures that has monetary consequences as well.
I hope Swarm can cover the computational cost to execute this function couple of times : )

Also I am wondering what would be the integration, do we want to integrate this code into redisitribution.sol or is this going to be just checked from bee node and if it doesn't pass the check they dont call other redis methods, which should be also fine.

Latter, because this contract address should be fixed and the Redistribution and other contract addresses can vary.

code is basically most about checking proper semver, which could be offloaded to offchain check in deployment code. Usually, that is good practice as onchain computing cost money.

IMO it is safer to handle this on-chain, since the chain can be seen as the backend-side of this architecture. On web2, admin methods also can be implemented on client side in many cases but that is still not a great practice. If Bee clients integrated EnvOracle and reacted on its events then a little mistake can cause failures that has monetary consequences as well. I hope Swarm can cover the computational cost to execute this function couple of times : )

yes, of course, actual  cost is not an issue here, it's more me as a solidity dev always thinking about making things less bloated and less expensive by default :)
I am also thinking about the process, actually when this changes, it will be changed through that multisig wallet, so I guess it makes sense that we don't make a mistake there in format, also would be really surprised we do as its rather simple variable to enter and check, so we would be really bad in entering and checking data if we do that :)
But yes, it's not an expensive thing to add and do and it would cost a lof more if we somehow make mistake and add wrong format, so ok lets do and be extra safe in that regard.

Also I am wondering what would be the integration, do we want to integrate this code into redisitribution.sol or is this going to be just checked from bee node and if it doesn't pass the check they dont call other redis methods, which should be also fine.

Latter, because this contract address should be fixed and the Redistribution and other contract addresses can vary.

This part is kind of obsolete, as when the redistribution addresses changes then the old contract is paused anyway so they cant use it.
The other thing is that we are aiming to proxy everything in some time so then redis addy would be the same.
Either way think we should add it at as interface to Redis as it would take less integration code for bee node.

I will make a commit with suggestions on how we could make lighter integration to the current workflow.

@nugaon check here a suggestion for integration.
https://github.com/ethersphere/storage-incentives/pull/134/commits/4614b819fa3405ab37e7c0ce81d64649b872ffaa
The reason for this will be that we don't need to change .sh script in /codegen  and also we don't need to add extra data to 007 step in the deploy process and adjust those 2 .json files in the root of the repo. This should also be easier for the bee node to just then call one method in ABI on the same contract and not need to map a new contract.
WDYT?

IMO, either we should handle this value in the Redistribution contract or keep it as a separated and not referenced smart contract.
Actually, that also can be a mechanism for bumping bee version to recognize the old Redis contract is not functioning anymore.

IMO, either we should handle this value in the Redistribution contract or keep it as a separated and not referenced smart contract.
Actually, that also can be a mechanism for bumping bee version to recognize the old Redis contract is not functioning anymore.

I am definitely more on the separation side, as redistribution contact is already big and bloated we don't need another thing added in there, also we could hit 24kb limit at some point and then we would need to again do some separation and refactoring. So I am all for this to be separated. Would like to hear arguments against integration through an interface as I don't see some at this point, but I would be fine with it if this is integrated with bee nodes directly. Not a problem IMO, just then this task needs more coding as it won't work without new go lang bindings, deployments exports, and what else might be missing for it. I made some doc here to list integrations demands, so please check here https://hackmd.io/ofFFKDh7S0aFV1f4n_yhvw

To continue on this issue, we just have a method for checking but not enforcing these checks in the game.
I know its not bullet proof as ppl can fork and change the version but we should think about also implementing that on SC level, with bee nodes submitting their version on commits/reveals
More info on freezings here https://hackmd.io/hyIicg1YQAqMxbtVj_yyqg

Per discussion on Matermost, this was decided that we won't use it in the end. The idea is that we just have a new redistribution contract that will force nodes to update and we won't have old nodes playing.
We can always rethink this down the line.


Fix berkshelf upload

This commit fixes the path from which we are running berks upload.

:+1: Thanks for fixing this! It looks like the vendor task was already doing the right thing, so :shipit:


[gui] Colormap object

PR relative to #146

[x] Move existing Colormap object from plot3d to plot
[x] Add missing functionalities to the object + signal
[x] Integrate the object in existing code
[ ] add management of the signal where needed + unit tests
[ ] check back widgets using colormap
[x] Update doc

Some question reminder :
should the function converting the colormap to a dict should be called getDict or toDict ?

or asDict ;)

I'd avoid getDict to keep get* for 'attributes' of the class.

_getDict?

I think I will go for toDict

Q:

now that the Color is integrated in the plot is it still interesting to conserve the examples/colorbar ?

the management of the colrmap by the colorbar will be part of an other PR ( #913 ), this should prevend not regression, the next one should fix remainging issues of #889

From my point of view the class Colormap in plot3D/scene/function.py should inherit from the general Colormap and the should probably be changed to. What do you think @t20100

Indeed, examples/colorbar.py is now pointless.
What would be interesting is to have an example of making a widget inherited from PlotWidget by adding actions, colorbar and tools (see #916).

The ColorBar widget is a widget which can be used without any dependency. Then an example showing the features is still useful.

I'd prefer to keep plot3d.scene separated from plot.Colormap because the whole scene API was made using Python properties rather than get/set and it does not directly depend on Qt (so no QObject in there), but the top-level plot3d widget API should use it.

@vallsv, agreed if it does not use Plot2D (which is what this PR provides).

Few comments:
from silx.gui.plot import Colormap as genColormap

It would be good to remove the as. Just use Colormap.

_DEFAULT_COLORMAPS = {
    'gray': 0,
    'reversed gray': 1,
    'red': 2,
    'green': 3,
    'blue': 4,
    'temperature': 5
}

It would be good to add a little comment about why this things have numbers.

NORMS = 'linear', 'log'
"""Tuple of supported normalizations."""

NORMALIZATIONS = ('linear', 'log')
"""Tuple of managed normalizations"""

There is a duplication code. It would be good to use only one of this.
I would prefer to have enum, but i am maybe the only one
It would be nice to have const for LOGARITHM  and LINEAR
If we want to expose the Colormap object as gui or plot module, which can be convenient, it would be nice to expose constants as static attributes from the object it-self.

Most of getter function does not have summary docstring. But i dont check the rendering of the doc anyway. I would prefer
Return the upper bounds of the colormap is exists, else None
:rtype: float, None

than
:return: the upper bounds of the colormap or None

You should implement __eq__ instead of compare, and also __hash__ can be very useful.


Unexpected error while viewing conformity details

Describe the bug
Unexpected Django error appeared in logs and browser while viewing conformity details of a machine.
📝 Logs

Internal Server Error: /api/schedules/refresh/
Traceback (most recent call last):
  File "/opt/alcali/.local/lib/python3.9/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/opt/alcali/.local/lib/python3.9/site-packages/django/core/handlers/base.py", line 181, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/opt/alcali/.local/lib/python3.9/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/opt/alcali/.local/lib/python3.9/site-packages/rest_framework/viewsets.py", line 125, in view
    return self.dispatch(request, *args, **kwargs)
  File "/opt/alcali/.local/lib/python3.9/site-packages/rest_framework/views.py", line 509, in dispatch
    response = self.handle_exception(exc)
  File "/opt/alcali/.local/lib/python3.9/site-packages/rest_framework/views.py", line 469, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/opt/alcali/.local/lib/python3.9/site-packages/rest_framework/views.py", line 480, in raise_uncaught_exception
    raise exc
  File "/opt/alcali/.local/lib/python3.9/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/opt/alcali/.local/lib/python3.9/site-packages/api/views/alcali.py", line 294, in refresh
    ret = refresh_schedules()
  File "/opt/alcali/.local/lib/python3.9/site-packages/api/backend/netapi.py", line 185, in refresh_schedules
    for job_name in minion_jobs:
TypeError: 'bool' object is not iterable

Version 3003.1.0

is it still the case with the last version?

Unable to reproduce again, just found it once...


Angular error with codegen and @client directive

Describe the bug
I can't generate code for a query, that looks for a client field.
To Reproduce

My GraphQL schema (client):

extend type Query {
    visibilityFilter: String
  }

My GraphQL operations:

query visibilityFilterQuery {
    visibilityFilter @client
}

My codegen.yml config file:

schema: http://localhost:1337/graphql
documents: ./src/**/*.graphql
generates:
  ./src/app/eddy/generated/graphql.ts:
    plugins:
      - typescript
      - typescript-operations
      - typescript-apollo-angular
config:
  ngModule: ../eddy.module#EddyModule

Expected behavior
When I run: graphql-codegen, I expect a service generated for visibilityFilterQuery. But I get the following error:
./src/app/eddy/generated/graphql.ts
    AggregateError:
        GraphQLDocumentError: Cannot query field "visibilityFilter" on ty
pe "Query".

Environment:

OS:
@graphql-codegen/typescript-apollo-angular 1.0.6:
NodeJS: v10.12.0

Additional context
The codegen works for all "normal" server side queries, AND "inline" client queries.

@dauledk
It looks like you didn't include your client-side schema as part of the codegen schema.
You should have a client-side schema file, for example client-schema.graphql with the following content:
directive @client on FIELD

extend type Query {
    visibilityFilter: String
}

Then, provide it to your codegen configuration:
schema: http://localhost:1337/graphql
documents: ./src/**/*.graphql
generates:
  ./src/app/eddy/generated/graphql.ts:
   schema: client-schema.graphql
    plugins:
      - typescript
      - typescript-operations
      - typescript-apollo-angular
config:
  ngModule: ../eddy.module#EddyModule

This way, the codegen will merge both schemas, and the documents will get validated against the merged schema.

@dotansimha thanks for the reply. Makes sense that I have to provide the schma to graphql-code-generator. But if I convert my client schema from client-schema.ts to client-schema.graphql, how do I provide the schema in the Apollo angular client?
It currently looks something like this:
import { typeDefs } from './client-schema';

export function createApollo(
    httpLink: HttpLink,
    utilities: UtilitiesService
  ) {
  return {
    typeDefs,
    link: concatAllLinks(httpLink, utilities),
    cache: createCache(),
    resolvers: resolvers
  };
}

@dauledk You can use .ts file, just make sure to export the schema as default export, and add require: ts-node/register to your codegen config file

@dotansimha  I could not find anything about require: ts-node/register in the codegen documentation? Is this a module I need to install, and where should it be places in the codegen.yml file?
And thanks for you help - our team is currently implementing Apollo. The help from the Apollo community has been awesome so far!

@dauledk https://graphql-code-generator.com/docs/getting-started/require-field


Update the details for IsImplicitlyDefined packages

I've been asked at least one million times about this. Since NuGet UI points to this section, I'm adding more details and instructions for what users should do.
cc @Rick-Anderson @damianedwards

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tools/Package-Manager-UI.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Edit this to see my raw rewrites. You had two .App so changed the 2nd to .All

For some packages, the Update button is disabled and a message appears saying that it's "Implicitly referenced by an SDK" (or "AutoReferenced"). This message indicates that the package, such as Microsoft.AspNetCore.App, Microsoft.NETCore.All or NETStandard.Library, is part of a larger framework or SDK and should not be updated independently. (Such packages are internally marked with <IsImplicitlyDefined>True</IsImplicitlyDefined>.) For example,  Microsoft.NETCore.App and Microsoft.AspNetCore.All are part of the .NET Core SDK, and the package version is not the same as the version of the runtime framework ~actually~ used by the application. You need to update your .NET Core installation to get new versions of the ASP.NET Core and .NET Core runtime. See this document for more details on .NET Core metapackages and versioning.

Feel free to keep ~actually~ - I'm OCD on deadwood.

@Rick-Anderson I took your advice and made a few tweaks. Microsoft.NETCore.All isn't a real package. It's easy to mix up Microsoft.NETCore.App and Microsoft.AspNetCore.App, so I've reduced the number of times these names are used and made a list instead.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tools/Package-Manager-UI.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

LGTM

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tools/Package-Manager-UI.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@anangaur I don't have merge permissions to this repo. Can you help me bring this in?

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/tools/Package-Manager-UI.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@CamSoper
Please take a look and merge when appropriate.

Ping! Anyone able to merge this one?

Thanks @nkolev92 !


SPFx v1.15 - Error when running Form Customizer solution with a specific configuration

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

Windows 11
Microsoft Edge 103.0.1264.44
SPFx 1.15.0
Node 14.18.1

Describe the bug / error
I'm getting the error below when I'm running a Form Customizer solution with a specific configuration:

Followed this documentation reference:
https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/get-started/building-form-customizer
Steps to reproduce

Generate a brand new Form Customizer solution with the Yeoman generator for the SharePoint Framework.
Run gulp serve --config=basics_EditForm (change the configuration name).

Expected behavior
Running this command should display the edit form for the item specified in the ID property.

@yhabersaat - thank you for reporting this issue.
Just checking - do you have the item with this ID in the list?
I've tested the behavior and it works correctly if you have the item in the list.
And it correctly shows the error if there is no item.

Hi @AJIXuMuK, my bad thank you for pointing the mistake. I confirm that everything is working as expected now! Sorry :-)


Melting of Avalanche comparison primops is wrong

something is wrong here (Ne)

  connectivePrim p = case p of
    Min.PrimRelationGt -> primAnd
    Min.PrimRelationGe -> primAnd
    Min.PrimRelationLt -> primAnd
    Min.PrimRelationLe -> primAnd
    Min.PrimRelationEq -> primAnd
    Min.PrimRelationNe -> primOr

consider melting (x /= Some 0)

(a, b) < (c, d) ==> a < b \/ (a == b /\ c < d)

right now it's just doing:
(a, b) < (c, d) ==> a < b /\ (c < d)

which is just wrong


Commonwealth Coat of Arms MIT license exclusion notice

Description
Appends Coat of Arms license exclusion notice to README.md, and to the footer.liquid include.
Related issues:

#97
#75

Additional information

Also adds a link to the DPMC CoA page: https://www.dpmc.gov.au/government/commonwealth-coat-arms.
Definition of Done

[ ] Content/documentation reviewed by Jools or someone in the #content-design team
[ ] UX reviewed by Gary or someone the Design team
[ ] Code reviewed by one of the core developers
[x] Acceptance Testing

[x] HTML5 validation (CircleCI)
[x] Accessibility testing & WCAG2 compliance

[ ] Stakeholder/PO review

Worth noting I’ve already amended the UI-Kit’s README.md file to this end.

There's some repetition on the home page.
If we have licensing info in both the footer and on the home page we should add where otherwise noted... to the home page instance.
I'd really like to cut down on the amount of stuff we have in the footer. Could we flip it to say, © Commonwealth of Australia. Where available, this work is licensed under the MIT License? Or like?

I've done this work https://github.com/AusDTO/dto-design-guide/pull/130


Water does not protect from dry explosions

An explosion, if detonated while dry, will destroy blocks on the other side of a liquid with "Liquid Multiplier: 0.0". Is this intended functionality? I've had a few players bring this to my attention now. Please see the following videos for examples.
Vanilla TNT functionality:
https://youtu.be/3QuTH6_dCWk
Blowable Obsidian:
https://youtu.be/KBGoqtZFI3U
Using Paper 1.16.2
BlowableObsidians v6.5
Config:
### How much to damage managed blocks
Explosion Settings:
  Default:
    Radius: 3.0
    Damage: 1.0
  TNT:
    Radius: 3.0
    Damage: 1.0
  Creeper:
    Radius: 3.0
    Damage: 0.1
  Wither Creation:
    Radius: 3.0
    Damage: 1.0
  Wither Projectile:
    Radius: 2.0
    Damage: 0.2
  Wither Block Eating:
    Damage: 0.3
## Explosions caused by other plugins:
  Custom Explosions:
    Radius: 1.0
    Damage: 0.5

### Send block health if 100% health.
Always Send Health: true

### How effective explosions damage inside water/lava.
### Damage is multiplied with this if inside a liquid.
Liquid Multiplier: 0.0

### When to send block health message.
Check:
  Item: 'BLAZE_ROD'
  Type: 'RIGHT_CLICK_BLOCK'

### Messages which can be translated.
### You may use <health> instead of <percent>%
Message:
  Block Health: '&9Block health: &c<percent>% hp'

### How much health the blowable blocks has.
### Use wildcard <blocks> for all blocks.
### You can also add more blocks here
Blocks Health:
  obsidian: 7.5
  enchantment_table: 7.5
  anvil: 7.5
  ender_chest: 7.5
  chest: 0.1
  spawner: 0.1
  shulker: 0.1
  shulker_box: 0.1
  white_shulker_box: 0.1
  orange_shulker_box: 0.1
  magenta_shulker_box: 0.1
  light_blue_shulker_box: 0.1
  yellow_shulker_box: 0.1
  lime_shulker_box: 0.1
  pink_shulker_box: 0.1
  gray_shulker_box: 0.1
  light_gray_shulker_box: 0.1
  cyan_shulker_box: 0.1
  purple_shulker_box: 0.1
  blue_shulker_box: 0.1
  brown_shulker_box: 0.1
  green_shulker_box: 0.1
  red_shulker_box: 0.1
  black_shulker_box: 0.1
  end_stone: 3.75
  cobblestone: 1.5
  <blocks>: 1.0

### Health regeneration
### How many seconds until the block health is back at max health.
Regeneration Time: 3600

### Protect bedrock below chosen Y level
Bedrock Protection: 10
# Nether has an override at Y 127

### Don't touch this
Config Version: 11

This is actually intented, but if you want this can be configurable.


linux: 5.9-rc7 -> 5.9-rc8

Motivation for this change
Update kernel to the new rc.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

cc @NeQuissimus (I think you want to be cced on linux bumps / can't we automate this somehow?)


Is this still active

Hi Brady,
I am very interested in WAMP and played around with your gem. It worked so far. But after including the new autobahn.js library (example here: http://crossbar.io/docs/Getting-started-with-Browser/) I could not get it working again. So my question is, if you are still working on this gem and if you are interested to update it.
All the best
Andy
http://autobahn.s3.amazonaws.com/autobahnjs/latest/autobahn.min.jgz

Here is the issue: http://stackoverflow.com/questions/34198566/websocket-error-during-websocket-handshake-sent-non-empty-sec-websocket-proto
What I get is this in the log:
negotiated realm1 as {"roles"=>{"caller"=>{"features"=>{"caller_identification"=>true, "progressive_call_results"=>true}}, "callee"=>{"features"=>{"caller_identification"=>true, "pattern_based_registration"=>true, "shared_registration"=>true, "progressive_call_results"=>true, "registration_revocation"=>true}}, "publisher"=>{"features"=>{"publisher_identification"=>true, "subscriber_blackwhite_listing"=>true, "publisher_exclusion"=>true}}, "subscriber"=>{"features"=>{"publisher_identification"=>true, "pattern_based_subscription"=>true, "subscription_revocation"=>true}}}}
[2016-02-20 03:20:33 +0100] c56ec0e2-9942-45f1-982e-722a850f24c9 prefixes: {"realm1"=>#<WAMP::Topic:0x007ffd999151d8 @uri={"roles"=>{"caller"=>{"features"=>{"caller_identification"=>true, "progressive_call_results"=>true}}, "callee"=>{"features"=>{"caller_identification"=>true, "pattern_based_registration"=>true, "shared_registration"=>true, "progressive_call_results"=>true, "registration_revocation"=>true}}, "publisher"=>{"features"=>{"publisher_identification"=>true, "subscriber_blackwhite_listing"=>true, "publisher_exclusion"=>true}}, "subscriber"=>{"features"=>{"publisher_identification"=>true, "pattern_based_subscription"=>true, "subscription_revocation"=>true}}}}, @clients=[]>}
[2016-02-20 03:20:51 +0100] d4356e91-917e-43c3-9b61-2a9099ccb146 connected
[2016-02-20 03:20:51 +0100] d4356e91-917e-43c3-9b61-2a9099ccb146 disconnected

Hey Andy,
I am not actively maintaining this gem. The reason it does not work with autobahn.js is because wamp-ruby uses the WAMP v1 sub-protocol, however the version of autobahn you linked me to uses WAMP v2.  WAMP v2 is very different from v1 and would require a complete re-write. I have always wanted to write a v2 client but just haven't had the time to do so.
Sorry I could not be of more help.

Hey Brady,
ok that's totally fine ;-). Thanks a lot for the answer and all the best.
Cheers
Andy


Rename package to org.roaringbitmap.v1 or org.roaringbitmap.v2

We are about to merge this API breaking changes to
https://github.com/RoaringBitmap/RoaringBitmap/tree/1.0-SNAPSHOT
It has been proposed in https://github.com/RoaringBitmap/RoaringBitmap/pull/461 that we rename the package.
I do not mind jumping directly to v2, but consider that we never reached v1... and I am not exactly sure why would skip v1.
cc @blacelle @richardstartin

I consider current package as some sort of v1.
We may argue current package is rathrr some sort of v0, like mvn 0.X
Then, moving package to v1, and mvn to 1.x looks hood to me.
Le jeu. 11 févr. 2021 à 20:05, Daniel Lemire<EMAIL_ADDRESS>a
écrit :

We are about to merge this API breaking changes to
https://github.com/RoaringBitmap/RoaringBitmap/tree/1.0-SNAPSHOT
It has been proposed in #461
https://github.com/RoaringBitmap/RoaringBitmap/pull/461 that we rename
the package.
I do not mind jumping directly to v2, but consider that we never reached
v1... and I am not exactly sure why would skip v1.
cc @blacelle https://github.com/blacelle @richardstartin
https://github.com/richardstartin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/RoaringBitmap/RoaringBitmap/issues/463, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQFCFZ3EM5K74QB3FFX6ALS6QS6LANCNFSM4XPL7EGQ
.

I’d just called it org.roaringbitmap.v1 and align it with the library version


adds missing @2x icons

This pull request fixes the following issues:

adds missing @2x icons
updates old icons with new design

can't find  filer/static/filer/icons/plainfolder.png

and plainfolder_32x32.png

@Brankub I just added the icons in every size from 512x512 to 16x16 and x2.

👍


GO term specificity proposal 2  (gene product specific regulation via catabolism)

Remove "gene product specific" regulation via catabolism.
This term, and its descendants
GO:0008054 negative regulation of cyclin-dependent protein serine/threonine kinase by cyclin degradation
cyclin is degraded by the Anaphase promoting complex. This does regulate Cyclin dependent-kinase but this seems a pretty convoluted way to represent this.
We would use
GO:0031145
anaphase-promoting complex-dependent proteasomal ubiquitin-dependent protein catabolic process
has_input (cyclin)
(note this term is now represented more succinctly as
"anaphase-promoting complex catabolic process"
this is clearer, and more informative because it captures the specific pathway, and the substrate

@pgaudet @thomaspd @dosumis

But this is easy to read and understand: "negative regulation of cyclin-dependent protein serine/threonine kinase by cyclin degradation"
While this is pretty hard to parse on first or even second reading - especially if you don't already understand the biology enough to know how to chunk it. (Best to avoid heavily stacked noun phrases like this):
"anaphase-promoting complex-dependent proteasomal ubiquitin-dependent protein catabolic process"
There's also nothing wrong with the logic of the first phrase - regulation may be indirect.
Here's an attempt to break it down to something like a LEGO model using annotation extensions (A LEGO model would be clearer and more accurate though.
'proteasomal ubiquitin-dependent protein catabolic process' has_agent anaphase-promoting complex, has_substrate 'cyclin', regulates cyclin-dependent protein serine/threonine kinase activity.
BTW 'proteasomal ubiquitin-dependent protein catabolic process' is a horrible name.  Wouldn't it be better to have 'ubiquitin-depend protein degredation by proteasome' ?

Some thoughts following discussion in editor's meeting:

Scratch the last suggestion. Given that the existing term is used to annotate APC and it's components, it makes no sense to an extension to with has_agent APC. But then again it also seems odd to have a term which names APC, specifically for the purpose of annotating APC and its components.  Surely you could just annotate APC components to any term that starts 'APC dependent...' without knowing any biology at all.  Isn't it better to annotate the APC complex using MF and BP terms that make clear what it does, and then propagate to genes using the contributes_to qualifier?

I can't make sense of the use case here without a nice textual description of the biology and then maybe a LEGO model.  Here's my quick attempt:

https://en.wikipedia.org/wiki/Anaphase-promoting_complex:
"Anaphase-Promoting Complex (also called the cyclosome or APC/C) is an E3 ubiquitin ligase that marks target cell cycle proteins for degradation by the 26S proteasome."
Ubiquitination regulates degradation of some target - regulating the overall level of activity of that target in a cell.  In this case the target is some cyclin and the result of regulating levels of this cyclin is regulation of the overall level of cyclin dependent kinase activity
Surely the primary annotations should be to
MF: E3 ubiquitin ligase activity
BP: protein ubiquitination leading to degradation # Made this up but it seems reasonable
The BO could then be extended:
protein ubiquitination leading to degradation

has_input some cyclin
regulates some cyclin dependent kinase activity

LEGO is clearly much better suited to capturing the full story, but this gets us some of the way there and uses terms that non-specialists can follow.  It should also be possible to make this fully compatible with a LEGO representation via the consistent use of patterns in the ontology, extensions and LEGO.

One proposal that looked like it might be favored was to merge GO:0008054 (the 'cyclin degradation' term)  into GO:0031145 (the 'APC-dependent catabolism' term). It was proposed to then eliminate the current primary term name of 8054, 'negative regulation of cyclin-dependent protein serine/threonine kinase by cyclin degradation' as a synonym and retain all the 'cyclin degradation'-related synonyms.

What did we decide... I still vote to get rid of the gene product specific catabolism terms, there are much cleaner ways to make these annotations (the suggested merge would work fine). Does this  need further discussion?

I don't recall a discussion or what was decided beyond setting this issue to 'pending'.  @paolaroncaglia , @ukemi , @dosumis , @mcourtot  - can someone remember more?

Sorry, can't remember more :-(

Me neither.

Here's what I suggest to close this issue:

[ ] merge GO:0008054 negative regulation of cyclin-dependent protein serine/threonine kinase by cyclin degradation into GO:0031145 anaphase-promoting complex-dependent catabolic process
[ ]  merge the children terms [pos/neg/regulation of cyclin catabolic process] into [pos/neg/regulation of cellular protein catabolic process]

I think that is what the proposed solution was....
I think I was supposed to update the tickets after the meeting. I found the older on my desktop, so there was clearly something I was still supposed to do here, but hadn't...

Implemented changes above.  Closing.


GEODE-6599: All subprojects should opt into configuration.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased merged against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit? Do you promise to Squash-Merge when the time comes?

[x] Does gradlew build run cleanly?

[n/a] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

This is not intended to be a real improvement, as we still have nonsensicle configuration of, for instance, java-library plugin being applied to non-Java subprojects.  The scope of this PR was intended to position each build.gradle to be reasonable independent of each other, so that the considerations about what each subproject should be can proceed relatively unfettered.

Also, despite the first commit claiming to have removed ide.gradle, that was the result of a mistaken initial branch.  Do note that the second-to-last commit reverts that removal.


Change response timeframe to next business day

👍

Should this also be updated?

I don't know if making that more specific really helps; but if you have a suggestion I'd be interested.


Load file to texture & apply transformations with stacked shaders

The time it takes to transmit data to GPU is non negligible.
Right now what we do is

mmap the file to RAM
create RAM buffers for vertices & colors & the indices of the elements that are worth rendering
transfer these buffers to GPU memory
draw
repeat

Instead we could

load the file into a texture (mmap on GPU?)
create vertices & their colors with shaders (tricky part*)
draw
repeat

This would make the host code much simpler.
Thus porting to Web or Android would be much simpler.
*The tricky part is that to do something like Algo3DCubeFrebet with shaders, one needs to somehow:

read multiple bytes from the file-texture
write a color to the output texture but at different coordinates
I believe that (x1,y1) --> (x2,y2,z) transformation is hard for a GPU.
Maybe compute shaders can help here?

@icehess Ideas?

If shaders can be used to fill a texture then things such as https://github.com/rikusalminen/triangles/blob/613a27d047f0caba934173448260b932e33b0924/shaders/icosphere/icosphere.glslv could be used as well.

https://www.khronos.org/opengl/wiki/Vertex_Texture_Fetch

Some experiments at https://github.com/voidstarHQ/voidstar/blob/b5182cc3758bae1ef1b718e5df37f68a40c24b12/hacking/argh.cc#L401


Add support for running scylla-manager-agent

@ShlomiBalalis  @slivne can you guys take a look on this PR ?

thanks @ShlomiBalalis
@roydahan would give it a final review it


Add possibility to set properties for each item

I'm using iterator plugin in Eclipse Che project https://github.com/eclipse/che-lib/blob/master/websocket-terminal/pom.xml
I need to build golang application for different architectures. It requires environment variable. I use maven exec plugin and would want to make pom more pretty with iterator plugin. But it is not possible to set these variables because I can't set properties for each item.
It would be really useful if iterator plugin supports such configuration:
<items>
    <item>
        <value>linux_amd64</value>
        <properties>
            <goos>linux</goos>
            <goarch>amd64</goarch>
        </properties>
    </item>
    <item>
        <value>linux_arm7</value>
        <properties>
            <goos>linux</goos>
            <goarch>arm</goarch>
            <goarm>7</goarm>
        </properties>
    </item>
</items>

Can you take a look with the current state and check if it works for you?

It works like a charm! Thank you a lot! I've just submitted a PR with usage of new syntax /eclipse/che-lib/pull/7

I think it's time to make it available via Maven Central...WDYT ?

Release to Central done. Take some time to synchronize...

Awesome! Thank you!


refactor: refact topology to support sharding db

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #234 (412861b) into dev (12c94c4) will increase coverage by 0.05%.
The diff coverage is 63.33%.

@@            Coverage Diff             @@
##              dev     #234      +/-   ##
==========================================
+ Coverage   42.82%   42.88%   +0.05%
==========================================
  Files          82       82
  Lines        9840     9857      +17
==========================================
+ Hits         4214     4227      +13
- Misses       5187     5189       +2
- Partials      439      441       +2

Impacted Files
Coverage Δ

pkg/topo/topo.go
75.86% <57.69%> (-1.92%)
:arrow_down:

pkg/misc/strings.go
90.24% <100.00%> (+0.50%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Question about the architecture

Hi, thanks for your great work.
But I am confused by your network architecture. If I understand correctly, in your paper, the contrastive loss is added after projection head $g(f(\cdot))$, while classification is conduct on features after the backbone $f$. While I am reading your code, the logits actually come out of DINO head (passing through three Linear layers) and features coming out at the same place.
Therefore, the network in your paper and implementation seem to be a little inconsistent. Could you please tell me which one is your standard practice? Thank you very much.

Oh, very sorry about that. It seems I misunderstood your codes. This issue has been solved.


InstallBuilder

This PR adds InstallBuilder installer generation for Windows.

This is ready for review and squashing! 😄


WIP: Migration to vector

Hello!
Here is the initial implementation of migration from arrays to vectors. It compiles but fails against some of unit tests. To figure out the root cause, i recompiled sources with enabled check-bounds Yep, seems like most of exceptions were 'index of bounds'. I wanted to debug in cabal-repl, but it failed to load due to Error: bytecode compiler can't handle unboxed tuples and sums. Cabal suggested to specify -fobject-code, but i couldn't google anything about that flag.
So, some assistance is required :)

P.S. diff shows realy huuuuge changeset because of hindent (default setup). If a special profile is used in Arithmoi, please, point me at so i can reduce the number of changed lines.

@tau3 when you run cabal repl or cabal new-repl, simply pass -fobject-code as a GHC option:
cabal (new-)repl --ghc-options=-fobject-code

This goes for any other GHC option(s) you may wish to pass to a GHCi session.

Cabal suggested to specify -fobject-code, but i couldn't google anything about that flag.

https://downloads.haskell.org/~ghc/8.4.1/docs/html/users_guide/ghci.html#compiling-to-object-code-inside-ghci

P.S. diff shows realy huuuuge changeset because of hindent (default setup). If a special profile is used in Arithmoi, please, point me at so i can reduce the number of changed lines.

I do not use hindent for artithmoi, so no agreed profile exists. We can discuss indentation profile as a separate issue, but in course of this PR I am not keen to review thousand lines. Please commit only non-whitespace changes. You can do it as suggested at https://stackoverflow.com/a/7149602 plus several manual tweaks.

@Bodigrim thanks for advices, but unfortunatelly git diff ... -U0 -w ... didn't do the trick with spaces in this case :disappointed:
also i encountered some weird vector bounds errors during debug (debugging itself was kinda stressing due to inability to work with unboxed code in interpreter mode).
probably this refactoring attempt should be considered as unseccessful.
@rockbmb thank you and good luck with yours!)

@tau3 don't call it unsuccessful just yet! You don't have to do everything at once.
You can just pick a module that is all four of:

small,
simple
few other modules depend on it
is still using Arrays instead of Vectors

and make the necessary refactoring. It won't be too difficult, and remember that after it compiles you have a friend in cabal (new-)test.

+1 to @rockbmb
It may be tough to migrate prime sieves from array to vector, but migration of other modules is also highly appreciated.

Closing due to the lack of activity. Anyway thanks for you efforts!


Taskresult type should be string if the type from taskspec is string

Changes
This commit fixes #5803. In tep75&76 we introduced object results. The object results is inferred from the emitted message, and if the message is of json format, then the result type could be object or array. We later validate the type with the type we defined in taskspec. So if the taskpec type is string it will fail validation. Based on the discussions in #5803, if the type in taskpec is string, we should keep the results as is and don't convert it to object or array.
/kind bug

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Taskresult type should be string if the type from taskspec is string

/retest

/approve

@chuangw6 sorry I forgot to cc you. please take a look

Thanks for fixing this. This is a blocker for us to upgrade to the latest release.
I wonder whether it makes sense to add notes to the released version (since the validation was added) with additional description like Known Issues.
Task can't emit string type results that can be parsed as json

Thanks for fixing this. This is a blocker for us to upgrade to the latest release.
I wonder whether it makes sense to add notes to the released version (since the validation was added) with additional description like Known Issues.
Task can't emit string type results that can be parsed as json

Sure. @zhouhaibing089  What did you mean by adding notes to the released version?

@jerop Can we merge this into v0.43 btw? Thanks!

/lgtm


Replace the card images of the project section with GIFs

Replace the card images of the project section with GIFs like the animated counter project

Well, can I do this with GIF zoom on hover?

Can I work on this issue?

@TidbitsJS Yes !! @Devenkapoor7303 It is assigned to @TidbitsJS for Now !! You can check other issues for Now !!

@Vishal-raj-1 Can i work on this issue.

@TidbitsJS Any update or shall i assign this to @ChitvanRamani22 ?

@TidbitsJS Any update or shall i assign this to @ChitvanRamani22 ?

So sorry for the delayed work ... I will create PR by today itself


fixed note style

Changed notes not using the style defined in our guidelines.

Open Publishing Build Service: The pull request content has been published and here are some sample preview links:

Docset 1: core-docs

Looks great.  :shipit:


Improve error message upon failing to resolve a claim

Fixes #723

This LGTM ,
Making note though that line 1495 to 1504 after resolved_result = yield self.session.wallet.resolve(uri) is overly complicated and should be improved.

Merged in 4e4eeae79cab9f044f6c2e51bdc7bfe02a954cea


Try to detect core before the popup opens.

Current logic for detecting uProxy chrome install:

in extension background: if we receive an onInstalled event, call the onInstalled callback
in the onInstalled callback: once we connect to the core, turn the icon green

However right now, the extension doesn't try to connect to the core until the popup is opened (which does not happen automatically if the user is installing from uproxy.org)
Adding core.connect to the background will make sure the extension can detect that the app has been installed, without needing the popup to be opened.
Tested by locally loading the new extension, and installing the app from uproxy.org

We think this might be fixed by https://github.com/uProxy/uproxy/pull/1819/

yep! fixed in #1819


Auto-reconnect in chainsync input plugin

This should be optional but enabled but default. We should reconnect with a number of retries and throw an error on failure. The chain sync should be setup again from scratch, but with our last known chain point(s) as the intersect point

Modifying snek to set a 1s keep-alive timeout for the chainsync input and running the following command will reproduce an "unexpected" disconnect fairly reliably after a few minutes. This is useful for testing.
./snek -filter-type foo -input-chainsync-bulk-mode=true -input-chainsync-intersect-tip=false -input-chainsync-intersect-point 39916796.e72579ff89dc9ed325b723a33624b596c08141c7bd573ecfff56a1f7229e4d09


Java: New Feature: OpenAI o1 model support

name: Feature request
about: Suggest an idea for this project

OpenAI o1 model introduced a number of unique parameters incompatible with other models (1) :

Developer Message as opposed to System Message
reasoning_effort parameter
max_completion_tokens

With o1 being released as GA we need to add the model support as soon as reasonable.
This overlaps with Bug report (2).

o1 announcement
SK BUG 9749

Created issue in th java repo: https://github.com/microsoft/semantic-kernel-java/issues/280


search functionality with ontology

@pvgenuchten this was a remark that came up from @KathiSchleidt  in a WP5 meeting:
What's the planned search approach? Utilizing an underlying Ontology that could sort the synonyms via sameAs, or something more towards Lucene?
not sure if the issue is relevant to the harvester, if not, please move to the relevant repo

We will for sure provide a vocabulary in the backend which facilitates reasoning on language labels, synonyms, sameas, maybe even narrower/broader.
The indexing mechanism could match relevant keywords based on that the above graph
At search time the options are limited for keyword matching, using current technology, elastic/solr may be better here. for the generic search (not keyword filter) parsing the querystrung using llm would provide additional value


Add a pygeos migration guide for Shapely 2.0

The Shapely 2.0 migration guide is now focused on shapely users, but we need to inform existing pygeos users as well.
@jorisvandenbossche

The change of segmentize's tolerance parameter to max_segment_length is one item that needs to be documented. https://github.com/shapely/shapely/pull/1506

The relevant changes from https://github.com/shapely/shapely/issues/1276 are:

preserve_topology=True now default in simplify function(https://github.com/shapely/shapely/pull/1392)
renamed apply to transform (https://github.com/shapely/shapely/pull/1393)
The buffer keyword quadsegs -> quad_segs (https://github.com/shapely/shapely/pull/1512)


⚠ Bofrost.at has degraded performance

In 5802d8d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1978 ms

Resolved: Bofrost.at performance has improved in 76b8525 after 6 minutes.


Updated card hover effect to prevent flickering

Enhanced card hover transition for seamless experience, eliminating flickering and ensuring smooth visual effects.
Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #686
Description
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
https://github.com/anuragverma108/SwapReads/assets/119524954/80fb984d-6308-4d3f-9105-5dbdfeee957d
After
https://github.com/anuragverma108/SwapReads/assets/119524954/797d7f97-990a-4c18-bd12-39998cb66ed8
Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

Conflicting files
assets/css/test-style.css
Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Add TimeoutExecutor decorator

The Executor violates SRP, so this PR is a first step to break this up into multiple independent classes.
This PR preserves full BC by making its timeout feature optional first and then using a TimeoutExecutor decorator instead. This can also be reused for other components, in particular clue/mdns-react. Also, this is needed for upcoming changes related to #12 and #19 which will introduce new executor classes that will reuse this timeout handling.
Builds on top of #35
Somewhat similar to https://github.com/reactphp/socket-client/pull/51

Rebased now that #47 is in :shipit:

Rebased again to fix new tests added via #45 :shipit:

@clue Are you planning to remove the $timeout parameter of the Executor constructor for v0.5?

@jsor Very likely, but not 100% decided yet. I'll make sure to set up a roadmap for this component soon :+1:


寫Google singleton 的 service 和 controller

[ ] 建立 google vision 的 api

took 3 hours
service寫好了
但是還沒推上來

took 0 hours
done


Add meostream.com bypass

 NOTICE
I dedicate any and all copyright interest in this software to the
public domain. I make this dedication for the benefit of the public at
large and to the detriment of my heirs and successors. I intend this
dedication to be an overt act of relinquishment in perpetuity of all
present and future rights to this software under copyright law.

Fixes: #38

Just take link from anchor lol
thanks to lem6ns for helping with the regex

[x] I made sure there are no unnecessary changes in the code*
[x] Tested on Chromium- Chrome OpenSUSE
[x] Tested on Firefox

* indicates required

Works on Chromium & Firefox.


Newline ignored when after last map element (and {:style :respect-nl} used)

If a newline follows the last map element, it will be ignored when {:style :respect-nl} is used:
(czprint "{a\nb\n}" {:parse-string? true :map {:respect-nl? true}})
{a
   b}

Fixed in 1.0.0.


Agnostic QT support

Hello,
Currently this library uses PyQt5, which means that project using PySide2 or 6, and PyQt6 cannot make use of this library. We could use QtPy to be compatible with all bindings.
I'm willing to do it, but I don't know is this project is still being maintained.

Hello I am facing same issue as I am on mac

Thanks for the suggestion!
@Flowake I updated the code to use QtPy instead of PyQt5.


can you add bdmv/dvd menu support. And yt-dlp support in mpv-android

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hello @aicynide, I added yt-dlp, DVD and Blu-ray support into my mpv repository:
https://github.com/MartinEesmaa/mpv-android-vvc/commit/0d95b8bab453594cd205fc7dc27840ba74b78761
https://github.com/MartinEesmaa/mpv-android-vvc/commit/16da2e8e0404e915f930d6f6a58c93bc49e5cbdb
I will build soon as possible for new mpv android with new additional features that you requested.

Martin Eesmaa

@aicynide, finished. :)
https://github.com/MartinEesmaa/VVCEasy/commit/0e673ecabff01c095e5d326872b1f6806f8fe821

Martin Eesmaa

Update: Sorry, I revert build back and this was crashed by libbluray and yt-dlp permission denied error 13. :(

Martin Eesmaa

Actually linux/windows mpv also doesn't support bluray menu

Please add bluray/dvd menu support in mpv. Only yt-dlp support is enough in mpv-android

@MartinEesmaa Please give mpv-android apk with yt-dlp support

Do you think you would be able to add bluray menu support in mpv?

@MartinEesmaa please build mpv-android with yt-dlp only

Hello @aicynide, it seems my device has permission denied to install yt-dlp in my custom build, but here is apk arm64 to test your device: https://mega.nz/file/7l900ZwS#iEfCbHCYNX_BwdDbeqSX_StVjHEyXeJqGejHeRZeph8
Universal architecture of x86, x86_64, arm64 and armeabi: https://mega.nz/file/3lNgEQAA#h1sxEYUvzS1E6cZLrJd3SpSeXDZZDEMqdSXQFsEuyas
If your device says permission denied, it may be custom build error than original build. I may ask to solve permission denied error 13 :/.

Martin Eesmaa

I'm also getting permission error

I'm also getting permission error

Yeah you're right. Same as well too for my device. I guess I will ask why my custom build gets permission error for some reason to mpv community for yt-dlp. :/

I decided to close the issue for some reasons that did not work my custom builds for Blu-ray and DVD menu on MPV Desktop and yt-dlp on MPV Android. Sorry, I tried to add your request features to my custom build, but did not work.
Feel free to reopen the issue anytime.
Regards

Martin Eesmaa


chore: CodeClimate Improvements

Summary
This PR resolves several code issues flagged in CodeClimate and Codacy.
Specifically, this PR does the following:

hash.cpp

use memmove vs raw loop.

crypto_helpers.h

add offset to join method.
add join w/offset test.
improve join test.

serializer.cpp

use join w/offset vs raw loop.
use std::accumulate vs raw loops.

transaction.cpp

initialize local vars in toArray() call.
use memmove vs strncpy & strncat.
use "%" PRIu32 vs "%d" in sprintf call.
use object.length() vs strlen.

Checklist

[x] Tests (if necessary)
[x] Ready to be merged

@faustbrian good to go 👍


Apple root

Apple ```
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

**Describe the solution you'd like**
A clear and concise description of what you want to happen.

**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.

**Additional context**
Add any other context or screenshots about the feature request here.

Closing this because your question didn't come through.  We can reopen if you edit the message with a better description.


Hello, I was wondering how much concurrency the egress service can have without a cluster deployment?

I am now concerned with the amount of egress concurrency, which determines that I need to cluster several egress services. Or I was hoping you could recommend a test.

This is dependent on the workload. Please run your own tests on specific hardware that you are targeting.


Visual glitches during resizing and persisting afterwards until window is moved

While resizing window by grabbing a border windows there are visual glitches, flashing and jumping content.
After we stop resizing we end up with a glitched window. Glitched window is still interactive and will stay like this until window is moved.
https://user-images.githubusercontent.com/10875340/165650462-b5019b65-029e-44a1-a158-c0bdfd1025ad.mp4
I think its a regress as #30 is saying its fixed. - Never mind its a different problem.

I apologize for the very slow response. I can't seem to reproduce the flickering/resizing glitchy behaviour. What platform are you on? And does it happen with an empty neovim configuration?

No problem. I'm glad you're back :)
I'm sorry, I didn't notice when it got fixed. Apparently while dragging the border box of Nvy it's no longer updated continuously and only updates once on release. Same on previous Nvy releases. Maybe it was changed in Neovim itself?
Can't reproduce the problem anymore, so I'm closing it.
Thank you for bringing such snappy and light Neovim client/GUI!


Polish pyacp module

[ ] Elemental and NodalData -> put into separate NameSpace
[ ] Class ACP -> rename to ACPServer or ACPInstance
[ ] Enums -> Move to separate namespace?
[ ] LinkedObjectHandling  -> move to separate namespace
[ ] TriangleMesh -> move to separate namespace
[ ] VectorData -> move to separate namespace
[ ] example_helpers -> move to separate namesapce

@roosre agreed overall, just some minor points:

Class ACP -> rename to ACPServer or ACPInstance

That was actually called ACPServer at some point (or ACPInstance, I don't remember), before we merged the "client" and "server" functionality into one. I'm now a bit on the fence whether we should rename it back.

Enums -> Move to separate namespace?

For this one I think I'll want to create a branch with the change and see how it changes the example code (when enums are used, not strings). I think it's not entirely clear whether to put all enums in one namespace, or group them by usage (e.g. all solid model - related enums). The former is somewhat odd, but grouping by usage will be difficult to remember especially since the main types are all top-level.

example_helpers -> move to separate namesapce

This is a separate namespace already.

@greschd , thanks for the response.
Enum: I wanted to remove this point from the list and keep it as it is. Sorry about that.
ACP -> ACPClient / ACP Instance: thanks for the info. Good to know and so let's discuss it next week.

@roosre is this one and #637 done, or is something left to do from your point of view?


Add tup

Seems like the GitHub releases are not up-to-date, better use http://gittup.org/tup/win32/ with the reverse property for checkver

@r15ch13 Not really sure how that works. That releases page is a bit weird because the last version number doesn't show up anywhere, it just gets hidden as "latest". I was assuming the github tags and distributed binaries lined up.

The “latest” version is also listed by version number (0.7.5). See http://gittup.org/tup/win32/?C=M;O=D


Ffmpeg camera not working

Great app, camera feed is fast, excellent.
On some there is error like
219 20:08:47 [Error] :  Error loading camera.door_12 history: Bad state: No element
Below configuration in HASS

platform: ffmpeg
name: Door 12
input: -rtsp_transport tcp -i rtsp://admin:xxxxx@<IP_ADDRESS>:554/Streaming/Channels/101/

Also this gives same error

platform: ffmpeg
name: Kitchen2
input: rtsp://admin:xxx@<IP_ADDRESS>:554/onvif2

Also this same error

platform: ffmpeg
name: Office2
input: rtsp://admin:xxxx@<IP_ADDRESS>:554/rtsp_live0

I have all the generic camera work perfectly

The error provided is not about streaming. Try to open any of this cameras and then copy the full lot there

Also ffmpeg problem.
0219 20:04:23 [Debug] :  Fetch complete successful
0219 20:04:44 [Debug] :  [Sending] ==> http://<IP_ADDRESS>:8123/api/camera_proxy_stream/camera.cam_1?token=7a5fe2be6764f2ccc04face8cf6e46fae635035396bdcb926a77c3cb25418944
0219 20:04:44 [Debug] :  [Sending] ==> http://<IP_ADDRESS>:8123/api/history/period/2019-02-18T20:04:44+0100?&filter_entity_id=camera.cam_1
0219 20:04:44 [Debug] :  [Received] <== {content-type: multipart/x-mixed-replace;boundary=ffserver, date: Tue, 19 Feb 2019 19:04:52 GMT, transfer-encoding: chunked, server: Python/3.6 aiohttp/3.5.4}
0219 20:04:44 [Error] :  Error loading camera.cam_1 history: Bad state: No element

Config:
camera:
  - platform: ffmpeg
    input: -rtsp_transport tcp -i rtsp://xxxxx:xxxxx@<IP_ADDRESS>:10554/tcp/av0_0
    name: CAM_1

After 3-4 seconds on camera entity app freeze.

19 20:52:38 [Debug] :  [Sending] ==> https://xxx.duckdns.org:443/api/camera_proxy_stream/camera.living2?token=ac64dc301af0fcb288810f29272c2676b697645c618a9835b616b5909449
0219 20:52:38 [Debug] :  [Sending] ==> https://xxx.duckdns.org:443/api/history/period/2019-02-18T20:52:38+0100?&filter_entity_id=camera.living2
0219 20:52:38 [Error] :  Error loading camera.living2 history: Bad state: No element

@Klagio i need the next line )) It should contain [Received] <== {content-type:

219 21:24:45 [Error] :  Error loading camera.living2 history: Bad state: No element
0219 21:24:45 [Debug] :  [Received] <== {content-type: multipart/x-mixed-replace;boundary=ffserver, transfer-encoding: chunked, date: Tue, 19 Feb 2019 20:25:25 GMT, server: Caddy,Python/3.6 aiohttp/3.5.4

If something will not work after 0.4.3 update, feel free to reopen or report new.


Deprecate the low level MD2, MD4 and MDC2 functions.

Use of the low level MD2, MD4 and MDC2 functions have been informally discouraged for a long time. We now formally deprecate them.
There is a change in the documentation which impacts MD5 as well but the corresponding code changes haven't been made yet.

[x] documentation is added or updated
[x] tests are added or updated

CHANGES entry in #10802

Merged to master.  Thanks for the feedback.


Issue with Precision and Recall

Hi all, i don't know if i can use this post in this way to solve my problem, but i'm using rival in my thesis work to evaluate the predictions of multiple recommender systems; my issue is then when i try to compute precision and recall of my results, recall is always set to 1.0 and precision is not correct (i made compute by hand and the results are diffents) i try 2 different configurations of test, the former with binarized score and the latter with  prediction confidence score, but noone give me the correct results; i'll write the contents of the two files i use to do the test; i think i'm loosing something so the meaning of the test change in my case.
The file are in the format : idUser (tab) idItem (tab) score
Left side: predicitions file | Right side: test file
First Configuration
2	13	1	|	2	13	1
2	19	1	|	2	19	0
2	50	1	|	2	50	1
2	251	1	|	2	251	1
2	257	1	|	2	257	1
2	279	0	|	2	279	1
2	280	0	|	2	280	0
2	281	0	|	2	281	0
2	290	0	|	2	290	0
2	292	1	|	2	292	1
2	297	1	|	2	297	1
2	298	1	|	2	298	0
2	299	0	|	2	299	1
2	301	1	|	2	301	1
2	303	1	|	2	303	1
2	307	1	|	2	307	0
2	308	0	|	2	308	0
2	312	0	|	2	312	0
2	313	1	|	2	313	1
2	314	0	|	2	314	0
2	315	1	|	2	315	0
2	316	1	|	2	316	1
Precision:	0.5454545454545454
Recall:	1.0
Second Configuration
2	13	0.5328112138740229	|	2	13	1
2	19	0.8095414650648188	|	2	19	0
2	50	0.9888651730555137	|	2	50	1
2	251	0.8796398906715018	|	2	251	1
2	257	0.8994154288885712	|	2	257	1
2	279	0.2031462463186414	|	2	279	1
2	280	0.137729028181314	|	2	280	0
2	281	0.12677275323108753	|	2	281	0
2	290	0.12267777345041282	|	2	290	0
2	292	0.6708907647085832	|	2	292	1
2	297	0.769468001849272	|	2	297	1
2	298	0.878495414483143	|	2	298	0
2	299	0.09460751859396674	|	2	299	1
2	301	0.5983346178722854	|	2	301	1
2	303	0.8094825203837621	|	2	303	1
2	307	0.7223299632360657	|	2	307	0
2	308	0.11311986767160687	|	2	308	0
2	312	0.1279286169284483	|	2	312	0
2	313	0.9889272842775845	|	2	313	1
2	314	0.0026441894503800604	|	2	314	0
2	315	0.955807550363639	|	2	315	0
2	316	0.965191057772059	|	2	316	1
Precision:	0.5454545454545454
Recall:	1.0
The prediction has a confusion matrix of
TP=10
FP=4
FN=2
TN=6
so metrics should be
precision: 0.714
recall: 0.833
As you can see they are very different.  Thanks in advance, Vincenzo.

Hi Vincenzo,
I will take a look at it as soon as possible. Can you tell us how you are calling the precision and recall metrics? If you are using your code, it will be useful to know how you are creating and printing the metrics. If you are using a script, it would also be helpful to know the commands you are using.
Thanks,
Alejandro

yes sure, This is my code:
....
SimpleParser sp = new SimpleParser();
DataModel dm = sp.parseData(new File("path_of_my_predictions_file"));
DataModel dmtest = sp.parseData(new File("path_of_my_test_file"));
Precision pr = new Precision(dm, dmtest);
pr.compute();
System.out.println(pr.getValue());
Recall rc = new Recall(dm, dmtest);
rc.compute();
System.out.println(rc.getValue());
....

Hi Vincenzo, sorry it took so long, but I thought it was going to be something related with how you called the metrics, so I needed to prepare a test case. In the end, the problem seems to be with your definition of precision (and recall): instead of using the definition from the classification literature, you should look at the one used in information retrieval (here), where the total number of retrieved documents are considered in the denominator. In your case, this means: 12 (positive) / 22 (retrieved) = 0.54.
Let me know if you have any comments,
Alejandro


use more robust remove_dir_all for Windows

Fixes #61.
I don't have a Windows machine to test this on, but it should do the trick.

Intereting, thank you!


Remove unnecessary lookup for IndexShard.

With new API IndexShard is passed as argument in onPrimaryShard/onReplicaShard
so can be used directly without the need of lookup through IndexService.

Codecov Report

:exclamation: No coverage uploaded for pull request base (es51@716795f). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             es51    #5707   +/-   ##
=======================================
  Coverage        ?   82.76%
  Complexity      ?    11769
=======================================
  Files           ?     1394
  Lines           ?    45302
  Branches        ?     5606
=======================================
  Hits            ?    37494
  Misses          ?     5208
  Partials        ?     2600

Impacted Files
Coverage Δ
Complexity Δ

...executor/transport/TransportShardUpsertAction.java
85.81% <ø> (ø)
72 <0> (?)

...executor/transport/TransportShardDeleteAction.java
80.76% <ø> (ø)
10 <0> (?)

...crate/executor/transport/TransportShardAction.java
86.48% <100%> (ø)
9 <0> (?)

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 716795f...fd3e164. Read the comment docs.


Influx inspect export bug under multiple retention policies in same database

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

Create a database named db2：CREATE DATABASE db2
Create a rp named rp2 on db2: CREATE RETENTION POLICY rp2 ON db2 DURATION 0s REPLICATION 1 SHARD DURATION 168h
Write some data into db2+autogen and db2+rp2 separately
Now export the data of db2 with influx_inspect export

The export file will contain the following lines:

# INFLUXDB EXPORT: 1677-09-21T08:18:26+08:05 - 2262-04-12T07:47:16+08:00
# DDL
CREATE DATABASE db2 WITH NAME rp2
CREATE DATABASE db2 WITH NAME autogen
# DML

Then import the file with influx -import, and the following error will occur:

retention policy conflicts with an existing policy

The reason:
CREATE DATABASE "db2" WITH NAME "autogen" will create a database called db2 with a new DEFAULT retention policy and specify the name of that retention policy
Expected behavior:
Describe what you expected to happen.
Fix the bug: The correct lines should be:
# DDL
CREATE DATABASE db2 WITH NAME autogen
CREATE RETENTION POLICY rp2 ON db2 DURATION 0s REPLICATION 1

Environment info:
InfluxDB 1.8

Fixed by #23197


Error 500 on /compile

HTTP Response Body:
A web handler threw an exception. Details:
data/haskell/user/[redact]/[redact].hs: hClose: resource exhausted (No space left on device)

This is on the https://code.world server, not a self-hosted instance.

Yikes!  We ran out of disk space on the main CodeWorld instance.  I've mitigated the problem by deleting the cache of compiled code, so it should work now.  I'll look into a longer-term solution now.


fix: the way to pass the input default value in useTextField story

On storybook, we were not able to change the input value in useTextField.stories.tsx because value was passed to useTextField as props so I changed to use defaultValue props instead of it.
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Confirm we can edit input content in useTextField.stories.tsx on storybook

🧢 Your Project:

Thanks for the interest.
Our storybook isn't documentation, it's for our own internal testing.
We do not wish to change this at this time.
I'm closing this, but if this is causing a real life problem, please open an issue.


Add low watermark metric

I really appreciate the high watermark metric.  I will be submitting a PR for low watermark metrics.

@rdonnelly3 this has been released in 0.3.0


How to change the code to make the headset shake hands with the server through static IP? I'm a beginner of rust.

I want to simply add my IP address to the code, but I can't find the place to change the code in Connection.rs.

oh，i added my ip after "send to" on line 29 in ALVR-master\alvr\client_core\src\connection_untils.rs,but it not work. Do I need to change the code of the server？

Why would you want to hardcode your cleint IP? The ALVR server already supports clients with a static IP

Why would you want to hardcode your cleint IP? The ALVR server already supports clients with a static IP

No,the IP is server IP, not Client IP.

I replaced BROADCAST with my IP in ALVR-master\alvr\client_core\src\connection_untils.rs ,and my server can discovery the headset, but It didn't work after I clicked the “Trust” button


Retrieving element values without range indexes in MarkLogic

I have the following situation in MarkLogic. I have a database with a large number of documents which each hold hundreds of fields, And I want to provide a feature where user need to sort with any attributes. I don't want to add all attributes in range index. So could I achieve this.
Important things is we required pagination.

I've had plenty of customers say, "We want to sort on any field". In practice though, sorting on most fields didn't make any sense. For example, if you have 1 million customers and you sort on last name, that's only going to be helpful for finding customers with last names starting with A or Z. Free text search is usually a far better approach.
Thus, I've found that sorting on date/dateTime fields to be very helpful; sorting on numeric fields to sometimes be helpful (bucket facets usually work better); and sorting on string fields to rarely be helpful. If a string field has a bounded set of values, a facet is almost always the best choice (though that of course requires a range index).
So my first recommendation is - consider which attributes of documents would really be valuable for sorting, and add range indexes to those, and you're done.
If that doesn't suffice, then you've probably already realized that sorting by a non-indexed field is not going to scale well, regardless of what database / search engine you use. The next option with MarkLogic would be to consider using the Optic API as a way of indexing values without the higher memory cost of range indexes (the Optic API will index rows via the triple index, which is not pinned into memory like range indexes are). You'd need to use something like a TDE to define all the attributes though so that ML can index rows. Given that you have hundreds of fields, that's going to be a large TDE that results in additional disk space being used for all those rows to be indexed.
The above approach is doable, but... I go back to my original point, which is that I've found it to be extremely rare for "Sort by any field" to be worth it in terms of cost/benefit. Figuring out the (likely small number of) fields that truly require sorting will likely save you a lot of time and money.

Closing, but please reply back if you'd like to continue the conversation about your customer's use case.


In csharp script using .netstandard library: The type 'List<>' is defined in an assembly that is not referenced.

I'm using a script referencing a assembly compiled with .net standard 2. When i use a property of type List'1, I'm getting the error in vscode:
The type 'List<>' is defined in an assembly that is not referenced. You must add a reference to assembly 'netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. [testValidation.csx]

Is possible to reference a assembly compiled with .net standard 2 from a csx file, and that the intellisense works properly in vscode?

@gabomgp Can you share your omnisharp log?

@gabomgp by default OmniSharp assumes net46 mode for resolving dependencies. You can add #! "netcoreapp2.0" directive at the top of your script to force the assembly resolution to assume .NET Core 2.0 mode.

@filipw  Thanks you very much. You resolved my question.


[OLD] feat(host): refactor kona-host to include library

Description
It is useful to be able to access the functionality of kona-host from an outside crate, so that projects leveraging kona can invoke it within their flow. For example, in the ZK context, we use kona-host out of the box in our witness generation step.
This PR moves the core kona-host functionality into a library, and refactors main.rs to use this library.
Tests
N/A - existing tests should ensure that this is safe.
Additional context
N/A
Metadata
N/A

CI isn't running. Rebuild on fresh branch and opened new PR: https://github.com/ethereum-optimism/kona/pull/361