MODPATBLK-84 Remove pubsub unregistering logic

MODPATBLK-84
Purpose
According to investigation in MODPUBSUB-147, when module is restarted it unregisters itself from pubsub after Okapi's call to DELETE _tenant and doesn't get registered again, because in some situations Okapi doesn't make a call to POST _tenant endpoint when module is started. This leads to mod-circulation being unable to publish events. mod-pubsub developers say that unregistering is unnecessary and some modules don't do it, because re-registering doesn't cause any issues. This was confirmed by re-registering manually on the snapshot env.
Approach
Unregistering logic was removed from DELETE _tenant.

INFO: lint-raml: Using lint-raml version: 1.4.5
INFO: lint-raml: Investigating and determining configuration: mod-patron-blocks/ramls
INFO: lint-raml-schema: Assessing schema files (https://dev.folio.org/guides/describe-schema/):
INFO: lint-raml-schema: Found 19 JSON schema files under directory './ramls'
ERROR: lint-raml-schema: loan-policy.json: Missing top-level "description".
ERROR: lint-raml-schema: loan-policy.json: Missing "description" for: description, holds, loanable, loansPolicy, name, pages, recalls, renewalsPolicy, requestManagement
ERROR: lint-raml-schema: period.json: Missing top-level "description".
INFO: lint-raml: Not assessing RAML/Schema or examples against schema, as per option '--json-only'.
ERROR: lint-raml: There were processing errors. See list above.

Note: When those ERRORs are fixed, then any INFO and WARNING messages will be gone too. See Jenkins "Artifacts" tab for LintRamlSchemaReport.

INFO: lint-raml: Using lint-raml version: 1.4.5
INFO: lint-raml: Investigating and determining configuration: mod-patron-blocks/ramls
INFO: lint-raml-schema: Assessing schema files (https://dev.folio.org/guides/describe-schema/):
INFO: lint-raml-schema: Found 19 JSON schema files under directory './ramls'
ERROR: lint-raml-schema: loan-policy.json: Missing top-level "description".
ERROR: lint-raml-schema: loan-policy.json: Missing "description" for: description, holds, loanable, loansPolicy, name, pages, recalls, renewalsPolicy, requestManagement
ERROR: lint-raml-schema: period.json: Missing top-level "description".
INFO: lint-raml: Not assessing RAML/Schema or examples against schema, as per option '--json-only'.
ERROR: lint-raml: There were processing errors. See list above.

Note: When those ERRORs are fixed, then any INFO and WARNING messages will be gone too. See Jenkins "Artifacts" tab for LintRamlSchemaReport.

INFO: lint-raml: Using lint-raml version: 1.4.5
INFO: lint-raml: Investigating and determining configuration: mod-patron-blocks/ramls
INFO: lint-raml-schema: Assessing schema files (https://dev.folio.org/guides/describe-schema/):
INFO: lint-raml-schema: Found 19 JSON schema files under directory './ramls'
ERROR: lint-raml-schema: loan-policy.json: Missing top-level "description".
ERROR: lint-raml-schema: loan-policy.json: Missing "description" for: description, holds, loanable, loansPolicy, name, pages, recalls, renewalsPolicy, requestManagement
ERROR: lint-raml-schema: period.json: Missing top-level "description".
INFO: lint-raml: Not assessing RAML/Schema or examples against schema, as per option '--json-only'.
ERROR: lint-raml: There were processing errors. See list above.

Note: When those ERRORs are fixed, then any INFO and WARNING messages will be gone too. See Jenkins "Artifacts" tab for LintRamlSchemaReport.


traincontroller issue - Stuck waiting for r_queue

Hi, the train controller seems to be forever waiting for the results in the result queue.
Anything I should look for in particular ?
The command I ran:
python traincontroller.py --logdir exp_dir --n-samples 4 --pop-size 4 --target-return 950 --display --max-workers 1 (I tried 4 workers, same, and 32 workers my RAM explodes)

As mentionned in the README, if you are on a headless server, you need to run the command using xvfb-run to simulate a display. Logs should be available in exp_dir/tmp. In exp_dir/tmp, you should have a couple of .err and .out files, can you cats one of the .err file, and the corresponding .out file ?

It was that, but it's not a headless display. Does the fact that I have xvfb installed explain it ?

Not as far as I know. Did you have time to cat the log files ? This could be related to an ill set DISPLAY env variable, but I'd have to look at the logs to have a clear idea. Is the training running properly now ?

It probably was that, I tried to reproduce and now it launches the window correctly.
What kind of values did you have when you reproduced the paper's result I'm having VAE around 37 MDRNN 1.0 And the controller reward's -8 (?)

For how many epochs was the controller trained ? The controller takes quite a long time to obtain decent results.

By the way your VAE loss is a bit high, I typically get around 33. You can have a look at the samples produced in the dir exp_logdir/vae/samples. Can you tell me if they look abnormal?

The VAE seems a bit high I agree, I had versions where it was slightly slower (33 or around that I am not exactly sure). I'm going to train more the controller.
http://nicolas.kwyk.fr/sample_1.png
http://nicolas.kwyk.fr/sample_2.png
http://nicolas.kwyk.fr/sample_3.png
http://nicolas.kwyk.fr/sample_4.png

Can you update me on wether the controller is properly training?

Controller got to 200 but nothing more and it seems to be oscillating somehow, which seems pretty wrong (I am trying to retrain from scratch to make sure things work, I happened to have had some OOM, as I'm training on my home workstation).

I am also having the same issue -- stopping at waiting for the r_queue and looping infinitely in while r_queue.empty():. I can see logs in my tmp dir, but the progress bar isn't increasing even if I have it running for 10 minutes. What could be the issue here? I'm running with the xvfb-run command on gcloud.


Permission denied when trying to enter the container

After following the setup, the container is constantly restarting as it hits a permission error. Any ideas what I could try?

I'm investigating this, and hope to have something to report tomorrow morning Pacific time (-0700)

@DeepVoltaire Can you paste the output of docker version?

That's the output from docker version

I still have the same issue even after trying with the latest changes and also trying to add Alex Hall's suggestion into the dockerfile. I tried cleaning everything up and rebuilding the image with running scripts/run-local.sh, even forcing BUILD=1, but still the same permission denied error.

Try this:

change the last line of Dockerfile to CMD ["sleep", "infinity"]
run scripts/run-local.sh in a separate terminal
run these commands and paste the output:

❯ docker compose exec ice bash
root@9c053fe69892:/code# id
uid=0(root) gid=0(root) groups=0(root)
root@9c053fe69892:/code# ls -ld ui
drwxr-xr-x 19 root root 608 Oct  4 16:48 ui
root@9c053fe69892:/code# ls -ld ui/.next
drwxr-xr-x 9 root root 288 Oct  4 21:09 ui/.next
root@9c053fe69892:/code# ls -ld ui/.next/package.json
-rw-r--r-- 1 root root 20 Oct  4 21:09 ui/.next/package.json

@DeepVoltaire Since going back and forth once a day will probably be pretty slow, I can pair-debug this with you any time Thursday or Friday: https://notime.zone/NDdd8-ynMUtv-

Thanks @lslunis I will try to get it to work on Friday and if it's still not working, I will gladly pair-debug with you!

@lslunis I could not make the time window, sorry!
There is no package.json inside ui/.next/, there is only a package.json inside ui/
> docker compose exec ice bash
root@d2a5b4d674d7:/code# id
uid=0(root) gid=0(root) groups=0(root)
root@d2a5b4d674d7:/code# ls -ld ui
drwxrwxr-x 9 1000 1000 4096 Oct  5 08:43 ui
root@d2a5b4d674d7:/code# ls -ld ui/.next
drwxr-xr-x 2 root root 4096 Oct  3 08:38 ui/.next
root@d2a5b4d674d7:/code# ls -ld ui/.next/package.json
ls: cannot access 'ui/.next/package.json': No such file or directory

Same error with the most recent master changes.

It looks like ‘ui/‘ isn’t owned by root, and I’m not sure how that’s possible. Are you running the docker daemon as a non-root user?
 I’m flexible for the next few days.

I will send you a message on Slack about fixing this live!
I can run "docker run hello-world" without sudo which I assume means I am running it as root. Here is the output of ls -lah /code from inside the container

We debugged this in a huddle, and the 1000:1000 ownership is coming from the host system via the volume mount. It’s unclear why this doesn’t affect anyone else using Docker Engine. For reference, this is happening with v2.10.2 of docker compose.

@DeepVoltaire were you able to work around this?

After installing docker desktop the error remains

The outputs now look like this, the files are now owned by root, but the file ui/.next/package.json still does not exist in the container. How should that exist? In the dockerfile I only see COPY ui/package.json ui/package-lock.json ui/, but this is only copying ui/package.json into /code/ui/package.json, right? And then COPY . . is copying everything into /code, but as there is nothing in ui/.next in this repo, nothing will be copied there.
> docker compose exec ice bash
root@d2a5b4d674d7:/code# id
uid=0(root) gid=0(root) groups=0(root)
root@d2a5b4d674d7:/code# ls -ld ui
drwxrwxr-x 9 root root 4096 Oct 10 11:11 ui
root@d2a5b4d674d7:/code# ls -ld ui/.next
drwxr-xr-x 2 nobody nogroup 4096 Oct  3 08:38 ui/.next
root@d2a5b4d674d7:/code# ls -ld ui/.next/package.json
ls: cannot access 'ui/.next/package.json': No such file or directory

ls -lah /code from inside the container

Let's follow up on Slack if you have any other ideas I could try.

I messaged you on Slack

@lslunis asked me if the ui/.next/ directory exists on the host, I checked, it did exist and was empty. Probably this prevented that ui/.next/package.json was created in the container. I deleted it with sudo rm -r ui/.next, rebuild the image and now everything works. Thank you very much for the help @lslunis !!


Optional forwards

This will change the default behavior of the library to exclude forward ip headers. Only when they are explicitly enabled, will they then work.
I am bumping the minor version of this as it will most likely break current installs.
I am going to leave this PR open for any feedback for a bit and then if there are no concerns, I will merge this in.

I took the opportunity to put a couple more changes into this PR. This is getting merged today.


Union in struct

Close #502

This change is

Ready for review

Reviewed 5 of 5 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

tests/union.c, line 65 at r2 (raw file):

void union_inside_struct()
{
	{int g;(void)g;}

What does this line do?

types/resolve.go, line 413 at r2 (raw file):

		}
	}
	inside := string(([]byte(name))[index : last+1])

Can you explain what is going on here?

Comments from Reviewable

Review status: 4 of 6 files reviewed at latest revision, 2 unresolved discussions.

tests/union.c, line 65 at r2 (raw file):
Previously, elliotchance (Elliot Chance) wrote…
What does this line do?

I removed this line.
That code help for searching the bug.

types/resolve.go, line 413 at r2 (raw file):
Previously, elliotchance (Elliot Chance) wrote…
Can you explain what is going on here?

// Example: 'union (anonymous union at tests/union.c:46:3)'
that line get string inside "(...)".

Comments from Reviewable

Reviewed 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

types/resolve.go, line 413 at r2 (raw file):
Previously, Konstantin8105 (Konstantin) wrote…
// Example: 'union (anonymous union at tests/union.c:46:3)'
that line get string inside "(...)".

Please add a comment to the code explaining what this code does and what and example looks like.

Comments from Reviewable

Review status: 4 of 6 files reviewed at latest revision, 1 unresolved discussion.

types/resolve.go, line 413 at r2 (raw file):
Previously, elliotchance (Elliot Chance) wrote…
Please add a comment to the code explaining what this code does and what and example looks like.

Done.

Comments from Reviewable

Reviewed 2 of 2 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable

Ready for review, merging


I would like to ask ng-admin to support a few ie, how to support more than IE8

Now I use the ng-admin project, how to support ie8. Thank you

Angular 1.3 has dropped support for IE8 (see https://docs.angularjs.org/guide/ie), so this isn't supported in ng-admin either.


Improve the contents of Coding/DataStructures/Introduction.md

Improve the content of the file.

@Iliveinbianries  can  u please assign this issue to me

Assigned to you ✅


Do not delete subprocesses when the type changed

Problem you would like to solve
I just had a diagram with some subprocesses. I thought i could change the subprocesses to be a manual task. That did not work (seems like a subprocess can not "globally" defined as manual), so i changed it back to collapsed subprocess. Now the content of the subprocess is lost.
Proposed solution

[ ] Add a hint "(removes content)" similar to our participant collapsing when changing a collapsed sub-process (with children!) to any other element (cf. https://github.com/camunda/camunda-modeler/issues/3291#issuecomment-1309894533)

Alternatives considered
or even disable the option with a hint
Additional context
No response

Thanks for your report.
Please double check if you can undo your changes. We do support it and we advice you to use it to revert "surprising" changes.

Just to double check: I guess you're imagining something like this (cf. "removes content" hint)?

i did not notice that my subprocess information is gone - i had about 4 subprocesses, changed all to be manual and then noticed that the "+" is gone and no hyperlink to any subprocess is available anymore. and of course i save my document after nearly every change, so maybe that helped to be not able to get back. its not that problematic as those subprocesses were only very small and simple ones. i already re-created them.

hyperlink to any subprocess is available anymore. and of course i save my document after nearly every change, so maybe that helped to be not able to get back.

Please tell me which modeler you're using. Undo/redo are not affected by safe, unless you discovered a critical bug in the Camunda Modeler (Desktop).

I use 5.5.0, i clicked about 10 times ctrl + z (thats default in  DE/CH for undo) but that did not recover my subprocesses.

I use 5.5.0, i clicked about 10 times ctrl + z (thats default in DE/CH for undo) but that did not recover my subprocesses.
Windows 10, Desktop version

I cannot reproduce this behavior using Camunda Modeler 5.5 and Windows 10 with a German keyboard.

I think we are mixing things now. I can also not reproduce that behaviour and i dont know what i exactly did, i will try to reproduce that when i find some time.
The other thing is (which is imho more important), that the user should be notified that he loses all subprocess information when he changes the task type

@damnmso We'll keep this feature request open. Indicating to the user that (invisible) contents are being removed is something we already do, cf. https://github.com/camunda/camunda-modeler/issues/3291#issuecomment-1309894533.


How are split moves performed?

I'm trying to play with a friend and we can't find how to perform a split move in the game. How is that done?

Hi Ethan
The shift key toggles between quantum and classical modes.
To perform a split, press shift once, and select the piece you want to move. Select the first position you want to place the piece at, and then select the second position. You have now moved the piece to two different position with a 50% chance of the piece being measured in each spot. Hope this helps!

@ayushidubal thanks! that helps :)


Games exported for iOS are not fullscreen/stretched despite the settings

Tested versions

Reproductible with Godot 4.3 RC 1 & Godot 4.3 Beta 3

System information
Godot v4.3.rc1 - macOS 14.5.0 - Vulkan (Mobile) - integrated Apple M3 Pro - Apple M3 Pro (12 Threads)
Issue description
/!\ iOS exports seem to be broken in some cases /!\
I want to make an iOS version of my game which displays just fine on Android devices. But when I run the game from XCode on my iPad, it isn't fullscreen/stretched despite the project settings. I also tried a bunch of hacks found on Google but 1) it doesn't work better and 2) I don't like hacks :D
You will find attached two pictures, one from my own game and another one from a MRP.

Steps to reproduce

Create a Godot project on MacOS with the expected project stretch settings
Export the project for iOS
Open the project with XCode and run it on a device which has a higher resolution than 1080 x 1920

Minimal reproduction project (MRP)
Archive.zip

Does it work on 4.2.2?
CC @bruvzg

Does it work on 4.2.2?
CC @bruvzg

I can do the test if you like. I was working with 4.2.1 before but only when I was working on the Android version.

That would be helpful, as there are some improvements and fixes to platform support in each minor release, but it can happen that they introduce regressions. Knowing if this is "just" a bug or a regression in 4.3 would help figure out the proper fix.

You have integer viewport scaling enabled in the project settings, this is how it will look on any device unless it resolution exactly integer multiple of whatever you set in the setting.

Tested with RC1. With fractional scaling, it's covering all screen on iPad. With integer scaling it had black bars on both iPad and Mac, I have not checked whether all scaling calculations are correct, but it seems like expected behavior, and it's not iOS specific.

Never saw that on Android but if you say it's the expected behaviour, you're the boss, boss! ^^ I'm gonna try with fractional scaling.
I put integer because I read it was better for 2D pixel games even though my current game isn't one. So how are you supposed to display pixel perfect fullscreen on iPads ?

Tested with RC1. With fractional scaling, it's covering all screen on iPad. With integer scaling it had black bars on both iPad and Mac, I have not checked whether all scaling calculations are correct, but it seems like expected behavior, and it's not iOS specific.

It works! <3 Thank you

Just a tiny related question: Window.min_size can only be set through code?


Pedantry is its own reward.

MAAS not MaaS.

Booooo you're right.
10:05 <kiko> the name is MAAS

Interesting commit message...  mas o menos.


Error when redirecting output on Windows

When I try to redirect the output of "conda-tree deptree" to a file or pipe through more or tee on Windows 10, I get the following error:
Traceback (most recent call last):
File "C:\Users\manow\anaconda3\Scripts\conda-tree-script.py", line 9, in
sys.exit(main())
File "C:\Users\manow\anaconda3\lib\site-packages\conda_tree.py", line 345, in main
print(''.join(complete_tree), end='')
File "C:\Users\manow\anaconda3\lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 21-22: character maps to
However, if I just let the output go to the console, I have no problem.

FYI, I tested this both from CMD and from bash (installed from posix conda meta-package), and I get the same error in either.

hello,
I don't use windows, so I'm not sure what this means, does this also happens with other python scripts ?
does setting set PYTHONIOENCODING=utf-8 helps ?

Setting PYTHONIOENCODING=utf-8 got rid of the crash, but the resulting
output file has many escape codes in it.  I see you have coloarama as a
dependency, so I'm guessing you're trying to color the output.  However,
interestingly, I don't see any colors when I output the results to the
console.
On Thu, Dec 16, 2021 at 9:33 AM Renan Valieris @.***>
wrote:

hello,
I don't use windows, so I'm not sure what this means, does this also
happens with other python scripts ?
does setting set PYTHONIOENCODING=utf-8 helps ?
—
Reply to this email directly, view it on GitHub
https://github.com/rvalieris/conda-tree/issues/16#issuecomment-995872479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6TT7ZQ6OHUNJLCL7EX5JDURH2CHANCNFSM5KEETZCA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

currently colorama is only being used to dim the package versions, like a dark gray-ish color, I don't know if cmd supports this.
there are 2 issues here I think:

there is a colorama.init() that detects windows and change the escape codes, the script is not calling this currently, so the escape codes are probably wrong.
the deptree output uses box drawing characters, without knowing the encoding, these characters might show up as gibberish in the file

try this:

add import colorama; colorama.init() to the top of the script
try setting PYTHONIOENCODING=utf-8 or PYTHONIOENCODING=utf-16 and check the file results

Sorry for the delayed reply; it looks like I missed this message when you
originally sent it.  Regardless, it appears that adding the  colorama.init()
call and setting PYTHONIOENCODING=utf-8 has fixed both issues.
--David
On Fri, Dec 17, 2021 at 6:57 AM Renan Valieris @.***>
wrote:

currently colorama is only being used to dim the package versions, like a
dark gray-ish color, I don't know if cmd supports this.
there are 2 issues here I think:

there is a colorama.init() that detects windows and change the
escape codes, the script is not calling this currently, so the escape codes
are probably wrong.
the deptree output uses box drawing characters
https://en.wikipedia.org/wiki/Box-drawing_character, without knowing
the encoding, these characters might show up as gibberish in the file

try this:

add import colorama; colorama.init() to the top of the script
try setting PYTHONIOENCODING=utf-8 or PYTHONIOENCODING=utf-16 and
check the file results

—
Reply to this email directly, view it on GitHub
https://github.com/rvalieris/conda-tree/issues/16#issuecomment-996667500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP6TT72LOMLVYCJ4HRT5T3TURMQT7ANCNFSM5KEETZCA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

More details can be found at tartley/colorama#319
cp1252.py is not showing the exact characters, but once I added import logging to that file, the error is thrown in mbcs.py instead:
UnicodeEncodeError: 'mbcs' codec can't encode character '\u251c' in position 14: invalid character


Leaflet with rotate functionality

Hi,
I would like to ask what is the status of this Leaflet featured version?Is this still being supported.I see it hasn't been merged in a while?Is this project abandoned?
Thanks,
Ionut

Hello,
Well, I am not working on this project anymore (I don't need that feature anymore), so it is pretty much abandonned, yes. If you merge this branch with leaflet master, it could work or it could not, I have no idea.
The real solution would be for rotation to be integrated into leaflet's core so that is part of the test suite and maintained as part of the whole project. But at the moment, it is not a priority for leaflet's maintainers and I don't think it will ever be (you need to make quite a lot of changes for rotate to work properly, which could have side effects).
If you really need a map with rotation, I suggest you look at OpenLayers instead, which supports it natively
Fabien


Error "Cannot invoke method get() on null object". Is there a problem?

Hello. run-android and error. Is there a problem?
* What went wrong:
A problem occurred configuring project ':app'.
> Could not resolve all dependencies for configuration ':app:_debugApk'.
   > A problem occurred configuring project ':react-native-onesignal'.
      > Failed to notify dependency resolution listener.
         > Cannot invoke method get() on null object
         > Cannot invoke method get() on null object

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

@shanematte Can you provide some more information..?
When does this happen? What version of the RN SDK are you using, can you post your package.json? Also, are you using expo or something?

Closing due to no response - did you manage to fix this issue? If not please reply and I'll be happy to reopen and help!


Added Booking.com Scraping

The method can retrieve all the hotel Names of a destination place at a start date and an end date.

@nikhil25803 The requested changes has been commited . Please verify

@nikhil25803 please verify as i have optimized the code and please merge it possible.


mxnet: use openblas, fix build

Motivation for this change
noticed it was failing when reviewing https://github.com/NixOS/nixpkgs/pull/87102
previous failure:
builder for '/nix/store/1zcd9mmm3dyyxcr588cszdclrsnjbga1-mxnet-1.6.0.drv' failed with exit code 1; last 10 log lines:
  -- Could not find OpenBLAS lib. Turning OpenBLAS_FOUND off
  CMake Error at cmake/Modules/FindOpenBLAS.cmake:82 (MESSAGE):
    Could not find OpenBLAS
  Call Stack (most recent call first):
    cmake/ChooseBlas.cmake:42 (find_package)
    CMakeLists.txt:310 (include)

  -- Configuring incomplete, errors occurred!
  See also "/build/apache-mxnet-src-1.6.0-incubating/build/CMakeFiles/CMakeOutput.log".

Things done

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

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

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

I'm not a big fan of having to do both blas blas.provider, is there a way that we can symLinkJoin them so that people only need to specify blas

AFAIK, the provider pass thru is pretty special to blas libraries

Result of nixpkgs-review pr 87118 1

1 package marked as broken and skipped:
- python27Packages.myfitnesspal

5 packages built:
- mxnet
- python27Packages.mxnet
- python37Packages.mxnet
- python37Packages.optuna
- python38Packages.mxnet

I would approve your push, but I can't approve my own PR. Lol

also, wouldn't it be better to get the liblapack.so from the blas.provider?
[12:24:46] jon@jon-desktop /home/jon/projects/nixpkgs (fix-mxnet)
$ nix-build -A blas.provider
/nix/store/hwdbqpqknffqnc30fjxzvrpmp2d3lcg2-openblas-0.3.9
[12:24:58] jon@jon-desktop /home/jon/projects/nixpkgs (fix-mxnet)
$ ls ./result/lib/lib*
./result/lib/libblas.so.3     ./result/lib/liblapack.so.3          ./result/lib/libopenblas.so.0
./result/lib/libcblas.so.3    ./result/lib/libopenblasp-r0.3.9.so
./result/lib/liblapacke.so.3  ./result/lib/libopenblas.so

the main thing I see, is that the blas.provider only provides the libraries with the .3 suffix, maybe we should add symlinks to their unversioned variants

Agreed, the current setup is a little clunky, though I'm not sure symlinkJoin would cover it: IIRC we need the actual package to be in buildInputs, so that setupHooks for doing things like adding to PKG_CONFIG_PATH and the CMake search path work.
@matthewbauer anything you think we should change here?

actually, this is specific to openblas
    ln -s $out/lib/libopenblas${stdenv.hostPlatform.extensions.sharedLibrary} $out/lib/libblas${stdenv.hostPlatform.extensions.sharedLibrary}${stdenv.lib.optionalString stdenv.hostPlatform.isLinux ".3"}
    ln -s $out/lib/libopenblas${stdenv.hostPlatform.extensions.sharedLibrary} $out/lib/libcblas${stdenv.hostPlatform.extensions.sharedLibrary}${stdenv.lib.optionalString stdenv.hostPlatform.isLinux ".3"}
    ln -s $out/lib/libopenblas${stdenv.hostPlatform.extensions.sharedLibrary} $out/lib/liblapack${stdenv.hostPlatform.extensions.sharedLibrary}${stdenv.lib.optionalString stdenv.hostPlatform.isLinux ".3"}
    ln -s $out/lib/libopenblas${stdenv.hostPlatform.extensions.sharedLibrary} $out/lib/liblapacke${stdenv.hostPlatform.extensions.sharedLibrary}${stdenv.lib.optionalString stdenv.hostPlatform.isLinux ".3"}

by modifying the openblas package (in the PR that's now linked to this thread), i was able to build this while only supplying blas.provider

currently blocked by  #87134

CC @matthewbauer I'm pretty sure the above is correct; I had to do something similar for pytorch. Perhaps we should add to the manual?

Yes that sounds right. Something in the manual would be helpful. We may need a "maintainers" section or something since this is more about how you make sure your package is correct vs. just how you use it.

@matthewbauer anything you think we should change here?

Adding the unversioned library seems like a good idea.

should I just target staging and get this merged?

Sounds good, let's just merge this on staging then so it can bake with all the other openblas / numpy changes. and we don't have to remember to circle back to it.

@GrahamcOfBorg build mxnet

There's a CI error.

@GrahamcOfBorg eval

pretty sure staging was broken at the time of PR creation


Can i use gzip compress in bulk-index api?

i try to use gzip compress refer this example code
in my case, i put compressed body into bulkIndexerItem (in example, line 209)
but response return status '0'
this is my compress function
func compressQueryByGZip(queryByteBuff *bytes.Buffer) (*bytes.Buffer, error) {
	buf := new(bytes.Buffer)
	w := gzip.NewWriter(buf)

	if _, err := w.Write(queryByteBuff.Bytes()); err != nil {
		return nil, err
	}

	if err := w.Close(); err != nil {
		return nil, err
	}

	return buf, nil
}

thanks

Compressing the individual items in the bulk indexing request won't work. Instead, the bulk items should be supplied uncompressed, and the final constructed HTTP request (containing all the bulk items) needs to be compressed as a single unit. The request compression also needs to be indicated to the server using the appropriate HTTP header.
PR https://github.com/elastic/go-elasticsearch/pull/302 (merged) adds a CompressRequestBody option to the client, that you can now use for this purpose. If you set that, it should work?


Dropping stream hourly on 0.1.7

Firmware Version | 0.1.7
Base Version | <IP_ADDRESS>_202011201756
Model Suffix | y21ga
I have the stream from motioneye to a localhost website and have seen 'Connection to camera lost' around the same time every hour (00:13) for about 2m at a time.  The webgui is inaccessible during this time. Most features off including cloud.
This is a new issue to me at least.


Default implementations for Cw721Execute and Cw721Query

Currently traits are implemented in cw721-base here:
https://github.com/CosmWasm/cw-nfts/blob/177a993dfb5a1a3164be1baf274f43b1ca53da53/contracts/cw721-base/src/execute.rs#L132
https://github.com/CosmWasm/cw-nfts/blob/177a993dfb5a1a3164be1baf274f43b1ca53da53/contracts/cw721-base/src/query.rs#L22
Any objection against moving this as default implementations traits in cw721 package? Here: https://github.com/CosmWasm/cw-nfts/blob/177a993dfb5a1a3164be1baf274f43b1ca53da53/packages/cw721/src/traits.rs#L18

@shanev, @larry0x, let me know what you think about this.

Currently traits are implemented in cw721-base here:
https://github.com/CosmWasm/cw-nfts/blob/177a993dfb5a1a3164be1baf274f43b1ca53da53/contracts/cw721-base/src/execute.rs#L132
https://github.com/CosmWasm/cw-nfts/blob/177a993dfb5a1a3164be1baf274f43b1ca53da53/contracts/cw721-base/src/query.rs#L22
Any objection against moving this as default implementations traits in cw721 package? Here:
https://github.com/CosmWasm/cw-nfts/blob/177a993dfb5a1a3164be1baf274f43b1ca53da53/packages/cw721/src/traits.rs#L18

Done in #161


Additional regex params needed for message validation

there were node ids in the kg and qg pydantic definitions that were of type GUID . these was causing a validation errors.
the solution present here appends the regex for GUIDs onto the existing regex for CURIES.

So there are a few things going on here.

Let's not comment-out old code. Remembering what we had before is what git is for.
The aim of this tool is to serve as a reflection of the Reasoner API. If ROBOKOP is not compliant, that's a problem with ROBOKOP. For unavoidable inconsistencies, one could monkey-patch reasoner-pydantic.
TRAPI actually does not constrain CURIEs this way. It only enforces that they be strings. The RFC definition of a CURIE is very complex and surprisingly permissive, so I think that's the right decision. We can update reasoner-pydantic accordingly.

So @patrickkwang  the upshot here is that the ids out of robokop are fine, and we're just overvalidating?


Cassandra 2.2.x - aggregates support, min, max etc

Since Cassandra 2.2.x is already stable, do Phantom supports min, max etc functions ? :)
http://christopher-batey.blogspot.com/2015/05/cassandra-aggregates-min-max-avg-group.html

Hi @mmatloka ,
We are in the process of releasing extensive support for all the fun features upcoming in Cassandra, including aggregates combined with extensive support for Spark. They will however only be available in the licensed version of phantom which we plan to release in Q1 2016.
It also bundles things like:

automated schema resolutions
automated SQL DB to Cassandra transitions

It will all be available for a very low license fee and we plan to use the income generated to fund development of our other open source work.
Regards.

Hi @mmatloka,
I'm going to close this as it's not really a relevant issue for the OSS project. We will keep everyone up-to-date on the developments of the enterprise version.
Regards.


8tracks download bug

Version : youtube-dl is up-to-date (2015.12.18)
Command : youtube-dl -f bestaudio --verbose http://8tracks.com/maurya-bendukuri/bollywood-mix
Output :
[debug] User config: []
[debug] Command-line args: [u'-f', u'bestaudio', u'--verbose', u'http://8tracks.com/maurya-bendukuri/bollywood-mix']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.12.18
[debug] Python version 2.7.10 - Linux-3.16.0-56-generic-x86_64-with-debian-jessie-sid
[debug] exe versions: avconv 9.18-6, avprobe 9.18-6
[debug] Proxy map: {}
[8tracks] bollywood-mix: Downloading webpage
[8tracks] bollywood-mix: Downloading song information 1/10
[8tracks] bollywood-mix: Downloading song information 2/10
[8tracks] bollywood-mix: Downloading song information 3/10
[8tracks] bollywood-mix: Downloading song information 4/10
[8tracks] bollywood-mix: Downloading song information 5/10
[8tracks] bollywood-mix: Waiting for 300 seconds
[8tracks] bollywood-mix: Downloading song information 5/10
[8tracks] bollywood-mix: Downloading song information 6/10
[8tracks] bollywood-mix: Waiting for 300 seconds
[8tracks] bollywood-mix: Downloading song information 6/10
[8tracks] bollywood-mix: Downloading song information 7/10
[8tracks] bollywood-mix: Waiting for 300 seconds
[8tracks] bollywood-mix: Downloading song information 7/10
[8tracks] bollywood-mix: Downloading song information 8/10
[8tracks] bollywood-mix: Waiting for 300 seconds
[8tracks] bollywood-mix: Downloading song information 8/10
ERROR: An extractor error has occured. (caused by KeyError(u'id',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 290, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/eighttracks.py", line 146, in _real_extract
    'id': compat_str(track_data['id']),
KeyError: u'id'
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 663, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 296, in extract
    raise ExtractorError('An extractor error has occured.', cause=e)
ExtractorError: An extractor error has occured. (caused by KeyError(u'id',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.```

Duplicate of #5029.

@dstftw  Is there a fix already?

No idea.


duplicate dir name in examples/cifar10_msra/data.py

I followed the neon course tutorial 02 and came across this error in examples/cifar10_msra/data.py. Apparently, the image path was duplicated with the out_dir in it.
Traceback (most recent call last):
  File "examples/cifar10_msra/data.py", line 132, in <module>
    generated_files = ingest_cifar10(args.out_dir, args.padded_size)
  File "examples/cifar10_msra/data.py", line 67, in ingest_cifar10
    im.save(os.path.join(out_dir, img_path), format='PNG')
  File "/Users/chengchingwen/Desktop/gitrep/neon/.venv3/lib/python3.6/site-packages/PIL/Image.py", line 1682, in save
    fp = builtins.open(filename, "wb")
FileNotFoundError: [Errno 2] No such file or directory: 'data/cifar10/data/cifar10/train/6/0.png'
(.venv3) zhengjiwendeMBP:neon chengchingwen$ python examples/cifar10_msra/data.py --out_dir data/cifar10/
0it [00:00, ?it/s]
Traceback (most recent call last):
  File "examples/cifar10_msra/data.py", line 132, in <module>
    generated_files = ingest_cifar10(args.out_dir, args.padded_size)
  File "examples/cifar10_msra/data.py", line 67, in ingest_cifar10
    im.save(os.path.join(out_dir, img_path), format='PNG')
  File "/Users/chengchingwen/Desktop/gitrep/neon/.venv3/lib/python3.6/site-packages/PIL/Image.py", line 1682, in save
    fp = builtins.open(filename, "wb")
FileNotFoundError: [Errno 2] No such file or directory: 'data/cifar10/data/cifar10/train/6/0.png'

Errors in cifar10_msra like above disappeared with neon 2.2. Please verify. Thanks, @chengchingwen !

Please re-open if errors occur in latest neon. https://github.com/NervanaSystems/neon/tree/latest


panic for non-mut variables when --no-lifetime is specified

This is pretty low-priority, since it only applies when --no-lifetime is specified, but Verus panics if you try to mutate a non-mut variable:
fn test() {
    let x: u32 = 5;
    takesmut(&mut x);
}

[Verus all-hands]
@tjhance has started on a patch.


Ensure all folders are accessible by appuser in the Docker container

What's changing
I run into an error trying to run the container, /home/appuser/.cache is not accessible for appuser
How to test it
Build the container and try to run it locally
I already...

[X] Tested the changes in a working environment to ensure they work as expected
[N/A ] Added some tests for any new functionality
[N/A ] Updated the documentation (both comments in code and under /docs)

Ok I made some changes so that models are correctly downloaded after configuring the user


Need to Organize Repo Structure

we should organize our additional pipeline stages into a project: src/stages
all other additions to spark can be their own projects either in core, or in src

One potential structure could have
-src
- stages
- image
- text
- udfs
- plot
- fuzzing
- codegen
- io

@mhamilton723 can this issue be closed?


I got this error after running npm run build

Description
at Object.writeBundle (/node_modules/vite-plugin-ssr/dist/esm/node/plugin/plugins/buildConfig.js:54:17)
at /node_modules/rollup/dist/es/shared/node-entry.js:19597:40
at async Promise.all (index 2)
at PluginDriver.hookParallel (/node_modules/rollup/dist/es/shared/node-entry.js:19525:9)
at /node_modules/rollup/dist/es/shared/node-entry.js:20521:13
at catchUnfinishedHookActions (/node_modules/rollup/dist/es/shared/node-entry.js:19942:16)
at build (/node_modules/vite/dist/node/chunks/dep-cNe07EU9.js:67481:22)
at CAC. (/node_modules/vite/dist/node/cli.js:842:9)

I need the full error message & stack.

x Build failed in 18.58s
error during build:
Error<EMAIL_ADDRESS>You stumbled upon a bug in vite-plugin-ssr's source code. Go to https://github.com/brillout/vite-plugin-ssr/issues/new and copy-paste this error; a maintainer will fix the bug (usually under 24 hours).
at Object.writeBundle (/node_modules/vite-plugin-ssr/dist/esm/node/plugin/plugins/buildConfig.js:54:17)
at /node_modules/rollup/dist/es/shared/node-entry.js:19597:40
at async Promise.all (index 2)
at PluginDriver.hookParallel (/node_modules/rollup/dist/es/shared/node-entry.js:19525:9)
at /node_modules/rollup/dist/es/shared/node-entry.js:20521:13
at catchUnfinishedHookActions (/node_modules/rollup/dist/es/shared/node-entry.js:19942:16)
at build (/node_modules/vite/dist/node/chunks/dep-cNe07EU9.js:67481:22)
at CAC. (/node_modules/vite/dist/node/cli.js:842:9)
Stack is Laravel Inertia with React and capacitor

Updated to the latest version. I'll re-open if you still get an error then.


EN Documentation

Could someone translate the documentation in english?

After succeeding in  configuring.

Hello, This is the English document

@roccomuso

@BruceYuj thanks!
Is it worth putting it as default language?

you mean putting the English document as the default?

@roccomuso

@BruceYuj yep.

@roccomuso ， That's fine. I make an issue to ask the author.

@roccomuso,  I finish it. You can see the effect.

great!


DRAFT: adds flow options to basic Router, as well as custom option/object passthrough

Description
This adds support for passthrough options (e.g. port in a Bun service), as well as the following specific options:

after: what you would pass to a .then() chain on the router.handle/router.fetch.
errors: what you would pass to a catch block.  if not defined, Router will still throw (uncaught).

Adds appx. 45 bytes.
Considerations
The bulk of the bytes added is to support the explicit try/catch block (and rethrow).  If we did not support errors in this, it would be closer to 460 bytes total (instead of 490).
Example Usage
import { Router, RouterType, error, json, withParams } from '../src/index'

const router = Router({
  port: 3000,
  after: json,
  error: error, // long form for clarity
})

router
  .get('/json', () => ({ foo: 'bar' }))
  .get('/throw', (a) => a.b.c) // this is caught!
  .all('*', () => error(404))

export default router

Type of Change (select one and follow subtasks)

[ ] Documentation (README.md)
[ ] Maintenance or repo-level work (e.g. linting, build, tests, refactoring, etc.)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

[ ] I have included test coverage

[ ] Breaking change (fix or feature that would cause existing functionality/userland code to not work as expected)

[ ] Explain why a breaking change is necessary:

[ ] This change requires (or is) a documentation update

[ ] I have added necessary local documentation (if appropriate)
[ ] I have added necessary itty.dev documentation (if appropriate)

I think this one will be killed in favor of AutoRouter.  It makes sense to keep the core as minimalist as possible for use-cases that demand it, while allowing AutoRouter to be the batteries-included version for tiny demos.

Killed in favor of #222


Sc tx commitment

Rough implementation of the ScTransactionCommitment with poseidon hash according to the zend_oo ScCommitmentTree_to_cryptolib draft implementation.

This PR is outdated, since we calculate ScTxCommitment using sc-cryptolib.


PolyPhen-2  Adapter

This adapter imports and processes data from PolyPhen-2 (Polymorphism Phenotyping v2), a tool that predicts the possible impact of amino acid substitutions on the structure and function of human proteins. The adapter:

Parses compressed PolyPhen-2 data files
Handles all possible nucleotide substitutions (A, C, G, T) for each genomic position
Extracts scores and predictions for both HumDiv and HumVar models
Generates nodes for each unique variant with properties including:

Reference and alternative alleles
PolyPhen-2 scores and predictions for HumDiv and HumVar

@mjsduncan did you check the code before merging it?
@kedistS It looks like the PolyPhen data is in BED format. BED files have a 0-based start and 1-based end. This means you have to add 1 to the start coordinate as we use 1-based coordinates in our knowledge base. See here. Cc @dawit-melka


Box<ToSql + ?Sized> instead of Box. And also remove Box

In https://github.com/jgallagher/rusqlite/pull/660 I added support for Box<T> where T: ToSql.
But as I see it does not work with unsized types. Instead of impl for Box<dyn ToSql> above is used.
I tried to put my Box<str> in ToSql::to_sql(**here**) but it tells me that str is unsized.
I think there must be impl ToSql + ?Sized for Box<T> instead of T: ToSql. And I also removed impl for Box<dyn ToSql> because it conflicts (and didn't work before with my example) with my new impl.

Hmm... So this actually fails some tests if I try it locally (try cargo test --features "array bundled csvtab series vtab")
For whatever reason travis tests are busted (Filing a bug). Can you make https://github.com/jgallagher/rusqlite/blob/master/.travis.yml#L44-L48 pass?

Oh, your patch didn't make these broken...


Package show model query

Sorry for the commit noise but as I was adding the mock for the package query I noticed that a lot of other mocks were in the App target instead of the test target and I moved them over as part of this change.
The relevant bit is really just the query method where I commented and the new test test_query_basic.

Is this ready to go?

Yes, it is!


xcp: 0.16.0 -> 0.18.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/tarka/xcp/releases.
meta.description for xcp is: An extended cp(1)
meta.homepage for xcp is: https://github.com/tarka/xcp
Updates performed

Rust 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 0.18.0 with grep in /nix/store/mvpg84s71h6bngkv1x19fpvrslcgds24-xcp-0.18.0
found 0.18.0 in filename of file in /nix/store/mvpg84s71h6bngkv1x19fpvrslcgds24-xcp-0.18.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
xcp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mvpg84s71h6bngkv1x19fpvrslcgds24-xcp-0.18.0 \
  --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 xcp https://github.com/r-ryantm/nixpkgs/archive/b8c666bc5d22bfcab50ea1492743c96191e0624e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b8c666bc5d22bfcab50ea1492743c96191e0624e#xcp

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

xcp

Maintainer pings
cc @legendofmiracles for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/tarka/xcp/releases.
meta.description for xcp is: An extended cp(1)
meta.homepage for xcp is: https://github.com/tarka/xcp
Updates performed

Rust 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 0.18.1 with grep in /nix/store/kq7ih62az51q674kxhbaiw5m9syic28l-xcp-0.18.1
found 0.18.1 in filename of file in /nix/store/kq7ih62az51q674kxhbaiw5m9syic28l-xcp-0.18.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
xcp

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/ae853f28babe2985aa2452bdc341d629d2795e57#xcp

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

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

1 package built:

xcp

Maintainer pings
cc @legendofmiracles for testing.


error: Inline macro println not found.

Are the code snippets only meant to be run locally or they are supposed to run online as well? Whenever I try to run them using https://book.cairo-lang.org, I get the error above.

yeah we had to update the code examples to support the latest version (with prints), but we still need to update the code runner to the last version. cc @cryptonerdcn

Will update it soon ser!

Are there any plans to have a server that would handle running the code? From what I can see it's currently a separate WASM that needs to be prepared and redeployed. In the Rust Book I saw they have some server that's evaluating the code under https://play.rust-lang.org/evaluate.json. I can imagine something like this should be quite easy to do using scarb binaries and a bit more flexible when it comes to updating.

In the future I'm thinking about having a dedicated playground with quizzes to do along reading the book.
I already know that https://cairovm.codes/ is doing this, and has some extra cool features associated. For the book I think having a WASMed version of Cairo deployed along is the best path to follow as everything is done client-side

Sorry for the late, got sick last weekend, but will release a new version of wasm-cairo soon.

@cryptonerdcn any update?

@enitrat Sorry for late. I got an allergic reaction to pollen allergy(a very common disease in Japan's spring) these weeks but am recovering now.
Still WIP.

would like to join this team

https://github.com/cairo-book/cairo-book/pull/728
Finished.


Mint Theme

I'm submitting a Bug report ;
When changing the theme to Mint, some components still displays data with colors that doesn't work with a light color theme
exp : Revenue, TreeView, Bubble Map and Line Map.

Hey @m0uj, have you followed the documentation and changed the code as per with step 3 https://akveo.github.io/ng2-admin/articles/011-changing-color-scheme/?

Yes, i didn't want to customize it, i want to use the Mint theme with its default color scheme.
But it has problems with the display of some components, you can test it.
The components mentioned above may need some fixed (they don't need to inherit some colors from the theme used)

Having the same issue here. Revenue and Tree components are broken in Mint theme.

Same observation, Slim loading bar, Hot tables, Pie-chart and Tree are broken. Any way to fix these?

same


Java indexer: mitigate Plugins caching KytheGraphs [phab:D2069]

Created by schroederc at 2017-11-29 22:33:05:
Refactor KytheJavacAnalyzer#registerPlugin(Plugin) into
KytheJavacAnalyzer#registerPlugin(Supplier) and construct fresh
Plugin instances for each compilation.

zarko accepted this revision at 2017-11-29 22:34:32

schroederc closed via commit 9f20a5f04778e7392c78256201b33753b08eabc7 at 2017-11-29 22:36:06


[BUG] added space in error messages

Describe the bug
when a ptsh command results in an error, the error message is prepended with a space
To Reproduce
run a ptsh command that you know will result in error, optionally compare the terminal output to a ptsh command that results in success
Expected behavior
the error message does not have a space prepended to it, making it consistent with the success output.
Screenshots/terminal output

Desktop

OS/Linux distro: linux mint

Closing, because all scripts has been replaced with programs in latest release


Is this package still being maintained?

Hi.
As the title said, is this package still being actively maintained? It seems to lack behind in compatibility with Omise API documentation.

I think they already move to https://github.com/opn-ooo/omise-go
but in the repo, I see import github.com/omise/omise-go/internal.
that means we can't use it 😂

This package is still being maintained. Currently reviewing old issues and pull requests with an aim to support more active maintenance in the upcoming months.


Is there an easy way to redirect an old url?

For example I have some old links to blog posts that look like this myblog.com/posts/some-blog-post but now it is located at myblog.com/some-blog-post.
Is there a way so that I can easily fix up all the old urls?

You can use aliases: https://www.getgutenberg.io/documentation/content/page/

Is it working for you?

Would that require me to change all the headers of my posts?

If you are not using a server, yes.


[WIP]Wildcard imports

Hi @trotterdylan! I read the conversation in #192 and I thought I'd take a shot at wildcard imports. The code should be functional, I've updated the tests too.
If the idea is the right one, the next step for me would be to start trimming that code down. I can isolate some code in reusable methods and remove some of the duplication.
Before I take this PR further, can you tell me if I'm heading in the right direction?

It struck me that I could write the code in Go, as I got more familiar with the codebase. It was a bit too late and I was closer to finishing the Python code. And I was afraid Go would be the better solution. :) I'll start working on that on Monday, excited about how this will turn out.
BTW, the Travis jobs for my commits seem to take a lot of time. Job #679 seems to have been running for over 50 mins now. Is that normal?

Sounds good.
The OS X job on Travis is slow. Usually it suffices to validate that the
Linux job was successful. It's usually pretty quick.
On Thu, Feb 16, 2017 at 11:23 AM Claudiu Coman<EMAIL_ADDRESS>wrote:

It struck me that I could write the code in Go, as I got more familiar
with the codebase. It was a bit too late and I was closer to finishing the
Python code. And I was afraid Go would be the better solution. :) I'll
start working on that on Monday, excited about how this will turn out.
BTW, the Travis jobs for my commits seem to take a lot of time. Job #679
seems to have been running for over 50 mins now. Is that normal?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/grumpy/pull/254#issuecomment-280431548, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHLe9R6fksWkmB1hIUCH4CLAYQemholks5rdKIhgaJpZM4MDVBG
.


Receiving Could not find class for puppet forge modules

Some background: I'm trying to configure ocd for the first time to test an upgrade of puppet from 4->5 and a refactor of some puppet code therein.  One of the first changes I'm making is upgrading modules in the Puppetfile to new versions.
Using Ubuntu on windows (windows subsystem for linux), I installed ocd and all it's dependencies onto my machine.  I created a .octocatalog-diff.cfg.rb and made sure I had a hiera.yaml file in place.  When I run octocatalog-diff -n node.domain.com, i'm getting an error:
Error: Evaluation Error: Error while evaluating a Method call, Could not find class ::resolv_conf for node.domain.com
Now, resolv_conf is a puppet forge module that's included via a hiera file with classes: -resolv_conf.  Does octocatalog-diff deal with modules from the forge?  I've tried installing the module via puppet module install saz-resolv_conf --version 3.3.0 but that didn't seem to help.
Ubuntu 16.04 on windows subsystem for Linux.
ruby 2.3.1p112
puppet-agent 5.5.0
octocatalog-diff 1.5.3

Your version of octocatalog-diff: <!-- type octocatalog-diff --version -->

I suggest trying with --no-parallel since sometimes output can get eaten as it tries to launch the two parallel scripts. This will make it run 2x slower (since it will compile the old and new catalogs one after the other, instead of at the same time) but this can be handy for debugging.
I also suggest that you have .octocatalog-bootstrap.sh start with -x for further debugging, e.g.:
#!/bin/bash -x


feat: add isInspectable and isAnimated properties to OpenWebViewOptions

Hey @riderx i added the possibility to inspect the inner website and the possibility to bypass webview opening animation

thanks for the work !


Please considering make the project description objective

I am looking at the http://shardingjdbc.io/index.html . There is a sentence makes me attention:

migrate legacy code almost zero costs

I prefer to re-consider about this. I understand zero costs based on JDBC interfaces compatible, and sharding based on SQL statement analysis. But I have to say there are costs from the design level, which definitely far from zero. And if a traditional system using join or some other complex sql features, it is hard to switch into Sharding MySQL (not just sharding-jdbc) easily.
In my perspective, I prefer compatible with JDBC and ORM frameworks

Yes, I accept it is not always zero costs if complicated SQL in legacy code, "zero costs" is the final objective.
I will change the doc to "compatible with JDBC and ORM frameworks" for now. Thank you for suggestion.

homepage has updated. Closing this issue.


update aliwangwang to version 3.4.1

update aliwangwang to version 3.4.1

Closed in favor of https://github.com/caskroom/homebrew-cask/pull/13468.


Deploy the ERC1820 on a private network

Hi!
I'm trying to deploy all the contracts from your repo on a private network (IBFT2), where the gas price is set to 0, but I'm unable to deploy the ERC1820
Since there is no gas fee in my network, I changed the rawTx to fit my needs, but when I deploy the contract with truffle, the contract address returned is Null
Do you have any idea how I can solve this issue ?
Thanks

Ok

Hi @soragXYZ ,
The ERC1820 deployment transaction shall not be modified otherwise, its address won't be "0x1820a4B7618BdE71Dce8cdc73aAB6C95905faD24" anymore.
As "0x1820a4B7618BdE71Dce8cdc73aAB6C95905faD24" is hardcoded, it shall not be different.
The only way we've managed to deploy ERC1820 on private networks with gas price set to 0, is to include erc1820 directly in the genesis block.


FR: add option to set URL for localAI server instead of openAI

With the availability of localAI (which offers an openAI compatible REST API) it would be great if this plugin offered the possibility to set an individual URL for the API to call.
https://github.com/go-skynet/LocalAI
Using localAI would mitigate concerns about working with any kind of sensitive or business related information for me and hopefully others as well.

Just merged this in #110 let me know if this works for you!


MLVPN with single link

Hi @zehome,
I've setup mlvpn on both server and client.
The client has two internet connections available, however, I'm trying it first with a single link.
The internet link I'm using is a 20Mbps one. Doing SpeedTest gives around 17Mbps (+/- 0.2Mbps).
Theorically, using this link with mlvpn should result in around 13.5 Mbps.
When I launch mlvpn and use it as the default internet route, I get in best cases 11Mbps.
Is that a bad configuration issue?
Thank you.

Can you check if mlvpn is consuming 100% CPU during your tests?
I’m asking because I find that especially on lower-end devices (e.g. a Ubiquiti EdgeRouter X SFP), it’s relatively easy to max out on CPU before maxing out on bandwidth when using mlvpn.


Custom error messages

I would like to be able to customize the error message. For example to be able to linking to a documentation in your project where it is explained how to write proper commits, or to give more details about what the whitelist/blacklist represent.

Are you wanting to wanting to bake that into the error messages displayed/printed in the build? You might want to take a look at the severity branch that I'm working on where I extract out the ability to leave issue (previously known as error) hints. The severity branch, BTW, is related to the severity issue.

This issue has been resolved by this pull request.

@bkuhlmann I think you misunderstood me, I didn't mean the error warning thing, but to allow to have custom error messages. For example, to add a link to documentation and to clarify the errors with regular expressions. In my case I have this error message:
Commit Subject Prefix Error. Use: /\A(?!\s)(\[api\])?(\[backend\])?(\[ci\])?(\[dist\])?(\[doc\])?(\[webui\])? [a-zA-Z]/.
and I would prefer having something like:
Commit Subject Prefix Error. Tag your commit properly. See link_to_documentation.
This could be configured with something like this (a message param):
:commit_subject_prefix:
  :enabled: true
  :severity: :error
  :message: "Tag your commit properly. See link_to_documentation"
  :whitelist:
    - \A(?!\s)(\[api\])?(\[backend\])?(\[ci\])?(\[dist\])?(\[doc\])?(\[webui\])? [a-zA-Z]

The :message (or :hint as it's known internally) would replace the default text then? It would also probably need a string placeholder (i.e. %s) so regular expression output could be displayed in the feedback as it does currently (or not, in the case where you wouldn't want the regular expressions in the output at all).
What you're suggesting feels a lot like i18n support where, in this case, could be used as alternate/custom output.
The link to documentation, while I understand would be handy, worries me a bit since it could get out of date rather quickly. Would need to think about how that could be generated or updated in an automated fashion so this gem isn't hard to maintain.

@bkuhlmann

The link to documentation, while I understand would be handy, worries me a bit since it could get out of date rather quickly. Would need to think about how that could be generated or updated in an automated fashion so this gem isn't hard to maintain.

well, my idea is just to allow the user to have his own error messages, so he could write whatever he want, documentation, explanation of what regular expression mean or any other thing that he think about. This was just an example of why this would be really useful.

The :message (or :hint as it's known internally) would replace the default text then? It would also probably need a string placeholder (i.e. %s) so regular expression output could be displayed in the feedback as it does currently (or not, in the case where you wouldn't want the regular expressions in the output at all).

I would leave the name of the cop that is failing and replacing the rest of the current error message with the one provided by the user, if any.

What you're suggesting feels a lot like i18n support where, in this case, could be used as alternate/custom output.

Not sure if I get what you exactly mean. What I suggest is just replace the current error message with the one provided by the user, without doing anything else. Just whatever the user introduced is what is rendered.

Yeah, that makes sense. I think that level of customization becomes more interesting if this gem needed i18n support (i.e. Internationalization). With i18n, one could provide translations in English, Spanish, German, etc. ...or even different versions of a specific language. With support like that, you could then customize any of the text output however you liked.
That said, not sure I want to support this yet. It's good that this gem is customizable as is but would like to wait and see if others have this issue before proceeding.

@bkuhlmann

Yeah, that makes sense. I think that level of customization becomes more interesting if this gem needed i18n support (i.e. Internationalization). With i18n, one could provide translations in English, Spanish, German, etc. ...or even different versions of a specific language. With support like that, you could then customize any of the text output however you liked.

:+1:

That said, not sure I want to support this yet. It's good that this gem is customizable as is but would like to wait and see if others have this issue before proceeding.

Maybe leaving the issue open then?
What is the problem of supporting it? It is not something complicate to implement/support, right?

Let's leave the issue closed for now to maintain Inbox Zero and keep the cognitive load low as possible. We can always search for this later if need be.
Adding i18n support would complicate the implementation and add more maintenance complexity. It'd be some effort to support so let's wait and see if the pain point grows in intensity before proceeding.


clustermesh: adapt global service annotations

Follow-up of cilium/cilium#23298, adding the new clustermesh service.cilium.io/global annotation alongside the previous one.
Signed-off-by: Marco Iorio<EMAIL_ADDRESS>

https://github.com/cilium/cilium/pull/23298 got merged. Marking this ready for review.


Add Localization Style Guides to TOC

Docs Build status updates of commit f2c8fa2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

globalization/localization/ministyleguides/mini-style-guide-polish.md
:white_check_mark:Succeeded
View

globalization/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Revert "WIP, BUG: Make plot_evoked() not plot GFP for single channel (#8774)"

cc @cbrnr

cc @cbrnr


One time commands [question]

Not an issue, just a question. Is there currently a way to add a one-time command? Not temporary permissions, but where a person gets access to a command to use it just once.

Not really possible, permissions aren't bound to commands only, but plugins can check for a certain permission anytime, when moving around, doing certain actions, running other commands, even when showing up command suggestions while typing them in chat!
While this at first seems like (and would be) a great idea at first, it isn't really possible due to the nature of "permissions are not always for commands, they can be checked at all times".

Thanks for the response. Should I understand, then, that the only way to implement this on a server would be to integrate this functionality into the plugin that manages the command?

Correct, it would be up to the plugin author to implement such feature, as they are the only ones that can fully control when the action is completed successfully.

Alright, thanks.


recovering SMPL coefficients

I'd like to extract SMPL coefficients like humannerf did (https://github.com/chungyiweng/humannerf?tab=readme-ov-file#metadatajson)
after WHAM, I could get results (['poses_body', 'poses_root_cam', 'betas', 'verts_cam', 'poses_root_world', 'trans_world', 'frame_id']),
and needed coefficients for humannerf are "poses", "betas", "cam_intrinsics", "cam_extrinsics"
I could easily find that "betas" are exactly same as results[0]["betas"],
and it seems "cam_extrinsics" is results[0]["poses_root_world"] + results[0]["trans_world"].
But in humannerf or VIBE, "poses" were (72,) array, while results[0]['poses_body'] were (23,3,3).
also I'm not sure about recovering cam_intrinsics. Maybe results[0]["poses_root_cam"] means cam_intrinsics?
Could you please let me know how to recover these coefficients?
It would be great if you also introduce me (poses_root_cam, verts_cam).

I think , for poses, you can use 'matrix_to_axis_angle' function in tranforms.py . (3x3 -> 3, which makes 24x3x3->24x3)
And..I'm really not sure about, but maybe you can use data from DPVO output...
Maybe you'd better check # ========= Store results ========= # part code on demo.py and your data!


Allow incomplete fact selection

Description
This change to the cartesian function allows a user to create a potential link without supplying values for every fact template.
If a fact template does not have a selected value, it is not replaced in the command. This is useful if the user has edited the command to remove a fact template. The change also allows for appropriate substitution of global facts (i.e. #{server}) when mixed with plugin facts.
fixes mitre/caldera#3030
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Unit test added.
Integration tested by adding to a local instance of Caldera and replicating some representative use cases.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works

Tested.


TLC

Adds python 3.12 classifier
CI now runs all python environments as declared in the python classifiers
Tox now runs all python environments as declared in the python classifiers

Python 3.6 passes all tests on my local computer but not CI. I've dropped it completely as it is no longer supported by Python.

Tests fail for Python 3.12. I'll take a deeper dive into why it fails at a later date. For now, adding tests for the additional Python environments should suffice.

I think a version bump and release would be nice after this, and any other open PRs, are merged.

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

Project coverage is 99.23%. Comparing base (7f24f78) to head (7ebe16d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage   99.23%   99.23%
=======================================
  Files          12       12
  Lines         784      784
=======================================
  Hits          778      778
  Misses          6        6

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


pull latest codes

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: jmtao:white_check_mark: zhxt:x: Shengzhao TaoShengzhao Tao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


artistInfoFileUpdated

Description
Issue related to artist info
Now artist info will redirect to correct page
Checklist

[X] I am making a proper pull request, not a spam.
[X] I've checked the issue list before deciding what to submit.

Please reference the issue you were assigned

Please reference the issue you were assigned

issue no: 249
https://github.com/nilisha-jais/Musicophilia/issues/249#issue-920486769

You should have filled the pr template properly!
Please follow this on how to resolve the conflicts

@niloysikdar Yes finally i did it😅
@nilisha-jais Actually i filled pr template correctly but when you had merged else pr lead to conflict on my page. However thanks, the link is very useful 😀

Great @Pratik57

Thanks @niloysikdar  😀


SE-0138: Minor update. Array.withUnsafeBytes is nonmutating.

The "mutating" keyword should not have been on this API. The example
code in SE-0138 uses it as a nonmutating API.
(The mutating keyword was a temporary workaround for a bug that has now been fixed on master.)

@swift-ci test and merge.


firefoxPackages.tor-browser: 8.0.2 -> 8.0.3

Minor security update.

[X] It builds.

Failure on aarch64-linux (full log)
Attempted: firefoxPackages.tor-browser
Partial log (click to expand)
   Compiling gkrust-shared v0.1.0 (/build/tor-browser/toolkit/library/rust/shared)
   Compiling gkrust v0.1.0 (/build/tor-browser/toolkit/library/rust)
    Finished release [optimized] target(s) in 23m 55s
make[3]: Leaving directory '/build/tor-browser/obj-aarch64-unknown-linux-gnu/toolkit/library/rust'
make[2]: Leaving directory '/build/tor-browser/obj-aarch64-unknown-linux-gnu'
make[1]: *** [/build/tor-browser/config/recurse.mk:33: compile] Error 2
make[1]: Leaving directory '/build/tor-browser/obj-aarch64-unknown-linux-gnu'
make: *** [/build/tor-browser/config/rules.mk:434: default] Error 2
builder for '/nix/store/9zdzsippbj540l1ngkbax945v21h5sdy-tor-browser-unwrapped-8.0.3.drv' failed with exit code 2
error: build of '/nix/store/9zdzsippbj540l1ngkbax945v21h5sdy-tor-browser-unwrapped-8.0.3.drv' failed

Timed out, unknown build status on x86_64-linux (full log)
Attempted: firefoxPackages.tor-browser
Partial log (click to expand)
cannot build derivation '/nix/store/vqhk2zgl0hiy6s1k22mnb54xjlzbxf55-hook.drv': 1 dependencies couldn't be built

trying https://github.com/SLNOS/tor-browser/archive/bd512ad9c40069adfc983f4f03dbd9d220cdf2f9.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   159    0   159    0     0    404      0 --:--:-- --:--:-- --:--:--   403
100  367M    0  367M    0     0  7995k      0 --:--:--  0:00:47 --:--:-- 7203k
unpacking source archive /build/bd512ad9c40069adfc983f4f03dbd9d220cdf2f9.tar.gz
cannot build derivation '/nix/store/506bifjy2781qwvp2s7cfvm96g71lh0p-tor-browser-unwrapped-8.0.3.drv': 3 dependencies couldn't be built
error: build of '/nix/store/506bifjy2781qwvp2s7cfvm96g71lh0p-tor-browser-unwrapped-8.0.3.drv' failed

@GrahamcOfBorg build firefoxPackages.tor-browser
/cc @joachifm


[Installing Failure]Compilation failed

after I install Rtools, I use command - install_github("galanisl/NetHypGeom") - trying to install it.
But in then end, I encounter a problem.
'''
ERROR: compilation failed for package 'NetHypGeom'
'''
How could I fix it? Could you please help me with it?

Did you install library devtools? Are you installing this on Windows? Could you please share you sessionInfo()?

Closed due to lack of feedback from @RiptideBo
My tests of Ubuntu 16.04 LTS and macOS Sierra and High Sierra do not present installation problems.


Make GM Intro message to change based on the notification preferences

Summary
For the GM as DM feature we updated the intro message for GMs, stating that you will be notified for all messages.
This PR updates that to show a message based on the notification preferences.
Ticket Link
Fix Mobile part of https://mattermost.atlassian.net/browse/MM-54541
Release Note
None if release on 2.9
NONE

@larkox what if the current server has not been updated to treat GM's as DM's ?

@larkox what if the current server has not been updated to treat GM's as DM's ?

That is handled outside of the changes. In line 148 of direct_channel.tsx we have a check so if the feature is not present, we use the old message.

Building app in separate branch.

Cherry pick is scheduled.


Support for Door close sensor in August Pro lock

The August Smart lock Pro comes with a door open / close sensor. At the moment the status Enum is limited to Locked and Unlocked states, not Locked and Open, etc etc.
I'll have a look at some point around what the API returns and maybe make a pull request if I can figure it out. But thought I'd capture it here in the first instance

I have two August Smart Locks, the normal ones, not pro (http://august.com/products/august-smart-lock/) and they come with "Door Sense" which is that additional sensor to install on the door which determines the open and close. I did run this API in debug mode and indeed found the extra attribute on the lock status. It is called "doorState". This is what I got: {"status":"unlocked","dateTime":"2018-04-12T15:26:49.881Z","isLockStatusChanged":false,"valid":true,"doorState":"closed"}
So, yeah, it's just a matter of returning an additional property on the lock status and then modify the home assistant component to "read that also".

Cross referencing to PR #4

#4 should take care of this


Specify stable to build image on Read the Docs

http://www.rubocop.org seems to be redirecting to Read the Docs.
https://docs.rubocop.org
By default, Read the Docs refers to latest (i.e. master branch).
https://docs.readthedocs.io/en/latest/config-file/v1.html#build-image
Here is the latest docs.
https://docs.rubocop.org/en/latest
However I think RuboCop users expect stable docs.
https://docs.rubocop.org/en/stable
This idea was what @pocke taught me before. Thanks!

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Thank you! It sounds great :+1:
But I'm not familiar Read the Docs, and I'm not sure what build.image option is.
I (and users) expect it redirects to /stable instead of /latest when I access http://docs.rubocop.org . If it will work, I think we should merge this pull request :+1:

Thanks! I will try it first.

It has not affected https://rubocop.readthedocs.io/en/latest yet. I will check it for a while whether the site will be updated.


⚠ Bofrost.lu has degraded performance

In e1fdd2b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 968 ms

Resolved: Bofrost.lu performance has improved in 6cf2d63 after 17 minutes.


Roll Engine from 334fa806add1 to 5f0f69f7c6d3 (27 revisions)

https://github.com/flutter/engine/compare/334fa806add1...5f0f69f7c6d3
2021-07-19<EMAIL_ADDRESS>Warn, but don't fail if prebuilt Dart SDK could not be fetched (flutter/engine#27546)
2021-07-19<EMAIL_ADDRESS>Roll Skia from 95cc53bafd5f to 8f553a769b2b (2 revisions) (flutter/engine#27544)
2021-07-19<EMAIL_ADDRESS>Roll Skia from 52d6bd49747f to 95cc53bafd5f (3 revisions) (flutter/engine#27540)
2021-07-19<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from vfv7CH0zH... to 897eI2xwc... (flutter/engine#27539)
2021-07-19<EMAIL_ADDRESS>Roll Skia from 1f261da41ea4 to 52d6bd49747f (1 revision) (flutter/engine#27537)
2021-07-19<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 1yUmKH13E... to FGuPZEZLt... (flutter/engine#27535)
2021-07-19<EMAIL_ADDRESS>Re-route githook for Windows (flutter/engine#27532)
2021-07-18<EMAIL_ADDRESS>Roll Dart SDK from 6be3bb459ef1 to b410651bd18e (2 revisions) (flutter/engine#27514)
2021-07-18<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from uhLYz8rWn... to vfv7CH0zH... (flutter/engine#27527)
2021-07-18<EMAIL_ADDRESS>Roll Skia from f02aa80ba91b to 1f261da41ea4 (1 revision) (flutter/engine#27525)
2021-07-18<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from jBy624LbH... to 1yUmKH13E... (flutter/engine#27524)
2021-07-18<EMAIL_ADDRESS>Roll Skia from 8837c919fb8b to f02aa80ba91b (1 revision) (flutter/engine#27523)
2021-07-18<EMAIL_ADDRESS>Roll Skia from 3d49efa8e177 to 8837c919fb8b (1 revision) (flutter/engine#27521)
2021-07-18<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 0PyjWN1eh... to uhLYz8rWn... (flutter/engine#27520)
2021-07-18<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from Y9N1cBuxa... to jBy624LbH... (flutter/engine#27517)
2021-07-17<EMAIL_ADDRESS>Roll Skia from 01e6273c46f0 to 3d49efa8e177 (1 revision) (flutter/engine#27516)
2021-07-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from JuA3-RaN-... to 0PyjWN1eh... (flutter/engine#27513)
2021-07-17<EMAIL_ADDRESS>Roll Skia from 11a20bbbf4d5 to 01e6273c46f0 (1 revision) (flutter/engine#27512)
2021-07-17<EMAIL_ADDRESS>Roll Skia from 7c79a871c084 to 11a20bbbf4d5 (1 revision) (flutter/engine#27511)
2021-07-17<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from ufdC-mDPS... to Y9N1cBuxa... (flutter/engine#27507)
2021-07-17<EMAIL_ADDRESS>Roll Dart SDK from c99d9ea1ac3a to 6be3bb459ef1 (1 revision) (flutter/engine#27505)
2021-07-17<EMAIL_ADDRESS>Roll Skia from c82ab0839f06 to 7c79a871c084 (1 revision) (flutter/engine#27504)
2021-07-17<EMAIL_ADDRESS>Roll Skia from fb0440e5bcb6 to c82ab0839f06 (2 revisions) (flutter/engine#27502)
2021-07-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from FwgLM4h6t... to JuA3-RaN-... (flutter/engine#27501)
2021-07-17<EMAIL_ADDRESS>Migrate non-scenic-based fuchsia tests to cfv2. (flutter/engine#27241)
2021-07-17<EMAIL_ADDRESS>Roll Skia from fe5d07a8e471 to fb0440e5bcb6 (1 revision) (flutter/engine#27500)
2021-07-17<EMAIL_ADDRESS>Roll Dart SDK from 207232b5abe0 to c99d9ea1ac3a (8 revisions) (flutter/engine#27499)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from ufdC-mDPS5KW to FGuPZEZLt_5Z
fuchsia/sdk/core/mac-amd64 from FwgLM4h6tJNU to 897eI2xwc8f5
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Gold has detected about 13 new digest(s) on patchset 27.
View them at https://flutter-gold.skia.org/cl/github/86670


Add redcap_bridge (replacement of #8)

Pull Request Test Coverage Report for Build 744456830

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 741116555:
0.0%

Covered Lines:
4

Relevant Lines:
4

💛  - Coveralls

\o/ test are passing!

Yeaha !
\o/

I'm happy also, but you're gonna have to explain to me what's redcap_bridge... Ahah ;)


v1.1.0

That is a very difficult pull request to merge: RipaEx does not have a DevNET at the moment and merging those changes directly in all nodes in the MainNET without testing can brake the blockchain currently live

I will merge into a branch v1.1.0 and test it in there: if the test is successful I will merge into master


Disable informational console logging

I have a suggestion for a less 'messy' dev experience.
Maybe it's possible to disable all the informational console logging of webpack-hot-middleware?
Instead of having this every time, with every refresh:

It would look like this, cleaner:

Warnings and errors will still be shown, just not all the console logging.

@jetmirshatri we're working on a nicer way to show the hot code reloading state (@evilrabbit_), and we'll be sure to remove those pesky logs.
Thanks for your contribution

@rauchg should we open an issue for this? 😄


Compile curl / busybox only for current camera

I am a little new to compiling things, having looked at how to compile using the wiki here, it is only for compiling firmware and adding program to firmware which would take up space. I want to use programs on the sd card under yi-hack-v* folder.
I have the original yi home camera with yi-hack-v3 initially and now v4 on there there was:
busybox-1.28
curl
I can see that there has been an attempt made to compile on mstar also:
https://github.com/roleoroleo/yi-hack-MStar/issues/77
Is there a allwinner version?

You could use the scripts as a template and copy the output binary manually.

Thanks for your response. I will attempt it and feedback.

So I have tried to both extract the file and create symlinks myself manually, however I keep getting
"Operation not permitted"
ln -s /tmp/sd/yi-hack/lib/libcrypto.so.1.1 /tmp/sd/yi-hack/lib/libcrypto.so
ln -s /tmp/sd/yi-hack/lib/libcurl.so.1.1 /tmp/sd/yi-hack/lib/libssl.so
ln -s /tmp/sd/yi-hack/lib/libcurl.so.4.6.0 /tmp/sd/yi-hack/lib/libcurl.so
ln -s /tmp/sd/yi-hack/lib/libcurl.so.4.6.0 /tmp/sd/yi-hack/lib/libcurl.so.4

root@xiaoyi:/tmp/sd/yi-hack/lib# tar -xvzf curl-7.68.0.tar.gz
curl
libcrypto.so
tar: can't create symlink libcrypto.so to libcrypto.so.1.1: Operation not permitted
libcrypto.so.1.1
libcurl.so
tar: can't create symlink libcurl.so to libcurl.so.4.6.0: Operation not permitted
libcurl.so.4
tar: can't create symlink libcurl.so.4 to libcurl.so.4.6.0: Operation not permitted
libcurl.so.4.6.0
libssl.so
tar: can't create symlink libssl.so to libssl.so.1.1: Operation not permitted
libssl.so.1.1

Think it is just a permissions issue

After some further looking I thought I might as well get everything else up and running, however it appears that the /tmp/sd/yi-hack folder on sd card isn't on the path by default? I thought it was looking at the "env.sh" and "system.sh" file.
Reckon maybe the init.sh in home/app might be overriding the path and library. After manually setting the path, I am back at the original issue o being able to sym link

You can't create symlinks on FAT32 file systems. It's not supported.
As a workaround you could create 3 files for each library with different names and same content.
The PATH variable is changed by system.sh: all programs launched from system.sh inherit this variable.
But if you login with telnet or ssh you haven't it. You can add it temporarily (only for the current session) sourcing the file env.sh:
. /home/yi-hack/script/env.sh

Okay, just did as you said, copied and renamed files. I suppose its not that big of an issue if on SD card anyway. I tested putting startup script with binaries on sd card and it seems to load the path and library just not in session. Not a big deal can just load the env.sh


feat(response_transfomer) add extract config

Summary
This is a PR for the issue 3938
Our case is around the integration of Kong + AWS Lambda plugin. When an error occurs in the Lambda end, the full response sent back by the Lambda is displayed on the response body, which includes the stack trace and the type of error.
Here's an example:
{
    "errorMessage": "{\"errorCode\":8,\"errorMessage\":\"actual error message\"}",
    "errorType": "java.lang.Exception",
    "stackTrace": [
        "sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)",
        "sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)",
        "sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)",
        "java.lang.reflect.Constructor.newInstance(Constructor.java:423)",
        "clojure.lang.Reflector.invokeConstructor(Reflector.java:180)",
        "shared.lambda$format_error.invokeStatic(lambda.clj:14)",
        "shared.lambda$format_error.invoke(lambda.clj:8)",
        "shared.lambda$handle.invokeStatic(lambda.clj:28)",
        "shared.lambda$handle.invoke(lambda.clj:17)",
        "clazzz.lambda$G__10962handleRequest.invokeStatic(lambda.clj:7)",
        "clazzz.lambda$G__10962handleRequest.invoke(lambda.clj:6)",
        "clazzz.lambda.handler.handleRequest(Unknown Source)"
    ]
}

The payload above contains way too much info and what we really want to send back is the contents of the errorMessage key. Since this is JSON content, we thought we could use the response-transfomer plugin to extract that information and send it back but the plugin does not support extraction.
So we decided to have a simple implementation of the use case above with the only assumption that the content of the key you want to extract is actually a stringified JSON.
As you can see by the tests, you can also combine with the other configuration options in the plugin allowing for manipulation of the content extracted. We thought that's a very powerful feature - initially we were not going to do this way but thought that could be really useful for other use cases.
One thing that I'm not clear on is the migrations - might need some help with that.
Thus the sequence goes like this now:
remove -> extract -> replace -> add -> append

Hope that makes sense. Happy to provide more information.
Full changelog

feat(response_transfomer) add extract config

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tarcio Saraiva seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@tarciosaraiva Thanks for the PR! We have looked at it and there's a bit of a concern that this may be a too narrow use-case for adding into the plugin... maybe if it also supported non-stringified versions this would make more sense as a general feature? Any thoughts?

@tarciosaraiva Thanks for the PR! We have looked at it and there's a bit of a concern that this may be a too narrow use-case for adding into the plugin... maybe if it also supported non-stringified versions this would make more sense as a general feature? Any thoughts?

Hey @hishamhm thanks for your reply. So I guess you have a point about non-stringified versions but it would be very hard for me to test it right now since I don't work at the place where this feature request was applicable anymore - sorry it's been more than 1 year since I opened this PR.
If you still think there's value in this I can try and get some scenarios running locally, no issues. Otherwise, feel free to close the PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tarcio Saraiva seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing because of lack of activity. We would like to accept this contribution if anyone wishes to pick this back up.


More upgrade

Pull Request
Description
Updates to:

Add recursive CNN models as options for the image encoder
Allow different image encoders for satellite and NWP with new optional parameter
Use effective capacities pulled from PVLive inside the app

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

Y

Code looks good.
Would you be able to add some model results. Perhaps a table of Forizong horionz and MAE for the old model and this one. Could just do hours 0,1,2,4,and 8?

Yeh sure, I'll add a table in tomorrow

nice thanks

I ran some tests on using the effective capacity instead of the installed capacity at test time for the current model used in the app. I used a dataset scraped from production from the 8th - 19th of June 2023. The results for using the 2 different capacities are shown below. These are the MAEs (in MW) compared to PVLive national.

installed capacity
effective capacity

0
277.591
176.887

1
267.336
183.76

2
258.972
197.684

3
244.926
218.806

4
228.007
231.098

5
216.842
252.164

6
219.066
253.204

7
223.344
263.131

8
228.136
244.427

9
220.702
237.81

10
231.86
232.174

11
242.839
222.232

12
249.38
229.147

13
265.02
210.434

14
254.916
212.988

15
254.345
215.57

These are the MAEs (in MW) compared to PVLive sum of GSPs.

installed capacity
effective capacity

0
208.622
98.4234

1
192.824
100.75

2
174.131
118.636

3
153.464
146.622

4
137.781
172.134

5
137.488
202.943

6
145.356
207.18

7
149.642
215.174

8
159.737
201.403

9
165.691
206.292

10
174.322
200.463

11
184.509
185.069

12
181.436
187.266

13
206.504
161.02

14
184.979
161.773

15
185.337
161.205

I ran some tests on using the effective capacity instead of the installed capacity at test time for the current model used in the app. I used a dataset scraped from production from the 8th - 19th of June 2023. The results for using the 2 different capacities are shown below. These are the MAEs (in MW) compared to PVLive national.
hours ahead	installed capacity	effective capacity
0.5	277.591	176.887
1	267.336	183.76
2	244.926	218.806
4	223.344	263.131
8	254.345	215.57
These are the MAEs (in MW) compared to PVLive sum of GSPs.
hours ahead	installed capacity	effective capacity
0.5	208.622	98.4234
1	192.824	100.75
2	153.464	146.622
4	149.642	215.174
8	185.337	161.205

Just to check are these for the same data? Did you take the data from SQL and then adjust it by using the effective capacity?

you coulndt bump the datamodel to 1.4.5 as well please?


Remove usages of ANDROID_SDK_ROOT in the engine

https://github.com/search?q=repo%3Aflutter%2Fengine ANDROID_SDK_ROOT&type=code
https://cs.android.com/android-studio/platform/tools/base/+/mirror-goog-studio-main:build-system/gradle-core/src/main/java/com/android/build/gradle/internal/SdkLocator.kt;l=197?q=Several environment variables and%2For system properties contain different paths to the SDK
https://cs.android.com/android-studio/platform/tools/base/+/24d480687a1d5f92d898dd49e23286cdf8b0e3fd
The reason for targeting the engine is that if both ANDROID_HOME and ANDROID_SDK_ROOT are set but have the same value then there is not an error. But the engine overrides the local android sdk when building so it causes build errors.

Just want to clarify, is this essentially tracking replacement of ANDROID_SDK_ROOT with ANDROID_HOME?

yes


Change fig sizes to better display on notebooks

Figure sizes were too large, and this would make difficult to visualize
the plots on jupyter notebooks and on our web tutorial.

Codecov Report

Merging #296 into master will increase coverage by 0.21%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #296      +/-   ##
==========================================
+ Coverage   80.01%   80.23%   +0.21%
==========================================
  Files          11       11
  Lines        3168     3167       -1
==========================================
+ Hits         2535     2541       +6
+ Misses        633      626       -7

Impacted Files
Coverage Δ

ross/rotor_assembly.py
92.72% <100%> (+0.12%)
:arrow_up:

ross/results.py
37.5% <29.41%> (+0.98%)
:arrow_up:

Continue to review full report at Codecov.

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


Problems with example (Visual Studio 2010)

Hi! I'm trying to use the library with visual studio 2010 with one example, but adding a new project to the library and copying the paths, the following error is shown:

How can I fix it?
Thanks

You need to link your application against the SimpleITK library.

I also tried to do that, but it returns me the same errors. Which .h files should I add to the linker dependences?
Thanks

Closing due to inactivity. Feel free to reopen if this is still an issue.


mac hyperctl

This page https://github.com/hyperhq/docs.hypercontainer.io/blob/master/get_started/install/mac.md
that is not linked from the docs (?) has a download link for a package that installs "hyper", but no "hyperctl" (like it should according to the latest commit message to that file)
Are you still maintaining mac builds? I think that hyperctl would be super useful to have as a pre-built thing.

The Mac version had been disabled, and the link to this page had been removed form the TOC.

@gnawux can you help me out building one? or is it not doable anymore?
make
Making all in mac_installer
make[1]: Nothing to be done for `all'.
go build -tags "static_build   exclude_graphdriver_btrfs libdm_no_deferred_remove" -ldflags "-X github.com/hyperhq/hyperd/utils.VERSION=0.8.0 -X github.com/hyperhq/hyperd/utils.GITCOMMIT=`git describe`" hyperd.go
hyperd.go:11:2: cannot find package "github.com/docker/docker/opts" in any of:
	/usr/local/Cellar/go/1.8/libexec/src/github.com/docker/docker/opts (from $GOROOT)
	/Users/mpa/.go/src/github.com/docker/docker/opts (from $GOPATH)
hyperd.go:20:2: cannot find package "github.com/docker/docker/pkg/parsers/kernel" in any of:
	/usr/local/Cellar/go/1.8/libexec/src/github.com/docker/docker/pkg/parsers/kernel (from $GOROOT)
	/Users/mpa/.go/src/github.com/docker/docker/pkg/parsers/kernel (from $GOPATH)

Mac OS X build is not available in current version, and I closed this issue. However, it's always welcome if you could work on it and submit PR for it if you like to make a client for Mac.