[terra-button] [terra-list] Resolve WDIO screenshot hover tests

Bug Report
Description
Created from this comment: https://github.com/cerner/terra-core/pull/2929#issuecomment-620058376
Need to investigate why byproduct of this fix #2929 does not work for screenshot tests - such that the hover tests no longer show the hover styles applied as intended by the tests. Fix could be to create an alternate test method or investigation why headless chrome doesn't capture hover style for test like this ../wdio/button-spec.js#L10-L19
Looks like it also is a problem in list, where this fix was first applied. Although -- not sure why, it didn't seem to cause an update for screenshots when it was first applied on Apr 10, 2019 on PR #2332 -- instead, it wasn't until Dec 2, 2019 that the screenshots changed.
Steps to Reproduce

Apply media styles similar to pull/2929/files#diff-c2c3bae...R18
Run screenshots tests
Screenshots fail unless updated with hover-style no longer visible.
Diff Example: ...snapshots/reference/en/chrome_tiny/button-spec/Neutral-Hovered[default].png

Additional Context / Screenshots

Emphasis-Hovered[default].png
Ghost-Hovered[default].png
Neutral-Hovered[default].png
De-emphasis-Hovered[default].png

Expected Behavior
WDIO tests for hover style should correctly capture the hover appearance styling in screenshots.
Notes
Fixes will also need to be added to https://github.com/cerner/terra-core/pull/2932 and any other companion similar issues in other repos.

Instead of depending on media queries to remove hover styles on Touch- devices. We can use ontouchstart value to check whether the device is touch device or not. Depending in this value we can restrict hover-styles by using :not() function in css like below:
    &:hover:not(.is-touch-device)

// before render set value for is-touch-device
   { 'is-touch-device': ('ontouchstart' in window) },

@supreethmr We discussed this proposal. Are there any other, non-javascript options that will resolve this issue?

Reason for considering ontouchstart event handler for fixing hover styles for touch-devices are :

ontouchstart is being used in terra-components like terra-form-select so I don't think there will be any compatibility  issues with this approach.

The code changes are minimal for using ontouchstart to restrict hover styles on touch devices.

Disadvantages with using @media queries are :

There are no standard media query available to support hover media feature in IE. -ms-high-contrast is marked as non-standard feature and might be changed in future releases of Microsoft. Reference : https://developer.mozilla.org/en-US/docs/Web/CSS/@media/-ms-high-contrast

Since our WDIO uses headless chrome for visual regression testing these hover style rule will apply for all tests irrespective of the viewport size. It seems that headless chrome doesn't support the @media (hover: hover) media query, could be similar to issue : (https://github.com/puppeteer/puppeteer/issues/4820). Due to this issue hover styles in WDIO tests are missing.

And Other CSS options like Pointer CSS media feature will also not work with headless chrome.
I think the other option would be to identify the touch devices using screen width in media queries. But I am not sure of the limitations with this approach.

Thank you for the investigation and documentaiton @supreethmr -- for reference, channelling my inner @bjankord and some background as to why we do not feel comfortable using this method as the solve.
We have very intentionally made it a general rule that Terra does not attempt to do user-agent, device, event, or feature detection within our JavaScript, because it can lead to unreliable confidence in being able to customize Terra applications, usually specifically for mobile, and falsely assumes user behavior based on device capability. This becomes dangerously close to an approach like Modernizr, where initialization detections are made and global classes are applied to base/body/html to apply to the full application -- which Terra has also intentionally rejected such methods in the past as it runs contrary to being browser/device agnostic.
(Yes, there are however a very small number of places where we have made special exceptions, usually only after much discussion, and agreement to limited usage until alternate methods can replace them.)
Having the complex media queries is not perfectly ideal either, but less concerning as an approach mainly since it is exclusively CSS.  Agree that the current 4-part query conflicts with WDIO, so not perfect here either.
Please hold @saket2403  - we are going to do have a discussion about approaches, since there is a secondary issue also being discussed for mobile apps that might be related in the solve, depending on how it's decided to handle one or both of these.

@saket2403 The updated direction for
cerner/terra-core#2932
https://github.com/cerner/terra-clinical/pull/652
https://github.com/cerner/terra-framework/pull/1095

