pip installs from git don't work correctly with pip freeze.

This is actually a bug with pip freeze, but having looked into it, it seems like a fix at that end is going to be really difficult or unlikely to be merged, so I'm filing here first.
If you do something like:
pip install -e git+https://github.com/limist/py-moneyed/@f9f4b8c4bfdded3433fbc88cd478d90ab2670eb0

And then do:
pip freeze

you will find that instead of listing the git URL, it shows:
py-moneyed==0.5.0

(Tested with latest pip==7.1.0). This is due to a bug in pip freeze, which assumes that the directory containing the installed module is the top-level VCS dir.
In the case of py-moneyed, the installed module ends up in: /your/virtualenv/src/py-moneyed/src. This is due to the folder layout, in which moneyed is within src/, instead of one level up, which seems to be the norm in every other Python project I can find (I did a quick scan down 10-15 projects on PyPI, and every one where I could quickly find the source used the other convention).
Technically this is a bug in pip. But looking into it, it's going to be extremely difficult or impossible for them to fix. When it comes to doing pip freeze, pip has a list of directories where modules are found. The only way to correctly detect a VCS install is to look in the directory for .git (etc.), and then recurse upwards. However, once you start recursing upwards, you risk all kinds of issues - for example, traversing file system boundaries which could trigger permission errors, network errors (for network mounted folders) etc. etc. I imagine the maintainers of pip are not going to want to include that behaviour, especially when the current approach, while incorrect, works really well for most Python projects.
So, I suggest that everything in src/ gets pulled up one level. This is as simple as:
git mv src/moneyed .

and then fixing the setup.py file. I'm very happy to submit a PR for this if you accept it - it's a really simple change. I've also checked that this doesn't screw things up with regards to existing patches and pull requests, and git seems to handle this fine.

I vaguely remember there was a convention/reason to use a top-level src/ dir, but that was a while ago, so go ahead and pull up that dir, thanks.


Cutsom company logos for the Chinese landing page.

The Chinese landing page should have the following logos in order:
Apple, Google, Microsoft, Alibaba and Tencent

Hi @ahmadabdolsaheb , i wanna work on it , can you help me in getting started with this issue !

Sure, we would like to replace Amazon and Spotify logos with Alibaba and Tencent when the clientLocale === 'chinese'.
Here is where the logos are displayed. Please follow the path to see where they are located. We would like similar svg react components for Alibaba and Tencent.
Then we would pull clientLocale similar to the following and display the above mentioned logos when the client is chinese.
https://github.com/freeCodeCamp/freeCodeCamp/blob/c0e24d735caa04e8a1874aef82fdc67b076fdee8/client/src/components/profile/components/Camper.js#L19

Thanks @ahmadabdolsaheb  :-)

Dear @ahmadabdolsaheb , i have followed all the steps for setting up locally but getting this error while running ' npm run develop ' .
Error in "F:\open source\freeCodeCamp\client\node_modules\gatsby-plugin-manifest\gatsby-node.js":
Something went wrong installing the "sharp" module
Cannot find module '../build/Release/sharp.node'
Require stack:

F:\open source\freeCodeCamp\client\node_modules\sharp\lib\constructor.js
F:\open source\freeCodeCamp\client\node_modules\sharp\lib\index.js
F:\open source\freeCodeCamp\client\node_modules\gatsby-plugin-manifest\safe-sharp.js
F:\open source\freeCodeCamp\client\node_modules\gatsby-plugin-manifest\gatsby-node.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\bootstrap\resolve-module-exports.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\bootstrap\load-plugins\validate.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\bootstrap\load-plugins\load.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\bootstrap\load-plugins\index.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\services\initialize.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\services\index.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\state-machines\develop\services.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\state-machines\develop\index.js
F:\open source\freeCodeCamp\client\node_modules\gatsby\dist\commands\develop-process.js
F:\open source\freeCodeCamp\client.cache\tmp-5024-mcD7XsGWBxgh

How do you create or find the svg's for those @ahmadabdolsaheb?

We followed the following steps for the other svgs:

We found the images for logos and turned them into svgs if necessary.
Optimize the size of svgs using an online tool ( there are a lot of them, here is one).
Rectified the optimized svg.


perl538Packages.CSSDOM: fix build

Description of changes
CSSDOM's test failed due to the deprecated use of a package seperator. There's a patch upstream, but it hasn't been merged yet. The corresponding ticket has been idle since August.
See https://rt.cpan.org/Public/Bug/Display.html?id=146661
This adds the patch directly to nixpkgs, because the perl bug tracker seems to refuse automated downloads.
ZHF: #265948

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Thx, waiting for staging-next to be merged as it contains 64f82333aae1657df4bb232320ed71f92816b9d1 affecting this package

Your commit fixes this already at the expense of removing a test. Mine uses an upstream patch and keeps the test. The question is do we care for having the test.

I prefer your solution of patching the test to be honest, feel free to update the PR to also remove my change.


Window size and positions are not restored correctly

This doesn't always happen so it is hard to to capture steps. But here is what I see when it happens:

Observe that even after resizing the workbench window, closing it and reopening it the size is not remebered

If this is a confirmed regression, it would help to know which change caused it. We maintain a node.js based CLI tool  vscode-bisect that automatically downloads and runs previous VS Code insiders and asks for each build if the issue reproduces. It takes up to 8 steps to find exactly the build that caused the regression.
Steps:

install node.js (if not yet done)
npx --yes @vscode/vscode-bisect@latest
follow the instructions until you found the offending build (its fine to leave the commit empty)
report back the commit range in this issue

Thanks!

I digged a little deeper and here are steps:

open a terminal
change into a workspace directory (e.g. vscode)
run code-insider .
change into another workspace directory
run code-insider .

Observe: the size and position of the second workspace window are not restored correctly

resize the second opened window
close the window
run code-insider . again

Observe: the window opens again with its default size.

Version: 1.85.0-insider (user setup)
Commit: f3da7cb46a8ffbea21998420dc673777715e55cf
Date: 2023-11-22T05:35:23.085Z
Electron: 25.9.4
ElectronBuildId: 25127168
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22635
Happens on Windows and WSL

Can repro in stable as well:
Version: 1.84.2 (user setup)
Commit: 1a5daa3a0231a0fbba4f14db7ec463cf99d7768e
Date: 2023-11-09T10:51:52.184Z
Electron: 25.9.2
ElectronBuildId: 24603566
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22635

Previously https://github.com/microsoft/vscode/issues/61838
Dirk, in this case you have other windows still open. We only persist state when you fully exit. To solve this, we need to keep state of windows associated to the workspace in the session. Turning into a feature request.

/duplicate https://github.com/microsoft/vscode/issues/170635

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Browser extension to run arRPC Server

Possible? The obvious advantage being that it is convenient and you don't need to have Node installed (or maybe you can provide prebuilt binaries)

Not possible, browser extensions can't run native things. It would be possible to run the client/mod everytime via an extension without having to use the console.

It would be possible to run the client/mod everytime via an extension without having to use the console.

Or a userscript, that would be better

Both of them doesn't seem to work

Here's for the userscript:

And here is for the extension:


mqtt websocket change to mqtt

Hello @donskytech
I'm trying instead of mqtt websocket with mqtt and it doesn't work for me,
do you know why?
mqtt websocket:
MQTT_BROKER=ws://<IP_ADDRESS>:9001/mqtt
mqtt:
MQTT_BROKER=mqtt://<IP_ADDRESS>:1883
thanks you.

Hey, if you want something like this then you can do it on the server side not on the browser side.  Use mqtt.js library on your app.js.

Hi @donskytech
Do you have any videos on YouTube or repository that can be viewed server side use mqtt.js library?

I find this one, a good introductory for this topic https://www.emqx.com/en/blog/how-to-use-mqtt-in-nodejs


No error yet not slide change (tried the demo server, own server, own server without reverse proxying)

I checked https://github.com/reveal/multiplex/issues/10 then https://github.com/reveal/multiplex/issues/12 but I still didn't find a solution.
I tried :

Glitch
Glitch remix
Reveal update (have 4.3.1 now)

own server in container directly exposed (HTTP/WS)

own server in container behind reverse proxy (HTTPS/WSS)

But no dice. It seems the connection is established properly, master and client connecting to the same server, trying to switch protocol but then nothing happens.
I'm consider re-implementing something similar with PeerJS since I'm a bit familiar with it but suggestions welcomed before I do.

You can try live with https://fabien.benetou.fr/Testing/35C32018?action=revealcontrol as master and https://fabien.benetou.fr/Testing/35C32018 as client. I didn't see anything via the console nor server-side :

root@ns3031697:/home/web/benetou.fr/fabien/cookbook# docker logs reveal-multiplex
reveal.js: Multiplex running on port 1948
(node:1) [DEP0106] DeprecationWarning: crypto.createCipher is deprecated.
(Use node --trace-deprecation ... to show where the warning was created)

Well I tried https://github.com/rajgoel/reveal.js-plugins/tree/master/seminar instead and it worked right away so moving to this for now.


Add sounds

add sounds to following actions:

[ ] exploration
[ ] hex travel

[ ] Lords sound (like resources sound)
[ ]  Level UP sound

[ ] running sound

fixed in #485


Resource Graphs: Add the rest of relations

Description

This could be easily parallelised for different categories, clusterwide/namespaced resources etc.

Find out how they are connected and add them in shared/components/ResourceGraph/relations/matchers.js and shared/components/ResourceGraph/relations/relations.js.

Configure them on their respective details view - for some resources (e.g. Namespaces) we don't need to show the graph at all:

or we could do away with shallow graph:

ingress
network policy
destination rule
sidecar
auth policy
svc entry
pvc
volume
subscription
git repo
issuer
certifcate
storage class
stateful set
deamon set
cron job
job

A task will be added for each one, on a need basis


PR for Windows automation

Issue:  https://github.com/rancher/qa-tasks/issues/151

Problem

