Is there an Authentication Strategy Roadmap For Nitro File-Based API?

Describe the feature
Is there a plan for the addition of route guards to the Nitro API framework?
I think this could be done in a similar manner to  vue-class-component's custom decorators. It would be really cool to add a  @Auth or  @Auth({role: "Admin"}) above a defineEventHandler() to take care of the permissions for a handler.
I am transitioning a Nuxt 2 app with an Express API to Nuxt 3, and have noticed that the Nuxt 3 docs do not have examples of authentication, that is the Nitro File Based API docs has no authentication strategies, patterns, or reference to an authentication module that one could use.
I see some libraries like @nuxt-alt/auth have been adapted, but they are not official, and nor do they support basic auth.
My previous Nuxt 2 app used custom JWT auth. It is a requirement. It appears that the current naive solution is as follows:

Create a login endpoint that gives user a JWT.
Add a server middleware handler server/middleware/myAuthHandler.ts that checks the JWT and sets an appropriate prop on the event
Given that the server middleware according to the docs cannot cut the request short when auth fails, literally add to each file-based endpoint's 'eventHandler' that requires auth, a check on the event (I mean the whole one file per endpoint is tedious enough).

Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

Looks like @sidebase/nuxt-auth demonstrates the defacto auth technique for Session Access and Route Protection. I am using supabase and so implement auth on the endpoints in a similar manner. But still, it would be tedious (AFAIK) to manage large numbers of endpoints without the likes express.js or asp.net route/role based auth schemes/ middleware that they support. I wrote a post about using Supabase auth with Nitro API: https://intellitect.com/blog/nuxt-3/


Update package.json

Is this not the development branch ?
https://github.com/Narzerus/angular-permission/pull/138#issuecomment-179069465

Plz move your PR to development branch.

It's funny, I saw your 31a80cdc7bd2b7a8f4d6c6d6b0321c0f985bef4a, if you wanna use your own, you don't need to ask me to move on development branch.


Close active apps?

Hey!
More of a functionality question, is there a similar functionality to "see" the active apps and close them. Basically the functionality described here in "Close All Apps". Currently when I press my 🔲 button to pull up all apps on my Pixel 6, nothing happens!

More information:
It does work in my profile that has google play services installed, but does not work in my google-less profile


Is it feasible to generate a RotateKeyGen without specifying EvalBootstrapSetup({4, 4}, {0, 0}, 16384)?

Is it feasible to generate a RotateKeyGen without specifying EvalBootstrapSetup({4, 4}, {0, 0}, 16384)?

However, the following error occurred at res = m_cc->EvalAdd(res, m_cc->EvalRotate(sum, 1024));.

This happens because $1024 \neq -1024$. I think that the actual rotation for $-1024$ is $16384-1024=15360$

However, after adding context->EvalBootstrapSetup({4, 4}, {0, 0}, 16384);, this issue did not occur. Apologies, I am a beginner in FHE.

No worries, I am still learning, too :-P.
The bootstrapping its self uses a lot of rotation keys, perhaps that setup generates the key also for $15360$.
To confirm, check the memory when you

generate the bootstrapping keys
vs
generate the bootstrapping keys + rotation keys 15360.

If they are the same, then the rotation key for $15360$ is already included in the ones used by the bootstrapping

Thank you very much!

Feel free to re-open if you have issues :)

Why is the error from layer 3 to FC so large?


Background palette is not being displayed correctly on Lynx

Hello, again;
I'm having a weird problem where a background image is not being displayed correctly on Lynx, even though the image is using the same palette as the demo images:

When running the application on Lynx, the image above is displayed with the wrong palette:

Checking both the image and the example images on GIMP, they seem to be using the exact same palette.
Does not seem to be an initialization problem, since if I replace the image with one of the demo images, it works:

Would this be a bug, or would there be some small little detail that I am missing?

Hi Haroldo. I have checked the image and found the problem.
While the 16 first colors are correct, the palette is actually 256 colors wide (with remaining colors assigned to white). I re-assigned the 16 colors lynx palette to image, and it works now.
Did you generate the image with your conversion script? If so, the code should ensure that palette is 16 colors wide.

Thanks for the tip; I'll have to take a look at it later; one possible problem that could happen is that the version of PIL that comes with 8bit-Unity does not seem to support indexed images with less than 16 colors.


[CPROD-736] Signing calls to management canister

I personally don't like using rebased-2022-02-17 branch because of these commits

But when I tried do a clean update of the ic fork, for some reason ledger_canister could not be found in remote (the discussion).


CodeMirror syntax highlight error

It looks like CodeMirror tries to highlight anything that seems to be html.

https://gist.github.com/211179a4846abab9178a3b85cfb84ace

We can disable all highlight to keep consistent.

@hackjutsu
This is caused by the mode setting of CodeMirror. If it is set correctly according to the code, its style will be fine. If not, while its auto mode detecting is not robust, highlighting could be wrong.
You can currently set mode: 'plaintext' to disable all highlighting. I'm still trying to implement changing mode according to filename.

Thanks~ The syntax highlight for CodeMirror is a nice-to-have feature, but not a must-have. I might want to stick to plain text before this feature becomes robust.

Fixed: https://github.com/hackjutsu/Lepton/commit/46abe43edd59650693bba1f961b16a905380cc0a


Bring back block node type display

When switchTextType is disabled we still want to show the human readable label of the active block node.
We should really use that space, as we already did.

Fixed via #264


mysql-replication support for PyMySQL>0.10.0 was introduced in v0.22

This might not be the way to safeguard against it, but for illustration, my upgrade of pg_chameleon to 2.0.18 pulled in version 0.21 of mysql-replication and 0.10.1 of PyMySQL
This resulted in charset_to_encoding errors which were addressed in version 0.22 of mysql-replication.
A manual bump of mysql-replication==0.22 fixed this for me.
If the requirement could be expressed in the package that would be nice.

thanks, I'm merging the PR.
I'm about to file a bug to the mysql-replication project as it seems there are still issues with the query event in the last release 0.30


DOC: <Issue related to /concepts/tracing>

Does this mean that after 400 days links to traces from experiments will return 404? Is there any way to retain traces beyond 400 days while still being able to use all of LangSmith's features?

Thanks for the question! I've forward this to the person in charge of this. Feel free to @ me if they don't follow up here in a reasonable timeframe.

Not at present


Players in incorrect mode when returning to Lobby

When players return to the lobby, they have all their inventory still and are in survival mode. This allows them to place items and break items in the lobby causing the game to get broken often on servers

You can turn on strict mode using the red sign in the room after card generation. This makes sure only operators can generate cards and start games, as well as putting everyone in adventure mode when in the lobby.


Update active-directory-application-manifest.md

Removed html tags from markdown

@slamidtfyn : Thanks for your contribution! The author, @CelesteDG, has been notified to review your proposed change.

@slamidtfyn - I'm sorry your PR was not handled promptly. We're trying to get current, and this article no longer is present in the repo, so I'm closing.


LightningCLI does not accept DDPPlugin as trainer argument

🐛 Bug
LightningCLI does not accept DDPPlugin as trainer argument
Code:
from pytorch_lightning.utilities.cli import LightningCLI
from pytorch_lightning import LightningModule, LightningDataModule
from pytorch_lightning.plugins import DDPPlugin
cli = LightningCLI(
    LightningModule,
    LightningDataModule,
    subclass_mode_model=True,
    subclass_mode_data=True,
    trainer_defaults={'plugins': [DDPPlugin(find_unused_parameters=False)]}
    #trainer_defaults={'plugins': DDPPlugin(find_unused_parameters=False)}
)

Error:
trainer.py: error: Configuration check failed :: Parser key "trainer.plugins": Value "[<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>]" does not validate against any of the types in typing.Union[typing.List[typing.Union[pytorch_lightning.plugins.base_plugin.Plugin, pytorch_lightning.plugins.environments.cluster_environment.ClusterEnvironment, str]], pytorch_lightning.plugins.base_plugin.Plugin, pytorch_lightning.plugins.environments.cluster_environment.ClusterEnvironment, str, NoneType]:
  - Value "<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>" does not validate against any of the types in typing.Union[pytorch_lightning.plugins.base_plugin.Plugin, pytorch_lightning.plugins.environments.cluster_environment.ClusterEnvironment, str]:
    - Expected an str or a Dict with a class_path entry but got "<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>"
    - Expected an str or a Dict with a class_path entry but got "<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>"
    - Expected a <class 'str'> but got "<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>"
  - Expected an str or a Dict with a class_path entry but got "[<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>]"
  - Expected an str or a Dict with a class_path entry but got "[<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>]"
  - Expected a <class 'str'> but got "[<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>]"
  - Expected a <class 'NoneType'> but got "[<pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fbf0fef3dc0>]"

Expected
Accept the argument as described in
https://pytorch-lightning.readthedocs.io/en/stable/benchmarking/performance.html#when-using-ddp-set-find-unused-parameters-false
Colab example
https://colab.research.google.com/drive/1MuOcvfWoujaU76jvtU85Y6jXSRnLLyHF?usp=sharing
Environment

CUDA:

GPU:

NVIDIA GeForce GTX 970

available:         True
version:           11.3

Packages:

numpy:             1.20.2
pyTorch_debug:     False
pyTorch_version:   1.8.1
pytorch-lightning: 1.3.8
tqdm:              4.61.0

System:

OS:                Linux
architecture:

64bit
ELF

processor:         x86_64
python:            3.8.10
version:           #66~20.04.1-Ubuntu SMP Thu Jun 17 11:14:10 UTC 2021

With the version of jsonargparse being released today I think this will not fail anymore. However, looking at this I am thinking that there is something that we might want to change related to plugins. Currently callbacks has the behavior that when added using trainer_defaults they will be persistent. This means if someone provides more plugins in a config file, these are concatenated to the ones defined in trainer_defaults. Also a callback can be added to be persistent but configurable. Should the behavior for plugins be the same?

Should the behavior for plugins be the same?

Yes, although multiple plugins is much more of a niche usecase

Yes, only 1 plugin from each type is supported.

Dear @tasptz,
From trying with the latest version from jsonargparse ?
Best,
T.C

The code snippet in the description does work with the new version of jsonargparse. I only tested --help and --print_config.

Yes, although multiple plugins is much more of a niche usecase

