[Issue #245] feat: delete paybutton

Description:
Solves #245. Users can delete paybuttons from the frontend (for that I made a very ugly button that simply works).
Test plan:
Go to http://localhost:3000/buttons and try to delete them. Also should be covered by tests.
Depends on:

[x] #302

I'm seeing some display issues on the buttons page.

Indeed, I didn't care about that however; I just tried to make a pipeline that worked and leave the front end for you :shrug: (it would take a lot longer for me to brute-force the CSS into making it work)

And actually there should already be some UI in place for deleting. If you you go to the button detail page and click the pencil icon, there is a delete button. If we can get this functionality there, that'd be ideal

Nice, I didn't know that. In this case, I will remove the ugly buttons from the button list and move this logic there.

Ready for review after dependency PRs are merged.


What is that? nat symbol star? Ugly! Confusing!
We can eliminate (probably?) the uses of nat and symbol if we never actually use unrefined types (see here) though I'm not yet sure how. This will reduce the type noise to just Api star which is easier to explain and less noisy. This is a big improvement!

These instances are a bit broken by this change:
instance HeaderEncode 'AccessControlAllowHeaders (Set (HeaderType Text)) where
  headerEncode _ = displaySetOpt . Set.map headerName

instance HeaderEncode 'AccessControlAllowHeaders [HeaderType Text] where
  headerEncode = uniqueSet

I could probably replace those with something like Set SomeHeader for SomeHeader = exists (h :: HeaderType Symbol) . Sing h. That's a possible way forward!


CICD: add CARGO_TERM_COLOR env var for colors in CI output

This hopefully makes the CI output a bit prettier :)
EDIT: Seems to have worked mostly. I want to figure out how to also enable color in the tests though.

Great, apparently it's due to an issue open since 2015 :)
https://github.com/rust-lang/cargo/issues/1983

Well, I can't get it to work in cargo test. I remove those changes and leave that for later.

While I log colors, i am not a big fan in the CI. it is clobbering the raw logs:
https://pipelines.actions.githubusercontent.com/serviceHosts/91055393-6e6f-4662-ba97-3acee74a8c7c/_apis/pipelines/1/runs/20716/signedlogcontent/6?urlExpires=2023-01-27T20%3A12%3A46.9850910Z&urlSigningMethod=HMACV1&urlSignature=gbO%2Be6ji8njlwtpV7mcX87sxHkEn2Ak7VdOYlUatM1o%3D

2022-12-06T07:55:24.0549810Z    Compiling proc-macro-error v1.0.4
2022-12-06T07:55:25.6700050Z    Compiling num-bigint v0.4.3
2022-12-06T07:55:25.7502200Z    Compiling lock_api v0.4.9
2022-12-06T07:55:26.2127390Z    Compiling filetime v0.2.18

Sure, I don't look at the raw logs much, but if you think it's important we can close this PR.


Is this a correct update?

I wasn't 100% clear on what was being described, so trying to write something more explicit.

@justinegeffen Can you help review this PR for clarity and read-ability?  I think the content is there, but worried the way it is worded is not as clear as it could be.


Build fails from AUR on Antergos

glxinfo.txt
glxinfo egl.txt
meson-log.txt
pakku AUR helper terminal output.txt
all egl packages found with pakku.txt
all installed packages.txt
After going over some of the tickets in /sway and /wlroots, I have found that it would be most beneficial to provide as many details about my Antergos installation and about the build process and failure that could possibly be beneficial in any way concievable to me. Do correct me if I am mistaken on this note.
I have tried to install sway and wlroots from the AUR -git packages due to the binaries available being out-of-date. As seen in the attachments, pkg-config located all build dependencies up to egl, but neither pkg-config nor cmake found the relevant binary. I have tried installing all packages found when searching egl in pacman other than those with nvidia in their name, since I don't think the AUR packages are relevant, but I included them as well in one of the files just in case, which is also why I included a list of all of my installed packages as well.
To note, the Wayland sessions of KDE Plasma and GNOME 3 work just fine (KDE's implementation is subpar relative to GNOME's, but that's neither here nor there in the end).
If this issue has already been referenced with the solution somewhere else where it is easily found, please forgive me (this is my first time opening a ticket and I have been looking around a bit for the issue).
I thank you in advance and have a nice day and good luck with the project, whatever the outcome of this issue is!

So, the issue seems to be Package xfixes was not found in the pkg-config search path.. pkg-config requires xfixes for egl, so the terminal output given is that egl was not found, when, in fact, it's xfixes that is missing (even though the package is installed).

You could also try to install the missing xfixes dependency, but I'm not sure it'll fix your issue. It should definitely be a dependency of mesa in the Antergos package.

Well, libxfixes is installed, so that did not help.


homu.io has been dead

http://homu.io/ has been dead.

we shift to https://github.com/bors-ng/bors-ng

complete to shift to bors-ng


Installation

Can someone guide me in the process of installisng this and get it configured ? Using Orca Slicer.
I found it unclear. I've tried and need some step by step help, please.
May help others too, from what I read online, who struggle to implement this.
My setup is:
. Ender 3 S1
. Raspberry Pi 4 running klipper (KIAUH)
Thanks.

No idea about Orca, but here's cura plugin (with instructions): https://github.com/pedrolamas/klipper-preprocessor

No idea about Orca, but here's cura plugin (with instructions): https://github.com/pedrolamas/klipper-preprocessor

It's a bit different with Cura as it uses KlipperPreprocessor, and Klipper Estimator being an option of that plugin.
Thanks anyway.
Anyone can help ?

i need also help with getting started with Orca Slicer, have do the readme on the pi, where runs klipper and moonraker, but doesnt work for me, Cant run klipper_estimator. A introduction would be nice.
Thanks

Hi @pcout,
OrcaSlicer installation is the same as for PrusaSlicer and SuperSlicer.
Best regards,
Lasse


Add: Added my name to the list, my first contribution! Also fixed a typo

Pull Request Requirements

[x] I have followed the directions and format under contributing.md
[x] My username is under the correct alphabetical category
[x] I added my github link
[x] I added a personal goal for this year

Congrats and Thank you!

awesome thanks great job!


OSError: Caught OSError in DataLoader worker process 0.

Hi, I'm having this issue when running !python tools/train.py configs/faster_rcnn/faster_rcnn_r50_fpn_1x_coco.py in google colab:
index created!
2021-07-20 04:56:56,971 - mmdet - INFO - Start running, host: root@5526dcebfd7a, work_dir: /content/drive/.shortcut-targets-by-id/1QI1uqs44zWhCK8Zc8WK7JuesHC_QjWqw/MyFPN/work_dirs/faster_rcnn_r50_fpn_1x_coco
2021-07-20 04:56:56,972 - mmdet - INFO - workflow: [('train', 1)], max: 12 epochs
Traceback (most recent call last):
  File "tools/train.py", line 188, in <module>
    main()
  File "tools/train.py", line 184, in main
    meta=meta)
  File "/content/drive/.shortcut-targets-by-id/1QI1uqs44zWhCK8Zc8WK7JuesHC_QjWqw/MyFPN/mmdet/apis/train.py", line 170, in train_detector
    runner.run(data_loaders, cfg.workflow)
  File "/usr/local/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 125, in run
    epoch_runner(data_loaders[i], **kwargs)
  File "/usr/local/lib/python3.7/site-packages/mmcv/runner/epoch_based_runner.py", line 47, in train
    for i, data_batch in enumerate(self.data_loader):
  File "/usr/local/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 435, in __next__
    data = self._next_data()
  File "/usr/local/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 1085, in _next_data
    return self._process_data(data)
  File "/usr/local/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 1111, in _process_data
    data.reraise()
  File "/usr/local/lib/python3.7/site-packages/torch/_utils.py", line 428, in reraise
    raise self.exc_type(msg)
OSError: Caught OSError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 198, in _worker_loop
    data = fetcher.fetch(index)
  File "/usr/local/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/usr/local/lib/python3.7/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/content/drive/.shortcut-targets-by-id/1QI1uqs44zWhCK8Zc8WK7JuesHC_QjWqw/MyFPN/mmdet/datasets/custom.py", line 194, in __getitem__
    data = self.prepare_train_img(idx)
  File "/content/drive/.shortcut-targets-by-id/1QI1uqs44zWhCK8Zc8WK7JuesHC_QjWqw/MyFPN/mmdet/datasets/custom.py", line 217, in prepare_train_img
    return self.pipeline(results)
  File "/content/drive/.shortcut-targets-by-id/1QI1uqs44zWhCK8Zc8WK7JuesHC_QjWqw/MyFPN/mmdet/datasets/pipelines/compose.py", line 40, in __call__
    data = t(data)
  File "/content/drive/.shortcut-targets-by-id/1QI1uqs44zWhCK8Zc8WK7JuesHC_QjWqw/MyFPN/mmdet/datasets/pipelines/loading.py", line 59, in __call__
    img_bytes = self.file_client.get(filename)
  File "/usr/local/lib/python3.7/site-packages/mmcv/fileio/file_client.py", line 306, in get
    return self.client.get(filepath)
  File "/usr/local/lib/python3.7/site-packages/mmcv/fileio/file_client.py", line 184, in get
    with open(filepath, 'rb') as f:
OSError: [Errno 5] Input/output error: 'data/coco/train2017/000000123190.jpg'

Problem 1:
OSError: Caught OSError in DataLoader worker process 0.
Solution:
Set worker_per_gpu = 0
Problem 2:
OSError: [Errno 5] Input/output error: 'data/coco/train2017/000000123190.jpg'
Solution:
Colab cannot access all the coco images at once, what I did was just keep pressing the rerun button until all of the images are seen by colab.

If we set work_per_gpu=0; the efficiency of training is very slow,  can you tell me how to improve it, please?


Investigage leveraging Interactive Window for debugging

See if we should leverage the interactive window for debugging, or if the console window gives enough value.

Closing since the console window supports node, and performs this functionality


How to open NerdTree when a file is selected from Startify?

How can I open NerdTree when a file is selected from the Startify menu, but not anywhere else. For example,
autocmd BufEnter * NERDTreeCWD

won't work because it will open on every new buffer.

Try putting this in your vimrc:
autocmd User StartifyBufferOpened NERDTreeCWD

Thanks!


Catch the error when deleting an item with protected related objects

It catches django.db.models.ProtectedError at deletion, which is raised when there are protected related objects linked to the item being deleted. An error message is shown to the user with those linked objects. To ease the customization, an extra method format_linked_objects() is added.
Any feedback are welcome, thanks!

