Interface federation proof of concept

This PR is a proof of concept for interface federation in
graphql-go-tools. It is not meant to be merged. It follows the
implementation strategy outline in jensneuse/graphql-go-tools#328.
Specifically, it uses the first concrete implementation of an interface
as the root type when determining data source ownership of a field. The
field configuration ForTypeField method was also updated to have use
the first concrete type. These changes are very hacky, but it helps
identify what needs to change to support federation.
Note that this implementation only works when a selection set is the
same for all concrete types that implement an interface; it doesn't
handle inline fragments with type-specific selections. The federation
code doesn't handle inline fragments at all, actually. This needs to be
fixed.
Related to inline fragments: federation also doesn't support union
types, which require inline fragments for all selection fields.
@csilvers had an interesting idea: expand all interface selections into
inline fragments (which use the concrete types as type conditions)
during the normalization process. This would be done before the
normalization step that merges inline fragments. Then federation and
union support would boil down to teaching the planner configuration
visitor and the GraphQL data source visitor how to handle inline
fragments at the top-level of upstream operations.

Closing all the old graphql-go-tools PRs. We ended up using Apollo Gateway to create query plans (which can be done statically), which we then execute via a small, custom Go execution engine.


WIP: Iter and Cursor Seek

This PR refactors the Iter RPC to use Honu seek that was added in v0.2.4 - however in the course of making this change (which I thought was going to be simple) I discovered some namespace handling bugs. Those bugs are resolved in github.com/rotationalio/honu #12 so this PR is blocked until that PR is merged.
The better namespace handling also allowed me to simplify some of the namespace logic in the code, and to guard against changes, I implemented another test for a standard series of access (Get, Put, Get, Delete, Get, Put, Get).
The last thing to do in this PR is to consider adding Seek to the cursor implementation.

Codecov Report

Merging #257 (5e394d8) into main (671713e) will increase coverage by 0.07%.
The diff coverage is 64.70%.

:exclamation: Current head 5e394d8 differs from pull request most recent head 138572e. Consider uploading reports for the commit 138572e to get more accurate results

@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
+ Coverage   43.78%   43.86%   +0.07%
==========================================
  Files          53       53
  Lines        6052     6046       -6
==========================================
+ Hits         2650     2652       +2
+ Misses       2926     2920       -6
+ Partials      476      474       -2

Impacted Files
Coverage Δ

pkg/trtl/honu.go
71.63% <64.70%> (-1.01%)
:arrow_down:

pkg/gds/gds.go
62.50% <0.00%> (+0.76%)
:arrow_up:

pkg/gds/admin.go
56.05% <0.00%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

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


How to do "SELECT EXISTS(SELECT 1 FROM ...)"?

Maybe need to add method query.AsExists() or any other for do this?

These are methods for adding conditions.
For example .HavingExists(...) will generate SELECT ... FROM ... HAVING EXISTS(...).
And .WhereExists(...) will generate SELECT ... WHERE EXISTS(...).
The Query class has a .AsCount() method that generates SELECT COUNT(*) FROM ....
My question is how to generate exactly SELECT EXISTS(...)
Maybe add a method like .AsExists() that will generate a query that will have SELECT EXISTS(...) in the view?


AttributeError: 'MjModel' object has no attribute 'eq_active'

