is there any high end sample app to refer and build high end business application

i am checking for any high-end sample app like adding many scene objects to the main scene and the output should be like listview or recycler view functionality.
The scene objects added to parent scene should be scroll very smooth and fast without any memory leaks.
The provided sample apps are pretty simple and not much helpful while writing for high-end business.
Is there any possibility to add sample app which covers

the UI should be like recylerview or listview(many scene objects with vertical scrolls)
setting image to scene object from network url

Thanks

Hi @sams89 .  What you're asking for is certainly possible.  I believe EvryPlace: https://www.oculus.com/experiences/gear-vr/1739742929422865/ (which is a GearVRf app) accomplishes this.  But I assume you're looking for source code on how to do this.
The good news is that our friends at SIDIA in Brazil are working on a more comprehensive 360 video player sample app that should provide source and demonstrate what you're looking for.  But you'll have to be a bit patient, its release is still several weeks away.  @ragner please keep this issue/thread in mind as your team develops this app.  Thank you.

Yes, It could be helpful if there is a comprehensive app.

Hi @thomasflynn,
We will prioritize this working to do a pull request soon.

@ragner when we can expect the PR commit?

@ragner any update on this?
cc @thomasflynn , @liaxim

Hi all, we have submitted a PR with video player example. See https://github.com/gearvrf/GearVRf-Demos/pull/630 for more details.

@sams89 The video player has been merged. See https://github.com/gearvrf/GearVRf-Demos/tree/master/gvr-videoplayer/app. Planning to close the issue in few days if you don't have any other comments.

@sams89 Evryplace developer here ;). It is not that hard to apply high quality of android views right now. GearVRf traveled long road from first versions on which we worked. Playing videos (normal or 360) is also quite easy. Only thing that you will have to handle is hover events and or clicks. And be aware of on which thread you are launching events :). Good luck


AYANEO Air Pro compatibility issues

Did an install of the Deck version, goes to desktop, won't go to game mode. Also the desktop is rotated the wrong way, it's upside down.

Interesting...and that same quirks file works in Chimera. I wonder if UBlue has some additional routine to handle rotation and they're stepping on each other.
We had to set displays as "external" so that Steam would allow access to the scaling slider in DeckUI mode.

Interesting...and that same quirks file works in Chimera. I wonder if UBlue has some additional routine to handle rotation and they're stepping on each other.
We had to set displays as "external" so that Steam would allow access to the scaling slider in DeckUI mode.

Nothing on our end that wouldn't be part of upstream Fedora.
The only custom rotation anything in this whole project should be in game scope session, which is just coming from Chimera anyway.

Fixed in next release.


Why doesn't net point to mock/net.js?

When using the plugin with esbuild my bundled code throws the following error:
TypeError: Cannot read properties of null (reading 'createConnection')

The lines of code responsible for triggering the error are:
var net = require_empty();
...
ForeverAgent.prototype.createConnection = net.createConnection;

The README.md says that net has a mock implementation mock/net.js which makes the functions createConnection noops.
However this line sets net to  mock/empty.js i.e. null:
https://github.com/niksy/node-stdlib-browser/blob/3e7cd7f3d115ac5c4593b550e7d8c4a82a0d4ac4/index.js#L34

Every mock implementation should be explicitly defined in your build/bundler configuration.

@niksy can you tell me how?

@marijanp you can try with aliases.


[Bug]-[2400]:Text overflows on evaluated value pop-up on a query page

Is there an existing issue for this?

[X] I have searched the existing issues

Description
On a query page when the evaluated value displays a long continuous string like a URL, the text overflows on the pop-over as seen in the image below.
Steps To Reproduce

On any query page use a URL param like appsmith.URL.fullPath or Audio1.url and observe the evaluated value pop-up

Public Sample App
No response
Version
Cloud

say 1 in 5 users have experienced this
Stats

Stat
Values

Reach
2400

Effort (months)
<>

this is no longer reproducible after the new evaluated value pane improvements


max user problem in ubuntu

hi
iam using the softheter server on ubuntu using tunnel with cascade connection
when users reach 40-50 people the server giving error authetntacion error for some clients .
i dont use any external authentication server exept the softheter .
both servers CPU and RAM usage are normal and under 50% and this is not a server hardware error
i think i will increase some function somwhere but i dont know where !!
max vpn session is set to unlimited

#1390

This issue has been resolved in the developer edition.

You are looking at it.

how could i install it in ubuntu 22.04 ?


API improvements and interface change

This PR improves the API such that it actually becomes usable. The API endpoints for creating and altering Shifts now work as expected and the non-REST-like API endpoints have been removed in favour of a PATCH to the Shift API endpoints. Tests have been added to ensure that the endpoints work as expected.
Also closes #435

I'll leave the review to @DeD1rk


Change the text color of days in month

How can I change the text color of days in Month Calendar ? Can you please add some of the functions in this library to customize the views.
Regards

Hi, you can implement your own view for month day where you can change: text color, background, font style etc. You can do it using setMonthDayLayout method in MonthCalendarConfiguration.Builder, where you should pass your own layout id and MonthDayDecorator implementation where you can customize what you want. Please reference to the source of MonthListCalendarInteractionActivity. Here is example:
MonthCalendarConfiguration.Builder builder = new MonthCalendarConfiguration.Builder(this);
        builder.setMonthDayLayout(R.layout.custom_month_day_layout, new MonthDayDecorator() {
            @Override
            public void onBindDayView(View view, Calendar monthDay, Calendar month, List<Event> eventList, boolean isSelected, boolean isToday) {
                if (!DateUtils.isSameMonth(month, monthDay)) {
                    view.setVisibility(View.GONE);
                    return;
                } else view.setVisibility(View.VISIBLE);
                TextView day = (TextView) view.findViewById(R.id.day_view);
                day.setText(String.valueOf(monthDay.get(Calendar.DAY_OF_MONTH)));
                day.setTextColor(Color.WHITE);

                view.setBackgroundColor(Color.TRANSPARENT);
                if (isToday) view.setBackgroundColor(Color.RED);
                if (isSelected) view.setBackgroundColor(Color.MAGENTA);
            }
        });

where:
view - your inflated layout;
monthDay - day of month;
month - current month;
eventList - list of events if you are displaying them or empty (you can use it to show event indicator in day view);
isSelected - informs is this day selected;
isToday - informs is this day today day.
Is it enough for you to customize your own day view?

Got It! Thank you so much for this awesome work :)

FYI you can do the same with week titles for month calendar using setDayWeekTitleLayout. ListCalendarConfiguration.Builder contains similar methods.

@Abdullah-Nasim And thank you for your feedback!!! :)


Fixes #491 update v1.4.0 changelog

I generated diffs in spec since v1.3.0 via following command line, which I used as a guide to update the 1.4.0 changes section:
git diff v1.3.0 HEAD ./docs/v1/P4Runtime-Spec.mdk > v1.3.0-spec-diffs.txt

Something went wrong with this MR, closing and will re-do.


Refactor and add more tests

Pull Request Test Coverage Report for Build 15

