Make getContinuousReportActionChain consider any optimistic action

Details
When generating reportActions optimistically, they are not given a previousReportActionID. This means that our rendering logic for generating the continuous chain of ReportActions in ReportActionsUtils.getContinuousReportActionChain has to have special handling of optimistic actions.
Currently, the way it works is we find the last non-pending action, then iterate from there to find the longest continuous chain, including any pending add actions.
This is complicated but works ok, except for that it only accounts for the optimistic ADD RBR pendingAction, and not for DELETE or UPDATE pendingActions.
This is meant to be a temporary solution - long term we should be optimistically generating previousReportActionID for any optimistic reportAction. We will create a follow-up task for that soon.
Fixed Issues
$ https://github.com/Expensify/App/issues/38844
Tests

Open a report
Go offline
Send a message
Delete the message
verify that the action appears as strikethrough, and the other actions on the report are unaffected

[x] Verify that no errors appear in the JS console

Offline tests
same as tests.
QA Steps

Follow the testing steps (simplified version of the original bug report)
Retest the original bug, verify that it's fixed.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/47436092/74105536-7638-496c-ac51-e87f843d84ae

MacOS: Desktop

discussed at length in slack here. Going to merge and CP to staging

🚀 Cherry-picked to staging by https://github.com/roryabraham in version: 1.4.56-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Cherry-picked to staging by https://github.com/roryabraham in version: 1.4.56-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.4.56-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[SourceKit] Allow generation of cursor info for declarations from solutions that haven’t aren’t applied to the AST

This has two benefits:

We can now report ambiguous variable types
We are more robust in the generation of results for declarations inside closures. If the closure has an error, we won’t apply the solution to the AST and thus any cursor info that tried to get types out of the AST would fail.

rdar://123845208

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test Linux


Fixes static placeholder display on admin

BEFORE:

AFTER:

/cc @mkoistinen @yakky

Coverage increased (+0.003%) to 88.515% when pulling 5a13ca3557228d04e0a4105e7589c56281590ed1 on czpython:fixes/3.3.x/staticplaceholder-display into d063c45bd0253aa6cc1d1bb5dfea1bd868d89528 on divio:develop.

LGTM

👍


Fix not to create csv if format is empty

Signed-off-by: Jiyeong Seok<EMAIL_ADDRESS>Description

Fix not to create csv if format is empty
Type of change

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

Dear @dd-jy , I will merge this PR. :smiley:


pull request for e2e test at 11/11/2016 13:15:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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.


fix defaut max_sync_log_lag

The default max_sync_log_lag is mistakenly set to default max_read_log_lag.

Coverage decreased (-0.005%) to 67.281% when pulling 826c5b4134b95a4f244554302577a6c71043546c on fix_default_max_sync_log_lag into 860c8de52ced88b4440f10bc0b07cd8a9c7441bd on master.


[docs] add docs of how to integrate with an existing react app

it would be great to have a step by step tutorial on docs of how to setup fbt in an existing React app project
which packages should we install?
how should we modify our babel config?
should we modify our webpack config?
how to translate new generated hash keys?

I am kind of confused how can I use it, I checked the example app and didn't find any packages called fbt in package.json. Also all the references I can find on npm is this and it doesn't say anything

I am kind of confused how can I use it, I checked the example app and didn't find any packages called fbt in package.json. Also all the references I can find on npm is this and it doesn't say anything

I have no idea what fbt-cli is, but the packages used in the demo-app are actually using the yarn workspaces.  You can look at my example npm-only app for the relevant npm modules:
https://github.com/jrwats/fbt-npm-app
We should be landing and finalizing version 1.0.0 for all of these modules this week.

@jrwats thank you so much for your effort, I checked the new version again. It's still not user friendly I have been trying to get it to work in my existing project without any success. I will continue to tinker and document the steps in a format an average developer can understand.

A +1 on this issue -- even after guessing at a bunch of steps (e.g. manually installing babel-node and related dependencies globally) by reading the sample app's package.json, we were unable to get fbt working in our existing application. fbt's promised feature set is super great, but right now there is zero documentation for consumers of the library to get started (versus developers building the library from source). :(

It took me an hour to add fbt to my react-native project.
@aldeka You don't need babel-node to run scripts from an npm package!

You can find diff for create-react-app project here...

It took me an hour to add fbt to my react-native project.
@aldeka You don't need babel-node to run scripts from an npm package!

can u upload a diff or sample  for
react-native app works with fbt ???

@Zbish  there is no big difference, you can use this diff

thanks,  will try
and it will work on react-native init app ??

Migrate to the latest version https://github.com/retyui/fbt-getting-started/commit/6d55f47a57f34e93edc53b410ec64f97916e30f1

thanks, will try
and it will work on react-native init app ??

@Zbish Did you ever get this working? Wondering if I should attempt to integrate into a React Native app or not. I can't seem to find anyone using it successfully

@vhesener, @Zbish, @retyui: here's some breaking news we haven't announced yet...
We released the Android NPM package and demo-app.  Take a look and let @dalmendray know if you hit any issues :)

I understand there's a demo app that proves this can work, but I think there's a slight disconnect in what we were expecting. I'm willing to help you get there, but I just need to know what steps need to be taken to bring this functionality into an existing React (Native) app.
There seems to be some missing magic to get from here to here. For example, where are the install steps and where are the import statements? Package documentation usually includes a typical cadence in the beginning pages in the docs. Here's a pseudo outline of what I would like to see before I know if I can use a any given package:

Installation

Add dependencies: yarn add fbt or maybe even yarn add https://github.com/facebookincubator/fbt.
add plugin to babel
if using RN version < 0.60, do some manual linking stuff

Getting Started

import { fbt } from  'fbt';
Add some Provider to the app
Use the <fbt /> component

I understand it's probably not too much effort to clone the demo app and figure out how to massage this to a working state. I'm not being lazy here, but I just want to know this use case is even supported and if you intend to add this type of documentation for future tinkerers. Candidly, if no one is using this package in a traditional way, I am reluctant to be on the fringe. There's strength in numbers with a larger community sharing the same problem sets.
Let me know your thoughts and if we can help. I think many of us would be willing to help you polish the usability if you were to share your vision regarding app integrations.


init() missing 2 required keyword-only arguments: 'alive' and 'up_to_date' after update to 2.4.5

[ ] I have read the Readme, including the Advanced section regarding debugging.

Describe the bug
Loss of shutters since the last version 2.4.5
Logger: custom_components.tahoma
Source: custom_components/tahoma/__init__.py:112
Integration: Somfy TaHoma (documentation, issues)
First occurred: 15:13:41 (1 occurrences)
Last logged: 15:13:41

__init__() missing 2 required keyword-only arguments: 'alive' and 'up_to_date'
Traceback (most recent call last):
  File "/config/custom_components/tahoma/__init__.py", line 112, in async_setup_entry
    gateways = await client.get_gateways()
  File "/usr/local/lib/python3.8/site-packages/backoff/_async.py", line 133, in retry
    ret = await target(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/pyhoma/client.py", line 135, in get_gateways
    gateways = [Gateway(**g) for g in humps.decamelize(response)]
  File "/usr/local/lib/python3.8/site-packages/pyhoma/client.py", line 135, in <listcomp>
    gateways = [Gateway(**g) for g in humps.decamelize(response)]
TypeError: __init__() missing 2 required keyword-only arguments: 'alive' and 'up_to_date'

Environment (please complete the following information):

hub: Somfy TaHoma
ha-tahoma version: 2.4.5
Home Assistant version: 2021.1.5
Platform: cover

Device: (if your problem is related to a specific device)

Model: [e.g. PositionableScreen, can be gathered from device list in HA]
Type: [e.g. io:DimmableLightIOComponent, can be gathered from device page in HA looking at Firmware] /config/devices/dashboard]

Additional context
Add any other context about the problem here.

Thanks for reporting @dckiller51. For now, I would advise to downgrade to v2.4.3.
Which hub are you using? (Somfy TaHoma, Connexoon, CozyTouch, etc.).

Could you give https://github.com/iMicknl/ha-tahoma/archive/fix/fix_gateway_params_issues.zip a try? Extract this file and place custom_components/tahoma in your custom_components folder.

Thanks for reporting @dckiller51. For now, I would advise to downgrade to v2.4.3.
Which hub are you using? (Somfy TaHoma, Connexoon, CozyTouch, etc.).
Somfy Tahoma
Yes downgrade to v2.4.3 works. I'm testing tonight, I'm not at home to send the files

Thanks @dckiller51. Do you perhaps have the Somfy TaHoma first generation? It is weird how their API is not standardized on all gateways...

Thanks @dckiller51. Do you perhaps have the Somfy TaHoma first generation? It is weird how their API is not standardized on all gateways...

This is the Tahoma V2 version 2020.6.4-1 5

That is weird, I have the same version (2020.6.4-15) and for me this error doesn't occur...

That is weird, I have the same version (2020.6.4-15) and for me this error doesn't occur...

I don't have the hub in HA device. The information, I recovered them in the official application.

@dckiller51 I know, that is the error you are facing in this issue :-). The fixed version should add your hub as an entity in Home Assistant.

Pourriez-vous essayer https://github.com/iMicknl/ha-tahoma/archive/fix/fix_gateway_params_issues.zip ? Extrayez ce fichier et placez-le custom_components/tahomadans votre custom_componentsdossier.

This version works. I do have two additional devices. (Tahoma V2 hub and Home Protect System hub). Thank you

Thanks! Your Home Protect System hub is also a gateway, that is what caused this issue. We will add this to the next version.

@iMicknl
Hello, Home Assistant 2021.2.0 updated suite. Mistake
Setup failed for tahoma: Requirements for tahoma not found<EMAIL_ADDRESS>

@dckiller51, sorry that branch has been removed already. You can download the latest version (v2.4.6) via HACS or via GitHub, where this fix is included.

@iMicknl
thank you very much for your speed


As a user, I want to subscribe to a blog post feed so I can read CDH updates in the feed reader of my choosing.