Currently there is no support to add Windows node as agent to a rke2 standalone cluster. The process of adding the agent is tedious and time consuming, specially during rke2 patch or minor release validation.
Solution

This PR adds the ability to automatically add Windows node as agent to a rke2 cluster
Testing

Engineering Testing
Manual Testing

Automated Testing

QA Testing Considerations

Regressions Considerations

@rancher-max @ShylajaDevadiga please review, made some changes to retry logic


render the first, top message in the "wall" LARGE

This feature has the target to reach the milestone at
https://groups.google.com/forum/#!msg/loklak/7azwyuslmTU/q9ueCSgwjZUJ
it also is based on https://github.com/loklak/loklak_webclient/issues/31
We want to see the first/top message in the timeline VERY LARGE. To show the message in a large form, increase the image size (take the large one, 400x400, link like
https://pbs.twimg.com/profile_images/577515909335511041/AFPxKLw3_400x400.png )
and display the message text in larger letters.
This way to display the message shall encourage people in conferences, who see the wall simultaneously during speeches, to tweet their own message to appear on the wall.

the given image link path  in the loklak json actually always ends '_bigger'. if you replace 'bigger' with 400x400 or 200x200 you get the picture which is bigger than 'bigger' :)

For the first message, do we display the images ?

yes, please!

I tested this and it looks good!
but...when the entries below the large image is updated, the whole list loads again. That should be done more smoothly, like shifting them on the screen below.

Oh, normally there is a transition animation, you have no animation at all ?

The first message in the wall is comparatively larger than the rest. I think this can be closed safely, if this needs more discussion, please feel free to reopen.


Goulart paul patch 1

I have just fixed this typo in  the original cleanup PR. I think this one is no needed.


BooleanField shows label twice

In 1.3.9, a BooleanField named checkbox will show "Checkbox: Checkbox" before the input control. Using the bootstrap FormRenderer.

Confirmed. Thanks for reporting.

This regression has been added in aa0be005a987bd354d54afff5127ac3aca923b7e and was part of release 1.3.8 and 1.3.9. Will be fixed in 1.3.10. Sorry for the inconvenince.


Update 02-analyze-spark.md

Module: 02
Lab/Demo: 01
Fixes #123
Changes proposed in this pull request:

make the codeblock follow the same typing as the rest of the document
If not it adds spaces in front of the codeblock throwing off proper spacing

The change in the syntax highlighting for the code block from lowercase p to uppercase P yields the same output in the .md file


Adding --include-source-info to options for descriptor output

Addressing https://github.com/os72/protoc-jar-maven-plugin/issues/36.
This PR adds the ability to optionally add the --include_source_info option when invoking protoc

Ack. It's not the intended use, but I guess we could overload it
How about just: if (outputOptions != null) cmd.add(outputOptions);?

For added flexibility, I've added the ability to have multiple outputOptions.

@os72 - awesome - thanks!


Implement reading of large objects

Part of the work for #554.

Seems like the Kokoro test failed due to some quota issues:
C++ exception with description "InstanceAdmin::DeleteCluster(parent=projects/cloud-cpp-testing-resources) Insufficient tokens for quota 'InstancesWriteGroup' and limit 'CLIENT_PROJECT-1d' of service 'bigtableadmin.googleapis.com' for consumer 'project_number:936212892354'.: InstanceAdmin::DeleteCluster(parent=projects/cloud-cpp-testing-resources) Insufficient tokens for quota 'InstancesWriteGroup' and limit 'CLIENT_PROJECT-1d' of service 'bigtableadmin.googleapis.com' for consumer 'project_number:936212892354'. [8=RESOURCE_EXHAUSTED] - ï¿½Insufficient tokens for quota 'InstancesWriteGroup' and limit 'CLIENT_PROJECT-1d' of service 'bigtableadmin.googleapis.com' for consumer 'project_number:936212892354'.ï¿½
#type.googleapis.com/google.rpc.Helpq

@sduskis or @mbrukman is there a way to fix this?

Connected with @coryan offline to resolve the API quota issue.


Corrections for IntelliJ configuration - fix initial build

make folder configuration for web interface module more versatile
(fixing error appearing when running gradlew idea before build
and creating configuration for generated sources folder)
configure folders for ubw importer module
enable annotation processing

Six of seven commits were necessary, next try in #106.

You don't have to create a new PR, just squash your changes and force push the changes to your pr branch. Or ask the owner to squash merge your pr, then  all commits get squashed by github before the merge is done. So there will only be one commit for the whole pr in the owners history.

Still exercising...


Primary and foreign key data type do not match

When adding constraints to our project we've discovered some datatype inconsistencies within our models.
For example in a fact table we specify the customer_key as varchar and in the dimension as varchar(50). By default Snowflake will create the fact table with varchar(16777216) which obviously is not the same as varchar(50).
Upon running the dbt build every test passes. But in the post hook when actually creating the constraints Snowflake will return the database error that keys are not matching.
When running with multiple threads its very hard to figure out which constraint has failed to build.
There's a couple of solutions to help finding the cause of the error:

When the database error occurs also log which constraint is causing it
When a foreign key is specified not only test the existence of the key, but also check the data type and fail / warn if there's a mismatch.

I usually try to avoid errors and report the condition to users so I like option 2. I will implement the check and let you know when it is available.


[docs] added prop docs TabelCell

Improved docs:
colSpan: Specifies the number of columns a cell should span.

This is a native prop. We don't document them on purpose. I would propose that we document this logic https://github.com/mui-org/material-ui/blob/1e4d2db346e0e31ac1b85d371ee9c375fb1545cc/packages/material-ui/src/TableCell/TableCell.js#L125
In the beginning of the API page.

@oliviertassinari thank you for explain
I think we need ref to this item that all prop native its work

@uxitten What do you think of this alternative language?

@oliviertassinari It's great

@uxitten Thanks


Bug fix, now can be use in tensorflow 2.6.4

部分Library路徑改了，（該死的Tensorflow，改了也沒什麽人說，還好github能看修改記錄）還有自己使用時候遇到'Functional' object has no attribute '_layers'也試著改了下,强烈建議使用pydotplus，pydot出現各種奇奇怪怪報錯。
Some of the Library paths have been changed, and I also tried to change the 'Functional' object has no attribute '_layers' when I used it myself. It is strongly recommended to use pydotplus, and pydot reported various strange errors.


Let's read kmod

What is kmod?
Quote from kmod's README:
kmod is a set of tools to handle common tasks with Linux kernel modules like insert,
remove, list, check properties, resolve dependencies and aliases.

Why kmod?
The kmod project is widely distributed:

It is part of the Android Open Source Project
Many Linux distributions ship it in their default installations

Notes
The code by itself is not exactly security relevant, since elevated permissions are used for almost all tools. The only aspect of interest might be the inspection of module files through modinfo which could be done with untrusted module files. In general, this allows open discussions about stability issues.

By now, Tuesday 8 pm (CET) works best for everyone, but feel free to offer other more fitting times.


cmd/update-report: always output new formulae.

There's near zero performance overhead and this seems good for discovery.
Marking as critical to get into 3.5.1 today.
Fixes #13412

Review period skipped due to critical label.


[Feature Request] Add a button to turn voice chat on or off

I would like to add a button to activate voice chat on my phone from the pause menu, Since I haven't found a way to make it work, I even connected a keyboard to it via OTG and it wasn't useful. I clarify that I already made the respective configurations from the menu and I have not had any improvement. I just hope I can be heard and see this feature in a few weeks or months, until then, bye

Idk but yeah same.....


Correct Wordpress admin bar location

Did not found a solution or comments on this, may be I am doing something wrong, but the code is quite simple:
I have index.twig template and everything works fine, until I login into admin -

Wordpress adds admin-bar HTML before index.twig template code itself.
Wordpress does not adds additional admin classes for  tag

looks like this:
In index.php file:
...
Timber::render(array('/pages/index/index.twig'), $context, 1);

my .twig file:
<!DOCTYPE html>

    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Klinta</title>
    <link rel="stylesheet" href="{{site.theme.link}}/css/bootstrap.min.css">
    <link rel="stylesheet" href="{{site.theme.link}}/style.css">
    {{ function('wp_head') }}

Result in WEB page, with logged in admin:
...

Rediģēt savu profilu

Atslēgties

                        Atslēgties

<!DOCTYPE html>

...
...

...

@arnisjuraga — that's really strange. The admin bar outputs via {{ function('wp_head') }} so it should come within those  tags. Can you remove that line (in the twig file) and see if you still see the admin bar output? My guess is you have a wp_head() call (or something calling it) in one of those PHP files

The HTML code for the admin bar actually outputs at the end of the document via wp_footer(). So check if you have that in your template.

@arnisjuraga Is this still something you're actively working on or need addressed? Please respond in two weeks or we'll close this thread

Hi! No, I am not working with Timber anymore, so - I cant tell anything new about this issue. If no one else is experiencing this, we can close it freely.


Feature request: Firebase dynamic links

Hi, I was wondering if you were thinking about adding support for Firebase Dynamic Links: https://firebase.google.com/docs/dynamic-links/
There is only one library for php so far, i have found: https://packagist.org/packages/yedincisenol/dynamic-links
I am already using kreait/firebase-php in our app so it would be lovely to not need more packages for one service 😄

same question

+1. I also need this feature.

I‘m on it and will publish a Draft-PR soon for you to test with. I‘m currently trying a different approach on handling things in the SDK, so please don‘t expect it to be production ready in the short term 🙈

@jeromegamez Thanks for update. Will help you test this

@swayok @jhoanborges @JanMikes Please have a look at #325 - I believe that this everything you can do in respect to Dynamic Links can be done. If something is not working, please let's discuss this over there :)

Release 4.32 includes the new Dynamic Links component, including Dynamic Link Analytics. 🎉

@jeromegamez Nice! Will give it a try within next several days =)


Fix duration=0 & stream disabled bugs

Overview
Fix the following bugs in openai:

During errors, the function trace is not exited before capturing the duration so the duration=0 inside the events.
When not streaming but stream monitoring is disabled, the function trace is exited early and events are not recorded.

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

