"Sourcemap points to missing source files" for all svg imports

Describe the bug
When using rollup-plugin-svelte-svg (https://github.com/codefeathers/rollup-plugin-svelte-svg), any imported svgs throw the warning:
Sourcemap for "/.../my-icon.svg.rollup-plugin.svelte" points to missing source files

The images load properly; the only problem is the erroneous warning message.
Reproduction
https://github.com/yowzadave/svg-import-demo
Running "vite" in this repository will reproduce the error.
Logs
No response
System Info
System:
    OS: macOS 12.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 86.17 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 17.3.1 - ~/.nvm/versions/node/v17.3.1/bin/node
    Yarn: 1.22.15 - ~/.nvm/versions/node/v17.3.1/bin/yarn
    npm: 8.3.0 - ~/.nvm/versions/node/v17.3.1/bin/npm
  Browsers:
    Chrome: 99.0.4844.83
    Firefox: 98.0.2
    Safari: 15.2
  npmPackages:
    @sveltejs/vite-plugin-svelte: ^1.0.0-next.30 => 1.0.0-next.40
    svelte: ^3.44.0 => 3.46.4
    vite: ^2.8.0 => 2.8.6

Severity
annoyance

The console warning comes directly from vite and is caused by rollup-plugin-svelte-svg's custom id format.
I recommend you check out unplugin-icons, or unocss with preset-icons as alternative solution or report this directly on rollup-plugin-svelte-svg.

Also a note: You can pass extensions: ['.svelte', '.svg'] to the plugin options. Another rollup plugin isn't really needed, unless you want to run svgo, and that can be done with https://github.com/svitejs/svelte-preprocess-svg too.

Thanks @dominikg and @bluwy--I'll try your suggestions!


Store and Retrieve Connected Timestamp

After #7275 is resolved. We should follow up and fix how we retrieve the "connected" timestamp. We should store the "connected" timestamp as soon as we are "connected". We would then retrieve this value in the getCurrentTimestamp function in the ContainerRuntime.
#8580 (comment)

The getCurrentTimestamp method to pass to GC only uses the timestamp from ops and not from the "connected" timestamp. So, this is not required anymore.


jest-runtime makes each module to have its own modules cache

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Printing the content of require.cache in several files, they are shown as being isolated ones from others instead of being shared as Node.js require() function does. Jest-runtime has its own require function, and in fact this behaviour seems to be on purposse to prevent modules do side-effects, but in fact that side-effects is exactly what does the react-native-mock-render module, so it doesn't work at all.
What is the expected behavior?
require.cache being shared by all the modules as does Node.js implementation, so replacing entries on it affect the full Node.js instance.
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.
  "jest": {
    "collectCoverageFrom": [
      "*.js",
      "config/**/*.js",
      "scripts/*.js",
      "src/**/*.js"
    ],
    "preset": "jest-react-native",
    "setupTestFrameworkScriptFile": "./src/setupTests.js",
    "snapshotSerializers": [
      "enzyme-to-json/serializer"
    ],
    "transformIgnorePatterns": [
      "node_modules/?!(react-router-native)"
    ]
  },

Jest: 21.2.1
Node.js: 9.2.0
npm: 5.5.1

Jest does not support require.cache. If you'd like to set a module, you can add it to setupFiles in a way that does jest.mock('module-name', () => { return moduleContents }, {virtual: true}).

Jest does not support require.cache. If you'd like to set a module, you can add it to setupFiles in a way that does jest.mock('module-name', () => { return moduleContents }, {virtual: true}).

Thanks for the advice, I'll give it a try.

Confirmed, it works. Thank you! :-D


fix(dropdown): fix focus and shrink

General summary

Fixed

Fixed and improved Dropdown shrinking system.
Fixed unwanted scroll to the top on the screen when opening a Dropdown.

Changed

Improved useInfiniteScroll to prevent conditionnal hooks.

Added

Added focusElement prop to the Popover component to be able to focus on open.

Fixed https://lumapps.atlassian.net/browse/LUM-11305
Fixed https://lumapps.atlassian.net/browse/LUM-11322
Screenshots

Check list

[ ] (if has style) Add need: review-integration label
[ ] (if has textual documentation) Add need: review-doc label
[x] (if has code) Add need: review-frontend label
[x] (if has react) Check through the react dev check list
[x] (if fix or feature) Add need: test label

Check out the contribution guide for general guidelines for submissions to the design system.

Test OK


Unknown OpenGL attribute - Windows 10

Saw this error on a surface pro 4.
SDL Error (-1): Unknown OpenGL attribute
   [Condition of type SDL2::SDL-RC-ERROR]

Restarts:
 0: [RETRY] Retry SLIME REPL evaluation request.
 1: [*ABORT] Return to SLIME's top level.
 2: [ABORT] abort thread (#<THREAD "repl-thread" RUNNING {1003828033}>)

Backtrace:
  0: (SDL2:GL-SET-ATTR :CONTEXT-PROFILE-MASK 1)
  1: ((:METHOD CEPL.HOST:REQUEST-CONTEXT (T T T T T T T T T T T T T T T)) 320 240 "CEPL REPL" NIL NIL 0 16 8 8 8 8 32 T NIL T) [fast-method]
  2: (CEPL.CONTEXT:MAKE-CONTEXT :WIDTH 320 :HEIGHT 240 :TITLE "CEPL REPL" :FULLSCREEN NIL :NO-FRAME NIL :ALPHA-SIZE 0 :DEPTH-SIZE 16 :STENCIL-SIZE 8 :RED-SIZE 8 :GREEN-SIZE 8 :BLUE-SIZE 8 :BUFFER-SIZE 32 :..
  3: (CEPL::INIT 320 240 "CEPL REPL" T)
  4: (CEPL:REPL 320 240)
  5: (SB-INT:SIMPLE-EVAL-IN-LEXENV (CEPL:REPL) #<NULL-LEXENV>)
  6: (EVAL (CEPL:REPL))

Which makes a pile of sense as I have just been assuming GL features to be available rather than querying for them.

I have temporarily decided to just eat the errors until I fix this properly.
See this commit for the hack: dc2e1bcedb1647c7aa6663eb56d343b9582e51c1

damn, I should have put this on cepl.sdl2 rep, ah well

I now require core contexts by default in cepl.sdl2 & cepl.glop. We do however search for an acceptable version in a more sensible way so we should now get whatever the highest supported core gl context is for a given machine


Video incorrect - Bot Channels Registration doesn't exist

Type of issue
Outdated article
Feedback
Hi, Video incorrect - Bot Channels Registration doesn't exist.
The video is under this text: See the following video to learn how to define message extension action commands:
At timestamp 1:06 (1min 6 sec), the speaker goes to Marketplace and searches for and clicks a new resource type "Bot Channels Registration". I can't find this. I can only see "Azure Bot". Should I select this one? Is there another tutorial I can follow that is up to date?
Thank you.
Page URL
https://learn.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/action-commands/define-action-command?tabs=developer-portal-for-teams%2Cdotnet
Content source URL
https://github.com/MicrosoftDocs/msteams-docs/blob/main/msteams-platform/messaging-extensions/how-to/action-commands/define-action-command.md
Author
@surbhigupta
Document Id
415b5d57-2d41-9540-55a7-2a33a4315e37

Hello @teams-exporter - Thanks for raising your query.
We will look into it and let you know if any further inputs required.

Hi @teams-exporter - Yes, the video seems to be outdated as now you have to select 'Azure Bot'. The remaining operations are mostly same. You can also refer this document to create Azure Bot resource.
https://learn.microsoft.com/en-us/azure/bot-service/abs-quickstart?view=azure-bot-service-4.0&tabs=userassigned#create-the-resource

We will have to remove the link to the video.


Remove boilerplate/duplicate code

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like

This could have been one struct
Describe alternatives you've considered
None
Additional context
None

Thanks for submitting a issue!
to assign this issue to yourself please comment @aurora-care-bear please assign me


UI query does not work if = is present in the value of a binary annotation

The current split logic blindly splits whole of the query by = and does not care if that might be present in the value. Hence, this binnary annotation query fails : http.uri=/foo?bar=3. i.e. Having another = in the value (/foo?bar=2).

Aah, we need to encodeURIComponent() it in the JavaScript layer, I guess,

I don't think that is the problem. When we are splitting, it should split only by the first = and the rest of them should stay as in, in the value. The limitation of this is = should not be in the key, but that is not a deal breaker.


The icon looks like Refresh.

Refer to trello card
https://trello.com/c/zcqk4xT6/39-the-icon-looks-like-refresh

Refer to the the icon on screen - Backup – Level 1 expanded Security Question
https://xd.adobe.com/view/306fd70c-6cc6-4d76-a5c4-46490c6831ef-653c/

@antuz123  check once

-Verified this issue on dev app v1.5.0(257)


Anno put dev

complete the features and corresponding unit tests.
Style check passed.
Doc added.

Annotation operation APIs are completed and tested. API documentation is added.


featureSupport partition stats cache

Support cache partition level stats and show partition level cached stats.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Explore: query settings broken

https://github.com/mlcraft-io/client-v2/assets/2017148/b7df6913-a03f-4e25-ad1d-5a91ad4f8db7
query settings are broken. Sometimes it's not possible to click on the checkbox. Checkbox title should be clickable too.

done


Enable smooth scrolling for navigation links.

Description: When a user clicks on a navigation link, the page should scroll smoothly to the corresponding section.
Labels: enhancement, navigation

Implemented smooth scrolling for all navigation links. Now, when users click on any link, the page scrolls seamlessly to the desired section, enhancing the user experience. Tested for functionality across different browsers


Strange shaking phenomenon while applying polar to cartesian for radar object

Hi, thanks for releasing this awesome Boreas Dataset.
I have recently attempted visualization using the code provided below. The attached video demonstrates the output result. I am puzzled by the persistent shaking of the radar image. It seems that I may have overlooked some crucial details, but I am uncertain about what they could be. I would greatly appreciate your assistance in addressing this issue. Thank you very much.
import matplotlib.pyplot as plt
import matplotlib.transforms as mtf
import numpy as np
import cv2
from pyboreas import BoreasDataset
from pyboreas.data.splits import obj_train, obj_test, obj_sample
from pyboreas.utils.utils import get_inverse_tf

bd = BoreasDataset('/data/Boreas/', split=obj_train)

seq = bd.sequences[10]  ## ** Try different sequences!

seq.filter_frames_gt()
#seq.synchronize_frames('lidar')
#print(seq.radar_frames)
bounds = [-75, 75, -75, 75, -5, 10]
index = 4  ## ** Try different indices in the sequence!

for index in range(len(seq.radar_frames)):
    rad = seq.get_radar(index)
    radar_cart = rad.polar_to_cart(0.25, 1000)  # from polar to cartesian
    radar_cart = cv2.addWeighted(radar_cart, 1, radar_cart, 1, 0)  # use to enhance the brightness
    cv2.imshow("radar", radar_cart)
    cv2.waitKey(10)

https://github.com/utiasASRL/pyboreas/assets/88025855/241bf756-4f6e-4a98-a496-bb047a8fede2

Hmm, I think the issue is that the polar_to_cart function is trying to work with the azimuth values stored in the polar image files. However, I remember that these were slightly corrupted during the boreas-objects-v1 sequence due to the images being compressed using a lossy compression. I have since replaced the lossy compression with a lossless image format.
For all the sequences other than boreas-objects-v1, this shouldn't be a problem.
Try using this function directly and setting fix_wobble to False:
https://github.com/utiasASRL/pyboreas/blob/0a687fa66387e24aea6335fd04fb242f7aece774/pyboreas/utils/radar.py#L59C5-L59C29
For a better fix, you could try and fit a linear slope to the azimuth value vs. row index and then use the corrected azimuth values in the polar_to_cart function.
I have some code for correcting timestamps that you may find useful:
import numpy as np
from sklearn.linear_model import RANSACRegressor as RANSAC

# d can be 8, 16, 32, 64, ...
def convert_to_byte_array(t, d=64):
    byte_array = np.zeros(d, dtype=np.uint8)
    for i in range(d-1, -1, -1):
        k = t >> i
        if (k & 1):
            byte_array[i] = 1
        else:
            byte_array[i] = 0
    out_array = np.zeros(d // 8, dtype=np.uint8)
    for i in range(d // 8):
        for j in range(8):
            out_array[i] += byte_array[i * 8 + j] * 2**j
    return out_array

def get_time_fit(times):
    ransac = RANSAC(min_samples=2, residual_threshold=2000, stop_n_inliers=int(0.98*times.shape[0]))
    X = np.asarray(range(times.shape[0])).reshape(-1, 1)
    reg = ransac.fit(X, times)
    p = np.polyfit(X[reg.inlier_mask_].squeeze(), times[reg.inlier_mask_].squeeze(), deg=1)
    p = [p[1], p[0]]
    return p

def fix_timestamps(raw_data, residual_threshold=2000):
    times = raw_data[:, :8].copy().view(np.int64)
    params = get_time_fit(times)
    tcomp = np.asarray(range(times.shape[0])).reshape(-1, 1) * params[1] + params[0]
    outlier = np.abs(times - tcomp) > residual_threshold
    data2 = raw_data.copy()
    for i in range(times.shape[0]):
        if outlier[i, 0]:
            data2[i, :8] = convert_to_byte_array(np.int64(round(i * params[1] + params[0])))
    return data2

So you could modify this code to create a function for fix_azimuths for example.
I'll look into fixing the data and uploading that to S3 when I get a chance.

Thank you for your reply. I will give it a try, and if I come across any new findings, I will report them here. Many thanks!


Test of new corrective flexbox class

So, I've mentioned elsewhere that there is a deficiency in the flexbox version of Foundation. This fix could come as a new class or could simply baked in (the latter is probably the way to go).
A so called "beam" class is meant to ensure that a flexbox based layout fills the viewport even when content is sparse.
Why?
• Makes it easy for sticky footer to stay at bottom no matter how sparse a page's content is.
• Heights of flexbox children are not affected by image load delay, which creates the nasty side effect of sudden height increases.
Why not set the height to 100%?

Let's say that you have just a bit of content in the main element. Let's also say that you'd like your footer to stay at the bottom. The temptation is to set the height of main to 100%; however, this causes your layout to take up more than 100% of the viewport.
So, why not use auto height for main? Not only will the footer not be at the bottom of the viewport, elements will "snap" in place as images load in.
In order to demonstrate this, I am including a new test file at (http://localhost:3000/flex-grid/column-row-beam.html).
EDIT: grammar, clarifcation

I think this is a pretty interesting idea.  @andycochran @brettsmason @rafibomb @zurbrandon would be interested in your thoughts.  (check out the branch & try the visual test)
Right now this pr is simply an example, but we could add this as a helper class... I think its pretty cool.  @zurbrandon is this something we'll get as a part of the apps grid work you're doing?

OK, awesome, had a conversation with @zurbrandon out of band and it sounds like we're going to be getting this functionality (but in a more flexible way, with a bunch of additional goodies) as a part of porting the apps grid we created for Foundation for Apps into Foundation for Sites.  We should have an initial PR for this sometime next week, targeting integration in 6.4.

With the above in mind going to close this PR.

@kball So glad to hear that ideas like this are in the works! Thanks for the update. I guess I'll hang back and wait for the PR!


Make sorting keys configurable

For example, to sort entries by line number, use:
require("trouble").setup({
  sort_keys = {
    "filename",
    "lnum",
    "col",
    "severity",
  }
})

The defaults remain the same.
Fixes: https://github.com/folke/trouble.nvim/issues/65

Any feedback on this?

Thanks!

Thank you!


DNAseq pipeline input file

I am using an aligned bam file from the non-GATK pipeline platform as the input file. In that case, both of the ref (original file is aligned with GRCh37) and the file format (not from BWA) are different from what I expected (using GRCh38 as ref and using BWA alignment). I found that the GATK pipeline could convert aligned bam file to unmapped one and redo alignment from their official website. Can this Azure-based GATK pipeline also do similar things (generate unmapped bam and align it again)?

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

ID: 796846ea-4dff-5c31-15f7-63fb61faccd0
Version Independent ID: 79c141f4-790b-4d18-6a04-7270e9f3438c
Content: DNASeq pipeline - Azure Databricks - Workspace
Content Source: databricks/applications/genomics/secondary/dnaseq-pipeline.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@YidiQin
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

@YidiQin
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

Thank you for your question @YidiQin .
@mssaperla I have been unable to locate information useful for answering this question.  Could you please route this to the appropriate expert?

Thank you for your question @YidiQin .
@mssaperla I have been unable to locate information useful for answering this question.  Could you please route this to the appropriate expert?

Thanks for your dedication to our documentation. Unfortunately, we have been unable to address your issue and apologize for the delayed response. We are closing this issue, but if you feel that it's still a concern, please let us know directly at<EMAIL_ADDRESS>#please-close

#please-close


Update jacoco

Update jacoco.
jacoco officially supports Java11 from 0.8.3
You can see changelogs below.

https://github.com/jacoco/jacoco/releases/tag/v0.8.3
https://github.com/jacoco/jacoco/releases/tag/v0.8.2

Hi @wreulicke, thank you for suggesting. Digdag now using Java8 but, let us try to upgrade. If it doesn't work, it will be reverted.

Thanks for your review.


Make loggers more consistent

Description
Changes proposed in this pull request:

set the same output format for both loggers

Related issue(s)

https://github.com/kyma-project/serverless-manager/issues/219

/retest

/retest

/test pre-serverless-operator-build


Updated logging.py for #9146

Updated logging.py for #9146
Vijay-Arora

Merging in Vijay-Arora/pytest


scenario-testing failing - Manually install make

scenario-testing action routinely fails because of make not being installed.

This is due to a breaking change in the 2.54.0 tag: https://github.com/Azure/azure-cli/pull/27567.  To fix this, we need to manually install make.

#154 fixes


How to run se3lam?

Hi dear experts,
Honor to write a letter to you, I want to know what is wrong with the running part?
Run rviz:
roscd se2lam   --- roscd: No such package/stack 'se2lam'
rosrun rviz rviz -d rviz.rviz  ---- picture on the left
Run se2lam:   ----  picture on the right
rosrun se2lam test_vn PATH_TO_DatasetRoom PATH_TO_ORBvoc.bin

thank you
best regards
William

Hi, you should learn ROS first. Finish the begin level tutorials, then you would know what build this project as a ROS package in the README means, and why your commands fail to run.


Feature request: How Do I add a new variable to a data set?

A common thing to want to do is to create a new DataArray and add it to an existing Dataset, but the answer does not seem to appear in the "How Do I?" list.
Thanks!

ds["name"] = data_array Also see Dataset.assign
A pull request would be very welcome!

OK, done. Per my comments, I don't have the ability to build and check the sphinx code, so please check before merging.

@dcherian
I know it's kind of overloading this issue, but if you would also LMK if there's a clean way to do compound querying, the way one can in Pandas, I will also put in something about that.
foo = data_array.where(np.logical_and(other_data_array['bar'] == 12, other_data_array['baz'] < 22))

...is all I have come up with and this gets bloated and unreadable really quickly if there are multiple conjuncts.  Infix conjunction seems not to work unless I take the .values of the data array subsets, which adds its own layer of bloat.
I figure this is something that lots of people will want to do, but it seems a lot harder than pd.DataFrame.query()

You can shorthen this using parantheses and &:
sel = (other_data_array['bar'] == 12) & (other_data_array['baz'] < 22)
foo = data_array.where(sel)

@mathause Thank you! That never occurred to me.
I thought it was something wrong with the way I was using conjunction, but now I believe it was just a matter of operator precedence.


pkg/trace/agent: improve normalizer

This change improves the normalization speed for a span by 18%
 name             old time/op    new time/op    delta
 Normalization-8     850ns ± 0%     693ns ± 0%  -18.47%

 name             old alloc/op   new alloc/op   delta
 Normalization-8    1.00kB ± 0%    0.95kB ± 0%   -4.80%

 name             old allocs/op  new allocs/op  delta
 Normalization-8      12.0 ± 0%      10.0 ± 0%  -16.67%

Followed @LotharSee's suggestion so everyone PTAL and let me know if happy :)


Refactor supportsFeature out of the icons directory

As discussed in https://github.com/dermotduffy/frigate-hass-card/pull/565 .

BTW, the original source of the function is from here.


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Codecov Report

Merging #2 (d8e3f4a) into master (9c4709c) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##             master      #2   +/-   ##
========================================
  Coverage      0.00%   0.00%
  Complexity      106     106
========================================
  Files             8       8
  Lines           368     368
========================================
  Misses          368     368

Impacted Files
Coverage Δ
Complexity Δ

src/CesantiaSupport.php
0.00% <0.00%> (ø)
7.00 <0.00> (ø)

src/PreviScraper.php
0.00% <0.00%> (ø)
5.00 <0.00> (ø)

src/Sources/AbstractSource.php
0.00% <ø> (ø)
8.00 <0.00> (ø)

src/Sources/Previred.php
0.00% <ø> (ø)
53.00 <0.00> (ø)

Continue to review full report at Codecov.

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


force upgrade markup (sysupgrade/luci-21.02)

The option "Force upgrade" is outside the container (Bootstrap theme). Tested Firefox/Chrome browsers with reset caches.

LuCI openwrt-21.02 branch (git-21.088.35541-d627abe) / OpenWrt 21.02-SNAPSHOT r15952+1-6090337

@dibdot the reason is this commit

I doubt that, rather I wonder why this label was defined as a button!? E.g.
[...]
body.push(E('p', {}, E('label', { 'class': 'btn alert-message danger' }, [
[...]

remove the "btn" part should fix this IMHO.

remove the "btn" part should fix this IMHO.
Don't know, maybe this will break other themes.

I made this new rule for bootstrap-theme, now the alert looks ok:
.modal .btn.danger { 	white-space: normal; }

@jow- now other buttons are affected by the problem:

With new css rule on cascade.css it looks like:
.modal .btn.danger, .modal .btn { 	white-space: normal; }

Probably worth reopening the issue.

Patch missed on the 21.02 branch.
https://github.com/openwrt/luci/blob/b39d9bf4bb88acd3120098c3a087e47331d1d757/modules/luci-mod-system/htdocs/luci-static/resources/view/system/flash.js#L280


[zh-cn] Part4:updated the term "群集" to "集群"

Signed-off-by: jacky<EMAIL_ADDRESS>
issue:https://github.com/kubernetes/website/issues/33735
fix issue part4 file typo

The files are not fully synced with upstream (English) version.
Feel free to /unhold once it's fully checked and synced.
/hold

@Sea-n

The files are not fully synced with upstream (English) version.
I know what u mean,so what should I do now,can u give some advice?

@Sea-n

The files are not fully synced with upstream (English) version.

I know what u mean,so what should I do now,can u give some advice?
@dapengJacky

Please refer to this link for command line instructions for synchronization in both Chinese and English.
https://kubernetes.io/zh-cn/docs/contribute/localization_zh/#更新译文

@Sea-n

The files are not fully synced with upstream (English) version.

I know what u mean,so what should I do now,can u give some advice?

@dapengJacky
Please refer to this link for command line instructions for synchronization in both Chinese and English.
https://kubernetes.io/zh-cn/docs/contribute/localization_zh/#更新译文

I have synced with upstream (English) version,please review again,thanks

please squash your PR into a single commit.

It would be great to revise some sentence to be more fluent.

thanks,I have fixed this

Ah, forgot to unhold since the content is synced.
/unhold
And overall looks good to me, wait for others to provide more suggestions.

/approve

/cc @ydFu
Could u help to review,thanks

@ydFu  thanks,I have fixed


[XamlUIBasics] CalendarView demo crashes with certain CalendarIdentifiers

Steps to reproduce

Check out the dev branch (2867cc13c46abcc3ba63267ba1a7db5beeca4db8)
Build and launch the AppUIBasics sample (Samples\XamlUIBasics\cs\AppUIBasics)
Navigate to the CalendarView demo
Select one of the following CalendarIdentifiers to crash the app:
ChineseLunarCalendar
JapaneseLunarCalendar
KoreanLunarCalendar
TaiwanLunarCalendar
VietnameseLunarCalendar

I've asked the feature team to investigate.


Devfile Ecosystem update

Signed-off-by: Kim Tsao<EMAIL_ADDRESS>What does this PR do / why we need it:
Introduces a new Devfile Ecosystem section to explain the big picture
Which issue(s) this PR fixes:
Fixes #?
https://github.com/devfile/api/issues/819
PR acceptance criteria:

[ ] Unit Tests
[ ] E2E Tests
[ ] Documentation

How to test changes / Special notes to the reviewer:
Tested locally with with yarn dev and yarn start

/approve

/lgtm


Node.js Web controller client UI improvements

New UI looks nice. Lots of files needed for the new font. Any other option? Minor comments: Controls should be spelled with only one l

Thanks, for the quick response. A few more minor comments.

I think it would look nicer with equal button size (for single letter keys). What do you think?
For left/right/up/down you could consider using the corresponding arrow symbols like on the keyboard.
Please update the README.md by replacing the current screenshot of the UI with one of yours.

Great work @Coder-Lars!


Sending a request with Accept-Encoding: deflate, gzip won't produce a compressed response

A request using the mentioned header doesn't produce a response with Content-Encoding: gzip or deflate as expected. I've tried with this setting, but doesn't change much:
:pipeline-transform #(doto % (.addLast "deflater" (io.netty.handler.codec.http.HttpContentCompressor.)))

I don't see the deflater being added to the pipeline in Aleph's code anywhere. If this is supposed to be default in Netty, what does it need to be triggered otherwise?
Thanks @ztellman !

Also tried with:
:pipeline-transform #(doto %
                                                               (.addLast "decoder" (io.netty.handler.codec.http.HttpRequestDecoder.))
                                                               (.addLast "encoder" (io.netty.handler.codec.http.HttpRequestEncoder.))
                                                               (.addLast "inflater" (io.netty.handler.codec.http.HttpContentDecompressor.))
                                                               (.addLast "deflater" (io.netty.handler.codec.http.HttpContentCompressor.)))

In order for this to work, I had to use:
#(doto % (.addAfter "http-server" "deflater" (HttpContentCompressor. 1)))

Seems reasonable, but what about adding this to the pipeline by default. It's quite standard.
Also, do you think it would make sense to move https://github.com/ztellman/aleph/blob/master/src/aleph/http/server.clj#L398 up the pipeline before the handler?
Well, I see the benefits of having it this way too, so any function added to pipeline-transform can reference anything behind in the pipeline, and be placed wherever is required.
I would just add that the only inconvenience here is that for such an important feature, the way to enable it is not documented and not easily discoverable unless familiar with Netty.

Hi, sorry for the late reply to this.  I don't think this is something that should be enabled by default, but I think there's a strong argument to be made for adding a parameter to start-server which exposes this as an option.
Patches are welcome, or I'll get to it myself.

Hi @ztellman !
Thanks  for your reply. What about adding it simply as documentation? That would also work.

I'd be fine with a documentation patch, too.

Closing this one as compression is already supported since https://github.com/ztellman/aleph/commit/d137b5f4657478b5401bcb504e560e13bcfc5cff


Backlogs

What is a backlog in GitHub terminology? Is it issue with label "type:backlog"?
How to associate backlog items with particular backlog?

It's well written in the README file:

TIP: Use no:milestone in the search field on your issue dashboard to find backlog items.


V0.4.23b1 win support

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_f1d9qbg0b9/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:67: invalid_url: https://jax.readthedocs.io/ : Not reachable: 403
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_aebmvdea9r/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:69: invalid_url: https://jax.readthedocs.io/ : Not reachable: 403
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_15_7qrw33m/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:67: invalid_url: https://jax.readthedocs.io/ : Not reachable: 403
===== Final Report: =====
0 Errors and 1 Warning were found


ARROW-16831: [Go] panic in ipc.Reader when string array offsets are invalid

Add a check for invalid offsets in a string array and panic. This
prevents panic'ing later when accessing the column data or
attempting to write it with ipc.Writer.

@chriscasola LGTM the only thing i'd like is at least one test for the recovers added to the reader and writer to have them catch the panics and return errors.

LGTM! Thanks much @chriscasola


draft: [RFC] typesafe secrets

Checklist

[ ] run make test
[ ] PR title must follow conventional-commit-standard
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message and code follows the Code of conduct

@Pagebakers great work!

@jensneuse Can you merge this, the build_test doesn't run


Import or sweep into app a LBC paper wallet.

The Issue
I want to be able to import a LBC paper wallet account into the LBRY app.
New feature request.
Expected behaviour
QR scan of the code or manual entry that then imports or sweeps the wallet contents into the current wallet or perhaps create an entirely new wallet, but I would think it just getting opened then imported would be best, so a transaction would occur.
Actual behaviour
no current way to import this wallet.  at least not automatically.

@painbank thanks for opening this! Will require https://github.com/lbryio/lbry/issues/1049 to be implemented on the daemon first.

Need new UI changes to support daemon v.30


Add task-based permissions shortcuts

https://issues.redhat.com/browse/MGDSTRM-6585
https://issues.redhat.com/browse/MGDSTRM-6841

Let's merged app-services-ui PR first
https://github.com/redhat-developer/app-services-ui/pull/326

Hi @ajaypratap003 , the Assign permission section in the screenshot above looks good! I'm only able to view the screenshots though, so I don't know what the new "Add permissions" split button looks like when opened, or the info-tip popover. Would you please add a screenshot for those items as well? Thanks!

@lclay2  I've added few more screenshots . Add permission is split button and dropddown will open on click icon.

I was in back-to-back meetings today, so didn't get a chance to post this one other comment...
The width of the menu should be wider than the toggle. Right now, they are the same width, and this forces the menu to be much taller than it needs to be due to the wrapping description text. If it's wider, then the height will be reduced. Ideally, the menu would not have vertical scrolling, but without seeing a different width, I'm not sure if that's completely attainable, but it's worth trying to eliminate the vertical scroll bar if possible.
We had a similar use case with this example, where we were able to modify the width of the menu relative to the toggle: https://619e1921b3e38b003afeb2be-cdfgnosvcu.chromatic.com/?path=/story/components-metrics-filterbytopic--does-not-break-with-long-words

I was in back-to-back meetings today, so didn't get a chance to post this one other comment...
The width of the menu should be wider than the toggle. Right now, they are the same width, and this forces the menu to be much taller than it needs to be due to the wrapping description text. If it's wider, then the height will be reduced. Ideally, the menu would not have vertical scrolling, but without seeing a different width, I'm not sure if that's completely attainable, but it's worth trying to eliminate the vertical scroll bar if possible.
We had a similar use case with this example, where we were able to modify the width of the menu relative to the toggle: https://619e1921b3e38b003afeb2be-cdfgnosvcu.chromatic.com/?path=/story/components-metrics-filterbytopic--does-not-break-with-long-words

@jgiardino  I've raised a new PR for this scrollbar issue. Please have a look.
https://github.com/bf2fc6cc711aee1a0c2a/kafka-ui/pull/590


Overlay effects not applied to animations

When aplying overlayEffect they are ony applied to one animation state (not necessarily the right one) when playing an animation, the overlayeffect does not change

What is the res path for overlay effect you are using?

res:/dx9/model/effect/shieldhardening.red but its the same for shieldboosting armor hardening and repairing. None of them follows animations

There is an animated version of that effect: res:/dx9/model/effect/shieldhardening_skinned.red. In general almost all overlay effects have an _skinned version that needs to be applied to animated ships.  Unfortunately there is no easy way to know when the ship is animated or not.

Would object.wrappedObjects[0].animation.animations.length not be sufficient?

Not necessarily, but it should work in 99% of cases

I tried it, the problem is that during the shipLoad Callback the ship is not yet fully loaded and wrappedObjects[0].animation.animations.length is still 0

Use if ('isSkinned' in sof.hull[hullName] && sof.hull[hullName].isSkinned === 1) for sof ships, and the above for non-sof objects.

Adding the skinned overlays solves the issue. Checking animations.length has to be done after animations have fully loaded.


Rename Project and Tool

We must not continue to use the name "Intel Power Gadget". Also this tool is not BenchExec-specific.

The new name is cpu-energy-meter. At least the documentation and the name of the binary need to be adapted.

Done in commit 934868cd5b6f3dfb582f6747b7e9b71f0dc8b7fc. The 'README.md' has been updated accordingly.


Add command to copy tasks from other files to a time tracking table

Something like this:
https://user-images.githubusercontent.com/20051042/165466847-fa795f3b-aafe-4c1d-b91e-6a6f21f1c768.mov

Solved in #13.


Added Geometry ToElement function

Signed-off-by: Nate Koenig<EMAIL_ADDRESS>🎉 New feature
Summary
Added ToElement function for the Geometry class.
Test it
Run the tests.
Checklist

[x] Signed all commits for DCO
[x] Added tests
[x] Updated documentation (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-04-13-fortress-edifice/1367/1


Setuptools scm

I'm really not sure what to do with this. The branch that's proposed merging causes the continuous integration tests to fail because they can't bootstrap. I suspect it will also cause issues in other bootstrap scenarios. This work if merged will likely cause disruption. And at least until the CI issues are sorted, it's not acceptable. I do invite you to grab the code and investigate the issues and devise solutions.


progress.fractionCompleted is always 0

[x] I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.

What did you do?
Used Alamofire with iOS 10 and progress.fractionCompleted worked well, until iOS 11.
Alamofire.download(
    url,
    method: .get,
    parameters: nil,
    encoding: JSONEncoding.default,
    headers: headers,
    to: {
        (temporaryURL, response) in
        let dir = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first!

        localPath = dir.appendingPathComponent((path as NSString).lastPathComponent)
        return (localPath!, DownloadRequest.DownloadOptions.removePreviousFile)

}).downloadProgress(closure: { (progress) in

    print("fractions = ", progress.fractionCompleted) // always zero
    onProgress(progress.fractionCompleted)

}).response(completionHandler: { (response) in
    ....
    onCompletion(path)
})

What did you expect to happen?
Downloading progress from 0 to 1
What happened instead?
Downloading progress is always 0
Alamofire Environment
Alamofire version: 4.5.1
Xcode version: 9.0
Swift version: 3
Platform(s) running Alamofire: iPhone 5S Simulator / iPhone 5S device / iPhone 6 (all of them on 11.0.2 (15A421)
macOS version running Xcode: 10.13
Demo Project
Unable to, just use the code above.

TaskDelegate.swift => line 400:
progressHandler.queue.async { progressHandler.closure(self.progress) }
I inserted print to debug what happened, thats what I can see:
[ALAMOFIRE_DEBUG] self.progress = <NSProgress: 0x604000134780> : Parent: 0x0 / Fraction completed: 0.0000 / Completed: 1408060 of -1
[ALAMOFIRE_DEBUG] self.progress = <NSProgress: 0x604000134780> : Parent: 0x0 / Fraction completed: 0.0000 / Completed: 1399964 of -1

Going to debug a little further.

Not sure, but found at stackoverflow, it says Content-Length is not sent by server. I do see there's no Content-Length. Closing this issue.


Allow filtering by multiple metadata key/value pairs

We would like to be able to aggregate lines data by filtering for multiple pairs of metadata key/values.
This is assumed to be an AND operation for now.  Specifying multiple key/value pairs for metadata filtering means that the line must have all of the metadata provided to be included in the aggregation.

LGTM :+1:

+1 I'm curious as to how mysql will handle this if we have a ton of metadata but we can take a look at that when/if it happens

:+1:

+1 Nice!


[BUG] Modifiers to spell damage apply to poison

My account name is EvanTheGray, character Kora_The_Chaos_Herald. I'm playing ED with Consuming Dark.
With ED selected as main skill, in Calc -> Poison you can see that modifiers to spell damage apply also to poison, which isn't supposed to happen.

Working as intended; Essence Drain has the stat "Modifiers to Spell Damage apply to this skill's Damage Over Time", which applies to all DoTs created by the skill, including Poison.


Updated for tidy csv output + more retries if no broadcast detected

For the argument "file", changed output file format to be more concise. Reformatted the output table to a tidy csv ready for import. The first line contains names, the second line units and the following lines contain the numeric values only.

Instead of 10 retries, 100 retries are done before exiting if no broadvcast detected.

This changes the output file format of the csv file.

Units are in header, not with values.
Datetime now ISO-formatted with " " instead of "T" as separator.
The logger is now more patient waiting for the sensor, retrying 100 instead of 10 times.


Add docker as a development stack

This is a docker-compose setup to get SecretSanta running on docker-compose for development purposes.

I'm going to need some time to try this out. :-)
But as long as nothing outside of the docker folder is impacted I'm fine with pulling this into master.
Great job!


Refactor Backend Code for Better Structure and Readability

Is there an existing issue for this?

[X] I have searched the existing issues

Bug Description?
Description:
The current backend code has several long and complicated files that handle too many tasks at once, making it hard to understand and maintain. I want to improve the code's structure while keeping everything working the same way.
Objectives:

Break It Down: Split large files into smaller, focused pieces that do one thing well.
Organize Files: Rearrange the file structure to make it easier to find related parts of the code.
Clear Naming: Use simple and consistent names for variables, functions, and files to improve clarity.
Document Everything: Write clear documentation to explain the new structure and how different parts work together. (i'll add this in a README.md file)

By making these changes, we aim to create a cleaner and more manageable codebase that will be easier to work with in the future.
Add Screenshots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue (Optional)
[X] I'm willing to provide further clarification or assistance if needed.

@tejasbenibagde okay


Symbol with dynamic value

Hi dear friend
I used this service and set the symbol value dynamically
But after changing the name of the currency, the chart is not updated to be loaded with the new currency
I even used several methods to reload the component manually after changing the name of the currency, but unfortunately it did not work.
Can you help me to solve my problem?

Hi! Would you give me a code demo here?

Hi! Would you give me a code demo here?

const currencyName = ref('')
const Option = computed(() => ({
width: '88vw',
height: '500',
symbol: currencyName,
interval: '60',
style: '1',
locale: 'en',
enable_publishing: false,
allow_symbol_change: true,
theme: 'dark'
}))

*currencyName value is filled from the api

Hi! Would you give me a code demo here?

const currencyName = ref('')
const Option = computed(() => ({
width: '88vw',
height: '500',
symbol: currencyName,
interval: '60',
style: '1',
locale: 'en',
enable_publishing: false,
allow_symbol_change: true,
theme: 'dark'
}))
<Chart
      class="d-flex flex-row justify-content-center"
      :options="Option"

/>
*currencyName value is filled from the api

key may help you:
<Chart :key="Option.symbol"   class="d-flex flex-row justify-content-center" :options="Option" />

I used this method
but after choosing a new currency, it does not show the chart at all

same problem here.
I am facing same problem

I managed to fix this by copying src/components/Chart.vue into my project and removing if (scriptExists()) return, which prevented from re-rendering.
Here is the component you can copy and paste into your project:
<template>
	<div :id="container" />
</template>

<script lang="ts">
import { defineComponent, onMounted, ref } from 'vue'

declare global {
  interface Window {
    TradingView: any;
  }
}
window.TradingView = window.TradingView || {}
export default defineComponent({
	name: 'Chart',
	props: {
		options: {
			type: Object,
			default: () => ({}),
		},
	},
	setup(props) {
		const container = ref('tradingview-chart')
		const scriptID = ref('tradingview-chart-script')

		const defualtOptions = {
			width: 980,
			height: 610,
			symbol: 'NASDAQ:AAPL',
			interval: 'D',
			timezone: 'Etc/UTC',
			theme: 'light',
			style: '1',
			locale: 'en',
			toolbar_bg: '#f1f3f6',
			enable_publishing: false,
			allow_symbol_change: true,
			container_id: 'tradingview-chart',
		}

		const canUseDOM = () => {
			return typeof window !== 'undefined' && window.document && window.document.createElement
		}

		const getScriptElement = () => {
			return document.getElementById(scriptID.value)
		}

		// const scriptExists = () => {
		// 	return getScriptElement() !== null
		// }

		const appendScript = (onload: () => void) => {
			if (!canUseDOM()) return
			// if (scriptExists()) return

			const script = document.createElement('script')
			script.id = scriptID.value
			script.type = 'text/javascript'
			script.async = true
			script.src = 'https://s3.tradingview.com/tv.js'
			script.onload = onload
			document.getElementsByTagName('head')[0].appendChild(script)
		}

		const initWidget = () => {
			setTimeout(() => {
				if (typeof window.TradingView === 'undefined') return
				const options: { container_id: string } = { ...defualtOptions, ...props.options }
				if (options.container_id !== container.value) {
					console.error('container_id in Chart component must be "tradingview-chart"')
					return
				}
				new window.TradingView.widget(Object.assign({ container_id: container }, options))
			}, 300)
		}

		onMounted(() => {
			appendScript(() => initWidget())
		})
		return { container }
	},
})
</script>

I am pretty sure there is a better way to fix this. I may take a look at it later.

I managed to fix this by copying src/components/Chart.vue into my project and removing if (scriptExists()) return, which prevented from re-rendering.
Here is the component you can copy and paste into your project:
<template>
	<div :id="container" />
</template>

<script lang="ts">
import { defineComponent, onMounted, ref } from 'vue'

declare global {
  interface Window {
    TradingView: any;
  }
}
window.TradingView = window.TradingView || {}
export default defineComponent({
	name: 'Chart',
	props: {
		options: {
			type: Object,
			default: () => ({}),
		},
	},
	setup(props) {
		const container = ref('tradingview-chart')
		const scriptID = ref('tradingview-chart-script')

		const defualtOptions = {
			width: 980,
			height: 610,
			symbol: 'NASDAQ:AAPL',
			interval: 'D',
			timezone: 'Etc/UTC',
			theme: 'light',
			style: '1',
			locale: 'en',
			toolbar_bg: '#f1f3f6',
			enable_publishing: false,
			allow_symbol_change: true,
			container_id: 'tradingview-chart',
		}

		const canUseDOM = () => {
			return typeof window !== 'undefined' && window.document && window.document.createElement
		}

		const getScriptElement = () => {
			return document.getElementById(scriptID.value)
		}

		// const scriptExists = () => {
		// 	return getScriptElement() !== null
		// }

		const appendScript = (onload: () => void) => {
			if (!canUseDOM()) return
			// if (scriptExists()) return

			const script = document.createElement('script')
			script.id = scriptID.value
			script.type = 'text/javascript'
			script.async = true
			script.src = 'https://s3.tradingview.com/tv.js'
			script.onload = onload
			document.getElementsByTagName('head')[0].appendChild(script)
		}

		const initWidget = () => {
			setTimeout(() => {
				if (typeof window.TradingView === 'undefined') return
				const options: { container_id: string } = { ...defualtOptions, ...props.options }
				if (options.container_id !== container.value) {
					console.error('container_id in Chart component must be "tradingview-chart"')
					return
				}
				new window.TradingView.widget(Object.assign({ container_id: container }, options))
			}, 300)
		}

		onMounted(() => {
			appendScript(() => initWidget())
		})
		return { container }
	},
})
</script>

I am pretty sure there is a better way to fix this. I may take a look at it later.

I think watching the props with watch() will fix the issue. I'm gonna refactor the whole code base and fix this issue

@ehsan-shv Yeah that was one of my thoughts. Are you about to implement it this week, or later? I am thinking about contributing to this project and this seems something I can handle.

@ehsan-shv Yeah that was one of my thoughts. Are you about to implement it this week, or later? I am thinking about contributing to this project and this seems something I can handle.

I'm thinking about using Vite for packaging the components for NPM and it takes time. if you are interested in contributing to and refactoring the project, please send your Discord account to my Email then we can discuss refactoring in detail.

I've just checked and dynamic props work in the latest version. So just install the latest version. Also, this example may help.

If there is an issue with dynamic props please reopen the issue.


Diagramms in readme not visible, broken links

.

These were broken in a recent site migration but are now resolved.


Idea to integrate a new unparam linter

Hi,
I came across this linter which checks for unused function parameters https://github.com/mvdan/unparam which is fairly new but probably nice to add in the future.
Kind regards,
Jerry

Nice, thanks!

Probably it should also be added to the README.md

Ah, alas, it relies on sort.Sort() which is only in Go 1.8. I had to revert it.

Thats a pitty :(

@alecthomas FYI: You meant sort.Slice()

Support for 1.7 fixed - was not able to find what Go versions gometalinter expects to be able to run with.

@mvdan thanks for your fast response.
@alecthomas can you reopen this issue and when you have time check it again?
Thanks!

was done by  #260


Possible need for a preprocessor

See https://github.com/halfnelson/svelte-native-preprocessor:

It performs the following transforms to provide a better developer experience when using Svelte-Native:

[x] Adds <svelte:options namespace="xmlns" /> to the component, ensuring the generated code is compatible with svelte-native
[x] Changes bind:text="{email}" to text="{email}" on:textChanged="{e => email = e.value}"

We at least need the namespace aspect, as if PR https://github.com/sveltejs/svelte/pull/5652 lands, we'll require such preprocessing in order to close https://github.com/shirakaba/svelte-nodegui/issues/9.
I'm unsure whether we'll need it for bind:text; I haven't yet tried that feature (and indeed may not be able to if event listener names are forced to lower case – though I'm not sure of the casing used by NodeGUI yet).

I've published the preprocessor and incorporated it into both this repo's playground app and the official svelte-nodegui-starter. It works perfectly, thanks @mrsauravsahu!

Ah thanks @shirakaba for taking the lead on this. Great to hear that! 😃


Drop "Why not start the conversation?" from empty dm-including: views

As discussed on CZO, we should drop the "Why not start the conversation?" line from empty dm-including: views.

@zulipbot claim


No-Connect Fix for SVG Export

This should fix #123 but the code is not in the same compact style as the original. I get a different error after fixing this (see below), which I think might be related to the KiCAD refactor that seems to be happening on the main development branch. It is hard to be sure with my current lack of familiarity with this codebase though.
Traceback (most recent call last):
  File "/home/jwright/repos/skidl/src/skidl/skidlbaseobj.py", line 50, in __getattr__
    return self.__getattribute__("fields")[key]
KeyError: 'draw'

Thanks for looking into this and offering a fix! You found the cause of the problem, after which I found a way to fix it with fewer changes to the code. Go ahead and test in the the development branch when you have the time. Thanks!

@devbisme
Thanks. The error reported in #123 seems to be gone on development, but the other error I posted above seems to be different now.
Traceback (most recent call last):
  File "/home/jwright/repos/skidl/src/skidl/skidlbaseobj.py", line 50, in __getattr__
    return self.__getattribute__("fields")[key]
KeyError: 'num_units'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/jwright/Downloads/chlorophyll_skidl_trial/detector.py", line 17, in <module>
    generate_svg()
  File "/home/jwright/repos/skidl/src/skidl/circuit.py", line 980, in generate_svg
    f.write(self.generate_netlistsvg_skin(net_stubs=net_stubs))
  File "/home/jwright/repos/skidl/src/skidl/circuit.py", line 691, in generate_netlistsvg_skin
    part_svg[name] = part.generate_svg_component(
  File "/home/jwright/repos/skidl/src/skidl/part.py", line 1075, in generate_svg_component
    return tool_modules[tool].gen_svg_comp(self, symtx=symtx, net_stubs=net_stubs)
  File "/home/jwright/repos/skidl/src/skidl/tools/kicad6/gen_svg.py", line 419, in gen_svg_comp
    for unit in range(1, part.num_units):
  File "/home/jwright/repos/skidl/src/skidl/part.py", line 504, in __getattr__
    return SkidlBaseObject.__getattr__(self, attr)
  File "/home/jwright/repos/skidl/src/skidl/skidlbaseobj.py", line 52, in __getattr__
    raise AttributeError
AttributeError

Here is the minimal code I use to reproduce #123 and show that it is fixed in development with KiCAD 6.0.11.
from skidl import (Part,
                   Net,
                   POWER,
                   generate_svg,
                   set_default_tool,
                   KICAD6)

set_default_tool(KICAD6)

pwr_net = Net("VCC")
pwr_net.drive = POWER

# Create the Op Amp Part and connect one pin
op_amp = Part(lib="Amplifier_Operational", name='OP07', template=False, footprint="Package_SO:SOID-8-N7_3.9x4.9mm_P1.27mm")
pwr_net += op_amp["V+"]

generate_svg()

I should probably also post how I am running the script from the Linux terminal for full reproducibility.
KICAD_SYMBOL_DIR="/usr/share/kicad/symbols" KICAD6_SYMBOL_DIR="/usr/share/kicad/symbols" python mre_for_123.py

I'll look into this...

Ok, thanks. I'll go ahead and close this PR since it is no longer needed.


Bazel sync failed

I tried with WSL and followed all the steps as -https://github.com/google/mediapipe/blob/master/mediapipe/docs/install.md#installing-on-windows-subsystem-for-linux-wsl and got the hello world program run perfectly fine but still, it says BAZEL  sync failed.

I just tried Bazel sync on macOS and it works Okay.

Do you run Android Studio on Windows or WSL? If it's on Windows, my guess is it still calls Windows' Bazel to build, which is not supported yet by MediaPipe.

I am running Android Studio on Windows. So how do I build the application(hand tracking) after installing Bazel and running hello world application through WSL? I am using the Windows 10 system.

You need to do the following steps to build the APKs:

Run setup_android_sdk_and_ndk.sh to download and setup Android SDK and NDK in WSL.

Run the build command to build the apps in WSL. For example, you need to run the following command to build the hand tracking app.

bazel build -c opt --config=android_arm64 mediapipe/examples/android/src/java/com/google/mediapipe/apps/multihandtrackinggpu

In Windows, go to the Windows directory of the Linux Subsystem to get the apk. The default directory of WSL is C:\Users\YourUsername\AppData\Local\Packages\CanonicalGroupLimited.UbuntuonWindows_SomeID\LocalState\rootfs\home

Use adb to install the apk.

i tried hello world program in ubuntu,but when i bazel run it,I report some errors.Can you tell me some  advices?


Unable to get keytocard subcerts to yubikey 5 NFC (piv) :-/

Hi there
Struggling to get anything to work with gpg, but trying to get my PIV sub certs onto the Yubikey for use with NixOS.
I have changed my 3 passwords for PIV in the yubikey manager  gui (cli's just crashed for me) and protected the management password with the admin pin.
Anyway, when doing a keytocard export I am entering my Yubikey management key and admin code (PUK) as specified, and it rejects the passwords and codes each time.
Driving me nuts, as not sure what I am doing wrong.
Also tried with the admin pin and then user pin, but that also failed, as well as the key password itself, all failed.
I've reset PIV a good few times due to password issues.
Any help or direction would be much appreciated :)

Try Resetting the card and provisioning it with default PIN values and using a simple password for GnuPG. Also try provisioning it on a different system.

Hi there
@drduh Thanks for this nice guide.
However, I also struggled with this problem for a while. (Incidentally, also on NixOS.)
My understanding after reading this issue is that RSA4096 keys are not supported by Yubikey 5 NFC's PIV.
I wonder if this limitation of the Yubikey should be dealt with in this guide?


[master] Upgrade to latest dependencies

Cron. -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced via:
./hack/update-deps.sh --upgrade --release v0.21
./hack/update-codegen.sh

Details:
bumping knative.dev/pkg 22f4898...75d66b5:
  > 75d66b5 preserve the stable webhook logger names we had before (# 1994)

/lgtm
/approve

/lgtm
/approve


[high] JAWS+Chrome: Licenses: The expandable sections are announced with no role or state

Action Performed:

Using JAWS+Chrome, open https://staging.new.expensify.com/
Tab to focus the "licenses" link press Enter key.
Press down arrow key to focus the expandable state sections and listen to the announcements.

Expected Result:
The expandable elements should be announced along with their button role and collapsed/expanded state.
Actual Result:
The expandable elements are announced as headings, no interactive role or state is announced.
HTML:

Alabama

Workaround:
No workaround
Area issue was found in:
Licenses page
Failed WCAG checkpoints
4.1.2
User impact:
The screen reader users are not informed about the functionality or current state of the controls.
Suggested resolution:
Implement the expandable sections using native <button> as suggested in issue 5181344. Use aria-expanded attribute on the buttons to announce their collapsed or expanded state accordingly. See: https://accessibility.huit.harvard.edu/technique-expandable-sections
Platform:
Where is this issue occurring?

Web

Version Number:
**Reproducible in staging?:**Yes
Reproducible in production?:
Notes/Photos/Videos: Any additional supporting documentation
Issue reported by:
https://user-images.githubusercontent.com/88733897/134720268-cc7e516d-db2e-4fb4-9d15-0854fbbd48a2.mp4

View all open jobs on GitHub

Same comment: So the terms of service page is hosted on UseDot and it's technically not part of the NewDot app. @stitesExpensify @kadiealexander do you still think it's worth addressing these issues, or should we punt and focus on the app itself?


feat: disable compression feature on router

Motivation and Context
I had this issue #904 after a lot of hours I couldn't figure out why compression is causing this issue.
I decided to had a config env setting to disable compression to bypass this issue (I don't need compression).

TODO

[ ] Tests or benchmark included
[ ] Documentation is changed or added on https://app.gitbook.com/
[ ] PR title must follow conventional-commit-standard

This PR can't be merged as it is. First, we need a config proposal.


New Device [Blink Camera]

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
I was about to add an entry for Amazon Blink Cameras, but noticed that the model shows up as "unknown" in the device list, and is missing in the device page, in HomeAssistant. How can we add battery notes to such devices?
Will adding the following into library.json work?:
        {
            "manufacturer": "Blink",
            "battery_type": "AA",
            "battery_quantity": 2
        }

Describe the solution you'd like
Any, as long as I can add battery notes to Blink XT2 cameras.
Describe alternatives you've considered
na
Additional context
na

Battery Notes requires a model to be present so this won't work.
For this device I'd suggest just adding a note manually.


Give total number of tests in result as well.

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

Project coverage is 100.00%. Comparing base (d3e71df) to head (dbca0e9).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          965       982   +17
=========================================
+ Hits           965       982   +17

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Is it ok to use my own RGBD dataset?

Thank you for such a great job！
Is it ok to use my own RGBD dataset? I have a mutli-room RGBD dataset, but I don't have IMU, LiDAR, Stereo and other data, can your code run on this dataset?
Thank you for your reply！

Yes you can run it with RGBD as long as you have a visual odometry software working for your data.
For example, use this launch file, remove the camera, lidar, and IMU nodes and modify the odometry node (currently lidar-based) for the RGB-D visual odometry solution of your choice.

@lajoiepy To supplement the above question: the RGB image of my RGBD dataset is in .jpg format, the Depth image is in .png format, and the pose is in .txt format. Will this still work with SWARM-SLAM? My concern is that only the .bag dataset format will work with SWARM-SLAM. If not, how can I convert the data format to meet the requirements? Thanks!

Hi! Indeed, you need to convert your image sequence to a ROS 2 bag. Here is a tutorial to create a bag from your data https://docs.ros.org/en/foxy/Tutorials/Advanced/Recording-A-Bag-From-Your-Own-Node-CPP.html . There is also a python interface for this.

If I modify your VIO module to a VO module (such as ORB2-SLAM), and the input of the system is an RGBD data set, can this experiment be said to be the experimental result of your paper? Because you mentioned "Our system supports inertia, lidar, stereo, and RGB-D sensing," in your paper, but doing the above experiment requires modifying your code, so I don't know. Because I now need a traditional C-SLAM comparison experiment with RGBD dataset (the worst is RGB dataset) as input.

The odometry source is external to our system. We provide rtabmap launch files as an example, but you can use the solution of your choice (e.g. ORB-SLAM) for odometry without modifying our code base. You only need to adapt the launch and config files accordingly.


Error after cloning calendar.vim into ~/.vim/plugin

hi, first of all thank you so much for making this wonderful plugin for vim, I didn't think two times to try it, but unfortunately I cant fix the problems by mi own, so I tried to isntall with the vim plugin manager:

I first cd int ~/.vim/plugin
git clone https://github.com/itchyny/calendar.vim
then when i start vim this output of "start.log" file comes, then vim starts normally
then when vim starts i type :Calendar, and i get the output of "cal.log" file
start.log
cal.log

I tried to put all inside calendar.vim folder inside a new "start" folder bot the same result.
By the way, i'm in Arch linux and this is my vim compilation:
compilation.txt

You can't clone this plugin to ~/.vim/plugin. Please use a plugin manager like vim-plug.

OMG!! sry, so I will try with my plugin manager, by the way is vundle, calendar.vim works whit vundle to??

Yes, it should work with vundle too.


Setting host! in before (:each or :all) does not work in Request-specs

What Ruby, Rails and RSpec versions are you using?
Ruby version: 3.3.6
Rails version: 8.0.0
RSpec version: 3.13

rspec-core 3.13.2
rspec-expectations 3.13.3
rspec-mocks 3.13.2
rspec-rails 7.1.0
rspec-support 3.13.1

Observed behaviour
I prepared this spec
require 'rails_helper'

# This spec is a basic test that verifies that the code is working.
RSpec.describe 'Basic API Tests' do
  describe 'GET /api/users/steviee/repos', type: :request do
    before :each do
      host! 'timeapi.io'
    end

    it 'returns successful response' do
      get "/api/time/current/zone?timeZone=Europe/Berlin"
      expect(response).to have_http_status(200)
      expect(response.body["timeZone"]).to eq('Europe/Berlin')
    end
  end
end

The above code is expected to run the request against the host timeapi.io, however it is run against <IP_ADDRESS> as per logs:
[c93d8682-67ca-4aad-9545-5705a736ee01] Started GET "/api/time/current/zone?timeZone=Europe/Berlin" for <IP_ADDRESS> at 2024-11-12 16:17:57 +0100
[c93d8682-67ca-4aad-9545-5705a736ee01]
[c93d8682-67ca-4aad-9545-5705a736ee01] ActionController::RoutingError (No route matches [GET] "/api/time/current/zone"):
[c93d8682-67ca-4aad-9545-5705a736ee01]
  TRANSACTION (0.0ms)  ROLLBACK TRANSACTION
F

Failures:

  1) Basic API Tests GET /api/users/steviee/repos returns successful response
     Failure/Error: expect(response).to have_http_status(200)
       expected the response to have status code 200 but it was 404
     # ./spec/accounts/basic_api_spec.rb:11:in `block (3 levels) in <top (required)>'

Finished in 0.22201 seconds (files took 0.5135 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/accounts/basic_api_spec.rb:9 # Basic API Tests GET /api/users/steviee/repos returns successful response

Expected behaviour
The expected behavior would be calling another host as per host!-setting.

Can you provide an example reproduction?
See above.
Created a new fress api-only Rails 8.0.0 app, added RSpec and the spec from above.
No further configuration on RSpec was done.

To clarify, what are you expecting to work here which isn't, the host! functionality is part of the Rails test helpers which sets the header HTTP_HOST for a test request that makes Rails act as if that domain name was used, but it doesn't affect the IP address any server is listening on, or a request is made from...

Ah, I see! So whatever I do, it won't call a remote endpoint, except if I use rest-client or some other library explicitly, correct?
Kind regards and thank you for the clarification! I missed that point explicitly mentioned and was expecting a different behavior.

Indeed, there is no built in functionality for testing remote systems.


Collection filtering and sorting

The collection template should include a toolbar containing filters, categories and sort options (ex: low to high price)

There is a nice and mostly complete example in the Collection template.


bcc/libbpf different behavior in attach event

What happened?
compare with same machine but with libbpf and bcc image logs:
libbpf version:
I0906 03:44:15.849230       1 libbpf_attacher.go:149] failed to attach perf event cpu_instructions_hc_reader: failed to open bpf perf event on cpu 0: no such file or directory
I0906 03:44:15.849254       1 libbpf_attacher.go:149] failed to attach perf event cache_miss_hc_reader: failed to open bpf perf event on cpu 0: no such file or directory
I0906 03:44:15.849262       1 libbpf_attacher.go:149] failed to attach perf event cpu_cycles_hc_reader: failed to open bpf perf event on cpu 0: no such file or directory
I0906 03:44:15.849270       1 libbpf_attacher.go:149] failed to attach perf event cpu_ref_cycles_hc_reader: failed to open bpf perf event on cpu 0: no such file or directory

I saw this in libbpf version (libbpf-latest) only but not on bcc version
which supposed to report ?
https://github.com/sustainable-computing-io/kepler/blob/main/pkg/bpfassets/attacher/bcc_attacher.go#L120
What did you expect to happen?
libbpf/bcc should behavior same in identical env?
How can we reproduce it (as minimally and precisely as possible)?
compare latest version
Anything else we need to know?
No response
Kepler image tag

Kubernetes version

$ kubectl version
# paste output here

Cloud provider or bare metal

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Kepler deployment config

For on kubernetes:
$ KEPLER_NAMESPACE=kepler

# provide kepler configmap
$ kubectl get configmap kepler-cfm -n ${KEPLER_NAMESPACE}
# paste output here

# provide kepler deployment description
$ kubectl describe deployment kepler-exporter -n ${KEPLER_NAMESPACE}

For standalone:
put your Kepler command argument here

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

Is it a VM or a BM?

it's a VM, but doesn't matter, I think they should be identical, do they?

Where is the bcc logs? I can only see the libbpf.
So, both bcc and libbpf should show warnings that the hardware counters cannot be collected because it is a VM.

I didn't paste it as I can't see in my log output..

In the BCC code we return if a hardware counter cannot be attached: https://github.com/sustainable-computing-io/kepler/blob/main/pkg/bpfassets/attacher/bcc_attacher.go#L115
The libbpf try to attach all hardware counters regardless if there was an error: https://github.com/sustainable-computing-io/kepler/blob/main/pkg/bpfassets/attacher/libbpf_attacher.go#L144

ok, will read more about the logic and see whether something can be improved

ok, I think above logic make sense so this should not be a problem


RFC proposal: IonQ integration

This is a proposal to integrate vendor IonQ's api into Cirq:
RFC: Cirq IonQ integration
See RFC document for details.

Ooops wrong doc shared, should be correct now.

This work is done.


Feature Request: default values

Like environs it would be useful to be able to define a "fallback" value if the requested VAR isn't found in the environment.

Even doing a Jinja test for defined fails if LEKTOR_TEST isn't in the env. eg:
{% if envvars('TEST') is defined %}optionif{% else %}optionelse{% endif %}

Will output error
raise EnvValidationError('Environment variable "{}" not set'.format(source_key), [message])
environs.EnvValidationError: Environment variable "LEKTOR_TEST" not set


Master to Live

Docs Build status updates of commit 3ba46a1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Add-UnifiedGroupLinks.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-ExchangeCertificate.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-M365DataAtRestEncryptionPolicyAssignment.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-RecoverableItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-Label.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Restore-RecoverableItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-Label.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-M365DataAtRestEncryptionPolicyAssignment.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-MyAnalyticsFeatureConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-User.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-DataClassification.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-TextExtraction.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/exchange.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/mapping/serviceMapping.json
:white_check_mark:Succeeded

skype/skype-ps/skype/Get-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Grant-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

teams/teams-ps/teams/New-CsGroupPolicyAssignment.md
:white_check_mark:Succeeded
View (teams-ps)

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

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

Docs Build status updates of commit 3ba46a1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md
:bulb:Suggestion
View (sharepoint-ps)
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Add-UnifiedGroupLinks.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-ExchangeCertificate.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-M365DataAtRestEncryptionPolicyAssignment.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-RecoverableItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-Label.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Restore-RecoverableItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-Label.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-LabelPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-M365DataAtRestEncryptionPolicyAssignment.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-MyAnalyticsFeatureConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-RetentionCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-User.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-DataClassification.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-M365DataAtRestEncryptionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Test-TextExtraction.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/exchange.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/mapping/serviceMapping.json
:white_check_mark:Succeeded

skype/skype-ps/skype/Get-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Grant-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/New-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Remove-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsFeedbackPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

teams/teams-ps/teams/New-CsGroupPolicyAssignment.md
:white_check_mark:Succeeded
View (teams-ps)

sharepoint/docs-conceptual/sharepoint-online/connect-sharepoint-online.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

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


Optimize AdaptiveRadixTree

Benchmark results:
Benchmark                                                      Mode  Cnt         Score   Error   Units
JavaTypeCacheBenchmark.readAdaptiveRadix                      thrpt    2       609.359           ops/s
JavaTypeCacheBenchmark.readAdaptiveRadix:gc.alloc.rate        thrpt    2      1814.269          MB/sec
JavaTypeCacheBenchmark.readAdaptiveRadix:gc.alloc.rate.norm   thrpt    2   3122809.369            B/op
JavaTypeCacheBenchmark.readAdaptiveRadix:gc.count             thrpt    2       169.000          counts
JavaTypeCacheBenchmark.readAdaptiveRadix:gc.time              thrpt    2        61.000              ms
JavaTypeCacheBenchmark.readSnappy                             thrpt    2       332.215           ops/s
JavaTypeCacheBenchmark.readSnappy:gc.alloc.rate               thrpt    2      3192.919          MB/sec
JavaTypeCacheBenchmark.readSnappy:gc.alloc.rate.norm          thrpt    2  10082058.492            B/op
JavaTypeCacheBenchmark.readSnappy:gc.count                    thrpt    2       179.000          counts
JavaTypeCacheBenchmark.readSnappy:gc.time                     thrpt    2        78.000              ms
JavaTypeCacheBenchmark.writeAdaptiveRadix                     thrpt    2       469.081           ops/s
JavaTypeCacheBenchmark.writeAdaptiveRadix:gc.alloc.rate       thrpt    2      4802.820          MB/sec
JavaTypeCacheBenchmark.writeAdaptiveRadix:gc.alloc.rate.norm  thrpt    2  10738906.909            B/op
JavaTypeCacheBenchmark.writeAdaptiveRadix:gc.count            thrpt    2       197.000          counts
JavaTypeCacheBenchmark.writeAdaptiveRadix:gc.time             thrpt    2       256.000              ms
JavaTypeCacheBenchmark.writeSnappy                            thrpt    2       313.278           ops/s
JavaTypeCacheBenchmark.writeSnappy:gc.alloc.rate              thrpt    2      3414.677          MB/sec
JavaTypeCacheBenchmark.writeSnappy:gc.alloc.rate.norm         thrpt    2  11435420.340            B/op
JavaTypeCacheBenchmark.writeSnappy:gc.count                   thrpt    2       133.000          counts
JavaTypeCacheBenchmark.writeSnappy:gc.time                    thrpt    2       174.000              ms

Additionally, here is the memory footprint for the two implementations:
Retained AdaptiveRadixTree size:    8425616 bytes
Retained Snappy size:               8613760 bytes

As can be seen reads are over 80% faster and writes about 50% faster. At the same time the allocations for reads drop by around 70%. For reads only around 5%. Finally, the memory footprint is now almost exactly the same (even slightly lower), so the implementation looks like it is better across the board. An additional bonus is that the implementation doesn't depend on Snappy.


P4001 The introspected database was empty

Bug description
I tried npx prisma db pull with a minimal SQLite database with a single table events. However the prisma schema is not updated.
I've validated that the database contains at least one table:
sqlite3 test.db

SQLite version 3.36.0 2021-06-18 18:58:49
Enter ".help" for usage hints.

sqlite> .tables
events

sqlite> .schema events
CREATE TABLE events(
id INTEGER PRIMARY KEY,
payload TEXT
created_at TEXT
);

The following error is produced (with DEBUG mode on):
npx prisma db pull
  prisma:loadEnv project root found at /Users/immi/Code/guarda/package.json +0ms
  prisma:tryLoadEnv Environment variables loaded from /Users/immi/Code/guarda/.env +0ms
Environment variables loaded from .env
  prisma:engines binaries to download libquery-engine, migration-engine, introspection-engine, prisma-fmt +0ms
Prisma schema loaded from prisma/schema.prisma
  prisma:getConfig Using CLI Query Engine (Node-API Library) at<EMAIL_ADDRESS>+0ms
Datasource "db": SQLite database "guarda.db" at "file:guarda.db"

Introspecting based on datasource defined in prisma/schema.prisma …
  prisma:introspectionEngine:rpc starting introspection engine with binary: /Users/immi/Code/guarda/node_modules/@prisma/engines/introspection-engine-darwin-arm64 +0ms
  prisma:introspectionEngine:rpc SENDING RPC CALL {"id":1,"jsonrpc":"2.0","method":"introspect","params":[{"schema":"// This is your Prisma schema file,\n// learn more about it in the docs: https://pris.ly/d/prisma-schema\n\ngenerator client {\n  provider = \"prisma-client-js\"\n}\n\ndatasource db {\n  provider = \"sqlite\"\n  url      = env(\"DATABASE_URL\")\n}\n"}]} +4ms
  prisma:introspectionEngine:rpc {
  prisma:introspectionEngine:rpc   jsonrpc: '2.0',
  prisma:introspectionEngine:rpc   error: {
  prisma:introspectionEngine:rpc     code: 4466,
  prisma:introspectionEngine:rpc     message: 'An error happened. Check the data field for details.',
  prisma:introspectionEngine:rpc     data: {
  prisma:introspectionEngine:rpc       is_panic: false,
  prisma:introspectionEngine:rpc       message: 'The introspected database was empty: file:guarda.db',
  prisma:introspectionEngine:rpc       meta: [Object],
  prisma:introspectionEngine:rpc       error_code: 'P4001'
  prisma:introspectionEngine:rpc     }
  prisma:introspectionEngine:rpc   },
  prisma:introspectionEngine:rpc   id: 1
  prisma:introspectionEngine:rpc } +6ms
Error: Error:
P4001 The introspected database was empty:

prisma db pull could not create any models in your schema.prisma file and you will not be able to generate Prisma Client with the prisma generate command.

To fix this, you have two options:

- manually create a table in your database.
- make sure the database connection URL inside the datasource block in schema.prisma points to a database that is not empty (it must contain at least one table).

Then you can run prisma db pull again.

    at Object.parse (/Users/immi/Code/guarda/node_modules/prisma/build/index.js:103378:17)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async main (/Users/immi/Code/guarda/node_modules/prisma/build/index.js:106144:18)

How to reproduce
Run npx prisma db pull on this sqlite db. The db file sits in the same directory where I invoke npx (and === pwd)
Expected behavior
The prisma schema is updated with the model from the events table.
Prisma information
my .env file:
DATABASE_URL="file:guarda.db"

my schema.prisma file:
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "sqlite"
  url      = env("DATABASE_URL")
}

Environment & setup

OS: Mac OS 12.0
Database: SQLite 3.36.0
Node.js version: v17.0.1

Prisma Version
prisma                  : 3.8.1
@prisma/client          : 3.8.1
Current platform        : darwin-arm64
Query Engine (Node-API) : libquery-engine 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at node_modules/@prisma/engines/migration-engine-darwin-arm64)
Introspection Engine    : introspection-core 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at node_modules/@prisma/engines/introspection-engine-darwin-arm64)
Format Binary           : prisma-fmt 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at node_modules/@prisma/engines/prisma-fmt-darwin-arm64)
Default Engines Hash    : 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f
Studio                  : 0.452.0

I actually get the following error message:
C:\Users\Jan\Documents\throwaway\11175>npx prisma db pull
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db": SQLite database "guarda.db" at "file:./guarda.db"

Introspecting based on datasource defined in prisma\schema.prisma …

✔ Introspected 1 model and wrote it into prisma\schema.prisma in 39ms

*** WARNING ***

These fields are not supported by the Prisma Client, because Prisma currently does not support their types.
- Model "events", field: "payload", original data type: "TEXT
created_at TEXT"

Run prisma generate to generate Prisma Client.

But at the same time this introspects as:
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "sqlite"
  url      = env("DATABASE_URL")
}

model events {
  id      Int                                  @id @default(autoincrement())
  payload Unsupported("TEXT
created_at TEXT")?
}

Note the invalid line break after TEXT.
That is happening because your original SQL is missing a , after payload TEXT. Seems SQLite happily pulls in the line break and everything after until it finds another , and uses that multi line string as the "type".

Yikes! Thank you for catching this.
I've tried with a new db, with a minimal events table, but still the same error.
Now if I use an absolute path in my DATABASE_URL it works. So it must be because of the relative path, but I don't really understand why.

OK I believe this is the same as #9649 as there sits an empty db with the same name in the ./prisma directory.

Ok, let's close this then. Seems I uncovered an unrelated problem, and your issues is better described with that other issue.


Path is incorrect escaped => activeClassName is not applied ()

Hello at all,
I am currently facing the problem that there is no activeClassName applied to any <NavLink /> components even if the path (to) definitely matches the url. I tried to find the reason why and I found that the problem is the following line: NavLink.js#L25.
Version
4.3.1
Test Case
E.g. my path is /studiengaenge/60/module and is escaped to \/studiengaenge\/60\/module in NavLink.js. After that matchPath compiles this path to /^\\\/studiengaenge\\\/60\\\/module\/?(?=\/|$)/i. The problem is, that <Route /> expects a pattern like /^\/studiengaenge\/60\/module\/?(?=\/|$)/i to be the current route and therefore the path is seen as not the current one. I think that the escaping should be removed. When I use the unescaped path as argument for <Route /> everything works as expected.
Expected Behavior
I expect that activeClassName is applied but it isn't. Pattern for <Route /> should be /^\/studiengaenge\/60\/module\/?(?=\/|$)/i.
Actual Behavior
Pattern for <Route /> is  /^\\\/studiengaenge\\\/60\\\/module\/?(?=\/|$)/i.

I am also not sure why <NavLink /> needs escaping at all. Shouldn't <Route /> be responsible for this?

@MeiKatz have you seen #6201?

Yes, this is a dupe of that issue.

@pshrmn Yes, I have seen this issue. But for me it seems that this issue was solved with changing path.resolve("node_modules") to "node_modules". I tried this but it didn't work for me. So I opened this new issue.

@pshrmn I will keep it in mind. Next time I will do so.


OCaml 4.08 support

This PR makes Reason build on OCaml 4.08.
It depends on https://github.com/let-def/merlin-extend/pull/9
The 4.08 CI steps are failing because of the above dependency. Perhaps we should vendor merlin-extend too as it's such a small dep?

Woohoo! Thanks as always.


Possible fix for the encoding bug

I had a very similar issue today, and starting to mess with backslashreplace (only available in python >= 3.5).
In case we want to still support python 3.4, the hotfix look like that:
# === PYTHON 3.0 - 3.4 SUPPORT ======================================================

# From https://gist.github.com/ynkdir/867347/c5e188a4886bc2dd71876c7e069a7b00b6c16c61
import sys
if sys.version_info >= (3, 0) and sys.version_info < (3, 5):
    import codecs

    _backslashreplace_errors = codecs.lookup_error("backslashreplace")

    def backslashreplace_errors(exc):
        if isinstance(exc, UnicodeDecodeError):
            u = "".join("\\x{0:02x}".format(c) for c in exc.object[exc.start:exc.end])
            return (u, exc.end)
        return _backslashreplace_errors(exc)

    codecs.register_error("backslashreplace", backslashreplace_errors)

Codecov Report

:exclamation: No coverage uploaded for pull request base (logging@45d71cb). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             logging       #1   +/-   ##
==========================================
  Coverage           ?   85.33%
==========================================
  Files              ?        3
  Lines              ?      600
  Branches           ?        0
==========================================
  Hits               ?      512
  Misses             ?       88
  Partials           ?        0

Impacted Files
Coverage Δ

bin/filecheck.py
82.3% <100%> (ø)

Continue to review full report at Codecov.

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


Upgrade to Bevy 0.4.

Hey @ndarilek ,  There are changes that are not related to bevy upgrade in this or i.e. code formatting, parameter aligning change...etc. They should not be part of this request.

Done.

@ndarilek Thank you for the pr.


CSUB-1160: Upgrade Ubuntu - test against testnet for Azure self-hosted runners

Description of proposed changes

Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

NOTE: only the last commit is relevant here. Should be merged into dev after #380.

Sucessfull deployment of self-hosted runner w/ Ubuntu 24.04 can seen here:
https://github.com/gluwa/creditcoin3/actions/runs/9384175960


setClickTypeDetection and single click type detection

I started to play with setClickTypeDetection to get more reactive click.
So i used setClickTypeDetection(false) but then i can t have single click with app gestures like move/zoom
The reason is that the click seems to be triggered on down and not up, and thus not canceled by move/zoom gestures.
What i would expect with setClickTypeDetection(false) is to not have long/double click types but still have single click working with gestures.
Or maybe i got it wrong and it is not what it is made for?

I have reviewed the behavior of setClickTypeDetection and indeed, it made little sense. Click event was triggered on touch, not on click. I have updated the behavior in master and added ClickInfo attribute to MapClickInfo, VectorTileClickInfo, etc to get additional information about the touch (like click duration).
Thanks for bringing this into attention.

@mtehver amazing that will be a great step forward! thanks. Are you going to look at this too https://github.com/CartoDB/mobile-sdk/issues/437 ? I would love to disable only double click to get faster click detection while keeping the ability to get long click

@farfromrefug With click duration available via ClickInfo you can do short/medium/long click detection easily yourself.

@mtehver i see ! You are right so you say setClickTypeDetection(false) then handle it myself?
It will work though i still something is wrong with double click handling. It should be possible to disable it and we should have very fast single click while being able to do long click

@mtehver i tried with setClickTypeDetection(false) and using duration to handle long press but it wont work!
A long press gesture is not triggered on on touch up but on long press timeout. I dont see a way to do it with setClickTypeDetection(false)

@farfromrefug Are you sure you have the latest version from master? There is no timeout after setClickTypeDetection(false), the event is triggered only on touch up.

@mtehver yes it is my point. No timeout so no way to hanfle longress that way

@farfromrefug Ok, I think I understand your point now. So you would like to have click triggered even when finger is not lifted after 500ms? And perhaps this 500ms should be configurable via options?

@mtehver yes you are right but let me sums up:

setClickTypeDetection(true):

there should be a way to disable double click
when double click is disabled there should be no more delay for single click. (the delay exist to detect double click)
options to customize long press timeout ?

setClickTypeDetection(false):

no dual/double/long ... detection
thus not sure you want to trigger after a customisable time out ? Maybe the current (new behavior) is what you intended for that option all along.

To conclude the whole point of my discussion was to aim for this:

single click on touch up with no delay (like we do with setClickTypeDetection(false))
no double click as i dont need it
long click feature with  optional timeout duration feature (if possible)

@farfromrefug This makes sense, in some cases the current long click detection mode may be a good fit. I have updated the master branch to reflect this proposal. There are now 2 additional methods in Options class: setDoubleClickDetection (default value true) and setLongClickDuration (default value 0.4, that is 400ms).

@mtehver it is perfect ! i added this https://github.com/CartoDB/mobile-sdk/pull/445 to fine tune it all

Thanks! #445 is merged and will be in final 4.4.1 release.

4.4.1 is released now, thus closing.


Scroll offset with dynamic sizes not working properly in React 16

Describe the bug
When adjusting scales of items, the scroll offset is maintained when items get bigger, but are lost in a React 16 environment when items get smaller.
Your minimal, reproducible example
https://codesandbox.io/p/devbox/funny-sound-9dvq49?file=%2Fsrc%2Fmain.tsx%3A92%2C34
Steps to reproduce

Scroll down
Zoom in (Scroll position is maintained, sometimes)
Zoom out (Scroll position is shifted down a third of the page, everytime)

Expected behavior
As a user, my scroll position should remain the same as my items scale, as it is in React 18
React 18 example: https://codesandbox.io/p/devbox/suspicious-orla-wcmzp8?file=%2Fsrc%2Fmain.tsx%3A96%2C37
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

OS: Mac - Browser: Chrome - Version: 120.0.6099.216

tanstack-virtual version
v3.0.1
TypeScript version
4.9.5 and 5.2.2
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

This modification to resizeItem appears to solve for me.
resizeItem = (item: VirtualItem, size: number) => {
    const itemSize = this.itemSizeCache.get(item.key) ?? item.size
    const delta = size - itemSize

    if (delta !== 0) {
      //
      // note change below
      //
      if (
        item.start < _this.scrollOffset + _this.scrollAdjustments
      ) {
        if (process.env.NODE_ENV !== 'production' && this.options.debug) {
          console.info('correction', delta)
        }

        this._scrollToOffset(this.scrollOffset, {
          adjustments: (this.scrollAdjustments += delta),
          behavior: undefined,
        })
      }

      this.pendingMeasuredCacheIndexes.push(item.index)
      this.itemSizeCache = new Map(this.itemSizeCache.set(item.key, size))

      this.notify(false)
    }
  }


Include version in tasks package directory (server or client issue?)

(maybe related to https://github.com/fractal-analytics-platform/fractal-server/issues/215)
Calling the client as
fractal task collect fractal-tasks-core --package-version 0.3.2

results into the creation of the folder ${FRACTAL_ROOT}/.fractal/fractal-tasks-core instead of something like ${FRACTAL_ROOT}/.fractal/fractal-tasks-core-0.3.2.
TO CHECK: is this a client or server problem?

The error became evident when trying to install two different versions at the same time, as their folders would have the same name.