Even though it is a niche use case I think it might be good to be the same as the callbacks. Right now if you do --print_config the yaml includes:
  plugins:
  - <pytorch_lightning.plugins.training_type.ddp.DDPPlugin object at 0x7fe185f6e630>

Which is not a valid value for the config. If it is like the callbacks then the object instance wouldn't appear.


delete key on empty line does not delete lf (3.1pre2)

I think this is new in 3.1, since I don't think I'd be so annoyed otherwise.
In a method, with the cursor on an empty line, if you press the delete key on the keyboard, I expect it to delete that lf or cr or whatever, and remove the empty line. It does not do this; when the cursor is on an empty line, the delete key does nothing.

actually the delete key is generally not working!

Think I'll fix this before I prepare pre-release3. fyi
On Thu, Apr 2, 2020 at 5:10 PM Lisa Almarode<EMAIL_ADDRESS>wrote:

actually the delete key is generally not working!
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/GemTalk/Jadeite/issues/761#issuecomment-608158643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIUI6JB6GK4ZP65ABFBX6F3RKUSQBANCNFSM4L3DY7WQ
.

--
Eric Winger
Staff Engineer
GemTalk Systems

Fixed in 3.1.0pre-release3. This should restore delete key behavior. Had it disabled by mistake except when there was a selection.


Show/hide tray icon

Description of the new feature / enhancement
Just an option to show/hide the PowerToys icon in the tray. Would be pretty nice =D
Scenario when this would be used?
For those who like to have no icon hided in the tray and let only the system icons or for who doesn't like icons in the tray and want to keep PowerToys running.
Supporting information
No response

Duplicate of #9526

Thanks for your suggestion! This has been raised in #9526. To centralize the discussion thread, please go to that thread for discussion.


AbstractArray constructor ignores the global argument

https://github.com/HebiRobotics/MatFileIO/blob/6b7cc2661a20523c3cbc3918977bc50d6e1127dd/src/main/java/us/hebi/matlab/mat/types/AbstractArray.java#L114-L118
This looks like a bug.  What does "global" mean, anyway?

Yes, thank you for pointing out the bug. global is a keyword that defines variables as globally accessible.
https://mathworks.com/help/matlab/ref/global.html
I personally never use it, so it must've slipped my testing.

Wow!  Didn't realize MATLAB had this behavior.  Happy to send a PR for this as well, once #22 gets sorted :)

Reading your link I'm realizing global probably only applies to root-level entries (which makes perfect sense). In that case I think global shouldn't be in the AbstractArray class at all, and should instead be treated like the name attribute.
What do you think about adding a global flag to NamedArray and maybe renaming the class
something semantically similar to MatlabVariable? It'd represent an array with a name and global flag that sits at the root level. This would match MATLAB semantics much better.

It's a good thought, but my gut is to leave it as-is.  Here's a couple reasons:

if a misformed MAT-file happens to have the global flag set in a non-root array, what would you do?  Ignore it?  Throw an error?
looking at the MatFile interface, the only way to get the isGlobal() value is through the Iterable<NamedArray> getEntries().  That's okay, but it's a little awkward for people who care about the isGlobal() but want to access via map or index

It's not crazy to change it, but it does seem like a bit of an impedance mismatch with the file format.  It's too bad that there's a mismatch between the underlying format and the semantic meaning, but there's too many MAT files out there to change it now ;-)

I'm ok with ignoring it. I believe that's what MATLAB is doing, and semantically it doesn't mean anything. For example, what would it mean if the 2nd variable of a cell array were global? There is no way to reference it without the cell array being global.

I assume that there are very few people who will ever care about this, so it probably doesn't need to have a particularly user friendly API. It'd be simple enough to iterate over the entries and create a globals map in user space. However, at this early stage I'm also not opposed to changing the interface to something that better matches (not sure what it'd be yet).

There are quite a few things that are in every array header that only get used in a subset of cases:

name: only used at the root level, otherwise empty
global flag: only used at the root level, otherwise ignored
logical flag: only used for numerical matrices, otherwise ignored
complex flag: only used for numerical matrices, otherwise ignored
number of non-zero-values: only used for sparse matrices, otherwise ignored

So far I've tried to treat the MAT-File format as an implementation detail, and tried to keep the public API close to the actual MATLAB behavior. The globals flag was an oversight on my part.
No MAT file generated by MATLAB should ever have the global flag set on a sub-element, so this would only come up when loading files created by 3rd party libraries that let users set invalid options.

I've committed the 1 line constructor fix. I'll keep the issue open for discussion. The more I think about it, the more I lean towards moving the global flag to the root level.
I'll create a PR for an API proposal.

One trick I'm doing right now is that I can treat a MatFile as a List<NamedArray>, and I can also do that for Struct.  That way I can model the data as a tree of NamedArray, and I can fit everything into that tree, and the code doesn't have to specific to root or child entries of a file.
If NamedArray becomes RootArray with a boolean isGlobal(), then I can't stuff the entire hierarchy into it anymore, and I have to treat children of a scalar struct differently than children of a file.  Or I could stuff everything into RootArray, but it's the same problem where isGlobal() is present everywhere, but only actually matters for root entries.
That's just my usecase though, and I'll be fine either way :)

I may be misunderstanding this, but is there really a difference between the two options? isGlobal() is still present everywhere and always false.
1) NamedArray

getName()
getValue().isGlobal() (always false)
getValue()

2) Variable

getName()
isGlobal() (always false)
getValue()

In Case 1 you could technically create an array that returns true, but IMO returning an incorrect value would be worse than returning a redundant / useless value.

Is there really a difference between the two options?

I don't think so, which is why I'm mildly in favor of the status quo :)

I meant for your use case ;)
I see benefits in following MATLAB semantics and making it harder for users to create invalid files, e.g., Mat5.newCell(1,1).set(0, myGlobalArray).
I'm also not particularly happy with the naming of NamedArray.  Variable may also be a weird name for a class, but it could potentially clean this up a bit, i.e., a MatFile has a local or global Variable that consists of a name and an Array value.
for(Variable variable : matFile.getVariables(){ ... }
I'll just test around with the API a bit. If it doesn't make the API cleaner, I'm ok leaving it as is.

Please take a look at #27. I think it makes the API a bit more readable.
I wasn't sure whether to rename MatFile::getEntries(), MatFile::addArray , Mat5Writer::writeArray etc., so I left those the same for now.
Aside from doing a search-replace from NamedArray to Variable there shouldn't be any other change required to your existing code.

Did another iteration and changed it again to MatFile.Entry. I think this should work well for your use case as well.

I like the MatFile.Entry name, especially in the context of Mat5Writer.writeEntry.  This will work well for my use case, and it definitely improves the public API semantics to better match the in-MATLAB experience.  Good call :)

Merged in #27. Thanks!
I may do another intermediate 0.4 release, but believe this is getting very close to a releasable 1.0.


Pulse switch output changes input.

Hi, I am trying to make a device with 3 buttons and 3 LEDs.
When you click a button the LED next to that button will blink and it will send a report of the button pressed.

This gives me something close to what I want, but when I click a button I get:
Zigbee2MQTT:info  2022-03-27 14:16:07: MQTT publish: topic 'zigbee2mqtt/0x00124b00061df10f', payload '{"linkquality":87,"state_l1":"OFF","state_l2":"OFF","state_l3":"OFF"}'
Then clicking button1 again:
Zigbee2MQTT:info  2022-03-27 14:16:09: MQTT publish: topic 'zigbee2mqtt/0x00124b00061df10f', payload '{"linkquality":94,"state_l1":"ON","state_l2":"OFF","state_l3":"OFF"}'
So it only sends the state not the button that was pressed, so I would need to compare messages to the previous message to see what was clicked.
If I untick the 'link to output' I get the following:
Zigbee2MQTT:info  2022-03-27 14:13:10: MQTT publish: topic 'zigbee2mqtt/0x00124b00061df10f', payload '{"action":"single_l1","linkquality":83,"state_l1":"OFF","state_l2":"OFF","state_l3":"OFF"}'
It now has an action I can use to see what button was clicked. Can I have the pulse switch (blinking LED) and the action? Thanks.

No, you cannot have the "action" value for a linked button.
In your case, state_l1 means Button #1, and state_l2 - Button #2.


Cpp formatting fix

Updated the code formatting to cover taintdag. Any changes in C++ code should relate to formatting.

@hbrodin  I think we're good on python typechecks. CI still fails due to polytracker tests, but that's going to be addressed in taint_dag branch.


Not sure how to use it?

First of all thanks for your work!
I was able to apply 1st PR as a patch and build my image with apko_image().
We plan use rules_apko to build repeatable base images.
But I can't realise next things:

How can I generate lock-file to use it in apko_image() to persist repeatability of generated image?
How can I consume output image generated by apko_image() with rules_oci or something else?

It's not completely ready for people to use but here's PR partially demonstrating how it works. https://github.com/chainguard-dev/rules_apko/pull/6

Thank you for clarification! Closing this.


Sign the content of the Agreement message, instead of just the header

Currently, on the Agreement message, only the header is BLS signed. This makes the rest of the message malleable. For safety purposes, it would be better to also sign the contents of the message.

This issue no longer applies, as the header no longer includes BLS signatures.


PhantomJS unable to interpret GUI objects through Xpath

