E5108: Error executing lua ....local/share/nvim/lazy/mini.nvim/lua/mini/statusline.lua:658: attempt to call field 'is_enabled' (a nil value)

Describe the bug
Hi, I followed the WSL installation to install neovim and kickstart.nvim. Below is the error I have experienced.

To Reproduce
wsl --install
wsl
sudo add-apt-repository ppa:neovim-ppa/unstable -y
sudo apt update
sudo apt install make gcc ripgrep unzip git xclip neovim
git clone https://github.com/nvim-lua/kickstart.nvim.git "${XDG_CONFIG_HOME:-$HOME/.config}"/nvim
nvim
Desktop

OS: OS: Ubuntu 20.04.6 LTS on Windows 10 x86_64
Kernel: <IP_ADDRESS>-microsoft-standard-WSL2
Terminal: Windows Terminal

Neovim Version
NVIM v0.10.0-dev
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3

I am getting this as well, reproduced with the same issue (also on WSL)

I am getting this as well, reproduced with the same issue (also on WSL with the same nvim version). I worked past it for now by disabling statusline in init.lua.

comment out lines that contains stautsline in init.lua file just work fine.

comment out lines that contains stautsline in init.lua file just work fine.

Not using the code that produced issue will, of course, work. Same as not using severely outdated Neovim development version, which is arguably the better solution.

comment out lines that contains stautsline in init.lua file just work fine.

Not using the code that produced issue will, of course, work. Same as not using severely outdated Neovim development version, which is arguably the better solution.

Thx a lot. I followed your suggestion and worked out. It's a better suggestion, all the plugins working now. i just downloaded newer NEOVIM to fix it. 0.11

Is this still an issue? I can't repro on released 0.10 or 0.11

This is only an issue if using 0.10 development version not sufficiently late in its develoent cycle. This should not happen on released 0.10 and later.


License file patterns

Implemented by a different PR.

Implemented by a different PR.

Mind linking to that PR for reference?


util.lualine.pretty_path enhancement

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
Now, util.lualine.pretty_path only do some highlight when a buffer is modified. For someone who are not really sensitive to color it is not very good. Hope some other elements show when a buffer has been changed.
Describe the solution you'd like
like add [+] when a buffer is modified
Describe alternatives you've considered
none
Additional context
No response

On the topic, does any find the default truncation of path to no more than 3 directories a little bit excessive?
Though I don't think dynamic truncation is possible with lualine at the moment.

Not excessive for me, but that's purely personal preference from my part.
You could still just copy the pretty_path function and make it look like you want.
Or we could add a trunc_level variable and check if it's, for example, 0, then don't do any truncating. We could also do the truncating based on the trunc_level. For example, if trunc_level = 5, then we could just unpack the parts table based on the trunc_level.
But again that begs the question of how a user would like to see it. Would he want to see the first 2 parts of the path, then the ellipses character and the last 3 parts at the end. Or would it be a 1 and 4 split? Or a 3 and 2 split?
This complicates the logic unnecessarily I believe and should be better left to the user to implement a similar function in their personal configuration and use that function in their lualine component to make it look exactly how they like.


Update attack-simulation-training-simulation-automations.md

The below lines should be removed. It seems Exclude users page does not exist under automations tab. It's misleading.

[!TIP]
Although you can't remove users from the list on this page, you can use the next Exclude users page to exclude specific users.
-->

#sign-off


Go stack traces are not shipped

Besides the pattern is declared here, it seems like Logagent is not able to parse Go stack traces.

@rabbitstack Do you have a sample of Go stack traces that are not getting parsed that you could include here?

Something like this:
panic: runtime error: index out of range

goroutine 1 [running]:
github.com/sematext/foo/beef.(*upbuilder).InitUprobes(0xc4202d3680)
    /home/johndoe/Sematext/dev-st/foo/src/github.com/sematext/foo/beef/uprobe_builder.go:94 +0x18
main.main.func1(0xc42045e000, 0x19b6180, 0x0, 0x0)
    /home/johndoe/Sematext/dev-st/foo/src/github.com/sematext/foo/cmd/foo/main.go:196 +0xc77
github.com/sematext/foo/vendor/github.com/spf13/cobra.(*Command).execute(0xc42045e000, 0xc4200a8170, 0x0, 0x0, 0xc42045e000, 0xc4200a8170)
    /home/johndoe/Sematext/dev-st/foo/src/github.com/sematext/foo/vendor/github.com/spf13/cobra/command.go:766 +0x2c1
github.com/sematext/foo/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0xc42045e000, 0x1, 0x100027b, 0x1)
    /home/johndoe/Sematext/dev-st/foo/src/github.com/sematext/foo/vendor/github.com/spf13/cobra/command.go:852 +0x30a
github.com/sematext/foo/vendor/github.com/spf13/cobra.(*Command).Execute(0xc42045e000, 0xc42045a000, 0xc42045e000)
    /home/johndoe/Sematext/dev-st/foo/src/github.com/sematext/foo/vendor/github.com/spf13/cobra/command.go:800 +0x2b
main.main()
    /home/johndoe/Sematext/dev-st/foo/src/github.com/sematext/foo/cmd/foo/main.go:307 +0x51c


Preview

What feature or improvement do you think would benefit Files?
The file preview is (just) not large enough to read texts in a pdf. No function to set it.
Requirements
preview function
Files Version
<IP_ADDRESS>
Windows Version
10
Comments
In Win Explorer is that possible (comfortable).
A readable is crucial to use the app with text files.

You can resize the review pane by holding down on the divider line and dragging it.

that does not change the size of the previewed file

That's because of the details pane. The app needs to be made taller as well.

no, obviously not

If you don't need details you can right click and enable show preview only

details are switched off already, see picture. that doesn't change anything.

There seems to be a bug where it hides the details but doesn't expand the preview, will take a look at this.

Thank you

Can you open a bug report to track this?


Should TagsFilter provide shortcuts for all, any, none predicates

Currently the TagFitler is great for filtering a single tag based on custom matching rules. If the user wants to filter for example a way based on if there is any destination* tag on it the way to do this is:
// matches `destination*`
osmium::TagFilter destinationFilter;
osmium::TagMatcher destinationMatcher{osmium::StringMatcher::prefix{"destination"}};
destinationFilter.add_rule(true, destinationMatcher);
// check if we got any match
osmium::TagsFilter::iterator first{std::cref(destinationFilter), way.tags().begin(), way.tags().end()};
const osmium::TagsFilter::iterator last{std::cref(destinationFilter), way.tags().end(), way.tags().end()};
const auto hasDestination = std::distance(first, last) != 0;

https://github.com/osmcode/libosmium/blob/f53a1e37e2c99e19270d09f5c52c810afe4718db/test/t/tags/test_tags_filter.cpp#L56-L76
which seems a bit clumsy to me. Maybe I'm missing something in the interface here.
Should there be a shortcut e.g. for all, any, none predicates if all the user wants to do is checking if a filter triggered or not?

That was fast! ;-)
Sorry for the noise then, couldn't find these.


refactor(compatibility): refactor stripe compatibility routes using web::resource

Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[x] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

fixes #961

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

run cargo check on the entire project
format the code

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

changes done


CLI example in a single commit

Just a replacement for PR #5 in a single commit

Sorry if sidetracking.
	if runtime.GOARCH != "arm" {
		fmt.Println("This program can be unpredictable on other machines rather than Raspberry Pi")
		os.Exit(1)
	}

Is this check suggested for users of this library? I have been relying on rpio.Open() to return an error on unsupported systems. Is there a case where rpio could cause a side-effect on other Linux systems?

in BSD gpio mapped to a different place (as far as i understand to /dev/gpioc{N}, and also /dev/mem exist), so probably this could trigger some unexpected behavior


Feature request: JS callouts extentability

Some kind of JS event callout hooks to extend functionality, eg.
I am playing around with something like this, could also be tied into things like inventory control and picking etc.
https://youtu.be/oZNod_Mhfro

Can you expand on what you mean by JS callouts? Maybe an example of integration with outside system?