[ ] revert and remove the ontouchstart detection and the 'is-touch-device' class, and return back to the compound media query `@media screen and (-ms-high-contrast: active), (-ms-high-contrast: none), (hover: hover) and (pointer: fine) { &:hover { ... } }.

[ ] since this prevents webdriver's ability to capture the hover visual style, we can reasonably shift to manual unscripted testing for all hover testing, so for all packages we are applying this media query, we can remove any corresponding hover wdio tests, plus remove the corresponding snapshots and test examples.


⚠ Bofrost.fr has degraded performance

In a1238ac, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 955 ms

Resolved: Bofrost.fr performance has improved in 7796cd0.


Page continues to read the previous/closed page image

With VoiceOver on, when the user is going item by item from How it works (step 3 of 6) to "Learn more about how it works" and comes back to step 3 by either clicking Close or by going through all 4 of it's tabs, it announces "Hand drawn illustration of mobile phone in a pocket. sending and receiving signals". It announces first when the user gets back and then continues to announce this when the user swipes right to read next item.
This is also the case for Step 5 for "Learn more about privacy". VoiceOver continues to announce "Privacy policy heading".
Please see video of the Step 3. I'm only swiping right and left. https://photos.app.goo.gl/TCFbUyq7SPJQM1WZ6
iOS 13.5.1 on iPhone 7

@mtropper can you take a look at this one?

@JuliannaR  this appears to be a bug that was introduced in Version 54. its not limited to these screens it happens in all of the screens that are opened in the overlay window as well. Do we know what change was made in that release?

I tested and It work properly in V53 and got broken in V54 on testflight

I think this is introduced when fixing for android focus. @smcmurtry do you remember something about Android and IOS having different approaches to focus? This might related to https://github.com/cds-snc/covid-shield-mobile/pull/766

I was also able to reproduce this going from the bottom menu to the "Change province or territory" screen and then clicking close. Focus is stuck on the "Province or territory" heading, which is offscreen after the page is closed.
This is due to the navigator change from #738. I ran git checkout f71d7aab38b64022e569098b1d70eb423846b300 src/navigation/MainNavigator.tsx to get the old navigator and this issue went away.
I'm not sure how to fix this - there is a useFocusEffect hook we can use with the new navigator, which we could use to do something with the autofocus when the page changes.

I made a PR for the how it works screen and privacy screen specifically and it is working on Android: #801. It uses a useEffect hook to manually focus on an element when the navigator focuses on a screen (or when an overlay window is closed)

@nickmly unfortunately the same issue continues in IOS. I reverted the changes from pull 738 (https://github.com/cds-snc/covid-shield-mobile/pull/738/commits/651ce631b81b7485479e7c1752add86ffb18d9ea) and things started to work as @smcmurtry but reverting made the fix you had made for Android to stop working.

Note: I think there are 3 problems here:

react-native-screen causes the main issue described in the ticket which is: VoiceOver / TalkBack reads screen that are being unmounted. Look like react-native-screen doesn't unmount it. It just hides the screen.
=> Suggestion for now: remove react-native-screen

src/screens/tutorial/views/ItemView.tsx  which is used for Tutorial screen. When pressing on close button, it's unmount > mount > unmount. This is the main problem that causing the iOS that I describe above.
=> Suggestion: keep digging to see why this happens.

AccessibilityInfo.isScreenReaderEnabled() is async + we have 200ms delay to start reading which introduce more delay. If two elements on the screen requesting for focus, it's unpredictable which one will take over. This is not likely to happen in theory as we set which element to auto focus on each screen. However, if 2nd case happens, then this happens.
=> Suggestion: in src/shared/useAccessibilityAutoFocus.ts
Replace AccessibilityInfo.isScreenReaderEnabled()  by using useAccessibilityService as we have a provider for it. So, we can get the enabled result instantly.
Replace delay function by queue function. The idea is that if we have more than one element requesting autofocus, after the timeout (200ms or whatever), the most recent element requesting focus will be read. The rest will be ignored.

cc @smcmurtry @nickmly


cannot enable 2-arg fts3_tokenizer on SQLite 3.20.0

the internal flag value was changed at 3.20.0 (and it seems the value was changed back to the previous value)
It is private value defined in sqliteInt.h, so it is better to avoid using it.
2-args fts3_tokenizer should be enabled by the public API(sqlite3_db_config) as this module did before.
The change may break a compatibility with APSW built with its Amalgamation method.

fixed in <IP_ADDRESS>


After process death, the quiz says "please check your internet connection" and throws you back

The good news is that I found no crashes in this regard :yum:

Part of the problems I'm encountering regarding handling network connections :man_facepalming:

Kindly outline any bugs you find. I will effect them all. Thanks

I actually tried to "quick fix" this one but it didn't work. This one is a bit more complex than I thought. :thinking:


Necessary diagnostic from eyring06jgr for the release of version2

Before you start, please read our contribution guidelines.
Please discuss your idea with the development team before getting started, to avoid disappointment later. The way to do this is to open a new issue on GitHub. If you are planning to modify an existing functionality, please discuss it with the original author(s) by tagging them in the issue.

Tasks

[ ] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[ ] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[ ] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[ ] Please use yamllint to check that your YAML files do not contain mistakes
[ ] (Only if really necessary) Add any additional dependencies needed for the diagnostic script to setup.py, esmvaltool/install/R/r_requirements.txt or esmvaltool/install/Julia/Project.toml (depending on the language of your script) and also to package/meta.yaml for conda dependencies (includes Python and others, but not R/Julia)
[ ] If new dependencies are introduced, check that the license is compatible with Apache2.0

New recipe/diagnostic

[ ] Add documentation for the recipe to the doc/sphinx/source/recipes folder and add a new entry to index.rst
[ ] Add provenance information

Partly addresses #1586 .

Thank you!
Can you please update the checklist here?

Thank you!
Can you please update the checklist here?

Done.

CircleCi is complaining about the style:
/test/esmvaltool/diag_scripts/eyring06jgr/eyring06jgr_fig05b.ncl:148:3: E265 block comment should start with '; '

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1146:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1147:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1148:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1149:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1150:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1151:23: E127 continuation line over-indented for visual indent

Can you have a look?

One thing to note is: CMIP6 data only runs with the added ESMValCore functionalities from the pull request #595 (https://github.com/ESMValGroup/ESMValCore/pull/595).

CircleCi is complaining about the style:
/test/esmvaltool/diag_scripts/eyring06jgr/eyring06jgr_fig05b.ncl:148:3: E265 block comment should start with '; '

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1146:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1147:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1148:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1149:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1150:23: E127 continuation line over-indented for visual indent

/test/esmvaltool/diag_scripts/shared/plot/aux_plotting.ncl:1151:23: E127 continuation line over-indented for visual indent

Can you have a look?

Will do.

Tested successfully.
There is only the minor thing with the documentation to fix.

Tested successfully.
There is only the minor thing with the documentation to fix.

Ok, thanks for testing! The documentation problem should be fixed now.

It's still complaining: I fixed the path, let's see what happens...

It's still complaining: I fixed the path, let's see what happens...

Didn't fix it, I assume. It seems like sphinx cannot find the folder where the figure is located. I had to create the folder new. I have double-checked the name of the figure and the name of the folder in the figure-path, and both are fine. So I am at a loss why there is an error...

Didn't fix it, I assume. It seems like sphinx cannot find the folder where the figure is located. I had to create the folder new. I have double-checked the name of the figure and the name of the folder in the figure-path, and both are fine. So I am at a loss why there is an error...

The path looks correct now, recipe/ was missing.

Didn't fix it, I assume. It seems like sphinx cannot find the folder where the figure is located. I had to create the folder new. I have double-checked the name of the figure and the name of the folder in the figure-path, and both are fine. So I am at a loss why there is an error...

The path looks correct now, recipe/ was missing.

AAAAARRRGGGHHH!
How many times can it still fail us?!

AAAAARRRGGGHHH!

We are getting there :crossed_fingers:

AAAAARRRGGGHHH!

We are getting there crossed_fingers

It passed!!

:partying_face: :partying_face: :partying_face:
M&B 1 - CircleCI 0

partying_face partying_face partying_face
M&B 1 - CircleCI 0

:partying_face: indeed!
Took us long enough!  :laughing:


WIP: ENH: Add logging mechanism

Closes GH-70.

introduces config.log_level to control pipeline logging, and config.mne_log_level to control MNE logging
adds config.gen_log_message() to generate a message string consisting of the message itself, the pipeline step, participant ID, session name, and run number
makes use of the coloredlogs package, which provides a great and cross-platform way to make the logs more readable by adding color or changing the font weight

works like a charm !
great work @hoechenberger

Thanks! We still need to tune which messages to show at which log level, and we can adjust the color schemes.


TST: special: mark a slow test as xslow

The test test_gammaincc in test_precompute_gammainc.py currently
takes about a minute to run; it appears to have become slower after
the changes to Arg. Mark it as xslow since it's not essential for it
to run all the time.

Thanks Josh


Added tlsConfig to ServiceMonitor

Want to add customization for scheme and for tlsConfig for ServiceMonitor

Merged in https://github.com/opencost/opencost-helm-chart/commit/59c2897db6df5a1323113bf8bea3acdc6bfd19b5


Success and error firing on startup

I've this code in my Vue
<a href="#" v-clipboard:copy="`some text`" v-clipboard:success="handleCopyStatus(true)" v-clipboard:error="handleCopyStatus(false)">Copy

and under the Javascript script I've this part
methods: {
  handleCopyStatus(status) {
    console.log(status);
    this.copied = status
  }
}

I noticed the console.log was fired setting the copied to true and false successively. Then when I click on my Copy button it actually doesn't change the copied value. The copy to clipboard does work tho.
What's happening here ? Is it a bug ?
PS : I'm new to VueJS so I might have done something wrong.

Got this issue too!

@Inndy ?

@Loschcode seems like it works if you create separate functions for success and error, and reference them by method name without ()

Can you provide a minimal working example for me?

@franzwilhelm I did that indeed, forgot to give the solution here ...
My Button
Hope it helps.

I followed this solution, but it only update successful status after I click on another button. I mean, it does not update instantly when I click the copy button, but it will update if I click any button after that.
Any idea?

the solution is to use it exactly as the in the docs
v-clipboard:copy="abc"
v-clipboard:success="onCopy"

onCopy(e) {
   console.log(e)
},

which will produce
{action: "copy", text: "abc", trigger: td, clearSelection: ƒ}

Did you solve it? I have the same problem.

the solution is to use it exactly as in the docs
v-clipboard:copy="abc"
v-clipboard:success="onCopy"

onCopy(e) {
   console.log(e)
},

which will produce
{action: "copy", text: "abc", trigger: el, clearSelection: ƒ}

Did you solve it? I have the same problem.


looking for example of "real-world" usage of carbonyl

Greetings,
this is not a support issue, but I don't know of a more appropriate channel to ask.
I am writing an introduction to Carbonyl for an IT magazine. I would like to complete it with some examples of real world usage of carbonyl, that is, beside its value as a programming endeavour... what /when/why / how do you use it as a browser? That is why and how you launch carbonyl instead of an ordinary browser, considering that it cannot do audio, and only renders very grainy images and videos? Or am I losing something big here?
Thanks,
M.

University of Waterloo is using Carbonyl to enable users to mount their OneDrive using a fancy script that behine the covers uses rclone. Rclone simplest method to enable the drive is to use a web-browser to grant the app access through Microsoft's login portal. It's a clunky solution but it works thanks to Carbonyl!

Thanks! That's exactly the kind of success stories I was looking for. The more the better, of course, thanks in advance to whoever will share more

I haven't used much but I see this can be used to log into networks guarded with captive portals without accessing the GUI. Most captive portal sites aren't HTTPS, so they use some JS to hash the password before sending to the server, which wouldn't work on existing CLI browsers. There are already some solutions like using WebDriver, but it requires a full-blown browser to be installed. Carbonyl can be a lightweight alternative to them.

I've made the Carbonyl Terminal client app for Windows/Linux/MacOS based on Windows Terminal / QTerminal. Have a try!

Carbonyl provides the ability to render interactive frame graphs in remote servers.

I've used carbonyl to troubleshoot an issue with vite. It showed me that the problem wasn't with vite at all, rather with the vs code insiders build I was using.

We deploy a lot of raspberry pi with wifi camera in it's own network. In order to access the camera we need to ssh to the raspi and use carbonyl to configure the wifi camera, if you are curious why use carbonyl, the camera has a very limited api support.

Why not just use an SSH tunnel and access web camera from a local browser?

Why not just use an SSH tunnel and access web camera from a local browser?

We don't directly ssh to the raspi using our local terminal, our shell is on a web browser which communicates to a server via websocket and then relays to the raspi via tunnel SSH. It's a weird structure, but i'll look into what you were suggesting. I'm not that profound in reverse tunneling.
I know that i can use my local terminal to use the reverse tunnel initiated by our raspi to our ssh server but i'm not too sure how to reverse tunnel the browser back to server and to my local at the current writing of this. But i'll definitely look into it!! Thanks for the suggestion, if it works i'll update you!! :D

https://superuser.com/questions/1855629/browse-a-web-page-through-ssh-need-to-access-router-web-interface-remotely-bu/1855639#1855639


🚀 | move music and playlist directories

Feature Description
What functionality would you like to see in your phoniebox?
To separate the phoniebox code from it's payload, we should not use a subdir of RPi-Jukebox-RFID to store data.
How do you envision the feature to work from a users perspective?
There is a common understanding, that music goes to ~/Music on desktop environments. IMHO, we should stick to that to make it easier to find for users.
The same concept should be used for playlists: ~/Playlists
TODO

[ ] Migration path for existing installations
[ ] Documentation has to be modified to reflect this (buzzwords to remember: nfs, cifs, samba, backup, sync)
[ ] Config changes for mpd, mopidy, web interface, etc

I think this feature will break hundreds of phonieboxes and leaves confused users behind. Anyone, who is just executing command lines that you're giving him or her, does not understand, what is happening. If you really consider to move files, there should be a really good migration path, that moves configuration files automatically and adjusts all configuration the user might done. From what I've seen in the code, it will be really hard to create a 100% solution for this.
I'm with you, that in a well designed software the config files are something that should be stored separately from the application. But we're here talking about a little Raspberry PI, which has only one function in this situation (play music and audio files for the kids). So in a matter of design aspect it is really cool to separate this. But in the matter of function and unnecessary changes this will break function and you'll gain nothing.
If you ever want to change this: Do it in version 3.0. May be in a separate project, where you can make some things better, that are currently not well designed. I took a longer look in the code. There are situations, where PHP calls a BASH that calls a PYTHON which uses BASH commands.
So Phoniebox works as it is. I'm looking forward for a version 3.0. But please, don't change this.


rejoin is expired?

everything was working fine on a test project... now I'm getting this error consistently, but no code has changed:
colyseus.js:2832 colyseus.js: server error: rejoin has been expired for npy0KR3Xc
I've cleared caches, restarted everything... nothing seems to make a difference.
What does this mean, and how can I fix it?
Thank you!
David

@davidhoare are you using allowReconnection during onLeave in the server? The rejoin is only allowed while allowReconnection is still pending. Cheers

colyseus.js: server error: rejoin has been expired for wG-jzWwI5
I have the same problem without using allowReconnection

I have encountered the same as @xubowenshizi - I can't remember exactly what I did, but either onAuth/onJoin were failing and/or erroring and this was the reported error.
@xubowenshizi Try debugging your onAuth/onJoin to see if there are any issues.

@xubowenshizi you must use allowReconnection to be able to use rejoin. See docs: http://colyseus.io/docs/api-room/#allowreconnection-client-seconds

@endel I think it might be a bug in error reporting. I had the same issue - I don't use the reconnection architecture from Colyseus in my game anywhere - however I randomly started getting the rejoin has been expired error.
It was something to do with my onAuth/onJoin failing and/or erroring, fixing that removed the issue.

If you guys happen to have a project that I can reproduce this please let me know, I've tried and cannot reproduce here using latest versions of client and server. 😭

turns out I traced my particular issue to an unrelated null variable that was causing a glitch and instead it was reporting this error...
Thanks for looking into it, everyone!

@davidhoare I and a few others seem to have had the same issue for 0.9.22, is that the version of Colyseus you are using? Could you be more specific about your solution in case I'm making a similar mistake? Thanks

Hey, @Loonride just helped to find the reason for this, apparently when the matchmaker retries in case of an odd situation, the joinOptions get mutated inside matchMaker.onJoinRoomRequest , then if it retries, it will try to perform a rejoin without asking:
https://github.com/gamestdio/colyseus/blob/766497e23eaa5b48a6594df02fd5fcac42498f08/src/Server.ts#L244-L250
Reopening until it gets fixed


Support builds with maven or gradle

Apache Ant is somewhat antiquated has a tool and doesn't allow for artifact dependency management. Would love to see builds using maven or gradle

This is implemented now and documented at https://docs.aion.network/docs/maven-and-aion4j
Does this resolve your issue?


scene: add wlr_scene_attach_output_layout

This is a helper to integrate wlr_scene with wlr_output_layout.
Depends on: https://github.com/swaywm/wlroots/pull/3117

cc @ifreund @kennylevinsen @ammen99
This uses the approach discussed on IRC.
The helper is just dumped in wlr_scene.h, but could be moved elsewhere if desirable.

I've moved the new helper implementation to a separate source file, but kept the declaration in the wlr_scene.h header for now.


TensorBoard example: net object has no attribute named_parameters?

Hi, I'm a bit new to PyTorch / Python so I apologize if I made a poor diagnosis in advance. My error is of the following form when running main.py
python main.py
Step [100/50000], Loss: 2.1832, Acc: 0.51

Traceback (most recent call last):
  File "main.py", line 94, in <module>
    for tag, value in net.named_parameters():
  File "/home/yenson/anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 237, in __getattr__
    return object.__getattr__(self, name)
AttributeError: type object 'object' has no attribute '__getattr__'```