49 of 51 (96.08%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 97.368%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/useFormState.js
32
34
94.12%

Totals

Change from base Build 13:
0.7%

Covered Lines:
50

Relevant Lines:
52

💛  - Coveralls


feat: support Angular 12

@joeldenning cannot make it working since it throws:

Error: You forgot to add 'mini-css-extract-plugin' plugin (i.e. { plugins: [new MiniCssExtractPlugin()] }), please read https://github.com/webpack-contrib/mini-css-extract-plugin#getting-started

https://stackoverflow.com/questions/65961356/how-to-get-mini-css-extract-plugin-to-work-with-webpack-5
Do you have any ideas?

We're currently removing mini-css-extract-plugin from the webpack config in favor of using style-loader, since it's hard to make sure the css is loaded from the correct url at the correct time with mini-css-extract-plugin. I created https://github.com/single-spa/single-spa-css to help with this, but haven't fully tested or incorporated it into react/vue/angular single-spa CLIs yet.
Does angular 12 just refuse to compile without that plugin? If so, first attempt would be to work around that. Second attempt would be to use single-spa-css in conjunction with single-spa-angular to ensure proper loading of css files.

Oh I just realized a more likely explanation - it's probably not Angular, but the mini-css-extract-plugin, that is throwing that error. The plugin requires a webpack loader and a webpack plugin - perhaps single-spa-angular is removing the plugin without removing the loader. See https://www.npmjs.com/package/mini-css-extract-plugin for docs. Sharing the relevant code in single-spa-angular below - my guess is that angular 12's webpack config is different such that we no longer enter the if statement on line 104
https://github.com/single-spa/single-spa-angular/blob/9c6b8fcf23fc5f167180608bba27309b9686f4d0/libs/single-spa-angular/webpack/index.ts#L94-L108

I've just tested the current single-spa-angular version with Angular 12.
One error that showed up was:
Error: node_modules/single-spa-angular/src/extra-providers.d.ts:12:5 - error TS2416: Property 'onPopState' in type 'SingleSpaPlatformLocation' is not assignable to the same property in base type 'ɵBrowserPlatformLocation'.
  Type '(fn: OnPopStateListener) => void' is not assignable to type '(fn: LocationChangeListener) => VoidFunction'.
    Type 'void' is not assignable to type 'VoidFunction'.

After changing the type of "onPopState" from void to VoidFunction, the error is gone
Another point is that I had to remove the aot=true option from the angular.json file, otherwise some errors showed up:
Error: ./src/main.single-spa.ts
Module build failed (from ./node_modules/@ngtools/webpack/src/ivy/index.js):
TypeError: angularCompiler.getNextProgram is not a function

I'm not sure if this has something to do with single-spa-angular at all, looks more like an angular issue I guess.

looks good - I'll merge and publish a new version later today or tomorrow. If I forget, feel free to remind me here to do it.

Reminder, please publish the new version, waiting for new version.

Am having trouble with the publish - yarn build does not create the directories and files I would expect. No package.json nor the main single-spa-angular.js bundles. Instead, it's creating the files below. I'll investigate further. @arturovt do you know what would cause this?
lib
lib/README.md
lib/lib
lib/lib/webpack
lib/lib/webpack/webpack-5
lib/lib/webpack/webpack-5/remove-mini-css-extract.d.ts
lib/lib/webpack/webpack-5/remove-mini-css-extract.js.map
lib/lib/webpack/webpack-5/remove-mini-css-extract.js
lib/lib/webpack/index.js
lib/lib/webpack/index.js.map
lib/lib/webpack/index.d.ts
lib/schematics
lib/schematics/schematics.json
lib/schematics/ng-add
lib/schematics/ng-add/tests
lib/schematics/ng-add/tests/utils.d.ts
lib/schematics/ng-add/tests/utils.js
lib/schematics/ng-add/tests/utils.js.map
lib/schematics/ng-add/index.js
lib/schematics/ng-add/rules
lib/schematics/ng-add/rules/update-configuration.d.ts
lib/schematics/ng-add/rules/add-dependencies.js.map
lib/schematics/ng-add/rules/update-configuration.js.map
lib/schematics/ng-add/rules/show-warning-if-routing-is-enabled.d.ts
lib/schematics/ng-add/rules/add-npm-scripts.js
lib/schematics/ng-add/rules/add-npm-scripts.d.ts
lib/schematics/ng-add/rules/add-dependencies.js
lib/schematics/ng-add/rules/show-warning-if-routing-is-enabled.js.map
lib/schematics/ng-add/rules/create-main-entry.js.map
lib/schematics/ng-add/rules/utils.d.ts
lib/schematics/ng-add/rules/update-configuration.js
lib/schematics/ng-add/rules/add-npm-scripts.js.map
lib/schematics/ng-add/rules/utils.js
lib/schematics/ng-add/rules/utils.js.map
lib/schematics/ng-add/rules/create-main-entry.js
lib/schematics/ng-add/rules/create-main-entry.d.ts
lib/schematics/ng-add/rules/show-warning-if-routing-is-enabled.js
lib/schematics/ng-add/rules/add-dependencies.d.ts
lib/schematics/ng-add/index.js.map
lib/schematics/ng-add/schema.json
lib/schematics/ng-add/schema.d.ts
lib/schematics/ng-add/index.d.ts
lib/schematics/ng-add/_files
lib/schematics/ng-add/_files/extra-webpack.config.js.template
lib/schematics/ng-add/_files/src
lib/schematics/ng-add/_files/src/app
lib/schematics/ng-add/_files/src/app/empty-route
lib/schematics/ng-add/_files/src/app/empty-route/empty-route.component.ts.template
lib/schematics/ng-add/_files/src/main.single-spa.ts.template
lib/schematics/ng-add/_files/src/single-spa
lib/schematics/ng-add/_files/src/single-spa/asset-url.ts.template
lib/schematics/ng-add/_files/src/single-spa/single-spa-props.ts.template

Seems to be a problem with the nx build inside of build:single-spa-angular. Here's the terminal output, but no files are created:
yarn build:single-spa-angular
yarn run v1.22.10
warning ../../package.json: No license field
$ nx build single-spa-angular && cpx README.md lib

> nx run single-spa-angular:build [existing outputs match the cache, left as is]
Building Angular Package

------------------------------------------------------------------------------
Building entry point 'single-spa-angular/internals'
------------------------------------------------------------------------------

------------------------------------------------------------------------------
Building entry point 'single-spa-angular/parcel'
------------------------------------------------------------------------------

------------------------------------------------------------------------------
Building entry point 'single-spa-angular'
------------------------------------------------------------------------------

------------------------------------------------------------------------------
Building entry point 'single-spa-angular/elements'
------------------------------------------------------------------------------

------------------------------------------------------------------------------
Built Angular Package
 - from: /Users/joeldenning/code/single-spa-angular/libs/single-spa-angular
 - to:   /Users/joeldenning/code/single-spa-angular/lib
------------------------------------------------------------------------------

Build at: 2021-06-01T21:34:26.804Z - Time: 8733ms

———————————————————————————————————————————————

>  NX   SUCCESS  Running target "build" succeeded

  Nx read the output from cache instead of running the command for 1 out of 1 tasks.

>  NX   CLOUD  See run details at https://nx.app/runs/uiU0OJewVRc

✨  Done in 1.86s.

$ find lib
lib
lib/README.md

@joeldenning I've pushed to master branch, could you pull and try again

That worked! Thanks. Publishing now. I've created the 4.x branch for long term maintenance of the 4.x releases

Released in https://github.com/single-spa/single-spa-angular/releases/tag/v5.0.0.

Documented in https://github.com/single-spa/single-spa.js.org/pull/456

I've just tried the new version with Angular 12, but I'm getting an error when using "ng serve"
 - configuration.output has an unknown property 'chunkLoadingGlobal'. These properties are valid:
   object { auxiliaryComment?, chunkCallbackName?, chunkFilename?, chunkLoadTimeout?, crossOriginLoading?, devtoolFallbackModuleFilenameTemplate?, devtoolLineToLine?, devtoolModuleFilenameTemplate?, devtoolNamespace?, filename?, futureEmitAssets?, globalObject?, hashDigest?, hashDigestLength?, hashFunction?, hashSalt?, hotUpdateChunkFilename?, hotUpdateFunction?, hotUpdateMainFilename?, jsonpFunction?, jsonpScriptType?, library?, libraryExport?, libraryTarget?, path?, pathinfo?, publicPath?, sourceMapFilename?, sourcePrefix?, strictModuleExceptionHandling?, umdNamedDefine?, webassemblyModuleFilename? }
   -> Options affecting the output of the compilation. `output` options tell webpack how to write the compiled files to disk.


Fix typo in help text for list cmd

The list command referenced a "--delete" flag that doesn't exist. Fixed this typo to "--deleted".
Also, regenerated docs, as it looks like that hasn't been done in awhile.

Thanks for regenerating the docs, too. It's apparently been quite a while.


Collect content for each entity ID provided in the solution set

User Story:

For each entity in a solution set, the operator controller is able to retrieve the associated packageName and bundle URL.

Background:

Once Deppy has identified which operators need to be on cluster, the Operator Controller must be able to retrieve the content that needs to be installed onto the cluster.

Acceptance Criteria:

For each entity in a solution set, a packageName and bundle URL are retrieved.
Failure to retrieve the packageName or bundleContent surfaces as an error in the Operator Cr.

Completed in https://github.com/operator-framework/operator-controller/pull/104


createMemoryHistory API not exposed

The v2.0.0-rc4 does not expose the createMemoryHistory API. From history (created from createMemoryHistory from history module) I was also able to call createLocation. I am hoping this will not be a problem v2.0.0.
I am creating programmatic routes but unable to do so just yet with what is available in the rc release to migrate away from v1.0.3 and history 1.17.0. Hope some of these breaking changes will stop soon. Its been a bumpy ride.

This will be exported in the next version: https://github.com/rackt/react-router/commit/ed630a5e8384d7f5a38f34384fb924735d95896e


method with empty regex sometimes raises a syntax error

def a(arg1, arg2)
end
def b(arg1, arg2, arg3)
end

a(//, "") # OK
a /s/, "" # OK
a "", // # OK
a %r{}, "" # OK
b "", //, 1 # OK
a //, "" # unexpected token: ,

Perhaps it has something to do with #6761 .
crystal version: 0.27.0
OS: macOS(Mojave)

According to playground, this code works with v0.25.1 but doesn't work with v0.26.0

I think this is a wont fix.
when the parses sees the partial expression a //, it is already decided that // is a division operator.
Something similar happen if you want to pass a tuple to a method without parenthesis a { is already a call with a block.
a(//, "") what you would need to type.
Similar to a({1}).


Constructor non-intuitive cancellation of small coefficient values

The main constructor contains the 3 lines:
# determine the last nonzero element and truncate a accordingly
a_last = max(1,findlast( p->(abs(p) > 2*eps(T)), a))
new(a[1:a_last], symbol(var))

That has a surprising consequence:
All higher coefficients are truncated, if their absolute value is below eps(). That generates an underflow-behaviour at high values. It would be easy to find p=Poly(..); p / 1000 * 1000 != p. There should be no such limitation, which restricts usability, for example when working with interpolating polynomials of higher degrees.
I recommend to replace abs(p) > eps(T) by p != zero(T)

I believe this is the case for T an integer or rational type. Can you use those?

I kind of agree with point 1.  If we are to keep the smallness condition it should be relative to other coefficients, not an absolute value like it is now.  The tests were all positive with
# a_last = max(1,findlast( p->(abs(p) > 2*eps(T)), a))
a_last = max(1,findlast(a))

but maybe I am missing some edge cases.
As for point 2. a[1:a_last] always returns a copy of a so I cannot reproduce the behavior you described.

jverzani.  I could use high precision or rational or integer arthmetic, of couse, but my problem is in the Float64 case, where eps ~ 2e-16.
pwl. Your are right with issue 2, which I removed already in the original comment.
I have here some test cases, which show the bad useability.
T1: I would expect to be  p(x) * a != 0 if p(x) != 0 and a != 0, but   Poly([0.0, 2e-8]) * 2e-8 == Poly(0.0).
T2: Imagine I want to interpolate at 5 points x the values y by a polynomial of degree 4. I try pf = polyfit(float([0:4]*1e4, [1, 0, 0, 0, 0]); but degree(pf) == 4 is false. Reason is that Poly cannot represent the highest coefficient, which gets below 2eps(). There would be a workaround by scaling x, but that is absolutely not necessary, because Float64 is capable of representing 1e-308.
With the same argument as forbidding higher coefficients below eps() it would be justified to set all results of floating point scalar operations to zero, if they are below eps() in absolute value ...
I would suggest to let the user decide, if he wants coefficients to be truncated by introducing a new utility function like polytruncate(p::Poly, tolerance) ... and in the normal case only remove exact zero.

There are arguments for or against. If you change the internal definition of eps to eps{F<:AbstractFloat}(x::Type{F}) = Base.zero(F) , then you get what you want. However, this means simple polynomial operations can go askew:
p = Poly([1,1]/10)
q = Poly([1,2]/10)
r = Poly([1,3]/10)
p + q - r  ## will have wrong degree

I don't see an easy way to let the user specify what is used for eps, but if you do, and it doesn't impact performance you should put together a pull request.

I believe my original intention was to avoid numeric instability in the evaluation of polynomials, division and root finding from the presence of non-significant leading zeros. eps(max(coef)) would probably have been a more correct cutoff, so that magnitude scaling didn't affect the result.


Added ProjectCodeTest that tests if all classes inside tests are final or abstract

I have treated that comment as a request!
ping @GrahamCampbell

Nice. :)

Look good :)

Now test internal too ;)


Train on new dataset

Can you share commands to train on a new dataset, please?

Sorry, but we don't plan to release the train code.

But you have mentioned that "We will release our model and newly labeled data as a strong foundation for future research. Codes and data....". How can your work can become a foundation for future research, if your are not planning to release the train code ? This statement in the paper might be misleading.


SelfDuration off in ubuntu/unix?

I'm seeing some weird results in a devcontainer based on ubuntu 20.04. SelfDuration is reporting 1min in a 1 seconds run.
Only me or unix-compatibility issue? Tried with both PS 7.0.6 and 7.1.3 in same image. Works fine on Windows using PS 7.1.3.
PS /workspaces/Profiler> $trace = Trace-Script { Start-Sleep -Seconds 1 }

Running in PowerShell 7.0.6.
Tracing...
Run  finished after 00:00:01.0016429
Tracing done. Got 5 trace events.
Processing 5 trace events.
Loading sources for 1 files.
Figuring out flow.
Sorting trace events to file lines.
Figuring out durations per line.
Counting averages and percentages.
Getting Top50 that take most percent of the run.
Getting Top50 with the longest average Duration.
Getting Top50 with the longest Duration.
Getting Top50 with the longest average SelfDuration.
Getting Top50 with the longest SelfDuration.
Getting Top50 with the most hits.
Progress: A: 00:01:40.0548300 (0 ms) -> A: 00:01:40.0540700 (-1 ms) -> A: 00:01:40.0669400 (13 ms)
Done. Try $trace.Top50 | Format-Table to get the report. There are also Top50Duration, Top50Average, Top50SelfDuration, Top50SelfAverage, Top50HitCount, All, Events and Files.

PS /workspaces/Profiler> [Profiler.Tracer]::Hits | ft SelfDuration, Text -AutoSize

SelfDuration     Text
------------     ----
00:00:00.0264400 & $ScriptBlock
00:00:00.0007600 {
00:01:40.0373800 Start-Sleep -Seconds 1
00:00:00.0066600 }
00:00:00.0194100 Set-PSDebug -Trace 0

Did not try it on anything other than 64 bit windows powershell 5 and 7. We will see when tests are written. And CI setup.

Yup I can see it on CI build now. https://github.com/nohwnd/Profiler/pull/21

$t = { [System.Diagnostics.Stopwatch]::GetTimestamp() }; $t1= &$t;Start-Sleep -milli 10; $t2=&$t;[timespan]::fromticks($t2-$t1).TotalMilliseconds
1033.65 <- it is 100 times higher

Fixed in #22


Package Revision Update for Passenger

Apparently rbenv needs to update the version number for passenger due to versioning in its PATH:
https://github.com/phusion/passenger/issues/2416
A recent update has the wrong PATH, which breaks it.  I would have thought this would be automated, but it is apparently not automated.  I tried building from source, but that didn't fix it.  Version 6.0.12_2 is installed, but reporting 6.0.12:
>>> 9:30:11 rich@HQ3 ~ ➜ brew install passenger
Warning: passenger 6.0.12_2 is already installed and up-to-date.
To reinstall 6.0.12_2, run:
  brew reinstall passenger
>>> 9:30:14 rich@HQ3 ~ ➜ brew reinstall passenger
==> Downloading https://ghcr.io/v2/homebrew/core/passenger/manifests/6.0.12_2
Already downloaded: /Users/rich/Library/Caches/Homebrew/downloads/62b87744e3a46a8be141cf2197b831ca44fe55d6d7f235592ea7941e32a46306--passenger-6.0.12_2.bottle_manifest.json
==> Downloading https://ghcr.io/v2/homebrew/core/passenger/blobs/sha256:53eda31b5562cfec0e19fd20b7255185f4d30fa3f4acc3e44d70fe4252fb
Already downloaded: /Users/rich/Library/Caches/Homebrew/downloads/cdeb67c2264d19452c04af682a230003aa09ad435f60f6c815cdba90e451ceab--passenger--6.0.12_2.arm64_monterey.bottle.tar.gz
==> Reinstalling passenger
==> Pouring passenger--6.0.12_2.arm64_monterey.bottle.tar.gz
==> Caveats
To activate Phusion Passenger for Nginx, run:
  brew install nginx
And add the following to /opt/homebrew/etc/nginx/nginx.conf at the top scope (outside http{}):
  load_module /opt/homebrew/opt/passenger/libexec/modules/ngx_http_passenger_module.so;
And add the following to /opt/homebrew/etc/nginx/nginx.conf in the http scope:
  passenger_root /opt/homebrew/opt/passenger/libexec/src/ruby_supportlib/phusion_passenger/locations.ini;
  passenger_ruby /usr/bin/ruby;

To activate Phusion Passenger for Apache, create /etc/apache2/other/passenger.conf:
  LoadModule passenger_module /opt/homebrew/opt/passenger/libexec/buildout/apache2/mod_passenger.so
  PassengerRoot /opt/homebrew/opt/passenger/libexec/src/ruby_supportlib/phusion_passenger/locations.ini
  PassengerDefaultRuby /usr/bin/ruby
==> Summary
🍺  /opt/homebrew/Cellar/passenger/6.0.12_2: 5,008 files, 245.3MB
==> Running `brew cleanup passenger`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).

>>> 9:30:49 rich@HQ3 ~ ➜ brew info passenger
passenger: stable 6.0.12 (bottled), HEAD
Server for Ruby, Python, and Node.js apps via Apache/NGINX
https://www.phusionpassenger.com/
/opt/homebrew/Cellar/passenger/6.0.12_2 (5,008 files, 245.3MB) *
  Poured from bottle on 2022-03-11 at 09:30:23
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/passenger.rb
License: MIT
==> Dependencies
Build: nginx ✔
Required<EMAIL_ADDRESS>✔, pcre ✔
==> Options
--HEAD
	Install HEAD version
==> Caveats
To activate Phusion Passenger for Nginx, run:
  brew install nginx
And add the following to /opt/homebrew/etc/nginx/nginx.conf at the top scope (outside http{}):
  load_module /opt/homebrew/opt/passenger/libexec/modules/ngx_http_passenger_module.so;
And add the following to /opt/homebrew/etc/nginx/nginx.conf in the http scope:
  passenger_root /opt/homebrew/opt/passenger/libexec/src/ruby_supportlib/phusion_passenger/locations.ini;
  passenger_ruby /usr/bin/ruby;

To activate Phusion Passenger for Apache, create /etc/apache2/other/passenger.conf:
  LoadModule passenger_module /opt/homebrew/opt/passenger/libexec/buildout/apache2/mod_passenger.so
  PassengerRoot /opt/homebrew/opt/passenger/libexec/src/ruby_supportlib/phusion_passenger/locations.ini
  PassengerDefaultRuby /usr/bin/ruby
==> Analytics
install: 275 (30 days), 1,048 (90 days), 5,879 (365 days)
install-on-request: 263 (30 days), 986 (90 days), 5,472 (365 days)
build-error: 0 (30 days)

Wrong location.


Sync readme

The docs -> readmy synchronization we have discussed

@Diggsey I guess you can merge this, checks related the PR are seem to pass

Thanks 👍


Upgrading from 1.x to 2.x, missing accessors

I'm currently in the process of upgrading my Rails application from 3.2 to 4.0 (and upwards to 5.1 but that's another story). As part of that upgrade, I changed simple_enum version from 1.6.9 to 2.3.1.
I find that newer accessors are not working, especially scopes.
Here is my model
class Season < ActiveRecord::Base
  as_enum :name, low: 0, medium: 1, high: 2
