Investigate a potential opinionated package management workflow

The Python extension in VS Code is investigating ways to improve our package management support tailored towards beginners with the option for extensibility for your preferred package manager tools via extensions.
Goals
Propose and implement an opinionated workflow as part of the Python extension for VS Code to:

Support beginners who are working with Python packages and virtual environments.
Streamline the package management story with the environment creation story.
Allow third-party package management extensions to integrate into the workflow to provide their own support

Proposal:
An opinionated workflow (ie. commands) that will integrate with the Python: Create Environment command in order to streamline environment creation and package management in the Python extension.
CLI command coverage we want to match in our opinionated workflow:

Create/update a dependency file
Read, resolve, and install dependencies from a file
Update dependencies
Remove a package
Add a package (and update the dependency file)
Create environment ✅
Run code ✅

Where and how to document dependencies:
Default should be to write to a requirements.txt file, including the package version that was installed. Opting to write the version, by default, supports the users' need for reproducibility and ease of collaborating on work. Given the nature of the work done by the target audience and our target usage, a pyproject.toml is out of scope as the target user typically does not build their project nor is there a direct way to only install dependencies listed in a pyproject.toml file.

Will there be a way for other extensions to use this facility? Maybe just through a command or something like that with looser coupling would be fine.
We'd need to :

Know that the current environment is supported (or opinionated I guess)
Send 'install this package' commands to the environment

VSCode command + cli tool similar to the code cli tool me thinks would be ideal.

Will there be a way for other extensions to use this facility? Maybe just through a command or something like that with looser coupling would be fine.
If I am understanding your comment correctly, yes, we have discussed the idea that we would allow other package management extensions to plug into the opinionated workflow but use their own commands. Need more investigation to see how this could be implemented.

Courtney, I'm the target user as described in this issue and in your blog post.  Usually I use conda, unless I have to use pip CLI.
My most frustrating experience is resolving package errors in an environment.  Sometimes, the environment says that it has inconsistencies and I can use conda doctor to troubleshoot which packages are inconsistent.  Other times, importing a package causes an error because that package has its own requirements.txt which conflicts with my environment.  This error can occur when that package hasn't updated (to newer) the versions of its dependencies in its requirements.txt.  But which dependency is causing the error isn't easily identifiable from the error's traceback.
P.S. conda's command to create a dependency file should be: conda env export > environment.yml

Current issue that I've encountered in VS Code about a package error https://github.com/microsoft/vscode-jupyter/issues/14993.  (I'm trying to update numexpr, but I'm unsuccessful and conda doesn't show any error.)

related https://github.com/microsoft/vscode-python/issues/21334


Breaking change: CA2247 - Argument passed to TaskCompletionSource constructor should be TaskCreationOptions enum instead of TaskContinuationOptions enum

Breaking change: CA2247 - Argument passed to TaskCompletionSource constructor should be TaskCreationOptions enum instead of TaskContinuationOptions enum
This new Roslyn analyzer will emit a warning if the user constructs a System.Threading.Tasks.TaskCompletionSource with a System.Threading.Tasks.TaskContinuationOptions enum value rather than a System.Threading.Tasks.TaskCreationOptions enum value.
Depending on the project's settings, the warning could show up as a build break.
Version introduced
Microsoft.CodeAnalysis.FxCopAnalyzers 3.3.0-beta1.final
Old behavior
The user could, without warning, construct a System.Threading.Tasks.TaskCompletionSource with a System.Threading.Tasks.TaskContinuationOptions enum value rather than a System.Threading.Tasks.TaskCreationOptions enum value.
New behavior
The user will now see the CA2247 warning suggesting to replace the System.Threading.Tasks.TaskContinuationOptions enum value with the corresponding System.Threading.Tasks.TaskCreationOptions enum value.
Reason for change
The TaskCompletionSource type has a constructor that accepts a System.Threading.Tasks.TaskCreationOptions enum value, and another constructor that accepts an Object. Accidentally passing a System.Threading.Tasks.TaskContinuationOptions enum value instead of a System.Threading.Tasks.TaskCreationOptions enum value will result in calling the Object-based constructor: it will compile and run, but it will not have the intended behavior.
Recommended action
Replace the System.Threading.Tasks.TaskContinuationOptions enum value with the corresponding System.Threading.Tasks.TaskCreationOptions enum value. It is not recommended to suppress this warning, since this rule almost always highlights a bug in the calling code.
Refer to the documentation for rule CA2247 for a detailed explanation on when to expect this rule violation to be diagnosed and how/when to suppress the warnings.
https://docs.microsoft.com/en-us/visualstudio/code-quality/ca2247
Category

Core .NET libraries

Affected APIs
Constructing a System.Threading.Tasks.TaskCompletionSource<TResult> with a with a System.Threading.Tasks.TaskContinuationOptions enum value.

Issue metadata

Issue type: breaking-change

cc @stephentoub @GrabYourPitchforks @jeffhandley

LGTM


Mock PA2 testcase 5_1

anyone knows what could be the error here?

Hey guys, i tried to reduce it in my calculations, all good thanks!


Change account prefix

Revert to rkt: prefix when https://github.com/iqlusioninc/relayer/issues/43 is fixed

I hope it is fixed, are you facing any issues?


integer value for option 'CFG_INT' is out of range

Hi all,
encountered weird behavior for parameter defined as CFG_INT. If I set to a value =>0x80000000 on 32bit system. Libconfuse (3.3) gives me conf.conf:17: integer value for option 'parameter' is out of range. What I am doing wrong that integer value is not valid?
Thanks,
Mike

CFG_INT is a signed integer. On a 32-bit system that means the maximum value is 0x7fffffff<PHONE_NUMBER>).
See #11 for more information.

thank you for your answer. So, it turns out to be hex radix error; if I set -2147483648, it gives me 0x80000000. 0x80000000 is valid signed 32bit integer.

Yeah, we use strtol() to convert the input value. The syntax 0nnnnn, 0xnnnn, 0bnnnn, or nnnn is only syntactic sugar to establish the radix argument to that function.  when you input 0x80000000 try to convert a postive hexadecimal value that is greater than the datatype can store.  The value 0x7fffffff is the largest positive value you can store.


Fix variable name in git_file_hash() error path

Also add a test case that hits this error path.

@StefanKarpinski not sure what other error paths should be tested like this, but I actually ran into this case while testing out a Base.BufferStream replacement.  ;)


"ReferenceError: exports is not defined" with webpack 2.2.1

I'm submitting a bug report
Webpack version:
2.2.1
Webpack Karma version:
2.0.2
Karma version:
1.5.0
Please tell us about your environment:
OSX 10.11.6
Browser:
Firefox 51.0.1 (64-bit)
Current behavior:
Firefox 51.0.0 (Mac OS X 10.11.0) ERROR
  ReferenceError: exports is not defined
  at test/test_index.coffee:22489
Firefox 51.0.0 (Mac OS X 10.11.0): Executed 0 of 0 ERROR (0.761 secs / 0 secs)

Expected/desired behavior:
Firefox 51.0.0 (Mac OS X 10.11.0): Executed 2688 of 2688 SUCCESS

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem along with a gist/jsbin of your webpack configuration.

Steps to reproduce:
$ NODE_ENV=test karma start

package.json:
https://gist.github.com/TaylorBriggs/1a37e3078e6c552f78d49d57838661a9
Karma config:
https://gist.github.com/TaylorBriggs/e4b395c07f9da76dd6adc343dff5020e

What is the expected behavior?
The tests should run as usual before updating to webpack 2.2.1 and karma-webpack 2.0.2

Got same problem

Tracked it down to a wepack 2 issue: https://github.com/webpack/webpack/issues/2631
Looks like I need to stop using add-module-exports and refactor to require('x').default.


Missing dependences in bundle when using plugins for loading main file

I am using my own plugin to perform conversion from typescript1.6 to babel to javascript.
Unfortunately when i am trying to bundle this using jspm bundle some deps that are used by typescript1.6 and some other module i am using in my program are not properly included in generated bundle.
I prepared small example that can be used to reproduce this issue, first by executing this In shell
$ jspm init
$ jspm install underscore
$ mkdir app
$ echo "U = require('underscore');exports.translate=function(){}" > plugin.js
$ echo "import _ from 'underscore';_" > app/main.js
$ cp app/main.js app/main2.js

and then adding this to config.js
  "packages": {
    "app": {
      "meta": {
        "main.js": {
          "loader": "plugin.js"
        }
      }
    }
  }

This only happens when plugin using file is used as first item in jspm bundle, so
jspm bundle app/main will generate broken bundle while jspm bundle app/main2 + app/main will make good one.

When transpiling TypeScript make sure to use the system module format output.


How to make a delay/gap time/pause in continuous scan

Hi, i really need the continuous scan, but at the same time i don't want it instantly scan every bar code in front of the camera. Because we need to have a sec or more to move the bar code.
In my case i make an increment every time a same bar code is scanned, because this its always reach 10 scan per 2-3 secs.
I already read some discussion in other discussion thread. Like check system.currentTimeMillis and compare it to a variable and then do something with if else condition, etc. Which is not working at all.
Do anyone have a suggestion or opinion?
Regards and Thanks

The easiest option would be to just filter out results if it is too close to the previous one. The ContinuousCaptureActivity sample already has code to discard the result if it's the same as the previous scan. All you need to do is add another check to compare the time since the last scan.

I did this by this code in BarcodeCallback's barcodeResult

            GlobalScope.launch(Dispatchers.Main) {
                barcodeView.barcodeView.stopDecoding()
                delay(100)
                toggleScan()
            }

private fun toggleScan() {
      barcodeView.decodeContinuous(callback)
  }

I did this by this code in BarcodeCallback's barcodeResult

            GlobalScope.launch(Dispatchers.Main) {
                delay(100)
                toggleScan()
            }

private fun toggleScan() {
      decoratedBarcodeView.decodeSingle(callback)
  }

Whats a GlobalScope in this and can you give a piece of code for callback


soda diff : compare the results of two scans

Intended use:
To see the effect of changes made to a data model, I would like to compare two scans. One of the two scans could be the latest scan made for a specific data model (which are stored in the Soda cloud).
This workflow would be interesting when combining it with dbt and a PR trigger:

Someone makes changes to a data model, in a feature branch
This person opens a PR
We run the data models for that specific branch
Compare the difference between the models with the main branch using soda diff.
Supply feedback about the effect of the changes (in a PR comment).

Describe the solution you'd like
A diff method that compares two scans.
Additional context
Add any other context or screenshots about the feature request here.

Curious about this one. How would you compare two scans? What is the criteria and how are they going to be interpreted?
The workflow looks good! I do think it is a needed feature, if we can get what we want to compare exactly

