Raycasting on Gltf

From the documentation it says :

GLTFloader :
<GLTFLoader> behaves as a <Group> containing meshes loaded from the .gltf or glb file.

But onClick or onMouseMove never get triggered !
I try also to remove :enableRaycasting="true" @click="onClick"
from GLTFLoader to add it to <Group>, but it's not working
Is it because all the initialisation appends before the gltf file being fetch ?
I dont know if it's an issue or an impossible use case :D
Thank you for the hard work on drie ! it should have 100k ⭐
<template>

      <Renderer ref="renderer" :antialias="true" resize="window">
        <PerspectiveCamera ref="camera" :position="[3, 3, 5]" :up="[0, 0, 1]" :far="200000">
          <TrackballControls :rotate-speed="0.01" :pan-speed="0.003"/>
        </PerspectiveCamera>
        <Scene>
          <AmbientLight ref="light" />

            <GLTFLoader
                :enableRaycasting="true" @click="onClick"
                url='/test-mesh/'
                @load="onCreation"
                @mousemove="onMouseMove"
            >
            </GLTFLoader>

            <AxesHelper :size="30"/>
        </Scene>
      </Renderer>

</template>

`

PS: I'm trying to figure out your documentation so I can contribute a bit

Thanks for reporting! I just forgot to use the magic manageParentRelationship in the GLTF and OBJ Loaders.
I tested click on GLTF and it works for me. Can you test all the events in your project? There may be another issues.
Note that onClick argument should be is too and not group (See Intersection interface. You can get the intersected Object3D using is[0].object.
Regarding the docs - most of it is autogenerated from source (props, events) and the rest contained in the <docs></docs> block of the .vue SFC. I need to add some tutorial on how to docs:dev and docs:build. It's a bit hacky now :)
I'll also add a guide on how to implement new components, but the library doesn't seem mature enough. I'm changing the internals all the time.

Working like a charm, I've tested all available @event (some more than others)
but I have a weird Error (probably a warning?!) :
AddMesh.vue?73eb:106
 Uncaught TypeError: Cannot read properties of undefined (reading 'face') at Proxy.onClick
(AddMesh.vue?73eb:106:1)

But the code compute well and it still run !
<template>
   ...
   <GLTFLoader
       url='/test-mesh/'
       @load="onCreation"
       enable-raycasting
       @click="onClick"
   >
   </GLTFLoader>
   ...
</template>

If you can't recreate it on your side nevermind, its still run !
Just I've got a bit of red in my console I feel bad eheh

It is really laggy compared to using :
window.addEventListener("pointerup", this.onClick, false);

this.onClick a method with my own raycasting management (not using enable-raycasting props)

I tested with pointerup and click, same result

I can show more code if needed

Sorry for the missleading information, after much more testing it's kindly the same... weird I might sleep more
Thank you for this fix!

You were right! The way raycasting was implemented was not performant. enableRaycasting always fired raycasting on @mouseMove even when you didn't listen to the event. This is expensive for large models when you are only interested in click event.
I have moved raycasting event listeners into props - callbacks. Please replace @click to :click etc. This way raycasting works on demand - if you only provide :onClick function - no raycasting will be performed on mouse move as expected.
Sorry for the API change, but this way it's much more performant. Please report any issues.
It's implemented in the latest version 1.0.0-alpha-1.34

Can I close this issue? It seems resolved.

you can close it!


misspelling

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/data-guide/relational-data/data-warehousing.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

I believe the "Dynamic scalability" row is correct for SQL DW (it supports dynamic scaling) although the term "scale up" may not be accurate for DW, as it scales by adding compute nodes.
I suggest reverting "No" to "Yes" but deleting the phrase "(scale out)"

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/data-guide/relational-data/data-warehousing.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fixed deprecated Code

Hey, I fixed the deprecated code in both the Control App and the test App and also updated the gradle. The app seems to be working perfectly given the new changes.

Since we've changed to a lambda function, I've slightly refactored the code to make it more readable.

Have tested the code and all runs well. So, I have no further concerns. Looks all good to me!


[Bug] Nhentai setting for displaying title as short title doesn't works as it should be

Device information

Tachiyomi version: Preview r280
Android version: 9
Device: Huawei ANE-LX2

Steps to reproduce

Open Nhentai settings
Change Display manga titles as: to short title
Open Nhentai, select a random manga
Title refresh itself to full title

Expected behavior
Title should remain as short
Actual behavior
Title changes to full
Other details
I tried same thing in Tachiyomi, it was working well.
(Sorry for my bad English.)

Added in https://github.com/jobobby04/TachiyomiSY/commit/99457526674d309431abdec17eb5fcb07b29768d

thank you


Add Tests to Bot vyper repos

Add tests to the following repos, so they can be audited.
Add the details of how to run the tests in the readme

[ ] https://github.com/VolumeFi/curve-limit-order-bot-vyper
[ ] https://github.com/VolumeFi/curve-twap-bot-vyper
[ ] https://github.com/VolumeFi/offramp-eure-bot

@webelf101 FYI - here are the requirements for test from our auditors https://docs.google.com/document/d/10_t7Kt814Otp-FMFK8mvCsxb3tX3wyu1Z9V4nhZxTY8/edit#heading=h.7lbrpb7hdwvw


Fix #681

Fix #681 https://beta.elementary.io/branch/fix-681/

Breaks small screens.

I am happy that this works on both small and large screens, with the only issue being pre-existing (background doesn't modify height to fit changes in window dimensions).


the pretrained backbone on IMAGENET?

No such file or directory: 'model_best.pth.tar'
Could I get the pretrained model on imagenet? Thanks!

Hi, yes I trained the backbone with several epochs (around 10-15) on ImageNet then used it for training YOLOv1. You can train it without pre-trained weights also but there is high probability of gradient vanishing. You can set pretrained weights to False on this line https://github.com/yakhyo/yolov1-pytorch/blob/main/main.py#L62
from
net = YOLOv1(pretrained_backbone=True).to(device)

to
net = YOLOv1(pretrained_backbone=False).to(device)

OK. Thanks.


Update modal.js

Short description of what this resolves:
Modal not visible, if modal HTML template is wrapped in a script tag.
Changes proposed in this pull request:
Change the usage example by removing the script tag around the template HTML or at least - add a note, which states it needs to be removed since version 1.7.7

Ionic Version: 1.x / 2.x
1.7.7 +
Fixes: #
The usage example shows the HTML template wrapped into a script tag. It seems since version 1.7.7 this is not valid any more. If the modal HTML template is wrapped into a script tag, only the backdrop will be shown, but not the modal itself. I've spent few hours and found the correct answer here: http://stackoverflow.com/questions/32552896/ionic-modal-not-showing#answer-33292269

Hello! Thanks for submitting a PR to us, we really appreciate it! It seems as though your PR has a failing test. Would you be interested in fixing this test with some extra commits? Also, just for future reference the version number that you are giving for your ionic build is actually the version number for the ionic-cli. Your current ionic library version number should be 1.2.4. (:

I've commited an update to my branch, but now the build server can not find my commit. I'm not sure what I'm doing wrong.
"Could not find commit 2eb53304874f2b0a3e41c0c071ead3b85692ec97 in the repo"
but it is in my repo:
https://github.com/venelinod/ionic/tree/patch-1

Hey @venelinod ! I really appreciate your PR! But, I am a little confused by it. The script tag should not be causing an issue, and that SO post is actually incorrect. Would you mind opening an issue about the problem your having, and then if needed we can make a PR for it then? I think we can solve this by just looking at your code. Thanks!


cHostsFile : Dynamic DNS Resolver on node side

Currently DNS resolving statically when creating mof on DSC Server.
This is not useful at all, and need to be resolve at node side.
Even node hosts is defined, Resolve-DnsName -Name <TargetDns> -Server <DnsServer> can be handle specific dns.
Would be prefer add Write Key to mof and handle in psm1.

Outside Dns Server can't be resolved when customize hosts file....?

Required to add -DnsOnly switch. It can be resolve even you have modified hosts.


Fix a typo in the generated docs

This simply fixes a small typo I have noticed today while checking the docs popup in VS code.

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

Thanks a lot for the PR!


[embedded] Use -1 as the swift_once predicate indicator, to match Darwin expectations in the compiler

The attached testcase demonstrates the problem: On Darwin, the compiler expects that after after swift_once completes, the predicate value is "-1", and under LTO the optimizer ends up seeing through the swift_once implementation and turning the failed expectation into a trap.
This PR is still a stopgap in terms of swift_once becoming "real", but the -1 value matching Darwin resolves the immediate problem and is harmless on non-Darwin.

@swift-ci please test

@swift-ci please test

@swift-ci please test Linux platform

@swift-ci please test Linux platform

@swift-ci please test Linux platform


Added support for docker secrets for the VSTS_TOKEN env

The primary purpose is to provide the PAT as a secret to the instance, this way no hard coded PAT in docker-compose or other initialization scripts. Secondly, based on using the directory hierarchy for the tag of the image, the VS2015 and VS2017 dockerfiles are referencing the wrong generated image tag; it is missing the "servercore" folder.

@smokedlinq,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@smokedlinq, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Can this get merged by someone at MSFT? We would much prefer to use docker secrets for the PAT than an environment variable.

I'm not responsible for the Windows images, but the change looks ok to me, and I don't believe there are any published images that need updating, so I'll go ahead and accept the change.


Launch debug

Environment data
dotnet --info output:
VS Code version:
C# Extension version:
OmniSharp log
Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected  behavior
A clear and concise description of what you expected to happen.
Actual behavior
Additional context
Add any other context about the problem here.

Dupe of https://github.com/dotnet/vscode-csharp/issues/6562 this got created by mistake.


[ERROR] Not exporting the video

Hello,
I have being testing the script multiple times, but I always have this error in the console.
The when I click generate it goes into the exporting tab but doesn't export (puts the video in the queue but does not export it)
Using model -> [ small.en ]
MarkIn: 86400
Rendering Audio for Transcription...
Audio Rendering Complete!
Transcribing Audio...
FFmpeg failed to read "C:\Users\Leona\AppData\Roaming\Blackmagic Design\DaVinci Resolve\Support\Fusion\Scripts\Utility\audio.mov".

Do you mark the beginning and end of clip with I and O?

Different poster but same issue with FFmpeg. On MacOS. I have been marking the clip with I and O. Tried multiple times.

That happened to me when I hadn't pulled the Custom Title effect:

Yes, i did mark the End and start.


Subscriptions are only working sporadically

Current behavior
Sometimes subscriptions work, sometimes they don't. There is no precise pattern. Our code has not changed in weeks, nor have our dependencies.
Reproduction
Setup real-time subscriptions with Apollo (I'm using apollo-client 2.3.2), or use the Playground. Observe how sometimes nothing happens on a CREATE or a UPDATE  mutation.
Expected behavior?
When creating/updating a subscribed node, it should trigger a CREATE or UPDATE mutation, everytime in realtime.
Thanks

Thanks. Can you prepare a minimal setup for which you confirmed this to happen?

In the console create a type like this:

type Post @model {
  id: ID! @isUnique
  title: String!
}

Manually, in the console, create to Posts, with as title Title1 and Title2

Then in the playground type this:

subscription updatePost {
  Post(
    filter: {
      mutation_in: [UPDATED]
    }
  ) {
    mutation
    node {
      title
    }
  }
}

Now go and change Title1 to foo

Observe in the playground if anything happened. Sometimes it does, sometime it doesn't, you might have to change the title multiple times before the issue occurs. In that case you have to repeat step
4, but instead of foo another value like foo2and so forth.

@marktani @dpetrick @nikolasburk @schickling this issue is still happening, is there anything we can do to help?

I'm investigating the issue and I'll try a few fixes over the next hour.

thanks, let's hope it does not


updated for CakePHP 3.6

Please, release a version or create a branch compatible with CakePHP 3.6.

Needs phpunit 6.


Migrate to sensorSelectionInfos

Today I happened to know that https://github.com/mujin/mujinplanningclientpy/blob/master/python/mujinplanningclient/binpickingplanningclient.py#L354 IsRobotOccludingBody signature is different from current planningcommon realtimerobottask3.py's signature, where sensorSelectionInfos needs to be used, not cameraname.
There should be some others, so please check robottask and fix the planningclient signature accordingly.
/cc @ntohge

I wander - how comes we only noticed it now? Was IsRobotOccludingBody never used before?

@andriymujin maybe mujinsimulator is the only user

sent mujinsimulator side MR, can use that for total testing


Add transaction flow

Adds the transaction flow and transaction provider.

Also not super friendly when on smaller screens. Doesn't seem to take into account the height of my screen so I can't always see the title of the modal. And i can't scroll either


Fixes Issue #185

Have a look to https://github.com/Keyang/node-csvtojson/issues/185 for explanation

Same here, this solves issue #185

Thanks for working on this solution! 💯

All tests passing now 😃

Sorry for the delay..Thanks for this solution.
I will push 1.1.8 containing this to npm soon.


semantic highlighting

tmlanguage based highlighting isn't ideal. Since we have to write an LSP for intellisense (#15), we might as well include semantic highlighting support.

Highlighting via tmlanguage looks good enough -- fee966b7b8e656b31a946e7ea686857e96caee9d


Atharvak/food on fork node

Description
[TODO: describe, in-detail, what issue this PR addresses and how it addresses it. Link to relevant Github Issues.]
Testing procedure
[TODO: describe, in-detail, how you tested this. The procedure must be detailed enough for the reviewer(s) to recreate it.]
Before opening a pull request

[ ] Format your code using black formatter python3 -m black .
[ ] Run your code through pylint and address all warnings/errors. The only warnings that are acceptable to not address is TODOs that should be addressed in a future PR. From the top-level ada_feeding directory, run: pylint --recursive=y --rcfile=.pylintrc ..

Before Merging

[ ] Squash & Merge

Closing this PR to give way to #104.

Closing this PR to give way to #104.


Prerelease: 4.1.0-alpha.2: publish error AgoraRTCError INVALID_OPERATION: audience can not publish stream on flutter web

On  Prerelease: 4.1.0-alpha.2
When trying to join a stream with the role set as Audience the error
publish error AgoraRTCError INVALID_OPERATION: audience can not publish stream
Seems to be an issue only with the web sdk, works fine on Android and iOS.
await engine.setChannelProfile(ChannelProfile.LiveBroadcasting);
await engine.setClientRole(ClientRole.Audience);
engine.joinChannel(null, "channelName", null, 0);
Let me know if you need more info.

only broadcaster can publish stream, you can refer to the web submodule which is a wrapper by javascript.

Thank you for your answer.
Yes i understand that, I am setting the role as Audience and joining a channel with a broadcaster present.
The same implementation works for iOS and Android but not for web, could it be that when the client is being created it is setting
{ codec: "vp8", mode: "rtc" }
where as the audience needs the codec live in order to join...
Ive followed the example on that branch however it doesnt seem to work

Any progress on this?

pls follow the dev/flutter2 branch, the alpha version is out-date.

Pls try agora_rtc_engine: ^5.0.0


Pade floods the server with VCard set requests

Pade appears to indefinitely try to set a VCard. On servers were VCards cannot be modified, this is causing problems. Pade shouldn't continuously retry failed requests.
The error below is logged very two seconds.
2018.08.17 09:36:49 ERROR [Jetty-QTP-BOSH-1112]:
org.jivesoftware.openfire.handler.IQvCardHandler - VCard provider is read-only.
java.lang.UnsupportedOperationException: VCard provider is read-only.
        at org.jivesoftware.openfire.vcard.VCardManager.setVCard(VCardManager.java:158)
        at org.jivesoftware.openfire.handler.IQvCardHandler.handleIQ(IQvCardHandler.java:90)

Note that VCards can contain binary information (a picture/avatar) - the data usage involved might be considerable.

change available in version 0.4.12

I was unable to reproduce the continuous retry flood errors.

I have been able to reproduce the flood error. It happens when openfire is running on my production linux server. It does not happen on my Windows dev server. The offending code is the strophe re-connection plugin. Now I have to it

cc @jcbrand

So, what change applied the fix?

So, what change applied the fix?

https://github.com/igniterealtime/Pade/commit/e36a2341fe0153ba8a6fa9430eba82583328d228


Create .resx files runtime + add strings to it

I cant found any possibility to create translations via app code. Its important to have in localization some possibilities to add new languages and strings on runtime.
I mean with it:
Create .resx dynamically via C# code and add localized strings to this .resx file. It is important for each translated modular app.

Have a look to this https://msdn.microsoft.com/en-us/library/gg418542(v=vs.110).aspx

If you need support at runtime then RESX is probably not a good solution. RESX is a compile-time resource definition.
With the new localization features in ASP.NET / DNX, I recommend writing a custom resource provider that works with whatever storage medium you have - perhaps a TXT file, a PO file, a database, or something else.
Have a look at how the default implementation in https://github.com/aspnet/Localization/tree/dev/src/Microsoft.Framework.Localization implements what's in https://github.com/aspnet/Localization/tree/dev/src/Microsoft.Framework.Localization.Abstractions to see how the concepts work together.

@Eilon can you check the links, seems something wrong

Fixed links, thanks.


XINDEX tests are failing in Python 2

   All Routines? No => No
161:
161:
161:
161:   Routine:
161:
161:   Traceback (most recent call last):
161:
161:     File "C:/Users/betsy.mcphail/development/VistA-Build/Testing/XINDEX/SurgeryTest.py", line 47, in <module>
161:       VistA.write(prefix + '*')
161:     File "C:/Users/betsy.mcphail/development/VistA/Python/vista\OSEHRAHelper.py", line 199, in write
161:       self.connection.write(encode(command) + encode('\r'))
161:     File "C:\Python27\lib\telnetlib.py", line 280, in write
161:       if IAC in buffer:
161:     File "C:\Users\betsy.mcphail\AppData\Roaming\Python\Python27\site-packages\future\types\newstr.py", line 127, in __contains__
161:       newkey = newstr(key)
161:     File "C:\Users\betsy.mcphail\AppData\Roaming\Python\Python27\site-packages\future\types\newstr.py", line 102, in __new__
161:       return super(newstr, cls).__new__(cls, value)
161:
161:   UnicodeDecodeError: 'ascii' codec can't decode byte 0xff in position 0:
161:   ordinal not in range(128)

Not supporting Python2 anymore.


{Issue #47} Synchronous spinner

Description of new feature, or changes
Added the ability to manually render single spinner steps, via:

'manual_step' argument when creating Halo class instance
'.step()' method to trigger single frame to render
'test_manual_step' test

Usage
with Halo(spinner='dots', manual_step=True) as spinner:
    for text in ["foo","bar"]*5:
        spinner.text = 'Loading {}'.format(text)
        spinner.step()
        do_something(text)

or
spinner = Halo(spinner="dots", manual_step=True)
spinner.start()
for text in ["foo", "bar"]*5:
    spinner.text=text
    spinner.step()
    do_something(text)
spinner.stop()

Checklist

[x] Your branch is up-to-date with the base branch
[x] You've included at least one test if this is a new feature
[x] All tests are passing

Related Issues and Discussions
Fixes #47 (new feature)
People to notify
@steelman

Pull Request Test Coverage Report for Build 414

3 of 6 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.8%) to 89.26%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

halo/halo.py
3
6
50.0%

Totals

Change from base Build 409:
-1.8%

Covered Lines:
915

Relevant Lines:
1005

💛  - Coveralls

I'm struggling to identify why the build is now failing with the following error:
Traceback (most recent call last):
  File "/opt/python/2.7.15/lib/python2.7/threading.py", line 801, in __bootstrap_inner
    self.run()
  File "/opt/python/2.7.15/lib/python2.7/threading.py", line 754, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/home/travis/build/manrajgrover/halo/halo/halo.py", line 402, in render
    if self._run_spinner.is_set():
AttributeError: 'NoneType' object has no attribute 'is_set'

The _run_spinner threading.Event is created straight after the _stop_spinner Event. The _stop_spinner doesn't trigger an error on the previous line in render, so both must exist when the first render call occurs. This makes me think that the _run_spinner Event must be being removed after the Thread is started? However, I can't reproduce the error locally.


Uncaught PermissionError in pathlib.py line 1232

i have a weird bug... there's some random file in one of my directories that is supposed to be docker exec-ed inside a container... this file contains some file paths like /root/.... now for whatever reason, aider is looking at this line and trying to open it... getting a "PermissionError: [Errno 13] Permission denied: '/root/'" within aider...
the specific filepath aider is complaining about exists only in one file (it's a shell script), and only in one line: ln -s /root//...
this is the aider output:
Aider version: 0.54.12
Python version: 3.9.18
Platform: Linux-5.13.0-52-generic-x86_64-with-glibc2.31
Python implementation: CPython
Virtual environment: No
OS: Linux 5.13.0-52-generic (64bit)
Git version: git version 2.25.1
An uncaught exception occurred:
Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
  File "main.py", line 623, in main
    repo_map = coder.get_repo_map()
  File "base_coder.py", line 599, in get_repo_map
    repo_content = self.repo_map.get_repo_map(
  File "repomap.py", line 124, in get_repo_map
    files_listing = self.get_ranked_tags_map(
  File "repomap.py", line 466, in get_ranked_tags_map
    result = self.get_ranked_tags_map_uncached(
  File "repomap.py", line 497, in get_ranked_tags_map_uncached
    ranked_tags = self.get_ranked_tags(
  File "repomap.py", line 309, in get_ranked_tags
    if not Path(fname).is_file():
  File "pathlib.py", line 1456, in is_file
    return S_ISREG(self.stat().st_mode)
  File "pathlib.py", line 1232, in stat
    return self._accessor.stat(self)
PermissionError: [Errno 13] Permission denied: '/root/<path_to_file>'

Thanks for trying aider and filing this issue.
The fix is available in the main branch. You can get it by installing the latest version from github:
aider --install-main-branch

# or...

python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.


compiling with go1.17rc1 make a binary that fatal with SIGSEGV

What version of Go are you using (go version)?

$ go version
go version go1.17rc1 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/user/go-darwin-amd64-bootstrap"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOVCS=""
GOVERSION="go1.17rc1"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/7w/flrplbwd0jg9lfhwcwjxvc_80000gn/T/go-build3902858131=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.17rc1 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.17rc1
uname -v: Darwin Kernel Version 20.5.0: Sat May  8 05:10:33 PDT 2021; root:xnu-7195.121.3~9/RELEASE_X86_64
ProductName:	macOS
ProductVersion:	11.4
BuildVersion:	20F71
lldb --version: lldb-1<IP_ADDRESS>
Apple Swift version 5.4 (swiftlang-1<IP_ADDRESS> clang-1<IP_ADDRESS>)

What did you do?
I compiled a Go source code with go1.17rc1!
What did you expect to see?
I expected the resulted binary to run successfully just like when compiled with go1.16.5
What did you see instead?
running the binary:
fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0xb01dfacedebac1e pc=0x7fff2057acbe]

runtime stack:
runtime: unexpected return pc for runtime.sigpanic called from 0x7fff2057acbe
stack: frame={sp:0x7ffeefbff918, fp:0x7ffeefbff968} stack=[0x7ffeefb809b8,0x7ffeefbffa20)
0x00007ffeefbff818:  0x01007ffeefbff838  0x0000000000000004
0x00007ffeefbff828:  0x000000000000001f  0x00007fff2057acbe
0x00007ffeefbff838:  0x0b01dfacedebac1e  0x0000000000000001
0x00007ffeefbff848:  0x0000000004039e11 <runtime.throw+0x0000000000000071>  0x00007ffeefbff8e8
0x00007ffeefbff858:  0x0000000005232c17  0x00007ffeefbff8a0
0x00007ffeefbff868:  0x000000000403a0c8 <runtime.fatalthrow.func1+0x0000000000000048>  0x00000000061d74a0
0x00007ffeefbff878:  0x0000000000000001  0x0000000000000001
0x00007ffeefbff888:  0x00007ffeefbff8e8  0x0000000004039e11 <runtime.throw+0x0000000000000071>
0x00007ffeefbff898:  0x00000000061d74a0  0x00007ffeefbff8d8
0x00007ffeefbff8a8:  0x000000000403a050 <runtime.fatalthrow+0x0000000000000050>  0x00007ffeefbff8b8
0x00007ffeefbff8b8:  0x000000000403a080 <runtime.fatalthrow.func1+0x0000000000000000>  0x00000000061d74a0
0x00007ffeefbff8c8:  0x0000000004039e11 <runtime.throw+0x0000000000000071>  0x00007ffeefbff8e8
0x00007ffeefbff8d8:  0x00007ffeefbff908  0x0000000004039e11 <runtime.throw+0x0000000000000071>
0x00007ffeefbff8e8:  0x00007ffeefbff8f0  0x0000000004039e40 <runtime.throw.func1+0x0000000000000000>
0x00007ffeefbff8f8:  0x0000000005244567  0x000000000000002a
0x00007ffeefbff908:  0x00007ffeefbff958  0x0000000004050436 <runtime.sigpanic+0x0000000000000396>
0x00007ffeefbff918: <0x0000000005244567  0x0000000004042cde <runtime.checkTimers+0x000000000000005e>
...

Thanks for reporting, @urbanishimwe .
Could you please include instructions for reproducing the  problem, e.g. what application you built and how you ran it when you encountered this crash? Thanks.

@urbanishimwe is your code using the golang.org/x/sys repo? If so, could you try updating that repo? (See #46645) Thanks.

@cherrymui it seems to update golang.org/x/sys(v0.0.0-20201101102859-da207088b7d1 => v0.0.0-20210630005230-0f9fa26af87c) fixes the issue!

Thanks for confirming.

$go version
go version go1.17 darwin/amd64

I'm using go1.17 from https://golang.org/doc/install and encountering the exact same problem when running a test.

@MuyaoXiao try go get -u golang.org/x/sys

go get -u golang.org/x/sys

Thank you! I missed out the comment above, silly me!


Can't upload P12 key

I cannot upload P12 key file, the page throws error. The key is password protected. Can anyone help?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25508921-can-t-upload-p12-key?utm_campaign=plugin&utm_content=tracker%2F1282891&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1282891&utm_medium=issues&utm_source=github).

The file used should be json now, still getting this issue with v1.1?

Sorry, I gave up on that long time ago, can't tell if it is working now. Thanks!


Dns proxy use original source address and port

Make Cilium DNS proxy transparent by using the original source address (and port) in upstream connections. Since clients can issue multiple requests from the same port without waiting for the responses in between, we must support multiple requests in flight at the same time on the client side as well. To this end we share the DNS client between all requests that use the same upstream 5-tuple. In order to not add additional goroutines per request, concurrent requests on the same client take turns receiving responses and then pass responses between each other rather than simply discarding responses with different IDs to their own request.
DNS library has not been built for concurrent access on the client, so we use mutexes to serialize writes and reads, respectively, but allow concurrent writes while a read is blocking. Based on the source code analysis this seems safe given our current use of the DNS library.
Cilium DNS proxy now uses the original pod's address as the source address towards the DNS servers.

In local testing cilium monitor now shows the original pod source address and port is also used on the overlay:
<- endpoint 1929 flow 0x0 , identity 3664->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
Policy verdict log: flow 0x0 local EP ID 1929, remote ID 36693, proto 17, egress, action redirect, auth: disabled, match L3-L4, <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
-> proxy port 37757 flow 0x0 , identity 3664->unknown state new ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
<- endpoint 1929 flow 0x0 , identity 3664->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
-> proxy port 37757 flow 0x0 , identity 3664->unknown state established ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
<- proxy flow 0x5164dc81 , identity 3664->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
-> overlay flow 0x5164dc81 , identity 3664->36693 state unknown ifindex cilium_vxlan orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
<- proxy flow 0xa0e9948a , identity 3664->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
-> overlay flow 0xa0e9948a , identity 3664->36693 state unknown ifindex cilium_vxlan orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
<- overlay flow 0x52db30f3 , identity 36693->unknown state unknown ifindex cilium_vxlan orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
-> proxy flow 0x52db30f3 , identity 36693->3664 state reply ifindex lxc53a2958fd80a orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
<- overlay flow 0x52db30f3 , identity 36693->unknown state unknown ifindex cilium_vxlan orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
-> proxy flow 0x52db30f3 , identity 36693->3664 state reply ifindex lxc53a2958fd80a orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
<- proxy flow 0x0 , identity unknown->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
-> endpoint 1929 flow 0x0 , identity 36693->3664 state reply ifindex lxc53a2958fd80a orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
<- endpoint 1929 flow 0x0 , identity 3664->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
-> proxy port 37757 flow 0x0 , identity 3664->unknown state established ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
<- proxy flow 0xe1185fe7 , identity 3664->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
-> overlay flow 0xe1185fe7 , identity 3664->36693 state unknown ifindex cilium_vxlan orig-ip <IP_ADDRESS>: <IP_ADDRESS>:43094 -> <IP_ADDRESS>:53 udp
<- overlay flow 0x52db30f3 , identity 36693->unknown state unknown ifindex cilium_vxlan orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
-> proxy flow 0x52db30f3 , identity 36693->3664 state reply ifindex lxc53a2958fd80a orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
<- proxy flow 0x0 , identity unknown->unknown state unknown ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp
-> endpoint 1929 flow 0x0 , identity 36693->3664 state reply ifindex lxc53a2958fd80a orig-ip <IP_ADDRESS>: <IP_ADDRESS>:53 -> <IP_ADDRESS>:43094 udp

dnsproxy gets timeouts waiting for response from the dns server when curl is issuing both IPv4 (A) and IPv6 (AAAA) queries at the same time. Works flawlessly if limiting to IPv4 only (curl -4 ...).
Functionally significant change is that now there are two upstream connections (one for A and other for AAAA records) at the same time, with the same 5-tuple. So the connections are fighting for the same packets. What I think happens on the timeout is that the "missing" response was delivered to the same socket as the successful one, but the connection does not read multiple answers from the same socket.

Added commit to share a DNS client between requests that share the same upstream 5-tuple.

One of the commits needs to be moved to cilium/dns.

Moved the refactoring commit to cilium/dns, added TryLock support in sasha-s/go-deadlock/pull/30

/test

marked as blocked as we need to decide if we wait for https://github.com/sasha-s/go-deadlock/pull/30 to merge or use my fork with it like in the 1st commit via go mod replace right now.

Avoiding the use of original source address if it is known to be from the host networking namespace (host endpoint or localhost).

/test

Rebased to restart CI in hopes cloudflare is better today.

/test

Rebased & switched to cilium/go-deadlock fork.

/test

@bimmlerd asked about the TryLock critical section (but GH would not let me comment on it directly):

Did you think about which locks are held outside of the DNS proxy when control flow enters these critical sections? Specifically, are we allowed to hold the DNSProxy lock here, or does that create a potential wait cycle involving Endpoints & ipcache?

No, as the section does not call anywhere else in the Cilium agent. Technically, we could move the functionality of the new dnsproxy.Client into the vendored dns.Client, which would make this point clearer. At this stage, however, I figured this is better reviewed here in one place.

Addressed @bimmlerd's comments.

/test

/test

/test

/test

/test

The ugly TryLock code is now placed in https://github.com/cilium/dns/pull/7 that we import here. The core logic is literally the same as before, but it is now packaged differently and unit testing has been added.

Unless objected here I will remove the commit that adds TryLock support for go-deadlock, as we no longer need it in cilium repo.

@pippolo84 @nebril Asking for your reviews on https://github.com/cilium/dns/pull/7...

@bimmlerd Use of the proxy lock is not eliminated as part of pushing the code that was already functionally dependent on the internal detail of cilium/dns (like the setting of the connection deadlines) into cilium/dns itself. This also made it possible to add unit testing by reusing testing already in there.

Removed the go-deadlock change and squashed commits to simplify this PR, which will make this much easier to backport as well.

Marked as blocked, waiting for https://github.com/cilium/dns/pull/7 to merge.

/test

Shared client support using the Go design proposed by @joamaki merged to github.com/cilium/dns, so removing the blocked label :-) Thank you @pippolo84 and @nebril for timely and detailed reviews!
This PR should now be in it's final shape!

@bimmlerd To add some info on folks who have not seen the update in cilium/dns (which are visible here if you click open the changes to the vendored files!):

TryLock is nowhere to be seen, as Jussi volunteered his nice Go pattern that only needed minor tweaks.
"global" lock is now only ever held for map update operations, never for anything that makes system calls (like opening or closing the connection)

/test


Fix #1538: Improvement for show_default showing in case of a flag the name instead of True/False

Addressing #1538

Made a PR: https://github.com/pallets/click/pull/1617


Lint on CI

[ ] Config lint rules (including coding style)
[x] Abort build when has lint warnings
[x] Lint updated files on CI, create baseline
[x] Comment on PR according to lint result

How to use:

Send PR as normal
Check the comments added by lint-bot
Fix issue locally and click 'Resolve conversation' under each comment
Commit and Push again
Back to step 2

@seabornlee also we should add style check similar to web3j to enforce consistent project style eg:
if (a.equals(b))
{
    //do something
}
else
{
    //the other thing
}

rather than
if (a.equals(b)) {
    //do something
} else {
    //the other thing
}

@seabornlee also we should add style check similar to web3j to enforce consistent project style eg:
if (a.equals(b))
{
    //do something
}
else
{
    //the other thing
}

rather than
if (a.equals(b)) {
    //do something
} else {
    //the other thing
}

Sure thing.

Hi @seabornlee I created a 'AlphaWalletStyle.xml' which contains the coding style.
This is a great mod, looking forward to the merge.

Awesome!


Add support for branches to config-repo

Hey thanks for this! Can you write a test a test to go with the code change?

I added a unit test to ensure the correct behavior.

Releasing as 0.6.4, thanks for helping out!


links to ckpts not working

Hi @MukundVarmaT, @peihaowang, thanks for sharing the ckpts. However the links to those ckpts seem broken now. Can you fix them? Thanks!

Hello @MukundVarmaT. This is superb work! I loved the idea! Kindly fix the broken links!

@bmmi7 @shounaknaik
apologies for the delay, I have fixed the broken links. Please let me know if anything does not work.


test_system.vim not passing with X11 forwarding and PuTTY

Note: this whole scenario requires vim to be configured with --with-x=yes.
When using X11Forwarding, authentication happens using the ~/.Xauthority file. In setup.vim, $HOME gets set to a non-existent path, thus breaking authentication. This, in turn, produces an error message when executing vim with some SSH clients (notably PuTTY and its derivative MobaXterm) along the lines of MobaXterm X11 proxy: Unsupported authorisation protocol on stderr.
This error message gets prepended to the desired $? in test_system.vim#L56, so a[0] contains 'M', not '0'. This leads to the actual error message that gets produced when running make test:
From test_system.vim:
Found errors in Test_system_exmode():
function RunTheTest[24]..Test_system_exmode line 7: Expected '0' but got 'M'

Since this isn't an error in how vim was compiled, but rather a result $HOME being unset, it shouldn't break testing.
It can be temporarily fixed by changing the value in setup.vim to something like /tmp/vimhome, creating that directory, and copying the ~/.Xauthority file in there. A better solution would probably be to either set $HOME temporarily, or to have the test simply ignore stderr, for example by changing test_system.vim#L52:
let cmd = ' -es -u NONE -c "source Xscript" +q 2>/dev/null; echo $?'

I've tracked the message down to this line in the PuTTY sources, after which it seems to be passed back to the remote host.
For completeness, here's the steps to reproduce the test failure:

Connect to a machine using PuTTY/MobaXterm with X11 forwarding enabled on both sides
Run make test on a vim instance that's compiled with +X11
Test_system_exmode() fails

Xiretza wrote:

Note: this whole scenario requires vim to be configured with --with-x=yes.
When using X11Forwarding, authentication happens using the ~/.Xauthority file. In setup.vim, $HOME gets set to a non-existent path, thus breaking authentication. This, in turn, produces an error message when executing vim with some SSH clients (notably PuTTY and its derivative MobaXterm) along the lines of MobaXterm X11 proxy: Unsupported authorisation protocol on stderr.
This error message gets prepended to the desired $? in test_system.vim#L56, so a[0] contains 'M', not '0'. This leads to the actual error message that gets produced when running make test:
>From test_system.vim:
Found errors in Test_system_exmode():
function RunTheTest[24]..Test_system_exmode line 7: Expected '0' but got 'M'

Since this isn't an error in how vim was compiled, but rather a result $HOME being unset, it shouldn't break testing.
It can be temporarily fixed by changing the value in setup.vim to something like /tmp/vimhome, creating that directory, and copying the ~/.Xauthority file in there. A better solution would probably be to either set $HOME temporarily, or to have the test simply ignore stderr, for example by changing test_system.vim#L52:
let cmd = ' -es -u NONE -c "source Xscript" +q 2>/dev/null; echo $?'

I've tracked the message down to this line in the PuTTY sources, after which it seems to be passed back to the remote host.
For completeness, here's the steps to reproduce the test failure:

Connect to a machine using PuTTY/MobaXterm with X11 forwarding enabled on both sides
Run make test on a vim instance that's compiled with +X11
Test_system_exmode() fails

First of all: it's possible to reset $DISPLAY to avoid Vim connecting to
the X server.
That extra error message gets in the way of the actual test output,
but it is otherwise harmless?  We could change "echo $?" into something
like "echo "result = $?"", and then search the output for "result = {nr}".
Would that work?
--
From "know your smileys":
<|-) Chinese
<|-( Chinese and doesn't like these kind of jokes
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

We could change "echo $?" into something like "echo "result = $?"", and then search the output for "result = {nr}".

That would of course work. An even better solution might be to set $XAUTHORITY though, according to man xauth it can be used to point to the .Xauthority file directly. This would avoid the error altogether.

We could change "echo $?" into something like "echo "result = $?"",
and then search the output for "result = {nr}".

That would of course work. An even better solution might be to set
$XAUTHORITY though, according to man
xauth it can be used to point to
the .Xauthority file directly. This would avoid the error
altogether.

OK then, can you try this in testdir/setup.vim:
" Only when the +eval feature is present.
if 1
" Make sure the .Xauthority file can be found after changing $HOME.
if $XAUTHORITY == ''
let $XAUTHORITY = $HOME . '/.Xauthority'
endif
" Make sure $HOME does not get read or written.
let $HOME = '/does/not/exist'
endif
--
hundred-and-one symptoms of being an internet addict:
98. The Alta Vista administrators ask you what sites are missing
in their index files.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Sorry for the wait. That solution works great, thanks!


Fix panic and tidy up exporter logic

While testing #71 I hit a bug when the API token has expired.
This causes a panic which makes the exporter crash and need a restart. It should just log the error.
2023/07/12 18:18:39 failed to get carbon intensity &fmt.wrapError{msg:"401 Unauthorized - {\"error\":\"Invalid auth-token\"}: received non-200 status", err:(*errors.errorString)(0x14000180ce0)}
panic: runtime error: index out of range [0] with length 0

goroutine 34 [running]:
github.com/thegreenwebfoundation/grid-intensity-go/cmd.(*Exporter).Collect(0x14000078280, 0x140000b6758?)
	/Users/ross/src/rossf7/grid-intensity-go/cmd/exporter.go:191 +0x2a8
github.com/prometheus/client_golang/prometheus.(*Registry).Gather.func1()
	/Users/ross/.gvm/pkgsets/go1.20.5/global/pkg/mod/github.com/prometheus/client_golang@v1.15.1/prometheus/registry.go:455 +0xd8
created by github.com/prometheus/client_golang/prometheus.(*Registry).Gather
	/Users/ross/.gvm/pkgsets/go1.20.5/global/pkg/mod/github.com/prometheus/client_golang@v1.15.1/prometheus/registry.go:547 +0x9f4
exit status 2

@mrchrisadams Can you also take a look at this PR when you have time. Thx!

Hi Ross! This looks great - thanks for the PR. I'm merging in.


UP-4799: Add sign in, sign out, and new user icons

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines

Description of change

https://issues.jasig.org/browse/UP-4799
Adds icons to give some additional visual interest to the current plain text "login" and "logout"

Coverage remained the same at 23.044% when pulling 2b4dc00007fb124458944a7b58c138f0b47f547c on ChristianMurphy:UP-4799 into 4140d3240ce2f3890e85e2e7170f3e8959c2d212 on Jasig:master.

Looks good.


Support for .Net Standard

Hello,
Are there any plans to support .Net Standard?
I tried to change the project to Standard 2.0 and added the MySql.Data 8.0.10-rc reference from Nuget without any compiler errors, but havent tested it yet.
So at first glance, there seems to be no code changes need to be made.

Yes sure, the coding used in MySqlBackup.NET is common C# methods and basic syntax. It is backward compatible supports .NET 2.0 as well as forward compatible until .NET Standard 2.0 and even future version.
I'll release the binary of .Net Standard at later time.

I faced a problem converting the code into .Net Standard 1.3
System.Timers does not exist in .NET Standard
some search in Google tells that System.Timers has to be replaced by System.Threading.Timers
and the coding is slightly different.
I will release .NET Standard version at a later time if I have solved the problem.

Can you just use .NetStandard2.0?

Sure

Hey adriancs2,
do you have any idea, when you will release the .NetStandard-Lib on Nuget?
Thanks kl1mm

excuse the renewed disturbing but are you still planning the .NetStandard implementation and the release on nuget?

i finally got some time to help you look into this.
i managed to convert to project to .Net Standard 2.0 and it compiled without any warnings.
Are you sure you have to converted the projects to 2.0?
If you have problems i can see if i can find time to do a PR with a raw upgrade of the two projects

Yes, you're welcome

dear @adriancs2
any netstandard release on nuget ?

@StevenLiSiwei Hi bro, the version of .NET Standard 2.0 is published at Nuget a few minutes ago at the time of writing this. You can may check it out.

@StevenLiSiwei You're welcome

Hi sir @adriancs2 i've been a fan of your work and really wanted to use the MySqlBackup.NET you did . but i am having trouble

An unhandled exception of type 'System.IO.FileLoadException' occurred in System.Windows.Forms.dll Additional information: Could not load file or assembly 'MySqlBackup, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' or one of its dependencies. A strongly-named assembly is required. (Exception from HRESULT: 0x80131044)

At first everything is working fine until i tried to "Publish" my work and that's where the error started i can not debug after making a publish -- i tried to remove add again and try the files you have in the folder but no luck -- how can i make it work again, i am not very pro when it comes to programming and i am just still exploring thank you :)
Details :
VB.NET
Visual Studio Pro 2015
Net Framework : 4.5
MySQL Connector : v6.9.8

@iamjohnpalencia go to properties of 'MySqlBackup' DLL and set Copy Local = True and then try publish again. Also make sure that DLL is including or not while publishing your project.

Yes, @iamjohnpalencia , you try out @sridhargoshika 's suggestion first and see if the problem get solved or not :)

Hello good day :) thank you for the response, unfortunately i did that already but the issue has not been solved. I think the issue is the MySql.data i think it does not have the terms or keyword you have used in Mysqlbackup.net
Scenario : When i use the MySql.data included in sir @adriancs2 's package -- it runs but it can't be publish saying MySql.data is not official or something like that. but the MySqlBackup.net is fine (not pointing as the cause of error) but when i use the official MySql.data from Oracle its pointing to the MySqlbackup.net so i think the cause of the problem is the Mysql.data
I just want to ask - is the Mysql.data included in the zip file of sir @adriancs2 is somewhat modified or not :) thanks a lot :) and i really admire pro_programmers like you guys <3

I did not modify MySQL.Data.DLL.
Your problem might be one of the causes below:

Different target build platform of x64 bit or x86 bit.
Your application build setup requires referenced DLL must be signed too (strong name). All the releases of MySqlBackup.NET are not signed. Therefore, I think you can try to recompile MySqlBackup.NET by signing it. You can try download the source code of MySql.Data.DLL which contains a Strong Name Key File (ConnectorNetPublicKey.snk) that you can used to sign MySqlBackup.NET.


refactor(EKS): node pool deletion everywhere should be consolidated to use the EKSNodePoolAPI.DeleteNodePool approach

Is your feature request related to a problem? Please describe.

Currently there are 3 implementations of deleting a node pool.
NodePoolAPI.DeleteNodePool:

DELETE {{url}}/api/v1/orgs/:orgId/clusters/:clusterId/nodepools/:nodePoolName
cluster.Service
clusteradapter.NodePoolManager
clusterworkflow.DeleteNodePoolWorkflow
clusterworkflow.DeleteNodePoolActivity
eksproviderworkflow.DeleteStackActivity

ClusterAPI.DeleteCluster:

DELETE {{url}}/api/v1/orgs/:orgId/clusters/:clusterId
cluster.Service
eksproviderdriver.EKSClusterDeleter
srccluster.EKSDeleteClusterWorkflow
eksproviderworkflow.DeleteInfrastructureWorkflow
eksproviderworkflow.DeleteStackActivity

LegacyClusterAPI.UpdateCluster:

PUT {{url}}/api/v1/orgs/:orgId/clusters/:clusterId
srcapi.ClusterAPI
eksproviderdriver.EKSClusterUpdater
srccluster.EKSUpdateClusterWorkflow
eksproviderworkflow.DeleteStackActivity

Describe the solution you'd like to see

We would like to consolidate all 3 of those to use the same logic, namely the EKSNodePoolAPI.DeleteNodePool implementation.
Describe alternatives you've considered

Additional context

Blocked by #3223
Blocks #3185

With #3232 the first one has been transitioned to the EKS distribution specific node pool API which is the target to use.


Improve landing page content

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/219
Originally created by @bencodezen on 2018-12-14 15:21:46

Per @jschatz1's comments from meltano.com#9:
Getting started on the front page section that would give them an intro to Meltano. Which could link them to the install section. Then also a installation link below it that would also link them to installation.
They want to know.

What is this thing. It should be clear from the home page. 99% of the people coming to the page will have absolutely no idea and they need to get an idea quickly.

Now that we like it how do I install it.

View 11 previous comments from the original issue on GitLab


[GH-506] Correctly link or embed attachments

Summary
Recognize embeddable attachments and just link the rest of the attachments.
Ticket Link
https://github.com/mattermost/mattermost-plugin-jira/issues/506

CI failing due to known error being fixed at https://github.com/mattermost/mattermost-plugin-jira/pull/513

@jfrerich ping

@larkox This PR does solve the problem. Text files as well as various other file extensions I've tried will now embed properly as a link.
One small regression I have found is that video files such as .mp4 are now treated the same way as well. When viewing the comments in the "old View", .mp4 will show an embedded link rather than a thumbnail the way images do. This is a regression from master where .mp4 showed a preview. This does not seem like a significant usability issue and only "Old View" is affected.
Not sure how much control you have over this but, can videos files that play in-line in a browser like .mp4 continue to be treated the same way as images?

Codecov Report

Merging #517 into master will decrease coverage by 0.20%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #517      +/-   ##
==========================================
- Coverage   34.14%   33.93%   -0.21%
==========================================
  Files          37       37
  Lines        4581     4603      +22
==========================================
- Hits         1564     1562       -2
- Misses       2868     2892      +24
  Partials      149      149

Impacted Files
Coverage Δ

server/client.go
8.79% <0.00%> (ø)

server/issue.go
16.37% <0.00%> (-0.08%)
:arrow_down:

server/utils.go
22.12% <0.00%> (-4.48%)
:arrow_down:

server/subscribe.go
65.55% <0.00%> (-0.40%)
:arrow_down:

Continue to review full report at Codecov.

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

@DHaussermann I added the mp4 MIME type so it considers mp4 files as embeddable. It should be working now.


customize output

hi
how can i customize output?
say something like a json response instead of full form code.

I am working on that currently will post soon...there is library available to convert html to json ....will be using that to convert

Wow
i never knew such library exists, can you tell me library name?

You can look into below..I am not getting time to work on this ....you can contribute if you are able to do this...
https://github.com/Jxck/html2json

Ok... thanks for link, i will check into it.


remove nowrap for perex

should fix second comment https://gitlab.skypicker.com/frontend/frontend/merge_requests/5849#note_784217
I decided to remove all ... as we had planned since the start, we were using it only because perexes were too long, but now content guys looks like they fixed it to the correct amount of characters, so we can remove ... and use two lines as it is in the zeplin desings.  LiveURL: https://smartfaq-fix-perex-faqs.surge.sh

:tada: This PR is included in version 1.8.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


auto shutdown

Description of the new feature / enhancement
a scheduled auto shutdown option with the PC's date and time
Scenario when this would be used?
when you are updating a software during the night or you simply have to go out and at a certain time you would like it to turn off because you expect the installation to be finished
Supporting information
No response

/dup #21605


Add custom extension to exclude generated marshallers/unmarshallers

This is required to support a new feature in azblob where blob names are base-64 encoded.

Also fix XML handling of ,chardata

Fixed in preview.49 of the code generator.


lightline.vim: Could not load colorscheme

I have added the colorscheme file in the wrong place and since that it stopped recognising all colorschemes. I have tried to reinstall the plugin but unfortunately it did not work. I have noticed the colorscheme manual too late, sorry. Any ideas how to fix?
I am using Plugged for plugin manager.
And my plugin file looks like below.

Thank you

I cannot understand what you mean, please describe

can you see the default colorsheme of lightline?
can you change the lightline colorsheme to another preset (ex. wombat)?
where exactly do you put your colorsheme file and how do you load it with vim-plug?


Issue/56

fixed the problem with the double 'backspace' on the location search bar

This seems to break the functionallity.
After I enter a location and start to search no location is selected.

=>


Remove threesdk and update docker file

Description

[x] Remove threesdk module (and usershell)
[ ] Update docker file

Checklist

[ ] Pre-commit hook is installed to do formatting checks before committing code...etc
[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings

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

Project coverage is 78.37%. Comparing base (cb51118) to head (0e68212).
Report is 22 commits behind head on development.

Additional details and impacted files
@@               Coverage Diff               @@
##           development     #611      +/-   ##
===============================================
+ Coverage        72.82%   78.37%   +5.55%
===============================================
  Files               78       73       -5
  Lines             3499     3251     -248
===============================================
  Hits              2548     2548
+ Misses             951      703     -248

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


replaced APIs that to be deprecated with the new ones

In Firebase V6, signInAndRetrieveDataWithCredential will be deprecated in favor of signInWithCredential. Replace them in the sample apps.

@jhuleatt this should not be merged until I/O (when Firebase SDK v6 comes out)


Multi-selection widget: Selection is hard to see

🐛 Bug
When using a multi-selection widget in napari, its selection is hard to see because the blue is hardly different from the grey:

To Reproduce
@magic_factory(quality=dict(widget_type='Select', choices=Quality))
def surface_quality_table(surface: SurfaceData, quality, napari_viewer:"napari.Viewer"):

See also:
https://forum.image.sc/t/generate-multi-selection-widget-from-magicgui-for-napari-plugins/72631/6

Expected behavior
Would be cool if we could change the stylesheet of the dark theme so that the selection color is more different from the background color.
Environment

Please copy and paste the information at napari info option in help menubar here:

napari: 0.4.17rc4
Platform: Windows-10-10.0.19044-SP0
Python: 3.9.13 | packaged by conda-forge | (main, May 27 2022, 16:51:29) [MSC v.1929 64 bit (AMD64)]
Qt: 5.15.2
PyQt5: 5.15.4
NumPy: 1.22.4
SciPy: 1.8.1
Dask: 2022.7.1
VisPy: 0.11.0
magicgui: 0.5.1
superqt: 0.3.3
in-n-out: 0.1.5
app-model: 0.0.9
npe2: 0.6.1

OpenGL:
- GL version: 4.6.0 NVIDIA 512.78
- MAX_TEXTURE_SIZE: 32768

Screens:
- screen 1: resolution 1920x1200, scale 1.0

Any other relevant information:

Additional context

@haesleinhuepf Could you post a longer code that will work out of the box? Or point a plugin from you bring it?

I think the code @brisvag can be used as a reproducer with small tweak.
In the napari console this should work:
from magicgui import magicgui
from enum import Enum, auto

class Choices(Enum):
    a = auto()
    b = auto()
    c = auto()
    d = auto()

@magicgui(x=dict(widget_type='Select', choices=Choices))
def test(x):
    print(x)

viewer.window.add_dock_widget(test)

In my case the highlight color is grey not blue, probably an OS thing:

I also cannot reproduce. So we need more input from @haesleinhuepf

this is how it looks on my windows with #5202


Use app/form/fields.json API to get fields of an app

Instead of form.json, use app/form/fields.json to get more
fields from the app.
More fields include Record Number in Group field etc.
This pull request does not include tests for this change,
because I have no kintone development environment on localhost.
If someone who has the kintone development environment on localhost provides an output of
app/form/fields.json of the environment, I will modify app_test.go and app_test_json.go with the output.

@ryoon
Thank you for your pull request!
Happy to inform you that it will be available in the next version.


Adds FFaker::Date.birthday

This adds FFaker::Date.birthday
I ran this test 10_000 times to verify correctness:
assert_random_between(today - 43*365..today - 42*365) { @tester.birthday(min_age: 42, max_age:42) }

Fixes #540

please add :birthday to assert_methods_are_deterministic

Are you ok with me adding TimeCop?

@marocchino thanks for all your help with this PR. I appreciate it.

Are you ok with me adding TimeCop?

Well, Personally, I don't like it. Rails treats it as a legacy that needs to be removed, so I often do the work of removing it.
However, other maintainers may have different thoughts.

OK, I believe this PR is ready for merging.

Are you ok with me adding TimeCop?

What for? For tests? I'm OK if it'd help.

Well, Personally, I don't like it.

Why?

Rails treats it as a legacy that needs to be removed, so I often do the work of removing it.

TimeCop doesn't belong to Rails, what the deal with Rails? It's like "FFaker treats RSpec as a legacy". TimeCop is under active development, as I see, and not even v1 for now:

@marocchino anything blocking the merging of this PR?

I was expecting another maintainer's approval, but it's okay as is.


Option to delete the second file when performing fake duplex

Pushing into Future as the duplex work isn't currently planned for Next.


Design a clean shutdown for connectTo

One of the problems other teams have is that when they cancel a thread that runs a multiplexer it shows mux errors like this one:
[piotr-la:cardano.node.LocalErrorPolicy:Error:44] [2020-07-03 12:35:17.35 UTC] [String "ErrorPolicyUnhandledApplicationException (MuxError {errorType = MuxBearerClosed, errorMsg = \"<socket: 29> closed when reading data, waiting on next header True\", errorStack = []})",String "ErrorPolicyTrace",String "LocalAddress {getFilePath = \"\"}"]

This pollutes test output.  This could be done either by classifying errors better or expose a clean shutdown method.

See slack discussion


Read highlighted content only using ocr-reader

Hi Team,
Is it possible to read only a highlighted  or underlined text?

Try consulting StackOverflow for questions like these. The issue tracker is for reporting bugs.


OpenShift Console URL is incorrect after Launcher deploys an app to OCP4.x

Description
Once the application is deployed by the Launcher, the OpenShift Console link takes to an error page.
Note this location may be slightly different with 4.2 developer console.
What should happen?
It should take to overview page with URL like
https://CONSOLE_URL/overview/ns/PROJECT_NAME
Note this location may be slightly different with 4.2 developer console.
What happens instead?
It goes to https://CONSOLE_URL/console/project/PROJECT_NAME/overview/
and returns Error 404 Not found
Do you have any idea why?
Update from 3.x to 4.x
Error log
If you have an error log, please paste it here.

fixed in the operator https://github.com/fabric8-launcher/launcher-operator/pull/52 you can now add oauth api in the "cr"


refactor(log): unify log format with mx-apps

MX-1423
Minor change on log format to match requirement in ticket.

Huh. Seems we don't have a test to cover the actual log format in our test suite 😅
Can you write one here? https://github.com/rewardops/rewardops-sdk-node/blob/master/test/utils/logger.test.js#L231-L296

Good call! Will do.


Update or Delete blog post capability in dashboard

User Story
AS A developer who writes about tech,
I WANT a CMS-style blog site,
SO THAT I can publish articles, blog posts, and my thoughts and opinions.

Acceptance Criteria
GIVEN a CMS-style blog site,
WHEN I click on one of my existing posts in the dashboard,
THEN I am able to delete or update my post and taken back to an updated dashboard.

Complete. See PR https://github.com/nestibry/umnbc-mod14-mvc-tech-blog/pull/24


CompType::Typeless is not handled when remotely replaying DX11 capture with Linux client

Description
I'm running QRenderDoc on Linux machine and want to replay a DX11 capture on remote host (doesn't matter if server is running on Windows or under Wine). In this case QRenderDoc uses OpenGL to render output.
In MakeGLFormat or one level higher in GLReplay::CreateProxyTexture fmt.compType is never tested for CompType::Typeless so DX11 typeless textures are displayed incorrectly.
Repro steps

On Linux connect to remote host which support DX11
Open D3DTemplate.zip
Select "DS" output in the only DrawIndexed
The texture is not displayed and all values of pixels are incorrect

Environment

RenderDoc build: 2019-09-08 nightly build.
Operating System: Linux + Windows

This is a tricky one, since GL has no native concept of typeless textures so we need to decide a type at create time, but we don't have it until it's later rendered. It's kind of a problem of mapping a more capable to a less capable API.
GL has texture views in an extension but they're ugly and awkward to set up, as well as needing a fallback path if they're not supported. At the moment even if the proxy texture was created somehow the type hint used at render time is ignored on GL because there's no equivalent for native GL display.
Even if it's ugly that's probably the only feasible fix, but this does make it more appealing to implement proxy support for Vulkan so that we can use that wherever possible instead of GL for local proxying on linux.

but this does make it more appealing to implement proxy support for Vulkan so that we can use that wherever possible instead of GL for local proxying on linux.

This would be nice. Is this something you plan to do in near future? Or it's something that would be nice to have some day.
In case it's not planned I may try to implement it. Am I right that to implement proxying next methods should be implemented:
  virtual ResourceId CreateProxyTexture(const TextureDescription &templateTex) = 0;
  virtual void SetProxyTextureData(ResourceId texid, uint32_t arrayIdx, uint32_t mip, byte *data,
                                   size_t dataSize) = 0;
  virtual bool IsTextureSupported(const ResourceFormat &format) = 0;

  virtual ResourceId CreateProxyBuffer(const BufferDescription &templateBuf) = 0;
  virtual void SetProxyBufferData(ResourceId bufid, byte *data, size_t dataSize) = 0;

Probably not in the near future, it's just a nice-to-have indeed. Especially it doesn't actually save any time for fixing this bug, because adding Vulkan proxy support doesn't help for people who don't have Vulkan.
Yes exactly, if you look in vk_replay.cpp there are a few functions that are not implemented yet. VulkanReplay::NeedRemapForFetch also needs an implementation in addition to what you have listed there.

This took longer than I thought to address but I've now done the necessary refactoring to allow for this display. I've also added tests that should hopefully ensure that in future the remote proxying doesn't break between different APIs, even if vulkan support is added in future.


Add a nexd socket check for an existing process running

Add a check to prevent more than one nexd being initialized
Added checks for OS on nexctl for unsupported OSs

Leveraging Russell's excellent socket ctl to check if an existing nexd service is running on Linux. @russellb I ended up duplicating the callNexd function in nexodus instead of moving local.go out of nexctl to preserve Version numbers set at build. Ty.
### Linux:

# nexctl nexd ... commands
$ sudo ./nexctl nexd version
nexctl version: dev
nexd version: dev

$ sudo ./nexctl nexd status
Status: Running

# Starting on Linux with an existing nexd process running

$ sudo ./nexd-dev-linux-amd64 --username kitteh1 --password floofykittens  https://try.nexodus.local
{"level":"fatal","ts":1680137512.865208,"caller":"nexd/main.go:205","msg":"existing nexd service already running"}

### Non-Linux Hosts:

$ sudo ./nexctl nexd version
nexctl version: dev
nexd ctl interface not currently supported on darwin

$ sudo ./nexctl nexd status
nexd ctl interface not currently supported on darwin


Add integration tests to pubsub

This adds integration tests for PubSub and makes Travis more tolerant if tests are taking long.

Coverage remained the same at 86.131% when pulling 4054e3d3e9f4a2b8b7fcc514f8c50c029d19feea on mziccard:pubsub-its into 42f0fe8eed6be53a7d17e7675547136d6f9b5b0a on GoogleCloudPlatform:pubsub-alpha.

/cc @lesv

LGTM - Cool, Thanks.


Add CLI to the Repository Management Tools

The repository management tools may either be imported into a Python module or used with the Python interpreter in interactive mode.
Build a CLI on top of the available functions/methods for users that prefer to also use the command-line.  Consider the use cases for the CLI: for simpler repositories?  Support all of the capabilities of the current tools?  The Go implementation of TUF provides a good example of how the CLI should behave and what to support.

Closing due to uncertain future of TUF cli. Re-visit with #811.


Make the :machine attribute required for machine_execute and machine_file lwrps.

resolves #449

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Rebased and merged in 79e5279a42494983c829a8ab6df4c23bfc7a8b90. Thanks @poliva83 !
In chef-provisioning-aws we intentionally make most attributes not required so that you can reference a machine later in a recipe without having to continue specifying all the attributes you used to create it. But in the case of machine_execute and machine_file I think it makes sense to make them required.
We'll need to release this as at least a minor version bump since this is technically a breaking change.

Thanks for merging @tyler-ball


font does not display numbers

On OSX, chrome :
0123456789 ABCDE !@#$%^&*()_+{}[]
displays as :

This is a problem for displaying device keys in Settings > Security and Privacy. Copy-pasting still works properly though.

No problems for me on Manjaro (FF and Chromium). Might be a font bug, can you try with another font maybe ?

Not present in 2.3.7

Present in 2.4.0 on OSX on firefox as well

Present on windows 10 pro, on Chrome, Firefox and Edge

I think the default font was changed in Element, since it had problems, but maybe this one had problems at this point of time

We have this line in theme-tchap.css (copied from element's theme-light.css and theme-dark.css) :
pre, code {
    font-family: Inconsolata, Twemoji, 'Apple Color Emoji', 'Segoe UI Emoji', 'Noto Color Emoji', Courier, monospace;
    font-size: 100% !important;
}

It looks like inconsolata is not displaying numbers, so Twemoji is displaying them, and Twemoji displays them as squares.
Not clear to me why inconsolata is not displaying the numbers.

On the same computer, this bug is :

Present on 2.4.0 on Firefox 91.3.0esr (64 bits)
Not present on 2.3.7 on Edge 101.0.1210.47 (64 bits)

This bug seems to come with version 2.4.0

I have made the fix in a PR.


Protect the main branch

If we do that, the Action can not update tool and tutorial lists
Possible solution

Protect main and have PR with changes instead of pushing directly
But someone needs to merge the PR every week

Allow only the CI to push directly
Not sure how to do that

Store the results in a different repo (as the research-software-ecosystem does)

I like @nsoranzo's idea!

I agree that would work, but is it not a somewhat strange way of using git ? :)

I did re-protect the main branch again, btw., I only unprotected to show in the demo how the CI is working.

It is the way GitHub Pages use the gh-pages branch, see e.g. https://docs.github.com/en/pages/getting-started-with-github-pages/configuring-a-publishing-source-for-your-github-pages-site
This is how I do it in my repo: https://github.com/TGAC/rfparser/blob/main/.github/workflows/deploy.yaml

@nsoranzo it seems that storing the results in a different branch does bring some new issues: https://github.com/galaxyproject/galaxy_codex/actions/runs/10338185162/job/28615936668
Maybe you have an idea how to handle this ?

I think that should do the trick, https://github.com/galaxyproject/galaxy_codex/pull/163

Allow only the CI to push directly
Not sure how to do that

Should be possible using a deploy key (you can use exceptions from protection for deploy keys)

Is fixed now, as the CI makes a PR for the updated stats.


Add support for Alpine morph and custom morphers

To replace morphdom wih Alpine morph:
Add to your base template:
<script defer src="https://unpkg.com/<EMAIL_ADDRESS><script defer<EMAIL_ADDRESS>
Add to your settings:
UNICORN = {
    "MORPHER": "alpine",
}

Ref: https://alpinejs.dev/plugins/morph

I created a demo project and recorded a video to show how it works:

Video
Sample project. Branch experiment/alpine-morph

@adamghill, when you have time, could you please take a look at 5f240f5f02e4fdd6cca65d078f1283ba2321b535 where I document the future state of custom morphers and let me know if you have any feedback? Thanks!

@adamghill, everything except tests is ready for the review.
If you're OK with my work so far, can you please pick it up from here. Alternatively, can you please help me understand which tests should I add and modify to make it work? Thanks!

@adamghill, when you have time, could you please take a look at 5f240f5

This looks like a good start! I'll probably tweak it a bit when I get a change, but thanks for writing up all of this.
I rebased main into this branch and have started working on the tests. I will continue to fix up the JavaScript tests and make sure things look good to get this into a mergeable state for the next release.

Thanks a lot for picking up where I left off, @adamghill! I'm looking forward to having these changes merged. 🙌

Here is the summary of the last commits that I made yesterday and today.
I was almost ready to use "alpine-morph" in production, but then I found that alpine.min.js, which is used when DEBUG is False, is not up-to-date with the source code.
Additionally, the getMorpher() function needs to be moved to a Unicorn namespace so that it can be accessed from the minified version.
Finally, I discovered that Alpine and Alpine morph will be loaded after our script (as they are loaded with

@imankulov Thanks for all your work for this PR. This got merged into main and should be available in 0.56.0. 🎉

@imankulov I ended up releasing 0.56.1 with another fix for the Alpine morpher. Can you try it out and let me know if you run into any problems?

@adamghill, I have tested version 0.56.1 with my codebase that uses the Alpine morpher, and everything appears to be functioning correctly 🎉
Thank you very much for completing and merging the PR.


Update from plotly 1.33.1 to 1.40.0 to fix #309

Easiest bug fix in the history of this project.

Looks great, thanks @AmandaBirmingham !


dummy PR to check the CI

Signed-off-by: Nitin Goyal<EMAIL_ADDRESS>

/test lvm-operator-bundle-e2e-aws

Once the aws-disk PR gets merged, the CI will pass then.


Idle CPU usage

Describe the bug
When spotify_player is not playing (only tested with the integrated streaming on), it still uses ~7% of 1 CPU core (on a modern Ryzen).
To Reproduce
Open the app.
Expected behavior
< 1%
Log and backtrace
I the default config, though suspect this has something to do with image.
Environment

OS: Arch
Application version: 0.14.0
Application features: -F notify,image,lyric-finder,pulseaudio-backend,streaming,media-control --no-default-features

FYI, it was ~0.7% of 1 CPU core for my M1 Macbook Air.  It would be helpful if you can provide a video/pictures for a demonstration.
Why do you think that 7% is not a good number but 1%? Do you have any baselines for comparison?

I'll try to compile with other feature sets, to see if I can narrow it down.
Compared to spotifyd, which uses 0-1% while idle. Playback should consume more cpu than being idle, but for now playback only uses ~2%, which is great!

Looks to be an issue even with all features disabled. I'll include the flamegraph. How often is the UI updated? IIRC, spotify-tui heavily throttled updates when the window wasn't focused (I don't know how to detect that in terminals).

How often is the UI updated

It's determined by app_refresh_duration_in_ms config option (default to be every 30ms). You can increase the value if the default one feels consuming too much CPU.

Thanks! Would it be possible to make that refresh interval 500ms when I'm not inputting anything into spotify_player, but 32ms when I interact?

Also seeing CPU usage around 7% too.

I'm seeing high CPU usage on my macbookpro11,3 under EndeavourOS usually in the mid 30% range. This is true while playing and running via cargo run --no-default-features. Generally a great alternative to spt but that's not really usable for me :disappointed:

I've been neglected the app's performance for a while as its CPU usage is relatively low for me. top also gives me around 7% w.r.t one core. The previous tool I used report CPU usage w.r.t all cores, so the number is 1/8 smaller.
Now, as I'm more interested in performance optimization, I'll come back to this issue and try something.

@aome510 great app so far, however, I see also a significant CPU usage with defaults settings on macbook M1. ncspot in comparison is neglibible.

@lbonvarl thanks for reporting. Can you share the number on your machines, e.g what are CPU usages of spotify_player and ncspot? It can provide a good baseline for me to optimize.
That said, kinda busy lately, so I didn't have the chance to pick up this yet.

Hello @aome510, I am on Macbook Pro 13in, 2020 M1 16GB Sonoma 14.5, using current iterm2 3.5 under zsh.
spotify_player sitting idle not playing is at 35% CPU on task manager with defaults setting. I installed via "cargo install spotify_player --features lyric-finder".

I also did samply "record ./spotify_player" from "spotify-player/target/debug" after a cargo build

In comparison, same machine here is the data for ncspot installed via homebrew.  CPU usage is when idle at 2-3%

Hello @aome510, do you need any extra data for this issue ? Let me know.

Same here. Around 5-7% on idle. Running on 24.05 nixos, pkg from unstable channel.


update vidhukant.com size

This PR is:

[ ] Adding a new domain
[x ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[ x] I used the uncompressed size of the site
[ x] I have included a link to the GTMetrix report
[ x] The domain is in the correct alphabetical order
[ x] This site is not an ultra lightweight site
[ x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[ x] Check to confirm

- domain: vidhukant.com
  url: https://vidhukant.com
  size: 28.3
  last_checked: 2023-08-27

GTMetrix Report: https://gtmetrix.com/reports/vidhukant.com/lWFlxkZQ/

Thanks for cleaning up the whitespace. I think it's time to add a YAML lint step to the build pipeline...🙈


Would like to introduce the Argo W/F to my organization

**Question: Have you read the mentoring guide? Please only submit this request for general mentoring (not GSoC).Y
Answer: Yes I have read the mentoring guide.
What is your background? Any experience with Go, Kubernetes, React, Typescript, etc.?
Answer: Minimal experience in react but has some experience Kubernetes .
Is there any particular issue you'd like to work on? You may want to check out the list of good first issues.
Answer: I would like to work with issue #7920
Do you want to work with your mentor async (i.e. by Slack messaging or issue comments), or sync (e.g. using Zoom video)? Please include your time-zone if you want to work sync.
Answer: (async/sync) (EST)

#7920

Hi @nagesh1977. I can try to help where I can. It sounds like you will be mainly contributing on the front end? Let me know what you might be looking for. I have worked on the front end a little bit but not a ton. I am still pretty new to React.

Hello Julie
Thanks for your response. I have slightly different requirements here. Let
me put up some details.
My Organization is in the journey of transformation from legacy technology
and we are introducing K8s. One big challenge is to address the workflow
orchestration over k8s, traditionally we are using the Autosys with
dedicated servers.
Therefore we are looking for options which will replace Autosys and we
found that ARGO workflow fits the bill.  Argo workflow has most of the
features we are looking for and our engineering team is eager to adopt it.
However I find that the documentation is not that elaborative in most
cases. I looking for in depth documentation covering mostly topics like
a) Role specific workflow execution.
b) Security features.
c) Rest API access.
Please let me know in case you can help with this.
Regards
Nagesh
On Mon, Dec 5, 2022 at 11:40 AM Julie Vogelman @.***>
wrote:

#7920 https://github.com/argoproj/argo-workflows/issues/7920
Hi @nagesh1977 https://github.com/nagesh1977. I can try to help where I
can. It sounds like you will be mainly contributing on the front end? Let
me know what you might be looking for. I have worked on the front end a
little bit but not a ton. I am still pretty new to React.
—
Reply to this email directly, view it on GitHub
https://github.com/argoproj/argo-workflows/issues/10171#issuecomment-1337695789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHRLHEE35FBSOX6OAW22X4DWLYLJHANCNFSM6AAAAAASTNEAUY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for clarifying the need. Unfortunately, what I know on those topics also comes from the online documentation. Have you tried starting a discussion here to ask your questions? Or alternatively going to the CNCF argo-workflows slack channel?

Thanks Jullie for the quick response.
Also let me be part of the project as well, i need time to understand the
code but let me know pick up some basic jira  to start with
On Fri, Dec 16, 2022 at 1:37 PM Julie Vogelman @.***>
wrote:

Thanks for clarifying the need. Unfortunately, what I know on those topics
also comes from the online documentation. Have you tried starting a
discussion here https://github.com/argoproj/argo-workflows/discussions
to ask your questions? Or alternatively going to the CNCF argo-workflows
slack channel?
—
Reply to this email directly, view it on GitHub
https://github.com/argoproj/argo-workflows/issues/10171#issuecomment-1355398197,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHRLHEBO764FRRFMQAKYR63WNSZE7ANCNFSM6AAAAAASTNEAUY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks Jullie for the quick response. Also let me be part of the project as well, i need time to understand the code but let me know pick up some basic jira to start with
…
On Fri, Dec 16, 2022 at 1:37 PM Julie Vogelman @.> wrote: Thanks for clarifying the need. Unfortunately, what I know on those topics also comes from the online documentation. Have you tried starting a discussion here https://github.com/argoproj/argo-workflows/discussions to ask your questions? Or alternatively going to the CNCF argo-workflows slack channel? — Reply to this email directly, view it on GitHub <#10171 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHRLHEBO764FRRFMQAKYR63WNSZE7ANCNFSM6AAAAAASTNEAUY . You are receiving this because you were mentioned.Message ID: @.>

Feel free to take an issue labeled good-first-issue. Some general things about where to find the code:
Key Files:
Controller:

pkg/apis/workflow/v1alpha1/workflow_types.go: defines Workflow spec
workflow/controller/controller.go: main loop for processing Workflows
workflow/controller/operator.go: called by controller.go to process a Workflow
Argo Server:
server/apiserver/argoserver.go

Key Directories:

cmd/argo: Argo CLI
cmd/argoexec: Executor running in Workflow Pods
docs
examples
manifests (uses kustomize)
pkg/apiclient: REST interface, Go client
server: Argo Server
workflow/controller

Closing as stale with no response in ~9 months


GUI is broken under Ununtu Gnome 16.04

the GUI can not be precisely operated using mouse (touchpad).
and keyboard also can't be used to operate on the GUI so the app basically is unable to configure.

Could you please provide more details?

installed from .deb, version v0.2.5
when the GUI configuration window is opened, my mouse pointer can not operate on them, as if they have different axis other than my desktop.
both scroll bar and drop down menu of sound style have this issue
but the quit button can be clicked in its right position
i already removed the app and re-install it using pip, still the same broken GUI issue
so now i use the CLI interface so that i can really enjoy the app
btw i don't know how to trigger the CLI interface from the main program so i must use something like python run_CLI.py -c as the entry point to the app.

Which Linux distribution you used?

Thank you, I will check it later.

i have the same problem and some times tickeys close from itself


Fixes for comparators

Fix argument matching for StructEqualWithExplanation and WrapperEqualWithExplanation in order to let people know they should change the comparators when they change the structures. As you'll notice from this PR, they don't always remember to do it. In case you think that these should be automatically generated: I fully agree, I sent a PR that did it, but it seems that I used a reflection library that wasn't nice enough. Maybe I'll try to do it again, but I won't mind if someone else does it before.

Fix line length issues.

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

For whatever it's worth, here's my previous PR:
https://github.com/kframework/kore/pull/382


Add a default user id on Android.

This renders like so,

Which isn't amazing, but I think the android_id part is important so people realize what it is, and so  if/when they set their own ID it's clear they're different...
#329

Looking at this again, maybe it's look nicer if it were just android:XXX so it renders as id:android:XXX...

Changed it to android:abcd


possible problem with current go snapshot

This doesn't affect us, because it isn't the term  we need, but reporting as it is presumably a problem:
from https://github.com/geneontology/go-ontology/issues/16329
From @kimrutherford
That took a bit of digging but it looks like it might be a problem with
the current snapshot.  We download this snapshot file and then process
it with owltools to create an OBO file that obeys the taxon
constraints:
http://purl.obolibrary.org/obo/go/snapshot/extensions/go-plus.owl
The OBO file from that doesn't contain GO:0060468
But if I do the same process with the release go-plus.owl:
http://purl.obolibrary.org/obo/go/extensions/go-plus.owl
the result does contain GO:0060468

When I look at http://purl.obolibrary.org/obo/go/snapshot/extensions/go-plus.owl, I do see GO:0060468. Maybe if @kimrutherford can paste the owltools command I can try to replicate.

ok maybe it is fixed now. I still don't see
+[Term]
+id: GO:0140538
+name: negative regulation of conjugation with zygote
+namespace: biological_process
+def: "A process that prevents a zygote from fusing with additional cells." [PMID:30089908]
+is_a: GO:0060467 ! negative regulation of fertilization
+property_value: term_tracker_item #16329 xsd:anyURI
+created_by: pg
+creation_date: 2020-11-07T07:50:23Z
as of today- I'll try tomorrow.
cheers,
v

Weird. There was a snapshot yesterday.  I see the term in the go-basic.obo file here http://snapshot.geneontology.org/ontology/index.html
Thanks, Pascale

Hmm, did anything else change at GO end?
Or at our end @kimrutherford ?

When I look at http://purl.obolibrary.org/obo/go/snapshot/extensions/go-plus.owl, I do see GO:0060468. Maybe if @kimrutherford can paste the owltools command I can try to replicate.

Hi Jim.
This is the command line we use to make a pombe-only subset:
curl -L http://purl.obolibrary.org/obo/go/snapshot/extensions/go-plus.owl > go-plus-snapshot-2020-11-12.owl
owltools go-plus-snapshot-2020-11-12.owl --reasoner elk  --make-species-subset --perform-macro-expansion false -t NCBITaxon:4896 -o -f obo --no-check go-plus-snapshot-2020-11-12.obo
grep GO:0060468 go-plus-snapshot-2020-11-12.obo

I used the 2020-04-06 release of owltools.  GO:0060468 is missing from the output.
If I do the same thing with the release go-plus.owl, GO:0060468 is in the output:
curl -L http://purl.obolibrary.org/obo/go/extensions/go-plus.owl > go-plus-release.owl
owltools ./go-plus-release.owl --reasoner elk  --make-species-subset --perform-macro-expansion false -t NCBITaxon:4896 -o -f obo --no-check go-plus-release.obo
grep GO:0060468 go-plus-release.obo

go-plus-release.obo:id: GO:0060468
go-plus-release.obo:relationship: part_of GO:0060468 ! prevention of polyspermy

Thanks for your help.

Sorry, I just noticed this:

ok maybe it is fixed now. I still don't see
+[Term]
+id: GO:0140538

Val, in the other issue (#16329) you mention GO:0060468 but in this issue you mention GO:0140538.  Which one is the problem?  GO:0140538 is not in the taxon-restricted output OBO file for the release or the snapshot.  Should it be?

Sorry, I might be conflating 2 issues, but I wondered if they were related.
I wanted to replace
GO:0060468 ! prevention of polyspermy
in
https://curation.pombase.org/pombe/curs/46c89f342f117ca5
by
GO:0140538: negative regulation of conjugation with zygote
So 2 unusual things occurred at the same time:

GO:0140538 did not  yet become available
GO:0060468 prevention of polyspermy became "UNKNOWN TERM"  in Canto  (so I could not re-approve the session while the new term came through) which would usually be possible.

Even though "GO:0060468 prevention of polyspermy" will eventually  become taxon restricted for fission yeast (might even be now) , it was not at the point when I looked. I could not see any obvious reason why this term would appear as "UNKNOWN TERM"
So, I had 2 odd things occurring in the same annotation session, I assumed there might be some connection, but it is also possibly just a coincidence......

@kimrutherford thanks, that was very helpful! I think I know what is going on. This is a consequence of the way --make-species-subset works. GO:0060468 is not restricted from fission yeast. However some terms it relates to are connected to 'multicellular organismal process'. It's confusing because I do not mean that 'in taxon' is being inferred across these relationships. --make-species-subset temporarily pretends that ALL terms are in_taxon fission yeast (for example). Then it queries for all unsatisfiable classes, and removes them.
The reason this has suddenly started happening is that we recently started bringing in additional axioms from CL. The problem axiom is 'germ cell' SubClassOf 'capable of' some 'multicellular organismal reproductive process'. Either we need to remove this, or remove the restriction 'multicellular organismal process' never_in_taxon 'Schizosaccharomyces pombe'.

@cmungall what do you think the fix is for this? Or @ValWood are you fine with this?

It depends what  you are asking
It wouldn't be the best solution to remove
'multicellular organismal process' never_in_taxon 'Schizosaccharomyces pombe'
because this high level taxon check captures/prevents a lot of taxon violations
GO:0060468  really IS a taxon violation. It's just that when had used it, this wasn't so clear, and it did not have a flag. Now have a taxon appropriate term, I requested a taxon restriction on GO:0060468, but I could not figure why we were already being prevented from using it.  So if these inferences can tell us terms which appear to be taxon inappropriate based on the reasoning, even though they are not flagged ass such  the current behaviour is  is fine.
Did I understand correctly? In summary, I would always  rather be told that something is incorrect than not be told that something is  incorrect.

@ValWood would you ever want to use 'fertilization' or 'germ cell' with S. pombe? I just googled and I guess they do a sort of fertilization (https://www.sciencedirect.com/science/article/pii/S0960982218312739).
I would expect both of these terms to be excluded by --make-species-subset as well.

that's OK. We use the related  fungal equivalent "conjugation with cellular fusion".
Yeast people are not partial to the words 'gamete' and 'fertilization' and the processes are sufficiently different to exclude.


adding hwflwctl support for NUCLEO_F401RE

adding changes to support hardware flow control in NUCLEO_F401RE Target.
3 files changed:

libraries/mbed/targets/hal/TARGET_STM/TARGET_STM32F4/TARGET_NUCLEO_F401RE
/PeripheralPins.c
libraries/mbed/targets/hal/TARGET_STM/TARGET_STM32F4/TARGET_NUCLEO_F401RE/device.h
libraries/mbed/targets/hal/TARGET_STM/TARGET_STM32F4/TARGET_NUCLEO_F401RE/objects.h

Test reports for singletest.py tested with GCC_ARM attached as a zip file.
report.zip

@bcostm @adustm

LGTM
@mridup Can you please sign https://developer.mbed.org/contributor_agreement/?


Integrate chocolatey package creation into build script

Fixes #41
@NikolayPianikov I'm creating the PR so you can review. We still have unexplained failures in AppVeyor and Travis, similar to the previous PR. Under Appveryor, we have integration test failures and under travis a complete failure of the nuget restore step. I feel like you understand your project better than I and may be able to figure this out.
As for the creation of the chocolatey package, I used the same logic as in the other extensions but did not reorganize the code in the same way. I'll leave it to you to keep it structured as you prefer or change it in the future.

@rprouse Can you suggest what's going wrong with the Travis builds? It started when I upgraded the packages.

@CharliePoole, I will try to reproduce issue on Linux/mono and hope it helps to find what is wrong

@CharliePoole looks like there is some problem during resolving dependencies in NUnit package 3.7.x, I've rolled back NUnit to 3.6.1 and it is working now

@NikolayPianikov - I believe the issue is caused by this repo downloading too old a version of nuget.exe. build.sh currently downloads v2.8.6, which I don't believe can handle the NUnit v3.7.1 NuGet package correctly.
Try replacing build.ps1 and build.sh with the latest versions from the link below, which download NuGet v4.1.0. (NuGet changed the download link for the latest version, which is why this repo is still fetching v2.8.6!)
https://github.com/cake-build/resources

@ChrisMaddock thank you, I will check

@CharliePoole, @ChrisMaddock, mono 3.2.8 still has error and I can't reproduce it locally, do you have any ideas?

Not sure sorry, we've dropped mono 3.2.8 support on the main projects.

Travis has recently upgraded the default Unix environment to Trusty - and looks like the package can't be found there? Maybe try adding dist: precise to the .travis.yml? (Or just dropping 3.2.8...that would be my preference! ;-) )

I got rid of mono 3.28, consistent with what we are doing with other projects. Travic-ci now passes.
We still have a failing integration test in the AppVeyor pr build. What do you want to do?

@CharliePoole I will ignore test while I do not find a problem

@CharliePoole don't you have any objections to merge PR?