Fixed Crate Position on M977

Fixed the position of the supply crates on the M977 vehicles, which were too high

Q
A

Bug fix?
yes

New feature?
no

Needs wipe?
no

Fixed issues

Description:
Fixed the position of the supply crates on the M977 vehicles, which were too high.
Content:

[ ] Content part

Successfully tested on:

[x] Local MP
[x] Dedicated MP

Compatibility checked with:
No mods required.

The Z-Value of the 2 trucks changed, because it is bassically the same truck.
I did check it with the supply crate type using the attachTo script command.
I checked it again now with all 3 types and increased each value a bit due to the ammo crate being a bit low.

Do I need to do anything else? If so, please tell me.
I think I addressed all the requested changes but they are still requested?
I find github very confusing, nothing like the git I am used to at work,
but I'd go through all this trouble again if I spot another bug, making
this awesome mission even better :)
Am Do., 29. Aug. 2019 um 12:35 Uhr schrieb Christian <
<EMAIL_ADDRESS>
@Wyqer requested changes on this pull request.
Also please add this as tweak listed in the changelog inside the Readme.md
Adapt it from the other entries concerning your mentioning.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KillahPotatoes/KP-Liberation/pull/660?email_source=notifications&email_token=ANBCHSE4WXXJZW5LVKEZYH3QG6RA7A5CNFSM4ISAIME2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCDCU6FA#pullrequestreview-281366292,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANBCHSEY2HIUGN5SQSXIASLQG6RA7ANCNFSM4ISAIMEQ
.

Do I  need to do anything else? If so, please tell me.
I think I addressed all the requested changes but they are still requested?
I find github very confusing, nothing like the git I am used to at work, but I'd go through all this trouble again if I spot another bug, making this awesome mission even better :)

I can create a new PR if you like, now that I know what I need to do.

No need. Squash merge is enough.

@FishAI
Merged and many thanks. Let me know, if you're also on our Discord. At least I haven't found a user names FishAI there.
@KillahPotatoes/kplib-testers
Even if it's merged, it would be great if you have a look at this in your tests of 0.96.6 before release. Thanks in advance.


Fixed #28109 -- Adjusted stacklevel of warnings raised when paginating unordered querysets.

Refs #26290.

@timgraham makes sense I'll adjust the release notes. I gave a try at testing it as well but abandoned after 20 minutes spent on it. Having slept on it I think the best avenue would be prevent the warning from being converted to an exception for the duration of the test but I haven't tried it yet. I'll give it a shot later today and if I don't come up with a solution I'll just commit it without a test 🤷♂ .

I couldn't figure out why assertRaisesMessage() works there (why is the warning converted to an exception). Or is it... because the test passes with assertWarns() also.

Oh, warnings.simplefilter("error", RuntimeWarning) in runtests.py.

yeah it's caused by the warnings.simplefilter('error', RuntimeWarning) call. I was thinking of doing a warnings.simplefilter('once', UnorderedObjectListWarning) in a warnings.catch_warnings() context and play around with module, file and lineno to assert it's working correctly.


Fixed rewriter for TypeScramber

The problem was that the type scrambler injected a reference of the ConfuserEx runtime framework into the target assembly instead of using the framework of the assembly.