Notes for testing

Provides both RSS and Atom feeds.  Should be linked at the bottom of blog index and blog detail pages and also included in the header for autodiscovery.
Easiest way to test is probably to install a browser RSS tool like chrome rss extension which will show the auto-discover url and also give you a nice preview of the content.

I installed the Chrome extension and used both subscribe buttons worked perfectly with it.


docs: fix broken link

@renomureza Please can you sign the CLA check and then we can merge this PR.

@thegreatercurve I just finished signing it.


adds preliminary typings for Typescript

Initial contribution to Issue #26.
Provides:

(Xstream-only) typing for Collection and pluck
inline documentation for both functions

Still to do:

typing for non-Xstream libraries (can it simply be done by typing makeCollection? if not, can typing be factorized as easy as the code?)
typing for gather and merge

Sorry for the delay in response.
This looks really good! I like the union type for adding sources together. I haven't really used generics in TS yet so this is informative.
I can merge this now if you like or I can wait until we've amassed a full definition.

I would merge it now, cause it's doing the job for me for now, so I can hardly promise that I will have time to improve it much in a close future... Sorry :-(

@aravantv great, I'll merge it now. Thanks for contributing this! It's a great start on a definition. I especially like the documentation.


ValueError on slurm batch system

A user of my pipeline with a Slurm cluster reported this traceback after upgrading his toil to 3.11.0:
10-19 09:39:58 luigi-interface ERROR [pid 40083] Worker Worker(salt=945881764, workers=16, host=holyhoekstra01.rc.fas.harvard.edu, username=lassance, pid=36709) failed    ToilTask: GenerateHints for rheMac3 using batchSystem Slurm
Traceback (most recent call last):
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/luigi/worker.py", line 194, in run
    new_deps = self._run_get_new_deps()
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/luigi/worker.py", line 131, in _run_get_new_deps
    task_gen = self.task.run()
  File "/n/home01/lassance/Comparative-Annotation-Toolkit/cat/__init__.py", line 1023, in run
    hints_db(self.hints_args, toil_options)
  File "/n/home01/lassance/Comparative-Annotation-Toolkit/cat/hints_db.py", line 81, in hints_db
    combined_hints = t.start(job)
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/common.py", line 727, in start
    return self._runMainLoop(rootJobGraph)
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/common.py", line 997, in _runMainLoop
    jobCache=self._jobCache).run()
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/leader.py", line 183, in run
    self.innerLoop()
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/leader.py", line 487, in innerLoop
    self.checkForDeadlocks()
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/leader.py", line 504, in checkForDeadlocks
    totalRunningJobs = len(self.batchSystem.getRunningBatchJobIDs())
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/batchSystems/abstractGridEngineBatchSystem.py", line 379, in getRunningBatchJobIDs
    batchIds = self.worker.getRunningJobIDs()
  File "/n/home01/lassance/.conda/envs/ENV_PROGRESSIVECACTUS/lib/python2.7/site-packages/toil/batchSystems/slurm.py", line 53, in getRunningJobIDs
    slurm_jobid, state, elapsed_time = values
ValueError: too many values to unpack

https://github.com/ComparativeGenomicsToolkit/Comparative-Annotation-Toolkit/issues/77#issuecomment-337684302

more on the context of this error: I was still running toil 3.6.0 before updating to 3.11.0.
I ran some tests on various releases (3.11.0, 3.10, 3.9.0, 3.8.0, 3.7.0), which all gave me the same error.
Went back to 3.6.0, and it seems to be working fine.

Trying to revisit this error. I realize that some problems are caused by the fact that our local deployment of Slurm relies on a modified version of the squeue command. Basically, some options are to be misinterpreted.
I made the following changes at line 48:
original:
lines = subprocess.check_output(['squeue', '-h', '--format', '%i %t %M']).split('\n')

corrected:
lines = subprocess.check_output(['squeue', '--noheader', '--format="%i %t %M"']).split('\n')

Can someone double-check this?

Thank you for tracking this down @lassancejm. The = sign after format looks like the correct option. Using '--noheader' instead of 'h' doesn't change anything. Can you submit a PR?


Feature Request: Temporarily Muted Words

Is your feature request related to a problem? Please describe.
When I am muting a keyword because it is the Discourse Of The Day and I am not interested, I will often want the mute to go away after a certain period of time (generally one day). This is not possible; I need to actually remember to remove the keyword mute, which is not ideal.
Describe the solution you'd like
An option to have the mute exist for a certain period of time, either by a simple one-option checkbox of "delete this mute after 24 hours", a dropdown with a set of options, or an input box that takes some sort of explicit structured timeframe (I do not like this option).
Describe alternatives you've considered
I have considered just remembering to go back and change things, but that seems unlikely to work. Also, having this as an explicitly supported piece of functionality makes it not only easier for people to do but also more endorsed in a sort of structural-social way, and I think this would be good for the vibes to some small degree.
Additional context
This is the screen in question:

I mentioned this to @pfrazee on bsky and suggested that I might be motivated to try to implement this. My understanding is that the relevant UI code for this is in MutedWords.tsx . This would also presumably require in turn changing useUpsertMutedWordsMutation to incorporate the timer, adding that to the schema for the muted words, and having something that intermittently checks and cleans up the muted words—I am not sure where the best place to do that is.
Things that come to mind for testing it: how it looks at different font levels, in different localizations, different browsers, and different mobile configurations.

any update on this?

This will be an option when #4801 is merged, which can happen after the backend changes in  https://github.com/bluesky-social/atproto/pull/2276 are landed.


Change villager trade hooks to use factory

Put as part of the 1.6 update since this will be a breaking change


feat(): Added fallthought option

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
N/A
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
This PR, Added fallthrough option

Thank you!


[Bug] Alignment of card changes as button rows are hidden

It seems that the title and temperature bar are aligned differently to the temperature text and buttons that appear within the arc of the temperature bar. As a result hiding all buttons in a row (i.e. all HVAC modes) changes the alignment of the text within the temperature arc. See below:

This problem is made even worse if you use a non-Versatile thermostat, which results in the preset buttons being hidden (see below). I also imagine this causes the row of sensor icons (motion, window, etc) to clip the temperature arc.

Edit, added photo of thermostat with no buttons to show the text shifted all the way up

That is possible but having no havc_mode is weird. This should not possible. You should always be able to turn-off and turn-on with heat or cool.

ok. Turning off an HVAC controlled by VTherm without turning off on VTherm it self is strange but wht not it is working like that.
I will try to have a look but I'm really not a UI expert... Will try to do my best.

Unless I missed something in the setup, the VTherm thermostats controlling my TRVs don't have an off hvac option. Is there a way of making a virtual off action?

The hvac_,modes of the VTherm in over_climate mode is the ones of the underlying.


Integrated terminal toggle shortcut works only when caps lock is ON

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:34:22.110Z)
OS Version: Linux x64 4.12.0-2-amd64
Extensions:
vscode-icons|rob|7.19.0

Steps to Reproduce:

Open VSCode
Press ^` shortcut (ctrl+`) (default shortcut for workbench.action.terminal.toggleTerminal) while caps lock is off. Nothing happens.
Press same shortcut while caps lock is on. Integrated terminal toggles.

Reproduces without extensions: Yes

@iMasoud Have you tried our troubleshooting guide? https://github.com/Microsoft/vscode/wiki/Keybinding-Issues#troubleshoot-linux-keybindings


Support for key material in PEM and PKCS12 formats

(rewritten to focus on requirements)
Kafka itself supports JKS, PKCS12, and PEM key formats.  In the Kubernetes world, PEM is common place.
In order for Kroxylicious to integrate seamlessly into the Kafka/Kubernetes worlds it should support those same key material formats when configuring TLS.  This applies to both the upstream side and the downstream side.   It would be a pain for the user if they had to configure Kroxylicious differently from how they chose to configure their Kafka.
KeytoolCertificateGenerator currently generates key material in the JKS format.  That format is Java specific and is quite dated: Java has supported the PKCS12 standard since Java 8.
Thinking about writers of Kroxylicious integration tests using the Junit5 extension to test TLS functionality.
Upstream
Assuming I have configured a Kafka cluster to use a signed certificate using the @Tls, I desire a clean way to retrieve the client trust and/or have  a mechanism to convert it to other formats. I want to do this so I can verify that Kroxylicious accepts client key material in all the supported forms.
I can currently do this by usingcluster.getKafkaClientConfiguration() and retrieving the key material by accessing the client properties ssl.keystore.location, ssl.keystore.password and ssl.key.password, but I need to assume the keystore type JKS, which is currently private knowledge of the Junit5 extension.
Downstream
Assuming I am using KeytoolCertificateGenerator to generate a certificate/private key pair for Kroxylicious, I want a easy way to generate key material in all key formats (JKS, PKCS12, and PEM).   I want to do this so I can verify that Kroxylicious accepts server key material in those same formats.

I am imagining that the @Tls annotation will accept an optional keyStoreType annotation parameter that lets the user specify JKS, PKCS12, or PEM.
As Kafka uses a different set of configuration parameters when key material is expressed as PEMs (ssl.keystore.key , ssl.keystore.certificate.chain, ssl.key.password) rather than (ssl.keystore.location, ssl.keystore.password, and ssl.key.password), it might makes sense for KeytoolCertificateGenerator to expose methods that provide/populate the Kafka configuration rather than methods such as getTrustStoreLocation that it has right now.

Closed in favour of https://github.com/kroxylicious/kroxylicious-junit5-extension/issues/131


DIST folder standalone

I have deployed the CSET SQL instance and have been able to successfully connect to the SQL instance from a remote system. When logging into the SQL instance using CSET Web interface we are presented with the gold ribbon at the top showing local instance.
Following the instructions, I am unable to find and download the DIST folder as the link points to an old link. Could you please provide any insight into why we would be receiving the local instance error when it is connected to the DB. Also, there are no webfile to be placed in the IIS for launching and not sure if this is part of the issue in the instructions\downloaded files

