Security: DotEnv loads DB password plaintext in $_SERVER

This might be considered and decided upon already, but I was surprised when I ran phpinfo() to check extensions that it includes Environment values and thus, my database password (in plaintext). Technically $_SERVER and environment variables should never be exposed, but practically there are a lot of stray calls to phpinfo() which normally isn't ideal but doesn't display anything so compromising as DB connection info.

I've never seen this be done. It's really up to the developer to make sure there are is no phpinfo() around in the code. It's the equivalent of trying to encrypt passwords for the reason that developers have echo $my_password somewhere in the code.
If an attacker gains access to your server, they're going to have access to your password anyways. I know AWS Encrypts DB passwords in transit when you're using KMS or Opsworks/ElasticBeanstalk
Most IDE's have a way to check for things like phpinfo().

Gotcha, I'll follow you on this and say "dev problem" - closing.

@MGatner Use https://www.php.net/manual/en/ini.core.php#ini.disable-functions

Thanks @Paradinight - cool feature I didn't know about! I wasn't so much worried about this for myself, but generally. But if others don't think it is the framework's responsibility to keep $_SERVER clean then I won't worry about it.

We should add a info in the documentation and in the env file.


What's the difference about deprecated in Diagnostic and Semantic Token?

Hi there. There is a SemanticTokenModifiers.deprecated and a DiagnosticTag.Deprecated.
In TypeScript, a variable tagged deprecated by jsDoc(// @deprecated) will be added a strikethrough.
I tried semantic one but in my VS Code, most color themes don't support adding strikethrough to a token.
So I guess ts language server may use diagnostic one to implement that.
Is that the best practice to add strikethrough? If few themes add strikethrough to the deprecated semantic tokens, what is the usage of this semantic token modifier?

As always LSP doesn't define detailed rendering so it is up to the client to decide how to render deprecated for both semantic tokens and tags.
We had DiagnosticTags long before semantic tokens. This is why both exists.  I would actually use tags since more clients will support them that semantic tokens.

@dbaeumer I got it. Thanks for your time


Add tag(s) with version numbers

Hi there,
we are in the process of pointing the FreeBSD ports collection to this repo as new location for LuaDBI. Currently the LuaDBI version in FreeBSD ports is "0.5". However we noticed that there are no version number tags in this repo.
Of course we could just map some git commit to version 0.5 on our side. But in a couple of months some other distribution might face the same problem. So it would be great if you could tag the appropriate git commit with something like "v0.5". Thanks :-)
References:

Prosody Issue: https://prosody.im/issues/issue/747
FreeBSD bug: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=212884

Checkout #32
I also wait it for a while :)

Hmm, that's weird and frustrating. I used the migration tool that Google provided. I'll see what I can do (but I'll be away the next few days at least, so I probably won't be able to handle it before next week).

@mp1409 I hope you realize that sources on luarocks server do not match to sources in this repo?
So making tags need after do sync. Checkout #32

I understand that it is an undesirable situation that the luarocks version does not match this repo. However, my priority is to fix LuaDBI on FreeBSD. And the FreeBSD LuaDBI port depends on the version 0.5 from the original Google Code repo, which exactly matches commit 58513d50e800f6a4d4e999f9682c747c087eab6f, and not the luarocks version. This means syncing to the luarocks version and tagging that as v0.5 might break the build on FreeBSD again...
Anyway, I can only make suggestions, it is upon the Prosody maintainers (i.e. @mwild1) to decide.

Is there any chance that we getting that or at least a 0.6 release?

Tagged version 0.5 as specified above.
Additionally, 0.6 is released.

@sparked435 thank you! I will see if we can update the ports in FreeBSD now.


Issue with large wordlists that are formatted poorly

I am realizing that there is an issue with reading large wordlists that have a lot of junk in them that is formatted poorly. (Not valid UTF-8)
I will probably just change the way I am reading files.  This will probably be fixed fairly easily.

This issue has been fixed with the current release.  Closed.


The first contact of the recon service should be in the background

When opening a document with v3.6 SNAPSHOT, the Recon Service gets checked. But, it blocks the user interface, and I think it should be done in the backgroud.