hi,
We have selenium code which is up and running different browser such as Safari, Chrome, FireFox. Our new requirement is to run those test cases on headless mode. Hence we have asked to use PhantomJS. While executing test cases we noticed that PhantomJS is not able to identify objects which has reference through Xpath and test cases failed. We are getting below errors.
[RuntimeError: RuntimeError
(NoSuchElement:7) An element could not be located on the page using the given search parameters.
Problem: Unable to find element with xpath '//*[@id='header']/descendant::div[@class='header-left']/h1'
 Callstack:
 -> element("//*[@id='header']/descendant::div[@class='header-left']/h1")
 -> waitFor("//*[@id='header']/descendant::div[@class='header-left']/h1",35000)

]
[RuntimeError: RuntimeError
(NoSuchElement:7) An element could not be located on the page using the given search parameters.
Problem: Unable to find element with xpath '//*[@id='localCurrencySelection']/select'
 Callstack:
 -> element("//*[@id='localCurrencySelection']/select")
 -> waitFor("//*[@id='localCurrencySelection']/select”,35000)

]
We also tried setSize ()  and client.Pause() with different parameter but none of them are working. Basically it is not identifying the objects at run time
Where as same test cases are working perfectly on real browsers. Our test cases are purely on NodeJS.
Please advice us how to proceed.
thanks,
Amiya

First, please make sure you are using PhantomJS 2.0.  1.9.x has many bugs and is no longer developed.
Second, XPath issues are without question WebKit's fault.  We do not have the manpower to fix WebKit bugs ourselves, or to cherry-pick fixes.  Therefore, if you can reproduce the problem using 2.0, then please try to reproduce it using a nightly build of WebKit.  If it does reproduce using nightly WebKit, please report it to the WebKit developers directly.   Conversely, if the problem does not reproduce using nightly WebKit, then it will eventually be fixed in PhantomJS when we pick up a newer version.  (Unfortunately, there is no schedule for that.)


Version Bump DNSPolicy

What
Move DNSPolicy to v1Alpha2

This issue is stale because it has been open for 60 days with no activity.


Can we keep RD Android running with screen off?

Description

It's similar to this https://github.com/baldurk/renderdoc/issues/1178 but since I'm using an OLED phone, I'd rather keep my screen off than having it displaying a static image while I'm working with RD on my computer...

Environment

RenderDoc version: any
Operating System: Android
Graphics API: Vulkan/GLES

But is it possible to add a flag or something to prevent the killing, like you did in the referenced issue?
Or is there a way to perhaps turn off the text overlays? Currently I just add a black screen in my game on four finger touch, but the texts are still there and pose a risk for burn-ins...

I don't expect so otherwise every app could set that and then nothing would be closed. Keeping the screen on is a way for games/etc to indicate that they are still running even without interactivity. If there is a way I'm happy to review a pull request.
You can disable the overlay if you wish using the in-application API.


Package request: gimp-devel

Project description
This is the 2.99 development branch of GIMP. I have a working build script (but macOS will need some workshopping).
Metadata

homepage URL: https://www.gimp.org/
source URL: https://www.gimp.org/downloads/devel/#source
license: GPL3.0 or later
platforms: linux, darwin

https://github.com/NixOS/nixpkgs/pull/67576


LCOE real for PV + Storage discrepancy in the cash flow 'send to Excel with equations' workbook.

Describe the bug
We noted a difference in the LCOE real as reported in the Summary tab compared to the cash flow workbook created with 'send to Excel with equations'. The values are equivalent when modelling PV only, but when modelling Storage only or PV + Storage, the discrepancy occurs. For our PV + Storage model, the Summary report shows a real LCOE is 16.91 R / kWh. In the excel workbook, the real LCOE is R 2.86. When I model the PV only system with a higher installation cost that matches the cost of the PV + Storage, the real LCOE is 2.53 R/kWh in both the summary report and the excel workbook. The 16.91 R / kWh seems much too high for the PV + storage.
To Reproduce
Steps to reproduce the behavior:

Create a 'PV-Battery, Commercial' model
Simulate
Note the real LCOE on the Summary Tab
Click on the 'Cash Flow' tab
'Send to Excel with equations'
Note the real LCOE on the 'Inputs and Outputs' tab

Expected behavior
I expect the reported real LCOE to be similar, maybe with 5% due to the limited functionality of the excel workbook model, not 5x or so as noted.
Screenshots
See attachments. Note the values are in Rand-cents, not USD-cents
Operating System and Version

OS: Windows 10 Enterprise, version 10.0.19045 19045
SAM Version 2022.11.21, 64 bit, updated to revision 1

Additional context

20230510 S8_pv_storage.xlsx
I attached the summary report and the excel workbook for the PV + storage project.

This discrepancy only seems to occur when I include a 'grid outage' profile profile.

@TheMakiran Thank you for reporting this issue. We'll need to check that the Excel workbook for SAM's "Send to Excel with Equations" feature is working correctly with grid outage data from SAM. In this special case (battery storage with grid outage), the Excel workbook is not replicating SAM's cash flow calculations correctly.

Also note issue with BTM battery O&M costs in Excel spreadsheet (need to check for Fuel Cell and other techs as well): https://sam.nrel.gov/forum/forum-general/4240

Also:
When using Detailed PV-battery model, in SAM cashflow includes battery operating expenses and information about energy discharged annualy:

But spreadsheet generated with your default file do not.

When using merchant model the are present:


Simplified Contributions and Automated development setup using Gitpod

This PR adds a Gitpod configuration file which will automate setting up the developer environment and make it easy for contributors to directly start working without having to worry about installing the dependencies.

Description
Added .gitpod.yml
This file automates installing the required dependencies, build and start the application. The preview is launched once the build has finished, you can go to the open ports tab and launch the application in the browser.

Click on this link to give it a try : https://gitpod.io/#https://github.com/anudeepreddy/carbon
It works great and it would be much better if someone from the team has a closer look at this and provide their reviews. I hope this helps the team and the dev community.

Hey @mfix22,
Rebased on top of master, please do have a look when you find time.


Fixes issue #295: Disallow hyphen at end of identifiers

Also extended test coverage of parser to 100%. (Although DEFCIRCUIT parsing is still broken and needs more unit tests)

No longer relevant. Issues have since been fixed.


Added NaszaKlasa OAuth2 support

Hi,
OAuth2 works fine and after authorization we get valid token. But If we want use it to get data about user from REST service. Each request must be signed with OAuth Consumer Request. The supported signature method is HMAC-SHA1. Also, for POST and PUT requests, Oauth Body Hash is mandatory.
Thats why

That's an odd implementation, but works, for me, thanks!


Preview format

I understand that current format is a default, ISO-consistent format, but it would be nice to have possibility to pass some text (probably translatable) as preview, so it can show labels like

5 hours 0 minutes 15 seconds

Thanks for that lib, regards!

@superdyzio I don't have enough time to work on those suggestions now, but PRs are welcome. However I think it would be better to split one suggestion per issue.

@ShinDarth ok, later I will divide my ideas into separate issues and in free time I'll start working on them - this issue will be related to preview / output format - as stands in title

resolved in #21


Add Preparatory tick box

As discussed at this afternoon's catch up.
Please add preparatory tick box to:

Attribute Terminologies profile
Terms profile
Data element code sets profile

This has been fixed and is on the test environment now.

Look OK, will test


Code Quality: Migrated CopyFileFromApp and MoveFileFromApp to CsWin32

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Part of #14505, #15000

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

I think you can put more two DeleteFileFromApp and DeleteFolderFromApp.

I think you can put more two DeleteFileFromApp and DeleteFolderFromApp.

Let's try to keep these PRs small in scope to make the review easier.


Crash on Firefox 72 on Mac (script is "blocked from extracting canvas data")

Console logs:
Error Blocked http://leaflet.github.io/Leaflet.heat/demo/ from extracting canvas data because no user input was detected. leaflet-heat.js:6:1045
Blocked http://leaflet.github.io/Leaflet.heat/demo/ from extracting canvas data because no user input was detected. leaflet-heat.js:6:1413

If it's any help, I found that it doesn't seem to happen on my Firefox 72.0.2 on macOS Catalina. If you replace http:// with https:// however then it prints errors about mixed active content and the map fails to load but I suppose that's a separate issue.

this thread popped up when I searched on
'Blocked digid.nl from extracting canvas data because no user input was detected.''
taken from the console warning:
'Blocked https://digid.nl/inloggen_app_qr?utf8=✓&app_verification_code[verification_code]=SMJX&button= from extracting canvas data because no user input was detected.'
that is in Nightly: 87.0a1 (2021-02-22) (64-bits) (Ubuntu 20.10)
It blocks me from logging via QR-code on a Dutch government site. (QR code is not displayed)


How to make chained calls to be on new lines

Is there an option to allow >= 3 (or configurable number) calls in a row on self to be wrapped?
a.do_something()
 .another_thing()
 .build();

#2263 sounds related

I would also like for chained calls to be on new lines as shown in @xahon's example.

Closing as a duplicate of #2263


pd.array().tolist() is not recognized

Describe the bug
.tolist() is not recognized as a method for a pandas array. You can compare with generating the stubs using stubgen -p pandas which does provide it in core/arrays/base.pyi:
class ExtensionArray:
  ....
  def tolist(self) -> list: ...

Should pandas-stub support this? Could be something wrong with my typechecker, but I don't see tolist in the corresponding pandas-stubs' pyi.
To Reproduce
import pandas as pd
return pd.array([1,2,3]).tolist()

Please complete the following information:

python version: 3.10.8
version of type checker: fork of pyright. Should be compatible with pyright 1.1.279
version of installed pandas-stubs: <IP_ADDRESS>304

Need to add tolist() to pandas-stubs\core\arrays\base.pyi , with appropriate test.  PR welcome.


Fix default folding value

It seems to default to false, but the docs say it defaults to true.

Hi @leMaik, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Thanks, this is already fixed on master.
https://github.com/Microsoft/vscode/blob/master/src/vs/editor/common/editorCommon.ts#L449
	/**
	 * Enable code folding
	 * Defaults to true in vscode and to false in monaco-editor.
	 */
	folding?: boolean;

@alexandrudima Oh, didn't check it there. When will it land here?

I plan to do it in the following days.


fix: don't split inner products that are already on device memory (PROOF-923)

Rationale for this change
Fix inner product so that it doesn't chunk inner products that are already in device memory
What changes are included in this PR?
Changes inner product so that it doesn't chunk when inputs are already in device memory.
Without this change, some of the tests flake on machines with multiple GPUs.
Are these changes tested?
Yes

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


add the missing author.

This website is based on the Lab Website Template.
See its documentation for working with this site:
https://greene-lab.gitbook.io/lab-website-template-docs

As the citations.yaml file is an automatically generated file; it does not need to be fixed.


strange behavior of quick links and join us in blog page

Description
join us and quick links section shrinks abnormally when hovered over.
I want to work on this issue.
Screenshots

hey @SauravMukherjee44  you can assign this issue to me I can work on it.

@HoneyTyagii @20RitikSingh  Someone else is already working on this #548
Closing the issue.


Superfluous '.' introduced

Browser name: Firefox
Browser version: 54
OctoLinker version: 4.8.0
URL and line number where issue occurs: https://github.com/silverwind/oui/commit/03504e482f50b0c9243123ed9e222b4981ae01ff#diff-2fbe63f0ca81ccc1151eb979b3ff1270L76
Expected behavior: require(".").search
Actual behavior: require(".")..search (note the extra '.')

Thanks for the report, @silverwind! I was able to reproduce the issue with OctoLinker 4.8.0, but not with v4.10.0, using the firefox-open script in package.json. However, v4.10.0 hasn't been released for Firefox yet, due to https://github.com/OctoLinker/browser-extension/issues/351, so I think we should close this issue in favor of that one.

Note that you can use Chrome Store Foxified to install the addon in Firefox from the Chrome Store.

OctoLinker v4.11.0 has been released with the fix and is now available from Mozilla: https://addons.mozilla.org/en-US/firefox/addon/octolinker/versions/?page=1#version-4.11.0

Fix confirmed, thanks!


Fix wrong NHWC shape parameters for cudnn conv2d

No way to test this part of the code due to no schedule.
xD
I think I can add a generic schedule just like NCHW for cudnn.

Here is the code I used for testing. If the existing test case is correct, then there is something wrong with mxnet import. And I also used a pre-train model for testing, but the output is garbage. I assume the shape was mess up somewhere.
data = mx.sym.var('data')
sym = mx.sym.Convolution(data, dilate=(1, 1), kernel=(4, 5), no_bias=False, num_filter=64, num_group=1, pad=[1, 1], stride=(1, 1), layout='NHWC')

arg_params = {}
aux_params = {}

dtype = 'float16'
input_shape = (1, 720, 1280, 3)

shape_dict = {'data': input_shape}
mod, params = relay.frontend.from_mxnet(sym, shape_dict, dtype, arg_params, aux_params)

func = mod["main"]

target = tvm.target.create('cuda -libs=cudnn')
with relay.build_config(opt_level=3):
    graph, lib, params = relay.build(func, target, params=params)

ctx = tvm.gpu(0)
x = np.ones(input_shape).astype(dtype)

m = graph_runtime.create(graph, lib, ctx)
m.set_input(input_name, tvm.nd.array(x))
m.set_input(**params)

m.run()
tvm_output = m.get_output(0)
print(tvm_output.shape)

Correct fix: https://github.com/apache/incubator-tvm/pull/4456


moodle: 3.8.2 -> 3.8.3

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/moodle/versions.
meta.description for moodle is: "Free and open-source learning management system (LMS) written in PHP"
meta.homepage for moodle is: "https://moodle.org/"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.8.3 with grep in /nix/store/wrqvnscgp17lrrhz5avw7prsm2ilspgn-moodle-3.8.3

directory tree listing: https://gist.github.com/0495143b9e812171f5e97ce59855d7cd

du listing: https://gist.github.com/f90811063f8ae0090067266808cf6a5e

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
moodle

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wrqvnscgp17lrrhz5avw7prsm2ilspgn-moodle-3.8.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A moodle https://github.com/r-ryantm/nixpkgs/archive/865f214e33dd61ad2c6371a387c1ccfb10a78d58.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wrqvnscgp17lrrhz5avw7prsm2ilspgn-moodle-3.8.3
ls -la /nix/store/wrqvnscgp17lrrhz5avw7prsm2ilspgn-moodle-3.8.3/bin

Security vulnerability report

Security report (click to expand)

CVEs resolved by this update:
none
CVEs introduced by this update:
none
CVEs present in both versions:

CVE-2019-15536

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- moodle

Maintainer pings
cc @aanderse for testing.

@GrahamcOfBorg test moodle

Changelog looks good and both bots are happy with it.


Codemove and Library exports introduction

This PR is completing the code move of all "Non-node.js" functionality into the matter.js package.
Additionally it introduces an "topic based" export structure for the packages which allows to export the funcions still simplified, but not in a "one export for anything" structure.
Tests are also moved from the matter-node repo to matter.js best possible. Some tests needed to stay in matter-node because they use CryptoNode functionalities and we do not (yet) have a pure JS Crypto lib. Because of the fact that we always run all tests it is made sure that still anything is tested as before. We can move them later too.
The PR is not changing any code beside the move relevant import changes, adjustments to tsconfig, eslint and package.jsons.
I also adjusted the Readmes to describe the exports and also streamline the Readme's a bit.
I think this PR is near to impossible to review using GitHub comparism view, so maybe explore the final code in https://github.com/Apollon77/matter.js/tree/codemove/packages directly.
And check the Readme's at https://github.com/Apollon77/matter.js/blob/codemove/packages/matter.js/README.md and https://github.com/Apollon77/matter.js/blob/codemove/packages/matter-node.js/README.md
The PR also incorporates the PR #113 from @lauckhart
fixes #62

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Apollon77:x: AlCalzoneYou have signed the CLA already but the status is still pending? Let us recheck it.

I was able to build this PR, run node ./dist/DeviceNode, pair the device with the Alexa App and see on & off commands arrive in terminal. The on & off commands continued to work after restarting matter-node, so persistence is working as well.
One tiny grammar error in both readme's: the L in cluster should be lower case : )  Exports Matter CLuster definitions and Default Cluster-Handlers

Typo fixed

PR enhanced by some tsconfig hints after feedback from @Controlhaus

Looks good, nice refactor 😃

Optimized tsconfig even more to allow local linking also for matter-node.js package

I was able to implement all the exports used in DeviceNode.ts in my own implementation successfully. Pairing and device control tested ok with Alexa.

The code is still highly inter-dependent and monolithic looking, but easier to work with now.

Should be easier to work with.  Approving as all compiles, all tests run, and examples are functional.

Makes sense from my admittedly limited perspective.  Thanks for the added documentation, that's helpful.
Here's some quick thoughts.  Probably best to accept this PR and iterate from there...  But these could change the public API so figured you might want to consider now.  LMK if it'd be better to open as separate issue(s) and/or PRs.

Storage currently acts like a singleton but the pattern is different vs. crypto/time/net.  Makes sense if there will be different storage pools or methods for a single platform in the future...  But I wonder if "default" storage could be handled by a singleton so everything platform-specific is configured the same way.
Much of the remaining code in DeviceNode and ControllerNode doesn't seem to be Node specific.  Could this be moved into base classes in matter.js or a new package?
Or even better, maybe have Device/Controller classes that handle platform specific bits via composition?  So e.g. QrCodePresenter (eww, just an example) similar to StorageManager.  The benefit would be that if I create BridgeDevice extends Device it could be used in either Node.js or React Native.  BridgeDevice extends DeviceNode would tie the implementation to Node.
R.e. duplicative singleton registration in DeviceNode and ControllerNode, maybe create MatterNode.registerProviders() or even just have import '@project-chip/matter-node' do registration on module initialization?  Would reduce redundancy but mostly relevant in light of previous point as the library user would use to initialize their platform.
If I understand correctly, except for the hard-coded On/Off Light endpoint, the call to start(), and a few missing extension points, DeviceNode looks like a good base class for any device!  Is that generally where you're headed?

Hi @lauckhart,
thank you very much for your review and your thoughts. This PR mainly focus on code structuring, so it do not touch existing functionality.
But I want to take the chance to answer your questions and give a bit background on the "history and future" ;-)
to 1.) We started as singleton for storage but then this broke in testing because there we needed a "controller" and a "device" storage in parallel and they have overlapping data to store. We will see if the planned high level API changes that, but for now it was the better and more flexible way
to 2.) Yes I also thought about that but that would have required several code changes in the "Device/ControllerNode class" but I wanted to do least possible "other code changes" becasue the PR gets big enough - additionally the new High level API (review coming soon after this got merged) will change all that dramatically - then exactly this happens :-)
to 3-5.) The High level API will be your friend ;-) lets discuss when I posted my proposal :-)

