notebook-history-s3 cronjob not saving data to S3

The on-start.sh script is downloading a file named "notebook-history-s3.py" using the following command:
wget https://raw.githubusercontent.com/aws-samples/amazon-sagemaker-notebook-instance-lifecycle-config-samples/master/scripts/notebook-history-s3/notebook-history-s3.py
But, when it creates a cronjob, it tries to execute a python file named "notebook_history_s3.py" which doesn't exist.
(crontab -l 2>/dev/null; echo "0 * * * * /usr/bin/python3 $PWD/notebook_history_s3.py") | crontab -
suggested change to fix the issue:
(crontab -l 2>/dev/null; echo "0 * * * * /usr/bin/python3 $PWD/notebook-history-s3.py") | crontab -

Thank you for pointing this out. We will work on resolving this in an upcoming PR.

The PR above has been merged, thank you for pointing out this issue. Closing this since it is resolved.


Implement a mark-and-sweep garbage collector

Issue summary
Context: We previously investigated the feasibility of using a mark-and-sweep garbage collector, but it was prohibitively expensive to scan values with RocksDB. New data shows that ParityDB can efficiently scan through all values. This opens the door for a more space-efficient mark-and-sweep GC.
Important caveats: We require state-roots going back 2 x chain_finality = 1800 epochs. Any data that is reachable within 1800 epochs may not be garbage collected.
The Forest database contains a persistent (i.e. immutable), directed acyclic graph. This makes implementing a mark-and-sweep garbage collector reasonably straightforward. The algorithm could look like this:

Scan through all IPLD values in the database and mark their hash value in a hash set. Collisions don't affect correctness, so the hash size matters little.
Wait for at least 1800 epochs after the scan.
Traverse the reachable graph from HEAD (including at least the last 1800 epochs) and remove reachable nodes from the hash set.
Scan the database again and remove all elements still in the hash set.

Notes:

If there is a hash collision, some data may not be garbage collected. Hash collisions will never cause reachable data to be garbage collected.
ParityDB does not store keys. This is fine in Forest since our keys are always the hash of the IPLD value.
Make sure we no longer use buffered writes and use this approach instead if applicable.

Other information and links
Some knowledge in regards to CID construction:

For the GraphDagCborBlake2b256 parityDB column we currently use this approach (V1, DAG_CBOR codec, Blake2b256 hash). We need to construct the CID manually, because this column does not have a btree_index therefore we have no way to iterate the keys. That's done due to performance reasons as DAG_CBOR is the most common type of key.
For the GraphFull column we have the btree-index so we can iterate the keys. Otherwise there is no way to know how to get the CID hash.

After looking into this, am I correct to assume that:
1. With this approach we can ditch the whole `old and new` db concept, as we can just operate on the current db, deleting only unreachable stuff from the previously collected hash set.

Yes.

2. We'll need to calculate hashes of IPLD values according to the specified algorithm, which is already implemented in forest.

Yes. In essence, a CID is the hash of an IPLD value. But the devil is in the details. We can talk through the details in slack.

3. We need to traverse the graph from HEAD to detect all the reachable nodes within 1800. The rest can be removed. Are there cases when we need more than 1800 epochs history?

Yes, 1800 is the absolute minimum, but many node operators want more history than that. The exact number of epochs to keep should be configurable (but still with 1800 as the minimum).

4. We aren't currently aiming at parallel DAG traversal, as far as I can see the current implementation is a recursive function that takes care of visiting everything that's reachable.

Indeed. When exporting a snapshot, the key-value pairs must be emitted in depth-first order. But for the GC, a parallel traversal would be great.

Additionally, once the above has been implemented we could look into parallel DAG traversal if that'd make sense performance-wise.
Please correct me if I'm wrong.

Looks like we're on the same page.

IIRC, the major blocker with parity-db is that, it's impossible to retrieve raw (CID) key with its iteration API, see paritytech/parity-db#187

We don't need to retrieve them. We can re-generate them from the IPLD value. It costs a bit of CPU time but isn't too bad.

I have got two questions:

How are we going to deal with GC progress endpoint and manual runs? I have currently opted to have EITHER manual OR automatic GC, with the current approach there is little sense to allow for both at the same time. But we can if needed.
We probably need to re-think the whole progress report thing too, as reporting GC progress now makes very little sense. What we could do is something like: 30% when we have marked all the keys, 60% when we filtered and 100% after sweeping, but it's pretty inaccurate. As far as I understand the GC progress report is a part of the filecoin rpc spec, would love some guidance here.

Re-opening, the feature was temporarily reverted. https://github.com/ChainSafe/forest/pull/3682


typo: prefx -> prefix

@nschonni : Thanks for your contribution! The author, @anavinahar, has been notified to review your proposed change.

Editorial fix confirmed.


fix: Transparent menu on MenuButton widget

Description
Fixes #7240
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manual
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/menubutton-bg-color

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.27 (0)
36.31 (-0.01)
33.43 (0)
54.81 (0)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
50.47 (-0.24)
38.26 (-0.87)
36.21 (0)
55.08 (-0.27)

@aswathkk please resolve conflict

@somangshu done!


Play again sound when clicking again on a tile

Once a tile has been selected its associated sound is played.
It would be nice to be able to play the sound again by clicking on the same tile (sometimes there can be noise around the player and it may be nice to repeat the sound once the noise has gone away ...).

Interesting suggestion that is definitely doable.
I am concerned about the sound stacking effect though.
For instance, how should the app behave when a selected tile is spam clicked?
Please discuss it here


TextField fullwidth outlined problem

It is not possible to interact with TextField provided with fullwidth and outlined props
https://codesandbox.io/s/38z2qjmn76

Duplicate: https://github.com/jamesmfriedman/rmwc/issues/229
Basically combining these two is not allowed by MDC spec:

NOTE: Do not use mdc-text-field--box or mdc-text-field--outlined to style a full width text field.


add: nix-shell interpreter

see: https://github.com/cachix/pre-commit-hooks.nix/issues/35

Reopened, since this is ok and plays well with #130 for the wired things happening on the second line.


Delete manifests/j/JanDeDobbeleer/OhMyPosh/7.47.0 directory

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

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

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run


Added SUI notifications product feature

@miq-bot add_label enhancement
@miq-bot add_label fine/no

@chriskacerguis here is the PR that is needed to fulfill https://github.com/ManageIQ/manageiq-ui-service/pull/1053

Is there really a feature to get notifications?  I mean, why would an admin ever turn that off? 🤔

@Fryguy it's for testing.  QE requested to have the feature turned off.  They stated that this was something that already exists in Classic UI.

That seems like a strange reason to create a whole customer-facing "product feature"

@Fryguy /shrug.  We did that to match what is in Classic UI (again, per QE)

I'll merge for now for consistency with classic UI and for prior parity, but for the record I am 👎  on this feature in general cause it makes no sense from a customer-facing POV...just one more thing the user has to ignore to make it difficult for them.


Getting the following error : "feature "" has a value outside the dataset"

When generating the counterfactual explanation, i'm getting an error related to a feature value which value is considered to be outside of the dataset. This is strange for me, since the variable is a dummy and is not allowed to vary. The variable name is not in the list of feature to vary. My target variable is continous. Perhaps i'm missing something, could someone help, please.
Sincerely yours

@noureini , could you please share some sample code or python stack trace?
Regards,

@gaugup, please find below the traceback

ValueError                                Traceback (most recent call last)
 in