The "Local Installation" gold ribbon is not displayed based on the physical location of the database.  Find the file LOCAL-INSTALLATION in the web root of the application.  Its existence causes the local install ribbon to display and tells CSET to use the user's Windows account for identity.  Removing or renaming that file will turn off the ribbon and allow CSET to display the login page to the user.


Add sub-daily capability to models

Description
Not an issue per se, but eventually we will want to go to the subdaily scale. RAVEN allows this relatively easily but some items in the chain will need to be updated to allow this. Not urgent but would be useful to the community.

This has been added for a while now to the templating it seems thanks to @huard  so it works by specifying the timestep variable in the model config. Will close this issue, but will add a notebook to show the implementation a bit later.


No events are being logged in Google Analytics

This is a separate issue from #19.
Events are not consistently being logged into Google Analytics.
I'm not completely sold that the issue is a result of using this package.
When I try to enter this url
https://ssl.google-analytics.com/collect?v=1&tid=<TRACKING_ID>&cid=5555&t=screenview
into the browser while looking at the google analytics' realtime view, I am unable to see the events being tracked. Some times events are being tracked however. I've tried switching out the TRACKING_ID with websites' tracking id and not an apps, and an event seems to be tracked. The only thing is that the event tracking is not consistent.
Possibilities for debugging:
* Try different query params
* Try different Tracking IDs
* Try different User Agents/devices

Has there been any progress made to figuring out why this is happening? I setup this component today and got a handful of hits then it went to nothing while I was trying to test. I confirmed the fetch call is returning a success, just nothing showing in GA.

Can you confirm you're using the latest version of the module? This looks like it might be an issue with caching, which was solved (to the best of my knowledge) in 0.1.0.

You should be seeing a &z= parameter in the requests coming from this module.

@lwansbrough I am on 0.1.0 and there is the &z parameter in the request.
I tried troubleshooting a bit by way of switching from a GET request to a POST request as it appears GA accepts either which didn't have any positive results.

So I tried to catch any errors on the fetch method used on Analytics.js and received the following error: TypeError: Network request failed Is this related to the issue?

I had this problem and found that the issue was the cd= parameter was not being included.
The cd parameter is required for screenview hits, so the example listed in the README (var screenView = new GAHits.ScreenView('GA Test', '1', 'com.example.app'); wouldn't work, right?
@the4dpatrick Your original request in #19 doesn't include a 5th parameter (cd) -- could that be your issue too?

@danleveille
I just tried another request https://ssl.google-analytics.com/collect?v=1&tid=<GA_ID>&cid=555adfasd5&t=screenview&cd=HighScore and still no luck.
So you've tested that events were being tracked by adding the cd= param to a request?

Yeah, when I remove the cd parameter, it doesn't record the event in GA. When I add it back in, it tracks properly.
Make sure you have all the required parameters. These parameters are all required: v, an, tid, cid, cd, t

It works! Good job!
https://ssl.google-analytics.com/collect?v=1&tid=<GA_ID>&cid=5555&t=screenview&cd=foo&an=name
Would you like to make the fixing PR? @danleveille

@the4dpatrick Nice! Glad to hear. I'm honestly a bit of a newbie with Github (and React), so I'll leave the PR to someone else. :stuck_out_tongue_closed_eyes:

That's a perfect reason to make your first Pull Request! I can help and write down each step with the end result being your first PR. I'd be more than happy with getting you started in open source :)
I'm away from my computer now, but I'll can get the steps to you later today.

On Dec 24, 2015, at 12:10 PM, Dan Leveille<EMAIL_ADDRESS>wrote:
@the4dpatrick Nice! Glad to hear. I'm honestly a bit of a newbie with Github (and React), so I'll leave the PR to someone else.
―
Reply to this email directly or view it on GitHub.

Nice work guys, if someone could make that PR that would be greatly appreciated!