@yenson-lau Please give me more information about your error.

Update till the most recent version of PyTorch solved this issue for me.

I still have this problem and I am using a current version on CUDA-X:
for name, param in network.named_parameters():     print(name, '\t\t', param.shape)
`AttributeError                            Traceback (most recent call last)
 in
1 # Accessing the Networks Parameters
----> 2 for name, param in network.named_parameters():
3     print(name, '\t\t', param.shape)
AttributeError: 'Network' object has no attribute 'named_parameters'`


Presentr animation from corner

May be, it will be useful for someone, or the author will add it to release (desirable, in 3.0 swift :) ).
The code of corner animation, based on author's animations.
import Foundation
import Presentr

public class CoverFromCornerAnimation: PresentrAnimation {

    private var fromBottom: Bool
    private var fromRight: Bool

    public init(fromBottom: Bool = false, fromRight: Bool = false) {
        self.fromBottom = fromBottom
        self.fromRight = fromRight
    }

    override public func transform(containerFrame: CGRect, finalFrame: CGRect) -> CGRect {
        var initialFrame = finalFrame
        if(fromBottom) {
            initialFrame.origin.y = containerFrame.size.height + initialFrame.size.height
        } else {
            initialFrame.origin.y = 0 - initialFrame.size.height
        }
        if(fromRight) {
            initialFrame.origin.x = containerFrame.size.width + initialFrame.size.width
        } else {
            initialFrame.origin.x = 0 - initialFrame.size.width
        }
        return initialFrame
    }

}

The usage is simple.
From left top corner:
presenter.transitionType = TransitionType.custom(CoverFromCornerAnimation())
From left bottom corner:
presenter.transitionType = TransitionType.custom(CoverFromCornerAnimation(fromBottom: true))
From right top corner:
presenter.transitionType = TransitionType.custom(CoverFromCornerAnimation(fromRight: true))
From right bottom corner:
presenter.transitionType = TransitionType.custom(CoverFromCornerAnimation(fromBottom: true, fromRight: true))

Interesting  new animation.
Could you open a Pull Request so I can test it out and further take a look into it.
Thanks.


Enable option to skip vertical interpolation in QGField if variables are already on uniform pseudoheight grid

https://github.com/csyhuang/hn2016_falwa/blob/1615b37a7f59dbbf335ac40e1931d63cd1525026/scripts/pre_process_data_in_hybrid_coordinates/vertical_interpolation_hybrid_to_pressure_poisson_filled_below_topo.py#L57
Optimize the interpolation procedures and refactor it into utility module of falwa.
[Update] After some investigation, the time needed seems inevitable unfortunately. Perhaps we can reduce the cost a bit by allowing QGField to take in evenly-spaced pseudoheight grid without doing interpolation. But this would not save much time, since the time sink is mostly at the hybrid coordinate conversion.
TODO: Move the vertical interpolation from f2py to scipy. Let f2py code do only QGPV and static stability computation.

This is done with commit a0d9252 😄 I will close this


[MemoryLeak] medLogger TeeStreams, TeeDevices and d

Based on https://github.com/medInria/medInria-public/pull/1116
Commits:

https://github.com/medInria/medInria-public/commit/3a76bbf219d1ce65c829e024a934439f160e458d
https://github.com/medInria/medInria-public/pull/1117/commits/b7431d770379fe9bb0474d26f5ffa8a0eda239dc

Solves several memory leaks in medLogger from teestreams, teedevices and d.
:m:

This pull request has been rebased


Clarification

@ifzhang Thank you for this work. I would like to get clarification of the Byte track.
My question is, can we call ByteTrack as the one-shot method  like FairMOT (On the Fairness of Detection and Re-identification in Multiple Object Tracking) and JDE (Towards real-time multi-object tracking) because combines both detection and tracking at once.
Or we can not categorize it into one shot method  because there is no re-id part?
Where does it fall??
Thank you.

have any answer?

@NaifahNurya @gyh420 byteTrack it's a technique for matching object between frames using motion model (here is kalman filter) it's contains beautiful technique for solving the problem of matching. (

First : doesn't contains appearance model
Second : you can go to part of tutorial inside bytetrack you can found the combination of FariMOT and ByteTrack


Introduce a CMake option to specify a plugin search path

The new PLUGIN_SEARCH_PATH option allows for specifying a default plugin
search path at build time. More precisely, the plugin_searchpath
attribute is assigned the value of PLUGIN_SEARCH_PATH by default.
This makes it possible for example to simple install custom plugins to
PLUGIN_SEARCH_PATH without having to fiddle around with environment
variables (i. e. OIIO_LIBRARY_PATH).

LGTM. I will merge this.

Is this ok for master, or do you need it backported to 1.5?

A backport to 1.5 would be very nice.

Done.

Thanks!

I am going to rename the internal name of the definition, however, to prefix it with OIIO rather than PVT.


App Feedback Questionnaire

Questionnaire for the user.
To be added as a link next to About Us, or as a pop-up.

Seems already completed.


Cross compile to scala 2.12 y scala 2.13

Some users want to use Shaclex using scala 2.12. Provide a cross compiled version

Done at version 0.1.46


Friend deleted his account, can't hide or block the conversation

I get the following errors in the chat window:
error unboxing chat message: Deleted
Error Code 100 - bad list value in method keybase.1.apiserver.Get
There are no buttons or any other options within the failed conversation.

Also receiving this error

thanks for the report @taohansen, we'll take a look this morning

Any updates on this @maxtaco?

I have the same issue. In my case its annoying cause I got a message, I wasnt able to read it and then the user was deleted. I have 2 notifications waiting and there's no way to delete them.


Update api.md

Fix inconsistent punctuation
Clarify explanations in 'skipped' and 'After' section

Hi @selfeki,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber


Question. Does there is opposite to this library like ESTree -> JSON

Hi! Thanks for your library.
I am looking for the opposite library to convert ObjectExpression from ESTree to JSON. Any thougout?

Good question. I have thought about this myself as well.
The problem is that estree may contain a lot more that what this library can produce. For example, estree may contain function calls, if-statements, and other control flow expressions that are simply not relevant to represent a simple value as estree-util-value-to-estree does.
I suppose it would be possible to write such a function where you limit support to specific value. I imagine it would look something like this:
export function estreeToValue(node: Expression) {
  switch(node.type) {
    case 'Literal':
      return node.value
    case 'UnaryExpression':
      if (node.operator === '-') {
        return -estreeToValue(node.argument)
      }
      if (node.operator === '+') {
        return +estreeToValue(node.argument)
      }
    // Etc…
  }
}

This will work now, but there is no guarantee a later version estree-util-value-to-estree won’t produce nodes that this new function can’t handle.
For example, one thing that’s still missing from estree-util-value-to-estree is common references.
const a = {}
const b = {
  // In the result of estree-util-value-to-estree b.c is not equal to b.d
  c: a,
  d: a
}

const r = {}
// This object will cause `estree-util-value-to-estree` to recursive indefinitely
r.r = r

To serialize such a value correctly, estree-util-value-to-estree would need to generate an iife. That’s going to be really hard to handle for an inverse algorithm.

I recommend to use the same approach as in the tests: Use astring to generate the value, then evaluate using the Function constructor. Of course you should only evaluate content you trust.
import assert from 'node:assert/strict'
import { generate } from 'astring'
import { valueToEstree } from 'estree-util-value-to-estree'

function estreeToValue(estree) {
  const code = generate(estree)
  const getValue = new Function(`return ${code}`)
  return getValue()
}

const value = { some: { complex: [,,, 'value',,, ] } }

assert.equal(
  estreeToValue(valueToEstree(value)),
  value
)

I’m closing this issue because it’s a question. Feel free to add more comments. It’s an interesting subject. :)

@remcohaszing thank you very much for your detailed answer! 🎉
I guess I will write something to parse only Literal (string/number/boolean) and ArrayExpression and ObjectExpression  nodes to be safe and avoid evaluating using the Function, the rest of things I'll just skip
const foo = {
  a: 'foo',
  b: true,
  c: 22,
  z: [...],
  d: {...}
}

Your best option to evaluate an estree expression is sval.
The upcoming minor version of value-to-estree release will add an option to preserve references, which also supports cyclical references. The next major version will flip this option, making it the default behaviour.


invalid yaml spec required for duplicate keys

In order to configure LDAP for rstudio-connect the following entry is required in the values file to allow multiple groups to login:
config:
  'LDAP "My Config"]:
    PermittedLoginGroup: cn=some-group
    PermittedLoginGroup: cn=other-group

