Grails 7: render(text: "...") not working

When using render(text: "...") in a controller, Sitemesh tries to find a layout and renders a 404:
2024-11-18T11:52:23.494+01:00 ERROR 743942 --- [o-auto-1-exec-8] org.sitemesh.webapp.WebAppContext        : Error 404 processing decorator '/layouts/_none_'

See example project with tests: https://github.com/matrei/sitemesh-bug-render-text

@matrei it sounds like the issue here isn't render(text:) it is that you are setting the contentType to text/html and sitemesh is configured to decorate all text/html content.

Isn't it valid to render adhoc html templates (i.e. for htmx)?  I don't think sitemesh should assume it needs to decorate if there's not a layout.

If a layout is not located using any of those conventions, the convention of last resort is to look for the application default layout which is grails-app/views/layouts/application.gsp.

https://gsp.grails.org/latest/guide/layouts.html

I would only expect this behavior if a view is being rendered.
In grails 6.2.x, if you have this controller:
class TestingController {
    def index() {
	    render text: 'Testing', contentType: 'text/html'
    }
}

It renders Testing and does not use the default layout.  In 7.x with this plugin, it's trying to render a layout - even though one doesn't exist.

I guess the other question is: does sitemesh also decorate templates now?

Does setting "text/plain" work?

@codeconsole Yes, that works.
I have updated https://github.com/matrei/sitemesh-bug-render-text with more tests to show that.
However, render(text: '
Hello World
', contentType: 'text/html') has been working for previous versions of Grails in grails-functional-tests since 2015.
https://github.com/grails/grails-functional-tests/blob/2df395bf60d42c4d596837453dc717e8c3d638e4/app1/grails-app/controllers/functionaltests/UploadController.groovy#L14

well, an easy solution for this would be to not have a default decorator.   In sitemesh 3 you can decorate anything, including static html pages.

well, an easy solution for this would be to not have a default decorator.

Would that be backwards compatible with how Grails < 7 works?

well, an easy solution for this would be to not have a default decorator.

Would that be backwards compatible with how Grails < 7 works?

no, but making everything work how it worked previously isn't necessarily a good thing.   The correct decision is figuring out which is the best way it should work.   Having a default decorator, from what I remember, is inefficient because it assumes every html page will be decorated.

I'm not as familiar with the updated sitemesh.  Do you have any documentation on the default decorator that I can refer to?  I assume the idea is you'd have to include the layout in the gsp if you want it decorated or in the render command?
In general, I'm not a fan of decorating explicit render(text: 'text here') calls from the controller.  If someone says I want to render some text, I'd only ever expected it rendered.  If they want to add the layout, they can always call a view or explicitly set the layout argument.
Another possible solution is to only apply the layout when it's a view, or if the layout: '' parameter is set.  This may be the closest to pre Grails 7 functionality and would make the most sense in my mind.

Looking more at the error message:
Error 404 processing decorator '/layouts/_none_'

I don't think the issue here is with the plugin. It is the fact that a layout is being set to _none_ and site mesh has no concept of a none layout.  If the layout was not set to none, there would be no error.
https://github.com/grails/grails-core/blob/cc5d4cae8d8453824849957caef9ae6635b2a52b/grails-web-common/src/main/groovy/org/grails/web/util/WebUtils.java#L64
https://github.com/grails/grails-core/blob/cc5d4cae8d8453824849957caef9ae6635b2a52b/grails-plugin-controllers/src/main/groovy/grails/artefact/controller/support/ResponseRenderer.groovy#L535-L544

I think we should consider removing all sitemesh references in ResponseRenderer. I am not sure if it makes sense to compile sitemesh logic into RestConteoller and Interceptor

Fixed by grails/grails-core#13908


Match object recursively (toMatchObject)

The discussion on recursive matching suggested here https://github.com/facebook/jest/issues/1793#issuecomment-250909796 was never continued in the PR. As far as I can tell toMatchObject does not match recursively, but it would be nice if this was a possibility.

@ismay How do these examples adapted from the unit tests for toMatchObject compare to your expectations for recursive matching, pardon the pun :)
describe('toMatchObject', () => {
  it('matches recursively', () => {
    expect({
      a: 'b',
      t: {
        x: {
          r: 'r'
        },
        z: 'z'
      }
    }).toMatchObject({
      t: {
        x: {
          r: 'r'
        }
      }
    });
    expect({
      a: 'b',
      t: {
        x: {
          r: 'r'
        },
        z: 'z'
      }
    }).not.toMatchObject({
      t: {
        l: {
          r: 'r'
        }
      }
    });
  });
});

Maybe I should have worded it differently, but what I mean is it matching:
x: {
  r: 'r'
}

In the test you mentioned, even though that's nested under t. To me that would be useful when testing if an object needs to contain something but you don't care about where exactly.

@ismay are you willing to ship a PR on that?

Definitely willing, but no time atm I'm afraid.

Cool, take your time, no worries! 🙂


Support building Fortran libraries

Would it be possible for this crate to support building Fortran code too?
It would allow to use existing Fortran code from Rust, even if this code is not available from the system: for example using a custom BLAS/Lapack on Windows, or some research code that might be hard to get otherwise.
I think that the current approach in this crate will work with Fortran compilers too: there are 3-4 majors compilers that uses different flags, and it should be possible to pattern-match on them to create the command line invocation.
It can also make sense to have a separated crate for this, but it would be largely copy-pasted from this one.
What do you think? I might try to contribute this if/when I find some time.

Seems reasonable to me!

I'm going to close this for now since I don't think it's something we necessarily want to actively add and I won't be pursuing myself, but PRs are of course always welcome!


Added option to let batch conversions continue on error

Closes #11

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          150       179   +29
=========================================
+ Hits           150       179   +29

Impacted Files
Coverage Δ

sas7bdat_converter/converter.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Deleted font sumernote.eot still referenced

The font file summernote.eot has been removed but it is still referenced by summernote.css.

Thanks for reporting. I'm handling this now...

Found a related issue: https://github.com/sapegin/grunt-webfont/issues/367

Just released <IP_ADDRESS>. Thanks!


Add noindex setting and default to noindex all pages in the starter kit

I've noticed there are quite a few Clean sites deployed unmodified, or partially modified, to the web. This leads to polluting search results for content the starter kit includes.
For example, searching "The Clean Starter Kit for Umbraco" includes results of the Clean Starter Kit default "about" page deployed by various people, unmodified.
I'm concerned that this is and will pollute search queries for conferences, meetups, 24 Days, Skrift, etc and flag these sites as duplicate content, damaging these site's rankings going forwards.
One option would be to replace all the content with lorem ipsum, or another would be to make all the default pages "noindex" so it'd be a conscious decision to allow search engine discovery of pages.

Hi @glombek
Sorry for the late reply, I did read your issue but got distracted and forgot to come back to it.
I think the best solution for this would be to add properties for index and follow and have them set to false on the sample content but default to true on new content.
I'll see if I can squeeze it in today, I need to do an update to include a post about umb.fyi too

That's a nice solution!
Thanks for contemplating my niche situation!


desync between users and server?

Discussed in https://github.com/itzg/docker-minecraft-server/discussions/1997

Originally posted by Traptonian March  8, 2023
i am using modded, but when im playing with my friend on a server and he opens a door it opens for him but not me? and then when i try to open the door it closes for me but not for him but he still cant get through?

This has nothing to do with running containerized.
Please don't open a discussion and an issue without waiting for some kind of response.


px4fmu_common rcS: fix MAVLINK_F test

fixes a 'test: syntax error' message on startup. There were two problems:

the expansion of $MAVLINK_F lead to multiple arguments in the test
when the variable contained spaces. Fixed with ""
the x prevents interpretation as a unary expression, if $MAVLINK_F starts
with a - character (in that case the expansion would be:
if [ -r 1200 ... and nsh interprets - as unary expression)

Makes sense, merged.


Automate cloud cluster stop on a schedule

The idea here is to be able to stop clusters during off hours to reduce costs.  This looks possible on Azure:
https://docs.microsoft.com/en-us/azure/automation/automation-solution-vm-management

CLOSING: This should be a NEONCLOUD feature rather than relying on cloud specific capabilities.  We can revisit this later if/when it becomes important.


Question: In PD cluster, whether the PD Leader will be the receiver for TiKV region/store heartbeat

After a quick look at PD source code, a question come to my mind.
In server/leader.go#leaderLoop, it seems that only one PD will be the leader, others will watch it util the PD Leader fail or be resigned, Does it means that no matter how PD instance I have started in the PD Cluster, the really working PD is only the leader? When TiKV send region/store heartbeat , does the PD Leader will be the only receiver?
If so, will the PD be performance bottleneck when TiKV cluster being enlarged and data grows up explosively?

Hi @yeyuqiang , thanks for your feedback.

When TiKV send region/store heartbeat , does the PD Leader will be the only receiver?

Yes. You may think other pd-servers are for disaster preparedness.

Will the PD be performance bottleneck when TiKV cluster being enlarged and data grows up explosively?

Yes. It could overload if there are too many regions. In our test environment, pd-server can smoothly handle heartbeats for about 500k regions.
If you observe the pd-server's CPU load is too high in a large cluster, you may consider adjusting tikv's region heartbeat config to a longer interval.

thanks for your reply.


Error on client.Activities.GetAthleteActivities

Same issue as:-
https://github.com/gabornemeth/StravaSharp/issues/3
but with these activity types.
"Error converting value "VirtualRun" to type 'StravaSharp.ActivityType'.
"Error converting value "Golf" to type 'StravaSharp.ActivityType'.
"Error converting value "Handcycle" to type 'StravaSharp.ActivityType'.
"Error converting value "Soccer" to type 'StravaSharp.ActivityType'.
All activities done between June 2019 and July 2019. So none of them are old.
So can't read 38 user's data (out of just over 16000 athletes).

There are also sailing, skateboard, velomobile and wheelchair.
PR incoming...

PR24 fixes this... awaiting attention from the powers that be to get it published

PR #24 is merged, this can be closed I guess?


chore: Bump spanner version

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.44%.  Consider uploading reports for the commit 8807686 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage   74.44%   74.44%
=======================================
  Files          34       34
  Lines        1119     1119
=======================================
  Hits          833      833
  Misses        286      286

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Android back button and command Navigator.of(context).maybePop(); beams to notFoundPage

Android back button and command Navigator.of(context).maybePop(); beams to notFoundPage
'/product/:id': (context, state, data) {
final productId = state.pathParameters['id'];
// productId sample 'ptk-1542'
int id = int.parse(productId.toString().substring(4));
return Product(id);
},
`
I checked the app with the Chrome browser, for example, when the app is in a certain path like '/product/5421', pressing the android back button or  Navigator.of(context).maybePop(); command returns to a path like '/product', which does not exist and gives the error not found.
And the next problem is that the starting address of my application is a splash screen and after two seconds I navigate the app to the home page, but pressing the back button returns to the splash screen instead of closing the application.
'/': (context, state, data) => const Splash(),
'/home': (context, state, data) => const FirstScreen(),
To transfer to the home page, I have written the following code in the initState of the splash page
Future.delayed(const Duration(seconds: 2)).then((c) => context.beamToReplacementNamed('/home'));
Beamer version: (^1.5.0)

Hey @bardiaabasirad :wave:
Thanks for creating an issue!

command returns to a path like '/product',

You can solve this by wrapping Product(id) screen in BeamPage and specifying popToNamed: '/'. Furthermore, this will get improved soon, so there will not be a need for those "hidden hacks".

but pressing the back button returns to the splash screen instead of closing the application

This seems to be related to default behavior of RoutesLocationBuilder which thinks that / and home are stacked. Zou can solve this in multiple ways

by defining your own BeamLocation where you would not return a stack of / and /home
by renaming / to /splash
by using context.beamToReplacementNamed('/home', stacked: false)

and similarly, this will also get improved soon in v2


Added MemeCommand to commands

I didn't see a problem with the website, it's working for me and making content. Now we can create some dank memes a la issue #212 !
To test: You have to input it correctly to display right in the url. the command formatting reads whitespace and is like this: /meme memename top-content bottom-content.
Ex: /meme goodguygreg this-is-the-top-text this-is-the-bottom-text
It should display correctly with show/hide and display the picture formatting correctly.

@ashanhol This was a fun one to test! works as expected. The list of memes really helps. :shipit:


Add CoinGecko Crypto Price app

add support for fetching and displaying the price data and image for any CoinGecko-API-supported coin
use typeahead search to find cryptocurrency to display price of and dropdown to display currency against
give option to show/hide price for up to two other currencies and show/hide the currency symbol and currency code for each
caches a specific coin data every 10mins (600secs) and caches the list of coins, currencies, and images once a day (86400secs)
based on @saltedlolly's digibyte-price app

I have read the CLA Document and I hereby sign the CLA

recheck

This looks awesome man


code formatting for python files

1, Formatting the code according to PEP8.
2, When looking into the Trace2TimeSeries problem, I am not sure if it is a bug in the Trace2TimeSeries function in util.converter.py from line 247 - 249, but I changed it to what I understand.
    # we don't require endtime in TimeSeries so ignore if it is not set
    if 'endtime' in trace.stats:
        t = trace_stats_dict['endtime']
        trace_stats_dict['endtime'] = t.timestamp

Have you considered formatting the codes using black. I think it's widely used for Python projects.

yes, I might integrate that into our CI here, but this branch is not only for formatting issues that can be handled by a formatter. We need to manually clean up the code and then apply the formatter.


False Positives for Anchor Links to Section Headers Following Fenced Code Blocks

If a section header follows a fenced code block, then markdown-link-check erroneously flags anchor links to that section header as 404s.

markedjs/marked#2483 makes me wonder if this can be fixed simply by updating marked to 4.0.17+.

Now that the upstream issue has been fixed, this issue can be addressed by upgrading marked to 4.0.18+.

should be fixed in latest release. if not, please re-open.


Equipped armor is deleted when mk5 is equipped

Any armor you have on just disintegrates as soon as you use the case
Either block the case from being used with armor on, or make equipped armor drop into your inventory/ground when case is used

first one ill do


It is possible use wrap_content height viewpager?

I need use this but ....

Without swipeable (open/close) - only animation show and dismiss
Wrap_content height
Replace bottom to top

I don't understand the desired result or the problem.


[DO NOT MERGE] Removed move from down

In the WebDeviceInputSystem, when we receive a pointerdown event, we will call the observers associated with down events (from within _pointerDownEvent) and then call the ones associated with move.  After looking more closely at this, I believe that we do not need to do this as the pointer position will still be updated in the InputManager's down code.  This PR removes this move call from the code that handled the pointerdown events.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14637/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14637/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14637/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14637/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14637/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14637/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14637/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14637/merge#BCU1XR#0

Closing for now.  Will reevaluate later


Fixes https://github.com/godotengine/godot/issues/80509.
Apparently the pr https://github.com/godotengine/godot/pull/76212 that was made to fix https://github.com/godotengine/godot/issues/76195 was wrong. I now tried with both MRPs with this pr and it fixes both for me. No "inexplicable holes" show up and the heightmap-from-texture also bakes its half-sphere correctly but more testers are welcome to finally close this heightmap bug-juggle-chapter.

Thanks!


Void type

Summary
The specification, examples, and grammar already allowed subroutines and externs to not return anything. This PR formalizes that the return type when no return value is provided is void, an unrealizable and uninstantiable type.
Details and comments
The grammar, parser, and associated tests have also been updated.

Minor comments aside, I'm happy with the technical content of this PR. I'm curious what the motivation for it was, though? We don't have pointer types or first-class functions, so I can't immediately see the need to be able to name the unit type.
The consistency is nice, but I'm not sure -> void is any clearer than the absence of a return type. As we use the keywords def or extern to introduce subroutine declarations, it's also superfluous grammatically (unlike in C, say).

void does not denote, or signify, the absence of a type. Quite the opposite.
Also, there is no such thing as the absence of a return type for functions. All functions return a type. There are not two distinct categories of functions - ones that return a type,  and others that don't.
If a function ever were not to return a type, it would never return.
Returning void from a function is indicated either by the special statement
return;

with no operand, or, by the absence of a return statement, in which case the function implicitly returns void after the last statement.
Just like when writing
def function_with_void_return(int i, double d) {
  // code goes here
}

the absence of an explicit return type after the right paren does not mean that the function does not return. It only means that the function returns void.
By definition, void is an incomplete type that does not have a size, cannot perform arithmetic and cannot be either an lvalue or an rvalue.
A void type and a pointer-to-void type are two different things that have very little in common. Since OQ3 does not have pointers of any kind, it cannot have pointers to void either.
A function that appears not to return anything is a function that returns this special void type that cannot be materialized.

By "absence of a return type", I was implying "absence of a specified return type".  I understand the distinction between zero types and unit types.
My additional comments about pointers and first-class functions were to contrast OQ 3 to other languages.  As I also said, OQ 3 doesn't have such objects, which are some of the reasons usually used to justify spelling out a unit type.  Since OQ 3 doesn't have them and in the one circumstance void could be valid in this PR, it's also the "default" (by omitting the -> <type>), I'm just curious as to what other reasons were discussed about why we might want to add the extra keyword.

I'm just curious as to what other reasons were discussed about why we might want to add the extra keyword.

Because the current Spec does not explain what is being returned from a function that does not appear to have a return type. Which implies there exists an assumption that functions that do not specify a return type do not return anything.
That assumption is incorrect. As I have already explained, functions must return something. For functions that do not specify a return type, the void type is the implicit something being returned.

For functions that do not specify a return type, the void type is the implicit something being returned.

This is really the essence of the PR in my mind. We are specifying something concretely that was hinted at before, just to ensure that there is no ambiguity possible.
And yes, Jake, I didn't make it clear enough in the first draft that the non-specified version is implicitly returning void, I'll amend the PR today to address that.

That's fair.  Just to be clear, I'm not questioning implicit returns of a void type, and I'm fine with the additional text surrounding such functions to remove ambiguity.  I'm only questioning if we need to add explicit -> void syntax.  Apologies if that wasn't as clear as it could have been.  I don't feel strongly about it either way.

I'm only questioning if we need to add explicit -> void syntax.

Nope. No syntax changes are necessary.The compiler will take care of figuring out that it's a void return type.

I would be fine removing the syntax changes if desired.

At the TSC meeting today we decided to leave the syntax changes as is. The current version is ready for review by all.

Per the discussion at last week's TSC meeting, I have removed the optional void return signature. All grammar and parser changes have been reverted and the specification text has been updated accordingly. The void keyword is now reserved for possible future use, so I added it back to the lexer, but there are currently no valid uses of it.

This PR is now ready for final review/approval by the WG members and TSC.


Can't test it at the moment, general questions

Firstly just wanted to say it looks cool, and looks like it works well!
I saw in the video on the vrchat discord that the feet aren't moving on the platform, are you teleporting or using a station to move them? How does the syncing work across clients? If it's using a station to move them, how does it handle turning? (otherwise if it's teleport that's an easy question: you don't have to handle turning)

Thanks for the compliments.

The feet were not moving cause I was in fullbody with locomotion off. Recorded a quick example in desktop

https://github.com/jetdog8808/Platform-Link/assets/36026251/2504047b-b9a2-49c6-95cc-d77b715a191f

I'm teleporting to move the player.
Syncing is done with vrchats builtin ik sync. So there will be about a half second delay between clients.


Update sbt-scoverage to 2.0.11

About this PR
📦 Updates org.scoverage:sbt-scoverage from 2.0.10 to 2.0.11
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (2.0.10).
You might want to review and update them manually.
website/package-lock.json

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scoverage", artifactId = "sbt-scoverage" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Done in https://github.com/data-tools/big-data-types/pull/414


[CodeClean] fix svace issue

Check null ptr when getting n-th tensor info.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #431. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Not working in VS code.

Extension Name: debug-auto-launch
Extension Version: 1.0.0
OS Version: Windows_NT x64 10.0.17134
VSCode version: 1.28.0

We have written the needed data into your clipboard. Please paste:

What is not working? Please provide more details.

This looks like a consequence of #60318


won't start up on non-SMP VM which is suprisingly a default

Ubuntu 13.10, LeoFS 1.0.0 binary package:
# ./leo_storage/bin/leo_storage console
Exec: /usr/local/leofs/1.0.0/leo_storage/erts-5.10.4/bin/erlexec -boot /usr/local/leofs/1.0.0/leo_storage/releases/1/leo_storage -mode minimal -config /usr/local/leofs/1.0.0/leo_storage/etc/app.config -args_file /usr/local/leofs/1.0.0/leo_storage/etc/vm.args -- console
Root: /usr/local/leofs/1.0.0/leo_storage
Erlang R16B03 (erts-5.10.4) [source] [64-bit halfword] [async-threads:32] [kernel-poll:true]

=INFO REPORT==== 9-Apr-2014::23:57:01 ===
    alarm_handler: {set,{system_memory_high_watermark,[]}}
id:leo_logger_file_i, path:"./log/app", filename:"info"
* opening log file is ["/usr/local/leofs/1.0.0/leo_storage/log/app/info.20140409.23"]
id:leo_logger_file_e, path:"./log/app", filename:"error"
* opening log file is ["/usr/local/leofs/1.0.0/leo_storage/log/app/error.20140409.23"]
enif_send: env==NULL on non-SMP VM[os_mon] memory supervisor port (memsup): Erlang has closed
[os_mon] cpu supervisor port (cpu_sup): Erlang has closed
Abort (core dumped)

...after enforcing the -smp VM flag, everything starts to work. The problem is that the flag should be added via cuttlefish which I don't know enough (yet) to make a patch. (Isn't the SMP default when no SMP flags are supplied? Does it mean that the ERTS shipped with leo_storage is not SMP intentionaly?)
I think the SMP VM should be default for all the apps -- at least until there is a good reason to stay with non-SMP. Is there such a reason in this case?
Why the non-SMP does not work? Is it a bug in the VM itself?! (It seems it has something to do with os_mon.)

We find this issue on leo_manager_0 with leofs-1.2.21 along with Centos7.0. also
=====
===== LOGGING STARTED Wed Apr 13 21:14:34 EDT 2016
=====
Exec: /usr/local/leofs/1.2.21/leo_manager_0/erts-<IP_ADDRESS>/bin/erlexec -boot /usr/local/leofs/1.2.21/leo_manager_0/releases/1/leo_manager -mode embedded -config /usr/local/leofs/1.2.21/leo_manager_0/etc/app.config -args_file /usr/local/leofs/1.2.21/leo_manager_0/etc/vm.args -- console start
Root: /usr/local/leofs/1.2.21/leo_manager_0
Erlang/OTP 17 [erts-<IP_ADDRESS>] [source] [64-bit halfword] [async-threads:32] [kernel-poll:true]

id:leo_logger_file_i, path:"./log/app", filename:"info"
* opening log file is /usr/local/leofs/1.2.21/leo_manager_0/log/app/info.20160413.21.3
id:leo_logger_file_e, path:"./log/app", filename:"error"
* opening log file is /usr/local/leofs/1.2.21/leo_manager_0/log/app/error.20160413.21.3
id:log_id_history_log, path:"./log/app", filename:"cmd_history"
* opening log file is /usr/local/leofs/1.2.21/leo_manager_0/log/app/cmd_history.20160413.21.3
enif_send: env==NULL on non-SMP VM

@PatelParas thanks for reporting.
As we have adopted leveldb as a backend for leo_mq from recent versions,
we need to enforce the -smp VM flag also on others(leo_manager, leo_gateway).


Enhancement/265/Redo-FAQ-section-with-new-styling

@ABCodez

I believe the buttons can be smaller in the mobile view, so that they are not taking too much space
also, try to make the accordion item text left align, somehow when the text break into another line, the text shift as well
can you try to remove the accordion card expand transition, it is the main reason why it felt janky

@teoh4770, I made a few changes based on ur feedback ! Everything should look a bit more cleaned up + smaller on mobile. The categories section still looks a bit big on mobile, the original design had it be an accordion as well but, I feel like that'd just be too much lol.
Check it out tho, and lmk what ya think :)

@ABCodez is better now for sure, but you can make them more consistent. You can improve the tab section in mobile view, by making it smaller(than the accordion) and draw less attention. Keep the ratio between the tabs and the accordion in mobile view the same the desktop view.

New styling for FAQ

Way better than the OG one. But personally thought that the heading size is slightly bigger? You can refer to the other section's header to see if the size is aligned.


Perfkit can't ssh to VMs once it is created in OpenStack environment

Running below command in OpensStack environment

./pkb.py  --cloud OpenStack --machine_type=m1.small --openstack_network=external-network-shared --os_type=centos7 --benchmarks=ping

Perfkit is creating VMs successfully but can not ssh into it for ping or iperf benchmark test. Getting Permission denied when it is trying to ssh to VMs.

2019-12-12 17:46:36,301 2dee6931 Thread-1 ping(1/1) vm_util.py:364 INFO     Running: ssh -A -p 22<EMAIL_ADDRESS>-2 -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o PreferredAuthentications=publickey -o PasswordAuthentication=no -o ConnectTimeout=5 -o GSSAPIAuthentication=no -o ServerAliveInterval=30 -o ServerAliveCountMax=10 -i /var/folders/sc/l_g86zr57qg0pr7jbxj3c3340000gn/T/perfkitbenchmarker/runs/2dee6931/perfkitbenchmarker_keyfile -o ControlPath="/var/folders/sc/l_g86zr57qg0pr7jbxj3c3340000gn/T/perfkitbenchmarker/runs/2dee6931/ssh/%h" -o ControlMaster=auto -o ControlPersist=30m hostname
2019-12-12 17:46:36,608 2dee6931 Thread-2 ping(1/1) vm_util.py:364 INFO     Running: ssh -A -p 22<EMAIL_ADDRESS>-2 -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o PreferredAuthentications=publickey -o PasswordAuthentication=no -o ConnectTimeout=5 -o GSSAPIAuthentication=no -o ServerAliveInterval=30 -o ServerAliveCountMax=10 -i /var/folders/sc/l_g86zr57qg0pr7jbxj3c3340000gn/T/perfkitbenchmarker/runs/2dee6931/perfkitbenchmarker_keyfile -o ControlPath="/var/folders/sc/l_g86zr57qg0pr7jbxj3c3340000gn/T/perfkitbenchmarker/runs/2dee6931/ssh/%h" -o ControlMaster=auto -o ControlPersist=30m hostname
2019-12-12 17:46:36,856 2dee6931 Thread-1 ping(1/1) vm_util.py:421 DEBUG    Ran: {ssh -A -p 22<EMAIL_ADDRESS>-2 -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o PreferredAuthentications=publickey -o PasswordAuthentication=no -o ConnectTimeout=5 -o GSSAPIAuthentication=no -o ServerAliveInterval=30 -o ServerAliveCountMax=10 -i /var/folders/sc/l_g86zr57qg0pr7jbxj3c3340000gn/T/perfkitbenchmarker/runs/2dee6931/perfkitbenchmarker_keyfile -o ControlPath="/var/folders/sc/l_g86zr57qg0pr7jbxj3c3340000gn/T/perfkitbenchmarker/runs/2dee6931/ssh/%h" -o ControlMaster=auto -o ControlPersist=30m hostname}

ReturnCode:255
STDOUT:
STDERR: Warning: Permanently added 'X.X.X.X' (ECDSA) to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey,gssapi-keyex,gssapi-with-mic).

I solved a very similar issue for myself.
first use queens version of packstack if you are using packstack, I ran into issues with stien.
Second and most important for SSH issues:
try lowering the MTU on your physical interface for the packstack/openstack server and the client you perform tests from.
I was able to debug and confirm this issue by running:
 ssh -vvv<EMAIL_ADDRESS>\
-2 -o StrictHostKeyChecking=no -o IdentitiesOnly=yes \
-o PreferredAuthentications=publickey -o PasswordAuthentication=no \
-o ConnectTimeout=5 -o GSSAPIAuthentication=no \
-o ServerAliveInterval=30 -o ServerAliveCountMax=10  \
-i /tmp/perfkitbenchmarker/runs/bccaf369/perfkitbenchmarker_keyfile

from the same server I was running the gperfkit tests.
the command I used on the packstack server to lower the MTU:
ifconfig enp2s0 mtu 1400
ifconfig br-ex mtu 1400

The bridge adaptor setting may not be required but I did it to be safe.
I also performed the same on the gperfkit host.
After reading through the source a bit, I believe this is the correct command to run against openstack:
./pkb.py \
--cloud=OpenStack \
--machine_type=m1.small \
--image='ubuntu-16.04' \
--benchmarks=ping --openstack_network=private \
--openstack_floating_ip_pool=external_network  \
--os_type=debian
note: I found a potential issue in the code where --openstack_network and --openstack_private_network (depreciated) actually check for a network named private and not the network supplied in the command line arguments.
I renamed private_network in horizon to private. I need to do further digging to confirm but I have it working finally.
note (2): I dont believe modifying the source to use ubuntu-16.04 is necessary and it does appear to slow down benchmarks by repeatedly trying to install the package "python" which does not exist in ubuntu 16.04+. This does not cause the benchmark to fail just slows it down.

So i would say your setting --os_type=centos7 but your not passing --openstack_image_username=centos
in your output its showing its attempting to log in with the user ubuntu, which if your using the centos images available for openstack, wont work as the user required is centos.

So i would say your setting --os_type=centos7 but your not passing --openstack_image_username=centos
in your output its showing its attempting to log in with the user ubuntu, which if your using the centos images available for openstack, wont work as the user required is centos.


Expose unit tracks as SuSample *

Synchronisation of music and visuals is a costly task; It currently requires the shader to contain the sequence and additional code to evaluate into "beat count, smooth beat count, beat envelope" values.
Would it be possible to expose envelopes tracks from Sointu to feed into the shader as texture for synchronization? I understand this requires additional memory and initialization code; so maybe this feature could be turned off by default and switched on per unit?
This would require a similiar infrastructure to what's required for #61 or #67, but for the exported assembly code.

There's the SYNC opcode which allows sending any signals to sync data, and sointu does not define this symbol, so you can make this symbol available anywhere in the code e.g. block of memory that you send as a texture the shader. Check out Adam intro on how this was done.
Does this solve your issue?

This is exactly what I wanted to do; thanks!
(This file gives info on how this works - this is in https://github.com/vsariola/adam/blob/main/intro/main.c#L1)


Fyusenet works in OpenGL but not in GLES2 and EGL

Good Afternoon,
I’ve been working with a target board with PowerVR GPU. The target board just has OpenGLES and EGL and doesn’t have OpenGL / glx / glfw / GLUT / GLU / GLVND etc.
I’ve been able to run the sample resnet, provided in fyusenet, successfully on a host Ubuntu system with OpenGL, glx and GLVND, but in the target board, the output of the resnet is always coming out to be the first class. I tried printing the intermediate layers to narrow down which layer is causing the issue and it came out to be the second last layer, which is defined in a file called deepgemmlayer.cpp. I used the enableIntermediateOutput() API in the fyusenet library and converting the binary raw intermediate feature maps to decimals. I think it has something to do with how the shader is getting executed by GLES-EGL combination. I've tried changing the precision values in the .vert and .frag shader programs assuming that the board hardware might not support all the operation required in the gemm layer, but it also ends up giving the same all zero output. Though the assumption was more on a test and try basis, as all the previous layer's feature maps are coming out to be non-zero floating points, similar to the host Ubuntu system.
Can anyone help me find out the issue why the shaders might be behaving in such a way that the layer’s output becomes all 0?
Thanks in advance for any help.

Hi @IssacXid ,
Sorry for getting back at you so late. First of all, can you tell me what GPU and OS you are running on ? On StackOverflow I saw that you mentioned PowerVR, which model exactly ?
Also, I have a personal fork with a bunch of updates, can you reproduce the same problem on that ?
If that problem persists we can look a bit deeper and try to figure out what's going on.

I got my hands on this over the weekend.
It has the IMG BXE-4-32 MC1 GPU, which should be quite a bit slower than the one that you are using, still working on getting the GPU drivers to work properly :-)
I saw that you mentioned GLES 2.0, I would like to note that FyuseNet requires GLES 3.0 to work. Though it should be possible
to get it to work with GLES 2.0 (with some adaptions on the shaders), I would not recommend it. Given that only the point-primitive layer in the end causes problems, I assume you are running under GLES 3.0 (it would have failed way earlier otherwise).
I already have an idea what could be wrong, will test it here locally, otherwise I will provide instructions here (should be a one-line fix).

Hello,
Thanks for testing it out while I couldn't access the board. I got it running, cloned and built the modified repository with some changes:

Setting the flag BUILD_TESTS OFF as it results in fetching gtest repository as we cannot connect the target board to internet.
In CMakeLists.txt, I replaced the arguement OpenGL in find_package() with OpenGLES and EGL, for which I've added the FindOpenGLES.cmake and FindEGL.cmake in the cmake directory.
Changed some occurences of GLES/gl.h with GLES2/gl2.h, GLES/glext.h with GLES2/gl2ext.h and GLES3/gl3ext.h with GLES3/gl2ext.h as per the files available in the target board.
Before reaching to the point of replicating the deepgemmlayer.cpp, I'm trying to debug another error that I didn't face with the original fyusenet repository:

  - GL_OVR_multiview2
terminate called after throwing an instance of 'fyusion::opengl::GLException'
  what():  /home/root/github_issue/fyusenet/fyusenet/base/buffermanager.cpp:724 [fyusion::fyusenet::BufferManager::Textn
Detailed error: Cannot parameterize texture (err=0x502)

Aborted

Do you know why this is occurring? I'll try to debug it in the meantime.

Hey @IssacXid ,
So to your point 2, that is actually a good idea. Would you mind setting up a PR into my fork with the changes in CMakeLists.txt and the cmake modules ?
Point 3 puzzles me a little bit. It is possible that different Linux distros handle things differently, but debian and Ubuntu put the headers into /usr/include/GLES. They do however also have copies in the other folders you mentioned. Perhaps I should do some homework on which pattern is more common among different distros and settle for a common theme here.
The error (0x502) that you see there is a long standing nuisance in my codebase which has to do with GLES and the support of RGB textures (as write targets). Don't get me started on that :-)
I set up a PR here which fixes that issue as well as another issue that I discovered when testing on the RISC-V board.
Your initial problem (the zero output in the GEMM layer) was already fixed in my fork, it was a simple line in the vertex shaders:
gl_PointSize = 1.0

Not all drivers seem to use the default (1.0) when not explicitly instructed to.
But there was another bug that only occured on weaker GPUs, which yields wrong results in convolution layers with larger kernels. That is also addressed and fixed in the PR above.
I could run the ResNet50 sample on the RISC-V board. The performance was not overwhelming (around 1800ms), but the GPU on that board is on the low end of the spectrum for PowerVR archs. So I hope that you get a bit more performance out of it.
Also the drivers that are available for my board do not seem to be really mature. For example I was not able to get any sane results when using 16-bit FP textures, I had to set the HIGH_PRECISION flag in the CMakeLists.txt to ON to make it work.
Anyway, let me know if this solves your issue.

Sure thing! 🚀 PR is up! Also, regarding gl_PointSize = 1.0, can you pinpoint to the exact files for that tweak? I'll try to make it work for original fyusenet if I can get the Resnet to run.
This is the GLInfo log for the target board:
 GL version: 3.2
GLSL version: 3.20 build 1.18@6276027
GPU vendor: Imagination Technologies
GPU renderer: PowerVR A-Series AXM-8-256
Caps:
  GL_MAX_TEXTURE_SIZE: 16384
  GL_MAX_VERTEX_ATTRIBS: 16
  GL_MAX_VERTEX_UNIFORM_VECTORS: 1024
  GL_MAX_VARYING_VECTORS: 15
  GL_MAX_VERTEX_OUTPUT_COMPONENTS: 64
  GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS: 144
  GL_MAX_VERTEX_TEXTURE_IMAGE_UNITS: 24
  GL_MAX_TEXTURE_IMAGE_UNITS: 24
  GL_MAX_FRAGMENT_UNIFORM_VECTORS: 1024
  GL_MAX_FRAGMENT_INPUT_COMPONENTS: 60
  GL_MAX_COLOR_ATTACHMENTS: 8
  GL_MAX_DRAW_BUFFERS: 8
  GL_MAX_FRAGMENT_UNIFORM_COMPONENTS: 4096
  GL_MAX_VERTEX_UNIFORM_COMPONENTS: 4096
  (I) GL_FRAGMENT_LOW: [15 15]
  (I) GL_FRAGMENT_MEDIUM: [15 15]
  (I) GL_FRAGMENT_HIGH: [31 31]
  (F) GL_FRAGMENT_LOW: [1 1] 8
  (F) GL_FRAGMENT_MEDIUM: [14 14] 10
  (F) GL_FRAGMENT_HIGH: [127 127] 23
  GL_MAX_UNIFORM_BUFFER_BINDINGS: 72
  GL_MAX_VERTEX_UNIFORM_BLOCKS: 12
  GL_MAX_FRAGMENT_UNIFORM_BLOCKS: 12
  GL_MAX_COMBINED_UNIFORM_BLOCKS: 72
  GL_MAX_UNIFORM_BLOCK_SIZE: 134217728
  GL_UNIFORM_BUFFER_OFFSET_ALIGNMENT: 128
  GL_MIN_PROGRAM_TEXEL_OFFSET: -8
  GL_MAX_PROGRAM_TEXEL_OFFSET: 7
  GL_MAX_COMBINED_VERTEX_UNIFORM_COPONENTS: 402657280
  GL_MAX_COMBINED_FRAGMENT_UNIFORM_COMPONENTS: 402657280
  GL_MAX_COMPUTE_IMAGE_UNIFORMS: 24
  GL_MAX_COMPUTE_SHADER_STORAGE_BLOCKS: 35
  GL_MAX_COMPUTE_SHADER_STORAGE_BLOCKS: 24
  GL_MAX_COMPUTE_SHADER_STORAGE_BLOCKS: 0
  GL_MAX_COMPUTE_UNIFORM_BLOCKS: 12
  GL_MAX_COMPUTE_UNIFORM_COMPONENTS: 1024
  GL_MAX_COMPUTE_WORK_GROUP_INVOCATIONS: 1024
  GL_MAX_COMPUTE_WORK_GROUP_SIZE: 1024 1024 1024
  GL_MAX_COMPUTE_WORK_GROUP_COUNT: 65535 65535 65535
GL_MAX_TEXTURE_BUFFER_SIZE: 65536
Extensions:
  - GL_ANDROID_extension_pack_es31a
  - GL_APPLE_texture_format_BGRA8888
  - GL_ARM_shader_framebuffer_fetch_depth_stencil
  - GL_EXT_blend_minmax
  - GL_EXT_buffer_storage
  - GL_EXT_clear_texture
  - GL_EXT_clip_control
  - GL_EXT_color_buffer_float
  - GL_EXT_color_buffer_half_float
  - GL_EXT_compressed_ETC1_RGB8_sub_texture
  - GL_EXT_conservative_depth
  - GL_EXT_copy_image
  - GL_EXT_discard_framebuffer
  - GL_EXT_draw_buffers
  - GL_EXT_draw_buffers_indexed
  - GL_EXT_draw_elements_base_vertex
  - GL_EXT_EGL_image_array
  - GL_EXT_float_blend
  - GL_EXT_geometry_point_size
  - GL_EXT_geometry_shader
  - GL_EXT_gpu_shader5
  - GL_EXT_memory_object
  - GL_EXT_memory_object_fd
  - GL_EXT_multi_draw_arrays
  - GL_EXT_multi_draw_indirect
  - GL_EXT_multisampled_render_to_texture
  - GL_EXT_multisampled_render_to_texture2
  - GL_EXT_occlusion_query_boolean
  - GL_EXT_polygon_offset_clamp
  - GL_EXT_primitive_bounding_box
  - GL_EXT_pvrtc_sRGB
  - GL_EXT_read_format_bgra
  - GL_EXT_robustness
  - GL_EXT_separate_shader_objects
  - GL_EXT_shader_framebuffer_fetch
  - GL_EXT_shader_group_vote
  - GL_EXT_shader_implicit_conversions
  - GL_EXT_shader_io_blocks
  - GL_EXT_shader_non_constant_global_initializers
  - GL_EXT_shader_pixel_local_storage
  - GL_EXT_shader_pixel_local_storage2
  - GL_EXT_shader_texture_lod
  - GL_EXT_shadow_samplers
  - GL_EXT_sparse_texture
  - GL_EXT_sRGB
  - GL_EXT_sRGB_write_control
  - GL_EXT_tessellation_point_size
  - GL_EXT_tessellation_shader
  - GL_EXT_texture_border_clamp
  - GL_EXT_texture_buffer
  - GL_EXT_texture_cube_map_array
  - GL_EXT_texture_filter_anisotropic
  - GL_EXT_texture_format_BGRA8888
  - GL_EXT_texture_format_sRGB_override
  - GL_EXT_texture_norm16
  - GL_EXT_texture_rg
  - GL_EXT_texture_shadow_lod
  - GL_EXT_texture_sRGB_decode
  - GL_EXT_texture_sRGB_R8
  - GL_EXT_texture_sRGB_RG8
  - GL_EXT_texture_type_2_10_10_10_REV
  - GL_EXT_unpack_subimage
  - GL_EXT_YUV_target
  - GL_EXT_texture_storage_compression
  - GL_IMG_framebuffer_downsample
  - GL_IMG_multisampled_render_to_texture
  - GL_IMG_program_binary
  - GL_IMG_read_format
  - GL_IMG_shader_binary
  - GL_IMG_texture_compression_pvrtc
  - GL_IMG_texture_compression_pvrtc2
  - GL_IMG_texture_format_BGRA8888
  - GL_IMG_texture_npot
  - GL_KHR_blend_equation_advanced
  - GL_KHR_blend_equation_advanced_coherent
  - GL_KHR_debug
  - GL_KHR_robustness
  - GL_KHR_texture_compression_astc_ldr
  - GL_OES_compressed_ETC1_RGB8_texture
  - GL_OES_depth24
  - GL_OES_depth_texture
  - GL_OES_depth_texture_cube_map
  - GL_OES_draw_buffers_indexed
  - GL_OES_draw_elements_base_vertex
  - GL_OES_EGL_image
  - GL_OES_EGL_image_external
  - GL_OES_EGL_image_external_essl3
  - GL_OES_EGL_sync
  - GL_OES_element_index_uint
  - GL_OES_fragment_precision_high
  - GL_OES_geometry_point_size
  - GL_OES_geometry_shader
  - GL_OES_get_program_binary
  - GL_OES_gpu_shader5
  - GL_OES_mapbuffer
  - GL_OES_packed_depth_stencil
  - GL_OES_required_internalformat
  - GL_OES_rgb8_rgba8
  - GL_OES_sample_shading
  - GL_OES_sample_variables
  - GL_OES_shader_image_atomic
  - GL_OES_shader_io_blocks
  - GL_OES_shader_multisample_interpolation
  - GL_OES_standard_derivatives
  - GL_OES_surfaceless_context
  - GL_OES_tessellation_point_size
  - GL_OES_tessellation_shader
  - GL_OES_texture_3D
  - GL_OES_texture_border_clamp
  - GL_OES_texture_buffer
  - GL_OES_texture_cube_map_array
  - GL_OES_texture_float
  - GL_OES_texture_half_float
  - GL_OES_texture_npot
  - GL_OES_texture_stencil8
  - GL_OES_texture_storage_multisample_2d_array
  - GL_OES_vertex_array_object
  - GL_OES_vertex_half_float
  - GL_OES_viewport_array
  - GL_OVR_multiview
  - GL_OVR_multiview2

Sorry, I clicked on the closed issue by mistake.

I made the changes in the original fyusenet repo  by adding gl_PointSize = 1.0 for the files: deepconv1x1_tiled.vert, deepdefault.frag and deepdefault.vert. The deepgemmlayer is now giving non-zero outputs, but resnet is giving the same wrong class for all the examples I ran. It is now coming out to be the class with index:936. Any suggestions on the fix?

That fix would be in my fork :-)
I highly recommend to go with my fork, it has a ton of updates compared to the original. The reason why those updates are not merged into the upstream repo are because of an unclear situation regarding licenses for the Llama-type LLMs (which my fork can run). I am quite sure that once those issues have been resolved, it will be merged to upstream. Also, I am the only one developing on FyuseNet, so you won't miss anything from the upstream fork for now :-)

Hello @mtnwrw ,
Thanks for informing! After building on the github branch: Fixes for deep convolutions on weak GPUs and EGL, I got this error when running sample networks:
fyusenet/gl/egl/glcontext_egl.cpp:201 [virtual void fyusion::opengl::GLContext::init()] threw GLException
Detailed error: Cannot initialize EGL extensions

Any workaround/suggestion?

Seems that your board offers none of the extensions to query multiple devices. I just added fallback code for that case which goes with the default display instead.

Yeah, I checked on the symbol 'eglGetProcAddress' in the libEGL.so, and it pointed to IMGeglGetProcAddress in libIMGegl.so and it also had the symbol IMGeglGetProcAddress.
objdump -D /usr/lib/libIMGegl.so | grep IMGeglGetProcAddress
0000000000010c70 <IMGeglGetProcAddress@@Base>:
   10c84:       b4000900        cbz     x0, 10da4 <IMGeglGetProcAddress@@Base+0x134>
   10cac:       b40007b5        cbz     x21, 10da0 <IMGeglGetProcAddress@@Base+0x130>
   10cd0:       14000004        b       10ce0 <IMGeglGetProcAddress@@Base+0x70>
   10d00:       340003a0        cbz     w0, 10d74 <IMGeglGetProcAddress@@Base+0x104>
   10d04:       54fffe81        b.ne    10cd4 <IMGeglGetProcAddress@@Base+0x64>  // b.any
   10d14:       54000820        b.eq    10e18 <IMGeglGetProcAddress@@Base+0x1a8>  // b.none
   10d18:       54000128        b.hi    10d3c <IMGeglGetProcAddress@@Base+0xcc>  // b.pmore
   10d1c:       34000700        cbz     w0, 10dfc <IMGeglGetProcAddress@@Base+0x18c>
   10d40:       54000501        b.ne    10de0 <IMGeglGetProcAddress@@Base+0x170>  // b.any
   10d4c:       34000340        cbz     w0, 10db4 <IMGeglGetProcAddress@@Base+0x144>
   10dc0:       54fffc81        b.ne    10d50 <IMGeglGetProcAddress@@Base+0xe0>  // b.any
   10e04:       34000160        cbz     w0, 10e30 <IMGeglGetProcAddress@@Base+0x1c0>
   10e0c:       b4000121        cbz     x1, 10e30 <IMGeglGetProcAddress@@Base+0x1c0>
   10e14:       17ffffd1        b       10d58 <IMGeglGetProcAddress@@Base+0xe8>
   10e20:       34000120        cbz     w0, 10e44 <IMGeglGetProcAddress@@Base+0x1d4>
   10e2c:       17ffffcb        b       10d58 <IMGeglGetProcAddress@@Base+0xe8>
   10e40:       17ffffda        b       10da8 <IMGeglGetProcAddress@@Base+0x138>
   10e50:       54fffea1        b.ne    10e24 <IMGeglGetProcAddress@@Base+0x1b4>  // b.any
   10e54:       17ffffdc        b       10dc4 <IMGeglGetProcAddress@@Base+0x154>

As long as your board only advertises one EGL device anyway, there will be no difference in the outcome. There are many boards and many desktop devices that offer more than one EGL device, in many cases the "other" devices are software-emulated, for example Mesa EGL. Some laptops with "Optimus" have NVIDIA, Intel and Mesa devices etc.

It worked!! Thanks for adding the patch. Btw, why did you remove the inference timing pipeline?

You mean in the sample apps ? Yeah, I forgot to remove those in the original release. The Engine itself gathers more detailed timings, I merged a PR which makes that interface available as part of the API, you might want to look into that, or just add those timing lines back to the samples, whatever is easier. What kind of timings do you get on your device btw ?
P.S.: I merged your PR but it needs a few amendments, will do those later and add you as reviewer.

For the resnet, it's coming out to ~540 ms. After enabling Engine timing, I'll share that too.

Got the engine timings(but the inference timing got increased a lot to ~1700 ms because of writing the result to file):
Total=84804 microsec
2 3954
3 3450
4 1665
5 427
6 1182
7 1381
8 971
9 847
10 333
11 758
12 692
13 679
14 343
15 664
16 666
17 908
18 1060
19 1092
20 1428
21 750
22 304
23 993
24 1144
25 626
26 277
27 614
28 1080
29 591
30 280
31 624
32 1176
33 688
34 859
35 1399
36 1896
37 1558
38 385
39 1495
40 2072
41 1584
42 297
43 1200
44 1625
45 1558
46 306
47 1090
48 1158
49 1282
50 303
51 1220
52 1371
53 1251
54 283
55 1041
56 1550
57 1612
58 1472
59 1794
60 2472
61 2015
62 386
63 2661
64 2126
65 1658
66 366
67 1389
68 1614
69 1267
70 1351
71 2026
72 2026
73 430

I suspect that the texture up/download might add a bit to the 540ms, which seems a bit slow. I think I might add a sample benchmark that uses asynchronous texture handling to get a bit closer to the actual GPU timings.

@IssacXid If you have some time, please check if that works out for you and add a review:
PR #10
Also, I added a ResNet50 benchmark standalone executable.

PR is merged. Closing ticket.


iptables_load wrapper does not obey iptables-restore exit status

Cookbook version
6.0.1
Chef-client version
14.14.29
Platform Details
Debian Jessie
Scenario
Add a wrong iptables line
Steps to Reproduce
Add a wrong iptables_rule and run Chef and execute /etc/network/if-pre-up.d/iptables_load succeeds even the command would give a syntax error.
Expected Result
iptables_load obeys the iptables-restore exit status
Actual Result
iptables_load always return 0

Hi,
We have since released version 7, I am going to close this but if you still an the issue on version 7 please can you re-open this or open a new issue :+1:
Thanks


Add a wiki: Etrian Odyssey Wiki

An anonymous user has submitted a new wiki via our submission form.
Link to origin wiki: https://etrian.fandom.com/wiki/Etrian_Odyssey_Wiki
Link to destination (independent) wiki: https://etrianodyssey.wiki/wiki/Main_Page

Destination wiki appears to be behind the origin wiki (1,457 vs, 146 content pages) with far fewer recent active edits.


DocumentSnapshot.ConvertTo System.ArgumentException

I get exception System.ArgumentException: "Unable to convert value type IntegerValue to System.Double" when trying to convert DocumentSnapshot to my attributed class.
I think this exception is excess because there is implicit conversion from integer to double in C#

I don't want to start adding every possible implicit conversion in - and I believe it would be dangerous to do so. If you've got a field which is an integer in Firestore, but you try to use it as a double in your attributed class, that means when the value is serialized back to Firestore, it would overwrite the integer value with a floating point value, and anything which was assuming it was an integer would then have a problem.
Basically, I think you need to be consistent in your schema everywhere, otherwise you're creating problems for yourself. Using implicit conversions when deserializing would delay finding those problems, rather than actually solving them.

Yeah but I don't want to have integer in Firestore, it supposed to be real number, but Firestore decises it for me because in some cases numbers are integer and I can't do anything with it

Yeah but I don't want to have integer in Firestore, it supposed to be real number, but Firestore decises it for me because in some cases numbers are integer and I can't do anything with it

What do you mean by "Firestore decides for me"? You should absolutely be able to store the type you want. I believe that storing the value with the type you want should be the focus of your efforts. What stored the integer value in the first place?

Application on Android or iOS creates document in Firestore using value from textbox (stores "cost") parsed to double. Sometimes cost has no decimal point so this value becomes integer in Firestore

Sometimes cost has no decimal point so this value becomes integer in Firestore

I don't believe it should do that if you've actually parsed it to a double. For example, in the Java SDK's UserDataConverter class, a double always ends up converted to DoubleValue
I suggest you check the code you're using on the Android and iOS applications: check the behaviour of each of them separately, and look at the document that gets stored. Fixing the code that's writing the document to do the right thing is going to be more productive than dealing with different types, IMO.

Great - thanks for letting us know.


Runtime: Refactor drivers and connectors

See details in Notion.
Task list:

[ ] Implement the refactors proposed in https://github.com/rilldata/rill/issues/1519
[x] Merge connectors into drivers as described in the proposal

[x] Implement the new interfaces
[x] Implement transporters for the connectors we currently support

[x] Implement Progress support in the connectors where it's possible

Partially closed by https://github.com/rilldata/rill/pull/2763. Will create new issues for other required changes.


We love your old GitBook site!

Hello @astaxie! Addison here from GitBook 😄
It seems like it's been a little bit, but I recently came across your old GitBook project—https://astaxie.gitbooks.io/build-web-application-with-golang/content/en/
I don't know the last time you checked out GitBook, but we've updated the product quite a bit from the version you were last using, and have added a suite of new features to help you customize, organize, and make overall more interactive and engaging sites in GitBook.
I'm reaching out because I'm working on a new project this month, where we'll be launching a new landing page on our homepage that showcases some of the different content that can be made on GitBook, and I think the content you've made would be perfect to feature for the initial launch.
The only thing is, that we're only aiming to include content that's on the latest version of GitBook, so it would be great to work with you on migrating it over.
Luckily, we have a tool in place that will automatically migrate your content if you have it still hosted on GitHub or GitLab (which it seems that you do! 😄)
Would you be interested in exploring this with me at all? Let me know, and I'd be happy to provide more information or even hop on a call to explain a bit more.
You can reach me here, or on email if that's easier—addison @ gitbook.com
Looking forward to hearing back from you!
Addison
Developer Relations @ GitBook

您好，您发的邮件已收到。我会尽快阅读。

这是来自QQ邮箱的自动回复邮件。您的邮件，我已经收到，O(∩_∩)O谢谢


Update gettingstarted.md

Added in this sentence - When you create the partner expense connection it will already be 'Linked' so you won't need to authorise this connection again.
Description
Please include a summary of the new content you're adding or the reason for the change. Please include relevant context.
Type of change
Please delete options that are not relevant.

[ ] New document(s)/updating existing
[ ] Fixes
[ ] Styling
[ ] Bug fix (non-breaking change which fixes an issue)

Sorry.... will sort out

this can be merged @Amy-Roberts


Add the constant pi/4

Resolves #76 .

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines, including the relevant style guides.
[x] Read and understand the Code of Conduct.
[x] Read and understood the licensing terms.
[x] Searched for existing issues and pull requests before submitting this pull request.
[x] Filed an issue (or an issue already existed) prior to submitting this pull request.
[x] Rebased onto latest develop.
[x] Submitted against develop branch.

Description

What is the purpose of this pull request?

This pull request:

is work done on RFC: add constant pi divided by four #76

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #76

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

@stdlib-js/reviewers

@chrisdamba Everything looks good. Would you mind amending the commit message to
Add constant pi divided by four

as we are adding a feature, rather than explicitly making a patch.
You should be able to git commit --amend locally and then force push. I believe this should update the PR. No need to resubmit a new PR.
Once done, should be clear to merge. Thanks!


jsnext:main

Hello there, thanks for this awesome package.
Webpack get's confused when loading react-google-button because it tries to parse the jsnext:main entrypoint but does not know how to handle export default from ..., class variables and JSX.
Shouldn't jsnext:main only contain ES2015 code?

Yes, that is correct. Surprised this wasn't caught until now, but thanks for pointing it out


Teach fsc/fsi how to use responsefile

ref #816
i need feedback on how to test this.
ok in tests\fsharpqa\Source\CompilerOptions\{fsc|fsi}\responseFiles.
But a good test without do output difference?
i think i can try to generate a specific error, and check for that one ( and response file fix the error ), a good candidate?

ok added tests and doc ( in --help, same text of roslyn )
ready to review/merge.

I looked over this briefly and everything looked fin to me.  It would be great to have this in.

ok, used a completely new error number, some cleanups, and added test for error if response file is missing

@otawfik-ms Any idea what the issue is with the timeouts?  Are they real failures or do the tests just take a long time? (We can up the timeouts if necessary)

@mmitche No. 120 minutes are much longer than what is expected. I'll send a PR to run the smoke tests only, until #697 is done, and I can look into existing failures.

@otawfik-ms I can also give you direct access to a machine for debugging if necessary

@otawfik-ms @mmitche can you can add test log to jenkins build artifacts, no ui, but we can download it if needed. Also the current fsharpqa
the net40,portable, etc
CSC : error CS0006: Metadata file 'd:\j\workspace\Microsoft_visualfsharp\net40' could not be found [d:\j\workspace\Microsoft_visualfsharp\net40,portable7,portable47,portable78,portable259,vs_prtest\vsintegration\src\unittests\Resources.MockTypeProviders\DefinitionLocationAttribute\DefinitionLocationAttribute.csproj]

is because of directory name,  d:\j\workspace\Microsoft_visualfsharp\net40,portable7,portable47,portable78,portable259,vs_prtest

@enricosada  you mean this?
http://dotnet-ci.cloudapp.net/job/Microsoft_visualfsharp/job/qa_suite_prtest/6/consoleFull

not the build log, i mean build artifacts. in jenkins you save file in build job, and download it later. for example compiled binaries or test results ( like https://ci.jenkins-ci.org/job/jenkins_2.0/1/ )

I'll add it in my pending changes now.

Changes are in. Both test results and build artifacts should be in Jenkins UI now.
I also restricted it to smoke tests for now. LMK if you see any more timeouts and I'll take a look.

@enricosada I get a weird stack trace if I call:
fsc.exe @""

Unhandled Exception: System.ArgumentException: The path is not of a legal form.
   at System.IO.Path.NormalizePath(String path, Boolean fullCheck, Int32 maxPath
Length, Boolean expandShortPaths)
   at System.IO.Path.GetFullPathInternal(String path)
   at System.IO.Path.GetFullPath(String path)
   at Microsoft.FSharp.Compiler.AbstractIL.Internal.Library.Shim.DefaultFileSyst
em.Microsoft-FSharp-Compiler-AbstractIL-Internal-Library-Shim-IFileSystem-GetFul
lPathShim(String fileName) in C:\dev\src\projects\github.com\visualfsharp\src\ab
sil\illib.fs:line 978
   at Microsoft.FSharp.Compiler.CompileOptions.processArg@255.Invoke(FSharpList`
1 args) in C:\dev\src\projects\github.com\visualfsharp\src\fsharp\CompileOptions
.fs:line 259
   at Microsoft.FSharp.Compiler.CompileOptions.ParseCompilerOptions(FSharpFunc`2
 collectOtherArgument, FSharpList`1 blocks, FSharpList`1 args) in C:\dev\src\pro
jects\github.com\visualfsharp\src\fsharp\CompileOptions.fs:line 370
   at Microsoft.FSharp.Compiler.Driver.ProcessCommandLineFlags(TcConfigBuilder t
cConfigB, FSharpFunc`2 setProcessThreadLocals, FSharpOption`1 lcidFromCodePage,
String[] argv) in C:\dev\src\projects\github.com\visualfsharp\src\fsharp\fsc.fs:
line 261
   at Microsoft.FSharp.Compiler.Driver.GetTcImportsFromCommandLine(String[] argv
, String defaultFSharpBinariesDir, String directoryBuildingFrom, FSharpOption`1
lcidFromCodePage, FSharpFunc`2 setProcessThreadLocals, FSharpFunc`2 displayBanne
rIfNeeded, Boolean optimizeForMemory, Exiter exiter, ErrorLoggerProvider errorLo
ggerProvider, DisposablesTracker disposables) in C:\dev\src\projects\github.com\
visualfsharp\src\fsharp\fsc.fs:line 321
   at Microsoft.FSharp.Core.Operators.Raise[T](Exception exn)
   at Microsoft.FSharp.Compiler.ErrorLogger.ErrorLoggerExtensions.ReraiseIfWatso