end

When I try finding the low season (or seasons), it will fail
Season.low #fail
Season.lows #fail

# error
NoMethodError: undefined method `lows' for #<Class:0x007f8415a11f08>

I believe it is because the accessor method where not generated at the time of the migration. Is there a way I can make it work?
I managed to get it working using the column_name but I believe it limits the use of simple_enum 🙃
Season.where(name_cd: 0)

Hi @nicolasrouanne I think this might be a duplicate of https://github.com/lwe/simple_enum/issues/137, I think my fix as per https://github.com/lwe/simple_enum/pull/138 should work, would you mind bundling from that branch and letting me know whether you still encounter any issues?

Hi @drobny, thanks for the quick answer 😄
Yes, indeed, #138 does fix it:
# app/models/season.rb
class Season < ActiveRecord::Base
  as_enum :name, low: 0, medium: 1, high: 2
end

# rails console
> Season.lows

+----+---------+---------------------------+---------------------------+
| id | name_cd | created_at                | updated_at                |
+----+---------+---------------------------+---------------------------+
| 1  | 0       | 2016-05-31 13:00:03 +0200 | 2016-05-31 13:00:03 +0200 |
+----+---------+---------------------------+---------------------------+

However, trying to not pluralize doesn't work
# app/models/season.rb
class Season < ActiveRecord::Base
  as_enum :name, low: 0, medium: 1, high: 2, pluralize_scopes: false
end

# rails console
> Season.low
NoMethodError: undefined method `low' for #<Class:0x007f8b6fb5f570>

However, I see the same behaviour with #138  than with gem version 2.3.0 which is said to work fine.
So I may be not fully understand how the feature works... 😉

That looks right to me, I am guessing you restarted / reloaded the console session after you added the pluralize_scope: false option in?

Yes @drobny I did restart the console (2 or 3 times 🤣) after adding pluralize_scopes: false. But, as I said, this doesn't work with 2.3.0 either. So maybe it's something else...

Mmm, have you got a repo I can have a look at to do some testing against? The pluralize_scopes option was only introduced in version 2.3.1

Hey @drobny, sorry for the late reply. Here is a simple repository reproducing the issue. Thx

Thanks @nicolasrouanne for doing that, will look into it as soon as possible

Looks like if you define it like
as_enum :name, low: 0, medium: 1, high: 2, pluralize_scopes: false it doesn't work because pluralize_scopes gets shoved in as a name of an option.
If you do something like
as_enum :name, [:low, :medium, :high], pluralize_scopes: false then it should work but whether that is not flexible enough or not is another matter. What are your thoughts?

Well it does work indeed!
However, what would happen if I want to use other indices? Such as
as_enum :name, low: 1, medium: 2, high: 3

I actually have models in my code where I start the index at 1 instead of 0 😉. And I do like the option of being able to manually decide which integer value I choose for each item.

I'm having the exact same issue. It's not clear from following this thread though what the root cause is. I'm also upgrade a rails app from 3.2 to 5.2 and going from 1.6.1 to 2.3.1
None of the solutions mentioned in this thread worked for me:

setting pluralize_scopes: false -- fail
I was already using an array of symbols e.g.  as_enum status: [:pending, :confirmed]
I saw in a related ticket that caching might be involved.

Thanks

@hraynaud can you try pointing your Gemfile to the master repo instead of 2.3.1 and see if that fixes it? I think the proposed fix has not been release as a new patch version yet

