System.AggregateException

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
cant open roblox
Bloxstrap Log
System.AggregateException: One or more errors occurred. (Deserialised application settings is null!)
[Inner Exception]
System.Exception: Deserialised application settings is null!

https://github.com/bloxstraplabs/bloxstrap/wiki/"Deserialised-application-settings-is-null!"-when-launching-Roblox


Releasing an alpha version does not tag the SDK

What happened?
Following a bot-initiated alpha release:
@release-bot release pulumi-aws v6.0.2-alpha.1

Observing that sdk/v6.0.2-alpha.1 tag has not been added to the repository.  Therefore the Go SDK is not released.
The job had a Java publishing failure but that seems like it should be unrelated.

Steps to reproduce
N/A
Output of pulumi about
N/A
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@t0yv0 what else needs to happen here going forward?

After the repo transitions to ci-mgmt workflows we can move it to ci-mgmt; something is not quite reliable with the workflow generation.

So in fact
release.yml include this:
  tag_sdk:
    name: tag_sdk
    needs: publish_sdk
    runs-on: ubuntu-latest
    steps:
    - name: Checkout Repo
      uses: actions/checkout@v3
    - name: Install pulumictl
      uses<EMAIL_ADDRESS>      with:
        repo: pulumi/pulumictl
    - name: Add SDK version tag
      run: git tag sdk/v$(pulumictl get version --language generic) && git push origin
        sdk/v$(pulumictl get version --language generic)

But it does not apply to alpha pre-releases, so that's probably what's wrong here. We should fix it so we can tag alpha releases seamlessly.


Error when building in electron-builder

Hi there just wanted to report a bug when building in Electron-builder for Electron Castlabs, it seems to be looking at the incorrect link when it goes to download the file
./node_modules/.bin/electron-builder --linux --win --x64
  • electron-builder  version=23.1.0 os=5.15.55-1-MANJARO
  • writing effective config  file=dist/builder-effective-config.yaml
  • packaging       platform=linux arch=x64 electron=19.0.6+wvcus appOutDir=dist/linux-unpacked
  ⨯ cannot resolve https://github.com/electron/electron/releases/download/v19.0.6+wvcus/electron-v19.0.6+wvcus-linux-x64.zip: status code 404
github.com/develar/app-builder/pkg/download.(*Downloader).follow
	/Volumes/data/Documents/app-builder/pkg/download/downloader.go:237
github.com/develar/app-builder/pkg/download.(*Downloader).DownloadNoRetry
	/Volumes/data/Documents/app-builder/pkg/download/downloader.go:128
github.com/develar/app-builder/pkg/download.(*Downloader).Download
	/Volumes/data/Documents/app-builder/pkg/download/downloader.go:112
github.com/develar/app-builder/pkg/electron.(*ElectronDownloader).doDownload
	/Volumes/data/Documents/app-builder/pkg/electron/electronDownloader.go:192
github.com/develar/app-builder/pkg/electron.(*ElectronDownloader).Download
	/Volumes/data/Documents/app-builder/pkg/electron/electronDownloader.go:177
github.com/develar/app-builder/pkg/electron.downloadElectron.func1.1
	/Volumes/data/Documents/app-builder/pkg/electron/electronDownloader.go:73
github.com/develar/app-builder/pkg/util.MapAsyncConcurrency.func2
	/Volumes/data/Documents/app-builder/pkg/util/async.go:68
runtime.goexit
	/usr/local/Cellar/go/1.17/libexec/src/runtime/asm_amd64.s:1581
  ⨯ /mnt/Storage/Electron-applications/PrimeVideos/node_modules/app-builder-bin/linux/x64/app-builder process failed ERR_ELECTRON_BUILDER_CANNOT_EXECUTE
Exit code:
1  failedTask=build stackTrace=Error: /mnt/Storage/Electron-applications/PrimeVideos/node_modules/app-builder-bin/linux/x64/app-builder process failed ERR_ELECTRON_BUILDER_CANNOT_EXECUTE
Exit code:
1
    at ChildProcess.<anonymous> (/mnt/Storage/Electron-applications/PrimeVideos/node_modules/builder-util/src/util.ts:250:14)
    at Object.onceWrapper (node:events:628:26)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1091:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

I installed it by doing
npm install "https://github.com/castlabs/electron-releases#v19.0.6+wvcus" --save-dev

But now it seems to be doing this and looking at the wrong address.

You are right, it does look for the wrong file if you don't tell electron-builder what to do. You should check out this FAQ 😄

No, electron-builder is meant to be used with stock Electron so it defaults to try and download from that repo. This is during the packaging step, npm/yarn will do the right thing during development (i.e. when installing in node_modules).

I know it is but it hasn't ever had a issue using electron castlabs in the past, did something change?
Hmm I added
  "electronDist": "node_modules/electron/dist"
}

To my package.json file but now I am running into this issue when building..
    /mnt/St/E/PrimeVideos  ./node_modules/.bin/electron-builder --linux --win --x64
  • electron-builder  version=23.1.0 os=5.15.55-1-MANJARO
  • loaded configuration  file=package.json ("build" field)
  • writing effective config  file=dist/builder-effective-config.yaml
  • packaging       platform=linux arch=x64 electron=19.0.6+wvcus appOutDir=dist/linux-unpacked
  • copying Electron  source=/mnt/Storage/Electron-applications/PrimeVideos/node_modules/electron/dist destination=/mnt/Storage/Electron-applications/PrimeVideos/dist/linux-unpacked
  • building        target=snap arch=x64 file=dist/PrimeVideos_1.0.0_amd64.snap
  • building        target=AppImage arch=x64 file=dist/PrimeVideos-1.0.0.AppImage
  • application Linux category is set to default "Utility"  reason=linux.category is not set and cannot map from macOS docs=https://www.electron.build/configuration/linux
  • packaging       platform=win32 arch=x64 electron=19.0.6+wvcus appOutDir=dist/win-unpacked
  • copying Electron  source=/mnt/Storage/Electron-applications/PrimeVideos/node_modules/electron/dist destination=/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked
  • application Linux category is set to default "Utility"  reason=linux.category is not set and cannot map from macOS docs=https://www.electron.build/configuration/linux
  ⨯ ENOENT: no such file or directory, rename '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/electron.exe' -> '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/PrimeVideos.exe'  failedTask=build stackTrace=Error: ENOENT: no such file or directory, rename '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/electron.exe' -> '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/PrimeVideos.exe'
    /mnt/St/E/PrimeVideos  ./node_modules/.bin/electron-builder --linux --win --x64
  • electron-builder  version=23.1.0 os=5.15.55-1-MANJARO
  • loaded configuration  file=package.json ("build" field)
  • writing effective config  file=dist/builder-effective-config.yaml
  • packaging       platform=linux arch=x64 electron=19.0.6+wvcus appOutDir=dist/linux-unpacked
  • copying Electron  source=/mnt/Storage/Electron-applications/PrimeVideos/node_modules/electron/dist destination=/mnt/Storage/Electron-applications/PrimeVideos/dist/linux-unpacked
  • building        target=snap arch=x64 file=dist/PrimeVideos_1.0.0_amd64.snap
  • building        target=AppImage arch=x64 file=dist/PrimeVideos-1.0.0.AppImage
  • application Linux category is set to default "Utility"  reason=linux.category is not set and cannot map from macOS docs=https://www.electron.build/configuration/linux
  • packaging       platform=win32 arch=x64 electron=19.0.6+wvcus appOutDir=dist/win-unpacked
  • copying Electron  source=/mnt/Storage/Electron-applications/PrimeVideos/node_modules/electron/dist destination=/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked
  • application Linux category is set to default "Utility"  reason=linux.category is not set and cannot map from macOS docs=https://www.electron.build/configuration/linux
  ⨯ ENOENT: no such file or directory, rename '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/electron.exe' -> '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/PrimeVideos.exe'  failedTask=build stackTrace=Error: ENOENT: no such file or directory, rename '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/electron.exe' -> '/mnt/Storage/Electron-applications/PrimeVideos/dist/win-unpacked/PrimeVideos.exe

Hm, maybe something has changed with electronDist since it is not working for you, I need to look into that.
But generally this has always been the case. As far as I know electron-builder has never had the ability detect and use ECS instead of stock Electron - so some configuration has always been necessary. I believe there are other ways to do it than what we suggest in the FAQ though, so maybe you were using some other solution before?

yeah I would look into it because it did use to build no problem without needing to add these extra lines to package.json

I'm not sure I follow your meaning, but since electron-builder is maintained by the Electron Community it will likely always require special treatment when using it with a fork that publishes separate binaries - as we do. I would be very surprised if support for ECS was directly added there, so extra configuration will likely always be necessary (and has always been).
About the electronDist issue I noticed that the logs say it wants to move electron.exe even though you are appear to be building for Linux, so something is clearly wrong there (since the executable on Linux is only named electron). Did you by any chance miss this quote in the FAQ?

Using this option lets electron-builder skip the download step but, as a consequence, only works on the platform that matches your installed distribution.

Yeah  think it had to do with it not adding the build line when installing Electron Castlabs, I made sure to add
"build": {
  "electronDownload": {
    "mirror": "https://github.com/castlabs/electron-releases/releases/download/v"
  }
}

to any projects that didn't get it and it fixes the issue but I have no idea why it wasn't adding it

I'll close this now, thanks for the help and support :smile: :+1:  @khwaaj


Class name collisions

I have two model classes, Scheduler and Strategy.  Inside Scheduler I have the following function:
def add_strategy(role, strategy_name)
  ...
  strategy = Strategy.find_or_create_by(:name => strategy_name)
  ...
end