Project coverage is 81.83%. Comparing base (b950070) to head (cd05a7c).
Report is 2 commits behind head on develop-ai-limited-preview-3.

Additional details and impacted files
@@                       Coverage Diff                        @@
##           develop-ai-limited-preview-3    #1096      +/-   ##
================================================================
- Coverage                         81.86%   81.83%   -0.03%
================================================================
  Files                               192      192
  Lines                             21253    21177      -76
  Branches                           3711     3699      -12
================================================================
- Hits                              17398    17331      -67
+ Misses                             2783     2777       -6
+ Partials                           1072     1069       -3

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


Refer to the API documentation

Fixes #949

[x] I agree that my contribution may be licensed either under MIT or any version of LGPL license.

Thank you for the PR. Looks good to me.
one little remark regarding your commit message: Could you please amend your commit message to reference  issue 949 instead 950?

Done


Add Matter test suite

See PR review on obsolete repo: https://github.com/MonicaisHer/matter-snap-testing-draft/pull/1
This PR's GitHub action has been tested in https://github.com/canonical/chip-tool-snap/pull/31
Testing
# clear temporary chip file
$ sudo rm -rf /tmp/chip_*

matter-snap-testing/test/suites/chip-tool$ go test -v -failfast -count 1 .
2024/01/08 13:53:46 [CLEAN]
2024/01/08 13:53:46 [exec] sudo snap remove --purge chip-tool
2024/01/08 13:53:46 [stderr] snap "chip-tool" is not installed
2024/01/08 13:53:46 [SETUP]
2024/01/08 13:53:46 [exec] sudo snap install chip-tool --channel=latest/edge
2024/01/08 13:53:48 [stdout] chip-tool (edge) v<IP_ADDRESS>+snap from Canonical IoT Labs installed
2024/01/08 13:53:48 [exec] sudo snap connect chip-tool:avahi-observe
2024/01/08 13:53:48 [exec] sudo snap connect chip-tool:bluez
2024/01/08 13:53:48 [exec] sudo snap connect chip-tool:process-control
=== RUN   TestMatterDeviceOperations
=== RUN   TestMatterDeviceOperations/Commission
    exec.go:19: [exec] sudo chip-tool pairing onnetwork<PHONE_NUMBER>1
=== RUN   TestMatterDeviceOperations/Control
    exec.go:19: [exec] sudo chip-tool onoff toggle 110 1
    main_test.go:130: Retry 1/10: Waiting for expected content in logs: CHIP:ZCL: Toggle ep1 on/off
    main_test.go:139: Found expected content in logs: CHIP:ZCL: Toggle ep1 on/off
--- PASS: TestMatterDeviceOperations (1.80s)
    --- PASS: TestMatterDeviceOperations/Commission (0.49s)
    --- PASS: TestMatterDeviceOperations/Control (1.32s)
PASS
2024/01/08 13:53:50 [TEARDOWN]
2024/01/08 13:53:50 [exec] (sudo journalctl --since "2024-01-08 13:53:46" --no-pager | grep "chip-tool"|| true) > chip-tool.log
Wrote snap logs to /home/mengyi/Desktop/881/matter-snap-testing/test/suites/chip-tool/chip-tool.log
2024/01/08 13:53:50 Removing installed snap: true
2024/01/08 13:53:50 [exec] sudo snap remove --purge chip-tool
2024/01/08 13:53:52 [stdout] chip-tool removed
ok      matter-snap-testing/test/suites/chip-tool       6.212s

Closing this pull request as development has shifted to MonicaisHer/matter-snap-testing#2 to maintain a clearer trace of the review history.


Fix missing/broken Staff & Wand recipes

Some Wand & Staff recipes like the Primal Staff and Witchwood are currently not working. (Scepters and Staffters work fine)
This is due to the research assigned to the arcane recipe not existing.
The Scepter and Staffter recipes were already using the "SCEPTRE" research when not explicitly set and as such always worked. In order to ensure Wands and Staffs work ive aligned the default researches to BASICTHAUMATURGY (Basic Wand Crafting) and ROD_greatwood_staff (Magic Staves) respectively.

It was tested in sp, so it should work


After upgraded to latest weaveexec weave images, the issue is still there: weave Net 2.4.1 on macosx 10.13.6

No DNS entry added after container started.
steps to repoduce:
Docker Version 18.06.1-ce-mac73 (26764)
install weave 2.4.1 by curl
update /usr/local/bin/weave with:
#SCRIPT_VERSION="2.4.1"
SCRIPT_VERSION="latest"
eval $(weave env)
docker run --name AAA bash /usr/bash
weave ps
No entry listed for new container.

It is a duplicate of https://github.com/weaveworks/weave/issues/3405 which fix hasn't been released yet.


[Windows] Fix #112: Return work area instead of display area

Replace rcWork with rcMonitor in GetPrimaryDisplay() function

Should keep size and add visibleSize field.
https://github.com/leanflutter/window_manager/blob/875a617271075c6d30790578b28ea5ea48ea3b5a/macos/Classes/WindowManager.swift#L476-L479

Will do

Todo: Add visiblePosition

Done, I think


multiuser environment

HI Jeremy,
given a multiuser environment on a machine running singularity, I would like to give the possibility to each user to manage their own RStudio . is this possible? i cannot figure out the port settings: each use will use a different one? or is a matter of credential/authentication?
Thanks for any hint..
Cheers
S.-

Since this is run in Singularity it will run as a single user. So anyone who successfully authenticates against this running server will have access to all the files under this user.
One solution would be to start an RStudio Server instance for each user, have them listen on Unix domain sockets (this gets around the port issue), then stand up a reverse proxy in front with a proper authentication mechanism that proxies the logged in user to the correct Unix domain socket.
This can be fairly complicated to set up and the Unix domain sockets are optional as you can just assign a different port to each user's server.


Move object helpers from LandingZoneTaskflowMixin to TaskflowTestMixin

While implementing #1634 I noticed that the usability of make_object() and make_md5_object() is not limited to landingzones tests. These should be moved.

Done.


Add option to skip archiving things in the 'plan' action

This is for cases where 'Plan' action is used in jobs with other actions that can just get the terraform plan from the action output.
It also avoids an 'assume role' which can impede other 'assume role' calls further down in the same job.

Change applied as part of https://github.com/Brightspace/terraform-workflows/pull/51


Stacks CLI to build & publish a Stack

Related to #614
What seems to be the problem?

As part of working on #614, I need to build & publish a Stack from source.
What could Crossplane do to help?

Provide a CLI utility I can use to build a Stack.
What does this look like when it's done?

kubectl plugin command to build a Stack
kubectl plugin command to publish a Stack
docs & examples

I'm working on this in a separate repo, and have an initial set of changes: https://github.com/suskin/crossplane-stack-cli/pull/1
publish is the least tested of the commands so far, but I'm feeling pretty good about the init+ local build flow. I'll be creating a document to describe a quick start with the commands.

I've made good progress on this and have a quick start guide which gives an overview of the commands. It's in the README of the personal repo that I'm currently doing this work in.

The code and initial documentation for build is checked into crossplaneio/crossplane-cli, but publish isn't quite polished enough yet to be considered done.


The case for Symplectic Solvers in ODE Solver

Problem: Conservation properties in certain ODE Solving (or Numerical Integration) schemes
I am working on orbital mechanics problems with Scipy and one issue that I have is linked to energy conservation properties - the IVP Solvers in Scipy are good (particularly DOP853) but they cannot conserve energy and there is a linear energy "leak". Therefore the calculated position diverges from the true position quadratically.
See the linear decrease in energy for the 10 day two-body dynamics problem for an Earth-bound satellite:

You can have a look at this detailed analysis as html here and as jupyter here.
This is just one simple example of a broad field of Hamiltonian Dynamics. Basically, there are a lot of physics problems that require energy conservation properties for the ODE solution. For short propagation runs DOP853 is perfectly fine, but for longer propagations the quadratic increase in error is simply not acceptable.
Therefore I believe the physics community would benefit greatly from a symplectic scheme.
See below for symplectic schemes to solve the same two-body problem above:

Proposed Solution: Let's add Symplectic Scheme support to ODE Solver!
The solution is not that complicated, but requires two steps in increasing complexity and API issues:

Introduction of a symplectic scheme: starting with leapfrog, though a 4th or 6th order scheme is not much more complicated - I have some sample codes to start with if anyone is interested
Introduction of Partitioned or Composite scheme: Partitioned Schemes (e.g. Partitioned Runge-Kutta) divide the problem into parts ("surrogate Hamiltonians"), so practically we solve more than one ODE with more than one solver (e.g. some part solved with a higher order scheme and some part lower).

The first is the minimum requirement. The second will significantly increase the performance of the solver.
I can upload sample codes or discuss the problem in greater detail if I am able to generate some interest. :)
Finally, this is probably a branch of a "not-so-new" discussion in scipy, see: https://github.com/scipy/scipy/pull/6326#issuecomment-386887108
Pinging @astrojuanlu @nmayorov and @rubendibattista, as I think they will be interested.

I think this would be a nice addition. This type of integration also arises from Port-Hamiltonian system simulations and I would really like it if I can use. Discussion of this sort typically starts at the mailing list https://scipy.org/scipylib/mailing-lists.html for SciPy.
The remaining issues are obviously how are we going to maintain it? Is there any bundled high-performance lib that we can use or are we going to roll-out our own etc. Another important issue is how much of the integration problems it will cover. By that I mean if it is working for 50% of the cases then we will get issues about the other 50% since symplectic integration will be officially supported and will create some expectations to work. So edge cases are quite important too.

Thanks for the reply, you think I should start a thread in the mailing list? I'm still new to the scipy community. So, any pointers would be very much welcome.
I don't know many other libraries that offer something similar in functionality, with the exception of JuliaDiffEq. That said, at the risk of stating the obvious, the ODE solver itself is rather inexpensive, but the function evaluations probably are. Therefore, I'm not sure whether a high performance library is actually needed.
Anyway, I'm here for questions and will support this effort any way that I can. :)

