npm audit reveals vulnerablities

While installing mempool I noticed the output warned about 18 vulnerabilities. I checked this via
admin@raspberrypi:/home/mempool/mempool/frontend $ npm audit
[...] [List of vulnerablities]
found 18 vulnerabilities (4 moderate, 14 high) in 1723 scanned packages   run npm audit fix to fix 18 of them.
They are caused by the packages  y18n, socket.io and elliptic. I checked this issue in the mempool-repo, but this wasn't even mentioned there. Should that cause concern, or is it already fixed in next milestones?

npm deps are already updated in upcoming v2.2 release, if user is concerned they can change tag to master

I was thinking about to provide a kind of script, which can test for this kind of vulnerabilities, shown by npm audit. So this should be executed, maybe once a month, for all installed programs via npm, not only mempool. Would this be a benefit or would the issues even be fixed by continuous upgrading the installations itself?

Ever since GitHub acquired npm the repo maintainers get notifications from GitHub all the time about npm dependency vulnerabilities, so I think if you fork mempool into your own GitHub account you can get the same notifications?

To avoid misunderstandings.
I'am in the view of a runner of the node. To make it clear, a not professional/experienced runner. I saw the warnings and fixed them by running npm audit fix.  I don't know if this was the correct way. I did it in a understanding of kind of responsibility for my node. So when I don't check and don't run every new release I see a risk for the runner. From my point of experience, I can't imagine if this risk would be significant.
So if there is no need to fix this warnings by the user, I'm fine with it.

The main problem is that RaspiBlitz hasn't shipped a new version of mempool in a long time, so once they do I think those warnings will go away.

I don't want to blame mempool. When I noticed it, I checked it on other installations (RTL, RPC), too. I received also warnings, which I fixed with npm audit fix. The question is for me, if the user should have the responsibility, which would be ok, but then he should be aware of and should have the chance to react to this issues.

v1.7RC2 is available to test here with mempool v2.1.2: https://github.com/rootzoll/raspiblitz/blob/dev/README.md#downloading-the-software
And a final v1.7 should now be out within a week .. @wiz sorry that it took so long, but it was a major update for RaspiBlitz .. the next one v1.7.1 will be faster and then can come with v2.2 of mempool.

I said v1.7.1 will be faster and then we decided to add c-lightning and make a big refactor of the setup process and schwups it August - LOL
But for the v1.7.1rc1 will already contain mempool v2.2.1 - so setting this issue to final testing.

mempool v2.2.2 tested and will be part of v1.7.1 final release.

I am running v1.7.2 and see similar warnings during mempool space and Thunder Hub installs. Should I attempt to fix these with npm audit fix or will this break upstream dependencies?
Also, should I upgrade npm from 7.24.0 to 8.7.0 or will this cause problems?
Thanks!


MGMT-10518: separate Driver container and Device Plugin into 2 daemon…

…sets
This PR contains the following:

separate driver container and device plugin container into 2
daemonsets
refactor module reconciliation code that deals with drvier container,
device plugin and build
refactor daemonset code that is responsible for creating daemonset
structure for driver container or device plugin
add unit test

/hold

Are you OK with sticking to that format for commit messages?
In the u/s repo (more than in the d/s repo) Jira tickets have less visibility and we expect contributor also out of Redhat to create PRs.
WDYT?

Are you OK with sticking to that format for commit messages?
In the u/s repo (more than in the d/s repo) Jira tickets have less visibility and we expect contributor also out of Redhat to create PRs.
WDYT?

For now i prefer to stick to this format, it helps to quickly identify the ticket, especially if the commit message
does not provide detailed explanations


