School informations (#53)

Unrestrict login page and change rules order

Codecov Report

Merging #54 (11624f1) into master (e9b7bfe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #54   +/-   ##
=======================================
  Coverage   93.76%   93.76%
=======================================
  Files          90       90
  Lines         930      930
  Branches       43       43
=======================================
  Hits          872      872
  Misses         58       58

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 e9b7bfe...11624f1. Read the comment docs.

Super, je devais faire le fix, merci 😁


update runv vendor

update runv to let hyperhq/runv#464 in, fix #573
I'll open another PR later to do cleanup stuff.
/cc @gnawux
Signed-off-by: Crazykev<EMAIL_ADDRESS>

This is also work for #575 / #570

"github.com/opencontainers/image-spec" is updated in Godeps.json, however, no related file is updated. Should it be removed in Godeps.json?

I'm not sure how this happened, but the current one seems more reasonable, for I grep in hyperd repo, we didn't import github.com/opencontainers/image-spec directly
crazykev@CK:~/go-project/src/github.com/hyperhq/hyperd$ grep -r "github.com\/opencontainers\/image-spec"
image/tarexport/load.go:	ociv1 "github.com/opencontainers/image-spec/specs-go/v1"
image/tarexport/save-oci.go:	imgspec "github.com/opencontainers/image-spec/specs-go"
image/tarexport/save-oci.go:	ociv1 "github.com/opencontainers/image-spec/specs-go/v1"
Binary file hyperd matches
Godeps/Godeps.json:			"ImportPath": "github.com/opencontainers/image-spec",
Binary file .git/index matches
vendor/github.com/opencontainers/image-spec/specs-go/v1/image_index.go:import "github.com/opencontainers/image-spec/specs-go"
vendor/github.com/opencontainers/image-spec/specs-go/v1/manifest.go:import "github.com/opencontainers/image-spec/specs-go"

Seems this TestSuite.TestSendContainerSignal test have high possibility fail to response...

@Crazykev looks like the container process quit before killed...

retest this please, hykins

looks like an runV issue, APIVersion Command result error before it is sent to hyperstart. Will file an issue in runV.

retest this please, hykins

good to find that.

The issue is come from the test case itself. It should wait before issuing a kill signal, otherwise, it may wait forever because the container has already exist.

#578 is aiming to fix the test case

Update, the issue should be an hyperstart issue ( hyperhq/hyperstart#285 ) if my further investigation is correct


avoid mapping device while start pause image

container would start failed ,while mapping device in pause image.

I have registered in linuxfoundation, what should i do next ?

@Eric-918 , thanks for your PRs; but I think this PR is for k8s and update godep here :).

k8s get device info from the config , I don't think this pr suits for k8s . Mapping device by this way is just an expedient for kube-mesos, more work need to do, include on mesos.

old PR, i'd like to close it.


wish: bypass / configure filename test?

Could an option be exposed to skip the /\.(js|jsx|es6)$/ test, or configure the regular expression? I am using javascript transformed/extracted from HTML, so this skips many of the files I want to lint. The behaviour is a little surprising also, IMO.
In case this helps anyone reading, my current workaround is to fool it:
        .transform({ continuous: true }, function (file, options) {
            return eslintify(file + '.js', options);
        })

Thanks for a very useful plugin, Evan et al.

Interesting use case. I could implement an opt.extensions, --extension combo similar to how top-level Browserify supports custom extensions if that would solve your need.
Like:
browserify x.js -t [ eslintify --continuous --extension html --extension haml ]

Programmatically like:
{ continuous: true, extensions: ["html", "haml"] }

This would be purely additive to the normal js/jsx/es6 it looks for.

Landed in 1.1.0, thanks!


Can't Install the gem

$ gem install remacs
Fetching: curses-1.2.0.gem (100%)
Building native extensions.  This could take a while...
ERROR:  Error installing remacs:
	ERROR: Failed to build gem native extension.

    /home/akamal/.rbenv/versions/2.1.3/bin/ruby extconf.rb
checking for tgetent() in -ltinfo... yes
checking for ncursesw/curses.h... no
checking for ncurses.h... no
checking for ncurses/curses.h... no
checking for curses_colr/curses.h... no
checking for curses.h... no
*** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
	--with-opt-dir
	--without-opt-dir
	--with-opt-include
	--without-opt-include=${opt-dir}/include
	--with-opt-lib
	--without-opt-lib=${opt-dir}/lib
	--with-make-prog
	--without-make-prog
	--srcdir=.
	--curdir
	--ruby=/home/akamal/.rbenv/versions/2.1.3/bin/ruby
	--with-curses-dir
	--without-curses-dir
	--with-curses-include
	--without-curses-include=${curses-dir}/include
	--with-curses-lib
	--without-curses-lib=${curses-dir}/lib
	--with-ncurses-dir
	--without-ncurses-dir
	--with-ncurses-include
	--without-ncurses-include=${ncurses-dir}/include
	--with-ncurses-lib
	--without-ncurses-lib=${ncurses-dir}/lib
	--with-termcap-dir
	--without-termcap-dir
	--with-termcap-include
	--without-termcap-include=${termcap-dir}/include
	--with-termcap-lib
	--without-termcap-lib=${termcap-dir}/lib
	--with-tinfolib
	--without-tinfolib

extconf failed, exit code 1

Gem files will remain installed in /home/akamal/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/curses-1.2.0 for inspection.
Results logged to /home/akamal/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/extensions/x86_64-linux/2.1.0-static/curses-1.2.0/gem_make.out

INFO :
Kernel: 4.4.0-62-generic x86_64 (64 bit) Distro: Ubuntu 14.04 trusty

$ ruby -v
ruby 2.1.3p242 (2014-09-19 revision 47630) [x86_64-linux]

$ gem -v
2.2.2

looks like your ruby setup can't install ncurses I guess installing the ncurses-dev package will solve it
can you try
sudo apt-get install libncurses-dev

and let me know if it solved this issue.
also note that this is still a WIP nothing to see there, only a screen that display the text you print.

sudo apt-get install libncurses-dev Solved the Problem

@AhmedKamal20 perfect, now notice we had to change the project name for collision with other famous project, review #2 for more details if you wish


#1319 Error log when dataType is "file" in @ApiImplicitParam (swagger-jaxrs)

added file as a primitive type

Can this be merged and released in 1.5.11 please? We have some continuous integration tests that are now failing because this bug is logging an error in our logs.

maybe ping @fehguy will help? this is intended as a fix for #1319, which was erroneously closed

@dklotz Thanks, I emailed him.

@msiemczyk @dklotz sorry for the lag.  Yes, this should be fine.


Benchmark can't be properly run on other machine

Hello,
compiled bin can't be properly run on another machine because of error  ': TemplateNotFound "report.tpl"'. If "templates" dir from .cabal is supplied with bin, and bin run with "-t templates/report.tpl", the report is missing pics.
Ubuntu OS.
Best regards,
vlatko

This is not a bug. You need .cabal/share/<your compiler>/criterion-<version>/templates too.

I think this issue should be re-opened.  The bench executable (which wraps criterion in a command-line executable) is running into this same problem:
https://github.com/Gabriel439/bench/issues/12
I would like to be able to distribute a standalone executable, but the baked-in runtime reference to the template file makes the --output option fail when the executable is distributed to another machine that doesn't have criterion installed

Hm. I think I also share @osa1's opinion that this isn't a bug—or at the very least, the issue lies in Cabal, not criterion.
As you've noted in https://github.com/Gabriel439/bench/issues/12#issuecomment-346892659, criterion uses Cabal's data-files feature to bundle several large files used for criterion's HTML reporting. This means that one cannot simply copy a criterion binary from machine to machine and expect it to work, as the appropriate data-files must also be present on the target machine.
For whatever reason, it seems that brew is not distributing the data-files correctly, leading to the mess in https://github.com/Gabriel439/bench/issues/12. Alas, I cannot offer any advice on how one might fix that, since I know nothing of how brew works internally.
(There are extreme measures one can take to eliminate the use of data-files entirely, such as by embedding the files directly into source code using the file-embed library. But that would massively increase binary sizes—and likely compilation times—so as a result, that's not an avenue I'm willing to pursue.)

One way forward here is to adopt the approach that the hyphenation and pandoc take: use file-embed to embed the data-files into the binary when a flag is enabled (pandoc calls this embed_data_files). This would add some complexity, but perhaps it's unavoidable for now, as I'll admit that I can't find a general solution to the data-files problem after doing some searching.

See #169 for an attempt at implementing the idea in https://github.com/bos/criterion/issues/45#issuecomment-346899126.

@Gabriel439, can you confirm if the new -fembed-data-files flag works for bench's purposes?

@RyanGlScott: Yeah, will do.  Give me a day or two to test this

@RyanGlScott: bench users confirmed that the new -fembed-data-files successfully produces a relocatable bench executable: https://github.com/Gabriel439/bench/issues/12#issuecomment-348633222
Thanks a lot for all of your help! 🙂

Awesome! I've uploaded criterion-<IP_ADDRESS> to Hackage with these changes.


Use Notation's PathManager in verification workflow

Signed-off-by: rgnote<EMAIL_ADDRESS>

Codecov Report

Merging #100 (8fa921e) into main (e698b39) will increase coverage by 0.03%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main     #100      +/-   ##
==========================================
+ Coverage   70.91%   70.94%   +0.03%
==========================================
  Files          35       35
  Lines        2135     2134       -1
==========================================
  Hits         1514     1514
+ Misses        500      499       -1
  Partials      121      121

Impacted Files
Coverage Δ

verification/verifier.go
76.08% <40.00%> (+0.81%)
:arrow_up:

verification/helpers.go
94.54% <100.00%> (ø)

verification/verifier_helpers.go
78.83% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Fix incorrect use of async driver in code sample creating CodecRegistry

This commit changes MongoClient to MongoClients to call getgetDefaultCodecRegistry() and adds value 'true' to automatic().

Rebasing and merging.  Thanks for the contribution!


add loc

There's also the variant "SLOC" for "Source Line Of Code". I remember using the tool sloccount a long time ago.

@Tarmil true. Could you submit a new edit with SLOC/LOC?


Install

How install?
Redme pls

I added how to install to readme, so close this issue.
If you have more request regarding to this, please feel free to reopen this.


Fix table browser selection

Fix issue where multiple rows couldnt be selected with the modifier key pressed.
Fixed issue where selected row states would persist after performing the select nodes from selected rows operation.
Fixed issue where selected row states would persist after changing between the node/edge table or loading a different network.
Fixed issue where selecting edges from selected rows would continue to select nodes instead.
Removed checkboxes as they didn't function properly and conflicted with the cell click event.

Hi Dylan @d2fong , Nice job on the fix!
And I'm still encountering a couple of issues with this branch:

It seems I can no longer edit the table cells. Was this functionality intentionally disabled?
When I press and hold the 'Shift' key to select multiple lines, it selects lines [M, N-1] instead of [M, N]. For example, if I click on line M first and then line N, it only selects up to N-1.
Another minor thing is that I cannot deselect the lines/columns anymore if I click blank areas in the table


Update improvements

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[X] a bug fix (Fix #637)

Description
I added a if which makes sure that the downloaded updates hash matches the currentHash entry in ripme.json. I also fixed the bug which caused ripme to think every update was new
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Coverage decreased (-0.06%) to 36.375% when pulling 17f3882057ecfd1166a711ea2017aadff40338f2 on cyian-1756:updateImprovements into 23ded9f6f4e748c2270d42faa5e3a154557141e2 on RipMeApp:master.

The failed test has nothing to do with these changes


WIP fix: stop listener

Related tasks

Depends on #45
Changes
Missing call of unRegisterBroadcastReceivers()

Checklist for self-check

[x] Commits and PRs have been filed according to the rules on the project.
[x] The author is marked as an assigne and assigned mandatory reviewers.
[x] Required labels marked
[x] Specified related tasks and/or related PRs.
[x] Specified Changes.
[ ] Attached videos/screenshots demonstrating the fix/feature.
[x] All unspecified fields in the PR description deleted.
[ ] New code covered by tests.

Checklist for reviewers

[x] CI passed successfully (with a green check mark).
[x] PR is atomic, by volume no more than 400 (+-) corrected lines (not including codogen).

Design:

[x] System design corresponds to the agreements on structure and architecture on the project.
[x] The code is decomposed into necessary and sufficient components.

Functionality:

[x] The code solves the problem.
[x] Any changes to the user interface are reasonable and look good.

Complexity:

[x] The code is clear, easy to read, functions are small, no more than 50 lines.
[x] The logic is not overcomplicated, there is no overengineering (no code sections that may be needed in the future, but no one knows about it).

Tests:

[ ] Updated or added tests for mandatory components.
[ ] The tests are correct, helpful, and well designed/developed.

Naming:

[x] The naming of variables, methods, classes and other components is understandable.

Comments:

[ ] The comments are understandable and helpful.

Documentation:

[x] All labels are correct
[ ] Technical documentation updated (after approval, updates last reviewer).

Codecov Report
Patch and project coverage have no change.

Comparison is base (9ee394c) 96.34% compared to head (492b335) 96.34%.

:exclamation: Current head 492b335 differs from pull request most recent head 8432fea. Consider uploading reports for the commit 8432fea to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #49   +/-   ##
=======================================
  Coverage   96.34%   96.34%
=======================================
  Files           4        4
  Lines          82       82
=======================================
  Hits           79       79
  Misses          3        3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Win32 chrome-finder fix

Right now, chrome-finder (part of the chrome-launcher being sliced off of Google's Lighthouse project) presumes the existence of a PROGRAMFILES(X86) environment variable which only exists on Windows x64 versions and tries to use it to create a path, which fails since it's undefined instead of a string.
This works around it by setting the env variable if it doesn't exist.

Did some preliminary testing after switching dev.3 to pre-release, works great!

#518

 #518 ⁃ [uploader] Windows 32 crashes when clicking on most links (like the footer)


Repeat animation

Hello everyone,
First of all thank you for the animation to the creators.
I would like my image repeat the animation after 12sec.
How can i do it? Could you kindly help me with this, please?
This would be much appreciated.
Thank you

That's only possible using javascript and is not in the scope of the library itself. Sorry, but you better seek for help on StackOverflow as there are lots of people willing to help on this.
I'm closing this as it's unrelated to the library itself.


log: constant doc is all on Ldate

Currently:
const (
	// Bits or'ed together to control what's printed.
	// There is no control over the order they appear (the order listed
	// here) or the format they present (as described in the comments).
	// The prefix is followed by a colon only when Llongfile or Lshortfile
	// is specified.
	// For example, flags Ldate | Ltime (or LstdFlags) produce,
	//	2009/01/23 01:23:23 message
	// while flags Ldate | Ltime | Lmicroseconds | Llongfile produce,
	//	2009/01/23 01:23:23.123123 /a/b/c/d.go:23: message
	Ldate         = 1 << iota     // the date in the local time zone: 2009/01/23
	Ltime                         // the time in the local time zone: 01:23:23
	Lmicroseconds                 // microsecond resolution: 01:23:23.123123.  assumes Ltime.
	Llongfile                     // full file name and line number: /a/b/c/d.go:23
	Lshortfile                    // final file name element and line number: d.go:23. overrides Llongfile
	LUTC                          // if Ldate or Ltime is set, use UTC rather than the local time zone
	LstdFlags     = Ldate | Ltime // initial values for the standard logger
)

This means when you parse the AST for log, the doc is on Ldate and no other constant, nor on the general declaration/block that they're all in.
Should be:
// Bits or'ed together to control what's printed.
// There is no control over the order they appear (the order listed
// here) or the format they present (as described in the comments).
// The prefix is followed by a colon only when Llongfile or Lshortfile
// is specified.
// For example, flags Ldate | Ltime (or LstdFlags) produce,
//	2009/01/23 01:23:23 message
// while flags Ldate | Ltime | Lmicroseconds | Llongfile produce,
//	2009/01/23 01:23:23.123123 /a/b/c/d.go:23: message
const (
	Ldate         = 1 << iota     // the date in the local time zone: 2009/01/23
	Ltime                         // the time in the local time zone: 01:23:23
	Lmicroseconds                 // microsecond resolution: 01:23:23.123123.  assumes Ltime.
	Llongfile                     // full file name and line number: /a/b/c/d.go:23
	Lshortfile                    // final file name element and line number: d.go:23. overrides Llongfile
	LUTC                          // if Ldate or Ltime is set, use UTC rather than the local time zone
	LstdFlags     = Ldate | Ltime // initial values for the standard logger
)

Thanks, want to send a change?

Sure thing

@ianlancetaylor Doesn't this break compat, though? Any tool parsing the log package, for example, will observe the change.

We promise compatibility for source code that imports the log package.  We don't promise that the log package will never change in any way detectable by using the go/parse package.

Proposed fix: https://go-review.googlesource.com/62671


Asset locking

Objective

Implements #15346.

Solution

This is pretty much the same as the design proposed in #15346.

Testing

A basic test in bevy_asset.

Showcase
fn my_task_spawning_system(my_mesh_handle: Res<MyMeshHandle>, mut meshes: ResMut<Assets<Mesh>>) {
    let handle: Handle<Mesh> = my_mesh_handle.0.clone();
    let mesh_arc: Arc<Mesh> = meshes.lock(&handle);

    AsyncComputeTaskPool::get().spawn(move async || {
        // Do something with the mesh_arc.
        std::thread::sleep(std::time::Duration::from_secs(mesh_arc.count_vertices()));
        println!("Done processing!");
    });
}

The primary new parts of the API are:

Assets::lock - Locks the asset if it exists
Assets::try_unlock - Attempts to unlock the asset if there are no more outstanding handles (otherwise the asset remains locked).

Migration Guide

Assets::get_or_insert_with may now return an error if the asset is locked.
Assets::get_mut may now return an error if the asset is locked (in addition to if the asset is missing). Consider using Assets::get_mut_or_clone if your type is Clone to avoid handling locking.
Assets::remove and Assets::remove_untracked now return an AssetPresence<A>. This requires you to handle the case where an asset is locked.
Assets::iter_mut now returns an (AssetId<A>, AssetPresenceMut<'a, A>) instead of (AssetId<A>, &'a mut A). This allows you to handle the locked case.
ReflectAsset::get_mut and ReflectAsset::get_unchecked_mut may now return an error if the asset is locked (in addition to if the asset is missing).
ReflectAsset::remove now returns a ReflectedAssetPresence to store both the locked and unlocked assets.

Just rebased onto main.

I rebased (#15281 had a big blast radius), and also included a new example to show off how you can use asset locking.
I still need to figure out how to implement get_mut_or_clone as mentioned in #15346. I'm running into borrow checker issues (something something polonius something). I'll take another stab at that tomorrow.

Ok, I've implemented get_mut_or_clone, and all the examples now use it. I've updated the migration guide to point users to get_mut_or_clone as well.
One thing that's kinda rough here is that iter_mut doesn't allow you to "clone to unlock". I'm not sure if this is worthwhile to solve, it may require a whole new struct to wrap around AssetPresenceMut, and that seems a little overcomplicated.

Going to wait on @cart to take another look at this design: he had opinions in the linked thread and I don't feel comfortable merging this without him :)


Add optional React 16 support

Description
This adds optional React 16 support by allowing the developer to modify the React version served using, e.g.:
import dash_renderer
dash_renderer.REACT_VERSION = '16.2.0'

The default REACT_VERSION remains 15.4.2
Other changes

Updated .eslintignore to ignore all minified react bundles (versus just 15.4.2)
Bumped version to 0.12.0

The mechanism of choosing version is not clear.
When you execute import dash_renderer file dash_renderer/__init__.py is executed and version is set to '15.4.2'.
If you set REACT_VERSION to '16.2.0' later on the next line, this has no effect.

@utyf You're totally right. I just realized this last night. I'm going to close this PR for now. There's another (ugly) monkeypatch example in this PR: https://github.com/plotly/dash-core-components/pull/160

maybe we need to do something like this:
def _set_react_version(react_version):
    assert react_version in _REACT_VERSION_TYPES
    # Dash renderer's dependencies get loaded in a special order by the server:
    # React bundles first, the renderer bundle at the very end.
    setattr(_this_module, '_js_dist_dependencies', [{
        'external_url': _REACT_VERSION_TO_URLS[react_version]['external_url'],
        'relative_package_path': _REACT_VERSION_TO_URLS[react_version]['relative_package_path'],
        'namespace': 'dash_renderer'
    }])

    setattr(_this_module, '_js_dist', [{
        'relative_package_path': 'bundle.js',
        "external_url": (
            'https://unpkg.com/dash-renderer@{}'
            '/dash_renderer/bundle.js'
        ).format(__version__),
        'namespace': 'dash_renderer'
    }])

_set_react_version('15.4.2')

and then, in the code:
import dash_renderer
dash_renderer._set_react_version('16.2.0')

That works. Adding and reopening with that. From the updated CHANGELOG:

Allows user to now choose between '15.4.2' and '16.2.0' for React versions
import dash_renderer

# Set the react version before setting up the Dash application
dash_renderer._set_react_version('16.2.0')

app = dash.Dash(...)

All previous tests still pass. Added another test to make sure that this actually updates the React version for a dash.Dash application initialized after dash_renderer._set_react_version has been called
Thanks, @utyf !

This looks good to me! Thank you very much for adding the test. Anything else you would like to add @mjclawar before I merge?

@chriddyp no I think this covers everything as long as you are OK with the dash_renderer._set_react_version method!
Probably also will want to bump the default minimum React version to 15.6 soon to at least warn other dash component authors about PropTypes before ideally moving entirely to 16. Not sure if you want a more rigorous roadmap about updating dash-renderer React versions.

@chriddyp anything else you need to see on this?

Any news on the topic?

@chriddyp Checking in to see if you want anything else here to accept the PR.
We're looking forward to using React 16 for all our Dash applications!

Sorry for the delay on this one! This looks good to me, I'll merge now and press a new release 🍻

just published: pip install dash-renderer==0.12.0


RDP NOT SUPORTED -N10.0.19041.3636

You can check RDP functionality with RDPCheck program.
Also you can configure advanced settings with RDPConf program.

Duplicate of https://github.com/stascorp/rdpwrap/issues/2381


attempt to call global 'require' (a nil value)

> love-release
/usr/bin/lua5.1: /usr/local/share/lua/5.1/love-release/pipes/conf.lua:23: conf.lua:3: attempt to call global 'require' (a nil value)
stack traceback:
	[C]: in function 'assert'
	/usr/local/share/lua/5.1/love-release/pipes/conf.lua:23: in function </usr/local/share/lua/5.1/love-release/pipes/conf.lua:14>
	(tail call): ?
	...luarocks/rocks/love-release/2.0.4-1/bin/love-release:11: in main chunk
	[C]: ?

This project uses loadconf to execute the conf.lua file in a safe environment, require is not included in this sandbox.
This is related to issue #27 and #49.
If you want to provide a conf.lua that works both in LÖVE and love-release, make sure you first check if you are running on LÖVE with something similar to:
if love.filesystem then
  --This is LÖVE
end

I don't have a config for love-release. Here is my conf.lua: https://github.com/egordorichev/neko8/blob/master/conf.lua
It was running ok, but something happened and it broke. Debian package building was broken all time.

love-release tries to load conf.lua even if you don't have specific configurations for love-release
Try to do something like
if not require then
  love.conf = function () end
else
  --Your code goes here
end

Oh, that works. Thank you :)


[Storage] Add better support for file copies from a share snapshot

Is your feature request related to a problem? Please describe.
Right now this is supported for a single file copy through specifying a --source-uri for a file that indicates a snapshot. This requires a developer to get the file url and append the snapshot name as a query parameter. The CLI can do better to make this scenario smoother.
Describe the solution you'd like
A clear and concise description of what you want to happen, a --snapshot parameter for both az storage file copy start/start-batch
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Describe the solution you'd like
Wanted to reopen this to see if we could add snapshot support for az storage file copy start-batch?
Thanks!


fix empty batch requests

To trigger new patch release we should change the prefix of the PR title to fix: empty batch requests


Create page for composing track reviews

Is your feature request related to a problem? Please describe.
We need a page that allows the user to compose a post, like a blog post, with their track review, something like writing a blog post on medium
Describe the solution you'd like
The route for the page can be track/:id/review and it should be acessible by clicking on the button "Compose review" on the track page.
The page should have a field to input a title for the review, another input to set the numeric score for the track (if it's not already set), a big input for composing the review and a submit button. For the big input, we found this component which is an editor with the features of the medium... :pleading_face:

Have a cancel button is a good idea too :thinking: and maybe, the possibility of having a "save as draft" button


Feature: private survey variables, or don't store the value of a variable

Related to
Web-Frontend (what users interact with), Ansible (task execution)
Impact
must have for enterprise usage
Missing Feature
There should be 'private' variables available in 'survey variables' that are not displayed to the user (like a password field) and are not saved in the task such that you cannot re-run the task and see the private variables in plaintext.
Currently, all survey variables are displayed in plaintext, and are stored in the tasks in the UI, so you can always go back and view what variable was sent with the task. This poses a security risk with sensitive variables (such as secret keys, passwords, etc.).
Implementation
Very similar to the current survey variables, but with a toggle/checkbox to make it a 'private' variable. This checkbox would make the field not readable to the end user, and would not save the variable value in the task.

Design
No response

Hi @mschmitt-nasuni ,
survey variables  passed to Ansible as "Extra vars". Therefore they cannot be private because user can run Ansible playbook and debug mode and see all extra vars.

Unfortunately I don't know how to workaround this. Let me know if you will have an idea.

Is there a way to maybe prompt for a variable instead? I know ansible has vars_prompt, but when I tried using that method, Semaphore hung on task execution because it's waiting for user input.

Hi @mschmitt-nasuni currently it is not possible, but this feature in the priority for implmementation.

Hi @mschmitt-nasuni
we added Secret type of survey variable. Will be available in following release.


New rule proposal: Literal Suffixes should be uppercase

New rule proposal: Literal Suffixes should be uppercase
Category: Readability
Rule: Literal suffixes should be uppercase
Rationale: Improves readability of code, following recommendation from C# specification
Bad:
long foo = 50l;
decimal bar = 50.4m;

Good:
long foo = 50L;
decimal bar = 50.4M;

See https://msdn.microsoft.com/en-us/library/aa664674(v=vs.71).aspx for the specification for integer literals, in particular the following paragraph:

As a matter of style, it is suggested that "L" be used instead of "l" when writing literals of type long, since it is easy to confuse the letter "l" with the digit "1".

:+1:

I'd suggest restrict this only for L/UL suffixes

I'm currently a "weak no" on this for the following reasons:

While it's clear to me that L is preferable to l, it's not so clear that this makes any difference for other suffixes. In fact, for the suffix m and the e which appears in floating point values, I almost feel like the lowercase version is more easily distinguished and possibly even more common in existing code.
I've never seen anyone use the lowercase suffix for integer literals.

@berkeleybross Have you looked for any instances of this syntax in open source projects?

I looked through my code, and I was mostly consistent on using lower case for m and f, and upper for L. I would be okay if the rule was consistent with lower for everything except L and possibly UL. It could also be configurable, but that really isn't in the StyleCop nature, which is provide one way of doing things. I did use L on integer constants, particularly when there were other constants that also has suffixes nearby. Example:
		private const long FileTimeOffset =<PHONE_NUMBER>00000000L;
		private const decimal MaxFileTime =<PHONE_NUMBER>999999999m;

The fact that I was only mostly consistent might argue for having this rule.

I'm now more of a :-1: on this because of CS0078.

I think there should be a consistent rule on the literal suffixes. I really don't care what it is, I can live with whatever is decided, but there should be one. It can't violate CS0078. I hate looking through my code and finding that I do something one way in one place and a different way in another place.

Since this isn't closed, I'd cast my vote as :+1:.

Not necessarily. This is subjective. One reason can be readability as m is more readable than M; so the current situation with the existing compiler warning CS0078 are desirable. I am inclined to a negative vote.

I think there should be a consistent rule on the literal suffixes. I really don't care what it is, I can live with whatever is decided, but there should be one. It can't violate CS0078. I hate looking through my code and finding that I do something one way in one place and a different way in another place.


What's going on？

HTTPSConnectionPool(host='darksearch.io', port=443): Max retries exceeded with url: /api/search?query=cats&page=1 (Caused by SSLError(SSLEOFError(8, 'EOF occurred in violation of protocol (_ssl.c:1129)')))

Getting the same error !!
UnboundLocalError: local variable 'json_data' referenced before assignment
Any solution provider ?

I'm having the same problem

Darkdump is fixed. The new version is ready for use. (Darkdump 2.0)


[Feature Request] Add ARM operation template to model provider level action

route: /provider/Micorosoft.Contoso/someAction

completed


proposal: 'go get’ tool support http or ftp directly download source file.

indeed, we only need a snapshot of source code, not all commit history of the code.
now ‘go get’ just support vcs(e.g. git), all them will pull a complete repo to local disk and then checkout a new
work copy of repo.
something i considered:

burden of network downloading
latest checkout work copy maybe not working(a expertise commit)

we can make ‘go get’ support directly download the source file(http or ftp) through go-import meta tag.
<meta name="go-import" content="github.com/golang/go file https://github.com/golang/go/releases/tag/go1.5beta2
">

I don't see this proposal as compelling. There's no way to make the -u flag work as-is, and if we set about devising a means of making it work then we start to reinvent git. I'm going to close this as nobody else has shown interest since it was proposed.


Error: Protocol error (Runtime.evaluate): Session closed. Most likely the page has been closed.

Keep getting this error:
UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Protocol error (Runtime.evaluate): Session closed. Most likely the page has been closed.
(node:18244) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Full error  using process.on('unhandledRejection', console.error)
Error: Protocol error (Runtime.evaluate): Session closed. Most likely the page has been closed.
    at Session.send (C:\...\puppeteer\lib\Connection.js:170:29)
    at Page._onConsoleAPI (C:\...\puppeteer\lib\Page.js:225:20)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)

Can you post code that repros the issue?

Speculatively closing this. Feel free to file a new issue or to write here to reopen.

Unhandled Rejection at: Promise Promise {   <rejected> Error: Protocol error (Runtime.evaluate): Session closed. Most likely the page has been closed.     at CDPSession.send (/Users/aayush/Documents/projects/bjAuto/node_modules/puppeteer/lib/Connection.js:185:29)     at ExecutionContext.evaluateHandle (/Users/aayush/Documents/projects/bjAuto/node_modules/puppeteer/lib/ExecutionContext.js:65:77)     at _documentPromise._contextPromise.then (/Users/aayush/Documents/projects/bjAuto/node_modules/puppeteer/lib/FrameManager.js:319:38)     at <anonymous> } reason: Error: Protocol error (Runtime.evaluate): Session closed. Most likely the page has been closed.
Getting this error on executing code below:
await page.goto(${properties[name][project_partition]['uploadUrl']}).catch( (e) => console.log(e));
Can't share the url with you, but my workflow is like this:
-> Login into rundeck
-> wait for login page
-> go to rundeck job(using page.goto())
This is where i get the above error, please help me understand it.
I'm able to execute it manually though, thanks


Add more specific to_dict types

Closes #148
Following the pandas to_dict spec, add an option for lists as well as maps

@ste-pool https://app.circleci.com/pipelines/github/VirtusLab/pandas-stubs/496/workflows/d14c782e-e81e-42a4-8900-fdbd37959462/jobs/507
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/snippets/test_frame.py:8: in <module>
    from pandas._typing import Column
E   ImportError: cannot import name 'Column' from 'pandas._typing' (/usr/local/lib/python3.8/dist-packages/pandas/_typing.py)

That's a bit surprising. It seems like it's impossible to import types that aren't in Pandas itself. That's no problem. Column is just an alias for Union[str, int] and in this case you can replace it with str.


[BUG] Players not working properly after exit Fullscreen

Describe the bug
I'm trying to make a GirdView with couple of videos. It works perfect(even when I switch videos randomly) on original screen. But once get into a full screen, and get back to the original screen, then the bug occurs.
Following unexpected situation occurs randomly (occurs together sometimes).
(1) Controller(play, pause, bottom bar, etc.) disappears but video still goes playing.
(2) Controller alives but plays audio only, with screen paused.
(3) Only play, pause buttons alive. Bottom bar disappears. Then, (2) happens to other videos.
To Reproduce
Below is my part of code. I'm using youtube_player_flutter: ^6.0.3+2
_controller = new List<YoutubePlayerController>();
_url = [
  "https://youtu.be/GAJgbyzSsxg",
  "https://youtu.be/93QS6CReUbQ",
  "https://youtu.be/qch5tdUQvYk",
  "https://youtu.be/-7XFvZmIcTE",
  "https://youtu.be/9w7e02i39lM",
  "https://youtu.be/HtNL9-35xaI",
  "https://youtu.be/baHYYZScKaw"
];
for (int i = 0; i < _url.length; i++) {
  YoutubePlayerController tmp = YoutubePlayerController(
    initialVideoId: YoutubePlayer.convertUrlToId(_url[i]),
    flags: YoutubePlayerFlags(autoPlay: false),
  );
  _controller.add(tmp);
}

// ...

child: GridView.count(
  crossAxisCount: 2,
  shrinkWrap: true,
  physics: NeverScrollableScrollPhysics(),
  crossAxisSpacing: 5,
  primary: true,
  children: List.generate(_url.length, (index) {
    return Center(
      child: YoutubePlayer(
        controller: _controller[index],
        onReady: (){},
      ),
    );
  }),
),

Technical Details:

Device: Samsung Galaxy S9+

Thank you for making good plugin :)

@sarbagyastha  i am also facing the same issue, video player after exit of fullscreen i am not able to make the video player fullscreen again.

Should fix with v7.0.0. Feel free to reopen if the issue persists.


Adding Multiple Programming Language Courses

🛠 Fixes Issue
Issue- #1468
👨💻 Changes proposed
I have added courses for the programming languages, catering to diverse learners and industry demands.
Added courses for the following programming languages:

Java
React
Python
Rust
Shell
SQL
Typescript
PHP
Scala
Perl
Lisp

If there are any specific programming languages or technology domains that you believe would be beneficial to include, I'd be more than happy to work on expanding the course catalog. If there are any guidelines or preferences regarding course additions, please let me know.
Thank you for your time and consideration. I'm looking forward to your feedback.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Look into this, there are some broken image links...
Also I don't think you additions match the pattern we are trying to implement. I would want the user to move right-left to look for his preferred course I think that looks better, currently it looks crowded.
Also make sure the size of images are same...

The issue regarding the broken images and the course layout has been resolved. If there are any further improvements needed please let me know.
Thank you!

hi @rohanbabbar04 ,
I hope you're doing well. I just wanted to check in on the status of my Pull Request. It seems that it hasn't been reviewed yet, and I'm eager to receive your feedback on the changes I made.

The latest commit now includes only the relevant changes without any formatting modifications.

hi @rohanbabbar04  ,
I just wanted to check in on the status of my Pull Request. It seems that it hasn't been reviewed yet, and I'm eager to receive your feedback on the changes I made.

Looks good thanks


Allow Symfony 5

Symfony 5 was just released. Is it possible to allow Symfony 5 in composer.json?

Yes, of course. I will add it.

Done


[YUNIKORN-1423] Update README.md in placement examples that how to update configuration

What is this PR for?
YUNIKORN-1371 updated the deployment, so YUNIKORN-1423 update the README.md to describe how to deploy the configuration.
What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[x] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-1423?
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

With the work that has just started to allow queue type changes the processing of placement rules will probably change.
Any follow up that would be needed should be hooked into that major work.


Provide static builds for multiple platforms

You might want to provide static builds for multiple platforms as releases on this github.
offtopic:
Also, if you need help with a website, I can design a website and publish it in my repo which you can pull and control yourself, as a github pages project.

You are right, I should do that. I'll create a release tag soon.
I guess the project could have a gh-page, if you feel like contributing a simple one, I'm happy to merge it.


GCC medium

cant find GCC medium (which we found before for another site).  is the PS1 without GCCH GCC medium?
please advise

will try other venues


Harmony control of HomeKit scenes?

I'm looking for a way to trigger HomeKit scenes from buttons on my Harmony. It looks like this plugin is designed in the opposite direction, triggering Harmony activities from HomeKit.
Is what I'm looking for possible?

Hi,
Indeed the plugin for homebridge does this opposite. I don’t know if any solution to do what you want, since there is no « trigger » that can be set with harmony. You can upvote this feature that would allow to do such things but I doubt Logitech will open his system ... https://community.logitech.com/s/question/0D53100006u0CzxCAE/feature-request-harmony-to-call-arbitrary-urls . There is some workaround suggested there also but didn’t try them myself

Although, you could use this to mount "virtual devices" into Homebridge, and then use those to control HomeKit scenes through automation...
create a Device using an old unused remote (like a Xmas light one), and present that to HomeBridge. When your Harmony Remote sends that on/off command, it should register in HomeBridge, right? and then use that device as the trigger for your scene


[PROPOSAL] Tutorial on using UBI in an OpenTelemetry centric world

What/Why
What are you proposing?
A tutorial on the how and why of UBI and Otel.  Added to https://opensearch.org/docs/latest/search-plugins/ubi/index/ list of tutorials.
What users have asked for this feature?
Otel is a core enabling tech on OpenSearch.
Stavros, Aruna, and other see value.

See https://github.com/opensearch-project/user-behavior-insights/issues/20 for the underlying code improvement to UBI.

[Catch All Triage - 1, 2, 3, 4, 5]

We're getting close to merging the Otel support, and then this can get movign!


chore(tikv/tikv): update additional_labels for plugin ti-community-label

What problem does this PR solve?
Problem Summary: update additional labels for plugin ti-community-label

/merge


Unused parameter in function causing calls to fail which don't provide it

The function _handle_data_upload takes a service_websocket poitional parameter which isn't actually used in the function.
Calling from handle_request doesn't provide an argument for this parameter, which lead to an error.

Thanks for reporting this, @hellkite500! Just opened #551 to fix this.


MINOR Moved a few log segment util methods from LocalLog to LogFileUtils

MINOR Moved a few log segment util methods from LocalLog to LogFileUtils.
This is related to the changes required for KAFKA-14481
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks @showuon for the review. Addressed them with the latest commit.

Test failures do not seem to be related to this change, merging these changes to trunk.

I didn't see this until now. I actually think the right approach is to push some of these methods to the respective classes (eg if it's related to an index, the method should be in the index class). I starting working on a PR for that and will submit it later.

@satishd How is this related to moving LogSegment? It doesn't seem like that was updated at all?

@satishd How is this related to moving LogSegment? It doesn't seem like that was updated at all?

Methods like the below have usages in LogSegment class. Those will be updated as part of KAFKA-14481 changes.
UnifiedLog.offsetIndexFile(dir, baseOffset, fileSuffix)
UnifiedLog.timeIndexFile(dir, baseOffset, fileSuffix)
UnifiedLog.transactionIndexFile(dir, baseOffset, fileSuffix)
UnifiedLog.logFile(dir, baseOffset, fileSuffix)


Fix slight cmdline parsing issue for job container/misc cleanup

I was using filepath.Clean to remove any initial '.' paths in the commandline.
However it changes all forward slashes to backslashes, which hinders commandlines like
the following:
"cmd /c ping <IP_ADDRESS>"
as the /c argument will be altered to \c.
Misc. cleanup involves cleaning up some comments, changing to FreeLibrary instead
of CloseHandle after loading kernel32 for signals, and checking if the stdio pipes
are nil before closing them.
Signed-off-by: Daniel Canter<EMAIL_ADDRESS>

@kevpar Going to close this as the work now is just cleaning up the path tests

Oh I'm out of it, I still need to remove the filepath.Clean call also, but that's small.


UI: Allow correcting the former clip

Sometimes I approve a clip and then I am reminded that something isn't just right, having a way to go back and correct that could be beneficial, usually other contribution systems only allow going to the previous item but no further.

Thanks @yarons  for raising this issue. Let's concentrate on the discussion in issue #17.


How to prevent intermixed logs

Sometimes, logging from different tasks will get mixed up:

How can we prevent this? Is there a place where we could add a mutex?
The issue seems to be specifically Rust logs and logs from esp-idf components mixing.

This happens specifically when configuring UART0 as stdin:
https://github.com/esp-rs/esp-idf-hal/issues/200

The intermixed logs are caused by writeln! (as used here) calling to write() multiple times when formatting is used. The following example illustrates this:
writeln!(
    EspStdout,
    "\x1b[0;{}m{} ({}) {}: {}\x1b[0m",
    color,
    Self::get_marker(record.metadata().level()),
    unsafe { esp_log_timestamp() },
    record.metadata().target(),
    record.args()
)

Will effectively execute this (pseudocode):
write("\x1b[0;");
write(color);
write("m");
write(Self::get_marker(record.metadata().level());
write(" ");
...
write(record.args);
write("\x1b[0m");

Stdout is locked for every single one of these individual write() calls, but it is not locked for the entire writeln!.

Problem is not easy to solve as we are - these days - doing cheap (as in no-alloc) logging which moreover is not truncating logs. To do this, we have to rely on writeln! as it streams the data to stdout rather than accumulating everything in a large buffer (a "string") and then outputting the accumulated string in a single call to the native ESP stdout C-level stream.
Putting a "mutex" "around" writeln! in Rust is only a partial solution, as the native C code won't know about the mutex, and you'll still be getting interleaved log output.
Also, and as you can see here, the rabbit hole is deeper than even that, because even if we were able to get our hands on the native C-level stdout mutex from newlib, we would still be getting interleaved logs if something is writing to stdout, and then another thing is hitting stderr, as long as one or the other thing does not "log" with a single call to e.g. C "fwrite" which resolves to "uart_write".
So yeah, I'm open to suggestions, but please think carefully for pros and cons, as I'm not seeing a perfect solution.

Also, and as you can see here, the rabbit hole is deeper than even that, because even if we were able to get our hands on the native C-level stdout mutex from newlib, we would still be getting interleaved logs if something is writing to stdout, and then another thing is hitting stderr, as long as one or the other thing does not "log" with a single call to e.g. C "fwrite" which resolves to "uart_write".

I'm thinking that actually the problem of interleaved logs (or I should rather say - outputs) - where one is coming from, stdout, and another - from stderr - is probably not solved in ESP IDF either, as ESP IDF logging is using the vprintf function, which would only lock the stdout file handle, but won't lock or even realize, that the stderr handle outputs on the same UART.
So... forgetting about the above ^^^... and looking at the C/POSIX docus, maybe we just need to call flockfile/funlockfile before/after writeln!?

It looks like this solved the issue. Thanks!

Thank you!


Add new about page text, plus new sub-pages and FAQ section

Doesn't contain any homepage changes. That might be a bit more involved with changes to layout and styling, so I think I separate pull request probably makes sense.

I guess Ira needs to have a look at these changes and sign them off? Is the best approach to merge to staging?

They are mostly just content that's already been agreed on so I think it should be fine to deploy.


Test lister

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

58 of 143 (40.56%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+28.8%) to 38.356%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/gh/lister.go
54
73
73.97%

internal/gh/printer.go
0
21
0.0%

internal/jira/cloner.go
4
49
8.16%

Files with Coverage Reduction
New Missed Lines
%

internal/gh/lister.go
1
75.73%

Totals

Change from base Build<PHONE_NUMBER>:
28.8%

Covered Lines:
84

Relevant Lines:
219

💛  - Coveralls


Support change tracking on inverse side of M:N relations

Is your feature request related to a problem? Please describe.
Currently, only the owning sides of M:N relations are checked for changes. Working with inverse side works as long as you are adding or removing specific and populated items, but it won't work if you try to add not loaded reference instead. Also calling removeAll() on inverse side is not working at all.
Describe the solution you'd like
We should probably check both sides of M:N relations, and try to compute the diff respecting both.
Describe alternatives you've considered
If this proves to be too hard, we should at least throw instead of silently ignoring those problematic actions.
Additional context
This has been one of the common pain points since inception.

Closing as implemented in v6 via #4798


Add UIAppearance support

MarkupKit should provide support for configuring UIAppearance elements declaratively, similar to how template properties are defined. This would provide functionality comparable to descendant or child selectors in CSS.
An Appearance.json file could be used to declare appearance properties. Size class-specific values could be declared in Appearance~vertical.json, etc. LMViewBuilder would automatically load these files at startup and add the necessary entries to UIAppearance.
For example:
{
    "UIBarButtonItem": {
        "tintColor": "#aa0000"
    },

    "UINavigationBar, UIBarButtonItem": {
        "tintColor": "#00aa00"
    }

    "UIPopoverController, UINavigationBar, UIBarButtonItem": {
        "tintColor": "#0000aa"
    }
}

Default templates would be applied via appearanceWhenContainedInInstancesOfClasses:. Size class-specific templates would use appearanceForTraitCollection:whenContainedInInstancesOfClasses: (it may be possible to use appearanceForTraitCollection:whenContainedInInstancesOfClasses: with an empty trait collection instead of appearanceWhenContainedInInstancesOfClasses:).
The last type in the list would represent the target type (the type on which the method would be called). Any preceding types would be passed in the containerTypes argument, in reverse order.
Trait collections would be applied as follows (horizontal/vertical):

default - none/none
"normal" - regular/regular
"horizontal" - regular/compact
"vertical" - compact/regular
"minimal" - compact/compact

Unfortunately, it seems like this may not work. Even though the methods of UIAppearance return instancetype, they don't actually return an instance of the actual type (e.g. UIButton). Instead, they return an instance of a private class named _UIAppearance. This class is not KVC-compliant, and also doesn't inherit the implementation of applyMarkupPropertyValue:forKey: from the actual type.

The intent of this enhancement will be resolved by #67. Additionally, it should be possible to use the features described in #67 to configure an appearance proxy, if desired. Closing.


Test if server works without kinect sdk (probably won't)

Seems to not work, but need to retest

Probably doesn't need it? Need more testing.

Server does work without kinect sdk. On plugging in the kinect 2.0, the kinect 2.0 runtime is installed.


TS-4584: Resource leak, CID 1254818 and 1254809

Coverity issues CID 1254818 and 1254809, a resource leak on file descriptor
and allocated memory.

[approve ci]

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/330/ for details.

This fails on clang-format, please re-run and push --force.

add to whitelist

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/331/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/437/ for details.

Could we use the ats_scoped_obj here? Or some other smart ptr? I'm ok either way,  but maybe we should encourage these smart pointer patterns?
This might change memory management though (malloc/free vs new/delete) ? Don't go out of your way though if it's going to be really complex.

I'm testing the scoped pointer approach now.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/481/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/378/ for details.

Linux build failed! See https://ci.trafficserver.apache.org/job/Github-Linux/434/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/537/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/538/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/435/ for details.

Scoped pointer approach posted.


Sending custom keycodes (e.g. Power) from TinyPilot?

I'm looking into whether we can use TinyPilot to remotely control some ChromeOS based devices.
These devices often have specific keys for things like "Search", or "Power".
I can get the keycodes for these - e.g. 0x98 is power.
However, is there any way to send custom keycodes to a device from TinyPilot?
And do you know if sending these will still work when the device is powered off?

Thanks for filing this!
That might be possible. I suspect that it won't work when the device is powered off, though.
Can you share screenshots of what appears when you push those keys while viewing keycode.info?

Didn't hear back, so I'm closing, but feel free to reopen if you'd like to share screenshots.


Pageloadstrategy not supported

require "marionette"

options = Marionette.chrome_options(args: ["headless"],  page_load_strategy: Marionette::PageLoadStrategy::Eager)
session = Marionette::WebDriver.create_session(:chrome, exe_path: "bin/chromedriver", capabilities: options)

Unhandled exception: invalid argument: cannot parse capability: goog:chromeOptions
from invalid argument: unrecognized chrome option: pageLoadStrategy (Marionette::Error::InvalidArgument)
  from ................/crystal/src/class.cr:148:0 in 'execute'
  from lib/marionette/src/marionette/session.cr:49:18 in 'start'
  from lib/marionette/src/marionette/session.cr:34:5 in 'start:capabilities'
  from lib/marionette/src/marionette/web_driver.cr:46:7 in 'get_session:capabilities'
  from lib/marionette/src/marionette/web_driver.cr:40:7 in 'create_session:exe_path:capabilities'
  from src/scripts/marionette.cr:4:1 in '__crystal_main'

Hmm good question. I'll have to look into this when I get time. I'd also happily accept a PR.

@Daniel-Worrall this should be fixed. Turns out pageLoadStrategy is a standard WebDriver option and not a chromedriver specific one, so all of the browser options objects now accept **capabilities which will be merged into the base capabilities object. Let me know if this works :)


"Found unexpected equivalence"

I'm writing a very custom Language implementation and I suspect that maybe there is a bug in my implementation that is causing this assertion in "egraph.rs" to fail. Question is, I'm not sure what sort of problem leads to "Found unexpected equivalence", so maybe the text of that assertion could be expanded to include details of what the likely cause is?

Yeah, that's a little confounding. That check is to make sure that the rebuild process went correctly, I'm not sure how it could have failed. Do you have some funky eq or hash implementations for your Language?

I do. I've found what the problem was. I was assuming that if a Language instance was created from the exact same syntax tree instance as another Language instance, then those two Language instances would always be equal - i.e. I had added an "optimisation" that allows me to short-circuit equality tests by skipping things like comparing the children Ids. I see now that since the children Ids are modified by egg, and that it relies on modified nodes no longer being equal to their past selves, this "optimisation" cannot work. I've removed it and the assertion failure is gone.
I think it might be worth mentioning something about the expected behaviour of equality on Language, though I appreciate I was taking a fairly liberal view of it.

Could you PR that? I'm not sure I fully understand what you were doing, so I'm not sure I can write the necessary addition to the Language documentation.

Closing for now. Feel free to re-open if needed!


tlapm ending abnormally with Invalid_argument("List.combine") in level or arity checking

TLAPS has been terminating unexpectedly on an existing proof since the pre-release https://github.com/tlaplus/tlapm/releases/tag/202208050903. This issue appears to be linked to level checking.
$ tools/tlaps15pre/bin/tlapm --cleanfp BlockingQueueFair.tla
(* fingerprints file ".tlacache/FiniteSets.tlaps/fingerprints" removed *)
(* created new ".tlacache/FiniteSets.tlaps/FiniteSets.thy" *)
(* fingerprints written in ".tlacache/FiniteSets.tlaps/fingerprints" *)
File "/workspaces/BlockingQueue/tools/tlaps15pre/lib/tlaps/FiniteSets.tla", line 1, character 1 to line 23, character 77:
[INFO]: All 0 obligation proved.
(* fingerprints file ".tlacache/TLAPS.tlaps/fingerprints" removed *)
(* created new ".tlacache/TLAPS.tlaps/TLAPS.thy" *)
(* fingerprints written in ".tlacache/TLAPS.tlaps/fingerprints" *)
File "./TLAPS.tla", line 1, character 1 to line 311, character 77:
[INFO]: All 0 obligation proved.
(* fingerprints file ".tlacache/BlockingQueue.tlaps/fingerprints" removed *)
(* created new ".tlacache/BlockingQueue.tlaps/BlockingQueue.thy" *)
(* fingerprints written in ".tlacache/BlockingQueue.tlaps/fingerprints" *)
File "./BlockingQueue.tla", line 1, character 1 to line 152, character 77:
[INFO]: All 1 obligation proved.
 tlapm ending abnormally with Invalid_argument("List.combine")
Raised at Stdlib.invalid_arg in file "stdlib.ml", line 30, characters 20-45
Called from Stdlib__list.combine in file "list.ml", line 305, characters 36-49
Called from E_levels.level_computation#expr.apply_operator.f in file "e_levels.ml", line 228, characters 28-55
Called from E_levels.level_computation#expr.apply_operator in file "e_levels.ml", line 234, characters 27-55
Called from E_levels.level_computation#expr in file "e_levels.ml", line 492, characters 23-40
Called from E_levels.level_computation#expr in file "e_levels.ml", line 620, characters 21-38
Called from E_action.lambdify in file "e_action.ml", line 2090, characters 12-39
Called from M_elab.lambdify_definition in file "m_elab.ml", line 244, characters 23-44
Called from M_elab.lambdify_enabled_cdot.object#definition in file "m_elab.ml", line 273, characters 21-46
Called from M_visit.map#module_unit in file "m_visit.ml", line 48, characters 12-47
Called from M_visit.map#module_units.f in file "m_visit.ml", line 33, characters 27-49
Called from Stdlib__list.fold_left in file "list.ml", line 121, characters 24-34
Called from M_visit.map#module_units in file "m_visit.ml", line 36, characters 31-75
Called from M_elab.lambdify_enabled_cdot in file "m_elab.ml", line 302, characters 20-47
Called from M_elab.instantiate in file "m_elab.ml", line 530, characters 15-44
Called from M_elab.normalize.spin.create_instance in file "m_elab.ml", line 1349, characters 16-98
Called from M_elab.normalize in file "m_elab.ml", line 1457, characters 12-66
Called from Tlapm.process_module in file "tlapm.ml", line 293, characters 29-68
Called from Tlapm.main.f in file "tlapm.ml", line 503, characters 23-43
Called from Stdlib__list.fold_left in file "list.ml", line 121, characters 24-34
Called from Tlapm.main in file "tlapm.ml", line 506, characters 13-40
Called from Tlapm.init in file "tlapm.ml", line 518, characters 8-33

For comparison, 1.4.6 accepts BlockingQueue.tla just fine:
$ tools/tlaps/bin/tlapm --cleanfp BlockingQueueFair.tla
(* fingerprints file "BlockingQueueFair.tlaps/fingerprints" removed *)
** Unexpanded symbols: ---

** Unexpanded symbols: IsInjective, Get, vars, Put

** Unexpanded symbols: IsInjective, Next, vars

** Unexpanded symbols: IsInjective, Next, vars

** Unexpanded symbols: Next, vars, Put

** Unexpanded symbols: Next, vars

** Unexpanded symbols: IsInjective, Next, vars

** Unexpanded symbols: Next, Get, vars

** Unexpanded symbols: Next, vars

** Unexpanded symbols: IsInjective, Next

** Unexpanded symbols: ---

** Unexpanded symbols: BQS!vars, Get, vars, Put, BQS!Next, TypeInv

** Unexpanded symbols: BQS!vars, Next, vars, BQS!Next, TypeInv

** Unexpanded symbols: BQS!vars, Next, vars, BQS!Next, TypeInv

** Unexpanded symbols: BQS!vars, Next, vars, BQS!Next, TypeInv

** Unexpanded symbols: BQS!Get, Next, BQS!NotifyOther, vars, Put, TypeInv

** Unexpanded symbols: BQS!vars, Next, vars, BQS!Next, TypeInv

** Unexpanded symbols: BQS!vars, Next, vars, BQS!Next, TypeInv

** Unexpanded symbols: Next, BQS!Put, Get, BQS!NotifyOther, vars, TypeInv

** Unexpanded symbols: BQS!Get, Next, BQS!Put, TypeInv

(* created new "BlockingQueueFair.tlaps/BlockingQueueFair.thy" *)
(* fingerprints written in "BlockingQueueFair.tlaps/fingerprints" *)
$ tools/tlaps/bin/tlapm --version
1.4.5 (build 33809)

TLAPS errors out right away with latest and greatest (6acb3cb):
-> % ~/.opam/5.1.0/bin/tlapm --version
6acb3cb

-> % ~/.opam/5.1.0/bin/tlapm --cleanfp BlockingQueueFair.tla
File "./BlockingQueue.tla", line 126, characters 21-28:
Error: Invalid number of arguments
 tlapm ending abnormally with Failure("Expr.Levels: ARITY")
Raised at Stdlib.failwith in file "stdlib.ml", line 29, characters 17-33
Called from Tlapm_lib__E_levels.level_computation#expr.apply_operator.f in file "src/expr/e_levels.ml", line 229, characters 20-49
Called from Tlapm_lib__E_levels.level_computation#expr.apply_operator in file "src/expr/e_levels.ml", line 236, characters 27-55
Called from Tlapm_lib__E_levels.level_computation#expr in file "src/expr/e_levels.ml", line 865, characters 21-36
Called from Tlapm_lib__E_levels.level_computation#expr in file "src/expr/e_levels.ml", line 622, characters 21-38
Called from Stdlib__List.rev_map.rmap_f in file "list.ml", line 105, characters 22-25
Called from Tlapm_lib__Ext.List.map in file "src/util/ext.ml", line 48, characters 22-37
Called from Tlapm_lib__E_levels.level_computation#expr.max_args_level in file "src/expr/e_levels.ml", line 148, characters 22-51
Called from Tlapm_lib__E_levels.level_computation#expr.level_info_from_args in file "src/expr/e_levels.ml", line 157, characters 43-66
Called from Tlapm_lib__E_levels.level_computation#expr in file "src/expr/e_levels.ml", line 606, characters 36-63
Called from Tlapm_lib__E_action.lambdify in file "src/expr/e_action.ml", line 1634, characters 12-39
Called from Tlapm_lib__M_elab.lambdify_expr in file "src/module/m_elab.ml" (inlined), line 228, characters 15-160
Called from Tlapm_lib__M_elab.lambdify_definition in file "src/module/m_elab.ml", line 243, characters 23-44
Called from Tlapm_lib__M_elab.lambdify_enabled_cdot.object#definition in file "src/module/m_elab.ml", line 272, characters 21-46
Called from Tlapm_lib__M_visit.map#module_unit in file "src/module/m_visit.ml", line 49, characters 12-47
Called from Tlapm_lib__M_visit.map#module_units.f in file "src/module/m_visit.ml", line 34, characters 27-49
Called from Stdlib__List.fold_left in file "list.ml", line 123, characters 24-34
Called from Tlapm_lib__M_visit.map#module_units in file "src/module/m_visit.ml", line 37, characters 31-75
Called from Tlapm_lib__M_elab.lambdify_enabled_cdot in file "src/module/m_elab.ml", line 301, characters 20-47
Called from Tlapm_lib__M_elab.instantiate in file "src/module/m_elab.ml", line 529, characters 15-44
Called from Tlapm_lib__M_elab.normalize.spin.create_instance in file "src/module/m_elab.ml", line 1166, characters 16-98
Called from Tlapm_lib__M_elab.normalize in file "src/module/m_elab.ml", line 1274, characters 12-66
Called from Tlapm_lib.process_module in file "src/tlapm_lib.ml", line 320, characters 29-68
Called from Tlapm_lib.main.f in file "src/tlapm_lib.ml", line 574, characters 23-43
Called from Stdlib__List.fold_left in file "list.ml", line 123, characters 24-34
Called from Tlapm_lib.main in file "src/tlapm_lib.ml", line 577, characters 13-40
Called from Tlapm_lib.init in file "src/tlapm_lib.ml", line 589, characters 8-33

That's the minimal example that fails similarly. B1 is parsed successfully, but when parsing B2, it fails at  B1 on Op(x).
---- MODULE B1 ----
Op(x) == TRUE
USE TRUE
OpAll == \A x : Op(x)
====

and
---- MODULE B2 ----
INSTANCE B1
====

Here, it is important to have an operator with an argument and USE of something before applying that operator.
@muenchnerkindl, maybe that points you to some particular place in the code?

Removing the USE from BlockingQueue.tla allows TLAPS to prove BlockingQueueSplit.tla.  However, BlockingQueueFair.tla fails with a new error that mentions EnabledWrapper, which is not defined or declared in my workspace:
-> % opam exec -- tlapm --nofp BlockingQueueFair.tla
File "./BlockingQueue.tla", line 116, characters 44-52:
[INFO]: Auto-expanding the definition of operator:  Put

File "./BlockingQueue.tla", line 36, characters 10-31:
[INFO]: Auto-expanding the definition of operator:  NotifyOther

File "./BlockingQueue.tla", line 38, characters 10-16:
[INFO]: Auto-expanding the definition of operator:  Wait

<unknown location>:
Error: Operator "EnabledWrapper" not found
 tlapm ending abnormally with Failure("Expr.Anon: 4")
Raised at Stdlib.failwith in file "stdlib.ml", line 29, characters 17-33
Called from Tlapm_lib__E_visit.map#expr in file "src/expr/e_visit.ml", line 273, characters 22-37
Called from Tlapm_lib__E_action.lambdify_action_operators#expand in file "src/expr/e_action.ml", line 941, characters 17-45
Called from Tlapm_lib__E_action.lambdify in file "src/expr/e_action.ml", line 1640, characters 12-142
Called from Tlapm_lib__M_elab.lambdify_expr in file "src/module/m_elab.ml" (inlined), line 228, characters 15-160
Called from Tlapm_lib__M_elab.lambdify_definition in file "src/module/m_elab.ml", line 243, characters 23-44
Called from Tlapm_lib__M_elab.lambdify_enabled_cdot.object#definition in file "src/module/m_elab.ml", line 272, characters 21-46
Called from Tlapm_lib__M_visit.map#module_unit in file "src/module/m_visit.ml", line 49, characters 12-47
Called from Tlapm_lib__M_visit.map#module_units.f in file "src/module/m_visit.ml", line 34, characters 27-49
Called from Stdlib__List.fold_left in file "list.ml", line 123, characters 24-34
Called from Tlapm_lib__M_visit.map#module_units in file "src/module/m_visit.ml", line 37, characters 31-75
Called from Tlapm_lib__M_elab.lambdify_enabled_cdot in file "src/module/m_elab.ml", line 301, characters 20-47
Called from Tlapm_lib__M_elab.instantiate in file "src/module/m_elab.ml", line 529, characters 15-44
Called from Tlapm_lib__M_elab.normalize.spin.create_instance in file "src/module/m_elab.ml", line 1166, characters 16-98
Called from Tlapm_lib__M_elab.normalize in file "src/module/m_elab.ml", line 1274, characters 12-66
Called from Tlapm_lib.process_module in file "src/tlapm_lib.ml", line 320, characters 29-68
Called from Tlapm_lib.main.f in file "src/tlapm_lib.ml", line 574, characters 23-43
Called from Stdlib__List.fold_left in file "list.ml", line 123, characters 24-34
Called from Tlapm_lib.main in file "src/tlapm_lib.ml", line 577, characters 13-40
Called from Tlapm_lib.init in file "src/tlapm_lib.ml", line 589, characters 8-33

The EnabledWrapper error appears to be coming from the two levels of refinement: BGFair refines BGSplit refines BQ.
-
 INSTANCE TLAPS

 (* Prove TypeInv inductive. *)
@@ -144,7 +143,7 @@ THEOREM ITypeInv == Spec => []TypeInv
   <2>4. QED
     BY <2>1, <2>2, <2>3 DEF Next
 <1>3. QED BY <1>1, <1>2, PTL DEF Spec
-
+====
 -----------------------------------------------------------------------------

diff --git a/BlockingQueueSplit.tla b/BlockingQueueSplit.tla
index afc10a0..ce224c9 100644
--- a/BlockingQueueSplit.tla
+++ b/BlockingQueueSplit.tla
@@ -73,8 +73,8 @@ Spec == Init /\ [][Next]_vars
 (* BlockingQueueSplit refines BlockingQueue. The refinement mapping is *)
 (* straight forward in this case. The union of waitSetC and waitSetP   *)
 (* maps to waitSet in the high-level spec BlockingQueue.               *)
+====
 A == INSTANCE BlockingQueue WITH waitSet <- (waitSetC \cup waitSetP)
-
 (* A!Spec is not a valid value in the config BlockingQueueSplit.cfg.   *)
 ASpec == A!Spec


Implement macro to enforce strong typing/direction in edges

Creates implement_add_edge_to macro that defines functions to specify strongly typed connections with the direction hardcoded.
This is a quick win implementation to solve a couple of footguns the system had, it does not intend on fixing all footguns. Since now is not the time to workshop a perfect solution this quick win felt like the way to go.
This unblocks copy pasting since it enumerates all connections in the system and makes it greppable.
Please try to use the implement_add_edge_to  macro for new connections instead of workspace_snapshot.add_edge directly.
The current macro does not support all use cases, like category nodes, and definitely needs expansion.
It checks if edge weight kind in the function call matches the discriminant defined in the macro call.
Example:
Before:
    // Component (this) --> Schema Variant
    workspace_snapshot
        .add_edge(
            id, // Ulid
            EdgeWeight::new(change_set, EdgeWeightKind::new_use())?,
            schema_variant_id, // Ulid
        )
        .await?;

After:
    implement_add_edge_to!(
        source_id: ComponentId,
        destination_id: SchemaVariantId,
        add_fn: add_edge_to_schema_variant,
        discriminant: EdgeWeightKindDiscriminants::Use,
        result: ComponentResult,
    );

    // Enforces that the source is of type ComponentId and destination is of type SchemaVariantId
    // Runtime checks that EdgeWeightKind variant is Use
    Component::add_edge_to_schema_variant(
        ctx,
        component.id,
        schema_variant_id,
        EdgeWeightKind::new_use(),
    )
    .await?;

Feedback is needed here!
Benefits that this creates:

Enforces connection direction
Enforces connected objects types
Runtime checks that the EdgeWeightKind passed is of the expected EdgeWeightKindDiscriminant
Makes possible connections enumerable

bors try

bors merge


Address the unnecessary generation of Jest snapshots

Describe the bug
When updating a component, and running jest --updateSnapshot, new snapshots will be generated that are not limited to the particular component.
To Reproduce
Steps to reproduce the behavior:

Go to any component, say Button
Do any minor CSS update
Run jest --updateSnapshot
Expect to see various new snapshots

Expected behavior
Only updated components should have updated snapshots
Screenshots

Closing, since that's a mistake on my part. These elements do include the Button component, so it's expected to be affected by them.


Add support for manually selecting scopes in the pre-value options

Both Facebook and Google allows developers to specify one or more scopes when authentication with their service, but the scopes are currently hardcoded Skybrud.Social.Umbraco. This should be changed for v1.0.4 (or possibly v1.0.3) so that the desired scope can be specified via the pre-value options.
I have already done some work on this, but need to wrap it up before it can be released. Here are some screenshots:

With the release of v1.0.3 , the Facebook property editor and prevalue editor has now support scopes similar to shown on the first screenshot in my post above.
To get v1.0.3 out the door, scopes for Google will first be supported in v1.0.4.

@abjerner Just checking if you made any start/progress on the Google scopes?
I'm working on a project where I need to set the AnalyticsScope. If the property-editor supported this, that would be great.  I'm happy to roll my sleeves up and help out if needed.  Just wanted to make sure you hadn't already made a start with it? (I couldn't tell from the repo)


Suggestions. Make x, y, z more easier to distinguish.

Overall, it was a great experience.:grin:
Here is my suggestions: add axes or mark quadrants with different colors.:smiley:

@Huxley7744 thanks for the feedback. Will see what we can do to make this a better experience.


Removed test folder

Yes, the test document does contain my telegram token. No, it no longer works. I am sorry.


Advice: Clearer and more adjustable parameters.

It is recommended to put the adjustable parameters in one block.

Hi, thanks for your interest! Could you please elaborate this or show some examples? Thanks

For example: the parameter that can control the size of the model like (169m,430m)

do you mean this? https://github.com/Hannibal046/nanoRWKV/blob/1b57cabde78b059052c729d623bc66118809f889/modeling_rwkv.py#L199-L207
or this?
https://github.com/Hannibal046/nanoRWKV/blob/1b57cabde78b059052c729d623bc66118809f889/train.py#L155-L160

Yes, but if we can use clear descriptions, we can let non-relevant practitioners learn more clearly.

Because the beginner may not want to train too large a model, or may be a rapid prototype iteration.
Note: It is possible to add a mixture of multiple low parameter models as a composite model.

Additional idea: you can add the ability to train a back-propagation low-parameter RLHF model mixed with a forward-propagation conventional model as a composite Bi-directional model

Thanks for your advice. But the purpose of this repo is to introduce RWKV while keeping in line with nanoGPT, so the above may be a little bit out of scope.


gofmt apply inconsistent whitespace around operator expressions

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

$ go version
go version go1.16.3 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="auto"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/abc/Library/Caches/go-build"
GOENV="/Users/abc/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/abc/go/pkg/mod"
GOOS="darwin"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.16.3/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.16.3/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16.3"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/tb/p57hxlwn0l760vcrm2vx_xvc0000gn/T/go-build969478116=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Run gofmt on:
x := 2*4
log.Println("%v", 3*5)

What did you expect to see?
x := 2 * 4
log.Println("%v", 3 * 5)

What did you see instead?
x := 2 * 4
log.Println("%v", 3*5)

dup of https://github.com/golang/go/issues/34426.


Add a footer

Just a minimal little thing with a copyright notice

Fixed in https://github.com/elementary/mvp/pull/13


Add ButtonCell

Hi,
It would be really useful to have a ButtonCell with a Button inside.
I will try to implement one, and if satisfied, I will make a PR.

@rsertelon @slugmandrew Any update on this? Would like to get a 0.9 release soon and have this in there!

I was wondering the same thing. If @rsertelon hasn't made any progress I'm willing to give it a shot.

Will try to do it today. If I cannot, then I'll tell you so that @slugmandrew can try to as he proposed (in the next days, I'm afraid I won't have time) ;)

I've finally had time to push this. Does it seem okay to you all?


chore: merge main back to beta

Merging main back to beta.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Submission]:

