[move-prover] transform a loop into a DAG

This commit breaks the back-edge of a loop and place the induction case
checking of the loop invariant in a newly created block.
The general procedure works as following (assuming the loop invariant expression is L):

Collect loop targets (T) - the set of variables that can be possibly mutated during the execution of the loop.

At the beginning of the loop header, insert the following statements:
- assert L;
- havoc T;
- assume L;

For each backedge in this loop:
- Create a new dummy block (say, block X) with only the following statements
- assert L;
- stop;
- In the source block of the backedge, replace the last statement (must be a jump or branch statement) with the new label of X

Motivation

simplifies encoding of loops
separation of loop invariant base case proving and induction case proving

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo xtest, the related test cases are updated.
Related PRs
#7946 (the first two commits are in fact commits in 7946, only the last commit matters to this PR).

Are we planning to land this before the upcoming release?

No, that is not the intention. My understanding is that we will land it at least after the release.

The PR description says that a separate dummy block is generated for each back edge.  I suppose you can use the same dummy block for all back edges of a loop.

My bad, the implementation is actually one dummy block for  the whole loop and stitch all backedges to that dummy block. I'll update the description.

More work items for this PR based on the discussion with Shaz:

Fix the wrong assumption that two loops cannot share the same header.
Write an analysis to collect mutable references that do not have its location/path changed in loop body and re-establish the assumptions after havoc.

The following items also come into discussion but may be handled in separate PRs after this PR lands:

Allow the use of old(.) in loop invariants.
Have separate error messages differentiating the violation of loop invariants upon entry and upon induction.

Now, if two loops share the same header (e.g., due to two continue statements in the loop body), we construct a fat-loop to capture both loops.

Added an analysis to collect mutable references that do not have its location/path changed in loop body and re-establish the assumptions after havoc.

The rules are captured in loop_analysis.rs, function targets().

Ported a few representative loop tests to move-prover, including

nested loops
loops with two back edges
loops with memory operations in it

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

@wrwg comments addressed and suggestions incorporated, all in the last 6 commits.

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

Rebased to master to resolve the conflict of automated doc file Diem.md, nothing changed

release binaries dependency change summary:
target packages:
  A disassembler 0.1.0 (workspace, path 'language/tools/disassembler')
    * features: default
  A move-coverage 0.1.0 (workspace, path 'language/tools/move-coverage')
    * features: default
  A colored 2.0.0 (direct third-party, crates.io)
    * features: [none]

/land

:broken_heart: Test Failed - ci-test-success

/land

:broken_heart: Test Failed - ci-test-success

Rebased to latest main to see if the land-blocking tests are passing.

/land


"Out of data" message is back

Aged Violet is almost out of videos, but it seems the last few won't come up for classification.  Instead, the interface is giving the "out of data" message when it tries to load Aged Violet videos to the queue.  Green Snowflake videos are still queuing ok.

Ouroboros' subject counts for that group were incorrect, so it thought all the remaining subjects were already active. I've reset the counts and manually activated the remaining subjects.

We finished Aged Violet, but the "out of data" message is still displayed.

Ah, the group hadn't been automatically marked as complete because it has some paused subjects. I've manually changed its status.


Exhaustive search for keyboard conflicts

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:38:23.027Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

html-snippets
abu
0.1.0

vscode-color
ans
0.4.5

npm-intellisense
chr
1.3.0

vscode-simpler-icons
dav
1.4.0

vscode-eslint
dba
1.3.2

vscode-instant-markdown
dba
1.3.0

tslint
eg2
1.0.9

vscode-npm-script
eg2
0.2.1

auto-rename-tag
for
0.0.14

icon-fonts
idl
1.8.1

angular2-switcher
inf
0.1.4

docthis
joe
0.4.8

Angular2
joh
2.7.0

vscode-todo
Mat
0.8.0

Angular-BeastCode
Mik
5.0.4

csharp
ms-
1.12.1

vs-keybindings
ms-
0.1.7

live-server-preview
neg
0.0.4

language-vscode-javascript-angular2
nwa
0.0.11

ionic2-snippets
oud
1.0.0

vscode-docker
Pet
0.0.18

simontest
Sim
0.17.2

vscode-fileutils
sle
2.7.0

ionic2
Tha
2.4.0

cordova-tools
vsm
1.2.12

After hitting another keyboard shortcut conflict I thought I'd do an exhaustive search for more so I've prepared a spreadsheet that highlights duplicate keyboard mapping to assist in the hunt.  Below is an attached Excel sheet that I produced using the steps above to help identify keyboard conflicts and the procedure I used to create this list.
I also found some cases where an identical mapping rule was duplicated which might cause problems with your code maintenance in the future.
Here the conflict that trigged my search:

"ctrl+shift+g" conflict: "Show SCM" and "workbench.action.files.openFileFolder"

Steps to Reproduce:

"Open Keybaord Shortcuts"
Edit "keybinding.json"
Select ALL (Ctrl-A) and Copy (Ctrl-C)
Visit https://konklone.io/json/ to convert JSON to CSV

Remove header comment (1 line) before "[" and and all trailing comments after last "]"

Download keyboard shortcuts as a CSV file.
Open in Excel and sort by 1st column ("key")
Use conditional formatting to highlight duplicate key mappings (1st column)

Example: http://www.excel-easy.com/examples/find-duplicates.html

Inspect to check for duplicate key mappings that may conflict.  I fully understand that you'd need to check the "when" condition.  Since we sorted the rules, this sheet does not record the original rule order which may also affect their interpretation. I'm not up on this but I'm assuming you are!
Happy conflict hunting :+1:
vscode-keyboard-conflict.xlsx

Found plenty more!

This issue was moved to Microsoft/vscode-vs-keybindings#6


Fix for invalid "changed" property on lvm batch.

It seems like lvm batch bluestore deployment checks incorrectly for the .changed property on ceph command output.
This fix seems to work as intended, but testing/comments required.

What version of ceph were you trying to deploy? I think the update for ceph-volume that includes the correct json is not yet released in luminous or mimic.

I am using latest mimic, 13.2.2, which added the batch commands.
I just tested with my changes running the playbook twice, and it seems to try and run the report on the device created last run, resulting in:
RuntimeError: Unable to use device, already a member of LVM
Not sure if my changes cause this state issue.
Does does the master branch assume you're running the latest ceph master?

Yes, the master branch of ceph-ansible does assume you want to install ceph@master. It has support for luminous and mimic as well, but the commits you need for batch do not exist in mimic yet. This PR that was merged a week or so ago is what you need and it's unreleased. https://github.com/ceph/ceph/pull/24588

@Roukoswarf @andrewschoen what's the status on this? The new package should be released by now right?

Well, if master is tracking the ceph master, then this issue is likely invalid and should be closed.
Which stable branch of ceph-ansible is exactly matched to the current mimic stable? I couldn't find one matched specifically.

stable-3.* branches use stable ceph releases, so 3.2 or 3.1.

When checking the stable-3.2 branch, the same code is present and will fail in the same way on mimic 13.2.2.

@andrewschoen what's your recommendation?

@andrewschoen up?

@leseb This PR will not work, there's really no issue here. The changed property has just not made it to any official ceph release yet. If master is installed from dev packages this will all work as expected. I expect this to fail in stable-3.2 as well as it's very close to master, I think stable-3.1 is going to be the best bet for installing a mimic release.

@andrewschoen thanks, I'm closing this based on Andrew's comment.


regex: let the user pick the flags

We might not want to use the insensitive case flag.
Leave it there by default but at least, there is a way to remove it or
use others like the multiline one.

Hi @Dimsday,
Thank you for having merged that. I see that you have created a new version (v1.1.0) but I don't see it available on my side.
Do I have to do something special on my side to update it? My Apps Engine Version is 1.27.1.
I cannot find which version has been published on the marketplace so I don't know exactly where is the issue.

Hello,
Sorry to bother you about that but on my side, I cannot upgrade to v1.1, still stuck with the v1.0 one.
Do we need to do something special to upgrade? Was the v1.1 published to the marketplace without issue?


NOISSUE - merge authz and authn into new service auth

What does this do?
New auth service will perform functions of authn service

issue token
identify
Additionally it will support groups features
assign member to a group ( member can be user or thing )
get group members

Codecov Report

Merging #1313 (06d85d7) into master (b2ccbae) will decrease coverage by 3.96%.
The diff coverage is 38.31%.

@@            Coverage Diff             @@
##           master    #1313      +/-   ##
==========================================
- Coverage   64.91%   60.95%   -3.97%
==========================================
  Files         117      110       -7
  Lines        8029     8582     +553
==========================================
+ Hits         5212     5231      +19
- Misses       2361     2906     +545
+ Partials      456      445      -11

Impacted Files
Coverage Δ

auth/api/http/responses.go
100.00% <ø> (ø)

auth/keys.go
100.00% <ø> (ø)

auth/postgres/groups.go
0.00% <0.00%> (ø)

bootstrap/service.go
74.34% <ø> (ø)

things/service.go
53.40% <ø> (-1.14%)
:arrow_down:

things/users/users.go
81.25% <0.00%> (-18.75%)
:arrow_down:

twins/service.go
69.38% <ø> (ø)

auth/api/grpc/requests.go
34.28% <34.28%> (ø)

auth/api/grpc/endpoint.go
38.46% <38.46%> (ø)

auth/service.go
43.96% <43.96%> (ø)

... and 21 more

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 b2ccbae...06d85d7. Read the comment docs.


Center loader in frontend

In the frontend a loader appears when making a request for surf data (thanks to @shaifulazh!). It would look better if this were centered instead of off to the left, like in the video,
See 3f3202cd9c80b64a368435ef196b5fdace472d66 for an idea of where to work
video (12).webm

3a397356e50da4aaf254204dbf4be59ca85b566e


[Beta - Bug] Rule twig statement