👍 Thanks, makes complete sense.  I found your discussions r.e. high-level API in Discord after I posted.  I had missed the elegantly named #matterjs-nodejs-node-matter channel 😁

Lol. Might get renamed soon ;-)


Detailed Mode

Hello,
I think a nice feature of your plugin would be a Detailed mode that would send every elements of a Sampler:

Request Header+Body
Response Header+Body

There should be an option to send Successful or Erroneous requests/responses.
During scripting, one would enable ERROR + SUCCESS sending to be able to search using Elastic Search power
During Load Testing, one would only enable ERROR  sending to be able to investigate when errors occur.
What do you think ?
Regards

I think that's a very good idea and it's extremely easy to implement :) !
I will get to it ASAP!
Thanks again for the nice suggestion Philippe!

Added debug and info mode in new version
https://github.com/delirius325/jmeter-elasticsearch-backend-listener/releases/tag/2.2.3
Would you happen to have any suggestions to the way the functionnality works or should I push to jmeter-plugins and Maven as well?

Hello Anthony,
Great !
I 'll have a closer look in the upcoming days, but for now my suggestions:

I would use isSuccessful instead of isResponseCodeOK
Why did you remove sending assertion results ?
Document the new feature by upgrading screenshot at minimum :-) . Non documented feature does not exist :-)
For the plugin and to show how great it is, you should propose a template dashboard for detailed and non detailed mode.

Regards
Philippe M.
@philmdot

Hey Philippe,

After thinking about it, not really sure why I removed assertion results.. I'll just add it back
I'll make sure to use isSuccessful instead of isResponseCodeOK
When I mark the version as release I'll document the new feature :-)
Great idea, I'll make a template dashboard for both modes. I'll even make one for build comparing when using Jenkins for automated performance testing.

Cheers,
Anthony G.

Hello,
The assertion results is the main reason I use this plugin :)

Hi,
Yeah no worries though, the release where I removed the assertion results isn't on Maven or JmeterPlugins. It's only a pre-release on this repo.
It'll be added back into the next pre-release.
Thanks,

Hello @delirius325 ,
I see you released today a new version but don't see any related commit.
Is this regular ?
Thanks

Hi Philippe,
I can't believe I actually forgot to push my changes... Wow... First thing I do in the morning...
😂

Thanks for pushing.
Code looks good to me although I am not sure I'll have time to test it soon.
Regards

IMO, default ES_TEST_MODE should be info which is a good option for Load Testing.
Debug is too verbose and might lead "lazy" user to think JMeter+ES Plugin does not scale :-)
Thanks

Hi Philippe,
Thanks for the feed back! Changed ES_TEST_MODE to info. https://github.com/delirius325/jmeter-elasticsearch-backend-listener/commit/e1e6d7287577ac78cb1c8d75a2aaead454f974a3
Only thing that's missing as of right now is the documentation. I'll get started on that ASAP.