(There seams to be a bug also, but that's for another issue…)
Proposed solution
Contact the Recon Service in the background.
Alternatives considered
Waiting.

How long is the wait? Is it especially burdensome?


What do we do after programming the board with open nic shell ?

I am currently working on SN1022 and I have programmed the board successfully. What is the next step? How do I connect my smart NIC to the host and the ethernet and send the data for the simple wire plugin that is provided and also for some other plugins that I can generate?

Hello @prateek-0328 ,
Same as other Alveo cards, you can install onic driver.
The only difference is that on U45N/SN1022, there is a AXI4-stream switch inside of box @250MHz. Please check https://github.com/Xilinx/open-nic-shell/blob/main/README.md. Especially, "MAC <-> QDMA subsystem 0 (On U45N, it is connected to host CPU) Select QDMA subsystem 0"
Hope it helps.


Remove version: CrystalDewWorld.CrystalDiskInfo.KureiKeiEdition version 8.12.4

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


removed the 'sudo' as this caused quite some trouble

As the title says.
In my case running the command as root caused quite some directories to be owned by root and this caused quite some problems.

So I did what was asked by the "Details" of the DCO step ...

I'll re try this huge piece of work ...


Fog Customizer icon added

Added a robust and world-class icon that emphasizes the creativity and expressiveness of such a wonderful venture of programming marvel

gg


PHP 84 support

PHP 8.4 was just released - please compile drivers with updated module api.

This is a duplicate of #1523.


vim-mode-change signalled twice

When entering insert mode from visual mode, vim-mode-change is signalled twice. Firstly with mode parameter as insert, then with normal.
Expected: it should only signal once changing to insert mode.
Repro: from normal mode press v, then c. (assuming there is a listener on the event).
exitVisualMode is called from enterInsertMode. exitVisualMode always signals a change to normal mode. vim.insertMode is already set to true at that point, so a check could be added there exitVisualMode:L3206.
I'm working on some unit tests, covering this bug and some other mode transitions.

Fixed in 5.55.0


"Schematic view of how things are done" link in the documentation is dead

Doc URL: https://sorl-thumbnail.readthedocs.io/en/latest/operation.html
Targeted URL: https://docs.google.com/drawings/edit?id=1wlE4LkQpzXd2a2Nxfjt6_j5NG7889dzMyf0V-xPAJSE&hl=en
Error:

@sorl, this was done a long time ago, but do you still have sources for this document?


ng build with deleteOutputPath not working

🐞 bug report
Affected Package
The issue is caused by package @angular/cli 10.1.7
Is this a regression?
Yes, the previous version in which this bug was not present was: 9.1.0
Description
Running ng build --deleteOutputPath=false causes the following error:
Unknown option: '--deleteOutputPath'
🔬 Minimal Reproduction
Create any new Angular 10 web application.
Run ng build --deleteOutputPath=false
🌍  Your Environment
Angular Version:

Angular CLI: 10.1.7
Node: 12.16.3
OS: win32 x64

Angular: 10.1.3
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, upgrade
Ivy Workspace:

Package                            Version
------------------------------------------------------------
@angular-devkit/architect          0.1001.7
@angular-devkit/build-angular      0.1001.7
@angular-devkit/build-ng-packagr   0.1001.7
@angular-devkit/core               10.1.7
@angular-devkit/schematics         10.1.7
@angular/cli                       10.1.7
@schematics/angular                10.1.7
@schematics/update                 0.1001.7
ng-packagr                         10.1.2
rxjs                               6.5.5
typescript                         4.0.5
webpack                            4.42.0

Can you share the build section of the angular.json?

@alan-agius4 I am using NX with custom webpack.config. Here is my build section from angular.json:
"build": {
          "builder": "@nrwl/web:build",
          "options": {
            "baseHref": ".",
            "buildLibsFromSource": true,
            "outputPath": "dist/apps/flww",
            "index": "apps/flww/src/index.html",
            "main": "apps/flww/src/main.ts",
            "tsConfig": "apps/flww/tsconfig.app.json",
            "webpackConfig": "apps/flww/webpack.dev",
            "progress": true,
            "scripts": [],
            "styles": [],
            "assets": []
          },

@KirillMetrik,
This issue is not caused by the official Angular CLI builder. Kindly open an issue with NX.
Thanks


feat: add web support

Close #20

Awesome going to check Web support this evening. 💯

Glad to hear it!
Let me know if you have any problems.

Works flawlessly. Maybe wanna add my VueJS example to the Readme? I'm currently writing a Medium article for the plugin I hope this is fine for you?
Usage

VueJs
import { FirebaseAuthentication } from '@robingenz/capacitor-firebase-authentication';

export default {
    name: 'FirebaseAuthentication',

    components: {
        //
    },

    data() {
        return {
            //
        };
    },

    methods: {

        async getCurrentUser () {
            const result = await FirebaseAuthentication.getCurrentUser();
            return result.user;
        },

        async getIdToken () {
            const result = await FirebaseAuthentication.getIdToken();
            return result.token;
        },

        async setLanguageCode () {
            await FirebaseAuthentication.setLanguageCode({ languageCode: 'en-US' });
        },

        async signInWithApple () {
            await FirebaseAuthentication.signInWithApple();
        },

        async signInWithFacebook () {
            await FirebaseAuthentication.signInWithFacebook();
        },

        async signInWithGithub () {
            await FirebaseAuthentication.signInWithGithub();
        },

        async signInWithGoogle () {
            await FirebaseAuthentication.signInWithGoogle();
        },

        async signInWithMicrosoft () {
            await FirebaseAuthentication.signInWithMicrosoft();
        },

        async signInWithPhoneNumber () {
            const { verificationId } = await FirebaseAuthentication.signInWithPhoneNumber(
                {
                    phoneNumber: '123456789',
                },
            );
            const verificationCode = window.prompt(
                'Please enter the verification code that was sent to your mobile device.',
            );
            await FirebaseAuthentication.signInWithPhoneNumber({
                verificationId,
                verificationCode,
            });
        },

        async signInWithTwitter () {
            await FirebaseAuthentication.signInWithTwitter();
        },

        async signInWithYahoo () {
            await FirebaseAuthentication.signInWithYahoo();
        },

        async signOut () {
            await FirebaseAuthentication.signOut();
        },

        async useAppLanguage () {
            await FirebaseAuthentication.useAppLanguage();
        }

    }

};

Angular
import { FirebaseAuthentication } from '@robingenz/capacitor-firebase-authentication';

const getCurrentUser = async () => {
  const result = await FirebaseAuthentication.getCurrentUser();
  return result.user;
};

const getIdToken = async () => {
  const result = await FirebaseAuthentication.getIdToken();
  return result.token;
};

const setLanguageCode = async () => {
  await FirebaseAuthentication.setLanguageCode({ languageCode: 'en-US' });
};

const signInWithApple = async () => {
  await FirebaseAuthentication.signInWithApple();
};

const signInWithFacebook = async () => {
  await FirebaseAuthentication.signInWithFacebook();
};

const signInWithGithub = async () => {
  await FirebaseAuthentication.signInWithGithub();
};

const signInWithGoogle = async () => {
  await FirebaseAuthentication.signInWithGoogle();
};

const signInWithMicrosoft = async () => {
  await FirebaseAuthentication.signInWithMicrosoft();
};

const signInWithPhoneNumber = async () => {
  const { verificationId } = await FirebaseAuthentication.signInWithPhoneNumber(
    {
      phoneNumber: '123456789',
    },
  );
  const verificationCode = window.prompt(
    'Please enter the verification code that was sent to your mobile device.',
  );
  await FirebaseAuthentication.signInWithPhoneNumber({
    verificationId,
    verificationCode,
  });
};

const signInWithTwitter = async () => {
  await FirebaseAuthentication.signInWithTwitter();
};

const signInWithYahoo = async () => {
  await FirebaseAuthentication.signInWithYahoo();
};

const signOut = async () => {
  await FirebaseAuthentication.signOut();
};

const useAppLanguage = async () => {
  await FirebaseAuthentication.useAppLanguage();
};

Glad to hear that!
Of course that's fine, thank you! I would love to read the article then.
The code in Usage is not framework-specific code. Just basic JavaScript examples.
So VueJS specific code doesn't fit in there very well.
But i would love to add a demo app to the README.md (like here).
Feel free to create a Vue app and send me the link.

Will there be a signInWithPhoneNumber support for web?


add Dispatch Reboot

fixes #443
fyi @ashawley @mdedetrich @xuwei-k @n8han @softprops

test run: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1128/consoleFull (404 til Jenkins recovers from his 72-hour NYE hangover)

new test run: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1129/consoleFull

hmm
[error] Fatal: multiple projects have the same artifacts visible in the same space.
[error]
[error]   net.databinder#unfiltered  from unfiltered and unfiltered, both visible in space "default"

not sure where that's coming from. I'll fiddle with it locally

doh, I forgot that in dbuild if you don't specify a branch you don't get the default branch, you get master

https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1130/consoleFull (404 til Jenkins finishes quaffing a fizzy, delicious glass of Alka-Seltzer)

hmm, this worked for me locally, but on Jenkins we get
[unfiltered] [error] Failed tests:
[unfiltered] [error] 	unfiltered.netty.SslServerSpec

let's see if it's intermittent: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1131/consoleFull

sigh, well let's just compile unfiltered's tests but not run them, then: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1132/consoleFull

I opened https://github.com/unfiltered/unfiltered/issues/364 on the test failure


typo in tooltip for "isShellCommand" in tasks.json

On Yosemite 10.10.5, Code v.0.10.3, hovering over the "isShellCommand" key in tasks.json gives the tooltip gives a typo: "external programm" instead of "external program".

(It doesn't have anything to do with the non-monospaced font I'm using - switching to Monaco gives me the same result.)
I traced this, to the function localize in vs/nls.js, which calls _format, but couldn't find anything weird in that invocation.
In a dev build of Code, on the current tip of master, I don't get the typo on the same tooltip, but I noticed that in this case the application launches as "Electron" and not "Visual Studio Code".
I was thinking this would be an easy fix and a nice way to get into the code. Are there any thoughts as to why this might be happening? Can anyone else reproduce it?

Whoops, so this was actually a typo in v10.10.3, but it's OK in master.


Isometric Tilemap Support

Basic isometric support with a new isoadventure demo can be found on my isometric branch
Most of my testing has been with the staggered isometric, which was the most difficult to get coordinate mapping to work properly.
Included test assets are pretty ugly, my first attempt at isometric tiles in blender.
It would be nice to support exposing tmx properties for layers/tiles so that walkability testing can be implemented but, that will need some more code cleanup and thought.

I wanted to give a bit of an update on this. I am leaving this branch as an example for myself and stepped back to do some refactoring of the tilemap code.
I am not really satisfied with the mess trying to use zIndex to allow an outside object to sort into the map, and I think it makes more sense to create ECS components for the map so that an object can have a map component that maps it into the map space.
The above reference isometric branch is non-breaking, and so is my current work.
I expect this will result in breaking changes, but, that should remove the need for tilemap consumer code to iterate the tiles adding SpaceComponents to each.

Hello @thecarp, would you mind if I used your branch to incorporate this into the current tilemap loading?


[Web Portal] Remove admin-lte layout dependencies

Most pages in web portal are using react and office-fabric framework now.
And our layout component (sidebar, navbar, background) still uses admin-lte which style pattern is different from office fabric/fluent.
I think it is time to remove the admin-lte depenedencies now.
TODO

Refactoring the layout component using office fabric and remove admin lte dependencies
Use ejs loader to implement a wrapper workaround for old jquery/ejs based pages.
Remove unused pages in web portal
Reorganize the polyfills and initialize functions (theme/font)

#4057


Quick fix generates invalid import

Environment data

Language Server version: 2023.10.40
OS and version: Linux
Python version: 3.11

Code Snippet
pyrightconfig.json
{
    "typeCheckingMode": "strict",
    "reportImplicitOverride": "error",
    "pythonVersion": "3.11",
    "include": [
        "**/*",
    ],
}

file.py
class A:
    def method(self):
        pass

class B(A):
    @override
    def method(self):
        pass

Repro Steps

Request a quick fixes on override

Actual behavior
Pylance suggests "from typing import override" even though that won't work unless I'm using Python 3.12.  The same is true for auto-import completions.
Expected behavior
There should be no provided quick fix.
Alternatively, I'd take the quick fix as long as I can get an error telling me it's not going to work based on the configured version of Python being too old.

Note also that if I explicitly have typing-extensions installed, I only get a quick fix from typing not typing-extensions.

new python.analysis.regenerateStdLibIndices option is added. if enabled, pylance will create stdlib indices per workspace rather than using prebuilt stdlib indices.

This issue has been fixed in prerelease version 2024.7.100, which we've just released. You can find the changelog here: CHANGELOG.md


Does Azure Map Conversion Note Apply to Shape Maps?

Does the note at the top of the page about Azure Maps general availability and converting Maps / Filled Maps to Azure Maps apply to Shape Maps? If you follow the link provided in the note to "convert maps and filled maps to Azure Maps visuals" the documentation only references Maps and Filled Maps, but does not reference Shape Maps.
Thank you!

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

ID: 6a301928-35b6-2b9d-3bbe-0dc48b237b07
Version Independent ID: 765bad52-68e2-985f-924e-4e7f334b62f0
Content: Use Shape maps in Power BI Desktop (Preview) - Power BI
Content Source: powerbi-docs/visuals/desktop-shape-map.md
Service: powerbi
Sub-service: pbi-visuals
GitHub Login: @msftrien
Microsoft Alias: mihart

Thanks for your comment, @PBrad. This issue has been assigned to the doc author for review.

It does not yet apply. I'll update this doc to remove that notice for now.

Thank you @msftrien and @bishalgoswami !

Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. Both the product and documentation might have changed since this issue was opened. Therefore, we are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.


[Bug] connected_devices entity showing incorrect information for node and connection type for clients on wired node

The problem
Recently changed a node from wireless to wired backhaul and now the connected devices entity sometimes displays totally incorrect information for wifi clients that are connected to the wired node.
wireless clients report they are wired and connected to the main router.
asus gui has correct information so this issue impacts the integration.
Your device model
RT-AX86S
Firmware type
Merlin
Firmware version
388.6
Integration version with the issue
0.29
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2024.2.1
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
here is what my connected devices is showing. all of the marked clients are incorrect and are actually connected to the node.
They are all wireless devices so not possible to be wired.

I have a similar issue with 0.28.0 where my nodes were showing up as using 2.4Ghz instead of my 5Ghz backhaul despite the Asus web UI confiming the 5Ghz backhaul. Reverted back to  0.22.0 as that has been the most stable version for me.

Please, allow me some time to get access to an AiMesh node and test this

Im facing the same issue. My device tracker reporting a wired connection and more troublesome, the wrong Node (Im trying to use the integration for room tracking with almost every important room containing an AiMesh node). Using v0.32

What I have ended up doing (for a number of reasons) is to stop using Aimesh nodes and setup each additional 'node' as an access point (AP) instead. For this you need to have all of the APs wired and not connected wireless. This is not always easy - I had to crawl around under my house with about 30cm space above (and lots of bugs - in live in Australia). Also your nodes wired will work sooo much better than mesh/wifi.
Overall, my opinion of aimesh is that it is not very clever and mainly marketing. The only real advantage is that you can see all of the connected clients in one view - but even this does not work particularly well as you have seen.
My dashboard looks like this now in HA and works perfectly all of the time.

and with each AP expanded to show clients...

Nice that you found a working solution! I will keeping using AiMesh, I like the "binding" feature of devices which are a bit stubborn sometimes. But I also found a workaround, I'm merely interested in room tracking and I noticed that the information was constantly flipping between 2/5ghz and wired. Thats why Im just ignoring the wired updates:
template:
  - sensor:
    - name: gj_connected_router
      state: >
        {% if 'ghz' in state_attr('device_tracker.gj_asus_wifi', 'connection_type') %}
          {% set node = state_attr('device_tracker.gj_asus_wifi', 'node') | upper if state_attr('device_tracker.gj_asus_wifi', 'node') != null else 'Away' %}
          {% set mapping =  states('sensor.routermapping') | from_json %}
          {{ mapping[node] if node in mapping else 'Away' }}
        {% else %}
          {{ this.state }}
        {% endif %}

Now it's perfect again:)


Contribution guideline for Interative Notebook needs to be modified.

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/community/contribute/docs
Description of issue (what needs changing):
Documentation under "Interactive notebooks" needs to be modified to accommodate issues created after direct editing of Jupyter Notebook in Colab.
Clear description.
Direct editing (on colab or using VSCode) Jupyter Notebook and committing as mentioned here adds additional unintended changes like prettifying and escapes unicode symbols.
For example, see here: https://github.com/tensorflow/docs/pull/1238
Related code commit : https://github.com/copperwiring/docs/commit/98f35604617d1ecc93b3dc75ac6ec4ab108536eb
Correct links
Yes
Parameters defined
N/A
Returns defined
N/A
Raises listed and defined
None.
Usage example
It is useful and needed for any PR request for Jupyter Notebook
Request visuals, if applicable
N/A
Submit a pull request?
I am planning to submit a PR to improve the documentation.

Thanks, @copperwiring
Unfortunately this is WAI since there are serialization differences between Colab versions. This is being tracked in an internal ticket (b/123414970)
cc: @csemike @corrieann


Vue3 installation and usage: a few pointers

I just installed and used this component in my Vue3/Quasar project. Here are the three things I struggeled with a little bit to get it to work. It's meant as a constructive critism. It may help future users. :-)
Don't install next, (2.0.0) but install latest (2.0.1), the import fix is a must-have.
I didn't know this thing was renderless, so took me a while to figure that out. I think renderless is better, and had almost selected a different component as I thought this one wasn't. I now realize that is what was meant by "Customizable"/"Unstyled". IMHO, you might want to clarify this somewhere.
The global component registration isn't necessary. Unless the buttons are used everywhere, it's probably better (because code splitting) to locally register them. Or not?? Right?
Quasar boot file for global registration:
import { boot } from 'quasar/wrappers'
import { SWhatsApp } from 'vue-socials'

export default boot(({ app }) => {
    app.component('SWhatsApp', SWhatsApp)
})

Local registration:
<script setup lang="ts">
    import { SWhatsApp } from 'vue-socials'

    const windowFeatures = {}
    const shareOptions = {
        number: '1(999)999-99-99',
        text: 'Hello world!',
    }

    const useNativeBehavior = false
    const onClose = () => {
        return
    }
    const onOpen = () => {
        return
    }
    const onBlock = () => {
        return
    }
    const onFocus = () => {
        return
    }
</script>

Usage (SVG from the Storybook demo).
The super-tiny-icons package might be good companion to this plugin.
                <s-whats-app
                    :window-features="windowFeatures"
                    :share-options="shareOptions"
                    :use-native-behavior="useNativeBehavior"
                    @popup-close="onClose"
                    @popup-open="onOpen"
                    @popup-block="onBlock"
                    @popup-focus="onFocus"
                >
                    <svg
                        xmlns="http://www.w3.org/2000/svg"
                        width="24"
                        height="24"
                        viewBox="0 0 24 24"
                        aria-hidden="true"
                        focusable="false"
                    >
                        <path
                            d="M.057 24l1.687-6.163c-1.041-1.804-1.588-3.849-1.587-5.946.003-6.556 5.338-11.891 11.893-11.891 3.181.001 6.167 1.24 8.413 3.488 2.245 2.248 3.481 5.236 3.48 8.414-.003 6.557-5.338 11.892-11.893 11.892-1.99-.001-3.951-.5-5.688-1.448l-6.305 1.654zm6.597-3.807c1.676.995 3.276 1.591 5.392 1.592 5.448 0 9.886-4.434 9.889-9.885.002-5.462-4.415-9.89-9.881-9.892-5.452 0-9.887 4.434-9.889 9.884-.001 2.225.651 3.891 1.746 5.634l-.999 3.648 3.742-.981zm11.387-5.464c-.074-.124-.272-.198-.57-.347-.297-.149-1.758-.868-2.031-.967-.272-.099-.47-.149-.669.149-.198.297-.768.967-.941 1.165-.173.198-.347.223-.644.074-.297-.149-1.255-.462-2.39-1.475-.883-.788-1.48-1.761-1.653-2.059-.173-.297-.018-.458.13-.606.134-.133.297-.347.446-.521.151-.172.2-.296.3-.495.099-.198.05-.372-.025-.521-.075-.148-.669-1.611-.916-2.206-.242-.579-.487-.501-.669-.51l-.57-.01c-.198 0-.52.074-.792.372s-1.04 1.016-1.04 2.479 1.065 2.876 1.213 3.074c.149.198 2.095 3.2 5.076 4.487.709.306 1.263.489 1.694.626.712.226 1.36.194 1.872.118.571-.085 1.758-.719 2.006-1.413.248-.695.248-1.29.173-1.414z"
                        />
                    </svg>
                </s-whats-app>

With Super-Tiny-Icons, add the import import stiWhatsApp from 'super-tiny-icons/images/svg/whatsapp.svg'
                <s-whats-app
                    :window-features="windowFeatures"
                    :share-options="shareOptions"
                    :use-native-behavior="useNativeBehavior"
                    @popup-close="onClose"
                    @popup-open="onOpen"
                    @popup-block="onBlock"
                    @popup-focus="onFocus"
                >
                    <img
                        :src="stiWhatsApp"
                        width="24"
                    />
                </s-whats-app>

Just my experience :-)

I apologize for that.
Components aren't fully renderless, it renders <a> tag, but icons aren't included, you're right. Global registration isn't recommended since it will increase bundle size. It's better to register the component at the place of use.
Your example with super-tiny-icons is correct, but don't forget to make image accessible.
<s-whats-app
  :window-features="windowFeatures"
  :share-options="shareOptions"
  :use-native-behavior="useNativeBehavior"
  @popup-close="onClose"
  @popup-open="onOpen"
  @popup-block="onBlock"
  @popup-focus="onFocus"
 >
   <img
      :src="stiWhatsApp"
      width="24"
      height="24"
      aria-hidden="true"
      alt=""
    />
 </s-whats-app>

Fixed in v2.0.2

That ARIA is a good addition!
Sidenote: This has nothing to do with vue-socials; choosing your image is mostly personal taste, I actually ended up with using Font-Awesome instead, because I wanted the icons with custom colors, which is not possible with super-tiny-icons, unless you edit the SVG or use JS SVG loaders. Just something to consider as well.


Can't install anymore

Installed as usual tried laravel 7,6,5.8
The command "'php' 'artisan' 'backpack:add-sidebar-content' '"<li' 'class="nav-item"><a' 'class="nav-link"' 'href="{{' 'backpack_url('''elfinder''')' '}}"><i' 'class="nav-icon' 'fa' 'fa-files-o">' '{{' 'trans('''backpack::crud.file_manager''')' '}}"'" failed.

Thank you for raising the issue @giojkd ! Indeed, there was a problem with the installation process - we were not correctly using the new Symphony\Process syntax.
I've just pushed a fix as 4.0.45. Please run composer update and php artisan backpack:install again, it should work fine this time. If it doesn't, please let us know.
Cheers!


feat: add spanish locale, fix: sidebar width (again)

Added spanish translation

now you removed the spanish translations?

does it work now?

no, it does not. Spanish locale is MISSING

PR is ready to merge. Flustix or rooot has to merge it tho, since i dont have permission for the main branch

i won't merge this as is - you are modifying the en_us locale.
can you fully remove the commits that do that and commit only the spanish locale file? (eg by force pushing)

nevermind me, im blind (it still showed the old PR state), this seems ready to be merged.

gh won't let me merge this


what's the valid mask

@Janspiry i want to train a inpainting diffuison by my own mask, some of them can get rational result but more of them get a black image. i wonder what's the valid mask, can someone give me a help or have somebody miss the same problem

i solve my problem by changing the picture storage format from JPEG to PNG. As the jpg format does not guarantee the binarization of images.

Thanks! will give it a try. Can I ask how's the model performance for you?

i load the pretrained model of inpainting_places and use 7w+ data, only after serveral epochs it perform great

So, I think we need to define a new function in 'mask.py' to load our own masks, right?
And I'm wondering if this generative way can maintain the diversity (that possessed by diffusion models) or not.
Hoping to get your insights about that.


:arrow_up: react-json-tree 0.2.0

this version should support react 0.14.

I'll close as I did this in https://github.com/gaearon/redux-devtools-log-monitor.


Turn Bat sprites into Birds

Fox among Bats will ultimately turn into Cat among Birds. The bats need to be reskinned into birds.

Steps for this Issue:

[ ] Make basic animation (colored prototype)
[ ] Outline the animation
[ ] Color the animation

Colored Prototype

That's an early version of the animation where every part of the bird has its own color:

New experimental animation! :)

