[TW#12274] Can't access SD card from task other than main

Accessing a file on a VFS mounted SD card from a task other than app_main doesn't work.  This is likely some sort of re-entrancy problem.  Attached is a very simple test case.
test_sd.zip

stdout here:
https://gist.github.com/jsarrett/a801b6967a0a089d41a2e19fe85af9bf

There are two issues observed with the code you have posted:

Illegalinstruction exception. In FreeRTOS, a task function is not allowed to return. Behaviour in this case is undefined, and is implementation-specific. In the ESP32 port, this causes an illegal instruction exception. When a task terminates, it needs to call vTaskDelete(NULL).

SDMMC driver error when communicating with the card from a task. This issue is caused by a race condition: the task code starts initializing the card, which main task code initializes GPIO4. GPIO4 is one of IOs used to communicate with the SD card. gpio_set_direction function changes the pin direction while the SDMMC peripheral is communicating with the card. This causes an error to be returned. To avoid this error, use a synchronization primitive to wait until SD card operation is complete, and then continue with the main task.

The example with both issues fixed can be found here: https://gist.github.com/igrr/57d9a0c4c4605f711634ee5c3e825d10

Indeed that fixed it.
Thanks for the second set of eyes, that was a pretty silly bug!
-James


Infer owl:inverseOf statements when ontology is ingested

I.e., if property p1 owl:inverseOf property p2, infer that p2 owl:inverseOf p1. This is a workflow convenience, because when Protégé saves an ontology it removes the inverse owl:inverseOf assertions. Vitro does this when the inverse is created in the editing interface, but not when ingesting the ontology data.

Moved to VIVO project JIRA.


[hotfix] Fixed charges in summary

Codecov Report

Merging #1136 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1136      +/-   ##
==========================================
+ Coverage   28.39%   28.41%   +0.02%
==========================================
  Files         342      342
  Lines       15086    15074      -12
  Branches     1348     1346       -2
==========================================
  Hits         4283     4283
+ Misses      10309    10297      -12
  Partials      494      494

Impacted Files
Coverage Δ

.../px/review_and_confirm/components/FullSummary.java
8.33% <0%> (+1.38%)
:arrow_up:

Continue to review full report at Codecov.

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


Create class for segmentation handling in agent

Background
When working with Atum data the Segmentation information has to be held and provided to the Atum routines.
Feature
Create a class which will hold the segmentation data and related setup.
The class then can either provide the Atum routines itself or work in tandem with an implicit class to DataFrame as an implicit parameter to the routines.
Expected basic functions:

Create a construtor with a map of Segmentation field name -> value of the segmentation
Have a constructor that create a new segmentation by providing additional pair(s) of Segmentation names and values - a sub-segmentation.

Proposed Solution
Class name Ideas:

AtumSegmentation or AtumSegmentationContext in case of implementation with implicits
AtumAgent in case of one class implementation

Pull req: https://github.com/AbsaOSS/atum-service/pull/45/files


Updated change log details

Docs Build status updates of commit 422195b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


firebase-messaging-sw.js has wrong SDK version

dpebot is missing the service worker's imports in its auto-dependency update PRs, which breaks the Firebase Cloud Messaging quickstart.

Fixed by this PR: https://github.com/GoogleCloudPlatform/repository-gardener/pull/109
You can see it working here: https://github.com/firebase/quickstart-js/pull/391/files#diff-7d78cef88f4a23512aa7fbe6633e800e


Update key related byron era cli commands with cardano-api

How do we reassure ourselves we've not broken something? We don't have that much automated testing of the Byron parts of the CLI. Perhaps we can check that the byron-to-shelley/ script still works: it uses the Byron genesis and key generation, and tx submission.
I've added some deserialisation tests and I will double check all the relevant scripts still work.

How do we reassure ourselves we've not broken something? We don't have that much automated testing of the Byron parts of the CLI. Perhaps we can check that the byron-to-shelley/ script still works: it uses the Byron genesis and key generation, and tx submission.
I've added some deserialisation tests and I will double check all the relevant scripts still work.

bors merge

bors merge


Page Multiple Roles For Each Incident

Hi there,
We noticed a change in behavior from previous versions where we are able to fire pages to multiple roles per instance. (e.g. page both the Commander and the Liaison). After merging v20220310 I'm noticing two different behaviors where:

It always only pages the Commander
When the Commander have is the same person as the Liaison, the pagerduty service id can get overwritten from Commander's service to Liaison's service. (seems to happen in flows.py within incident_create_flow function)

example of overwrite
6053` ^[[32mslack_1      |^[[0m DEBUG:Col ('participant_role_id', 'participant_role_assumed_at', 'participant_r     ole_renounced_at', 'participant_role_role', 'participant_role_participant_id', 'anon_1_participant_id'):/     usr/local/lib/python3.9/site-packages/sqlalchemy/log.py:debug:105
6054 ^[[32mslack_1      |^[[0m DEBUG:Row (1702, datetime.datetime(2022, 6, 2, 18, 34, 1, 358128), None, 'Incident Commander', 1170, 1170):/usr/local/lib/python3.9/site-packages/sqlalchemy/log.py:debug:105
6055 ^[[32mslack_1      |^[[0m DEBUG:Row (1703, datetime.datetime(2022, 6, 2, 18, 34, 2, 332127), None, 'Liaison', 1170, 1170):/usr/local/lib/python3.9/site-packages/sqlalchemy/log.py:debug:105
The service id now points at the Liaison's service
6067 ^[[32mslack_1      |^[[0m 2022-06-02 18:34:14,390 DEBUG sqlalchemy.engine.base.OptionEngine Col ('service_evergreen', 'service_evergreen_owner', 'service_evergreen_reminder_interval', 'service_evergreen_last_reminder_at', 'service_id', 'service_is_active', 'service_name', 'service_type', 'service_description', 'service_external_id', 'service_search_vector', 'service_project_id', 'service_created_at', 'service_updated_at')
6068 ^[[32mslack_1      |^[[0m 2022-06-02 18:34:14,390 DEBUG sqlalchemy.engine.base.OptionEngine Row (None, None, 90, datetime.datetime(2022, 1, 10, 17, 47, 37, 393480), 25, True, 'Product Incident Liaison', 'pagerduty-oncall', 'Liaison service for incidents handled by Product Support.', 'FAJIEWF', "'incid':2 'liaison':3 'product':1", 1, datetime.datetime(2022, 1, 14, 17, 46, 40, 896621), datetime.datetime(2022, 5, 20, 16, 28, 37, 423167))
My question would be is it possible to add the feature to page multiple roles or should we add in the feature ourselves and submit a pr upstream?

I fixed the overwriting issue here https://github.com/Netflix/dispatch/pull/2254


[5.1.5] Release notes

Wow, really appreciate how quick this is happening!

Wow, really appreciate how quick this is happening!

I was hopping for today, but things got a bit complicated, but now all validation pipelines are green and good, anyway I am ready to release the package 5.1.5, which will happen first thing Monday morning.
Also, a new release Microsoft.Data.SqlClient.AlwaysEncrypted.AzureKeyVaultProvider is coming next way.


feat: add GSO support

Description
PR takes care of adding GSO related instructions like Lockup and Withdraw excess collateral.
Screenshots

Need to increase the size of the config ix


Fix coordinate feedback

For coordinate questions: change the way coordinates are concatenated before being validated to avoid negative numbers erroneously being flagged as invalid.
Also implement invalid character checks in the same style as numeric questions.

I have some feedback about the numeric validation, but the requirements for that have changed a bit since we started. Since this is becoming increasingly separate from the original problem, I'll suggest that we undo the changes made to IsaacNumericQuestion here and make a separate PR for numeric cleanup. The coordinate stuff is looking good though!


WSL shell freezes upon automount while off from local network

Your Windows build number:  version 10.0.17134.472

What you're doing:  running WSL shell (here Debian 9) on my laptop, with automatic remote drive mounting (/etc/fstab), while being off from my local network.

myself@name:~$ cat /etc/fstab
LABEL=cloudimg-rootfs   /        ext4   defaults        0 0

\\localnetwork\remote_drive /mnt/remote_drive drvfs

Basically, while connected to the local network (\\localnetwork), everything works perfectly as expected (remote drives are being mounted automatically as I run the WSL shell).
On the other hand, being off from it results in an empty shell (no login) upon running the shell ; a ^C shuts down the window.
Alternatively, I discovered that rebooting the LxssManager Windows service allows to bypass the issue ; altough temporarily, which is somewhat annoying.
Does anyone have any idea to address this specific issue ?

Tagging this "file system" for the fact that WSL probably (possibly?) ought not freeze in the scenario, and certainly ought not require restarting LxssManager. I haven't actually reproduced here. But in general, sticking filesystems that might not be there in /etc/fstab is bad news on Real Linux™ too. This sort of thing is usually handled by more sophisticated mechanisms like systemd-automount (or similar), and WSL does not have a good systemd story right now (ref #994).
Work-around on WSL2 would be to test for the mount in your .bashrc, and if it isn't there, do a sudo mount.cifs ... on your share.

thanks for the reply !


Check contracts formatting

Feature Request
Check cairo contracts formatting in workflow by using scarb fmt --check.

Hello @franciszekjob can i work on this?

Hi @franciszekjob i will love to work on this issue

Hey @ScottyDavies , @ShantelPeters , you're welcome to open a PR for it :).


Zh cn

@all-contributors please add @Yuanxingzhu for translation

@all-contributors please add @Yuanxingzhu for translation

https://github.com/all-contributors please add @StarknetAstro for translation

@all-contributors please add @StarknetAstro for translation


Exclude the control-plane from our webhooks for psbinding webhook.

do not fight with azure's webhook for webhooks
Relates to #1590

/hold
Link the Azure issue?

It is linked from our issue, but it is this one : https://github.com/Azure/AKS/issues/1771
It is not clear it is an issue. They are doing a normal thing, and we are re-reconciling the object back to how we thing it should look. We should be smarter about mutation webhooks in general.

/unhold

/approve
/unhold

/test pull-knative-pkg-unit-tests


docs; fix example with 'continue'

This is a minor fix in the documentation.

nice catch, thanks for the PR!


Fixed password pattern in schema.yaml file

I have read the CLA Document and I hereby sign the CLA

recheck


Example wrong on http://jaxb.java.net/tutorial/section_3_1-Unmarshalling-and-Using-the-Data.html#Unmarshalling

The example code on the following page are incorrect:

http://jaxb.java.net/tutorial/section_3_1-Unmarshalling-and-Using-the-Data.html#Unmarshalling

It should be something like:
public  T unmarshal( Class docClass, InputStream inputStream )
throws JAXBException {
JAXBContext jc = JAXBContext.newInstance( docClass );
Unmarshaller u = jc.createUnmarshaller();
JAXBElement doc = (JAXBElement)u.unmarshal( inputStream );
return doc.getValue();
}
Another issue with this example is that it doesn't demonstrate caching the JAXBContext (which is thread safe). If the intent is not to show reusing the JAXBContext, then the example could be further simplified to be:
public  T unmarshal( Class docClass, InputStream inputStream )
throws JAXBException
{ return JAXB.unmarshal(inputStream, docClass); }
Environment
Online examples
Affected Versions
[2.2.7]

I believe this is not an issue any more


next: DateField reset segment based on DateValue

Describe the feature in detail (code, mocks, or screenshots encouraged)
Description
Provide a concise summary of the issue:
After implementing the DateField component, I noticed a challenge when trying to reset the segments individually after resetting the overall DateField value. Currently, the process is not straightforward or configurable, making it difficult to handle scenarios where resetting specific segments is required alongside the main DateField reset.

Steps to Reproduce

Implement a DateField component.
Attempt to reset the DateField value programmatically.
Observe that individual segments do not reset as expected.

Expected Behavior
When resetting the DateField value, the corresponding segments should also reset seamlessly without requiring additional manual configuration.

Actual Behavior
Resetting the DateField value does not reset the segments, making it cumbersome to handle specific use cases where segment-level resets are necessary.

Proposed Solution
Provide a built-in mechanism or API in the DateField component to reset individual segments along with the overall value, or document how this can be achieved with manual configuration.

Additional Context

Environment<EMAIL_ADDRESS>
What type of pull request would this be?
Enhancement
Provide relevant links or additional information.
No response

I'm not able to reproduce this behavior.
Are you referring to changing the value like this?
https://github.com/user-attachments/assets/23391b02-0149-48e4-afbe-950c0d40b1c0

Or do you mean reset as in no value at all?


Require /oem mount point for cos-setup-fs.service

This commit adds a dependency of cos-setup-fs.service with the /oem mount
point. In addition the timeout of the /oem and /usr/local is raised a
little so it is really unlikely they are missed on the first attempt to
mount them and their dependency to /etc is dropped. In fact we do not
care about what is mounted first /etc or /oem, we care that both are
mounted when cos-setup-fs.service is executed.

Looks good here! :+1:


There is an extra '...' next to the 'Customize Columns' in the toolbar

Storage Explorer Version: 1.32.0-dev (93)
Build Number: 20230928.9
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Ventura 13.5 (Apple M1 Pro)
Architecture: x64/x64/x64
How Found: From running testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Tables.
Create a table.
Hover on the 'Customize Columns' button in the toolbar.
Check whether there is no extra '...' next to the 'Customize Columns' in the toolbar.

Expected Experience
There is no extra '...' next to the 'Customize Columns' in the toolbar.
Actual Experience
There is an extra '...' next to the 'Customize Columns' in the toolbar.

Verified this issue on the main build 20231007.7. The tooltip has been updated!


I think you didn't renew the domain

I can't access to demo page on shortener.cagatay.pro

You absolutely right, thanks for feedback, I’ll reactivate demo shortener as soon as possible. ⚡

Redirected to https://shortener.cagatay.dev
Thx for feedback!

You're welcome, thanks


RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU.

Hello, i wanted to try your fork of sadtalker in colab, but I keep getting this error:
--------device----------- cpu Traceback (most recent call last):   File "inference.py", line 217, in <module>     main(args)   File "inference.py", line 43, in main     preprocess_model = CropAndExtract(sadtalker_paths, device)   File "/content/xtalker/src/utils/preprocess.py", line 49, in __init__     self.propress = Preprocesser(device)   File "/content/xtalker/src/utils/croper.py", line 22, in __init__     self.predictor = KeypointExtractor(device)   File "/content/xtalker/src/face3d/extract_kp_videos_safe.py", line 28, in __init__     self.detector = init_alignment_model('awing_fan',device=device, model_rootpath=root_path)      File "/usr/local/lib/python3.8/dist-packages/facexlib/alignment/__init__.py", line 19, in init_alignment_model     model.load_state_dict(torch.load(model_path)['state_dict'], strict=True)   File "/usr/local/lib/python3.8/dist-packages/torch/serialization.py", line 713, in load     return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)   File "/usr/local/lib/python3.8/dist-packages/torch/serialization.py", line 930, in _legacy_load     result = unpickler.load()   File "/usr/local/lib/python3.8/dist-packages/torch/serialization.py", line 876, in persistent_load     wrap_storage=restore_location(obj, location),   File "/usr/local/lib/python3.8/dist-packages/torch/serialization.py", line 175, in default_restore_location     result = fn(storage, location)   File "/usr/local/lib/python3.8/dist-packages/torch/serialization.py", line 152, in _cuda_deserialize     device = validate_cuda_device(location)   File "/usr/local/lib/python3.8/dist-packages/torch/serialization.py", line 136, in validate_cuda_device     raise RuntimeError('Attempting to deserialize object on a CUDA ' RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU.
I followed instructions, tried with the int8 branch and the main branch too. I tried making it work for hours but no success with and without GPU. Tried adding map_location=torch.device('cpu') on every instance of torch.load  in inference.py.
i changed the installed python version from python 3.10 to 3.8 because it's required by sadtalker. i believe i tried with python 3.10 too but i doubt the problem is with the python version.
Thanks in advance.

Hi @diabolo98 , this error lies in the facexlib dependencies, not in inference.py. Please in your /usr/local/lib/python3.8/dist-packages/facexlib/alignment/__init__.py , try edit the line
model.load_state_dict(torch.load(model_path)['state_dict'], strict=True)

to
model.load_state_dict(torch.load(model_path, map_location=device)['state_dict'], strict=True)

Thanks, i would"ve never thought of changing the init file. it fixed the issue, but xtalker still wouldn't work.
After it solved the problem above, I had another issue related to intel_extension_for_pytorch. I checked the installation guide and I found out needed to match the torch version, so I installed the correct version using !python3.8 -m pip install intel_extension_for_pytorch==2.0.0 -f https://developer.intel.com/ipex-whl-stable-cpu  but I get the error that the CPU doesn't handle AVX2, which it does.  Apparently this is a known problem and was only fixed in 2.0.1 of intel_extension_for_pytorch.
I was able to bypass it by changing /usr/local/lib/python3.8/dist-packages/intel_extension_for_pytorch/cpu/_cpu_isa.py
then I got an error about the CPU not having AVX512 and friends, which i then "fixed?" by changing dtype from bfloat16 to float line 78 of /content/xtalker/src/facerender/animate.py
now, I get this and honestly, i have no idea what its about :
 device========= cpu
---------device----------- cpu
0000: Audio2Coeff
0.9539964199066162
No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'
0001: AnimateFromCoeff
5.883184432983398
3DMM Extraction for source image
landmark Det:: 100% 1/1 [00:04<00:00,  4.24s/it]
3DMM Extraction In Video:: 100% 1/1 [00:00<00:00,  5.08it/s]
0002: preprocess_model generate
11.044325828552246
eyeblick? pose?
None
None
mel:: 100% 1452/1452 [00:00<00:00, 32997.74it/s]
audio2exp:: 100% 146/146 [00:08<00:00, 17.53it/s]
0003: audio_to_coeff generate...
23.92700743675232
/content/2023_08_21_21.06.47/00963-1453378001##output_1.mat
rank, p_num: 0, 1
[kp_detector]:
1.2990715503692627
[mapping]:
0.021234512329101562
0.014824390411376953
Face Renderer::   0% 0/726 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "inference.py", line 217, in <module>
    main(args)
  File "inference.py", line 152, in main
    result = animate_from_coeff.generate(data, save_dir, pic_path, crop_info, \
  File "/content/xtalker/src/facerender/animate.py", line 178, in generate
    predictions_video = make_animation(source_image, source_semantics, target_semantics,
  File "/content/xtalker/src/facerender/modules/make_animation.py", line 141, in make_animation
    out = generator(source_image, kp_source=kp_source, kp_driving=kp_norm)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/content/xtalker/src/facerender/modules/generator.py", line 212, in forward
    out = self.first(source_image)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/content/xtalker/src/facerender/modules/util.py", line 255, in forward
    out = self.conv(x)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/intel_extension_for_pytorch/nn/utils/_weight_prepack.py", line 127, in forward
    return torch.ops.torch_ipex.convolution_forward(x, self.weight, self.bias, self.ctx.get_data_handle(), self.weight_size, self.padding, self.stride, self.dilation)
  File "/usr/local/lib/python3.8/dist-packages/torch/_ops.py", line 502, in __call__
    return self._op(*args, **kwargs or {})
RuntimeError: could not create a primitive descriptor for a convolution forward propagation primitive

Here's the colab : https://colab.research.google.com/gist/diabolo98/dbffa21b297174058ca903763d45db8e/test-11111.ipynb

Hi @diabolo98 , I just noticed that you are doing the experiment on colab. I have not ever tested it on colab with cpu only. colab's cpus are limited. I just have a simple look on the cpu of colab and there are only 1 core with hyperthreading, which means the IOMP parallel optimization in xtalker should not be enabled, because it does not have parallel computation on only 1 physical core.  Of course, you are also not enabling that yet because I find the rank, p_num: 0, 1 in log. So it doesn't matter.
However, I think only the BFloat16 optimization can still make some performance improvement (1~2x). As you mentioned, you found that intel_extension_for_pytorch==2.0.0 did not work, so have you tried install the latest intel_extension_for_pytorch==2.0.100? Please do not change the bfloat16 to float because we need bfloat16 to do acceleration.
Regarding to the last error RuntimeError: could not create a primitive descriptor for a convolution forward propagation primitive, that is an oneDNN error. This is probably caused by that you change bfloat16 to float, which is a invalid path that may conflict the oneDNN kernel size or stride.  I recommend you to try the 2.0.100 ipex version firstly, keep bfloat16 and see whether the error exist.

int8 optimization itself does not depend on intel extension for pytorch here so it will not raise the above errors. But again, I've not tested it yet in colab environment. Will do some experiments at weekend.

I think colab CPU are too limited to be of any use here. I tried to use tortoise TTS that advertise a huge speed-up with half precision, deep speed and KV cache optimization, but on COLAB it says 4 hours for a few minutes of TTS so no matter how much you'll optimize xtalker it will probably still be unbearably slow. Nonetheless, I have a few questions :

Would the converted int8 model still work with the GPU ? I mean, the conversion won't make the model only works with CPU, right ?
Is colab specs enough for the conversion ?
Can i share the converted model afterwards to hugging face ?

Colab is truly limited in CPU because it tends to let you use GPU or TPU to do computing intensive work. My optimization is tested on Xeon Sapphire Rapids (check README), so I fully understand that you think it make no sense to do it on Colab.
I do not think the converted int8 model will work on GPU. My int8 optimization is based on https://github.com/intel/neural-compressor, which is mostly tested on Intel Xeon CPUs. I recommend you to try TensorRT if you really want to do int8 optimization on NV GPU.
I think Colab is enough for int8 conversion with CPU. However I guess it will be slow. You can share the converted model to HF if you want, but I think you should follow MIT license, which the original SadTalker has.
By the way, xtalker is irrelevant to TTS. You should not take the slow speed of any TTS component as the reason that xtalker will also be slow. BTW, I've tested tortoise TTS before. tortoise TTS is, as its name implies, SUPER slow. Try other alternatives :)

I usually try each component individually, so it is not a problem of using multiple models at once.
I know xtalker is irrelevant to TTS. I gave it as an example because they advertised a huge boost in speed on CPU, and so did bark TTS (or was it just bark, I tried both)  but despite that they're still extremely slow when using the COLAB CPU. On the other hand, tortoise TTS recently became relatively fast on colab GPU and became very usable. I hoped the int8 would've improved the speed on GPU, but as you said it wouldn't work. Also, Sadly my knowledge about AI is extremely limited, so I don't think I can use TensorRT to do the int8 conversion for GPU unless it's a drag and drop process that require relatively simple adaptation and a lot of documentation reading.
I think we've diverged a far bit from the main issue and the purpose of this repo and and I apologize for that.

No problem, your feedbacks are welcome :)


Set parent class for DFTD3Dispersion to avoid logger crash.

DFT-D3 crashes in verbose mode. Example:
from pyscf import gto, scf, dftd3
from pyscf.lib import logger

mol = gto.Mole()
mol.atom = ''' O    0.00000000    0.00000000   -0.11081188
               H   -0.00000000   -0.84695236    0.59109389
               H   -0.00000000    0.89830571    0.52404783 '''
mol.basis = 'cc-pvdz'
mol.verbose = logger.INFO
mol.build()

mf = dftd3.dftd3(scf.RHF(mol))
print(mf.kernel())

leads to the error message:
AttributeError: 'DFTD3Dispersion' object has no attribute 'stdout'

Codecov Report

Merging #537 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #537      +/-   ##
==========================================
- Coverage   82.29%   82.29%   -0.01%
==========================================
  Files         574      574
  Lines      109938   109938
==========================================
- Hits        90473    90472       -1
- Misses      19465    19466       +1

Impacted Files
Coverage Δ

pyscf/dftd3/itrf.py
78.57% <100%> (ø)
:arrow_up:

pyscf/pbc/cc/eom_kccsd_ghf.py
93.85% <0%> (-0.08%)
:arrow_down:

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 3e426b2...92e9c5c. Read the comment docs.


Signal 11 (segmentation fault) in do_sigsuspend

Hello,
I got the following segmentation fault during envoy deployment:
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Core was generated by `/usr/bin/envoy -c /etc/envoy/active-conf.json'.
Program terminated with signal 11, Segmentation fault.
#0  0x00007fad562e654b in do_sigsuspend (set=0x3c) at ../sysdeps/unix/sysv/linux/sigsuspend.c:60
60
The used version is:
[root@selfcare-dqwpj /]# curl -H "Accept: application/xml" -i -X GET  http://<IP_ADDRESS>:9901/server_info
HTTP/1.1 200 OK
content-type: text/plain; charset=UTF-8
cache-control: no-cache, max-age=0
x-content-type-options: nosniff
date: Fri, 22 Jun 2018 13:35:36 GMT
server: envoy
transfer-encoding: chunked
envoy 4dd49d8809f7aaa580538b3c228dd99a2fae92a4/1.6.0/Clean/RELEASE live<PHONE_NUMBER>92 0
Unfortunately the VM this core prouced is not accesible any more to get more symptoms.
Any ideas?
Thanks,
Apostolos

Sorry no there isn't much we can do with this. Please setup your prod environment to capture core dumps or at least back traces.


[vault-secrets-webhook] namespaced access to secrets

Is your feature request related to a problem? Please describe.
I'm currently trying out the vault-secrets-webhook on top of an existing Vault deployment.
I'm currently using a vault-template sidecar along with existing deployments to load secrets and trying to figure out how to migrate to the webhook.
For authentication to Vault I'm using a dedicated ServiceAccount in each namespace. Therefore the vault policies and the roles used to authenticate are also namespaced to the specific deployments running in the cluster. Apps can only access the secret their individual ServiceAccount from their namespace has access to.
As far as I understand the vault-secrets-webhook, access to all Vault data is authenticated with a single ServiceAccount, residing in the namespace of the webhook.
Describe the solution you'd like
From a security perspective, I would like to have an option to let the webhook use a ServiceAccount in the namespace of the to-be-mutated resource. Otherwise, it would be possible for any resource in any namespace to query for any secret that the webhook has access to.
Describe alternatives you've considered
It is quite possible, that this is already possible with the webhook and I didn't figure out. If not I would propose to allow the webhook to be configured to use a ServiceAccount in the namespace of the resource requesting a secret, possibly also auto-creating this service account

Hi @philband, I suggest to read and the blogs: https://banzaicloud.com/docs/bank-vaults/mutating-webhook/ but in a nutshell: when injecting into Pods entities always the ServiceAccount of that Pod is used to access Vault, let it be vault-env or the sidecar version.
For injecting into Kubernetes Secrets, the ServiceAccount of the webhook is used.

Hi @bonifaido, thank you first of all for the quick response.
I now realize, that my question was not as clear as I intended. How this works for adding secrets to deployments etc. is clear to me.
The question/request was intended to ask, whether it is possible to use a custom ServiceAccount for mutating Secrets and/or CRDs?

Yeah, that is not possible currently. And I wonder how could we implement that.

After reviewing the webhook codebase, I think I might have an idea on how to implement this feature.
I would propose to add an optional annotation vault.security.banzaicloud.io/vault-serviceaccount, which would hold the name of a service account in the namespace of the Secret, ConfigMap or CustomResource to be mutated.
If this annotation is present, the webhook shall query the k8s API for a ServiceAccount with that name in the namespace of the resource requesting mutation. From that, it should fetch the jwt associated with the ServiceAccount.
With this jwt, the webhook can authenticate against vault, as if the request came from the ServiceAccount specified in the annotation.
This ServiceAccount would have to be handed down to the vault client wrapper, replacing the default ServiceAccount token injected into the webhook-pod:
https://github.com/banzaicloud/bank-vaults/blob/7fadfb8416b0d1cd3ac5db5aba5d94e56ac50b8f/pkg/sdk/vault/client.go#L274
If I'm not mistaken, this would also require adjustments in the helm chart for the webhook (RBAC), to allow the webhook to query ServiceAccount resources.
Should you be agreeable to this proposal, I will try to make a PR for it.

The technical part of the proposal sounds right to me. But giving the webhook the right to query and read service account tokens sounds a bit a broad scope of privileges that concerns me. If the webhook as all the rights to read service accounts people won't install it, if we can make this more scoped to only a list of service accounts that might be okay.

Current state, according to the helm chart of the webhook:
https://github.com/banzaicloud/bank-vaults/blob/7fadfb8416b0d1cd3ac5db5aba5d94e56ac50b8f/charts/vault-secrets-webhook/templates/webhook-rbac.yaml#L8-L27
The webhook can already view and update Secrets and ConfigMaps across all namespaces.
So here is another idea:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: vault-webhook-mutator
rules:
  - apiGroups:
      - ""
    resources:
      - serviceaccounts
    verbs:
      - "create"
      - "get"
      - "update"
    resourceNames:
      - "vault-webhook-mutator"

We give the webhook permissions to create, get and update a ServiceAccount with a well-known name (e.g. vault-webhook-mutator), that is just used for the purposes of the mutation.
The jwt attached to the secret referenced by the ServiceAccount can already be read.
One thing to consider in this idea: Should the webhook auto-create the ServiceAccount if it is missing? Or is this a task to be done by the user?

Sounds good to me! 👍
I think we should leave the SA creation to the users IMHO.

@philband Have you made any progress on a PR for this feature? The lack of namespace isolation in the current solution makes it unusable for us.

@ron1 would be the configurable auto-creation of the ServiceAccount enough for you?

One thing to consider in this idea: Should the webhook auto-create the ServiceAccount if it is missing? Or is this a task to be done by the user?

@bonifaido Yes, configurable ServiceAccount creation would work for us. The GoDaddy Kubernetes External Secrets controller uses a simpler strategy to solve this problem which would also work for us. This controller introduces a namespace annotation to enforce naming conventions for backend keys as described here: https://github.com/godaddy/kubernetes-external-secrets#enforcing-naming-conventions-for-backend-keys. If managing specific, per-namespace service accounts is too cumbersome or introduces too much overhead, using naming conventions to control backend key access would be sufficient for us. WDYT?

I like this namespace annotation idea TBH. But this needs to be implemented in the webhook I guess.

This seems like a reasonably straight-forward validation that could be performed by a webhook against the Secret/ConfigMap upon creation or edit. Do you agree?
I suspect this approach would also solve issue https://github.com/banzaicloud/bank-vaults/issues/964 in a more streamlined manner than creating individual webhooks to control specific namespaces.

Great, I absolutely agree! Are you about to give it a stab @ron1?

@bonifaido When ConfigMaps and Secrets are parsed using envFrom in a Pod, is the serviceAccount of the Pod or the WebHook used when accessing Vault.

@bonifaido @philband After looking at the following code in the GoDaddy Kubernetes External Secrets controller that enforces the namespace isolation: https://github.com/godaddy/kubernetes-external-secrets/blob/71901209cf76491948072f675e1732cb7efdffbf/lib/poller.js#L204, I think the @philband idea of enforcing namespace isolation using standard Vault constructs rather than webhook-applied regex tests is preferred.
If a user wants Secrets/ConfigMaps mutated in a namespace, I think it is reasonable to require that they create a vault-webhook-mutator SA in that namespace and that they define a Vault Role and accompanying Policy for that SA. Attempting to create or update a Secret/ConfigMap that has a vault url in a namespace without an appropriately entitled vault-webhook-mutator SA would fail validation. Does that seem reasonable?

@bonifaido Note that the below comment on the vault-role annotation in doc section https://banzaicloud.com/docs/bank-vaults/mutating-webhook/#example leads me to believe that the vault-webhook-mutator already (by default) uses the default SA in the namespace to control access to vault for Secrets and ConfigMaps. Based on your comments above, I presume I am misinterpreting that comment included below. Or does the comment only apply to the use of Secrets and ConfigMaps referenced in Pods via envFrom?
      annotations:
        ...
        vault.security.banzaicloud.io/vault-role: "default" # optional, the default value is the name of the ServiceAccount the Pod runs in, in case of Secrets and ConfigMaps it is "default"

@bonifaido When ConfigMaps and Secrets are parsed using envFrom in a Pod, is the serviceAccount of the Pod or the WebHook used when accessing Vault. It is not clear to me in this doc: https://banzaicloud.com/docs/bank-vaults/mutating-webhook/.

When there is runtime entity mutated, like a Pod which has a serviceAccount then always its SA  token is used to access Vault, even when using envFrom, since that is just a "syntactic sugar".
For mutating ConfigMaps/Secrets/Any Objects the webhook SA is used.

@bonifaido @ron1 Do you guys have any ETA for when the new vault.security.banzaicloud.io/vault-serviceaccount annotation might be available to use? Thanks.

We don't have an ETA, for this, PRs are welcome! 🙂

Would this feature be compatible with OIDC issuer discovery https://banzaicloud.com/blog/kubernetes-oidc/ ?

@ron1 did you find any workaround for your use case ? The lack of namespace isolation makes it unusable for us too :(


Question on Stack Overflow

Hi! I'm a Dataverse developer and I just did a search for "Dataverse" on Stack Overflow and found this question: https://stackoverflow.com/questions/49433615/jupyter-notebook-modulenotfounderror-no-module-named-cfuzzyset
Can someone please post an answer there?
This is a neat project and I'm glad I found it! I especially like your visualization but I did notice a typo ("subjet" instead of "subject"). Here's a screenshot:

For my own reference, the datasets seem to be hosted in Harvard's installation of Dataverse: https://dataverse.harvard.edu/dataverse/Bioversity

Thanks for the feedback.
The typo is now fixed.

@marieALaporte thanks for fixing that typo! 👍  💯
Thanks also for replying to that person on Stack Overflow!


container: Allow output to be printed "live" and captured

This allows the user to see what is going on and mitigates
issues with travis-ci timing out when commands take too
long.
Also added code that simplified the use of
psq-travis-container-exec. There is now no longer
a need to specify a distro and release if a container
already exists and it has a .distroinfo file in it.

Changes Unknown when pulling e680591efa168bba9dfc803e545fdc75ac5bf7d0 on smspillaz:polysquare-travis-container.live-output into ** on polysquare:master**.

Changes Unknown when pulling aeefc71c290bfb9fabe84e7103e3f3b9876de35d on smspillaz:polysquare-travis-container.live-output into ** on polysquare:master**.

Changes Unknown when pulling a76cf7c3d6cce044f3717e6e969d126de8d13ff6 on smspillaz:polysquare-travis-container.live-output into ** on polysquare:master**.

Changes Unknown when pulling cb5abea79920bf850dc1a404411073b5055afec5 on smspillaz:polysquare-travis-container.live-output into ** on polysquare:master**.


[MINOR][LAUNCHER]Reuse EXECUTOR_MEMORY and EXECUTOR_CORES in SparkLauncher

What changes were proposed in this pull request?
Currently， EXECUTOR_MEMORY and EXECUTOR_CORES  in SparkLauncher is never used.
This PR will be reused them.
How was this patch tested?
existing test cases.

Can one of the admins verify this patch?

These do look removable, but I don't think we should bother with PRs to remove things like this one or two at a time.

@srowen
In fact, we can replace spark.executor.memory with SparkLauncher.EXECUTOR_MEMORY and replace spark.executor.cores with SparkLauncher.EXECUTOR_CORES. But in the process of modification, it also interrupts the consistency of other codes. So I chose to remove it.

OK, I'd either use these constants where they can be used within the launcher module, or remove them.

My only concern is that some external projects which rely on this static variables will be broken because of this change, since they're public available, we cannot stop users to rely on them (though they can also change the codes).

@srowen @jerryshao

Maybe @vanzin can weigh in here, because the real question is whether these constants in the launcher module are meant to be the single definition of them used throughout the code. core depends on launcher and uses these constants a little bit, but not consistently. Most of the other code doesn't seem to use it. That is, there are hundreds more changes like this you could make.
Consistency is good. In contrast, there are only about 8 usages of these constants outside the launcher module. Is it simpler to achieve some consistency by removing those usages? Then it seems like a small step backwards to not use them (and yet declare them), but is also much less change.
In the end, this is why I don't know if it's worth trying to standardize, because it is also hard to keep it standard anyway.

They're there mainly to declare the constant as a public API that must not change. (I'm not sure whether mima captures changes in constant values, since that's a binary breaking change, but that's the spirit of having these constants.)
I didn't change all of the usages when I introduced them because it would be really noisy. There are also a whole bunch of other constants that could be re-used throughout the code (basically all the constants declared in SparkLauncher). But I think there's no real need to change this - we can encourage new code to use the constants, but leave the old code there until it needs to be changed, to avoid noise.

Since there's no explicit +1, I would suggest to leave the code as it is.

thanks @srowen @jerryshao @vanzin


typo coinsiding > coinciding

https://github.com/savkov/bratutils/blob/0b3ba5c3336c99136dfb5711606cd35e932e83c5/src/bratutils/agreement.py#L444
many thanks for your work. saved me days.

Merged in #10


Mysql faster

import-full-data.sh progress bar if pv is installed
Docker directly inherits from ubuntu
use data.table (instead of data.frame) internally
removed some helper functions in data_acess.R for better readability
made numerical algo_id (implementation.id) in database obsolete
improved SQL Query for get_parameter_table: Does everything in one command
memoise caching without time limit but on disk
renaming: algo_id is always something like "mlr.classif.ranger"
renaming: parameter etc. to par_vals if they are actual values to not get confused with parameters that are the definition

@mboecker Please look at everything. If you totally dislike something change it / tell me.

By the way, the time for the first request is substantially higher for me now, but I think this is irrelevant if the queries afterwars are faster?


Edit transcripts and publish videos from Design System Day 2024 event 2

What
Edit the transcripts for all recordings from the face to face Design System Day event so they are accurate.
Why
So we can publish the videos
Who needs to work on this
Nora
Who needs to review this
Gareth Pryce, Lucky Nwosu (comms)
Done when

[x] we have edited Greg Macoy's talk
[x] we have edited Chika's talk
[x] we have edited the content panel recording
[x] we have edited Duncan's talk
[x] we have edited Michael & Alessandra's dual narrative recording
[x] we have edited Ishmael's talk
[x] we have Youtube thumbnail images for all sessions
[x] we have sent these to comms to be published

https://docs.google.com/spreadsheets/d/1GoB91TII6Rp6NtDXxQA_bdIjwgw4CaDMk39Pz67dkdQ/edit?gid=1503972698#gid=1503972698
Captions edited

sent to<EMAIL_ADDRESS>in comms team


Tradfri motion sensor event detection

Where are you using pytradfri (eg stand-alone, Home Assistant etc)
Stand-alone / raspbian / raspberry Pi 3 model B+
Version of pytradfri
master
Expected behaviour
Hi, first of all thank you for this great package.
This is more a question than an issue. I have a Tradfri motion sensor which I'd like to use for my homemade alarm system.
I have almost everything working :  lights control, sound warnings, snapshots sent by email, smartphone notifications and live streaming. The last thing that I need is the motion detection event to trigger all of this, and I have no idea on how to observe "motion sensor events" using pytradfri and the Tradfri motion sensor...
Any ideas or hints ?

Unfortunately it doesn't work as the gateway doesn't report changes to the motion sensor. We'll have to wait until IKEA exposes changes to the motion sensor in the gateway.

Ok thanks :) Meanwhile, I'll check the state of a bulb triggered by the motion sensor, that'll do the job as well.


