Adding support for fill colors directly in JSON (TC 11 feature).

With think-cell 11, we now have the feature to set the fill color with JSON. think-cell 11 is available on the More Downloads section at the bottom of the downloads page. The feature is described here: https://www.think-cell.com/en/product/whats-new.shtml?from=10&to=11

In order for this feature to work, the template file needs to utilize the "Use Excel Fill on Top" option in the template chart's color scheme. https://www.think-cell.com/support/manual/basicconcepts.shtml#subsect:colorscheme

I added this feature by adding an optional argument named fill. If it is set to None, no "fill" element will be added to the JSON.
So far, I have implemented this as a single color per Series, but it would be possible to expand filling each data point individually. In order to do this, I could expand the fill argument to also accept a 2d array that has len(series) lists each as long as that series data. This is not yet a required use case for me though.

Hey @colegrigsby! Thanks so much for this awesome PR! This sounds like a super interesting feature!
I'm currently a bit busy, but as soon as I'm done, I will make sure to run the tests, merge and release.
Have a great one!

Hi again @colegrigsby, couple of questions/clarifications before I merge this:

Is think-cell version 11 already available/stable? I am still running version 10 on my current machine. (see image below)
Do you mind running black in thinkcell.py and test_thinkcell.py? Black is part of the dev dependencies and such be run as black -l 79 name_of_file.py

Again, thank you very much for a great pull!

Hi @duarteocarmo -
It is not yet stable, but can be downloaded with the following instructions I was given from the think-cell support team.

"think-cell 11 is already available as a preview build on our website. If you request a link on our website: www.think-cell.com/downloads and follow the unique link we send you, you can find think-cell 11 under the More Downloads section at the bottom of the page."

I don't believe this will have any adverse effects on users running think-cell 10 as long as they do not try to use the fill option in the methods.
Also - Just added the reformat commits. Thank you!

Great @colegrigsby - will merge and include in the next release.
Again sorry for the delayed response, things have been a little hectic on my side 😅

Thanks for remaining active here @colegrigsby - the current version is still 10 on my side.
As soon as 11 is stable will make sure to pull this one in.
Thanks for the great work!

This works like a charm, thank you! I love that you raise a ValueError if the number of colors does not match the data they are to be assigned to.


Gave a strong name to CircuitBreaker.Net assembly

Fixes #4
The test assembly was given a strong name as well due to friendship relationship (InternalsVisibleTo()).

Awesome! Thank you for the contribution! I'll release the packages today\tomorrow.

Pushed v0.4.0 to NuGet.


Changes for ES2015 and linting

Changes include:

Alterations for ESLint
Alterations for Airbnb code standards
Changing var to let/const
Changing function() to () => where appropriate
Change ordering of functions so that functions weren't invoked prior
to being defined
Change some function parameters so they didn't have the same name as
the parent closure's variables (these are prefixed with "local" at
present)

Has been tested against demo.html and test.html to be the same as
previous state.

Fantastic work!


Pls fix error There are some server side issues

Pls fix error There are some server side issues in XLR8_BOMBER

@Almight384
i have made a modified version of xlr8 bomber this version is working all server side issues are fixed but script has not been updated by @anubhavanonymous
install and usage commands are here
https://github.com/bagarrattaa/xlr8mod

@Almight384
i have made a modified version of xlr8 bomber this version is working all server side issues are fixed but script has not been updated by @anubhavanonymous
install and usage commands are here
https://github.com/bagarrattaa/xlr8mod

Pls fix error There are some server side issues in XLR8_BOMBER

Everything fixed !
Check now 🧨

@Almight384 i have made a modified version of xlr8 bomber this version is working all server side issues are fixed but script has not been updated by @anubhavanonymous install and usage commands are here https://github.com/bagarrattaa/xlr8mod

Keep your crap to yourself
Don't defame my name


Sidebar is not closing properly

Does this issue occur when all extensions are disabled?: Yes/No
No

VS Code Version: Version: 1.61.2 stable build(user setup)
Commit: 6cba118ac49a1b88332f312a8f67186f7f3c1643

OS Version: Windows NT x64 10.0.22000 (Windows 11 Beta build)

Steps to Reproduce:
1.Open the sidebar and close it after some work

@deepak1556 have you seen anything like this reported. The sidebar seems properly hidden here since I can see the line numbers of the editor. Some gpu acceleration screen cheese perhaps?

can anybody tell what more info can I give?

Does the issue repro when you start with code --disable-gpu ?

Actually this problem happen very often and I noticed this when i closed the sidebar. This problem doesn't occur always. As it seemed odd to me I informed.
Whether I use --disable-gpu or just start it normally the problem not happening now.
So what should I do now?

https://user-images.githubusercontent.com/58502105/141681424-918b272c-ab14-404f-a6be-84a6c653f9e8.mp4

Guys I found that this problem enables when I use github dark default theme and reload the window with the panel opened. When the reload completes properly I close the panel and the issue recreates.
See This:
https://user-images.githubusercontent.com/58502105/141682558-2ef14d95-93d2-4bbe-bcac-bdf945942f79.mp4
See this in fullscreen to have a better look at the problem

Hey @deepak1556 sbatten, this issue might need further attention.
@SantoshPattnaik, you can help us out by closing this issue if the problem no longer exists, or adding more information.

@vscode-triage-bot @deepak1556 @sbatten
Sir the problem now also exists and the problem can be reproduced.
This prblm happens when I open the sidebar and click on any other button on the sidebar like extensions, search or anything which changes the sidebar width, after clicking on the button through which I first opened the sidebar, and closing it the prblm is reproduced.
You can refer the video (Vscode1.mp4)

@SantoshPattnaik now that you have a repro again, can you test with --disable-gpu

@SantoshPattnaik now that you have a repro again, can you test with --disable-gpu

YES, the prblm is gone after i uncommented disable-gpu-acceleration flag in argv.json.
I have opened and closed vs code 10 times after adding the flag and the prblm has gone.
After I restored the settings, the problm is happening 6-7 times in the 10 times open and close period.
I hope you understand
Well may I know is this a software problem or operating system issue as other software windows are rendering fine?


Bluetooth sensor integrated, showing the nearest Bluetooth device

Please fix the CLI first 😃

Created https://github.com/ChrisAllen1982/ha-google-home/pull/1 to @ChrisAllen1982's branch to correct the issues blocking this PR.

I've noticed I am not seeing the sensor.*_bt_devices sensor anywhere. I haven't had a chance to dig into why yet.

This is probably why.
Logger: homeassistant.components.sensor
Source: custom_components/google_home/sensor.py:91
Integration: Sensor (documentation, issues)
First occurred: 1:26:50 PM (1 occurrences)
Last logged: 1:26:50 PM

Error while setting up google_home platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/google_home/sensor.py", line 91, in async_setup_entry
    device.device_model,
AttributeError: 'GoogleHomeDevice' object has no attribute 'device_model'

One of my previous fixes used the wrong property: https://github.com/ChrisAllen1982/ha-google-home/pull/2

https://github.com/ChrisAllen1982/ha-google-home/pull/4 Fix the final linting errors, not sure why running it locally didn't flag these issues.

Please fix the CLI first 😃

I think this is ready for review now?

I'm relatively new to the GitHub workflow. Do I need to assign the pr to someone to review?

Generally someone will grab it.

@ChrisAllen1982, @HarvsG, @marcusds, guys, is anyone of you have any availability these days - feel free to grab this PR and move it further. We need to fix the requested changes and re-request review :)

Closing for lack of activity :( hopefully it will be picked up at some point down the road :)

Any news about this?


Failed to download plugin

