Missing source and model props in Table component d.ts

Missing source and model props in components\table\Table.d.ts
TS2339: Property 'model' does not exist on type 'IntrinsicAttributes & IntrinsicClassAttributes<Table> & TableProps & { children?: ReactNode; }'.

forget it, please update the documentation with the last version.


fix: ignore screenview tracking events from SwiftUI framework

Part of https://github.com/customerio/issues/issues/10950
Alternative approach from this PR which disables screenview tracking feature entirely.
I am introducing this approach to try and keep screenview tracking feature for customers that may use UIKit views inside of their SwiftUI app.
Tasks

[ ] QA testing in SwiftUI to verify automatic screenview tracking events get filtered out.
[ ] QA test UIKit app to verify events do not get filtered out.

I changed the implementation to include a SdkConfig option. Asking for code review again because of this.


Is it possible to migrate this to Stable Diffusion 3?

First of all, I would like to sincerely appreciate your impressive work on.
I have a question regarding the potential compatibility of this work. Would it be possible to migrate this approach to the Stable Diffusion 3 model? If so, are there any specific considerations or adjustments that would be necessary?
Looking forward to your guidance and insights. Thanks again for this outstanding contribution!

Thank you for your interest in our work! Yes, it is indeed possible to adapt our approach to Stable Diffusion 3. However, there are a few considerations and adjustments that would likely be necessary:

Our approach is specifically designed to align few-step models efficiently. As such, it would be important to leverage a distilled few-step version of Stable Diffusion 3, such as Stable Diffusion 3.5 Large Turbo.

Migrating to Stable Diffusion 3 may require upgrading our codebase to support newer versions of libraries like Accelerate, Diffusers, and Transformers.

If you'd like to work on this, feel free to reach out. I would be happy to help.

Thank you for sharing your insights!   I am currently experimenting with this, but I’m uncertain how to implement a function similar to ddim_step_with_logprob in a flow-based model.   I have made similar modifications based on diffusion-dpo and spo, but unfortunately, it hasn’t worked as expected.
Would you have any recommendations or insights on how to approach this?


Error in popout related to CSS

Describe the bug
I'm getting the following error message when opening a popout now (using itwinui-react 2.12.1 and appui 4.x)

NotAllowedError: Failed to set the 'adoptedStyleSheets' property on 'Document': Sharing constructed stylesheets in multiple documents is not allowed

I have reported the issue in the ITwinUI repo, but since copying styles to the popout is done by appui, it seems the fix should be done here.
Please refer to issue #1579 in iTwinUI for details.

Thanks, we'll have a look!

Fixing the copyStyles method in relation to another issue. It is fixing this issue as well

Hi @a-gagnon , a fix for this have been released in 4.6.0 released today, let us know if this fixes your issue!


Add validation for Membership#new

Currently, it is required that you use the auto-complete feature.
If you don't, things break.
Don't allow things to break.

Things don't break and a lookup is attempted if user doesn't use auto-complete.
b457bd1f7dfff1355d41ccf417cc8b18c6e6a0e0


Embed software metadata, publication and citation

As announced earlier, this PR provides additions to @ljgarcia's PR on ML and metadata (#214).
Let me know if I should put this up as a dedicated PR to main once #214 is merged.
I'd be happy about feedback regarding these changes, and will try to join today's call.

Thanks, these are great additions.

Sorry for the unintended close. You could target main with this PR now

@mhagdorn @CaptainSifff I've now removed the extraneous references in the competency overview and left only one that talk more generally about software metadata. In lieu of a canonical reference paper, this is the most comprehensive overview at the moment I think (although disclaimer: I'm an author). I can still remove it if you feel it shouldn't be there, and no hard feelings ;).


Deletion of CRs (service and binding) ends with an unstable state  for the CRs and the IBM Cloud Operator

Hi,   today we figured out following:
Our objective was
We wanted to delete instances of the "bindings and service" by removing the CR instances inside a project/namespace, because we would like to delete the services on IBM Cloud and setup new configurations for our application in the cluster.
Steps to reproduce:

Step 1: Delete  existing CRs (service and binding) of the IBM Cloud Operator  in a project or namespace
Step 2: Verify it will fail, the CRs will be greyed out but not be deleted.
Step 3: All deleted CRs now in an unstable status

Question:
How to avoid this or even better how to fix that unstable status?
Steps to get a stable status again:
We tried to following steps to get a stable status again, but we had no success at the end.

Deletion of the operator development project with the CRs failed (status terminating)
Deletion of IBM Cloud Services has no impact to the IBM Cloud Operator
We noticed that the IBM Cloud Operator restarts itself again and again
Deinstall the IBM Cloud Operator has no impact to the deletion or CRs and the srojects
Restart the master node of the OpenShift cluster
Restart the work nodes of the OpenShift cluster

Conclusion at the moment
Don't delete CRDs for the IBM Cloud Operator and don't delete projects that contains CRDs of the IBM Cloud Operator! The deletion will break the IBM Cloud Operator and you need to install it again and maybe ends in an unstable state for projects at the cluster.

We fixed it with more memory for our cluster and a reinstallation of the Operator once more and a clean-up in IBM Cloud.

Hey @thomassuedbroecker, thanks for posting your solution. Was the only issue memory pressure?
Deleting CR's should behave correctly, of course. 🙏

Hi @JohnStarich, we noticed: you should don't delete too fast too many IBM Cloud Operator CRDs , maybe that results in an unstable state and can be related to Writing a Kubernetes Operator: the Hard Parts - Sebastien Guilloux, Elastic .


mkdocs.org does not support https

changed https to http in link

Thanks @Ken-B.

Thank you for the wonderful package.


Can't run using npm run serve

i cloned the repo.
i did a npm install

i got:
found 35586 vulnerabilities (35554 low, 29 moderate, 3 high) -> i did not fix/audit those.
-> npm run serve results in:
ERROR  Failed to compile with 10 errors.
basically it seems the be a repeating pattern -> You may need an additional loader to handle the result of these loaders.
You may need an additional loader to handle the result of these loaders.
|
|     bingApiKey() {

  return this.config?.apiKeys?.bing;

|     },
|
Here's my EnviDat-RunServeErrors.txt
most probably i did something wrong. Maybe you can point me in a direction to solve this errors?
thanks - appreciate your contribution!
Christian

Hi Christian
Thanks for your interest and your feedback.
Yes right, the babel config is missing some entries. I've only added them in a feature branch.
I'm going to add it to the master aswell... should be fixed in around 10mins.
I'll reply again.

Okay done, could you try again please :)

now it works like a charm, thanks!
amazing work btw!
Christian


Cannot compile the Karate test runner file (TestRunner.java)

I have a maven project that has included the dependencies of the Karate libraries in its pom.xml, but I still could not compile the java file. It shows the error - "java: cannot access com.intuit.karate.Runner
class file for com.intuit.karate.Runner not found"
I do not know what else to tell Maven to include those libraries.  I am using IntelliJ. I am very new to Java. Could someone help? I had included my pom.xml and the testrunner java file.

@candrewtemenoscom sorry this is a question best asked on stack overflow: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue
if you are new to java, I recommend the VS Code plugin. you can create a feature file and just run it: https://marketplace.visualstudio.com/items?itemName=karatelabs.karate

Thank you for your prompt response and information. In fact, I installed the same plug-in in my IntelliJ IDE already. Now, I managed to compile my Java program with Karate, even though I am unsure what I had done to make it work! Anyway, the ultimate goal I want to achieve is to make a Java application (an executable) that I can run to run all the tests I had in the feature file. It seems I can only put the Java class and feature file in the test folder to make it work. But this way, the executable (jar file) would not run the feature file. How should I structure the project to make an executable to run all the feature tests after compile it? Thanks so much.

@candrewtemenoscom not something I recommend teams new to karate or java should do - and we don't claim to support this. all available info is here on stack-overflow: https://stackoverflow.com/a/56553194/143475


Factory method for array container deduplication

SUMMARY
Factory method for array container from given bitmap within given range deduplicated. Chosen variant, which is more similar to expression used in project - preferred word &= (word - 1) to word ^= word & -word.
What is ugly, that created common method is public, but it is used in both cases from private methods, but as the utility classes are not within the same package, it is the only way to do it. And as it is public, there is a strong assumption, that method parameters are valid, so it is dangerous when no limit checks are done in method not used internally only. May be some refactoring of both affected utility classes BitSetUtil and BufferBitSetUtil should be done.
Automated Checks

[x] I have run ./gradlew test and made sure that my PR does not break any unit test.
[x] I have run ./gradlew checkstyleMain or the equivalent and corrected the formatting warnings reported.

Would you extend to BufferBitSetUtil.java? We are trying to keep our implementation in sync.

Isn't this what you mean?

Merging. Thanks.


we need to investigate this problem

Transactions (most of the time) go to state -
Sorry, we are unable to locate this Transaction Hash
https://etherscan.io/tx/0xd6323e041c299aac90b12c4db02a0c6faff2650fc88f43dcba4c71b104f35d6a
This tx can't be found on Etherscan. You may have to rebroadcast the tx or create a new one as it likely didn't make it to the network.

Also read this -
<EMAIL_ADDRESS>


UP-4443:  Fixed the EntityIdentifier available from PersonAttributesG…

…roupDefinitionImpl;  it was creating unusable EIDs

Looks good!


sound: remove ctrl msg param

Summary of the PR
This PR is the result of splitting https://github.com/rust-vmm/vhost-device/pull/556. This PR addresses the required changes to remove the ControlMessage parameter from the set_param() and release() functions. In particular in set_param(), this PR replaces it with the VirtioSndPcmSetParams structure that is parsed in the device thus preventing the audio backend to manipulate descriptors.
Thanks for reviewing!
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR are signed (with git commit -s), and the commit
message has max 60 characters for the summary and max 75 characters for each
description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

FYI did not get to test it, I just woke up :) no problem of course, I think it looks fine and if there's anything we can fix it afterwards.


chore: Update msrv to 1.60

Updates rust-version field in manifests to follow to #814.

oh good catch!


Devstack plugin clean target hardcoded to :latest

The image in the devstack plugin when using clean.sh shouldn't be hardcoded to sg-core:latest. Instead it should remove the $SG_CORE_CONTAINER_IMAGE
https://github.com/infrawatch/sg-core/blob/3b1474493c32b939fe47c00a66eeca33c8a934bb/devstack/plugin.sh#L48
I see this as a nice opportunity to make sure the CI here is working correctly after we fix the current ceilometer issue. I plan to work on this once this is merged: https://review.opendev.org/c/openstack/ceilometer/+/898215

fixed in #123


Nested with items

Fixes issue #1186

Coverage decreased (-0.1%) to 88.562% when pulling 1f9567fe730719133416d008354419c5183c45a8 on manticore-projects:NestedWithItems into 7c21242e893abcd383357f002bd676ce01e33f0a on JSQLParser:master.

Kind reminder please.

Sorry. I thought I already merged it.


IO Port Restriction (D3-IOPR) expanded definition and link to ATT&CK T1200 Hardware Additions