this violates yaml spec (https://yaml.org/spec/1.2.2/) which requires keys be unique, and causes all kinds of problems when attempting to deploy via helm, kustomize, or any other tool. Right now it's not possible for us to deploy this and allow more than one group to login
similar problems exist for many other config values like Authorization.AdministratorRoleMapping which are supposed to be defined multiple times.
One fix for this would be to configure the configmap template helper function in helm to generate entries based on either strings, for single entries, or an array of strings, for multiple entries. The canonical solution for this is to check the typeOf the value and iterate accordingly:
# values.yaml
config:
  'LDAP "My Config"':
    PermittedLoginGroup:
    - cn=some-group
    - cn=other-group

When rendering key/values, we check if the value is a string and render directly, otherwise iterate and write your duplicate keys into the configmap:
# generation snippet
...
{{- range $key, $val := .items }}
{{- if kindIs "string" $val }}
  {{ $key }}: {{ $val }}
{{ else }}
{{ range $val }}
  {{ $key }}: {{ . }}
    {{- end }}
{{- end }}

Hi @scott-grimes  Thanks for submitting this issue!
We had a similar issue with the rstudio-workbench chart and came up with a similar workaround.
Using the library chart in the PR, this is the following output for your proposed values. Can you confirm whether this renders properly for your use case?
---
# Source: rstudio-connect/templates/configmap.yaml
(... snip ...)
data:
  rstudio-connect.gcfg: |
    [LDAP "My Config"]
    PermittedLoginGroup = cn=some-group
    PermittedLoginGroup = cn=other-group
    (... snip ...)

beautiful, that works great
# delme-values.yaml
config:
  'LDAP "My Config"':
    PermittedLoginGroup:
    - cn=some-group
    - cn=other-group

helm template . --values delme-values.yaml -s templates/configmap.yaml
yields
---
# Source: rstudio-connect/templates/configmap.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  name: RELEASE-NAME-rstudio-connect-config
  namespace: prod
data:
  rstudio-connect.gcfg: |
    [Authentication]
    Provider = password

    [HTTP]
    Listen = :3939

    [LDAP "My Config"]
    PermittedLoginGroup = cn=some-group
    PermittedLoginGroup = cn=other-group
...

Hi @scott-grimes
These changes have made it into all of the charts. For connect this is version 0.2.32.
I am closing this issue, but please re-open it if you run into problems using this.


Dirty write prevention in notebook content provider

With @jrieken 's fixes of reloading non-dirty notebook document if there are changes from local disk, we see "Content on disk is newer" less frequently (https://github.com/microsoft/vscode/issues/104209), but only now I saw that we tried to prevent false positive through
https://github.com/microsoft/vscode/blob/cda3b7e157fe0cbd0dfad66a30aa3cbd2ddd1e47/src/vs/platform/files/common/fileService.ts#L394-L414
which compares mtime, etag and size. We would adopt the same heuristics in notebook land.

Does this issue occur when all extensions are disabled?: Yes/No

@rebornix where is that code in notebook land and can you not just use the file service directly to benefit from this prevention? Or actually the ITextFileService even when you save the underlying resource of the notebook?

@bpasero we delegate file serialization and saving to extensions, and extensions will use fs in the ext host to save the document to file system. That's why we can't use ITextFileService directly. However we do want to listen to the file change and file stats of the resource URI.

If that is the case, how do you ever get a "Content on disk is newer" error if all file handling goes through the extension and not our file service?

They are still files of a provider, so we stat them "locally"

It's now replaced by the serializer api.


fin uni

please read updates to the README; added one contract, one javascript helper, and an interface for talking to Uni

Adding a test in for the new contract. Rationale for using uni is slipping with liquidity drain. See screenshot, compared to $2M on sushi.

Core functionality tested to work but I still have several reservation about this contract:

requires a certain amount of weth to be sent to the contract for withdraw to work

probably better to use the weth balance in the reward calc

there is no insurance claim function
there is no zap function for univ3 liquidity
no external claim function for just rewards
univ3 has low liquidity for FOLD. Might be better to use sushi, then can use erc4626

Any movement on this?

Moved: https://github.com/manifoldfinance/fold-staking


Add signing support to VSIX

Verified with vsce-sign and binlogs

https://dnceng.visualstudio.com/internal/_build/results?buildId=2531117&view=results

Thanks for the review!  Since this affects publishing (and we always run the publish pipeline on main changes), I'm going to hold off on merging for now until we've done our publishing for this week.

Merging this now that the release stuff is all merged.


i am getting command not found in new ubuntu 18.04 server

I was tried with snap also ..getting the same error

Aa per your screenshot, you are not using Ubuntu, its looks like a SUSE Linux, try use below command to update the patch
sudo zypper update
Thanks
DevOpsEnz


[MLIR] Fixing WRW invoker by setting zero to weights

This turns out to be the root cause of #1085. Somehow this isn't an issue before the use of atomic add but starts to cause regression after that. It is still not clear to me why atomic add is the trigger. After force resetting the weights tensor before hand, all regressions in WRW xdlops fp32 are gone.

It is still not clear to me why atomic add is the trigger.
Because when we use atomic add, the default value in weight tensor will be added to the result. The non-atomic-add method just write the result to the weight tensor.

@jerryyin There is a small thing to note: if the data is f32, the algorithm uses atomic add, but if using other data formats, this algorithm do not use atomic, so  no need to reset Wei.

@atamazov In a way this PR is a feature rather than a bug, based on:

By the time @ltqin contributed his atomic add in wrw, the set zero kernel is indeed not necessary
The current commit of MLIR get pulled into MIOpen is at the point before @ltqin contributed his PR
The regression you saw that happen in MLIR side is where we always build with the latest of MIOpen (the combination never happens in MIOpen repo yet)

I have confirmed locally that if we build MIOpen with a newer commit of MLIR the existing MIOpen ctest does catch all failure cases. Therefore the test will regress as expected and I haven't seen a need to add new ones just to catch this case.
Would you prefer me to dial the MLIR commit forward to the latest (which includes his feature)?

@jerryyin

Would you prefer me to dial the MLIR commit forward to the latest (which includes his feature)? Or else my next PR will require a newer MLIR commit, we can let it "naturally happen" over there.

Is it so that necessity of "set zero kernel" depends on MLIR version?
Is it so that existing MIOpen tests will fail if "set zero kernel" is necessary but is not there?

Is it so that necessity of "set zero kernel" depends on MLIR version?

That's correct. Introduction of a new instruction in fp16 routine happens fairly recently that requires an additional set zero operation. This PR should have been done by owner of @ltqin, the owner of this feature.

Is it so that existing MIOpen tests will fail if "set zero kernel" is necessary but is not there?

That's also correct. All fp16 test will fail if the set zero operation is not there.


atForm duplicate submition

i'm use semantic-ui and useraccount-unstyled,
the onSubmitHook triggered twice on all click, i think it because your submit button type='submit' and in addition, he has a 'submit' class.
so, if u remove one of them it will work perfectly
look at this semantic-ui issue
tks for your awesome package!!!

+1

https://github.com/meteor-useraccounts/core/issues/628


Thread system error value not preserved when tracing

When tracing an executable, the tracing code interferes with the thread's system error code, which can cause incorrect execution of the traced process.
Example:
#include <iostream>
#include <Windows.h>

int main()
{
    SetLastError(1337);

    std::cout << "Last error: " << GetLastError() << std::endl;
    std::cout << "Last error: " << GetLastError() << std::endl;
}

Output (without tracing):
Last error: 1337
Last error: 1337

Output (with tracing):
...
2024-04-04 20:15:07,773     INFO | Writing trace data of 1025258 bytes...
STDOUT : Last error: 0
STDOUT :
STDOUT : Last error: 0
STDOUT :

Hi! My testing seemed to indicate that the problems in Frida and Frinet are independent of each other – though, I found both issues in the course of the same investigation. Simon From: myr-syn<EMAIL_ADDRESS>Monday, June 3, 2024 at 11:54 AMTo: synacktiv/frinet<EMAIL_ADDRESS>Simon Zuckerbraun<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Re: [synacktiv/frinet] Thread system error value not preserved when tracing (Issue #5)Hi !Thank you for your contribution.We wonder whether this issue might be linked to Frida itself. We also noticed this commit from the 16.2.2 release:gumjs: Preserve thread’s system error over NativeCallback invocations. Thanks @HexKitchen!Have you been able to confirm that the problem is now solved without applying this patch?—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
We tested this pull request on arm64 and it broke the tracer for some reason, the pull request was reverted until we understand the problem better.


Support for Sourceloop Microservices in sourceloop/cli

Is your feature request related to a problem? Please describe.
There should be a way to initiate a microservice based on one of the microservices provided in sourceloop
Describe the solution you'd like
Something like -
ls microservice --ms=@sourceloop/authentication-service
Additional context
This can be done in the same way a local package is added

Done.


DSI uses ENV

Changes in this PR

consolidate DSI linking building logic into a dedicated class
return a URI that defaults to the production environment
add DSI_ENV to control the environment and link prefix
remove DFE_SIGN_IN_ISSUER and DFE_SIGN_IN_API_ENDPOINT

 DSI env for use in link logic


Model

Learning objectives
Set up associations between models.
Write SQL queries with Active Record.

Workflow
Create models
Create model classes for all entities as shown in the ERD diagram.
Set up associations between models.

Thanks @Lucas-Masaba


Add VulnerabilityStats() to for kernel vulnerabilities

See https://github.com/prometheus/node_exporter/issues/1046

@bobrik @discordianfish shouldn‘t it be 0 or 1 depending if this is mitigated (mitigation) or not (vulnerable).

@mjtrangoni, it's not that simple:
$ cat /sys/devices/system/cpu/vulnerabilities/l1tf
Mitigation: PTE Inversion; VMX: conditional cache flushes, SMT vulnerable

I think raw value is a good tradeoff: you want to know what kind of setups you have and possibly alert if mitigation is not what you expect it to be (think slow kernel fix vs fast microcode).

Thanks a lot for your contribution!
The library aims to provide a simple interface towards the procfs/sysfs filesystems. The provided interface should mirror the naming of these filesystems, and take away the burden from go developers to parse the content on their own.
The function name should mirror the naming of file system names wherever possible. The full path is quite long, I propose CPUVulnerabilities() ([]CPUVulnerability, error).
Furthermore, the intent of this library is to take away the burden of parsing the specific file formats and providing idiomatic golang datatypes instead. Therefore, I suggest to return a struct for each vulnerability instead of a map of strings which is not a clear interface.
type CPUVulnerability struct {
	CodeName   string
	State      string
	Mitigation string
}

Following the sysfs documentation at https://www.kernel.org/doc/Documentation/ABI/testing/sysfs-devices-system-cpu, CodeName would be the filename, State one of Not Affected, Vulnerable, or Mitigated, and Mitigation the value of $M. It might even make sense to define a type CPUVulnerabilityState and three constants for the three states to guarantee type safety for the state comparison.

I've updated the code. Let me know if that's what you had in mind and I'll work on docs and tests.

Looks like the tests failed due to missing copyright headers.

@bobrik Any chance you can rebase this one on latest master and add the copyright header?

I've created a new PR with these changes rebased on master.


Add python-shapely for wily and xenial

http://packages.ubuntu.com/xenial/python/python-shapely
http://packages.ubuntu.com/xenial/python/python3-shapely
Wily is EOL, but without it this seems incomplete.

There's Y.
However, I'm not sure, if there's something like yakkety_python3.


ImportError: cannot import name table_to_blockmanager

Is there a specific polars or pyarrow version needed?

I am getting this exception (cannot import name table_to_blockmanager) with polars '0.20.5' and pyarrow '10.0.1'.

Not sure if you're sstill having this issue, but I was able to resolve it by upgrading pyarrow. I went from 14.0.2 to 16.1.0 and it was resolved.


Block unauthenticated requests - Error 1272 or 5

Hi there, floyd from Pentagrid here (the "POISIONDAMAGE" guys)
I tried to use concealed position and it worked fine until I tried to install the printer with cp_client.exe. It seems deactivating the password protected sharing in the advanced sharing settings is breaking the exploit in this setup. The error message of cp_client.exe is:
Couldn't connect to the remote printer: 1272
Digging a little, I found that error 1272 is probably:
You can't access this shared folder because your organization's security policies block unauthenticated guest access. These policies help protect your PC from unsafe or malicious devices on the network.
And of course if I don't deactivate password protected sharing, I get:
Couldn't connect to the remote printer: 2
Which is access denied, because cp_client.exe didn't authenticate.
Some more debug info. This line fails:
https://github.com/jacob-baines/concealed_position/blob/c41ef9b95fbce456f7685209db44004e5a7dbf0b/src/cp_client/main.cpp#L54
OpenPrinter doc is here and is called with the first argument (pPrinterName) being an UNC path (\<name>):
https://docs.microsoft.com/en-us/windows/win32/printdocs/openprinter
So the question is which component is responsible for authentication on the UNC paths. Probably the machine sees that it is a non-authenticated share. So we need authentication. The question is if there is something that allows to accept any NTLM/Kerberos authentication the Windows machine is doing.
Any chance to fix this? Any other ideas?
Edit: I'm going to try to mount the server first (aka "net use Z: \Server\SharedFolder passwordGoesHere /USER:userAccountGoesHere /persistent:no"), then try the exploit again, I guess the UNC authentication part is transparent Windows magic to OpenPrinter and if the printer is alread "mounted" that might work

You can work around that with runas /netonly supplying a user of your evil printserver.

Oh, that sounds plausible, why didn't I think of that. Thanks for the hint. Will need to try this. Unsure if runas was allowed in the environment though. If I get back on that I'll let you know if it worked.


Connect to Daly BMS via TTL/UART

Why not allow direct connection via TTL to your PI3/4 device ?
It is possible and you can read data directly from Daly BMS. No need for wireless.

This is not part of this project.
Some users don't have the Pi near their BMS and in my case, I have 12BMS plus another 10 in mobile 12V batteries which I need to monitor. Cabling impossible.

OGG, I just watched your video. Most of us don't have 22 BMS to monitor and as you found, Bluetooth just isn't reliable enough. I'm still looking for a good solution that can look at several BMS through RS485/TTL/CAN. for more reliable monitoring that doesn't require a MCU for each BMS. I currently run the standalone program, HybridInverterMon, which can monitor 1 JKBMS over RS485 (Im working on adding additional JKBMS but I'm not familiar with C# or DOTNET) and not any other brand BMS. If this project can be expanded to include RS485 communication it sounds good for even up to 10 BMS. I also don't know why peterboards require 1 board per BMS because I thought CAN also supports multi devices on a bus. You of course have your batteries all over the garage for testing and what not so cabling would be a mess but for someone like me who only plan on a few batteries in the same space, this would be ideal.

It is on my todo, since most BMS just have the BT dongle sit on the UART, we just need to change the transport layer, the protocol should be the same
also see https://github.com/fl4p/batmon-ha/issues/22


PageUsers rerendering in an infinite loop

Hello, I think I have to ask you for help, I have tried for several days, but I can't solve it, I hope I can get your guidance, thank you very much!
I try to switch mock server to real API , After the switch,  when I directly type http://localhost:3000/admin/management/users  in the browser address bar, will cause PageUsers rerendering in an infinite loop.

But if when I first go to http://localhost:3000/admin/dashboard and click the MainMenu Admin from the TopBar, then the browser will jump to PageUsers, and everything is perfect, without infinite rendering.

but when I click the refresh button in the browser, infinite rendering loop happens again.
My question is:

when I directly in the browser address bar, type http://localhost:3000/admin/management/users, can cause PageUsers rendering in an infinite loop, Is this a normal situation?
If not, could you please give me some guidance to solve the problem?
Thank you very much!

export const useUserList = ({ page = 0, size = 10 } = {}, config: any = {}) => {
  const [{ users, count: totalItems, hasMore }, { isLoading: isLoading, error, refetch }] =
    usePaginatedQuery(
      getUsers,
      {
        orderBy: { id: "asc" },
        skip: size * (page - 1),
        take: size,
      },
      {
        keepPreviousData: true,
        ...config,
      }
    )

  const totalPages = Math.ceil(totalItems / size)

  return {
    users,
    totalItems,
    hasMore,
    totalPages,
    isLoading,
  }
}

@winner106 I think you need to update the next.config.js to replace the /app/ with /admin/ in the redirect part ☺

Thank you for your guidance. I added the following code in next.config.js:
  async rewrites() {
    return [
      {
        source: '/admin/:any*',
        destination: '/admin/',
      },
    ];
  },

but Error ERR_TOO_MANY_REDIRECTS is displayed in the browser, and all the pages cannot be opened.
Sorry, Maybe I have something to add. I used next file-base route to catch all the admin routes, . I wonder if it has any impact?

Thanks!


feat(Algebra/Group/Equiv/TypeTags): moving Multiplicative inside pi types

Add equivalences to move Multiplicative and Additive into the codomain of a pi type, both dependent and non-dependent.
    Multiplicative (∀ i : ι, K i) ≃* (∀ i : ι, Multiplicative (K i))

    Multiplicative (ι → G) ≃* (ι → Multiplicative G)

    Additive (∀ i : ι, K i) ≃+ (∀ i : ι, Additive (K i))

    Additive (ι → G) ≃+ (ι → Additive G)

From AperiodicMonotilesLean.

 If the file can afford to import the elementwise operations on pi types, then I would expect it to do so for pairs too.

Mathlib.Algebra.Group.Prod actually imports a lot more than should be needed just to define the operations (Mathlib.Algebra.Group.Opposite and Mathlib.Algebra.Group.Units.Hom).

If you make the change and push the commit, then CI will count exactly how bad it is.

Done, though I think putting these definitions instead in Mathlib.Algebra.Group.Prod, with the new import going the other way, would mean many fewer new imports in total (but be less consistent as a location).

Looking at the numbers above, I think this is telling us that every file downstream that imports Mathlib.Algebra.Group.Equiv.TypeTags already imports Mathlib.Algebra.Group.Prod anyway, so the change only affects the dependencies of this one file.

bors r+


Remove $viewBasePath from ViewRenderer, add documentation

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
✔

Fixed issues
-

👍


Create Loaders for Major Music Streaming Services

Fixes # (issue)
Create loaders for library endpoints of major music streaming services
Before submitting

-
-

a test for the integration
❌ REST APIs

an example notebook showing its use
✅

twitter
Who can review?
Tag maintainers/contributors who might be interested:
@eyurtsev

@jsj thanks for this! i'm happy to review (was planning to do so today). any reason to close this?

Thanks!

I was just experimenting with LangChain

It was untested

It felt like to me there should just be some generic REST API doc loader

Apps of this were unclear. It’s just a text track list. Perhaps something like a CharacterAI yourself with music. Would be cool to have actual audio but not possible for obvious reasons


Custom date range doesn't work properly in reports

From user email:
I found a small bug where when viewing charts using custom range period, the chart always display the data until today.
So the beginning date is correct, but the data shown always include everything until today, and not until the specified end date.

I can confirm the problem. In my tests, there's also some problem with the start date. For some reason it always starts at the 16th of the month where the range starts.


Open/Close button in the middle bar for Info panel is barely visible when the table/map is fully expanded in Manager

Describe the bug
Test app:
https://solutions.mapsdevext.arcgis.com/apps/instant/manager/index.html?appid=e708aede521647a08b030ceb9049b0cf
Open/Close button in the middle bar for Info panel does not show up when the table (split view and table) or map (map view) is fully expanded, so there is no way to open the Info panel in this state. if I hover over at the bottom left, I can see the tooltip of the button.

Expected Behavior
The button should be visible for all the layouts when table or map is expanded
Reproduction Steps & Sample

Launch the test app
Click on the Close button
Observe that tip of the button is visible at the bottom left

Other Relevant Info
No response

Looks good in devext


train mineral shard example is not compatible with baselines library

I can't get started train_minteral_shards.py example. Getting this error:
$ python train_mineral_shards.py
Traceback (most recent call last):
File "train_mineral_shards.py", line 14, in
from common.vec_env.subproc_vec_env import SubprocVecEnv
File "D:\devel\repositories\pysc2-examples\common_init_.py", line 1, in
from baselines.common.console_util import *
ImportError: No module named 'baselines.common.console_util'
btw, I am using windows, can't it be problem? I've tried build baselines from repository, but I've failed on atari.py dependency (not very compatible with windows). baselines0.1.4 or lower libraries can be installed using pip, but seem to be not sufficient for example. Please help how to proceed.

You need to install baselines library from the baselines Github repo.
pip install git+https://github.com/openai/baselines

Try this!

ok, I've switched to another machine with linux and I was able to install baselines from repository. (baselines with version 0.1.4 does not work). Now I am getting error while running this script:
python3 train_mineral_shards.py --algorithm=deepq --prioritized=True --dueling=True --timesteps=2000000 --exploration_fraction=0.2 --num_cpu=4 --lr=0.0005
algorithm : deepq
timesteps : 2000000
exploration_fraction : 0.2
prioritized : True
dueling : True
num_cpu : 4
lr : 0.0005
Traceback (most recent call last):
  File "train_mineral_shards.py", line 289, in <module>
    main()
  File "train_mineral_shards.py", line 97, in main
    visualize=True) as env:
  File "/home/snurkabill/.local/lib/python3.5/site-packages/pysc2/env/sc2_env.py", line 116, in __init__
    raise ValueError("All arguments must be passed as keyword arguments.")
ValueError: All arguments must be passed as keyword arguments.

also for example:
algorithm : acktr
timesteps : 2000000
exploration_fraction : 0.5
prioritized : True
dueling : True
num_cpu : 4
lr : 0.0005
Process Process-1:
Traceback (most recent call last):
  File "/usr/lib/python3.5/multiprocessing/process.py", line 249, in _bootstrap
    self.run()
  File "/usr/lib/python3.5/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/home/snurkabill/dev/repositories/pysc2-examples/common/vec_env/subproc_vec_env.py", line 15, in worker
    step_mul=1) as env:
  File "/home/snurkabill/.local/lib/python3.5/site-packages/pysc2/env/sc2_env.py", line 116, in __init__
    raise ValueError("All arguments must be passed as keyword arguments.")
ValueError: All arguments must be passed as keyword arguments.
Process Process-2:
Traceback (most recent call last):
  File "/usr/lib/python3.5/multiprocessing/process.py", line 249, in _bootstrap
    self.run()
  File "/usr/lib/python3.5/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/home/snurkabill/dev/repositories/pysc2-examples/common/vec_env/subproc_vec_env.py", line 15, in worker
    step_mul=1) as env:
  File "/home/snurkabill/.local/lib/python3.5/site-packages/pysc2/env/sc2_env.py", line 116, in __init__
    raise ValueError("All arguments must be passed as keyword arguments.")
