Configuration apply error from applying 2 Prometheus plugins against 1 Service fails to be parsed

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Configuration apply error from applying 2 Prometheus plugins against 1 Service fails to be parsed
Expected Behavior
No response
Steps To Reproduce
Given the following 2 plugins:

apiVersion: configuration.konghq.com/v1
config:
  latency_metrics: true
kind: KongPlugin
metadata:
  name: prometheus-1
plugin: prometheus
---
apiVersion: configuration.konghq.com/v1
config:
  latency_metrics: true
kind: KongPlugin
metadata:
  name: prometheus-2
plugin: prometheus

Assuming a Service named httpbin, when user annotates the Service (following https://docs.konghq.com/hub/kong-inc/prometheus/how-to/basic-example/ guide)
k annotate svc httpbin konghq.com/plugins=prometheus-1,prometheus-2

the error from Kong fails to get parsed in KIC and hence no event is generated:
KIC logs:
debug   Sending configuration to gateway clients        {"v": 1, "urls": ["https://<IP_ADDRESS>:8444"]}
debug   events  failed to apply Kong configuration to https://<IP_ADDRESS>:8444: failed posting new config to /config: got status code 400    {"v": 1, "type": "Warning", "object": {"kind":"Pod","namespace":"kong","name":"kong-controller-758d4d7bb5-mz4d6","apiVersion":"v1"}, "reason": "KongConfigurationApplyFailed"}
debug   No configuration change, skipping sync to Kong  {"url": "https://<IP_ADDRESS>:8444", "v": 1}
debug   Due to errors in the current config, the last valid config has been pushed to Gateways  {"v": 1}
error   dataplane-synchronizer  Could not update kong admin     {"error": "performing update for https://<IP_ADDRESS>:8444 failed: failed posting new config to /config: got status code 400"}

Acceptance Criteria

[ ] Services, Ingresses (and other k8s resources that can produce the problem) using multiple plugins of the same type in their konghq.com/plugins annotation are rejected

Kong Ingress Controller version
No response
Kubernetes version
No response
Anything else?
No response

Currently, events are generated properly

logs[ingress-controller] 2024-05-06T08:18:28Z        debug   Sending configuration to gateway clients        {"v": 1, "urls": ["https://<IP_ADDRESS>:8444", "https://<IP_ADDRESS>:8444"]}
[ingress-controller] 2024-05-06T08:18:28Z        debug   events  failed to apply Kong configuration to https://<IP_ADDRESS>:8444: failed posting new config to /config: got status code 400        {"v": 1, "type": "Warning", "object": {"kind":"Pod","namespace":"kong","name":"ingress-kong-5cb656bbf8-g997d","apiVersion":"v1"}, "reason": "KongConfigurationApplyFailed"}
[ingress-controller] 2024-05-06T08:18:28Z        debug   events  invalid plugin:prometheus: uniqueness violation: 'plugins' entity with primary key set to '4aeffa30-fd1b-5b9e-80d7-2dc1b0180f92' already declared       {"v": 1, "type": "Warning", "object": {"kind":"KongPlugin","namespace":"default","name":"prometheus-2","uid":"0d6617ff-8b4f-4d6e-844b-18fac946587f","apiVersion":"configuration.konghq.com/v1"}, "reason": "KongConfigurationApplyFailed"}
[ingress-controller] 2024-05-06T08:18:28Z        debug   No configuration change, skipping sync to Kong  {"url": "https://<IP_ADDRESS>:8444", "v": 1}

events...
default     2m59s       Warning   KongConfigurationApplyFailed   kongplugin/prometheus-1                   invalid plugin:prometheus: uniqueness violation: 'plugins' entity with primary key set to '4aeffa30-fd1b-5b9e-80d7-2dc1b0180f92' already declared
default     3m59s       Warning   KongConfigurationApplyFailed   kongplugin/prometheus-2                   invalid plugin:prometheus: uniqueness violation: 'plugins' entity with primary key set to '4aeffa30-fd1b-5b9e-80d7-2dc1b0180f92' already declared
...
kong        7m2s        Warning   KongConfigurationApplyFailed   pod/ingress-kong-5cb656bbf8-g997d         failed to apply Kong configuration to https://<IP_ADDRESS>:8444: failed posting new config to /config: got status code 400
kong        2m2s        Warning   KongConfigurationApplyFailed   pod/ingress-kong-5cb656bbf8-g997d         failed to apply Kong configuration to https://<IP_ADDRESS>:8444: failed posting new config to /config: got status code 400
...

So the part of improving the validation webhook to reject such invalid configuration only left


The close icon missed and html: true not work in the mini js file

Hi,
　　I got 2 issues in the course of my use:
1. when I include bootstrap.css file, the close icon X will not show. It means if closeConfirm: true, user will have no where to click to close the notify.
　　　2. Based on issue #4, I saw you added html: true to solve it. It did work when I use overhang.js, but seems still not work the mini file: overhang.min.js.
BTW, the version of bootstrap.css is 3.3.5, and the browser is Firefox v36.0.4.
Anyway thank you for sharing this pretty plugin :blush:!

Hey,
Thanks for reaching out! All the issues you have listed above have been fixed in the latest commits! I hope that helps!
Thanks,
Paul

hi Paul,
Both are passed ,thanks a lot!

Glad to be able to help!


[Enhancement] Remove varlen key limitation of persistent index

What type of PR is this：

[ ] bugfix
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/10469
Problem Summary(Required) ：

We support varlen key for persistent index for now, so we can remove the limitation for varlen key.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

run starrocks_admit_test

@mergify backport branch-2.4

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio update


Build: net8.0 - multitarget v6 library and update docs and tests

Description

Update library to contain a net8.0 targeted version.
Update the docs and tests to use net8.0
The majority of the code changes are in response to compiler warnings.
net8.0 Docs can be seen here
https://mudblazor-test.azurewebsites.net/
How Has This Been Tested?

Build
Browse docs
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

I did some exploring with https://mudblazor-test.azurewebsites.net/ and it looks all good.

We now also have a way to confirm the runtime the docs are running on:

_jsonSerializerOptions.AddContext<XYZContext>();

should be replaced to
_jsonSerializerOptions.TypeInfoResolver = JsonTypeInfoResolver.Combine(XYZContext.Default)

I fixed the obsolete method (thanks @ScarletKuro ).  https://mudblazor-test.azurewebsites.net/ is down due to over quota.  Seems like the free versions in Azure are very restrictive these days. Will reset in 3 hours


Allow cipher strings to be given using its standard name

Added pull request for issue #16169

[x] documentation is added or updated
[x] tests are added or updated

OTC: This is fine for master, but cannot be backported to 3.0 or 1.1.1

Could you please rebase to fix the conflict in CHANGES.md?

Could you please rebase to fix the conflict in CHANGES.md?

Done!

I now realized there is a CLA: trivial on the first commit. Could you please remove that via git rebase -i master and force push it?

I now realized there is a CLA: trivial on the first commit. Could you please remove that via git rebase -i master and force push it?

Done, did it turn out ok?

I've not seen your ICLA.  When did you send it?

@eriklax Please follow the instructions at https://www.openssl.org/policies/cla.html to send the ICLA.

@eriklax Please follow the instructions at https://www.openssl.org/policies/cla.html to send the ICLA.

[x] ICLA sent by email

The CI failures in the "minimal" builds look relevant.

Also please rebase to fix the CHANGES.md conflict.

The CI failures in the "minimal" builds look relevant.

Changed cipher in test

Also please rebase to fix the CHANGES.md conflict.

Done

This pull request is ready to merge

Merged to master branch. Thank you for your contribution.


Unit Production Spawn Location and Thanks :)

Hi,
Just want to say I am a huge fan of what you are doing here, and have been playing around, using your exposed values in blue print to add a percentage text render above buildings on their construction. I am not a Programmer however and blueprint is where my talents end.
A bug or more of a feature quest that immediately springs to mind is that there are no rally points for the buildings. Because of that, an unmoved spawned unit will stand next to the building and when another is spawned it will spawn in the same place causing the collisions to fight.
The offset/world location is also not exposed.

Thanks Again For your time spent on this, the components method is so much cleaner than other examples I have seen on the unreal marketplace :)

Hey @cptnight00!
Thanks for all the warm words! That's what keeps me going :)
In our current project, we already have rally points in place, and spawning in a circle around the building in case blocked spawn positions as well. I just haven't had the time to merge everything back yet.

Great to hear! :)

Added rally points in commit eb7f1d231c0bf44e1f5047e8cd7260d325ee6e19.


Data export: JSON

JSON format support needed in data exporter.

Done.
Fix will be included in 3.5.2

Can I export the Project (all queries) in the JSON format?
To save into github I need to convert all the queries in the JSON format, is there a way to do it?


When the app is full screen, the mouse moves to the top without feedback

Describe the bug
When the app is full screen on my mac, the mouse moves to the top without feedback
To Reproduce
Steps to reproduce the behavior:

Click the full screen button
Move the mouse to the top of the screen

Expected behavior
https://user-images.githubusercontent.com/45113206/132062041-6cd76710-18b9-490c-8654-ea82fd8123cf.mov
Screenshots
https://user-images.githubusercontent.com/45113206/132062197-82be8dec-960b-4a4e-909f-7e51beec2eaf.mov
Platform and Versions (required):

Operating System - Mac OS, version 10.15.7 X64

Node.js environment
  Node.js - 14.17.3
  @tauri-apps/cli - 1.0.0-beta.10
  @tauri-apps/api - 1.0.0-beta.8

Global packages
  npm - 7.21.1
  yarn - Not installed

Rust environment
  rustc - 1.54.0
  cargo - 1.54.0

App directory structure
/node_modules
/public
/src-tauri
/src

App
  tauri.rs - 1.0.0-beta.8
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - ../public
  devPath - ../public

Additional context
Add any other context about the problem here.
Stack Trace

related to tauri-apps/tao#188

fixed in tao's next branch and will be available in the next release.


Contact Mechanics crashes for the rough sphere of the contact.engineering paper

Same here for a different topography

@mcrot - can you check what is going on?

Yes, I'll have a look today.

I think the problem is the following: The new version of SurfaceTopography has version number 1.0 and does
not report the micro field. This was done for the versions before:

In my code I haven't checked that the micro field could be None. On each start a new database entry for this dependency is generated - this should not be allowed, I did not expect that. On retrieval of the on entry the result is then not unique as expected (several "None"s for micro field) and this generates an exception.

It would be good to to next release of SurfaceTopography with version number 1.0.1 (micro filled).
Additionally I will look for a quick fix which sets the field to zero, when missing.

Actually I've just deleted the double entries and it seems to work now. The "None" value should be handled in a similar way than 0. I'm not sure why the double values existed.

@sannant Now it seems to work. Could be check please?
Not sure yet how these double entries were created - the code should not allow this.
Maybe two stacks in parallel? One  started with "docker-compose" instead of "docker-stack"?

It is fixed thanks


Skype SDK does not load today

Hello,
I am having problems loading the Skype SDK. This is something that worked fine some days ago and even a few times yesterday morning.
Skype SDK does not load when I try to test the log in example:
<!doctype html>
<html>
<head>
  <title>My Skype Web SDK app</title>
  <script src="https://swx.cdn.skype.com/shared/v/1.2.15/SkypeBootstrap.min.js"></script>
</head>
<body>
  <script>
    Skype.initialize({ apiKey: 'a42fcebd-5b43-4b89-a065-74450fb91255' }, api => {
      var app = new api.application;
      app.signInManager.signIn ({
        username: '***',
        password: '***'
      }).then(() => {
        console.log("signed in as", app.personsAndGroupsManager.mePerson.displayName());
      }, err => {
        console.log("cannot sign in", err);
      });
    }, err => {
      console.log("cannot load the sdk package", err);
	  alert("cannot load the sdk package", err);
    });
  </script>
</body>
</html>

The network request that fails is: https://latest-swx.cdn.skype.com/jLync/master_0.4.1028/sdk.js
I cannot reach that URL from my navigator.
Any help would be appreciated.

Sorry, it was a network problem


[BUG] MultiTurnPerplexityMetric

Describe the bug
https://github.com/thu-coai/contk/blob/e6e3d641766e4ae2111f41e742be206cc8684d2c/contk/metric/metric.py#L147-L150
Use multiturn as batch_size in sub_metric
Expected behavior
Add some comments to explain this.

Done in #76

Reopen because error message is not right

Done in #107


create Boyer-Moore Majority Voting Algorithm

• Folder structure: C++->Sub-Arrays->Boyer-Moore Majority Voting Algorithm.cpp
• I have used this algorithm to find majority element (>N/2) occurrences
🛠 Issue (Number)
Issue no #235
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I have added detailed explanation of the code as well as the algorithm.
📷 Screenshots

Please note that I am not contributing under SWOC .
I learned about the event through this repo.
So it is entirely up to you whether or not to include the relevant tags.

Please note that I am not contributing under SWOC . I learned about the event through this repo. So it is entirely up to you whether or not to include the relevant tags.

Thanks for informing that


[Onboarding] Splash screen w/Tari disco ball

From #22

Create splash screen with Tari spinning loop video

Issue is complete.


Gh 125 update jackson

Related Issue

Resolve #125

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop-v2@177cdd8). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop-v2     #208   +/-   ##
=============================================
  Coverage              ?   81.16%
  Complexity            ?     1535
=============================================
  Files                 ?      185
  Lines                 ?     4274
  Branches              ?      430
=============================================
  Hits                  ?     3469
  Misses                ?      627
  Partials              ?      178

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 177cdd8...495d01c. Read the comment docs.


WIP: update mocha to 6

[x] update mocha package version
[x] remove co-mocha (does not support mocha 6) and generators in tests
[ ] update new configs

@DavertMik , I look into mochaFactory.
There are using of mocha with loadFiles and Runner.uncaught overriding.
In the ui.js file tags, inject, and others are added to Mocha.Test, Mocha.Suite and others.
Maybe, analogues CodeceptMocha/CodeceptTest/CodeceptSuite should be created as class extension instead of monkey patching Mocha classes themself?
Can do in this or other PR.

Maybe, analogues CodeceptMocha/CodeceptTest/CodeceptSuite should be created as class extension instead of monkey patching Mocha classes themself?

A good idea. That would be really nice!
Filenames can be like:
mocha/test.js
mocha/suite.js

I tried your branch locally. Works really great. Waiting to see this merged!

@elukoyanov are we ready?

It can be merged.
I'll make mocha class variation in other PR

Thank you!


Investigate Alizer Service Detection

/kind user-story
/kind epic
Which area this user story is related to?
/area alizer
User Story
As a developer I want to know specific types of services used inside an application, so that I can use this information in order to do automation actions, CI/CD etc.
An example, of this could be to detect types of databases used inside an application (e.g. PostgreSQL), or message brokers (e.g RabbitMQ). This way a developer can know which services should be also installed when using the application.
Acceptance Criteria

[x] https://github.com/devfile/api/issues/1213

As the investigation item has been completed closing this epic


Use Textual-terminal?

https://pypi.org/project/textual-terminal/
This would allow the user to replay the question without interrupting the input.

Not mature enough.


[SymbolGraph] Filter @_spi declarations

These definitely shouldn't be showing up in symbol graphs with a minimum access
level of public, and it's not clear whether another minimum access level
should show these. For now, filter them unconditionally.
rdar://62081711

@swift-ci Please smoke test

@swift-ci Please smoke test OS X platform

@swift-ci Please smoke test


filter slows down on large datasets

Issue
When using filter in small datasets there is minimal difference in processing time between datar's filter and pandas query; however, when working with large datasets, datar's filter slows dramatically
Steps to replicate
df_test = pd.read_excel("https://archive.ics.uci.edu/ml/machine-learning-databases/00352/Online%20Retail.xlsx", dtype={"Description": str})
df_test >>= mutate(country2 = f.Country, country3 = f.Country, country4 = f.Country, custid2 = f.CustomerID, custid3 =  f.CustomerID, custid4=  f.CustomerID, invoice2 =f.InvoiceDate, invoice3 =f.InvoiceDate, invoice4 =f.InvoiceDate, stock2 = f.StockCode, stock3 = f.StockCode)
filtered_df = df_test >> filter((f.Description =="WHITE HANGING HEART T-LIGHT HOLDER" )| (f.Description =="REGENCY CAKESTAND 3 TIER") )
Note: I artificially created more columns to show the issue.
Snapshot comparing small dataset vs large dataset using datar's filter and pandas query

