Add rkt prepare for preparing a container and allocating its uuid independent of rkt run

Then have rkt run-prepared accept just a uuid to run what was prepared at the uuid returned by rkt prepare.
rkt prepare a.aci b.aci c.aci will allocate the uuid by creating a directory under /var/lib/rkt/prepared, then perform all the image fetching and expanding rkt run would perform up to but excluding executing the stage1 run entrypoint.  The uuid is printed on the stdout following with the progress information, a --quiet option would be needed to restrict output to only the uuid for programmed invocations.
rkt run-prepared UUID will execute the container prepared at the UUID by renaming /var/lib/rkt/prepared to /var/lib/rkt/containers and jumping directly to the stage 1 entrypoint.
This might get a little awkward if options are supplied to rkt prepare which are persisted in the prepared container and contradicted by options supplied to rkt run-prepare, the global --debug option for example could possibly something in rkt prepare then rkt run-prepare without debugging enabled would produce debugged results.
/cc @yifan-gu @philips @jonboulle

Override with the options of rkt run-prepared seems to be more reasonable to me personally, sort of last/latest wins

Something along these lines? https://github.com/vcaputo/rocket/commit/270a496ab6e00ac50b1141f3647a1f22620e8e2a
/cc @yifan-gu @philips @eyakubovich

Just popped up from my mind: How does rkt run ${UUID} sound?

I found that after rkt prepare, I might get error from other operations (such as writing unit files to disk) and abort the creation of the container, so that uuid might be never used.
We might need to gc it...

What should rkt enter uuid do when given a uuid that has been prepared but not yet run? I guess it should fail because there is no container to enter yet. It would not find the uuid in containers/run/$uuid because it is still in containers/prepared/$uuid.

How is this interacting with overlayfs?
The benefit of overlayfs is that we have an expanded tree in the cas so we can then merge it with another directory to allow writes without changing the ACI tree. This expanded tree can be shared between executions of the image saving space and memory (when running several instances of an image at the same time). Since stage1 is an ACI it also benefits from this saving.
In my initial implementation I just mount the overlayfs in a new directory every time you execute an image instead of extracting it. If we do this in prepare, we would end up with lots of mounts that should be released later (some kind of gc). I don't like the idea of leaving mounted filesystems around.
A possible solution could be that rkt prepare fetches the image, prepares the runtime manifest (generating the UUID) and stops there. Then, rkt run will do the prepared -> containers renaming and mount the overlayfs. When the container exits, the overlayfs will be unmounted.
/cc @sgotti @vcaputo

Yesterday I threw these together to support rkt prepare and rkt run-prepared:
https://github.com/coreos/rocket/pull/556
https://github.com/coreos/rocket/pull/557
still need to update containers_test.go, but there's enough there to kick the tires.
I went ahead and made it so rkt status --wait UUID handles waiting for a container to get from uuid allocation down to exited, even if the container goes though a split prepare-run sequence.  For now rkt enter could just fail if the container isn't running, but we could give rkt enter a --wait option like rkt status.
It's not gcing prepared containers right now, that's a todo item; reclaiming stale prepared containers and stale uuids having no container.
@iaguis This should be independent of overlayfs, we're doing the same thing a normal rkt run does, just introducing an arbitrarily long delay between the prepare and run steps we're already performing.  Perhaps that has significant ramifications for resources pinned by the prepared images?

@yifan-gu I personally think rkt run already supports enough (too much) flexibility in its arguments, especially when combined with the support for abbreviated UUIDS we'd support for rkt run-prepared.  Running a prepared container requires no discovery and should have zero potential for ambiguity in terms of things like a file existing having the same name as the supplied UUID, for example.
What does everyone else think?

@iaguis This should be independent of overlayfs, we're doing the same thing a normal rkt run does, just introducing an arbitrarily long delay between the prepare and run steps we're already performing. Perhaps that has significant ramifications for resources pinned by the prepared images?

Yes, basically leaving the overlayfs mounts present between prepare and run-prepared. Maybe it's not important, especially if we mount it in a mount namespace. (#387)

@iaguis minimizing the startup latency and potential for failure of rkt run-prepared are both desirable.

@vcaputo mounting the overlayfs takes almost no time and we can verify the image's tree cache in rkt prepare to be sure rkt run-prepared will run correctly.
I think moving overlayfs mounting to run-prepared should be fine. If it's not supported by the system we would still do the extracting in prepare.
I implemented a POC of this in endocode/rocket:iaguis/prep (the last commit is the interesting one),

Implementation detail: I store the stage1 ID in a file in the container directory when running prepare so I know which stage1 I should mount in run-prepared. I get the Apps' image IDs from the CRM

This POC works and survives reboots. When overlayfs is not supported it still extracts in run-prepared but that can be easily moved to prepare.
PTAL.

@iaguis @vcaputo If a user removes (when implemented) an image (and so also the tree store) between prepare and run-prepared, is it ok for you that run-prepared fails? I think it can be ok.
Instead, if a way to avoid this is needed, a possible solution will be to check that the image isn't referenced in the CRM's of the prepared containers.

I don't think it's acceptable to permit removal of an image actively referenced, perhaps we need to maintain some kind of reference state?

