RuntimeError: no suitable implementation for this system

After nameko run helloworld,  appear
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:13> ignored
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:14> ignored
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:16> ignored
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:17> ignored
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:8> ignored
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:9> ignored
starting services: greeting_service
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:7> ignored
Exception AttributeError: "'_SocketDuckForFd' object has no attribute '_closed'" in <bound method _SocketDuckForFd.__del__ of _SocketDuckForFd:10> ignored
Traceback (most recent call last):
  File "/home/XXXXX/anaconda2/bin/nameko", line 11, in <module>
    sys.exit(main())
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/nameko/cli/main.py", line 104, in main
    args.main(args)
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/nameko/cli/commands.py", line 110, in main
    main(args)
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/nameko/cli/run.py", line 184, in main
    run(services, config, backdoor_port=args.backdoor_port)
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/nameko/cli/run.py", line 131, in run
    service_runner.start()
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/nameko/runners.py", line 66, in start
    SpawningProxy(self.containers).start()
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/nameko/utils/__init__.py", line 186, in spawning_method
    return list(pool.imap(call, self._items))
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/greenpool.py", line 175, in imap
    return self.starmap(function, six.moves.zip(*iterables))
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/greenpool.py", line 160, in starmap
    eventlet.spawn_n(self._do_map, function, iterable, gi)
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/greenthread.py", line 66, in spawn_n
    return _spawn_n(0, func, args, kwargs)[1]
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/greenthread.py", line 156, in _spawn_n
    hub = hubs.get_hub()
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/hubs/__init__.py", line 116, in get_hub
    use_hub()
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/hubs/__init__.py", line 70, in use_hub
    mod = get_default_hub()
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/hubs/__init__.py", line 38, in get_default_hub
    import eventlet.hubs.epolls
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/hubs/epolls.py", line 27, in <module>
    from eventlet.hubs.hub import BaseHub
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/hubs/hub.py", line 23, in <module>
    from eventlet.support import greenlets as greenlet, clear_sys_exc_info, monotonic, six
  File "/home/XXXXX/anaconda2/lib/python2.7/site-packages/eventlet/support/monotonic.py", line 167, in <module>
    raise RuntimeError('no suitable implementation for this system')
RuntimeError: no suitable implementation for this system

Please see #533 for an explanation of this.


Algorithm: Minimum Loss

Sorting problem
https://www.hackerrank.com/challenges/minimum-loss/problem

Python Solution
https://github.com/adityabisoi/ds-algo-solutions/pull/361

Can i do it in c++.

I can do it in java!

@yashshah14093 In which language?

@yashshah14093 In which language?

@Nivedita967 Solved using python. Already made a PR.

@Nivedita967 Can u assign me for c++?

@manishaag7 go for it.

I can do it in Java!

I will solve this in C++
please assign this to me.

@shash-2468 Go ahead

Solution's are there in all languages close this issue


Make "Contact Us" link more prominent on the front page.

Currently, it's not easy to find out how to contact the forseti team on the website.
When I go to the front page, nothing immediately sticks out, unless I do a lot of scrolling to the bottom, which seems not a good user experience since I am just guessing/hoping something might be down there.
I would expect to find a Contact Us link at the top menu bar.

I'm sure this will be super helpful to users.

#2887 fixes this.


mkdir: fail if no arguments are provided

https://github.com/uutils/coreutils/pull/6790 can be used instead of this for the no arguments fix, but we still need the fix in this PR for:
❯ /usr/bin/mkdir '' ; printf '%s\n' "${?:?}"
/usr/bin/mkdir: cannot create directory ‘’: No such file or directory
1

❯ coreutils mkdir '' ; printf '%s\n' "${?:?}"
0

I'll update this PR accordingly when #6790 is merged.

I had missed a "." in the test, fixed now

hmm, i am sorry, i took this PR: https://github.com/uutils/coreutils/pull/6790
as it seems to do the same thing but shorter

#6790 can be used instead of this for the no arguments fix, but we still need the fix in this PR for:
❯ /usr/bin/mkdir '' ; printf '%s\n' "${?:?}"
/usr/bin/mkdir: cannot create directory ‘’: No such file or directory
1

❯ coreutils mkdir '' ; printf '%s\n' "${?:?}"
0

I'll update this PR accordingly when #6790 is merged.

Going to just make a new PR instead


[WIP] Add OpenTracing configurability

This exposes OpenTracing configurability provided by APIcast into the APIcast operator.
Design:
A new spec.opentracing section has been added in the APIcast CR.
An example:
spec:
  opentracing:
    enabled: true
    tracingLibrary: jaeger
    tracingConfigRef: mysecretname

All attributes in the opentracing section are optional
If the enabled flag is not specified, it defaults to false
If the tracingLibrary flag is not specified, it defaults to jaeger
If the tracingConfigRef flag is not specified, it defaults to a default tracing configuration specific to the set tracingLirary. This default tracing configuration is provided in the APIcast image itself

The configuration is mounted into the APIcast pod by looking at a config key in the provided secret name and mounted in /opt/app-root/src/tracing-configs/tracing-config-<tracing_library_name>-<secret_name>, for example in opt/app-root/src/tracing-configs/tracing-config-jaeger-mysecretname
Pending:

[ ] Verification

With the current implementation the OpenTracing configuration is being provided with a Secret but in the official APIcast product documentation I see a ConfigMap is used. Maybe in this case a Secret is not needed and we should use a ConfigMap instead?

Ready for a first review @eguzki before I proceed to do some verifications

@eguzki I think this should be ready


Why does 'timedOutPerformingAction' get called ?

Scenario: "Alice calls Bob, Alice and Bob are connected to the room and can see each others video".
Issue: Cannot hear each others Audio and timedOutPerformingAction delegate method gets called and call ends.
For Info:
a. Local audio track is created for Bob and Alice.
b.TVIParticipant addedAudioTrack delegate method gets called for Bob and Alice.
Looked up apple docs on CallKit for timedOutPerformingAction which says:

Called when the provider performs the specified action times out.

Not sure what to make of it.

@ptankTwilio, I would really appreciate if you can advice on this issue which completely blocks us from using Twilio video call service.
look forward to hear from you.

@pravishanth Can you provide more details which action is getting timed out by inspecting the CXAction returned in timedOutPerformaingAction?
Regarding the no audio: since CallKit activates the AVAudioSession, it's important to use TVIAudioController's  CallKit APIs appropriately in the app. I would recommend comparing your application code with the VideoCallKitQuickStart example to debug the issue in your application.

@ptankTwilio,
Apologies for the delay in reporting back,
I tried to print the CXAction returned in
 func provider(_ provider: CXProvider, timedOutPerforming action: CXAction) { logMessage("Action in timedOutPerformingAction: \(action)") }
This is what I get, could not understand what failureReason=2 means,
<CXAnswerCallAction 0x14f3a730 UUID=DA5B2D26-CEDD-49A6-A2D3-7E29953659AF state=2 commitDate=2017-11-13 05:45:25 +0000 failureReason=2 callUUID=7DC79500-A97E-47B8-BE5B-1C98403FACA9 localLandscapeAspectRatio={0, 0} localPortraitAspectRatio={0, 0} dateConnected=(null)>
I also compared my code against  VideoCallKitQuickStart, and I can confirm that there are no changes in the way TVIAudioController is configured.

@pravishanth I am suspecting the AnswerCallAction is not being fulfilled in your app and that's why it is timing out. So, can you make sure your app is calling fulfill on CXAnswerCallAction? Our quickstart example executes the [CXAnswerCallAction fullfill] upon receiving the TVIRoomDelegate 's didConnectToRoom callback.
see following for reference :
https://github.com/twilio/video-quickstart-swift/blob/master/VideoCallKitQuickStart/ViewController%2BCallKit.swift#L73
https://github.com/twilio/video-quickstart-swift/blob/master/VideoCallKitQuickStart/ViewController.swift#L285

@ptankTwilio,
You were bang on, thanks so much for the advice.
The AnswerCallAction wasn't being fulfilled, since my performRoomConnect function wasn't passing a value to the callback.

thanks, i know problem, i must call reportOutgoingCallWithUUID


Black Screen

I followed your example in my own application step by step but I got black screen in camera. The onFrame method is called and the net.forward is called too but the rest does not execute. Then I cloned your repo and installed it but it is still the same.
Any ideas or clues to how to proceed?

Add
cameraBridgeViewBase.setCameraPermissionGranted();
to OnCreate() and test it!


Please invite me to the Recode-Hive GitHub Community Organization

Name
Tejashri Taral
Discord Username (if applicable)
Tejashri-Taral
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people


Allow clickthrough if swipe hasn't started yet

Is your feature request related to a problem? Please describe.
For my use case I need a swipeable list that is also clickable, which the library doesn't support currently.
I believe this could be a nice addition.
Describe the solution you'd like
Allow clickthrough when the swipe threshold hasn't been hit.
I would like into this myself but don't really have the time currently.
Describe alternatives you've considered
None

I don't follow :) Could you please describe in more details?
How somebody could click while swiping item?

I've got the same issue with this SwipeableList !
Sometimes I need to click multiple times on my switch to be able to toggle it. If I don't use the SwipeableList my Switch is working fine, it toggle instantly when I click on it.
Here is a video of what's going on :
https://user-images.githubusercontent.com/41306995/147000626-6c35ba4c-f598-43a0-8a08-d6cdcd405d9e.mp4

@bastiengrignon Could you tell me what kind of component/library are you using for this switcher? Or even better provide some code so I can see how it is built?