When I execute this method, I get the following error:
undefined method `find_or_create_by' for Mongoid::Relations::Cascading::Strategy:Class (NoMethodError)

If I execute Strategy.find_or_create_by in the console, there are no problems.  For some reason, Strategy is colliding with a Mongoid defined class.  I assume there is a good reason for this, so is there some alternative way of referencing my Strategy model/collection from within another Mongoid::Document model?

Hey mate. Mongoid::Relations::Cascading::Strategy shouldn't be causing a conflict here. That's a bug.
In the meantime, you can work around this by referencing "::Strategy". Eg:
def add_strategy(role, strategy_name)
  ...
  strategy = ::Strategy.find_or_create_by(:name => strategy_name)
  ...
end

BTW, I've confirmed this bug in 2.4.0 .

Thanks for the work around, I figured it was something like that.  Keep up the good work.


Express setup: Better error message when swe-agent docker image isn't found

For users with full docker setup:
When users forget to run docker pull sweagent/swe-agent:latest before starting run.py, they get an error like #110.
Instead, we should show something like "Docker image ... not found. Please make sure to run docker pull .... or consult the readme for more information"

import docker

client = docker.from_env()

image_name = 'image-name'
filterred_images = client.images.list(filters={'reference': image_name})

Can be tackled together with #123


Disable Tab and S-Tab

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
local cmp = require('cmp')

local cmp_mappings = cmp.mapping.preset.insert({
  ["<C-b>"] = cmp.mapping.scroll_docs(-4),
  ["<C-f>"] = cmp.mapping.scroll_docs(4),
  ["<C-y>"] = cmp.mapping.complete(),
  ["<C-e>"] = cmp.mapping.abort(),
  ["<CR>"] = cmp.mapping.confirm({ select = false }),
});

-- disable completion with tab
-- this helps with copilot setup
-- This doesn't seem to work though
cmp_mappings['<Tab>'] = nil
cmp_mappings['<S-Tab>'] = nil

cmp.setup({
  sources = {
    { name = 'nvim_lsp' },
    { name = 'path' },
    { name = 'luasnip' },
    { name = 'buffer',  keyword_length = 5 },
  },
  mapping = cmp_mappings,
})

Description
I want to disable Tab fully as I am using Copilot which I want to use Tab with. However, even if I try to "unbind" Tab it still is mapped when checking :map

Steps to reproduce
Any setup with cmp
Expected behavior
Tab should be able to disable
Actual behavior
Can't disable Tab
Additional context
Tried https://github.com/hrsh7th/nvim-cmp/issues/880#issuecomment-1094144400 but had no effect either

Closing, seemed to reset after a couple of tried of just disabling and enabling cmp and copilot. Seemed like something was cached?

@RobertBrunhage how did you disable/enabled cmp and copilot? Still unable to unmap tab.

@trigun539 I used what I described in the issue and after some restarts it just correctly unmapped it. Not sure if it somehow was cached or similar, but for me this was in the end, enough.
-- disable completion with tab
-- this helps with copilot setup
-- This doesn't seem to work though
cmp_mappings['<Tab>'] = nil
cmp_mappings['<S-Tab>'] = nil

The nil has specisl,meaning in Lua.
You should ise vim.NIL


Looks like I need some more packages installed to work

║ Host system is missing dependencies to run browsers. ║
║ Please install them with the following command:      ║
║                                                      ║
║     sudo playwright install-deps                     ║
║                                                      ║
║ Alternatively, use apt:                              ║
║     sudo apt-get install libatk1.0-0\                ║
║         libatk-bridge2.0-0\                          ║
║         libdrm2\                                     ║
║         libxkbcommon0\                               ║
║         libxcomposite1\                              ║
║         libxdamage1\                                 ║
║         libxfixes3\                                  ║
║         libxrandr2\                                  ║
║         libgbm1\                                     ║
║         libasound2\                                  ║
║         libatspi2.0-0                                ║
║                                                      ║
║ <3 Playwright Team

Hi @jonhilgart22 ,
Just to double check, did you add the 3rd party buildpack yet?

I ended up just building with a Dockerfile because my slug was too large. Thanks!


Homebrew install not working with my M1 silicon chip

brew config output
fatal: the remote end hung up unexpectedly
fatal: early EOF
fatal: index-pack failed
Error: Fetching /opt/homebrew/Library/Taps/homebrew/homebrew-core failed!
fatal: invalid upstream 'origin/master'
Failed during: /opt/homebrew/bin/brew update --force --quiet
brew doctor output
Not able to use 'Brew doctor' as install won't work

[X] The brew doctor above contains no "Warning" lines.

What were you trying to do (and why)?
Install Homebrew on my Macbook Pro M1 silicon chip.
What happened (include all command output)?
After using the Homebrew formulae in terminal i get a few fatal errors that rise:
command output:
HEAD is now at 9d62a13a3 Merge pull request #10700 from appropriate/docs-homebrew-and-java
error: Not a valid ref: refs/remotes/origin/master
fatal: ambiguous argument 'refs/remotes/origin/master': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git  [...] -- [...]'
fatal: the remote end hung up unexpectedly
fatal: early EOF
fatal: index-pack failed
Error: Fetching /opt/homebrew/Library/Taps/homebrew/homebrew-core failed!
fatal: invalid upstream 'origin/master'
Failed during: /opt/homebrew/bin/brew update --force --quiet
What did you expect to happen?
For homebrew to install
Step-by-step reproduction instructions (by running brew commands)
Terminal input:
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"

This is usually caused by something breaking the connection, such as security software. You can try temporarily disabling anything like that you may have installed. I also hear that Big Sur 11.3 will help a bit here.
Either way, this is unfortunately something disrupting a regular git fetch and not a brew bug. You can open a discussion, or reply to an existing one, for additional help.


Defining required extensions for a project

Is there a way to define a list of extensions that are required for a project, and have Visual Studio detect when they are missing? This would be similar to what is described in https://github.com/Microsoft/vscode/issues/5559.

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

ID: 6c50729d-6836-feee-b306-cfab96b36d17
Version Independent ID: afec4f9e-ac95-d414-62d7-c6a654721f5b
Content: Find and use Visual Studio extensions - Visual Studio
Content Source: docs/ide/finding-and-using-visual-studio-extensions.md
Product: visual-studio-dev15
GitHub Login: @gewarren
Microsoft Alias: gewarren

Visual Studio considered adding this functionality at one time, but unfortunately the feature was cut.

@gewarren according to this idea, this functionality is being added to vs2019 https://developercommunity.visualstudio.com/content/idea/351158/allow-to-automatically-install-extensionsplugins-p.html


Folder logic enhancement: folder per function group

Function groups tend to contain a lot of includes with meaningless names, which makes your repo very messy without folders. Older code is often structured in function groups.

or we serialize function groups like classes... all code in one file.

This would be huge, also the includes usually correspond to forms so there is some sense to the structure, it's just hard to navigate.

Newer code is also often structured in function groups as some functionality was never ported over to classes (rfc, update task, change documents, table maintenance, dynpro support, exits...).
The includes are, "easily", identifiable by their 3 character long suffix naming convention... Seriously though, I don't think a folder structure would help at all. You can only navigate larger function groups with IDE support where navigation to the relevant include is done automatically by selecting the component / modularization unit.

folders correspond to packages, having function groups on top of this would be confusing IMHO, but it could be a user setting
code should be serialized as close to what the developer sees, one include is one file
as a workaround, the developer can always move that one function group to a new package, or refactor the code to classes. Function groups are good for RFC, which is external communication, this would be nice to group in one package, and have all the business logic in a different package implemented with classes

It's not that easy to change package hierarchies, there's more bureaucracy involved and this actually complicates our downport just because I want to create some subpackages and relocate a bunch of objects there.
Well I am still doing that, but the plan for the function groups is to keep them where they are and slowly turn them into classes in the relevant subpackages. This might take a long time tohugh which is why I thought to solve it on the abapgit side.
Basically a function group on the system already looks like a folder, I just want it to follow the same principle in the repo. Of course this would be configurable just like full/prefix logic.
Btw many packages use dashes instead of underscores in names so they can't use the prefix logic, this would maybe also be a nice thing to support.

another idea, use https://docs.abapgit.org/ref-exits.html#change_tadir, to simulate the desired package layout
might need another/different exit for pull

It would make things tidier especially when packages have been littered with table maintenance generator objects, but I see this as a nice to have.
On the one hand serialising into one file would be the cleanest solution, conceptually function modules are the equivalent of methods in a class. But on the other hand the way we usually work with FMs is such that we usually don't know or care what function group it's in - I just want to call function X - so I might also want to find it by name in the Git repo.
So on balance I think separating a FG out into a package is the easiest and cleanest. Shouldn't take more than a couple of minutes, or am I missing something? @FreHu what bureaucracy problems are you referring to? In most organisations shifting stuff into a subpackage is no issue.

I don't have authorization to create packages :(. In sap for some things you need an overseer with special rights or to take a course or something. I would assume that some of our customers are similarly agile in their processes :)
This has some fun consequences - I've seen my colleague create an object with the wrong name and he didn't have authorization to delete it afterwards. I also have two transport requests I created in the wrong place that I'm not authorized to delete.
I'm gonna give up on this idea as there doesn't seem that much appreciation for it and it complicates things just to make the repo slightly prettier. It's probably better to pursue other things.

Urgh, must admit I've never worked with such restrictions, my sympathies. They obviously want control to keep order, so appeal to their sensibilities and suggest generated classes just pollute own-developed packages and should be moved into a subpackage... you may even get someone to do the work for you :-)

nothing matters except $$$. translate the bureaucracy into time waste times your hourly rate. then customers will at least listen.

not happening


Link to transactions in blockchain explorer (including switch that allows to "click out" of the app)

Design with link to Block Explorer https://zpl.io/2pv4NXy
When user clicks on link (Open in Block Explorer) display popup https://zpl.io/2yqOMew and only after clock "Open" Block Explorer opens in browser.

doesn't display popup https://zpl.io/2yqOMew

@iscander01 When tapping on a link make redirects to the kernel ID

@Denis-Asipenko done

сhecked on testnet.
version:2.1.6


Integration test improvements

Contains a hack to sleep while the Appc.getInfo call in the create module flow finishes
Adds a wait in the beforeEach of each test for the environment detection to finish
Only removes the tempDirectory if we created one as sometimes the test setup can barf before we get to creating it

Warnings

:warning:

src/test/integration/util/common.ts#L40 - src/test/integration/util/common.ts line 40 – Argument 'tiapp' should be typed with a non-any type. (@typescript-eslint/explicit-module-boundary-types)

Messages

:book:

:white_check_mark: All tests are passing
Nice one! All 32 tests are passing.

:book:

:floppy_disk: Here's the built vsix for this change.

  Generated by :no_entry_sign: dangerJS against ede09b25ee35d36f80ee36446fc4dc57ef79554f


Some etags in sessionData might not be deleted in deleteFile

Describe the bug
A clear and concise description of what the bug is.
When delete multiple files at once, some etags in sessionData might not be deleted. As await is not guaranteed ordering, there might be a race condition. For example, deleting 2 files, calling deleteFile('path1') in the first line and calling deleteFile('path2') in the second line. At storage.ts:line 637, the first call will get a copied of sessionData with etags: ['path1', 'path2'] and send a request to the server. With await keyword, this is queued up and the second call is processed. The second call will get a copied of sessionData with etags: ['path1', 'path2'] and send a request to the server. When the request1 get responded, 'path1' is deleted from the copied and the rest: ['path2'] get saved to sessionData. And when the request2 get responded, 'path2' is deleted from the copied and the rest: ['path1'] get saved to sessionData.
I think the way to fix this is to get a copied of sessionData after awaiting.
Also, this might happen at storage.ts:line 555 and 599 as well. At that time, the copied of sessionData and the real sessionData might be different and some etags can be missing due to overriding of the obsolete copied.
What version of Blockstack.js are you using?
I found this bug in 21.1.1 and I believe it also happens in the latest version
Is the bug present in the Blockstack CLI, Gaia hub, Connect or Blockstack Browser?
I think not
To Reproduce
Steps to reproduce the behavior:
Delete 2 files with deleteFile('path1') and deleteFile('path2')
Expected behavior
A clear and concise description of what you expected to happen.
Etags in sessionData should be deleted
Desktop (please complete the following information):

OS: [e.g. iOS] All
Browser [e.g. chrome, safari] All
Version [e.g. 22] found in 21.1.1 and believe also in the latest version

@yknl @agraebe have you guys observed this behavior

@yknl @agraebe have you guys observed this behavior

I hope @yknl could take a look 🙏🏼

I hope @yknl could take a look 🙏🏼

I did analysis on this issue and reached to a conclusion that there is no chance of happening of this issue.
Justification:
If you look into the body of deleteFile function it does
 const sessionData = this.userSession.store.getSessionData();

The above statement will return the reference of session data instance, not the actual deep copy of session data. So its a shared variable here.
Now assume we have concurrent calls to deleteFile 2, 10 on n number of calls. Now each call will get the reference to same
session data variable and will perform the delete file operation. See a chunk from body of deleteFile
     await deleteFromGaiaHub(path, gaiaHubConfig);
     delete sessionData.etags![path];
     this.userSession.store.setSessionData(sessionData);

Here sessionData is the same reference that we got in first line. The second line is deleting the file and the third line will delete the current etag and then setting the same reference again to user session on last line above . If you reassign the Object itself, the original won't be changed. So sessionData is a shared variable and it's 100% guaranteed that etags in sessionData will get deleted in case of concurrent calls to deleteFile function (ignoring the await).
Anyone may come up with counter arguments or else we should close this issue.
cc: @agraebe , @zone117x

closing this issue due to inactivity and the analysis above. If there's anything left to discuss, please reopen with more details as to how to reproduce the exact behavior. thanks!

I can reproduce the issue but I can't find the root cause. I've created a Github repo to demo it at https://github.com/witwitchayakarn/stacks-demo.

Clone the repo
Run yarn
Go to node_modules/blockstack/lib/storage/index.js to insert console.log at line: 738 and 745 (screenshot)
console.log('After deletion, sessionData: ', sessionData);
Run yarn run start
On the web app, after sign in, put a couple arbitrary files and then delete them all
In console, you can see that etags are not empty, still has some values (screenshot)

With this issue, when creating a new file with the same name, there will be an error about etag mismatch.
Please help. Thank you.
PS. I use the old version of Stacks.js but I believe it's the same as the new version.

I think, by default running in a browser, sessionStore is a localStorageStore (auth/src/userSession.ts:82). Calling this.userSession.store.getSessionData() would get a copied of data from the localStorage (auth/src/sessionStore.ts:91). So it's not a shared variable.

Thank @witwitchayakarn for the exact steps to reproduce. Yes the root cause is the localstorage and userSession is not a shared variavle considering browser localstorage.
Reopening the issue and working on to fix the issue.


Clarification: pyouroboros/ouroboros vs circa10a/ouroboros

Hello!
When I first switched over from vtech/watcher, the project I used was circa10a/ouroboros. Searching on docker hub, I see both circa10a/ouroboros(last updated 25 days ago) and pyouroboros/ouroboros (last updated 2hrs ago).
I don't see anything in the readme about renaming project so just being a bit paranoid (insert). Which should I use now? Thank you.

Closed issue because I just saw that notice about project being renamed was put on Dockerhub.


add an integration test with dropped packets during the handshake

Fixes #392.
Should be merged after #819.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #820   +/-   ##
=======================================
  Coverage   91.03%   91.03%
=======================================
  Files          87       87
  Lines        6063     6063
=======================================
  Hits         5519     5519
  Misses        345      345
  Partials      199      199

Continue to review full report at Codecov.

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


Content + Styling: Update content and font styling "Transform a Property"

Update copy blocks and align with header / body copy styling
Remove old copy
This ticket does NOT include updates to the green content cards
https://www.figma.com/proto/NAFkgq34abW6uJ0R7PW24T/Prototype---Clean-%26-Green-Philly?type=design&node-id=2592-31488&t=sdhkPVHItWMM3Xnn-0&scaling=min-zoom&page-id=187%3A12602&starting-point-node-id=2592%3A30019

I would like to be assigned this. I'd just like a little more clarification as to the specific changes this is requesting. Thank you.

To be more specific about my question. Is the desired outcome of this request to have the Transform a Property page look like the linked prototype but without the green content cards? Thanks.
@Moylena @thansidwell

@AZBL  yes! apologies for the delay

closed by #510


Timesync events: Use a pixit to set the ceiling

With the fix for isUrgent, this isn't required anymore


Incorrect version of Ninja installed using PIP on M1

Initially reported in https://github.com/ninja-build/ninja/issues/2049
On Mac when using ARM CPU the binary installed with PIP contains x86-64 executable instead of arm64.
Initially discovered in conjunction with Meson build system in https://github.com/versatica/mediasoup/issues/706 and can be tested by cloning that repo and doing npm i on Mac machine with M1 processor.

The universal2 build seems to be broken since #70 (marked as universal2 but only contains an x86-64 binary).
Will provide a PR to fix this.


Simulation development

This PR includes:

removal of FullOptic mode (not working in this new geometry)
change of default macro card to include /tbgeo commands
removal of fVertRot bool from main() parser, it is now controlled via a dedicated macro card command
inclusion of /tbgeo commands documentation in README.md

Accepting PR #17.


Add channel override for mattermost integration

Implement mattermost channel override as a configuration parameter.
Source: Slack Incoming Webhook API documentation here

To do:

[ ] Add "channel" param to mattermost config section (old and new version must work!)
[ ] Add a "mattermost" section to readme.md


Added comic archive formats

This adds comic reader based formats.
Cb* are formats that indicate the underlying file contains some special xml file. They are the appropriate zip, rar, gzip, etc archive under the hood. Epub is a popular book format and tar.gz is common archive format.

Thanks for the contribution. Please move the .epub addition to be after .eps to maintain alphabetical order. 👍

I'm on holiday for next 2 weeks, but will do that once I'm back. Thanks.

I'm back from holiday and I have fixed it up.


Minimal example doesn't run because of incorrect import

In the minimal example: https://starlite-api.github.io/starlite/#minimal-example
The Controller (my_app/controllers/user.py) imports the model with
from my_app.models import User

This fails to startup with:
ImportError: cannot import name 'User' from 'my_app.models' (unknown location)

because the User class is defined in:
my_app/models/user.py

This would work if there were a init.py file in my_app/models that imports all the model classes (as there is for the backend-starlite-postgres), but this isn't in the minimal example.
To fix, the import should be:
from my_app.models.user import User

It would also be nice if this code were in a standalone template repo to clone and use, instead of cutting and pasting it out of the docs.

Well, TBH its not meant to be a viable example with all the files included - there is an assumption there that python pacakges are in place etc.
We have an example project in the works, the problem is that its not finished yet.

The example project would be great. My point was more that it's almost a viable example if it just used explicit imports rather than relying on the package-level init imports that don't exist. Totally fine if it's not intended to be.

You can check this out- https://github.com/lesnik512/starlite-sqlalchemy-template


Update Tokyo Toshokan

@DoumanAsh Update Tokyo Toshokan, the description links don't work. You can link to details page: https://tokyotosho.info/details.php?id=917975
Remember to update: https://github.com/qbittorrent/qBittorrent/wiki/Unofficial-search-plugins
Thanks.

Done.
Thanks for reporting


[Bug] Unable to install Kernal Memory Core without deprecated packages

Context / Scenario
Trying to install Kernel Memory Core package through nuget package manager on dotnet 8 solution. The packages referenced are deprecated and alternative suggestion is the Microsoft.KernelMemory.Core package.
What happened?
When attempting to install Microsoft.KernelMemory.Core, I'm receiving the following errors stating unable to find the (deprecated) packages.
Expected: To install Microsoft.KernelMemory.Core without errors/installing deprecated packages
Importance
I cannot use Kernel Memory
Platform, Language, Versions
Dotnet 8.0
Windows
Visual Studio 2022
Web API + Console App
Relevant log output
Errors:

Error	NU1102	Unable to find package Microsoft.KernelMemory.Abstractions with version (>= 0.50.240502.2)
  - Found 71 version(s) in nuget.org [ Nearest version: 0.40.240430.1 ]
  - Found 0 version(s) in Microsoft Visual Studio Offline Packages	KernelMemory

Error	NU1102	Unable to find package Microsoft.KernelMemory.AI.TikToken with version (>= 0.50.240502.2)
  - Found 2 version(s) in nuget.org [ Nearest version: 0.40.240501.1 ]
  - Found 0 version(s) in Microsoft Visual Studio Offline Packages	KernelMemory

Error	NU1102	Unable to find package Microsoft.KernelMemory.AI.AzureOpenAI with version (>= 0.50.240502.2)
  - Found 32 version(s) in nuget.org [ Nearest version: 0.40.240501.1 ]
  - Found 0 version(s) in Microsoft Visual Studio Offline Packages	KernelMemory

Full Package Error List:

Microsoft.KernelMemory.MemoryDb.AzureAISearch
Microsoft.KernelMemory.AI.LlamaSharp
Microsoft.KernelMemory.DataFormats.AzureAIDocIntel
Microsoft.KernelMemory.ContentStorage.AzureBlobs
Microsoft.KernelMemory.AI.OpenAI
Microsoft.KernelMemory.Orchestration.RabbitMQ
Microsoft.KernelMemory.MemoryDb.Quadrant
Microsoft.KernelMemory.MemoryDb.Postgres
Microsoft.KernelMemory.Orchestration.AzureQueues
Microsoft.KernelMemory.MemoryDb.MongoDbAtlas
Microsoft.KernelMemory.MemoryDb.Redis

This should fix the problem https://github.com/microsoft/kernel-memory/pull/458 - New nugets to be published soon

Cheers, thanks for the promptness

Problem fixed, version 0.50.240504.7


Getting UnicodeDecodeError accessing trie read from file

Hi, I'm consistently getting the following error when trying to access a trie from a load or read from a file.
./read_trie_test.py
Traceback (most recent call last):
File "./read_trie_test.py", line 18, in
print(t.restore_key(0))
File "marisa_trie.pyx", line 324, in marisa_trie.Trie.restore_key (src/marisa_trie.cpp:6365)
File "marisa_trie.pyx", line 334, in marisa_trie.Trie.restore_key (src/marisa_trie.cpp:6299)
File "marisa_trie.pyx", line 62, in marisa_trie._get_key (src/marisa_trie.cpp:1615)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 10: invalid start byte
I get the same error if the following code is used...
for k in t.keys():
print(k)
and again the same error if I use:
t['someKey'] (or t[u'somekey']
The trie file reads in w/o any error and i've written the file using both trie.save() and trie.write()
and in writing file I've used a codec.open() and codec.write() to force utf-8 encoding
I'm not sure if this is similar  issue #10

ok, never mind. I was taking the examples a little to litterally
so i was loading a BytesTrie() into a constructed Trie() - once I switched to a constructed BytesTrie() it worked fine

I'm glad it is not a bug in the marisa-trie source code :)
Do you have any suggestions about how to change the docs to make them more clear regarding this?

So am I :)
so as for the documentation, at the end of the load/save section, I'd just call out, that the Trie() constructor will not load a RecordTrie or a BytesTrie even though it will not fail. You need to construct the Trie class that you are trying to load.
Alternatively, the load() methods could throw an exception if a trie file of the wrong type is presented.

Part of the problem here is that the BytesTrie class should offer a static method for loading. The thought process that I think both jottos and I encountered was:

Okay, my trie is saved, now I want to load it
Huh, that's weird, the load method requires you to already have a trie
I guess I'll create an empty trie first, how do I do that? Oh right, marisa_trie.Trie().

If you could call BytesTrie.load('trie.marisa') as a static method, it would be easier to not go astray.


Add god hand by ithoco

Song Name
god hand
Artist Name
ithoco
Source
Youtube
Youtube Link
fQXnXPRdIOY
Direct File Link
No response
Song ID
557117
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 6508283
Downloaded: No

idk im new here


HybridDictionary has indexer (accepting object) which recursively calls itself instead of calling to types overload.

Probably the intended implementation was:
    public object this[object key]
    {
        get { return ((IDictionary<TKey, TValue>)this)[(TKey)key]; }
        set { ((IDictionary<TKey, TValue>)this)[(TKey)key] = (TValue)value; }
    }

Team triage: This is probably dead code and could be deleted.


Problem using docker-kong via docker run, but works correctly via docker-compose

The docker-compose config below works as expected
version: '2.1'

services:
  kong:
    image: "kong:1.4.0-alpine"
    environment:
      KONG_ADMIN_ACCESS_LOG: /dev/stdout
      KONG_ADMIN_ERROR_LOG: /dev/stderr
      KONG_ADMIN_LISTEN: '<IP_ADDRESS>:8001'
      KONG_DATABASE: 'off'
      KONG_LOG_LEVEL: info
      KONG_PROXY_ACCESS_LOG: /dev/stdout
      KONG_PROXY_ERROR_LOG: /dev/stderr
    ports:
      - "8000:8000/tcp"
      - "8001:8001/tcp"
      - "8443:8443/tcp"
      - "8444:8444/tcp"
    healthcheck:
      test: ["CMD", "kong", "health"]
      interval: 10s
      timeout: 10s
      retries: 10
    restart: on-failure

however this equivalent docker run command does not:
docker run --rm -it \
  -e KONG_ADMIN_ACCESS_LOG=/dev/stdout \
  -e KONG_ADMIN_ERROR_LOG=/dev/stderr \
  -e KONG_ADMIN_LISTEN=<IP_ADDRESS>:8001 \
  -e KONG_DATABASE=off \
  -e KONG_LOG_LEVEL=info \
  -e KONG_PROXY_ACCESS_LOG=/dev/stdout \
  -e KONG_PROXY_ERROR_LOG=/dev/stderr \
  -p 8000:8000/tcp \
  -p 8001:8001/tcp \
  -p 8443:8443/tcp \
  -p 8444:8444/tcp \
  kong:1.4.0-alpine

Is there any reason this should be the case, or is this a bug?

Are you sure you don't want docker run -d instead of -it?

The -it shouldn't matter, it just means it will stop when I issue a CTRL-C, and also delete the container for me. I find this a convenient way of doing ad-hoc things like quick testing and such with containers.
That said I can't reproduce the problem anymore, the command is working now, so there must have been something local to my setup that was breaking this.
Apologies for the false report.


Upgrade to Swift 5

Done about up swift version to 5. I will release it. I close this PR. Thank you your contributions 👍
https://github.com/bannzai/SpreadsheetView/pull/256

No issues at all, happy to help :) cheers 🍻
On Thu, 8 Oct 2020 at 7:17 AM, bannzai<EMAIL_ADDRESS>wrote:

Done about up swift version to 5. I will release it. I close this PR.
Thank you your contributions 👍
#256 https://github.com/bannzai/SpreadsheetView/pull/256
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bannzai/SpreadsheetView/pull/240#issuecomment-705282334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APY4CEEDRJMZCWHSHRBGFDDSJUK4JANCNFSM4OLL2H5Q
.


Decimal property is not displayed correctly in Variables in debugger

From @bgrainger on July 29, 2016 1:16

VSCode Version: Version 1.3.1 (1.3.1) e6b4afa53e9c0f54edef1673de9001e9f0f547ae
OS Version: Mac OS X 10.10.5

Steps to Reproduce:
Create a .NET Core Console application with the following code:
namespace ConsoleApplication
{
    public class Program
    {
        public static void Main(string[] args)
        {
            var t = new Test { Value = 1.0m };
            // SET BREAKPOINT HERE
            System.Console.WriteLine(t.Value);
        }
    }

    public class Test
    {
        public decimal Value
        {
            get { return _value; }
            set { _value = value;}
        }

        private decimal _value;
    }
}

Debug it by pressing F5. Break on the Console.WriteLine line and look at the Variables pane. The value of Value is displayed incorrectly (but _value) is correct. The incorrect value changes each time the program is run.
Actual Output:

Expected Output:
Both Value and _value should show 1.0 [decimal].
Copied from original issue: Microsoft/vscode#9929

From @sjones204g on July 29, 2016 2:10
I'm getting this problem as well. VS Code 1.3.1 on Mac OSX 10.11.6.

The formatting of these values is done in the debug adapter of the C# extension.

@noahfalk @Mikem8361 I think this issue is a CoreCLR issue, but before I port this over, I wanted to know if you had a quick way to confirm/deny (ex: with mdbg)? The issue is that if we evaluate a function that returns a System.Decmial, unbox the resulting value, and then look at what memory backs the resulting value, the lower 64-bits are fine, but the high 64-bit look corrupt.

We don’t have a test specifically for this scenario, but I’m currently trying to repro with mdbg.  I’ll let you know the results.

Looks like I can repro the problem with mdbg on OS X and on Linux but not Windows. So it mostly likely a bug in coreclr.
mikem

I opened https://github.com/dotnet/coreclr/issues/6935 to track this issue.

The CoreCLR team has checked in a fix for their 1.1 release

H there,
When will this fix be available?
Cheers,

The fix for properties returning decimal (or any other 16 byte value type) did making it into .NET Core 1.1.
The second fix (dotnet/coreclr#7115) for funceval of functions passing in enregistered 16 byte value types didn't make it into 1.1.  It was too late (already past escrow) when I realized that.


Invalid text representation error when trying give string to integer key

Hi,
I have primary key as a integer.
When i try this: PATCH /api/quizzes/999999999 i get not found response - it's ok.
But, when i try this: PATCH /api/quizzes/somestring i get error 500 - "Invalid text representation".
It happend on PostgreSQL database which is very strict and throw error when we try select * from "quizzes" where "quizzes"."id" = somestring.

Yeah, you don't want the somestring to even hit your database, which is why you should use ID constraints:
https://laravel-json-api.readthedocs.io/en/latest/basics/routing/#id-constraints

Thanks a lot!


creating a fake input shifter makes makes memory footprint grow

When I create a new input shifter without actual input shifter chip in place (so the pins are floating) the firmware generates a lot of phantom events for the pins when data pin is pin 2 on Pro Micro or Uno firmware (I have a Nano to test with).
There are two things I noticed:

This only happens with Pro Micro and Uno firmwares (tested with Pro Micro and Nano that runs Uno fw)
This only happens when DATA is set to pin 2.
It doesnt seem to happen on Mega

The random events are kinda understandable since the pin is floating, but it only happens when the data pin is 2, and they happen
Is there something else going on that pin?
Also these fake events seem to bloat the memory footprint, but this could be a separate issue.

why bother? what motivates you to add a fake input shifter and then worry about weird side effects? If this happens with an actual shifter attached, then I'd say that is worth investigating.
"Nano" is also not supported at the moment.

I hope you don't mind if i close this. Maybe the recent fixes have actually solved it, because pins get properly intialized. Feel free to post feedback if you can test.


[JungleMail365] Updating outdated brand names

Updated brand names like below:
Microsoft Flow -> Microsoft Power Automate
Flow -> Power Automate (only in Descriptions & Summaries)
PowerApps -> Power Apps
LogicApps -> Logic Apps

@EnovaPoint-Lewis Please approve the changes and let me know if you have any questions.


Rose tesnet bug

Hi. i am finding bug, then used Rose testnet. I wish repay RUSD, but my transaction is failded. Thats brought me here. Leaving screen down here.

Hello, @Vindicator78 . There is a fix landed on testnet that should resolve DB Not Found Error.Can you please validate if this still an issue. Thank you in advance.

Thanks, I'm tested Rose second time. All options (stake, farm, swap, borrow & repay) are working. Transaction speed impruved too. You, guys, did good job!


Google Analytics Dashboard

Ian has this mostly hooked up now. Essentially you validate with your Google account and we can bring in Google analytics data from your site. Needs design and a few days of work.

@snide  - don't know if you're ever interested in pursuing this again, but I wonder if 18F's recently created analytics tool might make this doable. Instead of reading the analytics directly, it dumps the data into JSON for reading on the site (https://18f.gsa.gov/2015/03/19/how-we-built-analytics-usa-gov/)
It's open source: https://github.com/18F/analytics-reporter And, as always, I'm happy to help, if it's welcome. Here's the site they're publishing the data on: https://analytics.usa.gov/


Vector3.one is exposed, but reported as error

Describe the bug
Vector3.one is exposed, but reported as error
To Reproduce
Type Vector3.one
Expected behavior
No report.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Windows 10]
Analyzer Version: [e.g. 22]
Unity Version: [e.g. 2018.4.20f1]
VRCSDK Version: [e.g. 2<IP_ADDRESS>.57]
UdonSharp Version: [e.g. v0.19.6]
Editor Version: [e.g. Rider 2020.3]

Additional context
Add any other context about the problem here.

My Unity workspace was broken.


fix(connection-state-recovery): pin mongodb due breaking change

Between 5 -> 6, the return value of findOneAndDelete changed. It now returns the document or null directly, see changelog
By pinning the version we'll mitigate potential breaking changes in future.

Superseded by https://github.com/socketio/socket.io-mongo-adapter/commit/1a0488562f1e8b4171af20378aacfa43072980dd.
Thanks!

Well, you are still allowing any mongodb version. I would advise to at least consider capping it. What if mongdb version 7 rolls out and introduces a new breaking change?

What if mongdb version 7 rolls out and introduces a new breaking change?

Capping the version means that we will have to publish a new version with or without breaking change, doesn't it?

What if mongdb version 7 rolls out and introduces a new breaking change?

Capping the version means that we will have to publish a new version with or without breaking change, doesn't it?

It gives you a chance to test and address these breaking changes before they are released.


Statamic shows old version in their marketplace

Bug description
Current version of this package is: v4.12.0
Statamic market place version is: v4.11.1
https://statamic.com/addons/statamic/eloquent-driver
How to reproduce
Check here: https://statamic.com/addons/statamic/eloquent-driver
Logs
No response
Environment
No env needed

Additional details
No response

Thanks for letting us know, it should be fixed now!


Npmjs package doesn't display README.

Normally I believe this is taken from the README in the package directory, which is currently non existent for anchor/ts/.
See https://www.npmjs.com/package/@project-serum/anchor.

Still occurring on  on https://www.npmjs.com/package/@coral-xyz/anchor-cli


🐛 Mobile | Weekly leaderboard shows stale data

Hi team
Describe the Bug
There's a bug where the weekly leaderboard is showing stale data, so points rewarded from 2 weeks ago might be showing, and new data is missing.
Tasks

[x] Investigate
[x] Fix

Acceptance Criteria

[ ] Meets Definition of Done - https://github.com/SSWConsulting/SSW.Rewards.Mobile/blob/main/_docs/Definition-of-Done.md

Thanks!

Done - @AntPolkanov fixed this in #937


Adding tracking data (pose) to the plugin to allow 3d models to track to the Vuforia target

Expected Behavior

Having read issues #86 and #87 (similar questions but not exactly the same), this may not be possible, but it would be great to extend the plugin to be able to use tracking info (pose) rather than just registering when a target has been detected, so that I could enable three.js to add 3d models over images and Vumarks.
Although elements such as adding transparent webviews (if indeed that's the best way to go?) and passing tracking data to three.js are almost certainly outside the scope of this plugin, things such as adding the tracking data to the plugin are part of Vuforia image recognition.

Possible Solution

I'm not very experienced but I'm happy to have a go at doing this if no-one else has the time/inclination; but I'd really appreciate if someone could give me a few pointers on how to tackle it, to check I'm on the right track! I'm thinking:

add the Java tracking elements from the Vuforia SDK into the current cordova plugin
once Vuforia has detected a target, begin tracking, and also start three.js in a transparent Crosswalk webview above the camera
pass the tracking info from Vuforia in the camera activity to the transparent Crosswalk webview above it and allow three.js to put a 3d model above the Vuforia target.

Thanks!

Hi Flikker,
Thanks very much for taking the time to create this issue :) unfortunately this isn't possible with the current plugin - how it works at the moment is:

Cordova triggers the plugin
A new ViewController is created and launched

This view controller uses the native Vuforia SDK to monitor for targets
The user find a target, or triggers a close

The ViewController returns an object back to cordova

All of the indented points are effectively seperate and andboxed from the cordova part of your application. Launching the plugin effectively opens a mini-vuforia app of it's own. You're able to send messages in via a simple interface, then you just wait for a response. Unfortunately the vuforia side of your application is very opaque - there's no way to add elements on top using the cordova environment, this is why any additional rendering etc has to be done on a system-by-system basis.
For example, i'm not 100% on how I would pass an image from Cordova through the sandbox and into the Vuforia ViewController that could then be displayed, or a video for that matter.
I'm sorry this probably wasn't the answer you hoped for - you could always take my plugin and extend it to allow some more specific behaviors on the Vuforia side?


fix(runtime-core): TypeError when using html comment in teleport(fix #10747)

close #10747

@edison1105 I think assuming that all comments should be removed in production mode, then there shouldn't create vnodes for comments in production mode for performance reason. However, comment vnodes are created, also removed by nodeOpt.remove method during unmounting, except for traversStaticChildren, which lacks consistency. Indeed this inconsistency in handling caused the exception.

@tonliver Could you add a test case?

@tonliver Could you add a test case?

Sure, I'll work on it as soon as possible.

@edison1105 I've added some test cases.

@yyx990803 What else can I do?


Lights stop after about 20min

Possibly a memory leak.

Might need to lift WiFiClient from displayMessage.

Seems to be this; Serial.print(ESP.getFreeHeap()); reports smaller and smaller heap size each loop.

Resolved by moving jsonBuffer into loop().


I would like to join the org

Hello everyone 👋

Welcome.  The invite is now sent 👍

Thanks @jbampton


Bug Report: Incorrect Error Message for Month Field in Monthly Check-In Submission Form

Describe the bug
A clear and concise description of what the bug is.
In the new Monthly Check-In submission form, the first input field is labeled "Month." However, when the user attempts to submit an empty form, the field displays the error message "Title is required." This message should instead display "Month is required."
To Reproduce
Steps to reproduce the behavior:

Go to the Mentee Dashboard
Click on the "Submit Your Monthly Check-In" button.
Scroll to the bottom and click Submit without entering any data.
Observe the error message.

Expected behavior
A clear and concise description of what you expected to happen.
The error message "Title is required" should instead display "Month is required."
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Linux Mint 21.3
Browser: Chromium
Version: 129.0.6668.100

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Are you working on it?
No

I'll work on this


Upgrade svnkit to 1.8.12

@dsyer @ryanjbaxter @dussab Should we move svn support to a separate project because of licensing?


[Docs]: Update SSL & Custom Domain

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
n/a
Discord/slack/intercom Link
n/a
Describe the problem and improvement.
Update the SSL & Custom Domain

@harshilp24 - Is this duplicate?


Some SEO suggestions

[ ] Verify Domain in github and add to various repositories -> https://open-neuromorphic.org/
[ ] Link to socials at https://github.com/open-neuromorphic
https://open-neuromorphic.org/
https://discord.gg/C9bzWgNmqk
https://www.linkedin.com/groups/9267873
https://www.youtube.com/@openneuromorphic
[x] Optimize title & description at https://open-neuromorphic.org/
(Suggestion) Discover the Future of Computing with Open Neuromorphic: Your Hub for Open-Source Projects in Neuromorphic Research
[x] Include '- Open Neuromorphic' in page titles example: https://open-neuromorphic.org/p/snn-library-benchmarks/

Strategy wise, I would recommend targeting some keywords, 'neuromorphic computing', 'neuromorphic ai'. A good way to do this is through data architecture, possibly via the tagging mechanism.

Data structure like https://open-neuromorphic.org/neuromorphic-computing/articles faqs etc
Left column of site navigation should feature links that lead to these content sections.

I am open to help & collaboration! These are some small steps that can help keep things trending in the right direction. There is a ton more that is possible.
I added Open Neuromorphic to https://aimodels.org/ai-community-directory . I've recently started an article on Neuromorphic computing: https://aimodels.org/neuromorphic-computing and developing this page and subject further is what brought me here. :)

super nice, thanks for that! Would you mind opening some pull requests for that? Would definitely help to get better exposure

@biphasic Good suggestion - I've spent the last week studying Hugo and migrating an existing site to use it. I should be able to get some pull requests up once I finish the migration.

For the first point, the domain is verified for the ONM organization profile, but it says 'DNS check in progress' in the settings for the open-neuromorphic.github.io repo. Should I change this?

This is to look more at https://github.com/organizations/open-neuromorphic/settings/profile
This will have a number of fields (my github as example):

This is a likely landing page (second result for Open Neuromorphic), so it would be good to have at least Discord and Website on the org landing page.

done!

It doesn't show verified, though it's probably not that important. It's something I thought could potentially be a signal somewhere, but it's just a wild guess on my part. We can mark this complete! Looks good!

anoyher fyi: You might want to look into

And see if you can get neuromorphic * topics in there too. Again, it's probably not that important, but could be a signal somewhere.


RVM not being installed, wrong URL

The URL for RVM installer has changed. This pull fixes the url used in vcap/setup/install.
I have successfully installed using this change.

Sweet, we will pull this in soon.


Remove timeout loop in FluentConnection

~trying to understand the impact~
Looking at how we are launching Fluent from PyFluent, the timeout loop in FluentConnection is unnecessary. I have replaced it with a check_health call which will fail only if there is some issue at grpc communication level and it will expose the grpc error message in that case. Exposing the grpc error message here will help us to triage issues in user's system.
The start_timeout is used only when we wait for the server to finish writing the server-info file.
Tested by hardcoding a port in FluentConnection, so grpc connection will fail due to port-mismatch:
Old behaviour (fails after 60 s):
>>> solver = pyfluent.launch_fluent()
pyfluent.launcher ERROR: Exception caught - RuntimeError: The connection to the Fluent server could not be established within the configurable 60 second time limit.
Traceback (most recent call last):
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\launcher\launcher.py", line 750, in launch_fluent
    session = new_session.create_from_server_info_file(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\session.py", line 226, in create_from_server_info_file
    fluent_connection=FluentConnection(
                      ^^^^^^^^^^^^^^^^^
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\fluent_connection.py", line 283, in __init__
    raise RuntimeError(
RuntimeError: The connection to the Fluent server could not be established within the configurable 60 second time limit.

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

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\launcher\launcher.py", line 784, in launch_fluent
    raise LaunchFluentError(launch_cmd) from ex
ansys.fluent.core.launcher.launcher.LaunchFluentError:
Fluent Launch string: start "" "D:\ANSYSDev\vNNN\fluent\ntbin\win64\fluent.exe" 3ddp   -sifile="C:\Users\mkundu\AppData\Local\Temp\serverinfo-1aqp0o22.txt" -nm -hidden

New behaviour (fails immediately after Fluent is launched and shows the grpc error message):
>>> solver = pyfluent.launch_fluent()
pyfluent.launcher ERROR: Exception caught - RuntimeError: failed to connect to all addresses; last error: UNAVAILABLE: ipv6:%5B::1%5D:12345: Connection refused
Traceback (most recent call last):
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\launcher\launcher.py", line 745, in launch_fluent
    session = new_session.create_from_server_info_file(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\session.py", line 226, in create_from_server_info_file
    fluent_connection=FluentConnection(
                      ^^^^^^^^^^^^^^^^^
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\fluent_connection.py", line 272, in __init__
    self.health_check_service.check_health()
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\services\error_handler.py", line 15, in func
    raise RuntimeError(ex.details()) from None
RuntimeError: failed to connect to all addresses; last error: UNAVAILABLE: ipv6:%5B::1%5D:12345: Connection refused

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

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\ANSYSDev\PyFluentDev\pyfluent\src\ansys\fluent\core\launcher\launcher.py", line 779, in launch_fluent
    raise LaunchFluentError(launch_cmd) from ex
ansys.fluent.core.launcher.launcher.LaunchFluentError:
Fluent Launch string: start "" "D:\ANSYSDev\vNNN\fluent\ntbin\win64\fluent.exe" 3ddp   -sifile="C:\Users\mkundu\AppData\Local\Temp\serverinfo-z5hlusk9.txt" -nm -hidden

This is a great idea, but the test_recover_grpc_error_from_launch_error is going to leave unreachable zombies behind (watchdog also not going to work due to incorrect port number), and I have an idea of how to make this test work with containers as well, and clean up after itself so it doesn't leave zombies, going to see if I can make some suggestions
Edit: if you prefer, can merge this without the test for now, and then we can add more robust tests in a separate PR

Thanks @raph-luc, this is a good point. Feel free to merge your suggestions directly to this branch if you prefer. We'll merge the PR sometime next week.

Thanks @raph-luc, I've kept both tests. Both are useful examples of how to recover the grpc error in those scenarios.


fix(chips): remove button not accessible via keyboard

Fixes the user not being able to reach the chip remove button using the keyboard. Now the button will become tabbable once the chip around it is focused.
Fixes the remove button not doing anything on enter key presses.

Fixes #11896.

This was intentional; for a11y purposes, chips are removed via the delete key. I think that making the button show up for screen-reader uses would just make the interaction noisier.

In that case how should we handle #11896? Also I'm not sure whether there's any indication to the user that a chip is remove-able.

One thing we could do it so add instructions to the Accessibility section in the docs that the labeling should communicate this somehow. AFAIK there's no common pattern for indicating something is "delete-able".

No longer relevant and will be handled in the MDC chips.


Add View

Creating a subimage / view shouldn't allocate, unless you manually copy metadata etc.

The subview ends up with markedly different layout constraints.

Should be easy to implement now


Fix Swiftlint hash

See #201, there was an issue when installing latest swiftlint:
$> bundle install && rake
...
Downloading<EMAIL_ADDRESS>/Users/thomas.joulin/src/danger-ruby-swiftlint/ext/swiftlint/downloadSwiftlint.sh -u https://github.com/realm/SwiftLint/releases/download/0.55.1/portable_swiftlint.zip -d /Users/thomas.joulin/src/danger-ruby-swiftlint/ext/swiftlint/bin -a portable_swiftlint.zip -dh 506b49a6d71512b25a511bca01065503
Zip was corrupted, try again later.

When downloading manually, hash was different, after updating hash installation works

How did you find it !. I tried several times :(
GG


Improve Documentation Across All Classes and Functions

There are several classes and functions in our codebase that lack sufficient documentation, particularly regarding their arguments. This issue aims to address and rectify this gap.
The goal is to ensure that all classes and modules in the project have comprehensive and clear documentation. The documentation should include, but not limited to, an overview of what each class or function does, descriptions of all arguments and their expected types, and what the function returns if applicable.
This task will be considered successful when every class and function in the project is properly documented, enhancing readability and maintainability of our codebase.

Closing since this issue was for release v0.0.2.


Private method of Environment class

The method Environment._r is often used by other related classes holding an instance of the Environment object.
Accessing a private member of a class violates the SLF rule in Ruff (and Flake8): https://docs.astral.sh/ruff/rules/private-member-access/
I would suggest adding a public method r(...), or maybe more descriptive calc_source_mic_distances(...), in the environment class that calls the private method.

The member being private was introduced in %17bcf14 for a reason, I guess? @gherold

IIRC this was changed since the r trait does not always correspond to actual physical distances (as users may expect) but rather to the lengths of sound travel paths. As it was (is) unclear if and how the public interface of the Environment class should change to be more clear and versatile, the trait was set private at the time.
I agree that we should have a public trait here. Currently _r is the most important part of the Environment class, so we could simply make it public again and document its purpose better. Alternatively, we could derive a sound_travel_time trait from _r and c and use this in further processing. This would be more intuitive, e.g. for simple delay-and-sum beamforming, but would require significant changes in the code.

How about changing quickly to apparent_r ?


Add vpc-azs() & vpc-az-count()

Return info about AvailabilityZones for subnets of vpc(s)

Will recreate this PR as this one is showing things already present in master


Luigi doesn't output FAILURE processing time, end time and start time

When a task is failed the processing time is not recorded. As a result, the processing_time of a process left as initialized value.
{'job_id':'1479211214', 'datetime':'2016-11-15 04:12:44', 'task_id':'AdobeLoad_2016_11_14_page_summary_5281aeca7d', 'task_name':'AdobeLoad', 'task_date':'2016-11-14','task_report_name':'page_summary', 'start_time':'1479211963.7', 'end_time':'1479211964.12', 'process_time':'None', 'status':'FAILURE'}
{'job_id':'1479229552', 'datetime':'2016-11-15 09:07:09', 'task_id':'AdobeLoad_2016_11_14_page_summary_5281aeca7d', 'task_name':'AdobeLoad', 'task_date':'2016-11-14','task_report_name':'page_summary', 'start_time':'1479229554.92', 'end_time':'1479229629.66', 'process_time':'74.7349998951', 'status':'SUCCESS'}

Processing time for Failure events shouldn't be None or 0, therefore processing_time for FAILURE event should be calculated before exits.
Luigi let failed tasks have no processing time information. As a result, it could be hard to track failures. I think time information should be added to failed tasks.

What are does dicts? Did you create them yourself or is it a luigi future?

@Tarrasch is just an output not by luigi


COMPAT: Return expected response for MaxUpload and Delimiter params

Currently, the value of MaxUploads for listMultipartUpload API is
the number of MPU in a bucket. This is not the behavior of AWS. This
commit changes the API's behavior so the value of MaxUploads can be
specified by the user using the MaxUploads parameter. This commit also
returns the expected default of 1000 if no MaxUploads is defined.
If MaxUploads is defined as 0, the behavior of AWS is to return an
empty Uploads array and set IsTruncated to false. This commit also
adds this expected behavior.
Currently, when using the file-backend, if a Delimiter parameter is
defined and it does not result in a CommonPrefixes array that is
greater than 0, Delimiter is set to ''. This causes the XML response
to omit the Delimeter key. For compatibility with AWS, this commit
tests an associated change in Arsenal that fixes the Delimiter response:
https://github.com/scality/Arsenal/pull/197
A new test that directly compares the expected object has been added to
verify these behaviors.

@ironman-machine

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "dev/BF/handle_MaxUpload_values_in_listMPU", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6614 for CI status.

@ironman-machine with correct Arsenal branch
SCALITY_ARSENAL_BRANCH=dev/COMPAT/handle_delimter_multipartUploads

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "dev/BF/handle_MaxUpload_values_in_listMPU", "SCALITY_ARSENAL_BRANCH": "dev/COMPAT/handle_delimter_multipartUploads", "REPO_NAME": "S3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6615 for CI status.

@ironman-machine after fix of Resolution method is overspecified error
SCALITY_ARSENAL_BRANCH=dev/COMPAT/handle_delimter_multipartUploads

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "dev/BF/handle_MaxUpload_values_in_listMPU", "SCALITY_ARSENAL_BRANCH": "dev/COMPAT/handle_delimter_multipartUploads", "REPO_NAME": "S3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6616 for CI status.

@ironman-machine
SCALITY_ARSENAL_BRANCH=dev/COMPAT/handle_delimter_multipartUploads

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "dev/BF/handle_MaxUpload_values_in_listMPU", "SCALITY_ARSENAL_BRANCH": "dev/COMPAT/handle_delimter_multipartUploads", "REPO_NAME": "S3"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6795 for CI status.

This can be rebased and drop me arsenal dep. commit can be removed as the PR has been merged.

@ironman-machine after rebase and dropped dependency on Arsenal branch

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "dev/BF/handle_MaxUpload_values_in_listMPU", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6893 for CI status.

Re-triggered end-to-end and passed here.

@ironman-machine after rebase

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "dev/BF/handle_MaxUpload_values_in_listMPU", "DEFAULT_BRANCH": "master"}
Please follow http://ci.ironmann.io/gh/scality/Integration/7011 for CI status.


EXEC : [ERR!] error : Unexpected token   [ERR!] CODE : undefined

Hi,
I am getting the following exception while rebuilding the project from Visual Studio 2015. Could you have a look and let know what could be the possible cause.
tsd-debug log file : tsd-debug.txt
npm WARN<EMAIL_ADDRESS>No license field.
Unexpected token   [ERR!] cwd  : C:\Common\DEV\DEV2
ERR!] os   : Windows_NT 6.1.7601
[ERR!] argv : "C:\Program Files\nodejs\node.exe" "C:\Users\RakeshC\AppData\Roaming\npm\node_modules\tsd\build\cli.js" "install"
[ERR!] node : v4.4.2
[ERR!] tsd  : 0.6.5
EXEC : [ERR!] error : Unexpected token   [ERR!] CODE : undefined
[ERR!] Unexpected token EXEC : [ERR!] If you need help, you may report this error at:
https://github.com/DefinitelyTyped/tsd/issues
[ERR!] .tsdrc could not be retrieved
[ERR!] Please include the following file with any support request:
C:\Common\DEV\DEV2\tsd-debug.log
C:\Common\DEV\DEV2\DEV.Client.csproj(271,5): error MSB3073: The command "tsd install" exited with code 1.

Apparently, clearing local cache fixed the issue :)
ran tsd purge


Use std::launder when converting unsigned char* to T*

See the impl here: https://en.cppreference.com/w/cpp/types/aligned_storage
Which references: https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2016/p0137r1.html
Also.. change from char[N] to unsigned char[N].
This needs to happen in the Iterator type erasure code: https://github.com/chromium/subspace/blob/main/subspace/iter/sized_iterator.h#L80

https://github.com/chromium/subspace/commit/f667a462c2ced418a76ba62b722d378a99a62b66


Spike: figure out a path forward for misconfigured KongPlugin resources

Users always run into issues with malformed plugin configurations which brings down the entire reconciliation loop.
Figure out a way to avoid this from happening.
One are to explore will be to use Kong's /schema endpoint a lot more to verify if each KongPlugin is valid or not.
This will need to be cached to avoid performance issues when the network latency between the controller and Kong is significant (> 10s of milliseconds)

We have run into this problem many times. Our entire cluster goes down since the gateway fails if one plugin is misconfigured. This is a very concerning issue to us. We may drop Kong altogether because of it.

Are there any plans to address this issue?
Is there some kind of work around such as validating the plugin config in a CICD pipeline?

Please configure and install validating webhook: https://docs.konghq.com/kubernetes-ingress-controller/2.1.x/deployment/admission-webhook/#main
That will get rid of the problem in most cases.

That's great thanks for the quick response we will look in to this.

In #2195 we are addressing this problem holistically and attempting to do it in ways that wont technically require the validating webhook (however, please do install it). We're going to close this in favor of #2195, please subscribe to that issue to track the progress of improvements in this regard.


The '-i' alias for the '--connection-id' flag on the send message command not working

Describe the bug
The '-i' alias for the '--connection-id' flag on the send message command does not work
To Reproduce
Steps to reproduce the behavior:

Run the 'agent-cli message -i <> -m <>'

Expected behavior
Message should be sent
Aries CLI version (run aries-cli --version)
0.3.0
Desktop OS

OS: Ubuntu
Version: 20.04

Verbose logs
Run your command with the -vvv flag. See aries-cli --help for guidance.

Logs
< error: Found argument '--i' which wasn't expected, or isn't valid in this context

  If you tried to supply `--i` as a value rather than a flag, use `-- --i`

USAGE:
  agent-cli message --connection-id <CONNECTION_ID> --message <MESSAGE>

For more information try --help
>

Screenshots, screen recordings
If applicable.
Additional context
I think the issue is that the alias has been changed or removed, but this has not been reflected in the docs yet.

Seems it is indeed not working.
@blu3beri is the docs released from the main branch currently? Would be nice if you can update docs whenever breaking changes are made / feature are added, and that those will be released with the next version

Ah my apologies. The shorthand is -c.
I will have to check how we want to deploy the docs (based on a GitHub release or on push to main).

Okay!

Should I close the issue? Or leave it for the doc fix?

Should I close the issue? Or leave it for the doc fix?

Let's leave it for the doc fix to close it :).


[FLINK-33402] 2 CONTINUED

Continued from https://github.com/apache/flink/pull/23687

@tweise
Continuing this investigation. I tried reader only and the error did not go away
Looking at the logs again I still see the concurrency bug as reported in the original Jira thread.
We see the JM (Enumerator) send 'No More Splits event'

And shortly after the TM acts on this event and switches to the next source:

What I wonder is what is causing the reader to prematurely close? Synchronization seems to fix it.

@tweise requesting your review on this. I added additional details above ^


ship seedme python module with Visit 2.9

For the 2.9 release, we should ship with seedme support: http://seedme.org/
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2058
Status: Resolved
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: ship seedme python module with Visit 2.9
Assigned to: Cyrus Harrison
Category: -
Target version: 2.9
Author: Cyrus Harrison
Start: 11/10/2014
Due date:
% Done: 0%
Estimated time:
Created: 11/11/2014 01:39 pm
Updated: 03/10/2015 03:05 pm
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 3 - Medium
Expected Use: 3 - Occasional
OS: All
Support Group: Any
Description:
For the 2.9 release, we should ship with seedme support: http://seedme.org/
Comments:
Added the requests seedme python modules & support to build them with build_visitRC:Adding (bin) third_party/requests2.5.1.tar.gzTransmitting file data .Committed revision r25448.Adding (bin) third_party/seedmepythonclientcb1f3c409788.tar.gzTransmitting file data .Committed revision r25449.Sending svn_bin/bv_support/bv_python.shSending svn_bin/bv_support/helper_funcs.shTransmitting file data ..Committed revision r25452.Trunk:Sending svn_bin/bv_support/bv_python.shSending svn_bin/bv_support/helper_funcs.shTransmitting file data ..Committed revision r25454.Adding (bin) third_party/requests2.5.1.tar.gzAdding (bin) third_party/seedmepythonclient-cb1f3c409788.tar.gzCommitted revision r25455.-Cyrus Hi Everyone,*Added the Seedme python module to VisIts CLIRC:Sending src/svn_bin/bv_support/bv_python.shTransmitting file data .Committed revision r25526Deleting third_party/seedmepythonclientcb1f3c409788.tar.gzAdding (bin) third_party/seedmeseedmepythonclientc8818385bc88.tar.gzTransmitting file data .Committed revision r25527.Trunk:Sending svn_bin/bv_support/bv_python.shTransmitting file data .Committed revision r25529.Deleting third_party/seedmepythonclientcb1f3c409788.tar.gzAdding (bin) third_party/seedmeseedmepythonclientc8818385bc88.tar.gzCommitted revision r25530.-Cyrus Hi Everyone,Updated the version of seedme we will use for the 2.9 releaseRC:Sending src/svn_bin/bv_support/bv_python.shTransmitting file data .Committed revision r25782.Adding (bin) third_party/seedmeseedmepythonclient0a365da9c861.tar.gzTransmitting file data .Committed revision r25783.Trunk:Sending resources/help/en_US/relnotes2.9.0.htmlSending svn_bin/bv_support/bv_python.shTransmitting file data ..Committed revision r25787.Adding (bin) third_party/seedmeseedmepythonclient0a365da9c861.tar.gzCommitted revision r25786.PS: It looks like I picked up someones release notes changes.-Cyrus

2064_smime.p7s.zip


More sld tests

Coverage remained the same at 70.262% when pulling 5d508822bcdeb181c7cf4b10766c71e6c7800ebb on more-sld-tests3 into f4dd01ee59c026b9d743fecbe3fa2a3226a6b0f5 on master.

Coverage remained the same at 70.262% when pulling 5d508822bcdeb181c7cf4b10766c71e6c7800ebb on more-sld-tests3 into f4dd01ee59c026b9d743fecbe3fa2a3226a6b0f5 on master.

Coverage increased (+0.03%) to 70.298% when pulling 52fd0a6e8a969cf4519e2691bcb47ad5e93453bc on more-sld-tests3 into f4dd01ee59c026b9d743fecbe3fa2a3226a6b0f5 on master.

Coverage increased (+0.03%) to 70.335% when pulling 69195a368b0d1635da31794b87190e2b37bea04f on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.03%) to 70.335% when pulling b87b5d0573bd1586c64afc8a638d58c64477687e on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.05%) to 70.354% when pulling 1b4b424a7c8dcc023153495191086cf5ae697b7f on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.05%) to 70.354% when pulling 1b4b424a7c8dcc023153495191086cf5ae697b7f on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.05%) to 70.354% when pulling 676bdee01639c0e9de3cb993ebaca1df2f3c7101 on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.05%) to 70.354% when pulling cc154c82f590f646e570bbec204b2649f04aaf3f on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.05%) to 70.354% when pulling cc154c82f590f646e570bbec204b2649f04aaf3f on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.

Coverage increased (+0.3%) to 70.551% when pulling 65ff7a5363f09e4d5c4420ca9bb7964a0f240736 on more-sld-tests3 into 36bda0948ea78517a6d92e8a5a859e6f563ff14f on master.


Show QR code when using the --host flag

Describe the problem
When exposing the server via the --host flag, the URL shown needs to be typed into your phone/tablet's browser.
Describe the proposed solution
It would be convenient if a QR code is displayed on the terminal that can be scanned by your phone! Especially for people who don't have a Mac-iPhone combo.
Alternatives considered
No response
Importance
nice to have
Additional Information
I'm willing to work on this feature.

@benmccann correct me if I'm wrong but if this is implemented, I think this needs to be implemented in Vite.

An immediate alternative would be to open the URL in Chrome which shows you the QR code for that URL.
Or, you can search for "qr " in DDG and it will generate what you need.
E.g: https://duckduckgo.com/?q=qr+<IP_ADDRESS>&t=hx&va=g&ia=answer

Actually this is implemented in Kit:
https://github.com/sveltejs/kit/blob/5d477916d76e03690f73fea87379331c6bf3db22/packages/kit/src/cli.js#L223
Vite also prints a duplicate log message, but that will go away shortly with Vite 2.6.
I think the bigger question is whether we want to add this. I've never loaded anything on my phone during development and just use Chrome's mobile phone emulator instead

Well, it wouldn't be localhost:3000, it would be whatever the IP on your network is for your host machine, which is probably more of a pain to transcribe.

can't you just copypaste the cli url into a random qrcode generator and then bookmark the scanned url on the phone? I'd rather not spam everyone's cli with a qrcode relevant to some.
Another way to implement this could be a third-party vite plugin that shows the qrcode on a virtual page and logs a link to that page during startup

Made a POC for a Vite plugin that show the QR code in the terminal. Might turn it into an actual npm package but I haven't got the host resolve to match Vite's implementation:
import qr from 'qrcode-terminal'

const plugin = {
  name: 'qrcode',
  configureServer(server) {
    const host = server.config.server.host
    if (host === 'localhost') return
    if (server.httpServer?.listen) {
      const _listen = server.httpServer.listen
      const protocol = server.config.server.https ? 'https' : 'http'
      let timer
      server.httpServer.listen = function () {
        const port = arguments[0] || port || 3000
        const link = `${protocol}://${host}:${port}`
        clearTimeout(timer)
        timer = setTimeout(() => {
          qr.generate(link, { small: true }, (result) => {
            console.log('  Visit page on mobile:')
            console.log('  ' + result.replace(/\n/g, '\n  '))
          })
        }, 0)
        return _listen.apply(this, arguments)
      }
    }
  }
}

