Using ".js" in the page title breaks nginx

Describe the bug
if you create a new page with a title ending with .js, this breaks the nginx and renders the page inaccessible with "404 Not Found" errors just for this page.
Steps To Reproduce
Steps to reproduce the behavior:

Go to a book
Create a blank page
title it with something like "Node.js" (without quotes)
See error

Expected behavior
Should behave normally and open the page as normal.
Screenshots
If applicable, add screenshots to help explain your problem.
Your Configuration (please complete the following information):

Exact BookStack Version (Found in settings): BookStack v0.31.7
PHP Version:
Hosting Method (Nginx/Apache/Docker): docker

Additional context
It is pretty serious because you are stuck with this page. You cannot access it so you cannot delete, rename or move it.

Hi @Salvora,
Pages saved with such a title should not contain a . in the url path for the page. The example provided should save with a URL path ending with /page/nodejs. Is that the same behaviour with your instance? If not, can you provide the path you get redirect to after save?
My guess for this would be overly general rules written in the webserver config, with a rule looking for URLs ending with js instead of .js. It'll be wroth checking the webserver rules active on both the proxy in front of your container (If existing) and the webserver within the container.

Yes it doesn't contain an url path:
the page with a title koko.js in book dummy url
books/dummy/page/kokojs
but this url throws error anyway.

I use Home Assistant addon for BookStack so webserver is not in my control. I was thinking maybe the way the url arranged is wrong for this case.
If the problem is not the internal webserver, I can open a bug report on the addon page.

Thanks for confirming.
The URL example you've provided looks fine and as per expected, I would not want to add additional edge-case handling within the app itself to handle misconfigured web servers at all.
If this is definitely not something within your control then yeah, I'd also recommend raising with the BookStack hassio-addons project: https://github.com/hassio-addons/addon-bookstack. They're generally active. From a quick look I could not see anything in there that would cause this but I don't have a home assistant setup to test on.


Bring back tombstones to Head block

This PR reverts the changes done by https://github.com/prometheus/tsdb/pull/270 which is required for https://github.com/prometheus/prometheus/issues/6377.
One change that I noticed while fixing the tests is that even if all the samples are deleted in the tombstones, the querier would still return series but with 0 samples. Is this behaviour fine or should the querier not return any series at all?

That behaviour is okay, we'd need a bit much pre-processing otherwise. PromQL should in principle already deal with this, but add uniitests if you're unsure.

Then the current test coverage looks fine for this part where samples are verified.

What do benchmarks look like?
It also looks like once we delete data that you can't recreate it in that time range, which makes things simpler (and is reasonable semantically).

What do benchmarks look like?

Query benchmarks? Haven't done any benchmark for this change yet

It also looks like once we delete data that you can't recreate it in that time range

That was the behavior before prometheus/tsdb#270

We don't have any benchmarks covering tombstones in the head?

I don't see any, so, nope. What do we want to benchmark for tombstones? Queries/iterators?

It's not a use case we particularly care about performance wise, as you shouldn't be deleting from the head outside of rare mistakes you need to clean up so I wouldn't worry about it.
Probably should add a unittest to cover trying to add a sample in a time period that has been tombstones though.


[$500] SetupNode action failing when patch has been updated

Problem
When we modified only a patch without changes to the package-lock.json the node_modules are not reinstalled during the build process which means the patch is not applied
@adamgrzybowski Has updated the setupNode script to also listen to changes in the patch folder and if that is a case, apply the patch.
There is however still an issue which causes the setupNode action to fail now when patch is updated https://github.com/Expensify/App/actions/runs/5983634462/job/16233978359
Solution
Lets fix the action to apply the patch safely when there is a change in the patch file
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01373fad44757789f7
Upwork Job ID:<PHONE_NUMBER>560124928
Last Price Increase: 2023-08-28
Automatic offers:

shubham1206agra | Contributor | 26372375

Is this up for proposals? @mountiny or @shubham1206agra only gonna work on it?

@b4s36t4 we've hired for this - thank you for your interest!

@shubham1206agra mind giving an update? 🙏

Waiting for PR to be reviewed

payment time!

 I was mostly on leave last week. So, running quite behind. I will try to get to this issue soon.

nice!


Integration testing for external istiod

I don't think this is a release blocker, so I pushed this to 1.9. @linsun @ostromart please move this back to 1.8 if you think otherwise. Thanks!

I think a basic integration test is already created for this.  closing this, feel free to reopen or open a new issue for specific tests.

I think a basic integration test is already created for this.  closing this, feel free to reopen or open a new issue for specific tests.


switched to elemental2 Blob implementation

do not merge it for now

@tiagobento looks like it's good to go

@tomasdavidorg Hi Tomas, can you please verify this PR?


Add Active/Pressed State to Navbar Buttons

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have read the contributing guidelines

Describe the issue
Currently all navbar buttons have no add active/pressed state so they need to be added on button click.
Reduced test cases
No response
What operating system(s) are you seeing the problem on?
No response
What browser(s) are you seeing the problem on?
No response
What is the issue regarding ?
@trimble-oss/modus-web-components
What version of npm package are you using ?
No response
Priority
Medium
What product/project are you using Modus Components for ?
E-Builder/Branding
What is your team/division name ?
E-Builder/DarkKnights
Are you willing to contribute ?
Yes
Are you using Modus Web Components in production ?
No response

This ticket was discussed in a meeting with the two teams.  I'm waiting to hear if they need technical help to resolve their need here after they check their options with gainsight.


forge script --json always returns a zero status

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (2f4dc6c 2022-07-29T00:03:54.821792323Z)
What command(s) is the bug in?
forge script --json Contract
Operating System
Linux
Describe the bug
Errors in a solidity script caused by reverts, asserts, etc, induce a non-zero exit code. However, running the script with the --json option always returns zero.
$ forge script AlwaysReverts --json && echo "success"
[⠰] Compiling...
[⠔] Compiling 4 files with 0.8.14
[⠃] Compiling 6 files with 0.8.9
[⠊] Solc 0.8.9 finished in 1.27s
[⠆] Solc 0.8.14 finished in 1.29s
Compiler run successful

{"logs":[],"gas_used":299261,"returns":{}}
success

$ forge script AlwaysReverts && echo exited
[⠰] Compiling...
[⠑] Compiling 5 files with 0.8.9
[⠃] Solc 0.8.9 finished in 1.24s
Compiler run successful

Script failed.
Gas used: 299261

== Return ==
Error:
Script failed.

This is a surprising behavior and I'd expect the exit status when adding --json to work the same way without the option.

Confirmed that forge test --json exits with an error, but forge script --json does not. Going to add this to https://github.com/foundry-rs/foundry/issues/4444 to track


The Evaluation Time of AVS-Semantic Datasets

Thanks for your wonderful job!
I'm facing a question while performing the training process of the AVS-Semantic Datasets. The evaluation time of this dataset is too long especially the computing process of miou metrics. The evaluation time can be over 6 hours for the whole dataset. Have you meet the similar questions and how to solve it?

It is normal when I re-run the code of AVSS training. Could you provide more information about it?
Here is my evaluation logs(Only evaluation):


⚠ Bofrost.at has degraded performance

In fb21a42, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1760 ms

Resolved: Bofrost.at performance has improved in dc6e6e2 after 28 minutes.


Changing ThumbIndicatorWidth for ToggleSwitch does not work

What steps will reproduce this issue?
I am overriding ToggleSwitchButtonStyle for a ToggleSwitch like this:
<Style x:Key="SmallerMetroToggleSwitchButton" TargetType="{x:Type Controls:ToggleSwitchButton}" BasedOn="{StaticResource MetroToggleSwitchButton}">
  <Setter Property="Width" Value="40" />
  <Setter Property="Height" Value="20" />
  <Setter Property="ThumbIndicatorWidth" Value="4" />
</Style>

I see that Width and Height values are applied while thumb indicator width is not (it has default width).
Expected outcome
Thumb indicator width should be changed
--
Environment

MahApps.Metro v 1.3.0-ALPHA176
Windows 7
Visual Studio 2015
.NET Framework 4.5

Actually it works if I set ThumbIndicatorWidth directly in a ToggleSwitch e.g. like this: <ctrl:ToggleSwitch ThumbIndicatorWidth="7" /> so maybe it is not a bug.

@olegfeferman Yeah, this is by design, the ToggleSwitch use TemplateBinding for this values, so the values from the Style are overriden.


installation: fix prerequisite package versions

FIX Adds upper limit on versions of several prerequisite packages,
fixing newly observed Invenio 2.0 demo site installation troubles.

Reported-by: Dmitri Pisarev<EMAIL_ADDRESS>Signed-off-by: Tibor Simko<EMAIL_ADDRESS>

This should fix the installation troubles observed recently by @dimaip.
(CC-ing also @pamfilos who experienced CERN Open Data installation troubles lately as well.)

@tiborsimko thanks a lot for the fix! It fixed the web container, but the worker is still failing.

 Testing...

:+1: Verified.
Thanks a lot for taking care!

+1: Works for me..Thank you @tiborsimko

:dancer:

Changing to WIP while trying to get all the Travis tests pass...


Add a deprecation warning to validate_for if schema doesn't exist

This addresses this issue #460
Followed your lead by using the SynchronousTestCase . Any reason or particular features that drive using it instead of just TestCase?
Please let me know if you have any comments. Can't think of any other bright ideas regarding the warnings. First time using the module :) and feels like a pretty small PR so maybe I am missing something.

@Julian  updated the pr please take a look

Fixed the failing test on merge.
Really appreciated!


Add useAutoScroll() hook and context to fix autoscrolling

Fixes #105
This fixes a bunch of bugs in how the auto scrolling feature works.  Specifically:

Adds a new React context and hook useAutoScroll().  This keeps track of a React ref that points to the "bottom" of the message list (i.e. where we should auto scroll to in the messages), whether or not the UI should autoscroll, and also the current progress of scrolling (i.e., every time there's some new content added that requires autoscroll logic to fire).  Previously a bunch of this stuff was done as state in ChatBase, but I needed to share it across components.
When we render the NewMessage component, we use it as our "bottom."  This works both when doing a normal chat, and also retrying with a different LLM midway through the chat.
When the content of what we're streaming (i.e., a new AI Message) changes, we increment the scrollProgress in the autoscroll context, telling effects in the UI to fire and perform various autoscroll logic.
When we aren't streaming a response (i.e., scrollProgress === 0), we no longer scroll the whole chat (e.g., when deleting or editing a message part-way through).
I've increased the fudge-factor threshold for when to break autoscrolling as the user scrolls up.  Previously, it was happening too easily when the LLM added "tall" content (e.g., an iframe or mermaid diagram).  I think using 100 pixels is better.

A key idea to how this code works:

useChatOpenAI() is usable in different parts of the app without sharing logic.  That is, the streamingMessage you get back is specific to the component where you're working.  Retrying a message with an LLM won't trigger any logic in the main chat, which also uses the same hook (i.e., it's just a hook, not a shared context).
useAutoScroll() is shared across all the chat-related components in the app.  If you update something in it, it affects all other parts of the app (i.e., it's a context vs. only a hook).

I need to spend more time testing this, and thinking through the code, but I think it's just about ready to go, and makes the UI so much better to use!
Try it and see if you notice any problems.

@tarasglek when you have a chance, can you test this a bit?  I think it's good to go (minus bug with scrolling function stuff, which I'll do in follow-up).

I can't see any bugs with either prod or your version atm. If you feel this is better, lets land :)

OK, I'll do this and file a follow-up to figure out how to scroll function results into view.


On Android crashes on App start - Error code -20 when initializing native AudioRecord object.

Version of react-native-audio-recorder-player
2.5.0
Platforms you faced the error (IOS or Android or both?)
Android
Expected behavior
Works as in iOS
Actual behavior
The app doesn't even start, it crashes immediately every time.
Steps to reproduce the behavior
Install and follow the instructions.
React native version
0.60.5
I've installed the package like the readme says, on iOS no problem at all, on Android I've this error:
03-17 15:53:00.521 1516-21879/? I/AudioFlinger: AudioFlinger's thread 0xa9803c80 tid=21879 ready to run
2020-03-17 15:53:00.524 1516-11030/? E/AudioFlinger: not enough memory for AudioTrack size=131296
2020-03-17 15:53:00.524 1516-11030/? E/AudioFlinger: createRecordTrack_l() initCheck failed -12; no control block?
2020-03-17 15:53:00.527 11662-21739/? E/AudioRecord: AudioFlinger could not create record track, status: -12
2020-03-17 15:53:00.530 11662-21739/? E/AudioRecord-JNI: Error creating AudioRecord instance: initialization check failed with status -12.
2020-03-17 15:53:00.530 11662-21739/? E/android.media.AudioRecord: Error code -20 when initializing native AudioRecord object.

My AndroidManifest.xml has these permissions:
<uses-permission android:name="android.permission.INTERNET" />
    <uses-permission android:name="android.permission.CAMERA" />
    <uses-permission android:name="android.permission.VIBRATE" />
    <uses-permission android:name="android.permission.RECORD_AUDIO" />
    <uses-permission android:name="android.permission.MODIFY_AUDIO_SETTINGS" />
    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />

My project gradle is:
buildscript {
    ext {
        buildToolsVersion = "28.0.3"
        minSdkVersion = 19
        compileSdkVersion = 28
        targetSdkVersion = 28
        //supportLibVersion = "28.0.0"
        androidXAnnotation = "1.1.0"
        androidXBrowser = "1.0.0"
        googlePlayServicesVersion = "17.0.0"
        googlePlayServicesVisionVersion = "15.0.2"
        mediaCompatVersion = '1.0.1'
    }
    repositories {
        google()
        jcenter()
    }
    dependencies {
        classpath('com.android.tools.build:gradle:3.5.2')

        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

What did I do wrong?
Thanks a lot!
\m

Have you granted the permission in Android?
        const granted = await PermissionsAndroid.request(
          PermissionsAndroid.PERMISSIONS.WRITE_EXTERNAL_STORAGE,
          {
            title: 'Permissions for write access',
            message: 'Give permission to your storage to write a file',
            buttonPositive: 'ok',
          },
        );
        if (granted === PermissionsAndroid.RESULTS.GRANTED) {
          console.log('You can use the storage');
        } else {
          console.log('permission denied');
          return;
        }

Also I hope you to test Example app too. I've just done it now and seems to be working correctly.


Snappy data viewer for large DFs

This feature request is following from microsoft/vscode-python#5469
When I work with large pandas DFs, I relied on PyCharm's dataframe viewer. I liked it because it was very responsive and fast, though it wasn't without problem. With VSCode, however, it is nearly impossible to view large DFs, and many times it takes too much time to see it.
My guess is that PyChram programmed the viewer to work dynamically - it does not throw all of the data into the viewer, but some parts of data that is visible to users. As the user scrolls to left/right and up/down it instantly loads data to the viewer. Sometimes it was a little bit buggy but was still okay to use.
I hope vscode Data Viewer can handle large DFs like this.

This problem does not occur only with dataframes. Any type of large data has this behavior. Big numpy arrays, big lists, big dicts...

We will likely just rely on external extensions like Data Wrangler for data viewer functionality going forward - closing

Is it possible to use an external extension like Data Wrangler in a debug session? So that it opens Data Wrangler's data viewer when choosing "View Value in Data Viewer"?


Changes to support crd v1 and pipeline processes

Steps to prepare for the next release and align with CoP pipeline

LGTM


[INLONG-688] change additionstr to additionAtr in agent

[INLONG-688] change additionstr to additionAtr in agent

Codecov Report

Merging #522 (01d71cb) into master (08916d4) will decrease coverage by 0.07%.
The diff coverage is n/a.

:exclamation: Current head 01d71cb differs from pull request most recent head 3a2a646. Consider uploading reports for the commit 3a2a646 to get more accurate results

@@             Coverage Diff             @@
##             master    #522      +/-   ##
===========================================
- Coverage      7.73%   7.65%   -0.08%
+ Complexity      565     561       -4
===========================================
  Files           355     355
  Lines         31077   31077
  Branches       4970    4970
===========================================
- Hits           2403    2380      -23
- Misses        28185   28213      +28
+ Partials        489     484       -5

Impacted Files
Coverage Δ

...nlong/tubemq/corebase/policies/FlowCtrlResult.java
0.00% <0.00%> (-60.00%)
:arrow_down:

.../java/org/apache/flume/sink/tubemq/TubemqSink.java
51.42% <0.00%> (-4.00%)
:arrow_down:

.../inlong/tubemq/corebase/policies/FlowCtrlItem.java
36.66% <0.00%> (-3.34%)
:arrow_down:

.../tubemq/corebase/policies/FlowCtrlRuleHandler.java
42.52% <0.00%> (-2.06%)
:arrow_down:

Continue to review full report at Codecov.

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


Adding descriptions for cxx examples

@ajpmaclean : Please review.
I have tried to add appropriate descriptions for some of the Cxx Examples.
Please correct me if the descriptions provided are unnecessary.
I am referring the descriptions from https://vtk.org/doc/nightly/html/annotated.html

I think it is a good idea, thanks.

If you can, can you please update the Java and Python example descriptions at the same time?


Bug fix/create collections better preconditions

Pull Request Guidelines
Acceptance Checklist
Scope & Purpose
During a collection creation the collections are inserted with a TTL, after success the same object was inserted(persisted) without TTL, however if during this a FAILOVER did happen this remove TTL operation stored now invalid information in plan unintentionally. It is now protected by a more precise precondition

[ ] Bug-Fix for devel-branch (i.e. no need for backports?)
[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[ ] Strictly new functionality (i.e. a new feature / new option, no need for porting)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)
[ ] The behaviour change can only be verified via automatic tests

Related Information
(Please reference tickets / specification etc )

[x] Internal chaos QA

Testing & Verification
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(Include link to Jenkins run etc)
http://jenkins.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/4914/

Think about whether the new code you added is modular enough to be
easily testable by unit tests written with GTest / Catch. It is not good if your feature is so interconnected
that it prevents other people from writing their own unit gests. It should be possible
to use your code in future without extensively mocking your classes.
A bad example that required some extensive effort would be the storage engine API.

Documentation

All new Features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the changelog so that
developers and users have a concise overview.

[x] Added a Changelog Entry
[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the http API
[ ] Added Swagger examples for the http API

CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

Updated Jenkins PR:
http://jenkins.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/4917/


[SPARK-20407][TESTS] ParquetQuerySuite 'Enabling/disabling ignoreCorruptFiles' flaky test

What changes were proposed in this pull request?
SharedSQLContext.afterEach now calls DebugFilesystem.assertNoOpenStreams inside eventually.
SQLTestUtils withTempDir calls waitForTasksToFinish before deleting the directory.
How was this patch tested?
Added new test in ParquetQuerySuite based on the flaky test

Test build #75983 has started for PR 17701 at commit 4364c98.

Do you know any Jenkins build where this test was failed?

Fails rarely in our own jenkins. The test I added is just part of an existing test that is run 100 times, that one fails reliably.

A few minor comments, otherwise LGTM.

Test build #75991 has started for PR 17701 at commit adab242.

Test build #75991 has finished for PR 17701 at commit adab242.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75991/
Test PASSed.

Merging to master/branch-2.2. Thanks!


Data is reset between each request

Consider the following code:
require 'rails_helper'

RSpec.describe "Customer", type: :request, live: true do
  before :all do
    @client = StripeMock.start_client
  end

  after :all do
    StripeMock.stop_client
  end

  it "should create a new customer" do
    customer = Stripe::Customer.create
    get root_path
    Stripe::Customer.retrieve(customer.id)
  end
end

Running this code results in the following exception:
Failure/Error: Stripe::Customer.retrieve(customer.id)
Stripe::InvalidRequestError: No such customer: test_cus_1
It seems like any request resets the server data. However, that's not even the case, since:
it "should create a new customer" do
  customer = Stripe::Customer.create(email<EMAIL_ADDRESS>  get root_path
  puts @client.get_server_data(:customers)
  puts Stripe::Customer.list
end

outputs:
# puts @client.get_server_data(:customers)
<EMAIL_ADDRESS>:description=>"an auto-generated stripe customer data mock", :object=>"customer", :created=>1372126710, :id=>"test_cus_1", :livemode=>false, :delinquent=>false, :discount=>nil, :account_balance=>0, :currency=>nil, :sources=>{:object=>"list", :total_count=>0, :url=>"/v1/customers/test_cus_1/sources", :data=>[]}, :subscriptions=>{:object=>"list", :total_count=>0, :url=>"/v1/customers/test_cus_1/subscriptions", :data=>[]}, :default_source=>nil}}

# puts Stripe::Customer.list
{
  "object": "list",
  "data": [

  ],
  "url": "/v1/",
  "has_more": false
}

So the data is still on the server, but I can't retrieve it. Is this a bug or am I doing anything wrong?

I did some debugging and found that the issue was caused by StripeMock.start being called somewhere else in the stack, creating a new instance of StripeMock. Any request to Stripe was then handled by this new instance of StripeMock, instead of the server which still had the data.


migrate to typescript 1.6

Daily code health exercise.
Just a couple of fixes required to account for "stricter object literal assignment checks":
https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript#stricter-object-literal-assignment-checks

:+1:


Not able to add oAuth2.0 while creating enterprise.generateSignupUrl

This is my code:-
import requests
class Workingwithapi:
    def generatesignupUrl(self):
            API_ENDPOINT = "https://www.googleapis.com/androidenterprise/v1/enterprises/signupUrl"

            data = {
                    'callbackUrl': 'https://console.cloud.google.com/freetrial/signup/tos?_ga=2.48272614.1773331467.1589037178-1754419521.1588923374&_gac=1.205245988.1588949758.EAIaIQobChMI96v2i8Ok6QIVxHwrCh0qmQ9CEAAYASAAEgKMi_D_BwE'

            }

            r = requests.post(url=API_ENDPOINT, data=data)

            pastebin_url = r.text

            print("The information is :%s" % pastebin_url)

object1 = Workingwithapi()
object1.generatesignupUrl()
Im trying to get a response for the generateSignupUrl but not able to see the response due to this error :-
The information is :{
"error": {
"code": 401,
"message": "Request is missing required authentication credential. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.",
"errors": [
{
"message": "Login Required.",
"domain": "global",
"reason": "required",
"location": "Authorization",
"locationType": "header"
}
],
"status": "UNAUTHENTICATED"
}
}

@rajmani17234 Is there a tutorial you were following? You don't seem to be using this library (google-api-python-client) in your code.
If you're looking for documentation on using OAuth2 with Google APIs, please see https://developers.google.com/identity/protocols/oauth2

hey @busunkim96  could you please send your email-id where in I can  talk to you about this entire tutorial i'm encountering the issue

Could you paste the link here?

https://developers.google.com/android/work/play/emm-api/create-enterprise
So i'm following this tutorial where in the enrollment (https://developers.google.com/android/work/play/emm-api/create-enterprise#enrollment_process) section i'm trying to do the first step

Hi @rajmani17234,
I'm going to close off this issue due to inactivity. If you're still having trouble, please re-open it and we can take a look.


Why doesn't everything get a reset fn?

It seems like we only have a reset fn if stateful==true, but why? Almost all modules will require a reset due to pipelining registers? Why not just include reset for everything?
& then get rid of stateful annotation b/c this is not used for anything else?
actually, pipeline registers aren't even being reset currently!!

Perhaps systolic should just automagically build a reset for everything

Correction: systolic doesn't generate pipeline valid bits (only pipeline registers, which don't need to be reset). Valids are calculated based on the static delay. So, systolic modules only require a reset if they have explicit state.


Error while using resolutions.yml file

Hello team,
I am facing an error while trying to use the resolution file to resolve the violations.
Seeking your support to understand the issue. Please find the details on the input provided.
Note:-  While trying only for the vulnerabilities, it worked.
But the rule_violations, is throwing below attached error.
resolutions.yml file used.
vulnerabilities:
- id: "GHSA-ww39-953v-wcq6"
  reason: "INVALID_MATCH_VULNERABILITY"
  comment: "A vulnerability resolution which resolves only vulnerabilities for testing."

rule_violations:
- message: "The dependency NPM::ismyself:1.0.2 is licensed under the 'network-copyleft' categorized license AGPL-3.0-or-later. This requires approval of the OSPO."
  reason: "CANT_FIX_EXCEPTION"
  comment: "overide this policy."

Command used:
run:  ort/bin/ort report -i ./evaluation-result.yml -f WebApp --resolutions-file ./resolutions.yml -o ./

Error message received.

@saaskesaakura345  This type of error is what you normally get when your resolutions.yml is invalid YAML e.g. incorrect indentation or hidden zero-width charactor. I can't narrow down the issue without the original resolutions.yml file, recommend use you use your IDE YAML linter or https://www.yamllint.com/  to find the root cause of the issue.

@tsteenbe Thank you for your prompt response. I am super excited and eagerly looking forward exploring ORT tool.
Please find the resolutions.yml file that i am exploring ORT with.
The ID and message was taken from "evaluation-result.yml file"
vulnerabilities:
- id: "GHSA-ww39-953v-wcq6"
  reason: "INVALID_MATCH_VULNERABILITY"
  comment: "A vulnerability resolution which resolves only vulnerabilities for testing."

rule_violations:
- message: ".*NPM::ismyself:1.0.2 .*"
  reason: "CANT_FIX_EXCEPTION"
  comment: "overide this policy."

Now I am able to get a success scan, but the resolutions provided above are not reflected in report.
I am passing this file to the reporter module as below.
run:  ort/bin/ort report -i ./evaluation-result.yml -f WebApp --resolutions-file ./resolutions.yml -o ./

@saaskesaakura345  in .NPM::ismyself:1.0.2 ." there is a space after the 2 which might be the issue. FYI for these type of basic issues the quickest way to get support is actually via the ORT Slack channel.

You may try this:
resolutions:
  vulnerabilities:
  - id: "GHSA-ww39-953v-wcq6"
    reason: "INVALID_MATCH_VULNERABILITY"
    comment: "A vulnerability resolution which resolves only vulnerabilities for testing."
  rule_violations:
  - message: ".*NPM::ismyself:1.0.2 .*"
    reason: "CANT_FIX_EXCEPTION"
    comment: "overide this policy."

@tsteenbe and @fviernau , thank you for your inputs.
The issue is resolved now.
For furture refernce if anyone faces this issue, please find what worked for me.
Call the evaluator with  --resolutions-file ./resolutions.yml which is automatically used by the Reporter as well.


[Tests] Reorg/refactor of karma tests, add JSDOM for node tests and improve coverage

resolves #4039 and #3519
Removes isparta in favour of istanbul alpha with source map reconciliation support built in. Refactors several tests into shallow rendered unit tests. Adds JSDOM globals to the test suite so components dependent on DOM APIs can be effectively unit tested using enzyme.

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

@nathanmarks Really nice 😍.

@oliviertassinari awesome! thanks!


Error 500 when opening the site

Hello! After installation, Error 500 occurs when opening the site (php 7.4)
https://audiobook-online.com/ph7builder1721/
(a clear and concise description of what the bug is.)
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected Behavior
(a clear and concise description of what you expected to happen.)
Screenshots / Screencasts
(if applicable, add screenshots or screencast the scenario to help explain your issue the best you can.)

Hi @alex7070m
There must be an internal PHP error causing this 500 page you mentioned (on your last screenshot).
Can you enable the development mode to show the PHP errors (exceptions) on the page?
To do so, please edit _protected/app/configs/config.ini, and replace environment = production to environment = development, and then save it.
Then, go back to your web browser and  refresh the page.
From there, you will normally see an explanatory PHP error message. Let me know the error you will see there.
Have a great day! 😊

Thanks, I did as you wrote, such error message appeared.
Fatal error: Uncaught TypeError: strtolower() expects parameter 1 to be string, bool given in /home/audioboo/public_html/ph7builder1721/_protected/framework/Server/Server.class.php:165 Stack trace: #0 /home/audioboo/public_html/ph7builder1721/_protected/framework/Server/Server.class.php(165): strtolower(false) #1 /home/audioboo/public_html/ph7builder1721/_protected/app/Bootstrap.php(78): PH7\Framework\Server\Server::isRewriteMod() #2 /home/audioboo/public_html/ph7builder1721/index.php(45): PH7\Bootstrap->run() #3 {main} thrown in /home/audioboo/public_html/ph7builder1721/_protected/framework/Server/Server.class.php on line 165

Thank you. This issue has been already patched in 18.x default branch https://github.com/pH7Software/pH7-Social-Dating-CMS/commit/88c58904a65c015309fab4bef9a3f4d9118a0cd5
However, hasn’t been release in the zip archive yet.
In your website, can you go to protected/framework/Server/Server.class.php and copy/paste the content of this file https://raw.githubusercontent.com/pH7Software/pH7-Social-Dating-CMS/18.x/_protected/framework/Server/Server.class.php and save it

Same problem as https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/1061

Thank you! That helped.


Basic sveltekit app with workers fails to finish build (hangs right after success message)

Describe the bug
Sveltekit app with workers fails to build. npm run dev works.
// $lib/worker/foo.ts
self.addEventListener('message', (message) => console.log(message.data));

// $lib/worker/foo-instance.ts
export const fooInstance = new Worker(new URL('./foo', import.meta.url));

// +page.svelte
<button on:click={async () => {
  const { fooInstance } = (await import("$lib/worker/foo-instance"));
  fooInstance.postMessage(Math.random());
}}>test worker</button>

The build starts running and shows the The emitted file "vite-manifest.json" overwrites a previously emitted file of the same name.. Continues and even shows success message ✓ built in XYms, but immediatelly hangs after it. The process will not quit.
Reproduction
https://github.com/marianmeres/_sveltekit-app-worker-bug
https://stackblitz.com/edit/sveltejs-kit-template-default-wwv4gb
Logs
No response
System Info
System:
    OS: macOS 13.2.1
    CPU: (8) arm64 Apple M1
    Memory: 52.84 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 19.6.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.4.0 - /opt/homebrew/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 111.0.5563.110
    Firefox: 111.0.1
    Safari: 16.3
  npmPackages:
    svelte: ^3.56.0 => 3.57.0

Severity
blocking an upgrade
Additional Information
No response

I'm experiencing the same problem in a fresh repo.
Directly Imported Worker - Build Hangs
If you attempt to import a worker using the ?worker query suffix per the Vite docs, the npm run build commands hangs.  It appears a "vite-manifest.json" is created for the worker and overrides the "vite-manifest.json" file created for the application, as noted by the build output rendering chunks (10)...The emitted file "vite-manifest.json" overwrites a previously emitted file of the same name.
import MyWorker from './worker?worker'
const worker = new MyWorker()

Retrieve Worker as URL - Inlined w/ Wrong MIME Type
Alternatively, if you attempt to import a worker using the ?url query suffix, the build completes but is broken.  If you run npm run preview you'll see the worker is inlined as Base64, but the MIME type is wrong and set to data:video/mp2t (because .ts is a valid video file extension too).  This is mentioned in Issue 11823
import workerUrl from './worker?url';
const worker = new Worker(workerUrl, { type: 'module' });

So no matter which approach you take, there's no way to create an operational build with a Web Worker using SvelteKit currently.  This use to be operational.

Can confirm this is also happening to our project after updating SvelteKit

In version 1.15.0  the build no longer hangs but throws an error

@GavinSiver in version 1.15.0  the build no longer hangs but throws an error.

The error is occurring because the vite-manifest.json for the application is being overwritten by a new vite-manifest.json that is specific to the worker.  The build process attempts to look up an application file in the vite-manifest.json but the manifest only contains data for the worker, it no longer contains data for the application.
SvelteKit 1.15.0 Reproduction:
https://stackblitz.com/edit/sveltejs-kit-template-default-zzblkv?file=src/routes/+page.svelte

This appears to be a Vite issue, highlighted in the two issues below...
Vite Issue 10190
Vite Issue 12627
I haven't verified, but there appears to be a fix on the way:
https://github.com/vitejs/vite/pull/12661

In the meantime, adding this to the vite.config.js might help as a temporary workaround:
const config = {
        // ...
        // TODO: Remove once vite 4.3 is out
	worker: {
		plugins: [
			{
				name: 'remove-manifest',
				configResolved(c) {
					const manifestPlugin = c.worker.plugins.findIndex((p) => p.name === 'vite:manifest');
					c.worker.plugins.splice(manifestPlugin, 1);
					const ssrManifestPlugin = c.worker.plugins.findIndex(
						(p) => p.name === 'vite:ssr-manifest'
					);
					c.plugins.splice(ssrManifestPlugin, 1);
				}
			}
		]
	}
};
// ..

In the meantime, adding this to the vite.config.js might help as a temporary workaround:
const config = {
        // ...
        // TODO: Remove once vite 4.3 is out
	worker: {
		plugins: [
			{
				name: 'remove-manifest',
				configResolved(c) {
					const manifestPlugin = c.worker.plugins.findIndex((p) => p.name === 'vite:manifest');
					c.worker.plugins.splice(manifestPlugin, 1);
					const ssrManifestPlugin = c.worker.plugins.findIndex(
						(p) => p.name === 'vite:ssr-manifest'
					);
					c.plugins.splice(ssrManifestPlugin, 1);
				}
			}
		]
	}
};
// ..

I can confirm the workaround works in my project with the same problem. Thanks @dummdidumm

Vite 4.3-beta.2 has also been released with the fix above. There's also a ton of performance improvements in the release so would be good to bump as soon as possible too.

Upgrading to vite 4.3.1 fixed the issue for me

Closing as fixed as it's a Vite issue.


Linux support? [Resolved already. But official linux would be appreciated]

Geode Version
v2.0.0-Beta.20
Related Problem
Can you please add official linux support?
I Recently went back to linux mint. I Moved from windows to linux, To windows again,  then back. I Cant install geode though. I Tried installing it through proton in the same version, But it didnt work. The geode files exist and stuff, but geometry dash acts as if geode doesnt exist or anything.
Your Solution
Maybe make a linux .sh or something or anything that works with steam proton and linux? Idk.
Additional Information
No response

Nvm. But i would still appreciate an official linux version.

there isn't even official linux gd you have to use proton

Yeah but it can still access the files. But the exe has to be ran in proton, but it doesn't work unless you add advanced launch options. And I think maybe making an sh file that either modifies the steam launch options for it, or something else to make it simpler for other people.


ContentfulModel::Base doesn't provide a #hash method

ContentfulModel::Base doesn't provide a #hash method, so model instances cannot be effectively used as keys in Hashes, or easily filtered for duplicates.
For example, I have a Requirement model which has_one Category. After loading my requirements, I group them by category:
requirements.group_by(&:category)
  # => {<Category> => [<Requirement>, <Requirement>, …], …}

For the results of a single query, this works fine. However, sometimes I want to combine the results of two queries, to simulate a logical OR:
(some_requirements + other_requirements).uniq.group_by(&:category)

This throws up two problems, both arising from the lack of #hash:

Duplicate requirements from the two queries are different objects, and since they inherit the default implementation of #hash, they're not filtered out by #uniq.
Any category that was present in both sets of results ends up as two different keys in the resulting Hash, again because they are different objects.

Possible solutions
In my app, I added #hash and #eql? methods based closely on ActiveRecord's implementation:
  def eql?(other)
    super || other.instance_of?(self.class) && id.present? && other.id == id
  end

  def hash
    if id.present?
      self.class.hash ^ id.hash
    else
      super
    end
  end

Hey @georgebrock,
Thanks for the suggestion, I'm currently implementing the upgrade to the 2.0 CDA SDK and implemented a similar solution to this.
The release will be coming up in the next few weeks, with the initial PR coming out tomorrow.
Cheers


BitsAndBytesConfig Error when Installing Requirements

Running the latest version of ComfyUI and the custom nodes (as of 7/21) on Windows and the requirements installation is throwing this error:
File "X:\ComfyUI\ComfyUI\custom_nodes\Zuellni\Nodes\IF.py", line 42, in Load_Encoder
    "4-bit": BitsAndBytesConfig(
TypeError: BitsAndBytesConfig.__init__() got an unexpected keyword argument 'device_map'

Cannot import X:\ComfyUI\ComfyUI\custom_nodes\Zuellni module for custom nodes: BitsAndBytesConfig.__init__() got an unexpected keyword argument 'device_map'

This does cause the nodes to not import:
 0.4 seconds (IMPORT FAILED): X:\ComfyUI\ComfyUI\custom_nodes\Zuellni

It looks like there was something wrong with extra indexes in the requirements, try updating the repo and installing requirements again.

Sadly I'm still getting this error after pulling the most recent version. If it's just me then maybe there's something weird with my particular config - but from what I can tell the error is being caused by the device map argument in the IF.py around line 42
    _MODELS = {         "4-bit": BitsAndBytesConfig(             device_map="auto",             load_in_4bit=True,             bnb_4bit_use_double_quant=True,         ),         "8-bit": BitsAndBytesConfig(             device_map="auto",             load_in_8bit=True,         ),         "16-bit": None,     }

I moved device_map outside BitsAndBytesConfig, that might help though I'm not sure why I'm not getting this error at all.


CocoaLumberjack project does not build bitcode enabled artifacts

Hi,
The project file for CocoaLumberjack doesn't seem to have the proper settings for bitcode enabled.
I had to modify the project file to include:
-fembed-bitcode inside every non OSX target to obtain proper bitcode enabled artifacts.

Could you pull request those changes?

I ran into this, too, but I can't figure out what setting to fix. I'm using the Cocoalumberjack/Swift pod

Bitcode is enabled inside our project targets. For CocoaPods, you can control this as well. Everything should work (now that bitcode enabled is the default Xcode setting).

The issue is still there - but it is a little more subtle. Although all targets have "Bitcode enabled" (except on MacOS) in the Shared.xcconfig - any target (my Framework for instance) linking against CocoaLumberjack-static, will fail to build-for-profiliing, with this error:  ```
ld: bitcode bundle could not be generated because '/Users/myUser/Library/Developer/Xcode/DerivedData/mySDK-eporqzissbedhfazciqqygcaiqjx/Build/Products/Release-iphoneos/libCocoaLumberjack.a(DDASLLogger.o)' was built without full bitcode. All object files and libraries for bitcode must be generated from Xcode Archive or Install build file '/Users/myUser/Library/Developer/Xcode/DerivedData/mySDK-eporqzissbedhfazciqqygcaiqjx/Build/Products/Release-iphoneos/libCocoaLumberjack.a' for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)


fix: mark amount of ingredient as 0 when amount is not specified.

Change description

When an ingredient has a quantity of 0, remove the quantity from the ingredient, such as @salt{}.
Add "pinch", "cheese", "ground", "powdered", "seeds" to single_stop_words.
Add small_amount_words list to convert certain unit quantities from 0 to 1, such as @salt{1%pinch}.
Check if list length is greater than 1 for single_stop_words filter.

Related issues

Fixes #22

Additional information
N/A

@bubonicfred please review.

Heh, this fixes one of the other things I was going to raise from the Tikka Masala recipe:

@ground{1%tsp} @almonds{50%g} Counts almonds as 2 ingredients, is possibly confused with the ground cumin or coriander in the ingredients list, which go in under 'spices' NB: in new branch renders as ground @almonds{50%g}
It also means the coriander lower down comes out as @coriander{1%tsp} instead of @coriander{0} like before, this is technically incorrect in terms of the recipe.
(it's supposed to be some leaves) but this is most likely a limitation of recipe scraping.

@bubonicfred are you okay with the merge?

@nicholaswilde yes


ssl configuration

Hi
Looking for ssl configuration with hydra, Is it possible to configure ssl certificates?

Hello @getsachmail, configure in what sense? Can you provide a detailed use case?

Need https end point on all microservices insted of http, As of now we have configure hydra-router as https service and all internal microservice as http which is working good for us.

@getsachmail thanks. closing this issue.


pkg/achclient: remove useless idempotencyKey branch

Codecov Report

Merging #103 into master will increase coverage by 0.08%.
The diff coverage is n/a.


Unable to get ACTIONS_ID_TOKEN_REQUEST_URL env variable when PR from a fork to upstream

The ACTIONS_ID_TOKEN_REQUEST_URL env variable appears to be unavailable if this action is run on a pull request from a fork into an upstream branch.
It works perfectly fine when ran in the fork itself, or if the pull request is from a branch belonging to the upstream repository itself. It could be my mistake with configuration, or just a misunderstanding of how the action is supposed to work on my part. It'd be great if a fix could be provided for this, or at least a workaround where for this specific situation the step is skipped.
Here is an example of a run with the situation I'm describing: https://github.com/RimSort/RimSort/actions/runs/9338649185

Hi, and thanks for trying out our feature.
What a puzzling error! Glancing over your workflows, it sure looks like you're doing everything right.
That said… this error smells like a permissions issue, and once I read "from a fork" my spidey sense started tingling; for security & abuse prevention reasons, workflows in forks have all sorts of not-necessarily-intuitive behaviour.
I looked this up in our public docs and found the following link: https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs#changing-the-permissions-in-a-forked-repository

You can use the permissions key to add and remove read permissions for forked repositories, but typically you can't grant write access.

Which would explain why the workflow is not fetching the id-token. Is it possible that this setting is not enabled?
Caveat emptor: I suggest thinking carefully about what you might be enabling if you turn this feature on, i.e. you might inadvertently allow project outsiders to push packages / you might want to consider how to adequately scope permissions or workflow triggers (which I see some evidence of in the build file). I myself am not an expert on how to structure this properly, and am just raising the possibility.

Thanks for the suggestion! It does seem like a permission issue. I think a potentially safe way of doing this would be to just disable the attestation step for this specific case? Or at least let it safely fail only in this specific case. I'm not sure how to do this such that it safely fails in the use case of a PR into upstream from a fork but not within a fork itself.

I think a potentially safe way of doing this would be to just disable the attestation step for this specific case?

Without looking into your project specifically, my recommendation is as follows:

you want to attest any built artifacts that are distributed to end-users
often we build artifacts as part of the pull request process
but this model is kinda awkward in the context of open pull requests, SO:

What if you move the build artifact step to a workflow that is fired after merges are made to your main branch? That way you step over any permissions issues since it only fires in the context of your repo, not in the fork, and after a project maintainer has blessed it.

(if you're building artifacts in a pull request where the tokens don't have write access this might be wasted effort/compute anyways 🤔, cos the artifacts will by definition be thrown away)


nextjs parameterized-routing issue with multiple parameters

I tried parameterized-routing example for my project I need to declare 3 parameters so I add 2 more in this example, I am not getting correct way. I made few changes in example:
index.js

<Link href='/blog?id=first&uid=1&vals=abc' as='/blog/first/1/abc'>My first blog post</Link>

<Link href='/blog?id=second&uid=2&vals=xyz' as='/blog/second/2/xyz'>My second blog post</Link>

<Link href='/blog?id=last&uid=3&vals=aaa' as='/blog/last/3/aaa'>My last blog post</Link>

my Blog.js

export default class extends React.Component {
static getInitialProps ({ query: { id,uid,vals } }) {
return { id,uid,vals }
}
render () {
return

 My {this.props.id} blog post
uid ={this.props.uid} vals ={this.props.vals}

Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod
tempor incididunt ut labore et dolore magna aliqua.

} }

Issue is when I navigate this through index.js it works fine, but when I directly put route with parameter on addressbar it shows error page could not be found. What should I need to do for impliment multiple . parameters, For my project I share links to user's email with dynamic parameter, and I want user can direcly reach to his dynamic content on my website with that link.

https://github.com/zeit/next.js/issues/2833#issuecomment-414919347


Type-safety and autocomplete?

💬 Questions and Help
I can't use v3 in production yet due to some recently reported issues, but I had a question about Typescript usage. The types overall seem too flexible to guarantee that you're using existing tokens. I understand falling back to CSS, but it makes it harder to guarantee consistency and correctness.

Using object syntax, you can see things like CSS's global colors:

Using x.div, color doesn't autocomplete at all.
Using css string literals, there's no guarantee. I don't expect anything here; writing CSS syntax almost doesn't seem worth it these days.

I really appreciate the work and it certainly makes styled-components easier to use. In the long-term, however, I'm starting to consider options like stitches.dev since it seems hard to guarantee type-safety and DX in an automagic fashion. I guess I'm wondering if it's worth overriding the types or if there's anything else in the roadmap to solve these issues?

Hello @pstoica, xstyled is flexible by nature and fallback to CSS. If you need something stricter, then it is probably not the good solution for you.


Edited the space between the status bar items

This PR fixes ##145852
I appreciate feedback if something is missing.

@misolori I tried to do the tests locally after this review. However, I wasn't quite successful with that.
https://github.com/microsoft/vscode/wiki/Smoke-Test
The only way to target the space before the element is by using the before pseudo-selector. The only other alternative that comes to my mind is to change the class name here.
.monaco-workbench .part.statusbar > .items-container > .statusbar-item  span.codicon-mark-github::before { 	content: " "; 	white-space: pre; }
Again this may cause overwriting of other contents as I was not able to do the test locally.
If this is the right direction to go, then I can definitely work on the issue otherwise I don't have other suggestions or solutions at the moment.

Unfortunately that isn't a scalable solution as an extension can add multiple icons next to each other so it's not specific to this one use case. So we'd need a more generic solution here.

@misolori Is it possible to close this PR so that the issue remains open for others to contribute? Thank you.


Create a new ghost role that allows ghosts to possess some lizard plushies

About the PR

This PR adds a new variant of the lizard plushie that ghosts can possess. This plushie cannot move and can only speak (not limited like animals). All speech is accompanied by a Weh! sound.
Why / Balance

I wanted to add more ghost roles and saw a video of someone pretending to be a plushie with a chameleon projector and thought this would be funny and fit in with the spirit of space station 14
Technical details

This PR adds a new entry to the grand lottery to have a 1 in 100 chance to spawn a possessed lizard plushie
It also changes the way bulk lizard plushie crates work, but it appears to be unchanged to players. This allows me to add a 1 in 10 chance to spawn a possessed lizard plushie when a crate is bought.
These crates are the only way to obtain a possessed lizard plushie
Media

Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: The plushie factory seems to have been infiltrated by Nanotrasen's enemies and have trapped innocent souls within some of the lizard plushies. If any of these plushies are discovered, please report them to your local chaplain for an exorcism.

>give any item a ghost role
>call it possessed
>profit :trollface:

give any item a ghost role >call it possessed >profit :trollface:

When the wehpocalypse comes, I am making sure I will be spared

There is this PR allowing PAIs to be put inside plushies, which has the some effect of the plushie being able to talk.
#30805

There is this PR allowing PAIs to be put inside plushies, which has the same effect of the plushie being able to talk. #30805

These two systems together could create some funny rp where the pai is "inside" the plushie's head.
And does the pai system change the sound of the pai to match the plushie? Like how the plushie in this pr makes a weh! sound when talking?

i dont think this is needed, and if admins want to do shenanigans they can just create ghost role in one button. I think a better solution would be something like the pai pr mentioned before.

Agreed with Emisse
Meh

also, it has been discussed more than once that cluttering the tour with uninteresting options without gameplay is a bad idea, and we, on the contrary, try to remove such roles


Add Vietnamese language support

I want to add Vietnamese language support. I want to work on this issue. Would you please assign me to it?

Hi Susan, I'm working on updating the app and it's dependencies. There are some build errors right now which I'm sorting out. Once that's done, I'd love your help with Vietnamese localization. Thanks.


[SuggestionProvider] Suggest using a remote cache, if not done yet

If the profile looks like remote caching was not used, suggest setting it up. The suggestion mentions both --remote_cache and --disk_cache as options that you can combine, and links to Bazel documentation.
Fixes #127

Example output:
Suggestion: "Use remote caching"
Consider using remote caching to speed up builds: https://bazel.build/remote/caching
There are two kinds of remote caches you can set up. The first is configured with the Bazel flag `--remote_cache` and leverages a separate service for caching, which usually runs on a different machine. The second cache is configured with the Bazel flag `--disk_cache` and uses your local disk. It is possible and in many cases recommended to use both `--disk_cache` and `--remote_cache` together.
Rationale
A remote cache allows you to share build outputs between different Bazel clients. This can make builds significantly faster.


Increase electronic temperature for Fermi smearing when using xTB

Electronic temperature
Increasing the electronic temperature used for Fermi smearing (fractional occupation according to Fermi statistics) might improve the convergence in electronically challenging and less defined cases during early optimization steps with xtb.

I'd say that this would be a valuable extension and probably very easy to implement (we just have to connect the CLI command and toml entry with the flag for the xtb calculation).
Would you like to look into this, @jonathan-schoeps?

Yes i can have a look into this.


python310Packages.sqlalchemy-jsonfield: cleanup checkInputs

Description of changes

Things done

Built on platform(s)

[ ] 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.11 Release Notes (or backporting 22.05 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

[ ] Fits CONTRIBUTING.md.

@ofborg eval

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

4 packages built:

apache-airflow (python310Packages.apache-airflow)
python310Packages.sqlalchemy-jsonfield
python39Packages.apache-airflow
python39Packages.sqlalchemy-jsonfield


add metadata.json to apisnoop bucket that matches specified structure

Request:
A metadata.json in the same bucket as our endpoints.json, tests.json, etc. that matches the structure outlined below.
Short description:

Current state:    "job-version":  String,
    "master_os_image": String,
    "infra-commit": String,
    "node_os_image": String

- Desired state:
  ```json
    {
      "job": String,
      "bucket": String,
      "version": String,
      "job-version": String,
      "master_os_image": String,
      "infra-commit": String,
      "node_os_image": String,
      "pod": String,
      "passed": Boolean,
      "result": String,
      "timestamp": Number
    }

Merged in #188


[PAGC-1757] Add aoi argument to Reader::tile() to support clipped tiles

Reminder: Public repo
What / Why
This PR re-introduces the changes from #1 to support an aoi argument to the Reader::tile() method. This work is being done in tandem with SenteraLLC/titiler#3 to modernize the SenteraLLC fork of TiTiler.
I also wrote a bit of content in a new wiki: https://github.com/SenteraLLC/rio-tiler/wiki
QA Strategy

[ ] Merge Latest Master

QA 👍


Duplication Dragon egg with Claim's

My version are
spongeforge-1.12.2-2555-7.0.0-BETA-2758
LuckPerms-Sponge-4.0.36
griefprevention-1.12.2-<IP_ADDRESS>5
I have no mods installed just the plugins i listed above
Here how i do it i first get a ender dragon egg (vanilla)
and i make a large claim then go into survival
place the 1 egg in your claim. Then Left Click it but do not right click it
you most likely have to do that 2 time's then you will see the Egg on top and bottom essentally  2 eggs
within the same location but just 1 on top and 2nd on bottom
EDIT: also this don't happen without doing it on claims. and i made a fresh test server with all default config

This is a sponge bug when cancelling interact event. Report it to SpongeCommon.


Updated third party libraries

Codecov Report

Merging #7 (e98e318) into master (6a3e934) will increase coverage by 0.01%.
The diff coverage is 87.35%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
+ Coverage     86.27%   86.28%   +0.01%
- Complexity    15419    15642     +223
============================================
  Files          1005     1017      +12
  Lines         57631    58680    +1049
  Branches       5877     5950      +73
============================================
+ Hits          49720    50632     +912
- Misses         4857     4944      +87
- Partials       3054     3104      +50

Impacted Files
Coverage Δ

...va/io/pravega/client/admin/ReaderGroupManager.java
85.71% <ø> (ø)

...in/java/io/pravega/client/admin/StreamManager.java
0.00% <ø> (ø)

...client/control/impl/ControllerResolverFactory.java
82.85% <ø> (+1.37%)
:arrow_up:

...ava/io/pravega/client/state/impl/UpdateOrInit.java
100.00% <ø> (ø)

...in/java/io/pravega/client/stream/EventPointer.java
100.00% <ø> (ø)

...ssor/requesthandlers/AbstractRequestProcessor.java
82.81% <0.00%> (-1.32%)
:arrow_down:

...c/main/java/io/pravega/controller/store/Scope.java
25.00% <ø> (ø)

...oller/store/kvtable/ZookeeperKVTMetadataStore.java
63.82% <0.00%> (-2.84%)
:arrow_down:

...a/segmentstore/server/ContainerEventProcessor.java
100.00% <ø> (ø)

...tore/server/containers/StreamSegmentContainer.java
89.64% <0.00%> (-0.80%)
:arrow_down:

... and 139 more

Continue to review full report at Codecov.

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


reassess actual dependencies

Currently, requirements.txt is a bit over-zealous with versions: expan doesn't actually need the latest version (0.17) of scipy for example.
To increase backward compatibility, we should go over what's actually required. Potentially long-winded operation I guess but perhaps there's a tool to do it semi-automatically?

I'm not aware of that kind of tools, would be glad to learn about them.

@gbordyugov The current idea is to automatically generate a bunch of requirements.txt with different combinations of library versions, and do some kind of grid search to check the dependency.


Fix composer require example

Hey mate, sorry, just saw this. The package moved to a different namespace, but in any case, thank you for noticing the error!


Feature: More output format options

Is your feature request related to a problem? Please describe.
The generated table output is not always clear. Sometimes the lines break inconsistenly which dificults readability
Describe the solution you'd like
I would like to have an option to generate the output results in file or concise object form as JSON, so it could be saved or used as output from github workflows for example.
Additional context
I would like to help on the development of this improvement if it makes sense and could be achieved.

It's supported: run --format json, you can pipe it into a file as --format json > results.json

Hello @laurentsimon,
Thanks, that's exactly what I needed. I missed this in the documentation I'm sorry. I'm closing the issue.


[feature] How about allowing to modify the id attribute?

Hey mate, I see you're going a nice way allowing people to modify specific column names. How about allowing to modify and configure the id attribute? For example, I would love to change it to id_xxxxxx to match the rest of the schema.
Cheers,
P.

@ptheofan Hi, you can use any primary key field name without any troubles and without reconfiguring behavior.


NIFI-6350 Flow Controller Startup Error Misleading

When a node is starting, it displays a misleading/incorrect error message. It says, "Cluster is still in the process of voting on the appropriate Data Flow.", even when the cluster has already agreed on/voted on a Data Flow. This message shows up until the flow is fully initialized.
Updated to show a more generic error message that the flow is still initializing.
Description of PR
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Reviewing...

Thanks Patrick. Merged & closed.


[Feature] When the intelligent stop execution savepoint fails, use the most recent checkpoint as the savepoint

Purpose of the pull request.

手动执行保存点失败时，可以把最近的检查点作为保存点，用户通过最近一次保存点启动时，如果保存点没执行成功，就会从最近检查点启动


Code of Conduct file added

I have added.. please accept pull request.

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

sorry I can't

If without signing the agreement you can accept the pull request then its fine else leave.

Hi @sahilsahu246,
Do you mean that the link is not working or that you do not want to sign the CLA?
Thanks!
Elmer

actually its not working..means not redirecting..
On Thu, Oct 26, 2017 at 2:44 AM, Elmer Thomas<EMAIL_ADDRESS>wrote:

Hi @sahilsahu246 https://github.com/sahilsahu246,
Do you mean that the link is not working or that you do not want to sign
the CLA?
Thanks!
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/email-templates/pull/30#issuecomment-339473627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXnFSs4jXfhf9vme0S0SZ-shGlvKoc3cks5sv6SkgaJpZM4QGpQ3
.

--
Sahil Kumar Sahu

can the PR still be accepted..without signing
On Thu, Oct 26, 2017 at 3:17 AM, Sahil Kumar Sahu<EMAIL_ADDRESS>wrote:

actually its not working..means not redirecting..
On Thu, Oct 26, 2017 at 2:44 AM, Elmer Thomas<EMAIL_ADDRESS>wrote:

Hi @sahilsahu246 https://github.com/sahilsahu246,
Do you mean that the link is not working or that you do not want to sign
the CLA?
Thanks!
Elmer
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sendgrid/email-templates/pull/30#issuecomment-339473627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXnFSs4jXfhf9vme0S0SZ-shGlvKoc3cks5sv6SkgaJpZM4QGpQ3
.

--
Sahil Kumar Sahu

--
Sahil Kumar Sahu

@sahilsahu246 Can you please try again? I am able to get to the cla now.

This is already done by #7, anyway.


Tokens.verify returns an AuthyException rather than a Token if non-numeric or wrong length

Verifying a token returns:

Token object if the entered token is:

correct OR
incorrect AND is 6-10 numeric digits

AuthyException if the entered token is:

non-numeric
has <6 or >10 digits

This behavior is inconsistent for the following reasons:

Suppose an Authy app has the OTP set as the default 7 digits. It does not make sense for the response for entering an incorrect 9 digit code to be different from entering an incorrect 5 digit code. The client should also not be expected to hardcode a 6-10 digit validation.
AuthyException is hard to handle because it does not have a status or error code like the Token does, and does not match the responses in the Authy API documentation (https://www.twilio.com/docs/authy/api/one-time-passwords#verify-a-one-time-password).

Since the library cannot access what the expected number of digits in the OTP is for an Authy app, one potential solution is to create a Token object that has the same status code as the normal "incorrect token" response and add the current message in the AuthyException as the message.

Hi @mzhaiclh thank you for creating an issue. I think we should improve the consistency of our response.
We should only return a Token object if the OTP is correct in all other cases we should return an exception.
As you pointed out, currently the exception doesn't have all the info needed to properly identify the error, I think we should add it.

Awesome, thank you so much! :)


Make amount of rules to disable configurable

Follow-up 1f2284db0630807613ebe39c6b240153600f1d48
Summary of the changes / Why this improves CrateDB
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

I changed my mind on this one, once I tried to migrate more tests. This is useful to configure.


[hotfix] Bump all flink-1.18.0 to 1.18.1

Purpose

Linked issue: close #xxx

Tests

API and Format

Documentation

+1


Pascal Coin Wallet Error

The wallet was working fine but today when I open it, it display this error message I restarted and still it gives me this error.
" An error occurred during initialization. Application cannot continue:
Cannot open your wallet... Perhaps another instance of Pascal coin is active!
Invalid TWalletKeysExt steam
Application will close... "
EDIT: I uninstall and reinstalled it but still the same.
I have 1 account in the wallet what should I do ?

Delete all files and folders in AppData folder EXCEPT WALLETKEYS.DAT

In here right ?
C:\Users\Username\AppData\Roaming\PascalCoin
I tried but it didn't work , When I move WalletKeys.dat to other folder it worked but all my account and keys are gone... Luckily I exported my private keys before and imported it back to wallet....
But this is a big bug in the wallet what if I didn't back it up before ?
Not sure why but WalletKeys.dat got corrupted .
Please take a look and fix this bug..
Thanks for your help

The message says that "another instance is active", that means that your system is running the same App twice. That's not allowed.
Simply close other app or restart system.


Fix spelling & punctuation in device type contracts

patch: Fix spelling of while
patch: Fix spelling of eMMC
patch: Fix punctuation in partials

@acostach Can you just confirm if the changes made in the second commit are correct, that's all.

@balena-ci I self-certify!


remove deprecated errorMessage property

The errorMessage property from the ISummaryNack interface has been deprecated and will be removed from common\lib\protocol-definitions\src\protocol.ts.
All uses of errorMessage must be replaced with message.
Closes #8275

Please validate (in debugger) that indeed we get proper message, not errorMessage (for SPO & r11s)


Roads not syncing up

A friend and I connected to each other.
We can't see any roads the other person built, but money and building sync up fine.

Road syncing is not implemented yet. But that's probably the next major thing that will be worked on.

Roads do sync now, but upgrading or demolishing roads doesn't sync.

Only creation of roads has been implementet, the rest still needs some coding :)

What's the status of this issue now? @junkmail88 are you still having any issues with road syncing?


win loose message

Change dialogue message functionality so when you win or lose, the dialogue box buttons show 'Main Menu' and 'Reset Game' instead of simply 'Okay'.

F16 OK (50 pts)


Question on use-case

Sorry to open an issue for this.
I'm just curious, what's the reason I'd want to use Void over ()? Don't they serve the same purpose?
I've seen Void used in a few iron examples. I've replaced this in my own code with (), so I'm curious as to the reasoning behind its usage.
fn sample() -> Result<String, ()>;
// works just as well as
fn sample() -> Result<String, Void>;

impl SomeTrait for MyType {
    type Error = Void;
    // or
    type Error = ();
}

Not sure if I'm missing something, or if its just a style choice?

@gchp Sorry for not getting back to this, for some reason I wasn't watching this repo.
The difference between () and Void is that there are values of type (), whereas it is impossible to construct a value of type Void in code that is reachable.


Configure chainvine admin to redirect to Giveth all projects page.

Configure chainvine admin to redirect to Giveth all projects page. (one general link for all referral links).
Note: Chainvine devs will implement users to configure to the donate page of a specific project per link.

@MoeNick this is also working I configured the admin, but hmm ill report an issue.
I think we shouldn't allow users to change the redirect url, its bad ux and complex for them.
I don't see a way to disable it.

leaving in progress in the meantime


feat: allow test to be asynchrone

Description
allow use async in tests
Motivation and context
I'm using bruno/cli for integration testing and want to put some asynchronous stuff in the tests to allow queries to be run after getting the results to check data consistency.
How was this tested?
I added a new test file for test-rutime in bruno-js

This is great! Merged!
Many Thanks @LesageYann


nd-codegen not working v22.27.0

nd-codegen shows this error message after update
Unhandled exception. System.InvalidOperationException: The requested operation requires an element of type 'Object', but the target element has type 'Null'.
at NetDaemon.HassModel.CodeGenerator.Model.JsonExtensions.g__getServiceFields|1_1(String service, JsonElement element, <>c__DisplayClass1_0& ) in /home/runner/work/netdaemon/netdaemon/src/HassModel/NetDaemon.HassModel.CodeGenerator/Model/JsonExtensions.cs:line 60
at NetDaemon.HassModel.CodeGenerator.Model.JsonExtensions.g__getServices|1_0(JsonElement element, <>c__DisplayClass1_0& ) in /home/runner/work/netdaemon/netdaemon/src/HassModel/NetDaemon.HassModel.CodeGenerator/Model/JsonExtensions.cs:line 40
at NetDaemon.HassModel.CodeGenerator.Model.JsonExtensions.ToServicesResult(JsonElement element) in /home/runner/work/netdaemon/netdaemon/src/HassModel/NetDaemon.HassModel.CodeGenerator/Model/JsonExtensions.cs:line 27
at Program.<$>g__GetHaData|0_2(HomeAssistantSettings homeAssistantSettings) in /home/runner/work/netdaemon/netdaemon/src/HassModel/NetDaemon.HassModel.CodeGenerator/Program.cs:line 91
at Program.<$>g__GetHaData|0_2(HomeAssistantSettings homeAssistantSettings) in /home/runner/work/netdaemon/netdaemon/src/HassModel/NetDaemon.HassModel.CodeGenerator/Program.cs:line 95
at Program.$(String[] args) in /home/runner/work/netdaemon/netdaemon/src/HassModel/NetDaemon.HassModel.CodeGenerator/Program.cs:line 37
at Program.(String[] args)

Thank you for reporting the issue. Some questions.
Did you try going back to previous version and generate code. If so what is the latest working version that works for you?
Have you tried restarting HA prior to generate code?
Thanks for your help!

Last working version was 22.23.0 and the errored occurred after updating  to 22.27.0 without any other known change. After restoring a HA backup with the Netdaemon 22.23.0 and rebooting, codegen still doesn't work.

Please try the 22.27.2 release to see if your problem is fixed. Thanks!

Last working version was 22.23.0 and the errored occurred after updating to 22.27.0 without any other known change. After restoring a HA backup with the Netdaemon 22.23.0 and rebooting, codegen still doesn't work.

Can you please try the latest version 22.23.1?

Last working version was 22.23.0 and the errored occurred after updating to 22.27.0 without any other known change. After restoring a HA backup with the Netdaemon 22.23.0 and rebooting, codegen still doesn't work.

Can you please try the latest version 22.23.1?

Thank you for addressing the issue so quickly, unfortunately I still get the same issue even after after updating to version 22.27.2 of the home assistant addon and the codegen dotnet tool and rebooting all devices.


Removing old logic from EditSubmissionFragment

We don't plan to keep the EditSubmissionFragment around for the newer version of Ground, instead the edit flow will be handled by the Data Collection flow. This PR removes the logic which has already been replicated in the Data Collection flow while leaving the remaining logic for now for easy reference as we build out that functionality.
Towards #1078

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Prepare for Darwin.SR1

Pull actuator endpoints via spring-cloud/spring-cloud-app-starters-maven-plugins#27

Pull Tensorflow changes via spring-cloud-stream-app-starters/tensorflow#13

Both issues are addressed.


Closing integrated terminal makes UI hang

Issue Type: Bug
Since updating to today's (5/7) Insiders Build, I've run into an issue where closing the integrated terminal will cause VS Code to hang and pop a dialog asking me to reopen or wait. The problem is intermittent, but I frequently open and close the integrated terminal and have run into the issue 3 or 4 times today.
 Just hit the issue again.
Version 1.24.0-insider
Commit 8a07c8d935998d1b1153dde0c5b337189d35cea0
Date 2018-05-08T05:16:07.736Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

@martellaj when this happens next could you try run code-insiders --status from the command line and paste in the output?

@Tyriar Yes, I'll give it a go. Doing it in a terminal outside of VS Code will work, right?

@martellaj as long as a terminal is responsive (in a different window?)


bug: the sbcs on my ipad did not has an futb bin solutions and i can't solve it automatically by bot

Prerequisites

[X] I have searched for existing issues that already report this problem, without success.

Script Version
1.3.1
Platform
ios
Current Behavior
I cant solve abc automatically

Expected Behavior
.

Steps to Reproduce
.
Additional Information
.

Fixed


XPop Service Dies...

the server can run for some time and then when I tries to get the updated UNL from https://vl.xrplf.org/

Obtained ledger (binary) store/0/83/647/550 83647550 TX# 54
Obtained ledger (JSON object) store/0/83/647/550 83647550 Hash EF1AFB7DC1835EFFE9E2AD45A4802EB64DE9EB60408ABCB598C0F12C167930A9
Obtained ledger (binary) store/0/83/647/551 83647551 TX# 46
Obtained ledger (JSON object) store/0/83/647/551 83647551 Hash CBBA9EBB488B9290F39FF3DCA1E2B9F770E019EED84DF17A0B5B93AF290D5591
file:///usr/src/app/node_modules/node-fetch/src/index.js:108
			reject(new FetchError(`request to ${request.url} failed, reason: ${error.message}`, 'system', error));
			       ^

FetchError: request to https://vl.xrplf.org/ failed, reason:
    at ClientRequest.<anonymous> (file:///usr/src/app/node_modules/node-fetch/src/index.js:108:11)
    at ClientRequest.emit (node:events:514:28)
    at TLSSocket.socketErrorListener (node:_http_client:495:9)
    at TLSSocket.emit (node:events:514:28)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  type: 'system',
  errno: 'ETIMEDOUT',
  code: 'ETIMEDOUT',
  erroredSysCall: undefined
}

Node.js v20.9.0
Running backend version 3.1.0
Started Event Socket Service at TCP port 3000
<<<<< CONNECTING >>>>>
* * * CONNECTING wss://xrplcluster.com
* * * CONNECTING wss://s2.ripple.com
Fetched UNL https://vl.xrplf.org found validators 35
Obtained ledger (binary) store/0/83/647/553 83647553 TX# 51
Obtained ledger (JSON object) store/0/83/647/553 83647553 Hash<PHONE_NUMBER>D44776736548C92942A4FB9EA55B265B91B7CDB57C059DE3AD9F01
Obtained ledger (binary) store/0/83/647/554 83647554 TX# 39
Obtained ledger (JSON object) store/0/83/647/554 83647554 Hash 3EEAF5D99D5B9878C9236F33AD04F91374903E66DB60014107CA6D9463A1FC1C

it falls over and restarts

@lathanbritz I checked my logs on three different nodes and this never happened.
I suspect it's something local, DNS, ... fetching the URL (?)
I'll update the code to prevent it falling over if this happens and keep on using the old record, but I think this indicates some connectivity issues on either end of the connection, this isn't something we can actually fix in the code :( (Except for the falling over)

Rippled reports same issues in its log mainnet, alloy as I understand knows about the intermittent issue on the UNL fetch.

Even more interesting, so then there's an issue with the hosting provider (Netlify) I assume.

@lathanbritz Nice catch. I updated the code (publishing in a sec) to deal gracefully with not being able to fetch UNL;

If first call (and nothing known): error out and restart
If follow up call & previous results known: continue and try again every now and then


FIX Floating-point calculation in provision.sh

Line 24 in k8s/debug/provision.sh will not work in bash as bash does not support floating-point calculations. I changed that line and now the calculation is done using bc.

What platform are you on where you spotted this issue? I haven't seen this come up on archlinux, MacOS and WSL.

It comes up on Ubuntu with bash version 5.0

Ubuntu 18.04  or 20.04? Im just going to give this a try on different platforms before merging.

Pretty sure it's 20.04

Platform Checks

[ ] Arch Linux
[ ] Ubuntu 20.04
[ ] MacOS
[ ] WSL

Hmm ok, I'm seeing it calculate 15.50000000000000000000Gi on a 32 GiB system. I mean it is taking that number, but I don't think that is what we want. What are the memory resources on the machine you saw an issue on @AlpacaMax ?

So I saw this issue on a virtual machine with 4GB memory (it shows 3.8GiB in free -h). As a result it will try to compute 3.8 / 2, which gives me an error. After changing it to the version I wrote the calculation succeeded. Then minikube tells me my machine doesn't have enough resource for itself to run.

Yeah unfortunately minikube debugging requires quite a bit of resources to work. Sometime soon we may be creating a debug cluster that we can get you access to. We'll still merge once I can confirm it works on all the main things.


Student page should be independent of sidebar

Currently, there is a bug where a student will disapper when is it not visible in the sidebar. This is because studentpage depends on the data in studentStore, which may or may not contain the student, due to an applied filter, or not being loaded yet (for instance, when the student would be at page 5).
This was not an issue earlier, because the selected student was stored as a whole entity in projectStore.currentStudent. However, that breaks websockets.
I think the best solution here is to create something like projectStore.currentStudent and store it as a separate entity in the store, or store it in the state of studentPage. The studentPage could take a Student as an optional prop and thus borrow it from the sidebar, and if not given, should fetch the student. This is needed, because students can also be accessed with a hyperlink from a project (and may not be loaded into studentStore).
The one thing I'm not sure about is how this will work out with websockets. @Zelzahn , do you have any idea about that?

I added a redirect in #402 . Why did you need the current student in the store?

I added a redirect in #402 . Why did you need the current student in the store?

This has been discussed in a call between Lisa and me, for a better explanation

To be honest I'm not entirely following. Websockets should be working in #341, so this shouldn't be of issue anymore?


fix(ghost_text): ephemeral-like inline ghost-text workaround

This is a workaround to get inline ghost-text to work as expected without using ephemeral extmarks. See refs for context.
Refs:

https://github.com/hrsh7th/nvim-cmp/pull/1645
https://github.com/neovim/neovim/issues/24797

ping

@hrsh7th Please merge this PR to fix the ghost_text rendering. Thank you so much.

Thank you. I understand it.


OSC getLivePreview implementation

Hey
I need to implement the getLivePreview command from Google Open Spherical Camera API
https://developers.google.com/streetview/open-spherical-camera/reference/camera/getlivepreview
What i need to do is a POST with some data that will tell the endpoint i need the getLivePreview stream ... the output is a video stream of MOTION Jpegs at 15fps ... the pb is that i need a way to display them in javascript by executing a POST request (any other method will not work)
is there a way this library can help me implement that?
thanks

Hi @wowzaaa , this is very interesting, from my understanding if you send a POST request it should return something like this
---osclivepreview---
[frame data]

---osclivepreview---
[frame data]

...

---osclivepreview---

Then you can extract frames by split the data with boundary, for example

RNFetchBlob.fetch('POST', 'http://some.endpoint', headers, body)
.then((res) => {
   let frames = String(res.text()).split('---osclivepreview---')
   renderFrames(renderFrame)
})

basically it's possible, but I think there will be some performance issue when split and render the frames if the image resolution is high 😕

so frames[frames.length - 1] will have the latest frame right?
u mean because of the size of the response buffer?
will then get executed on every change of the stream? every 66 ms (  1 000 / 15 = 66.. ) ?

i will try to implement this and get back to u ... until now i could not get even a console log from the success function (i did it the old way ... without promises)

Well, after some investigation I think multipart/x-mixed-replace data will never reaches an end, each part will be considered as replacement of previous part, and that's why the success function never got called. If we're going to support this I need do some more tests, wondering if there's any way I can have a working sample for this ?

well in order to have that u need a one shot camera like a ricoh theta s, gear 360 or lg 360 ... that will be the only way to replicate the incoming stream

okay .. I will write a video streaming service to imitate the multipart/x-mixed-replace response, and see what can I do 👍

u'r a star .. i am here to test 💃 :P

but u understand what i am saying right?

"agent": "", // string - Path to ssh-agent's UNIX socket for ssh-agent-based user authentication. Windows users: set to 'pageant' for authenticating with Pageant or (actual) path to a cygwin "UNIX socket." Default: (none)

the question becomes ... how is the agent option useful to me? (a pageant user)

Hi @wkh237 , I'm having some trouble with the part response in ios, basically I'm updating an image component with the part data adding the code in the state and using it as the image source uri, it was working fine but now I'm having a "error for object 0x7fff4725a000: double free" in a breakpoint at
if(self.isServerPush)
    {
        [partBuffer appendData:data];
        return ;
    }

any idea on how to solve?
thanks

@fabio0  hi ,
Have you solved this problem?

@zzq1023
Hi, no I built a bridge for the Ricoh theta sdk for ios using a native View for the live streaming


update: add sphinx build ci

Codecov Report

:exclamation: No coverage uploaded for pull request base (update-0.3.0-test@7075496). Click here to learn what that means.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##             update-0.3.0-test       #3   +/-   ##
====================================================
  Coverage                     ?   86.97%
====================================================
  Files                        ?       44
  Lines                        ?     2711
  Branches                     ?      364
====================================================
  Hits                         ?     2358
  Misses                       ?      335
  Partials                     ?       18

Continue to review full report at Codecov.

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


class generation (class, constructor, attributes, methods)

These warnings correspond to changes in Generator.java
15:31:58.287 [main] WARN org.apache.velocity.deprecation - configuration key 'resource.loader' has been deprecated in favor of 'resource.loaders'
15:31:58.290 [main] WARN org.apache.velocity.deprecation - configuration key 'classpath.resource.loader.class' has been deprecated in favor of 'resource.loader.classpath.class'
15:31:58.290 [main] DEBUG org.apache.velocity - Initializing Velocity, Calling init()...
15:31:58.290 [main] DEBUG org.apache.velocity - Starting Apache Velocity v2.3
....


chef-solo --no-fork broken in Chef 12.0.3

(Just for context, trying to make chef-server use latest chef)
I get the following when running in --no-fork mode:
chef-solo --no-fork -l debug -c /opt/opscode/embedded/cookbooks/solo.rb -j /opt/opscode/embedded/cookbooks/dna.json
[2015-01-16T02:09:33+00:00] FATAL: NoMethodError: undefined method `size' for nil:NilClass

The above command runs fine if I remove the --no-fork part of the command.
Also looks like someone recently reported a similar issue via stackexchange.
Try as I might, I cannot get any more error output or a stack. It just straight crashes with that message.

DUP https://github.com/opscode/chef/pull/2710

Thanks.


Fix #189

This is a fix for the issue #189 about the xlim that did not really adapt to the selected curves because of the horizontal dotted black line.

I pulled the PR and it works as expected. thanks a lot @tanglef


iOS / tvOS 15 compatibility

The standard player UX completely changes in tvOS 15. Maybe we can get rid of some tricks, most notably to reload the info panel.

The reload trick has been removed on tvOS 15+. Still custom info panel do not reload without closing and reopening the player, but this is likely something we can report to Apple, or which might be fixed in a tvOS update.


Issue/build master fix inmanta core postgresql versions issue2 wrap api call in retry limited

Description
Wrap get_resources_for_agent API call with a retry_limited so that the test no longer fails if the agent didn't obtain a session yet by the time the call is made.
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box
- [ ] Attached issue to pull request

[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
- [ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Pull request rejected by merge tool. Failed to parse change entry file: Change entry /tmp/tmp2mt9ye89/changelogs/unreleased/build-master-fix-inmanta-core-postgresql-versions-issue2-wrap-API-call-in-retry-limited.yml does not match pydantic schema: 1 validation error for ChangeEntry
sections
none is not an allowed value (type=type_error.none.not_allowed)

Processing this pull request

Merged into branches master in<PHONE_NUMBER>6b0c368caec5ccdb45e5d10f39a29a

Processing #4444.


Add Gotify as notification option

Gotify is an extremely lightweight notification project, I love it, so I added it as a notification option.
And also, this PR solved this discussion.

Hey, would anyone be able to take a look at this PR when you have a moment?
Thank you!

Looks good, thanks!


fix(bc): correct case sensitivity for binding constraint term IDs

Description
This PR introduces changes to ensure consistent handling of case sensitivity in binding constraint term IDs across the backend and frontend. It also includes additional tests to validate this behavior.
Backend
Updated the update_constraint_term function, to handle term IDs in a case-insensitive manner. This ensures that term IDs are processed consistently, regardless of their case.
Modified ID comparisons and processing logic to use lowercase versions of term IDs, aligning with the frontend changes for uniformity and reducing the likelihood of mismatches due to case differences.
Frontend
In the OptionsList component, normalized the cluster IDs to lowercase. This change allow the frontend to display clusters names for BC terms regardless of their ID case.

In some weird update case I get this error, the issue seems related to the frontend sending invalid data (as youcan see in the network tab):
{
	"key": "filterSynthesis",
	"value": ""
}

while trying to update the cluster of a term, we should investigate on this, but this rarely happens @laurent-laporte-pro


Security policy

For security-conscious and especially commercial entities, a SECURITY.md file which shows up in the Github Security tab is quite helpful.
You can set a SECURITY.md at the project level or the organization level. For example here's a couple projects which set it at the organization level:

https://github.com/nodeca/.github
https://github.com/pandas-dev/.github/blob/master/SECURITY.md

Those organizations currently use Tidelift, which might be helpful because it's a third-party which means it's easier to trust than an individual. But I don't know too much about that - I think any security policy would be helpful.

We are students at the Australian National University and are completing the assignment for comp6120. We are very interested in this issue. Can you assign this issue to us?


[SPARK-34271][SQL][3.1] Use majorMinorPatchVersion for Hive version parsing

What changes were proposed in this pull request?

This PR backports SPARK-34271 (#31371).
Why are the changes needed?

I'm personally checking whether all the tests pass with Java 11 for thecurrent master and I found HiveExternalCatalogVersionsSuite fails.
The reason is that Spark 3.0.2 and 3.1.1 doesn't accept 2.3.8 as a hive metastore version.
HiveExternalCatalogVersionsSuite downloads Spark releases from https://dist.apache.org/repos/dist/release/spark/ and run test for each release. The Spark releases are 3.0.2 and 3.1.1 for the current master for now.
https://github.com/apache/spark/blob/e47e615c0ede9692fd3aa1098155e92e4fb50b7f/sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveExternalCatalogVersionsSuite.scala#L239-L259
With Java 11, the suite run with a hive metastore version which corresponds to the builtin Hive version and it's 2.3.8 for the current master.
https://github.com/apache/spark/blob/20750a3f9e13a2f02860859f87bbc38a18cba85e/sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveExternalCatalogVersionsSuite.scala#L62-L66
But branch-3.0 and branch-3.1 doesn't accept 2.3.8, the suite with Java 11 fails.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Existing tests with CI.

@dongjoon-hyun I don't think this is a blocker for 3.1.2 but FYI just in case.
To solve the problem, another solution would be changing HiveExternalCatalogVersionsSuite itself for the current master.
What do you think?
If this solution seems better, I'll open a backport PR for branch-3.0 too.

@dongjoon-hyun All right. I have another solution so I'll close this and open another test PR.

Thank you so much, @sarutak !


Upgrade docs dependencies.

Description
It looks like the dependencies in docs had fallen behind. I'm going to be heading into a documentation overhaul so want to make sure my local dev server is fast with next@latest - but figured I'd bump a bunch of other stuff while I'm here.
CLOSES TURBO-1863
Testing Instructions
Navigate around the preview build and look for anything broken.

I'm merging through 3 CI failures here:

The Turbopack tests are only running because the lockfile changed. They are irrelevant to the changes in this PR.
The formatting check is failing due to code rot. These issues will be fixed in an upcoming PR.


fix bugs in pregexp look-behind matching

handle newline properly
respect non-zero start position
use string output ports in replace functions for efficiency
tighten signatures for multiple-arity procedures
use brackets instead of parentheses where appropriate

I noticed this morning that when I ran file pregexp.tex, I got "UTF-8 Unicode text", which is different from all the other .tex files. There may be other lingering UTF-8 characters that I missed.

Thanks, I will look for non-ASCII characters in there and get it cleaned up soon. I may have time yet today.

In Emacs elisp, I ran (occur "[^[:ascii:]]"), and it showed the places you found and no others. Good job!