@vcaputo Given that an "image" is composed (aci in the blob store, imagemanifest in the im store and the tree store) I previously though only about active containers. An active container can take locks on the images (using #571) and keep them opened (see last part of https://github.com/coreos/rocket/issues/387#issuecomment-76631101 with a proposal to let shim.c and capsule keep multiple file descriptors opened).
But between prepare and run-prepared the above isn't possible so there's the need to save a reference somewhere. As the image owner is the cas, for clearness, it should be the one aware of the reference. It can expose a function to permit rkt to add/remove/get a reference to an image (and these references can be saved in a db table, for example a one to many relation between an "image ID" and the N containers UUID referencing it).
As can happen to get stale references (due to different kind of problems) a sort of GC for these references should be needed (for example during a "rkt gc", it can ask the cas for the list of all the container's UUID in the references, and remove the ones of nonexistent containers).
Perhaps it's becoming too much convoluted...


chore(publish): fix dev dep install on docgen when publishing

Also adds yarn install before building instantsearch.js for the website

NB: it doesn't work with yarn 1.1.0. But it works with 1.2.0

https://github.com/algolia/instantsearch.js/blob/2208f7d93c5c1fa906b3782764fd9ed5de006f09/scripts/docs/publish.sh#L29
Since the publish scripts sets NODE_ENV to production this will not install devDependencies no?
We could force NODE_ENV to development if that's the case in build.sh

It's updated @vvo


Implement weeklyCaseDensity

This PR addresses issue https://trello.com/c/mQAfsaWF/399-add-cdc-community-level-related-metrics-to-api
Please Check

[x] Are the JSON schemas updated?
[x] Are tests passing?

@mikelehen  https://data.covidactnow.org/snapshot/3100/v2/cbsa/10100.timeseries.json
https://data.covidactnow.org/snapshot/3100/v2/county/20033.timeseries.json
https://data.covidactnow.org/snapshot/3100/v2/county/51085.json
https://data.covidactnow.org/snapshot/3100/v2/state/AZ.json
https://data.covidactnow.org/snapshot/3100/v2/states.timeseries.json
https://github.com/covid-projections/covid-data-model/runs/5668362439?check_suite_focus=true


Protocol test suite

I am writing a Swift client for GoTalk and it would be excellent if there were a protocol test suite available. I have the base protocol written, but verifying correctness is a matter of trial and error.
I may tackle this myself, but thought I'd file an issue to gather thoughts first.
I imagined a data-driven suite would probably be most useful for different language implementations. Perhaps something where an implementation can load a JSON file (or whatever) and playback simulated communications with a server or client.
Thoughts?

Great idea!

2c048e74cc87385dbcc2c82abd6f12f38e4f5cba adds a test suite that tests some aspects of communication in practice.
To try it:
git pull && make test


Background-position for sprite icons is calculated wrong

Hello everyone,
Since I upgraded compass and compass-rails, the background-position related to sprite icons is calculated wrong in the CSS. The result is that the wrong icon is displayed on my website every time I call for a sprite icon in my CSS.
Here are the changes to my Gemfile that caused/triggered this issue:
-gem 'sass-rails', '~> 4.0.5'
+gem 'sass-rails', '~> 5.0.1'

-gem 'compass-rails', '~> 2.0.0'
+gem 'compass-rails', '~> 2.0.4'

-gem 'susy', '~> 1.0.9'
+gem 'susy', '~> 2.2.5'

-gem 'sass', '~> 3.2.19'
+gem 'sass', '~> 3.4.16'

-gem 'compass', '~> 0.12.7'
+gem 'compass', '~> 1.0.3'

My ruby version is 2.1.2, but I have also tried upgrading to 2.2.1 and 2.2.2 and the problem is still there.
I am using Rails 4.2
I am happy to provide you with any other information that you may need.
Many thanks
PS I am not sure if I should be posting this under Compass/compass instead.

This is a compass bug. This project is only related to the compass/rails integration.

OK, thanks. I am moving it to Compass.


Started adding plot echelle in peakbag

I'm putting the echelle plot into peakbag.  I'm using the echelle from lighkurve as a base plot and then overplotting the fitted mode frequencies.  It looks good but there are some challenges.
I think there is an issue with the lightkurve echelle which is based around a small problem with the extent keyword in imshow.  The lightkurve team are on it in terms of fixing this.

The legend is a bit big.

Need to add some docs!

The legend is a bit big.

I'd say increase the size of the plot a little as well as decrease the legend a little 👍

Also, looks sick!!

The legend is a bit big.

I'd say increase the size of the plot a little as well as decrease the legend a little 👍

How do I do that in the seismology class?
Is it seismology.plot_echelle(dnu, numax, ..., figsize=[16,9]) ?
@ojhall94

I thiiink you can do
fig = plt.figure(figsize=(x, y))
ax = fig.gca()
ax = seismology.plot_echelle(ax)

I think eventually it should be a generic plot module, so it can be used
for all steps.
So:

Raw power
Epsilon
Asy peakbag results
Peakbag results

And with optional ax as input so user can overplot each step.
On Mon, 22 Jul 2019, 14:33 Guy Davies<EMAIL_ADDRESS>wrote:

I'm putting the echelle plot into peakbag. I'm using the echelle from
lighkurve as a base plot and then overplotting the fitted mode frequencies.
It looks good but there are some challenges.
I think there is an issue with the lightkurve echelle which is based
around a small problem with the extent keyword in imshow. The lightkurve
team are on it in terms of fixing this.
You can view, comment on, or merge this pull request online at:
https://github.com/grd349/PBjam/pull/132
Commit Summary

Started adding plot echelle in peakbag

File Changes

M docs/example-advanced.ipynb
https://github.com/grd349/PBjam/pull/132/files#diff-0 (258)
M pbjam/peakbag.py
https://github.com/grd349/PBjam/pull/132/files#diff-1 (25)

Patch Links:

https://github.com/grd349/PBjam/pull/132.patch
https://github.com/grd349/PBjam/pull/132.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/grd349/PBjam/pull/132?email_source=notifications&email_token=AEJWO334RX27WQARYTYTLV3QAX4RPA5CNFSM4IF3NV5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HAWWJ2Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEJWO32VUMROAF46GFPEY5LQAX4RPANCNFSM4IF3NV5A
.

I thiiink you can do
fig = plt.figure(figsize=(x, y))
ax = fig.gca()
ax = seismology.plot_echelle(ax)

If that was in before then it's possibly been removed ...
def plot_echelle(self, deltanu=None, numax=None,
                 minimum_frequency=None, maximum_frequency=None,
                 smooth_filter_width=.1,
                 scale='linear', cmap='Blues'):

I'll leave it as subotimal for now so I can move on ...


Use fullchain cert in nginx

Symlink the fullchain certificate for use in nginx instead of the client certificate. This is per documentation here (https://www.digitalocean.com/community/tutorials/how-to-secure-nginx-with-let-s-encrypt-on-ubuntu-14-04).

Hey jpgrusling, can you send the merge request to gitlab repo : https://gitlab.com/adi90x/rancher-active-proxy.git ?
I am using gitlab CI to auto build go app and building docker image.
And for the moment Gitlab repo is mirrored to Github every hour, not sure how it will handle it if we merge something here :s
Will need to find a way to fully synchronise both repo but I am not sure if it is possible !

Merge in Gitlab , Thanks a lot


User/rf/spelling and wording 01 01 29

@fabry-training super, thank you for the fixes, please resolve the current merge conflicts so that I can merge your changes into the main branch


Client side cli

Very cool project
i noticed that the user needs to do sone technical things in order to setup things on their desktop
the steps seem to be here : https://github.com/antoniomika/sish/blob/main/deploy/gcloud.md
would a cli for users be a good idea ?
it would essentially automate as many of the steps as possible for a user to interact with an already setup server.

Thanks for the kind words!
The installation part of sish is a bit out of purview for the client, as the whole goal is to try and use SSH as much as possible.
Setup is only required once, and the google compute engine setup is pretty easy to follow using their interactive tutorials page (here). I'd prefer to leave install to operators as we provide docker images and binaries for all different types of operating systems and architectures. Happy to add more instructions or example manifests to the wiki though if there are some you'd like to contribute!


feat(util) add PUT request helper

This change helps in issuing PUT requests.
Newer versions of Kong now support specifying ID via PUT requests and
not POST and hence this is necessary.

Codecov Report

Merging #98 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   31.94%   31.94%
=======================================
  Files          33       33
  Lines        2880     2880
=======================================
  Hits          920      920
  Misses       1847     1847
  Partials      113      113

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 96b2f0b...772619d. Read the comment docs.


feat: open for all hashes from hashlib

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

Project coverage is 100.00%. Comparing base (227703e) to head (995110c).

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #64   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        12    -1
  Lines          227       233    +6
  Branches        19        21    +2
=========================================
+ Hits           227       233    +6

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


Hide tray program in dock when there is no dialog open

Very low priority improvement.
The tray icon on all platforms should not be visible in the "dock" if it does not have a dialog window open. Only with a window should it be visible like so:

Just need to add these strings:
LSUIElement
1
right before . Need to change Info.plist. Tested and works.


eachLike doesn't seem to generate proper o/p in Java

below is my DSL
.object("dataStorePathInfo")
         .stringMatcher("basePath", String.format("%s/%s/training-data/[a-z0-9]{20,24}", CUSTOMER, TRAINING))
          .eachLike("fileNames", PactDslJsonRootValue.stringType("abc.txt") ,1)
.closeObject()

o/p generated is as follows
"dataStorePathInfo": {
    "basePath": "acctestcustomer/acctesttraining/training-data/l0at879y7786mv5z30g1",
    "fileNames": [
         "\"abc.txt\""
    ]
}

I was expecting file name in o/p to be just "abc.txt" and not "\"abc.txt\""

Hi, I've hit this issue also - just to add that it appears to affect all the array matching methods (e.g. minArrayLike, maxArrayLike, etc)

Am also hitting this issue, it seems to stem from https://github.com/DiUS/pact-jvm/commit/17554aaaa99a0d2751cdee75b2af218fb72b5342

Released 3.3.9 with this fix

Great, let me check

3.3.9 version works for me! Thank you!


Vanity helper

Description

Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Dependency changes
[ ] Deployment
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

Checklist:

[ ] The diff is legible and has no extraneous changes
[ ] Complex code has been commented, including external interfaces
[ ] Tests have 100% code coverage
[ ] The base branch is either main, or there's a description of how to merge

Issue Resolution

Close, because too outdated.


Pascal casing for demo function names

adapter_on_scan_stop
adapter_on_scan_updated
etc

and for all variables!
var
  adapter_identifier: PChar;
  peripheral_identifier: PChar;
  peripheral_address: PChar;

Fixed with commit 087cbea4e23d1d6fb673ddddf305e1a343c4ca1d


[Event Request] - OnBeforeUnapplyCustLedgEntry in Codeunit 12 "Gen. Jnl.-Post Line"

Hello, please consider adding the following event to Codeunit 12 "Gen. Jnl.-Post Line"
    [IntegrationEvent(false, false)]
    local procedure OnBeforeUnapplyCustLedgEntry(GenJnlLine2: Record "Gen. Journal Line"; DtldCustLedgEntry: Record "Detailed Cust. Ledg. Entry"; var IsHandled: Boolean)
    begin
    end;

and call it accordingly inside the "UnapplyCustLedgEntry" procedure:
    procedure UnapplyCustLedgEntry(GenJnlLine2: Record "Gen. Journal Line"; DtldCustLedgEntry: Record "Detailed Cust. Ledg. Entry")
    var
        Cust: Record Customer;
        CustPostingGr: Record "Customer Posting Group";
        GenJnlLine: Record "Gen. Journal Line";
        DtldCustLedgEntry2: Record "Detailed Cust. Ledg. Entry";
        NewDtldCustLedgEntry: Record "Detailed Cust. Ledg. Entry";
        CustLedgEntry: Record "Cust. Ledger Entry";
        DtldCVLedgEntryBuf: Record "Detailed CV Ledg. Entry Buffer";
        VATEntry: Record "VAT Entry";
        TempVATEntry2: Record "VAT Entry" temporary;
        CurrencyLCY: Record Currency;
        OldCustLedgEntry: Record "Cust. Ledger Entry";
        TempDimPostingBuffer: Record "Dimension Posting Buffer" temporary;
#if not CLEAN19
        TempInvoicePostBuffer: Record "Invoice Post. Buffer" temporary;
#endif
        AdjAmount: array[4] of Decimal;
        NextDtldLedgEntryNo: Integer;
        UnapplyVATEntries: Boolean;
        PmtDiscTolExists: Boolean;
        UnAppliedBill: Boolean;
        IsHandled: Boolean;
    begin

        **OnBeforeUnapplyCustLedgEntry(GenJnlLine2, DtldCustLedgEntry, IsHandled);
        if IsHandled then
            exit;**

        GenJnlLine.TransferFields(GenJnlLine2);
        if GenJnlLine."Document Date" = 0D then
            GenJnlLine."Document Date" := GenJnlLine."Posting Date";
...
...

Thank you !

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52098.


Do not use "mrousavy" as  temp file name.

What feature or enhancement are you suggesting?
Simplify library migration from v2 to v3 and also improve user usability.
File.createTempFile("mrousavy", extension, context.cacheDir)
outputFile = File.createTempFile("mrousavy", fileType.toExtension(), context.cacheDir)
Is there any reason to use own nickname in temporary file?
What Platforms whould this feature/enhancement affect?
Android
Alternatives/Workarounds
I guess it is better to use "tmp_" instead of "mrousavy".
File.createTempFile("tmp_", extension, dir)
Additional information

[X] I agree to follow this project's Code of Conduct
[X] I searched for similar feature requests in this repository and found none.

Is there any reason to use own nickname in temporary file?

Yea - I spent more than 2,000 hours on VisionCamera and thousands of people use it without saying thanks or sponsoring the project - using mrousavy here just helps me see if an app is using VisionCamera internally, which is exciting to see.
It's only a temp file, you're supposed to remove it later anyways. (Either store to Gallery or upload to server).


Update docs related to toml file

Problem
Looks like in the new gleam release version is a mandatory filed.
==> foo

error: File IO failure

An error occurred while trying to parse this file:

    ./gleam.toml

The error message from the file IO library was:

    missing field `version` at line 1 column 1

The error was explanatory enough but I added the version like
version = 0.0.1

and the error changed to
The error message from the file IO library was:

    expected newline, found a period at line 2 column 14

which looks completely alien to people(me) whom don't use toml often.
Proposal
Include the version to the sample toml config in README file.

Hello! What version are you using? I didn't think this had been released

This has been succeeded, thank you


Use web hooks for caching

Redesign the caching mechanism so that it is capable of purging select cached content based on HTTP messages sent from Kentico Cloud to the app's web hook endpoint.

Consider extracting to a separate repo.

@JanLenoch can we close this? now that you implemented #54

Absolutely! (Just didn't have time to groom this repo.)


M-M: Create Python script to dynamically add Many to Many, using Practice Area

Blockers/Dependencies

#35
#24

Overview
Create Python script to dynamically add Many to Many, using Practice Area so that each asset can be marked as relevant to each practice area.  This will allow the front end interface to display all asset results that are relevant to any specific practice area.
Details

See #24 for advantage of creating autogeneration scripts.  If too complicated, this can be skipped.
practice areas: engineering, product, ui/ux, etc.

Action Items

See #24

Manual Verification

Use for generating M-M for #37

Automated Tests

No.  Resulting script is manually verified.

This draft will be complete once Ethan adds action items


[fix][offload]support offload prefetchrounds & assignmentThreads adjustable

This PR is for running tests for upstream PR

/pulsarbot rerun-failure-checks

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


Unable to match $PGM variable

The following rule, attempting to match $PGM fails:
configuration <T> <k> test </k> <program> $PGM:String </program> </T>
syntax Test ::= "test"
rule <k> test => .K </k> <program> S </program>

Note that if the <program> S </program> section is removed then the rule successfully matches.

Can you please tell me what version of K you are using?

The latest version, I pulled it straight from master yesterday.
(Commit ef11f905bf7d7a6ec27e4f0eec0b7d0ff435f3b0)

Can you please tell me what command you are passing to krun? I tested the following and it works:
$ cat test.test
"5"
$ kompile test.k
$ krun test.test

My bad, I did not realize that the input had to be enclosed in quotes. Sorry for the trouble.


JBrowse label PomBase tracks

Could you label the PomBase tracks?
DNA track
data_type: PomBase track
alleles: WT
first_author: Wood
growth_phase_or_response: vegetative growth
pmed_id:11859360
study_ID:not specified
Reverse and Forward strand features tracks:
data_type: PomBase track
alleles: WT
first_author: Wood
growth_phase_or_response: multiple (combined data)
pmed_id:22039153
study_ID:not specified

This is done now in pombase-dev: http://pombase-dev.bioinformatics.nz/jbrowse/?loc=III%3A713819..740481&tracks=PomBase forward strand features%2CDNA%2CPomBase reverse strand features&highlight=
I assumed that it got be (or had to be) done in the JBrowse config file but it has to go in the big table: pombe-embl/website/pombase_jbrowse_track_metadata.csv
If you need any tweaks you edit that file and check it into SVN.

It's now in the main site too.

gene page links aren't live yet, but URLs with pombase.org/jbrowse/ work, e.g.  http://www.pombase.org/jbrowse/?loc=III%3A713819..740481&tracks=PomBase forward strand features%2CDNA%2CPomBase reverse strand features&highlight=

ok super...


C++ big endian encoding of floating point fields doesn't seem to be working correctly

When using byteOrder="bigEndian" in message schema definition (littleEndian works fine) with simple
field definition of
<field name="field1" id="11" type="float"/>

The attempt to encode floating point value seems to drop the part after decimal point:
msg.field1(5.4f);
std::cout << "value=" << msg.field1() << std::endl;

prints "value=5" (without .4)
The dump of the buffer shows: { 0, 0, 160, 76 }
When the same piece of code executed with "littleEndian" byte order it prints "value=5.4" and dump of the buffer shows: { 205, 204, 172, 64 }
Reproduced with v1.7.4 on Ubuntu 16.04 64bit with gcc v5.4.1

I can confirm this bug can be recreated.

It also fails for double. The conversation to an integer for the bswap operation is truncating the value.

Fix put in place. @mjpt777 try it out and let me know if it works. Or I will create a test to exercise it.

@tmontgomery Your changes worked. OTF decoding was missed so I added that.


Conditionally Hide Routes in Sidebar

User Story
As a developer,
I want to have a hidden prop in routes.js
So that I can create routes respected by react-router, but not shown in Sidebar.js
Description
A hidden prop can be added in routes.js that would still be loaded into the react-router, but not rendered in the Sidenav.
This will allow devs to not have to manually create custom router logic for their nested routes.
See PR:
https://github.com/horizon-ui/horizon-ui-chakra/pull/22/files?diff=split&w=1 (whitespace removed for readability)

Hello, @vcolmenar ! Thanks for supporting Horizon! For the moment, I will not merge, however, in future Horizon 2.0 we might add this ass a new feature,
Regards, Vlad.


Update badges links

Fix the wrong badges links

Codecov Report

Merging #930 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #930      +/-   ##
==========================================
- Coverage   55.55%   55.53%   -0.03%
==========================================
  Files         237      237
  Lines       23877    23877
==========================================
- Hits        13265    13259       -6
- Misses       9397     9402       +5
- Partials     1215     1216       +1

Impacted Files
Coverage Δ

pkg/log/logger.go
56.14% <0%> (-1.61%)
:arrow_down:

pkg/module/http2/server.go
81.55% <0%> (-0.37%)
:arrow_down:

pkg/network/connection.go
33.75% <0%> (+0.36%)
:arrow_up:

Continue to review full report at Codecov.

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


[CARBONDATA-1522] Support preaggregate table creation and loading on streaming tables

Added support to create preaggregate datamap on streaming table.
Added support to load data into datamap after handoff is fired for streaming table
Added transaction support for datamap loading

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[X] Any interfaces changed?

[X] Any backward compatibility impacted?

[X] Document update required?

[X] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3232/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4465/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4466/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3233/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/3982/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/3983/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4475/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3243/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/3991/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3305/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4533/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3388/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4613/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4115/

Add 2-3 test cases for time series feature also

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3411/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4637/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3413/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4639/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3415/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4145/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4147/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4148/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4150/

@kunal642 Please rebase

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4155/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4646/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3420/

retest sdv please

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4157/

@ravipesala @manishgupta88 Build passed

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4681/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3454/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4683/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3456/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4190/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4191/

LGTM

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4684/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4192/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3457/


https://github.com/anhkgg/SuperRDP

https://github.com/anhkgg/SuperRDP
You can set the checkbox AutoSupport， then SuperRDP2 will automatically support the new version
Originally posted by @wxpanada in https://github.com/stascorp/rdpwrap/issues/1860#issuecomment-1131152970
servir para a versão 21h2 Compilação 19044.1645?

Not an RDPWrapper issue


Can we add the webhook creator id in the request body

[Enter feedback here]
Hi
I was wondering can we add the service hook creator/owner id in the request body so that we will know whose service hook is triggered.

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

ID: 4806b641-fdb1-c0e9-081c-c007807c7cdb
Version Independent ID: f59d8501-1285-be79-b084-8371ed8395bb
Content: Webhooks with Azure DevOps - Azure DevOps
Content Source: docs/service-hooks/services/webhooks.md
Service: azure-devops
Sub-service: azure-devops-service-hooks
GitHub Login: @aaronhallberg
Microsoft Alias: aaronha

@xiaochenyang30 -- thank you for your question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@aaronhallberg -- please look into this issue.


Is there a way to avoid closing the modal, like a "isClosable" attribute so that the modal stays open and the user cannot close it?

Unfortunately not currently.
But this is a great idea @eMatsiyana , will look at that and push it in the next release

@eMatsiyana couldn't you simply turn off the swipe to close and not include a close button?

@michaelwschultz ... That would have worked but unfortunately, after turning off the swipe, " swipeToClose={false}" doesn't turn off the close function:
If you tap outside the modal, it still closes.

Yes, currently you cannot deactivate the swipeToClose feature after the component has been mounted.
I'm will look for a fix this week-end

That's perfect @maxs15 ... Looking forward to the fix :+1:

The isDisabled property has been added in the last version 1.2.2.
It should answer your needs :)
I'm closing the issue


Add ID number to emails (or remove the request)

The email you get when the scraper has an error or the post is gone tells you to submit an ID number but doesn't include the ID number:
"Please include the URL and the ID # in your email and we will look into the problem as soon as possible."
We either need to add the number or remove that part and just tell them to include the URL.

@ericaryan check please.

Confirmed!


Bump build number, update recipe, noarch, gcc7/r3.5

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Bump Ubuntu 14.04 to 16.04.

Images started failing to build with Ubuntu 14.04.
This is a breaking change, but the next version will be 0.2.0 in any case.
This brings up the question whether we should just bump all images to use 18.04 so we can have a common base image for all of them. To keep supporting the oldest version is clearly not feasible in the long run since images will keep breaking.

bors try

bors r+


Adding AsTracking

I am trying to set tracking behavior to "NoTracking" globally and override it for some instances.
So I tried using "UseQueryTrackingBehavior(QueryTrackingBehavior.NoTracking)" in the OnConfiguring method of my DbContext
and override it for specifications like this "Query.AsNoTracking(false)" but it does not work.
I also tried to set it to true for all specifications and override it in another method without setting "UseQueryTrackingBehavior(QueryTrackingBehavior.NoTracking)" in my DbContext but it didn't work either. (it seems that if AsNoTracking with the value true or without value at all gets called on the query it's sets no tracking to true but if it's gets called with the value false does not do anything)

Is this still something you need? Reopen if someone wants this.

Up, related issue https://github.com/ardalis/Specification/issues/336


Update Creating-a-Crew-and-kick-it-off.md to add crewai-tools as dependency

Without installing crewai-tools, the getting started example does not work, throwing the following error
ModuleNotFoundError: No module named 'crewai_tools'

This updates the docs to include installing crewai-tools during the installation step.

Closing this in favor of a different approach


Looping back to first slide isn't working when roundLengths is set to true

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 5.0.4

Platform/Target and Browser Versions: Mac Chrome 79, Safari 13.0.2, Firefox 69.0.3

Note, this seems to be the same issue as #2249, which has since been closed.

What you did
Swiper config:
    new Swiper(slider, {
        direction: 'horizontal',
        loop: true,
        slidesPerView: 1,
        spaceBetween: 40,
        roundLengths: true,
        navigation: {
            nextEl: '[data-master-slider=next-button]',
            prevEl: '[data-master-slider=previous-button]',
        },
        simulateTouch: false,
        breakpoints: {
            1280: {
                slidesPerView: 3,
            },
            768: {
                slidesPerView: 2,
            },
        },
    });

Expected Behavior
Clicking on the right arrow should keep looping through the slider indefinitely
Actual Behavior
When reaching the last slide in the DOM, I can no longer click forward. It does loop correctly in the backwards direction. When I remove roundLengths: true it loops correctly, but then I get the blur bug on non-retina screens that is roundLengths exists to fix.

@dskoziol can you provide a CodePen / JSFiddle recreating the issue


As a Medulla User, when I click on the Page editor from the dashboard, it will give me a form to fill out for a new page CRD, or it will ask me to edit an existing CRD, so that I can start editiing a page

Summary

Goal is to start editing a page, ask user if new page, or existing page,

Time Requirements

1 day

Acceptance Criteria

If new page, then give form

If existing page, then redirect to object browser

Closed for not following issue template.


[Style] Smooth transition for the sidebar accordion

What would you like to share or ask?
The sidebar accordion on LinksHub kind of feels too janky on opening. So some ease-in, ease-out transition should be used for the accordion.
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Great, thanks

@rupali-codes Can I take up this issue? I currently have no issues assigned. All my PRs have been successfully merged.


Object as Interface not asserting properly

Bug Report
🔎 Search Terms
object as interface not asserting properly
🕗 Version & Regression Information
Typescript Version: 4.1.3

I was unable to test this on prior versions because I just started with TS

⏯ Playground Link
Playground link with relevant code
💻 Code
interface ServerRoute {
    path: string;
    method: string;
    options?: object;
}

const workingRouteDefinition: ServerRoute = {
    method: 'get',
    path: '/'
};

// errors due to missing path
const badRouteDefinition: ServerRoute = {
    method: 'get',
    // path: '/'
};

// doesnt error even though there is no path
const weirdRouteDefinition = {
    method: 'get',
    // path: '/'
} as ServerRoute;

// errors due to missing path, but not the extra key
const weirderRouteDefinition = {
    method: 'get',
    // path: '/',
    otherString: 'what'
} as ServerRoute;

Workbench Repro
This is primarily evident when doing something like export default {} as Interface in real code, but can happen when assigning consts as well in the above example case.
🙁 Actual behavior
Errors are not thrown where they should be.
🙂 Expected behavior
Errors due to missing keys should always be present, not just when there's a third unknown key added. Unknown keys should have errors as well.

as will downcast (weirdRouteDefinition) because this is what it's for, but it will not "side-cast" -- the expression type and the asserted type must be in some way related.

@RyanCavanaugh thanks for getting back to me! I think what's confusing is that adding an extra key to the object changes whether it errors due to a missing path or not. I would (personally) expect TS to consistently assert on the presence/absence of path if the casting is consistently applied. I could understand if adding otherString caused path to stop erroring as the casting changed, but it's weird to me that it causes path to start erroring when it otherwise wasnt.

This is described here: https://www.typescriptlang.org/docs/handbook/2/everyday-types.html#type-assertions
and here
https://www.typescriptlang.org/docs/handbook/type-compatibility.html

Okay, so I still think there's weird behavior happening. I'm going to try to explain using phrases from the documentation that you've so kindly linked.

The basic rule for TypeScript’s structural type system is that x is compatible with y if y has at least the same members as x.

Great, this absolutely make sense. All members of x must be present in y for a type to be considered compatible. This lines up with the behavior seen below in my first two examples:
const workingRouteDefinition: ServerRoute = {
    method: 'get',
    path: '/'
};

const badRouteDefinition: ServerRoute = {
    method: 'get',
};

workingRouteDefinition contains every member of ServerRoute, so this instantiation passes type checking. badRouteDefinition does not contain every member of ServerRoute, so this explodes during type checking.
Now we get to the cases that use type assertions rather than directly assigning types like the examples above do.
const weirdRouteDefinition = {
    method: 'get',
} as ServerRoute;

This weirdRouteDefinition unexpectedly passes the type assertion check. The docs from the type assertion section in v2 of the handbook states the following:

you can use a type assertion to specify a more specific type

TypeScript only allows type assertions which convert to a more specific or less specific version of a type

As far as I can tell, ServerRoute is a more specific version of the inferred type object, so this assertion is allowed by the compiler.
What doesn't make sense here is why a more specific type assertion is failing to error due to missing the path key.
If we then look at the final example, we'll see that the more specific assertion starts working, but only after a key is added that isn't present in the more specific assertion.
const weirderRouteDefinition = {
    method: 'get',
    otherString: 'what'
} as ServerRoute;

The above snippet will start throwing errors about missing path even though it seems like ServerRoute is actually a less specific type assertion due to the additional key.
In both of these examples, the number of overlaps between the type asserted object and the type is the same single member. I'm not sure why TS is considering one of these objects to be missing a path key while the other isn't missing the path key when the number of overlapping members between the two objects and the type they are being asserting on is the same.
I dont see anything in the documentation that explains this behavior, I actually feel as though the documentation states the the behavior should be the inverse.
The first example should error due to a missing path as the type being asserted against has more members than the object being checked which fails the "y has at least the same members as x" check from the type compatibility docs.
The second example I gave should also error due to missing the path key, like it is currently doing.
The following example I would expect to pass type checking, as it passes the type compatibility "y has at least the same members as x" check outlined in the docs, and it is a "less specific" type that is being asserted against.
const extendedRouteDefinition = {
    method: 'get',
	path: '/test',
    otherString: 'what'
} as ServerRoute;

If this can't convince you that there's inconsistent behavior here when compared to the docs I'll stop pestering you, but everything you cited seems to state that the behavior I'm experiencing is unexpected due to type compatibility checking and the fact that in the problematic scenario I'm specifying a "more specific" version of a type and en error isn't being thrown for a missing key.

This behavior is 100% consistent with the definition of an allowable type assertion - that the expression type and the asserted type are comparable (which is the same as "assignable to or from" for the cases shown here). Every example here can be shown to be correctly following that rule.
You seem to expect that type assertions should not be capable of downcasting, but downcasting is the primary use case of type assertions, which I think indicates some fundamental misapprehensions about what's supposed to be happening here.

Thanks for the explanations Ryan. Much appreciated.


DropTarget doesn't account for default props

Consider the following:
// I was the drop target type to be driven via the `props.type` value (which is derived from the component's default props)
@DropTarget(props => { console.log(props); return props.type; }, spec, collect))
export default class Sortable extends Component {
	static defaultProps = {
		type: "Test"
	}
}

However, the console statement returns an empty object ({}). Looks like defaultProps never gets passed to the DropTarget.

Yeah, it's a bit of a gotcha.. See http://gaearon.github.io/react-dnd/docs-faq.html#why-won-39-t-my-static-methods-and-properties-work-.
This will work as intended:
@DropTarget(props => { console.log(props); return props.type; }, spec, collect))
export default class Sortable extends Component {
}

Sortable.defaultProps = {
  type: "Test"
}

Inner component's default props are just not available to the outer component provided by React DnD.
The link above explains why I don't want to build heuristics for that into the library itself.

Thanks for the fast reply.
Unfortunately, I'm not sure this will work for my needs as I need defaultProps to call a function which generates a new unique type for each component instance.
@DropTarget(props => { console.log(props); return props.type; }, spec, collect))
export default class Sortable extends Component {
    static defaultProps = {
        // This should be a unique value for each Sortable instance
        type: generateUUID()
    }
}