Do you need to override listen? Or could you do it after the server was started and then call .address() to get the details?

I wouldn't want --host to show a QR code by default, but I wouldn't object at all to a --qr option to show a QR code.

@dominikg and I have published https://www.npmjs.com/package/vite-plugin-qrcode. Add it to config.kit.vite.plugins as [qrcode()] and a QR code will be shown when you set --host. Closing in favor of a userland solution.


Baseline for BUG: datawc does not work on a time axis.

@danlipsa thanks for merging

@doutriaux1 glad to help :smile:


[Bug]: unable to use proper tsconfig with ESM

Version
29.0.2
Steps to reproduce

https://codesandbox.io/s/purple-night-6bj7kx
open console
type yarn test

Expected behavior
test should work
Actual behavior
jest throwns an error
Top-level 'await' expressions are only allowed when the 'module' option is set to 'es2022', 'esnext', 'system', 'node16', or 'nodenext', and the 'target' option is set to 'es2017' or higher.

Debug log
{"context":{"allowJs":false,"logLevel":20,"namespace":"jest-preset","package":"ts-jest","version":"29.0.2"},"message":"creating jest presets not handling JavaScript files","sequence":1,"time":"2022-09-25T19:37:16.570Z"}
{"context":{"allowJs":false,"logLevel":20,"namespace":"jest-preset","package":"ts-jest","version":"29.0.2"},"message":"creating jest presets not handling JavaScript files","sequence":2,"time":"2022-09-25T19:37:16.596Z"}
{"context":{"logLevel":20,"namespace":"Importer","package":"ts-jest","version":"29.0.2"},"message":"creating Importer singleton","sequence":3,"time":"2022-09-25T19:37:17.718Z"}