add to_not matcher

error messages need help but should be an exercise for another PR imo

r? @samphippen @sgrif

Thoughts on having a NotMatcher and the result being to(not(whatever)) instead of to_not(whatever)?

oh i could do that too, i just did what it seemed like @samphippen was suggesting i do

there's going to be a separate not matcher eventually, but separately the existence of a to_not is useful, so this seems good as is.

@samphippen @sgrif shall i make a not matcher?

@ashleygwilliams if you want to, yeah!

wrrd, i'll file an issue


Increase fitted FFD generation test tolerance

Purpose

As per title, see #255 for more details
Expected time until merged

Type of change

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing

Checklist

[ ] I have run flake8 and black to make sure the Python code adheres to PEP-8 and is consistently formatted
[ ] I have formatted the Fortran code with fprettify or C/C++ code with clang-format as applicable
[ ] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

@eirikurj are you happy to merge this?


Patch 1

A couple of small changes -- the main one is that clicking in one of the blank sidebar regions now does not make the main text lose focus.

Any chances that the patches get integrated?
Especially the first ones are really simple and useful!

I concur


feat: add a way to cache state between restarts of Neovim

This is useful for features such as org-clock-in-last that has the ability to persist its "knowledge" of the last clocked in task. Org mode does this by saving to a cache file on system. This commit gives nvim-orgmode the same capability.
I'm creating this PR in what I consider a partially finished state. I'm primarily looking for feedback as to the design of this module. In the scenario we're all happy with the design then I can get some simple tests written and then promote this to a non draft PR.
Some quick notes:

This state management system uses the in-built promises library that nvim-orgmode bundles. This means there's some additional validation and steps taken than what might exist in fully synchronous code.
I use a metatable to ensure any accesses against the State table in which we assign new values are saved into the subtable data. This is so later on it is easy to determine what to save into the cache. Without it, I'd have to write additional logic explicitly excluding specific keys from the State table.
Seeing as the State object is a singleton is may seem weird to bother with a State:new() function. This actually is quite important, this sets up the metatable after all other functions are read in the file, ensuring they are not bound to the data subtable within the State object.
Reading the cache should work on all platforms as I'm "normalizing" the paths with vim.fs.normalize.

Any feedback (especially critical feedback) is desired on this. I'm looking to iron out kinks and find out if this is even desired 😃.

@kristijanhusak I'm primarily interested in what you think. If the above all sounds good and the code looks solid, I intend to get this merged and then turn around and implement at least org-clock-in-last with persistent state between Neovim

Thanks for the PR. I didn't give it a deeper look yet. I have one question regarding this:

Org mode does this by saving to a cache file on system

Can you point me where I can read more about this? I wasn't aware it is done in Emacs, at least not in this way. I assumed Emacs has a more advanced persistence.

So emacs is a little bit fancier than the method done above to persist state. That index file contains:
((:container
  ((elisp org-element--headline-cache)
   (elisp org-element--cache))
  :persist-file "26/0e7168-1132-40fb-9c4a-f9527c58b0eb" :associated
  (:hash "db811d99c98367fd949e030078d6c3dc" :file "/home/sam/org/sprints.org" :inode 1798784)
  :expiry 30 :last-access<PHONE_NUMBER>.4788425 :last-access-hr "2023-10-31T09:02:16-0500")
 (:container
  ((elisp org-element--headline-cache)
   (elisp org-element--cache))
  :persist-file "16/77b350-d23c-435f-a1af-79fab01dc901" :associated
  (:hash "9b28fb51f68a22e72bfe80dd6eea5713" :file "/home/sam/.notes" :inode 1798482)
  :expiry 30 :last-access<PHONE_NUMBER>.958719 :last-access-hr "2023-10-31T07:08:54-0500")
 (:container
  ((elisp org-element--headline-cache)
   (elisp org-element--cache))
  :persist-file "32/2a423d-7966-474a-b285-bec583e15678" :associated
  (:hash "f821ab3b4dde77f42b322d56138e8b6f" :file "/home/sam/Notes/Whitelist-API.org" :inode 1853863)
  :expiry 30 :last-access<PHONE_NUMBER>.2776916 :last-access-hr "2023-11-01T09:12:35-0500")
 (:container
  ((elisp org-element--headline-cache)
   (elisp org-element--cache))
  :persist-file "7a/71cd55-9320-4ced-8290-64f32f790869" :associated
  (:hash "0c752e61eadaa71b7e3408264c6a2349" :file "/home/sam/Notes/sprints.org" :inode 1709198)
  :expiry 30 :last-access<PHONE_NUMBER>.8752766 :last-access-hr "2023-10-30T02:09:43-0500")
 (:container
  ((index "3.1"))
  :persist-file "c0/b60f88-9bc6-45e0-a7f9-711167b5b71d" :associated nil :expiry never :last-access<PHONE_NUMBER>.2797225 :last-access-hr "2023-11-01T09:12:35-0500"))