There could be cython functions called inside DataFrame.query(), but I need to confirm.
However, before that, we can confirm whether it is because of the optimization inside DataFrame.query() or somewhere else. Could you try this:
conditions = (df_test.Description =="WHITE HANGING HEART T-LIGHT HOLDER" ) | (df_test.Description =="REGENCY CAKESTAN
D 3 TIER")

%%time
filter(df_test, conditions)

%%time
df_test[conditions]

Here is the output

Looks like that 20x time was optimized inside .query(). One of the reasons might be the data in the conditions are from inside the data frame itself.
What if you do:
descr = df_test.Description.copy()
df_test.query('(@descr == "WHITE HANGING HEART T-LIGHT HOLDER") | (@descr == "REGENCY CAKESTAND 3 TIER")')

The condition aggregation is also eating the time up. There is an extra calculation:
# check condition, conditions
    condition = Array([True] * _data.shape[0])
    for cond in conditions:
        if is_scalar(cond):
            cond = Array([cond] * _data.shape[0])
        condition = condition & cond

    condition[is_null(condition)] = False
    try:
        condition = condition.values.flatten()
    except AttributeError:
        ...

    out = _data[condition]

that I used an initial array with all Trues. Let's try to use the first condition passed in as the initial condition and see if it helps.
Might need a prun to profile which steps are eating up the time.

See: https://pwwang.github.io/datar/performance/#Aggregating-results-on-single-columns

Closing this as datar.dplyr.filter() now achieves similar performance as DataFrame.loc[].


Remove cfg hacks for old MSRV

@bors r+

:pushpin: Commit 1ef7e87a56eced77e7949aa8ed4908418d819536 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 1ef7e87a56eced77e7949aa8ed4908418d819536 with merge a5dfaa316254c86c109c78b375d238d8bdb41dbc...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 061a7e371475c633283d35df23e11c2ce9d343d4 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 061a7e371475c633283d35df23e11c2ce9d343d4 with merge 6478ec66e562a7054125e4f1b0f30021028e2328...

:broken_heart: Test failed - checks-actions

FYI, it is fine on openbsd

@bors r+

:pushpin: Commit ba4892f42a00a55cfbeca29614c019177fe719ac has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit ba4892f42a00a55cfbeca29614c019177fe719ac with merge 09105c5ea71a96299a55eca17ec873feeaa250fd...

:broken_heart: Test failed - checks-actions

@bors r+

:hourglass: Testing commit 7a4517bbfc46d8744be64d867eb7dd40dbcd4353 with merge 298b0a7ddcfa14747c6271268e57a61db1d22135...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit e2e6fd69f49925f95fd6b493217a7e44ef7564cd has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit e2e6fd69f49925f95fd6b493217a7e44ef7564cd with merge a0d9be16d818e16aa450d97839f51be7a4d8f714...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing a0d9be16d818e16aa450d97839f51be7a4d8f714 to main...

It's very surprising to me that there's no discussion or reference here to what the new MSRV should be. Was this discussed in an issue? If so, please link the issue in the PR description.
In my opinion it would be nice to stick to 1.63 for the time being, matching Tokio and a bunch of other downstreams that would like to support stable Debian.

This change only applies to the v1.0 branch (main) which is not released yet. The v0.2 branch still has the old MSRV.

Maybe the version should be changed on the main branch to avoid further confusion?


max sockets (not per host) ?

Is it possible? I don't want to modify any globals and I'd prefer not to write a queuer.

Actually, such a scenario would be better served with a rate limiter (such as rate-limit).


Add section on this in order to explain the new used in Promise section

Title is self explanatory, that part of the guide bothers me because I tried to only use concepts that were already previously presented but new just comes out of nowhere to punch people in the face on this example :(
let p = new Promise(resolve => {
  resolve(2)
})

this section might look like it's only about a bad part but I could use it to introduce partial application using .bind which is pretty cool

hey, i know you closed my issue, but just wanted to mention that my PR for classes mentioned the this and new operator.

Thanks @dewball345 I was starting to write this part I'll take a look into it!!

Its not fully completed but has most of it


Adding support for other platforms (ie: Odysee, Kick, etc.)

Would be nice to add support for newer platforms if possible.

Will you please make individual requests per platform so that users can ask for support on a per-platform basis? Thanks!


[ServiceBus] Fix #24741: Remove invalid arguments for entity update commands

Related command

az servicebus queue update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection --enable-session
az servicebus topic update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection
az servicebus topic subscription update --resource-group {} --namespace-name {} --name {} --enable-session

Description

The following properties of an entity cannot be updated post creation but the update the commands for entities have these arguments, which are invalid and fail as such

Partitioning (Queues & Topics)
Duplicate Detection (Queues & Topics)
Sessions (Queues & Subscriptions)

This PR removes the arguments for the update commands.
resolves #24741
Testing Guide

az servicebus queue update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection --enable-session
az servicebus topic update --resource-group {} --namespace-name {} --name {} --enable-partitioning --enable-duplicate-detection
az servicebus topic subscription update --resource-group {} --namespace-name {} --name {} --enable-session

The above commands currently fail with a 400 Bad Request, which is expected due to the invalid arguments that can't be updated post entity creation. With this PR, they should be removed from the documentation, command help, and error as invalid arguments directly.
History Notes

[ServiceBus] az servicebus queue update: Remove invalid arguments for queue update command
[ServiceBus] az servicebus topic update: Remove invalid arguments for topic update command
[ServiceBus] az servicebus topic subscription update: Remove invalid arguments for subscription update command

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

ServiceBus

@damodaravadhani Sure. I've added you as a contributor to my fork. Let me know what tests you have in mind and I could help out too, if needed.

Hi @PramodValavala-MSFT ,  this is breaking change and we only allow breaking change twice a year. Can you add deprecation and warnings for now and totally remove them in next breaking change window?

Oh I've already seen #24937 adding the deprecation warning. I'll close this PR then


Does arrangement in ReloadableViewLayoutAdapter need replace every time?

https://github.com/linkedin/LayoutKit/blob/master/LayoutKit/Views/ReloadableViewLayoutAdapter.swift#L91-L93
Say I have a table view with 1000 items, then I insert 1 item in it, it will regenerate the whole arrangement...
Could we just replace the updated items?

It does regenerate all arrangements. Now that BatchUpdates is a parameter (it wasn't initially), it would be possible to do the optimization you suggest. This optimization can only be done though when BatchUpdates is non-nil (otherwise we don't know what has changed).

@nicksnyder Sure, what I want to do is avoid regenerate all arrangements when possible.
I would like to send a PR about this if you agree. Thanks.

Yeah, a PR would be welcome, thanks!


BUG: display_schema page has bugs after schema change

Page: https://datalad.github.io/datalad-catalog/display_schema.html
Click on metadata_sources schema => no display.
Console output:
Failed to load resource: the server responded with a status of 404 ()
display_schema.js:519 WARNING: schema file could not be loaded:./assets/jsonschema__metadata_sources.json
/favicon.ico:1          Failed to load resource: the server responded with a status of 404 ()
vue.js:634 [Vue warn]: Error in render: "TypeError: Cannot read properties of undefined (reading 'title')"

(found in <Root>)
warn @ vue.js:634
vue.js:1906 TypeError: Cannot read properties of undefined (reading 'title')
    at Proxy.eval (eval at createFunction (vue.js:11698:14), <anonymous>:3:2545)
    at Vue._render (vue.js:3572:24)
    at Vue.updateComponent (vue.js:4082:23)
    at Watcher.get (vue.js:4494:27)
    at Watcher.run (vue.js:4569:24)
    at flushSchedulerQueue (vue.js:4327:15)
    at Array.<anonymous> (vue.js:1998:14)
    at flushCallbacks (vue.js:1924:16)
logError @ vue.js:1906
vue.js:634 [Vue warn]: Error in render: "TypeError: Cannot read properties of undefined (reading 'title')"

(found in <Root>)
warn @ vue.js:634
vue.js:1906 TypeError: Cannot read properties of undefined (reading 'title')
    at Proxy.eval (eval at createFunction (vue.js:11698:14), <anonymous>:3:2545)
    at Vue._render (vue.js:3572:24)
    at Vue.updateComponent (vue.js:4082:23)
    at Watcher.get (vue.js:4494:27)
    at Watcher.run (vue.js:4569:24)
    at flushSchedulerQueue (vue.js:4327:15)
    at Array.<anonymous> (vue.js:1998:14)
    at flushCallbacks (vue.js:1924:16)

This was fixed in subsequent changes to the display-schema content, but the fix was not explicitly documented.
NOTE: The display schema functionality could perhaps be useful to refactor and extract into its own package at some undefined point in the future. But this is not a priority for future datalad-catalog developments.


GPTChromaIndex indexing fails when documents contains an empty file

trying to index a folder containing an empty document fails when using GPTChromaIndex. reproduced easily by creating an empty doc in a folder of files, and at least by using simpledirectoryreader.
2023-03-21 12:14:31.215 Uncaught app exception
Traceback (most recent call last):
File "/lib/python3.10/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 565, in _run_script
exec(code, module.dict)
File "/github/ChatObsidian/pages/Index_to_chroma.py", line 106, in
build_chroma_index(documents, collection, reindex)
File "/github/ChatObsidian/utils/chroma.py", line 52, in build_chroma_index
index = GPTChromaIndex(documents, chroma_collection=_chroma_collection, embed_model=embed_model, prompt_helper=prompt_helper)
File "/lib/python3.10/site-packages/llama_index/indices/vector_store/vector_indices.py", line 500, in init
super().init(
File "/lib/python3.10/site-packages/llama_index/indices/vector_store/base.py", line 63, in init
super().init(
File "/lib/python3.10/site-packages/llama_index/indices/base.py", line 114, in init
self._index_struct = self.build_index_from_documents(documents)
File "/lib/python3.10/site-packages/llama_index/token_counter/token_counter.py", line 86, in wrapped_llm_predict
f_return_val = f(_self, *args, **kwargs)
File "/lib/python3.10/site-packages/llama_index/indices/base.py", line 286, in build_index_from_documents
return self._build_index_from_documents(documents)
File "/lib/python3.10/site-packages/llama_index/indices/vector_store/base.py", line 206, in _build_index_from_documents
self._add_document_to_index(index_struct, d)
File "/lib/python3.10/site-packages/llama_index/indices/vector_store/base.py", line 186, in _add_document_to_index
new_ids = self._vector_store.add(embedding_results)
File "/lib/python3.10/site-packages/llama_index/vector_stores/chroma.py", line 71, in add
self._collection.add(
File "/lib/python3.10/site-packages/chromadb/api/models/Collection.py", line 79, in add
ids = validate_ids(maybe_cast_one_to_many(ids))
File "/lib/python3.10/site-packages/chromadb/api/types.py", line 71, in maybe_cast_one_to_many
if isinstance(target[0], (int, float)):
IndexError: list index out of range
Seems more like a Chroma issue really. but I guess a workaround could be made either client side or on llama index.

Should be a quick fix!

This is fixed in the latest versions of llama-index
>>> from llama_index import GPTVectorStoreIndex, Document
>>> doc = Document('')
>>> index = GPTVectorStoreIndex.from_documents([doc])
>>>

>>> from llama_index import GPTVectorStoreIndex, Document
>>> index = GPTVectorStoreIndex.from_documents([])
>>>


Add tests for embedding services configurations classes

Currently there are no tests for embedding services configurations related java files under the package io.stargate.sgv2.jsonapi.service.embedding.configuration and so the changes like the ones in this PR https://github.com/stargate/data-api/pull/1783 don't get any test coverage.

hold on this for a little - we need to refactor all of the vectorize work in January for tables we can solve it then.


Carthage install not working for v 0.7.0

carthage update --platform iOS
returns an error:

Dependency "Player" has no shared framework schemes for any of the platforms: iOS
If you believe this to be an error, please file an issue with the maintainers at https://github.com/piemonte/Player/issues/new

hey @JakubMazur i added a shared framework scheme last year, let me know if you still encounter issues. thanks again.


Hi

Hi Roman. It is will be nice if such layer visualizer will be implemented in Android Studio.

Wouldn't that be cool! Probably not a relevant bug for this project, but I'll chat with the Studio team.


website: 403 Forbidden from golang.org

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.6.2 linux/amd64

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/nik/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?
$ go get -v github.com/revel/revel
Fetching https://golang.org/x/net/websocket?go-get=1
Parsing meta tags from https://golang.org/x/net/websocket?go-get=1 (status code 403)
package golang.org/x/net/websocket: unrecognized import path "golang.org/x/net/websocket" (parsing golang.org/x/net/websocket: XML syntax error on line 9: expected /> in element)
Fetching https://golang.org/x/sys/unix?go-get=1
Parsing meta tags from https://golang.org/x/sys/unix?go-get=1 (status code 403)
package golang.org/x/sys/unix: unrecognized import path "golang.org/x/sys/unix" (parsing golang.org/x/sys/unix: XML syntax error on line 9: expected /> in element)

curl https://golang.org/x/net/websocket?go-get=1 -v

Hostname was NOT found in DNS cache
Trying 2a00:1450:400e:803::2011...
Connected to golang.org (2a00:1450:400e:803::2011) port 443 (#0)
successfully set certificate verify locations:
CAfile: none
CApath: /etc/ssl/certs
SSLv3, TLS handshake, Client hello (1):
SSLv3, TLS handshake, Server hello (2):
SSLv3, TLS handshake, CERT (11):
SSLv3, TLS handshake, Server key exchange (12):
SSLv3, TLS handshake, Server finished (14):
SSLv3, TLS handshake, Client key exchange (16):
SSLv3, TLS change cipher, Client hello (1):
SSLv3, TLS handshake, Finished (20):
SSLv3, TLS change cipher, Client hello (1):
SSLv3, TLS handshake, Finished (20):
SSL connection using TLSv1.2 / ECDHE-RSA-AES128-GCM-SHA256
Server certificate:

   subject: C=US; ST=California; L=Mountain View; O=Google Inc; CN=misc-sni.google.com

   start date: 2016-05-04 09:19:41 GMT

   expire date: 2016-07-27 08:39:00 GMT

   subjectAltName: golang.org matched

   issuer: C=US; O=Google Inc; CN=Google Internet Authority G2

   SSL certificate verify ok.

GET /x/net/websocket?go-get=1 HTTP/1.1
User-Agent: curl/7.38.0
Host: golang.org
Accept: /

< HTTP/1.1 403 Forbidden
< Content-Type: text/html; charset=UTF-8
< Content-Length: 1594
< Date: Wed, 11 May 2016 14:15:19 GMT
<

Error 403 (Forbidden)!!1

403. That’s an error.
Your client does not have permission to get URL /x/net/websocket from this server.  That’s all we know.
* Connection #0 to host golang.org left intact

What did you expect to see?
working go get

What did you see instead?
403 forbidden

ip ad
...
inet <IP_ADDRESS>
inet6 2a03:b0c0:2:d0::b6:1001
...

CC @adg @jbuberel

@acidnik does visiting https://golang.org work for you?

@adg "for me" as in "from my home machine"? Yes it does.
"for me" as in "from my virtual machne at digital ocean?" No, it doesn't. See curl output above

@acidnik Right. I am asking you to clarify that you get the same result with just "curl https://golang.org/" from that machine.

These commands:
curl -v https://golang.org/x/net/websocket?go-get=1
curl https://golang.org/

Return 200 OK for me tested from a couple of different DO instances (CoreOS, Ubuntu) - returns a redirect or a page respectively. So I don't think there is a general problem. Suggest this is closed unless @acidnik wants to pursue it further.

@broady, could you file an IP geolocation bug? (see my previous ones)

We've made some changes since May.
@acidnik are you still seeing this? Please reply/re-open if you are.


Enable stylelint-config-standard-vue

Which issue, if any, is this issue related to?

See https://github.com/stylelint/stylelint-ecosystem-tester/issues/39

Is there anything in the PR that needs further explanation?

stylelint-config-standard-vue supports Stylelint from version 14 and later.
However the test suite is setup for Stylelint 14 specifically: https://github.com/ota-meshi/stylelint-config-standard-vue/blob/f60920e8476c81b250ca4896e81c57bedb5d6d9e/tests/fixtures/integrations/stylelint-config-standard-scss/package.json#L7-L10
Given that they want to support older Stylelint versions I am unsure if this will ever reliably pass.

@ota-meshi should we keep stylelint-config-standard-vue?


Deprecation of user management via terraform

We maintain a decently sized userbase on aiven. We use this provider to create users and assign them to their projects and use a merge request based process so teams can manage their members mostly autonomously with supervision only from the userbase-owning team.
We recently started getting deprecation warnings for our terraform-maintained userbase. When investigating this, I assumed that the aiven_project_user resource had outstayed its welcome and we have to migrate to aiven_organization_user instead. I was surprised to learn that that resource is also deprecated.
Then I found in https://github.com/aiven/terraform-provider-aiven/commit/0dd160c1b7427898a2fc27f7d14641f590dcd9a3 that "Invitation of organization users is not supported anymore via Terraform". Then I got to the "We recommend to use WebUI to create and organization user invitation" part and I'm like: :open_mouth:
Am I missing something?
How is this not a giant step back? Are we really going back from "everything as code" to "use your mouse and browser" (mouse of all things)?
What is the rationale behind this change?

Hey, @fbreckle! 👋
We have the same idea that it should be done via Terraform, but unfortunately, the API was developed in a way that is not compatible with IaaC tools in mind at the moment.
I will forward this information to the relevant team. Thank you for raising this question!

Do we have more info on the API roadmap we are quite disapointed and we keep using the deprecated ressource but we hope aiven is not gona cut those API before finding a solution

@jean-mercier-hivebrite Could you please open an official support ticket. There are other solutions in place but I need more information about how you are using Aiven in order to recommend the right solution.


Use webpack for bundling. Fix popup callback.

For some reason, I couldn't get vite to properly bundle without erroring at runtime. Webpack seemed to work, so going with that for now.
vite assets are still configured if anyone wants to take a stab at it.

FYI, I published -pre release version of this, so I can work on the examples and docs.


Consider using AST-based TypeScript code generation

See e.g.: https://nabeelvalley.co.za/docs/javascript/typescript-ast/
This might also be useful to make pluggable code generators, e.g. generating code specifically for Freon.
Is it possible to use a ParseExpression feature like Roslyn has?

At first sight this looks harder than just using a (text) template. But maybe first sight is misleading :-).

(See also: https://github.com/microsoft/TypeScript/wiki/Using-the-Compiler-API#creating-and-printing-a-typescript-ast)

Two advantages of this approach is that we're already using an intermediate representation, and that potentially things like cross-file imports could become a bit easier.


pagination shows less than OSCAR_PRODUCTS_PER_PAGE

I have defined OSCAR_PRODUCTS_PER_PAGE = 21  in the first page, but in some cases,
it is giving me 19.

There's not enough information here to be able to help... you're counting products in Javascript so we would need to see your templating logic. Also are you sure you have more than 19 products in the result set?


Missing o1 models support

Are there plans to support o1 models: o1-preview and o1-mini? I suppose they use already existing encodings, so it is not a problem to add support for them.

Can you confirm that these models use the encoding o200k_base?

Unfortunately, I haven't found any confirmation. When I asked ChatGPT and Perplexity, they both answered that this is cl100k_base. Sure, that's not reliable. I think it's better to wait until it is known for sure.

I've proposed support for the o1-mini and o1-preview models by mapping the "o1-" prefix to the "o200k_base" encoding in pull request #19.
This proposal is based on the recent update in tiktoken (see: https://github.com/openai/tiktoken/compare/0.7.0...0.8.0#diff-088caf12b3c3144a63ff973b98bc3eec7e53f4c1d75d11d49720cbcd2744fd30R72), where these models were added.

Fixed in #19


Add Bluesky for Ella

Hi @EllaKaye,
I'm working on adding a Bluesky integration for the R-Ladies bot and saw that you're also on Bluesky. Please let me know

if you agree to add your handle here as well so that the bot can use it to tag you when posting about your posts
if you're also OK to be featured in an R-Ladies starter pack

Thank you ☺

Thanks! I've only just signed up to Bluesky and yet to post, but I do plan to start properly engaging with it soon so, yes, please go ahead and add the integration to the bot.
I'd be honoured to be part of the RLadies starter pack. Thanks!

Amazing, thank you so much! :)

Ref #97  - Should be added to starter pack


parallel wavenet

@twidddj ! hi ,  do you plan release parallel wavenet?

Hi @maozhiqiang, thanks to your interest our project!
Yes, we have a plan to release that part. Currently, we are almost ready to evaluate.
We hope the part shows up as soon as possible.

hi @twidddj ! thanks for your excellent work, I'm looking forward to seeing the performance of parallel wavenet！

@twidddj Do you get quality voice now by parallel wavenet?This week, i just can get reasonable voice but still some noisy.


Add Pyramid Patterns | Java

Add Code files for various pyramid patterns under the Java Folder Section

Hey @aryashah2k , I can work on pyramid patterns for java. I have previously worked on Java Wave patterns. you can assign this issue to me.

Hey @aryashah2k , I can work on pyramid patterns for java. I have previously worked on Java Wave patterns. you can assign this issue to me.

Hey @aryashah2k , I can work on pyramid patterns for java. I have previously worked on Java Wave patterns. you can assign this issue to me.

Hey @aryashah2k , I can work on pyramid patterns for java. I have previously worked on Java Wave patterns. you can assign this issue to me.

Hey there @trial1user , that's some really amazing work. Thanks a lot for working on this. I have successfully merged the PR. Grateful to have your contributions towards this repository!

Hey there @trial1user , that's some really amazing work. Thanks a lot for working on this. I have successfully merged the PR. Grateful to have your contributions towards this repository!


Update Master to Main

We need to update 'Master' to 'Main' and make certain it is standard in all the docs

This was resolved in #493


Registration function error

The registration function doesnt fire in IOS and Android with the phonegap build compiled app. In the phonegap test app all is working fine. What can I do to fix this?

Sorry to be a jerk but I'm now closing issues that don't follow the issue submission guidelines. This plugin takes a lot of work to maintain and I can't afford the time required to solicit the required info from everyone.
I'm closing this issue but please feel free to re-open the issue if you provide the requested details from the new issue submission guidelines.
https://github.com/phonegap/phonegap-plugin-push/blob/master/.github/ISSUE_TEMPLATE.md


Topic recognition

This branch implemented a superpixel model that is trained on ADNI dataset. The codebase include dependecy files and  python files for data preparation, model buidling, model training+validation_testing and model use example. The trained model and dataset is not included in the codebase. After 60 epochs of training, the model achieves mean PSNR of 28.82 with loss of 0.0013 for training; and mean PSNR of 27.56 for testing, which is higher than mean PSNR of arbitary lower resolution images (25.96).

This is an initial inspection - no action is required yet
Difficulty: Normal
General Comments: Nice approach overall, uses keras dataset functions that create a validation set. However, requires the user to manually extract AD and NC into one big folder. Code exists to compute metrics on the test set. Training plot looks good. Network architecture unchanged compared to provided Keras example.
[Recognition Problem]
"modules.py":

Reimplementation of the Keras example without changes.

"dataset.py":

Reserves 20% of training dataset for validation.
Based on readme and dataset code, student manually extracts AD and NC images into one big folder.
Seems to be a nice implementation all round.

"train.py":

Implements callbacks and early stopping to save best model.
Computes metrics on whole test set after training (and loading best model).
Reports metrics by printing (average over dataset)

"predict.py":

Loads model correct.
Computes metrics on all images in the test set and prints image-by-image.
Displays predictions and stores them.
Computes the average performance over whole test set. Based on their divide by 10, it seems they only had 10 images in their test set.

"README.MD":

Dependencies in separate text file (though it is referenced in the readme).
Very brief and limited introduction into the model architecture.
Reconstructed images in-line with other submissions.
Provides usage information.
Training/Validation plot looks good.

[Commit Log]
Commit Log looks ok.

Hello Dr. Shakes,
Thank you for your feedback. Could I please just double check that I do not need to make any change now since "This is an initial inspection - no action is required yet"? Following that, would there be a second round of feedback and if so when is the deadline to incorporate them? Thank you for your time.
Kind regards,
Shan

发件人: branme96 @.>
发送时间: 2023年11月28日 11:14
收件人: shakes76/PatternAnalysis-2023 @.>
抄送: ikai @.>; State change @.>
主题: Re: [shakes76/PatternAnalysis-2023] Topic recognition (PR #157)
This is an initial inspection - no action is required yet
Difficulty: Normal
General Comments: Nice approach overall, uses keras dataset functions that create a validation set. However, requires the user to manually extract AD and NC into one big folder. Code exists to compute metrics on the test set. Training plot looks good. Network architecture unchanged compared to provided Keras example.
[Recognition Problem]
"modules.py":

Reimplementation of the Keras example without changes.

"dataset.py":

Reserves 20% of training dataset for validation.
Based on readme and dataset code, student manually extracts AD and NC images into one big folder.
Seems to be a nice implementation all round.

"train.py":

Implements callbacks and early stopping to save best model.
Computes metrics on whole test set after training (and loading best model).
Reports metrics by printing (average over dataset)

"predict.py":

Loads model correct.
Computes metrics on all images in the test set and prints image-by-image.
Displays predictions and stores them.
Computes the average performance over whole test set. Based on their divide by 10, it seems they only had 10 images in their test set.

"README.MD":

Dependencies in separate text file (though it is referenced in the readme).
Very brief and limited introduction into the model architecture.
Reconstructed images in-line with other submissions.
Provides usage information.
Training/Validation plot looks good.

[Commit Log]
Commit Log looks ok.
―
Reply to this email directly, view it on GitHubhttps://github.com/shakes76/PatternAnalysis-2023/pull/157#issuecomment-1828904495, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZXI3FZRNXPDLHHGHF5E4VLYGU3GPAVCNFSM6AAAAAA73OWVFWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRYHEYDINBZGU.
You are receiving this because you modified the open/close state.Message ID: @.***>

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Normal -5
Commit Log
Meaningful commit messages sometimes -1
Progressive commits used
Documentation
ReadMe acceptable, no problem statement -1
Model/technical explanation minimal -1
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in InCorrect Branch) -2
No Feedback required
Request Description minimal -1

Wrong branch used, please update branch to correct one and ensure repo READMEs are restored. Does not affect grade only the merge of your PR.


I will not let me equip little nightmare

🐛 Bug Report
Write a description of the bug here, give all the information you can about what it is, how it happened.
I type out +equippet little nightmare and it keeps saying I do not have the pet. Even tho I obtained it from nightmare mass

That's not a pet, or you do not own this pet.

This will be fixed in the next update - thanks for reporting.


z doesn't record cd

I deleted ~/.z file but new file doesn't created and history of cd doesn't recorded. How to this it? Thanks.

Are you using bash or zsh
Are you sure you are sourcing and not executing z.sh

I'm using bash.
I've installed it with brew and added to ~/.profile – . /usr/local/etc/profile.d/z.sh

Please, help. I'm really want in back)
z --add /somepath working, z s also, but cd doesn't record where i'm going.

something is probably overwritting PROMPT_COMMAND or it's not getting added for some reason. you can look and see if the z --add stuff is there when you echo $PROMPT_COMMAND

Thanks a lot! That helped. Another program override prompt. Will be cool to add info about it to installation or troubleshooting doc's section.


Please invite me to the GitHub Community Organization

Please invite me to the GitHub Community Organisation.

Name: RTK
Discord Username: RTK#0296
Additional Context: Youtube

I'm not new to OSS..

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Invitation sent for the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/EddieJaoudeCommunity/people
Tips for practising:
Customise your GitHub profile, here is a live stream on it https://www.youtube.com/watch?v=cT6GXCuS0Zo
Practice repo, instructions on how to add your name to the README in the README https://github.com/EddieJaoudeCommunity/hacktoberfest-practice
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps


Exception on Windows Desktop platform.

Hello. I'm trying to add dualscroll and It's driving me crazy. I've found your package and it is just what I was looking for.
However, it rises an exception when you change windows dimensions.
It can be reproduced by maximazing the windows, scroll the content and change the windows width or height or change both dimension at once.
Is there an issue that can be solved?. I am newbie in flutter.
════════ Exception caught by rendering library ═════════════════════════════════
The following assertion was thrown during paint():
Offset argument contained a NaN value.
'dart:ui/painting.dart':
Failed assertion: line 36 pos 10: ''
....
.....
══════════════════════════════════════════════════════════════════════════════
════════ Exception caught by gestures library ══════════════════════════════════
'package:flutter/src/widgets/single_child_scroll_view.dart': Failed assertion: line 583 pos 18: 'transformed == position + -_paintOffset': is not true.
single_child_scroll_view.dart:583
════════════════════════════════════════════════════════════════════════════════
════════ Exception caught by gestures library ══════════════════════════════════
'package:flutter/src/widgets/single_child_scroll_view.dart': Failed assertion: line 583 pos 18: 'transformed == position + -_paintOffset': is not true.
single_child_scroll_view.dart:583
════════════════════════════════════════════════════════════════════════════════
════════ Exception caught by gestures library ══════════════════════════════════
'package:flutter/src/widgets/single_child_scroll_view.dart': Failed assertion: line 583 pos 18: 'transformed == position + -_paintOffset': is not true.
single_child_scroll_view.dart:583
════════════════════════════════════════════════════════════════════════════════

Hello! Thank you for using dual_scroll! Sure i'll look into this matter. Can you give the output of flutter doctor -v?

flutter doctor -v

First of all, thank you so, so much for your reply.
PS C:\Users\gnard\VSCODE\scavium_wallet> flutter doctor -v
[√] Flutter (Channel stable, 3.13.4, on Microsoft Windows [Versi¢n 10.0.19045.3448], locale es-ES)
• Flutter version 3.13.4 on channel stable at C:\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 367f9ea16b (2 weeks ago), 2023-09-12 23:27:53 -0500
• Engine revision 9064459a8b
• Dart version 3.1.2
• DevTools version 2.25.0
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
• Android SDK at C:\Users\gnard\AppData\Local\Android\Sdk
• Platform android-33-ext4, build-tools 33.0.1
• ANDROID_HOME = C:\Users\gnard\AppData\Local\Android\Sdk
• Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-9505619)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.4)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.4.33213.308
• Windows 10 SDK version 10.0.22000.0
[√] Android Studio (version 2022.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-9505619)
[√] VS Code (version 1.82.2)
• VS Code at C:\Users\gnard\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.72.0
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Versi¢n 10.0.19045.3448]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 116.0.5845.188
• Edge (web)        • edge    • web-javascript • Microsoft Edge 115.0.1901.203
[√] Network resources
• All expected network resources are available.
• No issues found!

Thank you for sending the required details and video! I will look into this and reply as soon as possible.


Safer multi-user editing

When an intent or entity is opened for editing we should create a lock on it maybe an editing flag set to true/false. This flag is then removed once the users navigate away or save their changes.
We should think of a basic way to force the removal of the lock as well. I am thinking on the intent/entity list pages we add an unlock item to the menu, with a warning message like Only use if you are sure no one else is editing this intent.
On the intent screen the Edit button could show a lock icon when it is un-clickable because someone else is editing it.

Should we also implement a lock timeout? The lock automatically is removed after N minutes or N minutes of inactivity? Seems like that is hard to get right.

@marcoaleixo anything to add to this? I see this as the first step in allowing multiple people to be working in the same agent. It is closely tied to #198 .

@wrathagom I think that's exactly it.
Just to make sure I understood correctly, the page of intentions would show what intentions are already being edited to avoid that the trainer does not waste time opening an intention that already has someone editing.
Looking at my problem, I think a 5-minute timer is more than enough to ensure the entity has already been edited.

This is a totally different ball game in the new UI. Need to think about how it could/should work.


[New game]: Pokedex game

🎮 Game Request
In this we can select pokemons and play around with them
Point down the features
Points can be assigned to each pokemon
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Barplot

New FramedBar component + functions bar, barh, to create bar plots.
This includes a definition of  the BoxPainter type, which is needed for legends (see pull request #195 for the same topic regarding legends of FilledComponent objects).

I tested in v0.4 and it works. It threw some warnings about deprecated syntax, which I fixed in slangangular/Winston.jl@28909d0d82ac32a514acc975842cc53022e0b235.
Is there any way to "append" these changes to the pull request or do I have to open a new one?

Thanks, slangangular. I have merged your changes into my fork. So this pull request should be updated automatically.

Thanks!


Inline styles added to background image set in CSS in FF Dev edition

I have a background image that I want to appear in the bottom left corner of every slide (it's a company logo) so I set it in my CSS:
.remark-slide-content:not(.inverse) {
   background: #fff url('../public/img/red-logo.svg') no-repeat 20px bottom;
   background-size: 150px 75px;
}

This works great in most browsers, but in Firefox Developer Edition there's some JS (here, I believe) that overrides the position and size that I set in my CSS by adding inline styles to the element. The inline styles set the background image to its original size and position it in the middle of the slide.
Not sure why this is occurring in only FF Dev?

This happens in Chrome too: http://jsfiddle.net/hfom987y/
Is your logo larger than 908x681px?
If the background image's dimensions exceed those of the slide (908x681px), the element styles are added to make the background fit the slide as well as becoming centered both vertically and horizontally. This is intended as a helpful feature for people using the background-image slide property to set a background image that doesn't fit the slide. However, is applied of there exists a backgroundImage style on the slide, regardless of whether it has been applied via CSS or implicitly via the aforementioned slide property.
For people who write their own CSS auto-fit is probably just a annoying feature, so I guess perhaps only background images set using the slide property should be auto-fitted.
In your case, however, if the logo is larger than 908x681px, the easy fix is to use a scaled-down version of the image.

Thanks, scaling down the image did the trick!


Request: disallowSwitchCaseIndentation

If this flag is true, then case statements should live at the same indentation level as their parent switch, a'la Crockford style.
Valid:
switch (x) {
case 'b':
    break;
}

Invalid:
switch (x) {
    case 'b':
        break;
}

The validateIndentation rule already tracks which style you use and then enforces future usages to be consistent with the first one, so I'm going to close this for now. Thanks again, and please explore the docs at http://JSCS.info

@mikesherov Hmm, but what if the first instance it picks up is not the desired one?

@yaycmyk then you'll get a ton of errors in your codebase. I suppose we can always make this an explicit option too. Care to work on it?

Sure, happy to!


UI: Moving from SQL Query to Structure leaves query editor on screen.

Steps to reproduce:

Choose a table.
Choose SQL Query.
Choose Structure.

Query editor stays on the screen. Moving to some other tab and back to Structure fixes it.

Thanks, i'll fix it.

This is fixed in master.


Fix template

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c6e65c0). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #4   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?        14
  Lines             ?       138
  Branches          ?        13
==========================================
  Hits              ?       138
  Misses            ?         0
  Partials          ?         0

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 c6e65c0...4079a7d. Read the comment docs.


Add a few things to Metadata section of the Submission Review page

Section
Section Corner
MRRC Project (Yes or No)

validated


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_91fa8bdd-a4d9-4901-b3c8-6ceed02d4f41/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_91fa8bdd-a4d9-4901-b3c8-6ceed02d4f41.git

Docs Build status updates of commit b13ed67:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/opstest2/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix small TODOs

continuation of #310

Codecov Report

Merging #311 (1e1b4ad) into main (5c2f23e) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files

@@            Coverage Diff             @@
##             main     #311      +/-   ##
==========================================
- Coverage   85.84%   85.83%   -0.01%
==========================================
  Files          52       52
  Lines        5594     5593       -1
  Branches      855      854       -1
==========================================
- Hits         4802     4801       -1
  Misses        669      669
  Partials      123      123

Impacted Files
Coverage Δ

src/ott/geometry/grid.py
91.60% <ø> (ø)

src/ott/geometry/epsilon_scheduler.py
94.44% <100.00%> (-0.16%)
:arrow_down:

src/ott/geometry/geometry.py
91.08% <100.00%> (ø)

src/ott/geometry/low_rank.py
96.37% <100.00%> (ø)

src/ott/geometry/pointcloud.py
86.04% <100.00%> (ø)


remove url regex restriction and allow up to 128 characters

https://app.clubhouse.io/augur/story/13947/url-verification-doesn-t-take-long-extensions-in-to-account

Pull Request Test Coverage Report for Build 935

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 65.844%

Totals

Change from base Build 934:
0.004%

Covered Lines:
3531

Relevant Lines:
4859

💛  - Coveralls


search page url parameters

When you search in the web version, the url is always "https://bsky.app/search?q=".
This is bad because:

When you refresh the browser, it goes to the empty search page (showing suggested follows) instead of updating the search.
You can't save a search or share it.
It makes back behavior very unpredictable. I don't know how to explain this one, because I'm not sure what's happening. But if I leave the search results and then hit back, it might show the last search or it might show one several searches back.

Thanks!

It looks like this might well be improved/fixed by the changes in #3663?

@surfdude29 Oh! It is improved, but still not right. The first search is captured as a url, but if you search again from the results page it reverts to "https://bsky.app/search?q=".

@surfdude29 Oh! It is improved, but still not right. The first search is captured as a url, but if you search again from the results page it reverts to "https://bsky.app/search?q=".

That's helpful feedback, thanks. Hopefully that will be fixed when @haileyok has finished working on the PR and it's ready to go 🚀

awesome! thanks so much!!!

Thanks for the report! Does the behavior on https://social-app-pr-3663.onrender.com/ match your expectations?

sorry for not replying, I hate that github doesn't have notifications for submitted issues. It's late, but yeah! it was perfect. thanks!

sorry for not replying, I hate that github doesn't have notifications for submitted issues. but yeah! it was perfect. thanks!


add an endpoint to sync discord users

Consume a pre-created function and create an endpoint to sync discord users and add their joined at date.
coverage report:
lines for controller written by me are covered

Please create a ticket and link that here.

Hey, @yesyash here is the ticket.
https://github.com/Real-Dev-Squad/website-backend/issues/1155

Please create a ticket and link that here.

Hey, @yesyash here is the ticket.
#1155

Thanks, @RitikJaiswal75 added it to the PR description.


Superposition: Crash when loading

Hello,
To begin, thank you for all the work you do on dxvk.
I'm french, sorry if my english is not perfect.
I suppose I found a problem between two commits. I am having this on a computer equipped with GeForce GTX 680 running Unigine Superposition benchmark on Ubuntu 19.04.
You can found logs between these commits with this issue.
Since commit 6936da1, I can't run Unigine Superposition without issue. Application crashes when loading. Superposition benchmark run fine with commit 18aada2.
 Running fine.
Testing on other computer with GTX 680. Commit 6936da1, application crashes when loading. Commit 18aada2, running fine.
This problem is still present in master branch.
Hope that help you.
System information

GPU: GeForce GTX 680 and GeForce GTX 970
Driver: 418.52.18
Wine version:  >= 4.12.1
DXVK version: 1.3 Commit 6936da1

Log files

[Commit_]d3d11.log and [Commit_]dxgi.log::
18aada2_launcher_d3d11.log
18aada2_launcher_dxgi.log
6936da1_launcher_d3d11.log
6936da1_launcher_dxgi.log

I'm aware that Superposition doesn't work properly on some low-VRAM Nvidia GPUs, but I also don't plan to invest any significant amount of time into trying to debug this. It's likely related to #1100, which is a known problem.

No problem. Create this issue only if that can help you.
Thanks for your answer.


JSONEncoder: time.mktime raises an error on Windows

Python 2.7.7 32bits on Windows 7 with the Anaconda distribution (I don't like this OS for dev but, it's for work you know)
Raised OverflowError: mktime argument out of range

http://stackoverflow.com/questions/2518706/python-mktime-overflow-error
https://docs.python.org/2/library/time.html#time.mktime

If the input value cannot be represented as a valid time, either OverflowError or ValueError will be
raised (which depends on whether the invalid value is caught by Python or the underlying C libraries

I fix this with the following code:
EPOCH = datetime.datetime(1970, 1, 1)

class JSONEncoder(json.JSONEncoder):
    def default(self, obj):
        if isinstance(obj, pandas.tslib.Timestamp):
            #t = time.mktime(obj.timetuple()) - time.mktime((1970, 1, 1, 0, 0, 0, 0, 0, 0))
            day = (obj.to_pydatetime() - EPOCH).days
            return int(day * 24 * 3600 * 1000)
        if isinstance(obj, datetime.date):
            day = (datetime.datetime(obj.year, obj.month, obj.day) - EPOCH).days
            return int(day * 24 * 3600 * 1000)
        if isinstance(obj, datetime.datetime):
            day = (obj - EPOCH).days
            return int(day * 24 * 3600 * 1000)
        return json.JSONEncoder.default(self, obj)

Quite ugly to take into account the different "date" object representation. Not very happy with that.
When I read a little the pandas.io.json.json function, I think that it handles this case. It seems that there is some cython code.

time.mktime((1970,1,1,0,0,0,0,0,0)) returns -3600. on my GNU/Linux distrib
Windows does not seem to support dates before EPOCH (due to CPython, C lib? don't know). However, if I type time.mktime((1970,1,1,1,0,0,0,0,0)), I've got 0.0.


Update version, changes, generate docs for 1.1.0 release

Update version, changes, generate docs for 1.1.0 release

Merged to dev in #231


[BUG] It's easy to introduce (non-fatal) data restore errors in check_update.sh if new firstboot-type flags are created

Description
See https://matrix.to/#/!ZhEZYNzKBfpEAhtQIz:matrix.org/$xW4cEYllmP9OHDon0Sz4nBz2u8oQ_vCgrnGCEKRLU_g?via=matrix.org&via=nitro.chat&via=strugee.net for background discussion. tl;dr 0fc580eb9d1ff3f262bca5278c79b13a970ecae0 introduces a non-fatal ENOENT error during update because do_bluetooth_config has been introduced in the same update, so the first boot to create this file hasn't occurred yet. This is non-fatal currently but would become fatal if anyone ever added set -e, which isn't necessarily a bad idea.
Steps to Reproduce
No response
Relevant Code
No response
Other Notes
Suggested fix from Matrix:

yeah, I was thinking there's not really any reason to not do those "if exists" checks for every file to be moved. could just refactor all those moves into a shell function

Good catch; I neglected to add those checks for the few signal files that are restored but it's valid for those not to exist so it makes sense to check if they do


koord-scheduler: support log filtering failure reasons as needed

Signed-off-by: wangjianyu<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?
this PR solves issue #846
Ⅲ. Describe how to verify it

Ⅳ. Special notes for reviews
V. Checklist

[] I have written necessary docs and comments
[X] I have added necessary unit tests and integration tests
[X] All checks passed in make test

/lgtm
/approve

/milestone v1.1


Fix lint errors

I used the Khan Linter to resolve all lint errors.

@simeg thanks for the PR.  I'm not sure when we'll be able to review this... hopefully sometime this week.


File name escaping seems to fail

I'm running into this issue and can't seem to find a solution. When setting a source or destination that has spaces in it, it causes all kinds of issues and for a variety of reasons:

When entering a name with spaces, the spaces are escaped but the entire string isn't put into quotes. Simply escaping the name doesn't work. There's an SO question on this
When entering a name with spaces and trying to manually quote it, the quotes get escaped (which makes sense but doesn't offer a quick workaround)

I created a fork and it's far from perfect but here are a few situations it fails (and node-rsync currently fails):

when transferring at any point and escaping, there need to be quotes
when transferring from remote to local, escaping local paths can cause issues, escaping remote can cause issues
when transferring from local to remote, remote has to be escaped.

I'm currently doing this:

escape all paths unless it contains @ which designates it as remote
when going from local to remote, using the s argument.

It'd be great to be able to override the default escaping mechanism (such as by specifying a "true/false" flag in the function) and having "escape and quote all" default.

Okay, so I haven't been able to figure it out..at all. The results I'm getting are rather inconsistent. Anyways, I'd love some help on this. I don't mind writing the PR or implementation but how is escaping supposed to work? I read that you're supposed to quote AND escape the name like so:
'file\ name\ with\ space.pdf'

But this morning, my version of rsync ended up uploading a file with the slashes included.

Thank you for taking the time to try and figure this out. I'm not quite sure myself what the best solution is here. I did a quick test with a file and directory with spaces in them. It seems to work fine when the entire source and/or destination are quoted and have the spaces escaped, like so:
$ rsync "a\ text.file" "user@some.host.server:b\ dir"

Testing this is going to be hard cause it might also depend on the shell the command is executed in (I did not include that in my small test). Some work has to be done to figure out proper test cases that are know to work. These test cases will probably have to be figured out manually but running a bunch of commands on a machine.

So here's what I found out over the past few months. First: the PR I opened has been working without a hitch since the last commit. I'm using the repo to transfer files uploaded from Heroku to another server (and transfer them back). I'm dealing with a fair number of files in all kinds of different naming formats.
Here's the sample command I'm using to retrieve files:
var rsync = new Rsync()
          .shell(`ssh -o StrictHostKeyChecking=no -i ${sshKeyPath}`)
          .flags('az')
          .set('remove-source-files')
          .source([`${remoteUser}@${currentEnv.dataServer}:${cssPath}`, `${remoteUser}@${currentEnv.dataServer}:${htmlPath}`])
          .destination('/tmp/')
        ;

and to save them:
   var rsync = new Rsync()
      .shell(`ssh -o StrictHostKeyChecking=no -i ${process.cwd()}/provisioning/files/ssh/id_rsa`)
      .flags('azs')
      .source(`${req.files.file.path}`) //upload path of a file
      .destination(`${remoteUser}@${envConfig.dataServer}:${app.locals.path)}`)
    ;

I never truly got it to work so right now, I have to post-process the files that I upload so that they have a nice name. Otherwise, I can either setup the escaping to allow me to retrieve files or upload them but I can't do both reliably with the same code. I kept getting into a situation where escaping one way worked for uploading to a remote server but downloading made it fail and vice-versa.
There's some weird thing with escaping where you escape one way for src vs. dest and another way local vs remote. Each combination of the two produced a failure effect.
To explain the PR:
var match = filename.match('@'); // match to see if the server is remote

//if it is remote, escape everything
if (match && match.length > 0) {
   //escape quotes, backticks, apostrophes, $, &, and bunch of other stuff because people use those in EVERYTHING for some reason.
   return "'" + filename.replace(/(["'`\s\\\(\)\\$\\&])/g,'\\$1') + "'";
}

// if it is not remote, simply quote it because that works
return "'" + filename + "'";
```

Another major caveat to this PR (other than not reliably escaping for that weird grid of `src`/`dest`/`local`/`remote` combination), is that variable expansion essentially doesn't work due to the use of quotes.

Thanks for the detailed explanation.
The thing I don't like about the current solution is that it relies on the @ character in the string. I'd rather have a single way of treating each file path used in source or destination. This basically comes down to quoting all files, not just the ones containing an @ character.
Like you pointed out shell expansion does not work when quoting the file names. I wanted to hold on to this for convenience but I'm starting to get convinced that this is just not a good idea. We are constructing the command from JavaScript anyways so why not do file lookups  from JavaScript too (through glob for example). Also #40 kind of killed escaping already (partially).

I kept getting into a situation where escaping one way worked for uploading to a remote server but downloading made it fail and vice-versa.

This is odd. Could you provide a reproducible test case for this (using local files) so we can see if we can determine how this should work.

There's some weird thing with escaping where you escape one way for src vs. dest and another way local vs remote. Each combination of the two produced a failure effect.

I don't fully get this one but I think this should be solved by just always escaping file names with quotes. In order to do this escapeFileArg should be modified but beware that this is also used for include/exclude patterns. Some of the tests expected output should be changed in order to accommodate the changes. I would be nice if you could keep #37 in mind and maybe even get a little start with that.

I got around this by quoting the remote path myself.
For example:
const rsync = new Rsync()
    .set('rsh', 'ssh -p22')
    .source(`${username}@${host}:"${source}"`)
    .destination(destination)

Awesome!
I still use my snippet up above in production lol. And it works!


Ethan: Watch mytv-live.com

Ethan requests the watch of the watch_keyword mytv-live\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

mytv-live\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Ollie in Charcoal HQ


Exporter: Sequence animations simulations; in blender rendering

Feature request: Exporter
In blender rendering integration or a suggestion/workaround on exporting scene animation sequences.
Is your feature request related to a problem? Please describe.
I read the wiki on the exporter module current functionalities' support at

https://github.com/mitsuba-renderer/mitsuba-blender/wiki/Exporting-a-Blender-scene

"Future versions may support more fancy features, such as custom nodes for materials or in-blender rendering."
I'm going to try a few things myself, but I was wondering how could one use Mitsuba render engine directly in Blender? Or use the xml export and export a frame sequence similar to blender command line for animation rendering. Cameras objects would be nice as well.
Describe the solution you'd like
Currently I have a few scenes in Blender, and I use the LuxCore render engine to use the Metropolis method and easily get caustics along boids and ocean simulations.

I was wondering if I could get this exact project and render the animation sequence, so like a series of images I could later turn into a video... or even pipe ffmpeg to export a video too.
Describe alternatives you've considered
At first I thought of setting a different frame and sequentially export the xml corresponding to the frame, but it would be a lot easier if I could control the xml like the blender command line animation render options, perhaps mitsuba would give even greater control like different/multiple camera selection
Additional context
I was first interested in Mitsuba after reading Tizian Zeltner, Iliyan Georgiev, and Wenzel Jakob. 2020. Specular Manifold Sampling for Rendering High-Frequency Caustics and Glints. In Transactions on Graphics (Proceedings of SIGGRAPH) 39(4). The caustics were quickly calculated, I believe it could beat LuxCore current performance on CPUs.
This is what I am trying to accomplish with Mitsuba in my pipeline. I used the Metropolis method in LuxCore as a render engine, directly in Blender. I was able to do all sorts of fancy stuff like boids, ocean simulation, multiple cameras... but it takes ages for an animation sequence.
https://user-images.githubusercontent.com/28659434/204975973-81bd0d8e-d638-4e19-b0c2-dd67c09d4730.mp4
https://user-images.githubusercontent.com/28659434/204975921-56a895b8-1d63-457a-9ae1-b24db6567ff5.mp4
I would be very open to share any potential developments I come up with on trying to implement Mitsuba in my Blender scenes, or help in some other possible way.

Hello, thanks for the suggestion!
I believe that Mitsuba 3 does not support animation sequences (see the doc here). So describing the animation in the XML scene file is unfortunately not an option.
However, we are working on in-Blender rendering using Mitsuba which should solve your issue.
If you want to try to have a go at it, you can already find the previous (sadly broken) implementation of the renderer in this file.
Cheers!


[WIP] Allow the user to import transcripts into the Roadmap Planner

Development Notes
Any comments would be very helpful!

[ ] This would be good to release together with #266 since it currently overrides the user's schedule. (This is still open to discussion though)
[x] Lint checks will currently be failing due to updating older files that have ESLint warnings in them.
[x] Styles are still using the year modal styles, so this needs to be refactored.
[x] Button placement and other layout/styles are not finalized yet.

Description

Add package to parse the HTML without injecting content into the page
Extract the transferred credits as well as quarters and courses taken at UCI from the HTML Transcript
Group these quarters into academic years (in a planner-compatible format)
Set the current roadmap's planner based on these year entries

Screenshots
Import Dialog with Information (styles and WIP)

Steps to verify/test this change:

[ ] Verify changes work as expected on staging instance

Final Checks:

[ ] Verify successful deployment

Issues
Closes #443

@js0mmer do you think you could take a look at the files failing the lint check and update the dependency arrays for them?

Yeah

Interesting, yours allowed you to commit even with the dependency array warnings, but mine didn't.

Yeah, formatting just seemed to be messed up somehow.

I will leave the pre-existing dependency warnings to be resolved later, don't want to move too much stuff around in this PR.

I will leave the pre-existing dependency warnings to be resolved later, don't want to move too much stuff around in this PR that's out of scope.

yeah it's not giving an error on commit anymore so not going to worry about it

Going to request review on this once we decide what the behavior of the import should be.

Now that #440 is merged, I think this would be a great first feature to add to the changelog modal!

Included the changes we discussed yesterday regarding quarters and CC course credits @js0mmer

Still not a huge fan of just removing spaces but I do agree that it's better than essentially reusing 40 lines of code.


[Question] buildmode hooks?

Short Description
i am wondering how i do it for kyle's buildemode thing for ulx to stop players from taking or doing damage to builders, as i can't find any real examples.
https://steamcommunity.com/sharedfiles/filedetails/?id=1308900979

https://github.com/Stooberton/ACF-3/blob/1387113c1b8cc3c1c3e348331438a10c58338620/lua/acf/damage/damage_sv.lua#L482

Something like this?


Missing Intensity normalization prevents fits

Amplitudes generated with the expertsystem cannot be used in fits as a normalization of the intensity is missing. Hence the expertsystem is not compatible with the current version of tensorwaves.

Relates to https://github.com/ComPWA/expertsystem/pull/417

Relates to https://github.com/ComPWA/expertsystem/pull/417


Consolidate signing into one package

The signing implementation in simulation seem to have become useful across production scope code, among other places. Instead of having a split singing package, one at top level only containing BLS and one at sim mixing it all consolidate them both.
Move tests as necessary and split test files as placeholder for writing more tests. Simplify the test suit boilerplate that now has become redundant because of the new simpler package structure.

These packages have different goals, from what I can tell. The sim/signing package exists to manage and generate temporary keys.
Honestly, I expect a lot of this will change once we move most of the signing to the client. But @Kubuxu might be able to shed more light on that.

These packages have different goals, from what I can tell. The sim/signing package exists to manage and generate temporary keys.

I more or less agree. The current state of main however uses Fake signer (in sim/signing) across other production scope packages.
Would separating "generation of temporary keys" from the rest and leaving it in sim unblock this PR @Stebalien ?

My understanding is that the current situation is temporary and that main.go will drop the dependency, right?

Ok

Yes, the main will drop the dependency in the future. We still might want to use a fake signer for wider integration tests, though.

This is a quick refactor. Seems like it's best to to reopen it once there is a clearer idea on where we stand with main branch.


When using a local ruby installation it includes the local LOAD_PATH

The problem is that if the ruby interpreter uses the ruby already installed on the host, and that host is rbenv, rbenv will inject into its own LOAD_PATH which includes its gems folders.
This means that ruby_binary is not hermetic as it may inadvertently depend on gems that are not explicitly declared in the rule.
Not sure how to fix this, maybe use a better script to use the local ruby version, and try work around rbenvs magic (or use rbenvs magic to find the real interpreter path without LOAD_PATH injection)

We don’t have to use rbenv’s shims or wrappers for the ruby binary. It installs rubies in .rbenv/versions/2.6.5/bin/ruby — that would be a pure binary without any rbenv or Rvm magic.


Selection occurrences decorations are on top of diagnostics

Not sure if themeing.

They are behind the diagnostics. I don't think anything has changed.
1.10.2:

Oh I see, this is the theme I'm using: Ayu.

Ah now see it with the QuiteLight as well. The focus'ed selectionHighlight has no transparancy.


Provisioning fails against cPouta

There is a problem with current version of shade against cPouta nova API when creating servers with server groups. The underlying problem seems to be this shade issue:
https://storyboard.openstack.org/#!/story/2001174
Using shade 1.21 works, so a workaround is to install that specific version when populating the virtual environment for provisioning:
    pip install 'shade==1.21' dnspython funcsigs functools32

If your application can live with the increased risk of data loss,  another possible workaround is to disable server groups in tasks/vm_group_provisioning.yml by commenting out scheduler_hints:
    volume_size: 50
#    scheduler_hints: "{{ scheduler_hints | default({}) }}"
    wait: no

Looks like shade==1.21 works. Workaround updated


I wonder if Potion Core's Damage Resist Attribute...

Since this this mod has some Potion Core related things and the RLTweaker is a bugfix / performance. I wonder if Potion Core's Damage Resist Attribute can be fixed as it conflicts with Vanilla Damage Resist in a way. Here's a comment that I posted on Potion Core's curseforge page listing the details.
(Quote)

Potion Core's Resist has a conflict with Vanilla Resist if it's default value was changed by... say... Potion Tweaker. It's a mod that allows you to change the default values that potions give for strength potions, weakness potions, instant damage potions, etc.

Here's a breakdown between No Potion Core & Potion Core. With Potion Tweaker I changed the resist value from 0.2 (20%) to 0.1 (10%).

(0)R0= 1001 1001 1001 1001 | 0%
(-)R1= ____ 0901 ____ 0901 | 10%
(1)R2= 0801 0801 0801 0801 | 20%
(-)R3= ____ 0701 ____ 0701 | 30%
(2)R4= 0601 0601 0601 0601 | 40%
(-)R5= ____ 0501 ____ 0501 | 50%
(3)R6= 0400 0400 0400 0480 | 60%
(-)R7= ____ 0300 ____ 0420 | 70%
(4)R8= 0200 0200 0200 0320 | 80%
(-)R9= ____ 0100 ____ 0180 | 90%
(T)RT= 0000 0000 0000 0000 |100%

===========================
Resist 0.2 0.1 0.2PC 0.1PC

This has been looked into and this is due to the Resist in Potion Core being hard coded with 0.2D (20%) so it will always expect that level 5 is the max. If it goes above level 5, then the damage reduction ends up breaking.

One other bug occurs where if you have Vanilla Resist & Potion Core Resist attributes... you CANNOT have both at the same time if they reach high values otherwise the damage reduction will get a bit messy. Example...

Zombie Pigman hits me for 12 Damage with no resistance of any kind on World Difficulty Hard.

Resistance I (+20%) = 10 Damage Taken.
Resistance II (+40%) = 7 Damage Taken.
Resistance III (+60%) = 5 Damage Taken.
Resistance IV (+80%) = 2 Damage Taken.

+50% Potion Core Resist = 6 Damage Taken.
+100% Potion Core Resist = 0 Damage Taken.

I am now gonna add in scaling health but without any damage scaling, as it has a debug option to show your exact health value. This is what I get with both Resistances. Note that this can happen without scaling health being used.

+50% Potion Core Resistance + Resistance I (+20%) = 4 Damage Taken. (16.16 / 20.0 Health)
+50% Potion Core Resistance + Resistance II (+40%) = 3 Damage Taken. (17.119999 / 20 Health)
+50% Potion Core Resistance + Resistance III (+60%) = 3 Damage Taken. (17.119999 / 20 Health)
+50% Potion Core Resistance + Resistance IV (+80%) = 2 Damage Taken. (18.08 / 20.0 Health)

+100% Potion Core Resistance + Resistance I (+20%) = 2 Damage Taken. (18.08 / 20.0 Health)
+100% Potion Core Resistance + Resistance II (+40%) = 3 Damage Taken. (17.119999 / 20 Health)
+100% Potion Core Resistance + Resistance III (+60%) = 3 Damage Taken. (17.119999 / 20 Health)
+100% Potion Core Resistance + Resistance IV (+80%) = 2 Damage Taken. (18.08 / 20.0 Health)

Either +50% or +100% Potion Core Resistance + Resistance V (+100%) = 0 Damage Taken. (20.0 / 20.0)

=====

If either one is being used it's fine but if you have both then you'll run into conflicts such as the damage reduction not working the way it's suppose to, in some cases you may take more damage if you had higher resist.

I think that the Potion Core Resist should be it's own damage reduction attribute that isn't mixed with Vanilla Resist since currently it causes these conflicts to occur. It could go after Vanilla Resist reduces the damage so like if you took 50 damage, and had 50% vanilla resist, that lowers to 25 damage... and if you had 25% Potion Core resist... that lowers to 18.75 damage.

===========
I dunno if RLCraft has Potion Tweaker but even then there's still some sort of conflict going on when using both Vanilla Resist & Potion Core's Damage Resist. If you factor in Potion Tweaker and have it grant Resist by 10% per level instead of the default 20%... the DR doesn't add up correctly when yo go beyond Level 5. This doesn't happen if you aren't using Potion Core.
P.S. Also wonder if the HUD Element for showing your Damage Resist whenever you have Vanilla Resist or Potion Core Resist can be changed so it only shows the overlay whenever you have Potion Core's Damage Resist Attribute, and not when you have any Vanilla Resist, that is if it's possible to change the Potion Core Damage resist Attribute to be it's own damage reduction instead of being linked to Vanilla Resist.

I'm sorry, I somehow missed looking at this issue but thanks for the detailed suggestion.
Actually this is something for RLCraft and thus RLTweaker v1.12.2-0.5.6 has an option to fix the issue. But sadly the mixin from  Potion Tweaker to the EntityLivingBase class is very invasive which in fact just overrides all changes from RLTweaker. I suggest you to ask the Potion Tweaker developer to make his mixin less invasive to not override the RLTweaker changes.

It's alright and no problem, I gave it a test and looks like both Vanilla Resist & Potioncore Resist stack on top of each other correctly, for example...
Resist IV (80%) and Potioncore Resist (20%) will actually be 100% Resist in total, 0 damage taken.
Resist III (60%) and Potioncore Resist (40%) will actually be 100% Resist in total, 0 damage taken.
It also works fine if you happen to overflow the total resist.
Resist III (60%) and Potioncore Resist (100%) would be 160% Resist but acts as it's 100%, which makes sense since there's nothing really to gain beyond 100% Total Resist. As for the damage taken... it's still 0 where before it would've been some form of damage taken.
The calculation is fixed, though... I was hoping if Potioncore's Resist can be it's own damage reduction instead of stacking on top of the vanilla resist in general, unless that can't be done as has to be done on Potioncore's side. If it were to be it's own form of damage reduction. Rather than it stacking you'll have something like this.
...I dunno if vanilla goes first and then potioncore or if potioncore goes first and then vanilla but here's an example.
If vanilla resist goes first...
You took 50 damage, but you have 60% Vanilla Resist, that gets lowered to 20 damage but you also have 40% Potioncore Resist, so that 20 gets reduced to 12 damage.
If potioncore goes first...
You took 50 damage, but you have 40% Potioncore Resist, that gets lowered to 30 damage but you also have 60% Vanilla Resist, so that 30 gets reduced to 12 damage.
Looks like no matter the order it appears to be the same result. I prob got my math wrong, though. There's a potion that can lower your resist but I dunno if it's potioncore's or vanilla's resist.
If something reduced your Potioncore Resist... since there's a potion that can lower your resist.
You took 50 damage, but you have 60% Vanilla Resist, that gets lowered to 20 damage but you also have -50% Potioncore Resist, so that 20 gets increased to 30 damage.
If you already had some Potioncore Resist and something lowered it...
You took 50 damage, but you have 60% Vanilla Resist, that gets lowered to 20 damage but you also have 70% Potioncore Resist, however there's also -50% Potioncore Resist due to the potion, leading to only 20% Potioncore Resist. That 20 gets lowered to just 16 damage.
If it still stacked like it currently does...
You took 50 damage. You have 60% Vanilla Resist but you also have -50% Potioncore Resist, your total resist is 10% so the damage taken would be 45.

Again, I prob got my math wrong with this, but with so many modded reductions added from other mods, they don't all stack together but run their reductions prob related to their event, priority or modid. This is only a guess, though.
One thing to mention, if Potioncore Resist does become it's own damage reduction, the potion "Potion of Vulnerability" would prob need to be looked at so it affects Potioncore's Resist instead of affecting Vanilla Resist.
This was quite a write-up, so hopefully I explained it well.

I just reworked the resistance calculation again with 1.12.2-0.5.8. Now it works like a normal attribute with config options that let you configure the value to your liking.
Explanation of alternativeResistanceMode (because I forgot to add the description):
If you have two modifiers with amount=0.3 and operation=2 then this is the difference:
Normal Mode: 1 - ((1.0 * (1.0 + 0.3) * (1.0 * 0.3)) - 1) = 0.31 which means you get 69% less damage.
Alternative Mode: 1 - (1.0 * (1.0 - 0.3) * (1.0 - 0.3)) = 0.51 which means you get 49% less damage.


Remove @testable imports WIP

[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test

@swift-ci test windows

Sorry for the negative feedback, but this seems like a really bad idea. it opens all of these APIs to be used by anything in package for the (imo small) benefit of avoiding @testable.  instead please consider @_spi(Test) or something similar like swift-driver does. It's a lot more clear at the declaration site why the API is public than package.

This PR is a draft WIP and isn't ready for feedback.
I'm only trying it out to see if this enables incremental builds for bootstrap builds at all via any changes in code, which may not be the case in the end. At that point we won't care if @_spi is better than package if neither solve the actual problem.

@swift-ci test linux

@swift-ci test

@swift-ci test

@swift-ci test windows