Additional context
I know, there were similar issues, but according to them, the problem was solved by jest@29. Maybe my setup is a bit wrong here?
Environment
System:
    OS: Linux 5.13 Debian GNU/Linux 10 (buster) 10 (buster)
    CPU: (8) x64 Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
  Binaries:
    Node: 16.15.1 - ~/.nvm/versions/node/v16.15.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.15.1/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.15.1/bin/npm
  npmPackages:
    jest: 29.0.3 => 29.0.3

Fixed in #3833 and will be included in the next patch release


Integrate clearml logger

Blank

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Please run the make commands here to make sure your code is formatted correctly. There is for example at east 1 line that is too long: https://github.com/huggingface/transformers/blob/main/CONTRIBUTING.md#create-a-pull-request

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Add Eager reference to tools

Thanks, but this looks like very general, and not Hugo specific. We haven't got enough room for everything remotely related.


Explicit Nashorn dependency

Adding Nashorn dependency allows to upgrade JDK beyond version 11.

@zguo Please, review.


Deploying to AWS (serverless)

I tried to search around but did not successfully find a AWS serverless solution for Nuxt SSR. Is there something i have missed or is this not currently support?

This question is available on Nuxt.js community (#c1717)

I made this: https://github.com/phlogisticfugu/perfect-paca a while ago.
Waiting for Nuxt 1.0 to update it, and use the new vue-cli

@phlogisticfugu very interesting.. Would it make more sense to spin something up with Claudia.JS?

perfect-paca uses https://serverless.com/ actually, which does what claudia.js does, but supports more clouds.  Give it a try and I'm open to suggestions/PRs

@phlogisticfugu that looks like a better solution. I'm comparing that with the AWS codestar pipeline this week and will try to give feedback.

Hi, what is the status of this?
It would be very useful to me if I can deploy to AWS lambda and also cache pages on S3 or some other cache for frequently accessed pages.
Are there any other solutions for this?

I basically ended up using a pre-rendering solution with my nuxt project. But perfect-paca looks like a good solution.

Ok I'm starting to use perfect-paca, are there any other alternatives? well I'm go ahead with this and report back how it goes.

https://github.com/phlogisticfugu/perfect-paca/issues/3

a little late to the game @maziarz but if you don't have dynamic routes and want to go the route of s3+CloudFront I built a tool that auto-deploys Nuxt projects and easily plugs into Circle CI
https://github.com/acidjazz/aeonian

I am still looking for a working example utilising Express or Koa running on AWS Lambda / ApiG without serverless-framework, but merely CloudFormation/SAM. If anyone is up for creating such a sample project, please ping me :-)


