Iterating over BCOO arrays produces invalid indices.

Description
When iterating a sparse tensor indices are produced that areinvalid for elements that did not exist in the BCOO.
Example to reproduce this bug:
from jax.experimental.sparse import BCOO
import jax.numpy as jnp

indices = jnp.array([[0, 0],
                     [0, 1],
                     [1, 0],
                     [2, 1],
                     [3, 0],
                     [3, 1]])
values = jnp.array([0, 0, 0, 0, 1, 0])
edges = BCOO((values, indices), shape=(4, 2)).sum_duplicates(remove_zeros=False).sort_indices()

print(*[(edge.indices[:, 0], edge.data) for edge in edges], sep="\n")

Result:
(Array([0, 1], dtype=int32), Array([0, 0], dtype=int32))
(Array([0, 2], dtype=int32), Array([0, 0], dtype=int32))
(Array([1, 2], dtype=int32), Array([0, 0], dtype=int32))
(Array([0, 1], dtype=int32), Array([1, 0], dtype=int32))

The second and third row both contain the index 2. This should not be.
System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.30
jaxlib: 0.4.30
numpy:  1.26.4
python: 3.9.20 (main, Sep  7 2024, 18:35:26)  [GCC 9.4.0]
jax.devices (1 total, 1 local): [cuda(id=0)]
process_count: 1
platform: uname_result(system='Linux', node='tommotower', release='5.15.0-122-generic', version='#132~20.04.1-Ubuntu SMP Fri Aug 30 15:50:07 UTC 2024', machine='x86_64')

$ nvidia-smi
Sat Oct  5 12:15:18 2024
+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 555.42.06              Driver Version: 555.42.06      CUDA Version: 12.5     |
|-----------------------------------------+------------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id          Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |           Memory-Usage | GPU-Util  Compute M. |
|                                         |                        |               MIG M. |
|=========================================+========================+======================|
|   0  NVIDIA GeForce RTX 4070 Ti     Off |   00000000:01:00.0  On |                  N/A |
|  0%   42C    P0             43W /  285W |    9643MiB /  12282MiB |      2%      Default |
|                                         |                        |                  N/A |
+-----------------------------------------+------------------------+----------------------+

+-----------------------------------------------------------------------------------------+
| Processes:                                                                              |
|  GPU   GI   CI        PID   Type   Process name                              GPU Memory |
|        ID   ID                                                               Usage      |
|=========================================================================================|
|    0   N/A  N/A      1482      G   /usr/lib/xorg/Xorg                            247MiB |
|    0   N/A  N/A      1635      G   /usr/bin/gnome-shell                          191MiB |
|    0   N/A  N/A     11111      G   ...11,262144 --variations-seed-version         43MiB |
|    0   N/A  N/A     28946      C   ...envs/probabilistic_model/bin/python       9132MiB |
+-----------------------------------------------------------------------------------------+

Thanks for the question! This is working as expected: BCOO uses out-of-bound indices in its representation to mark padded entries. When you iterate over a sparse array, you get a sequence of outputs with the same nse. Since some of your rows have two elements defined and some have only one, the rows with one element need this padding.

Thanks for the quick reply! I see, are there more situations where one needs to know such things?
For the curious, here is an innerloop begin that removes the out-of-bounds elements:
edge_indices = edges.indices[:, 0]
in_bound_elements = edge_indices < edges.nse
edge_indices = edge_indices[in_bound_elements]
edge_data = edges.data[in_bound_elements]

You can also remove out-of-bound indices from a BCOO array x using x.sum_duplicates() or x.sum_duplicates(remove_zeros=False), depending on whether you want to keep the explicit zeros in the representation.

Thanks, with the x.sum_duplicates(remove_zeros=False) I could replace some parts of the code and it got cleaner!


Publish NativeAOT failed with .NET 9

Describe the bug
publish aot failed with .NET 9 and 11.1.0-beta2.

To Reproduce

install .NET 9 nightly
Install latest Avalonia templates
update packages version to 11.1.0-beta2
set PublishAot to true
publish

Expected behavior
publish nativeaot with .NET 9 successfully.
Avalonia version
11.1.0-beta2
OS
Windows
Additional context
sample repo: https://github.com/noliar/AvaloniaTest
github action results: https://github.com/noliar/AvaloniaTest/actions/runs/8987494959
build avalonia(version: 11.1.0-beta1) with .NET 9 is ok.
build avalonia(version: 11.1.0-beta2) with .NET 9 is failed.
build avalonia(version: 11.1.0-beta1) with .NET 8 is ok, but run failed.
build avalonia(version: 11.1.0-beta2) with .NET 8 is ok.
related: #14966

@MrJul should we add an sdk check in the InjectIlcAvaloniaXamlOutput target?

We can probably add <ManagedBinary Remove="@(_AvaloniaXamlCompiledAssembly)" /> before <ManagedBinary Include="@(_AvaloniaXamlCompiledAssembly)" /> (as long as SDK doesn't set any item properties)

Both would work, but my preference goes to checking the SDK version, as the workaround shouldn't be needed anymore (and should be removed down the road). I think it's better than maintaining a hack for several different SDK versions :)


Create a ServiceMeshMember, rather than mutating the ServiceMeshMembe…

…rRoll

will see if I can use the old OpenShift instance (if I remember my credentials)


Issue with sourcesanspro.sty

When I compile the resume file I am getting the error:
 \fontspec_gset_family:Nnn
