[BUG] Rate Limiting settings in Chatflow Configuration not persisting

Describe the bug
The Rate Limiting settings in Chatflow Configuration are not being saved correctly. After configuring and saving the Rate Limiting settings, switching to another tab and then returning to the Rate Limiting tab shows that the settings have not been retained.
To Reproduce
Steps to reproduce the behavior:

Go to 'Chatflow Configuration'
Click on 'Rate Limiting' tab
Enable Rate Limiting and configure settings
Click 'Save' button
Switch to another tab in Chatflow Configuration
Return to the 'Rate Limiting' tab
Observe that the settings are not retained and Rate Limiting is shown as Disabled

Expected behavior
After saving the Rate Limiting settings and returning to the tab, the previously configured settings should be displayed and Rate Limiting should remain Enabled.
Screenshots
https://github.com/user-attachments/assets/ae847101-dfde-4107-8a91-0b84e5f7f123
Setup

Installation: Docker
Flowise Version: 2.1.2
OS: Ubuntu 22.04
Browser: Chrome

thanks! we have a PR fixed here: https://github.com/FlowiseAI/Flowise/pull/3311
will be pushing to new release!

Thank you for your response. However, I noticed that PR #3311 (merged on Oct 5) is already included in Flowise 2.1.2 (released on Oct 10), which is the version where I'm experiencing this issue.
Could you help verify if the fix in PR #3311 fully addresses the Rate Limiting persistence problem?
I'm still observing the following behavior in version 2.1.2:

Rate Limiting settings are not persisting after:

Configuring and saving the settings
Switching to another configuration tab
Returning to the Rate Limiting tab

Reproduction environment:

Flowise Version: 2.1.2 (Docker)
OS: Ubuntu 22.04
Browser: Chrome

Timeline reference:

PR #3311 was merged on October 5, 2023
Flowise v2.1.2 was released on October 10, 2023

Let me know if you need any additional information to help investigate this issue.
Thank you for your time and assistance.

Thank you for your response. However, I noticed that PR #3311 (merged on Oct 5) is already included in Flowise 2.1.2 (released on Oct 10), which is the version where I'm experiencing this issue.
Could you help verify if the fix in PR #3311 fully addresses the Rate Limiting persistence problem?
I'm still observing the following behavior in version 2.1.2:

Rate Limiting settings are not persisting after:

Configuring and saving the settings
Switching to another configuration tab
Returning to the Rate Limiting tab

Reproduction environment:

Flowise Version: 2.1.2 (Docker)
OS: Ubuntu 22.04
Browser: Chrome

Timeline reference:

PR #3311 was merged on October 5, 2023
Flowise v2.1.2 was released on October 10, 2023

Let me know if you need any additional information to help investigate this issue.
Thank you for your time and assistance.


Enum value 'index' will result in error

When using an enum with a value 'index':
enum Stuff {
  index
}

This error pops up on generation:
[SEVERE] json_serializable:json_serializable on lib/graphql/graphql_gen.graphql.dart:

Bad state: Too many elements

Every other value seems to work.

This seem like a problem with your serialiser code generator rather than the client library.


Fixes to formatting in IOCTL documentation

I have made formatting edits, mostly changing from HTML to Markdown to make future edits easier.  Also, I fixed sample code blocks, including ones that had issues with spacing, which made the code ambiguous and confusing.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: AKhristov-SE sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

This pull request is quickly falling behind, I'm seeing more merge conflicts I'll have to resolve. Are there any updates?

Hi @AKhristov-SE, thanks for the extensive work on these files. Can you resolve the merge conflicts (which might be due to a build bug fixes for the IOCTL file types) and resubmit? Thanks.

@AKhristov-SE : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@AKhristov-SE : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

Do not merge this just yet...
Oh, too late.

Hi Karl,
Thanks for looking at this.  I will be submitting another pull request, as I was in the middle of integrating my changes when this got merged.
-Alex

ok, i reverted that pr as soon as i saw your message

Thanks. I'll have this done within the hour.

Hi @Karl-Bridge-Microsoft,
I went ahead and submitted another PR, and I'll do a second one a bit later. Sorry for the confusion.  I was just worried back when I submitted this request that I'd have to redo it, and that's kind of what I ended up having to do, so it was a bit frustrating.
-Alex


Fix: Pull up createAuthenticationMock()

This PR

[x] pulls up createAuthenticationMock()

Follows #957.
Related to #925.
Related to #959.
Related to #962.
Related to #964.
💁♂ Using it in all of the referenced PRs. While we could wait until any one of these gets merged, it's probably ok to pull this method up already (as I assume the open PRs will get merged eventually).

Closing in favour of #986.


policy for multiple components in maxp

TODO

[ ] add tests
[ ] finish docs

@sjsrey Is this ready to merge or are you still working on it?

@sjsrey Is this ready to merge or are you still working on it?

Good to go.


fix: Fail early if IMDS returns an error.

This catches errors earlier on OCI for example, where the PUT request
returns 405.

FYI @dsmeytis


Update test to 2.5.3

etd-robots and hydra_etd are now on 2.5.3

Coverage remained the same at ?% when pulling 88b86417dda73312f1602050ba64e905d83385c0 on jcoyne-patch-2 into 3212aed3c6139297d936a5a3aabb4069360942fb on master.


Support for permanent Service_Address objects

