Tweaks to  component

Well, that was more of an odyssey than I expected. The original issue reported by @scytacki was that in its steady-state, when viewed in a browser with caching disabled (not an expected end-user situation), leaf-pack continuously loads the images used for animation (i.e. the network panel continues to show activity for each animation frame). It was hypothesized that this might lead to performance issues as well, e.g. higher CPU usage.
The relatively simple fix which was straightforward to implement was to render all frames into the DOM and use CSS visibility to control which frame is visible rather than rendering only a single frame at a time into the DOM as we had been before. In a parallel universe that makes sense this would have been the end of the story. In this universe, however, it was just my entree into the rabbit hole.
With the fish animations, unlike any of the other animations, the animation occurs within an opacity transition, because the fish fade in gradually while they're animating/swimming. This opacity transition was handled with ReactTransitionGroup's <TransitionGroup> and <CSSTransition> components, which under the hood make use of a CSS transition on opacity. After making the change to control frame visibility through CSS visibility rather than through presence/absence in the DOM, there was obvious and distracting flashing that occurred. Looking closely it was apparent that the images were flickering between the properly translucent state consistent with their opacity and their final state somewhat randomly. After beating my head against this for several hours trying to figure out what I was doing wrong, I eventually came to the conclusion that browsers apparently can't handle simultaneously changing CSS properties while transitioning another CSS property (e.g. opacity). I tried using position (left, right) to control visibility instead of visibility but the effect was the same. This issue was raised on Stack Overflow for Safari years ago (by none other than React's Dan Abramov) but now appears to affect Chrome and Firefox on Mac and Windows as well.
Eventually, it occurred to me that I could eliminate the <TransitionGroup> and <CSSTransition> components and just handle the opacity inside the animation component itself with a timer. Now the opacity is handled by the <SimulationAnimation> component itself rather than relying on a CSS transition applied by the parent in the case of the fish animation. Phew!

Fixed -- see updated description above.


get current retry iteration count

Is there a way to get the current retry iteration value?  I've observed this.test._currentRetry and this._runnable._currentRetry and both counts are stuck on 0.

Looks like this is not needed any longer.  Mocha now has retries functionality built in https://github.com/mochajs/mocha/blob/master/bin/_mocha#L97


Not working on Windows 10.0.17133.1 (Spring Creators Update)

Hello, I upgraded to the new version of Windows and RDP Wrapper doesn't appear to work on this build. Just thought I'd let you guys know! Thanks for all your help.

Win 10 Home or Pro?
32-bit or 64-bit?
On Windows 10 Home you might need to get a copy of rfxvmt.dll and place it somewhere in your path (ex c:\windows\system32)

This is Win10 Pro, 64bit.

Same issue. To try and fix it, I copied rfxvmt.dll from \System32 and \SysWOW64 from a working copy of Windows 10 (same build, 17133.1) Pro, and just for sure the termsrv.dll from \System32, to a copy of Windows 10 Home (same build, 17133.1), which had previously worked with RDPWrapper 1.6.2
I ran the following commands on all 3 DLL's, the two copies of RFXVMT and the one of TERMSRV, then restarted:
net stop TermService
takeown.exe /A /F rfxvmt.dll (or termsrv.dll)
icacls.exe rfxvmt.dll /grant "PCADDR\user:F"

icacls.exe rfxvmt.dll /inheritance:d
icacls.exe rfxvmt.dll /setowner "NT SERVICE\TrustedInstaller"
icacls.exe rfxvmt.dll /grant "NT SERVICE\TrustedInstaller:F"

icacls.exe rfxvmt.dll /remove "NT AUTHORITY\SYSTEM"
icacls.exe rfxvmt.dll /grant "NT AUTHORITY\SYSTEM:RX"

icacls.exe rfxvmt.dll /remove "BUILTIN\Administrators"
icacls.exe rfxvmt.dll /grant "BUILTIN\Administrators:RX"

icacls.exe rfxvmt.dll /remove "PCADDR\user"

I still have the "Not listening"/"Not Supported" error in Listener State on the RDP Wrapper Configuration

Is there a planned update to this? I would like to be able to deploy via chocolatey?

please, upload termsrv.dll from Fall Creators Update

I got it from here https://www.mysysadmintips.com/-downloads-/Windows/Clients/termsrv_win_10.0.16299.15.zip

sorry, I mean Spring Creators Update

After dll from Fall Creators Update RDP successful not working

Rather than a kludgy fix, I paid less than $5 to get a Pro license key from eBay.

I have license

What was the fix for this?

