Bouncing too fast & collideWorldBounds makes the sprite stick to the wall

The sprite just stops moving in the direction which has the 3000 velocity after it hits the wall. But keeps moving in the direction which has 500 velocity.
Here you can just clone the repo & run the index.html
https://github.com/taylankasap/phaserjs-bounce-issue
I tested on Chrome, Firefox, IE, Windows & Ubuntu. All same.

This is known as 'tunneling', when a physic body is moving so fast it tunnels into (or sometimes even right through) another body. Arcade Physics has no checks in place to handle this, and no form of CCD, so you'll need to either keep your velocities in check, or move to a full body physics system.


[FEAT] Console log statements spam the console

Problem
I do a lot of plugin development for Obsidian.md, so I frequently open the console. I would like to disable console log messages for this plugin.
Solution
Add a setting that will enable/disable console log statements. Something like Enable debug messages. or Enable log messages

Thank you for your feature request, I will work on it.


Xamarin.UI.Tests doesn't test properly before releasing

Hi All.
I'm writing this compliant especially for them who are responsible for releasing cycle of Xamarin UI tests.
I'm starting with words that we really like this framework. It's friendly, easy and enough for our needs.
But previous several releases are total setbacks.

3.2.7 - 3.2.9 impossible to run real iOS device (problem with clear cache. there is some topic on GitHub)
4.0.0 - impossible to run start app. Local (Xamarin) problem appears from the scratch after calling ConfigureApp.Android

System.TypeInitializationException : The type initializer for 'Xamarin.UITest.ConfigureApp' threw an exception.
----> System.IO.FileNotFoundException : Could not load file or assembly 'Xamarin.UITest.Shared, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.

4.0.1 - can't work with Repl() . When the console is opened it writes that "This is not the real repl - this is just a placeholder".
Screenshot() doesn't work, always return null.
maybe else some functions don't work I couldn't check them yet.

Conclusion:

Consider testing common functions properly before realises
If you don't need this framework anymore than consider making it for open-source in order we solve problems which you solve not in a hurry
This framework can be awesome if you elevate his priority in your daily tasks

Hi @Viktor027 , Thank you for reaching out and for the fair suggestion!
We will take your notes into account!
Closing.


updated source with license/copyright headers

Coverage remained the same at 34.708% when pulling 43dc3cd6caa5ab8fa35fef47042f45ab2564d6d8 on license-copyright into 704b2390f93214e1a8fb9e2a2b1abd354dfe89be on master.


Actual running pod counts are different from the HPA-allocated

I have configured HPA for the Solrcloud pods when I see the HPA status from the Kubernetes cluster - it returns 10 pods are allocated, but from the nodes page in the Solrcloud admin UI - it shows more than the HPA allocated - (58) pods are running.
Versions:
SolrCloud 9.3
solr-operator 0.8.0
here are some logs from the cluster
solrcloud_annotations.txt
solrcloud_operator_logs.txt
solrcloud_output.txt

I think there are two issues here: The Solr Operator is deleting PVCs of pods before the pods are being deleted. That is obviously a problem, and likely causing the BalanceReplicas command to fail.
There is also a bug that results in the BalanceReplicas command continuously going even though it should be paused to start scaling down. The PVC issue mentioned above causes the BalanceReplicas command to fail, and thus this bug is seen.


build(deps): Upgrade and handle breaking changes in dependencies

In order to support upgrading dependencies in forklift-ui and benefit from some bug fixes in dependencies, this PR upgrades all the things we should try to keep fresh: peer dependencies consumers also depend on and will keep up to date (patternfly, yup, react, react-dom), and core code quality and docs stuff (typescript, semantic-release, storybook, jest, testing-library, eslint, rollup, babel, commitizen, prettier)
This PR replaces/adopts a lot of the changes in https://github.com/konveyor/lib-ui/pull/51, so that PR should be rebased or rewritten after this change so that it can focus on the yarn-to-npm change.
Notable upgrades include:

