Can ApplicationSets and App of Apps be used together in a monorepo?

I am trying to deploy a very big service consisting of different components with ArgoCD.
I'm using a single repo to define all the components and using ApplicationSets to divide config based on environments, so each environment gets one ApplicationSet (dev, staging, prod..) each. For each environment, the targetRevision will be different.
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
  name: dev
spec:
  generators:
  - git:
      repoURL:<repoURL>
      revision: <dev-commit>
      files:
      - path: deployments/dev/**/config.yaml
template:
...
    spec:
...
      source:
        repoURL: <repoURL>
        targetRevision: <dev-commit>
        path: **apps**
...

Here, my source path apps has a Helm Chart format with multiple App definitions inside, to signify each different component of the over-all service which makes the structure as an ApplicationSet deploying a single app per cluster. And that app brings up several child apps on the target cluster.
However, the problem is that all these App definitions need repoURL and targetRevision to be valid.
Is there a way I can send the targetRevision from the ApplicationSet to these child apps?
Otherwise, even if ApplicationSets are defining a new config for each environment based on the targetRevision mentioned in the AppSet yaml, the several Apps inside apps folder will have a static spec for targetRevision which is shared across environments, which is not desirable.
I want to be able to change only 1 value(targetRevision in this case) in ApplicationSet YAMLs and have all the downstream apps use this value as well.

@jgwest Do you have any comments on the above question?

Hello @AnjaliSrivastava29
We are also facing the same issue as we are trying to use ApplicationSet deploying a first Application which will then deploy others Applications.
Based on the following schema:
ApplicationSets  deploys
      -> Combined-App  which creates
                 -> App1
                 -> App2

This architecture supposes to control the Sync wave between waves. Deploying applications in a specific order.

I was able to use Helm plugin in the Appset and Application YAMLs to deliver the values to the most bottom level.

I was able to achieve this by doing the following.
ApplicationSet:
apiVersion: argoproj.io/v1alpha1
kind: ApplicationSet
metadata:
  name: dev-appset
  namespace: argocd
spec:
  generators:
  - git:
      repoURL: https://<repo where values.yaml file exist>
      revision: main
      files:
      - path: "apps/**/*.yaml"
  template:
    metadata:
      name: '{{name}}'
    spec:
      project: '{{project}}'
      source:
        chart: xyz
        repoURL: https://<packaged helm chart url>
        targetRevision: 1.0
        helm:
          values: |-
            {{values}}
      destination:
        name: cluster-1
        namespace: '{{namespace}}'

      syncPolicy:
        automated:
          prune: true
        syncOptions:
          - CreateNamespace=true

And this is how my different yaml files with values looked like. I discarded the I discarded my parent App of Apps manifest and also the child application manifests too.
Unlike App of Apps which shows the parent app with its child apps in the UI, the ApplicationSet does not show in the UI, only the child apps show.
name: app1
project: testing
namespace: app1
values: |
    fullnameOverride: app1
    image:
        repository: <docker image repo url>
    namespace: app1
    containerPort: 7080
    autoscaling:
        enabled: true
    ingress:
        host: app1.test.com

@faraktingi
Why do you need an ApplicationSet in your schema? Why you can't use just AppOfApps approach?

Hello @AnjaliSrivastava29 We are also facing the same issue as we are trying to use ApplicationSet deploying a first Application which will then deploy others Applications. Based on the following schema:
ApplicationSets  deploys
      -> Combined-App  which creates
                 -> App1
                 -> App2

This architecture supposes to control the Sync wave between waves. Deploying applications in a specific order.


A Comprehensive Upgrade

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Improved User Interface:
Visual Appeal: Enhance the overall visual appeal of the website with a modern and user-friendly design.
Clear Navigation: Ensure a clear and intuitive navigation structure for easy browsing.
Responsive Design: Optimize the website for different screen sizes to provide a seamless experience across devices.
2. Enhanced Search Functionality:
Advanced Search Options: Allow users to search by author, title, genre, publisher, or ISBN.
Autocomplete Suggestions: Provide suggestions as users type to improve search efficiency.
Filtering Options: Implement filters for various book attributes (e.g., price, publication year, language).
3. Personalized Recommendations:
Recommendation Engine: Utilize algorithms to recommend books based on the user's purchase history, browsing behavior, and preferences.
Personalized Categories: Create personalized book categories or collections based on user interests.
4. Social Media Integration:
Sharing and Reviews: Allow users to share books on social media and write reviews.
Social Login: Enable login options through popular social media platforms for convenience.
5. Wishlists and Gift Registries:
Wishlist Feature: Allow users to create wishlists and share them with others.
Gift Registry: Enable users to create gift registries for special occasions.
6. Mobile Optimization:
Mobile-Friendly Design: Ensure the website is fully optimized for mobile devices.
Mobile App: Consider developing a dedicated mobile app for a more seamless experience.
7. Payment and Shipping Options:
Secure Payment Gateways: Integrate with reliable payment gateways to ensure secure transactions.
Multiple Payment Methods: Offer various payment options (e.g., credit/debit cards, net banking, UPI).
Shipping Options and Tracking: Provide clear shipping options, estimated delivery times, and tracking information.
8. Customer Support:
Live Chat or Contact Form: Offer real-time customer support through live chat or a contact form.
FAQ Section: Create a comprehensive FAQ section to address common questions.
9. Improved Image Quality:
Higher Resolution Images: Use higher resolution book cover images to provide a better visual experience.
Zooming Functionality: Allow users to zoom in on book covers for a closer look.
10. User Reviews and Ratings:
Review System: Encourage users to leave reviews and ratings for books.
Average Ratings: Display average ratings for each book to help users make informed decisions.

Use Case
. Improved User Experience:
Enhanced Navigation: The cleaner layout and intuitive navigation would make it easier to find the books I'm looking for and explore different categories.
Personalized Recommendations: The personalized recommendation feature would help me discover new books that align with my interests, expanding my reading horizons.
Social Media Integration: The ability to share books on social media and connect with other book lovers would create a more engaging and interactive experience.
2. Increased Efficiency:
Advanced Search Functionality: The enhanced search options would allow me to quickly find books based on specific criteria, saving me time.
Mobile Optimization: The mobile-friendly design would enable me to browse and purchase books conveniently from my smartphone or tablet.
3. Enhanced Shopping Experience:
Wishlist and Gift Registries: The ability to create wishlists and gift registries would make it easier to organize my book collection and share my preferences with others.
Secure Payment and Shipping Options: The integration of reliable payment gateways and clear shipping information would provide a secure and hassle-free shopping experience.
4. Better Informed Decisions:
User Reviews and Ratings: Reading reviews and ratings from other users would help me make informed decisions about which books to purchase.
Book Descriptions and Details: Detailed book descriptions and information would provide a comprehensive understanding of each book before making a purchase.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

assign it to me sir
On Thu, 10 Oct 2024 at 00:42, github-actions[bot] @.***>
wrote:

Hi there! Thanks for opening this issue. We appreciate your contribution
to this open-source project. We aim to respond or assign your issue as soon
as possible.
—
Reply to this email directly, view it on GitHub
https://github.com/Trisha-tech/OnlineBookSales/issues/410#issuecomment-2403219459,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCN627U2USDQ54UZA3O4PMTZ2V523AVCNFSM6AAAAABPVGONMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBTGIYTSNBVHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Using LESS how do I extend .form-control class to apply to all form inputs

I am applying this theme to an application where I cannot add the .form-control class to the inputs themselves. A lot of these inputs are in tables and such so I would just like to use LESS to extend the .form-input class to be the default styles for all inputs unless overridden by specific styles. I seen this idea in another answer but I am unsure how to do it properly.

You mean you can not edit html ?   May be you could do something like
td input{
   //Now copy those css from .form-input into this place.
}

Closing due to inactivity. If this is still relevant, please let us know.
Thanks.


Purpose of the .fixed class

There's a .fixed class defined in dialog-polyfill.css, but it doesn't seem to be used anywhere in the library.
Is this class defined on purpose or is it just a remnant of some previous code?

It's left over from some experiment. The <dialog> technically loads in the viewport when you open it (so on a large page, you can scroll up/down and the dialog won't follow you), but honestly that's quite odd.
Maybe this is sort of related to what we should be doing for old IE support, just having an 'old IE/fixed/simple' class: #159

I've added some guidance on this in the README.


Add an indicator so we can easily determine if an account is in the CZGE_internal segment in split.

Summary:

What: Add section to account page to indicate whether an account is in the 'internal_user' split group.
Ticket: none
Env: none

Demos:

Notes:
We will use the CZGE_internal segment in split to test the new samples and trees tables.  Since we expect no change, it will be helpful to have a quick way to determine whether an account is in the segment.  This will not show for users who are not in the CZGE_internal segment.
Checklist:

[x] I merged latest <base branch>
[x] I manually verified the change
[x] I added labels to my PR
[x] I tested in multiple browsers
[ ] I added relevant unit tests
[ ] I have notified others of changes they need to make locally (migrations, jobs, package updates, etc)

Looks good! Long-term, this might be a nice place to stow indicators for any flags we want to watch generally. Only internal_user folk will get the section displayed, but within that section we could display a bunch of flag statuses. Or maybe just have some iterator that runs through all our flags and drops the values. But I'm just spit-balling long-term ideas, this is a great start.

Agreed!


Tags Cog: Add Option to List Tags Alphabetically

Currently, tags are listed by usage. This is not particularly useful if there are very many tags and you're trying to remember the specific name of a tag.
At a minimum, there should also be an option to view tags in alphabetical order. Moving the "___ uses" text to be beside the tag name rather than beneath it could help make the list easier to read.

The website has this functionality, complicating the bots in server use is a subpar ux compared to what we can offer on the website