ValueError: All arguments must be passed as keyword arguments.
Process Process-3:
Traceback (most recent call last):
  File "/usr/lib/python3.5/multiprocessing/process.py", line 249, in _bootstrap
    self.run()
  File "/usr/lib/python3.5/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/home/snurkabill/dev/repositories/pysc2-examples/common/vec_env/subproc_vec_env.py", line 15, in worker
    step_mul=1) as env:
  File "/home/snurkabill/.local/lib/python3.5/site-packages/pysc2/env/sc2_env.py", line 116, in __init__
    raise ValueError("All arguments must be passed as keyword arguments.")
ValueError: All arguments must be passed as keyword arguments.
Process Process-4:
Traceback (most recent call last):
  File "/usr/lib/python3.5/multiprocessing/process.py", line 249, in _bootstrap
    self.run()
  File "/usr/lib/python3.5/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/home/snurkabill/dev/repositories/pysc2-examples/common/vec_env/subproc_vec_env.py", line 15, in worker
    step_mul=1) as env:
  File "/home/snurkabill/.local/lib/python3.5/site-packages/pysc2/env/sc2_env.py", line 116, in __init__
    raise ValueError("All arguments must be passed as keyword arguments.")
ValueError: All arguments must be passed as keyword arguments.

Can you point me forward one more time, please?