TypeScript 3 to 4 (required the installation of tslib to handle some syntax)
Storybook 6.0 to 6.2 (removes the requirement of a direct dependency on webpack, so this PR removes webpack),
yup from 0.29.3 to 0.32.8 (propagates some breaking changes to consumers, see below)
PatternFly from 2020.10 to 2021.04 (https://www.patternfly.org/v4/developer-resources/release-notes)
React from 16.13 to 17.0 (we're not affected by any of their breaking changes)
eslint from 7.6 to 7.24
prettier 2.0 to 2.2
rollup 2.23 to 2.45

BREAKING CHANGE: The yup upgrade bundles its own TS types, removing the need to depend on @types/yup, and some of the type changes are breaking. yup.Schema has been replaced by yup.SchemaOf, and it correctly infers array types now so we were able to remove the hacky MaybeArraySchema type from useFormState. There have also been some behavior changes in some of the schema methods, particularly the .defined() and .required() methods. See those changes here: https://github.com/jquense/yup/blob/master/CHANGELOG.md (the items between 0.29.3 and 0.32.8). We felt it was a good idea to upgrade even though this is a little disruptive, because the new built-in TS types are less finicky and more helpful than the community-provided ones were.

LGTM! Thanks Mike.


Show similar documents

Each line should show a code bubble showing the number of similar lines in other documents.
Example: I am looking at:
### what is it
- a compiler

it shows a bubble saying "3 other compilers"
when clicking on the bubble, I see the other documents describing compilers as a list of references

might introduce a lot of noise that isn't particularly relevant
"compiler" would be a link right now, which contains a list of all compilers


Fixes #5545

Refactor out the SUBSCRIPTION tests for some better coverage and clarity. Ensure that a retention policy error will be returned if you attempt to drop a subscription on a policy that doesn't exist.
Removed meta.ErrRetentionPolicyNotFound since everything else uses influxdb. ErrRetentionPolicyNotFound()

Tested and it works :+1:

minor nit (that I'm not 100% sure on).  +1

@corylanou golint seems fine with it, though there were some pre-existing lint issues in files I touched in this PR, so I've tidied those up.

+1

:+1:


Add metadata and richlink (fixes #689)

https://richpreview.com/?url=https%3A%2F%2Fqualitycheck.encompass.bayesimpact.org%2F

Nice! This is a convenient tool. Note that we'll have to change this to Beacon Labs.. Might as well do that now for the title!

This will be changed when we review the code for Bayes Impact mentions


Timeline .load() .init()

I've checked the code in frame.js and I've just noticed that you don't use the .load function from the modules just the .init.
I'll use .load before .init to load all the resources and after all call .init to initialize everything related to the module.
I think it shouldn't be called any .init code if the rest of the .load effects are still peding because maybe a module could depends on another one.
The .load callback should be automated by the FRAME function instead of leaving it to the user, and include a load funciont on Timeline with a callback that could be used as global status of the loading (creating a progress bar and so on).
I've modified something similar to the following:
Timeline: function ()
{
  load: function (callback) {
    this.elementsToLoad=elements.length;
    for ( var i = 0, l = elements.length; i < l; i ++ ) {
        element[ i ].load(loadedCallback);
    }

  loadedCallBack: function (element) {
     this.elementsToLoad--;
     userLoadedElementCallback(element);
     if (this.elementsToLoad==0)
        this.init();
  }
}

What do you think?

Closing this because since 6449949b818bd36db2b151872e2f6f985ebcc15c the code base is now different. Open a new PR is needed.


iOS Play out of earpiece only

Checklist

[X] I made sure that the issue I am raising doesn't already exist

Use case / Problem
Using AudioContextIOS, I can't seem to find a combinations of a category and options to play audio out of the earpiece speaker only, i.e. On a call. I can't currently test on Android but it seems this is possible baed on https://github.com/bluefireteam/audioplayers/issues/1401 from @dariocavada.
Is this at all possible on   audioplayers: ^4.1.0? It was possible on  audioplayers: ^0.20.1
Thanks!
Proposal / Solution
AudioContextIOS that allows audio to played from just the earpiece speaker.
Example Code
No response
Affected platforms
iOS
Platform details
No response
Audio Files/URLs/Sources
No response
Mockups
No response
Related issues / more information
No response
Working on PR
no way

I cannot test, as I don't have a real device but this should do the trick:
  AudioPlayer.global.setGlobalAudioContext(
    AudioContext(
      iOS: AudioContextIOS(
        category: AVAudioSessionCategory.playAndRecord,
        options: const [
          AVAudioSessionOptions.mixWithOthers,
        ],
      ),
    ),
  );

Maybe you can also tell me, if there's a difference to:

AudioPlayer.global.setGlobalAudioContext(
AudioContext(
iOS: AudioContextIOS(
category: AVAudioSessionCategory.playback,
options: const [
AVAudioSessionOptions.mixWithOthers,
],
),
),
);

@RedHappyLlama did you succeed?

@Gustl22, yes, thanks very much for your help! Sorry for the late reply!
The first bit of code worked, the second did not work.

@RedHappyLlama Thank you, that helped a lot ;D

I cannot test, as I don't have a real device but this should do the trick:
  AudioPlayer.global.setGlobalAudioContext(
    AudioContext(
      iOS: AudioContextIOS(
        category: AVAudioSessionCategory.playAndRecord,
        options: const [
          AVAudioSessionOptions.mixWithOthers,
        ],
      ),
    ),
  );

Maybe you can also tell me, if there's a difference to:
  AudioPlayer.global.setGlobalAudioContext(
    AudioContext(
      iOS: AudioContextIOS(
        category: AVAudioSessionCategory.playback,
        options: const [
          AVAudioSessionOptions.mixWithOthers,
        ],
      ),
    ),
  );

There could also be a different behavior, if headphones are connected...

This solution works for me thanks.!


Feature/Show progress bar on each waypoint similar to shown at map

When a camper is on a waypoint of any block (e.g. HTML and CSS) he needs to know how far he is from completing that block. Checkout the proposal given below (Not a good UX though but clears the point):

I personally am all for further gamification, but someone will need to come up with a less obtrusive UI for it than that.

I agree with this sentiment but, as @ltegman says, it needs to be much less obtrusive.  We have been talking about moving away from the modals, even, for the end of challenge notification.

What about showing percentage number in the right bottom corner instead of using status bar on the top?

@SaintPeter @ltegman  The modal approach can prove to be useful if we are moving towards gamification. At the end of the challenge we can display stats and progression.

@hallaathrad, what do you think of this?

The gamification idea has been of great appeal to a lot of people (including myself) but I'm not sure I've ever heard @quincylarson speak for or against it.
That said, we could consider better options than the modal window (specially considering it's going away soonish, if I've understood correctly).
A progress bar would be shown for the challenges position in the block of content (e.g. HTML and CSS... Or JavaScript). And then at the end of each of these blocks an additional progress indicator for it's part of the stack/certification.
However, as an argument against progress bars, and similar progress indicators: people should not be misled into thinking this is a race or competition they could speed through. Coding is more like learning to paint than building a lego toy. Constance ans repetition rather than a linear process until some imaginary "completion".


Mybet command

closes #15

Add one of these options: https://deno.land/manual/tools/formatter#ignoring-code for succesful checks.

Add one of these options: https://deno.land/manual/tools/formatter#ignoring-code for successful checks.

@alexsc23 Well it is nothing wrong with the changed files. The file that causes the crash is not manually created, it is the bundled JavaScript. For reference #20

Fixed problem with deno fmt & lint.
But now fails at tests while connecting to mongodb.


Azure ADE : S2D File servers with Azure ADE

Hi team,
Just a points related to suppport matrix for Azure ADe.the documentation states following :
Enable encryption on VMs that are configured with Windows Storage Spaces beginning in Windows Server 2016.
Now if we talk about to achieve HA in file services we have to use S2D (Storage spaces) which includes CSV volumes. The encryption doesnot work/support with CSV volumes and hence the statement of support doesnot fully justify and can cause confusion as happened with us while defining the solution .I donot see any statement about support CSV volumes either.

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

ID: 7c60e59e-b306-3b47-dcad-56a3c03de428
Version Independent ID: 95ea9500-3d2e-6bef-a743-86e660e1ca86
Content: Overview - Azure Disk Encryption for IaaS VMs
Content Source: articles/security/azure-security-disk-encryption-overview.md
Service: security
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

Thanks for the feedback! We are currently investigating and will update you shortly.

Any further news on this one plz?

Any update on this topic ?
So this means that in Azure, the SQL HA scenario called "Always On with Failover cluster instances and Shared Storage " where S2D is used, doesn't support Azure Disk Encryption V1 or V2 ? What about Windiws Server 2019 ?

The article (which has since been moved to https://docs.microsoft.com/en-us/azure/virtual-machines/windows/disk-encryption-windows) lists "Encrypting VMs configured with Storage Spaces Direct (S2D), or Windows Server versions before 2016 configured with Windows Storage Spaces" as an unsupported scenario.  The supported VMs and images are listed here: https://docs.microsoft.com/en-us/azure/virtual-machines/windows/disk-encryption-overview#supported-vms-and-operating-systems; anything not mentioned is unsupported.
Thank you! #please-close


Fix UFL API change

Marked as draft until I confirm that this fixes Firedrake CI (https://github.com/firedrakeproject/firedrake/pull/3236).

I see that Firedrake PR pass with test so I'll merge. Thank you so much @connorjward and @pbrubeck !


[#noissue] Cleanup

Codecov Report

Merging #6985 into master will decrease coverage by 0.00%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master    #6985      +/-   ##
==========================================
- Coverage   39.03%   39.03%   -0.01%
==========================================
  Files        3229     3229
  Lines       94294    94284      -10
  Branches    11854    11854
==========================================
- Hits        36809    36801       -8
+ Misses      54529    54527       -2
  Partials     2956     2956

Impacted Files
Coverage Δ

...entExecuteMethodWithHttpUriRequestInterceptor.java
0.00% <0.00%> (ø)

...agent/plugin/proxy/apache/ApacheRequestParser.java
72.09% <100.00%> (+0.35%)
:arrow_up:

...point/agent/plugin/proxy/app/AppRequestParser.java
37.50% <100.00%> (-0.97%)
:arrow_down:

...t/agent/plugin/proxy/nginx/NginxRequestParser.java
47.61% <100.00%> (ø)

.../bootstrap/plugin/proxy/ProxyHttpHeaderParser.java
80.21% <100.00%> (-1.04%)
:arrow_down:

...p/pinpoint/rpc/stream/StreamChannelRepository.java
75.00% <0.00%> (-10.00%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
66.66% <0.00%> (-3.34%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.22% <0.00%> (-1.77%)
:arrow_down:

...m/navercorp/pinpoint/profiler/AgentInfoSender.java
84.15% <0.00%> (-1.00%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.40% <0.00%> (+0.32%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


fix rspec > 3 compatiblity

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


[Login] After logging in from Login.gov, check if the user's domain is allowed.

See the "/domains/allowed" on the backend

This was implemented into the Complete Your User Profile form.


Sentry.io for all teams

Create user guidance and runbooks for users to have access to Sentry.io.
This service is mostly self service so there is little guidance we would need to provide but we should discuss and confirm this. This will help define what documentation if any that we need.

Needs refinement - My initial views from conversations with David Read - https://mojdt.slack.com/archives/CPVD6398C/p1607693207014800

Done


Migrate to new 'config' concept

When building Swish-Web I moved to a new 'config' concept where Swish-Base doesn't need to know all the config available for each app.
The change was made retaining backward compatibility, but it would be good for all other apps to use this method.

Use the configManager.parseEnvironment method to tell the ConfigManager which environment variables to pull, and where to put them in the config
Use the configManager.getConfig method to retrieve the current state of the config - so the config manager doesn't need to know the required state too early

Swish 2.0 adds a getValue("key") method to the ConfigManager.
We should also move towards using that to retrieve config needed rather than using getConfig() (or the even older configManager.config property) to get the entire object.

The default config should also be set by the host app.
Currently it just contains default font and ligature settings, which are only used by the legacy GUI app.
Either allow the host to pass defaults, or remove the 'default' concept.


namespace: ga4gh or vrs?

My colleague Ian Fore raised a question about the CURIE namespace ga4gh belonging to other GA4GH workstreams. Does a vrs prefix namespace narrow the scope in the identifier here?
#https://github.com/ga4gh/vr-spec/blob/1598814955baeda62fe484b89bddaf70950cfd74/schema/ga4gh.yaml#L13

@annalu12 , @ianfore
We debated identifiers until we were all bruised an sore :-) in https://github.com/ga4gh/vr-spec/issues/32.
One of the considerations was whether the identifiers should be VRS-specific or GA4GH-wide. We decided that we wanted a GA4GH-wide standard to encourage interoperability within GA4GH, and, conversely, that we didn't want to bake GA4GH group names into the namespace. Also, some objects, like sequences now (and perhaps genes, etc later) transcend groups and need global scope.
So, we ended up proposing a GA4GH-wide identifier syntax to the Steering Committee and that was approved last September. slides.
So, that's a long way of saying that this decision is made and closed at this point.


data format error

In file "kitti_util.py",
Function:
def load_velo_scan(velo_filename, n_vec):
scan = np.fromfile(velo_filename, dtype=np.float64)
scan = scan.reshape((-1, n_vec))
return scan
"np.float64" is wrong, should be "np.float32"

Thank you for finding that bug.
I change the default datatype to "np.float32", and provide option "--dtype64" to support "np.float64" data.


Feature Request: Publish on docker hub

Would be great to publish decK to docker hub eg. for job definitions in k8s or smth

This is already available: https://hub.docker.com/r/hbagdi/deck.


Update login page with new design

Need to rewrite login page styling as per new design.
Design
Screenshot pending
[LINK TO FIGMA DESIGN] pending

When creating a new ticket, please adhere to one of the issue templates we have


[#3561] 1.7.0-RC1 release

(1.7.0-SNAPSHOT -> 1.7.0-RC1)

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3575      +/-   ##
==========================================
+ Coverage   40.07%   40.09%   +0.02%
==========================================
  Files        2186     2186
  Lines       71519    71519
  Branches     9831     9831
==========================================
+ Hits        28663    28679      +16
+ Misses      40109    40100       -9
+ Partials     2747     2740       -7

Impacted Files
Coverage Δ

...point/rpc/client/DefaultPinpointClientHandler.java
72.53% <0%> (+0.92%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.41% <0%> (+1.04%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
66.42% <0%> (+1.42%)
:arrow_up:

.../com/navercorp/pinpoint/profiler/context/Span.java
50% <0%> (+1.72%)
:arrow_up:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
83.58% <0%> (+2.98%)
:arrow_up:

...tor/metric/datasource/DefaultDataSourceMetric.java
78.78% <0%> (+3.03%)
:arrow_up:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
74.03% <0%> (+3.84%)
:arrow_up:

Continue to review full report at Codecov.

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


How do I disable automatic focus widget.

Press the up,down, left and right keys on the keyboard, will cause the automatic focus widget.
How do I disable this feature.

It can't be disabled!
https://docs.flutter.dev/cookbook/forms/focus
You'd have to create new widgets without the Focus widget. Why is it a problem?

I only want disabled button focus.
I set autofocus is false, but is invalid.
ex:
              TextButton(
                **autofocus: false,**
                child: const Text('Text Button'),
                onPressed: disabled
                    ? null
                    : () {
                        print('pressed text button');
                      },
              ),

You can try create a focus node with canRequestFocus: false.
final focusNode = FocusNode(canRequestFocus: false);

return TextButton(
  focusNode: focusNode,
);

Also, if the button is disabled (onPressed is null), the widget will be ignored in the focus scope
https://api.flutter.dev/flutter/widgets/FocusNode/canRequestFocus.html


SSIMULACRA 2

Major new version of the SSIMULACRA metric.
Changes compared to the original version:

works in XYB color space
uses 1-norm and 4-norm (instead of 1-norm and max-norm-after-downscaling)
penalizes both smoothing and ringing artifacts (instead of only penalizing ringing but not smoothing)
removed specific blockiness detection

The score weights were obtained by running Nelder-Mead simplex search, optimizing to minimize MSE and maximize Kendall and Pearson correlation for training data consisting of 17611 subjective quality scores, validated on separate validation data consisting of 4292 scores.
Results for the full dataset (almost 22k subjective scores):

Metric
Kendall correlation
Spearman
Pearson

PSNR
0.34677
0.49926
0.48178

SSIM
0.41612
0.58939
0.52543

SSIMULACRA
-0.5214
-0.7124
-0.6891

VMAF
0.61720
0.81549
0.77781

Butteraugli 2-norm
-0.6572
-0.8461
-0.8103

SSIMULACRA 2
0.74185
0.91695
0.90244

Results for just the validation set (4292 subjective scores):

Metric
Kendall correlation
Spearman
Pearson

SNR
0.35089
0.50092
0.50157

SSIM
0.45002
0.62777
0.55710

SSIMULACRA
-0.5881
-0.7845
-0.7808

VMAF
0.58799
0.78798
0.75031

Butteraugli 2-norm
-0.6226
-0.8127
-0.7834

SSIMULACRA 2
0.72163
0.90354
0.89504

Computing the mean absolute error between opinion scores (on a scale of 0 to 100) and SSIMULACRA 2 results, on the full set the MAE is 4.274 and on the validation set the MAE is 4.470.

Added a separate main file, added some more info to the help screen, and made it work for images with alpha too.

@jonsneyers I see that you are using coefficients correlations to evaluate metrics. You might be interested to learn that there is a new coefficient correlation in town, with very interesting properties and therefore maybe that this new meta-metric could help you compare and improve compressions metrics? https://news.ycombinator.com/item?id=29687613

Thanks for the pointer! Sounds interesting indeed.
Note however that most metrics require a nonlinear mapping to be used as MOS predictors, so in general, rank correlation (like Kendall's tau) is more useful than Pearson correlation to compare the performance of various metrics. But I have to read that paper...

I'm not an expert but I found this visualization quite impressive https://twitter.com/adad8m/status/1474754752193830912?s=21


[LIVY-729] Fix livy should not recover the killed session

What changes were proposed in this pull request?
Follows are steps to reproduce the problem:

Set livy.server.recovery.mode=recovery, and create an interactive session: session0 in yarn-cluster
kill the yarn application of the session
restart livy
livy try to recover session0, but application has been killed and driver does not exist, so client can not connect to driver, and exception was thrown as the image.
If the ip:port of the driver was reused by session1, client of session0 will try to connect to driver of session1, then driver will throw exception: Unexpected client ID.
Both the exception threw by livy and driver will confuse the user, and recover a lot of killed sessions will delay the recover of alive session.

How to fix:
When session was killed or dead, remove the session from the store
How was this patch tested?
Existed IT and UT.

@jerryshao @yiheng Could you help to review ?

Codecov Report

Merging #266 into master will increase coverage by 0.09%.
The diff coverage is 76.47%.

@@             Coverage Diff              @@
##             master     #266      +/-   ##
============================================
+ Coverage     68.09%   68.18%   +0.09%
- Complexity      939      945       +6
============================================
  Files           102      102
  Lines          5883     5897      +14
  Branches        893      896       +3
============================================
+ Hits           4006     4021      +15
  Misses         1302     1302
+ Partials        575      574       -1

Impacted Files
Coverage Δ
Complexity Δ

...che/livy/server/recovery/ZooKeeperStateStore.scala
77.27% <0%> (-1.8%)
17 <0> (ø)

...e/livy/server/interactive/InteractiveSession.scala
70% <100%> (+0.63%)
51 <0> (+5)
:arrow_up:

...he/livy/server/recovery/FileSystemStateStore.scala
62.5% <66.66%> (-0.55%)
11 <1> (ø)

...la/org/apache/livy/server/batch/BatchSession.scala
86.73% <66.66%> (+0.56%)
14 <0> (ø)
:arrow_down:

...ain/java/org/apache/livy/rsc/driver/RSCDriver.java
80.75% <0%> (+0.83%)
46% <0%> (+1%)
:arrow_up:

...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala
55.88% <0%> (+2.94%)
7% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

+1


Playback problem of ugoira converted to a video file with an uneven value as width/height.

Prerequisites

[x] Did you read FAQ section in readme.md?
[x] Did you test with the latest releases or commit ?
[x] Did you search for existing issues in Issues?

Description
Trying to play video files converted from ugoiras that have an uneven value as height or width (ex. width:600 height:335) results in graphical artifacts during playback.
And yes, I realize that this is not a problem with pixivutil per se, but there are ways to fix the uneven height/width values during ugoira-to-video conversion.
Steps to Reproduce

Download and convert ugoira image id 92936913 (NSFW) to a video file.
Play it through a video player (MPC-HC x64 in my case).

Expected behavior:
No graphical artifacts during playback.
Actual behavior:
Has graphical artifacts during playback.
Log file:
Log file not related to problem.
Versions
Latest
Possible fix to the problem
Inserting -vf pad='width=ceil(iw/2)*2:height=ceil(ih/2)*2' to ffmpegParam in config file.
This fixes the playback problem of video files that have an uneven value as height or width by padding the uneven height or width value with one line of black pixels.
Solution copied from https://stackoverflow.com/questions/20847674/ffmpeg-libx264-height-not-divisible-by-2
Basically this takes the width and height of the input file and divides them by two, rounds them up to the closest integer using ffmpeg "ceil" function and then multiplies by two. Will do nothing if height and width are already an even value.

Try my personal configuration, I downloaded your ugoira file and works like a charm
[FFmpeg]
ffmpeg = ffmpeg.exe
ffmpegCodec = libvpx-vp9
ffmpegExt = webm
ffmpegParam = -row-mt 1 -deadline good -crf 19 -vsync 2 -r 999 -pix_fmt yuv444p
webpCodec = libwebp
webpParam = -row-mt 1 -lossless 0 -q:v 90 -loop 0 -vsync 2 -r 999
gifParam = -filter_complex "[0:v]split[a][b];[a]palettegen=stats_mode=diff[p];[b][p]paletteuse=dither=bayer:bayer_scale=5:diff_mode=rectangle"
apngParam = -vf "setpts=PTS-STARTPTS,hqdn3d=1.5:1.5:6:6" -plays 0
verboseOutput = True

[Ugoira]
writeUgoiraInfo = False
createUgoira = True
deleteZipFile = True
createGif = False
createApng = False
deleteUgoira = True
createWebm = True
createWebp = False

Thanks for replying and sorry for late reply.
Looks like the culprit was the pixel formatting (-pix_fmt) set to yuv420p. Worked exactly as intended when changed to yuv444p though I does increase the file size quite a bit. I wonder if there is a way to force 444 formatting only when there are uneven values present in dimensions or if there are better ways of sorting this out without increasing filze sizes that much. Anyway, thanks for the help!


better install instructions

the install instructions are ambiguous if they want you to extract the contents of custom server stuff to the Titanfall 2 install folder, or id you should put the contents of the folder in the Titanfall 2 folder.
It's the latter, but of course I picked the wrong one the first time around. The instructions should be updated to clarify this,

Im still confused, I extracted all the zip files to my Titanfall 2 folder and updated the game through origin. when I launched nothing changed, am I missing something?

Im still confused, I extracted all the zip files to my Titanfall 2 folder and updated the game through origin. when I launched nothing changed, am I missing something?

you're better off asking at https://northstar.tf/discord

https://rentry.org/northstar-dl
i've made this guide on how to install it, if there anything you want to change/remove/add just let me know.

How To Install Northstar Guide
i've made this guide on how to install it, if there anything you want to change/remove/add just let me know.

updated the link

If we want to contribute to the wiki, there could be a docs folder in the Northstar repo which has an action to push markdown pages to the wiki.

How To Install Northstar Guide
i've made this guide on how to install it, if there anything you want to change/remove/add just let me know.

I would also add to the noted section that you cannot host Frontier Defense on the private servers. I had a couple friends download the source files and were a bit disappointed to see that they couldn't play that game mode.

the guide isnt about how to host, its how to install the client
On Fri, Dec 24, 2021 at 8:35 PM Christian @.***> wrote:

How To Install Northstar Guide https://rentry.org/northstar-guide
i've made this guide on how to install it, if there anything you want to
change/remove/add just let me know.
I would also add to the noted section that you cannot host Frontier
Defense on the private servers. I had a couple friends download the
source files and were a bit disappointed to see that they couldn't play
that game mode.
—
Reply to this email directly, view it on GitHub
https://github.com/R2Northstar/Northstar/issues/2#issuecomment-1000906362,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVDOXQPZYCP4VZ2RJ33S7FDUSSVPLANCNFSM5KWBSUQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Since this issue was opened we now also have a more proper wiki.
This also includes reworked install instructions which can be found here: https://r2northstar.gitbook.io/r2northstar-wiki/installing-northstar/basic-setup
@SIGSTACKFAULT (and everyone else in this thread) what's your impression? Are there things we should still change before closing this issue?

Closing this for now, feel free to contribute to the wiki if you think anything's missing.


crypto/x509: NullBytes and NullRawValue are incorrect

The current NullBytes and NullRawValue are as follows [] and {0 5 false [] []} respectively.
According to a the definition of RawValue this is incorrect
// A RawValue represents an undecoded ASN.1 object.\\
type RawValue struct {
	Class, Tag int
	IsCompound bool
	Bytes      []byte
	FullBytes  []byte // includes the tag and length
}

the values should instead be [5 0] and {0 5 false [] [5 0]} respectively.
This is currently causing parsing of rsaencrypted certs to fail

This is an issue with asn1 not with golang


hyper-v

[Enter feedback here]

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

ID: 1df65b2d-24ba-71b0-0648-14c21f5b1b26
Version Independent ID: a89c84df-6ef7-66a9-eec1-35ba94afdb1b
Content: Enable Hyper-V on Windows 10
Content Source: virtualization/hyper-v-on-windows/quick-start/enable-hyper-v.md
Product: windows-10-hyperv
GitHub Login: @scooley
Microsoft Alias: scooley

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


KEDA Kafka Scaler : Not able to set activationLagThreshold to '0' value (which is default value supported by KEDA)

Report
KEDA is supporting 0 as the default value for activationLagThreshold in kafka scaler. But if we set this field explicitly with the 0 value, KEDA is throwing below error:
"error":"error parsing kafka metadata: "activationLagThreshold" must be positive number"
A default should always behave the same whether you set it explicitly or leave it, if optional.

Expected Behavior
A default should always behave the same whether you set it explicitly or leave it, if optional.
So even if we set activationLagThreshold as 0, KEDA should not throw any error.
Actual Behavior
KEDA is throwing below error
"error":"error parsing kafka metadata: "activationLagThreshold" must be positive number"
Steps to Reproduce the Problem

Create a job/pod spec with kafka scaler
Set activationLagThreshold value to 0
Try to deploy the job/pod
Check keda operator logs

Logs from KEDA operator
example

KEDA Version
2.9.2
Kubernetes Version
1.26
Platform
Microsoft Azure
Scaler Details
Kafka Scaler
Anything else?
No response

This makes sense totally, are you willing to fix it?

@JorTurFer Yes, I would be glad to contribute. I'll raise a PR shortly.

@JorTurFer Yes, I would be glad to contribute. I'll raise a PR shortly. Can I get write access?

Nice! I assign this issue to you :)
Why do you need write access? For the PR, you should fork this repo and do the changes in your own fork (where you have write access) and the, open the PR against this repo

Thanks @JorTurFer
Raised the PR: https://github.com/kedacore/keda/pull/4143

I don't think that we do any other hotfix release before March (we have released v2.9.2 this week, and I'm releasing v2.8.2 right now), but for sure in the next minor release (v2.10 in March) will be included

Hi @JorTurFer,
Can you please share the exact date when we'll be releasing v2.10?
Thanks,

Hi @JorTurFer,
Can you please share the exact date when we'll be releasing v2.10?
Thanks,

We aim to do on 9th


Add a way to propagate test failure in aftereach block to the main UI

This basically allows recordIterationComplete to be called multiple times and each time it would write out a new test execution report. a little bit more IO but it shouldn't be too bad.

I'm concerned with the lack of tests for this.
For example:

Does this alway write out the report on failure/not on failure?
Does this work with multiple after blocks? (one throws, all throw, vs. none throw.)
Does this work with zero-after blocks? (one throws, all throw, vs. none throw.)
etc.

@cmeiklejohn  lmk what you think

Can you resolve the conflicts with the CHANGELOG and the build number?

@jiujiu1123 one thing we need to do here is make this option configurable via:

the annotation
a system property.

Can you please make those changes?  If you need guidance, look at how suppress_combinations is also used in both a.) the annotation (which takes first precedence) and then the system property.  It should default to on.

@jiujiu1123 disregard, I realized this is only reporting on the failure not controlling pass/fail, so you don't need the property/annotation.


add Toolhouse cookbook

Description
This PR adds a cookbook that demonstrates how to use Toolhouse with LlamaIndex on a workflow for sales prospecting. It introduces no new LlamaIndex dependencies. The notebook depends on the Toolhouse package.
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No (No new package)

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[ ] Yes
[x] No (No new version bump)

Type of Change
Please delete options that are not relevant.

[x] This change requires a documentation update

How Has This Been Tested?
Your pull-request will likely not be merged unless it is covered by some form of impactful unit testing.

[ ] I added new unit tests to cover this change
[ ] I believe this change is already covered by existing unit tests
[x] Added new notebook (that tests end-to-end). Note: the notebook requires user-specific API keys for Toolhouse and Groq. It can otherwise be executed as-is end-to-end.

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

@logan-markewich I made the requested changes and I added your suggestions regarding event logging.


RSI is not working as it is on TradingView

Expected Behavior
I would expect RSI on TradingView and in Lean, with absolutely the same parameters and candle prices, to be the same.
Here is a screenshot of TradingView RSI (period: 10, Resolution: Hour, MovingAverageType.Simple, BTCUSDT) (=~23)

The value of such an indicator at such point of time at such market (with such candles history) is ~23.
Actual Behavior
I have downloaded and verified that the data fully matches the data on tradingview.
Then I set-up the same indicator: RSI (period: 10, Resolution: Hour, MovingAverageType.Simple, BTCUSDT)
And it produces me this value: 38.47
(The log is: 20220723 16:09:54.208 TRACE:: Log: 30.06.2022 20:00:00 RSI: (38.47717))
Potential Solution
I see a lot of complaints about RSI in various issues. I think that it needs to be re-written. If we don't come to the solution in this issue, I could write my own version and submit it to you guys. But for now, I'd like to ask you if you know what the problem is, maybe you can help.
Reproducing the Problem
Here is the code that fully reproduces it:
https://github.com/borseno/QuantConnect.Lean.RSI.Issue
I have implemnted an algorithm that initializes RSI in the way I described above, and logged its value at a particular point of time. The result is 38, but it is expected that it is 23.
System Information
Windows 10, unactivated
Intel Core i5
Geforce 1080
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Hi @borseno ,
TradingView's RSI uses an exponential moving average in its calculations (I couldn't find a way to set another moving average) while you have selected MovingAverageType.Simple in LEAN. Please try using MovingAverageType.Exponential instead.
Could you please provide the RSI values from TradingView? All indicators in LEAN are verified against indicators of third-party software, so we can plug your data into the unit test and verify it.

Hi @AlexCatarino
I have explicitly selected SMA type of moving average on trading view.
LEAN Engine: Binance, period: 10, SMA, BTCUSDT, 30.06.2022 20:00:00 UTC = 38.47
TradingView: Binance, period: 10, SMA, BTCUSDT, 30.06.2022 20:00:00 UTC = 25.38

In order to select SMA in Lean, you should click on the gear button on the line where the data about indicator settings is written (the line where its written RSI 10 SMA)

Hi @borseno ,
Could you please provide the RSI values from TradingView? All indicators in LEAN are verified against indicators of third-party software, so we can plug your data into the unit test and verify it.

Hi @borseno ,
Could you please provide the RSI values from TradingView? All indicators in LEAN are verified against indicators of third-party software, so we can plug your data into the unit test and verify it.

I wrote them above.
LEAN Engine: Binance, period: 10, SMA, BTCUSDT, 30.06.2022 20:00:00 UTC = 38.47
TradingView: Binance, period: 10, SMA, BTCUSDT, 30.06.2022 20:00:00 UTC = 25.38

That message means that TradingView indicator gives 25.38 value for the Binance exchange, RSI period: 10, moving average type: SMA, symbol: BTCUSDT, the date: 30.06.2022 the time: 20:00:00 UTC

Where as the same setup in LEAN produces value of RSI to be 38.47.

If that doesnt make sense, please specify what other data do you need ?

Hey @borseno
I think we have a misunderstanding.
RSI(period, MovingAverageType.Simple) != SMA(RSI)
The argument MovingAverageType specifies how the RSI values are computed. TradingView, on the other hand, always uses MovingAverageType.Wilders which is also the default in LEAN. The SMA in TradingView means something else, they just apply the SMA on the RSI values. Does that make sense?
If you want to reproduce this in LEAN you can do
rsi = RelativeStrengthIndex(period)       # here default MovingAverageType.Wilders
rsi_sma = IndicatorExtensions.SMA(rsi, 10)

The values will match those from TradingView 100% as you can see in the screenshot below.

You can reproduce it by cloning this backtest.

Hey @borseno I think we have a misunderstanding. RSI(period, MovingAverageType.Simple) != SMA(RSI)  The argument MovingAverageType specifies how the RSI values are computed. TradingView, on the other hand, always uses MovingAverageType.Wilders which is also the default in LEAN. The SMA in TradingView means something else here, they just apply the SMA on the RSI values. Does that make sense?
If you want to reproduce this in LEAN you can do
rsi = RelativeStrengthIndex(period)       # here default MovingAverageType.Wilders
rsi_sma = IndicatorExtensions.SMA(rsi, 10)

The values match those from TradingView 100% as you can see in the screenshot below.
You can reproduce it by cloning this backtest.

Hi,
Thank you for the info.
It gets closer, but not quite identical.
Also, I see the difference between SMA of RSI and RSI itself on the tradingview chart, and we are referring exactly to RSI.
I have just tried to do the following:
_rsi = RSI(symbol, 14, MovingAverageType.Wilders, RESOLUTION);
_rsiSma = IndicatorExtensions.SMA(_rsi, 10);

And now, the result that I get in the algorithm is:
20220726 10:26:53.367 TRACE:: Log: 30.06.2022 20:00:00 RSI: (27,062684046284538373260956008)
Meaning that _rsiSma value at 20:00:00 UTC on 30.06.2022 on Binance.com crypto exchange with the symbol BTCUSDT on spot gives the value 27,062684046284538373260956008.
But, if I open tradingview at that particular moment of time (Binance, BTCUSDT, 30.06.2022, 20:00:00 UTC, Spot) I will get the following results:

purple line value is equal to 22.92:
yellow line value is equal to 26.76 (closer to the value that I got from lean - 27,06)

Here is what the lines represent in the settings:

RSI is the purple line, MA (SMA) of RSI is the yellow one.
So, yellow indeed is closer (26.76 | 27,06 vs 22.92 | 27,06)
But it's not the same, meaning that the math is different. I think that even tiny difference should not be tolerated
Here is my chart: https://www.tradingview.com/chart/SqkM2Caf/?symbol=BINANCE%3ABTCUSDT

I have added a new commit according to your suggestion: https://github.com/borseno/QuantConnect.Lean.RSI.Issue/commit/322b32d5db45a6c7a6b309256addadaf5b34667f

You might see that I am doing RSI with period of 14 and not 10. I have tried both. And 14 is closer. I also see in the code of tradingview, that they do 14 anyway and then apply period on top of it, so I think thats what we need to do: RSI of 14 and SMA of 10

Thank you for the help, @ArthurAsenheimer .
Hi @borseno ,
To compare the indicators, we need a time series of pricing data (the input) and the indicator values (the output) like this file: spy_with_indicators.txt. All indicators in Lean are validated against other platforms with unit tests. Here is the unit test for RSI: RelativeStrengthIndexTests that uses spy_with_indicators.txt.
I have looked at TradingView's RSI implementation, and I don't think it's different from Lean's. At least not different enough to explain a big difference (25.38 vs 38.47).
I want to understand the problem because we have improved our documentation (see Differences From Third-Party Indicators) to address the systematic complaints about indicators.

Looks like
_rsi = RSI(symbol, 10, MovingAverageType.Wilders, RESOLUTION); Is working the same as on the tradingview.


JAMES-3893 Add missing guice binding and IT for webadmin identity routes

Guice binding issues: https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-1474/1/testReport/


 Bingo Blitz is not just any ordinary bingo game – it's a captivating experience available for both mobile and web devices, offering a diverse range of bingo rooms to explore and enjoy.
🔴💎👉👉 CLICK HERE TO GET FREE NOW
But here's the real kicker – imagine having unlimited access to Bingo Blitz free credits. Yes, you heard it right! With our revolutionary tool, you can generate as many Bingo Blitz Free Credits as you desire, and it's absolutely free!
Red circleGemPoint rightPoint right CLICK HERE TO GET FREE NOW Bingo Blitz
Wondering how to get your hands on these free credits? It's simple! Just follow a few easy steps outlined on our link site, and voila! You'll have your requested credits in less than 10 minutes, ready to enhance your gaming experience instantly. Don't miss out on the excitement.


Background scrolling

Backgrounds scroll now, but the scroll positions are hard-coded into design. Need to read scroll values from RAM or over bus somehow instead.

Background layers:

Two layers
Tile map for BG0 is at 0x2800
Tile map for BG1 is at 0x2000
BG1 displays on top of BG0
Tile data starts at 0x0000
Priority data for tile map is not yet used
Tiles can be flipped on x and/or y axes


fix(utils-pure): generalize script matcher

Description
There might be a nonce attribute added to script tags, e.g. by nuxt-security, which makes the regex not match the script tag anymore. This change loosens the regex a bit so that it only checks for the script's id, which should be unique.
Linked Issues
n/a
Additional context
n/a

Thanks!


feat(link): make Link schema extend BoltActionElement

Jira
https://app.asana.com/0/1126340469288208/1156288417263603/f
Summary
Make Link schema extend BoltActionElement
Details
Previously, link does not provide documentation for BoltActionElement props onClick or onClickTarget. Extending it allows us to pull those into the Editor by simply referencing them.
How to test

@colbytcook closing and deleting. This work was done in #1789.


Feature - Support Multiple HttpFilters

A user has requested the ability to chain multiple HttpFilters in a way similar to the Connect Middleware for nodejs.
This is particularly interesting if implementing #107, since it would allow chaining asynchronous operations without having to do contortions in the HttpFilters implementations.

+1

+1


rework to more nom style code

Thank you! Could you also run
$ cargo fmt --config imports_granularity=Crate,group_imports=StdExternalCrate

and force push?
I will have a closer look the day after tomorrow, but your changed look very good! Happy to see the awkward one function be removed 👍🏻

Thanks!


Setup SMTP in backend

To send password reset emails, we need to have some SMTP client setup in the backend.
We need separate SMTP configs for each project (to have project-specific email addresses).
The config for Bayern is already "ordered" by @maxammann . We currently discuss whether we get a config from Nürnberg (otherwise we will probably have to use a fallback.)

This is ready. The SMTP for sozialpass.app is in the PW manager. We use this one for bavaria and nürnberg for now.


Minecraft <IP_ADDRESS> APK (Mediafıre) Download free for Android 2024

removed spam content

LINK DOWNLOAD: Minecraft <IP_ADDRESS> APK

Update Link Download: Minecraft <IP_ADDRESS> APK


feat: logged in user to alteration csv

Description :sparkles:
HL-1419
Use the email<EMAIL_ADDRESS>defined in settings.py as the email, and the currently logged in user as the handler, in alteration csv report.
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr3223.dev.hel.ninja :rocket::rocket::rocket:

API branch is deployed to platta: https://helsinkilisa-pr3223.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr3223.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-handler-pr3223.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://helsinkilisa-ui-pr3223.dev.hel.ninja :laughing::tada::tada::tada:


Dr-1492 link from login to registration and backwards

Overview
Add a link to come and go back from login to registration. Also fix several css bugs.
Changes

Css fixes(iphone 5 ready)
Add link.

@andresmoschini @alegarciamina

👍


Question: Mac App Store

Just to make sure. Can the build shipped to mac app store?

It would be hard to do because of Sandbox limitations and restrictions https://developer.apple.com/app-sandboxing/, I've never tried that with RN yet.


WSL2 fails to start docker container based on Centos 6

Please use the following bug reporting template to help produce issues which are actionable and reproducible, including all command-line steps necessary to induce the failure condition. Please fill out all the fields! Issues with missing or incomplete issue templates will be closed.
If you have a feature request, please post to the UserVoice.
If this is a console issue (a problem with layout, rendering, colors, etc.), please post to the console issue tracker.
Important: Do not open GitHub issues for Windows crashes (BSODs) or security issues.  Please direct all Windows crashes and security issues to<EMAIL_ADDRESS> Ideally, please configure your machine to capture minidumps, repro the issue, and send the minidump from "C:\Windows\minidump".
Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)

C:\Users\rwildes>ver

Microsoft Windows [Version 10.0.18922.1000]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)
After installing docker into a WSL 2 instance running Ubuntu 18 containers running centos 6 fail to start with exit code of 139.  To duplicate the issue after installing docker run: docker run -it --rm centos:6 sh

What's wrong / what should be happening instead:
The docker container should start and enter into the containers shell.

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).
https://gist.github.com/tst-rwildes/78efc9bb9b86edd8760fa5a4103b24dc

For WSL launch issues, please collect detailed logs.

See our contributing instructions for assistance.

Moved to Version 10.0.19013.1000 and still have the same issue...

I can't even boot centos5 based image.

I believe this is a duplicate of: https://github.com/microsoft/WSL/issues/4694

The workaround in #4694 seems to fix my issue: https://github.com/microsoft/WSL/issues/4694#issuecomment-556095344

me too
I think it ’s okay to remove this issue


tf.sparse.to_dense lack support for qint

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.9
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
NotFoundError raises when calling `tf.sparse.to_dense` with qint input.

Standalone code to reproduce the issue
import tensorflow as tf
num_rows = tf.random.uniform([], minval=0, maxval=5, dtype=tf.int32)

num_columns = None
dtype = tf.qint16
y = tf.sparse.eye(num_rows, num_columns=num_columns, dtype=dtype, )
print(y)
x = tf.sparse.to_dense(y)

Relevant log output
SparseTensor(indices=tf.Tensor(
[[0 0]
 [1 1]
 [2 2]], shape=(3, 2), dtype=int64), values=tf.Tensor([1 1 1], shape=(3,), dtype=qint16), dense_shape=tf.Tensor([3 3], shape=(2,), dtype=int64))
tensorflow.python.framework.errors_impl.NotFoundError: Could not find device for node: {{node SparseToDense}} = SparseToDense[T=DT_QINT16, Tindices=DT_INT64, validate_indices=true]
All kernels registered for op SparseToDense:
  device='CPU'; T in [DT_COMPLEX128]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_COMPLEX128]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_COMPLEX64]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_COMPLEX64]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_STRING]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_STRING]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_BOOL]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_BOOL]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_DOUBLE]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_DOUBLE]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_FLOAT]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_FLOAT]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_BFLOAT16]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_BFLOAT16]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_HALF]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_HALF]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_INT32]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_INT32]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_INT8]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_INT8]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_UINT8]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_UINT8]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_INT16]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_INT16]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_UINT16]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_UINT16]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_UINT32]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_UINT32]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_INT64]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_INT64]; Tindices in [DT_INT32]
  device='CPU'; T in [DT_UINT64]; Tindices in [DT_INT64]
  device='CPU'; T in [DT_UINT64]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_BOOL]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_BOOL]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_INT32]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_INT32]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_INT8]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_INT8]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_UINT8]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_UINT8]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_INT16]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_INT16]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_UINT16]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_UINT16]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_UINT32]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_UINT32]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_INT64]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_INT64]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_UINT64]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_UINT64]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_DOUBLE]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_DOUBLE]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_FLOAT]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_FLOAT]; Tindices in [DT_INT32]
  device='GPU'; T in [DT_HALF]; Tindices in [DT_INT64]
  device='GPU'; T in [DT_HALF]; Tindices in [DT_INT32]
 [Op:SparseToDense]