I tried to run eval.py,a new issue appeared like this:
Traceback (most recent call last):
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/oem/data/fb/新项目/task_conditioned_adaptation-main/src/run/eval.py", line 23, in
import dmc2gym, gym, mj_envs, mjrl.envs
File "/home/oem/data/fb/新项目/eai-vc/third_party/dmc2gym/dmc2gym/init.py", line 56, in
e = make(domain_name='finger', task_name='spin', frame_skip=2)
File "/home/oem/data/fb/新项目/eai-vc/third_party/dmc2gym/dmc2gym/init.py", line 52, in make
return gym.make(env_id)
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/gym/envs/registration.py", line 676, in make
return registry.make(id, **kwargs)
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/gym/envs/registration.py", line 520, in make
return spec.make(kwargs)
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/gym/envs/registration.py", line 140, in make
env = cls(_kwargs)
File "/home/oem/data/fb/新项目/eai-vc/third_party/dmc2gym/dmc2gym/wrappers.py", line 61, in init
self._env = suite.load(
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/suite/init.py", line 113, in load
return build_environment(domain_name, task_name, task_kwargs,
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/suite/init.py", line 148, in build_environment
env = domain.SUITEtask_name
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/suite/finger.py", line 52, in spin
physics = Physics.from_xml_string(*get_model_and_assets())
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/mujoco/engine.py", line 435, in from_xml_string
return cls.from_model(model)
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/mujoco/engine.py", line 418, in from_model
return cls(data)
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/mujoco/engine.py", line 122, in init
self._reload_from_data(data)
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/mujoco/engine.py", line 399, in _reload_from_data
model=index.struct_indexer(self.model, 'mjmodel', axis_indexers),
File "/home/oem/anaconda3/envs/eai-vc/lib/python3.8/site-packages/dm_control/mujoco/index.py", line 623, in struct_indexer
attr = getattr(struct, field_name)
AttributeError: 'MjModel' object has no attribute 'eq_active'

This error might come from the version of dm_control or mujoco you have installed. Have you followed the installation guidelines in the README? What versions of dm_control and mujoco do you have?

dm_control==1.0.5,mujoco==3.2.2，i followed your installation guidelines in the README，then i fixed some lackness of my conda env ,i may did something wrong then.What version these two need to be?

i solved this problem by pip install mujoco==2.3.5,thanks for your patience for me


Allow volume to be optional

A regular Secret resource can be used as an optional volume:
volumes:
- name: missing
  secret:
    secretName: missing
    optional: true

If the secret does not exist, it does not prevent the Pod from being created. Any volumeMounts using the secret volume are mounted as an empty dir (or file if subpath is used).
This is helpful in creating resources (Pods, Jobs, Deployments, etc) that work in different environments.
Is it possible to do the same for the SharedSecret resource?

I just realized that a SharedSecret can be created to reference a Secret that does not exist. In that case, what is mounted is an empty directory. This kind of allows for an optional SharedSecret. It would be nice to not have to create the SharedSecret at all though.


Amend needs and requirement

What does this pull request do?
service for amend needs and requirements
What is the intent behind these changes?
to allow pp to amend needs and requirements.

Needs changes to ReferralNotificationService to send emails to notify SP

What is the scope of this ticket? It seems to introduce changes for all these fields
  var furtherInformation: String? = null,
  var additionalNeedsInformation: String? = null,
  var accessibilityNeeds: String? = null,
  var needsInterpreter: Boolean? = null,
  var interpreterLanguage: String? = null,
  var hasAdditionalResponsibilities: Boolean? = null,
  var whenUnavailable: String? = null,

And that seems to be a lot.


AES key request

Hi,
I have managed to create a streaming locator with its associated Content Key Policy (with JWT token restriction) and its Streaming Policy (Predefined_ClearKey).
Now I am trying to request the AES key so my Azure Media Player can decrypt the encrypted content however I am getting a 401 response saying that the content key policy evaluation has failed. I think this is due to the fact that my STS is providing the player an invalid token.
How should I create a valid token? Are there any JavaScript SDK functions to create this token?
I would like to replicate the funcitonality of the GetTokenAsync function from the EncryptWithAES example found here
I think this guide does not completely specify how to create this token when using a token restricted content key policy.
Help will be verry much appreciated

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

ID: 2c75e8fa-2e91-d600-417c-2de6f9d98c21
Version Independent ID: 09063127-7d0a-db1d-c2d1-69f7fb3fa3ac
Content: Protect your content with Media Services dynamic encryption - Azure Media Services
Content Source: articles/media-services/latest/content-protection-overview.md
Service: media-services
GitHub Login: @Juliako
Microsoft Alias: juliako

@BernardoMB, Thanks for the question! We are investigating and will update you shortly.

@BernardoMB Here is a way to generate a JWT token in C#
https://docs.microsoft.com/en-us/azure/media-services/latest/protect-with-aes128#get-a-test-token
I did not find an example based on javascript. May be this can help : https://www.jonathan-petitcolas.com/2014/11/27/creating-json-web-token-in-javascript.html

@BernardoMB , Just wanted to understand if you need further assistance on this issue.

@BernardoMB  I will proceed with the closure of the issue. Please feel free to reopen if you still need assistance.


Fix incorrect inner component reference

The inner component is EditForm, not EditContact.

I noticed the same problem while translating this article. I believe this needs to be merged.


Make sure leading / in command is never required (but in console), also make sure to strip in all cases

Ref: https://github.com/UnvanquishedAssets/unvanquished_src.dpkdir/pull/11
We must be able to do:
teambind humans     hw:e   "modcase alt \"deconstruct marked\" deconstruct"

instead of:
teambind humans     hw:e   "modcase alt \"/deconstruct marked\" /deconstruct"

but this last one must always work even if / is not needed.
Stuff like delay already works like this. It's all about unifying things and making things right.

I don't see what the difference between these 2 is? They look exactly the same.

I assume it should be teambind humans     hw:e   "modcase alt \"deconstruct marked\" deconstruct" instead or something.

Fixed the first post.

if is the other one like that, that you have to put /foo else it does like vstr foo.


Default bucket?

Should we have a default bucket? It's annoying specifying it every time. This could be a module global, environment variable, or something else.
There's also the question of which other defaults we could want.
@tcram @DCSCHUS

I think specifying the default bucket as a environment variable by user might be the best option since staff outside of DECS may be using this tool.

A default bucket can be specified a few ways.
Environment variable - set ISD_S3_DEFAULT_BUCKET to desired bucket.
Command line - python isd_s3 --default_bucket [bucket name] or python isd_s3 -db [bucket name]
Config Module -
import config
config.set_default_bucket('bucket_name')

In .ini file
....
[default]
bucket = bucket_name
....


Support for clients and services in the rclc_executor

Add support for clients and services to the rclc_executor, in the style of rclc_executor_add_subscription (cf. https://github.com/ros2/rclc/blob/master/rclc/include/rclc/executor.h#L208).

Add support for clients and services to the rclc_executor, in the style of rclc_executor_add_subscription (cf. https://github.com/ros2/rclc/blob/master/rclc/include/rclc/executor.h#L208).

Is there any plan to support client and service in rclc_executor?

Yes. My colleague @JanStaschulat is already working on this in the branch https://github.com/micro-ROS/rclc/tree/feature/support_for_service_clients.

Yes. My colleague @JanStaschulat is already working on this in the branch https://github.com/micro-ROS/rclc/tree/feature/support_for_service_clients.

thanks, so great!

Services, clients and guard conditions were added in PR https://github.com/ros2/rclc/pull/26

Services, clients and guard conditions were added in PR https://github.com/ros2/rclc/pull/26


Add CodeClimate badge

Add badge below the title. It's good to see that repo in such a good shape!

Thanks!


[ci] Move static tests into separate job

Two longest jobs currently are test_unit and test_build. These have some steps that are don't depend on previous steps and therefore can be safely extract.
New job is called test_static which describes the steps good enough (format, lint, types, docs generation). Every (custom) step in this job could potentially run parallel to the other steps.

We only have 5 concurrent jobs available with Circle CI. Running 5 jobs means that a single pull request takes all the resource. Could that have a negative impact?

We only have 5 concurrent jobs available with Circle CI. Running 5 jobs means that a single pull request takes all the resource. Could that have a negative impact?

If we have 5 jobs available and this change doesn't go over the limit I don't see how this could cause any issues? If our builds can run in parallel then we were already starved anyway.

I was wondering if we would lose global throughput (because each test as a compressive yarn install setup), but even if we do, I agree with the approach, the feedback loop for the tests is very important, the shorter they are (for a given one), the more productive we can be 👍.

Sure it's not guarenteed that this will helps since I can't simulate heavy load on the CI environment. This is technically the better approach. We have to monitor how this works out (just like any other change).
If yarn install becomes the bottleneck we can scale the machine power. Just restoring cache already takes 30s which is quite long. Rrestoring node_modules/ might be the better approach. Another bottleneck might be compiling of pupeteer since it doesn't come with pre-built binaries.


Test if increase of controller-runtime version introduced new issues

Following up from issue https://github.com/goharbor/harbor-operator/pull/1034

Seems to work from our testing so far. We wait for final testing but close here for now.


Fix Xcode 11 build

Summary
Fixes build in Xcode 11 beta, the signature for __unused was changed. This adds a new check for the new style.
In my project, react-native version 0.49.5 was used.
I met this issue and found a solution, but I need to modify the code in node_moduels/reac-native by myself. It is not convenient for me to migrate the project.
I was hoping you guys would release react-native 0.49.6.

Changelog

[iOS] [Fixed] - Xcode 11 beta build
Test Plan
Build & run RNTester

Um... That just does not look right. Can we just have the same as #29071 but based on master?


WIP: PR for testing CI on MacOS

Adding extra logging in integration tests to capture more info on flaky MacOS CI runs

Closing PR for now, CI test builds can be triggered from branch for later testing.


⚠ Bofrost.nl has degraded performance

In 70c6fd8, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1016 ms

Resolved: Bofrost.nl performance has improved in c031823.


CB-3210. Refactor telemetry models (use objects instead of booleans)

use objects for boolean flags (reportDeploymentLogs and meteringEnabled)
removed databusEndpoint from WA request (only from request/response)
remove logging/attributes from telemetry and replace it with fluentAttributes

retest this please

I am afraid the SQL has some compatibility issues, and during the rollout, it will break all exsisting pods.


data: AxelGarciaK

Changes proposed
I have added my profile in LinkFree. 🎉
Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

@Krishnapro Thanks mate! 😁

Good catch @kumarsonsoff3, fixed!


Refactor test kits

Remove httpbin tests and ensure that their cases are covered in the main test kits
Review all tests and test kits to ensure that there is a unified test plan across all functionality and clients
consider: sync, async, clients, http, https, BASIC, DIGEST across all or most test cases by verb
Ensure active encoder/parser test cases for each verb

This has been addressed in the 0.14.0 release (development branch)


Correct ARM64 SIZE_OF_JMP from 8 to 16 and spell out 2*4 as 8.

This is a lot of bytes to overwrite, and it precludes
patching a number of small-ish functions, but it is how many.
For example, automatic following imports, has lead to system service stubs,
that were smaller than this (and aligned less than this).
Part of this change:
Change 112950 by REDMOND\galenh@galenh2 on 2016/11/07 16:24:57
Build_340: Fix compilation error on ARM64.  Fix jmp region calculations on ARM & ARM64.
Affected files ...
... //depot/969/3.0/README.TXT#58 edit
... //depot/969/3.0/src/creatwth.cpp#61 edit
... //depot/969/3.0/src/detours.cpp#69 edit
... //depot/969/3.0/src/detours.h#68 edit
... //depot/969/3.0/src/detver.h#46 edit
... //depot/969/3.0/src/disasm.cpp#56 edit
... //depot/969/3.0/src/image.cpp#51 edit
... //depot/969/3.0/src/Makefile#57 edit
... //depot/969/3.0/src/modules.cpp#56 edit
... //depot/969/3.0/src/uimports.cpp#48 edit

Pedro will be reducing this from 16 to 12 shortly.


Members: add Fidelity (FCAT)

Adds Fidelity Center for Applied Technology (FCAT) as member

Looks ok, but perhaps
 is better? (I think it's too wide with the 'Fidelity Center for Applied Technology' wording, but perhaps it works with just the FCAT?)

FCAT ACK'd existing logo

Send that tweet :rocket:


[WIP][NOT MERGE][EXTENSION] Inject Rule to fast fail gluten app if there are too much un-support operator

Why are the changes needed?

Fast fail if there are too much un-support operators in spark plan when enabled gluten.
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?

No

Codecov Report

Merging #5321 (aa3f654) into master (ffebc64) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #5321   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         590     590
  Lines       33436   33488   +52
  Branches     4422    4424    +2
======================================
- Misses      33436   33488   +52

see 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

cc @ulysses-you

thanks @ulysses-you @pan3793 , make some changes:

Rename fallback operator config key
make fallback operator as optional
make fall back operator can be changed in runtime case(Get this operators each call apply method)


CORE-7268 Handle empty component groups

CORE-7268 Handle empty component groups
All component groups gets prefixed with an extra item to avoid empty lists.

Closing this in favor of the already merged https://github.com/corda/corda-runtime-os/pull/2545
That one looks cleaner and more concise than this


ViewZone area's last line reports the next line number in the MouseDown event

monaco-editor version: 0.18.0
Browser: Google Chrome 76.0.3809.132 64bit
OS: Ubuntu 18.04.3
Steps or JS usage snippet reproducing the issue:

Initialize a Monaco editor instance.
Add a ViewZone using changeAccessor.addZone()
Set a editor.onMouseDown().
Try making console.log() for the e.target.position.lineNumber while clicking on the last line inside the ViewZone area.
You'll get the number of the next line, not the one having the ViewZone.

The mouse position returns the line above the view zone when the mouse is in the first half of the view zone and the line below the view zone when the mouse is in the second half of the view zone.


Add basic tests with real data

Want to merge this before #5

I rebased this on main and the new tests still pass.

Cool, looks good, thanks!


TAL's are not being placed if arin.tal is already existing

I deploy Routinator with ansible and thus place the arin.tal using get_url in Ansible. If you start routinator afterwards, it does not check if it has tals for all five RIRs but instead only runs with output of Arin.
Removing the tals directory creates all the tals and the fake Arin one.

We are going to change the whole strategy around setting up the TAL directory in one of the next versions and replace it with an init command.


testing new models

Hi Jonas, just a curiosity. What are the correct formats and steps to test new models in other languages, please? Please advise. Thank you.

Do you mean how to train and test new models? The README has example steps on how to do that. If you want to train a larger model, then check the READMEs of the pre-trained models which detail the training settings for those models.

Hi Michael, thank you. I meant to ask if corpora must be in .sh format in order to be prepared to be trained. Thank you.

What do you mean by .sh format?
On 31 May 2017 at 17:25, micsanMercy16<EMAIL_ADDRESS>wrote:

Hi Michael, thank you. I meant to ask if corpora must be in .sh format in
order to be prepared to be trained. Thank you.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/fairseq/issues/48#issuecomment-305354837,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADGSfgAxEipk5JUvp5pWPRXiBgnE_9Clks5r_gT7gaJpZM4NsSRj
.

I have some data I would like to train with fairseq and README says cd data, bash prepare-iwslt14.sh. What should I do to train my own data? Thank you.

This is what I get when I try to train my data, please advise. Thank you.
fairseq micheles$ fairseq train -sourcelang es -targetlang en -datadir data-bin/span-samples-sources.tokenized.es-en -model fconv -nenclayer 4 -nlayer 3 -dropout 0.2  -optim nag -lr 0.25 -clip 0.1 -momentum 0.99 -timeavg -bptt 0 -savedir trainings/fconv
/Users/micheles/torch/install/bin/luajit: ...es/torch/install/share/lua/5.1/fairseq/scripts/train.lua:29: assertion failed!
stack traceback:
[C]: in function 'assert'
...es/torch/install/share/lua/5.1/fairseq/scripts/train.lua:29: in main chunk
[C]: in function 'require'
...install/lib/luarocks/rocks/fairseq-cpu/scm-1/bin/fairseq:17: in main chunk
[C]: at 0x010513dbc0

@micsanMercy16 you'll need a machine with a GPU for training new models. The error message stems from train.lua, line 29. As you can see, the line above states that cuda is required for training.

Does it not require a -cpu instead?

The current training code does not support CPU training. You can, however,
generate with already trained model on CPU.
On 2 June 2017 at 10:01, micsanMercy16<EMAIL_ADDRESS>wrote:

Does it not require a -cpu instead?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/fairseq/issues/48#issuecomment-305852080,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADGSfuamQ7HJd_oaY8Y1Qp8sqvM5wTJgks5sAD_mgaJpZM4NsSRj
.


regarding output

This is very well written I really thanks a lot
This code is very helpful for me but i am not able to see the output (perspective images)
I have used this code (below),  foe showing output but the perspective image is not showing
I requested to please suggest to me or write a code for showing the perspective images.
if name == 'main':
img = im.imread('/content/drive/MyDrive/Street View 360_1.jpg')
equ = Equirectangular('/content/drive/MyDrive/Street View 360_1.jpg')    # Load equirectangular image
img = equ.GetPerspective(60, 0, 0, 720, 1080) # Specify parameters(FOV, theta, phi, height, width)
#persp.toEquirectangular(img, center_point)

`
from PIL import Image  # +++++++
import Equirec2Perspec as E2P
if name == 'main':
equ = E2P.Equirectangular('src/image.jpg')  # Load equirectangular image
#
# FOV unit is degree
# theta is z-axis angle(right direction is positive, left direction is negative)
# phi is y-axis angle(up direction positive, down direction negative)
# height and width is output image dimension
#
img = equ.GetPerspective(60, 0, 0, 720, 1080)  # Specify parameters(FOV, theta, phi, height, width)

img = img[:, :, ::-1]  # +++++++
img1 = Image.fromarray(img.astype('uint8')).convert('RGB')  # ++++++

img1.save("./out_img.jpg")  # +++++

`
You must add the line " # +++++".

Thank you so much sir for your response

Thank you so much sir for your response. This code is very important for usRegardsPankaj KumarPh.D Research scholarDepartment of Civil Engineering(Specialization: Transportation Engineering)Indian Institute of Technology KanpurKanpur - 208016, U.P. , INDIAContact: +91-<PHONE_NUMBER>On Aug 7, 2023 07:59, Thomas @.***> wrote:
`
from PIL import Image  # +++++++
import Equirec2Perspec as E2P
if name == 'main':
equ = E2P.Equirectangular('src/image.jpg')  # Load equirectangular image

FOV unit is degree
theta is z-axis angle(right direction is positive, left direction is negative)
phi is y-axis angle(up direction positive, down direction negative)
height and width is output image dimension

img = equ.GetPerspective(60, 0, 0, 720, 1080)  # Specify parameters(FOV, theta, phi, height, width)
img = img[:, :, ::-1]  # +++++++
img1 = Image.fromarray(img.astype('uint8')).convert('RGB')  # ++++++
img1.save("./out_img.jpg")  # +++++
`
You must add the line " # +++++".
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Current speed value's in dashboard config

Currently, every time I exit I get asked to save the dashboard even tho I have not changed anything. When I view the changes, it seems to be hard-coding the current speed values in de dashboard config. I don't see why it should do this since it changes every minute.
To Reproduce
edit dashboard and exit without changing anything and view the changelog.

Expected behavior
I should exit without a save prompt because there were no changes.
VERSIONS:

Browser Edge Version 114.0.1823.58
Grafana Version v10.0.1
Plugin Version 0.3.6

Additional context
Using Zabbix as the source for the data.

Thanks for pointing this out. It was something I had seen before but hadn't spent the time getting to the bottom of. I think that I've fixed it, but need to do some more testing to ensure I haven't accidentally broken something else. I'll close this issue when the update to fix this is released.
Long story short, since I originally wrote this part of the program, Grafana has added better state handling for the panel so some tricks I had implemented to reduce calls to redraw were no longer useful (and this "state written to dashboard JSON" was an outcome of that).

Alright, the new version is out on Grafana (it's v0.4.2). You might have to save your dashboard once (after updating the plugin) for this to fully take effect? But from my testing on the released version I can now update the dashboard to see the new values and leaving or trying to save brings up no new changes.

Seems to work now, thanks!


Unfolding of Electronic and Phonon Band Structures

Adapt our old scripts to yambopy

Create the folder "unfolding with the corresponding classes"

I pushed the branch "supercell" on my github remote including the unfold-band-netcdf.py file. Both this one and the supercell class are in qepy. If Henrique creates a branch of the same name on his remote, I can do a merge.


v:true is v:false returns one

In addition to producing two internal errors v:true is v:false and v:true == v:false produce one.

Fixed by recent patches.


UHF-8835: Reverted the alter for paragraph button.

What was done

Reverted the alter for paragraph button.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 13.07%. Comparing base (8f829b6) to head (deb8a8d).
 Consider uploading reports for the commit 16c7b3d to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #700   +/-   ##
=========================================
  Coverage     13.07%   13.07%
  Complexity      242      242
=========================================
  Files            28       28
  Lines           941      941
=========================================
  Hits            123      123
  Misses          818      818

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


Fix fuzz target binary name and source file path

Some projects had >0 build rate but 0 coverage on all benchmarks, because of their incorrect fuzz target binary name (and/or fuzz target path).
This PR fixes that.

capstone and pigweed are two special cases who cannot build with OSS-Fuzz.

/gcbrun exp -i -n dg -b missing-bin

CC @DavidKorczynski Is there some more reliable way we can get these mappings?

/gcbrun exp -i -n dg -b missing-bin -ns 10 -nf 5 -to 1800

/gcbrun exp -i -n dg -b missing-bin -ns 10 -nf 5 -to 1800 -f

/gcbrun exp -i -n dg-1 -b missing-bin -ns 10 -nf 5 -to 1800 -f

/gcbrun exp -i -n dg-1 -b missing-bin -ns 10 -nf 5 -to 1800 -f

https://llm-exp.oss-fuzz.com/Result-reports/ofg-pr/2024-08-03-524-dg-1-missing-bin/index.html

/gcbrun skip


build_labels includes masked image tokens?

Hi Authors,
in these lines, the function build_labels masked all the labels in positions up to the seq length of the embeddings. What differences would it make if one just use the caption?
To be more specific, now the code build a label with first part of the sequence (which has sequence length the same as the image) all set to -100, then the second part would be the actual text labels. Why would we need all the -100s? Why couldn't we just use text label ids?
Thanks a lot!

Hi,
positions with index -100 get ignored by default when using torch cross entropy loss. We don't want a loss on these positions because the model is supposed to predict the text.
Best,
Constantin

Hi,
positions with index -100 get ignored by default when using torch cross entropy loss. We don't want a loss on these positions because the model is supposed to predict the text.
Best,
Constantin

Yeah I get it. But why mask at all? Why not just cut that part out and only use the text that you are predicting?

I guess you are right, one could also truncate the logits and labels from the left for the amount of image positions.
At least we need some logic to mask the right-padding: since the amount of right-padding varies per batch item we cannot simply cut this part without breaking up the batch which would be awkward. Using the same function to mask the image positions seems like the most clean solution to me.
In the end it is a matter of taste :-)


implement support for approver group

Implements support for single approver groups for a namespace or page.
Fixes part of #21 .

Could you update the plugin documentation?

Will do. Thanks for merging!

Done!

Thank you!


My MC crash everytime I try to open it, this is what it says

crash-2024-08-06_12.50.29-client.txt
flush now
I have 3 mods, Pehkui, Bleach-Awaken, Kleiders.
It stops crashing when I delete Kleiders but its a library for the Bleach-awaken
pls help me😭

How is this an Origins mod issue?


Make a checkttl expiration event always be critical

At yelp we never send warnings to our more critical handlers (pagerduty).
I want to be able to allow people to set a ttl on ephemeral checks and give them at least the option to page, but if it is alway set to warning, that can never happen?
I don't believe in warnings. :(
If you don't agree with this change, would you accept a more complicated PR that... set multiple thresholds like the keepalive thresholds? (but I would rather KISS here and have it set to always be crit)
cc @hashbrowncipher @bobtfish

@portertech can I get some feedback on this and #972? I'm very eager to use this new feature but I don't want have to through the pain of forking again to make it usable.

I do not believe a check failing to execution in a timely manner (or at all) warrants a critical status. A check TTL event could be the result of a variety of causes, sometimes more than one. I too would prefer to keep it simple, a single numeric threshold, as it is now. You could use a Sensu filter to match events with a check status of 1 and the TTL check output: eval: value.start_with?("Last check execution"), and apply the filter to a special pagerduty handler.

I think it depends on the check, but I agree that the simplicity is good and that this specific policy can be applied in other ways.

But, I would be curious if there are others who share my opinion, so others please perk up if you also see this is a bug or have other alternatives.


Cleanup unused imports, sort other imports

This is the bog-standard PyCharm "Optimize Imports" which removes unused imports and also sorts imports to the standard  stdlib-thirdparty-local grouping

@jhoshiko @nreinicke this looks good to me but it's a repo-wide change that will impact pycon sprint participants, so i'll leave it to you to pick the timing for it.


Automatic Grade generation and Embbeded output

We need first generate some Grade using some value, like slots of proposals and then organize the proposals, for now just using a ordinary order and let the user reorder, using some code like: http://tympanus.net/codrops/2014/11/11/drag-and-drop-interaction-ideas
This would be nice, if we had some an html output to include in Static sites the accepted proposals for a event, this will facilitate the work of the Organizers in choose talks and build the html, will automate all the work, this is much related to the "Automatic Grade generation".

Closing this according the PR #86


SyntaxError: Cannot use import statement outside a module

Not sure why this is happening, using the examples..
import { v4 as uuidv4 } from 'uuid';
import { ComfyUIClient } from 'comfy-ui-client';
import type { Prompt } from 'comfy-ui-client';
// The ComfyUI server address
const SERVER_ADDRESS = '<IP_ADDRESS>:8188';
export const txt2img = async (
prompt: Prompt,
outputDir: string,
): Promise => {
// Create client ID
const clientId = uuidv4();
// Create client
const client = new ComfyUIClient(SERVER_ADDRESS, clientId);
// Connect to server
await client.connect();
// Get images
const images = await client.getImages(prompt);
// Save images to file
await client.saveImages(images, outputDir);
// Disconnect
await client.disconnect();
};

This is a full working demo around the code you provided. It's not an issue with this library.
client_demo.zip


How handle resource Type validation of File.exists? with rspec-puppet 2.3.0

Afternoon all
After upgrading to rspec-puppet 2.3.0, I've hit an issue whereby I've got a failing test[1] on a defined type [2][3], that's using a custom resource[4], which has got a File.exists? validate block[5]...
Obviously when testing on TravisCI, that file doesn't exist, and I've got no way of making it exist AFAIK...
What I could do with some pointers on is how to handle this scenario and make the test pass?
It's probably worth adding that this test passed with rspec-puppet 2.2.0[6]...
Any ideas?
Cheers
Gavin
[1] https://travis-ci.org/fatmcgav/fatmcgav-glassfish/jobs/100859787#L1467-L1472
[2] https://github.com/fatmcgav/fatmcgav-glassfish/blob/develop/manifests/create_domain.pp
[3] https://github.com/fatmcgav/fatmcgav-glassfish/blob/develop/spec/defines/create_domain_spec.rb#L134-L166
[4] https://github.com/fatmcgav/fatmcgav-glassfish/blob/develop/lib/puppet/type/domain.rb
[5] https://github.com/fatmcgav/fatmcgav-glassfish/blob/develop/lib/puppet/type/domain.rb#L79-L87
[6] https://travis-ci.org/fatmcgav/fatmcgav-glassfish/jobs/94887206#L252-L254

I don't understand how this ever worked. You need to either stub File.exists? for these tests or create the file in a :before block (and clean it up in :after).

Hehe, indeed..
I've tried stubbing File.exists?[1] for the /tmp/template.xml file, but that causes other errors[2]...
Any pointers?
[1] https://github.com/fatmcgav/fatmcgav-glassfish/commit/9dbf3d04b0ef215f49be5a48d21aafedce5c902c
[2] https://travis-ci.org/fatmcgav/fatmcgav-glassfish/jobs/101125286#L1470-L1478

I'll have to look into that on monday, as it explicitely tells you that
File.exists? was not called. Too confusing.
On 8 January 2016 at 18:24, Gavin Williams<EMAIL_ADDRESS>wrote:

Hehe, indeed..
I've tried stubbing File.exists?[1] for the /tmp/template.xml file, but
that causes other errors[2]...
Any pointers?
[1] fatmcgav/fatmcgav-glassfish@9dbf3d0
https://github.com/fatmcgav/fatmcgav-glassfish/commit/9dbf3d04b0ef215f49be5a48d21aafedce5c902c
[2]
https://travis-ci.org/fatmcgav/fatmcgav-glassfish/jobs/101125286#L1470-L1478
—
Reply to this email directly or view it on GitHub
https://github.com/rodjek/rspec-puppet/issues/348#issuecomment-170081512
.

Hehe, cheers...
On 8 Jan 2016 18:28, "David Schmitt"<EMAIL_ADDRESS>wrote:

I'll have to look into that on monday, as it explicitely tells you that
File.exists? was not called. Too confusing.
On 8 January 2016 at 18:24, Gavin Williams<EMAIL_ADDRESS>wrote:

Hehe, indeed..
I've tried stubbing File.exists?[1] for the /tmp/template.xml file, but
that causes other errors[2]...
Any pointers?
[1] fatmcgav/fatmcgav-glassfish@9dbf3d0
<
https://github.com/fatmcgav/fatmcgav-glassfish/commit/9dbf3d04b0ef215f49be5a48d21aafedce5c902c
[2]

https://travis-ci.org/fatmcgav/fatmcgav-glassfish/jobs/101125286#L1470-L1478

—
Reply to this email directly or view it on GitHub
<
https://github.com/rodjek/rspec-puppet/issues/348#issuecomment-170081512>
.

—
Reply to this email directly or view it on GitHub
https://github.com/rodjek/rspec-puppet/issues/348#issuecomment-170082568
.

It looks like this was introduced in 9c295b5de726128bc08acfb1efaeaf5987174563 ; that code calls #to_ral on resources which actually creates Puppet::Type instances and sets parameters - and setting a parameter triggers a call to the validate function. The validate functions are only meant to be applied on the agent during catalog application, whereas rspec-puppet unit tests are meant to only test the catalog, so by converting resources to their RAL equivalents this seems to break any type that might have validation functions like this.

Correction - any types that both have validation hooks that may touch the system and have autorequires will trigger this behavior. I believe that the autorequires are only available by creating an instance of the type, and setting up the type triggers that validation.
One option could be to monkey patch custom types and rip out the validation function when RSpec is running; short of that I don't see any good solutions.

Thanks for the analysis! to_ral sets up all relationships within the
catalog, which is required for the transitive dependency checking.  Otoh ,
the stubbing should have caught that, so I'll have another look at that
later today.
On Jan 8, 2016 11:23 PM, "Adrien Thebo"<EMAIL_ADDRESS>wrote:

Correction - any types that both have validation hooks that may touch the
system and have autorequires will trigger this behavior. I believe that the
autorequires are only available by creating an instance of the type, and
setting up the type triggers that validation.
One option could be to monkey patch custom types and rip out the
validation function when RSpec is running; short of that I don't see any
good solutions.
—
Reply to this email directly or view it on GitHub
https://github.com/rodjek/rspec-puppet/issues/348#issuecomment-170155098
.

Ah, of course! The catalog is built only once for the complete group (caching, yay!) so the second example will have the expectation, but not the compilation to match it. Replace expects with stubs to remove the assertion, while keeping the fake implementation.

@DavidS So what would you recommend?
Changing File.expects? with File.stubs??

In https://github.com/fatmcgav/fatmcgav-glassfish/commit/9dbf3d04b0ef215f49be5a48d21aafedce5c902c#diff-51c7ec588ec9f75fa3df5d5a7d2e7b50R159, change
File.expects(:exists?).with('/tmp/template.xml').returns(true).once

to
File.stubs(:exists?).with('/tmp/template.xml').returns(true)

Still seem to be getting the unexpected invocation error:
diff --git a/spec/defines/create_domain_spec.rb b/spec/defines/create_domain_spec.rb
index 1459722..8cf69c5 100644
--- a/spec/defines/create_domain_spec.rb
+++ b/spec/defines/create_domain_spec.rb
@@ -156,7 +156,7 @@ describe 'glassfish::create_domain' do
     end

     it do
-      File.expects(:exists?).with('/tmp/template.xml').returns(true).once
+      File.stubs(:exists?).with('/tmp/template.xml').returns(true)^M
       should contain_glassfish__create_service('test').with({
         'running'      => 'true',
         'mode'         => 'domain',

Results in:

Failures:

glassfish::create_domain with a domain template file specified should contain Glassfish::Create_service[test] that requires Domain[test]
Failure/Error: }).that_requires('Domain[test]')
Mocha::ExpectationError:
unexpected invocation: File.exists?('/Users/gavinw/Puppet/modules/fatmcgav/glassfish/spec/fixtures/modules/manifests/init.pp')
satisfied expectations:
- allowed any number of times, not yet invoked: #Puppet::Util::Feature:0x7fcb34386628.root?(any_parameters)
- allowed any number of times, not yet invoked: File.exists?('/tmp/template.xml')
# ./spec/defines/create_domain_spec.rb:165:in `block (3 levels) in <top (required)>'

OK, looks like I've come up with a work-around in a4a218ae410a943542ae27683d6193e9aad6c02c...
Am using Tempfile to create a file on FS, and passing that path through to the relevant places...
So will go-ahead and close this one off... Might look at updating the docs' for rspec-puppet to make this scenario clearer, as I can't be the only person this change has affected... :S

@fatmcgav thanks to working through this. Adding a note to the documentation is always great, but I also need to add that doing agent-side verification in the type is not very mainstream. I think this is a very vaguely defined area in puppet.


Update examples.rst

Update example coordinates to have non-zero data

@macrintr Thanks for improving the docs!
It looks like our CI has caught that your changes make some of the doc reference outputs incorrect.
You can run make doctest in the zetta_utils/docs folder to run the same tests locally.


how to  link the profile result with the business log  and async task using java

Please answer these questions before submitting your issue.

Why do you submit this issue?
Question or discussion

Question

What do you want to know?
-First Question: Is skywalking support log(business log) collect. Like alibaba's arms link the profile result with the business log.
-Second Question:  In java project, business logic process use thread pool,  how to bind traceContext to the task? and how to  analysis the async thread pool task consuming.  like cost ?

-First Question: Is skywalking support log(business log) collect. Like alibaba's arms link the profile result with the business log.

Please don't involve a commercial product in, that is not a reference for an open-source project.

Collecting logs in traces has been supported, and will be released in 8.4.0, you could read docs/en/setup/service-agent/java-agent/agent-optional-plugins/Logger-plugin.md
If you want a better way, lower load to your application(agent), consider waiting for the satellite release in 2021, https://github.com/apache/skywalking-satellite

-Second Question: In java project, business logic process use thread pool, how to bind traceContext to the task? and how to analysis the async thread pool task consuming. like cost ?

Across thread solution has been provided for a long time, https://github.com/apache/skywalking/blob/master/docs/en/setup/service-agent/java-agent/Application-toolkit-trace-cross-thread.md
Your questions are mostly included in the documentation. Please read them before submitting an issue.

thx，we just want to reduce dependence on commercial product, so we coming.

I know, but please let's keep discussion on tech perspective only.


The size of the image input to Swin Transformer

I notice that Swin Transformer uses pre-training parameters, and I would like to ask you whether the image size input into Swin Transformer is also 224*224?

No. The input size has two restrictions in default: (1) No more than 1.3*480p resolution; (2) The edge size should be 16N+1 (if MODEL_ALIGN_CORNERS=Ture) or 16N (if MODEL_ALIGN_CORNERS=False).

Sorry, I do not understand this. When encoder is SwinB, the initial weights were trained with Imagnet22k and input image size is 224*224. If your input size is not 224 * 224, the number of corresponding patches will change. Can you input the next linear layer? Maybe I have a wrong understanding of swinB~~
Thank you!

Swin Transformers perform like CNNs since sliding windows are used. In fact, you can use any input size you want.


Create 0000-01-02-jbhopale.md

Pull Commit and Merge to master completed

Successfully done with the first assignment


Can I use xterm for backend only without dom object?

I am going to create a terminal web project. But I want to separate frontend and backend. The backend, which is nodejs, will serve restful APIs for frontend. I checked that xterm.js works really awesome and want to include xterm.js in my backend.
So the scenario will be that frontend has an editor on the browser, it will send commands to backend when users hit enter. Backend will send the commands into xterm.js and listen on its output. Then send the output back to the frontend.  In other words, I want to emulate a terminal in my backend server.
I checked the xterm.js doc and found that xterm has to be created with a dom object. Is this possible to use xterm without dom object?

xterm.js is a frontend, it doesn't really work as a backend. You cannot create it without a DOM.

So the scenario will be that frontend has an editor on the browser, it will send commands to backend when users hit enter.

It sounds like you want a text box that just sends a command to a server which runs a child processes and sends the output? If that's all you want you should not use xterm.js/node-pty as they overcomplicate things and don't do what you want. What you can do if you do a setup like the xterm.js demo however is run interactive programs like git as it's a true terminal.

Yep, your understanding is correct. I need to use node-pty to handle the pty output from a backend process. And it is not easy to deal with ansi code. So I want to use xterm.js to help me to handle ansi characters. I see xterm is listening on websocket channel.
xterm is a javascript library which means it should work with nodejs backend application as well. How about I create an instance of xterm in the nodejs and listen on its output? Is it possible?

@zhaoyi0113 why not just run the command in a child process?

child process doesn't have pty. The sub-process I am going to create is a mongo shell process. It is an interactive terminal. I do need pty to handle the input and output. I see that when I call xterm.write it will parse the data and render it on the dom. Is it possible that I attach a listener on xterm to get the parsed output?

If you need a pty then use xterm.js on the client side then, that's what it's designed for.
The demo works by sending key/mouse events to the server which are handled by node-pty, node-pty then sends back both the output. Note that the output includes the input that was send from xterm.js after it's been evaluated by mongo. This is what you want, isn't it?

That's a good suggestion. But one more requirement is that I need to do some format on xterm output before rendering it on the dom. Is there a method or injector I can use to do that?

There isn't currently. The issue there though is more the renderer has a strict format and you couldn't do that even if it was running on some server.

I see there is a Parser.ts file. Is it used for parsing ansi characters? I wonder whether I can use this parser class independently?

Sounds like you need everything though..? The rendering, the input handling, the parsing. If you just take the Parser (ignoring the fact it changes) then you will still need to convert key presses, maintain a buffer, deal with rendering, etc.

I only need the parsing part and I don't want to handle the key press, render, cursor movement etc. I just need to take the data from pty output then parse it.


Search button does not search

Search is not implemented yet

Not ready for review yet.

Ready for review but still in need of a test for the search view.

This article happens to have the word "blockquote" and it's the only result.

:+1:

@miltontony landing, adds search, pls review in the morning.


Step Learning Rate Scheduler

Added a StepLearningRate scheduler and an example resnet training script for CIFAR-10 that uses it.
This is based on a recent PR that was not formed well. But, since @fchollet is interested in adding these schedulers to keras, I resurrected the PR fixing most of the issues and adding an example script that uses it in examples. This learning rate scheduler schedules a step learning rate scheduling that is common to train very deep networks such as GoogLeNet, resnets, WRN, etc...
The scheduler as implemented here could be used manually as well as automatically. If the user supplies a list of epochs, the scheduler drops the learning rate at those manually specified discrete epochs and if the user supplies a single 'patience' epoch, it will check if the monitored variable doesn't improve up to 'patience' amount of epochs and drops the learning rate by a user supplied drop amount. The auto mode is very similar to early stopping, the difference being instead of stopping this scheduler reduces the learning rate, and hence I shared a bunch of code from early stopping.
I have used the example code I shared here with n=4 and k=4 to result in WRN (wide residual network) 28x4 and reproduced (even slightly better) result with a similar implementation in Lasagne.
Note: I had to modify the SGD optimizer code in keras to allow only user supplied scheduler like this one and not end up with sporadic 'nans' when I set lr.decay in SGD in the current implementation.

I didn't run PEP8 on the code and it still has some commenting standard issues. I will fix it today. Other than that it passes the other builds.
I will also add tests if it is necessary. But, I have tested it thoroughly and I am using it in my projects.


Second paragraph needs correction.

In the second paragraph where it says -"By default, Terraform state is stored which isn't ideal for the following reasons:", the word --> 'locally' is missing.
The sentence should be -- "By default, Terraform state is stored locally which isn't ideal for the following reasons"

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

ID: 789076a3-5d6d-e5d9-a3f7-cc1d6351ca8a
Version Independent ID: e873b421-20fe-3415-0f4b-aacd7378c271
Content: Tutorial - Store Terraform state in Azure Storage
Content Source: articles/terraform/store-state-in-azure-storage.md
Service: terraform
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@cloudysingh Thanks for the feedback. I'm closing this as a duplicate of #561


Gosub - Return

Thank you for your work. I am using it as a scripting engine inside a c# application and it works perfect!
Is it possible to add a GOSUB - RETURN subroutine call?
Thanks in advance + best regards
JB

It is possible, but might be hard to implement as this is direct code interpreter.


Update Pages “home”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
91bb0fbac5d2f437787f1d22c5bd6f50e1da9d89

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/654f05b5b50baa0009d8d6b7


[Bug]: Making application Public, and making application Forkable are not treated as new commits in a git connected app.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Making application Public, and making application Forkable are not treated as new commits in a git connected app.
Steps To Reproduce

Create a git connected app, add a few pages/widgets etc on master, and create a few branches say test1 and test2 from master.
Now, in the test1 branch, From Settings - > Share and Embed -> Turn on "Make Application Public" and "Make Application Forkable"
Click on the Commit Changes, and note that there are no changes to commit.
Now merge this branch to test2 branch
Go to Settings - > Share and Embed in test2 branch.
Note that Make Application Public is enabled, but Make Application Forkable is disabled.

This behavior seems incorrect.

If the two settings are not required to be recognised by git, as they are app level changes, then both options should have been enabled even in test2 branch
If they are required to be recognised by git as new commits, then, they should appear as commits in the test1 modal, and on merging test1 to test2, test2 should have both the settings enabled.

Public Sample App
No response
Environment
Production
Severity
High (Blocker to building)
Issue video log
https://www.loom.com/share/6fc4f403faea48fd8283e0847e568354
Version
Cloud

Still we can repro it

@RakshaKShetty is this just for sharing or forkable? Sharing would not be treated as a git persisted property since permissions are local to an instance.

Closing as both these properties are meant to be tracked at instance level only.


README batch says only Python 3.7 supported, is this true?

The Python badge in the README says only Python 3.7 is supported. Is this true? I don't think so.
It'd probably be better to either fix or remove the version - rather than state something that's incorrect.

Closed by #2020


New version: Malwarebytes.Malwarebytes version <IP_ADDRESS>

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #175464

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

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

Microsoft Reviewers: Open in CodeFlow

The offline installer should be used, located here:
https://data-cdn.mbamupdates.com/web/mb5-setup-consumer/offline/MBSetup.exe


nvim-yarp fails when python3_host_prog is pointed towards virtualenv

I have created a python3 virtualenv specifically for the neovim and installed the packages required by nvim and its plugins there.
after setting g:python3_host_prog = '~/nvim/venv/bin/python' neovim starts as expected and :checkhealth is not giving me any error. but for some reason nvim-yarp keeps failing on startup flashing an error message on startup. it doesnt show error if i comment g:python3_host_prog line from my .nvimrc
OUTPUTS:
pip list of virtualenv:
(venv) [precrisk@prefros nvim]$ pip list
Package           Version
----------------- -------
astroid           2.3.3
autopep8          1.4.4
entrypoints       0.3
flake8            3.7.9
greenlet          0.4.15
isort             4.3.21
jedi              0.15.1
lazy-object-proxy 1.4.3
mccabe            0.6.1
msgpack           0.6.2
neovim            0.3.1
parso             0.5.1
pip               19.3.1
powerline-status  2.7
pycodestyle       2.5.0
pyflakes          2.1.1
pylint            2.4.4
pynvim            0.4.0
setuptools        41.2.0
six               1.13.0
wrapt             1.11.2

:checkhealth
 33 ## Python 2 provider (optional)
 34   - WARNING: No Python executable found that can `import neovim`. Using the first available executable for diagnostics.
 35   - ERROR: Python provider error:
 36     - ADVICE:
 37     ¦ - provider/pythonx: Could not load Python 2:
 38     ¦   ¦ /usr/bin/python2 does not have the "neovim" module. :help ¦provider-python¦
 39     ¦   ¦ /usr/bin/python2.7 does not have the "neovim" module. :help ¦provider-python¦
 40     ¦   ¦ python2.6 not found in search path or not executable.
 41     ¦   ¦ /usr/bin/python is Python 3.8 and cannot provide Python 2.
 42   - INFO: Executable: Not found
 43
 44 ## Python 3 provider (optional)
 45   - INFO: Using: g:python3_host_prog = "~/nvim/venv/bin/python"
 46   - INFO: Executable: /home/precrisk/nvim/venv/bin/python
 47   - INFO: Python version: 3.8.0
 48   - INFO: pynvim version: 0.4.0
 49   - OK: Latest pynvim is installed.

my project .nvimrc file with all nvim-yarp related config.
let g:python3_host_prog = '~/nvim/venv/bin/python'
call plug#begin()
Plug 'roxma/nvim-yarp'
Plug 'ncm2/ncm2'
Plug 'ncm2/ncm2-bufword'
Plug 'ncm2/ncm2-path'
Plug 'ncm2/ncm2-jedi'
call plug#end()
autocmd BufEnter * call ncm2#enable_for_buffer()
set completeopt=noinsert,menuone,noselect

error message during nvim startup. this doesnt show if i comment out g:python3_host_prog

You should expand ~.
let g:python3_host_prog = expand('~/nvim/venv/bin/python')

@Precrisk #39 fixes your problem.

Thankyou, after using expand its working again.


Ruby 2.6 and 2.7 support

We need to add support for Ruby 2.6 and 2.7.
At least, BigDecimal behaviour needs to be updated. BigDecimal.new is deprecated in 2.6 and removed in 2.7. Instead, use the BigDecimal() method. E.g.
-          BigDecimal.new(string)
+          BigDecimal(string)

We can also likely remove the bigdecimal dependency from this gem as it's standard.
Add 2.6 and 2.7 to CircleCI, and stop testing versions before 2.5.x (they're now long EOL).

Fixed by #16


UI Customisation?

Hey, I would like to customise the look of a few of the UI. Mainly the Message UI. I would also like a separate pm design which I know would mean changing some of the code as I would like it to stay on screen and allow the user to reply to the PM.
I would mainly like to know how to change the look of the UI first. Once I know that I will be able to change how the PM command works myself. Thanks.

Solved it, just used a plugin to change UI to Lua, put the lua into the message module and edited it so the data was passed through from the command.

Forgo to close the issue.


Fix typo

python -> Python

Learn Build status updates of commit 4e88bf7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/synapse-analytics/spark/apache-spark-azure-portal-add-libraries.md
:white_check_mark:Succeeded

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

#sign-off


Fault detection algorithm

Implement the fault detection algorithm from the paper:
A Diagnostic Method of Simultaneous Open-Switch Faults in Inverter-Fed Linear Induction
Motor Drive for Reliability Enhancement
Authors:Jung-Hyun Choi,Sanghoon Kim, Dong Sang Yoo, and Kyeong-Hwa Kim.
The algorithm should work for single open-switch and double fault in the same inverter's leg. According to the paper, this corresponds to the fault modes 1 (single leg fault) and 2 (single switch).
The cases necessary to test this two modes are:
(T1,T4), (T2,T5), (T3, T6), T1, T2, T3, T4, T5, T6.

The simulations moved to an external drive.
The cases already covered: (T1,T4), (T2,T5), (T3, T6), T1, T2, T3, T4, T5, T6 in full speed.
T1 with series resistance of 5 and 10 Ohms.
T1 with intermittent trigger for 20% of the time.


Gist support not working

Tried with your sample example as well for gist support but it is not working as expected.
http://codepen.io/ritz078/pen/gaNEYJ

Hi @exexzian ,
I am aware of this. It has been fixed in v4 that is currently merged in master. I am just busy writing docs for that and will release it in few days. You can use the v4 code and if you find any issue let me know here until the new docs are published .

thanks for the update @ritz078 .
Checked the updated example and v4 and yup it works fine.


Fix Codacy warnings

Run standardjs across all js files, fix all other issues by hand

@kc5nra can you approve my CR

Pull Request Test Coverage Report for Build 1947

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 79.202%

Files with Coverage Reduction
New Missed Lines
%

internal/mux/mux.go
2
92.5%

pkg/ice/candidate.go
4
91.11%

Totals

Change from base Build 1937:
0.02%

Covered Lines:
3850

Relevant Lines:
4861

💛  - Coveralls


fix(rollup-build): Modified rollup config

Modified Rollup.js config to include the following plugins as devDependencies:

rollup-plugin-polyfill-node
@rollup/plugin-commonjs
@rollup/plugin-json

Modified build step for rollup execution tsc && npx rollup -c

Removed cdn_bundle files

Updated dev/index.html to call Rollup generated bundle my-element.bundled.js

Removed unwanted code in the feature.

Removed rollup configuration
index.html solely used for demo purpose and not advisable to include a bundled output file, same case considering the production code.


sorting of child contexts

child contexts are seemlingy sorted randomly. there seem to be 2 orders: the leafs appear in order and the contexts with children too, but there is no overall order.

Moved to backlog. Tried to fix this, but it is not that easy. Maybe we should try it together @derntl

Fixed in current working branch. Will be in next release (v0.9 aka Isfahan)


⚠ Fortnite has degraded performance

In b519b95, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1656 ms

Resolved: Fortnite performance has improved in 2c442f1.


Widget: Bitcoin

Category
Widget
Overview
A very simple bitcoin widget to fetch current price in USD and EUR plus the current recommended fees per transaction. It uses the free mempool api
I intend to improve this widget over time but anyone is welcome to make it more useful
Screenshot (if applicable)

Also please note that your commits are unverified and not authored by you in any way, there is an issue with your git config. The committer is called "Your name"

Hello! Thanks for the PR, I just saw your LinkedIn post! Congrats on your contribution. After reviewing it quickly I can say that I do not approve of this Widget simply because it is not common enough, I'm sure a lot of people are interested in crypto but adding a widget only to display the price of bitcoin is not good enough. We already support iframe and the website https://coinlib.io/widgets?w_single_coin_id=859&w_single_pref_coin_id=1505&w_single_width=250&w_all_theme=Light#w_single can basically create this widget with the selection of the coin, the currency, …
However, you could most likely find a read API endpoint (without a library) that gives out that info and more (like the price for the last X days) and add options for example : selected coin (XRP, ADA, ETH, BTC…) , timeframe (24h, 7d, 30d, 1y) and display a cool graph using our included graph library. I know this is a lot of work but if you need help I can help you. Feel free to reach on discord about this.
Do you think my review is too harsh ? Please don't get frustrated by my refusal, I am very happy to welcome new contributors. It's just that we need to have high quality widgets. Let's work together on making a high quality "crypto" widget and not only a bitcoin one ;)

I made this widget out of my own need at this moment. I know it could be more elaborated and show fancy graphs and all that stuff. It was my intention to increment it over time but for now the most important thing is having BTC price at my fingertips without having to leave my dashboard.
As of add other coins I have no problem with that I just think they are irrelevant but as I said I welcome anyone wiling to make this widget more useful. I built it out of my own needs.

a Crypto.com study found that more than 219 million individual Bitcoin holders were recorded in late 2022

Anyway, your suggestions are indeed good ones and I will take them but I am extremely bad at styling frontend and I couldn't find any docs regarding development standards. This project is huge and it was quite hard to find myself in it, like I was very confused where to store translation file for instance, whether I could write the api my own way or should I follow some pattern etc. I used the weatherWidget as my reference.
 I am going on sorta trial and error basis here.
I don't consider your review harsh, it is direct to the point which is what I expect. I gladly accept your help, if you can take charge of the frontend that would be amazing and make things much more faster.
As of my git configs you are right, I had forgotten to set it up. I will if there is a way to fix the commits

You should try to implement things on your own. Try to understand how the weather widget works, remove the new dependency and use a public API. Then look again at the weather widget to understand options and how to use them. Then use these options in your API calls (locally using tanstack query or in the router like you did)
And then you should have a functioning crypto widget ;)
We simply cannot merge the "bitcoin" widget as it adds no value (because of the already existing iframe widget) the added cost in dependencies would make the app use more ram/storage as well.

@ajnart can you help me find out what I am missing.

For some reason I cant get data to the frontend, something is not working properly on my router I cant seem to find out what.  I know the urls are correct

@ajnart can you help me find out what I am missing.

For some reason I cant get data to the frontend, something is not working properly on my router I cant seem to find out what.  I know the urls are correct

It seems like you are calling the procedure without arguments, (symbol, name, links)
Ideally you just have a symbol, timeframe option and the procedure will return you the price/fluctuation of that asset in the correct timeframe. You just need to make it so that the option is then properly interpolated onto the URL that is used to get the data from the api.
But your editor should say that you are calling the procedure without correct arguments so I'm unsure 🫤 you can push your code and we can see where you have errors

After glancing at the code, I'm not sure why you have another call for "get initial data", I think you can just use the basic call

@ajnart can you help me find out what I am missing.

For some reason I cant get data to the frontend, something is not working properly on my router I cant seem to find out what.  I know the urls are correct

It seems like you are calling the procedure without arguments, (symbol, name, links)
Ideally you just have a symbol, timeframe option and the procedure will return you the price/fluctuation of that asset in the correct timeframe. You just need to make it so that the option is then properly interpolated onto the URL that is used to get the data from the api.
But your editor should say that you are calling the procedure without correct arguments so I'm unsure 🫤 you can push your code and we can see where you have errors

Those are not arguments, those are the zod schema I created for the response. It seems like the procedure is not awaiting response from the api. Also I defined default values for each procedure in case no input is passed

After glancing at the code, I'm not sure why you have another call for "get initial data", I think you can just use the basic call to populate your data

The reason a made getInitialData is simply to have both calls inside only one since they always happen together in order to update the widget

After glancing at the code, I'm not sure why you have another call for "get initial data", I think you can just use the basic call to populate your data

The reason a made getInitialData is simply to have both calls inside only one since they always happen together in order to update the widget

Why not just have one inside of one procedure ?

Isnt that what this is?
  getInitialData: publicProcedure
    .output(cryptoWidgetDataSchema)
    .query(async () => getInitialData()),

After glancing at the code, I'm not sure why you have another call for "get initial data", I think you can just use the basic call to populate your data

The reason a made getInitialData is simply to have both calls inside only one since they always happen together in order to update the widget

Why not just have one inside of one procedure ?

if I make this call:
api.cryptoCurrencies.getCoinGeckoCryptoData.useQuery()
or
api.cryptoCurrencies.getCoinGeckoCryptoMarketChart.useQuery()
it works but if I use getInitialData() it doesnt

After the request is made using fetch I get this:

Note that the url is correct formed. Even if I pass empty object to useQuery() in the frontend the url is well formed by the default values.


MakeapplyMaterialVariant Method Public in FilamentInstance Class

The current implementation restricts access to this method, and making it public will enable external classes or components to utilise this functionality

Thanks for your contribution, could you please sign the CLA so we can accept your patch?


Ensure session returns a UserInterface

zend-expressive-session does not serialize PHP objects directly, but instead passes them to json_encode(); this is done to prevent PHP de/serialization vulnerabilities.
As such, the logic within the PhpSession adapter needed to change: it needed to pull the relevant data from the UserInterface implementation (the username and role), and serialize that data; in turn, when pulling the data from the session, it needs to create a UserInterface instance.
For this latter, we use an anonymous class implementing the interface, and populating it based on the session data.

Ping @ezimuel to review.


Puts new proposition complete behaviour behind a feature flag.

Description of change
This puts the new behaviour of proposition complete functionality behind the feature flag. If proposition-documents feature flag is active, new functionality becomes enabled.
This adds a suite of tests for old behaviour in TestLegacyCompleteProposition class.
It also adds a utility function in the feature_flag app that gets the "state" of the feature flag.
Checklist

[X] Has a new newsfragment been created? Check changelog/README.rst for instructions

Codecov Report

Merging #1100 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1100      +/-   ##
===========================================
+ Coverage    96.54%   96.55%   +<.01%
===========================================
  Files          225      226       +1
  Lines         6347     6361      +14
  Branches       596      598       +2
===========================================
+ Hits          6128     6142      +14
  Misses         151      151
  Partials        68       68

Impacted Files
Coverage Δ

datahub/feature_flag/utils.py
100% <100%> (ø)

datahub/investment/proposition/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/investment/proposition/constants.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[Release v1.14.0-beta.0] Update cmd/cmctl's go.mod to v1.14.0-beta.0

This PR cmd/cmctl's go.mod to v1.14.0-beta.0 as part of the release process.
To the reviewer: the version changes in go.mod must be reviewed.
NONE

/kind cleanup
Will approve via tags since this is trivial


How can i know the progress of the upload (post)?

It seems that `onProgress' only shows the download progess?

Yes, this is known feature, closing as duplicate of #239

public void onProgress(long bytesWritten, long totalSize) {
// Progress notification
if(totalSize > 4096) {
Log.d("now: ", bytesWritten.toString();
}
}
but i get no log

Again, yes, there is no library provided upload onProgress, just download.

so sad


Bidmach at the limit

Hi,
Wondering if there were more benchmark available for bidmach with terabytes size data,
Using Kylix or not for popular ML algos ?
Thanks

You quickly run in to problems with hosting and serving data sets that size. The typical monthly cap for Comcast customers is 300GB.

Hi,
Suppose than datafeed is not an issue,
On Single machine, how much is the max typical size than Bid Mach can handle ?
(Ex: Logistic regression)
while keeping processing time linear with size
10Go ?
With Kylix , 10 To is fine ?
Kind regards
Arita
On 07 Apr 2016, at 08:17, Matthew Macy<EMAIL_ADDRESS>wrote:
You quickly run in to problems with hosting and serving data sets that size. The typical monthly cap for Comcast customers is 300GB.
―
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

We're working on a 1TB public dataset from Criteo via MS Azure. Would you believe downloading it takes about 2 weeks? We have about 1 day left, then we have to featurize it.
Its true we could probably do it a lot faster on EC2 with a cluster, but we estimated that the download/processing would take more than $1000. On the other hand we can do it for free as a background task on one of our machines and in few weeks, we'll upload the data to EC2. Its true we wont want to leave it there too long at 0.10/GB/mo = $100/mo, but once its there we'll be chewing machine cycles ata pretty good rate doing benchmarks.

Dear John,
I understand the material burden related to large dataset...
In theory, Bidmach should be able to large dataset (1To), is it some
batch offline processing in the computation process
(ie logistic regression) ?
Just thinking of the bottleneck to handle very large dataset (how it can
be overcome..)
Kind regards
Arita

jcanny<EMAIL_ADDRESS>Thursday, April 07, 2016 10:10 AM
We're working on a 1TB public dataset from Criteo via MS Azure. Would
you believe downloading it takes about 2 weeks? We have about 1 day
left, then we have to featurize it.
Its true we could probably do it a lot faster on EC2 with a cluster,
but we estimated that the download/processing would take more than
$1000. On the other hand we can do it for free as a background task on
one of our machines and in few weeks, we'll upload the data to EC2.
Its true we wont want to leave it there too long at 0.10/GB/mo =
$100/mo, but once its there we'll be chewing machine cycles ata pretty
good rate doing benchmarks.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/BIDData/BIDMach/issues/105#issuecomment-206641244

We're working on a 1TB public dataset from Criteo via MS Azure. Would you believe downloading it takes about 2 weeks? We have about 1 day left, then we have to featurize it.

"Never underestimate the bandwidth of a station wagon full of tapes." - Perhaps one could drop by Soda Hall with a 1+TB SSD to obtain a copy. Assuming the SSD could sustain a 500MB/s streaming write for the duration of the copy it would take less than 40 minutes.

Just wondering if the tests have finished, Thanks


Use a different QR code app

The barcode scanner app that is linked to from the display fingerprint menu item wants too many permissions for me to install it. I already have a QR code app (qr droid private) that can generate and scan codes. Why won't chatsecure use that?

Alternate solution: XPrivacy Pro


Have a way to force environment variable names to upper case regardless

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
In V1, given an OS that has case sensitive environment variable names: If you have a settings-object like the below (simplified from the pydantic docs):
class Settings(BaseSettings):
    foo: str

then: If only "FOO" is set, "FOO" is obviously used. Likewise if only "foo" is set, "foo" is used. But if both "foo" and "FOO" is set, "foo" is preferred. This is a bit surprising!
I'm used to environment variable names always being upper case, regardless. For that reason, I would like the ability to only read from upper case environment variable names regardless of what it looks like in the rest of the code. This could look like:
    class Config:
            force_uppercase_names = True  # Default: unset,  no change from current system

or
    class Config:
            environment_name_casing = "upper"  # Default: None/unset, no change from current system

At minimum, the docs should be updated with a warning that on an OS with case sensitive environment variable names and case_sensitive set to False, this will occur, to simplify tracking down the obscure bugs I can see this leading to.
Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

You can use case_sensitive config and set all your environment variables upper-case. Then you have to define your settigns fields upper-case as well. Here is an example in pydantic-settings:
from pydantic import ConfigDict

from pydantic_settings import BaseSettings

class Settings(BaseSettings):
    FOO: str

    model_config = ConfigDict(case_sensitive=True)

import os
os.environ['FOO'] = 'test1'
os.environ['foo'] = 'test'

print(Settings())
# FOO='test1'

If you don't want to use upper-case field names, you can use Field.validation_alias:
from pydantic import ConfigDict, Field

from pydantic_settings import BaseSettings

class Settings(BaseSettings):
    foo: str = Field(validation_alias='FOO')

    model_config = ConfigDict(case_sensitive=True)

import os
os.environ['FOO'] = 'test1'
os.environ['foo'] = 'test'

print(Settings())
# foo='test1'

Note: pydantic-settings is in alpha state. You can install it by pip install pydantic-settings --pre


Add high priority to DaemonSet

We are running an EKS cluster with Horizontal Pod Autoscaler and Karpenter for node autoscaling.
When several pods with our application are needed and no space is available in existing nodes, Karpenter, as expected, creates a
new node and tries to start all our pods in it, in addition to csi-secrets-store-provider-aws pod.
But in some occasion, our pods were created first, occupying all the node resources, and csi-secrets-store-provider-aws could not be created. This prevented our pods to work correctly, as they depend on secrets provided by csi-secrets-store-provider-aws.
As explained in https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/, would it be a good idea to add "priorityClassName: system-node-critical" to DaemonSet in aws-provider-installer.yaml?
We have done it with:
# verify that yq is installed
command -v yq >/dev/null 2>&1 || (echo "yq command not found. Please install it from https://mikefarah.gitbook.io/yq/"; exit 1)
curl -o aws-provider-installer.yaml https://raw.githubusercontent.com/aws/secrets-store-csi-driver-provider-aws/main/deployment/aws-provider-installer.yaml

# add priorityClassName: system-node-critical
# https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/
# https://kubernetes.io/docs/tasks/administer-cluster/guaranteed-scheduling-critical-addon-pods/
yq -i 'select(.kind=="DaemonSet") | .spec.template.spec.priorityClassName = "system-node-critical"' aws-provider-installer.yaml

kubectl apply -f aws-provider-installer.yaml

This is already a configuration option we expose in our Helm chart.

Closing the issue. Please feel free to open a new issue if additional support is required.


Fix Broken Layout

Broken layout, issue is opened here with details:
https://github.com/open-physiology/open-physiology-viewer/issues/205

Testing this from the latest master and I can see it's rendered differently from the image attached in the card mentioned.

Is this fine? from what I can see (screenshots attached) all the lyphs are attached to the chain, not sure how to reproduce this issue (using the same model in the top bar of the screenshot provided in the issue mentioned and enabling the same group).


remove old test_requests.py listing from MANIFEST.in

This is a trivial fix to address a warning pointed out in #4012.
test_requests.py was moved from the top level directory into the tests directory in 2.10.0 but the MANIFEST.in wasn't updated afterwards. Installing from setup.py produces a warning (no files found matching 'test_requests.py'). This patch will remove the unneeded listing and keep the warning from cluttering stderr.

Codecov Report

Merging #4015 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4015   +/-   ##
=======================================
  Coverage   89.74%   89.74%
=======================================
  Files          15       15
  Lines        1940     1940
=======================================
  Hits         1741     1741
  Misses        199      199

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 e4693eb...f4b9082. Read the comment docs.


Add a Gitter chat badge to README.md

Ichigochou/WheelEvent now has a Chat Room on Gitter
@storm1er has just created a chat room. You can visit it here: https://gitter.im/WheelEvent/Lobby.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

@Ichigochou pense à activer l'integration gitter quand tu auras le temps ici


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in d1daa38 after 5 minutes.


Crash while running in darkside mode

What is the bug?
I was running lightwalletd in darkside mode and I experienced crash of the server.
  And I can reproduce it anymore. My lighwalletd was built from master branch. I was running AdvancedReOrgTests test suite from iOS SDK project. This crash happened while testLongSync was running.
I am not sure if this is worth of a bug report.

Thanks for the report. I can't reproduce this problem, and I don't see how it can happen by looking at the code. Can you reproduce it?

And I can reproduce it anymore

Are you saying that you can't reproduce it? I wasn't sure.
By any chance, could you try the latest version of the code (5d174f7feb702dc19aec5b09d8be8b3d5b17ce45)? It's not tagged as a release, but I should do that soon. There have been several fixes since 0.4.13, so I'm wondering if this problem may have been fixed.

I am not able to reproduce it anymore. And it happened only once. I will fetch the latest code.

Thanks for trying to reproduce it, I'll close this now but if it happens again, please reopen it.


How to tf scans to PointCloud2 while TurtleBot3 LDS Lidar rotates clockwise ?

ISSUE TEMPLATE ver. 0.3.0

Which TurtleBot3 you have?

[x ] Burger
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[x ] ROS 1 Melodic Morenia
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[ ] etc (PLEASE, WRITE DOWN YOUR ROS VERSION HERE)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ x] Raspberry Pi 3
[ ] Intel Joule 570x
[ ] etc (PLEASE, WRITE DOWN YOUR SBC HERE)

Which OS you installed in SBC?

[x ] Raspbian
[ ] Ubuntu MATE 16.04 or later
[ ] Ubuntu Server 18.04 or later
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Which OS you installed in Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[ ] Linux Mint 18.x
[ ] Linux Mint 19.x
[x ] etc (PLEASE, WRITE DOWN YOUR OS) = Windows10

Write down software version and firmware version

Software version: [x.x.x]
Firmware version: [x.x.x]

Write down the commands you used in order

Tried laser_geometry::LaserProjection... transformLaserScanToPointCloud in C++ on Rasberry PI

Copy and Paste your error message on terminal

NO error = ran ok ... but did not produce correct results because LDS spins clockwise and ROS standard is for anti-clockwise scan points.  Therefore start of scan (0 degrees) at scan stamp time is actually at end of lidar rotation, while 359 degrees is the first scan to be read .. so "End time" at 359 degrees is negative time increment: which transformLaserScanToPointCloud does not seem to handle.

Please, describe detailedly what difficulty you are in

Therefore can you please advise:

what python or C++ functions should I use to correctly transform a moving Turtlebot3 LaserScan To PointCloud2 ?
and / or can the LDS lidar be set to spin anti-clockwise ?

Thanks
Michael Hallett

Hi,
You might want to checkout laser_geometry package to transform the laser scan into PointCloud.
http://wiki.ros.org/laser_geometry
Thank you.

Yes... I already had... and tried = transformLaserScanToPointCloud
... but this assume an anti-clockwise lidar (eg "start" time 0 is after "end" time @ 359 degrees)

@MikeHallettUK
I refered below thread to transform /scan to /cloud on my TurtleBot3 burger, and didn't have any problem creating the PointCloud2.
https://answers.ros.org/question/11232/how-to-turn-laser-scan-to-point-cloud-map/

Yes, I agree "transformLaserScanToPointCloud" runs .... but the point cloud is not correctly compensated for TB3 odom movement, because :
a) LDS Lidar rotates clockwise
b) your published scan time_increment is 20 x too small
????????????

@MikeHallettUK
I see. Though we cannot change the rotation of the LDS, I've fixed the incorrect time_increment value.
Could you try with updated code in the below hotfix-laserscan branch?
$ git clone -b hotfix-laserscan https://github.com/ROBOTIS-GIT/hls_lfcd_lds_driver.git

Thanks – will do. RegardsMike Hallett From: Will SonSent: 08 September 2020 02:39To: ROBOTIS-GIT/turtlebot3Cc: MikeHallettUK; MentionSubject: Re: [ROBOTIS-GIT/turtlebot3] How to tf scans to PointCloud2 while TurtleBot3 LDS Lidar rotates clockwise ? (#633) @MikeHallettUKI see. Though we cannot change the rotation of the LDS, I've fixed the incorrect time_increment value.Could you try with updated code in the below hotfix-laserscan branch?$ git clone -b hotfix-laserscan https://github.com/ROBOTIS-GIT/hls_lfcd_lds_driver.git—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

I do not believe this is "a specific case" for only my application.  This is fundamental to correcting any clockwise lidar scan for movement of any robot.  I am very surprised there seems to be no other issues raised about this.
**Q1) Has the TurtleBot3 LDS Lidar always been clockwise ?    I assume there are many TurtleBot3 LDS Lidar customers out there:  so I assume I am not the only one with a clockwise lidar scan ? ... so all customers must have this issue ?
( are most other Lidars (e.g. for ROS) anti-clockwise ? )
Q2) If so, then how does anyone using the standard ROS library produce a stable PointCloud2 from a scan while TurtleBot3 is moving ?**
The only partial fix I found was:
So I added this line into "hls_lfcd_lds_driver" to give negative increment...
scan->time_increment = -scan->time_increment;    // make negative to account for clockwise spin of lidar
which gives ...
rostopic echo /scan -n1
…
time_increment: -0.000559284118935
scan_time: 0.201342284679

Then "transformLaserScanToPointCloud" works quite well.
BUT.... then Cartographer SLAM no longer works with the negative scan time increment.
so the next question is can you make a change to Cartographer ~ "cartographer_turtlebot" ? to solve this clockwise spin of lidar issue...
a) does Cartographer have a parameter to set this ? in LUA ? ... I cannot find it.
b) or can the code that takes the scan in Cartographer be changed to accept either a negative scan time increment, or explicitly reverse the scan order some other way ?
Q3) What is the generix fix for the clockwise spin of any lidar within the ROS system (which is set positive for anti-clockwise spin) ?
... your sugestion of "installing the lidar LDS-01 upside down  (though you'll have to sacrifice some angles where supports are installed)" seems like a last resort.... since this surely can be fixed relatively easily in the software (without sacrifice of some angles where supports are installed).

I have now fixed this issue with Cartographer ROS SLAM, using a point-cloud as sensor input instead of the laser-scan directly;
by changing parameters in the turtlebot3_lds_2d.lua file in ~slam/config to:
num_laser_scans = 0,          { instead of = 1, }
num_point_clouds = 1,        { instead of = 0, }
So this works if you correct the laser scan-lag, converting it to a PointCloud2 (e.g with laser_geometry package routine "transformLaserScanToPointCloud"), and publishing this in a topic called "/points2" for SLAM Cartographer ROS to subscribe to.
In fact I also wrote a similar transform LaserScan To PointCloud routine in Python which seems to performs faster, and also feeds the required fixed PointCloud2 into Cartographer ROS.
I do not now have a "kink" in my "walls" when I approach them, and more importantly the whole Cartographer ROS map is much more stable with the Turtlebot3 spinning or turning.
For further discussion on this, with RVIZ screen shots showing the effect, see @ Using Cartographer ROS with clockwise Lidar and Python


Loading AppConfig feature flags into strong-type options

Describe the issue
So I've defined a feature flag configuration profile in AppConfig with two booleans e.g.

Run = true
Jump = false

For the ASP.NET 6 WebApplicationBuilder.Configuration object, it gets extended with AppConfig.
var builder = WebApplication.CreateBuilder(args);
builder.Configuration.AddAppConfig("appid", "envid", "configprofileid", TimeSpan.FromSeconds(30));

// This will not work.
// builder.Services.Configure<FeatureFlags>(builder.Configuration.GetSection(nameof(FeatureFlags)));

But how to read in all those feature flags into a consolidated options object? There's no real configuration section tied to those, is there?
public class FeatureFlags
{
	public bool Run { get; set; }
	public bool Jump { get; set; }
}

public class AppConfigModel : PageModel
{
	internal FeatureFlags FeatureFlags{ get; private set; }

	public AppConfigModel(IOptionsSnapshot<FeatureFlags> flagOptions)
	{
		this.FeatureFlags= flagOptions.Value;
	}

	public void OnGet()
	{
	}
}

Links
https://github.com/aws/aws-dotnet-extensions-configuration

@icelava Thanks for opening the issue. Could you please share screenshot(s) on what your AppConfig in AWS Console looks like and how you have defined a feature flag configuration profile in AppConfig with two booleans e.g.

Run = true
Jump = false

Thanks,
Ashish

It's just a feature flag config profile with flags defined.

It's just a feature flag config profile with flags defined.

@icelava Just for reference, debugged using the clone of the repository and defining the flags similar to your screenshot. The configuration is returned as a dictionary like below:

So supposed to be configuring for a Dictionary<sting, bool> object instead? But what's the configuration section name?
builder.Services.Configure<Dictionary<string, bool>>(builder.Configuration.GetSection("WHATCONFIGSECTIONNAME"));
If I place in the name of the configuration profile as config section name "FeatureFlags", all the page gets on construction is just an IOptionsSnapshot<Dictionary<string, bool>> of an empty dictionary.

Ok now that have some spare time to revisit this issue, I found that given the nature of the config parameter names returned from AppConfig to ASP.NET, it's not possible to consolidate all feature flags in one configuration profile into the same .NET strong-typed config object.
The class has to be written for each individual feature flag.
public class FeatureFlagRun
{
	public bool Enabled { get; set; }
}

Then the configuration builder draws in the feature flag as the "Run" config section alone.
builder.Services.Configure<FeatureFlagRun>(builder.Configuration.GetSection("Run"));
Given that this has to be done repeatedly for each feature flag, think it's just way more convenient to use free-form JSON.


SCP (from GreaseWeazle) to Flux to Victor 9k

Hello - it's having a bit of a hard time converting this to anything useful. I converted from SCP to flux, then tried to read as victor 9000. It was captured on an 80track 1.2MB drive, so was spinning at 360rpm. Single sided, so a 600kb victor disk, MS-DOS format.
v9kdump.zip

Interesting --- I see bad sectors down near track 0:
H.SS Tracks --->
0. 0 .B.B.........B..................................................................
0. 1 B...............................................................................
0. 2 .BB.............................................................................
0. 3 ..BB............................................................................
0. 4 B.BB............................................................................
0. 5 BB.B.................B..........................................................
0. 6 BB........................B.....................................................
0. 7 B..B.........B.B.....B..........................................................
0. 8 B.BB...B...B.BB.................................................................
0. 9 BBB.............................................................................
0.10 B............B.B................................................................
0.11 BB...........B.B................................................................
0.12 B..B...................................................................XXXXXXXXX
0.13 B..B..........B.............................................XXXXXXXXXXXXXXXXXXXX
0.14 B..B...........B................................XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.15 BBBB..........BB......................XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.16 B.B.........B..B.......B.B.XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.17 BBBB...B.....B..XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
0.18 B...XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

I'd be inclined to think it's just not a very good disk, but IIRC you said that you knew it was good? How? Do you have a real Victor 9K to test on? That suggests that my decoding could be improved (which is always true).
Also, given the triangular disk format, the MS-DOS disk format will have to be tweaked to work there. Do you know anything about this?

It does indeed work on a real V9k.  I'm not sure what MS-DOS format looks like on victor disks in truth as I've never managed to get one copied across.  Do you have a copy of the technical manual?

dos31.zip
Here's another one - this is an original factory duplicated disc (dos 3.1).

If you have access to a V9K, then what woud be invaluable is a file system disk image of one of these produced on the machine itself --- that would allow me to compare the data produced by FluxEngine with the known good data and figure out precisely where the bit errors are (and therefore find out why they're there).
(Also, wow! Aren't they super rare?)

I guess they are fairly rare. I got mine back in the early 90s. It's full of 6522s (thanks Chuck :))! - the issue is getting a file system image off the blasted thing. It is still working, I just checked it. The screen is a bit wobbly. I wonder if some of the old capacitors are feeling a bit unhappy. I think I need a null modem cable though to have any hope of getting anything off it...

One more... Here's a blank formatted disk from it (again, single sided).  This should be more predictable!
v9kblank.zip


HHH-12592

Add assertions to the test to ensure that the detached and merged collection are not the same, which currently fails.
I'll try reverting the fix and seeing if I can fix it a different way.
I've added a possible fix.
The tricky bit for this use case is that, even though the collection is lazy, the collection still gets loaded when merging the entity. This happens when the operation cascades to the collection.
The code that causes the eager load on merge is: https://github.com/hibernate/hibernate-orm/blob/master/hibernate-core/src/main/java/org/hibernate/loader/entity/CascadeEntityJoinWalker.java#L40-L44 .
The fix I came up with waits for the collection to be completely initialized, then set the initialized collection in the entity.
I'm not absolutely sure this is the right way to do this.
I  think it would be best if @sebersole and @barreiro both look at this.

@gbadner i had @barreiro take a look and he agreed with this in theory.  If we want to run with this I can certainly integrate it into 5.3.1 before we release.  If not, I'll open another PR to revert my original fix and we run without it.  Either way works for me at this point.

I'll merge it. Thanks!