What to compare is indeed a good question, a left that open since I do not know exactly know what we would like to see. Some ideas:

dataset metrics: count (only print when changed?)
schema changes:

number of columns
new/removed columns
data type changes?

column metrics (only show when changed?)
custom metrics (only show when changed?)
difference in tests:

summary: number of failures/successes

In dbt we could "simply" change the schema, so if we implement #485 changing the schema should be more straight forward.
Another question is where do we keep the scan results. Would the workflow be something like:
$ soda scan warehouse.yml tables/my_table.yml > pre_changes_scan_result.yml
$ # Make change to your data model
$ soda scan warehouse.yml tables/my_table.yml > post_changes_scan_result.yml
$ soda diff pre_changes_scan_result.yml post_change_scan_result.yml
Left:               Right
7 columns      8 columns
....


feat: implemented structured logging

Codecov Report

Merging #54 (ad176d6) into main (eb5bb27) will decrease coverage by 1.73%.
The diff coverage is 86.36%.

@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   92.33%   90.59%   -1.74%
==========================================
  Files           5        6       +1
  Lines         274      351      +77
==========================================
+ Hits          253      318      +65
- Misses         20       32      +12
  Partials        1        1

Impacted Files
Coverage Δ

pkg/openfeature/openfeature.go
76.08% <36.36%> (-12.49%)
:arrow_down:

pkg/openfeature/logger.go
66.66% <66.66%> (ø)

pkg/openfeature/client.go
96.61% <95.77%> (-0.55%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM, minor suggestions.
One thing I'm wondering is how to log in hooks. It's not immediately obvious to me if that's possible.

That would be something for the application author to decide, there's nothing stopping them from placing a logger inside their hook struct and using it in the callbacks.


Could not find a "psql" to execute

I've installed daemontools on Centos 7 x64. Is this a problem with path ?
[/]# envdir /etc/wal-e.d/env wal-e backup-push %p
wal_e.main   INFO     MSG: starting WAL-E
DETAIL: The subcommand is "backup-push".
STRUCTURED: time=2016-04-20T15:28:54.572057-00 pid=22865
could not find a "psql" to execute
psql: FATAL:  role "root" does not exist
wal_e.main   ERROR    MSG: could not run one or more external programs WAL-E depends upon
DETAIL: Could not get a connection to the database: note that superuser access
is required
Could not run the following programs, are they installed? psql, pv
STRUCTURED: time=2016-04-20T15:28:54.763567-00 pid=22865

Hi, I'm getting the same message, so i'm assuming start and stop backup commands will not work..
`

wal-e version
0.9.1
PostgreSQL 9.4.7 on x86_64-unknown-linux-gnu, compiled by gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-4), 64-bit
CentOS Linux release 7.1.1503 (Core)

wal-e --aws-instance-profile --s3-prefix=s3://backup backup-push /db/postgres/5432/data
wal_e.main   INFO     MSG: starting WAL-E
DETAIL: The subcommand is "backup-push".
STRUCTURED: time=2016-04-20T19:48:56.868954-00 pid=20882
could not find a "psql" to execute
could not find a "psql" to execute
could not find a "psql" to execute
wal_e.operator.backup INFO     MSG: start upload postgres version metadata
DETAIL: Uploading to s3://backup/basebackups_005/base_000000020000000000000019_00000040/extended_version.txt.
STRUCTURED: time=2016-04-20T19:48:58.744628-00 pid=20882
wal_e.operator.backup INFO     MSG: postgres version metadata upload complete
STRUCTURED: time=2016-04-20T19:48:59.723766-00 pid=20882
wal_e.worker.upload INFO     MSG: beginning volume compression
DETAIL: Building volume 0.
STRUCTURED: time=2016-04-20T19:48:59.803028-00 pid=20882
wal_e.worker.upload INFO     MSG: begin uploading a base backup volume
DETAIL: Uploading to "s3://backup/basebackups_005/base_000000020000000000000019_00000040/tar_partitions/part_00000000.tar.lzo".
STRUCTURED: time=2016-04-20T19:49:00.332021-00 pid=20882
wal_e.worker.upload INFO     MSG: finish uploading a base backup volume
DETAIL: Uploading to "s3://backup/basebackups_005/base_000000020000000000000019_00000040/tar_partitions/part_00000000.tar.lzo" complete at 6978.42KiB/s.
STRUCTURED: time=2016-04-20T19:49:01.995380-00 pid=20882
could not find a "psql" to execute
NOTICE:  pg_stop_backup complete, all required WAL segments have been archived`

I am getting the same issue. I have installed wall-e on my ec2 instance to backup my db files to S3. I am not able to understand what is the issue and how to resolve it.
Any help will be appreciated.
Below is the error I am getting
envdir /etc/wal-e.d/env wal-e backup-push /home/
wal_e.main   INFO     MSG: starting WAL-E
DETAIL: The subcommand is "backup-push".
STRUCTURED: time=2016-04-21T15:42:19.372926-00 pid=11198
wal_e.main   ERROR    MSG: could not run one or more external programs WAL-E depends upon
DETAIL: Could not run the following programs, are they installed? psql
STRUCTURED: time=2016-04-21T15:42:19.401187-00 pid=11198

I just reverting to wal-e 0.9.0, which made the errors go away.
pip install --upgrade 'git+https://github.com/wal-e/wal-e@583ef4a4a89b7ffe204cb3722c25a658dfc19898'

I think this issue relates to https://github.com/wal-e/wal-e/issues/234  - recent commit broke wal-e for CentOS 7 users.

I went ahead and applied this. Can I get have a tester?

sure - I can help test. What would you like me to do?
(I assumed it was in 0.9.1 already)

Sure, download and install the master branch and see if it fixes your problem.

I think these problems have been resolved in #234. Sorry about that bug, everyone.


WFLY-17910 Deprecate org.wildfly.clustering.web.annotation.Immutable for removal

https://issues.redhat.com/browse/WFLY-17910

@pferraro Should we do this? :-)

@rhusar / @pferraro Can we get a decision on if we should do this?  We have a couple of days to get the PR merged but would be good to have the decision so we know if this really needs to be on the list.

Hey @darranl, we have uncovered a problem with usage of this annotation where it doesn't work exactly as we have documented. So, let me close this until we fix it.


Parallel upload support

🗒 Summary
This branch incorporates the use of the joblib library to parallelize the ingress requests and transfers to S3 on the list of requested files. The number of concurrent "threads" to utilizes can be controlled with a new --num-threads command line option.
⚙ Test Data and/or Report
No unit tests have been affected or updated by this branch.
Parallel uploads have been tested in NGAP using a 4 core machine. Running with different settings for --num-threads produces the expected improvement in runtime (i.e. single thread -> 1 minute, 2 threads -> 30 s, 4 threads -> 15 s).
♻ Related Issues

Resolves #24

can you verify that the pinned versions in setup.cfg look alright so I can get this branch merged

@collinss-jpl appreciate the ping! The earlier notification must've gotten lost.


chore: ASP.NET Core 3

@joaopgrassi It's running in staging. Looks good so far.
I can't get the build to pass though.
It's funny that API builds fine, but Worker doesn't. And yet on the config yml file there's only some **/.Api.csproj so I'm confused how this build is configured


Log In Account

Is there any way to log in with an spotify account easily? I mean for example clicking a button and redirecting the user to the spotify log in or something like that?

I will take a look at this this week.
for future issues, if you have to paste long code, I suggest using the  toggle:

Expand

Bullet
Points
and stuff

Expand

* Bullet
* Points
* and stuff

moreover, you could also just reference the file (or mention the file name), if it is in this library. No need to paste the whole file.
And finally, you can use code highlighting with
```dart
<your code here>
```

I am not sure, how localhost is available on mobile devices. I think you have to give spotify a real responseUrl (e.g. your github profile page).
(I am not 100% sure of this, though. Maybe @rinukkusu knows more about this)

Just to have more inputs about this, does that mean you guys are thinking of adding support to other login flows available in the spotify sdk?

@ahmedo100 since spotify is not supporting any other login flows other than the one mentioned in their documentation, I would say, no.

This maybe a hack but what you could do is to include the spotify android and ios sdk into your flutter app and call the android login activity (for android) from your flutter app. Since you are then getting some sort of client secret from spotify, you could bind it in your flutter app.
I haven't tried this and I don't know if that is even possible, that is the first thing that came into my mind when I lookend into the android sdk documentation. If anyone knows more, that would be great.

Since we cannot help any further, I would like to close this issue @addreeh .


VSCode Not Invoked When Creating GitHub Codespace

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

VS Code Version: 1.81.0-insider (Universal)
OS Version: macOS 13.5

Steps to Reproduce:

Create GitHub Codespace
I see "Launching Visual Studio Code" message

@conradwt, thank you very much for filling this issue and apologies for not getting back to you on this until now. Based on the provided information, this seems to be an issue with GitHub Codespace so could you please file an issue in the Codespaces issue tracker. Thank you!


Digital Clock  #205

I have completed the Digital Clock website along with a readme file and website pic.
#205
Hope it gets merged.
Thank you.

@LiQuiD-404 The screenshot is already there in the readme section of this project. please take a look again.
Hope it gets merged now.
Thank you.


⚠ Fortnite has degraded performance

In 2f077ba, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1922 ms

Resolved: Fortnite performance has improved in 09c5e2f.


Not Working: (Mecanim) Humanoid import settings for rigged .glb model file

I am using UnityGLTF 2.12.0 for Importing .glb model files in Unity 2022.3.23f1
I've got several model files:
1.) One, which is rigged & skinned with the mecanim humanoid bone hierarchy (Hips > Ribs > Chest > etc ...)
2.) Another, which is rigged & skinned with meta avatar bone hierarchy (Spine.001 > Spine.002 > Spine.003 > etc ...)
If I export those files as .fbx and import it in Unity (editor, not runtime), I can choose "Humanoid" rig configuration and may need to retarget some spine bones - but both files are getting retargeted really well.
If I export those files as .glb and import it in Unity (editor, not runtime), I can choose "Mecanim Humanoid" and it always fails, stating that the "Hips"-bone is missing [It is not].
Did I miss a step configuring/exporting in Blender, in the Unity Editor or am I missing an extension?
As stated in the changelogs for version 2.11.0: "- add: support for humanoid import for Meta avatar bone hierarchies" - there should be a working solution for that.
Thank you guys in advance and for your work so far. I will appreciate any advice given!

Sorry, found out myself:
The ErrorMsg is a bit misleading, it states that the Bone named „Hips“ is not found - when the importer itself actually has a lookup table, with different naming conventions:
hips -> Hips
Hips_jnt -> Hips
mixamorig:Hips -> Hips
but never actually looks for „Hips“ with a capital Letter „H“.
To be sure your bone hierarchy is readable for the importer, take a look at „AvatarUtils.cs“ which has the dictionary „HumanSkeletonNames“ with different naming conventions.