git clone recursive

It might be useful to do a recursive clone here https://github.com/binder-project/binder/blob/master/binder/app.py#L103. This would allow a single git repo to aggregate notebooks from many other repos into a single binder image by including them as git submodules.

Fixed in PR #36


Added slack support for appveyor

We have a slack channel? :confused:

Yeah, I'm still mainly on Jabbr, but there is an auto inviter in the topic on Jabbr if you want to get an invite.

:shipit:


Add slide direction and overlay options

It would be nice to add different slide directions as required e.g.:
slideDirection: 'top' // left, bottom, right

As well as add an overlay (a little like a lightbox would do) for off-canvas hide-show elements e.g.:
overlayEnabled: false // true

It might be best to keep the overlay option out of the plugin - instead adding some additional callbacks that trigger on open/close. That way we have more flexibility in the future to do overlays or any other kind of DOM manipulation.

Let's separate out the overlay option in another ticket.
The slide direction enhancement has been completed and a pull request made: https://github.com/nomensa/jquery.hide-show/pull/21

For the slide direction, it will be best to use the open/close callbacks to be added. This will allow you to customise what happens. You could also update the 'speed' to allow 'none' or '0' in which case, the slideDown|Up will ignored.

I've removed the slide direction enhancement and added a callback at the end of the animation