[#2300][Bug] Long tail tasks in the Write Stage retry phase results in data loss.

What changes were proposed in this pull request?
The long-tail task in the previous Stage caused data loss, and this bug needs to be fixed.
Why are the changes needed?
Fix: #2300
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existential UT.

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

Project coverage is 51.65%. Comparing base (44bfd0d) to head (355f691).
Report is 6 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2301      +/-   ##
============================================
- Coverage     52.27%   51.65%   -0.62%
+ Complexity     3569     2942     -627
============================================
  Files           543      472      -71
  Lines         29797    22343    -7454
  Branches       2779     2049     -730
============================================
- Hits          15575    11542    -4033
+ Misses        13227    10084    -3143
+ Partials        995      717     -278

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


Should cache the compiled assembly

Currently the snippet is compiled every time the component is executed. It would be better if the result is cached.

Should switch stream implementation to MarkableStream and then mark the stream before the snippet is executed and reset it after the snippet.


Simplify route definition for xstatic route

In the following route definition the filename must always be truthy, so the route with a default seems to make no sense:
https://github.com/Open-MSS/MSS/blob/6bf94d35b4d49f1c24a161ff61a96542f9e6ac8b/mslib/index.py#L87-L96

Please assign me this issue !

Sure


Experimental change to remove getter assertions

Getter methods and assertions SHOULD be freely available via association/play of signs and signifiers.

I will rest it open for some time. I personally :+1: for this. But i am understanding it is huge breaking change and somebody will hate me when update.
I will see people response for this PR.

I love it! :+1: I have had false-positives several times because of getter assertions. Now I use the eslint rule no-unused-expressions to disallow use of getter assertions but that excludes a large section of the assertion api.

@dbrockman yep, known problem for getters. I have added Proxy wrapper for this, but it still works only in ff. Actually i still think that such 'unused expressions' is not right from linter view, but i do not think they could do it better.


Tag based exclusions

What would you like to be added?
Exclusions based on tags (instead of individual exclusions)
Why is this needed?
When managing files in subfolders currently there are settings to exclude individual files. This quickly becomes untenable in large nested directories.
Dataview based queries are read-only substantial functionality is removed if using a query to populate the database

The Excluded files feature is mostly for handling exceptions at this point. Would this comment solve your use case?

Yeah other comment will address the request. I'll close this thread


WASM backend files missing

In previous version of human, the files needed for WASM backend (e.g. tfjs-backend-wasm.wasm) are located in assets folder. However, these files have been missing since a certain version (I noticed it before but I supposed it was a temporary issue). I can get the files from tensorflow dependency folder, but I don't think this is how the library is supposed to work.

I've removed it a while ago and:

updated built-in dev server
updated default wasmPath: '../node_modules/@tensorflow/tfjs-backend-wasm/dist/'
updated all demos to download WASM from CDN:   wasmPath: 'https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>
Reason was two-fold - reduce size of the bundle, but more importantly due to change in Chrome 90 where it started throwing errors due to web server requirement for a specific header in order to be allowed to load WASM binaries.
Built-in dev server was updated to be fully compliant, but I had too many users reporting issues with their own web servers, so it was easier to update demo paths to use CDN location than explain again and again how web server must be properly configured.
I'm open to include WASM binaries in /assets if you have a valid reason?

Oh sorry I wasn't aware that the default path changed too (I manually pointed the path to the assets folder). If we can use the exact same files from the official tensorflow build, that's perfectly fine I suppose.

Or maybe you can update the info for configuration, and inform the users that they can mnually download the files from the above CDN, since not everyone using this library would need to install the dependencies (thus no node_modules folder)

I agree on doc updates, but as it's non-critical it will have to wait for a bit as I'm traveling for the next few weeks.

updated https://github.com/vladmandic/human/wiki/Backends and closing this issue


Missing Admin Links

I seem to be missing General Settings, Maintenance Options, Admin Activity Logs, and Template Diffs from the Admin links despite having full admin rights in the group Administrators. It is a fresh install and unedited..

What platform is this on, it has not been reproduced on current stable php versions

We're running PHP 5.4 on linux. I figured it may be because this is the 5.5.x conversion, but switching to PHP 5.5 did not fix it either.


Web interface: ModeS not being saved

As reported by a user:
https://www.reddit.com/r/stratux/comments/3p0js9/how_to_use_foreflight_ownship/cw38i5x
"I spent some time with this issue this morning. I'm using v0.4r2, with no GPS connected to the Stratux. The Ipad 2 is getting GPS data from an external GPS (Garmin GLO)
Preamble - I haven't used unix in 25+ years so I have no idea what I"m actually doing :)
I edited the stratux.conf file using the following commands (I put sudo in front of every command): kill -9 xxxx (gdl90 process) kill -9 xxxx (dump 1090 process) nano stratux.conf (put in the hex code, lowercase letters a4313a) poweroff, the poweron
When I restart the Stratux and launch the web interface (using the "Stratux" icon) - I see that for some reason the 1090 mode switch is off.
This is happening even throgh the ES_enabled is set to TRUE in the .CONF file
Sliding the switch to "ON" removes my Mode S code from the stratux.conf file.
It appears that the Web Interface doesn't like having a text value in the OwnshipModeS field. It further appears that the Web Interface rewrites the OwnshipModeS field to zero.
I re-editited the ModeS field and will avoide using the Web Interface to see if Foreflight will recognize it in the air.
"

After re-reading this, the behavior does not surprise me. The code assumes the mode-S value is uppercase because it insures it is upper case before storing it. Since the user edited the config file directly and then also used the web settings, the mode-S didn't match up.
I change the validation code handle this case.

I'm the original reporter for this issue.
I just checked and the behavior still exists and seems incorrect.   I edited the stratus.conf file to include "OwnshipModeS:A4313A"   After restarting I used the web interface.  The 1090 switch was set to off (incorrect, it was ON before and ES_Enabled:true is in the .conf file.)
Next, after setting the switch to on, the OwnshipModeS was reset to 0 in the .conf file.
Hope this helps
PS - I'd love to add a Feature Request to have the Mode S be editable in the Web Interface.  Is that allowed?

Confused - ModeS is editable from the web interface, no?

Not that I can find.   Not in Settings

The "mode S" for own ship is on the settings page. However, it's relatively new code so I do not believe it is in 0.4r2 pre-built image.

Thanks - that explains it.  Can't wait to try it (wish I was smart enough to set up a build environment).  Thanks to both of you for the excellent work.

I'm going to mark this as "no need to fix" since the webUI is now available with 0.4r3.


Feature: quota per mailbox

As mentioned in https://github.com/tomav/docker-mailserver/issues/1028, restricting quota per mailbox is an interesting feature. This PR implements a quota per mailbox.
Details:

New commands: setquota and delquota
New optional configuration file dovecot-quotas.cf
Quota warning (email sent to the user) when quota reached 80 and 95%, and when quota is below 80%
Dovecot-quota and Postfix-quota are now synced (use doveadm quota get -u user@domain for getting quota informations)
Supports imap-quota (IMAP clients requesting quota usage information)
Quota per mailbox is DISABLED when using SMTP_ONLY
Quota per mailbox is DISABLED when using LDAP_ENABLE (I don't use LDAP, feel free to implement it if you want, it should not be difficult)

TODO:

~Fix tests~
~When merged, update the wiki~

I can confirm imap-quota is working: tested with rainloop:

Looks like a nice feature, good work!


[Table Designer] Add "New Table" menu to Tables node in OE

This PR adds "New Table" context menu to Tables node in OE.

The end-to-end workflow to add a new table and persist the table to db works fine. (Along with my changes to DacFx)

does it depend on DacFx changes that is not in current STS version? @caohai

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 48.198%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
14526

Relevant Lines:
28633

💛  - Coveralls

does it depend on DacFx changes that is not in current STS version? @caohai

The button itself can show up with current STS version but it would fail to open the designer without related DacFx change.
Do you think it's ok to add the button this way? Or do you think we should do this after merging the DacFx changes and bumping STS version?

@caohai I prefer to do this after the STS vbump.

@caohai I prefer to do this after the STS vbump.

Sure I'll ping you again for this PR once we are done with the DacFx change and vbump.

@alanrenmsft


Add burner binary sensor

This implements #13 as binary_sensor.burner_status.

Hi @Boltgolt, Thanks for this! Wouldn't you think that it would be better to expose a toon.burner_on that would return a boolean in toonlib to implement this feature, instead of reaching into the thermostat state for this? If you do I would be happy to implement it in toonlib and you can refactor this. If not, I can just merge this. Let me know!

That's a great idea, burner_on would be so much clearer for anyone using the library. I originally had no idea what this property did and had to manually test it by turning on the burner.
I'll push a version using toon.burner_on later today.

Version 1.1.0 of toonlib (with the timeout problem) exposes a burner_on boolean property on toon object. Please also update the dependency to match on your update.

Thanks!


Redirect mdb-484 to https://www.fairfaxva.gov/home/showpublisheddocument/22883/638499935919070000

mdb-484 doesn't return any datasets. Should be deprecated and redirected to https://www.fairfaxva.gov/home/showpublisheddocument/22883/638499935919070000

403 error - example for #216


Remove unused tests

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: hzxuzhonghu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

.circleci/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren

I think this is a dupe of https://github.com/istio/istio/pull/14044
thanks for the fix though


Asio : Do not really merge

Still PoC.
Minimal project with lots of hacky things (like including from the result of macro)
Still works on my machine :-)
New MSVC project (I'll figure Cmake out later) with a synchronous ASIO-header only example of a HTTPS Get generator.
If you want to compile locally (and that would be lovely - please give feedback)

I'm using

Microsoft Visual Studio Community 2019
Version 16.10.1

Download asio-1-18-2
Point the MSVC-Include directories at the Header-only library
I only have tried x64 (Debug/Release) so far

First of all, thanks a lot for your interest in this library. It's really encouraging to not only have some users but even better when someone is willing to make improvements.
Making this library work with standalone asio hasn't really been very high on my list of priorities, but anything that could get more users is of course very welcome.
Considering the actual pull request, I haven't looked at it in detail but it is indeed quite a bit from being something I'm willing to merge. You admit so yourself :-)
I have looked a bit for other libraries that can use both boost::asio and standalone asio, but haven't really found anything. Maybe you could find something for inspiration on ways to do it.
Some general points:

Don't bother too much on the server part if you can avoid it. I plan to rewrite it quite a bit at some point. My initial idea was that this should be a drop in replacement for asio::ssl but that didn't turn out to be feasible but it still shows from the clunky API in the server implementation.
boost::system is being used for error codes/exceptions. I assume that could be replaced with std::system_error possibly as a compile time configuration option (e.g. using error_code = std::system_error::error_code or similar).
The boost::winapi parts can be removed and replaced with our own headers. I would really like to avoid having to include any Windows headers at all at some point.
I would prefer if the configuration was placed in config.hpp. Hopefully most can be done by some using aliases as I would really like to avoid too many macros i possible.
Ideally it should be possible to choose whether to use standalone asio at CMake configure time passing something like -DASIO_STANDALONE to CMake defaulting to using boost::asio like it is now.

Thanks for the effort so far and good luck with the code.

Some general answers :-)

Don't bother too much on the server part if you can avoid it. I plan to rewrite it quite a bit at some point. My initial idea was that this should be a drop in replacement for asio::ssl but that didn't turn out to be feasible but it still shows from the clunky API in the server implementation.
It's still very nice. Look how easy an HTTPS request can be..(Assuming you receive the reply in one go)
**
std::string reply;
TemplateDefinitions map{ {"host", host}, {"path", path} };
const auto rqst = ApplyTemplate(BasicGetRequest, map);
std::cout << rqst << std::endl;
try {
stream << rqst;
stream >> reply;
}
catch (const net::error_code& ec)
{
std::cerr << ec.message() << std::endl;
}
**

I assume that could be replaced with std::system_error possibly as a compile time configuration option (e.g. using error_code = std::system_error::error_code or similar).
Really there are only two uses for errors/exception in the code. Simple error_codes, so asio::error_code slots in, and an error with an additional string. Easy to emulate

The boost::winapi parts can be removed and replaced with our own headers. I would really like to avoid having to include any Windows headers at all at some point.
Its already like that, I created a new sub-directory called winapi. The includes pick this up instead of boost::winAPI.

I would prefer if the configuration was placed in config.hpp. Hopefully most can be done by some using aliases as I would really like to avoid too many macros i possible.
Will do

Ideally it should be possible to choose whether to use standalone asio at CMake configure time passing something like -DASIO_STANDALONE to CMake defaulting to using boost::asio like it is now
Already done

The next thing to do is fix up the boost build again. I sort of ploughed-through to get the examples compiling, and the boost-version has been left broken, just with macros, where it needs fixing.
It has to be said, that part of me wonders if we should drop the boost-bit, and just call it an asio-fork. However, I want your fixes (always the trade-off), so I feel like it's worth the try even if it is macro-hell :-)


Typedoc Improvements

Summary
Some typedoc improvements: There's now a root 'readme' for the typedocs, and we're only running typedoc once and configured it slightly differently.

Explain the motivation for making this change. What existing problem does the pull request solve?

Our typedocs previously weren't linked to each other correctly. So e.g. I wouldn't be able to take a look at the FileContent type when I was inside the FileSystem class. Also there were no links to FileSystem form the root of the docs, you'd have to know about webnative.fission.app/fs.
Test plan (required)
Hm. Ideally I want the docs to be rendered in github actions, but for that I'd need access to the fission account that belongs to webnative.fission.app (I think you have that @icidasset?).

I think auto-publishing to webnative.fission.app should be another PR, along with auto-publishing the npm module. Those two things should happen at the same time, after all.
I guess that's also a release-workflow kinda thing.


⚠ Jenkins Plugins Artifactory has degraded performance

In befee5e, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1206 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 8fe639d after 1 hour, 47 minutes.


Keep scrypter instance

Is the split between new_intializer and initialize necessary for performance? Could we just pass in the arguments from new_initializer to initialize?
I would like to try these changes with postcli but CI is failing.