hey @Phlegmati, can you please test your scenario with this comment? That should be fix it :)

Nice, good to hear :)


What is the intended use case for CPieceTable::IsPieceInvisible()?

My current understanding of CPieceTable::IsPieceInvisible() and CPlayBoardView::DoToolTipHitProcessing() is that a piece whose small tile color is the transparent color will not report who owns the piece in the tool tip or status bar.  Am I interpreting the code correctly?  If so, what is the intended use case for this?

I dig back into my aging memory cells to address this after I finish the PR for the Geomorphic stuff.
-Dale


unify rebuild_local_checksums method

redefinition of rebuild local checksums is reduced to redefinition of "get_remotes".
as discussed in #2127

@larshp please review : -)

@larshp: can we merge this? There are more and more conflicts to be resolved with every subsequent incoming change I need to integrate into this one...

something happend, yea, but I dont think conflicts is a big problem, only due to the way the changes for file src/zcl_abapgit_repo_online.clas.abap looks right now, let me try out a few things

the changes to repo_online should look somewhat like,
https://github.com/abapGit/abapGit/commit/800b80e2d129e699f1cb279a3b8c4d3bb0760de8

currently, it is

I'll go ahead and overwrite the file in the source branch

@larshp OK. I'm not sure what happened there, I just rebased the branch on abapGit/master a few times. Should I not have done that? I confess that I'm a bit new to this way of working, unfortunately, doing mostly ABAP or solo work, it looks like I got confused.

some of the changes might have reordered the methods, and git was successful pulling the upstream changes into the branch, which caused method duplication
Anyhow, I don't really understand why the remote status has something to do with the local checksums, but its outside of this PR scope => #3997
merging, thanks 👍


New ktlint rules

EnumEntryNameCase
NoEmptyFirstLineInMethodBlock

Both are from the ktlint experimental rule set so not enabled by default.

It's a little strange, verifyGeneratorOutput is failing but doesn't report what the errors actually are, so not sure how I can resolve it. Runs clean on my machine.

Codecov Report

Merging #2243 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2243      +/-   ##
============================================
+ Coverage     81.38%   81.42%   +0.04%
- Complexity     2074     2081       +7
============================================
  Files           343      345       +2
  Lines          5984     5993       +9
  Branches       1086     1086
============================================
+ Hits           4870     4880      +10
  Misses          532      532
+ Partials        582      581       -1

Impacted Files
Coverage Δ
Complexity Δ

...rmatting/wrappers/NoEmptyFirstLineInMethodBlock.kt
100% <100%> (ø)
2 <2> (?)

...ch/detekt/formatting/wrappers/EnumEntryNameCase.kt
100% <100%> (ø)
2 <2> (?)

...ab/arturbosch/detekt/formatting/KtLintMultiRule.kt
87.75% <100%> (+0.52%)
6 <0> (ø)
:arrow_down:

...turbosch/detekt/rules/style/UnusedPrivateMember.kt
93.47% <0%> (ø)
5% <0%> (ø)
:arrow_down:

...osch/detekt/rules/bugs/WrongEqualsTypeParameter.kt
100% <0%> (ø)
10% <0%> (+3%)
:arrow_up:

.../gitlab/arturbosch/detekt/rules/MethodSignature.kt
70% <0%> (+10%)
0% <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 5dfe7c5...f22d5e7. Read the comment docs.


Add time command

The time command is an easier way for users to create relative timestamps. The command allows you to say "/time amount: 2 unit: hours" to display a timestamp that says "in 2 hours". You can also change the way it is displayed to show a timestamp, datestamp, or datetimestamp.

The lang properties cleanup included in this closes #655.


Fix argument type generation with typed method binds

Fix a mistake in #59793 which causes the crash mentioned in https://github.com/godotengine/godot/pull/59793#issuecomment-1086620159

Thanks!


More loot tracker stuff (skilling-related)