Hello,
I just try the latest beta at this day, and i got for:
{{
pimcore_renderlet('newsteaser', {
type: 'object',
editmode: editmode,
controller: 'News',
action: 'emailNewsTeaser',
title: 'Drag a news object here',
height : '400px',
width: '100%'
})
}}
the message: ERROR A print statement should start with 1 space.
I think twigcs want inlined {{ pimcore_renderlet('newsteaser', {...
But as it the code seems to respect the standart twig rules. Maybe cariage return should be count like a space...
For information, i test the linting on this: https://github.com/pimcore/demo/tree/master/app/Resources/views
Thanks!

By the way i found the same problem on this code:
<thead>
    {%
        include 'Areas/print-product-table/spec_attribute/column-attribute-table-header.html.twig' with {
            'configArray': configArray,
            'classname': classname,
            'levels': levels,
            'currentLevel': 0
        }
    %}
</thead>

which return ERROR A tag statement should start with 1 space.
It seems always that carriage return are not acocunted like space, but should i think.

Hey,
I do not see any problem. Here's what those examples should look like for twigcs to pass:
<thead>
    {% include 'Areas/print-product-table/spec_attribute/column-attribute-table-header.html.twig' with {
        'configArray': configArray,
        'classname': classname,
        'levels': levels,
        'currentLevel': 0
    } %}
</thead>

{{ pimcore_renderlet('newsteaser', {
    type: 'object',
    editmode: editmode,
    controller: 'News',
    action: 'emailNewsTeaser',
    title: 'Drag a news object here',
    height: '400px',
    width: '100%'
}) }}

It seems to me this is related to #58

Thanks, yes exactly, it is same than #58
After reading more carefully twig standard norm, yes you are right Put one (and only one) space after the start of a delimiter ({{, {%, and {#) and before the end of a delimiter (}}, %}, and #}):
My bads, this issue can be close so!


🚀 Feature: Take into account number elements in a Table in the base theme

🔖 Feature description
The Backstage base theme has a table component with a pagination toolbar at the bottom which allows you to choose how many elements should be displayed in a card with that table (from now on mentioned as page size). It seems the table layout does not care about the actual number of elements displayed in the table and sizes itself to whatever the page size is, which leads to a not so nice experience where the user can choose a page size of 100 for a table that only has 10 elements to display.
It would add to the user experience if the table size took into account the actual number of elements displayed in the table rather than the page size
🎤 Context
It improves the user experience of the product out of the box. This would prevent for example, the user seeing a table with 10 elements and changing the page size to 20 only to get a table which is mostly a white page because there are only 11 elements to display
✌ Possible Implementation
Use the actual number of returned elements to be displayed rather than the page size to set the table size
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Can you find an example of a table in some other application or somewhere online that you can show us here with some idea on how you would like it to work? :pray:

Absolutely. The BigQuery UI is the one that comes to mind directly
See the 2 screenshots. Independent of the page size set at the bottom, the UI does not grow with the size of it but rather the number of rows shown (up to a limit of course)

@Rugvip I marked that I could help with this issue so I could perform an attempt on a PR. I just need a pointer on where exactly in the code  this change would be and I should be able to figure my way out from there.

@Rugvip can you re-open?


undefined method `markdown_link' when using bitbucket_cloud

Report
What did you do?
I want to add a link to my warning to a certain file, therefore I wanted to use markdown_link or html_link
Both methods are defined in the danger sourcecode: https://github.com/danger/danger/blob/bbf5d0d15ec314fc8529bcff62ae8ae04b9d78e4/lib/danger/danger_core/plugins/dangerfile_bitbucket_cloud_plugin.rb#L153
What did you expect to happen?
Inside the comment should be a markdown Table with correct links.
What happened instead?
Danger outputs [!] Invalid Dangerfile file: undefined method markdown_link for #\<Danger::Dangerfile:0x0000560fe45cf770> same thing for html_link
Your Environment

Bitrise / Bitbucket Cloud
Danger 5.12
What is your Dangerfile?

pictures = git.added_files.select { |file| file.end_with? "jpg" }
if pictures.any?
  fail "Do not commit images in jpg!"

  message = "### JPGs\n"
  message << "File\n"
  message << "| --- |\n"

  pictures.each do |pcitrue_file|
    message << "#{bitbucket_cloud.markdown_link(picture_file)}\n"
  end

  markdown message
end

Cool, yeah, sounds like a bug


Allow treating /_virtual_includes/ and bazel-out/ as -isystem

Like https://github.com/erenon/bazel_clang_tidy/pull/32, but as opt-in behavior. Additionally allow ignoring generated includes in bazel-out/
The opt-in part is handled via bool_flags, which needs Skylib.

ping on this @erenon, wdyt?

Hi, since a recent change, HeaderFilterRegex is no longer recommended: https://github.com/erenon/bazel_clang_tidy/commit/f43f9d61c201b314c62a3ebcf2d4a37f1a3b06f7
Instead, lib headers are added automatically. Does this affect this change?
re skylib: I'd like to keep this project dependency free as possible.


WIP: Bypass _concat_arrays to improve performance

This PR is a work in progress.
This patch bypasses _concat_arrays when loading examples into a device. It increases the performance of the MNIST example by 7~10% compared with the current implementation using chainerx on GPU (--batch-size=100 --device=cuda:0) in my environment. I believe it is especially useful for experiments with large mini-batch.

Add BatchableDatasetMixin, which has get_examples in addition to the DatasetMixin operators

The existing get_example produces an example, or a list of {tuples, dicts} of examples (e.g. List[Tuple[ndarray, int]]). In this setting, you need to convert them (e.g. concat_examples) to feed them into a device
get_examples produces Examples instance unlike get_example

Add Examples, which is a Sequence-like collection of examples

It keeps the examples as a dataset, or a {tuple, dict} of datasets (e.g. Tuple[List[ndarray], List[int]])
Its to_dataset method transfers the examples into the specified device in batch without any conversions
If the dataset is ndarray, to_dataset tries to slice it by using array indexing (dataset[indices]) instead of picking each example
(It has a same interface with collections.abc.Sequence but is not an actual instance of it to avoid the overhead of isinstance)

Add sample_from_dataset in chainer.dataset.examples, which is a convenient function to implement get_examples

e.g.:

class MyDataset(BatchableDatasetMixin):
    ...
    def get_examples(self, indices):
        order = ...
        padding_spec = ...
        return examples.sample_from_dataset((self._images, self._labels), indices, order, padding_spec)

Fix Iterators to handle datasets that have a get_examples method

next() of the iterator will return an Examples for a "batchable" dataset instead of an actual list of examples

Fix Updaters and Evaluators to skip running convert (e.g. concat_examples) if Iterator.next() returns Examples

This patch keeps backward compatibility with the customized Updaters in the wild, thanks to __iter__ method in Examples

Add resolve_device_spec function to reuse a part of to_device code in Examples

TODO

Write tests
Implement get_examples in TransformDataset
Fix more iterators in Chiainer Core and examples to support Examples
Fix iterators in ChainerMN

Some codes assume Iterator.next() returns list directly

Fix imagenet example (which uses DALI for pre-processing)

Maybe need to extend sample_from_dataset or to_dataset to accept a custom convert function

Confirm the specification

It seems some functions and links assume its inputs are ndarray instead of dataset objects (which just have __getitem__ and __len__). I can't judge Examples need to convert their types on behalf of concat_examples or not
padding in parallel updater

It seems that preprocessing and transmission to the device should be done in the iterator because Updater can be a bottleneck.

It will be superseded by #7117. Close this after it is merged.


Feature: Support for TagSpaces Tagging Conventions in Files

What feature or improvement do you think would benefit Files?
Integrating support for TagSpaces tagging conventions would significantly enhance the usability of Files for users who manage their files with TagSpaces tags. TagSpaces offers two tagging methods: an invasive method that incorporates tags directly into filenames within square brackets [tag], and a non-invasive method that utilizes JSON files stored in a .ts directory adjacent to the tagged files or folders. Enabling Files to recognize and display these tags in its "info panel" would provide a seamless experience for users who rely on TagSpaces for file organization, making it easier to view and manage tagged files directly within Files. This is a follow-up to this question on Discord.
Requirements

Implement a parser in Files that can recognize and extract tags from filenames formatted according to TagSpaces' invasive tagging convention (tags within square brackets).
Develop functionality to read and interpret JSON metadata files stored in .ts directories according to the non-invasive TagSpaces convention.
Display the extracted tags in the Files' "info panel" for easy viewing.
Ensure that tag data is updated in the Files interface when changes are made to filenames or JSON metadata files.
Provide settings options in Files to enable or disable TagSpaces tag recognition, accommodating users' preferences.
Conduct performance testing to ensure the new tagging features do not adversely affect the app’s performance.
Create documentation and help guides to assist users in utilizing the new TagSpaces tagging features in Files.

Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.4291
Comments
This feature would be particularly beneficial for users who use TagSpaces extensively for file management but wish to utilize the sleek and powerful interface of Files for their daily tasks. It bridges the gap between two powerful tools, enhancing the organizational capabilities of Files while adhering to users' established workflows.

Thanks for the feedback, I've added this to the project board.

How does TagSpaces work if a file is transferred or renamed?

@yaira2 Not sure, TBH. ¯\_(ツ)_/¯ Asked them a question here on Twitter, let's see if they respond.

If you rename/transfer such file outside of TagSpaces, the connection with meta file is lost, this happens also by the database based tagging solutions.

This is why we went with alternate data streams, they allow files to be moved externally without losing the tags.

Correct, there are limitations with either approach, but we had to choose one.
From our docs (https://files.community/docs/features/tags)

Tags are saved using alternate data streams (and not in a database). We found that alternate data streams is the best approach for Files because it allows items to be transferred between locations (even without using Files) without losing their tags. However, this system has some limitations:
Tags only work on drives formatted as NTFS
Tags may not work with all cloud storage providers

This feature request suggests that we provide an option for both methods. We'd prefer picking a single method but we'll leave this open for some time to gather additional feedback.

I have a propasal that should work

Central Database Storage: Implement a centralized database to store the paths of all tagged items. This will serve as the primary reference point for tagged data.
Tag Embedding: Concurrently, embed the unique tag identifier (GUID) directly into the tagged file’s metadata. Note: This method is applicable exclusively to NTFS file systems due to their support for alternate data streams.
Database Retrieval: Upon initiating the tagged items’ loading process, the system will first retrieve the list from the central database. It will then apply a filter to exclude any items that are no longer accessible or have been removed.
Search For Embedded Tags: Files can then scan the file system for items containing tag-related alternate data streams. It will eliminate any duplicates already accounted for in the database to prevent redundancy.


Publish 03/29/2022, 10:30 AM

Docs Build status updates of commit 4318ca3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

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

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

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

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

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionConnector.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionErrorReport.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionOperation.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionSyncResult.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionTeamMap.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionWfmTeam.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Get-CsTeamsShiftsConnectionWfmUser.md
:white_check_mark:Succeeded
View (teams-ps)

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

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

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

teams/teams-ps/teams/Remove-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-CsTeamsShiftsConnectionTeamMap.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-CsTeamsShiftsScheduleRecord.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsShiftsConnectionInstance.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Test-CsTeamsShiftsConnectionValidate.md
:white_check_mark:Succeeded
View (teams-ps)

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


Preview: Undo action using Ctrl+Z doesn't work after adding/removing tail

Steps to Reproduce

Add multi-tail arrow
Add any element
Add a tail to multi-tail arrow

Click Ctrl+Z

Actual behavior

Can't undo actions by Ctrl + Z buttons

Expected behavior

The tail is removed on the first undo by Ctrl + Z, the element is removed on the second redo and etc.

Environment Details

Nightly environment /5104-save-load-multitail-arrow/
OS: Windows 11 Enterprise
Browser: Chrome, Version 127.0.6533.120 (Official Build) (64-bit)

Additional context

Undo button works correctly, the issue is only with keyboard Ctrl + Z
The issue is detected during exploratory testing of https://github.com/epam/ketcher/issues/5056

Successfully verified; Undo/Redo actions work fine.

Environment details:
Ketcher Version: 2.25.0-rc.1 Build at 2024-09-03; 15:31:03
Indigo Version: 1.24.0-rc.1.0-gb8f099851-wasm32-wasm-clang-19.0.0
Chrome Version: Version 128.0.6613.113 (Official Build) (64-bit)
Windows 11 Enterprise


fix: greetings workflow access token naming convention

Updated greetings workflow action token naming convention
:warning: make sure to add a new personal access token in repository secrets with name GH_TOKEN and give access to issues and pull requests
Fixes #257
Type of change
Please give a X on it which is applicable

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactor Code
[ ] A documentation update
[X] Others(mentioned in the issue number)

How Has This Been Tested?
I have tested this workflow on my repo: https://github.com/aayushchugh/workflows/issues/1

Checklist:
give a X on it which is applicable

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

Amazing work @aayushchugh


Example explaining how to load values does not work

The example used (in section: Do not use value objects returned from methods or properties in the same query
) does not work, you have also to load description (w => w.Description):
ClientContext context = new ClientContext("http://SiteUrl");
Web web = context.Web;
context.Load(web, w => w.Title);
context.ExecuteQuery();
ListCreationInformation creationInfo = new ListCreationInformation();
creationInfo.Description = web.Description;
creationInfo.Title = web.Title;
SP.List newList = web.Lists.Add(creationInfo);
context.ExecuteQuery();

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

ID: 306bdabe-3314-0f4d-9199-1be256fd58de
Version Independent ID: e99a275e-bd93-ef8c-6fe6-ff81065e5f42
Content: Complete basic operations using SharePoint client library code
Content Source: docs/sp-add-ins/complete-basic-operations-using-sharepoint-client-library-code.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Thanks for your post. You can modify the Content Source directly and submit an pull request.

Close by #2907


Fix resolv.conf race condition

This hopefully fixes #607

I did a "quick" (it took over 2 minutes to run those commands) systemd-analyze dot | dot -Tsvg -o dot.svg and systemd-analyze plot > plot.svg to figure out which things start after network-online.target on one machine I have access to. One example is wireguard which does the following in its service:
cat<EMAIL_ADDRESS>[Unit]
Description=WireGuard via wg-quick(8) for %I
After=network-online.target nss-lookup.target
Wants=network-online.target nss-lookup.target
PartOf=wg-quick.target
Documentation=man:wg-quick(8)
Documentation=man:wg(8)
Documentation=https://www.wireguard.com/
Documentation=https://www.wireguard.com/quickstart/
Documentation=https://git.zx2c4.com/wireguard-tools/about/src/man/wg-quick.8
Documentation=https://git.zx2c4.com/wireguard-tools/about/src/man/wg.8

[Service]
Type=oneshot
RemainAfterExit=yes
ExecStart=/usr/bin/wg-quick up %i
ExecStop=/usr/bin/wg-quick down %i
ExecReload=/bin/bash -c 'exec /usr/bin/wg syncconf %i <(exec /usr/bin/wg-quick strip %i)'
Environment=WG_ENDPOINT_RESOLUTION_RETRIES=infinity

[Install]
WantedBy=multi-user.target

So I think we should set the following which is very similar to the suggestion from Erik from the archwiki to get what we want:
[Unit]
After=network-online.target nss-lookup.target
Wants=network-online.target nss-lookup.target


Remove github3.py as a dependency

Most of the functionality is used via IGitt, other than few org stuff, so ideally we should implement whatever is missing upstream and then remove github3.py

Please create upstream issues for whatever is missing in IGitt.

This task is to create upstream issues for whatever is missing in IGitt.
It should only be done by GCI participants that have completed "Your first coala issue" and a GCI task of an IGitt low+ issue


Add new backend interfaces

This PR introduces new interfaces for the backends to implement instead the way is done now.
This will make it easy to implement new backends.
The backend are required to implement the following interface:
// Backend is basic methods a backend must implement
type Backend interface {
	// Type must return backend unique type string, called by both Proxy and Controller
	Type() loadTestV1.LoadTestType

	// TransformLoadTestSpec should validate and transform LoadTestSpec, called by Proxy
	TransformLoadTestSpec(spec *loadTestV1.LoadTestSpec) error

	// Sync should create resources if not exists, called by Controller
	Sync(ctx context.Context, loadTest loadTestV1.LoadTest, reportURL string) error
	// Sync should update status with current resource state, called by Controller
	SyncStatus(ctx context.Context, loadTest loadTestV1.LoadTest, loadTestStatus *loadTestV1.LoadTestStatus) error
}

and optionally to those:
// BackendGetEnvConfig backend must implement this if it has Environment Variables
// This method is called by both commands, Proxy and Controller
type BackendGetEnvConfig interface {
	// GetEnvConfig get from backend a envConfig struct pointer
	GetEnvConfig() interface{}
}

// BackendSetDefaults backend must implement this if wants to set default values
// This method is called by both commands, Proxy and Controller
type BackendSetDefaults interface {
	// SetDefaultValues gives a chance to backend initialize itself
	SetDefaults()
}

// BackendSetLogger interface can be implemented by backend to receive an logger
// This method is called by both commands, Proxy and Controller
type BackendSetLogger interface {
	// SetLogger gives backend a logger instance
	SetLogger(*zap.Logger)
}

// BackendSetPodAnnotations interface can be implemented by backend to receive pod annotations
// This method is called only by command Controller
type BackendSetPodAnnotations interface {
	// SetLogger gives backend a logger instance
	SetPodAnnotations(map[string]string)
}

// BackendSetKubeClientSet interface can be implemented by backend to receive an kubeClientSet
// This method is called only by command Controller
type BackendSetKubeClientSet interface {
	// SetKubeClientSet gives backend a kubeClientSet instance
	SetKubeClientSet(kubernetes.Interface)
}

// BackendSetKangalClientSet interface can be implemented by backend to receive an kangalClientSet
// This method is called only by command Controller
type BackendSetKangalClientSet interface {
	// SetKangalClientSet gives backend a kangalClientSet instance
	SetKangalClientSet(clientSetV.Interface)
}

// BackendSetNamespaceLister interface can be implemented by backend to receive an namespaceLister
// This method is called only by command Controller
type BackendSetNamespaceLister interface {
	// SetNamespaceLister gives backend a namespaceLister instance
	SetNamespaceLister(coreListersV1.NamespaceLister)
}

An important change is those lines:
	Sync(ctx context.Context, loadTest loadTestV1.LoadTest, reportURL string) error
	SyncStatus(ctx context.Context, loadTest loadTestV1.LoadTest, loadTestStatus *loadTestV1.LoadTestStatus) error

Passing LoadTest by value and status by reference, this way backends cannot mutate LoadTest, but only LoadTestStatus

+1

+1


Cow: Watch xususa.com

Cow requests the watch of the watch_keyword xususa\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

xususa\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


plaintext password storage

should make this a priority for a "release" milestone.

Closing, should be Elysium

Re-opening to let @wastevens decide where this goes.

Closed.  Passwords are now encrypted.  Site wide password encryption secret is NOT committed.


Kitematic doesnt work behind corporate proxy

Expected behavior
Kitematic app should interpret OS level proxy settings or environment variables HTTP_PROXY, HTTPS_PROXY and work behind a proxy, or the UI should provide a config setting to provide proxy settings.
Actual behavior
Doesn't seem to be working as connections are refused and traffic seems to be routing straight to the endpoints and not through the proxy first.
Information about the Issue
Build 0.13.0, environment variables are set, and proxy pac file is retrievable.
HTTP_PROXY=http://user:pass@proxy:80
HTTPS_PROXY=http://user:pass@proxy:80

Steps to reproduce the behavior
download kitematic, and when trying to sign in to docker hub (the very first screen you come to) it just says connection refused. Additionally clicking skip, you cant pull up or login to the hub.

this seems to actually have fixed its-self, after installing the docker toolbox, and setting the proxy settings in the docker preferences menu I'm able to login to the hub and pull down images.

As long as you have the Use system proxy turned on it should work properly

Hi maybe that remarque it to late but i sware good it s not cleare aboot docker toolboxe on windows 7 I REAPTE WINDOWS 7 no good documentation are added to clarify how we can configure the proxy with virtualbox behin corporate proxy
Ok we have to add the environment variable http proxy and https proxy and no proxy (no cleare because the ip adresse provide in nat network should conflict or not the local ip)
Help use with taking care about configuration windows7
EXPLAIN US HOW we can resolve confkict
How we can use docker compse correctly inside cmd our instead cygwin or powershell or minwin ....
How we resolve tls problem even with authentication
We can t change os from windows 7 to 10
THANKS FOR ADVANCE TO BE CLEAR


Add DialTimeout option to remote commands

By default, in Go, Dial has no timeout, which mean that we may wait up to OS timeout (~2/3 minutes on Linux) during the first attempt while remote system system may be ready earlier.
With DialErrorLimit, it also provides better control on the maximum way time (even though the Retry parameters, 100ms base + 1.5x backoff are not publicly documented).

/run-acceptance-tests

👋 Any update on going forward with this PR?

/run-acceptance-tests

/run-acceptance-tests

/run-acceptance-tests

@iwahbe Can you help unblock this as a part of your Command investments?

This functionality merged as part of https://github.com/pulumi/pulumi-command/pull/268.


syntax error, unexpected '?'

Q
A

Bug report?
yes

I got error every time i send mail
Reference
https://github.com/swiftmailer/swiftmailer/blob/master/lib/classes/Swift/Mime/SimpleMessage.php
Function
getPriority()
 public function getPriority()
    {
        list($priority) = sscanf($this->getHeaderFieldModel('X-Priority'),
            '%[1-5]'
            );

        return $priority ?? 3;
    }

It looks like you are trying to use a Swiftmailer version that requires PHP 7 with PHP 5.

Exactly you are right .
Thanks


Omdbapi is going private (used in movie)

Issue Label:

[X] Bug
[ ] New feature
[ ] Enhancement
[ ] New component

Description:
So it seems Omdapi is going private in 05/08/17 (need Patreon to get api key), so that's going to make the movie script essentially not work.
Anyone happen to have any alternatives in mind?

If its a bug make sure to include this section.
OS and OS version:

[X] Mac
[X] Linux 32 Bit
[X] Linux 64 Bit
[X] Windows 32 Bit
[X] Windows 64 Bit
OS Version:

I am aware of this but at the moment cannot find any movie API's that will remain free and opensourced. For now movies is going to work until the api stops the key and then the script will break permanently...

Seems like https://www.themoviedb.org/ would be a good/only alternative i could find

They actually seem to have a decent API. When the movies script breaks hopefully I can have a replacement ready using this API.

also apparently Rotten Tomatoes also  has an api https://developer.fandango.com/Rotten_Tomatoes it needs applying tough

Of course using python to crawl a page like rotten tomatoes for info is also an option. theres a good framework  for that: https://scrapy.org/

I can configure my script according to your need https://github.com/N3V0N/TMDb

Using the the script by @N3V0N and modifications by me I have a backup movies script ready. For now it will sit in the movies folder under the filename tmdb. If the movies script ever breaks I will rename it to movies.deprecated and rename tmdb to movies. tmdb could be improved upon by grabbing more information. If it is able to grab more then the current movies or proves to be better in some other way but equal in information than tmdb will become the default.


fix(launch): slice memory bug in core process job input filtering

Description

Fixes WB-NNNNN
Fixes #NNNN

What does the PR do? Include a concise description of the PR contents.

[ ] I updated CHANGELOG.md, or it's not applicable

Testing
How was this PR tested?

#7556  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bcsherma and the rest of your teammates on  Graphite


[fix](cluster key) fix cluster key too many segments when schema change

What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
when schema change, the primary key index is not sorted, so the solution is buffering and merging some blocks, and then write them to one segment and start a new segment.
after this pr, the primary key index can be sorted in SegmentWriter.
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall


License for the code

If possible could an explicit license be listed for the implementation code?

Great point! Done.


TileLayersView call move

factory.get(TileLayersView.class, true, TileLayersController.class);
Can this code be moved elsewhere, can it be queue'd instead and picked up wherever instead of just having it exist within the MainWindoView?


[BUG] Basic sell garage feature

Hello @ImagicTheCat, there is a problem with this function, when player has vehicle out, he can sell the vehicle and keep it (vehicle won't despawn).
Thanks in advance for your amazing dev!

Yes I know, it's not so bad, unless your sell factor is less damageable than the rent factor.

At the beginning of your framework, I modified the garage function for to implement the "In/Out" state in BDD. Are you interested ?

You respawn the vehicles at their old out position ? If yes, it's something that I had in mind, but I don't know yet how to do it without breaking everything when your vehicle respawn.

Nop, when I spawn vehicle, I set state to "Out" in BDD.
When I despawn it, I set state to "In". And when player leave server, it reset state directly to "In".
Like this:
    local state = "in"
    q_set_vehgroup_state:bind("@user_id",user_id)
    q_set_vehgroup_state:bind("@vehicle",veh_type)
    q_set_vehgroup_state:bind("@state",state)
    q_set_vehgroup_state:execute()

Oh I see, I'm not interested, at this point vehicles only exist while in game.

Yes, but if player disconnect when another player is already in the car, we have a problem :D

The engine manage the car spawn/despawn, I have even saw issues with vehicles being lost (the engine respawn it, so the id doesn't match in basic garage) if you go too far away or if you teleport. Fixing a car can not work also because of the synchronization between players, causing the car to rollback to a not fixed state. That's the problem of client-side games trying to synchronize themselves, no idea how to smooth this.

Really strange, I made a lot of tests according that you describe, and I didn't noticed any losing vehicle.


Trying to get property of non-object in lucadegasperi/oauth2-server-laravel/src/Storage/FluentClient.php:187

Hi there,
I'm receiving this error in Lumen 5.1 when trying to request an access token with correct credentials. It seems that hydrateEntity expects the $result to be an object, however it's an array. Modifying the function to hydrate array params works, but is there something I'm doing wrong? I read another post about setting the PDO:Fetch to return an object, but I'm pretty sure that's the default behaviour already
Cheers,

Seems to be an issue with defining multiple database connections, rather than this. Although, perhaps defining the fetch type in the plugin might not be a bad idea if using Facades. If anyone has any idea as to why when defining more than one database the fetch type gets screwed up?
Cheers

Thanks for using this package!
For this kind of support questions, we'd like you to use the following resources:

Documentation
OAuth 2.0 Documentation
Laracasts
Laravel Slack
Stack Overflow

This will allow us to keep the issues for bug reporting only and make our work easier.


Add js license info for example page.

This is a follow-up to #257.

Coverage remained the same at 80.22% when pulling fec5963ff31911764e665a827c04683d0ca3cd30 on albsod:development into 24357d6da8d1b3f529bf9ba25d897bc199c5b78c on Textalk:development.

Coverage remained the same at 80.22% when pulling ea0627d0ab13b108afcee5559bf0f774f7f0f14b on albsod:development into 24357d6da8d1b3f529bf9ba25d897bc199c5b78c on Textalk:development.

Personally, I always try to use HTTPS when possible.  I wasn't aware of the speed difference though and I suppose there is no harm in using HTTP for the jquery URLs.
"I can see a case for using :// instead as that would remove it being blocked when going to browsing the https:// site."
OK, I'll change all the URLs to that.

Sorry it's actually // not ://
On Sun, Mar 22, 2015 at 12:03 AM, albsod<EMAIL_ADDRESS>wrote:

Personally, I always try to use HTTPS when possible.  I wasn't aware of the speed difference though and I suppose there is no harm in using HTTP for the jquery URLs.
"I can see a case for using :// instead as that would remove it being blocked when going to browsing the https:// site."
OK, I'll change all the URLs to that.
Reply to this email directly or view it on GitHub:
https://github.com/Textalk/angular-schema-form/pull/311#issuecomment-84468017

Coverage remained the same at 80.22% when pulling 72e66cbbfc06e61a0ee1c25577fae230f6d5a6ca on albsod:development into 24357d6da8d1b3f529bf9ba25d897bc199c5b78c on Textalk:development.


Dependency "illuminate/support" missing.

Using any search request throws error.

To reproduce install package in a new project:
composer init
composer require deschutesdesigngroupllc/perscom-php-sdk

Run the following code:
        $user = $connection
            ->users()
            ->search(filter: [new FilterObject('email', 'like', 'user@example.com')])
            ->json('data')[0] ?? null;

Temporarily I added illuminate/support to my own deps to fix since I don't know what direction you want to go with it.
illuminate/support does have quite some dependencies of its own. Do you really need these dependencies just for this little helper?
Up to you to either add the dependency to composer.json or remove usage of it...


How to use Osprey in my own dataset

Thank you for your amazing work. And here is the thing, what should i do to link your Osprey with my own dataset? I have some images collected by our team. Should i retrain the model to make the description of every area more accurate? how to do this? And because it is a folder of images not just one image that i can use in the demo. How to use your model in a large amount of images and collect all the result?

Hello, @joy91269
If you find that the generated description of our model is not so accurate for your domain data, you can consider finetuning the model with your own dataset.  To do this, ensure your data is formatted in accordance with the osprey-724k dataset we've provided.
When you wish to conduct inference on a collection of images, you'll need to run the demo code on your local machine. Additionally, you should craft a script that processes each image, feeding the each mask region into the model to obtain the corresponding answers.

I send you a email to your @zju.edu.com to ask more detail things.


Invalid cert type while trying to generate an certificate

I ran into a bit of a problem when I tried to generate a PEM certificate using the following command
$ sudo ./snpguest certificates PEM certs
ERROR: Invalid cert type
Error: Invalid cert type

Then I change the format and met the same problem
$ sudo ./snpguest certificates DER certs
ERROR: Invalid cert type
Error: Invalid cert type

I'm not quite sure if I read the tutorial wrong, but that seems to be how he wrote it
$ sudo ./snpguest certificates -h
snpguest-certificates 0.3.0
Certificates command to request cached certificates from the AMD PSP

USAGE:
    snpguest certificates <encoding> <certs-dir>

FLAGS:
    -h, --help       Prints help information
    -V, --version    Prints version information

ARGS:
    <encoding>     Specify encoding to use for certificates. [PEM | DER]
    <certs-dir>    Directory to store certificates in. Required if requesting an extended-report.

This command does not generate certificates. It simply extracts the existing certificates (ASK, ARK and VCEK) from the CPU certificates cache. You have to import these certificates first with snphost.

Got it. Thanks.

@DiWangShePi You can use the fetch subcommands to fetch the CA and VCEK certs, then use import to store them to the PSP.


[Playground Task]: Speed up loading

What needs to happen?
Playground takes about 7 seconds to load at the first take (the last vertical bar marks 8 seconds):

At the second take, it's just over 2 seconds (the last vertical bar marks 3 seconds):

We should:

See if we can speed up the first take.
See if we are using caching fully for subsequent takes.

Issue Priority
Priority: 2
Issue Component
Component: beam-playground

.take-issue


Wording is wrong in many places on this page

This sentence: "Azure Machine Learning designer is a drag-and-drop UI interface to build pipeline in Azure Machine Learning.
As shown in below GIF, you can build a pipeline visually by drag and drop your building blocks and connect them."
Should be:
Azure Machine Learning designer is a drag-and-drop UI interface for building machine learning pipelines in Azure Machine Learning Workspaces.
As shown in below GIF, you can build a pipeline visually by dragging and dropping building blocks and connecting them.

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

ID: 037b4303-07aa-b63f-4fff-3b1ce72f9d81
Version Independent ID: d0cfb6ab-249a-f0a9-08aa-ebddb2213b1b
Content: What is the Azure Machine Learning designer(v2)? - Azure Machine Learning
Content Source: articles/machine-learning/concept-designer.md
Service: machine-learning
Sub-service: core
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

@lferdinandMS
Thanks for your feedback! We will investigate and update as appropriate.

@lferdinandMS
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


regular non dynamic image tag images not displayed

Describe the bug
I am not trying to do a dynamic image just loading a regular  url via image src tag <img src="https://media.geeksforgeeks.org/wp-content/uploads/20190506164011/logo3.png" alt="Inage" /> and the image is not displayed.  To duplicate do normal npm install with vue, javascript, and default options.  Add an img src tag with full url in your App.vue and run npm run dev.
I have tested in just a regular vue vite app and the img tag works fine.
Picture of default electron-vite app with img src tag showing broken image above the electron logo

Electron-Vite Version
2.3.0
Electron Version
33.2.0
Vite Version
5.4.11
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

I figured it out. It is the default Content-Security-Policy in the index.html.  It does not allow showing remote images. I changed my img-src to *    ```

Further info https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Security-Policy/img-src


Decouple LockedApp schema from spin-core

This is another experiment in decoupling the all-important LockedApp object from spin-core (and therefore from wasmtime and all its instruments of CPU heating).  The previous unpublished efforts were (1) feature flag to compile a version of spin-app without the spin-core dependency (intrusive and felt like a maintenance catastrophe waiting to happen) and (2) similar to this but without spin-app re-exporting its existing API (larger blast radius).
I don't know if this is the right answer - a whole crate seems pretty heavyweight, and maybe harder to understand than a feature - so if other folks have better ideas then bring 'em on, I'm happy to do what works.

I'm generally fine with this, and in the long term I'd like to get rid of the spin-app crate entirely; I'm not really happy with the App abstraction in there.


Strange errors like: is imported by os?commonjs-external, but could not be resolved – treating it as an external dependency

Hey! I suddenly ran into strange errors, and I can not understand what I changed to make it happen. Please, can you help me debug?
In the browser it looks like this:

In the terminal like this:

In the browser console like this:

Here is my ".entry.js" file:
AdminJS.UserComponents = {}
import Component1 from '../dist/admin/ui/components/path'
AdminJS.UserComponents.Component1 = Component1
import Component2 from '../dist/admin/ui/components/photo/index'
AdminJS.UserComponents.Component2 = Component2
import Component3 from '../dist/admin/ui/components/dashboard'
AdminJS.UserComponents.Component3 = Component3
import Component4 from '../node_modules/@adminjs/upload/src/features/upload-file/components/edit'
AdminJS.UserComponents.Component4 = Component4
import Component5 from '../node_modules/@adminjs/upload/src/features/upload-file/components/list'
AdminJS.UserComponents.Component5 = Component5
import Component6 from '../node_modules/@adminjs/upload/src/features/upload-file/components/show'
AdminJS.UserComponents.Component6 = Component6
import Component7 from '../node_modules/@adminjs/upload/src/features/upload-file/components/edit'
AdminJS.UserComponents.Component7 = Component7
import Component8 from '../node_modules/@adminjs/upload/src/features/upload-file/components/list'
AdminJS.UserComponents.Component8 = Component8
import Component9 from '../node_modules/@adminjs/upload/src/features/upload-file/components/show'
AdminJS.UserComponents.Component9 = Component9
import Component10 from '../node_modules/@adminjs/upload/src/features/upload-file/components/edit'
AdminJS.UserComponents.Component10 = Component10
import Component11 from '../node_modules/@adminjs/upload/src/features/upload-file/components/list'
AdminJS.UserComponents.Component11 = Component11
import Component12 from '../node_modules/@adminjs/upload/src/features/upload-file/components/show'
AdminJS.UserComponents.Component12 = Component12

I can also provide access to the entire codebase. I will be very grateful for your help. Thank you!

one of your frontend components is probably importing a nodejs library

Understood. I had a utils.ts file that was used for both the backend and the frontend. And in it, I do import uploadFeature from "@adminjs/upload”, for my helper. It was this import that led to errors.


[SWAGGER/ Club/ GET]  Incorrectly described the parameter for endpoint '/api/club/{id}'

Environment: Windows 11 Pro Version 22H2 Build 22621.2428
Google Chrome 118.0.5993.118 (Official build) (64-bit)
Reproducible: always
Build found:
front 2024-01-04 d5b9ee
back 2024-01-04 7dd789
Preconditions
1.Swagger http://speak-ukrainian.eastus2.cloudapp.azure.com/dev/swagger-ui/index.html#/  is opened.
2. Token is gotten.
Steps to reproduce

Go to endpoint '/api/club/{id}' http://speak-ukrainian.eastus2.cloudapp.azure.com/dev/swagger-ui/index.html#/club/getClubById

Actual result
ID is present as a parameter.
The concept of a parameter is incorrectly described.

Expected result
ID should not be a parameter.
Detailed information about the path is available.

Related:
https://github.com/ita-social-projects/TeachUA/issues/2135
https://github.com/ita-social-projects/TeachUA/issues/2132
https://github.com/ita-social-projects/TeachUA/issues/2117
https://github.com/ita-social-projects/TeachUA/issues/2133
https://github.com/ita-social-projects/TeachUA/issues/2134


Package source link for ChocolateyGUI 0.15.0 in nuspec is wrong

current points to: https://github.com/chocolatey/ChocolateyGUI/tree/develop/ChocolateyPackage
instead of: https://github.com/chocolatey/ChocolateyGUI

seems to be a remnant of before the directory structure changed.
the correct url would probably be:
https://github.com/chocolatey/ChocolateyGUI/tree/develop/nuspec/chocolatey

@bcurran3 thank you very much for reporting that!


Lazy Read: Iterator objects are not correctly released

Describe the bug
Follow-on of NSE-383 and NSE-429

closed by #470


DEV-489 apply Lombok to registration service

SonarQube analysis reported 13 issues

 13 major

Watch the comments in this conversation to review them.
3 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 RegistrationInfo.java#L17: Remove the "credentials" field and declare it as a local variable in the relevant methods.
 RegistrationInfo.java#L18: Remove the "user" field and declare it as a local variable in the relevant methods.
 RegistrationInfo.java#L19: Remove the "team" field and declare it as a local variable in the relevant methods.

Coverage decreased (-0.04%) to 84.569% when pulling f9837d9c7d403e990149d08acd04d41e3be71fe4 on DEV-489 into 148053dcd43d21a731c5fd83fe786a223d72dd61 on master.


[Form Field v2] Tooltip for FF Helper text

Tooltip integration with FF for helper text in low screen real estate scenarios
Part of V2 Form Field features

@libertymayc do we have any control over tooltip position? I notice when width is restricted the tooltip is clipped, rather than appearing above or below.
Also, is there an agreed standard for tooltip reveal? Specifically:

The delay on keyboard focus - appears instantly (is that common convention with focus?
Animation on reveal - i thought the tooltip was supposed to have an animation in/out of some description? @navkaur76 @alycrys ?

Tooltip is clipped - this shouldn't happen. @bhoppers2008 could you please point out the story (steps to follow) where this is happening?
Delay on keyboard focus - floating-ui has a delay option only for hover. would take some effort to add it for keyboard focus and not sure it adds much value, as keyboard users don't normally care for animations and visual delays.
Animation on reveal - was meant to come later as an enhancement

Tooltip is clipped - this shouldn't happen. @bhoppers2008 could you please point out the story (steps to follow) where this is happening?
Delay on keyboard focus - floating-ui has a delay option only for hover. would take some effort to add it for keyboard focus and not sure it adds much value, as keyboard users don't normally care for animations and visual delays.
Animation on reveal - was meant to come later as an enhancement

@alycrys
Story: https://c8451ff8.saltdesignsystem-storybook.pages.dev/iframe?args=&id=core-form-field--helper-text-as-tooltip&viewMode=story

Open dev tools
Drag dev tools window so reduced space to either side of form field
Hover over FF
Tooltip appears clipped to left or right, rather than above (first choice) or below (second choice).
Another observation... alternate hover, tooltip moves from left to right. On right site, text wraps. On left side, text is on single line (both clipped).

On KB focus, works for me. Good to have a clear reason 👍
Animation... understood. Is there an issue to track it?

Tooltip is clipped - this shouldn't happen. @bhoppers2008 could you please point out the story (steps to follow) where this is happening?
Delay on keyboard focus - floating-ui has a delay option only for hover. would take some effort to add it for keyboard focus and not sure it adds much value, as keyboard users don't normally care for animations and visual delays.
Animation on reveal - was meant to come later as an enhancement

@alycrys Story: https://c8451ff8.saltdesignsystem-storybook.pages.dev/iframe?args=&id=core-form-field--helper-text-as-tooltip&viewMode=story

Open dev tools
Drag dev tools window so reduced space to either side of form field
Hover over FF
Tooltip appears clipped to left or right, rather than above (first choice) or below (second choice).
Another observation... alternate hover, tooltip moves from left to right. On right site, text wraps. On left side, text is on single line (both clipped).

On KB focus, works for me. Good to have a clear reason 👍
Animation... understood. Is there an issue to track it?

@bhoppers2008 this is only an issue if the screen is smaller than 365px, but not an issue if there is a smaller wrapper. As we're not dealing with the screen size at this point, I wouldn't count this as an issue for us to fix.
Don't think there is an issue for the animation. Maybe we should have one if that's definitely something we want to do


Fix "make scheduler"

Running make scheduler fails for me without this change. Please verify!

Duplicate of #162

I see. Your approach is even better because it doesn't run as root. I will close here. However, would be glad if @mblaschke @jousch could merge it. Thanks!


WIP Elewis refactor cmds readme

@jeff1010322 - I would love to get a review on this.
Things I want to continue to change ( in seperate pr )

Consts need moved/cleaned up... planning on creating a flags files
tui build out - still missing a few items in the new tui pkg
clean up of commands - I am thinking about creating a command wrapper ( similar to tui ) for verbs

Would love feed back on the overall layout.
I haven't had a chance to test a lot of this but it was largely just moving things around ... only functional change was adding the block height look up before submitting requests ( worth making sure I didn't miss a location for that )

I built and tested the commands and everything seems to be working for me.  There were a few verbs that are not being added to the commands currently: delete.go, show.go, and list.go
For the layout, the only thing that seems odd to me in the cmd layout is the separation between resources and verbs in parallel packages.  Like would it make sense to make the resources each their own package and include the verbs under the specific resource package even if that means there may be an occasional duplicate verb file for different resources?
Something like this:

- /cmd

   - /channel

      - channel.go

      - delete.go

      - list.go

      - lookup.go

      - execute.go

      - pause.go

   - /configuration

      - configuration.go

      - add.go

      - initialize.go

      - set.go

      - show.go

I thought about this originally... A
few things

I wanted to be explicit about the resource/verb pkgs to show clearly how new commands should be added and thought about
I feel this structure provides less code duplication going forward as more resources are added ( I.e the project resource we discussed )
I want to limit the amount of duplication even if it is just at the top level verb command
I had the verbs pkg under resources but felt that didn't make sense when the root cmd could have verbs ... I.e show

That said - I'm not in love with it.
Let's discuss this morning and I'll chat through what else I'm thinking.


Constant Rate Factor?

Hello, wondering if there are any plans for CRF or something similar? Maybe it's' not supported?
https://unix.stackexchange.com/questions/28803/how-can-i-reduce-a-videos-size-with-ffmpeg
Additional options that might be worth considering is setting the Constant Rate Factor, which lowers the average bit rate, but retains better quality. Vary the CRF between around 18 and 24 — the lower, the higher the bitrate.
ffmpeg -i input.mp4 -vcodec libx264 -crf 20 output.mp4

sure, open a PR.

sorry i have no idea how your underlying encoder works or if crf is even doable.  looks to work in x264, x265 and libvpx but not sure if that's what yours uses?

the code is all open source if you want to see how it works

it's beyond me...i'm gonna run some file size tests as I would love this to be a viable encoding option. thanks for your work on this :)

with some googling there is no CRF option
we use AVFoundation on iOS and MediaCodec on Android

ahhh thanks for that :)


boxes are all 30 weight which falls behind with gridinv and makes it inconvenient

all boxes are thirty weight which means that you can only fit three in a backpack, and exactly 4 in a duffel bag. this makes it really hard as det to get all your boxes in order. some boxes like the survival box are fine but the other boxes that are 4 tiles in size on upstream should be lowered to 15 to make it easier to fit them in the bag.

This will be obsoleted when we merge gridinv, but feel free to propose an alteration now. I am not sure if it will be accepted because you can inherently hold less stuff with listinv.

well a quick change, as said in the issue, is to make boxes 15 weight instead, or even 10 weight

We do not want to make boxes weigh less than they hold.


⚠ Bofrost.es has degraded performance

In 61477a8, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1557 ms

Resolved: Bofrost.es performance has improved in 883d2d2 after 9 minutes.


parse methods are returning an array

I saw your comment that you don't know why the result is an array. I think it's because the GO method has a dual return type: (output *js.Object, err error)
I'm very new to GO and gopherjs (in fact I found your repo after spending loo long spinning my wheels on trying to do this exact thing: build a HCL2 parser for node.js), but from what I have seen the convention in GO is that errors are not thrown but signaled using this dual-return type pattern. I suspect that if you changed the signature to:
func parseToObject(input string) (output *js.Object)

then the result via JS would not be enclosed in an array. I would hope that there is a way to make the JS version throw the error using the gopherjs lib. If I get some more time, I may try some things out and make a PR, but for now I just wanted to mention this.
But, thanks for making this repo. I was about to give up hope.

I'm having the same issue. I'm writing a loader to allow you to require() HCL files. My initial implementation used the parseToString method, but due to the returned array I had to use JSON.stringify(parseToObject(hcl)[0]) instead.


Add Jetbrains Gateway compatibility

Hi,
I am currently running IntelliJ in WSL2 and connecting via Jetbrains Gateway. The plugin is not compatible in this way forcing me to run a parallell Windows IntelliJ in order to run the plugin.
Is there any plan to implement compatibility with Gateway?
Thanks.

Hi! Thank you for your feedback. We have added this task to our backlog, but we cannot promise it will be completed quickly.


Add support for WAYFless URLs

Description

This PR adds support for WAYFless URLs.
There is a new called setting SAML WAYFless URL Template used for configuring acquisition link templates as described below:

Now it's available only for OPDS collections.
In order to add this config setting to other collection, you need:

Add SAMLWAYFlessConfiguration to the collection's configuration settings list:

add SAMLWAYFlessConfiguration class as a parent of their configuration class:

class MyCollectionSettings(SAMLWAYFlessConfiguration):
  pass

class MyCollection:
  SETTINGS = ... + MyCollectionSettings.to_settings()

OR add SAMLWAYFlessConfiguration's properties to the collection's SETTINGS list directly:

class MyCollection:
  SETTINGS = ... + SAMLWAYFlessConfiguration.to_settings()

Add mapping to CirculationAPI.default_fulfilment_post_processors_map:

 @property
    def default_fulfilment_post_processors_map(
        self,
    ) -> Dict[str, Type[CirculationFulfilmentPostProcessor]]:
        """Return a default mapping of protocols to fulfilment post-processors.

        :return: Mapping of protocols to fulfilment post-processors.
        """
        from core.opds_import import OPDSImporter

        from .saml.wayfless import SAMLWAYFlessAcquisitionLinkProcessor

        return {
            OPDSImporter.NAME: SAMLWAYFlessAcquisitionLinkProcessor,
        }

This will inject SAMLWAYFlessAcquisitionLinkProcessor to the fulfilment processing chain and will be calling it to do post-processing of FulfilmentInfo objects fulfilled by collection integrations.
Motivation and Context

The phrase "Where Are You From?" (WAYF) is often used to characterise identity provider discovery.
Generally speaking, a discovery service is a solution to the identity provider discovery problem, a longstanding problem in the federated identity management space when there are multiple identity providers available each corresponding to a specific organisation.
Discovery service provides a browser-based interface where a user selects his or her home organization (i.e., identity provider). A service provider uses this information to initiate SAML Web Browser SSO.
Please refer to examples below to understand how typical IdP discovery service pages look like:

d5c1400d11f3/Untitled.png)
To learn how a discovery service works, the SWITCH federation has an excellent series of demos that describe and illustrate how a discovery service integrates into a typical SAML flow.
A WAYFless URL, is specific to an institution with associated users and to a web-based service or resource.  It enables a user from an institution to gain federated access to the service or resource in a way that bypasses the "Where Are You From?" (WAYF) page or Discovery Service step in SAML based authentication and access protocols.
Circulation Manager does not support WAYFLess URLs since it has it own unique built-in discovery service:

Circulation Manager mentions all the configured by admins IdPs in the authentication document
Mobile apps allow patrons to choose an IdP they want to authenticate against:

However, many prominent Ebook Content Vendors such as ProQuest, Springer, Taylor and Francis, FULCRUM, etc. use WAYFLess URLs.
How Has This Been Tested?

I tested the script using the SAML testbed :

Comment out es, db, cm, cm-test services in docker-compose.yml.

Run the SAML testbed:

docker-compose up -d

Set up the local Circulation Manager as it's described in the SAML testbed's README file.

Set SAML WAYFless URL Template configuration setting of opds.hilbertteam.net collection to https://fsso.springer.com/saml/login?idp={idp}&targetUrl={targetUrl}:

Run circulation-test:

pyenv install 2.7.17
pyenv virtualenv 2.7.17 circulation-test
pyenv activate circulation-test
pip install -U pip
pip install -r requirements.txt
cd ..
FLASK_APP = circulation-test/circulation_test/__init__.py
FLASK_ENV = development
FLASK_DEBUG = 0
export CM_ACS=http://localhost:6500/saml_callback
export CM_AUTHENTICATION_DOCUMENT_URL=http://localhost:6500/authentication_document
export CM_LOANS_URL=http://localhost:6500/SAML/loans
export CM_GROUPS_URL=http://localhost:6500/SAML/groups

flask run

NOTE: Replace http://localhost:6500 with the local CM's URL.

Authenticate using different credentials from a list of built-in users.

In circulation-test first borrow A Dictionary in Hindi and English and then download it:

Notice that CM redirected to the Springer website because it was set up as SAML WAYFless URL Template:

Checklist

[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

I just pushed a commit that reorganizes at least some of the system into a mixin-based system. It's not expected to be usable in this form yet, and there are test failures. Thought I'd get the work so far pushed so that I can go back to working on LCP.

This appears to be ready to review.

@io7m Merging #180 added some conflicts. ☹

This is even more ready for review than before.

I can see no issues with the code  :+1:

I am going to merge this PR with a couple of caveats:

This PR contains changes to configuration settings support that are needed elsewhere, so there is pressure to merge this code. This part of the PR appears to work properly.
I am not 100% sure that the WAYFless URL portion of this PR works. I was unable to complete testing with the SAML testbed, which is not working properly (confirmed by @RishiDiwanTT) and needs an overhaul. However, whether it works or not, it does not appear to negatively impact integrations that do not need WAYFless support, so the need to get the configuration benefits of this PR outweigh concerns about whether this part works. We can fix this later, if needed.
There are errors in the testing instructions in this PR's "How Has This Been Tested?" section. We will need to update these instructions as we endeavor to complete testing outside the bounds of this PR.


Please Help

Can t find boot.elf anywere

The instructions clearly state how to obtain it. What's the issue?

Yes the instructions are straight foreword. But i tried 7 times and my RPCS3 never creates an boot.elf in my data\BLUS30443\
DS Version BLUS30443
RPCS3 v0.0.6-8410 Alpha [2019-07-22]

Ah, it's possible RPCS3 has changed its behavior regarding saving decrypted executables. I'll look into it.

Thank You !

Any luck?

Yeah it doesn't save boot.elf anymore since https://github.com/RPCS3/rpcs3/pull/5453. The build before that was 0.0.5-7715 (any build before that will also work), which you can download here: https://rpcs3.net/compatibility?b

Newer builds of rpcs3 now has a menu option for decrypting PS3 executables. Documentation fixed via ec8d6b0.


Alternative To Distance

Description

I noticed that you have code snippet for calculating the distance between two co-ordinates in 2d space. Would it be useful to add a code snippet which does the same but isn't confined to 2d space.

You could generalize it and make it x dimensional...

This is what I have come up with so far;
function distance(...coords){
    let length = Math.trunc(coords.length/2);
    let sum = coords.slice(0,length).reduce((accumulator,currentValue,currentIndex) => accumulator + (Math.pow(currentValue-coords[length+currentIndex],2)),0);
    return Math.sqrt(sum);
}

console.log(distance(10,0,10,2000,10,10)) //1990.0251254695254

Essentially

The function allows a number of args (coords)
Slices the coords array in half into length
Reduces length array by squaring the difference of each coordinate component and summing
Finally the sum is square rooted

What do you think?

This what I have come up with so far:
function distance(...coords){
    let length = Math.trunc(coords.length/2);
    let sum = coords.slice(0,length).reduce((accumulator,currentValue,currentIndex) => accumulator + (Math.pow(currentValue-coords[length+currentIndex],2)),0);
    return Math.sqrt(sum);
}

// (10,0,10) & (2000,10,10)
console.log(distanceV2(10,0,10,2000,10,10)) //1990.0251254695254

Essentially:

The function can take any number of args (coords)
coords is split into half, separating the two points
For each co-ordinate component I find the difference between the two points and square
Finally I square root the sum

I was wondering what you think?

I like the idea, but it shouldn't be called distance. distance is the existing, simplified 2D version. Should we call it vectorDistance to make sure it's understandable?

Sweet I will do pull request


Reading from stdin doesn't dispatch to shell

~/p/r/parallel ❯❯❯ echo 'echo 1; echo 2;' | gnu-parallel
2
~/p/r/parallel ❯❯❯ echo 'echo 1; echo 2;' | target/debug/parallel
1; echo 2;

Worse (for me)
~/p/r/parallel ❯❯❯ echo '( cd mydir; echo 2; )' | gnu-parallel
2
~/p/r/parallel ❯❯❯ echo '( cd mydir; echo 2; )' | target/debug/parallel
< just hangs forever >

This might be an intentional difference from gnu parallel? I like the fact I can put any arbitrarily complex shell command into the stdin, so I can make subshells, change directories, etc.

For some reason, this didn't occur to me.  I'll just have to set up some conditional compilation to account for Windows which does not feature Unix shells like sh.

This is now possible: https://github.com/mmstick/parallel/commit/cb908eb764a39993485e09ae356a9c08fc468c01


[Bug] Can't install helm on local k8s v 1.27.2

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
Helm Chart (apache/skywalking-kubernetes)
What happened
Tried to install as per link
  ~ export SKYWALKING_RELEASE_VERSION=4.3.0  # change the release version according to your need
export SKYWALKING_RELEASE_NAME=skywalking  # change the release name according to your scenario
export SKYWALKING_RELEASE_NAMESPACE=default
➜  ~ echo $SKYWALKING_RELEASE_NAME
skywalking
➜  ~ echo $SKYWALKING_RELEASE_NAMESPACE
default
➜  ~ echo $SKYWALKING_RELEASE_NAMESPACE
➜  ~ echo $SKYWALKING_RELEASE_NAMESPACE=skywalking
default=skywalking
➜  ~ helm install "${SKYWALKING_RELEASE_NAME}" \
  oci://registry-1.docker.io/apache/skywalking-helm \
  --version "${SKYWALKING_RELEASE_VERSION}" \
  -n "${SKYWALKING_RELEASE_NAMESPACE}" \
  --set oap.image.tag=9.2.0 \
  --set oap.storageType=elasticsearch \
  --set ui.image.tag=9.2.0
Pulled: registry-1.docker.io/apache/skywalking-helm:4.3.0
Digest: sha256:34f9abc41924aabbe7c71a0e5b0c548b67e2b8d7093f19e21fa8927309f00e4a
Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: resource mapping not found for name: "elasticsearch-master-pdb" namespace: "" from "": no matches for kind "PodDisruptionBudget" in version "policy/v1beta1"
ensure CRDs are installed first

What you expected to happen
i don't know. I'm junior at these stuff
How to reproduce
see in what happened
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

This should be fixed in https://github.com/apache/skywalking-kubernetes/pull/98, please try SKYWALKING_RELEASE_VERSION=4.4.0

fails for some other reason. i think there's some issue with pod affinity, elasticsearch nodes are not booting up.
Dunno


Add a new plugin fetchinvoice inside the archive bash script

Here we go, I arrived also in this repository with a PR :-)
During the test of new tag with lamp I received a following error
/data/user/0/com.lvaccaro.lamp/no_backup/lightningd/../plugins/fetchinvoice': opening pipe: No such file or directory
And this PR add the plugin fetchinvoice inside the archive.sh bash script

thanks mate
ack 57bd6ed47ff7d0b0c22415d977c4cc58b0b522da

@lvaccaro I missed also the plugin offers, I added now

2 plugins are better than 1
ack 193e37bc87aee25cb58f2d11c317f963d378b310


Android Compose Add a Scrollable List

Name of the Codelab or Codelab URL
https://developer.android.com/codelabs/basic-android-kotlin-compose-training-add-scrollable-list?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-3-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-training-add-scrollable-list#3
Describe the problem
error when using items() in LazyColumn -- The compiler says it needs an Int not a List
In which lesson and step of the codelab can this issue be found?
Lesson number + step number.
[Add a scrollable list] + Step 4
How to reproduce?
What are the exact steps to reproduce the problem?
Versions

What version of Android Studio are you using?
What API level are you targeting?

Additional information
This code:
@Composable
private fun AffirmationList(affirmationList: List, modifier: Modifier = Modifier) {
LazyColumn {
items(affirmationList){ affirmation ->
AffirmationCard(affirmation)
}
}
}
Should be replaced by this code:
@Composable
private fun AffirmationList(affirmationList: List, modifier: Modifier = Modifier) {
LazyColumn {
items(affirmationList.size){ affirmation ->
AffirmationCard(affirmationList[affirmation])
}
}
}
codelab: basic-android-compose-training-add-scrollable-list

The code in the example is correct. The items() function you are referring to is LazyDsl.items()
However, the function being referred to in the codelab (and found in the solution) is an extension function of LazyListScope. It's still defined in LazyDsl, but it is not the same and it can take a list as a parameter. See LazyListScope.items().
To ensure you are using the correct function, make sure to have the following import:
import androidx.compose.foundation.lazy.items


Elegant UI.

Design and Implement a more elegant UI for the extension.
Current UI

I'll take this one.
Any references for the UI?

Great!
Something in Material design will look Good.
Reference:

Kindly read this article for some tips on Chrome Extension designs: Design Tips


Problem with multi dimensions GPy

Hi there,
I am new to use GPy, I am trying to make a simple model that takes 2 input (xtrain1, xtrain2) and a single output so here's my code where data is an array of sensor readings and I am trying to estimate the coming readings
Xtrain1 = data[0:200]
Xtrain2 = data[23:223]

Ytrain = data[24:224]

Xtest1 = data[225:250]
Xtest2 = data[248:273]

Ytest = data[249:274]

Xtrain = [Xtrain1,Xtrain2]
Xtest = [Xtest1,Xtest2]

kernel = GPy.kern.RBF(input_dim=2, variance=1., lengthscale=1.)
m = GPy.models.GPRegression(Xtrain, Ytrain, kernel=kernel)

m.optimize()
simY, simMse = m.predict(Ytest)

Everything work as expected when I insert Xtrain1 and Yrtain to the model but when I tried to add another input Xtrain2 I got this error
'list' object has no attribute 'ndim'

ndim is an attribute for numpy ndarray, so I believe wrapping your Xtrain as a numpy array should solve this issue

Thank you for the reply, I changed my code like this
Xtrain1 = data[0:200]
Xtrain2 = data[23:223]

Ytrain = data[24:224]

Xtest1 = data[225:250]
Xtest2 = data[248:273]

Ytest = data[249:274]

Xtrain =np.array([Xtrain1,Xtrain2])
Xtest = np.array( [Xtest1,Xtest2])

kernel = GPy.kern.RBF(input_dim=2, variance=1., lengthscale=1.)
m = GPy.models.GPRegression(Xtrain, Ytrain, kernel=kernel)

m.optimize()
simY, simMse = m.predict(Ytest)

and now I got this error
....gp.py", line 41, in init
assert X.ndim == 2
AssertionError
any idea?


QOL: Fix vim-test to filter to single tests

:TestNearest support for busted stops at the file level. It would be nice for :TestNearest to find the nearest it() call and run that.
If vim-test could find that it call, extract the string and convert it to a Lua pattern, then pass that to --filter, then :TestNearest could totally work at a reasonable level.
The string would have to escape the following characters to make a literal matching pattern:
( ) . % + - * ? [ ^ $

Lua's pattern escape character is %.
This quality of life fix would improve my ability to run single tests instead of using my current hack using tags.

https://github.com/vim-test/vim-test/pull/598 is in the PR queue. I've got a local version for now with my branch to make this work.

Yay, it's merged!


file input request filepath.

hello flask,
I just need help regarding request.files['spam'].
why request.files not have filepath attribute?
here my stackflow issue http://stackoverflow.com/questions/41882959/get-path-of-file-uploaded-to-flask
that needed a fullpath attribute.
please kindly support it.

Browsers do not send meaningful information there to avoid leaking information the server has no need to know so it makes not much sense to expose more than just the filename in Flask.


Manual Play not working

First of all thanks for keeping this plugin alive,
When ever I use the manual play option it always defaults to casting on me (the astro), I saw the old issue where it was being caused by other plugins (MoAction, I use Reaction though) so I have disabled every plugin (Not uninstalled I dont know if that matters) and it still happens.
What is weird is that auto play will target the correct person and works fine but if I press play manually it always goes on me and I cant think of anything that would be intefering with it.
I have deleted Astro install/config,enabled deal cards in n second, reinstalled, disabled every other plugin and cant get it to work.
Thank you

Okay
So I reinstalled it manually and now Manual Play works in Trials/Raids but not outside of a duty which is fine, (Before it didnt work in trials for me either)
I dont know if it was the update or I did something wrong before but for now seems good..
Thanks again for the update!!

I've just updated the repo so it should be back to normal unless you are running the dev build i sent on discord.


ghcjs: Add ghc-8.0 stage2 packages, update shims.

Motivation for this change
Adds the stage 2 packages required by ghcjsHEAD.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Looks good!


Creating a CCE Cluster did not work and got "Page not found" error

Issue on page:        https://github.com/opentelekomcloud-docs/cloud-container-engine/blob/main/umn/source/instruction.rst
Creating a CCE Cluster did not work and got "Page not found" error

Thank you for your input. The solution is on the way.
(https://gitea.eco.tsi-dev.otc-service.com/docs/cloud-container-engine/pulls/59/files)

The link has been updated. Thanks.


side chain documentation improvements.

content review + addition by @shiwk

:white_check_mark: Build AElf <IP_ADDRESS>8 completed (commit https://github.com/AElfProject/AElf/commit/324eb29041 by @samfrb)

:white_check_mark: Build AElf <IP_ADDRESS>8 completed (commit https://github.com/AElfProject/AElf/commit/25840d3879 by @rosona)


Allow to configure if NodePort should be used

In our environment, we don't use Kubernetes service NodePort.
Could you please make this  configurable?

It should just be MultiPodPerHost false I believe.
https://github.com/aerospike/aerospike-kubernetes-operator/blob/master/deploy/crds/aerospike.com_aerospikeclusters_crd.yaml#L121
Does that work for you?

In the docs it is over here: https://github.com/aerospike/aerospike-kubernetes-operator/wiki/Configuration#spec

oops, yes, sorry
Thanks


ci  update ci config

Description
Update GitHub Actions:

build(deps): bump step-security/harden-runner from 2.9.0 to 2.9.1
build(deps): bump EmbarkStudios/cargo-deny-action from 2.0.0 to 2.0.1

Type of change

[X] CI

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] 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] Any dependent changes have been merged and published in downstream modules

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.32%. Comparing base (815e4cd) to head (af7bfe3).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #43   +/-   ##
=======================================
  Coverage   52.32%   52.32%
=======================================
  Files           4        4
  Lines         946      946
=======================================
  Hits          495      495
  Misses        451      451

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


some small problems

Describe the bug:

scan this QR  code, I get the following error:

Currently, more people use Discord for community communication. A Discord link should be added.

Steps to reproduce:
1.
3.
4.
Attu version:
Attu version:

will fix it in the next release


Allow packages that failed validation to be reuploaded

https://github.com/NuGet/NuGetGallery/issues/4678

Thank you !!!

Is this live yet? This just happened to one of my packages.


Adding omitted moon-gray color to bg and color classes

@oller This is our fault because we doing don't have documentation around how to contribute yet - but this change would go in the tachyons-skins repo here https://github.com/tachyons-css/tachyons-skins otherwise this will get overridden the next time someone updates any of the modules. I can make the change manually myself - or if you want to update that repo, I'd be more than happy to merge it in. Thanks so much for contributing and apologies for not having better docs yet.

No worries. I thought that might have been the case after I created the PR.
Shall get that done in the correct repo later today
On Mon, 28 Mar 2016, 12:04 Adam Morse<EMAIL_ADDRESS>wrote:

@oller https://github.com/oller This is our fault because we doing
don't have documentation around how to contribute yet - but this change
would go in the tachyons-skins repo here
https://github.com/tachyons-css/tachyons-skins otherwise this will get
overridden the next time someone updates any of the modules. I can make the
change manually myself - or if you want to update that repo, I'd be more
than happy to merge it in. Thanks so much for contributing and apologies
for not having better docs yet.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/tachyons-css/tachyons/pull/168#issuecomment-202341561


Add Windows support (MSVC)

Trying to do something with https://github.com/sodiumoxide/sodiumoxide/pull/258 and https://github.com/sodiumoxide/sodiumoxide/pull/200. Not very successfully.
The problem is vcpkg-rs requires some environment variables to be set, and these variables cannot be set in build.rs.

Also change README.md "Platform Compatibiility -> Windows" "In Progress" to "Yes". Also it would be good to add badge. See example in tox-rs:
[![Appveyor Build Status][appveyor-badge]][appveyor-url]

[appveyor-badge]: https://ci.appveyor.com/api/projects/status/y3y2hi6552qgmfr0/branch/master?svg=true
[appveyor-url]: https://ci.appveyor.com/project/kpp/tox/branch/master

And badge to Cargo.toml, example from tox-rs:
[badges]
appveyor = { repository = "kpp/tox", id = "y3y2hi6552qgmfr0" }

@Dylan-DPC give us the id of the project from AppveyorCI->Settings to setup the badge.

Should I also add a fallback (in the case when there's no libsodium in the system or required environment variables are not set) that downloads libsodium and links it statically?

Should I also add a fallback

No. It's another issue. See https://github.com/sodiumoxide/sodiumoxide/issues/234

Thanks a lot @suhr . I'm happy witih this. Is there anything else pending or (WIP) can be removed from the title and merged?

@Dylan-DPC It would be nice to add an appveyor badge.

Ah damn forgot about that :D

SVG image: https://ci.appveyor.com/api/projects/status/u05iy6wufw9ncdi7?svg=true

Although this build.rs passes CI, I don't like the algo behind it. It can be described in pseudolanguage:
find_libsodium():
1. If SODIUM_LIB_DIR is set, tell cargo to find library
2.1. If SODIUM_INC_DIR is set, return the value
2.2. Otherwise probe package using package manager and return the value of inc_dir

The point is that if the user specify any of env SODIUM_LIB_DIR, SODIUM_INC_DIR, SODIUM_STATIC we should use env vars only. We should use package managers only if there are no env variables set.
I don't want @suhr to implement correct algo since... well... we don't need this functionality. But in the future you guys will need to fix it.

We should use package managers only if there are no env variables set.

I disagree about SODIUM_STATIC: on linux both static and dynamic libraries are available via pkg-config. It makes sense to link statically with a system libsodium.

Will there be a case where CI doesn't have any of those values set? I know this should ideally mimic the way a developer builds the crate on windows, but we have other issues in that regard which we need to solve (such as #234 ), which should resolve this concern.

https://ci.appveyor.com/project/Dylan-DPC/sodiumoxide/builds/20147722/job/62pnocscycg3gm9h#L261

warning: redundant linker flag specified for library libsodium

I wonder what's that

bors: r+


can't search resistance option in korean client

In the picture, that option is fire resistance
In korean client, cant search only the resistance options
psudo searching is work. only blue letters are not work

Hi, thanks for creating an issue. Could you please provide the copied item data (CTRL + C). Thanks! I'll look into it.

아이템 희귀도: 희귀
혼란스러운 차양
태양의 관
퀄리티: +8% (augmented)
에너지 보호막: 168 (augmented)
요구사항:
레벨: 62
민첩: 60
지능: 122
홈: B-B-B-B
아이템 레벨: 71
역학 폭발 효과 범위 16% 증가 (enchant)
에너지 보호막 75% 증가
생명력 최대치 +62
화염 저항 +18%
냉기 저항 +31%
기절 및 막기 회복 16% 증가
에너지 보호막 최대치 +35 (crafted)
not the same item in the picture. but all of item have same problem.

Hi, thanks for the item data. i've found the error:
base_fire_damage_resistance_%
%1$+d%% to Fire Resistance
화염 저항 %1$+d%%

fire_damage_resistance_is_%
Fire Resistance is %1%%%
화염 저항 %1%%%

There are two stats with the same translation (in korean). This requires quite some effort to get resolved. I'll provide a quick workaround in the next version

0.6.8 (2020-03-19)

fix temporary damage_resistance mismatched with damage_resistance_is (#359)

Now it works!
Always feel thanks for using this. thank you!


Retrieving all input data using $request->all()

I have one middleware, which merges customerId to each incoming request.
if ( ! $request->has('customerId')) {
    $request->merge(['customerId' => auth()->user()->userId]);
}

When I'm fetching all input data using $request->all(), It's just works fine.

If $request is of type Illuminate\Http\Request.

But if $request is of type App\Http\Requests then I can't get customerId while fetching all data using $request->all().

Ask on the forums please.


Pass own props to mapStateToProps

The current design makes this un-ergonomic for things like grabbing url params from react-router.
This will also open up use cases like calling connect first to massage data and then grabbing those props

@ganemone this one too?

Could you add a test here?

Looks good, just need lint and flow passing

Done

!merge

@ganemone any ideas when the next version will ship?