Recommending the definition of IOPR be expanded to Port Security and MAC address restrictions for additions of new devices or hardware to the network. Through this expanded definition, D3-IOPR should protect against T1200 Hardware Additions.

Thank you for your contribution! We agree, we'll get this updated.


Uncaught TypeError: Cannot set property 'xLabels' of undefined

This pertains to the codebase on https://github.com/jhudson8/react-chartjs/pull/17
https://github.com/aaronhayes/react-chartjs/blob/master/lib/core.js#L45
Oddly, while it does update the chart with the pertinent labels, It's still throwing this error with no perceivable repercussions as new data is fed into the char.

In Firefox, instead of the xLabels error, I get:
TypeError: nextProps.data.labels is undefined

ahh yes, forgot to mention there is an error in chart.js itself. It has been fixed in a recently merged pull on https://github.com/nnnick/Chart.js. However a release has not been made yet.

closing this as it is not an issue with react-chartjs

Adding redraw as a prop fixed this issue for me.
https://stackoverflow.com/a/48930199


troubleshooting docs & tools is need

troubleshooting is difficult  with different status(Pod pending，pvc pending) without logs and events , suggest to provider troubleshooting docs & tools

We will plan it in our roadmap


Regression in 2.13+: Builds fail with "the tools version is unrecognized" error.

I have a build that runs find with plug-in 2.12. I'm using VS 2015 and MS Build 14.0.
Here is my full msbuild configuration:
msbuild {
        // .csproj, relative to the project root
        projectFile = file("src/main/asp.net/${vsProject}.csproj")

        // MsBuild project name (/p:Project=...)
        projectName = "${vsProject}"

        // Verbosity (/v:detailed, by default uses gradle logging level)
        //verbosity = 'detailed'

        // targets to execute (/t:Clean;Rebuild, no default)
        targets = ['Clean', 'Rebuild']

        // ### Project setting overrides ###

        // overrides project OutputPath
        destinationDir = "${buildDir}/msbuild/bin"

        // overrides project IntermediaryOutputPath
        intermediateDir = "${buildDir}/msbuild/obj"

        // Generates XML documentation file (from javadoc through custom DocLet)
        generateDoc = false

        // overrides project build configuration
        configuration = msbuildConfig

        // Other msbuild options can be set:
        // loggerAssembly, debugType, optimize, debugSymbols, platform, defineConstants ...

        // /Project setting overrides

        // ### Build properties (/p:SomeProperty=Value) ###

        parameters.DeployOnBuild = 'true'
        parameters.PublishProfile = 'Local Build Dir' // Publishes to build/site/
        parameters.VisualStudioVersion = '14.0' // VS 2015
    }

In 2.13+ builds fail with:
Error during project file parsing: Microsoft.Build.Exceptions.InvalidProjectFileException: The tools version "14.1" is unrecognized. Available tools versions are "14.0", "2.0", "3.5", "4.0".
This might have to do something with children projects. I have two child projects defined and configured in a similar way (to above). If I disable children projects, my main project passes.

Hello,
I had the same issue until I changed the ToolsVersion="12.0" to ToolsVersion="14.0" in the csproj. This is because 12.0 is an unknown tools version so the plugin automatically bump to the tools version of MSBuild in the ProjectFileParser (i.e. 14.1).
Regards

Hello all,
I have just installed visual studio enterprise 2015 edition. I get the error attached when i tried to create a new project.

I am new to visual studio and lost as to what to do next. Pls help

@Itzkwat That doesn't seem to be from our plugin, but rather from the project you're trying to open :)