However, if I move this outside of the class definition, then the UUID will not be new for each instance. It will be the same for all component.
Any ideas how to get around this issue?

Perhaps put that UUID in the state instead?

I don't think I can access the state in the DropTarget decorator definition, ie:
// This only gives me access to 'props'. Not 'state'
@DropTarget(props => { console.log(props); return props.type; }, spec, collect))

Unfortunately, I'm not sure this will work for my needs as I need defaultProps to call a function which generates a new unique type for each component instance.

As far as I know this is a wrong way of using defaultProps. It's against how it's supposed to work in React. They should be static. Here is an example of how this can break with React 0.14 compilation optimizations: https://github.com/facebook/react/issues/3228.
To achieve what you want, there is no other way but to wrap your whole component into another component that generates UUID in the state or its constructor and passes it down.


Link to File Upload broken if it contains Exotic Characters

Describe the bug
If files are uploaded with unusual characters such as – (en dash) in the file name, the link for downloading the submission (at least from the student's perspective) is broken.
To Reproduce

Create a file upload excercise
Upload a file with em dash or other exotic chars and submit
View your Submission
Click on download file
The link should not work

Expected behavior
The submission is shown/downloaded.
Screenshots

Environment
Does the download work on... ?
OS Windows 10:

Google Chrome (78.0.3904.97) DOES NOT WORK
Vivaldi (2.8.1664.40) DOES NOT WORK
Firefox (68.0.2) WORKS
Microsoft Edge (44.18362.387.0) WORKS

OS Android 9:

Google Chrome (78.0.3904.96) DOES NOT WORK
Vivaldi (2.7.1624.277) DOES NOT WORK
Firefox (68.2.0) WORKS

Additional context
Submission location: artemis.ase.in.tum.de/api/files/file-upload-exercises/769/submissions/452148/Deckblatt–AfI.pdf
This (and maybe some problems connected to this) also happened to some students after submitting their homwork as well as to some tutors grading the homework.
// Updated the system environment details as they are more important than initially thought.

We tested this on all supported browser. This seems to be an issue only for Vivaldi.
Please use a different browser in this case. We might fix this, but we cannot verify this with exotic browser environments.

I can now confirm that the Problem does not exist for Firefox (68.0.2, Win 10) on my PC.
However, the current Google Chrome Version 78.0.3904.97 (Offizieller Build) (64-Bit), Windows 10
and Google Chrome Version 78.0.3904.96 for Android 9 also have this problem.

Ok, then it seems that more browsers have this issue. We will try to normalize this URL so that the problem.
@ungaralex could you add url normalization here?

Update:
We got this problem for multiple submissions and were informed by tutors that they could not view or download the file they should review. As we got all submissions from the Artemis team we could check for which submissions/filenames this applies (in the CSV file):
These can't be opened at all
i.e. one clicks on "Submission File" it seems to open a new tab but it closes instantly
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/469045/assessment
469045;"/api/files/file-upload-exercises/770/submissions/469045/C:\Users\Baraa\Desktop\PGDP\Syntaxbaum.pdf";
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/460195/assessment
460195;"/api/files/file-upload-exercises/769/submissions/460195/C:\Users\49151\Desktop\PGDP\Übungsfolien\Scan2019-11-0917.16.03HausaufgabePGDPPinguinreihe.pdf";
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/439091/assessment
439091;"/api/files/file-upload-exercises/770/submissions/439091/C:\Users\erick\Documents\TUM\5.Semester\PGdP\Syntaxbaumwurzel.pdf";
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/460189/assessment
460189;"/api/files/file-upload-exercises/770/submissions/460189/C:\Users\49151\Desktop\PGDP\Übungsfolien\Scan2019-11-0917.09.14HausaufgabePGDPSynthaxbaum.pdf";
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/469040/assessment
469040;"/api/files/file-upload-exercises/769/submissions/469040/C:\Users\Baraa\Desktop\PGDP\Pinguinreihen.pdf";
https://artemis.ase.in.tum.de/#/file-upload-exercise/769/submission/439099/assessment
439099;"/api/files/file-upload-exercises/769/submissions/439099/C:\Users\erick\Documents\TUM\5.Semester\PGdP\Niugnip.jpg";
https://artemis.ase.in.tum.de/#/file-upload-exercise/769/submission/414483/assessment
414483;"/api/files/file-upload-exercises/769/submissions/414483/C:\Users\laeti\Desktop\UNI\pgdp\Pinguinreihe.jpg";
Here at least a new Tab is opened
but the "website cannot be reached" error is shown along with the url
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/467771/assessment
467771;"/api/files/file-upload-exercises/769/submissions/467771/Aufgabe-w03-h02—Pinguinreihen.pdf";"ge75qav"
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/460464/assessment
https://artemis.ase.in.tum.de/api/files/file-upload-exercises/769/submissions/460464/Woche03H02PinguinreihenGrubmüller.pdf
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/458638/assessment
https://artemis.ase.in.tum.de/api/files/file-upload-exercises/769/submissions/458638/PinguinReihe-RegulärerAusdruck.pdf
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/427447/assessment
https://artemis.ase.in.tum.de/api/files/file-upload-exercises/769/submissions/427447/Woche03H02-Pinguinreihen-NiklasClüsserath.pdf
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/443229/assessment
443229;https://artemis.ase.in.tum.de/api/files/file-upload-exercises/769/submissions/443229/PGDPWoche03Hausaufgabe02–ReguläreAusdrücke.pdf
Here one gets a 401 Api error
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/454407/assessment
454407;"/api/files/file-upload-exercises/770/submissions/454407/syntaxBaumAbgabe#.PNG";
{"type":"https://www.jhipster.tech/problem/problem-with-message","title":"Unauthorized","status":401,"detail":"Full authentication is required to access this resource","path":"/api/files/file-upload-exercises/770/submissions/454407/syntaxBaumAbgabe","message":"error.http.401"}

Update:
We got this problem for multiple submissions and were informed by tutors that they could not view or download the file they should review.

I should add that in many cases the browser is not important anymore, it won't work, even for Firefox.

Here at least a new Tab is opened

Only here switching the browser (e.g. to Firefox) works, because the problem are only the exotic characters in the filename.

I investigated the issue. Seems the full Windows path was used for students with Windows and Microsoft Edge as browser :-(
I found less than 10 entries with 'C:' in the database and fixed them manually in the filesystem and the database.
I implemented a bugfix consisting of 2 improvements:

We normalize the URL on the client side. This should solve most problems for existing file submissions with non ascii characters that need to be encoded in a URL

We now handle wrongly formatted filenames on the server side:
a) In case of windows paths, we only take the last element, i.e. the correct file name
b) we remove all illegal characters, i.e. everything except A-Z, a-z, 0-9 and . and -
c) In case the filename gets too short, i.e. less than 5 characters including the file extension (e.g. a file called ßßß.pdf), we prepend 'file' to make sure the filename has entries before the file extension (e.g. ßßß.pdf --> file.pdf)

