Problems encountered in visualization

After running the model, I want to use the “demo/demo.py” to randomly input a picture for detection.
python demo/demo.py --config-file configs/PASCAL_VOC/base-training/R101_FPN_base_training_split1.yml \--input input1.jpg
Here I can normally detect the results and output a picture with the detection results.But，in the fine tune model，after I do this
python demo/demo.py --config-file configs/PASCAL_VOC/split1/10shot_CL_IoU.yml \--input input1.jpg
The content of the error report is
AssertionError: get_event_storage() has to be called inside a 'with EventStorage(...)' context!
I appreciate anyone who can help me.

Or, who has tested the test results with a random picture after running all the models?Thanks！

Put storage = get_ event_storage () in roi_ heads.py Change to:
with EventStorage():
storage = get_event_storage()

Put storage = get_ event_storage () in roi_ heads.py Change to:
with EventStorage():
storage = get_event_storage()

Thank you very much！

Put storage = get_ event_storage () in roi_ heads.py Change to:
with EventStorage():
storage = get_event_storage()

Thank you very much！

I have tried this but occur another errpr, how to define EventStorage?


feat: Add Missing Methods for Execution details

Resolves #18

@sccalabr could you please address the failing CI checks?

@sccalabr Please attend to the failing tests.

@sccalabr All the CI tests are failing. Please look into it

The CI failures look to be coming from main branch. I'll open a PR soon to fix them


Abbreviations table in div with id?

In our mock Abbreviations.html file, the div containing the table doesn't have an id.
Does the real one?
Hopefully it does, and we can update (fix) the html, which makes a DITA error go away.

Fixed here: https://github.com/DeepBlueCLtd/LegacyMan/commit/f8f6d7e16bd2b1687d903433970cb0848e85689c


several issues with install guide