Codecov Report

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

@@          Coverage Diff          @@
##           master    #10   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          43     43
  Branches        7      7
=====================================
  Hits           43     43

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 4355b16...c3c2b3c. Read the comment docs.


Please hide archived pictures

I don't want my archived pictures showing on my screensaver.  Am I doing something wrong, or is that function not available?
Thanks!

I have the same issue.

This will be added to the backlog. It will show any photos in your account.

Closing issue. This has been fixed in Release 1.7 (https://github.com/chtaylo2/Roku-GooglePhotos/releases/tag/1.7)  Update has been pushed to Roku for publishing.

I'm also seeing archived photos in the screensaver. These photos aren't in any albums. I'm running Release v2.1 - Build 4.364.20180303
Otherwise, it's a great product.

I am also seeing archived photos in the screensaver. Is there a way to force a refresh after new photos are moved to the archive?


Update Request for resend dependency

Hey,
I've been setting up your resend plugin locally and am encountering an error described here. Since I'm getting the same ECONNREFUSED error, I can't really send emails.
Thank you for your work!

From that thread, it looks like it can be resolved by force updating the underlying resend-node dependency.  I'll officially update it in the next release.

This issue should now be resolved with plugin version 1.1.0, which uses the resend sdk package version 1.0.0.


Irmin pack inspector tool feedback

This issue aims to gather all the feedback/requests that might be done for the tool developed in this pull request once it is merged.

Some desired QOL enhancement:

If files already exist, take advantage of them, don't recompute.
Show offsets, not just entry indices
Move to children of node and commit
Prettier print than just Repr.pp for inode and commit. (e.g. decode dict indices, show children hash, child kind)
Move one entry at a time, not just per kind


import/resolver not working for webpack aliases in VSCode eslint

Hi, I'm using this plugin along with eslint in a project which as this folder structure:
project_root/
|_ server/
  |_ src/
  |_ .eslintrc
|_ client/
  |_ src/
  |_ .eslintrc
  |_ tools
    |_ webpack.config.js

I have defined some aliases in webpack like so:
/* client/webpack.config.js */
config = {
  resolve: {
    alias: {
        Common: path.resolve('src/common'),
        IP: path.resolve('src/apps/inv-planner'),
    },
    modules: ['node_modules', 'src'],
    extensions: ['.js'],
  },
}

This is my relevant eslint config:
    settings: {
        'import/resolver': {
            node: {
                moduleDirectory: ['node_modules', 'src'],
            },
            webpack: {
                config: `${__dirname}/tools/webpack.config.js`, // vscode doesn't resolve this
                // config: `./tools/webpack.config.js`, // vscode doesn't resolve this either
            },
        },
    },

I have opened the project_root folder in vscode, and all eslint rules work fine in VSCode, except for import/extensions, which keeps throwing errors for imports from the src/apps/inv-planner directory, but not from the src/common directory.
import AsyncRequests from 'IP/components/AsyncRequests'; // throws error
import Instrumentation from 'Common/extensions/instrumentation'; // doesn't throw

The eslint CLI works fine, but the vscode extension doesn't. Is there an issue in path resolution for non-root directories in this plugin?

There’s an issue in vscode’s repo itself, i believe. Something about it incorrectly setting the project dir.
Specifically, as long as it works on the CLI, the issue is in the IDE, and there’s nothing an individual plugin can do about it.

Okay, thanks for the pointers. I'll report this there


PHP 5.6.5 vs genghis v2.3.11

@bobthecow hi, I'm running genghis 2.3.11 with PHP 5.6.5, seeing below suggestion, adding 'always_populate_raw_post_data' to php.ini solves the problem. If you have time could you look into this? Just thought you might want to know.
Deprecated:  Automatically populating $HTTP_RAW_POST_DATA is deprecated and will be removed in a future version. To avoid this warning set 'always_populate_raw_post_data' to '-1' in php.ini and use the php://input stream instead. in Unknown on line 0

Warning:  Cannot modify header information - headers already sent in Unknown on line 0

Fatal error:  Uncaught exception 'Genghis_ErrorException' with message 'PHP warning:  Cannot modify header information - headers already sent' in ...

Hi! That warning has been fixed in the dev version of Genghis. It's here if you want to run it:
https://github.com/bobthecow/genghis/tree/develop
Otherwise, ignore the warning or set that flag in php.ini for now :)

It's working on the develop tree, will use this tree from now on. thanks for this app btw.

No problem :)


Blank screen with HTAN data

The interface shows for a bit and then the browser goes blank.
https://viewer.imaging.datacommons.cancer.gov/slim/studies/2.25.174442227011591161839224535127252003551/series/<IP_ADDRESS>.4.1.59<IP_ADDRESS>2175091.700331458.1655914616819.4.0
This message shows up in the developer console:
react-dom.production.min.js:189 Error: Pyramid of optical path "1" is different from reference pyramid.
    at O (viewer.js:1033:17)
    at new e (viewer.js:948:71)
    at ke (SlideViewer.tsx:111:24)
    at new n (SlideViewer.tsx:433:43)
    at Ba (react-dom.production.min.js:147:172)
    at Nu (react-dom.production.min.js:198:97)
    at xi (react-dom.production.min.js:292:172)
    at bs (react-dom.production.min.js:280:389)
    at gs (react-dom.production.min.js:280:320)
    at vs (react-dom.production.min.js:280:180)

Desired behavior would be at least an error message to the user and graceful handling.  Even better would be to handle this kind of data (although that may not be possible if the data is invalid for some reason).

Thanks @pieper for reporting the issue. We will need to investigate whether there is a problem with the data. However, I agree that the error handling will also need to be improved.

Errors that are thrown upon parsing of the data are difficult to catch/handle, because they happen deep in the library. It may be cleanest to introduce an error handler callback that allows the library to include a detailed message for known data issues that is explicitly intended for display to the user.

@hackermd since this is affecting IDC production viewer, it is a priority for IDC to at least know what is going on here. What can we do to expedite investigation of this issue?

cc: @Punzo

In general we need to improve the error handler in Slim.
I see 3 current issues regarding error handling
https://github.com/herrmannlab/slim/issues/106
https://github.com/herrmannlab/slim/issues/122
https://github.com/herrmannlab/slim/issues/123
We can talk the various specifics tomorrow, but in general error handler should not block the viewer. In addition, the error logger service should always report in a popup dialog or separate debug window (like we do for OHIF).
For customized error handler we can implement indeed callbacks that we can setup in the config file, e.g.: https://github.com/OHIF/Viewers/blob/master/platform/viewer/public/config/idc.js#L12
Anyway i have put this already in tomorrow agenda https://cornerstonejs.slack.com/archives/GP0DDMVMF/p1660725079247979

Regarding this issue: simply optical paths with different geometrical (e.g. orientation, spacing, etc) properties are not supported. We could add a registration phase, but it would be very expensive client side. Or it could be that simply the metadata are wrong, but then the viewer can't work if we can't trust the metadata.

@fedorov I have started to work on improving the error handling. As stated above, the viewer should definitely handle such errors more gracefully. Providing a meaningful error message to the user that contains the right level of detail will require some thought and development work.
To fix the underlying issue, we will also need to investigate the data, because the issue is most likely due to a problem with the data rather than a bug in the viewer. The viewer requires pyramids of different channels to have the same structure. That does not seem to be the case here.
Image with Optical Path Identifier "1":
=== Pyramid Level 0 ===
Total Pixel Matrix Rows/Columns: (29226, 33635)
Pixel Spacing: (0.000649983750406, 0.000649983750406)
Downsampling Factors: (1.0, 1.0)
=== Pyramid Level 1 ===
Total Pixel Matrix Rows/Columns: (14613, 16818)
Pixel Spacing: (0.001299928852712, 0.001299928852712)
Downsampling Factors: (2.0, 1.9999405398977286)
=== Pyramid Level 2 ===
Total Pixel Matrix Rows/Columns: (7307, 8409)
Pixel Spacing: (0.002599857705424, 0.002599857705424)
Downsampling Factors: (3.9997262898590393, 3.999881079795457)
=== Pyramid Level 3 ===
Total Pixel Matrix Rows/Columns: (3654, 4205)
Pixel Spacing: (0.005199097133154, 0.005199097133154)
Downsampling Factors: (7.998357963875205, 7.998810939357908)
=== Pyramid Level 4 ===
Total Pixel Matrix Rows/Columns: (1827, 2103)
Pixel Spacing: (0.010395722037524, 0.010395722037524)
Downsampling Factors: (15.99671592775041, 15.993818354731337)
=== Pyramid Level 5 ===
Total Pixel Matrix Rows/Columns: (914, 1052)
Pixel Spacing: (0.020781562209994, 0.020781562209994)
Downsampling Factors: (31.975929978118163, 31.972433460076047)
=== Pyramid Level 6 ===
Total Pixel Matrix Rows/Columns: (457, 526)
Pixel Spacing: (0.041563124419988, 0.041563124419988)
Downsampling Factors: (63.951859956236326, 63.944866920152094)

Image with Optical Path Identifier "0":
=== Pyramid Level 0 ===
Total Pixel Matrix Rows/Columns: (29226, 33635)
Pixel Spacing: (0.000649999976158, 0.000649999976158)
Downsampling Factors: (1.0, 1.0)
=== Pyramid Level 1 ===
Total Pixel Matrix Rows/Columns: (14613, 16818)
Pixel Spacing: (0.001299961303251, 0.001299961303251)
Downsampling Factors: (2.0, 1.9999405398977286)
=== Pyramid Level 2 ===
Total Pixel Matrix Rows/Columns: (7307, 8409)
Pixel Spacing: (0.002599922606502, 0.002599922606502)
Downsampling Factors: (3.9997262898590393, 3.999881079795457)
=== Pyramid Level 3 ===
Total Pixel Matrix Rows/Columns: (3654, 4205)
Pixel Spacing: (0.005199226919876, 0.005199226919876)
Downsampling Factors: (7.998357963875205, 7.998810939357908)
=== Pyramid Level 4 ===
Total Pixel Matrix Rows/Columns: (1827, 2103)
Pixel Spacing: (0.010395981549253, 0.010395981549253)
Downsampling Factors: (15.99671592775041, 15.993818354731337)
=== Pyramid Level 5 ===
Total Pixel Matrix Rows/Columns: (914, 1052)
Pixel Spacing: (0.020782080986767, 0.020782080986767)
Downsampling Factors: (31.975929978118163, 31.972433460076047)
=== Pyramid Level 6 ===
Total Pixel Matrix Rows/Columns: (457, 526)
Pixel Spacing: (0.041564161973534, 0.041564161973534)
Downsampling Factors: (63.951859956236326, 63.944866920152094)

