non-compatible racer binary with RUST_SRC_PATH

I've started using multirust and suddently racer stopped working for non std completions or go to definitions. I think we should add a warning in the installation instructions about racer-binary/rust-src-path and potentially get racer to throw a warning as well.  When invoked from the CLI, it would exit with status 0 in any case.
Any ideas how can we make this incompatibility issue more transparent to consumers of racer?

racer should work with multirust (that's how I use it).
I just realized that I can use go-to-definition only on use std::<mod> or on extern crate <lib1>, on top of the <lib> word. If I try to use it inside one of the nested modules, say for example in use <lib1>, it doesn't work. Is this expected?
To be more specific, I can point you to the repo I'm working on: https://github.com/ernestrc/lbd . If you check it out and open the file src/model/job.rs, using vim-racer, If you try to go-to-definition on top of  "fmt" in line 15, it works fine. On the other hand, when I try with "toml" in line 17, it doesn't find anything. If you go to the file src/lbdd.rs and try to do the same on top of "toml" in line 9, it will work. Is there a logical explanation for this?
Thanks mate!

I think it's because racer doesn't support the cargo '[[bin]]' field yet, so it is not finding src/lbdd.rs as the crate root. I'll add an enhancement issue for this

created #409

Brilliant! Thanks for all the work you've done with Racer, it's going to be important piece in the future of Rust. Do you think #409 it's doable for a newcomer to the code base?

Yes, I think it might be. Is quite an isolated piece of work

add function to cargo.rs to locate the root modules from the cargo.toml
hook it up to find_possible_crate_root_modules() in nameres.rs

Unfortunately that second function assumes there's only one root in a crate. It might work returning both roots though.


Make build reproducible

by sorting the input file list
to avoid random .o file link ordering
causing functions to be randomly ordered in the .so file
See https://reproducible-builds.org/ for why this is good.
Alternative approach to PR #29

@bmwiedemann thanks for proposing the changes, I've merged them.


Sliders cannot be moved using Chrome on Windows

As witnessed by @AlexanderWirtz yesterday. He doesn't know which version it was.

Sorry, was a bit busy to ask people which versio nthey were running. I noticed it for about 1/3 of the students using Windows laptops with Chrome. I asked those who could to switch to Firefox. That worked fine, which tells me it was indeed a Chrome issue.
Those Chrome users could type in slider settings, by the way.

@grdw, could you look into this?

Yesterday we used Chrome on Windows 7 and that worked just fine. :unamused:

Hmm... this looks weird.  Considering the date, can I assume this was Windows 10?

Quick overview:

Windows 7 - Chrome 48.0.2564.97  - Works
Windows 11 - Chrome 48.0.2564.97  - Works

I'm in the proces of checking Win 8 and 8.1 but they're still downloading :snail:

Could well be a specific Chrome version's problem. I asked the people to email me their versions but they never dod. Let's not spend too much time on this.

Thanks, @AlexanderWirtz . Closing this issue then.


Bug: When switching split window focus multiple times with , , switching hangs up

Bug: Notebook Editor, Interactive Window, Editor cells
Steps to cause the bug to occur

Switch from text file to interactive window numerous times using keyboard shortcuts, e.g.  alternating with

Actual behavior
Window switching works indefinitely
Expected behavior
Window switching stops happening after around 10 times, despite keys being pressed

Your Jupyter and/or Python environment

Jupyter server running: Local
Extension version: 2020.9.114305
VS Code version: 1.50.1
Setting python.jediEnabled: false
Setting python.languageServer: Microsoft
Python and/or Anaconda version: 3.8.5 Conda
OS: Windows 18636.1082 Enterprise
Virtual environment: conda

Dupe of #11808


Issues with the machine readable file for AsyncAPI 2.0.0

Describe the Issue
There are some discrepancies with the machine readable version of the AsyncAPI specification version 2.0.0.

on line 571  additionalItems are defined when the schema type is object.
When additionalProperties are indeed allowed on many of the schemas and should follow the patternProperties, how come it is always sat to false 🤔? I.e. on line 10
On line 726  same as 1.
On line 782 missing type: [object, array].

I checked issues and they are still valid.

@jonaslagoni wanna work on this?

There are 2 cases here
Case 1
@magicmatatjahu @jonaslagoni

On line 782 missing type: [object, array].

I thought that if there is no type in https://github.com/asyncapi/spec-json-schemas/blob/master/definitions/2.5.0/specificationExtension.json it means that any type is ok. No?
Case 2
This one is still valid and very trivial to fix:
One PR related to traits fix: additionalItems should be removed from https://github.com/asyncapi/spec-json-schemas/blob/master/definitions/2.5.0/operation.json#L35 and https://github.com/asyncapi/spec-json-schemas/blob/master/definitions/2.5.0/message.json#L157 as in both cases this is an object, not an array

I thought that if there is no type in https://github.com/asyncapi/spec-json-schemas/blob/master/definitions/2.5.0/specificationExtension.json it means that any type is ok. No?

Yea you are right, don't know why I suggested it was missing 🧐

Final scope for this one 👇🏼
additionalItems should be removed from https://github.com/asyncapi/spec-json-schemas/blob/master/definitions/2.5.0/operation.json#L35 and https://github.com/asyncapi/spec-json-schemas/blob/master/definitions/2.5.0/message.json#L157 as in both cases this is an object, not an array.
And it should be done in all versions, not 2.5 only


Newer docker image

It seems that the latest docker image does not contain the WHOAMI_PORT env feature. Could a newer docker image be created?

@ldez Would it be possible to setup github actions in this repository? If a tag is set then the github action could build an image and subsequently push it to dockerhub.

the GHactions is already set https://github.com/traefik/whoami/tree/master/.github/workflows

Ok thank you.


Added Humidity 2.0 Bricklet

Added Humidity 2.0 Bricklet.
Please can you revise and merge if ok.
Humidity 2.0 Bricklet has been released.
https://www.tinkerforge.com/en/doc/Hardware/Bricklets/Humidity_V2.html

Ok, checked it and code looks fine.  Added readme.md comments and merged.  Thanks for contributing and sorry for the delay in merging.


Making the breeze install task precompiled.

Fixes #15
When installing Breeze, it runs the script/precompile_tasks, but for some reason, it won't actually generate the file. I tried to copy what Avram is doing, but I'm not sure what's missing here...

@stephendolan that worked! I don't know why the other way wasn't working, but this lets me generate the binary and running the install is all compiled now

Nice, that's super clean!


Enable pipeline credscan

We should enable the common credscan tooling for the go CI pipelines.

Dup with https://github.com/Azure/azure-sdk-for-go/issues/17654


Wait rather than fail when lock is held

Running two instances of vulnix at the same time fails with this error:
zc.lockfile.LockError: Couldn't lock '/home/user/.cache/vulnix/Data.fs.lock'

vulnix should wait for the lock to become available rather than failing.  (This is what nix does.)
In particular, it would allow vulnix runs to be easily diffed against one another with diff <(vulnix ...) <(vulnix ...), which is valuable for confirming that a contemplated change has the desired effect.  Practical example:
$ nixos-rebuild build -I nixos-config="$PWD/configuration.nix"
$ vulns() { vulnix --whitelist ~/nixos-vulnerability-roundup/whitelists/nixos-19.09.toml "$@" | sed -r 's,/nix/store/[^/]{32},/nix/store/...,g'; }
$ diff -u <(vulns --system) <(vulns ./result)

Good point. Will look into this.


[MRG+1] move trait import out of try block

WIP following up on https://github.com/mne-tools/mne-python/pull/3383

travis not happy

Current coverage is 85.87% (diff: 100%)

Merging #3648 into master will decrease coverage by 1.52%

@@             master      #3648   diff @@
==========================================
  Files           343        343
  Lines         60764      60737    -27
  Methods           0          0
  Messages          0          0
  Branches       9301       9300     -1
==========================================
- Hits          53107      52157   -950
- Misses         4903       5883   +980
+ Partials       2754       2697    -57

Powered by Codecov. Last update 570eab2...a02e6f2

Tests pass now. I had to modify the nosetests call and the tab-checker to stop them from importing modules that import mayavi.

It seems ot be working now, @Eric89GXL your suggestions are implemented

can someone confirms that this works on a machine that does not have mayavi installed?

can someone confirms that this works on a machine that does not have mayavi installed?

Shouldn't Travis take care of it?

I suppose but let's be sure...

I threw a raise ImportError in my Mayavi __init__.py and get the expected:
$ nosetests mne/tests/test_docstring_parameters.py
..
----------------------------------------------------------------------
Ran 2 tests in 1.463s

OK
$ nosetests mne/gui/
SSSSSSSSSSS
----------------------------------------------------------------------
Ran 11 tests in 0.011s

OK (SKIP=11)

+1 for merge

thx @christianbrodbeck


Bottoms of droopy letters getting cut off in event titles

The gs are having a bad time.

Oof, I can't repro :/

I'm on Chrome Windows, maybe it's a platform-specific issue. I can try it out on Chrome Mac in the morning.

Fwiw, adding line-height: normal !important on the  holding that text fixes the issue. Not sure if that's useful debugging information or not. ¯\_(ツ)_/¯

That span should already have line-height: 1.2;, which is...more. No?

Some additional data from Chrome Windows:

Changing the window's zoom level to 90% displays the entire g, and changing it to 110% displays more of the g than the 100% zoom level, but not quite all of it.

Changing the line-height values to 1.21, 1.22, 1.23, or 1.24 do absolutely nothing at 100% zoom. Changing the line-height value to 1.25 suddenly makes the entire g show up. From some quick Googling it seems that Chrome may be doing some rounding down / int truncation on computed line-heights, possibly this truncation is bypassed when it has to calculate the line height itself via normal?

Ughhhhhhhh pixels, why are you the worsttttt.
Sounds cool, thanks so much for digging into it! I'll change it tomorrow :)

Looks like my gs are fine on Chrome Mac regardless of what zoom level I choose. Windows font rendering problem maybe?

Removing overflow: hidden from the .cpchip parent makes the droopy letters show completely. The same will happen with it if we make .cpchip display: inline-block. But any of this would be just a workaround for the font weirdness.
Remove "Segoe UI" and will have full droopy letters, albeit with an uglier font.
The entire problem is reproduced with this

        gyj

Ugh, this sounds like I should file a Chrome bug :)