Yes, the split is required to preserve allocations and compiled kernel.
I still need to work on this change, please don't try it yet

@fasmat, can you try it now?


number_guessing_game:UI Enhancements: Text, Color, and New Game/Try Again Button

hey @TusharKesarwani I would like to propose some UI enhancements for the website, including improvements to text and color, as well as the addition of a 'New Game' or 'Try Again' button. Furthermore, I suggest incorporating a visual reaction when the player wins to enhance the overall user experience.
Specifically, I recommend the following changes:
1.Text and Color: Standardize and optimize the text styles and color palette across the website for consistency and visual appeal.

New Game/Try Again Button: Introduce a dedicated button,,positioned on the page, to allow the user to start a new game or retry the current game. This will provide a clear and intuitive way for users to initiate a fresh game session.

Winning Reaction: Implement a visual reaction or feedback when the player successfully wins a game. This can be achieved through animations, confetti effects, or any other appropriate means, to celebrate the victory and engage the player.

These UI enhancements aim to improve the overall aesthetics and usability of the website, creating a more enjoyable and engaging experience for users.
I am willing to contribute to this issue  under SSOC '23 by working on the necessary code changes. Any guidance or suggestions regarding the implementation of these UI fixes would be greatly appreciated.

Game projects are not accepted


Commented out AlanAIComponent & added missing tags

Website failed to launch on vercel app. Issue was caused by AlanAI because API key was not provided, so website failed to load. I have commented the AlanAIComponent so website will work properly. Also few tags were missing. Also I have spare api key with 10k interactions i can add it to the website so users can use AlanAI for a while. Do let me know.

Website is somehow working now. 😑


jrc_mus-sc-zp104a

proofread tif found here: /avetissiane/tiff_slices/jrc_mus-sc-zp104a/s4_amira_proofread.tif
also wouldn't mind having a second set of eyes on this one

/avetissiane/tiff_slices/jrc_mus-sc-zp104a/s4_amira_proofread.zarr

jrc_mus-sc-zp104a.zarr/recon-1/labels/inference/segmentations/nuc

Thumbnail
/cellmap/nrs/data/jrc_mus-sc-zp104a/thumbnail_nuc.jpg
Ready for meshes

DOI created
https://doi.org/10.25378/janelia.26513224 https://figshare.com/s/562e89d71f6fe38afa24


libdwarf: version the shared object

Motivation for this change
libdwarf sets its soname to libdwarf.so.1 so anything that links -ldwarf is built to actually link against libdwarf.so.1. Install the versioned shared object and symlink libdwarf.so to it so binaries that link against libdwarf can be loaded and run successfully.
Fixes #20994. Should be backported to 16.09.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

This whole "installing by hand" is strange. I wonder if there's a make install or something (already), to avoid similar problems in future.

There isn't; the upstream Makefile.in includes an install rule, but all it does is echo "No install provided, see comments in the README". 😕

Backported, too. Thanks :-)


Add dark mode

Add a dark/light mode toggle with correct theme colors

Hi!
Please assign me to that ;)

Hi! Please assign me to that ;)

Hope you enjoy workign on it !


Spot extraction cleaned up

Replacing PR #86, part of #34.
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

@fionaRust #86 has now been approved by two reviewers.

I have looked through all of the code and it seems to make sense to me, and be logical. However I have left a couple of minor comments about some comments in the code.
I have also run all the tests and it said the unit tests, CLI tests and All tests ran successfully. Codacy and Travis are also fine.


Play command flags

Rationale
When playing a playlist, you sometimes want to shuffle the playlist before playing it. Sometimes you might even want to play it right after the current track, or even interrupt the current track and skip right away. These can be all achieved via flags and it was a much appreciated feature from Groovy.
Description
When using the play command, adding a flag modifies the behavior of the play command.
These were the available flags in Groovy (with the exception of -choose, which seems more appropriate as a search command). They were also available as shorthand flags with the first letter only:

-all: This instructs Groovy to queue all returned songs (...) when you queue a YouTube video in a playlist.
-shuffle: (...) randomize the order of songs in the playlist before it's added to the queue (...)
-next: (...) the song you queued is placed directly after the playing track in the queue. (...)
-jump: Instantly jumps to the track you just queued. (...)
-reverse: Queue the playlist in reverse order.

I'd like to return to this feature, since it's the one I miss the most and I want something to start out with the code as well. Do you think it makes sense to keep the flags in the query, or should I make these optional booleans in the slash command?

Definitely adding these as optional booleans is the way to go moving forward for any additional arguments imo!

@rafaeldamasceno I reckon this is a good chance to use select menus as they support both single and multiple selections.
Was it possible to use multiple flags on Groovy?

Was it possible to use multiple flags on Groovy?

Yes, it was.

@rafaeldamasceno I reckon this is a good chance to use select menus as they support both single and multiple selections.

Yes and no... Do we want to break the current flow of just posting the query and having the bot reply? We'd need to have this intermediate step to select the flags (which I don't even know if is possible to do in the same slash command as the one that receives the query), or create a new command for using the flags. None of these options seem good to me.

@rafaeldamasceno have you tested it out? I'm pretty sure it is pretty seamless using discord's built-in optionals and stuff

@afonsojramos No, but I'm pretty sure that select menus are not part of the command options, but instead part of the message components. This would mean having to use play (or any other command for this purpose), having the client render the select menu, and only then we would have the correct flags. This contrasts versus using play and having all the flags as optional boolean application command options, and when you hit enter, you are done.

@rafaeldamasceno You're absolutely right, I was confusing that with application command option choices.
Unsure of the limitations (this is still quite new) but maybe something like this? 🤷♂

@rafaeldamasceno Do you want me to assign you to this topic?

@rafaeldamasceno Do you want me to assign you to this topic?

Yes, please.

Just an FIY, after #135 we may use custom yt-dlp arguments to help us implement some flags.
Could be handy, but haven't checked in detail.

@rafaeldamasceno As discussed, moving the issue to @StaticRocket.

If @rafaeldamasceno is fine with it then sure!

@aquelemiguel Regarding the examples you gave for the options I presented in #140, they are both correct.
The only feasible ways I see to implement this are two:

The first one is to have it as booleans like /play query:query all:true shuffle:true where the all flags are optionals and the ones you want to activate need to be set to true, since it's the closest thing to flags (and unfortunately it's not looking like it will change anytime soon. I don't think it is cumbersome to set, but I agree it looks atrocious. The advantage is that there's helper text and it's obvious to the user what is going to happen when setting values to the flags.
The alternative would be /play query:query flags:as, where the flags parameter will have letters for each flag you want to activate. Since flags would be an optional, there isn't any need to add dashes or anything. For me, this is the way that would make it fastest for power users to interact with the bot and now that I think about it, is about the same way as Rythm worked and advertised these features (as advanced usage flags).

All different play modes implemented, closing.

Keeping it closed for now. If needed, let's open a separate issue regarding multiple flags for tracking. But realistically, we must wait until Discord supports some sort of variadic arguments to justify any changes.


Validation needed for already existing resource names chosen by the user.

Here are a few examples:

[ ]  Mi Sql instance name

[ ] 2. Postgres Server instance name

[ ] 3. Arc dc name, arc namespace.

It would be good to do these validations before exiting the corresponding wizard pages so that the user can be alerted about failures that they will run into during notebook execution.

Not something we're planning on doing currently


.WAV file header reader should ignore JUNK chunks.

Currently, any .WAV file with a JUNK tag in the header will produce errors when being processed by the audio_stream initializer. These chunks ought to be skipped over.

Issue has been resolved.


Feature request : Categorized Library items

At the moment it seems library items can't be distinguished from which library it is being rendered.
It would be very nice to have these library items categorized something like this.

@ad1992

@ad1992

I think this would be a useful feature. Would like to hear what the team thinks about it

I came here to suggest this exact same thing, it would make using the Library a lot easier

This is the only thing that prevents me from using Libraries more than I do. Cause I have so many libraries downloaded for different things I use Excalidraw for.

also a Recently Used category

+1 to this feature.

+1

+1


Java 17 instead of Java 11 (i.e. build with - and therefore effectively require)

Does anyone reading this have any thoughts, both Pro as well as Con, about switching this project from Java 11 to Java 17?
Recent contributors (@jejanim @chrisbloe @MazenAmria @mrdziuban @simonzkl @dassio @skyding1212  @rkumar0322 @simonzkl @bib-marko @ StruckCroissant @jamalmatgmail @gaetannandelec-ibboost @3l0w @Blanco27) any input? Do you MIND if this project were to require Java 17 instead of Java 11 in a next (future) release like 2.7.0 or 3.0.0 or whatever version number I chose for it?
This question has come up in #638.

Full disclosure: I personally actually don't really care either way. I just want to avoid loosing end-users, because of this... 🤣

That don't bother me, we moved our code base recently to java 17

Same here, i think it would be good given main project like Spring boot 3.0 / Hibernate 6 have moved to it