nable(Exception exn) in C:\dev\src\projects\github.com\visualfsharp\src\fsharp\E
rrorLogger.fs:line 299
   at Microsoft.FSharp.Compiler.ErrorLogger.ErrorLoggerExtensions.ErrorLogger.Er
rorRecovery(ErrorLogger x, Exception exn, range m) in C:\dev\src\projects\github
.com\visualfsharp\src\fsharp\ErrorLogger.fs:line 322
   at Microsoft.FSharp.Compiler.Driver.GetTcImportsFromCommandLine(String[] argv
, String defaultFSharpBinariesDir, String directoryBuildingFrom, FSharpOption`1
lcidFromCodePage, FSharpFunc`2 setProcessThreadLocals, FSharpFunc`2 displayBanne
rIfNeeded, Boolean optimizeForMemory, Exiter exiter, ErrorLoggerProvider errorLo
ggerProvider, DisposablesTracker disposables) in C:\dev\src\projects\github.com\
visualfsharp\src\fsharp\fsc.fs:line 328
   at Microsoft.FSharp.Compiler.Driver.main0(String[] argv, Boolean bannerAlread
yPrinted, Exiter exiter, ErrorLoggerProvider errorLoggerProvider, DisposablesTra
cker disposables) in C:\dev\src\projects\github.com\visualfsharp\src\fsharp\fsc.
fs:line 1769
   at Microsoft.FSharp.Compiler.Driver.typecheckAndCompile(String[] argv, Boolea
n bannerAlreadyPrinted, Exiter exiter, ErrorLoggerProvider errorLoggerProvider)
in C:\dev\src\projects\github.com\visualfsharp\src\fsharp\fsc.fs:line 1977
   at Microsoft.FSharp.Compiler.Driver.mainCompile(String[] argv, Boolean banner
AlreadyPrinted, Exiter exiter) in C:\dev\src\projects\github.com\visualfsharp\sr
c\fsharp\fsc.fs:line 1989
   at Microsoft.FSharp.Compiler.CommandLineMain.Driver.main(String[] argv) in C:
\dev\src\projects\github.com\visualfsharp\src\fsharp\fscmain.fs:line 270
   at Microsoft.FSharp.Compiler.CommandLineMain.main(String[] argv) in C:\dev\sr
c\projects\github.com\visualfsharp\src\fsharp\fscmain.fs:line 285
   at Microsoft.FSharp.Core.Operators.Raise[T](Exception exn)
   at Microsoft.FSharp.Compiler.ErrorLogger.ErrorLoggerExtensions.ReraiseIfWatso
nable(Exception exn) in C:\dev\src\projects\github.com\visualfsharp\src\fsharp\E
rrorLogger.fs:line 299
   at Microsoft.FSharp.Compiler.ErrorLogger.ErrorLoggerExtensions.ErrorLogger.Er
rorRecovery(ErrorLogger x, Exception exn, range m) in C:\dev\src\projects\github
.com\visualfsharp\src\fsharp\ErrorLogger.fs:line 322
   at Microsoft.FSharp.Compiler.CommandLineMain.main(String[] argv) in C:\dev\sr
c\projects\github.com\visualfsharp\src\fsharp\fscmain.fs:line 287

It seems something special is done when the file doesn't exist (it gives a short informative error message), it would be nice if it was doing the same (maybe with another message).
For example, csc does this:
C:\dev\src\projects\github.com\visualfsharp>csc @""
Microsoft (R) Visual C# Compiler version <IP_ADDRESS>204
Copyright (C) Microsoft Corporation. All rights reserved.

error CS2021: File name '' is empty, contains invalid characters, has a drive sp
ecification without an absolute path, or is too long
warning CS2008: No source files specified.
error CS1562: Outputs without source must have the /out option specified

@smoothdeveloper i check if file exists, but i forgot invalid path like empty. Good catch!! thx
d:\github\fsharp>Release\net40\bin\fsc.exe "@dd"
Microsoft (R) F# Compiler version (private)
Copyright (c) Microsoft Corporation. All Rights Reserved.

error FS3192: Response file '@dd' not found in 'd:\github\fsharp\dd'

ok, added error if empty or invalid response file name ( like @ )

merged into vs2015
Thanks for this contribution.

@KevinRansom  can you merge it in coreclr branch too? it's really useful for dotnet cli
btw why not master? the vs2015 it's the long term branch? it's not a breaking change

@enricosada it was pulled in master too I see.


added copyright

Signed-off-by: Alexandre Eichenberger<EMAIL_ADDRESS>Added copyright to files, as suggested by ONNX Steering Committee. Added all .h/hpp/cpp/md/py and CMakefiles in top dir, src/, include/, docs.
Did not add to files in test and utils (do we own this, I think so). Adding to utils may mess-up the doc-check.

migrated changes to #513


get insecure_ip from request

Thanks for such a useful code，but i want to get insecure_ip from request like：
    let remote_addr = req
        .extensions()
        .get::<axum::extract::ConnectInfo<SocketAddr>>()
        .map(|ci| ci.0);

But I don't know how to code，Looking forward to your answer, thank you

Extractors implement FromRequest


Allow Markdown-Rendering Without Providing an Env-Parameter

This PR fixes #116987
Since the introduction of the new feature for stabilizing images, passing an env-parameter to the markdown-it-parser became mandatory as seen here:
https://github.com/microsoft/vscode/blob/a47f3244b36ae1b792f9c14b2576720ca9d19c52/extensions/markdown-language-features/src/markdownEngine.ts#L161-L168
As my extension uses vscode's markdown-it-parser, having to provide an env-object with pre-defined properties is quite inconvenient. For that reason I created this PR which makes passing an env-object optional as the necessary env.containingImages-property is created on the fly, if necessary, using a new initialize_vscode_image_stabilizer-rule.

In order to create this PR I had to update the @types/markdown-it package.
However, changes to yarn.lock files seem to be blocked.
Is there some workaround for such cases?

Thanks! This will be in the next VS Code insiders build and is scheduled to go out with VS Code 1.54

Awesome thank you very much! ✨


Robot API and Dawn documentation

Replace the placeholder modal opened by the help button in Topbar with opening a new window showing docs for Dawn and the robot API. We should probably ask Runtime for updated Markdown or HTML for the current version of the API because the stuff in old Dawn is heavily outdated. If they’re busy, I think the workshop slides are more recent than the docs on GitHub.

Workshop Slides
Runtime API Description


Add Optional KeyName Param to Items() Function

As mentioned in https://github.com/Azure/bicep/issues/1427, I wish you could pass in a KeyName param to the items() function like items(myObj, "name"). This change would make it easier for users to work with Bicep properties that only accept NameValuePairs and not KeyValuePairs. One such property is SiteConfig/Appsettings and although I can use map((items(appsettings)), arg => {name: arg.key, value: arg.value}) to reconstruct the object list it feels inelegant.
I guess another way of going about this would be making NameValuePair and KeyValuePair interchangeable data types but I imagine those changes would be more work than my proposal.

Also, I realize this is a pipe dream but the NameValuePair stuff makes SiteConfig/Appsettings difficult to work with because you can't do StructuredTransforms on a parameter file when the key names are all the same. Additionally, the items() function only works on the first depth of an object and there is no way to recurse a child object (far as I know) to produce a flattened object. So, a single level appsettings.json could work but users will probably make the mistake of thinking they can nest and we can't validate a JSON schema against it yet.
If we had a flatten() function which could take
{
    "Foo": {
        "Bar1": "Val1",
        "Bar2": "Val2"
    }
}

and convert it to
{
    "Foo--Bar1": "Val1",
    "Foo--Bar2": "Val2"
}

that would allow us to use items() to convert it to NameValuePair and pass it to SiteConfig/Appsettings with minimal (user) effort. I imagine that it would look like this items(flatten(loadJsonContent('appsettings.json'), "--"), "name") in practice. Maybe that's where User Defined Functions will come in down the road? I guess I can't really think of where else a flatten() function would come in handy so... yeah.
Anyway, thanks for listening.

Out of interest, why do you feel like the map approach you suggested is inelegant? That would be my preferred approach to accomplish this scenario:
map((items(appsettings)), arg => {name: arg.key, value: arg.value})

Well, it's not terrible but NVP is a standard datatype so it felt like there ought to be a shorter, normal'ish looking method like passing in a param or casting the KVP to an NVP to get the right value. It took me longer than it should have to realize the only difference between the working and non-working object was just the key name. The appsettings property doesn't complain about being given an invalid object, it just fails.
If the name is seeded from the start the intent is clear that we want the format returned by items() just with a different name. Then again, if map() treats all objects as KVPs (I'm new to this btw) maybe I could have done a map(appsettings), arg => {name: arg.key, value: arg.value}) which would have been a little better but I've reinvented items() or names() rather :)
I get it though, it's nothing egregious.

After reviewing Azure/ResourceModules  using the Microsoft.Web/sites/config resource seems like it solves the issue best since it takes an object. Also has the ability to be placed into a module.
var appSettings = {
  custom_key: 'value'
}
var appInsightsValues = !empty(appInsightId) ? {
  APPINSIGHTS_INSTRUMENTATIONKEY: appInsight.properties.InstrumentationKey
  APPLICATIONINSIGHTS_CONNECTION_STRING: appInsight.properties.ConnectionString
} : {}
var expandedAppSettings = union(appSettingsKeyValuePairs, appInsightsValues)

resource appSettings 'Microsoft.Web/sites/config@2022-03-01' = {
  name: 'appsettings'
  kind: kind
  parent: app
  properties: expandedAppSettings
}

Since the primary use case for this feature request is no longer relevant going to close this out.


[SPARK-47795][DOCS] Supplement the doc of job schedule for K8S

What changes were proposed in this pull request?
This PR propose to supplement the doc of job schedule for K8S.
Why are the changes needed?
Spark document missing the description of job schedule for K8S.
Does this PR introduce any user-facing change?
'No'.
Just update document.
How was this patch tested?
Manual tests.
Was this patch authored or co-authored using generative AI tooling?
'No'.

Is this a required change, @beliefer?

I think we should add these document so as follows the other cluster mangers.

cc @yaooqinn @LuciferYang

I didn't even notice this page or section. The dropdown from the Navi Bar is enough for me.
Do we support scheduling jobs across applications? It's odd to me.

Nit: Use K8s instead of K8S, the former is the official abbreviation

Do we support scheduling jobs across applications? It's odd to me.

This section is about scheduling across applications.
Scheduling Within an Application section is related to jobs.

Let's wait for @yaooqinn 's final sign-off, @beliefer .

Merged to master
Thank you @beliefer @dongjoon-hyun @bjornjorgensen

@yaooqinn @dongjoon-hyun @bjornjorgensen Thank you!


After minimize can't get back to original layout

given this set up

Selecting minimize button gives:

Which isn't what i expected.  Sort of expected both tabs to show up.
Then if I try and maximize the ribbon I get:

@csekme I just made a pull request with the fix for this issue.
Paul--

@csekme I just made a pull request with the fix for this issue.
Paul--


Any interest in integrating an (optional) type-hinting DI callable resolver?

I've got a rough proof of concept that would allow Slim to look up and inject dependencies from any PSR compliant container, even the existing pimple container, via reflection. Adds no extra package dependencies to Slim.

The feature would be optional, as I suspect reflection isn't the fastest thing.
Opt-in, so no BC break.
Introduces two new files, a callable resolver and an invoker strategy. You can use one or the other or both depending if you want constructor injection, method/closure parameter injection, or both.

On one hand, it could be a nice feature to add, on the other it is a handful more files to maintain (Two for implementation and one+ for tests), so I thought I'd put it up for discussion. While this wouldn't trump solutions like php-di, this would allow out-of-the-box type hinted parameter injection with Slim.
Thoughts?

Does this work as a separate add-on project for Slim?

It certainly could. Plus if we went that route, we could probably replace my really simplified reflection with a more powerful  solution like PHP-DI/Invoker, a container agnostic library which provides almost all the functionality, just short of the callable resolver and invoker strategy.

You mean like the PHP-DI Slim Bridge?

Very much, except Slim Bridge is tightly coupled to the PHP-DI container, and what I'm proposing is much of the same functionality, but in a library that doesn't care which container you choose to use. So a bit like a stripped down Slim Bridge, sure.

I don't think we would want this into the core, but you are free to make this a separate add-on project.


Fixed bug with the horusec parser's date parser.

2022-02-02 is 10 characters long, but it only took the first 9.

Hi @bjhijmans please add a unit test that checks for the length of the date

I added a test. This one failed because the date was read as 2022-05-0 which couldn't be parsed, The first test file has a date of 2021-10-19 which was truncated to 2021-10-1 which was interpreted as 2021-10-01 . That 's why it didn't fail.


Resolve android PDF file attach issue over email.

. Resolve android PDF file attach issue over email.
. Added file in mods folder and copying that into node_modules via setup.sh script.

@hemantsain Please also add a fixed version number to be used in package.json


Synchronise SCMP types between python and go

Pr #967 added some new SCMP types to Go, but i forgot to update python. We should do a comparison to fix any differences.

Also: the description of UNREACH_PORT should also contain the possibility that the port is not open (i.e. no application has bound to that port).

Python is no longer supported, so we don't need to implement this any more.


LICENSE: add 3rd party license for Cadence Design Systems

Summary
Add 3rd party licenses for Cadence Design Systems to the LICENSE file.
Add 3rd party licenses for Tensilica Inc to the LICENSE file.
Impact
NONE
Testing
NONE

please ignore the remaining nxstyle errors

@jerpelea there is conflict need to resolve.


Use least_squares solver from ndarray-linalg

Use the least squares solver from ndarray-linalg instead of solving the normal equations explicitly.
Resolves #25

Codecov Report

Merging #125 (9f8b7f4) into master (908efde) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #125      +/-   ##
==========================================
- Coverage   58.32%   58.28%   -0.05%
==========================================
  Files          75       75
  Lines        6695     6693       -2
==========================================
- Hits         3905     3901       -4
- Misses       2790     2792       +2

Impacted Files
Coverage Δ

algorithms/linfa-linear/src/ols.rs
71.69% <100.00%> (-0.27%)
:arrow_down:

...rithms/linfa-trees/src/decision_trees/algorithm.rs
51.05% <0.00%> (-1.41%)
:arrow_down:

algorithms/linfa-kernel/src/lib.rs
70.76% <0.00%> (ø)

algorithms/linfa-pls/src/pls_generic.rs
68.71% <0.00%> (ø)

algorithms/linfa-hierarchical/src/lib.rs
50.00% <0.00%> (+0.74%)
:arrow_up:

algorithms/linfa-linear/src/float.rs
16.66% <0.00%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Make default language server

every time extension pylance ask for make default language server
and I clicked on yes and reload I do it this thing at least ten time
but still same notification pop up it just wasting time nothing more than that
Environment data

Language Server version: 1.1.53
OS and version: Window 10
Python version (& distribution if applicable, e.g. Anaconda): 3.8.1

Expected behaviour
it should start working as I click on yes and reload
Actual behaviour
but it's not working
Logs

XXX

Code Snippet / Additional information

XXX

@OmnipotentCoder, the problem is likely due to the language preference being specified in the user settings and workspace settings. You can verify this by checking the user settings - (Ctrl-Shift P) and selecting "Preferences: Open Settings (JSON))" and the workspace setting found in the settings.json file in the .vscode folder in the project root. Look for the setting "python.languageServer"
Work around:
Delete the "python.languageServer" setting in the settings.json file in the .vscode folder
Hope this works for you, please let us know. Additionally we will add this issue to our backlog. Thanks for the report

Well I also went in settings.json file and change the "python.languageServer" from "Microsoft" to "Pylance" but after that same notification of Pylance still pop up to make default language server.

The check looks into global settings. Do you happen to have multiple settings files or multi-root workspace with settings per workspace?

Nope i am just using python and C language in VS code

Can you post content of your settings.json (and also search your workspace for other possible settings.json in subfolders). Thanks.

when I download Pylance I changed language server from "Microsoft" to "Pylance" but still after that same notification pop up
to make default language server even I restart visual studio code
3-4 times.

When you click 'Yes' does python.languageServer change to Pylance?

That's also the workspace config. What about your user settings JSON? (You can find it by searching for JSON in the command pallete)

Nope when i click on notification to make pylance
default language server it doesn't work after restart it's still ask for to make default language server so then i just go to settings.json and change language server and then still it doesn't work even after restart same notification again pop up to make pylance default language server

If you delete the languageServer option and move it to your user settings, does that fix things? Can you post logs from the Python output window?

If you delete the languageServer option and move it to your user settings, does that fix things? Can you post logs from the Python output window?

Well i don't try to delete the language server option untill now but i will try this thing and i will send screenshot of this here but i can't able to send things right now and but in settings i can language server to none in both workspace and user also but it still doesn't work so now i just delete language server option then i will post the results here

My problem get solved after updating python extension today

thank you for your all support and advise

We should still look into what happens with multiple setting files though.

yes, you are right about this sir but still, thanks for helping me


Add kokoro configuration

Simple presubmit and continuous configurations.

Seems to be wirking - https://source.cloud.google.com/results/invocations/f632c3ea-eb37-4e8b-823c-665166372004/targets/foss-fpga-tools%2Ffpga-tool-perf%2Fpresubmit/log


fix failed tests on Mac:

no -Wlogical-op available on Mac
use pointer (%p)

Hmm, I'd like to keep -Wlogical-op on GCC, as it is an useful warning.
Mac OS X for some reason aliases clang as gcc. The scons CC=clang CXX=clang shown in readme should work, but it might be nice to autodetect that also.
The pointer cast seems reasonable.

Confirmed that "scons CC=clang CXX=clang++" is working under Mac OS X.
It's not needed to merge this into master.


Different color between form-label and form-check-lable in same Form

the form-check-label is darker than form-label
https://github.com/havit/Havit.Blazor/blob/f7a6e108bffd83afc05016854ad9d7ae68cc0498/Havit.Bootstrap/scss/_variables.scss#L28

@crdo I guess this is intentional in our Havit.Blazor theme?

It is, but the checkboxes should have the same label color. So it is a bug.

@crdo it looks like this issue is back on new v4.0

We will do a minor release with some hotfixes soon, I will have a look on this. It might not been properly merged to v4.


Why is there a license?

In general having a license is really important for different things e. g. driving, diving, killing. So congratulations that you have found one. To provide more specific information, it would be useful to know more about your license.

:-D
Actually, no it is not necessary to know more about this specific license, to answer the question WHY it is there in the first place.


Extend e2e test to verify that removal of Shipwright Build works

Follow up of https://github.com/shipwright-io/operator/issues/71
The fix has been made there, we need to extend e2e tests to cover the removal of a ShipwrightBuild instance.

Actual e2e tests, are installing the shipwright operator, then the shipwright component (controller, webhook..)
We also need e2e tests that

create a clusterstrategy
create a build
creates a buildrun
checks that all working well


The bugcheck 0x139 documentation has no description about Parameter 1 of value 30 (0x1e)

[Enter feedback here]
I got bugcheck 0x139 with Parameter 1 of value 30.
I looked up the document for the meaning of Parameter 1 of value 30, but found that this specific value is absent in the list.
What does Parameter 1 of value 30 mean?
If it is for internal use only, is it safe to just ignore it? Or is there anything I should do about it?
Thanks!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f1073c44-369a-9365-a2a7-ea5b064fb1af
Version Independent ID: 28187f46-abb9-c7cb-9b22-ddd4a2ec9d37
Content: Bug Check 0x139 KERNEL_SECURITY_CHECK_FAILURE - Windows drivers
Content Source: windows-driver-docs-pr/debugger/bug-check-0x139--kernel-security-check-failure.md
Product: windows-hardware
Technology: debugger
GitHub Login: @DOMARS
Microsoft Alias: domars

Hi @yujuiwu
Thanks for reaching out to the Windows Driver Developer Documentation team. Unfortunately, we don’t have the resources to offer individual technical advice to the broad array of customers who use the driver documentation.
You may have success posting your question in the following forums, where other driver developers and Microsoft engineers can be found. The community there is quite responsive. Best of luck with this issue.
•	windows-hardware-wdk - https://docs.microsoft.com/en-us/answers/topics/windows-hardware-wdk.html
•	OSR's Debugger forum -  https://community.osr.com/categories/windbg
This thread on the OSR forum describes this bug check.
https://community.osr.com/discussion/292848/kernel-security-check-failure-for-stack-overrun
Transient memory errors can be caused by physical failures and environment factors, and typically require no follow up actions.
If this bug check continues to occur, you could check in the event log and see if there is a specific activity that occurs right before the bug check and look further at the code associated with that event.
As is mentioned in the OSR thread, enabling driver verifier memory pool options, can be a good technique to narrow in on the initial memory write error that leads to other faults and failures downstream.
Thanks for your interest in the Windows bug check docs, and I hope you are able to find an appropriate resolution.


Bioc submission prep -- stage 1

Minor indenting and whitespace nits cleanup in the code to adhere to Google style guide.
Unit tests now follow the recommended strategy in testthat.
DESCRIPTION file updated to use the version number for new submission, and a real person as maintainer.
NAMESPACE file updated to explicitly import all symbols used in code and DESCRIPTION file updated accordingly. This step is optional and can be reverted if looks ugly.
shiny directory and travis file excluded from the R package build.

Stage 2 is to include runnable examples in all exported functions, and to write a simple vignette. These are required by Bioc but need authentication to the API so I am not sure how to proceed.

Hey Sid,
This looks awesome, but please don't remove your name from the Authors.  You did some major work here man.
~p

Ah, I see you updated your email.  Sorry :)

Very nice cleanup - thank you Sid!


Dependencies are misleading

The package.json lists good-listener, select and tiny-emitter as dependencies, meaning they will get installed when requiring clipboard.
But these are misleading. The main file points to the dist folder, where these dependencies are bundled inside the generated file.  I'm not sure if I understand your point exactly.
According to npm docs:

"dependencies": Packages required by your application in production.
"devDependencies": Packages that are only needed for local development and testing.

What would be your suggestion here? Can you send a PR to clarify?

Dependencies are okay, if you want to use src/clipboard.js instead of prebundled file from dist

@zenorocha currently, your entrypoint (dist/clipboard.js does not require good-listener, select and tiny-emitter in production. They are required by your webpack build process generating the dist file.

@stof i make a PR with using src/clipboard.js as entry oint for environments that support ES modules (eg webpack)

if #668 is merged, this issue should indeed be closed if dist/clipboard.js wants to keep bundling dependencies.

This merge will not change anything in dist file bundling

@ogonkov I know. This merge will make it needed to keep the dependencies as is though for modules. but then, there is the question whether the file for main should be updated too, or whether this issue should be closed without changing anything.

Note that for my own projects, #668 will be enough though (I'm using webpack)


Dropdown duplicates if select-element is replaced

After the select element is replaced using the jQuery replaceWith method, calling selectpicker("refresh") causes a duplicate dropdown to show up. It is nested into the markup of the original dropdown button.
See this jsfiddle.
Using the replaceWith method is mandatory for me because I use bootstrap-select in combination with Apache Wicket.

Is there anything I can do to help this getting fixed? I would really appreciate integrating bootstrap-select into our product.

Try calling $(".selectpicker").selectpicker("destroy"); before you use replaceWith.


resolve_async missing from NodeExtension

Describe the Bug
One new bug from graphql/type/definition.py", line 811 as TypeError: Query fields cannot be resolved. Cannot add sync-only extension(s) NodeExtension to the async resolver of Field hardware_details. Please add a resolve_async method to the extension(s).
As we can see at https://github.com/strawberry-graphql/strawberry/blob/main/strawberry/relay/fields.py#L59 there is no reference to resolve_async so we must assume this would never work with the underlying gql library.
The potential quick fix is as follows:
class NodeExtension(FieldExtension):
    def apply(self, field: StrawberryField) -> None:
        assert field.base_resolver is None

        if isinstance(field.type, StrawberryList):
            resolver = self.get_node_list_resolver(field)
        else:
            resolver = self.get_node_resolver(field)

        field.base_resolver = StrawberryResolver(resolver, type_override=field.type)

    async def resolve_async(
        self, next_: AsyncExtensionResolver, source: Any, info: Info, **kwargs: Any
    ) -> Coroutine[Any, Any, Any]:
        return await next_(source, info, **kwargs)

having a query of the following that inherits from relay.Node
@gql.django.type(
    hardware.models.Hardware,
    filters=HardwareFilter,
    order=HardwareOrder,
    pagination=True,
)
class Hardware(gql.relay.Node):
    ...

@strawberry.type
class Query:
    hardware_details: Hardware = gql.django.node(permission_classes=[IsAuthenticated])

Full stack trace:
Traceback (most recent call last):
  File "/Users/dylan/spice/backend/manage.py", line 22, in <module>
    main()
  File "/Users/dylan/spice/backend/manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/django/core/management/__init__.py", line 442, in execute_from_command_line
    utility.execute()
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/django/core/management/__init__.py", line 436, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/django/core/management/__init__.py", line 275, in fetch_command
    klass = load_command_class(app_name, subcommand)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/django/core/management/__init__.py", line 48, in load_command_class
    module = import_module("%s.management.commands.%s" % (app_name, name))
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/dylan/.pyenv/versions/3.11.3/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/Users/dylan/spice/backend/graphql_api/management/commands/generate_schema.py", line 4, in <module>
    from graphql_api.schema import schema
  File "/Users/dylan/spice/backend/graphql_api/schema.py", line 33, in <module>
    schema = Schema(query=Query, mutation=Mutation)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/strawberry/schema/schema.py", line 140, in __init__
    self._schema = GraphQLSchema(
                   ^^^^^^^^^^^^^^
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/graphql/type/schema.py", line 224, in __init__
    collect_referenced_types(query)
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/graphql/type/schema.py", line 432, in collect_referenced_types
    for field in named_type.fields.values():
                 ^^^^^^^^^^^^^^^^^
  File "/Users/dylan/.pyenv/versions/3.11.3/lib/python3.11/functools.py", line 1001, in __get__
    val = self.func(instance)
          ^^^^^^^^^^^^^^^^^^^
  File "/Users/dylan/spice/backend/.venv/lib/python3.11/site-packages/graphql/type/definition.py", line 811, in fields
    raise cls(f"{self.name} fields cannot be resolved. {error}") from error
TypeError: Query fields cannot be resolved. Cannot add sync-only extension(s) NodeExtension to the async resolver of Field hardware_details. Please add a resolve_async method to the extension(s).

System Information

Operating system: macOS Ventura 13.4 (22F66)
Strawberry version:

graphql-core               3.2.3
strawberry-django-plus     3.0.1
strawberry-graphql         0.186.1
strawberry-graphql-django  0.9.4

I determined this was actually due to the permission being added.
# produces error
hardware_details: Hardware = relay.node(permission_classes=[IsAuthenticated])
# works
hardware_details: Hardware = relay.node()

a simple async permission is what causes this path to be exposed.
import typing

from strawberry.permission import BasePermission
from strawberry.types import Info

class IsAuthenticated(BasePermission):
    message = "User is not authenticated"

    async def has_permission(self, source: typing.Any, info: Info, **kwargs) -> bool:
        user = await info.context.request.auser()
        return user.is_authenticated

Hey @djstein ,
Hrm, interesting! I think the new Permissions using Field Extensions will help with this issue. I'll also migrate strawberry-django-plus' permissions code to use it as well.
Either way the extension is indeed missing a resolve_async method, an oversight on my part. I'll do that :)


use json to store settings

This will allow more complex settings information (e.g. lists of things) to be stored in the future.
Note that this change will discard any existing settings (target name, and authentication information), so you may have to login again (or you can manually transfer your settings from ~/.yotta/config to ~/.yotta/config.json if you'd prefer.)
Example of a config.json file:
{
  "build": {
    "target": "x86-linux-native,*"
  },
  "keys": {
    "private": "abcdef",
    "public": "abcdef"
  },
  "github": {
    "authtoken": "abcdef"
  }
}

This pull request also makes yotta link respect a YOTTA_PREFIX environment variable.


Added slew rate feature to sim

Multicopter slew rate was only present in the px4fmu driver, added feature to pwm_out_sim.

Thanks! I do have one comment.

Yes please. Please also check the build errors. Thanks!

@imcnanie can you rebase on master? It also looks like you'll need to fix the code style. Try make format.

@imcnanie can you fix the code style on this?

Looks like this repository was already deleted.