Chapter08 - run_model.py return size mismatch

Hi after running the "train_model_conv.py", I tried to test the saved models using "run_model.py" using the below command but got size mismatch error as stated below.  Any idea how I could fix it?  Many thanks in advance.
Chapter08$ python3 run_model.py -d data/YNDX_160101_161231.csv -m saves/runs/mean_val-0.824.data -b 10 -n test --conv
Reading data/YNDX_160101_161231.csv
Read done, got 131542 rows, 99752 filtered, 0 open prices adjusted
Traceback (most recent call last):
File "run_model.py", line 35, in
net.load_state_dict(torch.load(args.model, map_location=lambda storage, loc: storage))
File "/home/lamhk/Deep-RL-Hands-On/venv/Deep-RL/lib/python3.5/site-packages/torch/nn/modules/module.py", line 719, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for DQNConv1D:
size mismatch for fc_val.0.weight: copying a param of torch.Size([512, 256]) from checkpoint, where the shape is torch.Size([512, 5376]) in current model.
size mismatch for fc_adv.0.weight: copying a param of torch.Size([512, 256]) from checkpoint, where the shape is torch.Size([512, 5376]) in current model.

Hi!
Amount of bars passed to the model (-b option) should match to amount used during the trainig. This value is set here: https://github.com/PacktPublishing/Deep-Reinforcement-Learning-Hands-On/blob/master/Chapter08/train_model_conv.py#L16, and equal to 50 by default. So, you should pass -b 50 in command line to match.
Alternative would be to change the constant in train_model_conv.py.

Hi Shumuma, it works as explained about.  Many thanks.


New Release request

Please publish a new release. I'd like to pull the code that allows for storing SSL key & cert as ENV variables. It will be much easier if they are in an official release instead of on the dev branch.
Please let me know if you'd like my assistance in documenting the changes to SSL options.
Also I noticed there are several spots where it says the gem is no longer available on RubyForge. However, your releases are being published on RubyGems without issue. I believe those disclaimers could be removed.

Sure I will get a new release out within the next day or two.
The rubyforge comment can indeed be removed.  It has been there ..... a long time.  Since the gem moved from rubyforge+subversion to gitorious/git.

Done as of a few minutes ago.

Wonderful! Thank you.
On Mon, Dec 10, 2018, 2:31 PM Guy M. Allard<EMAIL_ADDRESS>wrote:

Done as of a few minutes ago.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stompgem/stomp/issues/156#issuecomment-445942103, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAbN5ATOborsDotk8G2oWuNXGjFYl8i4ks5u3rZ2gaJpZM4ZLZFb
.

Hi. The new release does not appear to have the work that allows for ssl keys to be stored as text. For example it does not appear to have this commit in it: https://github.com/stompgem/stomp/commit/99db128ea427dae4f283dc69c5d9a0387389e1d2


Prometheus configuration

Currently inside for the prometheus contained the configuration file is from the MiniSpace/compose/prometheus/prometheus.yml, an update will need for the service on the web.
http://<IP_ADDRESS>:9090/targets?search=

We will not use the prometheus as too much of resources are required.

Server resources are not available now.

Currently it stabilized, but we will not use the prometheus container on server nor graphana.


Prevent people to enter payment processor fee >= expense.amount

People are making the mistake to input the expense amount in the payment processor fee.
We're seeing it in the ledger and this is causing issues. This should be prevented.
Also, on this screen, it looks weird that the "total amount paid" is pre-populated.

According to
SELECT "ExpenseId", "TransactionGroup", MAX("createdAt")
FROM "Transactions"
WHERE "kind" = 'EXPENSE' AND "type" = 'DEBIT' AND "deletedAt" IS NULL GROUP BY "TransactionGroup", "ExpenseId" HAVING COUNT(*) = 2

This issue is especially happening in 2023 so it's certainly linked to the second aspect mentioned and a regression.

Also, on this screen, it looks weird that the "total amount paid" is pre-populated.

The 0.00 is a placeholder; the input is not pre-populated. Trying to submit in this state will trigger a validation error:

If this is a typo and the intent was to say that the input is not pre-populated (with the expense amount) then I agree we can pre-fill it.


Cannot import the db from the flaskblog folder

I have done your stuff as you teach in the youtube channel.
When i want to import the db from the flaskblog floder the conda is giving the error "ImportError: cannot import name 'db' from 'flaskblog' (E:\anoconda install\envs\flaskwork\lib\site-packages\flask_init_.py)" how can i solve this problem

how are you writing that just Give a screenshot, Maybe There Would Be A Problem, in writing you Import

here is the attached code

i am getting same error, did you guys find the solution??


Fix handling of socket connection refusal.

Currently if the socket was never started you get an error about
the service being started.  But if the service was started and later
stopped, you get a useless error.
This change causes the error to always be the same for connection refused.
The error message was also repeating the address twice which looked bad.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@jwhonce @baude PTAL
BTW pypodman does not work in release because we did not install python3-psutil as a requires in Fedora or RHEL spec.

LGTM

LGTM
@rhatdan I'll put together a PR for the spec file update.

@rh-atomic-bot r+

@rh-atomic-bot retry

@jwhonce The reason the second field was removed, was that it was printing twice.  The first field was enough to get it printed.


Wrong title/name

Observed/problematic behavior
The name of the languages options in the settings is wrong

Expected behavior
It should say something like 'Languages"
Steps to reproduce
Go to Settings
Go to Other
Other
No response

see #1358


Multi-module support to identifier in ProvidedPlugin.

In case of needs to use "jquery" and "jquery-ui" by a same identifier like "$",

What kind of change does this PR introduce?
feature, refactoring

Did you add tests for your changes?
no

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
It is be able to add webpack config file by declaring first element as array like below.
new webpack.ProvidePlugin({
            $: [["jquery", "jquery-ui"]]
)}

And also it can be available to get specific property by adding elements like below.
new webpack.ProvidePlugin({
            UI : [["jquery", "jquery-ui"], "ui"]
)}

Thanks for you PR. sorry webpack@4 is deprecated. ( no features for this version )


Data Viewer available in regular debug

I don't know if the rest of the vsc allows this to be customized but it would be good to be able to reuse the existing Data Viewer in a regular debug session without running in an interactive notebook-esque environment.
Sometimes when debugging production code that deals with dataframes/numpy etc then it's beneficial to be able to inspect those dataframes in a tool rather than through programmatic inspection.

The variable inspection window is not configurable by us; it's entirely under VS Code's control. You might be able to get richer output through the debug console if either the object supports some richer output or you can pull in a package that can provide it for you.

How would I do that? Does the debug console support rich rendering? If so, are there any docs for that?

If by "rich rendering" you mean HTML, then no. That would once again require VS Code to add support for such a thing.

Dup of https://github.com/microsoft/vscode-python/issues/7063


Salsa: Not effectively using suggestion icons

Testing #2218

Also interesting: suggestions on console. shows them as methods which is good:

But suggestions on a commonjs module shows everything again using the blue brick icon:

Duplicate of #2169


Server zip file should not include the libraries folder, forge executable, or Minecraft

Including the correct forge installer and a script to kick that off will get the server set up by downloading the correct libraries for the server's operating system, as well as the Minecraft jar file.

This will be fixed with the next server release. It was a mistake on my part that Ewanm89 is fixing. The updated server pack for 9.4 should be released tomorrow.

This is sorted, so I'll close the issue. Only reason server files aren't already there is cos Curseforge is having trouble actually processing them for some reason


Message viewer tab missing page navigation buttons, displays only 100 but shows 500 messages consumed.

Describe the bug

Message viewer tab missing page navigation buttons
Select the topic.
Switch to messages tab (no filters).
Screen shows, 500 messages consumed
Data grid shows only 100 messages.
URL:
http://:8080/ui/clusters//all-topics//messages?filterQueryType=STRING_CONTAINS&attempt=1&limit=100&seekDirection=FORWARD&keySerde=String&valueSerde=String&seekType=BEGINNING&seekTo=0%3A%3A31979786
Even if i change limit=100 to limit=500 in url, still shows 100 events only.
http://:8080/ui/clusters//all-topics//messages?filterQueryType=STRING_CONTAINS&attempt=1&limit=500&seekDirection=FORWARD&keySerde=String&valueSerde=String&seekType=BEGINNING&seekTo=0%3A%3A31979786
Page navigation buttons are missing and unable to navigate pages for more events.
Set up

1. UI for Apache Kafka 10.14.2022 05:17:20 ( 2b5dd27 )
Steps to Reproduce

Steps to reproduce the behavior:

Expected behavior

Page navigation buttons should be added to navigate pages for events more than 100.
Screenshots

Additional context