Well, I've created one 🥇

@mariusbalaban can you link it in here, pretty please? I can find some people to look at it :)

Here it is

Sweet! Looks like it's assigned! I'll keep an eye on it, but I'll close this issue for now since I'd really like Chrome to fix this! 🙏

Yep, that's great.

Well, it has been decided: it's not a Chrome bug


Modifiers are binded for phenotypic features

Create a phenopacket .
goto phenotypic feature.
Add 2 or more phenotypic features .
select phenotypic feature 1 --> click edit --> goto edit more details.
Select Modifiers and select one modifier.
Click ok
Select phenotypic feature2 --> click edit --> goto edit more details.
Select Modifiers and expand
Error : Modifier that was selected for phenotypic feature 1 is shown as selected for phenotypic feature2. This should not be allowed.

tested


Decompile Windows' Structured Exception Handling (SEH)

It would be nice if the decompiler could correctly decompile functions that use structured exception handling.
This would involve recognizing exception registration setup code, the scope table, and properly hooking up the control flow.
https://web.archive.org/web/20150318041037/https://www.microsoft.com/msj/0197/exception/exception.aspx has a good overview of how it works.

Good suggestiion.  We've had supporting try/catch block scope on the books for a while.
The first step was recoverying the SEH information and laying it down so it could be used.
Our plan is to handle sub-function such as exceptions in an upcoming release.

Just piggy-backing this request to have SEH structures included. This is helpful when reversing exploits setting a malicious handler in the TIB.

bump

bump

Good suggestion. We've had supporting try/catch block scope on the books for a while. The first step was recoverying the SEH information and laying it down so it could be used.
Our plan is to handle sub-function such as exceptions in an upcoming release.

@emteere is this still work-in-progress?

Another reference is https://www.openrce.org/articles/full_view/21.


Update add1.2