We're considering upgrading to java 17 in the project which I'm using MariaDB4j in, so I'm fine with it.

OK, then let's do this - I'm switching 2.6.1 (unreleased) to 3.0.0 in #690 and will then build with Java 17 instead of 11 (perhaps not today anymore, but on another upcoming weekend or so).

Resolving now, as #691 just switched the build from Java 11 to 17 (on CI, for unreleased 3.0.0, watch #696).


[WIP] Discover featureparity

Informal opening to review the backend code as the maps and dates are finalized.

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-3983


How to make  my own customized transformers

I want to use my own data augmenter ,I don't want use image_aug_default .
So, how to make my own data augmenter .Can someone give me a tutorial.
Thanks.

Are you using gluon or symbol? You can use gluon.data.DataLoader or mx.image augmenters, respectively.

@eric-haibin-lin This has been inactive for a while. Can this be closed ?

Closing due to inactivity. Feel free to reopen if not resolved yet


feat(core): add IPC channel

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[ ] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

Is this mobile only feature?

I've also implemented it for desktop (it can easily be backported to v1) though the main idea here is to use it to send events from the Kotlin and Swift sides to the webview (eventually we'll also need a communication channel between mobile and Rust, but that's a little complicated since it involves static variables).

Yeah it is part of the public API @JonasKruckenberg

Can we hold off on merging this until I had the chance to review it tomorrow?

Sure @JonasKruckenberg


can not create a network using TFT payment option in chatflow

steps: network name --> TFT --> 1d --> IPV4 --> configure ip range myself & choose ip range for me
error : below picture

no wallet setted
jsx version

root@3bot:/sandbox/code/github/threefoldtech/jumpscaleX_threebot# git log -1
commit 5e0e42c60eebed766f92addbb03f24a55fb47e07 (HEAD -> development, origin/development, origin/HEAD)
Author: Rana Tarek Hassan<EMAIL_ADDRESS>Date:   Sun Apr 26 16:03:52 2020 +0200

    Use currency from network instead of asking user in deploy solutions chatflows

Currently TFT is not supported, but an error is returned from the explorer indicating this. Will just make it visible in the chatflow

verified


Bond balances in genesis

Motivation
We want to be able to bond balances in the genesis block
Related PRs
https://github.com/AleoHQ/snarkVM/pull/2308

Should this target the mainnet branch?

Closing in favor of https://github.com/AleoHQ/snarkOS/pull/3091


Wrong thumbnail :(

From the reporter: Google Chrome, version 47.0, newest build. However, on Firefox, it shows the correct thumbnail.
I can't repro on Chrome 47.0 on Mac.

No reports of this in almost a month so I'm calling it closed.


move include/global_project to adapters

resolves https://github.com/dbt-labs/dbt-core/issues/8925

Problem

Solution

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue

🎩 ran dbt docs generate && dbt docs serve off this branch & off main + jaffle_shop, manually inspected no differences.

[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[x] This PR includes type annotations for new and modified functions

TODO: tophat docs with 3.8

However, I have not verified that the macros get correctly inherited from an adapter in another repo (e.g. dbt-snowflake). Do you expect there to be an issue there? Put another way, will we need to do the same migration at the adapter level? I'm guessing that's not the case given that dbt-postgres was not updated. But I want to call it out explicitly.

I don't think this should be an issue for adapters defined in other repos since they are installed in the same manner as dbt-postgres. Unfortunately it'd take a lot of effort to test this incrementally because the other adapters do not have corresponding migration branches yet.


clang-tidy performance-faster-string-find

Optimize calls to std::string::find() and friends when the needle passed is a single character string literal. The character literal overload is more efficient.

Runs clang-tidy over the compilation database.
I may push more changes from clang-tidy results.

Nice :) Looking at clang-tidy now. Was it easy to setup and run on the whole project? Do you have an idea of how we could make this kinda automated with a script or something?

I already wrote a script for this some time ago, and I believe I put it in the root dir, but it's not there anymore. I must have deleted it... You can look at the linux cmake scripts to see how to build the OS with tidy.

Yeah it figured it out with the help of the docs, already had the tooling. On Mac:
ccmake ..
# turn on CMAKE_EXPORT_COMPILE_COMMANDS
make
<EMAIL_ADDRESS><options> # homebrew llvm


[ Tensor ] fix the dim type

Previously, the dimension type of tensor dim is unsigned int and it cause the error for bigger problems. This pr change the demeion type to size_t.
Also, This pr fix the user_data for stop callback to use user_data.
Resolves:
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2137. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jijoongmoon, api/ccapi/include/model.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.

:octocat: cibot: @jijoongmoon, api/ccapi/include/model.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2137-202303131106370.56025910377502-322b266711e05c8a5fc19109f451900ada9b1f92/.

:octocat: cibot: @jijoongmoon, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2137-202303131144320.46167492866516-e8a9c5887620e1fcd894ac7a2f62bd1fb8dffdfa/.

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.


Comparison Report: Add custom values page

Add a Custom Values page to the comparison report that compares the custom value measurements of the two runs.

Open: since we can't know on a semantic level what those custom values are, it does not make so much sense to mark changes as good=green, bad=red - we don't have an idea if an increase is good or bad. Also, in the example that is often used in load tests, the long runners are counted per scenario and usually there are different scenarios with a few long runners each on different load tests, but marking these in light gray as added/removed also does not quite hit it semantically...

Open: since we can't know on a semantic level what those custom values are, it does not make so much sense to mark changes as good=green, bad=red - we don't have an idea if an increase is good or bad.

True. Maybe we can use a yellowish color instead of read/green. I thought about using the absolute difference between the old and new value to determine the shade to apply, but then again, we don't know what is a small difference and what is a significant difference. So it's either just highlighting the cell with yellow if the values are different or not coloring the cell at all.

Also, in the example that is often used in load tests, the long runners are counted per scenario and usually there are different scenarios with a few long runners each on different load tests, but marking these in light gray as added/removed also does not quite hit it semantically...

I believe it is okay to mark custom value cells as "(added)" or "(removed)" if values appear in only one of the two reports. In general, comparing reports only makes sense for very similar runs, i.e. same scenario set, same load profile. So we should be fine here.


Update supported characters to reflect IBM Spectrum Protect 6.3.3 or later password requirements

From the v8.1.10 set password documentation:

set password
if you do not use an LDAP directory server to authenticate passwords
a b c d e f g h i j k l m n o p q r s t u v w x y z
A B C D E F G H I J K L M N O P Q R S T U V W X Y Z
0 1 2 3 4 5 6 7 8 9 . ! @ # $ % ^ & * _ - + = ` ( )
| { } [ ] : ; < > , ? / ~

Passwords are stored in the IBM Spectrum Protect server database and are not case-sensitive.
if you DO use an LDAP directory server to authenticate passwords
a b c d e f g h i j k l m n o p q r s t u v w x y z
A B C D E F G H I J K L M N O P Q R S T U V W X Y Z
0 1 2 3 4 5 6 7 8 9 . ! @ # $ % ^ & * _ - + = ` ( )
| { } [ ] : ; < > , ? / ~

Passwords are case-sensitive and are subject to more restrictions that can be imposed by LDAP policies.

References

https://www.ibm.com/support/knowledgecenter/SSEQVQ_8.1.10/client/r_cmd_setpassword.html

I'm also going to update the provided Python script to reflect the changes too.
If we get to the point where it's not maintainable (not likely, it's very small), we can drop it or heavily annotate it to indicate that it is no longer supported


[BUG] Missing New story button, missing new page button

Tested on Foundry version 9, build 269
Tested Storytellers:

0.3.3
0.3.4
0.4.0
0.4.1
1.0.0
1.0.1
1.0.3

None of the above versions had "new page button" or "new story button". Are these old features that no longer exist in any build?
Screenshot:
https://imgur.com/a/x4c2Bqr

@ArdonDaros
I suspect that you are using an old version of the mod with a relatively new version of foundry. In that case, the mod may not work correctly.
From version 1.0.0 only foundry v10 and higher is supported.

When you use the latest version, you should have a way to create history through the create journal button.


Add reconnect page help link to unconnected section

Resolves ticket #2374
Adds in a link into the unconnected section with a link to the appropriate section in the user guide to help users in reconnecting pages back into their form.

can we use the locales please

can we use the locales please

Now using locales


Fix endpoint weight lost

fix endpoint weight lost bug
fix convertxds unit tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature-istio_adapter@b9ca4c8). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             feature-istio_adapter    #1145   +/-   ##
========================================================
  Coverage                         ?   56.48%
========================================================
  Files                            ?      246
  Lines                            ?    23015
  Branches                         ?        0
========================================================
  Hits                             ?    13001
  Misses                           ?     8845
  Partials                         ?     1169

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 b9ca4c8...96c8c7c. Read the comment docs.


Use webpack4 hook that replaced deprecated plugin api

