Set up bulk file storage

One of the identified user needs is to be able to compare historical data to current data. A simple way to get this started is to:

Set up a backup server with lots of disk space
Add a function to the cli tool to create a gzipped tarball of the raw files and the sqlite metadata database
Establish a procedure for copying the file to the backup server, and conversely restoring it to the database for analysis.

This isn't needed for that user need - the program already lets you download the same source at different time points, have the raw files for both there at the same time, and have them in the raw tables at the same time in the DB.

@odscjames as I understand it, there's only room on the current server for one copy of all of the data - I'm currently running a download of all the Ukraine data for @duncandewhurst and it's already at 46G for just the raw files.

This is now set up and the script will be in place by the end of the current sprint. We've certainly gone longer than I was anticipating without needing it!


Can not find the table while the sharding key not transmit

the table user_buy  is shared into 16 tables as user_buy_{0..15}， sharding key is uid， the version:4.0.0-RC1  can not word while uid not transmit.
UserBuy db = new UserBuy();
db.setBuyOrderId(4902l);
db.setUid(11587537l); //transmit uid has no problem， but don't work while not transmit
List userBuys = userBuyDao.findList(db);
problem:
com.duowan.udb.exception.DataAccessException: get SQLException error
--- The error occurred while applying a parameter map.
--- Check the UserBuy.findList-InlineParameterMap.
--- Check the statement (query failed).
--- Cause: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Table 'hq_goods.user_buy' doesn't exist
but it works at the version 1.4.1 when use in the groupId com.dangdang
version：·

com.dangdang
sharding-jdbc-config-spring
1.4.1

com.dangdang
sharding-jdbc-config-common
1.4.1

commons-dbcp
commons-dbcp
1.4

Thank you so much!   I have solve the proble by the follow way:
it must config the attribute actual-data-nodes when do not transmit the sharding key

Can you try it with the last version (4.0.0-RC3-SNAPSHOT).

Oh, please try 4.0.0-RC2.

The version 1.x is too old, please update version 4.x


WIP feat/logs: electron-log impl

r? @krishnaIndia
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Media Library + Azure: illegal characters causing various problems

Windows (NTFS) prohibits the use of the following characters in file and folder names: ", *, :, <, >, ?, |, /, \.
Due to the differences in how files are stored in Azure Blob Storage (including the Development Storage of the Storage Emulator) most of these are actually accepted when renaming files in the Media Library when using Microsoft Azure Media Storage:

", <, >, |: Server error when calling the API to query media items ("Illegal characters in path."). For some of the illegal characters, the file name is changed up until the illegal character, but the rest of it is cut off. This error renders the Media Library unusable when opening a folder that contains an affected file.
*, :, ?: The file can't be downloaded to the local file system without changing its name (also true for the previous set of characters).
/: Creates a folder for each slash in the file name (and displayed as such in the Media Library navigation) and the file is at the bottom of the folder structure created by those slashes, but the file is displayed in the correct folder (where it was actually uploaded) in the Media Library (with a file name that has the folder structure defined by the slashes cut off).
\: This is the only illegal character that is handled correctly: A validation error is displayed when saving the file.

Suggestion: Trigger a validation error for all illegal characters.

Besides having added a bunch of characters, in #7577 I added a validation to the FileSystemStorageProvider to further prevent invalid characters in the path. Something similar could be done for Azure. It should be straightforward, but I am not volunteering because we don't use Azure storage and I would have no way to test or validate this.


Removing a thread from awaiting a response should be safer