Hi, @chris-chris . I've tried modify your examples but I somehow failed. Could you please provide solution to my problem? I would be very grateful :)

@snurkabill
Try adding
_only_use_kwargs=False
to constructor "with sc2_env.SC2Env("
Idk what is purpose of this code >.>

@snurkabill
Can you pull the latest code and try it again?
I fixed this issue recently :)

I still got that errors:
Traceback (most recent call last):
File "train_mineral_shards.py", line 13, in
from common.vec_env.subproc_vec_env import SubprocVecEnv
File "/home/howard/pysc2-examples/common/init.py", line 1, in
from baselines.common.console_util import *
ImportError: No module named 'baselines.common.console_util'

@mcdavid109
pip install git+https://github.com/openai/baselines
same thing "ImportError: No module named 'baselines.common.console_util'"
so, I tried

Step : git clone https://github.com/openai/baselines.git
Step : pip install -e . (in baselines directory)

and found 'baselines.common.console_util'
Environment : win10 + bash..  // bash app is very honey!

@Faur
I had this "mpi4py" problem too and I solved it this way:
1 sudo apt-get install libopenmpi-dev openmpi-bin openmpi-doc
2 pip install mpi4py

I have the same exact issue with snurkabill.
ImportError: No module named 'baselines.common.console_util'
I am using win10. Do I have to change to Linux?

@gokimsky no. I have the example running. I am running anaconda with a tensorflow-gpu installation on windows 10.
If you install the current atari-py first from their git repo and then baselines you do not need mpi:
git clone git@github.com:openai/atari-py.git
cd atari-py
pip install -e .
cd ..
git clone git@github.com:openai/baselines.git
cd baselines
pip install -e .
cd ..

The second problem is that the baselines API has changed, so the code here needs some changes.
I got the defeat_zerglings and enjoy_mineral_shads files running in my fork for now, take a look at my changes here:
https://github.com/chris-chris/pysc2-examples/compare/master...brean:baseline_changes
(I am not that experienced with deep learning yet so I will not directly provide a pull request and let you guys take a look at my code first, so feel free to copy stuff from it and merge it with this repo).

when i run the

python3 train_mineral_shards.py --algorithm=a2c
I got a similar problem,but this is the problem of my tensorflow.
The output is that:

Traceback (most recent call last):
File "train_mineral_shards.py", line 5, in
from baselines import deepq
File "/home/limenglin519/pysc2-examples/baselines/baselines/deepq/init.py", line 1, in
from baselines.deepq import models  # noqa
File "/home/limenglin519/pysc2-examples/baselines/baselines/deepq/models.py", line 2, in
import tensorflow.contrib.layers as layers
ImportError: No module named 'tensorflow.contrib'
I wuold be grateful ,if u can help me to solve this.


Enforce password confirmation when creating account

On the latest version (current develop branch), password confirmation is not required when creating an account. This should probably be fixed.

What do you mean by password confirmation? We don't have a "Enter the password again". This can be seen in many applications these days. The user should backup the seed phrase with which he can import the account again if he forgot the password.

Yes, I'm talking about entering the password again. It's not about preventing the user from forgetting the password, it's just a way to limit typos.

You are right, but I think for most users it would be a hassle.

If they wrote down the seed words, they can recover it that way. Although maybe we should be forcing them to prove they really wrote it down.

I strongly believe in having users confirm passwords-- it isn't an unnecessary point of friction for an end-user, especially if it acts as a check that down on access issues due to typo.
Security UI/UX is overwhelmingly important to get right around authentication, so having a confirmation is a good way to help us plan for when users make mistakes or do things with our software that we did not intend. (I.e. failing to write down a seed phrase or store it somewhere safely.)

Although maybe we should be forcing them to prove they really wrote it down.

interesting idea! @jessysaurusrex what do you think about this?
re: confirming password - i guess we might as well. could save a lot of people a lot of headaches.

It's interesting idea, but I cannot see a feasible way to accurately measure or enforce this security behavior in a way that it provides a benefit and does not introduce myriad problems on our end. At some point, security is a shared responsibility: we can (and should) encourage people to do the right thing and we should make it easy and we can make use of UI to do so in other places and in other ways.
But how do we get proof from the user in a way that doesn't feel like we're being bossy parents to the people who have invested in our work? Do we ask them to click through a check box saying that they've written something down, or serve a security warning that is more likely to be ignored than heeded? Do we ask someone to submit proof of writing it down by taking a picture of their password (a behavior we never want users to engage in) which is something that we can't quite verify or validate easily on our end?
tl;dr I think the confirmation box is sufficient. :)


Unresolved comment was collapsed on file open

Testing #194015
I reloaded the window, opened a PR file with an unresolved comment, and the comments panel opened ✅ but the comment itself had been collapsed.

Comments from you will always start collapsed: https://github.com/microsoft/vscode-pull-request-github/issues/3452


Validate input as="select"

I am using ember-easyform in conjunction with this plugin and it is working great so far. Though I just don't seem to be able to get it to validate a select-input. I am giving it a presence: true validator but even when I select something I am still getting the error 'can't be blank`.
Is there any way to get this to work with selects?

I am going to close this issue for now, feel free to reopen this issue if you feel like it's still relevant and we should take a look at it.


Field restrictions

morphling should support some kind of field value restrictions (e.g. "this string should be a most 30 characters long", "this int should be between 0 and 42") via HCofree.

Done in #10


Dynamically update sender parameters

The intension for this change is to fix that:
Previously, if there are no dimensions info, simulcast would never be enabled.
With this change simulcast will always be enabled (if its true in the options),
and the sender parameters will lazily be updated when dimensions info
is provided by the capturer. (even when dimensions change)
Many of the capturers cannot report "real dimensions" until they produce a frame,
it's only possible to estimate (which is in accurate)
Thoughts
What should be the desired behavior when the user provides VideoEncoding(max frameRate, maxBitrate) in the publish options, and Simulcast is enabled.

Thanks oniichan! I will need to re-read your Slack comments & JS PR !! 😁🙏


bin/kyuubi-zk-cli should has execute permission

Why are the changes needed?
bin/kyuubi-zk-cli should has execute permission
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1748 (cba11d0) into master (01f0ea8) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1748   +/-   ##
=========================================
  Coverage     59.52%   59.53%
- Complexity      232      233    +1
=========================================
  Files           268      268
  Lines         13315    13315
  Branches       1692     1692
=========================================
+ Hits           7926     7927    +1
  Misses         4709     4709
+ Partials        680      679    -1

Impacted Files
Coverage Δ

...ache/kyuubi/operation/KyuubiOperationManager.scala
95.00% <0.00%> (+2.50%)
:arrow_up:

Continue to review full report at Codecov.

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

thanks, merged to master and v1.4.1


Added support for all ChromeOptions features

Hello,
I need to pass the "debuggerArgument" to the ChromeOptions through the browserOptions argument of the Open Browser keyword.
This argument let Chrome to attach it to another browser (e.g. chromium) and control it with the regular keyword set.
The PR contains the changes in order to allow any unknown property to be correctly routed in the ChromeOptions class without requiring new releases of this project.
When a setting become consolidated in the ChromeOptions, the code can be then changed in order to use the appropriate property.
Thanks, Luciano

Including this to version <IP_ADDRESS>


opencv4 build error on windows

summary
Error: Building package opencv4:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
Package: opencv4:x64-windows
Vcpkg version: 2020.11.12-nohash
details
C:\vcpkg-master> .\vcpkg.exe install darknet[full]:x64-windows
Computing installation plan...
The following packages will be built and installed:

darknet[core,cuda,cudnn,full,opencv-cuda,weights,weights-train]:x64-windows -> 2020-08-30#1
opencv[core,cuda,dnn,ffmpeg,jpeg,opengl,png,tiff,webp]:x64-windows -> 4.3.0
opencv4[contrib,core,cuda,dnn,ffmpeg,jpeg,opengl,png,tiff,webp]:x64-windows -> 4.3.0#4
pthreads[core]:x64-windows -> 3.0.0-6
stb[core]:x64-windows -> 2020-09-14
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...