Please - can someone with freaky skills patch the 0.0.17133.1 alias 1803 64 bit Version (Enterprise)?
I know MS has killed the RTM Status ... but there are many people now out there sitting on a shitty 1803 Version without RDPWrap function ...8(
Please help ...

I've just found a solution on this link.
https://www.mysysadmintips.com/windows/clients/545-multiple-rdp-remote-desktop-sessions-in-windows-10
Thanks to Adam
_Windows 10 April 2018 Update 17134 RTM x64 build:
Find:
8B 99 3C 06 00 00 8B B9 38 06 00 00
Replace With:
B8 00 01 00 00 89 81 38 06 00 00 90
Cheers
Adam_

Yesterday night，my Win10 home edition/x64 automatically updated to Win10 1803, and now I found the RDPwrap not works...

Replace the C:\Windows\system32\termsrv.dll with the one from C:\Windows.old\system32\，after restart the computer, it works!

Replace the C:\Windows\system32\termsrv.dll with the one from C:\Windows.old\system32\，after restart the computer, it works! I confirm! Thank you!

That indeed works! In case you can't delete the file, you need to take ownership first.
Then you should be good to go.

how can i replace the DLL in win 10 Home？

I am in Same boat with Win10 1803 what do you mean take ownership first I can't seen to copy and paste from the windows.old on termsrv.dll please help

Check this link:
https://www.laptopmag.com/articles/take-ownership-folder-windows-10-using-file-explorer

I can also confirm it works when you replace the termsrv.dll with the one from Windows.old. Reboot isn't necessary; a restart of the remote desktop service is enough. Actually you have to turn it off in order to replace the file so just start it again and it works.

Update: On my system updated with the Spring Creator update with the termsrv.dll copied from Windows.old I get two logon-screens when connecting using RDP. After logging in on the first one I get a blank bluish screen for less than a second and then the logon screen again. After logging in the second time I get the normal desktop and everything works as before.

There is a thread for Windows 10.0.17134.48??

I also just updated to Windows 10 build 1803 17134.48.
I am getting Not Listening Not Supported when I run RDPConf. I downloaded the latest version 1.6.2 and ran update as well.
I tried to implement the suggested change - moving termsrv.dll from Windows.old, but there is no system32 directory under Windows.old. Searching for termsrv.dll in that tree turned up nothing.

Get the termsrv.dll from the link earlier in this thread - the post by gave92.

@gave92 @XenobyteDK the link you posted for termsrv.dll isn't working... anyone who could upload the x64 version of it?
Thank you.

same issue here...
win10 pro updated from 1709 to 1803 and even when replacing with termserv under windows.old it wont work, only one session allowed :(
any idea ? thx !

Hi All,
I am facing an issue ie. a new windows update is installed after the installation,my RDP Wrapper is not working. Kindly give solution to resolve this issue.

Replacing termsrv.dll with the link provided above resolved the issue for me.

In Command Prompt run: net stop TermService
Download termsrv.dll from:
https://www.mysysadmintips.com/-downloads-/Windows/Clients/termsrv_win_10.0.16299.15.zip
or
https://transfer.sh/LaY0f/termsrv.zip
and extract
Goto C:\Windows\System32 in explorer
Right-click > Properties > Security > Advanced
Make yourself the owner of the file and close all windows
Back in C:\Windows\System32, goto the Properties > Security tab again and hit Edit
Give yourself (or Administrator assuming you are admin of your box) modify rights
Copy and paste the newly extracted termsrv.dll into the folder
Hit replace when prompted
You should then run in Command Prompt: net start TermService

Then run RDPConf to confirm the Listener is in a "Listening" state, and then RDPCheck to confirm RDP access locally.
It worked for me, hope it works for others.


Kong 1.1.1 with MariaDB support

Hi,
The issue that I wanted to add this post is already closed - #1867 (MySQL support).
However I'd like to inform you that we've made public our version of Kong 1.1.1 with MariaDB support and additional changes like IPv6 support or Cassandra cluster topology.
Repo: https://github.com/nokia/kong/tree/nokia-1.1.1
Changelog: https://github.com/nokia/kong/blob/nokia-1.1.1/CHANGELOG.md#nokia-111
Of course I'm aware  that the newest version is on 2.x and that DB-less is right now more popular - however you may take a look on our implementation and consider supporting it together with PostgreSQL and Cassandra.
Thanks and regards,
Jarek
CC: @peacemakr, @tnucera, @li-wl, @MichelLiu,

Hello @jarekchr,
Thanks for your contributions!
At this point we do not have plans for adding MySQL support inside the official Kong repo and instead focusing more on improving the Hybrid mode deployment mode. But we may reevaluate this decision in a later time.


The product name is incorrect in Release Planner page

In below image the produce name should not be productsku health decistion support but Confidential ledger.

Fixed and deployed in PPE


Support macOS 12 dyld shared cache format

On macOS 12 the dyld shared cache format seems to have changed. At least this is the case in the Beta that I have running at the moment.
Instead of one cache file per architecture, the cache is now split up into multiple files, each being a little over 700MB. The filenames are:
/System/Library/dyld/dyld_shared_cache_x86_64
/System/Library/dyld/dyld_shared_cache_x86_64.1
/System/Library/dyld/dyld_shared_cache_x86_64.2
/System/Library/dyld/dyld_shared_cache_x86_64.3

(and similarly for the other architectures)
The old list of images (currently parsed by DyldCache::images) in each file is now empty.
Instead, the header points to a new list of images, let's call them images_across_all_chunks. For a given architecture, all cache chunk files have the same full list of images_across_all_chunks, with the same image addresses.
For example the x86_64 cache has the following list, in all cache chunk files:
<image_address> <path>
0x7ff800050000 /usr/lib/system/libsystem_blocks.dylib
0x7ff800052000 /usr/lib/system/libxpc.dylib
[...]
0x7ffb1bc46000 /System/iOSSupport/usr/lib/swift/libswiftWebKit.dylib
0x7ffb1bc4a000 /System/iOSSupport/usr/lib/swift/~libswiftPencilKit.dylib

However, the cache files differ in the mappings. It looks like each chunk covers a 4 GiB range of virtual addresses.
/System/Library/dyld/dyld_shared_cache_x86_64:
0x7ff800000000-0x7ff81fd10000 at file offset 0x0
0x7ff840000000-0x7ff8418c8000 at file offset 0x1fd10000
0x7ff8418c8000-0x7ff843664000 at file offset 0x215d8000
0x7ff880000000-0x7ff88bf08000 at file offset 0x23374000

/System/Library/dyld/dyld_shared_cache_x86_64.1:
0x7ff900000000-0x7ff9201e0000 at file offset 0x0
0x7ff940000000-0x7ff942878000 at file offset 0x201e0000
0x7ff942878000-0x7ff944664000 at file offset 0x22a58000
0x7ff980000000-0x7ff98cac4000 at file offset 0x24844000

/System/Library/dyld/dyld_shared_cache_x86_64.2:
0x7ffa00000000-0x7ffa1ff44000 at file offset 0x0
0x7ffa40000000-0x7ffa42304000 at file offset 0x1ff44000
0x7ffa42304000-0x7ffa44f70000 at file offset 0x22248000
0x7ffa80000000-0x7ffa88ea0000 at file offset 0x24eb4000

/System/Library/dyld/dyld_shared_cache_x86_64.3:
0x7ffb00000000-0x7ffb1bc58000 at file offset 0x0
0x7ffb40000000-0x7ffb41254000 at file offset 0x1bc58000
0x7ffb41254000-0x7ffb43d98000 at file offset 0x1ceac000
0x7ffb80000000-0x7ffb8adbc000 at file offset 0x1f9f0000

The images_across_all_chunks list can be accessed from the header like this:
#[derive(Debug, Clone, Copy)]
#[repr(C)]
pub struct DyldCacheHeader<E: Endian> {
    /// e.g. "dyld_v0    i386"
    pub magic: [u8; 16],
    /// file offset to first dyld_cache_mapping_info
    pub mapping_offset: U32<E>,
    /// number of dyld_cache_mapping_info entries
    pub mapping_count: U32<E>,
    /// file offset to first dyld_cache_image_info
    pub images_offset: U32<E>,
    /// number of dyld_cache_image_info entries
    pub images_count: U32<E>,
    /// base address of dyld when cache was built
    pub dyld_base_address: U64<E>,
    ///
    reserved: [u8; 408],
    /// file offset to first dyld_cache_image_info
    pub images_across_all_chunks_offset: U32<E>,
    /// number of dyld_cache_image_info entries
    pub images_across_all_chunks_count: U32<E>,
}

The mapping_offset in the build I have is 456. This is also the size of the header at the same time. (The list of mappings starts right after the header.)

It's probably a good idea to wait for macOS 12 to be released and for the new dyld source to become available, before making any changes to object.
This issue is just a heads-up.

The implementation I suggested above doesn't seem to work in all cases. For example, for arm64e libsystem_malloc.dylib, it looks like two chunks are needed in order to get symbol names: The list of symbols is in the first chunk and the string table is in the second chunk.

The strings are in the chunk which contains the __LINKEDIT segment.
(It looks like chunks are called "sub caches".)
All images in the arm64e dyld shared cache share the same __LINKEDIT segment, and it's in the .1 subcache. For example, on macOS 12.0.1, all images (in both subcaches) have a __LINKEDIT segment with the following address range:
0x21018c000-0x233c039fd at file offset 0x2a7f4000

And the subcache mappings are as follows:
dyld_shared_cache_arm64e:
0x180000000-0x1cf73c000 at file offset 0x0
0x1d173c000-0x1d4334000 at file offset 0x4f73c000
0x1d6334000-0x1d8f98000 at file offset 0x52334000
0x1d8f98000-0x1da6d0000 at file offset 0x54f98000
0x1da6d0000-0x1dd96c000 at file offset 0x566d0000
0x1df96c000-0x1df998000 at file offset 0x5996c0

dyld_shared_cache_arm64e.1:
0x1df998000-0x204628000 at file offset 0x0
0x206628000-0x2073a0000 at file offset 0x24c90000
0x2093a0000-0x20bb18000 at file offset 0x25a08000
0x20bb18000-0x20c81c000 at file offset 0x28180000
0x20c81c000-0x20d9e8000 at file offset 0x28e84000
0x20f9e8000-0x2371dc000 at file offset 0x2a050000

I have this working now.

It works with ipsw, but I know Golang isn't pleasant to read to some people;)

Thanks for the pointer! This suggests that I should also add support for the .symbols subcache. This seems to be only used on iOS, not on macOS.
For reference, ipsw's dsc header definition is here: https://github.com/blacktop/ipsw/blob/05af7d2bde570035b25e7bfdf316d9f363c0106c/hack/extras/Dyld.bt#L116-L124
(I've found a small issue with that header definition, filed as https://github.com/blacktop/ipsw/issues/49.)

Yes on the non-macOS caches all the local symbols are in the .symbol subCache.

The source for macOS Monterey dyld has been released now.
The new fields which I called images_across_all_subcaches_offset/_count are just called imagesOffset/Count, with the old fields renamed to imagesOffsetOld / imagesCountOld:
https://github.com/apple-oss-distributions/dyld/blob/5c9192436bb195e7a8fe61f22a229ee3d30d8222/cache-builder/dyld_cache_format.h#L99-L100


Set Active page call not working

the set_active_page(page_name) does not work. is there a way a different way to change the active page?

It's working fine for us.
Get the page name from the function get_pages() and set the active page using set_active_page(page_name). Followed the below code.
report.get_pages()
report.set_active_page("ReportSection1234")
report

Please provide more additional information if the issue still exists.


FR-12873 - vue2 clean mapEntitlemenetsState

mapEntitlemenetsState is not needed anymore

Why is it not needed any more?

will explain tomorrow f2f. (related to proxy of something that is changing)


Unable to Create SystemDSN Data Source after fresh ODBC driver install

Driver version
Version <IP_ADDRESS>
Client Operating System
Virtual Machine
Problem description
Problem: Trying to add a SystemDSN after using the Microsoft Driver Installer to install driver, yields this error.

Driver Exists

Referenced Path in Error Message Exists

Please advise

Solved: Had to download https://visualstudio.microsoft.com/downloads/#microsoft-visual-c-redistributable-for-visual-studio-2022 dependency for the Driver to work


As a CIF administrator, I want to set the funding stream once before creating the project, so that I can see customized forms for each funding stream

Description:
Funding streams will need separate forms, and supporting changing a project's funding stream will be:

a nightmare programming-wise
generating inconsistent data in the existing forms

Acceptance Criteria:
Given
When
Then
Development Checklist:

[ ] Checklist item
[ ] Checklist item
[ ] Checklist item
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

Hi @pbastia - to support this ticket, do we require some user research to more clearly identify where IA differs from EP? And then some UI design?

@LindsayMacfarlane  All the research! :D and a double-check with the business area, and a dev discussion on how to make this fit our database schema

@pbastia and @nanyangpro - this ticket seems to be the same as #1075? If so I can merge any details then close/archive this ticket.

I think so, yes (unless @pbastia suggest otherwise). Thanks @LindsayMacfarlane!


Separation of named/positional params lead to API duplication

For example, all the _named (vs the non-_named) functions on https://docs.rs/rusqlite/0.21.0/rusqlite/struct.Statement.html. In mozilla/application-services, we even have a extension trait which exists/started mainly as a way to get more methods for _named variants (also cached) and such: https://github.com/mozilla/application-services/blob/master/components/support/sql/src/conn_ext.rs
Really this is probably my biggest complaint about rusqlite's API. I don't really have a solution (the ones I've thought of all hit coherence issues or restrict the API in ways I wouldn't want to give up), but it seems like it should have an issue on file, since its a pain for maintenance. It's probably worth investigating what other non-orm (and non-macro) SQL crates do.

As far as know,

postgresql does not support named parameters, only $NNN. So you can do SELECT $1, $1 + 10;
JDBC does not support named parameters, only ?. So you must bind several times the same value.
SQLite does support named parameters but sqlite3_bind() functions support only indexes. sqlite3_bind_parameter_index() must be used to translate a symbolic parameter name into its equivalent numeric index.

So maybe we should have only non-_named functions and add a single method that do the translation: &[(&str, &dyn ToSql)] => IntoIterator<Item=ToSql> ?

I didn't realize that this was so uncommon, most of my professional experience has been with SQLite or MySQL (which also has them).
I'm aware that there's a cost, but my experience has definitely been that the readability gains can be substantial, and I've never seen the cost take any time compared to the query that generally follows. So I don't really like having there be a separate function for this...
Along the lines of your suggestion, something like
trait IntoParams {
    type Param: ToSql;
    // It's likely that this would be better as `Item = (i64, Self::Param)`
    // to avoid needing to reorder for named params...
    type Params: IntoIterator<Item = Self::Param>;
    fn into_params(self, stmt: &Statement) -> Self::Params;
}

might work (probably with more Result tho).
E.g. basically IntoIterator<Item: ToSql> but that passes the statement into its ctor, and each function could take params: P) where P: IntoParams or something (e.g. users probably would never call the trait directly).
These possibilities have coherence issues though (since it would likely need a blanket for the existing cases, which I think are important, to work)... Maybe having named params be a wrapper around the &[(&str, &dyn ToSql)]  (which get created by named_params!, or .into()...). would make it clear enough to the impl solver that the cases don't overlap. Dunno, I might poke around with it at some point.
What are your thoughts?


Failing to Resolve Dependancy

Pulling in the dependancy as per the README, results in 403s.
repositories {
    maven { url = uri("https://dl.bintray.com/47deg/helios") }
}

dependencies {
    compile "com.47deg:helios-core:0.2.0"
    compile "com.47deg:helios-parser:0.2.0"
    compile "com.47deg:helios-optics:0.2.0"
    kapt "com.47deg:helios-meta:0.2.0"
    kapt "com.47deg:helios-dsl-meta:0.2.0"
}

Similarly the issue occurs with a simple wget
wget https://dl.bintray.com/47deg/helios/com/47deg/helios-core/0.2.0/helios-core-0.2.0.pom

Can we get this project successfully published?

@dr4ke616 did you manage to overcome this?
is this project dead?

The wget command above still returns with a 403. So i suspect the issue is still present.


Microsoft.Automation swagger linter-check fix for sdk generation

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

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

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

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

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

Contribution checklist:

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

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

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

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

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

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

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

The breaking change can be ignored , as the object definition is not changed

The breaking change can be ignored , as the object definition is not changed

@akning-ms  could you force merge it , it's to fix the existing conflict in the PR https://github.com/Azure/azure-rest-api-specs/pull/12073 , so that the lintDiff can run normally

@akning-ms  could you force merge it , it's to fix the existing conflict in the PR https://github.com/Azure/azure-rest-api-specs/pull/12073 , so that the lintDiff can run normally

/azp run

/azp run


MGMT-8860: Update Hypershift to create the AgentCluster with minimal spec

MGMT-8860: Update Hypershift to create the AgentCluster with minimal spec

What this PR does / why we need it:
This PR updates (removes unnecessary fields) agentcluster crd. From this change agent-provider will take those fields from from hostedcontrolplane.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test capi-provider-agent-sanity

can you please explain why do we need this PR?

/retest

/approve

/lgtm

/test capi-provider-agent-sanity


Revert "Rollforward "[vm errors] Propagate status message to adapters""

Reverts move-language/move#949
Let's wait until we cleanly make it through the branch cut before adding in new features :pray:

This is the wrong approach. Instead we should create aptos-1.3 similar as we did with 1.2. Then all this roll back/forward/back is not needed.


Mf refactoring

Upper level changes for MF refactoring.  Edits here only cover top level terms.  There are separate branched for ATPase-coupled activities, receptors and transcription factor activities.
CC @thomaspd @cmungall @ukemi @vanaukenk

I should clarify - this is mostly upper level changes.  There are a few more low level ones that should probably be in other branches.  For now, it would be great if you could review the new upper level terms and changes to new upper level existing defs & comment here.  Please don't merge yet.

@thomaspd @pgaudet
Some feedback:

GO:0060089  name: molecular transducer activity
Update definition to: A compound molecular function in which an effector function is controlled by one or more regulatory functions.  (*was: components).


Changing product name

The documentation for buildProduct (http://akhikhl.github.io/gretty-doc/Product-generation.html)
Specifies that the product will be built to build/output/${project.name}.
The archive file when running archiveProduct creates a $project.name-$version zip.
It would be nice to be able to name the product that way as well.
The use case is that I currently download the latest zip and ln the product directory after unzipping it.
It would be nice if that directory was versioned as well so that it doesn't overwrite previous installs (for easier versioning/rolling back

This feature is available in my fork by adding includeVersion property:
product {
includeVersion = true
}

@saladinkzn provided the solution for this in Gretty 1.2.6. I am going to deliver Gretty 1.4.0, that will contain this solution as well.


Dev 1.1.0 datasource

In order to make it more convenient for users to manage data sources, we built a unified datasource module with the help of WeBank.These functions are included in the module：

DataSourceManager module.  DataSourceManager is the core module of data sources, which is used to manage all kinds of data sources in the big data platform, mainly including the addition, deletion, modification and release of data sources.Version management and other  functions;  #723
MetaManager module is used for docking metadata of various data sources in the big data platform, mainly including data source connection testing, metadata query and other functions; #724
The DataSource Client module contains the call logic of the Client side, so that users can operate data sources and obtain relevant metadata in the way of the Client side; #725
Added Kafka data source module, its role is to unify the management of Kafka data sources, Kafka is usually used for streaming computing applications; #726

pull code in the past
1.linkis_dml.sql legacy (linkis_configuration_config_key,linkis_configuration_config_value,linkis_manager_label) relevant information
2.linkis-start-all.sh startup script ext/linkis-common-start all datasources do have Spring information

closed for 1.1.0


Deny Rust warnings on CI

Fixes #816.
That issue was open for a while and didn't receive any negative feedback, so I don't expect this PR to be controversial. Thus, I'll merge it after I've tested that it works.

Coverage increased (+0.03%) to 57.081% when pulling 7d29e3ef4ec059793c75300fd60b9a4ce7c0bd62 on feature/deny-rust-warnings-on-ci into fcf92ba345db6110b78630b2eee86503e68100da on master.

Yay, it works as it should!

Now just waiting for Travis to build this again, and then I'll merge this.


Add additional signal reexports

This adds the following signals to the libc reexports:

Signal
Description

SIGTSTP
Stop typed at terminal

SIGPWR
Power failure (System V)

SIGTTIN
Terminal input for background process

SIGTTOU
Terminal output for background process

SIGURG
Urgent condition on socket (4.2BSD)

SIGVTALRM
Virtual alarm clock (4.2BSD)

SIGXCPU
CPU time limit exceeded (4.2BSD)

SIGXFSZ
File size limit exceeded (4.2BSD)

These are all the signals that are defined in libc which are not
aliases to other existing signals. The main useful one here is SIGTSTP
for handling ^Z, but adding all missing signals seems sensible.
The only other signal documented which I have not added is SIGEMT
(Emulator trap), since it is not exported by libc.

Hello
Can you provide the motivation why these should be re-exported? I'm not really much of a fan of re-exporting in general and it kind of clutters the docs and API surface to have them all there. Note that as these are just constants, so you can always use them directly from libc and everything will just work.
Are you 100% sure these are available on all reasonable platforms? I'm afraid the CI does not (yet) cover everything.

Can you provide the motivation why these should be re-exported? I'm not really much of a fan of re-exporting in general and it kind of clutters the docs and API surface to have them all there. Note that as these are just constants, so you can always use them directly from libc and everything will just work.

I use them from libc right now, but if the argument is that you don't need to reexport them, then signal-hook probably shouldn't reexport anything?

Are you 100% sure these are available on all reasonable platforms? I'm afraid the CI does not (yet) cover everything.

I am not. But of the signals I have added only SIGPWR is not part of the POSIX standard. I've considered only adding the missing POSIX signals, but some of the existing signals already aren't part of that (see SIGWINCH/SIGIO). I suppose SIGPWR is the only non-POSIX signal that is from System V though.

I use them from libc right now, but if the argument is that you don't need to reexport them, then signal-hook probably shouldn't reexport anything?

That actually was my original stance. But I got swayed by users asking for them for comfort, so they don't have to put both signal-hook and libc into Cargo.toml and the current state is a bit of a compromise between having the common one there and cluttering the docs and API with every signal nobody ever used.
So mostly I'm asking if you think these would be used commonly enough to be included and the additional comfort for some people would outweight the discomfort of others that have to scroll through them in docs. That SIGTSP seems like a good candidate, but I do have some doubts about eg. SIGXFSZ ever being used in practice.

I am not.

Give me a while, I have an idea how to add bunch of lightweight checks for a lot of platforms that I think are supported, now that I've migrated to github actions recently. Then we can rebase it on top of that and see.
As for them being POSIX… I'm specifically worried about android systems. That one doesn't seem to be fully posix compliant and there already are few hacks inside the code to deal with it.

So mostly I'm asking if you think these would be used commonly enough to be included and the additional comfort for some people would outweight the discomfort of others that have to scroll through them in docs. That SIGTSP seems like a good candidate, but I do have some doubts about eg. SIGXFSZ ever being used in practice.

The only reason for this PR is SIGTSTP, I've not used any of the others and have no need to do so in the future. But effectively this PR will eliminate the need for any future PR doing the exact same thing again. If there are reexports, I'd say one might as well reexport everything? Leaving out just one or two seems somewhat strange?

As for them being POSIX… I'm specifically worried about android systems. That one doesn't seem to be fully posix compliant and there already are few hacks inside the code to deal with it.

I have absolutely no clue at all about Android, so I can't help you there.

Leaving out just one or two seems somewhat strange?

That's true. But then, there are two questions:

Should the crate be moving in the direction of covering them all, or removing the ones that already seem like not really that useful. I certainly missed the opportunity when releasing 0.2 :-|.
What is, authoritatively, the complete list of signals on each platform? Not even the libc seems to have a good answer to that.

Anyway, do you think you could rebase this onto current master? I've added bunch of targets to be checked on the CI. If some of the signals are not supported on many of them, we somewhat easier life deciding about them.

Should the crate be moving in the direction of covering them all, or removing the ones that already seem like not really that useful. I certainly missed the opportunity when releasing 0.2 :-|.

Given how few signals these are and how all of them are just reexports from libc, I think the only thing that makes sense is to support all of them or none of them. Supporting only the "important" ones is arbitrary (who decides what is important?) and not beneficial (reexporting 1 signal vs 10 doesn't really make things more difficult).

What is, authoritatively, the complete list of signals on each platform? Not even the libc seems to have a good answer to that.

I'm not sure, all I've done is read man 7 signal and added every single signal that is in there which is exported by libc and not an alias for an existing signal. But this should definitely be a "complete" (except for SIGEMT) list of signals for Linux at least. If there are some BSD signals missing, someone can probably send the same PR for BSD? But I'd honestly assume that this PR covers most signals on Unix that are in use.

Anyway, do you think you could rebase this onto current master? I've added bunch of targets to be checked on the CI. If some of the signals are not supported on many of them, we somewhat easier life deciding about them.

I mean I can't tell you what to do. Personally I'm fine with just adding SIGTSTP. But assuming this works on all platforms, I think it makes the most sense to add all of them. That way there are no arbitrary decisions, either use libc for everything or use signal_hook for everything, no need to mix and match (which is basically the worst solution possible).

When I'm concidering #69 that might mandate some restructuring and therefore release of 0.3, eventually, I think I'm getting persuaded, in a way. I'd aim to eventually re-export all of them, but move them to a submodule, not to root.
I'll postpone merging this until I decide if I want to go about it by starting to throw breaking changes in, or if I want to start migrating through series of #[doc(hidden)] and #[deprecated] attributes. But I'll get back to it in few days.


Support 32-bit and 128-bit service data UUID (sandeepmistry/noble#625)

Take a look on #625 for more info..

does this need a test?

@jrobeson I think it's ok like this. We are still working on unit tests with @jacobrosenthal on a few branches.


Make test event log compression codec configurable

Switch to zstd for event log compression
Make compression codec configurable

Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

Thanks for the idea @tgravescs

build


Divide by zero in ParseDsdiffHeaderConfig()

divzero.wav.zip - Contains fuzzed input
Running  cli/wavpack divzero.wav with divzero.wav extracted from the attachment triggers a SIGFPE due to divide by zero in ParseDsdiffHeaderConfig (dsdiff.c:282).

This is fixed now, thanks!


Change Methods to Properties(Add new feature)

Properties Idea From
Balance
Gets or sets the balance left/right: -1 is 100% left : 0% right, 1 is 100% right : 0% left, 0 is equal volume left/right.
CanSeek
Gets a value indicating whether the position of the loaded audio file can be updated.
Volume
Gets or sets the playback volume 0 to 1 where 0 is no-sound and 1 is full volume.
Loop
Gets or sets whether the player will continuously repeat the currently playing sound.
Original Method
Task SetMuted(bool value);
Task SetVolume(int value);

#5


Fix/simplify evm amm 2

Simplifies the GatewayEVM class and amm-arb strategy by removing:

Allowances check before running strategy
Auto-approve in Gateway-related strategy strategy
GatewayPriceShim feature
EDIT: Updates the Connector Tiers for Epoch 4 polls

The unused clob Gateway connector is removed since clob_spot and clob_perp replace it.
Tips for QA

Test AMM-ARB and UniswapLP strategies
Test with approved and non-approved tokens
Non-approved tokens should throw error when trading

I will fix the failing tests, but this should be ready for testing

UniswapLP strategy - instance crashed after start
Steps to reproduce:
Create a uniswapLP strategy
Start the strategy
Actual:
Bot crashed, exit/stop strategy are not available
log file continue to grow size (to 5,6 GB when instance killed)
Expected:
Bot start a strategy and adding liquidity to the Pool
logs_conf_uniswap_v3_lp_1_filepart.log
2023-04-21 09:07:35,700 - 2147467 - hummingbot.client.hummingbot_application - INFO - Creating the clock with tick size: 1.0
2023-04-21 09:07:35,704 - 2147467 - hummingbot.client.hummingbot_application - INFO - start command initiated.
2023-04-21 09:07:35,704 - 2147467 - hummingbot.client.hummingbot_application - ERROR - 'GatewayEVMAMMLP' object has no attribute '_all_spenders'
Traceback (most recent call last):
  File "/home/ubuntu/simplegw6242/hummingbot/client/command/start_command.py", line 188, in start_market_making
    await self.wait_till_ready(self.kill_switch.start)
  File "/home/ubuntu/simplegw6242/hummingbot/client/command/start_command.py", line 37, in wait_till_ready
    all_ready = all([market.ready for market in self.markets.values()])
  File "/home/ubuntu/simplegw6242/hummingbot/client/command/start_command.py", line 37, in <listcomp>
    all_ready = all([market.ready for market in self.markets.values()])
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 862, in ready
    return all(self.status_dict.values())
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 875, in status_dict
    "allowances": self.has_allowances() if self._trading_required else True,
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 868, in has_allowances
    return ((len(self._allowances.values()) == len(self._tokens) * len(self._all_spenders)) and
AttributeError: 'GatewayEVMAMMLP' object has no attribute '_all_spenders'
2023-04-21 09:07:35,743 - 2147467 - hummingbot.client.hummingbot_application - ERROR - start_trade_monitor failed.
Traceback (most recent call last):
  File "/home/ubuntu/simplegw6242/hummingbot/client/ui/interface_utils.py", line 68, in start_trade_monitor
    if all(market.ready for market in hb.markets.values()):
  File "/home/ubuntu/simplegw6242/hummingbot/client/ui/interface_utils.py", line 68, in <genexpr>
    if all(market.ready for market in hb.markets.values()):
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 862, in ready
    return all(self.status_dict.values())
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 875, in status_dict
    "allowances": self.has_allowances() if self._trading_required else True,
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 868, in has_allowances
    return ((len(self._allowances.values()) == len(self._tokens) * len(self._all_spenders)) and
AttributeError: 'GatewayEVMAMMLP' object has no attribute '_all_spenders'
2023-04-21 09:07:35,744 - 2147467 - hummingbot.client.hummingbot_application - ERROR - start_trade_monitor failed.
Traceback (most recent call last):
  File "/home/ubuntu/simplegw6242/hummingbot/client/ui/interface_utils.py", line 68, in start_trade_monitor
    if all(market.ready for market in hb.markets.values()):
  File "/home/ubuntu/simplegw6242/hummingbot/client/ui/interface_utils.py", line 68, in <genexpr>
    if all(market.ready for market in hb.markets.values()):
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 862, in ready
    return all(self.status_dict.values())
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 875, in status_dict
    "allowances": self.has_allowances() if self._trading_required else True,
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 868, in has_allowances
    return ((len(self._allowances.values()) == len(self._tokens) * len(self._all_spenders)) and
AttributeError: 'GatewayEVMAMMLP' object has no attribute '_all_spenders'

amm_arb:
Bot place order only for connector_1 side (not connector2)
Steps to reproduce:

Create strategy using pangolin_avalanche + CEX or Paper connector (or uniswap_arbitrum_one and CEX or Paper connector)
Set up low min_profitability (-1 to -10) to get trades
Start the strategy

Actual:
Bot place order for connector_1 side only (same thing for uniswap_arbitrum_one)
Expected:
For each cycle bot place order for connector_1 then for connector_2
pangowaaxfiles.zip
logs_pangowavax.log
files.zip

commit ce06729
Steps to reproduce:
Create a uniswapLP strategy
Start the strategy
Actual:
Bot failed to start uniswapLP strategy
Expected:
Bot start a strategy and adding liquidity to the Pool
logs_conf_uniswap_v3_lp_2.log
2023-04-21 15:28:08,859 - 2161407 - hummingbot.client.hummingbot_application - INFO - start command initiated.
2023-04-21 15:28:08,919 - 2161407 - GatewayEVMAMMLP - INFO - Network status has changed to NetworkStatus.CONNECTED. Starting networking...
2023-04-21 15:28:09,001 - 2161407 - hummingbot.strategy.uniswap_v3_lp.uniswap_v3_lp - WARNING - uniswapLP_ethereum_arbitrum_one connector is not ready. Please wait...
2023-04-21 15:28:10,001 - 2161407 - hummingbot.strategy.uniswap_v3_lp.uniswap_v3_lp - WARNING - uniswapLP_ethereum_arbitrum_one connector is not ready. Please wait...
2023-04-21 15:28:11,000 - 2161407 - hummingbot.strategy.uniswap_v3_lp.uniswap_v3_lp - WARNING - uniswapLP_ethereum_arbitrum_one connector is not ready. Please wait...
2023-04-21 15:28:12,002 - 2161407 - hummingbot.strategy.uniswap_v3_lp.uniswap_v3_lp - WARNING - uniswapLP_ethereum_arbitrum_one connector is not ready. Please wait...
2023-04-21 15:28:12,004 - 2161407 - GatewayEVMAMMLP - ERROR - 'GatewayEVMAMMLP' object has no attribute 'amm_lp_orders'
Traceback (most recent call last):
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 934, in _status_polling_loop
    self.update_order_status(self.amm_lp_orders),
AttributeError: 'GatewayEVMAMMLP' object has no attribute 'amm_lp_orders'
2023-04-21 15:28:13,001 - 2161407 - hummingbot.strategy.uniswap_v3_lp.uniswap_v3_lp - WARNING - uniswapLP_ethereum_arbitrum_one connector is not ready. Please wait...
2023-04-21 15:28:13,001 - 2161407 - GatewayEVMAMMLP - ERROR - 'GatewayEVMAMMLP' object has no attribute 'amm_lp_orders'
Traceback (most recent call last):
  File "/home/ubuntu/simplegw6242/hummingbot/connector/gateway/amm/gateway_evm_amm_lp.py", line 934, in _status_polling_loop
    self.update_order_status(self.amm_lp_orders),
AttributeError: 'GatewayEVMAMMLP' object has no attribute 'amm_lp_orders'

For the AMM-ARB error, based on these logs_pangowavax.log, it looks like the bot thought the Avalanche BUY transaction was lost before placing the order for some reason:

2023-04-21 13:02:45,277 - 2157190 - hummingbot.strategy.amm_arb.amm_arb - INFO - Placing BUY order for 0.200000 WAVAX at pangolin_avalanche_avalanche at 18.165663050000000 price [clock=2023-04-21 13:02:45+00:00]

2023-04-21 13:02:45,278 - 2157190 - hummingbot.strategy.amm_arb.amm_arb - ERROR - Order buy-WAVAX-USDC-1682082165278130 seems to have failed in this arbitrage opportunity. Dropping Arbitrage Proposal.  [clock=2023-04-21 13:02:45+00:00]

2023-04-21 13:02:47,707 - 2157190 - hummingbot.connector.client_order_tracker - INFO - Created LIMIT BUY order buy-WAVAX-USDC-1682082165278130 for 0.200000 WAVAX-USDC.

I couldn't replicate this on my side with Sushiswap Avalanche and Kucoin Paper Trade, but will try Pangolin now.

@nikspz I was able to replicate the AMM-ARB error on development as well. I believe it occurs when there's an Gateway order takes too long and it's considered lost. When the user runs start again, the lost order in the DB prevents the strategy from executing more arbs, because the first order is considered failed as soon as it is sent.
I'll look at the UniswapLP bug, but I don't think the AMM-ARB bug is causes by this PR.

logs_conf_amm_arb_4.log

PR update:

Tested on local WSL Ubuntu20.04
Test AMM-ARB

Setup test using traderjoe-avalanche and pancakeswap-bsc
Check wallet balance using gateway connector-tokens ✅
Starting the strategy does not require any prompt ✅
Check profitability rate on each connector for ~30mins

if connector_1 is DEX and connector_2 is CEX ✅
if connector_2 is  CEX and connector_2 is DEX ✅

Test arbitrage opportunity

Does not do arbitrage if either if balance is insufficient ✅
Concurrent_orders:

If no, does not proceed with 2nd leg
If yes, does not wait for 1st leg to be completed.

Getting trade failed error
2023-04-24 12:29:23,142 - 9799 - GatewayEVMAMM - ERROR - Error submitting SELL swap order to traderjoe on avalanche for 0.700000 WAVAX-USDC 16.800273000000000.
Traceback (most recent call last):
  File "/home/rapcmia/github/hummingbot/6242/hummingbot/connector/gateway/amm/gateway_evm_amm.py", line 442, in _create_order
    order_result: Dict[str, Any] = await self._get_gateway_instance().amm_trade(
  File "/home/rapcmia/github/hummingbot/6242/hummingbot/core/gateway/gateway_http_client.py", line 493, in amm_trade
    return await self.api_request("post", "amm/trade", request_payload)
  File "/home/rapcmia/github/hummingbot/6242/hummingbot/core/gateway/gateway_http_client.py", line 218, in api_request
    raise e
  File "/home/rapcmia/github/hummingbot/6242/hummingbot/core/gateway/gateway_http_client.py", line 206, in api_request
    raise ValueError(f"Error on {method.upper()} {url} Error: {parsed_response}")
    ValueError: Error on POST https://localhost:15888/amm/trade Error: {'message': 'Swap price 16.80027286 lower than limitPrice 16.800273000000000', 'httpErrorCode': 500, 'errorCode': 1009, 'stack': 'Error: Swap price 16.80027286 lower than limitPrice 16.800273000000000\n    at /home/rapcmia/github/gateway/gateway-development/dist/src/connectors/uniswap/uniswap.controllers.js:163:23\n    at Generator.next (<anonymous>)\n    at fulfilled (/home/rapcmia/github/gateway/gateway-development/dist/src/connectors/uniswap/uniswap.controllers.js:5:58)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)'

On this scenario, the bot has an exact amount of asset to create order yet its not possible for this PR with an error of 'Swap price 16.80027286 lower than limitPrice 16.800273000000000'
- Crosscheck with development branch using the same config (attached) and able to complete arbitrage 💯

When the event that our wallet does not have enough to trade, it return an a trade failed error which is expected due to insufficient funds ✅

Test with approved and non-approved tokens

Assume to best test it with new wallet without assets and check client behavior

Setup wallet on pangolin-avalanche and uniswap-arbitrum and noticed that it tries to check for arbitrage opportunity but no display
When approving using gateway approve observed the expected behavior and error message below ✅:

Crosscheck with development branch and the behavior not the same, client would display ...waiting for allownce and manual approval

Steps to reproduce reported issues:

Setup this PR with latest gateway development branch
Setup also a development client for crosscheck
Connect preferred SPOT AMM DEX connector
Observed behavior for:

Checking allowances and approval or gateway approve command
Profitability rate and trade execution (txhash on chain explorer)
Check traceback on trade failed error and crosscheck with development branch
Setup a new wallet (0 balance) and redo steps above

6242.zip
Pending:

Non-approved tokens should throw error when trading (WIP)

Closing this PR and breaking it up into smaller PRs


style: update search suggestion

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #7078
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[ ] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Ecosystem Telegram?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at c6d3022
Improved UI and UX of SearchSuggestion.vue component. Hid empty search history, fixed trending tab layout, and added vertical space for search result items.

🤖 Generated by Copilot at c6d3022

The search suggestion component
Got a UI improvement
No more empty div
Better padding and alignment
And some vertical space for each element

pay 10 usd

😍 Perfect, I’ve sent the payout
💵 $10 @ 4.25 USD/DOT ~ 2.353 $DOT
🧗 13rv1SWoLg9Gb3tmvHPZxb7JbVy51BtMziX7k9WQGSJ7Kp3A
🔗 0x75649f02978da7c137785be2c1d211ffc3e0d82f5e855c515cef392680f028de
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Add Seq Module

What does this PR do?
New module to support the Seq logging server.
Why is it important?
Another module to make it easy for developers needing Seq to get testing quickly.
How to test this PR
Automated test added to demonstrate the Seq server accepts logs and that those logs can be read via the API.

Very glad you have PRed this in it was on my list of ones to do if someone hadn't :)

👋🏻 G'Day @montanehamilton ,
question: what is the use case for having Seq in a TC? I use Seq all the time .. love it! but ... I've never seen a use for it as a test container?
I'm really curious to learn why.

Logging is a requirement and ensuring applications can actually log and that those logs actually get to the logging provider

Hmm .. ok. So if I'm reading this correctly, you're testing that your logging configuration (to seq) is correct? Or you're testing either the MS-Logging to Serilog-Logging method you're using to log .. works?


Undefined namespace errors are misleading

When trying to open a sub-namespace that doesn't exist, the error message blames a parent namespace that does exist.
Repro steps
open System.Data
open System.Data.Common
open System.Data.Blah
open System.Data.Blah.Blah

Expected behavior
The error message on the third and fourth lines should be "The namespace or module 'Blah' is not defined."
Actual behavior
The error message is "The namespace 'Data' is not defined." even though the first two lines works fine.
Known workarounds
Check to make sure it's the sub-namespace that's missing and not the parent namespace as is indicated, by opening the parent namespace separately.
Related information
Although this in itself is a minor issue, there appears to be a systemic lack of meaningful error messages lately throughout the entire .NET stack. There used to be guidelines on handling errors and exceptions at Microsoft, but they don't seem to be adhered to anymore. I don't know where else to get in contact with devs, so I'm submitting this particular issue. I could report at least five more similar frustrations from today alone.
It's becoming extremely unproductive having to Google the meaning of every single unhandled exception and unhelpful error message. Half the time I land on a Github issue saying "This isn't a bug, you did XYZ wrong." Yes, the code itself is indeed working as designed, but if I'm doing something wrong, I need more contextualized information than a NullReferenceException from somewhere deep in the call stack, for example.
Sorry for the rant; if there's someone more appropriate to direct this message to, please let me know.
F# 4.1 on .NET Framework, Windows 10, in both Visual Studio and VS Code

I've noticed this error too


Issue/slow tests compiler

Description
I attempted to speed up the compiler tests
Performance trace prior to optimization:

20% time spent on module loading -> reduced reliance on std
29% of time spent verifying python dependencies -> patched it out of the compiler tests (caching was too complicated to be worth the effort, I tried) -> down to 8.28%

On master, prior to change, compiler tests take 191 on this branch 116, 75s time reduction
Performance graph before

Performance graph after

it does make the test a bit more verbose
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in e40588211f44f022d6c9afedb3cb9b811b223574

Processing #8070.

Processing #8070.

Processing #8070.


Rename Modelica.Electrical.Machines.Examples.ControlledDrives.Components

3fd3d6a736386e9c59a13c61559b38293d62d929 added Modelica.Electrical.Machines.Examples.ControlledDrives.Components. Should it be renamed to Modelica.Electrical.Machines.Examples.ControlledDrives.Utilities in order to match #2429?
@AHaumer It would be wise to first open a pull request for such large portions of proposed WIP changes. This makes it easier for other library developers to check, evaluate and finally approve your changes. Furthermore, there will be an issue number that can be referred to in the Release Notes.

Resolved by 406b3b1f2edd9438157af167c5a14f4f8a6f515a.


Build support for Java language in DBB

Hi,
We need to set up a build chain for the Java language on z/OS.
Unless I'm mistaken, Java language support is not implemented in DBB, in particular to manage artifacts dependencies.
Is there any work planned for Java language support in DBB?
Note that DBB is a Java application for z/OS environment. So the DBB team should have experience on the subject?
Thanks.

Relate to Build support for Java language in zAppbuild


Add provider 0.4.6 release note

Summary
Add release note for qiskit-ibmq-provider 0.4.6 (Qiskit 0.15.0)
Details and comments

Shouldn't this have been 0.5.0...

In retrospect it probably should have been. The changes were all very minor so I just went with a patch release.

That being said could you do 2 things before we merge this, switch the function and method references to links using the sphinx syntax and the pr numbers to links?

I cleaned things up a bit and added the reference links. I removed the PR numbers though since they were not used in the terra section.


Shows a warning if a user is linked to a motion as submitter or an election as candidate or anything else

Implements the "get_user_related_models"-presenter
Also implements a "PresenterService" for presenters

When a user should be deleted, then a dialog is appearing which shows to which meetings and committees the user is linked to. If the user is linked to something as submitter or candidate (...), their account is only deactivated. If a requesting user (within a meeting) has not the necessary permissions to perform this action, the user will be removed from the meeting instead of being deleted.
Fixes #495

I have also updated the orga-settings, so the key limit_of_users is requested


Profile cache level

What steps will reproduce the problem?
Process connect to several cache systems. For performance purposes programmers need to know how effectively use cache.
What is the expected result?
In debug panel: performance, time to do query.
What do you get instead?
Nothing in debug panel if I use memcache.
Additional info

Q
A

Yii version
2.0.9

PHP version
7.0.12

Operating system
Debian 8, Docker

https://github.com/yiisoft/yii2/pull/12911

Move to the cache package?

@rustamwin can't we measure time based on data collected via container proxy calls? https://github.com/yiisoft/yii-debug/blob/master/src/Proxy/ProxyLogTrait.php

Yes, of course.


Added tests

not sure if it covers everything, but its something

Tests passed and works OK. Thank you!


Create diderot.txt

Institution/School Name
Lycée Polyvalent Diderot
Instiution/School Website
http://www.diderot.org/
Email Users
[ x] Students
[ x] Academic/Teaching Staff
Other/Support Staff
Alumni
Education Levels
Please place an x between the square brackets if yes
Post-graduate research
Graduate School
[x ] College (University) or Undergraduate School or equivalent
Community College or equivalent
[ x] High School or equivalent
Page showing a >1 year formation :
http://www.diderot.org/index.php/l-enseignement-superieur/bts-sn-ir

@abrajeux Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!

Thank you @philipto ! Do you have any idea of how many time this will take to take effect ?


MRG: Simpler coil defs

This PR:

Removes the unnecessary X and Y point mag coil defs by using a simple rotation of the local coordinate system combined with the standard Z point mag def.
Simplifies the coil def machinery by making use of the new use_coil_def.
Makes it so that maxwell_filter uses the same coil defs as the rest of MNE. In some of the tests we still use use_coil_def(elekta_def_fname) just to cross-check high compat when using identical defs; in others, we don't bother (to check that processing is working properly with MNE defs). Some of the "check vs. MaxFilter" test tolerances thus needed to be lowered, but critically none of the shielding / data quality ones did, telling us that things are still working.

No need for whats_new.rst update as this should be transparent to users.

thx @larsoner


Implement suggestions from 25/11/16

Bug: If the header is 'emails' rather than 'email', the import CSV function fails silently on the client side.
Bug: Emails no longer include the sender's name (in the sense of 'Bob<EMAIL_ADDRESS>Buttons should be adjusted for conformity in the following manner. Buttons indicating success/progression should be placed on the left, and a consistent colour scheme needs to be in place. Sizing also needs to be larger and consistent.
Importing a CSV needs to be improved. At present, it's not clear that the CSV is actually uploading (in the sense that the file is being sent from the client to the server, not internally from a CSV to the db).
A 'test' functionality should be implemented that allows users to send out a quick test email from within a 'send campaign' view. This would allow them to assure themselves of the final email's rendering.
Change 'Preview' button text to 'Next Step'

Done:

Bug: If the header is 'emails' rather than 'email', the import CSV function fails silently on the client side.
Bug: Emails no longer include the sender's name (in the sense of 'Bob<EMAIL_ADDRESS>Change 'Preview' button text to 'Next Step'

Remaining:

Buttons should be adjusted for conformity in the following manner. Buttons indicating success/progression should be placed on the left, and a consistent colour scheme needs to be in place. Sizing also needs to be larger and consistent.
Importing a CSV needs to be improved. At present, it's not clear that the CSV is actually uploading (in the sense that the file is being sent from the client to the server, not internally from a CSV to the db).
A 'test' functionality should be implemented that allows users to send out a quick test email from within a 'send campaign' view. This would allow them to assure themselves of the final email's rendering.

Done:

Buttons should be adjusted for conformity in the following manner. Buttons indicating success/progression should be placed on the left, and a consistent colour scheme needs to be in place. Sizing also needs to be larger and consistent.
Importing a CSV needs to be improved. At present, it's not clear that the CSV is actually uploading (in the sense that the file is being sent from the client to the server, not internally from a CSV to the db).

Remaining:

A 'test' functionality should be implemented that allows users to send out a quick test email from within a 'send campaign' view. This would allow them to assure themselves of the final email's rendering.

Done:

A 'test' functionality should be implemented that allows users to send out a quick test email from within a 'send campaign' view. This would allow them to assure themselves of the final email's rendering.


Add support for Kafka 0.10+

Please add support for Kafka 0.10+

I'm using it with Kafka <IP_ADDRESS> with no major problems.

The wiki says as follows:
Create a topic with optional topic configs (<IP_ADDRESS> has different configs than 0.8.2+)
Delete topic (only supported on 0.8.2+ and remember set delete.topic.enable=true in broker config)
Topic list now indicates topics marked for deletion (only supported on 0.8.2+)
Do these features work well with kafka 0.10?

Has anyone enabled SSL inter broker communication / SASL_SSL auth with kafka in 0.10, and then tried to use kafka manager to manage the cluster?
It doesn't seem to work for me. Is this something which needs adding?

@KaimingWan I can state that topic deletion and marked for deletion works for Kafka <IP_ADDRESS>.

@falmp I have some problem in compiling kafka-manager. Would you mind sending me the code after compiling with sbt? My email is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Thanks!

@KaimingWan What kind of problem? Anyway, sent you the link.

@falmp I have solved the problem. Thanks anyway~

Hi all!
Any ETA Kafka 0.10+ ? :)

@freemanlutsk There is currently a pull request open to add support for 0.10 here https://github.com/yahoo/kafka-manager/pull/282

Appreciate the 0.10 support. I see a consumer.properties file in conf/ where the security.protocol is defaulted to PlainText. Due to HIPAA constraints we are only allowed to only open SSL listeners on our brokers, but it doesn't appear that trying to assign the security.protocol=SSL and attempting to assign the Keystore (ssl.keystore.location) and TrustStore (ssl.truststore.location) can work. Actually, when I trace the properties back to KafkaStateActor.scala it appears to only be concerned with the ConsumerConfig and not the CommonConfig.SSLConfig.
Granted I might be missing something, but this is the only passable UI for Kafka Management, and I don't want to give it up for security reasons.

@benMain this is work in progress, we are in process off adding SSL to our clusters and then we'll update KM

what version of SSL supports Kafka-Manager? How would you provide ssl.truststore information to KM? We can only use SSL on our Kafka cluster for security reasons.

Great Tool. any timeline on supporting SASL_SSL?

SSl part is really ambigous, i just succeed to get full information on full plaintext only


Remove Zimbabwe and direct links to Apply for QTS service from Teach in England

Remove Zimbabwe and direct links to service from Teach in England
Trello card
Context
Changes proposed in this pull request
Guidance to review

Hi @MaxineCl, welcome back...
This update is due to go live week commencing 8th Jan. I'll be back on 8th but if you had a chance to review between now and then, that would be fab. Thank you!


Bug in InteractionManager.prototype.processInteractive

I´m getting an error when having a mousedown on a button and on mousedown removing various displayObjects
Uncaught TypeError: Cannot read property 'visible' of undefined
It seems pixi is still looping through nulled(?) displayObjects.
So I suggest changing "if( !displayObject.visible)" to "if(!displayObject || !displayObject.visible)" here:
InteractionManager.prototype.processInteractive = function (point, displayObject, func, hitTest, interactive )
{
    if(!displayObject || !displayObject.visible)
    {
        return false;
    }

Here´s a testcase:
http://jsfiddle.net/Lsfkj1ng/7/

Happened to me this morning too !

I'm getting this error also in v3.0.6.

I am also having this problem. I set my texts into interactive mode and it seems to me, this only happen on object with interactivity on.
I am using PIXI v3.0.5.
Here is the code that I used when I encountered this problem:
// Setup
StartScene.prototype.setup = function()
{
this.mStage = new PIXI.Container();
this.mStyle =
{
font   : 'bold italic 36px Arial'
, fill   : '#F7EDCA'
, stroke : '#4a1850'
, strokeThickness : 5
}
this.mTitleStyle =
{
	  font   : 'bold italic 62px Arial'
	, fill   : '#F7EDCA'
	, stroke : '#4a1850'
	, strokeThickness : 5
}

var centerPos = {x : this.mScreenSize.width / 2.0, y : this.mScreenSize.height / 2.0};

this.mSnakeTitle = new PIXI.Text("Snaked!" , this.mTitleStyle);
this.mSnakeTitle.anchor.x = 0.5;
this.mSnakeTitle.anchor.y = 0.5;
this.mSnakeTitle.position.x = centerPos.x;
this.mSnakeTitle.position.y = centerPos.y - 90.0;

this.mEasyText = new PIXI.Text('Easy' , this.mStyle);
this.mEasyText.anchor.x = 0.5;
this.mEasyText.anchor.y = 0.5;
this.mEasyText.position.x = centerPos.x;
this.mEasyText.position.y = this.mSnakeTitle.y + 70.0;
this.mEasyText.interactive = true;
this.mEasyText.click = (function(e)
	{
		console.log("Easy!");
		this.mOnFinishCallback(Difficulty.Easy);
	}).bind(this);

this.mMediumText = new PIXI.Text('Medium' , this.mStyle);
this.mMediumText.anchor.x = 0.5;
this.mMediumText.anchor.y = 0.5;
this.mMediumText.position.x = centerPos.x;
this.mMediumText.position.y = this.mEasyText.y + 50.0;
this.mMediumText.interactive = true;
this.mMediumText.click = (function(e)
	{
		console.log("Medium!");
		this.mOnFinishCallback(Difficulty.Medium);
	}).bind(this);

this.mHardText = new PIXI.Text('Hard' , this.mStyle);
this.mHardText.anchor.x = 0.5;
this.mHardText.anchor.y = 0.5;
this.mHardText.position.x = centerPos.x;
this.mHardText.position.y = this.mMediumText.y + 50.0;
this.mHardText.interactive = true;
this.mHardText.click = (function(e)
	{
		console.log("Hard!");
		this.mOnFinishCallback(Difficulty.Hard);
	}).bind(this);

this.mStage.addChild(this.mSnakeTitle);
this.mStage.addChild(this.mEasyText);
this.mStage.addChild(this.mMediumText);
this.mStage.addChild(this.mHardText);

this.mRenderer.render(this.mStage);

}
//Callback
this.mOnFinishCallback = function(difficulty)
{
this.mStage.visible = false;
this.mStage.destroy(true);

//new MainGameScene(this.mRenderer , this.mScreenSize , difficulty);

}

Happened to me too !

Hi is there any suggested fix for this? I tried to replicate this with the example provided from the site. my aim is to destroy all the sprite I have on the screen to make way with the new scene.
When you add this line:
stage.destroy(true) or stage.destroy(false)
And you press the button, I am able to get the same error. I believe there is something wrong with what I am doing. That is, there must be another way on how to do screen transitions.

Happened to me too in v3.0.7 when calling
container.removeChildren();

@englercj do you know when the fix for this will be released? We have come across this as well, I can see it is in the dev branch but not master yet.
is is possible to release a patch update before 3.1?

@ahumphreys87 It is, but I'm pretty busy right now with other things. I opened #2071 as a way to try and make time for issues like this, but haven't seen much movement yet.


Order by rank (hasContactInfo + more)

To remove ambiguity, I will add a vivo:rank key to each hasContantInfo entry.  This uses the same setup  as the citation authors.  It's  likely the order will not be sequential, but more a generic rank.  I propose something like the sort callback.  sort((a,b)=>(a.rank?a.rank:100)-(b.rank?b.rank:100))
Seems pretty safe
['foo',{rank:10},{rank:null},{norank:1},{rank:2},"this",{rank:2}].sort((a,b)=>(a.rank?a.rank:100)-(b.rank?b.rank:100))

Best is always array.sort[]; ties mean order doesn't matter.

@qjhart I'm confused. What am I sorting other than the citation authors?

@spelkey-ucd As discussed in #23, most faculty will have multiple hasContactInfo entries.  I imagine that Kimmy expects only one title and organization on the people lists, and for the entries to be ordered on the individual page.  So for sure, you'll need to order the array in hasContactInfo.  If we have other items that ever need ranking, we'll always use the method above.

ohhhhhh gotcha thanks. Sorry,  I misread 23. I thought you were referring to the title property being either array or string.


Turn on Linting for Files

Turn on CheckStyle, SpotBugs, and JavaDoc linting for Azure Storage Files

Rolled up into #4051


changed single verticle bar(|) to or in src/dumps/Windows/win.py by gpu name detection line 128

Thank you for the report.
However, this will soon be replaced with a more sanitised and accurate method of obtaining the Codename of the current CPU, if possible. I will push the changes to experimental as soon as it's finished testing.

Fixed with latest pushes to main.


http: no cipher suite supported by both client and server when container contacting  ingress-endpoint (ingress-sbox)

What did you do?
We are hosting traefik in a docker swarm containing 3 managers. Things are working fine but after a week or so we start getting either of these errors continuously:
http: TLS handshake error from <IP_ADDRESS>:56437: tls: no cipher suite supported by both client and server
2017/07/28 06:18:17 server.go:2753: http: TLS handshake error from <IP_ADDRESS>:44674: tls: unsupported SSLv2 handshake received
2017/07/28 06:18:26 server.go:2753: http: TLS handshake error from <IP_ADDRESS>:48083: tls: client using inappropriate protocol fallback
2017/07/28 06:11:12 server.go:2753: http: TLS handshake error from <IP_ADDRESS>:54382: tls: client offered an unsupported, maximum protocol version of 300
What did you expect to see?
Once I hit the service endpoint, I should see our application JSON response.
What did you see instead?
Once this error starts showing, our service front end starting giving 502 bad gateway. But not always, 3 in 10 times 502 bad gateway occurs.
Output of traefik version: (What version of Traefik are you using?)

Docker version:
Client:
Version:      17.06.0-ce
API version:  1.30
Go version:   go1.8.3
Git commit:   02c1d87
Built:        Fri Jun 23 21:20:36 2017
OS/Arch:      linux/amd64
Server:
Version:      17.06.0-ce
API version:  1.30 (minimum version 1.12)
Go version:   go1.8.3
Git commit:   02c1d87
Built:        Fri Jun 23 21:21:56 2017
OS/Arch:      linux/amd64
Experimental: false
traefik.toml:
traefik.toml
accessLogsFile = "log/access.log"
traefikLogsFile = "log/traefik.log"
logLevel = "INFO"
defaultEntryPoints = ["https"]
[entryPoints]
[entryPoints.http]
address = ":80"
[entryPoints.http.redirect]
entryPoint = "https"
[entryPoints.https]
address = ":443"
[entryPoints.https.tls]
[[entryPoints.https.tls.certificates]]
CertFile = "/etc/traefik/traefik.cert"
KeyFile = "/etc/traefik/traefik.key"
[web]
address = ":8080"
ReadOnly = true
[docker]
endpoint = "unix:///var/run/docker.sock"
domain = "docker.localhost"
watch = true
exposedbydefault = true
swarmmode = true
I can see <IP_ADDRESS> as ingress-endpoint from ingress. Copying the detail of ingress:
The Strange thing is, it was working. Without any change suddenly fails followed by this error.
docker network inspect ingress.
[
{
"Name": "ingress",
"Id": "g91vlveu6s2u9c4tvzqdn2e8y",
"Created": "2017-07-27T10:01:09.111025266Z",
"Scope": "swarm",
"Driver": "overlay",
"EnableIPv6": false,
"IPAM": {
"Driver": "default",
"Options": null,
"Config": [
{
"Subnet": "<IP_ADDRESS>/16",
"Gateway": "<IP_ADDRESS>"
}
]
},
"Internal": false,
"Attachable": false,
"Ingress": true,
"ConfigFrom": {
"Network": ""
},
"ConfigOnly": false,
"Containers": {
"bfdee6bece2b0750796d39533793286f599ad3f38855f4518ac545cf02048473": {
"Name": "traefik.qml805v1p6e35ewm3e6f1qko8.zbyxdt4vawobelqp8pz7lxdp5",
"EndpointID": "6e0e5f3d4c6bede800d6948aaa6da6f6b4df0e51d4e9b43c3fc9be88385e966e",
"MacAddress": "02:42:0a:ff:00:07",
"IPv4Address": "<IP_ADDRESS>/16",
"IPv6Address": ""
},
"ingress-sbox": {
"Name": "ingress-endpoint",
"EndpointID": "10adf1878e093da50c7bfe7dcda2c0c1cdb5271db9b31c49553e4a031b20244b",
"MacAddress": "02:42:0a:ff:00:02",
"IPv4Address": "<IP_ADDRESS>/16",
"IPv6Address": ""
}
},
"Options": {
"com.docker.network.driver.overlay.vxlanid_list": "4096"
},
"Labels": {}
}
]

This issues is making us to reinstall docker environment every now and then.

@AchuthSankar This looks like an issue on Docker Swarm Mode. This repo is for the older project "Standalone Docker Swarm". Please open the issue on https://www.github.com/moby/moby.


Console/Admin session

I'm not really sure if this is a bug or something I'm doing wrong, but how do you connect to the console/admin session, i.e. the current logged-on session? For example using Microsoft RDP app in Android has and "admin" switch to allow connection to the current running session, but this doesn't work with RDP Wrapper. Instead, a new session opens. So is there a way to enable this in RDP Wrapper? Thanks.

Enable single session per user in config.

I actually meant is there a way to access the console session with multiple sessions enabled?

Use tscon console utility (if it doesn't exist, google it), it allows to connect to existing sessions.

Sure but tscon is a Windows program. When RDP-ing using Microsoft's own Android RDP app - which has an "Admin" option (meaning console session) - you still can't connect to the console session, just a new session. There's something with RDPWrap and multiple sessions enabled that somehow isn't "respecting" the Admin/console login request from non-Windows apps including Microsoft's own.

You need to enable "Single Session Per User" option in order to do that. There's no other solution.
If this is disabled, then you'll always get a new session.

Closed as solved. If not - reopen it.


Build Fail

I noticed that the Neural Net Test caused the build to fail. There is nothing wrong in the code, but unfortunately the algorithm is non-deterministic so there is always the chance it will fail, if we are very unlucky.
What should we do about these kinds of tests? I would like to be testing them in case we break something in the implementation, but there are times where this backfires and unluckily the build fails. Another solution I can think of is increasing the threshold for error, but this may also backfire (we make a change that makes the implementation worse, but we don't notice it because it doesn't surpass the threshold).
I think the best way to go is increase the threshold by a little (say, 0.33), at least for now.

For now we could make it so that such tests donot break the build but instead give a warning.
We would need a better solution because we have the same problem with tests for other modules like GA and random forest etc.

I agree with @Chipe1 -- we shouldn't have tests that can occasionally fail. Some remedies, any of which are good:

Use random.seed('something') to get deterministic results.
Widen the acceptance level of the test. For example, rather than insisting it get one test case right, run 100 test cases and test whether it got a certain percentage right.
Have some way of writing a warning, rather than failing the test.

Thanks for the feedback.
I like the idea of generating a lot of random test cases and running those. I will also see about giving the random generator a seed.

Thanks for #534.


Cognitive dissonance in command-line and configuration-file flags

The expectation is that command-line and configuration-file flags will be identical, with perhaps an extra
-- added by convention to command line flags.
In Cayley however we have this:

--db vs database
--dbpath vs db_path
--host vs listen_host (this one bit me hard.)

Is there a particular reason for this?

Fixed in https://github.com/cayleygraph/cayley/pull/492


Telework Headquarter Field Requirement for Pre-February Submitted Forms

Forms submitted before February 1st at noon, when the new requirement was added, do not have data for the Headquarter fields.
When a supervisor receives these forms, they are not able to approve/deny the form as the Headquarter fields are disabled and empty.
We have temporarily set the Headquarter fields to optional on production, to allow approval of all forms.
On test I have created a version where the fields are required for submitters, but not required for approval of the submission.
An alternative solution would be to allow these fields to be editable by the supervisor, but still required.
We should determine which solution is best for the business.

@Andrew-Vargas-bcgov I've discussed this with Sean (Policy) who likes the idea of the supervisor having final edit authority for the HQ field.  We also have had a message in the DJ box from a supervisor wanting to edit as their employee put the wrong address in the field so it works well for a couple of reasons.

@Stella-Archer should I make all of the HQ fields required too then?
This would make it so that submissions before the implementation of the HQ fields will force the supervisor to fill out those fields.
For submissions after HQ implementation, the submitter would be required to fill out the fields, but the supervisor could still edit.

HQ fields are now required for submitter and approver.
For forms submitted when HQ did not exist or was not required, the supervisor will now be required to fill those sections out.
Supervisor now has ability to edit the submitted HQ fields only.
pdf reflects the changes made by supervisors.

Po approved ready to deploy


Some members with Github as Auth Provider could not be shown in cluster member list

Internal reference: SURE-7818
Reported in 2.7.9
Some members (Auth Provider=Github) could not be shown in cluster member list.
The user exists on Github Enterprise and the user exists on the list under Add Cluster Member but not when using the filter string upon typing within the Select Member textbox.

Verified on v2.9-head ID: fd73f ✅

Fresh Install | v2.9-head | Org/Team/User Search
Fresh Install | v2.9-head | Team Access
Fresh Install | v2.9-head | Nested Memberships
Fresh Install | v2.9-head | Enable two (or more) orgs in GH provider
Upgrade | v2.8.4 → v2.9-head | GH Auth Enabled Before Upgrade | Org/Team/User Search
Upgrade | v2.8.4 → v2.9-head | GH Auth Enabled Before Upgrade | Team Access
Upgrade | v2.8.4 → v2.9-head | GH Auth Enabled Before Upgrade | Nested Memberships
Upgrade | v2.8.4 → v2.9-head | Enable two (or more) orgs in GH provider


Deal with LIS hub unavailability in the London GCP cluster

Context
The solution to https://github.com/2i2c-org/infrastructure/issues/1915 was to allow the cluster to create nodes in any of the three zones in the region, rather than just the -b zone. The impact of this is that NFS is now working across multiple zones which will have performance issues. We should move the cluster back to a single zone.
Which zone?
We can do one of the following two:

Restrict the cluster back to the -b zone
Move the cluster to the -a or -c zones

We are using the -b zone now as we thought there was a need for GPUs, but it doesn't seem that the LIS hub is using them right now.
Timescale?
What timescale should we aim to make this move on?
Proposal
No response
Updates and actions
No response

Let's move this back on Dec 1?

Renaming, as we're now considering moving out of London entirely. Matt is checking with his team to see if being inside the UK is a legal requirement.

Can this issue be closed?

I actually don’t think we fixed this


RModal is not defined on website using RequireJS

I have a script which dynamically loads rmodal.min.js on my clients' websites and then tries to show a popup as usual.
It looks somewhat like this:
(function() {
    // this piece loads rmodal.min.js
    var d = document, g = d.createElement('script');
    g.type = 'text/javascript'; g.async = false; g.defer = true; g.src = 'somepath/rmodal.min.js';
    d.head.appendChild(g);

    // this piece waits for the script to load and creates a popup
    g.onload = function() {
        var popup = new RModal(document.getElementById('myPopup'));
        popup.open();
    };
}();

Usually it works very well. However, I encountered a website that uses RequireJS v2.1.2 library. RModal detects this and tries to load itself using "define" function. Obviously it fails somehow, since then I get an JS error when trying to create a popup (or just check for the RModal in the console):
ReferenceError: RModal is not defined

The method of loading the script is not an issue, it might as well be static

If you're using AMD on your website you need to import RModal using AMD itself. The exports are generated in such a way that an AMD export is used if found.

If the website is using RequireJS then you need to import RModal using RequireJS itself. The library supports such exports, so you should have no issues if you import it using the already existing module loading mechanism on the website.

Thanks, that's what I did in the end. I detect whether RequireJS is available and use it to load RModal. If not, the default method is used. It looks somewhat like this:
if (typeof define === 'function' && define.amd) {
    // fix for websites using requirejs
    require(['somepath/rmodal.min.js'], function(rmodal) {
        window.RModal = rmodal;
        buildPopup();
    });
} else {
    // regular script loading method
    var d = document, g = d.createElement('script');
    g.type = 'text/javascript'; g.async = false; g.defer = true; g.src = 'somepath/rmodal.min.js';
    d.head.appendChild(g);
    g.onload = buildPopup;
}

It just took me a while since I've never encountered RequireJS before and docs for RModal only mention it is supported, not how to use it in such environment.
Eh, writing panzercode is not easy ;)


Recovery link for contact

create temp channel and create link (Primary device)
Put data to temp channel (Keeper contact)
Download it from temp channel and delete channel from keeper side (Primary device)

We have done away with level based backup


Real-time non-blocking mesh visualization

Thanks for your amazing work.
I am intending to display the 3D mesh reconstruction results in real time.
But every time, the program will be ended after showing the mesh in first time. I have tried ioff(), interactive=False but they didn't work.
Here is a example, I intend to visualize a different mesh (with the same face).
faces = np.array([[0,1,2], [1,2,3], [2,3,4]])
verts = [vert1, vert2, vert3]
for vertex in verts:
    mesh=Mesh([vertex,faces])
    show(mesh,axis=1,interactive=0)

The program will be stopped after running into show at the first time.

Thanks for your quick reply!
I have learnt the example
https://github.com/marcomusy/vedo/blob/master/examples/simulations/airplanes.py
There are some difference between the goals.
In examples/simulations/airplanes.py, only the pose of mesh is updated during the simulation.
While in my real-time application, I have to estimate the human 3D mesh from webcam. Here is my project:
https://github.com/Arthur151/ROMP
Therefore, I need to update the vertices in the loop. Is there a possible way of updating the vertices of the pre-defined mesh?

Thanks! I have fixed this problem. Your vedo helps a lot!  I will commit a new version of ROMP using vedo for visualization.

Yes, check out examples/basic/mesh_modify.py. This is only if you need to modify the vertex positions. Note that the global position of the object will not change (the modified obj will remain at nominal [0,0,0]).
If you also need to modify faces then the only way is to create a brand new mesh.
Very cool project btw!

Here is an issue about the texture.
It is really wierd that there is a blank gap between different area of UV map.

from vedo import *
import numpy as np

faces=np.load('face.npy')
verts=np.load('vertices.npy')
uvmap=np.load('uv_table.npy')

mesh = Mesh([verts, faces])
mesh.texture('SMPL_sampleTex_m.jpg',tcoords=uvmap)
show(mesh)

material.zip

i've seen this before.. i'll investigate the issue

Thanks! Looking forward to your reply.
Really looking forward to building a good demo togather for the researchers in my field.
Vedo is great.

Are you sure that your texture coordinate file is correct?
I'm getting a strange warning:
/home/musy/soft/anaconda3/lib/python3.7/site-packages/vtk/util/numpy_support.py:137: FutureWarning: Conversion of the second argument of issubdtype from `complex` to `np.complexfloating` is deprecated. In future, it will be treated as `np.complex128 == np.dtype(complex).type`.
  assert not numpy.issubdtype(z.dtype, complex), \

Also with pure vtk (no vedo) I cannot get rid of the problem... maybe I should ask in the vtk forum
import numpy as np

txu = 'data/guy/SMPL_sampleTex_m.jpg'
uvmap = np.load('data/guy/uv_table.npy')

####### with pure vtk:
# import vtk
# from vtk.util.numpy_support import numpy_to_vtk
# mreader = vtk.vtkPolyDataReader()
# mreader.SetFileName('data/guy/guy.vtk')
# mreader.Update()
# pd = mreader.GetOutput()
# mesh = vtk.vtkActor()
# mapper = vtk.vtkPolyDataMapper()
# mapper.SetInputData(pd)
# mesh.SetMapper(mapper)

# reader = vtk.vtkJPEGReader()
# reader.SetFileName(txu)
# reader.Update()
# pd.GetPointData().SetTCoords(numpy_to_vtk(uvmap))
# tu = vtk.vtkTexture()
# tu.SetInputData(reader.GetOutput())
# tu.SetInterpolate(False)
# # tu.SetRepeat(False)
# # tu.SetEdgeClamp(True)
# mesh.SetTexture(tu)

# renderer = vtk.vtkRenderer()
# renderer.AddActor(mesh)
# render_window = vtk.vtkRenderWindow()
# render_window.AddRenderer(renderer)
# interactor = vtk.vtkRenderWindowInteractor()
# interactor.SetRenderWindow(render_window)
# renderer.SetBackground(1, 1, 1)
# renderer.ResetCamera()
# render_window.Render()
# interactor.Start()
#######################

####### with vedo:
from vedo import Mesh, Picture, show
mesh = Mesh('data/guy/guy.vtk', c='w').lw(0.1).texture(txu, uvmap)
mlab = mesh.labels(uvmap, ratio=1, precision=2, font="VTK", c='k')
show([(mesh,mlab), Picture(txu)], shape=(1,2), axes=1, sharecam=False)

vtk is trying to interpolate the boundary jumps:

Have you tried by chance to visualize this mesh with some other software?

I really appreciate your effort. I have test the fbx file with the same uv map in the Blender and unity. They work fine. It is just wierd in vtk. I will make further check.

Yes, the reason stated in the link sounds reasonable. Here is it:
I just ran into this exact problem as well and found that the reason this happens is because VTK assumes there's a 1-to-1 relationship between points in the polydata and uv coordinates when rendering the actor and associated vtkTexture. However, in my case and the case of OP, there are neighboring triangles that are mapped to different sections the the image, so they have very different uv coordinates. The points that share these neighboring faces can only have one uv coordinate (or Tcoord) associated with it, but they actually need 2 (or more, depending on your case).
The points lie in the border between different part of UV map correspond to multiple uv texture.
But I can't understand his solution.
 My solution was to loop through and duplicate these points that lie on the the seams/borders and create a new vtkCellArray with triangles with these duplicated pointIds. Then I simply replaced the vtkPolyData Polys() list with the new triangles. It would have been much easier to duplicate the points and update the existing pointIds for each of the triangles that needed it, but I couldn't find a way to update the cells properly.

I kind of understand the second comment... but it looks rather complicated.. and it's non trivial to detect the seam of the texture.. I'll post this as an issue in the vtk forum let's see if they have some better solution..

Thanks!

I got an answer from the vtk develpers here, and indeed it looks like a non trivial issue in vtk.. As I understand there will be a dedicated filter which will deal with it in some clever ways... The solution of collapsing the triangle strip that links different UV regions seems to go in the right direction, but definitely it's not ideal as it modifies the mesh.. Here is a quick test.. which only alleviates the problem..:
import numpy as np
from vedo import *

txu = 'SMPL_sampleTex_m.jpg'
threshold = 4.0

mesh = Mesh('guy.vtk', c='w').texture(txu).lw(0.1)
# mesh.write("guy.ply") # doesn't work because of "warning complex.."
uvmap = mesh.getPointArray('tcoords')
mlab = mesh.labels(uvmap, precision=2, ratio=2, font="VTK", c='k')

grad = mesh.gradient("tcoords")
ugrad, vgrad = np.split(grad, 2, axis=1)
ugradm, vgradm = mag(ugrad), mag(vgrad)
gradm = np.log(ugradm*ugradm + vgradm*vgradm)

largegrad_ids = np.arange(mesh.N())[gradm>threshold]

# collapse faces that have large gradient
faces = mesh.faces()
points = mesh.points()
new_points = np.array(points)
for f in faces:
    if np.isin(f, largegrad_ids).all():
        new_points[f[0]] = new_points[f[1]] = np.mean(points[f], axis=0)
mesh.points(new_points)

mesh2 = mesh.clone().cmap('jet', gradm).addScalarBar()
mlab2a= mesh2.labels('id', font="VTK", c='k', justify="bottom-left")
mlab2b= mesh2.labels(gradm, precision=2, c='dg', justify="top-left", italic=1)

show([(mesh,mlab), (mesh2, mlab2a, mlab2b), Picture(txu)],
      N=3, axes=1, sharecam=False)

the other proposed solution to save to PLY and load it back, I could not test it because it seems that the numpy file of uvmap has some problems as per warning:
/home/musy/soft/anaconda3/lib/python3.7/site-packages/vtk/util/numpy_support.py:137: FutureWarning: Conversion of the second argument of issubdtype from `complex` to `np.complexfloating` is deprecated. In future, it will be treated as `np.complex128 == np.dtype(complex).type`.
  assert not numpy.issubdtype(z.dtype, complex), \

bottom line: I would wait for the vtk upstream solution of the dedicated filter.

Thanks a lot for your great effort!
Yes, your solution of collapsing triangles with large gradient does look better.

I thinks the warning occurred during saving pkl indicating that there are some problems in UV map. I will look into this.
Thanks again. We really appreciate your time and effort.

I have updated a new version of ROMP using vedo for visualization.
I will upload some demos later. Thanks!


Error Installing

Potential duplicate of a previous issue
Installation not working, on either the installer or delta patcher on windows.
Tried with "run as admin" to no avail.

Buckshot roulette got a patch recently, could this be because of that?

BRML has not been updated to the latest version yet. You're likely running v<IP_ADDRESS>, which we have not said we support yet.

Thanks - I'll keep a lookout for if yall do update it!


Multiple hyphenations per word in whitelist

If a word has multiple breakpoints you have to insert multiple entries in the whitelist. For example:
hy-phenation
hyphen-ation
hyphena-tion
If you add an an entry with all possible breaks like hy-phen-a-tion, lua-check-hyphen doesn't recognize correctly hyphenated words. It would be nice to add this feature.

This is finally fixed in version 0.4.


Document how to enable pop-up windows for Safari

https://www.technipages.com/safari-popup-blocker
Also, we can try to check a status of the pop-up window  blocking for Safari before test running.

waits for #912

waits for #912

The Multiple Browser Windows feature is not implemented in Safari. So, it's not possible to accomplish this task.


Logging v1 - error when viewing/downloading cattle-logging yaml

Setup

Rancher version: 2.6.8
Browser type & version: chrome 105.0.5195.125 (Official Build) (x86_64)

Describe the bug
Installed logging v1. when attempting to view/edit the cattle-logging yaml in the system project, error in UI. when attempting to download yaml, uncaught exception in dev tools, and no download initiated.
To Reproduce

provision a k8s 1.20 cluster RKE1 Rhel 8.6 (used in this test)
install logging v1
confirm logs are flowing to endpoint specified
goto projects/namespaces
attempt to edit or download catte-logging namespace yaml

Result
500 error in console
[Object object] error on UI
Expected Result
yaml opened for editing
Screenshots

Additional context

/backport v2.6.9

Logging v1 will be deprecated in 2.7.0 so fixing this is not a priority. However, we'll check if we need to remove this from the old UI along with Dashboard.

A 500 error is always an internal error on the server producing the API response, if it were a malformed request producing an error, we'd see something like a 300 or a 400 error. Recommend transfer to backend.

@ronhorton do you have details on the logging provider used? We're going to transfer this to rancher/rancher since it looks like a backend issue rather than UI.

i've been unable to reproduce this error. closing for now.


Intellisense is broken in the vue-ts vite environment

Describe the bug
Intellisense looks to be broken in vite single file components.
The editor does not seem to be able to parse the typescript.
If a create new string const a = 'asdf'; and in the next line type a.alsdslkfjfaasdfladskjf() , I would expect the editor to understand a is a string and that this gibberish method does not exist and give eslint errors. Instead: nothing is happening
Link to the blitz that caused the error
https://stackblitz.com/edit/vitejs-vite-vdxrot?file=src%2Fcomponents%2FHelloWorld.vue,package.json&terminal=dev
Steps to reproduce
Create a new vue-ts vite environment.
Open helloworld.vue
Create a new string: const a = 'asdf';
On the new line, type a and wait for autocompleting: nothing happens.
Then write gibberish like a.alsdslkfjfaasdfladskjf() and expect eslint to complain.
Expected behavior
I expect the editor to understand that a is a string and:

Autocomplete string methods
Give eslint errors for undefined function calls on a

Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 103
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 120786951,
  "usedJSHeapSize": 115439411,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = b0de7779cd2868347677a5b8bbbf526e914eb012

Additional context
No response

Hey @stam, thanks for reporting! Let's close this in favour of #821 and continue the discussion there.


help dein not working

minimal setup:
:help dein => Sorry no help for dein

Please don't ignore the issue template.
I have not ESP skills.

I will close it later.

Reproduced.


La'Tonia Mertica EW Mentee Assessment (to date)

as much as could complete on my own

Hello Those Powering EW,
Didn't see option to add reviewer(s) et cetera. Apologies if this pull request is in error in any way.
Thanks for this opportunity, please stay safe.


LCAM-1015

What
LCAM-1015
Describe what you did and why.

Disabled prosecutionConcluded listener in staging for testing

Checklist
Before you ask people to review this PR:

[ ] Tests should be passing: ./gradlew test
[ ] GitHub should not be reporting conflicts; you should have recently run git rebase main.
[ ] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[ ] You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

PR raised for testing purpose in Staging env. Closing this PR.


Rename Logs to Diagnostics

Diagnostics is much more descriptive since it covers images, videos and other non-textual output. It can also be ambiguous relative to FBSimulatorLogger

@facebook-github-bot import

reviewed IRL