forgive the typo in the git issue title - this pertains to the data upgrade guide.
I used this guide to upgrade a Canadian NAV2016 CU3 customer to NAV2018 CU3. Found several issues with the documented data upgrade procedure:
Task 1.4: modify 130400 CAL Test Runner and 130402 CAL Command Line Test Runner from the old dev environment
It states this step is needed for NAV2016 and earlier if these codeunits are present. In our nav 2016 dev environment, there was no "TestPermissions" data type in the C/AL editor, making it impossible to modify these codeunits as this step suggests. At any rate, our earlier merge of the application objects via the Merge-NavApplicationObject cmdlet ensured these codeunits would have the parameters that the data upgrade expects in the upgrade .fob my merge efforts produced.
Task 7: Clear Dynamics NAV Server instance records from old database
Microsoft should add another T-SQL task AFTER that step. If there are records in the "Debugger Breakpoint" table, the data upgrate will fail at task 9 (Import the upgraded application objects and upgrade toolkit objects into the converted database). Those records have to be purged.
This needs to be executed via SQL Management studio BEFORE task 9 is executed:
DELETE from [My NAV Database Name].[dbo].[Debugger Breakpoint]
Task 9: Import the upgraded application objects and upgrade toolkit objects into the converted database.
Documentation is INCOMPLETE for certain regions. In the case of my upgrade (Canadian NAV 2016 to NAV 2018), as per pre-requisite #2, I grabbed Upgrade9001100.FOB (the upgrade toolkit objects for NAV 2016 to NAV 2018). However, after following all the steps (import the nav 2018 app objects fob, THEN the toolkit fob)
however....
Step 13 (synchronize schema with validation) FAILED as it detected destructive changes. Below is the error I received from the admin powershell when I tried to sync the schema with validation:
Sync-NAVTenant : The schema synchronization may result in deleted data. The following destructive changes were detected:
Table: 4, Currency
Field: 10000, Symbol: Deleted
Table: 79, Company Information
Field: 10018, US Country/Region Code: Deleted
Field: 10019, Canada Country/Region Code: Deleted
Field: 10020, Mexico Country/Region Code: Deleted
Table: 81, Gen. Journal Line
Field: 10005, Check Exported: Deleted
Field: 10007, Check Transmitted: Deleted
Table: 311, Sales & Receivables Setup
Field: 10000, G/L Freight Account No.: Deleted
Examples of code to migrate data:
DataUpgradeMgt.SetTableSyncSetup(DATABASE::"Currency",[Upgrade Table Id],TableSynchSetup.Mode::[Mode]);
DataUpgradeMgt.SetTableSyncSetup(DATABASE::"Company Information",[Upgrade Table Id],TableSynchSetup.Mode::[Mode]);
DataUpgradeMgt.SetTableSyncSetup(DATABASE::"Gen. Journal Line",[Upgrade Table Id],TableSynchSetup.Mode::[Mode]);
DataUpgradeMgt.SetTableSyncSetup(DATABASE::"Sales & Receivables Setup",[Upgrade
This was very puzzling, until I discovered the .fob contents in the DVD > UpgradeToolkit > Local Objects. There is no mention of these localized objects in this guide, nor their importance to the data upgrade workflow. Secondly, I don't think the file should have been named Upgrade9001100.US.fob. This clearly affects both Canada and presumably Mexico as well. Shouldn't these localized files be named "UpgradeXXXXXXX.NA.fob"? For a new-comer who has never upgraded a database before, I found this missing piece of important information extremely frustrating.
At any rate, to resolve this I imported the following fobs (synchronize schema >> later)  in the following order for my 2016 to 2018 upgrade:

import the upgraded/merged .fob of ALL NAV2018 objects
import Upgrade9001100.FOB
import Upgrade9001100.US.FOB

Doing so allowed me to successfully sync with validation @ step 13 and proceed with finishing the data upgrade.
Please correct the documented steps accordingly.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9ec2bde3-7aa2-0d9b-2d60-0fc57aa85645
Version Independent ID: c37bcc83-3482-b533-8e9a-8feb56a6b88e
Content: Upgrading the Database - Dynamics NAV
Content Source: dynamics-nav/Upgrading-the-Data.md
Service: unspecified
GitHub Login: @jswymer
Microsoft Alias: jswymer

Thanks for the valuable feedback! The issues have been addressed, and the content updated accordingly. The updates will be available the next time we refresh the content.
As for the name of the local upgrade toolkit fob file, I spoke with the development team. As you experienced, the US fob file covers all the languages in the NA version. For whatever reason, the name has always included US, not NA. We realize that it is confusing, and will look into possibly changing this in the future.
Thanks again

Thank you for the follow-up - always happy to provide input


URL percent decode now only works for ASCII chars

I'm using an older version of cppnetlib and rely on boost::network::uri::decode().
It seems to work well in my tests.
However browsing master code I see in d5fb464947557b you limited it so decoder throws if the hex value is a non-ascii character.
What was rationale? That commit wasn't linked to any ticket etc. so I can't see why you would do that.
Clearly real browsers encode non-ascii chars in URLs. According to

RFC 3986 does not define according to which character encoding table non-ASCII characters (e.g. the umlauts ä, ö, ü) should be encoded. As URL encoding involves a pair of hexadecimal digits and as a pair of hexadecimal digits is equivalent to 8 bits, it would theoretically be possible to use one of the 8-bit code pages for non-ASCII characters (e.g. ISO-8859-1 for umlauts).

On the other hand, as many languages have their own 8-bit code page, handling all these different 8-bit code pages would be a quite cumbersome thing to do. Some languages do not even fit into an 8-bit code page (e.g. Chinese). Therefore, RFC 3629 proposes to use the UTF-8 character encoding table for non-ASCII characters. The following tool takes this into account and offers to choose between the ASCII character encoding table and the UTF-8 character encoding table. If you opt for the ASCII character encoding table, a warning message will pop up if the URL encoded/decoded text contains non-ASCII characters.

In practice all real browsers DO encode non-ascii characters (in whatever character set - that's a whole other topic) in URLs and our servers need to be able to decode that and not crash/fail on any non-ascii parameter input.
What was the rationale for the change? I'm now concerned that we will not be able to update cppnetlib version without breaking URL decoding...

Why are you looking at the master branch? What version of the library are you using? Can you please try the latest in 0.11-devel branch to see if thi is still the case?

Apologies, to be clearer: we are using 0.11.0 (according to CMakeLists.txt) and this bug is //not// present there. Nor do I see the offending commit in the 0.11-devel branch.

Why are you looking at the master branch?

Since there is no online reference documentation for URI class I ended up searching and finding the source here on github. Search results default to master.
I (perhaps mistakenly) created this issue on the assumptions that:

master branch commits are intended to one day make it into a future release
The commit linked above appears to //intentionally// break this feature - it's not an accidental regression that would be caught later by tests and so appears like it will be present in the next release of the library which would break ours and anyone else's servers (possibly silently until issues show up).

If any of my  assumptions are off then I apologise. Mostly I'm interested to know if there was a good reason for the change and if it was misguided as it appears to be from my experience, to have it rectified before it gets into a released version.

Apologies, wrong button.

Ah, right, thanks @banks -- Maybe @glynos has a good reason for implementing it this way (which may be that "the RFC says so"). Being restrictive in this regard may not be a good thing to do though, regardless of how we're currently implementing it.

As for future releases of the library, yes we should take this into account and make it clear that a 1.0 release will not be a backward-compatible release if we decide to stick with the restrictive implementation. Note @banks that we are not making promises of backwards compatibility between 0.x releases and a 1.0 release that's forthcoming.

Thanks Dean
I'm clear that pre-1.0 releases do not guarantee Backward-compatible upgrade path. My concern here is that new behaviour appears to be plain broken!
As far as I read, RFCs are light on the issue - see quote on original post - but in practice all browsers do send non ascii chars in URLs and refusing to handle them effectively means the uri class is unsuitable for real-world use.
On the plus side it's a trivial fix ;)

Sorry I just got back to this (I'm doing an issue triage and I see this one on the list). To circle back on this, I think it's worth undoing the restriction as you point out @banks -- do you have time to make this change?

I can no longer find commit d5fb464947557b in master branch nor see the offending code in decode.hpp. I've look back in history at the state of the file when I created the ticket and don't see the issue there either.
Did master branch history get rewritten at some point?
As far as I can see from browsing (not testing) the code I referred to in this issue is no longer in master branch nor in 0.12rc tag.
So I guess this can be closed, although I'd be more confident if I understood what happened to the repo's history since I opened this ticket :).
At anyrate this test would have failed given the problem commit so it seems safe to say this is no longer valid.

The master branch was rewritten (the changes were abandoned, but still exist on the "master-deprecated" branch). On this branch, the URI code was moved to a different repo because some people asked for a standalone URI library. Therefore, the commit you are referring to was done in the uri repo, not cpp-netlib:
https://github.com/cpp-netlib/uri/commit/d5fb464947557b3b080b3f6ea94c1bc1207bde49
The test you link to is using a much older version of the URI, the issue you describe remains.
Could you make a pull request on the uri repo?
@deanberris we should merge the URI back into master for cpp-netlib 1.0. The current situation is really confusing.

@glynos Ah I see.
The basic fix is (untested, not even compiled) https://github.com/banks/uri/commit/38b8cca17ec392223074cdef021310423cd6161d
But this is not a PR (yet) because I've not got time right away to:

rewrite the tests - at least one needs changing although I'd love to know they the last 2 "euro" symbol tests are only defined for MSVC or commented out respectively. Why should different platforms have different URL decoding behaviour?
check whether the same assumption that non-ascii chars are illegal is present during encoding.

If you have a dev environment setup already for that library it's likely you'd be able to do both those things in a fraction of the time it will take me to get round to it, but nonetheless I'll try to find time soon if you don't :)

Specifically, my interpretation of how this should behave to be useable in practice is:

for decode, any %XX with valid hex digits should be decoded to the raw byte without error. Figuring out what charset this byte is in is a thorny issue with no good answers but one that all web applications must deal with in their own way (by sniffing, or assuming it's in same charset as their pages, or assuming UTF-8 etc.)
for encode, any valid UTF-8 sequence other than the specific ascii subset allowed by RFC to be unencoded, should be converted into % encoding. For multibyte codepoints, the output should be % encoded UTF-8 bytes, for example, ☃ U+2603 is e2 98 83 in UTF-8 hex bytes so should be encoded in a URL param as %e2%98%83.

I think old versions of the library actually had exactly that behaviour so this is not news, just wanted to state the issue unambiguously.


"cargo package" crash: "slice index starts at 102 but ends at 101"

cargo package fails with thread '<main>' panicked at 'slice index starts at 102 but ends at 101', ../src/libcore/slice.rs:534 on jsgf/rocksdb-sys@bfb1a1d124d965784063dd9cea8f8d0a7bcc14d2.
The backtrace is:
$ RUST_BACKTRACE=1 cargo package
   Packaging rocksdb-sys v0.2.2 (file:///home/jeremy/src/rust/rocksdb-sys)
thread '<main>' panicked at 'slice index starts at 102 but ends at 101', ../src/libcore/slice.rs:534
stack backtrace:
   1:     0x5584de3c6710 - std::sys::backtrace::tracing::imp::write::h63529ecac330b5bc
   2:     0x5584de3cf6eb - std::panicking::default_hook::_$u7b$$u7b$closure$u7d$$u7d$::h0e99487804ecdaa3
   3:     0x5584de3cf36e - std::panicking::default_hook::h1f0a2403a716c4a6
   4:     0x5584de3b3eef - std::sys_common::unwind::begin_unwind_inner::heae36551373e36bd
   5:     0x5584de3b58a8 - std::sys_common::unwind::begin_unwind_fmt::hfdca93605af5caca
   6:     0x5584de3c5941 - rust_begin_unwind
   7:     0x5584de405ddf - core::panicking::panic_fmt::h139963e0c814ece1
   8:     0x5584de405fd3 - core::slice::slice_index_order_fail::hfd97ac60d33c912a
   9:     0x5584de164fd7 - tar::header::UstarHeader::_set_path::h17b0206550960dd5
  10:     0x5584de163d66 - tar::header::Header::_set_path::h31d689d2714e1449
  11:     0x5584ddf5d374 - cargo::ops::cargo_package::package::h08cdd994f43d4c04
  12:     0x5584dde57a5f - cargo::call_main_without_stdin::h31e15b3df12e109b
  13:     0x5584dde1cc56 - cargo::execute::hab577e023b3ac5e2
  14:     0x5584dde17943 - cargo::call_main_without_stdin::hf709651485746f28
  15:     0x5584dde14aa1 - cargo::main::h2b219a79f378c1ef
  16:     0x5584de3cefe4 - std::sys_common::unwind::try::try_fn::h512e768250918f40
  17:     0x5584de3c58cb - __rust_try
  18:     0x5584de3cea2b - std::rt::lang_start::h2dd73040e4b52b40
  19:     0x7f172c54457f - __libc_start_main
  20:     0x5584dde14428 - <unknown>

target/package/.rocksdb-sys-0.2.2.crate exists, but it looks like its missing the following files:
@@ -773,29 +778,3 @@
 rocksdb/utilities/spatialdb/utils.h
 rocksdb/utilities/table_properties_collectors/compact_on_deletion_collector.cc
 rocksdb/utilities/table_properties_collectors/compact_on_deletion_collector.h
-rocksdb/utilities/table_properties_collectors/compact_on_deletion_collector_test.cc
-rocksdb/utilities/transactions/optimistic_transaction_db_impl.cc
-rocksdb/utilities/transactions/optimistic_transaction_db_impl.h
-rocksdb/utilities/transactions/optimistic_transaction_impl.cc
-rocksdb/utilities/transactions/optimistic_transaction_impl.h
-rocksdb/utilities/transactions/optimistic_transaction_test.cc
-rocksdb/utilities/transactions/transaction_base.cc
-rocksdb/utilities/transactions/transaction_base.h
-rocksdb/utilities/transactions/transaction_db_impl.cc
-rocksdb/utilities/transactions/transaction_db_impl.h
-rocksdb/utilities/transactions/transaction_db_mutex_impl.cc
-rocksdb/utilities/transactions/transaction_db_mutex_impl.h
-rocksdb/utilities/transactions/transaction_impl.cc
-rocksdb/utilities/transactions/transaction_impl.h
-rocksdb/utilities/transactions/transaction_lock_mgr.cc
-rocksdb/utilities/transactions/transaction_lock_mgr.h
-rocksdb/utilities/transactions/transaction_test.cc
-rocksdb/utilities/transactions/transaction_util.cc
-rocksdb/utilities/transactions/transaction_util.h
-rocksdb/utilities/ttl/db_ttl_impl.cc
-rocksdb/utilities/ttl/db_ttl_impl.h
-rocksdb/utilities/ttl/ttl_test.cc
-rocksdb/utilities/write_batch_with_index/write_batch_with_index.cc
-rocksdb/utilities/write_batch_with_index/write_batch_with_index_internal.cc
-rocksdb/utilities/write_batch_with_index/write_batch_with_index_internal.h
-rocksdb/utilities/write_batch_with_index/write_batch_with_index_test.cc

Perhaps rocksdb/utilities/table_properties_collectors/compact_on_deletion_collector_test.cc is too long?

BTW, this is cargo installed by rustup on Linux:
$ cargo version
cargo 0.10.0-nightly (10ddd7d 2016-04-08)

This I believed was fixed in https://github.com/alexcrichton/tar-rs/commit/d08897ae6d7a1d7a5bb381e24c5402a8d15eca46 which was published in 0.4.5 of tar-rs and brought to Cargo in 11a0a316 which was published just a few days after the version you're using. I think this is fixed on master in this case so I'm gonna close, but thanks for the report!


write 0 if typename exp : ExpressionStatement

Now tests/behaviour/solidity/test/libsolidity/semanticTests/libraries/library_enum_as_an_expression.sol is passing
Previously, only ElementaryTypeNameExpression was being written to '0' excluding other types (eg. Enum, structs...)

#526


Is AddressBook IoC container similar?

Is AddressBook equivalent to an IoC Container where singleton instances of coordinators and routers are managed?

In current example it's role is similar to IoC - to keep access points (in form of the IActorRefs) to all major services in the program, and return them on demand. But similar properties (registering, unregistering and returning services) are also common to service discovery patterns - I think that treating address book as a local version of service discovery is better mental model. Since it's an actor and it's aware of it's purpose it may provide additional features, for example tracking and unregistering dead actors.


[FEATURE]save the lsp workspace folders environment variable

Is your feature request related to a problem? Please describe.
-- add path environment variable
vim.lsp.buf.add_workspace_folder

recovering after calling the SessionSave command did not recover the lsp workspace
Describe the solution you'd like
Run vim.lsp.buf.add_workspace_folder to see the previously added workspace
Describe alternatives you've considered
by pre_ save and SessionRestore
Additional context
1. nvim demo/
2. vim.lsp.buf.add_workspace_folder()
3. vim.inspect(vim.lsp.buf.list_workspace_folders())
4. exit nvim
5. nvim demo/
6. :SessionRestore
7. vim.inspect(vim.lsp.buf.list_workspace_folders())

Does it get saved if you manually rum mksess to create a session and manually restore it after?
What gets saved in the session is up to upstream Neovim and its available options. auto-session does not have control or implement extras on top of Neovim's supported session saving features.


Feature/add crypto libs

node crypto libraries removed
added bitcoinj to replace js crypto libraries

@jeandudey  You shouldn't use the compile, but with implementation the bitcoinJ lib gives an error when compiling


Only OPS can build in new worlds?

When I make a new world with multiverse, of instance a flat world, only ops can build on it. How do enable other players to build on it as well as ops? Or is this a bug issue? I'm running CraftBukkit 1.4.6 with Multiverse dev build 2.5

Not the case, locked due to inactivity.


AuthUser registration "password mismatch" - can't create new account.

Problem
Can't register a new user using the AuthUser tool.
When attempting to register a new user, using matching passwords results in "password mismatch" message display and unable to register new users.
Replicate

Use bloggy example or simplified AuthUser Model like so:

# AuthModel ---------------------------------------
from weppy.tools.auth.models import AuthUser

class User(AuthUser):
    pass

# Init------------------------------------------------
from weppy import App, DAL
from weppy.tools import Auth

app = App(__name__, template_folder="./views")

# [...]

# init database and auth
from starter_weppy.models.user import User

# init auth before passing db models due to dependencies
# on auth tables in the other models
db = DAL(app)
auth = Auth(
        app, db, usermodel=User
)

Run app and navigate to http://<IP_ADDRESS>:8000/account/register
Attempt to create a new user with valid matching password
Unable to create use and see "password mismatch" error

Version
0.7.9
OS
Windows, OSX, linux
Screenshots
From running bloggy (0.7.9)

This looks to be fixed on current version of master (02148e43bafdd509d26d4c4dd1bad3605871e6c3)
Looking forward to the next release :bowtie: !

@gi0baro - any chance we can get a 0.7.10 with current code? 0.7.9 user creation through AuthUser is definitely broken (at least on python 3.4 and 3.5).
Not sure how tests passed for 0.7.9 release but might be UI specific bugginess. Might be a good idea to roadmap some simple selenium testing to make sure crucial UI functionality like this works.

@mijdavis2 Yeah, I'm sorry about this, I should really implement some integration tests above auth system.
Will release a 0.7.10 later today, thank you for your patience :)
0.8 then should be probably ready the next week.

@mijdavis2 this is fixed in 0.7.10 :)

👍 Thanks!
If I can find the time I'll toy around with adding some lite selenium testing.


Cloning moment object changes date

Using the clone method is causing the date to change in Safari 8.0.3
This is an example of the code I am using.
var firstDay = startDate.clone();
var lastDay = endDate.clone();
console.log(startDate);
console.log(firstDay);
console.log(endDate);
console.log(lastDay);

This is what it returns:
[Log] Moment (daterangepicker.js, line 1095)
_d: Tue Jan 27 2015 00:00:00 GMT+1100 (AEDT)
_i:<PHONE_NUMBER>000
_isAMomentObject: true
_isUTC: false
_locale: Locale
_pf: Object
__proto__: Moment

[Log] Moment (daterangepicker.js, line 1096)
_d: Mon Jan 26 2015 12:00:00 GMT+1100 (AEDT)
_i:<PHONE_NUMBER>000
_isAMomentObject: true
_isUTC: false
_locale: Locale
_pf: Object
__proto__: Moment

[Log] Moment (daterangepicker.js, line 1097)
_d: Thu Apr 02 2015 00:00:00 GMT+1100 (AEDT)
_i:<PHONE_NUMBER>000
_isAMomentObject: true
_isUTC: false
_locale: Locale
_pf: Object
__proto__: Moment

[Log] Moment (daterangepicker.js, line 1098)
_d: Mon Apr 06 2015 01:59:59 GMT+1000 (AEST)
_i:<PHONE_NUMBER>000
_isAMomentObject: true
_isUTC: false
_locale: Locale
_pf: Object
__proto__: Moment

Am I doing something horribly wrong or is something wrong with moment js?

@Rambomst Can you create a jsFiddle that shows this error?

Only the timezone is not being preserved, but the date is the same, see the _i property. Could you post the _locale property here, so we could see what is the locale that moment is using in the clones.

I'm going to close this since we haven't heard back from you. Feel free to reopen with us if you're able to reproduce. We would be happy to look at it.

I have the same issue in Chrome. Here is code snippet
var endTime = moment(startTime).add(obj.duration, 'm');
startTime is changed after this. Thanks

Probably I should use something like moment.tz(startTime) since startTime was created using moment.tz? Thanks

@dmitin-bv the .add() method mutates the moment obj it's called on. You should instead do
moment(startTime).clone().add(obj.duration, 'm');

Similar problem. I am using clone() to clone the object and preserve the value. I am trying to dynamically generate date ranges inside a for loop.
While doing so, I noticed that _d property is correct, however, the _i property remains as old one.
This can be "hacked" by using next trick:
var newDate = moment(AppSettings.Formatting.minDate.clone().add(i, 'M').format(AppSettings.Formatting.dateFormat), AppSettings.Formatting.dateFormat);
Only AFTER explicitly applying format does the new moment.js object acquire new values.
Using only AppSettings.Formatting.minDate.clone() DOES NOT set _i property to correct value.

Am I doing something wrong and what?


Button height

The standard button component is 4px short of meeting the AAA criteria for target size on the height attribute. I know we only work to AA, but I noticed the radio buttons and checkboxes were increased to 44px, and 44px was presumably chosen for that reason?
The back link component also appears to leverage some :after css trickery to increase the target size above the threshold of 44px, presumably to hit the same WCAG criterion?
I'm wondering if the standard buttons have just been missed or if there is a deliberate design decision to keep them at 40px rather than 44px?
Thanks

more discussion here:
https://github.com/alphagov/govuk-frontend/issues/2060

more discussion here:
https://github.com/alphagov/govuk-frontend/issues/2060

Ah thanks @joelanman - I'm always unsure on what is a Design System issue and what is a GOVUK Frontend issue. Should have looked in both! Apologies.

Ah thanks @joelanman - I'm always unsure on what is a Design System issue and what is a GOVUK Frontend issue. Should have looked in both! Apologies.

no problem at all, it's hard to keep track of both

no problem at all, it's hard to keep track of both

@abbott567 are you happy that issue covers this as well? If so, can we close this as a duplicate?

@abbott567 are you happy that issue covers this as well? If so, can we close this as a duplicate?

Yeah, I think it does, as you specifically mentioned the buttons. Will close this in favour of: alphagov/govuk-frontend#2060. Thanks!

Yeah, I think it does, as you specifically mentioned the buttons. Will close this in favour of: alphagov/govuk-frontend#2060. Thanks!


Fix: handle Windows better, fixes #31

This should fix #31 on Windows, I haven't tested it on Linux though

Thanks, let me test this on Linux in the next hour!

Works for me!


Adding optional sidecar containers and volumes

This allows the use of optional sidecar containers to be added to the memcache deployment to handle things like metric collection or ssl termination.

/ok-to-test

/lgtm


:bug: navbar on big touch devices

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #4685
[ ] Requires deployment <>

Before submitting pull request, please make sure:

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

Optional

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

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

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

Before:

After:

hmm, looks like its fixed but when resizing the screen and switching from mobile size to tablet size, it still isnt ideal.
step one:

step two:

swithing to mobile view

somehow reproduced it in chrome. I think its when switching from mobile to desktop mode and resizing it in responsive mode.

somehow reproduced it in chrome. I think its when switching from mobile to desktop mode and resizing it in responsive mode.

That definitely shouldn't happen.
I'm trying to reproduce this right now (along with your comment from yesterday), and I'm failing to do so 🤔
I know why it happened before -> We were differentiating the navbar based on device type, i.e., mobile(touch) and desktop.
But we've switched from that to caring only about the device's viewport, so that should adjust accordingly.
@exezbcz I've pushed some changes to remove unused code, can you try to update the app and try again?

somehow reproduced it in chrome. I think its when switching from mobile to desktop mode and resizing it in responsive mode.

That definitely shouldn't happen.
I'm trying to reproduce this right now (along with your comment from yesterday), and I'm failing to do so 🤔
I know why it happened before -> We were differentiating the navbar based on device type, i.e., mobile(touch) and desktop.
But we've switched from that to caring only about the device's viewport, so that should adjust accordingly.
@exezbcz I've pushed some changes to remove unused code, can you try to update the app and try again?

i reproduced it on the beta, seems like here its fixed.
Tried it on chrome for ipad and safari, the issue is only with chrome when resizing and using split screen.
When adjusting the navbar on mobile, we should be extra cautious because it could break this again.

i reproduced it on the beta, seems like here its fixed.

okay, if there are any issues with this branch (not beta, we are fixing the beta here), let me know 😄

i reproduced it on the beta, seems like here its fixed.

okay, if there are any issues with this branch (not beta, we are fixing the beta here), let me know 😄
// still weird you got that result on this deploy before

yup, i mentioned that because you wrote that you dont know how to reproduce this earlier :D Nothing important

pay 20

😍 Perfect, I’ve sent the payout
💵 $20 @ 31.49 USD/KSM ~ 0.635 $KSM
🧗 FNyt7T1xdbhN7dagf7yGYCRJuE3R45VmTCE3tjzy1rKxa7y
🔗 0xc5585c49ff01b2db799707a7a2813b22f11dbc98423d55dec859cc1d18d9b1ff
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Release 0.4.0

[x] tag and build 0.4.0

https://github.com/sigstore/sigstore-java/releases/tag/v0.4.0

[x] release to maven central

https://repo1.maven.org/maven2/dev/sigstore/sigstore-java/0.4.0/

[ ] update project version and README

#393

[ ] release gradle plugin to gradle plugin portal (@vlsi)

FYI @patflynn @vlsi, tagging off of main@ 839d11381abcf885140eaa4721cbdd1ac189d296

AFAIK this is all done since the plugin is available


Extend functionality of the java-example

Closes https://github.com/FerretDB/dance/issues/530.
We need to enable assertions with maven by setting the environment variable MAVEN_OPTS="-ea".

Just noting that this jave example uses find while python example uses findone. So we have inconsistency between examples


Bot is providing some custom link in response

After creating rasa core model, When I am running python -m rasa_core.run -d models/dialogue command it is running on a server 5005 by default and said Bot loaded. Type a message and press enter (use ‘/stop’ to exit): but when I type some message like Hi it shows some link like: <IP_ADDRESS> - - [2018-09-12 10:05:43] “POST /webhooks/rest/webhook?stream=true&token= HTTP/1.1” 200 122 0.102086 Can anyone help me with this?
Now when I downgrade my rasa_core form 0.11.3 to 0.10.4 and build model again it works perfectly fine.
Is there any change in the new version which is not mentioned in the document or is it some error from my side? kindly let me know.

This isn't a response, it's a logger message, so this is working as expected


S3 Range Header

Here is the code I am currently trying to use when calling getting large items from S3. This file is 6.1BG and has never been able to be pulled down completely. I implemented a function that takes advantage of the Range header on S3 but I am one using it wrong (very possible) or it's broken.
// Download file from s3 to your local computer
// this will currently put the file in the
// directory you run it from. Need to add in
// some support for failover in case where TCP
// Dial timesout. Seems to happen somewhat often.
func downloadFileFromS3(file string) {
	svc := s3.New(&aws.Config{Region: "us-east-1"})

	params := &s3.GetObjectInput{
		Bucket: aws.String(s3Bucket),
		Key:    aws.String(file),
	}

	resp, err := svc.GetObject(params)

	if awserr := aws.Error(err); awserr != nil {
		// A service error occurred.
		fmt.Println("Error:", awserr.Code, awserr.Message)
	} else if err != nil {
		// A non-service error occurred.
		panic(err)
	}

	f, err := os.Create(file)
	if err != nil {
		fmt.Println(err)
	}
	defer f.Close()

	fmt.Println("starting download of " + file)
	w := bufio.NewWriter(f)
	_, err = w.ReadFrom(resp.Body)
	if err != nil {
		w.Flush()
		f.Close()
		if reflect.TypeOf(*resp).String() == "s3.GetObjectOutput" {
			retryDownload(file, resp)
		}
		fmt.Println(err)
	} else {
		// Pretty-print the response data.
		fmt.Println(awsutil.StringValue(resp))
	}

	w.Flush()
}

// On large files s3 is unreliable to complete the entire
// download and only has a 25% success rate. This function
// will try to complete the download by starting at the
// range that was successfully downloaded and appending
// to the file from that point.
func retryDownload(file string, obj *s3.GetObjectOutput) {
	fmt.Println("attempting to fix downloaded file...")

	svc := s3.New(&aws.Config{Region: "us-east-1"})

	params := &s3.GetObjectInput{
		Bucket: aws.String(s3Bucket),
		Key:    aws.String(file),
		Range:  aws.String("bytes=" + strconv.FormatInt(*obj.ContentLength, 10) + "-"),
	}

	fmt.Println("making request")
	resp, err := svc.GetObject(params)

	if awserr := aws.Error(err); awserr != nil {
		// A service error occurred.
		fmt.Println("Error:", awserr.Code, awserr.Message)
	} else if err != nil {
		// A non-service error occurred.
		panic(err)
	}

	fmt.Println("Opening file to append")
	f, err := os.OpenFile(file, os.O_RDWR|os.O_APPEND, 0666)
	if err != nil {
		fmt.Println(err)
	}
	defer f.Close()

	fmt.Println("trying to append")
	w := bufio.NewWriter(f)
	_, err = w.ReadFrom(resp.Body)
	if err != nil {
		w.Flush()
		f.Close()
		if reflect.TypeOf(*resp).String() == "s3.GetObjectOutput" {
			retryDownload(file, resp)
		}
		fmt.Println(err)
	} else {
		fmt.Println("in here...")
		// Pretty-print the response data.
		fmt.Println(awsutil.StringValue(resp))
	}
}

The errors that I get are as follows.
read tcp <IP_ADDRESS>:443: operation timed out
{
  AcceptRanges: "bytes",
  Body: &{0xc2082cf380 {0 0} false 0xc2082ce140 <nil> 0x161290},
  ContentLength:<PHONE_NUMBER>,
  ContentType: "application/x-gzip",
  ETag: "\"f0cf293fe2291cc0e4d8b72901be18fc-424\"",
  LastModified: 2015-03-15 20:12:50 +0000 UTC,
  Metadata: {
    S3cmd-Attrs: "uid:1000/gname:ubuntu/uname:ubuntu/gid:1000/mode:33204/mtime:1426407620/atime:1426407658/md5:cde9c42fbc30b5d68868de45455182db/ctime:1426449167"
  }
}

When i go to try and fix the issue using the second function I get.
attempting to fix downloaded file...
making request
Error: InvalidRange The requested range is not satisfiable
Opening file to append
trying to append
attempting to fix downloaded file...
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x55e0]

goroutine 1 [running]:
main.retryDownload(0xc2081e6620, 0x1d, 0xc2080356c0)
	/Users/mschuett/Go/src/github.com/michaeljs1990/scripts/cmd/rds-util/main.go:327 +0x230
main.retryDownload(0xc2081e6620, 0x1d, 0xc208034a90)
	/Users/mschuett/Go/src/github.com/michaeljs1990/scripts/cmd/rds-util/main.go:355 +0xba3
main.downloadFileFromS3(0xc2081e6620, 0x1d)
	/Users/mschuett/Go/src/github.com/michaeljs1990/scripts/cmd/rds-util/main.go:303 +0x7e8
main.fetchBackupsFromS3(0x0, 0x0, 0x0)
	/Users/mschuett/Go/src/github.com/michaeljs1990/scripts/cmd/rds-util/main.go:221 +0xed
main.runMigration()
	/Users/mschuett/Go/src/github.com/michaeljs1990/scripts/cmd/rds-util/main.go:149 +0x5b
main.main()
	/Users/mschuett/Go/src/github.com/michaeljs1990/scripts/cmd/rds-util/main.go:98 +0x517

goroutine 17 [syscall, 12 minutes, locked to thread]:
runtime.goexit()
	/usr/local/Cellar/go/1.4.2/libexec/src/runtime/asm_amd64.s:2232 +0x1

goroutine 15 [select]:
net/http.(*persistConn).writeLoop(0xc20825c210)
	/usr/local/Cellar/go/1.4.2/libexec/src/net/http/transport.go:945 +0x41d
created by net/http.(*Transport).dialConn
	/usr/local/Cellar/go/1.4.2/libexec/src/net/http/transport.go:661 +0xcbc

goroutine 14 [runnable]:
net/http.(*persistConn).readLoop(0xc20825c210)
	/usr/local/Cellar/go/1.4.2/libexec/src/net/http/transport.go:928 +0x9ce
created by net/http.(*Transport).dialConn
	/usr/local/Cellar/go/1.4.2/libexec/src/net/http/transport.go:660 +0xc9f
exit status 2

Sorry for bugging you so much :frowning:

It looks like what i would want is multipart download but I don't see it any place in the current API. I am guessing you could implement it fairly easily using goroutines and the Range header? I would be willing to do this although I am not sure if multipart download has an official spec it needs to meet?

I implemented multipart download locally if that is something that is on the roadmap for this sdk.

Hi @michaeljs1990 initially i think the retryDownload is failing because
Range:  aws.String("bytes=" + strconv.FormatInt(*obj.ContentLength, 10) + "-"),

produces a range request header value like:
bytes=<PHONE_NUMBER>-

This in incorrect when retryDownload is called for the first time due to downloadFileFromS3 encountering an error.  The value that should be used in the number of bytes written to your file.  So that the retryDownload starts off where the connection error occurred.
Also I'd suggest updating retryDownload to take the f file reader, and length of previous copy instead of s3.GetObjectOutput pointer.  This will ensure that the correct bytes will be requested.  This also prevents the need for the bufio and flushes if io.Copy is used instead.

Awesome thanks for the update, glad you got it working!
A managed downloader is something that would be very helpful to have and would go nicely with the managed uploader that was recently just committed.

Awesome, I'll clean it up over the weekend and submit a PR.


Handling of kernelspecs

I'm moving the discussion about kernelspecs from #271 to this issue.
@slavaGanzin @rgbkrk Please, could you clarify your position? Are you supporting making the installation of Jupyter a requirement to use Hydrogen?
The latest Ubuntu 16.04 LTS comes with IPython v2.4.1. This version of IPython doesn't support the command ipython kernelspec. This is very unfortunate and it's raising the hurdle users have to hop in order to use Hydrogen.
@rgbkrk Would any of the nteract libraries be ready to help us here? If not, I would consider contributing to them so that we can solve this unfortunate situation sooner than later.

I don't care about Ubuntu problems, but as I said before:
if jupyter/ipython kernelspec fails hydrogen should use kernelspec mappings that can be set in hydrogen settings. That means jupyter wouldn't be a requirement for hydrogen.
if we can get kernelspecs from ipython/jupyter, than that mappings should be merged internally (without showing them in settings) with kernelspecs from settings. And so if kernel was uninstalled from system it wouldn't be listed in kernel list (as now).
In this way you can manage only remote kernels (or specific ones) in settings.

kernelspecs is the library for querying kernel specs, though it only relies on jupyter-paths at the moment. We could introduce a way to check for ipython-paths as well as old IPython. Based on the fact that we "require" a kernel to be installed, even with jupyter, we likely need a hokey default kernel to be provided back from kernelspecs.

I don't care about Ubuntu problems
@n-riesco Life is ironic, even sarcastic - now I work on ubuntu 16.04 LTS (laughing allowed)

I read docs for ijavascript, jp-babel and installed env as described:
sudo apt-get install nodejs-legacy npm ipython ipython-notebook libzmq3-dev
and was discouraged with:
updateKernelSpecs: `jupyter kernelspec` failed Error: Command failed: /bin/sh -c jupyter kernelspec list --json --log-level=CRITICAL(…)
kernel-manager.coffee:71 updateKernelSpecs: `ipython kernelspec` failed Error: Command failed: /bin/sh -c ipython kernelspec list --json --log-level=CRITICAL(…)

after that I try to intall jupyter from pip and everything works like a magic.
So here is a question why do you recomend to use constantly outdated ubuntu repositories if anyone can get fresh and shiny from pip?
p.s. I hate ubuntu and idea about using LTS on desktop, so I'll switch.  So I have specified kernels in my settings. Update kernels on work machine doesn't update my kernelspecs because I have ones from my home. That's the weirdest behaviour I ever expect.
I'll implement that this way: https://github.com/nteract/hydrogen/issues/271#issuecomment-220771030
Current behaviour is a mess

To get the discussion in this issue re-started. Here are the main questions I thought we need to answer:

How to locate kernel specs

Currently: use jupyter kernelspec
Proposed: use a javascript-only library

How to set the setting kernelSpec

Currently: merge user setting with output of jupyter kernelspec
Proposed: kernelSpec only stores user setting

How to set the setting languageMappings

Currently: set by user to help with kernel languages that don't match grammar languages
Proposed: remove languageMappings and use only kernelMappings

How to set the setting kernelMappings

Currently: command hydrogen:select-kernel only shows kernels that match grammar language
Proposed: command hydrogen:select-kernel shows all kernels regardless of whether they match the grammar language

How to start a kernel for a given file:

Currently:

use the connection file ./hydrogen/connection.json
use the setting kernelMappings
match grammar and kernel languages

Proposed:

use the kernel spec ./hydrogen/kernel.json

Just a question at the beginning:
Do we need the kernelSpec setting if we make jupyter a dependency? This would simplify the code a lot and would make bullet point 1 very easy to achieve plus it is easy to install with pip/conda install jupyter and ipython kernel install on every system and easier than editing json. For my understanding this setting is here for useres who have an old ipython version, or am I missing something?

I would support the move to kernelspecs.
If we need the kernelSpec setting, I would prefer the proposed option.
I'm a bit confused with these settings and I think I'm not the only one. I would appreciate a merge of them if possible, so we can directly map kernel language names to kernel display names (eg. {'somestrangekernelname': 'Python 3'}). @slavaGanzin Somewhere you explained how you use them, but I couldn't find the issue, would your workflow work with this option?
I prefer the current option, because it's easier for new users. But I understand why the proposed option can be useful. I think the best way is a new setting to give the advanced user a choice.

On 05/07/16 22:49, Lukas Geiger wrote:

Just a question at the beginning:
Do we need the kernelSpec setting if we make |jupyter| a dependency?

I'm against making Jupyter a dependency. I'd like Hydrogen to run out of the box (i.e. no need to install ZMQ or Jupyter; we're far from achieving this).
Otherwise, we won't stop the constant flow of issues reporting problems with the installation of ZMQ and Jupyter.

This would simplify the code a lot and would make bullet point 1 very easy to achieve plus it is easy to install with |pip/conda install jupyter| and |ipython kernel install| on every system and easier than editing json. For my understanding this setting is here for useres who have an old |ipython| version, or am I missing something?

Ditto: we would have to deal with issues reported by users expecting Hydrogen to work with the IPython version in Ubuntu 16.04.

I would support the move to |kernelspecs| https://github.com/nteract/kernelspecs.

As explained above, I still believe in the merits of having a javascript-only solution.

If we need the kernelSpec setting, I would prefer the proposed option.

Me too.

I'm a bit confused with these settings and I think I'm not the only one. I would appreciate a merge of them if possible, so we can directly map kernel language names to kernel display names (eg. |{'somestrangekernelname': 'Python 3'}|). @slavaGanzin https://github.com/slavaGanzin Somewhere you explained how you use them, but I couldn't find the issue, would your workflow work with this option?

It was confusing because the code was using language names from too many sources (that sometimes were lowercased, but not always). This has been the source of many issue reports.
With PR #260, we reduced the sources of language names to 2:

lowercased Atom grammar language names (used to identify what kernels can run a given grammar language)
kernel display names (used to identify a kernel)

If we removed languageMappings, hydrogen:select-kernel won't list kernels that declare non-standard language names.
To workaround the removal of languageMappings, we change hydrogen:select-kernel to list all kernels, but this is very inconvenient for users like me with a large number of kernels.

I prefer the current option, because it's easier for new users. But I understand why the proposed option can be useful. I think the best way is a new setting to give the advanced user a choice.

I'm not against, the removal of languageMappings, but I'd like to make sure we provide the user the best experience we can:

e.g. having a setting to configure whether hydrogen:select-kernel shows all the kernels
e.g. having two commands hydrogen:select-kernel and hydrogen:select-non-standard-kernel
...

"javascript-only solution" is just hardcoded paths and it often fail.
Parsing jupyter kernelspecs should be moved to https://github.com/nteract/kernelspecs by design. And used by hydrogen as dependency.

Jupyter/ipython kernelspecs should be optional, but preferred way. Jupyter/ipython shouldn't be a dependency. But it should be recommended.

I think two options for selecting kernel is good choice.

languageMappings should be removed as redundant.

Closed via #351.


added insertion sort

Fixes #23
Short description of what this resolves:
Added a insertion sort page.

Features added in this pull request:

Introduction to Insertion Sort
Complete explanation on Insertion sort
Complexity Analysis of Insertion Sort
Code for Insertion Sort

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added code examples, test must pass
[x] Added documentation so that the program is self-explanatory and educational.
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

If you find anything not appropriate, please comment here, and I will look into it and try to get it fixed.

OK, I have committed all of your changes, sorry for the typos and markdown error. There were no guidelines for that, so I looked at other files and took a guess whether they were H₂ or H₃.
I have also removed the package-lock.json, NPM added or deleted some files due to some updates.
Thank you for all the help.

OK, I have committed all of your changes, sorry for the typos and markdown error. There were no guidelines for that, so I looked at other files and took a guess whether they were H₂ or H₃. I have also removed the package-lock.json, NPM added or deleted some files due to some updates. Thank you for all the help.

Ok No Problem man, if you want please update the contribution guideline raise an issue, and then make a PR updating the Contribution guidelines

Since the package-lock.json was already deleted, I am not able to make any edits or add new commits there. I was thinking merge this, then I will make a different pull request to add the package-lock.json which is currently present.
Unless u have some other better idea.

OK, I have committed all of your changes, sorry for the typos and markdown error. There were no guidelines for that, so I looked at other files and took a guess whether they were H₂ or H₃. I have also removed the package-lock.json, NPM added or deleted some files due to some updates. Thank you for all the help.

Ok No Problem man, if you want please update the contribution guideline raise an issue, and then make a PR updating the Contribution guidelines

Sure i will look into it and see if i can improve something that will help other contibutors!

ok done I have removed that package-lock.json file, its no longer going to affect any other previous files. This PR seems done now. Thank you for ur help. I guess i will move on to other issues XD.

Thank you for your Patience!


Add immut/hashset

Closes #498
This PR adds a immutable hash set that based on a HAMT. Basically just a copy from the immutable hash map.

Pull Request Test Coverage Report for Build 1613
Details

84 of 102 (82.35%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 88.4%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

immut/hashset/bucket.mbt
21
25
84.0%

immut/hashset/HAMT.mbt
48
62
77.42%

Totals

Change from base Build 1604:
-0.2%

Covered Lines:
2431

Relevant Lines:
2750

💛  - Coveralls

Pull Request Test Coverage Report for Build 1620
Details

84 of 102 (82.35%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 88.413%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

immut/hashset/bucket.mbt
21
25
84.0%

immut/hashset/HAMT.mbt
48
62
77.42%

Totals

Change from base Build 1619:
-0.2%

Covered Lines:
2434

Relevant Lines:
2753

💛  - Coveralls


Updated position and mouseclick in ConnectionPropertyDrawer

Added function to activate/deactivate ConnectionPropertyDrawer with respect to the dialog being open

Reduced size of ConnectionPropertyDrawer
Tried to data-bind source and sinks (not working)
Tried to change "Left click to start connection" to "Right click to end connection" only when the user places first point (not working)


Backwards compatibility for enums?

Is there anyway to do backwards compatibility for enums? a short example:
pub struct V1 {
    pub label: String,
}

pub struct V2 {
    pub label: String,
    pub password: String,
}

pub struct Latest {
    pub label: String,
    pub password: String,
    pub destination: String,
}

pub enum Door {
    V1(V1),
    V2(V2),
    Latest(Latest),
}

pub fn v1_to_v2(door: V1) -> V2 {
    V2 {
        label: door.label,
        password: "".to_string(),
    }
}

pub fn v2_to_latest(door: V2) -> Latest {
    Latest {
        label: door.label,
        password: door.password,
        destination: "".to_string(),
    }
}

pub fn is_latest(door: &Door) -> bool {
    matches!(door, Door::Latest(_))
}

pub fn any_to_latest(door: Door) -> Door {
    match door {
        Door::V1(door) => {
            let door = v1_to_v2(door);
            let door = v2_to_latest(door);
            Door::Latest(door)
        }
        Door::V2(door) => {
            let door = v2_to_latest(door);
            Door::Latest(door)
        }
        Door::Latest(door) => Door::Latest(door),
    }
}

or is there a better way to do this? Adding a bigger struct to Door and trying to deserialize an older version of it fails.

Have you seen this example? https://github.com/rkyv/rkyv/blob/master/examples/backwards_compat/src/main.rs

Have you seen this example? https://github.com/rkyv/rkyv/blob/master/examples/backwards_compat/src/main.rs

I guess a better example would be this:
pub enum Test {
    A(u16),
    B(u32),
}

pub fn main() {
    let test = Test::A(1);

    let mut serializer = AllocSerializer::<0>::default();
    serializer.serialize_value(&test).unwrap();
    let bytes = serializer.into_serializer().into_inner();

    println!("{:?}", bytes);
}

would print:
[0, 0, 1, 0, 0, 0, 0, 0]

however, if I add another element that increases the size of the enum:
pub enum Test {
    A(u16),
    B(u32),
    C(u64)
}

pub fn main() {
    // bytes from above
    let bytes: [u8; 8] = [0, 0, 1, 0, 0, 0, 0, 0];
    let archived = rkyv::check_archived_root::<Test>(&bytes[..]).unwrap();
    let deserialized: Test = archived.deserialize(&mut rkyv::Infallible).unwrap();

    println!("{:?}", deserialized);
}

it will fail to deserialize:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ContextError(ArchiveError(OutOfBounds { base: 0x7dcb6ff6e8, offset: -8, range: 0x7dcb6ff6e8..0x7dcb6ff6f0 }))', src\main.rs:199:66
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: process didn't exit successfully: `target\debug\rust-tests.exe` (exit code: 101)

i hope this makes it more clear.

When the C variant is added to Test, the size of Test increased from size_of::<Discriminant>() + size_of::<u32>() to size_of::<Discriminant>() + size_of::<u64>(). This breaks binary compatibility because even though the first two variants haven't changed, the overall enum has and rkyv checks for enough memory for an ArchivedTest before checking the enum discriminant. One way to work around this is to make sure that the size of Test never increases by boxing all of your enum fields:
#[derive(Archive, Serialize, Deserialize)]
pub enum Test {
    A(#[with(AsBox)] u16),
    B(#[with(AsBox)] u32),
    C(#[with(AsBox)] u64),
}

This isn't 100% perfect because if you add enough variants to increase the size of the discriminant, the size of the enum may change. But it should be good enough in most cases.

The discriminants for archived enums are now required to always be a single byte, so the previous example is now comprehensive. The linked example (backwards_compat.rs) looks comprehensive enough to describe how to add backwards compatibility support to your data format. Closing this as complete.


Updating the NPM packages dependencies

Updating the NPM packages for the following:

fernet
hubot
inflection
ip-address
chai
mocha
hubot-mock-adapter

This also adds the following new dependencies:

@coffeelint/cli
coffeescript, sprintfjs

Should someone be available for review, this should be in a state to be merged please.


how to change the resolution of the depth image?

The default w x h = 1280 x 1024.
How can we change it to other values? Any other options for the resolution?

The 3D data has the resolution of the camera sensor by default, since we compute a depth for each of the camera pixels.
You can change it by adjusting camera parameters like binning, scaling and AOI. See this tutorial for how to set parameters.


Yellow King - Minor fixes on translatable tags

Submission Checklist
Required

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Fixed wrong and missing data-i18n in some terms

Character Sheet Info Roll20 Internal Use only.


Add support for custom event listeners.

Addresses #6 by adding support for custom event listeners to be passed through the Lottie wrapper to bodymovin. Expects an array of event listener objects of the following form to be passed in.
const eventListeners = [
  { eventName: 'complete', callback: () => {console.log('complete render');} }
];

Thanks for this!


Implement pushing code to switch

A procedure and a cli function

Marked as important. This will simplify vlan changes.
capt find mac xx:xx:xx:xx:xx:xx
capt int x/x/x on x.x.x.x set vlan xxxx

Setting a VLAN should be: switch -> interface -> VLAN
More of a logical drill-down type sequence.
E.g.
capt switch x.x.x.x if x/x/x vlan xxxx
Also, allow range commands
capt switch x.x.x.x if range x/x/x-y vlan xxxx

capt conf x.x.x.x if x/x/x vlan xxxx

Can not have a subparser or parser attached to a parser. Therefore, the command implemented is:
capt change mac xx:xx:xx:xx:xx:xx --vlan yyyy
Completed.


I cannot debug code on my chrome browser

I have some js files that when I run them to debug, the browser gives me this error:
This site can’t be reachedlocalhost refused to connect.
Try:
Checking the connection
Checking the proxy and the firewall
ERR_CONNECTION_REFUSED

@DavidBenHorin reproducable steps please.

This means that your web server (like a webpack dev server) is not listening, or it's not on localhost:8080. You can change the address in your launch.json.


Error when selecting from database

Hello,
I am new to Vapor and running into some odd issues when I try to select from a MySQL database. It works locally (on OSX), but it does not work when I deploy to a prod environment.
In production, my docker container is running on a host in AWS EC2, connecting to an RDS instance. My other containers (which are not vapor 4.0) on the host can connect. Also, this is not a connection issue, I have command ("moments") which can successfully insert into the database. However, it seems as soon as I select there is a problem reading bytes.
It works locally, and so I cannot replicate locally, but fails every time I try to select using code like this:
LogMoment.query(on: req.db).limit(1).all()

MySQL Version: 5.6.37
The app was created using vapor-beta, because I wanted the app dockerized.
Here is the error:
Fatal error: file /build/.build/checkouts/mysql-nio/Sources/MySQLNIO/MySQLQueryCommand.swift, line 134
0x7fadc3c6088f
0x7fadc3fbd4e5
0x562aecb567d5, MySQLNIO.(MySQLQueryCommand in _2ACF6639C25D14CC4B926B0BFBB183FB).handle(packet: inout MySQLNIO.MySQLPacket, capabilities: MySQLNIO.MySQLProtocol.CapabilityFlags) throws -> MySQLNIO.MySQLCommandState at /build/.build/checkouts/mysql-nio/Sources/MySQLNIO/MySQLQueryCommand.swift:0
0x562aecb56b8f, protocol witness for MySQLNIO.MySQLCommand.handle(packet: inout MySQLNIO.MySQLPacket, capabilities: MySQLNIO.MySQLProtocol.CapabilityFlags) throws -> MySQLNIO.MySQLCommandState in conformance MySQLNIO.(MySQLQueryCommand in _2ACF6639C25D14CC4B926B0BFBB183FB) : MySQLNIO.MySQLCommand in MySQLNIO at /build/<compiler-generated>:0
0x562aecb417fe, MySQLNIO.MySQLConnectionHandler.channelRead(context: NIO.ChannelHandlerContext, data: NIO.NIOAny) -> () at /build/.build/checkouts/mysql-nio/Sources/MySQLNIO/MySQLConnectionHandler.swift:72
0x562aecbb1ee2, NIO.ChannelHandlerContext.(invokeChannelRead in _EEC863903996E9F191EBAFEB0FB0DFDD)(NIO.NIOAny) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/ChannelPipeline.swift:1339
0x562aecbb1f14, NIO.ChannelHandlerContext.(invokeChannelRead in _EEC863903996E9F191EBAFEB0FB0DFDD)(NIO.NIOAny) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/ChannelPipeline.swift:1341
0x562aecbae102, NIO.ChannelHandlerContext.fireChannelRead(NIO.NIOAny) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/ChannelPipeline.swift:1152
0x562aecb5db00, MySQLNIO.MySQLPacketDecoder.decode(context: NIO.ChannelHandlerContext, buffer: inout NIO.ByteBuffer) throws -> NIO.DecodingState at /build/.build/checkouts/mysql-nio/Sources/MySQLNIO/Packet/MySQLPacketDecoder.swift:32
0x562aecb5db78, protocol witness for NIO.ByteToMessageDecoder.decode(context: NIO.ChannelHandlerContext, buffer: inout NIO.ByteBuffer) throws -> NIO.DecodingState in conformance MySQLNIO.MySQLPacketDecoder : NIO.ByteToMessageDecoder in MySQLNIO at /build/<compiler-generated>:0
0x562aecbc96f6, closure #1 (inout A, inout NIO.ByteBuffer) throws -> NIO.DecodingState in NIO.ByteToMessageHandler.(decodeLoop in _F2A740607FEBA425AF6F8C49A24C0AD7)(context: NIO.ChannelHandlerContext, decodeMode: NIO.ByteToMessageHandler<A>.(DecodeMode in _F2A740607FEBA425AF6F8C49A24C0AD7)) throws -> NIO.(B2MDBuffer in _F2A740607FEBA425AF6F8C49A24C0AD7).BufferProcessingResult at /build/.build/checkouts/swift-nio/Sources/NIO/Codec.swift:567
0x562aecbc947e, NIO.ByteToMessageHandler.(withNextBuffer in _F2A740607FEBA425AF6F8C49A24C0AD7)(allowEmptyBuffer: Swift.Bool, _: (inout A, inout NIO.ByteBuffer) throws -> NIO.DecodingState) throws -> NIO.(B2MDBuffer in _F2A740607FEBA425AF6F8C49A24C0AD7).BufferProcessingResult at /build/.build/checkouts/swift-nio/Sources/NIO/Codec.swift:529
0x562aecbc947e, NIO.ByteToMessageHandler.(decodeLoop in _F2A740607FEBA425AF6F8C49A24C0AD7)(context: NIO.ChannelHandlerContext, decodeMode: NIO.ByteToMessageHandler<A>.(DecodeMode in _F2A740607FEBA425AF6F8C49A24C0AD7)) throws -> NIO.(B2MDBuffer in _F2A740607FEBA425AF6F8C49A24C0AD7).BufferProcessingResult at /build/.build/checkouts/swift-nio/Sources/NIO/Codec.swift:563
0x562aecbc9cf6, NIO.ByteToMessageHandler.channelRead(context: NIO.ChannelHandlerContext, data: NIO.NIOAny) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/Codec.swift:636
0x562aecbb1ee2, NIO.ChannelHandlerContext.(invokeChannelRead in _EEC863903996E9F191EBAFEB0FB0DFDD)(NIO.NIOAny) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/ChannelPipeline.swift:1339
0x562aecb914b6, NIO.ChannelPipeline.fireChannelRead0(NIO.NIOAny) -> () at .build/checkouts/swift-nio/Sources/NIO/ChannelPipeline.swift:829
0x562aecb914b6, NIO.BaseStreamSocketChannel.readFromSocket() throws -> NIO.BaseSocketChannel<A>.ReadResult at /build/.build/checkouts/swift-nio/Sources/NIO/BaseStreamSocketChannel.swift:107
0x562aecb8bf13
0x562aecc2658a, generic specialization <NIO.Socket> of NIO.BaseSocketChannel.readable() -> () at .build/checkouts/swift-nio/Sources/NIO/BaseSocketChannel.swift:1048
0x562aecc2658a, inlined generic function <NIO.Socket> of protocol witness for NIO.SelectableChannel.readable() -> () in conformance NIO.BaseSocketChannel<A> : NIO.SelectableChannel in NIO at /build/<compiler-generated>:1045
0x562aecc2658a, function signature specialization <Arg[2] = Dead> of generic specialization <NIO.SocketChannel> of NIO.SelectableEventLoop.handleEvent<A where A: NIO.SelectableChannel>(_: NIO.SelectorEventSet, channel: A) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/SelectableEventLoop.swift:323
0x562aecc23f4f, generic specialization <NIO.SocketChannel> of NIO.SelectableEventLoop.handleEvent<A where A: NIO.SelectableChannel>(_: NIO.SelectorEventSet, channel: A) -> () at /build/<compiler-generated>:0
0x562aecc23f4f, closure #1 (NIO.SelectorEvent<NIO.NIORegistration>) -> () in closure #2 () throws -> () in NIO.SelectableEventLoop.run() throws -> () at /build/.build/checkouts/swift-nio/Sources/NIO/SelectableEventLoop.swift:393
0x562aecc256c3, reabstraction thunk helper from @callee_guaranteed (@guaranteed NIO.SelectorEvent<NIO.NIORegistration>) -> (@error @owned Swift.Error) to @escaping @callee_guaranteed (@in_guaranteed NIO.SelectorEvent<NIO.NIORegistration>) -> (@error @owned Swift.Error) at /build/<compiler-generated>:0
0x562aecc256c3, partial apply forwarder for reabstraction thunk helper from @callee_guaranteed (@guaranteed NIO.SelectorEvent<NIO.NIORegistration>) -> (@error @owned Swift.Error) to @escaping @callee_guaranteed (@in_guaranteed NIO.SelectorEvent<NIO.NIORegistration>) -> (@error @owned Swift.Error) at /build/<compiler-generated>:0
0x562aecc2912f, NIO.Selector.whenReady(strategy: NIO.SelectorStrategy, _: (NIO.SelectorEvent<A>) throws -> ()) throws -> () at /build/.build/checkouts/swift-nio/Sources/NIO/Selector.swift:620
0x562aecc228b7, closure #2 () throws -> () in NIO.SelectableEventLoop.run() throws -> () at /build/.build/checkouts/swift-nio/Sources/NIO/SelectableEventLoop.swift:388
0x562aecc228b7, reabstraction thunk helper from @callee_guaranteed () -> (@error @owned Swift.Error) to @escaping @callee_guaranteed () -> (@out (), @error @owned Swift.Error) at /build/<compiler-generated>:0
0x562aecc228b7, generic specialization <()> of NIO.withAutoReleasePool<A>(() throws -> A) throws -> A at /build/.build/checkouts/swift-nio/Sources/NIO/SelectableEventLoop.swift:26
0x562aecc228b7, NIO.SelectableEventLoop.run() throws -> () at /build/.build/checkouts/swift-nio/Sources/NIO/SelectableEventLoop.swift:387
0x562aecbd9865, static NIO.MultiThreadedEventLoopGroup.(runTheLoop in _D5D78C61B22284700B9BD1ACFBC25157)(thread: NIO.NIOThread, canEventLoopBeShutdownIndividually: Swift.Bool, selectorFactory: () throws -> NIO.Selector<NIO.NIORegistration>, initializer: (NIO.NIOThread) -> (), _: (NIO.SelectableEventLoop) -> ()) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/EventLoop.swift:822
0x562aecbd9865, closure #1 (NIO.NIOThread) -> () in static NIO.MultiThreadedEventLoopGroup.(setupThreadAndEventLoop in _D5D78C61B22284700B9BD1ACFBC25157)(name: Swift.String, selectorFactory: () throws -> NIO.Selector<NIO.NIORegistration>, initializer: (NIO.NIOThread) -> ()) -> NIO.SelectableEventLoop at /build/.build/checkouts/swift-nio/Sources/NIO/EventLoop.swift:842
0x562aecbde919, partial apply forwarder for closure #1 (NIO.NIOThread) -> () in static NIO.MultiThreadedEventLoopGroup.(setupThreadAndEventLoop in _D5D78C61B22284700B9BD1ACFBC25157)(name: Swift.String, selectorFactory: () throws -> NIO.Selector<NIO.NIORegistration>, initializer: (NIO.NIOThread) -> ()) -> NIO.SelectableEventLoop at /build/<compiler-generated>:0
0x562aecc3b9de, reabstraction thunk helper from @escaping @callee_guaranteed (@guaranteed NIO.NIOThread) -> () to @escaping @callee_guaranteed (@in_guaranteed NIO.NIOThread) -> (@out ()) at /build/<compiler-generated>:0
0x562aecbde930, partial apply forwarder for reabstraction thunk helper from @escaping @callee_guaranteed (@guaranteed NIO.NIOThread) -> () to @escaping @callee_guaranteed (@in_guaranteed NIO.NIOThread) -> (@out ()) at /build/<compiler-generated>:0
0x562aecc3cbdd, closure #1 (Swift.Optional<Swift.UnsafeMutableRawPointer>) -> Swift.Optional<Swift.UnsafeMutableRawPointer> in static NIO.ThreadOpsPosix.run(handle: inout Swift.Optional<Swift.UInt>, args: NIO.Box<(body: (NIO.NIOThread) -> (), name: Swift.Optional<Swift.String>)>, detachThread: Swift.Bool) -> () at /build/.build/checkouts/swift-nio/Sources/NIO/ThreadPosix.swift:105
0x7fadc3c556da
0x7fadc1d9088e
0xffffffffffffffff

Package.swift:
import PackageDescription

let package = Package(
    name: "tickertracker",
    platforms: [
       .macOS(.v10_15)
    ],
    dependencies: [
        // 💧 A server-side Swift web framework.
        .package(url: "https://github.com/vapor/vapor.git", from: "4.0.0-rc"),
        .package(url: "https://github.com/vapor/fluent.git", from: "4.0.0-rc"),
        .package(url: "https://github.com/vapor/fluent-mysql-driver.git", from: "4.0.0-rc")
    ],
    targets: [
        .target(
            name: "App",
            dependencies: [
                .product(name: "Fluent", package: "fluent"),
                .product(name: "FluentMySQLDriver", package: "fluent-mysql-driver"),
                .product(name: "Vapor", package: "vapor")
            ],
            swiftSettings: [
                .unsafeFlags(["-cross-module-optimization"], .when(configuration: .release))
            ]
        ),
        .target(name: "Run", dependencies: [.target(name: "App")]),
        .testTarget(name: "AppTests", dependencies: [
            .target(name: "App"),
            .product(name: "XCTVapor", package: "vapor"),
        ])
    ]
)

Package.resolved:
  "object": {
    "pins": [
      {
        "package": "async-http-client",
        "repositoryURL": "https://github.com/swift-server/async-http-client.git",
        "state": {
          "branch": null,
          "revision": "037b70291941fe43de668066eb6fb802c5e181d2",
          "version": "1.1.1"
        }
      },
      {
        "package": "async-kit",
        "repositoryURL": "https://github.com/vapor/async-kit.git",
        "state": {
          "branch": null,
          "revision": "48a719dec79ea3cbbc71ca9b29f69df7cd5794cd",
          "version": "1.0.1"
        }
      },
      {
        "package": "console-kit",
        "repositoryURL": "https://github.com/vapor/console-kit.git",
        "state": {
          "branch": null,
          "revision": "7a97a5ea7fefe61cf2c943242113125b0f396a98",
          "version": "4.1.0"
        }
      },
      {
        "package": "fluent",
        "repositoryURL": "https://github.com/vapor/fluent.git",
        "state": {
          "branch": null,
          "revision": "dba22d5a6115092482669efe5492d800ace4da38",
          "version": "4.0.0-rc.2.2"
        }
      },
      {
        "package": "fluent-kit",
        "repositoryURL": "https://github.com/vapor/fluent-kit.git",
        "state": {
          "branch": null,
          "revision": "c73570c85d6661c04b8ea29a9f8eede67f6d4aae",
          "version": "1.0.0-rc.1.26"
        }
      },
      {
        "package": "fluent-mysql-driver",
        "repositoryURL": "https://github.com/vapor/fluent-mysql-driver.git",
        "state": {
          "branch": null,
          "revision": "74d95e80b998e3d143a0361cd0ecd8b43f2d9fd1",
          "version": "4.0.0-rc.1.3"
        }
      },
      {
        "package": "mysql-kit",
        "repositoryURL": "https://github.com/vapor/mysql-kit.git",
        "state": {
          "branch": null,
          "revision": "b59a08d77c3830b21f35fe744813640ef56a2ebb",
          "version": "4.0.0-rc.1.6"
        }
      },
      {
        "package": "mysql-nio",
        "repositoryURL": "https://github.com/vapor/mysql-nio.git",
        "state": {
          "branch": null,
          "revision": "6a9235582076122badc5428c8d5b6c41f9e40e1a",
          "version": "1.0.0-rc.2.2"
        }
      },
      {
        "package": "routing-kit",
        "repositoryURL": "https://github.com/vapor/routing-kit.git",
        "state": {
          "branch": null,
          "revision": "e7f2d5bd36dc65a9edb303541cb678515a7fece3",
          "version": "4.1.0"
        }
      },
      {
        "package": "sql-kit",
        "repositoryURL": "https://github.com/vapor/sql-kit.git",
        "state": {
          "branch": null,
          "revision": "1e0239616134dd45fa4894bfe83750d196e3a83d",
          "version": "3.0.0"
        }
      },
      {
        "package": "swift-backtrace",
        "repositoryURL": "https://github.com/swift-server/swift-backtrace.git",
        "state": {
          "branch": null,
          "revision": "f2fd8c4845a123419c348e0bc4b3839c414077d5",
          "version": "1.2.0"
        }
      },
      {
        "package": "swift-crypto",
        "repositoryURL": "https://github.com/apple/swift-crypto.git",
        "state": {
          "branch": null,
          "revision": "d67ac68d09a95443303e9d6e37b34e7ba101d5f1",
          "version": "1.0.1"
        }
      },
      {
        "package": "swift-log",
        "repositoryURL": "https://github.com/apple/swift-log.git",
        "state": {
          "branch": null,
          "revision": "74d7b91ceebc85daf387ebb206003f78813f71aa",
          "version": "1.2.0"
        }
      },
      {
        "package": "swift-metrics",
        "repositoryURL": "https://github.com/apple/swift-metrics.git",
        "state": {
          "branch": null,
          "revision": "708b960b4605abb20bc55d65abf6bad607252200",
          "version": "2.0.0"
        }
      },
      {
        "package": "swift-nio",
        "repositoryURL": "https://github.com/apple/swift-nio.git",
        "state": {
          "branch": null,
          "revision": "c5fa0b456524cd73dc3ddbb263d4f46c20b86ca3",
          "version": "2.17.0"
        }
      },
      {
        "package": "swift-nio-extras",
        "repositoryURL": "https://github.com/apple/swift-nio-extras.git",
        "state": {
          "branch": null,
          "revision": "f21a87da1353b97ab914d4e36599a09bd06e936b",
          "version": "1.5.0"
        }
      },
      {
        "package": "swift-nio-http2",
        "repositoryURL": "https://github.com/apple/swift-nio-http2.git",
        "state": {
          "branch": null,
          "revision": "b66a08e4bc53ab7c39fb03ab3678132e6ba5d12d",
          "version": "1.12.0"
        }
      },
      {
        "package": "swift-nio-ssl",
        "repositoryURL": "https://github.com/apple/swift-nio-ssl.git",
        "state": {
          "branch": null,
          "revision": "10e0e17dd47b594c3d864a063f343d716e33e5c1",
          "version": "2.7.3"
        }
      },
      {
        "package": "vapor",
        "repositoryURL": "https://github.com/vapor/vapor.git",
        "state": {
          "branch": null,
          "revision": "74bbf36fe0378fafe5413762affe8db6b871a380",
          "version": "4.5.1"
        }
      },
      {
        "package": "websocket-kit",
        "repositoryURL": "https://github.com/vapor/websocket-kit.git",
        "state": {
          "branch": null,
          "revision": "021edd1ca55451ad15b3e84da6b4064e4b877b34",
          "version": "2.1.0"
        }
      }
    ]
  },
  "version": 1
}

LogMoment (the model I'm just testing a select from):
import Vapor

final class LogMoment: Model, Content {
    init() {}

    static let schema = "logMoments"

    @ID(custom: .id) var id: Int?
    @Field(key: "startedAt") var startedAt: Date?
    @Field(key: "completedAt") var completedAt: Date?
    @Field(key: "allTickersCount") var allTickersCount: Int?
    @Field(key: "tickersProcessed") var tickersProcessed: Int?
    @Field(key: "tickersNotProcessed") var tickersNotProcessed: Int?
    @Field(key: "durationSeconds") var durationSeconds: Int?

    init(id: Int? = nil,
        startedAt: Date? = nil,
        completedAt: Date? = nil,
        allTickersCount: Int? = nil,
        tickersProcessed: Int? = nil,
        tickersNotProcessed: Int? = nil,
        durationSeconds: Int? = nil
) {
        self.id = id
        self.startedAt = startedAt
        self.completedAt = completedAt
        self.allTickersCount = allTickersCount
        self.tickersProcessed = tickersProcessed
        self.tickersNotProcessed = tickersNotProcessed
        self.durationSeconds = durationSeconds
    }
}

Controller I'm using (I recognize the class is TickerController, not Moment controller. I had the same issue with the TickerModel, so I wanted to test a different model):
struct TickerController {
    func index(req: Request) throws -> EventLoopFuture<[LogMoment]> {
        return LogMoment.query(on: req.db).limit(1).all()
    }
}

configure.swift:
import FluentMySQLDriver
import Vapor

// configures your application
public func configure(_ app: Application) throws {
    // uncomment to serve files from /Public folder
    // app.middleware.use(FileMiddleware(publicDirectory: app.directory.publicDirectory))

    app.databases.use(.mysql(
        hostname: Environment.get("DATABASE_HOST") ?? "****",
        username: Environment.get("DATABASE_USERNAME") ?? "****",
        password: Environment.get("DATABASE_PASSWORD") ?? "****",
        database: Environment.get("DATABASE_NAME") ?? "****",
        tlsConfiguration: .none
    ), as: .mysql)

//    app.migrations.add(CreateTodo())

    // register routes
    try routes(app)
    app.commands.use(MomentsCommand(), as: "moments")
}

I have tlsConfiguration: .none in there, because my of a SSL handshake error, when using .forClient()
Dockerfile:
# Build image
# ================================
FROM swift:5.2-bionic as build
WORKDIR /build

# First just resolve dependencies.
# This creates a cached layer that can be reused
# as long as your Package.swift/Package.resolved
# files do not change.
COPY ./Package.* ./
RUN swift package resolve

# Copy entire repo into container
COPY . .

# Compile with optimizations
RUN swift build --enable-test-discovery -c release

# ================================
# Run image
# ================================
FROM swift:5.2-bionic-slim

# Create a vapor user and group with /app as its home directory
RUN useradd --user-group --create-home --home-dir /app vapor

WORKDIR /app

# Copy build artifacts
COPY --from=build --chown=vapor:vapor /build/.build/release /app
# Uncomment the next line if you need to load resources from the `Public` directory
#COPY --from=build --chown=vapor:vapor /build/Public /app/Public

# Ensure all further commands run as the vapor user
USER vapor

# Start the Vapor service when the image is run, default to listening on 8080 in production environment
ENTRYPOINT ["./Run"]
CMD ["serve", "--env", "production", "--hostname", "<IP_ADDRESS>", "--port", "8080"]

docker-compose.yml:

services:
   tickertracker:
     image: ****
     expose:
       - 8001
     ports:
       - "8001:8001"
     restart: always
     command: ["serve", "--env", "production", "--hostname", "<IP_ADDRESS>", "--port", "8001"]
     environment:
       VIRTUAL_HOST: ****
       DATABASE_HOST: ****
       DATABASE_USERNAME: ****
       DATABASE_PASSWORD: ****
       DATABASE_NAME: ****
     container_name: tickertracker_prod
networks:
  default:
    external:
      name: nginx-proxy

Note, I have the port set to 8001, because I have multiple others containers on the host, so I am using docker-compose.yml to configure the port manually (which is also the reason for the nginx-proxy
I have tried to dig into why the fatalError() is being called, but I realized quickly this is way over my head.
Nothing seems to be out of date (in terms of my Package.resolved).
Any ideas?
Any other information that I can provide?

Sorry probably opened this is the wrong place.  Moved to:
https://github.com/vapor/fluent-mysql-driver/issues/177


I don't think that much people know that a new language must added inside items array of strings.xml of main values folder.

Thanks

@AbdurazaaqMohammed A way to fetch languages folders is needed to prevents problem like this in future.
I don't think that much people know that a new language must be added as array item inside strings.xml of main values folder. 😄

I can do it myself, I just forgot to do it in the last version, but I will try and set up a translation project for the app


Update to otel-go codecov presubmit

Fixes https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/issues/865
As a first pass, i'm applying some of the improvements that otel-go has implemented (https://github.com/open-telemetry/opentelemetry-go/pull/5440, https://github.com/open-telemetry/opentelemetry-go/pull/5534) to see if those fix our issues.
Codecov passed (both creating the coverage, and uploading it)!

Lint will be fixed after rebasing on https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/pull/867


App Configuration client.list_configuration_settings() unexpected keyword argument 'keys'

Package Name:
azure.appconfiguration

Package Version:
1.0.0

Operating System:
Mac

Python Version:
3.6.10

Describe the bug
Following the example for listing configuration settings (https://github.com/Azure/azure-sdk-for-python/tree/master/sdk/appconfiguration/azure-appconfiguration#list-configuration-settings), the for-each loop gives the error
TypeError: request() got an unexpected keyword argument 'labels'

suggesting that it cannot be iterated as shown.
To Reproduce
# set up client
client = AzureAppConfigurationClient.from_connection_string(connection_str)

# example given in the documentation
filtered_listed = client.list_configuration_settings(
    labels=["*Labe*"], keys=["*Ke*"]
)
for item in filtered_listed:
    pass

Expected behavior
Be able to run through the for-each loop without erroring. Please show a working example of how this iterator should be used.

Thanks for reporting the issue.
We have a fix for it and it will be available in next release.

Fixed in #12784


General housekeeping

Proposed changes:
Drop .NET 2.1 test support as it has officially reached EOL
Bump .NET Framework minimum version to 4.6.2 reason: .NET Framework 4.5.2, 4.6, 4.6.1 will reach End of Support on April 26, 2022
Add .NET 6.0 test support and update once more when RTM drops in November.
Move namespace to System.Threading.Tasks so people can leverage the code without having to import our specific namespace.
Update nuget API key as has expired
@jnm2 let me know if you're opposed to any of these.  I doubt you are but you know what they say about assumptions.

The target framework plan looks good to me.

Move namespace to System.Threading.Tasks so people can leverage the code without having to import our specific namespace.

This is a breaking change both for recompiling and for swapping the binary at runtime (e.g. when this package is transitively referenced on more than one path and the SDK picks a winner). I'm okay with this but this forces the version to 2.0.0.
Also, what do you think about not using a namespace? It would seem kind of odd to me that I would not need to import the System.Threading.Tasks namespace in order to do await FooAsync(); await BarAsync(); but then I would need to import it to switch to await (FooAsync(), BarAsync()); Perhaps this will become a moot point with the .NET 6 SDK adding default global usings when using C# 10+.

@jnm2 sorry for the late response but with RC1 dropping today I was probably going to release to nuget in concert with it.
I was definitely going to version it as 2.0
My assumption is/was that you would only be using this in async functions that return either Task or Task which are both in the Sytem.Threading.Tasks namespace ergo you would have already imported it.
Yes as far as the C# 10+ default global namespace bit it would become a moot point for people who have moved to the latest release.

Nice!


Release Token TS client bindings

This PR makes some minor fixes, moves the existing js dir to js-old, and moves the ts dir to js.
We should integrate the necessary changes from the js-old directory and delete it before merging.

All right, that was very tedious, but it's done! :crossed_fingers: on CI now

Are there any other "necessary changes" to port over that you know of? If not, I can nuke js-old

Incidentally, is there a reason this was built in a branch on the solana-labs repo, instead of from a fork?

Ah, was just my default since it's easier to manage. I can use a fork in the future if that's preferred?

I can use a fork in the future if that's preferred?

Yes, please :) Easier maintenance of the labs repos that way.

Anything holding this up?

I think we are good to go! I'll release the docs on Github Pages, just waiting to hear back from someone with access.


net/http: TestTransportPersistConnReadLoopEOF failures with Connection reset by peer on wasip1

#!watchflakes
post <- pkg == "net/http" && test == "TestTransportPersistConnReadLoopEOF" && `send: [Cc]onnection reset by peer`

Examples:

https://build.golang.org/log/db36c47e207d169418b1fb6cf8e729fc179ebeb9
https://build.golang.org/log/a384a8b3d229e534d42898bb547f5b7855d59205

--- FAIL: TestTransportPersistConnReadLoopEOF (0.00s)
    transport_internal_test.go:62: pc.closed = http.nothingWrittenError{error:(*net.OpError)(0x181c5a0)}, write tcp <IP_ADDRESS>:2-><IP_ADDRESS>:1: send: Connection reset by peer; want errServerClosedIdle or transportReadFromServerError
FAIL
FAIL	net/http	28.029s

See previously:

#49450
#22540

(CC @neild @golang/wasm)

Ah, I think this is related to #30938.
https://go.dev/cl/379554 changed the propagation of nothingWrittenError and also changed an earlier assertion in TestTransportPersistConnReadLoopEOF to allow it.

Hmm. I wonder if we just need to crank up rstAvoidanceDelay for this test. (Search for SetRSTAvoidanceDelay and apply a similar loop, perhaps?)


micro:bit game brightness broken recently

In this program, the two sprites now look the same on the real micro:bit. In the simulator, they appear to have the same brightness but the left one is flickering. I tried on live and beta.
let item = game.createSprite(1, 2)
let item2 = game.createSprite(3, 2)
item.set(LedSpriteProperty.Brightness, 1)

This is fixed.


Color of text changes

Issue Type: Bug
Copied an list in array format from Excel (text surrounded by double quotes with comma in between) into an array in .js file in VSCode.  Although no problems have been detected in the work space, the text color is white instead of brown.   If I reduce the length of the array sufficiently, the text color changes to brown.  No idea why this is happening.
VS Code version: Code 1.65.2 (c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1, 2022-03-10T14:33:55.248Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

Version: 1.65.2 (system setup)
Commit: c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
Date: 2022-03-10T14:33:55.248Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.22000
Steps
Get a list of cities in a country, e.g. Argentina from any website.
Copy list into Excel, remove columns leaving city names
Undo Wrap Text
Clear Hyperlinks
Sort A-Z.
Transpose list column to row
select all cells, format cells Custom, delete General, type "@",   This surrounds names in double quotes with a trailing comma, select all names,
In a .js file create array var argentina = [];
Then paste the names between the brackets.  // The text color is be white, expected brown
Delete about half of the array, the text color changes to brown

Type a backslash before each double quote, Github remove them after submitting.

Get a list of cities in a country, e.g. Argentina from any website.
Copy list into Excel, remove columns leaving city names
Undo Wrap Text
Clear Hyperlinks
Sort A-Z.
Transpose list column to row
select all cells, format cells Custom, delete General, type "@", This surrounds names in double quotes with a trailing comma, select all names,

I doubt these are minimal and free-of-interpretation steps: So please simply attach the file/contents that you are pasting


The ASSERT_EQ_EVENTUALLY macro expression may not compile with complex expressions

The ASSERT_EQ_EVENTUALLY macro expression evaluation works better if expression is evaluated using parenthesis since it is checked for equality with the == operand and without the extra parenthesis the equality may not work correctly depending on the expression.
Centos 7 Builds failed due to the following warning:
ClientReplicatedMapListenerTest.cpp:161:125: error: suggest parentheses around comparison in operand of ‘==’ [-Werror=parentheses] 02:20:04                  ASSERT_TRUE_EVENTUALLY(listener->keys.size() == 1U && listener->keys.pop() == 1 && listener->addCount.get() == 1);
Also removed the unused code ASSERT_NE_EVENTUALLY.

Windows test FAILed.

Linux test PASSed.

Windows test PASSed.

Linux test PASSed.

Linux test PASSed.

Windows test FAILed.

verify-windows

Windows test FAILed.

verify-windows

Windows test PASSed.


Use slash-syntax instead of old in-syntax in build.sbt

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

Thanks!


Attempt to make MaxRequestBufferSizeTests.LargeUpload less flaky

This is for the release/2.1 branch, but should probably be ported forward to master so the test can be unskipped.
Hopefully addresses https://github.com/aspnet/AspNetCore-Internal/issues/1775
See https://github.com/aspnet/AspNetCore-Internal/issues/1970 for the master version.

@halter73 is this needed in the current patch builds? I believe 2.1 builds have passed AspNetCore for the next release i.e. the builds succeeded (though a few tests may yet fail in the Test: Windows (Server 2012) build). Don't really want to back them up.
Might be fine for 2.2 either way because those AspNetCore builds are blocked by other PRs and their validations.

Skipping is fine for 2.1. I would like to get _dataLength to 100MB in all branches we can though.

@Eilon this is a test-only fix, what's the shiproom process here. Do we need to bring it there?

It's "Tell Mode" (send email only) for test-only changes. If we already have the final 2.1 build for this patch, let's get it in now. If not, maybe we wait until that's done so we don't bork anything by mistake.

For shiproom: This is a test-only fix to improve build stability. Please let us know when we're good to merge this into the 2.1 branch.

Don't need any shiproom labels, just merge when ready.


[Bug] unnecessary-string-conversion should not be raised in if condition

What happened?
I am using a Set variable if with a condition based on the value of an environment variable. Without quotes around the variable python won't recognize it as a string but as a variable name
What command/code did you try to run?
    ${output}       Set Variable If
    ...             any(substring in "%{ENV_VALUE}" for substring in @{PROTECTED_ENV_VALUES})
    ...             ${TRUE}
    ...             ${FALSE}

What is the full error message?
unnecessary-string-conversion is raised
What did you expect to happen instead?
unnecessary-string-conversion should not be raised
Operating System
No response
Robocop version
4.1.0

What about using %ENV_VALUE? (without brackets) Or only $.. supports passing variable directly?

I've tried that and unfortunately it does not get evaluated, only $ supports the removal of brackets in python evaluation

Thanks for confirming this. I will also check this and adjust the rule so it doesn't raise if it's impossible to avoid string conversion. And environment variables are strings already so there is no "unnecessary string conversion" anyway :)

Released in 4.2.0


Stop participation in the challenge

Added feature to stop participation in the challenge by the host.

Changes proposed in this pull request:

Added feature to stop participation in the challenge by the challenge host.

Link to live demo: http://pr-189-evalai.surge.sh

Codecov Report

Merging #189 into master will increase coverage by 0.51%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #189      +/-   ##
==========================================
+ Coverage   51.58%   52.09%   +0.51%
==========================================
  Files          65       63       -2
  Lines        3569     3411     -158
  Branches      404      385      -19
==========================================
- Hits         1841     1777      -64
+ Misses       1635     1540      -95
- Partials       93       94       +1

Impacted Files
Coverage Δ

...rc/app/components/challenge/challenge.component.ts
93.06% <14.28%> (-5.87%)
:arrow_down:

src/app/services/auth.service.ts
24.05% <0%> (-17.3%)
:arrow_down:

src/app/components/auth/auth.component.ts
90.9% <0%> (-2.43%)
:arrow_down:

...lengelist/challengecard/challengecard.component.ts
75% <0%> (ø)
:arrow_up:

...ts/auth/reset-password/reset-password.component.ts

...ssword-confirm/reset-password-confirm.component.ts

src/app/services/endpoints.service.ts
66.27% <0%> (+0.72%)
:arrow_up:

...rc/app/components/utility/input/input.component.ts
57% <0%> (+1%)
:arrow_up:

src/app/services/global.service.ts
40.42% <0%> (+2.22%)
:arrow_up:

...onents/auth/verify-email/verify-email.component.ts
77.77% <0%> (+8.21%)
:arrow_up:

... and 2 more

Impacted Files
Coverage Δ

...rc/app/components/challenge/challenge.component.ts
93.06% <14.28%> (-5.87%)
:arrow_down:

src/app/services/auth.service.ts
24.05% <0%> (-17.3%)
:arrow_down:

src/app/components/auth/auth.component.ts
90.9% <0%> (-2.43%)
:arrow_down:

...lengelist/challengecard/challengecard.component.ts
75% <0%> (ø)
:arrow_up:

...ts/auth/reset-password/reset-password.component.ts

...ssword-confirm/reset-password-confirm.component.ts

src/app/services/endpoints.service.ts
66.27% <0%> (+0.72%)
:arrow_up:

...rc/app/components/utility/input/input.component.ts
57% <0%> (+1%)
:arrow_up:

src/app/services/global.service.ts
40.42% <0%> (+2.22%)
:arrow_up:

...onents/auth/verify-email/verify-email.component.ts
77.77% <0%> (+8.21%)
:arrow_up:

... and 2 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 fbfd1f7...7f44566. Read the comment docs.

Thanks @sanketbansal for the PR, Closing this as it is opened here after resolving conflicts and with some little modifications.


[4pt]  - Display list of items on the Homepage - Student A

Create the main part of the homepage that keeps the layout from the wireframe:

When the page loads, the webapp retrieves data from  the selected API and shows the list of items on screen with the basic data (e.g. title + image).
Prepare also "Comments" and "Reservations" buttons. They should be doing nothing - just being displayed.
This task does not include displaying the number of likes for each item.
This task does not include the counter of the items.

great work this is the pull request: https://github.com/shakerAbuDrais/FreeEpicGames/pull/23


Allow table size customization

Allow the user to adjust the table size by setting its row/column size.

15d23aacd6869208fe8b1a238d25aebd14750f03
Implements option to customize row/column size. Update incoming after plugin has been publicly published.


Optimize Performance by Pre-sanitizing Sequence Before Existence Check

Hi,
Firstly, I'd like to extend my gratitude for your exceptional work.
I've encountered a potential issue regarding the verification of sequences within the .h5 file.
It appears that while the .h5 file stores sanitized versions of SMILES strings as here:
https://github.com/samsledje/ConPLex_dev/blob/6e7e27748092be86e0401206b90c7630f0d83c96/src/featurizers/base.py#L133
The preload function searches for non-sanitized versions: https://github.com/samsledje/ConPLex_dev/blob/6e7e27748092be86e0401206b90c7630f0d83c96/src/featurizers/base.py#L149-L151
This seems negligible for simpler, quicker featurizers like MorganFeaturizer. However, it becomes problematic with custom featurizers that require significant time to encode SMILES strings. Upon attempting to preload embeddings from the .h5 file after the initial run, the system fails to locate the sanitized sequences due to its search for non-sanitized versions.
To address this issue, I propose the following modifications.

Hi @cankobanz, thanks for submitting this pull request! This version of the code is meant to be a snapshot of the code at paper submission, to be used for replicating results from the manuscript. Please re-submit this pull request to the main repository (https://github.com/samsledje/ConPLex) and I will look it over there?