When a thread's already been removed or not in it for some reason it should just silently pass.
Traceback (most recent call last):
File "/home/bots/discord_bots/GPT3Discord/models/index_model.py", line 699, in index_webpage
    raise ValueError(
ValueError: Invalid URL or could not connect to the provided URL.
The summary is None
Ignoring exception in on_message
Traceback (most recent call last):
  File "/home/bots/.local/lib/python3.9/site-packages/discord/client.py", line 378, in _run_event
    await coro(*args, **kwargs)
  File "/home/bots/discord_bots/GPT3Discord/cogs/index_service_cog.py", line 168, in on_message
    self.thread_awaiting_responses.remove(message.channel.id)
ValueError: list.remove(x): x not in list

I will see what I can do


Documentation on available input and lead times

I'm attempting to use this notebook for rollout: https://github.com/NASA-IMPACT/Prithvi-WxC/blob/develop/examples/PrithviWxC_rollout.ipynb and I want to try some different input and lead times. For example,
lead_time = 48
input_time = -12

However, most of the time I get error: "There doesn't seem to be any valid data."
Is there a good resource to find what combinations of input and lead times are available?

You may need to modify the number of MERRA2 data files you've downloaded. The data loader will look at the files you have and will only produce samples for which you have the data. For example, if you've only downloaded one day of data, then you can't use a lead time of 48 hours as there would be no way to get the target.
This also highlights something that we need to add to the dataloader: an inference flag.


add isLayoutRtl

Suggest you add isLayoutRtl to ViewUtils. isLayoutRtl is mostly used in rtl support app.

Try the version of 1.25.9.


Pretrained CRIS model

Are there any plans to host a pre-trained CRIS model on e.g. huggingface?
Looking forward to trying out the model!

+1

Yes. We will release the pretrained models ASAP.

@DerrickWang005 Any updates on this or timeline?

+1

Hii @DerrickWang005, Can you upload the pre-trained model?

Hi @DerrickWang005 any update on releasing a pre-trained model? Thanks!

+1

+1

+1

While we wait for the official pretrained model, I have trained CRIS on refcoco using the standard training procedure (default parameters, 50 epochs). The model is available here.
I should be able to keep it there for a couple of years, I think it will be enough for the authors to publish their model 😄

@loris2222 do you have similar score on benchmark datasets?

I am using the model for another purpose so I haven't saved the test performance, but you are welcome to try and report the results here. Anyways, qualitatively results are quite good.

Wow @loris2222 thank you so much. By the way, do you have any report or documentation regarding the results (qualitatively and quantitatively). It would be really nice and helpful to see the results and read more about it. Great work!

@loris2222 @sanchit88 I have some bad news(
test your model on evaluation RefCOCO data and I have this(
IoU=8.03  Pr@50: 0.17  Pr@60: 0.17  Pr@70: 0.17  Pr@80: 0.09  Pr@90: 0.09

@DerrickWang005 Should you please share pre-trained CRIS model with benchmark score(

I have found the wandb final log:
{   "time/batch": 3.7636070251464844,   "time/data": 0.017482995986938477,   "training/lr": 0.00001,   "training/loss": 0.014677105471491814,   "training/iou": 89.92655944824219,   "training/prec@50": 100,   "_timestamp":<PHONE_NUMBER>.9539435,   "_runtime": 74153.98540353775,   "_step": 33700,   "_wandb": {     "runtime": 74615   } }
Maybe there is something wrong in your configuration?

Yeah possibly, but results on training data were not published, so they still might be reasonable. By the way VOC is an entirely different dataset and it seems to be doing well as you see from the example (not cherry picked). As I said I'm using these models for something else so I don't think I will be able to run another 20h of training...

Yes. We will release the pretrained models ASAP.

Hi just want to ask if is there any update. I'm planning to do some further research with your model, could you please upload the checkpoints?

@loris2222 @MaslikovEgor @OptimusLime @willyfh @qiruiw @saurabhya
Just want to report my test result on @loris2222 's model(RefCOCO val):
2022-10-27 18:05:35 | INFO     | __main__:76 - => loading checkpoint 'exp/refcoco/CRIS_R50/best_model.pth'
2022-10-27 18:05:36 | INFO     | __main__:79 - => loaded checkpoint 'exp/refcoco/CRIS_R50/best_model.pth'
Inference:: 100%|###############################################| 3811/3811 [06:45<00:00,  9.39it/s]
2022-10-27 18:12:22 | INFO     | engine.engine:198 - => Metric Calculation <=
2022-10-27 18:12:22 | INFO     | engine.engine:211 - IoU=69.06
2022-10-27 18:12:22 | INFO     | engine.engine:213 - Pr@50: 80.93.
2022-10-27 18:12:22 | INFO     | engine.engine:213 - Pr@60: 76.44.
2022-10-27 18:12:22 | INFO     | engine.engine:213 - Pr@70: 69.83.
2022-10-27 18:12:22 | INFO     | engine.engine:213 - Pr@80: 53.51.
2022-10-27 18:12:22 | INFO     | engine.engine:213 - Pr@90: 17.09.

The result reported by the authors is 69.52, I think his mode is fine(69.06), thanks(a lot) for sharing!

Glad to be of help :)
So @MaslikovEgor did you check again to see whether there were problems in your pipeline?

Thanks for the pre-trained model @loris2222. I am still trying to better understand the functionalities and I was wondering if you could detail on how to load the weights and use the model?

Just put the linked .pth in <CRIS_root_folder>/exp/refcoco/CRIS_50 and follow the instructions from this repo's readme.

I did the following:

Clone the repo to colab;
Update <CRIS_root_folder>/config/refcoco/cris_r50.yaml to consider the .pth as clip_pretrain: /content/best_model.pth;
Run evaluation code in readme;

# e.g., Evaluation on the val-set of the RefCOCO dataset
CUDA_VISIBLE_DEVICES=0 python -u test.py \
      --config config/refcoco/cris_r50.yaml \
      --opts TEST.test_split val-test \
             TEST.test_lmdb datasets/lmdb/refcocog_g/val.lmdb

Got the error below. I do not understand the root cause..
2022-10-27 16:34:07.855 | ERROR    | main::90 - An error has been caught in function '', process 'MainProcess' (222), thread 'MainThread'<PHONE_NUMBER>88256):
Traceback (most recent call last):

File "/content/CRIS.pytorch/test.py", line 90, in
main()
└ <function main at 0x7f308a052b00>

File "/content/CRIS.pytorch/test.py", line 70, in main
model, _ = build_segmenter(args)
│               └ CfgNode({'dataset': 'refcoco', 'train_lmdb': 'datasets/lmdb/refcoco/train.lmdb', 'train_split': 'train', 'val_lmdb': 'dataset...
└ <function build_segmenter at 0x7f309690e290>
File "/content/CRIS.pytorch/model/init.py", line 33, in build_segmenter
model = CRIS(args)
│    └ CfgNode({'dataset': 'refcoco', 'train_lmdb': 'datasets/lmdb/refcoco/train.lmdb', 'train_split': 'train', 'val_lmdb': 'dataset...
└ <class 'model.segmenter.CRIS'>
File "/content/CRIS.pytorch/model/segmenter.py", line 15, in init
map_location="cpu").eval()
File "/usr/local/lib/python3.7/dist-packages/torch/jit/_serialization.py", line 162, in load
cpp_module = torch._C.import_ir_module(cu, str(f), map_location, _extra_files)
│     │  │                │       │   │             └ {}
│     │  │                │       │   └ device(type='cpu')
│     │  │                │       └ '/content/best_model.pth'
│     │  │                └ <torch.jit.CompilationUnit object at 0x7f308a068b30>
│     │  └ <built-in method import_ir_module of PyCapsule object at 0x7f3098a21a80>
│     └ <module 'torch._C' from '/usr/local/lib/python3.7/dist-packages/torch/_C.cpython-37m-x86_64-linux-gnu.so'>
└ <module 'torch' from '/usr/local/lib/python3.7/dist-packages/torch/init.py'>
RuntimeError: PytorchStreamReader failed reading zip archive: failed finding central directory

@RubenBMHMendes You have to download pre-trained CLIP first(it's not best_model.pth) from here , it's from the official CLIP repo.
Then put best_model.pth at <CRIS_root_folder>/exp/refcoco/CRIS_50 and you are all set.

@BingliangLI thanks for your answer! I downloaded the RN50 model and created a pretrained folder in <CRIS_root_folder> and added best_model.pth in the correct directory but I got the same error as before.
Does anyone have a minimum working example on this? I can also share the colab notebook.

I have found the wandb final log for my training process:
{ "time/batch": 3.7636070251464844, "time/data": 0.017482995986938477, "training/lr": 0.00001, "training/loss": 0.014677105471491814, "training/iou": 89.92655944824219, "training/prec@50": 100, "_timestamp":<PHONE_NUMBER>.9539435, "_runtime": 74153.98540353775, "_step": 33700, "_wandb": { "runtime": 74615 } }
Qualitative results on VOC
Maybe there is something wrong in your configuration?

Hi @loris2222, how did you extract the segmentation mask with a custom image using your best_model?

Yes. We will release the pretrained models ASAP.

@DerrickWang005, Any update on this?


Improve Report interface

Too much of the underlying implementation (ctypes) shows through.
Additionally, no clear mechanism for getting specific sensors (e.g. temperature sensor 5).
No mechanism to normalize units for values, e.g. Celsius for temperature.

Known temperature mappings for the Aquaero 6 XT:
0: temperature sensor 1
1: temperature sensor 2
2: temperature sensor 3
3: temperature sensor 4
4: temperature sensor 5
5: temperature sensor 6
6: temperature sensor 7
7: temperature sensor 8
8: temperature sensor 9
9: temperature sensor 10
10: temperature sensor 11
11: temperature sensor 12
12: temperature sensor 13
13: temperature sensor 14
14: temperature sensor 15
15: temperature sensor 16
16: software sensor 1
17: software sensor 2
18: software sensor 3
19: software sensor 4
20: software sensor 5
21: software sensor 6
22: software sensor 7
23: software sensor 8
24: virtual sensor 1
25: virtual sensor 2
26: virtual sensor 3
27: virtual sensor 4

42: fan amplifier 1
43: fan amplifier 2
44: fan amplifier 3
45: fan amplifier 4


v2 - get rid of OpenApiReaderRegistry

Having a static instance with concurrent dictionaries is going to at best negatively impact performance, at worst introduce race conditions.
In addition to that, the API itself it not obvious.
All the places where we call "GetReader" from, are places where we have access to the settings.
I suggest we nuke that class and make the readers a property of the settings class instead, with defaults.

I'm fine with removing the static instance, as long as we are not losing any functionality.  Using reader settings seems fine to me.


org.w3c.dom Java package collision

I keep getting "problems" about the package org.w3c.dom is accessible from more than one module after moving to Java 14 from 8.  After doing research, what I thought was the culprit was the indirect inclusion of the xml-apis dependency which I promptly exluded using the following in my build.gradle:
configurations {
   all*.exclude group: 'xml-apis', module: 'xml-apis'
 }

I verified this did indeed remove this dependency and everything compiles cleanly from the command line.  however, even after limiting modules using
--limit-module  java.base,java.desktop,java.sql,java.naming
gradle arguments, I still have a bunch of "problems" in the console related to this package collision across multiple module:
The package javax.xml.parsers is accessible from more than one module: <unnamed>, java.xml
The package org.w3c.dom is accessible from more than one module: <unnamed>, java.xml

What am I missing?  Is this an issue with vscode or do I need to do something else?  I verified the JAR which my sample app builds no longer includes any JARs which would have duplicates of this package.
I have a sample vscode project which reproduces the issue here:
https://github.com/mentaloaf/vscode-examples.git
I am using Java 14:
java version "14.0.1" 2020-04-14
Java(TM) SE Runtime Environment (build 14.0.1+7)
Java HotSpot(TM) 64-Bit Server VM (build 14.0.1+7, mixed mode, sharing)

My vscode and related system versions:
Version: 1.51.1
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-11T01:11:34.018Z (3 wks ago)
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 18.7.0

Any suggestions?
Thanks!

/extJava


Id to profile

Hello.
How to get friend profile with id? Regards.

var friend = User.GetUserFromID(id)
Helps to use the intellisense to see the various methods etc.

Yes, intellisense is king with Tweetinvi. Sorry for the documentation, I am trying my best with the time I have.
Cheers,
Linvi


feat: add toast feedback to annotation templates

SUMMARY
This PR adds toast messages as a feedback to user actions: creating an annotation template and editing annotation template.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
https://user-images.githubusercontent.com/15073128/150381907-aafb48f3-8de1-4b64-9a30-13ff370967c0.mov
TESTING INSTRUCTIONS
Create an annotation template, verify that toast message appears.
Edit an annotation template, verify that toast message appears.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC @yousoph @rusackas
https://app.shortcut.com/preset/story/35387/annotation-layer-templates-system-feedback

/testenv up

@jess-dillard / @kasiazjc if you feel like taking a look to validate from the design side, you can check the ephemeral environment link above. :)

@rusackas Looks good to me!


Rename "master" to "main" on all repositories

See https://github.com/github/renaming
Github and Git changes the default branch name from "master" to "main", recognizing that "master" is offensive to some people.
Cultural issues aside, in order to keep naming of branches consistent across all projects and avoid papercuts, I suggest we rename "master" to "main" on all repositories.

Currently it seems "master" is more commonly used though. Renaming now might cause more trouble than it's worth.

Closing since "master" is still used, except new projects.


Escape user input before creating RegExp

Un-sanitized user input causes SyntaxError when trying to create a regular expression. Example: try typing "("

Component Screenshot:

Description (short): escaped special characters before using in RegExp

:+1:

👍


yarn

⚠ IMPORTANT ⚠ Please check the following list before proceeding. If you ignore this issue template, your issue will be directly closed.

[x] Read the docs.
[ ] Make sure the code is up to date. (Some bugs have been fixed in the latest code)
[ ] This is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Describe the bug
A clear and concise description of what the bug is..
Reproduction
Please describe the steps of the problem in detail to ensure that we can restore the correct problem
System Info

Operating System:
Node version:
Package manager (npm/yarn/pnpm) and version:

yarn安装以来一直卡住

设置国内镜像源，删除yarn.lock，重新yarn install


Bump version, update changelog

Updates the version in preparation for a 0.4.0 release with the latest changes.

Coverage remained the same at 100.0% when pulling a63afb6a79d7ce8af564a94a25c3e9405c0b284e on 0.4.0-release into c923eef2883c74b9dfcd0688867d992f9418b25d on master.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 88 ms

Resolved: ClamAV performance has improved in 3f5cd2b after 5 minutes.


adnilg issues

Clicking:

shows:

Double scrollbar:

I assume you click the whole menu, but I thought maybe you should be able to click the numbers?

These links seem to open up to broken urls:

Clicking around opened:
https://smart-starfish-5.localtunnel.me/?icloud%2Fproduction%2Fdeployments%2F%3Ca+class=%22link-mleft%22+target%3D%22_blank%22+onclick%3D%27window.open(%22https%3A%2F%2Fsmart-starfish-5.localtunnel.me%2Fglinda%2Ftasks%2F56115%2Fview%2Fhealth.carnival%3Fname%3Dcondor&filterType=correct%22)%3Breturn+false%3B%27%3Eundefined%3C%2Fa%3E

Popup cut off:

Clicking:

Then trying to submit:

How do you make these verifications change?

Nothing I do seems to change them.  Is this a bug?

Another broken event handler:

These seem like links but do nothing on click:

Verifying doesn't seem to work:

I'm not sure if this is a bug or not.

Search doesn't seem to work:

This might be a UI bug, but after you click Create Requests it the button stays the same, making it more likely someone will keep clicking it:

Another minor bit of UI weirdness:

There is a lot of space between the checkbox and content.  I wouldn't have noticed if the horizontal space on my monitor wasn't so large.

Creating a freeze didn't seem to work:

Nothing appeared in the list.

This is probably a UI bug, but if you click on 10, you can't see pages to the right of it:

This is likely due to the slow internet connection, but the "refresh" icon stays visible for a long time, then goes away:

After it goes away, I can type in the input:

But this seems odd / wrong?
I think this form is broken.

Probably a UI bug, but hovering over the column, just below the column text, shows a cursor pointer.

Changing the form doesn't seem to change the content below:

Creating a deployment warns about a key:

This sort of warning usually means a bug.


No method for managing curations

Hi,
Thanks for this great tool.
I miss the method for creating and mantaining curations, especially for bad words when you need to hide all results from the bad words search criteria.
It's tedious to click hide in every document returned by the search criteria.
Has this been considered ?

Hi @niloct. We do have an API for managing curations, but unfortunately the Python client does not currently support those. However, you can still interact with these API endpoints directly: https://swiftype.com/documentation/app-search/api/curations.

I just implemented it for myself here fiddling with the client.py
On Tue, Oct 22, 2019 at 2:57 PM Jason Stoltzfus<EMAIL_ADDRESS>wrote:

Hi @niloct https://github.com/niloct. We do have an API for managing
curations, but unfortunately the Python client does not currently support
those. However, you can still interact with these API endpoints directly:
https://swiftype.com/documentation/app-search/api/curations.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/elastic/app-search-python/issues/17?email_source=notifications&email_token=AALVMJ2APNCMG4673OMRDWLQP45HNA5CNFSM4JDSUFCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB6UM7A#issuecomment-545080956,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALVMJ4CHCS7UQJS325KNH3QP45HNANCNFSM4JDSUFCA
.

--
Nilo César Teixeira
<EMAIL_ADDRESS>(55) (11) 98571-5314


Revert upgrade hook

What this PR does / why we need it:
The upgrade hook added yesterday doesn't fix anything and actually breaks the chart. Reverting.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm


[release 4.5] Bug 1926754: Update maven agent to 3.6

/bugzilla refresh

/bugzilla refresh

/lgtm

@waveywaves

this appears to be an RFE (per the linked bug), we don't generally backport features to z-streams, especially not 4.5 which is in maintenance support only.

seems odd you've changed the content w/o renaming the "maven-3.5" directories.

@bparees Maven 3.5 is EOL which leaves 4.5 with no supported Maven agents for Jenkins. Therefore we aimed to backport to 4.5 as well, despite being in maintenance mode.

not renaming the directories still seems like an odd choice but i don't want to hold you up from landing this, so i'm approving.

@bparees agree about the directories
@waveywaves any reason for not adding a new directory for maven-3.6?

@siamaksade the initial fix has been done by a new comer on the project, and it has been LGTMed too quickly while I was not here.
for 3.11 , I have created a different directory, so, we should align. However, 4.x directory should be name agent-maven  to reflect the final image name which is version agnostic.
To complete the renaming, a PR within https://github.com/openshift/ocp-build-data/blob/openshift-4.5/images/jenkins-agent-maven-35-rhel7.yml should be done and that also may have impacts on ART build process.

one of the test pipeline is failing with the following error
First time build. Skipping changelog.
[Pipeline] }
[Pipeline] // stage
[Pipeline] stage
[Pipeline] { (Build WAR)
[Pipeline] sh
+ mvn clean package -Popenshift
<EMAIL_ADDRESS>line 1: mvn: command not found
[Pipeline] }
[Pipeline] // stage

/hold

/retest

/retet

/retest

/close

/reopen

/reopen


Limit annotations to specific graphs and show metrics for all graphs

Love the zabbix plugin for Grafana.  It's absolutely amazing.  Thank you.
I'd love it if we can restrict certain annotations to specific graphs.  If I have a dashboard for a host with CPU, Memory and Network graphs, with annotations for CPU and Memory, I'd like to only see the CPU triggers on the CPU graph, Memory triggers on Memory graph, etc.
I can see that there may occasionally be a benefit to seeing all triggers on all graphs simultaneously when researching an issue historically.
Also, would it be possible to provide an option when using linked cross hairs to show metric data for points on each graph simultaneously?
Thanks!

Hi! Thanks for feedback!
Unfortunately, this is Grafana feature only. And now you can't bind annotation to specified panel. But I also want to use this feature, and I'll create relative issue in grafana.

Hi, was it this feature added?
Thanks.

Yes, it was implemented in grafana, so I can try to add this to Zabbix plugin.

Hi, do you still intend to implement this feature?
It would be a great function
Thanks for the amazing plugin!


Failed to build Android apk target due to empty option string

Description of the bug:
After upgrading to the latest android studio plugin, full apk build from IDE is broken due to empty option string.
This regression is related to https://github.com/bazelbuild/intellij/commit/3f39fcadb2082216dcd56497668adb500d819ecb#diff-60594ed699457ac04f3d9c936c335815f810037083f52c5a978dc1206ed74327R112-R115
which introduced an empty string into the build options
Command: bzl build --tool_tag=ijwb:AndroidStudio --tool_tag=version:2023.01.24 --output_groups=+android_deploy_info "" --curses=no --color=yes --progress_in_terminal_title=no --build_event_binary_file=/var/folders/04/l0dbn8vx6k3d22cwy8c9r0200000gn/T/intellij-bep-7494bc2c-849e-4270-bf43-c592044deeaa --nobuild_event_binary_file_path_conversion -- //apps/apk:apk

What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
build apk target from IDE
Which Intellij IDE are you using? Please provide the specific version.
Android Studio 2021.3
What programming languages and tools are you using? Please provide specific versions.
No response
What Bazel plugin version are you using?
v2023.01.24
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

A simple (and ugly) workaround to this is creating a bazel.sh file like this: https://github.com/bazelbuild/intellij/issues/1922#issuecomment-1339097708
And then setting the Bazel binary location in Android Studio.

Close this as it's fixed in https://github.com/bazelbuild/intellij/pull/4527


move TODO item to correct heading

I think this TODO was misplaced, under integers instead of pointers. This was confusing to me for a while!

Ah you are probably right, thanks!


fixed backgound change bug issue-170

fixed bug:
#170
#173

hey can you please give this PR a hacktober fest acctepted label


AI: Crystal AI's and Rain's link seems to be invalid.

Any chance you can track down the new URLs and public a pull request? If not I can always try to fix it later on.

Crystal AI's link https://github.com/igiagkiozis/CrystalAI

Closing this as I just merged a PR that should fix these links. Unfortunately I had to remove Rain because it seems to have been deprecated by its creator.


[5.7][stdlib] add explanations to _debugPrecondition calls

Add explanatory messages to _debugPrecondition calls in withMemoryRebound.
This is a cherry-pick of https://github.com/apple/swift/pull/60634.
Resolves rdar://98826137.

@swift-ci please test


Add CMake Interface Library

I am very interested in using your library. I would like to use FetchContent and have access to your library through add_subdirectory. However, your library does not create a convenience target to use with target_link_libraries. While it is little effort for me to add the necessary include directory myself, I would much rather just "link" against your header only library like so:
target_link_libraries(MyTarget PRIVATE Boost::ut)

I have included an import prefix to mirror Boost's component linking behavior.
Defining an interface library through CMake with the appropriate include directories will make this possible.
The changes might look like the following.
add_library(ut INTERFACE)
add_library(Boost::ut ALIAS ut)
target_include_directories(ut INTERFACE $<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>)

This code also paves the way for the interface target to be exported and included with find_package.

Thanks, @athrun22, it should be fixed now, let me know if it works for you :thinking:

@krzysztof-jusiak, thanks! The interface library works perfectly 😃😃😃


Crestle pricing changed

Crestle GPU type changed. It is no longer Tesla K80 as described here. As mentioned in the FAQ, now they switched to NVIDIA Tesla P4 and accordingly the pricing changed to 2.40 $/h (which btw seems to me too expensive for a beginner).

Thanks for raising this! Any chance you might be able to provide a PR fixing it?

Hi all,
piggybacking on this topic, I am not sure if crestle would continue to support individual projects, but rather shifts its focus on offering solutions only for companies. At least this is how I understand it:
Link to blogpost
In particular "From now through the end of August you will continue to have access to your instances with support. During this time, make sure to download your data, models and complete your current work. As of August 31, 2019 the service and your data will no longer be accessible on the platform."
Do you also understand it as I do?  I was planning to actually use crestle.ai and I had started a test project...

Thanks I've removed Crestle from our docs now.


Allow/fix aggregation includedBuilds

What is your use-case and why do you need this feature?
kover integrates with Gradle's dependency resolution system by exposing a kover configuration to aggregate projects
As includedBuilds by convention exposes a dependency coordinate (implicit dependencySubstitution), declaring it as a kover dependency is expected to aggregate the child build into the main report.
dependencies {
    kover(":build-logic")
}

This setup does not break the build, the correct koverArtifact[TOTAL] variant is resolved for the included build, but the report does not aggregate it.

https://scans.gradle.com/s/2nb3nedv4ntsm/dependencies?focusedDependency=WzAsMCwyLFswLDAsWzJdXV0&toggled=W1swXSxbMCwwXV0
Running ./gradlew :koverHtmlReport produces a report only containing main build modules:

While running ./gradlew :build-logic:koverHtmlReport, is correctly computing the build-logic modules (some now not aggregated in the root one):

Describe the solution you'd like
Included Builds should be transparently resolved as any other kover dependency and included in the aggregated report. Variants are there, it's seems to a matter of the internal plugin resolution strategy.

[!IMPORTANT]
I provided a sample project at https://github.com/gmazzo/kover-demo-aggregation-issue/?tab=readme-ov-file#includedbuilds-not-aggregating

Unfortunately, Composite Builds are not supported at the moment, there are a number of technical limitations why it does not work in its current form.

A task has already been opened to design support for such types of builds.

Closed as a duplicate of https://github.com/Kotlin/kotlinx-kover/issues/487


Allow ApplePay's onPaymentAuthorized event to complete with merchant provided status and customized errors.

The OnPaymentAuthorized event on ApplePay can complete with the ApplePayAuthorizationResult object that allows custom error messages targeted for specific fields using ApplePayError objects.
Currently Adyen is only passing positive or negative status towards ApplePay without any way of customizing negative outcomes.
Summary

Fixed issue:  This PR passes along the response from onPaymentAuthorized promise towards the ApplePaysession.completePayment

Is there anything I can modify to address the typescript type error ? I am not sure where it would be

Is there anything I can modify to address the typescript type error ? I am not sure where it would be

https://github.com/Adyen/adyen-web/runs/5601030699?check_suite_focus=true
response is of type unknown which is not valid to be passed to the completePayment function. So the type needs to be explicitly set. Next to it, the change needs to be backwards compatible

Going to close this PR as the proposed changes isn't backwards compatible. Feel free to reach out if you find an alternative solution to that. We can always re-open this PR.
Regardless of this, I added to our backlog this improvement 👍


Wrapper element gets wrong height if content changes when sticky

The height assigned to sticky-element-container when the element becomes sticky appears to be calculated before isSticky in the yielded hash becomes true, which can result in either a gap between the sticky element and subsequent content (if the element gets smaller when sticky) or the sticky element covering up some of the subsequent content (if the element gets larger when sticky).
In order to test this / see it in action, you can add the following extra columns to the demo app:

    {{#sticky-element class="sticky" as |sticky|}}
      {{#if sticky.isSticky}}

      {{else}}

      {{/if}}

small, not sticky to bottom, off view, shrinks when sticky

        {{sticky-debug sticky}}

    {{/sticky-element}}

This bit isn't sticky.

    {{#sticky-element class="sticky" as |sticky|}}
      {{#if sticky.isSticky}}

      {{else}}

      {{/if}}

small, not sticky to bottom, off view, grows when sticky

        {{sticky-debug sticky}}

    {{/sticky-element}}

This bit isn't sticky.

When you scroll far enough for the elements to stick to the top, the first column will have a gap between the sticky element and the "This bit isn't sticky" text, whereas the "isn't sticky" text in the second column will disappear behind the sticky element.

@pauln I tried your example, and I see what you are describing. However I am not sure if we should consider that "wrong". In general I think I would expect the behavior to be as it is. Imagine you have a navbar that you want to stick to the top when scrolling down, you wouldn't want the following content to reflow and "jump", rather it should be unaffected and continue to scroll down smoothly, right!?
What's your use case where you would expect a different behavior?

 There's a fixed header that, when sticky, collapses the logos in order to make more room for the table.

Wit the current implementation, the moment the element get stickied a gap between the header and the table is introduced.


Use cache as an external Redis store

Description
Instead of using in-mem cache we should use an external Redis store.
Screenshots
No response
Additional information
No response

Done https://github.com/origranot/reduced.to/pull/353


Update README.md

Updated the readme file for the todo-mvp-loaders branch with the following changes:

Consistent terminology and style.
Added a new 'What you need' section to list relevant pre-requisite topics.
Extensive links to other readme files in this project, as well as external reference documentation.
New heading and structure.
Updated formatting.
Fixed various grammatical issues.

Great job, thanks!


Base exception, fixes #207

Why as interface? You mean like a marker interface? What is the difference with an abstract class in thise case?

That way we can use SPL exceptions as well and extend whichever we want. For example UnresolvableCurrencyPairException should be invalid argument:
UnresolvableCurrencyPairException extends \InvalidArgumentException implements Exception

:+1:

@sagikazarmark Good idea. You do not see that many marker interfaces in PHP.

https://github.com/php-http/httplug/blob/master/src/Exception.php
https://github.com/lstrojny/fxmlrpc/blob/master/src/fXmlRpc/Exception/ExceptionInterface.php
https://github.com/FriendsOfSymfony/FOSHttpCache/blob/master/src/Exception/HttpCacheExceptionInterface.php
There are a few :wink:

Can you squash commits?

Merge it in?


cascading dialog windows

Brother Volkan,
Is there a way to open each new popup dialog slightly offset both vertically and horizontally from the previous popup dialog window.  This will give user an idea/context of all the previous windows. Assuming all windows will be styled to have fixed height and width
Thanks

See here https://github.com/volkanceylan/Serenity/issues/239

If only could people thank and close issue. Thanks @Estrusco

Brother Volkan,
I am truly very sorry.  It is truly lazyness on my part.  I should have as you mentioned above thanked and closed the issue.  Thanks Estrusco. Too late to close as its already closed but for future I promise to improve myself with regards.


operatorPosition=maintain not respected in conditional type

Describe the bug
dprint-plugin-typescript version: main
Input Code
export function getIterator<I extends readonly any[] | ReadonlySet<any> | ReadonlyESMap<any, any> | undefined>(iterable: I): Iterator<
    I extends ReadonlyESMap<infer K, infer V> ? [K, V] :
    I extends ReadonlySet<infer T> ? T :
    I extends readonly (infer T)[] ? T :
    I extends undefined ? undefined :
    never>;

Expected Output
export function getIterator<I extends readonly any[] | ReadonlySet<any> | ReadonlyESMap<any, any> | undefined>(iterable: I): Iterator<
    I extends ReadonlyESMap<infer K, infer V> ? [K, V] :
    I extends ReadonlySet<infer T> ? T :
    I extends readonly (infer T)[] ? T :
    I extends undefined ? undefined :
    never
>;

(or something)
Actual Output
export function getIterator<I extends readonly any[] | ReadonlySet<any> | ReadonlyESMap<any, any> | undefined>(iterable: I): Iterator<
    I extends ReadonlyESMap<infer K, infer V> ? [K, V]
        : I extends ReadonlySet<infer T> ? T
        : I extends readonly (infer T)[] ? T
        : I extends undefined ? undefined
        : never
>;

Note how the operators are on the right now.
From: https://github.com/microsoft/TypeScript/compare/main...jakebailey:TypeScript:dprint#diff-8eee8edc4cdf82aaa3b0c0ba3c8c74fa80ac6d5214540a460c49fb2ee1031b25

I goofed up the indention in the "expected" snippet; it's the operators that matter.

It seems actually that there is no conditional type operator position config at the moment. I will add one probably tomorrow.

Thanks!


chore(release): 2.17.0-rc.2

Version bump in package.json and package-lock.json for release 2.17.0-rc.2

:tada: This PR is included in version 2.17.0-rc.3 :tada:
The release is available on:

npm package (@pre/rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


image_types_resin: Optimize the partition alignment size for SDHC

It is very common for most SDHC cards to have an erase block size
of 4MiB.  Therefore it is optimal to align all partitions to a
8192 sectors. The 4MiB sizes are actually recommended by the
SD card specifications.
Signed-off-by: Jon Nettleton<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Hi @linux4kix. There is a confusion here. We are already using 4M as alignment size. You can see later in the code that we multiply that by 1024.

@linux4kix can we close this PR?

yes feel free.
On Fri, Jun 16, 2017 at 7:36 PM, Andrei Gherzan<EMAIL_ADDRESS>wrote:

@linux4kix https://github.com/linux4kix can we close this PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/resin-os/meta-resin/pull/651#issuecomment-309073760,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGPpaT0zV3VhD64o_1cmu9yK2u2JmS6ks5sEq8AgaJpZM4NEFVX
.


Failing unit tests.

test/unittests is failing for all jobs because of the lines:
https://github.com/IQTLabs/AISonobuoy/blob/main/.github/workflows/test.yml#L20-L24
the file being referenced was removed as part of #1040
The subsequent block starting at https://github.com/IQTLabs/AISonobuoy/blob/main/.github/workflows/test.yml#L25 is also dependent on PiBuoyV2 and therefore seems likely to fail.

The workflow contained in https://github.com/IQTLabs/AISonobuoy/blob/main/.github/workflows/docker.yml is also not functioning at the moment because no directories contain a dockerfile and should be removed.


Add capture_error method to Python base SDK

Related issue https://linear.app/serverless/issue/SC-561/report-detailed-error-information-in-case-of-error-resolution
Description

Add serverlessSdk.capture_error method
Add documentation for the new method

Testing done
Unit tested

@medikoo ready for review please 🙏

Looks very good 👍
Let's follow up with the release, and update to the AWS Lambda SDK and release over there.
Great work @selcukcihan 💯

Thanks @medikoo Release PR => https://github.com/serverless/console/pull/522


Fix double dispose in BlobBatch

Make BlobBatch resilient to multiple disposes

/azp run net - storage - tests

We should update the Blob Batch changelog - https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/storage/Azure.Storage.Blobs.Batch/CHANGELOG.md
Let's hold off on merging this until after the GA release we are doing today.

API change check
API changes are not detected in this pull request.

The build is failing for an unrelated reason.

CI is fixed in main

/azp run net - storage - ci


nps-utils doclets not rendering

I just added doclets to my new module nps-utils here and as you can see, it's able to render the Overview (the README), but it's not rendering the API (based on the code's JSDoc in the src direcotry). Any tips are appreciated!

Could you remove the packageJson from .doclets.yml?

That didn't work 😕


draft docs reorg, minor edits

Here's the big first PR -- mostly reorg, some content tweaks, and some TODOs for outstanding content that we've discussed.

Thanks!

lgtm


Fix typo leanable to learnable.

Description
Fix typo in documents
Motivation and Context
#746
Typo of leanable
How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[ ] Fix bugs
[ ] Add new feature
[x] Update documentation

Thanks so much!

No problem! I will be more active after one or two months. Still have some techincal debts in my own projects. Look forward to contribute.


NTTabs errors if window "zoomed" with ctrl-w o

The shortcut ctrl-w o maximizes a pane :h CTRL-W_o. It can be hit again to undo the minimization. If nerdtreetabs is toggled during the zoom in time, it gets in a bad way. It will no longer be open when you go to any other tab (even after zooming out). If you switch to another tab after opening it in zoomed mode, this error appears:
Error detected while processing function <SNR>76_TabEnterHandler..<SNR>76_RestoreNERDTreeViewIfPossible:
line   16:
E16: Invalid range: 2wincmd w
Error detected while processing function <SNR>76_TabEnterHandler..<SNR>76_RestoreFocus:
line    2:
E16: Invalid range: 2wincmd w

It doesn't seem possible to get back to the regular nerdtree setup until vim is restarted(!).

I get it all the time as well, but during tab cycling. See issue linked above.

It happens for me when I cycle through tabs when NERDTree tab is open as by default. Any solutions for this.

I get a similar error:
Error detected while processing function <SNR>55_TabEnterHandler[16]..<SNR>55_RestoreNERDTreeViewIfPossible:
line   20:
E16: Invalid range: 2wincmd w
Press ENTER or type command to continue

I don't know how to trigger/reproduce the error, but it happens usually after codding a few hours.
After the error is initially triggered you get the error message above every time when I change the tab with NerdTree open.
@jistr do you have any idea how to fix this?


ERROR No Broadlink device found

Hello,
I am trying to use it with "lookup" device type (because I can't set a fixed IP for my broadlink on my router) but I am getting the error: ERROR No Broadlink device found
Here's my custom.conf
# Type of device. Valid options are 'lookup', 'multiple_lookup', 'rm', 'rm4', 'sp1', 'sp2', 'a1', 'mp1', 'dooya', 'bg1'
device_type = 'lookup' # use lookup

# lookup parameters
lookup_timeout = 20
local_address = '<IP_ADDRESS>'

# parameters for direct connection
#device_host = ''
#device_mac  = ''

# MQTT connection parameters
mqtt_broker = 'localhost'       # default: 'localhost'
mqtt_port = 1883                # default: 1883
mqtt_clientid = 'broadlink'
mqtt_username = 'XXXXXXX'
mqtt_password = 'YYYYYYY'
mqtt_topic_prefix = 'broadlink/'
mqtt_multiple_subprefix_format = '{type}_{mac_nic}/' # use only with 'multiple_lookup' device type (allowed parameters are {type}, {host}, {mac}, {mac_nic})
#mqtt_birth_topic = 'clients/broadlink'
#mqtt_birth_payload = 'Hello!'
mqtt_will_topic = 'clients/broadlink'
mqtt_will_payload = 'Adios!'

When I log into my routers config page, get the IP address, set it on "device_host" along with "device_mac" everything works OK.
But I can't rely on that because every now and then my router changes the broadlink's IP.
It is possible to use broadlink-mqtt without static IP, right?
What am I doing wrong??
Thanks.

Ok, I found out.
Actually my raspberry pi was connected to a VPN I forgot about.
When trying to "lookup" for the broadlink it was probably using the network interface from the VPN instead of the "regular" interface on the same network as the broadlink.
After disabling the VPN interface it was working again.
Now I need to make it work with the VPN interface enabled. Any ideas?
Maybe I could set an IPtables rule that when connecting on specific port it has to use the "regular" interface (instead of the VPN)...
Anyone knows what port broadlink-mqtt uses to both "lookup" and to connect to broadlink devices?
Thanks.

If someone stumble upon this on the future: I figured out that my clients VPN connection config file was explicitly forcing any traffic  out from the VPN interface (AllowedIPs = <IP_ADDRESS>/0).
After changing that to the IP of the VPN (<IP_ADDRESS>/24 or whatever IP you set to your VPN), everything seems to be working now.

There is local_address parameter in the config
Please set there IP of the computer (not Broadlink device) provided by the router


Support for gdsl files (intellisense addition)

Some projects which support groovy additions have gdsl files for the additional linting, and custom objects.
For example Artifactory User Plugins:
https://jfrog.com/help/r/jfrog-integrations-documentation/the-artifactory-public-api-papi
https://github.com/jfrog/artifactory-user-plugins/blob/master/ArtifactoryUserPlugins.gdsl
Is it possible to include this in the Plugin, so I can utilize the additional intellisense?
Or is this maybe already possible ? (I did not find anything with a quick search, so I have no Idea.)
Thanks

@Bloodiko we could handle .gdsl files for formatting and linting, but not for intellisense, because VsCode Groovy Lint does not cover that area
Are you inteteste by .gdsl management anyway ? :)

Well linting is also a good stepup.
But don't feel pressured.
If there's more demand for it, you may revisit this.
It's not like this is a breaking issue or smth 🙂

I have switched to groovy-guru (dont-shave-the-yak) for intellisense.
This helps a lot.
So this is not needed for me anymore, since it also supports the custom jars for intellisense, basically rendering the gdsl file unnecessary.


Missing ;

https://github.com/flokosiol/kirby-focus/blob/master/fields/focus/assets/css/focus.css#L2

Damn. Will fix it soon. Thanks for reporting.

I fixed it on the development branch and added a config.php option to go fullwidth again:
c::set('focus.field.fullwidth', true);

Would be awesome, if you could test it before I merge it to the master branch, @SimonWpt.


Dont bind  in quickfix window.

plugin seems to be binding <CR> to :<C-U>call multiple_cursors#new("v"). This causes every plugin to fail which uses a quickfix window.

Closing due to inactivity. Feel free to submit a PR to fix this

The promise still remain. Could you resolve it? @faceleg

If someone submits a PR then we'll be able to resolve it


Edge error on selection when defaultEdges are connected to Redux store

Describe the Bug
If the user pass to DefaultEdges prop a variable containing a portion of store state of Redux and an edge selection operation is performed, Cannot add property selected, object is not extensible error is shown. There are some sort of shallow copy of value passed to DefaultEdges that causes an immutability error.
This doesn't happen with DefaultNodes and with zustand instead Redux as state manager.
Your Example Website or App
https://codesandbox.io/s/sandpack-project-forked-c03e86?file=/App.js
Steps to Reproduce the Bug or Issue
Click on the edge or select multiple nodes with "Shift" selection
Expected behavior
No errors should be displayed, or not allow the user to pass a direct reference of Redux state as prop.
Screenshots or Videos

Platform

OS: macOS
Browser: Chrome
Version: v10
State manager: Redux (v7.2.6) + Redux Toolkit (v.1.8.0)

Additional context
No response

Hi @alessandropisu! I think this might be a problem with using ImmerJS in Redux Toolkit. Internally, we are not cloning the edge objects, so it might be that you are passing immutable ImmerJS objects into React Flow. This might not be a problem with nodes as we are parsing them into a slightly different data structure internally. Does this help?

Hello @chrtze, thanks for your answer, the biggest doubt was about the error only during edges selection and not on nodes (I forgot they have different structures internally in ReactFlow).
For those who have the same problem and are reading: just deep clone your edges array before pass them to DefaultEdges prop :)

That's a good solution for now @alessandropisu . Thanks for sharing :)

Guys I just stumbled upon the same issue and while deep-copying the edges fixes the issue itself I have another weird behavior - The first time when I try to drag a multi-selected group my selection is automatically unselected. Then after the second time it works.
I'm using Recoil, controlled RF props (edges and nodes), and then right before I render RF I have something like this:
  // Fix multi-selection of edges - https://github.com/wbkd/react-flow/issues/1993
  const edges = useMemo(() => JSON.parse(JSON.stringify(diagram.edges)), [diagram.edges]);

  return <ReactFlow nodes={diagram.nodes} edges={edges} ... />

Here's a video preview but if it's not obvious I can also try to create a Codepen repro:
https://user-images.githubusercontent.com/2777825/173307516-57389899-0532-47ac-ba46-2be0bfdc9aa4.mov
Any ideas @moklick ?

@nkalinov did you ever solve this? do you have any example using React Flow with Recoil?

This error should be fixed in the latest version (I think that we fixed it in v10.3.0)


Prevent ?> from being parsed by comments

@maxbrunsfeld I'm not really sure how to fix the failing test. I think it has to do with comments ending with ?\n eating the following line. Is the regex eating the newline?

@joshvera What happens if you add \n to things that can't be matched following a ?, and make it end with an optional ??
seq(choice('//', '#'), repeat(/[^?\n]|\?[^>\n]/), optional('?')),

@joshvera Sorry, I should have checked more thoroughly. The tests pass, but closing in a comment wasn't tested for. The following has an error node in the comment, instead of closing the PHP block
#teste
#teste2
<?php

#ahahah
#ahhfhf

echo '#ola'; #  ?>

I'll think some more on this.

@joshvera OK, this seems to work better
seq(choice('//', '#'), repeat(/[^?\n]|\?[^>\n]/), optional('?\n')),

Tests:
<?php echo '#ola'; // ?>
not php

(program [0, 0] - [2, 0]
  (echo_statement [0, 6] - [0, 18]
    (string [0, 11] - [0, 17]))
  (comment [0, 19] - [0, 22])
  (text [1, 0] - [2, 0]))

<?php echo '#ola'; // ?
is php

(program [0, 0] - [2, 0]
  (echo_statement [0, 6] - [0, 18]
    (string [0, 11] - [0, 17]))
  (comment [0, 19] - [1, 0])
  (ERROR [1, 0] - [1, 6]
    (name [1, 0] - [1, 2])
    (name [1, 3] - [1, 6])))

I don't know which line endings need to be accounted for, but the idea would be the same.
The downside to this is that we capture \n if it ends with ?, so the range is different. It would be preferable to not capture this newline in the range, but still require it be there.
@maxbrunsfeld Does this sound possible?

I see two solutions here:

It may be cleaner to parse PHP the way that we currently parse other templating languages like ERB and EJS in Atom. We use a two-pass approach, where the first pass just parses the templating tags (in this case <?php and ?>), and the second pass parses just the contents of the tags using Tree-sitter's included_ranges API.
This way, it would be impossible for any token to ever interfere with the PHP tags: those tags would be parsed first, before even looking at the actual PHP code. I believe this is a more direct reflection of how PHP is actually parsed - the PHP tags are located before any other code is parsed.

In the short term, we could probably address this problem by handling comments using the external scanner, so that we could explicitly check for the sequence ?> and stop if we find it.


fix(operator): Update k8s related package to 0.19.9, kubeflow/common to 0.3.6 WIP

Fixes #1306

[ ] Test it locally

@Jeffwan @zw0610 WDYT
There are some bugs reported by our internal customers. If we decide to freeze it, we can hold it and I will fix it internally.

How long should we hold? what are the blocking issues? @Jeffwan

/retest

@gaocegege  @johnugeorge @terrytangyuan
There're similar changes in all-in-one-operator branch https://github.com/kubeflow/tf-operator/pull/1290 https://github.com/kubeflow/tf-operator/pull/1292

OK SGTM

@gaocegege When you get a chance, let's just bump the version and send a separate PR? please help cherry-pick the changes to v1.2-branch

SGTM


Spamming 'wifi: Haven't to connect to a suitable AP now!' (IDFGH-2731)

Environment

Development Kit:  none
Module or chip used: ESP32-WROVER-B
IDF version: v3.3.1
Build System: Make
Compiler version: 1.22.0-80-g6c4433a
Operating System: macOS
Power Supply: USB

Problem Description
This morning I noticed that one of my boards was spamming the message 'wifi: Haven't to connect to a suitable AP now!'' on the console. For example:
W (40478) wifi: Haven't to connect to a suitable AP now!
W (40598) wifi: Haven't to connect to a suitable AP now!
W (40708) wifi: Haven't to connect to a suitable AP now!
W (40828) wifi: Haven't to connect to a suitable AP now!
W (40948) wifi: Haven't to connect to a suitable AP now!
W (41068) wifi: Haven't to connect to a suitable AP now!
W (41188) wifi: Haven't to connect to a suitable AP now!
W (41298) wifi: Haven't to connect to a suitable AP now!
W (41418) wifi: Haven't to connect to a suitable AP now!
W (41538) wifi: Haven't to connect to a suitable AP now!
W (41648) wifi: Haven't to connect to a suitable AP now!
W (41768) wifi: Haven't to connect to a suitable AP now!
W (41888) wifi: Haven't to connect to a suitable AP now!
W (42008) wifi: Haven't to connect to a suitable AP now!
W (42118) wifi: Haven't to connect to a suitable AP now!
W (42238) wifi: Haven't to connect to a suitable AP now!

I have power cycled the board and have reflashed it several times and this persists. It started after I purged all connectable wifi networks from the device - thus esp_wifi_connect() is not being called.
This is a situation I test regularly, and have tested as recently as last night.
My WiFi initialization is as follows:
event_group = xEventGroupCreate();
xx_ERROR_CHECK_RET(esp_event_loop_init(event_handler, NULL));
wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
xx_ERROR_CHECK_RET(esp_wifi_init(&cfg));
xx_ERROR_CHECK_RET(esp_wifi_set_storage(WIFI_STORAGE_RAM));
xx_ERROR_CHECK_RET(esp_wifi_set_mode(WIFI_MODE_STA));
xx_ERROR_CHECK_RET(esp_wifi_start());
xx_ERROR_CHECK_RET(esp_wifi_set_ps(WIFI_PS_MIN_MODEM));

After this initialization I start a task which performs site scans and attempt to connect.
I have disabled the task entirely, and commented parts of the initialization. The warning starts as soon as esp_wifi_start() is called, even if I never perform a scan or attempt esp_wifi_connection().
I have searched the ESP-IDF code for this error and not found it, so I assume it is part of the WiFi blob.
Can someone tell me what state causes this warning to appear?
Thanks,
Jason

Seems very similar to #4778 that I reported recently...

@vonnieda This log shows that you called the function esp_wifi_get_ap_info() before you had connected to an AP.  If you want to get the scan results and find an AP to connect, you need to call function esp_wifi_scan_get_ap_records()

Hi @zhangyanjiaoesp, thank you for the response. This lead me to find all instances where I am calling esp_wifi_sta_get_ap_info and indeed, there is one case where I call it often enough to explain this behavior. Thanks!


Test latest .NET 8 preview

Installed .NET 8.0.100-preview.6.23330.14 (July 11th, 2023).
maui, maui-ios and maiu-android workloads 8.0.0-preview.6.8686
The issues are logged here https://github.com/SSWConsulting/SSW.Rewards.Mobile/pull/406
In a nutshell, Android has 2 regression issues for carousel and input box.
There are still some issues for both platforms.


Added mdp-0.93

A command-line based markdown presentation tool.

Won't work with $(out) because it's a variable, not a shell command.

Please squash commits into one, and force push it to update PR.

Won't work with $(out) because it's a variable, not a shell command.

TL;DR: it will, I tested it :-)
makeFlags isn't passed through the shell. Perhaps you're thinking of makeFlagsArray?
For make, $(foo) is just a plain variable. $foo and ${foo} will also work, but are in IMO even more confusing because they look just like shell variables. At least with $(foo), it's obvious that foo isn't a command and so no shell can be involved.

I also tried it and I get this error with $(out).
/nix/store/frmyf8fdq5xhm035mr87wn6kj0c278g7-stdenv/setup: line 829: out: command not found

That's... weird. Like this?
-  installPhase = "make install PREFIX=$out";
+  makeFlags = "PREFIX=$(out)";

Oops, no.  My mistake.  Pushed new change with makeFlags.  I definitely prefer this way.  I was still using installPhase.

Thanks! Travis approves. Pushed.

@nckx haven't you seen my comment?

See above. Chromium + github are horrible here atm. Sorry.
FWIW:
nckx@dm1 ~/Hacking/nixpkgs $ grep -r 'makeFlags = ' pkgs/ | grep '$(out)' | wc -l
117
nckx@dm1 ~/Hacking/nixpkgs $ grep -r 'makeFlags = ' pkgs/ | grep '[' | wc -l
194
nckx@dm1 ~/Hacking/nixpkgs $ grep -r 'makeFlags = ' pkgs/ | grep -v '[' | wc -l
223
I'll certainly pay attention to that in future, but the tree is almost maliciously inconsistent about such things.

:crying_cat_face:

:bouquet:

I'll keep that in mind in the future.  And if I update this package.


fix(material/core): remove tabindex from mat-option

Remove the tabindex attribute added to MatOption components. MatOption does not need tabindex because the parent component manages focus by setting aria-activedescendant attribute. Previously, MatOption set tabindex but was also a referenced by aria-activedescendant. This was not the correct ARIA semantics. Align closer to ARIA spec by using only aria-activedescendant rather than both.
Tabindex="-1" seems to be causing a problem in angular#26861 where VoiceOver with Firefox moves DOM focus from the combobox to the option when opening the listbox popup.
Unblocks angular#26861.

Since this was last reviewed, I moved _getTabIndex back to MatOptionBase. That's where the method originally was. Moving that method caused the typescript types to break for some situations. That's because some members expect a MatOption and adding a _getTabIndex method to LegacyMatOption makes it no longer type compatible with MatOption.

How was this not considered a breaking change? By removing the tabindex from the MatOption component, you make it not focusable. So $event.relatedTarget in the focusout event will always be null, no mater where on the screen you click. With the tabindex attribute on the MatOption, $event.relatedTarget will be set to one of the MatOption elements if you click inside the dropdown and null if you click outside. I am not sure how you are supposed to tell if you clicked inside or outside of the dropdown without this. Either way I can fix it by manually adding the tabindex, but I feel this should have been considered a breaking change and not pushed out in a patch release.


Add sortSmallArrayBy

It would be nice to have a variant that works on SmallArray#s.
My use case is that I want to use this to implement sorting for my rrb-vector library, which uses SmallArrays (the benchmarks are very promising).

Sounds like a good use case, though I don't see how sorting a SmallArray would help. For rrb-vector, isn't the data spread across multiple SmallArrays?
In general, the idea with this library is to copy the data into a MutableArray# (if it's not already in one), sort it, and copy it back out. I have used it in a similar case, sorting a sequence, which could be likely be adapted for rrb-vector with small changes:
https://github.com/meooow25/seqn/blob/775acc81790377153c5236e5403363abd7af1606/src/Data/Seqn/Internal/Seq.hs#L1235-L1241

Sounds like a good use case, though I don't see how sorting a SmallArray would help. For rrb-vector, isn't the data spread across multiple SmallArrays?

The arrays I use are actually slices of SmallArrays. My idea is to build the tree out of adjacent slices of the sorted array, to avoid copying.
The disadvantage of this approach is that the whole array is kept around, even if only a small part of it is still used.

My idea is to build the tree out of adjacent slices of the sorted array, to avoid copying.

Oh interesting...

The disadvantage of this approach is that the whole array is kept around, even if only a small part of it is still used.

Yes, I'm not sure this pays off, only to avoid one batch of copying. I imagine it is also worse for GC if a SmallArray# is actually huge.

Back to the issue, I'm reluctant to add this because a SmallArray# is typically, well, small. Data.Primitive.SmallArray says

The card size is 128, so for uses much larger than that, Array would likely be superior.

At these sizes, insertion sort may be fast enough to not require more complex algorithms.

Yeah, you're probably right.


Direktiv Repository Mirror: GitLab user and PAT token not being used

We're using GitLab as the workflow repository. If we want to sync to GitLab (in our mirroring) then we need to be able to support the following type of configuration:

We created a PAT token, but we cannot get it working without actually having the token completely visible in the URL
so something like:
<EMAIL_ADDRESS>There needs to be an option to configure a username as part of the mirror configuration, however, not sure how this will affect the GitHub syncrhonisation?

Fixed in 0.7.1-rc1


Added Math.log function

:deciduous_tree:

This is incredible.
Two things:

Can you rebase the latest master into your pull request to resolve the conflicts?
Can you re-add the newline at the end of the file? Per http://stackoverflow.com/a/729795

Yep - should be good now!

Amazing.


MHD TI issues

There are many reported issues with MHD as multiple projects implement. This work item would look to resolve the current backlog of issues and provide a revision. Note that some of the issues reported likely will result in significant change to MHD and thus MHDS. such as changing ITI-65 to an $operation, and support for Bundles as documents.
https://github.com/IHE/ITI.MHD/issues

https://chat.fhir.org/#narrow/stream/179223-ihe/topic/MHD.20revision

@oliveregger please nominate the most important MHD/MHDS issues in comment here.

First try from my side, additional input welcome, we should probably tackle all issues ...

Identifier discussion #90, #92, #93
DocumentReference.date #94
ReplaceTo behaviour #100
enhancements https://github.com/IHE/ITI.MHD/issues?q=is%3Aissue+is%3Aopen+label%3Aenhancement

Candidate Editor: John, Oliver
Developer resources: Simone, Luke


CSS for standard names tables HTML pages

I notice that all HTML pages for standard names tables are rendered in the browser without any styling, despite the rest of the CF Conventions site being styled with Bootstrap. Is there a reason that no styling has been applied to the HTML tables? If not, I think it would be better to apply some consistent (i.e. Bootstrap or some light customisation of) styling to the tables, for cross-site consistency in presentation if nothing else but probably making them a little easier to read and search through as well.
Unless anyone conveys of any reason to not add styling, I can put in a pull request to implement it quickly, as I have already tried out the built tables locally with some CSS. with the Bootstrap stylesheet used elsewhere in the site. I decided it would be better to use it with another very small stylesheet to customise the table elements themselves as they were otherwise cramped (see details below).

Demo of a HTML table page with (suggested) styling applied
All screenshots shown here are taking the HTML version of the table for v.71 as an example, as linked here for ease of comparison to the current (absence of) style (in the web sense).  I opened the pages locally in Firefox (68.0.1, 64-bit) and applied these sheets via the developer console, but of course they can (if I get the go-ahead) be coded into the headers of the HTML so they are applied permanently to the rendered pages on the site.
With just the Bootstrap stylesheet
There is already a Bootstrap CSS sheet applied to pages on the site, namely Data/media/css/bootstrap.min.css, so I started by trying that out on the page to see how it looked. While it generally looks clean and clear:

the table itself has become cramped vertically, and cells have lost padding against the borders, as Bootstrap must amend the element & border spacing or similar:

Suggested style: Bootstrap stylesheet + small stylesheet to customise
To get over the above, I played around and found the following CSS customisations:
td {
  height: 10px;
  line-height: 30px;
}

table td {
    padding-left: 8px;
    padding-right: 8px;
}

table th {
    padding-left: 8px;
    padding-right: 8px;
}

tr:hover {
  background-color: #f5f5f5;
}

would keep the consistent Bootstrap look and feel but stop the table itself (the most important part, of course!) being styled in a way that lacks spacing & looks cluttered.
With both the Bootstrap and the above stylesheets, we get the following (including greying of the row being hovered on to help match up names and units columns, etc.), which I propose as a candidate for the overall CSS to apply to the tables HTML pages:

Of course, people might prefer something a little different (e.g. a striped table, maybe?) & perhaps there could some further styling "bells & whistles" added.

@sadielbartholomew I can't think of a good reason not to use CSS (of course that doesn't mean that there isn't one!). It might be daunting to get this integrated with however they're built and deployed though - you might want to coordinate with Alison, as she probably knows of any pitfalls. My preference is for striped tables, beyond that I'm a known table slob, willing to consume the ugliest table, as long as it is indeed tabular. I'd support this initiative.

I don't know about the technicalities of sytle sheets, but I certainly support bringing the standard name table in line with the look and feel of the rest of the CF web site.

Thanks @erget, I'll definitely aim to coordinate with Alison. I did suspect some deployment work might be required, since all the HTML table page outputs were in the build directory of each version so somehow the line(s) to specify to use certain stylesheets would have to get processed into all of those. Happy to do that though. Hopefully Alison can help with that, so I will get in touch with her to ask. Thanks for the pointer.

My preference is for striped tables, beyond that I'm a known table slob, willing to consume the ugliest table, as long as it is indeed tabular.

:smile: Fair enough, I am sure there are many who don't care one bit as long as long as the tables are readable! But I do think unstyled HTML has the potential to look quite dated to some. It is certainly uncommon to see it on the general web nowadays (at least, outside of the personal webpages of academics!) If this gets to PR stage I will try out a striped table and I can show a screenshot so we can decide if we think that looks & reads better.

Closed by #404.


no support of "com.fibaro.FGT001" devices

Hi,
Devices with the type "com.fibaro.FGT001" don't appear in accessories list

@vkamenski can you test version 1.1.7 ?

@ilcato Hi, yes it works. Thank you for your work!


window.ethereum.send('eth_accounts').then doesn't work

Fixed in develop branch. Will be a part of 5.1.0 release.


Add placeholder pages for Domain Apps

It would be helpful for developers using this template, to be able to build upon predefined pages for different App users.
As a reference you could take the interface of the PHP-app-skeleton developed by Ondics:

In case of this PHP app there are placeholder pages for Users, Developers, Admins, and a Rest-interface. There can also be more placeholders (e.g. for Documentation), but in my opinion those 4 should be enough in most cases.

Would be great if you made a pull request with this. The app skeletons use Ionic due to various reasons (ease of use, UI toolkit etc.). If you do the pull request, please create the pages with the Ionic style.
The Ionic style closely resembles Googles Material Design. You should look into it, if you haven't until now.

Alright it took me a while, but I worked out an interface similar to the one used in the PHP app.
After talking with Dr Menschig we actually do NOT want an overwiew page like is shown in the picture above. Instead every site should be accessible through a popup menu in the side. Therefore I added an addidtional Menu button on the top bar, which toggles an overlay menu with all the buttons.
The individual sites are working well, but there are actually some problems with the linking-system of different buttons. Andy, I used your ionicpage-branch as a reference on how to link different sites. However using the [navPush]="admin" seems to create some issues, or maybe I just haven't fully understand Ionics nav-system... Using href="/#/admin" seems more reliable, as it actually opens the link instead of going through the Ionic system.
Which system do we want to use in the end?
As I'm not allowed create a branch in this repo, I had to copy this project to my github: https://github.com/RobinFischer/nodejs-app-skeleton
Please tell me if I'm on the right track.

Here is how you can create a pull request (i.e. add to this project):
https://yangsu.github.io/pull-request-tutorial/
And yes, the navPush is broken in the branch (that's the reason it was not commited to master). We are still working on a fix.
Alright it took me a while, but I worked out an interface similar to the one used in the PHP app.
-> my docker is broken right now, did you change the UI? the screenshots are still the same

Maybe you should use the alpine version, and not node:boron. It saves you a lot of space and I heard you want to make it run on a RPi.
we actually do NOT want an overwiew page like is shown in the picture above.
-> that's ok, you can modify the code as you wish and pull changes we make from upstream. If it's a useful change we'll ask you for a pull request!

Thanks for the info. I figured out that my project wasn't properly forked, but now I would be able to do a pull request.
However I'll wait until you fix the navPush feature, as it seems that’s what should be used in the end.
The UI currently looks like this:

You can see the menu only includes the 4 main user-roles. Inside the role system I'm still using the old PHP system, which isn't ideal. The goal is to move the navigation interface is moved out of the way, so the sites are only filled with the actual content. For the role system we could either use a sub-menu in the side, or a tab system (which I think would fit better).
Regarding the RPi: that would be a nice to have, but we also have a backup system without an ARM processor. I'll try to stay close to the master branch, so it makes the merging easier.


snallygaster: 0.0.11 -> 0.0.12

Motivation for this change
Update to latest upstream release 0.0.12
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 158335 run on x86_64-linux 1

1 package built:

snallygaster


add support for sending data over TLS

Currently AppScope only supports sending data over raw TCP sockets, however when sending data to a SaaS destination TLS is a requirement.

So, to test with a self-signed certificate w/Logstream...  Used this command where logstream is running to generate a private key (domain.key) and a self-signed cert (domain.crt).
openssl req \
       -newkey rsa:2048 -nodes -keyout domain.key \
       -x509 -days 365 -out domain.crt

Then enabled TLS for the AppScope Source, and provided the Private Key Path and Certificate Path like this:

Then configured AppScope like this:
cribl:
  enable: true
  transport:
    type: tcp
    host: <IP_ADDRESS>
    port: 10090
    tls:
      enable: true
      validateserver: true         # false is like curl's -k option
      cacertpath: '/home/ubuntu/jrc/appscope/certs/domain.crt'
          # cacertpath can be used to specify a private CA.
          # If left empty, AppScope will try to use public root certs
          # that are provided by the linux distro.

Then ran scope like this SCOPE_LOG_LEVEL=info LD_PRELOAD=lib/linux/libscope.so top.
Inspected the contents of /tmp/scope.log to see output like this:
Scope: top(pid:7550): getting DNS info for tcp <IP_ADDRESS>:10090
Scope: top(pid:7550): fd:5 connect to <IP_ADDRESS>:10090 is pending
Scope: top(pid:7550): fd:5 connect successful
Scope: top(pid:7550): fd:996 establishing tls session
Scope: top(pid:7550): fd:996 tls session established

To Dos:

Create SCOPE_ env vars equivalent to the AppScope scope.yml tls configuration shown above.
Update help text (currently in os/linux/os.c) to describe the new tls env vars
Build issues:
3A) fix libscope.so dependency on GLIBC_2.25 (previously 2.12 was the latest required) per test/linux/glibcvertest
3B) do we need to do anything with the -D defines for OPENSSLDIR, ENGINESDIR, and MODULESDIR?
Provide cloud-friendly defaults for SCOPE_CRIBL mode: tls enable: true, validateserver: true, cacertpath: ''
Rework remoteConfig() to use tls for command/response with logstream.  Validate with LogStream Cloud.
CLI needs to be updated for tls

A couple of technical notes:

when we statically linked openssl into libscope.so, we had two conflicting symbols...  SSL_read and SSL_write had one definition in src/wrap.c and another in libssl.a.  To solve this, I followed the pattern described here: https://stackoverflow.com/questions/6940384/how-to-deal-with-symbol-collisions-between-statically-linked-libraries
when we first statically linked openssl into libscope.so, all public openssl symbols were added to libscope.so.  To avoid this, we introduced a version script (libscope.map) to blacklist all of the symbols that were added by the inclusion of libssl.a (well, with the exception of SSL_read and SSL_write described immediately above). Here's a description on the topic:  https://www.gnu.org/software/gnulib/manual/html_node/LD-Version-Scripts.html

Here's a table for high-level validation scenarios we want to run through.
For "Is Valid Config : true" scenarios, we want to verify that:

Data flows to logstream
If run with SCOPE_LOG_LEVEL=info, /tmp/scope.log should reflect the success
Host app should run unaffected (including no segfault on termination)

For "Is Valid Config : false" scenarios, we want to verify that:

No data flows to logstream
If run with SCOPE_LOG_LEVEL=info, /tmp/scope.log should indicate the error
Host app should run unaffected (including no segfault on termination)

No TLS
No Server Validation
Self-Signed
Publicly Trusted
Mismatched TLS Config 1
Mismatched TLS Config 2
Not Trusted

Is Valid Config
true
true
true
true
false
false
false

Logstream Location
X
X
X
Cloud
X
X
X

Logstream TLS Enabled
No
Yes
Yes
Yes
Yes
No
Yes

AppScope TLS Enabled
false
true
true
true
false
true
true

AppScope ValidateServer
X
false
true
true
X
X
true

AppScope CaCertPath
X
X
Valid path to local cert
empty string
X
X
No local cert and No public trust

Test findings:
A) For Mismatched TLS Config 1, when I run this as my scoped process: SCOPE_LOG_LEVEL=info LD_PRELOAD=lib/linux/libscope.so top, I never see an error in /tmp/scope.log...  Presumably because logstream is terminating our connection from the remote side.  If we can detect this and report it, that would be good.
Scope: top(pid:549): fd:8 connect successful
Scope: top(pid:549): getting DNS info for tcp <IP_ADDRESS>:9109
Scope: top(pid:549): fd:8 connect to <IP_ADDRESS>:9109 is pending
Scope: top(pid:549): fd:8 connect successful
Scope: top(pid:549): getting DNS info for tcp <IP_ADDRESS>:9109
Scope: top(pid:549): fd:8 connect to <IP_ADDRESS>:9109 is pending
Scope: top(pid:549): fd:8 connect successful

B) For Mismatched TLS Config 2 and Not Trusted when I run this as my scoped process: SCOPE_LOG_LEVEL=info LD_PRELOAD=lib/linux/libscope.so top I get a Segmentation fault (core dumped) when I "ctl-c" the top process.

All of the work completed against #164 has been published from the feature/send_tls branch with a tag "v0.7.0-rc2" (See ToDos above for what work was completed in this ticket).
To break the tls work up a little, I've created #286.  All uncompleted ToDos have been moved to #285, where work will continue.

These review comments were made in a walkthrough of this ticket, and are resolved here:

[x] Switch from make build_sw to make for openssl build (from review of #164)
[x] Create note describing how recursive mutex could help debug in src/transport.c (from review of #164)
[x] Rename functions in cfg.[hc] to add Tls to validateserver and cacertpath (from review of #164)

Other review comments were added to the "to do" list in #286.


Anka macOS version is out of date

The macos CI failed on a recent build of the master branch, with the following error message:
==> ./configure --disable-warnings-as-errors --with-boot-jdk-jvmargs=-Duser.home
Last 15 lines from /Users/jenkins-slave/Library/Logs/Homebrew/openjdk/01.configure:
checking whether the compiler supports GNU C++... yes
checking whether /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang++ accepts -g... yes
checking for /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang++ option to enable C++11 features... none needed
checking how to run the C preprocessor... /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang -E
checking how to run the C++ preprocessor... /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang++ -E
configure: Using clang linker version 409.12 [@(#)PROGRAM:ld  PROJECT:ld64-409.12]
checking for ar... /usr/bin/ar
checking for lipo... /usr/bin/lipo
checking for otool... /usr/bin/otool
checking for install_name_tool... /usr/bin/install_name_tool
checking for metal... [not found]
checking if metal can be run using xcrun... no
configure: error: XCode tool 'metal' neither found in path nor with xcrun
/private/tmp/openjdk-20211107-32797-14snxaf/jdk17u-jdk-17.0.1-ga/build/.configure-support/generated-configure.sh: line 84: 5: Bad file descriptor
configure exiting with result code 1

Do not report this issue to Homebrew/brew or Homebrew/core!

These open issues may also help:
openjdk@11 11.0.13 https://github.com/Homebrew/homebrew-core/pull/87638
openjdk: try to reproduce java.lang.UnsatisfiedLinkError https://github.com/Homebrew/homebrew-core/pull/84886
OpenJDK is somewhat broken on newer MacOS instances, console is flooded with errors when using JMeter, AdoptOpenJDK has no issues https://github.com/Homebrew/homebrew-core/issues/66953

Error: You are using macOS 10.14.
We (and Apple) do not provide support for this old version.
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Twitter or any other official channels. You are responsible for resolving
any issues you experience while you are running this
old version.

Error: A newer Command Line Tools release is available.
Update them from Software Update in System Preferences or run:
  softwareupdate --all --install --force

If that doesn't show you any updates, run:
  sudo rm -rf /Library/Developer/CommandLineTools
  sudo xcode-select --install

Alternatively, manually download them from:
  https://developer.apple.com/download/all/.

You should download the Command Line Tools for Xcode 11.3.1.

script returned exit code 1

This looks like a question of upgrading the version of macOS + command line tools that's installed on the Anka test runner machine.

We're not going to use Anka in the future.


Authenticate via SSH key

Not sure, if this feature is available, could not find it.

Found the reloaded version, can be closed

I think you can close


Can't run go-errcheck when saving go file

When using doom emacs edit a go file, got error below:

export GOPATH=$HOME/go
export PATH=$PATH:$GOPATH/bin

$ ls $HOME/go
bin pkg src

$ ls $HOME/go/bin
asmfmt  dlv  errcheck  fillstruct  gocode  gocode-gomod  godef  gogetdoc  goimports  golint  gometalinter  gomodifytags  gorename  gotags  gounit  guru  iferr  impl  keyify  motion

$ whereis go
go: /usr/local/go

~/.doom.d/init.el
(doom! :input
    go
    flycheck
    ...
)

Please provide 1) your M-x doom/info and 2) the output of M-; (executable-find "go").

@hlissner

Got nil

This appears to be a PATH issue. Either:

Your envvar file is outdated. doom env refresh will regenerate it. You'll need to do this whenever your shell environment is changed (i.e. your shell dotfiles).
 You need to do this in ~/.bashrc, because doom env reads your shell environment from an interactive (non-login) shell, which only reads .bashrc, and not .bash_profile.

@hlissner
Thank you for your answer! I tried your way ~/.emacs.d/bin/doom env refresh, got:
Regenerating envvars file at "~/.emacs.d/.local/env"
Scraping env from ’/bin/bash -ic /bin/env’

Then edit a go file and save it. The error didn't happened again!
I'm using the zsh shell.

It's odd that it thinks your shell is bash, however, since it resolved your issue I guess it's a happy accident.
I'll consider this resolved, but if this gives you trouble in the future, try adding (setq shell-file-name (executable-find "zsh")) in ~/.doom.d/init.el. Thanks for bringing it to my attention!


HPE Nimble not usable with OpenShift Virtualization

The HPE CSI driver does not work as expected in ReadWriteMany mode using a raw block device.
During a VM live migration, the following happens.

VM is running in pod PA on node A
Live migration starts
pod PB is started on node B
VM memory/state is copied to PB on node B
PA shuts down, PB resumes the VM

The VM disk is a volume that is attached to 2 pods (source migration and destination) from steps 2 to 4. Once the migration finishes the source pod (PA) is deleted and the VM runs on the destination (PB). The problem is that once PA shuts down the volume is set to offline by the CSI driver, and the destination (PB) loses access to the volume, pausing the VM with EIO as its storage is gone, so instead of running on the destination node, the VM is now paused/hung because it lost access to its disk/volume.
The problem is reproducible by starting 2 pods on 2 different nodes, sharing a raw block RWX volume. No need for live migration or virtualisation.
Once the first pod shuts down the remaining pod loses access to the volume, because the nimble side makes the backing volume go offline and the iscsi connection is abruptly closed while one of the pods is still using it.

Create test-pvc with Block and RWX, on HPE CSI StorageClass.
Create 2 pods, running on 2 different nodes, using the shared PVC

apiVersion: v1
kind: Pod
metadata:
name: example
labels:
app: httpd
namespace: default
spec:
containers:
- name: httpd
image: 'image-registry.openshift-image-registry.svc:5000/openshift/httpd:latest'
ports:
- containerPort: 8080
volumes:
- name: raw
persistentVolumeClaim:
claimName: test-pvc

Shutdown one pod

Other pod loses LUN access, LUN is offline on HPE Nimble dashboard

Logs for the HPE CSI driver shows the request to offline the LUN that is still used by the other pod:
2022-08-22T21:32:44.661195588Z 21:32:44 DEBUG [co.ni.no.gr.GroupMgmtClient]] (executor-thread-88) 7 * Sending client request on thread executor-thread-88
2022-08-22T21:32:44.661195588Z 7 > PUT https://<IP_ADDRESS>:5392/v1/volumes/066b77f2277cc0b66d00000000000000000000098e
2022-08-22T21:32:44.661195588Z 7 > Content-Type: application/json
2022-08-22T21:32:44.661195588Z 7 > X-Auth-Token: ****
2022-08-22T21:32:44.661195588Z {"data":{"online":false,"force":true}}

I should not offline the volume, so PB does not lose access to it. This should only be offlined once all pods sharing it stop using it. Perhaps missing some refcount.

The team is aware of this issue and what happens is that the CSI Unpublish call revokes access to all initiators when it should only revoke access to the source initiator and not offline the volume. The RWX volumeMode: Block PVC should not have been provisioned in the first place as this has not been tested.
I'll update this thread if I hear about a potential bugfix in a future release.

Digging more into this, this item is specifically around RWX block mode PVs. If the Nimble CSI can support RWX Filesystem PVC/PVs, I'd recommend scoping the title to something more precise, and list possible workarounds.

The CSI driver supports volumeMode: Block for all accessModes (except RWOP). It's the CSP implementations that breaks. The official HPE CSPs are not open source at this time but we're working on a resolution.

@datamattsson Any update on the status on this ?

@pipopopo we've identified what needs to be done and engineering is working on it. No ETA as of yet but it will be part of the next release of the CSI driver.

For anyone feeling edgy there's a publicly available CSP image with the fix.
If you want to use it with the current operator you need to disable the Nimble and Alletra6K CSP in the CSV (it's parameterized .spec.disable.nimble: true and .spec.disable.alletra6000: true). Then deploy the CSP with a resource manifest from here and change the image to quay.io/datamattsson/alletra-6000-and-nimble-csp:block-rwx.
This is not meant for production use, the fix will be part of the next CSI driver release which is TBD at this point.

@datamattsson Any update for this  issue ？
I meet  this issue on alletra 9k also

Alletra 5/6K and Nimble will be supported in the next release. Alletra 9K, Primera and 3PAR does not have an ETA at this time.

Fixed in v2.4.0.

When do you think that 3PAR could OpenShift Virtualization will be supported using 3PAR.

When do you think that 3PAR could OpenShift Virtualization will be supported using 3PAR.

I'd recommend creating a new issue in this repo for 3PAR, and it can be tracked separately.

Alletra 5/6K and Nimble will be supported in the next release. Alletra 9K, Primera and 3PAR does not have an ETA at this time.
Can you clarify where we landed with this for v2.4.0?   I have a customer needing Alletra 9K support

@therevoman the 2.4.0 release that includes Nimble/Alletra5/6K has been delayed for OpenShift. You can still install the CSI driver with the Helm chart but it's unsupported by Red Hat. However, that release will be available in the next few days in the Red Hat catalog so I would encourage customers/partners to wait for that.
For backends with 3PAR pedigree (3PAR/Primera/Alletra 9K/MP) you'll have to wait for 2.4.1 which is in the works. ETA is unknown at this time.


Horizontal line is not drawing in 0.40.6

Horizontal line were not drawing in fl_chart 0.40.6. I had to decrease the version back to 0.40.5 to make it work. Here is some part of my code used.
gridData: FlGridData(
                show: true,
                drawVerticalLine: false,
                drawHorizontalLine: true,
                getDrawingHorizontalLine: (_) {
                  return FlLine(
                    color: AppColor.grey,
                    strokeWidth: 0.5,
                  );
                },
              )

As described in the issue's template. You need to provide a reproducible code.


Cannot execute a trans (when deployed to Docker and connected to a Pentaho Repository)

How to reproduce

Deploy webSpoon as a Docker container
Connect to a Pentaho Repository (running on the Docker host, so not on localhost)
Create a trans and save it to the repository
Run

Workaround:

Deploy webSpoon in a non-Docker env, or
Do not use Pentaho Repository

Here is the environment
OS: macOS 10.12.6
Docker: 17.06.0-ce
Docker image: hiromuhota/webspoon:<IP_ADDRESS>-full
Here are the error messages of the Docker container:
2017/08/23 17:43:36 - Spoon - Using legacy execution engine
2017/08/23 17:43:36 - Spoon - Transformation opened.
2017/08/23 17:43:36 - Spoon - Launching transformation [Transformation 1]...
2017/08/23 17:43:36 - Spoon - Started the transformation execution.
2017/08/23 17:43:36 - Transformation 1 - Dispatching started for transformation [Transformation 1]
2017/08/23 17:43:36 - Spoon - ERROR (version <IP_ADDRESS>-12, build 1 from 2017-05-16 17.18.02 by buildguy) : Transformation 1: preparing transformation execution failed
2017/08/23 17:43:36 - Spoon - ERROR (version <IP_ADDRESS>-12, build 1 from 2017-05-16 17.18.02 by buildguy) : org.pentaho.di.core.exception.KettleTransException:
2017/08/23 17:43:36 - Spoon - There was an error calculating the change data capture date range, it probably involved log table null.
2017/08/23 17:43:36 - Spoon -
2017/08/23 17:43:36 - Spoon - No log table has been defined for the transformation.
2017/08/23 17:43:36 - Spoon -
2017/08/23 17:43:36 - Spoon -
2017/08/23 17:43:36 - Spoon - 	at org.pentaho.di.trans.Trans.calculateBatchIdAndDateRange(Trans.java:2238)
2017/08/23 17:43:36 - Spoon - 	at org.pentaho.di.trans.Trans.prepareExecution(Trans.java:985)
2017/08/23 17:43:36 - Spoon - 	at org.pentaho.di.ui.spoon.trans.TransGraph$26.run(TransGraph.java:4287)
2017/08/23 17:43:36 - Spoon - 	at java.lang.Thread.run(Thread.java:745)
2017/08/23 17:43:36 - Spoon - Caused by: org.pentaho.di.core.exception.KettleTransException:
2017/08/23 17:43:36 - Spoon - No log table has been defined for the transformation.
2017/08/23 17:43:36 - Spoon -
2017/08/23 17:43:36 - Spoon - 	at org.pentaho.di.trans.Trans.calculateBatchIdAndDateRange(Trans.java:2051)
2017/08/23 17:43:36 - Spoon - 	... 3 more

Irrelevant.
This was not an issue of webSpoon, but a miss-configuration of logging.


[doc] Add connection.pool.size and connect.max-retries option in mysql document

We missed document when introduced mysql connection pool in https://github.com/ververica/flink-cdc-connectors/pull/534, we should add the introduce configuration options in document.

please assign to me

Thanks for taking this issue @eastfisher, assigned to you.

Thanks for taking this issue @eastfisher, assigned to you.

Hi, I've submitted a PR #857 to fix this issue, please take a look. ^_^


[ci] tests.config.tests.test_config_all_upper_case failing

It appears that the tests.config.tests.test_config_all_upper_case test is failing across a number of different platforms on the master branch:

@cachedout I'm aware, I just didn't get around to fix it yet. It's an easy fix, I'll open a PR


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220219163632807 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220219163632807 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14819

Relevant Lines:
29137

💛  - Coveralls


Minor copy-paste error in all the samples.

You're saying set ``YOUR_ENVIRONMENT_ID but then in the actual config, it's "ENVIRONMENTID" like:
  # For SaaS, set `YOUR_ENVIRONMENT_ID` to your environment ID.
  # For Managed, change the apiUrl address.
  # For instructions on how to determine the environment ID and how to configure the apiUrl address, see https://www.dynatrace.com/support/help/reference/dynatrace-concepts/environment-id/.
  apiUrl: https://ENVIRONMENTID.live.dynatrace.com/api

E.g. https://github.com/Dynatrace/dynatrace-operator/blob/2b9a13edd8c21d2652f1e0db431c34242ad2c584/assets/samples/classicFullStack.yaml#LL7-L11C55

This inconsistency should indeed be removed, thanks!


Cache store does not support tagging (Laratrust)

I tried to install Laratrust for roles and permission.
But when I add the "LaratrustUserTrait" to my User Model it gives me an error:
"This cache store does not support tagging"
More details: https://flareapp.io/share/DPyGVJ59#F69
How could I solve this? I read on google people suggest to change CACHE_DRIVER in ENV to array, but this doesn't work for me.

Hi. If you want your cache to be tenant-aware, you need to use a driver that supports tagging, e.g. Redis.
If you don't need tenant-aware cache, comment out the CacheTenancyBootstrapper in your tenancy.php config file.

Thanks I commented out the CacheTenancyBootstrapper and that solved it. Will look into Redis in the future!

Glad it works :)
If you ever need to use caching in your app, Redis is a really good option.
It's extremely performant and it requires practically zero configuration. Just install it and set the cache driver to Redis.
And if you want to learn about using Redis yourself, not just through Laravel's cache, I recommend this course on Laracasts: https://laracasts.com/series/learn-laravel-and-redis-through-examples


allow image manipulation on presigned urls

very much needed - are there any workarounds?

You can use another image transformation service such as on Cloudflare or Vercel, and send the entire presigned url with signature to their endpoint.
With Cloudflare you need the pro plan on your domain with image transformation enabled, and make your image src something like https://images.your.domain/cdn-cgi/image/width=400,fit=cover/{presignedUrl}

thanks for your reply and succestion. I am using Nhost pro plan and looking for a solution within that scope.

this is now supported, if someone wants me to update their cloud version DM me or drop me an email with your subdomain

Will this feature also be available in nhost-dart?

Awesome, thanks!


[Automation] az automation: Add new command group schedule software-update-configuration software-update-configuration runs software-update-configuration machine-runs

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

az automation schedule: Add new command group to support managing schedule
az automation software-update-configuration: Add new command group support managing software-update-configuration
az automation software-update-configuration runs: Add new command group support managing software-update-configuration runs
az automation software-update-configuration machine-runs: Add new command group support managing software-update-configuration machine-runs
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

Automation


Add lock battery reminder automation to HA

As someone with automated, battery-powered locks, I want to be reminded when the batteries need to be changed so that the system is fully autonomous.

[x] Research expected battery length for devices -> approximately every 6 months, so 5 month HA reminder cadence
[ ] Discuss the appropriate amount of slack to be safe
[ ] Add a reminder to the HA install to charge the batteries

Don't think it's critical at all, just a nice plus.