We need to verify this during testing and will deploy this with the next release.
Let me know if you find additional cases that are not covered in the 2 improvements above and feel free to reopen the issue then.

The files with backslash can now be opened, but for files with "website cannot be reached" the error still persists. i.e. these submissions:
Problem: English Dash
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/467771/assessment
Problem: Umlaut?
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/460464/assessment
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/458638/assessment
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/427447/assessment
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/443229/assessment

The changes have not yet been deployed. We plan to deploy it tonight after some additional testing.

Suggestion would be to use this regex [^\x00-\x7F]

We have a even stricter one, so this should be fine

The new fix doesn't seem to work in Chrome and Safari, but only in Firefox.
See example
https://artemis.ase.in.tum.de/#/file-upload-exercise/770/submission/458638/assessment

I get the same error, but I do not understand why. I tested it again locally and on our test server and it works correctly in Chrome. It might be an encoding problem. Maybe the file was uploaded on a Windows machine with a different encoding than our server is used to and now the problematic ä is differently encoded.
The good thing is, in the future there won't be such problem because we replace such characters.
Would be great if you can use Firefox in such cases for now, because it would be really hard for us to fix the problem


massconvert for hdbtabletype

Hi Jung,
Do you plan to support massConvert from hdbdd (Type) to hdbtable or hdbtabletype.
For example:
context GLOBAL {
type AUDIT {
	CREATED: UTCTimestamp;
	CREATEDBY: Integer64;
	CHANGED: UTCTimestamp;
	CHANGEDBY: Integer64;
};

entity TTO{
	key PO_TYPE: String(32);
	PRIO: Integer;
};

};
During massconvert, "entity" is converted but not the "type". I have around 100 such types needs to be converted to hdbtable or hdbtabletype.
Do you able to take this enhancement request?
Regards,
Karthick

