click back button return blank page

Hi, I found the click back button not working on some browsers, like ios safari (ios 8.1.1, not sure if other version works.) and also some browsers which run on android.
these browsers all return a blank page after click the history back button.
this is the demo issue link:
http://test.chemm.com/animsition/
your demo works okay in my iphone, that's weird.
plz advise why this issue happened.
thx.

ok, now I figure out why this happened. This issue is from Back/Forward cache, solution is add these code to the js:
window.onpageshow = function(event) {
    if (event.persisted) {
        window.location.reload()     // you can add whatever you want to execute
    }
};

or:
function Reload() {
	try {
	    var headElement = document.getElementsByTagName("head")[0];
	    if (headElement && headElement.innerHTML)
	        headElement.innerHTML += " ";
	    } catch (e) {}
	}

	// ! Reloads on every visit in mobile safari
	if ((/iphone|ipod|ipad.*os 5/gi).test(navigator.appVersion)) {
    window.onpageshow = function(evt) {
        if (evt.persisted) {
            document.body.style.display = "none";
            location.reload();
        }
    };
}

the above code is focus for the ios version.


Pull out IndexSiteSteps.sendFollowupMessage

The logic from that method should be pulled out into FollowupMessage class. Also, if mail send fails, then a comment should be posted to Github.

Decided to remove SendEmail logic for now and just respond only via Github comment. Mainly because it introduces too much complexity/configs and risk for failure.

FollowupMessage  not needed for now

@charlesmike pls index

@charlesmike pls index

@amihaiemil Index site action started. I'll let you know of the progress...

@charlesmike pls index

@@amihaiemil the repository's name must match the format owner.github.io or it must have a project website on branch gh-pages

@charlesmike pls index

@amihaiemil Action finished successfully. See logs the logs for more details.
If you notice any inexplicable errors, please open an issue here


Fix ChainerX CMake test dependencies

Previously the main function was included in chainerx via chainerx_testing, if CHAINERX_BUILD_TEST was set. Removed main from chainerx_testing.
Avoid referencing gtest multiple times.
ChainerX submodules are included in chainerx. Removed redundancy.
Use CUDA_ADD_EXECUTABLE instead of add_executable for chainerx_cuda_test.

Jenkins, test this please

Jenkins, test this please

Jenkins CI test (for commit 26c48db0f4aecc2c52ab2b17fa32f17ae0a2c479, target branch master) succeeded!


Trainable joint_label

Hi, thank you for your great work.
I want to make joint_label trainable so I can examine the partitions obtained by Hyperformer.
To this end, I added the following line to the init method of Model class in Hyperformer.py:
self.joint_label = nn.Parameter(torch.randn(num_point, 3))

Then I modified the forward method of unit_vit so it now looks like this:
def forward(self, x, joint_label, groups):
        ## more efficient implementation
        # label = F.one_hot(torch.tensor(joint_label)).float().to(x.device)
        label = joint_label.softmax(dim=1)
        z = x @ (label / label.sum(dim=0, keepdim=True))

        # w/o proj
        # z = z.permute(3, 0, 1, 2)
        # w/ proj
        z = self.pe_proj(z)
        e = z @ label.T

I'm training the network on the UCLA dataset, but the joint_label.softmax(dim=1) is hardly changing.
What am I getting wrong here?

Hi,
your implementation seems correct. I would suggest:

try to increase the number of predefined groups, as the actual group dimension will be determined by the training (some groups will be assigned no joints at all)

try to initialize the  label matrix with torch.ones, since torch.randn() tend to generate small values around zero, which could be a reason for the slight change

try to train on NTU datasets, and see whether it makes a difference

Please give me your feedback after trying out my suggestions.

or maybe you just made a simple mistake, the code you provided seems to use the joint_label given as input, instead of the self.joint_label parameter. In such situation, the external joint_label input will not be optimised unless you have explicitly added it to the optimizer.

Thank you for the quick response!
I forgot to mention that I propagated the learnable version of joint_label instead of the fixed one so I don't believe this is the source of the problem. I also verified it by checking that the optimizer has a grad value for joint_label (the gradients are very small though, which is what I'm trying to solve).
Will check the options that you've mentioned and see if they resolve the issue.

Wanted to update that I've tried your suggestions but unfortunately the problem still persists.
Do you recall perhaps how you implemented it?

Wanted to update that I've tried your suggestions but unfortunately the problem still persists. Do you recall perhaps how you implemented it?
I double-checked my implementation by taking a fixed joint_label and computing e in two ways:

Your original code
By transforming the joint labels to one-hot vectors so I get the binarised partition matrix H (as it's denoted in the paper) and use my implementation

I got identical values for e using either way.

I still think your implementation should be correct, but unfortunately the exact code I used is now unavailable (the work is done during my internship at the company).
However, I am thinking if there's some gradients, no matter it's small or not, it will lead to a different group label matrix after training. Then you apply softmax on this learned label matrix, won't it give you a partition?

It gives me the initialised partition, since the gradients are too small to change it


[ENHANCEMENT] Create a Stats Page

Split the Records page into Stats & Records pages.
Records tables would stay on Records page, bottom section with refreshing tables and stats would move to Stats page.
Open the page with all time stats.
Have filters to view individual seasons. Allow selecting multiple seasons (ex. To compare first 3 vs last 3 seasons)
Some new stats and features to add for this page:

Schedule Comparisons

Same as Other Schedules google sheet and as this site: https://theffhub.com/ScheduleComparison?site=sleeper&leagueId=925192539278303232&userId=&s2=&swid=

Charts for wins/points/standings over the course of each season and all time

Could use Chris's charts for these with some tweaks.

Created with 5f6bcc673a042bd86e05ad56c9c66562e3a6aea6


Update es.json - spanish translation according to the official NOSOLOROL

Adding the Spanish translation according to the official translation of the NOSOLOROL publisher.
Changes / Comments
Adding the Spanish translation according to the official translation of the NOSOLOROL publisher.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hello @MiguelZamoraRey
Your Pull Request includes changes to translation files. Unfortunately, we no longer accept translations submitted outside of Crowdin. To learn more about Crowdin and how to submit translations through Crowdin, please see our article on How to Become a Translation Volunteer. Thanks for all you do!


Updates for week of Apr 22

clarify when ?setdefaultversionid is required when creating a new Resource

one or more Versions, for example:
rID doesn't exist
POST /GROUPs/gID/RESOURCEs/rID/versions + body=collection of Versions
collection is a map (no order) so we need guidance on which is to be "default"

clarify that ?setdefaultversionid is processed before we apply the Resource's properties (IOW, we update the new "default version", not the old one) - for example:

defaultVersionID=v3
PUT /GROUPs/gID/RESOURCEs/rID?setdefaultversionid=v5
v5 is updated, not v3

Approved on the 4/25 call


Provide working plunkr to show multimodule setup

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[x] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
I cannot figure out how to use ngx-translate throughout a main-module and several child-modules. I have a shared library that is included into the main-project, but I cannot manage to get them both to work with the same instance of ngx-translate. My child-module is in a separate library, installed via npm and then imported into the project.
Expected/desired behavior
Have a working plunkr example where I can see how I have to declare the modules.
Reproduction of the problem
There is a plunkr here, that kind of works, but still does not do the translation: https://plnkr.co/edit/LVmIuI1Xw9vFn0IuC2jW?p=preview
What I've tried
I basically managed to get every case working, BUT the one I need:

No provider for TranslateService
No provider for TranslateStore etc.

Right now I get No provider for TranslateService and this is my setup:
shared.module:
@NgModule({
    exports: [
        TranslateModule
    ],
    imports: [
        TranslateModule
    ]
})
export class SharedModule {
    static forRoot(config: any): ModuleWithProviders {
        return {
            ngModule: SharedModule,
            providers: [
                config.apiConfig,
                _more providers_
            ]
        };
    }
}

app.module:
// AoT requires an exported function for factories
export function createTranslateLoader(http: HttpClient) {
  return new TranslateHttpLoader(http, './assets/i18n/', '.lang.json');
}

@NgModule({
  imports: [
    SharedModule.forRoot({
      apiConfig: {
        provide: ApiConfig,
        useFactory: LoadApiConfig
      }
    }),
    TranslateModule.forRoot({
      loader: {
        provide: TranslateLoader,
        useFactory: (createTranslateLoader),
        deps: [HttpClient]
      },
      isolate: false
    })
  ],
  providers: [
  ],
  bootstrap: [AppComponent]
})

Please tell us about your environment:

ngx-translate version: 7.2.0

Angular version: 4.3.5

Browser: [all ]

@tommueller I know it's been a while but have you managed to make it work? I'm kinda in the same boat ATM and can't find any working solution anywhere.

@jo-russ Nope, sorry. We actually migrated to transloco in the meantime.


Run tests under wasmtime as part of the CI

@sbc100 I have patches in my wasi-sdk fork that fix the tests and run them via wasmtime; if you don't mind, I'd like to open another PR to port my patches?

@sbc100 I have patches in my wasi-sdk fork that fix the tests and run them via wasmtime; if you don't mind, I'd like to open another PR to port my patches?

Hmm.. this PR is 2.5 years old!  Please feel free to replace it with something newer.

Shall we close this one given #278 has landed for some time?


Full review

Class suggestions (e. btn-primary, btn-info) was not working, so I have fixed the syntax.
In alphabetical order, I have started from the beginning. I have made some little changes and added some default values that will help in learning or reminding.
I am removing some components like custom forms which are no longer used. (default form elements are custom now)
In the latest commits, I will add some missing components like offcanvas, toasts and spinners.

@uguryuruk Thank you for your contributions, we are maintaining the HTML snippet in different folder. I'll post the HTML codes in this repo, so that you can modify it directly. And while we generate it will create the snippet in all possible formats.
Currently I'm in process of creating the snippet-generator plugin, once its done. it will be more useful.
Till now, kindly don't commit. since we should not modify the snippets.code-snippets directly.

@anburocky3 Ok, I am waiting for it. I guess it is like Ashok Koyi's project. I have sent some commits to him but he did't see yet.
As soon as you publish it, I will update that files.

@uguryuruk Sorry for wasting your time, i have created an commit which will list all templates, you can modify it and send an PR.
Contribution Guide: https://github.com/anburocky3/bootstrap5-snippets/blob/master/CONTRIBUTION.md
Currently will close this PR and you can modify the HTML files which is located in src/templates and send a fresh PR.
Thank you for your contribution and sorry for wasting your time.


[$500] Web -  An error occurs in the console when we send a link and an image

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Action Performed:

Login >> Open console
Click on plus sign >> Create a chat
Copy and paste a link in the chat >> Click on send
Click on plus sign >> Click on "Add attachment"
Select an image >> Send image (In this instance we can see the errors in the console, error: "Unhandled Promise Rejection: DataCloneError: Failed to store record in an IDBObjectStore: BlobURLs are not yet supported.")

Expected Result:
This process should be done without errors.
Actual Result:
This process is done with errors.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/93399543/202feda1-997b-4666-b5d6-9c27b8524b41
Expensify/Expensify Issue URL:
Issue reported by: @eusalazar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694188803526719
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016f8d9f9c0ae33a1d
Upwork Job ID:<PHONE_NUMBER>381185536
Last Price Increase: 2023-09-12

I think this is browser limitations, and only happens when you're using Private Mode.

Blob URLs are temporary URLs that represent the data of a Blob or File object. They are typically used to display images or other media directly in a browser window. The error message is indicating that Blob URLs are not supported in the context where you're trying to use them (possibly inside IndexedDB).

Yes can confirm that this is happening only on the Private Mode.

@zukilover @EliteDevSolution @studentofcoding Did you happen to check the root cause (besides the bug being reproducible in private mode only) i.e. what is causing this to fail in private mode and is it something that we can fix?

@s77rt This seems to be a webkit bug. It's been reported here before: https://bugs.webkit.org/show_bug.cgi?id=188438

Not overdue. I think we can close being a browser bug. cc @JmillsExpensify @isabelastisser

I have already several checks about this issue.
Unfortunately I can not reproduction such as above issue.

Closing this based on the comments above.


Fx.0 format cannot work

method _compose_float_string() in file _output.py (line: 219)
this fix will cause error .write method output of a FotranFormatWritter('Fx.0')
if (d == 0):
return '*' * w

Hi @brendanarnold, I'm running into this same bug.
In FORTRAN when I format a float with 0 precision, like "f6.0", I get the number with a decimal with no trailing numbers.
Example:
program printformat
implicit none

    real :: number

    number = 99999.01

    Print "(f6.0)", number

end program printformat

OUT:
99999.

But in Python with fortranformt when I try the same formatting, I get ******
from fortranformat import FortranRecordWriter
record_line = FortranRecordWriter("f6.0")
record_line.write([99999.01])

OUT:
'******'

Which version are you using? I've tried to recreate locally and it seems to work

Sorry, my bad that test was actually after I had applied some local changes - verified a bug

Fix is now in 1.1.0


Binary Releases

Having a link to download some binary releases would be super helpful for non-developers.

Windows
Intel Mac
Arm Mac
Intel Linux

This issue is still important, but is being removed from the "Ready-As-Example" milestone. It's a "nice to have" thing for developers, but totally not required.

Much of this work is happening in #31.

Intel Mac version runs on Arm Mac, Arm Mac can be a separate issue. This issue is likely complete, and will be closed once Linux releases are tested, and the README is updated.

Releases aren't yet signed, but that can be resolved on a per OS basis. Complete: https://github.com/TheDiscordian/disco-chat/releases


Testcafe fails to load universal-cookie correctly

What is your Test Scenario?
When running tests that import universal-cookie testcafe throws a type error.
What is the Current behavior?
When trying to use TypeScript and universal-cookie testcafe errors with:
TypeError: universal_cookie_1.default is not a constructor

This appears to be something odd in the way testcafe is building the typescript as universal cookie works fine in other situations.
What is the Expected behavior?
The tests run and pass or fail based on the actual test case.
What is your web application and your TestCafe test code?
https://github.com/st3mc5/testcafe-bug

Your website URL (or attach your complete example):

Your complete test code (or attach your test files):

 import Cookies from "universal-cookie";

fixture`Getting Started`.page`http://localhost:3001/`;

test("My first test", async (t) => {
  const cookies = new Cookies();
});

Your complete configuration file (if any):

none

Your complete test report:

 yarn testcafe -c 5 chrome:headless './e2e/cookietest.e2e.ts'
yarn run v1.22.10
$ /home/steve/alinta/testcafe-bug/node_modules/.bin/testcafe -c 5 chrome:headless ./e2e/cookietest.e2e.ts
 Running tests in:
 - Chrome 89.0.4389.90 / Linux 0.0

 Getting Started
 ✖ My first test

   1) TypeError: universal_cookie_1.default is not a constructor

      Browser: Chrome 89.0.4389.90 / Linux 0.0

         2 |import Cookies from "universal-cookie";
         3 |
         4 |fixture`Getting Started`.page`http://localhost:3001/`;
         5 |
         6 |test("My first test", async (t) => {
       > 7 |  const cookies = new Cookies();
         8 |});
         9 |

         at <anonymous> (/home/steve/alinta/testcafe-bug/e2e/cookietest.e2e.ts:7:19)
         at <anonymous> (/home/steve/alinta/testcafe-bug/e2e/cookietest.e2e.ts:8:71)
         at __awaiter (/home/steve/alinta/testcafe-bug/e2e/cookietest.e2e.ts:4:12)
         at <anonymous> (/home/steve/alinta/testcafe-bug/e2e/cookietest.e2e.ts:6:35)

   2) undefinedFailed to load the page at "http://localhost:3001/".
      Increase the value of the "pageRequestTimeout" variable, enable the "retryTestPages" option, or use quarantine mode to perform additional attempts to execute this test.
      You can find troubleshooting information for this issue at "https://go.devexpress.com/TestCafe_FAQ_ARequestHasFailed.aspx".

      Error details:
      Failed to find a DNS-record for the resource at "http://localhost:3001/".

      Browser: Chrome 89.0.4389.90 / Linux 0.0

 1/1 failed (0s)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Screenshots:
![image](https://user-images.githubusercontent.com/74217363/133962582-8e7808af-05e6-4767-ba09-f8f9c9d55a70.png)

Steps to Reproduce:

Using example codebase run yarn testcafe -c 5 chrome:headless './e2e/cookietest.e2e.ts'

Your Environment details:

testcafe version:                   1.16.0
node.js version:                    v12.19.1
command-line arguments:  yarn testcafe -c 5 chrome:headless './e2e/cookietest.e2e.ts'
browser name and version: Chrome but N/A
platform and version:          WSL Ubuntu and Azure Piplines
other:                                   May be related to ESM loading: https://github.com/DevExpress/testcafe/issues/6535 ?

This is not an issue with TestCafe. Please refer the following thread:
https://github.com/reactivestack/cookies/issues/187
You can fix the issue using the TestCafe configuration file (https://testcafe.io/documentation/402638/reference/configuration-file) with the following content:
{
  "compilerOptions": {
    "typescript": {
      "options": {
        "esModuleInterop": true
      }
    }
  }
}


Home Assistant won't start on DSM 6

**Home Assistant release (hass --version):**20150526-1
**Python release (python3 --version):**3.4.1-4
**Component/platform:**DSM 6.0 fresh install on a Synology 215j
Description of problem:
The package won't start : launch failure during repair package

That version is a year old.. Not a maintained package
On Fri, Apr 1, 2016, 01:17 alexkoch89<EMAIL_ADDRESS>wrote:

*Home Assistant release (hass --version):*20150526-1
*Python release (python3 --version):*3.4.1-4
*Component/platform:*DSM 6.0 fresh install on a Synology 215j
Description of problem:
The package won't start : launch failure during repair package
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/issues/1668


Fix prerendering logic

Hi,
Thank you for creating this wonderful package!
I discovered an alternative method to measure the actual size in the contexts. By using the width and height parameters of the measure function instead of placing and hiding contents, we can resolve the skipped numbering issue mentioned in issue #26.
Please review this change. I appreciate your consideration.

Hi ✨
Wonderful! I'll take a look at the code and run some tests before merging
Thank you!


New Crowdin updates

Crowdin decided to remove these, I guess they somehow got removed in the source file. They've been added back in cf6fb37dc59da60dd5110ca0a721422ae418ae71 so I've just been waiting for it to resync.


mec15xxevb_assy6853: tests/boards/mec15xxevb_assy6853/i2c_api/ failed

Describe the bug
test_i2c_pca95xx in tests/boards/mec15xxevb_assy6853/i2c_api/ has assertion failure.
To Reproduce
Steps to reproduce the behavior:

west build -b mec15xxevb_assy6853 -p auto tests/boards/mec15xxevb_assy6853/i2c_api/
west flash
See error

Expected behavior
Test suite should pass.
Logs and console output
*** Booting Zephyr OS build v2.5.0-rc1-133-gdfd8e7522d11  ***
Running test suite i2c_test
===================================================================
START - test_i2c_pca95xx
E: Write error: -60

    Assertion failed at WEST_TOPDIR/zephyr/tests/boards/mec15xxevb_assy6853/i2c_api/src/main.c:48: test_i2c_pca95xx: (ret == 0 is false)
Failed to write data to i2c device
 FAIL - test_i2c_pca95xx
===================================================================
Test suite i2c_test failed.
===================================================================
PROJECT EXECUTION FAILED

Environment (please complete the following information):

Linux Ubuntu 18.04.4 LTS
zephyr-v2.5.0-rc1-133-gdfd8e7522d11
Toolchain zephyr-sdk-0.12.0

need to make some jumpers setting to change to use I2C01 port according to the README.txt file in tests/boards/mec15xxevb_assy6853/i2c_api/, this will be passed.

tested after jumper setting according to README.txt file, it was passed.
*** Booting Zephyr OS build v2.5.0-rc1-191-g8595a390a7b5  ***
Running test suite i2c_test
START - test_i2c_pca95xx
PASS - test_i2c_pca95xx
Test suite i2c_test succeeded
PROJECT EXECUTION SUCCESSFUL


[BUG] Stock tracking information alignment

Describe the bug
The alignment of the table is not consistency. For example, the customer name, will it be good to make it consistency or is there a purpose to be like this?
This is took from https://demo.inventree.org/stock/item/232/
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://demo.inventree.org/stock/item/232/
Scroll down to 'Stock Tracking Information'

Expected behavior

Screenshots

Deployment Method

[x] Docker
[ ] Bare Metal

Version Information
Demo

One more, after the stock returned from customer, I couldn't find it in the location.

Hi @wengtad thanks for the report! If you submit more of them please include the version information block as the demo is changing daily and it adds some steps to find out what version contains a bug.
Would you be willing to submit a fix? Seems to be a simple css / html thing. You do not have to but I always try to encourage ppl. to try with the easy fixes. You could always ping me for advice / to review.

One more, after the stock returned from customer, I couldn't find it in the location.

Could you try to reproduce with full steps and check if you looked in the same location as you returned it to? It is easy to switch the different locations in the head, which happens to me sometimes when debugging. If it still appears pls. open a new issue so we can keep this one clean. Thank you for your intrest in InvenTree!

One more, after the stock returned from customer, I couldn't find it in the location.

Could you try to reproduce with full steps and check if you looked in the same location as you returned it to? It is easy to switch the different locations in the head, which happens to me sometimes when debugging. If it still appears pls. open a new issue so we can keep this one clean. Thank you for your intrest in InvenTree!

Yup, I will open a new issue after my finding, I will try to reproduce using demo site and with steps, not sure whether I use it in a correct way or not.


Create multi-layer topology

Create multi-layer topology based on the scenario manager's request

close this pr due to some conflicts on merge, create pr #104 instead of this one


refactor: Replace Table's fullWidth custom style with USWDS utility class

Summary

Replaces a custom CSS rule with a USWDS utility class

Related Issues or PRs
#2178
How To Test

View in Storybook: http://localhost:9009/?path=/story/components-table--full-width

Screenshots (optional)

Closing this branch, this work was merged in #2316. Thank you for the contribution @sawyerh


Can't create a new env

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

JavaScript Framework
Next.js
Amplify APIs
Authentication, Analytics, REST API, GraphQL API, Storage
Amplify Categories
auth, storage, function, api, analytics
Environment information

# Put output below this line
  System:
    OS: Linux 4.19 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (5) x64 Intel(R) Core(TM) i7-9700KF CPU @ 3.60GHz
    Memory: 1.84 GB / 3.84 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 12.18.3 - ~/.nvm/versions/node/v12.18.3/bin/node
    Yarn: 1.18.0 - /usr/bin/yarn
    npm: 6.14.6 - ~/.nvm/versions/node/v12.18.3/bin/npm
  npmPackages:
    @babel/core: 7.12.10 => 7.12.10 (7.14.5)
    @testing-library/dom: 7.28.1 => 7.28.1 (7.31.2)
    @testing-library/jest-dom: 5.11.6 => 5.11.6
    @testing-library/react: 11.2.1 => 11.2.1
    @testing-library/user-event: 12.3.0 => 12.3.0
    @types/node: 14.14.20 => 14.14.20 (15.12.2, 14.17.3)
    @types/react: 17.0.0 => 17.0.0 (17.0.11)
    babel-jest: 26.6.3 => 26.6.3
    babel-loader: 8.2.2 => 8.2.2
    cz-conventional-changelog: 3.3.0 => 3.3.0 (3.2.0)
    jest: 26.6.3 => 26.6.3
    jest-sonar-reporter: 2.0.0 => 2.0.0
    typescript: 4.0.5 => 4.0.5 (3.9.9)
  npmGlobalPackages:
    @aws-amplify/cli: 4.50.2
    bit-bin: 14.8.8
    create-react-app: 4.0.3
    depcheck: 1.2.0
    jest: 26.6.3
    lerna: 3.22.1
    locize-cli: 7.6.6
    now: 20.1.1
    npm: 6.14.6
    prerendercloud-server: 0.5.0
    roast: 2.3.17
    serverless: 2.10.0
    source-map-explorer: 2.5.0
    vercel: 20.1.1

Describe the bug
I tried to create a new env with amplify env add, choose "prod" as new env's name and Amplify Admin UI for authentication method. It opened Amplify Admin UI in my browser, I validated my choice but once back on my terminal I had an error telling me that I don't have the right permissions.
I retried to create a new env with the same name "prod" but this time I choose AWS profile as authentication method and this is the output:
➜  students git:(master) amplify env remove prod
No environment found with the corresponding name provided
➜  students git:(master) amplify env add
Note: It is recommended to run this command from the root of your app directory
? Do you want to use an existing environment? No
? Enter a name for the environment prod
Using default provider  awscloudformation
? Select the authentication method you want to use: AWS profile

For more information on AWS Profiles, see:
https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-profiles.html

? Please choose the profile you want to use default
⠇ Initializing project in the cloud...An error occurred when creating the CloudFormation stack
✖ Root stack creation failed
AlreadyExistsException: Stack [amplify-students-prod-211214] already exists
    at Request.extractError (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/protocol/query.js:50:29)
    at Request.callListeners (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at Request.emit (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at callNextListener (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:96:12)
    at IncomingMessage.onEnd (/home/paul/.nvm/versions/node/v12.18.3/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/event_listeners.js:313:13)
    at IncomingMessage.emit (events.js:327:22)
    at IncomingMessage.EventEmitter.emit (domain.js:483:12)
    at endReadableNT (_stream_readable.js:1220:12)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  code: 'AlreadyExistsException',
  time: 2021-06-23T20:33:40.810Z,
  requestId: '346e616e-5fe0-4b5c-88d7-e3ce1ad8f5b1',
  statusCode: 400,
  retryable: false,
  retryDelay: 38.3637349356734
}
➜  students git:(master)

Expected behavior
Amplify should have clean the previous attempt
Reproduction steps
amplify env add
=> create a new env with Amplify Admin UI authentication method
FAILS
amplify env add again
=> create a new env with another authentication method
aws-exports.js
/* eslint-disable */
// WARNING: DO NOT EDIT. This file is automatically generated by AWS Amplify. It will be overwritten.

const awsmobile = {
    "aws_project_region": "ap-southeast-1",
    "aws_cognito_identity_pool_id": "ap-southeast-1:3928d3f5-abb9-4cd7-96d5-bbf1e11e601b",
    "aws_cognito_region": "ap-southeast-1",
    "aws_user_pools_id": "ap-southeast-1_toIr1ygEg",
    "aws_user_pools_web_client_id": "233j2993ca074vpr1cod7d0jii",
    "oauth": {},
    "aws_appsync_graphqlEndpoint": "https://fbtuf6bm4bcrllp2ix5sdfmyjq.appsync-api.ap-southeast-1.amazonaws.com/graphql",
    "aws_appsync_region": "ap-southeast-1",
    "aws_appsync_authenticationType": "AMAZON_COGNITO_USER_POOLS",
    "aws_appsync_apiKey": "da2-mgsxxjw53jfyzjhhactyn3fomi",
    "aws_cloud_logic_custom": [
        {
            "name": "rest",
            "endpoint": "https://dxf5rgz8x3.execute-api.ap-southeast-1.amazonaws.com/dev",
            "region": "ap-southeast-1"
        }
    ],
    "aws_user_files_s3_bucket": "applyfuture-students-content162403-dev",
    "aws_user_files_s3_bucket_region": "ap-southeast-1",
    "aws_mobile_analytics_app_id": "1b1231e1f78d402cbc2c5b7de6fc577c",
    "aws_mobile_analytics_app_region": "us-west-2"
};

export default awsmobile;

Based on the description I think this problem relates to the temporary credentials Admin UI service is generating.


[Snyk] Upgrade zod from 3.22.2 to 3.22.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade zod from 3.22.2 to 3.22.4.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 22 days ago, on 2023-10-04.

The recommended version fixes:

Severity
Issue
PriorityScore (*)
Exploit Maturity

Regular Expression Denial of Service (ReDoS) SNYK-JS-ZOD-5925617
696/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.5
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Release notes

Package name: zod

3.22.4 - 2023-10-04Commits:

d931ea3 Lint
8e634bd Fix prettier
4018d88 docs: add @ sanity-typed/zod to ecosystem (#2731)
15ba5a4 docs: add zod-sandbox to README ecosystem links (#2707)
699ccae Export jsdoc with @ deprecated when building (#2717)
dfe3719 Fix sanity-typed links (#2840)
cd7991e fix ulid regex (#2225)
7cb4ba2 Remove stalebot
9340fd5 Lazy emojiRegex
e7a9b9b 3.22.4

3.22.3 - 2023-10-03Commits:

1e23990 Commit
9bd3879 docs: remove obsolete text about readonly types (#2676)
f59be09 clarify datetime ISO 8601 (#2673)
64dcc8e Update sponsors
18115a8 Formatting
28c1927 Update sponsors
ad2ee9c 2718 Updated Custom Schemas documentation example to use type narrowing (#2778)
ae0f7a2 docs: update ref to discriminated-unions docs (#2485)
2ba00fe [2609] fix ReDoS vulnerability in email regex (#2824)
1e61d76 3.22.3

3.22.2 - 2023-08-19Commits:

13d9e6b Fix lint
0d49f10 docs: add typeschema to ecosystem (#2626)
8e4af7b X to Zod: add app.quicktype.io (#2668)
792b3ef Fix superrefine types

    from zod GitHub release notes

Commit messages

Package name: zod

e7a9b9b 3.22.4
9340fd5 Lazy emojiRegex
7cb4ba2 Remove stalebot
cd7991e fix ulid regex (#2225)
dfe3719 Fix sanity-typed links (#2840)
699ccae Export jsdoc with `@ deprecated` when building (#2717)
15ba5a4 docs: add `zod-sandbox` to README ecosystem links (#2707)
4018d88 docs: add @ sanity-typed/zod to ecosystem (#2731)
8e634bd Fix prettier
d931ea3 Lint
1e61d76 3.22.3
2ba00fe [2609] fix ReDoS vulnerability in email regex (#2824)
ae0f7a2 docs: update ref to discriminated-unions docs (#2485)
ad2ee9c 2718 Updated Custom Schemas documentation example to use type narrowing (#2778)
28c1927 Update sponsors
18115a8 Formatting
64dcc8e Update sponsors
f59be09 clarify datetime ISO 8601 (#2673)
9bd3879 docs: remove obsolete text about readonly types (#2676)
1e23990 Commit

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

Comparison is base (8300fdd) 81.66% compared to head (5fa4c7d) 81.66%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   81.66%   81.66%
=======================================
  Files           4        4
  Lines          60       60
  Branches       11       11
=======================================
  Hits           49       49
  Misses         11       11

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


fix(workflow): add load option and fix image tag

Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>What this PR does?:

add load option to pull request workflow so images that are built during integration-tests are loaded into the local docker daemon before running the tests
fix image name in build workflow for node-disk-operator

Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Continuation of https://github.com/openebs/node-disk-manager/pull/548
Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Codecov Report

Merging #551 (a393e57) into master (0e11c48) will increase coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #551      +/-   ##
==========================================
+ Coverage   48.20%   48.38%   +0.17%
==========================================
  Files          73       73
  Lines        3365     3365
==========================================
+ Hits         1622     1628       +6
+ Misses       1588     1580       -8
- Partials      155      157       +2

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/udevprobe.go
65.36% <0.00%> (-1.68%)
:arrow_down:

cmd/ndm_daemonset/probe/addhandler.go
74.22% <0.00%> (+3.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Hide the button

Preparation

[X] No one has asked for this feature before in other issues (even the closed ones)

Describe the feature
Can you do hide the button as a feature because i have an oled display and i don't want the pixel burn in my screen

The apk is perfect

Disable the service in the app or via the notification.


Default value for edges method

In documentation:
func edges(insets: UIEdgeInsets = default) -> Maker

The insets for setting relations for superview. UIEdgeInsets.zero - default insets.

But when I try to ommit UIEdgeInsets value, for example:
tableView.configureFrame { maker in
    maker.edges()
}

I see an error:

Ambiguous use of 'edges'

Yea, you're right, there are two edges methods and it's not possible to use edges(insets:) with default value - bug. I will remove the default value in next update.
Thanks for remark!


⚠ Avatar API Endpoint has degraded performance

In 6dcc3e7, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4742 ms

Resolved: Avatar API Endpoint performance has improved in e78303a after 21 minutes.


[alert_handler] escalation ping response fail question

Hi Michael,
Two clarifications about escalation ping fail:
1). From the spec, responding to esc_p/n ping response requires: resp_p/n toggle with a "1010" pattern. From the current design, if the "1010" pattern does not come right after one cycle of esc_tx_i.esc_p/n, then the integ_fail_o signal will also be asserted. Please refer to the simulation screenshot here:

Is this behavior expected?
If so, could you update the design spec graph please? (I think at timestamp 10-11 12-13, the integ_fail_o should also be set)

2). If escalation response resp_p/n comes with pattern "1010" but not the "correct cycle" (not sure if this is understandable, please refer to the screenshot). Then the ping_ok_o pattern won't recognize it as a ping_response, but count as integ_fail. Is this behavior expected? If so, do you think is necessary to clarify that in the spec as well?

Thanks for your time Michael!
Cindy

Hey Cindy!
So in both cases the design is behaving as intended. The 1010 response must come in the next cycle, no matter what (this is a bit different from the alert rx/tx where we have an asynchronous variant that can tolerate some delay).
I'll update the spec accordingly. Thanks for flagging these discrepancies!

Thanks for the quick response and your confirmation Michael.

Np. One thing to note here is that there is an ambiguity between ping and integrity failures in this particular case. I decided to just allow both to trigger, since otherwise I would have to suppress the integrity failure - which is not preferable from a security standpoint.
Hence, I would expect that the design first raises an integrity alert, and then, after the timeout of the ping timer, it should also raise a ping fail alert in addition to that. Can you double check whether that is actually happening?

Hi Michael, yes, I do see both the signal_int_alert and ping_timeout interrupt triggered in the second scenario :)

ok great, thanks for confirming!


Embedding styles in shadow DOM

When we pass shadow : 'open' or shadow : 'closed' we would probably expect the styles to be included in the shadow root.
Is there any way to accomplish this?
.welcome {
    color: red;
}

React component
import './Welcome.css';

export function Welcome(props: { name: string }) {
    return

Welcome, {props.name}!

        <slot></slot>

}

export default Welcome;

To create the web component
const WelcomeWC = r2wc(Welcome, {
  props: {
    name: "string"
  },
  shadow: "open"
});

customElements.define("welcome-wc", WelcomeWC);

Usage
<welcome-wc>
  </welcome-wc>

I'm wondering the same, no styles seems to be included in the shadow dom

First of all, this would entirely depend on how your bundler/build tool deals with CSS imports, and the constraints you have in your project. With that said, you can make it work, but at the moment it looks like you must roll your own solution.
In my situation, I am using esbuild directly, and one of my constraints is that I must end up with a single JS file as output from the bundler. I found an excellent plugin for the Bun bundler called bun-lightningcss. I use esbuild, but it was dead simple to modify it so that it works with esbuild, since Bun's API is almost identical.
Without modifications, the bun-lightningcss plugin does a couple things:

It creates a virtual module called inject-styles. It exports a function injectStyles that adds all CSS code passed onto it into a `;
}

class StyledHelloWC extends r2wc(App, {
  props: { name: "string" },
  shadow: "open",
}) {
  connectedCallback() {
    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
    // @ts-ignore
    super.connectedCallback();
    this.shadowRoot?.appendChild(template.content.cloneNode(true));
  }
}

customElements.define("hello-tailwind-wc", StyledHelloWC);

And in [vite.config.ts](https://github.com/wataruoguchi/poc-spa-gh-pages/blob/main/packages/react-tailwind-fragment/vite.config.ts) I have
plugins: [
        ...defaultConfig.plugins,
        cssInjectedByJsPlugin({
          injectCode: (cssCode: string) => {
            return `window.__styles = ${cssCode}`;
          },
        }),
      ],

Hi, actually am not using vite, bun_lightningcss, etc..
i have developed my project using react CRA https://create-react-app.dev/. and using SCSS for styles. now i implementing r2wc and its working without shadomDom. but i want to build my with shadomDom enabled. am not sure how to achieve this.
If anyone have idea let me know, it will help me lot.
@danielegarciav @shogoroy @wataruoguchi @seesharper @se-andbjo


Optimize decimal_length_minus_1

As discussed in https://github.com/dtolnay/dragonbox/issues/1, the algorithm for calculating the decimal length is rather inefficient, since the number of digits can be computed with minimal branching using a shr, add, and a table lookup instruction as described here.
The current implementation is:
JKJ_FORCEINLINE static constexpr std::uint32_t decimal_length_minus_1(std::uint32_t const v) {
    assert(v <<PHONE_NUMBER>);
    if (v >= 100000000) { return 8; }
    if (v >= 10000000) { return 7; }
    if (v >= 1000000) { return 6; }
    if (v >= 100000) { return 5; }
    if (v >= 10000) { return 4; }
    if (v >= 1000) { return 3; }
    if (v >= 100) { return 2; }
    if (v >= 10) { return 1; }
    return 0;
}

Which optimizes quite poorly (for example, on x86):
decimal_length_minus_1(unsigned int):
        mov     eax, 8
        cmp     edi, 99999999
        ja      .L1
        mov     eax, 7
        cmp     edi, 9999999
        ja      .L1
        mov     eax, 6
        cmp     edi, 999999
        ja      .L1
        mov     eax, 5
        cmp     edi, 99999
        ja      .L1
        mov     eax, 4
        cmp     edi, 9999
        ja      .L1
        mov     eax, 3
        cmp     edi, 999
        ja      .L1
        mov     eax, 2
        cmp     edi, 99
        ja      .L1
        xor     eax, eax
        cmp     edi, 9
        seta    al
.L1:
        ret

Using a built-in ctlz function, we can make this much more efficient:
#ifdef _MSC_VER
#include <intrin.h>
#endif

JKJ_FORCEINLINE static std::uint32_t ctlz(std::uint32_t v) {
    // Optimize for GCC/Clang when possible.
#ifdef __has_builtin
  #if __has_builtin(__builtin_clz)
    return __builtin_clz(v);
  #endif
#endif

    // Optimize for MSVC
#ifdef _MSC_VER
  #if defined(_M_X64) || defined(_M_ARM64)
    unsigned long leading_zero = 0;
    _BitScanReverse(&leading_zero, v);
    return static_cast<std::uint32_t>(31 - leading_zero);
  #endif
#endif

    // Fallback algorithm
    int count = 0;
    if (v & std::uint32_t(0xffff0000)) {
        v >>= 16;
        count += 16;
    }
    if (v & std::uint32_t(0xff00)) {
        v >>= 8;
        count += 8;
    }
    if (v & std::uint32_t(0xf0)) {
        v >>= 4;
        count += 4;
    }
    if (v & std::uint32_t(0xc)) {
        v >>= 2;
        count += 2;
    }
    if (v & std::uint32_t(0x2)) {
        v >>= 1;
        count += 1;
    }
    return 31 - count;
}

JKJ_FORCEINLINE static std::uint32_t fast_log2(std::uint32_t const v) {
    return 31 - ctlz(v | 1);
}

JKJ_FORCEINLINE static std::uint32_t decimal_length_minus_1(std::uint32_t const v) {
    static constexpr std::uint32_t table[] = {9, 99, 999, 9999, 99999, 999999, 9999999, 99999999, 999999999};
    std::uint32_t y = (9 * fast_log2(v)) >> 5;
    y += v > table[y];
    return y;
}

Most of the bloat is just to ensure we have a portable way to count leading zeros. And this optimizes quite nicely with the built-in optimizations:
decimal_length_minus_1(unsigned int):
        mov     eax, edi
        mov     edx, 31
        or      eax, 1
        bsr     eax, eax
        xor     eax, 31
        sub     edx, eax
        lea     eax, [rdx+rdx*8]
        shr     eax, 5
        mov     edx, eax
        cmp     DWORD PTR decimal_length_minus_1(unsigned int)::table[0+rdx*4], edi
        adc     eax, 0
        ret

For MSVC on x86_64, we have:
unsigned int decimal_length_minus_1(unsigned int) PROC                    ; decimal_length_minus_1, COMDAT
        xor     eax, eax
        mov     edx, ecx
        or      edx, 1
        bsr     r8d, edx
        lea     r9d, DWORD PTR [r8+r8*8]
        shr     r9d, 5
        lea     r8, OFFSET FLAT:unsigned int const * const `unsigned int decimal_length_minus_1(unsigned int)'::`2'::table
        cmp     ecx, DWORD PTR [r8+r9*4]
        seta    al
        add     eax, r9d
        ret     0
unsigned int decimal_length_minus_1(unsigned int) ENDP

And for MSVC on ARM64, we have:
|unsigned int decimal_length_minus_1(unsigned int)| PROC              ; decimal_length_minus_1
        orr         w8,w0,#1
        clz         w9,w8
        mov         w10,#0x1F
        sub         w10,w10,w9
        add         w8,w10,w10,lsl #3
        adrp        x9,|unsigned int const * const `unsigned int decimal_length_minus_1(unsigned int)'::`2'::table|
        add         x9,x9,|unsigned int const * const `unsigned int decimal_length_minus_1(unsigned int)'::`2'::table|
        lsr         w11,w8,#5
        ldr         w8,[x9,w11 uxtw #2]
        cmp         w0,w8
        csethi      w10
        add         w0,w10,w11
        ret

        ENDP  ; |unsigned int decimal_length_minus_1(unsigned int)|, decimal_length_minus_1

Since the fallback ctlz algorithm isn't very good, we could do a simple optimization for MSVC and GCC/Clang, and for another other compiler keep the old algorithm.

Good idea, but may I propose another optimization?
JKJ_FORCEINLINE static std::uint32_t fast_log2(std::uint32_t const v) {
    return 31 - ctlz(v | 1);
}

Change - to ^, and GCC will emit three opcodes less.
...though MSVC prefers -, so may want to ifdef it. (Clang optimizes both equally.)

Good idea, but may I propose another optimization?
JKJ_FORCEINLINE static std::uint32_t fast_log2(std::uint32_t const v) {
    return 31 - ctlz(v | 1);
}

Change - to ^, and GCC will emit three opcodes less.
...though MSVC prefers -, so may want to ifdef it. (Clang optimizes both equally.)

Unfortunately, this fails on 1 important condition (also, using the __builtin_clz may be undefined if v is 0): when v is exactly 1, since it xors out the 1-bit and then sets it to 0.

Ooh that could work nicely too (sorry, my initial reply assumed you meant ctlz(v ^ 1), which would not work).

I'm actually aware of this trick. The reason why I'm hesitant to applying it is that actually it results in slower (even if shorter) code in terms of the uniform average performance, b/c most numbers are long (i.e., have many digits). Of course in practice short numbers occur more often than long numbers so uniform average performance is not the best metric, but it is not a negligible metric neither. Also, I think the resulting performance boost/regression may not be in the robustly measurable range.
However, I think at least the code size benefit is obvious, and I guess it does have a meaningful impact on inlinability.
Currently I cannot afford enough time to carefully analyze the benefit/cost of this. Maybe later someday.

I'm actually aware of this trick. The reason why I'm hesitant to applying it is that actually it results in slower (even if shorter) code in terms of the uniform average performance, b/c most numbers are long (i.e., have many digits). Of course in practice short numbers occur more often than long numbers so uniform average performance is not the best metric, but it is not a negligible metric neither.

Just from the implementation standpoint, this is definitely true for how binary64 calls the algorithm, since by default the policy isn't to trim trailing zeros, so there's a high likelihood even with common floats of having a large number of trailing zeros. But, this isn't necessarily true for binary32 representations, which trim trailing zeros prior to this being called, or if trailing zeros is called for binary64, for common floats.
Also, this effectively amortizes the time of calculating the number of digits, while providing good performance for common cases, which I believe should be a desirable property.

Also, I think the resulting performance boost/regression may not be in the robustly measurable range.

This probably isn't a bottleneck, so I'd have to agree.

Currently I cannot afford enough time to carefully analyze the benefit/cost of this. Maybe later someday.

I look forward to it. This probably minimally impacts overall performance, just due to other bottlenecks in the room, so I understand this is very low priority.

Another candidate: https://godbolt.org/z/czrK918KP.
(Note that the input number is never 0 in our case so we don't need to take care of that special case.)

Another candidate: https://godbolt.org/z/czrK918KP.
(Note that the input number is never 0 in our case so we don't need to take care of that special case.)
I think this implementation will be slightly better than what @Alexhuszagh proposed, as it uses one less table entries. It uses one or two more instructions, but as adc is usually quite slow, I expect this one to actually perform better than the one with adc.
But anyway, I'm leaning towards the side of leaving it as it is, since the suggested changes don't seem to provide a better performance in general. But I will leave this issue open until someone provides a more comprehensive analysis.

Looks good, I'll probably benchmark this with a few contrived examples to see if it makes any difference. If not, I'll close this.

not if i benchmark them first
I took five ilog10 designs, reimplemented them, and tossed them into Quick-Bench: https://quick-bench.com/q/ohRRzsfTxtVf7sAFLZ_D6q8fcWU
ilog10a is the current one as quoted in OP; it simply compares to powers of 10 in descending order.
ilog10b is a trivial variant of ilog10a, which compares to 100000 first, halving the worst-case number of comparisons (but adding one more comparison for large numbers).
ilog10c is OP's proposal, using log2 to find which power of 10 to compare to (except I replaced *9>>5 with *9/32 to remove a set of parentheses - it optimizes back to bitshift).
ilog10d is ilog10c, but another lookup table instead of *9/32.
ilog10e is jeon's counterproposal, which uses addition and shift instead of a conditional on the relevant power of 10.
All are tested four times. First with somewhat randomly distributed odd numbers 1 through 0x1FFFFFFF = 536870911 (it stops below<PHONE_NUMBER> to not hit ilog10a's assert), second with numbers 1 through 255, and third with each number length picked sequentially (length 1 then 2 then 3 then 4...), and finally numbers of uniformly distributed random length.
The answers are quite conclusive: jeon's counterproposal, E, has vastly superior performance characteristics on all input distributions (though D wins on code size ignoring tables, and A wins in readability by beginners, if speed isn't your top priority).
A and B are fast for large and sequential-length numbers - large numbers obviously let the branch predictor always say the same thing, and sequential length yields fully predictable branch schedule (unlike large ones, where every ~5.36th number is 8 digits - hard to predict that kind of non-integers).
But A and B suffer on small inputs, and explode completely if the input length is random and the branch predictor has nothing to go on. C and D have slower best case (jeon is right, adc is slow), but their speed remains constant across the board, and they easily defeat A and B at the random lengths.
E narrowly beats A and B on the cyclic length, and like C and D, it's constant time; it easily wins on the large numbers, and absolutely crushes all competition on the random lengths.

@Alcaro Thanks a lot.

D is uniformly slightly slower than C. (Their speed does vary a little; I suspect either it's just noise, or the branch predictor does something to the adc.)

This is expected. Multiply by 9 can be done very fast by abusing the lea instruction, so *9/32 is nothing but lea and then shr. Definitely better than a table lookup.
It seems that GCC actually favors C over E. Don't know why. Looking at the assembly, I think Clang actually does a better job here, so it's likely GCC's fault.
So it sounds reasonable to actually adapt ilog10e. I'll update the code shortly.

ilog10e is jeon's counterproposal, which uses a different table, such that the conditional can be replaced with an addition and a few bitshifts. A quite clever design.

This is not my idea. I recall I saw this when I was reading Dr. Lemire's blog. I'll find the one who suggested this and correctly attribute the inventor through the comment.

That'd be George Spelvin, and everyone who contributed to the versions his idea is based on (Josh Bleecher Snyder,
Kendall Willets, and of course Daniel Lemire himself). https://lemire.me/blog/2021/06/03/computing-the-number-of-digits-of-an-integer-even-faster/#comment-586067
Willets' idea (the blog OP, henceforth known as G) is also worth testing. While it needs a bigger table (256 bytes for 32bit inputs, 1KB for 64bit - E only needs 36 or 152, respectively), it needs fewer instructions, at least for 32bit inputs on 64bit hardware. I also extended it to 64bit inputs, to see how much it suffers from the extra adc and table access. https://quick-bench.com/q/-Ar6_ubBXxxUeesHRZB1yQdgf0k
Turns out G is the fastest; G64 is identical to E, but G32 defeats both. And on GCC, the two Gs are identical, and a fair bit faster than E. I don't know how the extra adc is free; I vaguely suspect instruction scheduling shenanigans, possibly involving false dependencies. https://quick-bench.com/q/07u1DKbzK9MOD8HH7Nj8QGUK7K8
Considering how rarely it's called (once per formatted integer), I'm personally leaning towards favoring size over speed (i.e. E), but it's your choice.
(C++20 added ilog2 to the standard library, as std::countl_zero / std::bit_width. C++26 should totally add an ilog10.)

Since I implemented a faster jeaiii-style printing algorithm (692f67f4baa76cda7327c1cf43e0aa8277950a8c) which does not compute the length upfront (rather, it does a binary search and specialize each branch), there is no need to implement the trick discussed in this issue. So I close this.


CB-10580 EnvAdmin can perform actions on datalake, datahubs - fix tests

See detailed description in the commit message.
http://ci-cloudbreak.eng.hortonworks.com/job/cloudbreak-real-ums-test/2832

retest this please

retest this please


Documentation Temnplates lose highlights when viewing individual templates

TypeScript Version:  2.6.0-dev.201xxxxx
Expected behavior:
When I visit documentation for templates the list of templates are shown and on the side bar templates is highlighted. I expect that when I visit each template the menu should keep templates as highlighted as i am within that section. For example visiting this page should not remove highlight from templates.
I expect that Templates menu would stay highlighted as long as I am visiting pages under Templates section.
Actual behavior:
When visiting template list page menu indicates that I am on template page where as on visiting any sub pages it keeps the "Declaration Files" section open but removes highlight from Templates section which makes it unclear which section this file belongs to. User can see from URL that this file belongs within template section as a work around. Sharing this link with others also cause confusion as they dont know which section to go to see more templates.

This isn't relevant in the new design


Detect click on each radar chart labels

I want to detect and get radar chart label when there's click on it to append a particular text for each label.
How can I do this ?
Thanks in advance !

@Voldacar21 this is not currently possible and would require substantial changes to the radar scale to make this work

In fact, I tried to find something via the click event but I found nothing.
It's too bad !
A provision in a future release then?
Thanks anyway !

Is there any update on this?

I have created #6549 which links to a JSfiddle containing a workaround to achieve this for version 2.8.0.


readme: add debian based deps

@cfernhout @erezsh I installed this on my linux box to make sure the instructions work, and this is what I needed to install 👍🏻

LGTM


Wrong collapsed text when setting a NSAttributedString using both HTML and "regular" attributes.

Hi there,
I'm setting my attributed string the following way:
let fontFamily = "font-family: \(fontName);"
let fontSize = "font-size: \(fontSize);"
let color = "color: \(htmlRGBColor);"
let htmlString = "
\(self)
"

let attributedString = try! NSMutableAttributedString(
    data: htmlString.dataUsingEncoding(NSUTF16StringEncoding)!,
    options: [NSDocumentTypeDocumentAttribute:NSHTMLTextDocumentType],
    documentAttributes: nil)

if let paragraphStyle = aParagraphStyle {
    attributedString.addAttribute(NSParagraphStyleAttributeName, value: paragraphStyle, range: NSRange(location: 0, length: attributedString.length))
}

For some reason, when I set the text alignement and the line spacing through the additional paragraph style attribute, bot the collapsedAttributedLink and the ellipsis are not displayed anymore.
I observed that getCollapsedTextForText() consistently returns the expanded text (instead of the collapsed one) and I supsect it is because, in this case, linesForWidth(_ :) always returns a a wrong array of CTLinesRef, but I'm struggling to go forward in the debugging.
Any idea?
Thank you for your help!
UPDATE :
A workaround for the specific line-height issue (one of the variables I set on my NSParagraphStyle) is to set it via HTML styles.
However, I cannot set the text alignement using the same technique as the text-align property is exclusively available with block tags and a bug in iOS (libxml?) prevents me from doing so as it adds an unwanted extra space at the bottom of the string (see this issue)
Hence, being able to add most of the attributes via the addAttributed:value:range method would be really convenient 😄

Hi
For me when I set text it works well but when I set attributed text in ExpandableLabel it shows an empty label even if the attributed text has no attributed. I tried both View Controller and also in
TableViewCell.
This shows the text with “...read more” tailing
label.text = text
This shows an empty label
label.attributedText = NSAttributedString(string: text)


Avatar migrations/seeds

Description
Add migrations for user/team avatars
Add table for generic predefined avatars
Add seeds for avatars
Type of change

[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] Created migrations. Migrated successfully and confirmed via pgAdmin
[x] Rolled back migrations. Migration rolled back successfully and confirmed via pgAdmin
[x] Ran seeds successfully and verified via pgAdmin

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[x] 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
[ ] 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] There are no merge conflicts

:just_do_it:

 Create migrations/seeds for user/team avatars (avatar phase 1)


more utils like raffle manager?

Hello!
I've been perusing the latest commits, and noticed that there is a raffle manager built-in. Are there plans to include any other type of stream-specific functionality, such as bank heists, betting, etc? Or would you leave that up to pull requests?

I am planning to add more functions but no complete stream-specific functionalities. Like all the utilities in twitch-irc, these functions are just making it easier to code your application. The reason why I am not adding full functionalities like that is because I want to keep tmi.js (and twitch-irc) as a framework. However, I am planning to create a forums and will be sharing some scripts, such as bank heists, betting, rates limit, etc.. but I'll do that only when I'm done with the website.

Ok, makes sense! With that said, does that mean they should be broken out into a separate module? Perhaps just a simple set of stream tools by itself?

@celluj34 Sorry for not responding, I never saw your reply.. I am thinking about creating simple scripts that everyone will be able to use instead.. Still need to think about it though :smile:


[SPARK-16029][SPARKR] SparkR add dropTempView and deprecate dropTempTable

What changes were proposed in this pull request?
Add dropTempView and deprecate dropTempTable
How was this patch tested?
unit tests
@shivaram @liancheng

Test build #60757 has started for PR 13753 at commit 0b2fe6f.

LGTM pending Jenkins. Thanks!

Test build #60762 has started for PR 13753 at commit d70154c.

Test build #60757 has finished for PR 13753 at commit 0b2fe6f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60757/
Test PASSed.

Test build #60762 has finished for PR 13753 at commit d70154c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60762/
Test PASSed.

Merged build finished. Test PASSed.

Thanks @felixcheung @liancheng - LGTM. Merging this to master, branch-2.0
Are there any other catalog functions that have changed in Spark 2.0 that we also expose in SparkR ? Or are createTempTable and dropTempTable the only ones ?

Only registerTempTable and dropTempTable have name changes.

@shivaram I think we've covered all of them.


problem with spiderifier code on Mapbox.

Hi, I followed the code but it doesn't work. The map does not show up. I get the code here: https://github.com/bewithjonam/mapboxgl-spiderifier/blob/master/docs/example-mapbox-gl-cluster-spiderfy.html
Would you be kind enough to help me identify what's wrong here?
https://glitch.com/edit/#!/emphasized-atom-clover?path=index.html%3A85%3A33
This is a cluster map I made: https://glittery-unexpected-comic.glitch.me
I want to add spiderifier with popup box.
Thank you very much!

Which version of mapbox-gl-js are you using. What error are you getting?
The examples demonstrate all the different options in this library. You can find all the working examples here:
https://bewithjonam.github.io/mapboxgl-spiderifier/docs/index.html (All examples list)
https://bewithjonam.github.io/mapboxgl-spiderifier/docs/example-mapbox-gl-cluster-spiderfy.html

I am using the latest mapbox-gl-js. I saw your examples. I made a cluster map with spider legs (that;s your example 2). My problem now is to add popup box to spider legs (like your example 3). However, I add the code you gave on example 3, https://github.com/bewithjonam/mapboxgl-spiderifier , the spider leg simple disappears and no popup box either. The problem seems to be this area:
initializeLeg: function(spiderLeg) {
var $spiderPinCustom = $('', {class: 'spider-point-circle'});
This is my code. Would you be kind enough to help me what's going wrong with the customize pin popups here? From line 404 and below. The codes above this line work well.
https://github.com/lixiao321/spider-leg-map/blob/main/leg without popup
Thank you very much!

Please check this doc sample. https://github.com/bewithjonam/mapboxgl-spiderifier#custom-pins-with-popup
You need to use the initializeLeg option while creating spiderfier and give setup popups on hover.  In case of filing issues, please provide a reproable jsfiddle or something like that for quick reference. The html file you attached has errors.
I am closing this issue. Not sure, if I will be able to help like this in future.  Will make the enhance the docs better in a couple of days.
Fix for the html that you gave. https://gist.github.com/bewithjonam/68e2fb92d692b110e96e12c9dccb9fbe
Changes mentioned below.
<script
      src="https://code.jquery.com/jquery-3.5.1.min.js"
      integrity="sha256-9/aliU8dGd2tb6OSsuzixeV4y/faTqgFtohetphbbj0="
      crossorigin="anonymous"></script>

      var map = new mapboxgl.Map({
        container: "map",
        style: "mapbox://styles/mapbox/dark-v10",
        center: [40.876, 34.577],
        zoom: 5.81
      });

      spiderifier = new MapboxglSpiderifier(map, {
        customPin: true,
        initializeLeg: function(spiderLeg) {
          var popup = new mapboxgl.Popup({
            closeButton: true,
            closeOnClick: false,
            offset: MapboxglSpiderifier.popupOffsetForSpiderLeg(spiderLeg)
          });

          popup.setHTML(spiderLeg.feature.description);
          spiderLeg.mapboxMarker.setPopup(popup);

          $(spiderLeg.elements.container)
            .on('mouseenter', function(){
              popup.addTo(map);
            map.getCanvas().style.cursor = "pointer";
            })
            .on('mouseleave', function(){
            map.getCanvas().style.cursor = "";
              popup.remove();
          });
        }
      });


Homepage article card not displaying correct aspect ratio [Safari v15.4]

https://carbondesignsystem.com/#latest-news-and-articles
current [Safari 15.4]

expected

@jeanservaas Another thought I had for these article cards will help solve for when images are on white or g10 backgrounds. Maybe the bottom card can be g100 as we did for w3.ibm.com/design


PIP command is not functional

'pip' is not recognized as an internal or external command,
operable program or batch file.
pip install --upgrade --upgrade-strategy only-if-needed c:\temp\sqlmlutils-0.7.2.zip

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

ID: bb790fba-2d19-b1a7-3935-6ec5f9207fcc
Version Independent ID: f350cf6b-0fa8-6d76-8433-f3f752e62ecf
Content: Install Python packages with pip - SQL Server Machine Learning Services
Content Source: docs/advanced-analytics/package-management/install-additional-python-packages-on-sql-server.md
Product: sql
Technology: machine-learning
GitHub Login: @garyericson
Microsoft Alias: garye

@JamieL22 Thank you for the feedback, please refer to pip installation for instructions on how to install.
The documentation has been updated with a link and is pending review.


added batch mode to plugin install documentation

Signed-off-by: ariamarble<EMAIL_ADDRESS>Description
Adds documentation of the --batch option for the installation of plugins
Issues Resolved
#731
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @ariamarble, the commit from my inline comment suggestion is flagging the DCO signoff error.
Could you instead of "committing suggested change," just make a separate change and do the regular DCO signoff with your name?
Sorry i should have mentioned that - i always use the inline comment tool to be precise, but i guess the "commit suggested change" feature creates a garbled DCO signature.
thanks,
Alice


add min and max hops flags for advertise command

Adding min and max hops flags to bos advertise comand.
Files changed:
bos
services/advertise.js

https://github.com/alexbosworth/balanceofsatoshis/issues/156

Made the changes.


Request for rtl8723bu driver on Intel NUC

This driver is already present on Generic x86_64 but we don't offer ESR releases for that device type.
We will need to consider the extra space required and whether or not this driver will fit in the rootfs of Intel NUC.

It looks like this firmware was added in v2.89.5+rev1 but has not been backported or included in any ESR releases.
https://github.com/balena-os/balena-intel/commit/3e1e1c659fa90d145e82677abc16fa33e0dc750b


Add table aws_glue_crawler

References
https://docs.aws.amazon.com/cli/latest/reference/glue/get-crawlers.html
I'd like to have it in order to retreive which crawlers are running (state field)
Thanks
Matteo

Hi @matteoredaelli, Thank you for raising this issue with us. We will look into it.

@matteoredaelli, we have added the table aws_glue_crawler, it will be released today.

thanks! I can see it!!!
Bye
Matteo


Error running Llama2 example on AMD Ryzen 7 8845HS

Hi,
I am trying to run the llama2 example and have followed the transformers example setup. However, the ctypes module which qlinear is dependent on is giving a FileNotFoundError
Traceback (most recent call last):
...
File "C:\Users\...\RyzenAI-SW\example\transformers\third_party\_ffi\base.py", line 64, in _load_lib
    lib = ctypes.CDLL(lib_path[0], ctypes.RTLD_GLOBAL)
  File "C:\Users\...\anaconda3\envs\ryzenai-transformers\lib\ctypes\__init__.py", line 374, in __init__
    self._handle = _dlopen(self._name, mode)
FileNotFoundError: Could not find module 'C:\Users\...\RyzenAI-SW\example\transformers\third_party\lib\tvm_ipu.dll' (or one of its dependencies). Try using the full path with constructor syntax.

I have confirmed that the tvm_ipu DLL file is present.
My configurations are as below:

Driver - AMD Ryzen 7 8845HS
OS - Windows 11
Using Microsoft (R) C/C++ Optimizing Compiler Version 19.29.30154 for x64

In which step you getting this error? Are you following this page starting from Step 1?
https://github.com/amd/RyzenAI-SW/tree/main/example/transformers
Also this version requires Visual Studio 2019 "Desktop development with C++" package for Step 3 "Build dependencies"
We are going to release new version very soon that will require Visual Studio 2022.

I have followed through from step 1 (download repo) to step 3 (build dependencies) and am only getting an error when I ran python run_awq.py --w_bit 4 --task quantize. The error occurred when importing qlinear.
I've also installed the "Desktop development with C++" package with Visual Studio 2019 Enterprise. Will await the new version in the meantime.

Your error is very odd because this llama2 flow was published in February month. I know many people have already tried it, and everything worked for them by following the steps, but I had not heard about this error before.
Having said that
Are you using Conda CMD prompt (not Powershell)?
Did your step 3 (pip install ops\cpp --force-reinstall) was successful?
If the above two are correct, can you try installing this in your conda environment? This is just a suggestion without knowing it will solve the issue or not
conda install -c conda-forge cffi

You can try the above what I suggested.. Otherwise you might look into the current "main" branch where we published the new flow. I am closing this issue now


Bootstrap 5.0.0beta1 requires popper.js >= 2

The latest Bootstrap version (5.0.0beta1) depends on popper2. Unfortunately this gem depends on popper_js gem < 2. And the 2.0 version is not available :(

https://popper.js.org
It's only available through npm?

https://popper.js.org
It's only available through npm?


Snap 2760

Provided implementation for fetching ldap groups of the user.
The code checks if the memberOf Attribute is present in user information & uses that .
If it is missing, then it recursively checks the groups , if the user is a member of the group or not.
If it is then the group & all its parents groups are added to the returned list.
Refactored the existing code of fetching users of the ldap group to make the code common to both the requirements ( of fetching users of a group & for fetching all the groups of the user)
Is precheckin with -Pstore clean?
Precheckin is yet to be run
ReleaseNotes changes
(Does this change require an entry in ReleaseNotes? If yes, has it been added to it?)
documentation needs to be updated.
Other PRs
snappydata repo pr
(Does this change require changes in other projects- snappydata, spark, spark-jobserver, aqp? Add the links of PR of the other subprojects that are related to this change)

@sumwale @kneeraj @ashetkar  @suranjan  , pls update ....


Merge aframe-shaders into this repo

We're no longer using https://github.com/aframevr/aframe-shaders so we could close that repo and add another entry here to store the community shaders.

@dmarcos @ngokevin we agree on this one too right?

Yeah. The shader entry is in, just need to update the Inspector branch.

:+1:


Docker-compose does not activates all the containers

Description
Upon running the <docker-compose -f docker-compose.yml -f docker-compose.override.yml up> all the containers are not started mentioned in the docker-compose file. Some containers throw certain errors mentioned below
To Reproduce
◦	docker-compose up -d
◦	docker-compose stop ibf-dashboard
◦	cd interfaces/IBF-dashboard
◦	npm start

Expected behaviour
All containers mentioned in the docker-compose file /IBF-system/docker-compose.yml be up and running.
Application should load and respond properly on the localhost:4200.
Screenshots
Attached below.
Desktop (please complete the following information):

OS: MAC OS
Browser : chrome
Version : 92.*

Workaround
The containers ibf-system_ibf-pipeline , ibf-system_ibf-api-service do not start
The container  and  do not start automatically via the docker compose up command. The workaround for this would be, manually start the container  and  and run this command.

docker compose exec ibf-pipeline python3 runPipeline.py

These below commands come in handy when you face such issue:
1 docker-compose -f docker-compose.yml -f docker-compose.override.yml up
2 docker compose exec ibf-api-service bash
3 docker compose exec ibf-pipeline python3 runPipeline.py
4 docker-compose exec ibf-api-service npm run seed
5 docker-compose up -d ibf-api-service
6 docker-compose logs -f ibf-api-service
7 docker-compose restart ibf-api-service
8 docker-compose exec ibf-api-service npm install
Use them as and when needed

@SwatiTh I don't agree at all that this is a known/accepted issue.
The errors you mention here should not happen, and to be honest I don't think you can run the system correctly (which you are doing now) if these issues happen. Are you still expericieng these errors?

I am closing this issue, because it has been solved by the new migration-setup.


Rich text block image decorators don't show up in latest versions of plugin

Using a rich text block and an image decorator, you can set up a way to show images in-line with text as described in the UMG Rich Text Block documentation.
In the latest versions of C4U (v1.15 and up) the image shows in PIE and everything's working as expected. However, in packaged games (tried with shipping and DebugGame), the image does not show. Rather, the HTML tag shows, e.g. <img id="logo"/>
In a prior version of C4U (v1.10.1) the image does show properly in both contexts.
I uploaded an example project for reproducing the issue. If you use plugin version v1.10.1 and package the samples project, you'll see:

In v1.17.0, you'll see:

as a clue, I'm seeing the following error:
LogSlateStyle: Warning: Unable to find Slate Widget Style 'img'. Using FTextBlockStyle defaults instead.
In a conversation with Epic, their engineer said:

It looks like the 'img' is being treated as a style name from the style table instead of being recognized as a decorator. It's not immediately clear how a plugin would be causing the decorator class to be missing or unrecognized, you could drop some breakpoints in URichTextBlockImageDecorator to see if it's functions are being called (meaning it exists but isn't finding the image) or if it's missing altogether. The Supports function in there will return false if a matching brush isn't found and will fall back to looking for a style, you could set bWarnIfMissing to true to log out warnings whenever the decorator looks for a specific image and doesn't find it.

@joseph-kaile can you please take a look at this one?


Endless loop connecting to mpd server

Hi @sludgefeast
really great that you tackled implementing better support for multiple mpd servers!
I tried your development branch in the AndroidStudio VM and ran into an issue if the default mpd server is not reachable.
After opening the app the dialog "Connection Failed" pops up correctly. Clicking on the "Settings" button takes you to the ConnectionSettingsActivity. But it also instantly triggers the onResume() of this activity which tries to connect to the (unreachable) mpd server. The connection attempt fails and the "Connection Failed" dialog is shown once again.
Avoiding the connection attempt in onResume() in the ConnectionSettingsActivity seems to fix this.

Hi @chme!
I fixed this issue. Thanks for your feedback. In the current phase there are still many issues left - so feel free to test, report or maybe fix some issues.

I am sorry for taking so long to get back to you. Thanks for fixing this issue!


Investigate tags and workflow versions for tools

In 1.6, tool versions are loaded with the label tags, but they are now loaded under the label workflowVersions. However, the tags label still exists, but it seems to always be null. Should tags still exist for tools? And does the 1.6 CLI rely on looking for tags instead of workflowVersions.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-1010

Oh, maybe I misheard. So this isn't non-deterministic?
That sounds like a backwards compatibility thing then (i.e. provide the old field to maintain some form of compatibility to old clients)

You probably heard correctly. It's just when I was writing the ticket I realized what I said during standup wasn't actually the case. But thought should bring it up anyway and since I didn't quite get what gary said about what he's noticed, this might be related? Or maybe his was an ES only issue.

I think what @garyluu meant was that when Java objects are converted and imported into elastic search, he has noticed the same thing (a rename of fields).
So it sounds like this is similar to #2860 or https://github.com/dockstore/dockstore/pull/2887
When renaming fields, we sometimes have to leave the old one in place either with the data in the old format or just empty to avoid crashing old clients.
Why was performance testing picking this up though? Does it rely on the old field?

During standup, I noted that before my fix, elasticsearch verification facet was not working at all (completely missing).  The verification facet pulls in ALL the tools/workflows and looks for tags.verified in each entry and still found nothing which is why I believed that it's deterministically null/empty for all of them. See https://github.com/dockstore/dockstore-ui2/commit/134232f2c0e2027f20e0cb56d8b99fd8a19585f3#diff-5c2de616fd18de33171ea09ce84291d1 for me removing tags.verified. In the current search, it's using workflowVersions.verificationSources instead of tags.verificationSources.

Why was performance testing picking this up though? Does it rely on the old field?

Yes, it picks a random container and then navigates to a the default version, or first version, if there is no default version, like what the UI is doing when you click on one of the containers on the home page. It is looking at the now old field to figure that out.
We can change it to check for workflowVersions instead.


added myself

Hey, Please merge my pull request <3.

Add details

Hey, Please help me to do this I'm unable to understand this.
On Sat, 26 Sep, 2020, 3:47 PM Chandu<EMAIL_ADDRESS>wrote:

Add details
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenSourceTogether/Hacktoberfest-2020/pull/93#issuecomment-699475020,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUDPZON6X7D3HUKKKIUS3DSHW5SZANCNFSM4R2ZWCDQ
.