Which definitely is more complex than what I'm implementing here.

@kristijanhusak
Found it, source code for org-persist can be found in a mirrored Github repository here.
Or you can directly get it by cloning from Savannah, git clone git://git.sv.gnu.org/emacs/org-mode.git && cd org-mode && $EDITOR lisp/org-persist.el . It has a fair bit of its own documentation in the top of the file. I was unable to access said docs in Emacs 🤷.

Also, it would be great to write some tests around this to ensure it works as expected.

Absolutely. I just wanted to check that this looked good in the first place before writing up tests etc. only to be told that it's a no go.
I'll get those tests written after resolving your comments in a sec here.

Still need to write a test for the self-healing.

As a heads up @kristijanhusak this should be good to go. If you're happy with this, I'm going to begin implementing some missing orgmode features that can use persistence.

@kristijanhusak Let me know if you see anything else.

@treatybreaker how did you test this manually?
I tried setting up a keymap like this to ensure it's working but it doesn't seem to do anything:
vim.keymap.set('n', '<leader>u', function()
  local state = require('orgmode.state.state')
  return state:load():finally(function()
    print('LOADED', state.data)
    state.field = 'value'
    state:save()
  end):next(function()
    print('SAVED')
    vim.print('UPDATED STATE', state.data)
  end)
end)