Sorry for the delay, I'm running way behind the notifications. Yes, a mailing list entry would be really nice and also helps to harvest more opinions from rather technical audience.

@egemenimre
As far I understand the formulation of Hamiltonian dynamics ODE requires 2 parts:
dp / dt = -dH / dq
dq / dt =  dH / dp

which requires 2 callables or a callable returning 2 vectors. Am I correct?
Does it make sense to implement it within solve_ivp requiring changes to what fun returns? Also do you know how to support "dense output"/event handling capabilities for partitioned schemes? I guess  the general idea is that all solvers in solve_ivp must support these 2 features.
I believe it might be the case to introduce another function like solve_symplectic_ivp.

@mdhaber

Is there any bundled high-performance lib that we can use or are we going to roll-out our own etc.

Boost has some ODE solvers in odeint.  Note that Boost is not currently in scipy, but there are ongoing efforts to find a way to include it.  I am currently working on clang-based Cython wrapper generator (cythonator) to call stats classes in Boost.Math to replace problematic scipy stats functions, but it might be able to be used to generate wrappers for Boost.odeint as well.
The wrapper generator is still very much a WIP, but it is generating good Cython code for the simple Boost.Math distribution classes we are interested in.  I'm happy to give it a go with cythonator or help out with writing Cython wrappers if these ODE solvers are in-line with what you are looking for.

@mdhaber

Is there any bundled high-performance lib that we can use or are we going to roll-out our own etc.

Boost has some ODE solvers in odeint. Note that Boost is not currently in scipy, but there are ongoing efforts to find a way to include it. I am currently working on clang-based Cython wrapper generator (cythonator) to call stats classes in Boost.Math to replace problematic scipy stats functions, but it might be able to be used to generate wrappers for Boost.odeint as well.
The wrapper generator is still very much a WIP, but it is generating good Cython code for the simple Boost.Math distribution classes we are interested in. I'm happy to give it a go with cythonator or help out with writing Cython wrappers if these ODE solvers are in-line with what you are looking for.

I have completely forgotten to reply to this topic. The answer is an empathic yes, Boost ODEINT has support for symplectic and "simple symplectic" systems.
https://www.boost.org/doc/libs/1_66_0/libs/numeric/odeint/doc/html/boost_numeric_odeint/concepts/symplectic_system.html
The n-body problem is a classic Astrodynamics problem where conservation of energy is important (where the symplectic integrators excel).
https://www.boost.org/doc/libs/1_66_0/libs/numeric/odeint/doc/html/boost_numeric_odeint/tutorial/solar_system.html

@egemenimre
As far I understand the formulation of Hamiltonian dynamics ODE requires 2 parts:
dp / dt = -dH / dq
dq / dt =  dH / dp

which requires 2 callables or a callable returning 2 vectors. Am I correct?
Does it make sense to implement it within solve_ivp requiring changes to what fun returns? Also do you know how to support "dense output"/event handling capabilities for partitioned schemes? I guess the general idea is that all solvers in solve_ivp must support these 2 features.
I believe it might be the case to introduce another function like solve_symplectic_ivp.

Apologies for completely forgetting about this thread (life was too chaotic back in Sep)
Yes, it does require two parts, but you integrate them together. For example:
https://en.wikipedia.org/wiki/Leapfrog_integration
So there should be two sets of differential equations (though in many cases dH/dq = p hence trivial). Actually, it is possible to define multiple Hamiltonians where H = H_0 + H_1 + H_2 where separate schemes can be set up to integrate them separately. You can view H_0 and H_1 as the kinetic and potential energy, or H_0 as some of the potential field contributors and H_1 and H_2 as other contributors. An example is a satellite where:
H_0 = 1/2 v^2 - mu / r    (Keplerian or two body term)
H_1 = potential due to Earth oblateness
H_2 = remaining terms of the potential
This enables building composite schemes, where:

one step is integrated taking into account H_0, with a scheme S_0. A result for this step is generated
The result above is the initial condition for H_1, with a scheme S_1. A result for this step is generated
The result above is the initial condition for H_2, with a scheme S_2. A result for this step is generated

It's actually a little bit more complicated than that, but this is how a composite scheme can be built.
For event handling, they don't behave any different than other numerical schemes, as you have a new state at the end of each step. And the event checks for some parameter at the end of that step (e.g. whether RSS of the coordinates equal a certain value).

For what it's worth, I'd be thrilled to see symplectic methods in scipy.

I took a stab at exposing the symplectic ODE solver in boost via a simple to use function. The boost library has a lot of functionality that I boiled down into a function with the following signature
typedef std::vector< double > pq_t;
typedef pq_t (*f_pq2dpq_t)(pq_t);
typedef std::tuple<
    std::vector<pq_t>,
    std::vector<pq_t>,
    std::vector<double>> result;

result solve_ivp_symplectic (
    double dt, double t0, double tn,
    f_pq2dpq_t q2dp, f_pq2dpq_t p2dq,
    pq_t q0, pq_t p0)

The gist with all the code is here
This function can be used with the standard interface in scipy's ivp_solver by using a helper function like this, but this wrapper will waste half of its compute, so it might be better to actually bite the bullet and expose the more general API . The con will be that it will not match the standard ODE api, but it will be twice as fast.
def solve_ivp_symplectic_cython(dy, t_span, y0, dt, ...):
     dim = y0.shape[0]
     assert dim % 2 == 0
     dim_pq = dim // 2
     def f_dqdt(p):
           y = [0,..., 0, *p]
           return dy(y)[:dim_pq]

    def f_dpdt(q):
         y = [*q, 0, ..., 0]
         return dy(y)[dim_pq:]

    # call the cpp wrapper
    return solve_ivp_symplectic(dt, *t_span, f_dpdt, f_dqdt, y0[:dim_pq], y0[dim_pq:])

In the following demo-image I was trying to solve a simple system, whose solution should be a circle in the p,q space.
dq/dt = p
dp/dt = -q

The solution is q(t) = r cos(a + t), p(t) = r sin(a + t), where r and a are determined by the initial value.  and we can see that the symplectic system performs the best with RK45 a very close-second. Ideally the trajectory should be a perfect circle, and RK45 makes a circle that's a little bit thicker than the symplectic integrator.

The Julia community has done amazing work to develop their ODE solver package, lots of inspiration to be found there. In particular in the "GeometricIntegrator" package when it comes to symplectic solvers. https://github.com/JuliaGNI/GeometricIntegrators.jl
For what it's worth, I've recently written explicit and implicit RK solver for partitioned Hamiltonian systems of ODE in Cython. It does not have many features as is, but was written with performance in mind (linked with BLAS for the dense linear algebra for instance)
The code is far from perfect, but I'm still leaving this here (mainly because I'd like this discussion to gain some traction!). I'd be thrilled to answer any question anyone might have regarding it.
https://github.com/gabrielfougeron/choreo/blob/main/choreo/Choreo_scipy_plus_ODE.py
https://github.com/gabrielfougeron/choreo/blob/main/choreo/Choreo_cython_scipy_plus_ODE.pyx


maureen solution

I had trouble pushing my git branch, so I had to push it without branching. After attempting to make this game many times I decided to study the solution and finally understood how it's done. However, I don't understand why "this" needs to be used on line 28 (typing in my variable "square" instead of "this" prevents the game from working). I figured that maybe it only works when I type "this" because the variable "square" is inside of an anonymous function.
I also attempted to do the bonus question but had too much getting the brushes to change color according to the input.

Inside event listener functions, this always refers to the element that was clicked on. If you already have that element saved to a variable, you can use that variable instead: either is fine.


PR: Spydata file loading now uncompress files inside a temp folder

Fixes #3917
This error was caused due to write permission restrictions inside a folder. Now the files are uncompressed in a temporal folder, which allows to bypass any write restriction present in the directory where the spydata file is located.

@ccordoba12 This PR is ready to go!

Very good job @andfoy!! Specially for noticing that we needed to uncompress spydata files in a temp dir in order to load them :-)
Just a recommendation for next time: please try to avoid so many merges, or in case you do them, please squash them at the end ;-)


Extended Type Assertions

Addresses Issue #271

Now that's what I call a pull request ;-)
But on a serious not, well done. Thanks for that. I'll have to look at those changes a bit more and clean-up the code a bit, so it'll probably take me a couple of days to merge the changes.
Any change you could provide me with an updated version of that section in the wiki so that I can copy it over as soon as 3.5 is released?

Dennis, take a lo K at the associates issue. Is that good for the wiki?

Should be good enough. Thanks.


Initial mecha parser

:)

This needs testing with many different kind of messages. It is not clear if all tags are always available in a PRIVMSG. I does seem like they should be, as tag values just tend to be empty instead of not present.

@kristoff-it Haven't tested this latest rebase (partly because I cannot render emotes), but this should mimic the behavior of your emote parsing

Thank you for the rebase work, much appreciated


make press release to fix consept. and send press release for game developper.

~- write below https://docs.google.com/document/d/1N9LyDrt3VTRez2JtOlnvpTXl_UNyU8sdtORXlrmyQA4/edit~
~- refer below article. 穴埋めで作る、インディゲーム開発者のための簡単なプレスリリースの作り https://note.com/gamecast/n/nae90d756bf04~
~- reflect comments.~
~- send when to send is best.~
~- make list to send press release. https://note.com/akutaba/n/n029b58128250 https://vr-tips.lipronext.com/vr_content/vr-media/~
~- make press release page.~
~- make zip file include image, word and pdf.~
~- make sentence to send to form.~
~- write up game review.~
~- change cover image and repost the article.~

send press release to form.

sent except foreign media.