I've got the same problem with 4.16
Error during project file parsing: Microsoft.Build.Exceptions.InvalidProjectFileException: The tools version "14.1" is unrecognized. Available tools versions are "14.0", "2.0", "3.5", "4.0".  D:\dev\Webservices\WonderLib\WonderLib.csproj
at Microsoft.Build.Shared.ProjectErrorUtilities.ThrowInvalidProject(String errorSubCategoryResourceName, IElementLocation elementLocation, String resourceName, Object[] args)
at Microsoft.Build.Evaluation.Project.Data.InitializeForEvaluation(IToolsetProvider toolsetProvider)
at Microsoft.Build.Evaluation.Evaluator4..ctor(IEvaluatorData4 data, ProjectRootElement projectRootElement, ProjectLoadSettings loadSettings, Int32 maxNodeCount, PropertyDictionary1 environmentProperties, ILoggingService loggingService, IItemFactory2 itemFactory, IToolsetProvider toolsetProvider, ProjectRootElementCache projectRootElementCache, BuildEventContext buildEventContext, ProjectInstance projectInstanceIfAnyForDebuggerOnly)
at Microsoft.Build.Evaluation.Project.ReevaluateIfNecessary(ILoggingService loggingServiceForEvaluation)
at Microsoft.Build.Evaluation.Project.Initialize(IDictionary2 globalProperties, String toolsVersion, String subToolsetVersion, ProjectLoadSettings loadSettings)    at Microsoft.Build.Evaluation.Project..ctor(String projectFile, IDictionary2 globalProperties, String toolsVersion, String subToolsetVersion, ProjectCollection projectCollection, ProjectLoadSettings loadSettings)
at Microsoft.Build.Evaluation.ProjectCollection.LoadProject(String fileName, IDictionary2 globalProperties, String toolsVersion)    at Microsoft.Build.Evaluation.ProjectCollection.LoadProject(String fileName)    at ProjectFileParser.ProjectHelpers.LoadProject(String fullPath, IDictionary2 args)
at ProjectFileParser.ProjectHelpers.<>c__DisplayClass0_0.b__1(ProjectInSolution p)
at System.Linq.Enumerable.WhereSelectEnumerableIterator2.MoveNext()    at System.Linq.Buffer1..ctor(IEnumerable1 source)    at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)
at ProjectFileParser.ProjectHelpers.GetProjects(SolutionFile solution, IDictionary`2 args)
at ProjectFileParser.Program.ParseSolution(String file, JObject args)
at ProjectFileParser.Program.Parse(String file, JObject args)
at ProjectFileParser.Program.Main(String[] args)

Did you try to update the csproj's ToolsVersion as per @adescamps-ullink 's comment?

No, I didn't.
Now I directly used command line MSBuild.exe mysolution.sln to build my solution

@ngyukman @adescamps-ullink I think we should take the highest of available, or maybe the closest to the one requested 🤔

I've upgraded to com.ullink.gradle:gradle-msbuild-plugin:2.18 and am trying to build a simple vs2017 project (that, of course, builds in visual studio 17) and am getting the error above, i.e.;
Error during project file parsing: Microsoft.Build.Exceptions.InvalidProjectFileException: The tools version "14.1" is unrecognized
I too don't want to change a visual studio maintained file from ToolsVersion="15.0" to ToolsVersion="14.0".  Where is the code that converts the "15.0" to "14.1"?
Thanks in advance,
Jim

hi @JamesPoli
can you upgrade to the latest?
There is a fix around the tools version in
https://github.com/Ullink/gradle-msbuild-plugin/releases/tag/2.19

Ok, will do!  I'll post back with the results.

Ng Yuk Man, that worked.  I believe it was a red herring though, I had VS2017 Professional and Enterprise installed on my machine, and only used the Professional version.  It appears code that looks for the VS version must have gotten mixed up.  I completely uninstalled all VS2017 versions and re-installed only VS2017 Professional and it's all good.


Fix Query with no key parse

Description of the Change
Fix https://github.com/Badgerati/Pode/issues/1065

Test was falsely passing as apparently:
$c = [System.Collections.Specialized.NameValueCollection]::new()
$c    $c.Add($null, 'blue')
# Key is ""

Is not the same as:
$c = [System.Web.HttpUtility]::ParseQueryString('?blue')
# Key is $null

Get-PodeCount was filtering  '?blue' but not '?blue&foo=bar' which was not consistent, now it will not filter no keys.


Abandon webapp-meta

Hi!
I just found this package in one of the projects I started maintaining / cleaning up. I assume this repo was created, then re-created as webapp-pack, so I can safely remove this one.
It still has ca. 1k installs per day (and has 3 dependents). Perhaps it might be useful to abandon this repo to reduce confusion.
What do you think?
Cheers

Will do, thank you.


Add language field to config page

Resolve #
Description:
Checklist:
You should check all boxes before the PR is ready. If a box does not apply, check it to acknowledge it.

[x] ISSUE NUMBER. You linked the issue number (Ex: Resolve #XXX).
[x] PRE-COMMIT. You ran pre-commit on all commits, or else, you
ran pre-commit run --all-files at the end.
[x] USER CHANGES. The changes are added to CHANGELOG.md and the documentation, if they impact
our users.
[x] DEV CHANGES.

Update the documentation if this PR changes how to develop/launch on the app.
Update the README files and our wiki for any big design decisions, if relevant.
Add unit tests, docstrings, typing and comments for complex sections.

Thank you for the helpful suggestions and taking your time in reviewing it @JosephMarinier  :)


'Null' is not a subtype of type 'InterfaceType' in type cast, _ContextTypeVisitor

Thread 0 main thread
CastError: type 'Null' is not a subtype of type 'InterfaceType' in type cast
at _ContextTypeVisitor.visitListLiteral(feature_computer.dart:827)
at ListLiteralImpl.accept(ast.dart:7324)
at FeatureComputer.computeContextType(feature_computer.dart:162)
at CompletionGetSuggestions2Handler.handle.<fn>(completion_get_suggestions2.dart:191)
at <asynchronous gap>(async)
at OperationPerformanceImpl.runAsync(operation_performance.dart:158)

Adding debug output in https://dart-review.googlesource.com/c/sdk/+/243840.

@scheglov: do you have and idea on how to reproduce this or is that what the debug output is for?

No, I don't know how to reproduce this.
If I knew, I would not need adding the debug output :-)


Sparklines

Initial work on sparklines. For this, they are just an in-memory set of [timestamp, float] tuples. Merge semantics are easy, as it is just set-union with timestamp for uniqueness. Timestamps expire as reports expire.
The graphs are a bit sub-optimal, but probably good enough to be useful, so will keep relatively basic for first release.
Todo:

[x] Clean up code
[x] Add tests
[x] Add other sparklines (will need some generalizing of the code)
[x] UI work (in particular, render "latest value" inline after sparkline)
[x] Squash commits

Potential other sparklines to add:

container memory usage
cpu usage (do we report this)

Threads

TCP Connections

stuff from docker?

At the moment, because we get new data every 15 seconds (and the sparklines are 15 seconds long), every 15 seconds the sparklines entirely redraw. It would be nice to smooth that out.

Another thought: we might want to come up with a way to deal with units - memory usage will be in bytes, but we might want to display it in bytes.  Similarly, CPU in %, with min and max pegged at 0 and 100, or latency in ms, or QPS in... QPS.

First two commits are pulled out into: https://github.com/weaveworks/scope/pull/631
@tomwilkie @davkal , PTAL, thanks.

Metric data are sent even for non-metric fields:
{
            "key": "Operating system",
            "value_major": "darwin",
            "metric": {
              "samples": null,
              "min": 0,
              "max": 0,
              "first": "0001-01-01T00:00:00Z",
              "last": "0001-01-01T00:00:00Z"
            }
          },

This roughly doubles the response size of a node details request compared to master.

I'd change the width to 48:

Otherwise they are too dominant.

I'd also add title attribute so that a tooltip shows the length of the timeseries.

@davkal by "length" do you mean number of samples, timespan, or both?

@paulbellamy whatever we can say for sure. At the minimum "Last 15 secs", even better "Last 15 sec, 1 sample/sec, min: 0.6, max: 1.8, avg: 1.2".

Tooltip looks great! I'd still vote for smaller width, e.g. 48px. Otherwise LGTM! Well done, jsx :heart: Paul

When you're ready, I think this need to be squashed and split into 3 PRs - the plumbing (changes to report and render/ etc), the exposing of metrics in Docker and Host probe packages, and the UI work.

Assuming this is green, LGTM.


Pion drops connection to OME after few seconds

Pion seemingly connects to OME https://github.com/AirenSoft/OvenMediaEngine
ome-to-pion-ome-1             | [2023-10-26 15:00:42.273] I [SPICE-t3478:19] ICE | ice_port.cpp:606  | Turn client has connected : <ClientSocket: 0xffff8f907010, #36, Connected, TCP, Nonblocking, <IP_ADDRESS>:50068>

but after few seconds drops the connection
ome-to-pion-ome-1             | [2023-10-26 15:00:42.276] I [SPRtcSig-t3333:10] Monitor | stream_metrics.cpp:114  | A new session has started playing #default#app/fixme on the WebRTC publisher. WebRTC(1)/Stream total(1)/App total(1)
ome-to-pion-ome-1             | [2023-10-26 15:00:42.276] I [SPRtcSig-t3333:10] ICE | ice_port.cpp:368  | Added session: 0 (ufrag: qj2PGt:ghrTrwNEJpQijbMf)
ome-to-pion-webrtc-pion-1     | 2023/10/26 15:00:47 Blocking forever
ome-to-pion-ome-1             | [2023-10-26 15:01:12.408] I [SPICE-t3478:19] ICE | ice_port.cpp:620  | Turn client has disconnected : <ClientSocket: 0xffff8f907010, #36, Disconnected, TCP, Nonblocking, <IP_ADDRESS>:50068>
ome-to-pion-webrtc-pion-1     | Peer Connection State has changed: failed
ome-to-pion-webrtc-pion-1     | Peer Connection has gone to failed exiting
ome-to-pion-ome-1             | [2023-10-26 15:01:12.420] I [DQICETmout:13] ICE | ice_port.cpp:436  | Removed session(0) from ICEPort | ice_seesions_with_id count(0) ice_sessions_with_ufrag(0) ice_sessions_with_address_pair(0)
ome-to-pion-ome-1             | [2023-10-26 15:01:12.420] I [DQICETmout:13] WebRTC Publisher | webrtc_publisher.cpp:649  | IcePort is disconnected. : (#default#app/fixme/100) reason(6)
ome-to-pion-ome-1             | [2023-10-26 15:01:12.420] I [DQICETmout:13] WebRTC Publisher | webrtc_publisher.cpp:551  | Stop command received : #default#app/fixme/100
ome-to-pion-ome-1             | [2023-10-26 15:01:12.420] W [DQICETmout:13] ICE | ice_port.cpp:527  | Agent [Unknow, 0] has expired
ome-to-pion-ome-1             | [2023-10-26 15:01:12.421] I [DQICETmout:13] Monitor | stream_metrics.cpp:136  | A session has been stopped playing #default#app/fixme on the WebRTC publisher. Concurrent Viewers[WebRTC(0)/Stream total(0)/App total(0)]
ome-to-pion-ome-1             | [2023-10-26 15:01:12.421] I [DQICETmout:13] Signalling | rtc_signalling_server.cpp:450  | Client is disconnected: HttpConnection(0xffff8c801010) : WebSocket <ClientSocket: 0xffff8f840a10, #34, Connected, TCP, Nonblocking, <IP_ADDRESS>:58704> TLS(Disabled) (#default#app / fixme, ufrag: local: qj2PGt, remote: ghrTrwNEJpQijbMf)
ome-to-pion-ome-1             | [2023-10-26 15:01:12.504] I [SPRtcSig-t3333:10] HTTP.Server | http_server.cpp:216  | Client(<ClientSocket: 0xffff8f840a10, #34, Closed, TCP, Nonblocking, <IP_ADDRESS>:58704>) has been disconnected by *:3333

Repro is here https://github.com/wildfluss/ome-to-pion which is basically reflect example but with OME signaling and few tweaks which I've tried to do by reading other examples (eg unreal-pixel-streaming from example-webrtc-applications )
Disclaimer: I'm WebRTC n00b

@wildfluss - did you ever get to the bottom of this? I am having an issue that seems superficially similar (disconnects after exactly 30 seconds), only the remote is using libjuice not OME. It appears at least in my case that the connection is actually being dropped by the remote (DTLS CloseNotify comes from the remote) with Pion, but with Chrome all is OK. I suspect that Pion is not doing something quite right with keepalives, but despite looking into this for several days and looking at tcpdumps I haven't been able to figure it out.

I suspect that Pion is not doing something quite right with keepalives

@adriancable , unfortunately, same here, Pion lacks documentation and I could not figure out what was that in about a day and gave up on using it, otherwise could be great library but taking into account amount of net layers involved in the protocol it would just take far too long to understand whether its Pion not compliant / remote is funky or what you know :(


Publish PPA automatically

Inspired by:
https://github.com/yuezk/publish-ppa-package/blob/main/build.sh
This script needs to be updated to reflect the cargo vendoring step needed (see https://github.com/fujiapple852/trippy/blob/master/debian/README.debian).
Note that this script is able to publish for all supported Ubuntu versions.

Step (roughyl) to publish manually (for 0.10.0 on Jammy):
docker run -it -v (pwd):/data ubuntu:jammy

apt update && apt install build-essential devscripts debhelper cargo wget

cd /data

wget https://github.com/fujiapple852/trippy/archive/refs/tags/0.10.0.tar.gz
mv 0.10.0.tar.gz.1 trippy_0.10.0.orig.tar.gz

mkdir build
mv debian build
cd build

# build vendored dep crates
./debian/rules vendor

# setup gpg and import launchpad key pair
# gpg --import ...

# build PPA source package
debuild --prepend-path ~/.cargo/bin -S -sa

# publish to launchpad
dput ppa:fujiapple/trippy ../trippy_0.10.0-1ubuntu0.1~jammy1_source.changes

Issues found when publishing 0.11.0.
1: The downloaded tarball doesn't have the .1 suffix:
mv 0.10.0.tar.gz.1 trippy_0.10.0.orig.tar.gz

I just changed the command to remove it.
2: the changelog file needs to be updated for each release, rather than appended to.  I did this by hand and manually patched the tar file as I had already cut the github release before updating the version.
trippy (0.11.0-1ubuntu0.1~jammy1) jammy; urgency=medium

  * 0.11.0 release.

 -- Fuji Apple<EMAIL_ADDRESS>Sun, 11 Aug 2024 20:55:00 +0800

3: For gpg --import I copied the launchpad_secret_key.pgp to build and imported with gpg --import launchpad_secret_key.pgp
4: debuild failed with:
error: unwanted binary file: debian/trippy_0.11.0.orig.tar.gz

Because I had debian/trippy_0.11.0.orig.tar.gz as well as in the git repo root (the parent dir of build). We need the later, not the former, so deleted it.
5: debuild failed with:
dpkg-source: error: aborting due to unexpected upstream changes, see /tmp/trippy_0.11.0-1ubuntu0.1~jammy1.diff.NQ5XoT
dpkg-source: info: Hint: make sure the version in debian/changelog matches the unpacked source tree

This is the point when I manually patched the tar file (#2 above), but that didn't fix it (but still needed, next time do it before the github release).  The actual error was:
dpkg-source: warning: file build/launchpad_secret_key.pgp has no final newline (either original or modified version)
dpkg-source: info: local changes detected, the modified files are:
 build/launchpad_secret_key.pgp

I moved the key to the parent directory and that fixed it.


OAuth support

Overview
Add OAuth support for public-facing use cases.
The OAuth flow:

Unauthenticated requests are redirected to a page with a list of providers.
Client selects one provider from the list of configured providers (Google and GitHub are implemented.)
Client is redirected to the provider to approve Waiter's access to the client's credentials.
Provider redirects user back to Waiter with a token that can be used by Waiter to verify the user.
After authentication, client is redirected to original location, with path, query-string, and hash preserved.

OAuth requires an application to be registered with the provider.  Upon registering an application, the provider provides a client-id and a client-secret.  These values are configured via settings, where loading these values via environment variables is preferred for security.
Providers also require the URI redirecting back to the application to be specified when registering.  Some providers allow multiple callback URIs (e.g. Google), while others only one (e.g. GitHub).  Providers that allow multiple redirect URIs to be specified are preferred, since Waiter will likely be hosting multiple applications with different hostnames.  Each hostname hosted by Waiter will need to be registered.  Using a provider that supports only one redirect URI limits usage of the OAuth support to one hostname.
OAuth is, at its core, not a method of authenticating users, but it can be used for authentication if the provider has a way to look up a user's verified email address using the access_token provided.  Open ID Connect is a standardized flow for this, but while Google supports Open ID Connect, GitHub does not.
The user's email address is used as the principal for Waiter.  This implementation still expects a single run-as-user to be configured since we can't reliably map a principal to a username that makes sense to the scheduler.
If per-user instances are desired, a ServiceDescriptionBuilder can be used to add principal metadata to the service description, ensuring unique services per user.
Integration tests for OAuth would be difficult, given that interactive authorization by a user is required.
Demo
After trying to visit localhost:9091/settings, the browser is redirected:

After choosing Google as the provider:

GitHub has its own login page.
After successfully logging in, browser is redirected back to localhost:9091/settings.

Please rebase.


Vulkan: WorldEnvironment Glow effect's size is resolution-dependent

Related to https://github.com/godotengine/godot/issues/56911.
Godot version
4.0.dev (fe6c65a1a)
Still reproducible as of 4.0.dev (ce6de717f)
System information
Fedora 34, GeForce GTX 1080 (NVIDIA 470.74)
Issue description
The glow effect's size is resolution-dependent. This means the glow effect will appear to be larger as the rendering resolution decreases.
This means the rendering will change in unintended ways depending on the user's monitor resolution. Also, if we implement dynamic resolution scaling and FSR in the future, we need to make sure every effect is as a resolution-independent as possible.
https://user-images.githubusercontent.com/180032/139929516-f7eb7492-15c7-4b7e-9f81-a4a8b4d302ab.mp4
Steps to reproduce

Enable Glow in WorldEnvironment.
Create a MeshInstance with a SphereMesh.
Add a StandardMaterial3D on the SphereMesh, with a bright albedo and/or emission value (to make glow kick in).
Decrease the 3D resolution scale factor at run-time using get_viewport().scale_3d -= 0.1.

Minimal reproduction project
https://github.com/Calinou/test_fxaa_supersample_master
Press the Up / Down arrow keys to adjust the 3D rendering scale. When doing so, watch the glow effect on the floor become larger as the resolution decreases.

On Reddit, someone reported a similar issue on master but with the opposite happening. The glow effect became weaker at lower resolutions (possibly because details were too small to be caught in the mipmaps)? https://www.reddit.com/r/GlobalOffensive/comments/qu9psf/hidden_easter_egg_on_ravine_music_plays_from_this/

I managed to ameliorate the issue a bit by sliding the glow levels around.
The results are okay if the scaling is a power of 2 (0.5, 2, ...) but kind of iffy in-between.
Also it doesn't work that well if the user uses the outer glow levels.
https://user-images.githubusercontent.com/9743949/151017405-833bdf9b-e28e-4250-be2a-1770df4f7312.mp4
The depth of field effect is stable throughout the scaling range though, so maybe it is possible to reimplement the glow shader in a similar fashion.
Anyway, I pushed the changes to a branch if anybody wants to have a look.

@ceLoFaN This looks pretty good, especially for depth of field – feel free to open a pull request for this :slightly_smiling_face:


Deployed API PoC

As an API program owner I want a deployed "mock"/sample implementation of my API (a PoC) that potential users of my API can access to explore the functionality, and to support their collaboration with me on the API spec and usage models.
DoD: a URL exists that a client can make calls against that will return a sample data response.  This is likely implemented generated stub API (based on swagger file), deployed in a lab instance, behind kong, and automated via Jenkins.

What's the URL fro accessing the PoC API?

last question was for @f-w

https://lab-bca-api.bcdevexchange.org, explorable using https://lab-bca-api.bcdevexchange.org/docs, for example, enter quadra in address field.


Refactor of pull/push/copy

do not merge

At this point, this PR is ready for review

At this point, this PR is ready for review


add make release-bin target

What type of PR is this:
Adds new release-bin make target.
This is for the homebrew package, and it could be also used by another packaging system.
The problem is that currently, odo-dev formulae  uses make bin, this is a problem because this build odo with dev Segment key. This means that MacOS users installing odo via homebrew don't show up in our production telemetry.
new make release-bin uses prod Segment key and it should be used only in packaging systems.

What does this PR do / why we need it:
Which issue(s) this PR fixes:

PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

What type of PR is this:
Adds new release-bin make target.
This is for the homebrew package, and it could be also used by another packaging system.
The problem is that currently, odo-dev formulae  uses make bin, this is a problem because this build odo with dev Segment key. This means that MacOS users installing odo via homebrew don't show up in our production telemetry.

Just curious, because I am not familiar with Homebrew :-)
Could the odo-dev formula download the right binaries (and checksums) directly from https://developers.redhat.com/content-gateway/rest/mirror/pub/openshift-v4/clients/odo (at least when a build from sources is not required)?

/retest

/override Kubernetes-Integration-Tests/Kubernetes-Integration-Tests
/override OpenShift-Integration-tests/OpenShift-Integration-tests
/override ci/prow/v4.11-integration-e2e
Non code-related changes
The failing tests are fixed by another PR


chore(test): fix localization fallback config

Description
This PR reverts the localization test configuration changes that broke the fallback locale test - https://github.com/payloadcms/payload/pull/6981/commits/abd9f7a15cf174ff1ddd856e387dce4453380038#diff-52f2ded97e96189e2bcc0c1f69cc1399726fd2d2382c9bbc295166521f783e34L301

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Chore (non-breaking change which does not add functionality)

Checklist:

[x] Existing test suite passes locally with my changes

@kapilbarad thanks! Looks like this has since been addressed. Closing.


regex variable defined as string treated as a compiled regex

@bsmithyman @fwkoch
import properties

class TestUID(properties.String):
    class_info = 'a child of properties.String'
    regex = 'abc'

class TestResource(properties.HasProperties):
    uid = TestUID('id of the resource')

resource = TestResource()
resource.uid = 'abc'

The code above throws an exception.  I believe the variable regex above is allowed to be a string, but it's treated as a regex in the validate method.

Thanks, @riscy - The reason validate assumes this is compiled is because in the setter coerces strings to compiled regex - see here: https://github.com/aranzgeo/properties/blob/v0.3.5/properties/basic.py#L962 - The way you define TestUID, however, overrides the @property regex with just a string (so the setter is no longer called).
Currently, you can replicate the above class in a way that calls the setter with something like:
import properties

class TestUID(properties.String):
    class_info = '...'

    def __init__(self, *args, **kwargs):
        kwargs.update({'regex': 'abc'})
        return super(TestUID, self).__init__(*args, **kwargs)

However, it would be nice to define a class as you do above. This has implications beyond regex though. You could override any attribute (well, @property) in a properties.Property class this way. That would mean moving all setter validation to somewhere it can be run if the @property is overridden - eg on class construction or inside validate...

I see!  Thanks for the explanation and the solution.


⚠ gorzdrav.org has degraded performance

In 61d69e9, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 4646 ms

Resolved: gorzdrav.org performance has improved in c0de829.


CompositeFrame schema disagrees with implementation

The CompositeFrame JSON schema defines this object as either being an array of frames, or as being a single frame. However, the implementation of CompositeFrame only supports the array of frames. Which is correct? There are no tests for the latter case, and removing support for it would allow for simplified code since it would mean that the CompositeFrame schema version would not have to be bumped every time that the frame schema changes.

Since the single frame case was not supported in code to begin with, I am strongly in favor of removing it altogether. However, I can't say I entirely understand all the use cases, so feedback is welcome.

GWCS uses only an array of frames. I think it's OK to remove support for single frame.


Making VDiagram Taller

Per https://internal.cida.usgs.gov/jira/browse/AQCU-615
Adjusting from 6 to 11

instead of


Basic Pagination Implementation

Implement a general pagination setup for all get routes for larger testing sets.

https://laravel.com/docs/8.x/eloquent-resources#pagination

Where exactly ?

@slowloris-x Currently all controller index methods return all results found in the database. Giving the option of pagination to these functions would be the aim!


oops something went wrong

the page didn't load google maps correctly. see java script console for details..........
every time i log in. please halp looked for fixes

Maybe API isn't correct? I had that issue before just generate a new one

Same problem, i did generate new api, unless i typed in wrong. It wont paste into the map. Still getting this error?

Do you have google maps geocoding Api and google maps JavaScript Api enabled

I do think i do..looks like
On Sun, Aug 7, 2016 at 4:50 PM, chintqm03<EMAIL_ADDRESS>wrote:

Do you have google maps geocoding Api and google maps JavaScript Api
enabled
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mchristopher/PokemonGo-DesktopMap/issues/717#issuecomment-238110800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT7OGE6BswFgI4Dpsj_rJ_3L_nSi4Dvtks5qdlMfgaJpZM4Jek-S
.

Same problem for me as well

Sigh.  Me too.  I dunno if this helps, but I'm getting this:
Uncaught TypeError: Cannot read property 'getSouthWest' of undefined


Make image 35+M smaller

ARM image is already above 250M and fails to build, and amd64 image is at 249M today.
Once we figure out other places to save space we can revisit this container.

Hey @eriknordmark -- lets hold off on this and just use it in our private builds for now as need it. I'd rather not see it merge, because if we're killing LISP -- that's a separate discussion and we should take it as such.
For immediate needs -- I don't think there's any blockers right now (aside from ARM which I'm going to fix shortly)


Integrated terminal loses environment variables.

Does this issue occur when all extensions are disabled?: Yes/No
yes

VS Code Version: 1.90.0
OS Version: win10 22H2 19045.4291

Steps to Reproduce:

When I start the integrated terminal and enter commands such as 'gcc -v (scoop installed)', they don't work, but they work in an external terminal. Similarly, I've lost the environment variables for several tools installed through scoop, making them unusable within VS Code.

did this work in earlier versions? what shell are you using?

The issue arose after updating to v1.9. The screenshot shows Nushell, and other shells such as pwsh have the same problem.

does disabling shell integration, terminal.integrated.shellIntegration fix it for you?

The issue has been fixed, possibly a random bug after a computer reboot.


Create returnTheNextNumber.py

This program will return whether Friday 13th comes in the given month or not.

[ ] Task Complete


Is there a way to limit calculated combinations to just 1 result

I'm interested in getting just the first possibility of a regex.  e,g. the first decimal in /0.\d+/ .I was hoping to avoid CPU of generating all possibilities before running the callback to exit at first result.
Thanks,
Damion

I think this is the way to go https://github.com/fent/randexp.js?

Ah, well that is where I started and it is great.  But they randomize choices across alternatives, e.g. (a|b|c) yeilds any of a,b,c randomly, rather than a, then b, then c.  I was hoping for the latter.

Managed to find a solution with randexp - by tuning randomizing function always to select first value.
Thanks for your time!


Print pid of nodes when started on Windows

This is an interim change to facilitate tracking down if specific dtests are consistently hanging. Windows only and just a little verbosity on node start.

This won't work on Python 3, sorry. -1

Updated to use six print version to stderr.

+1! Thanks.


feat: setup WIF for devtools image push

This setups up WIF for use with google-github-actions/auth to push devtools image

This has been applied


Upper part of the application window is cut off when snapped

Describe the bug
When a UWP/WinUI application is snapped to one side (in desktop or tablet mode), the upper part of its window is cut off.
Steps to reproduce the bug

Steps to reproduce the behavior:

Open a UWP app (e.g. the store)
Snap it to the left side (e.g. via keyboard short)
Oberserve that the upper part of the window is cut off.

Expected behavior

Screenshots

Version Info

NuGet package version:

This template needs to be updated to include the most recent released Windows version...

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2019 Update (18363)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

Can't be reproduced.
You'd need to provide screenshots and detailed information about your display hardware/settings/configuration to get anywhere with this.

Screenshot of Microsoft Edge when maximized:

Window snapped to the left with Win+Left shortcut:

I can see this issue on a Surface Go (1820x1200 Resolution, 150% scaling) and on a Desktop PC (4k resolution, 200% scaling), both running Windows 1909.

There must be some edge-case, non-standard setting or software on both systems. Otherwise it'd be easy to repro and heavily reported by now. Try logging into Safe Mode or a different user account to see how it appears there.
The Windows Sandbox shows it working fine:

Can't be reproduced.
You'd need to provide screenshots and detailed information about your display hardware/settings/configuration to get anywhere with this.

Easy to reproduce.

@shaheedmalik
The OP clearly stated:

When a UWP/WinUI application is snapped to one side (in desktop or tablet mode)

It is not reproducible in desktop mode on any of my devices.
If it only repros in tablet mode then that is an important detail and the issue should be corrected, including the title.

I tested it again on another machine (with a completely unrelated non-microsoft user account) and see the same issue. All machines are on build 18363.476 and I can reproduce it in desktop and tablet mode.

If anyone outside of @lhak can repro this in a normal Windows desktop environment, then please let us know.
Until then this is the difference between an issue impacting:

800+ million desktop users
~1 million tablet mode users
? users in desktop or tablet mode under very specific, yet unknown circumstances

It's easily reproducible in both modes.

Open window in desktop mode. Win+Up, Win+Left

So if you specifically:

First hit the Win+Up to maximize a window
Hit Win+Left/Right to snap the window

then the issue can be repro'd (goes away if you his Win+L/R again).
However, this happens to all windows, not just UWP/WinUI ones. So this is a Windows 10 issue that belongs in the Feedback Hub, rather than something that should be addressed here.

So if you specifically:

First maximize a window
Then hit Win+Left/Right to snap the window

Then the issue can be repro'd (and goes away if you hit Win+L/R again). Maximizing first wasn't part of the original steps.
However, this happens to all windows, not just UWP/WinUI ones. So this is a Windows 10 issue that belongs in the Feedback Hub, rather than something that should be addressed here. @lhak
@shaheedmalik Thanks for the double downvotes btw!

The issue also occurs when you snap a window at the the top of the screen (via mouse) to maximize it.

@lhak thank you for opening this issue. As @kmgallahan pointed out, the problem isn't limited to UWP apps and won't be fixed in WinUI. Feedback Hub is definitely the best place for these types of issues, so please feel free to upvote the feedback I opened based on the information that was provided here.

Sorry to bump up this closed issue, but I can't seem to open your link for the feedback issue @YuliKl is there a Issue ID or smth I can search for? I'd like to vote for it because apparrently I'm one of the 800million users affected by this. Though I wouldn't have noticed without this issue.

Yulia's issue is internal and I don't know how to make it public. @jasonwurzel (or anyone) can you file a Feedback Hub item? I'll link it to the one Yulia created.


Serve the web page from main.py

something like
from fastapi.staticfiles import StaticFiles
app = FastAPI()
app.mount("/static", StaticFiles(directory="static"), name="static")

#63 solves it i think


Implement sphere_to_plane

[x] Implementation
[x] Tests
[x] Docs

Closes #181.

On second thought, I'm unsure if I got this right. Should x_origin and y_origin be lon_origin and lat_origin instead? x_origin and y_origin don't seem useful.

On second thought, I'm unsure if I got this right. Should x_origin and y_origin be lon_origin and lat_origin instead? x_origin and y_origin don't seem useful.

Yes. My understanding is that you set the origin, for example, (0˚, 50˚N) of a domain [-10˚W, 10˚E] and [40˚N, 75˚N] that will map to [0,0] in the tangent plane.


Feature/add test pypi publish add spectree link

This adds a link to Spectree in the docs, and adds an Action to push the library to Test PyPi on release.

We should probably rename the existing workflow as it's called "Python package" but it doesn't do any packaging :confused:


Mistakes repository for Ruby

Part of #300

LGTM plz check if everything runs with the errors fixed, I can do that tomorrow as well


Add docker compose for the production and development server

This is the name of the image of this specific container not of the
project, so it should be the name of the container with a tag.
Transcendence is the name of the whole project (which should be in .env btw
as
COMPOSE_PROJECT_NAME=ft_transcendence
)
Using latest tag is bad practice (some reading
https://vsupalov.com/docker-latest-tag/). For instance if you have another
project running another container named web with no tag, which is gonna be
autotagged latest, it's going to be a mess
On Thu, Dec 22, 2022, 11:18 Valentin @.***> wrote:

@.**** commented on this pull request.
In docker-compose.yml
https://github.com/vfurmane/ft_transcendence/pull/1#discussion_r1055305977
:

@@ -5,6 +5,8 @@ services:
build:
context: .
dockerfile: ./apps/web/Dockerfile

container_name: web
image: web:transcendence

Why is the name of the image web and the tag transcendence? Isn't it
supposed to be the name of the project and the version (e.g.
ft_transcendence:latest)?
—
Reply to this email directly, view it on GitHub
https://github.com/vfurmane/ft_transcendence/pull/1#pullrequestreview-1227439706,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGAERW4UHUIVDASITY2I2PTWOQTF3ANCNFSM6AAAAAATFQRG4M
.
You are receiving this because your review was requested.Message ID:
@.***>


Use onClick for FormlyFieldConfig fetched from db

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[X ] support request

Current behavior
When i load FormlyFieldConfig from external resources i'm not able to call any function
Expected behavior
Minimal reproduction of the problem with instructions
see https://stackblitz.com/edit/angular-t9xn4r
What is the motivation / use case for changing the behavior?
I would like to call a function from an externally defined FormlyFieldConfig
Please tell us about your environment:

Angular version: 6.1.X
**Ngx-formly: 4.7.2
Browser: all

use bind(this), see https://stackblitz.com/edit/angular-t9xn4r-kpx1pc


Region editor uses linear filtering, always

Godot version
v4.0.beta.custom_build (3229194df)
System information
Windows 10, Vulkan, NVIDIA GeForce RTX 3070
Issue description
The region editor always displays textures using linear filtering, despite the Sprite node having filter set to nearest.
Reproducible in v4.0.beta2

Related to #52332 and possibly #60150.
While trying to come up with a fix to this, I found that it is most likely due to how TextureRegionEditor is drawing the sprite (similar to #52332).
https://github.com/godotengine/godot/blob/6b986f18554105671c474ec08ca02a19d9cd74a2/editor/plugins/texture_region_editor_plugin.cpp#L88-L91
It seems that using draw_texture, the final appearance depends on the panel's filter state for some reason (shouldn't the zoom transform be scaling the texture before it is drawn?)
I was able to fix the issue by changing the panel's filter to match the filter of the object being edited, but this (I think) subtly affects all of the gizmos and lines being rendered. I don't like this solution. It also neglects the "inherit" filter mode, but I think https://github.com/godotengine/godot/pull/66546 will be able to solve that part.
I am going to continue working on this, but I am still new to the codebase so any advice would be appreciated :)
Steps to reproduce

Create a sprite node with a small resolution picture (e.g. pixel art) to make the linear filter more noticeable.
Enable region
Set filter to Nearest
Open region edit and zoom

Minimal reproduction project
No response

After some more experimentation it seems like the easiest and least intrusive fix to this issue as well as #52332 would be to add a "set filter" command to the rendering server so that the filter mode for a canvas item can be overridden on a per "render item" basis (which was also suggested in #52332 by @groud). This seems pretty straightforward to implement if modeled after CommandTransform, but I'm not confident I know what impact that would have.


Zend\Code\Scanner\MethodScanner does not account for scalar type declarations

I believe the issue is with MethodScanner and not ParameterScanner, as this is what is actually parsing the PHP tokens - it correctly ignores non-scalar type declarations such as array and callable. From what I can tell, string, int, bool and float are tokens of type T_STRING. As a result, the scanner believes they are classnames. This can result in classnames that simply don't exist.
I'm not really sure how this can be worked around inside the scanner. As far as I can tell, no PHP version checks are performed on the code being parsed, which means there's no reliable way of detecting whether or not string is an actual classname in PHP <7.0 code, or a scalar type declaration in PHP >=7.0 code.
Perhaps there needs to be some way to globally set the PHP version of the code being scanned?

As an aside to this, it would be nice if ParameterScanner and MethodScanner were able to handle non-class type declarations of any form.

It looks like there's a related issue with classes that have a leading namespace separator. MethodScanner doesn't appear to detect this, and fails to pass the token onto ParameterScanner. ParameterScanner then uses NameInformation to resolve the classname as if it is relative to the current namespace, rather than as an FQCN.
The issue is definitely with MethodScanner. If you create a completely separate instance of ParameterScanner and pass it just the tokens for a single parameter, including the leading T_NS_SEPARATOR token, it will correctly identify the FQCN, and NameInformation will correctly resolve it as an FQCN.

I've managed to create a patch to resolve the second issue, see #60. The original problem, incorrectly detecting scalar type declarations, is still an issue.

I had to re-submit the PR as #61.

This repository has been closed and moved to laminas/laminas-code; a new issue has been opened at https://github.com/laminas/laminas-code/issues/27.


Incorrect assert,  Compiler warning fix

Hi,
Please check proposed changes.
Regards,
Deon Marais

Thank you!


update Dockerfile

add -ldflags '-s -w' in Dockerfile
remove the main.go in build

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

@hurricane1988 Thanks for the contribution. Would you sign your commit like below?:
git commit -s --amend
git push -f

@wanjunlei @wrongerror @tpiperatgod , please help to review

@wrongerror could you review my request?

Could you please describe the purpose of this PR? @hurricane1988

@hurricane1988 Thanks for the contribution. Would you sign your commit like below?:
git commit -s --amend
git push -f

@wanjunlei @wrongerror @tpiperatgod , please help to review

@hurricane1988 You need to sign your commit first. And then you'll need to examine why the E2E test failed with your PR?

@hurricane1988 Thanks for the contribution. Would you sign your commit like below?:
git commit -s --amend
git push -f

@wanjunlei @wrongerror @tpiperatgod , please help to review

@hurricane1988 You need to sign your commit first. And then you'll need to examine why the E2E test failed with your PR?

OK, I will examine E2E test first.

@hurricane1988 Your commits look a bit messed up. You may need to rebase or squash your commit:
git remote add upstream https://github.com/OpenFunction/OpenFunction.git
git fetch --all
git checkout main
git rebase upstream/main
git commit -s --amend
git push -f

Could you please describe the purpose of this PR? @hurricane1988
@wrongerror refer to https://go.dev/ref/mod, go mod tidy method "go mod tidy ensures that the go.mod file matches the source code in the module. It adds any missing module requirements necessary to build the current module’s packages and dependencies, and it removes requirements on modules that don’t provide any relevant packages", and go mod download method "The go mod download command downloads the named modules into the module cache.", I think we could speed up the building time with "go mod tidy" method.


Error when using DTBook to EPUB 3 script (ttsConfig.xml file does not exist)

This is the error:
Input not found: file does not exist: C:\Users\ngaines\AppData\Roaming\pipeline-ui\ttsConfig.xml

The issue was reported by @nicolegaines.

is this with RC1?

Yes.

Yes, TTS is enabled
No voices are selected as brand new install and therefore no config is generated yet presumably
I suggest ideally error with instructions to select TTS settings etc.

Does the job finish with an error? Is the conversion completed?

yes, Error
Input not found: file does not exist: C:\Users\PaulW\AppData\Roaming\pipeline-ui\ttsConfig.xml
conversion is not complete

If you are able to send log files, you can email me marisa [dot] demeglio [at] gmail and I can have a look.

@marisademeglio Thanks for the quick fix!


Retain error types in the API response

Currently, only query errors retain their type in the API response.
If other Torii errors do as well instead of being stringified, the client can restore the error response to torii::Error or an equivalent type.
This allows for more reliable error handling

Could be replaced by #1455 in the future


Added pysatMagVect

@rstoneback I think my recent PR (#23) is what created the conflict here, specifically the switch from using code_url to code in the project metadata for the URL of the source code. Sorry about that!

Addressed in #28


Convert to .5 format

Converting to the .5 format for beanstalk
Adding vendorized versions

@tboerger everything ok for you? let me know if I need to change anything :)

LGTM

@bradrydzewski If you're ok to merge this in. I can also take over this plugin and start publishing images.


MRO X2.1-777

Describe the bug
after flashing PX4 to the FC, QGC doesn´t see it anymore via USB
To Reproduce
Steps to reproduce the behavior:
unpack it
QGC connects to Arduplane (ChibiOS)
mro_x21-777_default.px4 flash successful to:
Device: ArduPilot ChibiOS
Version: 5
Board ID:136
(prebuild bin or from source)
after the flash procedure and reboot to PX4
QGC doesn´t find it anymore via USB
there is also nothing special to find on the SD
no logs available
Expected behavior
QGC should connect to it after reboot
Is there an adapted boot loader available to change it from ChibiOS to NuttX ?
also in PX4 discussion:
https://discuss.px4.io/t/px4-on-mro-x2-1-777/17938

I'll take a look, it might be QGC not recognizing it for some reason.

it seems that the boot process does not finish, the led which flashes fast for a short time before the board is ready does not stop flashing, like when the update of the IO board does not finish.
Flashed back to ChibiOS ArduPlane QGC connects immediately without problems in the moment when the led stops blinking.

@dagar can I do tests with changed start scripts to look in which module the start procedure stops?
e.g. flash a version without io-unit

Should be fixed in https://github.com/PX4/Firmware/pull/15521.


Ready Event after loading data and displaying the same

Sometimes when we are loading a lot of data during initialization, the chart area stays blank and unresponsive for a few seconds or more.
It would be useful to have a ready event to signal that the chart has loaded the data and drawing is complete.
We can then show a overlay wait animation, etc. to help users understand what is happening.
This event can also be used to lazy load data in batches for a better UX

Sometimes when we are loading a lot of data during initialization, the chart area stays blank and unresponsive for a few seconds or more.

How much data you put to the library? Afaik it works fine even with thousands of points very quickly. Can you provide an example please?

It would be useful to have a ready event to signal that the chart has loaded the data and drawing is complete.

See #450

Thank you for the prompt response.
In one instance we had to load 150k points. it worked but the wait was long and we want to hint the end-user with a wait animation or something.

Now that 150k example is not working anymore..
Here is a test page I created for that - https://ifiva.com/test.html
Well we are loading 2 different data series - 1.candles 2. markers (small
file) and we also create a table to scroll to the candle as per marker. So
we cannot lazy load on scroll / view.
I could load incrementally if we had a way to know when the previous data
load was successful by an event.
ᐧ
On Tue, May 25, 2021 at 8:42 PM Evgeniy Timokhov @.***>
wrote:

In one instance we had to load 150k points. it worked but the wait was
long and we want to hint the end-user with a wait animation or something.
Is it possible to provide example of that? Probably there are points of
improvements.
Also, is there any reason why you put all the data at once instead of
portion them due scrolling/scaling?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tradingview/lightweight-charts/issues/746#issuecomment-847947773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD32QZP4K7MC7EONSJUHLTTPO44XANCNFSM45LIXG7A
.

--
Dr. Praveen Baratam
Tel: +91-98495-92355
https://www.linkedin.com/in/praveenbaratam/

Here is a test page I created for that - ifiva.com/test.html

@praveenbm5 What creds I need to use to access the page?

you can use username: test and password: test

It would be useful to have a ready event to signal that the chart has loaded the data and drawing is complete.

I think #450 is what you need.
From the profiler I noticed that you parse huge csv file and it takes 15s on my pc but the library's setData call and the first paint only takes 1.3s and 700ms respectively (this is with enabled profiler). Probably if you optimize loading the data it might be much faster.


[qqmusic] Clean description/lyrics

The extractor puts lyrics into the description and it's really useful. However the lyrics are filled with lots of extraneous "\n", not linebreaks but actual text, "" (slash) and "n", making it difficult to read since it looks like just a big block of text.
$ youtube-dl --get-description "http://y.qq.com/#type=song&mid=001LmqZ03vqdCg"
[ti:Infinity]\n[ar:Mariah Carey]\n[al:#1 To Infinity]\n[by:]\n[offset:0]\n[00:00.10]Infinity - Mariah Carey\n[00:00.20]\n[00:04.12]Ooh Aah, Ooh Aah, Ooh Aah, Ooh Aah\n[00:10.18]\n[00:10.74]Ooh Aah, Ooh Aah, Ooh Aah, Ooh Aah\n[00:17.19]\n[00:18.40]Why you mad\n[00:19.15]\n[00:19.66]Talkin' 'bout you're mad\n[00:20.79]\n[00:21.49]Could it be that you just lost the best you've ever had\n[00:24.62]<-- snip -->

This change replaces the \n with real linebreaks. After patching, it also looks to be a valid [LRC] (http://en.wikipedia.org/wiki/LRC_(file_format)) format.
$ python -m youtube_dl --get-description "http://y.qq.com/#type=song&mid=001LmqZ03vqdCg"
[ti:Infinity]
[ar:Mariah Carey]
[al:#1 To Infinity]
[by:]
[offset:0]
[00:00.10]Infinity - Mariah Carey
[00:00.20]
[00:04.12]Ooh Aah, Ooh Aah, Ooh Aah, Ooh Aah
[00:10.18]
[00:10.74]Ooh Aah, Ooh Aah, Ooh Aah, Ooh Aah
[00:17.19]
[00:18.40]Why you mad
[00:19.15]
[00:19.66]Talkin' 'bout you're mad
[00:20.79]
[00:21.49]Could it be that you just lost the best you've ever had
# -- snip --

Thanks for the fix. I've adopted your idea in b813d8c


Make demo with just an abstract

https://levin-lab-dev.vercel.app/paper-producer

Heh first couple sentences kinda work.


Norminette recognizes a function as a global variable

Describe the bug
Subject Prototype:
Erroneous code
static	char*(*sprep(char j)) (char *s, t_td *data);
Additional infos

OS: Xubuntu 20.04
python --version: 3.8.10
norminette -v: 3.3.1

Additional context
How to fix it?

Hello,
The proper way to define such a function prototype is :
static char *(*sprep(char j))(char *s, t_td *data);
Have a good day


BUG in file association ico path generation

This was accidentally posted in the RemoteApp Tool issues. Reposting here. Thanks @seflerZ

Easy fix. Only two and a half years late!


Set client_secret to nubank login after api changes

fixes #2

Nice! I had added a client secret before but was uncertain if it was indeed necessary


Fix problems found on deployed app

Description
The deployed app has a problem with the search bar because the communication to Outstatic leads to the app trying to read a file that is not there.
This needs to be fixed ASAP, even if the solution is temporary. After that,  a more permanent solution can be sought after.

This has been fixed for the moment and version 1.0.1 has been deployed to the production env.


Export-SqlLogins

OS - Window Server 2012R2
SQL - 2014
PS - 4.0
Export-SqlLogins exports all the database logins eventhough selected database like below,
Export-SqlLogin -SqlInstance Prod1 -Database Test1 -FilePath '\Location\backup\login.sql'

Please include version of dbatools module you are using...

I can duplicate this in the latest version under development though.

I tested in Beta 0.9.22

closed via https://github.com/sqlcollaborative/dbatools/pull/3296


spelling correction

there is not a README.rst file.

Thanks! This SDK is technically deprecated, but we appreciate it.


The test datasets used for evaluation metrics

Thank you very much for your contribution and for sharing it. I have always been curious about the evaluation metrics for co-speech, and I would like to ask whether the test datasets used for the metrics in your paper are the same as the ones used for Camn. I noticed that the test datasets in your code are somewhat different from Camn in terms of LMDB loading. If you could spare some time to answer this, I would be very grateful.

Hi lovemino, the test set should be the same as CaMN except that I converted the Euler rotations into axis-angle format. Therefore, the autoencoders to compute Fréchet Distances are also retrained for axis-angle rotations. Note that the Frechet Distance code of CaMN did not turn on evaluation mode, while the results in our paper are after correcting this issue.

Hi lovemino, the test set should be the same as CaMN except that I converted the Euler rotations into axis-angle format. Therefore, the autoencoders to compute Fréchet Distances are also retrained for axis-angle rotations. Note that the Frechet Distance code of CaMN did not turn on evaluation mode, while the results in our paper are after correcting this issue.

Thank you very much for your reply. I found your paper to be meticulously written and it is certainly a valuable read. However, as I attempted to reproduce the experimental results, I encountered some difficulties. Specifically, I found that the LMDB used in CAMN caused errors when used in your project.
Would it be possible for you to provide the processed LMDB file for the test datasets? I would greatly appreciate it.
Thank you for your contribution and response.

Hi, this is because I am using the latest version of lmdb. You can try replacing the lmdb in BEAT with latest version to generate the data cache. Regarding releasing the processed test dataset, I would need to check the license of the dataset. Let me know if upgrading lmdb helps :)
am very grateful for your prompt reply and look forward to you making the LMDB public. Once again, I want to express my appreciation for your contribution and I look forward to your future papers.Thank you very much.

Based on your response, I used the build_cache function from your beat.py and the test set from CAMN to generate a new data.mdb file. Then, using the ges_axis_angle_300 weights and the CAMN test code, I calculated the axis_angle gestures of the 141 upper body joints (before converting to a matrix) during model inference. However, The resulting FDG was 4282.439,  which is 10 times higher than the FDG: 438.93reported in your paper. I would like to ask if you could kindly provide the code and files you used to calculate these metrics. This would help me accurately reproduce the metrics reported in your paper.

Hi, this is because I am using the latest version of lmdb. You can try replacing the lmdb in BEAT with latest version to generate the data cache. Regarding releasing the processed test dataset, I would need to check the license of the dataset. Let me know if upgrading lmdb helps :)

Hi lovemino, before sharing the code, here are some traps you can check:

Have you turned on the eval mode?
Have you normalized the motion before feeding them into AutoEncoder of FGD?
What is your FGD of CaMN with our autoencoder checkpoint (convert to axis-angle)? Is it also very large?

Hi lovemino, before sharing the code, here are some traps you can check:

Have you turned on the eval mode?
Have you normalized the motion before feeding them into AutoEncoder of FGD?
What is your FGD of CaMN with our autoencoder checkpoint (convert to axis-angle)? Is it also very large?
Thank you for your reply!

I turned on the eval mode
I calculated FGD before "denorm_out = out_motions * test_dataset.std_pose_axis_angle + test_dataset.mean_pose_axis_angle"
I haven't calculated it on Camn with your autoencoder yet.

Below is the FGD calculation code, which is a replication of the metric evaluation in the test_arbitrary_len function, referenced from CAMN.

Hello,
Following your suggestion, I tested CaMN using your ges_axis_angle_300.bin. I converted the results and dataset of CaMN from Euler Angles to Axis-Angle using the conversion scripts you provided. Additionally, I used the mean and std.npy files and your ges_axis_angle_300.bin, but the resulting FGD is 800.22, which is different from the 1635.44 reported in your paper. This issue has troubled me for a long time. If you could kindly provide your test dataset via email, I would be immensely grateful. I appreciate your work and your response.
Thank you.

Hi lovemino, before sharing the code, here are some traps you can check:

Have you turned on the eval mode?
Have you normalized the motion before feeding them into AutoEncoder of FGD?
What is your FGD of CaMN with our autoencoder checkpoint (convert to axis-angle)? Is it also very large?

Hello,
Following your suggestion, I tested CaMN using your ges_axis_angle_300.bin. I converted the results and dataset of CaMN from Euler Angles to Axis-Angle using the conversion scripts you provided. Additionally, I used the mean and std.npy files and your ges_axis_angle_300.bin, but the resulting FGD is 800.22, which is different from the 1635.44 reported in your paper. This issue has troubled me for a long time. If you could kindly provide your test dataset via email, I would be immensely grateful. I appreciate your work and your response.
Thank you.

Hi liubeibei, here is the link for the processed test set of BEAT. Please comply with the original license and restrictions of the BEAT dataset. Cheers!

I am truly grateful for your willingness to provide the test set; you are a real lifesaver. If possible, could you also provide the relevant code for testing the FGD metric? Following the evaluation metrics written in your train function results in an error in motion_autoencoder because CaMN uses a sliding window to generate multiple latents. According to your method, it only gets one latent with shape (1, 34, 192) for batch size 1, whereas CaMN gets around 83 latents through sliding window, resulting in a shape of (83, 34, 192).
yours:
 latent_out = self.eval_model(outputs[:, :34, :].float())   latent_ori = self.eval_model(motions[:, :34, :].float())
CaMN
`            for j in range(num_divs):
if j == 0:
cat_results = myoutputs[:,jstride:jstride+pose_length, :]#[83, 34, 141]
cat_targets = tar_pose2[:,jstride:jstride+pose_length, :]
else:
cat_results = torch.cat([cat_results, myoutputs[:,jstride:jstride+pose_length, :]], 0)
cat_targets = torch.cat([cat_targets, tar_pose2[:,jstride:jstride+pose_length, :]], 0)
        latent_out = self.eval_model(cat_results.float())
        latent_ori = self.eval_model(cat_targets.float())`

So, whether I follow your method in the train function or use a sliding window like CaMN, I still don't get the correct metric. If you could provide the testing code, I would be incredibly grateful.

Thank you.

Hi liubeibei, here is the link for the processed test set of BEAT. Please comply with the original license and restrictions of the BEAT dataset. Cheers!

I am truly grateful for your willingness to provide the test set; you are a real lifesaver. If possible, could you also provide the relevant code for testing the FGD metric? Following the evaluation metrics written in your train function results in an error in motion_autoencoder because CaMN uses a sliding window to generate multiple latents. According to your method, it only gets one latent with shape (1, 34, 192) for batch size 1, whereas CaMN gets around 83 latents through sliding window, resulting in a shape of (83, 34, 192). yours:  latent_out = self.eval_model(outputs[:, :34, :].float()) latent_ori = self.eval_model(motions[:, :34, :].float())
CaMN:
        for j in range(num_divs):
                if j == 0:
                    cat_results = myoutputs[:,j*stride:j*stride+pose_length, :]#[83, 34, 141]
                    cat_targets = tar_pose2[:,j*stride:j*stride+pose_length, :]
                else:
                    cat_results = torch.cat([cat_results, myoutputs[:,j*stride:j*stride+pose_length, :]], 0)
                    cat_targets = torch.cat([cat_targets, tar_pose2[:,j*stride:j*stride+pose_length, :]], 0)
            latent_out = self.eval_model(cat_results.float())
            latent_ori = self.eval_model(cat_targets.float())

One guess is that according to the data.mdb you provided, the shape of the pose/pose_axis_angle variable in the test_dataset I read is [855, 141], not [256, 34, 141] as you commented.
So, whether I follow your method in the train function or use a sliding window like CaMN, I still don't get the correct metric. If you could provide the testing code, I would be incredibly grateful.  Thank you.

Hello, I have a simple question. Did you use the Beat 2, 4, 6, 8 dataset but with a different processing method, so you retrained the autoencoder? When comparing with CaMN, did you convert the results from the CaMN model to Axis-Angle to calculate the metrics, or did you retrain the entire CaMN model using your processed dataset? Thank you very much! I would be very grateful if you could answer.


:scope pseudo-class isn't supported in IE yet

The :scope pseudo-class isn't supported in IE11 yet, and using it in a querySelector or querySelectorAll call throws a syntax error. We're currently doing that in the component's init function (see here), and because of the syntax error that IE throws, it results in all of our components failing to initialize.
I've found a polyfill that seems to solve the problem in my limited testing, so we might be able to simply add this: http://stackoverflow.com/a/17989803/3137977

The polyfill approach is fine with me, but let's add it as a separate file. Maybe src/polyfills/scope.js. We'll also need to add it to the build in the Rakefile.


codegen statements no longer generating graphql files

Amplify CLI Version
10.8.1
Question
I am upgrading CLI from 6.1.1 to 10.8.1 (it's quite a leap!) and running into issues with with queries.graphql, mutations.graphql and subscriptions.graphql file generation.
Configuration
Amplify CLI 10.8.1
.graphqlconfig.yml
projects:
  acme-project:
    schemaPath: amplify/backend/api/acme-project/build/schema.graphql
    includes:
      - amplify-generated/graphql/**/*.ts
    excludes:
      - ./amplify/**
    extensions:
      amplify:
        codeGenTarget: typescript
        docsFilePath: amplify-generated/graphql
        region: us-west-2
        apiId: abcdefg
        maxDepth: 11
extensions:
  amplify:
    version: 5

Problem
When codeGenTarget: typescript is specified in .graphqlconfig.yml, none of these work to generate .graphql files:

amplify codegen generates only .ts files
amplify codegen statements generates only .ts files

queries.graphql, mutations.graphql and subscriptions.graphql files are not generated/updated in the path specified by docsFilePath or in any other path in the system.
I was able to confirm that with amplify CLI version 6.1.1 running
amplify codegen statements
works as intended. With 6.1.1 all three .graphql files are being generated and put in amplify-generated/graphql alongside mutations.ts, queries.ts and subscriptions.ts files.
However, if I change codeGenTarget to say
codeGenTarget: graphql
queries.graphql, mutations.graphql and subscriptions.graphql files are correctly generated. But then mutations.ts, queries.ts and subscriptions.ts are not generated.
And I need both .graphql and .ts files. That's how greedy I am!
The only way to currently work around this problem for me was creating a script and swapping codeGenTarget values in the .graphqlconfig.yml file on the fly so that it says codeGenTarget: graphql before running amplify codegen statements and then change it back to codeGenTarget: typescript before running amplify codegen again.
That way I had all six files created/updated correctly, but it's a nasty workaround.
Question
Is it an expected behavior in the current CLI version or am I missing something obvious? The documentation was not particularly helpful in this case.

Hi @sebekz 👋 thanks for raising this issue. You mentioned you were using CLI version 6.1.1 but your env info says you're using v10.8.1. Is this intentional or were you referring to a different amplify library?
In any case, this seems like expected behavior to me. The codeGenTarget determines the type of files generated. The graphql option doesn't seem to be an option exposed by the CLI prompt amplify codegen configure and has to be set manually. The workaround mentioned here may be necessary.
I'll confirm with the team and follow up with their response.

Thank you for responding @chrisbonifacio. Let me rephrase what I said to answer your question.

Originally we were using version 6.1.1 of the amplify CLI
Now we are upgrading to the latest amplify version which at this point is 10.8.1
The codegen behavior is different between these two CLI versions with the same configuration specified
Specifically, the older version is capable of generating both typescript and graphql files with codegen target set as typescript
The latest version is no longer capable of generating both typescript and graphql files without hacks

Are you saying that the 6.1.1 behavior was a bug that was ultimately fixed?

Hi @sebekz, Thanks for reporting. This is a known feature-request we're tracking in the codegen repository here. Closing this as a duplicate.


[FEATURE] Add /version endpoint to AppConfig Building Block

Is your feature request related to a problem? Please describe.
Currently, we don't have a way to retrieve the Building Block's version number through API.
Describe the solution you'd like
We need to add a /version endpoint that returns the current version number. OpenAPI specification YAML file needs to be updated with this endpoint. Basically, the building block needs to capture its docker image tag and return that in the response. Response Content-Type can be text/plain. If reading the docker image tag from within the container is not straightforward, the building block can read the version number in the YAML file.
Additional context
Epic: #738

Added configs_version_search method which reads the appconfig yaml file and returns the version number. Added the path to yaml file. http://localhost:5000/app/configs/version returns the version in plain text format.

Hi, Minu, Yes, this looks fine. Please make sure that the response Content-Type is text/plain. Thanks.


fix: change behaviour of envOrNumber to return default if environment is a…

See #363 and #359

Yep, this was along with the other refactoring I did without a pr... Thanks for fixing this!

No problem, a few users got hit hard by the nvidia-api session retry every 0ms :sweat_smile:


Add natural-induction and Fin

Checklist:

[x] Meaningful commit message - please not Update build-constraints.yml

[x] At least 30 minutes have passed since Hackage upload

[x] On your own machine, in a new directory, you have successfully run the following set of commands (replace $package with the name of the package that is submitted, $version is the version of the package you want to get into Stackage):
stack unpack $package-$version
cd $package-$version
stack init --resolver nightly
stack build --resolver nightly --haddock --test --bench --no-run-benchmarks

@strake, a honest question: I'm curious why you reinvent all these packages:

categoryfor http://hackage.haskell.org/package/semigroupoids
, Peano, Fin, natural-induction for http://hackage.haskell.org/package/fin
lenz for http://hackage.haskell.org/package/lens or http://hackage.haskell.org/package/microlens
constraint for http://hackage.haskell.org/package/constraints
... http://hackage.haskell.org/package/hs-functors looks like

and so on. What's wrong with Edwards or others packages?

@phadej
In a word: dependencies.
Consider lens, the most egregious example: when i began using lenses, these were the dependencies: aeson, array, base, base-orphans, bifunctors, bytestring, comonad, constraints, containers, contravariant, distributive, exceptions, filepath, ghc-prim, hashable, mtl, parallel, primitive, profunctors, reflection, scientific, semigroupoids, semigroups, split, tagged, template-haskell, text, transformers, transformers-compat, unordered-containers, utf8-string, vector, void, zlib
This is insane, particularly for a package which one can compat with not depending on it at all. (microlens had not been published yet, and anyhow i have other concerns with it.)
The other examples mostly have the same problem, to a lesser degree.
In some cases, e.g. category, it was merely exploring other design space (e.g. for category, not having the functional dependencies on Functor, which lets one define more general instances but sometimes leads to ambiguities — a trade-off).
A counterexample would be reflection: it depends on only base and template-haskell (since GHC 8.0), so i use it freely.

For the record:

microlens-<IP_ADDRESS> Uploaded at Sun Apr 19 19:49:36 UTC 2015
lenz-<IP_ADDRESS> Uploaded Tue Dec 22 17:34:08 UTC 2015 (8 month later).

fin has only two dependencies in addition to GHC boot libs with GHC-8.2.2: hashable (and therefore text, which is bundled with GHC-8.4). fin also lists 6 other packages solving that problem. Now there are 8. (I really tried to improve situation by making comprehensive and well documented library, I'm sad to see it's not good enough).

http://hackage.haskell.org/package/Peano-0.0.4/docs/Data-Peano.html (uploaded 2012, less instances)
https://hackage.haskell.org/package/peano-<IP_ADDRESS>/docs/Data-Peano.html (your package, 2015, more instances)

I had written and begun privately using lenz and Fin (then called fin) before microlens and fin were published. (I think the lesson for me here is it's better to publish sooner to avert potential wasted effort from others.)
I'm willing to work with you and/or the authors/keepers of these packages to try to reach a consensus, if you think it's worthwhile. I not like ecosystem fragmentation either.

One issue is that the practice of some of your packages to only case-differ from existing package names can be problematic. We've long intended to put a policy in place on hackage to prevent this, and probably will soon (though existing clashes will be grandfathered in). It would be nice if there was a deprecation path for such things -- i.e. deprecate the case differing package to point to the prior, more widely used package (e.g. https://hackage.haskell.org/package/cabal) , and then upload what had been the case differing package under a new name.


malloc doesn't return NULL when run out of heap

When esp-open-rtos malloc runs out of memory malloc and pvPortMalloc both does not return NULL even if there is no more heap memory available. In conclusion I can allocate over 2M of memory before I get into problem. The pointers are indeed increasing and are indeed "writable" (no  error given), and written value retained. Not sure where the writes end up at all?
Want to test that all the indicated memory by xPortGetFreeHeapSize actually was allocable, this is in relation to the reported issue of https://github.com/SuperHouse/esp-open-rtos/issues/75
Giving back memory beyond the "free memory" seems to crash, however.
void* a[60];
int c = 0;

int t = 0;
int z = 16384;
// TODO: keep allocated in free list and deallocate?
while (z > 0) {
    //while (xPortGetFreeHeapSize() < z) z = z/2;  // enable for safe allocs!
    void* p = pvPortMalloc(z);
    //void* p = malloc(z);
    if (!p)
        z = z/2;
    else {
        printf("%u %d %d %d\n", (unsigned int)p, z, t, xPortGetFreeHeapSize());
        t += z;
        // test is writable
        int* x = p;
        *x = 4711;
        if (*x != 4711) printf("--- Write failed! %d\n", *x);
        a[c++] = p;
        if (c == 60) break; // before it crashes!
    }
}
for (int i = 0; i < c; i++) {
    free(a[i]);
    printf("%u %d\n", (unsigned int)a[i], xPortGetFreeHeapSize());
}

(heap)
1073717248 16384 0 7204
1073733640 16384 16384 -9188
1073750032 16384 32768 -25580
1073766424 16384 49152 -41972
1073782816 16384 65536 -58364
1073799208 16384 81920 -74756
1073815600 16384 98304 -91148
1073831992 16384 114688 -107540
1073848384 16384 131072 -123932
1073864776 16384 147456 -140324
1073881168 16384 163840 -156716
1073897560 16384 180224 -173108
...
1075766248 16384 2048000 -2041796
1075782640 16384 2064384 -2058188
1075799032 16384 2080768 -2074580
1075815424 16384 2097152 -2090972
1075831816 16384 2113536 -2107364
Fatal exception (3):
epc1=0x401065f8
epc2=0x00000000
epc3=0x40221bd6
excvaddr=0x4020240c
depc=0x00000000
excsave1=0x3fff61b0

I know the implementation of sbrk in newlib's libnosys does not do any checks against heap size. It's easy enough to write a better one, for instance I've used this on STM32F4 (with my own custom linker script that provides _heap and _eheap for the start and end of the heap):
void *_sbrk(int incr)
{
	extern uint8_t _heap[];
	extern uint8_t _eheap[];
	static uint8_t *heapend=_heap;

	if(heapend+incr>_eheap)
	{
		errno=ENOMEM;
		return (void *)-1;
	}

	uint8_t *newheapstart=heapend;
	heapend+=incr;

	return newheapstart;
}

Sorry for the silence replying on this issue, @yesco. Busy time of year!
Yes, I think a smarter _sbrk is the way to go. The _sbrk we're using is here:
https://github.com/SuperHouse/esp-open-rtos/blob/master/core/newlib_syscalls.c#L16
There even was stack/heap collision code in there to detect this kind of overtflow, but it's been commented out with a TODO (well done me.)
(The "stack" in question that we care about is the "supervisor stack" that's running in _start/main, before any RTOS tasks run. The RTOS tasks take their own stack space from the heap.
So the FreeRTOS-friendly way to detect where "stack/heap collision" happens is one of two ways - either we add heap end symbols like in @DagAgren's version above, which fixes the "supervisor stack" to a fixed size. Or we mark the "supervisor stack high water mark" at the time we start the RTOS scheduler (which is the last function call that the supervisor stack ever makes, it never returns), and then in _sbrk we consider the stack pointer to be either the current stack pointer (if _sbrk was called outside an RTOS task) or the supervisor high water mark (if _sbrk was called within an RTOS task stack.)

I haven't looked at this in detail, so I may be missing something, but since, as you say, the scheduler never returns, and if I remember, there's no way for stack-local variables to be passed into the scheduler, why do we care about the stack at all after that point?  No need to record a high water mark at all, just consider the whole supervisor stack to be free memory after we start the scheduler..
One potential issue with your second approach in cases where people are doing significant work outside of the RTOS scheduler is that it doesn't guarantee any minimum amount of stack space, so it would be possible for somebody to successfully allocate a bunch of memory, and then make a call to another function which uses some more of the stack and in the process overwrites the malloc'd memory with the (now larger than when sbrk checked) stack..
I'm thinking a hybrid of the two: reserve a fixed (compile-time or even run-time configurable) size for the supervisor stack, but once we're in the RTOS, ignore that and consider the supervisor stack to be free memory..

Just clobbering the supervisor stack made me deeply uncomfortable at first thought, but I think you're right that it should work fine.
The hybrid approach sounds good, I like that we can have a reasonably large default stack size but then totally overwrite it once the scheduler is running. Making it runtime modifiable should be a simple implementation - it can just be a global variable that the scheduler sets to zero as it starts.

I consider this bug quite serious.
I have a feeling from the discussion that we're waiting for the ultimately and best solution instead of having a good enough one and improve it later. ;-)
I'm not sure why we just not enabled the check code? It was said it was commented out. Is it faulty/incorrect, dangerous? Any stacks allocated later are allocated from the heap (at least according to some docs I read), so the issue remains in knowing where the heap stops. There is a discussion regarding the supervisor thread that I don't follow completely as I'm new to rtos world, it seems indicated that the
But that contract of malloc not returning null when out of memory is not honored is quite serious and may well be the source of most instabilities observed by people trying to use esp-open-rtos. For example it causes networking to crash without any good explanation as seen in #101. Actually, any rtos app with networking enabled would easily be crashed by just doing enough connections in short time...

There even was stack/heap collision code in there to detect this kind of overtflow,
but it's been commented out with a TODO (well done me.)

Hi yesco,
It's fixed in the 'feature/tests' branch already (I even have unit tests in there to prove it!) I was hoping to merge the tests after I had them a bit more fleshed out - but it's dragging on a bit so perhaps I'll merge what I have now.

OK. I missed that. Great!
On Mar 1, 2016 4:33 PM, "Angus Gratton"<EMAIL_ADDRESS>wrote:

Hi yesco,
It's fixed in the 'feature/tests' branch already (I even have unit tests
in there to prove it!) I was hoping to merge the tests after I had them a
bit more fleshed out - but it's dragging on a bit so perhaps I'll merge
what I have now.
—
Reply to this email directly or view it on GitHub
https://github.com/SuperHouse/esp-open-rtos/issues/76#issuecomment-190632959
.

Sorry this dragged on for so long, the feature/tests branch is still a work in progress so I cherry-picked this one fix into master.


Add neutral things

Add neutral things on the ground:

Plants proto::TYPE_GREEN.
Neutral elements proto::TYPE_BROWN.
Add collider in case needed.

Add jump points / zones


Add MinIO fixture to get full test coverage across S3 unit tests

Right now, our S3 unit tests point to public AWS buckets, but this comes with several limitations like the inability to test writes and the inability to test authorized reads.
Instead of hosting special CI infrastructure that exposes buckets for testing, we should figure out how to spin up an instance of MinIO inside the unit test framework that creates these buckets locally. Then we should develop test coverage for the following

public read, 1MB file, 1GB file, both path/virtual hosted buckets
authorized read, 1MB file, 1GB file, both path/virtual hosted buckets
authorized write, 1MB file, 1GB file (requires resolution of #39 first), both path/virtual hosted buckets
ls of a public prefix (i.e. a bucket with no access/secret keys), both path/virtual hosted buckets
ls of a private prefix (i.e. a bucket that requires access/secret keys), both path/virtual hosted buckets

Adding @rw2 to the assignee of this ticket.
Most of the requested items have already been done; simply need to review the requested test coverage versus the status quo and, if any are missing, contribute them.


Activating light gizmo causes light to move

Bug repro: https://www.babylonjs-playground.com/#9DXCHP
Expected result: The light gizmo should have no effect on the light direction.
Current result: When scene is returned immediately (no export at all), the light gizmo toggle reverses the light direction (direction = -1 * direction).
Comments:
This bug does not occur in playground #KX53VK#27 (left handed).
This bug does not occur in the sandbox (playground #9DXCHP exported using Inspector tool).
The light is initially oriented front to back (0,0,1).

as a note, this bug is not related to handedness. this can be reproduced in a left handed coordinate system as well, since the default gizmo is aligned with the world origin. Any transformed light will be reset to point to local origin once activated.


HMR issues with ESM

This file has a problem with ES module:
https://github.com/webpack/webpack/blob/main/hot/poll.js

As per https://webpack.js.org/api/hot-module-replacement/ it should have conditional switch to import.meta.webpackHot

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us help you by recreating the issue using one of our templates.
Looks like some loader/plugins generate non ESM compatibility code, without more infromation we can't help you, sorry


Dataset during pretrained.

Thanks for great work, I wonder is there other dataset that is not english-based you used during training?

obviously not

Thanks for the reply,Can you tell me the pipeline  if I would like to train the model with Chinese dataset(suppose the data is already labeled).

Thanks for the reply,Can you tell me the pipeline if I would like to train the model with Chinese dataset(suppose the data is already labeled).

I  think just modify the tokenizer and instructions is OK, but my concern is that I can't reproduce result. Did you have this question ?

I think the reason you can not reproduce the result is that the pretrained model you used is only trained on english only], there's another pretrained model trained on multiple language.

But I dont notice that in his paper 😂, I'm also insterest in this model, would you mind leave me your wechat ? I look forward to further communication with you. ❤
my wechat: SupritYoung
email<EMAIL_ADDRESS>

But I dont notice that in his paper joy, I'm also insterest in this model, would you mind leave me your wechat ? I look forward to further communication with you. heart my wechat: SupritYoung email<EMAIL_ADDRESS>
Sure, emm.. just so you know I used to study computer vision, just started to working on NLP.

tk-instruct is finetuned on instructions based on the T5 model which is just in English. However, you can use the one @SupritYoung  posted. There are two variants of mtk which is the 3B and 11B variant. However, 3B would require significant compute requirements. You can also try finetuning using the mt5 base as well. But this would require instruction tuning that will have to be done additionally.