Github repo
https://github.com/Jayshah25/Quantum-Glasses
Description
Visualize effect of Quantum Gates on a single qubit via Bloch Sphere Simulation!
Email
<EMAIL_ADDRESS>Alternatives
The only alternative is to code up a circuit (qiskit.QuantumCircuit) and then use visualize_transition from qiskit.visualization. Quantum Glasses implementation wraps all of this into a simple beginner-friendly Tkinter Software.
License
Apache License 2.0
Affiliations
No response
Tags
No response
Website
No response

The tag for this project could be "quantum software" since this is a tkinter implementation.

Hi @Jayshah25 :)
Can you redo your submission with full information please :

you need to add a name in the issue title
add your tag from the list (if you don't find something, that correspond please tell us)
Software is to much generic, indeed every member of the ecosystem is a software. Maybe plugin or widget could be more appropriate ?

Also you need to have at least one test to be able to pass the check, also can you add a requirements.txt file with the python dependencies ?

ok, I will redo the submission while keeping all these points in mind. And maybe widget tag is indeed appropriate.

I think visualization or game tag is most appropriate. There is at least one other project that would fit a visualization so I'm in favour of adding it.

I feel "visualization" fits better than "game". I have opened the new submission issue here: https://github.com/qiskit-community/ecosystem/issues/478


BUG: Add Star Rating to Canteen Profile

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
We need to display a star rating on the canteen profile to provide users with a quick visual representation of the canteen's overall rating based on user feedback. This will help users make informed decisions and enhance the overall user experience.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Kindly reupload the image

Hi @hustlerZzZ,
Please check it now

Hi @hustlerZzZ, Please check it now

Ok do this


Adds RRUFF reader to scidatalib.io.rruff

Partially closes issue #50
Work includes:

Adding the functionality to read RRUFF files as SciData objects
Adds testing for this new functionality
Adds a test dataset for the RRUFF file format
Fixes to JCAMP reader found during adding the RRUFF reader
Adds a read_get_description method to the scidatalib.io.jcamp module
Adds ability to read in list for @type in the SciData.__addtoc method

Work still needed to close out issue #50 is to add the ability to include the dataseries.
Will do once we finish up #43

Thanks, @JohnsonDylan! :+1:


The way to understand additional dcgm-exporter Prometheus metric type

Hi! I'm trying to enable additional metrics in the exporter. According to the doc section Changing Metrics I need to add csv metric description here /etc/dcgm-exporter/default-counters.csv, but to be able to enable it I need to know Prometheus type of a metric.
Is there a way somehow understand metric type from that page below?
https://docs.nvidia.com/datacenter/dcgm/latest/dcgm-api/dcgm-api-field-ids.html
For example: DCGM_FI_DEV_NAME

Hi @k0nstantinv,
We do not have a direct list of the dcgm-Prometheus or dcgm-collectd types' correspondence.
You could get some ideas from this collectd script.
The metric in your example cannot be represented as a Prometheus metric value and instead should be used as a Prometheus metric label. See an example here.
I hope that will help you.
WBR,
Nik

@nikkon-dev thanks a lot!  One notice about enabling labels. One of the latest release of exporter required to be able to enable any label. For example enabling labels on 2.1.4-2.2.0 ends up fatal
time="2022-11-02T10:41:03Z" level=fatal msg="Could not find Prometheus metry type label"


Bump SBT Assembly to 0.14.7

requirement for #86

I'm not even sure why do we have sbt-asembly in iglu client. It is a library. @asoltysik, could you please change it to "Remove SBT assembly"?