The bats are black birds now. Their eyes glow red when they chase the player. (5ef87f441fd5a271b440627c7f172df7f89c2a57)


Moving commands don't work on DK-8-KZ

With NEJE DK-8-KZ Left, Right, Up and Down buttons do nothing.
I checked the code and found that it sends single byte (e.g. 0xf5) commands instead of (0xf5, 0x03) as native application does. The second byte is probably a number of pixels to move laser on.
It may be because of the difference between versions of software. But older (or different?) models of the printer could possible accept also two-byte commands.
UPDATE: After double-checking I found that in my version it uses the following commands to move the laser:
up: {0xf5, 0x01}
down:  {0xf5, 0x02}
left: {0xf5, 0x03}
right: {0xf5, 0x04}
Not sure if can be fit in the same protocol.

I have an NEJE JZ-5 and the move commands didn't work either. But as I didn't have a need the feature I was too lazy to check for the issue and left it as is. Poor excuse but that was the reason. :P
So first of all, thanks a lot for providing the codes. I will later check if these also work for mine.
Nevertheless, I'm currently unsure how I should implement it. As I have no idea if these buttons work for any machine at this time (or if there's actually someone who uses them), I don't feel like doing a hard-coded replacement. Probably a dropdown with the possibility to select the machine (or similar) is the most reasonable solution. Of course, feel free to provide better solutions.

Also I know how to retrieve burned pixels during printing:
connect(_ezGraver->serialPort().get(), &QSerialPort::readyRead, this,
&MainWindow::readyRead);
...
void MainWindow::readyRead()
{
QByteArray data = _ezGraver->serialPort()->read(1024);
//qDebug() << "received " << data.size() << " bytes: " << data.toHex();
if ((data.size() == 5) && (data[0] == 0xff))
{
    // pixel burned
    int x = data[1]*100 + data[2];
    int y = data[3]*100 + data[4];
    //_ui->progress->setValue(y * EzGraver::ImageHeight + x);
    // or
    // image->setPixel(x,y,...)
} else
if ((data.size() > 0) && (data[0] == 0x66))
{
    // done
    //_ui->progress->setValue(EzGraver::ImageHeight *

EzGraver::ImageWidth);
}
}
2017-04-10 17:47 GMT+03:00 Christoph Amrein<EMAIL_ADDRESS>
I have an NEJE JZ-5 and the move commands didn't work either. But as I
didn't have a need the feature I was too lazy to check for the issue and
left it as is. Poor excuse but that was the reason. :P
So first of all, thanks a lot for providing the codes. I will later check
if these also work for mine.
Nevertheless, I'm currently unsure how I should implement it. As I have no
idea if these buttons work for any machine at this time (or if there's
actually someone who uses them), I don't feel like doing a hard-coded
replacement. Probably a dropdown with the possibility to select the machine
(or similar) is the most reasonable solution. Of course, feel free to
provide better solutions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/camrein/EzGraver/issues/18#issuecomment-292972148,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADa0tSRGgccElVMhtU7n4ODNEg70vW_Dks5rukEDgaJpZM4M41oJ
.