[Fix #12926] Fix false positive for Style/SuperArguments

Fixes #12926. The following snippet demonstrates the issue:
class A
  def positional_arg(a)
    puts a
  end

  def block_arg(&block)
    yield
  end
end

class B < A
  def positional_arg(a = nil)
    a = 'b'
    super
  end

  def block_arg(&block)
    block = proc { puts 'b' }
    super
  end
end

B.new.positional_arg('a')
B.new.positional_arg
B.new.block_arg { puts 'a' }
B.new.block_arg

Modifying a methods block argument passes along the original block to super.
This fix is overly defensive and will miss some cases that could be autocorrected. This however would make this very complicated:
def foo(&blk)
  super
  blk = proc {} # Technically fine
end

The same goes for nested method definitions

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] 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.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Probably this should be mentioned in the cop's docs, so people are not surprised by this conservative behavior.

Ok, sure. I've also added a link to the ruby bugtracker issue I made

Thanks!


Update the Czech localization - cs_cz.json

Updated the cs_cz localization to not look like ChatGPT merged with Google Translate (which it was)

Thank you! Merging now


RuntimeHandler support

As part of the new Kubernetes RuntimeClass feature, we're adding a new field to the CRI's RunPodSandboxRequest: runtime_handler (https://github.com/kubernetes/kubernetes/pull/67518).
As discussed in the proposal (under RuntimeHandler), this new field is designed to select between multiple "handler" configurations, and deprecate the binary "trusted" and "untrusted" runtime selection.
In order to support this new feature, the following items need to be addressed:

[x] Map the runtime_handler field to a runtime configuration, deprecating the io.kubernetes.cri.untrusted-workload annotation.
[x] Reject requests that cannot be mapped to a valid runtime configuration
[x] (optional) Support multiple (non-binary) named runtime configurations
[ ] Integration tests

Proposal

Deprecate plugins.cri.containerd.untrusted_workload_runtime. If it is provided, give it the implicit configuration name of untrusted.

(optional) Deprecate the DefaultRuntime field as well, favoring a runtime with the name of default or ''

Introduce a new ContainerdConfig field: Runtimes map[string]Runtime

Specifying both the untrusted_workload_runtime and a Runtimes['untrusted'] is a configuration error (ditto for default, if we go that route).

Modify criService.getSandboxRuntime to take the runtime handler as a parameter, and map it to the correct runtime config.

Return an error if it cannot be mapped.
If both the untrusted annotation & runtime_handler are specified, prefer the runtime handler EDIT: return an error

If this proposal sounds good, I'm happy to open a PR.

+1 for this proposal. This is tightly related with kata shimv2 integration as well.
We may also need  a item about refactoring the CRI and e2e tests.  plugins.cri.containerd.untrusted_workload_runtime is referred in the tests for quite a while.

It might be hard to add an integration test, unless we can have another runtime running on the node.

First 3 items are done. Integration test is harder to add now without a test alternative runtime. Punt to next release.

We are running runtimeclass test for containerd in cluster e2e now https://k8s-testgrid.appspot.com/sig-node-containerd#e2e-gci
I think we can close this one.


Unwanted tags in Obsidian Vault

Followed the tuto from Nicole Van der  Hoeven. In the obsidian Vault I created from the quartz folder I'm getting some unwanted tags (see picture, only the people tag is from me).

Guess it's related to the fact that the quartz repositories include some .md files with some # in it.
Example path/to/vault/node_modules/chalk/readme.md containing "...will downsample an RGB value of #FF0000 ..."
Any idea on how to solve this issue ?
Thx in advance for your help.
By the way congrats for the job done with Quartz !

Did you create a vault from the quartz folder? You should create a vault from the content folder inside the quartz folder.
The problem you are experiencing now is because the npm modules that quartz depend on are included in your vault. Changing the vault to the content folder should fix this issue.

Thx Very much for the answer. and the reactivity !


Fix build

Closes #{put_issue_number_here}
Subject

Codecov Report

:exclamation: No coverage uploaded for pull request base (1.0.x@35c3433). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             1.0.x       #1   +/-   ##
========================================
  Coverage         ?   64.70%
  Complexity       ?       73
========================================
  Files            ?       10
  Lines            ?      255
  Branches         ?        0
========================================
  Hits             ?      165
  Misses           ?       90
  Partials         ?        0

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 35c3433...a900fda. Read the comment docs.


Added editor page

This doesn't include the shell script for bower updates yet. This is a pretty large PR due to the new inclusion of popcorn-editor

Tests pass :fireworks:

Next time we'll have to be a bit more anal about the commit message so that we can trace this back to a bug.


Porting to JJazzLab

Just FYI, I will try to port your walking-bass plugin to JJazzLab -just need to find time ;-)
My goal is to initiate a new JJazzLab plugin dedicated to swing-style backing track generation, eventually using more advanced algorithms and AI.

Thanks for the heads up, Jerome! Let me know if you have any questions on the code.
Just to get it out of the way, according to the License, you are free to use the code as long as the copyright statement is included, and I'd request an attribution be made in the "About" box.  I look forward to seeing the result!


Kaprekar number

Hey @ming-tsai ,
I have added kaprekar number program
Please review it
closes: #240
Thanks

Hey @cyberwizard1001 please review it

@cyberwizard1001 check it if there any mistake then tell me
Thanks

Algorithm checks out, can be merged, @ming-tsai


FYP: filter deleted content?

claim_id:  d5eac2b2c5995ad60a87ea65ef7852bb3ace5f02

Can filter at app side, but would it be more efficient if FYP does it?

Believe Johnny has this on his list..doesn't happen too often, but should be handled there. Can't hurt for us to improve if it's quick also


docs: Correct typo in documentation

Contributor checklist

[ ] Bug fixes include regression tests
[ ] Features include unit/acceptance tests

🚀 Thank you for your contribution! 🚀


2024-05-28_01-31-31_master-next_aws-iot-device-sdk-python-v2

Automatically created.

/run

/run


[Bug] Error when clicking on a composite traintuple with status failed

Setup
Local deployement of Substra using skaffold
Latest commit on master (tag: 0.0.17)
Description of the bug
On the models page: http://substra-frontend.node-1.com/model
I have a composite traintuple with the status 'failed'. When trying to click on it, the page becomes entirely blank.
Opening the dev tools gives me this error:
react-12ba27.js:2 TypeError: Cannot read property 'hash' of null

This happens with two different failed composite traintuples, no problem with failed traintuples and aggregatetuples, nor with 'done' tuple.
The error seems to come from this:

In case of failure, outHeadModel.outModel and outTrunkModel.outModel are null instead of something like this: outModel: {hash: "77ac6ae57ef74bc465a676e584c71a4d6f0ae083a5bb29e4a532a38e77355732"}

This has been fixed in #61 :)


Remove outdated wiki content

Related Ticket(s)
#1237
Description
Changelog
Removed

folder: Design-guideline-drafts/
folder: Design-office-hours/
file: wiki/Resources.md

@shixiedesign Let me put a PR together to get the component page links updated and pointing to the new content guidelines section. Just a jiffy!


Make sure times are rounded when track event moves

Also, the _round function is now a static function on Butter, named
round. numberOfDecimalPlaces is a static field on Butter as well, and
controls how many decimal places Butter.round rounds to.

This ticket got resolved yesterday, closing this.
https://webmademovies.lighthouseapp.com/projects/65733-butter/tickets/256-timecodes-should-round-to-1-decimal


make date-fns version strict in package.json

[x] I have changed my target branch to develop :facepunch:

Closes #
Description
Continuation of #418

@cherniavskii  I will not deploy new version of pickers now. And I will be available on 27.05. So everything can be stacked.
If it will be some breaking change from material-ui side feel free to publish repository fork to npm and mention that in the readme :)

@dmtrKovalenko can you at least deploy docs now?

@cherniavskii OK Im deploying now


Add Dockerfile

I saw #3 and figured it wouldn't be hard to add a simple Dockerfile to this project. Is this something you want in this repo?

Honestly, I personally hate Docker and don't want it, but I guess we should accept this PR because you had all this trouble.

Don't worry about me, no need to merge this if you don't want it in the codebase.

@0xtrr can you update the Dockerfile to include a command to build the Tailwind CSS bundle?
This is the command:
tailwind -i tailwind.css -o static/tailwind-bundle.min.css --minify

But you also have to install the Tailwind CLI.

@0xtrr can you update the Dockerfile to include a command to build the Tailwind CSS bundle?
This is the command:
tailwind -i tailwind.css -o static/tailwind-bundle.min.css --minify

But you also have to install the Tailwind CLI.

https://github.com/fiatjaf/njump/pull/9


readability of single coice lists with long texts

I need to display a single choice list with 5 items. The text of the items is rather long -- 4 to 7 lines on my Motorola Moto E. The radio buttons are vertically centered on the text. (android:gravity="center_vertical" of the radio buttons in res/layout/md_listitem_singlechoice.xml). As a result  it is very hard for the user to understand where one item begins and the next one ends.
It would be a great help, if I could configure the gravity or if it was TOP by default.

This doesn't sound like it's really related to this library, as much as trying to do too much in a dialog.
You could use a custom list adapter that modifies text size and gravity, though.

Hey Aidan,
Thanks for looking at this. I could do this using a custom list adapter and a custom view, but it would be so much easier for me if you'd put android:gravity="top"  instead of android:gravity="center_vertical" on the radio buttons in res/layout/md_listitem_singlechoice.xml. I don't think there would be an impact on existing views, which have only one line of text per item.
Why do you think I'm trying to do too much in a dialog. What option would you recommend?
Thanks,
Navid

The current gravity is center_vertical because that's what stock dialogs use, and that's what the Material guidelines specify (which is the purpose of this library).
If items are too long for a dialog, you could put them in a basic Activity that displays a list and uses Activity results.


Check if the passed in node is a LocalProjectTreeItem

Fixes https://github.com/microsoft/vscode-azurefunctions/issues/3369
Because of this change, the Local Project tree item will get passed in to deploy which is what's causing the error.
I think when we use the new wrapper, since we pass in node and nodes , it might be easier for us to tell when a node was right-clicked vs. just selected, though I'm not sure how VS Code passes us those args yet.
I assume that once that changes, we can change how we do this check.

