changed the image size of the card

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 0.46%. Comparing base (03869d8) to head (6dd8d5f).
Report is 5 commits behind head on main.

Files with missing lines
Patch %
Lines

src/components/Dashboard/DashboardMain.tsx
0.00%
2 Missing :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##            main     #36      +/-   ##
========================================
- Coverage   0.46%   0.46%   -0.01%
========================================
  Files         41      41
  Lines       3234    3237       +3
  Branches      42      42
========================================
  Hits          15      15
- Misses      3179    3182       +3
  Partials      40      40

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


ETN Icon

Hello & thanks for your contribution! Please refer to contributing guidelines for all info about PR submissions (just adding the SVG isn't enough). Thanks!


Cannot set port with :MPSetPort, :MPRepl drops to rshell

Thanks for making this plugin, it was exactly what I was looking for! A couple minor bugs:
Firstly I am running

Neovim NVIM v0.11.0-dev-1106+gf7e32fb6e6
Linux Mint 22 Cinnamon
Raspberry Pi Pico MicroPython v1.24.0 on 2024-10-25; Raspberry Pi Pico W with RP2040

When I run the Neovim command :MPSetPort the following is displayed:

There appears to be no ports listed to choose from
When I type the correct port (/dev/ttyACM0) and press enter, the window disappears and a No selection made message appears in the command window

When I run the bash command rshell -l I get the following output:
Serial Device: /dev/ttyS0
[ ... ]
Serial Device: /dev/ttyS31
USB Serial Device 2e8a:0005 with vendor 'MicroPython' serial 'e6614c311b712739' intf 'Board CDC' found @/dev/ttyACM0 *

The .ampy file created by :MPInit has a default port setting of dev/ttyUSB0. If I manually change this to /dev/ttyACM0 I am able to run the other commands successfully.
Sub-issue: :MPRepl drops me at an rshell prompt instead, but issuing the repl command gets to the REPL. Perhaps line 9 in /lua/micropython_nvim/repl.lua, which is currently:
local repl_assembled_command = string.format('rshell -p %s', _G['AMPY_PORT'])

Should be
local repl_assembled_command = string.format('rshell -p %s repl', _G['AMPY_PORT'])

And another function could be made for rshell?

Hey buddy,
Thanks for letting me know about these. I have fixed them both. I have made the the port filtering more flexible by filtering by
the following:
  local patterns = {
    '/dev/ttyUSB*', -- USB-to-Serial adapters
    '/dev/ttyACM*', -- Arduino and similar CDC devices
    '/dev/ttyS*', -- Hardware serial ports
    '/dev/tty.usbmodem*',
  }

Let me know if you have any other issues


Add IAM example to add users to a project

Supplemental stand-alone  example to add users to a project.

Can we start the eval/review for this?
/cc @krwenholz @jaivasanth-google

@adamharwayne who owns the internal bug for this.

Refactored to community folder; moved PR specific instructions to folder


corrupted double linked list

When attempting to install rbnacl-libsodium I get the following.
daniel@daniel-Latitude-D630:~/src/libsodium-1.0.1$ gem install rbnacl-libsodium
Building native extensions.  This could take a while...
*** Error in `/usr/bin/ld': corrupted double-linked list: 0x09abe5b0 ***
It then proceeds to behave as if the gem installed. However, when I "require 'rbnacl/libsodium'" I get a whole slew of problems:
daniel@daniel-Latitude-D630:~/Sandbox$ ruby Encrypt.rb
/home/daniel/.rubies/ruby-2.1.3/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require': cannot load such file -- rbnacl/libsodium (LoadError) 	from /home/daniel/.rubies/ruby-2.1.3/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require'
from Encrypt.rb:4:in `'
If it's any help, I'm using Ruby version 2.1.3 and I'm attempting to get this to work on lubuntu 14.04

This sounds like a bug in your version of ld


Allow a node to connect from both input and output.

Hi @jerosoler,
Can I create a node which connects from both input and output.

A node can connect to another node. Or to multiple nodes.
Nodes can be connected from outputs to inputs.
Nodes cannot be connected from inputs to outputs.
A node cannot connect to itself.
View from input to output:

https://github.com/jerosoler/Drawflow/issues/503


per-package-target doesn't seem to work?

Problem
I'm attempting to use the per-package-target unstable feature on nightly Cargo and it's being refused. Presumably I am holding it wrong, but I haven't been able to work out how.
Builds always fail with
error: failed to parse manifest at `/tmp/per-package-target/Cargo.toml`

Caused by:
  unknown cargo feature `per-package-target`

I've posted a minimal repro repo whose contents are extracted from Cargo's own test case for the feature. It does not build. (I have confirmed that the Cargo tests are, in fact, passing on Cargo's CI.)
I can only find one repo making use of the feature, kannader by the feature's author, and it also doesn't build with any nightly version I've tried from November to now -- their build is using nix to pull in a toolchain at a specific commit from somewhere odd, with a TODO.
Steps

git clone https://github.com/cbiffle/cargo-per-package-target-repro-case.git
cd cargo-per-package-target-repro-case
cargo build

Notes
Output of cargo version:
cargo 1.53.0-nightly (65d57e6f3 2021-04-04)
(pinned by rust-toolchain)

I'm going to close, as it should be available as of nightly-2021-04-28.  If you have any issues testing it, please feel free to open a new issue.


[WIP] Preload virtual field's uses only if it's not already included.

Fixes an issue seen in #5205 where:

a vm compare report had cpu_total_cores included via the hardware association:
cpu_total_cores is a virtual column on VmOrTemplate but not included in the report

db: VmOrTemplate
...

include:
...
  hardware:
    group: Properties
    columns:
    - cpu_total_cores
    - memory_mb
...

col_order:
- name
...
- hardware.cpu_total_cores

It's an edge case where we try to preload the virtual column's "uses" even though
it's already being preloaded via the include in the report.
This could also be fixed by changing the report to use the virtual column on VmOrTemplate:
db: VmOrTemplate
...

cols:
- cpu_total_cores
...

include:
...
  hardware:
    group: Properties
    columns:
    - memory_mb
...

col_order:
- name
...
- cpu_total_cores

cc @bdunne @Fryguy @matthewd
Note, I think the correct thing to do is change the report to use the virtual column but, it still feels like a bug in the code I changed above.  We should either warn people, "hey, there's a virtual column that you might be what you want to use" or skip the preload of the "uses" if they're already in the "include" as I did in this PR.
Thoughts?

If this is a bug, I can write a test that exposes it... it was a bit gnarly to do so I'd like to spend the time to simplify the test.

@jrafanie Since we're currently exposing the fact that some of these attributes live on the associated hardware, I think it's misleading to instead use the virtual column, don't you?  cc @gmcculloug

I agree with @bdunne.  Changing to use the virtual column on the VM would also move where the value is displayed in the drift screen.  It would no longer appear grouped under hardware with the other hardware properties.

@Fryguy @bdunne @gmcculloug Updated... I found a better way to fix this.  tldr: We were trying to use preloading of virtual columns on the main table when an included table's column happened to have the same exact column name as the virtual column.  I added a test also.  Throw :tomato:

@jrafanie The test failures seem to be because you don't have models for your new tables
Also, can you suppress the table creation logging in order to keep the build output clean?

@jrafanie The test failures seem to be because you don't have models for your new tables

@Fryguy Actually, it was because I was calling reflections instead of reflections_with_virtual.
I finally got this thing working with what I think are pretty good tests.  cc @bdunne
Keep me honest...

:+1: LGTM, Nice work!
@gmcculloug @Fryguy @chessbyte This one is important

@matthewd @Fryguy Please review


support for react 16.9/17.0

when using with react 16.9 we have this warning:

Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

Move data fetching code or side effects to componentDidUpdate.
If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: AnimatedNumber

Hey, @madmed88 @nanexcool @ameyms @phiresky @a-axton
As of React 16.9.x it seems the authors are beginning their transition to a full hooks or hooks preferred component strategy. This latest change requires that all componentWillMount, componentWillReceiveProps & componentWillUpdate calls, including those within bundle dependency packages, be prefixed with the UNSAFE_ prefix in order to dismiss the below console warnings. Of which, in my latest app where I'm using power-select I get about 4 per instance. I'd love to continue using react-animated-number for all my numerical needs, but have some stricter demands from my colleagues on console warnings, if you would kindly check into this, I'd be most appreciative. Thanks
Btw been a fan of your library for a years now and have used it in just about every react project I've worked on.
https://reactjs.org/blog/2018/03/27/update-on-async-rendering.html

Hey, @madmed88 @nanexcool @ameyms @phiresky @a-axton
As of React 16.9.x it seems the authors are beginning their transition to a full hooks or hooks preferred component strategy. This latest change requires that all componentWillMount, componentWillReceiveProps & componentWillUpdate calls, including those within bundle dependency packages, be prefixed with the UNSAFE_ prefix in order to dismiss the below console warnings. Of which, in my latest app where I'm using power-select I get about 4 per instance. I'd love to continue using react-animated-number for all my numerical needs, but have some stricter demands from my colleagues on console warnings, if you would kindly check into this, I'd be most appreciative. Thanks
Btw been a fan of your library for a years now and have used it in just about every react project I've worked on.
https://reactjs.org/blog/2018/03/27/update-on-async-rendering.html

We used https://www.npmjs.com/package/patch-package to cover packages who are not updated yet.

Hey @ameyms, is there interest in accepting a pull request to address this?

I also get this warning:
componentWillReceiveProps has been renamed, and is not recommended for use.


Alignment Issue with Dark and Light Mode Toggle in Responsive View (Below 800px)

Category

[ ] Documentation
[ ] Resource Addition
[ ] Codebase
[x] User Interface
[ ] Feature Request

Description
I noticed that the Dark and Light mode toggle is not aligned with the other navigation items when the screen width is below 800px. This alignment issue affects the visual consistency of the user interface.
Steps to Reproduce
1.Open the application on a device with a screen width below 800px.
2. Navigate to the navigation bar.
3. Observe the alignment of the Dark and Light mode toggle.

Interested to contribute in this issue

[x] Ready to Contribute

@mathanraj0601 Nice catch. The thing is, we don't want to have the toggle button as an entry to the navbar that we can see below 800px. Instead, we want it to be located at the left side of the hamburger icon.
This repository is currently participating in JGEC Winter of Code (JWOC) and we have decided that during this period, all the open issues will only be assigned to JWOC participants. If you have registered for this event, then please mention that in your comment. If not, please mention that as well because in that case, this issue will not be assigned to you unfortunately.

Thank you, @Sriparno08!  I'm not currently participating in JWOC, but no worries—I'm looking forward to contributing in the future after the event. For anyone in JWOC who is interested in tackling this issue, go ahead and work on it 🚀. Happy open source, happy coding! ✨

Thank you for informing, @mathanraj0601 and great to hear that you are interested in contributing to this project in the future 👍

Hi can i work on this issue? from JWoC

@Sriparno08 Please assign this issue to me under JWOC.

@antilneeraj You're assigned.
@AntonyJudeShaman The mentee who is assigned to this issue is chosen because he/she was the first one to ask for assignment with the 'JWOC' (Case-Sensitive) keyword in his/her comment.

@antilneeraj You can make it like this:

Don't change the button as another mentee is already working on it.

@Sriparno08 I understood that I only need to align the toggle and should not perform any changes in the appearance of the toggle button right?

Yeah, correct 👍

@antilneeraj You're assigned.
@AntonyJudeShaman The mentee who is assigned to this issue is chosen because he/she was the first one to ask for assignment with the 'JWOC' (Case-Sensitive) keyword in his/her comment.

Oh then the JWoC in my comment means a JCB bulldozer? Can u please visit the JWoC website and see how they have mentioned it?

@AntonyJudeShaman As we saw on Discord, there is a comment that says this:

Comment with I would like to work on that issue under JWOC (mentioning JWOC in a comment is compulsory)

Above, both the JWOC keywords are capitalized. Plus, we are advised to use the case-sensitive 'JWOC' label on our issues. That's why, we have decided to look for the case-sensitive 'JWOC' keyword in the issue comments to keep mentees on the safe side because otherwise, the assignee's PR may not get counted for an issue even though he/she wrote the keyword in their comment.
For this issue, it will be unfair to unassign the current assignee. So, we are keeping the assignment intact.
From the next time, please write the case-sensitive 'JWOC' keyword in your comment to ensure that your hard work is rewarded if you make a PR for an issue and avoid any kind of confusion.
Lastly, apologies for all the inconvenience caused. There's a lot of guidelines that we're all getting used to as time passes.

@AntonyJudeShaman Since many PRs are getting merged without the assignee mentioning 'JWOC' as case-sensitive, we have decided to just look for the keyword in the comments.
So, from the next time, there is no need to make 'JWOC' case-sensitive. Just make sure you mention it in your comment.
@antilneeraj You also note the above.

Alright 👍


contrib/: expand font setting capabilities

ZFSBootMenu binary releases use a default font size that's barely suitable for 2k screens and entirely unsuitable for 4k screens. In an attempt to fix this, a new script has been added that introduces automatic console font resizing.
Utilizing the Terminus font, we can pick a range of bold font sizes to better match screen resolutions. Working backwards from the largest to the smallest, the hook simply runs setfont and then checks if COLUMNS is at least 100 characters. If it is not, the next font is selected and it tries again.
Based on documentation, Terminus fonts prefixed with a 'v' have all mappings/codepages in them; so they should support the maximal number of languages.
contrib/20-console-autosize.sh is included in all release and recovery images. The automatic font setting behavior can be disabled via zbm.autosize=(0|off).
contrib/console-init.sh has been renamed to ensure it precedes this hook, since 10-console-init.sh calls console_init from Dracut, which is hard-coded to set a console font. Since console_init also performs setting a keymap, it should always be executed.
The legacy behavior of setting a font via rd.vconsole.font overrides/disables zbm.autosize.
TODO:

[ ] Document zbm.autosize in man pages

Maybe it's time to split contrib in two, moving scripts we actually use to a top-level hooks tree that can have some official documentation and claim official sanction.

The contrib/ directory has multiple different classifications of script right now:
"Official":

ZFSBootMenu early hook scripts (console related)
ZFSBootMenu late hook scripts (xhci related)
generate-zbm syslinux.conf support

"Unofficial":

generate-zbm esp-sync
ZFSBootMenu early hook luks-unlock
ZFSBootMenu late hook snapshot before kexec
Helper script to drive the build process, to enable dropbear in ZBM

How much sorting should be done here? contrib/ can be a catch-all for anything unofficial. Officially supported hooks should be placed into a proper hooks/{zfsbootmenu/...,generate-zbm/...} directory structure and then copied somewhere on the filesystem in the Makefile.

+1 on using contrib only for unofficial scripts

I agree we shouldn't be organizing the unofficial stuff in contrib. Just dump it there and let people wade around. As for hooks, what about
hooks/early-setup.d
hooks/setup.d
hooks/teardown.d
hooks/generate-zbm.pre.d
hooks/generate-zbm.post.d

The Makefile can install the whole tree to /usr/share/zfsbootmenu/hooks. Then, we can ship a default configuraton that looks in /etc/zfsbootmenu/hooks/* and either:

Just tell people to symlink what they want in these directories, or
Add logic that looks in both /etc/zfsbootmenu/hooks (or another, configurable path) plus $zfsbootmenu_module_root/hooks and lets the /etc/zfsbootmenu variant shadow the system location (e.g., by symlinking to /dev/null to disable system hooks).

I favor option 1 because it's the simpler of the two.

I'd prefer to have users symlink in scripts that they want to enable. There's no masking logic then that we have to figure out; if a script or symlink is present in the right directory, it's added (or executed) as needed.
I'd like to add a helper function to zfsbootmenu/install-helpers.sh that knows how to find the ZBM run-time hooks directories that copies in files to the ZBM image.  That would replace the need for explicitly defining files in zfsbootmenu_early_setup, zfsbootmenu_setup and zfsbootmenu_teardown variables in dracut.conf.d/zfsbootmenu.conf (and the equivalent for initcpio).


How to cook fried chicken using tdlib?

You can choose the right meat in the store
Take the 24 cores computer with 64GB RAM
Put the meat in the computer block. Run the tdlib assembly on all cores with ninja

The most important: Do not repeat at home

@isopen I tried to assemble a library. Everything turned out the first time.
I try in the language "Go

What was the solution here? I'm currently having the same problem.

hmmm, i can't seem to get this to work with brisket. any luck?

Same. No luck with brisket or skirt steak. I think it may only be for chicken, or perhaps poultry in general. I've had minimal success with corned beef. Maybe if you also corn the poultry?

how are you seasoning your meats and how long are you letting them marinate?
and no, I wouldn't suggest corning the poultry. the skin gets all shriveled like old man's balls.

Any luck? This is preventing my production-ready project from launching.

No luck. My team ended up just going to KFC. Eat Fresh.


A15-4-4: Functions that allocate are considered non-throwing

Affected rules

A15-4-4

Description
A15-4-4/MissingNoExcept.ql flags functions which are not noexcept when no exception is explicitly thrown by them (or the functions they call). Unfortunately, this doesn't take into a account the fact that std::bad_alloc can be implicitly thrown by functions which allocate.
Example
void AppendTo(std::string& string) {
  // rules/A15-4-4/MissingNoExcept.bqrs:
  //   Function AppendTo could be declared noexcept(true).
  static_cast<void>(string.append("foo"));
}

Thanks for the report! This requires expanding our modelling of the exception behaviour of standard library functions, and deciding on default behaviour for different queries (ideally this one would over approximate the exceptions thrown, vs. the rules that look for missing annotations, which we may under approximate).


AttributeError: 'Namespace' object has no attribute 'output'

Hi there,
I am getting this error when running rMETL detection (v1.0.2). It does generate the signatures and brokes like this:
Traceback (most recent call last):
  File "/apps/0_external_tools/rMETL/rMETL.py", line 57, in <module>
    parseArgs()
  File "/apps/0_external_tools/rMETL/rMETL.py", line 54, in parseArgs
    STAGES[args.stage](args.options)
  File "/apps/0_external_tools/rMETL/src/rMETL_extraction.py", line 566, in run
    load_sam_multi_processes(args)
  File "/apps/0_external_tools/rMETL/src/rMETL_extraction.py", line 500, in load_sam_multi_processes
    output_p = args.output if args.output.endswith('/') else "%s/"%(args.output)
AttributeError: 'Namespace' object has no attribute 'output'

My command has been:
python2.7 rMETL detection -t 8 -x ont
/pathToBam/sample.bam
/pathToReference/hs37d5.fa
/pathTooutputDir
/pathTooutputDir/sampleName
/pathTooutputDir existed already, but /pathTooutputDir/sampleName didn't, but don't think is related to that since you have a mkdir command.
Shouldn't it be args.output_dir instead?
Thank you in advance!
Sonia

Hi @Soniazumalave
Thanks for your bug reporting. I am sorry for my carelessness. As you mentioned, I miswrote output_dir into output and an AttributeError was raised. Now I have fix this bug and please use the latest version of  rMETL in your work.
If you have any questions about rMETL, please contact with me.
Thanks for using rMETL.
Cheers!

Hi @tjiangHIT,
Thanks for your answer. It does work now! I am closing this issue.
Regards,
Sonia


Missing Flesh 1.21

The latest contagion 1.21 version 1.2.0 is missing the flesh from zombies.

I just created a new game with just Contagion, killed a few zombies and no issues.

could be a bug regarding with the mod Charm... somehow some mods just dont get along with other mods. I would reconsider re testing again since I got a bit ahead of myself.

Have you check it with turning on Milk Drinking option? since I turned that on.

ill be closing this since I'll be using the milk curing method. till I find the cause.

I created a new instance and it still doesn't drop any Contagious Flesh..

it's set to 5%, so on average you have to kill 20 Zombies for 1 Contagious Flesh.

so that's why. Will it affect Looting enchantment soon?

I was also thinking maybe making the value configurable


[Feature] Configurable baseline directory

I would like to raise a feature request and if you are interested I could help to implement it as well.
I think it would be awesome if the user could pass in an environment variable with a path for the baseline.json file.
My idea is that I commit the baseline.json file to the source control but with the current implementation I can not do that.

@B3zo0 the use case for compare mode and the baseline file was to be able to run local comparisons between different builds. Having the file outside git inside is allowing bundle-stats to use it even after a branch change.
Can you please tell me more about your use case? Personally, I don't see too many benefits in committing the file, but I don't think the change will impact the current behaviour. Feel free to create a pull request & let me know if you have any questions ;)

I'm going to close this issue because I went with https://relative-ci.com/documentation/setup/webpack-plugin
It is easier and cleaner approach.

Is there a way to specify a different baseline? Even outside the directory, like the ~/.cache.
Sometimes my build fails and I do rm -rf node_modules and the baseline file is gone :cry:

@romariolopezc there are couple ways to achieve this now:

If you use the CLI, you can pass the baseline path as the last argument

bundle-stats CURRENT_WEBPACK_STATS_PATH BASELINE_WEBPACK_STATS_PATH

bundle-stats is using find-cache-dir to resolve the cache directory, you can override that at run time with CACHE_DIR env variable


AESEncrypt->AESDecrypt not functioning properly

All the other encrypt/decrypt pairs appear to work except for the AESEncrypt->AESDecrypt.  If you AESEncrypt a string with any key, AESDecrypt does not output that same string with the same key.  In fact, it doesn't seem to output anything.

Hi @DannyBoyk,
Have you checked that the "Output result" argument for the AES Encrypt operation is set to "Ciphertext"? The default is "Show all", but the decrypt operation won't understand the format unless it's just the ciphertext.
Example

Thank you for that.  I was able to duplicate what you did only specifying a key.  I should amend my first statement to indicate I was also specifying an IV in hex.  My IV was 16 bytes in length.  Even using the same IV in the decrypt block, the message fails to decrypt.  Is it expecting an IV of a specific length?

Thanks for reporting this, I'm getting the same problem when an IV is specified. I'll look into it shortly.

from what I random test with the DES encryption also has the same issue even though I put the same key,( or IV and salt) I got blank output for some reasons. I guess, there will be something wrong in the decryption.

I have been playing around with this. Looks like some issues with what it accepts. AES should only accept 16, 24 or 32 byte keys. DES should only allow 8 byte keys. These functions also require a IV for modes that don't use IVs. I also noticed that the "NoPadding" option doesn't always work.


Error: "Failed to start API Mocking. Running cleanup tasks"

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v16.13.0
Amplify CLI Version
10.0.0
What operating system are you using?
M1 Mac w/ rosetta terminal
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
I've tried w/ making changes and w/o making changes.  Either way I always get the same error.
Amplify Categories
auth, api
Amplify Commands
Not applicable
Describe the bug
I'm getting the following error:
➜  amplify mock
Failed to start API Mocking. Running cleanup tasks.
🛑 Cannot read property 'stop' of undefined

I've been able to run amplify mock function but not amplify mock api.  I'm just learning to use amplify but I've tried it on about 4 different test projects over the last week, all with different schemas and different levels of complexity and so far I have yet to get amplify mock api to run.  The error happens on v10 and v10.2.1 (v10.2.1 offers much better error messaging 👍 )
Expected behavior
I would expect to be able to mock the api
Reproduction steps

amplify init (choose all defaults, authenticate via AWS Profile)

amplify add api (choose "single object with fields")

amplify mock api.

GraphQL schema(s)

# Put schemas below this line
input AMPLIFY { globalAuthRule: AuthRule = { allow: public } } # FOR TESTING ONLY!

type Todo @model {
  id: ID!
  name: String!
  description: String
}

Project Identifier
d34d70d508b392d59285bf7088ddb485
Log output

# Put your logs below this line

Additional information
No response

That was it, I didn't have java installed 🤦.  Thanks!
Is this documented somewhere?

Hey @gxxcastillo , glad to hear that you are back up and running. This is currently documented in https://docs.amplify.aws/cli/usage/mock/.
Closing the issue for now, please feel free in reaching out to us again.

I think it may make sense to re-open this and change the error message since Amplify is intended to be user-friendly


Colour of Chips is wrong when selecting anything other than "Alle"

see colour of chips in screenshots attached

Do you meen the darker state? that is the "click" state.
But you are right by saying, it should be the more brilliant color after the view is refreshed, if that is what you mean.

Exatly, I mean the brilliant Green. On the first picture, before selecting any chip, it's the correct Green. As soon as you chose another one or go back to "Alle" it's the darker one.

This is the default behavior of the color-theme provided by material-ui. Maybe someone that is more experienced with that can take over from here?

I checked on some other items (buttons etc) on our app - behavior is the same. as long as the "pressed" state is active, it is displayed as shown...


DataGrid : Fix sorting when there is Header row (#7645)

Description
MudDataGrid has an issue with the HeaderCell component when the Header is added and there is no Column on a sort.
This PR fixes #7645
How Has This Been Tested?
A new DataGrid unit test was added to verify if sorting does not throw NullReferenceException and if sorting is still working as expected.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

@henon I suggest to merge it without @tjscience review as it just fixed a null exception.

Thanks @timlunev


Manually adding breadcrumbs to a sentry event, only works for iOS and not Android

Platform
Flutter Mobile
Obfuscation
Disabled
Debug Info
Disabled
Doctor
[✓] Flutter (Channel stable, 3.19.1, on macOS 14.2.1 23C71 darwin-arm64, locale da-DK)
• Flutter version 3.19.1 on channel stable at /Users/hurley/development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision abb292a07e (7 weeks ago), 2024-02-20 14:35:05 -0800
• Engine revision 04817c99c9
• Dart version 3.3.0
• DevTools version 2.31.1
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at /Users/hurley/Library/Android/sdk
• Platform android-34, build-tools 34.0.0
• Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 15C500b
• CocoaPods version 1.15.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2023.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
[✓] IntelliJ IDEA Community Edition (version 2023.3.4)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 78.1.1
• Dart plugin version 233.13763.5
[✓] Connected device (4 available)
• SM G960F (mobile)  • 221c919427027ece                     • android-arm64  • Android 10 (API 29)
• iPhone 15 (mobile) • ACAFE1FD-6161-4C5F-9F0A-B5D12D03ED29 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-2 (simulator)
• macOS (desktop)    • macos                                • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
• Chrome (web)       • chrome                               • web-javascript • Google Chrome 123.0.6312.122
[✓] Network resources
• All expected network resources are available.
• No issues found!
Version
7.18.0
Steps to Reproduce

Capture an exception
Try to manually add breadcrumbs to the SentryEvent in beforeSend()
Return the event with the added breadcrumb

Expected Result
That the sentry event is sent to sentry and that we can see the added breadcrumb in the sentry event
Actual Result
The breadcrumb is only added to the events that are logged by iOS devices, and if an android device tries to create an event, the breadcrumb is not there.
Are you willing to submit a PR?
None

SentryEvent? beforeSend(SentryEvent event, {Hint? hint}) {
event.breadcrumbs?.add(
Breadcrumb(
message: "Added breadcrumb",
timestamp: DateTime.now().toUtc(),
level: SentryLevel.info));
return event;
}
The following code gives adds the breadcrumb from iOS devices, but not if it is an Android device

Thank you for reporting, I can confirm this, we'll take a look

This should be fixed in v8, can you confirm @denrase ? since we removed the code that removes breadcrumbs

Awesome, ill leave the issue open until it has been released. Thank you!

Just confirmed with 8.0, that both iOS and Android have the added breadcrumbs.


End of game resolved before some extra actions

At the end of the game, the points for war are counted before all (extra) player actions are resolved, potentially leading to wrong war results.
For example, having the ability to pick up a card from the discard pile is performed after the war points are counted. If a red card is picked in this example, it is de-facto ignored at the end of the game

Thanks a lot for this report. There were already some glitches like this in the past, but I thought I had fixed the issue.
The case of the discarded cards is actually specifically handled before end of game events, which is why I'm surprised to see you have this bug. I will look again.
This area (about the game state / extra actions) needs to be reworked anyway, it was on my todo list :)

@stevasale I can't reproduce your exact example. I'm playing Halikarnassus B, and using the last turn of the game to upgrade the wonder, and pick a war card from the discard pile to get my military up to win over my neighbours.
It did work, and the score reflects it.
Could you please elaborate on how you saw this problem?

I am sorry if those cases which can influence the outcome of the war are already taken care of. I just saw that the war points were already calculated even though not all extra player actions were done and assumed from there, that potentially something could go wrong.
The example given was just the most obvious case that came to mind.
However, if the extra actions of Babylon B or Olympia B are performed after the war points have been calculated, then still something could go wrong.
Come to think of it, i think I noticed this with Babylon B, where the seventh card I picked was a war card, and it was literally pointless. However, it's been quite some time ago, so I am not too sure any more.

Thanks a lot for the precisions. I just tested and you are actually right regarding Babylon B and Olympia B.
For some reason their effect is not reflected in the score board (it's not just about war actually, copying guilds should give points).
There is actually an even worse issue with Babylon B:
https://github.com/joffrey-bion/seven-wonders/issues/125

I'm closing this issue for the 2 more specific ones.


Migrate to Ballerina Swanlake Preview 1

@daneshk @chamil321 - I've simplified the error handling since there's no real usage for so many error types and constants. Pls let me know your opinion.

@daneshk @chamil321 - I've simplified the error handling since there's no real usage for so many error types and constants. Pls let me know your opinion.

Yes, Let's go with a simple option now. If needed we can improve it later


Avoid work if signed assembly already exists.

If the signed assembly folder already contains the output assembly and its Mvid matches the Mvid of the current assembly, avoid all the work and just use the existing output assembly.
This helps with performance and also build incrementality. If the output assembly is not touched unnecessarily it's timestamp won't change and it won't invalidate the rest of the build as it currently does.

Thanks!


Issue 10

Updating the markdown table with latest variable names

I think the changes look good, but the black box test isn't running - https://github.com/dangoslen/constable-github-action/pull/35/checks?check_run_id=1130752918
Not sure, but it looks like something is happening when trying to generate the table

how do we go about fixing this?

@yash01234 Run npm run package command after you update the index.js file. That will make changes to the dist/index.js file as well coz there's a difference in those 2 files.

Also, can you take index.js again from main branch and copy your code for the table in that. I think you are using the old index.js file. May be the checks are not passing because of that.

@yash01234 you need to update the dist/index.js file using the npm run package command. Once you do that, just commit and push those changes and it should be good.

When I run (npm run package) command I get an error. I go onto check the logs and I see- (error missing script: markdown-table) as an error for some reason

interesting. sounds like you have an odd environment that isn't including everything in your node modules directory.
have you done npm install since the markdown-table was added to the package.json?

Yes works now. Thanks :)

awesome! feel free to merge

Done


Django LazyObject is not considered in mutation resolvers

Describe the Bug
The user from Django's request.user is set to a LazyObject in the AuthenticationMiddleware. I believe a LazyObject is always an Iterable even if the underlying object isn't because LazyObject implements iter. When passing the request.user to mutations resolvers.update, it will think its an Iterable and call list(instance). However, this causes an error because the user object isn't actually an iterable.
This could be resolved by doing a check at the beginning of the resolvers similar to how its done in django:
if isinstance(instance, LazyObject):
  instance = instance.__reduce__()[1][0]

Though for me to fix this, I just passed user._wrapped to the resolver.
This would also affect the delete mutation resolver as well.
However, I believe this exact scenario is the only case where this would happen. I didn't see references to LazyObject wrapping models except for this case. So since its obscure, maybe its not worth considering? Just creating this issue to track what I ran into and see if y'all have any thoughts.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Hey @ryanprobus,
Interesting... I don't think I have any use case for those in my projects, so I've never stumbled upon this.
I think it is fine for us to add an if isinstance(instance, LazyObject) there and in other places that makes sense. Do you want to try to open a PR for that? I'll gladly review it :)

Fixed by https://github.com/strawberry-graphql/strawberry-django/pull/338


supabase vectorstore - first cut

First cut of a supabase vectorstore loosely patterned on the langchainjs equivalent. Doesn't support async operations which is a limitation of the supabase python client.

I'm overriding from_texts in an incompatible manner having added additional arguments. How do you suggest I modify to pass the linter?

@dev2049 Modified per your suggestions. mypy is still unhappy with the method signature overrides:
langchain/vectorstores/supabase.py:95: error: Signature of "from_documents" incompatible with supertype "VectorStore"  [override]
langchain/vectorstores/supabase.py:95: note:      Superclass:
langchain/vectorstores/supabase.py:95: note:          @classmethod
langchain/vectorstores/supabase.py:95: note:          def from_documents(cls, documents: List[Document], embedding: Embeddings, **kwargs: Any) -> SupabaseVectorStore

Note that the signature for from_documents hasn't changed other than 3 additional explicit args.

Note that the signature for from_documents hasn't changed other than 3 additional explicit args.

guessing you have to include **kwargs at the end. but also note that base class VectorStore has default from_documents implementation that probably works for you use case
    @classmethod
    def from_documents(
        cls: Type[VST],
        documents: List[Document],
        embedding: Embeddings,
        **kwargs: Any,
    ) -> VST:
        """Return VectorStore initialized from documents and embeddings."""
        texts = [d.page_content for d in documents]
        metadatas = [d.metadata for d in documents]
        return cls.from_texts(texts, embedding, metadatas=metadatas, **kwargs)

@danielchalef excited to see this! has been in typescript for a while, excited to getting python to feature parity ;) thanks for adding

note that base class VectorStore has default from_documents implementation that probably works for you use case

Good catch. Thanks!
mypy doesn't like my previous use of a keyword-only argument marker. Have made the additional arguments Optional and checking for presence at runtime. Not ideal but can't think of an alternative.

@hwchase17 Done!

I've followed this: https://python.langchain.com/en/latest/modules/indexes/vectorstores/examples/supabase.html
but when i do similarity_search it responds with postgrest.exceptions.APIError: {'code': '42P01', 'details': None, 'hint': None, 'message': 'relation "docstore" does not exist'} from Supabase. Is there something in the SQL script via the tutorial that isn't correct?

Create a table called "docstore" or modify the Postgres function to use a the table name reflecting your schema.
I'll cut a PR later to correct the example function.

Create a table called "docstore" or modify the Postgres function to use the table name reflecting your schema.
I'll cut a PR later to correct the example function.

Ohhh my bad! Thank you so much apologies for me being silly

Any way to query from_existing_index, like in Pinecone and in the JS implementation?

Sounds like a great feature, though assumptions would need to be made around embedding model. I don't have bandwidth to work on this, but am sure the langchain team would appreciate the contribution.

Any way to query from_existing_index, like in Pinecone and in the JS implementation?

did you found any way?

Any way to query from_existing_index, like in Pinecone and in the JS implementation?

did you found any way?

Nope. Went back to Pinecone.

@kasem-sm @rapcal I believe it's just this?
supabase: Client = create_client(supabase_url, supabase_key)
    embeddings = OpenAIEmbeddings()
    vector_store = SupabaseVectorStoreWithMetadataFiltering(table_name='documents', embedding=embeddings, client=supabase, query_name='match_documents')

That said the supabase integration is lacking functionality such as filtering, but I think this should work for just loading it as an index. Maybe I'll push a pr for the docs.

@rapcal @kasem-sm If you have a few set fields you want to filter with, and not too much dynamic filtering, I got it working quite nicely by implementing my own Vector Store class. I explained a bit more here https://github.com/hwchase17/langchain/pull/5379#issuecomment-1610775206, and if needed I can share some minimal reproducible code if someone wants it.


Using Link only updates path and doesn't change anything in the DOM

Hi there, first of all thanks for this great lib!
I wanted to build an application with it, so I copied and renamed parts of the overview example. I used links everywhere, but only the url is getting updated and not the content (a full page reload does the job, though).
What am I missing?
My routes are defined here, the route handler here and the links here.
Thanks for your help!

Does removing the PureRenderMixin do anything?

Yes, that did the trick. Well, could have thought about that, thanks for your help!

No prob!


PodOptions should take ImagePullSecrets

Change Overview
This modification will enable PodOptions to take ImagePullSecrets as an input when creating a Pod.
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[x] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues

fixes #issue-number

Test Plan

[ ] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

CI error is unrelated. There was another PR to update the linters - let me merge that

@pavannd1 May not be unrelated unless we are no longer checking gocognit? That test is a bit poorly written and has a lot of cases/complexity. Either way I added an ignore since its just a test

I had missed the actual error. Let's merge this 👍🏼

@pavannd1 Not merging yet because there are some other consequences, will discuss in slack.


Improvements in blob storage support for SQL Migration.

Adding auto refresh dialog

Auto refresh dialog. Clicking on a different option or clicking outside will close the dialog

Auto refresh in other screens

Adding new columns for file, changing cutover icon, and adding auto refresh.

Pull Request Test Coverage Report for Build 929890791

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 43.369%

Totals

Change from base Build 926472147:
0.002%

Covered Lines:
26602

Relevant Lines:
55739

💛  - Coveralls

I would suggest to split the button to 2 separate components: text component indicating the refresh frequency, 'Edit' button to change the frequency.
As a user I would also like to see the remaining time for next refresh.

why not use all the horizontal spaces, as you can see the text is wrapped here. maybe you should left-align the content.

@alanrenmsft, had to remove the refresh dialog based on the UX feedback from the PMs. It will be replaced by a configuration setting in the future.


Switch cell type from the cell menu

Iteration towards https://github.com/jupyterlab/retrolab/issues/259
Update the menus to:

Rename the top-level Run to Cell like in the current classic notebook v6
Add submenu to change the cell type

RetroLab

Notebook v6

TODO: this should not affect the JupyterLab menus:

Reported upstream in https://github.com/jupyterlab/jupyterlab/issues/11754

A temporary workaround would be to add the menu items programmatically in the plugin instead of using the settings system.

Let's merge, and we can investigate how to properly use the settings system as a follow-up (https://github.com/jupyterlab/jupyterlab/issues/11754).

@meeseeksdev please backport to 0.3.x


Pass the argument entity.table_ref() instead of just entity.

Pass the reference instead of the entity itself.

Hey @josh-codes, I have added some commits, please check

Will test (:

@tyt2y3 why was this merged? it is using the beta 2021 of rust version and had an outdated version string. 0.3.2

@tyt2y3 why was this merged? it is using the beta 2021 version of rust version and had an outdated version string. 0.3.2

I don't get it. Is it specific to this PR?

I'm confused

The final changeset does not touch the Cargo.toml

@tyt2y3 why was this merged? it is using the beta 2021 version of rust version and had an outdated version string. 0.3.2

I think you undo the changes already

<img width="393" alt="Screenshot 2021-11-17 at 20 45 49" src="https://user-images.githubusercontent.com/43808099/142203217-165bf35b-34db-4c7d-9d2b-7cc3060eb4ed.png"

also, if it doesn't get compile with those settings, shouldn't we just use the correct settings?

@tyt2y3 why was this merged? it is using the beta 2021 version of rust version and had an outdated version string. 0.3.2

I think you undo the changes already

I did, but I wanted to ask first.

Ahhhh I got what you mean

You mean why is this still 0.3.1. Correct?
https://github.com/SeaQL/sea-orm/blob/0deedddf2809da0fa81e9a54b24bea198410ad4d/Cargo.toml#L4-L14

No worries, we will soon be releasing 0.4.0

No worries, we will soon be releasing 0.4.0

Yes, and also, why is it version 2021, that's a beta version of rust.

No worries, we will soon be releasing 0.4.0

Yes, and also, why is it version 2021, that's a beta version of rust.

I think Rust Edition 2021 is stable now.
https://blog.rust-lang.org/2021/10/21/Rust-1.56.0.html#whats-in-1560-stable


how I can  update values in component to global state

Example: user_name to navbar, any guidance. Thanks!

A new version and guide has been released:
https://elm-spa.dev/guide
With the latest version, you can update using the save function in a Page.application to save data to Shared.Model
If you have any other questions– please join us in #elm-spa-users on the Elm Slack


Fix issue #830: Improve compatibility with editor config, including *…

….{ sections
Fixes issue #830

Codecov Report

Merging #1145 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1145      +/-   ##
===========================================
+ Coverage    95.79%   95.79%   +<.01%
===========================================
  Files           49       49
  Lines         3541     3547       +6
===========================================
+ Hits          3392     3398       +6
  Misses         149      149

Impacted Files
Coverage Δ

tests/test_isort.py
99.67% <ø> (ø)
:arrow_up:

isort/settings.py
90% <100%> (+0.19%)
:arrow_up:

Continue to review full report at Codecov.

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


Breakpoint classes not working in Vue

Hello, I've recently installed Tailwind for my Vue project. It took some time to get it work but finally, it worked, even with code completion in VS Code.
The problem I am facing right now is that I cannot use the breakpoints anywhere in my project.

        // Content

Watch a gif of it.
What I want to achieve is to have the div be 1/3 when on desktop, full width on mobile. What I am getting is full width everywhere. I can't seem to find anything else like this on the internet either.
I'd like to mention that I am using VueJS, if that's of any help. Any ideas?
Thanks in advance.

If anyone has the same problem again, here's the solution I found:
Eventually found the solution after a break and finding a Vue project that uses Tailwind. I needed to install postcss-preset-env and add it to postcss.config.js.
npm install postcss-preset-env --save-dev
postcss.config.js
module.exports = {
  plugins: [
    require('postcss-preset-env')({ stage: 0 }),
    require('tailwindcss')('tailwind.js'),
    require('autoprefixer')
  ]
}


chore: convert view to TypeScript

Refs #293. ~Draft, based on #310.~
With using a typed protocol now, it also uncovered a couple of things that were outdated with the latest view changes.

Rebased and ready for review now.


Deploy BETYdb API to TERRA instances

Scott just committed the v0 API to the BETYdb master  branch https://github.com/PecanProject/bety/commit/494767e03af17dc33eb590eba3367da90af34823
Please:

Create a new release
deploy these to the existing TERRA instances of BETYdb (mepp, terra-test, and terra-ref)

@robkooper when will the test and references instances be migrated to their more permanent location (#65)?

@gsrohde it is not clear from the online docs (betydb.org/api/docs) how to construct an API call from the current api docs. e.g. shouldn't https://www.betydb.org/api/beta/citations:50 work?

@dlebauer It’s https://www.betydb.org/api/beta/citations/50, and you have to add a query string that includes key=<your api key>.  I’ve thought about removing the requirement for a key so that you could easily view the result in a browser once logged in; or, if using curl, you could use the -u username:password option instead of a key.  Though, actually, if you are logged in, even a bad key (or just ?key=) will work.
Regarding the API docs, it does refer to the path GET /api/beta/citations/:id, so it should be clear that you need a forward slash after citations.  What’s not clear, perhaps because this automatically-generated documentation is too “Rails-developer-centric”, is that the placeholder of the id parameters is called “:id”, so that when you replace this with a number, you should be replacing the colon as well.

What’s not clear, perhaps because this automatically-generated documentation is too “Rails-developer-centric”

This is where examples come in. I can't recall - does apipie and / or the testing suite provide a framework for this?
The other bit that is not clear to me is how to chain parameters together multiple parameters.


setup mkdocs doc

[x] setup basic material
[ ] deploy mkdocs with github action after release

well done!!


Redis cache issue with Entity framework model classes - dotnet core 3.1

Summary of the issue
First call insert the value into redis successfully but the second one always display the below error;
"Could not load type 'Castle.Proxies.OrganizationProxy, DynamicProxyGenAssembly2, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. Try add TypeCache.RegisterResolveType to resolve your type if the resolving continues to fail."
On the other hand, if I use a DTO in my select statement then no problem occurs.
Note: I used the redis configuration exactly as explained.
I have a feeling that this issue is happening with FirstOrDefault() & First()
Environment
.NET Core SDK version: 3.1.0
Microsoft.EntityFrameworkCore version: 3.1.0
EFSecondLevelCache.Core version: 2.9.1

Example code/Steps to reproduce:

return _context.Organization.Where(o => o.OrganizationId == orgId).Cacheable().FirstOrDefault();

## Output:

Exception message:
Message = "Could not load type 'Castle.Proxies.OrganizationProxy, DynamicProxyGenAssembly2, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. Try add TypeCache.RegisterResolveType to resolve your type if the resolving continues to fail."
Full Stack trace:
" at CacheManager.Core.Internal.TypeCache.GetType(String type)\r\n   at CacheManager.Redis.RedisValueConverter.CacheManager.Redis.IRedisValueConverter<System.Object>.FromRedisValue(RedisValue value, String type)\r\n   at CacheManager.Redis.RedisCacheHandle1.FromRedisValue(RedisValue value, String valueType)\r\n   at CacheManager.Redis.RedisCacheHandle1.GetCacheItemAndVersion(String key, String region, Int32& version)\r\n   at CacheManager.Redis.RedisCacheHandle1.GetCacheItemInternal(String key, String region)\r\n   at CacheManager.Redis.RedisCacheHandle1.GetCacheItemInternal(String key)\r\n   at CacheManager.Core.Internal.BaseCache1.GetCacheItem(String key)\r\n   at CacheManager.Core.BaseCacheManager1.GetCacheItemInternal(String key, String region)\r\n   at CacheManager.Core.BaseCacheManager1.GetCacheItemInternal(String key)\r\n   at CacheManager.Core.Internal.BaseCache1.GetCacheItem(String key)\r\n   at CacheManager.Core.Internal.BaseCache`1.Get(String key)\r\n   at EFSecondLevelCache.Core.EFCacheServiceProvider.GetValue(String cacheKey)\r\n   at EFSecondLevelCache.Core.EFMaterializer.readFromCache[T](Expression expression)\r\n   at "

EF Core uses Castle.Proxies to support lazy loading. Also other libraries such as AutoMapper are using it too. Sometimes it means you have initialized auto-mapper more than once or you are not referencing correct dependencies in different libraries of your project (For using lazy loading, the package Microsoft.EntityFrameworkCore.Proxies needs to be added to the project references.). Also SecondLevelCache always uses eager loading. So if you have enabled lazy loading, don't use it with this library.

but this is not happening when using .ToList()

I can't reproduce your issue. To test it, First I enabled the redis provider and then called the https://localhost:5001/ which maps to the FirstOrDefaultAsync sample. It works.


Is it possible that when we scan a QR code it opens a tab where we can show the data of a User ? Currently its not opening a browser tab Automatically

Hi @adhasvbdhrafay this is something that depends on how your client / reader handles URLs. The QR code itself only provides a simple string - possibly an URL - and no instructions on how to handle them. Good luck!

Hi @adhasvbdhrafay this is something that depends on how your client / reader handles URLs. The QR code itself only provides a simple string - possibly an URL - and no instructions on how to handle them. Good luck!


Feature request: Shallow and sparse clones of registry

To support monorepos where the granted registry can be just one part of a huge repo.
git clone --depth ...
https://www.git-scm.com/docs/git-sparse-checkout

Maybe a separate request, but also cloning/pulling with GIT_LFS_SKIP_SMUDGE=1 to avoid long delays when big LFS files get added to a monorepo would be nice (when LFS filters have been installed globally). Could be an optional feature, ideally on by default.


Docker -p ip:hostPort:containerPort and ip::containerPort not compatible with IPv6 addresses

Docker --publish is not compatible with IPv6 if one wants to use he IP:host_port:container_port argument format.
Even when address is provided in the well-known form of [IPv6] it does not work:
docker: Invalid format to parse.  [2001:470:6267:324::c2]:53:53 should match template ip:hostPort:containerPort. See 'docker run --help'.

@MalteJ: could you take a look at that?

@emsi yes, this would require DNAT on IPv6 or setting up a proxy for this port.
As we do not want to use NAT with IPv6 we do not support publishing container ports on the host's IPv6 address.
If you want to use the host's IPv6 address you should consider running your Docker container with docker run --network host

I'm not sure if I was clear. I'm talking about what is implemented now. Now when I use dual-stack on docker host I can use -p to publish some ports. If I call -p without providing IP the docker_proxy binds to :::port which is both ip and ipv6. See #11520. So it listens on both stacks. I can limit it to ipv4 if I provide IP but I cannot limit it to ipv6 the same way.

Hey @MalteJ, ipv6 NAT is used in many networks. (As much as i hate NATs...) ipv6 NAT does have important use cases. :/  Anyway, we're certainly running into this problem as we create clusters of ipv6 services. We need to run multiple docker containers in the same host with ipv6 and publish ports from all of them. the docker run --network host is a poor workaround IMO. Is this something that the new networking tooling may fix? Or do you really plan on not supporting IPv6 NAT?

@jbenet I'd be excited to hear your use case :)
Regarding libnetwork roadmap @mrunalp probably has the best overview.

ping @mavenugo @mrjana perhaps you can have a look if this is (or will be) on the roadmap?

thanks @thaJeztah . We are planning on addressing a bunch of ipv6 issues for 1.9. Will add this to the 1.9 milestone.

thanks @mavenugo !

For those watching this issue, I’ve sent https://github.com/docker/docker/pull/20315 which contains a working implementation that addresses this issue. Not sure whether discussion is supposed to happen on the PR or here.

Still not working with 1.12.1, build 23cf638
docker: Invalid format to parse. [::1]:1000:8000 should match template ip:hostPort:containerPort.

Yes. https://github.com/docker/libnetwork/pull/1367 is still not merged. https://github.com/docker/go-connections/pull/17 is done, but I think the vendored copy of go-connections might not be updated yet.

They replaced https://github.com/docker/go-connections/pull/17 with https://github.com/docker/go-connections/pull/19, but nothing is happening :/

looks like this was automatically closed because https://github.com/docker/libnetwork/pull/1367 was merged.
reopening because those changes are not yet vendored in this repository

Could someone take care of the vendoring please? I’m too pressed on time to figure out how that’s done.

https://github.com/docker/go-connections/pull/19 needs to be merged before this gets vendored into this repository. Just want to point that out in case the other Docker developers missed that.

It's been nearly 2 years and this issue still persists 😢

Reopening as it's not vendored yet 👼


Update openstack-ansible SHA

This commit updates the openstack-ansible SHA in preparation for the
next release.
The following issues are addressed by this change:
Connected https://github.com/rcbops/rpc-openstack/issues/1130
Connected https://github.com/rcbops/rpc-openstack/issues/1131
Connected https://github.com/rcbops/rpc-openstack/issues/1134

RPC-AIO-Liberty-Upgrade-PR: Fail

recheck_upgrade

RPC-AIO-Liberty-Upgrade-PR: Fail

Upgrades are failing due to a change upstream. I've created at upstream fix and am currently testing it with a separate pull request

👍


Fix gles3 shader to actually multiply specular light by rev_amount for fog calculations.

This resolves the 'additive' anomaly seen for dark colors of fog mentioned in #12454.
Bugsquad edit: Fixes #12454.

The implementation of fog looks strange to me. Multiplying all lighting components separately just to add the fog afterwards seems pretty wasteful...
From a physically-based-rendering view, a single mix(color, fog_color, fog_density) should be sufficient.

@CptPotato The problem is that Godot saves specular and diffuse to separate render targets, so there is not a "final color" to blend to.

Interesting, so I guess godot is using this semi-deferred approach to allow for post effects like subsurface scattering. Thanks for the clarification.

Thanks!

@CptPotato Yes exactly, rendering to diffuse and specular separately is a bit more expensive, but it allows doing SSAO, SSR, and SSS properly, even with a forward rendering.

Cherry-picked for 3.0.7.


Global Regex Search with Whole Word Only Searches Open Files If Search Includes Newline

Type: Bug
Make a global Whole Word, RegEx search that returns both open files and files in the project folder(s). It will return multiple files. The search must match something at the end of a line.
Add a newline to the search, using \n. It will incorrectly only show open files that have a newline.
Turning off Whole Word will correctly include all files that match the search string.
The search should either include all files that match or no files that match, instead of including open files only.
VS Code version: Code 1.95.1 (65edc4939843c90c34d61f4ce11704f09d3e5cb6, 2024-10-31T05:14:54.222Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 2611)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
31.69GB (3.58GB free)

Process Argv
--crash-reporter-id 5983b7a4-e461-42af-8bc3-bfe1619ec9de

Screen Reader
no

VM
50%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
12th Gen Intel(R) Core(TM) i7-1255U (12 x 0)

Memory (System)
15.47GB (8.44GB free)

VM
0%

Extensions (35)

Extension
Author (truncated)
Version

extension-path
act
0.0.4

lua-debug
act
2.0.10

batch-replacer
ang
1.2.1

vscode-markdownlint
Dav
0.56.0

gitlens
eam
15.6.3

gc-excelviewer
Gra
4.2.62

rainbow-csv
mec
3.12.0

vscode-docker
ms-
1.29.3

makefile-tools
ms-
0.11.13

quicktype
qui
23.0.170

java
red
1.36.0

vscode-yaml
red
1.15.0

markdown-preview-enhanced
shd
0.8.14

intellicode-api-usage-examples
Vis
0.2.9

vscodeintellicode
Vis
1.3.2

vscode-gradle
vsc
3.16.4

vscode-java-debug
vsc
0.58.1

vscode-java-dependency
vsc
0.24.0

vscode-java-pack
vsc
0.29.0

vscode-java-test
vsc
0.42.0

vscode-maven
vsc
0.44.0

markdown-all-in-one
yzh
3.6.2

vscode-nginx-conf
ahm
0.3.5

jsonl
Alp
0.0.1

Handlebars
and
0.4.1

jsdoc-markdown-highlighting
bie
0.0.1

javascript-ejs-support
Dig
1.3.3

vscode-graphql-syntax
Gra
1.3.8

hjson
lak
0.3.2

dotenv
mik
1.0.1

jupyter-keymap
ms-
1.1.2

remote-ssh
ms-
0.115.0

remote-ssh-edit
ms-
0.87.0

remote-wsl
ms-
0.88.5

remote-explorer
ms-
0.4.3

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
vscaat:30438848
c4g48928:30535728
azure-dev_surveyone:30548225
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
0ee40948:31013168
dvdeprecation:31068756
dwnewjupyter:31046869
newcmakeconfigv2:31071590
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
wkspc-onlycs-t:31132770
wkspc-ranged-t:31151552
cf971741:31144450
iacca1:31171482
notype1cf:31157160
5fd0e150:31155592
dwcopilot:31170013

This is not the first time I've run into this bug; previously, I found that particular combinations with \n led to odd behavior with Global Search. But this is the first time I drilled down into it and experimented, rather than just working around it as quickly as possible.

prob need to use \r?\n instead
open files are normalized to \n
while file search looks at the og file
which is \r\n in Windows

What exactly is your search term?

@roblourens Why would Whole Word results vary by whether the file was open or not?  Using \r?\n or \n does not change the behaviour reported by @mb-realpage.

Sorry, I skimmed a bit and thought it was a slightly different issue.
Any differences in results between open and closed files comes down to the fact that they are searched differently- via searching the editor text model or using ripgrep. See https://github.com/microsoft/vscode/wiki/Search-Issues#notes-on-regular-expression-support for some details.


Create a test plan

Even though the Bookmarks app is pretty limited in functionality, it would make sense to have a manual test plan fo the basic UI functionality so it's harder to regress when adding new features.

Rudimentary test plan is now in place. See<PHONE_NUMBER>51d848531afc20bf82531b58aad47e.


File content before upload

On a current project I need to access the file contents before upload to check for confidential informations and to replace it by an encrypted version of itself.
I also need to access the content of a received file to decrypt it on browser side.
Is it possible in one of the provided events to access and modify the content of the files?

Hello @theWeird ,
Sure, take a look on:

.pipe()
Other option is to get File contents via FileReader , see example. Modify it to meet your needs, and then convert it to base64 with .readAsDataURL() method. And upload file as base64

pipe is exactly what I need.
Thank you for your suggestion.

Hi @theWeird ,
I'm glad it fits your needs.
If you don't mind - share your sample code with .pipe() usage to our wiki.
Please, support this project by:

Star on GitHub
Star on Atmosphere


Process Hierarchy Model

Design

Please change the ordering of processes in the connect process window to Last modified on top, and the oldest on the bottom.

It will be more convenient if user can choose multiple processes to connect on one card, so they don't need to do it multiple times.

When linked process changed name, it should be updated in the current process.

@sir-larsen

@QiJin-Bouvet This is ready for test. Note that the issues you have mentioned is still being fixed on the backend: #803

Tested with commit a3b7c9f in TEST.

Text should shown in Italic font instead of  and
No chips shown for selected user/label in filter
when no process match filter, a grey filed shown

@QiJin-Bouvet Ready for test. Please make sure that infinite loading and the other form steps works correctly here aswell since I did make some changes to that part of the code.

Tested with commit 13b3a96 in TEST.
Fixed:

Text between < i > and < / i > is shown as bold
selected user/label shown as chips in the top left corner
UI improved when no matched process found
Need to fix:
Horizontal and vertical scrolling bar shown at the step before confirming connection
Still not able to fill all fields of a SubActivity/Waiting card at the same time

For me, the scrollbars are always there. But the thumb is only available when there is really need to scroll.

There is a bug that causes the process to not center when opening the "Connect Process" window in the linked processes page in chromium-based browsers. I suspect upgrading to the newest reactflow version will fix this. Here is a related patch note.

@QiJin-Bouvet Ready for test

Tested with commit c1ae181 in TEST env.
Process Hierarchy Model works fine.
Content been removed problem will be solved in issue #832


PickPhotoAsync PhotoSize,CompressionQuality and CustomPhotoSize not working in UWP

Please fill out either the bug or feature request section and remove whatever section you are not using.
Bug
var file =  await CrossMedia.Current.PickPhotoAsync(new PickMediaOptions
{
PhotoSize = PhotoSize.Custom,
CustomPhotoSize = 80
});
The above code does not apply the mentioned settings to the picked photo. The output is same as the original picture (Dimensions and Size).
Version Number of Plugin: 2.6.1
Device Tested On: WIndows 10 PC (1607-14393.693)
Simulator Tested On:
Expected Behavior
Actual Behavior
Steps to reproduce the Behavior
Use the above code I've mentioned.
Feature Request:
Please fill in what you would like

That is correct, they are not supported, only iOS and Android

Only photo size is supported and adjusts the resolution: https://github.com/jamesmontemagno/MediaPlugin/blob/master/src/Media.Plugin.WindowsStore/MediaImplementation.cs#L144

Thanks James for the clarification. Is UWP support coming or is it left out?

I have no plans currently, however anyone can add support for the features.


DOCX -> CiceroMark

It would be useful to be able to create contracts from existing MS Word DOCX files.
https://github.com/mwilliamson/mammoth.js#readme
https://pandoc.org
This should be integrated with #145

@dselman Can we consider this issue closed in #153 ?

Superseded by: https://github.com/accordproject/markdown-transform/issues/247


Tui input widget

Add widgets to the tui renderer, and a general purpose cursor struct for any renderers.
Todo:

[x] Textbox

[x] Basic Keyboard Editing

 Insert Mode

[x] Mouse Support

[x] size

[x] maxlength

[x] Use terminal cursor

[x] Checkbox
[x] Password

[x] Hide characters

[x] Button

[x] onclick

[x] Number

[x] Limit to only numbers

[x] Up/Down with arrow keys

[x] Up/Down buttons

[x] Range

[x] Left/Right with keyboard

[x] Mouse Support

[x] Min/Max

[x] Step

Depends on:
#400
#390
Demo:
https://user-images.githubusercontent.com/66571940/167262687-a8423500-00ae-4ac6-883a-899b59b646f8.mp4
Future work: pass through global attributes

This is so exciting!!!!!!!!!!

This is awesome, can't wait for this to be merged.

Do you need any help resolving conflicts here?

Do you need any help resolving conflicts here?

I've copied over most of the changes to a up to date branch. I need to update some things that changed with templates but hopefully I can finish up tomorrow

Conflicts fixed, but this branch has a bunch of files marked as changed that should not be. #646 has the same changes recreated from master


zasm: 4.2.6 -> 4.4.7, fix darwin build and cross-compilation

Motivation for this change
Package version bump and cross-compilation fix (test with nix build .#pkgsCross.raspberryPi.zasm)
@SuperSandro2000 please check to see if it's still broken on aarch64.
Things done

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

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

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

Result of nixpkgs-review pr 113402 at cf708951 run on x86_64-linux 1

1 package failed to build:

zasm

2 suggestions:

warning: missing-phase-hooks
configurePhase should probably contain runHook preConfigure and runHook postConfigure.
Near pkgs/development/compilers/zasm/default.nix:24:3:
   |
24 |   configurePhase = ''
   |   ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/compilers/zasm/default.nix:35:3:
   |
35 |   installPhase = ''
   |   ^

It builds for me on darwin.

@GrahamcOfBorg build zasm

[x] Build for me on NixOS x86-64

@siraben Does this build for you still on MacOS? I tried to build this via nixpkgs-review on Darwin and it failed for me.
❯❯❯  nixpkgs-review pr 113402 --eval local
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/113402/head:refs/nixpkgs-review/1
remote: Enumerating objects: 7, done.
remote: Counting objects: 100% (7/7), done.
remote: Compressing objects: 100% (2/2), done.
remote: Total 7 (delta 4), reused 7 (delta 4), pack-reused 0
Unpacking objects: 100% (7/7), 1.10 KiB | 188.00 KiB/s, done.
From https://github.com/NixOS/nixpkgs
   478c52b7e54..f1bb6330342  master                -> refs/nixpkgs-review/0
 + c4a3e021a9a...a4e2d7e0c60 refs/pull/113402/head -> refs/nixpkgs-review/1  (forced update)
$ git worktree add /Users/marek/.cache/nixpkgs-review/pr-113402/nixpkgs f1bb633034222cc151d92dffe9dba438016c9e68
Preparing worktree (detached HEAD f1bb6330342)
Updating files: 100% (24454/24454), done.
HEAD is now at f1bb6330342 Merge pull request #113378 from siraben/knightos-build-graph-reduction
$ nix-env -f /Users/marek/.cache/nixpkgs-review/pr-113402/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit a4e2d7e0c60ae6c27affd9073d1dac3b13497fe0
Automatic merge went well; stopped before committing as requested
$ nix-env -f /Users/marek/.cache/nixpkgs-review/pr-113402/nixpkgs -qaP --xml --out-path --show-trace --meta
1 package added:
zasm (init at 4.4.7)

warning: ignoring the user-specified setting 'experimental-features', because it is a restricted setting and you are not a trusted user
$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /Users/marek/.cache/nixpkgs-review/pr-113402/build.nix
warning: ignoring the user-specified setting 'experimental-features', because it is a restricted setting and you are not a trusted user
warning: ignoring the user-specified setting 'sandbox', because it is a restricted setting and you are not a trusted user
hash mismatch in fixed-output derivation '/nix/store/0dn4khwiycs6izfdxxfywjq1lk18904x-source':
  wanted: sha256:0rpz9x62njvm7fr4x2qc6w9didb5glx5f3amnb3bwjrjj23dxxir
  got:    sha256:0lfql6vmalsqljwjkv0wpxlcgr63qnj3rwh0vnq5r9jqcbkzq5sw
cannot build derivation '/nix/store/yix1xyra1izkiz5jp4j78k87wrkamm9w-zasm-4.4.7.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/258saj44qw17qdwaz8cbbdh21jf3dvr2-env.drv': 1 dependencies couldn't be built
error: --- Error ------------------------------------------------------------------------------------------------------------------------------------------------------- nix
build of '/nix/store/258saj44qw17qdwaz8cbbdh21jf3dvr2-env.drv' failed

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/113402

1 package failed to build:
zasm

warning: ignoring the user-specified setting 'experimental-features', because it is a restricted setting and you are not a trusted user
error: --- Error ------------------------------------------------------------------------------------------------------------------------------------------------------- nix
build log of '/nix/store/yix1xyra1izkiz5jp4j78k87wrkamm9w-zasm-4.4.7.drv' is not available
$ nix-shell /Users/marek/.cache/nixpkgs-review/pr-113402/shell.nix

I think the change of sha256 fixed NixOS but broke MacOS?

@siraben Does this build for you still on MacOS? I tried to build this via nixpkgs-review on Darwin and it failed for me.
I think the change of sha256 fixed NixOS but broke MacOS?

You are right, the build fails for me now
➜  nixpkgs git:(zasm-xcompile) ✗ nix build .#zasm; tb
error: hash mismatch in fixed-output derivation '/nix/store/pnrhmr65hbrckdybzfay81cc2rf7whch-source.drv':
         specified: sha256-OfbehpAyS77GslUNVzp9ZbXYEjcMi06yO3VLK0xP/2Y=
            got:    sha256-XBf852JYplyw3QDyPKTFw+THaL8c7Cm5pFhTVbeh2FE=
error: 1 dependencies of derivation '/nix/store/yix1xyra1izkiz5jp4j78k87wrkamm9w-zasm-4.4.7.drv' failed to build

I vaguely remember that I had some issues with this as well when I was originally packaging this and that's why I ended up restricting platforms so much. Anyway I think we should try to make this work at least for x86 Darwin and Linux.

Perhaps fetching the tarball will be more deterministic?

I believe I have a compromise. libs-src will be fetched with fetchFromGitHub but src will be fetched with fetchurl.

Correction: that will not solve the issue since 1. GitHub changes the tarball generation from time to time and 2. fetchFromGitHub already does a fetchzip.

Ok should be good now. The culprit was the Test folder in the repository which had files with weird names.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 113402 run on x86_64-darwin 1

1 package built:

zasm

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
zasm:

configurePhase should probably contain runHook preConfigure and runHook postConfigure.
Near pkgs/development/compilers/zasm/default.nix:24:3:
   |
24 |   configurePhase = ''
   |   ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-phase-hooks.md
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/compilers/zasm/default.nix:35:3:
   |
35 |   installPhase = ''
   |   ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-phase-hooks.md

@ofborg eval

NIXPKGS_ALLOW_UNSUPPORTED_SYSTEM=1 nix-build . -A pkgsCross.aarch64-multiplatform.zasm
configuring
building
g++ -c -m64      -pipe -Os -Wall -DNDEBUG -DQT_NO_DEBUG -D_REENTRANT -std=c++11 -Wno-multichar -I. -I.. -I../Source -I../Libraries -o Error.o ../Source/Error.cpp
make: g++: No such file or directory
make: *** [Makefile:64: Error.o] Error 127
builder for '/nix/store/vds63bafys60fdrb4i37nijmx1sw5q9x-zasm-4.2.6-aarch64-unknown-linux-gnu.drv' failed with exit code 2
error: build of '/nix/store/vds63bafys60fdrb4i37nijmx1sw5q9x-zasm-4.2.6-aarch64-unknown-linux-gnu.drv' failed


Rename RegisterBank to RegisterBlock

Rename RegisterBank to RegisterBlock
Collections of registers logically colocated together are currently
called "Register Banks". This term can introduce confusion because it
may be conflated with "Register Banking" defined various ARM specs[^1]:

Register banking refers to providing multiple copies of a register at
the same address.

Instead, industry specs use a variety of names, from "cluster" in ARM
KEIL's CMSIS/SVD format to "file" in IP-XACT (IEEE 1685). These names
each have pros and cons, but overall "block" seems to have the least
ambiguity and draws a nice parallel to IP blocks, which typically have
their own MMIO space.
[^1] e.g. ARM Generic Interrupt Controller Architecture Version 2.0,
https://developer.arm.com/documentation/ihi0048/latest/
Fixes #67

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test


General code improvements.

Hello, I contributed some improvements.

Cover edge case when only out of order activities are present https://github.com/skilldeliver/revolut-stocks/blob/main/libs/parsers/revolut.py#L165
Using pipenv for dependencies workflow
Using black for formatting
Using flake8 for linter

Thank you for the PR. Unfortunately, I can't see much improvement in the proposed changes. Docker is already providing an isolated environment, so I don't see the reason for introducing an additional one.


New playbook backend: VMWScript engine + example playbook

This PR

Adds a new vmwscript backend for playbooks (see pkg/cli/backend/vmwscript)
Adds support for print-only and test runs so that it's easy to see the input to the backends without running it.
Adds a playbook based on the examples in pkg/x/vmwscript/examples.

README.md becomes the help text of the playbook CLI
Two commands indexed by index.yml: building DockerEE template and launching a swarm.

See examples/playbooks/vmwscript/README.md for documentation on how to use.
Signed-off-by: David Chung<EMAIL_ADDRESS>

Ah I forgot to add that...  just need a new flag so the user can just specify it like vcenter URL... will add that before merge.  Thanks!


Add no warnings to slice variation test

Description
Added no warnings check to Variation tests, for early catch of errors.
Use case
Sometimes we have warnins at variation tests, that are errors, but still tests pass. So we need to add no warnings check
Benefits
Early and automatic bug catch
Possible Drawbacks
This repo will fail until versions are fixed
Testing
it is test changes
If so, do the tests pass/fail?
yes

@nwillhoft  thanks a lot, i didnt notice the commented line. Just fixed.


ENI mode attaches secondary ENIs in dubious subnets by default

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The Cilium AWS ENI IPAM mode will, by default, attach ENIs in the least full subnet whose VPC/AZ matches the subnet of the VPC/AZ of the primary ENI. When running in an EKS cluster with both public and private subnets, this means that Cilium is quite likely to attach an ENI in the "wrong" subnet: i.e., an ENI in a public subnet on a private node, or an ENI in a private subnet on a public node. This causes all sorts of surprising connectivity issues for the pods when a node gets full enough that pods start getting attached to the secondary ENI.
There are several other folks on the community Slack who have run into this:

https://cilium.slack.com/archives/C53TG4J4R/p1654800394686379
https://cilium.slack.com/archives/C1MATJ5U5/p1652110014858529

Using the subnet-ids or subnet-filters configuration options is not a workaround, AFAICT, because there is no set of subnets that is valid for all nodes. We need the public nodes to consider only the public subnets and the private nodes to consider only the private subnets.
Cilium Version
v11.4
Kernel Version
Linux ip-10-2-68-192.ec2.internal 5.4.190-107.353.amzn2.x86_64 #1 SMP Wed Apr 27 21:16:35 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
v1.22.10-eks-84b4fe6
Sysdump
No response
Relevant log output
No response
Anything else?
I hacked up a version of Cilium to unconditionally have the desired behavior: https://github.com/MaterializeInc/cilium/commit/49ca0ca4fe129dbda4861eb82879e1d93909a1c4. The patch is quite simple and has the desired effect:
diff --git a/pkg/aws/metadata/metadata.go b/pkg/aws/metadata/metadata.go
index e109b62478e..ea119b54111 100644
--- a/pkg/aws/metadata/metadata.go
+++ b/pkg/aws/metadata/metadata.go
@@ -39,7 +39,7 @@ func getMetadata(client *imds.Client, path string) (string, error) {
 }

 // GetInstanceMetadata returns required AWS metadatas
-func GetInstanceMetadata() (instanceID, instanceType, availabilityZone, vpcID string, err error) {
+func GetInstanceMetadata() (instanceID, instanceType, availabilityZone, vpcID, subnetID string, err error) {
 	client, err := newClient()
 	if err != nil {
 		return
@@ -64,6 +64,11 @@ func GetInstanceMetadata() (instanceID, instanceType, availabilityZone, vpcID st
 	if err != nil {
 		return
 	}
+	subnetIDPath := fmt.Sprintf("network/interfaces/macs/%s/subnet-id", eth0MAC)
+	subnetID, err = getMetadata(client, subnetIDPath)
+	if err != nil {
+		return
+	}

 	availabilityZone, err = getMetadata(client, "placement/availability-zone")
 	if err != nil {
diff --git a/pkg/nodediscovery/nodediscovery.go b/pkg/nodediscovery/nodediscovery.go
index b59d2087381..09194ac79a9 100644
--- a/pkg/nodediscovery/nodediscovery.go
+++ b/pkg/nodediscovery/nodediscovery.go
@@ -461,7 +461,7 @@ func (n *NodeDiscovery) mutateNodeResource(nodeResource *ciliumv2.CiliumNode) er
 	case ipamOption.IPAMENI:
 		// set ENI field in the node only when the ENI ipam is specified
 		nodeResource.Spec.ENI = eniTypes.ENISpec{}
-		instanceID, instanceType, availabilityZone, vpcID, err := metadata.GetInstanceMetadata()
+		instanceID, instanceType, availabilityZone, vpcID, subnetID, err := metadata.GetInstanceMetadata()
 		if err != nil {
 			log.WithError(err).Fatal("Unable to retrieve InstanceID of own EC2 instance")
 		}
@@ -479,6 +479,7 @@ func (n *NodeDiscovery) mutateNodeResource(nodeResource *ciliumv2.CiliumNode) er
 		// determine the appropriate value to place inside the resource.
 		nodeResource.Spec.ENI.VpcID = vpcID
 		nodeResource.Spec.ENI.FirstInterfaceIndex = getInt(defaults.ENIFirstInterfaceIndex)
+		nodeResource.Spec.ENI.SubnetIDs = []string{subnetID}

 		if c := n.NetConf; c != nil {
 			if c.IPAM.MinAllocate != 0 {

I imagine to submit this as a PR we'd need to make this behavior toggleable with an option like constrain-to-primary-eni-subnets: {true,false}—plus tests. I'm afraid I won't have the time to do that, but hopefully someone else can run with this.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think need new crd ip pool, that controll which subnet is used.

I agree that we can make the selection a bit smarter and picking the same subnet as the primary ENI is probably a good choice as well.
But I don't think I understand why using a filter in spec.eni.subnet-tags is not a feasible here. You can tag all the relevant subnets, and you can also define the tag filter per node if you use a node-specific CNI configuration:
https://docs.cilium.io/en/v1.11/concepts/networking/ipam/eni/#eni-creation

What do folks running on AWS normally do to customize the CNI per node? Run another DaemonSet that installs the config? Bake a custom AMI per node group?

Besides the globally shared CNIconfig (which can be deployed via Helm), users who are using a custom CNI config per node group seem to relying on cloud-init or the overrideBootstrapCommand in eksctl.
But yes, I agree that it is a bit fiddly.
I would not want Cilium the actually set the nodeResource.Spec.ENI.SubnetIDs based on the auto-detection (since this will prevent the operator from picking different subnets if the initial one runs out of IPs), but having a mechanism which makes the operator prefer the primany ENI's subnet if nothing else is specified does sound like very reasonable behavior to me.

Not stale. Can we disable the stale bot on this issue?


[Feature]: Phi-3 vision -- allow multiple images as Microsoft shows can be done

🚀 The feature, motivation and pitch
i.e. instead of this:
https://github.com/vllm-project/vllm/blob/main/vllm/entrypoints/openai/serving_chat.py#L138-L140
allow multiple images.
Idea is that many models trained for 1 image actually work well with multiple, and blocking usage inhibits exploration of what models are capable of.
E.g. would be good for microsoft/Phi-3-vision-128k-instruct
In HF transformers, Phi-3 handles multiple images just fine.  I've used it just fine as well.
It's also an officially supported task from Microsoft:
https://github.com/microsoft/Phi-3CookBook/blob/main/md/03.Inference/Vision_Inference.md#3-comparison-of-multiple-images
Alternatives
None
Additional context
openai.BadRequestError: Error code: 400 - {'object': 'error', 'message': "Multiple 'image_url' input is currently not supported.", 'type': 'BadRequestError', 'param': None, 'code': 400}

This is on our roadmap in #4197, we will work on that after supporting dynamic image size and streamlining the configuration arguments.


Push Notifications for Mendix 9

As discussed with @ConnorLand: This is work in progress, please do not merge. New documentation and docs structure for (push) notifications will be added here.

I also need to update the spaces.yml doc with the new Frontend space.


[Kusto] Polish the API

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run net - kusto - mgmt

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Kusto

/azp run net - kusto - mgmt


xcbuild: 0.1.2-pre -> archived, fix build

Update to the last available version as the project has been archived, also fix the build.
Previous failure:
https://hydra.nixos.org/build/142008015/nixlog/2
[...]
FAILED: Libraries/plist/CMakeFiles/plist.dir/Sources/Format/Encoding.cpp.o
/nix/store/zzvq5qwlm2xikawfqxb0q8gl2bw391a9-gcc-wrapper-10.2.0/bin/g++ -Dplist_EXPORTS -I/nix/store/pj89fimswl2c12dazdfy8cyka7037s6r-libxml2-2.9.10-dev/include/libxml2 -I../Libraries/plist/Headers -I../Libraries/plist/PrivateHeaders -I../Libraries/libutil/Headers -I../Libraries/ext/Headers -std=c++11 -fno-rtti -fno-exceptions -O3 -DNDEBUG -fPIC -Wall -Werror -Wno-multichar -Wno-sign-compare -fdiagnostics-color -MD -MT Libraries/plist/CMakeFiles/plist.dir/Sources/Format/Encoding.cpp.o -MF Libraries/plist/CMakeFiles/plist.dir/Sources/Format/Encoding.cpp.o.d -o Libraries/plist/CMakeFiles/plist.dir/Sources/Format/Encoding.cpp.o -c ../Libraries/plist/Sources/Format/Encoding.cpp
../Libraries/plist/Sources/Format/Encoding.cpp: In static member function 'static std::vector<unsigned char> plist::Format::Encodings::BOM(plist::Format::Encoding)':
../Libraries/plist/Sources/Format/Encoding.cpp:75:5: error: 'abort' was not declared in this scope
   75 |     abort();
      |     ^~~~~
../Libraries/plist/Sources/Format/Encoding.cpp: In function 'Endian EncodingEndian(plist::Format::Encoding)':
../Libraries/plist/Sources/Format/Encoding.cpp:105:5: error: 'abort' was not declared in this scope
  105 |     abort();
      |     ^~~~~
[...]

Motivation for this change
ZHF: #122042
@jonringer (ping meant for @NixOS/nixos-release-managers)
Things done

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

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

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

Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

xcbuild
xcbuild6Hook
xcbuildHook
xcodebuild6

@copumpkin @matthewbauer pinging you as you are listed as maintainers.

Oof, looks like this causes a world-rebuild on darwin. Anything we can do to prevent this, or should I just target staging instead?
Maybe it makes sense to move the import into the #if defined(__linux__) block instead? But abort() is in <cstdlib>, that should also hold true for darwin. I guess it's included through another header on darwin, but not on linux.

Result of nixpkgs-review pr 123106 at c70d6fb5 run on aarch64-linux 1

2 packages built successfully:

xcbuild (xcodebuild)
xcodebuild6

As I just saw in aliases.nix, other archived projects have instead completely been removed from nixpkgs: https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/aliases.nix#L362
Should we keep this alive, or remove it instead? Don't know what the policy here is.

Since that was added a year ago, I would just remove it.
Thanks for doing this :)

@veprbl the package just throws an error for users because of the alias. What would you rather have?

@jonringer The original question was whether we should remove xcbuild because it may become abandonware, and IMO the general answer should be "it depends". And, in this case we can't remove xcbuild unless we find something to replace, as many things depend on this to build.

@jonringer The original question was whether we should remove xcbuild because it may become abandonware, and IMO the general answer should be "it depends". And, in this case we can't remove xcbuild unless we find something to replace, as many things depend on this to build.

sure, but anyone on darwin trying to build one of those packages will be met with an error, and that error has been there for the past year

@jonringer What error are you talking about. It builds just fine.

apparently these PRs are bleeding together.... there's no error

apparently these PRs are bleeding together.... there's no error

Actually there is probably still one mentioned in the OP https://hydra.nixos.org/build/142008015, but that is on x86_64-linux (likely because of the gcc10). If the version bump fixes it, we should do it. Otherwise, it's fine to mark xcbuild as broken on Linux.

Actually there is probably still one mentioned in the OP https://hydra.nixos.org/build/142008015, but that is on x86_64-linux (likely because of the gcc10). If the version bump fixes it, we should do it. Otherwise, it's fine to mark xcbuild as broken on Linux.

Yes, the build I wanted to fix is the x86_64-linux build. The version bump alone does not fix the build, the small patch is also required. If we mark it as broken on linux and don't want to include a patch, we could instead also remove the x86_64-linux version, because the upstream repo is archived, so the build is not expected to be fixed upstream. Otherwise we fix the build for linux with the small patch, and keep the linux version around as well, then we just have to deal with the world-rebuild on darwin.
What's the preferred solution?

/rebase staging