Will resolve/fix  #3

thanks! released version 1.0.2:

added backward compatibility for old webpack API (<4.0.0)
webpack 4.* should not throw warning anymore.


refactor: remove usage of assert from cache.js [#973]

Type:
The following has been addressed in the PR:
remove all import assert from 'assert'; in favor of Jest matchers in cache.js

@juanpicado can you kick off the test_node8 again in CircleCI.  Looks to be a test unrelated to the changes made


How do I properly install this?

Hi, I'm new to CakePHP. I've followed the instructions, but I don't have the proper layout. I've copied the layouts from your examples folder and transferred them to src/Template/Layout/TwitterBootstrap, and extended a view to try it out but the layout did not change. I also tried to replace CakePHP's ../Layout/default.ctp with bootstrap-ui's default.ctp and still nothing. What am I missing?
Edit: also when I inspect element, it looks right, as it uses the bootstrap's style sheet and script, along with Jquery's.

The default layout assumes you have the CSS and JS assets installed. Before using the example layouts, make sure you get the default one working.
If you are using bower, instructions are included in README for installing the assets.
Another alternative would be to replace the cake app default layout by the one included in the plugin, but make sure you comment out the public $layout = 'BootstrapUI.default'; line in AppView. You will also need to change the assets' paths to CDN or wherever you have them installed.

I have solved the problem. I already had it installed, the CSS and JS assets. The problem was where bootstrap-ui is looking for them. It looks at myapp/webroot/css/bootstrap/bootstrap.css when I had it at myapp/webroot/css/bootstrap.css. I just had to edit the extra path out, then it works!
Thanks


Implementing a Custom Runner

Hi,
In my project, I have 2 datasets that include different objects in them and I have to train a detector that alternatingly processes one dataset and the other. Thus, I modified the Faster RCNN detector to have 2 RoI and RPN heads but one common backbone and neck. The required pipeline pseudocode is given below:
def train_one_iter():
   model.set_head(1)
   data1 = dataloader1.next()
   loss = model(data1)
   loss.backward()
   optimizer.step()

   model.set_head(2)
   data2 = dataloader2.next()
   loss = model(data2)
   loss.backward()
   optimizer.step()

However, the current runners that I observed do not allow for custom head changes and alternating dataset pass together. Is there any module that makes this kind of a pipeline applicable? If not, how can I modify the current runners to have this kind of training structure? Are there any examples for this purpose?
Thanks.

@barisbatuhan We are refactoring the runner, which will be released next month. You can wait until release time to implement it will be easier.


Mat/Par - UX Improvement - Coming early or extending leave

Add information about coming early or extending maternity/parental leave in the email send to applicants on approval

To be sent to all applicants on approval

Copy

Returning early or Extending leave

You must also inform your supervisor if -

You’re requesting an extension to your leave, and you have not taken the maximum parental leave and extended childcare leave (combined maximum of 78 weeks total).
You’re wanting to return earlier than the date you indicated on your digital application.

Ask your supervisor to submit an AskMyHR service request for "My Team or Organization" under the following category "Leave & Time Off > Maternity, Parental, Adoption" advising that you are returning early or extending your leave. AskMyHR will process the request and confirm.

@fazil-ey do you want the last paragraph in the ticket to be a bullet point too?
Lastly, do you want this to go at the end of the email, above the "NOTE"?

@Andrew-Vargas-bcgov  above NOTE
I have updated how the bullet points should be. It should go as the last bullet point with sub points

hey @Andrew-Vargas-bcgov can you change the last bullets to include. main bullet like so - ( for some reason Zenhub is showing this in the preview but not updating the ticket to reflect the change)

thx @Andrew looks good!

I do not see this in the latest emails. Has this been released?

Getting the content in the emails


filter in voting dialog not working

see pic:

I tried special signs like ! or ä
also normal chars from a-z

but it seems the filter does not apply them

version: <IP_ADDRESS>, on linux

Same for me. It filters when two or more characters are typed in the input in my case. v<IP_ADDRESS>(Ubuntu)

@MagixInTheAir good finding
if not bug, then this text change could be fine:

"Filter (in Title field, with 2 chars)"

Fixed !


Scrollbar div still takes up space even if there's no scrollbar

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Even when there is no need for a scrollbar, the scrollbar div is still rendered and has a non-zero size.

Steps to Reproduce

Make sure that scrollbars are enabled (macOS only?)
Open a file whose contents can fit horizontally or vertically on the screen without a scrollbar.
Through devtools, notice that there are scrollbar divs taking up 20px.

Expected behavior: No scrollbar div if there's no scrollbar
Actual behavior: Scrollbar div is there
Reproduces how often: 100%
Versions
Atom    : 1.19.0-dev-9e64abef3
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0

@50Wliu I always render the dummy divs because we use them to measure the width of scrollbars for use in various calculations. Maybe we can apply a CSS class to the scrollbar divs when the scrollbars are actually visible and only change the cursor then?

Basically, can you clarify the incorrect behavior that you're seeing as a result of the scrollbars always being rendered? Is it just the cursor styling or might it be other things as well?

Right now, the only thing I've noticed as a direct effect of this is the incorrect cursor styling.  Though I suppose that any theme that adds extra styles to .horizontal/vertical-scrollbar will have those styles appear even when visually, there is no scrollbar.  For example:

I'd like to second @50Wliu's point:
For very small editors the (invisible) scrollbars take up most of the editor's area, hence the cursor will be incorrect for most of it:

@as-cii If you want to look into this one tomorrow, here are some thoughts: I think we may be able to hide the dummy scrollbars and only show them when they're actually needed or forceScrollbarsVisible is true. There are places where we assume they're rendered though, and I'm not sure if we'll need more sophisticated update logic to ensure that any time we assume they are rendered that they actually are.
Another approach might be to force their height / width to 0 unless forceScrollbarsVisible is true.


Updates for version 1.6.0

Docs Build status updates of commit 483b2a5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

prosepy/docs-ref-conceptual/knownissues.md
:white_check_mark:Succeeded

prosepy/docs-ref-conceptual/relnotes.md
:white_check_mark:Succeeded

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


Update YYEVAPlayer.h

change delegate from assign to weak, avoid crash when some player working in repeat mode not finished but still own the delegate trigger crash.

how to reproduce this crash:

new a vc
play a repeat video with repeat count e.g. 100 times
return last vc before 100 times finished
crashed


Bug: Hydration Error causing application to break

Initial rendering of images causing application to break :
This is because of dynamic rendering with SSR I believe.
The error says :

that that UI doesn't match with which was render on server.
and didn't expect server HTML to contained  tag - which I looked into it-

Causing Hydration errors
Here are some screenshot:

Solution: As the fluent UI don't support SSR - we can work around with state by manually telling Next.JS when to render them
Could you assign this to me

@IrfanshaikH-7 Thanks, I've assigned it to you.

Could you please review it


ver 1.0.0 improvements

Coverage remained the same at 90.894% when pulling 69dbf392957cf5e02101197e96d8818dd397fea9 on kk/ver-1.0.0-fixes into 3958eea09d6440bbc4cc787304c2ec4135f023f4 on master.


fix some methods and classes missing from readthedocs documentation

Some of the methods and classes were missing in the sphinx generated documentation due to some errors in the intersphinx mapping and dependencies.  This PR resolves those documentation issues.
How I validated: I changed the latest readthedocs build to point to this branch, and then validated that the documentation was finally building.  I switched the branch back to master branch.

Codecov Report

Merging #509 (f5d4f7a) into master (33ca855) will decrease coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #509      +/-   ##
==========================================
- Coverage   86.62%   86.33%   -0.30%
==========================================
  Files          62       62
  Lines        4046     3959      -87
==========================================
- Hits         3505     3418      -87
  Misses        541      541

Flag
Coverage Δ

unittests
86.33% <ø> (-0.30%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...erpret_community/mimic/models/explainable_model.py
81.57% <0.00%> (-3.21%)
:arrow_down:

...ython/interpret_community/common/base_explainer.py
86.95% <0.00%> (-1.05%)
:arrow_down:

...hon/interpret_community/explanation/explanation.py
84.63% <0.00%> (-0.79%)
:arrow_down:

...mmunity/_internal/raw_explain/data_mapper_utils.py
80.85% <0.00%> (-0.79%)
:arrow_down:

...n/interpret_community/shap/gpu_kernel_explainer.py
22.55% <0.00%> (-0.58%)
:arrow_down:

...hon/interpret_community/mimic/models/tree_model.py
90.90% <0.00%> (-0.48%)
:arrow_down:

...interpret_community/mimic/models/lightgbm_model.py
85.63% <0.00%> (-0.33%)
:arrow_down:

python/interpret_community/common/aggregate.py
90.32% <0.00%> (-0.31%)
:arrow_down:

...n/interpret_community/mimic/models/linear_model.py
93.50% <0.00%> (-0.16%)
:arrow_down:

...community/_internal/raw_explain/feature_mappers.py
95.72% <0.00%> (-0.15%)
:arrow_down:

... and 13 more

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 33ca855...f5d4f7a. Read the comment docs.


Do we need NotebookManager?

The docstring for NotebookManager:
/**
 * It is not possible to 'open' an already open notebook i.e.
 * a notebook that has not been shutdown by Pluto.
 *
 * This manager manages the currently
 * open notebooks and gives us ids of already open notebooks
 * so that we can directly go to the **edit** url.
 */

This is not true: you can open a file that is already open, and Pluto will just send you the ID of the already running notebook.
The Pluto server already keeps track of the list of running notebooks, and it seems error-prone to also keep track of this list in the electron process. We could:

Not keep track of it at all, where do we need it? Perhaps we can make a change to Pluto's REST API to avoid this need? E.g. add a header that says notebook-was-already-running: true?
If it is necessary somewhere, request the notebook list on demand, without storing the result.

There are certain features that require this list, but I'd say performing this query on-demand is the best way to go


focal loss什么时候能支持softmax?

你好，focal loss什么时候能支持softmax?

For now, we do not do that because it is rarely used.


Text is displaying in code block

This text section is displaying in a code section.  That does not seem to be the intention.  It is not clear to me whether I fixed it.  Someone may need to review and edit.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Queued Subjects coming from different Subject Sets, despite specifying Subject Set

Issue Report

When requesting a queue of Subjects from the /queued endpoint, if a Subject Set is specified, sometimes the endpoint returns Subjects from multiple Sets instead.
This issue only appears when the user is not logged in.
If the user logged in, the queue only returns Subjects from the specified Set, as expected.
Context: the error was noticed on the Anti-Slavery Manuscripts project, where users are given the choice of transcribing letters from different time periods (subject sets). This is mostly noticeable on staging, where the ASM uses a very small set of Subjects and Subject Sets.

Testing
Setup:

On staging, we've created project 1764 / wgranger-test/anti-slavery-testing
The project has one workflow, workflow 3017
The workflow has 5 Subject Sets, with one Subject each:

subject set 4420: [ subject 73330 ]
subject set 4422: [ subject 73331 ]
subject set 4423: [ subject 73332 ]
subject set 4424: [ subject 73333 ]
subject set 4425: [ subject 73334 ]

Expectation:

When I request /queued?subject_set=4422, I should receive only [73331]

Note: Anti-Slavery Manuscripts's staging is tested mostly on localhost, but a potential quick test can be done via https://www.antislaverymanuscripts.org/?env=staging
Testing 1: User isn't logged in, attempts to select Subject Set 4422.
Request: GET https://panoptes-staging.zooniverse.org/api/subjects/queued?http_cache=true&workflow_id=3017&subject_set_id=4422
Response: Response body contains 5 Subjects: [73333, 73331, 72815, 73330, 73332, 73334]. This is unexpected. (Wait, where did 72815 even come from? That's from Subject Set 4384.)
Response will always contain those Subjects, in that order.

*Testing 2: User is logged in as user zootester1, attempts to select Subject Set 4422
Request: GET https://panoptes-staging.zooniverse.org/api/subjects/queued?http_cache=true&workflow_id=3017&subject_set_id=4422
Response: Response body contains 1 Subject: [73331]; This is expected.

Status
Open for investigation, tagging @camallen . Let me know if I need to share project setup details and/or other API calls to rule out errors on the front end/project setup side of things.
I'm not sure how to describe the priority here; one one hand, I'd like to see this fixed for the ASM beta test on 8 Nov. On the other hand, the project is otherwise functioning, and it's possible for users to actually classify Subjects, they just can't choose the ones they want. On the third mutated hand growing out of my chest, if this really is an issue (and not just poor API calls/project setup on my behalf) then other projects may be affected, which can't be good.

The workflow is set to use the default subject selector, which recently changed to always be Designator. Designator doesn't do grouped selection (yet). I've switched it over to Cellect. @shaunanoordin Did this fix your issue?
@camallen should the workflow bypass designator and use built-in selection for grouped?

@marten thanks for looking into this, if we don't have on the horizon plans to support grouped selection then yeah we should modify the code path to deal with this. I think this is pretty local to us right now though so happy to just switch the workflow selectors to deal with it.
@shaunanoordin Also noting there is a bug with cellect if all the data is retired it'll switch back to a random group, not the one you asked for. It's on my list to work on but please do remind me as it'll only start showing up once the data starts retiring for your workflow.

@marten SWEET. 👍  The switch to Cellect now fixes the issue. As either a logged in user or a not-logged-in user, when I choose a Subject Set, I am only served Subjects from that Set. This has been confirmed on staging.
@camallen Noted on the bug with cellect, and thanks for the alert! I wouldn't be too worried about having that retirement issue settled for ASM so soon (although I assume this is related to the "retired images seen again" issue of WildCam), but I'll have to pay closer in a few months.


Update to ember-pouch

As part of the update to Ember 1.13 (see #143), we need to update our ember data adapter to [ember-pouch] (https://github.com/nolanlawson/ember-pouch).

We may need to migrate data as part of this update.

We definitely need a migration script.  Working on that now.

Migration script is now available at https://github.com/HospitalRun/hospitalrun-server/blob/master/utils/pouch-mig.js

:+1:


Use Markdown to render MCQ content

Features

MCQ options rendered in markdown
Hints rendered in markdown

Notifications now accept Elements as well

Pull Request Test Coverage Report for Build 644

4 of 7 (57.14%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 26.057%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/workspace/MCQChooser.tsx
2
5
40.0%

Totals

Change from base Build 641:
0.01%

Covered Lines:
886

Relevant Lines:
2891

💛  - Coveralls


Add pay account api object

Codecov Report

Merging #30 (616b837) into main (9581b75) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #30   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           51        53    +2
  Lines          805       823   +18
=========================================
+ Hits           805       823   +18

Impacted Files
Coverage Δ

aioalfacrm/client.py
100.00% <100.00%> (ø)

aioalfacrm/crud_objects/__init__.py
100.00% <100.00%> (ø)

aioalfacrm/crud_objects/pay_account.py
100.00% <100.00%> (ø)

aioalfacrm/models/__init__.py
100.00% <100.00%> (ø)

aioalfacrm/models/pay_account.py
100.00% <100.00%> (ø)

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 9581b75...616b837. Read the comment docs.


Alex Hill - Write Up 1 Submission

Submission

Grade:
Problem 1: 2/2
Problem 2: 0/2
Output of testing/test_all.sh, showing differences in incorrect output:
Testing problem1
Success testing problem1 case 1 in 0.00 seconds.
Testing problem2
Failure testing problem2 case 1, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 2, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 3, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 4, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 5, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 6, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 7, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers

When using test all it will give you an error because Problem 2 requires user input. I believe test_all.sh does not account for this? Or maybe I just did it wrong idk.
Cheers,
Alex Hill

On Sep 21, 2016, at 12:26, Donald Pinckney<EMAIL_ADDRESS>wrote:
Grade:
Problem 1: 2/2
Problem 2: 0/2
Output of testing/test_all.sh, showing differences in incorrect output:
Testing problem1
Success testing problem1 case 1 in 0.00 seconds.
Testing problem2
Failure testing problem2 case 1, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 2, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 3, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 4, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 5, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 6, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 7, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

test_all.sh automatically feeds in "user" input.
You did it right conceptually, but you put extra printing into your code that was not specified in problem2.README, so the testing script says that your output does not match correctly.
I'll talk about it today.
Donald Pinckney
Sent from my iPhone

On Sep 21, 2016, at 1:38 PM, Alex<EMAIL_ADDRESS>wrote:
When using test all it will give you an error because Problem 2 requires user input. I believe test_all.sh does not account for this? Or maybe I just did it wrong idk.
Cheers,
Alex Hill

On Sep 21, 2016, at 12:26, Donald Pinckney<EMAIL_ADDRESS>wrote:
Grade:
Problem 1: 2/2
Problem 2: 0/2
Output of testing/test_all.sh, showing differences in incorrect output:
Testing problem1
Success testing problem1 case 1 in 0.00 seconds.
Testing problem2
Failure testing problem2 case 1, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 2, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 3, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 4, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 5, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 6, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
Failure testing problem2 case 7, since there was a difference in output:
1,2d0
< Please enter 4 numbers
< This is the sum, mean, product, and geometric mean of your numbers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


build!: upgrade engines field to >=8.10.0

This drops support for all Node.js versions < 8.10.0, please upgrade
BREAKING CHANGE: library now requires Node >= 8.10.0

This breaks usage with firebase cloud functions. Firebase cloud functions require engines to be "8" or "10" and won't accept any other values. Due to that, this library fails to compile when attempting to deploy cloud functions firebase deploy --only functions with
i  functions: preparing functions directory for uploading...

Error: Error occurred while parsing your function triggers.

Error: @grpc/grpc-js only works on Node ^8.13.0 || >=10.10.0


Multi GPU training problem

[ ] I have marked all applicable categories:

[x] exception-raising bug
[ ] RL algorithm bug
[ ] system worker bug
[ ] system utils bug
[ ] code design/refactor
[ ] documentation request
[ ] new feature request

[x] I have visited the readme and doc
[x] I have searched through the issue tracker and pr tracker
[x] I have mentioned version numbers, operating system and environment, where applicable:import ding, torch, sys
print(ding.__version__, torch.__version__, sys.version, sys.platform)

My DING's version is f1bf66. My pytorch version is 1.7.1+cu101. My system is Linux 3.7.11 (default, Jul 27 2021, 14:32:16) \n[GCC 7.5.0].
I follow the docs' guidance to enable multi-gpu training. I add a config term config.policy.learn.multi_gpu=True in demo/simple_rl/ppo_train.py. But I get the following exception:
WARNING:root:If you want to use numba to speed up segment tree, please install numba first
pygame 1.9.6
Hello from the pygame community. https://www.pygame.org/contribute.html
[ENV] Setting seed: 0
Traceback (most recent call last):
File "imgppo_train.py", line 189, in
main(main_config)
File "imgppo_train.py", line 158, in main
policy = PPOPolicy(cfg.policy, model=model)
File "/home/qhzhang/code/DI-engine/ding/policy/base_policy.py", line 81, in init
self._init_multi_gpu_setting(model)
File "/home/qhzhang/code/DI-engine/ding/policy/base_policy.py", line 101, in _init_multi_gpu_setting
broadcast(param.data, 0)
File "/home/qhzhang/anaconda3/envs/didrive/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py", line 859, in broadcast
_check_default_pg()
File "/home/qhzhang/anaconda3/envs/didrive/lib/python3.7/site-packages/torch/distributed/distributed_c10d.py", line 211, in _check_default_pg
"Default process group is not initialized"
AssertionError: Default process group is not initialized

The file demo/simple_rl/ppo_train.py is in DI-drive repo

You can refer to this demo and add DistContext in your program

Thanks for your suggestion. But I add DistContext and get the following error:
WARNING:root:If you want to use numba to speed up segment tree, please install numba first
pygame 1.9.6
Hello from the pygame community. https://www.pygame.org/contribute.html
Traceback (most recent call last):
File "imgppo_train.py", line 189, in
with DistContext():
File "/home/qhzhang/code/DI-engine/ding/utils/pytorch_ddp_dist_helper.py", line 114, in enter
dist_init()
File "/home/qhzhang/code/DI-engine/ding/utils/pytorch_ddp_dist_helper.py", line 81, in dist_init
raise RuntimeError("please indicate rank explicitly in dist_init method")
RuntimeError: please indicate rank explicitly in dist_init method

Are you run multi-gpu program in your own machine? And I guess you should prepare related environment variables here.
We test this parts in slurm before so it would be a bit inconvenient in local machine. And you can add your own advice for multi-gpu training in local machine.

I am running multi-gpu program in my own machine so your code based on slurm environment does not work.
I have gone through the relevant code about multi-gpu part. I think using torch.distributed.launch may help user to use your multi-gpu code in local machine.
But another problem for me is that I am actually using DI-drive and one di-drive program will launch several (like 8) carla environments. When using torch.distributed.launch, the carla will be launched multiple times (like 8*N, the N is gpu num).
So I find a quick fix using torch.DataParallel to do multi-gpu training without your API.

OK, we will support torch.DataParallel in next version, for some cases like you encountered.

Hi, @PaParaZz1
I have the same error, can you provide a demo of how to use multi-GPU in the custom environment?
Thanks


Error running continuous integration (jenkins)

I am going through the Docker book (Version: v1.12.3 (32f7583)) on OSx version 10.10.5
Going though the continuous-integration examples I am running into a few errors.

Running the build job, I always get a failure.
In order to debug why, here are the steps I followed:

1. docker run -it -v /srv/docker/jenkins/jenkins:/var/jenkins_home -v /var/run/docker.sock:/var/run/docker.sock jamtur01/jenkins /bin/bash

jenkins@081198b5b3d3:/$ls -la /var/jenkins_home/jobs/Docker_Test_Job/workspace/..
total 16
drwxr-xr-x  8 jenkins jenkins  272 Nov 22 21:24 .
drwxr-xr-x  3 jenkins jenkins  102 Nov 22 21:19 ..
drwxr-xr-x  9 jenkins jenkins  306 Nov 22 21:26 builds
-rw-r--r--  1 jenkins jenkins 2227 Nov 22 21:23 config.xml
lrwxrwxrwx  1 jenkins jenkins   22 Nov 22 21:24 lastStable -> builds/lastStableBuild
lrwxrwxrwx  1 jenkins jenkins   26 Nov 22 21:24 lastSuccessful -> builds/lastSuccessfulBuild
-rw-r--r--  1 jenkins jenkins    2 Nov 22 21:24 nextBuildNumber
drwxr-xr-x 14 jenkins jenkins  476 Nov 22 21:24 workspace

-- Checking the workspace folder --

jenkins@081198b5b3d3:/$ ls -la /var/jenkins_home/jobs/Docker_Test_Job/workspace/
total 36
drwxr-xr-x 14 jenkins jenkins   476 Nov 22 21:24 .
drwxr-xr-x  8 jenkins jenkins   272 Nov 22 21:24 ..
drwxr-xr-x 13 jenkins jenkins   442 Nov 22 21:24 .git
-rw-r--r--  1 jenkins jenkins   159 Nov 22 21:24 .gitignore
-rw-r--r--  1 jenkins jenkins   202 Nov 22 21:24 Dockerfile
-rw-r--r--  1 jenkins jenkins    61 Nov 22 21:24 Gemfile
-rw-r--r--  1 jenkins jenkins   418 Nov 22 21:24 Gemfile.lock
-rw-r--r--  1 jenkins jenkins 10273 Nov 22 21:24 LICENSE
-rw-r--r--  1 jenkins jenkins   142 Nov 22 21:24 README.md
-rw-r--r--  1 jenkins jenkins   265 Nov 22 21:24 Rakefile
drwxr-xr-x  3 jenkins jenkins   102 Nov 22 21:24 centos
drwxr-xr-x  3 jenkins jenkins   102 Nov 22 21:24 debian
drwxr-xr-x  3 jenkins jenkins   102 Nov 22 21:24 spec
drwxr-xr-x  3 jenkins jenkins   102 Nov 22 21:24 ubuntu

sudo docker run -it -v /var/jenkins_home/jobs/Docker_Test_Job/workspace/..:/opt/project/  /bin/bash

root@2fc88b826ef7: cd /opt/project/workspace
root@2fc88b826ef7:/opt/project/workspace# ls -la
total 8
drwxr-xr-x 2 root root 4096 Nov 18 21:01 .
drwxr-xr-x 3 1000 1000 4096 Nov 18 21:01 ..

It looks like after running step# 3,
-v /var/jenkins_home/jobs/Docker_Test_Job/workspace/..:/opt/project/
this doesn't work as expected.
It doesn't look like the directory /var/jenkins_home/jobs/Docker_Test_Job/workspace/ is mounted to
/opt/project/
Do you have any insight on this?

I'm having the same problem using v17.03.0 of the book on macOS 10.12.5. I followed these instructions:
> sudo mkdir -p /var/jenkins_home
> cd /var/jenkins_home
> sudo chown -R 1000 /var/jenkins_home

I successfully completed the chapter using a VirtualBox CentOS as host (i.e., CentOS VM running docker that creates a Jenkins container that itself uses docker to create a test run) and file sharing from macOS with the CentOS host (and sharing internally with Jenkins).
That may work for others who want to work through the chapters while waiting for this issue to resolve.

Same problem on Windows 10, book version v17.12.0-ce.
opt/project/workspace directory is created during mounting but is empty, build fails on rake spec

Similar problem as declanjryan while making the home directories and changing ownership.
Using book version:
Version: v18.03.1-ce (d498111)
The only way I could get the jenkins web interface to work on MacOSX, was to run docker like this:
sudo docker run -u root -d -p 8080:8080 -p 50000:50000
-v jenkins-data:/var/jenkins_home
-v /var/run/docker.sock:/var/run/docker.sock
--name jenkins
jamtur01/jenkins
Interesting that the first -v flag in listing 5.70 in the book calls for:
-v /var/jenkins_home:/var/jenkins_home
but I could not get this to work for the life of me, changing permissions and ownership and even trying /private/var/jenkins_home as James recommended for the Mac OS X users in Listing 5.68 on page 184, to no avail. Being new to docker, I'm not sure of the significance of the jenkin-data:/var/jenkins_home mapping but that seemed to at least get the container up and running so I could log into the web interface.
However, another thing I suggest is to select the "Install Suggested plugins" options on the Customize Jenkins, during the initial setup, otherwise, when James has you create your first build job on page 190, selecting the Git repository option under "Source Code Management" is not there as an option and to a rookie such as myself, you are left baffled, wondering how to enable this option.
Having said all that, even after I was able to specify the  git repository James setup for us, I was getting errors on the first build:
docker: Error response from daemon: OCI runtime create failed: container_linux.go:348: starting container process caused "exec: \"/bin/\": permission denied": unknown.
+ CONTAINER=8e6ff1aa4c0312c34ed9d32f4a8a87e7ca8e712e9bcf2f1aa3f61e5ea3d14f60
Build step 'Execute shell' marked build as failure
Recording test results
ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error?
Finished: FAILURE

Not sure if this is a result of the odd setting of the fist -v flag above or if this is something within the container itself or code that was provided in James repo he provided.


Refactor rank jobs

Original PR: https://github.com/twosigma/Cook/pull/344

Can we get this merged?


Help my if no problem

I have a voice device using the voice phone for Siri with Braille language input keys for the visually impaired, and it is done through scheduled tasks so that the user is on the move to distinguish an imaginary one for ease of use. These commands are connected to the network before logging into the iPhone account.

Thank you, but do I do the link research with MQtt tools or the command application?

openssl pkcs12 -export -clcerts -in client.crt -inkey client.key -out client.p12

eca1751569957f0aad1446b6547eb38ccc07d68d


Detect cycle in an undirected graph using BFS

💥 Proposal
Add a cpp code to detect cycle in an undirected graph using BFS. The current detect cycle code uses dfs and I want to add extra approach using BFS
Have you read the Contributing Guidelines on Pull Requests?
YES

/assign


ISO8061Date format drops milliseconds

Currently, if you call
azure.ISO8061Date.format(new Date());
We will return something like this:
2011-12-17T21:31:46.0000773Z
If you then try to convert this back into a date, the milliseconds will be set to zero. The format that works is the following:
2011-12-17T21:31:46.773Z

Should be fixed.


Documentation

When this plugin appeared I tried to instal and use it  following the rough procedure. I failed, im not a fb user so was a little lost.
Returning to it to have another crack, the documentation has vanished .. any chance of a detailed walkthrough with screenshots of fb ?

Hi, thanks for submitting an issue.
Unfortunately, creating and maintaining documentation in relation to how Facebook operate their developer access - is outside the scope of this plugin.
I am however, happy to assist you (and others) through this thread, as I too faced some frustration when obtaining an access token from Facebook.

#1 remains open for updating the plugins own documentation by the way too.

A good place to start would be these links:

https://yasithlokuge.medium.com/how-to-generate-a-never-expiring-facebook-page-access-token-24ac5c1a95f1
https://developers.facebook.com/docs/graph-api/reference/v12.0/group

Note, the first link refers to a Page Access Token, which is the same process when requesting a Group Access Token. The only difference being the permissions, which the second link provides for a Group.

Another important gotcha, is Facebook requires app verification in order to post to a Group (even to one you own).
To get around this, I created a Test App and used that instead.
For your Group members to see your Jellyfin posts, they must also be admins of the Group page.

#6
A much easier way could be for a Login / Continue with Facebook button to be added.
This would remove the need for manual retrieval and input of an access token.
I've created an issue to investigate this.

Closing.
Feel free to open another issue 😃


Uncaught TypeError: (deepmerge__WEBPACK_IMPORTED_MODULE_0___namespace_cache || deepmerge__WEBPACK_IMPORTED_MODULE_0___namespace_cache) is not a function

When attempting to set up the library I am getting this error:
Uncaught TypeError: (deepmerge__WEBPACK_IMPORTED_MODULE_0___namespace_cache || deepmerge__WEBPACK_IMPORTED_MODULE_0___namespace_cache) is not a function

it's connected with webback module import / export. Fix is to change in node_modules/ngrx-store-idb/fesm2020/ngrx-store-idb
from
import * as deepmerge from 'deepmerge';
to
import deepmerge from 'deepmerge';
hope author will fix it also

@radacovsky any update on this? :)

Hi,
As I said it is working with that patch - so it will be great to update library


Intro Summary and Structure

add the intro, the summary, and plan structure of the tutorial

DONE!


Update URL in README, update colour on line 117 on index.html

117 changed to orange.
Update 118 to blue


www/nginx: lodash update

dependency update

@fabianfrz the diff for lodash in frr doesn't appear to be different (looks like there only an extra end of line in it)

@AdSchellevis I copy and pasted the lib from https://lodash.com/ and maybe it was already 4.17.11.

@fabianfrz probably, better leave frr unchanged then.

@fabianfrz thanks!

@AdSchellevis that went fast…

GitHub is still complaining: Upgrade lodash to version 4.17.13 or later.

we can switch to https://www.npmjs.com/package/lodash .14 which has been released 5 days ago.

yes please :)


How can I select also Weekends day (Sat & Sun)?

I saw on the demo that weekends always are disabled. Is there anyway I can select these days?
And the demo on plunker doesn't work. Please check it again.
Thank you!

Thanks for trying this module.
It works for me. I can select weekends too.

Is your environment different? e.g., Windows, IE11

thanks for your reply.
I tried again on Chrome and it worked ok but when I tried it on FireFox, it looks like this:

windows env?

I have pushed 0.5.6 with FF update, please test.
I have also tested with weekends, and it worked fine for me.

thank you for your update. I would like to ask which version will work with Angular2 RC4? (I still use RC4 for my current project.)

This tag should work with RC4
https://github.com/ng2-ui/ng2-datetime-picker/tree/0.4.0

Updated to RC5 and it's working properly now, thank you!!!


Issue 150

Pull Request Title
Dark/Light Mode Implementation
Description
Implements Dark mode for the website with the use of Shadcn UI. Also makes Appbar responsive when viewing in smaller sized screens.
Resolves #150
Linked Issues

Fixes #[Issue Number]
Closes #159

Type of Change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Changes

Added custom text and background utility classes which can be used in the future for designing the ui without writing css colour properties.
Implemented a dark/light mode toggle using Shadcn ui which lets you switch between 3 options Dark,Light and System default.
Changed the Appbar ti make it responsive when using the website in a smaller screen

Screenshots/Recordings

https://github.com/aadeshkulkarni/figuringout/assets/109036913/18242c20-fba4-4448-804e-166d34e001fe
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there are no similar/duplicate pull requests regarding the same issue
[x] My changes follow the project's coding guidelines and best practices
[x] Code is formatted properly and lint check passes successfully
[ ] I have made corresponding changes to the documentation (if applicable)

Additional Notes

@vverma022 - npm build fails

@vverma022 - npm build fails

I have some dependencies not installed ?

@vverma022 - npm build fails

I have some dependencies not installed ?

You need to run the npm run build command on the frontend project to check & fix errors.

The border color for dark mode is very light. It's creating a bad UX. Can you please keep the border to border-slate-800 from border-gray-200 ?
Dark mode: The default text color can be a darker shade across the app. text-gray-100/200 / text-slate-100/200

The border color for dark mode is very light. It's creating a bad UX. Can you please keep the border to border-slate-800 from border-gray-200 ?

Dark mode: The default text color can be a darker shade across the app. text-gray-100/200 / text-slate-100/200

Alright will implement these changes


Override arguments for specific packages

I would like to have overridden arguments for specific packages using the --override flag. How can I do this for WAU?

Not supported yet as it depends on packages and we run WAU for all installed ones (expect black listed apps).

bump

bump

Bump

Bump


Feature/lexia core5

Description & motivation
This bundle maps Lexia Core5 assessment results. Lexia Core5 is a slightly awkward fit for the Ed-Fi assessment model because the results are a snapshot of the students' usage and performance in the Lexia platform year to date. This bundle was written with studentAssessment records at the student-year level, resulting in the records being overwritten with the most updated data each day rather than creating a historical record. We decided on this approach to avoid an enormous amount of very slowly changing data being pushed to the ODS with daily runs, but I could imagine a partner wanting to preserve snapshots every week or month. We could use project composition to customize the bundle in that case.
PR Merge Priority:

Low - not aware of any partners that are using this besides Jeffco but I think its worth having available

Tests and QC done:

Successfully sent records to Jeffco's dev ODS and their team approved the structure (before deciding to take a side load approach due to the desire for a full historical record of student performance).

Assessment Identifier Decision
The assessment identifier is just 'Lexia Core5' because it is a single subject literacy test. There may be other Lexia tests that are mapped to their own assessment identifiers in the future.

@sleblanc23 trying to get as many open PRs merged as possible so have a few asks on this one:

Can you remove Lexia from the assessment identifier? Appears to be the vendor and for consistency I'm working on only capturing the vendor via the namespace
Can you move the grade level descriptors from the assessment template to a seed and build out the json object in the earthmover yaml? Like we did here

@jalvord1 edits are done & confirmed it still runs!