I'm afraid that wouldn't be in the scope of this tool.  We read the SQL/runtime definition of the object (regardless of origin/design time type) and then convert that to a new target.  But what you are describing would require input of the existing hdbdd and processing it directly.

Looks like the issue here is just the :: in the original object name which isn’t valid in CAP CDS. You can use the noColons option to replace those with a period instead
Replace :: in table/view path with
dot        [boolean] [default: false]

Thanks but still it is returning the same error and program hasn't terminate.
hana-cli inspectTable -s HFP_INTERNAL -t HFP_PMT.TABLE::GLOBAL.AUDIT -o hdbtable -a --noColons
Using Connection Configuration loaded via default-env-admin.json
CDS compilation failed
.cds:1:22-23: Error: Extraneous ‘:’, expecting ‹Identifier›
.cds:1:23-35: Error: No artifact has been found with name “GLOBAL_AUDIT” (in entity:“HFP_PMT_TABLE”)
And my main request is do you able to support this massConvert also ?

You need to add “true” after noColons

Sorry to trouble you again, I am unable to see any effect of using --noColons true in the command.
output CDS generates cds file with ::.
hana-cli inspectTable -s HFP_INTERNAL -t HFP_PMT.TABLE::GLOBAL.AUDIT --noColons true -a -o hdbtable --hana
Using Connection Configuration loaded via default-env-admin.json
CDS compilation failed
.cds:1:22-23: Error: Extraneous ‘:’, expecting ‹Identifier›
.cds:1:23-35: Error: No artifact has been found with name “GLOBAL_AUDIT” (in entity:“HFP_PMT_TABLE”)
^C
C:\Users\I312836>hana-cli inspectTable -s HFP_INTERNAL -t HFP_PMT.TABLE::GLOBAL.AUDIT --noColons true -a -o cds --hana
Using Connection Configuration loaded via default-env-admin.json
@cds.persistence.exists
Entity HFP_PMT_TABLE::GLOBAL_AUDIT {
CREATED: Timestamp  @title: 'CREATED' ;
CREATEDBY: Integer64  @title: 'CREATEDBY' ;
CHANGED: Timestamp  @title: 'CHANGED' ;
CHANGEDBY: Integer64  @title: 'CHANGEDBY' ;
}

You said you wanted to do the massConvert.  The noColons option is in the massConvert command. https://github.com/SAP-samples/hana-developer-cli-tool-example#massconvert

Just published a new version with two things that should help. I added the noColons option to inspectTable:

And the massConvert command will now find user defined types (Table Types):


Error running

When I follow the README setup instructions exactly, I get this error when I run:
MainApplication.java:42: error: cannot find symbol
          new LANScanReactModule()

If I do what has worked with other native modules, that is run react-native link react-native-lanscan and also add an import to MainApplication.java like so:
import com.example.reactnativelanscan.LANScanReactModule;