@drobny @lwe Thanks for the reply. Sorry it took me a couple of days to get back to this. Pointing to the master branch works.
Thanks for your help.

👍 awsome, thanks for checking, so I guess I'll release that as an upgrade, so everyone can benefit.

published 2.3.2 to rubygems, https://rubygems.org/gems/simple_enum/versions/2.3.2


Bug: adding/changing labels and taints have no effect when running the plan again

If a cluster is defined, it cannot have taints and labels added or changed.

@otavio This is expected, I will clarify it in the docs. Not everything can be changed once the cluster is up and running.

Couldn't it be reworked using the kubectl? so it'll allow the provisioning to keep it in a consistent state. Or do you see some problem in doing it?

The thing is - If it doesn't touch Hetzner, there is no need to do it through our terraform provider. As you said, it can be done through kubectl or even probably the Kubernetes terraform provider.
I fully understand the need to keep state; that's why we built Kube-Hetzner as a module. Now, I would be curious if you see an easy fix that would allow the labels to be re-applied. But it seems out of scope for me. Yes, setting an initial state is okay because we have an opportunity to do so, but later, what is well supported is adding and removing nodes.
@phaer @mnencia What do you think?

@otavio I will make sure to update the docs. Thanks for this precision, it's indeed important that our users know what to expect.

@otavio Maybe you could have a look at https://github.com/xunleii/terraform-module-k3s/blob/master/server_nodes.tf#L265
And create a PR if you really wanted this functionality :)

In fact, kube-hetzner could use the module and avoid duplicating the effort.

@otavio We won't add a dependency on another module at this point, but as @HenkVanMaanen said, PRs are always welcome to add the functionality you want!

Closing this as "not planned", but as said above, contributions are always welcome!


[sdbusplus] add new port

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[ ] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

Added meson option async, to turn off internal impl of event loop, to use boost::asio only. It is both beneficial to reduce compile time of the port (if it is not needed don't compile it). Secondly, libc++ still does not implement internals of async.
Hacks to work with old version of meson where added.

Updating meson is blocked by the tool RFC PR. Furthermore updating meson is no simple task since it requires changes to vcpkg_confgure_meson and how compiler flags are setup. (I mean I already did the work the problem is the tool blocker.)

@Neumann-A , should I try to patch the current version instead?
Add support for cpp23 compiler flag.

Pipeline is failing https://dev.azure.com/vcpkg/public/_build/results?buildId=94519&view=logs&j=f79cfdd7-47a8-597f-8f57-dc3e21a8f2ad&t=da63cf11-1f12-503b-6d64-3b2fb713c172&l=2682
Should be resolved after merge of this PR: https://github.com/microsoft/vcpkg/pull/33999

@FrankXie05 can you review this PR?

@FrankXie05  is there any progress with the reviewing of this PR?

Date-Version updated, @FrankXie05 @autoantwort are there any blockers for reviewing the PR. We are ready to make changes and work with you to get this merged.

All features and usage have been tested successfully on x64-linux.

Will the asio-only feature & patch remain to be maintained in vcpkg forever?
Does the asio-only feature fulfil the "features are additive" requirement?

Giving upstream until March 29 to respond


Login refactor

[ ] We need to refactor the whole ClusterLogin concept as separate files and serialize these as kubeconfig extension properties.

[x] Persist ClusterLogin properties as kubeconfig extensions
[ ] Rename ClusterLogin to something else that will make more sense (ConfigExtensions?)
[ ] Refactor the neon-cli login related commands
[ ] Refactor cluster prepare and setup

[ ] Test:

[ ] KubernetesFixture
[ ] WinDesktop

CLOSING: This really needs to happen as part of #1729


Resizing download button layout for mobile

Pre-Flight checklist

[X] Screenshots and design review for any changes that affect layout or styling

[X] Mobile width screenshots

[X] Testing

[X] Code is tested
[X] Changes have been manually tested

What are the relevant tickets?
Closes https://github.com/mitodl/ocw-hugo-themes/issues/943.
What's this PR do?
Modifies the mobile layout of the download button and related text, so that there is no overlap between the button and the text.
How should this be manually tested?
Start the course with yarn start:course and verify that the layout looks as expected in both desktop (unchanged by this PR) and mobile views. It may be necessary to first delete the public subfolder in the course content, to make sure that no prior layouts are cached.
Screenshots (if appropriate)

The screenshot looks good !
Unrelated:   Why does FAQ links to the zendesk home page ?   is there any FAQ specific to the course download ?

@Ferdi Is this the FAQ that should be linked? https://mitocw.zendesk.com/hc/en-us/articles/4414681093659-I-have-downloaded-an-MIT-OpenCourseWare-course-but-I-can-t-access-the-materials-How-do-I-get-started-


bindings.node is not a valid Win32 application

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Packager Version:

14.0.6

Electron Version:

6.0.9

Electron Rebuild:

1.8.6

Operating System:

win32 10.0.17763 (x64)

Last Known Working Electron Packager version::

N/A

Expected Behavior

Ran "npx electron-packager . --all --overwrite"
Generated for all platforms
All platforms running well
Actual Behavior

Ran "npx electron-packager . --all --overwrite"
Generated for all platforms
Only Win32-ia32 is not running as per expected behavior
Error log:
Error<EMAIL_ADDRESS>is not a valid Win32 application.
<EMAIL_ADDRESS>    at process.func (electron/js2c/asar.js:155)
    at process.func [as dlopen] (electron/js2c/asar.js:155)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:828)
    at Object.func (electron/js2c/asar.js:155)
    at Object.func [as .node] (electron/js2c/asar.js:155)
    at Module.load (internal/modules/cjs/loader.js:645)
    at Function.Module._load (internal/modules/cjs/loader.js:560)
    at Module.require (internal/modules/cjs/loader.js:685)
    at require (internal/modules/cjs/helpers.js:16)
    at bindings (D:\MyProject-win32-ia32\resources\app\node_modules\@serialport\bindings\node_modules\bindings\bindings.js:112)
To Reproduce

CLI: npx electron-packager . --all --overwrite
Additional Information

I cloned the electron-quick-start guide from https://github.com/electron/electron-quick-start
Based on that code, I created a server app using express js https://www.npmjs.com/package/express
The server will be used to print to a thermal printer using escpos https://www.npmjs.com/package/escpos

This is not a bug in Electron Packager. Native node modules will not work if you use --all for a couple of reasons:

You're not using the electron-rebuild hook for Packager
Due to limitations with the Node.js ecosystem, it is not currently possible to cross-compile native modules (i.e., compile a native module on a Windows host for a macOS target).

You will need to use the rebuild hook, and then build the Electron app on each OS you intend on targeting.


Doc browser: version selected does not work

When there are multiple API versions, the dropdown selector does not work.
The 'li' nodes are in the DOM, but when clicking the dropdown doesn't show them.
This is happening for my in Chrome (Mac) in case it matters.
This is easily reproducible using the praxis-example-app

I'll take a look. I wonder if we should produce something like the example app, but quite a maximal version that excersizes most of the features of Praxis. I dev using some production apps, but they don't really take advantage of all features.


Don't add a type schema if it is already in the spec

See marshmallow-code/apispec#402.  apispec_oneofschema should not attempt to add a schema if it is already in the spec.

Thanks.


SSL implementation database-to-marklogic

Putting in verify mode as this is still not an obvious candidate feature.

Can we talk about this one @grechaw? I'm not clear on where it came from and whether its an RFE.

Sure thing. SSL was a SHOULD in Justin's original requirements.    We have not ensured it's implemented anywhere... since it introduced key signing it's kind of busy work.
SSL between browser and app is more important than SSL between middle tier and MarkLogic, so this should probably be priorized below what, OR just be set to N/A.  It's a good thing to demo as far as management API is concerned, and if set up should not affect the app, but it seems wrong to have it in the 8.0-2 timeframe regardless.

okay. (i put it in 8.0-2 just to make sure we got it triaged now). I'll put in N/A as we consider it.

