[lit] Stub out lit feature detection on non-Apple platforms

Quick for an oversight on my part that slipped through CI in my previous PR (https://github.com/apple/swift/pull/23455): the TSan runtime now has a link-time dependency on libdispatch (unfortunately).
This temporarily disables the following two tests on Linux to unbreak CI build.
Swift(linux-x86_64) :: Sanitizers/witness_table_lookup.swift
Swift(linux-x86_64) :: Sanitizers/tsan.swift
The above two tests are also the only 2 TSan (executable) tests that are run on Linux. Once I fix the TODO mentioned in this PR, I will go through all REQUIRES: tsan_runtime occurences and try to enable them by using libdispatch/foundation instead of objc_interop.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 50c82a5ac99cb5466c487f0d25db7b487d341f47

@swift-ci Please test Linux

Build failed
Swift Test Linux Platform
Git Sha - 50c82a5ac99cb5466c487f0d25db7b487d341f47

@swift-ci Please test macosx smoke

@swift-ci Please smoke test macosx

@swift-ci Please smoke test osx

@ jrose-apple: Are you okay with the updated patch? This should unblock https://github.com/apple/swift/pull/23508

I'm a little iffy on whether or not this will work because I'm not sure if we're consistent about "dispatch" meaning "Swift Dispatch" or "C Dispatch", but maybe it won't matter. It's okay for unblocking your other work, though, so go for it.

I'm not sure if we're consistent about "dispatch" meaning "Swift Dispatch" or "C Dispatch", but maybe it won't matter.

Good point. Here (the lit feature) means "Swift Dispatch", i.e., everything required for import Dispatch. "C Dispatch" is required for -sanitize=thread when linking. "Swift Dispatch" implies "C Dispatch" (but not the other way around). Ideally, these would all be separate things.

This broke the test suite on Windows!

Can you, uh, tell us how it broke the test suite on Windows?

apple/swift#23585 should repair the issue as a quick repair, looking into what it would take to properly detect this in apple/swift#23588

Can you, uh, tell us how it broke the test suite on Windows?

lit.py: C:\agent\_work\1\s\llvm\utils\lit\lit\TestingConfig.py:102: fatal: unable to parse config file 'C:/agent/_work/1/s/swift\\test\\lit.cfg', traceback: Traceback (most recent call last):
  File "C:\agent\_work\1\s\llvm\utils\lit\lit\TestingConfig.py", line 89, in load_from_path
    exec(compile(data, path, 'exec'), cfg_globals, None)
  File "C:/agent/_work/1/s/swift\test\lit.cfg", line 1503, in <module>
    config.available_features.remove('tsan_runtime')
KeyError: 'tsan_runtime'


Clean up mis-merge

Issues about the generator being a mess don't make much sense if the reporter leaves the code in the same sorry state.

Issues about the generator being a mess don't make much sense if the reporter leaves the code in the same sorry state.

To clarify my comments refer more to how the generator works, currently the generator mostly one monolithic function


wc: unexpected leading space when reporting two or more numbers for a given file

What happens now (using uutils wc):
$ echo "hello" > /tmp/test.txt && ./target/debug/coreutils wc -lc /tmp/test.txt
 1 6 /tmp/test.txt

(Notice the extra leading space in the output.)
What I expected (using GNU wc):
$ echo "hello" > /tmp/test.txt && wc -lc /tmp/test.txt
1 6 /tmp/test.txt

Which wc were you testing with?
I tried wc from coreutils 8.32 and got this output:
$ echo "hello" | wc -lc
      1       6

So the fix should be to add more whitespaces, not less?

For some reason, the output format when reading from stdin differs from the output format when reading from a file.
$ echo "hello" > /tmp/test.txt && wc -lc /tmp/test.txt
1 6 /tmp/test.txt
$ echo "hello" | wc -lc
      1       6

I was referring only to the output format when reading from a file.

By the way, I am testing on Ubuntu 18.04, with wc 8.28.


Update README.md

Remove last parm (email address) from google.auth.JWT() call. It causes "Error authorizing with JWT [Error: unauthorized_client]" and nothing here explains why so I believe it's an error.

It looks as if my changes haven't taken which is for the better. The issue comes down to this: If you're using an API that requires impersonation, you must provide an authorized account. If you don't need impersonation, you can omit that parameter.
If you do need impersonation for a service account, visit the following as a domain admin to authorize a given service account for one or more scopes: https://admin.google.com/AdminHome?chromeless=1#OGX:ManageOauthClients
I found that information very hard to come by because the need isn't clearly addressed and the provided error doesn't lead to the appropriate documentation. In my case, it was available (if outdated) at https://developers.google.com/admin-sdk/reports/v1/guides/delegation


SMART mode stops charging

When I charge in SMART mode and the total current on one phase goes (a little bit) over the limit I set. (25A) the charging current goes down as expected, which is great as it prevents the mains fuses to blow. :)
It starts going down from 16A to i.e. 11A and then 6A. Unfortunately it also ends up stopping the charge for a minute and then restart again. I do think the smartEVSE is overreacting a bit. I would prefer that it always goes to i.e. 6A mode and slowly increase from there if there is room for it. If 6A is still too much it should shut off charging of course.
Not sure if stopping the charge is an issue or if the algorythm reacts too fast. If it's the latter it would be nice to have a configuration setting in the menu for the algo: fast, normal, slow or an algo loop speed or something.
I update my P1 data using mqtt every 1 second so that should deliver the data fast enough?
version 1.80, 3 phases 16A, mains 25A (with a 1 / 3 phase switch on AUTO)

When less then MinCurrent is available, the SmartEVSE has no other choice then to stop charging...

You could try one of the 3.5 alpha releases floating around, it has several improvements....

One of the improvements is that it will only update the charge current if new data comes in.
Mind you, you should only upload "fresh" data through your MQTT interface; so if your P1 interface gives new data every 5s, only upload every 5s !!

Thanks for all the info: I will test and report back!

