Failing xo linter on master

With the release of<EMAIL_ADDRESS>the linter fails on master.
See https://github.com/eslint/eslint/issues/3759
Pinning to<EMAIL_ADDRESS>worked on my branch.

No.

But.. Butt..
➜  generator-nm git:(master) ✗ npm test

><EMAIL_ADDRESS>test /Users/jamestalmage/WebstormProjects/generator-nm
> xo && mocha

/Users/jamestalmage/WebstormProjects/generator-nm/app/index.js
  12:16  error  Properties should be quoted as `default` is a reserved word  quote-props
  36:6   error  Properties should be quoted as `default` is a reserved word  quote-props

✖ 2 problems (2 errors, 0 warnings)
npm ERR! Test failed.  See above for more details.
➜  generator-nm git:(master) ✗

And that's after an rm -rf nod_modules and reinstall.

I'm not actually recommending pinning xo, just making you aware there is an upstream issue.

@ bdebef730619 (1.6.2 release) now fails the build.
https://travis-ci.org/jamestalmage/generator-nm/jobs/79963505

@sindresorhus see my last comment on the eslint issue. It does not appear to be an eslint problem. xo specifically sets the keywords: true options on the quote-props rule.
I'll leave it up to you as to whether this is an issue here or with xo.
Here is a repro using just pure eslint:
https://github.com/jamestalmage/eslint-3762-repro
If you toggle it to keywords: false in package.json the linter passes.

@jamestalmage Yes, you're right, sorry about that. The ESLint docs were pretty ambiguous. The keyword option is only useful for ES3 as in ES5 those are not reserved in object literals. I removed the keyword option in a later XO version. It's fixed in<EMAIL_ADDRESS>


Restructure

Builds off #46.
Restructures how we do crate generation.
We now have a state machine UserCrate which has three states:

StateGenerated: Constructed from a fn_oid or components this can output lib.rs or Cargo.tomls and can state change via provision() into a StateProvisioned
StateProvisioned: Resident to disk, able to state change via build() to StateBuilt
StateBuilt: A shared object now exists in the workdir, holds that path.
StateLoaded: A  loaded shared object (with a late initialized symbol)

Adds some testing around the code.
No significant changes to parsing/generation were done, it was mostly movement.