Prints are never shown, and org-cache.json file is never created.
Also, tests are not actually checking anything. For example, if you change this line https://github.com/nvim-orgmode/orgmode/pull/624/files#diff-23e5eb566ad6dac12d0c71f0d549d5e273bd9ed2c79b837688a0ad21e0946dd7R52 to expect something that it should not get, tests still pass:
-- This passes, even though it shouldn't
assert.are.equal(state.my_var, 'hello world foo')

Something seems to be off. Can you check? Thanks.

I was testing by shoving require("orgmode.state.state") into Orgmode's top level init.lua.
I believe I found the issue which was caused by the early return. I remember not doing an early return for a reason, I just couldn't remember why. Now I do.
In terms of testing, I believe the tests aren't valid because they're expecting a synchronous context and since the promises are in the "background" the tests complete before the actual callbacks have finished. 🫤

Alright, I've fixed some functionality in the State:load function so it all should be good there. Works on my side.
Now I get to figure out async testing with plenary.

@kristijanhusak
This was uhhh, painful. Async code in lua is the very definition of not fun.
I think this should be good now, I have the tests responding correctly now by abusing vim.wait. And I did some minor fixes to the State module. One of those is tracking the State:saved context which particularly useful for tests.
I am really hoping this is good at this point. If it's not then I'm going to keep on grinding until it's in a good state.
Take a look and let me know your findings.

As a note, Neovim didn't seemingly create those stdpaths at startup with the minimal init... I had to create them during the minimal init setup.

Awesome, now it works :)
vim.wait is a way to go. I would even suggest introducing load_sync() and save_sync() as part of the State class, where it would do the same thing you do in tests. Then you can leverage that in tests and we also have it as an option if needed.
Feel free to increase timeout to 1000 and use an interval of 5. I did similar thing to ensure that org files are loaded.

@kristijanhusak
Added those functions as well as a State:wipe function. In the State module I had to do some fancy stuff with State:save as I realized there can be multiple saves running as once. Tracking this in the context now. For the sync functions I went with an interval of 20 with a default timeout of 500 ms. If that's too low, let me know and I can do a quick commit to change that.
Man... async code and multiple execution is hell on earth. Very useful, but my god.
Looking forward to more hellish findings 😉.

Working on replicating the failure in 0.9.1.

Fixed.

Let me know if you see anything else 🙂

I'll look into implementing some features that take advantage of this come this weekend or next week as a heads up.


Refactor to use arrow go bindings

In https://github.com/apache/arrow/pull/11538 support for writing to parquet files will get merged into upstream arrow for golang, does it make sense to drop use of the pure go parquet library? The C++ bindings are likely better performance.

Interesting. We can experiment with that yes. Will take a look at some point.


Remove columnattrs=true query arg and move into PQL

Original ticket below - title was changed, read comments for details.
Description
Currently it is possible to set column attributes on non-inverted frames, but there is no way that I know of to retrieve them. A Bitmap query using the column ID will fail as follows:
curl localhost:10101/index/repository/query \
     -X POST \
     -d 'Bitmap(frame="stargazer", repo_id=10)'
{"error":"Bitmap() cannot retrieve columns unless inverse storage enabled"}

Success criteria (What criteria will consider this ticket closeable?)
We need to either disallow SetColumnAttrs if inverse frames is not enabled, or provide some way to retrieve column attrs without the bitmap.

@jaffee Adding ?columnAttrs=true returns column attrs even for frames where inverseEnabled==false:
curl -XPOST 'localhost:10101/index/myindex/query?columnAttrs=true' -d 'Bitmap(frame="f1", rowID=1)'
{"results":[{"attrs":{},"bits":[100]}],"columnAttrs":[{"id":100,"attrs":{"a":1,"b":2,"frame":"f1"}}]}

ah, good tip, thanks @yuce . Does that seem a bit strange to you? (or anyone?) to use a query arg to change the behavior of PQL?

I think we definitely should have another look at PQL.

Let's move the columnAttrs=true arg into PQL and get rid of the query arg.

Adding columnAttrs=true to PQL directly has the potential to open a can of worms.

Which PQL calls would support columnAttrs=true ? What happens in a nested query? (e.g., the user can specify it in the outmost call or one or more of the inner calls)
That requires a change in proto files.
Some client library queries become worse. e.g., we have to change the Go client, so it's like: frame.Bitmap(5, true) or frame.BitmapWithColumns(5) (if we will support columnAttrs for other calls too, then we need to have a lot of similar looking functions).

All good points - sounds like we'll need to put more thought into this.
Do you think it's ok the way it is @yuce ? How do the client libraries expose this functionality now?

We can add a new call, maybe something like: ReturnProfiles(...) which can be used at the outermost of bitmap queries: ReturnProfiles(Bitmap(frame="foo", rowID=5)). If the innermost query is not a bitmap query, that would return a syntax error (like: ReturnProfiles(SetBit(...)))
Currently, the users set Columns: true in their QueryOptions object, which is used to set the Columns field of the protobuf QueryRequest object.

closing this and moving discussion to the proposal in #700


basic weapon ammo

just basic reloading the weapons
No animation for reloading.
No sound for reloading.

Thanks for creating this! If you could, please create a fork with new features as I'd like to keep the Starter Kit itself very basic and thus no new features will be added


MECH - Add a pause after a fast forward jump is triggered

Things noticed after testing
The player might slide occasionally after sprinting


migrate/fresh --migrationTable --migrationPath

What steps will reproduce the problem?
/yii migrate/fresh --migrationPath=@app/modules/<module>/migrations --migrationTable=<module>_migrations
What is the expected result?
Run migrate/down for all the migrations in the --migrationPath
What do you get instead?
Deletes all the tables from the database and run migrate/up in --migrationPath

"Are you sure you want to reset the database and start the migration from the beginning?"
What if there are tables generated from multiple migrationPath or migrationNamespaces? Since migration supports migrationPath/migrationNamespaces I think fresh should also respect both.

if you have multple sources the workflow for refresh would be:
./yii migrate/fresh --migrationPath=@app/modules/<module1>/migrations --migrationTable=<module1>_migrations
./yii migrate/up --migrationPath=@app/modules/<module2>/migrations --migrationTable=<module2>_migrations
...

@cebe Understood. But warning message of "actionFresh" says
"Are you sure you want to reset the database and start the migration from the beginning".
Here "start the migration from the beginning" makes no sense. Since it only runs the migrate/up in "migrations" or path specified in migrationPath.
If migrate/fresh use --migrationPath for migrate/up it should use --migrationPath for truncate also.
I suggest to change the "function truncateDatabase()" to the following
$this->interactive = false; $this->actionDown("all");
The developer can also ensure that the rollback (actionDown()) is working properly for all migrations specified in the migrationPath
#14664 @thyseus @samdark

When i implemented the migrate/fresh command i had this feature from Laravel 5.5 in mind:
https://laravel-news.com/migrate-fresh
There it says:

What is also beneficial with this new command is [...] it made the up and down methods in migrations optional. Now you can leave off the down and still quickly reset your local database.

Indeed having multiple sources of migrations is not really covered by this feature. I wouldn't mind to change the description sentence of this console command to cover this.

@thyseus understood. So
laravel migrate:fresh => yii migrate/fresh
laravel migrate:refresh => yii migrate/redo all


install kwok during make cluster-sync to work with simulated VMIs