yes, but most tags users are probably not power users.
How many people even on Clemson CPSC discord are going to take the time to go to the clembot dashboard, log in, select the server, go to the tags list, and find the tag they're looking for? Probably not many TBH.
It is inconvenient to leave the discord app for this functionality and creates a barrier impeding the mass usage of tags (or rather, to the discovery of previously unknown/un-used tags)

Ideally everyone 😁 its much easier then trying to figure out the complex strucutre for a  given command. adding more complexity when we have a much simpler approach isnt helpful. HOWEVER adding a link to the tags part of the dashboard IN the tags embed is an excellent idea. Give people something to click  on

didnt mean to close

Can I be added to this issue?


Packal Link doesn't work anymore

The packal link from the Readme does not work anymore:
http://www.packal.org/workflow/simplelogin-workflow

+1
Is this going to be updated?

Unfortunately, I don't think Packal is maintained anymore - I have not been able to add a new workflow there.. However, the workflow can be installed from the releases page on this GitHub project.. And it will update itself automatically from GitHub as well.


Add combobox primitive

What does it do?

Adds a new combobox primitive to be used in refactoring of the combobox made in a composable manor

Why is it needed?

needed to abstract the accessibility logic of the DS combobox

How to test it?

automated tests 🪄

I need to write some documentation, but from a code perspective i think this is good.

Some of the "stolen" code is a bit wild. I trust your judgement but wondering if it might be worth adding unit tests?

Can add unit tests if you'd like?

When I test the primitive in storybook, after selecting a value I am unable to clear the input, is that the expected behavior for the primitive?

When you say "clear" the input, do you mean with a button? or at all? because you should be able to delete the string, that might be a bug.

Yeah when you try to delete the string it just refills once the value is empty: design-system-git-feature-listbox-strapijs.vercel.app/?path=/story/design-system-primitives-combobox--basic-usage

Thanks for spotting that Mark, i added a new test and fixed the issue 😄

From functional testing:

the focus outline of the clear button doesn't look good

the element has a focus outline even though it is disabled (when clicking on the label)

From functional testing:

the focus outline of the clear button doesn't look good

the element has a focus outline even though it is disabled (when clicking on the label)

I haven't concerted the DS combobox to the Primitive yet, thats another task :)


npm-check and yarn-check actions

Tested manually.
Some screenshots (note we would not show notification if 0 packages to install):

Don't know enough typescript to review the code part

We could make a simple package with these features and ship them.
On Tue, May 2, 2023 at 4:42 PM Tyler Jang @.***> wrote:

@.**** approved this pull request.
Looks good, with important notes.
Tangentially, I was wondering if we should have some basic runtime utils
files in plugins. For common things like notification rendering, sarif
parsing, etc. Not sure how difficult that would be to wire up, though...
In .trunk/trunk.yaml
https://github.com/trunk-io/plugins/pull/273#discussion_r1183134478:

@@ -53,6 +53,7 @@ actions:
shell: bash

enabled:

npm-check

Can you add both to the top level readme
In actions/npm-check/npm_check.js
https://github.com/trunk-io/plugins/pull/273#discussion_r1183135729:

@@ -0,0 +1,54 @@
+#!/usr/bin/env node