@the4dpatrick Haha. Good point! I'll give it a shot. What should the actual fix should be?
In ScreenView.js:
constructor(appName, appVersion, appId, appInstallerId, screenName, experiment) {

I'm thinking the first two args should be appName (an) and screenName (cd), since those two args are the only ones required by GA. The others are optional.
Is that a good solution?

Yup that looks like the correct solution to me.

Looks good.
I realize my mistake was that I was not including the 4th arg for screenName. I'd be sure to update the README too @danleveille

Events are being sent again?

I haven’t used this repo for a while, but I can take a look at what I did. I’m don’t remember if by the end it worked or not.

On Thursday, Feb 02, 2017 at 5:15, Abe Herbert<EMAIL_ADDRESS>(mailto:notifications@github.com)> wrote:
Events are being sent again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/lwansbrough/react-native-google-analytics/issues/21#issuecomment-276953990), or mute the thread (https://github.com/notifications/unsubscribe-auth/ADjI6yIxllGYtMoPkUgZ4rohlDpBgv9mks5rYdb9gaJpZM4GwhKl).


Newbie100%: 502 error and [YOUR_SERVER_FQDN]?

Background
I set up Ubuntu 18.04.1 LTS on my cloud server and it gives me a public ip(lets say <IP_ADDRESS>) and an intranet IP.
My issue
I dont have a doman name so I set my FDQN with my public ip.
When i access my public ip, it returns 502 bad gateway error with colored favicon of huggin.
I tried troubleshoot with 'sudo bundle exec rake production:check' and it returns green ok everything is fine.
What should I do now? THANKS!

When I click @, you four guys pop up... sorry to bother you guys like this but I really like to know how to solve the issue... apologize again for any inconveniences.....
@brynary @amoeba @necrodome @raucao

What I tried: ... too many bad returns...

Troubleshooting - sudo bundle exec rake production:check
Return: [  OK  ] Everything is fine

Troubleshooting - Nginx error log /var/log/nginx/huginn_error.log
Return: connect() to unix:/home/huginn/huginn/tmp/sockets/unicorn.socket failed
BUT i did follow the guide to set the Procfile as below:

Troubleshooting - Unicorn log /home/huginn/huginn/log/unicorn.log
2.1 my unicorn.rb as below:

2.2 Gem dependencies not all installed, wonder why as below

Nginx is trying to connect to the socket but can't.
ls -alF /home/huginn/huginn/tmp/sockets/unicorn.socket
What comes back?

Nginx is trying to connect to the socket but can't.
ls -alF /home/huginn/huginn/tmp/sockets/unicorn.socket
What comes back?

ls: cannot access '/home/huginn/huginn/tmp/sockets/unicorn.socket': No such file or directory
@virtadpt no such file... and thank you for pick up my issue!

Yep, that's your problem.  No unicorn.socket, nothing for Nginx to connect to.  What happens when you try to start up the Unicorn server?
rake production:export


Strange behavior trying to upgrade iree-base-compiler Python package

What happened?
When old iree-base-compiler and iree-base-runtime packages are installed [into a venv] and one attempts to upgrade both, the iree-base-compiler package appears to not be upgraded:
pip install -f https://iree.dev/pip-release-links.html --upgrade --pre \
  iree-base-compiler \
  iree-base-runtime

Requirement already satisfied: iree-base-compiler in /groups/aig_sharks/actions-runner-nod-gpu5/_work/_tool/Python/3.11.9/x64/lib/python3.11/site-packages (3.0.0rc20241114)
Collecting iree-base-compiler
  Downloading https://github.com/iree-org/iree/releases/download/iree-3.1.0rc20241126/iree_base_compiler-3.1.0rc20241126-cp311-cp311-manylinux_2_27_x86_64.manylinux_2_28_x86_64.whl (71.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 71.3/71.3 MB 49.5 MB/s eta 0:00:00
Requirement already satisfied: iree-base-runtime in /groups/aig_sharks/actions-runner-nod-gpu5/_work/_tool/Python/3.11.9/x64/lib/python3.11/site-packages (3.1.0rc20241125)
Collecting iree-base-runtime
  Downloading https://github.com/iree-org/iree/releases/download/iree-3.1.0rc20241126/iree_base_runtime-3.1.0rc20241126-cp311-cp311-manylinux_2_28_x86_64.whl (8.1 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 8.1/8.1 MB 51.4 MB/s eta 0:00:00
Installing collected packages: iree-base-runtime, iree-base-compiler
  Attempting uninstall: iree-base-runtime
    Found existing installation: iree-base-runtime 3.1.0rc20241125
    Uninstalling iree-base-runtime-3.1.0rc20241125:
      Successfully uninstalled iree-base-runtime-3.1.0rc20241125
  Attempting uninstall: iree-base-compiler
    Found existing installation: iree-base-compiler 3.0.0rc20241114
    Uninstalling iree-base-compiler-3.0.0rc20241114:
      Successfully uninstalled iree-base-compiler-3.0.0rc20241114
Successfully installed iree-base-compiler-3.0.0rc20241114 iree-base-runtime-3.1.0rc20241126

https://github.com/nod-ai/shark-ai/actions/runs/12021435366/job/33546714200#step:5:203
Steps to reproduce your issue

Create a venv
Install old packages
Try to upgrade to new packages
See that iree-base-compiler is still using an older version

We should test with stable packages, nightly packages, and dev packages. The upgrade path for stable -> stable should work, while upgrading/downgrading with nightly or dev releases can require extra steps (like deleting the venv).
What component(s) does this issue relate to?
Python
Version information
Observed with packages newer than 3.0.0. Should check if 2.9.0 had this issue too, since https://github.com/iree-org/iree/pull/19156 and other packaging-related changes landed more recently.
Additional context
It's possible that https://github.com/iree-org/iree/pull/19156 broke the upgrade path somehow in the process of fixing the version reporting (https://github.com/iree-org/iree/issues/19155).

Have you any other packages installed in your venv that depend on one of the IREE packages (e.g. iree-turbine?) and causes trouble with the dependency resolution?

The workflow runs in https://github.com/nod-ai/shark-ai do have other packages installed, both from a pip cache and from prior steps.
I saw similar, possibly related, behavior on Friday with just iree-base-compiler and iree-base-runtime installed (Discord conversation here). I didn't grab full logs or repro steps though, just this:
Running command to install package:
  pip install --force --upgrade --no-index --force-reinstall --ignore-installed --no-cache-dir --no-deps /home/.iree/bisect/11915893210/iree_base_compiler-3.1.0.dev0+5b0740c97a33edce29e753b14b9ff04789afcc53-cp311-cp311-manylinux_2_27_x86_64.manylinux_2_28_x86_64.whl

Processing /home/.iree/bisect/11915893210/iree_base_compiler-3.1.0.dev0+5b0740c97a33edce29e753b14b9ff04789afcc53-cp311-cp311-manylinux_2_27_x86_64.manylinux_2_28_x86_64.whl
Installing collected packages: iree-base-compiler
Successfully installed iree-base-compiler-3.0.0.dev0+4c0fd906bfd1e8888654b3e8a967ef977bb84403

(attempted to install 3.1.0.dev0+5b0740c97a33edce29e753b14b9ff04789afcc53 from a local .whl file, actually installed 3.0.0.dev0+4c0fd906bfd1e8888654b3e8a967ef977bb84403)


python3Packages.hkavr: init at 0.0.5

Description of changes
Library for interacting with Harman Kardon AVR controllers
https://github.com/Devqon/hkavr
This is a Home Assistant dependency.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 170314 run on x86_64-linux 1

2 packages built:

python310Packages.hkavr
python39Packages.hkavr

Result of nixpkgs-review pr 170314 run on aarch64-linux 1

2 packages built:

python310Packages.hkavr
python39Packages.hkavr


pls help me, kernel for  XIAOMI3TD

I want a custom kernel for xiaomi 3 pisces.
kali nethunter   HID cannot use

If you say "I want", nobody will ever help you.
If you say "I am searching for" or "I would like to have", somebody might consider helping you.

DEVICE_CODENAME: pisces (see http://en.miui.com/thread-252582-1-1.html )
SOC: Nvidia Tegra 4 AP40 (see https://www.devicespecifications.com/en/model/69422983 )
KERNEL_SOURCE: https://github.com/MiCode/Xiaomi_Kernel_OpenSource/tree/pisces-kk-oss =>  or https://android.googlesource.com/kernel/tegra/

Check the Makefile in the kernel src to get the kernel version of your device.
Search for a suitable patch and apply it.
Compile your kernel again with make.


Prodigy Optimizer.

Hello!
First of all, I would like to thank the devs for their effort in maintaining this repository.
I would like to suggest an update.
Update the optimizer to Prodigy.
Prodigy is a new optimizer that converges much faster and easier than Adam. It is the direct evolution of Dadptation.
Prodigy has been widely used in Stable Diffusion now for delivering faster results and allowing the choice of LR to be automatic.
I implemented PRODIGY in my local environment and it worked very well.
It's not very difficult to implement and modify.
In this repository, you can find the paper and the requirements to use Prodigy.
From optimize args I used the standards we are currently using in SD.
"decouple=True" "weight_decay=0.01" "d_coef=2" "use_bias_correction=True" "safeguard_warmup=True"
I really think this can speed up model training and deliver better results. It would also be recommended to trigger an early stop system to lock the training at the moment it starts to want to overfit.
I also implemented in my local env a wandb integration, but for that i removed the tensorboard. So i dont know if it valid for here, but i like more Wandb since we can acess it from anywhere.
https://github.com/konstmish/prodigy
So that's it.
Any doubts you can contact me!
Have a good day!

It would also be recommended to trigger an early stop system to lock the training at the moment it starts to want to overfit.

How would you do the detection of overfitting? Any tips?


[Feature Request] Ability to sort plugin results

There are times where I really wish that I could manually re-order plugin results. For example, when I search for weather, there are about 2 dozen results from File Finder before I actually see results from the Weather Plugin. Similar results have been observed using the IP plugin. This is super frustrating and basically diminishes the usefulness of the plugins.
Expected Behavior

Weather results will be displayed first

Actual Behavior

Weather results are buried by dozens of File Finder results

Steps to reproduce the behavior

Search

Versions

Zazu:

App Environment: production
App Version: 0.4.0
Electron Version: 1.3.13
Node Version: 6.5.0
Chrome Version: 52.0.2743.82

OS: Windows 10 Enterprise, Build 14393.693
GIT: N/A

Log:
https://gist.github.com/nalexander50/305743bcd99d76d0e2ca82dc794b6ffe

Just for giggles, I tried having Weather as the very first plugin in my plugins array in hopes that the search results would key off the array ordering. Unfortunately, I am still seeing the same behavior where Weather is placed at the bottom.

Sorting is fun. Basically it works like this:

Prefix Script (with id)
Root Script (with id)
Prefix Script
Root Script

There is also logic that if something with an id is clicked on, it's ranked higher next time.
In this case you are looking at the file finder, which is a Root Script with an id and wanting it higher than the Weather plugin which is a prefix script without an id.
The fastest way to fix this is to add an id to the weather plugin, since I didn't when I created it but should have.
It could be interesting for users to specify they always want to rank some plugins higher or lower than others, but I'm unsure what the api would look like.
If you want to try and add an id to the weather plugin you can go here

I added the id to the weather plugin here.


Add Jerry API to check whether a given area in memory is a snapshot

JerryScript-DCO-1.0-Signed-off-by: François Baldassari<EMAIL_ADDRESS>

In jerry_snapshot_header_t the version is an uint32_t value. Furthermore jerry_is_snapshot is too generic, since it is not possible to tell from a random data whether it is a snapshot. I think you wish to check whether the snapshot can be loaded. In other words, you know the area contains a snapshot, but the system may not be able to load it.

Any plans to update this PR?

I am closing this PR, please reopen it if you work on it again.


Missing getter definitions

Hi,
when running the new 0.7.0 I get the following errors. Probably, that is connected to the statical code analysis at pub.dev.

Thank you :)

Oh ok. Looks like some deprecation issues with the analyzer. I'll check.

Done in 0.7.1

Seems to work now :)


create missing DNS entry. fix https://github.com/kubernetes-incubator…

…/external-dns/issues/964

@yverbin please sign the CLA and format your go code, then we will have a look 🙂.

@yverbin please sign the CLA and format your go code, then we will have a look 🙂.
I forgot to specify my email in repo config (facepalm)...


Master Issue: https://github.com/apache/pulsar/issues/12105
Motivation
Because the PIP is not completely implemented, we should not publish the interface changes in 2.10.
Modifications

Revert the core PR for PIP 97.

Verifying this change
This change is trivial, because the original change was completely backwards compatible.
Documentation

[x] no-need-doc

@codelipenghui - since we need to do a second RC for 2.10.0, I realized I should revert PIP 97 while I have the chance. I plan to complete the work soon, but it didn't make 2.10.

I am looking into why non of the tests ran.

@codelipenghui - I had opened this PR with the intention of making sure tests pass. Apparently, we don't have tests configured to run on these PRs--I thought we did. I am going to merge this now since I believe it is the correct change and will therefore get included in the next 2.10 RC.

Ok, thanks for the update, any reason for reverting the change?

@codelipenghui - I reverted the commit that was responsible for updating the interfaces because none of the other work has been completed. It'd be a bit misleading for third parties to implement the interface since it is still synchronous. I plan to implement PIP 97 for Pulsar 2.11.


Check bid key exists before logging

Would it be possible for the caller sends a block hash/slot combo that doesn't exist in the bids map, and relay also responded nil for result? In this case, I think originalResp.relays will result in a panic

Codecov Report

Merging #232 (e264ae1) into main (e63e526) will decrease coverage by 1.63%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##             main     #232      +/-   ##
==========================================
- Coverage   84.14%   82.50%   -1.64%
==========================================
  Files           5        5
  Lines         599      606       +7
==========================================
- Hits          504      500       -4
- Misses         71       80       +9
- Partials       24       26       +2

Flag
Coverage Δ

unittests
82.50% <11.11%> (-1.64%)
:arrow_down:

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

Impacted Files
Coverage Δ

server/service.go
81.86% <11.11%> (-2.82%)
:arrow_down:

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

Thanks! But shouldn't be an issue, because if not found then it would return nil, which strings.Join can handle: https://go.dev/play/p/V1KzwgD_0eZ

please comment again if i'm missing something!
thanks for the contribution terence, as always 🙏


refactor: create ASTs with inner dependencies

Directly connected realizes can fuse,
eg. the 3 E_128_10 kernels in Adam: https://tiny-tools-client.vercel.app/?id=6bbb1e19fad04b2db5c36448d015e36a

I see why this has to be done. I think the API can be cleaner (membufs should be inbufs?), but will merge for now.


Use Rust stable

It appears that this runtime and its dependencies successfully compile with Rust stable (1.59.0+, I didn't test for anything lower). So we should probably switch to using stable over nightly soon.
We should also document a minimum supported Rust version (MSRV).

Note for me: it seems like at least one of the unstable features that were used were custom profiles. These were stabilised in v1.57.

We use the --out-dir option on Cargo, which is still unstable: https://doc.rust-lang.org/nightly/cargo/reference/unstable.html#out-dir
This could be implemented manually if we really wanted to. From what I can tell this is the only blocker for this issue.

If you do that then you'll probably need an escape hatch to use nightly explicitly in LF. Users could bring in dependencies that require nightly.

That's a good point!

Actually you don't necessarily need to address that point explicitly. Most people using more than one version of the Rust compiler simultaneously will set the default toolchain or an override using rustup. So if we are only invoking cargo, we allow that to implicitly mean any toolchain version a developer may have set according to their need.

That's also a good point! I would say that the best plan for now would be to not use unstable features for Cargo and remove the nightly flag when compiling, so that people can still use nightly like you described. If for some reason there's still a need for defining nightly in the LF file itself, we can always add that later.


regulation of poly(A)-specific ribonuclease activity

id: GO:new1
name: regulation of poly(A)-specific ribonuclease activity
namespace: biological_process
def: "Any process that modulates the rate, frequency or extent of the catalysis of the exonucleolytic cleavage of poly(A) to 5'-AMP." [GOC:mah,PMID:29932902]
intersection_of: GO:0008150 ! biological_process
intersection_of: regulates GO:0004535 ! poly(A)-specific ribonuclease activity
if you also need asserted links, they would be:
is_a: GO:1901917 ! regulation of exoribonuclease activity
relationship: regulates GO:0004535 ! poly(A)-specific ribonuclease activity
and + and - subtypes, following same pattern
id: GO:new2
name: negative regulation of poly(A)-specific ribonuclease activity
id: GO:new3
name: positive regulation of poly(A)-specific ribonuclease activity
to use to annotate effect of S. pombe Pabp RNA binding on Ccr4 and Caf1 nuclease activities in the context of removing poly(A) tails

edited to add pos reg request

any progress?

Slipped through in vacation; will do later today.

+id: GO:0106246
+name: regulation of poly(A)-specific ribonuclease activity
+id: GO:0106247
+name: negative regulation of poly(A)-specific ribonuclease activity
+name: positive regulation of poly(A)-specific ribonuclease activity
+namespace: biological_process
Done.

hooray, thanks!


Add 'palette' option to reach edges of arcs in canvas

The edges of arcs suffer from not exactly being the replacedColor but being close enough to look like the fill was not finished.

By supplying a palette, we can ensure those edge colors are set to the new pixel color.

Check out the added demo. As you will see, this still hits the ~40ms benchmark.
I played around with cacheing the results of isInPalette but did not see a suitable increase in performance:
    private isInPalette(pixel: ColorRGBA): boolean {
        if (!this._palette) {
            return true
        }

        const hex = (pixel.r << 24) | (pixel.g << 16) | (pixel.b << 8) | pixel.a
        const cacheResult = this._isInPaletteCache[hex]

        if (cacheResult === undefined) {
            const inPalette = this._palette.findIndex((c) => isSameColor(c, pixel)) > -1
            this._isInPaletteCache[hex] = inPalette
            return inPalette
        }

        return cacheResult
    }

Hello @kierangilliam ,
Thank you so much for your contribution!
Your interest and efforts are really appreciated.
However, I don't think the palette option fits well with this package.
I would like to keep the codebase and usage as simple as possible.
Your idea is really useful and nice. A palette of irreplaceable colors could come handy in many situations.
It is worth publishing it as a separate package. I would be glad to help if you decide to go that way.
Thank you once more for your contribution! The idea is good.
Best regards,
@pavelkukov


cargo-tauri: 1.7.1-unstable-2024-08-16 -> 2.0.6

Changelog: https://github.com/tauri-apps/tauri/releases/tag/tauri-v2.0.1
Diff: https://github.com/tauri-apps/tauri/compare/2b61447dfc167ec11724f99671bf9e2de0bf6768...tauri-v2.0.1
This release contains breaking changes. See the migration guide
cargo-tauri_1 has also been introduced in order to keep other packages that still haven't updated to v2 building. It should be removed in 25.05 as it will no longer be maintained upstream

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Also add a eval warning to Nix/NixOs.

Nothing is removed, so I don't believe there is much of a need; I also wouldn't want to cause evaluation warnings for all packages in the tree that still use v1. The release notes should do well enough for now as breaking updates are expected in new releases, and then a throw can be used for cargo-tauri_1 in 25.05 once it's no longer needed here

It seems like @ofborg is failing since it missing maintainers.

There’s currently a merge conflict with master due to recent changes: kiwitalk has been removed from nixpkgs, and both insulator2 and modrinth-app-unwrapped have been updated.

Merge conflicts have been fixed, cargo-tauri was bumped from 2.0.1 -> 2.0.6, cargo-tauri-1 was bumped from 1.8.0 -> 1.8.1, and both should (hopefully) now build with the new Darwin stdenv (i.e., without the old explicit frameworks)

There's a new merge conflict in the release notes

hey, as i needed tauri 2 as well i forked your progress and started from there. its included in my pr: #355062
which can be seen as a duplicate of this one, i just bumped the version further (2.1.0) and removed the uneccessary cargo.lock (as it can be taken from src) and wanted to share my progress (also, it has no conflicts ;))

Cargo.lock is vendored, because as mentioned previously, doing "${src}/Cargo.lock" is a usage of import-from-derivation(IFD) and isn't allowed in nixpkgs. It's reasonable, in this case, to vendor Cargo.lock.

The pnpmDeps hash also had to be updated. Thanks for the patch though, @HannesGitH!

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 346277

aarch64-linux

:fast_forward: 3 packages marked as broken and skipped:

alcom
modrinth-app
modrinth-app-unwrapped

:fast_forward: 1 package blacklisted:

nixos-install-tools

:x: 1 package failed to build:

gitbutler

:white_check_mark: 9 packages built:

cargo-tauri
cargo-tauri_1
en-croissant
gale
insulator2
mouse-actions-gui
pot
surrealist
wealthfolio

GitButler was previously failing. See #355157

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 346277

x86_64-darwin

:fast_forward: 4 packages marked as broken and skipped:

alcom
alexandria
cinny-desktop
insulator2

:x: 3 packages failed to build:

gitbutler
modrinth-app
modrinth-app-unwrapped

:white_check_mark: 3 packages built:

cargo-tauri
cargo-tauri_1
en-croissant

aarch64-darwin

:fast_forward: 4 packages marked as broken and skipped:

alcom
alexandria
cinny-desktop
insulator2

:x: 1 package failed to build:

gitbutler

:white_check_mark: 5 packages built:

cargo-tauri
cargo-tauri_1
en-croissant
modrinth-app
modrinth-app-unwrapped

modrinth-app fails on x86_64-darwin due to an existing build failure of turbo. gitbutler fails for the same reason as above

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 346277
x86_64-darwin
⏩ 4 packages marked as broken and skipped:
❌ 3 packages failed to build:
✅ 3 packages built:
aarch64-darwin
⏩ 4 packages marked as broken and skipped:
❌ 1 package failed to build:
✅ 5 packages built:
modrinth-app fails on x86_64-darwin due to an existing build failure of turbo. gitbutler fails for the same reason as above

How did you do this?

an existing build failure of turbo

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 346277

x86_64-linux

:fast_forward: 1 package marked as broken and skipped:

cinny-desktop

:fast_forward: 2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

:x: 3 packages failed to build:

alcom
alexandria
gitbutler

:white_check_mark: 12 packages built:

cargo-tauri
cargo-tauri_1
en-croissant
gale
insulator2
modrinth-app
modrinth-app-unwrapped
mouse-actions-gui
pot
surrealist
treedome
wealthfolio

aarch64-linux

:fast_forward: 4 packages marked as broken and skipped:

alcom
cinny-desktop
modrinth-app
modrinth-app-unwrapped

:fast_forward: 2 packages blacklisted:

nixos-install-tools
tests.nixos-functions.nixos-test

:x: 8 packages failed to build:

en-croissant
gale
gitbutler
insulator2
mouse-actions-gui
pot
surrealist
wealthfolio

:white_check_mark: 2 packages built:

cargo-tauri
cargo-tauri_1

x86_64-darwin

:fast_forward: 4 packages marked as broken and skipped:

alcom
alexandria
cinny-desktop
insulator2

:x: 3 packages failed to build:

gitbutler
modrinth-app
modrinth-app-unwrapped

:white_check_mark: 3 packages built:

cargo-tauri
cargo-tauri_1
en-croissant

aarch64-darwin

:fast_forward: 4 packages marked as broken and skipped:

alcom
alexandria
cinny-desktop
insulator2

:x: 1 package failed to build:

gitbutler

:white_check_mark: 5 packages built:

cargo-tauri
cargo-tauri_1
en-croissant
modrinth-app
modrinth-app-unwrapped

Could you rebase and change alcom to use cargo-tauri_1?

I will disregard gitbutler failures, as they will be fixed with #355366, modrinth-app seems to require a similar fix (not in this PR), the aarch64-linux failures are all reporting failed to build app: Target aarch64-unknown-linux-gnu does not exist. Please run rustup target list to see the available targets. which might be a regression as I can also see it on master.

I would be inclined to merge this as is, but when building some of the aarch64-linux packages touched by this PR it fails for me with failed to build app: Target aarch64-unknown-linux-gnu does not exist. Please run `rustup target list` to see the available targets..
I don't have an actual arm builder. I use binfmt.emulatedSystems, so that might be the issue here?

Seems so. This is the result when building on an Ampere server
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 346277

aarch64-linux

:fast_forward: 3 packages marked as broken and skipped:

alcom
modrinth-app
modrinth-app-unwrapped

:fast_forward: 1 package blacklisted:

nixos-install-tools

:x: 1 package failed to build:

gitbutler

:white_check_mark: 9 packages built:

cargo-tauri
cargo-tauri_1
en-croissant
gale
insulator2
mouse-actions-gui
pot
surrealist
wealthfolio


fix(middleware): Correlate responses by index when server does not re…

…turn ID

By the way, this closes https://github.com/nodkz/react-relay-network-layer/issues/55

Awesome work! 👍
Tnx!
New package version will be published from minute to minute.

No problem, thanks for greating this library :taco: !


Doc: Mention possibility of " for inline PHPLDAPADMIN_LDAP_HOSTS

The docs (main README) could mention how to customize the phpLDAPAdmin-Config using arrays.
Current state
The YAML-example uses an array for force_may:
- ldap.example.org:
  - server:
    - tls: true
    - port: 636
    - force_may: array('uidNumber','gidNumber','sambaSID')
  - login:
    - bind_id: cn=admin,dc=example,dc=org
    - bind_pass: p0p!
  - auto_number:
    - min: 1000
- ldap2.example.org
- ldap3.example.org

However the inline example does not use any array:
docker run --env PHPLDAPADMIN_LDAP_HOSTS="#PYTHON2BASH:[{'ldap.example.org': [{'server': [{'tls': True}]},{'login': [{'bind_id': 'cn=admin,dc=example,dc=org'}]}]}, 'ldap2.example.org', 'ldap3.example.org']" --detach osixia/phpldapadmin:0.6.12

Proposed solution
For example, the complete YAML-example could be translated into inline. (This might be too verbose, though). The need for escaping might be spelled out explicitely.
docker run --env PHPLDAPADMIN_LDAP_HOSTS="#PYTHON2BASH:[{'ldap.example.org': [{'server': [{'tls': True}, {'port': 636}, {'force_may': \"array('uidNumber','gidNumber','sambaSID')\"}]}, {'login': [{'bind_id': 'cn=admin,dc=example,dc=org'}, {'bind_pass': 'p0p!'}]}, {'auto_number': [{'min': 1000}]}]}, 'ldap2.example.org', 'ldap3.example.org']" --detach osixia/phpldapadmin:0.6.12

Motivation
I spent a few hours trying to define multiple base DNs for my server to be able to access the cn=config. I finally realized that it is possible to escape the inner " that are put around the array.
If there is an alternative way of customizing the phpLDAPAdmin-Config, then maybe it could be clarified. The docs suggest there is, but I do not understand what it is (apart from maybe mounting a volume to image/environment/default.yaml thus overriding the default.yml?).

Faced same problem with auto_number, would be excellent to add some docs about using array() with env vars configuration.

The documentation seems wrong, I tried setting PHPLDAPADMIN_LDAP_HOSTS with your Python code and it worked, it overrides the default config with my server's config!

Possibly worth adding some documentation/comments about setting this in a docker-compose.yml as a multiline string (I found this much easier to read and ultimately helped me find what was wrong with the syntax in my entries), e.g.
version: '3.7'

services:
  phpldapadmin:
    image: osixia/phpldapadmin:0.7.2
    environment:
      PHPLDAPADMIN_LDAP_HOSTS: "#PYTHON2BASH:[
{'ldap.example.com':[
  {'server': [
    {'tls': True},
    {'port': 389},
    {'force_may': \"array('uidNumber','gidNumber','sambaSID')\"},
    {'base': \"array('dc=example,dc=com')\"}
   ]
  },
  {'login': [
    {'bind_id': 'cn=admin,dc=example,dc=com'},
    {'bind_pass': '${LDAP_ADMIN_PASSWORD:-default_password}'}
   ]
  },
  {'auto_number': [
    {'min': \"array('uidNumber'=>1000,'gidNumber'=>1000)\"}
   ]
  }
 ]
}
]"
...


[a6000] Restarts after shutter release, while using a dummy battery connected to powerbank

everything works fine, after the tweak,
i got this issue only on when i took pictures on dummy battery
(which works before the tweak) it would restart,
but if i change the battery to non dummy it will take pictures fine,
but dummy battery on any video mode works fine,
ACTUAL SEQUENCE: Press shutter > Shuts down> shutter release> restarts
i did only follow the youtube guides and instruction, and disable the limit, and thats it...
is there an fix or at least revert it back to original settings?
my firmware was 3.20
thanks and hope this well get resolved...
more power

That doesn't sound like something that can be caused by the tweaks, but who knows...
To go back, open the app, reset all the settings to their initial values and uninstall the app.

That sounds like a voltage drop from the dummy battery.


Cannot find module 'cldr-data/defaultContent.json'

Hey,
I am using l10ns library and today I noticed that none of the commands are working. Whenever I try to search, update or call interface I get error:

module.js:540
    throw err;
    ^

Error: Cannot find module 'cldr-data/defaultContent.json'
    at Function.Module._resolveFilename (module.js:538:15)
    at Function.Module._load (module.js:468:25)
    at Module.require (module.js:587:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (<project_directory>\node_modules\l10ns\libraries\MessageFormat\index.js:16:26)
    at Module._compile (module.js:643:30)
    at Object.Module._extensions..js (module.js:654:10)
    at Module.load (module.js:556:32)
    at tryModuleLoad (module.js:499:12)
    at Function.Module._load (module.js:491:3)

I have version 2.4.11 installed, I tried to install version 2.4.41, but I get same error.
Is there anyway to fix this?

Which node version are you using?

Node version is 9.9.0.

Hey, bumping the issue.
Did you manage to check what might be an issue?

@D4rkL1ch Right now, I only get the error https://github.com/tinganho/l10ns/issues/160. Which I don't really know how to fix. I think NPM changed something.
But I guess your OS is not Mac? Which OS are you using?

I am using Windows and node.js for Windows. My colleagues using Linux also do get that error.

I will try to check this out as soon as I can, though I usually cannot get pass #160. I might endup inlining the whole CLDR data node_module.

@rhalukongor thanks, that does work.

@tinganho can you please push a working version for node8+ on npmjs?
"cldr-data": "33.0.0" works for me.
I fork it: hexadecy/l10ns
But I don't know how to correctly build this project...

@hexadecy I published a new version. Could you give it a try?
On Mac I have to run with --unsafe-perm flag:
sudo npm install l10ns -g --unsafe-perm

@tinganho yes it's working on Windows 10, node v8.11.3 and npm 6.4.1.
Command prompt as administrator:
npm install l10ns -g


ci: Add tests for embedded Hubble

This is a successor of #10828 - it is using the same basic Gingko skeleton, the Hubble CLI itself however is deployed now as a DaemonSet via Helm. This allows it to use the same tolerations as the cilium agent pods (DaemonSet do not seem to support pod affinities). This is required to ensure the Hubble client (which connects via UNIX domain socket) is only running on nodes which actually run Cilium (notably, we don't want Hubble on the k8s3 CI node).

test-me-please

Coverage decreased (-0.007%) to 44.659% when pulling 860fd0b14df5b0ff0b4bbfb3d1f788642cd3deed on pr/gandro/hubble-ginkgo-tests into 7e2c7b3b811573941d2920ba45625778ebe2a680 on master.

test-me-please

There will still be follow-up work, but this seems to work reliably. Marking ready for review.


fix(kafka): Make user permissions recursive

Corrects access to subdirectories

Depends on: https://github.com/chainguard-images/images/pull/2171


Driver should allow a DRDY line to be assigned and monitor this for end of conversion

added with #6


[DOC-573] Version aliases

Description

Upstream PRs

3.10:
3.11:
3.12:

Search version is not found at first page load.

Search fixed as of last commit


fix(lint): Fix flake8 lint

zip(a, b, strict=True)

strict is a new feature in python3.10, we can just ignore it since we use python 3.8


Implement HLS Custom Op for standalone thresholding

This implements a hls custom op that maps directly to multithresholds. It is useful when multithresholds cannot be fused into MVAUs.
Generally, this op can implement any activation function including ReLU so this PR affects #77

Closing this, will merge multiple features into a single PR

I would actually prefer several smaller PRs instead of one large one.


Add topologies to satisfy all 3 testbeds for each vendor

Other changes:

add cisco xrd to presubmit
update the presubmit test to use new flags for knebind
add kne skip reset option to fptest binding

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 56.854%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

topologies/binding/new.go
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

topologies/binding/new.go
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
1236

Relevant Lines:
2174

💛  - Coveralls

/gcbrun


Strange server behavior and crashes.

We've been experiencing strange behavior with the mod on our server. For the most part it seems to work fine when you start a new Redstone Control build but after the chunk has unloaded (server restart, people moving away from that chunk, or people logging off) the data of the blocks seem to be potentially invalid or forgotten (I'm not sure what is happening)?
The first indication of something wrong in our log is the following:
[16:10:15] [Server thread/WARN] [rs_ctr API]: It appears the used up Signal Link IDs info wasn't properly saved to disk. IDs may have been assigned duplicate!
[16:10:15] [Server thread/WARN] [rs_ctr API]: It appears the used up Signal Link IDs info wasn't properly saved to disk. IDs may have been assigned duplicate!
[16:10:53] [Server thread/FATAL] [cd4017be_lib]: invalid TileEntity state: chunk = null, unloaded = true, world = net.minecraft.world.WorldServer@3f96d007, pos = BlockPos{x=540, y=85, z=217}

Where the last line is the person breaking and replacing the block that is behaving strangely (I think the error happens on breaking of the block).
This strange behavior also applies to the Processor Assembly and Circuit Designer which on the client when opening the interface immediately closes the interface again until you break and replace those blocks.
On one particular frustrating experience the server actually barked a bit more that just the three lines above:
[16:27:34] [Server thread/FATAL] [VanillaFix]: Error executing task
java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at java.util.concurrent.FutureTask.report(FutureTask.java:122) ~[?:1.8.0_252]
        at java.util.concurrent.FutureTask.get(FutureTask.java:192) ~[?:1.8.0_252]
        at net.minecraft.util.Util.runTask(SourceFile:531) [h.class:?]
        at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:723) [MinecraftServer.class:?]
        at net.minecraft.server.dedicated.DedicatedServer.updateTimeLightAndEntities(DedicatedServer.java:397) [nz.class:?]
        at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:668) [MinecraftServer.class:?]
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:526) [MinecraftServer.class:?]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_252]
Caused by: java.lang.NullPointerException
        at cd4017be.rs_ctr.port.SplitPlug.onPortModified(SplitPlug.java:195) ~[SplitPlug.class:?]
        at cd4017be.api.rs_ctr.port.Link.<init>(Link.java:96) ~[Link.class:?]
        at cd4017be.api.rs_ctr.port.Port.connect(Port.java:76) ~[Port.class:?]
        at cd4017be.rs_ctr.port.WireBranch.addLink(WireBranch.java:167) ~[WireBranch.class:?]
        at cd4017be.rs_ctr.port.SplitPlug.addLink(SplitPlug.java:175) ~[SplitPlug.class:?]
        at cd4017be.rs_ctr.item.ItemWireCon.doAttach(ItemWireCon.java:102) ~[ItemWireCon.class:?]
        at cd4017be.api.rs_ctr.port.MountedPort.onInteract(MountedPort.java:114) ~[MountedPort.class:?]
        at cd4017be.rs_ctr.tileentity.Gate.onActivated(Gate.java:110) ~[Gate.class:?]
        at cd4017be.lib.block.AdvancedBlock.onBlockActivated(AdvancedBlock.java:217) ~[AdvancedBlock.class:?]
        at net.minecraft.server.management.PlayerInteractionManager.processRightClickBlock(PlayerInteractionManager.java:449) ~[or.class:?]
        at net.minecraft.network.NetHandlerPlayServer.processTryUseItemOnBlock(NetHandlerPlayServer.java:741) ~[pa.class:?]
        at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.processPacket(SourceFile:55) ~[ma.class:?]
        at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.processPacket(SourceFile:11) ~[ma.class:?]
        at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) ~[hv$1.class:?]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[?:1.8.0_252]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_252]
        at net.minecraft.util.Util.runTask(SourceFile:529) ~[h.class:?]
        ... 5 more