Thanks for all the info: I will test and report back!
@dingo35 No luck with Smart EVSE V3 Version: 15:31:52 @jan 8 2024 The display tells me access denied and I cannot control the charge mode. Can I build the latest myself maybe? (which branch)
edit 2: Moving back to a slower interval (10 seconds) seems to work quite well! I found that turning on the dishwasher can create a peak of 19A on one of the phases. As the minimal charge current is 6A (can't go lower?) there is no room for charging anymore. Therefore I now set my mains to 26A (small risk the fuse will blow one time... :P)

For the edit2
As @dingo35 said, stop charging under 6A is normal behavior.
Standard min charge is 6A but some cars (e.g. Tesla) allow 5A charge
I've rebuild v1.8.3 to allow min charging at 5A
If you need the source code changes, I'll provide you

There is no version 1.8.3
If your LCD screen says "access denied" you need to press the menu button or choose the mode on the webserver. Please RTFM :
https://github.com/dingo35/SmartEVSE-3.5

This was minly a user / configuration error so I will close this one

There is no version 1.8.3
If your LCD screen says "access denied" you need to press the menu button or choose the mode on the webserver. Please RTFM : https://github.com/dingo35/SmartEVSE-3.5

Sorry!
I have made a mistake. I wanted to say both v1.7.3 & v1.8.0


radoneye-scan.py bleak.exc.BleakDBusError: [org.bluez.Error.InvalidArguments]

Hi,
Just in case someone sees this : I'm trying to set up radoneye on a Raspberry Pi 3b.
Its running OS is not fresh and I have (not yet) no handy SD card to try on a fresh install.
I've installed python3.8 from sources, and pip3 install --requirements.txt work fine.
When running radoneye-scan.py I get immediately following error

  File "/home/somewhere/radoneye-reader/radoneye-venv-38/lib/python3.8/site-packages/bleak/backends/bluezdbus/utils.py", line 22, in assert_reply
    raise BleakDBusError(reply.error_name, reply.body)
bleak.exc.BleakDBusError: [org.bluez.Error.InvalidArguments] Invalid arguments in method call

By any chance would that ring any bell ?
radoneye-scan.py is pretty much the simplest script possible around BleakScanner.discover() so it's obviously not an issue with the script itself. But maybe you'll have an idea / already met a similar issue.
I set export BLEAK_LOGGING=1
and then I get nice output first that looks like this
bleak.backends.bluezdbus.manager MainThread DEBUG: initial properties: {'/org/bluez': {'org.freedesktop.DBus.Introspectable': {}, 'org.bluez.AgentManager1': {}, 'org.bluez.ProfileManager1': {}}, '/org/bluez/hci0': {'org.freedesktop.DBus.Introspectable': {}, 'org.bluez.Adapter1': {'Address': 'XX:XX:XX:XX:XX:XX', 'Name': 'SomeName', 'Alias': 'SomeAlias', 'Class': 0, 'Powered': True, 'Discoverable': False, 'DiscoverableTimeout': 180, 'Pairable': True, 'PairableTimeout': 0, 'Discovering': False,

etc...

thanks anyway for the job done here !
Additional info
Using Bluez 9.99-10
kernel 4.14.78-4
maybe that's all related.

works on an up to date Pi, so closing. Sorry for the noise.


cli: implement list --shared argument

Adds new --shared, --shared-by, and --shared-with arguments to the
list CLI command to display workflows along with their sharing
information.
Closes reanahub/reana-client#687

Latest version before rebase:  5a0c4e4

Ready to be reviewed, squashed and merged


Lowest Common Ancestor in Binary Tree in C

💥 Proposal
(A clear and concise description of what the proposal is.)
I would implement lowest common ancestor(LCA) in binary tree using C.
Lowest Common Ancestor is defined between two nodes node1 and node2 as the lowest node in tree that has both node1 and node2 as descendants(a node can be descendant of itself)
I am GSSoC'21 participant and would like to work on this issue.
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes

/assign


Switching between multiple PDFs causes crash

When loading and switching between multiple PDFs, the browser tab will crash after n times. This can be reproduced on the documentation page by clicking "Next PDF" approximately 30 times (I imagine this will vary depending on the computer).
Inspecting the memory usage shows a new Javascript VM instance being created every time the PDF is changed. Essentially it acts like a memory leak. Eventually it overwhelms the tab and crashes.
Additional context

I used PDFDocumentProxy.destory() to destroy the previous document instance and terminate the worker when source of pdf has changed.
Now, it seems that every JavaScript VM instance created before can be destroyed normally.
Before

After

see https://github.com/TaTo30/vue-pdf/pull/105


OSX: reduce threshold (getting a much more fluid scrolling)

I'm honestly not sure how scrolling is behaving for other platforms/operative systems, but it's been working a little oddly these past months for OSX, after some digging in my local I found that having a small value like 10 improves things a lot getting me a more fluid scrolling experience.
Note: I didn't have so much time to poke with this, so I don't know how your building system works, when tested in my local I just made the change to the minified file (/Applications/Oni.app/Contents/Resources/app/lib/browser/vendor.bundle.js), so please let me know if i'm missing to push any bundle or something else.
Thanks.

I'll give this a test when I'm home on my Windows Surface and Desktop, so we get both trackpad + trackpad events, since they are both treated differently.

@jaanauati sorry for the delay on trying this out been quite busy lately but will give this a go once I get a moment

Tested on Arch, looks good to me :+1:

I tested this with both a mouse and a trackpad on Max OS X High Sierra and it worked nicely.
Just a thought on the side: If we want to make this even smoother and more accurate, could we directly use the fontHeightInPixels as the scrolling threshold and then, instead of ScrollWheelDown/Up keys, send <C-E>/<C-Y> for scrolling only one line? Could this possibly create issues when users have remapped <C-E> or <C-Y> to something else?
Maybe we could even decide based on the _scrollDelta to send one-line(<C-E>/<C-Y>), three-line (<ScrollWheel*>), half-page (<C-D>/<C-U>) or full-page (<C-ScrollWheel*>) keys?
In any case, what you built so far already seems like a great improvement. Let's merge it once @CrossR was able to verify that it works on Windows :)

Tested this on my Surface and with an MX Master and I didn't notice any difference, so if its helping on Mac, thats great!


Should we use result is Ok<T> for returned type of isOkAndForResult?

If we do this, the below problem will happen:

declare const bar: Result<number, string>;
if (isOkAndForResult(bar, (v) => v > 0)) {
    bar; // This is `Ok<number>`
} else {
    bar; // This is narrowed as `Err<string>`, but this should also be `Ok<(num <= 0)>`.
}


how to remove the uninitialized warning?

./mshadow/mshadow/tensor.h: In function ‘void mshadow::MapExp(mshadow::TRValue<R, mshadow::gpu, dim, DType>*, const mshadow::expr::Exp<E, DType, etype>&) [with Saver = mshadow::sv::saveto; R = mshadow::Tensor<mshadow::gpu, 2, float>; int dim = 2; DType = float; E = mshadow::expr::BinaryMapExp<mshadow::op::minus, mshadow::expr::BinaryMapExp<mshadow::op::div, mshadow::Tensor<mshadow::gpu, 2, float>, mshadow::expr::BinaryMapExp<mshadow::op::plus, mshadow::Tensor<mshadow::gpu, 2, float>, mshadow::expr::ScalarExp<float>, float, 1>, float, 1>, mshadow::expr::BinaryMapExp<mshadow::op::div, mshadow::expr::BinaryMapExp<mshadow::op::minus, mshadow::expr::ScalarExp<float>, mshadow::Tensor<mshadow::gpu, 2, float>, float, 1>, mshadow::expr::BinaryMapExp<mshadow::op::plus, mshadow::expr::BinaryMapExp<mshadow::op::minus, mshadow::expr::ScalarExp<float>, mshadow::Tensor<mshadow::gpu, 2, float>, float, 1>, mshadow::expr::ScalarExp<float>, float, 1>, float, 1>, float, 1>; int etype = 1]’:
./mshadow/mshadow/tensor.h:67:1: warning: ‘shape2.mshadow::Shape<2>::shape_[1u]’ may be used uninitialized in this function [-Wmaybe-uninitialized]
       this->shape_[i] = s[i];
 ^
./mshadow/mshadow/././expr_engine-inl.h:361:13: note: ‘shape2’ was declared here
     Shape<dim> shape2 = ShapeCheck<dim, TB>::Check(t.rhs_);
             ^

the following code creates the up warning:
    tdata_out = F<mshadow_op::log>(tdata_in + MIN_NUM) * tlabel +
                (1.0f - tlabel) * F<mshadow_op::log>(1.0f - tdata_in + MIN_NUM);
    tdata_out = tdata_out * tmarklabel * -1;

I think the expression has no problem.
But how it to cause the warning?

Likely because some source of your tensor's shape was not intialized?

I don't think it can work when any tensor's shape was not initialized. isn't it?
tensor must have shape or cannot exist.

I don't think it can work when any tensor's shape was not initialized. isn't it?
tensor must have shape, otherwise it cannot exist.

@tqchen

@piiswrong

@vchuravy

Please send a minimal configuration that reproduces this warning
https://gist.github.com/

@starimpact The best way to get help is to paste something that others can reproduce, like a snippet that can be compiled  and produces the warning you mentioned. Otherwise, it is very hard for others to help you, as guessing the context of what might happened on your side, so hard for others to take actions, and reply to your issue.
This warning is quite clear, saying that there is an assignment, and source shape array of the assignment might not be initialized.  This is likely due to for example, shape array of t_data or t_label are not completely initialized. Check how did you created the t_data and t_label, if some shape content you used was not initialized.

For example, a better way on raising the issue could be something like
The following code gives the warning blabla, and there appears to be nothing wrong on this code, etc.
#include <mshadow/tensor.h>

int main() {
   mshadow::Tensor<cpu, 2> data = mshadow::NewTensor<cpu, 2>(Shape2(1, 2));
   mshadow::Tensor<cpu, 2> out = mshadow::NewTensor<cpu, 2>(Shape2(1, 2));
   data = 0.1f;
   float MIN_NUM = 3.0
    out = F<mshadow_op::log>(data + MIN_NUM) * data +
                (1.0f - data) * F<mshadow_op::log>(1.0f - data + MIN_NUM);
    return 0;
}

In this way, others can directly copy this snippet, reproduce the warning they had, and fix the potential problem. If you cannot get the snippet that reproduces the error, think about the difference between the snippet version and the code you have, is there any gap that causes the problems?
There are two types of issues that the developers can and will effectively answer

A snippet that is minimal and reproducible which indicates some problems
A question regard to the system, i.e. "can we read element data from GPU tensor"

The answer will be yes or no.

The question such as "I have this snippet of code that seems to be wrong, but I do not know why" is hard to take action, and usually will be ignored due to this reason

😄

i also find that when compile the newest offical mxnet(without any change), it will report this uninitialized warning.

fixed by https://github.com/dmlc/mxnet/pull/643


Starting a container is failing for Java based azure functions

ERROR - Container  for site  has exited, failing site start
Container  didn't respond to HTTP pings on port: 80, failing site start. See container logs for debugging
I am seeing this issue for all possible tries I did. By setting both PORT and/or WEBSITES_PORT in app setting. Not sure what is causing this error.
docker run -d -p 80:80 --name  -e WEBSITE_CORS_ALLOWED_ORIGINS= -e WEBSITE_CORS_SUPPORT_CREDENTIALS=False -e PORT=80 -e WEBSITES_ENABLE_APP_SERVICE_STORAGE=false -e WEBSITES_PORT=80 -e WEBSITE_SITE_NAME= -e WEBSITE_AUTH_ENABLED=False -e WEBSITE_ROLE_INSTANCE_ID=0 -e WEBSITE_HOSTNAME=.azurewebsites.net -e WEBSITE_INSTANCE_ID=4e34fd1dcf287fe6e10b93d54f0687cf933ff0e6150f35901216ccdf393ab071 -e HTTP_LOGGING_ENABLED=1

@mohantdurgan, Thanks for leveraging Azure docs feedback channel to raise this question. Could you please share the Azure documentation for which this feedback is applicable to?
--This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.

@mohantdurgan,  Just checking in to see if you have had a chance to see the previous response. Kindly let us know (share the doc URL) if you need further assistance  on this issue or is good to be closed.

mohantdurgan, Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply and we will gladly continue the discussion.


Can't stage changing in parent repository

Type: Bug
Steps:

setup root repository
setup in and child directory subdirectory which files also accessible to parent repository
change any file in child repository (it has to be appear in both repositories iin SCM tab)
click on "+" for this file in parent repository

Expected: staged changed file in parent repository
Actual: nothing. This file can be staged only in child repository
This bug appear in one of latest builds of VSCode. In 1.86 or near previous all was fine.
 Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

@SailorMax, thanks for the recording. It seems to me that you have 3 repositories that were discovered by VS Code. Are these repositories nested? What happens if you try to stage the file using the Terminal?

Do you mind sharing the exact folder structure and the exact location of the various .git folders? Also, is the file that you are staging actually part of two district git repositories? If so, how did you add the file to both repositories? Thanks!

Steps for 'test-repo/'-dir:

git init
touch foo.txt
mkdir foo-dir
touch foo-dir/foo2.txt
git add *
git commit -m 'init'
cd foo-dir
git init
git add *
git commit -m 'init'
open vscode with 'test-repo/'-dir
modify both files
open at SCM
try to stage changes of foo2.txt in 'test-repo'


fix(vue-app): static payload fetching bugfixes

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Fix regression with webpack in nuxt-edge as window.__NUXT_IMPORT__ gets undefined

There are two theories either webpack is wrongly tree-shaking top level window setters or using an internal other than globalThis for that condition either way exlicit install call seems fixing issue (also makes jsonp.js sideeffect-less)

Fix issue that _pagePayload is overwritten by payload prefetches (reported by @StephanGerbeth via https://github.com/nuxt/image/pull/136)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #8701 (43d4d56) into dev (845d7c0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #8701   +/-   ##
=======================================
  Coverage   66.05%   66.05%
=======================================
  Files          92       92
  Lines        4010     4010
  Branches     1104     1104
=======================================
  Hits         2649     2649
  Misses       1095     1095
  Partials      266      266

Flag
Coverage Δ

unittests
66.05% <ø> (ø)

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

Continue to review full report at Codecov.

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

Codecov Report

Merging #8701 (43d4d56) into dev (845d7c0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #8701   +/-   ##
=======================================
  Coverage   66.05%   66.05%
=======================================
  Files          92       92
  Lines        4010     4010
  Branches     1104     1104
=======================================
  Hits         2649     2649
  Misses       1095     1095
  Partials      266      266

Flag
Coverage Δ

unittests
66.05% <ø> (ø)

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

Continue to review full report at Codecov.

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


laravel/lumen Migration command issue

Laravel Lumen Version: 8
PHP Version: 8

Description:
Issue with this file vendor/illuminate/database/Console/Migrations/MigrateCommand.php
signature and description have type defined as String  where other many vendor files don't have type defined explicitly causing issue error
Type of Illuminate\Database\Console\Migrations\MigrateCommand::$signature must be string (as in class Illuminate\Console\Command) {"exception":"[object] (Symfony\\Component\\ErrorHandler\\Error\\FatalError(code: 0): Type of Illuminate\\Database\\Console\\Migrations\\MigrateCommand::$signature must be string (as in class Illuminate\\Console\\Command) at vendor/illuminate/database/Console/Migrations/MigrateCommand.php:11)
Steps To Reproduce:
Just add any command file for lumen
https://laravel.com/docs/8.x/artisan#command-structure

These files don't have this type implemented: https://github.com/laravel/framework/blob/8.x/src/Illuminate/Database/Console/Migrations/MigrateCommand.php

This is for lumen application

Lumen uses this command directly


Define accepted and failed properties as empty array

While using the count method on property failed, the value must implements Countable.
The method isAccepted uses the count method on $this->failed but without be sure this is an array.
Newer php versions might throw an exception.
EDIT:
this issue is already been fixed here: https://github.com/CMTelecom/messaging-php/pull/7
You can close this PR and merge the PR opened by Rojtjo last week.

Hej, the issue is resolved now with the Pull Request by Rojtjo #7 as you suggested :) Thanks


Nested rows discussion

I have Viewport.react.js which is:
render() {
        return (
            <div>
                {this.props.children}
                <Footer />
                <ModalRoot />
            </div>
        );
    }

And I have the next route config:
<Route component={require(`${pagePath}/Viewport.react`)}>
        <Route path='/suppliers' onEnter={onRouteEnter} component={require(`${pagePath}/SuppliersView.react`)} />
        <Route path='/suppliers/supplier/:supplierId' component={require(`${pagePath}/SupplierView.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId' component={require(`${pagePath}/ProductView.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId/version/:versionId' component={require(`${pagePath}/VersionView.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId/version/:versionId/screen/:screenId' component={require(`${pagePath}/ScreenDesigner.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId/version/:versionId/screen/:screenId/grid_detail_view/:gridId' component={require(`${pagePath}/GridPopupDesigner.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId/preview/version/:versionId/:destiny' component={require(`${pagePath}/PreviewView.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId/preview/version/:versionId/screen/:screenId' component={require(`${pagePath}/ScreenPreviewView.react`)} />
        <Route path='/suppliers/supplier/:supplierId/product/:productId/preview/version/:versionId/screen/:screenId/grid_detail_view/:gridId/:gridDetailViewId' component={require(`${pagePath}/ScreenPreviewView.react`)} />
        <Redirect from="/" to="/suppliers" />
    </Route>

All these nested routes are practically rendered inside Viewport.react and this is desired behavior. For example when I here /suppliers/supplier/:supplierId/product/:productId/version/:versionId/screen/:screenId I want to render ScreenDesigner.react page directly inside Viewport.react but preserve all metadata of all parents for this page. I DONT WANT TO CREATE supplier page inside suppliers then product page inside supplier and so on.
I haven't seen anywhere in docs and examples description of possibility to create route config in this manner. But I strongly believe that this approach is working and useful.
React-router still lets me to create route config like this but now (I was on 0.13.x react-router) I can hardly deal with such routes. I really appreciate your great job guys but this situation makes me crazy.

Use index routes, like
<Route path="suppliers">
  <IndexRoute component={Suppliers} />
  <Route path="supplier/:supplierId">
    <IndexRoute component={Supplier} />
    {/* ... */}
  </Route>
</Route>


Question about Reverse Huber Loss

Hi @iro-cp I'm taking a look on the paper and got confused on the section 3.2 (Loss Function)

Where c

Questions

c Will be a scalar related to 20% (1/5) max error on the batch correct?
When I need to select between L1 and Quadratic loss. More specifically the  condition |x| < c. Does the |x| need to be summed over all it's elements in order to compare with this scalar number "c"?

Example
Considering the vector |x| with a batch of 3 elements as ...

My worst error will be 6, so c will be 1/5*6
c = 1.2
So summing all elements

|x| will be 9, then on this case the quadratic part of the reversed Huber would be selected
Or we are completely wrong and we should calculate B(x) for each element of |x| on the batch and then sum all those losses and divide by the batch size?

Hey,
To answer your first question, the way we calculate c is correct like this (also in your example).
The second part of your example should be different though: Instead of summing up elements and then deciding whether to apply L1 or quadratic Loss, this decision actually happens per element.
However, the "elements" are now all the image pixels. So every sample in your batch corresponds to an image and every pixel of the image gets penalized by either term depending on whether its difference to the ground truth value is smaller or larger than c.
Given the differences in your example, you would apply L1 loss to the first element, and quadratic on the other two. Then sum up.

Hi @iro-cp thanks for the help.

Can you please tell me if my implementation of the berHu loss is correct:
def berhu_loss(labels, predictions, scope=None):
    if labels is None:
        raise ValueError("labels must not be None.")
    if predictions is None:
        raise ValueError("predictions must not be None.")
    with tf.name_scope(scope, "berhu_loss",
                        (predictions, labels)) as scope:
        predictions = tf.to_float(predictions)
        labels = tf.to_float(labels)

        # Make sure shape do match
        predictions.get_shape().assert_is_compatible_with(labels.get_shape())

        # Get absolute error for each pixel in batch
        abs_error = tf.abs(tf.subtract(predictions, labels), name='abs_error')

        # Calculate threshold c from max error
        c = 0.2 * tf.reduce_max(abs_error)

        # if, then, else
        berHu_loss = tf.where(abs_error <= c,
                       abs_error,
                      (tf.square(abs_error) + tf.square(c))/(2*c))

        loss = tf.reduce_sum(berHu_loss)

    return loss


Pod install issue.

Hi James,
Hopefully you are doing good, I was trying your VideoCore project unfortunately pod install is not working for SampleBrodcast project because some missing "boost" project link.
I would be thankful if you can fix this.
Also I got few questions, actually I am looking to play & publish rtmp streams to red5 server, do this project cover this all ?
Thanks,
Asif

There are compatibility issues with Red5.  See #186 for discussion about the cocoapods.


News App

Can I create a news app on Flask Using the news API available?

Sure ! Show your Hacktoberfest Creativity :tada:

@omrawal Any further updates on the project ?

I m very close to completion but this week I have my College Exams so is it okay if I complete and create a pull request on 9th Oct i.e. Saturday

@omrawal No issues. Just commit few changes at your free time once before 9th Oct and tag us here. So we can see project track on your fork itself. Take your time !


Followup questions have no submit button.

This leads so a confusing user experience since the initial form requires the user to submit.

#42

This is kind of fixed, as in, there's a button to make you feel better. But really the design of the site as a whole should probably be revisited in the future.


Note structure refactoring

This PR addresses remaining parts of #350.

What is missing here? Do you think someone else can take over for this week?

What is missing here? Do you think someone else can take over for this week?

I'll try to write down some goals I have for this later today/tomorrow.

Closing as stale.


Defining @action in class layout

What is a recommended way of adding @actions in the class pages?
I have my variables and derived variables initialised in the __init__ for full encapsulation but I am hitting a problem when trying to define an action
Have something like this:
class Page:
    some_service: Service

    variable: Variable
    derived_variable: DerivedVariable

    def __init__(self, some_service: Service):
        self.some_service = some_service

        self.variable = Variable(default="default")

        self.derived_variable = DerivedVariable(
            func=self.some_service.get_derived_variable,
            variables=[self.variable],
        )

    @action
    async def reset_variable(self, ctx: action.Ctx, resolving_stage, selected_transaction):
        await ctx.update(variable=self.variable, value="")

It doesnt like that there is a self in the function...
Should it be something like this instead:
variable = Variable(default="default")

@action
async def reset_variable(ctx: action.Ctx, resolving_stage, selected_transaction):
    await ctx.update(variable=self.variable, value="updated")

class Page:
    some_service: Service

    derived_variable: DerivedVariable

    def __init__(self, some_service: Service):
        self.some_service = some_service

        self.derived_variable = DerivedVariable(
            func=self.some_service.get_derived_variable,
            variables=[variable],
        )

Its a bit annoying that I have to keep Variable and DerivedVariable defined separetly

Hi @Vlados09, thank you for reporting this!
This is indeed a bug and we will look into fixing it, for the mean time you could write your page into functional form so that Variable and DerivedVariable can be kept together or do the approach you mentioned. Apologies for the inconvenience.


Incorrect sample reference in OAuth flow for Tabs

When I go to the Overview section of Use 3rd party Oauth IDP providers in Tabs, the code sample referenced there is of how to build a Bot.

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

ID: 6d5b8b0f-3da3-8642-6edf-dc6608a830cf
Version Independent ID: f93cdabd-f6ab-29d0-f3c9-78deb21ca521
Content: Enable authentication using third-party OAuth provider - Teams
Content Source: msteams-platform/tabs/how-to/authentication/auth-flow-tab.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@surbhigupta12 - Can you please help to update the code sample links for C# and nodejs as below.
In this section https://learn.microsoft.com/en-us/microsoftteams/platform/tabs/how-to/authentication/auth-flow-tab#code-sample,
Sample name - app complete auth
Description - This sample demos authentication in bot, tab and messaging extention with sso, facebook using user name and password.
C# - https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/main/samples/app-complete-auth/csharp
Node.js - https://github.com/OfficeDev/Microsoft-Teams-Samples/tree/main/samples/app-complete-auth/nodejs


Use true-case-path to fix re-entrancy issue on Windows

Fixes #1895
When using node-sass through multiple entry points (as can happen when using webpack) on Windows, the casing of the drive letter in the path can be inconsistent. Since Node is case-sensitive to a fault, this results in Node attempting to create a second instance of the native binding, which results in a Module did not self-register error.
The fix is to use true-case-path to normalize the binding path, to prevent unintended duplicate bindings from occurring.

Please remove the package-lock.json

package-lock.json is generated automatically by npm 5, so I've also added it to .gitignore

Thanks!

No problem. Is there a timeline for the next patch release? Currently this bug is blocking some development on my team.

@MLoughry  +1 for this annoying issue :)

Best estimate is some time this week


Fix validation of lambda aliases in task ARNs

You can specify a specific alias of a function from a step function to
enable the pinning of step function versions to specific versions of
Lambdas. This change adds this syntax to the task ARN regex.
Fixes #38
I've tested this change with the project that exhibited the issue in the first place and it seems to be working well. Let me know if you can think of a test case I've missed.

Shipped in 1.4.1, thanks for your contribution!


Expose test FTP server class

From g.rodola on July 16, 2013 22:44:26
See: https://groups.google.com/forum/#!topic/pyftpdlib/l7j-6UTGz7c Summary:
FTPd class in test/test_ftpd.py [1] should be exposed as a public API because
it's useful for testing FTP clients.

[1]
https://code.google.com/p/pyftpdlib/source/browse/trunk/test/test_ftpd.py?spec=svn1219&r=1219#211

Original issue: http://code.google.com/p/pyftpdlib/issues/detail?id=267

Is this fixed by https://github.com/giampaolo/pyftpdlib/blob/master/pyftpdlib/servers.py#L528 ThreadedFTPServer class)?


convertFromHTMLToContentBlocks has only support for h1, and h2. Missing other headers

At the moment convertFromHTMLToContentBlocks does not support other headers

Attached a PR for it on https://github.com/facebook/draft-js/pull/207

Thanks -- sorry about that!


Add Apache Kafka Kubernetes Add-on

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

What is the outcome that you are trying to reach?
Adding Kafka Operator available to use as Kafka add-on
Describe the solution you would like
Use Strimzi Operator as a Kubernetes add-on to be used to install Kafka Clusters
Describe alternatives you have considered
The first Alternative is to install Confluent for Kubernetes, but favored the opensource solution for the first step and we can add confluent later. Also confluent comes with more components that may not be required.
We discussed adding the add-on in the DoEKS repo, but preferred to keep the add-on (operator) here and the complete solution to be implemented in DoEKS repo.
Additional context
This is a per-requisite step to this issue : https://github.com/awslabs/data-on-eks/issues/22

@vara-bonthu


Very large static files yield very low performance

I am attempting to virtualize a service that normally returns very large payloads (in excess of 30 MB).  The original server can return this data in under 3 seconds on average (also a Java service) but WireMock will take over 25 seconds. I understand this may not be a typical use-case for the tool but considering some of the horrible alternatives out there I'd much rather stick to using this.
If I am following the code correctly this line here would cause the entire file to be read into memory before it would ultimately be sent over the wire in chunks: https://github.com/tomakehurst/wiremock/blob/cc49a1848e262cdbc328020715a0d815c37d4393/src/main/java/com/github/tomakehurst/wiremock/standalone/RemoteMappingsLoader.java#L67

Can you provide a bit more detail about your setup? It would help to know:

Are you using the latest WireMock version?
How are you running it (CLI parameters + environment)?
Have you disabled chunked encoding?
Your stub mapping JSON

BTW, the code above is just for publishing to a remote WireMock instance so it shouldn't get executed in the normal run of serving responses. This is where body files actually get sent in the response: https://github.com/tomakehurst/wiremock/blob/master/src/main/java/com/github/tomakehurst/wiremock/servlet/WireMockHandlerDispatchingServlet.java#L223


Projectile appears not to clear out project paths that no longer exist?

E.g., if you move a project to a new directory, projectile won't remove it from tab completion. This seems that it could be checked periodically?
How best to address without adding large overhead to startup ?

Projectile has a command for removing nonexistent projects. If you don't want to run it manually, I would recommend investigating package midnight, which is bundled with Emacs. It's designed for this sort of thing (running commands periodically).

This appears to be a good solution. Maybe I'll look into it next week.


Make species number 1-based in list file

It would be better for biologist users if the species numbers start at 1 and not zero.

Subtask of issue #22 https://github.com/eeg-ebe/HaplowebMaker/issues/22


Add PATCH support Glossary and GlossaryTerms

Feature
#2517
Describe the task
This adds support PATCH APIs and tests for Glossary and Glossary terms

Reopening to fix issues


HDDS-7362. Bring back the mechanism to share LRU cache between RocksDB in datanodes

What changes were proposed in this pull request?
This PR brings back the mechanism to share LRU cache between RocksDB in datanodes
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7362
How was this patch tested?
Standard CI.

hmm just for myself learning~ when is the timing a DatanodeDBProfile object being created?

@lokeshj1703 @ChenSammi As mentioned in the original discussion, please have a look at this PR.

Hi @ChenSammi, If this PR can reduce the problem of too many LRUCache objects in DN. I think we can cherry-pick it to branch-1.3.  If there is no problem I will turn off revert PR HDDS-7442.

Thanks @duongkame for the fix. The change make sense. In addition, Did you check the dump of DN? Just make sure if there are still many LRUCache objects.

Yes, I did. Please see below:
duong@fubar ~ % jmap -histo:live 81172 | grep Managed
 144:           104           7488  org.apache.hadoop.hdds.utils.db.managed.ManagedColumnFamilyOptions
 388:             4           1056  org.apache.ratis.thirdparty.io.grpc.internal.ManagedChannelImpl
 400:            21           1008  org.apache.hadoop.hdds.utils.db.managed.ManagedDBOptions
 548:             4            544  org.apache.hadoop.hdds.utils.db.managed.ManagedBlockBasedTableConfig
 572:            21            504  org.apache.hadoop.hdds.utils.db.managed.ManagedCheckpoint
 573:            21            504  org.apache.hadoop.hdds.utils.db.managed.ManagedRocksDB
 574:            21            504  org.apache.hadoop.hdds.utils.db.managed.ManagedWriteOptions
1340:             4             96  org.apache.hadoop.hdds.utils.db.managed.ManagedBloomFilter
1341:             4             96  org.apache.hadoop.hdds.utils.db.managed.ManagedLRUCache
1398:             4             96  org.apache.ratis.thirdparty.io.grpc.internal.ManagedChannelImpl$1ChannelCallTracerFactory

LGTM + 1.  Thanks @duongkame .


Authorization branch

[x] install the cancancan gem
[x] add the ability
[x] add the authorization
[x] modify the application controller
[x] add the validation
[x] add the destroy method for food and recipes
[x] modify the view for the food and recipes

Hi @soesandarwin2201
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

No Linters errors 💯
All requirements are complete 💯

Optional suggestions
Kindly check comment belov
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Reading and writing to/from the pty

Reading from the pty has to be blocking and threaded so that writing can still happen. I'm not entirely sure which of the pty's threads are blocked or locked or anything, but this seems to do the trick.
Just a heads up that I had an issue testing with my usual starship prompt - maybe there's funky utf-16 codes that aren't properly translated?

Great work so far. Didn't have the chance to look at it on my pc yet, but I think i can get to that over the weekend.

This isn't quite right yet as it's still going through the main loop. The read needs to be blocking to be awaited properly, but since the pty is in a mutex it has to be locked on the read.

@iXialumy I'm continuing on other work to get it feature complete. Should I just keep making PRs into this repo?

@ulyssesdotcodes yes please. If you want to update a PR incrementally until completion you could also declare it as a draft pr. Otherwise I will just merge your changes somewhat regularly :)


1388 rename items to lines

I want to merge this change because...
Fixies #1388
Pull Request Checklist

[x] Privileged views and APIs are guarded by proper permission checks.
[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Database queries are optimized and the number of queries is constant.
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Python code quality checks pass: pycodestyle, pydocstyle, pylint.
[x] JavaScript code quality checks pass: eslint.

Codecov Report

Merging #1414 into master will increase coverage by 0.23%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master    #1414      +/-   ##
==========================================
+ Coverage    82.9%   83.14%   +0.23%
==========================================
  Files         128      129       +1
  Lines        5775     5855      +80
  Branches      663      665       +2
==========================================
+ Hits         4788     4868      +80
- Misses        830      831       +1
+ Partials      157      156       -1

Impacted Files
Coverage Δ

saleor/dashboard/order/utils.py
100% <ø> (ø)
:arrow_up:

saleor/userprofile/views.py
36.11% <ø> (ø)
:arrow_up:

saleor/dashboard/views.py
53.12% <ø> (ø)
:arrow_up:

saleor/dashboard/order/views.py
56.95% <ø> (ø)
:arrow_up:

saleor/dashboard/order/forms.py
73.39% <0%> (ø)
:arrow_up:

saleor/order/models.py
88.62% <100%> (-0.4%)
:arrow_down:

saleor/dashboard/customer/filters.py
100% <0%> (ø)
:arrow_up:

saleor/dashboard/shipping/filters.py
100% <0%> (ø)
:arrow_up:

saleor/dashboard/order/filters.py
100% <0%> (ø)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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


App Mockup

User Story
As a Developer, I would like a mockup so that it serves a reference for frontend development.
Detailed Description
The mockup includes the main pages of the app, designed in Figma
Mockup Design on Figma
Acceptance Criteria

[x] The Figma mockup includes all key pages (e.g., Homepage, Workouts, Meals, Dashboard)
[ ] Mockup is reviewed and and ready for front end implementation
[x] The Figma link is added to the issue for easy access

Looks good, very detailed.


Add CompliancePST from DEQ/DERR

Summary

Proposed Readable Name: Utah Petroleum Storage Tanks Compliance
Proposed Internal Dataset Name: CompliancePST
Proposed External Dataset Name: petroleum_storage_tanks_compliance
Proposed Data Category: Environment
Data Description: A table related to FactilityPST that shows compliance data.
Source Agency/Entity: DEQ/DERR
Contact: Rick Saathoff
Update Frequency: Nightly via DEQ Enviro pallet

This is a replacement for DEQMAP_TankUST that adds above-ground tank data. This table is related to FacilityUST.
The data should be available in
1 Check [x] all the areas where you expect the data to show up.

[x] Internal SGID
[x] Open SGID
[x] ArcGIS Online
[x] Open Data

The data is of high quality

[ ] Sweeper checks have run and passed (@stdavis on 2022/00/00)
[ ] The minimum requirements for metadata are populated (@stdavis on 2022/00/00)
[ ] The data complies with our domain rules (@stdavis on 2022/00/00)

Where is the data source
Choose one.

[ ] Internal SGID
[ ] Farm from AGOL (agol item url)
[x] Farm from DEQ Database via enviro pallet

Action items

Assign a person who should complete the task by replacing name with their github @name.
Check [x] the box when the task is completed and add the date of completion.
~Strike~ out all items that do not apply.

[ ] Add data to the Internal SGID (name, completed: 2022/00/00)
~- [ ] Configure forklift for Farm from AGOL (name, completed: 2022/00/00)~
[ ] Update relevant gis.utah.gov data pages (name, completed: 2022/00/00)
[ ] Add a Stewardship record (name, completed: 2022/00/00)
[ ] Complete a SGID.META.AGOLItems record (@stdavis, completed: 2022/00/00)
~- [ ] Complete an AGOLItems_shelved record for any static or shelved item (name, completed: 2022/00/00)~

:robot: Automation validation

Assign yourself or someone to check the item by replacing name with their github @name.
Check [x] the box and add the date of verification 2020/01/01 when the task is verified.
~Strike~ out all items that do not apply.

[ ] Open SGID via cloudb (@steveoh on 2022/00/00)
[ ] ArcGIS Online via forklift (@stdavis on 2022/00/00)
[ ] Auditor ran successfully (@jacobdadams on 2022/00/00)
[ ] Open Data (@stdavis on 2022/00/00)
[ ] gis.utah.gov data pages (@stdavis on 2022/00/00)
[ ] sgid-index (@steveoh on 2022/00/00)

Notification

[x] Twitter (@steveoh)

Group Task Assignments

Check [x] the box when you have assigned all the tasks relevant to your group.

[ ] Data Team (@gregbunce)
[x] Dev Team (@steveoh)
[ ] Cadastre Team (@rkelson)

scott, are you comfortable doing the data pages and the stewardship sheet for these replacement layers or would you like me to handle that?

conductor results for Environment.CompliancePST

check
status

internal sgid
:no_entry:

meta table

- item id
:no_entry:

- item name
:no_entry:

- geometry type
:no_entry:

stewardship
:no_entry: Did not find Environment.CompliancePST in the worksheet

conductor results for tasks - 185

check
status

@stdavis has completed 0 out of 7 tasks
:no_entry:

@rkelson has completed 0 out of 3 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

@jacobdadams has completed 0 out of 1 tasks
:no_entry:

gregbunce has completed 1 out of 1 tasks
:+1:

The client has approved the data and we are good to add it to SGID. I'm still waiting on metadata. I asked Rick Saathoff to help me with it.
Here's the data:
CompliancePST.gdb.zip

scott, are you comfortable doing the data pages and the stewardship sheet for these replacement layers or would you like me to handle that?

@gregbunce Same answer as the other issue. I'd love the help if you have the bandwidth

@ZachBeck or @MichaelAGRC Would you mind loading this data (.zip is in a previous comment) into internal and granting it the appropriate permissions? I can add the metadata later.

@stdavis this is loaded now.

conductor results for tasks - 185

check
status

@stdavis has completed 1 out of 7 tasks
:no_entry:

ZachBeck has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 3 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

@jacobdadams has completed 0 out of 1 tasks
:no_entry:

gregbunce has completed 1 out of 1 tasks
:+1:

conductor results for Environment.CompliancePST

check
status

internal sgid
:+1:

meta table

- item id
:no_entry:

- item name
:no_entry:

- geometry type
:no_entry:

stewardship
:no_entry: Did not find Environment.CompliancePST in the worksheet

conductor results for tasks - 185

check
status

@stdavis has completed 4 out of 7 tasks
:no_entry:

ZachBeck has completed 1 out of 1 tasks
:+1:

@rkelson has completed 0 out of 3 tasks
:no_entry:

@steveoh has completed 2 out of 4 tasks
:no_entry:

jacobdadams has completed 1 out of 1 tasks
:+1:

gregbunce has completed 1 out of 1 tasks
:+1:

conductor results for Environment.CompliancePST

check
status

internal sgid
:+1:

meta table

- item id
:+1:

- item name
:+1:

- geometry type
:+1:

open sgid
:+1:

open data
:+1:

arcgis online
:+1:

stewardship
:no_entry: Did not find Environment.CompliancePST in the worksheet

this is a table that does not need data page or stewardship changes

Great. Scott, will you close this if it is complete.


Moved the metadata into setup.cfg

Whats the difference with #113?

Whats the difference with #113?

Removed everything related to generation of cmaps. And removed explicit parsing - setup() does it itself.
The problem of setup.py is that it is executable code of arbitrary structure. setup.cfg is a metadata file, not code. So putting metadata into a separate file with rigid structure is beneficial. Both for automated tools and humans. Though, as I have said, neither GitHub, nor bandit, nor dependencies.io have support for these files, even though the parser for its format is readily available. Though they don't have support to setup.py too, because in order to extract data from setup.py one has to execute it, and it is a security issue. Of course some heuristics using ast.parse is possible, but IMHO makes no sense.

I'm not really sure why we should do it if there is no real benefit.
Reasons not to do it:

might not work on python2 (SO)
I think more people are familiar with setup.py that there are with setup.cfg. Might be wrong on this.
might introduce new errors, like the shebang that is hardcoded for python3

I might be a bit conservative here, but I don't see the point of changing nicely formatted working code if there is also weirdly formatted broken code in the rest of the repo. Lets focus on that first and close this PR.

might not work on python2 (SO)

he had used distutil

I think more people are familiar with setup.py that there are with setup.cfg

familiarity is not needed, the format is trivial and intuitively undestandable. It mirrors setup args.

might introduce new errors, like the shebang that is hardcoded for python3

isn't shebang a comment, so when called by python 2, it is 8gnored.

Any progress merging this?

I suggest we not merge this PR and close it since it does not solve any problem (as far as I can see).
@KOLANICH, please make sure in the future that you open an issue to discuss a bug/problem/enhancement/feature before creating a PR.

I'm closing this because I don't see the need of using a setup.cfg.
Also apparently it was a mandatory file earlier, but not anymore.

I don't see the need*

is a bulletproof argument.


standard makefile,fix not work no  macos

on macos , make binaries not compile contained

Could you please sign off your commits (try git commit -s) in order to pass CI checks?
https://github.com/containerd/project/blob/master/CONTRIBUTING.md#sign-your-work

Could you please sign off your commits (try git commit -s) in order to pass CI checks?
https://github.com/containerd/project/blob/master/CONTRIBUTING.md#sign-your-work

tks, already signed off.

tks, already signed off.

irrelevant failed test
Error:         losetup_test.go:96: assertion failed: expected [/dev/loop3] (length 1) to have length 0

irrelevant failed test
Error:         losetup_test.go:96: assertion failed: expected [/dev/loop3] (length 1) to have length 0

I know, tks for you explanation!

I know, tks for you explanation!


Progress is broken

Progress bar goes over 100% and it's not accurate.
This problem will be fixed soon...

fixed!


Steinbock container keeps exiting

When I try to open the container in docker it says running then exits immediately (less than 1 second). Could you help me solve this issue?

Hi @cuiriouscancerscientist, thanks for reaching out.
What exactly do you mean by "open the container in docker"? Is there a specific command you are running?

Thank you, when I click "run" it will say running then exit immediately within a second.

Please note that the steinbock Docker container is operated using the command line. Steinbock does not ship with a graphical user interface, nor does it provide a "run" button of some sort.

I was trying to run it in docker. After I have the Steinbock container in docker I thought I had to click run to use the CLI in docker to use the steinbock pipeline. If that is incorrect could you explain what I should be doing instead, please?

Ah, now I was able follow, thanks for the clarifications! To run the steinbock Docker container, please open the command prompt (Windows) or shell (Linux/MacOS) and follow the installation instructions. For example, on MacOS/Linux, after installing prerequisites such as Docker, you would start by configuring a steinbock alias by typing the respective alias... command. The Docker Desktop user interface is not required to run steinbock.

Hello again,
I have followed the installation instructions however when I get to this command line "docker run -v "C:\Data":/data ghcr.io/bodenmillergroup/steinbock:0.15.0." I get the help text.
More background: Following the steps I was able to successfully download docker and it seems to be working. I have the steinbock container on docker, which exits a second after I hit run. I have tried other containers e.g. alpine and it seems to run fine on docker. Once I am up to the aforementioned command line I have not been able to move beyond this step.
I am new to this so thank you for your patients.

I have followed the installation instructions however when I get to this command line "docker run -v "C:\Data":/data ghcr.io/bodenmillergroup/steinbock:0.15.0." I get the help text.

This is perfectly normal and expected, since you do not specify a command (available commands are shown in the help text). Please follow the installation instructions to set up a steinbock command alias for the current session using doskey. As noted in the usage instructions, this is a prerequisite for running the commands as documented online and in the help text.

Thank you!


Message Attribute is not getting serialised in the message object.

Describe the bug
The Message Object contains a member called Message Attribute, which should contain the SQS Message attributes.
Expected behaviour
The Message Object from the JustSaying Models should contain a Message Attribute.
Actual behaviour
The Message Object from the JustSaying Models does not contain Message Attribute.
System information:

OS: MacOS 11
Library Version 6.0.5
.NET version 5.0

Tracking documentation for this feature in #944.


delay reloadData to fix overscrolling

Description
When changing the tab and scroll up to top - by tapping the selected tab again - the contentOffset gets broken that leads to an overscrolling (big empty box on top). This happens because of the requested reloadData - that will calculate a new contentSize.
Fix at the moment: first scroll on top, then reloadData. To achieve that we delay the reloading of data a bit.
Link to Jira

Screenshots
https://user-images.githubusercontent.com/2675578/158641091-c4b4adcb-fc4a-471f-b7e1-67cd1c111d3a.mov

@KaiTeuber & @marcussc
FYI: This issue is not completely fixed, if a certificate is deleted and then the "Certificates" tab bar icon is tapped, the tab still over scrolls.


update test-sampleMetadata

Why are you submitting this pull request
This pull request addresses the creation of unit tests for study metadata/combined metadata methods.
Is this pull request related to a bug report
No.
Is this pull request related to a feature request
Yes. This pull request is related to issue #219.
Additional context
First draft unit tests checking that:

All studies from curatedMetagenomicData (in the metadata.csv file) are present in sampleMetadata.
All of the required columns are present in sampleMetadata.
sampleMetadata and combined_metadata are the same.

The combined_metadata file was updated by running the script combined_metadata.R

I added this unit test in 7a2a2c3, will close for now and we can discuss at the meeting tomorrow.


This module of yours is very difficult to use, and it cannot be run by running a simple example on the official website. Please give the developers some help, otherwise don't do it

from pywinauto.application import Application
app = Application(backend="win32").start('notepad.exe')
print(app.Properties.print_control_identifiers())
output：
Traceback (most recent call last):
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\application.py", line 248, in __resolve_control
ctrl = wait_until_passes(
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\timings.py", line 458, in wait_until_passes
raise err
pywinauto.timings.TimeoutError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\sheng\PycharmProjects\quantitative_transaction\pywinauto_api\pa_2.py", line 20, in
print(app.Properties.print_control_identifiers())
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\application.py", line 585, in print_control_identifiers
this_ctrl = self.__resolve_control(self.criteria)[-1]
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\application.py", line 259, in __resolve_control
raise e.original_exception
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\timings.py", line 436, in wait_until_passes
func_val = func(*args, **kwargs)
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\application.py", line 201, in __get_ctrl
dialog = self.backend.generic_wrapper_class(findwindows.find_element(**criteria[0]))
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\findwindows.py", line 84, in find_element
elements = find_elements(**kwargs)
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\findwindows.py", line 303, in find_elements
elements = findbestmatch.find_best_control_matches(best_match, wrapped_elems)
File "C:\Users\sheng\PycharmVenv\quantitative_transaction\lib\site-packages\pywinauto\findbestmatch.py", line 533, in find_best_control_matches
raise MatchError(items = name_control_map.keys(), tofind = search_text)
pywinauto.findbestmatch.MatchError: Could not find 'Properties' in 'dict_keys(['Notepad', '无标题 - 记事本', '无标题 - 记事本Notepad'])'

If you cannot determine the "problem" here, you really shouldn't dabble with programming.

I have successfully run the program through other methods, and have viewed various types of information, not your garbage sample code! Could you please update it locally? You can write several pages of bugs in this module, okay?

You should solve the 421 problem first, and then talk about other things

https://www.youtube.com/watch?v=uTLsI2asox4

If you cannot determine the "problem" here, you really shouldn't dabble with programming.

While I don't agree with author, your approach (basically gatekeeping) doesn't really help. You weren't born genius to understand programming and how-to debug code from young age. There is never a place for comment like this in open source community.

@haisheng666 First the code you've tried is not exactly the same as in README examples. It's a combination of codes from 2 different code samples. Copy-pasting any piece of code without understanding is not always a good idea.
Second the error message is pretty clear: Could not find 'Properties' in 'dict_keys(['Notepad', '无标题 - 记事本', '无标题 - 记事本Notepad'])' which means that the caption of main window is '无标题 - 记事本', '无标题 - 记事本Notepad' while you expect "Properties" or something similar to it. Of course, it's not equal to the real window title. Reading Python traceback carefully is not very hard skill. If you're new to Python, maybe it's worth learning some basic Python course.
Also the Getting Started Guide is among recommended links to read first. Maybe this guide is not the easiest guide in the world, but GUI automation is also not the easiest area to learn as timing issues are pretty common, for example. pywinauto provides many APIs for handling such issues, and we work on simplifying it during our free time.
All, please be more patient and calm. I'd prefer to see only technical and professional discussions.


0.4.32

js-sdsl 4.1.0 not working properly -> downgraded to 4.0.3

Codecov Report

Merging #44 (2c7057d) into main (2e59571) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   41.29%   41.29%
=======================================
  Files         113      113
  Lines        9578     9578
  Branches     1448     1448
=======================================
  Hits         3955     3955
  Misses       5195     5195
  Partials      428      428

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Change cap. RequiresMACAddress to true

Description
Related to https://github.com/projectcalico/calicoctl/issues/1817 .
libnetwork: GET /IpamDriver.GetCapabilities
{"RequiresMACAddress":false}
instructs docker do not change MAC address inside container.
So at container inspect it's empty:
"MacAddress": ""
I can see calico netplugin response with MAC provided, but seems that Docker ignores MacAddress field:
libnetwork-plugin[32378]: time="2018-04-26T13:51:40Z" level=info msg="CreateEndpoint response" JSON="{"Interface":{"Address":"","AddressIPv6":"","MacAddress":"EE:EE:EE:EE:EE:EE"}}"``

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: caseydavenport:white_check_mark: djlwilder:white_check_mark: tomdee:x: CharlieR-o-o-tYou have signed the CLA already but the status is still pending? Let us recheck it.


As a dev, I want company fields to be correctly persisted during analysis registration

Description
Context
While running a test script that registers an analysis, I noticed that the company_website field (and possibly other company fields) is missing from the result despite being present in the input analysis_json.
Test code:
import asyncio
import json
import os

from azure.storage.blob import BlobServiceClient

import datastore
import datastore.db as db
import datastore.fertiscan as fertiscan

# Environment variables setup
BLOB_CONNECTION_STRING = os.environ["FERTISCAN_STORAGE_URL"]
DB_CONNECTION_STRING = os.environ.get("FERTISCAN_DB_URL")
DB_SCHEMA = os.environ.get("FERTISCAN_SCHEMA_TESTING")

# Define the analysis data as a Python dictionary
analysis_json = {
    "company_name": None,
    "company_address": None,
    "company_website": "lesgranulaines.com",
    "company_phone_number": None,
    "manufacturer_name": None,
    "manufacturer_address": None,
    "manufacturer_website": None,
    "manufacturer_phone_number": None,
    "fertiliser_name": "FERTILISANT NATUREL TOUT USAGE / ALL-PURPOSE NATURAL FERTILISER",
    "registration_number": None,
    "lot_number": "L-1",
    "weight": [],
    "density": None,
    "volume": None,
    "npk": "10-0-4",
    "guaranteed_analysis_en": {
        "title": "Guaranteed minimum analysis",
        "nutrients": [
            {"nutrient": "Total nitrogen (N)", "value": 10.0, "unit": "%"},
            {"nutrient": "Soluble potash (K2O)", "value": 0.4, "unit": "%"},
            {"nutrient": "Organic matter", "value": 61.5, "unit": "%"},
            {"nutrient": "Maximum moisture content", "value": 14.8, "unit": "%"},
        ],
    },
    "guaranteed_analysis_fr": {
        "title": "Analyse minimum garantie",
        "nutrients": [
            {"nutrient": "Azote totale", "value": 10.0, "unit": "%"},
            {"nutrient": "Potasse soluble", "value": 0.4, "unit": "%"},
            {"nutrient": "Matière organique", "value": 61.5, "unit": "%"},
            {"nutrient": "Teneur maximum en humidité", "value": 14.8, "unit": "%"},
        ],
    },
    "cautions_en": ["Wash your hands after use.", "Keep away from children."],
    "cautions_fr": [
        "Se laver les mains après chaque utilisation.",
        "Garder hors de la portée des enfants.",
    ],
    "instructions_en": [
        "Garden: incorporate 150 to 200 grams of Granu-Laines per m2 into first top 10 cm of soil before seeding.",
        "Tomato: incorporate 150 grams underneath each plant.",
        "Cucumber and squash: Incorporate 100 grams underneath each plant.",
        "For cultivating in pots or large containers, mix in 50 grams of Granu-Laines per 4 liters of soil.",
        "For any questions, please contact your garden center.",
    ],
    "instructions_fr": [
        "Pour votre potager, incorporer 150 à 200g de Granu-Laines par m2 de sol.",
        "Concombre et courge: incorporer 100g sous chaque plante.",
        "Tomate: incorporer 150g sous chaque plant.",
        "Pour la culture en pot ou en bac, incorporer 50g par 4 litres de terre.",
        "Et en cas de besoin, renseignez-vous à votre centre jardin.",
    ],
    "ingredients_en": [{"nutrient": "Ingredients", "value": 100.0, "unit": "%"}],
    "ingredients_fr": [{"nutrient": "Ingrédients", "value": 100.0, "unit": "%"}],
}

async def run_register_analysis():
    # Setup DB connection
    con = db.connect_db(DB_CONNECTION_STRING, DB_SCHEMA)
    cursor = con.cursor()

    # Create search path
    db.create_search_path(con, cursor, DB_SCHEMA)

    # Setup user
    user_email =<EMAIL_ADDRESS>    tier = "example-tier"
    user = await datastore.new_user(cursor, user_email, BLOB_CONNECTION_STRING, tier)
    user_id = datastore.User.get_id(user)

    # Setup Blob storage container
    blob_service_client = BlobServiceClient.from_connection_string(
        BLOB_CONNECTION_STRING
    )
    container_name = f"example-user-{str(user_id)}"

    try:
        container_client = blob_service_client.get_container_client(container_name)
        container_client.get_container_properties()
    except Exception:
        container_client = blob_service_client.create_container(container_name)

    # Register analysis without committing
    analysis = await fertiscan.register_analysis(
        cursor,
        container_client,
        user_id,
        [],
        analysis_json,
    )

    print("Registered Analysis (Raw):", analysis)

    # Retrieve full inspection object after registration
    inspection_id = analysis.get("inspection_id")
    stored_inspection = await fertiscan.get_full_inspection_json(cursor, inspection_id)
    print("Full Inspection JSON:", stored_inspection)

    # Write the full inspection JSON to a file
    file_name = f"inspection_{inspection_id}.json"
    with open(file_name, "w") as json_file:
        json.dump(json.loads(stored_inspection), json_file, indent=4)

    print(f"Full inspection JSON written to {file_name}")

    # Rollback to avoid committing any changes
    con.rollback()
    db.end_query(con, cursor)
    container_client.delete_container()

# Run the script
if __name__ == "__main__":
    asyncio.run(run_register_analysis())

Problem Statement
The company_website field (and possibly other company fields like company_name, company_address, company_phone_number) is set in the input data but is missing in the result. Specifically, in the result JSON, the company object shows null values for all fields, including the website.
Here is the result after running the script:
{
    "inspection_id": "6a9e22c5-8c1e-4bb2-beea-279c9b50dd2d",
    "inspection_comment": null,
    "verified": false,
    "company": {
        "id": null,
        "name": null,
        "address": null,
        "website": null,     <-------  website not present
        "phone_number": null
    },
    ...
}

Acceptance Criteria

Ensure that company_website is properly saved and retrieved in the inspection result.
Investigate if other fields in the company and manufacturer objects are also not being saved properly and fix the issue.
Investigate if updating an inspection presents the same issue.
Confirm that once the analysis is registered, the data persists as expected in the full inspection object.
Tests are added or updated to validate the persistence of company data during analysis registration.

So i do not know how this issue happens. What schema are you using?
The metadata is returning a json with the correct data for the company. The db also does not modify the fields of a company (it also protect us from inserting all empty data like in your test).
name_string := input_json->'company'->>'name';
	address_string := input_json->'company'->>'address';
	website_string := input_json->'company'->>'website';
	phone_number_string := input_json->'company'->>'phone_number';
	IF COALESCE(name_string,
		address_string,
		website_string,
		phone_number_string,
		 '') <> ''
	THEN
		company_id := "fertiscan_0.0.14".new_organization_info_located(
			input_json->'company'->>'name',
			input_json->'company'->>'address',
			input_json->'company'->>'website',
			input_json->'company'->>'phone_number',
			FALSE
		);

What schema are you using? Does the json from build_inspection_import has a company? I tested the module and it seems to work.
The database also work for me when inserting data.
We have tests that are suppose to check the 📦metadata/ to make sure it build the jsons correctly and we also have tests to check the upload of data works correctly.
What schema are you using? Are the functions correctly build in your DB?
Until I get more info on where this issue lies for you, I wont really be able to investigate it further on my end since I'm not even able to replicate it and the tools seems to be working as intended.

Thanks. I've added the schema and latest PR.

The error was simply a join problem excluding records when there are no Location


Add the bitwise builtins to the metatheory

This adds the bitwise builtins specified in CIP-0121 and CIP-0122 to the Agda code in plutus-meatatheory.  Adding these has exposed a deficiency in the Agda code.  Types in UPLC can have multiple size measures but the metatheory currently only knows about the standard ones:  see issue #6367.  We're expecting conformance tests for the bitwise operations shortly, but the costing problem means that the buudget tests will fail for some of these when agda-conformance is run.  To make this a bit more manageable I've modified the conformance tests so that you can separately specify a list of budget tests that are expected to fail.  We can use this to turn off the problematic budget tests until we fix the Agda costing problem.

This fixes #6303.


Kinesis LZO S3 Sink: change configuration to use S3 region instead of full endpoint URI

Replacing this:
  s3 {
    endpoint: "http://s3-eu-west-1.amazonaws.com"
    bucket: "mybucket" # Put s3 Bucket here
  }

with this:
  s3 {
    region: "eu-west-1"
    bucket: "mybucket" # Put s3 Bucket here
  }

Bringing forwards...


GEP: Mesh Service Binding

What would you like to be added:
As the GAMMA initiative seeks to create a model for service mesh implementations within Gateway API, it is necessary to have a mechanism for binding routes to services
This GEP covers the work to design and implement this mechanism..
Why this is needed:
As @shaneutt originally suggested, the first step will be implementing a provisional GEP that sets out the terms of what we're doing - the "What" and "Why" of the solution, and then once we're on the same page, we'll talk more about the "How".
Why this is needed:
Without a way to bind routes to services, no further work can continue

/kind gep
/area mesh

I've drafted a Google doc for more fluid discussion of this GEP

It would appear the permissions on this google doc are restrictive.

Apologies, the doc should be accessible now!

Based on recent conversations where we are trying to harden Service binding as the main way to do Gateway API for Mesh, this is still an ongoing effort and is not stale.
/remove-lifecycle stale

I think this will be complete with the release of v1.1

/remove-lifecycle rotten
This is basically finished, but the GEP in the GEP list is listed as GEP-1426, not GEP-1294. We should really fix that, then this can be closed. See https://gateway-api.sigs.k8s.io/geps/gep-1426/ for the details. (May need a redirect from https://gateway-api.sigs.k8s.io/geps/gep-1426/ to gep-1294 once the change is made as well).

/remove-lifecycle rotten

This last item was fixed by #3359. Closing.
/close


Draft: Remove more unnecessary conditional run layer calls

This PR is a work-in-progress that I've hit a (hopefully) minor stumbling block on.
Overview
@tgrant-nv was looking at some of our shader ptx awhile ago and pointed out that a surprisingly high amount of the total instructions were just conditional layer call checks (due to all the stores and loads surrounding calls). The concern was that this was slowing down OptiX module compilation.
OSL has some basic checks in place to remove unnecessary conditional layer calls, but this patch takes things further. The high level strategy is:

Mark all our conditional layer calls during llvm generation
Run a custom llvm optimization pass that associates calls with their basic blocks
For each call, walk up llvm's dominator tree to find basic blocks guaranteed to have already run
If a guaranteed basic block already contains the same call as the current one, remove the current one

It doesn't help the compilation times for every shader, but we've seen speedups as high as 5x on some of our most expensive shaders.
Work in progress
The problem I've run into is that the optimization pass introduces quite a bit of overhead to the ptx generation. Most assets take 10-20% longer to generate ptx with this patch. For us, it's probably still a good trade to turn on for GPU (module compilation is more expensive than ptx generation), but I was hoping some more eyes could help improve things. Especially since I've never written a LLVM pass before, maybe I'm doing things particularly inefficiently.
Any feedback or ideas would be appreciated!
Tests

Checklist:

[ ] I have read the contribution guidelines.
[ ] I have previously submitted a Contributor License Agreement.
[ ] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
[ ] My code follows the prevailing code style of this project.

This sounds very promising. A 10-20% increase in JIT time isn't so bad, if it increases the shader performance or even reduces the PTX compile time. We'll take a closer look and see what recommendations we can make.

This sounds very promising. A 10-20% increase in JIT time isn't so bad, if it increases the shader performance or even reduces the PTX compile time. We'll take a closer look and see what recommendations we can make.

Thanks! Looking forward to your findings.

Is this ready to be a non-draft?

It's close. It turned out the speed hit we've been seeing is because removing so many run-layer checks made the remaining ones more palatable for inlining. So LLVM's inlining pass times are growing quite a bit.
Now that it's been merged, I want to test this patch internally on #1680, since that also has inlining ramifications and I want to make sure everything interacts okay.


Lib for Windows

Hi! Please make some lib for Windows. It would be commercial I guess

You can compile it to windows. It just doesn't work :D
See #13


_find_new_data_files returns incorrect path when saving in subdirectories of datapath

Error Log:
{{{
Traceback (most recent call last):
File "/usr/local/bin/smt", line 5, in
pkg_resources.run_script('Sumatra==0.5.2', 'smt')
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 499, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 1235, in run_script
execfile(script_filename, namespace, namespace)
File "/usr/local/lib/python2.7/dist-packages/Sumatra-0.5.2-py2.7.egg/EGG-INFO/scripts/smt", line 31, in
main(sys.argv[2:])
File "/usr/local/lib/python2.7/dist-packages/Sumatra-0.5.2-py2.7.egg/sumatra/commands.py", line 328, in run
launch_mode=launch_mode)
File "/usr/local/lib/python2.7/dist-packages/Sumatra-0.5.2-py2.7.egg/sumatra/projects.py", line 173, in launch
record.run(with_label=self.data_label)
File "/usr/local/lib/python2.7/dist-packages/Sumatra-0.5.2-py2.7.egg/sumatra/records.py", line 158, in run
self.output_data = self.datastore.find_new_data(self.timestamp)
File "/usr/local/lib/python2.7/dist-packages/Sumatra-0.5.2-py2.7.egg/sumatra/datastore/filesystem.py", line 117, in find_new_data
for path in self._find_new_data_files(timestamp)]
File "/usr/local/lib/python2.7/dist-packages/Sumatra-0.5.2-py2.7.egg/sumatra/datastore/filesystem.py", line 27, in init
raise IOError("File %s does not exist" % self.full_path)
IOError: File data/raphs/flat/N-1000_v1.xml.gz does not exist
}}}
when data was saved to data/graphs/flat/N-1000_v1.xml.gz, where --datapath=data/
Proposed solution:
in datastore/filesystem.py line 108:
relative_path = os.path.join(root[length_dataroot-1:],file)
Imported from Bitbucket issue 192,  created by Felix11H on 11-27-2013 at 16:31, last modified: 01-21-2014 at 15:29

The problem arises because of the trailing slash in datapath.  This problem is already fixed in the development version, since we normalize the root path using abspath().
Trac comment by apdavison on 01-21-2014 at 15:29


Enable leap day to be included in PV simulations using weather data

This pull request enables February 29 to be included in a detailed PV simulation when inputting weather via the weather data variable. It also enables annual outputs for an 8760 simulation that includes Feb 29 but not Dec 31, specifically for our reV/REeDS folks (looking at you @grantbuster @mrossol ). Fixes #494 . It does NOT enable handling 8784 data, which is still on our wishlist.
After merging into patch, can we ask either @grantbuster or @mrossol to test a new ssc library to make sure this works for you? If so, let us know which OS and we can get that built.

@janinefreeman  for the win! I think the easiest way to test this is either:

To have @dguittet  clone reV and run the tests from a python env with the new ssc
Have @dguittet  help us set-up a custom pysam env on eagle that we can run the tests from.

@MRossol I think I can also help test from the reV side of things. @dguittet has given me an ssc executable before that I was able to patch into the reV env for a test.

@MRossol @grantbuster I'll upload a new pysam distribution to Box, and you can update the python env you're running reV in (whether locally or on Eagle) by doing pip install <path to pysam.whl>. What platform should I build it for?

@dguittet see the failed test here: https://github.com/NREL/reV/runs/1487453080?check_suite_focus=true.
Let's go with Linux, that way @grantbuster can test locally or I can test it on Eagle.

Here's the link to the PySAM v 2.2.1 whl for linux Python 3.7. Install via pip install <whl file>. Please note it's not fully vetted yet and is intended just to check the leap year tests.

@MRossol I uploaded a build for the wrong branch! Sorry, just updating now. Do you have a ETA for when the testing will be done-- would it be possible to do before EOD tomorrow, when we plan to release the patch?

@dguittet Non-annual data with length of 8784 will trigger the function to check if it's a year, but it won't pass, so it will be flagged as non-annual data. Feb 29 is allowed through in non-annual data now, though.

Hey @dguittet it looks like our tests pass! Thanks for implementing that.
Can you clarify what the improvements were on the ssc side of things? It is now more sensitive to the input dates but we still cant run non-8760s? Can we run partial timeseries, like a week of data?

@grantbuster Thanks for testing! We had made some changes in version 2020.2.29 to allow non-8760 data when entered using the weather data structure- so yes, you can run a week at a time (or any other length of time). The additions I had made to allow that made the code rely more on the timestamps in the weather data, and I neglected to have it allow Feb 29. However, when you run non-8760 data, you can't get "annual" results, because they would be meaningless. So the code checked that 8760 data (or multiples thereof) was in fact a complete, evenly spaced, Jan 1-Dec 31 year before it would report annual values. For the purposes of reV, this pull request also modifies that function so that it will now allow Jan 1-Dec 30 if Feb 29 is present, and still report annual values.

Awesome thanks @janinefreeman !


Cannot start creating config file

Hi,
Tried with a couple of versions. Image starts but it suddenly fails. I can see this in the logs
writing ssl cert and private key to '/opt/ejabberd/ssl/host.pem'...
Missing ssl cert for your xmpp domain. Generating snakeoil ssl cert for example.cat...
Generating a 2048 bit RSA private key
....+++
..............................+++
writing new private key to '/tmp/selfsigned.key'
-----
Writing ssl cert and private key to '/opt/ejabberd/ssl/example.cat.pem'...
Generating ejabberd config file...
Traceback (most recent call last):
  File "<string>", line 7, in <module>
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 969, in render
	return self.environment.handle_exception(exc_info, True)
  File "/usr/lib/python2.7/dist-packages/jinja2/environment.py", line 742, in handle_exception
	reraise(exc_type, exc_value, tb)
  File "<template>", line 133, in top-level template code
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 11: ordinal not in range(128)

Hi, I can't reproduce this error. How do you start the container? Do you use volumes or your own config templates? Does the Quick Start example work for you?

My apologies,
As you confirmed me it was working I double checked my run parameters and found a weird character passed as env var.
This was throwing an error as ejabberd was not able to parse it.


XML package requests

Could you please add the following packages to Nanobox pkgsrc?

libxml2-dev
libxslt1-dev

Thanks!

Note that -dev packages are a Debian/Ubuntu convention to slim down the number of installed files on user systems. In pkgsrc, this distinction is not made, so the files provided by the requested packages in Ubuntu are actually already available in the xml2 and xslt packages.

Great, libxml2 and libxslt worked. Thank you!


terraform-azurerm-aistudio: Make AI Studio services location configurable

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ X ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

N/A

Any log messages given by the failure

N/A

Expected/desired behavior

Allow AI Services region configuration to be different from default #AzureSandbox location to allow the sandbox to be provisioned in a different region than where the AI Services region is located. This is necessary due to capacity constraints and limited availabiity of AI Services in some regions.

OS and Version?

N/A

Versions

#AzureSandbox v2.13.0

Mention any other details that might be useful

Thanks! We'll be in touch soon.

backed out this change, AI services need to be in the same region as compute resources.


load() only works for Python 3

The following code in libconf.py:
    if isinstance(f.read(0), bytes):
        raise TypeError("libconf.load() input file must by unicode")

Only works for file objects opened in Python 3.  It should be something like:
if sys.version_info >= (3,0,0):
    # for Python 3
    if isinstance(s, bytes):
        raise TypeError("libconf.load() input file must be unicode")
else:
    # for Python 2
    if isinstance(s, unicode):
        raise TypeError("libconf.load() input file must be unicode")

Including @tgiannak, who found this issue.

This replacement code came from: http://stackoverflow.com/questions/4987327/how-do-i-check-if-a-string-is-unicode-or-ascii

Thanks for using libconf and for taking the time to report this!
Could you please provide a code snippet of how you're using libconf and what the error message is? There are several tests in test/test_e2e.py that pass on Travis running on Python 2.7 (there is a bytes alias for str since 2.6). I'm not sure how your usage differs. Getting to the bottom of it would be interesting, maybe we can improve the overall situation for Python 2.7 users a bit. The strict unicode requirement is probably relatively uncommon on 2.7.

This test code (after creating the named file with some content) shows the situation, since it's exactly the sort of check that libconf is making:
with open('scraper_mon.cfg') as f:
    contents = f.read()
    f.seek(0)
    if isinstance(f.read(0), bytes):
        raise TypeError("FAILED")

This works fine if run under Python 3, but not 2.7 here.  Sadly, I can't include the full test, since it's part of an internal project.  Thanks!

Could you try io.open()? The builtin open() produces bytes objects on Python2, while libconf expects unicode. The unicode requirement is documented, though I see that my own example in README.md is wrong (using the builtin open()).

With io.open it seems to work.

Hi @jwiegley, is it OK if I close this issue? io.open() works and release 1.01 fixes the documentation bug in README.rst. If you have any further issues with libconf's unicode handling on Python2, we can always open a followup issue.


In some of my projects I get a cryptic compile error

Happens only with some versions of the compiler e.g dmd-2.100.0.
``
../../external/argparse/source/argparse/package.d(425,6): Error: typesafe variadic function parameternameof typestring[]cannot be markedreturn`

my local version of argparse is pointing to master

mhh .. somehow dub pulls -dip1000 in :/ seems to be my mistake then!

I do get dmd errors from time to time. For example, I recently got "illegal instruction" inside dmd with my local change.
That's why every commit to master is verified: https://github.com/andrey-zherikov/argparse/runs/7848427132?check_suite_focus=true

mhh .. somehow dub pulls -dip1000 in :/ seems to be my mistake then!

Does this mean that there is no issue?

Not sure yet ... it behaves better, when I remove the dip1000 switch, but the question is, if argparse should already look forward to dip1000 ... will test a little more.

yes ... I tested again, and with dmd-2.100.0 and the addition  "dflags": ["-dip1000"] to argparse's dub.json I get exactly the described error. So it was a dub dependency of mine, that declared ´-dip1000` which got promoted to my main application that uses argparse ... so i think we can close this issue or rename it to something like "support dip1000"

Hey, I got this issue today when porting to latest LDC 1.30.0.
It seems that the code is violating some DIP1000 rules, because typesafe variadic function parameters can live on the stack, and therefore need to either use .dup, or know the length at compile-time instead of relying on a possible stack allocation or mark it as scope if you are sure this won't escape. The main reason for this to happen is that returning auto make the compiler infer attributes.
I believe the .dup option is fine here, since NamedArgument is used as an UDA and therefore the value is CTFEable, not leading to a GC allocation after-all.
The following patch fixes this and another other DIP1000 issue:
diff --git a/source/argparse/internal.d b/source/argparse/internal.d
index 50117d0..54112bf 100644
--- a/source/argparse/internal.d
+++ b/source/argparse/internal.d
@@ -617,7 +617,7 @@ package alias ParsingArgument(alias symbol, alias uda, ArgumentInfo info, RECEIV
         }
     };

-package auto ParsingSubCommandArgument(COMMAND_TYPE, CommandInfo info, RECEIVER, alias symbol, bool completionMode)(const CommandArguments!RECEIVER* parentArguments)
+package auto ParsingSubCommandArgument(COMMAND_TYPE, CommandInfo info, RECEIVER, alias symbol, bool completionMode)(const scope CommandArguments!RECEIVER* parentArguments)
 {
     return delegate(Config* config, ref Parser parser, const ref Parser.Argument arg, bool isDefaultCmd, ref RECEIVER receiver)
     {
diff --git a/source/argparse/package.d b/source/argparse/package.d
index ae32332..ac885e0 100644
--- a/source/argparse/package.d
+++ b/source/argparse/package.d
@@ -424,7 +424,7 @@ auto PositionalArgument(uint pos, string name)

 auto NamedArgument(string[] name...)
 {
-    return ArgumentUDA!(ValueParseFunctions!(void, void, void, void, void, void))(ArgumentInfo(name)).Optional();
+    return ArgumentUDA!(ValueParseFunctions!(void, void, void, void, void, void))(ArgumentInfo(name.dup)).Optional();
 }

 auto NamedArgument(string name)
@@ -1751,4 +1751,4 @@ unittest
     assert(test("auto")   == Config.StylingMode.autodetect);
     assert(test("never")  == Config.StylingMode.off);
     assert(test("")       == Config.StylingMode.autodetect);
-}
\ No newline at end of file
+}

Thanks for diving into this! I'll take a look once I'm done with current work


Objectron for Human detection

System information (Please provide as much relevant information as possible)

MediaPipe Solution (you are using): Objectron
Programming language : typescript/javascript
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state: Current objectron models identify specific objects like chair, shoe etc. This request is to create a model for 3D bounding box detection of humans. The object detection detects the humans as bounding boxes, but getting a 3D bounding box around and tracking it would open up new avenues to use mediapipe.
Will this change the current api? How? No, it should not because it is only adding a new object model which outputs a bounding box similar to existing one
Who will benefit with this feature?
Mobile users as well as adds to IoT applications
Please specify the use cases for this feature:
Clothing apps, Covid like 6 ft distance alerting, Robotics edge compute ODOA systems

Hi @SRai22,
This looks interesting feature request! We are sharing this request internally and based on discussion will prioritize among others. Thank you!

Hello @SRai22,
We are ending support for these MediaPipe Legacy Solutions, but upgrading the others. However, the libraries, documentation, and source code for all the MediapPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions. However, We can see that this feature has been added to the new MediaPipe solution and Demo to detect person can been seen here. Thank you

@SRai22,
Thank you reverting back and actively using our solutions.
As this feature has been already added here, Can we go ahead and mark the issue as closed?

I'll have to disagree on the subject of this feature being added already. The request was more specific on identifying 3D bounding boxes around human in the scene similar to the existing 'cup objectron', 'shoe objectron', 'chair objectron'. But since the support on objectron is no longer continued, I think you can close this issue.
Thanks

@SRai22,
Apologies for the misunderstanding. We would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. Thank you


Fix/edit decimal amount

Codecov Report

Merging #1118 (b4345a0) into master (6d9f964) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1118      +/-   ##
==========================================
- Coverage   44.74%   44.73%   -0.01%
==========================================
  Files         329      330       +1
  Lines        6325     6344      +19
  Branches      960      962       +2
==========================================
+ Hits         2830     2838       +8
- Misses       3425     3436      +11
  Partials       70       70

Impacted Files
Coverage Δ

...c/transaction/CreateTx/AmountField/AmountField.tsx
69.23% <0.00%> (-13.70%)
:arrow_down:

src/blockchains/tokens.ts
73.33% <0.00%> (-10.00%)
:arrow_down:

src/transaction/CreateTx/CreateTx.tsx
100.00% <0.00%> (ø)

src/_tests.ts
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Circular dependency detected

I've done a few deploys with the automation scripts previously, however all of a sudden I'm getting the following when attempting to run:
{"code":"InvalidTemplate","message":"Deployment template validation failed: 'Circular dependency detected on resource: '/subscriptions/MYSUBSCRIPTIONID/resourceGroups/rcshrchatdemo/providers/Microsoft.Web/sites/rcshrchatfunc'. Please see https://aka.ms/arm-syntax-resources for usage details.'."}
I've tried numerous Resource Group Name and Prefix combinations. I've ensured all my other deploys where cleanly deleted. And have updated my fork with the latest here.
Not sure what else to try.

My bad, found the bug and fixing it, Will close the issue upon bug fix and quick testing.

@PrintsCharming  - Fixed. Let me know if any further issues.