I'm using Chakra UI for the Switch component
Here is a part of my code of list of alarms :
<SwipeableList fullSwipe={ true } style={ { backgroundColor: 'transparent' } } type={ ListType.IOS }>
  {
    alarms.map((alarm) => (
        <SwipeableListItem fullSwipe={ true } trailingActions={ trailingActions(alarm) } key={ uuidv4() }>

                        <div className={ `text-4xl ${ textColor } space-x-1` }>
                            { alarm.hour }:{ alarm.minute }

                            <SunIcon color={ alarm.lights ? 'yellow.500' : 'gray.500' }/>

                        <Weekdays weekdays={ alarm.weekdays }/>

                <Switch size="lg" defaultChecked={ alarm.activation }
                        onChange={ (event) => updateAlarmState(alarm, event) }/>

        </SwipeableListItem>
    ))
  }
</SwipeableList>

If I comment the <SwipeableListItem/>, I can toggle my switch by pressing it only once as it should 😉

@bastiengrignon I used your code, build an app with create-react-app, added chakra and tailwind and in my case everything is working OK:

There must be some other implementation issue that I cannot reproduce from the part of your code.

@marekrozmus Ok this is weird, I'll try to delete this component and try again from scratch to see if I did something wrong 🤔
Thanks for your quick response !

This one is closed by Release v1.5.0

@all-contributors please add @segersniels for ideas


"Beautify" UpdateLeaderBoard code, Fix "board" command

Edits:

Made UpdateLeaderboard.js look better
Fix "board" command

Board command fix credits to @proxiemind

Awesome 🎉


Line highlighting while stepping

Highlight Code lines (micro when micro step, macro when macro step)

Controller (ng service)
Trigger on Button Click/Error/Breakpoint etc
Priority highlighting: Error > Breakpoint > Current Step Line (Only Step Mode)

Lost for Macro steps after merging pull request #201


INT-20 Cancellation token

The idea behind this code is to have the possibility to pass cancellationToken from UiPath custom activities to the client.
If a token is canceled in the UiPath project somewhere, all async methods within the client will be canceled as well.
This will require a change in the UiPath custom activities project according to https://stackoverflow.com/a/11191070 <- Addition section of the answer.

The idea behind this code is to have the possibility to pass cancellationToken from UiPath custom activities to the client. If a token is canceled in the UiPath project somewhere, all async methods within the client will be canceled as well.
This will require a change in the UiPath custom activities project according to https://stackoverflow.com/a/11191070 <- Addition section of the answer.

The idea behind this code is to have the possibility to pass cancellationToken from UiPath custom activities to the client. If a token is canceled in the UiPath project somewhere, all async methods within the client will be canceled as well.
This will require a change in the UiPath custom activities project according to https://stackoverflow.com/a/11191070 <- Addition section of the answer.

Resolved conflicts caused by editor config PR merge.

Resolved conflicts caused by editor config PR merge.


WIP: Fix/bch tx build

Confirmed working!
Note. This PR works only if I also modify line recovery.service.ts:83
From: addressType: payload.addressType || "P2SH",
To: "P2SH",


replace mdBook with Hugo for site generation

An attempt to get multi-language support with a simple site generator: https://gohugo.io/
Research issue: https://github.com/vacp2p/docs.dappconnect.dev/issues/7
Theme: https://themes.gohugo.io/themes/hugo-book/
I've dropped the Waku specs part for simplicity of the PR, you can re-add it later yourself @D4nte .

With some manual adjustment to the CI Job I deployed this version to https://dev-docs.dappconnect.dev/.

Fixed links and added instructions to run git submodule update --init.

Merged into develop and master rebased on develop. Both branches deployed.


npm not being installed

Looks like version 6 of node doesn't install NPM. This means later build steps fail when building the docker image

Does this issue still need to be open?


Tensorflow implementation

Hi,
Are the authors aware of any tensorflow implementation of GLNet? If so, could you please point to it.
Thanks!

Sorry, we currently don’t have a plan for implementing a tensorflow version.


X and Y properties are not updating properly.

I have 2 lists of Vue draggables which are placed in 2 different components.
They are equal ( one is a map editor for tables in a restaurant and the other one is the actual tables, so they are equal )
Whenever I change the position in the map editor, the actual tables are not updating properly even though the props that hold the position is passed properly to the other component, but apparently X and Y properties are not updating properly, it only works if I re-render the tables by destroying the component.
Any ideas? :D

This is a duplicate of #14.


Redesign Modin to align with DataFrame Algebra

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. What kind of performance improvements would you like to see with this new API?
Redesign Modin's architecture to implement the DataFrame Algebra proposed in the vision paper found here. Predecessor to #2204

@devin-petersohn

@devin-petersohn

@RehanSD, please update the link to a newer paper in the issue description. I suppose this one should be https://people.eecs.berkeley.edu/~totemtang/paper/Modin.pdf.

We have resolved the code portion of this refactor - we also need to update the documentation to reflect the new language.


Ordon quality of life

Skip first round of sumo

From my testing it didn't break anything , the flag don't open bo door so it's fine


Can't create automations with hassio 0.80.0b1

Home Assistant release with the issue:
0.80.0b1
Last working Home Assistant release (if known):
0.79.3
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io installed on a RPi3
Component/platform:
Automation Editor
Description of problem:
Since updating to 0.80.0b1 I am unable to create new automations via the editor. After clicking on the plus icon, all I get is a blank screen with the title 'New Automation'.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
automation: !include automations.yaml

Additional information:
In addition to not being able to create a new automations, a couple of existing ones are not working. In the editor, when clicking on them, I'm greeted with a blank page (just like when trying to create a new automation).

I have the same problem on 0.80.0b3.
Do you mean you brought back automations.yaml to configuration.yaml, or that you started from scratch?

Same problem with 0.80.0b5, the editor works with about 50% of the of the old automations, the rest displays a blank screen and I'm not able to create new automations since it only shows a blank screen.
When I use the editor for my scripts it works fine.
No error messages in the log and the url that shows in the top of the editor display contains the right automation id:

I have the same problem on 0.80.0b3.
Do you mean you brought back automations.yaml to configuration.yaml, or that you started from scratch?
It has to be noted that the automations themselves work fine. Just the web editor is unavailable.

I started from scratch and that seems to have solved the problem, but only temporarily. Issue reoccured in 0.80.0b2/3/4/5. Issue still there today after updating to 0.80.0

So, lets understand this.  Broken in 0.80.0b(x) code and carried over to the production release?

So, lets understand this. Broken in 0.80.0b(x) code and carried over to the production release?

That does seem to be a troubling possibility.
Does this problem not occur with the Lovelace interface?
What cool features are developers using that would allow this bug to be missed in release testing?  I want to be using those features :)

I just updated to 0.80.0 and have the same issue. New automation page is blank and nothing happens when I click add trigger, condition, or action on existing automations.

On 0.79.0, none of my existing automations can be edited. Chrome console provides 404 error for all. I can create a new automation via the frontend though.

Is there any workaround for this or is the only way to restore the functionality to go back to the previous release?

This is fixed in 0.80.1 released an hour ago. https://github.com/home-assistant/home-assistant/releases/tag/0.80.1

I still cannot edit old automations though - they give me a 404 error in the Google Chrome console: GET https://[myHAip:port]/api/config/automation/config/aquarium_turn_on_time 404 File not Found. The last part is the unique ID of the automation - all of my automations have an unique ID.

I've updated to 0.80.1 and the automation screen is still blank when after clicking the + button.  For what it's worth, my automations file is empty as I'm doing a rebuild.

Same here - no change with 0.80.1 - blank page for new automations.  I only have two automations, one of them can be edited, the other cannot.

Tagging @balloob to escalate

I still cannot edit old automations though - they give me a 404 error in the Google Chrome console: GET https://[myHAip:port]/api/config/automation/config/aquarium_turn_on_time 404 File not Found. The last part is the unique ID of the automation - all of my automations have an unique ID.

Also 0.80.3 did not fix my automation editing, still seeing GET 404 errors in the console while trying to open existing automations.
I am using lovelace, HA 0.80.3, having all automations in several files in the automation directory, linked from configurationyaml by a automation: !include_dir_merge_list automation/

This issue has been resolved. Please close all tabs that have HA open to help the cache get cleared.


Feature request - env options

We set some env options dynamically on our builds due to github not preserving them between steps, and in our case they're computed. It would be great if there was an option like run_init that accepted something arbitrary to run before the gradle command

Hi @ZacSweers,
Is that a common practice? Could you please share some pointers showing how this would be done from a workflow point of view?

I don't have a public example but there re some cases where we run commands with local env vars
FOO=bar ./gradlew build

A better example would be env vars that gradle or other plugins (like the android plugin seek)
https://docs.gradle.org/current/userguide/build_environment.html#sec:gradle_environment_variables
We may dynamically set the JDK in a conditional way that can't be statically defined in github actions' env argument alone, hence wanting a script to manage it.

Hi Zac,
I can see how this could be useful. Thanks for clarifying.
I don't have that immediate need so it may take a while before I implement that myself.
Happy to review a pull request though ;)
It is somewhat related to #18 and should be made consistent form the YAML point of view.

With<EMAIL_ADDRESS>it is now possible to use the action to configure Gradle and use a standard run action for the actual Gradle invocation. This should allow you to benefit from the caching this action provides, but retain your existing step that invokes Gradle, including any scripted setup of the environment.


Clarified OAuth2 documentation

Fixed misspelling and grammar in OAuth2 documentation

Thanks for the PR @galapagosfinch. The changes should have been applied in the gh-pages branch, specifically this file. If you're interested in applying these updates, can you please resubmit a new PR against the gh-pages branch. Thanks!


ADU Sku and Locations property at account level

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@visingl , this appears to be a new api version. Can you go thru the first comment of this PR and check on appropriate items?  Did you add this version to rpsaasmaster branch  before? It's supposed to add this version to private repo if it's private preview.

@raych1 As suggested updated first comment

/azp run

@raych1 As suggested updated first comment

@visingl , can you read through the first comment of this PR?
ARM API Review Checklist need your selection to trigger ARM further review. In addition, you need to have first commit as the copy of preview version then have followed commit for the update. So that, the differences is easy to be reviewed.


Binary exits silently in Docker container when connecting to database