Is your feature request related to a problem? Please describe:
sig-scale is working on integrating https://kwok.sigs.k8s.io/ for stress and scale testing the kubevirt stack. For this to work, there needs to be an option to install kwok in make cluster-sync
Describe the solution you'd like:
Running the following command should
KWOK_DEPLOY=true make cluster-sync

install kwok stack to the kind cluster created by make cluster-sync
create the kubevirt related resources (role, rolebinding, stage vmi-ready, stage vmi-delete), similar to this https://github.com/kwok-ci/kubevirt-demo/tree/main/kustomize

cc @rthallisey

/assign


Show only leading in SliverAppBar

I have page with expandable SliverAppBar and I want it to show only leading like int the image below when it collapsed. How can I do that?

Hi @egprojects
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


RemoteStructServiceProvider Custom Header dose't sent to backend

Steps to Reproduce
This is my code
import { Editor } from 'ketcher-react';
import 'ketcher-react/dist/index/css';
import { RemoteStructServiceProvider } from 'ketcher-core';

const structServiceProvider = new RemoteStructServiceProvider(
  'http://localhost:8002' + '/api/eln/ketcher/v2',  // this my proxy url to '/api/v2'
  {
  	'X-Token': 'token',
    'custom header': 'value', // optionally you can add custom headers object
  },
);

const MyComponent = () => {
  console.log('headers', structServiceProvider.customHeaders);
  // log headers {X-Token: 'token', custom header: 'value'}
  structServiceProvider.customHeaders = { 'X-Value': 'value' }
  // this also does't show on browser request headers
  return (
    <Editor
      staticResourcesUrl={'/'}
      structServiceProvider={structServiceProvider}
    />
  );
};

Actual behavior
In the browser Request Headers, it doesn't show any X-Token, custom header, X-Value, and backend service doesn't recevice custom headers.
Expected behavior
The Browser send custom headers to backend.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 22.04 and indigo-service in docker.
Browser: Chrome, edge
Version: <IP_ADDRESS>78.140

Ketcher version [e.g. v2.4.2].
2.26.0 and 2.25.0
Test case
Add test case number (in EPMLSOPKET-* format) if bug was found during test case run.
Additional context
React 18.3.1

I notice there are some routes without custom headers. /info is don't have, but /calculate with. So, I need split unauthorizad routes and authorizad routes, then proxy with different ways?


Modules for Calculating Molecular Interaction Fields and Alignment to Principal Axes/PMI Descriptors

A new module for calculating Molecular Interaction Fields (MIFs) has been added along with Real Value Vector and a Real Value Grid data structures for storage of grids.
Another feature is the Alignment to Principal Axes Code which enables aligning molecules to their principal axes and calculating principal moments of inertia (PMI) descriptors.