Getting this error when I try to use this plugin:
Plugins are disabled because an error has occurred." error="failed to download plugin github.com/GiGInnovationLabs/traefikgeoip2: failed to call service: Get "https://plugins.traefik.io/public/download/github.com/GiGInnovationLabs/traefikgeoip2/v5.0.1\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
I saw that you had the same issue before https://community.traefik.io/t/how-to-import-plugin-that-uses-unsafe-package/11186/8
Look like it persists again :(

Maybe there is a way to build Traefik with your plugin and not download it in runtime?

Hello

use local plugins, https://traefik.io/blog/using-private-plugins-in-traefik-proxy-2-5/
report to traefik, not here.

Understood. Thanks.

Created post on Traefik community https://community.traefik.io/t/failed-to-download-plugin/17155


Hide avg and stdev until voting is finished

Closes https://github.com/github/internal-tools/issues/314

I just deployed it! 👍  🐦


Added Discontinuation Report

Overview
fix #1473
Test Notes/Sample Code

Notes:

Closing for now. May re-visit in future release.


Unable to start songbird

The following endpoint returns error 403
https://songbird.flare.network/ext/info

You need to be whitelisted

Thanks!
i am whitelisted now


Metrics stream not closing connections

Hi,
Way back in the day an issue was raised whereby connections to the metric stream were being held open when no command data was available: https://github.com/Netflix/Hystrix/issues/85
Since the rewrite of the HystrixMetricsStreamServlet it would appear that this behaviour has slipped back in.
If no data is available the onNext of the subscriber in HystrixSampleSseServlet is never called, and hence the connection is never checked, thus connections are held forever.
This can be replicated using the hystrix example webapp by just curling the metrics stream upon app start up.
[norvalp@mbp ~]$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
[norvalp@mbp ~]$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
[norvalp@mbp ~]$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
[norvalp@mbp ~]$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
[norvalp@mbp ~]$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
[norvalp@mbp ~]$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream

<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"/>
<title>Error 503 MaxConcurrentConnections reached: 5</title>

HTTP ERROR 503

Problem accessing /hystrix-examples-webapp/hystrix.stream. Reason:

    MaxConcurrentConnections reached: 5

Powered by Jetty://

[norvalp@mbp ~]$

Came here to report this same issue, netstat is showing the sockets in a CLOSE_WAIT state indicating that the server is the culprit.
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        1      0 <IP_ADDRESS>:8080       <IP_ADDRESS>:41290       CLOSE_WAIT  -
tcp        1      0 <IP_ADDRESS>:8080       <IP_ADDRESS>:41382       CLOSE_WAIT  -
tcp        1      0 <IP_ADDRESS>:8080       <IP_ADDRESS>:41572       CLOSE_WAIT  -
tcp        1      0 <IP_ADDRESS>:8080       <IP_ADDRESS>:41478       CLOSE_WAIT  -
tcp        1      0 <IP_ADDRESS>:8080       <IP_ADDRESS>:41196       CLOSE_WAIT  -

I'll investigate

I just added this branch with some printlns to the stream servlet: https://github.com/mattrjacobs/Hystrix/tree/understand-1430-regression
When I run the Jetty server in hystrix-examples-webapp, it's behaving the way I expect.
I'm able to run curls against that server, and the server properly cleans up when the client process ends.  Is there anything different about this attempt at a reproduction and your cases, @DangerPete / @mattnelson ?
Sample server output when I repeatedly start a curl and terminate with Ctrl+C:
1481568960959 :<PHONE_NUMBER>@qtp-1695903109-7 Number of connections : 1
1481568961964 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568961965 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568961965 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568961966 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568962240 :<PHONE_NUMBER>@qtp-1695903109-7 Number of connections : 1
1481568963245 : RxIoScheduler-7 Caught IOException, can release the server thread
1481568963246 : RxIoScheduler-7 Caught IOException, can release the server thread
1481568963246 : RxIoScheduler-7 Caught IOException, can release the server thread
1481568963246 : RxIoScheduler-7 Caught IOException, can release the server thread
1481568963247 : RxIoScheduler-7 Caught IOException, can release the server thread
1481568963793 :<PHONE_NUMBER>@qtp-1695903109-7 Number of connections : 1
1481568964298 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568964299 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568964299 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568964299 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568964300 : RxIoScheduler-6 Caught IOException, can release the server thread
1481568964799 : RxIoScheduler-6 Caught IOException, can release the server thread

What version of jetty is hystrix-examples-webapp using? I am using 9.3 which, from what I've seen , appears to have some different connection handling compared to previous versions.

It was using whatever the stock Gradle Jetty plugin was using, which wasn't clear.  I switched to use Gretty and was able to choose between Jetty 7/8/9 and Tomcat 7/8.  All of them worked locally as expected.
Are you able to replicate those findings?  The Gretty config is pushed to my personal fork: https://github.com/mattrjacobs/Hystrix/tree/understand-1430-regression

I was able to recreate the error by loading the hystrix stream before calling the demo page. This lines up with what I've been seeing on our services as well. Since we are using turbine to aggregate as soon as the service is UP, turbine begins to aggregate. It isn't until the first command is executed that the stream recovers and closes the abandoned connections.
curl logs
shell@laptop:~$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
shell@laptop:~$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
shell@laptop:~$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
shell@laptop:~$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
shell@laptop:~$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream
^C
shell@laptop:~$ curl http://localhost:8989/hystrix-examples-webapp/hystrix.stream

<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
<title>Error 503 MaxConcurrentConnections reached: 5</title>

HTTP ERROR 503

Problem accessing /hystrix-examples-webapp/hystrix.stream.   Reopening until the release happens and the fix is verified.  Thanks for the help in getting to a repro case.

When can we expect a new release?

Just released 1.5.9


Error: can not connect input to output

After the commit Prefer for loop over lambda (#181)
I can not connect the input of the Nodes with the output of another Nodes.
And only connect the output ->  input
variant first output -> input - works:

variant second output -> input does not work now:

Thanks,
regards jetru.

I can not access a computer until Monday, sorry. It is not happening to me and I don't know how to reproduce it

@paceholder @facontidavide sorry maybe it was before this commit
May 28, it worked, only today drew attention to what does not work
I will study this problem

Please revert one commit at a time to see when it was introduced. Cheers

@facontidavide sorry, in commit "Commits on May 29, 2018" already no work
I will try next

Please open another issue if there is some problems.


Listenened field clears.

Trying to implement a Dependent Dropdown. Seems to work fine except for a couple of issues

Need to initialise dependent dropdown first time into screen based on current district_id. I can probably work that out

BUT, immediately after selecting a district, the district dropdown clears!!! The Location dropdown is populated correctly with the locations for the selected district, but the District dropdown is clear!  Been hitting my head against this for a while now. What is happening

Please help.
<--relevant code below-->
DistrictListener.php
class DistrictListener extends Listener
{
    /**
     * List of field names for which values will be listened.
     *
     * @var string[]
     */
    protected $targets = ['event.district_id'];

    /**
     * What screen method should be called
     * as a source for an asynchronous request.
     *
     * The name of the method must
     * begin with the prefix "async"
     *
     * @var string
     */
    protected $asyncMethod = 'asyncFromDistrict';

    /**
     * @return Layout[]
     */
    protected function layouts(): iterable
    {
        // dump($this->query->get('location'));
        return [
            Layout::rows([
                Select::make('event.district_id')
                ->title('District')
                ->fromModel(District::class, 'district_name')
                ->empty('--everywhere--'),

                Select::make('event_id.location_id')
                ->title('Location')
                ->options($this->query->get('location')?$this->query->get('location'):[]),
   }

EventEditScreen.php
   public function asyncFromDistrict($district_id)
    {
        return [
            'location' =>   Location::where('district_id',$district_id)->pluck('description','id')->toArray()
        ];
    }

Can your provide the version of Orchid you are using please

I will close this for now as inactive.

Definitely not inactive. Sorry, been away for a few days. Using latest version of Orchid, latest laravel. Don't have development machine with me so can't quote version numbers sorry but first thing I did when I first had this problem was to make sure I was on latest versions.

@jrushworth Feel free to reopen this if you have any additional information.

you have to set the values in listener layout not to lose them. So, you have to listen for the values for all you put in your listener class.
for ex : if you put fields a, b, c in your Listener Layout, then you should listen fields a,b,c and set their own value inside listener class.


Fix StringRef issue in ExecutionEngine

ExecutionEngine::setBackendName() is getting a llvm::StringRef as input. in some cases, the reference is invalid after calling hostManager_.reset(), and when trying to use it in the next few lines you'll get an unexpected behavior (probably failing on line 56 check).
suggesting to use the member string that gets updated in the beginning of the function instead of using the input itself.
Summary:
Documentation:
[Optional Fixes #issue]
Test Plan:
Please see a detailed explanation of how to fill out the fields in the relevant sections in PULL_REQUEST.md.

@jsubag


Reset jled object when effect changes

Coverage decreased (-0.07%) to 96.104% when pulling eb96fae0dc7ffea6604b847430bf51aba16d02f2 on reset_jled_object_when_effect_changes into 8647eb6c261e8ab88868ccc82046a8e67e0693b8 on master.


bumping to latest versions of codemirror

@Sodik i think this will fix MC syntax highlighting. i had declared some strict equal package versions. with this pr we switch to compatible versioning.

verify

verify

verify


Adding minimal support for Zig

Why is this change being made?
Today SizeBench fails to open binaries/PDBs produced by Zig.  There are a few issues with Zig's toolchain, but SizeBench can still be made functional with some minimal changes until Zig can produce better PDBs.
In the future it would be good if Zig produced real command lines data for compilands (it comes through now as just "0"), and had its own CV_CFL_LANG enum value from DIA, but in the meantime we can special-case code that says it is CV_CFL_LANG_C and built with a tool name of "zig" to assume this is Zig code.
Zig also links in by default a library that uses __float128 so we should support that 16-byte floating point type, which benefits GCC and Clang as well, which I believe can both compile that type.
Zig also produces the longest, most convoluted COFF Group names I've ever seen which exposed bugs in how COFF Group names got converted to XAML property paths in the UI code.  That's fixed now, though honestly Zig shouldn't generate COFF Group names so absurd.
Briefly summarize what changed

Add Zig to the list of compiland/tool languages
Add support for 16-byte floating point types
Escape COFF Group names in XAML property path bindings

How was the change tested?
Added a Zig RealPETests set of tests that check for loading symbols, type layouts, etc.  Also check that the Zig language is detected correctly on a compiland, and that the built-in __float128-using function parses correctly.
PR Checklist

[x] Contributor License Agreement (CLA) has been signed. If not, go here and sign the CLA
[x] Changes have been validated
[x] Documentation updated. Please add or update any docs in the repo as necessary.

Thank you so much for working on this! 🥰


Incorrect documentation in context menu

The right-click context menu has some incorrect information:

a. The key binding for "toggle documentation" is apparently ctrl-j ctrl-d (not just ctrl-d).
b. "Go to definition" key binding is ctrl-j ctrl-g (not just ctrl-g).
c. The "Go to definition" command name is misspelled.

I think that's the wrong picture?

Yup, it was. My bad.

Still misspelled. Also, don't seem to work (e.g. just does nothing, or just very slow to respond) most of the time?

The incorrect keybindings are an upstream issue: https://github.com/atom/atom/issues/17222
@vtjnash Which of those don't work for you? Do note that all of those commands only work for symbols that are loaded in the current session. Additionally, Go to definition only works for objects that carry line info with them (since we're not doing any static analysis).

It seemed like go-to-definition and toggle-documentation didn't give feedback on whether they're still running the query or why they might be failing

Oh I see. Usually the progress indicator in the lower left of the editor should give you that info.


[useStore, api] = create() is deprecated and will be removed in v4

hi guys

[useStore, api] = create() is deprecated and will be removed in v4

how else will I get access to API?
btw I had to go back to using API because I, weirdly, got a

An unhandled error was caught from submitForm() Error: Invalid hook call. Hooks can only be called inside of the body of a function component

in a formik onSubmit form call... very weird.
But anyhow, it seems to work via api. How else is access to api done?

Hi, sorry for the confusion.
const [useStore, api] = create(...)

can be written
const useStore = create(...)
const api = useStore

in v3. It looks tricky, but it works.

If you really hesitate to do the above, the alternative way is:
import createStore from 'zustand/vanilla'
import createHook from 'zustand'

const api = createStore(...)
const useStore = createHook(api)

so
useStore and api is the same thing now, that's what you are saying?
so I can do
const useStore = create(...);
useStore.getState()...

???

useStore and api is the same thing now, that's what you are saying?

Yeah, it's like overloaded.

const useStore = create(...);
useStore.getState()...

Right. Check readme: https://github.com/react-spring/zustand#readingwriting-state-and-reacting-to-changes-outside-of-components

that would leave the weird hook problem from above though.

Do you mean Error: Invalid hook call.? I don't think it happens.

that would leave the weird hook problem from above though.
Do you mean Error: Invalid hook call.? I don't think it happens.

yeah you were right. I somehow had brain fog from working too much even though, it isn't as clear anyhow.
I did basically, after  a lot of simplification, did this:
const App = () => {

return    <Formik
            initialValues={{
            }}
            onSubmit={(values, formikHelpers) => {
            const {count, inc} = useStore();
                console.error("count", count);
            }
            }
        >

which doesnt work.
This was the fix:
const App = () => {
            const {count, inc} = useStore();

return    <Formik
            initialValues={{
            }}
            onSubmit={(values, formikHelpers) => {

                console.error("count", count);
            }
            }
        >


annotating anndata object with clusters and tnse info from .csv file

Dear,
Thanks for this fantastic package. I saved one of my analysis to get .csv files output using the cmd adata.write_csv('./ouput').  I modified some of these files (the obs.csv and obsm) for pecific purposes related to a collaborative project while respecting the dimensions and the order of the cells. How can I reload back these files into an anndata object?
adata = sc.read_csv('./ouput/*') ??

The documentation for AnnData.write_csvs tells you

It is not possible to recover the full AnnData from the output of this function. Use write() for this.

But if you have nothing else but those files, you can of course try to use pandas.read_csv to read the .obs and .var dataframes and do something like
adata = AnnData(
    pd.read_csv('output/X.csv'),
    pd.read_csv('output/obs.csv'),
    pd.read_csv('output/var.csv'),
    { # adata.uns
        'some_thing': pd.read_csv('output/some_thing.csv'),
    },
    pd.read_csv('output/obsm.csv'),
    pd.read_csv('output/varm.csv'),
)

You might have to fiddle with parameters to pandas.read_csv, like index_col, and obsm/varm might not be able to be specified as data frames.

From a gene matrix, tsne and cluster .csv files obtained from cell ranger output I was able to load these into scanpy and display a tsne plot that look exactly like the output of cellranger cloupe file. This is great thanks!


Implement RingProof SNARK using TE coordinates

Implement Twisted Edwards constrains for Ring proof: https://github.com/w3f/ring-proof/tree/skalman--allow-ec-in-te

https://github.com/w3f/ring-proof/pull/44
https://github.com/w3f/ring-proof/pull/45


Check if Conditions are used

There already is a check to see if the specified Parameters are actually used (https://github.com/awslabs/cfn-python-lint/blob/master/src/cfnlint/rules/parameters/Used.py)
Would be nice to have the same kind of check for Conditions

Any feedback on the 2 PR's for this?


Crash with hunger owerhaul, when hunger owerhaul not presents.

mariculture looks for apple core's iedible method, and causes random crash on startup.

fixed


DataGrid ScrollViewer issue

DataGrid bound to ObservableCollection ignores property changes.
In my case, issue occured when vertical scroll appears. If i do some scroll - it works as expected.
Also it works well when container does not have vertical scroll.

AvaloniaDataGridRefreshIssue.zip

This issue is actually being caused by your example code. The custom implementation of DataGridItem.Equals prevents rows from updating when their DataContext changes because == returns true. There are some complicated internals surrounding how rows are recycled that causes this to only occur in DataGrids with visible vertical ScrollBars that have not yet been scrolled.


Cervical spine distracting: Copyedits

Lessened the tone and definitiveness that torso/chest injuries should always be considered distracting
Made each study header-2
formatting changes

Codecov Report

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

@@           Coverage Diff           @@
##           master      #73   +/-   ##
=======================================
  Coverage   50.32%   50.32%
=======================================
  Files          14       14
  Lines         155      155
  Branches       43       43
=======================================
  Hits           78       78
  Misses         77       77

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 09a53c0...dc8ca16. Read the comment docs.


** BUILD FAILED **

error: No signing certificate "Mac Development" found: No "Mac Development" signing certificate matching team ID "EF37D6UQZ9" with a private key was found. (in target 'GLTFQuickLook' from project 'GLTFQuickLook')

Thank you for your report!
You don't need signing if you locally build and run GLTFQuickLook.
Could you try selecting "None" for "Team" and "Sign to Run Locally" for "Signing Certificate"? and I guess you can build it.

I'm going to remove my signing settings from the repository to avoid the confusion.


Prevent Log Spam: Summarize message encoding errors by default.

In order to give the user full access to all information, a configuration option SummarizeMessageEncodingErrors was introduced, which defaults to true, but can be set to false when more information is needed.
Also had to adapt a large number of Test cases.

Checklist
The following aspects have been respected by the author of this pull request, confirmed by both pull request assignee and reviewer:

Adherence to coding conventions

[x] Pull Request Assignee
[x] Reviewer

Adherence to javadoc conventions

[x] Pull Request Assignee
[x] Reviewer

Changelog update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

README update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

config update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[x] Reviewer

SDCcc executable ran against a test device (if necessary)

[x] Pull Request Assignee
[x] Reviewer

A binding for your newly added configuration key is missing, resulting in the Guice/MissingImplementation error.

Currently the executable cannot be executed, there is a mismatch with your configuration keys.
Exception in thread "main" java.io.IOException: Unknown configuration key 	at com.draeger.medical.sdccc.guice.TomlConfigParser.parseKey(TomlConfigParser.java:94) 	at com.draeger.medical.sdccc.guice.TomlConfigParser.parseToml(TomlConfigParser.java:79) 	at com.draeger.medical.sdccc.TestSuite.createTestRunInjector(TestSuite.java:461) 	at com.draeger.medical.sdccc.TestSuite.main(TestSuite.java:588)

Currently the executable cannot be executed, there is a mismatch with your configuration keys.
Exception in thread "main" java.io.IOException: Unknown configuration key at com.draeger.medical.sdccc.guice.TomlConfigParser.parseKey(TomlConfigParser.java:94) at com.draeger.medical.sdccc.guice.TomlConfigParser.parseToml(TomlConfigParser.java:79) at com.draeger.medical.sdccc.TestSuite.createTestRunInjector(TestSuite.java:461) at com.draeger.medical.sdccc.TestSuite.main(TestSuite.java:588)

Problem Resolved.

Since we re-enable using the charset specified in a message to decode it, this PR resolves Issue #48 .

A binding for your newly added configuration key is missing, resulting in the Guice/MissingImplementation error.

This is the same as below. Also resolved.

TODO: remove Limitation that SDCcc can only decode messages encoded in UTF-8 from README.md.

TODO: remove Limitation that SDCcc can only decode messages encoded in UTF-8 from README.md.

Done.


camelcase rule disallows our standard snake-case for static methods

In our coding standards we say that:

Static functions are named all lower case with underscores separating words in the name.

But this breaks currently in a TS file:
class Foo {
   public static usd_from_gbp(amount: number): number {
      // whatever
   }
}

Currently the camelcase rule and @typescript/camelcase both do not support disabling the rule for class members/properties at all, let alone for static functions only.
I opened an issue for to support this, since the base rule already has a { "properties": "never" } option that disables it for object properties. (This option has no affect on class members)
https://github.com/typescript-eslint/typescript-eslint/issues/590

There is a new rule that should fix this problem, if correctly implemented. @typescript-eslint/naming-convention
This rule provides much more granular control over naming of each type of identifier (variable, property, method, type, etc.). However, we will need to research and test exactly what combination of properties to use with this.
One thing to note, it does not seem to differentiate between object and class properties. We often create objects to match a schema of a certain api (i.e. config objects passed as parameters. If the api doesn't follow our naming conventions this rule will give a "false" error.

Resolved with https://github.com/silvermine/eslint-config-silvermine/pull/89


chatglm4v-9b全量微调full报错：File "/usr/local/Python3.10.0/lib/python3.10/site-packages/deepspeed/runtime/zero/stage3.py", line 149, in init [rank0]:     self.dtype = self.optimizer.param_groups[0]['params'][0].dtype [rank0]: IndexError: list index out of range

[rank0]: Traceback (most recent call last):
[rank0]:   File "/home/nfs/appnfs/sxh/code/swift-latest/examples/pytorch/llm/llm_sft.py", line 6, in
[rank0]:     output = sft_main()
[rank0]:   File "/home/nfs/appnfs/sxh/code/swift-latest/swift/utils/run_utils.py", line 32, in x_main
[rank0]:     result = llm_x(args, **kwargs)
[rank0]:   File "/home/nfs/appnfs/sxh/code/swift-latest/swift/llm/sft.py", line 439, in llm_sft
[rank0]:     trainer.train(training_args.resume_from_checkpoint)
[rank0]:   File "/home/nfs/appnfs/sxh/code/swift-latest/swift/trainers/mixin.py", line 409, in train
[rank0]:     res = super().train(resume_from_checkpoint, *args, **kwargs)
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/transformers/trainer.py", line 1948, in train
[rank0]:     return inner_training_loop(
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/transformers/trainer.py", line 2105, in _inner_training_loop
[rank0]:     model, self.optimizer = self.accelerator.prepare(self.model, self.optimizer)
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/accelerate/accelerator.py", line 1318, in prepare
[rank0]:     result = self._prepare_deepspeed(*args)
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/accelerate/accelerator.py", line 1815, in _prepare_deepspeed
[rank0]:     engine, optimizer, _, lr_scheduler = deepspeed.initialize(**kwargs)
[rank0]:   File "/home/nfs/appnfs/sxh/code/swift-latest/swift/llm/utils/template.py", line 344, in _initialize
[rank0]:     res = _old_initialize(*args, **kwargs)
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/deepspeed/init.py", line 176, in initialize
[rank0]:     engine = DeepSpeedEngine(args=args,
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 307, in init
[rank0]:     self._configure_optimizer(optimizer, model_parameters)
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 1257, in _configure_optimizer
[rank0]:     self.optimizer = self._configure_zero_optimizer(basic_optimizer)
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 1579, in _configure_zero_optimizer
[rank0]:     optimizer = DeepSpeedZeroOptimizer_Stage3(
[rank0]:   File "/usr/local/Python3.10.0/lib/python3.10/site-packages/deepspeed/runtime/zero/stage3.py", line 149, in init
[rank0]:     self.dtype = self.optimizer.param_groups[0]['params'][0].dtype
[rank0]: IndexError: list index out of range

已经解决，需要使用官方配置的deepspeed文件


About map of using multi-scale.

Did you try traing dota by using multi-scale? if yes, could you tell the mAP？

yeah，we trained ARS-DETR with Swin-T by ms and here are the results:
AP50: 0.7612842596809277
AP75: 0.5236644646118254
mAP: 0.4841383593699928

Thanks for your response.


Fix for issue #620

This PR fixes the error in issue #620
The previous implementation can lead to division by zero or extremely large exponential values that can lead to overflow problems for constrained problems.
This PR fixes these errors as follows:

Regularization with epsilon: The small constant epsilon is added to each value of point[obj_index] before raising it to the power of x. This prevents the base of the exponentiation from becoming too small, which can lead to extremely large results when x is large.
Logarithm Handling: The logarithm calculation is adjusted to prevent taking the log of non-positive numbers.
Zero Handling: If the denominator (denominator) or the derivative (ff) is zero, the function returns a fallback value to avoid division by zero.

For which problem do you get this error? That would help me debugging the code

For which problem do you get this error? That would help me debugging the code

This: https://gist.github.com/muazhari/deed94f293489e95a75e95c057f5e98f

On my machine, I don't get that error. I have also tried with different seeds

Another error again in this: https://gist.github.com/muazhari/0825646b3767402b1899881836b36c42

I don't know if mixed variable modification to the GA algorithm is interoperable with agemoea2. I think it should because it uses the same abstraction in the superset class (GA) itself, and is not specific to agemoea2.

On my machine, I don't get that error. I have also tried with different seeds

damn, how? My machine is currently a bit random as in my last GitHub issues to Pymoo, wsl, and Ray. Is it because intel stability issue? haha.

On my machine, I don't get that error. I have also tried with different seeds

I tried to run 3 times and still got the same error. Can you try to execute in different log levels? Or make sure the stdout/stderr not obstructed?

Another error again in this: https://gist.github.com/muazhari/0825646b3767402b1899881836b36c42
I don't know if mixed variable modification to the GA algorithm is interoperable with agemoea2. I think it should because it uses the same abstraction in the superset class (GA), and is not specific to agemoea2.

I also don't get this error on my chain (M3 Mac). But the last commit should consider/handle division with NaN and Inf value. This should prevent the error you have

Actually, I am looking at the stack trace you have. It seems you are not using the code after my commits as the line f += np.power(point[obj_index], x) in your stack trace is now f += np.power(point[obj_index] + epsilon, x), which is the latest code.

My bad. After ensuring to use your latest commit, both codes above did not display any errors.

For which problem do you get this error? That would help me debugging the code

This: https://gist.github.com/muazhari/deed94f293489e95a75e95c057f5e98f

Sorry, this first gist still has an error. Unknowingly, the stack trace can't displayed to the stdout when using logger/try-catch. After I removed it, the stack trace gave the error. Please rerun the modified gist.

For which problem do you get this error? That would help me debugging the code

This: https://gist.github.com/muazhari/deed94f293489e95a75e95c057f5e98f

Sorry, this first gist still has an error (the second gist is fixed). I don't know why the stack trace can't be displayed to the stdout when using logger/try-catch. After I removed it, the stack trace displayed the error. Please rerun the modified gist.

It should be fixed with the last commit

In the second gist (https://gist.github.com/muazhari/0825646b3767402b1899881836b36c42), comes another warning:

In the first gist (https://gist.github.com/muazhari/deed94f293489e95a75e95c057f5e98f), still has an earlier error.

I just pushed a much stronger/more robust way to check for overflow errors. Let me know if u still have problems/errors

The last two commits should fix the issues in both gits. Let me know if you encounter more issues

I have merged the code for now. Please open a new PR if another fix is needed.

Thanks @apanichella for taking care of it!


V2.1.2 Ternary crashes exported exe

Operating system or device - Godot version:
Win 7 (x64) desktop
Godot v2.1.2
Issue description:
The use of the ternary operator causes the exported .exe to crashes. The applications runs perfectly fine when in editor but not when exported.
The export debug reports the following error:

SCRIPT ERROR: GDScript::load_byte_code: Parse Error: Expected end of statement (var)
At: res://TernaryTest.gdc :6
Error: GDScript::load_byte_code: Method/Function Failed, returning: ERR_PARSE_ERROR
At: modules\gdscript\gd_script.cpp:720

Steps to reproduce:

Use the ternary operator in your code i.e. var frame = 0 if (!challenge.unlocked) else 1
Export project
Run exported project

Link to minimal example project:
Test project

@gonzo191 did you update export templates also?

@volzhs OK it works now. I was not aware I had to do that.

That's one reason to use the Steam version, it'll update the export templates along with everything else.


Fix Bugzilla issue 24827: maxElement does not handle opAssign correctly.

Rebindable2 did not handle types with opAssign correctly, which affected both minElement and maxElement. Namely, Rebindable2 assigned to memory which was not properly initialized when the correct solution in such a situation is to use copyEmplace. Assignment works when assignment is just a memcpy, but in the general case, opAssign needs to have a properly initialized object in order to work correctly. copyEmplace instead copies the object and then places the copy into the unitialized memory, so it avoids assigning to uninitialized memory.
This commit also adds additional tests for types with destructors (which do get opAssign automatically) and types with postblit constructors or copy constructors to try to ensure that the code is doing the correct thing in those cases with regards to copying, assignment, and destruction.
https://issues.dlang.org/show_bug.cgi?id=24829 was found in the process, and this does not fix that. Namely, types which cannot be assigned to and which also have a postblit constructor or copy constructor do not get copied correctly. So, among the tests added here are commented out tests for that case, since they're an altered version of some of the enabled tests. However, fixing that issue would be involved enough that I'm not attempting to fix it at this time.
@FeepingCreature

Oh man I don't remember. I'll have a look during the week I guess, but don't let that stop this merge. So long as the existing tests pass I'm happy. (This case wasn't on my radar because we never use nontrivial assignments/copy constructors anywhere.) Definitely good on you for testcasing the weird things.

Rebindable2 assigned to memory which was not properly initialized when the correct solution in such a situation is to use copyEmplace. Assignment works when assignment is just a memcpy, but in the general case, opAssign needs to have a properly initialized object in order to work correctly.

Hang on, I don't understand this. The whole point of the code is that it doesn't call opAssign, so why would opAssign make problems here?

Rebindable2 assigned to memory which was not properly initialized when the correct solution in such a situation is to use copyEmplace. Assignment works when assignment is just a memcpy, but in the general case, opAssign needs to have a properly initialized object in order to work correctly.

Hang on, I don't understand this. The whole point of the ubyte[]/void[] Payload code is that it doesn't call opAssign (or copy constructors), so why would opAssign make problems here?

I don't know of any situation where not calling opAssign or copy constructors when they exist is the right thing to do when copying an object unless you're moving the object instead of copying it. Otherwise, you just break how the type works. If an object is assigned to or copied, it needs to be done properly. I honestly do not understand why the version where useQualifierCast is false even exists. I've tried to test for it to make sure that it works properly, but I don't know what problem it's trying to solve - especially since Rebindable2 has no tests whatsoever prior to these changes. The only testing it's had is the testing that's done for the code that uses it.
That being said, the main issue that is being fixed here is that the existing code assigns to memory that hasn't been properly initialized, and that only works properly if there is no opAssign. Without these changes, in the case where useQualifierCast is true, opAssign is being used even though the memory hasn't been initialized properly. This fixes it so that in such a case, copyEmplace is used instead so that a copy is created and then moved into the uninitialized memory. In the case where useQualiferCast is false, this also fits, since it's specifically not using assignment at all (and if it's also supposed to avoid copying, that should have been tested for along with assignment). It was previously copying without calling the copy constructor or postblit constructor, but that's just wrong behavior for any use case that I'm aware of, because any type that defines those is going to designed with the idea that they will be called for any copy.
What this commit does not fix is the issue where copying Rebindable2 does not trigger a copy constructor or postblit constructor if T has one, and useQualifierCast is false (since to fix such a case, Rebindable2 needs the appropriate postblit constructor or copy constructor, and the compiler does not generate one, because a static array of ubyte or void is used). In such a case, the object is not copied correctly and gets blitted instead, but that's the behavior that it's had since Rebindable2 was added.

Actually, looking at the code again, the copy constructor was being called before in the case where useQualifierCast was false. It's just that instead of using copyEmplace, it created a union to copy the object into and then didn't call the destructor on the union, presumably because you don't want to destroy the object, because the array holds its memory at that point. However, copyEmplace should make that trick unnecessary. So, the copy was being done correctly there, but the copy constructor or postblit constructor still isn't called when the Rebindable2 as a whole is copied and useQualifierCast is false.
Either way, the main issue that I was trying to fix here was that opAssign was being called on uninitialized memory when useQualifierCast is true. That needs to use copyEmplace instead, which it now does with these changes.
So, the useQualifierCast is true case should be fixed, and the useQualifierCast is false case should be effectively unchanged. It's just that it now uses copyEmplace to make the copy into the array instead of creating the extra union to do the copy without triggering the destructor.

I don't know of any situation where not calling opAssign or copy constructors when they exist is the right thing to do when copying an object unless you're moving the object instead of copying it.

The way that Rebindable2 is intended to model this situation is that the value is first destroyed and then recreated. For an example of a type that must bypass opAssign, consider a humble immutable struct S { SysTime a; }. SysTime has opAssign, but it can obviously not be called in an immutable variable.


16279 - Custom downloader

Contains new parallel custom download manager to download large files faster. It uses http requests with "Range" header param to specify part size. Supports proxy also. It was developed quite fast and some parts of code could be later improved.

Stay tuned, more comments will be added.


Small refactoring

@tony-xia
This is interesting. Could you:

File an issue that describes the bug you're fixing.
In the fix, we need a test (or 2) that fails before the fix and passes after.

Thanks @cleemullins . I am not sure whether I can find one test (or 2) that fails before the fix and passes after, because it's a code refactoring which keeps the same input, output and logic.

I pushed this code to a branch and ran our CI system against it. That's found here:
https://github.com/microsoft/botbuilder-dotnet/pull/2023
Based on that, I'm merging this in.


Ceph :How to specify NIC name for cluster network

Hi,
As part of ceph-deployment , For Cluster network, is ther any way to specify NIC name instead of subnet.?
I have two same subnets configured for different interfaces.

I don't think so because that's how ceph in using the cluster network value in the ceph config.
http://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#cluster-network
http://docs.ceph.com/docs/nautilus/rados/configuration/network-config-ref/#id2

Thanks @dsavineau


AWS EKS Istio routing does not work

Platform: EKS, Minikube
k8s: v1.10.4
Istio: v1.0.2
So I'm stuck and have no clue what to do I have two namespaces with automatic Istio injection: demo and default. I've created Deployment inside of each namespace:
---
kind: Service
apiVersion: v1
metadata:
  name: demo-app-svc-2
  namespace: demo
  labels:
    app: demo-app-2
spec:
  selector:
    app: demo-app-2
  ports:
  - name: web
    port: 80
    targetPort: web
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: demo-app-2
  namespace: demo
  labels:
    app: demo-app-2
spec:
  replicas: 2
  selector:
    matchLabels:
      app: demo-app-2
  template:
    metadata:
      labels:
        app: demo-app-2
    spec:
      containers:
      - name: demo-app
        image: shmatovd/demo-app:0.0.9
        ports:
        - name: web
          containerPort: 8080

---
kind: Service
apiVersion: v1
metadata:
  name: demo-app-svc
  labels:
    app: demo-app
    version: v1
spec:
  selector:
    app: demo-app
  ports:
  - name: web
    port: 80
    targetPort: web
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: demo-app
  labels:
    app: demo-app
    version: v1
spec:
  replicas: 1
  selector:
    matchLabels:
      app: demo-app
      version: v1
  template:
    metadata:
      labels:
        app: demo-app
        version: v1
    spec:
      containers:
      - name: demo-app
        image: shmatovd/demo-app:0.0.9
        ports:
        - name: web
          containerPort: 8080

And created some routing rules with rewrite from /reviews to /metrics
---
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: demo-app-vs
spec:
  hosts:
  - demo-app-svc.default.svc.cluster.local
  http:
  - match:
    - uri:
        prefix: /reviews
    rewrite:
      uri: /metrics
    route:
    - destination:
        host: demo-app-svc.default.svc.cluster.local
        subset: v1
  - route:
    - destination:
        host: demo-app-svc.default.svc.cluster.local
        subset: v1
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: demo-app-dr
spec:
  host: demo-app-svc.default.svc.cluster.local
  subsets:
  - name: v1
    labels:
      version: v1

when I'm trying to curl from demo namespace:
curl demo-app-svc.default.svc.cluster.local/reviews

I got no rewrites, got no logs about this request in Envoy sidecars on both Pods. Do I do something wrong? But if I try to point this routing to some external resource like wikipedia.org it works perfectly!
Istio-system namespace
NAME                                        READY     STATUS    RESTARTS   AGE
istio-citadel-769b85bf84-762dd              1/1       Running   0          20h
istio-egressgateway-677c95648f-mz555        1/1       Running   0          20h
istio-galley-5c65774d47-c8nq9               1/1       Running   0          20h
istio-ingressgateway-6fd6575b8b-4p55z       1/1       Running   0          20h
istio-pilot-65f4cfb764-pl8xr                2/2       Running   0          20h
istio-policy-5b9945744b-zgjv8               2/2       Running   0          20h
istio-sidecar-injector-75bfd779c9-x9jm5     1/1       Running   0          20h
istio-statsd-prom-bridge-7f44bb5ddb-6fjcv   1/1       Running   0          20h
istio-telemetry-5fc7ccc5b7-dxxhp            2/2       Running   0          20h
prometheus-84bd4b9796-btgj2                 1/1       Running   0          20h

demo namespace
NAME                              READY     STATUS    RESTARTS   AGE
pod/demo-app-2-7b747bc88f-6tdnr   2/2       Running   0          24m
pod/demo-app-2-7b747bc88f-jpv2l   2/2       Running   0          24m

NAME                     TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)   AGE
service/demo-app-svc-2   ClusterIP   <IP_ADDRESS>   <none>        80/TCP    24m

NAME                         DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/demo-app-2   2         2         2            2           24m

NAME                                    DESIRED   CURRENT   READY     AGE
replicaset.apps/demo-app-2-7b747bc88f   2         2         2         24m

default namespace
NAME                           READY     STATUS    RESTARTS   AGE
pod/demo-app-69b6ddd94-ms52g   2/2       Running   0          19h

NAME                   TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)   AGE
service/demo-app-svc   ClusterIP   <IP_ADDRESS>   <none>        80/TCP    19h
service/kubernetes     ClusterIP   <IP_ADDRESS>       <none>        443/TCP   1d

NAME                       DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/demo-app   1         1         1            1           19h

NAME                                 DESIRED   CURRENT   READY     AGE
replicaset.apps/demo-app-69b6ddd94   1         1         1         19h

@dshmatov Istio has a naming porting convention.

The port names must be of the form protocol-suffix with http, http2, grpc, mongo, or redis as the protocol in order to take advantage of Istio’s routing features.

Please, try to rename your service port to something like this:
kind: Service
apiVersion: v1
metadata:
  name: demo-app-svc-2
  namespace: demo
  labels:
    app: demo-app-2
spec:
  selector:
    app: demo-app-2
  ports:
  - name: http-web   # <------------------------------------
    port: 80
    targetPort: web

Thank you @eduardobaitello ! It helped me a lot, but I was able to find out the solution by pointing Service to 8081 (you may use whatever port you want, except 80 ) and route traffic in VirtualService destination rule to this port, like:
kind: Service
apiVersion: v1
metadata:
  name: demo-app-svc-2
  namespace: demo
  labels:
    app: demo-app-2
spec:
  selector:
    app: demo-app-2
  ports:
  - name: web
    port: 8081    # <---------------------------------------------------
    targetPort: web

Also, see https://istio.io/docs/setup/kubernetes/spec-requirements.


No license file.

The repository has no MIT License file. It should be added for easier categorization and identification.

Fixed


Nested recyclerViews with firestoreRecyclerAdapter not showing all items in the subcollection of nested Recycler

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a GitHub issue.
For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: _____
Android OS version: _____
Google Play Services version: _____
Firebase/Play Services SDK version: _____
FirebaseUI version: ____

Step 3: Describe the problem:
Steps to reproduce:

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
// TODO(you): code here to reproduce the problem

@ImakeDepressedBots please use the issue template when filing an issue.  For general programming help, try StackOverflow.


imp(fee): capital efficient fee middleware

Description
closes: #5509
Commit Message / Changelog Entry
imp(fee): capital efficient fee middleware

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Also, if the fees are all in one-denom, then we should definitely have tests that the multi-denom case works as we expect

@colin-axner and @crodriguezvega, I will open a separate issue and PR for docs, since this PR is time sensitive and should not be blocked on documentation nits

@AdityaSripal I think the tests have a healthy mix of hard-coded (like defaultFees) and computed values. If all values were hard-coded, it'd have been a nightmare to update tests.
Also, the current state of tests reflects the state of tests before the PR. I think such a refactor should be a separate issue if others agree to it


[1.11] Don't bind dcos-dns on k8s cluster dns ip address

High-level description
dcos-net binds dns servers on all ip addresses on start. It works fine on clean node, but once k8s start coredns on <IP_ADDRESS>, dcos-net tries to bind on this ip as well and crashes with eaddrinuse error. To fix it we have to add <IP_ADDRESS> ip address to the blacklist.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS-21486 Don't bind dcos-dns on k8s cluster dns ip address

Related tickets (optional)
Other tickets related to this change:
Checklist for all PRs

[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: [link to CI job test results for component]
[x] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
https://github.com/mesosphere/dcos-kubernetes/pull/386/files#diff-965e73ab742e8a42e6874e28e5edda57R358
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label [Ready For Review]

@urbanserj do we need this PR against 1.11-GA branch?

@asridharan 1.11.0-GA is for 1.11.0 GA, 1.11 is for 1.11.1, master for 1.12
1.11.0 GA https://github.com/dcos/dcos/pull/2533
1.11.1 https://github.com/dcos/dcos/pull/2529
1.12 https://github.com/dcos/dcos/pull/2531


fix: zero content case for Badge

FX-2777
Description
Badge is hidden when content is 0. In order to fix this behavior, the showZero property was added to the reused <MuiBadge/>.
How to test

Navigate to Badge Story book and display the badge with content={0}, it should be displayed (https://picasso.toptal.net/SPT-NULL-fix-zero-badges-case/?path=/story/components-badge--badge)

Screenshots

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure that tests pass by running yarn test
[x] Ensure that visuals tests pass by running yarn test:visual. If not - check the documentation how to fix visual tests
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests

Breaking change

[x] codemod is created and showcased in the changeset
[x] test alpha package of Picasso in StaffPortal

PR commands

List of available commands:

@toptal-bot run all - Run whole pipeline
@toptal-bot run build - Check build
@toptal-bot run visual - Run visual tests
@toptal-bot run deploy:documentation - Deploy documentation
@toptal-bot run package:alpha-release - Release alpha version

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-bot run all

Ran yarn test, same errors as in master (will be fixed in https://github.com/toptal/picasso/pull/2764)


Drives not showing up on home screen

Describe the bug
Drives do not appear on home screen until opening one of the library folders, and sometimes not even that works.
To Reproduce
Steps to reproduce the behavior:

Open files
Go to click on the C: Drive
wait where did it go

Expected behavior
The buttons for the C: drive and all other drives should be there
Screenshots

Desktop (please complete the following information):

OS Version: Windows 10 20H2 19042.782
App version: <IP_ADDRESS>

Additional context
Terminating, Resetting, and Repairing from the Settings app does not fix it, and neither does a simple close and reopen.
Log file
debug.txt
debug_fulltrust.txt

@KibSquib48 does it work when you type a drive path e.g. "C:\" in the navigationbar?

@KibSquib48 does it work when you type a drive path e.g. "C:\" in the navigationbar?

Typing in the path of one of the non-showing drives works and contents is displayed correctly.

Typing in the path of one of the non-showing drives works and contents is displayed correctly.

@SimonCDalzell do you experience the same? Is all your drives missing or just some? Could you post the log files a screenshot of the sidebar?
Are the drive missing from the sidebar only or from the home page too?

@SimonCDalzell do you experience the same? Is all your drives missing or just some? Could you post the log files a screenshot of the sidebar?
Are the drive missing from the sidebar only or from the home page too?

@KibSquib48 does it work when you type a drive path e.g. "C:" in the navigationbar?
Did the missing drive by chance reappear after installing OneDrive?

Installing OneDrive fixed it, and typing C:\ worked too

@KibSquib48 does it work when you type a drive path e.g. "C:" in the navigationbar?
Did the missing drive by chance reappear after installing OneDrive?

Installing OneDrive fixed it, and typing C:\ worked too

@SimonCDalzell do you experience the same?

are all your drives missing or just some?
could you post the log files and a screenshot of the sidebar?
are the drive missing from the sidebar only or from the home page too?

Just some. I believe the two drives that appear correctly have AHCI Hotswap enabled in the bios, while the ones that do not show have it disabled. I can test this in a sec.
I've attached the logs and a screenshot
The drives are missing from the sidebar and from the home page.
Probably worth mentioning that I'm on Windows 1809.

debug.txt
debug_fulltrust.txt

@SimonCDalzell do you experience the same?

are all your drives missing or just some?
could you post the log files and a screenshot of the sidebar?
are the drive missing from the sidebar only or from the home page too?

Just some. I believe the two drives that appear correctly have AHCI Hotswap enabled in the bios, while the ones that do not show have it disabled. I can test this in a sec.
I've attached the logs and a screenshot
The drives are missing from the sidebar and from the home page.
Probably worth mentioning that I'm on Windows 1809.

debug.txt
debug_fulltrust.txt

@SimonCDalzell could you verify that:
In Windows Settings > Privacy > File system permission is turned on for Files

@SimonCDalzell could you verify that:
In Windows Settings > Privacy > File system permission is turned on for Files

@SimonCDalzell could you verify that:
In Windows Settings > Privacy > File system permission is turned on for Files

That solved it. I had not received the prompt to enable file access. I then unplugged the drives U and V, and next time I launched files was prompted to enable file system permissions.

@SimonCDalzell could you verify that:
In Windows Settings > Privacy > File system permission is turned on for Files

That solved it. I had not received the prompt to enable file access. I then unplugged the drives U and V, and next time I launched files was prompted to enable file system permissions.

@SimonCDalzell thanks for confirmation, the prompt is only shown if Files is not able to access any fixed drive. What kind of drive where U and V? Usb HDDs?

@SimonCDalzell thanks for confirmation, the prompt is only shown if Files is not able to access any fixed drive. What kind of drive where U and V? Usb HDDs?

@SimonCDalzell thanks for confirmation, the prompt is only shown if Files is not able to access any fixed drive. What kind of drive where U and V? Usb HDDs?

All are internal drives connected to motherboard SATA ports. I unplugged U and V for an unrelated reason and happened to notice the prompt for file access. As far as I remember of my configuration the only difference between the drives is the AHCI Hotswap mode. Happy to investigate more thoroughly though.

@SimonCDalzell thanks for confirmation, the prompt is only shown if Files is not able to access any fixed drive. What kind of drive where U and V? Usb HDDs?

All are internal drives connected to motherboard SATA ports. I unplugged U and V for an unrelated reason and happened to notice the prompt for file access. As far as I remember of my configuration the only difference between the drives is the AHCI Hotswap mode. Happy to investigate more thoroughly though.

In my case it takes a long time to load drives.

In my case it takes a long time to load drives.

@XPoppyX thanks for the report. That's unfortunately expected after #3177. Working on it :)

@XPoppyX thanks for the report. That's unfortunately expected after #3177. Working on it :)

@XPoppyX, if you are in a position to do so, please try out the commits from PR #3257.
This PR should allow local drives to appear almost instantly, while allowing cloud drives to appear a little later.
Or if you disconnect drive D (which appears to be a network or mapped drive), does that populate your drives list quicker?

@XPoppyX, if you are in a position to do so, please try out the commits from PR #3257.
This PR should allow local drives to appear almost instantly, while allowing cloud drives to appear a little later.
Or if you disconnect drive D (which appears to be a network or mapped drive), does that populate your drives list quicker?

Start faster now (without disconnecting the network drive)

Start faster now (without disconnecting the network drive)

@XPoppyX  thanks for trying that. Would you say it is acceptable performance with your setup or do you think there is room for improvement?

@XPoppyX  thanks for trying that. Would you say it is acceptable performance with your setup or do you think there is room for improvement?

Well, any speed improvement is welcome, because it is still slower than the native file explorer.
2 seconds vs. less than 400 ms
I guess because it's UWP it complicates things more

Well, any speed improvement is welcome, because it is still slower than the native file explorer.
2 seconds vs. less than 400 ms
I guess because it's UWP it complicates things more

UWP is a strange beast. Unlike WinForms which had minimal difference between debug and release modes, UWP apps are so much slower in debug mode and it makes it hard to troubleshoot issues.
I agree Files is slower at launching that file explorer, and I have been personally trying to improve this. Still a few ideas in the works to propose but I don't know if it would be in the direction everyone wants to go.

UWP is a strange beast. Unlike WinForms which had minimal difference between debug and release modes, UWP apps are so much slower in debug mode and it makes it hard to troubleshoot issues.
I agree Files is slower at launching that file explorer, and I have been personally trying to improve this. Still a few ideas in the works to propose but I don't know if it would be in the direction everyone wants to go.

I have an issue where instead of my OneDrive showing up, sometimes it shows the C Drive twice.

I have an issue where instead of my OneDrive showing up, sometimes it shows the C Drive twice.

@scottkuhl but if you open a new tab, the new tab sidebar shows the correct items, right?

@scottkuhl but if you open a new tab, the new tab sidebar shows the correct items, right?

@gave92 @scottkuhl I have had something similar occur while finishing the sync'd SharePoint folder integration. The last entry added to the side bar is intermittently duplicated, or even stranger, causing the last-1 item to be rendered as a duplicate of the last item.
Locally I have code logging all of the items discovered, logging the order they are being added to the sidebar, and then dump out the current items from MainPage.SideBarItems. What is confusing me is all of this logging shows exactly what I expect - everything is correct, in the right order, and there is no duplication. But the NavigationView is not displaying them as they appear in MainPage.SideBarItems.
I am digging through the microsoft-ui-xaml repo now for related issues to see if this is a known problem.

@gave92 @scottkuhl I have had something similar occur while finishing the sync'd SharePoint folder integration. The last entry added to the side bar is intermittently duplicated, or even stranger, causing the last-1 item to be rendered as a duplicate of the last item.
Locally I have code logging all of the items discovered, logging the order they are being added to the sidebar, and then dump out the current items from MainPage.SideBarItems. What is confusing me is all of this logging shows exactly what I expect - everything is correct, in the right order, and there is no duplication. But the NavigationView is not displaying them as they appear in MainPage.SideBarItems.
I am digging through the microsoft-ui-xaml repo now for related issues to see if this is a known problem.

@AndyMDoyle yeah that's what I meant by "visual duplicates", looks like a bug in NavigationView. Calling a Collection.Reset on the sidebar list usually fixes the visualization.

@AndyMDoyle yeah that's what I meant by "visual duplicates", looks like a bug in NavigationView. Calling a Collection.Reset on the sidebar list usually fixes the visualization.

@scottkuhl but if you open a new tab, the new tab sidebar shows the correct items, right?

@gave92  That is correct.

@scottkuhl but if you open a new tab, the new tab sidebar shows the correct items, right?

@gave92  That is correct.

@KibSquib48 This issue should be resolved with #3350.


Sample configs for diff deployments scenarios

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
Hi - Thanks for the awesome product!
Given the variety of config options , it would be  really helpful if we have sample configs 1) with proxy 2) multi-provider for Ollama , Llama.ccp etc by model. 3) Embedding providers , others that u think would help get the best out of this.
Thanks!
Solution
No response

@aiseei this is a great idea! I created 2 initial examples here. If you have other ideas, let me know and I'll gladly add those as well (or am welcoming of PRs) https://docs.continue.dev/setup/examples


Handle overflows while calculating cost of the block in gas-metering algorithm

Copy from https://github.com/paritytech/wasm-instrument/pull/29

As we already discussed with @SabaunT,
I offer approach where Cost { overflows: Vec<u8>, remainder: u32 }, where sum of overflows elements equals amount of times we should call gas(u32::MAX)
It's more optimal by memory and reduces amount of allocations in overflow cases.
For example, if we ran into 5 overflows, current approach will twice reallocate vec and contain [u32::MAX, u32::MAX, u32::MAX, u32::MAX, u32::MAX, reminder], while offered one: Cost { overflows: [5], reminder }

Whether this solution is checked on gear-tech/gear#1353

Yes, it was tested with current changes and works good. By the way, there is a new cost_overflow test in gas_metering module, which tests the expected set of instructions in case of overflows.


Detect duplication of name

Currently "John Doe" and "John  Doe" are considered distinct.
Is this behavior desirable?

They should be considered as a same person

Be careful. Does this count as overzealous input validation?

It could be because although no names contain >= 2 white spaces between words, if users wrongly type someone's name with many spaces they can't edit it to the right one if we consider "John Doe" and "John  Doe" as the same name.
Then should we change the way of checking validation of name?


SWITCHYARD-2892 Maintain SwitchYard console

Can one of the admins verify this patch?

ok to test

merged


iOS: Enable views to be nested within

Previously, only Text and Image could be nested within Text. Now, any
view can be nested within Text. One restriction of this feature is
that developers must give inline views a width and a height via
the style prop.
Previously, inline Images were supported by using iOS's built-in support
for rendering images with an NSAttributedString via NSTextAttachment.
However, NSAttributedString doesn't support rendering arbitrary views.
This change adds support for nesting views within Text by creating one
NSTextAttachment per inline view. The NSTextAttachments act as placeholders.
They are set to be the size of the corresponding view. After the text is
laid out, we query the text system to find out where it has positioned each
NSTextAttachment. We then position the views to be at those locations.
This commit also contains a change in RCTShadowText.m
_setParagraphStyleOnAttributedString:heightOfTallestSubview:. It now only sets
lineHeight, textAlign, and writingDirection when they've actually changed. This
avoids unnecessarily dirtying the shadow tree which is important in some inline
view scenarios.
Here's a screen shot demoing an inline view and an inline image:

Adam Comella
Microsoft Corp.

@rigdern updated the pull request.

@rigdern updated the pull request.

@facebook-github-bot import

Thanks for importing. If you are an FB employee go to Phabricator to review.

Thanks for importing. If you are an FB employee go to Phabricator to review.

This looks great!
I'm not sure the way that you're handling vertical alignment is correct though. It seems like you're basically aligning the views to the top of the line rect, but I would expect them to be aligned with the baseline of the text by default, which is how iOS normally positions text attachments, and how inline images were aligned on iOS previously.
Baseline alignment isn't always desirable, and we should probably consider ways to allow the developer to adjust the offset, as well as ensuring consistency with Android, but for now I'd like to ensure we remain consistent with how this worked on iOS previously.

@rigdern updated the pull request.

@nicklockwood I updated the PR to use 0.0 for the y coordinate like it did before my change. Hopefully the PR is good to go now.
Here are some questions/comments about future work:
If I wanted to support customizing the vertical alignment of inline views, what would you recommend? A new style on View that only works on inline views and is a no-op on other views (e.g. inlineVerticalAlignmentwith values like "bottom" and "baseline")?
We're currently working on adding a similar feature to the Android implementation.
Adam Comella
Microsoft Corp.

This looks good. I'll need to do some testing with our internal apps just to check that dropping support for automatically detecting image size hasn't broken anything.

If I wanted to support customizing the vertical alignment of inline views, what would you recommend? A new style on View that only works on inline views and is a no-op on other views (e.g. inlineVerticalAlignmentwith values like "bottom" and "baseline")?

That's a good question. A simple solution would just be to respect the existing "top" and/or "marginTop" styles to adjust the view by a relative offset. That should work pretty-much out of the box on iOS since NSTextAttachment respects the bounds.origin and treats it as a relative offset (although we might want to flip the y value since it follows the Mac convention of y pointing up the screen instead of down IIRC).
Having a verticalAlignment property that follows the CSS spec would be nice as well, although personally I've never found it very useful in practice because visually aligning images with text usually involves pixel-level adjustment.
Either way, the key thing is that the behavior should match between iOS and Android so we don't need to fork the styles every time we use inline views.

@facebook-github-bot import

cc @dmmiller for thoughts on alignment API

Thanks for importing. If you are an FB employee go to Phabricator to review.

@rigdern the alignment is still wrong. It seems like the problem is that [layoutManager boundingRectForGlyphRange:range inTextContainer:textContainer] doesn't return the actual frame of the NSTextAttachment, but a frame whose height matches the height of the row.
You've compensated for this by replacing the width/height with the actual width/height of the view (line 141 or RCTShadowText), but that doesn't fix the origin, which is still incorrect as it places the view at the top of the row frame instead of aligning it with the baseline.
I was able to improve this slightly by changing the logic to
  CGRect childFrame = [layoutManager boundingRectForGlyphRange:range inTextContainer:textContainer];
  childFrame = (CGRect){{
    RCTRoundPixelValue(childFrame.origin.x),
    RCTRoundPixelValue(childFrame.origin.y + childFrame.size.height - height)
  }, {
    RCTRoundPixelValue(width),
    RCTRoundPixelValue(height)
  }};

So that the view is aligned with the bottom of the frame instead of the top, but that's still not the same as the baseline, which is what it should be. There's presumably some way to get the actual NSTextAttachment frame instead of the frame for the placeholder glyph, but I couldn't find it after a few minutes of searching.

@rigdern updated the pull request.

@nicklockwood Sorry about that. The vertical alignment should be good now. I tested it with short and tall images and compared the results with the original implementation.
Here's what the UIExplorer example now looks like:

I did this as a separate commit so that you can see exactly what I changed. Let me know if you need me to squash it. In general, does your team prefer pull request revisions to be done as separate commits or squashed into the original commit?

@nicklockwood Is there anything else blocking this change from being merged?

@rigdern sorry for the delay. This looks good, thanks!
@facebook-github-bot import

Thanks for importing. If you are an FB employee go to Phabricator to review.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

This regressed one of our application startup metrics by 5%. Any clue where this could be coming from? Otherwise, I'd prefer to revert this for now, until we get this metrics-neutral.

@javache Without doing some analysis, I can't tell which part of this change slowed down your startup metrics. Just reverting this change fixes your startup metrics? Does your app use a lot of inline images?
@nicklockwood, do you have any thoughts on this?

@rigdern unfortunately I haven't been able to work out what's causing the regression. I'm going to revert this for now and try landing it again next week after our branch cut.
I did find a few obvious inefficiencies, such as not stripping the views associated with RawText (these are regular UIViews, so your check for RCTText views doesn't strip them), but I tried fixing that and it had no measurable effect.
I'm guessing that the problem must be related to the extra logic added to the layout, but I don't have any ideas beyond that right now.

@nicklockwood Can you provide any details about the scenario such as how many inline images there are and how the perf measurements are being taken (e.g. are you using some markers built into RN or does the app do the instrumentation itself? If the app is doing its own instrumentation, what specific timeframe is being measured and what is being used to take the measurements?)

@nicklockwood I see that this got checked in again. Were you able to fix the performance problem? If so, what was the cause?

@rigdern yes. I made some changes in RCTText which seemed to eliminate the regression, I'll annotate them for you in the commit.

@nicklockwood Thanks for the explanations!


excel list out of existing vocabulary in cms

Motivation

possibility to extract an excel list containing vocabulary already existing in the cms for each "berufsbild"
Proposed Solution

https://issues.tuerantuer.org/browse/IDEA-58
Alternatives

Additional Context

@timoludwig I have tried a solution for this issue on the branch feature/export_to_excel. I didn't open a PR for it, because I'm not sure that what I have is the best solution for it. I tried using the csv library. After playing around with it for a while now, I'm not sure this is the best library for it, because of two reasons: a. the csv library only export to csv files. I think in most cases this should be sufficient since Excel supports also csv, but there might be edge-cases in which we would need a .xls(x)-file and b. as far as I could tell it's not possible to create more than one CSV file. However I would expect, that if I select three disciplines that I get one csv file for every discipline (ideally also named after that discipline). But since the export works via the return of the response, I couldn't figure out a way to have more than one file. For now I appended all selected discipline in the same file (also not sorted).
Do you think it's worth the effort to spend more time on the csv library and try to fix the mentioned issues or do you think I should rather try a different library (for example import-export or pandas)

@JoeyStk Before trying to implement the export ourselves, I think we should first try to get this up and running:
https://django-import-export.readthedocs.io/en/latest/
If it doesn't work out for whatever reason, we can continue looking for alternatives that require more manual work.


Speed up compile of autohashable

@Lutzifer awesome work, mind solving the conflict that appeared after I merged your other pr?

solved

Should this also update templates test project?

yeah I think so

ah, didn't run the tests for this one actually 🤭

@krzysztofzablocki I will think about joining

@Lutzifer any updates on the templates here?


problem executing code

### Are there solutions to make the code work?
These are the errors that made the program not work:
The autoreload extension is already loaded. To reload it, use:
%reload_ext autoreload
------------ Parameters -------------
camera_lr: 0.01
ckpt: checkpoint/stylegan2-ffhq-config-f.pt
coarse_min: 32
color_transfer:<PHONE_NUMBER>0.0
contextual: 0.1
cx_layers: ['relu3_4', 'relu2_2', 'relu1_2']
e4e_ckpt: checkpoint/e4e_ffhq_encode.pt
e4e_size: 256
encoder_ckpt: checkpoint/encoder/checkpoint_b.pt
encoder_name: b
encoder_size: 256
eye: 0.1
gaussian: 0.75
generator_size: 1024
init_latent: None
input: dataset/أنا1_01.png
log_dir: log/
log_freq: 10
log_visual_freq: 1000
lr: 0.1
mix_layer_range: [10, 18]
noise_ramp: 0.75
noise_regularize: 50000.0
noise_strength: 0.0
rand_seed: None
recon_size: 256
results_dir: results/
spectral_sensitivity: b
vgg: 1
vggface: 0.3
wplus_step: [250, 750]
b-G0.75-init(10,18)-s256-vgg1-vggface0.3-eye0.1-color1.0e+10-cx0.1(relu3_4,relu2_2,relu1_2)-NR5.0e+04-lr0.1_0.01-c32-wp(250,750)
FileNotFoundError                         Traceback (most recent call last)
 in ()
19         results_dir="results/"
20 ))
---> 21 main(args)
5 frames
/content/Time-Travel-Rephotography/projector.py in main(args)
113     # initialize
114     with torch.no_grad():
--> 115         init = Initializer(args).to(device)
116         latent_init = init(imgs_orig)
117
/content/Time-Travel-Rephotography/tools/initialize.py in init(self, args)
114
115         assert args.encoder_size is not None
--> 116         self.color_encoder = create_color_encoder(args)
117         self.color_encoder.eval()
118         self.color_encoder_size = args.encoder_size
/content/Time-Travel-Rephotography/tools/initialize.py in create_color_encoder(args)
73 def create_color_encoder(args: Namespace):
74     encoder = Encoder(1, args.encoder_size, 512)
---> 75     ckpt = torch.load(args.encoder_ckpt)
76     encoder.load_state_dict(ckpt["model"])
77     return encoder
/usr/local/lib/python3.7/dist-packages/torch/serialization.py in load(f, map_location, pickle_module, **pickle_load_args)
592         pickle_load_args['encoding'] = 'utf-8'
593
--> 594     with _open_file_like(f, 'rb') as opened_file:
595         if _is_zipfile(opened_file):
596             # The zipfile reader is going to advance the current file position.
/usr/local/lib/python3.7/dist-packages/torch/serialization.py in _open_file_like(name_or_buffer, mode)
228 def _open_file_like(name_or_buffer, mode):
229     if _is_path(name_or_buffer):
--> 230         return _open_file(name_or_buffer, mode)
231     else:
232         if 'w' in mode:
/usr/local/lib/python3.7/dist-packages/torch/serialization.py in init(self, name, mode)
209 class _open_file(_opener):
210     def init(self, name, mode):
--> 211         super(_open_file, self).init(open(name, mode))
212
213     def exit(self, *args):
FileNotFoundError: [Errno 2] No such file or directory: 'checkpoint/encoder/checkpoint_b.pt'

For the "File not found" error, it's because too many people have downloaded the files so Google isn't letting you DL them via API.
I downloaded the models to my gdrive, then mounted it and copied directly from my gdrive. That solved that issue.

For the "File not found" error, it's because too many people have downloaded the files so Google isn't letting you DL them via API.
I downloaded the models to my gdrive, then mounted it and copied directly from my gdrive. That solved that issue.

Hello, please forgive my ignorance but I am somewhat new to Colab, albeit I have been using Time Travel Rephotography for a month. The solution you have offered there sounds like it will fix the current issue (one I posted a couple of days ago here) but I am not sure 'where' to download the models from nor how to 'copy' into the TTR Colab demo. Would you be so kind as to write a quick explainer?


Changed definition of pauli-Y gate

Description of Issue (if Applicable)
Definition of pauli-Y gate (in terms of inner product) was incorrect. The signs were flipped.
Fixes #your_issue_number_here
What Changes Were Made?
Changed the signs.
How Was This Tested?
Using a pen and a piece of paper.
Please provide any screenshots if applicable.
Anything Else We Should Know

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


Adding a public typealias for Database

Description
Added a CouchDatabase typealias to the Database class.
Motivation and Context
Because of https://github.com/Kitura/Swift-Kuery-ORM/issues/22
Checklist:

[x] I have submitted a CLA form
N/A If applicable, I have updated the documentation accordingly.
N/A If applicable, I have added tests to cover my changes.

@mbarnach Anyway the PR gets merged?

@dannys42 This PR as the same goal as https://github.com/Kitura/Swift-Kuery-ORM/pull/132, if you're able to merge it as well


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 9a7b61d:
:clock10: Full build: the average full build time is 7 min(s) 22 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 9a7b61d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Tainted canvas error if you are cropping an image from url

ImageCropper.tsx:273 Uncaught (in promise) DOMException: Failed to execute 'toBlob' on 'HTMLCanvasElement': Tainted canvases may not be exported.

This is an issue with your crop preview and not specific to this lib - checking google - https://www.google.com/search?q=Uncaught+(in+promise)+DOMException%3A+Failed+to+execute+'toBlob'+on+'HTMLCanvasElement'%3A+Tainted+canvases+may+not+be+exported.&oq=Uncaught+(in+promise)+DOMException%3A+Failed+to+execute+'toBlob'+on+'HTMLCanvasElement'%3A+Tainted+canvases+may+not+be+exported.&aqs=chrome..69i57.1028j0j7&sourceid=chrome&ie=UTF-8

You can add crossorigin="anonymous" to ReactCrop

Yeah it's not your issue but leave a note or something, not all of us are canvas experts, that's why we use the library. Thank you for your great work but don't migrate issues.

You can add crossorigin="anonymous" to ReactCrop

this, doesnt load the image if seen on network tab, it just shows failed.


Delete hdfs directory

If so how to download SST files which will be ingest into nebula?

@darionyaphet http://jira.nebula-graph.io:8081/projects/NG/issues/NG-98?filter=allissues

2.0 is also need, so close it.


chore(kube-testing): Implement comments from PR 3145

copilot:all

Current dependencies on/for this PR:

main

PR #3164   👈

This comment was auto-generated by Graphite.


3rd party libraries in use are tracked and analyzed for security risk

DevSecOps team needs to tell us what tools to use to do this.
Then we will update all the component team tickets with that information.

Per #308, DevSecOps recommendation is for everyone to use the same tool. Both Snyk and SourceClear are good options here. Prior recommendation from the team is to start with Snyk since it's open source (SourceClear requires procurement).


Bring back python botocore and fix build failure.

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Bring back reverted commit c48477cee344f9733dc423c73daef88f02d9404b
Fix build issues by moving BR on python3-pip from check section to general BR section.
This is because the build pyproject-rpm-macro 'pyproject-wheel' uses 'pip wheel' to generate project wheel artifacts.
This requires dependency on pip even when building without check section

Change Log

Adding python-botocore Spec in CBL-Mariner
Fix build issues by moving BR on python3-pip from check section to general BR section.
Fix check section. Add BR dependencies.

Does this affect the toolchain?
NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Local Build

Yes, the test (%check) sections pass for this package.


Don't throw 500 error on attempted directory browse

This should be a 403 error instead, because the --static mechanism doesn't allow directory browsing.

Spotted this while working on #731


feat(time): add meridian template {a}/{A}

Brief Information
This pull request is in the type of:

[ ] bug fixing
[x] new feature
[ ] others

What does this PR do?
I added a meridian template to the time format.

Fixed issues

Details
Before: What was the problem?
There wasn't a template to show the meridian.

After: How does it behave after the fixing?
Now it's possible to use the template {a} | {A} to show the meridian on the time formatting

Document Info
One of the following should be checked.

[ ] This PR doesn't relate to document changes
[x] The document should be updated later
[ ] The document changes have been made in apache/echarts-doc#xxx

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

Thanks for the contribution @maurodesouza! I've waited a long time for this one.
@plainheart any idea when you plan on releasing 5.5.1?


Need to add oAuth

@lpww just fleshed out a simple API client at lunch.
Usage:
// In script in repo
var stravaApi = require('./')({
    token: 'xxxxxxx'
});

// Get club activities
stravaApi.get('clubs/133132/activities', function(err, data) {
    console.log(data);
});

On second thoughts, you might have access issues to that club.  Try a simple method:
stravaApi.get('athlete', function(err, data) {
    console.log(data);
});

This is awesome, I'll test it out a little later! I'l also look into the oauth handler

Yeah should be a simple token exchange.  The ingredients will be there.  Some sort of generate login link and then token exchange method should cover it.  The actual oAuth stuff will be handled by the API

So it would be something like
stravaApi.authenticateUser(callback); //returns a login link that calls the callback once authorized

Probably a bit convoluted.  We'd then have to make a system for handling the login flow too.  We don't want to dictate that, so given the client is initialised with:
var stravaApi({
    id: 000,
    secret: 'xxx'
});

You need a method that returns an authentication URL with options to set the scope: http://strava.github.io/api/v3/oauth/#get-authorize
Then with the returned code, you need to swap it for an access_token: http://strava.github.io/api/v3/oauth/#post-token
So we don't want to dictate how a user actually gets the code for exchange, just allow them to make that exchange and obtain a permanent access_token!

Oh yeah, that solution would only return the temporary token to the user. I forgot that you have to do the exchange. It's the permanent token we want to return!

Fixed in #3


Add user type checking decorator

Add a decorator that will check to see if the user is a bot/has specific permissions.
This will also need users to have specific permission levels. A new DB table will need to be added. And some new columns.

https://discordpy.readthedocs.io/en/rewrite/ext/commands/api.html#discord.ext.commands.has_permissions
Might this decorator do for permission checks?
Also the user object has .bot to check if it is a bot.
https://discordpy.readthedocs.io/en/latest/api.html#discord.User.bot

I think this would be the least complicated way to manage it. I will be adding a permission tier check that uses user roles and can be set at runtime in the config.ini file.