So I am trying to deploy my app inside a Docker container and have come across a strange issue. The app exits silently (without any panics or errors) when connecting to the database. My connection string is correct since I can run the app locally with it.
I tried building in release mode locally and checking with that and it still worked fine.
I enabled trace logs and got the following:
api_1             | 2022-11-04T14:36:55.157393Z TRACE query_core::executor::loader: Loading Postgres query connector...
api_1             | 2022-11-04T14:36:55.157654Z  INFO quaint::pooled: Starting a postgresql pool with 9 connections.
api_1             | 2022-11-04T14:36:55.157764Z TRACE query_core::executor::loader: Loaded Postgres query connector.
api_1             | 2022-11-04T14:36:55.170918Z DEBUG mobc: creating new connection from manager

After that it just exits. I am not sure what the issue is and whether its just a container thing.
My Dockerfile's build section in case it sheds some light:
FROM rust:1.64-alpine as builder
RUN apk upgrade --update-cache --available
RUN apk add --no-cache musl-dev
RUN apk add pkgconfig
RUN apk add libressl-dev
RUN rm -rf /var/cache/apk/*
RUN rustup target add x86_64-unknown-linux-musl
WORKDIR /usr/src
RUN USER=root cargo new app
WORKDIR /usr/src/app
COPY Cargo.toml Cargo.lock ./
RUN cargo build --target x86_64-unknown-linux-musl --release
COPY src ./src
COPY prisma ./prisma
RUN touch ./src/main.rs
RUN cargo build --target x86_64-unknown-linux-musl --release --locked

This is quite bizzare, can you make a reproduction repo that triggers this? Don't really want to scaffold my own example around a single dockerfile.

Here is a small reproduction repo: https://github.com/affanshahid/prisma-rust-silent-exit-reproduction. I checked the error code after the exit and it seems it is a SEGFAULT (139).

Not sure why but I can't for the life of me get that example running. Have had to change x86_64 to aarch64 since I'm on M1 but I'm getting the following:
app_1  | exec /bin/prisma-rust-silent-exit-reproduction: no such file or directory
prisma-rust-silent-exit-reproduction_app_1 exited with code 1

I think the COPY is working but I'm not sure why it's failing to locate the built binary.

I googled around a bit and found out others have reported similar issues with alpine + rust, it might have to do with the MUSL libc, there are apparently ways to fix this but I just ended up changing my builder image to rust:1.64 and my final image to gcr.io/distroless/cc. Its a bit of an increase in size but I can live with it.
If you are still interested, you could probably just do cargo run ... somewhere in the builder image and you'd get the segfault.
The issue can be closed I think since it seems it doesn't have anything to do with the lib.

Hmm ok. Alpine has always been an awkward situation with Prisma, and with PCR static linking instead of dynamically like Prisma JS I'm not sure what should be done.


mysql structure

Hi,
i didn't find how to create mysql database.
There is no documentation or i can't find it .
Can you help me please ?
Thanks,

Apologies, I thought that had been uploaded.
The main dev on this is away at the moment, we’ll get the .sql file with the CREATE statements in uploaded early next week.
Oliver.

On 15 Apr 2016, at 14:39, Guillaume Roche<EMAIL_ADDRESS>wrote:
Hi,
i didn't find how to create mysql database.
There is no documentation or i can't find it .
Can you help me please ?

Hi @guigeek208
There should be a new file added towards the end of the day.
Thought i had uploaded it previously.
Best

This one worked for me:
CREATE TABLE data (
timestamp timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP,
locationID tinyint(4) NOT NULL,
event tinyint(4) NOT NULL
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

Hi @guigeek208
Check the new documentation
or
@igordutra 's statement works aswell
Best


Silk.NET.Direct3D11.ID3D10Blob Missing Methods

Hi,
The Silk.NET.Direct3D11.ID3D10Blob structure appears to be missing these two methods:

ID3D10Blob::GetBufferPointer
ID3D10Blob::GetBufferSize

The struct itself contains nothing:

EDIT: The workaround for this seems to be to use IDxcBlob for now, as it is an alias of ID3D10Blob. However, I don't think this is a safe workaround given that Microsoft may change IDxcBlob structure/functionality later down the line and diverge from ID3D10Blob.

Hi there. ID3D10Blob is in Silk.NET.Core.Native - unfortunate that it seems to have lit up in D3D11 as well. Will look at removing that as that's a bit of a binding bug.
https://github.com/dotnet/Silk.NET/blob/main/src/Core/Silk.NET.Core/Native/Structs/ID3D10Blob.gen.cs

Hi Perksey, That's brilliant. Thanks for the really quick response and solution.


Kaku stuck on "Loading"

Bug
Steps to reproduce the bug
I think that it's because I "killed" all the kaku processes (didn't send the SIGNINT signal?). So maybe there is something that should be closed on exit that didn't get close.

Ubuntu Mate 16.04
Linux 4.13.0-36-generic #40~16.04.1-Ubuntu SMP Fri Feb 16 23:25:58 UTC 2018 x86_64
Which platform are you using to listen to music?
YouTube
Which track format are you using?
best video (or best audio)
Is there any error coming from Inspector?
I can't open it.
Others
The loader icon even does not move. It starts moving at the beginning but then it just stops, I can't even click on the Minimize/Maximize/Close buttons.

Sorry, yes :smile: It's 1.9.0, I have it on my package manager as 1.9.0-1, and I've got confused

Do you have a link/description/screenshot to the package manager?
I do not remember publishing this to the package manager :thinking:

close old issue and will open a new one if needed.


Snapshot requires refactored Makefile

Currently the Makefile in /snapshot is only suitable for development on Linux machines. Running make all on OS X builds a container that will not run.

/assign @ianchakeres


Support for laravel 6.x ?

Hi,
Could you please add support for laravel 6.x?
Regards, Matthijs

Hi!
I haven't plans for adding support of laravel 6, because this package actuality is lost - he isnt needed anymore in most cases.
Could you tell, whats your problem? Maybe I can help you in any way?


Discoverability on the BOM Server

Creating this based on Discussion on Slack:

Quick question on SBOM retrieval via API. Is the assumption that a requestor already knows the SBOM URN? Are there plans to support URN identification by unique identifiers like PURL, CPE or SWID? Or is this left to implementer to determine? Completely understand there may be multiple SBOM for a given identifier.

We've discussed discoverability. Currently not specified. I'd recommend creating a ticket for the spec to support it.

Discoverability based on PURL/CPE will be very helpful for projects. It would also enable the creation of "public BOM servers", where one could host (say) SBOMs for every package on a specific distro or package manager.
Slack Ref: https://cyclonedx.slack.com/archives/C024GCLFJDR/p1652538141714249

is this request in the correct repo?
should it move to https://github.com/CycloneDX/cyclonedx-bom-repo-server ?

There's two standardization questions here:

URN identification via unique identifiers like PURL/CPE/SWID for a given SBOM.
Serving SBOMs via such identifiers on a server, one of which might be the bom-repo-server.

Till (1) is settled (ideally in the specification), I'm not sure if it can even be taken up by any SBOM server. As it stands, the SBOM metadata does not include a stable identifier for itself.
As a sample, this is what alpine:latest scanned via Syft shows up as:
{
  "$schema": "http://cyclonedx.org/schema/bom-1.4.schema.json",
  "bomFormat": "CycloneDX",
  "specVersion": "1.4",
  "serialNumber": "urn:uuid:16e4a07b-1174-4c6d-8783-f9d15c546924",
  "version": 1,
  "metadata": {
    "timestamp": "2023-07-27T10:12:21Z",
    "tools": [
      {
        "vendor": "anchore",
        "name": "syft",
        "version": "0.85.0"
      }
    ],
    "component": {
      "bom-ref": "5f08cdaf1f225f20",
      "type": "container",
      "name": "alpine:edge",
      "version": "sha256:12b034ab685c2031f14d2d99dff9db7049e05051c6dbce8905781d23c286b868"
    }
  },

Since this is a container image, which have unique PURLs, a discoverability mechanism would mean adding an identifier like:
pkg:docker/library/alpine@latest or pkg:docker/library/alpine@12b034ab685c2031f14d2d99dff9db7049e05051c6dbce8905781d23c286b868 inside the SBOM itself. Syft additionally adds a "operating-system" component at the end:
{
      "type": "operating-system",
      "name": "alpine",
      "version": "3.18_alpha20230329",
      "description": "Alpine Linux edge",
      "swid": {
        "tagId": "alpine",
        "name": "alpine",
        "version": "3.18_alpha20230329"
      },
      "externalReferences": [
        {
          "url": "https://gitlab.alpinelinux.org/alpine/aports/-/issues",
          "type": "issue-tracker"
        },
        {
          "url": "https://alpinelinux.org/",
          "type": "website"
        }
      ],
      "properties": [
        {
          "name": "syft:distro:id",
          "value": "alpine"
        },
        {
          "name": "syft:distro:prettyName",
          "value": "Alpine Linux edge"
        },
        {
          "name": "syft:distro:versionID",
          "value": "3.18_alpha20230329"
        }
      ]
    }

But this does not include a PURL (there is no PURL type for operating-systems), and is a description of the operating system inside the container image, its not a top-level identifier for what was scanned.

As it stands, the SBOM metadata does not include a stable identifier for itself.

The combination of serialNumber and version form the identity of the BOM. This identity can be expressed via BOM-Link. The BOM Exchange API currently supports this.  There are plans to support discoverability based on the metadata>component via its purl, cpe, or swid.

Does Syft add:
"swid": {
  "tagId": "alpine",
  "name": "alpine",
  "version": "3.18_alpha20230329"
}

If so, that's so completely wrong. Nothing about that markup is correct.

this topic might be related/referring to https://github.com/CycloneDX/transparency-exchange-api


php mail, avoid form refresh page

I have a form, my plugin read the $_POST var and give back the result to $twigVariables. All seem to work, but I want to avoid the page reload.
index.twig
 <form role="form" id="contact-form" method="post" data-toggle="validator">
...
...
</form>
{% if msg %}
<div class="alert alert-{{ error == false ? "success" : "danger" }} > {{ msg }} </div>
{% endif %}

plugin.php (onPageRendering)
if($_SERVER["REQUEST_METHOD"] == "POST"){
...
...
$twigVariables['error'] = $error; // false or true
$twigVariables['msg'] = $msg; // success or fail message
}

Any hint to solve the issue?

To my knowledge, reloading the page is just how POST works.
The only way around this is to use Javascript to send the POST data instead.
However, if you do this, you won't get that data back into Pico or Twig, since all the execution is done server-side before the page is rendered.  You'd have to also use Javascript to handle your success/error message as well.
Assuming you're a bit more technically inclined, check out what I did for my Freelancer port.
The original Start Bootstrap contact form used a proprietary server-side script for validation (to upsell their premium service), so I had to replace it if I wanted validation to work in my version.
I wrote this Javascript file to provide very basic browser-based form validation (something built-in to the browser, but requiring a script to utilize), and to send the POST data via an Ajax request without reloading the page. 😁
I have to warn you though that I am NOT a Javascript developer, and there might be some problems with my implementation.  I cobbled it together from a few different tutorials, using the pieces I specifically needed.
This contact form also activates several styles provided by the original theme and its underlying Bootstrap base to highlight any errors the browser-based validation detects when you try to send the form.
I can't say it wouldn't be a bit of a research project, but using Javascript and a client-side script is as far as I know the ONLY way to have a form that sends without reloading.  Trust me, I looked for any excuse I could find to not have to dive into JS for it. 😂
So, hopefully this will push you in the right direction.  Wish I could be more help on this, but I barely managed it myself when I did.  I even have a big disclaimer in the readme that my Javascript could be a complete mess and probably shouldn't be trusted! 😅
Let me know if you have any other questions though and I'll be happy to help where I can. 😉

Ok, I have already a similar JS to make the magic happen (it works with a separate mail.php). But I want to make a plugin, so I build a fake mail.php and read its output with the js:
class PHPmail extends AbstractPicoPlugin
{
    public function onRequestUrl(&$url)
    {
        // If url/mail.php, then true
        if ($url == 'mail.php') {
            $this->phpMail = true;
        }
    }

    public function onPagesLoaded()
    if ($this->phpMail  and  _POST){
            ...
            $output = 'success';
            die ($output);
    }
}

If I point to localhost/mail.php this give me the same output as if there was the php file on the server, so it seems to work.
But ajax fails
    $('#contact-form').on('submit',function(){

    // Add text right after clicking
    $('.alert').text('Loading...');

    var form = $(this);
    $.ajax({
        method: "POST",
        url: "mail.php",
    ...

There is definitely something I do not understand ... The js does not give me positive or negative results, but it remains stuck on Loading..., I probably cannot request the fake mail.php from the js or the onPagesLoaded is wrong?

Self reply. Just add a header($_SERVER['SERVER_PROTOCOL'] . ' 200 OK'); and JS works.

Sorry, been away for a few days and I need to catch up on things. 😅
Glad you got it worked out.
As I said before, I'm WAY out of my league when dealing with Javascript, PHP, or POST requests, so I'm not sure I could have helped much anyway. 😅


Support for custom retrievers: common base type for EvaluateRetrieval's retriever param

The current design of EvaluateRetrieval is already supporting multiple different retrievers. See https://github.com/beir-cellar/beir/blob/bb8a0582d8a82d6f4fddee8b668b1ebaa42985ef/beir/retrieval/evaluation.py#L14
However according to typings the supplied retrievers are limited to those implemented in BEIR. If we want to integrate BEIR in other tools (like e.g. haystack) a custom retriever is basically easy to implement (see https://github.com/deepset-ai/haystack/pull/2333). This is due to the fact that all retrievers share a common interface of course. However the typings on EvaluateRetrieval do not account for this explicitly. A common abstract base class instead of a Tuple of all implemented retrievers would make this extensibility more comprehensive.

Hi @tstadel,
Thank you for bringing up this issue. I agree with your point. Could you provide an implementation that you have in mind for a common abstract base class and provide a pull request for that? I can check and merge it in the future.
Also, happy to see BEIR integrated within haystack.
Kind Regards,
Nandan Thakur

Hey @NThakur20,
sure. I'll submit a PR in the next days.


Bugfix/6.4/tdi 38395

What is the current behavior? (You can also link to an open issue here)
Wrong behavior in "Guess Query" feature when there are no columns in the schema.
What is the new behavior?
Error message is shown
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[x] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
https://jira.talendforge.org/browse/TDI-38395

Pull request test result: failure :x:


update for VS 16.10.0

VS 16.10.0 includes support for P0645R10  Text Formatting but it isn't mentioned on this page. I'm interested to know its status. Is it fully implemented? Are there pieces missing?

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

ID: d606e0aa-36fd-97f2-6e65-af0f9b05b24c
Version Independent ID: 207d361a-397e-d687-8828-a152ebc4e572
Content: Microsoft C++ language and standard library conformance table
Content Source: docs/overview/visual-cpp-language-conformance.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

@PaulTopping
Some product features have come in hot, and the 16.10 doc updates aren't all ready to be published yet. We'll have updates to this particular page next week. You can also expect a C++ team blog post about <format> support in the next week or so. For details on what's implemented, you can check the status of the project on GitHub and correlate it to the release. This is apparently an area of active standardization development, with some LWG Defect Reports coming in that will affect C++20 implementations, so expect changes to continue in the future.


add normalizeurls to remove chrome-extension://

This needs some more thought to be sure it covers all the bases, and tests, but I think the solution could look something like this.

Are we sure that we want to send those kind of errors in the first place?
cc @benvinegar @MaxBittker

This issue has a lot of history, and I think we addressed it on Sentry server (e.g. it will parse chrome-extension:// frames correctly now).
Closing.


lts-mainline-longterm >=6.1.16 build failed

The builds are failing because the kernel build now requires rust packages which aren't available in Jammy, but they are being back-ported. The rust bindgen-0.56 package  is currently in proposed. When that package makes it through to the universe repo I'll resubmit the latest kernel build.
This may be the end of the line for focal though.

The builds are failing because the kernel build now requires rust packages which aren't available in Jammy, but they are being back-ported. The rust bindgen-0.56 package is currently in proposed. When that package makes it through to the universe repo I'll resubmit the latest kernel build.
This may be the end of the line for focal though.

Sadly. Does 5.15 now also require rust? I ask because from some version 5.15 on the focal stopped loading normally, the interface is there, but most of the peripherals do not work, because of this I had to switch to 6.1

Concerning focal 5.15 is already officially supported via hwe

This may be the end of the line for focal though.
Even for 6.1? Were some rust dependencies introduced in the change from 6.1.15 to .16?

It looks like the bindgen package has arrived in jammy and 6.2.6 has had a successful build.
WRT focal: the early versions of 6.2 were built without rust, so I've updated the script to back-out those dependencies on 6.2 kernels for focal, and 6.2.6 is building now. I'm not sure what this will mean for 6.3 and newer kernels. It's likely that focal users will have to stick on the latest longterm series of 6.1.x, because the 6.2 kernel will only have a short support period.
In any case Jammy is the current LTS and it's been out for almost 12 months, it's time to upgrade people ;-)

Concerning focal the 5.15 kernel is officially supported via hwe so you do not have to build it your self or use the ppa

This may be the end of the line for focal though.

Even for 6.1? Were some rust dependencies introduced in the change from 6.1.15 to .16?

The 6.1 kernel is the release that added Rust support, but the rust dependencies in Ubuntu kernels were added in the 6.2.3/6.1.16 time frame, so 6.1.16+ is also failing. I can remove the rust dependency from the 6.1 series kernels, and track that LTS as the final focal kernel. All newer kernels are going to require rust.

work-around in 75acf7599eb64f9dbffe3dd417a7bdeb7d7e0e1c

work-around in 75acf75

Looks simple. What are the pitfalls of this method?

@TuxInvader I tried to build the image on the last version of builder and still get the following error on the launchpad:
The following packages have unmet dependencies:
 sbuild-build-depends-main-dummy : Depends: rustc-1.62 but it is not installable
                                   Depends: rust-1.62-src but it is not installable
                                   Depends: rustfmt-1.62 but it is not installable
                                   Depends: bindgen-0.56 but it is not installable

What am i doing wrong?

I'm trying to build because I can't install the version from your repository, it gives an error about incompatible dependencies linux-generic-6.01 and linux-headers-6.1.19-060119-generic

It should be fixed now. The change to remove the rust dependencies broke the rename process.The 6.1.19 package should be fixed on my PPA too.

This is still an issue. The bindgen-0.56 package has not made it into universe yet, and is still in proposed for Jammy. I may need to disable rust in jammy while we wait for it to progress through the verification process. Ubuntu bug is here: https://bugs.launchpad.net/ubuntu/+source/rustc-1.62/+bug/1993183

@TuxInvader what happened with 6.1 builds for focal ?

The Ubuntu repos aren't syncing the changes from mainline builds anymore so I had no working tags to checkout. I fixed the issue by adding support for kernel.org as a git source and applying the ubuntu crack.bundle. I need to update the docs and submit the changes to thise repo, but there are now images available.


Smoothquant LLaMA builds not working on 0.8.0 release

System Info
GPU : NVIDIA A100 80GB
package version
tensorrt-9.2.0.post12.dev5-cp310-none-linux_x86_64.whl
[TensorRT-LLM] TensorRT-LLM version: <IP_ADDRESS>.0
Who can help?
@Tracin @byshiue
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction

Installation
python -m pip install tensorrt_llm==0.8.0 --extra-index-url https://pypi.nvidia.com

Create smoothquant checkpoint for LLaMA
python ./examples/llama/convert_checkpoint.py --model_dir ~/Llama-2-13b-chat-hf  --output_dir ~/fp16-tp4-sq5 --dtype float16 --tp_size 4 --smoothquant 0.5 --per_token --per_channel --workers 4

Expected behavior
Checkpoint should be created.
actual behavior
Error at line - https://github.com/NVIDIA/TensorRT-LLM/blob/v0.8.0/examples/llama/convert_checkpoint.py#L1502
ValueError: You are trying to save a non contiguous tensor: transformer.layers.0.attention.qkv.weight which is not allowed. It either means you are trying to save tensors which are reference of each other in which case it's recommended to save only the full tensors, and reslice at load time, or simply call .contiguous() on your tensor to pack it before saving.
additional notes
No such error seen on release 0.7.1
My guess is that the function get_tllm_linear_sq_weight returns some non-contiguous tensors.

Thank you for the report. We will fix it in next update. As you mentioned, you could add .contiguous() for
results[prefix + 'weight'] = torch.from_numpy(cur_weights).t().clone()

in get_tllm_linear_sq_weight.

Qwen has the same error, When I run
python3 examples/qwen/convert_checkpoint.py \
    --model_dir ./Qwen-72B-Chat-Int4/ \
    --output_dir./Qwen-72B-Chat-Int4-TRT/tllm_checkpoint_1gpu_gptq/ \
    --dtype float16 \
    --use_weight_only \
    --weight_only_precision int4_gptq \
    --tp_size 4 \
    --pp_size 1 \
    --per_group

I got:
ValueError: You are trying to save a non contiguous tensor: transformer.layers.0.mlp.gate.weights_scaling_factor which is not allowed. It either means you are trying to save tensors which are reference of each other in which case it's recommended to save only the full tensors, and reslice at load time, or simply call .contiguous() on your tensor to pack it before saving.


Missing auto indenting inside of switch blocks

VSCode Version: 0.10.11
OS Version: OSX, Win10
Scenario 1:
Create a switch statement:

Create a new line above default:
Inside that line type case 0:
Hit enter
Expected (normal VS product behavior): The editor should auto indent after hitting enter
Actual: No indent occurs. See below

Scenario 2:
Create a switch statement. Use the same as how scenario 1 started.
Move the cursor to after the 'break;' Hit enter.
Cursor will be at the same Ch spot as the b in break.
Type case 0:
Expected (normal VS product behavior): The editor should auto un-indent case 0: after typing the colon
Actual: No indent occurs. Looks like the following:

Sorry about the large code pictures. Appears they greatly expand when the pictures are pasted on a Mac

This isn't limited to switch statements. If statements, for example have the same issue.

@akshita31 Can you see if this still repros?

Yes it does reproduce. Will look into this.

@akshita31 Did we fix this?

Not yet.


Phantom colors and lingering edit mode in saved color panel

Bug Description
Somehow phantom colors + lingering edit mode appeared in the saved color panel
Expected Behaviour
This shouldn't happen
Steps to Reproduce
Unable to recreate. Contact @bethanylang here or in Slack.
Screenshots

Additional Context

Plugin Version:
WordPress Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Verified in QA


Issue #2328: added encoding annotation

Encoding annotation only; jaxrs2 implementation will be in a separate PR.

This PR is for issue #2328


I don´t want to see the print from cudaFromNumpy()

Hi!
Do anyone know if you can turnoff the print from cudaFromNumpy()?

Same problem ... I did the following and just recompile. (it's same procedure in the Jetson Inference code too)
Go to the file python/bindings/PyNumPy.cpp and delete or comment out the line 119;
//printf(LOG_PY_UTILS "cudaFromNumpy()  ndarray dim %i = %li\n", n, dims[n]);`
This disables the cudafromNumpy messages
and ...
Go to the file python/bindings/PyCUDA.cpp and delete or comment out the line 33;
//printf(LOG_PY_UTILS "freeing CUDA mapped memory\n");
and ...
Go to the file python/bindings/PyCUDA.cpp and delete or comment out the line 111;
//printf(LOG_PY_UTILS "freeing cudaMalloc memory\n");

@tttdevelop  Thanks! :-)

If you check out the dev branch of jetson-inference, these are nonow disabled by default. And you can control the logging level from the command line.

Thanks Dusty-nv!


Request permission is not working in iOS but works well in Android.

HI,
Permission check and multiplecheck is working fine in iOS but not in the case of request am i missing something i have follwed the same steps given that works well in android but not in iOS.

Check https://github.com/yonahforst/react-native-permissions/issues/161#issuecomment-332822089


Add Conic Gradient support

Conic gradients is a new draft promototed without any implementation, but with SVG polyfill.
@jonathantneal rewrote polyfill to PostCSS: https://github.com/jonathantneal/postcss-conic-gradient/issues
But the main question is, how to install Cairo.

With brew, this can be as simple as:
brew install cairo

There are installation instructions available, and those instructions worked well with travis.

It makes all cssnext installation process more complicated. Maybe we can remove Cairdo dependency? For example by creating SVG XML by hands?

Maybe. Right now, Cairdo is used to generate the Canvas which is then used to generate the PNG. The original did something like document.createElement('canvas').

Why PNG and not origin vector format?

It may be possible, but this would require an entire redesign of the polyfill.

This is totally cool. That said, the cairo deps seems overkill :/
How does the polyfill behave if cairo is not availabe (error or simple message) ?
What would be cool is that the plugin might not say anything until someone really try to use it. And if cairo is not available, a simple postcss warning will be really cool instead of a breaking error. If the plugin might behave like this, I would be really happy to integrate this feature !

I think PNG solution is good only for first implementation. But maybe we should wait for pure SVG implementation for cssnext without Cairo dependency?

in my experience, projects requiring cairo have always been a pain to install and run

We will probably wait for (or do) a cairo free version then.

postcss-cssnext has been deprecated in favor or postcss-preset-env. Read more at https://moox.io/blog/deprecating-cssnext/


swc asdoc

the fd some sdk not work
exec executable="${FLEX_HOME}/bin/asdoc.bat" failonerror="true"
			 arg line="-source-path 'src'"
			 arg line="-doc-sources 'src'"
			 arg line="-output 'lib/tempDoc'"
			 arg line="-keep-xml='true'"
			 arg line="-skip-xsl='true'"
		exec
this is the working one,need change 2 this.

Where does it have to be changed?

https://github.com/matrix3d/matrix3d.github.io/blame/master/_posts/2016-1-13-flashdevelop-ane-gcc-x86.md#L48
this line

I don't understand where this build.xml comes from - the only FD template with a build.xml is different:
https://github.com/fdorg/flashdevelop/blob/development/FlashDevelop/Bin/Debug/Projects/135 ActionScript 3 - AS3 Library SWC/build.xml.template

you build.xml use the ant asdoc task.my xml use the asdoc.bat from the sdk/bin.
a lot of pep can not work with the ant.
http://stackoverflow.com/questions/15377944/adobe-asc-2-0-and-asdoc
replace your xml 22-27 2 my.xml 48-55,will work good


<EMAIL_ADDRESS>Download failed

kitty: <IP_ADDRESS> -> <IP_ADDRESS>
Updating one outdated app:
Updating 'kitty' (<IP_ADDRESS> -> <IP_ADDRESS>)
Downloading new version
OperationStopped: C:\Users\emada\scoop\apps\scoop\current\lib\install.ps1:130
Line |
130 |          throw $e
|          ~~~~~~~~
| The remote server returned an error: (404) Not Found.

59e4fa6bc57695073213ce8a17a89bb7c689ad30


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter wifilight for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.wifilight
   -
ERRORS:

[x] :heavy_exclamation_mark: [E114] Admin support not specified. Please add "common.adminUI.config = json|materialize|html|none"

WARNINGS:

[ ] :eyes: [W105] Missing suggested translation into uk of "common.titleLang" in io-package.json.
[ ] :eyes: [W109] Missing suggested translation into uk of "common.desc" in io-package.json.
[ ] :eyes: [W145] Missing suggested translation into uk of some "common.news" in io-package.json.
[ ] :eyes: [W170] "common.keywords" should not contain "iobroker, adapter, smart home" io-package.json

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
Note: This issue replaces issue #408
@mcm1957 for evidence

Thanks for processing and closing this issue.
I will recheck for new issues later. So it's no problem if this issue has been closed premature.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.  Let's work together for the best user experience.
your
ioBroker Check and Service Bot


Enable member completions for import types

Fixes one of the points brought up in this comment.

@sheetalkamat Can I get a rereview?


Torch Load Warning causing test suite to fail.

🐛 Describe the bug
Personally, I'm unable to follow the codepath, but it seems that the recent warning introcued in pytroch 2.4 is causing the test_transforms_v2 tests to fail due to the very strict warnings.
https://github.com/pytorch/vision/blob/main/test/test_transforms_v2.py#L57
2024-08-24T19:20:41.2823509Z _____ TestResizedCrop.test_transform[make_segmentation_mask-ratio-value2] ______
2024-08-24T19:20:41.2825077Z test/test_transforms_v2.py:3659: in test_transform
2024-08-24T19:20:41.2825572Z     check_transform(
2024-08-24T19:20:41.2826263Z test/test_transforms_v2.py:397: in check_transform
2024-08-24T19:20:41.2826701Z     pickle.loads(pickle.dumps(transform))
2024-08-24T19:20:41.2827710Z ../_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.10/site-packages/torch/storage.py:414: in _load_from_bytes
2024-08-24T19:20:41.2828812Z     return torch.load(io.BytesIO(b))
2024-08-24T19:20:41.2829850Z ../_test_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_plac/lib/python3.10/site-packages/torch/serialization.py:1040: in load
2024-08-24T19:20:41.2830545Z     warnings.warn(
2024-08-24T19:20:41.2832296Z E   FutureWarning: You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.

Versions
Pytorch 2.4
Conda-forge
sample log from https://github.com/conda-forge/torchvision-feedstock/pull/94:
181.log.txt

It looks like the issue might be in line 57 of test_transforms_v2.py
pytestmark = [pytest.mark.filterwarnings("error")]

I could be wrong. The comment above it suggests that warnings are turned into errors, and your log suggests that you're using torch.load 'with weights_only=False'. Is this explicitly called in your arguments or are you relying on the default value?

I am running the test suite when building the package for conda-forge.
I've removed that line to move forward with our builds, but I mostly wanted to flag this to the pytorch team as a whole.
We sometimes find ourselves a little more bleeding edge at the conda-forge builds, but I found it strange since torchvision officially supports pytorch 2.4.0 and that is when that warning first appeared.

Thanks for the report @hmaarrfk .
I think the error you're observing has already been addressed in both torchvision and pytorch codebases.
I had opened https://github.com/pytorch/vision/issues/8517 in torchvision and https://github.com/pytorch/pytorch/issues/130242 in pytorch, and https://github.com/pytorch/pytorch/pull/130246 was the fix. It may not have been picked up in pytorch 2.4 but hopefully it should be included in pytorch 2.4.1 (and in any case, in 2.5).

great thanks!

My reading of the solution is that you specified False which stops triggering the warning, but you left the "unsecure loading".
hmm, i have to say, one of the things i dislike the most about pytorch is its usage of pickle to load and store data structures.
I understand it enables fast iteration, but these security problems are a long standing issue in the design of pickle. However I also understand that it is difficult to move away from storage formats.
So good luck working through this migration (torchvision + pytorch!).
We are looking forward to continuing to be users of your software!
Hopefully you don't find my opening of issues too annoying.


Can I use .pb file or .ckpt file as my weights?

I have already trained my data, but its saved as .ckpt file.
Is there have any way to use .ckpt file as weights in this ROS package?

You can't, did you train using pytorch? Try this pytorch darknet

Seems like #242, maybe @dregner has a solution for you.


Published paper

Can you tell me the title of the paper corresponding to this project.
Thank you

https://github.com/hayashi-laboratory/deeplabcut-analysis/issues/2#issuecomment-2279776177


Does this work on the Elite series 2?

Elite series 2 just came out, and unfortunately it still has the same limitation in the paddles, not being able to be mapped as their own button

Unfortunately it doesn't even work with the regular Elite anymore because
Microsoft removed the APIs I was using :(
On Mon, Nov 4, 2019, 8:39 PM arvz<EMAIL_ADDRESS>wrote:

Elite series 2 just came out, and unfortunately it still has the same
limitation in the paddles, not being able to be mapped as their own button
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/saqneo/Elite/issues/7?email_source=notifications&email_token=AD2FGZBGIXPD4CM4FY4E7F3QSD2GZA5CNFSM4JI5QB4KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HWZDEKQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD2FGZDVHOVCEGM3B4SWADTQSD2GZANCNFSM4JI5QB4A
.

I haven't looked into it any closer yet, but there seems to be an api that allows reading of the paddles state: https://docs.microsoft.com/en-us/uwp/api/windows.gaming.input.gamepadbuttons

Yeah, last time I took a look this API was hidden so I wasn't able to use
it effectively (rather not require copying files like I do with the current
implementation since so many people have had trouble with it). Maybe it's
possible now but I haven't tried to get it working on current versions of
Windows; I'll give updates if I ever take another shot.
On Mon, Dec 23, 2019, 10:09 AM mooop12<EMAIL_ADDRESS>wrote:

I haven't looked into it any closer yet, but there seems to be an api that
allows reading of the paddles state:
https://docs.microsoft.com/en-us/uwp/api/windows.gaming.input.gamepadbuttons
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/saqneo/Elite/issues/7?email_source=notifications&email_token=AD2FGZC3NRMQF36O4C463CLQ2DPC3A5CNFSM4JI5QB4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHRNN5Y#issuecomment-568514295,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD2FGZFERQPRFNE47UGNOVDQ2DPC3ANCNFSM4JI5QB4A
.

Quickly I ported the app to use the new api, but it does not seem to register the paddles, all other buttons are working. I tried an empty profile in the controller and I tried the default one, with the same effect. Any idea what this might be about?

Iirc I wasn't able to instantiate the correct type of gamepad device
because the factory wasn't available to construct the one I thought would
support paddles (visible in the dll via reflection).
If you're a strong coder you could try instantiating the object in C++ via
COM, which I believe is what the app uses to communicate with the
controller since it existed long before the new APIs. Or maybe they've
changed it; if you're familiar with ildasm you can use reflection to find
out.
I'm home for the holidays right now so I can't take a look directly,
hopefully this is informative though.
On Mon, Dec 23, 2019, 1:21 PM mooop12<EMAIL_ADDRESS>wrote:

Quickly I ported the app to use the new api, but it does not seem to
register the paddles, all other buttons are working. I tried an empty
profile in the controller and I tried the default one, with the same
effect. Any idea what this might be about?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/saqneo/Elite/issues/7?email_source=notifications&email_token=AD2FGZF7STBKXVR4F2FJC7DQ2EFSFA5CNFSM4JI5QB4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHRYODY#issuecomment-568559375,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD2FGZC2ZSI4HITBNE3BZ7LQ2EFSFANCNFSM4JI5QB4A
.

C++ is beyond my area of expertise I'm afraid. I'm not even sure which library implements these methods, the aforementioned namespace only contains externs for those methods and I'm not sure how to check which library it calls.
I also tried RawGameController (it would make sense for it to return raw, unmapped data, right?), but the effect is the same, paddles are mapped to the buttons from the Xbox Accessories app.
I hope you can get it working when you're back, good luck. In the meantime, Merry Christmas.


Change github.com links to the new home

As the title says, pretty much a replace of github.com/boson-project/[func|faas] with the new coordinates.

(CI failure is fixed in #419)

(CI failure is fixed in #419)

Re-running CI jobs now that #419 has landed. Hopefully that will succeed.

/lgtm
TY

/lgtm

/lgtm


New version: Bitnami.ReportServerCommunity version <IP_ADDRESS>66-0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.1 schema?

Microsoft Reviewers: Open in CodeFlow

I'm not sure how to fix the https://github.com/microsoft/winget-pkgs/labels/Validation-Unattended-Failed

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot waivers Add Validation-Unattended-Failed


Undesired tostring()/format() results for large numbers

While #97 by @tims-bsquare did fix handling of large integers for the most part, it introduced a few bits of undesired behavior with how Lua converts large numbers to strings that would be great to continue to improve if possible.
tostring()
I missed this while reviewing that PR, but suffixing large integer values with ".0" did end up breaking my application. This is exemplified by the test case:
        const res = await engine.doString(`return tostring(value)`)

        // Since it's a float in Lua it will be prefixed with .0 from tostring()
        expect(res).to.be.equal(`${String(value)}.0`)

The following patch can be made to lua's tostring as a workaround for this:
    const tostringOrig = lua.global.get('tostring');
    lua.global.set('tostring', (val: any) => {
      const str: string = tostringOrig(val);
      if (typeof val === 'number' && Number.isInteger(val) && str.endsWith('.0')) {
        return str.slice(0, -2);
      }
      return str;
    });

Is this something wasmoon could integrate in some way?
string.format() / concatenation
This one is a bit trickier. Here is some test Lua code:
local val =<PHONE_NUMBER>0
print("TEST1 "..val) -- > TEST1<PHONE_NUMBER>0.0
print(("TEST2 %s"):format(val)) -- > TEST2<PHONE_NUMBER>0.0
print(("TEST3 %d"):format(val)) -- Error: test.lua:74: bad argument #1 to 'format' (number has no integer representation)

The first two prints behave as I would expect given the current implementation. It would be awesome if they would match the stock lua implementation without the ".0" suffix, as above.
The last print crashing is definitely not ideal though. For what it's worth, the stock lua implementation results in "TEST3<PHONE_NUMBER>0" for this one as well.

I think this one might not be sensible, or possible to fix, though I'm open to suggestions.
I'm not keen on overriding the tostring behaviour because that just hides that larger numbers are represented as doubles rather than integers so I think I'd like to keep it in line with Lua's expected print behaviour.
WASM/Emscripten only supports up to 32 bit integers but does support 64 bit doubles whereas when Lua runs on your host machine that supports both 64 bit integers and doubles so for your case that large number can be represented as an integer but in WASM it can't. During my last PR I updated the README.md to document the number behaviour https://github.com/ceifa/wasmoon#numbers
The new behaviour now matches fengari at least, an even more mature project than this one, if they don't have a better solution yet I'm not sure I'll think of one. https://github.com/fengari-lua/fengari#integers

@ceifa you can close this one too now thanks to @Sapu94 's research and PR :)


Allow setting license(s) via chart values

Dear SonarSource team,
it would be nice to be able to set license(s) via chart values for non-CE instances of SonarQube. Other product's charts like from Artifactory or the Atlassian Data Center charts are good examples for that. Being able to do this allows engineers to inject all kind of secrets into the application from a Vault or similar just during the Helm deployment.
I'm looking forward for your feedback
Thank you and many regards
Patrick

Hi @pathod,
Thanks for the report, this feature is indeed not supported by the helm chart as of now.
Since it is a new feature request, could you please open a new thread in the forum https://community.sonarsource.com/c/suggestions/features?

Hi @leo-geoffroy-sonarsource,
before closing this issue, it would be interesting to know, whether SonarQube supports another way besides the UI to set licenses (I couldn't find any). If there is another a way, preferentially via the filesystem, I could contribute a pull request to introduce this feature.

Hi @pathob ,
there is none for the filesystem. There is an API endpoint (POST api/editions/set_license) but using it would result in similar downside as the change admin password hook, therefor we will probably not implement this without a change of the application in the helm chart. As there is currently no feature request for this, you should go ahead and create this feature request as stated by my colleague @leo-geoffroy-sonarsource


print stack trace only in debug mode

Related to #1359

test

test package_version#latest

test package_version#latest


feat(refresh-coda-cache workflow): add workflow.

Adds a new workflow that can be triggered manually to refresh cached data. Based off of https://github.com/StampyAI/stampy-ui/blob/e9647cdaa27110c515cf47160a0ad81651b62c1d/.github/workflows/questions-encodings.yml
Requires a PUSH_AUTH_TOKEN secret with a PAT with write permissions on the repo code.

@Aprillion would you be willing to create and add the PAT to the repo secrets? It can be a fine-grained PAT with only write privileges for the repo.
One other thing about this workflow is that it seems bypass branch protection rules. I set master to require a PR on my fork but was still able to run this workflow. Not a deal-breaker IMO, but something to be aware of.

oof, I had to enroll the org to allow PAT first in https://github.com/organizations/StampyAI/settings/personal-access-tokens, now I can change the Resource owner in https://github.com/settings/personal-access-tokens/new to StampyAI to be able to select this repo and generate the PAT 😓
I created one with the Contents permission with longest possible expiration (1 year) and copied it to a new repository secret PUSH_AUTH_TOKEN in https://github.com/StampyAI/stampy-ui/settings/secrets/actions, so hopefully it will work 🤞

@Aprillion sounds good, thank you 🙇


recvMsg reads past the end of the buffer

According to cmsg(3), cmsg_len includes header data and alignment, but the function cmsgExtractData from Network/Socket/Msg/CMsgHdr.hsc ignores that fact.
As a result, recvMsg always return some extra garbage data past the end of buffer.
A possible fix may be implemented this way:
diff --git a/Network/Socket/Msg/CMsgHdr.hsc b/Network/Socket/Msg/CMsgHdr.hsc
index 24bac7d..0d26c95 100644
--- a/Network/Socket/Msg/CMsgHdr.hsc
+++ b/Network/Socket/Msg/CMsgHdr.hsc
@@ -106,9 +106,10 @@ cmsgExtractData :: Ptr CMsgHdr -> IO (Maybe B.ByteString)
 cmsgExtractData p = do
     let dataPtr = castPtr $ c_cmsg_data p
     dataLen <- cmsghdrLen <$> peek p
+    let dataSize = (fromIntegral dataLen) - (fromIntegral $ c_cmsg_len 0)
     if dataPtr == nullPtr
         then return Nothing
-        else return.Just =<< B.packCStringLen (dataPtr, fromIntegral dataLen)
+        else return.Just =<< B.packCStringLen (dataPtr, dataSize)

 peekCMsg :: Ptr CMsgHdr -> IO (Maybe CMsg)
 peekCMsg pCMsgHdr =

Make this a pull request please.

Make this a pull request please.

Make this a pull request please.

Sorry for the long wait. Merged.


Expose the port_open method to Automate Methods from PortScanner

Purpose or Intent
Added a utility instance that can be fetched from the service handle.
On that utility instance we can call the port_open method.
Links

PR https://github.com/ManageIQ/manageiq/pull/10406

Steps for Testing/QA
Automate Method Script
open = $evm.utils.port_open('<IP_ADDRESS>', 22)
$evm.log(:info, "ssh port open on <IP_ADDRESS> #{open}")
or
utils = $evm.utils
open = utils.port_open('<IP_ADDRESS>', 22)
$evm.log(:info, "ssh port open on <IP_ADDRESS> #{open}")

@gmcculloug @Fryguy
Please review

@Fryguy @gmcculloug
Please review

@Fryguy @gmcculloug
Please review

@Fryguy @gmcculloug
Please review

@Fryguy @gmcculloug
Please review

Moved to new repo ManageIQ/manageiq-automation_engine
PR https://github.com/ManageIQ/manageiq-automation_engine/pull/13


Support reference to schema by file path

Support identifying the schema to be tested by file path, eg:
./postman/schemas/myapi.yaml
It's a best practice in CI and GitOps to reference assets that exist in the repository.  This capability would ease integration of the generator in CI pipelines.  Because the tests are tied to a specific asset in a specific commit it will also be auditable.

I've added the ability to reference a schema by URL in PR #6. I'll take a look at using a file path as wll.

I actually found a way that you can do this as follows:
name: Automated API tests using Postman CLI

on: push

jobs:
  automated-api-tests:
    runs-on: ubuntu-latest
    steps:
      - name: Read README.md
         id: schema
         uses: jaywcjlove/github-action-read-file@main
         with:
             branch: development
             path: <path-to-schema.yaml>
      - name: Install Postman CLI
         uses: actions/checkout@v4
         run: |
          curl -o- "https://dl-cli.pstmn.io/install/linux64.sh" | sh
      - name: Login to Postman CLI
         run: postman login --with-api-key ${{ secrets.POSTMAN_API_KEY }}
      - name: Run API tests
         run: |
          postman collection run "<collection-id> --env-var "env-schema=${{ steps.schema.outputs.content }}" --env-var "env-server=<server URL>"

I don't see a way of doing in the scripts themselves, so using actions to do the work for us works well.


Modem chat implement transmit idle

Update the modem chat module to use the transmit idle event. This addition removes the need for the process_timeout parameter passed to the modem_chat module in the init function, so this PR removes all in-tree setting of this property.

Shouldn't this change be mentioned in the release notes?

@henrikbrixandersen Yes, this is the last PR of a series of PRs that have been merged over the last few weeks to implement transmit idle throughout the modem subsystem, I will create a PR to document the addition after this one :)


check and set IPv6 salt settings during migration (bsc#1227172)

What does this PR change?
see title
Test coverage

No tests

[x] DONE

Links
Issue(s): https://github.com/SUSE/spacewalk/issues/24711

[x] DONE

Changelogs
Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository
If you don't need a changelog check, please mark this checkbox:

[ ] No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)
Before you merge
Check How to branch and merge properly!

@cbosdo I haven't tested it, I'm going to do that now, but in the meanwhile it would be great if you think it's the right approach, thanks!

Bug was at the end closed as invalid. And I tend to agree. With migration, we should not modify this config. People could have had ipv6 disabled intentionally, e.g. for lack of firewalling solution, etc.
With new install, that is different question.

Indeed, let's close it: it's dangerous


Incorrect parsing of times without separators ('20101112', etc)

This works:
moment('2010', 'YYYY').format('YYYY');
"2010"

These don't work:
moment('201211', 'YYYYMM').format('YYYY');
"201210"
moment('20101112', 'YYYYMMDD').format('YYYY');
"NaN"

In order to keep the speed of the parser relatively fast and the filesize of the library relatively small, input strings require separators.
Even if speed and size were not an issue, there is still the problem on non-zero indexed months and days and other edge cases that would cause an implementation of this to be buggy.
moment('2010111', 'YYYYMD'); // Jan 11 2010 or Nov 1 2010

I have an idea for a new parser that I may or may not build someday. Instead of spitting out a date, it spits out an array of dates with a estimated accuracy for each. That way, each developer could set the threshold of accuracy that they will accept, and if the input was not accurate, they can prompt the user to confirm based on the estimated matches.
All this to say, I won't be adding support for un-separated input strings at this time.


added a few new methods

added 3 new method to root finding and 2 new method for interpolation

lgtm!


When to destroy activity's presenter

We need a more reliable way to detect when to destroy activity's presenter.
Here is where the issue that started the discussion: https://github.com/konmik/nucleus/issues/42#issuecomment-204233175
TL/DR: activity's finish() can be called some time after onPause, so this code will leak the activity's  presenter: https://github.com/konmik/nucleus/blob/master/nucleus/src/main/java/nucleus/view/NucleusActivity.java#L76
Possible solutions:
a) Leverage retained fragments. This will solve the presenter destruction issue but we will leak background tasks sometimes.
b) Extend PresenterLifecycleDelegate lifecycle to explicitly call onDestroy.
c) Install system-wide lifecycle listener registerActivityLifecycleCallbacks.

if a activity may not restart forever,why not leak the background tasks?

Because some background tasks can be very resource-consuming (streaming, image processing, etc).

I think the system will drop an activity only after a certain time(
maybe several minutes).The resource-consuming tasks will be completed in the certain time and the result have already been saved with the help of saveInstanceState.
if the tasks is long enough to exceed the certain time.Just let it be a tasks with no end.Tasks will accumulate in such situation,which waste the system resource and the system will crash in the end.

Android never drops activities after some time. It drops them only with the entire process.

No,If an activity is paused or stopped, the system can drop the activity from memory by either asking it to finish, or simply killing its process

Do you have any references to docs or tests?
I created a test once which runs 100500 activities one on top of another - just to check if Android will finish some of background Activities when it becames short on memory. It didn't.

There is also a discussion on stackoverflow somewhere with the same conclusion.

Yes, doc I think the system drop the activity in this situation because of  time but not memory.

:D such an obvious lie in official docs. I never had a sutuation when Android finishes my Activities without my own command. It can finish them indirectly if I say FLAG_ACTIVITY_CLEAR**, but it never finishes them by itself.
How much time do you think we need for activity to be finished because of time?

I've found something interesting: http://developer.android.com/guide/components/tasks-and-back-stack.html
"If the user leaves a task for a long time, the system clears the task of all activities except the root activity. "

http://developer.android.com/guide/topics/manifest/activity-element.html#always

Ok,It said 30 minutes .I happened to see it before and I forget where,you find it.

It is always nice to know more info about these lifecycles, thanks for mentioning them! :)

Hey, did you red this article? i think it is really nice way to bound to Activity lifecycle.

@bejibx I hope you're kidding. This ugly workaround using one of the most buggy Android APIs will never make its way to my code. Never.

Workaround is kind of ugly, true, but it seems to me pretty much every possible solution to beat Activity lifecycle is ugly. Still I'm a bit confused, because loaders seems to work like a charm in my projects before.

@bejibx you're lucky then. Their lifecycle is not the same as Activity has and not the same as process has. So, they obviously add complexity instead of reducing it. And, in addition to the added complexity, it has so many callbacks and unneeded API complexities like sending arguments in a Bundle, forceLoad... they just suck.

Nucleus exists only because loaders do not work.

I got your point and I'm definitely agree with you about Loaders API over-complexity. I hope I'll have some time on this weekends to investigate Android Activity and Loader sources to see if system somehow diferentiate restart from complete destroy.

https://github.com/konmik/nucleus/commit/44c7722e32d1d54eb95f126586c79f114c4b14a0

44c7722 do not fix the situaction in case 2 "restart-activity" in your post.In that situation,the method isFinishing still returns false.

@duo2005duo Which post do you mean?

This one?
"Case 2: An Activity restart happens when a user has set "Don't keep activities" checkbox in Developer's settings and another activity becomes topmost." ?

@konmik  Yes

I think it will have isFinishing during its next onDestroy.

@konmik this is true, just checked it in sample project.

Of course.But do you think the retained fragment lifecycle is similar to current presenter lifecycle？Combining staticholder and saveInstanceState just do the same with retained fragment.I think it voliate KISS.

And I think the loader lifecycle is the same with retain fragment.They are both NonConfigurationInstances.

@duo2005duo the idea is to not have the same background task running twice. Without having a static variable we can't detect if the task is still running. Ratained fragments do not fit because they get destroyed during "case 2".

Yes,But I think case 2 is not so important,The user will not do this except the developer,so I said "similar". And it still need to be tested whether it will have isFinishing during its next onDestroy for cleaning task  we talked about yesterday

And I think it will not have its fininishing,the activity will not destroy twice without have a second onCreate.

@duo2005duo You're right this is not very important. But why complicate things and use fragments if we can do the same without them? Fragments is not the only thing we need presenters for.
It will have a second onCreate and a second onDestroy with isFinishing. I tested.

The result is interesting !:-D

Also I found this mentioned earlier stackoverflow conversation about Android destroying activities. So I think when lead Android engineer says

The only memory management that impacts activity lifecycle is the global memory across all processes, as Android decides that it is running low on memory and so need to kill background processes to get some back.

we can trust him.

I think I found this rare case when we can leak our presenters because isFinishing() returns false.
What we need is "Do not keep Activities" developer option enabled and following backstack:
Activity 1 -> Activity 2 -> [Activity 3]
So at this point Activity 1 and 2 was destroyed with isFinishing() == false and we expect to destroy our pesenters when user navigates to them later with back button. Now imagine if we use up navigation from Activity 3 to Activity 1. onDestroy on Activity 2 will not be called (tested) and we'll lost our presenter.

@bejibx You're right, we will have a leak in this case. I was already thinking about this.
I think we need to recommend android:alwaysRetainTaskState flag to prevent such leaks.

While this flag doesn't help with "Do not keep Activities" it will help with Android finishing activities by itself.

But did you test if isFinishing() == false when Android destroys Activities this way? I was trying to test it yesterday - I leaved my task for something around 3 hours and when I checked, it turned out that Android just killed my process. Maybe thats because my test was incorrect - I had only 1 task.
Also I tested what is gonna happen with flag android:clearTaskOnLaunch="true". Find out isFinishing() == true on all destroyed Activities.

Also just found this interresting stackoverflow article.

final ActivityRecord resetTaskIfNeededLocked(ActivityRecord taskTop,
            ActivityRecord newActivity) {
        boolean forceReset =
                (newActivity.info.flags & ActivityInfo.FLAG_CLEAR_TASK_ON_LAUNCH) != 0;
        if (ACTIVITY_INACTIVE_RESET_TIME > 0
                && taskTop.task.getInactiveDuration() > ACTIVITY_INACTIVE_RESET_TIME) {
            if ((newActivity.info.flags & ActivityInfo.FLAG_ALWAYS_RETAIN_TASK_STATE) == 0) {
                forceReset = true;
            }
        }
//...

If we look at ActivityStack#resetTaskIfNeededLocked(ActivityRecord, ActivityRecord) function we can see that no matter why activity should be killed - because of inactive reset time came off or because clearTaskOnLaunch flag was setted on Activity, actual Activity-by-time-removal mechanism stays the same, it only affects forceReset flag. So I assume if isFinishing() correctly returns true in my second test, we'll have same result for Activities destroyed by inactive reset mechanism.

I'm thinking about retained fragments and onRetainCustomNonConfigurationInstance :(

Here is a possible implementation, BTW: https://github.com/square/flow/blob/master/flow/src/main/java/flow/InternalLifecycleIntegration.java
I need to test retained fragments over edge cases.

Calm down for a moment, we just need to test first what happens with Activity when system destroys it because of time. It is confirmed this could be easily reproduced on Android 2.3:

This behaviour can be easily reproduced on devices running Gingerbread and earlier. Launch an app and create some back history, then hit the home button and wait half an hour. Launch the app again from the home screen and the state has been cleared as if it were starting a new task. Perfect.
I'll test this on emulator tonight or tomorrow if I there is no time today and report results.

@bejibx Even if it will call onDestroy with isFinishing this will not solve the "Do not keep activities" + "Clear stack" issue.

moving to retained fragments will also automatically solve fragment's presenter lifecycle problem where we need to manually destroy presenter.

We will have sometimes several instances of the same background task running but it will, at least, be a predictable scenario every other android developer is facing.

@konmik  The RxJava will stop the unneccessary task instances as soon as possible with the help of isUnscribed(). So it's OK.

Hey, can we use isChangingConfigurations() instead of isFinishing()? So presenter will NOT be destroyed only if it configuration change incoming.

@bejibx awesome. I will try it.

3.0.0-beta is on maven

https://github.com/konmik/nucleus/commit/2863995e40d65d144843f91e0e646a5d4be6248d


doc: fix checkpoint syslog page

:tada: This PR is included in version 2.25.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Error displaying /history

When the number of ISSUES + MAINTENANCE is greater than 25 and you try to access to /history an error screen appears.

I have this too. Here is a log from our instance:
==> /var/lib/staytus/log/puma.error.log <==
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04] Started GET "/history" for <IP_ADDRESS> at 2018-02-20 11:10:11 +0100
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04] Processing by PagesController#history as HTML
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]   Site Load (0.8ms)  SELECT  `sites`.* FROM `sites` ORDER BY `sites`.`id` ASC LIMIT 1
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]   Rendering content/themes/company/views/pages/history.html.erb within layouts/company
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]   HistoryItem Load (1.0ms)  SELECT  `history_items`.* FROM `history_items` ORDER BY `history_items`.`date` DESC LIMIT 25 OFFSET 0
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]   Maintenance Load (1.2ms)  SELECT `maintenances`.* FROM `maintenances` WHERE `maintenances`.`id` IN (20, 14, 16, 17, 10, 19, 18, 15, 9, 13, 12, 8, 11, 6, 3, 2, 7, 1, 5, 4)
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]   Issue Load (0.8ms)  SELECT `issues`.* FROM `issues` WHERE `issues`.`id` IN (6, 5, 4, 3, 2)
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]    (0.9ms)  SELECT COUNT(*) FROM `history_items`
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]   Rendered content/themes/company/views/pages/history.html.erb within layouts/company (61.6ms)
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04] Completed 500 Internal Server Error in 75ms (ActiveRecord: 4.8ms)
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04] ActionView::Template::Error (unable to convert unpermitted parameters to hash):
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     11:     <%= first_page_tag unless current_page.first? %>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     12:     <%= prev_page_tag unless current_page.first? %>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     13:     <% each_page do |page| -%>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     14:       <% if page.left_outer? || page.right_outer? || page.inside_window? -%>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     15:         <%= page_tag page %>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     16:       <% elsif !page.was_truncated? -%>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]     17:         <%= gap_tag %>
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04]
[9c0c0987-85e0-40c6-ac74-34ffe90f2e04] app/controllers/application_controller.rb:22:in `set_time_zone'

Are you running the latest version of Staytus? This issue was fixed quite recently.

I just did the updrade and unfortunately, I'm still having the issue. Tried both, public and admin pages.
Then, I tried to restart with procodile but I keep getting the error message.
Any idea?


⚠ Bofrost.lu has degraded performance

In f5aa393, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1543 ms

Resolved: Bofrost.lu performance has improved in 80bf74c after 13 minutes.


Feature/563 referrer can submit application

Context
Allow a referrer to submit an application.

Changes in this PR

Allow submission of an application
Build application document on submission
Mark check your answers task as incomplete if a previously answered task is changed
Require confirmation checkbox to be checked before submission

Out of scope:

Checking all answers are complete (incomplete applications can still be submitted)
Confirmation page changes

Note: I wanted to include a new fixture for applicationTranslatedDocument.json to match the other CAS services, but I ran out of time. Maybe this can be included in the task for creating the submitted application view.
Screenshots of UI changes
Before

After

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Are any changes required to the e2e tests?
[ ] If you've added a new route, have you added a new
auditEvent? (see server/routes/apply.ts for examples)
[ ] Are there environment variables or other infrastructure configuration which needs to be included in this release?
[ ] Are there any data migrations required. Automatic or manual?
[ ] Does this rely on changes being deployed to the CAS API?

Post merge checklist
Once we've merged it will be auto-deployed to the dev environment.
Find the build-and-deploy job in CircleCI.
Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

 Referrer can submit application


[feature]: changes in blog read button

Related Issue
Closes: #1458

Description

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@akshitagupta15june ok I will change it to either bright yellow or orange

@akshitagupta15june  I tried using your suggested color for maybe bright yellow or orange but they dont reall suit that well. Rather than that, i will show you some colors you can check them out and then find out whatever suits accordingly, I can change the shades in the color itself for better appeal.

Still as the background is dark then the colour of button should be bright yellow or light blue and then bold text


Minor changes

Coverage remained the same at 85.83% when pulling f6070ea5afea4fcab12a18fa38ab2a3d0ba32b2e on JYL123:minor_changes into c8da45bc22ef8027fae05108a6470a1e63ec3f48 on CS2103AUG2017-W11-B1:master.