2 #features_vary=list(filter(r.match, continuous_features_df_lait_cf))
3 query_instances_lait = x_train[2:3]
----> 4 genetic_lait = exp_genetic_lait.generate_counterfactuals(query_instances_lait,
5                                                              total_CFs=2,
6                                                              features_to_vary=['Positionnement: lait_vendu_par_sfp', 'has a value outside the dataset'],
~\anaconda3\lib\site-packages\dice_ml\explainer_interfaces\explainer_base.py in generate_counterfactuals(self, query_instances, total_CFs, desired_class, desired_range, permitted_range, features_to_vary, stopping_threshold, posthoc_sparsity_param, posthoc_sparsity_algorithm, verbose, **kwargs)
82             #query_instances = query_instances.to_dict("records")
83         for query_instance in query_instances_list:
---> 84             res = self._generate_counterfactuals(query_instance, total_CFs,
85                     desired_class=desired_class,
86                     desired_range=desired_range,
~\anaconda3\lib\site-packages\dice_ml\explainer_interfaces\dice_genetic.py in _generate_counterfactuals(self, query_instance, total_CFs, initialization, desired_range, desired_class, proximity_weight, diversity_weight, categorical_penalty, algorithm, features_to_vary, permitted_range, yloss_type, diversity_loss_type, feature_weights, stopping_threshold, posthoc_sparsity_param, posthoc_sparsity_algorithm, maxiterations, verbose)
260         self.start_time = timeit.default_timer()
261
--> 262         features_to_vary = self.setup(features_to_vary, permitted_range, query_instance, feature_weights)
263
264         # Prepares user defined query_instance for DiCE.
~\anaconda3\lib\site-packages\dice_ml\explainer_interfaces\explainer_base.py in setup(self, features_to_vary, permitted_range, query_instance, feature_weights)
104         else: # compute the new ranges based on user input
105             self.feature_range, feature_ranges_orig = self.data_interface.get_features_range(permitted_range)
--> 106         self.check_query_instance_validity(features_to_vary, permitted_range, query_instance, feature_ranges_orig)
107
108         # check feature MAD validity and throw warnings
~\anaconda3\lib\site-packages\dice_ml\explainer_interfaces\explainer_base.py in check_query_instance_validity(self, features_to_vary, permitted_range, query_instance, feature_ranges_orig)
114         for feature in self.data_interface.categorical_feature_names:
115             if query_instance[feature].values[0] not in feature_ranges_orig[feature]:
--> 116                 raise ValueError("Feature", feature, "has a value outside the dataset.")
117
118             if feature not in features_to_vary and permitted_range is not None:
ValueError: ('Feature', 'Invariant: Montagne', 'has a value outside the dataset.')

@gaugup And the code is :
query_instances_lait = x_train[2:3]
genetic_lait = exp_genetic_lait.generate_counterfactuals(query_instances_lait,
total_CFs=2,
features_to_vary=['Positionnement: lait_vendu_par_sfp', 'has a value outside the dataset'],
desired_range=[1.5,2])
genetic_lait.visualize_as_dataframe(show_only_changes=True)

@gaugup  It is now working, I transform the dummy variable into a categorical one and declare it as such, before running my model and generating the counterfactuals.
Thanks

great. Was this due to documentation not clear?
Could you close the issue if there is nothing actionable to do in this issue?
Regards,

Yes, the documentation was not clear on how to handle dummy variables, but it turns out that one has to transform them into categorical variable before generating the counterfactuals.


Turn off Mailchimp user tracking

https://trello.com/c/Qk3SHoov/
We no longer need to track opens/clicks for individual users on the Mailchimp DOS Opportunities email campaigns (see ticket for details).
We will still be able to track anonymised clicks per campaign via analytics, using the utm suffix on the links.

For future reference, could you post a link to the Mailchimp documentation where you found the meaning of thse settings? I'm struggling to find it with Google.

For future reference, could you post a link to the Mailchimp documentation where you found the meaning of these settings? I'm struggling to find it with Google.

Here's the Mailchimp API reference - it's under campaigns then you have to expand the tracking. It's not obvious and you can't Ctrl+F it 😿


Module does not intsall under prestashop 1.7

This module does not install.

It does, when you download the zip from GitHub, you have to remove -master from the folder name in the zip.
Best regards


API-1835: Update the static pod state controller to use Apply

proof pr at https://github.com/openshift/cluster-kube-apiserver-operator/pull/1749

/hold
holding off for a proof pr.

Lgtm. But needs to pass tests

/test unit

/lgtm

ok, the StaticPodsDegraded condition was properly updated in . this pr can be safely merged.

/hold cancel


Allow user to configure number of threads

When calling build_aomp.sh there is no control over the number of
threads used, it is calculated in aomp_common_vars. For some users
controlling the number of threads might be beneficial. This commit
simply changes the initial name used for discovering the number of
threads and check if NUM_THREADS is already defined by the user, if
not follows the same logic as before.

hi Jorgen
please see: https://github.com/ROCm-Developer-Tools/aomp/pull/216
bikeshedding the spelling of your patch on NUM_THREADS


⚠ Bofrost.lu has degraded performance

In beec691, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1845 ms

Resolved: Bofrost.lu performance has improved in 80fd15e after 9 minutes.


AP performance = '-1.0'

I got the performance as follows:
maskrcnn_benchmark.inference INFO: OrderedDict([('bbox', OrderedDict([('AP', -1.0), ('AP50', -1.0), ('AP75', -1.0), ('APs', -1.0), ('APm', -1.0), ('APl', -1.0)])), ('segm', OrderedDict([('AP', -1.0), ('AP50', -1.0), ('AP75', -1.0), ('APs', -1.0), ('APm', -1.0), ('APl', -1.0)]))])
The performance both on COCO2014 and own datasets is the same. I just changed the data path in path_catalog.py before training.
Is there anyone who has some ideas or the same experience as me?
Thx

hi, please, can you tell me what command you executed to get these results? when i run tools/test_net.py on a custom dataset, i get an error reported in #99

Hi @jpainam , I used the same command you reported in #99.


Create language_level_df.R

closes #1

I am going to write more functions for this package, and I am going to base branches on this branch since this function is useful in them.

I'm reworking the function now to just take long format.

The function language_level_df() now takes ValueTable in a long format instead of wide.
NB, I'm not changing the column "ID" in the ValueTable, which is a combination of Parameter_ID and Language_ID. I am however changing the Language_ID column. If you want,  I can set it  so that it also change sthe  ID column, up to you.

@SimonGreenhill where are the comments I should work through?

here in the binarise branch. I was working over there as those functions seem to be needed there as well.

here in the binarise branch. I was working over there as those functions seem to be needed there as well.

Aha, okay. I'll go over there and work on the two in unison then.


Save database

Hello,
How to restore it on reboot or to save the database when running it in a container?
I'm losing it when I restart the container.
A new database is made on reboot and get this error :
APIError(code=-1013): Invalid quantity.

Hi, sorry for the late reply, i was a total noob w/ docker at the time.
Docker is mainly stateless, but the docker custom script try to mitigate this. Are you using the docker script or doing this by hand?
Please note that the docker method is currently unmaintained as i am still near a noob and I don't see any real advantage of using docker over virtualenv


ARROW-11518: [C++][Parquet] Fix buffer allocation in parquet reader

9432

 Quality gate: Passed
3 Issues

 3 Code issues

Details


(WIP) K8S jobs

Work in progress
The aim of this PR is to be able to switch between docker jobs (through docker api via mounted socket) to k8s jobs (through k8s api via network).

[ ] Settings parameter to switch between docker / k8s task job
[ ] Build image through kaniko
[ ] Launch k8s pods instead of pure docker container
[ ] Mount k8s volumes for ML materials instead of docker binded volume

Close PR as it will not be merged. We surely need to reflect on this feature !


[docs] Demo SearchIcons crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

Visit https://mui.com/components/material-icons/?query=what
??
demo SearchIcons crashes

Your Environment

Tech
Version

MUI
v5.2.5

netlify deploy
https://61c098e520eae9000736c0df--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.71 Safari/537.36

@Regiss05  Please hard refresh the docs page. The latest version is v5.2.8.


Added package io.github.axisangles.ktmath

Added new kotlin math library, io.github.axisangles.ktmath, to replace jME3.math
includes Quaternion, Matrix3, Vector3, and EulerAngles.

Contains mathematically propper operator overloads for matrices, vectors and quaternions.
Has a small focus on numerical stability, trying to guarantee at least 6 sig figs on most operations.
Contains several improvements over standard algorithms for converting to eulerangles.
Provides 6 orders of eulerangles.
Provides for homomorphic conversion from quaternion to matrix.
Provides for matrix orthonormalization.

Why not include as a lib or a fork?

(through gradle)

We discussed it and it's worth keeping it in here XD. Gonna start reading :P


commons/colors.tex not found in package

Describe the bug
I installed moderncv using tlmgr, which says the package repo is https://ctan.math.washington.edu/tex-archive/systems/texlive/tlnet. Compiling any document results in an error of commons/colors.tex not found.
After some quick investigation, I found the commons in the directory for documents (some-path/doc/latex/moderncv in my case), rather than where it should be (some-path/tex/latex/moderncv in my case). So I assume maybe something's wrong with packaging for distribution?
Expected Behavior
Actual Behavior
Minimum Working Example to Reproduce the Problem
Specifications

moderncv version and source: v2.4.0
LaTeX compiler (pdfLaTeX, XeLaTeX, etc.) and version:
Platform (operating system, LaTeX editor, etc.): WSL2 with Ubuntu 22.04

Additional context

The package is located at https://mirrors.ctan.org/macros/latex/contrib/moderncv/ and it seems that the folder structure matches the structure of the uploaded .tar.gz:
tar -tvf moderncv-v2.4.0-0-ge8a2096.tar.gz 'moderncv/commons/colors.tex'
-rw-rw-r-- root/root      1516 2024-07-10 07:34 moderncv/commons/colors.tex

This in turn matches the directory structure of the source code (besides the moderncv prefix created with the tar archive).
May I ask if you are running tlmgr in usermode? Also did you install texlive manually or from the Ubuntu repository?
I am trying to understand your current setup in case different versions of texlive or tlmgr could be the reason for that problem?

After updating the moderncv package using tlmgr, I'm seeing the same issue:
l.25 \input{commons/colors}
                           ^^M
/usr/local/texlive/2024/texmf-dist/tex/latex/moderncv/moderncviconsawesome.sty:25:  ==> Fatal error occurred, no output PDF file produced!
Transcript written on lebenslauf.log.
Latexmk: Missing input file 'commons/colors.tex' (or dependence on it) from following:
  ! LaTeX Error: File `commons/colors.tex' not found.

Looking at the tar file from my mirror, the whole commons/ directory seems to be missing: https://mirror.physik.tu-berlin.de/pub/CTAN/systems/texlive/tlnet/archive/moderncv.r71767.tar.xz. This corresponds to the contents of my /usr/local/texlive/2024/texmf-dist/tex/latex/moderncv directory, which is also missing that directory.
Specifications

moderncv version and source: 2.4.0
LaTeX compiler (pdfLaTeX, XeLaTeX, etc.) and version: pdfTeX 3.141592653-2.6-1.40.26 (TeX Live 2024)
Platform (operating system, LaTeX editor, etc.): macOS 14.5, Latexmk 4.85, VSCode 1.91.1, LaTeX Workshop v10.1.0

Thanks to everyone for reporting this issue.
The problem is now understood: The missing file name prefix for colors.tex was responsible for bundling that file into the documentation archive on ctan (for a more detailed explanation see linked PR).
Once this PR is merged I will immediately create a new patch version 2.4.1 for ctan. So bear with me a little more and the patch will be incoming soon...

I submitted the new patched version v2.4.1 to CTAN just now.
So after the distribution to all mirrors it should be available within the next 24h.


Please add on off to toggle options!

Love the package, and the left-right scroll-through-time toggle. But I only use the package at very specific times (such as when I've stepped away for a moment. It works excellent when I need to regather my thoughts.) Please please please add!!!

I was hoping for a setting or even a keyboard shortcut under packages in
the the menu-bar
On Fri, Feb 23, 2018 at 6:22 AM Mathieu Dutour<EMAIL_ADDRESS>wrote:

not sure what you are asking for, there is already a setting and a command
to disable/enable it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mathieudutour/scroll-through-time/issues/15#issuecomment-367983200,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjDuZ47uCXv4ykDtxODV_f6ZOp3uXDKcks5tXp9agaJpZM4SQrRs
.


Docker Support

It seems that getting the dependencies installed just right for this project is quite tricky, so I decided to write a Dockerfile to eliminate/standardize that headache.
It ended up being fairly straightforward with the final usage going from:
python extract_scenes.py example_scenes.py WarpSquare
to
docker run --rm -v "$PWD"/files:/app/files manim example_scenes.py WarpSquare
which, while longer, doesn't add any more work for the user and could be hidden behind an alias or shell script if desired.
Any thoughts on if this is something you'd be interested in?
Also, I had to revert the requirements.txt changes to using my version of aggdraw and pillow, I couldn't get it to work using your versions.
How do you setup aggdraw and pillow? I can try to make it work with your versions.
Keep up the great work!

thanks for this!

I'm trying the Docker method but I got this error message when trying to execute the last step:
flag provided but not defined: -vpwd/files:/app/files
Thanks

Hmm. Not sure how that happened. Try copy and pasting it. You need to have a space between -v and "$PWD"

Oh, I see. I was trying to use the readme command. Now it's working :)

Ah, yea, I realized that command would fail if you had spaces in the path, I should update that if this PR gets merged.

This is great, however in addition to a Linux -v switch, you might want to document a similar construct for Windows.

@nesteruk true, I don't have any experience with docker on windows, it looks like its just a matter of changing the env var from "$PWD" to "%CD%", is that accurate?

Well I couldn’t get it to work with the quotes placed where you had them. I run it with
docker run -v "//r/:/app/files/movies" …
%CD% is indeed the current directory.
From: Scott Opell<EMAIL_ADDRESS>Sent: 11 September 2016 20:40
To: 3b1b/manim<EMAIL_ADDRESS>Cc: Dmitri Nesteruk<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [3b1b/manim] Docker Support (#7)
@nesteruk https://github.com/nesteruk  true, I don't have any experience with docker on windows, it looks like its just a matter of changing the env var from "$PWD" to "%CD%", is that accurate?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/3b1b/manim/pull/7#issuecomment-246192977 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAcoNrG2i-b_3tCJ4RzHL3dBO-d7qbcTks5qpD0NgaJpZM4Jn4Pr .  https://github.com/notifications/beacon/AAcoNjuy_Xk0PyuH1yqFI8P_xneKVc8Wks5qpD0NgaJpZM4Jn4Pr.gif

Oh, and you completely forgot the LaTeX part in the docker file. That’s actually something I’m struggling with right now – I install blang/latex, but the ordinary latex command doesn’t work for me. Perhaps you could integrate LaTeX in the docker file in the correct way.
From: Scott Opell<EMAIL_ADDRESS>Sent: 11 September 2016 20:40
To: 3b1b/manim<EMAIL_ADDRESS>Cc: Dmitri Nesteruk<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [3b1b/manim] Docker Support (#7)
@nesteruk https://github.com/nesteruk  true, I don't have any experience with docker on windows, it looks like its just a matter of changing the env var from "$PWD" to "%CD%", is that accurate?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/3b1b/manim/pull/7#issuecomment-246192977 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAcoNrG2i-b_3tCJ4RzHL3dBO-d7qbcTks5qpD0NgaJpZM4Jn4Pr .  https://github.com/notifications/beacon/AAcoNjuy_Xk0PyuH1yqFI8P_xneKVc8Wks5qpD0NgaJpZM4Jn4Pr.gif

Is this still working?
This is what I am getting:
$ sudo  docker run --rm -v "$PWD"/files:/app/files manim example_scenes.py WarpSquare
Unable to find image 'manim:latest' locally
Pulling repository docker.io/library/manim
docker: Error: image library/manim not found.
See 'docker run --help'.
Thanks,

@nesteruk What is a good file to generate to test latex support? I can add it in, but the full package is huge so I'd rather just add in exactly what's needed for this project.
@tonypower I haven't submitted it to docker hub so you'll need to build it manually. So clone my fork, then run the docker build command mentioned in my readme. Then that command should work as expected

Hi scottpell,
Thank for your reply.
I cloned your fork, but they only readme file I see is: README.md
which doesn't mention anything about the docker build command.
Sorry, but I am a newbie with docker.
Is there any other readme file?
Thanks,

Here's a link. It should be in there so make sure you have the right repo cloned
https://github.com/scottopell/manim/blob/c2c592679e76d5e5663bed1ce650702648b192f1/README.md#docker-method

Hi scottpell,
Thanks again. This is what I get:
$ pip install requirements.txt
Collecting requirements.txt
Could not find a version that satisfies the requirement requirements.txt (from versions: )
No matching distribution found for requirements.txt
aanjos@bot ~/Downloads/manim $ cat requirements.txt
colour==0.1.2
numpy==1.11.0
Pillow==1.7.8
progressbar==2.3
scipy==0.17.1
tqdm==4.7.1
I am using Mint 18.
Am I doing anything wrong?
Thanks,

Why are you running that command? Docker removes the need to install using pip. Only follow the directions i linked you to

LOL
Thanks once again.
What am I doing stupid right now?
$ docker build -t manim .
unable to prepare context: unable to evaluate symlinks in Dockerfile path: lstat /home/aanjos/Downloads/Dockerfile: no such file or directory
That happens both if I am inside manin or outside of it.
Please be patient ;)
Thanks,

To install LaTeX, just add this line:
RUN apt-get install texlive-full -y
That's all you need to do. Warning: LaTeX installation is massive, around 2Gig.

@nesteruk yea, I realize its big, which is why I said I'd prefer to only include whatever is necessary for this project. Which scene/command uses tex?
and @tonypower, your docker install doesn't seem to be setup correctly. I would post on stackoverflow or something similar and try to fix that issue, its kind of outside of the scope of this pull request/issue.

Well, everything in manim uses TeX. You don't have to use it, but I think
that's the whole point, if you use it for math.
I sell online courses on math so for me it's essential.
On 17 September 2016 at 19:47, Scott Opell<EMAIL_ADDRESS>wrote:

@nesteruk https://github.com/nesteruk yea, I realize its big, which is
why I said I'd prefer to only include whatever is necessary for this
project. Which scene/command uses tex?
and @tonypower https://github.com/tonypower, your docker install
doesn't seem to be setup correctly. I would post on stackoverflow or
something similar and try to fix that issue, its kind of outside of the
scope of this pull request/issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/3b1b/manim/pull/7#issuecomment-247787706, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAcoNoZrgxhF0CbavZ5WfsrR2UTHwVeOks5qrBm8gaJpZM4Jn4Pr
.

--
Dmitri Nesteruk
<EMAIL_ADDRESS>skype: dmitri.nesteruk

How to deal with this problem? It seems the directory is okay but exception still shows:
Exception:
Redefine MEDIA_DIR in constants.py to point to
a valid directory where movies and images will
be written


🐛 [flutterfire_ui] [flutterfire_auth] facebook sign in gives email verification page; email verification message never sent

Bug report
Describe the bug
When using the flutterfire_ui example, click to sign on via facebook. After entering credentials, an email verification screen pops up. This behavior is different than for other providers for example the google provider. No email is ever sent to verify, so cannot use facebook to login. However user does appear in user tree for firebase application.
Steps to reproduce
Steps to reproduce the behavior:

Make sure you have a google cloud account setup / firebase account setup. Need facebook "client id" configured like the documentation suggests: https://firebase.flutter.dev/docs/ui/auth/configuring-providers#facebook
Make the flutterfire_ui example work. For example I put a flutterfire configuration into the example folder / application of the example source tree. This made the example run.
flutter run -d chrome

As an aside: Note that upon startup I see an error related to the email verification screen
flutter run -d chrome
Launching lib/main.dart on Chrome in debug mode...
../lib/src/auth/screens/email_verification_screen.dart:97:22: Warning: Operand of null-aware operation '!' has
type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart'
 ('../../../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
    SchedulerBinding.instance!.addPostFrameCallback(_sendEmailVerification);
                     ^
Waiting for connection from debug service on Chrome...             39.9s
This app is linked to the debug
service:
ws://<IP_ADDRESS>:40585/PNAA4hgmnmU
=/ws
Debug service listening on
ws://<IP_ADDRESS>:40585/PNAA4hgmnmU
=/ws

Click sign on with facebook. Enter valid account credentials that are allowed to sign in with.

Email verification screen displayed. No email sent. Unable to login via facebook.

Expected behavior
If we display an email verification screen that says to check our email, then email will be sent.
Details in the flutterfire_ui documentation will describe the email verification workflow.

you need to add scope of email, did you add it?
you also can use
https://pub.dev/packages/eartho_one
which managing everything for you, with ready ui and logic.

Hi @colin-social, Thanks for filing the issue. Can you please confirm if above comment helps resolve the issue?

you need to add scope of email, did you add it?

Hello,
It's not clear what you are suggesting. What is the scope of email? Where should it be added? Is it documented somewhere?

you also can use
https://pub.dev/packages/eartho_one
which managing everything for you, with ready ui and logic.

When I take a look at it, it looks like a separate project. It looks to be a commercial project and seperate from flutterfire. At the current time I would like to evaluate flutterfire.

@maheshmnj @dvird in the bug report note the error in the email verification screen as shown in the output. Could this be causing the issue:
../lib/src/auth/screens/email_verification_screen.dart:97:22: Warning: Operand of null-aware operation '!' has
type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart'
 ('../../../../flutter/packages/flutter/lib/src/scheduler/binding.dart').
    SchedulerBinding.instance!.addPostFrameCallback(_sendEmailVerification);

Then perhaps what needs to be done is the following:
https://firebase.flutter.dev/docs/ui/auth/building-a-custom-ui#building-a-custom-sign-in-screen

Looking at this again. I deleted the existing user (because the gmail address is shared and I used google sign-in). This allowed me to look at the bug again. I missed the following that is output to the dart terminal when the "Verify Your Email" screen is displayed
Error: Error: installApp
property must be a boolean when
specified.

    at
    Object.createErrorWithStack
    (http://localhost:46697/dart
    _sdk.js:5093:12)
    at Error._throw
    (http://localhost:46697/dart
    _sdk.js:20399:18)
    at Error.throwWithStackTrace
    (http://localhost:46697/dart
    _sdk.js:20396:18)
    at
    async._AsyncCallbackEntry.ne
    w.callback
    (http://localhost:46697/dart
    _sdk.js:40921:18)
    at Object._microtaskLoop
    (http://localhost:46697/dart
    _sdk.js:40778:13)
    at _startMicrotaskLoop
    (http://localhost:46697/dart
    _sdk.js:40784:13)
    at
    http://localhost:46697/dart_
    sdk.js:36261:9

Looking this up and I see a reference to a closed bug:
https://github.com/firebase/flutterfire/issues/3581

I created a facebook developer account. The facebook account has a gmail account entered as it's email. It seems like it would take a bit of effort to try to create another facebook developer account and email and it should just work

Could this be causing the issue:

No, that warning is related to null safety and in no way related to this,
Can you check if you get any error in chrome console when the email sending fails?

I posted the details above but also made a screen capture

https://imgur.com/a/LNN0HnH

Thanks for the info @colin-social, Labeling the issue for further insights from the team.

I can't reproduce this issue on the latest version of the firebase_ui.
The reason why email verification is required for facebook, but is not required for google is that firebase treats some OAuth providers as "trusted" and doesn't require to verify email. Facebook is not a trusted provider.
I was able to sign in using Facebook, got verification email, got to the profile screen after email verification.
I assume this was fixed in #9343


boost_plugin_loader: 0.2.1-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository boost_plugin_loader to 0.2.1-1:

upstream repository: git@github.com:tesseract-robotics/boost_plugin_loader.git
release repository: https://github.com/tesseract-robotics-release/boost_plugin_loader-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.2.0-1

boost_plugin_loader
* Improved error messaging (#14 <https://github.com/marip8/boost_plugin_loader/issues/14>)
* Only catch plugin loader exception
* Fix issue not using library names returned from getAllLibraryNames
* Contributors: Levi Armstrong, Michael Ripperger

@marip8 a rebase should fix the failing CI job.

@marip8 a rebase should fix the failing CI job.

Done


Make the "Report a bug" and "Request a Feature" buttons more prominent

Is your feature request related to a problem?
It's not visible right away if someone is having a problem.
Describe the solution you'd like
Either fixed on one of the sides or top of the editing window.

It is already quite prominent, so we keep it like this for now, but thanks for the suggestion


Creating a Bitmap from a ZipArchiveEntry stream throws an error

Describe the bug
Heyo! When using Avalonia.Media.Imaging.Bitmap, using new Bitmap() on a Stream provided by a System.IO.Compression.ZipArchiveEntry throws the error "Unable to load bitmap from provided data". In my testing on the same file, System.Drawing.Bitmap works with no issues.
To Reproduce
I have the following code:
using ZipArchive archive = ZipFile.OpenRead(path);
foreach (ZipArchiveEntry entry in archive.Entries)
    if (entry.Name == "cover.jpg")
    {
        CoverImage.Source = new Bitmap(entry.Open()); //Error is thrown here
        break;
    }

I have also tried Bitmap b = new Bitmap(entry.Open()); instead with the same result.
Expected behavior
No response
Avalonia version
11.1.4
OS
No response
Additional context
No response

I can reproduce. As a workaround, copying to a MemoryStream beforehand works for me. Probably still a bug though so keep the issue open.
eg.
using var entryStream = entry.Open();
using var memory = new MemoryStream();
entryStream.CopyTo(memory);
memory.Seek(0, SeekOrigin.Begin);

CoverImage.Source = new Bitmap(memory);

For reproducing:
using ZipArchive archive = ZipFile.OpenRead(@"D:\avalonia-logo.zip"); // Fixup path
foreach (ZipArchiveEntry entry in archive.Entries)
{
    if (entry.Name == "avalonia-logo.png")
    {
        using var entryStream = entry.Open();
        using var bitmap = new Bitmap(entryStream);
    }
}

avalonia-logo.zip
The culprit could be an unsupported scenario in SkiaSharp:
https://github.com/AvaloniaUI/Avalonia/blob/a04af7cebf3d49b81c228cbb6c18eb2e860ada58/src/Skia/Avalonia.Skia/ImmutableBitmap.cs#L24-L30
I don't know the SK* APIs well, but Avalonia could write some specialized code for DeflateStream. It lacks certain capabilities such as Position, Length, and seeking.


Add logical planer and select caching

This commit adds the first version of a logical planner with the
capability to unify matchers between different selectors.
The MergeSelectsOptimizer traverses the AST and identifies the most
selective matcher for each individual metric. It then replaces
less selective matchers with the most selective one, and adds an additional
filters to ensure correctness.
The physical plan can then cache results for identical selectors which leads
to fewer network calls and faster series retrieval operations.

I added a benchmark for a binary query like
sum(http_requests_total{code="200"}) / sum(http_requests_total)

Here are the results:
goos: darwin
goarch: arm64
pkg: github.com/thanos-community/promql-engine/engine
BenchmarkMergeSelectorsOptimizer
BenchmarkMergeSelectorsOptimizer/withoutOptimizers
BenchmarkMergeSelectorsOptimizer/withoutOptimizers-8         	      37	  31050148 ns/op	44220461 B/op	  275174 allocs/op
BenchmarkMergeSelectorsOptimizer/withOptimizers
BenchmarkMergeSelectorsOptimizer/withOptimizers-8            	      40	  27853710 ns/op	40639897 B/op	  225706 allocs/op
PASS

I would expect the optimization to work even better in Thanos where we fetch data over the network. With a local TSDB, I believe we can reference the same series without copying them in both the nominator and denominator. In Thanos, both selectors would hold copies of the data which would increase memory usage even more, compared to not having the optimization.
I also added an explicit flag to enable/disable optimizers. Should this be set to true or false by default?

@yeya24 yes https://github.com/thanos-io/thanos/issues/4407 should be fixed by this. We don't cache decoded samples, this is hard because we decode chunks in time-based steps. But we do cache fetched series, which means encoded chunks will be cached. Maybe as a next improvement, we can figure out how to only decode shared chunks once and reuse them in all operators that need them.

I am wondering if we need the flag for optimization? In which case we want to disable it?

Ok, let's go with that. I will change the flag to DisableOptimizers. The use case I see is for debugging purposes if some queries return wrong results.

Ok, let's go with that. I will change the flag to DisableOptimizers. The use case I see is for debugging purposes if some queries return wrong results.

Maybe this could also cover https://github.com/thanos-community/promql-engine/issues/66?

Thanks everyone for the comments, they should all be resolved now. Let's merge this and start playing with it.

💪🏽


Eventually we want to have support for a SYNONYM_OF relationship

This should be drawable between characters and possibly? states.

I know you've talked about this in the background. Could we possible add this to the agenda just so that non-programers can have an understanding of what functionality this provides/limits?

That's fine, let's add it to the google docs agenda for next week, but I'm going to close this as a duplicate of #55


Emoticon to differentiate idea (memos plugin) and event (big calendar plugin)

An optional configuration can be interesting to differentiate ideas and events.
For example, putting 💡 before an idea and 📅 before an event.

Yes, you can do it right now by adding css, I will push an update to let you choose which emoji you want before the specific journal

like, you can use this now:
.rbc-event.TODO {
    background-color: #660066
}

.rbc-event.TODO .rbc-event-content::before {
    content: "📮 "
}

.rbc-event.DONE {
    background-color: #006600
}

.rbc-event.DONE .rbc-event-content::before {
    content: "✅ "
}

.rbc-event.JOURNAL {
    background-color: #7A7A7A
}

.rbc-event.JOURNAL .rbc-event-content::before {
    content: "📅 "
}

Thanks!


Add support for multiple hosts in ingress

This is a breaking change so we need to bump to a major release.
Removed old:

ingress.host
ingress.path

and introduced:

ingress.hosts
ingress.hosts[0].paths

Issue: https://github.com/runatlantis/helm-charts/issues/7

Can we do this in a non-backwards breaking way? If host is not empty then use it, otherwise iterate through hosts.

hey @lkysow sure it's possible just it adds a bit more complexity and it could be odd in the sense that there is already the hosts. But it's definitely feasible

@lkysow have one more look. Right now we still support the old single host if it's set

@lkysow any update on this?


using "gap" property in <j:HGroup> is ignored or throw error (SDK0.9.7 royale-asjs_jsonly #1053)

Hi (...Carlos ?),
Here is the test case :
<j:Application xmlns:fx="http://ns.adobe.com/mxml/2009"
	xmlns:j="library://ns.apache.org/royale/jewel"
	xmlns:js="library://ns.apache.org/royale/basic"
	xmlns:local="*" >

	<j:valuesImpl>
		<js:SimpleCSSValuesImpl />
	</j:valuesImpl>

	<j:initialView>
		<j:View width="100%" height="100%">
			<j:HGroup gap="40">
				<j:Button text="BT1"/>
				<j:Button text="BT2"/>
			</j:HGroup>
		</j:View>
	</j:initialView>
</j:Application>

Found issue when using gap property :
if set to 20, then it seem have no effet
if set to 40, then throw following error

Regards

Hi Fred,
thanks I found a bug since the limit value to pass is "10" but I was showing other one.
I committed now the fix.
Thanks
Carlos


Changed Eigen include in CMakeLists.

Eigen3.4 made problems.
The changed include should fix the issue.

@yannickBurkhardt could you approve this?


Feature request: visual rendering of ordered2, write pick of ordered1

I tried to move our platform to using ordered2 instead of ordered1 and realized since nearly all our objects are partially transparent, none are pickable anymore.
It would be useful for us to be able to pick the front most object, even if fully transparent, when blending in ordered2. Ordered2 2 gives the unique advantage of being to set the blending based on the shader code itself.
We often use fully transparent object as overlays but still want the user to interact with them.
Thanks for the consideration.
Would this mode be called something new?
Ps. I tried to add the write pick to the simple transparency pass but it didn't seem to update the write pick.

One challenge we talked about before in the pick_write merge request was how one would handle overlapping transparent objects.

Idea 1: Random.
Idea 2: Use the depth. buffer this might requie the ideas from xref: https://github.com/pygfx/pygfx/issues/748

I was down on myself for not being able to figure this out.
I think that the second commit in https://github.com/pygfx/pygfx/pull/781 allows for write picking and "ordered2" rendering.
However, I understand that this "other option" might be too much maintenance in light of the competing architectural approach from #748 so feel free to completely ignore it. Leaving it here for onlookers to see.
I'll probably vendor the subclassing required to make it work for our application.

I think that if we had access to a function
def set_custom_blend_mode(self, blend_mode, blender):

in WgpuRenderer it would help avoid the need for my other PR.


Paging Example is broken

The Paging Example at: http://listjs.com/examples/paging.html isn't working with the Chrome Console showing:
GET http://listjs.com/plugins/paging/list.paging.js 404 (Not Found)
-Neville

Damn, thanks! It's fixed now!


Migration Guide Please - Foundation 5 - 6

A migration guide is needed sooner rather than later.

+1

+1 (x 1000)

+1

Rolling this into #7246. We're still working on it!

Has a 5 to 6 guide been created yet?

Migrating from v4 to v5
Migrating from v5 to v6 (unofficial compilation)
All Foundation releases

@ncoden thanks! these are actually resources I have compiled on my on as well. Do you happen to know if there are plans for an 'official' guide?

Highly doubt an "official" guide will happen.
My best advice? Don't migrate an existing project to a new version of Foundation unless you're planning a redevelopment/redesign. There's really not much of a point.

Yes, we tried to migrate from 5 to 6 and after a week of effort, we gave up and moved back to Foundation 5 :(
We will eventually migrate to Bootstrap4 as it seems easier than upgrading to Foundation6.

See https://getbootstrap.com/docs/4.0/migration/
Migrating now to a totally different framework is much more work.


Enable recycling of rids on graph database

Provide flag which can be overwritten on storage level, whether or not storage should recycle deleted rids. It should be set to "recycle rids" values for graph database.

Not relevant anymore.


p5 nutshell: Suggest .nextsame and .nextwith once they've been implemented in Rakudo.

Original context:
https://github.com/perl6/doc/commit/97c24a51878b27797dcd9f06645438adaaa68def#diff-41d8ba5d58c8de99795c9c3779f777f6R233

They are already there, and here are a few examples to tap from.


Update Readme

The TYPO3 client now uses sentry/sdk

I'm wondering if it's worth keeping listing the package also in the "3rd party integrations using old SDK 1.x" section for users still looking for the old SDK...

@ste93cry we can discuss that in a separate PR. Anyway, since the readme on master is the first document that a user reaches here, I would prefer to leave it.


Change the import path for the macaroon bakery to use a new v3 version.

The bakery v3 version at github.com/go-macaroon-bakery/macaroon-bakery/v3 picks up mgo from the juju fork github.com/juju/mgo/v2 instead of `gopkg.in/mgo.v2.
The juju version has a few bug fixes which are not upstream.
In theory there'd be a go mod file and the version would go to v2 - the reason for this PR is that the bakery library uses an init() function to register a protocol type - if both gopkg.in/mgo.v2 and github.com/go-macaroon-bakery/macaroon-bakery/v3 are in use then both attempt to do the registration and there's an error logged (this will become a panic in the future). So we need this lxc library and any downstream users to be using the same bakery version.
Did we want to do the go mod conversion to accommodate this change?

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Looks like this PR is making CI very sad...
Also, why is bakery pulling in mgo? Any chance packages can be split in a way that the client doesn't pull in the server?

Looks like this PR is making CI very sad...
Also, why is bakery pulling in mgo? Any chance packages can be split in a way that the client doesn't pull in the server?

bakery has a root key store which can be backed by either mongo or postgresql, so it pulls in those db libraries for that purpose.
Ideally client and server would be separate so additional changes to the macaroon bakery library would be needed.
The CI is failing due to failing to pull the new bakery v3 dependency at compile time. It worked locally so I'll need dig a bit deeper to see what's happening.

@stgraber the build issue breaking CI tests was that lxc/lxd does not explicitly maintain a go mod file so that when go mod init is run as part of the build step, it only looks for @latest which pulls the default repo branch. For macaroon-bakery this is now been set to v3 so it works.
The postgresql and mongo stuff is all in a sub package which lxd/lxc does not import so there's no pressing need right now to split things out if we can land this change to unblock things.

@wallyworld you still need to sign off your commit before we can merge this :)

jenkins: test this please

https://jenkins.linuxcontainers.org/job/lxd-github-pull-test/3022/
For some reason Github didn't auto-update for it...

@stgraber I've change the macaroon bakery mod file to set the module path to use gopkg.in again (v3 in place of v2). This should allow a non go mod build to complete.

jenkins: test this please

@wallyworld looks like we're getting closer but still not quite there.


Build errors in watchOS and tvOS

watchOS
This error came up where you were calling an extension to AppKitOrUIKitHostingControllerProtocol. I noticed that most of the file is hidden from watchOS, except for this extension, though. You can probably solve this by hiding the extension from watchOS.
#if !os(watchOS) // Add this
extension AppKitOrUIKitHostingControllerProtocol {
    // ...
}
#endif // And this

tvOS
This one came up at if base?.isPagingEnabled ?? false (Under _UIHostingScrollViewRootView).
Errors
watchOS
.../Sources/Intermodular/Helpers/AppKit or UIKit/AppKitOrUIKitHostingControllerProtocol.swift:38:11: Use of undeclared type 'AppKitOrUIKitHostingControllerProtocol'

tvOS
.../Sources/Intermodular/Helpers/UIKit/UIHostingScrollView.swift:192:22: 'isPagingEnabled' is unavailable in tvOS

@BenSova fixed, thank you for raising the issue!


Decouple MongoProvider from MongoDB charm

Introduction
At present MongoProvider makes calls to methods in MongoDB charm. This needs to be decoupled using custom events.

fixed in current main branch


Heavenly Clouds zone: change Flighty to remove itself after being triggered and to use a tooltip instead of adding to the card description

This is the same way Electric (from the Storm zone) does it, and I think the UX is better. You don't get cards with super tiny text, and it's clear when the mod has been triggered already this combat.

Yeah, I think this is the way, thanks for doing that.


Remove tags feature

Pls rmb check that ALL functions are working
Fixes #160

Codecov Report

Merging #246 (65220ea) into master (6cfd3bb) will decrease coverage by 0.94%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master     #246      +/-   ##
============================================
- Coverage     69.89%   68.94%   -0.95%
+ Complexity      590      569      -21
============================================
  Files           100       98       -2
  Lines          2182     2109      -73
  Branches        181      174       -7
============================================
- Hits           1525     1454      -71
  Misses          566      566
+ Partials         91       89       -2

Impacted Files
Coverage Δ

src/main/java/donnafin/logic/PersonAdapter.java
100.00% <ø> (ø)

src/main/java/donnafin/logic/parser/CliSyntax.java
91.66% <ø> (-0.65%)
:arrow_down:

...rc/main/java/donnafin/logic/parser/ParserUtil.java
84.88% <ø> (-1.72%)
:arrow_down:

.../main/java/donnafin/model/util/SampleDataUtil.java
80.55% <ø> (-3.89%)
:arrow_down:

src/main/java/donnafin/ui/PersonCard.java
86.66% <ø> (-2.23%)
:arrow_down:

src/main/java/donnafin/model/person/Person.java
86.84% <50.00%> (-1.26%)
:arrow_down:

...n/java/donnafin/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

.../main/java/donnafin/storage/JsonAdaptedPerson.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


OAK-8633: Add warn logs if we add/update a string property larger than 100KB

I think we should simply allow config via command line params - that'd significantly reduce the plumbing to move configured value to intended location.

@catholicon yes we can enable these via command line params  and that will reduce plumbing but in my opinion we should only have those params via command line which have major impact over application. The changes presented here are not that that big to have these enabled via commandline params. Also with osgi config framework, we can persist these and we don't have to add command line params each time  we restart application.

@catholicon  suggested changes are incorporated in PR #155 .
Closing this PR.


Work with IT to access email.gwu.edu

You can contact IT through<EMAIL_ADDRESS>or by visiting them on the third floor of Himmelfarb library.

Hi @MiguelMazumder has IT been able to help you with this?


Unpacked files are not being deleted after import

Unpacked files are extracted, imported but are not being deleted, Log:
2022/01/05 08:13:46.633034 sonarr.go:116: [Sonarr] Updated (http://<IP_ADDRESS>:8989): 1 Items Queued
2022/01/05 08:13:46.633104 sonarr.go:151: [DEBUG] Sonarr (http://<IP_ADDRESS>:8989): downloading (torrent:5%): FBI.International.S01E09.1080p.WEB.H264-CAKES (Ep: 24261)
2022/01/05 08:13:46.633148 logs.go:143: [Unpackerr] Queue: [0 waiting] [0 queued] [0 extracting] [0 extracted] [0 imported] [0 failed] [0 deleted]
2022/01/05 08:13:46.633179 logs.go:145: [Unpackerr] Totals: [0 retries] [0 finished] [0|0 webhooks] [0|0 cmdhooks] [stacks; event:0, hook:0, del:0]
2022/01/05 08:14:46.623129 logs.go:143: [Unpackerr] Queue: [0 waiting] [0 queued] [0 extracting] [0 extracted] [0 imported] [0 failed] [0 deleted]
2022/01/05 08:14:46.623217 logs.go:145: [Unpackerr] Totals: [0 retries] [0 finished] [0|0 webhooks] [0|0 cmdhooks] [stacks; event:0, hook:0, del:0]
2022/01/05 08:15:46.626561 radarr.go:117: [Radarr] Updated (http://<IP_ADDRESS>:7878): 0 Items Queued
2022/01/05 08:15:46.635677 sonarr.go:116: [Sonarr] Updated (http://<IP_ADDRESS>:8989): 1 Items Queued
2022/01/05 08:15:46.641935 handlers.go:75: [Sonarr] Waiting for Start Delay: FBI.International.S01E09.1080p.WEB.H264-CAKES (1m0s remains)
2022/01/05 08:15:46.641997 sonarr.go:151: [DEBUG] Sonarr (http://<IP_ADDRESS>:8989): completed (torrent:100%): FBI.International.S01E09.1080p.WEB.H264-CAKES (Ep: 24261)
2022/01/05 08:15:46.642490 handlers.go:59: [DEBUG] Sonarr: Status: FBI.International.S01E09.1080p.WEB.H264-CAKES (Waiting, pre-Queue, elapsed: 0s)
2022/01/05 08:15:46.642559 logs.go:143: [Unpackerr] Queue: [1 waiting] [0 queued] [0 extracting] [0 extracted] [0 imported] [0 failed] [0 deleted]
2022/01/05 08:15:46.642589 logs.go:145: [Unpackerr] Totals: [0 retries] [0 finished] [0|0 webhooks] [0|0 cmdhooks] [stacks; event:0, hook:0, del:0]

It's currently 08:27 and the file's still there.
my unpackerr.conf:
debug = true
quiet = true
log_file = "/config/unpackerr.log"
log_files = 10
log_file_mb = 10
interval = "2m"
start_delay = "1m"
retry_delay = "5m"
parallel = 1
file_mode = "0777"
dir_mode = "0777"

[[sonarr]]
  url = "http://<IP_ADDRESS>:8989"
  api_key = "XXX"
  paths = ["/DATA01/downloads/complete/SONARR"]
  protocols = "torrent"
  timeout = "10s"
  delete_delay = "5m"
  delete_orig = false

[[radarr]]
  url = "http://<IP_ADDRESS>:7878"
  api_key = "XXX"
  paths = ["/DATA01/downloads/complete/RADARR"]
  protocols = "torrent"
  timeout = "10s"
  delete_delay = "5m"
  delete_orig = false

Using the hotio/unpackerr docker image.

Unpacked files are extracted, imported but are not being deleted, Log:

That's less than 1 second of log lines, and I do not see anything being extracted here. In fact, according to the totals, nothing has been extracted nor imported.

It's currently 08:27 and the file's still there.

Which file(s)?

previous log lines were just repeats of this:

2022/01/05 08:07:46.629235 radarr.go:117: [Radarr] Updated (http://<IP_ADDRESS>:7878): 0 Items Queued
2022/01/05 08:07:46.635192 sonarr.go:116: [Sonarr] Updated (http://<IP_ADDRESS>:8989): 0 Items Queued
2022/01/05 08:07:46.635266 logs.go:143: [Unpackerr] Queue: [0 waiting] [0 queued] [0 extracting] [0 extracted] [0 imported] [0 failed] [0 deleted]
2022/01/05 08:07:46.635286 logs.go:145: [Unpackerr] Totals: [0 retries] [0 finished] [0|0 webhooks] [0|0 cmdhooks] [stacks; event:0, hook:0, del:0]
2022/01/05 08:08:46.625636 logs.go:143: [Unpackerr] Queue: [0 waiting] [0 queued] [0 extracting] [0 extracted] [0 imported] [0 failed] [0 deleted]
2022/01/05 08:08:46.625718 logs.go:145: [Unpackerr] Totals: [0 retries] [0 finished] [0|0 webhooks] [0|0 cmdhooks] [stacks; event:0, hook:0, del:0]

fbi.international.s01e09.1080p.web.h264-cakes.mkv was most certainly extracted to the same folder as the archives
the above extracted file...
I agree that the log is a bit light on details, thus this ticket.

Can you share the log file, please? Do you see any files or folder with unpackerr in their name? What makes you believe this application created that mkv file? (That should be in the log file.)

While working on this I've discovered the tool i used to monitor the logs (Klogg) somehow prevented the log file from being written.
I'm monitoring using Notepad++ atm, and will share a fresh log next time.

When unpackerr extracts something it also writes a txt file. Do you see that file? The logs from 8:15 to 8:20 are all we need...


Clarify connector type for SQL Server

I was trying to set this up as a demo and ran into some confusion with the SQL Server connector. By default the list of connectors shows the built-in ones, and the one for SQL Server doesn't support data gateways.

@eugman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@eugman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@eugman, thanks! I'll approve and sign off now.

#sign-off


error to use PoN(panel of normal) as background for CNVkit

I made a PoN by using CNKkit below,
pon_build-cnvkit-background.cnn
Then I tried to use this *.cnn file as background (PoN) to running CNVkit, but It did not run on them. (It did run well without *.cnn )
This is yaml file that I used for this,
details:
- algorithm:
    aligner: bwa
    align_split_size: 5000000
    recalibrate: false
    realign: false
    variantcaller: false
    remove_lcr: true
    svcaller: [cnvkit]
    background:
      cnv_reference:
        cnvkit: /PoN/testsample/pon_build-cnvkit-background.cnn
  analysis: variant2
  description: NTP17-004880
  files:
  - ../input/testsanple_R1_001.fastq.gz
  - ../input/testsample_R2_001.fastq.gz
  genome_build: GRCh37
  metadata:
    batch: test
upload:
  dir: ../final

Is this yaml file correct?
As I know, once I made reference file (*.cnn/PoN), I can use it continuously for the different analysis, instead of making PoN  every time again. Is it right?

Thanks for the question and sorry about the issues. Your configuration looks right and that should run CNVkit using your cnn file as background. Could you describe more about what behavior or error you're seeing? I'm not able to replicate a problem using a similar configuration so I must be missing something. Thanks much for the help debugging.

Thanks for following up with the error. It looks from the error message like the file might have been specified as a list in the configuration, something like:
    background:
      cnv_reference:
        cnvkit:  [/CNV/mk_PoN/work/PoN_toy_2/final/testsample/pon_build-cnvkit-background.cnn]

Is that possible? In your example above it's a single item which should work correctly, but I can replicate the same error if I make the input a list. Hope this explains and resolves the problem.

@chapmanb
Thanks for your following up.
I double checked my yaml file. It did not specify the path of *cnn as a list.
background:
      cnv_reference:
        cnvkit: /CNV/mk_PoN/work/PoN_toy_2/final/testsample/pon_build-cnvkit-background.cnn

Then, error message was little bit different like below,
/bcbio/stable/anaconda/lib/python2.7/site-packages/bcbio/pipeline/run_info.py in _clean_background(data={'algorithm': {'adapters': [], 'align_split_size': 5000000, 'aligner': 'bwa', 'archive': None, 'background': {'cnv_reference': {'cnvkit': '/CNV/mk_PoN/work/PoN_toy_2/final/testsample/pon_build-cnvkit-background.cnn'}}, 'bam_clean': False, 'coverage_interval': None, 'effects': 'snpeff', 'ensemble': None, 'exclude_regions': [], ...}, 'analysis': 'variant2', 'description': 'NTP17-004880', 'files': ['/CNV/mk_PoN/collect_NS/toy_result...ie/positve/NTP17-004880-021218_S1_R1_001.fastq.gz', '/CNV/mk_PoN/collect_NS/toy_result...ie/positve/NTP17-004880-021218_S1_R2_001.fastq.gz'], 'genome_build': 'GRCh37', 'lane': '1', 'metadata': {'batch': 'test', 'phenotype': ''}, 'rgnames': {'lane': 'NTP17-004880', 'lb': None, 'pl': 'illumina', 'pu': 'NTP17-004880', 'rg': 'NTP17-004880', 'sample': 'NTP17-004880'}, 'upload': {'dir': '/CNV/bcbio_runs/toytest/final', 'run_id': ''}})
    375         if isinstance(val, basestring):
    376             out["variant"] = _file_to_abs(val, [os.getcwd()])
    377         elif isinstance(val, dict):
    378             for k, v in val.items():
    379                 if k in allowed_keys:
--> 380                     out[k] = _file_to_abs(v, [os.getcwd()])
        out = {}
        k = 'cnv_reference'
        v = {'cnvkit': '/CNV/mk_PoN/work/PoN_toy_2/final/testsample/pon_build-cnvkit-background.cnn'}
    381                 else:
    382                     errors.append("Unexpected key: %s" % k)
    383         else:
    384             errors.append("Unexpected input: %s" % val)

In this error message, the path of *.cnn file was not specified as a list.

Thanks for following up with the changed error message. Which version of bcbio are you currently running? Based on the code I can see in the traceback it looks like an older version without support for the background references. This was added in 1.1.3 (https://github.com/bcbio/bcbio-nextgen/commit/be17357a202051e693360591abee7a7c22760f79). Is it possible you're running an older version? Hope this helps explain the issue.

@chapmanb
Thanks for your reply. Currently, I'm using 1.1.2 version. I guess the issue that I had here is due to the 1.1.2 version not supporting the background reference. Let me try newer version for this. Thanks again!

@chapmanb
Version 1.1.5 of bcbio is properly working on this. It recognized *.cnn file as a background reference well! Thanks a lot!

Thanks everyone for working this out!


Setup test data for oil-gas unittesting

there are some scripts and data on nasgul from before. these should be packaged into an azure blob so they can be properly interfaced with github actions.
required for #101

Example data available here: https://pysilcam.blob.core.windows.net/test-data/test-data_oilgas.zip


Setting up a RCP Borderrouter via ot-ctl: No such file or directory

Describe the bug As I am quite new to openthread and setting up thread-mesh-networks, I thought you might help me on an issue I just ran into. I tried to set up a Openthread Borderrouter via Raspberry Pi3B and a nrf52840 USB Dongle as RCP. I essentially followed this tutorial https://github.com/openthread/openthread/issues/5311 and came to the point where the three services

avahi-daemon.service
otbr-agent.service
otbr-web.service

are running when typing sudo systemctl status. Till this point I got no issues, which indicates I set up the Borderrouter correctly.
However, typing sudo ot-ctl dataset init new leads to an error which says "connect session failed: No such file or directory".
This step (sudo ot-ctl dataset init new) is also listed on the official openthread site (https://openthread.io/guides/border-router/external-commissioning?hl=en&comm=ot-commissioner), where it says that you need to form the network on the command line of OpenThread POSIX.
Current working directory was ~/ot-br-posix, maybe the chosen directory was not correct?
I found the root directory of ot-ctl on the Raspberry Pi under: ~/ot-br-posix/build/otbr/third_party/openthread/repo/src/posix, but even if changing the working directory to this path, the above shown command (sudo ot-ctl dataset init new) leads to the same issue (connect session failed: No such file or directory).
At this point I am not sure whether ot-ctl was installed correctly..
I hope that you can help me :)
EDIT: The command sudo nrfutil dfu usb-serial -pkg out.zip -p /dev/ttyACM0 -b 115200 led to an error that says: "No trigger interface found for device with serial number: C8A8AA709BED, Product ID: 0xCAFE and Vendor ID..."
I just copied the command above, perhaps perhaps the hex-file flashed to the Dongle is not correct?

@MarcelloDavis , the "connect session failed: No such file or directory" error indicates that otbr-agent process is not actually running. You can validate whether the otbr-agent process is running by doing something like this:
$ ps -ax | grep otbr-agent
 1874 ?        Ss     0:00 /usr/sbin/otbr-agent -I wpan0 spinel+hdlc+uart:///dev/ttyACM0
 2043 pts/0    S+     0:00 grep --color=auto otbr-agent

The most common reason why otbr-agent is not running is because the RCP is not configured with the correct firmware image.
You can find otbr-agent logs in /var/log/messages, can you provide those logs?

Closing stale issue.

Hello,
I have successfully developed the RCP border router. Now I am trying to connect a thread node which consists of Raspberry pi with nRF52840 dongle with ot-cli-ftd.hex file flashed over it.
But no CLI commands run over it, and I cant connect it with the border router. How can i setup this thread node . Do I need to install the openthread or ot-br-posix on the pi, or any other package

Hello,
I have successfully developed the RCP border router. Now I am trying to connect a thread node which consists of Raspberry pi with nRF52840 dongle with ot-cli-ftd.hex file flashed over it.
But no CLI commands run over it, and I cant connect it with the border router. How can i setup this thread node . Do I need to install the openthread or ot-br-posix on the pi, or any other package

@sohaibbinaltaf , you may find the Thread Border Router codelab helpful.
BTW, please open a new issue for new questions.

@jwhui Thank you


Grid vs spacing

ffe-grid(-react) comes with gutters, which makes it difficult to nest grids or use it in any other context than page layout. This has caused some issues in cases where it has been used for internal layout in components or sections. At the moment ffe-grid is the only alternative for creating layouts, and we need to figure out how to handle this in components.
One suggestion (originally intended to standardize spacing in general) is to remove all hardcoded spacing from the grid and other components, to be replaced by a set of predefined variables throughout the design system. In the case of ffe-grid, could removing the gutters altogether in favor of letting the contents of each column deal with spacing be a possible way forward?
Input wanted!

If we have to go down that road I would much rather see an option to turn the padding off than it being removed. I'd prefer to not have to synchronize spacing across all the components in my app just to have content line up vertically across rows.
Or maybe rename this to ffe-layout and create a new grid or something, though to be honest there are already tons of grid-libraries around one could use if the goal is just to not have to deal with flexbox in components?

What is the reason padding preferred over margins? I see margin collapse as useful for solving some of these issues.

Another suggestion is to add a --no-spacing (or something) modifier to .ffe-grid in order to allow for using the grid in internal/component layouts. That way we won't have to refactor every implementation, but still be able to nest grids within grids.


Bluetooth: Make BT_GATT_ATTRIBUTE macro definition C++ compatible

In C++ designated initializers require that

All members are initialized
Designators used in the expression must appear in the same order as the
data members.

Signed-off-by: Alexey Eremin<EMAIL_ADDRESS>

I suspect we have lots of places in the tree that violate this. To you have some reference to the C++ specification where this requirement is stated?

I suspect we have lots of places in the tree that violate this. To you have some reference to the C++ specification where this requirement is stated?

Not sure about the standard but I am quite sure we’ve fixed this elsewhere in Zephyr already

I suspect we have lots of places in the tree that violate this. Do you have some reference to the C++ specification where this requirement is stated?

See e.g. https://en.cppreference.com/w/cpp/language/aggregate_initialization , Designated initializers.
One way to prevent is from happening is to add more compilation tests (or just add more code into existing one). Let me know if you would prefer me to do that.
Hm, actually there is another place having the same problem: BT_LE_ADV_CONN_NAME macro (and similar ones). They are based on BT_LE_ADV_PARAM which doesn't  initialize id member. Should I add corresponding change to this PR or it's better to send another one?

See e.g. https://en.cppreference.com/w/cpp/language/aggregate_initialization , Designated initializers.

@aeremin maybe I missed it, but I only see that stating "all designators used in the expression must appear in the same order as the data members of T", meaning you don't have to list all struct members. The document even gives an example of where one member was omitted.

@aeremin maybe I missed it, but I only see that stating "all designators used in the expression must appear in the same order as the data members of T", meaning you don't have to list all struct members. The document even gives an example of where one member was omitted.

I think you are actually right. I mixed both things together as I am using older compiler
(arm-none-eabi-gcc.exe (GNU Tools for Arm Embedded Processors 7-2018-q2-update) 7.3.1 20180622 (release) [ARM/embedded-7-branch revision 261907]) which doesn't support "partial" initializers. It's actually even more nuanced:
struct Foo {
  int a, b;
};

Foo x = {.a = 5}; // will compile
Foo y = {.b = 5}; // won't compile

(but my compiler is actually what Zephyr documentation recommends to use on Windows and also the latest one available in chocolatey, so it can be useful to support it)

(but my compiler is actually what Zephyr documentation recommends to use on Windows and also the latest one available in chocolatey, so it can be useful to support it)

Definitely!

Hm, actually there is another place having the same problem: BT_LE_ADV_CONN_NAME macro (and similar ones). They are based on BT_LE_ADV_PARAM which doesn't initialize id member. Should I add corresponding change to this PR or it's better to send another one?

I think we could just add the fix to this PR and also fix all other the instances we find.
bt_le_adv_param::id should be initialized to BT_ID_DEFAULT (0)

(but my compiler is actually what Zephyr documentation recommends to use on Windows and also the latest one available in chocolatey, so it can be useful to support it)

I've updated our Getting Started Guide. You can request the maintainer on Chocolatey to update to the latest:
https://github.com/zephyrproject-rtos/zephyr/pull/23186

@carlescufi Please clarify if we should go forward with this PR then. Given the compiler version support. I'm not sure which way we are going with this one now.

I am also a bit lost: do you want to support 7-2018-q2-update or not? If not - sure, I personally can deal with it and will only leave reordering (and not initializing everything) in this PR.

I am also a bit lost: do you want to support 7-2018-q2-update or not? If not - sure, I personally can deal with it and will only leave reordering (and not initializing everything) in this PR.

I am personally not against supporting 7-2018-q2-update, but if that is the goal then you should explicitly say so in the commit message. We tend not to support old compiler versions, but I am not against supporting it in this case. I only fixed the documentation to clearly state that the latest GNU Arm Embedded toolchain works fine.
So here's my suggestion:
If you need 7-2018-q2-update support, I am OK with this, but you are going to have to fix a lot of other instances of this problem. If you can update to GCC 9 then let's just change the bare minimum required, by amending this PR.


[KOGITO-2978] Run all non native BDD tests in Kogito PR check

Signed-off-by: Karel Suta<EMAIL_ADDRESS>https://issues.redhat.com/browse/KOGITO-2978
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

Jenkins, test this

Jenkins, test this

Jenkins, test this

Jenkins, test this

Jenkins, test this


Most Issues Raised, Solved and Documented Through Discord Chat Room (link)

The "International Crypto Research Group" Discord is a small group of traders / crypto trading bot manufacturers who test mostly using Roibal Bot and Gekko Bots. Within this group is a ton of info on Triangular Arbitrage as well as raised errors and solutions.
A link is available to join here: https://discord.gg/GGMvbCJ

Also be sure to visit youtube channel in order to learn about the functionality of programs: https://www.youtube.com/channel/UCVTnyT4fUxYkvawbggo8-AQ


Problem with sending a few messages in a row with different lengths

Hi @djc @mayfield @bnewbold
I am using nnpy to connect to my robot. It is at the following format:
push = nnpy.Socket(nnpy.AF_SP, nnpy.PUSH)
push.connect(rep_address)
str1 = '{"category":["Main","Tasks"],"name":"Command-1","data":{}}'
push.send(str1)
time.sleep(1)
push.close()
After sending this Command-1 as a message, I send some Command-2 (with a format similar to the above code) to do something else with the robot. But, since the length of the two messages is different (for example the second message is shorter), there will be some unknown characters at the end of the second message when I send it and the robot does not recognize it.
I tried to solve this by adding a lot of spaces at the end of the second message line (because my messages are somehow considered as JSON), but it becomes intractable when the number of commands goes to around 10 (I also tried to make it smarter by comparing the length of each message with the maximum length of the messages being sent before that, and add extra spaces at the end of the message if it's shorter than the max, but still doesn't work all the time!).
So, I thought a better idea could be available for this issue. It seems that there is a buffer-like variable that the messages are being overwritten on that. If so, that could be resolved if I have access to that variable to clear it every time before sending a new message.
Please let me know if you think different, or you have any idea to resolve this issue.
Thanks

@Ahmadreza-Shahrokhshahi I don't think this project or nanomsg itself are actively maintained.  I'm also not involved, but I just wanted to respond in case nobody else does.

@mayfield Thanks for your response. I was wondering if you could come up with some ideas. I can see that there is a function of send in socket.py :

I am trying to clear the buf befor being used each time.

I'm not familiar with ffi.new, but I wouldn't be surprised if there were
a lot of complicated library interop issues with that technique, doubly so
on python2.  If you're on python3, I'd probably just send it "my data".encode() as the data argument and see if that helps.
On Fri, Sep 24, 2021 at 2:36 PM Ahmadreza-Shahrokhshahi <
@.***> wrote:

@mayfield https://github.com/mayfield Thanks for your response. I was
wondering if you could come up with some ideas. I can see that there is a
function of send in socket.py :
[image: image]
https://user-images.githubusercontent.com/44307719/134736610-b4e88ca1-f523-4c12-8183-0c2801418f6a.png
I am trying to clear the buf befor being used each time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nanomsg/nnpy/issues/61#issuecomment-926904762, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCANC4WVWABWAACPZ2R7DUDTOLTANCNFSM5EWR2PHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I'm not familiar with ffi.new, but I wouldn't be surprised if there were a lot of complicated library interop issues with that technique, doubly so on python2. If you're on python3, I'd probably just send it "my data".encode() as the data argument and see if that helps.

I'm using python3. The other problem is that the file socket.py that I mentioned is located at the following address:
${SRC}/venv/lib/python3.6/site-packages/nnpy assuming $SRC is the source of my code, but it seems it is not the file being used in the code when calling nnpy.Socket.send (because I have commented the send function and it worked). Do you have any idea what else could be the location of this socket.py file that is running by the code?


Customiezable service worker path

To support wider scenarios that would be great to make service worker path a parameter. So for example adding different channels for anonymous ad logged in user becomes possible.

Just created a PR for this (#67)

This feature is now live in version v0.0.11.. closing.


Force delete when soft-deletable

Is there any way I can force delete soft-deletable entity?

@simPod, I have answered your question on stackoverflow.
link
You can easily remove and re-add the softdeletable listener temporarily.


ROS 2 bag play error

Hello, i'm trying to go through the tutorial (https://nvidia-isaac-ros.github.io/repositories_and_packages/isaac_ros_visual_slam/isaac_ros_visual_slam/index.html#quickstart): i've set the environment up and met all the requirements. The ROS node and the RViz launch fine, but when i'm trying to launch the rosbag file i'm getting this error:
admin@LOQ-16IRH8:/workspaces/isaac_ros-dev$ ros2 bag play src/isaac_ros_visual_slam/isaac_ros_visual_slam/test/test_cases/rosbags/small_pol_test/
[ERROR] [1701683426.900077380] [rosbag2_storage]: Could not open 'src/isaac_ros_visual_slam/isaac_ros_visual_slam/test/test_cases/rosbags/small_pol_test/small_pol_test_0.db3' with 'sqlite3'. Error: Failed to setup storage. Error: Error when processing SQL statement. SQLite error (26): file is not a database
[ERROR] [1701683426.900103663] [rosbag2_storage]: Could not load/open plugin with storage id 'sqlite3'
No storage could be initialized. Abort

i've tried to fix it installing rosbag2 plugin with apt-get install ros-humble-rosbag2 but this didn't help.
How can i fix it?

@jaiveersinghNV
Great thanks, this helped. Maybe it's necessary to mention git lfs pull command in the tutorial? I can't find neither in the Slam tutorial, nor in the Environment Setup.


Version 2.0 release candidate - I need your help for bug testing and feature requests

Hi all. I wanted to apologies for my inactivity lately and to make an exciting announcement. Version 2.0 is around the corner!
I wanted to get peoples feedback on the v2.0-beta. Have you encountered any bugs using it in your usual workflow? Are there any features you suggest? And for the coders among you, feel free to suggest ways to improve code readability. And remember, pull requests are always welcome.

I just downloaded it and noticed many changes.
for me, the most important feature is now gone: the ability to set the laser on/off commands.
I'm using this extension to generate instructions for plotting, not cutting, so custom commands for on/off were really handy.
What is the difference between the Standard G-Code and the Fan-Controlled G-Code?

The Fan-Controlled-Gcode interface is a standard interface with the only difference being that the laser is powered on using the Fan-power gcode command instead of Tool-power.
But you make a good point. It's not a very intuitive system and I might as well let people choose the command. It's pretty easy, I'll do that tonight.
Thanks for your feedback! :)

thanks, sounds great.
I've also looked into the code, and the underlying library svg-to-gcode can accept a custom header and footer, but neither of the two default G-Code Compilers makes use of it.
I see the debugging code is still there, but I can't seem to access it from Inkscape. Is there an easy way?

I am not sure this is what you are asking but I added documentation about header and footer some weeks ago : https://github.com/JTechPhotonics/J-Tech-Photonics-Laser-Tool/blob/master/README.md#custom-g-code-header-and-footer

I am not sure this is what you are asking but I added documentation about header and footer some weeks ago : https://github.com/JTechPhotonics/J-Tech-Photonics-Laser-Tool/blob/master/README.md#custom-g-code-header-and-footer

I was just looking at the code and saw that the G-Code compilers get called without those parameters set. Since the Custom Header and Footer files were only documented for the old branch I assumed that this feature was gone.

To clarify, the custom header/footer files are missing from the wrapper branch. I'll re-introduce the feature asap. I was planning on doing that last night however a few major bugs got in the way.
If all goes well, tomorrow I'll publish a new release candidate wich isn't missing any features. I'll still need your help finding bugs and useful features before It's ready for a full release.

Ok. I published my changes under the v2.0-beta.2 tag (still in the v2.0 release candidate).
Custom header and footer support was re-introduced as well as debug tracers and reference points. + A bunch of bug fixes.
There are two know bugs, one involves edge cases with ellipses and the other is that transform attributes are ignored.

Ok I had a quick look at it now. I'll need to modify it for my usecase as I have to use completely different commands for "laser on" and "laser off" while the current code only allows changes to the powerlevel. But it should not be too hard to implement.

Attached a svg file link.
https://my.hidrive.com/lnk/1VY0rpwz
With 0.92 the gcode is ok. With 1.01 the gcode is wrong (even negative coordinates), with the new beta it fails with errors:
Traceback (most recent call last):
File "laser.py", line 227, in
effect.run()
File "C:\Program Files\Inkscape\share\inkscape\extensions\inkex\base.py", line 140, in run
self.save_raw(self.effect())
File "laser.py", line 90, in effect
curves = parse_string(etree.tostring(self.document))  # Parse an svg file into geometric curves
File "C:\Program Files\Inkscape\share\inkscape\extensions\svg_to_gcode\svg_parser_helper_functions.py", line 86, in parse_string
return parse_root(root, canvas_height, transform, draw_hidden)
File "C:\Program Files\Inkscape\share\inkscape\extensions\svg_to_gcode\svg_parser_helper_functions.py", line 66, in parse_root
curves.extend(parse_root(element, canvas_height, transform, False, visible))
File "C:\Program Files\Inkscape\share\inkscape\extensions\svg_to_gcode\svg_parser_helper_functions.py", line 62, in parse_root
path = Path(element.attrib['d'], canvas_height, transform, transform)
KeyError: 'd'

There was an error in the file. I have deleted it. Now it works with beta 2 but gcode is absolutly wrong. Negative coordinates!
Also as mentioned by ninef: teh laser on/off commands are missing.
The new link and some wrong gcode:
https://my.hidrive.com/lnk/UlYULs87
G1 F 3000 X 217.1 Y -630.6;
M3 S255;
G1 X<PHONE_NUMBER>0000003 Y -627.8258000000001;
G1 X<PHONE_NUMBER>0000002 Y -624.4384000000002;

I made a small test file, and it shows strange paths on the top left (see attached screenshot).
as far as I can tell, these paths are not actually in the generated Gcode.
Feature suggestions:
*It would be nice to have the ability to see which G-code got generated from which path in the debug view (e.g. in the object properties description)
*the debug paths should have arrows indicating the move direction
*it would be great if the interface had the option to add/subtract "global" coordinate offsets. (e.g. my pen is about 40mm to the left of the machine center)

@uwefrank I think the buggy behavior is caused by the transform attribute. I still need to add support for parsing transforms (a pretty important feature).

@nineff Great suggestions. I think I'll implement all three. Maybe it's time to create a proper feature tracker.
No idea what the strange paths are. Could you share the file?

@nineff Great suggestions. I think I'll implement all three. Maybe it's time to create a proper feature tracker.
No idea what the strange paths are. Could you share the file?

sure, here you go:
(i had to zip it because github does not allow svg files directly)
Zeichnung.zip

I just published what might become the stable version if y'all don't find any bugs. (v2.0-beta.3 tag, still in the v2.0 release candidate).

Added support for transformations. (Solves most bugs)
Beautiful arrows
Custom printer origin and bed size
Translate and scale G-code
A bunch of minor bug-fixes

Known issues:

Elliptical arcs can act-out for certain edge cases

Please try it out and let me know if you find any bugs or if you think it's ready to go stable. Thank you all for you're helping out!

@nineff Adding G-code to object descriptions is a bit of a mess to implement right now. I like the idea but it will have to come after the first stable release.

I just published what might become the stable version if y'all don't find any bugs. (v2.0-beta.3 tag, still in the v2.0 release candidate).
* Added support for transformations. (Solves most bugs)

* Beautiful arrows

* Custom printer origin and bed size

* Translate and scale G-code

* A bunch of minor bug-fixes

Known issues:
* Elliptical arcs can act-out for certain edge cases

Please try it out and let me know if you find any bugs or if you think it's ready to go stable. Thank you all for you're helping out!

great, I will test it later today or tomorrow

Update: It fails with the following error if i use a spiral:
C:\Program Files\Inkscape\share\inkscape\extensions\svg_to_gcode\compiler\_compiler.py:50: UserWarning: Compile with an empty body (no curves). Is this intentional?
  warnings.warn("Compile with an empty body (no curves). Is this intentional?")

Moreover, it then deletes the spiral from the file!
spiral.zip

@nineff I tested it and the extension is actually working correctly in that case.  You accidentally placed the spiral in a debug layer so of course it was deleted and not drawn when you ran the extension.

well thats explains it, but it should then be harder to accidentally do this ;)

so now I did some more intensive testing:

BUG: The resulting G-Code apparently only uses the feedrate for the travel speed and not the cutting speed.
BUG: I don't see any arrow-heads in the debug view, just red lines
Request: please use the file-choosing dialog for the Output Directory/File and the custom header/footer code
(see here https://wiki.inkscape.org/wiki/index.php?title=Extension_GUI_Reference#File_chooser)
is there any reason why the Approximation Tolerance is entered as string and not as float?

Request: Color code different movement types in the debug view (e.g. cutting vs moving, potentially each pass a different color?)
Request: Please let us define different commands for laser on/off again via the UI, not just the power level for one command.

so now I did some more intensive testing:

BUG: The resulting G-Code apparently only uses the feedrate for the travel speed and not the cutting speed.

Feedrate can be simlpy corrected by changing line 98/99 in _compiler.py of current release candidate to:
```
code = [self.interface.laser_off(), self.interface.set_movement_speed(self.movement_speed), self.interface.linear_move(start.x, start.y),self.interface.set_movement_speed(self.cutting_speed),
self.interface.set_laser_power(1)]

Hi,
I'd just like to add, like @nineff I also use this extension for plotting so retaining the M03 and M05 commands would be very useful.
@nineff: how are you modifying the g-code files to make them worth with your plotter?

Version 2.0 is out! Thank you all for your help finding bugs and features. Let me know if you encounter any bugs or have feature recommendations by creating a new issue.
It's still missing the colored movement lines. If anyone wants to add them please do.


Mismatched datetime formatting in variable replacement

Describe the bug
When updating a datetime parameter of the factory using a JSON file and -Stage, the datetime is parsed before replacing the parameter. The replaced datetime is then formatted, but not in the manner expected, with the timezone identifier omitted, resulting in an error on deployment:
Start-AzDataFactoryV2Trigger `
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
HTTP Status Code: BadRequest Error Code: BadRequest Error Message: ErrorCode=InvalidWorkflowTriggerRecurrence,
ErrorMessage=The recurrence of trigger has an invalid start time '2023-02-27T00:30:00.0000000' zone designator
for time zone 'UTC'. Expected start time format with zone designator is 'yyyy-MM-ddTHH:mm:ssZ'. Request Id:
... Timestamp (Utc): ...

To Reproduce
Try to deploy a factory configuration using JSON variable replacement, with action "update".
Version of PowerShell module of:
Powershell 7.3.3
azure.datafactory.tools 1.3.0

Are you able to update one of the unit tests to present this bug?


[BUG] Bad exception handling causing crashes

Describe the bug
Due to bad exception handling, the program crashes every time an unexpected behaviour is encountered.
To Reproduce
Steps to reproduce the behavior:

Whenever the program is expecting a type of input or consequence and that doesn't happen

Expected behavior
The program should be able to continue it's operations when it meets an exception if that exception is handled, but almost none are, thus causing the crashes.
Desktop (please complete the following information):

OS: All Supported Operating Systems

As of v1.0.0, most of these unexpected crashes won't happen anymore.


Not correct shown tags images from stack deploy

Description
I have a few problems with the images that are created after deploy services from docker stack.yml.

Some images generally do not have tags, however some images that are deployed from the same stack and created simultaneously!? (why 2 or 3 identical images) have and do not name tag, the images don't get tags and the tag column in the list of images will be empty.

Why in the list I see 2 or 3 identical images if I do not scale containers and my container runs on one node of 3.

After a little analysis, I can say that at the moment of the peel, a new container starts to be created, then goes to shutdown without messages in the logs,  the previously run container changes its binding image and it continues to work correctly.
Supposedly no changes were made to the service, so the new container did not start, but just updated the image on the already running, BUT, in the list of images an additional image is created (a duplicate on an open container)
I tried to find an explanation in the knowledge bases, but I never got enough on this behavior.
Renaming the images manually every time does not fit, because a day can deploy stack 10-20 times
Remove duplicated images by manually comparing where actually used and where notused it is also not feasible
Tell me please how to solve this problem.

Steps to reproduce the issue:
1.deploy a stack
2. check tags for the images
3. observe image tag is empty
docker info
`Client:
Context:    dockswarmssl
Debug Mode: false
Plugins:
buildx: Docker Buildx (Docker Inc., v0.8.2)
compose: Docker Compose (Docker Inc., v2.6.1)
extension: Manages Docker extensions (Docker Inc., v0.2.7)
sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc., 0.6.0)
scan: Docker Scan (Docker Inc., v0.17.0)
Server:
Containers: 10
Running: 6
Paused: 0
Stopped: 4
Images: 18
Server Version: 20.10.17
Storage Driver: overlay2
Backing Filesystem: xfs
Supports d_type: true
Native Overlay Diff: true
userxattr: false
Logging Driver: json-file
Cgroup Driver: cgroupfs
Cgroup Version: 1
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: active
NodeID: kcop4hec8novqspkjyvc10t6n
Is Manager: true
ClusterID: 3p9ws0irom4c9mes439frcbjj
Managers: 1
Nodes: 3
Default Address Pool: <IP_ADDRESS>/8
SubnetSize: 24
Data Path Port: 4789
Orchestration:
Task History Retention Limit: 5
Raft:
Snapshot Interval: 10000
Number of Old Snapshots to Retain: 0
Heartbeat Tick: 1
Election Tick: 10
Dispatcher:
Heartbeat Period: 5 seconds
CA Configuration:
Expiry Duration: 3 months
Force Rotate: 0
Autolock Managers: false
Root Rotation In Progress: false
Node Address: <IP_ADDRESS>
Manager Addresses:
<IP_ADDRESS>:2377
Runtimes: runc io.containerd.runc.v2 io.containerd.runtime.v1.linux
Default Runtime: runc
Init Binary: docker-init
containerd version: 10c12954828e7c7c9b6e0ea9b0c02b01407d3ae1
runc version: v1.1.2-0-ga916309
init version: de40ad0
Security Options:
seccomp
Profile: default
Kernel Version: 3.10.0-1160.71.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.701GiB
Name: mss-docker00.sodrugestvo.local
ID: S77A:WBER:ODTY:74IZ:DX7L:C2CU:QG6F:YR5P:GJN7:ZOJL:CRBC:65TF
Docker Root Dir: /var/lib/docker
Debug Mode: false
HTTP Proxy: http://wsa.sodrugestvo.local:3128/
HTTPS Proxy: http://wsa.sodrugestvo.local:3128/
No Proxy: localhost,<IP_ADDRESS>,.sodrugestvo.local,.sodru.com
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false
Default Address Pools:
Base: <IP_ADDRESS>/16, Size: 24`
Portainer version:
Portainer Business Edition 2.16.1
Platform (linux)

Bug description
A clear and concise description of what the bug is.
Expected behavior
A clear and concise description of what you expected to happen.
Portainer Logs
Provide the logs of your Portainer container or Service.
You can see how here
Steps to reproduce the issue:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Technical details:

Portainer version:
Docker version (managed by Portainer):
Kubernetes version (managed by Portainer):
Platform (windows/linux):
Command used to start Portainer (docker run -p 9443:9443 portainer/portainer):
Browser:
Use Case (delete as appropriate): Using Portainer at Home, Using Portainer in a Commercial setup.
Have you reviewed our technical documentation and knowledge base? Yes/No

Additional context
Add any other context about the problem here.

@Kotnstantin
When you see images listed with :none it is because when Swarm pulls images, it does so by digest/hash of the image.
Thanks!

@Kotnstantin
Can you provide the following:
[1] Stack code to test with
I created 2 Stacks.  One with Private Registry and one with Dockerhub Images in a Docker Swarm Environment:

Thanks!