Note that the Pixel Spacing is different although the Total Pixel Matrix Rows/Columns are the same (the Imaged Volume Height/Width are also different).
That information can also be found in the console:

@dclunie could you kindly help us debug these data sets?

Thank you @hackermd and @Punzo - let's wait to hear from David. To me, it makes sense to add some logic to the converter to detect this kind of issues at the time of conversion.

I agree @fedorov. Ideally, we would avoid these issues upon conversion. At the same time, we want the viewer to be as robust as possible and at least handle such errors gracefully.

Interesting - not sure why there is a 0.002 % difference between the pixel spacing values in the first place - I will look into this.
That said, it might be better if the viewer didn't try to compare floating point values exactly, as opposed to within a specified tolerance:
http://randomascii.wordpress.com/2012/02/25/comparing-floating-point-numbers-2012-edition/

the current eps tolerance is 1.e-6. We could make it less strict if needed.
https://github.com/herrmannlab/dicom-microscopy-viewer/blob/b8393862f3d23f7f3c750505b36873dba4e46a0f/src/utils.js#L421

Related to https://github.com/OHIF/Viewers/issues/2869

Perhaps a percentage rather than an absolute value might be better

@dclunie as @Punzo pointed out, we already use a tolerance for comparing floating-point values. We could choose a different tolerance (considering that many values are in millimeter unit and we need high precision). However, I don't think this issue is merely a problem with floating-point arithmetic. I tend argue that if images are supposed to have the same pyramid structure (and especially if they have the same Pyramid UID), then the values for Pixel Spacing shall - at least in theory - be identical between the two instances.  In fact, DICOM Part 3 Section A.1.2.30 Multi-Resolution Pyramid IE explicitly states:

Each layer is encoded as a separate DICOM Image, each of which has a uniform resolution (same value for Pixel Spacing (0028,0030)).

In my opinion, if rounding errors get introduced during processing (conversion, resampling, etc.), then it is the responsibility of the equipment that performs the processing and generates the DICOM data set to resolve any inconsistencies and ensure that constraints that are imposed by the standard (e.g., pyramid structure) are actually met. We need to make sure that all instances in a series have the same Series Instance UID, Frame of Reference UID, etc. We should apply the same rigor to the constraints of other information entities, including multi-resolution pyramids.
One could argue that a 0.002 % (~1 nanometer) difference doesn't matter in practice and should be tolerated, but problems like these are in my experience often indications of wider problems with a data set. I don't want to push back on improving the application logic (there is definitely room for improvement - in particular with respect to error handling and reporting), but we can't expect the viewer to always provide a workaround for problematic data.

I completely agree, and I will look into why it is occurring.
Practical experience with radiology 3D data over the years suggests though that having too small an epsilon causes problems. Since the viewer is not being used as a quality control tool, making it too demanding doesn't really help anyone, though in this case if it had just worked, we wouldn't have noticed the problem in the data.

Fully agreed! We will need to find the right balance between being strict enough for the app to function properly (including not introducing significant errors with spatial coordinates when creating annotations) and being tolerant enough not to just abort for every small inconsistency in the data. We wanted to be very strict by default to be on the safe side.
As discussed during the IDC earlier today, I would advise against changing the tolerance for the floating-point value comparison in general. We probably all agree that the two values are indeed different. Instead, I would suggest handling the fact that the values are different at a higher level. In this case, we could allow the value of Pixel Spacing to be different between instances by a certain tolerance, which is bigger than the tolerance that we use to determine whether two floating-point values are the same. I would suggest specifying that tolerance in millimeter (or micrometer) unit. We could also make that configurable either upon installation/deployment or at runtime (as suggested by @fedorov).
The main question is how tolerant do we want to be?

@dclunie I've started to refactor the code to provide a tolerance for the values of the Pixel Spacing and the Y/X Offset in Slide Coordinate System attributes. Now, I am wondering what tolerance we should use and realized that there are some complications.
The offset of the images in the slide coordinate system should generally be the same, but since the offset is (or shall be) specified with respect to the center of the top left pixel, the values may nevertheless vary between images. How much deviation should be expect/tolerate? The expected difference could in principle be computed exactly, but my guess is that a lot of implementations may get this wrong, for example by not updating the values when downsampling (not sure whether ours is even fully standard compliant in this regard?).
For Pixel Spacing, we may want a different absolute tolerance for each resolution level, because a small deviation at the top of the pyramid may translate to a very large deviation at the bottom of the pyramid.
What are your thoughts?

Has this been addressed by https://github.com/ImagingDataCommons/slim/pull/134? Need to review.


GDB reverse debug

Are there any plans to support GDB reverse debug?
reverse debug allows retracing the steps the program took to get to the current breakpoint. In order to do this one mus run record after start the program can than be analysed with reverse-step akin to step.
This GDB feature is (I think) similar to Microsoft's IntelliTrace in VisualStudio

It's already implemented since d704af2501168126f6979b2ac784d59d84b3fc98


Use lodash.mapvalues instead of whole lodash

Thanks @rodweb for this PR.
I don't know why TravisCI doesn't build it though it should for each PR.

Anyway, I can try merging so I can possibly revert if it fails.
I think lodash is such a popular library that more than 90% of projects already use it but if it matters to you to have smaller node_modules, I'm ok with it.

@rodweb
it's published in<EMAIL_ADDRESS>

@rodweb
it's published in<EMAIL_ADDRESS>
Thank you, sir!


Babel Deprecation (returning a string from a visitor method)

I've been seeing this on all of my integration tests lately.
[BABEL] myapp/tests/integration/components/my-integration-test.js:
Returning a string from a visitor method will be removed in the future.
String building is NOT a substitute for AST generation.
String building leads to terrible performance due to the additional parsing
overhead and will lead to extremely brittle transformers. For those extreme cases
where you're dealing with strings from a foreign source you may continue to use
`path.replaceWithSourceString(code)`. Please don't abuse this. Bad plugins hurt the ecosystem.

What version of ember-cli-htmlbars-inline-precompileare you using npm ls ember-cli-htmlbars-inline-precompile)? This should be fixed in v0.1.2 ...

This is still happening on v0.1.2 on my TravisCI build:
I added this to my .travis.yml
cat node_modules/ember-cli-htmlbars-inline-precompile/package.json
{
  "name": "ember-cli-htmlbars-inline-precompile",
  "version": "0.1.2",
   ...
}

and I'm still getting the error. Oddly enough, I'm not getting this when running my tests on OS X. I'm happy to do more debugging if that would be helpful.

Oh boi, I found typo on the PR which should have fixed this. replaceWithSourceString -> replaceWithStringSource. New version should be up in a few minutes. Thanks for your report!

Nice find! Thank you for the prompt fix :beers:

Okidok, v0.1.3 is released which should contain the fix. Can you give it a try an close this one if it has been fixed? :beers:

Coming up.
On Tue, Jun 30, 2015 at 2:09 PM Clemens Müller<EMAIL_ADDRESS>wrote:

Okidok, v0.1.3 is released which should contain the fix. Can you give it
a try an close this one if it has been fixed? [image: :beers:]
—
Reply to this email directly or view it on GitHub
https://github.com/pangratz/ember-cli-htmlbars-inline-precompile/issues/13#issuecomment-117326216
.

It's fixed with v0.1.3. Thanks again.

:+1:


Improve performance load shape

Hi dear
I have idea for improve shape world map performance
But i dont know is it best way or not if you have any better idea i need to do it
My idea : convert map1 to bitmap image and stop full drawing and show it until next map1 activity

Dear @ZimaSystem not sure that I understand what you mean. But you can try at first do these changes in your own fork. If they really improve performance, please submit them as pull request. We'll review it, and if everything fine then we merge it into repository. Also, before submitting PR, please read this link.

Okey thank you for your response , i try to write it ...

@ZimaSystem do you have plans to do something on this issue? Otherwise I'll close it.


[Exam] Ability to take exercise dates from exam

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[ ] Server: I added multiple integration tests (Spring) related to the features
[ ] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[ ] Server: I implemented the changes with a good performance and prevented too many database calls
[ ] Server: I documented the Java code using JavaDoc style.
[ ] Client: I added multiple integration tests (Jest) related to the features
[ ] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[ ] Client: I documented the TypeScript code using JSDoc style.
[ ] Client: I added multiple screenshots/screencasts of my UI changes
[ ] Client: I translated all the newly inserted strings into German and English

Motivation and Context

This simplifies creating exercises for exams, because instructors can just copy the necessary dates from the exam with the click of a button
Description

Steps for Testing

Log in to Artemis
Navigate to Course Administration
...

Screenshots

Closed. Other approach was decided upon in #1865


Peer-to-peer code review

Nice project implementation Ceejay 🎉✔💪
Keep it up!
Cheers
Elisha

thanks


Documentation for @analogjs/vitest-angular is using imports from @analogjs/vite-plugin-angular

Please provide the environment you discovered this bug in.
Docs: https://github.com/analogjs/analog/blob/11577aa08794b547ac0dd504859c2484dd3fd627/packages/vitest-angular/README.md
Probably also https://analogjs.org/docs/features/testing/vitest
Which area/package is the issue in?
vitest-angular
Description
While upgrading my (plain Angular) project from @analogjs/vite-plugin-angular to @analogjs/vitest-angular I noticed that the docs for @analogjs/vitest-angular contain
import angular from '@analogjs/vite-plugin-angular';