Is your feature request related to a problem? Please describe.
When deploying forwarding virtual servers, a Service_Address object for some prefix (example <IP_ADDRESS>/0) must be created separately, so that ARP and ICMP-echo can be disabled for that object. When such an object will be used by multiple tenants, the Service_Address object is first created in /Common/Shared declaration, and can then be referenced from the other tenants' Service declarations using pointers.
Problem is that in this scenario, when the last tenant Service (vs) referencing to that Service_Address is deleted, also the Service_Address in /Common/Shared is deleted.
This causes subsequent posts of new tenants' declarations referring to this Service_Address object to fail, because the object is no longer in /Common/Shared.
Example declarations:
In tenant /Common:
"Common": {
"class": "Tenant",
"Shared": {
"class": "Application",
"template": "shared",
"vaAny_ipv4": {
"class": "Service_Address",
"arpEnabled": false,
"icmpEcho": "disable",
"routeAdvertisement": "disable",
"virtualAddress": "<IP_ADDRESS>/0"
},
...
Then in a tenant:
"APN01": {
"class": "Tenant",
"defaultRouteDomain": 0,
"GiLAN": {
"class": "Application",
"template": "generic",
"vsANY_out": {
"class": "Service_L4",
"label": "Default VS for any",
"layer4": "any",
"translateServerAddress": false,
"translateServerPort": false,
"virtualAddresses": [
{"bigip": "/Common/Shared/vaAny_ipv4"}
],
"shareAddresses": true,
"allowVlans": [
{ "bigip":"/Common/VLANNAME" }
],
"virtualPort": 0,
"profileL4": { "bigip": "/Common/Shared/fastL4any" },
"snat": "none",
"policyNAT": {"use": "cgnat_policy"},
"persistenceMethods": []
},
Later, when the APN01 tenant is deleted for whatever reason, the Service_Address /Common/Shared/vaAny_ipv4 is also deleted as side-effect, even if declared in different tenant and shareAddresses is true in the Services (tested on 3.20).
After this, when either the deleted tenant is recreated using same declaration, or a new tenant also using the vaAny_ipv4 is posted, the declaration fails due to the object missing from /Common/Shared.
Workarounds exists, like using dummy vs with a Virtual_Address containing that Service_Address, or instructing ops team to always re-apply the /Common declaration before posting new tenants, but neither are very elegant.
Describe the solution you'd like
I would like to have an option to mark  Service_Address objects as permanent, ie they would not be deleted even if the last virtual server referencing that Service_Address is deleted. I think it would be useful especially when using AS3 to create forwarding virtual servers for different tenants, who very often need to have destination any (either IPv4 or IPv6) and having ARP and ICMP echo disabled for correct function.
This feature would improve the idempotency for AS3 when you split the configuration over multiple AS3-files (one for Common, one for the tenant(s)), so that there is no need to re-apply the Common declaration if you have deleted the last non-common tenants from the device.
Describe alternatives you've considered
Using a dummy Service that "holds" the Service_Address in /Common to prevent it from being deleted.
Also considered to instruct operators to always first re-apply existing /Common declaration before posting new tenants.
Additional context

We have AUTOTOOL-1657 in our internal product backlog to look into disabling this auto-delete behavior for virtual-address objects.


npm version is missing btn-success class

I don't know how this happened but the npm module despite having the same version number (0.4.7) is different than version on github.
.btn.btn-success {
  background: #32b643;
  border-color: #2faa3f;
  color: #fff;

Please disregard. Was looking at wrong version (0.4.6).

The difference in npm is only in dist folder- all files in dist are still from (0.4.6).

@jakubrpawlowski Checking.

@jakubrpawlowski my fault. it seems dist is still old version.

@jakubrpawlowski Fixed and published.


Suggestion: Also flag Matlab warnings with CI

While there is a GitHub Action to check for errors in code, it is not currently checking for warnings, which could be useful. The only caveat is that if Matlab produces a lot of unhelpful warnings, failing the CI for every single one is probably not that useful. I haven't actually used it recently enough to know whether this is likely to be a problem or not though.
In order to check for warnings too, you'll need the following change:
diff --git a/buildfile.m b/buildfile.m
index edab52b..dce3cba 100644
--- a/buildfile.m
+++ b/buildfile.m
@@ -3,7 +3,7 @@ import matlab.buildtool.tasks.CodeIssuesTask
 plan = buildplan(localfunctions);

 task = "check";
-plan(task) = CodeIssuesTask;
+plan(task) = CodeIssuesTask(WarningThreshold=0);
 plan.DefaultTasks = task;

 end

I think such a change can improve user experience if implemented along with configuring the Code Analyzer.
-plan(task) = CodeIssuesTask;
+plan(task) = CodeIssuesTask(WarningThreshold=0, Configuration='codeAnalyzerConfiguration.json');

The manually configured Code Analyzer could prevent unhelpful or unwanted warnings during CI checks and regular work on the project.

That's a good idea. I'm not that familiar with Matlab's warnings these days, but it's good to know you can suppress unhelpful ones.
I suppose for implementing this we'd probably want to leave most warnings enabled to start with and let users disable the ones they don't like (ideally we should tell them how to do this in the documentation).


Swift3

Updated dependencies, framework, tests and example project

this is looking great, could you set correct Xcode version in travis.yml file so it passes CI?

Thank you!


fix: randomize cloud storage bucket name

Randomize the cloud storage bucket name for sample vertex_ai/index.

Error in logs is unrelated to this sample


katalyst-agent: get reclaimed resource.katalyst.kubewharf.io/reclaimed_memory capacity failed: resource memory last report value not found

What happened?

What did you expect to happen?
GetReportContentResponse from headroom-reporter-plugin Endpoint success.
How can we reproduce it (as minimally and precisely as possible)?
Use the image of the latest main branch
Software version

$ <software> version
# paste output here

@zhy76 Does this error appear within two minutes after starting? cc @luomingmeng

@zhy76 Does this error appear within two minutes after starting? cc @luomingmeng

No, This error persists in the katalyst-agent log after startup

It seems that the version of v0.3.1 does not have this error, but the latest version of the main branch does.

This issue is due to malachite not being deployed, which can be fixed by helm install malachite -n malachite-system --create-namespace kubewharf/malachite, I will close this.


pull request for e2e test at 12/30/2016 00:26:13

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


Update sbt-scalajs, scalajs-library_2.13, ... to 1.10.1

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-library_2.13
org.scala-js:scalajs-test-bridge_2.13

from 1.10.0 to 1.10.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.10.0).
You might want to review and update them manually.
modules/ws-client/yarn.lock

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

@Mergifyio rebase


Migrate from TSLint to ESLint

TSLint is being deprecated in favor of ESLint. We should investigate the migration process and what rules we want to enable.
https://github.com/palantir/tslint/issues/4534
https://medium.com/palantir/tslint-in-2019-1a144c2317a9

https://github.com/typescript-eslint/typescript-eslint


Update player docs URL to use HTTPS

Coverage decreased (-0.02%) to 76.21% when pulling f2586af057794e230c4c2191f08f2c879c735dc4 on davidjb:patch-2 into 10266c584aad3bca2fff4efa5e85eaf92cc25973 on clappr:master.


Mac support

Based on
https://docs.nwjs.io/en/latest/For Developers/Contributing to NW.js/
https://github.com/nwjs/nw.js/blob/3e895da916088e9858484b4690ce982dc3aeba32/src/api/tray/tray_mac.mm
https://github.com/AnaghSharma/Ambar-Xamarin
What would you like to be added:
Why is this needed:
For which Platform:

[ ] All
[ ] WPF
[ ] WinUI
[x] Uno
[x] Core

Anything else we need to know?

Explore https://github.com/davidortinau/WeatherTwentyOne


error 'encaps tag. MIssing annotation'

I have a react mutation I'm using, and under the getConfigs method, I am getting the error 'encaps tag. Missing annotation'. As far as I can tell, I've set up the method exactly as the Mutation docs have directed. I'm not even sure what this error message means.
Here is my code
getConfigs () {
    return [ {
      type: 'REQUIRED_CHILDREN',
      children: [
        Relay.QL`
          fragment on RegisterTenantPayload {
            tenant {
              url
            }
          }
        `
      ]
   } ]
 }

Here is the error

A reference to the docs
https://facebook.github.io/relay/docs/guides-mutations.html - under REQUIRED_CHILDREN
Any idea what this error is being cause by?

I had the same problem. Looks like flow does not like tagged templates inside of an array.
This here works for example:
const childQuery:any = Relay.QL`
  fragment on RegisterTenantPayload {
    tenant {
      url
    }
  }
`;

getConfigs () {
    return [ {
      type: 'REQUIRED_CHILDREN',
      children: [
        childQuery
      ]
   } ]
 }

I also got the same problem... you can also solve it by adding a return type to getConfigs like so:
  getConfigs (): any {
    return [{
      type: 'REQUIRED_CHILDREN',
      children: [
        Relay.QL`
          fragment on MyPayload {
            name
          }
        `
      ],
    }];
  }

Closing since this issue hasn’t seen a lot of activity. Sorry about that 😣
Please reopen with a reproduction on https://flow.org/try if you are still having issues!


Update ScreensShots.md

I have make use of the latest Flutter UI practices and improve the structure and styling of this code
In the upgraded code:

The code structure follows the standard Flutter app structure with a main entry point and a MyApp widget as the root widget.
The MyApp widget is a StatelessWidget that sets up the basic app scaffold with an app bar and a body.
The UI components are displayed in a Column widget inside the body of the Scaffold.
Each component is represented by a ComponentCard widget, which is a custom widget created to display an image and a title.
The image path and title for each component are passed as parameters to the ComponentCard widget.
The ComponentCard widget uses a Card to display the image and title, with appropriate padding and styling.

Make sure to update the image paths in the code to match the actual paths of this images. Also, consider adjusting the image dimensions to fit the specific requirements..
Pull Request Template
Description:
Briefly describe the purpose and context of this pull request.
Changes Made:
Provide a clear and concise overview of the specific changes made in this PR.
Related Issue:
If applicable, reference the related issue number(s) here.
Screenshots/GIFs:
If applicable, include any relevant screenshots or GIFs to visually demonstrate the changes.
Testing Done:
Outline the testing steps you have taken to ensure the changes function as intended.
Additional Notes:
Add any additional information or notes that are relevant to this PR.
Checklist:

[ ] The code follows the project's coding style guidelines.
[ ] Unit tests have been added or updated to cover the changes.
[ ] Documentation, if applicable, has been updated to reflect the changes.
[ ] All new and existing tests pass successfully.

Please review and merge this PR at your earliest convenience.
Thank you!

Must perview my code .

preview my code


CDS frame in output

Hi,
thanks for developing Finder. I was able to run it without problems using Singularity.
I got a question about the output files, FINDER_BRAKER_PROT.gtf in particular.
The result of my run doesn't report the CDS frame (8-th column of the gtf file) in the FINDER_BRAKER_PROT.gtf output.
Is this a bug in my run or is this intentional?
Best, Lars

Hello Sagnik,
thank you for the prompt reply.
Best,
Lars


Add Christian Hauf to list of AUTHORIZED_CONTRIBUTORS.

This PR is to add Christian Hauf to list of AUTHORIZED_CONTRIBUTORS. We have a signed docusign CLA from them now.

@cb1kenobi @caspahouzer Review and merge please so we can re-pull in the PR that Hans reversed.


Document conda install with conda-forge

📚 Documentation
installing from conda is a nice way to get around some of the wheel issues on vispy (https://github.com/napari/napari/issues/1340) and psutil (https://github.com/napari/napari/issues/1474)
but we don't have it documented... and someone with a fresh conda environment might also not have conda-forge in their channels (https://github.com/napari/napari/issues/1474#issuecomment-660332049).  This should be documented in the readme and at napari.org

fwiw I prefer the form:
conda install -c conda-forge napari

than advising users to change their config.

I think we can probably close this? Installing from conda is included in:
https://napari.org/tutorials/fundamentals/installation

I think we can probably close this? Installing from conda is included in:
https://napari.org/tutorials/fundamentals/installation
Yes looks like it


Improve toasts, error handling; Accept localeIdAdapter

Catch errors and handle errors/success with toast
Along with some changes to the promise chain of importing in the Smartling adapter plugin (related PR), this update catches errors in the promises made with importing translations and creating a new task and returns them as toasts, and also awaits all the promises until everything is fully complete, showing the success toast only when the final success is returned.
Success toast:

Error toast:

Disable button during import and task creation
While the import is in progress, we want this visually reflected on the button so the user knows that it's working in the background and doesn't try to click again.
importFile and handleImport need to be async employ await so that setIsBusy(false) isn't executed until the task is complete.
While loading (setIsBusy === true), the button is disabled and the content changes to Loading... with no icon, because the button doesn't have a lot of room.
Also updates the button to be disabled when there's no progress at all or if it's 0.
While new task job is running, disables "Create job" button.
Accept localeIdAdapter for custom ID transforms
Locales may have different IDs in the translation vendor and in Sanity. This adds an option to the plugin that accepts a function localeIdAdapter, which will be passed the translation vendor locale id and is expected to return the Sanity locale id. The translation vendor id will be used for getTranslation, but then the Sanity id is used for context.importTranslation.
Using the localeIdAdapter here removes the need to implement it across multiple adapters and allows default adapter configs to be used when the only required change is the locale ids.
Example implementation with sanity-plugin-studio-smartling adapter:
// deskStructure.js
import { defaultFieldLevelConfig } from 'sanity-plugin-studio-smartling'
import { TranslationsTab } from 'sanity-translations-tab'

const locales = {}
supportedLanguages.forEach(({ smartlingId, id }) => {
  locales[smartlingId || id] = id
})

export const getDefaultDocumentNode = (props) => {
  return S.document().views([
    S.view.form(),
    S.view.component(TranslationsTab).title('Smartling').options({
     ...defaultFieldLevelConfig,
     localeIdAdapter: (smartlingId) => locales[smartlingId]
    })
  ])
}

Provide task refresh button
After successful job creation, reset form and get refreshed task from translation provider. Because, in my testing experience, the newly added locale isn't always available immediately upon job creation, provide a "Refresh status" button so the user can grab it fresh themselves, without reloading page.

All of this looks good! I'm going to version bump and release on the main branch. (Semver says minor version bump (e.g., 1.3.0) for new functionality!)


SAA-618: attendance stats list

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.02 :tada:

Comparison is base (91047f3) 84.78% compared to head (b29631e) 84.81%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #411      +/-   ##
============================================
+ Coverage     84.78%   84.81%   +0.02%
- Complexity     2033     2036       +3
============================================
  Files           278      278
  Lines          7750     7762      +12
  Branches        438      438
============================================
+ Hits           6571     6583      +12
  Misses          933      933
  Partials        246      246

Impacted Files
Coverage Δ

...ppsactivitiesmanagementapi/entity/AllAttendance.kt
96.42% <100.00%> (+0.97%)
:arrow_up:

...mppsactivitiesmanagementapi/model/AllAttendance.kt
96.00% <100.00%> (+1.26%)
:arrow_up:

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


Feature Request - File Title

It'd be great to have an optional file title horizontally aligned with the buttons at the top of the screen.
Love the product, thanks!

This is done by #257.

@molnarmark How does the deploy cycle of carbon work? I saw that the update for Safari has been pushed to master but the website has not been updated. Do you have some sort of milestone system or anything in particular?

I've made some contributons that are not yet live as of now aswell, i think they collect quite a few updates before pushing to production.
@briandennis will be able to answer you clearly though.

Hey @molnarmark and @SirWindfield. We are having a slower release cycle this time since we have a lot of new features going in. We don't really have a standard practice, and usually will redeploy on every PR merged


Added PageController

Added PageController to control CupertinoOnboardingPage pages and updated very_good_analysis dependency.

This is a great addition to this package. I would love to see this on the next version @ivirtex.

No problem, just doing my part to make things awesome

This is a great addition to this package. I would love to see this on the next version @ivirtex.

Sure, this is definitely coming in the next version. I'm just waiting for @vijaykarthiktk to apply suggested changes ;)

Thanks guys, this package looks great!

Added in the newest version :)


My code using spidev crashes with free(): invalid pointer

Hi,
My project, which consists of an infinite loop that reads from the SPI interface and performs calculations on the data, crashes after several days of running.  I've looked extensively for a way to troubleshoot the problem, but I've hit a dead end, hence why I'm opening this issue.
To be clear - the crash is not a Python traceback. It appears to be a C++ error, and it simply says:
free(): invalid pointer
Aborted

Other instances of this issue that I've seen online usually include a memory address - the error I'm getting does not. The above output is the only text sent to my terminal before my program stops running.
My project is here if you want to inspect how I'm using the spidev library.

Since this isn't a Python issue, I am unable to catch the problem with Python's error handling features.  For anyone reading this in the future, a workaround is to configure your application as a service so it can be restarted automatically after a crash:
https://medium.com/@benmorel/creating-a-linux-service-with-systemd-611b5c8b91d6

I see a python3[...]: free(): invalid pointer message too when my service restarts every 60hrs or so. Running a service which restarts, so no big issue but nice if a fix is available.

It is a security feature in most c stdlibs to abort execution immediately if free is called on an invalid address. If you have the time, you can compile the python extension with ASAN, this should give a detailed crash report.

Why are you so certain that spidev is the root cause of all this? I can see that you are using some other libraries as well. This influx thing stood out particularly to me. As you are only using xfer2, I took a short look at its implementation and could not find any obvious bug leading to the described behaviour. So if you really want to trace this bug down, you should compile influx with ASAN as well. I think spidev is not guilty :) If this is too much work / you are inexperienced in this, it might be sufficient to run python with valgrind, e.g. PYTHONMALLOC=malloc valgrind python ./yourscript.py.

Thanks KoWu. I have not heard of ASAN but I would be interested in recompiling the libraries to get some more info. I'll probably try with valgrind first (which I'm not familiar with either, but Google is quite awesome in that regard).
I suspected spidev first because I have had doubts of the performance of my hardware (Raspberry Pi 3b+), and out of the few libraries I'm using, spidev is doing the most work. My project reads from the SPI interface about 12 thousand times per second.    Also, I did not suspect the influxdb library as the cause because it implements the requests and socket libraries which are much, much bigger projects and have been field tested on a much larger scale than spidev.  I think my suspicion is reasonably placed, but I am committed to trying to pinpoint the problem wherever it lies since I opened this issue.
I didn't know how to continue troubleshooting without a Python traceback, so thank you for providing me with some ideas on how to dig deeper. I'll report back with some findings after exploring the valgrind option.

Confirmed that this issue exists, either in pure spidev or something that it relies upon, using the following code running in Python3 against the shipped spidev 3.4.
I've also had multiple user reports of a project that relies on spidev periodically crashing (plus prior encountered with the same issue that I thought were fixed in 3.4):
import spidev
import time

bus = spidev.SpiDev(0, 0)

bus.mode = 0
bus.lsbfirst = False
bus.max_speed_hz = 80 * 1000000

bytes_total = 0
transfers_total = 0
last_update = time.time()

while True:
    bus.xfer([0b10101010] * 4096)
    bytes_total += 4096
    transfers_total += 1
    if time.time() - last_update >= 5.0:
        print(time.time(), bytes_total, transfers_total)
        last_update = time.time()

After 1.04 million writes, it fails with free(): invalid pointer. This is approximately 21 minutes of continuous testing time.
I'm confused, because I rigorously tested spidev 3.4 due to prior encountered with this issue. In this case however, my isolated tests encountered a Segmentation Fault in spidev.
See: https://github.com/pimoroni/mopidy-pidi/issues/3
And: https://github.com/pimoroni/st7735-python/pull/7 (which mentions testing to 1.8 million display updates, which is considerably in excess of the conditions above)
Either my prior testing was insufficient (the scant mention of 1.8 million cycles seems to suggest otherwise) or something has broken/regressed in SPI elsewhere in the Raspberry Pi kernel-space that is causing this problem.

I'm not too hot on this level of C/C++ debugging, but I did run valgrind against my minimal example script and ran into this burst of errors before the big finale:
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864A48: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B10: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B1C: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Use of uninitialised value of size 4
==20337==    at 0x4864B24: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B28: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B44: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B58: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B70: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864B84: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864BB0: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864BDC: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864BE4: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864BFC: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864C14: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864C28: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864C44: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
==20337== Conditional jump or move depends on uninitialised value(s)
==20337==    at 0x4864C54: ??? (in /usr/lib/arm-linux-gnueabihf/libarmmem-v7l.so)
==20337==
10561.25 - 4,260,139,008b 1,040,073
10621.26 - 4,284,350,464b 1,045,984
==20337== Invalid free() / delete / delete[] / realloc()
==20337==    at 0x4848D14: free (vg_replace_malloc.c:538)
==20337==    by 0x1B4F3F: ??? (in /home/pi/spitest/venv/bin/python3)
==20337==  Address 0x435174 is in the BSS segment of /home/pi/spitest/venv/bin/python3

The notable excerpt being:
10621.26 - 4,284,350,464b 1,045,984
==20337== Invalid free() / delete / delete[] / realloc()
==20337==    at 0x4848D14: free (vg_replace_malloc.c:538)
==20337==    by 0x1B4F3F: ??? (in /home/pi/spitest/venv/bin/python3)
==20337==  Address 0x435174 is in the BSS segment of /home/pi/spitest/venv/bin/python3
==20337==

Interestingly (and again this might be coincidence) this appears to be right at point at which the number of bytes transmitted advances from < 2^32 to >= 2^32.
This output is not, however, sufficient for me to reach any conclusion as to where it might be occuring. But since I have a Pi 4 this points to a potential memory leak, perhaps?

Now I'm reasonably sure we're leaking obscene amounts of RAM, I have whittled the code down and eyeballed it to see where that might be happening. malloc and free calls seem to match up, but I've noticed some issues around Python reference counting. Specifically here:
https://github.com/doceme/py-spidev/blob/f543ca1a8cdc982e6fdfa5402ca9316f8f84bc7a/spidev_module.c#L588-L591
As near as I can tell, this code is missing a call to Py_DECREF(val) after assigning the sequence item to the list.
I'm now running tests against this change and will prep a PR.

Thank you for the PR @Gadgetoid! I compiled this library from master using this commit and loaded it into my Python environment for testing. My application would typically crash after about 2 days, so I'll keep an eye on it and report back in several days.

Thanks @David00 for reporting this and thanks @Gadgetoid for diving in with the analysis and fix. I really appreciate it.

No worries! This manifested as one of our plugins crashing Mopidy for a number of users, so it shot up my priority list.
Doing some more sleuthing and raising a few more PRs that we can hopefully wrap up into a nice hardened release.

Hi, I just wanted to follow up and say that @Gadgetoid's commit seems to have resolved the crashing problem. My application has been running steady for over 4 days now without encountering the issue!
@doceme, are you able to release a new version with this fix on PyPi?

@David00, thanks for the feedback. Version 3.5 has been released on PyPI.

Version 3.5 seems to have solved my problem too! Running for 7 days now, before crashed every 3 days.
Thanks!


Suggestion to provide reach and valuable dual write filters examples in F&O and CE

Hi, can you please verify, correct and amend the following examples then add it to this page if you find it reasonable and helpful for the page visitors:

FO side filters:

A.	Equal:
Single Enum filter: (FieldName == EnumType::Test)
Multiple Enum filters: ((FieldName == EnumType::Test) && (FieldName != EnumType::Test))
Single String filter: (FieldName like "Test")
Multiple String filters: ((FieldName like "Test") && (!(FieldName like "Test")))
B.	Not equal:
Enum filter: (FieldName != EnumType::None)
String filter: (!(FieldName like "Test"))

CE side filters:

A.	Equal:
Single Enum filter: FieldName == ‘Test’
Multiple Enum filters: FieldName == ‘Test’ and FieldName == ‘Test’
Single String filter: FieldName eq ‘Test’
Multiple String filters: FieldName eq ‘Test’ and FieldName eq ‘Test’
B.	Not equal:
Enum filter: FieldName != ‘Test’ (not sure)
String filter: FieldName ne ‘Test’

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

ID: 1063123b-f794-e863-2cf2-231a499c68c6
Version Independent ID: 8c0c29c2-e3a2-809d-ce9e-6ca3b51c1d1b
Content: Customize table and column mappings - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/dual-write/customizing-mappings.md
Service: dynamics-365-cross-app
GitHub Login: @NHelgren
Microsoft Alias: nhelgren

@sheko89, thank you for providing this feedback!
@NHelgren, please consider updating the content based on this feedback.

Bug 637838 has been logged to track this issue.

Closed as won't fix.


itest: fix wrong node queried for balance

Was looking at the channel force close test and found the wrong node was used to query the balance. (weird as it should have been caught by the test?)
testChannelForceClosure has only two relevant participants, Alice(not net.Alice) and Carol, seems like net.Bob doesn't belong here.

The itest fails, with the following line,
carol's balance is incorrect: expected 1.05 BTC got 1.0498683 BTC
There's 13170 sats mismatched, still digging...

The itest fails, with the following line,
carol's balance is incorrect: expected 1.05 BTC got 1.0498683 BTC
There's 13170 sats mismatched, still digging...

Turns out that was the miner fee.

Very weird this ever worked. I assume because Bob just always had a high balance that was larger than craolExpectedBalance?

Yep I think so. When we set up the test we net.Bob will always have some coins.


Direct Message Restriction

Internal issue at https://its-pro.ucsd.edu/projects/MA/issues/MA-39?filter=allopenissues

Implemented changes see feature/direct-messages  on mobileplatform-notifications
Hash: f196094fe09db706418854ebcba230427f7f2b16

This issue is being closed due to platform migration and may be revisited at a later date.


Deferred dart:js library not work if loadLibrary in the other library

Dart VM version: 1.23.0 (Fri Apr 21 03:07:31 2017) on "macos_x64"
library a;

import 'dialogs.dart' deferred as dialog;

/**
 * Handle offline
 */
Future loadDialogLib(void onReady(_))
=> dialog.loadLibrary().then(onReady)
    .catchError(handleDeferedLoadingError);

void openDialogA() {
  loadDialogLib((_)//works fine
    =>  dialog.DialogA.open());
}

library b;

import 'a.dart' show loadDialogLib;
import 'dialogs.dart' deferred as dialog;

void openDialogB() {
  loadDialogLib((_)//error
    =>  dialog.DialogB.open());
}

NoSuchMethodError: No top-level getter 'dialog.DialogB' declared.
Receiver: top-level
Tried calling: dialog.DialogB

I believe this is working as intended.  Each deferred import (i.e., the import, not the target) must be explicitly loaded via loadLibrary.  In other words, dialog is not available to b until b imports it.
In general, this allows implementations to aggressively defer parts of dialog only used by b even if it needs to loads other parts (those used by a) earlier.
@floitschG - is this correct?

I agree that this is working as intended.
Each import must be loaded individually.


Issue with pulling api city names

When api.cities is used to pull cities for a country (say India), the name of the country also appear on the list of cities.  Also some of the strings for city names has "\n" within the string.  Would it be possible to fix this?

Hey @als0076 - thanks for pointing this out. This appears to be an issue with the OpenAQ API itself, not the python wrapper, so it may be a good idea to open an issue on that repository instead.

@dhhagan okay I will do. Thanks for responding so quickly!


[CI:DOCS]enabled base container create tests

enabling the base container create tests which allow us to get more into
the stop, kill, etc tests.
Signed-off-by: baude<EMAIL_ADDRESS>

/lgtm
/hold

/hold cancel


Change the LEDs on a gamepad

It would be cool to be able to change the LEDs on an Xbox 360 controller, then other gamepads too. To start with, I am just focusing on Linux.
What do we know?
We can send an event to /dev/input/eventN to change the caps lock led
import inputs
caps_lock = inputs.devices.keyboards[0].leds[0]
caps_lock.on()
caps_lock.off()

This sends an event in the normal evdev event format to the correct character device.
We can send an event to /dev/input/eventN to make the Gamepad rumble
import inputs
gamepad = inputs.devices.gamepads[0]
gamepad.set_vibration(1,1,1000)

On Linux, this does some interesting voodoo (in the method __get_vibration_code ) which creates a struct which is then encoded in hex and then sent as the code in the normal evdev event format to the correct character device.
We can get the current state of the gamepad led:
import inputs
gamepad_led = inputs.devices.gamepads[0].leds[0]
gamepad_led.status()

We cannot yet send an event to /dev/input/eventN to change the led,
either we don't know the right voodoo or there needs a tiny bit of
work in the linux driver.
We can (as root) bypass evdev and change brightness file and it will change the led. This is not the way.
Inputs runs as non-root and so we need to use /dev/input/eventN like with the caps lock light and rumble.
We have some info in the Linux source code.
We know all the brightness values ('commands'):
XBOX_STYLE_LED_CONTROL = {
    0: 'off',
    1: 'all blink, then previous setting',
    2: '1/top-left blink, then on',
    3: '2/top-right blink, then on',
    4: '3/bottom-left blink, then on',
    5: '4/bottom-right blink, then on',
    6: '1/top-left on',
    7: '2/top-right on',
    8: '3/bottom-left on',
    9: '4/bottom-right on',
    10: 'rotate',
    11: 'blink, based on previous setting',
    12: 'slow blink, based on previous setting',
    13: 'rotate with two lights',
    14: 'persistent slow all blink',
    15: 'blink once, then previous setting'
}

We know (based on this) that after the evdev phrase, the actual controller is expecting
0x01, 0x03 then the command.
Apparently, this also works on Max OS X
Useful Links

Kernel Input Documentation
Evdev source
Xpad source

Somehow I need to make a friend who is an expert in the kernel side of things, maybe one of the follow will know the answers: @paroj @Gottox @flibitijibibo @dtor @marcusfolkesson @auxym @TheUbuntuGuy

Nope, I added a oneliner to the kernel. Not an expert at all.

Haha, same here. I copy-pasted a few lines and changed a device id, sorry :/

Make that three.


Base64 directions

For OSX users, base64 commands with -d and -w are mapped differently.

gbase64 -w 0 is the same as base64 | tr -d '\n'. The latter works on both BSD and GNU.
I'm concerned that base64 -d might be a bug...
If you change the code, then it will all just work and documentation won't be needed. :)


BZ-1268219 - MySQL case sensitive table names - task list view error

jbpm-console-ng showcase with the queries adaptation for case sensitive db in master

+1, similar change already applied in kie-wb-distributions:
https://github.com/droolsjbpm/kie-wb-distributions/commit/08f2b571adc2b31597c436cde48e3bc3f1c673c6

It seems that exactly same changes are needed in multiple places. Would it be possible to share these queries between jbpm-console-ng-showcase, kie-wb and kie-drools-wb?

merged in master


Handles inclusion of Remote Path in FTP filename

Fixes error in #944's comment.
Will want @ahafele to maybe test in either dev or stage first before merging and deploying to production just in case there are additional issues in other parts of either the VMA or data-export.

I tested this for a couple of interfaces on stage. For some interfaces (like the one that prompted this report) the remote path name now needs to be included in the regex, for others they are working as-is. How can we instruct staff to know when to include the remote path in the regex for file fetching - for existing and future interfaces?

We have another example of an interface that has stopped fetching files - https://sul-libsys-airflow-prod.stanford.edu/vendor_management/interfaces/45

I tested the new vendor that was reported today and it appears to be the same issue. When I update the regex to include the pathname then the files are retrieved - https://sul-libsys-airflow-stage.stanford.edu/vendor_management/interfaces/145
Though the files are retrieved they are not being processed and I don't see in the DAGs any info about why (no errors etc.) You can see in the test from yesterday that the files have not been processed and have a status of loading
https://sul-libsys-airflow-stage.stanford.edu/vendor_management/interfaces/139

Keeping this PR around until this PR #957 is tested and reviewed to preserve the comments.

closing in favor of merged PR #957


Pty

Windows PowerShell
For Windows PowerShell 5.1 issues, suggestions, or feature requests please use the following link instead:
Windows PowerShell UserVoice
This repository is ONLY for PowerShell Core 6 issues.

@phyoyathupty, please either update your issue's title and body to provide relevant information, or, if you created the issue accidentally, close it.


[Style] Enhance the About Us page

Description
I want to improve the About Us page as its written in the simple text will put it into the box and when it will be hover then it will show  the details and also its a responsive too and matches the color theme
Assign me this issue under GSSOC24
Screenshots
Before Hover

After Hover

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

@Rakesh9100 this issue can be assigned

Provide the example screenshot of how it will look after your changes @SaiyamTuteja

I have added the screenshot above which shows that it will be in simple and clean heading in rectangular box and after we hover on it it will change the state to the paragraph of the mission , vision , goal

After Hover

after hovering the light purple color shows the transaction and shows the text and its responsive too

I need the screenshot example of how the actual page of website will look after your changes, it is not clear from your screenshot that what will you do @SaiyamTuteja

This is the Overview after putting this into the about us page

Okay go ahead @SaiyamTuteja

Any update on this @SaiyamTuteja

Working on it

Check this PR #1799 i have solved the issue which you are facing


Document handling of package version not found

Fix #433

Looks like a typo slipped through:
"If no version information found on package.json, ⚠ symbol vill be displayed."
I would consider changing it to:
"If no version information is found in package.json, the ⚠ symbol will be displayed."


Tech/Civic Tree and Civilopedia autofocus on the search bar

It seems we have a feature in our mod description that may no longer function:  (From SteamWorkshop thread)

Tech/Civic Tree and Civilopedia now autofocus the searchbar

In tech and civic tree this works but not in civilopedia for me at least, meaning when i go in the civilopedia i have to click the search bar with the mouse manually instead of already being selected on opening the window. This seems veeer minor, but if you use it often it is sooo annoying and very uncomfortable.

Civilopedia is no longer a part of CQUI.
Long time ago Pedia was in CQUI because there was a history and a back button. This functionality is in the base game, so probably was removed.
And the base version is missing AutoFocus.

If you want to add it back then 1 line is missing in OnOpenCivilopedia():
Controls.SearchEditBox:TakeFocus();

I think we could get away with function replacement this time if that's all we plan to add to the Civilopedia interface.

Closing as this is fixed


[Bug] sshx doesn't work on certain CLI apps

When running commands like flyctl apps create or gh auth login (after pressing enter a few times), sshx gets stuck and I can't even ctrl+c out of the command. This works for me in my normal terminal though.

It might be related to #60


Icon for the .spe extension of the spacengine template engine.

Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Please open an issue first.


Plotting histos and saving to pdf/png in script

Hi,
I've been trying Histbook for some experimental analysis and it seems very good! However I'd like to run over data samples and create plots of e.g. signal vs background(s) for many plottable variables, and save them to png/pdf.
As it is, I can do this via VegaLite using a web browser, by pressing save, but cannot seem to find a way to save directly to a file in the script without opening a browser, so that one can run e.g. in a headless gnu screen session or some sort of cluster/batch mode.
Is there a simple way plots can be exported/saved in Histbook without going through a web browser?
Thanks,
Alex

This is one of the reasons for VegaScope. That still goes through a web browser, but it can invoke the browser to save as PNG/SVG (use Inkscape or equivalent to convert SVG to PDF). Pop-ups need to be unblocked for VegaScope to have permissions to do that.
It might be a better option to install [vega-lite as an npm package] (https://www.npmjs.com/package/vega-lite), which of course means running nodejs, a non-browser JavaScript environment. This surely has vega-lite to PNG/SVG converters, and there are npm packages that do SVG to PDF conversion.
As you can see, I haven't figured out what would be a good workflow for this, but there are some leads. I don't know if nodejs would be considered an installation burden— it steps out of the Python ecosystem, but it's a well-maintained ecosystem on its own. (A lot of web services use it on the backend to unify server and client code.)
I'm going to leave this issue open, hoping to collect ideas.
Oh, there's another option: histbook Hists can be turned into Pandas DataFrames as easily as vega-lite, and Pandas plots as Matplotlib. I didn't want to bring this dependency into histbook itself (outputting pure JSON is zero-dependency), but you can get to it though Pandas. Just be sure to use the line-drawing and bar chart features of Pandas plotting— "histogram" features aren't relevant because the data passed to Pandas are already aggregated into bins.


Robert lowe solution

Comfort score, 5 out of 5
Completeness score,  5 out of 5

👍 very well done. mad prop$ on marking your references to jquery collections with $


copy_dir with gitignore options

This will adhere to the gitignore file found in the directory to copy and also allows include / exclude globs to be specified for further filtering.

I am overtaking this PR as discussed.
To summarize what we discussed and what I understood (please correct if necessary):

The ignore crate shouldn't be used anymore because it is tailored towards git and we possibly need another syntax (because the ! prefix for negation is a yaml builtin)
The copy_dir function should be integrated into recipe processing so that the license_file key can use a list of pathes and filters and that gets processed properly. Later this integration should also be integrated for other keys in the recipe.

Yep, that sounds good!


fix(aws_sdk.js): support put records in kinesis

Currently support just the first record

:tada: This PR is included in version 1.107.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add opted_in for providers

Context
UCAS Transition
Changes proposed in this pull request
Add opted_in to providers and scope v1/providers to only return when true

 incremental transition - tech work

@benilovj do we know which providers are going to be "opted in"?
I can knock up a rake task or migration to update those..

@benilovj do we know which providers are going to be "opted in"?
I can knock up a rake task or migration to update those..

Not yet – let's leave that as a separate task later, once we've decided and agreed the cohorts.


Use cp -p in Makefile to preserve last modified date

Replaces all invocations of cp with cp -p to preserve the last modified date when installing the yosys header files. This means that after make install a subsequent call to make will only compile the files that have been modified, instead of a full recompile.

This would be problematic in situations where one has two different yosys source directories and uses make install in one of them to install the files over the previously installed files from the other directory in an alternating pattern. Something like this is not uncommon during development. In this situation you always want to make sure that every file that has been changed by make install also has the current time as modification time.
A possible solution would be to e.g. use rsync -c to install. This will update the timestamp on the target if the target is different from the source, and leave the target untouched otherwise. But I'm hesitant to add a dependency to rsync for this.

This means that after make install a subsequent call to make will only compile the files that have been modified, instead of a full recompile.

I don't really understand what you mean here. Calling make install && make will not trigger a full recompile:
clifford@euklid:~/Work/yosys$ make install
[Makefile.conf] INSTALL_SUDO := sudo
sudo mkdir -p /usr/local/bin
sudo install yosys yosys-config yosys-abc yosys-filterlib yosys-smtbmc /usr/local/bin
sudo mkdir -p /usr/local/share/yosys
sudo cp -r share/. /usr/local/share/yosys/.

clifford@euklid:~/Work/yosys$ make
[Makefile.conf] INSTALL_SUDO := sudo

  Build successful.

clifford@euklid:~/Work/yosys$

Are you talking about running make somewhere else? Like a Yosys plugin?

It triggers a full recompile on my system because nearly every source file depends on the headers installed in /usr/local/share/yosys

It triggers a full recompile on my system because nearly every source file depends on the headers installed in /usr/local/share/yosys

None of the yosys source files depend on anything in /usr/local/share/yosys.

Sure. After running
cp share/include/kernel/yosys.h /usr/local/share/yosys/include/kernel/yosys.h
make --debug outputs this:
Reading makefiles...
Updating goal targets....
 File `all' does not exist.
   File `top-all' does not exist.
       Prerequisite `/usr/local/share/yosys/include/kernel/yosys.h' is newer than target `libs/subcircuit/subcircuit.o'.
      Must remake target `libs/subcircuit/subcircuit.o'.
[  8%] Building libs/subcircuit/subcircuit.o
      Successfully remade target file `libs/subcircuit/subcircuit.o'.
       Prerequisite `/usr/local/share/yosys/include/kernel/yosys.h' is newer than target `frontends/ast/ast.o'.
      Must remake target `frontends/ast/ast.o'.
[ 12%] Building frontends/ast/ast.o
      Successfully remade target file `frontends/ast/ast.o'.
       Prerequisite `/usr/local/share/yosys/include/kernel/yosys.h' is newer than target `frontends/ast/genrtlil.o'.
      Must remake target `frontends/ast/genrtlil.o'.

Can you tell where this comes from? E.g. why does libs/subcircuit/subcircuit.o depend on /usr/local/share/yosys/include/kernel/yosys.h on your machine? Is it listed in libs/subcircuit/subcircuit.d? Why? Does your libs/subcircuit/subcircuit.d list the correct dependency to kernel/yosys.h?
E.g. what output do you get for this grep command:
clifford@euklid:~/Work/yosys$ grep yosys.h libs/subcircuit/subcircuit.d
  /usr/include/assert.h kernel/yosys.h \

What command are you using to build subcircuit.o? I.e. what output do you get for this:
clifford@euklid:~/Work/yosys$ rm -f libs/subcircuit/subcircuit.o; make PRETTY=0 libs/subcircuit/subcircuit.o
[Makefile.conf] INSTALL_SUDO := sudo
mkdir -p libs/subcircuit/
clang -o libs/subcircuit/subcircuit.o -c  -Wall -Wextra -ggdb -I. -I"./" -MD -D_YOSYS_ -fPIC -I/usr/local/include -std=c++11 -Os -DYOSYS_ENABLE_READLINE -DYOSYS_ENABLE_PLUGINS  -I/usr/include/tcl8.6 -DYOSYS_ENABLE_TCL -DYOSYS_ENABLE_ABC -DYOSYS_ENABLE_COVER libs/subcircuit/subcircuit.cc

Last question for this post: Do you have any bogus symlinks from /usr/include/... to /usr/local/share/yosys/include/...?

grep yosys.h libs/subcircuit/subcircuit.d outputs
/usr/include/sys/_types/_mode_t.h kernel/yosys.h \
and
make PRETTY=0 libs/subcircuit/subcircuit.o outputs
mkdir -p libs/subcircuit/
clang -o libs/subcircuit/subcircuit.o -c  -Wall -Wextra -ggdb -I. -I"./" -MD -D_YOSYS_ -fPIC -I/usr/local/include -I/opt/local/include -I/usr/local/opt/readline/include -I/usr/local/Cellar/libffi/3.0.13/lib/libffi-3.0.13/include -std=c++11 -Os -DYOSYS_ENABLE_READLINE -DYOSYS_ENABLE_PLUGINS  -I/usr/include/tcl8.5 -DYOSYS_ENABLE_TCL -DYOSYS_ENABLE_ABC -DYOSYS_ENABLE_COVER libs/subcircuit/subcircuit.cc

(I am on a mac if that helps)

grep yosys.h libs/subcircuit/subcircuit.d outputs
/usr/include/sys/_types/_mode_t.h kernel/yosys.h \

Interesting. If libs/subcircuit/subcircuit.d does not list /usr/local/share/yosys/include/kernel/yosys.h, why does subcircuit.o depend on it? (I know, this are hard questions. But since I cannot reproduce the issue here, asking hard questions is pretty much the only thing I can do to figure out whats going on.)

(I am on a mac if that helps)

I don't have a mac and never had one. So if there is some mac-specific thing that would be useful to know about to understand what's going on here: I would not know about it.. :(

I just updated my make to 4.2.1 and it still does it.
Oh well, if this isn't a problem for anybody else, I'll just keep it on my local copy.


Add support for preparing Deal Data

Hi,
I have created a module in storage SDK to process the user specified file and fetch the data required for deal making in FVM. Below is the sample code to invoke the getPrepData() module.
async function main () {
const dealData = await client.getPrepData('README.md')
console.log(dealData)
}
main()
If this PR goes through I will create another PR to update the docs. Please let me know if you have questions.
Thank you!

The change in the README was inadvertent. I am sorry I did not realise it.

No worries, PR is quite good. Let @shomix take a look.


Cannot default array of objects to an empty array

Prerequisites

[x] I have read the documentation;
[x] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.

Description
When creating an array of objects using minItems there is no way to default the array to an emtpy array. If the array is empty the package with fill it with the structure of the object with all of its keys having values of undefined.
Steps to Reproduce
Shared playground
Expected behavior
When an array of objects has a default value of [] and minItems set higher than 1 it should default to an empty array with a validation error saying that the array must have at least minItems.
Actual behavior
minItems number of empty objects are injected into the array to satisfy the minItems constraint. If the object has any required fields validation with throw an error about the empty object not having that field instead of the array not having enough items.
Version
1.0.4

I think the underlying problem here is actually that the defaults weren't being applied to form correctly. The PR #1034 which was merged yesterday fixes this (you can test by checking out master and running the playground locally).
@glasserc when do you think 1.0.5 will be pushed to NPM?

I can probably cut a release today. I am trying to figure out how to handle #991.

I am still getting the same issue on master 😞


Add FirefoxOS

https://github.com/mozilla-b2g/moztt

This is Fira Sans, I believe. https://github.com/mozilla/Fira


[DNM] Investigate RocksDb 6.29 sysbench regression

What is changed and how it works?
Issue Number: Ref #13095
What's Changed:
None

Related changes
Check List
Release note
None

/build

/run-build

/run-build

/run-build

/run-build

/run-build


Rollback with database migrate (teardown)?

Hi,
looking at the docs & code I see no possibility to do a database migration together with a rollback. We have migration scripts using Phinx which have up and down methods to build and destroy database changes. Sometimes it is essentially necessary to revert database changes if rolling back a deploy because the previous release might not work with newly changed database.
Any hints or suggestions on that?
Thx in advance and best regards
tpinne

Have you tried hooking into the process via before and rollback in your hooks.php?

@mbrodala is correct, hooking onto the events of the rollback is the correct way to go about it


Use UCX wheels

This PR updates UCX to stop bundling UCX libraries, instead preferring to get them from the new UCX wheels that we are publishing from https://github.com/rapidsai/ucx-wheels.

/merge


Startup issues

Maybe just MacOS?
                                                                                (base) ➜  dassie git:(main) rm -rf local
(base) ➜  dassie git:(main) pnpm start

><EMAIL_ADDRESS>start /Users/nicholasdudfield/projects/dassie
> node packages/app-dev/bin/start-development-server.js

  Dassie//dev

  Starting development server...

  Debug UI: https://localhost/ <- Start here

‼ child exited code=1 signal=null
‼ error in actor actor="NodeActorsMapped[0]" path="RootActor.RunNodesActor.NodeActorsMapped[0]"

Error: child exited
    at ChildProcess.handleChildExit (/Users/nicholasdudfield/projects/dassie/packages/app-dev/src/backend/functions/run-child-process.ts:58:20)
    at ChildProcess.emit (node:events:514:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:294:12)

‼ error starting nodes

Error: child exited
    at ChildProcess.handleChildExit (/Users/nicholasdudfield/projects/dassie/packages/app-dev/src/backend/functions/run-child-process.ts:58:20)
    at ChildProcess.emit (node:events:514:28)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:294:12)

! proxy connection refused, maybe the node is still starting up port=5003
! proxy connection refused, maybe the node is still starting up port=5003
! proxy connection refused, maybe the node is still starting up port=5003
! proxy connection refused, maybe the node is still starting up port=5003
! proxy connection refused, maybe the node is still starting up port=5003
! proxy connection refused, maybe the node is still starting up port=5003
! proxy connection refused, maybe the node is still starting up port=5002
! proxy connection refused, maybe the node is still starting up port=5002
! proxy connection refused, maybe the node is still starting up port=5002
! proxy connection refused, maybe the node is still starting up port=5002
! proxy connection refused, maybe the node is still starting up port=5002
! proxy connection refused, maybe the node is still starting up port=5002
! proxy connection refused, maybe the node is still starting up port=5004
! proxy connection refused, maybe the node is still starting up port=5004
! proxy connection refused, maybe the node is still starting up port=5004
! proxy connection refused, maybe the node is still starting up port=5004
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001
! proxy connection refused, maybe the node is still starting up port=5001

It's alive!
First it's started for me in many moons


chore: Select component refactoring - TimeSeriesColumnControl - Iteration 5

SUMMARY
It replaces the react-select Select component with the Antdesign one in the TimeSeriesColumnControl component.
BEFORE
https://user-images.githubusercontent.com/60598000/130768485-f694c7ec-b6e3-43d0-b178-93a8b7326be8.mp4
AFTER
https://user-images.githubusercontent.com/60598000/130768576-9d61f25f-c677-490b-b16b-ce2130f441df.mp4
TESTING INSTRUCTIONS

Open a Time-series Table Chart in Explore
Add time columns and observe the dropdown behavior

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@rosemarie-chiu @jinghua-qa

/testenv up

To fix TypeError: handleOnSearch.cancel is not a function in your tests you have two options:

Don't mock debounce and just account for the async behavior.
Include the cancel function is your mock.

jest.mock('lodash/debounce', () => (fn: Function) => ({
  ...fn,
  cancel: jest.fn(),
}));


Chef13 chef15

Description
Fix problems with a name collision over filters.  It was used as a method in the helper module and as a resource property.  It worked in chef 14, but not in chef 13 or 15.
Issues Resolved
#158
#159
#162
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

This change is

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger

Do you think we should rip out the code that makes this work on chef 12?  The chef 13 fix is incidental, the same fix is needed for chef 15.
Thanks,
Mark

Yeah Chef 12 is now EOL, and we should be ideally removing code that makes that work so we have less to maintain going forward.
Cheers,
Dan

Hi Dan,
IMO ripping out the 1 method that supports chef 12 would be a breaking change.  This PR fixes something that is really broken for chef client version 15 and needs to get merged.  How about we get this merged and released and come back to the chef 12 question later?
thanks,
Mark


Data Types in python

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # <issue_number_here>

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ X] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ X] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[ ] I follow Contributing Guidelines & Code of conduct of this project.
[ ] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] My changes generates no new warnings.
[X ] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@adityabisoi Close this pr


Bump dependencies

Changed dependencies
No dependencies bumped.
Requirement changes
No requirements changed.

whyyy did i update npm to v10 :sob:


updated warning.default to #DA8301

Updated warning.default from bright yellow to a darker orange yellow  #FFD33D --> #DA8301

@georgewrmarshall  Althought I created this PR, it might make things easier if the description-update branch is merged first, or the yellow update & its description update may conflict. We can either delete this branch and do one step at a time. Lmk


Fails to build on mipsel / m68k / powerpc / sh4

Not sure, if it even was intended to run on these arches. I use the library, b/c it is pulled in by luametatex. I notice that linking of the libs fails on a few CPU architectures (mipsel / m68k / powerpc / sh4) when linking:
./obj-mipsel-linux-gnu/./source/libraries/mimalloc/src/arena.c:477:(.text+0x1440): undefined reference to `__atomic_load_8'
/usr/bin/ld: ./obj-mipsel-linux-gnu/./source/libraries/mimalloc/src/arena.c:479:(.text+0x1480): undefined reference to `__atomic_fetch_add_8'
/usr/bin/ld: ./obj-mipsel-linux-gnu/./source/libraries/mimalloc/src/arena.c:482:(.text+0x14e4): undefined reference to `__atomic_store_8'
/usr/bin/ld: ./obj-mipsel-linux-gnu/./source/libraries/mimalloc/src/arena.c:471:(.text+0x1370): undefined reference to `__atomic_load_8'
/usr/bin/ld: ./obj-mipsel-linux-gnu/./source/libraries/mimalloc/src/arena.c:479:(.text+0x1458): undefined reference to `__atomic_fetch_add_8'
/usr/bin/ld: ./obj-mipsel-linux-gnu/./source/libraries/mimalloc/src/arena.c:482:(.text+0x14d8): undefined reference to `__atomic_store_8'
collect2: error: ld returned 1 exit status

There a lot more error messages like these and at least they are common to all these CPU arches. I build (and link) using the gcc compiler suite and as far as I found out on these arches the library libatomic1 needs to linked. Maybe I can patch that in my Makefile.
Hmm, the CMakeLists.txt says:
option(MI_USE_LIBATOMIC     "Explicitly link with -latomic (on older systems) (deprecated and detected automatically)" OFF)
..but that doesn't seem to work reliably. At least in the Debian package they do something like:
LIBATOMIC=OFF
ifneq (,$(filter $(DEB_HOST_ARCH), armel m68k mips mipsel powerpc powerpcspe sh4))
  LIBATOMIC=ON
endif

configure (...) -DMI_USE_LIBATOMIC=$(LIBATOMIC)

OK, I'm able to reproduce the issue inside an m68k emulated by qemu. I use Debian and gcc-13. I installed the packages needed to build the Debian package, then downloaded the 2.1.2 tar ball (and "git clone" as 2nd try). The build fails at the end:
[ 80%] Linking C static library libmimalloc.a
[ 80%] Built target mimalloc-static
[ 82%] Building C object CMakeFiles/mimalloc-obj.dir/src/static.c.o
[ 82%] Built target mimalloc-obj
[ 85%] Generating mimalloc.o
[ 85%] Built target mimalloc-obj-target
[ 87%] Building C object CMakeFiles/mimalloc-test-api.dir/test/test-api.c.o
[ 90%] Linking C executable mimalloc-test-api
/usr/bin/ld: libmimalloc.so.2.1: undefined reference to `__atomic_load_8'
/usr/bin/ld: libmimalloc.so.2.1: undefined reference to `__atomic_store_8'
/usr/bin/ld: libmimalloc.so.2.1: undefined reference to `__atomic_fetch_add_8'
/usr/bin/ld: libmimalloc.so.2.1: undefined reference to `__atomic_compare_exchange_8'
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/mimalloc-test-api.dir/build.make:100: mimalloc-test-api] Error 1
make[1]: *** [CMakeFiles/Makefile2:199: CMakeFiles/mimalloc-test-api.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

I could upload the full build log, but I guess it is not of any help.


Crashes after several hours that corrupts all saves

I am using the Bannerkings at war collection (https://next.nexusmods.com/mountandblade2bannerlord/collections/pnzkoo?tab=mods) and constantly getting this crash after several hours of game play. (often crashes around 20 days). Afterwards, earlier saves of that campaign are also corrupted regardless if the saves were from before the crash happened. Had a few extra save files and it did not help.
After disabling a bunch of other mods, disabling bannerkings actually gets a different crash report but its likely due to the save being incompatible. (mscorlib null value sorta thing)
The current callstack is:
Exception information
Type: System.NullReferenceException
Message: Object reference not set to an instance of an object.
Source: TaleWorlds.CampaignSystem
CallStack:
at Hero TaleWorlds.CampaignSystem.Party.PartyComponents.BanditPartyComponent.get_PartyOwner()
at Hero TaleWorlds.CampaignSystem.Party.PartyBase.get_Owner()
at IFaction TaleWorlds.CampaignSystem.Party.MobileParty.get_MapFaction()
at void TaleWorlds.CampaignSystem.MapEvents.MapEvent.Update()
at void TaleWorlds.CampaignSystem.MapEvents.MapEventManager.Tick()
at void TaleWorlds.CampaignSystem.Campaign.Tick()
at void TaleWorlds.CampaignSystem.GameState.MapState.OnMapModeTick(float dt)
at void TaleWorlds.CampaignSystem.GameState.MapState.OnTick(float dt)
at void TaleWorlds.Core.GameStateManager.OnTick(float dt)
at void TaleWorlds.Core.Game.OnTick(float dt)
at void TaleWorlds.Core.GameManagerBase.OnTick_Patch9(GameManagerBase this, float dt)
at void TaleWorlds.MountAndBlade.Module.OnApplicationTick_Patch2(Module this, float dt)

enhanced stacktrace:
Frame: Hero TaleWorlds.CampaignSystem.Party.PartyComponents.BanditPartyComponent.get_PartyOwner() (IL Offset: 0)
Module: UNKNOWN
Method: virtual TaleWorlds.CampaignSystem.Hero TaleWorlds.CampaignSystem.Party.PartyComponents.BanditPartyComponent::get_PartyOwner()
Method From Stackframe Issue: False

Frame: Hero TaleWorlds.CampaignSystem.Party.PartyBase.get_Owner() (IL Offset: 0)
Module: UNKNOWN
Method: TaleWorlds.CampaignSystem.Hero TaleWorlds.CampaignSystem.Party.PartyBase::get_Owner()
Method From Stackframe Issue: False

Frame: IFaction TaleWorlds.CampaignSystem.Party.MobileParty.get_MapFaction() (IL Offset: 0)
Module: UNKNOWN
Method: virtual TaleWorlds.CampaignSystem.IFaction TaleWorlds.CampaignSystem.Party.MobileParty::get_MapFaction()
Method From Stackframe Issue: False

Frame: void TaleWorlds.CampaignSystem.MapEvents.MapEvent.Update() (IL Offset: 41)
Module: UNKNOWN
Method: System.Void TaleWorlds.CampaignSystem.MapEvents.MapEvent::Update()
Method From Stackframe Issue: False

Frame: void TaleWorlds.CampaignSystem.MapEvents.MapEventManager.Tick() (IL Offset: 92)
Module: UNKNOWN
Method: System.Void TaleWorlds.CampaignSystem.MapEvents.MapEventManager::Tick()
Method From Stackframe Issue: False

Frame: void TaleWorlds.CampaignSystem.Campaign.Tick() (IL Offset: 68)
Module: UNKNOWN
Method: System.Void TaleWorlds.CampaignSystem.Campaign::Tick()
Method From Stackframe Issue: False

Frame: void TaleWorlds.CampaignSystem.GameState.MapState.OnMapModeTick(float dt) (IL Offset: 140)
Module: UNKNOWN
Method: System.Void TaleWorlds.CampaignSystem.GameState.MapState::OnMapModeTick(System.Single dt)
Method From Stackframe Issue: False

Frame: void TaleWorlds.CampaignSystem.GameState.MapState.OnTick(float dt) (IL Offset: 77)
Module: UNKNOWN
Method: virtual System.Void TaleWorlds.CampaignSystem.GameState.MapState::OnTick(System.Single dt)
Method From Stackframe Issue: False

Frame: void TaleWorlds.Core.GameStateManager.OnTick(float dt) (IL Offset: 47)
Module: UNKNOWN
Method: System.Void TaleWorlds.Core.GameStateManager::OnTick(System.Single dt)
Method From Stackframe Issue: False

Frame: void TaleWorlds.Core.Game.OnTick(float dt) (IL Offset: 0)
Module: UNKNOWN
Method: System.Void TaleWorlds.Core.Game::OnTick(System.Single dt)
Method From Stackframe Issue: False

Frame: void TaleWorlds.Core.GameManagerBase.OnTick_Patch9(GameManagerBase this, float dt) (IL Offset: -1)
Module: UNKNOWN
Method: static System.Void TaleWorlds.Core.GameManagerBase.OnTick_Patch9(TaleWorlds.Core.GameManagerBase this, System.Single dt)
Method From Stackframe Issue: False

Frame: void TaleWorlds.MountAndBlade.Module.OnApplicationTick_Patch2(Module this, float dt) (IL Offset: -1)
Module: UNKNOWN
Method: static System.Void TaleWorlds.MountAndBlade.Module.OnApplicationTick_Patch2(TaleWorlds.MountAndBlade.Module this, System.Single dt)
Method From Stackframe Issue: False

Mod Version
tested on vers 1.14, modlist is the Bannerkings at war collection
Installed Modules
[+ Harmony (Bannerlord.Harmony, v<IP_ADDRESS>)](javascript:;)
[+ ButterLib (Bannerlord.ButterLib, v<IP_ADDRESS>)](javascript:;)
[+ UIExtenderEx (Bannerlord.UIExtenderEx, v<IP_ADDRESS>)](javascript:;)
[+ Mod Configuration Menu v5 (Bannerlord.MBOptionScreen, v<IP_ADDRESS>)](javascript:;)
[+ Native (Native, v<IP_ADDRESS>)](javascript:;)
[+ SandBox Core (SandBoxCore, v<IP_ADDRESS>)](javascript:;)
[+ Sandbox (Sandbox, v<IP_ADDRESS>)](javascript:;)
[+ Achievement Unblocker (AchievementUnblocker, v<IP_ADDRESS>)](javascript:;)
[+ StoryMode (StoryMode, v<IP_ADDRESS>)](javascript:;)
[+ CustomBattle (CustomBattle, v<IP_ADDRESS>)](javascript:;)
[+ Birth and Death Options (BirthAndDeath, v<IP_ADDRESS>)](javascript:;)
[+ Arena Overhaul (ArenaOverhaul, v<IP_ADDRESS>)](javascript:;)
[+ BannerLib (BannerLib, v<IP_ADDRESS>)](javascript:;)
[+ Banner Fix (BannerFix, v<IP_ADDRESS>)](javascript:;)
[+ Better HUD (BetterHUD, v<IP_ADDRESS>)](javascript:;)
[+ Better Time (BetterTime, v<IP_ADDRESS>)](javascript:;)
[+ Character Manager (CharacterManager, v<IP_ADDRESS>)](javascript:;)
[+ Character Reload (CharacterReload, e<IP_ADDRESS>)](javascript:;)
[+ Cheats (Cheats, v<IP_ADDRESS>)](javascript:;)
[+ Raise your Torch (RaiseYourTorch, v<IP_ADDRESS>)](javascript:;)
[+ CustomizableTroopWages (CustomizableTroopWages, v<IP_ADDRESS>)](javascript:;)
[+ EquipBestItem (EquipBestItem, v<IP_ADDRESS>)](javascript:;)
[+ Fourberie (Fourberie, v<IP_ADDRESS>)](javascript:;)
[+ HoldCourt (HoldCourt, v<IP_ADDRESS>)](javascript:;)
[+ Diplomacy (Bannerlord.Diplomacy, v<IP_ADDRESS>)](javascript:;)
[+ Improved Garrisons (ImprovedGarrisons, v<IP_ADDRESS>)](javascript:;)
[+ KillBanditsRaiseRelations (KillBanditsRaiseRelations, v<IP_ADDRESS>)](javascript:;)
[+ Realistic Weather (RealisticWeather, v<IP_ADDRESS>)](javascript:;)
[+ MBFastDialogue (MBFastDialogue, v<IP_ADDRESS>)](javascript:;)
[+ Dismemberment (Plus) for v1.1.3 (DismembermentPlus, v<IP_ADDRESS>)](javascript:;)
[+ (RBM) Realistic Battle Mod (RBM, v<IP_ADDRESS>)](javascript:;)
[+ Open Source Armory (OpenSourceArmory, v<IP_ADDRESS>)](javascript:;)
[+ Open Source Armory: RBM Patch (OpenSourceArmoryRBM, v<IP_ADDRESS>)](javascript:;)
[+ Adonnay's Troop Changer (AdonnaysTroopChanger, v<IP_ADDRESS>)](javascript:;)
[+ Tier 7 Troop Unlocker Harmony (T7TroopUnlocker, v<IP_ADDRESS>)](javascript:;)
[+ Banner Editor (BannerEditor, v<IP_ADDRESS>)](javascript:;)
[+ Hot Butter (hotbutter, v<IP_ADDRESS>)](javascript:;)
[+ Random Events (RandomEvents, v<IP_ADDRESS>)](javascript:;)
[+ Aggregated Income (SortedIncome, v<IP_ADDRESS>)](javascript:;)
[+ Tournaments XPanded (TournamentsXPanded, v<IP_ADDRESS>)](javascript:;)
[+ UnlimitedCAP (UnlimitedCAP, v<IP_ADDRESS>)](javascript:;)
[+ UnlimitedWS (UnlimitedWS, v<IP_ADDRESS>)](javascript:;)
[+ Surrender Tweaks (SurrenderTweaks, v<IP_ADDRESS>)](javascript:;)
[+ Immersive Battlefields (ImmersiveBattlefields, v<IP_ADDRESS>)](javascript:;)
[+ Baby Growth Mod (BabyGrowthMod, v<IP_ADDRESS>)](javascript:;)
[+ You Keep What You Kill (YouKeepWhatYouKill, v<IP_ADDRESS>)](javascript:;)
[+ KnowEveryone (KnowEveryone, v<IP_ADDRESS>)](javascript:;)
[+ Women in Calradia (WomeninCalradia, v<IP_ADDRESS>)](javascript:;)
[+ CheyronMod (CheyronMod, v<IP_ADDRESS>)](javascript:;)
[+ Detailed Character Creation (zzCharacterCreation, v<IP_ADDRESS>)](javascript:;)
[+ My Little Warband (MyLittleWarband, v<IP_ADDRESS>)](javascript:;)
[+ No More Clones (NoMoreClones, v<IP_ADDRESS>)](javascript:;)
[+ SandBox Siblings (SandBoxSiblings, v<IP_ADDRESS>)](javascript:;)
[+ POC Color Randomizer mod (PocColor, v<IP_ADDRESS>)](javascript:;)
[+ Settlement Icons Redesign (SettlementIconRedesign, v<IP_ADDRESS>)](javascript:;)
[+ The Color of Profits (TheColorOfProfits, v<IP_ADDRESS>)](javascript:;)
[+ Workshops Completed (WorkshopsCompleted, v<IP_ADDRESS>)](javascript:;)
[+ Better Smithing Continued (BetterSmithingContinued, v<IP_ADDRESS>)](javascript:;)
[+ Primae Noctis (PrimaeNoctisBLord, v<IP_ADDRESS>)](javascript:;)
[+ WarlordsBattlefield (WarlordsBattlefield, e<IP_ADDRESS>)](javascript:;)
[+ Xorberax's Legacy for v1.1.3 (XorberaxLegacy, v<IP_ADDRESS>)](javascript:;)
[+ RTS Camera (RTSCamera, v<IP_ADDRESS>)](javascript:;)
[+ BannerKings (BannerKings, v<IP_ADDRESS>)](javascript:;)

This happened to me as well. Did a bit of debugging with dnSpy and it seems a party object has some invalid fields, these will be the values in my case. Modified the code to remove the event from the queue and I finally managed to get past the exception. Maybe this will help developers

Hi, I have an error just like this blocking my campaign (with the same version, and pretty much same mods). I'm ready to try using dnSpy to solve this, is it manageable without any coding experience ?

I'm facing the same issue.
Crash report: https://report.butr.link/B5F74B.html

There a bin file patch for Calradia Expanded on the Banner Kings discord server #test-version. You'll find the file on the Dec 8, 2022 post. Had to copy over the bin in the CE module. I could even load my old corrupted save after.


chore: bump dlp to 0.7.0

[ ] Tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

I'd like it if we started using releasetool for staging this stuff :)   Unless there's a fire burnin', can you work with @crwilcox to make sure releasetool works for node.js, and use that to stage the release?

Ah I see. Can we merge this one in first? The release is due today. @JustinBeckwith


Render README.md to /about

@somarlyonks Better to make the markdown render compatible with common use as we may render dev blogs or things like that.


CEO-243 Add Users/QAQC to Review page

Purpose

Adds the User & QA/QC Assignments to the Project Review page.
Related Issues
Closes CEO-243
Submission Checklist

[x] Commits include the JIRA issue and the #review hashtag (e.g. CEO-### #review <comment>)
[x] Code passes linter rules (npm run eslint/clj-kondo --lint src)

Screenshots
Equal User Method / SME Quality Method

Percent User Method / Overlap Quality Method

Equal User Method / None Quality Method

src/js/project/ReviewForm.js
78, 88, Showing the user assignment but not QC will show a blank square.  I can be convinced that its ok to show neither if there is no assignment, but at least QC needs to show "none" if user assignments is visible.
src/js/project/CreateProjectWizard.js
145, remove console.log
200, If the user goes to /review-project first, this information does not need to be reloaded.  This should also address your fixme since this should only be called from /create-project.
206, you dont need to reset institutionUserList to [], thats its default, and you only call the api once.
208, remove this alert and return a Promise.reject.  Errors are handled in the Promise.all
src/js/project/ManageProject.js
47, remove console.log. This block can be one line
50, Return a Promise.reject() instead.  The error messaging is handled by the Promise.all.
23, this isnt quite safe. I think it mostly works because getProjectById will normally return first.  setting the context is async like setting state.  Pass synchronous projectId from getProjectById.  You have two options

Chain off getProjectById

    getProjectById = projectId => fetch(`/get-project-by-id?projectId=${projectId}`)
        .then(response => (response.ok ? response.json() : Promise.reject(response)))
        .then(data => {
            if (data === "") {
                alert("No project found with ID " + projectId + ".");
                window.location = "/home";
            } else {
                this.context.setProjectDetails(data);
                this.context.setContextState({originalProject: data});
                return data.institution;
            }
        })
        .then(institution => this.getInstitutionUserList(institution))
        .catch(() => Promise.reject("Error retrieving the project."));

Keep in Promise.all()

    getProjectDetails = () => Promise.all([
        this.getProjectById(this.context.projectId),
        this.getProjectImagery(this.context.projectId),
        this.getProjectPlots(this.context.projectId)
    ])
        .then(resp => this.getInstitutionUserList(resp[0]))
        .catch(response => {
            console.log(response);
            alert("Error retrieving the project info. See console for details.");
        });

    getProjectById = projectId => fetch(`/get-project-by-id?projectId=${projectId}`)
        .then(response => (response.ok ? response.json() : Promise.reject(response)))
        .then(data => {
            if (data === "") {
                alert("No project found with ID " + projectId + ".");
                window.location = "/home";
            } else {
                this.context.setProjectDetails(data);
                this.context.setContextState({originalProject: data});
                return data.institution;
            }
        })
        .catch(() => Promise.reject("Error retrieving the project."));

50, you dont need to reset institutionUserList to [], thats its default, and you only call the api once.

src/js/project/CreateProjectWizard.js
145, remove console.log
I didn't touch this part of the codebase. Should this be replaced with Promise.reject(response)?


Ability to disable unwanted HVAC modes

It would be great to disable unwanted HVAC modes.
I'm right now testing one a single thermostat, to evaluate whenever hass is finally ready to switch over to, and I see certain few problem, one of which is that for example toggle thermostat, it goes from OFF to HEAT straight away. Sounds great, but TVR that I'm using (control over climate), HEAT means wide open thermostat not caring about any temp -> leading to boiler running 100% all the time changing the room into sauna.
Also I'm a bit puzzled (but I know that I'm stupid), why frost protection is not an actual hvac mode, but instead it's a preset ... HASS toggle works on hvac modes, not presets. When people want to "switch off the heating" they understand that "intelligent heating system" will self protect to not allow house to freeze.

Hello @tomtom13 ,
HA works like this. I understand if it not the way you want to use it, but I can't do anything for you. I will not change the HA behavior.
I close the issue because it is not a Versatile Thermostat issue. Reopen if I'm wrong.

I won't reopen on a whim, but I can continue ;)
So my problem is that some thermostats that I've got by default on "heat" just go full open ignoring any temp set point - for those valid option is auto. Other thermostats work in heat accepting setpoints, but in auto they readjusts them self to internal pre programmed schedule - which goes unnoticed by VHT.
From my perspective after reading the documentation of HA there are two options:

during the creation of thermostat, an integration provides a list of allowed HVAC modes (yes those have to confirm to predefined enums) and one could just limit the list to: OFF/AUTO or OFF/HEAT
there is an integration "better thermostat" that allows to chose, what should be a default mode operation for heating: HEAT or AUTO. However that integration has it's own problems, not limited to memory leaks, infinite loops on reconfiguration, no central boiler control, no ability to control switch type heater etc.


Define in6_ifreq only if linux/ipv6.h is not included

When building on termux:
linux_pytun.c: #include <arpa/inet.h>
then arpa/inet.h: #include <netinet/in.h>
then netinet/in.h: #include <linux/ipv6.h>
which resulted in the redefinition of in6_freq.
This workaround detects if linux/ipv6.h is already included then define in6_ifreq.😊

Thanks for your PR!

🤔 In the newer Linux version the ipv6.h marker is _IPV6_H.
Would be it a good idea to omit in6_ifreq and include linux/ipv6.h? (but this imports lots of unused symbols)

In general, since you mentioned this is available only for newer Linux versions, sounds like its a worse choice since it would compile for less people

In general, since you mentioned this is available only for newer Linux versions, sounds like its a worse choice since it would compile for less people

I’m sorry for the misunderstood sentence….
In Termux I cannot compile because linux/ipv6.h is included so there are multiple definitions of in6_ifreq, one in linux/ipv6.h, one in linux_pytun.c.
This workaround detects if linux/ipv6.h was included by #define _UAPI_IPV6_H in linux/ipv6.h. However, I found it changed to #define _IPV6_H in the newer Linux version, so in6_ifreq will still be redefined. If we directly include linux/ipv6.h, these won’t happen 😉

sounds better. feel free to create a new PR for that :)


Missing some files in Baml project

Severity	Code	Description	Project	File	Line	Suppression State
Error	CS2001	Source file 'F:\Projects\OpenSource\obfuscar\obfuscar\Baml\..\ThirdParty\ILSpy\ILSpy.BamlDecompiler\BAML\BamlRecords.cs' could not be found.	Baml (netstandard1.1)	F:\Projects\OpenSource\obfuscar\obfuscar\Baml\CSC	1	Active
Error	CS2001	Source file 'F:\Projects\OpenSource\obfuscar\obfuscar\Baml\..\ThirdParty\ILSpy\ILSpy.BamlDecompiler\BAML\BamlReader.cs' could not be found.	Baml (netstandard1.1)	F:\Projects\OpenSource\obfuscar\obfuscar\Baml\CSC	1	Active
Error	CS2001	Source file 'F:\Projects\OpenSource\obfuscar\obfuscar\Baml\..\ThirdParty\ILSpy\ILSpy.BamlDecompiler\BAML\BamlDocument.cs' could not be found.	Baml (netstandard1.1)	F:\Projects\OpenSource\obfuscar\obfuscar\Baml\CSC	1	Active
Error	CS2001	Source file 'F:\Projects\OpenSource\obfuscar\obfuscar\Baml\..\ThirdParty\ILSpy\ILSpy.BamlDecompiler\BAML\BamlWriter.cs' could not be found.	Baml (netstandard1.1)	F:\Projects\OpenSource\obfuscar\obfuscar\Baml\CSC	1	Active

Duplicate to #96


Fix IOS build process

The container build for IOS seems to fail. Fix this and investigate if the compilers are wrong.
ETA: 2h

The compilers are a bit odd and flags are being replaced foolishly. But it did not work in any other way. Keep it until it breaks.


Allow channel for stereo split

Allow channel to be built in the XML in order to split L/R stereo audio into separate videos.
QA Steps:
Run rspec ./spec/video_format_editing_spec.rb

That's the correct format. I was using an incorrect format originally that somehow worked in my vagrant machine but threw an error when the spec was ran anywhere else.

Specs pass.


[BUG] Security Analytics Detector creation overwrites Data-Stream template Index-Patterns in OpenSearch

What is the bug?
When creating a Security Analytics Detector with a data-stream "Data source" in OpenSearch, the index-patterns defined in the data-stream template is being overwritten. The creation of the detector modifies the existing index pattern to match the specific data stream used by the detector and adds a new link to a .opensearch-sap-alias-mappings-component... component_template. This action renders the template unusable for other data streams that were designed to match the original index pattern.
How can one reproduce the bug?
Steps to reproduce the behavior:

Define a data-stream template with an index pattern, e.g., "ss4o_logs-dns-bind9.querylog*".
Define a data stream named "ss4o_logs-dns_bind9.querylog-prod" to use the aforementioned template.
Define another data stream named "ss4o_logs-dns_bind9.querylog-test" that uses the same template.
Create a Security Analytics Detector for the production environment, with "Data Source" set to "ss4o_logs-dns_bind9.querylog-prod".
Inspect the data-stream template after the detector creation.
Notice that the original index-pattern "ss4o_logs-dns-bind9.querylog*" has been changed to "ss4o_logs-dns-bind9.querylog-prod*".
Additionally, identify that the template has acquired a new link to a component template named ".opensearch-sap-alias-mappings-component-ss4o_logs-dns_bind9.querylog-prod".
Realize that the data-stream template is no longer valid for "ss4o_logs-dns_bind9.querylog-test" due to the index pattern modification.

What is the expected behavior?
Creating a Security Analytics Detector should link a new component template to the template used by the "Data Source" without modifying the existing index pattern in the template. This would allow the template to remain valid for all data streams matching the original index pattern design.
Actual Result: The creation of the Security Analytics Detector causes an alteration in the data-stream template index pattern, from "ss4o_logs-dns-bind9.querylog*" in our example to a more specific pattern matching the production data stream. In effect, this breaks the template for any other data stream that was supposed to use the same pattern, such as "ss4o_logs-dns_bind9.querylog-test".
What is your host/environment?

OS: Red Hat Enterprise Linux release 9.4
Version Opensearch 2.14
Plugins

 # /usr/share/opensearch/bin/opensearch-plugin list
opensearch-alerting
opensearch-anomaly-detection
opensearch-asynchronous-search
opensearch-cross-cluster-replication
opensearch-custom-codecs
opensearch-flow-framework
opensearch-geospatial
opensearch-index-management
opensearch-job-scheduler
opensearch-knn
opensearch-ml
opensearch-neural-search
opensearch-notifications
opensearch-notifications-core
opensearch-observability
opensearch-performance-analyzer
opensearch-reports-scheduler
opensearch-security
opensearch-security-analytics
opensearch-skills
opensearch-sql

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
This issue occurs only if the .opensearch-sap-alias-mappings-component* component template doesn't exist, and its reference is not already included in the data-stream template. The system should ensure that any new configurations added to cater to the detector do not negatively impact the underlying template structures and their reusability.
The code responsible for this behavior is probably:

https://github.com/opensearch-project/security-analytics/blob/main/src/main/java/org/opensearch/securityanalytics/mapper/IndexTemplateManager.java#L184
https://github.com/opensearch-project/security-analytics/blob/main/src/main/java/org/opensearch/securityanalytics/mapper/IndexTemplateManager.java#L219

[Catch All Triage - 1, 2, 3]

Any thoughts on how to mitigate this?  We ran into same issue just not with data stream indexes templates.  We had simple index template, created a detector, and the detector then changed the index pattern within the template.  That rendered the new indexes to not be found/created using the right template and the effects domino'ed from there.