Released version 2.2.3 on GitHub. Will deploy to maven and create PR to jmeter-plugins ASAP.
https://github.com/delirius325/jmeter-elasticsearch-backend-listener/releases/tag/2.2.3

Thanks.
The documentation is in wiki ? thanks

@pmouawad Yes is it in the Wiki, still a work in progress though!


Title

Type of change

Bug fix
New feature
Improvement (improvement to code, performance, etc)
Test update
Documentation update

Description

Additional details

Related issues

Looks like this PR was pulled against the wrong branch.
This PR should be closed and re-pulled against the 1.3 branch (rather than master) for review.


Crash with large amount of data

I'm trying to transfer a relatively big amount of data (few MB) and it crashes quite often.
What happened is "buffer.byteLength" returns undefined as "buffer" happens to be actually a header ("CHUNKS:2:b" for instance). Then "totalByteSize" becomes NaN and " input.forEach(function(chunk)" throws exception (RangeErrror: Source too large ...or something...). Didn't dig into it more...

@nickich I'll have a look into it when I get a chance.  For the most part I'd probably encourage trying to use rtc-dcstream instead as that is being actively used in application development by the Coviu team (and I know they they are definitely sending some BIG files over the data channel).


Flesh out Cargo.toml more

These are all the fields recommended here: https://doc.rust-lang.org/cargo/reference/publishing.html#before-publishing-a-new-crate

The Cargo.toml docs say:

If no URL is specified in the manifest file, crates.io will automatically link your crate to the corresponding docs.rs page.

so I'll skip that field it and just let the default fill in itself.


Followme does not need local cloudformation script

I rebuild (which removes repos)
I try to configure (to clone repos) so i can use plugins because they're not loaded
I hit followme (but no plugins folder)
Madcore Core CLI - Deep Learning & Machine Intelligence Infrastructure Controller
v<IP_ADDRESS>+ge20c961 Licensed under MIT (c) 2015-2017 Madcore Ltd - https://madcore.ai

2017-03-29 09:46:51,575 [DEBUG] madcore: initialize_app
2017-03-29 09:46:51,579 [DEBUG] madcore: prepare_to_run_command Followme
2017-03-29 09:46:52,667 [INFO] madcore.cmds.followme: Core Followme: Your public IP detected as: <IP_ADDRESS>
2017-03-29 09:46:52,668 [INFO] madcore.cmds.followme: Updating 'MADCORE-FollowMe' Stack...
2017-03-29 09:46:52,669 [ERROR] madcore: [Errno 2] No such file or directory: u'/Users/polfilm/.madcore/cloudformation/sgfm.json'
2017-03-29 09:46:52,669 [DEBUG] madcore: Got an error: [Errno 2] No such file or directory: u'/Users/polfilm/.madcore/cloudformation/sgfm.json'

We know exactly the name of the stack and region in which it is, Do we need the CF file to be on the disk? Can you look up --use-previous-template
http://docs.aws.amazon.com/cli/latest/reference/cloudformation/update-stack.html

But we have there defined all the mapping like:
          { "IpProtocol" : "tcp", "FromPort" : "22", "ToPort" : "22", "CidrIp" :  { "Fn::Join" : [ "", [ { "Ref" : "FollowMeIpAddress"}, "/32"] ]}  },
          { "IpProtocol" : "tcp", "FromPort" : "443", "ToPort" : "443", "CidrIp" :  { "Fn::Join" : [ "", [ { "Ref" : "FollowMeIpAddress"}, "/32"] ]}  },
          { "IpProtocol" : "tcp", "FromPort" : "8080", "ToPort" : "8080", "CidrIp" :  { "Fn::Join" : [ "", [ { "Ref" : "FollowMeIpAddress"}, "/32"] ]}  },
          { "IpProtocol" : "tcp", "FromPort" : "80", "ToPort" : "80", "CidrIp" :  "<IP_ADDRESS>/0"},
          { "IpProtocol" : "tcp", "FromPort" : "0", "ToPort" : "65535", "CidrIp" :  "<IP_ADDRESS>/24"  }
        ]


Add net host information

While the suggestion is appreciated --net=host introduces possibilities for port conflicts with running services or other containers on a Docker host. It also doesn't account for any firewalling or other security that may need to be adjusted to protect or allow access a container. Running with Docker internal networking and specific mapped ports allows for known conflicts to be remapped as needed and allows the Docker internal processes to handle firewalling.


[Alert] Add a dense/small variant

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Based on the docs currently it's not possible to have a small/large Alert component and I didn't find a simple way to customize the appearance. A new preset or a "complete" customization example would be nice
Examples 🌈
No response
Motivation 🔦
No response

Hey @alippai, thanks for the suggestion! A couple of thoughts:

it's not possible to have a small/large Alert component

What would you expect from a small Alert? Less padding and smaller font size?

I didn't find a simple way to customize the appearance

Have you tried using either the sx prop or the theme (you could create your own Alert variant with this approach)?

Exactly the great questions here! I'm not a designer and this is what I'm looking for. A design system can have the answer, I'm not sure Google Material or Joy has any opinions on the question. I'm well aware of sx but I don't know what to change, the styling is complex (try changing just the padding or just the font size, you'll see)

It's interesting to think about it! Here's an example just eye-balling it. Let me know what you think about it.
None of the examples I have close by has a size prop to it (Mantine, Evergreen, and, Shoelace)... however, Joy will probably support it - given the global variant approach that we're betting on.

I'd be interested in an alert size that matches button size.  For example, here's an alert next to a button (both with default sizing)

The height mismatch is visually unpleasant, and becomes more so when the button has size="small":

I'm also very open to alternative/preferred approaches for inline status indicators (which, ideally, would include both icon and text).

This would be particularly useful when conditionally rendering an Alert on the same line as a button / checkbox / switch.
The larger size of the Alert causes the entire UI to jump up and down, and forces the use of a minHeight or a transparent Skeleton somewhere.
Playing with the Alert's sx={{height: ... }} to make it smaller generally breaks the look of the component.


Typo fix

Q
A

content fix?
yes

New article?
no

Related issues?
fixes

What's in this Pull Request?
Typo fix.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/web-parts/get-started/build-a-hello-world-web-part.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx Jems, all contributions are always welcome.


*: update all documentation and examples on use of pv-provisioner

[skip ci]
Updated the documentation wherever it is assumed that the operator will create a storage class by default.
The example backup spec example/example-etcd-cluster-with-backup.yaml has been changed to use the custom storage class example/example-storage-class-gce-pd.yaml.

lgtm


HNSW CPU Hierarchy

This PR adds an option to build the full HNSW hierarchy on the CPU when converting a CAGRA index to an hnswlib index. It also updates hnswlib to v0.7.0

Does this need to be added to the Conda recipes as well?

@jakirkham no actually we don't want to use the hnswlib conda package because their python distribution messes up with our python version requirements. I will be removing it from our conda dev environments as well.
We'll rely on CMake instead to get us hnswlib. There's no double compilation issue anyway since it's a header-only library.

Oh ok. Hadn't not looked into the nuances of this package or our usage of it
Indeed if we don't use the Python package, we should drop it
There doesn't appear to be a header-only C++ package currently. So just fetching it from source seems like the thing to do
Are the headers just used in the build to produce a library? Or are we shipping their headers too? If the latter, we may want to look at siloing it somehow to avoid clobbering or affecting other installs a user may have

@jakirkham we don't need to ship the headers, they are self-contained within the libcuvs shared object.

/merge


Fix potential access of null pointer (pp)

Fixes #1001. If |pp| is NULL, a segfault would occur on line 370.

+1

+1 will merge.

+1 from me; merged at a9da48157c90b4b6d19d912666ad2183b5a16a1f  thanks!


Fix the data backup/restore feature

Change the data backup & restore feature and implement a feature that allows the user to backup & restore their data in the form of CSV.

Is this being worked on?

Nope @logickoder, if you want you can start working on it.

Can I be assigned please

Yeah sure

Create your branch through master.

Should I entirely replace the backup/restore with CSV format or should it be an option for the user?

Entirely replace with backup/restore with CSV

Okay, makes sense


sudo for uniconr start/stop/restart

Closing. Already merged https://github.com/capistrano-plugins/capistrano-unicorn-nginx/pull/55


vSphere new implementation

Missing parts:

Collecting tags from vCenter (requires an additional library/REST calls). Will be done in a separate PR.
Collecting per-instance metrics (instance is basically an extra dimension for metrics. For example, vsphere.cpu.usage can be tagged with instance:%d where the integer is the CPU core identifier). For now we do not collect those as it is expensive but we should add the option to collect them. Will be in a separate PR.

Major things to look for:

The way filters are implemented
The way concurrency is implemented
Any opinion regarding the config or implementation designs.

Note: Because we moved the legacy vsphere.py file to vsphere_legacy.py and because the new implementation also has to be named vsphere.py, git tries to show a diff between the two implementation which is a bit confusing. You can look the latest version of the vsphere.py file just here
Note2: The integration is running against our lab in our dddev environment.
Note3: I haven't checked yet if metadata.csv is up to date but in theory it should.
Note4: Don't mind the number of lines, most of it is either big fixture files or just moving the legacy code somewhere else.

Question: Should ressource_filters expect metrics to be prefixed with vsphere or not?

Question2: Should we allow collection_type: both

Remaining work: add empty_default_hostname: true to the config by default. And add the hostname field to all metrics/service_check/event where needed


Error on pull and push records to splunk

Suppose I have 300 million events in my local Splunk instance. I write a chunked streaming search command read records from Splunk and write to Splunk then. Then I got a error:
ChunkedExternProcessor - Failure writing result chunk, buffer full. External process possibly failed to read its stdin.
Seems it's a very common issue when handling big scale data. Maybe it's a bug?
Another issue is when I write a big data to Splunk, I can also get this exception sometime:
File=search_command.py, Line=1016, IOError at "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/internals.py", line 820 : [Errno 32] Broken pipe
Traceback:
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/search_command.py", line 801, in _process_protocol_v2
    self._execute(ifile, None)
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/streaming_command.py", line 54, in _execute
    SearchCommand._execute(self, ifile, self.stream)
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/search_command.py", line 869, in _execute
    self._record_writer.write_records(process(self._records(ifile)))
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/internals.py", line 528, in write_records
    write_record(record)
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/internals.py", line 651, in _write_record
    self.flush(partial=True)
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/internals.py", line 774, in flush
    self._write_chunk(metadata, self._buffer.getvalue())
  File "/Applications/Splunk/etc/apps/searchcommands_app/bin/packages/splunklib/searchcommands/internals.py", line 820, in _write_chunk
    write(body)