I get this error (in addition to the above error):
MainApplication.java:18: error: cannot find symbol
import com.example.reactnativelanscan.LANScanReactModule;
                                     ^

Thoughts?

Hi !
Did you try importing from the correct package in MainApplication.java that is import com.odinvt.lanscan.LANScanReactModule ?
If so then you probably didn't put the include lines in the correct gradle files.
You should put this :
project(':react-native-lanscan').projectDir = new File(settingsDir, '../node_modules/react-native-lanscan/android')
inside the file MyReactNativeProject/android/settings.gradle. This basically tells gradle that if you include a module with the id :react-native-lanscan it should look for the source files of the module in the folder mentioned.
Then in the same file there should be a line include ':app' you just need to add there the ':react-native-lanscan module so the line should look like include ':app', ':react-native-lanscan' or if you use other react native plugins that have native code it looks like include ':app', ':react-native-lanscan', ':plugin2', ':plugin3'.
Then look for the file MyReactNativeProject/android/app/build.gradle (the one inside the app folder not the one inside the android folder) and look for the dependencies block and add the line compile project(":react-native-lanscan") to it so it should look something like this :
dependencies {
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile "com.android.support:appcompat-v7:23.0.1"
    compile "com.facebook.react:react-native:+"  // From node_modules
    compile project(':react-native-lanscan')
    compile project(':plugin2')
    compile project(':plugin3')
}

again depending on which modules your project depends on.
If you went through all these steps correctly and still get the error then probably gradle didn't compile the module correctly or has corrupt dependencies cached if so try deleting the .gradle/ folder inside your project's android folder and the one in your home folder under %USER_HOME%/.gradle/caches. Try also deleting the MyReactNativeProject/android/app/build/ folder if you have one then run react-native run-android inside your project folder.
If your project compiles correctly and you don't get any errors in your command line but you still do in Android Studio then try refreshing your project and using the gradle clean feature of android studio as that clears AS's cache as well and rerun.
Let me know how that works for you. Cheers !

The problem is here: https://github.com/Odinvt/react-native-lanscan/blob/f9185b65163a3dad4ba6ce8704c7183ec971386a/android/src/main/AndroidManifest.xml#L2
I guess react-native link uses AndroidManifest.xml to generate the import line in MainActivity.java.

Yes you're right ! I'll solve that as soon as I can. Thanks !

Hi @timscott, were you able to resolve this?


Tests that fail

FAILED grizli/tests/test_fits.py::FittingTools::test_redshift_fit - OSError: /Users/momcheva/dev/grizli_conf/templates/fsps/fsps_QSF_12_v3_nolines_001.dat not found.
FAILED grizli/tests/test_jwst.py::JWSTHeaders::test_process_headers - AttributeError: 'ASDFValidator' object has no attribute 'hdulist'

Hmm... I'm not running into this failure.
pytest reports no errors and the only thing I did differently from the installation guide, was to set $GRIZLI to a location different from ${HOME}/grizli (I set $HOME/.config/grizli).


Adds a k8s-cluster module to community

This change adds a module to create a k8s cluster.
This initial implementation seeks to achieve simplicity by exposing a limited set of functionality which will grow over time where there is clear value. As part of this objective the following functionality has been excluded:

Autopilot is disabled
Auto-provisioning of new node pools is disabled
Network policies are not supported
General addon configuration is not supported
Only regional cluster is supported

Follow on change is coming with basic documentation for module and example.
Tested: Manually deployed 2 clusters into the same project using the provided example as a basis.

[x] Is unit test coverage still above 80%?
[x] Have you updated all applicable documentation?
[x] Have you followed the guidelines in our Contributing document?

In parallel to your review I will update the naming of the module and look into tf-lint failures. It should not affect the discussed topics.


add CI workflow to build and push multiubuntu image on change

Feature Request
Short Description
add a CI workflow to build and push multiubuntu image whenever a change detected https://github.com/kubearmor/KubeArmor/blob/main/examples/multiubuntu/build/Dockerfile

I'd like to take this up!

hi @sikehish would you like to work on this issue together?

hi @sikehish would you like to work on this issue together?

sure, but Ive never teamed up to solve an issue, so Im not very familiar. Also how would we raise a PR?

hi @sikehish , can we continue this convo in slack?

hi @sikehish , can we continue this convo in slack?

sure, or maybe discord?

yeah sure, can i get your tag?

yeah sure, can i get your tag?

hisham5

Feature Request
Short Description
add a CI workflow to build and push multiubuntu image whenever a change detected https://github.com/kubearmor/KubeArmor/blob/main/examples/multiubuntu/build/Dockerfile

hI @rksharma95 ,
I've raised a PR #1878. Let me know if any changes are to be made.

hey @rksharma95 is this issue open ?

hey @rksharma95 is this issue open ?

Im working on it. Was a bit busy with exams :)

Are you still wotking on this issue?

Are you still wotking on this issue?

The workflow's done. I had a query that which is still unanswered on the PR. Hence wasn't able to proceed

Please see my comments if they are relevant to you!

Please see my comments if they are relevant to you!

Sure, thank you for the assistance:))


brew doctor warns about erdtree (1.8.1) not having an associated brew formula?

brew doctor
Warning: Some installed kegs have no formulae!
This means they were either deleted or installed manually.
You should find replacements for the following formulae:
erdtree

Are you using an M1 mac by chance?

Yes, I am. On 12 Apr 2023, at 17:37, Benji Nguyen @.***> wrote:
Are you using an M1 mac by chance?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@gbstan could I get the location of your et binary?
$ which et

Yes, sure. I am running homebrew via rosetta on my M1 mac:
which et
/usr/local/bin/et

The warning from brew doctor has now disappeared.
Strange.

Great! I'm going to close the issue for now but feel free to re-open if you come across it again.

I'm seeing the same error:
Warning: Some installed kegs have no formulae!
This means they were either deleted or installed manually.
You should find replacements for the following formulae:
  erdtree

I'm on a M1 mac, and when I run which et there is no output.


changed SetFavoritePokemonMessage pokemon_id to uint64

For some reason when the pokemon_id is set to fixed64, the API returns with the result of 2 which is ERROR_POKEMON_NOT_FOUND (File proof) when passed a Pokemon ID.  However, when set to uint64, it works.
Same type of idea here: https://github.com/tejado/pgoapi/issues/59 and https://github.com/PokemonGoF/PokemonGo-Bot/issues/452#issuecomment-234920567
Setting nickname does work, so no worries there.

Since this has already been changed in POGOProtos I just updated the proto files which took care of the issue. Thanks for the pointer!


Sort by block top, block left, top, then left

By #444, the sort keys were block top, block left, top, then bottom.
However, it should be block top, block left, top, then left as indicated by the following comment.

sort pdf_word_list by page, block top then block left, top, then left

This PR will fix this issue.
Actually, #444 should have failed. This PR needs a unit test that would fail on #444.

Codecov Report

Merging #449 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #449   +/-   ##
=======================================
  Coverage   83.20%   83.20%
=======================================
  Files          88       88
  Lines        4554     4554
  Branches      837      837
=======================================
  Hits         3789     3789
  Misses        572      572
  Partials      193      193

Flag
Coverage Δ

#unittests
83.20% <ø> (ø)

Impacted Files
Coverage Δ

src/fonduer/parser/visual_linker.py
83.57% <ø> (ø)

2ac486d demonstrates #444 does not sort pdf_word_list by left.

Can you resolve the conflict so we can merge?

I resolved the conflict. This PR is ready to be merged.
This PR is basically a rework of #444, where the sort keys were "block top, block left, top, then bottom" instead of "block top, block left, top, then left".
Hence I don't think we need to mention this in the CHANGELOG.


Updating and fixing issues with testing controllers article and sample

Fixes #1916 and #1847

@RickAndMSFT please review


[Bug]: Date Picker widget, Date Format dropdown introduces scrollbar in the body.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
In the property pane, when the Date Format property dropdown is opened, it adds a scrollbar to the body rather than the dropdown options area itself.
Steps To Reproduce

Environment
Production
Version
Cloud

Working fine on #7361


Update the source information

Type: Missing
Summary:
Update the source information
Details:
With the rise in supply chain attacks and OSS dependencies being used as a attack vector, Microsoft is working with our ecosystem partners, such as the Linux Foundation's OpenSSF, to enable OSS consumers to track packages back to their public sources.
Resolution:
This PR was created to add this value, ensuring future releases will include this provenance information.
Published NPM packages with repository information:
* minipass-pipeline
Affected definitions:

minipass-pipeline 1.2.4

Author's NPM profile points to this GitHub user account. Approved.


Reduce file descriptor usage

An instance of rita uses more than ten thousand file descriptors in it's normal state. That's crazy and needs to be reduced.
Just looking as lsof it seems to be mostly async/await related.

Some experimentation here shows that this is probably the number of independent async/await executors we have running in various threads. Separating out the executors has been useful for preventing unexpected blocking behavior from causing issues in unrelated parts of the program by hotting the executor thread but clearly is coming at a cost.


firefox: 86.0 -> 86.0.1

Motivation for this change
https://www.mozilla.org/en-US/firefox/86.0.1/releasenotes/
Only applies to staging-next, as the 86.0 upgrade is stuck in there.
Also this doesn't cause too many rebuilds, its only in there because it depends on a newer nss version.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

That was quick. How did your testing go? Mine isn't finished yet, but Firefox hangs on start and then crashes on Sway (Wayland).

I had a transitive dep fail and figured I'd just merge it and wait for staging to stabilize in the next couple of days to test. Can always revert before it hits master anyhow.
If you're not confident these changes are AOK I'm happy to open a reverting PR :)

Mhm, rebuilt as part of my system derivation and now I can't reproduce it anymore. Also ran the tests over night, they are working fine.


Random functions break when JAX_DISABLE_JIT=1