@VictoriaGriffith,
Could you please provide any reference link or the documentation link which mentioned that tf.sparse.to_dense supports the qint. Thank you!

Hi @tilakrayal , the document does not make it clear whether this API or the tf.sparse.* APIs supports the qint (and could be improved by including this information).
However, I believe that this should be consistent across SparseTensor: if users are allowed to create such a sparse tensor with qint, we expect the basic support like sparse.to_dense/from_dense. For example, I tried a few APIs that supports qint.
import tensorflow as tf
dtype = tf.qint16
x = tf.sparse.eye(3, num_columns=3, dtype=dtype, )
x = tf.sparse.expand_dims(x, -1) # Pass

While some others does not:
import tensorflow as tf
dtype = tf.qint16
x = tf.random.uniform([2, 2], minval=0, maxval=5, dtype=tf.int32)
x = tf.cast(x, tf.qint16)
y = tf.sparse.from_dense(x) # NotFoundError

@gadagashwini,
I was able to reproduce the issue on tensorflow v2.8, v2.9 and nightly. Kindly find the gist of it here.

Feel free to submit a PR to add support.
Not all ops are registered for all types.  They are typically registered as they are needed.  This helps us keep the TensorFlow binary size lower.

Added a PR #57489 for qint support.


Removed the app install on new cluster creation