Closing this because it will take a completely new PR at this point.
@dfhahn : if you still are interested in integrating this (and I'd like to see that if you are willing to do the work), please update the PR to be relative to the current master and re-submit.


[SMALLFIX] Fix Permission when listStatus in AbstractFileSystem

Fix a bug in listStatus that Alluxio Hadoop API does not return correct mode/user/group
some cleanup

To test,   run bin/hadoop fs -ls alluxio://localhost:19998/aaa
before
-rw-rw-rw-   3          0 2016-11-19 17:13 alluxio://localhost:19998/aaa/bbb
-rw-rw-rw-   3          0 2016-11-19 17:13 alluxio://localhost:19998/aaa/ccc

after
-rw-r--r--   3 binfan staff          0 2016-11-19 17:13 alluxio://localhost:19998/aaa/bbb
-rw-r--r--   3 binfan staff          0 2016-11-19 17:13 alluxio://localhost:19998/aaa/ccc

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12357/
Test PASSed.

@calvinjia  will add one after https://github.com/Alluxio/alluxio/pull/4354 merged


Core dumped when initializing a pyclass on Linux only

Your operating system and version: Linux (Ubuntu 18/20, doesn't matter)
Your python version: Python3.8 (3.6, 3.7 also)
How did you install python (e.g. apt or pyenv)? Did you use a virtualenv?:  No.

So, this bug is weird. It happens only on Linux (not on macOS nor Windows, I've tested).
I'm rewriting python-ext-wasm, a project that uses Pyo3. Here is the PR, https://github.com/wasmerio/python-ext-wasm/pull/212.
To build:
$ git clone https://gtihub.com/Hywan/python-ext-wasm
$ cd python-ext-wasm
$ git fetch
$ git checkout -b reboot
$ just prelude
$ source .env/bin/activate
$ just build

To test, comment everything inside the tests/test_module.py file except line 20 to 21 (def test_compile_wat) as it is a small test case involving very few lines of code, and it fails.
Then run with:
$ just test tests/test_module.py
…
Fatal Python error: Aborted
[stack trace]
Aborted (core dumped)

I'm currently tracing what happens. The Module pyclass is defined as: struct Module { inner: wasmer::Module }. If I change to inner: u32, it works. So it seems that wasmer::Module is not liked by Pyo3 or Python.
The Module pyclass is defined in src/module.rs. The wasmer::Module is defined in https://github.com/wasmerio/wasmer/blob/80290e48055655a63f05f2c329f2ace4adbc712f/lib/api/src/module.rs#L33-L37 (for the record: struct Module { store: Store, artifact: Arc<dyn Artifact> }).
I'm digging for the moment. PyClassInitiliazer.create_cell_from_subtype is working so far. Since it is the last thing returned by the __new__ implementation, I don't understand where it can fail. Or maybe the class isn't well initialized.
Important: It works on macOS and Windows. It fails only on Linux. It's not random, but for some Module (Python class), it works! If you take the def test_exports() test case, in the same test_module.py file, you'll see that it works.
It drives me crazy. Help :-).

I've rewritten the test case to separate each line, and see dbg statements from Pyo3 in-between.
def test_compile_wat():
    print("--> store\n")
    store = Store()
    print("/ store\n");

    print("--> module\n")
    module = Module(store, '(module)')
    print("/ module\n")

    print("--> isinstance")
    assert isinstance(module, Module)
    print("/ isinstance")

The Fatal Python error: Aborted comes after / isinstance. It seems to be related to the destructor of the object.

I can reproduce the issue locally, this reliably causes crashes:
>>> from wasmer import Store, Module
>>> store = Store()
>>> module = Module(store, '(module)')
>>> del store
>>> del module
[1]    32123 abort (core dumped)  ipython

Switching the order of the dels fixes the problem. There was an issue with unsendable classes which got fixed in #1058 and #1060, but I could still reproduce the issue after cherry-picking the commits on 0.11.1.
I took a quick look at the Store and Module implementations but didn't have to time to dig through all of that whether Module steals a reference to the &Store. That could lead to a use-after-free and cause the breakage. But that's just a blind stab, I haven't checked for any of that. At least within the Python module everything looked sound.

So.
The destructor of Store is called first. It is called by PyCell::py_drop. The current code (as of 0.11.1, same on master at the time of writing) is the following:
ManuallyDrop::drop(&mut self.inner.value);
self.dict.clear_dict(py);
self.weakref.clear_weakrefs(self.as_ptr(), py);
self.inner.ob_base.py_drop(py);

So the destructor Store is called on the Rust (impl Drop for Store), then dict, weakref and inner are executed correctly.
Then, the destructor of Module is called:

ManuallyDrop::drop is OK, the destructor on the Rust side is called (impl Drop for Module).
dict.clear_dict is failing.

Second run is OK. Having printf fixes the problem. Yepee…
To be continued :-p.

Most definitely not related to unsendable, I removed the attribute and the issue persists.
It's not even necessary to delete any variables, it sometimes crashes while cleaning up after exiting Python:
>>> from wasmer import Store, Module
>>> store = Store()
>>> module = Module(store, '(module)')
Do you really want to exit ([y]/n)?
[1]    7632 abort (core dumped)  ipython

Yes, I come to the same conclusion.

So.
The destructor of Store is called first. It is called by PyCell::py_drop. The current code (as of 0.11.1, same on master at the time of writing) is the following:
ManuallyDrop::drop(&mut self.inner.value);
self.dict.clear_dict(py);
self.weakref.clear_weakrefs(self.as_ptr(), py);
self.inner.ob_base.py_drop(py);

So the destructor Store is called on the Rust (impl Drop for Store), then dict, weakref and inner are executed correctly.
Then, the destructor of Module is called:
* `ManuallyDrop::drop` is OK, the destructor on the Rust side is called (`impl Drop for Module`).

* `dict.clear_dict` is failing.

I'm actually seeing failure before dict.clear_dict:
    unsafe fn py_drop(&mut self, py: Python) {
        println!("enter pydrop");
        ManuallyDrop::drop(&mut self.inner.value);
        println!("dropped");
        self.dict.clear_dict(py);
        println!("cleared dict");
        self.weakref.clear_weakrefs(self.as_ptr(), py);
        self.inner.ob_base.py_drop(py);
    }

"dropped" never gets printed. So my hunch is that something within the module is gone once the store is gone?

That's what I'm digging right now. It doesn't seem related to Pyo3. Want to be sure before closing the issue :-).

Many thanks @sebpuetz for helping investigate this! I don't see any obvious red flags from the source I just scanned. Are you able to reproduce this issue using cargo run or in a test instead of the Python REPL? Should be possible to create Py<Store>, borrow() it to create a module (and eventually Py<Module>), and then drop the two objects.
It doesn't seem like this is a PyO3 issue at the moment, but happy to be proved wrong!

That reminder made me take another look and it's not related to pyo3:
    #[test]
    fn test2() {
        let store = wasmer::Store::default();
        let module = wasmer::Module::new(&store, "(module)").unwrap();
        mem::drop(store);
        mem::drop(module); // abort
    }

Stepping through the code in a debugger shows that the abort happens while dropping wasmer_engine_jit::unwind::systemv::UnwindRegistry. More specifically, the last call before abort is __deregister_frame in  UnwindRegistry::drop. __deregister_frame is an extern "C" function.
A fix is switching the order of the struct members in wasmer::Module:
pub struct Module {
    artifact: Arc<dyn Artifact>,
    store: Store,
}

Artifact holds a pointer to a Vec owned by Store that gets passed to __deregister_frame. If the Store goes out first, the pointer in Artifact's UnwindRegistry becomes invalid but gets used anyways.
Allocation happens here: https://github.com/wasmerio/wasmer/blob/80290e48055655a63f05f2c329f2ace4adbc712f/lib/engine-jit/src/artifact.rs#L165-L166
Subverting lifetimes happens here: https://github.com/wasmerio/wasmer/blob/80290e48055655a63f05f2c329f2ace4adbc712f/lib/engine-jit/src/unwind/systemv.rs#L86
Assuming dead-pointers are still alive happens here: https://github.com/wasmerio/wasmer/blob/80290e48055655a63f05f2c329f2ace4adbc712f/lib/engine-jit/src/unwind/systemv.rs#L104-L106
I think you can close this, I'll open an issue over at wasmer.

Superb sleuthing, thanks again!

It's not related to Pyo3. I found the bug in Wasmer, https://github.com/wasmerio/wasmer/pull/1581. Thanks for your time and sorry for the noise!


Custom certificate alias not being used when using HC5 AsyncHTTPConduit

The HttpContext is not available in the AsyncClientConnectionOperator, so we have to provide our own TLS strategy on construction.

@ffang  sorry, another one for you please :)

LGTM!


0 uses singular word instead of plural word

When the value hits 0, the plural word should be used instead of the singular word.
Currently it shows 0 day instead of 0 days. See also https://ell.stackexchange.com/a/51183.

Interesting.
Should be optional by a parameter, because if I take French as an example, both solutions are valid.


Add a service to provide Aliases for Table Names

No functionality changes in this PR, just an interface addition. Existing tests should suffice.

With this change, the response will have the alias name. We need to test this case. If a compute engine calls getTable for table 'abc' and Metacat returns the Table object with name 'xyz', will the engine fail?

@ajoymajumdar that's a good point. For now, I've updated the getTable() API to return the TableDto with the alias. We'll address other APIs on a case-by-case basis.


Add a C-API

This way nnnoiseless can be a drop-in replacement to rnnoise.
If there is interest I can send you a PR for that.

I guess I'm not opposed because I don't think it would add much maintenance burden. But I'd be honestly surprised if a non-rust project decided to use nnnoiseless instead of rnnoise. Why introduce the extra build-system complications of adding a rust dependency when there's a perfectly good C library available?

It makes easy to compare the two implementations, and if the rust one is faster, you pay having cargo-c installed to build it, but beside that it is exactly the same for the C project.
I'll show you soon :)

Closed by #7


Py2 qt4

Solve jpg supporting for py2-qt4 branch in Windows by adding plugins.
Add auto save while toggle to previous image.
Add Simgle Class Mode to speed up annotation.
Add openPrevImg and openNextImg action to advanced mode.
Add Change default save dir to tool bar.
Fix crash bug while choosing a point to delete.

Looks cool. I will spare some time to test and verify it manually.

@GarrickLin
SingleMode looks nice. However, after my test, I have some problems after trying your patch:

Cannot use hotkey 'D' and 'A' to switch images after annotating the image
Is it necessary to remove other classes from predefined_classes.txt ?

Actually it does not conflict with any other hotkeys
Not necessary
Cannot reproduce in my Win10 machine

I run the py2-qt4 branch in my Win10 machine. Actually I just add very few codes to inplement this functionality.
labelImg.py line 291-300
        self.singleClassMode = QAction("Single Class Mode", self)
        self.singleClassMode.setShortcut("Ctrl+Shift+S")
        self.singleClassMode.setCheckable(True)
        self.lastLabel = None
        addActions(self.menus.view, (
            self.singleClassMode,
            labels, advancedMode, None,
            hideAll, showAll, None,
            zoomIn, zoomOut, zoomOrg, None,
            fitWindow, fitWidth))

labelImg.py line 639-643
        if self.singleClassMode.isChecked() and self.lastLabel:
            text = self.lastLabel
        else:
            text = self.labelDialog.popUp()
            self.lastLabel = text

You can just add those code to your own code and test wether it works.

Hi @GarrickLin ,
I know the root cause. The branch is py-qt4 instead of master. Num 3 is the old problem, it has been fixed in master branch. Also, I will sync your code regarding 'single mode and auto saving' to master branch

Uploading issue_template.md…


add facility to set root password

I don't think it's good best practice to allow the root account to have a password. Before we merge this, I would like to discuss it with SCS team security next Wednesday.

I don't think it's good best practice to allow the root account to have a password. Before we merge this, I would like to discuss it with SCS team security next Wednesday.

Its okay, when you disable SSH login as root (we should give that advice here).
In situations of major system problems (a completely  filled up partition, hardware problems, crashed systems processes, overload, ...)  it is sometimes useful to have the possibility to use a root password on the bmc console to have the possibility to analyze the situation without rebooting.
(Especially when the /etc/hosts or /etc/hostname file is broken and "sudo" refuses to give you administrative access.)

We'll discuss this on Wednesday. I prepared https://github.com/osism/ansible-collection-commons/pull/672 in favor of this PR to be able to set the passwords of all user accounts.

We'll discuss this on Wednesday. I prepared #672 in favor of this PR to be able to set the passwords of all user accounts. This way we do not have to make this explicit for the root user account.

"when we have an operator account with password and possibility to elevate privileges there is no need to set password for root"
If you really want to set the password for the root account you can now use the osism.commons.user role for this: https://github.com/osism/ansible-collection-commons/blob/main/roles/user/defaults/main.yml#L16-L22


LE: Can't get automatic HTTPS to work on Google Container Engine (GKE)

I have been trying to get automatic HTTPS to work on my Kubernetes cluster on GKE. I was starting caddy with caddy -log stdout -agree -email<EMAIL_ADDRESS>and using the following Caddyfile:
alexferl.com {
    gzip
}

www.alexferl.com {
    redir https://alexferl.com
}

I get the following error:
2015/12/08 23:14:36 [INFO] acme: Obtaining bundled certificates for alexferl.com, www.alexferl.com
2015/12/08 23:14:36 [INFO] acme: Trying to solve HTTP-01
2015/12/08 23:14:38 [INFO] acme: Trying to solve TLS-SNI-01

2015/12/08 23:14:39 [alexferl.com] failed to get certificate: The server could not validate our request.
[www.alexferl.com] failed to get certificate: The server could not validate our request

It seems the LE server can't connect back to me, but I have a load-balancer with a static IP (<IP_ADDRESS>) pointing to the Kubernetes workers on ports 80 and 443 and DNS entries for alexferl.com and www.alexferl.com pointing to that static IP so in theory it should work.
$ host alexferl.com
alexferl.com has address <IP_ADDRESS>
$ host www.alexferl.com
www.alexferl.com has address <IP_ADDRESS>

I have also tried various different Caddyfile configs (just alexferl.com, just www.alexferl.com etc.) without success.
Any ideas?

Are you sure you can guarantee that the load balancer is always proxying the connection from the ACME server to the machine that's running Caddy totally transparently? SSL termination will break it the TLS-SNI-01 challenge, and proxying HTTP-01 insufficiently will cause that challenge to break, too. Given that there's nothing useful in the process log, I'm guessing the requests aren't even getting to your machine.
Until the DNS challenge is implemented by Let's Encrypt, you're on your own if you must be behind a load balancer other than Caddy. :-/

It's using the network (L3) load-balancer so it is not terminating SSL and just forwards the whole TCP stream to the workers. I was thinking it would work, but it seems that it doesn't since I have found that people on AWS with ELB have the same issue so it's really a lets-encrypt "problem" and not the caddy implementation of lets-encrypt.
Oh well! Thank you!

Interesting - but, at least you are not alone! :smile: Best of luck.

@admiralobvious FWIW the DNS challenge will be available in Caddy 0.9, which means it should work well behind load balancers.

@mholt Awesome! I'll give it a try once 0.9 is released. Thank you!


Added improving-key-expiration-in-reddis-at-twitter

It's a useful article to understand how Twitter has made changes to its Redis clusters recently.

@avidLearnerInProgress Cool! Thanks Chirag!


FileInput Label Rendering Fix

Addresses issue #212
Add a fix_file_input_label function to renderers.py to add a new line after the label for FileInput widget.
Rendering after the fix

The fix itself looks good to me, but you also added a bunch of unrelated files. Can you please remove them?

@xi How exactly do I do that now?

One more to go (pyenv.cfg) 👍

@dyve Yup done!

🎉


feature request - webui hosted proc args

When one wants to add an argument before any existing ones, you need to remove the existing ones and add them back - would be nice if we could insert variables in the correct place and possibly be able to move the order around as well

I've been testing a few different ways to handle env variables and arguments and I think the best way for this might just be a simple semicolon separated text input. I've switched to that for now, but will keep this request in mind if we switch to another way of handling these config items sometime later.


Cannot open composer, hangs on 'starting server'

Tried uninstalling, reinstalling GA version, reinstalling nightly, and always end in the same: when I start composer the blue screen at the beginning stays there forever.

Any workarounds or suggestions on how to gather error information / logs? The same installers worked in other computers.

@carlosscastro you can set the environment variable DEBUG=composer* and start the app from the command line.

@carlosscastro assigning to you to get the team some debugging logs. I expect you will see an error


Remove invalid ** usages from dummyapp/.gitignore

https://git-scm.com/docs/gitignore says that double asterisk patterns in gitignore are only valid when used with directory separators, i.e. **/..., .../**, and .../**/....
I noticed this when using ripgrep. It seems that its interpretation of ignore patterns is stricter than Git's:
$ rg something
./vendor/bundle/ruby/2.4.0/gems/rollbar-2.18.0/spec/dummyapp/.gitignore: line 24: error parsing glob '**.war': invalid use of **; must be one path component
./vendor/bundle/ruby/2.4.0/gems/rollbar-2.18.0/spec/dummyapp/.gitignore: line 58: error parsing glob '**.orig': invalid use of **; must be one path component
./vendor/bundle/ruby/2.4.0/gems/rollbar-2.18.0/spec/dummyapp/.gitignore: line 73: error parsing glob '**.swp': invalid use of **; must be one path component
...

I think **/*.x is more in line with what the original intent was.

Okay, change made. Please take another look. Thanks!

Great PR. Thanks @allenap . Merged into master


Stop training button

(1) Instead of running for a set number of epochs, run until the user presses a "stop training" button.
Solving this issue would also solve the following:
(2) The user should not be able to spam the "train {federated, decentralised}" button. Solving (1) would mean pressing the button again would stop (pause) training.
Further enhancement, requires (1) & (2) beforehand. I'll open another issue if enough people vote for this:
(3) Allow the user to enter their own desired number of training epochs, or a training time limit, through the UI.

1,2: perfect. 3: maybe let's postpone. easier for us to think of training of each task going on forever, and users can come and go (so start/stop basically just means join/leave)


Create 2018-02-08.md

Adds meeting minutes for the 2018-02-08 CommComm meeting.
Closes #238

Going to merge this 👍


fix(deps): update terraform ibm to latest for the deployable architecture solution

This PR contains the following updates:

Package
Type
Update
Change

ibm (source)
required_provider
patch
1.70.0 -> 1.70.1

Release Notes

IBM-Cloud/terraform-provider-ibm (ibm)
v1.70.1
Compare Source
BugFixes

Fix test bucket endpoints (5695)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

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

GitHub release
v8.13.5

Your semantic-release bot :package::rocket:


Nits for adapter

Hey there! Awesome work!
Just a quick thing, you don't need to inherit file adapter, we do prototype comparison to make sure the adapter implements the right methods.
I'm not familiar with createBlockBlobFromText but the data you receive is expected to be a Buffer.
On the same note getFileData 's promise is supposedly resolving with a Buffer.

Thanks for the tips! createBlockBlobFromText accepts buffers (I admit, the naming isn't exactly transparent...), but I made sure that we return a buffer, too. Updated in 0.2.3 :bow:.

OK wonderful!


Update to latest ember, ember-cli and ember-data

Also updates tests using ember-watson.
The addon is still tested in release, beta and canary, but it's not tested in Ember 2.0.2 anymore. I can bring it back if you have any special reason to test that version.

Thanks :-)


React kitchen sink example for discussion

Greetings,
I have been experimenting with vite and vitest and as a big fan of Storybook I thought I'd build an example app that shows how these tools can work together. The aim is to share some code for the benefit of others, but much of the config I have used has been cherry picked from various issues logged here in the past.
While everything does work there were some quirks with storybook-builder-vite and I thought it might be worth a discussion (I don't see discussions enabled for this repo hence I'm opening an issue instead, I hope that's OK.)
The repo is here: https://github.com/robcaldecott/vite-kitchen-sink
On the Storybook side it uses msw for API mocking, an old react-intl plugin for switching between English/Finnish, interactions (with the debug panel enabled), a dark-mode addon that I've wired up to MUI5 and support for the formatjs babel plugin that I use for automatic message IDs.
Thoughts on the developer experience:

Changing the babel config feels pretty brittle and although it works I am looking forward to a more elegant way to tweak the Vite react config.
Adding a bunch of packages to config.optimizeDeps seems like another area where the DX could improve. Some of these (like @storybook/components) are not top-level deps so would not be obvious unless you came looking for an issue where this has done before. It would be great if there was a way to automate some of this config.
My first choice i18n addon was actually storybook-react-intl as it gives you a new toolbar icon for language switching and is actively maintained. Sadly it does not co-exist with vite because of issues with a dependency (storybook-i18n) but it's possible this could be fixed with more config but I was out of my depth here.
Issues with jest-mock meant a preview-head.html was required plus some more tweaks. The Storybook interactions feature has so much potential it would be great if we can get this working with vite out of the box. I wonder if there is mileage in a future interaction addon including vitest support out of the box.
There is still a bit of flicker when switching stories but it's pretty much the same experience as it is with webpack and create-react-app but is much, much faster.

In my opinion this project is an amazing addition to the vite and Storybook ecosystem: the sample repo I created even uses stories for the unit tests and I had no problem getting this to work despite using vitest which is not even advertised as being production ready.
I'd like to thank the maintainers of this repo for putting this builder together.  👏

Nice work @robcaldecott :) Thanks for the great feedback.
For config.optimizeDeps I would recommend using https://github.com/antfu/vite-plugin-optimize-persist. I use it in my projects and it works great :)


New Fixes

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


Update browserslist dependency version

"browserslist" was updated to 4.16.8 to fix the "Regular Expression
Denial of Service" vulnerability. Find more info
@https://www.npmjs.com/advisories/1747

What kind of change does this PR introduce?

Did you add tests for your changes?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

You can do it locally, we use ^


batt_entities invalid configuration

I have the latest update installed

But can't use batt_entities feature

My configuration:
  - platform: mitemp_bt
    rounding: True
    decimals: 1
    period: 60
    log_spikes: False
    use_median: False
    active_scan: False
    hci_interface: 0
    batt_entities: True

How can I fix that?

Hmmm... Have you tried reinstalling the component?

I found, const.py file was not updated by HACS

Oh, actually files were not updated by HACS, despite it shows that sensor was updated...

HACS has recently been updated with download fixes. Do you have the latest?

I do

I can suggest trying to remove the component (+ comment out its configuration), stop HA, start HA, install the component 🤷♂

Thanks, I managed to install update by changing the files with the editor basically. Everything works

Could be related to the change in your configuration.yaml by adding the new option, batt_entities: True, while the old version is still loaded in home assistant. First a restart is required before you can add the new config variable.


IllegalStateException in closure-compiler-linter-1.0-SNAPSHOT.jar

I successfully compiled linter with mvn -DskipTests -pl externs\pom.xml,pom-linter.xml, but can't launch it the right way:
closure-compiler$ java -jar target/closure-compiler-linter-1.0-SNAPSHOT.jar
Exception in thread "main" java.lang.IllegalStateException: Cannot set value to a final field 'com.google.javascript.jscomp.Linter.files'.
        at org.kohsuke.args4j.spi.Setters.create(Setters.java:32)
        at org.kohsuke.args4j.ClassParser.parse(ClassParser.java:38)
        at org.kohsuke.args4j.CmdLineParser.<init>(CmdLineParser.java:96)
        at org.kohsuke.args4j.CmdLineParser.<init>(CmdLineParser.java:71)
        at com.google.javascript.jscomp.Linter.run(Linter.java:52)
        at com.google.javascript.jscomp.Linter.main(Linter.java:48)

Tried in cmd.exe and bash for windows.
Maybe there is a way to run linter from closure-compiler? --checks-only seems not enough.

You can run the compiler with --jscomp_warning=lintChecks --checks_only and that will be reasonably close to what the linter does. But we should fix this. I suspect what happened is that our internal analysis tools "helpfully" pointed out that a field was never assigned and so could be made final, without realizing that args4j sets it through reflection.


Memory consumption

Actually, your parser consume a lot of memory in Xcode 9 beta (both 1 and 2) with Swift 4. The problem is with appending string during parsing XML in Parser.swift. Appending founded string into currentValue allocate new memory, but still hold old one. I'm not sure if it's problem of implementation of calling delegate from XMLParser or they changed something and transfer responsibility of management into implementation of the delegate. Solution is fold processing of func parser(_ parser: XMLParser, foundCharacters string: String) into autoreleasepool closure.
Does have anybody else that problem with Swift 4?

Does it only happen in Debug, or in Release build with optimisations on?

I do. XCode 9 final. In both debug and release mode.
I have ~1MB binary as B64. During parsing over 12GB was allocated. Then I killed the app.

autoreleasepool is ok for small XML. For my part this solution not work for large XML. (NSXMLParserErrorDomain erreur : 1)

For Swift 3.2 and Xcode Version 9.1 (9B55) the problem is also present.

I managed to fix issue by changing method body:
@objc func parser(_ parser: XMLParser, foundCharacters string: String) {
    currentValue.append(string.trimmingCharacters(in: CharacterSet.whitespacesAndNewlines))
    currentElement?.value = currentValue.isEmpty ? nil : currentValue
}

Hi, thanks @milanhorvatovic for reporting this. I'm not really sure why this happens, but I've merged #131 and I'd appreciate if someone can confirm is it fixed now with the latest version 4.2.0?
Also, maybe someone can make PR with a unit test using large sample XML which is causing this bug, then everybody can look for a fix?


Searching for "decoded" from landing keeps me on landing after click

https://canary.kodadot.xyz/

https://github.com/kodadot/nft-gallery/assets/5887929/f00f7ee0-6766-46d2-bb86-a1e76637ace8

apparently @vikiival forgot to update polysearch
UPDATE collections SET chain = 'ahk' WHERE chain = 'stmn';
UPDATE collections SET id = chain || '-' || collection_id WHERE chain = 'ahk';

now should be ok


Provide input prefix completer function

Currently, the Completer function which can be added to command only receives fully completed words as arguments.
For finer-granular control it would be awesome to add also the prefix added so far to the arguments list for the completer function. This would allow completions which depend on the current input, like ls /usr/loc + TAB does complete only on directories found in /usr starting with loc (so the completer function would need to know that the user typed in /usr/loc to perform this kind of dynamic completion.
For backwards compatibility it's probably out of scope to change the signature of https://github.com/abiosoft/ishell/blob/8b8aa74a85127192b469453a72fd7971b47481fb/command.go#L29, so I suggest to a new member CompleterWithPrefix.

https://github.com/abiosoft/ishell/pull/119


'i' symbol description on cards in Dashboard page

Information is not displaying when we mouse hover on info button

Description has given


taphold, long press

Hi,
I don't know how to contact you, so I am writing here.
Is there possibility  to provide same functionality for something like "taphold"(jquery), "press"(hammer.js) on event.
Thanks

Hi Drago,
I feel it's beyond the scope of this library but if you look at the code I'm pretty sure it would be easy for you to adapt it for "taphold" and "press".
Cheers,
Michael

thanks, honestly I already tried modify your code, but i have problems at the begging. I got error after page load:
TypeError: View is undefined
View = View.class || View;
can you help  me to solve this problem?


Attempted to divide by zero error when creating a sprite with only one texture

This error seems to only happens when i make a sprite with only one frame
How to reproduce:

Add an image to the project
Right click on it, Create 2D sprite then save
Then try to drag and drop onto the scene
Error when dropping [SpriteTools.SpriteDropObject](arg:0) - Attempted to divide by zero.

This should be fixed now. Let me know if you have any other issues with it :)


Issue with loading third party library in Angular 6

Issue with loading third party library which is dependence on another third party library.
In my case, i'm using bootstrap(v3.3.7) which is required JQuery. I included these libraries as per angular suggested way and successfully compile the application, if run it in browser i got a console error.
Versions
"dependencies": {
    "@angular/animations": "^6.0.0",
    "@angular/common": "^6.0.0",
    "@angular/compiler": "^6.0.0",
    "@angular/core": "^6.0.0",
    "@angular/forms": "^6.0.0",
    "@angular/http": "^6.0.0",
    "@angular/platform-browser": "^6.0.0",
    "@angular/platform-browser-dynamic": "^6.0.0",
    "@angular/router": "^6.0.0",
    "@types/bootstrap": "^3.3.33",
    "@types/jquery": "^3.3.1",
    "bootstrap": "^3.3.7",
    "core-js": "^2.5.4",
    "jquery": "3.3.1",
    "malihu-custom-scrollbar-plugin": "3.1.5",
    "rxjs": "^6.0.0",
    "zone.js": "^0.8.26"
  }

Sample code
angular.json

app.component.ts

Console Error

did u add bootstrap js? also u can use CDN for them

You are setting jQuery as a global, however you are importing using a direct import. This will result in having jQuery twice in your application.
See https://github.com/angular/angular-cli/wiki/stories-global-scripts#using-global-libraries-inside-your-app