Pickpocketing, fishing, mlm, etc. Anything with variable yield, really. (i.e. H.A.M. members, fishing spots that yield multiple types of fish, mlm giving different ores, and so on.)
It'd be nice for collecting approximate drop rates without bugging jmods on twitter.
Farming would be neat but would probably be better suited to have its own plugin to track yield, as it would require extra work for tracking how many you get per patch (and you'd probably want a custom display).

Duplicate of #4126


The material of "Cesium3DTileset" is missing

// Cesium.VERSION = 1.98

const viewer = new Cesium.Viewer("root", {
    imageryProvider: new Cesium.SingleTileImageryProvider({
        url: "./Cesium/Assets/Textures/globe_warm.jpg",
    }),
});

const scene = viewer.scene;
const tileset = scene.primitives.add(new Cesium.Cesium3DTileset({
    url: "http://<IP_ADDRESS>/api/t-gis/bim-api/assets/" + "G52692538c5fd4afebccb752db47af6c8" + "/tileset.json?tenantId=c2af3f3ce08349a79811359c231de88e",
}));
viewer.flyTo(tileset)

But it is displayed normally in 1.86 and below.

Hi @sunvictor, thanks for the report.
I'm having trouble accessing the tileset at the url you provided in the example above. Is it possible to provide us with the data?
Also to confirm your 3D Tiles data is valid, you could also try running it through the 3d-tiles-validator and let us know the results.
Based on the changes that went out with 1.87 nothing in particular jumps out to me as something that would break this behavior.

I'm closing this issue due to inactivity. If you believe this is still an issue, please feel free to re-open. Thanks!


LD_PRELOAD Failure on Debian Stretch (9.4)

I have compiled the code using clang on debian 9.4 as per instructions in README. But I am getting error in LD_PRELOAD
Error
$ LD_PRELOAD=libmesh.so git status
ERROR: ld.so: object 'libmesh.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
clang++-3.9 --version
clang version 3.9.1-9 (tags/RELEASE_391/rc2)
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin
uname -a
Linux dal2mdspkgm02.corporate.local 4.9.0-6-amd64 #1 SMP Debian 4.9.88-1+deb9u1 (2018-05-07) x86_64 GNU/Linux

oh no, sorry for the trouble!  and you ran sudo make install after make on the line before that?

Sorry, I missed make install.  I got more focussed on getting it compiled across diff versions of gcc/clang and in the melee lost this important thing. Thanks for pointing it.

ok glad it was that easy!
@raman-gupta-tivo were you having troubles with either GCC/clang, or just looking to compare things?

When I first ran the command configure && make then I ran into compilation issues. On further investigation I came across similar issue (#16) raised by another user. After trying clang3.9 compiler and code changes mentioned in that issue I was able to build the shared object.
One more thing: will using mesh memory allocator also improve performance of python applications, since python interpreter is a C program? I understand that python uses its own memory allocator and performs garbage collection, but for experimenting I tried LD_PRELOAD=libmesh.so python my_python_script.py, but could not see any speed improvement.

oh thanks, if thats the only issue with supporting clang3.9, might as well just change the atomic declaration there.
I haven't benchmarked python recently, but the default allocation scheme they use is very fast; I wouldn't expect speed improvements.  Also, in python2 it isn't configurable.  I would try LD_PRELOAD=libmesh.so PYTHONMALLOC=malloc python3 my_python_script.py, and if you have issues lets open a new issue to look into python3 perf!
Thanks for the feedback!

this should be fixed after #34, closing.


add analytics tracking for make offer and view offer buttons

The type of this PR is: Chore

This PR resolves PURCHASE-2492
Description

This adds analytics tracking for the Make Offer and View Offer buttons.
PR Checklist (tick all before merging)

[ ] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a CHANGELOG.yml entry or my changes do not require one.

Should I be checking in Podfile.lock?

Should I be checking in Podfile.lock?

I haven't been, but I'm curious too. I keep needing to run pod install --repo-update for the app to build. @MounirDhahri do you know what the best-practice is for this?

It's fine if you check in Podfile.lock.

Since we are making sure that everyone is using the same version of cocoapods using the Gemfile we know for sure that it's going to change only when necessary.

I do however recommend spending some time to just test if whatever got updated did not break something in the app, just like updating any regular npm package.

You could as well create a PR with only the pod changes like I did here https://github.com/artsy/eigen/pull/4547

Oh, I see. Thanks for sharing the best-practice, Mounir 🙌

You could as well create a PR with only the pod changes like I did here #4547

Ahh this is good to know. Thanks all.
@erikdstock you okay with merging this?


An API to the get cached value synchronously

Sometimes we want to be able to read the cached value from the store synchronously.
For example, on Ktor's default request as requested in this feature
interface Database {
  suspend fun readUsername(): String?
  val cachedUsername: String?
}

defaultRequest {
  database.readUsername()?.let { header("X-User-ID", it) } // wont compile
  database.cachedUsername?.let { header("X-User-ID", it) }
}

Added in 0.3.0


sdk/StackReference: Add outputDetailsAsync

Adds the Java equivalent of StackReference.GetOutputDetails
that was added to:

Go SDK in pulumi/pulumi#12034
Python SDK in pulumi/pulumi#12071
Node SDK in pulumi/pulumi#12072
.NET SDK in pulumi/pulumi-dotnet#103

In Java, this adds a new StackReference.outputDetailsAsync method
which returns a CompletableFuture of a StackReferenceOutputDetails.
The object has optional value and secretValue fields,
at most one of which is set depending on whether the value is a secret.
Neither field is set if the value is absent.
Refs pulumi/pulumi#10839, pulumi/pulumi#5035

Current dependencies on/for this PR:

main

PR #986   👈

This comment was auto-generated by Graphite.


Slackin is installed already, but...

Hi, I'm new in Nodejs world. I installed slackin on my server and it's running now.

But I don´t now yet how can I create the page to invite my users. How can I create the Realtime badge, SVG and Langing page.
Can someone help me? Thanks.

@julian-nunezm you can add the realtime badge and invite by loading the script into your HTML document. Once the script loads, it will insert its own iframe and viola, you have your button!

So I just have a basic html made up:

And when the page loads - all I get is white.  help?

You need the ip/hostname of your slackin server not your subdomain.slack.com


Looking for a mantainer

Hello everyone,
I'm not using Meteor anymore, also I'm not updating this project.
Is someone willing to take the lead on this project?

ping @theodorDiaconu, (https://github.com/cult-of-coders/meteor-apollo-accounts)

Thank you for consideration, however we do not have bandwidth. Great package @nicolaslopezj very elegant approach of blending meteor's accounts with GraphQL

You should post this on meteor forums, if it is not done already. MDG could even take this one in, imho. Good job @nicolaslopezj


EventEmitter on Models

Sails version:0.12.4
Node version:6.4.0
NPM version:3.10.3
Operating system:OSX 10.12.1

Hello, first of all, I love sails. It's a great MVC framework for node, I would like to know it would be a good thing to add EventEmitter to Models, I think it would be an enhancement for watching Models inside the app, not restricted to sockets.
The only way to listen for events on a Model, you can use the current API for websockets
http://sailsjs.org/documentation/reference/web-sockets/resourceful-pub-sub
Is there any way I can listen for model events from anywhere else, a hook for example?

ok, fixed!

@agustinkassis my experience has been that using EventEmitters in app-level code built on Node.js can sometimes lead to maintainability issues.  But you can use lifecycle callbacks to achieve this effect.  Hope that helps!

I would use lifecycle callbacks to emit the event, precisely eventEmitter is better for maintainability as it provides loose coupling, there should be a way to listen for model events from hooks as not related code shouldn't be embedded inside a lifecycle method.


When long pressing text to copy text, you cannot drag the slider

Microsoft Windows [版本 10.0.26100.2454]
(c) Microsoft Corporation。保留所有权利。
D:\Program Files\scrcpy>adb connect <IP_ADDRESS>:5555
already connected to <IP_ADDRESS>:5555
D:\Program Files\scrcpy>scrcpy -s <IP_ADDRESS>:5555 -m1920 --max-fps=25 --no-audio -S -w
scrcpy 3.0 https://github.com/Genymobile/scrcpy
INFO: ADB device found:
INFO:         (tcpip)  <IP_ADDRESS>:5555             device  NX595J
INFO:     --> (tcpip)  <IP_ADDRESS>:5555             device  NX712J
INFO:           (usb)  adb-D1231381046D-Zbib0p._adb-tls-connect._tcp            device  NX712J
D:\Program Files\scrcpy\scrcpy-server: 1 file pushed, 0 skipped. 47.6 MB/s (90224 bytes in 0.002s)
[server] INFO: Device: [nubia] nubia NX712J (Android 14)
INFO: Renderer: direct3d
INFO: Texture: 864x1920
[server] INFO: Device display turned off

screenshot

screenshot

screenshot


Setting RigidBody2D.sleeping=true doesn't invoke sleeping_state_changed()

Godot version: not so stable 3.2.1
Issue description: closely related to #39366.
Setting RigidBody2D.sleeping property to true (just haven't tried with false) doesn't invoke sleeping_state_changed() signal callback. The callback sometimes invoked though, when engine sets the node to a woken state.
Minimal reproduction project: use MRP of #39366.
Make sure cbasen.gd contains
func _ready()->void:
	connect("sleeping_state_changed",self,"_on_sleeping_state")

func _on_sleeping_state():
	print("%s %s" % [get_name(), "SL" if sleeping else "w"])

and main.gd
func _ready():
	$Node2D.position.x+=1
	call_deferred("sleepefy")

func sleepefy()->void:
	print("sleepefy n1, but NOT n2")
	$Node2D/n1.sleeping=true

Run and observe that _on_sleeping_state() is NOT invoked after sleepefy() executes $Node2D/n1.sleeping=true.

Guys... Please stop calling the builds stable. This is important!
I have never-ever-ever seen in my life and 20+ years programming career software, that uses so big version numbers, labels itself as "stable" and still so full of various bugs and glitches! I consider myself a "quite" experienced C++ developer (something reflected in my profile), but I'm having to really struggle with making Godot work. I hardly imagine emotions of those who are less experienced and are stuck with numerous Godots issues... Just don't lie to your customers, that it's a stable thing. If it fails not too often, it doesn't make it deserving stable badge. Early pre-alpha is suitable much better. That doesn't make anybody love the project less, they will just know in advance what to expect.

This is by design. If you want to your manual change to send a signal then you must use the emit_signal function.

I think this might just be a documentation issue.
The current documentation says that sleeping_state_changed is emitted when the sleeping property changes, but that's not true. It's emitted when the sleeping state of the RigidBody changed in PhysicsDirectBodyState2D and the sleeping property is thus updated by the physics engine.
In this MRP, the body is sleeping at the start. The sleeping state is set to true from script, which is the same as the previous value, and since the physics state is not being updated due to the chosen sleeping status, there's no sleeping_state_changed to propagate.
I'd suggest making the signal's documentation clearer (both for 2D and 3D rigid bodies).

Errr... Well... Why not to update the docs then?
It really confuses when you read that "Emitted when sleeping changes." and it turns out, that it is actually not at all true.

@akien-mga Remi, thanks! You're right. The issue is in the docs then...

I may try to make an update to the docs, if some help needed...Need to find the docs contribution guide first...

@Arech It's here: https://docs.godotengine.org/en/latest/community/contributing/updating_the_class_reference.html

Made the pull..
Also, I'd suggest to append these rephrased @akien-mga words to a cut-out note somewhere in https://docs.godotengine.org/en/latest/getting_started/step_by_step/signals.html

Signals in Godot are emitted by the engine when it does something noteworthy internally. Signals are not raised to notify you of your own action - you don't need a callback to react to you setting some object's property, as you can call a callback manually or emit your own signal in that same code block.

That would be really helpful to know in advance for a rookie. But I don't get where are the sources of that part of docs located.

@Arech The documentation pages are located in this repository: https://github.com/godotengine/godot-docs
Specifically, you'll want to modify getting_started/step_by_step/signals.rst and open a pull request there.

@Calinou thank you for your help!


Baton management

To avoid multiple people trying to write to the beamline at the same time we should have a baton. It probably makes sense for this baton to be in dodal so that anyone using it obeys the baton
Acceptance Criteria

There is a discussion about how to handle this in dodal
There is some documentation on what the solution looks like

this is not a problem in blueapi as far as I am aware.
can we define the scenario in more concrete terms? Is it about different RE instances?

The main thing we're trying to avoid is GDA and Hyperion trying to use the beamline at the same time. Therefore there needs to be an external baton that each application looks at to tell them if they're in charge or not.

we could build a low level lock-and-baton system with redis, or in an etc-d key.
third party solutions include:

https://developer.hashicorp.com/consul/docs/dynamic-app-config/sessions - (https://github.com/hashicorp/consul)
https://zookeeper.apache.org/doc/r3.1.2/recipes.html (https://github.com/apache/zookeeper)

@tpoliaw  I searched on slack for messages with the baton keyword and thought you  may have suggestions - also amygdala might need to coordinate to make this work


Ensure Final Newline in .json File Generation Process

Summary
The current process responsible for generating .json files is not adding a final newline at the end of the file. This missing newline can cause issues with certain tools that rely on this newline for correct parsing. To prevent potential problems, we need to update the generation process to include a final newline in the .json files it generates.
Tasks

[ ] Investigate the current implementation of the .json file generation process
[ ] Modify the process to append a final newline character at the end of the generated .json files
[ ] Test the updated process to ensure the newline is correctly added and does not impact the existing functionality
[ ] Document the changes made to the .json file generation process

Acceptance Criteria

.json files generated by the process should include a final newline character at the end of the file
The updated process should not introduce any regressions or impact existing functionality
Documentation should be updated to reflect the changes made to the .json file generation process

@JolanThomassin could you add to your PR ai-cfia/ailab-db#96 a newline to the json files generated by the script ? Add this to the GitHub issue description closes ai-cfia/api-test#13


Consul watcher never emits 'error' event if connection to consul is lost

Hello. I'm using consul client for service discovery.  My code looks like:
const consulCli = require('consul');
const Promise = require('bluebird');

const client = consulCli({
  host,
  port,
  promisify: Promise.fromCallback
});

const options = {
  service: 'my-service-name'
};

client
      .watch({
        method: client.catalog.service.nodes,
        options,
        wait: '2m'
      })
      .on('change', (data) => {
        console.log('CHANGE');
      })
      .on('error', (err) => {
        console.log('ERROR');
      });

I run this code locally on my machine and consul is running on remote machine. When network connection is up, changes coming (it logs CHANGE). But when I shutdown my network connection consul client never steps into neither change event, nor in error event.
Such behavior only appear when network disconnect happens after establishing consul client connection (line const client = consulCli({).
Steps 2 reproduce:

Connect to consul via consul client library
Create watcher via client
Wait some time
Disconnect from network
See result - no error event emitted

Also when I enable connection again consul client never receive nether change nor error event. It looks like it's getting stuck somewhere inside request.
If consul host is already down papi lib throws timeout exception and it's handled well.
UPD: Seems it happens not always. Also specifying defaults with timeout seems resolves this issue (but I think that some timeout should be set by default to value passed in wait + some seconds.

Sorry for not getting this fixed earlier, and thanks for the detailed issue report.

I don't know if this is related, but I see a similar problem with kv.get(). I'm wondering if the problem is in get and got fixed in the wrong spot.

@SkeLLLa  You don't happen to know what version of consul this issue was introduced do you? Asking for a friend.


beacon integration : Maid-952

Changes Unknown when pulling 30b4a5e332943bfe55a6e27790f249e450e6f1cd on mmoadeli:MAID-592-test into ** on dirvine:master**.

Reviewed files:

examples/beacon_client.rs @ r1
examples/beacon_server.rs @ r1
src/beacon.rs @ r1
src/connection_manager.rs @ r1
src/transport.rs @ r1

Comments from the review on Reviewable.io


Add language_hints support to detect_full_text.

Fixes: #3124
Towards:  #3132.
Support language_hints argument for detect_full_text().

It appears this has broken the build, but it ACTUALLY appears that there is a broken version of grpcio==1.20 (published today)?

:(


Input Boolean cannot be used as heater in climate.generic_thermostat

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version): 0.40
Python release (python3 --version): 3.4.2
Component/platform:
Climate.generic_thermostat
Description of problem:
I would like to use an input_boolean as my heater but the climate.generic_thermostat component will not switch the input Boolean on or off as demand for heating changes. I assume this is because the component is calling the switch.turn_on service rather than homeassistant.turn_on
Expected:
Input_boolean is switched on and off by thermostat
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

Suggest that you try to change the code to use homeassistant.turn_on. If that works, you should make pull request.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum


change to examples

This PR:

adds/changes/removes etc

Testing
Description on how {APP-NAME} can be tested.

[ ] fresh install works
[ ] upgrade from previous version works

Other testing
Description of features to additionally test for {APP-NAME} installations.

[ ] check reconciliation of existing resources after upgrading
[ ] X still works after upgrade
[ ] Y is installed correctly

Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.
[ ] Update /examples if required.

I just ran into this issue so maybe we should have included how to set:
  extraConfigs:
  - kind: secret
    name: container-registries-configuration
    namespace: giantswarm
    priority: 25

But this is extremely giantswarm specific.

@kopiczko  You ran into what issue?

Docker limits

docker limit addressed in readme https://github.com/giantswarm/cluster-cloud-director/pull/119/commits/d2bb7649592b03e17570a276a536b7cdbb77904b


[Meru][New Pricing] Amplitude tracking

Description
Adding Amplitude tracking for the new Pro plan.
Implements: https://github.com/dust-tt/dust/issues/4317
Added

PageViewed event for the /w/wid/subscribe page (paywall page).
SubscriptionCreated event
FairUsageDialogViewed event
Make sure we flush the event ClickedEnterpriseContactUs before redirecting to the Google form cc @clmrn

cc @gabhubert @Yutcam

Risk

Deploy Plan

PMRW.

Concerns about taxonomy discussed here: https://dust4ai.slack.com/archives/C050SM8NSPK/p1710924049171739?thread_ts=1710921883.189469&cid=C050SM8NSPK


Separate requests

I have two different methods with
@NeedsPermission({Manifest.permission.CAMERA, Manifest.permission.WRITE_EXTERNAL_STORAGE})
for pick avatar and background.
And I want in @OnShowRationale method show different dialogs for avatar and background explanation.
How can I do this?

@3383389 then we recommend separating method per each permission. We just wrapped requestPermissions of support library.

for example I have
@NeedsPermission({Manifest.permission.CAMERA, Manifest.permission.WRITE_EXTERNAL_STORAGE})     public void getAvatar() {         ...     } @NeedsPermission({Manifest.permission.CAMERA, Manifest.permission.WRITE_EXTERNAL_STORAGE})     public void getCover() {         ...     } @ OnShowRationale({Manifest.permission.CAMERA, Manifest.permission.WRITE_EXTERNAL_STORAGE})     void showAvatarDialog() {         ...     }
and I can't separate OnShowRationale methods because permissions are the same.
Maybe we can use something like RequestCode in default permissions?

I meant separating method per each permission but I guess you can use v4's PermissionChecker or our light-weight wrapper!

Thank you, I'll try


User page

I'm trying to fix some issues of the repositories.
I fixed user page but I have 2 alternatives.
Which one you are going to use?

Hi, what are the URLs of the pages? I thought there was only meant to be one user page, with different content depending on what profile fields are filled out.

On 26 Jun 2016, at 23:07, chack1172<EMAIL_ADDRESS>wrote:
I'm trying to fix some issues of the repositories.
I fixed user page but I have 2 alternatives.
Which one you are going to use?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

euantorano the url is only one:
mysite.com/public/index.php/user/1/Admin
So the correct page is the second image?

Yes, the second one looks right to me. The two are similar, the only difference is that empty fields are hidden in the second image.

There are some issues with the styling though. The registration date should all be on one line, same with visit date. Paging @justinsoltesz!

Ok, I have another questions.
If I go to Admin profile I see the two profile fields because I set them in the database.
If I go to another user (e.g. Registered) I see the two fields empty, why they aren't hidden?

They should be hidden, there's obviously something missing in the query. This method probably needs to exclude NULL profile fields: https://github.com/mybb/mybb2/blob/master/app/Database/Repositories/Eloquent/ProfileFieldRepository.php#L66
I'll have a look tonight after work.

euantorano I only changed function hasForProfileField() in app/Database/Repositories/Eloquent/ because $user is not passes/empty/null.

	public function hasForProfileField(User $user=NULL, ProfileField $profileField)
	{
        if($user === NULL)
            return $this->userProfileField->where('profile_field_id', $profileField->id)
                ->count() > 0;
        else
            return $this->userProfileField->where('user_id', $user->getId())
                ->where('profile_field_id', $profileField->id)
                ->count() > 0;
	}

Yep, that would also do it. $user is null for guests (which will be changed soon).

Why is it null for guests? It shouldn't contain user data we are looking at?

Oops, it shouldn't be null actually. I was thinking $user was the currently logged in user. It should never be null for that method. if it is, there's another problem somewhere.
I'm still at work right now so it's difficult for me to debug until I get home unfortunately.


Incorrect CRTM AMSU-A Spectral Coeff file on hera

This is a similar issue to that already addressed in Issue 963
The AMSU-A Metop-C spectral coefficient file needs special treatment as the one supplied with the default file name is not correct.   The details on what to do are in that issue.
The correct file has the following cksum:
1237040622 12196 amsua_metop-c.SpcCoeff.bin

This one is from orion at /work/noaa/epic/role-epic/spack-stack/orion/spack-stack-1.6.0/envs/unified-env/install/intel/2022.0.2/crtm-fix-<IP_ADDRESS>_emc-ezbyzji/fix
On Hera at /scratch1/NCEPDEV/nems/role.epic/spack-stack/spack-stack-1.6.0/envs/unified-env-rocky8/install/intel/2021.5.0/crtm-fix-<IP_ADDRESS>_emc-bm46d3q/fix (which is the path given by $CRTM_FIX when the appropriate modules are loaded as specified in the develop version of GSI) the cksum is:
1692416252 1252 amsua_metop-c.SpcCoeff.bin

This causes a failure when trying to run the develop version of GSI.

@DavidHuber-NOAA Could you take a look at this if you have time?

Hercules check
Log into Hercules.  Load GSI modulefiles/gsi_hercules.intel and cd $CRTM_FIX.  The correct amsua_metop-c.SpcCoeff.bin is installed on Hercules
hercules-login-4:/work/noaa/epic/role-epic/spack-stack/hercules/spack-stack-1.6.0/envs/unified-env/install/intel/2021.9.0/crtm-fix-<IP_ADDRESS>_emc-2os2hw2/fix$ cksum amsua_metop-c.SpcCoeff.bin
1237040622 12196 amsua_metop-c.SpcCoeff.bin

Thank you @ADCollard for reporting this problem.  This issue prompted me to check GSI regression tests.  The global_4denvar test sets ta2tb=.false.,  This should be changed to ta2tb=.true. since this is how operations runs the global GSI.

@AlexanderRichert-NOAA @climbfuji @ADCollard I verified this does need to be fixed.  The procedure outlined in #963 should work fine again here.  This also needs to be fixed on Jet Rocky 8 installations.

@RatkoVasic-NOAA Can you work with @DavidHuber-NOAA to fix this, please?

@DavidHuber-NOAA does this mean we just have to copy amsua_metop-c.SpcCoeff.bin to all our installations? If yes, which one we know is correct? And I can use that one to populate to all environments/versions.

@RatkoVasic-NOAA This only applies to spack-stack v1.6.0 Rocky 8 installations on Hera and Jet and only needs to be applied to the unified-env-rocky8 environment (the gsi-addon-rocky8 uses the unified-env-rocky8 installation of crtm-fix).
A good copy of the fix file can be found here:
/scratch1/NCEPDEV/nems/role.epic/spack-stack/spack-stack-1.5.1/envs/unified-env-rocky8/install/intel/2021.5.0/crtm-fix-2.4.0_emc-sh2qz24/fix/amsua_metop-c.SpcCoeff.bin

@DavidHuber-NOAA thanks! I'm going to update it.

@DavidHuber-NOAA BTW, I checked other files as well. Beside amsua_metop-c.SpcCoeff.bin, there are two more files that differ:
atms_j2.SpcCoeff.bin
atms_n21.SpcCoeff.bin
Do you know if that is expected?

@RatkoVasic-NOAA I'm not certain on this.
@ADCollard Do you know if these changes are expected between crtm-fix v2.4.0 and v<IP_ADDRESS>?

@RatkoVasic-NOAA @DavidHuber-NOAA Thanks for doing this!   This whole sorry saga started with atms_n21!  Hopefully for <IP_ADDRESS> the cksums should be as follows:
andrew.collard@dlogin06:/apps/ops/para/libs/intel/<IP_ADDRESS>4/crtm/<IP_ADDRESS>/fix> cksum atms_n21.SpcCoeff.bin
3709800805 1960 atms_n21.SpcCoeff.bin
andrew.collard@dlogin06:/apps/ops/para/libs/intel/<IP_ADDRESS>4/crtm/<IP_ADDRESS>/fix> cksum atms_j2.SpcCoeff.bin
4188410122 1960 atms_j2.SpcCoeff.bin

@ADCollard thanks, it is correct in spack-stack-1.6.0/crtm-fix-<IP_ADDRESS>

The checksums and sizes are both correct now:
1237040622 12196 /lfs4/HFIP/hfv3gfs/role.epic/spack-stack/spack-stack-1.6.0/envs/unified-env-rocky8/install/intel/2021.5.0/crtm-fix-<IP_ADDRESS>_emc-z2uz6da/fix/amsua_metop-c.SpcCoeff.bin

1237040622 12196 /scratch1/NCEPDEV/nems/role.epic/spack-stack/spack-stack-1.6.0/envs/unified-env-rocky8/install/intel/2021.5.0/crtm-fix-<IP_ADDRESS>_emc-bm46d3q/fix/amsua_metop-c.SpcCoeff.bin

I think this can now be closed.  Thanks @RatkoVasic-NOAA!

Thank you both!


custom parameter in answer url for voice api

Hi Team,
I am using Nexmo for sending sms and text-to-speech voice call.
everything is working fine but I need to add custom parameter in answer url json for voice api.
please check below code:
$call = $client->calls()->create([
'to' => [[
'type' => 'phone',
'number' => '919619666397'
]],
'from' => [
'type' => 'phone',
'number' => '919619666397'
],
'answer_url' => ['https://www.efundbook.com/efb_nexmo/interest.json'],
]);
In above code, there static json file in answer_url.
Any time I execute the code it says same speech on voice call.
I need to change speech dynamically.
can  I use php file for 'answer_url' that will echo custom json out put. so that my voice api will say hello with customer name.
basically i need to change json text dynamically .
Please help me to get this done, thanks in advance.

Hi @ahmedhashmi21,
You can use a PHP script as your answer_url to customise the response. So long as the URL called returns JSON, Nexmo does not mind how the response is generated.
Unfortunately, there is no way to replace answer_url with JSON, but that's a great suggestion that I'll put to our product team!
Hope that helps

Hi Michael,
Thanks for providing resolution on this. I have tried php script for answer_url but it didn't work for me.
Could you please provide php script example so that I can use the same.
Thanks

Hi Michael,
Thanks for providing code. but i have only last issue, when I execute code it send blank call it is reading file.
Please check my code.
$call = $client->calls()->create([
'to' => [[
'type' => 'phone',
'number' => '919619666397'
]],
'from' => [
'type' => 'phone',
'number' => '919619666397'
],
'answer_url' => ['https://www.efundbook.com/efb_nexmo/dynamic_json.php'],
]);
dynamic_json.php
 'talk',
    'voiceName' => 'Russell',
    'text' => 'Hi, This is Russel.'
  ]
]);
?>
If above code works then I am done with it. please revert on this I would really appreciate it.
Thanks

It looks as though you're using the same number for to and from. The from number must be a number that you own on Nexmo.
Your answer_url looks good to me though!

Okay, but same code is working when I replace answer_url with json file.
As you said, I have changed from number but still I am getting blank call.

Ah, sorry @ahmedhashmi21. Nexmo require a file with the content type application/json, and you're serving up text/html. Try this:
<?php

header('Content-Type: application/json');

echo json_encode([
  [
    'action' => 'talk',
    'voiceName' => 'Russell',
    'text' => 'Hi, This is Russell'
  ]
]);```

Hi Michael,
Thank you very much, Its working for me. really appreciate it!!!

No problem @ahmedhashmi21, thanks for using Nexmo and please let us know if you have any other issues.

hii can you tell me how to make  dynamic ncco in asp.net

Code: To send Text to speech
string NEXMO_CALL_UUID = Session["UUID"].ToString();
string TEXT = "Hello I am Kimberly.This is a text to speech sample for voice alert message using nexmo";
        var result = Client.Call.BeginTalk(NEXMO_CALL_UUID, new Call.TalkCommand
        {
            text = TEXT,
            voice_name = "Kimberly"
        });

I have successfully done an outbound call but when i try to synthesized text to speech along with the call and  whenever  i execute this,result give me an exception "value can not be null.Parameter name is value.".
give me some suggestion


docs: update README.md

woking -> working

Interesting but not planned :(

What about changing enforce working to detonating your pagers ?

Oops I relized that the word "woking" stops will make CI fail


prepare a FEDRAMP env variable for use

for OSD-0249

the lint failure is because the fedramp variable isn't being used yet

/lgtm


Tests for automatic column adjustment feature

Thanks for creating this.
Tip: it's nicer to get smaller, focused PRs. Like,

the dictionary changes could be a PR on their own
moving the function about the test file could have been another PR
adding the new tests could have been a third PR.

That said, these are all small; and I'm taking them anyway :)
bonus: Doing smaller, focused CLs will also help you use git bisect (which is very helpful).

Heh, CL is for Change Request. PR is for Pull Request. They mean essentially the same thing. Sorry that I sometimes use one or the other. "pull" is a git/github thing. Like the generic category of thing that "git" is, is a Revision Control System. The generic name for a pull request is a change request. In my normal work, the term CL is used. When on github, I try to use PR, but sometimes I forget and write CL.
The "prefer small changes" is also encouraged at my work. I didn't make particularly small changes in the past, but now I do and I'm seeing the wisdom in it. For some (the initial paper I read on the topic) the advantage is in reviewing the change - keeping it small is more efficient for someone reading/approving the change.
To me, the big win is bisecting the code to see where something broke for the purpose of fixing it. If git bisect narrows it down to a small change, it's much easier to work out the bug and fix it. Conversely, if git bisects down to a pile of changes that all happened together, it's less clear what caused the bug being investigated.
In places I've worked that were concerned with having too many small changes, they were optimizing for the log of the change history. So if that's the priority you'd want to do the opposite (e.g. combine changes together). I don't optimize for reading the history, but it's just a choice.
Hmm, I think I should write up this choice so that others will know. I should make a style guide too.

Oh that's good to know. Having a style guide and contribution guide would be pretty nice as well! But... where is the L in "Change request"?

Oh my, I wrote that too early in the morning. CL is Change List.
For the commits being grouped together. I've been using the "Squash and Merge" button (which is the default) to bring in PRs. It looks like there's also an option to bring in the changes as separate changes (without squashing them).
Overall, this may just be a thread of me mistyping things and pressing the wrong buttons to pull changes. Thank you for bearing with me.

Oh I see. :+1: Thanks!


IOS keyboard slow or not responding simulator IOS 16.0

Details

IOS keyboard slow or not responding when typing in IOS simulator  IOS 16.0, High CPU usage also found.

Target Platform: IOS
Target OS version/browser: IOS 16.0
Devices:iPhone 14 Pro Max Simulator
Code to reproduce:
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(

        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {

      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {

    return Scaffold(
      appBar: AppBar(

        title: Text(widget.title),
      ),
      body: Center(

        child: Column(

          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const TextField(
              decoration: InputDecoration(
                border: OutlineInputBorder(),
                labelText: 'Password',
              ),
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

Logs

Logs

flutter analyze
Analyzing flutter_application_1...
No issues found! (ran in 3.7s)

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.3, on macOS 12.6 21G115 darwin-x64, locale en-IN)
    • Flutter version 3.3.3 on channel stable at /Users/xxx/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18a827f393 (4 days ago), 2022-09-28 10:03:14 -0700
    • Engine revision 5c984c26eb
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/xxx/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /Users/xxx/Library/Android/sdk
    • Java binary at: /Applications/Android Studio Preview.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[!] Android Studio
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (1 available)
    • macOS (desktop) • macos • darwin-x64 • macOS 12.6 21G115 darwin-x64

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

No issues with Android

@vinukn335 I tested this thing on iPhone, but I didn't find it slow or not responding behavior as yours.
Could you provide a recorded video for this? Is there any special log from the console when entering the keyboard?

https://user-images.githubusercontent.com/47185599/193616396-382bcdf5-4e62-440f-a116-36d4a78398f1.mp4
@huycozy I did not tested in real iphone, I am facing this issue in iPhone 14 Pro Max simulator.

@vinukn335 I tested on the same simulator too, but still can not see the issue.

Demo
https://user-images.githubusercontent.com/104349824/193727066-1818d5b3-4b4f-4ee8-b051-e4d80e9cf20e.mov

Could you check macOS's Activity Monitor app to see if the emulator take too much memory or CPU at that time?
Also, does the application work well without TextField?

https://user-images.githubusercontent.com/47185599/193857394-479ade8b-2531-4858-b0fa-ad97fb34b0c3.mov
This application works well with or without TextField,  but when trying to type with IOS soft keyboard, feels slowness or hang.

@vinukn335 thanks for providing a recorded video. Please help to clarify something below: (sorry for missing these questions above)

Does this issue happen on other IOS simulators (different iOS versions)?
Could you also try replicating this issue on some OS's native apps (such as Messages or Reminders)?

This issue happen in all ios simulators. No issues in Messages or Reminders
https://user-images.githubusercontent.com/47185599/194001598-35cf9b12-50f9-4197-9414-1725532127f0.mov

This issue is so weird that I can't reproduce it or find a way to dig dive into it. I will label this issue for further insights from the team.

Any chance your disk is nearing full?  Do you have a pending macOS update?

@jmagman no MacOS update pending. This Mac is not M1/M2  based, Intel Based processor.

for me this bug is not just in ios keyboard, but everytime you trying to input in text field in flutter app. this bug happened since ios 16, now i'm using flutter 3.3.4 and still having this issue

@huycozy have you tried with ios 16, cz this issue is only happend in ios 16, just run you ios 16 simulator, such as iphone 14, and try to input in textfield in your flutter app
@jmagman i'm sure its not related to disk, its just a bug with flutter in ios 16, i'm running a mac monterey 12.6 with 16 gb ram, my NVME SSDE still have 70+ GB empty space. once again this issue is only happen in ios 16

@noobcoder95 yes, please see the demo at https://github.com/flutter/flutter/issues/112762#issuecomment-1266343304. I ran on iPhone 14 Pro Max Simulator, iOS 16.0. And I'm also using a Mac Intel chip, macOS 12.6 (same as OP)

Works fine on my machine:
[!] Flutter (Channel main, 3.5.0-6.0.pre.26, on macOS 12.6 21G115 darwin-x64, locale en-US)
...
    • iPhone 14 Pro Max (mobile) • E23C2E6D-6F51-48DD-B81D-516B3671AB08 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-1 (simulator)

https://user-images.githubusercontent.com/682784/195957196-f80b7ef0-186e-4422-a624-fe39cf5596a9.mov
Does this reproduce on a non-Flutter, vanilla iOS app?  What if you delete and recreate the simulator?

@jmagman please see my previous post, there is no keyboard issue with non-flutter ios apps. Same issue with recreating simulator.

We also encountered this problem(#113823), is there a solution?
Devices:
iPhone Xs Max  iOS15.6.1
iPhone X iOS13.6
The keyboard freeze is the same as in this video：
https://user-images.githubusercontent.com/13282734/200276770-1d2d305b-aaaf-435b-9b1a-596a05b61121.mp4

Can you attach an Instruments time profile recording?  A recording of activity monitor isn't really actionable, and none of us could reproduce it...


fix(iroh-net)!: improve magicsock's shutdown story

Description

Waits for connections to actually be closed by calling quinn::Endpoint::wait_idle after close.
Consumes the MagicEndpoint in an effort to express the fact that closing an endpoint is a terminal operation.
Adds logging to closing the connections in case it's deemed necessary
Changes MagicSock::poll_recv to return Poll::Pending on close instead of an error. More info on the logic behind this change below.
Changes MagicSock::poll_send to not error if nothing is being sent.

Breaking Changes
MagicEndpoint::close now consumes the endpoint.
Notes & open questions
Shutdown MagicSock when quinn is done with it has proven something we can't do reliably.

Drop based solutions (close when the MagicSock is dropped) are unreliable because the socket is held by multiple components on quinn, including spawned detached tasks. There is no really any way to trigger the dropping of these tasks/components from outside quinn, and would require quite en effort to change it to happen inside quinn.
Close based solutions were rejected. The objective here was to stop polling the socket when the endpoint was both closed (close called) and all connections gracefully finalized. The reasoning here is that quinn both receives and sends frames after close to read new connection attempts and gracefully reject them. This is a fair argument on their side, despite it being clearly a limitation for a reliable freeing of resources from quinn.
Taking into account the fact that the socket will be polled after closed, both to send and receive, returning an error from these will always produce the logging error quinn::endpoint: I/O error: connection closed we have been chasing, even after the quinn::Endpoint has been dropped. Therefore changing poll_recv to return Poll::Pending addresses this annoyance.
Note that the part about gracefully shutting down is actually done by calling quinn::Endpoing::wait_idle and that the (now averted) log error in quinn doesn't really tells us whether shutdown was or not gracefull.
Note that the above point creates an API disparity between poll_send and poll_recv: one will error on close, the other will return Pending. I find this OK if MagicSock is simply a part of our implementation, but right now it's also part of our public API. I wonder if the MagicSock makes sense to users on its own or if we should remove it from the public API.
NOTE that conversation with quinn's maintainers to get a better api that allows to understand when all resources as freed has started. But this is playing the long game and we need to solve this on our end for now.

Change checklist

[x] Self-review.
[x] Documentation updates if relevant.
[ ] ~Tests if relevant.~
[x] All breaking changes documented.

* Drop based solutions (close when the `MagicSock` is dropped) are unreliable because the socket is held by multiple components on `quinn`, including spawned detached tasks. There is no really any way to trigger the dropping of these tasks/components from outside `quinn`, and would require quite en effort to change it to happen inside `quinn`.

I haven't gone through all the pain of trying to implement this, so might have this wrong.  Could you explain why the dropping of MagicSock by Quinn is not a suitable signal that it is done with it and can really be closed?  I still don't fully grasp this, apologies.  Is this mostly around the difficulty of achieving that drop inside of our MagicEndpoint while the MagicEndpoint itself is not yet dropped?
The most recent comment at https://github.com/quinn-rs/quinn/pull/1828#issuecomment-2073594181 also seems to think that dropping of the AsyncUdpSocket is a signal that all resources are released.
Also I think once Quinn drops the AsyncUdpSocket you can also be sure there's no more clone of the socket lingering around in some cloned connection, endpoint or whatever internal tasks.

Signal on drop has two problems mainly:

MagicEndpoint: Clone which makes for this horrible footgun:

let ep1 = MagicEndpoint::new();
let ep2 = ep1.clone();
ep2.close().await // this is a deadlock

Keeping track of clones is not easy so even in the presence of a big warning to users, I find such an API honestly unacceptable. It's not like a function xyz must be called before or should run inside a tokio runtime which are much more easily verifiable. It would be a you can only call this after dropping all clones of MagicEndpoint2. Clearly when theAsyncUdpSocketis dropped.. well it's dropped. The thing is that we can't reliably ensure this happened even if we drop thequinn::Endpoint, because it's the last thing to go. In fact, take a code such as this (where the MagicSockhad no changes topoll_*`
let msock = MagicSock::new();
let quinn_ep = quinn::Endpoint::new_with_abstract_socket(msock.clone());
drop(quinn_ep)
msock.close().await; // here the endpoint has been dropped but more likely than not, the `AsyncUdpSocket` **hasn't** been dropped yet. So more often than not you will _still_ get that error bubbled up as the quinn error log.

The main problem is for sure the first one and it makes it IMO simply not worth it going such a convoluted path when a far simpler solutions suffices

@divagant-martian thanks for the clear summary on the problems of drop!


Can we explicitly state that MDAV should not be disabled and must be in passive mode as a pre-req?

[Enter feedback here]
It seems that we don't explicitly state that you cannot have MDAV disabled in order for this to work (or will it still work if you completely disable MDAV?  I have doubts but do not know for sure).
Many customers with 3rd party AV explicitly disable MDAV despite the fact that there is passive mode in W10.

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

ID: b8da58e5-9cd8-8ccf-75a9-327c14ebaa76
Version Independent ID: 24a29218-59e1-7a1d-c8c7-e61424b5f9ee
Content: Endpoint detection and response in block mode - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/edr-in-block-mode.md
Product: w10
Technology: windows
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@vboyev-MSFT - I think we have covered this part in document Q&A section here at https://docs.microsoft.com/en-us/windows/security/threat-protection/microsoft-defender-atp/edr-in-block-mode#will-edr-in-block-mode-have-any-impact-on-a-users-antivirus-protection
So the statement "No. EDR in block mode does not affect third-party antivirus protection running on users' devices. EDR in block mode kicks in if the primary antivirus solution misses something, or if there is a post-breach detection. EDR in block mode works just like Microsoft Defender Antivirus in passive mode, with the additional steps of blocking and remediating malicious artifacts or behaviors that are detected." says the same thing in different wording.
Thanks
Imran.

@joinimran - Why are the requirements for EDR in Block mode to successfully function buried in the FAQ?
"Ensure MDAV is not disabled by policy and that cloud based protection is enabled" is a clear, direct statement that has very little room for interpretation - why would we want to state that in ambiguous indirect manner to leave doubt in a customer's mind about precisely what is required for this feature to work?.


Rule set instructions outdated

The following message displays on the rule set group page now
"The cache expiration, cache key query string and origin group override actions are deprecated. These deprecated actions can still work normally, but your rule set cannot be changed. You need to replace them with new route configuration override action before changing your rule set."

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

ID: 6090e37b-c633-0542-888e-eb3e81e73c75
Version Independent ID: 6e940c90-4d54-bd2b-b01b-2267c071542a
Content: Tutorial: Manually configure Azure Front Door for Azure Static Web Apps
Content Source: articles/static-web-apps/front-door-manual.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

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

@mmyers299 We have gone ahead and added a note in the doc about this recent change until the doc author can update the documention.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Removed boost/mpl methods in favour of new methods on openvdb::TypeList

Signed-off-by: Nick Avramoussis<EMAIL_ADDRESS>One important consideration in this PR is compile times - my implementation probably differs from boost so this may have been impacted (could have been improved as well as impaired). Need to test.

Very cool stuff, nice one! I like the readability of the new methods much better than the old ones.
Would be good to get the TestTypes::testTypeList() unit test updated to test this new functionality, otherwise I'm happy with this.

I've added a set of static asserts to TestTypes::testTypeList() to test these new methods

This is great. In case you weren't aware, the Houdini plugin is failing at the moment and will likely need updating to use this (see the Azure job).

In case you weren't aware, the Houdini plugin is failing at the moment and will likely need updating to use this (see the Azure job).

Oops thanks, will fix this

I'd quite like @kmuseth to look over this change before we merge it in.

I won't merge any of these boost PRs until we've got consensus in the tsc

I'll review it today!

@kmuseth I've added documentation to all the type internal meta-programming structures - I think I went a bit overboard, I added doxygen documentation but didn't realise that there exists a @cond OPENVDB_TYPES_INTERNAL doxygen rule which disabled this documentation generation by default. Oh well, it's there should we ever want it in the future!
Regarding the comment on compile times, this was something I wanted to test and hadn't yet. I didn't anticipate any adverse effects but I wanted to make sure that this solution performed at least as fast as the boost solution. I've tested with gcc and am getting similar results (across 32 threads, building the .so, the binaries and the tests, obviously no ccache):
before: real 4m52.941s
after: real 4m53.310s

great, thanks Nick


glsl-to-spv-worker assumes there is no vertex shader present

In time the worker should be adapted to be able to pass a vertex shader through to runspv.

Obsolete.


[DNM] v2.9 related version bumps

What does this PR do?
Bumps dependencies for v2.9 which are followed by rancher/rancher and rancher/shepherd
Which issue(s) this PR fixes (optional, in fixes #(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #103
Checklist:

[x] Squashed commits into logical changes
[x] Documentation
[x] GitHub Actions:
AKS - P0, SupportMatrix
EKS - P0, SupportMatrix
GKE - P0, SupportMatrix

Special notes for your reviewer:

Will create branch release/v2.8 out of current main, before merge.
Separate issue for failing jobs: https://github.com/rancher/hosted-providers-e2e/issues/117

The PR looks fine to me, but can we branch out to release-v2.8 before merging this?

The PR looks fine to me, but can we branch out to release-v2.8 before merging this?

I have created a new branch called release-v2.8.

The PR looks fine to me, but can we branch out to release-v2.8 before merging this?

I have created a new branch called release-v2.8.

nit: Branch name - release/v2.8

The PR looks fine to me, but can we branch out to release-v2.8 before merging this?

I have created a new branch called release-v2.8.

nit: Branch name - release/v2.8

Done. Also, once this is done we might need to delete the scheduled jobs from release/v2.8.


Request to change configuration of hydra r-updates jobset

I'd like to request a change to the configuration of the r-updates jobset (https://hydra.nixos.org/jobset/nixpkgs/r-updates) to alter the nixpkgs input to point at the r-updates branch in my fork (https://github.com/jbedo/nixpkgs.git) rather than peti's. As one of the current codeowners of r/rPackages I would like to be able to manage staging and testing of this tree.
Moving r-updates to a branch of nixpkgs rather than my fork, as is done with haskell-updates, would also be fine with me if that's more appropriate.

I'd say a branch in nixpkgs is preferable as it makes contributing, sharing the work among multiple people and changes in maintainers easier.
cc @grahamc @domenkozar who can make these kinds of changes.

It may also be a good idea to move r-nixpkgs.nix from peti/ci to nixpkgs as, say, pkgs/top-level/release-r.nix, so the jobset can be changed with less hassle.

We should expand the platforms to darwin & aarch64 too if there's capacity.


[W6.4h][T11-B3]Lim Zheng Kai

@limzk1994
Some comments added. Please close the PR after reading comments.


Link doesn't work

Steps to reproduce the behavior:

Go to https://github.com/florinpop17/app-ideas/blob/master/Projects/1-Beginner/Calculator-App.md
In section 'Example projects' click on 'Sample Calculator'

Screenshot

The github profile for sevlasnog does not appear to even exist. Should this link just be abolished and removed?

Hello everyone 👋
I hope this message finds you well. As you may know, the original repository for app ideas has not been updated in a while and is no longer actively maintained. However, I'm excited to let you know that a new repository has been created to continue this project (https://github.com/vincent0426/app-ideas).
The new repository aims to provide a comprehensive list of practical and achievable app ideas that developers of all levels can work on to improve their skills and build their portfolios. We welcome contributions from anyone who wants to add their own app ideas or improve the existing ones, and we are also looking for volunteers who can help maintain the repository and keep it up to date.
If you are interested in contributing or maintaining this project, please head over to the new repository (https://github.com/vincent0426/app-ideas) ⭐ and take a look at the existing issues and pull requests. Feel free to open a new issue if you have an app idea to suggest or if you have any questions about the project.
Thank you for your attention, and I hope to see you in the new repository soon! 💖

⚠ This message was generated by a robot 🤖, if this message reached you, consider leaving a star for this robot, repository link
⭐ https://github.com/aldairandre/github-bot ⭐

Hello everyone 👋
I hope this message finds you well. As you may know, the original repository for app ideas has not been updated in a while and is no longer actively maintained. However, I'm excited to let you know that a new repository has been created to continue this project (https://github.com/vincent0426/app-ideas).
The new repository aims to provide a comprehensive list of practical and achievable app ideas that developers of all levels can work on to improve their skills and build their portfolios. We welcome contributions from anyone who wants to add their own app ideas or improve the existing ones, and we are also looking for volunteers who can help maintain the repository and keep it up to date.
If you are interested in contributing or maintaining this project, please head over to the new repository (https://github.com/vincent0426/app-ideas) ⭐ and take a look at the existing issues and pull requests. Feel free to open a new issue if you have an app idea to suggest or if you have any questions about the project.
Thank you for your attention, and I hope to see you in the new repository soon! 💖

⚠ This message was generated by a robot 🤖, if this message reached you, consider leaving a star for this robot, repository link
⭐ https://github.com/aldairandre/github-bot ⭐


config: Make the data directory configurable

Adds a new category of config called "paths" which is intended to
allow the user to configure various paths. At the moment, it supports
one - the data directory. By setting this, you can control where image
files are stored without having to give a CLI argument.
This is based on #1192.
I will need it for configuring default stage1 directory.

I decided to postpone renaming the --dir flag to --data for now. This can be done in a separate PR.

TBD: Update the CHANGELOG, after the 0.12.0 is released.

LGTM

Rebased to master and updated the documentation.

needs to be rebased again.

Rebased.

LGTM


Prepend the inlined function name to inlined variables.

As title.
Fixes #7380
Fixes #7365

Thanks for the review @esc.


[WFLY-19888] Move to SE 17 as the minimum SE version

https://issues.redhat.com/browse/WFLY-19888

LGTM

Thanks @jasondlee and @jamezp

@bstansberry FYI the root README will also need an update as this specifies Java 11 as the minimum.


Prisma doesn't know which engines to download with Linux Mint

Bug description
prisma:warn Prisma doesn't know which engines to download for the Linux distro "org.freedesktop.platform". Falling back to Prisma engines built "debian".
Please report your experience by creating an issue at https://github.com/prisma/prisma/issues so we can add your distro to the list of known supported distros.
prisma:warn Prisma doesn't know which engines to download for the Linux distro "org.freedesktop.platform". Falling back to Prisma engines built "debian".
Please report your experience by creating an issue at https://github.com/prisma/prisma/issues so we can add your distro to the list of known supported distros.
This is just a prisma error, failing to detect my system.
For full system details check below. (doubt github will allow me to deposit all of this data)
https://termbin.com/lddw
How to reproduce

Expected behavior
No error occuring.
Prisma information
// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema
generator client {
provider = "prisma-client-js"
previewFeatures = ["jsonProtocol"]
}
datasource db {
provider = "postgresql"
url = env("POSTGRES_PRISMA_URL") // uses connection pooling
directUrl = env("POSTGRES_URL_NON_POOLING") // uses a direct connection
}
model Account {
id                 String  @id @default(cuid())
userId             String
type               String
provider           String
providerAccountId  String
refresh_token      String?  @db.Text
access_token       String?  @db.Text
expires_at         Int?
token_type         String?
scope              String?
id_token           String?  @db.Text
session_state      String?
user User @relation(fields: [userId], references: [id], onDelete: Cascade)
@@unique([provider, providerAccountId])
@@index([userId])
}
model Session {
id           String   @id @default(cuid())
sessionToken String   @unique
userId       String
expires      DateTime
user         User     @relation(fields: [userId], references: [id], onDelete: Cascade)
@@index([userId])
}
model User {
id            String    @id @default(cuid())
name          String?
email         String?   @unique
emailVerified DateTime?
image         String?
accounts      Account[]
sessions      Session[]
}
model VerificationToken {
identifier String
token      String   @unique
expires    DateTime
@@unique([identifier, token])
}
model auth {
id       Int       @id @default(autoincrement())
email    String    @unique @db.VarChar(255)
name     String?   @db.VarChar(255)
username String    @unique @db.VarChar(255)
password String?   @db.VarChar(255)
time     DateTime? @default(now()) @db.Timestamptz(6)
role     String?   @default("User") @db.VarChar(255)
}
model users {
id        Int      @id @default(autoincrement())
username  String?  @db.VarChar(255)
userid    String?  @db.VarChar(255)
banstatus Boolean? @default(false)
banreason String?  @db.VarChar(2550)
role      String?  @default("User") @db.VarChar(255)
}
Environment & setup

OS: Linux Mint 21.3 Virginia base: Ubuntu 22.04 jammy
Database: Postgres
Node.js version: 20.9.0

Prisma Version
4.16.2

Hi @Fr3akyMurk, this warning indicates that wasn't able to detect your Linux Mint distro as distro belonging to a "well-known" Linux family, and thus it fell back to downloading the Prisma Engines compiled for Debian.
Thankfully, the Debian-built engines should be completely compatible with your system, so you should be able to run Prisma without any problem, despite that warning (which we acknowledge).
Could you please report back whether the rest of Prisma functionality works well for your use case? Thanks

Hi @jkomyno , Sorry, i forgot to respond.
Yes, the Debian built-in engines work flawlessly, at least to my current knowledge.
I can report to you if something happens, this issue was opened mainly so you could see if an update can be pushed to prevent this error from appearing again; at least on mint distros. (it can get pretty flooded and i prefer error-less consoles for in case i need debug statements or error logs.)

I'll close this, as the fallback/default engine should work as expected, so only a report would be needed if there is an issue, let us know if that's the case.
Note that we changed the message, to make it informational only in https://github.com/prisma/prisma/pull/23243
The updated message will be part of the next release 5.11.0, next week.


Flaky e2e test: Addon update should propagate add-on file changes (Failed 3 times in the last 30 runs. Stability: 90 %)

Master tracking issue: #9312
Error message as follows (pretty uninformative - need to fix that):
/go/src/github.com/GoogleCloudPlatform/kubernetes/_output/dockerized/go/src/github.com/GoogleCloudPlatform/kubernetes/test/e2e/addon_update.go:321
Expected error:
<*errors.errorString | 0xc208029760>: {
s: "timed out waiting for the condition",
}
timed out waiting for the condition
not to have occurred

New test introduced 3 days ago in #9493.

@marekbiskup will you be able to look into this one?

I'm taking it now

@quinton-hoole , how can I find the failed runs of the test?

The error message is from the wait.Poll function. I'll create another wait.PollMsg that takes an error message to return something more meaningful that "error waiting for the condition".
e.g. "waiting for the condition: pod is successfully scheduled" .

#10171 improves the error messages from this test.
I have no idea where to find logs from the system from a failed execution. So it is not possible to investigate the error without reproducing it.
@quinton-hoole , are there any plans for gathering system logs (not test logs) from a failed execution of a ne2e test?
For now I'll just increase the timeout. Its in #10172.
In the meantime I'll try to reproduce the issue with a lower timeout.

@marekbiskup The logs are already being uploaded to Google Cloud Storage.  See the "Google Cloud Storage Upload Report" link in the left menu when you're looking at a failed Jenkins run.
I'm not sure that increasing the timeout is a good idea.  Do you have any data to suggest that it is a good solution?

For example, when this test passes, it always takes 32-35 seconds.  I don't think that increasing the timeout to 3 minutes will improve the success rate.  It will only make failures take a lot longer.

@quinton-hoole , you're right - I found a bug in production code.
It may happen that the addon-update script lists the add-on directory and finds a few files. Then it tries to create these add-ons. But they may disappear before they are used. The script is "robust", so it retries creating the file 100 times with 10 sec delay.
That's why we may have a 20-minute delay in creating an add-on in some cases.
What's worse, if the definition of one add-on is incorrect (and create will fail) - the add-on update script will be blocked for 20 minutes on each invocation.
I'm on a way to fixing it (as soon as I find a good way :)  )

Ok, my plan is to get rid of the retry loop - maybe retry 3 times with a 1s delay.
If something goes wrong and the objects are not created, they will be created in the next round - after 10 minutes.
I will also go back to 1min timeout in the test.

This test is still failing, approximately as often as it always was.

My apologies!  I was completely confused. It has not failed since the merge of #10237

https://pantheon.corp.google.com/storage/browser/kubernetes-jenkins/pr-logs/pull/22305/kubernetes-pull-build-test-e2e-gce/31209/?debugUI=CLOUD
http://kubekins.dls.corp.google.com:8081/job/kubernetes-pull-build-test-e2e-gce/31209/consoleFull
Looks like we just didn't create an addon service:
Addon update
/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/addon_update.go:323
  should propagate add-on file changes [It]
  /go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/addon_update.go:322

  Expected error:
      <*errors.errorString | 0xc208406ef0>: {
          s: "error waiting for service e2e-tests-addon-update-test-aefkd/addon-test-updated to appear: timed out waiting for the condition",
      }
      error waiting for service e2e-tests-addon-update-test-aefkd/addon-test-updated to appear: timed out waiting for the condition
  not to have occurred

@bprashanth does that close this issue?

I haven't investigated the last failure but it hasn't seemed to have occured since. I don't know who owns addon-update, but it's probably not team/cluster. I think this falls under control-plane, so I'll let them decide.

Please add team/test-infra (or assign back to me) if there is more for our team to do.

@davidopp to delegate

Hey @mikedanese how would you like to own addon updater? Currently implemented in https://github.com/kubernetes/kubernetes/blob/master/cluster/saltbase/salt/kube-addons/kube-addon-update.sh
but I assume your cluster deployment work will eventually sweep this up.

This seems not flakey anymore


add test for ocm_handle_root_policy_duration metric

ref:

https://issues.redhat.com/browse/ACM-2513

Signed-off-by: Will Kutler<EMAIL_ADDRESS>

@willkutler Would you please add the Jira issue URL to the commit message so that it can be tracked in stolostron?

@willkutler Would you please add the Jira issue URL to the commit message so that it can be tracked in stolostron?

Done!

@dhaiducek looks good to me! I just made those rewording changes and squashed


Windows list ports

Here's an implementation for Windows that builds on the already existent Linux implementation. Seems to work just fine with my limited experimentation.

@Susurrus You already have an open PR for adding serial port enumeration (#14). Please amend to that PR instead of opening new ones.


Fix General Accounting Unit CRLP Reset to 0 bug

We fixed an issue that could cause a single General Accounting Unit rollup record to have zeroed out totals
Critical Changes
Changes
Issues Closed
Fixes #6204
Community Ideas Delivered
Features Intended for Future Release
Features for Elevate Customers
New Metadata
Deleted Metadata

@force2b @andrewyu-salesforce Added unit tests.  Would appreciate your review.
@sam-knox No labels, but would appreciate your eye on the PR description.
Thanks all!!

@force2b @dfuller114 Thank you for the suggestions.  I've implemented your feedback and would appreciate another look.  Thanks!


Changes for Move exec operations from ContainerOperations

Hi teams
I saw #261 that was up for grabs and I jumped in.
Cheers
Giancarlo Lelli

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

Awesome thanks! LGTM


Refactoring ge 44 defining a form for displaying content

fix-44

fixes #44

fix #44

 (0)PR Refactoring ge 44 defining a form for displaying content #50


Fixed Issue #373 - Dragging Carousel Cards Sometimes Selects Text Instead

Created a PR that solved issue #373.

@rohansx Merging this PR Closes issue #373