It seems when jax is disabled with JAX_DISABLE_JIT=1, the random functions don't work:
> jax.random.truncated_normal(key, -2., 2., (1,), jnp.float32)

gives
.../jax/random.py in _truncated_normal(key, lower, upper, shape, dtype)
    506     shape = lax.broadcast_shapes(lower.shape, upper.shape)
    507   else:
--> 508     _check_shape("truncated_normal", shape, lower.shape, upper.shape)
    509
    510   sqrt2 = onp.array(onp.sqrt(2), dtype)

AttributeError: 'float' object has no attribute 'shape'

I think we could use onp.shape(lower) and onp.shape(upper) there to avoid this. Though this also makes me realize that PRNG without jit will be pretty slow!

Thanks @mattjj !


Keep trying to connect on startup

Right now, the client attempts to connect at startup. If it can't, it gives up. Better behavior would be to continue retrying.

This is built into nes already.


[Bug]: Github file error when pulling from main branch

FAQ check

[X] Yes, I checked the FAQ and my question has not been addressed.

Instrument
Other (any stage)
What happened?
When fetching upstream from main branch to my forked repository using the Github desktop app, I get an error regarding a file in the tests/data/WFC3 (see below) folder.
Error traceback output
error: invalid path 'tests/data/WFC3/cal/leapdir/leap-seconds.3881174400?'
Updating d479abe..b4d1844
What operating system are you using?
Windows 11
What version of Python are you running?
No response
What Python packages do you have installed?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

So this was just on fetching? This file is a reference file for WFC3 testing code - I'm surprised it could cause an issue. Maybe it's that it doesn't have an acceptable file extension for Windows?

Oh wait, the issue is almost 100% because the filename ends in a question mark which the windows filesystem will get really angry about. Let me try to edit how that file gets saved to avoid that headache for Windows users. Out of curiosity, do you normally use windows for eureka? I was wondering how well Windows might work with Eureka

Ah I see. I assumed it was a Windows problem haha. Thanks so much!
I don't use Windows for Eureka! so can't comment sorry! I run Eureka! on the clusters at Warwick, but I don't like using Github on the terminal when it comes to looking at big commits and file changes, the Github desktop app better for visualising.


When rubyzip is used to modify a DOCX file, the resulting file can't be opened

Take a DOCX file (which is just a ZIP file with a different extension). Use Zip::ZipFile to remove one file from the archive, and replace it with an exact, identical one. The resulting file won't be openable by Microsoft Word, OpenOffice, etc.
require "zip/zipfilesystem"
f1 = File.binread("test.docx")
Zip::ZipFile.open("test.docx") { |z| s = z.file.read("word/document.xml"); z.file.delete("word/document.xml"); z.open("word/document.xml","w") { |f| f.write(s) } }
f2 = File.binread("test.docx")
f1 == f2   => false
f1.bytes.take(15)
=> [80, 75, 3, 4, 20, 0, 6, 0, 8, 0, 0, 0, 33, 0, 111]
f2.bytes.take(15)
=> [80, 75, 3, 4, 10, 0, 0, 0, 8, 0, 0, 0, 33, 0, 111]
After the ZIP file is read and written by rubyzip, a couple of bytes in the header are changed. I don't know enough about the ZIP format to understand why. But when I do the same thing with a standalone ZIP utility, it doesn't do that, and the resulting DOCX file can be opened properly.

I'm running into the same issue.  I have made a patch, but still doesn't fix the docx problem.  Here is my commit, maybe you could point me in the right direction. Thanks!
https://github.com/jondruse/rubyzip/commit/cbd2df07e5c2219ca4f2618fb66001f30ef1cb14

OK.
It's not error in version.
As I understand after recreating of any file in docx zip archive, some files stay brokens.
The main issue in different cdirOffset.
Sorry but I'm not familiar with zip format and need more time to understanding what is going wrong.

I'm have a very similar problem and I believe it's being caused from the same thing. I'm also using windows zip files.
As far as I can tell, the actual zip entries contained within the zip remain unchanged but the file header and the central directory are modified and something in there is causing the error.
I have a feeling it's being caused by the windows directory in the zip file and not the actual windows file itself. 2KB are being added to the total size of the file when the file is just opened and closed using rubyzip and this 2KB appears to be coming from the way rubyzip is repackaging the directory ZipEntry when it closes the file.
I used one of your test zip files 5entry.zip, opened and closed each of the files individually in notepad and resaved the zip file. At this point it works fine and can all be re-extracted properly. If I then open and close using rubyzip, the file becomes corrupt and is no longer extractable.
I wrote a script to pull apart the central directory record at the end of the file and here's what I got for before and after:
https://gist.github.com/1831665
The very first record is the 'end of central directory' record and then each record after that are the actual zip entries themselves. Alex is right, the relative offsets are all different, though the records just may have been written in a different order..  You'll notice small other small differences but the main difference I think is in that first zip entry referring to the directory. The file modification date has changed and the compressed size is now 2KB for some reason?
I'm still looking at this myself, but hoping the gist might help someone else find the error before me..

I was able to get docx files to work correctly by using write_buffer() instead of open().  Maybe that will help lead to the issue.  But my working example is here;
https://gist.github.com/e7d2855435654e1ebc52

Thanks Jon, your example works with my broken one also. Think I know where the issue is now.. 2 secs

I'm still having a problem getting docx files to open correctly after zipping them with RubyZip. Is this issue going to be fixed or are we left with jondruse's code as the only solution to the issue?

I was able to solve this using this code, though I ultimately took the approach of shelling zip out to the command line, as it was a more direct solution for my needs.

@tinabel I have been unable to replicate the issue with the code you posted above - i.e. if I remove the docx specific bit in put_into_archive and use your code to recursively zip a directory, I can unzip the generated archive and there are no differences between the original files and their zipped/unzipped counterparts. I assume I'm missing something; do you have an example docx file that triggers the issue?
Also which platform and version of rubyzip are you using?
Thanks.


[Not Working] Focus on element with tab key, activate click handler using enter key

I want to trigger the click event when navigating by using tab key and getting focus on a non clickable element, after which I want to use enter to activate the element. However the code below does not fire the click event, I also tried the returnKey event which does not work either.

 Login

How do I get this to work? Can you change your code to allow click binding on focusable elements?

click works on pretty much everything. Make a repro.

Expected
To be able to trigger click or returnKey event on non default focusable elements. Elements are made focusable by adding tabindex attribute
Result
Event is not triggered
Steps to reproduce

Hit tab until focus is on the wanted element
Hit enter/return key to activate element (expected to trigger click or returnKey)
Console should log Triggered login

<template name="form">

Expected

To be able to trigger click or returnKey event on non default focusable elements. Elements are made focusable by adding tabindex attribute

Result

Event is not triggered

Steps to reproduce

Hit tab until focus is on the wanted element

Hit enter/return key to activate element (expected to trigger click or returnKey)

Console should log Triggered login

	<form>
		<input type="text" name="email" placeholder="E-mail address" data-bind="value: email, focused: emailFocus">
		<input type="password" name="password" placeholder="Password" data-bind="value: password, returnKey: login, focused: passwordFocus">

Div below does not work when getting focus by using tab key and then hit enter key to perform "login"

Login

Input[type="button"] below performs "login" as expected

		<input type="button" data-bind="click: login, returnKey: login" value="Login" />
	</form>
</template>

if (Meteor.isClient) {
	Template.form.viewmodel(
		{
			email: '',
			password: '',
			emailFocus: true,
			passwordFocus: false,
			login: function() {
				console.log('Triggered login');
			}
		}
	);
}

Update to 1.9.14
Thanks


Destroy all objects in an schema/db when deleting users

When deleting an user, only objects known by CartoDB (i.e.: cartodbified tables) are deleted from the user db. If the user has created tables/views via the SQL API, they won't get deleted.

For normal users, that means that the database is not completely deleted (we leak databases).
For organizational users, the user cannot be deleted:

First time clicking the delete button: the user is deleted from central, credential are deleted from redis and the know objects are dropped. An error message (flash) is shown indicating that the users schema could not be deleted.
From here on, it gives a 500 error with no information. The user cannot access any API's to delete the objects (credentials deleted), so only solution is manual deletion from the db (via support ticket).

It might be worth it to create a lib/carto/user_database_manager.rb that would deal with all operations related to the user database. Then refactor other code to avoid the in_database stuff. Benefits would be:

Less fat user model
Centralised, one point of failure code for database stuff
clean?

Summoning @juanignaciosl

There' already 2 classes that handle that: DBQueries and DBService, both at app/models/user.

Still subject to some refactor, but ok, too verbose!


Add support for custom shipping setups and several packages

Hi,
We are running a Dokan setup with support for multi-vendor orders and multiple shipping packages. To make that work with Dintero we have had to make some changes to this great plugin:

Make it possible to disable shipping handling inside express checkout (we want to handle it completely on our end).
Add support for multiple shipping packages (not making the assumption its just one).
Add filter to make it possible to override shipping line items and options.

PS: Its not unlikely that there are some bugs lurking since we have only tested it for our use-case with embedded express checkout.

Thanks for the pull request, @voldern. Will pass it on to Krokedil for review.

Hi @voldern, thanks for this!
We have assigned @MichaelBengtsson to do a code review on this when he gets the time and will get back to you if we have any questions related to that. After that we will then do some more QA related testing to make sure that other functionality works as expected before we can merge it in.

Thank you for your swift feedback!
Looking more into the code when resolving issue #158 I see that there are more places where shipping is handled in the plugin, especially related to the orders portion of the code, so it will need some more changes to also handle the different checkout options that we are currently not using. I've also divided the added filters into two different kinds, but they might be possible to consolidate into one kind of filter or use a different distinction then what I have.