+// trunk-ignore-all(eslint)
+const npmCheck = require("npm-check");
+const YAML = require("yaml");
+
+// TODO(lauri): Find more stable icon path
+const ICON_PATH = "https://avatars.githubusercontent.com/u/6078720";
+
+const pluralize = (count, singular, plural) => {

return count == 1 ? singular : plural;

=== is the paradigm throughout, I think?
In actions/npm-check/plugin.yaml
https://github.com/trunk-io/plugins/pull/273#discussion_r1183136468:

@@ -0,0 +1,15 @@
+version: 0.1
+actions:

definitions:

id: npm-check

 display_name: NPM Check

 runtime: node

 packages_file: package.json

 triggers:

   - schedule: 24h

   - files: [package.json, package-lock.json]

 run: actions/npm-check/npm_check.js

I think this (and yarn) should be ${cwd}/npm_check.js
In actions/npm-check/npm_check.js
https://github.com/trunk-io/plugins/pull/273#discussion_r1183136874:

.then((current) => {

const uninstalled = current.get("packages").filter((p) => {

 return !p.isInstalled;

});
if (uninstalled.length == 0) {

 const yaml = YAML.stringify({ notifications_to_delete: ["npm-check"] });

 console.log(yaml);

 return;

}
const uninstalled_count = uninstalled.length;
const yaml = YAML.stringify({

 notifications: [

   {

     id: "npm-check",

     title: "Check npm packages",

     message: `${uninstalled_count} npm ${pluralize(

What's our current story on message vs rendered?
In actions/npm-check/npm_check.js
https://github.com/trunk-io/plugins/pull/273#discussion_r1183137353:

   {

     id: "npm-check",

     title: "Check npm packages",

     message: `${uninstalled_count} npm ${pluralize(

       uninstalled_count,

       "package",

       "packages"

     )} ${pluralize(uninstalled_count, "needs", "need")} to be installed\n`,

     commands: [{ run: "npm install", title: "npm install" }],

     icon: ICON_PATH,

   },

 ],

});
console.log(yaml);
})
.catch((err) => {

Should this render an error notification? Or just a nonzero exit code and
show up on the next trunk check run? No strong opinions just want to make
sure it's not spammy
In actions/yarn-check/plugin.yaml
https://github.com/trunk-io/plugins/pull/273#discussion_r1183137883:

@@ -0,0 +1,15 @@
+version: 0.1
+actions:

definitions:

id: yarn-check

 display_name: Yarn Check

 runtime: node

 packages_file: package.json

 triggers:

   - schedule: 24h

   - files: [package.json, yarn.lock]

 run: ${plugin}/actions/yarn-check/yarn_check.js

prefer ${cwd}/yarn_check.js
In actions/npm-check/npm_check.js
https://github.com/trunk-io/plugins/pull/273#discussion_r1183135619:

@@ -0,0 +1,54 @@
+#!/usr/bin/env node

+// trunk-ignore-all(eslint)
+const npmCheck = require("npm-check");
+const YAML = require("yaml");
+
+// TODO(lauri): Find more stable icon path
+const ICON_PATH = "https://avatars.githubusercontent.com/u/6078720";
Agree. This should be doable if you pass in ${plugin} or ${cwd} as an
input to this script, right?
—
Reply to this email directly, view it on GitHub
https://github.com/trunk-io/plugins/pull/273#pullrequestreview-1410009986,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJVCPQKVPWNZPR3KVNDL7TXEGLVLANCNFSM6AAAAAAXSLRD6E
.
You are receiving this because your review was requested.Message ID:
@.***>

--
Eli Schleifer | Founder, Co-CEO | trunk.io

Follow-up work:
https://linear.app/trunk/issue/TRUNK-7003/create-npm-and-python-packages-for-creating-trunk-action-notifications
https://linear.app/trunk/issue/TRUNK-6654/distinguish-between-notifications-for-cli-and-those-for-vscode


bin links seem to cause issues when installing via npm

Installing poly-crypto via npm in the standard way results in errors during chmod for the cli files:
npm install poly-crypto
npm ERR! code ENOENT
npm ERR! syscall chmod
npm ERR! path /home/ubuntu/myapp/node_modules/poly-crypto/node ./cli/aes-pass-encrypt.js
npm ERR! errno -2
npm ERR! enoent ENOENT: no such file or directory, chmod '/home/ubuntu/myapp/node_modules/poly-crypto/node ./cli/aes-pass-encrypt.js'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2021-06-07T11_28_10_440Z-debug.log

Using the --no-bin-links option however, succeeds:
npm install poly-crypto --no-bin-links
added 4 packages, changed 1 package, and audited 237 packages in 40s
found 0 vulnerabilities

Details:
node --version
v16.3.0
npm --version
7.16.0

To me it seems the chmod operation takes the wrong path and thus failes.

Can you confirm on the solution above ?
I tried to do "npm install poly-crypto --no-bin-links" but still can not deploy correctly into Heroku.
The same ENOENT error occurs.

This was fixed with version 2.0.3.


Make react query's queryKey optional.

Fix #975
This changes the query options to mark queryKey as optional in function arguments. This allows the use___ hooks to add query options but not force overriding the query key as well. The query key is set internally in the generated code if not present so should be safe to make optional
Example - this no longer yields a typescript error about the queryKey option:
const { data: pets, refetch } = useListPets(undefined, 1, {
    query: {
      staleTime: 5000
    }
  });

Looks like tests are failing with this change?

I will close this one because I opened another one with Partial. I will go forward with that for now to be able to do a release asap. Let me know if it's an issue for you. Or if you see any other problems with that. Thanks a lot @leniency


Issue importing azure app service certificate into webapp

I believe this was running in AZ CLI 2.8.0, was working locally but failing in ADO, upgraded my local to reflect the same version now im able to reproduce the issue locally as well.
This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp config ssl import
Errors:
The command failed with an unexpected error. Here is the traceback:
'GraphClient' object has no attribute 'service_principals'
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 718, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/appservice/commands.py", line 39, in _ex_handler
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/appservice/custom.py", line 2700, in import_ssl_cert
    if not _check_service_principal_permissions(cmd, kv_resource_group_name, kv_name, kv_subscription):
  File "/opt/az/lib/python3.10/site-packages/azure/cli/command_modules/appservice/custom.py", line 2779, in _check_service_principal_permissions
    graph_sp_client = _graph_client_factory(cmd.cli_ctx).service_principals
AttributeError: 'GraphClient' object has no attribute 'service_principals'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp config ssl import --resource-group {} --name {} --key-vault {} --key-vault-certificate-name {} --debug

Expected Behavior
Environment Summary
Linux-4.4.0-19041-Microsoft-x86_64-with-glibc2.27, Ubuntu 18.04.4 LTS
Python 3.10.4
Installer: DEB

azure-cli 2.37.0

Extensions:
application-insights 0.1.1
logic 0.1.0
portal 0.1.1

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

route to CXP team

Duplicate of this issue https://github.com/Azure/azure-cli/issues/22761. Work around is provided in the other issue and we will continue to track this bug there, therefore closing this as duplicate.

Will track in #/22722


Does the implementation of reverseFlatMap cause non-private method used?

Description:
My lab5 is required to resubmit due to non-private used, I wonder if it’s the implementation of reverseflatmap result in this issue(i call it in f() in jsh file). Because I delete it and redefine the f() by using flatMap only, and then it passes the test cases.
Can anyone tell if the implementation of reverseflatmap violate non-private? (my map flatmap and reversemap all don’t state private either)
Screenshots

An extra public reverseFlatMap method will result in non-private methods used error. You should only have public methods which are used in the test cases given. Getters should also be private.

in general, public methods should only be the ones stated in the question, any other public methods will be flagged out, hope this helps!

Unfortunately, this implementation does not work when the lab assignment was updated. You just need to use flatMap appropriately in level5.jsh to achieve the reverse effect.
Consider looking at these #295 and #293.

the lab assignment was updated so we are not allowed to have helper function in level5 anymore... try to just use one recursive function, think more on how to connect the logs in a reversed manner using flatmap

you can try to swap the order around with the methods that you already have within the Log class

I hope it all worked out for you, @IYKYKY. If you find it great, you might want to close this issue. Thank you!


Cant resize or move popup window

Tried just about everything to get the window resize and make it popup where i want it to without success, maybe take a look and fix size  retainment.

@boe323 are you referring to popup window from the PowerToys settings?

Sorry it took so long to get back....im not that savvy when it comes to reporting things lol. Yes im referring to the settings  panel of power toys, it cannot be set to a certain area and wont retain the window size .

Just to mention that it still persists through out every update.

@boe323
this is currently by design until fixed by https://github.com/microsoft/PowerToys/issues/1445.

Ok.....thanks for getting back.


Issue at first connection using config flow

First of all, thank you very much for developing the component.
There are many stoves using this system (mine is Edilkamin) and being able to integrate them in home assistant is a big step forward.
I have a problem when adding the integration. I think it is related to hottohpy.
The first time I try to add the stove, I get the error "Unexpected exception".
In the home assistant log it says the following:
2021-11-19 13:21:50 ERROR (MainThread) [custom_components.hottoh.config_flow] Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/hottoh/config_flow.py", line 87, in async_step_user
info = await validate_input(self.hass, user_input)
File "/config/custom_components/hottoh/config_flow.py", line 39, in validate_input
info = await async_connect_or_timeout(hass, hottoh)
File "/config/custom_components/hottoh/__init__.py", line 247, in async_connect_or_timeout
name = hottoh.get_name()
File "/usr/local/lib/python3.9/site-packages/hottohpy/hottoh.py", line 256, in get_name
if self.get_manufacturer() is None:
File "/usr/local/lib/python3.9/site-packages/hottohpy/hottoh.py", line 267, in get_manufacturer
return self._getManufacturer()
File "/usr/local/lib/python3.9/site-packages/hottohpy/hottoh.py", line 290, in _getManufacturer
return str(self._data[StoveRegisters.INDEX_MANUFACTURER])
AttributeError: 'Hottoh' object has no attribute '_data'

If I try to add it again, the error becomes "Failed to connect", and the log shows this:
2021-11-19 13:22:45 ERROR (hottohpy) [hottohpy.client] Can't connect to <IP_ADDRESS>, error: [Errno 106] Socket is connected
2021-11-19 13:22:45 ERROR (hottohpy) [hottohpy.client] Can't connect to <IP_ADDRESS>, error: [Errno 106] Socket is connected
2021-11-19 13:22:45 ERROR (hottohpy) [hottohpy.client] Can't connect to <IP_ADDRESS>, error: [Errno 106] Socket is connected
2021-11-19 13:22:45 ERROR (hottohpy) [hottohpy.client] Unable to connect to <IP_ADDRESS>
2021-11-19 13:22:45 WARNING (hottohpy) [hottohpy.hottoh] Unexpectedly disconnected from Hottoh <IP_ADDRESS>, code Unable to connect to Hottoh at <IP_ADDRESS>

I think that due to the first error, the socket is somehow left open, and it cannot be reopened.
I have seen that you have uploaded a new version of hottohpy (0.0.17), I have tried updating the dependencies to this version, but the problem persists.

Hello,
there is two issues here:
-1- your first error should be corrected with hottohpy version 0.0.18, I corrected the error.
-2- for the second, I know I still have some problem when the integration is created at first, HA needs to reboot, I need to work on that.
Since this is my first integration and also my first python experience, I will try to do my best ;-)

I've just push new hottohpy version with improvements for the config flow (0.0.21)
and also pushed new version of the integration.
I would also be interested by the vendor value of your stove, to add the builder name:

Wow! That's fast!!
It may be your first integration, but it's certainly a very useful integration and one that I'm sure many people will appreciate.
I have just tested the new version. I will try to describe It's whole behavior.
First of all, after starting home assistant (even before configuring the integration) I got this error:
2021-11-20 00:31:45 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 197, in async_init
flow, result = await task
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 224, in _async_init
result = await self._async_handle_step(flow, flow.init_step, data, init_done)
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 320, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
File "/config/custom_components/hottoh/config_flow.py", line 70, in async_step_dhcp
info = await validate_input(self.hass, [host, HOTTOH_DEFAULT_PORT])
File "/config/custom_components/hottoh/config_flow.py", line 38, in validate_input
address=data[CONF_HOST],
TypeError: list indices must be integers or slices, not str

I had already seen this error with the previous version. I have also seen something similar trying to run hottohpy on my desktop PC.
Despite the error, I have tried to configure the integration.
The first time I did it, it added "stove 85" to the list of integrations. However, a red error appeared indicating that the integration could not load. It also did not add any devices or entities.
I checked the logs and saw this:
2021-11-20 00:37:47 ERROR (hottohpy) [hottohpy.client] Socket already connected to <IP_ADDRESS>
2021-11-20 00:37:47 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:47 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:47 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:47 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:47 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:47 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor

...... More than 600 entries redacted .......

2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:57 WARNING (MainThread) [homeassistant.config_entries] Config entry 'Stove 85' for hottoh integration not ready yet; Retrying in background
2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor
2021-11-20 00:37:57 ERROR (hottohpy-client) [hottohpy.client] Connection Error : [Errno 9] Bad file descriptor

After seeing the flood, I deleted "Stove 85". The flood has stopped.
Then I have tried adding the integration again. This time, the error message "Failed to connect" appears, the config form is not closed and no "Stove 85" was added. The log show:
2021-11-20 00:49:22 ERROR (hottohpy) [hottohpy.client] Can't connect to <IP_ADDRESS>, error: [Errno 9] Bad file descriptor
2021-11-20 00:49:22 ERROR (hottohpy) [hottohpy.client] Can't connect to <IP_ADDRESS>, error: [Errno 9] Bad file descriptor
2021-11-20 00:49:22 ERROR (hottohpy) [hottohpy.client] Can't connect to <IP_ADDRESS>, error: [Errno 9] Bad file descriptor
2021-11-20 00:49:22 ERROR (hottohpy) [hottohpy.client] Unable to connect to <IP_ADDRESS>
2021-11-20 00:49:22 WARNING (hottohpy) [hottohpy.hottoh] Unexpectedly disconnected from Hottoh <IP_ADDRESS>, code Unable to connect to Hottoh at <IP_ADDRESS>

These lines appears each time I try to add the stove, but only 5 lines each time, not flooding.
I hope I have explained it well enough.
Thank you very much for your quick response, if I can help you in any way do not hesitate to ask me.

Interesting.
After restarting Home Assistant again, I have managed to add the stove successfully.
Here is the "manufacturer" that appears:

Well, there still seems to be a problem:

This message appears 40.000 times in ten minutes.

Ok...
There were two problem,
first, I have try to enable discovery form dhcp to automatically add Hottoh integration to HA, but on my side it didn't detect it so I have not been able to test it correctly -> It have been removed from now
second, it should be better in version 0.0.22 of hottohpy and version 0.04 of this integration.
You will still have an error just after adding the integration, but after few seconds, the climate and sensors will arrive. This is because the stove did not accept more than one simultaneous connection.
Can you tell me if it's better ?
I'm also actually thinking of removing the config_flow and go back to configuration.yaml because, I have added three preset (away, eco and comfort) and there is no way to reconfigure them without going trough configuration.yaml
Going back to configuration.yaml will also really simply the code for all the error you got ;-)

You are quicker at fixing problems than I am at testing them. Hahah
I can confirm that it now works perfectly.
As you pointed out, it gave connection errors just after adding the integration. But after that everything works fine.
I see that it exposes much less data than can be seen/controlled in jeedom. But this is version 0.0.22, and just being able to turn the stove on and off from home assistant is a huge step forward. You are on the right track.
No doubt there is work ahead, my python knowledge is quite basic but I'll be happy to help you in any way I can. As for example in the translation of the strings to spanish.
In my opinion, I think the config flow is very convenient for certain types of integrations. However, I believe that for this particular one, the .yaml configuration is perfectly valid. People are not adding stoves to their homes every day.
So, if this simplifies the code and avoids potential problems, I think going back to .yaml is a good option.
Thank you again for your work.

Ok cool thanks for your feedback !
There is other data points than I can output from the integration

but I know, my stove did not support all of them and I'm not sur to have all possible.

Mine is a water stove, and the application allows these parameters to be displayed/controlled:

In the main screen:

[ ] In the upper left corner there is an icon indicating whether the water pump is running or not.

[X] In the middle there is an icon indicating the current status. This is already exposed, however, I have seen the following:
When the pellets are running out, the cooker makes a beep. However, neither the app icon nor the home assistant "action" entity change, it is kept on "heating".
If the stove is switched off due to lack of pellets, then yes, the entity "action" changes to "stopping" and then to "end pellet".
I don't know if it would be possible  to read an extra status indicating "low pellet". But it would be a very useful feature, to be able to notify about the lack of pellets via home assistant.

[X] Below the icon, it shows the current environment temperature, which is already exposed.

[ ] Finally, the water temperature is displayed. I guess it corresponds to the index #17 of your screenshot.

In the status screen:

[ ] Firstly, the power at which the stove is operating. Guess it is parameter #22.

[X] Set and Environment, already exposed.

[ ] Water temperature, the same as shown in the main screen.

[X] Fumes temperature, already exposed.

[X] Fumes fan speed, already exposed. As you see in the snapshot, the unit seems to be g/m and not rpm. But not sure if this is correct in the app.

[ ] Last one, air exchanger fan speed. This is not exposed and I think it must be #27, #30 or #33.

In the control screen:

[X] Firstly, the Eco mode, already exposed. I really don't know if it has any real effect on my stove.

[ ] Next, the Chrono mode, not exposed. For me it is secondary, because thanks to the integration, it will be possible to schedule the stove using home assistant.

[ ] Next, the programmed power. I don't know how it behaves because the stove adjusts this value automatically according to the temperatures. It's not exposed but I guess it is #23.

[X] Next, the environment setpoint. Already exposed.

[ ] Last, and very important In my opinion, the air exchanger fan speed. I can change it from 0 - 5 and "Auto".  I suppose it will be #28, #31 or #34.

As you can see, the app does not let you adjust the water temperature setpoint. But, according to your screenshots, I think it is possible to adjust it using parameter #18. This would be very very useful in my opinion, especially because it is not possible from the app.
I'm sorry for rambling, It is not my intention to give you so much work, only that you know which parameters are interesting in the case of water stoves.
Feel free to move this to a new issue or enhancement (or six, or seven ;) ). This is probably not the best place to discuss it.


initial test on Balatro+ (App Store and Apple Arcade edition)

If anyone's thinking of adding support for the Mac App Store version of Balatro, and the Apple Arcade ("Balatro+") release, here's some initial information from a couple of hours playing around with it. Caveats: I have very little lua experience and even less rust, nor am I an iOS/macOS developer, so consider everything here vague at best and unverified. I haven't even bought Balatro on Steam, so all my experience is with the AA version on iPadOS and macOS.
All the exploration below is with the macOS version of Balatro+, though I'm guessing the packages for the other Apple OSes have very few differences: the source includes the logic for iOS/iPadOS and tvOS. Similarly, it would surprise me if there's any major difference between the non-Arcade App Store package and the Apple Arcade Balatro+ package.
The Balatro+ (Apple Arcade version) package includes:

the game's (updated for Apple OSes and APIs) lua source and assets in Contents/Resources/game/

a custom binary launcher: Contents/MacOS/Balatro

a license.txt which specifies:
Clockwork Valley ("Licensor") grants LocalThunk, inc ("Licensee") a non-exclusive, non-transferable, revocable license to access and use the software solely for the purpose of assisting in the development of Balatro Arcade.

(According to the file, Clockwork Valley is Maarten de Meyer. The file also lists these open source components: LOVE, ENet, FreeType, GLAD, glslang, Kepler Project's lua-compat-5.3, lua-enet, LuaJIT, Lua's UTF-8 module, LuaSocket, LZ4, LodePNG, TinyEXR, UTF8-CPP, xxHash, dr_flac, stb_image, libmpg123, OpenAL Soft.)
My guess is that the Balatro binary is basically love but with extra logic compiled in for exposing Core Data, CloudKit, GameKit, and other Apple APIs, all gathered into a love.platform table.
Fascinatingly – and wonderfully – most if not all of the new logic that uses those APIs seems to  be out in the relative open in the included lua source, rather than more hidden in that custom binary. You can, for example, see lots of specific handling for tvOS.
So that's the good news. Now for some bad news:

you can't modify the lua source, even as root - I'm guessing it's all covered by SIP
the Balatro binary appears to be closed off to the dyld hijacking trick that the macOS version of lovely uses. (I tried compiling and using the much simpler tool on that page, and verified that it worked on the standard love binary.)

At this point I need to stress that I am not an experienced C or macOS hacker, and I may well have missed some major windows that are still open. More importantly, the Balatro binary could have been written with command line arguments that allow for custom logic injection, even though the standard love command doesn't have any. (Maybe we should just ask Maarten de Meyer.)
Talking of the standard love command, what happens when you point a normal love2d build at Balatro+'s game folder? It actually gets pretty far: it produces a nice big window, loads the m6x11plus.ttf font, and uses it to display the traceback showing that it got about three lines into main.lua's love.load event before hitting a call to love.platform.earlyInit - which, of course, doesn't exist in the standard love2d API.
But that's far enough to show that those new APIs are what's getting in the way, and fortunately all the interaction points are clear in the lua code. Some of those methods are easily stubbed/faked, but others have been moved in as the primary load-bearing I/O points for loading and saving the player's profile and progress.
The main issue I hit with lovely: currently, on macOS, it demands that love exist within a .app folder - why?  I was using a homebrew-installed binary and ended up commenting those lines out and rebuilding. (Yes, I could have faked it, but I was in the source anyway.)
I'm leaving this here for now. I'm happy to answer questions and try things out, but please don't expect me to do much more work on this. Others who have more experience will likely get much further, faster.

Does the balatro .app have a Frameworks/Lua.framework from the apple arcade? If not it's probably like the iOS version, which I've been working on trying to get lovely to work on, but haven't quite got success

@WilsontheWolf No Frameworks/ filder and no .framework files anywhere. Yup, I'm fairly sure this is the same as the iOS version.
I've made a little more progress since my first message.
The Balatro binary seems to accept a couple of the same flags that love does:
/Applications/Balatro.app/Contents/MacOS/Balatro --version prints LOVE 11.5 (Mysterious Mysteries) - so yes, this is basically a love build with extra stuff thrown in.
/Applications/Balatro.app/Contents/MacOS/Balatro --game $NAME looks for a folder relative to the game's save data location, which is ~/Library/Containers/com.playstack.balatroarcade/Data/
I tried copying the game/ folder (which has all the lua source and resources) out of /Applications/ and into somewhere more usable, then added a symlink to it in the Data/ folder above.
/Applications/Balatro.app/Contents/MacOS/Balatro --game game tries to run the copied game folder but produces this error:
Error: [love "boot.lua"]:278: module 'conf' not found:
	no field package.preload['conf']
	no file 'conf' in LOVE paths.
	no file './conf.lua'
	no file '/usr/local/share/luajit-2.1/conf.lua'
	no file '/usr/local/share/lua/5.1/conf.lua'
	no file '/usr/local/share/lua/5.1/conf/init.lua'
	no file './conf.so'
	no file '/usr/local/lib/lua/5.1/conf.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'
stack traceback:
	[love "boot.lua"]:354: in function <[love "boot.lua"]:350>
	[C]: in function 'error'
	[love "boot.lua"]:278: in function <[love "boot.lua"]:126>
	[C]: in function 'xpcall'
	[love "boot.lua"]:364: in function <[love "boot.lua"]:357>
	[C]: in function 'xpcall'

... which is weird, since there's definitely a conf.lua in that folder. I've also tried running this command while my working directory is the game folder (so ./conf.lua exists) but it still can't find it.

Okay. In that case, you can probably extract the game.love and stick it into a regular macos build of balatro, and then you should be able to use lovely

... and minutes later, some actual success:
The problem seems to be that the Balatro binary can't properly read files outside of the /Applications/Balatro.app/ folder. (See, I'd probably know things like this if I was a proper macOS dev.) So I (as root) copied /Contents/Resources/game/ to /Contents/Resources/moddedgame/, tweaked some of the strings in the en-us.lua locale file, and ran
/Applications/Balatro.app/Contents/MacOS/Balatro --game '/Applications/Balatro.app/Contents/Resources/moddedgame/'

... and it booted up fine, and showed the altered strings. Even better, it loaded my saved game profile as normal.
So I'm thinking that a modding process that copies the source over and runs the lua transformations on the copied version may be enough here?

@WilsontheWolf Yep, either keeping it unzipped or compressing to a .love file, either should work as long as it ends up in the Balatro.app folder.

So the reason why it cant see files outside of the Balatro.app folder is because of App Sandbox.
It can access files in the .app and ~/Library/Containers/com.playstack.balatroarcade
Also the app has com.apple.security.cs.disable-library-validation so injection should be possible with a code signed library.

Also the Apple Arcade version on the Mac is not the same as the non Apple Arcade version since the non Apple Arcade version is iOS only.

Just tested what you did earlier and having it in ~/Library/Containers/com.playstack.balatroarcade/Data/ and running with --game works.

bump to see y'alls progress

Another thing to know. The binary file in Balatro.app is FairPlay encrypted so doing stuff with the binary is most likely a dead end. It won't be easy to mod the Apple Arcade version without just manually patching the game files.


Circular reference and attributes.

Moved from https://github.com/fsharp/fsharp/issues/565
Repro steps
Paste the following code in FSI
open System

[<AttributeUsage(AttributeTargets.Property)>]
type MyAttribute() =
    inherit Attribute()
    member this.Foo () =
        new MyClass(1)
and MyClass(someValue) =
    [<MyAttribute()>]
    member this.SomeValue : int = someValue
;;

Expected behavior
It would compile to
type MyAttribute =
  class
    new : unit -> MyAttribute
    member Foo : unit -> MyClass
  end
and MyClass =
  class
    new : someValue:int -> MyClass
    member SomeValue : int
  end

Actual behavior
      [<MyAttribute()>]
  ------^^^^^^^^^^^

stdin(90,7): error FS0848: A custom attribute must invoke an object constructor

Known workarounds
Do it in C#.
Related information
Ran in fsi
Microsoft (R) F# Interactive version 14.0.23413.0

And in Visual Studio 2015 Update 2 with using a DLL with .NET 4.6.1, and the <IP_ADDRESS> F# Runtime.
This is done on a Windows 8.1 machine, but can reproduce on a Windows 10 (latest patches, until the one from 4/12/2016, non-Insider branch).
Use case:
namespace Test

open System
open System.ComponentModel.DataAnnotations

[<AttributeUsage(AttributeTargets.Property, AllowMultiple = false, Inherited = true)>]
type MyAttribute() =
    class
    inherit ValidationAttribute ()

    override this.IsValid (value: Object, validationContext: ValidationContext) =
        match validationContext.ObjectInstance with
        | :? MyClass as item ->
            // TODO more validation
            ValidationResult.Success
        | _ ->
            new ValidationResult("No no no")
    end
and MyClass(someValue) =
    [<MyAttribute>]
    member this.SomeValue : int = someValue

Note this is a good test case for https://github.com/fsharp/FSharpLangDesign/blob/master/RFCs/FS-1009-mutually-referential-types-and-modules-single-scope.md
The code would become the more pleasant:
namespace rec Test

open System
open System.ComponentModel.DataAnnotations

[<AttributeUsage(AttributeTargets.Property, AllowMultiple = false, Inherited = true)>]
type MyAttribute() =
    inherit ValidationAttribute ()

    override this.IsValid (value: Object, validationContext: ValidationContext) =
        match validationContext.ObjectInstance with
        | :? MyClass as item ->    ValidationResult.Success
        | _ ->     new ValidationResult("No no no")

[<MyAttribute>]
type MyClass(someValue) =
    member this.SomeValue : int = someValue

@dsyme notice that You're applying [<MyAttribute>] to the class, it should be applied to the member, not sure that makes a difference.

Ah yes, thanks. Now fixed above.

As a workaround, you could define your types in a signature file first, this seems to work:
Foo.fsi
namespace Foo

open System

[<AttributeUsage(AttributeTargets.Property)>]
type MyAttribute =
    inherit System.Attribute

    new : unit -> MyAttribute

    member Foo : unit -> MyClass

and MyClass =
    new : someValue : int -> MyClass

    [<MyAttribute()>]
    member SomeValue : int

Foo.fs
namespace Foo

open System

[<AttributeUsage(AttributeTargets.Property)>]
type MyAttribute() =
    inherit Attribute()

    member this.Foo () =
        new MyClass(1)

and MyClass(someValue) =
    // [<MyAttribute()>] -> specified in the fsi, still appears in compiled code
    member this.SomeValue : int = someValue

This was fixed by the implementation of RFC FS-1009


After cross compiling the AVS SDK. How to check and  run the sample app on iOS?

Briefly summarize your issue:

It is not an issue, it is just a question. I have completed all the steps successfully which are necessary for cross-compiling for iOS here: https://github.com/alexa/avs-device-sdk/wiki/How-to-cross-compile-the-AVS-Device-SDK-for-iOS
Now how can I run my sample app on the simulator?
What is the expected behavior?

I want to run the sample app on iOS
What behavior are you observing?

I am not able to run the app
Provide the steps to reproduce the issue, if applicable:

I have done all the compilation which is necessary to build AVS SDK on iOS
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

Tell us what hardware you're using:

[ *] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ *] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi @ShahShwetaj,
Unfortunately, we don't yet support the SDK on iOS, so I do not have any simple advice.  Of course, you can pursue creating an iOS application that wraps the SDK's native code, but you will need to to the work of integrating the SDK to the platform (e.g. microphone, speaker, media player, etc.).
Best regards,
-SWH

I am closing this issue due to inactivity. Please feel free to re-open it if it has been closed in error.

Unfortunately, I'm stuck with the same issue. I'm not able to find any help to integrate Alexa in my iOS app.
@ShahShwetaj did you find any solution?


Correct syntax errors

Corrected some of the syntax errors. I've seen that you are not using go convention for function comment, is there a reason for it? I can fix these for you if you as well.

Sure! I'll appreciate it.


Updated dependency db-key to version 0.1

Updated dependency db-key to version 0.1 and made some necessary modification to fit the new version. But I haven't updated tests and the documentation yet.

This would be great if it was merged. Whenever I use this crate, cargo-outdated will complain about db-key.
Does this repository not run GitHub actions? Might be good to add as well, so nothing breaks.


(FACT-1241) Use dmidecode from puppet-agent when available

This retargets #1212 against master

This requires puppetlabs/puppet-agent#387

:+1: waiting on dependent PR before merging.

@peterhuene @haus has +1'd that PR, so if you think it's fine you can probably hit it to get this one in

Wait on agent CI (currently borked) or merge?

:+1: to merge now, as per-commit facter#master is green, and only daily job is failing due to pxp-agent on solaris.


Fix entry point id lookup in initial actions collection

When an attack graph gets constructed, nodes are added to it with IDs that are sequential and map exactly to the nodes' indices in the graph's node list. (In case we are loading from a file, the node IDs are used as described in the file; but this makes no difference to the rest of the discussion).
In the simulator, when an attack graph gets loaded, pruning of nonviable and unnecessary nodes may happen:
https://github.com/mal-lang/mal-simulator/blob/599e47a21eaa302b5ac9c509342082c19d3c5b9b/malsim/sims/mal_simulator.py#L83-L85
(BTW, wouldn't it be more fitting if prune_unviable_unnecessary) was a sim setting?)
This will result in indices in attack_graph.nodes not mapping to the node.ids anymore. That's fine, we have the MalSimulator._id_to_index and _index_to_id properties to follow the nodes in the list.
The issue here, when collecting the initial actions, is that there is no reason to go from the node.id to the list index. If  truncation is in place this will result in the wrong node being picked up. There is no way for the user to control this, at least if they start with a handcrafted model file, or one generated via Securicad (and potentially the KTH UI tool), together with a MAL file; node IDs do not appear in any of those.
I believe there is a broader issue in the simulator when truncation happens, i.e. there may be more places where the translation between indices and node IDs is either missing or messing. For example a few lines below my edit, for the defender.

False alert, all seems good here.


health_check failed: No such file or directory @ dir_s_mkdir - /app/tmp/cache/AC2/CA0.

Today I´m using this lib with following setup:
Ruby: 3.2.0
Rails: <IP_ADDRESS>
health_check.rb:
config.standard_checks = ["site", "cache", "database", "rabbitmq", "redis", "sidekiq-redis"]

the health check endpoint is hit several times from Kubernetes readiness and liveness probes, load balance health checks and external SaaS of monitoring.
And belong a period of one minute, for example, always appears few errors like this:
health_check failed: No such file or directory @ dir_s_mkdir - /app/tmp/cache/AC2/CA0.
Completed 500 Internal Server Error in 2ms (Views: 0.2ms | ActiveRecord: 0.0ms | MongoDB: 0.0ms | Allocations: 407)

Feels like a concurrent hit is trying to use the same cache object and conflict (while one hit is clean the cache object, another concurrent hit is trying to use the object that is being erased in the cache)
I can´t clean this kind of error and is not an error of directory permission.
Did somebody have an issue close to these before?!?!

I remember seeing a few cases in the codebase where concurrency support/thread safety needs to be taken into account. So I'm inclined to agree with your prognosis.


changes in Braid.py, minor space details elsewhere

This makes some changes towards using polynomials in braid.py. I have added sketches of doctests, that should be filled and checked to behave as expected.
Also removing trailing whitespaces in a few other files

ping ?

Thank you @fchapoton ! I have been swamped lately. I will get to this soon. Thanks for the ping.

Excellent, thank you. Proper documentation along with using the polynomial data structure needs to be done throughout. Thank you for starting this!


Add a mechanism to configure the video output format

It's currently hardcoded to the first available option (which is the lowest supported resolution).

Duplicate of #7.


Bring updated datepicker to rg2

https://github.com/RiotGear/rg-datepicker/pull/12#event-420117396

Fancy writing some tests for it?!

Sure. Any tests you want besides:

Options Passthrough: https://github.com/RiotGear/rg/blob/master/components/date/rg-date.tag#L57-L68
Locale: https://github.com/RiotGear/rg/blob/master/components/date/rg-date.tag#L75-L83
Buffers are Correct: https://github.com/RiotGear/rg/blob/master/components/date/rg-date.tag#L108-L125
Date Change event fire: https://github.com/RiotGear/rg/blob/master/components/date/rg-date.tag#L140-L146
Shortcuts Work: https://github.com/RiotGear/rg/blob/master/components/date/rg-date.tag#L148-L177
Showing/Hiding Work: https://github.com/RiotGear/rg/blob/master/components/date/rg-date.tag#L85-L91

That's a good start! If you run the karma tests and check the coverage page to see what else needs doing. 100% is ideal 😉

I'm only talking about the date picker though btw!


Add an opt in option to use imagemin instead of wasm libs

Trying to fix https://github.com/cyrilwanner/next-optimized-images/issues/195 i ended up patching the loader to use image-min so i can build the images, so i was thinking maybe we could add an opt in option to use system libs instead of wasm, it should be a little bit faster and it can use as much memory as the system has. I am willing to make a PR, but i wanted to check in with you first to discuss about the configuration structure. When i have time i'll make a proposal :).

Since you are talking about memory problems I guess you had the same RangeError: WebAssembly.Memory(): could not allocate memory?
Would be really happy to see your proposal :)

@andreisergiu98 Do you mind sharing your solution? I am stuck. My prod used to work with the previous version (though slow). When I migrated to canary v10, my 2G RAM VPS is not able to build site that has only 4 images! Now I am stuck and I don't want to roll back. BTW, works perfectly fine on my laptop with plenty of RAM.

@semyou I cloned this repository and replaced wasm-codecs from src/optimize/jpeg.ts and src/optimize/png.ts with imagemin (with mozjpeg and pngquant plugins), then i added it into the project as a yarn workspace. I think yarn resolution would also work.

Also need a solution for RangeError: WebAssembly.Memory(): could not allocate memory, subscribing.


Add natural error translation for RPC

This update adds support for RPC_STATUS so that you can easily turn it into an HRESULT and use automatic error propagation with ? and Result for RPC functions.
Note I also opened this issue with the Win32 metadata: https://github.com/microsoft/win32metadata/issues/1857

@lhecker @zadjii-msft - this should simplify things for your RPC interop.


Continuous Integration

Set up continuous integration with TravisCi to run tests, build docs and deploy docs.

closed by scitran/data@9aba149969a4d86d18cf0d731fbd562cc0e8bfeb
TravisCI is configured to run tests.  Travis will only build and deploy docs for the master branch if tests pass.  Docs are not deployed for feature branches.


Create plan for splitting out DS code into a separate extension

Reasons for doing this:

Make the python extension smaller
Ship on a different schedule
Clearer to show we support other languages than Python
Clearer separation of code bases
Ship gather in a different extension that might not be entirely OSS

Different options:
Option 1 (Dependent Extensions):

Interactive window command and code lenses remain in the python extension
Kernel/UI/Data viewers/VS code notebook move to new DS extension
Python extension exports process creation and activation code
Running one of the code lenses or opening an ipynb file will prompt to download the jupyter extension
Jupyter extension doesn't actually need the python extension to be there
Jupyter extension by itself will ask to download python extension if python kernel is used
Python extension still bundles the jupyter extension

Option 2 (Bundled Extensions):

All DS code moves into separate extension
Python extension exports process creation and activation code
Jupyter extension is bundled with the python extension

Option 3 (Multiple bundled extensions):

DS code moves into multiple extensions

VS code renderers
VS code loading
VS code kernel
Interactive window
Data viewing

Python extension exports necessary process creation and activation code
Whole thing ships in a large bundle.

Not necessary at the moment.


Provide a way to require Creative Commons license in submission form

Describe the bug
I need the Creative Commons step to be required in submission process. I tried changing "mandatory" to "true" in step definitions (item-submission.xml) but it didn't work.
...
To Reproduce

Change configuration "mandatory" to "true"
Restart tomcat
Start new submission
Fill all required fields, except cc license
Submit submission
Done, but shouldn't

Expected behavior
Expected that form validation does not allow the submission to be submitted

Pulling this over to 7.6 board to see if we can find a volunteer.  I suspect the "mandatory" flag isn't being properly checked on the Angular UI side of things (just a quick guess though).

Revisiting this ticket as a duplicate was just submitted in https://github.com/DSpace/DSpace/issues/9603
This is a misunderstanding of "mandatory", as "mandatory" in the Submission form just means the step is always displayed. It does not require fields to be filled out.
See https://github.com/DSpace/DSpace/issues/9603#issuecomment-2124524264

Not a bug, just poor wording. According to the documentation:

"CC License" section (id="cclicense"): This step allows the user to (optionally) assign a Creative Commons license to a particular Item

This is meant to be optional. "mandatory" is a different thing

mandatory (attribute): [true|false] When true, the step's section is displayed by default to all users in the UI. When false, the step is not displayed and must be activated explicitly by the user by selecting it in the UI or supplying data of interest to the section.

That said, I'm leaving this ticket open and recategorizing as a feature request.
It's currently not possible to require all users to fill out the CC license field. In the future, this might be possible via a cc.license.required configuration which controlled whether the field could be left empty or not. (This would be similar to how we have a webui.submit.upload.required field to decide if uploads are required or not.)

@tdonohue
we can work on this task

Thanks @atarix83 .  I'll assign this to you then

Hello i have provided a pr for this issue :)  https://github.com/DSpace/DSpace/pull/9639


Unset variables that interfere with named directories

If left defined, ~script_location appears in the prompt when navigating to
powerlevel9k's checkout directory (due to setopt autonamedirs, it's a named
directory)

#562 also does this and more.

@docwhat yes. You are right. Sorry, I was about to review your PR, but that requires some more testing. ;) Will do soon.

I've been using since I posted it.
If you give me some test cases I can try them out for you.

@docwhat That would be awesome! I'll write them down in #562

@cdlm - Thanks so much for the PR, and welcome to P9k! =)
@docwhat @dritter - Since we are in the process of merging #562, I want to keep things clean. You mention above that #562 also fixes this problem. Does that mean that this PR isn't necessary as it is a subset of #562?

Correct!

Yeah somehow I missed #562 before sending this PR, but it includes the same fix…

No problem, @cdlm! I just merged #562, so we should be good, here.
Note that #562 was merged into next, not master, so you'll need to be on our next branch to see the change.
@cdlm - This was a good PR, and I look forward to your next contribution =)


Fix javadoc publishing of maven artifacts

The sources artifact published to Maven Central only contains Kotlin files. The javadoc artifact is empty.
Consequently, users cannot look up the Javadoc of a method from within the IDE and code-completion provides ugly parameter names like HttpUrl.get($this$toHttpUrlOrNull).
Please publish full Javadoc to Maven Central. Hopefully that will fix parameter names and let users look up the Javadoc on the fly.
(I personally consider this issue to be a bug, not a feature request, but I leave it up to you to decide)

Definite bug and thanks for flagging!

@cowwoc The sources artifact should only contain kotlin files.  OkHttp is implemented in Kotlin.

Could hook up dokkaJavadoc for the Maven artifact.
https://blog.jetbrains.com/kotlin/2020/08/dokka-preview-based-on-kotlin-1-4-0-rc/

Hopefully it's improving but when I looked at this previously the upgrade wasn't a tap in
https://github.com/square/okhttp/pull/6342
https://github.com/Kotlin/dokka/issues/1592

@swankjesse We should be okay so long as code-completion and Javadoc works properly in the IDE. Normally I like to be able to debug into the source-code, but so long as the method, parameter names are correct with Javadoc I think it's fine :)

@cowwoc source appears to be publishing fine https://repo1.maven.org/maven2/com/squareup/okhttp3/okhttp/4.9.0/
Likely there is something else going wrong with your IDE stopping this working.  It works correctly for me in downstream projects with Intellij. It contains all the source code as *.kt files.
But publishing javadoc artifacts should be fixed.

@yschimke The sources are there but if you CTRL+LMB click in IDEA it will open the decompiler instead of the sources. This is likely a side-effect of your usage of Kotlin. Adding Javadoc should be enough I guess.

I think if you just create your own dokka task and name it javadocJar before calling withJavadocJar() it'll work fine out of the box.

@ansman I'm not an expert on dokka, if it's that simple we'd happily accept a PR enabling that.

It is that simple, unfortunately dokka 0.10 does not support javadoc generation on JDK 11. I tried upgrading to dokka 1.4.20 but I hit the same issue as you filed: https://github.com/Kotlin/dokka/issues/1592

Working for 5.0, closing as low priority.  Sorry.
https://repo1.maven.org/maven2/com/squareup/okhttp3/okhttp/5.0.0-alpha.11/


Feature Request: Capturing Network Traffic via Packet Monitor

Packet Monitor (PktMon) was introduced since Windows 10.0.19041.
Will PerfView support that?

@yang-er, there aren't any current plans to do something specific around Packet Monitor, but it looks like Packet Monitor produces an ETL file that is consumable via NetMon, and PerfView should already be able to open this ETL file.  Is that what you're looking for?

As my understand, currently PerfView would combine the netsh trace result and standard ETL result into one ETL file.
So here I am thinking about getting a combined ETL file with standard ETL events (Kernel, Process, etc) and PktMon ETL events.

Insight Client should be able to view traces combining multiple event providers and format them into a text file.
NETMON can do this, as well, but events that require TMF files for metadata won't be formatted properly.


allow overrides in nginx configuration

as the configuration file is set now, it is impossible to http add headers because the hardcoded location directives precede the customization part. I propose to put the custom environment variable $CUSTOM_SERVER_CONFIG before the location sections.

Could you give an example of an override you'd want to do, and where in the resulting nginx config file it would need to be?  You can see the file that gets generated by running:
docker run -rm IMAGE_NAME cat /etc/nginx/mushed.conf

One serious limitation here is that nginx won't let you duplicate a setting in a block; it will quit with an error rather than start. So overrides may not be possible doing what we're doing nginx-boot.sh script that we're using to handle the customizations. But if I know what you're trying to do, we can figure out if this is the way to do it.


Search Lagging and only highlight

Expected Behavior
In the previous version (before 5.2, I believe) the search is fast and only show request that we search.
Actual Behavior
As you can see here, it's only highligh it

Reproduction Steps

Search something
Check the list

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2023.5.5
What operating system are you using?
Windows
Operating System Version
Windows 10 22H2
Installation method
Auto Update
Last Known Working Insomnia version
2023.5.2 Beta

Same issue [Search is not working]
Version: Insomnia 2023.5.5
Build date: 8/19/2023
OS: Darwin arm64 22.5.0
Electron: 25.2.0
Node: 18.15.0
Node ABI: 116
V8: <IP_ADDRESS>-electron.0
Architecture: arm64

Same issue，only highlight.

same here, since version 2023.5.0
my current version:
Version: Insomnia 2023.5.5
Build date: 18. 8. 2023
OS: Windows_NT x64 10.0.19045
Electron: 25.2.0
Node: 18.15.0
Node ABI: 116
V8: <IP_ADDRESS>-electron.0
Architecture: x64

duplicate of https://github.com/Kong/insomnia/issues/6288


Added Anti Money Laundering Check & Report

This adds AML functionality in accordance with the following API Documentation.
Stylistically and structurally, it follows the Identity Check formula, but swaps some key elements to point it to the new report type.
Note About Testing: I tried writing a test for this, but since the Onfido Token provided doesn't have AML Report functionality enabled on the account, I wasn't able to get a positive result. Please advise on what you'd like me to do here.

Nightly build failed. From what I gather, this is a travis issue, not a code one.


Available for GCC or not??

[Enter feedback here]
On this page you say this is available for GCC tenants.  Here, https://docs.microsoft.com/en-us/microsoft-365/admin/usage-analytics/enable-usage-analytics?view=o365-worldwide
you say it is not.  Is Microsoft 365 Usage Analytics available for GCC or not?

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

ID: 65732911-21e9-0902-51fa-0d3ae0caaa7a
Version Independent ID: 3682fff5-36fc-1d69-96ef-88f7f73225d5
Content: Enable Microsoft 365 usage analytics - Microsoft 365 admin
Content Source: microsoft-365/admin/usage-analytics/enable-usage-analytics.md
Service: o365-administration
GitHub Login: @efrene
Microsoft Alias: efrene

@leighannfletcher - Thanks so much for the feedback. Sorry for the confusion.  This article will be updated today to link to the article that describes how to use the Usage Analytics report in a Microsoft 365 GCC tenant.  Viewing Usage Analytics data in a Microsoft 365 GCC tenant is supported, but your need to download a Power BI template that is not available in the Power BI Marketplace (described in the other article). Again, sorry for the confusion, and thank you for noting this to us.


🐛Bug: Listbox elements should get data-highlighted on mouseover

Describe the bug
Without it, things like this can happen which are not pleasant. I'd like to be able to style using a single selector, regardless if it's highlighted via mouse or via keyboard, and have only one element be highlighted at a given time. Basically a single source of truth for where I'm at regardless of what device I'm using (keyboard/mouse).
Hover styles can still be used in addition to this if one would like 😃
https://github.com/melt-ui/melt-ui/assets/64506580/c0c6866c-50b0-4fe2-8957-666cbb955892

I don't think this should be the case, especially since it limits flexibility. You can see that currently, we closely resemble the APG pattern:
https://github.com/melt-ui/melt-ui/assets/26071571/cf532712-958d-4a13-8eee-048443e52517
The focus is always on the trigger. There is a keyboard highlighted item, and there's a separate hovered item.

Okay yeah that's cool for the examples. Anyone can still apply a different hover class and get the same effect, whereas if you don't want that double highlighted effect there's no way around it currently since that source of truth is managed internal to the builder.

We can export the highlightedItem from the builders. That way they can set it on mouse over


Update EKS-D 1.25

Issue #, if available:
Description of changes:

Updated EKS-D 1.25 to second prod release. This change includes a fix to the pause version

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


VIS-2-beta: input val - CSS text size and others gets ignored and line break before HTML append

Describe the bug
It does not apply CSS text size and other CSS settings to the "input val" widget.
To Reproduce
Change the text look via font-size or font-weight, it will not change.
Screenshots & Logfiles
The upper indicator has the same text size and it shows the correct size, but the input fileld ignores these settings.

If I use "HTML anhängen" / "HTML append" then it applies the CSS settings to the appended text, but not to the input text.
Another issue is, that the appanded text gets added after a line break instead into the same line.

Versions:

Adapter version: v2.0.16

It shouldn't.
To apply font size to input element you must define the class
.big-font input[type="text"]{
    font-size: 24px;
}

and then apply it to widget

Result:


feat: playing with basic rag scripts

Progress on https://github.com/ErikBjare/gptme/issues/59

Codecov Report
Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.37%. Comparing base (6d1471e) to head (f5ffa83).

Files with missing lines
Patch %
Lines

gptme/dirs.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
- Coverage   79.82%   78.37%   -1.46%
==========================================
  Files          52       52
  Lines        3142     3144       +2
==========================================
- Hits         2508     2464      -44
- Misses        634      680      +46

Flag
Coverage Δ

anthropic/claude-3-haiku-20240307
?

openai/gpt-4o-mini
78.37% <66.66%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

We have https://github.com/ErikBjare/gptme-rag and the wrapping rag tool now


Fuzzy matching on unknown method error message

Category
Other
Describe the feature you'd like to request
The function ast::Id::to_meth errors if the underlying Id doesn't name a method. It would be useful to fuzzy match the Id against all the possible methods to provide the user with guesses as to what they may have met
Describe alternatives you've considered
Leave error message as is
Additional context
No response
Is this something that you'd be interested in working on?

[X] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

It may be worth considering a more general "add suggestions for typos" to error messages. E.g., the ones mentioned in this comment: https://github.com/cedar-policy/cedar/pull/222#discussion_r1278119832


3 SNS Services are generated using ACK

Description of your changes
Topic, PlatformApplication and PlatformEndpoint resources are generated using ACK. Though support for PlatformEndpoint is poor since ACK recognizes CreatePlatformEndpoint call but not GetEndpointAttributes or DeleteEndpoint calls since they are named different (no Platform as prefix).
Existing SNSTopic in notification.aws.crossplane.io group has been removed in favor of Topic resource in sns.aws.crossplane.io

Fixes https://github.com/crossplane/provider-aws/issues/451
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
I was able to test Topic only since other two depended on having an external service that is not available in AWS (push notifications)

@krishchow thanks for the review! this should be ready for the final review.

As a side note, do you think it's a good idea to change the codecov check for PRs? Odds are most of the generated resources are going to fail that check given that we aren't explicitly adding test cases

@muvaf I did some testing with a Topic and was not seeing it successfully reconcile.  It created the topic, but did not sync.
Here's the output from the topic resource:
apiVersion: sns.aws.crossplane.io/v1alpha1
kind: Topic
metadata:
  annotations:
    crossplane.io/external-name: arn:aws:sns:region:account_number:test-topic1
  finalizers:
  - finalizer.managedresource.crossplane.io
  name: test-topic1
spec:
  forProvider:
    name: test-topic1
    region: region
  providerConfigRef:
    name: default
status:
  atProvider: {}
  conditions:
  - lastTransitionTime: "2021-03-31T13:21:31Z"
    message: 'observe failed: failed to describe Topic: InvalidParameter: 1 validation
      error(s) found.'
    reason: ReconcileError
    status: "False"
    type: Synced

Here's the call to AWS:
2021/03/31 13:21:31 <CreateTopicResponse xmlns="http://sns.amazonaws.com/doc/2010-03-31/">
  <CreateTopicResult>
    <TopicArn>arn:aws:sns:region:account_number:test-topic1</TopicArn>
  </CreateTopicResult>
  <ResponseMetadata>
    <RequestId>RequestId</RequestId>
  </ResponseMetadata>
</CreateTopicResponse>

debug logs from the provider-aws container:
628359d847a1","apiVersion":"sns.aws.crossplane.io/v1alpha1","resourceVersion":"194354"}, "reason": "CannotObserveExternalResource", "message": "failed to describe Topic: InvalidParameter: 1 validation error(s) found."}

@muvaf - with your latest commit I was able to create/edit/delete a sns topic and it late initialized the policy correctly.  LGTM

@muvaf will you rebase and i can test again ?


React: Team page 80% build work

The team page is built and configured by building all the necessary components to finish off the page.
Status of the page is :80% build
Currently, it is only missing the data cards for all members and social links since I'm planning to use the component used in the footer which is currently under review on other PR (build by another participant) 👍
We will finish off the page after the components are available in the code base.
Until then mentors kindly review and merge this as of now!
Preview: (Go to the team)
https://page-setup-preview-mecc8td1q.now.sh/

@rabeehrz there you go! Ignore the box-shadow classes for now on global styles will remove them after the page is done! I need it there for now! (I have removed other shadows not in use)
You may review again!

@rabeehrz there you go! Ignore the box-shadow classes for now on global styles will remove them after the page is done! I need it there for now! (I have removed other shadows not in use)
You may review again!

Awesome! Looks good! 💯

@abhu-A-J Can you please change the style of the cards and transition effects for the team page. @anushbhatia can help you with suggestions or let me know if you need anything

I was planning to chase the UI of cards aftee the master repo has it! That way it will be easier for us to build it on react since we can plan about the components. At this point we are simply following the main website. Will work on it as possible. Thanks

@smaranjitghose Sir The team page is finally completed for the react version. 😅
Kindly review the same !
Link to preview work: https://page-setup-preview.now.sh/
Also from next time can we make PR for components individually rather than one whole page?
since it gets very difficult to  manage one whole page

@smaranjitghose Sir The team page is finally completed for the react version. 😅
Kindly review the same !
Link to preview work: https://page-setup-preview.now.sh/
Also from next time can we make PR for components individually (or maybe multiple components ) rather than one whole page?
since it gets very difficult to  manage one whole page with so many components at a time

@smaranjitghose Sir The team page is finally completed for the react version. 😅
Kindly review the same !
Link to preview work: https://page-setup-preview.now.sh/
Also from next time can we make PR for components individually (or maybe multiple components ) rather than one whole page?
since it gets very difficult to manage one whole page with so many components at a time

Sure, you can give separate PRS for each component


Added schema hook for setAttr #127

@dnalagatla can we update README.md with setAttribute as well?

[ ] as you observed, we should also remove this code and replace it with a call to the setAttribute hook


[TableBody] Add new property: stripedRowsStyles

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

This resolves issue #5286. Description:

One can set stripedRows prop for a  component, and even rows will get different background color. But how can this color be changed? Or not just the color, but even some other style properties.

Demo:
import React from 'react';
import {Table, TableBody, TableHeader, TableHeaderColumn, TableRow, TableRowColumn} from 'material-ui/Table';

const TableExampleSimple = () => (
  <Table>
    <TableHeader>
      <TableRow>
        <TableHeaderColumn>ID</TableHeaderColumn>
        <TableHeaderColumn>Name</TableHeaderColumn>
        <TableHeaderColumn>Status</TableHeaderColumn>
      </TableRow>
    </TableHeader>
    <TableBody stripedRows={true} stripedRowsStyle={{backgroundColor: 'red', color: 'blue'}}>
      <TableRow>
        <TableRowColumn >1</TableRowColumn>
        <TableRowColumn>John Smith</TableRowColumn>
        <TableRowColumn>Employed</TableRowColumn>
      </TableRow>
      <TableRow>
        <TableRowColumn>2</TableRowColumn>
        <TableRowColumn>Randal White</TableRowColumn>
        <TableRowColumn>Unemployed</TableRowColumn>
      </TableRow>
      <TableRow>
        <TableRowColumn>3</TableRowColumn>
        <TableRowColumn>Stephanie Sanders</TableRowColumn>
        <TableRowColumn>Employed</TableRowColumn>
      </TableRow>
      <TableRow>
        <TableRowColumn>4</TableRowColumn>
        <TableRowColumn>Steve Brown</TableRowColumn>
        <TableRowColumn>Employed</TableRowColumn>
      </TableRow>
    </TableBody>
  </Table>
);

export default TableExampleSimple;

The Table component was partially / fully migrated on the next branch.
We are now focusing on releasing out the next branch.
We plan to stop the support of the migrated components.
I'm gonna close this PR. Thanks for your time.
Would be great if you check out the new component. Any PR is welcomed.

Ok, I'll have a look at the new component. Thanks.


bump jupyterhub chart

https://github.com/jupyterhub/zero-to-jupyterhub/compare/7174419...5ce02a2
This gets the values-passthrough changes in jupyterhub/zero-to-jupyterhub#941, which removes the workaround for using extraConfigMap, instead relying on the passthrough of .Values.custom.

Failure is due to https://github.com/jupyterhub/zero-to-jupyterhub-k8s/pull/970


Production Deployment Size

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
Current Production build is 1.6M prior to gzip.
Expected behavior
Angular 2 has really raised the stakes in file size, although there are techniques that the build could take advantage of (ngc and tree shaking) to further reduce the size.
Minimal reproduction of the problem with instructions
$ npm run build.prod
$ du -d 1 -h dist/prod
1.6M    dist/prod
1.1M    dist/prod/js
528K    dist/prod/assets
4.0K    dist/prod/css

What is the motivation / use case for changing the behavior?
Advanced seed project should encapsulate the best ideas of building an Angular2 project.
Please tell us about your environment:

OSX, N/A, NPM, None

Angular Seed Version: 8c892c4
Node: node --version = 6.9.2

The Angular ICO is 361K. I don't usually deal with ICO files that are larger than 32x32, but the comparable PNG file can be as small as 21K.

My apologies, I should of read a little further.
$ npm run build.prod.aot
$ du -d 1 -h dist/prod
1.4M    dist/prod
888K    dist/prod/js
528K    dist/prod/assets
4.0K    dist/prod/css

Any opportunity to squeeze more out, or is this the minimal size we can achieve with Angular 2 these days? (Not withstanding the assets folder of course)

@oravecz Can always squeeze more ;)  The rollup build will up although there's some more challenges to get rollup working here. It's not documented in the readme here however the task exists documented here in the parent:
https://github.com/mgechev/angular-seed#tree-shaking-with-rollup


in_http: Add support for Application/x-ndjson

Which issue(s) this PR fixes:
Fixes #3511
What this PR does / why we need it:
This enable in_http plugin to recognize "ndjson" format, which is
basically a list of JSON objects separated by "\n".
Here is a typical ndjson message looks like:
{"foo": "bar"}
{"buz": "hoge"}

See http://ndjson.org/ for more details.
Docs Changes:
Required. I'm going to submit a documentation patch  soonish.
Release Note:
in_http: Add support for "application/x-ndjson"

Thanks!


Test - Create Invitation

Test Description:
Test if the user can create an invitation

Test case successfully implemented for create invitation service


fixed #83

Thanks for your contribution, I would just like some feedback from native speakers!

I can help, but I didn't understand what is the matter on this thread :)

This pull request tries to fix #83.
This commit makes
"sexta que vem"
"sexta feira da semana que vem"
"sexta-feira da próxima semana"

equal to 2013/2/22 when the current date is Tuesday Feb 12, 2013


[Event Request] Table 37 - "Sales Line" HandleOnBeforeValidatePlannedShipmentDate

Dear Microsoft,
For field "Sales Line"."Planned Shipment Date", we would like to have the ability to skip the default validation. Therefore we like to see the following publisher added.
field(5795; "Planned Shipment Date"; Date)
{
    AccessByPermission = TableData "Order Promising Line" = R;
    Caption = 'Planned Shipment Date';
    trigger OnValidate()
    var
        IsHandled : boolean; //New local variable
    begin
//Start new publisher
         HandleOnBeforeValidatePlannedShipmentDate(Rec;IsHandled);
         if IsHandled then
             exit;
//End new publisher
        TestStatusOpen();
        if "Planned Shipment Date" <> 0D then begin
            PlannedShipmentDateCalculated := true;
            Validate("Shipment Date", CalcShipmentDate);
        end;
    end;
}

New Publisher
[IntegrationEvent(false,false)]
local procedure HandleOnBeforeValidatePlannedShipmentDate(var SalesLine : record "Sales Line";var IsHandled : boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Small improvement to readability of the middleware section of docs

The following is my personal feedback, as a newcomer to redux, reading the docs for the first time.
While going through the docs, I finally reached the section about middleware.
Since the middleware section is probably the 'hardest' to grasp, as someone who is completely new to redux, I would gladly take any simplifications that it could give me.
One such 'simplification', that I think could help, is having the next(action) named something more descriptive, e.g. providedDispatcher(action) or something along those lines.
I am sure there must be a reason as to why it is named next(), that at this time goes over my head, but truth is I always had to keep a 'manual' mental reminder for what next() actually represents, every time I encountered it in the doc, in order to help me keep track of what I was reading.

“Next” means dispatch of the next middleware in chain. Maybe we can call it nextDispatch?
It’s a convention from server side frameworks that would yield control to the next middleware by calling next argument.

@gaearon Coming from a server-side background, I would vote for keeping the API unchanged. Consistency is important, it's one of the compelling reasons in favor of JS everywhere.
Your explanation above is clear and should be added to the docs for those not familiar with the middleware concept.

I find nextDispatch to be an improvement over next as far as clarity-at-first-glance is concerned.
In the end of the day, try to see it from the perspective of someone who, yours trully, is oblivious of that naming convention from server side frameworks. Descriptive names for such things are probably going to help keep with the flow. That's just my opinion.

@mkarajohn It's not always nextDispatch, it's either nextMiddleware or dispatch. This could be clarified in the docs.

Your explanation above is clear and should be added to the docs for those not familiar with the middleware concept.

Happy to accept a PR doing this. Since leaving an issue hanging here is not very productive, I’ll close it, but I’m happy to continue the discussion in a PR that adds an appropriate note.
Thanks!


[RESOLVED] IDRAC_83 has been restarted (uptime < 10m)

Problem started at 19:44:58 on 2023.05.26
Problem name: IDRAC_83 has been restarted (uptime < 10m)
Host: IDRAC_83
Severity: Warning
Operational data: 00:03:33
Original problem ID: 22585
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23186&eventid=22585

Problem has been resolved in 6m 29s at 19:51:27 on 2023.05.26
Problem name: IDRAC_83 has been restarted (uptime < 10m)
Host: IDRAC_83
Severity: Warning
Original problem ID: 22585


Scroll CupertinoContextMenu if items overflow

Fixes https://github.com/flutter/flutter/issues/55025
Native Behavior:
https://github.com/user-attachments/assets/4ebed3e3-ab4f-4e94-8b47-e9c957f245ef
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Blocked on https://github.com/flutter/flutter/issues/154040 to correctly set the margins of the scrollbar.

Blocked on #154040 to correctly set the margins of the scrollbar.

Talking with @Piinks offline, it seems like the native margin is different in this case because of the border radius of the context menu. So it makes sense to expose mainAxisMargin in this PR so it can be customized for this use case.


Should the Nav Items use RouterLinks in Flow?

The current implementation has regular anchors as part of the template of the nav-item element, but would it make sense to use RouterLinks provided by Flow?
Maybe that would automagically solve the following issues:

https://github.com/vaadin/vcf-nav/issues/9
https://github.com/vaadin/vcf-nav/issues/10
https://github.com/vaadin/vcf-nav/issues/13

Created a web component issue based on this issue.


"Read more" cookie footer buggy and not compliant with law

If you open the page "https://dbeaver.io" and, inside the cookie notification, click on the "Read more", this just openes the same page again in a new tab.
Also there is no option to change the settings ("opt-out"), even if it's mentioned in the label.
This is not compliant to the law to my knowledge and is a bug which should be fixed asap.

Thank you for the bug report.

Fixed


[20.09] element: 1.7.24 -> 1.7.25

Motivation for this change
Backport PR for #119233. For some reason element-desktop is breaking:
Keytar isn't installed; secure key storage is disabled.
Seshat isn't installed, event indexing is disabled.
/home/ma27/.config/Element exists: yes
/home/ma27/.config/Riot exists: no
No update_base_url is defined: auto update is disabled
(node:6375) UnhandledPromiseRejectionWarning: TypeError: mainWindow.webContents.session.setSpellCheckerEnabled is not a function
    at App.<anonymous> (/nix/store/ydd7z7rjvbjzw5y4qnj2waxrvmbvdbbc-element-desktop-1.7.25/share/element/electron/src/electron-main.js:947:36)
(node:6375) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:6375) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

We probably need electron 12 to fix this, but I didn't push this for now since I don't know and don't use element-desktop.
Things done

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

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

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

Yeah, have you tried, actually running element-desktop @drewrisinger ? Because that's the problem I have right now :)

Yeah, have you tried, actually running element-desktop @drewrisinger ? Because that's the problem I have right now :)

Nope. When I did, I got the same error you did. I'm not familiar enough with electron/element-desktop to contribute anything, sorry.

This needs Electron 12. Backport:
https://github.com/NixOS/nixpkgs/pull/117621
https://github.com/NixOS/nixpkgs/pull/119578