This is ready for Peer Review #21


Http proxy redirect (not following) occasional test failure fix

I've pulled in this related PR into this one (they touch same code and work on very similar issues)


inlined_vector<unique_ptr> doesn't work

Most member functions of inlined_vector need to be disabled for non-copyable types like unique_ptr.

Fixed


IPyWidgets not rendered when switching renderers

Create a cell with the following code

import ipywidgets as widgets
widgets.IntSlider()

Run the cell
Confirm slider is displayed
Switch renderer to plain text
Switch back to widget renderer & nothing is displayed.

We should be able to re-render the widget as we have all of the information.
We need to account for the fact that the widget was disposed & it needs to be re-rendered.

Changing to debt as no one has reported this as fixing this is a good architectural cleanup (i.e. debt)

This can be fixed with IPyWidgets 8,
However closing this issue as its unlikely anyone would change renderers
We haven't had users reoprting this (except internally for those testing renderers)


Preserve file paths

@alexeyxo probably best to review this commit-by-commit. The last commit is a cleanup that removes a ton of duplicated files (so the diff is enormous).

link to the commits with whitespace removed:
https://github.com/tamird/protobuf-swift/commit/18aed768d69c3c7f9b64069be9dda684a3f5554e?w=1
https://github.com/tamird/protobuf-swift/commit/9fa0c06f53a272e6b70ace7a95fb58401552d825?w=1
https://github.com/tamird/protobuf-swift/commit/66584eb714112d4630458e0576ea942c46a347db?w=1