I think maybe you guys want to know this.

Running into this problem as well. Is there any known workaround (like maybe not using chunked)?

Hey devs, any updates on this one? Just making sure I should use this wrapper at all (issue dating Aug 2016 and still not fixed).

This issue is discussed in Splunk Answers ticket 686940 (https://answers.splunk.com/answers/686940) and I have offered a proposed SDK-side workaround (that is actually also an SDK efficiency improvement) in GitHub here:
https://github.com/TiVo/splunk-sdk-python/commit/5188f7d709cadd80e786692b371a64c4ae0991d2


Raspberry

Hi, it can be connected to raspberry pi 3 ?

Hello, I am sorry that this's for Arduino.
If you want to feature your Pi with CAN Bus, you can try Serial CAN Bus module.
Thanks.


Fix enableFeaturePreview failure on template project execution

Fix https://github.com/gradle/gradle/issues/10454
Previously, lower Gradle version might not be able to execute the template project
generated because of it doesn't have some features. This commit fixes it by catching
the exception.

Thanks @wolfs !


Support for firefox

This PR tries to add basic support for firefox.
To launch taiko with firefox:

Download and install firefox nightly
Set TAIKO_BROWSER_PATH to firefox nightly's executable.

Example: TAIKO_BROWSER_PATH="/Applications/Firefoxnightly.app/Contents/MacOS/firefox" taiko
Known Issues for now:

Highlighting element on action is set to false as overlay domain is not supported
openTab/closeTab does not work as expected since New is not available in firefox yet
Autofocus on first input field does not happen
file:// protocol does not emit events
waitFoNavigation does not wait for network calls and frame loads

Can you also update the documentation and FAQ? For example you can change
It is a node library with a clear and concise API to automate the chrome browser

to
It is a node library with a clear and concise API to automate the chromium and firefox browsers

Also add a section with the title Firefox support to list what does not work and what is different.


[Threading][Win32] Fix a warning.

The RTL_SRWLOCK_INIT declaration got reformatted and doesn't exactly match what's in windows.h, so we're getting warnings during builds. Fix to match what it says in the system header.
rdar://105400572

@swift-ci Please smoke test

@swift-ci please smoke test macOS platform


Is this in active development?

It's just intended to be an example, so while I wouldn't say it's actively developed, we want to make sure it stays useful. Is there something specific you had in mind?


Bad Opacity rendering with Shapes or Border

Describe the bug
When you apply opacity on a Shape or Border, the effect of opacity is applied separately on the stroke and on the background which doesn't correspond to the desired rendering, not the WPF behavior.
To Reproduce
Compare the rendering of the XAML code below between Avalonia and WPF (or any other design tools).

Expected behavior
The opacity must be applied on the final rendering of the Shape/Border, and not on the stroke/background separately as you can see in the screenshot below (WPF on the left, Avalonia on the right). If the stroke is Opaque, we must not see the background below it by transparency when Opacity is not 1.0.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Version 11 preview 6

Additional context
Also would be awesome if before final 11 release you can add some kind of BackgroundSizing feature on bordered elements and shapes to choose if we want to draw the border above the background, outside the background, or center as it is currently the case (but which is the less useful usage).

Duplicate of https://github.com/AvaloniaUI/Avalonia/issues/9227
And alternatively of this one https://github.com/AvaloniaUI/Avalonia/issues/9946

Have you tried adding .With(new SkiaOptions { UseOpacitySaveLayer = true }); to your AppBuilder?

@Gillibald Thank you so much for your reply. I added the options and it solved the issue.
But as I understand by reading the related post, that is not enabled by default for performance reason?
However I am not sure if enabling the option still cause the unnecessary textures creation, or if you already have added conditional code to reduce the overhead cost? In fact I don't know if it is all Black or all White...
Also, if I keep the option disabled, is there any trick such as an attached property or something to choose to use SaveLayer only on a specific container? Or do I have to reimplement rendering of Shapes and border? (but I may still have issue if one container have multiple childs and opacity).
Thanks!

UseOpacitySaveLayer was introduced as an immediate solution that will be replaced by some smart layering algorithm that automatically picks the best performing code path. We can't enable it per visual so you either have to enable it globally or not. The performance impact is theoretical. We did not do any benchmarks. So as long as you don't see skyrocketing memory usage it should be safe to have it enabled.


Move over to OrderedDict

In order to preserve the order of key's and values between pre-processing, post-processing, and saving, we should use OrderedDict to store input files.

We should also move from https://github.com/JuliaCollections/DataStructures.jl to https://github.com/JuliaCollections/OrderedCollections.jl

For TOML files, you can create a new Parser, swapping root from TOMLDict to an OrderedDict

YAML directly allows for dicttype=OrderedDict

JSON also has dicttype=OrderedDict


Deprecation warnings

Describe the bug
when vcpkg is used as toolchain for cmake. it generates a lot of deprecation warning:
CMake Warning (dev) at scripts/cmake/vcpkg_find_acquire_program.cmake:70 (cmake_parse_arguments):
  An empty string was given as the value after the INTERPRETER keyword.
  Policy CMP0174 is not set, so cmake_parse_arguments() will unset the
  arg_INTERPRETER variable rather than setting it to an empty string.
Call Stack (most recent call first):
  scripts/cmake/vcpkg_find_acquire_program.cmake:225 (z_vcpkg_find_acquire_program_find_internal)
  C:/Users/ASUS/AppData/Local/vcpkg/registries/git-trees/fa93b14c1581057e1d14650f5f15c9b0757dd548/portfile.cmake:114 (vcpkg_find_acquire_program)
  scripts/ports.cmake:192 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.

this warning repeats many times for different files or lines of code including vcpkg.cmake:40, vcpkg.cmake:878, vcpkg_find_acquire_program.cmake:70, vcpkg_find_acquire_program.cmake:30
Environment

OS: Windows
Compiler: MSVC
Installed packages using manifest mode: glad, glfw3

To Reproduce
just create a project with vcpkg in manifest mode and add glfw3 and glad port and try to compile the program then you get those warnings
Expected behavior
No deprecation warning
Additional context
I know i can suppress it with -Wno-dev but i guess this would suppress also my own cmake warnings.

The policy CMP0174 was added in CMake 3.31.
Vcpkg internally uses CMake 3.30 so you shouldn't be seeing this warning:
https://github.com/microsoft/vcpkg/blob/41626fd77bf42f29e8f7e43dc1f2f05780588cde/scripts/vcpkgTools.xml#L17-L23
Do you use VCPKG_FORCE_SYSTEM_BINARIES environment variable?

Vcpkg internally uses CMake 3.30 so you shouldn't be seeing this warning

When CMake 3.31 is already in the PATH and CMake 3.30 is not unpacked yet, vcpkg will use it happily.
That's how I analyzed https://github.com/microsoft/vcpkg/issues/41719.
And yes, the fix is easy. But 3.31 isn't released yet, and that new behavior has a regression. So there is no point in opening a PR.

When CMake 3.31 is already in the PATH and CMake 3.30 is not unpacked yet, vcpkg will use it happily.

Wow. I guess I never experienced this because I always use --x-abi-tools-use-exact-versions.


Add receiver device controllers

Checks

[ ] Debugging completed
[ ] Please debug me

Related Issue
This PR closes #145.

Codecov Report
Base: 81.70% // Head: 78.02% // Decreases project coverage by -3.68% :warning:

Coverage data is based on head (19832aa) compared to base (669fefb).
Patch coverage: 60.46% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
- Coverage   81.70%   78.02%   -3.69%
==========================================
  Files          47       52       +5
  Lines        2433     2753     +320
==========================================
+ Hits         1988     2148     +160
- Misses        445      605     +160

Impacted Files
Coverage Δ

necst/ctrl/antenna/motor.py
26.78% <18.18%> (-3.65%)
:arrow_down:

necst/rx/spectrometer.py
27.35% <26.19%> (+1.74%)
:arrow_up:

necst/ctrl/antenna/encoder.py
41.37% <33.33%> (+1.37%)
:arrow_up:

necst/rx/sis_bias.py
39.39% <39.39%> (ø)

necst/rx/signal_generator.py
42.42% <42.42%> (ø)

necst/ctrl/calibrator/chopper.py
37.93% <42.85%> (ø)

necst/rx/attenuator.py
43.75% <43.75%> (ø)

necst/core/clock.py
44.00% <44.00%> (ø)

necst/rx/thermometer.py
50.00% <50.00%> (ø)

necst/ctrl/antenna/pid_controller.py
89.07% <66.66%> (-1.53%)
:arrow_down:

... and 25 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[FEATURE] Enable mypy check in pre-commit-hook

Since we have type hints, I would also suggest to enable mypy check in pre-commit-hook.
Setup example, I put mypy setting in pyproject.toml: https://github.com/mmwave-capture-std/mmwave-capture-std/blob/8f3332814aba442003e0a3e103629d2331c30f7d/.pre-commit-config.yaml#L15

404a80d


Tested user features.

Description
Tested user features
Fixes #141
Status
READY
Type of change

Bug Fix.

How Has This Been Tested?
-The tests passed.
Checklist:

[ ] I have made corresponding changes to the documentation (if applicable)
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Aguardando revisão. @lucaaas @luizguilherme5

Aguardando nova revisão. @lucaaas


Exception thrown in copy constructor when copying empty optional

https://github.com/martinmoene/optional-bare/blob/0bb1d183bcee1e854c4ea196b533252c51f98b81/include/nonstd/optional.hpp#L179
May be I don't understand something, but how this line is supposed to work if method value() always checks assert(has_value())?
As i understand it we will always get exception when trying to copy empty optional.

Thanks @Andoku , for spotting & reporting this.
Added the missing test case and corrected the behaviour.

@martinmoene I think assignment operator has the same problem

Ah thanks,will look at that too.


fix(DATAHUB-305): ensure Dropdown opacity classes don't collide

This PR fixes the bug that currently when logged in, we cannot access the user nav because the classes opacity-0 and opacity-100 are simultaneously present and opacity-0 takes precedence which results in the dropdown being invisible.
In this PR it is ensured that the classes cannot exist at the same time.

Great Catch! Thx for fixing! 🙏
Consider this as approved! (I'm on mobile right now)

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


Change to install Roboto fonts by npm

Remove the Roboto fonts into source code or project. These are install by npm.

If I do a fresh install of MagicMirror (and npm install), the Roboto fonts will not load because the filenames in the roboto.css file does not match the actual font file names.
Example from roboto.css :
url("node_modules/roboto-fontface/fonts/roboto-condensed/RobotoCondensed-Light.ttf") format("truetype");
Actual filename is :
/node_modules/roboto-fontface/fonts/roboto-condensed/Roboto-Condensed-Light.ttf
I just wanted to verify that others can reproduce this issue as well before I proposed a fix for this...
@roramirez Can you confirm that your fonts do not get loaded as well?

 My mistake :(
There a way to make a test for the css?

@roramirez I've made an attempt at an e2e test case and added it to the PR #986. I think I inadvertently also found the reason why the vendor_spec.js test case isn't working at this moment (hit and miss) and I will try to test my theory; if it works I will submit a PR for that as well...

I stay expectant :O,
Do you mean this?
https://github.com/MichMich/MagicMirror/pull/1000/files#diff-a7d779483335cc417b43ff18051866adR45


Custom Fields in Jobs to be done format

Description of the change:
Reason for the change:
Link to original source:

Closes #
@ksigler7

Hello @Whatthefoxsays,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Evaluate opportunities to ease the experience of new contributors

Is your feature request related to a problem? Please describe.
We hear from pvlib users that unfamiliarity with git/github is sufficient to prevent them from making the jump from user to contributor.  Ideally, sufficient documentation would exist to prevent this kind of procedural/tooling barrier from derailing contributions.
Our Contributing page our primary documentation to help people get started with contributing to pvlib.  More generally, many resources for learning git/github and open-source contributing are available.  Perhaps these guides are insufficient somehow, or perhaps people are not aware of them.
Describe the solution you'd like
Evaluate where and how we can improve our documentation to make first contributions easier.  Feedback from people who recently began contributing to open-source projects would be very valuable on this point.
Describe alternatives you've considered
Some community members have suggested recording a video of walking through the process of making a first PR.  I see value here, but it may be largely redundant with existing guides elsewhere on the internet, and also risks falling out of date if somehow the contribution process changes (an example: when we switched from master to main).

Instead of creating a new video, we could add link to existing youtube resources for how to create a PR for beginners? Additionally, for absolute beginners who have only used Github for browsing, resources for how  to download git for desktop and use it would be helpful as well.

Hey team,
I am planning on posting this unofficial blog post once the PR for the transformer model gets merged.  It outlines my personal workflow for stuff like this, and the workflow might be different than how other people use python since I am normally working in other programming languages, but my hope is that the more resources there are, the easier it becomes.
Open to any suggestions for things that are missing.  I already know I should add:

a link to the contributing page
download of the flake8 extension for vscode

https://kurt-rhee.github.io/2024/05/11/how-to-contribute-to-pvlib

nice job with that post @kurt-rhee , I think the screenshots in particular are very helpful. I am a new contributor and at first (slowly getting used to it now) I found aspects of the git interface hard to navigate since there are a lot of symbols and terms to get to grips with. I think the definitions are super useful too --- any user can find them elsewhere from external pages but it's much easier and more convenient just to have them all in one place so that's good too

@kurt-rhee as a fellow VSCoder, may I know what is the benefit in using the integrated Git? Does it change the opened files or so when you change branches? I learnt how to use GitHub desktop so I never bothered in switching to it.
Btw, that post is amazing. I may try to expand on other topics in the future: rST, sphinx-gallery or CI/CD so in the future we can gather all those posts in a pvlib wiki page.

@echedey-ls there is no benefit other than the fact that you don't have to leave the IDE to use it.  I've used github desktop and command line in the past and I don't see any downsides to any of them except for maybe the steep learning curve of the command line for new users.

Great. Thank you very much @kurt-rhee !!

I think a definition of the good first issue tag and a link to such issues added to the contributing page to encourage new users to contribute. To be honest, I did not really know what this tag meant until I saw #2089 from @echedey-ls and the discussion with @AdamRJensen that concluded the issue could be left for newcomers. This would help people who want to contribute but don't have any ideas for PRs or aren't familiar with browsing through existing issues. Bear in mind if they are completely new to github, they are probably still working out what an issue and a PR actually is, so explaining this and directing them over to the good first issue tag might help.

After a brief conversation with @RDaxini and lurking his LinkedIn profile, I found that not everyone that wants to contribute here has ever used GitHub (or equivalent). I suggest that an starting point for the page should be an introduction to what it is and why it is used. And for unfamiliar users, recommend doing the GitHub hello world. My experience with that guide is that it is very useful for new people, as once I was, and other fellow students I have worked with.

Sorry @RDaxini, I updated the date of the post and that updates the post link.  I have updated my comment above.


Bump MUSL library

This PR contains the changes needed to update/compile and test using the MUSL library 1.1.21.
  Bump MUSL library to 1.1.21
  Fan out `tsearch_avl.c` from libc/CMakeLists.txt
  Ignore compiler warning on MUSL memmove.c
  Add new include directory needed by MUSL 1.1.21
  Fix enclave/core/__secs_to_tm.c compilation
  Change attribute from weak to __weak__ in defs.h
  Add more files to oelibc
  Mark strptime functional test as unsupported

From:
https://github.com/Microsoft/openenclave/pull/1492#pullrequestreview-207610457
3rdparty/musl/musl/configure:34

@mikbras, @anakrish MUSL added a new configuration modifier:

--enable-visibility     use global visibility options to optimize PIC [no]

Is there any additional change to the way we want to use PIC in building the enclave binary? I assume not.

@CodeMonkeyLeet :
I was unsure how to solve the circular dependency here, so I took a stab at it:
https://github.com/Microsoft/openenclave/pull/1498/commits/7bb4c857c979bc991cfa7b66ac60e782f0803da3
Please let me know how you want me to change it.

bors try

bors r+

bors r-

bors try

bors try

There are a bunch of failures with the Windows stages giving:
Failed to extract linuxDebug18.04.tar.gz

This is related to the CI Windows Elf build stage when it tries to pull a Linux build archive from a Linux node (using stash / unstash Jenkins pipeline primitives). Digging for more info about this, I found this in Jenkins:
java.io.EOFException: Unexpected end of ZLIB input stream
	at com.jcraft.jzlib.InflaterInputStream.fill(InflaterInputStream.java:186)
...
...
...
	at hudson.FilePath.readFromTar(FilePath.java:2623)
Also:   hudson.remoting.Channel$CallSiteStackTrace: Remote call to JNLP4-connect connection from IP_ADDRESS/IP_ADDRESS:49687
		at hudson.remoting.Channel.attachCallSiteStackTrace(Channel.java:1741)
...
...
...
		at java.lang.Thread.run(Thread.java:748)
Caused: java.io.IOException: Failed to extract linuxDebug18.04.tar.gz
	at hudson.FilePath.readFromTar(FilePath.java:2633)
...
...
	at java.lang.Thread.run(Unknown Source)
Finished: FAILURE

However, this is not consistent and it seems to be Jenkins-related. I'll keep an eye on this error, and investigate.
Failures are not related to this PR.

@ionutbalutoiu I saw that error as well yesterday! It was in another PR, and I simply re-ran a bors command and the error didn't repro. I wonder what's causing this

@johnkord @CodeMonkeyLeet Is there anything holding this PR to be merged ?

bors r+

@ionutbalutoiu I thought @johnkord had successfully merged it already based on the last comment, but apparently that's not the case … just ran the r+ and we'll see if it hits the Jenkins issue.

Part of the issue: https://github.com/Microsoft/openenclave/issues/1456


Did a bit more cleanup of the hash index

This failed because diesel does not work on nightly right now.
We should either do as they do and allow failures on nightly or alternatively we should just rerun whenever they fix it.


pickle for bbs

I want to convert bbs and topo to pickle format, but it seems only topo can be converted. I encountered the following error:
.conda/envs/pormake/lib/python3.9/site-packages/pormake/database.py", line 30, in init
if not self.bb_dir.exists():
AttributeError: 'str' object has no attribute 'exists'

I have fixed all bugs you mentioned (both in GitHub and PyPI).
Thank you!


[3223] Inconsistencies in IDictionary, ICollection, IList, etc

Fixes #3223 .
Changes proposed in this pull request:
Added missing and removed extra:

System.Collections.ICollection: Missing SyncRoot and IsSynchronized
System.Collections.IDictionary: Mismatching inheritances/interface references, extra 'Remove()method andContainsKey()instead ofContains()` implementation
System.Collections.IDictionaryEnumerator: Completely missing
System.Collections.DictionaryEntry: Completely missing
System.Collections.IList: Missing IsFixedSize

Propagated the interface contract to the classes referring to the interfaces and implemented like they are in .NET whenever possible (parallel code serialized).
Added also units tests.

Branch and features rebased and pushed to dev (thus git does not detect the changes were been merged).

Thank You - we encountered these issues recently.


TODO: investigate base distribution for the word "stupid" and see if we can collect a dataset to stop the false-positives for things like "Sorry I was stupid".

We can help by providing feedback (0 or 1) with communityId:"StackOverflow" on live comments, however since we are "only" a Community team we do not have access directly to SE database as NickLarsen.

Thanks for all the feedback!
We've just released TOXICITY@5 and also SEVERE_TOXICITY@1, these behave a good deal better on examples like this (although of course still a more work to do!). Would love to know if they work any better for you?


Fix for CakePHP 3.5.x

I've popped in a change for the latest version of CakePHP - 3.5.x. to remove the strict warnings.

Your stickler.yml configuration file could not be parsed. We got the following error:
while parsing a block mapping
  in "<string>", line 3, column 5:
        standard: CakePHP
        ^
expected <block end>, but found '<scalar>'
  in "<string>", line 4, column 29:
        extensions: '.php,.ctp' branches:
                                ^


DataStore never starts sync if the App started when device was offline even when invoking DataStore.start

Describe the bug
Original issue: https://github.com/aws-amplify/amplify-flutter/issues/1440

Condition

Turn off network
Run App with network turned off
Save 3 models say Todo1, Todo2 and Todo3

Observation

DataStore saved all 3 models into local DB
DataStore pushed all 3 models into mutationOutBox
There was no sign that sync engine was invoked to process mutationOutBox events
Turn on network
Invoking Amplify.DataStore.start , but below logic actually prevented restarting of the sync engine
https://github.com/aws-amplify/amplify-ios/blob/e66dfbc6265613d98000e0f2eb88fd4df380eb10/AmplifyPlugins/DataStore/AWSDataStoreCategoryPlugin/AWSDataStorePlugin.swift#L141-L144

Steps To Reproduce
Steps to reproduce the behavior:
See above description

Expected behavior
DataStore should start sync in this scenario.
Amplify Framework Version
1.22.0
Amplify Categories
API, DataStore
Dependency manager
Cocoapods
Swift version
5
CLI version
The latest
Xcode version
13.2.1
Relevant log output
No response
Is this a regression?
No
Regression additional context
No response
Device
iPhone 13 simulators
iOS Version
15.2
Specific to simulators
No response
Additional context
No response

Hi @HuiSF , thanks for the details, i'll give it a try to reproduce the unexpected behavior with Amplify.DataStore.start and provide an update when i can here

@royjit was able to reproduce this on a real device, the issue is the URLError case that is received by DataStore is not handled by the retry handler. When the retry advice is false, the attempt is treated as a non-retryable error and deleted from the outgoing mutation queue

change has been merged and released


Request to add download button

Hii developers if this app ..pls can u guys add a section for watching chinese dramas also? I think it would be great if u would introduce an extension called kisskh cos it have all dramas ..and can u guys please make the videos available there to download in the app itself too instead of in internal memory of phone please? It's a request..Hoping to hear from u guys

Hi, Chinese drama is already there, I will add a search page for drama so you can search it. Downloading or offline watching is not possible currently.

Hii can u please tell when will u add download button ?
On Mon, 4 Mar, 2024, 1:50 am Sanjeev Mishra, @.***>
wrote:

Hi, Chinese drama is already there, I will add a search page for drama so
you can search it. Downloading or offline watching is not possible
currently.
—
Reply to this email directly, view it on GitHub
https://github.com/PyPiSan/KinAni/issues/19#issuecomment-1975288476, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BGBA4AJLYI3Y3KC7YVGJEDLYWOAYRAVCNFSM6AAAAABEEAHAV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZVGI4DQNBXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


ISPN-7129 Duplicate cache names shouldn't be allowed

https://issues.jboss.org/browse/ISPN-7129
Removed the ParserOverrideTest after discussion with @galderz, since we have templates now.

@jmarkos @galderz I think as long as the functionality of extending the default cache still works, ParserOverrideTest can stay. Not sure how the template overrides are tested anyway...
I'd also like to see a test that actually has a duplicate cache name and throws the exception.

@danberindei the test cannot stay if my changes are to be accepted, because it fails with them (as it has multiple caches with the same name). So if we want to continue allowing overriding a cache configuration with another cache with the same name (which I think is more often a mistake on the users part than an intention), this PR and the jira can be closed.

@jmarkos Sorry, I didn't mean it should stay exactly the same, although TBH I was hoping some of it was dedicated to testing how the inheritance of configuration from the default cache to the named caches used to work. I see that's not the case, and in fact it doesn't work any more :)
I think there would still be some value in updating this test to check extending a template works and that extending another cache doesn't, unless there's another test for that already. WDYT?

Named caches still inherit the configuration of the default cache, with this configuration
    <cache-container name="c1" default-cache="mydefault">
        <local-cache name="mydefault">
            <eviction max-entries="100" />
        </local-cache>
        <local-cache name="inherit"/>
    </cache-container>

the 'inherit' cache will also have max-entries=100.
This
        <local-cache name="mydefault">
            <eviction max-entries="100" />
        </local-cache>
        <local-cache name="inherit" configuration="mydefault"/>

throws "Exception in thread "main" org.infinispan.commons.CacheConfigurationException: ISPN000375: Cannot use configuration 'mydefault' as a template" but I haven't found a test for it.
The only test for the templates I found (in core) is this one https://github.com/infinispan/infinispan/blob/master/core/src/test/java/org/infinispan/configuration/parsing/UnifiedXmlFileParsingTest.java.
I can add a test that verifies you cannot define duplicate cache names, would that be enough?

So what's the current/desired status? I think that cache xyz should not inherit anything from the default cache, as this is rather cryptic; if user wants default cache and custom cache inheriting similar configuration, he should use
<cache-container name="c1" default-cache="mydefault">
    <local-cache-template name="common"><!-- whatever --></local-cache-template>
    <local-cache name="mydefault" configuration="common"/>
    <local-cache name="inherit" configuration="common"> <!-- add something --></local-cache>
</cache-container>

I think inheriting from the default cache was a concept that was useful before templates, now it seems like a relict.

I think default cache inheritance should go.
@jmarkos, since there might be different opinions, it might be good if a dev thread is started to finalise the decision?

Rebased and added a test (to XmlFileParsingTest, not sure if that's the right place).

Needs rebasing. Also, this PR depends on #4744.

Still needs rebasing. Closing for now. @jmarkos please reopen when you've got around to sorting this out. Thanks for the contribution!

I was waiting for https://github.com/infinispan/infinispan/pull/4744 to get merged, since it looked like there could be conflicts. I'll look at this asap.


Update PTO entries for January 2024 to reflect in leave tables

Description
Currently, PTO's from January 2024 are not reflected in the leave management table.
What is the expected behaviour?
All leaves from 1st January 2024 should be reflected in https://app.miru.so/leave-management
What is the current behaviour?
leaves from February 2024 are shown in leave management

Completed https://www.notion.so/saeloun/6afc9500b2cc42af86be55f37894fb09?v=b4528f26f4424af7beaf036262796cb3&p=1d0d4b33c3f7402085bfd3f8cba8ecb0&pm=s


Github Release of 2.2

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
Issue to track the Github release of KIC 2.2
Proposed Solution
No response
Additional information
No response
Acceptance Criteria

[ ] https://github.com/Kong/kubernetes-ingress-controller/blob/main/RELEASE.md

waiting to give #2156 a chance to get into 2.2, but it's not a technical blocker so if we can't move it along in a couple days I'll release around it.


The sus macro is sus

The @sus macro says that GlowupVibes is sussing. However, the REPL is the one sussing...
julia> @sus "The red one is sus"
┌ Warning: The red one is sus
└ @ Main ~/.julia/packages/GlowupVibes/Rkk2K/src/GlowupVibes.jl:25

julia> @warn "The red one is sus"
┌ Warning: The red one is sus
└ @ Main REPL[9]:1

This is as far as I've gotten so far.
using Base.CoreLogging: logmsg_code, LogLevel, _min_enabled_level, current_logger_for_env, shouldlog, handle_message, @_sourceinfo, logging_error
using Base.CoreLogging: Debug, Info, Warn, Error

macro sus(exs...)
     logmsg_code((@_sourceinfo)..., :Warn, exs...)
end

julia> using GlowupVibes

julia> x = 5
5

julia> @sus "Hello world" x
┌ Warning: Hello world
│   x = 5
└ @ Main REPL[7]:1

My earlier attempt was to build a macrocall expression but this has problems displaying x as above.
macro sus(exs...)
    Expr(:macrocall, Symbol("@warn"), __source__, exs...)
end

Can you elaborate some more on what is sus? Is it that it doesn't quote the correct line of source code in the log message?

Facts. The REPL[8] is sus not GlowupVibes.jl:25.


Changed the uncommit_transaction and unvoid_transaction methods to pass empty hash for model

When calling the uncommit_transaction and unvoid_transaction methods the following exception was being raised. This was due to not passing a model to the post method. These transactions do not require a model so I modified it to pass an empty hash.
Failure/Error: request.url("#{encode_path(path)}?#{URI.encode_www_form(options)}")
     NoMethodError:
       undefined method `map' for "22.7.0":String
       Did you mean?  tap
     # /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/uri/common.rb:419:in `encode_www_form'
     # ./lib/avatax/request.rb:33:in `block in request'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/connection.rb:513:in `block in run_request'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/connection.rb:530:in `block in build_request'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/request.rb:56:in `block in create'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/request.rb:55:in `tap'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/request.rb:55:in `create'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/connection.rb:526:in `build_request'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/connection.rb:508:in `run_request'
     # /Library/Ruby/Gems/2.6.0/gems/faraday-1.10.0/lib/faraday/connection.rb:281:in `post'
     # ./lib/avatax/request.rb:26:in `request'
     # ./lib/avatax/request.rb:14:in `post'
     # ./lib/avatax/client/transactions.rb:731:in `uncommit_transaction'
     # ./spec/avatax/client/transactions_spec.rb:42:in `block (3 levels) in <top (required)>'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:254:in `instance_exec'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:254:in `block in run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:496:in `block in with_around_and_singleton_context_hooks'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:453:in `block in with_around_example_hooks'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/hooks.rb:464:in `block in run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/hooks.rb:602:in `run_around_example_hooks_for'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/hooks.rb:464:in `run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:453:in `with_around_example_hooks'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:496:in `with_around_and_singleton_context_hooks'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example.rb:251:in `run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:627:in `block in run_examples'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:623:in `map'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:623:in `run_examples'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:589:in `run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:590:in `block in run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:590:in `map'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/example_group.rb:590:in `run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:113:in `block (3 levels) in run_specs'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:113:in `map'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:113:in `block (2 levels) in run_specs'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/configuration.rb:1835:in `with_suite_hooks'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:112:in `block in run_specs'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/reporter.rb:77:in `report'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:111:in `run_specs'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:87:in `run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:71:in `run'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/lib/rspec/core/runner.rb:45:in `invoke'
     # /Library/Ruby/Gems/2.6.0/gems/rspec-core-3.5.4/exe/rspec:4:in `<top (required)>'
     # /usr/local/bin/rspec:23:in `load'
     # /usr/local/bin/rspec:23:in `<main>'

Any change we can get this fix moving?

Yes I will take a look at this for the next release.

@burkematthew @trevorj06  Fixed in Ruby SDK version 23.3.0. Let me know if there are any problems