Thanks a lot for the code to compute and display progress. I have already experimented with possibilities to display the progress earlier (with the progress bar and an estimated time). There were some things that didn't quite fit in a desirable quality of the result without having to hacky code.
Long story short, I like your idea with simply computing the progress with the pixel position. This way, there is no need to distinguish between continue and restart actions.

So it took some time, but I have finally integrated the support for different protocols with pull request https://github.com/camrein/EzGraver/pull/24.

Tested on my device. Works correct. Thank you!


No option to change button text language.

After opening the picker, the button on the top right corner, not able to change the language of that button to English.

Is the right button refer to the confirm button?

No matter what button it is, there's a text delegate to implement the custom text. Since you didn't follow the issue template, this issue will be lock and close now.
Please follow the instruction in the pinned issue.


DMA receive transaction timed out

Hi Brandon,
I am trying to install your driver for using with DMA on petalinux 2017.4 and run into some problems.
Here is what I have done:
root@zynq_petalinux:/lib/modules/4.9.0-xilinx-v2017.4/extra# insmod ./xilinx-axidma.ko
xilinx_axidma: loading out-of-tree module taints kernel.
axidma: axidma_dma.c: axidma_dma_init: 706: DMA: Found 1 transmit channels and 1 receive channels.
axidma: axidma_dma.c: axidma_dma_init: 708: VDMA: Found 0 transmit channels and 0 receive channels.

It is OK.
root@zynq_petalinux:/usr/bin# ./axidma_benchmark -i 1 -o 1
AXI DMA Benchmark Parameters:
        Transmit Buffer Size: 1.00 Mb
        Receive Buffer Size: 1.00 Mb
        Number of DMA Transfers: 1000 transfers