[16:27:37] [Server thread/FATAL] [VanillaFix]: Error executing task
java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at java.util.concurrent.FutureTask.report(FutureTask.java:122) ~[?:1.8.0_252]
        at java.util.concurrent.FutureTask.get(FutureTask.java:192) ~[?:1.8.0_252]
        at net.minecraft.util.Util.runTask(SourceFile:531) [h.class:?]
        at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:723) [MinecraftServer.class:?]
        at net.minecraft.server.dedicated.DedicatedServer.updateTimeLightAndEntities(DedicatedServer.java:397) [nz.class:?]
        at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:668) [MinecraftServer.class:?]
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:526) [MinecraftServer.class:?]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_252]
Caused by: java.lang.NullPointerException
        at cd4017be.rs_ctr.port.WireBranch.addLink(WireBranch.java:168) ~[WireBranch.class:?]
        at cd4017be.rs_ctr.port.SplitPlug.addLink(SplitPlug.java:175) ~[SplitPlug.class:?]
        at cd4017be.rs_ctr.port.SplitPlug.addLink(SplitPlug.java:180) ~[SplitPlug.class:?]
        at cd4017be.rs_ctr.item.ItemWireCon.doAttach(ItemWireCon.java:102) ~[ItemWireCon.class:?]
        at cd4017be.api.rs_ctr.wire.IWiredConnector$IWiredConnectorItem.doAttach(IWiredConnector.java:88) ~[IWiredConnector$IWiredConnectorItem.class:?]
        at cd4017be.api.rs_ctr.wire.RelayPort.onInteract(RelayPort.java:89) ~[RelayPort.class:?]
        at cd4017be.rs_ctr.tileentity.Gate.onActivated(Gate.java:110) ~[Gate.class:?]
        at cd4017be.lib.block.AdvancedBlock.onBlockActivated(AdvancedBlock.java:217) ~[AdvancedBlock.class:?]
        at net.minecraft.server.management.PlayerInteractionManager.processRightClickBlock(PlayerInteractionManager.java:449) ~[or.class:?]
        at net.minecraft.network.NetHandlerPlayServer.processTryUseItemOnBlock(NetHandlerPlayServer.java:741) ~[pa.class:?]
        at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.processPacket(SourceFile:55) ~[ma.class:?]
        at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.processPacket(SourceFile:11) ~[ma.class:?]
        at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) ~[hv$1.class:?]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[?:1.8.0_252]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_252]
        at net.minecraft.util.Util.runTask(SourceFile:529) ~[h.class:?]
        ... 5 more