I can not take this pull reguest.
Not correctly named class "package_name + class name".
Shall be as follows:
package = "google.protobuf"
message TestA {

}

internal struct Google
{
     internal struct Protobuf
     {
          internal class TestA {}
     }
}

In other proto files:
extension Google.Protobuf
{
      internal class TestB {}
}

Make a separate pull request with corrections for # 24

https://github.com/tamird/protobuf-swift/commit/18aed768d69c3c7f9b64069be9dda684a3f5554e
I agree with this fix.

@alexeyxo yes, the behaviour in this PR is not correct, but neither is the behaviour prior to this PR (no wrapping internal structs are ever generated. Perhaps you can help make the other necessary changes?

I started doing.
https://github.com/alexeyxo/protobuf-swift/commit/758289504a3bb0c1a21831771f6688f8b06842ba https://github.com/alexeyxo/protobuf-swift/commit/ccd779e2ac501d2f7dd3d49bec42aeffe0dcd23e https://github.com/alexeyxo/protobuf-swift/commit/414bc5ce7587edaf522fda010a3820480715ad35
I will do it at weekend.

after rebasing this on top of #30 i can get things to compile!

@alexeyxo turns out my filename fix isn't sufficient because the swift compiler does this:
...
<unknown>:0: error: filename "Verify.pb.swift" used twice: '/foo/Verify.pb.swift' and '/bar/Verify.pb.swift'
<unknown>:0: note: filenames are used to distinguish private declarations with the same name
Command /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swiftc failed with exit code 1


Drawing walls in Chrome not working

Hi
I can draw walls in Edge, but in Chrome Version 62.0.3202.94 (Official Build) (64-bit) its not working.

+1


Addition of Social Media Handles to the website.

LinkedIN
Instagram
GitHUB
Links will be updated when the handles are created. Till then create a dummy footer.

Will reopen it in some time.


DRILL-4823: Fix OOM while trying to prune partitions with reasonable …

…data size

@jinfengni , could you please review?

+1   Could possibly extend the same use of the ConstantValueHolderCache for visitor methods of other types; though this would make the code more cluttered.

@jinfengni  could you also take a quick look at this since the code change is in the Interpreter.

+1
The revised patch looks good to me.  @arina-ielchiieva , thanks for your pull request!


Release

Remove duplicate jobs #43

Travis tests have failed
Hey @bluzi,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
Node.js: 8

     npm run lint

><EMAIL_ADDRESS>lint /home/travis/build/bluzi/travis-buddy
> eslint .

/home/travis/build/bluzi/travis-buddy/handlers/failure.handler.js
  24:3  error  Unexpected 'debugger' statement                                                                                no-debugger
  25:3  error  The body of a for-in should be wrapped in an if statement to filter unwanted properties from the prototype                                                             guard-for-in
  25:3  error  for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array  no-restricted-syntax

✖ 3 problems (3 errors, 0 warnings)
  1 error, 0 warnings potentially fixable with the `--fix` option.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>lint: `eslint .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!


Fix: IO Exception on closed file

About this PR:
While running a query. with RESULT_STORE_TYPE as file, the execution completes but shows below error while displaying result.
I/O exception opening closed file

So as a fix, created a PR.

Hi @jczhong84,
Can you please have a look at this PR too?

Thanks for the fix!
please fix the lint errors

@najma-kunju there is still lint failure

@jczhong84, have fixed the current lint error


[WIP]Zoom centered to the cursor position

Do not merge yet!
Zoom centered to the mouse cursor similar to https://stackoverflow.com/questions/30002361/image-zoom-centered-on-mouse-position.

Suggestion for this; completely split zoom vs draw as a toggle
I find myself accidentally zooming while trying to prepare an ignore area


Allow undefined borgbackup_servers and borgbackup_management groups.

When there is no group defined, the conditional to find a value in a list is broken and passes as true.
This PR sets facts to ensure that undefined group lists are regarded as empty.
Without this change, the hosts groups would need to be defined as empty groups:
[borgbackup_servers]
# Empty...

[borgbackup_management]
# Empty...

Thanks, I missed that indeed.


Upgrade prometheus_client

Currently, yoshi pins the prometheus_client dependency to 0.5.0 which is more than 3yo. Latest is 0.14.
Having such an old version prevents the installation of other libs next to luigi in a conda env.

Yoshi? I suppose you mean luigi.
I have learnt the hard way that for python, it is preferable to specify ranges of dependency versions. I guess a newer version would also pass the test suite. May I suggest that you send a pull request with a range specification and a newer version in tox.ini? I guess that should be sufficient.

Closed by https://github.com/spotify/luigi/pull/3163


Wrap script/genkeys in Docker

script/genkeys is handy. We should put it in a Docker container so that we can share it more easily among different cloudpipe repositories.

This is done in :key: keymaster.


[cmake] Make dispatch/Foundation non required

When building using an install tree which already contains
dispatch/Foundation, a dispatch/foundation packages do not need to be
specified since they are already automatically linked against.

@swift-ci please test platform


carpenter unreported build failure

Describe the bug
carpenter action passes but output says:
2022-10-12T05:43:41Z	info		Missing pyproject.toml
2022-10-12T05:43:42Z	info		Failed requirements check, not building: arcaflow-plugin-template-python latest

I'm going to open an issue on carpenter for failing if not executed but if pyproject.toml is requried, we need to add it here too.

Reported on https://github.com/arcalot/arcaflow-plugin-image-builder/issues/11


Question - change pinMode of GpioPinDigitalMultipurpose

Hello,
Thanks for the great project.
I currently initialize all pins to PinMode.INPUT - but would dynamically light to change the mode depending on need to PinMode.OUTPUT.
I get
com.pi4j.io.gpio.exception.InvalidPinModeException: Invalid pin mode on pin [GPIO 13]; cannot setState() when pin mode is [input]
Is there a way to do this? By perhaps deprovisioning and re-provisioning ?
Thanks kindly.

This has been discussed in the V2-project with a reference to an implementation in V1:
https://github.com/Pi4J/pi4j-v2/issues/26