Using transmit channel 0 and receive channel 1.
axidma: axidma_dma.c: axidma_start_transfer: 298: DMA receive transaction timed out.`

What's the problem?Thank you.

Sometimes the error like this:
https://forums.xilinx.com/t5/Embedded-Linux/ZYNQ-7010-AXI-DMA-driver-Fail-TIMEOUT/td-p/833457

When i use DMA with SG mode,the problem has been solved.

Interesting, I'm going to have to make a note of this. The driver should work regardless of whether or not it's in SG mode. Something may have changed in the Xilinx driver. I'm going to close this issue since it's resolved.


Implement baseline Continuous Integration

[ ] add some unit tests
[ ] connect to TravisCI (on padamson account)
[ ] verify triggering of automated tests (on padamson account)
[ ] connect to TravisCI (on arichar6 account)
[ ] add CONTRIBUTING w/info on testing
[ ] add some documentation with Sphinx
[ ] connect to readthedocs.org
[ ] verify triggering of automated update to docs/docstrings
[ ] update CONTRIBUTING w/info on docs

Move issue to PPD Team fork and close


MP Defaults Test Automation v1.0

Context
Create one script that complete the following actions for MP defaults: Create, Edit, Evaluate, Revert to Offical Record

would like some clarification from the EPA on if every field in the create modal should be filled out

Test Complete


Support for OpenRouter

OpenRouter  is a great API alternative to the original OpenAI API as you already have access to all the models, including gpt-4-32k for example.
I was able to make it work locally:
prompt_node = PromptNode(
    "openai/gpt-4-32k",
    api_key=openai_api_key,
    default_prompt_template=prompt_template,
    model_kwargs={"api_base": "https://openrouter.ai/api/v1"},
)

https://github.com/deepset-ai/haystack/blob/main/haystack/nodes/prompt/invocation_layer/open_ai.py
    @property
    def headers(self) -> Dict[str, str]:
        headers = {"Authorization": f"Bearer {self.api_key}", "Content-Type": "application/json"}
        if self.openai_organization:
            headers["OpenAI-Organization"] = self.openai_organization

        if "openrouter.ai" in self.api_base:
            headers["HTTP-Referer"] = "http://test.test"
            headers["X-Title"] = "Test"

        return headers

See their Docs  for explanation of the difference to the official API.

We're evaluating different frameworks to use and a must is OpenRouter integration. Would be great if this was implemented. There's a growing number of developers using OpenRouter and Haystack would surely benefit from this.


iOS. Background listener not calling when send FCM Push Notification

Have an issue in iOS app background state and FCM Push Notification.
Steps to reproduce:

The backend sends FCM Push Notification.
iOS receives Push Notifications. Alert appears.
onBackground callback was not triggered.

Unfortunately, neither onBackground nor onMessage was not triggered.
Are there any additional parameters I have to provide for FCM to stimulate onBackground callback?
Received remote notification request 627E-B950 [ waking: 0, hasAlertContent: 1, hasSound: 1 hasBadge: 0 hasContentAvailable: 0 hasMutableContent: 1 pushType: Alert]
Remote notification request 627E-B950 can be modified: 1
Process delivery of push notification 627E-B950

Hello @ObranS, have you found a fix for this? On iOS we are sending the notification throuh APNs, but the background handler is still not called.


Touchid and kSecAttrAccessible

add support #32
add support for protecting item with touchID.
add promise for delete and add

@evollu, thank you for your effort again 👍 , is it a breaking change?

No. It is just additional feature

Don't worry. Thank you!!


Make it more clear when executing contract, attached eth goes to the contract

Actually in this situation we do not currently know where the ETH goes, it just disappears from our account 🤔. To know where it went, we would need to have ether transfers, which could come with new multicall...


[bug] Taskbar Overlay Issue on Windows 11 with Transparency and Fullscreen Configuration

Describe the bug
I'm facing an issue related to the UI on Windows 11 when working with a fresh new Tauri project. When the window configuration is set to both transparent and fullscreen. The taskbar is seemingly sent behind other opened windows, or it appears as if other windows are brought to the foreground and overlay the taskbar. This occurs despite having certain window elements placed behind the taskbar.
Reproduction

Create a new Tauri project on Windows 11 using the command npm create tauri-app@latest.
Modify the tauri.conf.json file to add "transparent": true and "fullscreen": true to the Window configuration.
Update src/styles.css to make the background-color: transparent.
Make sure there are some window elements positioned behind the taskbar (Paint or Notepad) for visual verification.
Run the Tauri application npm run tauri dev.

Expected behavior
The taskbar should maintain its normal position above all other windows, even when the Tauri application is set to be transparent and fullscreen.
Platform and versions
[✔] Environment
    - OS: Windows 10.0.22621 X64
    ✔ WebView2: 114.0.1823.58
    ✔ MSVC:
        - Visual Studio Build Tools 2022
        - Visual Studio Community 2022
    ✔ rustc: 1.70.0 (90c541806 2023-05-31)
    ✔ Cargo: 1.70.0 (ec8a8a0ca 2023-04-25)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 18.16.0
    - yarn: 1.22.19
    - npm: 9.6.7

[-] Packages
    - tauri [RUST]: 1.4.1
    - tauri-build [RUST]: 1.4.0
    - wry [RUST]: 0.24.3
    - tao [RUST]: 0.16.2
    - @tauri-apps/api [NPM]: not installed!
    - @tauri-apps/cli [NPM]: 1.4.0

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../src
    - devPath: ../src

Stack trace
No response
Additional context
Here is a video demonstration: Once the Tauri application is running, the taskbar seems to be sent behind the other opened windows, or these windows are moved into the foreground, overlaying the taskbar.
https://github.com/tauri-apps/tauri/assets/16103757/68b24708-e170-4fc4-bb87-9fab58622903

Here is another demo, where I start a non-fullscreen window with the same height/width as my monitor, but I move the position by 20 pixels to the right. When the transparent window (striped background) appears I resize it, so that it covers the 20 pixel gap I created, which will then cause the same overlay issue:
https://github.com/tauri-apps/tauri/assets/16103757/6fa0171e-89d2-44c7-885e-42de0c435f8d
This is the Window configuration I used:
"windows": [
  {
    "decorations": false,
    "transparent": true,
    "fullscreen": false,
    "alwaysOnTop": true,
    "visible": true,
    "focus": true,
    "skipTaskbar": true,
    "resizable": true,
    "title": "taskbar-bug",
    "width": 2560,
    "height": 1440,
    "x": 20.0,
    "y": 0.0
  }
]

I've also noticed that if I change the x and y value in the Window configuration to 0 and then set the LogicalPosition in the frontend to move 1 pixel to the right, then the overlay issue will not appear despite the window being the same size as my display.
await appWindow.setPosition(new LogicalPosition(1, 0));

This is the intended behavior of the underlying Windows API, from msdn docs:

Setting the value of fFullscreen to TRUE, the Shell treats this window as a full-screen window, and the taskbar is moved to the bottom of the z-order when this window is active

@amrbashir In my previous post https://github.com/tauri-apps/tauri/issues/7328#issuecomment-1619012632, I've provided a separate demo that illustrates the same problem when fullscreen is set to false.

I guess I should've included the full remark section:

Setting the value of fFullscreen to TRUE, the Shell treats this window as a full-screen window, and the taskbar is moved to the bottom of the z-order when this window is active. Setting the value of fFullscreen to FALSE removes the full-screen marking, but does not cause the Shell to treat the window as though it were definitely not full-screen. With a FALSE fFullscreen value, the Shell depends on its automatic detection facility to specify how the window should be treated, possibly still flagging the window as full-screen.

Even if the fullscreen is set to false, The Windows Shell will try to auto-detect if this Windows is a fullscreen window (in your case, when it is resized and covers the whole screen), it will mark it as a fullscreen window which will move the taskbar to the bottom of the z-order

The Windows Shell is automatically classifying the application as fullscreen, even though it is not explicitly set to be. This can result in the taskbar being rendered behind other open windows. This is likely due to the Shell's automatic detection system, which flags a window as fullscreen if it's resized to occupy the entire screen, even if the fullscreen Window option is set to false.
We can use the SetPropW() function before showing the window. This will signal to the Shell that the window should not be treated as a fullscreen window. As a result, the Shell won't adjust the taskbar's z-order (the order of overlapping windows along the z-axis, which determines which windows appear in front of others), ensuring that the taskbar maintains its position.
use windows::Win32::UI::WindowsAndMessaging::SetPropW;
use windows::Win32::Foundation::HANDLE;
use windows::core::PCWSTR;
use wchar::wchz;

let window = app_handle.get_window("main").unwrap();
let handle = HANDLE(true as isize);
let mut name: Vec<u16> = wchz!("NonRudeHWND").to_vec();
unsafe {
    SetPropW(window.hwnd().unwrap(), PCWSTR(name.as_mut_ptr()), handle)
};
window.show();

This is a rare use-case and I don't think we should be adding an API for it. We will consider if more users ask for it.


chore: remove redundant system-object metadata file

Current dependencies on/for this PR:

main

PR #167  (this PR)

This comment was auto-generated by Graphite.


[why-tcpudp] [ja] Refine ja translation about SCTP-over-UDP

original

SCTP is a reliable transport protocol with streams, and for WebRTC
there are even existing implementations using it over UDP.

Current translate lack of meaning about "over".
see also
WebRTC スタックコトハジメ

そのため UDP 上に無理矢理載せて使用するという方式をとっています。

@inductor Could you review this?


[Feature Request] If No Package is installed and --x-json is specified, vcpkg list should be empty

Is your feature request related to a problem? Please describe.
A bit of an annoyance
I was writing an app of sorts that more tightly integrates vcpkg and CMake
During that, for obvious reasons, I had to access vcpkg list
So as I was developing an app, I used vcpkg list --x-json
When packages are Installed, output is
{
...blah blah blah
Useful info
}
When nothing is installed
Output is
No packages are installed. Did you mean search?
Proposed solution
Show output as {} (an Empty Object)
Describe alternatives you've considered
On my side, I am checking for an Exception from JSON.parse currently
Additional context
Not really
It's also experimental so unstable behaviour is both fair and expected
Edit:-
Fixing This
The place to fix this would be
https://github.com/microsoft/vcpkg/blob/2a6442cb934de2f99c8ac5bbd8a9dcf5538aee98/toolsrc/src/vcpkg/commands.list.cpp#L96
Out here if JSON Mode is enabled, it should output {}
Should I create a PR?
I can help out here
        if (installed_ipv.empty())
        {
            if(args.output_json())
                     System::print2("{}");
            else
                     System::print2("No packages are installed. Did you mean `search`?\n");
            Checks::exit_success(VCPKG_LINE_INFO);
        }

The original message may be useful for users, however, show output as {} (an Empty Object) also make sense, @strega-nil, could you help confirm this issue?

This makes sense to me, although I'd do System::print2(Json::stringify(Json::Object(), {})).

Yeah that would be the better method
Sorry

Do you want to open a PR?

@strega-nil just opened a PR
Thanks for the help!!!


Documentation: Clarify how to run generator-tests

Original issue 187 created by soi-toolkit on 2011-11-03T08:52:27.000Z:
Developer docs on how to run the generator-tests needs to be updated to reflect the fact that the source-code generators have moved to a project of their own (i.e. no longer resides within the eclipse-plugin).
The text from:
http://code.google.com/p/soi-toolkit/wiki/DeveloperGuidelines#Running_tests
Generator tests
The generator is for now both tested and built manually using the built in support in Eclipse for building Eclipse-plugins.
Use Eclipse to run all tests in the project soitoolkit-generator-plugin.
needs to be changed to something like:
Generator tests
The generator tests are in the soitoolkit-generator project.
To run the tests:

Configure your environment in: soitoolkit-generator/src/main/resources/soi_toolkit_generator_plugin_default_preferences.properties
Run tests using Eclipse or Maven [TODO: maven example].

Also consider if the description for building the eclipse-plugin is affected by the generator extraction.

Documented here:
https://github.com/soi-toolkit/soi-toolkit.github.io/wiki/Developer Guidelines


Add instructions for install.sh to README

This is for https://github.com/jamiesykescreode/cdev-environment-local/issues/17

I'm not sure why this is complaining of a conflict....

Oh, it's complaining because I already did this in the install.sh branch, and my fork doesn't have an updated 1.x branch.
I will close this and think about what I've done


Updated Deployments concepts doc

This PR updated the concept doc for Deployments. This is a part of broader efforts to enhance the workloads controller docs. The corresponding example nginx deployment YAML file was also updated to reflect the removal of selector defaulting for apps/v1beta2 in kubernetes/kubernetes#50164.
@janetkuo

This change is

/assign @janetkuo

We can merge this as soon as we get the final LGTM from @janetkuo.

Tech part LGTM, wait for another docs review


opentelemetrytracer: allow configuring flush_interval_ms and min_flush_spans with tracer config

Title: allow configuring flush_interval_ms and min_flush_spans with tracer config
Description:
The OpenTelemetry tracer has two runtime settings that control the behavior of when spans are exported. One is flush_interval_ms which controls the timeout for a export to happen and the other is min_flush_spans which controls the size of the batch to trigger also a export.
Both of these today need to be enabled via the runtime config. I may be lacking context but when using Istio, to be able to change these I would need to deploy a ConfigMap and add a annotation to all my pods in order for Istio to know that it needs to "merge" this config when injecting the sidecar proxies.
I'm not sure if deploying configmaps is a common thing and users are used to it, but as a OpenTelemetry contributor/user, such configurations are part of the "SDK" configuration. For ex, the batch processor has similar parameters. Similarly, the OTel collector also has the same.
So I thought Envoy could also expose these two properties as part of the "normal" OTel tracer:
provider:
  name: envoy.tracers.opentelemetry
  typed_config:
    "@type": type.googleapis.com/envoy.config.trace.v3.OpenTelemetryConfig
    service_name: envoy-HTTP-exporter
    [other config]
    flush_interval_ms: 6000 <<<<<< NEW
    min_flush_spans: 512    <<<<<< NEW

[optional Relevant Links:]
https://github.com/envoyproxy/envoy/issues/35997#issuecomment-2388492731

CC @AlexanderEllis

Can we make this never stale?

Am I correct in my current understanding of the OTel Tracer implementation.
Is it currently using a default of min_flush_spans to 5 spans?
https://github.com/envoyproxy/envoy/blob/52b8519fb5c9f94e00e57c129c1eb1a6eaf9f5a7/source/extensions/tracers/opentelemetry/tracer.cc#L247
Does that mean a batch will have at most 5 spans before exporting it to the collector?
https://github.com/envoyproxy/envoy/blob/52b8519fb5c9f94e00e57c129c1eb1a6eaf9f5a7/source/extensions/tracers/opentelemetry/tracer.cc#L248

Yes, that's also my understanding. The values are very low. Together with this change in allowing them to be configured like the others settings are, I'd also like to increase the defaults, so they align with the BatchSpanProcessor spec: https://opentelemetry.io/docs/specs/otel/trace/sdk/#batching-processor

And there isn't any way to set them now because the values are absent in the proto correct?

You can set them yes. They are set via the runtime config https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/operations/runtime
For example, when using Istio, they can be globably set via the MeshConfig:ProxyConfig https://istio.io/latest/docs/reference/config/istio.mesh.v1alpha1/#ProxyConfig like so:
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
  meshConfig:
    defaultConfig:
      runtimeValues:
        tracing.opentelemetry.flush_interval_ms: "300000" <<<<< ADD HERE

Those land then as a static layered config in Envoy:
      - name: global config
        static_layer:
          envoy.deprecated_features:envoy.config.listener.v3.Listener.hidden_envoy_deprecated_use_original_dst: true
          envoy.reloadable_features.http_reject_path_with_fragment: false
          overload.global_downstream_max_connections: "2147483647"
          re2.max_program_size.error_level: "32768"
          tracing.opentelemetry.flush_interval_ms: "300000"

Can we make this never stale?


Upgrade oneDNN to v2.2.2

Description
This change upgrades oneDNN used on master branch to v2.2.2.
Checklist
Changes

[x] Change oneDNN tag

@mxnet-bot run ci [windows-cpu]

@mxnet-bot run ci [website, windows-cpu]

i think this pr will fix website pipeline https://github.com/apache/incubator-mxnet/pull/20263

@mxnet-bot run ci [unix-cpu]

merged #20263, you can try to rebase : )

@mxnet-bot run ci [unix-gpu]


Update broken link to post install steps

Proposed changes
I clicked the link, and it was broken. I updated the link to go to the right page.

Can you please double-check the link? It works for me: https://docs.docker.com/engine/installation/linux/docker-ce/fedora/, go to the bottom and the link URL is https://docs.docker.com/engine/installation/linux/linux-postinstall/, which works for me. Your fix actually breaks it as the file is a directory up from the fedora.md which is in the docker-ce/ subdirectory.

Misty,
I apologize. The link is indeed broken--I just fixed broke a working link. There are 3 links to postinstall steps, and one of the three is different.


UI API for deleting a shopcart and a product

IBM Cloud toolchain: Delivery Pipeline deployed shopcarts to dev, including commit 19b31029e1f2b01cb8dad985bbd1529e7bdb2782

IBM Cloud toolchain: Delivery Pipeline deployed shopcarts to prod, including commit 19b31029e1f2b01cb8dad985bbd1529e7bdb2782


I412 dhc network path

Thermo-hydraulic modeling of district heating and cooling networks.
This function performs thermal and hydraulic calculation of a "well-defined" network, namely, the plant/consumer substations, piping routes and the pipe properties (length/diameter/heat transfer coefficient) are already specified. This model is valid for networks with a tree topology (i.e., no loops) with one or multiple plants, but requires the output from each plant to be pre-defined (at the moment, it is assumed all plants supply the same share of the total demand).
The function does the following: (i) calculate substation heat exchanger specification, (ii) import network edges and nodes (from either csv files or shapefiles), (iii) assign pipe properties, (iv) for each time step, calculate network temperatures, pressures, plant requirements and pressure losses.
This pull request solves issues #393 , #407 , #408 , #409 , #412 . @shanshanhsieh @martin-mosteiro

@BhargavaKrishnaSreepathi would you mind reviewing this Pull Request?

Hey @JIMENOFONSECA @daren-thomas , can you pass me the 'manual of how to check the pull request'.

You may ask it to the authors of the pull request directly.
Best, Jimeno
On 10 Feb 2017, at 10:58, BhargavaKrishnaSreepathi <notifications<EMAIL_ADDRESS>wrote:
Hey @JIMENOFONSECAhttps://github.com/JIMENOFONSECA @daren-thomashttps://github.com/daren-thomas , can you pass me the 'manual of how to check the pull request'.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/architecture-building-systems/CEAforArcGIS/pull/476#issuecomment-278844722, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIjrgsId2oWKN-IZQbFSg_zST6zwOyRsks5ra9JDgaJpZM4L61Uk.

@shanshanhsieh @martin-mosteiro : can you let me know what is the output expected after these changes? and what are the results i need to look at to verify this request

@shanshanhsieh in the mean time please solve the current conflicts of the branch...

@JIMENOFONSECA @shanshanhsieh I took the liberty to solve the conflicts!
@BhargavaKrishnaSreepathi @shanshanhsieh probably makes sense to clarify this in person on Monday?

@BhargavaKrishnaSreepathi is this working in your computer? where should I start to test this? maybe a guide could help..

Hey @JIMENOFONSECA , I think it makes more sense to test this Pull request when the network optimization part is done. I am actually trying to figure out the procedure for optimization now

@BhargavaKrishnaSreepathi just for usability. It would be great to have this optimization working separately of the whole conversion system. The reason is that we can have this function activated if multiple network configurations want to be analyzed, before using the optimization of conversion systems.

@JIMENOFONSECA : Noted , Actually working on a structure similar to using 'multi-processing', where we can either use it or let the default setting run.

@martin-mosteiro could you solve conflicts and create a readme.md in the folder to understand how to execute the work and test it? @daren-thomas does this need to be included in the dodo tests?

yes, it should be included in the dodo tests. we need to formalize how to prove it is still working. also unit tests should be written. I guess I'll have to hold a talk on Test Driven Development someday...

@shanshanhsieh is this ready to be merged?

@JIMENOFONSECA  It is currently being reviewed by @BhargavaKrishnaSreepathi

@JIMENOFONSECA @shanshanhsieh : We have been working on this branch. Currently, Shanshan is simulating the thermal and pressure losses in Simulink. Once we have the estimated losses from Simulink, we will compare with the losses calculated from the network file generated by @shanshanhsieh and @martin-mosteiro. Hopefully this will be ready by next week to be merged

@BhargavaKrishnaSreepathi closing this PR until ready. it would need to pass updates from the other branches first.


Use PROJECT_SOURCE_DIR for CMAKE_MODULE_PATH

The error that occurs using CMAKE_SOURCE_DIR for CMAKE_MODULE_PATH is, that the HIDAPI  package could not be found due to the missing config file FindHIDAPI.cmake. So it is not possible to use OpenHMD in other CMake projects. Using  PROJECT_SOURCE_DIR instead of CMAKE_SOURCE_DIR solves the issue.
For more details see the CMake documentation:
CMAKE_SOURCE_DIR is the path to the top level of the source tree (https://cmake.org/cmake/help/v3.1/variable/CMAKE_SOURCE_DIR.html). It differs, if OpenHMD is included to other projects.
PROJECT_SOURCE_DIR is the top level source directory for the current project. This is the source directory of the most recent project() command. (https://cmake.org/cmake/help/v3.1/variable/PROJECT_SOURCE_DIR.html). Due to the project is defined in the top level CMakeLists.txt the correct path will be used to find the modules.

@ChristophHaag: Can you review it?

Sounds good to me. I don't think this will break anything, or can anyone else think of something?

Hmm interesting, never had issues with using OpenHMD in other CMake projects like this. I want to look into this a bit before merging to check some build setups and see what makes the most sense.

@TheOnlyJoey did you check build setups if merge is possible?

@ChristophHaag @TheOnlyJoey: Some progress? Did you need further informations?

Please don't spam the issue tracker with questions about updates, things are done when they are done ;)


Provide a dummy store for redux-dependent fixtures

Need
As a user
I want to be able to load components/fixtures that need a store
So that I can preview them.

Deliverables
Fixtures that need data from a redux store do not throw an error on selection, and load normally.
Solution
Wrap the rendered component in a <Provider> component, to supply a dummy store for it.

@flavius-tirnacop-hs @valentin-radulescu-hs @catalin-miron-hs Please review changes.

LGTM

@catalin-miron-hs @andrei-picus-hs I addressed all the comments in the new commits. Please tell me if there's anything left for me to fix.


WIP Allow wait template to run the remainder of the script

New feature: allow wait template to run the remainder of the script
Description:
This allows the Wait Template script to use a timer to continue running the script if timeout is reached vs the current behaviour of exiting.
This is achieved by setting a new flag 'proceed' to 'true' in the config.
By default this flag is set to 'false' and is optional to ensure this is not a breaking change.
For this flag to work the optional 'timeout' is required to be set also.
Pull request in home-assistant.github.io with documentation (if applicable):
home-assistant/home-assistant.github.io#5954
Example entry for configuration.yaml (if applicable):
# if the front door is open and it's night time turn on the light
# when the occupant reaches the first sensor, turn the light off
# if the timeout triggers, turn the light off anyway
automation:
  - id: '15'
    alias: Door Open Light On
    trigger:
    - entity_id: binary_sensor.front
      platform: state
      to: 'on'
    condition:
    - condition: state
      entity_id: light.entrance
      state: 'off'
    - after: '17:00'
      before: '6:00'
      condition: time
    action:
    - data:
        entity_id:
        - light.entrance
      service: homeassistant.turn_on
    - wait_template: '{{ is_state("binary_sensor.entrance", "on") }}'
      timeout: 00:01:00
      continue_on_timeout: 'true'
    - alias: ''
      data:
        entity_id:
        - light.entrance
      service: homeassistant.turn_off

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

Hi @lhovo,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @lhovo,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Thanks for your contribution. There are several outstanding issues in your pull request you have to resolve before review.

Please do not change file mod 100644 → 100755
Please fill in PR template's "Example entry for configuration.yaml" section, in your case template and script
Please add unit test, home-assistant/tests/helpers/test_script.py
Please fix the link to your document PR (remove the space between # and issue number)

Please do not change file mod 100644 → 100755

Whoops this was a mistake

Please fill in PR template's "Example entry for configuration.yaml" section, in your case, maybe template and script

I think I have done this now?

Please add unit test, home-assistant/tests/helpers/test_script.py

Added

Please fix the link to your document PR (remove the space between # and issue number)

Done

Thanks for your patience walking me through my first home-assistant PR

Regarding Example entry for configuration.yaml

new flag 'proceed' to 'true' in the config.

I want to see the "example config" to demo how to set proceed

Wait Template script to use a timer to continue running the script

I want to see a example to demo "Wait Template script to use a timer"

Thanks, I was battling with the correct word to use.
I was originally trying to stay away from 'continue' as it's a key word in programming.
I have moved the "if CONF_CONTINUE" inside the _async_set_timeout.
It's passing the tests presently, but I would prefer to integrate it with my HA to test it some more to ensure the logic is 100% correct (assuming the test is correct, it should be fine)
I will endeavour to do this tomorrow

Not convinced the code is working as expected, when running on my home system.
Will do more testing soon, in the meantime I'm going to mark this as WIP

I think we should consider for this to be the default.

I would love for this to be the default, though that would make it a breaking change.

I agree with @amelchio that this should be the default behavior.

I have spent the last week testing the present code and it is working as expected on my system

As I have only just started to use Home Assistant and contribute back,
I am unsure on the steps required to change the default behaviour.
I can change the code and update the docs, is there anything else that needs to be done?
Like a vote between core developers?

Having @balloob on board is enough :)
You tag the PR with "breaking change" and add a small note at the top describing the incompatibility. This note is copied to the release note and that's that.

Thanks for being so inviting to your project!


Feature/hee 262 hee 272 search filter updates

For HEE-272 I've added 'hee:event', 'hee:caseStudy', 'hee:listingPage' to include in the search results page and some test content.
For HEE-262 I wasn't sure whether there'd be a requirement to reintroduce this functionality with more relevant filter types, so have commented this out in the ftl template for now.

Hi @smanivasag - thanks for reviewing this. I've addressed most of your comments. A couple of areas for contention are:

list-item.ftl

the entire code probably needs refactoring if we're to display these like the listing pages to use common macros. thought it may be best for design team to review what we have currently before further effort spent on this.

repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[1]/searchDocument/hippo-resource

I wasn't sure if the extra nodes are duplicates and need to be removed. Also wether we can safely remove hippo-resource files?

Whether we ought to remove the comments, as we can always refer back to a previous commit, if we need to re-introduce.

Hi @smanivasag - thanks for reviewing this. I've addressed most of your comments. A couple of areas for contention are:

list-item.ftl

the entire code probably needs refactoring if we're to display these like the listing pages to use common macros. thought it may be best for design team to review what we have currently before further effort spent on this.

repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[1]/searchDocument/hippo-resource

I wasn't sure if the extra nodes are duplicates and need to be removed. Also wether we can safely remove hippo-resource files?

Whether we ought to remove the comments, as we can always refer back to a previous commit, if we need to re-introduce.

Hi @Raq-HEE - Thanks for addressing all the review comments. Have made 29d75f73c74148f08988df7a3638153f96ca1aa9 commit containing minor fixes. Could you review ?

Re. list-item.ftl refactoring - Sure, I am happy for design team to feedback on this.

Re. hippo-resource - hippo-resource are seem to be empty placeholder files that brXM creates when the author doesn't add a file for the document field i.e. It seems to be default file for empty document fields. I think the following files can safely be removed as they are not being referenced in repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank.yaml:

repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[1]/searchDocument/hippo-resource
repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[2]/searchDocument/hippo-resource

But, I think we still need the following files as they are being referenced in repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank.yaml:
repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[1]/strategyDocument/hippo-resource
repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[2]/strategyDocument/hippo-resource
repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[3]/strategyDocument/hippo-resource

Re. commenting out the filter functionality - I would probably lean towards removing the commented out filter functionality snippets as we can always refer back the previous commit if required as you mentioned. A new team member may not know that the filter functionality existed previously, but this doesn't qualifies to keep the commented out snippets. Thanks!

Update
Re. list-item.ftl refactoring - The PR will be deployed to dev for design team review (as agreed with Robin).
Re. hippo-resource - Removed the following unreferenced hippo-resource files:
repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[1]/searchDocument/hippo-resource
repository-data/development/src/main/resources/hcm-content/content/documents/lks/search-bank/search-search-bank/search-search-bank[2]/searchDocument/hippo-resource

Have committed this work as part of e38e7f6de5d2661022fb9335491f41b182c64352.
Re. commenting out the filter functionality - Removed the commented out search filter functionality code snippets from freemarker templates as well as from component java class as this functionality could be referenced back via previous commits later if required. Have committed this work as part of 0a06f7ebec9c14884820c1a53e1a306256b7765f.


the trait bound f32: From<f64> is not satisfied error on nightly

Describe the bug
While compiling epaint, this error occurs:
error[E0277]: the trait bound `f32: From<f64>` is not satisfied
    --> epaint-0.26.2/src/tessellator.rs:1816:34
     |
1816 |         let stroke = Stroke::new(2.0, Color32::from_rgb(150, 255, 150));
     |                      ----------- ^^^ the trait `From<f64>` is not implemented for `f32`, which is required by `{float}: Into<f32>`
     |                      |
     |                      required by a bound introduced by this call
     |
     = help: the following other types implement trait `From<T>`:
               <f32 as From<bool>>
               <f32 as From<f16>>
               <f32 as From<i16>>
               <f32 as From<i8>>
               <f32 as From<u16>>
               <f32 as From<u8>>
     = note: required for `f64` to implement `Into<f32>`
note: required by a bound in `stroke::Stroke::new`
    --> epaint-0.26.2/src/stroke.rs:23:28
     |
23   |     pub fn new(width: impl Into<f32>, color: impl Into<Color32>) -> Self {
     |                            ^^^^^^^^^ required by this bound in `Stroke::new`

To Reproduce
Steps to reproduce the behavior:

Use latest nightly rust
Build epaint

This will be fixed in next nightly, however the impl that broke inference might be readded in the future, so we might want to make this code more robust against inference errors anyway.


Fixed windows install client

Support for specified architectures

Should we update the document as well? I think we need to define OS_ARCH environment variable when we compile the node client.

Yes I think the documentation needs to be updated, I also think os_arch is an optional field, if not defined it will use this \vcpkg_installed\bin\ to find dependencies


Greenkeeper/default/eslint plugin import 2.20.1

This update doesn't seem to break anything, and I've merged in the latest changes from develop.

Coverage remained the same at 84.878% when pulling 957bfde518821f659c0349e432acea47dafded9d on greenkeeper/default/eslint-plugin-import-2.20.1 into 78ad3eab107fb5fbb77dcecccf5439fb0aa575fa on develop.


Download file getting Error 400: File download has failed or was canceled

I have set up 2 different accounts using tdlight/td.
They are running on two different servers and having the same issue when trying to download different files.
it.tdlight.client.TelegramError: 400: File download has failed or was canceled
	at it.tdlight.client.Result.get(Result.java:36)

I changed the log verbosity and this appears on the attempt to download the file:
[FileManager.cpp:2386][#1][!FileManager]	Need to redownload file {id}(0): [Error : 400 : Can't find real file path]

This is not the error you are looking for, but the actual error should be also logged with verbosity level 2.

@levlam Thank you for the fast reply.
Yeah, I've tried logged with verbosity level 2, but didn't find anything interesting.
So I'm downloading the file in parts each one with a max of 10 Mb. The problem is happening because these 10 Mb parts are taking too much time to download like +1 minute. The request gets a timeout from my side.
If we think about files of up to 4Gb that can take hours (even if a let the download runs without closing)
I understand that can be not related to the lib itself, but is unusable because I have had the code running for years and never took time to download just 10 Mb. Using an instance with up to 5 Gb of bandwidth this is odd.
Do you have some idea or reason that could start happening right now?


Can't define function without prototype

int f();
int f(int i) { return i; }
thread 'main' panicked at 'should not have an error declaring a function: IncompatibleSignature("f", Signature { params: [], returns: [AbiParam { value_type: types::I32, purpose: Normal, extension: None, location: Unassigned }], call_conv: SystemV }, Signature { params: [AbiParam { value_type: types::I32, purpose: Normal, extension: None, location: Unassigned }], returns: [AbiParam { value_type: types::I32, purpose: Normal, extension: None, location: Unassigned }], call_conv: SystemV })', src/libcore/result.rs:1084:5

Crazy idea: instead of silently allowing you to do this like literally every other compiler, take the first function call as registering the parameters for the function to be that of the promoted arguments. Then, any conflicting function call, definition, or declaration following will be an error. This seems to be allowed by the C standard:
If  the  expression  that  denotes  the  called  function  has  a  type  that  does  not  include  a prototype,  ... [and] the  number  of  arguments  does  not  equal  the  number  of  parameters,  the behavior is undefined. If  the  function  is  defined  with  a  type  that  does  not  include  a  prototype,  and  the  types  of the  arguments  after  promotion  are  not  compatible  with  those  of  the  parameters  after promotion, the behavior is undefined, except for the following cases:
— one promoted   type   is   a   signed   integer   type,   the   other   promoted   type   is   the corresponding unsigned integer type, and the value is representable in both types;
— both  types  are  pointers  to  qualified  or  unqualified  versions  of  a  character  type  or void.

promoted signed/unsigned integers aren't a problem since cranelift doesn't distinguish between them for values
pointers are (currently) always represented by the same IR type

If so, I don't see a reason not to do it, especially since adding this in Cranelift will be harder than just disallowing it.

This breaks for function pointers.  Then updating one type would update all pointers to it as well.

That still breaks, if int f(), g(); are declared they could take different arguments, and if the registry special-cases functions without parameters then it still doesn't solve the function pointer problem.

Maybe reuse Self::update_forward_declarations? That led to a lot of bugs with the ir module, though ...

We could just forbid pointers to functions without parameters altogether, that seems like a pretty niche use case...

Even registering the parameters isn't allowed, see https://github.com/CraneStation/cranelift/issues/1088.
Maybe a better way is to only declare a function if it's called with the parameters seen.
https://stackoverflow.com/questions/434763/are-prototypes-required-for-all-functions-in-c89-c90-or-c99

That requires becoming standards compliant for integer literals for the common case to work, though.
Note how this says that existing signature has int f(long), while the actual prototype is int f(int) (local prototype, published code has different output):
int f();
int main() {
        return f(1);
}
int f(int i) { return i; }
[src/ir.rs:59] func_type = FunctionType {
    return_type: Int(
        true,
    ),
    params: [],
    varargs: false,
}
thread 'main' panicked at 'should not have an error declaring a function: IncompatibleSignature("f", Signature { params: [AbiParam { value_type: types::I64, purpose: Normal, extension: None, location: Unassigned }], returns: [AbiParam { value_type: types::I32, purpose: Normal, extension: None, location: Unassigned }], call_conv: SystemV }, Signature { params: [AbiParam { value_type: types::I32, purpose: Normal, extension: None, location: Unassigned }], returns: [AbiParam { value_type: types::I32, purpose: Normal, extension: None, location: Unassigned }], call_conv: SystemV })', src/libcore/result.rs:1084:5


CPVM/SSVM fails to start on KVM/Ubuntu 22.04 with 4.19.1 RC1 using local storage

Upon upgrading my env to 4.19.1 RC1 (from <IP_ADDRESS>), on destroying systemvms, one of them wouldn't come up (SSVM & CPVM, I had more CPVM failures) which fails due to the following seen on the agent logs:
2024-07-11 15:37:46,746 INFO  [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-4:null) (logid:fc1036c0) Trying to fetch storage pool 8aa9768c-cbcf-4e8e-8875-f94a7f9445b6 from libvirt
2024-07-11 15:37:46,747 WARN  [cloud.agent.Agent] (agentRequest-Handler-4:null) (logid:fc1036c0) Caught:
com.cloud.utils.exception.CloudRuntimeException: Could not fetch storage pool 8aa9768c-cbcf-4e8e-8875-f94a7f9445b6 from libvirt due to org.libvirt.LibvirtException: Storage pool not found: no storage pool with matching uuid '8aa9768c-cbcf-4e8e-8875-f94a7f9445b6'
        at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.getStoragePool(KVMStoragePoolManager.java:280)
        at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.getStoragePool(KVMStoragePoolManager.java:266)
        at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.disconnectPhysicalDisksViaVmSpec(KVMStoragePoolManager.java:241)
        at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtStartCommandWrapper.execute(LibvirtStartCommandWrapper.java:160)
        at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtStartCommandWrapper.execute(LibvirtStartCommandWrapper.java:49)
        at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtRequestWrapper.execute(LibvirtRequestWrapper.java:78)
        at com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(LibvirtComputingResource.java:1929)
        at com.cloud.agent.Agent.processRequest(Agent.java:683)

On the mgmt server side, this is seen:
2024-07-11 16:36:01,674 ERROR [c.c.v.VmWorkJobHandlerProxy] (Work-Job-Executor-57:ctx-0417fe01 job-2431/job-10081 ctx-0c64ca8a) (logid:81e6bbc0) Invocation exception, caused by: com.cloud.utils.exception.CloudRuntimeException: We cannot stop VM instance {"id":824,"instanceName":"v-824-VM","type":"ConsoleProxy","uuid":"506d0e73-b286-4de6-9df4-46b570eeacd5"} when it is in state Starting
2024-07-11 16:36:01,674 INFO  [c.c.v.VmWorkJobHandlerProxy] (Work-Job-Executor-57:ctx-0417fe01 job-2431/job-10081 ctx-0c64ca8a) (logid:81e6bbc0) Rethrow exception com.cloud.utils.exception.CloudRuntimeException: We cannot stop VM instance {"id":824,"instanceName":"v-824-VM","type":"ConsoleProxy","uuid":"506d0e73-b286-4de6-9df4-46b570eeacd5"} when it is in state Starting
2024-07-11 16:36:01,674 DEBUG [c.c.v.VmWorkJobDispatcher] (Work-Job-Executor-57:ctx-0417fe01 job-2431/job-10081) (logid:81e6bbc0) Done with run of VM work job: com.cloud.vm.VmWorkStop for VM 824, job origin: 2431
2024-07-11 16:36:01,674 ERROR [c.c.v.VmWorkJobDispatcher] (Work-Job-Executor-57:ctx-0417fe01 job-2431/job-10081) (logid:81e6bbc0) Unable to complete AsyncJobVO: {id:10081, userId: 1, accountId: 1, instanceType: null, instanceId: null, cmd: com.cloud.vm.VmWorkStop, cmdInfo: rO0ABXNyABdjb20uY2xvdWQudm0uVm1Xb3JrU3RvcALQ4GymiWjjAgABWgAHY2xlYW51cHhyABNjb20uY2xvdWQudm0uVm1Xb3Jrn5m2VvAlZ2sCAARKAAlhY2NvdW50SWRKAAZ1c2VySWRKAAR2bUlkTAALaGFuZGxlck5hbWV0ABJMamF2YS9sYW5nL1N0cmluZzt4cAAAAAAAAAABAAAAAAAAAAEAAAAAAAADOHQAGVZpcnR1YWxNYWNoaW5lTWFuYWdlckltcGwA, cmdVersion: 0, status: IN_PROGRESS, processStatus: 0, resultCode: 0, result: null, initMsid:<PHONE_NUMBER>30305, completeMsid: null, lastUpdated: null, lastPolled: null, created: Thu Jul 11 16:36:01 IST 2024, removed: null}, job origin:2431
com.cloud.utils.exception.CloudRuntimeException: We cannot stop VM instance {"id":824,"instanceName":"v-824-VM","type":"ConsoleProxy","uuid":"506d0e73-b286-4de6-9df4-46b570eeacd5"} when it is in state Starting
        at com.cloud.vm.VirtualMachineManagerImpl.advanceStop(VirtualMachineManagerImpl.java:2107)
        at com.cloud.vm.VirtualMachineManagerImpl.orchestrateStop(VirtualMachineManagerImpl.java:1980)
        at com.cloud.vm.VirtualMachineManagerImpl.orchestrateStop(VirtualMachineManagerImpl.java:5505)
        at jdk.internal.reflect.GeneratedMethodAccessor977.invoke(Unknown Source)

Restarting the agent a couple of times, somehow fixed it. I'm also not sure if it's something to do with my env or with 4.19.1 RC1
Update - in my env, I had a specific IP range (just two IPs) reserved for systemvms, I removed it and relax the systemvm Ip range strictness (reservation) to false; but still same issue.
ISSUE TYPE

Bug Report

COMPONENT NAME

KVM, SSVM, CPVM, Local storage

CLOUDSTACK VERSION

<IP_ADDRESS> RC1

CONFIGURATION

Adv zone + KVM Ubuntu 22.04
OS / ENVIRONMENT

KVM Ubuntu 22.04

@sureshanaparti assigned to you to triage, I don't know if it's a blocker or an issue in my env.
cc @shwstppr @vishesh92 @weizhouapache @DaanHoogland @harikrishna-patnala

Digging further, I found the specific pool it is not able to find is the local storage pool:
root@kvm1:/home/rohit# virsh pool-dumpxml 8aa9768c-cbcf-4e8e-8875-f94a7f9445b6
<pool type='dir'>
  <name>8aa9768c-cbcf-4e8e-8875-f94a7f9445b6</name>
  <uuid>8aa9768c-cbcf-4e8e-8875-f94a7f9445b6</uuid>
  <capacity unit='bytes'>982240026624</capacity>
  <allocation unit='bytes'>198888828928</allocation>
  <available unit='bytes'>783351197696</available>
  <source>
  </source>
  <target>
    <path>/var/lib/libvirt/images</path>
    <permissions>
      <mode>0711</mode>
      <owner>0</owner>
      <group>0</group>
    </permissions>
  </target>
</pool>

In my env, the local storage pool for the same host were removed/added once; it may be possible that somewhere/somehow it's picking an old pool that's removed cc @vishesh92 @sureshanaparti

Root cause - this returns true always for local storage - https://github.com/apache/cloudstack/blob/4.19/server/src/main/java/com/cloud/storage/StorageManagerImpl.java#L2795

Fixed in #9376


volatile example is incorrect

In Guide to Volatile Keyword, it is implied that SharedObject is racy if volatile keyword is not used, but that adding volatile fixes the race condition. In fact, the volatile keyword does not prevent any race conditions related to incrementing the counter.
Volatile in the example does prevent two threads from disagreeing about what the current count is, if they simply getCount() at the same time.
Volatile in the example does not prevent race conditions, where the value of count can be incorrect.
Removing volatile does not introduce any race conditions in the code. Only, two threads may disagree about the current count, which is not a race condition.
Volatile is not interchangeable with AtomicInteger here, as seems to be implied.
Moreover, volatile does not prevent any race conditions in code that may be calling incrementCount() or getCount().

Hey Ryanne,
you're right, volatile does not prevent any race conditions.
The article mentions that in multithread applications, two issues should be taken into considerations: mutual exclusion and visibility. Volatile deals with visibility, while synchronized - with mutual exclusion.
Namely, synchronized (and not volatile) is useful for resolving race conditions as you've correctly pointed out.


[v4] Backdrop opacity

There's already backdrop-color variable for customization. but opacity is hard-coded in the element.style.
It would be great to give a change to modify backdrop opacity with global variable.

Thanks for the issue! So the issue here is actually that the opacity is added to the backdrop in the animation (enter/leave) of the component. Which component are you looking to change the opacity on?

https://github.com/ionic-team/ionic/blob/master/core/src/components/popover/animations/md.enter.ts#L74

how can i change the backdrop opacity of an overlay sidemenu? please help

This problem existed for over 2 years now... https://github.com/ionic-team/ionic/issues/9105
Why can't we get a css variable for target opacity or something?

Same here... We've customers that need custom opacity (not interested in MD spec) and we're unable to give them this requirement. I was looking for a workaround withou success.
I think --ion-backdrop-color should define the opacity himself rgba(0, 0, 0, .32) granting user to override it easily.

Please see my comment here: https://github.com/ionic-team/ionic/issues/17669#issuecomment-470243422
At the moment it can't be done using a CSS variable, but the opacity is set by the animation and it is possible to pass a custom animation. I mentioned there that we need to look into a way to set it easier, but it is possible.
Customizing animations via the config
So where you see the following line you would change it from 0.32 to whatever number you'd like:
backdropAnimation.fromTo('opacity', 0.01, 0.32);

In your theme, workaround:
:root {
ion-backdrop {
opacity: 0.5 !important;
}
}

This is a real issue for us, as we cannot change the opacity of the backdrop which conflicts with the desired designs. When changing it via SCSS it will break the fade animation. Any news on this?

This was working for me for a long time and now all of a sudden, it does nothing. Even if I try to change the opacity in inspect element, literally nothing happens. I can do anything else like change colors in inspect element. Does someone know what is going on?
ion-modal ion-backdrop {     opacity: 0.6 !important; }

This was working for me for a long time and now all of a sudden, it does nothing.

I shouldn't have worked. Perhaps you prevented the fade animation somehow via some side effect.
As mentioned before, the css value actually isn't used/respected in the animation and the css rule even if working, broke it.
You can see this happen on your page, where the backdrop has the desired 0.9 opacity for a split second before the animation starts.
As you can see from a quick GH-repo search, the animations all use different magic inline values for the opacity.
This is frankly terrible, I don't understand it and I don't think there is an elegant way to change these values.

It's very disheartening to see simple issues like this be ignored or take forever to fix.

It's very disheartening to see simple issues like this be ignored or take forever to fix.

I have only started building with Ionic since V3. So that is disappointing to hear. They usually do fix bugs pretty fast. But I guess this isn't really a "bug" per-say. I don't know... Hope they do make this more robust for us. Seems like a simple CSS4 variable added to an active class would be so much easier.

In case anyone here is still looking for a solution:
When creating a new loader, action-sheet, modal... etc, you have an option to provide enterAnimation and leaveAnimation.
Those two properties take a callback function of the following signature:
((Animation: Animation, baseEl: any, opts?: any) => Promise<Animation>

I had some strange behaviour using that signature. In some cases I received Animation and baseEl correctly; then it gets called immediately after with only baseEl as the first param. See example below for a working callback function.
Animation: a class that you can create a new instance of and start building your own animation.
This class is exported from a directory called animation-old so I assume that it's deprecated. There is another method called createAnimation which returns an Animation object. It seems to work fine. Here is an example of that function standalone:
import { createAnimation } from '@ionic/core';

...

// have your element ready
const someElement: HTMLElement = document.querySelector('...'); // or can be retrieved with Angular using ElementRef.nativeElement

// call createAnimation to get an animation instance
const animation = createAnimation();

// add the element to the the animation instance, so anything we do applies to it
animation.addElement(someElement);

// set the starting behavior / style
animation.beforeStyles({
  opacity: 0,
});

// set the duration and easing of animation
// we can chain all function calls (including all of the above)
animation.duration(250).easing('ease-in');

// play the animation
animation.play();

Here's an example of how you would use it for a custom loading animation.
The baseEl argument in the callback function is your base element depending on where you're using this enterAnimation callback. Which could be HTMLIonLoadingElement, HTMLIonModalElement ... etc
import { LoadingController } from '@ionic/angular';

...

constructor(private loadingCtrl: LoadingController) {}

...

showLoader() {
  this.loadingCtrl.create({
    // your options here
      enterAnimation: ((baseEl: HTMLIonLoadingElement) => {

        // create a backdrop animation object to set behaviour for the backdrop only
        const backdropAnimation = createAnimation()
          .addElement(baseEl.querySelector('ion-backdrop'))
          .fromTo('opacity', 0, 0.9)
          .duration(250)
          .easing('ease-in');

        // create a content animation to animate the loading content separately (we want to give it full opacity in this case)
        const contentAnimation = createAnimation()
          .addElement(baseEl.querySelector('.loading-wrapper'))
          .fromTo('opacity', 0, 1)
          .duration(250)
          .easing('ease-in');

        // merge them into one root animation and return that
        // you can optionally apply animations to the root component
        return createAnimation()
          .addAnimation([
            backdropAnimation,
            contentAnimation,
          ]);
      }) as any, // wrap the whole callback in brackets and apply `as any` since the signature doesn't match the interface
  });
}

Keep in mind that if you provide call back for enterAnimation or leaveAnimation you will be overriding the default enter or leave animations respectively. The default values for z-index, opacity, or other properties might prevent you from seeing the component completely until you define values for them in your animation using from, to, fromTo, beforeStyles, or afterStyles.

@ihadeed Thank you for the working example! However, it's not very smooth loading on my end. So I just set showBackdrop to false and then just did some css3 animation to fade in a background on the ion-modal. Just doing my own backdrop instead of using ionic's

@ShaneJohnsonCC I tried it with an ion-loading only, it worked well and it's smooth.
I think for a modal you might need additional animations to control the modal itself to set the position/height... etc. Might be worth looking into the source code to find the default animation that they use, and extend that manually (copy, paste, and modify/add as needed).

@ihadeed Makes perfect sense! Appreciate your help man

+1 on this

Try this:
ion-popover {
  background: rgba(0, 0, 0, 0.7);
}

Thanks for the feedback everyone. While we would love to fix every issue in a timely manner, there are over 700 issues and those with workarounds are not going to receive our immediate attention as we have critical issues without workarounds. We always welcome PRs in order to push these issues to the top of the list. 🙂
I created a PR for this here that just needs review: https://github.com/ionic-team/ionic/pull/19533
I released a dev build if anyone would like to try it out: 4.11.0-dev.201910021700.08b22da
However this is based on our next branch which is going to end up being a major release with breaking changes, any breaking change has a deprecation warning in latest 4.x, see the commits on the branch for a full list: https://github.com/ionic-team/ionic/commits/next
Thanks!!


Changing Network Interfaces setting for specific MAC requires service restart

Reference internal bug 518266
Add a note to bullet on Network Interfaces to restart the WDS service when making this change to ensure the configuration is properly saved. (also reference doc bug 512129)
For WDS-less PXE responder, service name "SccmPxe", display name "ConfigMgr PXE Responder Service"

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

ID: b49b7692-2fba-a5b6-cc24-3b2621b7f5dd
Version Independent ID: bbc48abf-20cb-18f4-0db4-467cb0408b30
Content: Manage distribution points - Configuration Manager
Content Source: sccm/core/servers/deploy/configure/install-and-configure-distribution-points.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

I updated this with the 1806 content, which will publish soon. I'm closing this issue.


Add comment in auto generated files

add comment in auto generated files that they are auto generated, ie. should not be edited manually

This could be done by adding a $comment at the beginning of the document.
https://json-schema.org/understanding-json-schema/reference/generic.html#comments

it's only possible for SAP to implement this issue, as the code for JSON generation is proprietary

I have talked to the team, today. It agrees to add a $comment.

Suggestion:
"This file is autogenerated, do not edit manually, see https://github.com/SAP/abap-file-formats for more information"