The only commented code here is a little TODO idea for me (Trigger) and a test introduced in a separate PR (#46 ).
Also this is a draft and not ready for review.

Alright I'm happy with this now.

Will merge after CI


Unable to change comment color

Description
Hi, and thanks for a beautiful theme!
I'm trying to set a different color for the comments, as they are a bit too low contrast for me, but am unsuccesful. I am probably doing something silly, though... Config:
return {
    "catppuccin/nvim",
    name = "catppuccin",
    disabled = false,
    priority = 1000,
    config = function()
        require'catppuccin'.setup({
            -- Doesn't work. Why?
            highlight_overrides = {
                mocha = function(colors)
                    local mocha = require("catppuccin.palettes").get_palette "mocha"
                    return {
                        comment = { fg = mocha.flamingo }
                    }
                end
            },
            integrations = {
                cmp = true,
                gitsigns = true,
                nvimtree = true,
                treesitter = true,
                notify = true,
                mini = true,
            }
        })
    end
}

Neovim version
NVIM v0.10.0-dev-741+g996dd36c77
Build type: RelWithDebInfo
LuaJIT 2.1.1702233742
Run "nvim -V1 -v" for more info

Terminal and multiplexer
foot, no mulitplexer
Catppuccin version / branch / rev
commit 6853cc8
Steps to reproduce

create a comment
have it be the color I want...

Expected behavior
To be able to change color of comment
Actual behavior
Unable to change color
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "catppuccin/nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("catppuccin")
-- add anything else here

The highlight group should be Comment with uppercase C instead of comment

Hmmm... changing the case doesn't seem to have any effect...


Implement full archive for Chat

Inject ChatManager.
Use ChatManager to get the chat channels for the site(context):
public List<ChatChannel> getContextChannels(String context, boolean lazy);
For each channel get the total number of messages since the date the channel was created.
Use paging to get them all.

Done in m1


Add information how to set the proxy for teams on Linux

Thanks for porting Microsoft Teams to Linux. While playing with this behind a cooperate firewall and a proxy set-up I could not login. It seems you are using somehow a chromium backend and one could easily pass the proxy to the executable:
teams --proxy-server=http://localhost:3128

Maybe a link to the Microsoft Code network settings page is an option: https://code.visualstudio.com/docs/setup/network#_proxy-server-support

@graugans Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@graugans  Thank you for raising this issue! I'll be looking into it!

@graugans Would you be able to link the documentation page you were referring to when you made this post? Without that, I won't be able to help in your issue, thank you!

@TokyoScarab The link is already included in the initial issue. https://code.visualstudio.com/docs/setup/network#_proxy-server-support

@graugans  We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@scanum unfortunatly I can not reopen the issue. Only organisation-members/contributors/people-with-write-access-to-repo/something-similar have the ability to reopen issues. The best place to put this information I do see at the moment is: https://docs.microsoft.com/en-us/microsoftteams/get-clients#linux Or: https://docs.microsoft.com/en-us/microsoftteams/known-issues#linux
It seems it is my fault that I have expected that the document team at Microsoft does know where the best place to put information about the Network configuration.


feat: checker to verify before room creation

Issue Template
Description
As we discussed in discord regarding this. I'm just adding it here.
https://discord.com/channels/1173844241542287482/1173844241542287485/1264884109574934569
Steps to Reproduce

Step 1
Step 2
Step 3

Expected Behavior
Please describe what you expected to happen.
Actual Behavior
Please describe what actually happened.
Screenshots
If applicable, add screenshots to help explain the issue.
Environment

Operating System:
Browser (if applicable):
Version (if applicable):

Additional Information
Add any other relevant information about the issue here.

Just note some idea:

Setting a token check endpoint (from config or when create token)
Media server call check endpoint for ensuring it valid

Some concern:

How to config it?
If embedded inside token, it can make token too big

@giangndm thank you.
How to config it?

I think it should be set from server level config. Most of the time it will be a fixed URL.

@giangndm thank you.

How to config it?

I think it should be set from server level config. Most of the time it will be a fixed URL.

But we have multiple zones with multiple media servers. Do you think it makes us more likely to misconfigure some servers?

In this case setting up in token will be better idea. Also user will aware of if they want to use this feature & need to make sure that they've added in every server.


fix: runtime build problem

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.皮锟 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add K8s 1.27 Support for downstream RKE2 clusters

This is a child ticket of the parent ticket https://github.com/rancher/rancher/issues/44098.
Todo:

[x] update KDM
[x]  update KDM URL in Rancher - covered by https://github.com/rancher/rancher/pull/44146

The fix can be validated in the latest v2.7.11-head

can be validated in the latest v2.7.11-head image

tested as part of regression tests in rancher, tracked in Qase. All tests are passing on 2.7.11-alpha3


Dependency change to fix compilation errors results in no features generated

Starting dev mode without all of the correct dependencies (no features generated on startup) add the correct dependencies to pom.xml, no additional features are generated when features should be generated. Workaround is "o" and Enter or saving a source file to trigger feature generation again.
To reproduce

Using the finish directory of the multi-module guide: https://github.com/openliberty/guide-maven-multimodules
Add API usage, ex. simple jaxrs application and resource classes and an healthcheck class to /war module.
Remove the umbrella Jakarta EE and MicroProfile dependencies from the war/pom.xml.
Start dev mode, compilation should be unsuccessful and no features should be generated
Add the correct dependencies to the pom.xml to fix the compilation errors. Features generation runs before failing classes are recompiled no additional features are generated. Source compilation occurs after liberty:deploy, but feature generation does not run again."o" and Enter or saving a source file should trigger feature generation again and the correct features are generated and installed.

[INFO] ************************************************************************
[INFO] guide-maven-multimodules-jar source compilation was successful.
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:8: error: package org.eclipse.microprofile.health does not exist
import org.eclipse.microprofile.health.Liveness;
                                      ^
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:9: error: package org.eclipse.microprofile.health does not exist
import org.eclipse.microprofile.health.HealthCheck;
                                      ^
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:10: error: package org.eclipse.microprofile.health does not exist
import org.eclipse.microprofile.health.HealthCheckResponse;
                                      ^
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:14: error: cannot find symbol
public class SystemLivenessCheck implements HealthCheck {
                                            ^
  symbol: class HealthCheck
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:12: error: cannot find symbol
@Liveness
 ^
  symbol: class Liveness
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:17: error: cannot find symbol
    public HealthCheckResponse call() {
           ^
  symbol:   class HealthCheckResponse
  location: class SystemLivenessCheck
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:16: error: method does not override or implement a method from a supertype
    @Override
    ^
/Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/war/src/main/java/io/openliberty/guides/multimodules/web/SystemLivenessCheck.java:22: error: cannot find symbol
        return HealthCheckResponse.named(
               ^
  symbol:   variable HealthCheckResponse
  location: class SystemLivenessCheck
8 errors
[INFO] guide-maven-multimodules-war source compilation had errors.
[INFO] guide-maven-multimodules-ear tests compilation was successful.
[INFO] [AUDIT   ] CWWKT0017I: Web application removed (default_host): http://<IP_ADDRESS>:9080/converter/
[INFO] [AUDIT   ] CWWKZ0009I: The application guide-maven-multimodules-ear has stopped successfully.
[INFO] [AUDIT   ] CWWKT0016I: Web application available (default_host): http://<IP_ADDRESS>:9080/converter/
[INFO] [AUDIT   ] CWWKZ0003I: The application guide-maven-multimodules-ear updated in 0.104 seconds.
[INFO] Running liberty:generate-features
[INFO] No additional features were generated.
[INFO] Running liberty:deploy
[INFO] CWWKM2102I: Using artifact based assembly archive : io.openliberty:openliberty-kernel:null:<IP_ADDRESS>:zip.
[INFO] CWWKM2102I: Using installDirectory : /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/target/liberty/wlp.
[INFO] CWWKM2102I: Using serverName : defaultServer.
[INFO] CWWKM2102I: Using serverDirectory : /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/target/liberty/wlp/usr/servers/defaultServer.
[INFO] Copying 2 files to /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/target/liberty/wlp/usr/servers/defaultServer
[INFO] CWWKM2144I: Update server configuration file server.xml from /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/src/main/liberty/config/server.xml.
[INFO] CWWKM2185I: The liberty-maven-plugin configuration parameter "appsDirectory" value defaults to "apps".
[INFO] CWWKM2160I: Installing application guide-maven-multimodules-ear.ear.xml.
[INFO] CWWKM2010I: Searching for CWWKZ0001I.*guide-maven-multimodules-ear in /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/target/liberty/wlp/usr/servers/defaultServer/logs/messages.log. This search will timeout after 40 seconds.
[INFO] CWWKM2015I: Match number: 1 is [10/05/22 15:43:11:690 EDT] 00000039 com.ibm.ws.app.manager.AppMessageHelper                      A CWWKZ0001I: Application guide-maven-multimodules-ear started in 0.495 seconds..
[INFO] guide-maven-multimodules-war source compilation was successful.
[INFO] guide-maven-multimodules-ear tests compilation was successful.
[INFO] [AUDIT   ] CWWKG0016I: Starting server configuration update.
[INFO] [AUDIT   ] CWWKG0093A: Processing configuration drop-ins resource: /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/target/liberty/wlp/usr/servers/defaultServer/configDropins/overrides/generated-features.xml
[INFO] [AUDIT   ] CWWKG0093A: Processing configuration drop-ins resource: /Users/kathrynkodama/devex/featureGeneration/snapshot-testing-04-22/guide-maven-multimodules/finish/ear/target/liberty/wlp/usr/servers/defaultServer/configDropins/overrides/liberty-plugin-variable-config.xml
[INFO] [AUDIT   ] CWWKG0018I: The server configuration was not updated. No functional changes were detected.
[INFO] [AUDIT   ] CWWKT0017I: Web application removed (default_host): http://<IP_ADDRESS>:9080/converter/
[INFO] [AUDIT   ] CWWKZ0009I: The application guide-maven-multimodules-ear has stopped successfully.
[INFO] [AUDIT   ] CWWKT0016I: Web application available (default_host): http://<IP_ADDRESS>:9080/converter/
[INFO] [AUDIT   ] CWWKZ0003I: The application guide-maven-multimodules-ear updated in 0.076 seconds.

Mac OS Java 1.8 Maven 3.8.4

Was able to reproduce this with a non-umbrella dependency addition to a single module project on Windows - Maven 3.6.3 - Java 1.8.0


When will you load your code into github?

I read your paper and want to apply your method to my own problems. Your method is much more superior than other VAE methods. I am looking forward to trying your code. Thank you very much.

Thank you Hanjun-Dai for loading your code into github. Could you please add more information about your code, such as how to use it, an ipython notebook example, what libraries are needed, and so on.

Hi there, please see my updated README, and let me know if there's any question.


Make sure callback id matches the source file name during local run

Summary
Make sure the callback_id matches the source file name during local run.
Details
Right now we require the callback_id to match the filename in LoadFunctionModule. This works fine for the lambda runtime where we control the files that are passed to the runtime, but when we try to run the developer's local files we are throwing an unclear error when the name of the file didn't match the callback_id.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

FYI @curtisallen this should not affect the Lambda context at all, since Lambda does not use the local-run.ts file. ROSI can keep using the existing tag of this repo it uses, or update to latest - shouldn't matter.


AaaLib still has no tests

Affected version(s)
1.0.60
Is this a regression?
NO - This bug also affected the previous version
Context
There are no tests in this repository.
Also, quite obviously, no coverage reports.
Expected behavior
PR checks should include running tests.
The release workflow should, too.
Actual behavior
dotnet test is executed in the above-mentioned workflows, but it does nothing, as there are no tests.
(I'm getting boring, I know.)
How to reproduce
n/a
Configuration
n/a
Other information
AaaLib being an incubator for Tenacom's procedures and internal standards, it's not only a matter of writing tests. Which unit testing library do we use? Do we also use some helper library, e.g. FluentAssertions? Etc.
It's important to have a standard, so people can "jump" between projects and feel at home.

I'm currently researching libraries and tools we can use. I'm going to open a discussion with the results, as it seems a better approach than stuffing them in an issue.

Find the discussion here: #11


Factorial should only support unsigned integer

Action

[ ] Rename ValueError::FactorialOnNonNumeric to ValueError::FactorialOnUnsignedInteger

It already existed as a FactorialOnNegativeNumeric type.


/var/task/main: /lib64/libc.so.6: version `GLIBC_2.34' not found

Awesome project 👍
Before all I must say that I'm not directly using your pre-build releases. So feel free to close this issue if you think it's not adequate.
I've copy/pasted your repository and adapted it for my needs but I'm getting the following runtime errors when running my Lambda:
/var/task/main: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by /var/task/main)
/var/task/main: /lib64/libc.so.6: version `GLIBC_2.32' not found (required by /var/task/main)

What could be wrong?
This is my install script:
SHARP_VERSION="0.33.2"

echo ""
echo "Install and compile sharp v${SHARP_VERSION} for AWS Lambda Linux AMI environment"

# Install and compile Sharp for the AWS Lambda Linux AMI environment.
# Tells to explicitly install libvips in node_modules instead of using global libvips
SHARP_IGNORE_GLOBAL_LIBVIPS=1 \
npm install sharp@${SHARP_VERSION} \
  --save=false \
  --os=linux \
  --cpu=arm64 \
  --libc=glibc

# Remove musl-based binary
rm -rf ./node_modules/@img/*-linuxmusl*

And my build script:
# Build sharp using ESBuild
./node_modules/.bin/esbuild \
  --bundle ./node_modules/sharp/ \
  --outfile=index.js \
  --minify \
  --format=cjs \
  --platform=node

# Move index.js
mkdir -p ./dist/nodejs/node_modules/sharp/lib
mv index.js ./dist/nodejs/node_modules/sharp/lib/

# Copy
cp -a node_modules/sharp/package.json ./dist/nodejs/node_modules/sharp/
cp -a node_modules/sharp/LICENSE ./dist/nodejs/node_modules/sharp/
cp -a node_modules/@img ./dist/nodejs/node_modules/

I'm running everything using GitHub Actions using ubuntu-22.04 the same as you.
Along with actions/setup-node@v4
- name: Set up Node.js
  uses: actions/setup-node@v4
  with:
    node-version: 20.x

Is it because I'm building using ubuntu-22.04 that uses libc6-dev version 2.35? 🤔

Name
Version

libc++-dev
1:14.0-55~exp2

libc++abi-dev
1:14.0-55~exp2

libc6-dev
2.35-0ubuntu3.6

libcurl4
7.81.0-1ubuntu1.15

You are using the same ubuntu-22.04 so how is it working for you? 🤔 (is it?)

Hum sorry ok it's much more related to aws-lambda-go

https://github.com/aws/aws-lambda-go/issues/340
https://stackoverflow.com/questions/71940179/error-lib-x86-64-linux-gnu-libc-so-6-version-glibc-2-34-not-found
https://github.com/aws-amplify/amplify-cli/issues/12577


[PLT-8396] Add loading spinner to MFA sign in button

Summary

Add loading spinner to MFA sign in button
Also convert LoginMfa into pure component and add unit tests

Ticket Link
Jira ticket: PLT-8396
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)
[x] Includes text changes and localization file (.../i18n/en.json) updates

@esethna Could you please help review the text Submitting... if that's the correct word to use?

Thanks! I thought I've updated that :)

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0ef515f9e23017ef3.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-0ef515f9e23017ef3


Read-only file system

when trying to run the container, i get this error: install: cannot create regular file '/target/browser-box': Read-only file system
i used this command: sudo docker run -it --rm   --volume /usr/local/bin:/target   --env BROWSER_BOX_REPO=sameersbn   sameersbn/browser-box:1.0.1-3 install
is there a quick workaround or fix?
i would really appreciate if i could get this to work, it would save a ton of time
thanks :)

When I run according to the instructions -
Run:
docker run  -it --rm  --volume /usr/local/bin:/target   --env BROWSER_BOX_REPO=sameersbn sameersbn/browser-box:1.0.1-3 install
Error:
install: cannot create regular file '/target/browser-box': Permission denied
This may be a slightly different error but still permission related. So I added a few options based on recent docker knowledge I've gotten.  So the result is the below works!
docker run --privileged  -it --rm  --volume /usr/local/bin:/target  -e "container=docker" -v /sys/fs/cgroup:/sys/fs/cgroup -v /tmp/.X11-unix:/tmp/.X11-unix  --env BROWSER_BOX_REPO=sameersbn sameersbn/browser-box:1.0.1-3 install


Empty guide in EPUB 2 not detected

Trying to validate an EPUB 2 with an empty <guide></guide> in the OPF does not return any error or warning.
Trying to validate an EPUB 3 with an empty <guide></guide> in the OPF does return:
ERROR(RSC-005): test.epub/OEBPS/content.opf(41,9): Error while parsing file 'element "guide" incomplete; missing required element "reference"'.

The EPUB 2 spec (assuming this: http://www.idpf.org/epub/20/spec/OPF_2.0.1_draft.htm ) clearly implies that the <guide> is optional, but, if present, it should have at least one child.
I think the .rng currently in use has a "zeroOrMore" instead of a "oneOrMore", which was brought in by the patch in issue #24 .
So the question is: is an empty <guide> </guide> legal in EPUB 2 or is it an epubcheck bug?

Looks like a bug to me, OPF 2.0.1 says

Within the package there may be one guide element, containing one or more reference elements.

Good catch Alberto!

@rdeltour can this be fixed in the EpubCheck schema directly or does it need to be fixed in the official spec schema and only then can be used in EpubCheck?

Tobias, that object is declared in the same file as the public method that returns it. I’m not a java expert but beside IKVM, it didn’t look right having a “Public” method returning a “Private” object.
If a “Public” method is returning a object, that object should be declared “Public” also. This just looks like a simple oversight.
From: Tobias Fischer<EMAIL_ADDRESS>Sent: Monday, May 16, 2016 6:40 AM
To: IDPF/epubcheck<EMAIL_ADDRESS>Subject: Re: [IDPF/epubcheck] Empty guide in EPUB 2 not detected (#663)
@rdeltour https://github.com/rdeltour  can this be fixed in the EpubCheck schema directly or does it need to be fixed in the official spec schema and only then can be used in EpubCheck?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/IDPF/epubcheck/issues/663#issuecomment-219415329 https://github.com/notifications/beacon/ARwjWpG00gtwWyEJmL4l9YuYvvMIno_0ks5qCGW2gaJpZM4G1294.gif

Yes, it was an oversight.  My apologies.

From<EMAIL_ADDRESS>Sent: 5/16/2016 6:00 AM
To<EMAIL_ADDRESS>Subject: Re: [IDPF/epubcheck] Empty guide in EPUB 2 not detected (#663)
Tobias, that object is declared in the same file as the public method that returns it. I’m not a java expert but beside IKVM, it didn’t look right having a “Public” method returning a “Private” object.
If a “Public” method is returning a object, that object should be declared “Public” also. This just looks like a simple oversight.
From: Tobias Fischer<EMAIL_ADDRESS>Sent: Monday, May 16, 2016 6:40 AM
To: IDPF/epubcheck<EMAIL_ADDRESS>Subject: Re: [IDPF/epubcheck] Empty guide in EPUB 2 not detected (#663)
@rdeltour https://github.com/rdeltour  can this be fixed in the EpubCheck schema directly or does it need to be fixed in the official spec schema and only then can be used in EpubCheck?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/IDPF/epubcheck/issues/663#issuecomment-219415329 https://github.com/notifications/beacon/ARwjWpG00gtwWyEJmL4l9YuYvvMIno_0ks5qCGW2gaJpZM4G1294.gif

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/IDPF/epubcheck/issues/663#issuecomment-219418946

No problem. Thanks for your efforts!
From: Steve Antoch<EMAIL_ADDRESS>Sent: Monday, May 16, 2016 7:07 AM
To: IDPF/epubcheck<EMAIL_ADDRESS>Cc: jajessup<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [IDPF/epubcheck] Empty guide in EPUB 2 not detected (#663)
Yes, it was an oversight. My apologies.

From<EMAIL_ADDRESS>Sent: 5/16/2016 6:00 AM
To<EMAIL_ADDRESS>Subject: Re: [IDPF/epubcheck] Empty guide in EPUB 2 not detected (#663)
Tobias, that object is declared in the same file as the public method that returns it. I’m not a java expert but beside IKVM, it didn’t look right having a “Public” method returning a “Private” object.
If a “Public” method is returning a object, that object should be declared “Public” also. This just looks like a simple oversight.
From: Tobias Fischer<EMAIL_ADDRESS>Sent: Monday, May 16, 2016 6:40 AM
To: IDPF/epubcheck<EMAIL_ADDRESS><EMAIL_ADDRESS>>
Subject: Re: [IDPF/epubcheck] Empty guide in EPUB 2 not detected (#663)
@rdeltour https://github.com/rdeltour can this be fixed in the EpubCheck schema directly or does it need to be fixed in the official spec schema and only then can be used in EpubCheck?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/IDPF/epubcheck/issues/663#issuecomment-219415329 https://github.com/notifications/beacon/ARwjWpG00gtwWyEJmL4l9YuYvvMIno_0ks5qCGW2gaJpZM4G1294.gif

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/IDPF/epubcheck/issues/663#issuecomment-219418946
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/IDPF/epubcheck/issues/663#issuecomment-219420334 https://github.com/notifications/beacon/ARwjWsP5mW_oS89-q-AZKH7jAcR1qxE6ks5qCGwQgaJpZM4G1294.gif

This seems to be fixed in latest revision of the OPF schema at http://www.idpf.org/epub/20/spec/OPF_2.0_latest.htm#AppendixA but hasn't been updated in the schema copy in EpubCheck.

Has been fixed in PR #724.


when the EMS benefit will again be available for activation

Is there an updated timeline on when the EMS benefit will again be available for activation?

@LegoSCCMGuy Thank you for the question.
EMS author @evanwindom should know more about this.

@LegoSCCMGuy - Thanks for reaching out.  We're making progress here and anticipate a resolution soon.  Thanks for your patience.
If you haven't already activated the Office 365 Developer subscription, there's a new benefit that will solve this issue for you.  We're now offering Microsoft 365 (M365), which includes both the Office 365 Developer and EMS benefits.  Look for the Microsoft 365 benefit tile in the Tools category at https://my.visualstudio.com/benefits.
(If you have already activated the Office 365 Developer subscription, you won't be able to activate M365.)

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


Integrate Packaging for Bootstrap.

Summary
Include packaging of the Bootstrap sources and compiled files.
Details
There are three options of packaging Bootstrap.

Node.js and in particular the npm package manager.
Packager included in Ruby.
Other options such as Webpack.
Probably the preferred option is Node.js with npm because it provides extra options via the npm package manager.

After selecting one of the above options, integrate the packaging of Bootstrap in our oscm-portal project under a "node_modules" separate folder.
Acceptance criteria
Bootstrap version 4.3.1 is packaged properly. Sass files refer to the correct directories of Bootstrap located under "node_modules"
Additional context
N/A.

I am not clear about this task. In my understanding we are talking of development tools for creating and customizing stylesheets and not of mandatory runtime for OSCM. So there is nothing to be packaged in the oscm-portal.

https://github.com/servicecatalog/oscm-dockerbuild/pull/258

The bootstrap sources are still found in the OSCM repo here. Please remove them.

https://github.com/servicecatalog/oscm-jenkinsscripts/pull/5
https://github.com/servicecatalog/oscm/pull/1027


Instead of 'You' present the user in the 'Access Granted' overview

Instead of showing "You" in the Access Granted overview we should present the user' email address.

I'll happily raise a pull request for this. Thoughts @jpassing?


Only run load_subclasses after app is initialized

In rails 6.1, nothing was calling descendants or subclasses during the app initialization.
Rails changed in 7.0 to call subclasses from reload_schema_from_cache here: https://github.com/rails/rails/commit/6f30cc09a7ad21898d856059e498ab9ad2ae64a4
It also changed to call descendants on the callback class (self) insead of the ActiveSupport::DescendantsTracker here:
https://github.com/rails/rails/commit/ffae3bd8d69f9ed1ae185e960d7a38ec17118a4d
We are not expecting to be called from these locations.
We can make this rails 7 compatible by ensuring the descendant loader loading of subclasses until after the app is booted, which was the implicit behavior previously.
(Extracted from https://github.com/ManageIQ/manageiq/pull/22873)

FYI, to demonstrate rails 6.1 didn't hit this code path during rails boot:
diff --git a/lib/extensions/descendant_loader.rb b/lib/extensions/descendant_loader.rb
index 488ab2500f..09455b37f3 100644
--- a/lib/extensions/descendant_loader.rb
+++ b/lib/extensions/descendant_loader.rb
@@ -263,6 +263,7 @@ class DescendantLoader

   module ArDescendantsWithLoader
     def descendants
+      raise if !defined? @loaded_descendants
       if Vmdb::Application.instance.initialized? && !defined? @loaded_descendants
         @loaded_descendants = true
         DescendantLoader.instance.load_subclasses(self)
@@ -275,6 +276,7 @@ class DescendantLoader
     # https://github.com/rails/rails/commit/8f8aa857e084b76b1120edaa9bb9ce03ba1e6a19
     # We need to get in front of it, like we do for descendants.
     def subclasses
+      raise if !defined? @loaded_descendants
       if Vmdb::Application.instance.initialized? && !defined? @loaded_descendants
         @loaded_descendants = true
         DescendantLoader.instance.load_subclasses(self)

It loads in console and rails runner:
joerafaniello@Joes-MBP manageiq % bin/rails c
** ManageIQ master, codename: Radjabov
Loading development environment (Rails <IP_ADDRESS>)
irb(main):001:0> exit
joerafaniello@Joes-MBP manageiq % bin/rails r ''
** ManageIQ master, codename: Radjabov

As soon as I try to load descendants after app initialization, my hack to raise shows we hit this code path.
In rails 7, this is hit during application boot.
joerafaniello@Joes-MBP manageiq % bin/rails r 'Vm.descendants'
** ManageIQ master, codename: Radjabov
/Users/joerafaniello/Code/manageiq/lib/extensions/descendant_loader.rb:266:in `descendants': unhandled exception
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/railties-<IP_ADDRESS>/lib/rails/commands/runner/runner_command.rb:45:in `<main>'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/railties-<IP_ADDRESS>/lib/rails/commands/runner/runner_command.rb:45:in `eval'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/railties-<IP_ADDRESS>/lib/rails/commands/runner/runner_command.rb:45:in `perform'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/thor-1.3.0/lib/thor/command.rb:28:in `run'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/thor-1.3.0/lib/thor/invocation.rb:127:in `invoke_command'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/thor-1.3.0/lib/thor.rb:527:in `dispatch'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/railties-<IP_ADDRESS>/lib/rails/command/base.rb:69:in `perform'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/railties-<IP_ADDRESS>/lib/rails/command.rb:48:in `invoke'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/railties-<IP_ADDRESS>/lib/rails/commands.rb:18:in `<main>'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/bootsnap-1.18.3/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:30:in `require'
	from /Users/joerafaniello/.gem/ruby/3.1.4/gems/bootsnap-1.18.3/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:30:in `require'
	from bin/rails:4:in `<main>'


feat(website): add chaindrop to ecosystem page

Closes #14305

@2manslkh can we take this out of draft? also, do they have a more square / circular icon, would look better for them.

Chaindrop is already added in another PR


Pest3 support

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

Project coverage is 100.00%. Comparing base (42c6bcd) to head (89645d1).

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #26   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        23        23
===========================================
  Files              2         2
  Lines             56        56
===========================================
  Hits              56        56

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


WebUI crashing on Colab

Here is the error message I see on Colab for this projet:
AttributeError                            Traceback (most recent call last)
 in
14         txt2img=webui.wrap_gradio_gpu_call(modules.txt2img.txt2img),
15         img2img=webui.wrap_gradio_gpu_call(modules.img2img.img2img),
---> 16         run_extras=webui.wrap_gradio_gpu_call(webui.run_extras),
17         run_pnginfo=webui.run_pnginfo
18 )
AttributeError: module 'webui' has no attribute 'run_extras'

As a workaround you can replace 16-17 lines like this:
run_extras=webui.wrap_gradio_gpu_call(modules.extras.run_extras),
run_pnginfo=modules.extras.run_pnginfo

Yeah it looks like there has been changed with WebUI.  I only set this up as a auto login for someone and cleaned up install method. I'll see if I can patch that when I get some time. May have to use Voldemort or another version of you can't patch


Feature/adjust info screen to announce new release features for 2.7

This PR brings the new version 27. features info screen
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-8295

@SwissMountainFrog
In the other versions the bullet points were always bold, is this also the case here and the screenshot is not accurate?
Here's the screen from version 2.5:

@pascal-brause
Thanks for the feedback, @SwissMountainFrog told me that this is tracked in EXPOSUREAPP-8510.


makeExtendableWithAttributes for public usage

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[x] I have looked at the Documentation
[x] I have read the F.A.Q.
[x] I have filled out this issue template.

Issue Info

Info
Value

Platform
ios

Platform Version
10.3

SnapKit Version
3.2.0

Integration Method
cocoapods

Issue Description
This is more like feature request
Is there is a way to use something similar to makeExtendableWithAttributes inside of makeConstraints block in my application?
It would be nice to be able to create a variable with constraint attribute to use it inside makeConstraints. Example: I have a view that can be filled from fight to left of from left to right. In current implementation constraints creation looks like that:
...
switch fillMode {
case .leftToRight:
  myView.snp.makeConstraints { make in
    make.left.equalToSuperview()
    ...
  }
case .rightToLeft:
  myView.snp.makeConstraints { make in
    make.right.equalToSuperview()
    ...
  }
}

Code duplication is bad so it would be much nicer like that:
...
let attribute: ConstraintAttributes
switch fillMode {
case .leftToRight:
  attribute = .left
case .rightToLeft:
  attribute = .right
}

myView.snp.makeConstraints { make in
  // This is just an example
  make.with(attribute).equalToSuperview()
   ...
}

Hm this is a pretty good suggestion, however, at the same time I find the new concise code ambiguous. The reader of this code now has to go find out what "attribute" is and what value it stores.
Most code it is valuable to strictly DRY it up but with auto layout I find it sometimes better to be a bit more verbose so readability is maintained years later.

@robertjpayne I agree that developers should not mess with auto layout code.
My example is a rare use case so maybe it's not worth it to support that.
The other solution to this problem is to decompose repeated make-code to another method with make as a parameter. This is not that bad 🤷♂
Thank you for quick answer!

I need to find a few ways to allow users to hook SnapKit and provide extensions for the API. Specifically a lot of users may want their own shortcuts but I'm unwilling to support them directly as I like to keep the API as true and "one way to do something" as possible.


[FEATURE] Add update field to scale component command

During our preparation for 1.0, @brooksmtownsend and I were chatting about the ergonomics of scale vs update in the control API. Essentially, once you start a component, its image reference we store in lattice data can only change if you update the reference, even if you scale to 0. This is a safety measure that prevents a user from unintentionally upgrading/downgrading a component when scaling or unintentionally running two different versions on two different hosts. In order to streamline API usage when the actual intent is to update the image ref, we think we should add an update boolean field to the ScaleComponentCommand that would indicate to a host that it should simultaneously perform the update operation along with anything else from the scale command. wash can then expose this as a flag for those users who may consume the imperative ctl API
Additional context
One thing that we also need to capture in our docs is that the ctl API is a "dumb" API that doesn't have any smarts around handling everything that is running. This is intentional so anyone can build their own extensions on top of it, just like how we build wadm. This should be made extra clear in explaining why the API is designed the way it is. We purposefully don't build anything in around coordinating simultaneous updates unless using a higher level tool like wadm.
We're not opposed to adding something like coordinating an update, but we would like to see some actual user production use cases before we build that API
Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[X] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

To clarify, this could be added post-1.0 as it is an addition to the API and not a breaking change

hey @thomastaylor312 , looking at  the code here) { and then here... it appears that one can do both a scale & update simultaneously with some warning message about the component refs not being the same... the scale task still proceeds with the provided component_ref input which may not be the same as the existing known ref (ie, entry.image_reference).
based on the observation above, assuming it holds... the change to make would be to

add the update flag as suggested: when true, use the provided component_ref. if false, always use the existing actor ref entry.image_ref.
update the predicate that displays the warning message regarding the mismatch between existing actor ref and the input actor ref

I can try working on this in the next couple of weeks since I already worked with code around host update/scale and wash #1978. I am motivated to understand the waswCloud internals, so why not contribute?

@thomastaylor312 yes, let's do it


Do not allow freshness to move errors out of the current file, ensure json documents are deeply unfreshened and fully widened

Fixes #34997
So, the two issues at play:

The error was only reported on the second call. This was because the error span was moved into the json document and then discarded (which is why the error had no elaboration - it had already been elaborated in the discarded error). We now have checks in the error span moving code that ensure the span we're moving to is in the same file as the input location, at least, so we'll not discard any errors like that again. (Side note: the fresh literal span moving code is much less safe than the other span moving code, as it looks up locations from symbols, rather than the AST - it would be nicer to remake it to be more like checkTypeRelatedToAndOptionallyElaborate and only traverse the AST)
That there was an error at all was caused by a bug in how object literal freshness freshness was performed for json documents. getRegularTypeOfObjectLiteral is used in checking in a few places to get a relatively shallow unfreshed object, while getWidenedType does full deep unfreshening, accounting for arrays and object contexts (do note: these are cached separately, as they are observably different transforms). We were using getRegularTypeOfObjectLiteral outside of the isRelatedTo context it is appropriate within, and should have used getWidenedType instead, and now we do. This should also have the side effect of producing the nicer context-sensitive object types we produce during assignments from json document types, too (as can be seen in the jsDeclarationsJson.types baseline).

Just because I’m curious...

This was because the error span was moved into the json document and then discarded

Since the error was... erroneous, and caused by freshness checks, what prevented the second error from being discarded in the same way as the first?

The second error used the cached relationship result, which skips the expanded elaboration and error span moving code.


I didn't find my issue here, can anyone help me?

Internet connection issue. Check your antivirus to make sure that it isn't preventing Bloxstrap from connecting to the internet


Popup is infinitely long at the bottom

Describe the bug
Each popup has no end, the popup always goes down to the bottom of the screen.
To Reproduce
Steps to reproduce the behavior:
I have this issue with all my popups. I even created a new clean page in my dashboard and tried with the sample from the readme (https://github.com/Clooos/Bubble-Card/tree/main?tab=readme-ov-file#pop-up). Its not working.
I tried to adjust the margin settings in the popup card but without success.
Expected behavior
The popup should not be longer than the content.
Screenshots

YAML
If applicable, add any relevant YAML code.
type: vertical-stack
cards:
  - type: custom:bubble-card
    card_type: pop-up
    hash: '#kitchen'
    name: Kitchen
    icon: mdi:fridge
    entity: light.kitchen

Informations (please complete the following information):

OS: Windows/Android
Browser/App: Chrome
Bubble Card version: 2.2.4
Home Assistant version: 2024.9.3

Additional context
Add any other context about the problem here.
Thank you! 🍻

Same here... I'll post back if I figure out what's going on.


Add onError to GithubService

#378 add a error prompt output using onError, and I add the missing onError to GithubService.
Should be merged after #378.

LGTM
Yeah, you're right, I didn't take that into account


dns-status-internal healcheck crashes

Another crash, same context as here.
time="2021-01-13T21:10:11Z" level=info msg="Found instance namespace: kuberhealthy"
time="2021-01-13T21:10:11Z" level=info msg="Kuberhealthy is located in the kuberhealthy namespace."
time="2021-01-13T21:10:11Z" level=info msg="Check time limit set to: 14m47.427936725s"
time="2021-01-13T21:10:11Z" level=info msg="Check pod is running on node: aks-default-15766151-vmss000000"
time="2021-01-13T21:10:11Z" level=debug msg="Getting pod: dns-status-internal-1610572204 in order to get its node information"
time="2021-01-13T21:10:11Z" level=error msg="Error waiting for node to reach minimum age: pods \"dns-status-internal-1610572204\" is forbidden: User \"system:serviceaccount:kuberhealthy:default\" cannot get resource \"pods\" in API group \"\" in the namespace \"kuberhealthy\""
time="2021-01-13T21:10:11Z" level=debug msg="Checking if the kuberhealthy endpoint: http://kuberhealthy.kuberhealthy.svc.cluster.local/externalCheckStatus is ready."
time="2021-01-13T21:10:11Z" level=debug msg="http://kuberhealthy.kuberhealthy.svc.cluster.local/externalCheckStatus is ready."
time="2021-01-13T21:10:11Z" level=debug msg="Kuberhealthy endpoint: http://kuberhealthy.kuberhealthy.svc.cluster.local/externalCheckStatus is ready. Proceeding to run check."
time="2021-01-13T21:10:11Z" level=debug msg="Getting pod: dns-status-internal-1610572204 in order to get its node information"
time="2021-01-13T21:10:11Z" level=error msg="Error waiting for kube proxy to be ready: error getting kuberhealthy pod: pods \"dns-status-internal-1610572204\" is forbidden: User \"system:serviceaccount:kuberhealthy:default\" cannot get resource \"pods\" in API group \"\" in the namespace \"kuberhealthy\""
time="2021-01-13T21:10:11Z" level=info msg="Running DNS status checker"
time="2021-01-13T21:10:11Z" level=info msg="DNS Status check testing hostname: kubernetes.default"
time="2021-01-13T21:10:11Z" level=info msg="DNS Status check determined that kubernetes.default was OK."
2021/01/13 21:10:11 checkClient: DEBUG: Reporting SUCCESS
2021/01/13 21:10:11 checkClient: DEBUG: Sending report with error length of:0
2021/01/13 21:10:11 checkClient: DEBUG: Sending report with ok state of:true
2021/01/13 21:10:11 checkClient: INFO: Using kuberhealthy reporting URL:http://kuberhealthy.kuberhealthy.svc.cluster.local/externalCheckStatus
2021/01/13 21:10:11 checkClient: DEBUG: Making POST request to kuberhealthy:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x11f1473]
goroutine 1 [running]:
github.com/Comcast/kuberhealthy/v2/pkg/checks/external/checkclient.sendReport(0x202a688, 0x0, 0x0, 0x1, 0xc0003265e8, 0xc0002edd74)
   /build/pkg/checks/external/checkclient/main.go:99 +0x4e3
github.com/Comcast/kuberhealthy/v2/pkg/checks/external/checkclient.ReportSuccess(0xc0002eddf0, 0xc0003265a0)
   /build/pkg/checks/external/checkclient/main.go:44 +0x7e
main.reportKHSuccess(0xc0002eddc8, 0xc0002edd70)
   /build/cmd/dns-resolution-check/main.go:182 +0x2d
main.(*Checker).Run(0xc0002f5ee0, 0xc00026fe40, 0xc0002eded0, 0x2)
   /build/cmd/dns-resolution-check/main.go:161 +0x204
main.main()
   /build/cmd/dns-resolution-check/main.go:119 +0x3ce
stream closed

Completed #40 instead.

Completed #40 instead.


[Gateway] Hydrate StakeGateway Message and Add Tests

Summary
Human Summary

Add stake field to the stake and gateway proto messages
Update validation
Update and add unit tests relating to

genesis
cli
staking

Hydrate the stake keeper method with the correct logic
Add makefile helpers

Very similar to #59 (may cause merge conflicts on makefile and .gitignore so double check before merging)
AI Summary

Summary generated by Reviewpad on 17 Oct 23 07:22 UTC
This pull request includes changes to multiple files. Here is a summary:

The config.go file has a single change where the initSDKConfig() function was renamed to InitSDKConfig().

The main.go file has the following changes:

Added package imports.
Added the InitSDKConfig() function call in the init() function.
Added test cases and functions related to the GatewayModuleAddress() and GatewayRemove() functions.

The expected_keepers.go file has changes related to the AccountKeeper and BankKeeper implementations:

Added a //go:generate directive for generating mock implementations.
Imported packages for sdk and types.
Modified the BankKeeper interface by removing the SpendableCoins method and adding the DelegateCoinsFromAccountToModule method.

The config.yml file has changes to the name fields of multiple accounts, where the names have been changed from "portalX" to "gatewayX".

The tests_gateway_msg_stake.go file contains test cases for the MsgStakeGateway struct in the types package. The tests cover various scenarios such as invalid addresses, invalid stakes, and valid addresses with valid stakes.

The genesis_test.go file has changes related to the validation of the GenesisState struct in the gateway/types package. It includes import statements, variable declarations, and test cases for validating the genesis state.

The mocks.go file has been added to the testutil/gateway/mocks directory. This file declares the package for dynamically generated mocks.

The helpers_test.go file has been added to the x/gateway/client/cli package. It includes test cases and helper functions for the gateway module in the CLI package.

The genesis.go file in the x/gateway/types package has changes related to importing the sdk package, adding validation for the stake amount, and checking for duplicated index.

The gateway.proto file has changes related to import statements, a new option for the address field in the Gateway message, and the addition of a stake field in the Gateway message.

The Makefile has been updated with new targets and actions related to testing, code generation, and gating.

The root.go file has changes related to import statements and the renaming of the initSDKConfig() function.

The errors.go file has changes related to import statements, error variable updates, and the addition of new error variables.

The gateway.go file in the testutil/keeper directory has changes related to package imports, function declarations, and variable declarations.

The message_stake_gateway.go file has changes related to import statements, function signature modifications, struct field additions, and validation.

The dependencies.txt file has changes related to the addition, removal, and update of dependencies.

The openapi.yml file has changes to several sections, including the addition of the stake object in the "paths" and "definitions" sections.

The tx_stake_gateway_test.go file has been added to the x/gateway/client/cli package. It includes test cases for the StakeGateway command in the gateway client CLI.

The .gitignore file has changes related to ignoring new file patterns.

The query_gateway_test.go file has changes related to package imports, function removal, and modifications in the TestShowGateway function.

The tx_stake_gateway.go file has changes related to import statements and modifications in the stake-gateway command.

The tx.proto file has changes related to import statements and modifications in the MsgStakeGateway message.

Please review these changes and let me know if you need further assistance.

Issue
Fixes (in part) 19
Type of change
Select one or more:

[x] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing

[x] Run all unit tests: make test_all_unit
[x] Verify Localnet manually: See the instructions [here](TODO: add link to instructions)

Sanity Checklist

[x] I have tested my changes using the available tooling
[x] I have performed a self-review of my own code
[x] I have commented my code, updated documentation and left TODOs throughout the codebase

@h5law Also note that this is not "Hydration," but this is an implementation of the StakeGateway message. I recommend asking ChatGPT about the difference if you want to learn more.

@Olshansk have addressed the comments and caught up with #59 only thing I removed one of your helpers in the network test utils, I couldn't get it to work with the types. It only worked for the application genesis state and I couldn't figure out a good way to make it generic without stepping into encoding and decoding any types so thought best this be put into the helpers that call this method so simplicities sake.


Consider using Microsoft.Extensions.Caching.Memory for all caches

Currently we use Microsoft.Extensions.Caching.Memory for our first level query cache and our own custom caching for all other things across the EF Core stack. Since there is no automatic eviction in our custom caching code it would be possible to make EF consume all memory in caching by just doing enough things in an application, e.g. queries with many different combinations of parameters enter the second level query cache, many models are created dynamically, etc.

Another thing we should consider here is the new explicit compiled query implementation. As far as I understand the delegates returned from the API are bound directly to the compiled query plans, which means that the more distinct compiled queries an application use the more memory will be used that cannot be freed up. We could instead use a unique identifier to point to an entry in the query cache like we do in EF6.

We have done enough here with the changes for 3.0. We can revisit in the future if something else comes up.


Update to 1.20.1

1.20.1 has been released for awhile now. It would be nice to propagate this update now.
I've verified this version to be working on my own personal server for 2 days.
There aren't really any changes, this just updates gradle properties mostly


add dockerignore

doesn't do much here, but I suppose ignoring the Makefiles and friends can't hurt.


How can I use a field of an object as data source?

such as spinner.setItems(List obj),but  just show obj.field in spinner

Hi, could you explain more details?
And you can get the selected field via OnSpinnerItemSelectedListener.
setOnSpinnerItemSelectedListener<String> { oldIndex, oldItem, newIndex, newText ->
   toast("$text selected!")
}

Could the T type of method params setItems(List objs) be a javabean in addition to a String?Then use a String type field of the javaBean as item?

Of course, you can use your custom type, but you should create your own adapter. Reference this custom adapter.
If you use not your customized adapter, the default adapter receives a String type which is the most used generally.


Installation with pip in edit mode

Added files to let the user/developer to install in edit mode (pip install -e .)

setup.py
requirements.txt
pyproject.toml

IMPORTANT: if the app.py is still the script to run it should contain a "main" function, otherwise it will be not possible to
call directly ebook2audiobookXTTS as an app name.
Worth to advise the user/develpoer to install a virtual python env v3.10 min, v3.11 max, with for i.e. conda
(I tried with python 3.11 and everything is ok)
conda create --prefix /path/to/env-folder python=3.11
conda activate  /path/to/env-folder
...
conda deactivate

WOH! 😲🤩
Dang! you did way more than I would of done!
This is Awesome!

wait. I have to fix some important things yet. next PR ok?

oops, ill revert it
was about to ask about the main thing lol

Reverted

ok I do the fix and run a PR again

I await your updated merge 👍

Hit me up when it's ready
I'm still new to this lol

ok no worries, I'm done for today, tomorrow all will be good


feat: implement Serde (de)serializer for Ipld enum

It's now possible to serialize and deserialize the Ipld enum with Serde.
As the Serde data model doesn't support custom types, we use the newtype
struct hack, which is used by many other libraries as well. We introduce
a newtype struct with a mostly unique/hard to clash with name. This struct
wraps the CID.
Serde de(serializers) that support CIDs, can then make use of it, others
will see a strange looking struct in their ouput.

I think at one point I needed Eq, Ord and PartialOrd for something fundamental. I can't recall for what. As things seems to work without it, I'd just not implement them for now.
In regards to "should we serialize to xyz", I think it makes sense to mirror serde_jsons behaviour as IPLDs data model is so similar and I as a user would also expect it to work in a similar way. WDYT @Stebalien.
All other things should now be fixed.

In regards to "should we serialize to xyz", I think it makes sense to mirror serde_jsons behaviour as IPLDs data model is so similar and I as a user would also expect it to work in a similar way. WDYT @Stebalien.

I think IPLD is significantly different:

Serde's attitude is "just do the thing". I.e., as long as it round-trips with serde, it's good enough.
For us, we very much care about the final representation and about not having any magic.

If we find that not having some specific magic is becoming a problem, we always add more magic as necessary. But it's hard to take it away if we find that it's confusing and/or unexpected.
For example:

Given a unit type, I wouldn't expect it to encode to whatever the unit type's name is in rust. Really, I'd expect either:

The user to skip the field (i.e., it's a bug).
The unit type to encode to some constant.

Given an enum with no "values", I'd expect it to encode to the tag values (1, 2, 3, etc.) or some "key". But, in most cases, I'd expect the user to specify the key.

@Stebalien thanks for explaning. I had another look and came to the same conclusions. Here's the current state:

Ipld::Null now serializes to None
unit and unit structs error. They can't neither be serialized, nor deserialized. I think symmetry makes sense here.
The only thing I haven't done that you brought up is about enums with e.g. repr(u8). I think representing as enum by default as string makes sense. If you want the actual integer value, you should be able to use serde_repr.

The only thing I haven't done that you brought up is about enums with e.g. repr(u8). I think representing as enum by default as string makes sense. If you want the actual integer value, you should be able to use serde_repr.

Fair enough. We can also revisit the other decisions later if we change our minds.

Oh, and most importantly I forgot to mention: thanks @Stebalien for your in-depth thoughtful review. It's amazing how much thought you put into this and how many things you spot that I've missed. I really also want to get this right. Thanks!


code cant render ui

VS Code Version: 1.54.1
OS Version: debian 10

Steps to Reproduce:

just open vscode

Does this issue occur when all extensions are disabled?: No, I deleted $HOME/.vscode directory

Are you running inside a vm ?
Disable gpu acceleration to workaround https://code.visualstudio.com/updates/v1_40#_disable-gpu-acceleration

Are you running inside a vm ?
Disable gpu acceleration to workaround https://code.visualstudio.com/updates/v1_40#_disable-gpu-acceleration
Yes. OK, I will try it later

Are you running inside a vm ?
Disable gpu acceleration to workaround https://code.visualstudio.com/updates/v1_40#_disable-gpu-acceleration

Yes. OK, I'll try it later

Merging to https://github.com/microsoft/vscode/issues/111290


Normalization of Citation Format Keys

I have many bibliography entries with different casing for the search keys. Could we have the picker normalize the entry keys before parsing the citations? Here is an example where a citation doesn't show up nicely in the picker even though it is valid bibtex:

Here's a normal one that is working:

Jep, that's a bug. Should definitely normalize the keys. Thanks for pointing this out. I will have a look at it.

Should be fixed with #37
Let me know if any issues remain!


Add docker build to PR builds; fix paths in Dockerfile

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

Comparison is base (7300295) 68.71% compared to head (91f3340) 68.71%.

:exclamation: Current head 91f3340 differs from pull request most recent head 3002e62. Consider uploading reports for the commit 3002e62 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage   68.71%   68.71%
=======================================
  Files          47       47
  Lines        4482     4482
=======================================
  Hits         3080     3080
- Misses       1207     1209    +2
+ Partials      195      193    -2

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


Parse co-published PubIDs

There are many PubIDs that are co-published. This means that the different organizations share the same "standard number". This also means that the "standard number syntax" will either comply with the IEEE syntax, or the syntax of the other publisher.
ASTM format:
IEEE/ ASTM SI 10-2010 (Revision of IEEE/ASTM SI 10-2002)

ANSI format:
IEEE/ANSI Std 24-1984 (Revision of 24-1977)
IEEE/ANSI Std 484-1987

ISO, IEC or ISO/IEC format but with the IEEE P applied:
IEEE/IEC P60076-57-1202, July 2014
IEEE/IEC P60076-57-129_FDIS, July2017
IEEE/IEC P60079-30-1 D5, January 2022
IEEE/IEC P60079-30-2 D4, January 2022
IEEE/IEC/ISO P80005-1/FDIS_March 2012
IEEE/ISO/IEC 8802-1Q-2020/Amd31-2021
IEEE/ISO/IEC 8802-3:2021/Amd 4-2021
IEEE/ISO/IEC 8802-3:2021/Amd7-2021
IEEE/ISO/IEC 8802-3:2021/Amd8-2021

Purely ISO, IEC or ISO/IEC format:
IEC/IEEE 60076-16 Edition 2.0 2018-09
IEC/IEEE 60076-57-1202:2017 Edition 1.0 2017-05
IEC/IEEE 60076-57-129:2017
ISO/IEC/IEEE 24748-5:2017(E)
ISO/IEC/IEEE 24765:2010(E)
ISO/IEC/IEEE 24765:2017(E)
ISO/IEC/IEEE 24774:2021(E)
IEEE/ISO/IEC 8802-3:2021/Amd 4-2021
IEEE/ISO/IEC 8802-3:2021/Amd7-2021
IEEE/ISO/IEC 8802-3:2021/Amd8-2021

@ronaldtse we agreed before to use pubid-iso to parse ISO identifiers. But should it be on the "pubid-ieee" level or "relaton" level? I believe "relaton" should decide what parser to use...
Also there are difference with what pubid-iso can parse now and what we have in our PubIDs list. Most of "ISO" identifiers there pubid-iso will not be able to parse without fixes, for example:
ISO/IEC 10861 : 1994 [ANSI/IEEE Std 1296, 1994 Edition]
ISO/IEC 13213 : 1994 [ANSI/IEEE Std 1212, 1994 Edition]
ISO/IEC 14515-1:2000 IEEE Std 2003.1-2000

we agreed before to use "pubid-iso" to parse ISO identifiers. Should it be on the "pubid-ieee" level or "relaton" level? I believe "relaton" should decide what parser to use...

I believe pubid-ieee should implement the link to re-use pubid-iso. Relaton technically does not do anything with the PubID except that it needs a PubID to identify a bibliographic item.
In the examples:

ISO/IEC 10861 : 1994 [ANSI/IEEE Std 1296, 1994 Edition] is ISO/IEC 10861:1994 with an alternative PubID ANSI/IEEE Std 1296, 1994 Edition
ISO/IEC 13213 : 1994 [ANSI/IEEE Std 1212, 1994 Edition] is ISO/IEC 13213:1994 with an alternative PubID [ANSI/IEEE Std 1212, 1994 Edition
ISO/IEC 14515-1:2000 IEEE Std 2003.1-2000 is ISO/IEC 14515-1:2000 with an alternative PubID IEEE Std 2003.1-2000

We could update pubid-iso to handle these, or make pubid-ieee handle them?

We could update pubid-iso to handle these, or make pubid-ieee handle them?

To parse them in pubid-ieee I need to duplicate parsing code from pubid-iso. Better to update pubid-iso for these identifiers.
Also I have an idea, maybe we should introduce something like this https://github.com/relaton/relaton/blob/main/spec/relaton/registry_spec.rb for PubIDs? e.g.:
expect(PubID::Registry.parse("ISO/IEC 13213")).to be_instance_of PubId::Iso::Identifier
expect(PubID::Registry.parse("IEEE/ANSI Std 484-1987")).to be_instance_of PubId::Ieee::Identifier

I fully agree with this. The challenge is on what is considered an ISO vs IEEE identifier. For example,
expect(PubID::Registry.parse("ISO/IEC 13213")).to be_instance_of PubId::Iso::Identifier
expect(PubID::Registry.parse("IEEE/ANSI Std 484-1987")).to be_instance_of PubId::Ieee::Identifier

There are however some challenges. Look at these identifiers:
ISO/IEEE DIS P11073-10418 D13, January 2011
ISO/IEEE DIS P11073-10418/D15, June 2011
ISO/IEEE DIS P11073-10418_D8, July 2010
ISO/IEEE P11073-20601a/D29, July 2010
ISO/IEEE P11073-20601a/D31, August 2010

The numbers are in IEEE format (Pnnnn-{part}/D{draft}), but starts with ISO. The first 3 even uses the ISO stage code DIS. From the ISO perspective, the P and D parts do not make any sense. These identifiers can only make sense from the IEEE perspective.

The numbers are in IEEE format (Pnnnn-{part}/D{draft}), but starts with ISO. The first 3 even uses the ISO stage code DIS. From the ISO perspective, the P and D parts do not make any sense. These identifiers can only make sense from the IEEE perspective.

So we should check not only the prefix but also if it's parsable by matched module.
For "ISO/IEEE P11073-20601a/D31, August 2010" by splitting publishers: "ISO" and "IEEE" we can check which module can parse it first and return correct instance.

For "ISO/IEEE P11073-20601a/D31, August 2010" by splitting publishers: "ISO" and "IEEE" we can check which module can parse it first and return correct instance.

Perhaps. I do not know whether we can know which parse operation is correct by simply comparing the outcomes though. Maybe the ISO version will fail once it encounters "Pnnnn"?
I'm not sure if we should handle this via parse rules where we can integrate multiple Parslet sub-rules...

Perhaps. I do not know whether we can know which parse operation is correct by simply comparing the outcomes though. Maybe the ISO version will fail once it encounters "Pnnnn"?

Yes, one parser will fail. If both succeeded to parse, it means both should return correct results.

I'm not sure if we should handle this via parse rules where we can integrate multiple Parslet sub-rules...

I also like the idea to include parse rules from another gem. For example we can use "pubid-iso" Parslet rules inside "pubid-ieee" to parse ISO part for identifiers like: "ISO/IEC13210: 1994 (E) ANSI/IEEE Std 1003.3-1991"

Yeah let's try out these ideas!

@ronaldtse I'm trying to parse this identifiers, for me it looks like identifier with 2 dual-PubIDs, but I believe I could be wrong.
"IEEE Std 802.5r and IEEE 802.5j, 1998 Edition (ISO/IEC 8802-5:1998/Amd.1)"
What is strange here it represend 2 dual-PubIDs with different format (first using " and " and second one use identifier inside brackets).
What is "IEEE Std 802.5r and IEEE 802.5j" here? (maybe this could help https://ieeexplore.ieee.org/document/827772)

According to this page:
http://gwise.itwelzel.biz/CDROMs/NASK4/Documents/IEEEStandards/IEEE802.5.html

"IEEE Std 802.5r and IEEE 802.5j" were combined and published as "ISO/IEC 8802-5:1998/Amd.1".
IEEE Std 802.5r and IEEE 802.5j are two separate IEEE Amendments. As we recall, IEEE Amendments use a lowercase letter after the number (802.5r means Amendment "r" of 802.5).


Handle JDK 10 version string

JDK 10 version string does not end with a " anymore which broke the regexp and sbt launcher would not even attempt to launch sbt under JDK 10.

Is the validation error a result of a flaky test on windows? I was not able to find the full log of what went wrong.

The log download button is towards the top-right:

Anyways the error seems to be a 404 error:
[00:01:48] jdk9 v<IP_ADDRESS> [Approved]
[00:01:48] jdk9 package files install completed. Performing other installation steps.
[00:01:49] Downloading JDK from http://download.oracle.com/otn-pub/java/jdk/9.0.1+11/jdk-9.0.1_windows-x64_bin.exe
[00:01:51] ERROR: Exception calling "DownloadFile" with "2" argument(s): "The remote server returned an error: (404) Not Found."
[00:01:52] The install of jdk9 was NOT successful.
[00:01:52] Error while running 'C:\ProgramData\chocolatey\lib\jdk9\tools\chocolateyInstall.ps1'.
[00:01:52]  See log for details.
[00:01:54]
[00:01:54] Chocolatey installed 0/1 packages. 1 packages failed.
[00:01:54]  See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
[00:01:54]
[00:01:54] Failures
[00:01:54]  - jdk9 (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\jdk9\tools\chocolateyInstall.ps1'.
[00:01:54]  See log for details.
[00:01:54] Command exited with code -1

Thanks for merging. Seems that the validation error is because of the outdated chocolatey package. The issue is being tracked here: https://chocolatey.org/packages/jdk9

I'm still getting this error upon trying to launch sbt with jdk 10:
[process_args] java_version = '10'

The Java Development Kit (JDK) installation you have is not up to date.
requires at least version 1.6+, you have
version 10

Please go to http://www.oracle.com/technetwork/java/javase/downloads/ and download
a valid JDK and install before running .

I'm using the rpm repository provided by this page and I launched sbt within a directory that had an explicit sbt 1.1.1 build.properties file.

I think we need a new release of sbt-launcher as there were more fixed to the java version parsing.


Ignore the nf_test_output parameter

nf-test introduces a custom parameter called nf_test_output. This should be ignored by default to ensure interoperability between the plugin and nf-test

A disclaimer should also be added to the doc that this parameter gets ignored by default


Setup travis

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@661f6bd). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             master     #1   +/-   ##
=======================================
  Coverage          ?   100%
=======================================
  Files             ?      3
  Lines             ?     41
  Branches          ?      0
=======================================
  Hits              ?     41
  Misses            ?      0
  Partials          ?      0

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 661f6bd...fe4b1a6. Read the comment docs.


Update timeline Redis backend for scalability

Replace aioredis
Add info on using Redis backend
Use client_id in mqtt client
Make sure event datetime has timezone info
Add PUBLISH_FREQUENCY
Use milliseconds timestamps in timeline
Add BATCH_SIZE; Use Lua for consuming timeline events
Make in-memory backend compatible with Redis
Switch to MQTT v5 protocol

Superseded with #39.

Superseded with https://github.com/karol-brejna-i/retailstoreofthefuture/pull/39.


Automatically generate a dummyMethodToEnforceBundling function, to make bundling in ios easier

currently it can be done manually. automate this can save 3 line of code.
also, we can make it a purely no-op, thus even easier to use.

oh I see! that func does not have impl, indeed!


Fix URI Path Reference

7.e: Change {var_uri_path_1} and {var_uri_path_1} are used to {var_uri_path_1} and {var_uri_path_2} are used
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8e83f232-bd7f-e2d3-1ff7-b40dc1d12e49
Version Independent ID: 12ec7b33-fc00-0428-69cf-38d70ff46948
Content: Rewrite HTTP request and response headers in portal - Azure Application Gateway
Content Source: articles/application-gateway/rewrite-http-headers-portal.md
Service: application-gateway
GitHub Login: @abshamsft
Microsoft Alias: absha

@simonkurtz-MSFT, looks like you have provided the wrong doc footer here. Hope you were referring to URL rewrite instead of Header rewrite.

Yes, sorry, I must have had multiple tabs open. It's URL Rewrite.
Simon Kurtz
he/him
Senior Customer Engineer
Midwest Customer Success Unit
(404) 877-2602
<EMAIL_ADDRESS>[Microsoft Logo]
If you have feedback about my work, please let my manager, Frank @.***?subject=Feedback%20Regarding%20Simon%20Kurtz>, know.
Upcoming Out Of Office: 8/9/21
From: suvasara @.>
Sent: Wednesday, August 4, 2021 1:36 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Simon Kurtz @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Fix URI Path Reference (#79287)
@simonkurtz-MSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsimonkurtz-MSFT&data=04|01|simon.kurtz%40microsoft.com|323437666665479c765708d9576e577f|72f988bf86f141af91ab2d7cd011db47|1|0|637636953694182678|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=19Gduq7ZlGWHeU1sVAVIsOB45q9XqwSzZx3do%2BF5tiM%3D&reserved=0, looks like you have provided the wrong doc footer here. Hope you were referring to URL rewritehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fapplication-gateway%2Frewrite-url-portal&data=04|01|simon.kurtz%40microsoft.com|323437666665479c765708d9576e577f|72f988bf86f141af91ab2d7cd011db47|1|0|637636953694192632|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VLLhHcmWXPUSaY3Rp6U2yC%2B9S%2F%2BY%2BC0rO0qVX3Ch%2Fw0%3D&reserved=0 instead of Header rewrite.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F79287%23issuecomment-892843817&data=04|01|simon.kurtz%40microsoft.com|323437666665479c765708d9576e577f|72f988bf86f141af91ab2d7cd011db47|1|0|637636953694192632|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=FyuXJIGq0QJGwAJkURNCp0ZCu%2BiSOG3J%2ByRm59uGF6M%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUHBQRK6A3BHAMAPCYMUTGLT3F3APANCNFSM5BRRXSLQ&data=04|01|simon.kurtz%40microsoft.com|323437666665479c765708d9576e577f|72f988bf86f141af91ab2d7cd011db47|1|0|637636953694202589|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=IaIwjbos%2BuHps6W9ftvG2%2FwRRZwxyKHxdwL2mip07Wk%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|simon.kurtz%40microsoft.com|323437666665479c765708d9576e577f|72f988bf86f141af91ab2d7cd011db47|1|0|637636953694202589|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Q1WNaVMp3j%2BCUlMDorrt0F8aakzrClCBpix4mAcEcU0%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26utm_campaign%3Dnotification-email&data=04|01|simon.kurtz%40microsoft.com|323437666665479c765708d9576e577f|72f988bf86f141af91ab2d7cd011db47|1|0|637636953694202589|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=xdag6bIdBa1xcUJJ9Kx4lIMj9wSHWJpzBcg3LRtCvjM%3D&reserved=0.

@simonkurtz-MSFT , I have updated the URI Path Reference in step 7.e and it will be reflected shortly.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


No hierarchy in the tree view panel

Hi,
As you can see I have no hierarchy in the tree view.
Do you have any idea from where it comes? (It is not the UI theme, I tyed with "Atom Dark" and had the same result).
I checked the generated code too and there is only one <ul> (no subtrees).
Thanks,
Damien

Same Problem on LaTeX with section and subsections! (Sould a make a own issue?)

Sorry for that, now symbols and their relationship are generated by ctags. For now ctags can't parse php or Latex as well as java and c++, and that's why there is no hierarchy in the tree view.
I'm tring to overcome this problem by using a myself parser, but it's not easy...

Thank you for your efforts @xndcn :+1:

Im having the same issue with Sass files, making the list barely usable due to multiple entries for generic symbols such as div, a, etc.
One way to get around this would be to limit the depth but surely having the list show the code nesting would be awesome.

The same is happening with JavaScript... CTags seems to not correctly parse modern js.
And regex rules are wrong too, it will only detect functions. Although I believe it is best like this right now, until hierarchy actually works.
Thank you for this awesome plugin!

Until hierarchy is fixed, I think colorizing and bolding functions, etc would help.

What ctags version do you use? Is there an upstream ticket for this?

+1

+1 to hierarchy
@jzumbrun : Colorizing is being tracked in issue #26 .
@gpascualg / @fuzzy76 : https://github.com/xndcn/symbols-tree-view/blob/master/lib/.ctags has the current version. I was thinking of submitting a PR for better JavaScript tags based on merging with https://gist.github.com/tim-smart/199338  .


Quiet Mode icon is not properly loading

Dev Home version
0.1301.472.0
Windows build number
10.0.26100.1000
Other software
OS Build Version: 10.0.26100.1000.amd64fre.ge_release_we_adept.240331-1700
.NET Version: .NET 8.0.3
Steps to reproduce the bug
Navigate to Windows Customization
Expected result
All the icons are properly loaded for the features inside of Windows Customization.

Fixed in the latest build


wrong colors in README

The image on GitHub has colors that don't fit the ones used in the table:

Whereas https://ke-complex-modifications.pqrs.org/#caps_lock_enhancement has the right colors:

我已经收到邮件，有空多联系~~


Swift2 update

Update to swift 2 syntax

The project has been updated for Swift 2.2/Xcode 7.3 via https://github.com/mutualmobile/VIPER-SWIFT/pull/15.


Fix ACE model missing license

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1023   +/-   ##
=======================================
  Coverage   29.77%   29.77%
=======================================
  Files         174      174
  Lines        4634     4634
  Branches     1220     1220
=======================================
  Hits         1380     1380
  Misses       3254     3254

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 2ceb0a0...4e6d869. Read the comment docs.


chore: bdd-spring-1585369875 to 0.0.1

chore: Promote bdd-spring-1585369875 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Darwin mdns fix

Problem
Use after free in darwin mdns causing flakiness. Also GetAddrInfo hangs if the interface is local ony.
Draft currently - opening PR to allow fixes from other devs.
Change overview
Use safe string copies to fix use after free, default to localhost on interface -1.
Testing
Tested by Andy

Closed because andy opened one with the updates on already - https://github.com/cecille/connectedhomeip/pull/1

Nope, I don't know how to read - Andy's PR is on this branch.

I'm sort of curious. Does the resolve for interface id -1 still fails once the memory corruption thing is fixed ?
For some context I'm running Big Sur and  applying this patch locally:
diff --git a/src/platform/Darwin/DnssdImpl.cpp b/src/platform/Darwin/DnssdImpl.cpp
index 8cb8d609e7..852ac72ae2 100644
--- a/src/platform/Darwin/DnssdImpl.cpp
+++ b/src/platform/Darwin/DnssdImpl.cpp
@@ -524,7 +524,7 @@ CHIP_ERROR ChipDnssdPublishService(const DnssdService * service)
         MdnsContexts::GetInstance().SetHostname(service->mHostName);
     }
     std::string regtype  = GetFullTypeWithSubTypes(service->mType, service->mProtocol, service->mSubTypes, service->mSubTypeSize);
-    uint32_t interfaceId = GetInterfaceId(service->mInterface);
+    uint32_t interfaceId = static_cast<uint32_t>(-1); //GetInterfaceId(service->mInterface);

     TXTRecordRef record;
     char buffer[kDnssdTextMaxSize];

And then launching the all-clusters-app: ./out/debug/standalone/chip-all-clusters-app then dns-sd -B _matterc._udp results into:
Browsing for _matterc._udp
DATE: ---Fri 19 Nov 2021---
17:35:16.524  ...STARTING...
Timestamp     A/R    Flags  if Domain               Service Type         Instance Name
17:35:16.525  Add        2  -1 local.               _matterc._udp.       C478BFCF4596349A

Running ./out/debug/standalone/chip-tool pairing onnetwork 0x12345 20202021 results into:
[1637339818779] [26712:46895140] CHIP: [IN] UDP::Init bind&listen port=5550
[1637339818779] [26712:46895140] CHIP: [IN] UDP::Init bound to port=5550
[1637339818779] [26712:46895140] CHIP: [IN] UDP::Init bind&listen port=5550
[1637339818779] [26712:46895140] CHIP: [IN] UDP::Init bound to port=5550
[1637339818779] [26712:46895140] CHIP: [IN] TransportMgr initialized
[1637339818779] [26712:46895152] CHIP: [DL] _OnPlatformEvent
[1637339818779] [26712:46895152] CHIP: [DL] Mdns: OnBrowseAdd  name: C478BFCF4596349A, type: _matterc._udp., domain: local., interface: -1
[1637339818780] [26712:46895152] CHIP: [DL] Resolved interface id:<PHONE_NUMBER>
[1637339818780] [26712:46895152] CHIP: [DL] Mdns: OnGetAddrInfo hostname:MacBook-Pro.local.
[1637339818780] [26712:46895152] CHIP: [TOO] Discovered Device: ::1:5540

I think the symptoms we are seeing here are slightly different, because the dns-sd command line tool also cannot resolve an address on these computers. I think you may be correct that there is a firewall blocking something to do with bonjour on our network. Not sure how common this will be across companies, but at the very least we know the memory fix alone does not get our devs unblocked.

fast track: platform specific change

fast track: platform specific change

/rebase


Add ability to calculate minimum free energy

This PR adds the ability to calculate minimum free energy of a given structure (in dot-bracket notation) and its sequence (needed for #145 ). The calculations compute the correct results for the given tests. The code still needs to be cleaned up and documented well.
@TimothyStiles I have to include a file for the energy parameters needed for the calculation. I was wondering if you'd prefer the three mfe files (mfe.go, mfe_test.go and energy_params.go) to be put in a folder, or leave them as is?

@TimothyStiles This PR is ready for review.
I've documented the code the best I can and only two functions need to be documented now (which I'll work on, but may take a while). Till then though, the rest of the mfe.go and mfe_test.go code is ready to be reviewed.
I need to work on energy_params.go, but that's just a lot of renaming variables and formatting the code to Go's style so should be done tomorrow.

@v-raja Is this PR stale given #176?

Nope. #176 parses energy param files into an EnergyParams struct. This PR actually figures out the secondary structure and uses the relevant energy values from the EnergyParams struct to calculate the minimum free energy.

Closed in favor of #173


[DOCS] GestureHandlerRootView hint for rn-gesture-handler v2

Given that the library is based on react-native-gesture-handler, docs should also have a hint on the v2 of the gesture handler -> for android, the NotificationsProvider should be placed within the GestureHandlerRootView

Closing this issue. Documentation directs to react-native-gesture-handler official docs with all installation guidelines.


BUG 2264002: rbd: log sitestatuses and description

This commit logs sitestatues and description in
GetVolumeReplicationInfo RPC call for better
debuging.
Fixes: #4430
Signed-off-by: Yati Padia<EMAIL_ADDRESS>(cherry picked from commit cdc73a73519282abb13db3fbf25c77ed6110d46d)

/hold to add another missing commit

/unhold
/lgtm

@yati1998 @Madhu-1 can you please confirm that the changes are available in the devel and release-4.16 branch too?
See https://github.com/red-hat-storage/ceph-csi/tree/devel/redhat#backports for the process that we are required to follow:

change in upstream
backport/sync to downstream devel branch
backport to downstream release-4.16 branch
backport to downstream release-4.15 branch <-- this PR

If this process isn't followed, we may run into a regression if we forget to backport the changes to any of the other branches.

@nixpanic yes changes are already in development as part of regular sync and 4.16 will be updated shortly as part of automation


Use latin hypercube sampler to initialize GP optimizations

This could help to ensure that the global minimum is actually found, and that any potential local minima are included in the cross-validation calculation (see #41).

Added GP hyperpriors and dramatically improved GP hyperparameter optimization in #55 and earlier commits. With these changes, maximizing the GP marginal likelihood seems to work well in general for approxposterior's use cases.


Add navmesh functions

This extension will allow users to access navmesh data from maps. Most if not all bugs have been ironed out by now so I'm opening this up for review.

okay so i have no idea why it's complaining about trailing whitespace then. seems like it's going for every line, super odd.

It's when you have spaces or tabs at the end of a line. I don't see any in the git editor, but might be there in the actual file.

Actually, I don't see any either. Maybe its the windows style newline.

I had thought so too, but nah there aren't any. You can download the file and see for yourself

Using Notepad++ 7.4.1

Ok, i'll review it sometime. The main worry I see in this code is potentially infinite loops.

so i've got a question, when using registerType, is there an explanation for what exactly the arguments are meant to do?
figured i'd ask since I somehow fixed not being able to assign a navmesh to an array/table entry
read something about a serializer. maybe use a combination of game.GetMap, game.GetMapVersion, nav:GetID? although since it's possible to create navmeshes independant of the bsp, there's not really a way to reliably checksum it.

Look at the source code, the parameters should be named and have some info about them. If not, I can check it out when I get some time.

oh okay found it, it was extpp.lua (extension preprocessor)
that's just amazing to me that it just reads the lua file straight like that. wow.

amazingly inefficient and hard to understand for the average lua user, ye.

Yes, there is CNavMesh:IsValid() that the global IsValid uses, which checks if the navmesh is not a NULL mesh such as the NULL_MESH specified.
If it’s about the function using retval as an argument, then I still don’t have a clue myself. That was more or less interpreted from the other available types such as CEffectData, Entity and Ranger data.
Sent from my iPhone

On 26 Oct 2019, at 12:17, TomyLobo<EMAIL_ADDRESS>wrote:

@TomyLobo commented on this pull request.
In lua/entities/gmod_wire_expression2/core/custom/nav.lua:

+local NULL_VEC = Vector()
+local NULL_TABLE = {}

+local aStarLoopCheck = 0
+local ASTAR_OP_MULTIPLIER = 10
+
+local Clamp = math.Clamp
+local floor = math.floor
+
+--------------------
+
+registerType("navmesh", "xnv", NULL_MESH,

nil,
nil,
function(retval)

if retval == nil then return end

I'm not sure, but doesn't IsValid already cover this?
Also, does IsValid work on CNavMeshes?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

astar todo: save a table of every navmesh by ID that has previously been closed and then reopened: increment its value by 1. if a navmesh exceeds a threshold value (opens and closes too many times) then we've entered a loop and should exit the function with false.

Is this ready now?

Nah mate. Not until I’m sure of myself.
Sent from my iPhone

On 6 Nov 2019, at 20:19, Sparky<EMAIL_ADDRESS>wrote:

Is this ready now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Is it ready at least to review?

probably not
tempted to just drop this on the workshop as an unofficial extension instead since i can no longer verify that it's up to scratch with a clear head. this is proving more stress than it was worth.

Want someone to spot you? Comon one more line! You can do it!! Just one more!

Focusing on school things right now. Just let the pull sit tight for a bit until I can come back to it, please. Shouldn’t take more than a month.
Sent from my iPhone

On 7 Nov 2019, at 11:39, Sparky<EMAIL_ADDRESS>wrote:

Want someone to spot you? Comon one more line! You can do it!! Just one more!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This will need to be worked on in the navext branch on my fork. my master will just be the current version of wiremod.


Feature Request: make the bot have a status

Hey, @stanleyyin3 , plzz elaborate on what you wish to do.