+1 from an internal conversation about configuring Samplestack Java http proxy  to use SSL to connect to ML back end (https://lists.marklogic.com/thread/7cbgokj4cfb3lcyo)


console logging in google-startup-scripts

Hi,
The rsyslog config in google-startup-scripts/etc/rsyslog.d/90-google.conf configures kernel and other messages to be dumped to the console, which makes console virtually impossible. Is there are reason for this?
Regards
Joe

We include kernel messages in case there is a failure on boot. The latest GitHub release contains a new Linux guest environment which limits the other logs that are written to the console.


Use flexible layout for hlist directives

Problem
The current hlist can look bad on mobile or when using many columns.
Reproducible Project
https://docs.blender.org/api/current/bpy.types.AlphaUnderSequence.html
Error Logs/Results

As you can see the list items extend beyond viewport when the viewport is narrow.
Expected Results
The columns should break into new rows.
Environment Info

Python Version: 3.9.x
Sphinx Version: 3.5.1
RTD Theme Version: 0.5.1

This is relatively easy to fix using display: flex and flex-flow: row wrap


Adds BelongsTo is NotOptional

The purpose of this PR is to see if others in the community would want this cop.
When a large monolith repo did the Rails 4 -> Rails 5 migration, we modified many of our Active Model belongs_to with an optional: true See blog post for why.
Rails 4 codebase:
class User
            belongs_to "company"
end

Rails 5 codebase:
class User
            belongs_to "company", optional: true
end

In many cases, the database column required the column, so the optional: true was misleading. We now what to remove optional:true when the FK is required.
Ps. I'm happy to clean up commits, etc if we want this PR.

Before submitting the PR make sure the following are checked:

[y] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[?] Wrote good commit messages.
[n] Commit message starts with [Fix #issue-number] (if the related issue exists).
[y] Feature branch is up-to-date with master (if not - rebase it).
[n] Squashed related commits together.
[y] Added tests.
[n] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[n ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[ ] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

This could be useful, but also tricky to implement properly. The primary/foreign key is not always id and the table name is not always the name of the association since they're all configurable:
belongs_to :parent, foreign_key: :uuid
belongs_to :parent, primary_key: :uuid
belongs_to :parent, class_name: 'Owner'
belongs_to :parent, polymorphic: true

I think the only way to do it right is va reflection, like:
MyModel.primary_key
MyModel.reflect_on_all_associations


prod clean install fails on "Invalid requirement: 'api/'"

When I do a clean prod install with a tarball built from master, I get an error running the install script. This has happened with several clean installs. But if I restart it, it completes. I don't see this in a different branch that hasn't been updated in a few days.
There's a recent commit that added a new requirement api/ in requirements/common.txt, which was formerly in dev.txt. Is it something about that?
https://github.com/zulip/zulip/commit/ce2c3fbb06bc18c9966ed6051eb83579cf6e3d39
Here's the last part of running /root/zulip/scripts/setup/install
Successfully installed pip wheel
Cleaning up...
Invalid requirement: 'api/'
It looks like a path. Does it exist ?
/srv/zulip-venv-cache/27ae06af0f1e1022553ce4c635e88440c06a6263/zulip-venv/local/lib/python2.7/site-packages/pip/_vendor/requests/packages/urllib3/util/ssl_.py:318: SNIMissingWarning: An HTTPS request has been made, but the SNI (Subject Name Indication) extension to TLS is not available on this platform. This may cause the server to present an incorrect TLS certificate, which can cause validation failures. You can upgrade to a newer version of Python to solve this. For more information, see https://urllib3.readthedocs.io/en/latest/security.html#snimissingwarning.
  SNIMissingWarning
/srv/zulip-venv-cache/27ae06af0f1e1022553ce4c635e88440c06a6263/zulip-venv/local/lib/python2.7/site-packages/pip/_vendor/requests/packages/urllib3/util/ssl_.py:122: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. You can upgrade to a newer version of Python to solve this. For more information, see https://urllib3.readthedocs.io/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
+ apt-get -y install build-essential libffi-dev libfreetype6-dev libz-dev libjpeg-dev libldap2-dev libmemcached-dev python3-dev python-dev python-pip python-virtualenv libxml2-dev libxslt1-dev libpq-dev
+ sudo rm -rf /srv/zulip-venv-cache/27ae06af0f1e1022553ce4c635e88440c06a6263/zulip-venv
+ sudo mkdir -p /srv/zulip-venv-cache/27ae06af0f1e1022553ce4c635e88440c06a6263/zulip-venv
+ sudo virtualenv -p python2 /srv/zulip-venv-cache/27ae06af0f1e1022553ce4c635e88440c06a6263/zulip-venv
+ sudo chown -R 0:0 /srv/zulip-venv-cache/27ae06af0f1e1022553ce4c635e88440c06a6263/zulip-venv
+ pip install -U setuptools
+ pip install --upgrade pip wheel
+ pip install --no-deps --requirement /root/zulip/requirements/py2_prod.txt

Error running a subcommand of /root/zulip/scripts/lib/create-production-venv: pip install --no-deps --requirement /root/zulip/requirements/py2_prod.txt
Actual error output for the subcommand is just above this.

Traceback (most recent call last):
  File "/root/zulip/scripts/lib/create-production-venv", line 28, in <module>
    virtualenv_args=['-p', 'python{}'.format(python_version)])
  File "/root/zulip/scripts/lib/setup_venv.py", line 215, in setup_virtualenv
    do_setup_virtualenv(cached_venv_path, requirements_file, virtualenv_args or [])
  File "/root/zulip/scripts/lib/setup_venv.py", line 249, in do_setup_virtualenv
    run(["pip", "install", "--no-deps", "--requirement", requirements_file])
  File "/root/zulip/scripts/lib/zulip_tools.py", line 98, in run
    subprocess.check_call(args, **kwargs)
  File "/usr/lib/python2.7/subprocess.py", line 540, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['pip', 'install', '--no-deps', '--requirement', '/root/zulip/requirements/py2_prod.txt']' returned non-zero exit status 1

Ahh, yeah, I bet this code only works if your current working direction is right.  Let me see if I can find a simple patch.

I think this might fix this; can you test @feorlen ?
diff --git a/scripts/lib/create-production-venv b/scripts/lib/create-production-venv
index 28a35a6..3d578d9 100755
--- a/scripts/lib/create-production-venv
+++ b/scripts/lib/create-production-venv
@@ -21,6 +21,10 @@ run(["apt-get", "-y", "install"] + VENV_DEPENDENCIES)

 python_version = sys.version_info[0]

+# Set the current working directory to the Zulip checkout, so the api/
+# relative path in requirements/common.txt works.
+os.chdir(ZULIP_PATH)
+
 venv_name = "zulip-venv" if sys.version_info[0] == 2 else "zulip-py3-venv"
 cached_venv_path = setup_virtualenv(
     os.path.join(args.deploy_path, venv_name),

@timabbott it works fine now, cool!

Cool, will merge that fix, thanks for testing @feorlen!


Update Link to Documentation & Discord Link (S)

Summary
As a DAO user I would like to be able go to the latest Documentation.
AC

[ ] Updated link to documentation on https://dao.hypha.earth/hypha/ to: https://notepad.hypha.earth/DBKEBKmIRMqqn9heCRf21w
[ ] Add Discord Icon (and link) to the support page: https://dao.hypha.earth/hypha/support
[ ] Headline: Need support?
[ ] Copy: Please read our Documentation for more info. If you are stuck with a problem you can also reach out to us on discord in the "dho-support" channel.
[ ] Link to documentation: https://notepad.hypha.earth/DBKEBKmIRMqqn9heCRf21w
[ ] Link to Discord: https://discord.com/channels/722537361480613950/732285564384051323

Design

(same as on Dashboard)

This was replaced with a setting on the DAO so it is dynamic now


Runtime: restrict behaviour to cygwin

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

__int128 is not available as an extension on MSVC nor clang for Windows.
Restrict the behaviour to cygwin only.

@compnerd, @gribozavr, I'm sorry so late comment.
clang with -fms-extension (without -fms-compatibility) can recognize the __int128, and that worked well for 64bit MSVC.
I think this will work for 32bit MSVC, but will not return two word type for 64bit MSVC. (this means two uintptr_t type not DWORD). How about change this part work for both 32/64bit MSVC using #if/#endif?

Well, cl nor clang-cl seem to recognize __int128 when I tested it (on x86, x86_64, nor ARM).  Does it actually work with cl?

@compnerd, As you know, the driver clang-cl invokes compiler including flags -fms-extension, -fms-compatibility, and -fms-compatibility-version=xx. clang-cl does not recognize __int128.
I used clang with option -fms-extension and -fms-compatibility-version=19 without -fms-compatibility. The clang with the options could recognize __int128 and compile MSVC2015 (at least for Update 2) library headers with some more options.
This approach needed lesser patch to Swift source and could generate some Swift stdlib for 64bit MSVC.


Let observers see job icons

Resolves #18550

make it an action (similar to enable/disable other ghosts)

Looks like the underlying systems would need an API extension for this sort of thing. @PrPleGoo
Closing for now because adding an inventory to ghosts and letting them toggle the HUD by equipping entities is probably not the best solution to this problem.


Buildings / Research costing +1 level

$elementLevel will never be set to either $PLANET[$resource[$Element]] or $USER[$resource[$Element]] therefore the cost displayed and taken from the planet will always be +1 level even when the Building / Research doesn't already exist on the planet.
Eg. Jumpgate
Factor is set to 2.00 in vars table
M 2000000, C 4000000, D 2000000 is the level one cost
although without the patch the current level one cost is:
M 4000000, C 8000000, D 4000000

i already posted a fix for this some time ago.

we had this in our game as well but we did a simple fix to just do
$elementLevel = $forLevel - 1;

And that would get the correct level.

So its a duplicate of #241?

If it were a duplicate why does the issue exist in a fresh pull from the master branch? referenced by the jumpgate example above
As explained in the original post this:
} elseif (isset($forLevel)) {
$elementLevel = $forLevel;
}
Will always result in the value of the building being set to the current level +1  for buildings and research instead of the below code being executed
} elseif (isset($PLANET[$resource[$Element]])) {
$elementLevel = $PLANET[$resource[$Element]];
} elseif (isset($USER[$resource[$Element]])) {
$elementLevel = $USER[$resource[$Element]];
}
As these are all elseif's when the any one condition is satisfied the rest of the chain will not be executed.
This is the problem as all buildings / research will have a $forlevel param passed into the function it will always result in the current level being +1

Maybe fixed in another branch of jan he accept pull request and he did 5months ago with what I posted ...
But maybe he didn't change dB values yet

I will look into this issue the weekend.

thats why we subtracted 1 from the for level to get the correct level to build and get the correct cost and build time.


no matching package named boring found issue

While upgrading rust to 1.71.0, we found some build failure as shown below:
cargo build --offline --lib --features ffi --release

error: no matching package named `boring` found
location searched: registry `crates-io`
required by package `quiche v0.17.2 (/private/tmp/cloudflare-quiche-20230719-97831-1ky1m5c/quiche)`
As a reminder, you're using offline mode (--offline) which can sometimes cause surprising resolution failures, if this error is too confusing you may wish to retry without the offline flag.

full build log, https://github.com/Homebrew/homebrew-core/actions/runs/5594719599/job/15158499681?pr=136514

looks like flaky build issue with the build cache, verified in https://github.com/chenrui333/github-action-test/actions/runs/5687099519


Release notes page update

Consider linking to Changelog instead of empty page for Release notes

PR to make the change:
https://github.com/cds-snc/gcds-docs/pull/209


Custom Linters

There is a fair amount of specific domain knowledge needed to write a linter (what HAML node you're looking for etc.) but we should at least allow people to go ahead and include their own custom linters. I previously forked this repo to add my custom linters, but I am very eager to remove that fork and just put the linters in the codebase.
This approach was very heavily influenced by Rubocop's require configuration.
Resolves https://github.com/brigade/haml-lint/issues/96

Thanks @daveallie!


Type field updated in Dataset API

Can one of the admins verify this patch?

Ok to test

retest this please

retest this please


How to avoid renaming columns when broadcasting

It seems like TSFrames overrides the behaviour of broadcast such that it renames the column names when a function is broadcast across a TSFrame.  How can I stop this from happening?
E.g.:
log_ts = log.(ts)
5×2 TSFrame with Int64 Index
 Index  var1_log     var2_log
 Int64  Float64      Float64
───────────────────────────────
    -2  -Inf         -1.38629
    -1    -1.09861   -0.693147
     0    -0.405465  -0.287682
     1     0.0        0.0
     2     0.0        0.0

How can I get the column names to be var1 and var2 without renaming them?  I tried passing renamecols to Base.broadcasted(log, ts; renamecols = false) but that doesn't seem to do anything.

I think the columns are getting renamed even when renamecols is false:
https://github.com/xKDR/TSFrames.jl/blob/305267dd2bd1ddcc94253e97efe01621e6d0cac5/src/broadcasting.jl#L3-L7


Power generator preset doesn't show filled generator:source field

Hi all,
I don't understand why power=generator preset is making the field generator:source field not visible once filled.
The preset shows appropriate generator:method and generator:type but no source when not empty. Is there any limit that prevent more than 5 fields to be displayed on iD?

According to wiki, usage of generator:source, generator:method or generator:type requires a power=generator tag on the same object
See :
https://wiki.openstreetmap.org/wiki/Key:generator:source
https://wiki.openstreetmap.org/wiki/Key:generator:method
https://wiki.openstreetmap.org/wiki/Key:generator:type
A warning should be displayed to encourage users to complete such objects with power=generator if missing
All the best

@flacombe This is happening because once you set generator:source to hydro, the preset changes to the more specific "Water Turbine" (in English). Because the tag comes along with the preset, we don't need to show a field for it.
I know this isn't always obvious. We should probably find a way to indicate that the preset changed, possibly as suggested in https://github.com/openstreetmap/iD/issues/5611#issuecomment-447674974.

Here's a demo of what likely happened.

Thank you @quincylvania to make this clear to me, I didn't noticed the preset changing.
Instead of making the preset label highlight when changing, iD could display lightly which fields are involved in the preset (with an additional section initially hidden ? or when clicking on I ?)
Problem is not only when the preset changes, but when you click on an already defined object and don't see the field in appropriate section. The point is to know what does the preset means exactly (water turbine can also be implied by generator:type=francis_turbine here).
All the best

iD could display lightly which fields are involved in the preset

This would be tricky since we generally hide fields that affect the preset (as evidenced by this issue).
If you're interested in what tags a preset requires, I encourage you to browse the preset definition files and exam the tags property.
https://github.com/openstreetmap/iD/blob/5edd1080706e8b8eb55db929eff3705bfda95c33/data/presets/presets/power/generator/source/hydro.json#L27-L30

I encourage you to browse the preset definition files and examine the tags property.
Thanks @quincylvania

It's not about me, but how users in general may understand presets meaning.
No problem to hide those fields at first, but retrieving them in an additional category on the menu will certainly improve understanding and sharing of mapping practices. Not everyone is able to browse config file or sources of iD.
It can be a special color/icon on the "all fields" section to highlight which fields are actually involved in the preset as well.
Such a feature can prevent to open issues like I did, at a glance.
My 2 cts


Mongo JSON utils

https://github.com/NexesJS/NQL/issues/2
This a rewrite of mongo json utilities (reason) that used to be based on regex'es and now operate only on Mongo JSON objects.

I'd suggest to merge this straight into our PR and then we can iterate on the missing pieces 👍
Our PR is WIP + HOLD anyway.


Template servlet doesn't respect welcome-file-list

As seen in https://github.com/jmiguelsamper/thymeleaf-extras-sqldialect/blob/master/src/main/java/org/thymeleafextras/config/StandaloneThymeleafServlet.java#L98, the template servlet doesn't respect entries in the welcome-file-list section of web.xml.

Ah, this is only if the servlet is also configured to respond to / requests.


✏ Rewordings in docs/en/docs/tutorial/debugging.md

A suggestion to change some words in __name__ == "__main__" section of the debugging.md to simplify some expressions.
And also replaced the word "automatic" in order not to mix up with automatic variable definition 🤔 (as I understand it has another meaning in this context).

Thanks!
I actually prefer the original words as the variables are not being set by the user, or by FastAPI or anything else, they just "have the value". Of course, this becomes a lot dependent on personal preference, but anyway, I prefer the previous wording.
I had never heard the term "automatic variable", I guess that comes from C++ and Java, maybe. But anyway, I updated it to not use that exact term.
Thanks! ☕


Some examples with covectors and some basic properties of next.

Working with covectors is quite difficult, but seems possible for the simple things I tried.
I proved some things about how later and Id type interact. In particular there is a proof that later Id implies Id of nexts. This only uses the (defined) induction principle for the Id type (the J rule).
For some of the properties it seems necessary to have the eta rule for next (the rule next [x <- t] x = t).
Not sure how important those properties are though.

Ah, yes, without a system on next we should have eta rules for it, otherwise there's no other reasoning principle available.
I'll look into the escaped tag error, it's definitely a bug somewhere.

Fixed!
By the way, are we happy that the following definition typechecks
idnext (A : U) (a b : |> A) (p : |> [x' <- a, y' <- b] (Id A x' y')) : Id (|> A) (next [x' <- a] x') (next [x' <- b] x')
   = <i> next [p' <- p] (p' @ i)

rather than needing to bind x' and y' in the local delayed substitution, like this?
   = <i> next [x' <- a, y' <- b, (p' : Id A x' y') <- p] (p' @ i)

The good thing is that they are definitionally equal.

Thanks for fixing it.
I am not entirely happy that the definition
idnext (A : U) (a b : |> A) (p : |> [x' <- a, y' <- b] (Id A x' y')) : Id (|> A) (next [x' <- a] x') (next [x' <- b] x')
    = <i> next [p' <- p] (p' @ i)

typechecks. At least I think with the rules we have in the paper (modulo a wishful extension with the cubical stuff) we would need to bind
     = <i> next [x' <- a, y' <- b, (p' : Id A x' y') <- p] (p' @ i)
as you say, because the term p' @ i is not typeable in the context without x' and y'.
On the other hand I also don't see what could go wrong by allowing this, although it is not clear to me at the moment what the semantics of such a term without x' and y' would be.

What's actually happening is that in both cases we get the following for p'
p' : Id A x' y' (next x' = a, next y' = b)
What's a bit sketchy is the way we justify the ability to form these thunks/delayed bindings next x' = a: we take care to only use them inside an outer next or |> by keeping track of the Tags but that's only in the implementation.
Maybe we can just say that [ a <- t ] is elaborated to xi[a <- t] for  t : |> xi A.

Unless we find something wrong I think your suggestion is a good one.


RFE: Make Aiven Operator more GitOps Friendly

The Aiven Operator is great and I can deploy a DB (sample using MYSQL) using it
apiVersion: aiven.io/v1alpha1
kind: MySQL
metadata:
  name: mysql-sample
spec:
  authSecretRef:
    name: aiven-token
    key: token
  connInfoSecretTarget:
    name: mysql-secret
  project: myproj
  cloudName: google-europe-west1
  plan: startup-4

However this stores the credentials in a secret. as specified under spec.connInfoSecretTarget
Usually (and especially true when doing GitOps with Flux/Argo CD), the secret needs to be stored in Git. More specifically it's stored in Git via Vault, Sealed Secrets, and/or External Secret Operator.
I would like to be able to set up connection parameters beforehand (or as much as I can) and have the Aiven Operator "respect" these parameters (i.e. use the parameters to set up the DB).

Hey.

The Aiven Operator is great and I can deploy a DB (sample using MYSQL) using it

Thank you, that's very nice of you.
Would this work for you?
Say you have applied this yaml stored in git:
apiVersion: v1
kind: Secret
metadata:
  name: mysecret
type: Opaque
data:
  username: YWRtaW4=
  password: bXktc2VjcmV0

---

apiVersion: aiven.io/v1alpha1
kind: MySQL
metadata:
  name: mysql-sample
spec:
  connInfoSecretTarget:
    name: mysecret

if you provide connInfoSecretTarget.name pointing to an existing Secret then the Operator uses that creds to set
if no Secret found, it creates a new one (as it does right now)

The only thing is that we can't validate the Secret.

1. if you provide `connInfoSecretTarget.name` pointing to an existing `Secret` then the Operator uses that creds to set

2. if no `Secret` found, it creates a new one (as it does right now)

Yes, this sounds perfectly reasonable

Few cons:
1. we can't validate the `Secret`.

2. for multiple reasons we won't be able to detect secret changes. Reconciliation stops once it has been successfully applied. To do so we mark our resources with annotations. But we never managed secrets so far.

I think those cons are fine to live with. Documentation should make this clear though.
Thanks! 🙌


Cannot join to the waterdog proxy through another proxy server

Version of Waterdog

All latest
Expected Behavior

Connect to the server through 2 proxies
Actual Behavior

Second proxy throws error
Crashdump, Backtrace or Other Files

https://hastebin.com/acejevoxoz.bash
Describe the bug

⬆
To Reproduce

Connect to the first proxy server that is set to the second proxy server
Screenshots

Server software used:

Server - Waterdog (localhost)
Server - Waterdog (VPS)
Server - PocketMine (VPS) (Nothing about connect in the console)

Additional context

Ip forward turned on

Need to set the middle proxy to offline mode. Using 2 proxies like that in a chain is definitely not an intended usecase, but should work if you disable online mode (Set to offline) the second proxy


How to install JellyBamm package

Pybamm is a very useful and effective lithium battery pack. I want to explore the spatial temperature distribution of cylindrical batteries, but I encountered some problems when installing JellyBamm. I don't know how to install it. I hope you can give me some help with installation. Thank you！

Currently JellyBaMM is not distributed through PyPI, so you shall have to install from source. There are instructions in the GitHub repository for JellyBaMM: https://github.com/TomTranter/JellyBaMM. You may also also try the
pip install git+https://github.com/TomTranter/JellyBaMM.git

command to let pip install it from the repository instead of having to clone it.


mongodb: 4.0.12 -> 4.2.8

Not strictly an upgrade, but adds a new mongodb-4_2 target with the
current mongodb from that branch.

Motivation for this change
Current mongodb isn't packaged, would like it to be.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[X] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

addressed comments, squashed out the errors i'd made, noticed that github doesn't preserve that history- apologies for that. regardless, PTAL. thanks!

/status needs_merger

@GrahamcOfBorg build mongodb-4_2

Oh, right. It's unfree now.

actually just caught an issue with tests- current test is using cross-version clients, and 3.4 to 4.2 doesn't work. quick update inbound.

In the meantime please prefix the commit messages with their scope, like you did on the first one

sure. mind if i just squash them out now that you've reviewed?

That's also fine, yes

/status needs_merger


Where is the Device Toolbar?

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
https://developers.google.com/web/tools/chrome-devtools/device-mode/
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.
Instructions on where to find, or how to open the Device Toolbar.  It states to "Click Toggle Device Toolbar" - no indication of where these words are located nor what to click or toggle.  It assumes that the toolbar is already opened.  Does not instruct how to open it.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.

The button is located in between "Elements" and the inspect icon (the leftmost one) in the DevTools toolbar. if you browse the article with images enabled you can see the same icon next to the "Click Toggle Device Toolbar".
You can also toggle Device Mode by hitting Ctrl/Cmd+Shift+M.


Jwt claims disabled

$subejct
Related Issues:

https://github.com/wso2/product-is/issues/20880

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/11550281938
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/11550281938

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/11550281938
Status: success


Problem occurred while unzipping the file 'data.zip'!

There is something wrong about your file 'data.zip' while unzipping, and they are empty after unzipping.

Fixed. Now it should be compatible with most unzip softwares.
Tested: Windows 10 built-in, Linux unzip and 7z.


Move topic names onto the parameter server

This should be done to all topics. The names should probably be grouped on the server, such as topics/imu.
The current code has the topic names hard-coded into each node, which makes it very easy to use the wrong name, and cumbersome to change names.

 Read config from yaml file


please make host part of OpenAIConfig

Hi,
Open AI is starting to deploy environments to Azure.  Would be great to have host part of OpenAIConfig, so createHttpClient can take from config, rather than hardcoded.  I couldn't see an easy way to override as many classes are marked as internal.
Thankyou.

Hello @ChrisGClark - I've added your suggestion to the latest release (v3.0.0) of the library! please give it a try :)

Amazing super fast!  Thanks a lot and also for the lib.

i'll use latest release today  :-)


CLOUDSTACK-8610. Unable to attach 7th Disk to Windows Server 2012 R2 …

…instance.
During disk attach, while trying to obtain the controller key for SCSI controller, look for device with the generic SCSI controller type i.e. VirtualSCSIController.
While trying to obtain the SCSI id to attach a disk to, CS should ignore the reserved SCSI id 7. But this is not being honored in case of VMs with SCSI controller of type 'VirtualLsiLogicSASController'. And so in case of Windows 2012 R2 VMs, CS chooses to attach the 7th disk on the reserved SCSI id and this fails on vCenter.

LGTM.
Device id 7 on virtual SCSI (can be of any sub type : LsiLogic, LsiSAS, BusLogic, Paravirtual) controller is reserved for controller itself, rest all virtual nodes could be used for virtual disks.

Looks good to me.

Looks like all checks are good and there are 2 LGTM too.
Should we merge this branch?

please add (unit) tests

+1 for @DaanHoogland's comments: please add unit tests to cover the changes.

Who wants to step in and finish this work? It seems the original author is not able to finish it. If no one steps in, we'll have to close the PR without merging it so please help :-).

LGTM, merging. Someone needs to add the unit test, later. I'm trying to build my skills around vmware related codebase.


Discord API Connection Error

I think this is likely a synchronisation error between TeX-Bot running & Discord. Restarting the application may fix it.

Closed due to lack of further discussion.


Create template

fixes #

Checklist:

[x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[x] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Add .. versionchanged:: entries in any relevant code docs.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

Do not use other people's projects on GitHub to practise using Git/GitHub.


[MAIN] Splitting signal functions into indiviual files for better mod…

Here is what I did
Here is why I did it
Here's ideally what I want second eyes on.

Closing as this was an example


Adding Textarea length field

According to our discussion in #14, This pull request adds a length field under the bottom right corner of the text field. Not sure what should the color/text of the field be, but this can be easily changed.
I've never used ember before so feel free to comment on whatever you feel can be done better :smile:

Edit : Adding the image.

Pretty clean code, i'm fascinated by following the same coding style, great job,
i have some design comments that will finalize it:

we should remove "Length :" from the indicator, that will leave us with the number only.
we should pull the div up to overlay the textarea, it should be a number on the textarea itself.
we should add a "padding-bottom" to the textarea so that the last line in it doesn't intersect with the length number.

otherwise everything is done perfectly, thank you.

Thanks :D
I tried the overlay thing. The problem is that when you scroll, the padding doesn't help anymore.

Aha i get it, ok, let me merge it and work on that, i may find some kind of solution for it.


windows 10

is this not  useful in win10? when open  with file   thow error “ Invalid chmod input data”

That's because the returned array name is empty. Thus parsing “chmod” when the error occurred.I added a filter function on it.
function delEmpty($v)
{
if ($v['name']===null || $v['name']==="")   //当数组name为null,或者为空的时候删除改元素
return false;
}
return true;
}
$list = array_filter($list,"delEmpty");

The above that there is a mistake, after the serial number to delete the chaos, I re modified the next, only the first element to deal with.
if($list[0]['name']===null||$list[0]['name']==='')
{
array_splice($list, 0, 1);
}

Hi.... let me check in a ftp server thru windows.
Btw, if you pass an empty value in rights response property, should work fine.
After it, you can disable the permissions in frontend extending the config file

thanks  !


Skip error.stack_trace if empty

Opening this PR for discussion.
Some exceptions in Elasticsearch don't contain a stacktrace, e.g.  NotMasterException.
For these error.stack_trace typically just repeats what's already present in error.type and error.message, having the field  doesn't add any value.
Can we skip error.stack_trace in this case?

Hey @mosche , sorry for the delayed response.

For these error.stack_trace typically just repeats what's already present in error.type and error.message, having the field doesn't add any value

This feels like a very elasticsearch specific edge-case, therefore I'm not sure whether we want to add this.
I also think that in general this would lead to a worse user-experience: E.g. if one is filtering on specifically error.stack_trace to contain a word, those errors would suddenly not match the search anymore, which isn't intuitive without knowledge of this "feature".


SILOptimizer: treat a destroy_addr as modification

When performing the r-value elimination pass repeatedly, we may have
inserted a destroy_addr of the source value.  In such a case, it is
important to treat the destroy_addr as a modification (as you may not
use the value any longer).  This seems to match the behaviour observed
on macOS (and theoretically Windows).  Although this appears to extra
copy, it does not misuse memory.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please benchmark

SR-13985, rdar://problem/72614608

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

FlattenListLoop
1629
2501
+53.5%
0.65x (?)

FlattenListFlatMap
4636
6618
+42.8%
0.70x (?)

NSStringConversion.MutableCopy.Rebridge
776
873
+12.5%
0.89x (?)

Improvement
OLD
NEW
DELTA
RATIO

CharIteration_utf16_unicodeScalars
5120
4360
-14.8%
1.17x (?)

NSStringConversion.UTF8
867
785
-9.5%
1.10x (?)

Code size: -O
Performance: -Osize

Improvement
OLD
NEW
DELTA
RATIO

FlattenListLoop
2503
1950
-22.1%
1.28x (?)

FlattenListFlatMap
6713
5989
-10.8%
1.12x (?)

Code size: -Osize
Performance: -Onone

Improvement
OLD
NEW
DELTA
RATIO

DataAppendBytesMedium
7320
6740
-7.9%
1.09x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

CC: @gottesmm @eeckstein

@swift-ci please test

This fix looks correct. To be clear, the problem was the assumptions that a destroy_addr modifies memory of the value being destroyed. It does not. However, the TempRValue code does need to check for either a modification of the source value or the end of the source value's lifetime.

@atrick my assumption was the latter part of the statement - that destroy_addr has marked the memory as being "invalid" (that is no guarantees can be made about the state of the memory).  Further use after the instruction is UB.  Based upon that, I made the assumption that it is okay to treat the destroy_addr as a load (which is what I feel like is the grey area - because its technically not a load).
However, thank you for verifying that this is a correct change!

Also, I would like a SIL test case for this.

Also, I would like a SIL test case for this.

Yes, I do have a SIL test case that I will of course add.

Yes, I do have a SIL test case that I will of course add.

@atrick There is a good documentation of this at the line which I mentioned in my previous comment: https://github.com/apple/swift/blob/949b0c0cc3632b0c278a33a4022dd280cf31215a/include/swift/SIL/SILInstruction.h#L379
Though, it's probably a bit hidden. I added references to this comment in the comments of the API functions: https://github.com/apple/swift/pull/35262

@atrick There is a good documentation of this at the line which I mentioned in my previous comment: https://github.com/apple/swift/blob/949b0c0cc3632b0c278a33a4022dd280cf31215a/include/swift/SIL/SILInstruction.h#L379
Though, it's probably a bit hidden. I added references to this comment in the comments of the API functions: https://github.com/apple/swift/pull/35262

@compnerd This is an interesting bug. Even the SIL test case only reproduces on linux, but not on MacOS. I'll take a look.

@compnerd This is an interesting bug. Even the SIL test case only reproduces on linux, but not on MacOS. I'll take a look.

I found the problem. This is something completely different. I'll put up a PR with a fix.

I found the problem. This is something completely different. I'll put up a PR with a fix.

Thanks @eeckstein!

Thanks @eeckstein!


Rename header.js to header.js

Fix: #21

Hi, thanks for helping. Actually someone else was doing this issue. Please only start an issue if you are assigned to it.
I appreciate the changes in any case. Though here it seems there are unrelated changes to what the issue #21 is asking.

@Niravprajapati1 thank you, the changes look good. Looks like there are merge conflicts. Do you know how to resolve them? Let me know if you don't know.

test is falling for table.js

Thanks 👍


Reduce flake8 max-line-length

Description
Reduced flake8 maximum length from 155 to 154 and changed some line exceeding the maximum line length.
Changed Behaviour
This won't change any functionality, it just improves the readability.
Fixes
Fixes #3105
Type of change
Choose which options apply, and delete the ones which do not apply.

Bug fix
Update to human readable text: Documentation/error messages/comments
Code maintenance/cleanup

thanks for this PR - your first commit looks good. then you merged in master, where someone else had already reduced the line length to 154 (in commit #3130) in a slightly different way - so now there isn't much left of your PR! but thats ok.
so you can make the line length even smaller, to end up with a merged PR for issue #3105 - or you can move on to looking at the issues tagged outreachy: https://github.com/Parsl/parsl/issues?q=is%3Aissue+is%3Aopen+label%3Aoutreachy to look for stuff you'd like to learn more about/work on.

Thank you for the feedback, @benclifford
Sure, I'll make the further line change in the issue #3105 and then proceed to other issues tagged  as outreachy.

Hey @benclifford
I made a new commit to reduce the maximum length further.
Please guide me and review the change.

ok, mostly looks good - there are a few unnecessary empty lines added that you can see here in https://github.com/Parsl/parsl/pull/3131/files in slurm.py and in db_manager.py. Those probably come from your first bit of work on this branch where you made the first changes a little bit differently than the PR that got merged (PR #3130). It would be good to remove those changes because they make this PR look more messy than it needs to be, and that can be awkward when someone looks back through git history.

Appreciate the guidance, @benclifford
You're an amazing mentor. I've resolved the changes as per your feedback, and I hope it looks good now.
Thank you.


feat: what is atsign

What I did
Developed a shortcode button for 'what is atSign'. Intended to be a navigator to atSign's description for newcomers.

How I did it
Followed the git workflow and asked discord community for guidance. Went through Hugo's docs to understand some parts of the codebase.

How to verify
Running the server with these changes and navigating to the landing page shows a new section below 'Learn all about how the atPlatform works' and 'I want to...' card group.

Describe the changelog
Made the shortcode button component and linked it to be used in _index.md in content/docs/
Navigation button was showing up as inline, so introduced a line break to show it in a new line.

One line summary
Shows a new 'What is atSign' section on the landing page. Fixes #159

Hey @utkershrajvenshi, thanks for taking this on. You rock 😎
This is what it looks like (for other reviewers)
A few things I think would be great 👍🏼

Add an "i" for info icon within the button

Remove the "Want to learn more about atSign?" prompt because the button already has that info in it (just a small UX enhancement).
As for the company name, it is "Atsign," not "atSign." "atSign" refers to our product while the company name is "Atsign." So please change it to "What is Atsign?"

Hey @TylerTrott. Sorry I somehow missed this notification. I will address the review comments tomorrow.

Hey @TylerTrott. Sorry I somehow missed this notification. I will address the review comments tomorrow.

Hey @utkershrajvenshi we have someone else taking on this issue with great progress as of now in #192
We are only accepting one PR for closing this issue so it is up to you if you want to continue working on this.

Hi @JeremyTubongbanua @TylerTrott , apologies for my less-than-punctual behaviour while working on this issue. I could and should have planned my work more efficiently.
As for the review comments, I have incorporated them and the page looks like below:

I wasn't able to get the exact info icon referenced in an earlier review comment, so had to make do with this one for the time being. Please provide your inputs on it...
Again, I will strive to be better when it comes to ownership of issues...

Thanks @JeremyTubongbanua @XavierChanth @TylerTrott . Could you please also add hacktoberfest-accepted label for easier tracking and verification?


transform data

Excuse me, how do you convert multispectral image to npy file? Thank you very much for your reply.

You can use np.save to save a np array, then you can get a npy file!


performance improvements

reuse meminfo
reuse shape
put input names, input values, output names, output values, run options all in one memory segment
reuse node names
use confined scope
multioutput test

Codecov Report

Merging #66 (19d8bc2) into master (12fd74f) will decrease coverage by 0.02%.
The diff coverage is 95.40%.

@@             Coverage Diff              @@
##             master      #66      +/-   ##
============================================
- Coverage     80.70%   80.68%   -0.03%
- Complexity      307      312       +5
============================================
  Files            39       39
  Lines          1135     1139       +4
  Branches         70       69       -1
============================================
+ Hits            916      919       +3
  Misses          191      191
- Partials         28       29       +1

Impacted Files
Coverage Δ

...main/java/com/jyuzawa/onnxruntime/MapInfoImpl.java
100.00% <ø> (ø)

.../java/com/jyuzawa/onnxruntime/OnnxMapLongImpl.java
100.00% <ø> (ø)

...ava/com/jyuzawa/onnxruntime/OnnxMapStringImpl.java
40.00% <ø> (ø)

...java/com/jyuzawa/onnxruntime/OnnxOptionalImpl.java
0.00% <ø> (ø)

...java/com/jyuzawa/onnxruntime/OnnxSequenceImpl.java
75.00% <ø> (ø)

...va/com/jyuzawa/onnxruntime/OnnxTensorByteImpl.java
40.00% <ø> (ø)

.../com/jyuzawa/onnxruntime/OnnxTensorDoubleImpl.java
40.00% <ø> (ø)

...a/com/jyuzawa/onnxruntime/OnnxTensorFloatImpl.java
100.00% <ø> (ø)

...n/java/com/jyuzawa/onnxruntime/OnnxTensorImpl.java
78.26% <ø> (-3.89%)
:arrow_down:

...ava/com/jyuzawa/onnxruntime/OnnxTensorIntImpl.java
40.00% <ø> (ø)

... and 17 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


:warning: Action Required: Replace Deprecated gcr.io/kubebuilder/kube-rbac-proxy

Description
:warning: The image gcr.io/kubebuilder/kube-rbac-proxy is deprecated and will become unavailable.
You must move as soon as possible, sometime from early 2025, the GCR will go away.

Unfortunately, we're unable to provide any guarantees regarding timelines or potential extensions at this time. Images provided under GRC will be unavailable from March 18, 2025, as per announcement. However, gcr.io/kubebuilder/may be unavailable before this date due to efforts to deprecate infrastructure.

If your project uses gcr.io/kubebuilder/kube-rbac-proxy, it will be affected.
Your project may fail to work if the image cannot be pulled. You must take action as soon as possible.

However, if your project is no longer using this image, no action is required, and you can close this issue.

Using the image gcr.io/kubebuilder/kube-rbac-proxy?
kube-rbac-proxy was historically used to protect the metrics endpoint. However, its usage has been discontinued in Kubebuilder. The default scaffold now leverages the WithAuthenticationAndAuthorization feature provided by Controller-Runtime.
This feature provides integrated support for securing metrics endpoints by embedding authentication (authn) and authorization (authz) mechanisms directly into the controller manager's metrics server, replacing the need for (https://github.com/brancz/kube-rbac-proxy) to secure metrics endpoints.
What To Do?
You must replace the deprecated image gcr.io/kubebuilder/kube-rbac-proxy with an alternative approach. For example:

Update your project to use WithAuthenticationAndAuthorization:

You can fully upgrade your project to use the latest scaffolds provided by the tool or manually make the necessary changes. Refer to the FAQ and Discussion for detailed instructions on how to manually update your project and test the changes.

Alternatively, replace the image with another trusted source at your own risk, as its usage has been discontinued in Kubebuilder.

For further information, suggestions, and guidance:

📖 FAQ and Discussion
💬 Join the Slack channel: #kubebuilder.

NOTE: This issue was opened automatically as part of our efforts to identify projects that might be affected and to raise awareness about this change within the community. If your project is no longer using this image, feel free to close this issue.

We sincerely apologize for any inconvenience this may cause.
Thank you for your cooperation and understanding! :pray:

Updating to use quay.io/openshift/origin-kube-rbac-proxy:4.18 for now

Hi @alegacy,
I can see that you are using kubebuilder: GitHub Link.
I suggest waiting for the next release of the tool and upgrading your project.
You can run kubebuilder re-generate to re-scaffold everything and just add your code on top.
This will also enable the feature to provide certificates for WithAuthenticationAndAuthorization and make your project production-ready. Currently, using kube-rbac-proxy with WithAuthenticationAndAuthorization without certificates is not ideal for production.
See my PR adding this feature: PR #4400.
But WithAuthenticationAndAuthorization is a better option:

Eliminates the need for the deprecated kube-rbac-proxy image, reducing complexity.
Ensures updates, bug fixes, and improvements in Controller-Runtime are naturally integrated.
Reduces risks by removing dependency on external images like kube-rbac-proxy.
Minimizes extra checks for updates or compatibility, streamlining development workflows.

The only downside that I hope will soon be addressed is tracked in: https://github.com/kubernetes-sigs/controller-runtime/issues/2781. We need to improve it to add  /healthz endpoint on the metrics server and add support for alwaysAllow paths (e.g., /healthz) and alwaysAllowGroups (e.g., system:masters) to prevent metrics outages during kube-apiserver networking issues, ensuring availability during critical times.

@camilamacedo86  Looks like this is specifically targeted to only handle the authn/authz for the metrics endpoint.   While we do have some use cases where this can be implemented using the method provided (i.e., on the metrics endpoint of our operator) we have other use cases where this won't be possible -- or at least doesn't appear to be a simple drop-in replacement.
For instance, we currently use the kube-rbac-proxy to do authn/authz for non-metrics API endpoints which are implemented by our microservices.  These are not metrics endpoints.  These are application level API endpoints.  We currently use kube-rbac-proxy to protect those rather than owning more code to implement that functionality directly in our microservices.
Yes, it would be possible to write those handlers in a similar way as the filters.WithAuthenticationAndAuthorization and have them work for our non-metrics API endpoints, but we do anticipate needing to leverage the OIDC capabilities of the existing kube-rbac-proxy functionality to secure our application using an external OIDC authentication server.
Again, yes, we could write custom handlers to do that but short of a ready-to-go drop-in replacement we'd have to duplicate and own code that already exists in kube-rbac-proxy.
Is there a ready-to-go library implementation that we could use that would instantiate an HTTP middleware filter (e.g., func(http.Handler) http.Handler) capable of doing OIDC to an external OIDC server?

Hi @alegacy
However, the most important here is that you cannot use the upstream image gcr.io/kubebuilder/kube-rbac-proxy; any version of this project using it will be impacted when the image disappears. That means your solution will not work at all (unless the image is found in the node cache or it is running in a cluster with all mirrored)
The feature in the controller runtime seems like target authentication and authorization primarily for metrics endpoints. You can raise an RFE there or look for alternatives. But if you are happy with the kube-rbac-proxy solution and it already provides what you need, you might stick with it. However, unfurtunilly, you can no longer use the image from this Registry.
If you already replaced used it from another place (at your own risk) and are happy with please feel free to close this one.


assertion failed: !self.0.access(cs).freeq.is_empty() in rtic-sync 1.0.1 when trying to send data to channel

panicked at 'assertion failed: !self.0.access(cs).freeq.is_empty()', /home/serg/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rtic-sync-1.0.1/src/channel.rs:301:17

I've tried checking the code myself, but it is not trivial, so any help would be appreciated

Hi, thanks for the report!
Do you have an example we can use to reproduce this?
I checked tests and this assert is quite well tested so I'm not sure what's wrong, any help to narrow it down would be appreciated!

I was hitting it consistently for some time and now it's gone.
I'll try to find a case where it is reproducing consistently and show you how to reproduce it. I think it was related to publishing messages before reader started to read them.

Thanks, I hope you can trigger it again!

Hi @pistolero I just wanted to check-in if you ran into this again?
I have a theory on what can happen, and would like to check if this is similar to what you did.
If you at a higher priority use try_send and at the same time use send at a lower priority I think the free queue can become inconsistent if hit just right.
I've not been able to reproduce it in tests though...
Were you doing something similar to this? I've tested extensively with different dropping/forgetting of the receiver and were unfortunately not able to reproduce :(

Not yet, had no chance to get back to this 😓
I think it was happening when queue was full, I had a software thread which was posting data to queue periodically and this assertion was failing when receiver of the queue were blocked (there were no Ethernet connection).


Editorial comments

The word "also" is overused (21 instances).  Taking a fresh read, it really stands out.  Suggest removing it in almost all instances to help tighten the language up.

Acknowledged.

All changes have been incorporated.