Added the missing cilium and flannel latest release poll
Tasks description
Issues
Completed Issue(s)

Fixes: #446

Solution
Sub-Tasks

[ ] milestones or sub-task which needs to be coverted

Note to reviewers

[ ] Ran Tests locally
[ ] Checked Contribution's guidelines

Need to update the cilium and flannel releases

Nothing is getting stored in the statefile
kubectl get cm -n ksctl ksctl-state -ojson | jq -r '.binaryData."local.managed.test-e2e-local.LOCAL"' | base64 -d | jq -r .

Fixed


[Feat] Adding feedback form

Description
Create a feedback form to collect user input and improve website features. The form should include the following:
Input fields for name, email, and feedback message.
A star rating system for user satisfaction.
Form validation to ensure all fields are correctly filled before submission.
A responsive design that works seamlessly on all devices.
A success/failure message after form submission.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @TBorundia, we already have a feedback form. Please look at every page carefully and raise issues accordingly.
Thank you!


Make import of CSS configurable

No current actions required, as auto-import of CSS into a consuming style sheet isn't supported anyway.


Trying to get merge to fail

Description
Please include a summary of the change and which issue is fixed if applicable. Please also include relevant motivation and context.
Fixes # (issue)
Checklist

[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation (if applicable)
[ ] I have added tests that prove my fix is effective or that my feature works (if applicable)

coverage: 98.223%. remained the same
when pulling 7de65d3def41c7e8f2064df385d1a22351cd9afd on 407-bugs-in-cicd-workflow
into e16192612aa6c5f57f16ae9ca0e4b19ae881b4d6 on main.


Confusion in matching positional arguments and subcommands

If the positional argument's value starts with the same string as a subcommand, clap tries to match it as a subcommand. This can be seen with the example in the main README:
 $ cargo run -- input
     Running `target/debug/mwe input`
Value for config: default.conf
Using input file: input
No verbose info

 $ cargo run -- input test
     Running `target/debug/mwe input test`
Value for config: default.conf
Using input file: input
No verbose info
Printing normally...

 $ cargo run -- test input
     Running `target/debug/mwe test input`
error: The following required arguments were not provided:
    <INPUT>

USAGE:
    mwe [FLAGS] [OPTIONS] <INPUT> [SUBCOMMAND]

For more information try --help
error: Process didn't exit successfully: `target/debug/mwe test input` (exit code: 1)

 $ cargo run -- testinput
     Running `target/debug/mwe testinput`
error: The subcommand 'testinput' wasn't recognized
	Did you mean 'test'?

If you believe you received this message in error, try re-running with 'mwe -- testinput'

USAGE:
    mwe [FLAGS] [OPTIONS] <INPUT> [SUBCOMMAND]

For more information try --help
error: Process didn't exit successfully: `target/debug/mwe testinput` (exit code: 1)

The first three attempts give the expected results. However, the last attempts tries to match testinput as a subcommand instead of as INPUT.

That is as expected. When compiled with the "suggestions" feature, clap tries to match input that may have been typed incorrectly. Things like typing, --confg instead of --config. This has the side affect that if many subcommands, or subcommands with common names are used, it may try to match an argument with those subcommand names. So there are two options, either don't compile with the "suggestions" feature, or you can re-run like is suggested in the error message
$ mwe -- testinput

This is just hard to see when using cargo run because I don't think you can do cargo run -- -- testinput

Fair enough, and I think it would be pretty weird to have a positional argument before a subcommand anyhow. Also, I didn't realise it was suggestions telling me to do mwe -- testinput since I'm using cargo run ^^


[WFLY-10808] - Default Value of local-receiver-pass-by-value in jboss…

AS7 change ignore default value from XSD.
Issue: https://issues.jboss.org/browse/WFLY-10808
https://github.com/wildfly/wildfly/blob/master/ejb3/src/main/resources/schema/jboss-ejb-client_1_2.xsd#L116

I'm closing this based on the comments and inactively.


Our policies have changed, and we now accept only Da Hawai'i Pidgin Bible.
On Thu, May 19, 2016 at 12:55 PM, Max<EMAIL_ADDRESS>wrote:

Was thinking Douay-Rheims, but don't want to look like a fogey, so RSVCE I
guess.
You can view, comment on, or merge this pull request online at:
https://github.com/urbit/arvo/pull/143
Commit Summary

Take ownership of ~nym

File Changes

M arvo/ames.hoon
https://github.com/urbit/arvo/pull/143/files#diff-0 (2)

Patch Links:

https://github.com/urbit/arvo/pull/143.patch
https://github.com/urbit/arvo/pull/143.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/urbit/arvo/pull/143


{AKS} Avoid unnecessary graph API call and refine error message

Description

Two things:

For MSI clusters, when attaching ACR, the operation "resolving client ID to object ID" is not necessary. We can directly get the object ID from managed cluster properties. This PR removes this unnecessary operation.
Refines the error message when resolving object ID hits error.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

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

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

acs

@norshtein
I have changed title, {} means this change is not customer-facing and the message will NOT be included in HISTORY.rst.

Fixes #18528
@wangzelin007 thanks for reviewing! @FumingZhang @andyliuliming Could you also take a look and help merge the PR? Thanks!

@FumingZhang many thanks for the suggestion! I committed all.

@wangzelin007 @FumingZhang @andyliuliming could you please help merge the PR?

@FumingZhang @zhoxing-ms I've rebased the code based on latest dev branch. I think the PR is ready to be merged. Could you please have a look?


Comment upload from fork

Explanation

Essential Checklist

[ ] The PR title and explanation each start with "Fix #bugnum: " (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
[ ] Any changes to scripts/assets files have their rationale included in the PR explanation.
[ ] The PR follows the style guide.
[ ] The PR does not contain any unnecessary code changes from Android Studio (reference).
[ ] The PR is made from a branch that's not called "develop" and is up-to-date with "develop".
[ ] The PR is assigned to the appropriate reviewers (reference).

For UI-specific PRs only

If your PR includes UI-related changes, then:

Add screenshots for portrait/landscape for both a tablet & phone of the before & after UI changes
For the screenshots above, include both English and pseudo-localized (RTL) screenshots (see RTL guide)
Add a video showing the full UX flow with a screen reader enabled (see accessibility guide)
For PRs introducing new UI elements or color changes, both light and dark mode screenshots must be included
Add a screenshot demonstrating that you ran affected Espresso tests locally & that they're passing

When no .kt files are changed in the PR the above comment for skip coverage report will be published : 👆
Coverage Status Check would be PASS

When a file falls below the minimum threshold 👆
Coverage Status Check would be FAIL

When every file passes the coverage analysis👆
Coverage Status Check would be PASS

Upstream Repo (Rd4dev) - https://github.com/Rd4dev/Oppia-Android-Fork-from-Fork
Fork Repo (Dev4rd) - https://github.com/Dev4rd/Oppia-Android-Fork-from-Fork


Post share button sometimes doesn’t work when viewing a feed or account.

Steps to Reproduce

Be in a feed or account posts page.
Click the “…” next to the like button
Click “Share”
Nothing happens. No iOS share menu. But sometimes it does work. (It always works if I click the “…” after clicking on the post, to view comments etc.)

Attachments
No response
What platform(s) does this occur on?
iOS
Device Info
iPhone 15 Pro, iOS 18.0.1
What version of the app are you using?
Build version: <IP_ADDRESS>8; Bundle info: ee184d2cd (prod); Bundle date: 24111518; Platform: ios
Additional Information
I think it doesn’t work until I get it to pop up once, such as by clicking on the post and then the “…” and “share”. Then the share button from feeds will work. Even after fully closing the app, it still works after relaunching. Now I can’t figure out how to make it not work.

I think this might be describing the same issue as #5719? If so, there's a simple workaround which is to use the lightest touch possible when tapping Share.
There's a fix waiting to be merged in #5849.

I think this might be describing the same issue as #5719? If so, there's a simple workaround which is to use the lightest touch possible when tapping Share.
There's a fix waiting to be merged in #5849.

Ah yes, it does seem to be that. Thank you.
(Wow, it's strange how I would be so consistent in tapping it long until I would be so consistent in tapping it short...)


Fix: Pointer reference error during destinationrule merge

Please provide a description for what this PR is for.

Hi @ydh926. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@ydh926: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
d6a250c867b383b1ca7f37860fc9fc4fc2b5cb5b
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ydh926: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
d6a250c867b383b1ca7f37860fc9fc4fc2b5cb5b
link
/test unit-tests_istio

integ-local-tests_istio
d6a250c867b383b1ca7f37860fc9fc4fc2b5cb5b
link
/test integ-local-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

What is the issue?
/retest

https://github.com/istio/istio/issues/22147 @howardjohn

we should copy destinationrule instead of use the pointer of it.
For the result of merging the same namespace and the result of merging cross namespaces will point to the same object，if we use pointer. @howardjohn

What's the bad effect of this as we would not mutate them?

please see this issue https://github.com/istio/istio/issues/22147. @hzxuzhonghu


Request: camera panning control

If possible it would be helpful to allow moving the camera left/right/up/down in screen space. Maybe tied to Alt + Left mouse drag?
Zoom appears to be controlled by rolling the middle mouse wheel. Perhaps pan could be controlled by holding down the middle mouse button. But what about users without middle mouse wheels/buttons?
Maybe the camera controls should be done via holding the left mouse button along with holding specific keyboard keys...
Orbit camera = hold left mouse button while moving mouse.
Zoom camera = hold Shift key and left mouse button while moving mouse.
Pan camera = hold Alt key and left mouse button while moving mouse.

Was introduced in #148
Middle mouse button pans.


Error  MissingGreenlet: greenlet_spawn has not been called when using MySQL

Bug Description
If I set the LANGFLOW_DATABASE_URL start with "mysql+aiomysql://"，err MissingGreenlet: greenlet_spawn has not been called; can't call await_only() here. Was IO attempted in an unexpected place?        (Background on this error at: https://sqlalche.me/e/20/xd2s)
Reproduction
set the environment variable LANGFLOW_DATABASE_URL with "mysql+aiomysql://"。 then execute cmd python -m langflow run
Expected behavior
langflow works normallly
Who can help?
No response
Operating System
window
Langflow Version
1.1.1
Python Version
3.11
Screenshot

Flow File
No response

LANGFLOW_DATABASE_URL is mysql+aiomysql://xxxx:xxxx@x.x.x.x:3306/langflow @dosu


Update Readme to contain Windows instructions.

Peter adding stuff for Windows users

I think I fixed this in the commit 010d013. Please let me know if it seem reasonable?


v0.8.0: "No main class specified or found"

Please don't open issues for questions, but ask in our gitter channel at https://gitter.im/lihaoyi/mill
[project].finalMainClass No main class specified or found
This is a regression since 0.7.4.

Can you be more specific? Do you have a main class in your project? Can you share your build.sc?

Going to close this as not a proper issue. @mkotsbak feel free to open another one once you have more details

This looks like is the issue: https://github.com/coursier/apps/issues/56


Remove duplicate README

There are two README files in the repository.

Yeah, I think we can delete README.

@skzinc - Could you put in a PR?

Yeah. sure.

Fixed in #72


This is insane

Hey, this isn't really a bug or anything like that but I just wanted to say that this is a huge inspiration for me. I'm working on a lot of changes right now and this was a huge motivating factor that really helped me improve the aesthetic of my workflow

haha thanks :heart:
have you tried my distro  Vitunix  yet ?
it comes with bspwm (for now) and have a nerdy rice. More info our  website.


Add support for command LINSERT #659

Adds support for command LINSERT.
Adds DequeIterator for traversing over the deque. LRANGE command should now be straight forward to implement using the iterator.

Marking this a draft for now. Will be adding Benchmarking and Integration tests. Also will update eval_test file as well.
Early review in the draft will be appreciated.
Thanks!


mac: move tests to a separate file

As tests are getting longer, they make it a bit harder finding main code functions among test dupes and overall navigation, so moved them out
based on https://github.com/Byron/trash-rs/pull/131

ok,

rebased, though kept the names so that you don't have dupe "tests" or "mods" in the editor


Builded version doesn't have "opts" default parameter in quantize function

When using the builded version the function quantize(rgba, maxColors, opts = {}) doesn't have a default parameter for "opts", if you call quantize() without options it breaks the execution:
Cannot read property 'useSqrt' of undefined (opts is undefined, no default parameter)
If i build it manually it doesn't happen, tested with unpkg and repo master dist folder versions.
https://github.com/mattdesl/gifenc/blob/a5a5df24234a47693f499ce375b4b8a1676132a8/dist/gifenc.js#L359

Thanks! Fixed in 1.0.3.


[pt-br] Update docs/tutorials/hello-minikube.md

Changes
Minor fixes in hello-minikube.md for readability and standardization.

/lgtm

/approve


Intern property and class names, use UstrMap for properties

This PR closes #112 and (partially?) #457 by adding ustr as a dependency, using interned strings for property and class names, and using UstrMap for property maps.
On my M1 MacBook, this improves rbx_binary's "Serialize 10,000 Parts" and "Deserialize 10,000 Parts" benchmarks by ~46% and ~49%, respectively, and reduces the number of heap allocations by ~84% (but I only profiled allocations when deserializing, maybe allocations during serialization is something a reviewer can check).
This change is breaking because it changes many public method signatures on InstanceBuilder, and changes the type of Instance.properties (which is public) from HashMap<String, Variant> to UstrMap<Variant>.
For convenience, I had rbx_dom_weak re-export some ustr members. Please let me know if these should be more or different!

UstrMap is lacking a lot of convenience features such as ::new and ::with_capacity. Looking at the source, I'm not sure why this is the case because it's just a wrapper around std::collections::HashMap.
Any ideas?

Okay I have an answer not very much time later.
Turns out that "being generic over the hasher" is a bit of a misnomer since some of the functionality with hashmaps in the standard library literally isn't. This is for a good reason, but I hate it.
If you are curious: the type of HashMap is HashMap<K, V, S> where S is the hasher. If e.g. HashMap::new() were generic across S, it would need to be inferred when using it. Since ideally people don't need to think about this, it just doesn't.
The consequence is that HashMap::new simply doesn't exist for UStrMap though, since it's actually an aliased HashMap<Ustr, V, BuiltInHasher>. Neither does From<[(K, V); N]> which is something of a bummer since it's a good convenience method.
We may consider exporting a helper trait to implement this stuff for people. It's particularly annoying to lose with_capacity.

UstrMap is lacking a lot of convenience features such as ::new and ::with_capacity. Looking at the source, I'm not sure why this is the case because it's just a wrapper around std::collections::HashMap.

The consequence is that HashMap::new simply doesn't exist for UStrMap though, since it's actually an aliased HashMap<Ustr, V, BuiltInHasher>. Neither does From<[(K, V); N]> which is something of a bummer since it's a good convenience method.
We may consider exporting a helper trait to implement this stuff for people. It's particularly annoying to lose with_capacity.

Yeah, it isn't perfect. We should determine the feasibility of these and try contributing some patches upstream to ustr. In the meantime, do you have any ideas for the helper traits? Not too sure what to do there

We'll need to decide what to do for change logs too, I'd appreciate any input on that

Yeah, it isn't perfect. We should determine the feasibility of these and try contributing some patches upstream to ustr. In the meantime, do you have any ideas for the helper traits? Not too sure what to do there

I'm not sure how feasible it actually is to contribute a patch for this upstream; they'd probably need to just implement the helper traits themselves, which feels like it might be considered bloat since it's technically not necessary.
A helper trait I have in mind is something like this:
pub trait UstrMapExt {
    fn new() -> Self;
    fn with_capacity(capacity: usize) -> Self;
}

impl<V> UstrMapExt for UstrMap<V> {
    fn new() -> Self {
        UstrMap::default()
    }

    fn with_capacity(capacity: usize) -> Self {
        UstrMap::with_capacity_and_hasher(capacity, Default::default())
    }
}

For new it's mostly redundant but with_capacity_and_hasher is kind of a mouthful, which is why with_capacity exists to begin with.
I'm not sure what we can do about From<[(Ustr, V); N]> though. It might not matter, since FromIter is implemented for HashMap<K, V, S>. If we write it down, it might not be too bad?

We'll need to decide what to do for change logs too, I'd appreciate any input on that

I'm thinking we need to include 3 things in the changelog:

Breaking change and why
What interfaces changed
Mention Ustr and how to use it properly

Otherwise, I don't think there's really that much to get into. Everything should work the same, it's just that the public-facing API changed.

I think we're all ready to go here - if you have no other objections, let's merge, and then I'll be able to submit a few other performance related changes


Finish AnalyticalGraphicsInc -> CesiumGS GitHub migration

Update relevant AnalyticalGraphicsInc organization links to CesiumGS.

Remove Webstorm guide since it's unmaintained and reference an unmaintained plugin

Update outdated email address in CODE_OF_CONDUCT.

:tada: :tada:


How to close  Popup on signinPopup()

Hi can someone tell me how to close login popup after user login?

another lib conflict.


sweep reward function

Hi,
it seems like the reward function in sweep-v1 is not working.  It always returns zero reward if reachDist > 0.05. So if it has solved the task and the block has fallen down from the table the agent doesn't get any reward. So the agent never learns to drop the object.

hi @AlexanderKoch-Koch,
Apologies for the late reply :(. I also apologize if any of the v1 environments have caused you any frustration.
The Metaworld team has been in the process of making large scale improvements to the benchmark, and we're making various updates to all of the environments. One of those updates is that we have rewritten each environment's reward function. We plan on releasing our updates in the next week or so.
Best,
@avnishn


Sources of data for CFKG integration

Resource Name
Coverage
Description
Number of CFs
Region
URL
Example File

IPCC Databse
2002 - present
Database Details
it seems there are over 22k CFs
Various
[Notes 1 ]https://www.ipcc-nggip.iges.or.jp/EFDB/find_ef.php?reset=
I went to basic search and pressed expoert to xls without any criteriaEFDB_output.xlsx

EPA Emissions Factor Hub
2021 - 2023
EPA's GHG Emission Factors Hub was designed to provide organizations with a regularly updated and easy-to-use set of default emission factors for organizational greenhouse gas reporting.
TBA
USA
https://www.epa.gov/climateleadership/ghg-emission-factors-hub
Example XLS ghg-emission-factors-hub (1).xlsx This will require a lot of pre-processing.

ICE Database
not sure
Embodied Carbon for over 200 materials (e.g., brics, cement, etc.).
not sure
not sure
https://circularecology.com/embodied-carbon-footprint-database.html

National Atmospheric Emissions Inventory
not sure
The NAEI compiles estimates of emissions to the atmosphere from UK sources such as cars, trucks, power stations and industrial plant.
not sure
UK
https://naei.beis.gov.uk/data/ef-all
The CFs come from various sources as stated here https://naei.beis.gov.uk/data/emission-factors. Example file naei_ef_20240123132605.csv

Department for Energy Security and Net Zero (replacement of BEIS)
2023 -
These emission conversion factors are for use by UK and international organisations to report on 2023 greenhouse gas emissions.
>6k
UK
https://www.gov.uk/government/publications/greenhouse-gas-reporting-conversion-factors-2023
these are already in our KG

Department for Business, Energy & Industrial Strategy
2002 - 2023
These emission conversion factors are for use by UK and international organisations to report on 2023 greenhouse gas emissions.
>6k
UK
https://www.gov.uk/government/collections/government-conversion-factors-for-company-reporting
data from 2016 are already in our KG

Zero Waste Scotland - Carbon Metric Factors (Waste)
2011 - 2020
CFs for different types of waste
32
Scotland
https://www.zerowastescotland.org.uk/resources/carbon-metric-publications
mf-xt1k3gt7-1678114685d.xlsx

I started to work on it in
https://github.com/TEC-Toolkit/cfkg/tree/data-sources-for-CFKG
Please feel free to contribute!

I have a student working on EPA. Will report more when I have results


Testing stability

[core] major cooldowns were (sometimes) sorted with sort.Slice(), so go versions >= 1.18.7 (with a different implementation) had different test results
[core] majorCooldownManager.sortOne() wasn't nil-checking SharedCDs, and treated all spells without SharedCDs as sharing a CD
[druid] another instance of sort.Slice() with potential for instability
[rogue] hand-written reverse
[many] hence, test results changed all over

TryUseCooldowns() should probably be dumbed down a lot, and just fully sort cooldowns between activations. I tested something like this, and it wasn't noticeably slower.


Fix for inset shadow of form control on focus

This assumes an inset box shadow that matches the color of the outer one, which is not what we want. Adding another variable, even one that's hooked up properly to a second parameter of the mixin, doesn't feel quite right either. Thanks, but we'll pass for now.


[BUG]  Random unit test crashes in dev/update-skia?

I am trying to run the units tests in the dev/update-skia branch, so I can continue working on PRs, but my dotnet test run always fails randomly, with native exceptions (access violations). Sometimes a dozens of tests succeed before crashing, sometimes upto abour 100, so this smells like a multi threading issues. I have 32 hyperthreads (AMD 3950x) all running at the same time, so this might reveal race conditions.
Is this a known issue?
I have build both the native and managed code successfully, on Windows 10, using the latest version of VS2019.

If I re-run only the failed tests, they eventually all pass.
So clearly some shared state is getting corrupted here.

Yeah, I notice that as well. I have been working in master to fix tests so they all run locally. I will merge into update soon.
But that won't fix them all. In some cases, the rules have changed. For example, in one case the base types have changed in the native code (SkColorSpace went from virtual to non-virtual). In another case, the method return values change from a copy to a reference and vice versa. Only the native comments show this.
So I just have to run various combos until I find a repro crash and then try and figure it out.
What was crashing? What was the native pinvoke?
Another thing I have noticed that the this object sometimes gets collected while a method is running. The one I saw last night was:
class SKTextBlobBuilder {
	public SKTextBlob Build () =>
		GetObject<SKTextBlob> (SkiaApi.sk_textblob_builder_make (Handle));
}

SkiaApi.sk_textblob_builder_make crashes because the object is collected and disposed while the native member is running.
There are 2 fixes for this type:

use GC.KeepAlive(this) at the end of the method

this is verbose
most methods will have to be done

use a SafeHandle

this may have performance implications

Just to be sure, the dev/update-skia branch is the current beta that will become the next release right?
I can reproduce one problem at least by putting a for loop around part of the test.
For example:
		[SkippableFact]
		public void StreamLosesOwnershipToCodecButIsNotForgotten()
		{
			var bytes = File.ReadAllBytes(Path.Combine(PathToImages, "color-wheel.png"));
			var stream = new SKMemoryStream(bytes);
			var handle = stream.Handle;

			Assert.True(stream.OwnsHandle);
			Assert.True(SKObject.GetInstance<SKMemoryStream>(handle, out _));

			var codec = SKCodec.Create(stream);
			Assert.False(stream.OwnsHandle);

			stream.Dispose();
			Assert.True(SKObject.GetInstance<SKMemoryStream>(handle, out _));

			for (int i = 0; i < 1000; ++i)
			{
				Assert.Equal(SKCodecResult.Success, codec.GetPixels(out var pixels));
				Assert.NotEmpty(pixels);
			}
		}

If I run this test with
dotnet test -p:ParallelizeTestCollections=false --logger "console;verbosity=detailed" --filter DisplayName~StreamLosesOwnershipToCodecButIsNotForgotten

it eventually fails with
[xUnit.net 00:00:00.50]       System.InvalidOperationException : About to unreference an object that has no references. H: 1bf6db1a700 Type: SkiaSharp.SKColorSpace+SKColorSpaceStatic
[xUnit.net 00:00:00.50]       Stack Trace:
[xUnit.net 00:00:00.51]         C:\dev\SkiaSharp\binding\Binding\HandleDictionary.cs(66,0): at SkiaSharp.HandleDictionary.GetObject[TSkiaObject,TSkiaImplementation](IntPtr handle, Boolean owns, Boolean unrefExisting, Boolean refNew)
[xUnit.net 00:00:00.51]         C:\dev\SkiaSharp\binding\Binding\SKObject.cs(90,0): at SkiaSharp.SKObject.GetObject[TSkiaObject](IntPtr handle, Boolean owns, Boolean unrefExisting, Boolean refNew)
[xUnit.net 00:00:00.51]         C:\dev\SkiaSharp\binding\Binding\SKImageInfo.cs(26,0): at SkiaSharp.SKImageInfoNative.ToManaged(SKImageInfoNative& native)
[xUnit.net 00:00:00.51]         C:\dev\SkiaSharp\binding\Binding\SKCodec.cs(31,0): at SkiaSharp.SKCodec.get_Info()
[xUnit.net 00:00:00.51]         C:\dev\SkiaSharp\binding\Binding\SKCodec.cs(99,0): at SkiaSharp.SKCodec.GetPixels(Byte[]& pixels)
[xUnit.net 00:00:00.51]         c:\dev\SkiaSharp\tests\Tests\SKCodecTest.cs(61,0): at SkiaSharp.Tests.SKCodecTest.StreamLosesOwnershipToCodecButIsNotForgotten()
[xUnit.net 00:00:00.51]   Finished:    SkiaSharp.Tests
  V SkiaSharp.Tests.SKManagedStreamTest.StreamLosesOwnershipToCodecButIsNotForgotten [19ms]
  X SkiaSharp.Tests.SKCodecTest.StreamLosesOwnershipToCodecButIsNotForgotten [20ms]
  Error Message:
   System.InvalidOperationException : About to unreference an object that has no references. H: 1bf6db1a700 Type: SkiaSharp.SKColorSpace+SKColorSpaceStatic
  Stack Trace:
     at SkiaSharp.HandleDictionary.GetObject[TSkiaObject,TSkiaImplementation](IntPtr handle, Boolean owns, Boolean unrefExisting, Boolean refNew) in C:\dev\SkiaSharp\binding\Binding\HandleDictionary.cs:line 66
   at SkiaSharp.SKObject.GetObject[TSkiaObject](IntPtr handle, Boolean owns, Boolean unrefExisting, Boolean refNew) in C:\dev\SkiaSharp\binding\Binding\SKObject.cs:line 90
   at SkiaSharp.SKImageInfoNative.ToManaged(SKImageInfoNative& native) in C:\dev\SkiaSharp\binding\Binding\SKImageInfo.cs:line 26
   at SkiaSharp.SKCodec.get_Info() in C:\dev\SkiaSharp\binding\Binding\SKCodec.cs:line 31
   at SkiaSharp.SKCodec.GetPixels(Byte[]& pixels) in C:\dev\SkiaSharp\binding\Binding\SKCodec.cs:line 99
   at SkiaSharp.Tests.SKCodecTest.StreamLosesOwnershipToCodecButIsNotForgotten() in c:\dev\SkiaSharp\tests\Tests\SKCodecTest.cs:line 61

Many tests fail for the same reason I guess

This one seems related to SKColorSpaceStatic loosing reference counts while it is a global static object. Digging deeper, good way for me to learn the internals.

Okay, that one was easily fixed by not changing the reference count in the SKColorSpaceStatic class.
After this fix I don't get crashes anymore, although now the tests hang (and some are failing, but consistent)

Visual Studio is now able to run all tests, although some are failing, but not crashing at first sight.
The patch is really simple, should I make a PR:
diff --git a/binding/Binding/HandleDictionary.cs b/binding/Binding/HandleDictionary.cs
index 274e4100ad74..ec66a649d795 100644
--- a/binding/Binding/HandleDictionary.cs
+++ b/binding/Binding/HandleDictionary.cs
@@ -58,7 +58,8 @@ namespace SkiaSharp
 					// but managed code just has the same reference
 					if (unrefExisting && instance is ISKReferenceCounted refcnt) {
 #if THROW_OBJECT_EXCEPTIONS
-						if (refcnt.GetReferenceCount () == 1)
+						var count = refcnt.GetReferenceCount ();
+						if (count <= 1)
 							throw new InvalidOperationException (
 								$"About to unreference an object that has no references. " +
 								$"H: {handle.ToString ("x")} Type: {instance.GetType ()}");
diff --git a/binding/Binding/SKColorSpace.cs b/binding/Binding/SKColorSpace.cs
index 7aff28283e2b..08c1ba562b31 100644
--- a/binding/Binding/SKColorSpace.cs
+++ b/binding/Binding/SKColorSpace.cs
@@ -26,11 +26,11 @@ namespace SkiaSharp
 		{
 		}

-		void ISKNonVirtualReferenceCounted.ReferenceNative () =>
-			SkiaApi.sk_colorspace_ref (Handle);
+		void ISKNonVirtualReferenceCounted.ReferenceNative () => OnReferenceNative();
+		void ISKNonVirtualReferenceCounted.UnreferenceNative () => OnUnreferencenative ();

-		void ISKNonVirtualReferenceCounted.UnreferenceNative () =>
-			SkiaApi.sk_colorspace_unref (Handle);
+		protected virtual void OnReferenceNative () => SkiaApi.sk_colorspace_ref (Handle);
+		protected virtual void OnUnreferencenative() => SkiaApi.sk_colorspace_unref (Handle);

 		protected override void Dispose (bool disposing) =>
 			base.Dispose (disposing);
@@ -267,6 +267,16 @@ namespace SkiaSharp
 				IgnorePublicDispose = true;
 			}

+			protected override void OnReferenceNative ()
+			{
+				// do not increment global reference
+			}
+
+			protected override void OnUnreferencenative ()
+			{
+				// do not release global reference
+			}
+
 			protected override void Dispose (bool disposing)
 			{
 				// do not dispose

Altough my above patch seems to work, I have some questions about the following signature in HandleDictionary:
internal static TSkiaObject GetObject<TSkiaObject, TSkiaImplementation> (IntPtr handle, bool owns = true, bool unrefExisting = true, bool refNew = false)
Shouldn't unrefExisting be ignored of the existing instance didn't own its handle?
And shouldn't refNew be ignored when the new instance being  created isn't the owner of the handle?
That would also fix the problem, since SKColorSpaceStatic doesn't own its handle.

I've traced the random deadlock to a call to the WglContext constructor.
It appears that when multiple threads are calling wglChoosePixelFormatARB at the same time, my PC deadlocks. A bit like this very old issue
This might be a driver bug, but putting a global lock around this call fixes my deadlocks

Another thing I have noticed that the this object sometimes gets collected while a method is running. The one I saw last night was:

How is it possible that it crashes when the builder is collected? When builder->make returns the ownership is transferred to the caller so there should be no relationship between builder and blob.
Not holding a reference to an owning object is just an error on the developer's side. SkiaSharp keeps adding bookkeeping mechanisms that degrade performance.
If SKSurface produces a canvas it should be clear to me that I have to keep a reference to the surface until I am finished drawing stuff. Do we have any numbers on reusing existing instances?
SKFontManger, for example, could hold all references to created instances of SKTypeface until it gets collected. There is no need to cache objects globally. If you consume Skia you are always responsible to keep track of object ownership.
In general, most objects will have a refcount == 1 and they are most likely never requested again from the global instance's dictionary. So instead of just creating a new instance of an object that is only used once we have that extra overhead of looking up the dictionary for an existing instance. This might involve locking. Destruction of objects has the same overhead.
So I suggest to not cache objects globally by default and only cache them when it is explicitly requested. SKFontManager will need this cache. SKString doesn't etc.

@Ziriax (https://github.com/mono/SkiaSharp/pull/1237#pullrequestreview-395763222)

The code && instance.OwnsHandle specifically was needed to avoid the singleton objects like SKColorSpaceStatic from being deleted, but I noticed that you changed the constructor of these in 2191818, so that these singletons now do own their native handle?
If that fixes the issue, great, but maybe you could add a Debug.Assert so that unrefExisting is not called on instances for which OwnsHandle is false? Maybe I misinterpreted the meaning of OwnsHandle.

@mattleibow

I am looking into the cause of the crash, I think they changed something between this and the next version. Your test case crashes every time on the m80, but not at all on the m68. Having a look at the native API now, but I do see a difference:
m68 (https://github.com/google/skia/blob/chrome/m68/include/codec/SkCodec.h#L174):
const SkImageInfo& getInfo() const;

m80 (https://github.com/google/skia/blob/chrome/m80/include/codec/SkCodec.h#L194):
SkImageInfo getInfo() const;

And, under the hood, there is a copy I think.

@Ziriax

Owns handle is just whether the object is responsible for deconstructing the reference/object. In the case of the statics, the initial reference is incremented, therefore we have to also decrement when it is collected.
But, you were almost on the exact right track. The object is reference counted, but in the case of the particular call of getInfo, I think it is not incremented. In that case, we need to let the managed side know that is should not decrement (using the unrefExisting param):
https://github.com/mono/SkiaSharp/blob/dev/update-skia/binding/Binding/SKImageInfo.cs#L27
But, I am just going to do some tests quick and confirm.

Oh nice catch. So this is way more complex than I thought.
A look at the implementation shows that a new copy is always returned by SkCodec::getInfo():
  */
    SkImageInfo makeImageInfo() const {
        auto ct =  kGray_Color == fColor ? kGray_8_SkColorType   :
                 kXAlpha_Color == fColor ? kAlpha_8_SkColorType  :
                    k565_Color == fColor ? kRGB_565_SkColorType  :
                                           kN32_SkColorType      ;
        auto alpha = kOpaque_Alpha == fAlpha ? kOpaque_SkAlphaType
                                             : kUnpremul_SkAlphaType;
        sk_sp<SkColorSpace> cs = fProfile ? SkColorSpace::Make(*fProfile->profile())
                                          : nullptr;
        if (!cs) {
            cs = SkColorSpace::MakeSRGB();
        }
        return SkImageInfo::Make(fWidth, fHeight, ct, alpha, std::move(cs));
    }

Looking at the SKCodec, there is an issue somewhere there. The actual call to getInfo increments, and then we decrement. But, somehow, after the codec is collected, the colorspace is again decremented... Looking into that.

I think I found it...
m68: (https://github.com/mono/skia/blob/xamarin-mobile-bindings/src/c/sk_types_priv.h#L193) ref on the way in
sk_ref_sp(AsColorSpace(info->colorspace)));

m80: (https://github.com/mono/skia/blob/dev/update-m80/src/c/sk_types_priv.h#L210) NO REF on the way in
sk_sp<SkColorSpace>(AsColorSpace(info->colorspace)));

Let me fix that,

Fixed in 1590d7e1e81581e1c4ab1252b7d3dcb563d1a8e1 and https://github.com/mono/skia/commit/47de7965be076311e824589b0464a4e1e08d54eb