[16:27:40] [Server thread/FATAL] [VanillaFix]: Error executing task
java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at java.util.concurrent.FutureTask.report(FutureTask.java:122) ~[?:1.8.0_252]
        at java.util.concurrent.FutureTask.get(FutureTask.java:192) ~[?:1.8.0_252]
        at net.minecraft.util.Util.runTask(SourceFile:531) [h.class:?]
        at net.minecraft.server.MinecraftServer.updateTimeLightAndEntities(MinecraftServer.java:723) [MinecraftServer.class:?]
        at net.minecraft.server.dedicated.DedicatedServer.updateTimeLightAndEntities(DedicatedServer.java:397) [nz.class:?]
        at net.minecraft.server.MinecraftServer.tick(MinecraftServer.java:668) [MinecraftServer.class:?]
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:526) [MinecraftServer.class:?]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_252]
Caused by: java.lang.NullPointerException
        at cd4017be.rs_ctr.port.SplitPlug.onPortModified(SplitPlug.java:195) ~[SplitPlug.class:?]
        at cd4017be.api.rs_ctr.port.Link.<init>(Link.java:96) ~[Link.class:?]
        at cd4017be.api.rs_ctr.port.Port.connect(Port.java:76) ~[Port.class:?]
        at cd4017be.rs_ctr.port.WireBranch.addLink(WireBranch.java:167) ~[WireBranch.class:?]
        at cd4017be.rs_ctr.port.SplitPlug.addLink(SplitPlug.java:175) ~[SplitPlug.class:?]
        at cd4017be.rs_ctr.port.SplitPlug.addLink(SplitPlug.java:180) ~[SplitPlug.class:?]
        at cd4017be.rs_ctr.item.ItemWireCon.doAttach(ItemWireCon.java:102) ~[ItemWireCon.class:?]
        at cd4017be.api.rs_ctr.port.MountedPort.onInteract(MountedPort.java:114) ~[MountedPort.class:?]
        at cd4017be.rs_ctr.tileentity.Gate.onActivated(Gate.java:110) ~[Gate.class:?]
        at cd4017be.lib.block.AdvancedBlock.onBlockActivated(AdvancedBlock.java:217) ~[AdvancedBlock.class:?]
        at net.minecraft.server.management.PlayerInteractionManager.processRightClickBlock(PlayerInteractionManager.java:449) ~[or.class:?]
        at net.minecraft.network.NetHandlerPlayServer.processTryUseItemOnBlock(NetHandlerPlayServer.java:741) ~[pa.class:?]
        at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.processPacket(SourceFile:55) ~[ma.class:?]
        at net.minecraft.network.play.client.CPacketPlayerTryUseItemOnBlock.processPacket(SourceFile:11) ~[ma.class:?]
        at net.minecraft.network.PacketThreadUtil$1.run(SourceFile:13) ~[hv$1.class:?]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) ~[?:1.8.0_252]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_252]
        at net.minecraft.util.Util.runTask(SourceFile:529) ~[h.class:?]
        ... 5 more