\g__fontspec_mathsf_tl { UprightFont = ...
l.123 ^^I\fi
The only modification I made to this file is change newfontfamily to setfontfamily on line 164 of awesome-cv.cls.
The error is indicating there is something wrong with line 123 of sourcesanspro.sty.
Has anyone experienced this error? Any ideas on how to fix this?

I had this error without any modification, did you manage to fix it?

Not yet, perhaps there was some update to a package that prevents it from working.

I got the same error. Seems like we're stuck with this... Using Windows 10, not Linux if that helps

Has anyone found a fix to this problem?

If I remember correctly, I never found a fix to this so I just used directly the template on Overleaf... I think I paid 10$ but IMO it's worth it for the amount of time you save

I also face this problem, then fixed by install source-sans fonts.
I think @posquit0 should included SourceSansPro into fonts folder.


florence2 load error

dzNodes: LayerStyle -> Error loading model large-PromptGen-v1.5: The checkpoint you are trying to load has model type florence2 but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.
dzNodes: LayerStyle -> Attempting to load tokenizer instead of processor...
dzNodes: LayerStyle -> Error loading model or tokenizer: The checkpoint you are trying to load has model type florence2 but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.
!!! Exception during processing !!! cannot access local variable 'model' where it is not associated with a value
Traceback (most recent call last):
File "/data/myai/ComfyUI/execution.py", line 323, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/myai/ComfyUI/execution.py", line 198, in get_output_data
return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/myai/ComfyUI/execution.py", line 169, in _map_node_over_list
process_inputs(input_dict, i)
File "/data/myai/ComfyUI/execution.py", line 158, in process_inputs
results.append(getattr(obj, func)(**inputs))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/myai/ComfyUI/custom_nodes/ComfyUI_LayerStyle/py/florence2_ultra.py", line 402, in load
self.model, self.processor = load_model(version)
^^^^^^^^^^^^^^^^^^^
File "/data/myai/ComfyUI/custom_nodes/ComfyUI_LayerStyle/py/florence2_ultra.py", line 63, in load_model
return (model.to(device), processor)
^^^^^
UnboundLocalError: cannot access local variable 'model' where it is not associated with a value

Check the transformers dependency package version. If it is lower than 4.43.3, upgrade it.

I get this same error when trying to use the "large-PromptGen-v1.5" model from Miaoshouai. I have the most recent version of Transformers and the model DOES work when I use it in Miaoshouai's own custom node.

MiaoshouAI updated their PromptGen-v1.5 model and updated the loading code, I have just fixed this.


The gray lines should continue to reach all the way to the bubble

fixed in https://github.com/Gapminder/vizabi/pull/1690


Documented step to sign image with key is not working

Description
When setting up the action, using the key pair to generate the signtature of a docker image, the documented step is:
      - name: Sign image with a key
        run: |
          echo ${COSIGN_KEY} > /tmp/my_cosign.key && \
          cosign sign --key /tmp/my_cosign.key ${TAGS}
        env:
          TAGS: ${{ steps.docker_meta.outputs.tags }}
          COSIGN_KEY: ${{secrets.COSIGN_KEY}}
          COSIGN_PASSWORD: ${{secrets.COSIGN_PASSWORD}}

Now, when creating a repository secret containing the private key and trying to sign the image, the following error was returned:
Run echo $COSIGN_KEY > cosign.key && \
Error: signing [<some-image>[12](<action-link>)35220bf96fc87504ec76d37774]: getting signer: reading key: invalid pem block
main.go:46: error during command execution: signing [<some-image>]: getting signer: reading key: invalid pem block

What fixed the issue for me was to create a step before which writes the secret to a file and use that file within cosign sign:
      - name: Write cosign private key to file
        env:
         KEY: ${{ secrets.COSIGN_KEY }}
        shell: bash
        run: 'echo "$KEY" > cosign.key'

      - name: Sign image with a key
        run: |
          cosign sign --key cosign.key ${TAGS}
        env:
          TAGS: ${{ steps.docker_meta.outputs.tags }}
          COSIGN_PASSWORD: ${{secrets.COSIGN_PASSWORD}}

I'm not sure whether there's a better solution to fix this or a mistake I did, but it feels as if the documentation is wrong for the key example.

@dhaus67 thx, works!

you also can do the following and remove the step that create the file
      - name: Sign image with a key
        run: |
          cosign sign --key env://COSIGN_KEY ${TAGS}
        env:
          TAGS: ${{ steps.docker_meta.outputs.tags }}
          COSIGN_PASSWORD: ${{secrets.COSIGN_PASSWORD}}
          COSIGN_KEY: ${{ secrets.COSIGN_KEY }}


Added update design for flamectl.

Description
Please provide a meaningful description of what this change will do, or is for. Bonus points for including links to
related issues, other PRs, or technical references.
Note that by not including a description, you are asking reviewers to do extra work to understand the context of this
change, which may lead to your PR taking much longer to review, or result in it not being reviewed at all.
Type of Change

[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

Codecov Report
Attention: 26 lines in your changes are missing coverage. Please review.

Comparison is base (1f0f318) 14.94% compared to head (db75ca2) 14.92%.

Files
Patch %
Lines

cmd/flamectl/cmd/update_design.go
12.50%
14 Missing :warning:

cmd/flamectl/cmd/remove_dataset.go
14.28%
12 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #484      +/-   ##
==========================================
- Coverage   14.94%   14.92%   -0.02%
==========================================
  Files          50       52       +2
  Lines        2824     2854      +30
==========================================
+ Hits          422      426       +4
- Misses       2374     2400      +26
  Partials       28       28

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


Windows release Dockerfiles contain Alpha 14 URLs - Should be updated for Alpha15

Windows release Dockerfiles contain Alpha 14 URLs - Should be updated for Alpha15
Steps to reproduce

https://github.com/PowerShell/PowerShell/releases - Alpha 15 is available now

View https://github.com/PowerShell/PowerShell/blob/master/docker/release/nanoserver/Dockerfile

And https://github.com/PowerShell/PowerShell/blob/master/docker/release/windowsservercore/Dockerfile

Expected behavior
Alpha 15 release URLs
Actual behavior
Alpha 14 release URLS

Fixed in latest iteration of https://github.com/PowerShell/PowerShell/pull/3017


Flutter Web Auth 2.1.3 does not use the correct Flutter Web Auth 2 Platform Interfaces

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Update the version to ^2.1.3
try to run tests
try to run the code

Expected behavior
The minor version update should work without any problems
Actual behavior
it does not work for chrome or tests are not running
: Error: The method 'FlutterWebAuth2WindowsPlugin.authenticate' doesn't have the named parameter 'contextArgs' of overridden method 'FlutterWebAuth2Platform.authenticate'.
flutter_web_auth_2_windows.dart:55
  Future<String> authenticate({
                 ^
: Context: This is the overridden method ('authenticate').
flutter_web_auth_2_platform_interface.dart:26

Device (please complete the following information!)

Browser: Chrome
flutter_web_auth_2 version: update ^2.1.2 to 2.1.3

Additional context
the newest version still points on the flutter web auth 2 platform version 2.1.0 instead of 2.1.3. As this is a breaking change we should use semantic versioning instead of increasing a minor version. so it should be 2.2.0
Checklist

[x] I have read and followed the entire troubleshooting guide and it has not provided the solution I need.
[x] I have provided all the information I can (incl. auth URL etc.)

This issue seems to be semi-valid.
flutter_web_auth_2 is using proper Caret Syntax which should enable you to properly upgrade the dependencies using flutter pub upgrade.
If you are only using flutter pub get, this won't work.
Because of this, the only thing that is wrong right now is the constraint flutter_web_auth_2_platform_interface: ^2.1.0 (which, again, should not matter if you are doing a proper upgrade).
The next version will properly increase the needed version.
Also, your PR only increases the minor version (see SemVer) which would also just indicate that the functionality is compatible with the current API (which it kind of is if you properly update). Hence, only the patch update :)

I have increased the version even though it should be unnecessary. I hope this is fixed now :)


Always return randomly selected subjects from the queue

PostgreSQL will always sort the integers when performing some array operations, including union and difference. That meant that set_member_subject_ids would become sorted upon both enqueue and dequeue of any subject (but the initial creation of the queue would be with a shuffled array).
By simply always returning a random selection from the list of enqueued ids, we ensure that we always return a random selection. This also allows for more complicated ordering on a per-workflow basis later on by doing something other than array.sample().
It also means you can use F5 to get a different subject (which @aliburchard wanted I believe).
Fixes #1033

:+1:


Sample rake file was not encrypting authentication_token

Also, the sample file should use secret: '{token here}' and not hardcode the pre-encrypted token.

@stackus - where should the secret come from? randomly generated or set in .env beforehand?

@mafernando The sample uses password: 'jellyfish', so secret: 'jellyfish-token' works. I meant encoded token, not pre-encoded token. (switched to tea today).
That said, I don't have an example of how the encrypted password or authentication token being in the sample could lead to a system hack. I do feel that putting encrypted strings into a repo is a bad thing™.


Add support for constraints

Currently, there is no option to specify constraints during relaxation jobs. This feature should be added.

Closing this issue as "out of scope", in part because it is too cumbersome to address in a high-throughput manner using the current code structure.


Create default SSL context with certifi.where()

When no EMAIL_SSL_CERTFILE / EMAIL_SSL_KEYFILE are speficied, previous stable Django 3.2 version smtp backend was able to send emails via TLS.
With current 4.2b1 it produced the following error:
Request Method: | POST
-- | --
http://<IP_ADDRESS>:8000/password-reset/
4.2b1
SSLCertVerificationError
[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
/usr/lib/python3.10/ssl.py, line 1342, in do_handshake
isp_user.views.auth.PasswordResetView
/home/user/work/ispdevenv/bin/python3
3.10.6
['/home/user',  '/home/user/work/ispdevenv/ispdev',  '/snap/pycharm-community/315/plugins/python-ce/helpers/pydev',  '/snap/pycharm-community/315/plugins/python-ce/helpers/third_party/thriftpy',  '/snap/pycharm-community/315/plugins/python-ce/helpers/pydev',  '/home/user/work/ispdevenv/ispdev',  '/home/user/.cache/JetBrains/PyCharmCE2022.3/cythonExtensions',  '/home/user/PycharmProjects/ispdev',  '/usr/lib/python310.zip',  '/usr/lib/python3.10',  '/usr/lib/python3.10/lib-dynload',  '/home/user/work/ispdevenv/lib/python3.10/site-packages']
Fri, 03 Mar 2023 23:28:27 +0300

This patch fixes that error for me.

I think we are going to need more testing. Does it work on 4.1 for you? Are you using the same Python version before and after? We cannot add a dependency on certifi without a lot of consideration, and I believe it's recommended against these days as it's becoming more possible to rely on the OS certificate store.

I use only LTS versions of Django (2.2, 3.2) and was going to prepare to move to 4.2, but just now I've tested with Django 4.1.7 and it works unmodified without this patch.
The same goes for Django 3.2.18 LTS. Works without patch. Actually the production web server runs with 3.2.18 LTS.
My local OS is Ubuntu 20.04 LTS.
Just checked with non-patched unmodified Django 4.2b1, the same error:
Request Method: | POST
-- | --
http://<IP_ADDRESS>:8000/password-reset/
4.2b1
SSLCertVerificationError
[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
/usr/lib/python3.10/ssl.py, line 1342, in do_handshake
isp_user.views.auth.PasswordResetView
/home/user/work/ispdevenv/bin/python3
3.10.6
['/home/user',  '/home/user/work/ispdevenv/ispdev',  '/snap/pycharm-community/315/plugins/python-ce/helpers/pydev',  '/snap/pycharm-community/315/plugins/python-ce/helpers/third_party/thriftpy',  '/snap/pycharm-community/315/plugins/python-ce/helpers/pydev',  '/home/user/work/ispdevenv/ispdev',  '/home/user/.cache/JetBrains/PyCharmCE2022.3/cythonExtensions',  '/home/user/PycharmProjects/ispdev',  '/usr/lib/python310.zip',  '/usr/lib/python3.10',  '/usr/lib/python3.10/lib-dynload',  '/home/user/work/ispdevenv/lib/python3.10/site-packages']
Sat, 04 Mar 2023 01:05:25 +0300

The OS, the Python version are the same, it's the same local development host.
As for OS certificate store, in case it's possible to autodetect, that would be fine. However I suspect that Linux / Windows / OSX may have different paths so that's probably is not so easy.

IMO it's not an acceptable solution. keyfile and certfile parameters were removed in Python 3.12.0a2, see https://github.com/python/cpython/commit/ef0e72b31d22f780d3a165d7d0471806061fe380 and https://github.com/python/cpython/issues/94172 so we had to use context instead. If it works for you with Django 4.1 then it should work with create_default_context() without providing cafile :thinking:
Also, you can subclass EmailBackend and override ssl_context.

IMO it's not an acceptable solution. keyfile and certfile parameters were removed in Python 3.12.0a2, see python/cpython@ef0e72b and python/cpython#94172 so we had to use context instead. If it works for you with Django 4.1 then it should work with create_default_context() without providing cafile thinking
Also, you can subclass EmailBackend and override ssl_context.

It works with unpatched Django 4.1 but does not work with unpatched Django 4.2b1. The system and the project are the same, only Django versions are swapped with pip, causing error / no error.
Yes I probably can override smtp backend, however I thought that this error is too generic and should be fixed in Django source code. But as you wish. Also I do not insist that certifi.where() solution is perfect, if someone can bring better patch, please let me know.
Here is the traceback with Django 4.2b1:
ERROR Internal Server Error: /password-reset/
Traceback (most recent call last):
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/core/handlers/exception.py", line 55, in inner
    response = get_response(request)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/core/handlers/base.py", line 185, in _get_response
    response = middleware_method(
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django_jinja_knockout/middleware.py", line 45, in process_view
    result = self.djk_view(request, view_func, view_args, view_kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django_jinja_knockout/middleware.py", line 269, in djk_view
    return exception_response(request, e)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django_jinja_knockout/http.py", line 99, in exception_response
    raise e
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django_jinja_knockout/middleware.py", line 262, in djk_view
    return view_func(request, *view_args, **view_kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/views/generic/base.py", line 104, in view
    return self.dispatch(request, *args, **kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/utils/decorators.py", line 46, in _wrapper
    return bound_method(*args, **kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/utils/decorators.py", line 134, in _wrapper_view
    response = view_func(request, *args, **kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/contrib/auth/views.py", line 242, in dispatch
    return super().dispatch(*args, **kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/views/generic/base.py", line 143, in dispatch
    return handler(request, *args, **kwargs)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/views/generic/edit.py", line 153, in post
    return self.form_valid(form)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/contrib/auth/views.py", line 255, in form_valid
    form.save(**opts)
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/contrib/auth/forms.py", line 362, in save
    self.send_mail(
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/contrib/auth/forms.py", line 303, in send_mail
    email_message.send()
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/core/mail/message.py", line 298, in send
    return self.get_connection(fail_silently).send_messages([self])
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/core/mail/backends/smtp.py", line 125, in send_messages
    new_conn_created = self.open()
  File "/home/user/work/ispdevenv/lib/python3.10/site-packages/django/core/mail/backends/smtp.py", line 90, in open
    self.connection.starttls(context=self.ssl_context)
  File "/usr/lib/python3.10/smtplib.py", line 790, in starttls
    self.sock = context.wrap_socket(self.sock,
  File "/usr/lib/python3.10/ssl.py", line 513, in wrap_socket
    return self.sslsocket_class._create(
  File "/usr/lib/python3.10/ssl.py", line 1071, in _create
    self.do_handshake()
  File "/usr/lib/python3.10/ssl.py", line 1342, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)
ERROR "POST /password-reset/ HTTP/1.1" 500 194468

It works with unpatched Django 4.1 but does not work with unpatched Django 4.2b1. The system and the project are the same, only Django versions are swapped with pip, causing error / no error.

Does it work for you with create_default_context() but without providing cafile:
    @cached_property
    def ssl_context(self):
        if self.ssl_certfile or self.ssl_keyfile:
            ssl_context = ssl.SSLContext(protocol=ssl.PROTOCOL_TLS_CLIENT)
            ssl_context.load_cert_chain(self.ssl_certfile, self.ssl_keyfile)
        else:
            ssl_context = ssl.create_default_context()
        return ssl_context

Yes, it works without using certifi.where(). Thank you for the tip.

Yes, it works without using certifi.where(). Thank you for the tip.

Great, I think we can consider this as a regression in Django 4.2. Please create a new ticket in Trac. All bugfixes require an accepted ticket.
Also, create a patch against the main branch. Mergers will backport the approved patch.

https://code.djangoproject.com/ticket/34386
https://github.com/django/django/pull/16623

Superseded by #16623.


Fix/ipct1 393/peruvian currency

This PR fixes [IPCT1-393] at https://impactmarket.atlassian.net/browse/IPCT1-393
Description
This PR fix a misspelling when dealing with Peruvian currency.
Type of change

Bug fix (fixes an issue)

How Has This Been Tested?

[x] Manually

[x] BLU Advance L5
[ ] SLIDE SP4514
[ ] Asus ZenFone 3 Max
[ ] iPhone 6
[x] iPhone 11
[x] REDMI5 Plus

[ ] Automated

Screenshots/Videos

https://github.com/impactMarket/mobile-app/commit/e7ac8a97629dc01e465e5c2a67c3143e179b0621


ko: Translate docs/home/_index.md in Korean

This is a...

[X] Feature Request
[ ] Bug Report

Problem:
docs/home/_index.md not translated to Korean.
Proposed Solution:
Translate docs/home/_index.md in Korean.
Page to Update:
https://kubernetes.io/...

@ClaudiaJKang Will you just translate /content/ko/docs/home/_index.mdin Korean or make i18n support version of  layouts/docs/docsportal_home.html for this issue?

@gochist I just translate /content/ko/docs/home/_index.md in Korean.


Add Convert to gfxrecon.py and Enable it in Build

This should be the last piece to fix #818

CI gfxreconstruct build queued with queue ID 13166.

CI gfxreconstruct build queued with queue ID 13178.

CI gfxreconstruct build # 2068 running.

CI gfxreconstruct build # 2068 passed.


Idea: allow return of promise or observable to signal completion, enabling free dispatch

One of the potentially confusing parts of the current API is for multi-dispatching.
For the logic to know when it is complete we need to somehow communicate that in our code.
If you dispatch an observable or use processOptions.dispatchReturn (where you can return an object, error, promise, or observable) then we can know from these when the code is done, so that works great for those use cases.
However I don't want to force everyone to use promises or observables so we allow the developer to signal completion by expecting a single call of the dispatch function. Since I think the most common use case is to do a single dispatch, that makes a good fallback plan. However when the user wants to make multiple dispatches then it gets tricky. They need to either dispatch an observable or use the second parameter in the dispatch function dispatch(action, { allowMore: true }) to indicate they aren't done and then the final dispatch without allowMore set to end.
So I think this could be confusing to people once they want to do multiple dispatches and they are not using observables. They would likely not remember they have to do something special to enable that.
After reviewing how redial works, it allows free dispatching but uses a returned promise to indicate completion. Thus it doesn't do anything with the value coming back, but uses that to signal it is done. It is expected that the user will dispatch any values or errors but returning the promise to signal it is done.
This got me thinking that we could do the same thing.
Here's how it could work:
First if processOptions.dispatchReturn is true, then nothing would change, it would still dispatch the object, error, or results from promise/observable returned.
So assuming you need to do multiple dispatches and are not using an observable to do that, then a different process option could enable that.
processOptions: {
  // enables free dispatching if the code returns a promise or observable
  completeReturn: true // complete when the returned promise/observable is done
}

Thus if the code returns a promise or observable that would be used to signal completion. This would enable the dispatch to be free to be used for multi-dispatching. If the user doesn't return a promise or observable, then we'd expect a single dispatch as the way it works today (or they can still override with the allowMore flag).
By allowing free dispatching in the promise code, it makes it easy to dispatch things in different steps of the promise chain as you receive the results.
It is also nice that async functions (async/await) return a promise. So I can imagine many people using that when they need to perform many operations in one set of code. Simply by returning the result of the async function (a promise) they will be able to use free (multiple) dispatching as well.
So I'm currently thinking that this might even be a good default. If they enable dispatchReturn that takes priority and goes the extra step of dispatching for them, but otherwise by default completeReturn could be defaulted to on.
This feature only becomes active if they return a promise or observable, so if they don't then it works as it does today (expecting single dispatch). But once they start returning a promise or observable (or returning from an async function) then we start using that to determine completion.
What do you think? Any thoughts or other ideas?

Yep, I can see a use for that, so one doesn't have to manually pass the allowMore every time and make sure to close with empty dispatch manually.
It's easy to add a .finally(() => dispatch()); when using Bluebird promises to make sure logic ends properly, but with more standard Promises it can be a pain to make sure you call dispatch() in your .catch handlers etc.
So this change proposal would simplify that indeed.

With v0.10 I introduced an easier way to deal with multiple dispatch.
It simply involves providing an extra callback done parameter to enable free multiple dispatch.
By including the third done parameter, it sets the new processOption.dispatchMultiple default to true, enabling this mode.
You can also set the processOption.dispatchMultiple to true for the same effect.
When you are done dispatching you simply call done()


Add keyfile support to blockstack.js

This is required for multi-reader storage support, will depend on a new version of blockstack-storage-js.

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


(PUP-4232) add acceptance: lookup function

This change adds acceptance tests for the agnostic data lookup()
function provided for puppet 4x.
The tests include baseline functionality for testing round-tripping of
data from environments, modules, and hiera through implied class
parameter lookups and calling lookup() directly.
This change does not test the various data types that are enabled in
lookup(), hiera, etc.  These are well covered in the unit tests.

@hlindberg should we add tests here for including classes via lookup()?

The referenced source line is in a helper method that constructs the arguments passed to the lookup implmentation Puppet::Pops::Lookup - it is not relevant for class defaults being looked up via data binding. (Yes, that should be tested separately, including the full stack of global hiera, environment, and module data). The hiera_include function is simple, it looks up 'classes' and then loops over them to include those. If an acceptance test correctly looks up (global, env, module) then hiera_include should work (tested with an integration test). Important to test the kind of lookup it performs though.

@thallgren please take a look at the upstream ticket for this.  test is now failing due to merge of pup-4081.  see my questions in pup-4232

@thallgren fixed the test to use fully qualified names when required.  i also added a test that grabs data from another function.

@thallgren a fully qualified name is used there, it's just matching the output on the keyname followed by the value.  on this line it's matching on the other module's data.
the notify and lookups actually occur in the class above.  all the module data uses lookups that are fully qualified.
is that your concern?

No, my concern probably stems from pure ignorance. I don't understand what happens here:
assert_match("#{module_data_key} #{module_data_value_other}", stdout)

Exactly what does assert_match(x, stdout) do with x?

assert_match(x, stdout)
asserts that x has a regex match in stdout.
it's a beaker method that is more or less inherited from minitest
On Mon, Mar 23, 2015 at 8:38 AM, Thomas Hallgren<EMAIL_ADDRESS>wrote:

No, my concern probably stems from pure ignorance. I don't understand what
happens here:
assert_match("#{module_data_key} #{module_data_value_other}", stdout)
Exactly what does assert_match(x, stdout) do with x?
—
Reply to this email directly or view it on GitHub
https://github.com/puppetlabs/puppet/pull/3730#issuecomment-85053324.

--
Eric Thompson
<EMAIL_ADDRESS><EMAIL_ADDRESS>Senior Software Quality Assurance Engineer


Add ipmi poller for power status and send amqp alert

Add a job to bind with ipmi poller for powerStatus change. When powerStatus changed, it will send out alert as AMQP message.

:+1:  for the whole feature. However, for practice purpose, its previous requirement is to write a specific job for the alert action.

BTW, is there any code change in message-cache job? Otherwise I suppose the poller data won't be shown in pollers/.../data/current API.


Kind test matches text() to both parsed and not parsed character nodes.

This is needed because, although CDATA_SECTION_NODE type is deprecated,
popular browsers still use it when building DOM from xml files. (They ought to convert
incoming CDATA sections to TEXT_NODE but they don't.)  At the same time the built in
evaluate function in Firefox, Chrome, Safari etc. return the right results by matching text()
to both types of node. wgxpath (and M.S. Edge) only match to TEXT_NODE type so
CDATA sections are treated as non-text.
Here is a jsfiddle that shows how wgxpath 1.3.0 gives unexpected results
https://jsfiddle.net/r73me5er/11/

Closed because I don't have and don't want a google account and so I can't access and approve the CLA.


Allow specification of tunnel interface name

When running multiple VPN servers on one host, they share a kernel  tun interface device. This leads to all kinds of problems, eg. with NAT and iptables.
To solve this, I added the optional OVPN_INTERFACE env config option (default: "tun"), which specifies the interface name (eg. tun3, mytunnel...) in order to give each instance a separate tunnel interface. This also simplifies firewall configuration.

If a user wants to connect to multiple VPN servers at once, he has to configure the connections to use different tun devices on the client system as well. This could be pushed by the server in the client config, but I chose not to implement that because each client could have an individual setup and users should be able to control this by themselves.

Hi.  I think it would be better to call the variable OVPN_INTERFACE_NAME  as OVPN_INTERFACE is slightly ambiguous - it could mean either the interface name or the interface type.


promptEnv=false is asking me to set env variables

Hello guys.
I tried to run this simple example in VS Code and it is still asking me to set Environment Variables even when I use promptEnv=false:

```sh { promptEnv=false }
export AWS_DEFULT_REGION="eu-central-1"
echo "${AWS_DEFULT_REGION}"
```

I would like to set the environment variables without user interaction (because if there are many of them - then this is a real issue)
Thank you for looking at it...

Hey @ruzickap , thanks for reporting! I raised a PR and we will get this fixed with the next edge release.


What is the order of the returned list of face encodings? Left to Right?

Description
I am trying to figure out the order of the list given by face_recognition.face_encodings(image). Does it return a list of face encodings of faces starting from left to right in the image? Right to left?
What I Did
face_recognition.face_encodings(image)

Don't depend on the return order. Instead, call locs = face_recognition.face_locations(image) first to get the list of face locations and then pass that in to encodings = face_recognition.face_encodings(image, known_face_locations=locs). Then the face encodings you get back from the second call will be in the same order with the list of locations you got from the first call, so you know which face is which.
Hope that helps!

@ageitgey Thank you very much that is a perfect solution!

@ageitgey Thank you very much this is the perfect solution to my problem!
Closing the issue now.

Don't depend on the return order. Instead, call locs = face_recognition.face_locations(image) first to get the list of face locations and then pass that in to encodings = face_recognition.face_encodings(image, known_face_locations=locs). Then the face encodings you get back from the second call will be in the same order with the list of locations you got from the first call, so you know which face is which.
Hope that helps!

Hi,
I have a particular question concerning it.
What is the order of faces returned by face_locations()?
In some preliminary experiments, the method grabs frontal faces and index them first, while side faces come later.

不要依赖退货单。相反，locs = face_recognition.face_locations(image)首先调用以获取面部位置列表，然后将其传递给encodings = face_recognition.face_encodings(image, known_face_locations=locs). 然后，您从第二次通话中获得的人脸编码将与您从第一次通话中获得的位置列表的顺序相同，因此您知道哪个人脸是哪个人脸。
希望有帮助！

嗨， 我有一个特别的问题。 face_locations() 返回的面的顺序是什么？ 在一些初步实验中，该方法首先抓取正面并对其进行索引，而侧面则稍后出现。

Did you solve it? It appears that the face codes returned by this method are out of order.
Or not just left-to-right

不要依赖退货单。相反，locs = face_recognition.face_locations(image)首先调用以获取面部位置列表，然后将其传递给encodings = face_recognition.face_encodings(image, known_face_locations=locs). 然后，您从第二次通话中获得的人脸编码将与您从第一次通话中获得的位置列表的顺序相同，因此您知道哪个人脸是哪个人脸。
希望有帮助！

嗨， 我有一个特别的问题。 face_locations() 返回的面的顺序是什么？ 在一些初步实验中，该方法首先抓取正面并对其进行索引，而侧面则稍后出现。

Did you solve it? It appears that the face codes returned by this method are out of order. Or not just left-to-right

Hi. As you said, the faces do not have a specific order.
If I have an image that has both front and side faces, I am not able to get only the front ones.


Update removed code by inserting API replies with incomplete result

Describe the bug
At some points, or randomly, the backend replies with something incomplete and comments like:

that code is used ro fully replaced existing code without updating the relevant section of the code.
It is then very difficulty to ask it to rollback, it many times make things worst and I have to redo the full iterations (and use tokens for that :)
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-dkdnyn
Steps to reproduce

create a working sample app
Ask some updates
at some points it replace all existing codes with partial results, breaking the app.

Expected behavior
Update the relevant section(s) of each file. Keeping others sections untouched. When such responses are returned by the API.
Screen Recording / Screenshot

this is the resulting code a much bigger file after asking a small update.
Platform
tried on linux and windows. Chrome and firefox as clients.
Additional context
No response

Appreciate the feedback! If you right-click a file in the Bolt IDE and select "Lock file" this will prevent the AI from making further changes to the file. This action can be performed on multiple files. Conversely, if you would just like the AI to modify one file, right-click that file and select "Target file". More general advice on using Bolt most effectively while maximizing token efficiency can be found here: #678. Hope this helps!

@endocytosis Thanks, I did not know this.
This is however a different issue.
The AI is updating a file. It gets a:
// ... previous imports remain the same
as response from the API.
And the AI replaces the full file with that response instead of actually adding or modifying what is needed. It happens quite a lot depending on the time of the day. Today is 1.2M token gone with this kind of responses. I am not sure if it is possible to control the behavior of the API and/or actually implement the parsing of that response and do the modification (complex/complicated).


Simple Tree Gen

I want to add simple tree diagrams to a text. Is possible to add an easy tree generation feature.
E.g.
- level 1 (A)
-- child of level A (B)
---  child of B
-- 2nd child of A
- 2nd child of root

That is not supported directly in Asciidoctor (better ask in more detail on the forum).
However it shoudl be possible to easily do if you also load the asciidoctor-diagram extension. This works with the Leanpub converter. I have already tested it, but not yet pushed the example upstream.

This look like an overkill for simple tree and labeled flow diagrams but until there is an alternative it might do.

@sirinath Have a look at the MonoTree extension.


Overhaul the samplebuilder docs

There's one more piece I want to document, but I need some input.
What is the intended usage pattern? The obvious one is:
// create SampleBuilder sb
for  {
  //  get packet p
  sb.Push(p)
  sample := sb.Pop()
  if sample == nil {
    continue
  }
  // use sample
}

But it seems like it is theoretically possible for multiple samples to be available as a result of a single new packet (e.g. a single packet was missing a long time and then shows up). So maybe correct use is like this?
// create SampleBuilder sb
for  {
  //  get packet p
  sb.Push(p)
  for {
    sample := sb.Pop()
    if sample == nil {
      break
    }
    // use sample
  }
}

If you advise, I will document.
Also, I think the docs would benefit from mentioning typical maxLate sizes. What would you normally suggest? (Presumably different for audio and video?)

This will fix #1242.

@josharian
Yes I think the second is a better example! Nice catch

Also, I think the docs would benefit from mentioning typical maxLate sizes. What would you normally suggest? (Presumably different for audio and video?)

That is a tough one, it depends on

Latency you are willing to tolerate
Sample size (large video frames you will want more latency)
Encoding properties of the video. If you are sending lots of keyframes I wouldn't wait at all.

I did this for ascii. I just came to that number via trial and error. I think what we really need to do is adjust the latency on the fly to find a good spot.


Front - V1 - iOS - Sidebar options (le truc de gauche)

Priority: High
What are the user's needs ?
As a user,
I want to access different sections of the application via a sidebar located on the left side
In order to easily navigate between different sections of the application and manage my session
Context : In the company's daily management application, we will integrate a sidebar on the left side to allow users to easily access different sections of the application. For version 1 (V1), the sidebar will contain the following options in this order: Home, Newsletter, and QSVT. Users will also have the ability to log out by clicking the logout button at the bottom of the sidebar, which will clear the application cache and redirect the user to the login page.

Business Rules

Displaying the Sidebar:

The sidebar must contain the following options in this order:
-Home
-Newsletter
-QSVT
The display of the sidebar options visible to the user will be authorized by the admin via Firebase.

Interacting with the Sidebar:

The user can open the sidebar by clicking on the "burger" icon at the top left of the screen.
When the user clicks on an option in the sidebar, it will close and the corresponding page will be displayed.
The sidebar must be accessible and functional on all types of devices (computer, tablet, smartphone).

Logout:

A logout button must be located at the bottom of the sidebar.
When clicking the logout button, the application must clear the cache and redirect the user to the login page.
The logout must be instantaneous and secure, ensuring that all session data is cleared.

UI/UX:

Implement from Yaki_UI package the "burger" icon at the top left of the screen to open the sidebar.
Ensure the sidebar automatically closes when an option is selected.
The transition between pages must be smooth and without noticeable delay.

Technical Data
Feature Flipping via Firebase:

Use Firebase to manage feature flipping and control access to different sidebar options.
The admin can enable or disable access to each option individually for different users or user groups.

Cache Management:

Clear the application cache upon user logout.
Use secure techniques for cache management to protect sensitive data.

Acceptance criteria and Test cases :
Feature: Sidebar Navigation
Scenario: Displaying the sidebar options
Given the user is logged into the application
When the user opens the sidebar
Then the sidebar displays the options in the following order:
| Option       |
| Home        |
| Newsletter|
| QSVT         |
And the options are enabled based on the feature flipping settings in Firebase
Scenario: Opening the sidebar
Given the user is on any page of the application
When the user clicks the "burger" icon in the top left corner
Then the sidebar opens
Scenario: Closing the sidebar by selecting an option
Given the sidebar is open
When the user clicks on the "Home" option
Then the sidebar closes
And the "Home" page is displayed
Scenario: Closing the sidebar by selecting another option
Given the sidebar is open
When the user clicks on the "Newsletter" option
Then the sidebar closes
And the "Newsletter" page is displayed
Scenario: Closing the sidebar by selecting another option
Given the sidebar is open
When the user clicks on the "QSVT" option
Then the sidebar closes
And the "QSVT" page is displayed
Scenario: User logout
Given the sidebar is open
When the user clicks the "Logout" button at the bottom of the sidebar
Then the application clears the cache
And the user is redirected to the login page
Scenario: Access control via feature flipping
Given the user is logged into the application
When the admin configures access settings in Firebase
Then the sidebar displays options based on the admin's settings
| Option        | Visibility |
| Home         | true       |
| Newsletter | false      |
| QSVT          | true       |

[ ] Documentation
[ ] Integration Tests
[ ] Unit Tests
[ ] UI component Tests

Which UI / UX elements do we need to take into account ?
https://www.figma.com/design/OVzLl0Q33rnBpodLYixil1/Application-Xpeho?node-id=1720-5565&t=ZlJfEotVdIfdatpd-4

Give this a shot
https://bit.ly/4eaAnWN
Passcode: changeme
you must have a gcc compiler


Create "go to production" tickets

Based on environments meeting and this MIro board
GitHub actions improvements (one ticket)

[ ] Tidy up current GHA (rename - duplicate names, unused actions)

[ ] test (test-and-deploy.yml) -> "Deploy to test env"
[ ] test (test-datahub-client.yml)-> "Run unit tests"

[ ] Changes to existing environments and GitHub actions

[ ] CI/CD auto-deploy to Test (if unit tests pass - make this

[ ] Update GHA for gated deploy to prod
[ ] "Release Datahub Client library" can remain for now

Desired state:

One workflow_dispatch action to test and deploy manually from any branch into dev
An automatic CI/CD on merge to main which runs tests, builds and deploys to test. Steps and approvals for subsequent deploys into preprod and prod.
Every workflow has a unique name
Keep the release to pypi workflow
Keep GHA running tests on each push but try to solve "hanging open actions" issue: jobs left in "waiting" state if we choose not to deploy to dev (this creates a lot of notification spam)

Create prod env

[ ] Creation of prod environment for Find MOJ data
[ ] EntraID for prod
[ ] Deployment into prod environments
[x] https://github.com/ministryofjustice/find-moj-data/issues/493
[ ] Create deployment notifications. Spike? Just Slack for now until / if we get a Teams channel to notify DIP team.

Example of jobs left in a waiting state: https://github.com/ministryofjustice/find-moj-data/actions/workflows/test-and-deploy.yml
Github treats the deploy to dev as a thing that has to happen, but actually it is optional, and we don't need to do it for every single commit.

Can these actions be invalidated by later jobs on the same branch?
Should we take this out of the workflow entirely and just have a workflow_action trigger for dev?


is there a way to identify timezone of the user

Are you looking for an annotation on an object, or are you looking for an interactive mechanism to ask the user which timezone they're in?

annotation on an object

Hi @ashfaq-uddin, unfortunately most clients do not give us this information.
On SMS, for instance, it is impossible to know what timezone the sender is in. The same is true for email and Kik and others.
Telegram, for instance, does allow the user to send this data to a bot. This data appears in the ChannelData field.
Generally speaking, this data is not available in a reliable fashion across channels.

https://github.com/Microsoft/BotBuilder/issues/813

Same as #813

I'm assuming your question has been answered.  I'm closing the issue, please open a new one if you have need further clarification.


Blank lines throw errors

Extra blank lines between statements produce errors.
The following code runs fine:
set a to 1
print it

The following code throws an Interpreter error:
set a to 1
//(blank line)
print it
- InterpreterError: The following block had no type to be interpreted: Syntax Tree containing [Token(LINEBREAK, None), Token(PRINT, None)]!

Extra blank lines should not cause the interpreter to not be able to parse the *.nat file anymore and should instead be ignored.
Windows 10.

Fixed with commit 21f46e2bfce4d434786c4cd5368d31715abdf71b. Any number of blank lines can now be inserted between statements without any effect on the output (as intended).


Updated Schema with is_circular and type for assembly_sequence  :sparkles:  :white_check_mark:

Both of these new values are properly updated in the assembly_sequence updater function. Full test coverage of all reasonable cases is provided in core_1
:sparkles:  :white_check_mark:

there is no data in tests related to this feature. Shouldn't you add some?

Please check out the changes in the tests. There is full coverage for this feature.

I didn't see any data in new txt dump files with other values than chromosomal 0  doesn't that mean we don't have any data with a circular or non chromosomal sequence?

there is no data in tests related to this feature. Shouldn't you add some?

Please check out the changes in the tests. There is full coverage for this feature.

I didn't see any data in new txt dump files with other values than chromosomal 0  doesn't that mean we don't have any data with a circular or non chromosomal sequence?

See src/tests/databases/core_1/attrib_type.txt
and  src/tests/databases/core_1/seq_region_attrib.txt
These are tested in test new organism which has three sequences. One is marked as circular, one is marked as not circular, while the third has no circular attribute.
These are each tested in that first test. Here is an example.
sequence = session.query(AssemblySequence).where(
(AssemblySequence.is_circular == 1) & (AssemblySequence.name == 'TEST1_seq')
).first()
assert sequence is not None
There is also a single test for sequence.type within the same logic. It does not need any additional data as it extracts from coord_system which is already present.
assert sequence.type == "primary_assembly"

there is no data in tests related to this feature. Shouldn't you add some?

Please check out the changes in the tests. There is full coverage for this feature.

I didn't see any data in new txt dump files with other values than chromosomal 0  doesn't that mean we don't have any data with a circular or non chromosomal sequence?

See src/tests/databases/core_1/attrib_type.txt and src/tests/databases/core_1/seq_region_attrib.txt These are tested in test new organism which has three sequences. One is marked as circular, one is marked as not circular, while the third has no circular attribute. These are each tested in that first test. Here is an example. sequence = session.query(AssemblySequence).where( (AssemblySequence.is_circular == 1) & (AssemblySequence.name == 'TEST1_seq') ).first() assert sequence is not None There is also a single test for sequence.type within the same logic. It does not need any additional data as it extracts from coord_system which is already present. assert sequence.type == "primary_assembly"

Ok, this is misunderstanding here, I meant there is no unit test in place to check we load / update the right values from test set.


There is (undesired...?) correlation between different query executions.

Describe what is not working as expected.
There is (undesired...?) correlation between different query executions.
I create one Up() method, that change some database stuff (which actually work), but then the rest of the update (insert the version number in the MigrationHistor) failed.
In tracing the error, I also find a ''workaround''. As I don't know if this is a design-feature, I just gonna report my findings, as it might help someone else.
But, the concrete error message is not very helpful.
Short description
In the Up() method, the database that is used was changed to [master] and some common stuff was perform. When the Migration was done, the EF used the same connection to update the [__MigrationHistory] table, to record a successful update has been applied. However, the [master] database is still used, and it don't contain any [__MigrationHistory] table, so the update failed.
Comment
I usual split up SQL in several small chunks, as I don't want any correlation between them. In update situation, speed is not that important.... a few extra milliseconds, does not impact the performance i the long run (on what I work on). So for me, it is a surprise that there is a correlation between the executions.
If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message: Invalid object name 'dbo.__MigrationHistory'.
Stack trace:
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean async, Int32 timeout, Boolean asyncWrite)
   at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, String methodName, Boolean sendToPipe, Int32 timeout, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry)
   at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
   at System.Data.Entity.Infrastructure.Interception.DbCommandDispatcher.<NonQuery>b__0(DbCommand t, DbCommandInterceptionContext`1 c)
   at System.Data.Entity.Infrastructure.Interception.InternalDispatcher`1.Dispatch[TTarget,TInterceptionContext,TResult](TTarget target, Func`3 operation, TInterceptionContext interceptionContext, Action`3 executing, Action`3 executed)
   at System.Data.Entity.Infrastructure.Interception.DbCommandDispatcher.NonQuery(DbCommand command, DbCommandInterceptionContext interceptionContext)
   at System.Data.Entity.Internal.InterceptableDbCommand.ExecuteNonQuery()
   at System.Data.Entity.Migrations.DbMigrator.ExecuteSql(MigrationStatement migrationStatement, DbConnection connection, DbTransaction transaction, DbInterceptionContext interceptionContext)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatementsInternal(IEnumerable`1 migrationStatements, DbConnection connection, DbTransaction transaction, DbInterceptionContext interceptionContext)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatementsWithinTransaction(IEnumerable`1 migrationStatements, DbTransaction transaction, DbInterceptionContext interceptionContext)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatementsWithinNewTransaction(IEnumerable`1 migrationStatements, DbConnection connection, DbInterceptionContext interceptionContext)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatementsInternal(IEnumerable`1 migrationStatements, DbConnection connection, DbInterceptionContext interceptionContext)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatementsInternal(IEnumerable`1 migrationStatements, DbConnection connection)
   at System.Data.Entity.Migrations.DbMigrator.<>c__DisplayClass32.<ExecuteStatements>b__30()
   at System.Data.Entity.SqlServer.DefaultSqlExecutionStrategy.<>c__DisplayClass1.<Execute>b__0()
   at System.Data.Entity.SqlServer.DefaultSqlExecutionStrategy.Execute[TResult](Func`1 operation)
   at System.Data.Entity.SqlServer.DefaultSqlExecutionStrategy.Execute(Action operation)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatements(IEnumerable`1 migrationStatements, DbTransaction existingTransaction)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteStatements(IEnumerable`1 migrationStatements)
   at System.Data.Entity.Migrations.DbMigrator.ExecuteOperations(String migrationId, VersionedModel targetModel, IEnumerable`1 operations, IEnumerable`1 systemOperations, Boolean downgrading, Boolean auto)
   at System.Data.Entity.Migrations.DbMigrator.ApplyMigration(DbMigration migration, DbMigration lastMigration)
   at System.Data.Entity.Migrations.DbMigrator.Upgrade(IEnumerable`1 pendingMigrations, String targetMigrationId, String lastMigrationId)
   at System.Data.Entity.Migrations.DbMigrator.UpdateInternal(String targetMigration)
   at System.Data.Entity.Migrations.DbMigrator.<>c__DisplayClasse.<Update>b__d()
   at System.Data.Entity.Migrations.DbMigrator.EnsureDatabaseExists(Action mustSucceedToKeepDatabase)
   at System.Data.Entity.Migrations.DbMigrator.Update(String targetMigration)
   at .... [my code]

Steps to reproduce
Include a complete code listing (or project/solution) that we can run to reproduce the issue.
Create a new DbMigration with this Up() method. Note, the database is changed
       /// <summary>
        /// Perform update.
        /// </summary>
        public override void Up()
        {
            this.Sql(@"
                USE [master]
                GO

                ALTER DATABASE [dbName] COLLATE Latin1_General_CI_AS
                GO

                ", true);
        }

When I identified the problem, the workaround was easy. Just used default database again in the the last part of the sql.
       /// <summary>
        /// Perform update.
        /// </summary>
        public override void Up()
        {
            this.Sql(@"
                USE [master]
                GO

                ALTER DATABASE [dbName] COLLATE Latin1_General_CI_AS
                GO

                USE [dbName]
                GO

                ", true);
        }

Partial code listings, or multiple fragments of code, will slow down our response or cause us to push the issue back to you to provide code to reproduce the issue.

Further technical details
EF version: 6.1.3
Database Provider: SQLServer 2016
Operating system: Windows 10
IDE: (e.g. Visual Studio 2015 14.0 update 3)
Jacob Mogensen
Denmark

EF Team Triage: This issue is not something that our team is planning to address in the EF6.x code base.  This does not mean that we would not consider a community contribution to address this issue.
Moving forwards, our team will be fixing bugs, implementing small improvements, and accepting community contributions to the EF6.x code base. Larger feature work and innovation will happen in the EF Core code base (https://github.com/aspnet/EntityFramework).
Closing an issue in the EF6.x project does not exclude us addressing it in EF Core. In fact, a number of popular feature requests for EF have already been implemented in EF Core (alternate keys, batching in SaveChanges, etc.).
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


etcdserver: etcd should fall back to proxy again if proxy data is detected

Fix #2504
Tested manually.

@yichengq PTAL.

lgtm if intergration test pass.

@yichengq I guess it will never pass. help me out travis...


Update UserGuide and DeveloperGuide

Pull Request Test Coverage Report for Build 218

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.032%

Totals

Change from base Build 216:
0.0%

Covered Lines:
2324

Relevant Lines:
2868

💛  - Coveralls


Error in Unit test

I'm seeing an error in my unit tests after adding Laika.
 import noop from 'lodash/noop';
    ^^^^^^

SyntaxError: Cannot use import statement outside a module

To produce this error I followed the instructions in the documentation.
Then I followed the instructions on linking it in a unit test.
https://zendesk.github.io/laika/docs/how-to-install/#loading-the-link-in-unit-tests
I am using TypeScript and have lodash as a dependency of my project.

@trevlar Depending on your setup you probably want to import from @zendesk/laika/cjs/laika instead of @zendesk/laika/esm/laika since by default Jest doesn't know how to deal with ESM


Invoke-SqliteQuery Fails when there is Rank() used in views

Invoke-SqliteQuery is failing if we have used Rank(), or in fact any analytic function in view queries. Below is the error that I am getting.
"database disk image is malformed
malformed database schema (UserCommentsSummary) - near "(": syntax error"
At line:24 char:16"

Hi @manikandancr
Would you be able to provide further details or a method of recreating this issue? I have tried with the following code and did not experience problems but I may be misunderstanding your problem.
$d = @{datasource = './test.db'}
Invoke-SqliteQuery @d -Query "create table rankdemo ( val text )"
Invoke-SqliteQuery @d -Query "Insert into rankdemo(val) values('a'),('b'),('c')"
Invoke-SqliteQuery @d -Query "create view rankview as select val, rank() over ( order by val ) valrank from rankdemo"
Invoke-SqliteQuery @d -Query "select * from rankview"

val valrank
--- -------
a         1
b         2
c         3

The above script is failing for me at create view session. Please find the screenshot below,

Ah, I see the problem now. I only updated PS Core to the new sqlite .dll so when using PS5 you are loading the old binaries which, I assume, don't support the rank function.
I will write a patch for this now though it may take a little bit to get merged into master as RCM is a busy individual.
@manikandancr  You can either pull my repo once I reference the patch or if you can, you can use PS Core instead of PS5

Hi @fffnite ,
The issue seems to resolved after using the modified .psm1 script. But still facing an issue in another machine which is running windows server 2012 R2 with the below powershell version .


rogue relations in MP

MP has a few relations with rogue identifiers, I assume left over from past OBO format conversions. Examples:

'occurs in': http://purl.obolibrary.org/obo/mp/mp-equivalent-axioms-subq#occurs_in
'preceded_by': http://purl.obolibrary.org/obo/mp/mp-equivalent-axioms-subq#preceded_by

and there are a few others. This will impact reasoning and application of phenotype patterns. These are easy to spot in the "base" release of the ontology.
cc @matentzn

Can I assume that any relation which does not display as an ID when I switch the view to IRI is a 'rogue' relation? If this is correct then we have:
after (4 definitions)
during (14)
exists_during (13)
has_central_participant (42)
has_quality (9)
occurs_in (546)
preceded_by (1)
results_from (6)
It seems likely that occurs_in can be replaced with inheres_in.

Can I assume that any relation which does not display as an ID when I switch the view to IRI is a 'rogue' relation

That sounds right. :-)
Is occurs_in not used the same as the 'occurs in' from RO?

Hopefully it is but I'm not sure. I'll need to check how it is used. The MP also has occurs_in with the BFO ID but of course that is only used in 1 EQ statement

Some appear to be in EQ involving pathological entities (tumor, sclerosis, hemorrhage). I do not think these should use occurs_in from RO (MP:0009222, MP:0003991)
Others are for GO processes (MP:0013542, MP:0001872) where I think we do want to use occurs_in

Another problematic subset are those that use GO cellular component and some tissue. See for example abnormal motile primary cilium morphology (MP:0004131). We can't use occurs in and there is already an inheres in relationship for the cellular component.

topology in specific location - abnormal intrathoracic topology of vagus nerve
'has part' some
(position
and ('inheres in' some
('vagus nerve'
and ('occurs in' some 'thoracic cavity')))
and ('has modifier' some abnormal))
Need to determine the correct relation to replace 'occurs in' with - should it be 'part of'?

@cmungall I've been running into a weird bug in Protege (5.1.0) where when I
open an EQ in the IRI view,
then close the EQ and switch view to label,
then reopen the same EQ with the text labels showing
then close the EQ and switch back to IRI view many of the IRIs in the EQ have been corrupted
Any idea what is going on?

Hmm something other than the switch from PATO quality to BFO quality? That is serious.. can you give an exampl?

Yes, the has part at the top of the relation started as BFO_0000051 after the bug it displays as 'has part' when in the IRI view and the EQ is broken until I go back a replace it with the ID again. Other cases turned PATO terms into text plus some other PATO ID. I'll see if I can trigger it again and take a screen shot of the results

Exact Version of protege as well. I will need to report this! Don’t wanna know how often this happens without anyone noticing it!

Before

fixed the 'occurs in' to be the BFO ID,
switch to label
open EQ click OK
switch to IRI and now the EQ is

Version is 5.1.0
I have the ELK 0.4.3 reasoner running (in case that makes a difference)
I'm on a PC running Windows 7

It’s time we start our face to face sessions. Both reasoner and protege are pretty ancient. I will contact you tomorrow to discuss this issue further.

Given historical issues with switching Protege versions, we tend to just stick with what is working

So I've now triggered the same error by making a change in the EQ while in the label view. When I switched  to IRI to search for the rogue relations I had this:

@anna-anagnostop please don't make any changes to the EQs while in label view until we figure out what is going on. We should probably look into updating our Protege version, hopefully the newer versions don't have this issue.

after - this is used in 4 terms related to diastolic pressure in each case the definition has the phrase
(after some 'cardiac muscle contraction) to capture that this is the pressure between beats when the heart muscle is relaxed
This is in contrast to the systolic terms that use 'exists during'. Which is also a rogue relation

Hey Sue, would you mind trying the installing the latest protege version in parallel to the one you have (so you can continue using the old one if problems occur) and repeat the steps you described above and see whether the problem persists? @rsgoncalves is a member of the Protege team, and he will relay the issue to the development team. If you can make it a proper recipe: (1) I click this, (2) add that (3) change this (4) change that (similar to what you did, with a spice more detail), that would be great and help Rafael with debugging! Thaanks a lot!

I couldn't quite pin down the problem looking at the above posts, so as @matentzn said, it would really help to have detailed steps to replicate the issue. If possible, please try those steps in the latest Protege 5.5 as there are substantial changes. Thanks :) I'll stay tuned

I'll work on getting the 5.5 version on my machine next week. I have some other things I need to get done today.

@rsgoncalves @matentzn I installed 5.5 and tried to recreate the bug and I am still seeing the same bug in 5.5
Steps:

open mp-edit.owl
select term, I picked abnormal neuron physiology (MP:0004811) at random
in label view double click on the Equivalent to field to open the EQ
click OK to close the EQ
switch to IRI view and see the mess
Pictures of the EQ before
label:

IRI:

Pictures of the EQ after
label (note no apparent change)

IRI (note all the plain text in the EQ now)

I did not run the reasoner
I did not make any changes to the EQ just opened and closed using OK.

Same thing happens if I click the edit button to open the EQ
Same thing happens if I click cancel to close
Same thing happens if I click the red X to close
Another example where terms appear to get broken down into component parts
term is abnormal blood-brain barrier function
pre-IRI

post IRI

@dosumis Suggested this might be a rendering issue and not a real corruption. To test I did the steps above, saved the file, then reopened the file and checked the EQ. Turns out David is correct, when the file is reopened the IRI display is back to normal. So weird rendering issue but no permanent change to the file.

Wow, seems the bug was fixed for Protege 5.6 (next version). @rsgoncalves this still leaves open the question of how Protege label rendering disambiguates duplicate labels. We have to create annoying workarounds such as this one to deal with it. Now I see you have this cool new feature with adding the short form in brackets (loving it!). Can you see a user friendly way to add that to the class expression editor to avoid the duplicate label problem?

Thanks Nico. We'll have a look at your post and create an issue for it.

Can you see a user friendly way to add that to the class expression editor to avoid the duplicate label problem?

Will look into this, but my initial thought is that it sounds doable.  Might require some not insignificant changes to the OWL API parser though.

Just a heads up for this:

this still leaves open the question of how Protege label rendering disambiguates duplicate labels. We have to create annoying workarounds such as this one to deal with it.

5.6.0 will select a term that is defined in the active ontology over one that isn't, and then select a term that is most commonly used in the active ontology over one that isn't.  The upshot of this, for HP, is that typing 'quality' will bind to the PATO term rather than the BFO term.  See screenshots here:
https://github.com/protegeproject/protege/issues/869#issuecomment-479968839

Thanks @matthewhorridge! That sounds great!

superseded by #3753


Object detection module with Zed Mini

hi
are you planning to make the Object detection module compatible with the ZED Mini? Why this module is only available for the ZED2, is there a hardware limitation on the ZED-M preventing object detection to work properly?
thanks

There is some HW limitations for the ZED-M due to its small baseline and smaller FoV compared to ZED-M. Results will not work as expected, therefore object detection is limited to ZED2 which filled all those requirements.


Java: Bump lombok version.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@barshaul please re-review


Change "tags" to "skills"

That is what they are, after all. Most other job searching sites call them skills, and even though I'm using them as tags, they should be called an easier to understand name.

https://github.com/NicksPatties/megaresume/pull/61
This change fixes this. Although the data is still listed as tags, the UI calls them skills now.


add riot.route.parser for user defined path parser

Current riot.route limiting path / hash parser to only support simple hash, with this, user can customize the parser to handle custom hash like #!/user/1/edit or #!/user?sort=created

What a fantastic idea!
This makes router super flexible and it still stays minimal. Totally want to merge this.
Can you change: var parser = function(path) to function parser(path)` to make it similar to other function definitions.
Also, can you rebase from the latest master and make a new PR so I can automatically merge.
Again – thank you for this creative solution.

And perhaps you have the energy to extend the documentation with a more complex parser example to handle things like ?sort=created?

Thank you for response,
update the pull request to inline with route function style
add complex parser sample in document and provides test in test/route.html also
add fix riot.route.exec document to riot.route.exec(callback) also

Awesome, thank you!!
Will be published today.


Getting file ID once uploaded

Hi,
Is there a way to get the file ID after it is stored? Without the ID, I can't download the files. Is there a way to get the ID or listing the files and looping over them is the only way to achieve the results? Thank you in advance for your help.

Looks like this is the solution:
https://github.com/nao-pon/flysystem-google-drive/blob/master/example/GoogleUpload.php#L72
Closing for now.


wrong link on "next step" button of this page (suggestion for right link provided)

[Enter feedback here]
At the bottom of the page "https://learn.microsoft.com/en-us/azure/defender-for-iot/organizations/architecture" the "next step" is described as "understand your network architecture". however, the link beneath that blue button links to the very same page  which the button is presented on: "https://learn.microsoft.com/en-us/azure/defender-for-iot/organizations/architecture"
That doesn't make much sense. It would rather make more sense to link to this page (if that topic was the intent behind the button): https://learn.microsoft.com/en-us/azure/defender-for-iot/organizations/best-practices/understand-network-architecture

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

ID: b4d4c7a0-0278-3807-0e67-6e5b6f914db0
Version Independent ID: f1d31c39-4e99-cf52-d666-3b53378c9184
Content: System architecture for OT/IoT monitoring - Microsoft Defender for IoT
Content Source: articles/defender-for-iot/organizations/architecture.md
Service: defender-for-iot
GitHub Login: @limwainstein
Microsoft Alias: lwainstein

@limwainstein
Please review it.

@mcteufelms
Thank you for bringing this to our attention.
I've delegated this to content author @limwainstein, who will review it and offer their insightful opinions.

Thanks for catching this @mcteufelms! I'm adding this to our task list and we'll handle as soon as we can.

Hi @mcteufelms, I've opened this PR to fix the link: https://github.com/MicrosoftDocs/azure-docs-pr/pull/266840. The change will be visible on https://learn.microsoft.com/en-us/azure/defender-for-iot/organizations/architecture later today.

#please-close


Verify deployments based on container image digests, not tagging

For https://github.com/wellcomecollection/platform/issues/5597. This is a new approach to verifying whether a release is successfully deployed, using the image digests rather than tags managed by weco-deploy.

This patch introduces the notion of a "task spec", which is meant to be an unambiguous statement of what images we expect to be running in a particular service at a given time:
{
  "task_definition": "arn:aws:ecs:eu-west-1:756629837203:task-definition/matcher:4",
  "images": {
    "app": {
      "uri": "ecr.public.123/matcher:env.prod",
      "digest": "sha256:301faf69ff1b1a06166b550ac225a7d7b631e97e83ac9d476bf4e31f8911c576"
    },
    "logging": {
      "uri": "ecr.public.123/logging:env.prod",
      "digest": "sha256:c1bf75cfe97e8765cd838d1579c54a9ca7e460846cfabbc832194938c7aa130b"
    }
  }
}

In particular, because we have floating tags in our task definitions, these image digests may change – we get these digests by asking "what is the digest of the image currently specified by this tag?".
This is defined by TaskSpec in the models.py file.

When we call the new has_up_to_date_tasks() method, we work out an equivalent images object for every current task, e.g. a task might have:
{
  "app": {
    "uri": "ecr.public.123/matcher:env.prod",
    "digest": "sha256:26353e694b8d7b150c6f1b094f5e38b7d83b0f7e1757377465976710b1bbe840"
  },
  "logging": {
    "uri": "ecr.public.123/logging:env.prod",
    "digest": "<none>"
  }
}

Here the app container is running an old Docker image, and the logging container hasn't pulled an image yet, so we don't have a digest.

Compare the expected value and the actual value. If they're the same, this task has been successfully deployed. If they're different, we're still waiting for the deployment to complete.

Benefits of this approach

We ditch any awareness of 'releases' when checking if images are deployed correctly, which I think is behind the recent logic problems – we'll check the complete set of services specified by this project are up-to-date, with no awareness of what services might have been updated recently.
Long-term I want us to entirely get away from 'releases' and 'deployments'; this is just one step in that direction.

We no longer have to set or manage the deployment:label tag. Success is determined by running the correct images, not by a separately-managed value.

Because it just uses the image tags, it's not tied to ECS. In particular, if we were planning to deploy containers into some non-ECS environment, we could reuse a lot of code if we can extract image URIs and currently-running digests. 👀

Testing
Doing automated testing of weco-deploy has been a PITA in the past and I haven't really addressed that here. I have been deploying the staging catalogue API back and forth between latest and env.prod to see what happens; it does seem to place new containers and wait for them to be correct.
This is the output while it's waiting:
Task bafda7ab8e30412585a413cd3dfc806b in stage-search-api has the wrong containers:
- app: wrong image digest
    expected: sha256:0438d140618343b26e543015c0afb83fa9b69d185c51e9bed7d392d0f25b29f8
    actual:   sha256:26353e694b8d7b150c6f1b094f5e38b7d83b0f7e1757377465976710b1bbe840

Task c57dba43ccf94f9ca5429e69092cf680 in stage-search-api has the wrong containers:
- app: wrong image digest
    expected: sha256:0438d140618343b26e543015c0afb83fa9b69d185c51e9bed7d392d0f25b29f8
    actual:   sha256:26353e694b8d7b150c6f1b094f5e38b7d83b0f7e1757377465976710b1bbe840

So I think every image digest will be tied to (at least) one ref tag, so how about something like:
    expected: sha256:0438d140618343b26e543015c0afb83fa9b69d185c51e9bed7d392d0f25b29f8
              ref.58ddae35c9e7867acd3a2df137ece9984d24f7de
    actual:   sha256:26353e694b8d7b150c6f1b094f5e38b7d83b0f7e1757377465976710b1bbe840
              ref.d2e53d3069a6873d06446547983fae08898066ae

Would that be more useful?

or even better, use the code we've got elsewhere:
    expected: sha256:0438d140618343b26e543015c0afb83fa9b69d185c51e9bed7d392d0f25b29f8
              commit 58ddae3 (Remove the old 'api' repository)
    actual:   sha256:26353e694b8d7b150c6f1b094f5e38b7d83b0f7e1757377465976710b1bbe840
              commit d2e53d3 (Merge pull request #539 from wellcomecollection/bump-tf-ecs-module-version)

Think that's good - nit would be to switch the lines so the first thing you read is the commit, followed by the digest for clarity

Okay, this is what the latest version looks like:
Task e72397d4082e4655b54ef54d558694e1 in stage-concepts-api has the wrong containers:
- app: wrong image digest
    expected: commit a25ebce: Merge pull request #541 from wellcomecollection/remove-old-repo
              sha256:271ef4ad16e6c6a565aa3584afef90ad7f89deaeceaee619d9d6ef3ee6d65506
    actual:   commit e5a461e: Merge pull request #536 from wellcomecollection/bump-scala-libs-to-v32.17.0
              sha256:c1bf75cfe97e8765cd838d1579c54a9ca7e460846cfabbc832194938c7aa130b

Trying again in 10s (waited 3s).

Still waiting for task 642c68622f7b424eb6c0bc7c7130d5d6 in stage-search-api to stop
Still waiting for task e72397d4082e4655b54ef54d558694e1 in stage-search-api to stop

It has the new commit explanations as discussed above, but it only gives the detailed report once – so rather than spamming your terminal with output, it just tells you "those tasks I told you about before are still going".

And one more change, flipping it around to put the service name first:
stage-concepts-api: task a2c51067fcee42a49c8e5e6218e881d4 has the wrong containers:
- app: wrong image digest
    expected: commit e5a461e: Merge pull request #536 from wellcomecollection/bump-scala-libs-to-v32.17.0
              sha256:c1bf75cfe97e8765cd838d1579c54a9ca7e460846cfabbc832194938c7aa130b
    actual:   commit a25ebce: Merge pull request #541 from wellcomecollection/remove-old-repo
              sha256:271ef4ad16e6c6a565aa3584afef90ad7f89deaeceaee619d9d6ef3ee6d65506
Trying again in 10s (waited 3s).
stage-search-api: still waiting for task 1c0c98e84c8b40efa95c003f5c72e22e to stop
stage-search-api: still waiting for task 72d12bca5e6a4b74a6cbd18bbfe5c737 to stop
stage-search-api: still waiting for task e58940e87dee42699da4b82bcb902eb9 to stop

@jamieparkinson This feels like it's getting a little large and unwieldy; let me know if you'd like to split off some of the unrelated pieces into a smaller PR for easier review.


is the efficiency of CFR tied to the efficiency of state.child()?

From the comments introducing Python games:

Python games are significantly slower than C++, but it may still be suitable
for prototyping or for small games.
It is possible to run C++ algorithms on Python implemented games, This is likely
to have good performance if the algorithm simply extracts a game tree and then
works with that (e.g. CFR algorithms). It is likely to be poor if the algorithm
relies on processing and updating states as it goes, e.g. MCTS.

My interpretation of this is that if CFR extracts a game tree and works with that, then any slowness in state.apply_action() or state.clone() won't matter at all once we're done extracting the game tree, which makes sense and sounds great!
My question is: do the openspiel implementations of CFR actually "simply extract a game tree and then
work with that"?  From skimming the source of open_spiel/algorithms/cfr.cc, it looks like each call to EvaluateAndUpdatePolicy() traverses the game tree with the recursive ComputeCounterFactualRegretForActionProbs(), which looks like it calls state.Child(action).  So each call to EvaluateAndUpdatePolicy() calls state.Child(action) for each (state, action) pair in the game.
Is the comment wrong, did I misinterpret the comment, or did I misunderstand the CFR code?

tagging @elkhrt because I think you wrote the comment

My interpretation of this is that if CFR extracts a game tree and works with that, then any slowness in state.apply_action() or state.clone() won't matter at all once we're done extracting the game tree, which makes sense and sounds great!
My question is: do the openspiel implementations of CFR actually "simply extract a game tree and then
work with that"?

No, our CFR implementations don't do this. They walk the game tree procedurally on every iteration via states, applying actions, getting the next children states, and looking up information states from the keys from those states.
I know @michalsustr talked about making a CFR based on algorithms/info_state_trees, but I can't seem to find it now -- maybe it was never submitted?

Gotcha, thanks for the answer!  Should I close this now, or leave it open to track the comment update?

I know @michalsustr talked about making a CFR of the form you're describing.. based on algorithms/infostate_trees, but I can't seem to find it now -- maybe it was never submitted?

Yes, I have a version that prebuilds the infostate trees, computes full gradients on the sequence form and propagates it through the individual trees. It's a bit of code, I didn't get to publishing it yet, but maybe now I can in the coming weeks.

Cool, thanks guys. Yes @VitamintK, leave it open as a reminder to update the comment, thanks!


Nightly builds link is broken

Describe the bug
On page https://www.nuget.org/downloads
The link to nightly builds is broken:

// cc @nkolev92 , @zivkan

Next step: deploy!


[feat] I want to copy a file in the project and then paste it on the system desktop. How do I achieve this?

Describe the problem
I want to copy a file in the project and then paste it on the system desktop. How do I achieve this?
Describe the solution you'd like
I hope there is an API that can directly input the file URL I know and write it to the clipboard. I can paste it with ctrl+v on the system desktop.
Alternatives considered
No response
Additional context
No response

I need it too!!!
😥😥😥


Can we get previous generated File Size Report With Current Report ?

Can we get previous generated Bundle Size in generated HTML file or Json file ?

No, it only generates one file to output path per time.
You should handle it by yourself.

Thank for your response


& is overlapped in package & Deploy section

Bug Report

So I took a quick look into this because I suspected that this is an issue related to how the docs site handles different screen widths. From what I could quickly gather via the Firefox Inspector this seems to be a problem with scaling between a width of 992px and 1347px. We use a tool called Hugo for generating our docs site and I'm not entirely sure what is done behind the scenes for handling scaling.
@asmacdo do you happen to know off the top of your head if this is a simple fix?

Idk, but I imagine this could be Hugo, the Docsy theme, or our hacks on top of our Docsy fork.
We should be able to bump the Hugo version relatively easily, and it's worth doing even if it doesn't fix this issue.
To bump Docsy would require us to integrate with Docsy properly, rather than the patch we use in our fork.

Bug Report


Mouse down detection appears to be broken

Detecting whether left/right mouse button is down appears to return true for both buttons if either is down.
Perhaps updating to the latest raylib sources will resolve the problem.

Latest commits in raylib fixed this issue.
https://github.com/raysan5/raylib/commit/3a073db0290e004cafaf2fdc6f1dbbfc990ae68e
There was also a bug with key-input
https://github.com/raysan5/raylib/commit/6b8f30964dc9490fb4725a7c4764d793202ee48f


pSet Properties is not working.

Describe the bug 📝
https://github.com/IFCjs/components/blob/big-restructure/src/ifc/IfcPropertiesUtils/index.ts#L8
I am working with this feature but not get the success. Can you help me on this please?
I am using the **Advance IFC JS course for implementation. **
There is error when it's matching the type of property. Somewhere "IFCPROJECT" is matching with type.
Used the Defined the functions for pSet:

I tried with static expressID but it is also giving the error, see above screenshots.

Reproduction ▶
No response
Steps to reproduce 🔢
No response
System Info 💻
openbim-components

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x] Make sure this is a IFC.js components issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[x] Check that this is a concrete bug. For Q&A join our Discord Chat Server.
[x] The provided reproduction is a minimal reproducible example of the bug.

Is this a duplicate of #100?

Hi,
pSet Properties is not working. · Issue #101 · IFCjs/components · GitHub https://github.com/IFCjs/components/issues/101  and pSet Properties is not working. · Issue #100 · IFCjs/components · GitHub https://github.com/IFCjs/components/issues/100  is related. Yes you can say that It is same.
Shailendra Singh
SISO Software’s India Pvt. Ltd.
B-3/61, Safdarjung Enclave, IN-110029 Delhi, India.
Contact: +91-9911682739
http://www.siso.net/ www.siso.net
From: Antonio González Viegas @.>
Sent: Thursday, November 2, 2023 5:10 PM
To: IFCjs/components @.>
Cc: shailendra26-siso @.>; Author @.>
Subject: Re: [IFCjs/components] pSet Properties is not working. (Issue #101)
Is this a duplicate of #100 https://github.com/IFCjs/components/issues/100 ?
—
Reply to this email directly, view it on GitHub https://github.com/IFCjs/components/issues/101#issuecomment-1790566043 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A4QNQLLI3PGMP5YJIQ3YEA3YCOBCBAVCNFSM6AAAAAA3GRP7QOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJQGU3DMMBUGM .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/A4QNQLJ5SWEU27XPCPK7R2TYCOBCBA5CNFSM6AAAAAA3GRP7QOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTKXHPJW.gif Message ID: @.*** @.***> >

Ok, got it. I already mentioned this in another issue, but can you please answer in Github directly instead of answering your email? Your messages are getting bigger and bigger and it might make this a bit hard to follow. We would also need a full minimal application that we can run where we can reproduce this to fix it. Thanks a lot in advance!

@shailendra26-siso it's a bit hard to tell what is going on in your code, but a couple of things to keep in mind:

In other cases you've said you're still using web-ifc-viewer. Keep in mind, web-ifc-viewer and openbim-components are not compatible between them, in other words you can't work with them interchangeably in the same codebase.
In openbim-components, the IFC entity types are not strings, but numbers. So, if you want to take all entities from a particular type, then you would have to find it by its type numbers (you can find that in web-ifc).
Once again, it's a bit hard to know what is going on in your code, but from the screenshots you have shown the error tell me you are working with a variable named t that you are not defining anywhere.

Hope this helps somehow.

This is solved in PR https://github.com/IFCjs/components/pull/214.


the calculation of rho in step_lm() seems wrong

Thanks for the wonderful work!!
When I read the source code, I was confused by the calculation of rho in this line. I'll paste it here.
double rho = (y0 - yi) / (d.dot(lm_lambda_ * d - b));
However, because we want to evaluate the similarity between the original model and the approximate model, I think the correct formula should be as follows:
double rho = (y0 - yi) / (d.dot((H + lm_lambda_ * Eigen::Matrix<double, 6, 6>::Identity()) * d + 2 * b)) * (-1.0);
I will attach my proof here.

Thank you for reporting this issue.
I simply implemented the algorithm appear in Fig.1 in this paper, and I suppose rho is derived with some approximations. Although the formula you provided may be better, I'm not very sure. Note that this part brings only a very small difference in convergence speed.


added custom domain for phinetwork.eth to phi.vote

added custom domain for phinetwork.eth to phi.vote
Hi! What is your PR about?

[ ] Add or edit a skin
[x] Add a custom domain for your space
[ ] Add an alias to migrate your space

Originally posted by @Phinetwork in https://github.com/snapshot-labs/snapshot-spaces/pull/2197

What is the need for this issue?

What is the need for this issue?

This is already merged it wasn't at the time of this issue request thanks it's all set


[BUG] log ingestion example missing information

Describe the bug
I was following the log ingestion example in the repo and found that a few key steps make the readme incomplete. They are:

No instructions on how to set a custom opensearch config. This is useful if i want to play around with other config options from opensearch
The default auth credentials are not mentioned. Once i startup the compose file, i am not able to login to the Dashboard. turns out it is admin:admin
Once i login, i am not able to find the sample log index apache-logs in the index management view. Adding more logs to the test.log file shows that the DNS resolution for the default fluent bit container fails.
The common apache log examples do not map the timestamp as a date field on OpenSearch, making exploring them based on time impossible without adding a custom mapping. Given that the logs are ingested automatically once data is added, this means that the mapping has to happen as soon as the cluster is up, and if not, will need a reindex just to get the time fields mapped correctly for the demo. All this adds friction to a user who just wants to quickly setup a demo that they can play with and get a feel for the log ingestion flow. The timestamp field can be manually mapped and is a part of common apache log. Making this map by default to a date type would help demo the capability of data prepper a lot better.

To Reproduce
Steps to reproduce the behavior:

Go to 'log'
Observe the issues mentioned above

Expected behavior
None of the mentioned issues
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Ubuntu 20.04 LTS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

i havent been able to get the example working myself.  is the data-prepper part needed any longer?  the documentation mentions it as one of the components/service defined in the docker-compose.ym, but it is not there longer.
is the data-prepper component needed in general or can you feed from fluent-bit directly to opensearch?  if so, how?
im more interested in aggregation of application logs, not so much http logs.  i figured it would be simple to adapt this example to do that..but no luck.

all of the above has been observed, and still haven't resolved
Once i login, i am not able to find the sample log index apache-logs in the index management view. Adding more logs to the test.log file shows that the DNS resolution for the default fluent bit container fails.
Which basically means as a working example its pointless.

Also observed that the network on the data prepper run command had to be altered to
--network "log-ingestion_opensearch-net"
as it picks up the root directory for the compose network.

Actually just to add to the above
3. Once I manually created the Index over the apache-logs source name, the data started appearing. My reading was that this step was automicatilly covered.


Arrow keys and return key trigger NSBeep on 10.11

Whenever I hit the arrow keys or the enter key, now a system sound ("error" sound) occurs.
OSX El Capitan 10.11.13
glfw version : master HEAD

Works perfectly thanks for the quick fix!


feat: define constant CSS_CLASSES

related to #827

checkbox
chips
dialog
drawer
list
floating-button
icon-button

Codecov Report

Merging #982 into rc0.15.0 will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##           rc0.15.0     #982      +/-   ##
============================================
+ Coverage     94.13%   94.19%   +0.06%
============================================
  Files            86       93       +7
  Lines          3649     3689      +40
  Branches        578      578
============================================
+ Hits           3435     3475      +40
  Misses           90       90
  Partials        124      124

Impacted Files
Coverage Δ

packages/dialog/constant.ts
100% <100%> (ø)

packages/checkbox/NativeControl.tsx
100% <100%> (ø)
:arrow_up:

packages/list/ListGroup.tsx
100% <100%> (ø)
:arrow_up:

packages/checkbox/index.tsx
88.6% <100%> (ø)
:arrow_up:

packages/dialog/index.tsx
98.02% <100%> (-0.02%)
:arrow_down:

packages/dialog/DialogContent.tsx
100% <100%> (ø)
:arrow_up:

packages/chips/index.tsx
100% <100%> (ø)
:arrow_up:

packages/drawer/AppContent.tsx
50% <100%> (+5.55%)
:arrow_up:

packages/chips/chip/constant.ts
100% <100%> (ø)

packages/drawer/Content.tsx
50% <100%> (+5.55%)
:arrow_up:

... and 33 more

Continue to review full report at Codecov.

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

Please merge this PR first!

Please merge this PR first!