-- Using cached C:/vcpkg-master/downloads/opencv-opencv-4.3.0.tar.gz
-- Cleaning sources at C:/vcpkg-master/buildtrees/opencv4/src/4.3.0-58520c6ae3.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/vcpkg-master/downloads/opencv-opencv-4.3.0.tar.gz
-- Applying patch 0001-disable-downloading.patch
-- Applying patch 0002-install-options.patch
-- Applying patch 0003-force-package-requirements.patch
-- Applying patch 0004-fix-policy-CMP0057.patch
-- Applying patch 0006-fix-vtk9.patch
-- Applying patch 0006-jpeg2000_getref.patch
-- Applying patch 0009-fix-uwp.patch
-- Applying patch 0010-fix-interface_link_libraries.patch
-- Using source at C:/vcpkg-master/buildtrees/opencv4/src/4.3.0-58520c6ae3.clean
-- Using cached C:/vcpkg-master/downloads/opencv-cache/tiny_dnn/adb1c512e09ca2c7a6faef36f9c53e59-v1.0.0a3.tar.gz
-- Using cached C:/vcpkg-master/downloads/opencv-cache/nvidia_optical_flow/ca5acedee6cb45d0ec610a6732de5c15-79c6cee80a2df9a196f20afd6b598a9810964c32.zip
-- Using cached C:/vcpkg-master/downloads/opencv-opencv_contrib-4.3.0.tar.gz
-- Cleaning sources at C:/vcpkg-master/buildtrees/opencv4/src/4.3.0-1c7c3cc613.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/vcpkg-master/downloads/opencv-opencv_contrib-4.3.0.tar.gz
-- Applying patch 0005-add-missing-stdexcept-include.patch
-- Applying patch 0007-fix-vtk9-contrib.patch
-- Using source at C:/vcpkg-master/buildtrees/opencv4/src/4.3.0-1c7c3cc613.clean
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/0ea90e7a8f3f7876d450e4149c97c74f-boostdesc_bgm.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/232c966b13651bd0e46a1497b0852191-boostdesc_bgm_bi.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/324426a24fa56ad9c5b8e3e0b3e5303e-boostdesc_bgm_hd.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/202e1b3e9fec871b04da31f7f016679f-boostdesc_binboost_064.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/98ea99d399965c03d555cef3ea502a0b-boostdesc_binboost_128.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/e6dcfa9f647779eb1ce446a8d759b6ea-boostdesc_binboost_256.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/boostdesc/0ae0675534aa318d9668f2a179c2a052-boostdesc_lbgm.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/vgg/e8d0dcd54d1bcfdc29203d011a797179-vgg_generated_48.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/vgg/7126a5d9a8884ebca5aea5d63d677225-vgg_generated_64.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/vgg/7cd47228edec52b6d82f46511af325c5-vgg_generated_80.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/xfeatures2d/vgg/151805e03568c9f490a5e3a872777b75-vgg_generated_120.i
-- Using cached C:/vcpkg-master/downloads/opencv-cache/data/7505c44ca4eb54b4ab1e4777cb96ac05-face_landmark_model.dat
-- Configuring x64-windows
-- Building x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:145 (message):
Command failed: C:/vcpkg-master/downloads/tools/cmake-3.18.4-windows/cmake-3.18.4-win32-x86/bin/cmake.exe --build . --config Debug --target install -- -v -j41
Working Directory: C:/vcpkg-master/buildtrees/opencv4/x64-windows-dbg
See logs for more information:
C:\vcpkg-master\buildtrees\opencv4\install-x64-windows-dbg-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_build_cmake.cmake:96 (vcpkg_execute_build_process)
scripts/cmake/vcpkg_install_cmake.cmake:27 (vcpkg_build_cmake)
ports/opencv4/portfile.cmake:362 (vcpkg_install_cmake)
scripts/ports.cmake:136 (include)
Error: Building package opencv4:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
Package: opencv4:x64-windows
Vcpkg version: 2020.11.12-nohash
Additionally, attach any relevant sections from the log files above.

#15055

@dorbodwolf ,Thanks for posting this issue,Duplicate to #15055


Cannot create note because note is empty

I'm using JP mining note and I've been trying all day to troubleshoot this and have reinstalled everything multiple times, but whenever I attempt to create a new anki card, it just says the title.

Read this and see if it helps you: https://github.com/killergerbah/asbplayer/issues/220#issuecomment-1501124166

No response


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1746 ms

Resolved: Bofrost.lu performance has improved in e8e35c1 after 49 minutes.


Add a license to the repository

I recently came across your project Quantum-Computing-Qiskit and I noticed that the repository github.com/minnukota381/Quantum-Computing-Qiskit currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

I have fixed this in : 3ad2e4d89732a3af0bbf801558a9e0dc590153d1

Thanks you so much for your contribution sir!

Thank you!

Sir! I’m currently working on a few small projects in quantum computing, and I’d love to hear more about what you’re working on at IBM and how quantum is being applied in real-world scenarios. Could you please connect with me sir?


gpio : Add support for input pull up/down modes

Make use of 'active_low' interface in sysfs for configuring input pin
in pull up / pull down mode. C++ binding also has been added.
Signed-off-by: Manivannan Sadhasivam<EMAIL_ADDRESS>

Merged! Thanks alot


Documentation issue

Hi,
Please check following URL...
https://github.com/cemdervis/SharpConfig/wiki/How-to-add-custom-object-string-conversions
In ConvertToString(), '{', '}' is used to delimit string, but in ConvertFromString(), '[', ']' is used to trim/parse it.
Is it type error or my confusion?
2nd issue is, since it is generic class, why 'object' is used as value in input and as return type?
Thanks

Hey Ketan-Desai,
thanks for submitting this issue.

The documentation is indeed incorrect, it should be "[{0};{1}]".
The class itself is not generic, but its base class is, see here.
This is so that you don't have to implement the "ConvertibleType" property (and other things in the future) in your own class. Object is passed because it has to be, there is no other general way. Generic type parameters are out of the question, because Setting.SetValue(object) exists and is the preferred method to use. The generic overloads of SetValue may be removed in the future, as they have provided little benefit.

Fixed the documentation.


Explain how to add a new model

Closes #525

It's ok for me @PierreGtc. But as this is core, related to what you have recently implemented, and involves an important part of the contributor part, I'm also asking Alex and Robin to review it as well.

And @sliwy and @martinwimpff :)

Thanks already for the explanations. In general, I think we need to try to make braindecode usage as simple and transparent and possible. Some concrete questions:

is each and every of these mandatory parameters necessary?
is it necessary that each of these is specified via ... =None?
there are use cases where one wants a model that outputs batch_size,n_times,n_targets, e.g. n_targets maybe number of classes to predict or variables to regress. This explanation seems in conflict with that? What is n_outputs exactly meant to be?

is each and every of these mandatory parameters necessary?
It won’t fail if the class doesn’t have all the parameters but I think it’s simpler for the users if all the models allow for all these parameters.
is it necessary that each of these is specified via ... =None?
They can have a not-None default value.
there are use cases where one wants a model that outputs batch_size,n_times,n_targets, e.g. n_targets maybe number of classes to predict or variables to regress. This explanation seems in conflict with that? What is n_outputs exactly meant to be?.
Hmmm I did not know we had this case with 3D outputs.
Is this the cropped decoding?
Which models can have a 3D output?

n_outputs replaced  n_classes. I changed it to also make sense if the model is used for regression.

In general one may want to have a temporal dimension in the output for segmentation-like tasks and we do that as well e.g., here https://braindecode.org/dev/auto_examples/advanced_training/plot_bcic_iv_4_ecog_cropped.html#sphx-glr-auto-examples-advanced-training-plot-bcic-iv-4-ecog-cropped-py.
To me this is not only in case one does cropped decoding way we do it, in general, you can have scenarios where you want a time dimension in your output, like if you want to temporally segment the signal.

Do you have ideas on how to explain this n_outputs parameters @robintibor?

Thanks for the work and great explanations. I feel might be more sensible to put this at the end of the CONTRIBUTING.md file rather than somewhere in the middle... if you want to make it more visible you could add a pointer at the point it is currently written to say this part (how to add a model) comes at the end... Otherwise to me in current form it breaks a bit the structure as it is quite long and specific to one type of contribution. Also felt it might be nice to add a bit more info about the mandatory parameters meaning, maybe copy their docstring there and make clear that some of them may not be used/needed by the model and that that is fine. What do you think?

Yes I agree, it's much longer than the other sections. I put it at the end and added the definitions of the parameters.

For me, it looks good, and we already have two approvals. As this document is in GitHub only and still needs to be incorporated into the documentation (in another PR). I will apply the merge for now and work on adjusting it further in another PR that I will do. I hope you don't mind @robintibor.

Thank you @PierreGtch!


Added K Framework

Currently only has support for 64-bit Linux

Squash all your commits firstly. (Do rebase -i and push --force)

Fails to build: http://hydra.nixos.org/build/25067189/nixlog/1/raw

Reverted in 69b648e (master) and 9b22f38 (release-15.09).

I'm currently waiting on an issue in mvn2nix (https://github.com/NixOS/mvn2nix-maven-plugin/issues/4) to fix this package. Once that's solved, I'll open another pull request for the new K Framework package.

Ping? I think your referenced issue has been fixed upstream since the "multiple modules" bit was merged in a few months ago.
Unfortunately I'm currently getting an error that seems to be due to lack of "sha1" files for various resources, but I've never used maven before so I'm not sure if there's a good way around that or not :).
Anyway :+1: there definitely should be a k-framework Nix package! :)
I've had some success (I think) packaging up 4.0.0 from a release tarball but a)it'd be better to "build" it ourselves I think, and b)my use actually requires a newer version from git.
Anyway, if you don't mind looking into this again that'd be great, let me know if there's anything I can do to help or test!

@dtzWill I see that you're in Urbana; would you be interested in meeting up IRL to talk about this? (though I am fairly busy over the next few weeks).

(we connected on IRC :))

Better yet have a nix definition implemented in K. Possible to get this package in?


Configure Azure Pipelines

This configuration builds the application and produces builds for both Debug and Release configurations.

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


Why Download center keeps saying "No Internet Connection", can't download any model...

Version: 5.5.0
Plateform: Win10

Please check the error log and paste the output here.

I went through 'C:\Users\me\AppData\Local\Programs\Ultimate Vocal Remover' directory, didn't find any log file, any hint please?

OK, here is it:
Last Error Received:
Process: Online Data Refresh
If this error persists, please contact the developers with the error details.
Raw Error Details:
URLError: "<urlopen error [Errno 11004] getaddrinfo failed>"
Traceback Error: "
File "UVR.py", line 3440, in online_check
File "urllib\request.py", line 214, in urlopen
File "urllib\request.py", line 517, in open
File "urllib\request.py", line 534, in _open
File "urllib\request.py", line 494, in _call_chain
File "urllib\request.py", line 1389, in https_open
File "urllib\request.py", line 1349, in do_open
"
Error Time Stamp [2022-12-31 11:57:29]
seems that GFW blocked github address " https://raw.githubusercontent.com/TRvlvr/application_data/main/filelists/download_checks.json"
I am going to test it through a VPN.