export default defineConfig(({ mode }) => ({
  plugins: [angular()],

which should probably be
import { vitestBuilder } from '@analogjs/vitest-angular';

export default defineConfig(({ mode }) => {
  return {
    plugins: [vitestBuilder],

I guess that also https://analogjs.org/docs/features/testing/vitest should be migrated to use  '@analogjs/vitest-angular', at least for the 'manual installation' section?
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

This is not a bug but setup as intended. vitest-angular is a builder that lets you run the Vitest runner. The @analogjs/vite-plugin-angular package is the plugin that handles Angular compilation.
So you still need both. We do need to update the guide the use vitest-angular:test instead of @analogjs/platform:vitest in the angular.json

Oh, this makes so much more sense and I got it to work now. Thank you for the quick answer and sorry for creating a bug.


Prevent race condition

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
This PR is work in progress.
This fixes following problems of #1888 :

[x] race condition on cache mode
[ ]  race condition on non-cache mode

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.03%) to 76.237% when pulling a260842e9a8ee89f859b3a320e87bc5060da8405 on node-red-hitachi:avoid-race-condition into 66ee27c5fa054d5f1269d91f0bbaf6b6e0e4c41d on node-red:master.

I need a bit more time to review this. I can see what it's doing, but I think there may be a cleaner way to achieve it - but I got it wrong last time, so need to be extra careful this time.

Ok - I've merged this change, then made the changes I've described above in 9777af7cb5f0eca4767cef8ad7820b6ce4b2b288
I am fairly sure this is a good change - I have not been able to reproduce the original error with it, but would appreciate your review and testing.

@knolleary
I am sorry for my late reply and thank you for your review and fix.
As you commented, the above changes were unnecessary. these were my mistakes.
Thank you.


Insert  element when converting from text to HTML

When converting text to HTML replace what looks like horizontal lines drawn with ASCII characters with <hr>.
Examples:
text
---------------------------
some other text
===========================
more text
-=-=-=-=-=-=-=-=-=-=-=-=-=-
scissors below
-- >8 --
other direction
-- 8< --
end

I'm not quite happy with the approach in PR #473. But that's the functionality we want.

Hey guys! I'd like to begin by taking up this bug (applying for GSoC!), it'd be great if someone could guide me around with it!

I'd like to work on this enhancement. At the moment I'm trying to implement it using regex.

This should be straightforward to implement with a regex.
I would definitely use TDD for this (write the unit-tests first)!

@cketti I've implemented this. Please check PR #2300 .


MTC - preliminary error handling functions

Added skeleton functions for error handling; processErrorMessage, processEmergencyMessage, processSdoMessage, processNmtMessage.
Implementing error codes and their description from https://www.emdrive-mobility.com/portfolio/emdrive-500/ controller.
Logging will be fixed when logging is updated on master, just being used as a placeholder.
Working with @TristCrocker
WIP

Another more in general thing, we wouldn't want to be logging things like "ERROR_DRV_HS_L1" or "ERROR_DRV_HS_L2".
Log messages should be clear enough that we immediately know what's happening instead of needing to check the datasheet/some other manual

I would like the error to stay in the log message, though. This makes it easier to understand whats going on with respect to the manual.

The most recent commit does a lot more than just merging...

Going to recommit with a more explicit explanation on the changes made

Just as a note, "some changes implemented" is a very unhelpful commit message

Seems like you added _deps/googletest-src by accident. please remove again.


Wavmix for the ESC50 dataset

Hello, thanks a lot for you amazing work and for publishing the code!
I was trying to run the ex_esc50.py with wavmix=True but got the error:
RuntimeError: "nll_loss_forward_no_reduce_cuda_kernel_index" not implemented for 'Double'

since when using wavmix the ground truth is not an integer anymore.
Would it not be more appropriate to use the KL-divergence as loss function instead of the crossentropy?

Hi, Thank you! yes you can use loss functions that accept vectors instead of integers, Here is CE implementation.
For this to work you need to convert the targets from an integer to onehot vector here


ADM-350 T5E - [frontend] [Config] - Pipeline tool section

Summary
BuildKite for pipelineTool is default value when required data set as Lead time for changes, Deployment frequency, Change failure rate, Mean time to recovery
GoCD is another option for pipelineTool
token field is empty and required flag is *
Given correct token of pipeline and click Verify button, it has loading effect
It has Verified and Reset buttons once Verify success
Before
Description

After
Description

Note
Null

All review status and fixed feedback have already shown into this PR #172, will close this PR due to this branch will not be used.  @guzhongren


Tracker doesn't record pages loaded via View Transitions

This is visible on my website. If you click "Blog", a view transition happens, the content is fetched from the server, then that content is smoothly added to the page (it doesn't look like a full page load took place, but it did).
In the network inspector, you'll notice there's no call to /collect, so Counterscale doesn't capture the page view. But it should be recorded.
(Note my blog is using Astro, so there's a possibility this is an Astro thing.)

Probably related to #83


Integrate es6 transpiler like Babel or Typescript

To build #217 and extendible to whole code base to avoid delays of implementation on es6 features (yeah we still support Node4,5 and 6)
(it's gonna be needed by the UI, why don't use it for back-end and take advance of ES next features).

Why not compile before publish it to npm? I don't think bundle compiler/transpiler with verdaccio is a good idea...

Yeah, that is the point, compile and then publish ES5.

Good. I think i can refactoring current code (logger) to template string.

Yeah, please check the branch refactor-ui, there is a extreme make over of all api endpoints

Thanks for the help I really appreciate it :)

Babel 6 was the winner. Closing.


Clean readme

This cleans the readme file, and adds an issue template for people who want to request their game be added.
I think there are some improvements that could be made with some of the wording of the issue template.

Looks good! My only thought is that it might be good to have a template for the other categories as well? Or have one template with a drop down for what type of item you're submitting?
Some more info that may be worth working into the comments:
The TOML schema is the same for all of the categories, the main difference is just that ecosystem entries usually use the source field to automatically pull various bits of data from Crates.io or Github. Specifically, it will autofill:

repository_url
crate_url (crates source only)
homepage_url (if they've set that on the crate/repo)
description

This can be overridden with the explicit options, though.
Also, we generally don't have images for crates/resources, as they don't look great in the single column layout those pages use.


Provide a TestNGListener that can be invoked when a test fails due to a timeout

TestNG Version

Note: only the latest version is supported

Expected behavior
TestNG provides a new timeout listener which can be implemented and injected into TestNG by the user and which would get invoked whenever a @Test method times out due to the time-out attribute specified in the @Test
Actual behavior
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample

Please, share the test case (as small as possible) which shows the issue

More details are available in https://stackoverflow.com/questions/53280681/bind-a-method-with-time-out-event-in-testng

ping @juherr - WDYT ?


Dead links in docs

title: Mesh Picking
image:
description: Learn how to pick meshes in Babylon.js.
keywords: diving deeper, meshes, interactions
further-reading:
- title: Mesh Overview
url: /divingDeeper/mesh/creation/set
- title: Forum Pick with Ray
url: https://www.html5gamedevs.com/topic/26503-scenepickwithray-blues/
- title: Forum World Local Ray
url: https://www.html5gamedevs.com/topic/26602-worldlocalray/
- title: Forum Gobal to Local
url: https://www.html5gamedevs.com/topic/7599-convert-global-coordinates-to-local-coordinates/
video-overview:
video-content:
All of the html5gamedevs.com links do not work.

@PirateJC - we need to remove all of the html5gamedevs links, as they are all offline

This is being addressed with the following PR:
https://github.com/BabylonJS/Documentation/pull/622


Update to Java 17 and retire Java 11 tagged images

Notes for Reviewers

[ ] The commit history must be preserved - please use the rebase-merge or standard merge option instead of squash-merge
[ ] Sync up with the author before merging

With 8 and 11 dropped there could be some minor cleanup in docker-entrypoint.sh.  UseParNewGC was removed in 10 and UseConcMarkSweepGC was removed in 14.
https://github.com/Graylog2/graylog-docker/blob/5.0/docker-entrypoint.sh#L11-L22

Thanks for the reminder @coffee-squirrel :+1: Much appreciated!

Refs: https://github.com/Graylog2/graylog-plugin-enterprise/issues/3997

This has been superseded by other commits.
The last missing part was moved to https://github.com/Graylog2/graylog-docker/pull/227


drop package names in functions

Currently, the list of function names supported in expr/functions.go uses a package-style naming convention, i.e., pkgname.func().  Until and unless we support a package import concept in zql this seems clunky and like overkill, and it's burdensome to have to type the package name.
Perhaps this is a bikeshed topic, but the proposal here is to drop the package name prefix from our library of function names, e.g., Strings.tolower() becomes tolower(), Math.log() becomes log() etc.

We should also

unify the different variants of length() into a single len() function
get rid of functions that are deprecated with :casts
use snake_case

I'll plan to write the ZQL user docs for these soon, tracked in #1621.


Running tests at the REPL

With the Cognitect test runner, you can run all your tests while via the REPL by simply doing
(require '[cognitect.test-runner])
(cognitect.test-runner/test {})

I was wondering if the same could be done with cljs-test-runner.
An experiment shows that it is probably possible via a macro.
https://github.com/mfikes/cljs-test-runner/tree/repl
The output of this experiment is below.
Before delving into this too far, wanted to share and see what your thoughts are.
Mikes-iMac:cljs-bean mfikes$ clj -A:repl/node
ClojureScript 1.10.520
cljs.user=> (require 'cljs-bean.core-test)
nil
cljs.user=> (require 'cljs-test-runner.test-runner)
nil
cljs.user=> (cljs-test-runner.test-runner/run-tests {})
WARNING: Use of undeclared Var doo.runner/set-entry-point! at line 31 cljs-test-runner-out/gen/cljs_test_runner/gen.cljs
WARNING: Use of undeclared Var doo.runner/karma? at line 31 cljs-test-runner-out/gen/cljs_test_runner/gen.cljs
WARNING: Use of undeclared Var jx.reporter.karma/start at line 31 cljs-test-runner-out/gen/cljs_test_runner/gen.cljs

Testing cljs-bean.core-test
{:result true, :num-tests 100, :seed<PHONE_NUMBER>726, :time-elapsed-ms 262, :test-var "roundtrip-1"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>989, :time-elapsed-ms 144, :test-var "roundtrip-2"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>133, :time-elapsed-ms 102, :test-var "roundtrip-3"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>235, :time-elapsed-ms 91, :test-var "roundtrip-4"}

Ran 72 tests containing 510 assertions.
0 failures, 0 errors.
0

Oh that's really cool! I'm totally okay with this, it's a small change and it looks like it'll be a great addition for some people. Namely you :smile:
If you'd like to submit a PR for this I'll give it a once over, I can't see why this can't be part of the main project though, it's not really bloat. Great work, as always!

@Olical Thanks. Since you are cool with this, I'll put together a proper PR for review.

Hey Mike, just wondering if you ever had this code working? I'd love to see what you were thinking about even if it doesn't work? I can maybe pick this torch up eventually.

@Olical Indeed there was some issue that I encountered when trying to solve this that was difficult to overcome. I failed to record it but I'll see if I can find the original branch and re-construct what the problem was.

@Olical OK, I took a look to see where I had become stuck. The experimental branch where I was progressing at the time is https://github.com/mfikes/cljs-test-runner/tree/repl
One salient aspect of the design being pursued there is that, since you are already in an established REPL, there is no need to involve doo to set up a ClojureScript / JavaScript runtime within which to run tests: In the REPL case, all that is needed is to (a) dynamically generate the synthetic namespace as cljs-test-runner already does, and (b) to simply require the synthetically generated namespace.
Another salient aspect is that I was ignoring for the time being any concerns surrounding self-hosted ClojureScript, and instead just focusing on trying to get conventional JVM ClojureScript working.
The macro in that branch is set up to do parts (a) and (b) as described above. The part where I was getting stuck was (b): The macro essentially expands into a require form:
(cljs-test-runner.test-runner/run-tests {})

would, essentially expand to be
(require 'cljs-test-runner.gen :reload)

In fact, if you were to issue that particular require in a REPL, the tests would run. But they don't and instead you get an error. This error is easy to reproduce independently of this project:
If you were to evaluate (require 'clojure.set) but instead first look at its macro expansion and instead directly evaluate its expansion: (ns* (:require 'clojure.set)) you will see an error. Digging a little, the reason is that if you essentially evaluate an ns* form in the REPL, the Clojure implementation of the REPL first wraps the entire form in a construct that will set *1, handle exceptions, etc., and then evaluate that wrapped form. The problem with this is that the resulting ns* form is no longer a top-level form, but instead is buried inside the wrapping form. And since it is not a top-level form, something down in the ClojureScript analyzer or compiler is not happy.
You can easily cause the same error directly in a REPL if you, for example supply your own wrapping: (do (require 'clojure.set))
This is essentially what derails the attempt to evaluate (cljs-test-runner.test-runner/run-tests {}).
Now, given this, the REPL is hard-coded to know not to wrap a form for which the first symbol is require, amongst other symbols. If you revise the REPL code to do the same for the symbol cljs-test-runner.test-runner/run-tests, you can get things to work.
Here is an example:
cljs.user=> (require 'cljs-test-runner.test-runner)
nil
cljs.user=> (cljs-test-runner.test-runner/run-tests {})

Testing cljs-bean.core-test
{:result true, :num-tests 100, :seed<PHONE_NUMBER>223, :time-elapsed-ms 298, :test-var "roundtrip-1"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>522, :time-elapsed-ms 145, :test-var "roundtrip-2"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>667, :time-elapsed-ms 133, :test-var "roundtrip-3"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>801, :time-elapsed-ms 163, :test-var "roundtrip-4"}
{:result true, :num-tests 100, :seed<PHONE_NUMBER>964, :time-elapsed-ms 170, :test-var "roundtrip-5"}

Testing cljs-bean.from.cljs.core-test

Testing cljs-bean.transit-test

Ran 111 tests containing 735 assertions.
0 failures, 0 errors.
nil
cljs.user=>

So, now knowing the root of the problem, the challenge is to devise some sort of solution for it. Perhaps instead of having the macro evaluate to a require form, it could make calls upon the ClojureScript compiler to attempt to cause the same side effect, or maybe there is some clever way to set the :wrap option for REPLs so that cljs-test-runner.test-runner/run-tests or run-tests is detected to be this particular macro and to not wrap it.
So that's where things sit now.
If you wanted to try hacking on it some to see if progress can be made, let me know.
If you decide to go down that path, feel free to take my branch with its experimental changes as a basis for hacking---you can see evidence of the things I was trying and some commented out code and bits that are hard coded in places.
Otherwise, I may get back to this at some point.
(I do feel that this feature would be a really nice thing to have in general---you often want to run all of your tests while at the REPL.)


Sui Code Bug or Feature Request sui wallet zero balance

Steps to Reproduce Issue
sui wallet error message coins disappeared now zero balance
https://ethoswallet.github.io/2048-demo/
Further to my previous bug i  have now achieved higher score but when i went to mint and link to sui wallet its taken all coins in wallet  and come up with this error message which was over 2million coins now zero balance which were in wallet and i now got this error message in wallet zero balance and no minted achievement from high score  please see attached photos
e.g.

Call function .
Use return value and call .

Expected Result
Specify what outcome you expected should have resulted, but didn't.
minted reward :) for high score
e.g.
Expected  to return 42.
coins still in wallet
Actual Result
Specify what the actual unexpected outcome was.
error message and now zero balance of coins
e.g.
 returned 41.
System Information

OS:
Compiler:
![20221027_184842](https://user-images.githubusercontent.com/116558527/198364264-5f298ce7-474e-4cec-bc8d-6
![20221027_184851](https://us

er-images.githubusercontent.com/116558527/198364290-e96169ef-2cce-4d56-a9a6-a0c1d17e62f3.jpg)
9b0134db83b.jpg)

iv just added more coins but all the nfts have disappeared from wallet aswell :( and i got error code on that nft i just minted :) but is showing in wallet but others have gone i hope i am putting these in right place for you as im not a programmer :( please advise if i should be putting these bugs elsewhere kind regards starlew77<EMAIL_ADDRESS>

Duplicate of #5813.


Describe extension benefits and features in a specific homepage

https://lokalise-document-preview.terales.info/


Start to add support for Electra

This PR makes the necessary changes (adds new structures and updates versioned structures) for Electra. Sorry, it contains a lot of new files and it lacks tests. I would like to get pull in these changes into an electra branch, so that multiple people (myself included) can add tests later, in smaller PRs.
After this PR gets merged, I'll make the equivalent PR for go-builder-client. We need to wait because go-eth2-client is a dependency and it currently needs a replace directive.

I added two nolint:gocyclo comments for these:
spec/versionedsignedbeaconblock.go:272:1: cyclomatic complexity 31 of func `(*VersionedSignedBeaconBlock).Attestations` is high (> 30) (gocyclo)
func (v *VersionedSignedBeaconBlock) Attestations() ([]VersionedAttestation, error) {
^
spec/electra/beaconblockbody_json.go:71:1: cyclomatic complexity 31 of func `(*BeaconBlockBody).UnmarshalJSON` is high (> 30) (gocyclo)
func (b *BeaconBlockBody) UnmarshalJSON(input []byte) error {
^

Not really sure how I would make these less complex.

The Electra changes themselves look good.
There are a fair number of non-Electra changes in here.  Would it be possible to remove these from this PR, to avoid additional pain when we come to merge this branch back into master?  Anything non-cosmetic can be pushed to a separate PR to be included directly into master.  Thanks.

Yes, good suggestion. I'll make those changes first thing tomorrow morning.

@mcdee Could you fix the linter issues for me? I'm not sure why there are magic number findings now.


Fix prompt for lambda role arn

Describe your changes and why you are making these changes
Fixes prompt for lambda role arn
Related issue number (if any)
Loom demo (if any)
Checklist before requesting a review

[x] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[x] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@hsubbaraj-spiral, what's the status of this PR?


[Bug] Incorrect org name searched

Is there an existing issue for this?

[X] I have searched the existing issues

What product(s) are you seeing the problem on?
OpenChallenges
Current behavior
This org appears in very first search results of organization dropdown filter. It seems it displays the member information , other than the name of organization.

Expected behavior
Display the name of organization.
Anything else?
No response
Commit ID
No response
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for the catch, @rrchai !
Looks like the organizations sheet was listing member information for some of the rows.  These rows have been updated to list the organization info only.
CC: @gaiaandreoletti


Compiling razor project with GetCompilationAsync not running the source generators on .NET SDK 7.0.4

On .NET SDK 7.0.307 I was able to use GetCompilationAsync to compile razor projects, however after updating to .NET SDK 7.0.4, it does not work anymore.
The issue might be related to this Roslyn issue, but I am not sure.
As I have described on that issue after updating to v7.0.4 a workaround needs to be applied to be able to load the project successfully. After loading the project successfully, the compilation succeeds too, but the razor-generated files cannot be seen in the syntax trees.

I'm having issues with GetCompilationAsync to show razor files.
I have the following code
var solution = await workspace.OpenSolutionAsync(solutionFile.FullName);
var project = solution.Projects.FirstOrDefault(x => x.Name?.EndsWith(projectName, StringComparison.OrdinalIgnoreCase) ?? false);
var compilation = await project.GetCompilationAsync();
foreach (var document in project.Documents.Where(x => x.FilePath?.EndsWith("*.razor.cs", StringComparison.OrdinalIgnoreCase) == true))
{
                var syntaxTree = await document.GetSyntaxTreeAsync();

                if (syntaxTree is null)
                {
                    continue;
                }

                var rootNode = await document.GetSyntaxRootAsync();

                if (rootNode is null)
                {
                    continue;
                }

                var semanticModel = compilation.GetSemanticModel(syntaxTree);

                var symbol = semanticModel.GetDeclaredSymbol(syntaxDeclaration);

                if (symbol is null)
                {
                    WriteError("Could not find symbol for " + syntaxDeclaration);
                    continue;
                }

                ITypeSymbol? variableType = symbol switch
                {
                    IPropertySymbol propertySymbol => propertySymbol.Type,
                    IFieldSymbol fieldSymbol => fieldSymbol.Type,
                    _ => null,
                };

                var isComponentType = variableType.AllInterfaces.Any(x => x.Name.Equals("IComponent"));
}

My expectation is for a interface called IComponent to exist in a blazor application, and the source generator to have run.
In 7.0.3 this is the outcome, in 7.0.4 I get no interfaces in the AllInterfaces

@jasonmalinowski @chsienki I assume you two are the experts here, not sure if this is a deliberate change, but it seems odd that multiple people have started hitting it today.

We had another report of 7.0.400 breaking things due to the NuGet.Frameworks version problem, so that could be it here. That only shipped last week I think. @glennawatson: if you add an event handler for MSBuildWorkspace.WorkspaceFailed, is that being raised with an error while you call OpenSolutionAsync?

What version of Microsoft.CodeAnalysis.* are you using?
Razor in 7.0.400 has a hard dependency on the 4.7 version of the compiler, so won't load if you're using 4.6.0.
I was able to repro using the above code when running on 4.6.0 but everything works as expected if you upgrade the code analysis packages to 4.7.0-2.final (note: that's pre-release because VS 17.7 isn't out yet, but is needed for the razor source generator in 7.0.400)

@chsienki using 4.6.0. Will try the prerelease version today.

@jasonmalinowski As an aside, we (the compiler side) are raising an AnalyzerLoadFailed event in this case with the 'this analyzer needs a newer compiler message', but it doesn't look like its handled by the workspace code that's calling into it.

@jasonmalinowski i didn't see anything unusual in the workspace failed events.

@chsienki Oh ouch, that's not good; we should probably raise that somwhere. Also:

note: that's pre-release because VS 17.7 isn't out yet, but is needed for the razor source generator in 7.0.400

We shipped 17.7 last week, looks like we're still working on shipping packages though.

I can also confirm that targeting 4.7.0-2.final version of Microsoft.CodeAnalysis.*  fixes the issue. So, I guess this issue can be closed.
However, it would be nice to have some sort of indicator when this sort of dependency problem happens, instead of silently skipping the code generation.
Also, I am not super happy with the fact that I had to reference a prerelease version of a nugget package to make it work.

The 4.7.0 packages should be out soon, there's a bit of a hangup due to some dependency issues we're having to sort out.

However, it would be nice to have some sort of indicator when this sort of dependency problem happens

Agreed; let's retitle this bug to reflect that fact!


Let small changes to the source code take effect in running container

Is it a duplicate question?
No, to the best of my knowledge.
Describe your problem
I am using the Docker version of uptime-kuma, and want to make a small change to the UI. So I exec into the running container and I modified the src/pages/DashboardHome.vue file. But nothing changed in my browser.
I have tried to switch browsers (to avoid cache), to restart the running container.
Is there a specific way for me to make the change effect? Thanks!

Great point! The whole /src folder can probably be removed as well. But at this point most of the image size is in node_modules, whereas /src is just ~1MB, so the benefit is marginal.


Error when filter result from outer join more than 2 table

CrateDB version: 1.0.5
JVM version: 1.8.0_92
OS version / environment description: CentOS 7
Problem description:
Error when filter result after outer join more than two table.

SQLActionException[SQLParseException: Couldn't create executionContexts from
NodeOperations: {2=NodeOp{ phase{id=2/nested-loop, nodes=[88GNmORSQ-myfx6d0D6APw], dist=BROADCAST}, downstreamNodes=[88GNmORSQ-myfx6d0D6APw], downstreamPhase=5, downstreamInputId=0}, 0=NodeOp{ phase{id=0/collect, nodes=[88GNmORSQ-myfx6d0D6APw], dist=BROADCAST}, downstreamNodes=[88GNmORSQ-myfx6d0D6APw], downstreamPhase=2, downstreamInputId=0}, 1=NodeOp{ phase{id=1/collect, nodes=[88GNmORSQ-myfx6d0D6APw], dist=SAME_NODE}, downstreamNodes=[88GNmORSQ-myfx6d0D6APw], downstreamPhase=2, downstreamInputId=1}, 6=NodeOp{ phase{id=6/fetchPhase, nodes=[88GNmORSQ-myfx6d0D6APw, Xjr6Sx8UTK-6_c7B7bSTtA]}, downstreamNodes=[], downstreamPhase=2147483647, downstreamInputId=0}}
Leafs: [NestedLoopPhase{executionPhaseId=5, name=nested-loop, joinType=LEFT, joinCondition=op_=(InputColumn{index=2, type=integer},InputColumn{index=6, type=integer}), outputTypes=[integer, integer, string, integer, string, integer, string], jobId=0d998bac-dd27-48b0-9f20-665a3fd10d19, executionNodes=[88GNmORSQ-myfx6d0D6APw]}]
target-sources: {2=[0, 1], 5=[2]}
original-error: Can't handle Symbol RELCOL(join.join.a.b.c, 4)]

Steps to reproduce:
 CREATE TABLE IF NOT EXISTS "db"."order" (
 "order_id" STRING ,
 "customer_id" INT,
 "company_id" INT
 )
 CLUSTERED INTO 1 SHARDS;

  CREATE TABLE IF NOT EXISTS "db"."mt_customer" (
 "customer_id" INT ,
 "customer_name" STRING
 )
 CLUSTERED INTO 1 SHARDS;

  CREATE TABLE IF NOT EXISTS "db"."mt_company" (
 "company_id" INT ,
 "company_name" STRING
 )
 CLUSTERED INTO 1 SHARDS;

insert into "db.order" ("order_id", "customer_id", "company_id") values ('ORDER#1', 1, 1);
insert into "db.mt_customer" ("customer_id", "customer_name") values (1,'customer_1');
insert into "db.mt_company" ("company_id", "company_name") values (1,'company_1');

Query:
[FAILED]
select * from ("db.order" a left join "db.mt_customer" b on a.customer_id = b.customer_id) left join "db.mt_company" c on a.company_id = c.company_id where b.customer_id = '1';
[SUCCESS]
select * from "db.order" a left join "db.mt_customer" b on a.customer_id = b.customer_id where b.customer_id = '1';

Thanks for reproting, we'll take a look at this.

this is also what we need

The issue has been fixed and will be available with the upcoming 1.1.3 stable release.

I'm trying the queries of DLT1412 in Crate 1.2.0 and got the same error. Isn't this fixed in 1.2.0 yet?


Fixed passing style to DrawerNavItem

Fixes # BLUI-3120

Changes proposed in this Pull Request:

Added style to DrawerNavItem props

To Test:

yarn start:showcase

You still need to fix the issue of passing in a nav item object with sx or style definitions via the items prop on a DrawerNavGroup component.

I would like to see a description of this issue as well as test instructions too.

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

eed to fix the issue of passing in a nav item object with sx or style definitions via the items prop on a DrawerNavGroup componen

Tried following style and sx prop

It shows below output

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

We checked on @ektasawant 's machine as well, it is working as expected. @emclaug2 @jeffvg Can you please verify on your systems (please clear cache ) and checkout bug/3120-drawer-nav-item-example-style-fix in react-showcase-demo to see drawer nav item has red background?

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

We checked on @ektasawant 's machine as well, it is working as expected. @emclaug2 @jeffvg Can you please verify on your systems (please clear cache ) and checkout bug/3120-drawer-nav-item-example-style-fix in react-showcase-demo to see drawer nav item has red background?

Thats because I made a commit to fix it 🤦♂

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

@bkarambe, the typescript error is gone, but the styles still aren't applied via the items prop on a DrawerNavGroup component.

We checked on @ektasawant 's machine as well, it is working as expected. @emclaug2 @jeffvg Can you please verify on your systems (please clear cache ) and checkout bug/3120-drawer-nav-item-example-style-fix in react-showcase-demo to see drawer nav item has red background?

Thats because I made a commit to fix it 🤦♂

Sure. Thanks

looking at this now...

general question....I noticed that some styles are listed twice like this..why?

Using the same example as @ektasawant this works
But when I try using like this, the backgroundColor is applied and the navItem root style isn't applied
Also when checking out other sx overrides, I ran into the BluiDrawerNavItem-active not working because its a reserved global slot name. So I tried out the alternative .BluiDrawerNavItem-titleActive and that worked for me and we don't have that documented in the sx overrides table in the doc

also not able to apply styles to BluiDrawerNavItem-chevron
BluiDrawerNavItem-expandIcon seems to be good.

Using the same example as @ektasawant this works
But when I try using like this, the backgroundColor is applied and the navItem root style isn't applied
Also when checking out other sx overrides, I ran into the BluiDrawerNavItem-active not working because its a reserved global slot name. So I tried out the alternative .BluiDrawerNavItem-titleActive and that worked for me and we don't have that documented in the sx overrides table in the doc

also not able to apply styles to BluiDrawerNavItem-chevron
BluiDrawerNavItem-expandIcon seems to be good.

Passing '& .BluiDrawerNavItem-root' in sx won't work, because it is a root component and should be written without a space ( '&.BluiDrawerNavItem-root' ) which will work.

BluiDrawerNavItem-active : Updated the Drawer.md

For BluiDrawerNavItem-chevron, one prop chevron should be true. I tried it and working fine:

sx:{backgroundColor: '#ca3c3d', '& .BluiDrawerNavItem-chevron': {mt: '10px'}},
chevron: true,
chevronColor: 'blue',
<img width="1792" alt="Screenshot 2022-06-14 at 3 06 23 PM" src="https://user-images.githubusercontent.com/25982779/173550883-1bba7c58-43ed-46b7-aefb-97e8badfff0b.png">

should BluiDrawerNavItem-active be renamed to BluiDrawerNavItem-titleActive in the doc?
currently DOM shows two BluiDrawerNavItem-titleActive (no BluiDrawerNavItem-active)

should BluiDrawerNavItem-active be renamed to BluiDrawerNavItem-titleActive in the doc?
currently DOM shows two BluiDrawerNavItem-titleActive (no BluiDrawerNavItem-active)

active and titleActive are two different classes applied on different elements. We can't replaced it with BluiDrawerNavItem-titleActive


?  not fount TDR registery entry in windows 10 ...

[Enter feedback here]

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

ID: 652e03eb-a834-6c30-44c1-e9c917a94926
Version Independent ID: bca54aa8-2da8-e389-aa74-f66bf3ab65ab
Content: Testing and debugging TDR - Windows drivers
Content Source: windows-driver-docs-pr/display/tdr-registry-keys.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

i  add the two  missing entry in regedit , that's seem ok  :
[HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\GraphicsDrivers]
"TdrDelay"=dword:0000003c
"TdrDdiDelay"=dword:0000003c

i  add the two  missing entry in regedit , that's seem ok  :
[HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\GraphicsDrivers]
"TdrDelay"=dword:0000003c
"TdrDdiDelay"=dword:0000003c

@rogermachin
this article  author is cleared  included note
These registry keys should not be manipulated by end users, or by applications outside of targeted testing or debugging during driver development.
If you are end-user of windows 10, then you can add the registry keys for debugging process, not the regular process.

@rogermachin
this article  author is cleared  included note
These registry keys should not be manipulated by end users, or by applications outside of targeted testing or debugging during driver development.
If you are end-user of windows 10, then you can add the registry keys for debugging process, not the regular process.

@lorihollasch , I  verified the article  in the edit process, and i found One  table  format is  included in source content , but that table is not reflected in public view in website

@lorihollasch , I  verified the article  in the edit process, and i found One  table  format is  included in source content , but that table is not reflected in public view in website

@e0i  @nam31   @lorihollasch, please  verify PR #2655 and approve if it correct.
https://github.com/MicrosoftDocs/windows-driver-docs/pull/2655

@e0i  @nam31   @lorihollasch, please  verify PR #2655 and approve if it correct.
https://github.com/MicrosoftDocs/windows-driver-docs/pull/2655

@rogermachin , this page (and everything under https://docs.microsoft.com/en-us/windows-hardware/drivers) is intended for hardware/software driver developers, and not for end users. Per this page, we do not recommend that the registry edits be done by end users or apps.

@rogermachin , this page (and everything under https://docs.microsoft.com/en-us/windows-hardware/drivers) is intended for hardware/software driver developers, and not for end users. Per this page, we do not recommend that the registry edits be done by end users or apps.

@RAJU2529, @lorihollasch  , I understand that it's not intended for manipulation by the end-user. But, if I were to check the value of TDRdelay, how should I do it?

Yes, I know that. My main intention was to get the timeout values and show them to the user.

@rawwar . Sorry that's not possible.

thanks for your's answer  .
bests regards ,
Roger

De : VARADHARAJAN K @.>
Envoyé : mardi 9 mars 2021 06:39
À : MicrosoftDocs/windows-driver-docs @.>
Cc : rogermachin @.>; Mention @.>
Objet : Re: [MicrosoftDocs/windows-driver-docs] ? not fount TDR registery entry in windows 10 ... (#2652)
@rawwarhttps://github.com/rawwar . Sorry that's not possible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-driver-docs/issues/2652#issuecomment-793429558, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AES3KI2U3KY5S2U5AUQCOU3TCWYAHANCNFSM4WNUPJZQ.


[DOCS] Update Docker related docs

Description
Docker Desktop released v4.21 today, which changed the way to run WasmEdge containers. We need to update the docs.

Screenshots
No response
Additional information
No response

https://github.com/anshikavashistha/docs/commit/c7e96b2ca0a326474ad7319d744997983141cb81
@alabulei1 PTAL


BEP021: Electrophysiological Derivatives

This PR aims to integrate BEP021 (Electrophysiological Derivatives) into the BIDS specification.
It is a work in progress (a draft PR). As a group we decided in several meetings that we might profit from taking the agreed upon aspects of the BEP in the Google Doc and formalizing them within this PR.
Please comment both in this thread, as well as with inline code reviews as to what you'd like to see changed / improved.
You are welcome to add changes directly via Pull Requests to this branch: https://github.com/bids-standard/bids-specification/tree/bep021
My personal goal is to keep adding to this PR over the next days/weeks so that all applicable points from the BEP Google Doc are incorporated here.
To Do:

[ ] Carry over changes from the Google Doc
[ ] Review internally with the BEP021 team for completeness
[ ] Community review
[ ] Merging into the specification

General Links:

Rendered preview
BEP repo
BEP Google Doc

Related PRs:

https://github.com/bids-standard/bids-specification/pull/1614
https://github.com/bids-standard/bids-specification/pull/1537
https://github.com/bids-standard/bids-specification/pull/1613

cc @christinerogers @CPernet @guiomar @robertoostenveld @arnodelorme @dorahermes

I am closing this, because I do not expect to work on this, and nobody else seems to, either. Furthermore, we do not lose a lot of discussions or comments by closing this here.
Whoever wants to continue this work may open a new PR from the bep021 branch. https://github.com/bids-standard/bids-specification/tree/bep021

Should we aslo remove the BEP from the listing of active BEP on the BIDS website?

In my opinion yes, because it is not active, but I am no longer a moderator of that BEP.

well, not active is sort of .. there are low hanging fruits in other derivative parts that needs sorted out before we can work again on that one


Make it work without basepath

When I try to open the built files locally, or on a server on a subpath, the _next folder fails to load. Yes I can set the basePath in the book configuration, but it should be possible without it so the hyperbook is portable.

Since hyperbook uses nextjs under the hood, I am not sure how I would implement this. Their documentation also does not provide a adaptive setting for the basepath https://nextjs.org/docs/api-reference/next.config.js/basepath. If you have an idea for how implemented it please let me now.

my idea was to use relative include paths instead of absolute ones so the application does not have to mind, idk if that works in nextjs though

I think that this is not how nextjs works, therefore I will close this one until nextjs supports it.


Refactoring and fix bug - default AP name given in fresh install

Fallback AP now gets SSID ESP-RFID-xxxxxx (last 6 digits of mac).
Also refactored to remove code duplication, new method startAP(ssid, password=NULL) rather than duplicating AP creation.

That was quick. Actually i did take that note to myself.


chore(prettier) add prettier to the toolchain

same setup as yarn, except no bracket spacing, since it conflicts with algolia-eslint

very cool, let's kill syntax


Appending to resource attributes (#134)

This is the parsing part for #134.
Could you review my changes ?
I still need to do the interpretation/resolving part but I thought smaller PRs are better.
Thanks

Note that I have added -funbox-strict-fields -O2  to the ghc-options. This was just a try. Do we keep them ?

Is TopSpurious used somewhere still ? I can only see these used here but never with TopSpurious.

I think unbox-strict-fields can be detrimental for memory usage in some cases where you can't share the reference to a resource. I don't think this is a case that happens in this codebase, so it should be strictly (haha) better.


List mycete on the Try Matrix Now page

For more exposure you will want a listing, probably on https://matrix.org/docs/projects/try-matrix-now.html#other •Near the top of that Try Matrix Now page, it instructs to file a PR in https://github.com/matrix-org/matrix-doc •to get your project listed. Good luck.

At some point I will. Want to get it feature complete first. - gonna close this until then :D - Thanks for the recommendation!

Plenty of projects on that page are in Alpha state. But, I assumed since you put the word out on social media linked to Matrix development, that you wanted others aware of the project. And, in Open Source, people often collaborate, even those not closely involved. But it's your call. Anyway, I suggest setting up a Matrix room for your project, which I would join. Good luck.

alright, you convinced me :D

Added in https://github.com/matrix-org/matrix.org/pull/65


Update WiFi Links

Remove link shorteners, as they're no longer allowed by PS, and remove the Youtube channel as it is no longer in use.

Link shorteners are no longer allowed?


support pointers using default tag

This fixes #47 where default was not setting pointer types.

coverage: 94.695% (+0.04%) from 94.66%
when pulling 668ba6e6187bc62a8cd79c1a21139401d6313857 on default-support-pointers
into 4214131ed726fa5c2ec3f1d943ac4e696ed523fe on master.


(Windows) [vite] Internal server error: Missing "./dist/vue.js" export in "@islands/hydration" package, after run of npm run dev

[x] I have read the troubleshooting section before opening an issue.
[x] I have tried upgrading iles and vite.

Description 📖
On Windows run of the npm run dev command will produce errors:

[vite] Internal server error: Missing "./dist/vue.js" export in "@islands/hydration" package
      at bail (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:33612:8)
      at resolve (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:33689:10)
      at resolveExports (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34675:12)
      at resolveDeepImport (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34693:31)
      at tryNodeResolve (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34410:20)
      at Context.resolveId (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34147:32)
      at Object.resolveId (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:35469:55)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at async Context.resolve (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:35234:23)
      at async Object.resolveId (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:35469:32)

[vite] Internal server error: Missing "./dist/hydration.js" export in "@islands/hydration" package
      at bail (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:33612:8)
      at resolve (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:33689:10)
      at resolveExports (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34675:12)
      at resolveDeepImport (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34693:31)
      at tryNodeResolve (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34410:20)
      at Context.resolveId (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:34147:32)
      at Object.resolveId (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:35469:55)
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at async Context.resolve (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:35234:23)
      at async Object.resolveId (file:///C:/Projects/iles-app/node_modules/vite/dist/node/chunks/dep-74663fff.js:35469:32)

Similar to #161, maybe the file:// prefix is also needed for running the dev script on Windows.
Reproduction 🐞
On Windows:
npm init iles@latest
cd iles-app
npm i
npm run dev

Dependencies Info
Run npx iles info and pnpm list (or npm list) and provide the output:
iles v0.8.3 vite v3.0.8

<EMAIL_ADDRESS>C:\Projects\iles-app
+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>

Screenshots 📷

Hi, thanks for the report!
Seems unusual that this wasn't reported earlier if it's related to paths, since the users from #161 would have detected it?
Which package manager are you using?

Which package manager are you using?

@ElMassimo NPM.
npm -v
8.5.5

node -v
v16.15.0

I'm also seeing this problem. It occurs when adding an interactive component with any of the client:* directives. The reason I didn't report it was because until now the site I'm building is 100% statically generated.
Interestingly I've deployed and it seems to work in production, this looks like a problem in dev only.
My bet is that we're missing a file:// somewhere like @mrauhu suggested. @ElMassimo if you need me to post you anything let me know. In the mean time here is the error I'm seeing my end:

Thanks for reporting!
Fixed in @islands/hydration v0.8.1.

I've just upgraded to 0.8.4 and I'm still seeing this problem. I suggest we reopen this issue.

@ohuu Can you open a new issue with information about your iles.config.ts and related plugins?
I tested 0.8.4 in Windows and it's starting the dev server correctly, so I'll need more information to debug it.

My bad. Turns out yarn doesn't automatically upgrade indirect dependencies, so I was still on<EMAIL_ADDRESS>0.8.1 does indeed fix the issue. Sorry for the noise


anndata "obs" are cells, not genes

Had some issues loading our data and realized that there was a transpose issue. Looking at the code, I believe there's a bug. The obs of an anndata object should be the cells, not the genes. The genes are var. So on the plus side for you, no transpose is necessary for loading.
Correct me if I'm wrong, as I may have just confused myself...

Hi, we tested the code using TabulaMuris data and I think they used cells as obs and genes as var. Is it more common to use obs as cells? I can add an extra parameter and set the default to be obs=cell to the class constructor. Thanks for catching that! 😊

That explains it! I made that file and I did it wrong–which led to it working fine with scVI but then our newer files were incompatible. The problem was I didn't search for any examples when I used anndata.read_mtx and assumed it was in the right orientation. This notebook shows them transposing the result.
I'll update the h5ad file on S3 (you might be the only group to use it so far, since no one else has said anything).

I see - thanks! I can fix that (also need to fix our test files to pass the unit tests ^ ^)


KAFKA-13204: assignor name conflict check

Add the partition assignor name conflicting check to avoid wrong assignor used. Also add tests.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ableegoldman , please take a look when available. Thank you.

Test failure is unrelated: StoreQueryIntegrationTest.shouldQueryStoresAfterAddingAndRemovingStreamThread

Merged to trunk


how can I run gulp serve in production ?

how to keep it running in production ?

Like any other node server. But it's incorrect and you shouldn't do this.


Dynamic serializer fields are not supported

Hi, I'm trying to use a serializer that adds fields dynamically. However, XLSXRenderer is not able to pick up those fields.
Here's a simple example:
class MySerializer(serializers.Serializer):
    field_1 = serializers.CharField()

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self.fields["dynamic_field_1"] = serializers.CharField(label="Dynamic Field", allow_null=True)

In the output XLSX, only field_1 is included.
I believe the use of serializer.get_fields() is the root cause:
https://github.com/django-commons/drf-excel/blob/0d47d7bdfcfbe8d09e6186bfea46f8cdc76e2adc/drf_excel/renderers.py#L277
The default implementation of get_fields() in BaseSerializer returns self._declared_fields.  This can be resolved by referencing to serializer.fields instead.
Let me know what you think. I can create an MR to fix it.
IMO, I think we should update this line to use serializer.fields as well.
https://github.com/django-commons/drf-excel/blob/0d47d7bdfcfbe8d09e6186bfea46f8cdc76e2adc/drf_excel/renderers.py#L156C18-L156C29

Interesting... Didn't know it was a thing, is it something well supported by DRF? I'm wondering if there would be any unintended side effects. If you can make a PR with some good testing, I'll be happy to look into it, yes.

There's another reference to get_fields() here: https://github.com/django-commons/drf-excel/blob/0d47d7bdfcfbe8d09e6186bfea46f8cdc76e2adc/drf_excel/renderers.py#L244
We should review and make sure there wouldn't be side-effects there.

I thought serializer.fields was already calling get_fields() anyway... but I might be wrong, I haven't looked at that code in a bit.

@rptmat57 you're right. serializer.fields is calling get_fields(). However, if extra fields are being added dynamically to serializer.fields in the constructor, get_fields() of the BaseSerializer will not be able to include them. Hence, I think it'd be better to retrieve all fields with serializer.fields rather than serializer.get_fields()


fix(md-3782): remove totals from view data android

Changes:

Removed totals from data view
Removed screen guides
Fix search button clicking when dragged instead of tapped

:tada: This PR is included in version 1.1.0-alpha.60 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add helpers to filter email messages

In my use case I was trying to fetch email messages received after some date with an IMAPS server (imap-mail.outlook.com). My first attempt was:
ReceiveMailSession s = ...;
EmailFilter emailFilter = new EmailFilter();
emailFilter.searchTerm = new ReceivedDateTerm(ComparisonTerm.GE, d);
s.receive(emailFilter,null);

but this wasn't returning anything.
Creating a ReceivedDateSearchTerm (similarly to SentDateSearchTerm depicted at http://www.codejava.net/java-ee/javamail/using-javamail-for-searching-e-mail-messages) I was able to correctly fetch messages.
Why not to add a bunch of common filter implementation (starting at least from FromFieldSearchTerm, SentDateSearchTerm ContentSearchTerm contained in that page plus similar implementation for filtering ReceivedDate and/or Subject)?

Heya, one question here, which I dont understand:
EmailFilter do have for example a from() that creates a term for filtering based on from field.
Sure, we can add more, but was from() working for you?
EmailFilter.filter().from("...");

Would you be so kind to check if this is working, and then I can add more implementations :)

Hi Igor,
in my tests I didn't have to filter the from field...looking at FromFieldSearchTerm example at the link above they do:
package net.codejava.mail;

import javax.mail.Address;
import javax.mail.Message;
import javax.mail.MessagingException;
import javax.mail.search.SearchTerm;

public class FromFieldSearchTerm extends SearchTerm {
    private String fromEmail;

    public FromFieldSearchTerm(String fromEmail) {
        this.fromEmail = fromEmail;
    }

    @Override
    public boolean match(Message message) {
        try {
            Address[] fromAddress = message.getFrom();
            if (fromAddress != null && fromAddress.length > 0) {
                if (fromAddress[0].toString().contains(fromEmail)) {
                    return true;
                }
            }
        } catch (MessagingException ex) {
            ex.printStackTrace();
        }

        return false;
    }

}

Sure, I will add more, good idea!

Added missing filters for dates, message size, and text content...
Enjoy!


Return derived() from operator--() in iterator_interface.

Previously, this returned the underlying adapted iterator.
Fixes #50.

Thanks for the Ffix!

No - thank YOU - for a fantastic and very useful library!
On Thu, 9 Dec 2021 at 12:03, Zach Laine @.***> wrote:

Thanks for the fix!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/boostorg/stl_interfaces/pull/56#issuecomment-989359571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANEP6RV5KVJI57OVQEVSXY3UP757FANCNFSM5HRPKJWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Add edgeextension support

Add support for edge extensions to bypass getting the w3cManifest

Hi @am385, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Andrew Messick). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@boyofgreen Hi Jeff, I also found that there is another addition needed for edge extension usage in the main application.


Cargo.toml: include = [] missing some specified paths

Problem
include = []

is not actually including all specified files.
Steps

I have a crate that contains an otherwise unreferenced subcrate.
When I set it to be included in the .crate via Cargo.tomls include key:
include = ["*", "/subcrate"]

It does not end up in the .crate archive generated by cargo package.
Possible Solution(s)
include = [...] should work on all specified files or directories.
Output of cargo version:
cargo 1.40.0-nightly (a429e8cc4 2019-10-04)

It is intentional that a .crate only includes one package.  Can you say more about why you want another package included?

I want to ship an optional binary with my crate that is not installed by default (cargo install) but only when explicitly specified (cargo install rootcrate subcrate).
The subcrate shares functionality with the root crate but cannot depend on it as that would bloat compiletime from 2 seconds to several minutes.

We probably won't extend crate files to include multiple packages.  cargo install already supports the ability to specify multiple crates, so I'm not sure I understand the difference.  Can't they be published separately?

Without more detail, I don't think this is actionable.  I don't think we will allow a .crate to include multiple packages.  There might be other ways to address what you are trying to do, but there's not enough detail here.  If you want to include a more detailed description of how the packages are related (preferably a concrete example) and the problem this is trying to solve (build times?), feel free to comment.


(Chore) Update android app to be targeted API level 30

Description

migrated RN to 0.65.1 (minimal version supporting API 30 without a lot of issues)
upgraded packages & tools to correspond a new RN version
adjusted configs
fixed eslint issues
fixed build for web
fixed build for Android

No IOS build adjustments were performed in this PR.
They would be done when we'll work on IOS version.
About #3317
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

@sirpy regarding the branch keys issue:
i tried different approaches - nothing helped. This is a known issue of the latest versions and there's still no fix:
BranchMetrics/react-native-branch-deep-linking-attribution#336 (comment)
As branch.json doesn't generates from github secrets (like Sentry or google-services.json) we could use this solution as temporary
I suggest to move branch.json issue to the separate task and go back to it later

checked web & native build from bare checkout + npm ci

checked web & native build from bare checkout + yarn

@sirpy opened https://app.zenhub.com/workspaces/gooddollar-5d50833888a83c6880d3e345/issues/gooddollar/gooddapp/3446 for the branch issue


handle VESPA_CONFIGSERVERS

also some refactoring
also, make C++ code (in defaults) handle same variable variants,
like services__addr_configserver
also, update some error messages to indicate that VESPA_CONFIGSERVERS
is the variable we expected to use

@bratseth please review and merge
@hmusum FYI

Great, but don't we also need to change CloudConfigInstallVariables.scala?

CloudConfigInstallVariables.scala: looks like it should be changed, but i don't know enough scala to do so safely, so @gjoranv needs to look at it.

@arnej27959 and @bratseth, I added handling for standalone. Verbose, but legible.


Secret object type for deeper integration

Summary of the new feature / enhancement
User story
As a PowerShell module author, I want to use secrets in my custom functions without needing to learn about and run the individual cmdlets that deal with secrets.
Current Experience
Currently, module authors need to use the Get-Secret cmdlet from the SecretManagement module to retrieve secrets within their functions. This requires additional code and makes it less convenient to work with secrets stored in a vault.
Enhanced Experience
With the proposed enhancement, module authors can use a new [Secret] type for their function parameters. This type allows referencing the name of a secret stored in a vault the user has access to, providing a deep integration with the PowerShell Secret Store.
Proposed technical implementation details (optional)
In the following mock-ups, the first example demonstrates the current experience. In the second example, the author enjoys integration with secret store without needing to learn about and handle the integration. Although this mock-up is overly simple, the implementation could handle additional scenarios such as argument completion (list of secrets) and handling securestring vs plain text.
# Current approach using Get-Secret
function Invoke-SecureAction {
    param (
        [string]$SecretName
    )

    $SecretValue = Get-Secret -Name $SecretName
    # Perform secure action using $SecretValue
}

# Proposed approach with [Secret] type
function Invoke-SecureAction {
    param (
        [Secret]$MySecret
    )

    # Perform secure action using $MySecret.Value
}

Very limited POC.
class secret {
    [ValidateNotNullOrEmpty()]
    [string]$Name
    [secureString] hidden $Secret

    secret ($Name) {
        $this.Name = $Name
        [secureString]$this.Secret = Get-Secret -Name $Name
    }

    [string]asplaintext () {
        [string]$asplaintext = Get-Secret -Name $this.Name -AsPlainText
        return $asplaintext
    }
}
``

It's a reasonably convenient approach, I've always like how Microsoft.Extensions.Configuration basically takes a lot of complex inputs and boils them down to a key-value store.
Thoughts:

What if I, as a user, don't want to use secretmanagement to give you my credential? You'd need to instruct module authors to use parametersets probably which is an even more complicated scenario.
What if I don't want to use the default vault? How am I going to specify it? And how is your class going to know?

@JustinGrote good point. could have an optional parameter for vault name. I only run into that when I store the same name in multiple vaults, but it happens.


Adds error when no config

@OKNoah Thanks for adding!
Can you:

[ ] Add a test for the no client use case in client-test
[ ] Reduce line length to 120 characters
[ ] Run npm test with no errors before committing

Thanks!


Sudoku-solver-4x4: Invalid input alert. #7

Now when an invalid input is given, a message is displayed inside html of the website itself, without using the "alert" function.
The grid gets reset, after the message is shown, thus allowing the user to re-enter the values without refreshing the page.
Both these cases were solved..

@Spandan-Bhattacharya  Please could you review this and let me know the feedback

Can you give some screenshot of it in action?

sure

Can you give some screenshot of it in action?

https://github.com/Spandan-Bhattacharya/Solve_it/assets/115262737/c7011184-ffb3-4213-b0f3-f8478c71273e
@Spandan-Bhattacharya check this video

Yeah the alert is good, but for the reset part, make sure that the whole grid is reset, not just the last input character.

Yeah the alert is good, but for the reset part, make sure that the whole grid is reset, not just the last input character.

@Spandan-Bhattacharya I can but resetting whole grid is made as another issue(#5) ...so I implemented this...What shoud I do?

Okay i see, then it's fine.
Good Job!

Okay i see, then it's fine. Good Job!

Thank You !

Merged! Good Job!