That yt video is an example, and the desc is pretty large.
Just events for current selected item etc.
I just hacked in an xhr for example to pass data to a host I control in this example...
function createRowHighlightHandler(rowid, refs) {
...
     URL = "http://<IP_ADDRESS>/?ledindex="+rowid.replace("bomrow","");
    let xhr = new XMLHttpRequest();
    xhr.open("GET", URL,true);
    xhr.send();

I see. Well these are hardly JS callouts, more like http callbacks but it's semantics.
I will think about good way to implement it. What events other than highlight do you think would be useful to have callbacks for?

sorry , I meant callouts in JS, not actual 'JS callouts' is that a thing?
functionally to callout to a users control function or trigger an event for customization or action.

I'm not an expert but probably in pure JS apps it's a thing, not in web since browsers sandbox everything.

ok well to be clear the xhr call has nothing to do with this.
Might be able to just get away with event listeners, but it depends on the scope of how to allow including user code assets. And what data we need to pass to the function etc

I agree, it's better to make it generic and not limited to http requests.
I think I'll add ability to inject user js code at bom generation time and that code will be able to add listeners to some events like highlight change, checkbox state change, etc. Then you can do whatever you want in your code.

Have you thought about this anymore?

Yes, I have a plan to implement the injectable user js code with listeners as described above and also injectable css and html headers/footers. Just have to find time to implement it.

This is now supported. In InteractiveHtmlBom/web/user-file-examples there are some examples you can copy over into InteractiveHtmlBom/web to check how they work and edit as you like.
I will add more info to the wiki soon.


CI/CD tweaks - Auto release using github actions

on:
  push:
    branches:
      - master

jobs:
  publish-npm:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-node@v4
        with:
          node-version: 20
          registry-url: https://registry.npmjs.org/
      - run: git config --global user.name "GitHub CD bot"
      - run: git config --global user.email<EMAIL_ADDRESS>      - name: Install deps
        run: npm i
      - name: Run tests
        run: npm run test
      - name: Build the app
        run: export NODE_OPTIONS=--openssl-legacy-provider && npm run build
      - run: npx semantic-release
        env:
          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      # push the version changes to GitHub
      - run: git add package.json package-lock.json && git commit -m'update version' && git push
        env:
          # The secret is passed automatically. Nothing to configure.
          github-token: ${{ secrets.GITHUB_TOKEN }}

Hi! Thanks for reporting this issue. Unfortunately, I don't have the capacity to automate it at the moment.


IE11: setDraftEditorSelection crashes with error 800a025e on removeAllRanges when getRangeAt(0).getClientRects().length === 0

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/stopachka/m6z0xn4r/.
When selection.removeAllRanges() is called in setDraftEditorSelection, IE throws an error JavaScript runtime error: Could not complete the operation due to error 800a025e if selection.rangeCount === 0 || selection.getRangeAt(0).getClientRects().length === 0. The root cause is discussed here: https://stackoverflow.com/questions/22914075/javascript-error-800a025e-using-range-selector/37580789#37580789
What is the expected behavior?
The selection's removeAllRanges() is successfully called.
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
0.10.5, IE11. No.
Additional comments
I believe the solution for this is to check for the above scenario and not call removeAllRanges(), which I will be happy to send out a PR for. I suppose it's a safe change given that either rangeCount === 0 or the first range has getClientRects().length === 0, but I'm happy to hear what others have to say.
function removeAllRangesSafe(selection: Object) {
  // IE throws Error 800a025e under the below circumstance
  if (document.body.createTextRange) {
    const willCrashInIE = selection.rangeCount === 0 || selection.getRangeAt(0).getClientRects().length === 0;
    if (willCrashInIE) {
      return;
    }
  }

  selection.removeAllRanges();
}

Alternatively, we can be extra safe by preventing swallowing all errors from IE?
function removeAllRangesSafe(selection: Object) {
  // IE throws Error 800a025e at times
  if (document.body.createTextRange) {
    try {
      selection.removeAllRanges();
    } catch (e) {
      // Swallow
    }
  } else {
      selection.removeAllRanges();
  }
}

My team is experiencing the same 800a025e error but I've tracked it down to using a component in blockRendererFn:
<Editor
  (... bunch of props omitted)
  blockRendererFn={customBlockRenderer}
 />

import ListItem from "components/ListItem";

export default function customBlockRenderer(contentBlock) {
  (... bunch of conditions omitted)
  return {
      component: ListItem
  }
}

Works fine if I remove component: ListItem but obviously doesn't render our component
AND
Its cropping up with our wrapper in a custom block renderer
const blockRenderMap = Immutable.Map({
  Table: {
    wrapper: <Table />,
  }
});
...
const extendedBlockRenderMap = DefaultDraftBlockRenderMap.merge(blockRenderMap);
...

<Editor
  (... bunch of props omitted)
  blockRenderMap={extendedBlockRenderMap}
/>

Am pretty stumped, and I am unsure if these interact with removeAllRanges?

huh. What version are you on @tbredin?

@mttwc I actually considered opening a PR with your solution. Only problem I ran into, was to "provide the steps to reproduce and if possible a minimal demo of the problem"..
I just couldn't find a practical solution for demoing and simply reproducing this issue. Mainly because jsfiddle, codepen and alike are not supporting IE11.
For now I forked the repo and published a npm package with the fix https://www.npmjs.com/package/pons-draft-js


Overlapping text on cards - tablet view

Loo k at the example below the issue around overlapping text.
A css style needs to be added to limit in tablet view the title where it may span 3 lines. Need to reduce to two then have an overlap.

Closing as duplicate.


Provide Error in Editor For Attempted C Style Casting

This issue was originally filed by @daftspaniel

What steps will reproduce the problem?
      int b = 123;
      int a = (int)(b/2);
What is the expected output? What do you see instead?
The 'int a' line should really be a error reported in the editor rather than at runtime. A C style programmer will type this instinctively or it will be hit when converting code.
Runtime error:
Breaking on exception: Class '_Type' has no instance method 'call'.
What version of the product are you using? On what operating system?
Dart r23200 Win7 64bit
Please provide any additional information below.

@kevmoo - think this can be closed as fixed. Repro Gives an error on Dartpad.


Enter world failed with Marium's Soulslike Weaponry mod

Mod Name
Marium's Soulslike Weaponry
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/mariums-soulslike-weaponry
Describe the bug
Game crashed when enter the world
Steps to reproduce
No response
Logs
World Generation: Stable 	Type: Integrated Server (map_client.txt) 	Is Modded: Definitely; Client brand changed to 'forge'; Server brand changed to 'forge' 	Launched Version: 1.20.1 	Sinytra Connector: 1.0.0-beta.25+1.20.1 		SINYTRA CONNECTOR IS PRESENT! 		Please verify issues are not caused by Connector before reporting them to mod authors. If you're unsure, file a report on Connector's issue tracker. 		Connector's issue tracker can be found at https://github.com/Sinytra/Connector/issues. 		 Game crashed! Crash report saved to: #@!@#  Using SimpleLogger

Additional context
I'm using forge 47.2.6
Mod list here, without the two mods I crossed out in red.

Full log documents here
minecraft-exported-crash-info-2023-11-22T22-29-28.zip

Is 47.2+ supported with Connector now? I thought it was just forgified fabric api

You sure?? Marium’s Soulslike Weaponery works for me on Forgified Fabric API on 47.2.6 and Connector Beta 25

You sure?? Marium’s Soulslike Weaponery works for me on Forgified Fabric API on 47.2.6 and Connector Beta 25

Did u tried to creat and enter a world? My game crashed at that moment

You sure?? Marium’s Soulslike Weaponery works for me on Forgified Fabric API on 47.2.6 and Connector Beta 25

Would u mind to tell me ur java's version?

downgrade soulslike weaponry below ver 1.0.0.

You sure?? Marium’s Soulslike Weaponery works for me on Forgified Fabric API on 47.2.6 and Connector Beta 25

Would u mind to tell me ur java's version?

My Java Version was 1.20.1 Forge 47.2.6

You sure?? Marium’s Soulslike Weaponery works for me on Forgified Fabric API on 47.2.6 and Connector Beta 25

Did u tried to creat and enter a world? My game crashed at that moment

Yes and It didn't crash for me

Geckolib Fabric works with Marium’s Soulslike Weaponery and yes I need the Forge version of Geckolib to run Iron Spells and spellbooks because Geckolib Forge and Fabric are not compatible together. Marium’s Soulslike Weaponery new update crashes the game.

I've just released the version 1.0.2 that fixes the mixin confliction, so it will now work with connector.

I've just released the version 1.0.2 that fixes the mixin confliction, so it will now work with connector.

Godness!  thanks u soooooo much sir!

Marium’sSoulslikeWeaponery 1.0.2 crashes on Connector beta 29

Worked fine for me when using the recommended forge version and geckolib fabric... did it say anything specific in the crash log?

I confirm to my great misfortune, the mod is incompatible :(

What do you mean. It works for me on Connector Beta 36.

Am using Marium’s Soulslike Weaponery 1.02

Yes so the latest version 1.20.1...It's weird for me, it's impossible to make it work...:(

Have you installed a mod in addition to connector and Marium’s Soulslike Weaponery?

Connector beta 36
Forgified Fabric API: 0.91.0+1.10.8+1.20.1
Forge 4.7.13
Geckolib-fabric-1.20.1-4.4.2

And do you use other forge mods with this?

Nope

Maybe that's the difference, our server has around 300 mods :/

Am running the mods on client that's why

Confirmed to work on Forge 47.1.3 using the following mods:

Connector-1.0.0-beta.36+1.20.1
fabric-api-0.92.0+1.11.1+1.20.1
geckolib-fabric-1.20.1-4.4.2
soulslike-weaponry-1.0.2-1.20-fabric


Generate and download logical enclosure dump

Scenario/Intent
How can I get via ansible the generated logical enclosure dump file from oneview_logical_enclosure module?  I tried to do it with the standard get_url ansible module without success.
Steps to Reproduce
    - name: Generates a support dump for the Logical Enclosure
      oneview_logical_enclosure:
        state: dumped
        data:
            name: "{{ logical_enclosure_name }}"
            dump:
              errorCode: "MyDump16"
              encrypt: "true"
              excludeApplianceDump: "false"

    - name: Download logical enclosure dump
      get_url:
        url: "https://{{ lookup('env','ONEVIEWSDK_IP') }}{{ generated_dump_uri }}"
        dest: "{{ generated_dump_uri | basename }}"
        url_username: "XXX"
        url_password: "XXX"
        validate_certs: no
        force_basic_auth: yes

Expected Result
Download dump file.
Actual Result
fatal: [localhost]: FAILED! => {"changed": false, "dest": "oneview.home-MyDump16-L-2018_03_01-15_34_49.623983.sdmp", "msg": "Request failed", "response": "HTTP Error 401: Unauthorized", "state": "absent", "status_code": 401, "url": "https://<IP_ADDRESS>/rest/appliance/support-dumps/oneview.home-MyDump16-L-2018_03_01-15_34_49.623983.sdmp"}

@joseba - Sorry for the delayed response. You can try with the below example:

name: Download logical enclosure dump
get_url:
url: "https://{{ lookup('env','ONEVIEWSDK_IP') }}/{{ generated_dump_uri }}"
dest: "{{ playbook_dir }}/{{ generated_dump_uri | basename }}"
headers: 'auth:,X-Api-Version:'
validate_certs: no
use_proxy: no
force_basic_auth: yes

where  is the SessionID.

Closing this since there was no query further.


The following command produces an error

Describe the bug
A clear and concise description of what the bug is.
Internally producing an invalid argument trying to load config.yaml for the model .
To Reproduce
Steps to reproduce the behavior:

What command or script did you run?

from layoutparser.models import Detectron2LayoutModel

model = Detectron2LayoutModel('lp://PubLayNet/mask_rcnn_X_101_32x8d_FPN_3x/config',
                              extra_config=["MODEL.ROI_HEADS.SCORE_THRESH_TEST", 0.5],
                              label_map={0: "Text", 1: "Title", 2: "List", 3: "Table", 4: "Figure"})

Environment

Please describe your Platform [Windows/MacOS/Linux] - windows 10
Please show the Layout Parser version -
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error traceback here.
Traceback (most recent call last):
File "C:\layoutparserdirty\layout_processor.py", line 3, in <module>
   model = Detectron2LayoutModel('lp://PubLayNet/mask_rcnn_X_101_32x8d_FPN_3x/config',
 File "C:\layoutparserdirty\venv\lib\site-packages\layoutparser\models\detectron2\layoutmodel.py", line 91, in __init__
   config_path = PathManager.get_local_path(config_path)
 File "C:\layoutparserdirty\venv\lib\site-packages\iopath\common\file_io.py", line 1197, in get_local_path
   bret = handler._get_local_path(path, force=force, **kwargs)
 File "C:\layoutparserdirty\venv\lib\site-packages\layoutparser\models\detectron2\catalog.py", line 136, in _get_local_path
   return PathManager.get_local_path(model_url, **kwargs)
 File "C:\layoutparserdirty\venv\lib\site-packages\iopath\common\file_io.py", line 1197, in get_local_path
   bret = handler._get_local_path(path, force=force, **kwargs)
 File "C:\layoutparserdirty\venv\lib\site-packages\iopath\common\file_io.py", line 794, in _get_local_path
   with file_lock(cached):
 File "C:\layoutparserdirty\venv\lib\site-packages\portalocker\utils.py", line 157, in __enter__
   return self.acquire()
 File "C:\layoutparserdirty\venv\lib\site-packages\portalocker\utils.py", line 239, in acquire
   fh = self._get_fh()
 File "C:\layoutparserdirty\venv\lib\site-packages\portalocker\utils.py", line 289, in _get_fh
   return open(self.filename, self.mode, **self.file_open_kwargs)
OSError: [Errno 22] Invalid argument: 'C:\\Users\\nfeliccia/.torch/iopath_cache\\s/nau5ut6zgthunil\\config.yaml?dl=1.lock'

Process finished with exit code 1
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

OSError: [Errno 22] Invalid argument: 'C:\Users\arvinder.sharma/.torch/iopath_cache\s/nau5ut6zgthunil\config.yaml?dl=1.lock'
I am getting same error. Running this on windows 10. its working in google colab but not in Jupter notebook or when run through flask.
Let me know how to resolve it.


Shoult not suggest incompatible update of dependency

Version(s)
0.2.0
Describe the bug
Scala CLI suggests updating a dependency even if it would create a binary incompatibility.
To Reproduce
//> using lib "com.softwaremill.sttp.client3::upickle::3.8.13"
//> using lib "com.lihaoyi::upickle::2.0.0"

$ scala-cli compile bug.scala
Downloading 3 dependencies
Compiling project (Scala 3.2.2, JVM)
Compiled project (Scala 3.2.2, JVM)
[hint] ./bug.scala:2:16
[hint] "upickle is outdated, update to 3.0.0"
[hint]      upickle 2.0.0 -> com.lihaoyi::upickle::3.0.0
[hint] //> using lib "com.lihaoyi::upickle::2.0.0"
[hint]                ^^^^^^^^^^^^^^^^^^^^^^^^^^^

Expected behaviour
Scala CLI should not suggest to update upickle to 3.0.0, without suggesting to update sttp to a upickle-3.0.0-binary-compatible version.

A related bug or improvement is https://github.com/VirtusLab/scala-cli/issues/1927


Will authorization be retained?

I use this link to authorize.
https://login.eveonline.com/oauth/token
Many characters have been authorized, and is there any way to convert them directly to the authorization generated by such links?
https://login.eveonline.com/v2/oauth/token
Or in future plans, all characters will be reauthorized again?

Take a look at https://github.com/ccpgames/eveonline-third-party-documentation/blob/fca100f5909acbc18ad364cbb08ea49784ac28ab/docs/sso/jwt-refreshtokens.md#upgrading-from-the-old-tokens-to-jwt
This information will also need to be included somewhere in the new docs.

So when was it going to be discarded, and how long would I be able to use it, after all, it would require some code modification

@fwfmiao the plan is to phase out the v1 oauth routes sometime in 2019, but expect an actual announcement with some time to move over before that is official. But the v2 routes are backwards compatible so you should be able to simply switch your oauth route to v2 and it will continue on as it always has.
@CarbonAlabel you are definitely correct that this info needs to be in the docs.


Fixes #53 Merge c-input with attr props

Can you include a snippet of the compiled output for us to review? It looks like you're using extend (or the xt helper for short) which seems like a good approach.

Ok, I'll add better code comments, but the following is the generated code:
__tag(out,
      simple_hello_tag,
      __extend(__extend({}, {name: 'Jane', adult: true}), {
        "name": "Frank"
      }));

There are two extends because we don't want to modify the original object. The inner extend creates a new object with the properties of the c-input expression merged in. The outer extend extends the inner object with the properties from the attributes.

Looking good


Implement automatic inference of qid shape for quantum states

Follow up to #3419 . Addresses #2904 .

@mpharrigan I've addressed your comments, please have another look.

A couple of comments:

There is also cirq.to_valid_state_vector that should make use of this functionality - states._state_like_to_state_tensor seems to do a bunch of qid shape inference itself...

After playing around with the code, it felt a bit fragile and hard to understand the rules just by discovery...it might be that I'm not smart enough for it or I'm misusing the tool :)

I wonder if we want to adopt a "use minimal dimensions where possible" - it would allow for infer_qid_shape(1) to be (2,1)
I wonder if we want to adopt automated casting - e.g. infer_qid_shape([1,0]) fails now but  infer_qid_shape([1,1j]) doesn't...what do we think about automated casting to complex?

I wonder if we want to adopt automated casting - e.g. infer_qid_shape([1,0]) fails now but infer_qid_shape([1,1j]) doesn't...what do we think about automated casting to complex?

I think this is by design to allow integer specification of basis states vs complex amplitudes

I wonder if we want to adopt automated casting - e.g. infer_qid_shape([1,0]) fails now but infer_qid_shape([1,1j]) doesn't...what do we think about automated casting to complex?

I think this is by design to allow integer specification of basis states vs complex amplitudes

I'm confused - list of ints wouldn't be integer specification of basis states as far as I understand, infer_qid_shape([1,0]) is the |0> state, it's just not complex...

I wonder if we want to adopt automated casting - e.g. infer_qid_shape([1,0]) fails now but infer_qid_shape([1,1j]) doesn't...what do we think about automated casting to complex?

I think this is by design to allow integer specification of basis states vs complex amplitudes

I'm confused - list of ints wouldn't be integer specification of basis states as far as I understand, infer_qid_shape([1,0]) is the |0> state, it's just not complex...

Oh, I got it - it could be looked at as a multiqubit integer state |10>...ahhh...

A couple of comments:
* There is also `cirq.to_valid_state_vector` that should make use of this functionality - `states._state_like_to_state_tensor` seems to do a bunch of qid shape inference itself...

After playing around with the code, it felt a bit fragile and hard to understand the rules just by discovery...it might be that I'm not smart enough for it or I'm misusing the tool :)
* I wonder if we want to adopt a "use minimal dimensions where possible" - it would allow for `infer_qid_shape(1)` to be `(2,1)` --- python world precedence: `bin(3) -> 0b11'

* I wonder if we want to adopt automated casting - e.g. `infer_qid_shape([1,0])` fails now but  `infer_qid_shape([1,1j])` doesn't...what do we think about automated casting to complex?

I'll edit states._state_like_to_state_tensor to use this, and I'll also implement your suggestion to use minimum dimensions for integer states.

@balopat I've addressed your comments, please have another look.

@balopat did Kevin address your comments?

Automerge cancelled: Only collaborators with write permission can use automerge.

Automerge cancelled: A required status check is not present.
Missing statuses: ['cla/google']

@kevinsung can you sign the CLA?

@mpharrigan I signed one years ago; do you have any idea why the check would stop going through all of a sudden?

All is good on the CLA front, I think either Cirqbot might have a bug here or there was a CLA bot flake? I suspect the former.


Document does not explain several parameters about $watch

What problem does this feature solve?
https://vuejs.org/v2/api/#vm-watch
About $watch, the document just mention the options about deep and immediate.
But after review the source code, there are four options about $watch :

deep
user
lazy
sync

  // options
  if (options) {
    this.deep = !!options.deep;
    this.user = !!options.user;
    this.lazy = !!options.lazy;
    this.sync = !!options.sync;
  } else {
    this.deep = this.user = this.lazy = this.sync = false;
  }

What does the proposed API look like?
Document must talk more about sync and user

Some options are intentionally omitted because they are not part of the public API.


chore(release): Release : 2021 JUN 01

ATTENTION : MAJOR UPDATE
There's a new release of JSCAD, including MAJOR CHANGES from several pull requests.
The maths classes / APIs have been adjusted to the same API as gl-matrix. This has the following affects:

improved performance as API signatures are now fixed (no more ...params)
improved performance in the use of math objects (less allocations)
improved interoperability with gl-matrix (no need to learn a different API)

Note: See the attached performance results.
If your designs use the maths API then updates are required.
The function signitures for ALL maths objects have changed; line2, line3, mat4, plane, vec2, vec3, vec4.
Especially review the use of maths..transform() and maths..scale() function calls.
The designs with the largest impact are those using extrudeFromSlices(), which typically uses maths.
Features
modeling: rework math objects to conform with gl-matrix (#804)
modeling: added aboutEqualNormals() to math utils
modeling: added fromVectorRotation() to mat4
modeling: added projectionOfPoint() to plane
modeling: new project() function added to extrusions (#844)
modeling: new scission() function added to booleans (#846)
Bug Fixes
modeling: fixed use of vec3.snap in triangulatePolygons
modeling: rewrote vec3 orthogonal() to do the correct thing, and adjusted test suites
modeling: update and correct TypeScript type definitions
Special thanks to @minodisk for the TypeScript changes!
Successfully published:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
All Submissions:

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Does your submission pass tests?

JSCAD Performance20210519.pdf


Patch 1

added the two files.

pushing to master not allowed, will push them to the correct branch


Drop fails with empty lists and to drop at bottom of the list.

Mikhael reported me that
"looks like once the list is empty in the demos, you cannot drag the items back to it.
I thought it's because  's size becomes 0, but even after I forced it to some min val, I still can't drag items to an empty list.
Is it a bug or is there a setting I need to enable?"
It is a bug introduced by commit https://github.com/a5hik/ng-sortable/commit/3cfc2f005395fcbe356fa9e7ad67d85d1a598b07 .

Thanks for the quick fix!


Database Population - Matrix Systems

Purpose
Currently we have been defining our schema within a respective database through the /create-db API. As we prepare to migrate our product into an AWS ECS container, we must ensure that this functionality is encompassed within the startup of Matrix Systems.
Expected Behavior
Locally upon starting Matrix Systems utilizing docker-compose or run.py Matrix Systems should be able to create all necessary tables within our remote database as well as populated any default data deemed necessary (Roles).

The user agent is able to successfully post data to our route via the api


Listeners are not supported?

Am I just looking wrong, or Listeners are not supported?

Aha, its hidden in HTTP and port variables, sry...


Release 1.1.0

Test evidence : https://gist.github.com/bbaugher/de1094f5903907ac2e80
Issues : https://github.com/cerner/cerner_kafka/issues?q=milestone%3A1.1.0+is%3Aclosed

+1

+1

Project has been released.


Native property access doesn’t work in production

When running m3 in a production build using native properties doesn’t work, as megamorphicModelProxyHandler isn’t defined, and the proxy constructor throws an error. It’s only defined inside an if (DEBUG) statement, making it work fine in development builds.
Would it be sufficient to move the declaration outside of that if statement, or is there a reason it’s inside it?
Code ref: https://github.com/hjdivad/ember-m3/blob/master/addon/model.js#L115

@mogstad thanks very much for reporting this; we should have this fixed soon (probably today); see #1376


Loaded Images Have Zero Height and Width

I'm trying to use this script to determine when images have finished loading so that I can draw the images to an html5 canvas.
In the 'always' callback, the image usually (not always) has a width and height of zero, which makes it impossible to set the canvas to the proper size or to draw the image with the correct parameters.
Here is the code (coffeescript):
https://gist.github.com/564ccd4410e1fccc8cb9

Hmm, I seem to have this problem as well. It's not consistently reproducible, so it's hard to know if a solution actually fixes the problem. I'll let you know if I find something.

I have debugged it and I can reproduce it consistently. The problem is at imagesloaded.js line 240:
var resource = cache[ this.img.src ] || new Resource( this.img.src );
If a resource is found in the cache but the browser decides to redownload the image anyway, imagesloaded will say these images are loaded, but actually they are not. Removing the cache fixes the problem for me:
var resource = new Resource( this.img.src );

There is a no-cache branch. Using this branch fixed the problem for me.

Unfortunately the no-cache branch solution didn't work for me. In my particular case, the 1st time the image loads, the width/height of the image is reported as zero (0) after the always event handler occurs. If I reload the page, the image loads just fine.

I've found that in my case the problem can be avoided by reusing Image instances that have the same uri. When trying to track a fresh Image instance that uses a src uri that have been previously tracked, the problem happens pretty consistently on the 2nd image load when running IE 10 in debug mode from the Visual Studio debugger.

imagesLoaded v3.2.0 has removed the internal cache. Try it out and report back if you feel this issue has been resolved.

sorry for bringing this up but the problem still exists with the latest version.
random behavier, tested on apache cordova.


Typo error

Just wanna say that there is a typo in almost all your CSS variables backgroud instead of background.  😞

Thank you for issue, I'll fix it in the next version


druid post aggregation KeyError for arithmetic division type

added a customed post aggregation in superset, but got KeyError.
Environment
(please complete the following information):

superset version: 0.28.1/0.29.0rc6 , install by pip
python version: 3.6.8

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

callstack:
2019-05-15 10:40:59,534:ERROR:root:'totalActionTimeMs'
Traceback (most recent call last):
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/viz.py", line 406, in get_df_payload
df = self.get_df(query_obj)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/viz.py", line 211, in get_df
self.results = self.datasource.query(query_obj)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 1342, in query
client=client, query_obj=query_obj, phase=2)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 940, in get_query_str
return self.run_query(client=client, phase=phase, **query_obj)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 1141, in run_query
metrics_dict)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 903, in metrics_and_post_aggs
postagg, post_aggs, saved_agg_names, visited_postaggs, metrics_dict)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 869, in resolve_postagg
required_fields, metrics_dict)
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 830, in find_postaggs_for
metrics_dict[name] for name in postagg_names
File "/home/work/superset-venv/lib/python3.6/site-packages/superset/connectors/druid/models.py", line 831, in
if metrics_dict[name].metric_type == POST_AGG_TYPE
KeyError: 'totalActionTimeMs'
superset customed metric
type: postagg
json:
{
"type": "arithmetic",
"name": "avgActionTimeInMs",
"fn": "/",
"fields": [
{
"type": "fieldAccess",
"name": "totalActionTimeMs",
"fieldName": "totalActionTimeMs"
},
{
"type": "fieldAccess",
"name": "countRows",
"fieldName": "countRows"
}
]
}
DRUID metic spec
    "metricsSpec": [
        {
            "type": "count",
            "name": "countRows"
        },
        {
            "type" : "doubleSum",
            "name" : "totalActionTimeMs",
            "fieldName" : "actionTimeInMs"
        }
    ],

@david-z-johnson I'd suggest to test 0.32 or later to get some attention, there are a ton of fixes between your version and the most recent ones..

@elukey , I have tried on latest version: 0.32.0rc2,  got same result as mentioned before.
Below is the error callstack:
2019-05-16 20:56:54,345:ERROR:root:'countRows'
Traceback (most recent call last):
File "/home/monitor/incubator-superset/superset/viz.py", line 410, in get_df_payload
df = self.get_df(query_obj)
File "/home/monitor/incubator-superset/superset/viz.py", line 213, in get_df
self.results = self.datasource.query(query_obj)
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 1286, in query
client=client, query_obj=query_obj, phase=2)
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 883, in get_query_str
return self.run_query(client=client, phase=phase, **query_obj)
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 1084, in run_query
metrics_dict)
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 846, in metrics_and_post_aggs
postagg, post_aggs, saved_agg_names, visited_postaggs, metrics_dict)
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 812, in resolve_postagg
required_fields, metrics_dict)
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 773, in find_postaggs_for
metrics_dict[name] for name in postagg_names
File "/home/monitor/incubator-superset/superset/connectors/druid/models.py", line 774, in
if metrics_dict[name].metric_type == POST_AGG_TYPE
KeyError: 'countRows'

The might be something wrong with the Druid post aggregation, I haven't checked closely. Can you confirm that it works as expected querying druid?

@elukey , I checked with below DRUID query, it works for DRUID http query json below:
{
"queryType": "groupBy",
"dataSource": "xxxxxxxxxx",
"granularity": "day",
"dimensions": [
"actionName"
],
"aggregations": [
{
"type": "count",
"name": "countRows"
},
{
"type": "doubleSum",
"name": "totalActionTimeMs",
"fieldName": "actionTimeInMs"
}
],
"postAggregations": [
{
"type": "arithmetic",
"name": "avgActionTimeInMs",
"fn": "/",
"fields": [
{
"type": "fieldAccess",
"name": "totalActionTimeMs",
"fieldName": "totalActionTimeMs"
},
{
"type": "fieldAccess",
"name": "countRows",
"fieldName": "countRows"
}
]
}
],
"intervals": [
"2019-05-01T00:00:00.000Z/2019-05-16T00:00:00.000Z"
]
}

Follow up question: when you check the datasource via Sources -> Druid datasources -> your_datasource do you see the countRows metric, or are you defining it in another way? (trying to understand since I don't have a ton of experience with Druid)
My goal is to reproduce the issue, but so far I was able only when adding name of metrics that were not listed in the datasource's details.

@elukey , the fields countRows and totalActionTimeMs are both can be seen in Superset DRUID data source.
The DRUID metric spec:
    "metricsSpec": [
        {
            "type": "count",
            "name": "countRows"
        },
        {
            "type" : "doubleSum",
            "name" : "totalActionTimeMs",
            "fieldName" : "actionTimeInMs"
        }
    ]

I found my the cause, it is my fault.
In superset customed metric, for fieldName, I should use superset's filed name instead of DRUID metric filed name.

@david-z-johnson What do you meant by superset's filed name?


Fix/pip

:tada: This PR is included in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Basic version of this library

I wanted to use it in some of my user scripts, and I was going to propose this library to @y252328, to use it in his user script Instagram_Download_Button as well.
Looking at the code now, it is too large and far more than the code of our user scripts.
Maybe someday you will consider creating a super small version of this library, where it is specific to one website and with few basic options ,to make the code smaller. I would do it myself, but I'm not sure I fully grasp how to use this library, let alone understand the code to create a smaller version. 😄

That is complicated for me to understand at the moment. So I can't give an opinion. 😄


Processing Network: Add Automatic Zoom Adjustment After Adding New Command

The processing network doesn't zoom out after adding a command. So you have to either zoom out yourself or pull the screen so that you can click on the next artifact. This can be burdensome when running long workflows. It would be nice if, after you add a command, the processing network automatically adjusts the zoom so you can easily click on the new artifact to add your next command.

New requests:

[ ] Add support for certain jobs to be executed locally in the head node, even on systems submitting jobs on Torque. This would involve an environment variable that could be set, and metadata added to a table to direct which jobs should be run where. Original


Delete execution directory on flow finishing

This PR moves execution directory cleanup out from CleanerThread, and deletes execution directory on flow finishing. The motivation is to make sure that execution directory gets deleted on time without leaving any orphan disk-consuming dirs. Previously CleanerThread periodically traverses every execution dir periodically and deletes any directories which haven't been modified for some configurable period. The logic is cumbersome and error-prone as it requires checking against shared data structure like running flows to make sure it's not deleting execution directories which are in use.

@li-ygerchikov thanks, see my answer inline
Would we ever want to keep execution dir, e.g. for debugging purposes?

yes, we kept it for debugging purpose but we never actually look at the execution dir for that purpose from my experience.

Have we captured all the logs and everything that's worth saving from it before we delete it after flow finishes?

yes, log will be persisted to mysql before the deletion

Are there any situations where handleEvent is not called for a given flow -- e.g. if executor dies or is killed, or if execution terminates abnormally?

i have tested the case where execution is killed. When executor is shutdown, there's code to handle deletion all execution dirs.

The main purpose of this code as PR suggests is to simplify the logic and make sure all execution directories get deleted on time to avoid disk penalty.


sort min_imports alphabetically

Since there is a call to qsort it seems like sorting is intended here. However, currently qsort is called on Name, but apparently this does not do the correct thing, since when using this in mathlib, I observed that the output was not sorted. We fix this by converting to string earlier and using the (· < ·) function to compare the strings.

friendly pinging @semorrison, because I'm not sure my pull request was seen...

I think the underlying issue is that the sorting on Name
#reduce Name.cmp `M.A `M.B -- (✓) .lt
#reduce Name.cmp `M.A.B `M.B -- (⨯) .gt

Could we first have a Lean Issue about this? Or confirmation that this is working as intended?
This happens because Name.cmp does the following:

M.A.B vs. M.B
M.A vs. M
A vs anonymous --> greater than --> entire result is "gt"

If leanprover/lean4#4944 is not addressed or found invalid, then your fix here is the correct thing to do. Let's give them some time to look at the issue.

SGTM!
We could also just fix it now with a remark to revert the fix if the behavior of Name.cmp changes, I suppose...

https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/Does.20Name.2Ecmp.20do.20the.20correct.20thing.3F

@mo271 what do you think of #24, does that look a bit cleaner?


ocamlPackages.menhir: 20181026 -> 20181113

Motivation for this change
New rule syntax.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Is the old still valid?

Quoting from the change log:

The old syntax remains available; the user chooses between the two syntaxes on a per-rule basis.

Sounds good. Then the upgrade should be not a big deal.


can we turn back on Sched or somewhere else continue to host schedule, speakers, etc.?

Need to point funders to it in connection with funding reports post-Summit.

Whoops, for some reason a refresh brought it back to live for me. Never mind. Thx.


Refactoring participantsDefinition

Merge with Refactoring gameDefinition
https://github.com/dexterneo/liveBBStats/issues/61


C4702 Unreachable Code Warnings in GRPC Generated Header Files Related to abort()

What version of gRPC and what language are you using?
Version: 1.10.0
Language: c++
What operating system (Linux, Windows,...) and version?
OS: Windows 10
What runtime / compiler are you using (e.g. python version or version of gcc)
Compiler: Visual Studio 2017
What did you do?
I was checking compiler warnings of the cpp files generated using grpc.
What did you expect to see?
I expected to see no compiler warnings from cpp files generated using grpc.
What did you see instead?
VS 2017 gave C4702 Unreachable Code warnings in lots of .h header files generated by grpc. Because there is return statement after abort() in the generated grpc code. I assume the issue is from following example code in grpc/src/compiler/cpp_generator.cc file.
printer->Print(
vars,
"// disable synchronous version of this method\n"
"::grpc::Status $Method$("
"::grpc::ServerContext context, const $Request$* request, "
"$Response$* response) override {\n"
"  abort();\n"
"  return ::grpc::Status(::grpc::StatusCode::UNIMPLEMENTED, "");\n"
"}\n");
In code like this, the return statement is unreachable in the eyes of VS 2017 and hence triggers C4702 warnings.
Anything else we should know about your project / environment?

This issue is important because the warnings come from header files which cannot be easily disabled in a small scope (header files can be included from anywhere in our code). And we don't want to suppress the C4702 Unreachable Code warning globally in our project because the warning is efficacious in finding bugs.

Hi, just want to know is the problem going to be fixed and in which release plan? Thanks!

This isn't really on our radar, as "fixing" this would cause the OTHER warning that we're not returning anything in a function that has a return type. Trying to clean up generated code is a sisyphean task. The best I could suggest as a quick workaround in the meantime would be to do something along the lines:
#pragma warning(push)
#pragma warning(disable: 4702)
#include "some_header.grpc.pb.h"
#pragma warning(pop)

Okay. We suppressed the warning from the header files. Thanks for the work around.


How to change animstate of one particular entity?

Working on a resolver that works by reading animstate and I was wondering how I would go about reading each individual entities animstate and then resolving them individually.
Currently I'm pretty sure I'm just reading info from the entity with the highest index and then applying my resolving method from that info to all of the other entities.
Thanks in advance

`for (int i = 1; i < 65; i++) {
Entity* entity = interfaces->entityList->getEntity();
if (!entity)
continue;
AnimState* animState = entity->getAnimState();

}`

`for (int i = 1; i < 65; i++) {
Entity* entity = interfaces->entityList->getEntity();
if (!entity)
continue;
AnimState* animState = entity->getAnimState();

}`

you can create a func to resolve each entity individually, and then loop through them

you can create a func to resolve each entity individually, and then loop through them


Bug: MQTT disconnects before sleep, grace period unreliable

The program works perfectly, but whenever I resume my computer from sleep, Hass agent reports 'MQTT failed to connect' which is kind of annoying to dismiss the notification event though it reconnects just fine (after resuming).
To Reproduce
Steps to reproduce the behavior:

Let your windows session go to sleep
Resume
Message should pop up

Expected behavior
Hass agent should detect when the computer goes to sleep so as to not notify that it can't connect to MQTT.

Windows 11
HASS.Agent version: 2022.12.0-beta4

Have you tried setting this to a higher value:

So after testing it yesterday, I set the option you highlited to ~16-17 minutes and the sleep time of my pc to 15 minutes. What I found today is that the message occurs anyway and I had the feeling that it was before going to sleep, but it happens the instant that the pc resumes.
Any ideas?

Interesting, thanks for checking that out! Hadn't occured to me that the disconnect can happen before going to sleep, that can mess with the counter.
I'll improve it and let you know

Awesome, let me know if you need logs or anything else.

I've merged all tickets related to this issue into a central pinned one: [#204]
Closing this one to keep it in one place.


SMEP

Should we require, Supervisor Mode Execution Protection (SMEP), or something like it. The OS kernel shall only read data from the address space of the kernel. App note, the kernel share not read data from user mode.
Looks like Linux has support
http://vulnfactory.org/blog/2011/06/05/smep-what-is-it-and-how-to-beat-it-on-linux/
Windows also supports SMEP.
not sure about other OS'es

Interesting idea.  Maybe add it as a GitHub issue?  The question is whether
we'd provide value by 'requiring' it if everybody does it, and whether our
testing would provide some value too.
On Mon, Jun 1, 2015 at 8:26 PM, bourdett<EMAIL_ADDRESS>wrote:

Should we require, Supervisor Mode Execution Protection (SMEP), or
something like it. The OS kernel shall only read data from the address
space of the kernel. App note, the kernel share not read data from user
mode.
Looks like Linux has support
http://vulnfactory.org/blog/2011/06/05/smep-what-is-it-and-how-to-beat-it-on-linux/
Windows also supports SMEP.
not sure about other OS'es
—
Reply to this email directly or view it on GitHub
https://github.com/commoncriteria/operatingsystem/issues/53.

--

Jeffrey Blank
410-854-8675
Operating Systems and Applications Division
Systems and Technologies Analysis Group
NSA Information Assurance

While SMEP and SMAP are nice to haves - firstly they're hardware backed (and we can't push requirements down to hardware), and secondly they're still not deployed that widely in OSs (I doubt some of the OS vendors that are in the TC will be able to confirm that their OS does them and then we'll have a barrage of complaints). If I was to use a modern OS, I wouldn't say it was negligent if it didn't support SMEP and SMAP whereas I'd be very surprised if it didn't do ASLR or user memory separate for example.

Available in Solaris, Linux, Mac OS X, and Windows... This is tempting. We could add wording that says if the underlying platform doesn't support it, just as we did for App on OS.

We should hold the same position as we do for any hardware feature such as DEP or ASLR. If the underlying hardware supports it, it should be utilized by the OS. Also, if the all four of those OS vendors support it already, it doesn't seem unreasonable to have it as a requirement.

Fair enough - while the OSs support it we weren't able to mandate some parts of secure boot as that's down to the type of underlying hardware (does the UEFI support it etc) - if we take the same approach as ASLR and DEP - then I think it's ok though.

Not this version.


docs: add new discord server

V programming language Discord server

Please check the following before submitting your pull request:

[x] Submit one item per pull request.
[x] Additions are inserted preserving alphabetical order.
[x] You have searched the repository for any relevant issues or PRs, including closed ones.
[x] Any chat you are adding to the list is active.
[x] The pull request title is informative, unlike "Update README.md".

Thanks !


Adds the Building Permits data set to the manifest, and fixes a couple loading issues

Building permits data has been loaded into the AWS database. No need to re-run
Solves:

Low Memory error that showed up in the read_csv method
Non-printing characters at the start of the file (i.e. in the header name of the first column) now removed

@salomoneb This fixes the building permit issue. You should pull the latest dev branch, but no need to run the script - I already ran it for the 2016 opendata.dc.gov data you pointed to.


[ZEPPELIN-3850] Fix run all paragraph behavior

What is this PR for?
"Run all paragraphs" button runs only the first paragraph of the note.
This PR fixes the problem.
What type of PR is it?
Bug Fix
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3850
How should this be tested?
Unittest included
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

Thanks @felixcheung for review. Merge to master.


Update response json example

Update response json example as per newest Protocol and Wireformat specification.

Thanks, good catch!

@skrjanec If you plan further contributions, you would need to sign the eclipse contributor agreements.


cargo --list shows blank explanation for the miri command on windows

Problem
On my machine (win10, x86-64) this is the (truncated) output of cargo --list
Installed Commands:
...
    logout               Remove an API token from the registry locally
    metadata             Output the resolved dependencies of a package, the concrete used versions including overrides, in machine-readable format
    miri
    new                  Create a new cargo package at <path>
    owner                Manage the owners of a crate on the registry
...

The table row for 'miri' contains no brief explanation like the other lines. I'm assuming this is not intended.
Perhaps this is related to the fact that cargo-miri is not available for the windows-msvc toolchain.
Steps

In a Windows environment, run cargo --list
Verify that the row for 'miri' contains no corresponding explanations like the other rows.

Possible Solution(s)
No response
Notes
No response
Version
cargo 1.78.0 (54d8815d0 2024-03-26)
release: 1.78.0
commit-hash: 54d8815d04fa3816edc207bbc4dd36bf18014dbc
commit-date: 2024-03-26
host: x86_64-pc-windows-msvc
libgit2: 1.7.2 (sys:0.18.2 vendored)
libcurl: 8.6.0-DEV (sys:0.4.72+curl-8.6.0 vendored ssl:Schannel)
os: Windows 10.0.19045 (Windows 10 Home) [64-bit]

Thanks for the report. This is a known issue tracked in https://github.com/rust-lang/cargo/issues/10662. I'll close this in favor of that. Let us know if this should remain open for any reason.

iirc cargo miri is distributed through rustup.  So we should decide whether to bake-in the description like cargo fmt and cargo clippy or put this as a dividing line and say we need a more general solution.  Re-opening for now for us to weigh that.


i18n: Attempt to translate search suggestions.

Translate search suggestions if translations are available.

Continued from #13274.

CI looks like it's failing still?

I wasn't able to provision locally after rebasing, due to network issues. I pushed it up to run the CI because I made some changes to remove some conflicts.

@vinitS101 can you look at updating this?  I think the big churns to search suggestion code from the pills feature are done.

@timabbott  Would capitalization with toLocaleUpperCase() for the translated lower case string's first character be an acceptable solution here?
We currently capitalize the 1st character manually in search_suggestion.js with toUpperCase().

Jun 21 19:46:19 + ./tools/check-capitalization --no-generate

Jun 21 19:46:20 Strings not properly capitalized in frontend:

Jun 21 19:46:20 alerted messages

Jun 21 19:46:20 all messages

Jun 21 19:46:20 all public streams in organization

Jun 21 19:46:20 exclude __description__

Jun 21 19:46:20 exclude sent by me

Jun 21 19:46:20 messages with one or more attachment

Jun 21 19:46:20 messages with one or more image

Jun 21 19:46:20 messages with one or more link

Jun 21 19:46:20 private messages

Jun 21 19:46:20 sent by me

Jun 21 19:46:20 starred messages

Jun 21 19:46:20 unread messages

Jun 21 19:46:20 See https://zulip.readthedocs.io/en/latest/translating/translating.html#capitalization

Jun 21 19:46:20 Failed!

[CI is failing because of the lack in capitalization.]](https://app.circleci.com/pipelines/github/vinitS101/zulip/143/workflows/5ba5d275-d3a0-4aec-8532-768c5ca25d19/jobs/489)

I think that we could make the commit structure better by splitting this single commit into:

Adding the changes on capitalization, so it would facilitate proper translations.
Adding the Translate: tag on tests as a prep commit for your new feature, if possible.
Do the toUpperCase() replacement.
Finally, translate

I've pushed a commit that removes the second translated: from strings in tests. It uses the negated_description concept to avoid translating the strings twice.

I think that we could make the commit structure better by splitting this single commit into:
1. Adding the changes on capitalization, so it would facilitate proper translations.

2. Adding the `Translate:` tag on tests as a prep commit for your new feature, if possible.

3. Do the `toUpperCase()` replacement.

4. Finally, translate search suggestions.

This is great, thanks @vinitS101! :)

As I mentioned on CZO earlier:

The tests are an effect of the internationalization changes. That has to be one commit (or whichever is first, will fail).

The change to toLocaleUpperCase could be a different commit, but it's a one line change. Also, it could break the search suggestions if the locale is a different language.

I could break this down into commits, but I've mostly avoided that because it might cause individual commits to fail CI.

@timabbott Can you please review this? The 2nd commit is a temp commit that uses negated_description to avoid translating strings twice.
I have already only rebased this PR for now, but running prettier would be a lot easier if I squashed these commits into one (if that is the way we want to go).

hey @vinitS101 is this issue solved ?
can I work on this.

Hey @vinitS101  does this Pr solved the isssue that you are trying to solve or not because again this issue is created .

@ritikchauhan-01 This PR has been in a finished state for a while, and was pending reviews. Most changes in the commit are tests and changes to strings so that it appropriately matches with the translation (and subsequent translation).
The primary change was the use of toLocaleUpperCase() which is what I was waiting an approval on. Please feel free to continue this work in a fresh PR if you would like to get this reviewed and finished.

@ritikchauhan-01 Are you working on a fresh pr for it.
If you are not working on it can I start working for this.

If you are not working on it can I start working for this.

yeah. you start working on it.

@vinitS101 I tried making changes similar to the this pr in search_suggestions.js but when I run dev server with language of my dev server set to German I get suggestions in english only.
Can you please help me if I am doing something wrong or incomplete.
Also zulip uses i18next for internationalization so how does i18n.t() works for translation(not in my case though). It is also not imported in search_suggestions.js.

@m-e-l-u-h-a-n That happens because these strings don't have translations and thus default to English. I believe you can use ./manage.py makemessages to create translations.json files and then add some translations for these new strings for testing.

@m-e-l-u-h-a-n That happens because these strings don't have translations and thus default to English. I believe you can use ./manage.py makemessages to create translations.json files and then add some translations for these new strings for testing.


all: inconsistent hyphenation

Grepping for //.*[a-zA-Z]-\n (see CL 53510) yields some examples of phrases that are inconsistently hyphenated in the tree. Two prominent ones:

"floating-point" vs "floating point"
"run-time" vs "run time"

Low priority, but we should pick one and use it consistently. (And also do a more targeted grep to find other hyphenation inconsistencies.)

We should also be consistent about how to punctuate em-dashes. We currently have a mix of two or three hyphens, with or without AP-style spaces. The typographically-correct thing to do would be to use u+2014 (em dash) consistently, in which case I would suggest also using spaces: many editors render an em-dash as a single-width character in fixed-width fonts, so a single em dash without spaces often appears too narrow.
(Even if we don't use the em dash, we should be consistent about spacing.)

Finally, should we use typographically-correct en dashes instead of hyphens for things like algorithms (Miller–Rabin), ranges (a–z, 0–9, R7–R25), and contrasting compounds (Child–Sibling order)?
(http://www.thepunctuationguide.com/hyphen-and-dashes.html is a great reference on the various dashes and their usage.)

should we use typographically-correct en dashes instead of hyphens

Yes!

a single em dash without spaces often appears too narrow

That's a pity, but em dashes are properly used without spaces.

(But even if we don't use the em dash, we should be consistent about spacing.)

Yes, please.
On the en/em dash front, an alternative is to use two or three hypens and teach godoc to combine them. But I'd rather just use the rune to begin with.
(I just started pondered writing a tool to reflow comments, using some heuristics to trade off between semantic line breaks, reasonable line length, consistent line length, and punctuation niceties...and decided to beat a hast retreat. It would be kind of awesome for Go to be the language with the best automated comment layout engine, though.)

Another one to add to the list is UTF-8 encoded vs. UTF-8-encoded.

@bontibon I don't think there is a hard-and-fast rule for compound adjectives.
I would tend to use “UTF-8 encoded” as a predicate adjective, as in “If the text is UTF-8 encoded, …”, but “UTF-8-encoded” as an ordinary adjective, as in “For UTF-8-encoded text, …”. However, in the latter case I would almost always look for a less awkward way to write the sentence, for example, “For text encoded in UTF-8, …”.

Low priority, but we should pick one and use it consistently.

This is not true. "run time" is a noun; "run-time" is an adjective. "Run-time initialization happens at run time."


Testing your app that uses SAQ

I find I'm needing to do a lot of mocking to  unit test my app that uses SAQ.
It would be great to have a test double of Queue that basically does NOPs.
What would be needed to make this comprehensive enough to be included in the library as test helper?

imo this is out of scope and it's better to just add redis + saq to your unit tests
the alternative would be you'd have to basically implement redis
i'm not sure what the state of this is yet https://pypi.org/project/fakeredis/
but you could look into it

I am specifically talking of a lightweight test double, not something comprehensive.
e.g:

confirming that a task got queued
running a task

The latter is easier as you can just call it, but generating a valid ctx is something that, if needed, can be done transparently.
The prior really just needs to replace the Queue.enque() method.
It's not for testing that saq itself works right, so assumption is that retries, delayed execution all works just fine.
It could probably be done with no replacement of redis itself.

is using unittest mock not enough?

It's hard, I need the job object back when queueing so basically I need to implement all the logic for that in the mocks.
I have a small prototype:
@patch('magneto.config.Config.queue', new=TestQueue())
class TestSaq(IsolatedAsyncioTestCase):

    async def test_queue(self):
        job = await queue.enqueue('add', val1=3, val2=5, timeout=10)

        self.assertIsInstance(job, Job)
        self.assertEqual(job.function, 'add')
        self.assertEqual(job.timeout, 10)
        self.assertEqual(job.status, Status.QUEUED)
        self.assertEqual(job.kwargs, {'val1': 3, 'val2': 5})

And a simple double:
class TestQueue(Queue):
    def __init__(self) -> None:
        super().__init__(redis=AsyncMock(spec=Redis))

    async def enqueue(self, job_or_func: str | Job, **kwargs: t.Any) -> Job:

        job_kwargs: dict[str, t.Any] = {}

        for k, v in kwargs.items():
            if k in Job.__dataclass_fields__:  # pylint: disable=no-member
                job_kwargs[k] = v
            else:
                job_kwargs.setdefault("kwargs", {})[k] = v

        if isinstance(job_or_func, str):
            job = Job(function=job_or_func, **job_kwargs)
        else:
            job = job_or_func

            for k, v in job_kwargs.items():
                setattr(job, k, v)

        job.queue = self
        job.queued = now()
        job.status = Status.QUEUED

        return job

If there is some small refactoring in the saq.queue.Queue class that TestQueue can be made significantly smaller.
If this is expanded to also make calling tasks a little easier (e.g. generating a valid ctx), and handle queue.apply()/queue.map() it would probably suit the needs of 80% of users.


Couple errors on unity Asset package with .NET 4.x

It seems that in the Unity Asset the file UnityExtensions.cs line 124 variable should be _op instead of op.
And in the LoadTextureHelper.UseCases.Await.cs it's crashing because it seems LoadTextureAsync needs a GetAwaiter?

Thank you for the notice. I'll look into it and publish the updated asset store package. Could you tell me what Unity version do you use?

2018.1

Should be fixed now. The NuGet and releases are updated. The asset store package will be available as soon as Unity approves it. Thanks for raising this!


How to set shipping information to invoice?

Hi, I was able to set shipTo but I was unable to find a way to add additional shipping information , Is there any way to set shippingAddress, shipPhoneNumber, shipCity, shipCompanyName, ShipPostalCode, ShipState, ShipCountry etc..
Thanks

You would have that information on an new/existing contact, and then use setShipToContactName on the invoice

Intacct\Functions\Company\ContactCreate and Intacct\Functions\Company\ContactUpdate are two included functions


Slidev logo styles wrong

Hi, thanks for maintaining this repo, it has been super helpful!
I am the creator the Slidev, and I recently noticed the included Logo of Slidev seems have broken style.
It should look like (https://sli.dev, https://sli.dev/logo.svg):

But in this collection it becomes:

I don't know why it happened, but could you help fix it? I am here if you need any help (I have the original design in Figma). Thank you!

Hey @antfu
Oops! sorry!
Updating it now

Updated in 5c1d9713d5ae7b26a5fd26cbd2927d359c328fb5
Thanks

Thank you!


Adding commands from tmTodoTXT, with keyboard shortcuts

Would you consider adding commands like these as well, as a complement to syntax highlighting? https://github.com/sideshowcoder/tmTodoTXT/tree/master/Commands

OK, I've looked over the list of commands there, and there's now an issue for each one:

Archive Done Tasks = #10
Create like current = #30
Set task date = #31
Set task done = #12

I'm going to track the functionality for each of those issues and close this one.


🎮 Game Request
send good guys to heaven and losers to hell
Point down the features
+1 for each correct reveal and combos extras are also included

Only Elves got pointy ears
Elves are 2m minimum, while
Dwarves are 1.5m maximum
Elves weigh 80kg max, while Orcs
minimum weight is 90kg
Human don't live after 100yo • Elves got purple skin, Orcs green skin
You'll soon have to check race names, like humans who got first name and last name

.... and probably more inventive
tricks you'll have to discover
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @S-ishita 👋!
@kunjgit here,
We have this annoucement to make 📢!!
💥 Special Announcement:
It's the Call to all the designers 🔥 🎨
As you all know that GameZone has crossed 100 games 💗 , it's time that we all come up together and make the UI even better!!
We are redesigning the thumbnails, make sure you come up with the aesthetic and cool ideas 🚀 !
You are supposed to share the 4 different templates that can be used. Make sure that the size is: 1280 px X 720 px.
The frame should be such that there is only a single image of the game that's taking up the major portion of the frame. Any kind of text should not be there.
The design should be catchy and aesthetic! 🔥
Note: You are not supposed to create any issue right now for the same.
You have to share different designs right now in Discord  and Project Admin would be selecting one design and would be assigning the issue. After that only, you have to start working on it.

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Support for io.js v3.0.0

Looks like a library Massive depends on, deasync, is not compatible with iojs.  Any thoughts on whether it can be replaced?

deasync provided the needed sync capabilities better for our purposes than other libraries. I can;t speak for Rob, but Massive is, in fact, a Node-based library. Without stepping into the "schism" around all that, I think you're welcome to see if you can find something which works as well. If you do, let us know (or shoot us a PR!).
All that said, deasync is in a rather "bleeding edge" state as I recall, and may evolve, or be abaondoned (or be made compatible with io.js).
I think we are open to feedback, but when I was digging around trying to find a clean way to wrap some things up, deasync was by far the easiest to use.

the concept of de-async seems so foreign in node/js.  Keep in mind iojs is the next node, it's being merged/finalized as we speak.
what are you using deasync for?  I'll poke around in the code and see if I can come up with something and help!

I originally wanted a Sync "load" method, so one would have the option of letting a host application bootstrap without diving immediately into nested callbacks. When Massive spins up, it loads a bunch of stuff and caches it. It seemed like there were cases where this might be attractive.
Rob added a bunch of additional sync methods, I am guessing for use with his Meteor.JS stuff (but I could be wrong). It is a node-based application, but there are cases where some sync love is not a bad thing. So long as you are intentional about it.
We'd love the help! :-)

A growing use case for Node is as a utility tool - not just as the main executor for a web site/API. In some cases people use it for simple scripting of "things" - like data analysis and loading/unloading of sample/analytical data. This used to be the sole domain of Python - but more people are turning to Node because it's so simple to use.
So, Massive has sync abilities in there to accommodate. Given that I made this thing for me, mostly, I put it in there :). I hear you about it being "foreign" to Node - and I'm a very big advocate against doing things synchronously - but there are times when it just needs to be done (like when your app starts and you want a Massive instance).
I'm not going to implement changes for a future version of something - if Node bumps and deasync breaks, then we'll deal with it when it happens.


Adding ASCII ART Text builder (For inPageBuilder)

Hi Elius, is there anything that I can help you with?

Hello! Nice to hear that! If you want, of course you can help me to solve some issues. Since this is a new feature, you can propose this add-on wit a pull request, so I'll review and if it's all ok, I will release it.
In this particular case: it could be difficult because to make a generator of ASCII art you need to have a font archive and I think this can take up a lot of space. You could take a cue from an open source library that already has this thing. However, you may need to make it an installable plugin from a second package .however, if you want to help me, keep in mind that:

Must be a function that returns an Array of StyledElements
It must be able to be added to a common PageBuilder.
No other files should be modified. Only fresh files should suffice.

Thank you very much for helping.
If I think it is a valid proposal, I will be happy to release your contribution.

Hello! Nice to hear that! If you want, of course you can help me to solve some issues. Since this is a new feature, you can propose this add-on wit a pull request, so I'll review and if it's all ok, I will release it.
In this particular case: it could be difficult because to make a generator of ASCII art you need to have a font archive and I think this can take up a lot of space. You could take a cue from an open source library that already has this thing. However, you may need to make it an installable plugin from a second package .however, if you want to help me, keep in mind that:

Must be a function that returns an Array of StyledElements
It must be able to be added to a common PageBuilder.
No other files should be modified. Only fresh files should suffice.

Thank you very much for helping.
If I think it is a valid proposal, I will be happy to release your contribution.

Thanks for the explanation. After my frustration with Ink and Blessed I've just found your library today and it seems very promising to me. Though, I haven't used it yet but I would love to be part of it.
I actually didn't mean to help with this specific feature because I don't know much about it (but I'm willing to learn it if needed). If you don't mind, could you kindly give me a beginner friendly guideline about how I can get started with this project. Is there any prerequisites (other than Node.js and Typescript) that I should have before studying the source code?
Thank you 💝.

Thanks for the explanation. After my frustration with Ink and Blessed I've just found your library today and it seems very promising to me. Though, I haven't used it yet but I would love to be part of it.
I actually didn't mean to help with this specific feature because I don't know much about it (but I'm willing to learn it if needed). If you don't mind, could you kindly give me a beginner friendly guideline about how I can get started with this project. Is there any prerequisites (other than Node.js and Typescript) that I should have before studying the source code?
Thank you 💝.

Thanks a lot for the interest!
To start with something easier, you can focus on this other issue:
https://github.com/Elius94/console-gui-tools/issues/54
It should be a quick work to add this feature to the progress bar class.
So, if you upload a PR, I will happy to check it and if it's all fine, I will merge it :)

Is there any prerequisites (other than Node.js and Typescript) that I should have before studying the source code?

No, you only need Node (14 or higher) and all the developer dependency (see in Package.json)

Clone the repo or fork it.
Open your text editor inside.
Edit.
Run npm run build
Make a new example file in the examples folder and make it works with your code and your patch
Once you have checked that all is fine, make a pull request directly from your fork.

Finally I'll check your mod by myself.


Airflow dag not being created

Affected module
UI
Describe the bug
Airflow dag not being created
To Reproduce
I installed an application using Helm. Application and packages version 0.13.0
When trying to create a dags, an error occurs:
[2022-12-20 15:25:06,185] {deploy.py:112} ERROR - Failed to import dag_file [/opt/airflow/dags/test_metadata_HGcN2EBl.py]: Failed to generate workflow [test_metadata_HGcN2EBl] verify config is correct: Could not get service from type pipelineService
[2022-12-20 15:25:06,185] {deploy.py:83} DEBUG - Traceback (most recent call last):
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/workflows/workflow_factory.py", line 74, in build_dag
    workflow = workflow_builder.build()
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/workflows/workflow_builder.py", line 50, in build
    dag = build_fn(self.airflow_pipeline)
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/workflows/ingestion/metadata.py", line 61, in build_metadata_dag
    workflow_config = build_metadata_workflow_config(ingestion_pipeline)
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/workflows/ingestion/metadata.py", line 45, in build_metadata_workflow_config
    source=build_source(ingestion_pipeline),
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/workflows/ingestion/common.py", line 182, in build_source
    raise InvalidServiceException(f"Could not get service from type {service_type}")
openmetadata_managed_apis.workflows.ingestion.common.InvalidServiceException: Could not get service from type pipelineService

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/api/routes/deploy.py", line 68, in deploy_dag
    response = deployer.deploy()
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/operations/deploy.py", line 165, in deploy
    dag_py_file = self.store_and_validate_dag_file(dag_runner_config)
  File "/home/airflow/.local/lib/python3.9/site-packages/openmetadata_managed_apis/operations/deploy.py", line 113, in store_and_validate_dag_file
    raise exc

I don't see any problem in json dag.
... 		}, 	"service": { 		"id": "f5a53f56-da12-48e8-8daf-4fc3666e9661", 		"type": "pipelineService", 		"name": "airflow_test", 		"fullyQualifiedName": "airflow_test", 		"description": "airflow_test", 		"displayName": null, 		"deleted": false, 		"href": null 	}, ...
Expected behavior
A clear and concise description of what you expected to happen.
Version:
dockerhub
openmetadata/ingestion                            0.13.0
openmetadata/server                               0.13.0
Additional context
Add any other context about the problem here.

Hi @Armadik, this is failing when Airflow tries to GET the service from the OpenMetadata server.
Could you please validate if your ingestion-bot has proper permissions to run the call? You can get the token from Settings > Bots > Ingestion bot and then run a cURL request similar to
curl --location --request GET 'http://<server>:8585/api/v1/services/databaseServices/name/<service name>' \
--header 'Accept: application/json' \
--header 'Authorization: Bearer <token>'

I'm having the same error, and executing this gets me {"code":403,"message":"Principal: CatalogPrincipal{name='ingestion-bot'} operations [ViewAll] not allowed"}

@pmbrull Thanks for the quick response. I noticed that the problem does not occur when using authentication provider: "basic". In my schema I am using custom-oidc integration with keycloak. It's not entirely clear to me what jwt I should and where to put it?
{"code":403,"message":"Principal: CatalogPrincipal{name='ingestion-bot'} operations [ViewAll] not allowed"}

Hi @Armadik and @martsec, the bug will be fixed on 0.13.1 by this PR https://github.com/open-metadata/OpenMetadata/pull/9351. As a workaround, you can assign the Ingestion bot role to the bot if it is not in {your_domain}/bots/ingestion-bot.

@Armadik for other SSO providers, they are the ones generating the SSO for the auth. So to curl endpoints under keycloack, their docs should cover how to get that going.
In any case, I'd recommend sticking to JWT tokens for bots and adding the missing role as @nahuelverdugo pointed above.
Please, confirm if that is working for you and we'll close this issue as the bug has already been fixed.
Thanks

Thank you very much, Adding the role solved my problem!


Fix forced image width issue

@ariganu on rotonde mentioned an issue with images smaller than 500px in width being scaled up:

This will fix that issue. Since the same css rule has a max-width of 500px, the image will still be contained within the parent's boundaries.

cheers :)
We should probably make larger ones clickable to view them full size too I reckon.


Rest Api support that enable and disable job

TODO write wiki, but it's difficult, *&^%$

Here is the wiki, https://github.com/vipshop/Saturn/wiki/Saturn-Console-Rest-Api, haha


MXLastMessage text and attributedText are always nil

After start session I have tried to call all types of methods like session.fixRoomsSummariesLastMessage() also tried to reset it, but still those properties are always nil.
The only piece of code, that helped me to retrieve it is finding it by eventId. Could you, please tell the best way to retrieve it? Or what should be the correct flow of getting the room last message text
`private func getMessage() {
        session.event(withEventId: summary.lastMessage.eventId, inRoom: room.roomId) { response in
            guard let message = response.value?.content["body"] as? String else {
                self.lastMessageEvent = "No message"
                return
            }
            self.lastMessageEvent = message
        }
}`

Also, as a new user to Matrix SDK, with all respect to your team and developers, who left awesome comments in the project, I would like to ask you to improve the documentation for the SDK for iOS, since your repository is the only source of the information (except your comments on the code, which actually helped me a lot) and your console sample app is out of date and not supported and has not straightforward and easily understandable logic, which is really time consuming. It is not the best way to master your SDK for developers that way.

Same as https://github.com/matrix-org/matrix-ios-sdk/issues/1782


[app-test] fix invalid conf file

set valid conf file path for nnstreamer
Signed-off-by: Jaeyun Jung<EMAIL_ADDRESS>

:memo: Version: 1.20190225. Thank you for submitting PR #516. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


Add requestTimeout option

Sorry I didn't see this sooner. Taking a look now.

This change looks helpful, but it needs tests and documentation. Tests should be added to operation-helper.specs.js and explanation of the new option should be added to the README.


WIP: Adding ability to provide custom editors

Addresses #162

This isn't ready to merge by any means. I got something working so I could wrap my brain around the problem. Now I'm wondering what the best interface could be. I believe I have a change that may not be backwards compatible - fieldValue is now being passed from TableBody to TableEditColumn as an attribute.

This is a working example:

@abhirao, it's seems look like good at first sight. Maybe can released on v3.0.0. How about it? or did you want to released in near future?

@AllenFang Thank you very much for creating this library! Sorry I was traveling last week and didn't get a chance to respond to you.
I'm not sure what the timeline is for v3.0.0 but I think our team would like the custom cell editing sooner. I can definitely help with this.

@abhirao, that's awesome, I'll expect your contribution:) So I just keep this PR open and wait your confirm for completion.
BTW, if you complete this PR and if I almost finish v3.0.0, I'll release them together.

I analyzed the backwards compatibility of this PR and I think it should be fine.  The only prop type that is changing is for editable columns for the field values.
My current implementation of custom editors is following a different pattern than custom filters. I'm going to do a little refactor to make the customEditor work the same way as filter.

@AllenFang I've updated the interface for my custom editor implementation to match the custom filter interface.
Custom Editor:
<TableHeaderColumn             dataField='price'             dataFormat={ priceFormatter }             customEditor={ { getElement: createPriceEditor, customEditorParameters: { currencies: currencies } } }>             Product Price           </TableHeaderColumn>
Custom Filter:
<TableHeaderColumn dataField='isInStock' filter={ { type: 'CustomFilter', getElement: getCustomFilter, customFilterParameters: { textOK: 'yes', textNOK: 'no' } } }>Product Is In Stock</TableHeaderColumn>
I think it's good to go. What do you think? Let me know if/how I can help with documentation.

@abhirao, that's awesome :) I'll spend some time to review this PR. But need some time, it's busy in these day ;(
Anyway, you did a great job and thanks you!

@AllenFang You're welcome.
Thank you very much for all you do keep this project going 🙇 .

Just a passion on it to keep me to contribute this repo. I'm busy in the recent, but I think I can back in next month! Anyway, if u r interesting for contributing this repo at long term, welcome to discuss with me :)

@abhirao, I need to say thanks agin. This PR has been released on v2.4.0. Thanks you :) That's awesome!


No script runtime error when rendering kte templates on latest IntelliJ

I am using JTE as part of a Gradle pre-compiled script plugin to generate buildconfig  and IntelliJ IDE is started showing No script runtime was found in the classpath error . Everything was fine recently and i am not sure since when it started having this IDE issue. Any idea what's happening here? By the way, gradle build is working fine as expected, the issue is only on IDE.
Env
IntelliJ IDEA 2023.2.1 Preview (Ultimate Edition)
         Build #IU-232.9559.10, built on August 9, 2023
         org.jusecase.jte-intellij (2.1.0)
         Kotlin: 232-1.9.0-IJ9559.10
Jte.        :  3.0.2
Gradle      :  8.3-rc-4
Kotlin.     :  1.9.0

JTE Config
// jte  = "3.0.2"
plugins {
  `kotlin-dsl`
  alias(libs.plugins.jte)
  ...
}

jte {
  contentType = gg.jte.ContentType.Plain
  sourceDirectory = sourceSets.main.map { it.resources.srcDirs.first().toPath() }
  generate()
}

dependencies {
  implementation(libs.jte.runtime)
}

IDE Error
No script runtime was found in the classpath: class 'kotlin.script.templates.standard.ScriptTemplateWithArgs' not found. Please add kotlin-script-runtime.jar to the module dependencies.

Hmm, that's weird. So I assume it started with the IntelliJ upgrade and worked normally before?
I had a quick look at the plugin and found this:
https://github.com/casid/jte-intellij/blob/6193b3870016f42155de503660a2985d761df8ac/src/main/java/org/jusecase/jte/intellij/language/KteKotlinLanguageInjector.java#L293
We explicitly request Kotlin as language here, but there's an overload to startInjtecting that takes a file extension as second parameter. I'll give it a try if this change helps to force Kotlin instead of Kotlin Script:
registrar.startInjecting(KotlinLanguage.INSTANCE, "kt");
Unfortunately I won't be able to test it until somewhere next week.
If you need a fix for it sooner, you could try to check out the plugin, apply the fix, build a local version of the plugin and test with it.

So I assume it started with the IntelliJ upgrade and worked normally before?

I think so.

registrar.startInjecting(KotlinLanguage.INSTANCE, "kt");

Yup, that did the trick. I built and updated the plugin and it seems ok now.

Aww, thanks for testing and the confirmation!
I've applied the fix and published a new plugin version. Will probably take 1-2 days to go through JetBrains Review.


[BUGFIX] Calling remove_at to manipulate elements when using Telescope UI causes items list to become malformed

Fixes #627

this might be the same fix #573

ok, this looks pretty good, let me make sure this is all good

dad can we get a merge

@ThePrimeagen is this good to merge?


Add chainermn support to apply_to_iterator

import time

from chainer.iterators import SerialIterator
import chainermn
from chainercv.utils import apply_to_iterator

comm = chainermn.create_communicator('naive')

def func(xs):
    time.sleep(1)
    ys = [x * x for x in xs]
    print('{}: {} -> {}'.format(comm.rank, xs, ys))
    return ys

if comm.rank == 0:
    dataset = [
        (3, 'one'), (4, 'one'), (5, 'nine'), (2, 'six'),
        (5, 'three'), (5, 'eight'), (9, 'seven'), (9, 'zero')]
    iter_ = SerialIterator(dataset, batch_size=5, repeat=False, shuffle=False)
    in_values, out_values, rest_values = apply_to_iterator(
        func, iter_, comm=comm)
    for in_, out, rest in zip(in_values[0], out_values[0], rest_values[0]):
        print(in_, out, rest)
else:
    apply_to_iterator(func, None, comm=comm)

with 3 workers
2: [5] -> [25]
1: [5, 2] -> [25, 4]
0: [3, 4] -> [9, 16]
3 9 one
4 16 one
5 25 nine
2 4 six
5 25 three
2: [9] -> [81]
1: [9] -> [81]
0: [5] -> [25]
5 25 eight
9 81 seven
9 81 zero

COCO example (note that I reduced the size of dataset)
w/o comm
100 of 100 samples, 4.87 samples/sec
mmAP (all): 0.39965793
mmAP (large): 0.5002278
mmAP (medium): 0.45323664
mmAP (small): 0.2703926

w/ comm (4 workers)
100 of 100 samples, 14.48 samples/sec
mmAP (all): 0.39965793
mmAP (large): 0.5002278
mmAP (medium): 0.45323664
mmAP (small): 0.2703926

CamVid example
w/o comm
233 of 233 samples, 16.07 samples/sec
                    Sky : 0.8801
               Building : 0.6662
                   Pole : 0.1924
                   Road : 0.8720
               Pavement : 0.6394
                   Tree : 0.6242
             SignSymbol : 0.1888
                  Fence : 0.2119
                    Car : 0.6421
             Pedestrian : 0.2704
              Bicyclist : 0.2389
==================================
               mean IoU : 0.4933
 Class average accuracy : 0.6730
Global average accuracy : 0.8255

w/ comm (4 workers, note that 233 % 4 is not 0)
233 of 233 samples, 34.42 samples/sec
                    Sky : 0.8801
               Building : 0.6662
                   Pole : 0.1924
                   Road : 0.8720
               Pavement : 0.6394
                   Tree : 0.6242
             SignSymbol : 0.1888
                  Fence : 0.2119
                    Car : 0.6421
             Pedestrian : 0.2704
              Bicyclist : 0.2389
==================================
               mean IoU : 0.4933
 Class average accuracy : 0.6730
Global average accuracy : 0.8255


Update to rust-url 1.0

Do not merge yet: rust-url 1.0 is not published yet and may still get breaking changes. The goal of this PR for now is to demonstrate API usage.
Depends on https://github.com/servo/rust-url/pull/176

Adapted in https://github.com/alexcrichton/git2-rs/commit/93b468419feae3599f201877100cbc28c3389045, thanks!


fix   (ide): Specify Java/Go/Kotlin/TS in .bazelproject

Merged as https://github.com/enola-dev/enola/commit/91dfb2a5ed96beddba027e5a7698ad53ce1c2f6a as part of #637.


Badge request by Cerebrux

I am confident that I deserve these gorgeous badges

skilltree badges have been relocate to GDrive.


Clarify keystore type with SYM_ENCRYPT

A couple things in encryption docs I found slightly misleading.

SYM_ENCRYPT and ASYM_ENCRYPT need to go above NAKACK2.
You can use other keystore formats with SYM_ENCRYPT with the keystore_type config attribute.

Hi @oraNod
No, the encryption protocols need to go below NAKACK2!! Take a look at the sample configs.

Thanks @belaban ! I thought you were going to hit on this. It's something I find slightly confusing. Bear with me if this is something that should be obvious of self-explanatory. I understand that the ordering matters so that NAKACK2 is also encrypted.
When I look at the example configs, it appears to me that the *_ENCRYPT protocols occur in the stack before NAKACK2. This is why it lead me to believe there was a typo (before = above). With the Infinispan docs, we use stack.combine="INSERT_BEFORE" like this:
<ASYM_ENCRYPT ... stack.combine="INSERT_BEFORE" stack.position="pbcast.NAKACK2"/>

Maybe I'm thinking about the order of the stack in the wrong way after looking at Infinispan configs.
I'll revert the changes in this PR but wanted to explain. Hopefully it makes sense and thank you for the corrections.

Yes, this is confusing! I always talk about protocols wrt their position in the running stack, not the position in the config.
Note that I once wanted to change the config, to reflect the actual running stack, but this caused an uproar, as all configs would have had to be changed... :-)

I see. Thanks for explaining the background there! I've changed it back.

@belaban Thanks again for explaining how JGroups docs are built. I updated the PR to remove the commit with all those unintentional whitespace changes (it was my editor).
I also looked into SYM_ENCRYPT.java to make sure I applied changes in the correct place and then did a local build. It seems like the changes I made are outside the generated docs.

Thx Done!