Added support for usage of add command (Closes #52). This PR covers:

Ensuring that the parsing of arguments with the command word add is updated with the new class.
Ensuring that the UI is updated with the new class
The program can compile and run without a problem.

If one of these points above still has an issue please let me know.
Note:

Email and Tag are not removed from parameters
Phone is a compulsory parameter
Tests have yet to be updated

@Jayne1010 could you fix the failing CI checks, they mostly look like checkstyle errors.

Closes #9 also


use Gonzalos template

Not sure if it's exactly what you mean, but I already updated tasks and other stuff to the "latest template"


brew install instructions for nvm

Oh, I forgot about this, the brew version sucks and it's much easier to use their installer as it'll modify your shell rc files etc


Add a collapsible debug panel to display agent's state information

from 4th task in #368
Agent information should contain:

Agent ID
Agent Score
Speed (needs to be added)
Position (needs to be added)
Heading (needs to be added)
Lane ID (needs to be added)

Build vehicle observation section in the panel

AC:

[ ] add the collapsible panel on the left hand side, have toggles to trigger which information to show
[ ] panel shows agent state information

Closed with #580

Closed with #580


Error:  Unexpected response: 404

Describe the bug
I tried to use the actions for downloading the release assets based on the following options

latest field by setting it to true and removing other fields like Tag
setting latest field to false and assigning a Tag version

In both the cases i get different error regarding the field:
I get Error: [getlatestRelease] Unexpected response: 404 when i use the latest field
If i use the tag instead of latest field then i get the same error for the tag
Error: [getReleaseByTag] Unexpected response: 404
Kindly let me know on how to solve this issue.
Action Environment (please complete the following information):

OS: [ubuntu-latest]

@spalaniappan-pg Are you using this action to download assets from a private repo or a public one?
Would you mind sharing your action step/yaml here

Hi Yes its a Private repo.
Below is the step i tried:
- uses: robinraju/release-downloader@v1
  with:
    repository: "owner/repo"
    latest: true
    fileName: "Capture.PNG"
    zipBall: true

Below is the list of file i have for downloading from release asset:

Fyi, at start i already checkout the project to get full access.

Can you check if the secret TOKEN_UPLOAD_RELEASE contains a personal access token of the user/organization containing the repository from which you are downloading the release asset (with enough permissions)
cf: https://docs.github.com/en/github/authenticating-to-github/keeping-your-account-and-data-secure/creating-a-personal-access-token
I see this as the problem with your token

Hi Thanks for the suggestion.
Just to update, i tried to create new access token and test the actions in a private repository to test the functionality to be sure that there is no issue with my organization tokens, it basically works when the zipBall or tarBall is set to false. When either one of these 2 fields are set to True, then an error occurs. Its a new error code.
Please find the image on the error.

@spalaniappan-pg I found an issue within the download logic that prevented it from downloading tarball/zipball
A fix for it is now merged into main. You can use this action from its main branch as follows.
   uses: robinraju/release-downloader@main

I can create a new release once you confirm it works for you.

Closing this issue as the fix for it is now available in the latest release
Option for downloading all files is now changed to fileName: "*" I think that makes more sense than leaving it empty.


feat: Add standard vue app template to tendermint/vue

Not sure we need a Dockerfile, though

It was part of the starport scaffolded one I think


Basic Testing

Simple testing to check the whole pipeline.
Usage
pip install pytest
pytest test/

Closed and moved to @lazyoracle /c3 for testing


Documentation of .layers

Hi @Koncopd,
would you write a proper documentation of the .layers attribute you built?  Currently, it's only a very non-informative stub:
https://anndata.readthedocs.io/en/latest/anndata.AnnData.layers.html
It should contain a reference to loompy, scvelo and very basic examples.
Here, even the heading in the table is missing:
https://anndata.readthedocs.io/en/latest/anndata.AnnData.html

Hi, @falexwolf,
Yes, i'll add the needed details.

Great, I saw https://github.com/theislab/anndata/commit/4358768532b24877f9b07b260009a4a43a3e4b8e!


Fix serialization for redis >=5.X

Related to onetimesecret/onetimesecret#238
I chose this over return MultiJson.encode(v) , let me know if you prefer one
Add a test to ensure it's not crashing after the patch

Great work figuring out this issue and adding the test coverage. I continued on #7 which has all the details on the approach I ended up going with (spoiler: raise an exception).


Improve tests and coverage

Current coverage:
89.67% Statements 217/242
87.68% Branches 121/138
61.29% Functions 19/31
89.41% Lines 211/236

Not bad but more coverage would be nice.

99.17% Statements 240/242
89.86% Branches 124/138
70.97% Functions 22/31
99.15% Lines 234/236


Peer Dependency issue

This package is not up to date with the latest version of stylelint.

@drew0530 can you explain the issue you're seeing in more detail? The error in your screenshot looks like it's coming from stylelint-config-prettier (a deprecated package) requiring a lower version of stylelint, which has nothing to do with this package, stylelint-config-prettier-scss.

It appears to me that stylelint-config-prettier-scss requires version 15.0.0, and I have 15.10.0 installed, which is providing me a warning.

Sorry for the ambiguity.

@drew0530 that is not an error. This package requires any version of stylelint >=15, which is valid in your case. The screenshot you shared shows the error I described above. Do you have stylelint-config-prettier installed? If so, remove it and the error will go away.

That solved the issue, thank you.


question

Hi,
is the time (for row2 and row3 of input.csv) defined as time between those two samples, the absolute time? why is there a negative time?
0,1,0,0,-52.430000 us
0,1,0,0,-52.428333 us

when i open the coverted vcd file with GTKWave and move the cursor to the left, i can see:
CLK=1
CS=0
DIN=0
DOUT=0

but the csv file shows the first sample with, why are csv and vcd different? CLK=0 and CS=1...
"CLK","CS","DOUT","DIN","Time"
0,1,0,0,-52.430000 us

Thanks,
A

Hi,

why does the signal CLK of input.csv not use quotation marks like the other signals do it?

No idea why, this is the raw output generated by our signal analyzer equipment.

is the time (for row2 and row3 of input.csv) defined as time between those two samples, the absolute time? why is there a negative time?

It starts at negative values I suppose because some synchronization reason of our signal analyzer. If you see line #119:
https://github.com/carlos-jenkins/csv2vcd/blob/master/bin/csv2vcd#L119
Actually the CSV file is interpreted starting at time 0 ("breakpoint"). All previous lines are ignored. This is the expected behavior according to our documentation.
Now, the time between rows is an absolute time. The main difference between the CSV output generated by the IXIA and the VCD is that the CSV has the state of all signals at absolute time steps (samples). The VCD has the timestamp of signal changes and which signal changed, making it a lot more lightweight (in our example, CSV == 24MB, VCD = 150KB).

cool, did you test your tool with some traces?
do you know a tool which does vcd2csv?

great, did you test your tool with some traces?

Sure, we developed this to debug some issues with some signals we were having, and this and GtkWave helped us to find the reason behind them.

do you know any other tools to visualize vcd besides GTKWave?

https://en.wikipedia.org/wiki/Waveform_viewer#List_of_the_best-known_viewers

do you know a tool which converts vcd into a csv?

Do, but it can be easily programmed, is not that difficult. If you see, csv2vcd is quite small and straightforward.

ok.
how can you convert a csv file with float values to vcd?

It really depends on the format of your CSV. If you read the comments on the get_parts() function it will tell you exactly what it is supposed to do.
https://github.com/carlos-jenkins/csv2vcd/blob/master/bin/csv2vcd#L71

my csv looks like that:
0.0,1.5,0.6,2.3,-52.415000 us

can you support that?

Floating point variables? So I suppose we're not in digital world anymore, and I assume your signal analyzer is reading some analog values. I'm not sure if VCD format allows other than 0/1, but according to Wikipedia:

The value change section contains a series of time-ordered value changes for the signals in a given simulation model. For scalar (single bit) signal the format is signal value denoted by 0 or 1 followed immediately by the signal identifier with no space between the value and the signal identifier. For vector (multi-bit) signals the format is signal value denoted by letter 'b' or 'B' followed by the value in binary format followed by space and then the signal identifier. Value for real variables is denoted by letter 'r' or 'R' followed by the data using %.16g printf() format followed by space and then the variable identifier.

So, maybe, if you read the variables as float and then output them as "r followed by the data using 16g printf() format followed by space and then the variable identifier" it could work.
I can try to sketch something if you provide me a relevant CSV file to test.
Regards.

Hi, I just created a new branch called real_variables that support real/float (analog?) signals in a CSV files:
https://github.com/carlos-jenkins/csv2vcd/compare/real_variables#diff-3d244f291ec130623f7f444cff889484
The changes were trivial. It seems to be working with the following example CSV file:
"CLK","CS","DOUT","DIN","Time"
0.0,0.0,0.0,0.0,-10.000 ns
0.0,0.0,0.0,0.0,-8.333 ns
0.0,0.0,0.0,0.0,-6.667 ns
0.0,0.0,0.0,0.0,-5.000 ns
0.0,0.0,0.0,0.0,-3.333 ns
0.0,0.0,0.0,0.0,-1.667 ns
1.0,0.0,0.0,0.0,0 s
2.0,0.0,0.0,0.0,1.667 ns
3.0,1.2,0.0,0.0,3.333 ns
3.5,3.4,0.0,0.0,5.000 ns
4.1,2.2,0.0,0.0,6.667 ns
2.3,2.1,0.0,0.0,8.333 ns

To view the resulting VCD file in GtkWave I had to add the signal (with the Append button) then select the signal, right click, Data Format -> Analog -> Step.
Regards

by floats i mean analog signals.
great, let me test if that works.

Hello Mr. Carlos Jenkins,
I tried to convert real data csv file to vcd with your scripts, but unfortunately, it's not successful. I could see the generated vcd files contains some random variable names (not the same in CSV file). I am also attaching the csv file for your reference. Could you please help me resolve the issue (I am attaching the respective *xls file here)?
Book1.xlsx
Thanks in advance!
Regards,
Mohan


Added recommended Cache-Control header for images

As described here:
https://developers.google.com/web/tools/lighthouse/audits/cache-policy
Included tests.

Hi @jmyrland,
Thanks very much for the pull request. I read into the commits you provided and it led me to provide a more comprehensive solution- you can now add response headers as part of the module options. The readme has been updated along with a credit/mention at the bottom for your contribution.
In the README, search for the imageHeaders option. By default, it will follow the Google recommendations of 1 day as per your pull request.
I also had to refactor some of the cypress tests as when I changed the assertions to incorrect values (e.g. status return of something other than 200) the tests still passed.
Version 1.3.3 is the latest version at the time of writing.
Thanks again for your feedback and contribution.

Great stuff @barryrld will check out the new version soon 👍


django 2.2.14

Checklist

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

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

@conda-forge-admin please rerender

travis fail looks like network :snowflake:
fatal: unable to access 'https://github.com/conda-forge/feedstock-outputs.git/': Could not resolve host: github.com

@conda-forge-admin, please restart ci


Ground truth: spaces before and after text?

I've created *.exp0.gt.txt as a base for manual ground truth creation using Shreeshrii's shell script and the files contain a space before and after the text (no newlines etc). Example:
01127778-001.exp0.gt.txt: " Verfahrenstechnik / -70 B 2813 "
01127778-002.exp0.gt.txt: " Verfahrenstechnik, Forschung und Lehre, (Zsgest. Ue "
01127778-003.exp0.gt.txt: " verf. von Kurt Schiefer und Kurt Boekmer,) "
01127778-004.exp0.gt.txt: " Düsseldorf: Verfahrenstechnische Ges, im Verein Deut- ! "
01127778-005.exp0.gt.txt: " scher Ingenieure 1967. 185 S, 8° "
01127778-006.exp0.gt.txt: " Frühere Ausg. 3. u.d.T.: Verfahrenstechnik im In- und "
01127778-007.exp0.gt.txt: " Ausland. "

... but The-Hallucination-Effect states
»Example 2: Your training text frequently includes a Space at the beginning of your sentences or at the end. Might result in slow training, non-convergence & even model corruption.«
My Question: Spaces or not?
The 1 line images are very tight, no blank space before/after; example:

ok, ocrd-testset.zip *.gt.txt contain no spaces before/after, but \n

@jbarth-ubhd, I see there is no space or no new line at the end of the *.gt.txt

I'll see newlines (4th line below):
jb@xxx:~/Downloads/ocrd-testset> cat *.gt.txt|od -c|head
0000000   i   c   h       d   e   n   k   e   .       A   b   e   r
0000020   w   a   s       d   i   e     305 277   e   l   i   g   e
0000040   F   r   a   u       G   e   h   e   i   m   r 303 244   t   h
0000060   i   n  \n 342 200 236   D   a   s       k   a   n   n       i
0000100   c   h       n   i   c   h   t   ,       c   '   e   s   t

Ground truth line text must not have spaces before or after the text.
It may end with a linefeed (which gets added automatically by many editors).

Just tried it with https://github.com/tesseract-ocr/tesstrain/issues/7 and https://github.com/ocropus/hocr-tools/blob/master/hocr-extract-images , the generated .gt files contain spaces before & after:
308-119.exp0.gt.txt: " | == zz NN NN ANNE NZZ SE anli : "
308-120.exp0.gt.txt: " <C3><BC>ber 1 BONS DD DD SS EN U = NS utfer]pras "
308-121.exp0.gt.txt: " Datei ihrem unfeligen zZ <E2><80><94> SS AN . LEE KA 5 XS Ode bot. "
308-122.exp0.gt.txt: " N ein SH 7 DD SS 7 ea san Zn EFF LEE Z<E2><80><94>_-- BEN UNE x "

bump

Just tried it again with #7 and https://github.com/ocropus/hocr-tools/blob/master/hocr-extract-images , the generated .exp0.gt.txt files contain spaces before & after

Then I assume that the original data (hOCR) already contains such spaces. Do you have a link to an example?

The .hocr' does not contain spaces: <span ...>abcdefg`
See https://digi.ub.uni-heidelberg.de/diglitData/v/tesstrain-issue-335.zip for a complete test environment; main script is Shreeshrii-script.

The spaces before and after the line occur, if your hocr file is indented.
hocr-extract-images uses regex to replace multiple spaces (caused by indentation) with one space. see line 20


Consider renaming master branch to main

The community as a whole is moving away from master, and main seems to be a common choice.  Now would be a good time to do it, before the library gets more adoption.
GitHub allows fairly transparent branch renaming from this page, though cloned repos would need to be updated manually.

Fine with me. We will need to update a few things (Read the Docs, etc.) but that should be relative straightforward. Shall we first finalize the currently open pull requests and then rename?

There's actually no requirement that we land the outstanding pull requests first (GitHub will automatically update the target branch) so this could be done at any time.


exotica: 6.2.0-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository exotica to 6.2.0-1:

upstream repository: https://github.com/ipab-slmc/exotica.git
release repository: https://github.com/ipab-slmc/exotica-release.git
distro file: noetic/distribution.yaml
bloom version: 0.10.7
previous version for package: 6.1.1-1

exotica
* Replace dead link
* Contributors: Wolfgang Merkt

exotica_aico_solver

No changes

exotica_cartpole_dynamics_solver
* Python code improvements from static analysis
* Contributors: Wolfgang Merkt

exotica_collision_scene_fcl_latest
* Switch to FCL on Noetic, use fcl_catkin on older distros (#735 <https://github.com/ipab-slmc/exotica/issues/735>)
* Contributors: Wolfgang Merkt

exotica_core
* Add EndPoseTask::GetTaskJacobian
* Fix build issues with Eigen 3.3.9; enable compilation of exotica_core on Conda (#740 <https://github.com/ipab-slmc/exotica/issues/740>)
* Performance improvements (#741 <https://github.com/ipab-slmc/exotica/issues/741>)
* DynamicTimeIndexedShootingProblem: Allocation-free control cost computation
* DynamicsSolver: dStateDelta and ddStateDelta return previously allocated transition derivative
* DynamicsSolver: Add in-place StateDelta
* Fix installation directives for cmake directory
* Deactivate Boost bind warnings arising from ROS messages
* Update minimum CMake version for add_initializer.cmake
* XMLLoader: Fix for #737 <https://github.com/ipab-slmc/exotica/issues/737>
* Scene: Fix uninitialised quaternion
* Contributors: Wolfgang Merkt

exotica_core_task_maps
* PointToPlane: Add PositiveOnly mode for use as inequality term (#744 <https://github.com/ipab-slmc/exotica/issues/744>)
* CoM: Change debug markers
* test_maps: Turn off JointVelocityLimit test
* SphereCollision: Remove unnecessary allocations (#741 <https://github.com/ipab-slmc/exotica/issues/741>)
* Contributors: Wolfgang Merkt

exotica_ddp_solver
* Performance improvements (#741 <https://github.com/ipab-slmc/exotica/issues/741>)
* FDDP: Fix allocation
* Use in-place StateDelta
* Contributors: Wolfgang Merkt

exotica_double_integrator_dynamics_solver

No changes

exotica_dynamics_solvers

No changes

exotica_examples
* Fix example_shapes
* Contributors: Wolfgang Merkt

exotica_ik_solver

No changes

exotica_ilqg_solver

No changes

exotica_ilqr_solver

No changes

exotica_levenberg_marquardt_solver

No changes

exotica_ompl_control_solver

No changes

exotica_ompl_solver

No changes

exotica_pendulum_dynamics_solver
* Python code improvements from static analysis
* Contributors: Wolfgang Merkt

exotica_pinocchio_dynamics_solver
* Use clang for Pinocchio dynamics solver (#746 <https://github.com/ipab-slmc/exotica/issues/746>)
* Contributors: Wolfgang Merkt

exotica_python
* Trajectory from array in python (#743 <https://github.com/ipab-slmc/exotica/issues/743>)
* Add EndPoseTask::GetTaskJacobian
* Fix build issues with Eigen 3.3.9 (#740 <https://github.com/ipab-slmc/exotica/issues/740>)
* Performance improvements (#741 <https://github.com/ipab-slmc/exotica/issues/741>)
* Remove temporary workaround, formatting
* Contributors: Vladimir Ivan, Wolfgang Merkt

exotica_quadrotor_dynamics_solver
* Performance improvements (#741 <https://github.com/ipab-slmc/exotica/issues/741>)
* Python code improvements from static analysis
* Contributors: Wolfgang Merkt

exotica_scipy_solver
* Code improvements
* Contributors: Wolfgang Merkt

exotica_time_indexed_rrt_connect_solver

No changes

Holding for Noetic sync


add netstandard2.0 support

I realised that one of the reasons I didn't use your library before was because it targets netstandard2.1 and the project I need to use it in is net462 (and I don't have the liberty to upgrade just yet). So here's a PR "backporting" to netstandard2.0. I've run tests against the new target by manually switching the test project over to another target (eg to net462 or netcoreapp2.1). When I leave all frameworks in place (ie multiple TargetFrameworks), it seems like tests interact with each other as the first assembly passes and the second fails to complete with dotnet test. However, all tests, run individually, pass (please verify!), so I'm assuming it's some interaction with the server.

Thanks for the PR.
I will have a look at this in the next few days.  Supporting netstandard2.0 would be nice for older projects, but as the whole netstandard is getting deprecated and many advanced APIs are not available in version 2.0 (true Memory/Span for  example) I have to take that into consideration.

sure, but netstandard2.0 supports net461+, and there's plenty of people still using net46x because it's going to be supported for many years to come; anyway, it's your project, you decide - i tried to keep to using the built-in stuff for !netstandard2.0 and only kludge through for netstandard2.0 users.

closing due to inactivity


[MetaSchedule] TorchBench tuning script: add task extraction mode

This PR adds the task extraction mode to the TorchBench tuning script. The task extraction mode will only import PyTorch model into TVM, extract tuning tasks from it and save them for further use.
This helps the use case where tasks should be extracted from a machine with GPU and then be tuned on a machine with powerful CPU with rpc workers.
cc @junrushao @zxybazh

CC: @zxybazh


Guideline for migrate existing AWS Calico helm to tigera operator

Please help provide a solution which can help use migrate existing AWS Calico helm to tigera operator. The solution should work for both new installation and existing AWS Calico installation.
I am doing auto update with terraform.

Have you followed this doc which the deprecation notice on aws-calico chart on amazon-vpc-cni-k8s point to?
If you do not think that is sufficient or it doesn't work I would suggest creating an Issue on projectcalico/calico as that is the place where something like that would be documented.
You should also include what you have tried from the document (and any variations) and what went wrong.
Since it seems like this is already provided I'm going to close this issue. If you believe this issue needs to be opened back up please explain why and I can do so.


DO NOT MERGE: Move away from actions/script to node scripts

This is an experiment and not suitable for merging in its current form.
TODO:
[ ] Tests will need updating/reworking
[ ] Need to figure out DSN for publish vs forks (useful for testing)
[ ] Need to figure out a cleaner method for making this change (i.e. smaller PRs etc)
[ ] Re-use common code (for example: The logic for creating a session with a status)

Love this! It was about time Pinocchio to become a real boy.

Thanks @BYK, please let me know if you have any concerns with this approach or anything I should keep in mind (I don't have the context on any previous decisions).

Going to close this in favor of smaller PRs so it's easier to review.

I don't have the context on any previous decisions

I'm mostly around to answer any specific questions you may have. Mostly, I was a lone wolf while working on this and actions/script got me the speed. Then I extracted the scripts into separate files for testing purposes but never got around to making this a "proper" action :)


Release 0.8.1

Coverage remained the same at 100.0% when pulling d1b95f5be69c76dd88fc725cf20ba5aec569efa2 on rodjek:release-0.8.1 into d0c1711eb2512a28c03025f33dc9c065e9e5bfda on camptocamp:master.


trealla: 2.55.22 -> 2.55.25

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/trealla-prolog/trealla/releases.
meta.description for trealla is: Compact, efficient Prolog interpreter written in ANSI C
meta.homepage for trealla is: https://trealla-prolog.github.io/trealla/
meta.changelog for trealla is: https://github.com/trealla-prolog/trealla/releases/tag/v2.55.25
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.55.25 with grep in /nix/store/180pgbl7ird92lhvhrfnvvl0jsiqh832-trealla-2.55.25
found 2.55.25 in filename of file in /nix/store/180pgbl7ird92lhvhrfnvvl0jsiqh832-trealla-2.55.25

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

trealla

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/180pgbl7ird92lhvhrfnvvl0jsiqh832-trealla-2.55.25 \
  --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 trealla https://github.com/r-ryantm/nixpkgs/archive/a48228d5bb369fe5bb3b78f45a71fd90e73a45aa.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a48228d5bb369fe5bb3b78f45a71fd90e73a45aa#trealla

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/180pgbl7ird92lhvhrfnvvl0jsiqh832-trealla-2.55.25
ls -la /nix/store/180pgbl7ird92lhvhrfnvvl0jsiqh832-trealla-2.55.25/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 built:

trealla

Maintainer pings
cc @siraben @AndersonTorres for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge

@NixOS/nixpkgs-merge-bot merge


[Python Track2] [azure-eventhubs] [5.0.0b2] Iot hub connection string raises exception with EventProcessor

The following error is raised in v5.0.0b2 when using a vanilla iothubowner connection string similar to the way in EventProcessor example
uamqp.errors.AMQPConnectionError: Failed to open mgmt link: MgmtOpenStatus.Error

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "c:\users\me\.venv37\lib\site-packages\azure\eventhub\eventprocessor\event_processor.py", line 98, in start
    partition_ids = await self._eventhub_client.get_partition_ids()
  File "c:\users\me\.venv37\lib\site-packages\azure\eventhub\aio\client_async.py", line 156, in get_partition_ids
    return (await self.get_properties())['partition_ids']
  File "c:\users\me\.venv37\lib\site-packages\azure\eventhub\aio\client_async.py", line 139, in get_properties
    response = await self._management_request(mgmt_msg, op_type=b'com.microsoft:eventhub')
  File "c:\users\me\.venv37\lib\site-packages\azure\eventhub\aio\client_async.py", line 120, in _management_request
    await self._handle_exception(exception, retry_count, max_retries)
  File "c:\users\me\.venv37\lib\site-packages\azure\eventhub\aio\client_async.py", line 98, in _handle_exception
    await _handle_exception(exception, retry_count, max_retries, self)
  File "c:\users\me\.venv37\lib\site-packages\azure\eventhub\aio\error_async.py", line 67, in _handle_exception
    raise error
azure.eventhub.error.ConnectError: Failed to open mgmt link: MgmtOpenStatus.Error
Failed to open mgmt link: MgmtOpenStatus.Error

Yes, currently could you use the EventHub-compatible endpoint connecting string for the IoT hubs before we resolve this issue?
We'll fix this issue.

Yah...Will do. Thanks.


Concept for fully randomized overworld items

I mentioned this in the Discord, but I want to jot it down here as well so it doesn't get lost to the sands of time.
As I understand it, the reason progressive items have their own special "pick from a list"-style randomization is that the mod doesn't see item locations, it only sees item IDs. Since most overworld item locations contain things like Titanite Shards that aren't at all unique, the mod isn't able to distinguish whether the player picked up a Titanite Shard in High Wall of Lothric or bought one in Firelink Shrine.
But I think this issue could be solved without needing to access additional data in the mod itself. The parameter file and map formats are well-understood, so—before booting the game—you could edit the params to add a few hundred additional fake items that exist only to have unique IDs, and then edit the maps to point all the overworld progressive pickups at those fake items. Then you could have an option on the randomizer server to track whether progressive items have been faked out and add them to the item pool if so.
Edit: Actually, you wouldn't even need to generate the params files as part of the mod. All you'd have to do is generate them once (maybe using the C# libraries that other software uses) and provide them as a zip file for people to use along with ModEngine.

I might be interested in working on implementing this if there's an appetite for it. I don't think the map and param formats are so complicated that it would be prohibitively difficult to port them to Python, which opens up all kinds of possibilities: showing the names of items in other games, putting items in the shrine maiden shop, randomizing starting equipment, and so on.

Hi, yes that's exactly why I ended up using a progressive location system for this first implementation.
I didn't try to modify the game files yet but it would be very interesting to create a new set of itemIDs to identify each location.
I still see some problems with it, but some could be fixed by changing the mod :

The mod doesn't handle properly a stack of the same item, which will prevent picking up a stack of titanite even if 'fakeItems' is set to false.
The mod doesn't hook the right method called after a buy from a merchant, which means even if the merchant inventory is shuffled the mod couldn't send the check to the server.
Having a single .dll file is a convenient installation process, if another file is involved I would probably need to create an installer to check the modengine setup first and then setup AP properly. Otherwise the amount of support requests would increase to much.

There's definitly appetite to support a maximum amount of features and would love to invest more time on this project to support this kind of idea :)

Unfortunately my skills end at the ASM boundary, so I can't help fix those issues directly, but I may be able to suggest workarounds.

The mod doesn't handle properly a stack of the same item, which will prevent picking up a stack of titanite even if 'fakeItems' is set to false.

As in, the mod can't push multiple items of the same type into inventory? This may be less of an issue if all the items are randomized at the map level. As you don't have any cross-game items that are stacks, you should be fine.

The mod doesn't hook the right method called after a buy from a merchant, which means even if the merchant inventory is shuffled the mod couldn't send the check to the server.

This is tricky. The easiest workaround is to say "cross-game items never go in shops", which is fine but kind of unsatisfying (especially because it makes the "get an ash -> get some souls -> get a progression item" flow much harder). The other possibility would be to check items in inventory rather than items as you pick them up, since you'd be able to treat each progression item as a unique key item. That would definitely be harder to implement, but it might fix #14 as well as a side effect.

Having a single .dll file is a convenient installation process, if another file is involved I would probably need to create an installer to check the modengine setup first and then setup AP properly. Otherwise the amount of support requests would increase to much.

I think the way the offline randomizer handles this is to distribute a .zip of files and just say "unzip these into the DS3 directory". ModEngine supports chained mods, so this could just look like:

dinput8.dll (modengine)
archipelago.dll (this mod)
modengine.ini (set up to load both DLLs)
archipelago/ (the directory with the actual randomization)

Another option would be to collaborate with @thefifthmatt (author of the offline randomizer) to teach that to connect to an Archipelago server and handle all the local generation. This might be the best option in general, since it also gives you enemy randomization effectively for free. I've done some work on that codebase already.

it's still early and a bit hacky, but I have a functional prototype of connecting @gracenotes' offline DS3 randomizer to the Archipelago server and randomizing items offline:

This isn't particularly useful yet, but it'll eventually allow Archipelago to put any item in any location the offline randomizer knows about, which is pretty much everywhere. It'll also allow it to randomize cool stuff like the Path of the Dragon gesture, display other games' item names within DS3, and share enemy randomizer settings and seeds for competitive runs.
Major work still to be done includes:

Injecting items from other worlds. From the offline randomizer's perspective this is just a matter of adding some extra rows to the item database, but a bigger question is how the Archipelago mod should figure out that the unknown item we just picked up is actually Archipelago ID 123.

Adding all the new potential locations to the Archipelago server (probably just in a branch for now). There are a lot of random pickups in DS3 once you add in all the shards, little soul items, shops, and so on.

Fixing the shop issue @Marechal-L described above. This may not actually require any ASM wizardry in the end—the offline randomizer is capable of adding new event scripts, so relevant shop items could be replaced with fakes that trigger an event that adds the real item to the inventory. Event items are definitely visible to the mod.

Making all of this tracker-friendly. I'm not sure how much of a priority this is, but because the mod still can't tell where an item was acquired any location that has a non-unique item will never get checked off on Archipelago's tracker page without some extra work. We could potentially fix this by giving each location its own unique synthetic item and letting the mod deliver the actual object to the player.

Making more headway here! If anyone's interested in following along, I'm pushing pretty regularly to github.com/nex3/Dark-Souls-III-Archipelago-client/ and github.com/nex3/SoulsRandomizers, and I'm going to start a branch on github.com/nex3/Archipelago soon as well.
I've got the offline randomizer generating custom items to represent items in other worlds:

...and I've also found another hook to use to detect items purchased from shops. This is actually less work than I thought it might be!

This is all really exciting stuff!  Just so I have a better understanding of the intended results here:
-Offline Generator connects to multiworld server to fetch items/locations based on the player yaml options.
-Generator patches and randomizes the game world to abide by those locations/items, shuffling anything else not included with presumably filler items, if necessary.
All of this accomplished by using invisible event flags on the correct locations?  Essentially, allowing for both to be arbitrary and highly customizable?
Does this account for player saves, and resuming sessions?  From what I can gather, the Generator itself becomes the Client, and patching the game once should be sufficient for the duration of the multiworld, even if across multiple sessions.  How would a user handle multiple player slots?  Just repatch the game?  Would that nuke player progress?  Obviously the server remembers what you've done, but I felt like asking because this sounds like a great step forward.
Got me all jazzed up thinking about the possibilities!  Thank you for your effort.

This is all really exciting stuff! Just so I have a better understanding of the intended results here:
-Offline Generator connects to multiworld server to fetch items/locations based on the player yaml options. -Generator patches and randomizes the game world to abide by those locations/items, shuffling anything else not included with presumably filler items, if necessary.

That's basically it, yeah. My plan is to make the Archipelago server do all the heavy lifting for figuring out the locations for almost everything, but the offline randomizer will probably still handle a few details like shop prices, non-unique enemy drops, and starting character loadouts (seeded by the server so it's consistent for races, of course). It'll also handle all the enemy randomization itself, with options sent down from the server.

All of this accomplished by using invisible event flags on the correct locations? Essentially, allowing for both to be arbitrary and highly customizable?

Mostly we don't even need to use the event system (although they may be necessary for edge cases like Path of the Dragon). The offline randomizer is able to modify the map files, drop tables, and so on. It's already very sophisticated—that's one of the big advantages of using it. (The other is that it knows how to read data from the DS3 files on disk so we don't need to store a whole copy of DS3 on the server.)

Does this account for player saves, and resuming sessions? From what I can gather, the Generator itself becomes the Client, and patching the game once should be sufficient for the duration of the multiworld, even if across multiple sessions. How would a user handle multiple player slots? Just repatch the game? Would that nuke player progress? Obviously the server remembers what you've done, but I felt like asking because this sounds like a great step forward.

It's possible to have multiple concurrent saves if you keep the mod directories separate and don't load into the wrong save at the wrong time. I think the UX will be something like this:

We give the user a zip file to download that contains ModEngine's dinput8.dll, modengine.ini, and an archipelago/ directory that contains the offline randomizer and the online Archipelago mod.
We tell the user "extract this zip in your Dark Souls III directory, then run archipelago/DS3Randomizer.exe.
Once that's done, they can just run DarkSoulsIII.exe and it'll work.

If a user wants multiple different saves, they just need to keep a different archipelago/ directory for each one, and make sure whichever one they want to use is the one with the correct name.

All right, I've got both parts of this fully functional. I've run an end-to-end test with the current server implementation and I've verified that it works to get items locally, send items to other worlds, and receive items from other worlds (it would be weird if that broke but stranger things have happened). You can test it out yourself by downloading DS3-Archi-Stack.zip and following the instructions above.
Anything further here is going to require updating the server as well. Would it make sense to start landing the work I already have in this repo? Or would you prefer I wait until I have a parallel server PR as well and land a whole lot of changes at once?

At best I fill a Community Manager sort of role in the Archipelago world, as I've never actually developed any code myself outside of plain text documentation.  I'll offer my opinion based on observing development, and hopefully that's useful for your purposes.  You'll likely get more specific implementation instruction in #archipelago-dev, as you've done so far.
Obviously @Marechal-L is the relevant Code Owner to consider for merging this progress into their currently existing implementation.  They (or possibly @Br00ty) will end up being tagged in any DS3-related server PRs for the main Archipelago repository, and will therefore need to approve changes.  It would then need to pass the usual unit-tests and Core Dev review prior to inclusion, which is definitely the slowest part of the process by far.  However, this is usually mitigated by successful Beta Tests and code review from other non-core-dev interested parties.  Judging by how previous PRs in the DS3 space have been reviewed, there are quite a few parties interested in pushing this along.
I would recommend compiling an Archipelago build that supports your changes, scheduling a Beta Test (or hopefully, more than one) in the #planning section of the AP Discord server, and distributing your build during that session.  This allows for some stress testing, and the session can be referenced in your actual PR against Archipelago.  Alternatively, you could schedule a Beta Test Async in the #community-run-asyncs channel to do something a little more long-form.  Many of the longer games go this route as it gives you an entire forum post to collect information and bug reports.  The main victory there is Archipelago has a lot of edge-cases and cross-game interactions that are not easy to design for without actually throwing it against the wall.
I made a guide for hosting both sync and async sessions in the #get-started section of the community Discord server, for the purpose of using the AginahBot commands to schedule them.  You're always welcome to schedule testing privately as well, though public sessions have the advantage of more players (aka more data points) and a better public record of testing.  The big takeaway is that PR approval is frequently slow in this project due to the small number of core developers, and whatever we can do to test functionality against the rest of the project in advance is a boon to speeding up that process.
Sorry for longposting, I don't think I forgot anything relevant?  That comment won't age well, I'm sure.

Oh I'm pretty bad at reading.  If you're pretty confident that the PR against the Client repo is compatible with the Archipleago server behavior as-is, then yeah it'd probably make a lot of sense to just PR against this repo now as a prelude to server changes.  Sorry, I'm a bit blind sometimes and completely missed the actual question while ranting.

The main asterisk on sending a PR as-is is that this also changes the way users need to set up the randomizer. I'm happy to update the documentation for this as well, but it's a bit more complex than just doing an in-place replacement of this client.

If so, then it might make a lot of sense to open a Draft PR with a comment explaining the new setup, and then commit doc changes alongside it once it's converted to a normal PR.  That way people will see the intended changes sooner and consider testing them, while not holding things back over the docs being prepared immediately.

This looks amazing, I'll take a look later this day

I've always wanted to use Matts version of the rando for AP, so seeing it come to light has no qualms with me. There was talks of adding a button to the offline rando for Enemizer, but never needed to as it just works with base rando as it is.
When it comes to changes, I usually always hit up Marech and ask "Hey mind if blahblahblah?" just to be transparent and make sure they know whats going on with their implementation, as they are the original person that made DS3 even happen in AP.
All that being said, I will be excited to stress test this and report anything to help progress this version more.

Latest update: this is now fully working, end-to-end, including:

Randomizing items in shops (including cross-world items).
Enemy randomizer (including exposing settings through Archipelago and choosing Storm Ruler's location relative to where Yhorm ends up).

There are a few quality-of-life changes I want to make, and I want to update the documentation and do a full multiworld runthrough with some friends, but once I'm pretty confident it's in a solid place I'll package up the current offline randomizer and send all of this out for review.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
xinming365, @diulama, @lempere, @ianidi, @khadgachy07, @teebui, @mutahhirkhan, @littleboycoding, @qzz0518, @JustAPotato0916, @damonliu1991, @huemy221338, @helnwein, @gtaschuk, @sumitpatel93, @kapdap, @sheikhimtiaz, @LIYANG-UST, @fruitcow, @mdastronaut

Email you got about github hiring is fake
Visit: https://github.com/Marechal-L/Dark-Souls-III-Archipelago-client/issues/17 to find the content which you received in the mail. Just a friendly developer letting you know of the scam.
xinming365, @diulama, @lempere, @ianidi, @khadgachy07, @teebui, @mutahhirkhan, @littleboycoding, @qzz0518, @JustAPotato0916, @damonliu1991, @huemy221338, @helnwein, @gtaschuk, @sumitpatel93, @kapdap, @sheikhimtiaz, @LIYANG-UST, @fruitcow, @mdastronaut

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards, GitHub Recruitment Team xinming365, @diulama, @lempere, @ianidi, @khadgachy07, @teebui, @mutahhirkhan, @littleboycoding, @qzz0518, @JustAPotato0916, @damonliu1991, @huemy221338, @helnwein, @gtaschuk, @sumitpatel93, @kapdap, @sheikhimtiaz, @LIYANG-UST, @fruitcow, @mdastronaut


404 Error, did you delete the website?

Hey FaceDev, yesterday, i saw your video and i wanted to sign up, but

Anyways i hope i didnt mess with your time and i hope you fix it

The website has closed
Duplicate of #8 and #7

Host it on your own domain


Service Request by Ava Hamilton - Access to Tracker data

[x] Qualtrics Service Request Form was received from User
[x] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[x] Problem FULLY RESOLVED and Issue is Closed
[ ] Post-Service e-mail was sent to Ava to let her know we set up her limited access account

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

From: david lounsbury
Sent: Wednesday, August 18, 2021 6:56 PM
To: Hamilton, Ava D<EMAIL_ADDRESS>Cc: Nasim Sabounchi<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>Subject: RE: IPA tracker data access
Hi Eva:
At the moment we are still very much piloting the tool with the Wave 1 counties.
Functionality is quite good, but we are still working to enhance its utility and, in spots, still debugging.
And as for the completeness and the quality of the data that specifies each county’s set of IPAs/Smart Strategies (and linked Measures, Partners, Venues, Resources, Challenges), we are at a critical juncture.
As we are working with them to clean up their IPAs/Smart Strategies, we are also starting to help counties enter MONITORING data (for IPAs/Smart Strategy Measures, Partners, Venues, Resources, Challenges). These data are so important to the study, as they will document implementation success (or failure!). And these data are still largely either not yet entered and/or are not being captured.
All this is to say that the data in the Tracker is not ready for any sort of analyses. Frankly, I think we need another 3-4 weeks and at least one more ORCATT reporting cycle to really see clean and complete SPECIFICATION data, and then probably 3-4 more weeks to start to see useful MONITORING data.
You raise an important point about the process for HCS staff requesting and working with the Tracker data. I would like to consult with Nabila, Tim, and Bruce R, among others, about this, too.
But in the meantime, what I am comfortable doing is giving you a username and a password that maps you to our ‘practice county,’ so you can see the tool’s structure and its functionality, including how data is extracted (down loaded). None of the data is real, but you can make up data to see how things work. If this is agreeable, submit a ‘Request Service’ record using the following page: https://github.com/dlounsbu/NYHEALing-IPAtracker-support.
Relatedly, please note that we have not written any turnkey reports yet, but we are starting to design them. I know we are both interested in baseline values for selected strategy measures, as well as in ‘planned’ vs. ‘actual’ trajectories for these strategy measures. Your collaboration on this front is very welcome.  Let’s discuss further, soon.
Best and thanks,
David
From: Hamilton, Ava D<EMAIL_ADDRESS>Sent: Wednesday, August 18, 2021 5:24 PM
To: david lounsbury<EMAIL_ADDRESS>Cc: Nasim Sabounchi<EMAIL_ADDRESS>Subject: IPA tracker data access
CAUTION: This email comes from an external source; the attachments and/or links may compromise our secure environment. Do not open or click on suspicious emails. Please click on the “Phish Alert” button on the top right of the Outlook dashboard to report any suspicious emails.
Hi David,
I hope you are well. I appreciated your HCS presentation a few weeks ago about the IPA tracker and how it can create data reports. I spoke with Nasim last week, and she mentioned that you are in charge of data acccess. I am not sure what the process is for those working on the HCS, but I am very interested in looking at/working with the data. Could you let me know what would be a good way to proceed?
Thanks so much,
Ava
Ava Hamilton
Senior Data Analyst
Department of Epidemiology
Columbia University
Mailman School of Public Health
722 West 168th Street, #726
New York, NY 10032

From: David Lounsbury<EMAIL_ADDRESS>Sent: Thursday, August 19, 2021 10:34 AM
To: david lounsbury<EMAIL_ADDRESS>Subject: IPA Tracker Support Request Form Submitted
IPA SERVICE REQUEST SUMMARY

CONTACT INFORMATION:

Ava  Hamilton
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER>

COUNTY

All counties

WHICH FEATURE(S):

REPORTING: Downloading data to create a table, figure or other summary report

BRIEF PROBLEM DESCRIPTION:

No issue, would just like to request access

URGENCY:

25

ANYTHING ELSE TO SHARE:

NA

I set up a user name and password for Ava. She will have access to only the xPracticeCountyx for now. Once our data is more complete and once we consider policies for HCS staff accessing Tracker data, we can revisit her access.

From: david lounsbury
Sent: Thursday, August 19, 2021 12:04 PM
To: Hamilton, Ava D<EMAIL_ADDRESS>Subject: RE: [EXTERNAL] RE: IPA tracker data access
Hi Ava:
See if you can log into the Tracker now. You will only be able to work with the Practice County for now!
Your username is: xxx
Your password is: xxx
Let me know if you have any problems.
DL


Secret Derivation Path

I have secret's bot derived from the seed showing insufficient fee (balance too low to send tx)
No matter whether I override the derivation path to secret network (529 or 118), the issue persists.
on keplr, the balance for Secret on the same seed as the other accounts (which reflect the actual balance the bot should have) is more than 20 SCRT (plenty to use).
do i have to transfer this balance to another address or do i have to adjust restake?

@gaia sorry for the delay - I can't think what would cause this if you can't find a balance under either derivation path. 529 is the 'correct' one for Secret I believe, and that's what I'd expect Keplr to be using. Assuming it's a small amount, the easiest option is probably to use 529 (correctSlip44 config option) then just send a small amount of tokens to the address REStake outputs. Not ideal but I'm not quite sure what's going on there, I haven't heard any similar reports

thanks tom. PR 3411 should allow me to use the 529 address.


MudTable Checkbox Visual Issues

Bug type
Component
Component name
MudTable
What happened?
I have seen a few different bugs regarding the visualization of the a selected item in a table not being checked when really it is selected.
Expected behavior
For both scenarios, I expect the check boxes would appear checked.
Reproduction link
https://try.mudblazor.com/snippet/QYGlbbPEqkhYAQeI
Reproduction steps

Multi-select with table filtering. Code copied from documentation and added multi selection: https://try.mudblazor.com/snippet/QYGlbbPEqkhYAQeI

If you start with the standard "select all" checkbox it will work
If you enter "s", press enter, then select all, I am seeing all but the first box are visually selected
If you enter "c", press enter, then select all, random check boxes are visually selected

Multi-select with table filtering and virtualization. Code copied from documentation and added multi-select, virtualize, and inserted a height: https://try.mudblazor.com/snippet/wamPFbbEgbRCcBQM

Set the rows per page to 50, select all, and scroll down the page

Relevant log output
No response
Version (bug)
6.0.2
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

The same as #2669

Will this issue be fixed? I have encountered these issues as well.

When using filtering in a table and doing selections on the filtered subset, the selected checkbox gets lost when removing the filter and showing the full list again.

https://try.mudblazor.com/snippet/wEGmYdbJcyMKYCMm

When using a table with virtualisation and a fixed height, the selected checkbox gets lost. Select an item from the top, scroll down and then back up. The checkbox is not set anymore. Probably the items get re-rendered then and the checkbox state is lost. Internally, the item is still in the list of selected items.

https://try.mudblazor.com/snippet/GOcwuRPzGINpbIOu
These minmal examples reproduce the issue very well. Tested with latest MudBlazor 6.0.7.

Hi - we are unfortunately still bound to .NET5 for our project. Would it be possible to get that patch into the 5.x version of MudBlazor?

Thanks - will take a look!

PR into the net5 branch ready for review: https://github.com/MudBlazor/MudBlazor/pull/5189
Thanks!

The bug is fixed without virtualizaton, but with virtualization enabled it is still present in 6.0.15. The selected checkbox gets lost when it gets re-rendered due to virtualization. But once you click on another element, it suddenly gets selected again. Probably because an UI update renders everything again.
Can be verified with the same snippet I posted above:
https://try.mudblazor.com/snippet/GOcwuRPzGINpbIOu
https://user-images.githubusercontent.com/79685270/188882681-2dddd27f-334b-49af-83ba-45f3049893ce.mp4


[TASK] Add Check on grid object for grid sampler

Issue Description
Is your feature request related to a problem? Please describe.
In the grid sampler a check is not being performed on the grid object when type is set to CDF.
Describe the solution you'd like
A check that make sure that provided values are in the [0,1] interval
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

checklist passed.


Feature Request: Option to represent italic with single asterisks

Thanks for developing this great extension and making it open source!
It would be convenient for the tool to to offer an option to have italic format represented in Markdown output as text enclosed in single asterisks (eg. italic text becomes *italic text*) instead of underscores (eg. italic text becomes _italic text_), which are used currently.
This "italic format" option would also represent text that is both bold and italicized using triple asterisks instead of nested double asterisks and single underscores (eg. really important text becomes ***really important text*** instead of really important text becoming **_really important text_**).
Use Case: I have a markdown application (wiki.js) configured to render underscore-enclosed text as underlined instead of italic. I know this underlining behaviour is not standard markdown, but single asterisks for italic is standard markdown, and it would be very helpful to have this option.

Another problem with using underscores for italic rather than asterisks is that underscores do not work for italicizing portions of words, whereas asterisks do. There's a note about this in The Markdown Guide: https://www.markdownguide.org/basic-syntax/#italic-best-practices
For example...
This works: "It's levi*o*sa not levio*sa*" > "It's leviosa not leviosa"
This doesn't work:"It's levi_o_sa not levio_sa_" > "It's levi_o_sa not levio_sa_"
This presents another use case for the suggested option. Also, it makes me consider whether the default should be asterisks (perhaps the more robust option) or underscores (the existing behaviour). I'll leave that up to the maintainer(s) to decide though!

This configurability could also potentially be extended to bold formatting (double underscores or double asterisks); however, just the change for italic format is suffice for the 2 use cases I shared above.

/* now the default for italic/bold markup in Markdown.


Air Gapped experience consolidation proposal

Description of changes:
In the last few months, we have built multiple features aiming in the same direction but they organically grew in different directions. This design aims to combine them into a more cohesive solution.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve cancel

Just a thought for the future - it would be nice to build a list of registry implementations that support custom OCI artifacts so we can quickly respond to customers in the event they have preferred tool sets that include a registry.

/approve


Is it possible to use custom steps in a FormStep?

I am trying to create a Form step with custom steps. I am met with a serialization issue, however:
java.lang.RuntimeException: Unable to start activity ComponentInfo{com.test.example/org.researchstack.backbone.ui.ViewTaskActivity}: java.lang.RuntimeException: Parcelable encountered IOException reading a Serializable object (name = org.researchstack.backbone.task.OrderedTask)
Everything is serializable on both sides, as far as I can tell. Any ideas?

@suomi35, see this for a working implementation. If you add some more info, I might be able to help troubleshoot.

It's hard to say without seeing the code for you CustomSteps, but if I had to guess it would be that one of them does not fully implement Serializable.  For Serializable to work, you must provide a default constructor for that class and make sure that all the member variables of your CustomSteps are also Serializable.

I was able to get this working. I was using only StepLayout instead of a StepBody. I converted all of my custom steps to use StepBody and it works fine now.
Thanks @jdkizer9 for the example!


Grid should have overrides for phone, mobile and desktop

The Block component of the Grid allows for setting column sizes that apply to desktop only. Mobile overrides whatever block size is specified to be the full size (or in this case, width).
What we need is to have block sizes that can be overridden by mobile and tablet. For example:
<Block classes={'e-row'}>
<Block classes={'brick brick-3 brick-6-tablet brick-12-mobile'}>brick-1</Block>
<Block classes={'brick brick-3 brick-6-tablet brick-12-mobile'}>brick-1</Block>
<Block classes={'brick brick-6 brick-12-tablet brick-12-mobile'}>brick-6</Block>
</Block>

In the above example:

Desktop, we would have one row, three columns, with size 3, 3, and 6.
Tablet, we would have two rows, first column would have size 6,6 and last row would have col 12
Phone, there would be three rows, each row size 12

Hello Barry, you can try the new responsive grid http://getessence.io/core/grid-system.
tkanks again for the hint!


rangeEnable bug: return ErrFrameFieldsNotAllowed if f.rangeEnabled

Overview

frame's rangeEnable should set = true when creating a frame with option RangeEnable=True
return ErrFrameFieldsNotAllowed when f.rangeEnabled=False

Pull request checklist

[ ] I have read the contributing guide.
[ ] I have agreed to the Contributor License Agreement.
[ ] I have updated the documentation.
[ ] I have resolved any merge conflicts.
[ ] I have included tests that cover my changes.
[ ] All new and existing tests pass.

Code review checklist
This is the checklist that the reviewer will follow while reviewing your pull request. You do not need to do anything with this checklist, but be aware of what the reviewer will be looking for.

[ ] Ensure that any changes to external docs have been included in this pull request.
[ ] If the changes require that minor/major versions need to be updated, tag the PR appropriately.
[ ] Ensure the new code is properly commented and follows Idiomatic Go.
[ ] Check that tests have been written and that they cover the new functionality.
[ ] Run tests and ensure they pass.
[ ] Build and run the code, performing any applicable integration testing.

@linhvo I believe these bugs are addressed in PR #840

@travisturner: got it. I'm closing this PR


[perf] set a fetch size to reduce peak memory usage

@dr-itz I am a little confused by this enhancement.  So this change will limit the default to only fetch at most 100 but we can enable it via config to be any value?  Should 100 be a default size here?  I think even if this is a good default we need an option to explicitly disable the fetch size (e.g. -1 will not set fetch limit).

Note that it's not a limit, just tells the driver how many rows to fetch at once from the server. For most drivers, the default is to fetch all rows at once which can use quite a bit of memory.
Anyway, it's just a hint for the driver. 0 means default behavior (fetch all at once), so no special case like -1 is necessary. We could also not set it when it's 0...and maybe even leave the default at 0...dunno. But the config option is important for performance tuning.
A fetch size of 100 is a good default, mostly from experience, also it won't affect most queries in a Rails app. And for anything that fetches a few thousand rows it will decrease memory usage and increase speed. The "perfect" fetch size of course depends on the data being fetched, but the difference between 100 and e.g. 500 is usually so small in terms of performance that 100 or 200 is good enough.

@dr-itz thanks for the explanation.  I personally feel like the default should be the JDBC default but I guess I am not someone who has a lot to bring to the discussion.  @kares @rdubya what do you guys think about this?

no strong opinion here - would maybe leave the default (explicit) 'fetch-less' and allow tuning with a config option (+ driver specific properties). been a while so I believe we should review jdbc driver (mysql/pg) docs on this, or maybe even how the native ones behave. also having some raw benchmark numbers on queries where this makes sense, would be useful?

I don't know enough about how that will affect things to have too much of an opinion. One question that does pop up in my mind though is how this works with things like pg_badger. We can't use prepared statements because of pg_badger since the connections to the actual database are shared among the connections to pg_badger, so the statement might get prepared on one connection and then executed on another. Will this fetching setting have the same problem or does it count as one continuous connection?

@dr-itz @kares @rdubya  I guess based on the collective feedback we have 2 leave default unless we learn something, 1 no opinion but wonders something (pg_badger), and your submission which would be to enable this.  Seems best path for the moment would be to accept this PR BUT change the default to 0 (which will be default fetch behavior).  This would allow this to be set and if we learn that we have good bench results or knowledge that a particular fetch size is best then a second PR can set that value.

Thanks for the feedback. pg_badger and friends is something I haven't considered. I changed it now to a complete opt-in behavior, not event trying to set the fetch size if it's 0 as there might be driver bugs around.

@dr-itz thanks again.  At some point we will need to update our docs on the project and provide current info for configuration: https://github.com/jruby/activerecord-jdbc-adapter/wiki/Configuration.

Just wanted to correct something I said, the technology we use is called pgbouncer not pg_badger 🤦♂, but the comment still applies. pg_badger is another tool that wouldn't be involved in this case.


JHU -> FH

Purpose/implementation Section
What changes are being implemented in this Pull Request?
There's some remnant JHU things in the OTTR_Template we need to fix this.

This'll do.


chore: Async api callback

Test URLs:

Before: https://main--icicidirect--aemsites.hlx.live/research/equity

After: https://api-callback--icicidirect--aemsites.hlx.live/research/equity

[ ] New Blocks introduced in this PR
If yes, please provide details below

Block Name
Documentation

For e.g. carousel
doc-link

@sdmcraft IMO exporting the getHostUrl was missed before the merging.
I am including it my PR https://github.com/aemsites/icicidirect/pull/92


SNOW-944696: SQL Compilation Error when running snow streamlit deploy command

SnowCLI version
2.0.0-dev
Python version
3.9.6
Platform
macOS-13.5.2-arm64-arm-64bit
What happened
When I run the following command:
snow streamlit deploy cars-app --file=Home.py -c mashsf --replace --query-warehouse MED_COMPUTE_WH
I get a SQL compilation error
FYI - I have already granted permissions as found in this link: https://docs.snowflake.com/en/developer-guide/streamlit/about-streamlit#label-streamlit-access-requirements
Console output
Exception message:
001003 (42000): SQL compilation error:
syntax error line 1 at position 32 unexpected '-'.
syntax error line 1 at position 32 unexpected '-'.
syntax error line 2 at position 0 unexpected 'ROOT_LOCATION'.

How to reproduce
I'm not sure.

Adding in a few screen shots of the deploy command with --debug stacktrace.

@mashhype Thanks fr the information but the full SQL error is missing from the screenshots (the text after SQL complation error). However you posted that you see there unexpected 'ROOT_LOCATION' please check if the account you use is part of streamlit PrPr.


[ARC-35] security vulnerability in markdown

Resolved markdown vulnerability by removing markdown.js library and adding a block of code to handle pulling issue keys from reference text.

Can you check if the right email is setup on your local git config, as I don't see proper identity on the commits

Have you tested the basic functionality with this fix?

Yep, tested the functionality out on master and made sure I saw the same behaviour on this branch with the new logic. Only diff is that the master keys are in lowercase (converted via the markdown library)

Can you check if the right email is setup on your local git config, as I don't see proper identity on the commits

Guess you're looking for my email attached to this account and not my atlassian email (which is what was in my gitconfig). I just updated it. Do I need to make another commit now so you can confirm?

Yes, its better to update the commit, I see now  in other PR for new commits its fine

Yes, its better to update the commit, I see now in other PR for new commits its fine

I just merged my other PR and then merged master into this one. Anything else I need to do to get this approved?

@rachellerathbone Please note that after merging into master, deployment is manually triggered by us. First, it's deployed on stage, we do the basic tests and then push to prod.


File format import/export

I'm having heavy color banding issues due to the heavy compression of h.264 to h.264. Are there higher-quality file format options for import/export?

I'm doing VFX work with this software, I'll love to have some sort of ffmpeg export option, so I can choose my own preferred lossless video codec or image sequences.


Is it possible to create task queue dynamically

I want to create a node server for scrapping using puppeteer (pass search term in GET request to scrap google search results)
currently my server is not able process more then 5 parallel request after its goes out of memory

I don't understand your question. Closing this for now.
Feel free to reopen, but please provide some more numbers (how much is your memory?) and specify your questions.


[enhancement] exit code parameter for cli

Been thinking of a way to automatically trigger success/error sounds based on whether the previous command succeeded on the terminal.
One way to accomplish this would be an --exit-code (-e) argument to be used as such:
% ls; chime --exit-code=$?

$? in bash and zsh stores the exit code of the previous command. An exit-code argument would allow the user to input it into chime, triggering an error on a non-zero exit code, success otherwise.
To be honest I wish there was a more automated way to handle this, but this is the best I can come up with at the moment!

Yes I agree that chiming with respect to the exit code would be wonderful. Maybe there's some way to pipe | or to redirect > into the chime command?

Maybe there's some way to pipe | or to redirect > into the chime command?

Haven't come across anything straightforward. The only way I could think of to getting the exit code it into stdin would be:
% command; echo $? | chime

Another option is for Chime acts as a wrapper for a command, running the command using os.system, or something like that. It would then have access to the exit code. This seems a bit complex to me, as chime would take on a bit of responsibility that the arbitrary command works as expected, and it might end up being a bit fragile. Open to ideas though!

This is a very old issue, but I'd like to add this for anyone stumbling upon it from their search engine:
In Linux, you can use logical operators to achieve largely the same functionality.
command && chime || chime error
This would produce the success sound if command is successful, otherwise it would produce the error sound.


Cyclic mapping issue in projection?

Hi, I have a test case that does not behave correctly when doing projection. (@Data is Lombok's way of generating getters and setters, asserts are using AssertJ syntax).
@Data
private static class Action {
    UUID id;
    Action previous;
}

@Data
private static class ActionDTO {
    UUID id;
    UUID previousId;
}

@Test
public void testPreviousLink() {
    ActionDTO dto = new ActionDTO();
    dto.setPreviousId(UUID.randomUUID());
    dto.setId(UUID.randomUUID());

    Action model = modelMapper.map(dto, Action.class);

    assertThat(model)
            .isInstanceOf(Action.class);

    assertThat(model.getId())
            .isEqualTo(dto.getId());

    assertThat(model.getPrevious())
            .isNotNull() // fails here when strict
            .isInstanceOf(Action.class);

    assertThat(model.getPrevious().getId())
            .isEqualTo(dto.getPreviousId()); // fails here with normal matching
}

This test fails on the last assert, because model.getPrevious().getId() is null. It fails on the third assert (isNotNull) when strict matching is enabled.

So there is indeed an explicit type cycle prevention in the code. It is present in org.modelmapper.internal.ImplicitMappingBuilder#matchSource in the form of
  if (!doneMatching
      && isMatchable(accessor.getType())
      && (!sourceTypes.contains(accessor.getType()) || accessor instanceof ValueReaderPropertyInfo))
    matchSource(TypeInfoRegistry.typeInfoFor(accessor, configuration), destinationMutator);

The condition starting with !sourceTypes is responsible for that.
I understand that infinite recursion must be prevented, but this actually prevents a perfectly valid use case of trying to convert a child-parent relationship too.


Field MorphTo::make('Causer', 'causer') not displaying anything.

In fields for the ActivityLog resource: MorphTo::make('Causer', 'causer'),  doesn't return a displayable value. It simply shows a dash for all activities. Meanwhile the database has the correct causer type and causer id.
Did you need to add anything to the user model to accept the relationship? or to the Nova User resource?

What version of nova are you using? Can you provide some more detailed information?

@bolechen
In the fields for the ActivityLog resource  MorphTo::make('Causer', 'causer')  shows a dash like I highlighted in the yellow part below.

SO I had to extend the class and using a text field I generated the link to the user object. Not ideal but works as you can see above  the next column to the yellow one.
public function fields(Request $request)
    {
        return [
...
            MorphTo::make('Causer', 'causer'),
            Text::make('Causer', 'causer_id')->displayUsing(function ($user_id) {
                return ''. strtoupper(\App\Models\User::find($user_id)->username) . '';
            })->asHtml(),
....

We're still using Nova 3.

@bolechen Found the answer. Might want to add to the documentation that the Causer requires that the Causer Nova resource has the title defined for it to display on the index and select inputs on the form view.
/**
 * Get the value that should be displayed to represent the resource.
 *
 * @return string
 */
public function title()
{
    return $this->name;
}

Or  public static $title = 'name';
Ref: https://nova.laravel.com/docs/1.0/resources/relationships.html#title-attributes


Handle BaseReader going away before an open file.

Our application doesn't close any open SICD files explicitly and maintains a reference to them throughout the application, so they're not garbage collected until shutdown.  At that point BaseReader has been destructed before our open file and we got a ton of errors like:
Exception ignored in: <function BaseReader.del at 0x7fd05b307a30>
Traceback (most recent call last):
File<EMAIL_ADDRESS>line 463, in del
File<EMAIL_ADDRESS>line 2450, in close
AttributeError: 'NoneType' object has no attribute 'close'
I haven't been able to come up with a standalone replication of this problem, but this works for me.

I don't know why the garbage collection would set the BaseReader reference to None.  If you replace BaseReader.close(self) with super().close(), does the error go away?

Gets further, but now DataSegment is None at a later point https://github.com/ngageoint/sarpy/blob/3354207e6e8d8920416d1248c9c626c7d5a86b47/sarpy/io/general/base.py#L439
Exception ignored in: <function BaseReader.__del__ at 0x7f87f307f910>
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 463, in __del__
  File<EMAIL_ADDRESS>line 2450, in close
  File<EMAIL_ADDRESS>line 439, in close
TypeError: isinstance() arg 2 must be a type, a tuple of types, or a union

Unfortunately python does not guarantee that __del__ methods will work during shutdown.  See the second bullet of the warning on __del__’s documentation

Unfortunately, I don’t think it is practical to make SARPy's __del__ methods accommodate running during shutdown.  The exact order of events during shutdown isn't defined or guaranteed, which could lead to a cat-and-mouse game of trying to make the __del__ methods run without error.


Updated the internal documention for the GDScript class.

Thanks, I had changed it there this morning too and forgot to commit it. It's been updated.


Replacing containerd GRPC API with client

This commit

Replaces the usage of containerd GRPC APIs with the containerd client for all operations related to containerd.
Updated containerd to v1.0alpha4+
Updated runc to v1.0.0

Signed-off-by: Abhinandan Prativadi<EMAIL_ADDRESS>

@abhinandanpb Thanks! Ping me when you are ready, and feel free to tell me if you have any questions. :)

I have signed the linux foundation cla

@Random-Liu updated PR. Thanks for reviewing the changes multiple times :)

@abhinandanpb LGTM, let's wait for the test result.

Done.

LGTM
Will merge after test passes.
@abhinandanpb Thanks a lot! :)

Fixes https://github.com/kubernetes-incubator/cri-containerd/issues/86.
Fixes https://github.com/kubernetes-incubator/cri-containerd/issues/49.


Registering of 4me in the Azure Active Directory application gallery : no access

I need to list  our application 4me (see https://4me.com ) in the in  the Azure Active Directory application gallery.  I followed the instructions in this article . When trying to login on the  Application Network Portal with my Microsoft account<EMAIL_ADDRESS>I get an error 'We're sorry, but<EMAIL_ADDRESS>can't be found in the microsoft.sharepoint.com directory.'  What can I do to proceed?

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

ID: 806e36a4-f5ae-bc07-e7e2-ea9d2ab98131
Version Independent ID: 3dea2974-8f2b-0e21-b7a2-b1381cf127b6
Content: List your application in the Azure Active Directory application gallery
Content Source: articles/active-directory/develop/active-directory-app-gallery-listing.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

Thanks for the feedback! We are looking into this and will get back to you shortly.

@wouterwyns Please contact Azure AD SSO Integration Team -<EMAIL_ADDRESS>for your access issues.

@wouterwyns We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

This application is already listed in the gallery now.
#please-close


ResourceLinks Escaped Incorrectly

Editing OccurrenceNote in preview (html) mode causes RC links to be escaped incorrectly.
Occurs in PROD (1.0.5)

open TN
edit OccurrenceNote in preview mode. (Edit anything in the block)
click outside of the field (necessary in 1.0.5)
toggle preview off
NOTE: the rc link syntax is bad now

e.g.,
“I, John the elder, am writing” (See: [[rc://en/ta/man/translate/figs-explicit]])
is incorrectly escaped as:
“I, John the elder, am writing” (See: \[\[rc://\*/ta/man/translate/figs-explicit\]\])

This issue reminds of a thought I've had several times over the past few months. Why do we permit editing in both Markdown and HTML? Issues like this on show that bi-directional translations are not lossless... which means we will always be fighting battles like this.
Github, Zulip, etc. only allow you to view the HTML (see preview button on this very input box!).
If that is the best practice, shouldn't we follow?
@ancientTexts-net @birchamp

This issue reminds of a thought I've had several times over the past few months. Why do we permit editing in both Markdown and HTML? Issues like this on show that bi-directional translations are not lossless... which means we will always be fighting battles like this.
Github, Zulip, etc. only allow you to view the HTML (see preview button on this very input box!).
If that is the best practice, shouldn't we follow?
@ancientTexts-net @birchamp

Not seeing this issue in translationCore Create - v1.1.0-rc.3. Able to edit Occurrence Notes without mangling the RC link.

Not seeing this issue in translationCore Create - v1.1.0-rc.3. Able to edit Occurrence Notes without mangling the RC link.


[ACR] Remove support for Germany cloud

We should remove the support for Germany cloud in our SDK as the cloud has already been deprecated.

Done in PR https://github.com/Azure/azure-sdk-for-python/pull/27675


fix(acs-identity): remove duplicate identities path from example endpoints

Changelog
This is an update to an existing version to fix a swagger quality issue that affects our documentation. It just updates documentation. So shouldn't need any changes in SDKs etc.
It can be released as soon as is suitable.
In all of our examples for ACS Identities, we mistakingly added an extra "identities" to our endpoint variable. This leads to the examples reading as: "identities/identities".
You can see this here:

(https://docs.microsoft.com/en-us/rest/api/communication/communicationidentity/create)
Contribution checklist:

[X] I commit to follow the Breaking Change Policy of “no breaking changes
[X] I have reviewed the documentation for the workflow.
[X] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[X] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

#sign-off

#sign-off