Wow. I can repro the Cannot read properties of undefined (reading 'createClient') error, but not the isLinux one. Do you know how to repro that?

I haven't been able to repro it either. Still kinda poking around, but really not sure how it's happening. What's crazy is once you have selected something, even if you unselect it, it still passes that tree item in. Super annoying. Only way to unselect something is to reload the window.

Only way to unselect something is to reload the window.

I noticed the same thing. Weird.


Confusing text in instructions for findAlphabet question

https://github.com/google/tie/blob/master/assets/questions/findAlphabet.js#L232
The word 'frequency' on this line and 'return them in alphabetical order' are both confusing. The first one is not related to the question, and the second might confuse the user as to which alphabetical ordering we are asking for.
It might make more sense to change the sentence to something like "In case there is not enough information to order multiple characters, return them in the conventional alphabetical order".

Fixed in #254, thanks @eyurko!


Add welcome banner before parcel begins

Talking to @eriktrautman today, we want to make sure every developer knows what to expect after running the commands posted on the homepage. At the time of this writing, those commands are simply:
# Drop this in your CLI and run it
npx create-near-app your-awesome-project
cd your-awesome-project
yarn
yarn dev

Folks unfamiliar with local web development may be unfamiliar with opening links to a localhost site. Furthermore, those same folks may think that the script is still running since they don't see their normal terminal return to the expected prompt.
Solution: let's add some banner thats conveys that they've successfully got a NEAR app running and to open the localhost link in their browser of choice.

Solution: let's add some banner thats conveys that they've successfully got a NEAR app running and to open the localhost link in their browser of choice.

there is currently such message:
Server running at http://localhost:1234
✨  Built in 23.36s.

I'd argue if you cannot figure out what's going on (server got started, build completed), no kind of banner would help (much deeper education required).
As for opening browser, it's trivial to do with Parcel:
https://parceljs.org/cli.html#open-in-browser

Laughed out loud. I don't think this is easy for everyone. From the Developer eXperience standpoint, it's our job to make this stuff super easy for folks. For example, we want CS 101 folks to be able to pick this up. Their classes are all about C++, not web. They may have never seen their Terminal "hang" like this before.
I'd hate to turn them away saying, "Sorry, Erik, you're just not cut out for this."

While it's not high priority, it's also not high effort and can only help.

For example, we want CS 101 folks to be able to pick this up. Their classes are all about C++, not web.

My point is that they won't be able to pick our stuff up by seeing snippet and copy-pasting it. We won't help much by showing more prominent banner with the stuff that already is displayed.
Impatient web devs and people just learning CS should be directed to different entry points altogether. Like right now if you even want to go through tutorials in docs (https://docs.nearprotocol.com/docs/tutorials/zero-to-hero) you need to have far more understanding of web stuff than CS 101. We don't even point folks to any resources allowing to learn basic web concepts required. So if we are any serious about this – effort has to be much bigger than a banner here.

I've checked create-react-app and I think it makes sense to do same stuff:

install dependencies automatically
show some basic instructions in console

https://github.com/nearprotocol/create-near-app/pull/186

Linked PR looks good to me, feel free to merge and close. Or if there are other improvements, no rush

create-near-app 0.9.1 released


Importing OpenAPI without type causes requestBody to be null

Describe the bug
If type is not specified as part of the requestBody schema it will be null when importing an OpenAPI yaml-file.
To Reproduce
This file reproduces the bug:
openapi: 3.0.1
info:
  title: My API
  version: '1.0'
paths:
  /pet1:
    post:
      requestBody:
        content:
          application/json:
            schema:
              type: object
              properties:
                petId:
                  type: string
  /pet2:
    post:
      requestBody:
        content:
          application/json:
            schema:
              properties:
                petId:
                  type: string

pet1 will be imported correctly, but not pet2, see images below.
Expected behavior
Both pet1 and pet2 should have a JSON structure in the requestBody after import.
Screenshots

Desktop (please complete the following information):

OS: Windows 10.0.19042
Installation Method: Installer
App Version 2020.5.2

Additional context
I tried checking the OpenAPI specification in order to make sure if this was a required parameter or not. My interpretation is that type is optional something since it is not marked with REQUIRED, in the specification, https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.3.md#schema-object
I will admit that I am not absolutely sure on this, but all other parsers that I tried this in works as expected.

This problem still exists on the latest version, tested with
Version: Insomnia 2021.3.0
Release date: 4/28/2021
OS: Windows_NT x64 10.0.19042
Electron: 9.3.5
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.73.0-DEV OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.9 WinIDN libssh2/1.9.0_DEV nghttp2/1.41.0


Canvas.drawAtlas with blendMode has odd behavior

Steps to Reproduce

take coast.jpg sample image from examples/image_list app
run the following code:

drawAtlas() async {
  final data = await File('coast.jpg').readAsBytes();
  final image = await decodeImageFromList(data.buffer.asUint8List());

  final recorder = ui.PictureRecorder();
  final canvas = Canvas(recorder);
  double y = 0;
  for (final blendMode in BlendMode.values) {
    canvas.drawAtlas(
      image,
      [RSTransform(1, 0, 0, y)],
      const [Rect.fromLTWH(300, 300, 100, 100)],
      [Colors.green],
      blendMode,
      null,
      Paint(),
    );
    canvas.drawAtlas(
      image,
      [RSTransform(1, 0, 100, y)],
      const [Rect.fromLTWH(300, 300, 100, 100)],
      null,
      null,
      null,
      Paint()
        ..colorFilter = ui.ColorFilter.mode(Colors.green, blendMode),
    );
    TextPainter(text: TextSpan(text: blendMode.toString(), style: TextStyle(color: Colors.black)), textDirection: TextDirection.ltr)
      ..layout()
      ..paint(canvas, Offset(0, y));
    y += 100;
  }

  final outImage = await recorder.endRecording().toImage(200, y.ceil());
  final outData = await outImage.toByteData(format: ui.ImageByteFormat.png);
  await File('coast_atlas.png').writeAsBytes(outData!.buffer.asUint8List());
  print('done');
}

and the output coast_atlas.png looks like:

Expected results:
both images in each row should be the same
Actual results:
only few of them look the same, for example: plus / modulate / screen / darken / lighten / difference / exclusion / multiply
Flutter doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 2.10.0-1.0.pre.1, on LMDE 4 (debbie) 4.19.0-8-amd64, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 4.0)
[✓] Android Studio (version 3.4)
[✓] Android Studio (version 3.6)
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Community Edition (version 2017.1)
[✓] VS Code (version 1.62.0)
[✓] Connected device (2 available)
[✓] HTTP Host Availability
• No issues found!

Hi @pskink, Thanks for filing the issue. I tried to render images as per your code sample but I see a blank screen. Could you please share a complete reproducible code sample?

flutter doctor -v
import 'dart:io';
import 'dart:ui' as ui;
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(home: MyHomePage());
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Column(
        children: [
          CustomPaint(
            painter: ImagePainter(),
          )
        ],
      ),
    );
  }
}

// FOR PAINTING POLYGONS
class ImagePainter extends CustomPainter {
  @override
  void paint(Canvas canvas, Size size) {
    drawAtlas();
  }

  @override
  bool shouldRepaint(CustomPainter oldDelegate) {
    return true;
  }

  drawAtlas() async {
    final data = await File('/Users/mahesh/Downloads/coast.jpeg').readAsBytes();
    final image = await decodeImageFromList(data.buffer.asUint8List());

    final recorder = ui.PictureRecorder();
    final canvas = Canvas(recorder);
    double y = 0;
    for (final blendMode in BlendMode.values) {
      canvas.drawAtlas(
        image,
        [RSTransform(1, 0, 0, y)],
        const [Rect.fromLTWH(300, 300, 100, 100)],
        [Colors.green],
        blendMode,
        null,
        Paint(),
      );
      canvas.drawAtlas(
        image,
        [RSTransform(1, 0, 100, y)],
        const [Rect.fromLTWH(300, 300, 100, 100)],
        null,
        null,
        null,
        Paint()..colorFilter = ui.ColorFilter.mode(Colors.green, blendMode),
      );
      TextPainter(
          text: TextSpan(
              text: blendMode.toString(),
              style: const TextStyle(color: Colors.black)),
          textDirection: TextDirection.ltr)
        ..layout()
        ..paint(canvas, Offset(0, y));
      y += 100;
    }

    final outImage = await recorder.endRecording().toImage(200, y.ceil());
    final outData = await outImage.toByteData(format: ui.ImageByteFormat.png);
    await File('/Users/mahesh/Downloads/coast.jpeg')
        .writeAsBytes(outData!.buffer.asUint8List());
  }
}

thanks

you need a different output file, for example:
     await File('/Users/mahesh/Downloads/coast_output.pmg')
        .writeAsBytes(outData!.buffer.asUint8List());

Ah, I see thanks, I was able to reproduce the issue. I am seeing the same output behavior on the stable and the master channel.  I am not sure about the expected output labeling this issue for further insights from the team.

Author's Output
Triage Stable
Triage Master

flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.8.1 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (3 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.32.0

[✓] Connected device (4 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5555                     • android-arm64  • Android 11 (API 30)
    • iPhone 13 Pro (mobile) • 0FA2FB93-2A89-4FDB-8293-DF66188F1E54 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-0
      (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 96.0.4664.110

• No issues found!

[✓] Flutter (Channel master, 2.9.0-1.0.pre.427, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.9.0-1.0.pre.427 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6eb3400dc1 (4 days ago), 2022-01-20 20:25:16 -0500
    • Engine revision cd65368f9d
    • Dart version 2.17.0 (build 2.17.0-31.0.dev)
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.32.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 97.0.4692.99
    ! Device <IP_ADDRESS>:5555 is offline.

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

code sample
import 'dart:io';
import 'dart:ui' as ui;
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(home: MyHomePage());
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  drawAtlas() async {
    final data = await File('/Users/mahesh/Downloads/coast.jpeg').readAsBytes();
    final image = await decodeImageFromList(data.buffer.asUint8List());

    final recorder = ui.PictureRecorder();
    final canvas = Canvas(recorder);
    double y = 0;
    for (final blendMode in BlendMode.values) {
      canvas.drawAtlas(
        image,
        [RSTransform(1, 0, 0, y)],
        const [Rect.fromLTWH(300, 300, 100, 100)],
        [Colors.green],
        blendMode,
        null,
        Paint(),
      );
      canvas.drawAtlas(
        image,
        [RSTransform(1, 0, 100, y)],
        const [Rect.fromLTWH(300, 300, 100, 100)],
        null,
        null,
        null,
        Paint()..colorFilter = ui.ColorFilter.mode(Colors.green, blendMode),
      );
      TextPainter(
          text: TextSpan(
              text: blendMode.toString(),
              style: const TextStyle(color: Colors.black)),
          textDirection: TextDirection.ltr)
        ..layout()
        ..paint(canvas, Offset(0, y));
      y += 100;
    }

    final outImage = await recorder.endRecording().toImage(200, y.ceil());
    final outData = await outImage.toByteData(format: ui.ImageByteFormat.png);
    await File('/Users/mahesh/Downloads/coast_output.jpeg')
        .writeAsBytes(outData!.buffer.asUint8List());
  }

  @override
  Widget build(BuildContext context) {
    drawAtlas();
    return Scaffold(body: Container());
  }
}

This is working as intended:
With per atlas colors, the image is the src and the color is the dst (see see https://api.flutter.dev/flutter/dart-ui/Canvas/drawAtlas.html)
With the ColorFilter the color is the src and the image is the dst (see https://api.flutter.dev/flutter/dart-ui/ColorFilter/ColorFilter.mode.html)


Revert "Update proto files for pagination (#2138)"

This reverts commit 0056320f7060d1fbc101dd735c24cdd0bc247b4b.

The reverted commit was incorrect due to using an old version of google.golang.org/protobuf


[OC MVP] Lease details missing port numbers

Describe the bug
To Reproduce
Steps to reproduce the behavior:

Deploy an RPC node with Console
Check lease details - notice no port numbers listed
Repeat same with CLI or Cloudmos

Expected behavior
Expect to see port numbers in lease details the way we do with cloudmos and CLI
Screenshots

Assigned P1 because we have heard feedback from users about this


Misc cleanups

These are just a few random minor changes, intended to make things a bit easier to maintain. Hopefully nothing controversial.

@mortoray: Good point, removed that PNG.
We already have targeted tests for shape and brush, and the SdfImage-implementation was actually somewhat buggy (it interpreted the stroke-width wrong, and drew it twice as wide, hence the corresponding change to make it look the same).


Add changelog and bump version for proposed 4.0.1 initial bi-weekly release

re https://github.com/apache/arrow-rs/issues/292
NOTE: This PR targets the active_release branch, NOT master
Rationale:
Begin preparing for the first bi weekly release
Changes:

Add changelog (see rendered version)
I made some manual edits to the changelog while reviewing them. I left the touchups in a separate commit
Update version to 4.0.1

Questions

Should we call this 4.1.0 (there are some "featurettes" and new APIs such as https://github.com/apache/arrow-rs/issues/25 and #210)?

Notes:

Changelog was made using the code from https://github.com/apache/arrow-rs/pull/289 (thanks @jorge!) via CHANGELOG_GITHUB_TOKEN=$ARROW_GITHUB_API_TOKEN ./change_log.sh

Verson was updated using (on a Mac):
sed -i '' -e 's/5.0.0-SNAPSHOT/4.0.1/g' `find . -name 'Cargo.toml'`

Codecov Report

Merging #298 (ff86a0e) into active_release (3665296) will decrease coverage by 0.00%.
The diff coverage is n/a.

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

@@                Coverage Diff                 @@
##           active_release     #298      +/-   ##
==================================================
- Coverage           82.49%   82.49%   -0.01%
==================================================
  Files                 162      162
  Lines               43980    43980
==================================================
- Hits                36283    36282       -1
- Misses               7697     7698       +1

Impacted Files
Coverage Δ

parquet/src/encodings/encoding.rs
94.85% <0.00%> (-0.20%)
:arrow_down:

Continue to review full report at Codecov.

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

Try again in #305


Fix LOD dithering artifacts

Fixes #1388, #1359
This PR modifies the ML_DitherFade material layer to prevent some flickering pixels that could be seen on tilesets. These were most visible on Google Photorealistic 3D Tiles, and especially when the tileset was selected. You can see it in 12_CesiumGoogleMapsTiles behind the intro billboard, or check out #1388 for an example.
Before

After

I will also test if this addresses #1104, TBD.

Unfortunately this does not address #1104. ☹

Code looks good, and this is working well, no matter if LOD transitions are enabled or disabled. Merging.


Add better hash URL support.

This PR comes with following enhancements related to hash url changes.

Add scrolling to given id related to hash
Hash changes won't trigger getInitialProps

@rauchg take this now.


Add issue template for new learn.microsoft.com feedback control

Adds a dedicated issue template for reporting problems on learn.microsoft.com-hosted API ref pages and READMEs.
Current experience:
If a customer wants to report an issue on this page, they click the This page button:

After clicking, the doc's metadata is added to the issue description:

Future experience:
This new feedback control will replace the existing one:

When clicking Open a documentation issue, the new issue template added in this PR will populate with the appropriate metadata. One benefit will be that such issues are created in this engineering repo, where the content lives.

/check-enforcer override


Modernize bindings in Todos sample app

@filblue Thanks for taking the time to explore and share this. We've actually gotten a PR for this before, but opted not to take this change:
https://github.com/pointfreeco/swift-composable-architecture/pull/784#discussion_r708677611
Because ViewState and ViewAction makes things more complex, we think it's better to keep things simple as a demo. I think we'll probably want to develop an article on best practices for bindable state and actions in the future, though, for guidance here.

@stephencelis That's great! I look forward to seeing the new tools. But please keep in mind that not everyone will be on iOS 16. We, in particular, are supporting iOS 14 and above. It would be great if the new navigation tools don't depend on the new NavigationStack APIs.


⚠ WSINGRESOS - OXXO has degraded performance

In cb15713, WSINGRESOS - OXXO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/compartido/wspagosoxxo.r) experienced degraded performance:

HTTP code: 200
Response time: 2480 ms

Resolved: WSINGRESOS - OXXO performance has improved in 5146f46 after 21 minutes.


MORE events

Closes #112 #159 #153 and #138

This doesn't' actually close 112. I thought there'd be a system that listens for construction events. If that's too much of a pain, then do what you did, but it isn't ideal.

also i feel like the math challenge should be a single challenge which creates a math problem for the players to solve. having a bunch of static ones will mess up the challenge pool in terms of probability (plus it's kinda lame)

also i feel like the math challenge should be a single challenge which creates a math problem for the players to solve. having a bunch of static ones will mess up the challenge pool in terms of probability (plus it's kinda lame)

do you have a method of sending text to individual (or all) clients without sending the actual locale string

also i feel like the math challenge should be a single challenge which creates a math problem for the players to solve. having a bunch of static ones will mess up the challenge pool in terms of probability (plus it's kinda lame)

do you have a method of modifying the text sent to individual (or all) clients

it could be the same question for every client, but just have it randomize the answer (c+p sayphrasemodifiersystem)

also you could just put it in chat or use the overlay.

i just fucking realized

god i'm so sleep deprived rn

it could be the same question for every client, but just have it randomize the answer

what

did most of it, merge when you approve


ARXIVNG-875 add form field and validation for policy acknowledgement

This PR creates the form field for policy acknowledgement and adds a simple yes/no verification on the field, raising an error in the event that the box is not checked.  Pretty sure this is in line with the state of where the previous pages are.  Some of this was done a bit blindly -- followed the pattern for event creation from authorship and verify_user.
Note: I noticed that the handy progress bar links aren't going to work any more due to not having submission_id passed and also we don't want to have the skip forward...took a short stab at how that might work but ended up aborting the effort for now. It's a separate task from this PR and possibly out of scope for the sprint.

Pull Request Test Coverage Report for Build 53

0 of 13 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 1.487%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

submit/controllers/policy.py
0
6
0.0%

submit/routes/ui.py
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

submit/controllers/policy.py
1
0.0%

Totals

Change from base Build 51:
-0.05%

Covered Lines:
4

Relevant Lines:
269

💛  - Coveralls

Ok. Let's not let the method question block this PR, but we should migrate
to POST for all views.  Maybe a
separate small ticket (god help us)
On Wed, Jun 13, 2018, 2:52 PM eawoods<EMAIL_ADDRESS>wrote:

@eawoods commented on this pull request.
In submit/templates/submit/policy.html
https://github.com/cul-it/arxiv-submission-ui/pull/6#discussion_r195197509
:

@@ -14,16 +14,19 @@ Acknowledge policy statement

For third-party submissions, I have obtained pre-authorization from arXiv
     to submit as a third-party submitter.

-
+
The previous pages are GET...not sure why we chose that after the first
one. I suppose if we used POST then we wouldn't need to send the
submission_id...right?
Tried POST quickly and current configuration returns a 405. Something to
talk about for all pages?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/cul-it/arxiv-submission-ui/pull/6#discussion_r195197509,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSqysYpClcxD7uaQ6jaGRioBlpMQvWnks5t8V9VgaJpZM4UmYCB
.


Weird screen output behaviour: alive and analyse "forgotten"?

During the big colliding flow simulation, I observed weird behavior on screen:
It should dislpay every 100 timesteps as can be seen from 4000 upwards, however for the 5000 upwards: some are missing?

In another simulation, it just forgot to disaply the analyse at 6000 time steps:

I have no idea what is going on here...but I wanted to at least report it here, maybe it is a bug.

Hmm, weird... Which command did you run to get this output? Is it reproducible on your system?

 trixi_include("examples/tree_2d_dgsem/elixir_euler_colliding_flow_amr.jl", tspan=(0.0, 25.0))
It is reproducible in the sense that it happens occasionally and most likely in every run it seems. It is not reproducible in the sense that starting the same simulation two times in a row, it does not behave the same - it might show up, or not!?
Very weird.

It's not a bug in the strict sense. With error-based step size control, the integrator can reject steps whenever the error estimate is far off. In that case, the step is repeated before callbacks are activated. Thus, a callback might not be called for a specific step number.

There are three things we could do here.

Ignore that your output is missing (maybe add it to the FAQ/troubleshooting section in the docs).
Add additional logic that remembers whether there was an analysis output at the last desired time step and run analysis if not.
Change the interface of the analysis callback etc. to print results based on desired simulation times and not time step counts.

Hm, I see. Is it not possible to only count the real number of time steps, i.e., the time step that really advances the solution?
(Regarding your suggestion: option 3 is the one that should be in Trixi from the start and is something imo we should put in asap)

(Regarding your suggestion: option 3 is the one that should be in Trixi from the start and is something imo we should put in asap)

I put it on our TODO list for Trixi.jl v0.4


Touch ripple effect

Should be looking nice on touchscreens
Like https://www.shadertoy.com/view/ll2GRD but with decay

ok, i did the base stuff
https://github.com/57r31/matrix
run with effect=ripples

I'll take a look tonight! Also any feedback you may have on the ability to make changes to this project is valuable in its own right 😊

Short version: I developed your proof of concept into a solid feature, with ripples propagating out from the click position. Up to five simultaneous ripples at a time. Adjusted their aspect ratio to appear circular, and also gave the glyphs a tint similar to what's in the Resurrections shot, which brighten slightly in intensity where the ripples are. Also added webcam support, but that's a separate opt-in so no one is caught off-guard when the browser requests camera access.
Since a different kind of ripple already exists in the project, I think I'll rename the effect "mirror".
I'll upload what I've got later tonight and link you to it. I'll also add an Attribution section to the readme, where contributors' names go. If that's all right with you, just let me know what moniker you'd like to be represented as.

Mirror effect
Mirror effect with camera

wow nice, thanks!


Tensor.to(Tensor) doesn't change dtype of Tensor.

PyTorch outputs torch.float64
import torch
x = torch.linspace(0, 1, 100, dtype=torch.float32)
y = torch.linspace(0, 1, 100, dtype=torch.float64)
x.to(y).dtype

TorchSharp (8005af93ef3ebf66f3b246a4ccbe4ba791f7f9f9) outputs Float32.
// commit 8005af93ef3ebf66f3b246a4ccbe4ba791f7f9f9
using TorchSharp;
var x = torch.linspace(0, 1, 100, dtype: torch.float32);
var y = torch.linspace(0, 1, 100, dtype: torch.float64);
x.to(y).dtype

TorchSharp provides Tensor.type_as(Tensor) to change Tensor.dtype, however, this difference from PyTorch is confusing when poirting from PyTorch. Also in many case, cast is used to apply operators to two tensors which dtype and device of them need to be compatible.

@kaiidams -- this will be fixed in the next release (which I will release today).


[WIP] Deluge client switch

Motivation for changes:
Stop requiring a deluge install to connect to deluge daemon. This allows easier configuration on flexget installs which connect to remote deluge daemons, as well as python 3 support in the deluge plugin.
Detailed changes:

Switch to https://github.com/JohnDoee/deluge-client as deluge client library.

To Do:

[ ] Test it more.
[ ] Implement get_localhost_auth ourselves?
[x] Autodetect deluge version? https://github.com/JohnDoee/deluge-client/issues/11
[ ] Code can probably be cleaned up a bit more, mostly just a copy/paste from the old callback version still.

https://github.com/JohnDoee/deluge-client/issues/5 and other timeout issues might cause problems but will see if I can't get that working properly while adding auto-detection.

I made something like get_localhost_auth already, the resolution should be very similar to the existing Deluge.
https://github.com/JohnDoee/autotorrent/blob/develop/autotorrent/clients/deluge.py#L55-L96

Tried this on MacOs 10.13 with no success.
First I've got this error:
2018-05-01 00:40 CRITICAL plugin                        Plugin `flexget.plugins.clients.deluge2` failed to import dependencies
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/flexget/plugin.py", line 396, in _load_plugins_from_dirs
    __import__(module_name)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/flexget/plugins/clients/deluge2.py", line 15, in <module>
    from utils.pathscrub import pathscrub
ImportError: No module named utils.pathscrub

So I had to change import in deluge2.py:
from utils.pathscrub import pathscrub to from flexget.utils.pathscrub import pathscrub
as well as from utils.template import RenderError to  from flexget.utils.template import RenderError
Then I've got this error.
2018-05-01 00:44 DEBUG    deluge        tv1             Error importing deluge: dynamic module does not define init function (init_openssl)
2018-05-01 00:44 CRITICAL task          tv1             Plugin `deluge` cannot be used because dependency `deluge` is missing.
2018-05-01 00:44 DEBUG    task          tv1             Deluge >=1.2 module and it's dependencies required. ImportError: dynamic module does not define init function (init_openssl)

Any ideas?

Sorry, I realized I had to use deluge3 plugin in config. I've created a pull request with working solution.
Initially I had an error with Deluge 1.3.15 installed :
TypeError: __init__() got an unexpected keyword argument 'deluge_version'

So I had to omit deluge_version in DelugeRPCClient setup and everything worked like a charm.

I will try to clean the code a bit when I get time

Ideally, I'd like it if we made a wrapper around DelugeRPCClient to get rid of the direct procedure calls in the plugin(s)

@cvium Not sure what you mean by this?
I think probably the one thing we should do is implement a very simple version of the get_localhost_auth, so as to not break everybodies setup who is relying on this when we release it.

@gazpachoking Just that deluge-client is very barebones and I don't think calling client.call('core.get_torrents_status', {}, ['name']) directly in the plugin is very nice. If the API changes, it's just easier to maintain if the Deluge plugins just do something like client.get_torrents_status('name', 'path').

@cvium Not sure I think wrapping is worth it. What about if we just enable a nicer call format in the new client? I think this PR would essentially make the deluge-client calls the same as the native deluge client calls.
https://github.com/JohnDoee/deluge-client/pull/19

Specifying a filter under from_deluge in config.yml does not filter the entries.
the self.get_torrents_status call on line 220 only sends one argument, it should send the filter as the second argument.


Blurry top bar when active normal text has a high intesity

With the airy theme, the "Preview Window" text is quite clear.

With the darker theme, the "Preview Window" text is quite blurry.

It appears this is directly related to the intensity bar on the right hand side for the Normal Text option. If the intensity is dropped, that specific piece of text doesn't look blurry anymore, but the text evidently looks a lot darker.

Sounds like a duplicate of #33? Try #54 (set the fonts again with the patch applied to update the config).


Dependency org.sonatype.sisu:sisu-inject-plexus uses old (patched) guava version

I am using the database-migrator within my project (as of #209). I notice when using mvn dependency:tree that the graph looks like this:
INFO] \- org.javalite:db-migrator-maven-plugin:jar:2.0:compile
[INFO]    +- org.apache.maven:maven-plugin-api:jar:3.0.5:compile
[INFO]    |  +- org.apache.maven:maven-model:jar:3.0.5:compile
[INFO]    |  +- org.apache.maven:maven-artifact:jar:3.0.5:compile
[INFO]    |  \- org.sonatype.sisu:sisu-inject-plexus:jar:2.3.0:compile
[INFO]    |     \- org.sonatype.sisu:sisu-inject-bean:jar:2.3.0:compile
[INFO]    |        \- org.sonatype.sisu:sisu-guice:jar:no_aop:3.1.0:compile
[INFO]    |           \- org.sonatype.sisu:sisu-guava:jar:0.9.9:compile

I use guava version 23.5-jre but still get run-time conflicts. Although I could probably exclude this dependency - I think it would be worth noting that org.sonatype.sisu:sisu-guava:jar:0.9.9 seems to be a patched old version of guava.
I'm aware I use the plugin for a different use-case (ie not as maven plugin, but in my webapp) - yet it would be an improvement for anyone wanting to run migrations when booting their (web)app without the need to use maven.

@stefanhendriks you can always use exclusions in your pom if you are using it not as a maven plugin. Since the code you are running has nothing to do with Maven, exclusion of a dependency you do not want on a classpath will do what you need.

@ipolevoy yes I did exclude it and it solved my problems I had with mixed guava dependencies. It looks like I can run the db migrator code fine without breaking anything at runtime. I wonder what org.sonatype.sisu:sisu-inject-plexus does though?

@stefanhendriks I'm guessing it injects dependencies into a plugin when it is executed by Maven. I do not think you need it for your case.


Add a CircleCI hello world test

This should avoid the "no tests" complaints from circleci while we're
still setting it up.

jenkins retest


Mir-bridge-ready-service with standard-srv-library

this adds a service to the mir bridge, which signals whether the mir bridge is ready (subscribed to all necessary topics = operational).
Using only standard srv message (from std_srvs.srv import Trigger)

Ready for merge


 I may implement it myself unless you object to it.
It would make this:
$ scp -O thingino.bin root@<IP_ADDRESS>:/tmp/
$ ssh<EMAIL_ADDRESS>sysupgrade /tmp/thingino.bin

Or this:
$ cat thingino.bin | ssh<EMAIL_ADDRESS>tee /tmp/thingino.bin >/dev/null
$ ssh<EMAIL_ADDRESS>sysupgrade /tmp/thingino.bin

Be simplified to just:
$ cat thingino.bin | ssh<EMAIL_ADDRESS>sysupgrade -

So, any objections?

we have a script in the repo for this, check scripts/fw_ota.sh

That script's awesome. The only downside is that the script would need to be downloaded prior to using. But I think it's worth the price, especially considering it also does a checksum check, which I think is very important.