:white_check_mark: Build ConfuserEx 944 completed (commit https://github.com/mkaring/ConfuserEx/commit/a9489f7dff by @mkaring)


Projective lighting

There are multiple aspects to this:

Shadow mapping (r1 had basic, variance, and screen-space shadows)
Shadow-mapped point lights (r1 didn't have these)
Mutable frustum light volumes (r1 only had immutable versions and used a pool)

Shadow mapping may be affected by #21

The current implementation still has back projection issues.
If the viewer stands behind the projective light, and a surface becomes between the light and the viewer, the GREATER_THAN depth test will result in the light being back-projected onto the surface:


Site is Down

I'm aware, but unfortunately can't do anything about it. Check the #swarfarm-support channel in the unofficial discord server for updates.


Add a switch to start _quake mode with focus mode off.

In the current implementation, the quake mode will ignore the launchMode setting and always launch in focus mode. This is fine for most users. But for the users who also want the tabs to be visible, the only way I think we can do that is to toggle focus mode from the command palette. I think starting the quake mode with an explicit switch should launch it in non focus mode. It should also respect the launchMode setting if this is defined in the configuration for the global summon action.
Maybe like this:
wt -w _quake --nonfocus

or
{
"keys": "win+`",
"launchMode": "default",
"command": {
"action": "globalSummon",
"name": "_quake",
"dropdownDuration": 200,
"toggleVisibility": true,
"monitor": "toMouse",
"desktop": "toCurrent"
}
}

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #9992 for more discussion.
/dup #9992


Bump scim2 dependency version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6886013584

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6886013584
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6890227842


Move sidenav arrow from right to left

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[ ] ✨ New Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 👷 Example Application
[ ] 🧑💻 Code Snippet
[ ] 🎨 Design
[ ] 📖 Content
[ ] 🧪 Tests
[ ] 🔖 Release
[ ] 🚩 Other

Description

This PR [adds/removes/fixes/replaces] this [feature/bug/etc].
Related Tickets & Documents

Resolves #
Mobile & Desktop Screenshots/Recordings

Added code snippets?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed

Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

No tests? Add a note

Added to documentation?

[ ] 📜 readme
[ ] 📜 contributing.md
[ ] 📓 general documentation
[ ] 🙅 no documentation needed

No docs? Add a note

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Hi, can you provide info on what was changed and why? I don't see any difference in how the chevron is displayed compared to the existing site

Hi, can you provide info on what was changed and why? I don't see any difference in how the chevron is displayed compared to the existing site

sure,
Arrow orientation has changed from right to left


Issue after installed 22.0

Describe the bug
I recieved an error after installing the newest version of blinkpy (22.0) via Pip
To Reproduce
Steps to reproduce the behavior:

Install v22
Run test script to download a single camera photo

Expected behavior
No error
blinkpy version (not needed if filling out Home Assistant version): 22
Log Output/Additional Information
2023-08-17 08:14:02,911 [INFO] **** Starting up ...
2023-08-17 08:14:02,911 [DEBUG] Using selector: EpollSelector
Traceback (most recent call last):
  File "blink-grab-one.py", line 58, in <module>
    auth = Auth(json_load("data/blink-creds.json"))
  File "/XXXXXX/blinkphotograb/env/lib/python3.8/site-packages/blinkpy/auth.py", line 33, in __init__
    self.token = login_data.get("token", None)
AttributeError: 'coroutine' object has no attribute 'get'
sys:1: RuntimeWarning: coroutine 'json_load' was never awaited
Exception ignored in: <function ClientSession.__del__ at 0x7f5493abc790>
Traceback (most recent call last):
  File "/XXXXXX/blinkphotograb/env/lib/python3.8/site-packages/aiohttp/client.py", line 348, in __del__
  File "/usr/lib/python3.8/asyncio/base_events.py", line 1740, in call_exception_handler
  File "/usr/lib/python3.8/logging/__init__.py", line 1475, in error
  File "/usr/lib/python3.8/logging/__init__.py", line 1589, in _log
  File "/usr/lib/python3.8/logging/__init__.py", line 1599, in handle
  File "/usr/lib/python3.8/logging/__init__.py", line 1661, in callHandlers
  File "/usr/lib/python3.8/logging/__init__.py", line 954, in handle
  File "/usr/lib/python3.8/logging/__init__.py", line 1186, in emit
  File "/usr/lib/python3.8/logging/__init__.py", line 1176, in _open
NameError: name 'open' is not defined

Easiest way to convert would be to look at the included blinksync.py or the blinkapp.py - both have been modified to work with async.

Here is the script with sensitive data removed (and some other functions that made calls to pushbullet etc..)
from blinkpy.blinkpy import Blink
from blinkpy.auth import Auth
from blinkpy.helpers.util import json_load
from PIL import Image, ImageFont, ImageDraw
import time
import datetime
import logging
import json
import os

logDir = "logs/"
programName = "blink-grab-one"
cameraName = "Back Deck"

# logging block
nowAsString = datetime.datetime.now().strftime("%Y%m")
logging.basicConfig(
    level=logging.DEBUG,
    format="%(asctime)s [%(levelname)s] %(message)s",
    handlers=[
#        logging.FileHandler("{}{}-{}.log".format(logDir, programName, nowAsString)),
        logging.StreamHandler()
    ]
)

def addTextToImage(file, text):
    img = Image.open(file)
    width, height = img.size
    draw = ImageDraw.Draw(img)
    font = ImageFont.truetype('data/Ubuntu-Regular.ttf', 25)
    draw.text((5,height-50), text, fill='white', font=font, stroke_width=2, stroke_fill='black')
    img.save(file)

logging.info("**** Starting up ...")
blink = Blink()
auth = Auth(json_load("data/blink-creds.json"))
blink.auth = auth
blink.start()
logging.debug("Logged in to blink..")

cnt = 0
pushFile = ""
for name, camera in blink.cameras.items():
    if name == cameraName:
        # take photo
        camera.snap_picture()
        # pause
        time.sleep(10)
        blink.refresh(force=True)
        # download to local file
        nowAsString = datetime.datetime.now().strftime("%Y%m%d.%H%M%S")
        filename = "data/{}.{}.jpg".format(name.replace(' ','.'), nowAsString)
        camera.image_to_file(filename)
        # add timestamp
        imageText = "{}.{}".format(name, nowAsString)
        addTextToImage(filename, imageText)
        logging.info(f'grabbed image {imageText} for camera {name} to file {filename}')
        logging.info(f'  attributes: {camera.attributes}')

I finally had some cycles this past weekend to re-write using asyncio and that resolved the issue.
Thank you again for keeping this lib up to date

Thanks for following up and testing!  This was a BIG rework and I won't be surprised by a bug or two :)


Won't render horizontal lines or vector symbols generated by katex.js

Most TeX commands rendered to HTML via katex.js result in proper canvas rendering via html2canvas.js with a notable exception of horizontal lines as in \frac{x}{y} which fails to render the horizontal line in the fraction as in the following example generated by this implementation.

Here is an HTML rendering using katex.js.

Here is a canvas rendering using html2canvas.js via katex.js.

Here you can see the HTML encoding generated by katex.js for the TeX code \overline{a}.  Perhaps this reveals why the horizontal line is not being rendered in the canvas with html2canvas.js.

html2canvas.js also does not render vector symbols generated by katex.js.  Here you can see the HTML rendering and the canvas rendering below.

Is this still an issue with v1.0.0? If so, could you please share an example on jsfiddle.


Update MD file on new branch 2017-22-14_22-11-08

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix: adjust declaration map paths

closes #7793
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Should we do #8825 first? Otherwise we'll need to leave reminders for ourselves to update stuff later

We can, yes. I was just trying to carve out bits of package that I can do without waiting on that decision - merge conflicts are probably minimal, if we do #8825 first.

closing in favour of #8825

In favor of means that you want to incorporate this into that PR? Because this fix is independent

I created #11287 for this, feel free to chime in!

I ended up writing

perl -pi -e 's/("(\.\.\/)+)\.\.\/\.\.\/(?=src\/lib\/)/\1/g' dist/**/*.d.ts.map
perl -pi -e 's/("(\.\.\/)+)\.\.\/\.\.\/(?=src\/lib\/)/\1/g' dist/*.d.ts.map

and adding sh fixpaths to the end of the package script in package.json
note that it's 2 levels too deep now


Difference in LCP measurements in web-vitals library and Lighthouse (No throttling)

I had implemented the web-vitals library in the website and ran lighthouse from dev tools. As shown in the screenshots, for the same run, web-vitals library is reporting LCP at 4.2s, whereas Lighthouse is reporting LCP at 7.9s. Could you please tell me, why am I getting different results in the same run?

Ciao @AnchalGupta3, the key of the answer is that you are comparing 🍊 with 🍏. One is a Lab Data and the other is Field Data. Even if at the core might seems the same, the logic and heuristic behind those two are different and they should never be compared. Use the Lab Data to prevent quick regression before deploying in Prod, or better understand opportunity Lighthouse will advice to address, and use Field Data with a large set of users to see how your product will perform differently with different device, network and platforms.

@AnchalGupta3 is this still an issue? I would expect the numbers to be the same for the same run, so presumably this was a bug that has been fixed?


crash when v-for child element is manually removed from the DOM

If manually delete v-for element from JQuery and then update v-for source VUE will crash trying to remove corresponding element
Following code works with VUE 0.12.x, but not working in the latest alpha
http://jsfiddle.net/hdpgrxp4/15/
There is no sense in removing element manually, that was legacy code, which was in separate file. Anyway maybe it worth to handle this situation.

v-repeat/v-for was never designed to work in situations like this, so I'd say this is a wont-fix.


SNOW-1791701: Support for Anaconda with pandas

I'm currently unable to run Snowpark with pandas in an Anaconda environment.
Creating the environment:
conda create -n sf -c conda-forge python=3.11 ipykernel pandas modin snowflake-snowpark-python[modin]

Next, if I run this
import snowflake.snowpark.modin.plugin

I get the following error:
RuntimeError: The pandas version installed (2.2.3) does not match the supported pandas version in Snowpark pandas (2.2.1). Run `pip install "snowflake-snowpark-python[modin]"` to resolve.

It works if I remove snowflake-snowpark-python with conda and reinstall with pip, but this is not recommended.

Re-raised as https://github.com/conda-forge/snowflake-snowpark-python-feedstock/issues/62


hi, how to get player structure??

cheat engine? or what?

yes cheat engine is what i use

thx bro


feat: add docs for grid and layout proposal

FX-3859
Description
This pull requests updates breakpoints and adds decision on layouts in Picasso and their compatibility with BASE design. Please see https://github.com/toptal/picasso/blob/FX-3859-add-grids-layout/docs/decisions/14-page-layout.md for the proposal.
Development checks

[N/A] Add changeset according to guidelines (if needed)
[N/A] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[N/A] Annotate all props in component with documentation
[N/A] Create examples for component
[N/A] Ensure that deployed demo has expected results and good examples
[N/A] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[N/A] Covered with tests
[ ] ⚠ Decision is made and added to the file before merge

Breaking change

[N/A] codemod is created and showcased in the changeset
[N/A] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

Merging due to 👍's in https://toptal-core.slack.com/archives/C052MRF4QJV/p1681838691318619


Bump pandas version and remove timestamp workarounds

Issue #, if available: #177
Description of changes: Pandas 0.25.0 was released, which fixes an issues with timestamp frequencies in the case of "W" and "M" frequency. This PR removes related workarounds. See here for related tests in the 0.25.x branch.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-230/1 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-230/1/index.html

Codecov Report

Merging #230 into master will decrease coverage by 0.61%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #230      +/-   ##
==========================================
- Coverage   81.85%   81.24%   -0.62%
==========================================
  Files         130      130
  Lines        7397     7396       -1
==========================================
- Hits         6055     6009      -46
- Misses       1342     1387      +45

Impacted Files
Coverage Δ

src/gluonts/model/npts/_model.py
95.65% <100%> (ø)
:arrow_up:

src/gluonts/transform.py
83.36% <100%> (-0.04%)
:arrow_down:

src/gluonts/dataset/repository/_artificial.py
35.29% <0%> (-64.71%)
:arrow_down:

...c/gluonts/dataset/artificial/generate_synthetic.py
12.94% <0%> (-10.59%)
:arrow_down:

src/gluonts/dataset/artificial/_base.py
36.67% <0%> (-6.6%)
:arrow_down:

src/gluonts/dataset/repository/datasets.py
75% <0%> (-5.56%)
:arrow_down:


Bug: unable to add Custom Items Heights for Multi Item Select DropDown

Issue comes when the last items are selected,
For reproducing we can use the examples - https://pastebin.com/iP8F6H0L
DropdownButton2 as Multiselect Dropdown with Checkboxes with items of different heights like dividers
Error: 'package:flutter/src/rendering/stack.dart': Failed assertion: line 749 pos 12: 'child != null': is not true.                                  #0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:51:61)                                  #1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:40:5)                                  #2      RenderIndexedStack._childAtIndex (package:flutter/src/rendering/stack.dart:749:12)                                  #3      RenderIndexedStack.paintStack (package:flutter/src/rendering/stack.dart:776:29)                                  #4      RenderStack.paint (package:flutter/src/rendering/stack.dart:670:7)                                  #5      RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2853:7)                                  #6      PaintingContext.paintChild (package:flutter/src/rendering/object.dart:253:13)                                  #7      RenderShiftedBox.paint (package:flutter/src/rendering/shifted_box.dart:74:15)                                  #8      RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2853:7)                                  #9      PaintingContext.paintChild (package:flutter/src/rendering/object.dart:253:13)                                  #10     RenderBoxContainerDefaultsMixin.defaultPaint (package:flutter/src/rendering/box.dart:2924:15)                                  #11     RenderFlex.paint (package:flutter/src/rendering/flex.dart:1089:7)                                  #12     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2853:7)                                  #13     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:253:13)                                  #14     RenderShiftedBox.paint (package:flutter/src/rendering/shifted_box.dart:74:15)                                  #15     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2853:7)                                  #16     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:253:13)                                  #17     RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:144:15)                                  #18     RenderObject._paintWithContext (package:flutter/src/rendering/object.dart:2853:7)                                  #19     PaintingContext.paintChild (package:flutter/src/rendering/object.dart:253:13)                                  #20     RenderProxyBoxMixin.paint (package:flutter/src/rendering/proxy_box.dart:144:15)

Please could you try this example
and select the last items
` // ignore_for_file: type=lint
import 'package:dropdown_button2/dropdown_button2.dart';
import 'package:flutter/material.dart';
class DummyScreen extends StatefulWidget {
const DummyScreen({super.key});
@override
State createState() => _DummyScreenState();
}
class _DummyScreenState extends State {
final List items = [
'Item1',
'Item2',
'Item3',
'Item4',
'Item5',
'Item6',
];
List selectedItems = [];
@override
Widget build(BuildContext context) => Scaffold(
body: Center(
child: DropdownButtonHideUnderline(
child: DropdownButton2(
isExpanded: true,
hint: Align(
alignment: AlignmentDirectional.center,
child: Text(
'Select Items',
style: TextStyle(
fontSize: 14,
color: Theme.of(context).hintColor,
),
),
),
items: _addDividersAfterItems(items),
//Use last selected item as the current value so if we've limited menu height, it scroll to last item.
value: selectedItems.isEmpty ? null : selectedItems.last,
onChanged: (value) {},
selectedItemBuilder: (context) {
return items.map(
(item) {
return Container(
alignment: AlignmentDirectional.center,
padding: const EdgeInsets.symmetric(horizontal: 16.0),
child: Text(
selectedItems.join(', '),
style: const TextStyle(
fontSize: 14,
overflow: TextOverflow.ellipsis,
),
maxLines: 1,
),
);
},
).toList();
},
menuItemStyleData: MenuItemStyleData(
padding: const EdgeInsets.symmetric(horizontal: 8.0),
customHeights: _getCustomItemsHeights(),
),
),
),
),
);
List<DropdownMenuItem> _addDividersAfterItems(List items) {
List<DropdownMenuItem> _menuItems = [];
for (var item in items) {
_menuItems.addAll(
[
DropdownMenuItem(
value: item,
child: StatefulBuilder(
builder: (context, menuSetState) {
final _isSelected = selectedItems.contains(item);
return InkWell(
onTap: () {
_isSelected
? selectedItems.remove(item)
: selectedItems.add(item);
//This rebuilds the StatefulWidget to update the button's text
setState(() {});
//This rebuilds the dropdownMenu Widget to update the check mark
menuSetState(() {});
},
child: Container(
height: double.infinity,
padding: const EdgeInsets.symmetric(horizontal: 16.0),
child: Row(
children: [
_isSelected
? const Icon(Icons.check_box_outlined)
: const Icon(Icons.check_box_outline_blank),
const SizedBox(width: 16),
Text(
item,
style: const TextStyle(
fontSize: 14,
),
),
],
),
),
);
},
),
),
//If it's last item, we will not add Divider after it.
if (item != items.last)
const DropdownMenuItem(
enabled: false,
child: Divider(),
),
],
);
}
return _menuItems;
}
List _getCustomItemsHeights() {
List _itemsHeights = [];
for (var i = 0; i < (items.length * 2) - 1; i++) {
if (i.isEven) {
_itemsHeights.add(40);
}
//Dividers indexes will be the odd indexes
if (i.isOdd) {
_itemsHeights.add(4);
}
}
return _itemsHeights;
}
}
`

Please could you test this code:-
https://pastebin.com/iP8F6H0L

This is the combination of DropdownButton2 with items of different heights like dividers and Multiselect Dropdown with Checkboxes

Doctor summary (to see all details, run flutter doctor -v): [✓] Flutter (Channel stable, 3.7.6, on macOS 13.2.1 22D68 darwin-arm64, locale     en-DE) [✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1) [✓] Xcode - develop for iOS and macOS (Xcode 14.2) [✓] Chrome - develop for the web [✓] Android Studio (version 2022.1) [✓] IntelliJ IDEA Ultimate Edition (version 2022.3.2) [✓] IntelliJ IDEA Ultimate Edition (version 2022.1) [✓] VS Code (version 1.76.0) [✓] Connected device (3 available) [✓] HTTP Host Availability

selectedItemBuilder is expecting the same length of your items. You're giving it your actual items without dividers (dividers are just disabled items but they should be counted).
I'll try to enhance dividers to not be counted as items or alternatively make selectedItemBuilder return a widget instead of List.
For now, you should consider adding dividers to your items inside selectedItemBuilder and it'll work.

@dhirajhimani have you been able to find a workaround? I am facing the same issue.

selectedItemBuilder is expecting the same length of your items. You're giving it your actual items without dividers (dividers are just disabled items but they should be counted).
I'll try to enhance dividers to not be counted as items or alternatively make selectedItemBuilder return a widget instead of List.
For now, you should consider adding dividers to your items inside selectedItemBuilder and it'll work.

Unfortunately, even after trying this work around I am facing the same issue too.
Could you please guide me a little on what im doing wrong?
 selectedItemBuilder: (context) {                               return selectedItems.map(                                 (item) {                                   return Container(                                     alignment: AlignmentDirectional.center,                                     child: Column(                                       children: [                                         Text(                                           selectedItems.join(', '),                                           softWrap: true,                                           style: const TextStyle(                                             fontFamily: "Arial",                                             fontSize: 16,                                             color: Color.fromRGBO(0, 0, 0, 1),                                             overflow: TextOverflow.ellipsis,                                           ),                                           maxLines: 1,                                         ),                                         Divider(),                                       ],                                     ),                                   );                                 },                               ).toList();

@giridat selectedItemBuilder should consider dividers items same as items


Allow import syntax for json in jest.config.ts

🚀 Feature Proposal
jest.config.ts should be able to use import syntax for json files in addition to the require syntax.
Eg. use resolveJsonModule: true

Motivation

In a multi-package repo, I'd like to import package.json to use as the display name for each jest project and take advantage of modern TS syntax, intellisense, and autocomplete.
Example

// jest.config.ts

// Currently working
const { something } = require("./someFile.json");

// Feature request
import  { something } from "./someFile.json";

Pitch
This would be a simple, backwards-compatible, quality-of-life addition for those of us who appreciate the usefulness of taking advantage of TS wherever we can.
I've done a bit of digging on my own, and I've narrowed down the problem area to the loadTSConfigFile function in the jest-config package.
https://github.com/facebook/jest/blob/bc50e7f360ab1845abbaa0b3ad788caead0d3174/packages/jest-config/src/readConfigFileAndSetRootDir.ts#L78-L90
Adding resolveJsonModule: true to the compilerOptions allowed me to use both import and require and is exactly the change I am requesting.
If there's any other information I could provide that would help, please let me know. Thanks!!

I'd have thought ts-node read the user config? We need to ensure we get CJS back in order do be able to load it via require, but other than that we should just use whatever config the user has configured

When parsing jest.config.ts, ts-node is registered directly with its own compiler options. See the snippet I linked.

Having ts-node merge instead of ignore user config seems like the correct solution then. PR welcome 🙂

@SimenB I agree that seems like the more proper, long-term solution. I do see that as havinga bigger buy-in and being more prone to edge cases though. I can try to find some time to work on that in my spare time, but it wouldn't be for a while.
Adding:
registerer = require('ts-node').register({
       compilerOptions: {
         module: 'CommonJS',
         resolveJsonModule: true, // This line here
       },
     });

would satisfy this issue for me for now while having a much smaller buy-in and still being fully backwards-compatible since the current require syntax would still work.
I can open a PR with the above mentioned change and create a separate issue about merging in the tsconfig from ts-node. Thoughts?

I'd prefer not to add more custom stuff. Seems to me to be a missing feature (bug?) of ts-node - they do a shallow merge (so our compilerOptions overrides user config) rather than a deep merge: https://github.com/TypeStrong/ts-node/blob/03355c15210731d948713a5edc624d4f215d1113/src/index.ts#L460-L464
Might be worth an issue over there for them to either deep merge by default, or allowing users to specify a way to configure the config after resolution

So, I did a bit more testing, and it looks like my specific problem was that I did not have a tsconfig.json` in my project root since I'm using a multipackage repo. Adding that (and including the resolveJsonModule option) fixed this issue for me.

Cool, so it does work?

Yes, although now I'm not sure how haha. I'm fine with closing this issue though.

Regardless, I think the correct way to solve this is having ts-node deeply merge options. If that is already the case, great! 😀 If not, I think an issue should be opened with them, and we could change what we do if needed afterwards


Sitemap is not generated on nuxt build

Describe the bug
This is my sitemap configuration:
sitemap: {
    exclude: [
      'aviso-legal',
      'politica-de-cookies',
      'politica-de-privacidad',
      'terminos-y-condiciones'
    ]
  },

Then I have the site url:
site: {
    url: 'https://...'
  },

When I run nuxt generate it outputs the sitemap properly, but when I run nuxt build the sitemap is not there.
Reproduction
No response
System / Nuxt Info
------------------------------
- Operating System: Darwin
- Node Version:     v20.8.0
- Nuxt Version:     3.7.4
- CLI Version:      3.9.0
- Nitro Version:    2.6.3
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, site, app, modules, sitemap, build, css, vite, vuetify, postcss, runtimeConfig, nitro, content, sourcemap, experimental
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

The sitemap will not be prerendered when you use nuxt build. It will be generated at runtime?
If I'm misunderstanding, then maybe try out the new v4 release and you can clarify further if you still have issues.

but if you have some pages pre-generated by configuration, I think the ideal is for the sitemap.xml to be created once the build command is run. I'm testing v4 and is the same thing.
I have to run npm run generate to generate the sitemap.
Couldn't we have a separate command to generate the sitemap at will?

I don't follow the logic sorry. Why is the ideal to generate it at build time? This would break any sitemap using dynamic URLs. Is there an issue you're facing generating it at runtime?
If you want to prerender it at build just add it to the prerender list
export default defineNuxtConfig({
  nitro: {
    prerender: {
      routes: ['/sitemap.xml']
    }
  }
})

I understand. In my case in concrete, I have everything generated with statics (prerender) except one route, which is SSR. Therefore, what I expect is that the sitemap of all the static ones is generated except for that specific route.
Don't know if it's the best, but at least it's what I would expect when running the npm run build command.


[hotfix][docs] Fix some error about the example of stateful source function in state documentatition

What is the purpose of the change
Fix some error about the example of stateful source function in state documentatition
Brief change log

Variable 'offset' needs be initialized before be used
When restoreState, offset should equals the sum of state

+1

@fhueske,hi, I think it's wrong.
because state redistribution happens when the parallelism of the operator is changed. Conceptually, the state in the checkpoint is the concatenated list of all lists returned by the parallel transformation function instances. When restoring from a checkpoint, the list is divided into sub-lists that are assigned to each parallel function instance.

@fhueske, developers will misunderstand the usage of ListCheckpointed which described in the document.
detail description in bellow file.
ListCheckpointed.java

+1，right

It doesn't make any sense to add up offsets. I can see how the current example is flawed (since it just ignores part of the state), but the correct fix is to rework the example such that the state can be aggregated on restore in a meaningful way.

@fhueske okay, I get your point.
@zentol, it's better to use total to replace offset in the document, do you think so?

No, that's not sufficient. Even if we're saying that they are just counts, then adding them up still doesn't make sense either, or rather is quite questionable.
For example, if you had a run with parallelism=2, then both sources would emit 1, 2, 3, ... ,X and 1, 2, 3, ... ,Y respectively, but when you restore with parallelism=1 the source would continue from X + Y. I have a very hard time figuring out a use-case where this makes sense.
I'm closing this PR since changing this example (if we really decide to do it) goes beyond the scope of a hotfix.


mechanism for a homeserver to suggest a default jitsi server to use for clients

As a homeserver admin, I might like to suggest a default jitsi server other than jitsi.riot.im to people who connect with riot to my server.

It's unclear how this would interact with the jitsi.preferredDomain riot config option.
It's also somewhat unclear whether this belongs in the matrix protocol at all, given jitsi integration is a riot-specific extension (so perhaps what we need is a way for servers to expose client-specific configuration parameters in a generic way?).

Jitsi is in the spec under the widgets banner, but indeed there's no definition for how clients acquire their defaults.

Could possibly use https://github.com/matrix-org/matrix-doc/pull/2301 or https://github.com/matrix-org/matrix-doc/pull/2063

As end user perspective: The unique thing with Jitsi within Matrix (or Riot/client) is that instead of "just" a widget, there exist dedicated call button for it, making it in some sense part of at least Riot, if not Matrix.
As developer'ish perspective: Most proper 'author' to define what Jitsi ought to be used would be room admins, which would bring us back into integration manager(s) and such. Does current way of using IM also apply to riot call button method? This method would ofcourse need the IM alltogether and generally focus to make custom one ever so easier to deploy.
Next best thing would be the client well-known, that would at least point HS users towards "HS-trusted" Jitsi, while that method loses control over room specific one.
Maybe even user-overridable setting, while I don't know feasibility of that..
And ofcourse all of these could be supported at once, within well thought priorities.
And just as I was about to finish, indeed somthing like @uhoreg just linked could also work... Thing is that most privacy-minded persons will have something to say no matter what is done, at the same time this shouldn't be too hard, especially in end-user perspective.. While the issue there remains that "who should control what Jitsi-instance is used".

So far the proposal is for clients to just do it through .well-known, as they're allowed to: https://github.com/matrix-org/matrix-react-sdk/pull/4348

I like the idea of user default Jitsi offer in .well-known, if room wants to offer somwthing else, I guess they still can(?)

For anyone else stumbling across this issue:
According to the documentation added in https://github.com/vector-im/element-web/pull/13047
Element already supports using the client .well-known for the default jitsi server

Indeed it does.
See second half of https://github.com/vector-im/element-web/blob/develop/docs/jitsi.md#configuring-element-to-use-your-self-hosted-jitsi-server

Yep, it came more or less after this ticket, so I guess closing as implemented should be done :)

Would probably be good for it to documented in the spec rather than a client's doc

I think I did add the good line on nginx
return 200 '{"m.homeserver": {"base_url": "https://matrix.defis.info/"},"im.vector.riot.jitsi": {"preferredDomain": "ourjitsi.defis.info"}}';
but when I want to add a jitsi widget, it keep offering me a jitsi.riot.im room

instead of a ourjitsi.defis.info
Should I write "preferredDomain": "https://ourjitsi.defis.info/" instead of "preferredDomain": "ourjitsi.defis.info" ?

That config only affects the call button. When using an integrations manager like your screenshot shows it always uses the one it has configured internally.

OK, got it. Thank you for pointing this out.
Still, it's not working.
If I click on video call button in a conference, It creates a jitsi conference to https://app.element.io/jitsi.html#xxxxxxyyyyyy and not to ourjitsi.defis.info
So I probably write something wrong but what?

That jitsi.html is just a wrapper. After clicking join conference see whether the iframe inside is pointing.

https://jitsi.riot.im/JitsiMsrfdhjwrugogrlgvaloyubg


include default SimpleTimeCluster configuration

Offline PR #1275 removes default SimpleTimeCluster configuration from source code, and hence requires full configuration in fcl. This PR accounts for the remaining configuration in the SimpleTimeCluster defined here, which then propagates to validation jobs.

Hello @edcallaghan did you have any chance to run a trigger test with the proposed new configuration? Once you implement the change I requested and confirm you did a test run we can go ahead and merge it.

@gianipez with everything at head, that command does run without error. But switching mu2e_trig_config to include these changes, I get the following complaint about incomplete definitions:

   ************************** Mu2e Offline **************************
     art v3_14_03    root v6_28_10a    KinKal v03_00_01
     build  /exp/mu2e/app/users/ejc3/mu2e/latest
     build  sl7-prof-e28-p056    05/25/24 17:17:43
   ******************************************************************
Warning in <UnknownClass::SetDisplay>: DISPLAY not set, setting it to <IP_ADDRESS>:0.0
Conditions file: /exp/mu2e/app/users/ejc3/mu2e/latest/Offline/ConditionsService/data/conditions_01.txt
Conditions lines: 38  hash:<PHONE_NUMBER>555253397
DbService  MDC2020_best v1_3   mu2e_conditions_prd
DbService  textFiles:
GlobalConstants file: /exp/mu2e/app/users/ejc3/mu2e/latest/Offline/GlobalConstantsService/data/globalConstants_01.txt
GlobalConstants lines: 146  hash:<PHONE_NUMBER>4763074497
Setting KinKal scattering Dahl-Lynch fraction to 0.995
BkgANNSHU weights Offline/Mu2eKinKal/data/TrainBkgTrigger.dat cut 0.2 freezing ( 0x1 : Inactive )
Setting KinKal scattering Dahl-Lynch fraction to 0.995
BkgANNSHU weights Offline/Mu2eKinKal/data/TrainBkgTrigger.dat cut 0.2 freezing ( 0x1 : Inactive )
Setting KinKal scattering Dahl-Lynch fraction to 0.995
BkgANNSHU weights Offline/Mu2eKinKal/data/TrainBkgTrigger.dat cut 0.2 freezing ( 0x1 : Inactive )
%MSG-s ArtException:  MergeTriggerInfo:tprHelixUeTriggerInfoMerger@Construction  03-Jun-2024 20:07:16 CDT ModuleConstruction
cet::exception caught in art
---- Configuration BEGIN

  ====================================================================================================

  !! The following modules have been misconfigured: !!

  ----------------------------------------------------------------------------------------------------

  Module label: TTtimeClusterFinder
  module_type : TimeClusterFinder

  Any parameters prefaced with '#' are optional.
  Missing parameters:

   -    HitSelectionBits: [
   -       <string>,
   -       ...
   -    ]

   -    HitBackgroundBits: [
   -       <string>,
   -       ...
   -    ]

   -    CaloClusterMinE: <float>

   -    CaloClusterWeight: <float>

   -    DtMax: <float>

   -    MinNHits: <unsigned int>

   -    MinKeepHitMVA: <float>

   -    MinAddHitMVA: <float>

   -    MaxdPhi: <float>

   -    Tmin: <float>

   -    Tmax: <float>

   -    Tbin: <float>

   -    AveragePitch: <float>

   -    Ymin: <float>

   -    RefineClusters: <bool>

   -    PrefilterCluster: <bool>

   -    RecoverHits: <bool>

   -    PeakWidth: <int>

  ----------------------------------------------------------------------------------------------------

  Module label: TTtimeClusterFinderUe
  module_type : TimeClusterFinder

  Any parameters prefaced with '#' are optional.
  Missing parameters:

   -    HitSelectionBits: [
   -       <string>,
   -       ...
   -    ]

   -    HitBackgroundBits: [
   -       <string>,
   -       ...
   -    ]

   -    CaloClusterMinE: <float>

   -    CaloClusterWeight: <float>

   -    DtMax: <float>

   -    MinNHits: <unsigned int>

   -    MinKeepHitMVA: <float>

   -    MinAddHitMVA: <float>

   -    MaxdPhi: <float>

   -    Tmin: <float>

   -    Tmax: <float>

   -    Tbin: <float>

   -    Ymin: <float>

   -    RefineClusters: <bool>

   -    PrefilterCluster: <bool>

   -    RecoverHits: <bool>

   -    PeakWidth: <int>

  ====================================================================================================
---- Configuration END
%MSG
Art has completed and will exit with status 9.```

Dumping the config when running at head, the configuration is fine. But these are definitions for `TimeClusterFinder` modules, and this PR only changes a definition for a `SimpleTimeCluster` module, and AFAICT there is no coupling between them. So I must be missing something, because I do not see a connection.

@gianipez with everything at head, that command does run without error. But switching mu2e_trig_config to include these changes, I get the following complaint about incomplete definitions:
$ mu2e -c mu2e_trig_config/test/triggerTest.fcl -s /pnfs/mu2e/tape/phy-sim/dig/mu2e/NoPrimaryMix1BBUntriggered/MDC2020r_best_v1_0/art/0a/fa/dig.mu2e.NoPrimaryMix1BBUntriggered.MDC2020r_best_v1_0.001202_00000240.art -n 100

   ************************** Mu2e Offline **************************
     art v3_14_03    root v6_28_10a    KinKal v03_00_01
     build  /exp/mu2e/app/users/ejc3/mu2e/latest
     build  sl7-prof-e28-p056    05/25/24 17:17:43
   ******************************************************************
Warning in <UnknownClass::SetDisplay>: DISPLAY not set, setting it to <IP_ADDRESS>:0.0
Conditions file: /exp/mu2e/app/users/ejc3/mu2e/latest/Offline/ConditionsService/data/conditions_01.txt
Conditions lines: 38  hash:<PHONE_NUMBER>555253397
DbService  MDC2020_best v1_3   mu2e_conditions_prd
DbService  textFiles:
GlobalConstants file: /exp/mu2e/app/users/ejc3/mu2e/latest/Offline/GlobalConstantsService/data/globalConstants_01.txt
GlobalConstants lines: 146  hash:<PHONE_NUMBER>4763074497
Setting KinKal scattering Dahl-Lynch fraction to 0.995
BkgANNSHU weights Offline/Mu2eKinKal/data/TrainBkgTrigger.dat cut 0.2 freezing ( 0x1 : Inactive )
Setting KinKal scattering Dahl-Lynch fraction to 0.995
BkgANNSHU weights Offline/Mu2eKinKal/data/TrainBkgTrigger.dat cut 0.2 freezing ( 0x1 : Inactive )
Setting KinKal scattering Dahl-Lynch fraction to 0.995
BkgANNSHU weights Offline/Mu2eKinKal/data/TrainBkgTrigger.dat cut 0.2 freezing ( 0x1 : Inactive )
%MSG-s ArtException:  MergeTriggerInfo:tprHelixUeTriggerInfoMerger@Construction  03-Jun-2024 20:07:16 CDT ModuleConstruction
cet::exception caught in art
---- Configuration BEGIN

  ====================================================================================================

  !! The following modules have been misconfigured: !!

  ----------------------------------------------------------------------------------------------------

  Module label: TTtimeClusterFinder
  module_type : TimeClusterFinder

  Any parameters prefaced with '#' are optional.
  Missing parameters:

   -    HitSelectionBits: [
   -       <string>,
   -       ...
   -    ]

   -    HitBackgroundBits: [
   -       <string>,
   -       ...
   -    ]

   -    CaloClusterMinE: <float>

   -    CaloClusterWeight: <float>

   -    DtMax: <float>

   -    MinNHits: <unsigned int>

   -    MinKeepHitMVA: <float>

   -    MinAddHitMVA: <float>

   -    MaxdPhi: <float>

   -    Tmin: <float>

   -    Tmax: <float>

   -    Tbin: <float>

   -    AveragePitch: <float>

   -    Ymin: <float>

   -    RefineClusters: <bool>

   -    PrefilterCluster: <bool>

   -    RecoverHits: <bool>

   -    PeakWidth: <int>

  ----------------------------------------------------------------------------------------------------

  Module label: TTtimeClusterFinderUe
  module_type : TimeClusterFinder

  Any parameters prefaced with '#' are optional.
  Missing parameters:

   -    HitSelectionBits: [
   -       <string>,
   -       ...
   -    ]

   -    HitBackgroundBits: [
   -       <string>,
   -       ...
   -    ]

   -    CaloClusterMinE: <float>

   -    CaloClusterWeight: <float>

   -    DtMax: <float>

   -    MinNHits: <unsigned int>

   -    MinKeepHitMVA: <float>

   -    MinAddHitMVA: <float>

   -    MaxdPhi: <float>

   -    Tmin: <float>

   -    Tmax: <float>

   -    Tbin: <float>

   -    Ymin: <float>

   -    RefineClusters: <bool>

   -    PrefilterCluster: <bool>

   -    RecoverHits: <bool>

   -    PeakWidth: <int>

  ====================================================================================================
---- Configuration END
%MSG
Art has completed and will exit with status 9.

Dumping the config when running at head, the configuration is fine. But these are definitions for TimeClusterFinder modules, and this PR only changes a definition for a SimpleTimeCluster module, and AFAICT there is no coupling between them. So I must be missing something, because I do not see a connection.

This is happening because https://github.com/Mu2e/Offline/blob/main/CosmicReco/fcl/prolog.fcl contains an incomplete module instance of TimeClusterFinderModule (lines from 10 to 20)  name in the exact way as in  https://github.com/Mu2e/Offline/blob/main/TrkPatRec/fcl/prolog.fcl (lines 21 to 51).
IMHO I would move the module instances referring to TimeClusterFinder currently present in CosmicReco/fcl/prolog.fcl in TrkPatRec/fcl/prolog.fcl to make it more maintainable. @kutschke and @brownd1978 can comment on this.

You are the right, it is the include. I tested that at head but must have goofed, because doing it again now I see that it does introduce the error.

I am 99% sure that I understand this at the big picture level.  Still a few details to go.  In Offline/CosmicReco/fcl/prolog.fcl, a fcl table named TimeClusterFinder is defined at outermost scope.  It appears that the include overrides another definition of TimeClusterFinder, also at outermost scope.  I am not sure where the other definition is but I know how to find out.
So the core problem is that both of these definitions are out outermost scope and not nested into a namespace.
Here is a proposed forward that I am not in love with but it seems the most expedient.

Revert to Ed's original solution of just copying the full configuration from CosmicReco/fcl/prolog.fcl into core/trigProducers.fcl.  Test and merge this PR.
In a second PR add a namespace to CosmicReco/fcl/prolog.fcl
In a third PR find the other fcl defintions that need to be namespaced and do it.
In a fourth PR go back to using #include and @table for Ed's change to core/trigProducers.fcl
Scrub our other prologs for namespaces that should be there but are not.

Giani: on what time scale can you look after 3?  Ed can you look after 1, 2, 4?

Hello Rob,
Thanks for the planning. For me working on point (3) I’d like to make it clear that I think we should organize the prolog fcl files as in CalPatRec/fcl/prolog.fcl, where EVERYTHING is kept within a namespace. Dave used a different style, so I want to be sure that changing the prolog files as I suggest in TrkPatRec and other directories is OK with everybody.
Once I receive the green light, I need no more than 2 hr for making the changes and testing them.
Gianantonio Pezzullo, Ph.D. (he/him/his)
@.***

Address: Department of Physics, Yale University, 56 Hillhouse, New Haven, CT-06511, USA
Phone   : +1<PHONE_NUMBER>

On Jun 4, 2024, at 6:53 PM, Rob Kutschke @.***> wrote:
I am 99% sure that I understand this at the big picture level. Still a few details to go. In Offline/CosmicReco/fcl/prolog.fcl, a fcl table named TimeClusterFinder is defined at outermost scope. It appears that the include overrides another definition of TimeClusterFinder, also at outermost scope. I am not sure where the other definition is but I know how to find out.
So the core problem is that both of these definitions are out outermost scope and not nested into a namespace.
Here is a proposed forward that I am not in love with but it seems the most expedient.
Revert to Ed's original solution of just copying the full configuration from CosmicReco/fcl/prolog.fcl into core/trigProducers.fcl. Test and merge this PR.
In a second PR add a namespace to CosmicReco/fcl/prolog.fcl
In a third PR find the other fcl defintions that need to be namespaced and do it.
In a fourth PR go back to using #include and @table https://github.com/table for Ed's change to core/trigProducers.fcl
Scrub our other prologs for namespaces that should be there but are not.
Giani: on what time scale can you look after 3? Ed can you look after 1, 2, 4?
—
Reply to this email directly, view it on GitHub https://github.com/Mu2e/mu2e_trig_config/pull/28#issuecomment-2147991039, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGCD3EC6SGDDMO7UU3FCZ43ZFXWHNAVCNFSM6AAAAABIP74F2SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXHE4TCMBTHE.
You are receiving this because you were mentioned.

@brownd1978 Adding you to the discussion.
The issue today is that both of
  Offline/CosmicReco/fcl/prolog.fcl
  Offline/TrkPatRec/fcl/prolog.fcl

define a table named TimeClusterFinder at outermost scope.  So there is a collision if these are ever included in the same file.  There is a at least one use case for which they will both need to be visible in the same fcl file: for an off-spill trigger that runs the on-spill paths plus the through-going cosmic paths.
We can solve the immediate problem by updating  Offline/CosmicReco/fcl/prolog.fcl
CosmicReco CosmicReco : {
   # existing content goes in here.
}

There will be follow-on work to find all of the places that need to be updated.
For future proofing, I suggest we also do the same for   Offline/TrkPatRec/fcl/prolog.fcl  and other prolog.fcl that do not use the "namespace" style.  Again there will be follow-on work.   Giani says he will do that work.   I think that TrkPatRec is your code so I want to check if you on board with this or do you want to go another direction?

Here is a proposed forward that I am not in love with but it seems the most expedient.
...
Giani: on what time scale can you look after 3? Ed can you look after 1, 2, 4?

I am happy with this plan, and can take point on #s 1, 2, and 4; but #2 will involve getting a greenlight from stakeholders mentioned in Offline issue #1277 and so may have a longer turnaround then the others.

I am happy with this plan, and can take point on #s (1), (2), and (4); but (2) will involve getting a greenlight from stakeholders mentioned in Offline issue #1277 and so may have a longer turnaround then the others.

Thanks Ed.  Please proceed with 1.  After the test that  Giani wanted pass, please push.  Then we can merge this PR and Offline#1275 .   After that please make a draft PR for to address 2 and we can  involve the stakeholders during the review.
Step 3 can proceed in parallel.
Does that work for you?

That sounds good to me. I should be able to pick this back up in the afternoon.

@kutschke @gianipez The latest commit reverts to a direct configuration, as opposed to a table override of a table from an Offline prolog. Both of the test fcls from under .../test/ run without error.

Thanks Ed.  @gianipez - the ball is in your court.

@gianipez Do you an estimate for when you can get to this?

@gianipez Do you an estimate for when you can get to this?

I'm still waiting for the green light from Dave.


how to export the whole project with images and fonts?

I'm new to XD to flutter. I like this tool, but I am confused by the lack of documentation and manuals. so I have a file xd. I want to create a flutter project from this file. I want to import all the images and all fonts. is it possible? any advice, I didn’t find how to do it.

Have you looked at the README?
https://github.com/AdobeXD/xd-to-flutter-plugin/blob/master/README.md
Feel free to reopen this if you feel like it doesn't cover specific topics well enough.


sample-code issues

need to create build/ folder.

https://github.com/FS1360472174/nosql-cassandra/commit/66b31d1cff22c2da7df4c9c7f70c50db75234c17


The top-bar settings are missing on the trading screen

I added it to the trading screen template, but since that screen was not componentized we have to add it to the other trading screens manually. Should do that to keep them aligned.

Fixed, I copied the top-bar into all the trading screen instances!


Adds ability to define arbitrary attribute fields in SidreDataCollection [tracer-dev]

Refactors the FieldMap in DataCollection into its own self-contained class
as a map from std::string to T*, where T is the template parameter.
Implements the field map for GridFunctions and QuadratureFunctions in terms of this class
Adds functions to SidreDataCollection to register named attribute fields (Array<int> fields associated with the mesh or boundary mesh's elements)
Attribute fields are registered with the mesh blueprint
Minor fixes for dumping vtk point meshes.

I wasn't sure if this warranted a note in the CHANGELOG.  If so, here is what I wanted to add:

Added ability to register named attribute fields with a SidreDataCollection. Attribute fields associate a single integer with each element in the mesh.

It might also be a good idea to add a note about improved support for point meshes.

I think the changes are orthogonal to those in PR#394 for the conduit blueprint data collection (https://github.com/mfem/mfem/pull/394/files)

@kennyweiss, I simplified the implementation of Mesh::PrintVTK a bit, take a look just in case.

Thanks @v-dobrev.
The Mesh::PrintVTK changes look good on first glance.  I will run it with my Point mesh examples and let you know if there are any problems.


Entrance/exit from collection not supported by VirtualizedList

Description
Entrance/exit from collection not supported by VirtualizedList. No components can be set as any collection role. With support for "List" or "Grid" roles would these would likely work as expected.
This issue is covered in-depth by parent issue #30861, please check there for more details.
React Native version:
0.63

Hi @amarlette , I would like to pick this up. Let me know if this is resolved or still open issue?

https://github.com/facebook/react-native/pull/34267


Add subexpressions

If you call a function whose results are not otherwise immediately available they should be added as a separate expression. e.g. if you have: foo x y = bar x y + foo y z then it would be useful to have dedicated values for bar x y and foo y z than than just $result.

I'm starting to work on this one...


Live examples in the documentation

Would be awesome to have embedded examples of the actual file-picker running in the documentation. Could be

running against demo.owncloud.com as a backend (kind of contradicts https://github.com/owncloud/file-picker/issues/43)
have a fake bearer token + a minimal mock in place for folder listing

Idea came up in the last reviews from @michaelstingl 💪

@kulmann do you mean file-picker should be in https://docs.owncloud.com ?
In case, pls file a docs issue for this 😄

Hey @mmattel, we have docs within this repo that get deployed to our dev docs at https://owncloud.dev/integration/file_picker/ ;-) As of now nothing to be taken care of by you. Thanks for checking! :-)

Idea came up in the last reviews from @michaelstingl 💪

Inspiration from Uppy docs and Dropbox Chooser, Saver, Embedder docs.


⚠ Bofrost.de has degraded performance

In 1bffb25, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1659 ms

Resolved: Bofrost.de performance has improved in a2aad92 after 19 minutes.


update producer.ex to remove compilation warning

warning: Stream.chunk/4 is deprecated. Use Stream.chunk_every/4 instead
lib/parallel_stream/producer.ex:16: ParallelStream.Producer.build!/4

Coverage remained the same at 100.0% when pulling e15d5b599b7c163e20ffabde008e041987d582b8 on kjinho:patch-1 into 75788a6db3bae77af515bb9d0cd213182f17d709 on beatrichartz:master.

Any update on this?


Update Ionic sass to v1.0.1

I tried out your branch and could not get it too compile, there's an error. On line 50 of the main file you put a ";" instand of a normal comma.

I'm going to update this myself.


marked-terminal@6 is deprecated

There is a deprecation warning on the marked-terminal dependency:
$ yarn
warning @arethetypeswrong/cli ><EMAIL_ADDRESS>should have been major release. Use 7.0.0

Do you need help in upgrading it?

Did my best in #186


Fixed typos and removed duplicate lines

There were some typos on line 152, 154, 155 and 156 "-generic" and a duplicate of line 155 which I removed

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

@code2exe
thank you for the inputs.
#sign-off


Add/country indicators

Changes

Add country indicators

This PR doesn't introduce:

[x] typos
[x] conflict markers
[x] unwanted comments
[x] temporary files, auto-generated files or secret keys
[x] console.log meant for debugging
[x] codegen errors

Closing in favor of https://github.com/IFRCGo/go-web-app/pull/518


Interpolar animation

How is it possible to update the marker location smoothly on flutter mobile app.
With the current approach the marker is jusmping to the new location. I wanted to have a smooth movement like uber vehicles LatLong are from firestore.
here is my current Approach
var markerIdVal = value.docs[b].id;

            final MarkerId markerId = MarkerId(markerIdVal);

            final Marker marker = Marker(
              markerId: markerId,
              position: LatLng(latitude, longitude),
              icon: carIcon,
);

Hi @heeshala,
This issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/ . You may also explore the official  maps sample here
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you.


Updated to Alpine 3.7 and glibc 2.27-r0

Update to Alpine 3.7 and glibc 2.27-r0. Tested with Server JRE 8u161-b12 and Confluence 6.7.0 without any issues so far.
I also updated the description on how to use a specific Java version.
It looks like it's only possible to download the last two versions from the public area and you would need to sign in with an Oracle account to download archived versions!? Beside that, I was only able to download when specifying a dedicated hash for every Java version..
Is there another trick or how did you, for example, build the image centos.server-jre.8.151 (available on Docker hub) a month ago? The Java CentOS Dockerfile uses still 8.121-b13..

Yes, Oracle changes download requirements from time to time. The download page hash has been introduced two years ago.

Thank for the update!


Fix RLHF loss metrics & single-gpu training script

This PR fixes:

the actor/critic mean loss calculation
step-3 training script for 1.3b model on single gpu
some typos

@microsoft-github-policy-service agree


Text Input with typing feedback (iOS) doesn't work

Characters and Words are not read back when setting is enabled.
/cc @cbracken

This is now a dup of https://github.com/flutter/flutter/issues/12786.


Task cancellation

In some situation it appears that some node task takes too long for some reasons.
In order to not block the service forever, we might want to cancel a task to allow the task list to continue to be processed.
Cancellation is indeed what we can think of in the first place to solve this kind of problem. But it brings quite some questions:

cancelling a task means we stop the execution of an action in the middle, which could cause inconsistent state of the service, how do we prevent that
shoulnd't we instead give the developer more tools to avoid having this kind of situation at all ? (https://github.com/threefoldtech/0-templates/issues/216)

From a code perspective I don't think implementing this is going to be hard. From the top of my head I can see:

Add a cancel action on the Task class
Add and API endpoint like DELETE /services/{service_guid}/task_list/{task_guid}
Exposing this interface on the service proxy too

@0xIslamTaha @serboctor


chore: bump rustc versions

Looks like -Os is now taken much more seriously by rustc at the expense of the cycle consumption. @luc-blaeser what about switching to -O2 or such?

Obsolete


Github configuration

Take care of branch protection, release, etc.

@westonpace
Thank you for working on this.
Let's mark this close. If we get new requests, we can always open an issue.


No more fail on error for st2ctl

Removes set -e on error, which allows services to start/stop cleanly when NG_INIT=true

+1

merge

Same! :fireworks: :tada:
Was annoying if any service down.


conda installation

Hi, I am testing out NOTEBOOKS_ . Do you have any wiki's on how to install bioconda?
Cheers,
Matt

Hello Matt! We don't support conda for now, we only support pip. BUT! There's a new image for BioPython that you can try out!


select toDateTime(now()) result is not correct

By using the sql_alchemy-clickhouse to connect the clickhouse server execute
query select toDateTime(now());
the result is not correct :
result is
10:15:49

However useing clickhouse client command line to execute the same query
the result is
18:16:75
and this result is correct

apache-superset        0.38.1
infi.clickhouse-orm    2.1.0
sqlalchemy-clickhouse master
Python 3.6.8
datetime is not support!!!
superset is  hand down!!!


How I get MySQL Error Code.

I want error handling with error code of mysql.
example) 1602 Duplicate key.
But I enable knex debug is true, I can't get it with knex.
Can knex get error code?

Exception got from database is the one thrown by the db driver. So you can check if mysql driver stores the error code in the exception. Knex doesnt extract that.


add new logo for frsca

Signed-off-by: pxp928<EMAIL_ADDRESS>

I don't know why the lint doesn't like inline HTML when it's the only way to do stuff like aligning images.

Created PR #340 to fix the CI issue for vendoring


Implementation of W3C verifiable conditions in/for the VC library

Definition of done:

The W3C Verifiable Conditions structure of the DID Document is implemented and ready for the issue() function

Research started here
https://docs.google.com/spreadsheets/d/1zyFlATImpzndK9wu8J9DaWslZqITNM-5VCUxgGVn5oE

compliance with JWS
https://www.rfc-editor.org/rfc/rfc7519#section-4.1.5

need to use nested JWT
https://www.rfc-editor.org/rfc/rfc7519#section-7.1

Done. Please review

https://github.com/Tonomy-Foundation/did-jwt/pull/1/files
https://github.com/Tonomy-Foundation/did-jwt-vc/pull/1/files
https://github.com/Tonomy-Foundation/Antelope-SSI-Toolkit/pull/8/files


Can I adjust the focus of the lens?

I'm using kalibr to calibrate the realsense d435i RGB camera and the instruction says it'd be better to Ensure that your sensor is in focus. kalibr offers a ROS node kalibr_camera_focus but this node asks the user to adjust the focus of the lens.
Is the focus fixed physically? If it were not, can I adjust the focus of the lens?

Hi @niebayes  The focus of RealSense cameras is fixed, as confirmed in the link below
https://github.com/IntelRealSense/librealsense/issues/7814

got it.


don't use ClampToRange in modular

Don't do unnecessary clamping and use of wider types in modular transforms and in the weighted predictor.
Small speedup of 1-4%, for example:
Modular falcon:
before: 4064 x 2704, geomean: 24.53 MP/s [21.31, 24.99], 30 reps, 4 threads.
after: 4064 x 2704, geomean: 24.77 MP/s [21.54, 25.62], 30 reps, 4 threads.
Modular default speed:
before: 4064 x 2704, geomean: 14.51 MP/s [13.62, 14.76], 30 reps, 4 threads.
after: 4064 x 2704, geomean: 14.61 MP/s [13.73, 14.99], 30 reps, 4 threads.
Modular default speed -Q 80:
before: 4064 x 2704, geomean: 17.86 MP/s [15.21, 18.50], 30 reps, 4 threads.
after: 4064 x 2704, geomean: 18.13 MP/s [14.79, 18.53], 30 reps, 4 threads.
Modular thunder:
before: 4064 x 2704, geomean: 49.24 MP/s [40.27, 51.04], 30 reps, 4 threads.
after: 4064 x 2704, geomean: 51.29 MP/s [42.49, 53.30], 30 reps, 4 threads.

Can you also remove ClampToRange from common.h?


Fix frame update bug for dynamic inputs

Fixes https://github.com/livebook-dev/kino/issues/345

@jannikbecher you got it right, thanks! I made a couple related updates in #2219 and added a test :)


Add types ?

I just found an issue in numpy's usage of spins that would have been caught by types,
but as spin does not seem to have types (and the py.typed marker), mypy just stop inferring.

I wouldn't mind having types.
Just out of curiosity, what is the error that would have been caught?

https://github.com/numpy/numpy/pull/26304, there was a migration from subprocess.run to  spin.utils.run and the usage of the return value of spin.util.run was improper when the subprocess would fail.

Types scaffolding has been put in place with #188. I was going to add a py.typed file, but after some headscratching this does not seem to pertain to our situation (we don't provide any data structures that other libraries need)? But, happy to be corrected; I have a branch with it set up.
From here, we can continue typing the rest of spin.


Please release the new code: v.1.3.0 is missing the notify script settings

I need the full set of keepalive configuration attributes you have in your development code:
:smtp_alert => true # Enables SMTP notifications
:notify_master => nil # Script to be run on transition to MASTER state
:notify_backup => nil # Script to be run on transition to BACKUP state
:notify_fault => nil # Script to be run when check_script fails
Do you now when the new version will be released?

+1

@LuigiClemente-Awin @uzyexe i'm going to work on polishing the documentation and doing the last bit of prep to get 2.0 out the door, so expect to see improved support for these keywords in the next week or two at most.


feat: mandatory suppression explanations

Summary
This pull request makes suppression explanations mandatory.

Suppression linting requires explanations.
CLI and LSP decisions add placeholder suppression explanations.
A new --suppression-explanation review CLI flag is supported.
Documentation explains that explanations are mandatory.

Resolves #972
Test Plan
./rome test

@bitpshr: is this PR changes also how rome check --review works? In theory now, when deciding to suppress the rule, we should prompt the reason why that rule must be suppressed.

@sebmck I worry about opening up something like suppression explanations to configuration since their benefit is fairly objective: including them will always increase suppression understanding, whereas not including them only ever saves keystrokes. I feel that this is similar to implicit return statements or any other keystroke-saving-but-technically-bad-practice rule, and I think Rome should lean on objectivity in these types of cases rather than opening them up.
@ematipico rome check --review inserts a placeholder suppression explanation for now. We can definitely improve the flow to accept user input instead; I'll have to walk the CLI command code a bit more to properly implement this.

I worry about making them mandatory without an option to make them optional. These serious of "microaggressions" that Rome has I fear will really stunt adoption. I think it's a good balance having it mandatory by default, where we can impose our opinions, while still making it configurable for those opinionated who will likely seek out a configuration option when they encounter it as a blocker.

Good point about configuration facilitating greater Rome adoption @sebmck. I also think that rules and configuration options can be tightened as trust in the tooling builds across the community.
Configuration option added, ready for another pass.


Add limited support for captured vars and athrow

Only the primitive type variables captured from a method are supported.
athrow is supported only with SparkException and is converted to raise_error.
Following additional changes have also been made:

A check to reject lambdas with void return type has been added.
An operand stack bug fix for consturctor calls has been added.

This commit also simplifies the code that handles method calls.
Signed-off-by: Sean Lee<EMAIL_ADDRESS>

@seanprime7 ok to take this PR off draft mode?

took this PR off draft mode.

Thanks @seanprime7


Error message seen when removing projects.

acorn version v0.5.1-28-g099c492+099c492f
Remove an existing project.
Project gets removed successfully but we see an error message.
% acorn project rm test
  ✗  ERROR:   "test" not found

@sangee2004 I'm interested in contributing to acorn and would like to work on this if no one is working on it
Thanks

@rmodpur Thank you for your interest in contributing to acorn. Please submit a PR and team will get involved in getting the PR reviewed and moving forward with it. Thanks.

Tested with acorn version v0.6.0-rc1-8-gc6ffc99d+c6ffc99d
No error message presented to user when removing projects:
%acorn project rm testnew1
testnew1

Thanks for the fix @rmodpur!


Link governance more prominently

Our governance document is quite hidden right now. It comes up fine if you google "prometheus governance", but if you start at our prometheus.io landing page, you have to click on Community and then find the link to the governance document on that page. The overview confusingly states that we have joined the CNCF  "to clarify the project's governance structure".
While the latter is easy to update, I think we should have a more prominent link to it, e.g. as an item in the Introduction  section.
Since I'm not sure what the best place is, I have created this issue (rather than a pull request with suggested changes).
Discuss! :o)

Both places are fine to me, but I think we should have one canonical place and clickpath.


https://github.com/HUAWEI11-111/special-happiness/issues/3

https://github.com/HUAWEI11-111/special-happiness/issues/3

Originally posted by @Setr43 in https://github.com/Setr43/solid-rotary-phone/issues/4#issuecomment-1168602865

Originally posted by @Setr43 in https://github.com/Setr43/cautious-train/issues/2#issuecomment-1168603818

Originally posted by @Setr43 in https://github.com/HUAWEI11-111/special-happiness/issues/4#issuecomment-1168604143

Originally posted by @Setr43 in https://github.com/Setr43/redesigned-system/issues/1#issuecomment-1168604370

https://github.com/HUAWEI11-111/special-happiness/issues/3

Originally posted by @Setr43 in https://github.com/Setr43/solid-rotary-phone/issues/4#issuecomment-1168602865

Originally posted by @Setr43 in https://github.com/Setr43/cautious-train/issues/2#issuecomment-1168603818

Originally posted by @Setr43 in https://github.com/HUAWEI11-111/special-happiness/issues/4#issuecomment-1168604143

Originally posted by @Setr43 in https://github.com/Setr43/redesigned-system/issues/1#issuecomment-1168604370


added support for role_arn in cloudwatch events for specifying roles

what

added role_arn to be able to specify a specific role for when dealing with sending events cross-account

why

lacks support for role_arn when the target requires a different role

references
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/cloudwatch_event_rule#role_arn

closes #25

/terratest


feature request: find occurrences

It would be great if merlin could search for an identifier and return a list of locations for all occurrences of this identifier either in the current buffer or the whole project (depending on an option).
This would allow implementing editor services such as mark occurrences (example for Java in this screenshot):

and search:

the :MerlinRename functionality is great but does not yet seem to work for module aliases. that would be a nice addition. e.g.
module A = Array
and then call MerlinRename on A to change it globally to Arr or whatever.


NameError: name 'Union' is not defined

I'm trying to install jsme notebook in jupyter notebook via rdkit using from jsme_notebook.rdkit import JSMERDKit . But it's showing the error as mentioned in title .

This is a bug I introduced from when I added enhancement #1


Azure Image Classification: switch local and azure training disable training step

Reproduce step

start azure training
go to environment page
select local training and switch back to envrionment
train step is disable

Verified.


IndexError: list index out of range

Hi
Thanks for your quick and great job!
I want to test on 3 images but get the following error when calling segment_anything_annotator.py
any insights?
Traceback (most recent call last):   File "segment_anything_annotator.py", line 26, in <module>     editor = Editor(   File "/mnt/data/polymore/salt/salt/editor.py", line 57, in __init__     ) = self.dataset_explorer.get_image_data(self.image_id)   File "/mnt/data/polymore/salt/salt/dataset_explorer.py", line 146, in get_image_data     image_name = self.coco_json["images"][image_id]["file_name"] IndexError: list index out of range
Thanks in advance

Can you paste the contents of your images folder?

@Terrylala
Can you please share the contents of your images dir?
Mine would be a violation of NDA.
Thanks for you support!

I am also facing the same issue!!!

Uploading test_image.zip…

test_image.zip

The dataset should be in the following format. I tried your data and it worked for me.
<dataset_name>/images/*.jpg
<dataset_name>/embeddings/*.npy


Add managedosversion finalizer

This PR is a pre-requisite to #759
A new controller has been added to manage the ManagedOSVersion finalizer.
This will prevent deletion of ManagedOSVersion that are still referenced by any ManagedOSImage.
To my knowledge there is no other direct reference to a ManagedOSVersion. The UI uses a cached list of versions when creating a new SeedImage, but this is presentation layer only, there's no reference to a ManagedOSVersion from a SeedImage.

Also accidentally found how it looks like when the finalizer is preventing the deletion.
So even without UI changes, this is looking good.


Production Schema Is Unknown

Problem
I am about 200 hours into my project and haven't had any issues until just this morning.  After adding new schema, Prisma on the production side doesn't recognize the new schema.

I figured something was missing in the production schema.prisma file, but after comparing it to what I have locally they match.

After trying to move the data onto the user object, this is the error I got from production (heroku)

Things I've tried

Diff checking production and local schema
Rolling back migrations
Creating a different table
Querying the data multiple ways
Delete and reinstall node_modules
Screaming into a pillow

Versions
@prisma/cli 2.4.1
@prisma/client 2.0.0

Checking the obvious things:

Database tables are present?

I would also suggest to update your @prisma/client to the newest version 2.4.1 to match your CLI.

Yep, DB was present.  I ended up deleting the production project and rebuilding it which fixed the problem.  But I still don't know what was causing the issue since /prisma/schema.prisma and /node_modules/.prisma/client/schema.prisma both had those "unknown" fields listed.

I am glad you were able to fix the problem. Since we don't know the cause here it is very hard for me to reproduce this.
If this is fixed for now, I suggest we close the issue.


fixing ci

For now we support clang. Ci fixed here: https://github.com/johannes-graeter/anywho/pull/2


CSX todo list (partial)

As discussed with Dan:

fCCD live (at least counters for total intensity and a couple of ROIs);
Count_time problem with the fCCD camera;
live check on data (written to disk?), reverse lookup from scan_id to file name;
cleanup (deconfig) not performed on abort and hanging on resume;
RL libraries, scans, trajectories, meshes and coverage of the detector (existing to be implemented).

@cmazzoli I'd like to check up on the progress on this list of items.

fCCD live (at least counters for total intensity and a couple of ROIs);

What does "live" mean here?

Count_time problem with the fCCD camera;

This is still a known issue.

live check on data (written to disk?), reverse lookup from scan_id to file name;

We have deployed this at XPD. We'll add it at CSX at our next visit.

cleanup (deconfig) not performed on abort and hanging on resume

I believe this has been fixed in the release you got yesterday. Please speak up if it happens again.

RL libraries, scans, trajectories, meshes and coverage of the detector (existing to be implemented).

Can you elaborate on this? If we discussed it in person, forgive me for not remembering what it means.

@cmazzoli What do you mean for 'RL libraries' ?
Can you elaborate on the count time issue?

Please open separate issues with more details if anything still remains from this list.


fixing broken link

The README had the wrong link for /features/node-client-api

Please issue the pull request against the develop branch

@ehennum I mistyped a command while trying to change my PR and accidentally pushed straight to the develop branch. Doh. Well, mission accomplished.

No worries, thanks for the fix


Update to new macOS spelling

I think I noticed a few more:
Some filters to ease searching
find . -name "*.md" | xargs grep -i mac | grep -v machine | grep -v macOS
find . -name "*.rb" | xargs grep -i mac | grep -v :mac | grep -v "mac?" | grep -v machine | grep -v -i macos | grep -v mac: | grep -v platform_slug

Let's merge these first, and do more replacements in a separate PR 👍


ts: missing instruction data leads to wrong error

in the ts client, when you forget to add instruction data e.g.
await program.rpc.updateFoo({
      accounts: {
        foo: foo.publicKey,
        authority: program.provider.wallet.publicKey,
      },
    })

instead of
await program.rpc.updateFoo(new BN(1234), {
      accounts: {
        foo: foo.publicKey,
        authority: program.provider.wallet.publicKey,
      },
    })

you get an error saying that the first account is not provided (this is from the zero-copy tests)
Error: Invalid arguments: foo not provided.

found the cause:
the cause is that splitArgsAndCtx expects well-formed input. If the input does not match the idl, it returns incorrect output. In the case of the error in the issue description, it returns the accounts inside the ixArgs return value and leaves the ctx return value empty. the validateAccounts function iterates over the accounts and checks whether the account is inside ctx. None of them are and foo is the first one so we get the error above.

fixing this is not straighforward. because instruction arguments can be anything, it is difficult if not impossible to know when intruction arguments end and the context begins.
(the following variable names are made-up)
functionArguments are the args the user passes into the ix function.
You easily know that sth is wrong if the functionArguments.length < idlIxArgs.length
but if functionArguments.length === idlIxArgs.length like in the example from the issue description and there's an ixArg missing, the split function will simply view the context arg as an ixArg.
So you could add a check that the 2nd value in the tuple returned from the split function must be at least 1. But that doesnt work because there are functions like the sample initialize you get when you init an anchor project which do not take expect accounts.
So you could check in the idl whether the ix expects accounts and only do the check inside split if idlIx.accounts.length != 0 but that also doesnt work because the check should be there also in cases where there are no accounts but there are e.g. preInstructions
So what does work is to check whether the last value of the first value in the tuple from the split function is not a ctx argument so youd have to check that none of the context fields like preInstructions etc. exist on it. That does not look very clean to me.
So my suggestion is, instead of creating the ix functions with variable amounts of arguments, only let them take 2 arguments: function (ixArgs: any[], ctx: Context) or even just 1 if ixArgs were a part of ctx. I dont see a reason why accounts should be in the same object as sth like postInstructions but not in the same object with ixArgs. In fact, I'd argue it'd be better to only have it be with ixArgs in one object function (ixData: {accounts:[], ixArgs:[]}, ctx: Context)
If you really wanted, you could wrap all this in an object that can then be made optional so there can continue to be functions like initialize with no input but dont think we should design for functions which pratically do not occur.
Whether we do 1 or 2 arguments or not, putting ixArgs into an array would allow us to remove the split function altogether and with it this bug


Fix fp16 issue for image classification

What does this PR do?

Image classification: casts inputs to fp16 in the dataset when training in fp16

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Leftover .ldb file

While validating the design for a feature on top of leveldb, we've been stress-testing a database using a workload consisting of Put() and Delete(). We consistently see a .ldb file being left behind, so it seems like compaction doesn't delete all the old files.
This may be related to #573, which also suggests a potential accounting problem in compaction.

Yep.

The first level file should be full of data that is soon deleted, but it sticks around forever.

Hi Guys,
Any update on this issue, I am facing the same issue.
In my case I have a stream of incoming data which is converted into a struct, saved while processing then removed.
Same as @dmurph.  But after a while, my amount of ldb files just grows and grows. All not used.
Because, I wrote a small binary that does only CompactRange from Start to Finish and all get removed.

@dmurph Can you please dust off your experiment's code and answer @cmumford's question above?

@pedrinimm That might be a different issue - can you give a range of 'growth' that you see? Or is it just the 00005.ldb file that sticks around?
@cmumford I will investigate, I'm not sure.

@cmumford Yes - I just re-ran my test with a persistent directory instead of a new one, and it keeps the 000005.ldb file around even after the second run of the test.

Updated the test to use a persistent folder: https://chromium-review.googlesource.com/c/chromium/src/+/1080339

In my team, we are seeing the same issue in one of our large enterprise customer projects. Old .ldb files are being left without getting deleted. It eventually leads to filling up the disk after two days. Is there a fix or workaround or any advice from the Level DB team to avoid this critical problem?
Thank you.

@nysenthil check that your code doesn't have any unreleased snapshots or iterators. Also check the log (*.log) that your Env may be creating for evidence of problems. This bug is only for a single *.ldb file being left behind - I'm unaware of a bug that fails to delete many *.ldb files.

Hi Chris,
The problem I described above is much more related to this open issue by not purging the old .ldb files: Disk space used by the DB grows without bounds #603


Fix matchMedia queries with multiple dimensions

Regex global flag was missing, so only the first dimension in a media query was being converted.

I can add tests if needed, but the current matchMedia tests are very barebones.

Adding a test for this specific case would be appreciated - it will help us gradually improve test coverage and this was obviously a gotcha during development.

Added a basic test, just checks that the new Function doesn't throw a SyntaxError.

Out of interest, does this get merged then released? Or do we need to work around it locally for a bit.

It doesn't hit the CDN until we release 1.3, but as soon as it's merged you could check out and use master if you wanted the fix right away.

Maybe we could continuously deploy to latest.polyfill.io or something
similar?
On 11 February 2015 at 17:46, Andrew Betts<EMAIL_ADDRESS>wrote:

It doesn't hit the CDN until we release 1.3, but as soon as it's merged
you could check out and use master if you wanted the fix right away.
—
Reply to this email directly or view it on GitHub
https://github.com/Financial-Times/polyfill-service/pull/289#issuecomment-73927521
.

--
Sam Giles
Developer
FT Labs
For Origami questions<EMAIL_ADDRESS>Number One Southwark Bridge, London SE1 9HL
--

This email was sent by a company owned by Pearson plc, registered office at
80 Strand, London WC2R 0RL.  Registered in England and Wales with company
number 53723.

i think in this case we have a workaround

CI won't work here because you're using a fork.  FT employees do not need to use forks, so it's easier for me if you use a branch in the FT account.  I'll take your word for it on the test!


Deduping mechanism in pinot

Is there a way to dedup similar rows? Will help with dealing with duplicate rows during kafka consumption and if partial data is ingested through batch mechanism (dedup the overlapping segment with the data consumed through realtime)

@npawar
This can be a good transformer to add to filter duplicate events.

There is a table config setting to aggregate metrics. If all your dimension values are same, then the metric values are added, and the rows are squished. Is this what you mean by "similar?"
There are some limitations, but give it a try

https://docs.pinot.apache.org/basics/components/table#tableindexconfig

I could use aggregate and unique, but I am thinking of following case which I believe wouldn't return accurate results, correct me if I am wrong or if there is an alternate solution.
With "aggregated" (meaning all column values are same including the timestamp and assume timestamp is in microseconds) if there are two rows of same value, count would become 2. Usually for querying purposes we would be interested in a slightly higher time-granularity (across some set of columns, for example 5m) then the overall counts we get would be doubled rite?
If we are looking at individual rows, we can get away with aggregate/unique sort of approach.

nm, I could actually just do count(*) instead of sum(count column) 🤦

What is the suggested solution to this problem at the moment?

Dedup feature is added in #8708


Docs for FileStream constructors are confusing about accessing devices

The API reference for various constructors of FileStream contain these two paragraphs:

The .NET Framework does not support direct access to physical disks through paths that are device names, such as "\.\PHYSICALDRIVE0 ".

Note
path is not required to be a file stored on disk; it can be any part of a system that supports access through streams. For example, depending on the system, this class can access a physical device.

On first reading, I thought the two paragraphs are contradicting each other: the first says I can't access physical disks, the second says I can.
On second reading, I realized the paragraphs don't contradict each other: I can access physical devices, just not physical disks.
Is this interpretation correct?
In any case, I think this is confusing. Considering that I think device access is not a common requirement, should the two paragraphs just be deleted? If device access is actually possible this way, maybe it should be explained in a separate article?

@JeremyKuhne, @ianhays can you guys help with this one?

This one is a bit complicated. I discuss the various formats in one of my blog posts. We do allow device path syntax in .NET 4.6.2 and above as well as .NET Core.
I'm fine with deleting the snippets mentioned. It might be good to add an article about .NET path formats on Windows based on the above posts. I'm happy to help review such a change.


Clarify some formulations in the README.md file.

I find the documentation highly incomplete and only saying half of the truth. Here is some attempt to improve it a bit.

I like your suggestions, except the phrase "Nextera Mate Pair linker" is not really used in official documentation (see below). I prefer to stick with "Nextera Mate Pair adapter" or "Nextera Mate Pair junction adapter".
https://www.illumina.com/documents/products/technotes/technote_nextera_matepair_data_processing.pdf

Actually it does not remove P1/P2 adapters either.

"Nextera Mate Pair linker" is not really used in official documentation

I find the official documentation unclear. It should distinguish between adapter and linker, and especially trimming and splitting programs should take care about proper phrasing as the actions are very different. NxTrim as you explained me does only the splitting part, and assumes trimming was done prior to using NxTrim (it would be more complicated&slower to do afterwards due to the RF reads being reverse-complemented). Do you really object that much? ;-)

Actually it does not remove P1/P2 adapters either.

So what is the TruSeq_Adapter aka IndexSP on https://github.com/sequencing/NxTrim/blob/master/matepair.cpp#L10 needed for?
And what is the TruSeq3/2 sequence at https://github.com/sequencing/NxTrim/blob/master/matepair.cpp#L11 needed for?
I assumed in both cases it is used to detect the left most end of Illumina adapter (to trim away rightwards from the start).
In both cases you are missing an A nucleotide which appears in the data because of the sticky end being introduced using Klenow fragment. Proper queries should be:
>TruSeq3/1 aka TruSeq_Adapter aka IndexSP
aGATCGGAAGAGCACACGTCTGAACTCCAGTCAC
>TruSeq3/2
aGATCGGAAGAGCGTCGTGTAGGGAAAGAGTGTA

So what is the TruSeq_Adapter aka IndexSP on >https://github.com/sequencing/NxTrim/blob/master/matepair.cpp#L10 needed for?
And what is the TruSeq3/2 sequence at >https://github.com/sequencing/NxTrim/blob/master/matepair.cpp#L11 needed for?

I assumed in both cases it is used to detect the left most end of Illumina adapter (to trim away rightwards >from the start).

This code is actually to catch the case where the DNA fragment is less than 2x the readlength. This means that R1 can read into the external adapter on the other side of the fragment.
eg.
R1------------------------>
    AAAAAAAAAAAABBBB

where BBBB is the external adapter. NxTrim will (hopefully) see this and remove BBBB accordingly.
This is an edge case that hopefully does not happen often but of course depends on sample preparation. In my data it is <0.5% of read pairs but your mileage may vary.
I agree that users should be able to specify their own adapters as arguments and will add some code for this.

Thanks for your input, I have incorporated many of your suggestions in the current README.


Feat: Add google provider paths to the keystone public pages config

This is needed for the Google provider to launch from and return to.
Also noticed a spelling mistake, not sure if that path was being used?

I created this PR because it's specifically what I needed added, but this will obviously be an issue for other providers. So I guess the question is, do you want to add all the providers here or just a select few and add instructions into the README on how to add your own publicPages config if you're using another one? Or something else?
I'm happy to amend the PR with whatever direction is preferred, or leave it to you if you have other plans. 🙂

Thanks PR @jordie23, good point re public pages. I'll have a think about how we might do this better (open to ideas if you have any). Happy to Merge this change while we think through how how to solve the issue of further providers public pages.


fix: runtime init panic

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

fix: https://github.com/kcl-lang/cli/issues/114

fix: runtime init panic

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

18 of 18 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 32.867%

Totals

Change from base Build<PHONE_NUMBER>4:
0.07%

Covered Lines:
5703

Relevant Lines:
17352

💛  - Coveralls


processmanager: tmux

fire/forget long running processes like servers and such in tmux

https://github.com/js-next/js-ng/commit/c802f4bdba247ce30b0125142462b2186834f948


Update dependencies

Hi, I would like to know when the following dependencies will be updated:

Hi @CharlesMoreira1,
I've updated a bunch of dependencies with #25.
These will be included in the next release. I'll close this issue. Please re-open if this re-surfaces with the next version.


TestDashboards and TestDashboards/Mixer failed in release tests

https://k8s-gubernator.appspot.com/build/istio-prow/pull/istio-releases_daily-release/510/daily-e2e-rbac-auth-default/461/
https://k8s-gubernator.appspot.com/build/istio-prow/pull/istio-releases_daily-release/510/daily-e2e-rbac-auth/453/
Both looked green before but started failing today, and they blocked the daily release.
The recent postsubmit run was green however.
https://k8s-gubernator.appspot.com/build/istio-prow/e2e-suite-rbac-auth/1386/
So they may very well be flaky.

same as #4842?

Indeed. Thanks @linsun
Moved the info over and closing this.


OrderCompleteEvent bugfix

We were missing OrderCompleteEvents that came from TIF orders being booted. This PR adds them 😊 I also added a warning when using subscribe_transactions() with non-Triton RPCs, and lowered the default log level to WARNING

Example placeOrder with 5 sec TIF:


Improve episode's summary

This PR aims to make episode's summary section more readable converting \n into <br> and rendering URLs into links.
Before:

After:

@hugoabonizio great! Thanks a lot.
I will merge and deploy it.
For the future, any commit that changes the behavior of the code is either a feature or a fix. In this case since it is improving and not solving any issue, both would be features.
Once again, thanks!

@marceloboeira I'm sorry, you're right! :smile:

no problem, I am just really crazy about commit messages...


Skillswipe

Skillswipe doesn't check if the player is out of xp before giving xp I think? Beast-xpwithdraw makes the player always have 1 xp left, so maybe that's causing the enchant to continue triggering?

Heard this is fixed @kicjow so I'll close it first XD