@czhou Excuse me but would you mind teaching me how to fix it? I failed even I have setup global proxy😭.

@czhou Excuse me but would you mind teaching me how to fix it? I failed even I have setup global proxy😭.

I have fixed it. I edit the hosts file and it works.

What lines to edit the hosts files for solving No internet Connection error? can you please share?

I am experiencing the same problem, please let me know how to fix it, thanks!


Use allele change log to identify errors that escaped detection

If most alleles in a session have errors and some don't, those might be mistakes as well and be correct by chance.

duplicate of #34


[FEATURE] command, tcpSocket and grpc livenessProbe and readinessProbe support

Detailed description
It would be ideal to support the command, tcpSocket and grpc methods for livenessProbe and readinessProbe.
Context
Today, only the httpGet method for livenessProbe and readinessProbe is supported.
command is another very popular method, like illustrated here with grpc apps: https://github.com/GoogleCloudPlatform/microservices-demo/blob/main/release/kubernetes-manifests.yaml#L61.
So for example, today, this below is not supported:
apiVersion: score.dev/v1b1
metadata:
  name: adservice
containers:
  adservice:
    image: gcr.io/google-samples/microservices-demo/adservice:v0.6.0
    variables:
      PORT: "9555"
    livenessProbe:
      exec:
        command:
          - /bin/grpc_health_probe
          - '-addr=:9555'
    readinessProbe:
      exec:
        command:
          - /bin/grpc_health_probe
          - '-addr=:9555'

Possible implementation
Humanitec already supports tcpSocket and command in addition to httpGet.
Note: gRPC (only supported since Kubernetes 1.24+) is not yet supported in Humanitec. The alternative is to use the command method by using the grpc_health_probe binary.
Additional information
Configure Liveness, Readiness and Startup Probes in Kubernetes.

The most important one here is the command one IMHO. Because it allows folks to embed a binary and simulate the same http/grpc/tcp probes in a general way.


capture git versioning information in manifest

For every generated jar, capture git versioning information in Java manifest.
This allows to tell at runtime exact git commit id that this jar was built from.

This will generate MANIFEST.MF with contents like this:
Manifest-Version: 1.0
Implementation-Title: presto-main
Implementation-Version: 0.101-SNAPSHOT
Archiver-Version: Plexus Archiver
Built-By: vadimm
Specification-Title: presto-main
Implementation-Vendor-Id: com.facebook.presto
Created-By: Apache Maven 3.3.1
Build-Jdk: 1.8.0_40
Specification-Version: 0.101-SNAPSHOT
Implementation-URL: https://github.com/facebook/presto/presto-main
Name: com.facebook.presto:presto-main
Git-Branch: mf-git-info
Git-Commit-Id-Describe: 0.100-95-ge1d65c0
Git-Remote-Origin-Url: https://github.com/facebook/presto
Build-Time: 2015-04-08T17:54:58-0700
Git-Commit-Id: e1d65c02a1d34e2b9aa1cdd925852b347a11fb49

Changed this to not use manifest sections, and also removed git branch and git remote.

Now manifest looks like this:
Manifest-Version: 1.0
Implementation-Title: presto-main
Implementation-Version: 0.101-SNAPSHOT
Archiver-Version: Plexus Archiver
Built-By: vadimm
Specification-Title: presto-main
Implementation-Vendor-Id: com.facebook.presto
Git-Commit-Id: a57b4a659426a395aa59f8a53be9698673f1e148
Git-Commit-Id-Describe: 0.100-95-ga57b4a6
Build-Time: 2015-04-08T18:10:26-0700
Created-By: Apache Maven 3.3.1
Build-Jdk: 1.8.0_40
Specification-Version: 0.101-SNAPSHOT
Implementation-URL: https://github.com/facebook/presto/presto-main

Per Martin request, changed it again to use Git-Commit-Id-Describe: 0.100-95-ga57b4a6 as Implementation-Version.
Manifest now looks like this:
Manifest-Version: 1.0
Implementation-Title: presto-main
Implementation-Version: 0.100-106-g7ddd3fc
Archiver-Version: Plexus Archiver
Built-By: vadimm
Specification-Title: presto-main
Implementation-Vendor-Id: com.facebook.presto
Git-Commit-Id: 7ddd3fc0e0b5aa96abb50653552902ec458a6dee
Build-Time: 2015-04-09T14:18:43-0700
Created-By: Apache Maven 3.3.1
Build-Jdk: 1.8.0_40
Specification-Version: 0.101-SNAPSHOT
Implementation-URL: https://github.com/facebook/presto/presto-main
Also, all places that use version, they use this code:
version = PrestoServer.class.getPackage().getImplementationVersion();
which now returns "0.100-106-g7ddd3fc" instead of 0.101-SNAPSHOT.
Also, "presto-cli --version" now returns:
Presto CLI 0.100-106-g7ddd3fc

This pull request can be obsolete if this one is approved instead:
https://github.com/airlift/airbase/pull/61

This is now included in airbase


plumbing/revlist: input as a slice of hashes instead of commits

Now the input of the method Objects inside revlist package is a slice of hashes instead of commits. Hashes can be from Blobs, Trees and Commits objects.
ObjectStorer now is used to obtain the object content using hashes slice.
This PR fix #222. Now a test into upload_pack_test.go file is not skipped anymore.
Remove code from remote.go and server.go that is not necessary.

Codecov Report

Merging #292 into master will increase coverage by <.01%.
The diff coverage is 76.08%.

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
+ Coverage   77.03%   77.03%   +<.01%
==========================================
  Files         116      116
  Lines        7902     7904       +2
==========================================
+ Hits         6087     6089       +2
  Misses       1161     1161
  Partials      654      654

Impacted Files
Coverage Δ

plumbing/transport/server/server.go
58.21% <100%> (-2.42%)
:x:

remote.go
69.61% <100%> (+0.38%)
:white_check_mark:

plumbing/revlist/revlist.go
78.78% <71.05%> (+5.2%)
:white_check_mark:

Continue to review full report at Codecov.

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


[Bug]:  Same name can be added to multiple tab in TABS widget

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The user can add the same name to the tab and we are not evaluating to show an error message this brings in conflict in identifying the Default Tab the
Steps To Reproduce

Add a Tab widget
Add multiple tab
rename the tab. with same name
and observe

Environment
Release
Version
Cloud

This is the name and not the key/id, Allowing duplicate is a general practice


Upgrade handler v0.6.8

Include a higher max wasm query gas limit to 5M (currently 2M hardcoded)

Include the StoreUpgrade for deleted module interchainqueries in the official v0.6.8
We need to do this because it was done in v0.6.7 which was only applied on our dev environments and we will now have a v0.6.8-rc for them, where v0.6.8 will include the storeUpgrade and be applicable only for mainnet


this seems to be not working with updates

I used it before and it worked. now, I am using python 2.7.3, pandas 0.13.1. it gives no result by calling summary(). positions, prices, report, equity, trades are empty. but trade_price is fine.

I think it is.


Datastreams index template creation or check for existence

Resolves : #284
Current behaviour :
When data streams is enabled, a new index template is created by default
New behaviour :
If data.streams.existing.index.template.name is provided, and if that index template does not exist, a template will be created with that name, else no template is created.
If data.streams.existing.index.template.name is not provided, a default template is always created

@reta @willyborankin can any of you ptal ?

@reta thank you for the great review.


bugfix: random label assignment

np.unique(support_y) will sort the unique values. Hence, the same label was assigned to the same class more often than random.

@lmzintgraf Have you tested it yet?

Yes; my model got 30% accuracy pre-update in the inner loop, i.e., when I gave it a new batch of classes with (seemingly) random labels during training, and before doing any updates on that data. After shuffling the labels I now get 20% as expected.
Here's an illustration of what's happening: Let's say the 64 training classes have "class labels" c_0-c_64. When sampling a task, you randomly sample 5 classes from these, and assign them the "target labels" t_0-t_5. E.g., let's say I picked images from the classes [c_38, c_0, c_15, c_3, c_35]. Because np.unique() sorts them, we would assign [c_0=t_0, c_3=t_1, ..., c_38=t_4]. This means for example that true class c_0 always gets the label t_0. More general, lower class labels have a higher chance of getting low target labels, and higher class labels have a higher chance of getting high target labels.


Make constructor of ListAssert that takes a Stream public

Summary
The constructor ListAssert(java.util.stream.Stream<? extends ELEMENT> actual) is currently protected. Due to constraints of Qulice and Elegant Objects (i.e. no static methods allowed), I'd like to be able to use the constructors. Their visibility must be public in order for them to be used anywhere. Please make these constructors public.
Example
new ListAssert<>(
    new ArrayList<>().stream()
).isEmpty();

Done.

Wow you're quick! 🥇 Thanks!


Nothing Happens after Start

What are the steps to reproduce this issue?
I have the following config
module.exports = {
    api: {
        output: {
            workspace: "src/",
            schemas: "src/services/models",
        },
        input: {
            target: "https://qa.betsybot.xyz/betsyapi-qa/swagger/v1/swagger.json",
        },
    },
};

I run npx orval@latest
I get the following output
> npx orval@latest

🍻 Start orval v6.15.0 - A swagger client generator for typescript

And nothing else.
What happens?
Nothing happens after starting
What were you expecting to happen?
Models to be generated.
Any logs, error output, etc?
N/A
Any other comments?
Not really sure what else to add here.
Happy to answer any questions that come through.
I am running a AMD 5900x Processor
What versions are you using?
Operating System: Window 10
Package Version: Orval @ Latest - Node @ v18.15.0
Browser Version: Chrome 113

Hi, i did some investigations on this. Here is what I found out for now:

The definition is loaded from the url correctly
Same behaivour when using the same definition as a local file
The input transformer receives the openapi spec as expected

Could it be that there is a circular dependency somewhere? There is definitely some processing going on when looking into the task manager.

I did an investigation yesterday night and found where is the problem in the code.  If you check here I already tried to avoid that case but apparently in your case that doesn't work properly

Same here.
I downgraded to version 6.13.1 and it solved

I think this is the same as #967

Duplicate of https://github.com/anymaniax/orval/issues/967 so  I will leave that ticket open if you want to subscribe to that ticket.