This was during an attempted fixing of the strangely broken Redstone Control build. During this rebuild the client (JennyLeeP) also crashed with the following crash (multiple times):
https://pastebin.com/05UySxF5
After a bit of fiddling later the client (JennyLeeP) had to go for a minute and logged off. About a minute later after she logged back on the server crashed with the following crash:
https://pastebin.com/G2V4zus2
I'm not sure if this all is related to a single issue or multiple problems. Hope you can fix these as currently the mod seems unusable on our server.

This is the full server log of the events described above:
2020-06-02-2.log

i have the same problem but in my case some link ID`s are Permanently Broken

I've just compiled a mod update that mainly adds a Block Teleporter and some utility features. But alongside my efforts to make everything in the mod compatible with being teleported around, I also rewrote code related to the errors and crashes you had and they shouldn't happen anymore.
You can download it here and the required library version here (the topmost .jar file under Assets). I may later add a beta release on curse forge as well.
You should backup your world because the save data structure for wire connections changed, so once blocks are loaded in that version, you can't go back anymore without breaking everything. Also old connections from before the update will behave a bit strange when removing them.
Issues that are probably not fixed yet are:

Signal Link IDs: might be related to world backups that don't catch the file because it's saved at the wrong time or caused when the server stops in an unusual way. The consequences of this issue are buggy wire connections in case new wires are placed and they get assigned the same IDs as some existing ones.
invalid TileEntity state: It appears that TileEntities are sometimes not initialized correctly from chunk load. I haven'd yet figured out why but added some debug code that prints this message whenever it happens and attempts to correct the invalid state which would otherwise lead to a crash. This might have caused the closing GUI issue. It would be nice to know what the blocks at {x=29, y=63, z=-2229} and {x=540, y=85, z=217} were and whether they had buggy behavior afterwards.

Signal Link IDs: might be related to world backups that don't catch the file because it's saved at the wrong time or caused when the server stops in an unusual way. The consequences of this issue are buggy wire connections in case new wires are placed and they get assigned the same IDs as some existing ones.

Our backup mechanism is pretty much the standard for minecraft servers. We ensure that minecraft isn't saving to its files, then we copy the files and afterwards we enable the saving. This way we don't copy files that are being written to at the same time causing corruption. If your mod is writing to files while the server saving is disabled this may cause corrupted backups however we have not rolled back to a backup since we added your mod so even if those files were corrupted during the backup copy phase this should not affect the live server.
In case you are curious this is our backup script: https://pastebin.com/6Rb5n99C

invalid TileEntity state: It appears that TileEntities are sometimes not initialized correctly from chunk load. I haven'd yet figured out why but added some debug code that prints this message whenever it happens and attempts to correct the invalid state which would otherwise lead to a crash. This might have caused the closing GUI issue. It would be nice to know what the blocks at {x=29, y=63, z=-2229} and {x=540, y=85, z=217} were and whether they had buggy behavior afterwards.

The block at {x=540, y=85, z=217}:  (I may have placed the assembler and designer the other way around when I placed them back).
I'm not sure what was at {x=29, y=63, z=-2229} as that is JennyLeeP's base and currently its just a standard block at that location:

I'll have to ask her when she gets back what was at that location.

You can download it here and the required library version here

Done and seems to work so far, however at this time I don't have much as JennyLeeP is the one that started using the mod the most. I'll have to ask her if things seem better at her base:
 (I have a very vague idea of what this all is but she likes to hide wires so I'm not going to touch this :P ).
We'll be testing the new version and I'll probably report back here.
Thanks a bunch!

So far ... well ... things didn't go too well. JennyLeeP was rebuilding here stuff as expected it would be broken but hopefully fixed after she rebuild everything (once again).
She left for a little bit and while I tried to rejoin the server after a little bit away it decided to immediately crash with the following crashlog:
https://pastebin.com/DEYQ5xbC
The server log has all kinds of interesting lines such as:
[22:03:45] [Server thread/FATAL] [cd4017be_lib]: invalid TileEntity state: chunk = null, unloaded = true, world = net.minecraft.world.WorldServer@318b65e9, pos = BlockPos{x=21, y=63, z=-2229}

But these were during the rebuilding.
 I'm not sure what to make of the info
Here is the full log: latest.log
I'm not sure if JennyLeeP rebuild everything or just parts of her build.

The block in question  {x=29, y=63, z=-2229}  that Noxy is referencing at my (JennyLeeP's) base was a control panel.
The panel had 1 switch , 3 lamps, 1 slider , and a 1x4 text module.
I have rebuilt all 4 systems multiple time snow and I can confirm that the issues have not been resolved.
Most of the issues I have witnessed so far happen on chunk unload.

Deactivating the force chunk loading in forge prevented the game from crashing instantly

So, now I got the Signal Link ID file to save alongside the regular world save whenever there were new connections added (previously it would only save during server shutdown). This should fix ID collisions in the future.
I figured out that the new crash was indirectly caused by the Link ID collision as well, so I additionally improved the way these collisions are handled to prevent such crashes.
Haven't made much progress with the invalid TileEntity thing except adding more debug logging. The problem is that I can't reproduce it in my dev environment or my test mod packs. According to your logs it happens only rarely. The circuit designer being affected confirms that it caused GUI insta-close issue, I made a workaround fix for that.
New updates:
CD4017BE_lib
RedstoneControl

I have the same problem with CD4017BE_lib-1.12.2-6.5.1.jar and RedstoneControl-1.12.2-<IP_ADDRESS>.jar:

start a new Redstone Control build but after the chunk has unloaded (server restart, people moving away from that chunk, or people logging off) the data of the blocks seem to be potentially invalid or forgotten (I'm not sure what is happening)?

I can not click on any buttons and levers on panels. It happens every day.

https://img.bymarcin.com/2020-07-30_01-21-41_DC0dt7pVIOF6ffpZ.png
https://img.bymarcin.com/2020-07-30_01-21-55_BBTyhjLTgwdGy4mO.png


How to login without password?

When we try social authentication, we don't need to put password yet? But, it's keep asking me password.
Undefined index: password

The method you would use to log a user in after social authentication, would be something like Auth::user()->loginUsingId($id) or Auth::user()->login($user). Neither of which request a password, it sounds like you're using attempt.


Invalid options error with Emacs26

It is showing a "Invalid options for slot timer in spin" error with today's master branch. And I suspect it affects some behavior of Emacs. Is there anyone experiencing the same thing or having any solutions?
My system:
Arch Linux
Emacs 26

I'm guessing the problem here is due to old .elc files. Try deleting your ~/.emacs.d/elpa directory and restarting.


[Tiny-fix] Add show user stmt impl.

Problem Summary:
Show User stmt always result empty because ShowExecutor.java miss the impl of handleShowUser
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (Yes)
Has document been added or modified: (Yes)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

My mistake. This stmt is used to show current login user

There are UT failed. You can click details and see the detail by click as guest

@jackwener OK, I will rebase it and fix regression

@jackwener UT has been resolved.


Pie chart: first slice should be North, not East

I found a todo comment in the code for this but I could not find a corresponding issue so I thought I'd create one to prevent the matter being forgotten.
In SP4 the first slice in a pie chart was positioned at the north of the chart.
In SP5 the first slice in a pie chart is positioned at the east of the chart.
Please could it be moved back to north or a 'first position' type option be added?
code ref: https://github.com/ScottPlot/ScottPlot/blob/fb3b1c84af3c361bf1eb8f4286ad1d4e7038afc5/src/ScottPlot5/ScottPlot5/Plottables/Pie.cs#L47-L52

Hi @johndoh, thanks for the suggestion! I agree we should make that the default, but also we could add a public property something like RotationDegrees to let the user control that as well.
I see that rotation is only applied at this line, so maybe it's as easy as adding 90 here?
https://github.com/ScottPlot/ScottPlot/blob/fb3b1c84af3c361bf1eb8f4286ad1d4e7038afc5/src/ScottPlot5/ScottPlot5/Plottables/Pie.cs#L66


Return directly lambda function instead to assign it to a variable

I dont think this is good because it breaks code. when you use return statement nothing will be executed after that. So we need $cPath_array so to check later and return the $tmp_array.
Of course, that function can be improved be replacing sizeof (which is an alias of count) with count and change the for loop including $n=count($cPath_array) or even use foreach loop.

As this function is called from application top on:
398      $cPath_array = tep_parse_category_path($cPath);
399	    $cPath = implode('_', $cPath_array);
400	    $current_category_id = $cPath_array[(sizeof($cPath_array)-1)];

and its purpose is only to convert all $cPath elements to integer make sense to me return directly the reworked $cPath array from the function.

What about:
// make sure no duplicate category IDs exist which could lock the server in a loop

Yes, now I understand you are right.
Just left unchanged previous code and added array uniqui to this commit that makes the same thing

Change to array flip method as it is faster


Make keep .rodata in a distinct segment?

lucetc artifacts currently result in .so that have .text and .rodata merged together. From readelf -l a.out:
 Section to Segment mapping:
  Segment Sections...
   00     .hash .gnu.hash .dynsym .dynstr .rela.dyn .text .rodata
   01     .dynamic .data
   02     .dynamic
   03
   04     .dynamic

as a result, pages in segment 0 are all executable:
Program Headers:
  Type           Offset             VirtAddr           PhysAddr
                 FileSiz            MemSiz              Flags  Align
  LOAD           0x0000000000000000 0x0000000000000000 0x0000000000000000
                 0x0000000000021a30 0x0000000000021a30  R E    0x200000

(important bit being the E flag!)
Do we want to put .rodata in a read-only (no-execute) segment?

Yes!


readme typo fix and add --purge to helm delete

typo fix and add --purge to helm delete

Thank you for contributing to kubernetes/charts. Before you submit this PR we'd like to
make sure you are aware of our technical requirements and best practices:

https://github.com/kubernetes/charts/blob/master/CONTRIBUTING.md#technical-requirements
https://github.com/kubernetes/helm/tree/master/docs/chart_best_practices

For a quick overview across what we will look at reviewing your PR, please read
our review guidelines:

https://github.com/kubernetes/charts/blob/master/REVIEW_GUIDELINES.md

Following our best practices right from the start will accelerate the review process and
help get your PR merged quicker.
When updates to your PR are requested, please add new commits and do not squash the
history. This will make it easier to identify new changes. The PR will be squashed
anyways when it is merged. Thanks.

/ok-to-test

/lgtm


[0.9.1-rc1] Database not found error not in json

When writing into a non existing database, the error message is not in a JSON structure like other error messages.
curl -X POST 'http://<IP_ADDRESS>:8086/write?db=notexist' --data-binary @TEST_VALUES
database not found: "notexist"
curl -X POST 'http://<IP_ADDRESS>:8086/write?db=test&u=notexist&p=password' --data-binary @TEST_VALUES
{"error":"user not found"}
I think it is better to embedded all error messages in a JSON structure.

This seems to be related to these lines or the way errors are handled. A search reveals that "user not found" is never returned directly, but rather as an error which may be relevant.

This should not be too difficult. Simply trace through the code to learn where errors are created, and use Go's built-in JSON encoding to spit out a JSON error.

Grabbing this.

Hey @jademcgough still working on this issue?

@aneshas will try to have a PR ready this weekend, I'm still ramping up my golang so I spent a few hours just familiarizing myself with the codebase :)

@jademcgough Oh, ok no problem, just let us know if you decide to pass it on :)


chore: revert nuxt version

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[ ] Refactoring

Context

[ ] Closes #<issue_number>
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[ ] My DOT address: Payout

Screenshot 📸

[ ] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at ba37afc
Fixed the nuxt version and updated some rollup plugins in pnpm-lock.yaml and package.json. This was done to improve the stability and compatibility of the project dependencies.

🤖 Generated by Copilot at ba37afc

To avoid any breaking changes
We fixed the nuxt version ranges
And updated the lock
With rollup and replace in stock
And added some requiresBuild changes

What's the issue with nuxt version?

safe to close I guess, didn't fix this issue https://github.com/kodadot/nft-gallery/issues/7989#issuecomment-1806755257


How to get started... trying to debug and it gives me message "Application not fond"

First run, trying to comple the Hello world, that has some trace statements in it.
Can't seem to get the PROJECT / DEBUG command to work. It says haxe -debug...., build succeeded, Done(0), then says "Application not found".
Will be glad to pay for some help.

What type of project?
FD version? (Help -> About...)
Haxe version?


[CI:DOCS]add return info for prereview

begin to add return info and status codes for errors so that we can
review the apiv2
Signed-off-by: baude<EMAIL_ADDRESS>

LGTM
although I don't really understand what is going on.  I figure this is just input to a tool that describes what the return codes are when generating API.

LGTM
although I don't really understand what is going on. I figure this is just input to a tool that describes what the return codes are when generating API.

It is an intermediate step for tool input yes.  I havent figured out how the tooling quite works yet so I just took the time to record it and somewhat verify that we are doing the right things.

/lgtm


Help with the code snippet generation part

Could you help me with this step? which command(s) can I run to do this part of the README:  you can generate the code snippets for each example of ExeDS with your model.?

Sorry for the late reply. You can use the the (preprocessing scripts)[https://github.com/Jun-jie-Huang/ExeDS#2-preprocessing] to generate the examples for the model. Then you can train the inference with the model.


Figure out cause of timeouts for full list-images, and adjust

We want to make more use of the all resources list-images API, but in our dev environment for instance, it tends to time out.
Possible remedies:

if there's too much data, introduce a summary mode or the like so that we send only (say) the current image, the latest (filtered) image, and the distance between them.
if there's too much calculation, um ... figure out how if we can do less calculation.

Related #913

Ops! Closing as duplicate of #913.