Hey, thanks for the detailed report!
We've consumed 500 messages because it's the default polling limit, but displayed just 100 of them.
We know about the pagination regression (#2488), we'll take a look soon.
Please let me know if you have any other questions.
Closing as a duplicate.


Issue 9/support metrics for 90x

This supports reporting and QC of the 90X coverage samples.
Currently the 60X and 70X (autosome only) coverage metrics are not generated by the normal pipeline and getting these from NGRD team.
Scandium also provides these metrics but they are autosome+x, y.
Takes two input files:

Exemplar merge report (download from Exemplar LIMS)
TopMed-60-70-Cov-Autosome_only (from NGRD team)

Outputs an Excel file with two tabs:

TOPMed weekly report tab3
TMQC with results PASS or FAIL

New QC metrics cutoffs for 90X coverage samples are set to:

Average Coverage : 90X
Per coverage > 95% for 60X
Per coverage > 90% for 70X
Bases at Q20 > 205 x 10^9

NOTE:
Development of the testing and packaging will be done on other branches once this is approved and merged in.

@itsmarcy Thanks!  I completely agree with you about the packaging.  I  would like to do the packaging.  I will speak to you if I run into any issue.


BAccordion & item to script setup

Due to a previously unknown syntax limitation https://vuejs.org/guide/typescript/composition-api.html#syntax-limitations
A reference to an interface or object literal type in **the same file**
This means that the organization that was hoped to be brought from the src/types/components dir, is less useful. According to Hiws, a member of the original Boostrap-vue project, he heard that there was talks in the Vue discord server stating that importing types from other files may be released in Vue v3.3. For now, importing types from other files is not possible. So, perhaps in a future release of Vue, we may be able to clean up the code -- while also properly being able to import extension types. For now, we are stuck doing it the hard way.

For some odd reason, I think the original BAccordionItem.vue (the file before the move to it's own dir) file isn't deleting on the PR. Some weirdness going on there. If it doesn't get removed in this. I'll submit another PR with it removed along with the next couple of changes.

Also, a question, in future components, the build errors that are plaguing the library, they are usually stemmed from trying to assign a html attribute to null. This is allowed in Vue, as null or undefined values are not rendered, but TS comes with the error. If we simply leave them to be undefined, the attributes will not be rendered, this could possibly fix almost all of the "cannot assign null to HTMLelement..." errors. While also keeping them unrendered. Should we assign them to undefined instead?

looks like it was deleted. I will have to review the error as I am not familiar with the use case. Would you provide an inline example?

looks like it was deleted. I will have to review the error as I am not familiar with the use case. Would you provide an inline example?
looks good otherwise!

An example of our error or how vue doesn't render undefined?

An example of our error or how vue doesn't render undefined?

Yeah, its been a while since I last built the library and documentation but never came across the undefined error.

An example of our error or how vue doesn't render undefined?

Yeah, its been a while since I last built the library and documentation but never came across the undefined error.

In BFormSelect.vue
Type 'string | null' is not assignable to type 'string | undefined'.
  Type 'null' is not assignable to type 'string | undefined'.ts(2322)

runtime-dom.d.ts(794, 3): The expected type comes from property 'form' which is declared here on type 'ElementAttrs<SelectHTMLAttributes>'

This is similar to the other issues, however, undefined is an allowed value, which means it will be properly unrendered if we explicitly set it to undefined. This is mainly a TS issue, as if we had some sort of Vue/types then it would be addressed there that null is allowed as an attributes value.


Readme Improvement for SIFT ALGO

https://github.com/akshitagupta15june/Face-X/tree/master/Recognition-Algorithms/Recognition using SIFT
ADD MORE INFORMATION ABOUT THIS ALGO AND THE WAY OF EXECUTING IT.

Assign this to me

Mam,I have update readme of Resnet50,so please merge pull request of that also

Assigning to u

Pull request is generated for this
On Fri, Mar 12, 2021, 11:39 AKSHITA GUPTA @.***> wrote:

Assigned #305 https://github.com/akshitagupta15june/Face-X/issues/305
to @utkarsh147-del https://github.com/utkarsh147-del.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/akshitagupta15june/Face-X/issues/305#event-4448917471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARS4LCNIUCVURMJ5HEE5B23TDGVX5ANCNFSM4ZAXFANA
.


fix: Set button type to prevent reload of page when clicked

Description
For most browsers the default type of button is submit, this could cause the page to reload when clicked. Setting button type to button, prevents this behavior.
Fixes #15502
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
[Manual](https://github.com/appsmithorg/TestSmith/issues/2025)
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=48bba88

tested and verified on DP


feat: Extend User struct by adding Data, Name and Segment

Closes #466

@vladanpaunovic Can you please update the repo settings so that changes after a ✅ do not dismiss it 😄

@vladanpaunovic Can you please update the repo settings so that changes after a ✅ do not dismiss it 😄

Done. Let me know if it works (or not).


[PackageModel] Change ClangModule baseclass to Module

This looks like a monotonic improvement, although I think we need a number of cleanups in the areas this touches, but it is a little hard to articulate exactly what they are.

I plan to do some cleanup around here, starting with modulemap generation. It also needs unit test coverage


Graph layer drag refactor

Update graph example to maange drag state on its own, using mouse handlers and queryObject().
I branched off of #741, so once that's landed I'll just rebase this to remove the duplicate commit from #741.
(Also, looks like this is the first use of queryLayers() in our examples. Works like a charm, thx @Pessimistress 🙏 🙆

Along with #741, this will close #648.


fix: pin versions of ol and ol-mapbox-style

and remove openlayers from shared modules in module federation config.

Replaced by #1515.


DG: Update user stories to match current product

Fixes #252

Codecov Report

Merging #256 (69bd5b7) into master (6f2e1c2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #256   +/-   ##
=========================================
  Coverage     67.99%   67.99%
  Complexity      829      829
=========================================
  Files           121      121
  Lines          3034     3034
  Branches        441      441
=========================================
  Hits           2063     2063
  Misses          785      785
  Partials        186      186

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


fix(Line2): allow points to be added/removed, fix defineExpose ref

Problem
After initial creation, additional points could not be added to Cientos' Line2 component. The on-screen component would not display the additional points.
Solution
Recreate the Line2 geometry to accommodate the additional points.
Problem
The defineExposed instance is undefined because here ...
<template>
  <primitive
    :ref="lineRef"
    :object="line"
  />
</template>

... :ref should be ref.
Solution
Remove the colon from :ref.
Meta
Closes #493

Mysterious Netlify errors here. Seemingly unrelated to the PR, according to the Netlify build logs. Closing and resubmitting a new PR.


Speed up listdir

instead of fetching all objects starting with the first letter of the prefix (which is time consuming due to the sheer number of objects on the buckets), use the whole prefix, so for instance for the sentry-sdk package, the prefix to use would be:
sentry-sdk instead of just s
It does also change any - in the package name to _, as it seems to be that all packages with dashes in their name are stored with underscores instead.

Gosh, ok this is surprising to me. I can't fathom why we would only use the first letter in the search. Which makes me worried, as could there actually be a reason for this?

It might be due to some package names having PascalCase but being saved on disk with camelCase. That's the only reason I can think of, but when looking at the list of package names in one project as an example, everything was working fine. I guess we will know where it breaks after we update... 🤷 💥

@msf do you remember - was there a particular reason to just use the first character?

should we just give this a try and see how it plays out? I think it's pretty easy to revert. My only concern is I don't know how the binary gets built and pushed to s3
It seems there's a branch called [circleci-upload-arm64](https://github.com/citymapper/go-minipypi/commit/c086ef0d378e31c0fa50faf3b2eeaaca7f7ca1d4) that builds both amd64 and arm64 but it doesn't seem to work due to an old ssh key being used to fetch from github
https://app.circleci.com/pipelines/github/citymapper/go-minipypi/1/workflows/c317b25b-195e-4da8-ab5b-730a4ab0ba0c/jobs/58

@leibowitz yes ok, as arguably its pretty broken right now, i'd say just go for it. We may have to be responsive though with any problems caused (hopefully none, but we haven't changed this in so long). I can't remember how it is built + packaged though (maybe manually?)

Hello, totally didn't realise I was still subscribed to this repo. Hope you are all doing well!
I think this is because package names in PyPi are case insensitive. (i.e. if you have Django and django in your requirements file both should install the same package). Moving to the prefix search means that this is no-longer a dropin replacement for PyPi.
A simple thing to do here might be to use this trick on the first 2-3 characters of the path (i.e. make 4/8 list requests instead of 2 which should cut down the number keys that get returned in each by a lot)

Thanks @simongibbons for the insight. From what I saw in our requirements.txt, we use the correct names (spelled properly in term of case sensitivity)
It would definitely break if a package name is not spelled properly, but we can always work around this by updating the package name in the requirements.txt file. Sounds like a minor and solvable issue to be able to solve the timing out one – which is blocking and where the only workaround is to increase the timeout to larger and larger values

the other "better" option would be to store all the files lowercase on s3, and change the request to lowercase when looking for the prefix... but I can't be bothered 🙊

Thanks @simongibbons !
@leibowitz I think that we should try what simon suggests, since we don't want to rename all the files on S3.

We make 4 requests instead of 2. If first chars are "AB" we make prefix requests for

AA
Aa
aA
aa

If this is still too low, we can try 8 requests: AAA, AAa, AaA, etc

I disagree. I would rather keep it simple, and do the query as the package name is specified in the requirements.txt – and fail when it doesn't mach. The failures will be quite obvious, and we can amend the package name in the requirements.txt to fix the build on CircleCI.
Doing 4/8 requests per package name is not going to help the requests being slow issue imho

What I'm trying to say is this boils down to how ListObjectsPages works. As the doc mentions (and the name suggests), it iterates through list of content, 1000 elements at at time.
In some cases, we have multiple thousands of packages with the same letter. Multiple requests will be triggered, and AFAIK this is based on the StartAfter parameter to query the next page. Therefore it can only do these requests in sequence. In other words, the more objects it has to go through, the longer it will take. And there's no way around it. At least that I know of
The best "solution" imho is to not do multiple requests, and fetch only the packages matching the complete name (prefix)

yes, the indexing on s3, is purely prefix based and pip installer was case
insensitive and did two queries for a package, changing the casing..
at the time we didn’t has that many packages so, searching for first letter
(twice, casing!) was simple and cacheable and easy.. -and because i was
reverse engineering the pip behavior and not studying carefully i didn’t
wann go into the complications of “what if the package is called xYz”,
(that is weird casings of characters..) whixh would for caching for
quadratic combinations of lookups.. and caching all those s3 list queries..
but quite honestly, it was simplicity and doing it quickly without thinking
nor knowing too much about pip behavior
in essence, it was something that worked just fine for just 2
On Wed, 15 Dec 2021 at 16:04, Emil R. Vaughan @.***>
wrote:

@msf https://github.com/msf do you remember - was there a particular
reason to just use the first character?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/citymapper/go-minipypi/pull/19#issuecomment-994932808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH3DGBWHQ7BX67PNVHEQLURC4ATANCNFSM5KDNHNKA
.

--
Miguel Mascarenhas Filipe

btw, nowadays there are pip proxies/caches that are super easy to
use/maintain, my minipipy is “old tech” now
On Fri, 17 Dec 2021 at 18:31, Miguel Mascarenhas Filipe <
@.***> wrote:

yes, the indexing on s3, is purely prefix based and pip installer was case
insensitive and did two queries for a package, changing the casing..
at the time we didn’t has that many packages so, searching for first
letter (twice, casing!) was simple and cacheable and easy.. -and because i
was reverse engineering the pip behavior and not studying carefully i
didn’t wann go into the complications of “what if the package is called
xYz”, (that is weird casings of characters..) whixh would for caching for
quadratic combinations of lookups.. and caching all those s3 list queries..
but quite honestly, it was simplicity and doing it quickly without
thinking nor knowing too much about pip behavior
in essence, it was something that worked just fine for just 2
On Wed, 15 Dec 2021 at 16:04, Emil R. Vaughan @.***>
wrote:

@msf https://github.com/msf do you remember - was there a particular
reason to just use the first character?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/citymapper/go-minipypi/pull/19#issuecomment-994932808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH3DGBWHQ7BX67PNVHEQLURC4ATANCNFSM5KDNHNKA
.

--
Miguel Mascarenhas Filipe

--
Miguel Mascarenhas Filipe

another option is to have a long running go-minipypi server running and
update the url on circleci configs to use that instead of a shortlived
instance
that way all builds share the s3 lookup cache
On Fri, 17 Dec 2021 at 18:33, Miguel Mascarenhas Filipe <
@.***> wrote:

btw, nowadays there are pip proxies/caches that are super easy to
use/maintain, my minipipy is “old tech” now
On Fri, 17 Dec 2021 at 18:31, Miguel Mascarenhas Filipe <
@.***> wrote:

yes, the indexing on s3, is purely prefix based and pip installer was
case insensitive and did two queries for a package, changing the casing..
at the time we didn’t has that many packages so, searching for first
letter (twice, casing!) was simple and cacheable and easy.. -and because i
was reverse engineering the pip behavior and not studying carefully i
didn’t wann go into the complications of “what if the package is called
xYz”, (that is weird casings of characters..) whixh would for caching for
quadratic combinations of lookups.. and caching all those s3 list queries..
but quite honestly, it was simplicity and doing it quickly without
thinking nor knowing too much about pip behavior
in essence, it was something that worked just fine for just 2
On Wed, 15 Dec 2021 at 16:04, Emil R. Vaughan @.***>
wrote:

@msf https://github.com/msf do you remember - was there a particular
reason to just use the first character?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/citymapper/go-minipypi/pull/19#issuecomment-994932808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH3DGBWHQ7BX67PNVHEQLURC4ATANCNFSM5KDNHNKA
.

--
Miguel Mascarenhas Filipe

--
Miguel Mascarenhas Filipe

--
Miguel Mascarenhas Filipe

at unbabel we host our own pypi server and it is quite painless, speaking
against my creation, maybe that would be a more robust fix?
https://github.com/pypiserver/pypiserver
but hey, I do like my creation.. and using a simple s3 bucket as a wheel
repository.. But please be aware that go-minipypi was my first Go program,
written initially on a saturday with a tcpdump and netcat -l to see what
pip install -r xx was asking for.. and simply solving for that.
I was lazy and didn't check how wheel files should be uploaded or how to
handle the nuances of pip.. and those nuances might be different nowadays..
I guess I'm secretly happy that my code is still being used :-)
hugs to everyone, covid free and full of good spirits!
cheers
On Fri, Dec 17, 2021 at 6:40 PM Miguel Mascarenhas Filipe <
@.***> wrote:

another option is to have a long running go-minipypi server running and
update the url on circleci configs to use that instead of a shortlived
instance
that way all builds share the s3 lookup cache
On Fri, 17 Dec 2021 at 18:33, Miguel Mascarenhas Filipe <
@.***> wrote:

btw, nowadays there are pip proxies/caches that are super easy to
use/maintain, my minipipy is “old tech” now
On Fri, 17 Dec 2021 at 18:31, Miguel Mascarenhas Filipe <
@.***> wrote:

yes, the indexing on s3, is purely prefix based and pip installer was
case insensitive and did two queries for a package, changing the casing..
at the time we didn’t has that many packages so, searching for first
letter (twice, casing!) was simple and cacheable and easy.. -and because i
was reverse engineering the pip behavior and not studying carefully i
didn’t wann go into the complications of “what if the package is called
xYz”, (that is weird casings of characters..) whixh would for caching for
quadratic combinations of lookups.. and caching all those s3 list queries..
but quite honestly, it was simplicity and doing it quickly without
thinking nor knowing too much about pip behavior
in essence, it was something that worked just fine for just 2
On Wed, 15 Dec 2021 at 16:04, Emil R. Vaughan @.***>
wrote:

@msf https://github.com/msf do you remember - was there a particular
reason to just use the first character?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/citymapper/go-minipypi/pull/19#issuecomment-994932808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH3DGBWHQ7BX67PNVHEQLURC4ATANCNFSM5KDNHNKA
.

--
Miguel Mascarenhas Filipe

--
Miguel Mascarenhas Filipe

--
Miguel Mascarenhas Filipe

--
Miguel Mascarenhas Filipe

thanks @msf 👍

@leibowitz I'm concerned that this will break things, since Python authors will assume package names are case insensitive - this applies both to requirements.txt and also to packages' setup.py where they may reference other packages.
I believe in our case we can fix the speed of go-minipypi by moving some unneeded old versions out of the packages bucket. (I shared this with you on Notion). Would it be worth trying that first? (Long term, we'll move everything to Code Artifact, so we don't need a permanent fix for this.

@emil79 removing packages from the s3 bucket we use is, as you pointed out, only a temporary fix. And it is independent from this change. In other words, this can be done with or without this change.
And I agree that moving to Code artifact is the way forward. As I said before, this was my attempt at fixing the issues we are facing while using go-minipypi in its current form. The fact that S3 bucket are case sensitive, but pypi isn't, means this idea of using an s3 bucket is always going to be flawed. My suggestion is to bite the bullet now, and not have to come back to this issue ever again. If someone has issues using go-minipypi, it will be apparent what the problem is when a package is not being resolved (and easily fixed by changing the case in the name)
I could even add a logging message when a package name lookup is returning a 404, to notify the user to double-check if the name of the package is spelled properly (in regards to case)
That and bumping the major version number, and highlight the breaking change.
But if you are still feeling against merging this change as-is, that's fine. I just don't want to add more requests, as we discussed, as I feel this is just repeating the original issue (due to how the S3 ListObjects API works)


Validate html

Practically super-confusing when html is being rendered wrong, unclear why, and turns out there's unclosed tag in template. Hello https://github.com/developit/htm/issues/91

https://github.com/dy/xhtm/issues/3


Add body/ring geo columns for center coordinates and radius in pixels

Add columns in the body geo metadata files:

Center (X,Y) coordinate for any body in or near the FOV

CENTER_X_COORDINATE, CENTER_Y_COORDINATE
Describe X as the axis that is traditionally displayed horizontally, Y as the axis that is traditionally displayed vertically
NULL_CONSTANT is -99999999
FORMAT is 8.2

Radius of the body in pixels (even if it's not in the FOV)

RADIUS_IN_PIXELS

Also add x/y coordinates for ring geo

See Sec. <IP_ADDRESS> in https://docs.google.com/document/d/1JVEhksHlru2jk8ZW6WPyZeGxReG0ccB-3GEQ2vyX4vg/edit


Is it possible to retrospectively update a table cell?

Hi
I'm trying to make a table like below:

Where each row represents a battle ( the last row being current battle).
I need to be able to go back and update the winner column? right now i'm having to add an extra row with the rest of the columns empty
Thanks

@RedEyedDog Currently not possible, but I guess we could add a setter to the TableRow indexer.
https://github.com/spectreconsole/spectre.console/blob/main/src/Spectre.Console/Widgets/Table/TableRow.cs#L30

that would be great.. you make it sound and look easy. I'll expect it by the end of the day haha. Would really Improve the functionality a lot :)

@RedEyedDog Haha, I don't have time to do anything about this, but perhaps you want to send a PR? 😄

if you think its as simple as adding a setter.. ill take a look no problem

could you possibly explain what kind of project this is and how to build it and test before I issue a PR. I cant see any developer onboarding instructions :O

Just found Contributing.md lol let me read that first

@RedEyedDog There is a solution file in the src folder which you can open in Visual Studio.
To run a full build with packaging, restore all tools from a command prompt using dotnet tool restore and then dotnet cake.
Also take a look at: https://github.com/spectreconsole/spectre.console/blob/main/CONTRIBUTING.md
When I think about it, I think we would need to add a new method here: https://github.com/spectreconsole/spectre.console/blob/main/src/Spectre.Console/Widgets/Table/TableRowCollection.cs such as Update(int row, int column, IRenderable item) so we can lock the row collection.

Ok so instead of adding a setter for indexing.. you would prefer an Update method? I can do that sir no problem.
I'll go try and figure it out now ;)

@RedEyedDog Awesome. Let me know if you run into any problems and I'll try to help out!

Hey looking at the code.. I think you want me to add update method to the Table class and then use "row" for index into his Rows collection and "column" to index TableRow?

@RedEyedDog We would want to lock the TableRowCollection when making the change, so I think we need to add an update method to the TableRowCollection. We could add an extension method to Table later on that calls the new Update method.

thanks.. i'm thinking somthing like this:

@RedEyedDog Sounds like a plan. Would be great if you could add tests for this as well.

ok great.. easy peasy lemon squeezy ;). Ill add some unit tests no problem

I'm just trying to test locally. I have built successfully and I go to "spectre.console\src\Spectre.Console\bin\Debug\netstandard2.0" and grab the dll. I then copy,  paste into my projects bin folder... but I'm still not seeing my new method?
I have also added an extension method for the table like so:

and I cant see it

Sorry realised what I was doing, smelly brain fart :@

Worked a treat

:)

I'm thinking this naming might be better? what do you think

Hey I have issued a PR but its Failing on documentation and not sure why exactly

@RedEyedDog Yeah, that's not your fault. We're currently having some problems with documentation and the net 6.0 rc SDK.
I will review your PR tonight when I have some spare time 👍

Closed via #546


[SPARK-16164][SQL] Update CombineFilters to try to construct predicates with child predicate first

What changes were proposed in this pull request?
This PR changes CombineFilters to compose the final predicate condition by using (child predicate AND parent predicate) instead of (parent predicate AND child predicate). This is a best effort approach. Some other optimization rules may destroy this order by reorganizing conjunctive predicates.
Reported Error Scenario
Chris McCubbin reported a bug when he used StringIndexer in an ML pipeline with additional filters. It seems that during filter pushdown, we changed the ordering in the logical plan.
import org.apache.spark.ml.feature._
val df1 = (0 until 3).map(_.toString).toDF
val indexer = new StringIndexer()
  .setInputCol("value")
  .setOutputCol("idx")
  .setHandleInvalid("skip")
  .fit(df1)
val df2 = (0 until 5).map(_.toString).toDF
val predictions = indexer.transform(df2)
predictions.show() // this is okay
predictions.where('idx > 2).show() // this will throw an exception

Please see the notebook at https://databricks-prod-cloudfront.cloud.databricks.com/public/4027ec902e239c93eaaa8714f173bcfc/1233855/2159162931615821/588180/latest.html for error messages.
How was this patch tested?
Pass the Jenkins tests (including a new testcase).

Test build #61105 has started for PR 13872 at commit 2b21fd7.

cc: @liancheng

@dongjoon-hyun Thanks for the work! However, I think the optimizer should have the freedom to reorder predicate evaluation order. For example, we may evaluate cheap predicates first in order to shortcut expensive predicates.
The problem here is that, to enable this kind of optimization, we actually require filter predicates to be both deterministic and full, while in case of SPARK-16164, one of the UDF is a partial function (only defined on a partial range of all possible input values), and thus rely on other predicates evaluated before itself to reduce input range.
In a word, it's not safe for applications to make the assumption that filter predicates are always evaluated in exactly the same order they are defined in the original query.

Sure, I fully agree with your view. That's the declarative language.
However, we can provide more natural order as a default order like in this PR. As you see, without considering this issue, we had better update CombineFilters in this way.
By the way, theoretically, we can reorder the unit predicate of the conjunctive form by their dependency if we can do more sophisticate analysis. But, that's not considered here.
I think there is no harm to change CombineFilters like this. And this PR definitely reduces the possibility of this kind of errors.

I think I had better change the title of this PR. (I just copied from the JIRA.)
Does that will reduce your concern a little bit?

For any conclusion, thank you for review, @mengxr and @liancheng !

Test build #61105 has finished for PR 13872 at commit 2b21fd7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61105/
Test PASSed.

My major concern is that even the reported case is "fixed" by this PR, in general user applications shouldn't rely on predicate evaluation order. Other than that, the change made in CombineFilters LGTM.
@rxin Shall we have this in 2.0?

@liancheng @rxin In the code, I didn't use UDF explicitly in a filter expression. It is like the following:
filter b > 0
set a = udf(b)
filter a > 2

and the optimizer merged them into
filter udf(b) > 2 and b > 0

This could happen for any UDFs that throw exceptions. Are we assuming that UDFs never throw exceptions?
The patch LGTM and I think we should merge it into 2.0.

I think it'd be difficult to guarantee ordering all the time, but since the change is small we might as well take this tiny step to ensure stronger compatibility.

LGTM

Thank you, @rxin . I hope so, too.

Merged into master and branch-2.0. Thanks!

Thank you, @mengxr , @liancheng , and @rxin .


Inserting rows removes macro assignment in button for ONLYOFFICE

Description
You may assign a macro to a text box or shape in ONLYOFFICE spreadsheets. When using the InsertRows function in excelize and saving such spreadsheets, the macro assignment is  lost.
Steps to reproduce the issue:

Create an empty spreadsheet in ONLYOFFICE
In the ribbon, click Insert > Text Box and create a text box with some text
Right-click the text box and click Assign Macro and assign Macros 1
Write some Go code using excelize to open the spreadsheet, call InsertRows and then Save and observe that the saved spreadsheet loses the macro assignment for the text box

I have attached an example spreadsheet below:
ONLYOFFICE Macro Example.xlsx
You can open this in ONLYOFFICE and click the Run Macro button which will place some text in A1.
Now, run the following code to open, insert rows and save the spreadsheet:
package main

import (
	"log"

	"github.com/xuri/excelize/v2"
)

func main() {
	f, err := excelize.OpenFile("ONLYOFFICE Macro Example.xlsx")
	if err != nil {
		log.Fatalf("unable to open spreadsheet: %v", err)
	}
	defer f.Close()

	if err := f.InsertRows("Sheet1", 5, 1); err != nil {
		log.Fatalf("unable to insert rows: %v", err)
	}

	if err := f.SaveAs("Output.xlsx"); err != nil {
		log.Fatalf("unable to save file: %v", err)
	}
}

If you now open Output.xlsx, you'll see that you can no longer click the button as no macro is assigned. This only seems to occur when inserting rows; updating values doesn't cause the problem.
Update: This problem occurs with any operation which inserts or removes rows and columns
Describe the results you received:
The text box is no longer clickable after saving and the macro assignment is lost.
Describe the results you expected:
The macro assignment should be retained.
Output of go version:
go version go1.22.3 windows/amd64

Excelize version or commit ID:
v2.8.1

I also tested this with the latest master branch on GitHub with the same results sadly.
Environment details (OS, Microsoft Excel™ version, physical, etc.):
Windows 11 Pro
ONLYOFFICE 8.1
Thanks in advance for all your help
Fotis

After examining the compressed XML, I can see the problem occurs in the drawing object / text box (under xl\drawings\drawing1.xml).
The assignment is present here:
        <xdr:sp macro="jsaProject_aebd0d9e67b34cf89bc153ffe4dff966">

But is lost after saving when rows or columns are inserted or deleted:
        <xdr:sp>

Cheers
Fotis

Thanks for your feedback. This issue has been fixed, please upgrade to the master branch code by go get -u github.com/xuri/excelize/v2@master, and this patch will be released in the next version.

Thanks for your feedback. This issue has been fixed, please upgrade to the master branch code by go get -u github.com/xuri/excelize/v2@master, and this patch will be released in the next version.

Thank you so much for the fix, works perfectly! 😄


Update to latest IcedFrisby version

See this issue for more information.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @PyvesB!

  Generated by :no_entry_sign: dangerJS against 06fb5f789362358861327cd03bb12b58daf8ff54

The CI results weren't being picked up, I did a no-op force-push to unblock things. Could someone please put another 👍 ?


use stack to build

I make stack used to build.
stack is very useful. If you use stack, no need to prepare GHC, cabal-install and so on manually. stack downloads them and dependency packages and then build sjsp.

Could you make it infomative for both cabal and stack users?

OK

Thank you.


No error sent to Rollbar

When an error is raised from the query execution, even setting the Debug::RETHROW_INTERNAL_EXCEPTIONS or Debug::RETHROW_UNSAFE_EXCEPTIONS, the exception is catched in the outer loop but is not sent to Rollbar.
Is like Rollbar not liking the exception, because even logging it doesn’t work.
try{
   GraphQL::executeQuery(/* query */)->toArray(/* debug flags */);
}
catch(Exception $e){
  /* here the exception comes, as a PDOException in this case, so it looks like it is re-thrown properly */
  Rollbar::log(Level::ERROR, “this is sent”);
  Rollbar::log(Level::ERROR, $e); /* this is not */
}

Removing the catch, the error is not sent.
Generating the error outside the executeQuery call, the error is sent.
So, what’s the difference in $e if in both cases get_class($e) returns the same PDOException?

This doesn't sound like an issue with this library - exceptions are re-thrown and caught properly. Must be some problem with Rollbar.
It could have something to do with PDOException nuances with code. See this for more details: https://www.php.net/manual/en/class.pdoexception.php#95812
Closing this but feel free to re-open if you still think it relates to this library.


Update steampipe commit reference to include new HCL config to make plugin startup timeout configurable

Update the steampipe commit reference to include new HCL config changes made in https://github.com/turbot/steampipe/commit/8ba2cc5645135804de83962c550812ec5596b441.
Steampipe issue: https://github.com/turbot/steampipe/issues/4320

https://github.com/turbot/steampipe-postgres-fdw/commit/363caad6bde5c706c74f0d26a470384edaca0030


Implement basic Parent bot to Skill service to service authentication (Dotnet)

Implements https://github.com/microsoft/botframework-sdk/issues/5590

the work required has been merged.  see PR #2755


No connection to Travis

This should be connected to travis and the badge for passing/failing should be included in the README

Done


How do you install libev on windows 10?

I downloaded the libev files.
Doesn't explain how it's used or installed.
and cmake for libuwsc throws errors for not able to find libev...
how to make cmake compile libuwsc with libev?
Thanks.

afaik there is no way to install libev on windows, see #32


Error after implementing TurbSim: GF wind array was exhausted at 0.5 seconds

Dear Community!
I am running FASTFarm v3.5 on Windows 10. My model consists of two small wind turbine models (D=1.2m). I wanted to implement TurbSim and did generate the .bts but now I get this error:

I have tried some different grid setups but nothing changes. Here is my input file for the TurbSim:
TurbSim.txt
Also: Where can I set that the full-field wind data files are generated periodically as requested here: https://openfast.readthedocs.io/en/main/source/user/fast.farm/ModelGuidance.html#inflow-wind-generation ?
Thank you for the help!
All the best,
Sara

Dear @lusoalman,
To generate periodic wind data, you should set UsableTime = "ALL" in your TurbSim input file.  Does that solver your issue?
FYI: When generating inflow for use with FAST.Farm, I would recommend using the scripts available through the Python toolbox (https://github.com/OpenFAST/python-toolbox/tree/main/pyFAST/fastfarm).
Best regards,

Dear @jjonkman,
Thank you for the quick answer! I tried giving "ALL" as an input for the usable time before but I get a numerical error:
Invalid numerical input for file ".\TurbSim.inp".
The error occurred while trying to read the usable output time.
All the best,
Sara

Dear @lusoalman,
It appears that you are using an old version of TurbSim (v1.5).  I suggest upgrading to the TurbSim v2.0 (available here: https://www.nrel.gov/wind/nwtc/turbsim.html) or the version of TurbSim available in this OpenFAST GitHub repository (which contains a few small updates relative v2.0).
Best regards,

Dear @jjonkman,
Ah yes, that worked.  But I need to figure out some other errors I get now when running TurbSim:

TurbSim_UsrSpec.txt
Thanks a lot and all the best,
Sara

Dear @lusoalman,
An error in the Cholesky factorization results from having too high (i.e., unphysical) coherence.  Presumably the issue in your case is that your domain is very small for model scale, so, the coherence parameters may need to be customized for the scale.
I would say that it is more common to keep the simulations at full scale, and scale-up model-scale experimental data to full scale when doing validation (assuming that is what you are doing).
Please also note that the FAST.Farm modeling guidance recommends setting SCMod2 and SCMod3 to "IEC" in TurbSim to ensure that you will get transverse coherence in TurbSim and wake meandering in FAST.Farm.
Best regards,

Dear @jjonkman,
Thank you for your answer.
I understand the issue but I got confused with the scaling now. Should I increase the grid and hub size and scale it down to my application? Do I do that by setting ScaleIEC to 2 and changing the Coherent Turbulence Scaling Parameters ?

Dear @jjonkman,
Okay, thanks a lot! I will try changing the Spatial Coherence Parameters.
All the best,
Sara

Dear @jjonkman
I am trying to simulate the disturbed wind Inflow. I used the matlab code [https://forums.nrel.gov/t/binary-wnd-specification/180/5] to use it for inflow wind (.wnd) for FAST.Farm but I have faced error.
Reading a 76x19 grid (1500 m wide, 5 m to 455 m above ground) with a characteristic wind speed
of 10.347 m/s.
Processed 334 time steps of 0.33333-Hz full-field data (999 seconds).
Farm_Initialize:AWAE_Init:AWAE_UpdateStates:InflowWind_CalcOutput:CalculateOutput:IfW_FlowField_Ge
tVelAcc:Grid3DField_GetCell: Error: GF wind array was exhausted at 3 seconds (trying to access
data at -2.7985 seconds). IT_Lo=0, IT_HI=1
InflowWind_CalcOutput:InflowWind_GetSpatialAverage:CalculateOutput:IfW_FlowField_GetVelAcc:Grid3DF
ield_GetCell: Error: GF wind array was exhausted at 3 seconds (trying to access data at -247.1
seconds). IT_Lo=-82, IT_HI=-81
InflowWind_CalcOutput:InflowWind_GetHubValues:CalculateOutput:IfW_FlowField_GetVelAcc:Grid3DField_
GetCell: Error: GF wind array was exhausted at 3 seconds (trying to access data at -261.56
seconds). IT_Lo=-87, IT_HI=-86
The data of Inflow wind, I got from .dtk with dt_low = 3s, as the time step, and the dt_low for my new simulation is also 3s, and dt_high = 0.3s. (.sum) file I followed the specification mentioned in the link as below,
T CLOCKWISE
230.00 HUB HEIGHT
10.347 UBAR
6.000 TI(u)
6.000 TI(v)
6.000 TI(w)
0.000 HEIGHT OFFSET
when I value Height OFFSET =0, the Running Inflow shows " (1500 m wide, 5 m to 455 m above ground)" which is my desire.

Dear @majidpoli,
My guess is the problem is in your FAST.Farm set up in the X direction (or time).  The script you are using does not generate wind data that is periodic in time (as recommended in the FAST.Farm modeling guidance when using synthetic wind data), so, your wind data will have a finite length of time, which is equivalent to a fixed length in the X direction.  Where have a placed the wind turbines and what are the dimensions and locations of your FAST.Farm low- and high-resolution domains?
Best regards,

3.0       DT_Low        Time step for low-resolution wind data interpolation; will be used as the global FAST.Farm time step (s)
0.3      DT_High       Time step for high-resolution wind data interpolation (s) [>0.0]
155    NX_Low        Number of low-resolution spatial nodes in X direction for wind data interpolation (-) [>=2]
76.0    NY_Low        Number of low-resolution spatial nodes in Y direction for wind data interpolation (-) [>=2]
19.0     NZ_Low        Number of low-resolution spatial nodes in Z direction for wind data interpolation (-) [>=2]
0.0    X0_Low        Origin of low-resolution spatial nodes in X direction for wind data interpolation (m)
-750.0    Y0_Low        Origin of low-resolution spatial nodes in Y direction for wind data interpolation (m)
5.0    Z0_Low        Origin of low-resolution spatial nodes in Z direction for wind data interpolation (m)
45.0       dX_Low        Spacing of low-resolution spatial nodes in X direction for wind data interpolation (m) [>0.0]
20.0       dY_Low        Spacing of low-resolution spatial nodes in Y direction for wind data interpolation (m) [>0.0]
25.0       dZ_Low        Spacing of low-resolution spatial nodes in Z direction for wind data interpolation (m) [>0.0]
57.0       NX_High       Number of high-resolution spatial nodes in X direction for wind data interpolation (-) [>=2]
57.0       NY_High       Number of high-resolution spatial nodes in Y direction for wind data interpolation (-) [>=2]
87         NZ_High       Number of high-resolution spatial nodes in Z direction for wind data interpolation (-) [>=2]
--- WIND TURBINES ---
3         NumTurbines   Number of wind turbines (-) [>=1] [last 6 columns below used only for Mod_AmbWind=2 or 3]
WT_X       WT_Y   WT_Z   WT_FASTInFile   	X0_High    Y0_High   Z0_High   dX_High   dY_High   dZ_High
(m)        (m)    (m)    (string)     		(m)        (m)       (m)       (m)       (m)       (m)
1200.0      0.0    0.0      "fst\WT1.fst"          1060.0     -140.0      5.0       5.0      5.0      5.0
2400.0      0.0    0.0      "fst\cWT2.fst"          2260.0     -140.0      5.0       5.0      5.0      5.0
3600.0      0.0    0.0      "fst\WT3.fst"          3460.0     -140.0      5.0       5.0      5.0      5.0

Dear @majidpoli,
Indeed, it appears that the flow in the X direction does not encompass your wind farm because your inflow data is not assumed to be periodic by InflowWind.  For wind data that is not periodic, InflowWind will start the wind box 0.5*MIN(GridWidth, GridHeight)  downstream of X =0 (which is X = 225 m for your wind box), and the box of length UBAR*UsableTime (which equals 10336.6 m for your wind box) will propagate downstream at a speed of UBAR.  However, your turbines and domains extend further downstream, and so, do not lie within the wind box at the start of the simulation.  To ensure that the domain lies within the wind box, you'll have to shift the entire domain upwind.
FYI: I notice that your high-resolution domain has a smaller time step and spatial resolution than your wind box, but InflowWind can only interpolate wind data, not add refinement, so, this added resolution will not improving the solution in your case.  This is why we typically recommend using Mod_AmbWind = 3 when using synthetic wind data in FAST.Farm, which allows you to use use distinct synthetic wind boxes for the low- and high-resolution domains that have appropriate resolution tailored to each domain.
Best regards,

Dear @jjonkman,
Thanks for the fast response and your commitment,
I used the normalized wind speed discussed in "Table C-2: TurbSim tower data binary file grid format"
I respected the wind box in the farm, and mentioned the "PERIODIC" in the .sum file.
so now FAST.Farm could read the .wnd file.
Thanks


fix(storage-network): incompatible with Multus version above 4.0.0

https://github.com/longhorn/longhorn/issues/6953

@mergify backport v1.5.x

@mergify backport v1.4.x


Request Invoicing safe app

Name/Description
Name: Request Invoicing
Description: The easiest way to get your invoices paid in crypto
Type

New addition

Compatible Networks
- Mainnet
- Rinkeby

Code for review
https://github.com/RequestNetwork/invoicing
IPFS hash/App URL
https://invoicing.request.network/

Hello!
Thanks for the contribution, the app has been added to the default list 🎉


Network 20180101

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Since add the new 2018-01-01 directory, also need to change the readme

No modification for AutorestCI/azure-sdk-for-ruby

No modification for AutorestCI/azure-sdk-for-python


WSDC format with 1-minute prep times between speeches

This is used in all high school tournaments in Estonia. It helps a single judge to adequately keep track of the prep time between speeches.
If this is a debate format submission, please provide some information about the circuit, league or tournaments where this format is used:
By submitting this pull request, I agree to license my contributions under the MIT License.

All done, thanks @loorm!


Need help to get started

I don't know if this is an issue or that I do know enough about wsdl.  I'm trying to convert the onvif library , but I only get empty definitions.
First I can not get the glob to work with **/*.wsdl ( this may be a known issue #44 )
And If I target one file the definition emitted is empty. Is there something wrong with the wsdl files from onvif?
npx wsdl-tsclient --verbose .\wsdl\ver10\device\wsdl\devicemgmt.wsdl -o ./generated/

Hi,
shouldn't it be npx wsdl-tsclient --verbose ./wsdl/ver10/device/wsdl\devicemgmt.wsdl -o ./generated/ instead?


Android app: parameter "clear" for action button doesn't work with action "view"

Android app (test build from Discord / https://phil.nopaste.net/T7d7JOvNRH?d=1&f=app-play-debug.apk&a=uVxXKVl4wn) is not behaving correctly with "view" action and the "clear" parameter set to true.
In this case my app clears the notification, but does NOT execute the "view" action.
Example request:
{
    "topic": "transfer",
    "message": "I got a new pickup point",
    "actions": [
      {
        "action": "view",
        "label": "meet me here",
        "url": "geo:47.421087468386155, 10.985437463216053",
        "clear": true
      }
    ]
  }

If the "clear" parameter is set to false (or not set at all, as false is the default), the action is executed.

Fixed in https://github.com/binwiederhier/ntfy-android/commit/e4255212effc2865fbdfc77b637e7b43a2fa0b97


Fix ticks alignment

I wanted to send this PR a bit early to get feedback on the overall approach.  I need to update the tests still, so you can ignore that they're failing for now
You can see a screenshot of this PR in https://github.com/chartjs/Chart.js/issues/4600

Simon's feedback was that I look at using the auto skipper with the time scale. I'll close this for the time being as I investigate that possibility


Toon component calling the Toon api too much

Home Assistant release with the issue:
0.89.1
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
hassio
Component/platform:
https://www.home-assistant.io/components/toon/
Description of problem:
The toon component seems to calling the Toon api too much
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
toon:
  client_id: ***
  client_secret: ****

Traceback (if applicable):
toonapilib.toonapilibexceptions.IncompleteStatus: {'fault': {'faultstring': 'Rate limit quota violation. Quota limit  exceeded. Identifier : *******', 'detail': {'errorcode': 'policies.ratelimit.QuotaViolation'}}}

Additional information:
n/a

I confirm the same issue as @MatthijsDJ already posted. Before upgrading  I was using the custom component from https://github.com/krocat/ToonHA which worked without any issues for the last 5-6 months.

Same here.

Just got the same problem. Used to work for a few days but after few subsequent HA restarts i started to get this.

See: https://github.com/home-assistant/home-assistant/issues/21010#issuecomment-471358123

I see in https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/toon/climate.py that the update interval is set at 30 seconds. Which is 2880 API calls a day.
SUPPORT_FLAGS = SUPPORT_TARGET_TEMPERATURE | SUPPORT_OPERATION_MODE

MIN_TIME_BETWEEN_UPDATES = timedelta(seconds=5)
SCAN_INTERVAL = timedelta(seconds=30)

But the Toon API docs say the following... max 100 or 500 calls a day (not sure at what it depends)

To protect our quality of service we have set a quotum of 10 calls per minute per display. To discourage the implementation of longer lasting continuous interaction a quotum of 100 calls per display per days applies on top of this. But to give you more flexibility during development you are allowed to consume 60 calls per minute and 500 calls per day per display to make sure you can play around as desired.

This is weird, I've seen the limitation in the documentation, but was unable to hit it.
The previous existing custom component is also polling at a rate of 30 seconds, which has not been an issue for a long time.
I'll look into a solution for this.

Does it maybe have something to do with the appname you give to https://developer.toon.eu/user/me/apps when you sign up to a developer account. maybe they only accept an special amount of request for a specific appneme.
because the new component has been released everybody is now maybe signing up with the same appname like: hassio or homeassistant could that cause any conflicts?
The toonapilib with Krocat/ToonHA has worked for me the last year flawlessly. but right now there are problems just when the new component is released.

I am reverting back to the Krocat/ToonHA versionto see if i have problems with that version.

@scheric The easy solution for this, it to adhere to the limits imposed by the docs.
Suggested:

Polling time of 5 minutes (300 seconds), which will cause 288 requests a day (out of 500).
Making SCAN_INTERVAL configurable for the user in case somebody wants different values.

I like the configurable scan interval because it has worked perfectly with 30 seconds interval.

@scheric It might be the appname but also Toon went to a new api. Probably the limit wasn't an issue on the old api? Hence no problems with it in the past.
Regarding the app name, I didn't used a generic name for it so I guess that isn't the issue.

@MatthijsDJ
I have used https://github.com/krocat/ToonHA for the last year that version uses the same (new) API  that this new component uses. It even uses the same toonapilib
I did not have problems with https://github.com/krocat/ToonHA for the last year. but now i stept on this new component i am having problems.

I am now going to test the https://github.com/krocat/ToonHA version for a week and look for errors.

@scheric Ok, I was not aware of that. Might it be that this new component gets more data (sensors) from toon and does more api calls?

@MatthijsDJ It does not, since the toonapilib is the one that limits the number of requests.

or maybe due to increased interest, Toon is now enforcing the limit more thoroughly?

important findings after one night.
I am now running https://github.com/krocat/ToonHA . i have changed the toonapilib refreshtime to STATE_CACHING_SECONDS = 5 I have hooked up an template sensor that tracks the refresh time.

I only reseved some
toonapilib.toonapilibexceptions.IncompleteStatus: {'code': 500, 'message': 'Internal Server Error', 'description': 'Error communicating with Toon'}

errors.
I have not seen any quota violation errors and i can see my my full log.

I had the same issue when I did use a common name: Home Assistant.
Now it's running fine for 2 days after I changed the appname to something random.

I reverted to https://github.com/krocat/ToonHA and for the last 3 days works perfect with no modification of developer appname, refresh time or toonapilib version.

I've changed my appname last night, and for now it's running...

Altough I didn't used a common changed my app name aswell.. Will have a look the logs wether this fix it,

Interesting conclusion if this becomes true... that would be a bug we'd need to report with Eneco/Toon themselves.

@frenck Changed the name, but didn't fixed the issue.  Also if I look the error message the "rate_limit" is based on your Developer account with Toon.
Learning Python since yesterday so might use this issue to speed up learing ;-)

@frenck, I also changed the name, worked for al little while but now it's broken again...

Thanks for letting me know!
I'm currently finishing another thing and will dive into this asap. (Unless someone beats me to it of course) :+1:

I celebrated too early, sorry guys :(
Looking for another solution, let you know if I find some .

@frenck, would really like to help you but don't know where to start :p ...

@frenck
I haven't thoroughly tested this change but could this be an solution?
https://github.com/home-assistant/home-assistant/compare/dev...scheric:scheric-toon-quota-violation

@scheric No, that is not a solution. I've left a comment on your PR as well.

I'm testing a workaround for now (based on the temporary changed cache TTL on the toonapilib). This will take off the heat.
I also found the source of the issue at hand and started implementing a better solution to this problem overall.

Can confirm that the temporary fix is working.

Thanks for letting me know @MatthijsDJ, I'm currently about 60-70% done with the "real" solution to this problem.

This is still ongoing, but currently blocked due to the climate 1.0 change. PR is open.

Merged, part of v0.96.


Add manual testing for ViewGroup and FindCommonTiming

Codecov Report

Merging #224 (7396c48) into master (ac2d097) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 7396c48 differs from pull request most recent head a9d8612. Consider uploading reports for the commit a9d8612 to get more accurate results

@@            Coverage Diff            @@
##             master     #224   +/-   ##
=========================================
  Coverage     67.09%   67.09%
  Complexity      695      695
=========================================
  Files           107      107
  Lines          2425     2425
  Branches        330      330
=========================================
  Hits           1627     1627
  Misses          671      671
  Partials        127      127

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 ac2d097...a9d8612. Read the comment docs.

Looks good.


hide more redundancy in default view

We should hide any GO terms /extension which do not add anything based on hierarchy.
This was on the old tracker, but I can't find the ticket so it's possible we though it was finished.
There is a little bit outstanding.
So for example:

This will mean that "protein kinase activity"  becomes totally redundant in compact view and only appears in full view with  supporting refs
here, we do not need to show
protein kinase activity has substrate cdr2
because it is covered by
serine/threonine protein kinase activity directly_activates cdr2 part_of regulation of cell size
(we can also  probably equivalize extensions on pom1 which seem to be describing the same thing)

duplicate of https://github.com/pombase/website/issues/45


Unable to set custom FQDN for WAC

Unfortunately I am not able to set FQDN for Windows Admin Center installation. Not even if wildcard SSL cert is chosen

Hi guys, sorry for bothering you again. Can you advise a workaround or name ETA for the feature delivery? It is holding our POC back
Br&Thx

This is not currently on our roadmap. Please create a suggestion for this to be tracked.


View PDF files in the browser

Is that useful?

Codecov Report

Merging #51 into master will decrease coverage by 0.22%.
The diff coverage is 60%.

@@             Coverage Diff              @@
##             master      #51      +/-   ##
============================================
- Coverage       100%   99.77%   -0.23%
- Complexity      352      353       +1
============================================
  Files            25       25
  Lines           903      901       -2
============================================
- Hits            903      899       -4
- Misses            0        2       +2

Impacted Files
Coverage Δ
Complexity Δ

backend/Controllers/DownloadController.php
96.29% <60%> (-3.71%)
15 <0> (+1)

backend/Services/Storage/Filesystem.php
100% <0%> (ø)
58% <0%> (ø)
:arrow_down:

backend/Controllers/routes.php
100% <0%> (ø)
0% <0%> (ø)
:arrow_down:

backend/Services/Tmpfs/Adapters/Tmpfs.php
100% <0%> (ø)
20% <0%> (ø)
:arrow_down:

backend/Services/View/Adapters/Vuejs.php
100% <0%> (ø)
5% <0%> (ø)
:arrow_down:

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 4616836...c523c5b. Read the comment docs.

Very useful!
Thanks


Do not raise when model version path is s3

#73

Hi Everyone,
is there any chance this bug fix is going to be merged and released?
Thanks so much in advance


Wrong null check in web-context initialisation listener

https://github.com/quartz-scheduler/quartz/blob/9f9e400733f51f7cb658e3319fc2c140ab8af938/quartz-core/src/main/java/org/quartz/ee/servlet/QuartzInitializerListener.java#L163
Seems like it's checking for shutdownPref, and then it parses shutdownWaitPref.
This leads to ignoring the configuration value quartz:wait-on-shutdown if quartz:shutdown-on-unload is not configured. This leads to a race condition in Tomcat that leaves the scheduler threads alive enough that Tomcat calls then thread leak.

Current work-around: if quartz:wait-on-shutdown is needed declaring quartz:shutdown-on-unload is needed.

Looks like there is a PR#212 for this already

New PR #322 has been raise for this fix


aio app add/delete service

Description

[x]  Needs to bump @adobe/generator-aio-console once changes in adobe/generator-aio-console#11 are released
[x] to be merged after #354

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Note tests are failing in gh-actions for now because adobe/generator-aio-console#11 is not merged and released yet. On local machine with npm linking tests are done

Note tests are failing in gh-actions for now because adobe/generator-aio-console#11 is not merged and released yet. On local machine with npm linking tests are done

Note tests are failing in gh-actions for now because adobe/generator-aio-console#11 is not merged and released yet. On local machine with npm linking tests are done

Turned out to be trickier than this, all TheCommand.run() in jest tests require all the commands, which in turns requires generator-aio-console/lib/console-cli.js from the new add service commands. Which does some operations on inquirer, however inquirer is mocked in jest.setup and needs to bee mocked in app use tests, which make the require fail.
Fixed this in the console generator, by delaying the prompt setup in an init method

Note tests are failing in gh-actions for now because adobe/generator-aio-console#11 is not merged and released yet. On local machine with npm linking tests are done

Turned out to be trickier than this, all TheCommand.run() in jest tests require all the commands, which in turns requires generator-aio-console/lib/console-cli.js from the new add service commands. Which does some operations on inquirer, however inquirer is mocked in jest.setup and needs to bee mocked in app use tests, which make the require fail.
Fixed this in the console generator, by delaying the prompt setup in an init method

ready to review :)

ready to review :)

I have an issue, see the log:
❯ aio console ws select<PHONE_NUMBER>344598433
Retrieving the Workspace with id:<PHONE_NUMBER>344598433... done
Workspace selected Stage
You are currently in:
1. Org: Adobe IO DEMO
2. Project: ShazFF8
3. Workspace: Stage
❯ aio app add service
Workspace MyWorkspace is currently subscribed to the following services:
[
  "Lightroom / Camera Raw API",
  "Adobe Analytics"
]
? Add Services to the Workspace MyWorkspace? Clone Services from another Workspace in the same Project
? Select Workspace:
❯ Production
  Stage

How does aio app add service determine which workspace I am in? After I selected a new workspace via aio console ws select, it still used the previous workspace.

Hi @shazron, aio console ws select is writing to the global config file. And app plugin is using .aio.

Edit: I see that the workspace is via .aio. How do we change the workspace?

The old way is to select globally and aio app use. But https://github.com/adobe/aio-cli-plugin-app/pull/368 exposes an easier aio app use -w <wskname>

closing in favor of #368


Update ru_RU.json (Translation)

Please submit translation requests using our Crowdin page: https://crowdin.com/project/pufferpanel


Cannot schedule pods on the Windows node

After completing the setup for the 2 nodes-setup (1x Linux control plane node and 1x Windows worker node), and ensuring that the configuration is finished on both nodes (due to #188 this currently means pasting the 1-antrea.ps1 file in an SSH connection on the Windows node (the file cannot be accessed by going to C:\sync\windows) for whatever reason)) trying to schedule a pod on the Windows node doesn't work as expected:
  Warning  FailedCreatePodSandBox  22s (x105 over 23m)  kubelet  (combined from similar events): Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "9125a4d13f634963a8f635f987c5be41d826fb5f9400ff2aa80706db6a929da5": rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing open \\\\.\\pipe\\cnisock: The system cannot find the file specified."

There is an issue opened on the antrea repo here - currently closed due to inactivity - which further leads to this issue on the containerd repo. As the discussion in the latter doesn't seem to suggest any fix coming soon, is there any workaround for now?

For this issue in particular it's related with the missing OpenSSL installation
https://github.com/kubernetes-sigs/sig-windows-dev-tools/issues/188

As @knabben mentioned, related to #188. Closing as dup

/close


Haskell wrapper

Not an issue, per se, but just so you know it exists.  I made a Haskell wrapper for ripser.
It takes a container of items and a distance function, produces a lower-distance-matrix, opens a ripser process, sends the LDM on stdin and then parses the result into a list of Barcodes (using a data structure from  https://hackage.haskell.org/package/Persistence-2.0.3/docs/Persistence-Filtration.html).
The wrapper can be found on github here: https://github.com/adamConnerSax/ripser-haskell.

Thank you!


Feature/editor layer

Description

Adds the editor layer of the framework with the basic fuctionality added:

ImGui-Bundle Integration,
EditorPanelSystem,
Gizmos using ImGuizmo:

Transformation gizmos,
Camera gizmos.

The editor can be attached and detached using a flag.
Minor code clean up.

Related Issues

Closes #7
Closes #8
Closes #9
Closes #13

Editor seems to work fine on my end! 👏

Great work @johnoyo ! I added a commit that allows all of my tests to run successfully, please all remember to run all tests after each PR.


Fixes to llvm backend for iele

Includes:

support for matching and compacting overloaded productions
update to latest version of Array in K
make llvm backend compatible with building a library
code to free kore memory
bug fixes

Jenkins: test this please


Cost Management subscription scheduled action module

Description

Adds a module for Cost Management scheduled actions.
Closes #266
Adding a new module

[x] A proposal has been submitted and approved.
[x] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[x] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.

Splitting into separate PRs


[DO NOT MERGE] Plat 454 aks

- What I did
- How I did it
- How to verify it
- Description for the changelog

@yellowmegaman @ita-sammann is this PR still needed?


Handlebar if eq works on template editor but fails API usage

Test code is as follows:
{{#if (eq sub_01 "1" )}}
Dear Sir
{{else if (eq sub_01 "2")}}
Dear Madame
{{else}}
Dear Customer
{{/if}}
Test data is as follows:
{
"sub_01": "2"
}
When testing via web template editor, change sub_01 from 1, 2, 3 etc results in the expected output in the preview pane. However, when sending through the API, the else condition is always caught. I can confirm the sub_01 field is being passed through correctly as I can return it in the email itself, the if/then logic just never evaluates properly.

@OpTiCz27
Hello,
Looks like you are trying to use a helper we do not support. You can look here at our docs to see what we currently support.
For what you are looking to do I give a current example here.
If you have further issues or questions please let us know.

Thanks. So it looks like the helper is implemented via template builder but not through the API.
I guess we'll have to transform our json so that each value becomes a key with a value of true.
So rather than something like "gender": "male" or "gender": "female", we'd have to nest it as something like "gender": { "male": true } and say {{#if gender.male}}.
That should work, it'll just become a bit ugly to manage when there are many possible values. Thank you for the input @kylearoberts

This issue appears to be resolved and is now being closed. If you have more questions or feel the issue has been closed in error, please open another issue. If you need direct support, please contact our Support team and they can help you out. You can access support contact options by logging into https://support.sendgrid.com.
SendGrid Docs Team


Update jquery.number.min.js

numbers lower then 1, like 0.5... it just shows as 5.00

This has been done in a newer version.


Cannot watch StarCraft II replays

Firstly, I ran command like this:
python3 src/main.py --config=qmix_smac --env-config=sc2 with env_args.map_name=3m save_replay=True save_model=True
and then I loaded model by running:
python3 src/main.py --config=qmix --env-config=sc2 with env_args.map_name=3m checkpoint_path=results/models/qmix_smac_parallel__2019-03-20_19-06-37/ runner=episode
Note that I have to use "qmix" rather than "qmix_smac" because I found that using "qmix_smac" along with "runner=episode" would go wrong.(I don't know the difference between qmix and qmix_smac). Then I got .sc2replay file in StarCraftII folder. But when I ran:
python3 -m pysc2.bin.play --norender --rgb_minimap_size 0 --replay NAME.SC2Replay
I got message like this without any replay:

smac_2019-03-21-07-04-12.SC2Replay
pygame 1.9.4
Hello from the pygame community. https://www.pygame.org/contribute.html
I0321 18:50:29.104659<PHONE_NUMBER>28192 sc_process.py:110] Launching SC2: /home/aaron/StarCraftII/Versions/Base70154/SC2_x64 -listen <IP_ADDRESS> -port 17276 -dataDir /home/aaron/StarCraftII/ -tempDir /tmp/sc-bvn2ia27/ -displayMode 0 -windowwidth 640 -windowheight 480 -windowx 50 -windowy 50 -dataVersion 94596A85191583AD2EBFAE28C5D532DB -osmesapath libOSMesa.so.8
I0321 18:50:29.107152<PHONE_NUMBER>28192 remote_controller.py:163] Connecting to: ws://<IP_ADDRESS>:17276/sc2api, attempt: 0, running: True
Version: B70326 (SC2.2018Season4)
Build: Nov 27 2018 03:26:30
Command Line: '"/home/aaron/StarCraftII/Versions/Base70154/SC2_x64" -listen <IP_ADDRESS> -port 17276 -dataDir /home/aaron/StarCraftII/ -tempDir /tmp/sc-bvn2ia27/ -displayMode 0 -windowwidth 640 -windowheight 480 -windowx 50 -windowy 50 -dataVersion 94596A85191583AD2EBFAE28C5D532DB -osmesapath libOSMesa.so.8'
Starting up...
Startup Phase 1 complete
I0321 18:50:30.110396<PHONE_NUMBER>28192 remote_controller.py:163] Connecting to: ws://<IP_ADDRESS>:17276/sc2api, attempt: 1, running: True
Startup Phase 2 complete
Attempting to initialize OSMesa from file libOSMesa.so.8 ...
Successfully loaded OSMesa library!
Running CGLSimpleDevice::HALInit...
Calling glGetString: 0x7ff0b6671da0
Version: 2.1 Mesa 18.2.2
Vendor: Brian Paul
Renderer: Mesa OffScreen
OpenGL initialized!
Listening on: <IP_ADDRESS>:17276
Startup Phase 3 complete. Ready for commands.
I0321 18:50:31.111603<PHONE_NUMBER>28192 remote_controller.py:163] Connecting to: ws://<IP_ADDRESS>:17276/sc2api, attempt: 2, running: True
Could not find map name for file: /tmp/sc-bvn2ia27/StarCraft II/TempReplayInfo.SC2Replay
----------------------- Replay info ------------------------
map_name: "Just Another StarCraft II Map"
local_map_path: "SMAC_Maps/3m.SC2Map"
player_info {
player_info {
player_id: 1
race_requested: Terran
race_actual: Terran
}
player_result {
player_id: 1
result: Tie
}
player_apm: 1808
}
game_duration_loops: 4558
game_duration_seconds:<PHONE_NUMBER>90625
game_version: "<IP_ADDRESS>326"
data_build: 70326
base_build: 70154
data_version: "94596A85191583AD2EBFAE28C5D532DB"
Configuring interface options
Configure: raw interface disabled
Configure: feature layer interface enabled
Configure: score interface enabled
Configure: render interface disabled
Launching next game.
Next launch phase started: 2
Next launch phase started: 3
Next launch phase started: 4
Next launch phase started: 5
Next launch phase started: 6
Next launch phase started: 7
Next launch phase started: 8
Starting replay 'TempStartReplay.SC2Replay'
Game has started.
Score:  100
Result:  [player_id: 1
result: Undecided
]
RequestQuit command received.
Closing Application...
unable to parse websocket frame.
Terminate action already called.
Entering core terminate.
Core shutdown finished.
I0321 18:54:00.920968<PHONE_NUMBER>28192 sc_process.py:201] Shutdown gracefully.
I0321 18:54:00.921149<PHONE_NUMBER>28192 sc_process.py:182] Shutdown with return code: 0

smac_2019-03-21-07-04-12.SC2Replay_1.zip

Which OS are you using?

Which OS are you using?

I am using Ubuntu 18.04

Which OS are you using?

I am using Ubuntu 18.04

Unfortunately, the SC2 replays cannot be watched using the Linux version of StarCraft II as it doesn't have a graphical component. You will need to use either the Mac or Windows version of the StarCraft II client.

Which OS are you using?

I am using Ubuntu 18.04

Unfortunately, the SC2 replays cannot be watched using the Linux version of StarCraft II as it doesn't have a graphical component. You will need to use either the Mac or Windows version of the StarCraft II client.

I installed starcraft2 client on windows. But it says the .sc2replay file cannot be open due to the lack of map file. However, when I tried the following command on Ubuntu:
python -m pysc2.bin.play --replay ~/StarCraftII/Replays/smac_2019-03-21-07-04-12.SC2Replay
I got the following picture, but the replay stuck there.

The problem was solved by copying Maps directory to C:\Program Files (x86)\StarCraft II.

Thank you! If you're using linux and install StarcraftII with wine, it would also work after copying the maps to Program Files (x86) of wine

Thank you! If you're using linux and install StarcraftII with wine, it would also work after copying the maps to Program Files (x86) of wine

Hello! When I try to watch the replay on StarCraft 2 (5.0 windows version) it says "Unable to open map", I tried copying all the maps into a folder but it doesn't work, do you know any way to watch that right now?


feat: (flatten, unflatten) Add the transformKey opt.

Here's the related issue: https://github.com/hughsk/flat/issues/79

lgtm


[QUESTION] How to filter by referenced collection primary key?

Let's take next relation
class Child
{
   int Id {get;set;}
}
class Owner
{
    int Id {get;set;}
    Child Child {get;set;}
}

Which is registered in next way
Database.Mapper.Entity<Child>()
     .Id(e => e.Id, false);
Database.Mapper.Entity<Owner>()
     .DbRef(e => e.Child)
     .Id(e => e.Id, false);

On this point I understand, Owner document will contain only id of child document.
But how should I filter Owner documents by Child id?
I can use next code:
Collection
    .Include(o => o.Child)
    .Find(Query.EQ($"{nameof(Owner.Child)}.{nameof(Child.Id)}", new BsonValue(idValue))))

But is it the best way to do filter by referenced document id? Because code above requires Include, when I don't really need child document, except of Id.
Also, how should I ensure index for it?
Thanks.

Hi @maxkatz6, if you don't need to include you can filter by $id key. Like this:
collection.Query()
    .Where("$.Child.$id = @0", idValue)
    .ToList()

@mbdavid thanks!
I will try it


Better return type for mode estimation

The Optim.jl interface returns a ModeResult, a type we define for holding the outcome of the optimisation. For compatibility the Optimisation.jl interface currently returns the same type, but it's not quite a natural fit for it. We might consider changing the return type before making the switch of interfaces official and possibly deprecating the Optim.jl one. Might also consider aligning with similar types from SciML.
This came up in review of #2221 with @torfjelde, https://github.com/TuringLang/Turing.jl/pull/2221/files#r1608363430.

Out of curiosity, besides the type of the parameter vector (#2363), what other features or changes would you want for the new and improved return type?

One thing I was dissatisfied with was the awkward nesting of data about the optimisation, and some commonly needed things being hard to access. For instance, the value of the objective function is stored three times: in -result.lp, in result.optim_result.objective, and, if OptimJL was used as the backend, in result.optim_result.original.minimum. However, something like convergence information, which is often of great interest, I think is only accessible through result.optim_result.original.
I'm not sure what the best thing to do is for intuitive usability, while keeping all the data reported by Optimization.jl and the backend optimiser accessible to users who know what they are doing. Maybe just storing the thing returned by Optimization.jl and writing getter functions to get the often-needed bits from it?
Also, we should store a flag for whether this was MLE or MAP, and maybe also the model object.
The current return type is a left-over from the time when the only optimiser we supported was Optim.jl, and life was simpler.

Thanks, that makes sense. As an end-user, my main complaint about usability is the one in #2363, and the current structure seems basically fine to me. A MAP/MLE flag would be good. The model object is already stored in result.f.model, so maybe it just needs a getter function...in general, writing some well-documented getter functions seems like an easy way to improve usability.
Thinking about it as an end-user, the main things I want to be able to get easily from a mode estimate, whether as fields or from getter functions:

Parameter estimates, accessible by name (a dimension or nparameters getter function would be nice)
Objective/log-probability value
Objective function, to do diagnostics, likelihood profiling, etc.
Optimization.jl results, to drill down into as needed
What algorithm was used
MAP vs. MLE
Coefficient table/standard errors

One more wish to add to the list: it would be great if there was a way to get generated_quantities from ModeResults!

One more wish to add to the list: it would be great if there was a way to get generated_quantities from ModeResults!

This will be trivial if we take the VarNamedVector approach @mhauru :)


ISPN-12948 AuthenticationLDAPIT and AuthenticationKerberosIT thread leaks

https://issues.redhat.com/browse/ISPN-12948
Increase the grace period for leaked threads to stop.
Apache Mina's AbstractPollingIoProcessor wakes every 1s
to check for stop requests.

@ryanemerson I'm pretty sure the NPE is caused by ISPN-12923 and fixed by https://github.com/infinispan/infinispan/pull/9500

Thanks @danberindei


vaultwarden-sqlite: 1.28.0 -> 1.28.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vaultwarden-sqlite is: Unofficial Bitwarden compatible server written in Rust
meta.homepage for vaultwarden-sqlite is: https://github.com/dani-garcia/vaultwarden
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.28.1 with grep in /nix/store/w58j9cw5iihgr1s5pf48nknc9s8z4i4m-vaultwarden-1.28.1
found 1.28.1 in filename of file in /nix/store/w58j9cw5iihgr1s5pf48nknc9s8z4i4m-vaultwarden-1.28.1

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
nixos-install-tools
tests.nixos-functions.nixos-test
vaultwarden
vaultwarden-mysql
vaultwarden-postgresql
vaultwarden-sqlite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w58j9cw5iihgr1s5pf48nknc9s8z4i4m-vaultwarden-1.28.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A vaultwarden-sqlite https://github.com/r-ryantm/nixpkgs/archive/2a961b35c967c0cc9c42d5680e806a119f3b34e0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w58j9cw5iihgr1s5pf48nknc9s8z4i4m-vaultwarden-1.28.1
ls -la /nix/store/w58j9cw5iihgr1s5pf48nknc9s8z4i4m-vaultwarden-1.28.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package blacklisted:

nixos-install-tools

3 packages built:

vaultwarden
vaultwarden-mysql
vaultwarden-postgresql

Maintainer pings
cc @msteen @ivan for testing.

Wrong attribute name.

Addressed by https://github.com/NixOS/nixpkgs/pull/225506


Workflow to label issues failed with 401

See also https://github.com/microsoft/azure-pipelines-tasks/issues/17107.
A workflow for https://github.com/microsoft/azure-pipelines-tasks/issues/17252 failed with an error:
Error: Failed request: (401)
    at RestClient.<anonymous> (/home/runner/work/azure-pipelines-tasks/azure-pipelines-tasks/node_modules/typed-rest-client/RestClient.js:202:31)
    at Generator.next (<anonymous>)
    at fulfilled (/home/runner/work/azure-pipelines-tasks/azure-pipelines-tasks/node_modules/typed-rest-client/RestClient.js:6:58)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  statusCode: 401

See https://github.com/microsoft/azure-pipelines-tasks/actions/runs/3518879727/jobs/5898253969#step:6:126

Hi @abatishchev  thanks for reporting! Fixed.

Hey @Roman-Shchukin it's failing again:
Error: {"$id":"1","customProperties":{"Descriptor":null,"IdentityDisplayName":null,"Token":null,"RequestedPermissions":0,"NamespaceId":"00000000-0000-0000-0000-000000000000"},"innerException":null,"message":"Access Denied: The Personal Access Token used has expired.","typeName":"Microsoft.VisualStudio.Services.Security.AccessCheckException, Microsoft.VisualStudio.Services.WebApi","typeKey":"AccessCheckException","errorCode":0,"eventId":3000}

See https://github.com/microsoft/azure-pipelines-tasks/actions/runs/5698212298/job/15445847160#step:6:99


fix: alpha disclaimer toast

ToastProvider.js: in the alphaDisclaimerToast fix a typo and add an 'id' so it won't create duplicates of itself.

This change is

:tada: This PR is included in version 1.0.0-dev.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ROI Feat extraction bug

Here 3 values are expected to be unpacked. But only 1 value is returned by the function coco_demo.run_on_opencv_image(img) . Because of this, an error is thrown when I run the roi extraction code in extract_feats_roi.py .
Did you use a modified predictor.py file which would return 3 values instead ?
The current predictor.py file I am using is one provided here by facebookresearch.
Or, i guess a possibility is that the extract_feats_roi.py was outdated because this line has an undefined variable (though the issue in this line hardly matters. we can comment it out)

Hi, yes, please comment it out and find the maskrcnn_code.zip in google drive.

Hi, yes, please comment it out and find the maskrcnn_code.zip in google drive.