@voldern Thank you for the PR!
The code looks solid overall, except for the small things i pointed out in my review. But we would need to test it internally before we can merge this. The main concern i have is to make sure its compatible with other plugins that i know we worked with early in the plugins development for compatibility.
I would also want to make sure that the changes are applied everywhere its needed before merging this as well. As you stated in your last comment. I also see some things in our code originally that could be improved to make this change a bit more clean, with less complexity and more customization using filters. But we will need to look into that internally as well.

Hi @mntzrr, sorry for the late reply. I didn't get notifications for updates to this PR.
We have decided to change our setup to move away from multi-vendor support, so we should be able to switch to the mainline of the plugin instead. But if you still want to support this in the plugin then feel free to take over the changes and adjust.
I think the changes that you have suggested in this thread makes sense :+1:


Test MetaCoq 1.0.

NixOS/nixpkgs#183106.

@vbgl @siraben FYI we seem to have a build issue of SMTCoq again.
See https://github.com/coq-community/coq-nix-toolbox/actions/runs/3053476045/jobs/4924509541
Extract:
  CXX      regexp.lo
../../../../../src/util/regexp.cpp: In static member function 'static size_t CVC4::String::maxSize()':
../../../../../src/util/regexp.cpp:422:15: error: 'numeric_limits' is not a member of 'std'
  422 |   return std::numeric_limits<size_t>::max();
      |               ^~~~~~~~~~~~~~
../../../../../src/util/regexp.cpp:422:36: error: expected primary-expression before '>' token
  422 |   return std::numeric_limits<size_t>::max();
      |                                    ^
../../../../../src/util/regexp.cpp:422:42: error: no matching function for call to 'max()'
  422 |   return std::numeric_limits<size_t>::max();
      |                                     ~~~~~^~

Unfortunately, the fact that SMTCoq is stuck on Coq 8.13 and thus not in the default coqPackages probably means that it will be broken by changes in external dependencies without this being noticed when it happens. If CVC4 were packaged separately, this probably wouldn't happen as much. But this probably isn't a realistic thing to consider while SMTCoq is stuck on CVC4 1.6 (IIUC from https://github.com/smtcoq/smtcoq/issues/107#issuecomment-1146764924), which is four years old, while the latest CVC4 version is from last year and CVC4 itself is discontinued in favor of CVC5.

Superseded.


User Names being double escaped

html entities should not be written out

Merged into dev, considering fixed with introduction of LDAP auth.


LockVisualizer

About the PR
Splits EntityStorageVisualizerSystem into its own LockVisualizerSystem with LockVisualsComponent.
Besides, fixes deployable barrier deployed state not being shown when locked.
Why / Balance
Was required in TODO in EntityStorageVisualizerSystem.
Besides, I wanted to make GlassBox lock state work so I've faced with the problem of LockVisuals not being geneirc.
Technical details
Methods in systems that update the lock state were cleaned up in favour of using a more generic LockVisualizer in prototype.
Basically, LockVisualizerSystem is just copy-pasted part of EntityStorageVisualizerSystem with some fixes.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog
N/A

It's little messy in LockVisualizerSystem but any suggestion will be appreciated.
LockVisualsComponent besides StorageUnlocked has StateStorageUnlocked for storages that surely must have "unlocked" state.
But if you want to make an entity with unlocked state, you will have to put it manually in LockVisuals in prototype.

Will try to clean that mess


Update with latest nf-core/template

Description of the bug
Update with latest nf-core/template
Command used and terminal output
No response
Relevant files
No response
System information
No response

On me ;-)


Descrepency between what is reported by the json net_worth output and what is shown on the web page

net_worth reports a value that is $130k higher than what is reported by the actual web page
Any idea why that might be?

Can you please use the bug template to include your version of mintapi, the call you are using, and any screenshots to demo this? We don't have any information needed to debug this.

Sorry, version of mintapi is current (2.6)
Running on Chrome Version 103.0.5060.114 (Official Build) (arm64)
ChromeDriver 103.0.5060.53
Python 3.9.13
MacOS 12.4 (21F79)
json from mintapi:
{
"net_worth": 1707764.8900000001
}
Screen Grab:

A difference of 127,570.63
Nothing in my accounts seems to add up to that amount.

Sorry if this is a really dumb question, but you are definitely using the same account for the web vs mintapi, right? :D

@mdupuy my best advice for you is to export your account data and then perform a manual calculation similar to our handling: taking active accounts, you subtract the current balance of Loan and Credit Accounts from the current balance of all other accounts.  This is what your true Mint "Net Worth" would be.  You also may check the Trend of your Net Worth (which you can pull using MintAPI now) and see if you notice where any sort of "deviation" might come into play.


Nuxt Layer uno.config is included by itself in relative layers

UnoCSS version
0.61.0
Describe the bug
When trying to utilize the automatic merging of uno.config files in Nuxt layers, it appears the final layer file is also included in the mergeConfigs even when defining the export default as mergeConfig result. Therefore seems to be create a circular reference.
Reproduction
Create a Nuxt project with relative nested layers.
/base    // base layer
/app      // app layer, extends base layer
/app/label // final layer, extends app layer

each containing there own uno.config.ts.
Upon generating the .nuxt/uno.config.mjs, the included configs also include /app/label/uno.config.ts but that file is also used as the default config for unocss. The documentation states to include the generated file and use mergeConfigs to extend the generated config, but that won't work as the generated config already included the label config file.
#/app/label/uno.config.ts
import { mergeConfigs } from '@unocss/core'
import config from './.nuxt/uno.config.mjs' // /app/label/uno.config.ts is imported in .nuxt/uno.config.mjs now in a circular reference

export default mergeConfigs([config, {
  // your overrides
}])

The error message that is generating when starting Nuxt:
Cannot start nuxt:  Cannot convert undefined or null to object                                                                                14:02:36

  at Function.entries (<anonymous>)
  at node_modules\@unocss\core\dist\index.cjs:574:50
  at Array.map (<anonymous>)
  at mergeConfigs (node_modules\@unocss\core\dist\index.cjs:574:26)
  at C:/Work/Projects/my-project/app/label/uno.config.ts:16:26
  ....

System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

We temporarily closed this due to the lack of enough information. We could not identify whether it was a bug or a userland misconfiguration with the given info.
Please provide a minimal reproduction to reopen the issue.
Thanks.

Why reproduction is required

So I've been concocting a sandbox to reproduce this behaviour and understand it much clearer now. It's not needed to import the ./.nuxt/uno.config.mjs file but it does seem rules and preflights are not merged in configs and I do really think in Nuxt layers that merging rules and preflights would be beneficial.


Handle permission denied scenario

Overview of discussion from slack with @saschatimme:
When a socket connection is dropped elm-phoenix will keep attempting to reconnect (on a backoff schedule). When the connection is first dropped the onAbnormalClose subscription is triggered, subsequent failed attempts to reconnect however don't trigger onAbnormalClose. The intention was to prevent the app being spammed with onAbnormalClose msgs. In the following scenario this is problematic:

Socket connects successfully with a JWT token
Socket connection is dropped
Socket attempts to reconnect but JWT is now stale

(2) causes onAbnormalClose is triggered but (3) is considered the second abnormal close so onAbnormalClose is not triggered. Ideally it would be possible to trigger different subscriptions e.g. onNetworkDown or onPermissionDenied. However the websocket spec deliberately conceals the reason for the bad connection (see https://developer.mozilla.org/en-US/docs/Web/API/CloseEvent for a list of response codes). Frustratingly the http connection that the websocket uses does actually report a 403, but the websocket API doesn't allow access to this status code.
To resolve the stale JWT scenario above it's necessary to trigger the onAbnormalClose on every failed connection (the websocket spec doesn't distinguish between a dropped connection or a failed attempt to connect). This allows the app to check whether the JWT is fresh and to take action and refresh it if necessary.
This PR changes the behaviour for onClose, onNormalClose and onAbnormalClose to always trigger regardless of whether or not it's the first attempt (previously established by checking the current state of the socket).
To test the behaviour run the example and load it in the browser, kill the server then restart it. The console gives an overview of what's happening (we have discussed using Wallaby to add test coverage but will add those at a later date).

Looks great


[help] made a fork to mirror the screen (rather than the camera) but can't get the video stream to work.. I'm in over my head

Hi, I was talking with Izzy (from f-droid) the other day.. and we got to discussing screen mirroring: Google Cast vs. AirPlay vs. Miracast.. and how there isn't a good open-source app to stream it over RTSP. Of course, libstreaming immediately popped to my mind.. and I accepted the challenge to write one (quickly). I just pushed my repo, but it isn't quite finished. The app builds and everything looks to be working correctly.. but clients immediately disconnect after requesting the video stream, which makes me think it has something to do with how I'm piping the Surface (from a VirtualDisplay) to a MediaCodec and finally to the Packetizer; either the data isn't flowing, or the format isn't what the Packetizer is expecting.. honestly, I have no idea.
I was wondering if you'd be so kind to take a quick look. I'm guessing that you know this stuff backwards and forwards.. and could probably spot the problem with a glance.
I attached an apk that was built from the code in the initial commit, since there's no tagged releases yet. (The .zip file extension was added so github would allow it.)
RTSP-ScreenCaster-release.apk.zip
As far as the design goes, my changes focus mainly on the VideoStream and SessionBuilder classes.. removing anything having to do with the Camera, and adding some code to initialize a MediaProjection. Otherwise, it's mostly the same. This app corresponds to your example1. Once it's working, I plan to include another one that corresponds to your example3, but running as another foreground Service.
Anyway, any help would be greatly appreciated. And btw.. great library! ..really great code!! :smiley:

Oh, I should also mention.. that the VideoQuality attributes (width, height, dpi) are all obtained from DisplayMetrics that match the physical screen.

Nevermind.. I got it (mostly) working. The biggest issue now is that the Surface doesn't update anywhere close to 60 fps..