Sheep slows down the website

The framerate of the canvas with the sheep decreases considerably on larger screens. This is a problem because it affects the web page's performance even when the sheep is not moving.
Factors that decrease performance:

Using a slower browser
Using an old/low budget/low-powered device
Using a large screen (e.g. 4K)

You can try it by zooming out. You'll notice that scrolling the page becomes more laggy the further you zoom out.

Did not experience any lag in chrome, I also looked at the frame meter and it constantly stayed at 140 frames no matter the zoom :/

I noticed a slight lag in Firefox on my private laptop. When I zoomed to 50%, I got around 15 fps. In Chrome, I had to zoom out to 25% until the framerate got lower than 60 fps. On my phone, it worked smoothly, but it almost crashed when I enabled the desktop page.


C++ interop

Crystal allows for the use of C, however my question is if there are plans to have interop for C++ as well. Well, there are cleary not, but wouldn't it be cool? It would complicate things quite a bit but Rust for example has C++ interop as their planned feature for 2017.
Some of us love scripting languages yet need them to be efficient, and I'm sure C++ interop would be used as much as C if not more. C is often used for UNIX software. C++ is used for industrial software where OO is necessary. I don't think people who use a lot of C like OOP. And I think most would agree C++14/17 and later is a much better language than C++03 or even C11. (Please no flaming...)
Another argument for this: it'd be a huge differentiator from languages like Python, Ruby, and Perl where C++ developers have to work with a CFFI and then wrap objects around it. Having it be seamlessly available in the language would open up the gateway to having it work with game engines and various industrial-grade solutions. It would also save a lot of people from having to use Python anywhere. :)
Just look at this shit: https://docs.python.org/2/extending/extending.html
PyArg_ParseTuple(args, "s", &command)  # why even live?

I'd personally recommend it, but after 1.0.

See https://github.com/crystal-lang/crystal/issues/3098

Okay it is obvious using C++ libs in Crystal would be hard to do.
But why not just take a look at how the Rust guys will be doing it?

D interop has plenty of limitations. No constructors\destructors, no stdcall and i'm pretty sure no STL or other templates in the functions interface (of course they can be inside implementation, but same is true for C interop). Is it much better than C interop?

@kennymalac Any link to Rust code implementing C++ interop?

Some links regarding the topic:

https://www.reddit.com/r/rust/comments/3cj1fg/rust_c_interop/
https://www.reddit.com/r/rust/comments/4yz59h/cc_interop/
https://internals.rust-lang.org/t/interfacing-d-to-legacy-c-code-a-summary-of-a-competing-languages-capabilities/1406
http://swig.org/

By the way, libclang would help, both for parsing C++ headers obviously, but also for mangling symbols, targeting the MSVC or GNU "standards".

Closing as duplicate of #3098 as Sija brought up


Sanitize user-input on file name

Sanitize user-input before it get passed into the DOM.
Prevent things like "" from being executed. This isn't a XSS attack as the server seems to be santizing the path as well, so this is just to prevent from people copy pasting the incorrect/bad file-names and get themself XSS'ed.

Codecov Report

Merging #17666 (b07a164) into main (5233051) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #17666      +/-   ##
==========================================
- Coverage   45.36%   45.32%   -0.04%
==========================================
  Files         800      800
  Lines       89169    89169
==========================================
- Hits        40448    40415      -33
- Misses      42217    42257      +40
+ Partials     6504     6497       -7

Impacted Files
Coverage Δ

modules/git/error.go
23.43% <0.00%> (-26.57%)
:arrow_down:

models/gpg_key_common.go
59.67% <0.00%> (-4.84%)
:arrow_down:

modules/git/repo.go
40.99% <0.00%> (-3.16%)
:arrow_down:

modules/notification/mail/mail.go
35.29% <0.00%> (-2.95%)
:arrow_down:

modules/notification/ui/ui.go
60.86% <0.00%> (-1.45%)
:arrow_down:

modules/queue/workerpool.go
48.85% <0.00%> (-1.15%)
:arrow_down:

services/pull/pull.go
40.77% <0.00%> (-1.02%)
:arrow_down:

models/notification.go
64.71% <0.00%> (-0.87%)
:arrow_down:

models/issue_comment.go
53.05% <0.00%> (+0.28%)
:arrow_up:

routers/api/v1/repo/pull.go
29.35% <0.00%> (+0.51%)
:arrow_up:

... and 4 more

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 5233051...b07a164. Read the comment docs.


Enable NDC by default

What changed?
Enable NDC replication by default

Why?
Migrate from 2DC to NDC

How did you test it?
Unit tests, integration tests and run in beta/prod cluster

Potential risks
In the worst case, the replication stops running.

Coverage decreased (-0.6%) to 68.135% when pulling a39da754f392cdc12c59b389874b8d76deaa3241 on remove_2dc into 6d1a17aee46c72429b2fad3849d3a1da49582fcf on master.


SPA templates need to be updated for endpoint routing

@JamesNK Seems like you left these puppies behind :(

Awww don't call them puppies, it makes me sad to think of puppies being left behind.

@rynowak That way I encourage you both to fix them. 🐶

Could I get some context on this? A link to the issue for the work which drove this would be helpful. A link to existing examples or descriptions of best practices would also be good.

Look at the configure method on the startup class for the razor pages templates. Those use endpoint routing. You can probably figure out the rest from there. If not, @rynowak or @JamesNK can probably help you, but I trust you'll figure out.

@javiercn is this statement from the routing docs no longer true for 3.0?

Note
With the release of endpoint routing in ASP.NET Core 2.2, endpoint linking is limited to MVC/Razor Pages actions and pages. The expansions of endpoint-linking capabilities is planned for future releases.


An idea needed for translate subject when this one was in config  message options

Currently isn't  possible to perform auto translate when used options value. F.e. config
'sxmail' => [
        'configs' => [
            'default' => [
               // ...
                'message' => [
                    'options' => [
                        'subject'  => 'Hey no body is perfect',
                    ],
                ],
            ],

this 'Hey no body is perfect' needed to be translated  for different countries. I have no idea how to done this. Please help me :) thx!
BTW
// TODO: DI never  initiate an instance via new operator maybe we need  $this->getMessage() and I can use $this->setMessage($message// from factory) to
/**
     * Compose a new message.
     *
     * @param   mixed   $body   Accepts instance of ViewModel, string and null.
     * @param   string  $mimeType
     *
     * @return  \Zend\Mail\Message
     * @throws  \SxMail\Exception\InvalidArgumentException
     */
    public function compose($body = null, $mimeType = null)
    {
        // Supported types are null, ViewModel and string.
        if (null !== $body && !is_string($body) && !($body instanceof ViewModel)) {
            throw new InvalidArgumentException(
                'Invalid value supplied. Expected null, string or instance of Zend\View\Model\ViewModel.'
            );
        }

        $body    = $this->manipulateBody($body, $mimeType);
        $message = new Message; // TODO: (DI fault) never  initiate via new operator maybe $this->getMessage() and I can use $this->setMessage($message// from factory)

        $message->setBody($body);

        if ($this->config['message']['generate_alternative_body'] && count($body->getParts()) > 1) {
            $message->getHeaders()->get('content-type')->setType('multipart/alternative');
        }

        $this->applyMessageHeaders($message);
        $this->applyMessageOptions($message);

        return $message;
    }

@olekhy Sorry for the late reply.
You could translate the subject on an event yourself. Otherwise, a pull request that allows you to translate automatically (like other modules and components do) would be very welcome!

You could translate the subject on an event yourself.

meaning handle bootstrap event to and rewrite subject in configuration with one translated?
for me is a performance impact.

@olekhy To me, too. You can use the message returned by compose and set it yourself. Otherwise the only thing that remains really is the translator implementation (which isn't that difficult to do).


Performance of baselines on your modified SMAC

Hello! Thank you for sharing your code and RODE is a very interesting work!
However, I am confused by the experimental results. We found that you modified the SMAC environment, which is not mentioned in your paper. And when we carried out QMIX on your modified SMAC, we found that QMIX achieved performances comparable to RODE on hard scenarios like MMM2. We use five independent runs for each algorithms to get convincing results. So we suspect that there may be an unfair comparison of the results in the paper. The performance improvement of RODE may benefit from the modified SMAC environment rather than the role-based mechanism.
It may be a careless mistake. Could you check the performance of QMIX or other baselines in your modified SMAC?

We first clarify why we modified the SMAC environments. SMAC actions are related to the identity of enemies (an attack action is aimed at a specific enemy). Every time the environment is reset, the same action ID may attack different enemies, resulting in the change of action semantics, which makes it impossible to learn action representation. So we sorted enemies.
When we submitted our paper, we didn't realize that this change can also improve the performance of QMIX. After we got aware of this issue, we tested QMIX on this changed SMAC benchmark. RODE still outperforms on most of super hard scenarios like corridor and 6h_vs_8z.

Thank you very much for your reply!
I believe it is not a deliberate mistake. However, we think you should state in a prominent position in Readme.md that you have changed the SMAC environment, which will avoid misleading followers.
The performance of RODE in the paper has absolute advantages over other baselines, but you know, this is not true in practice. Other baselines, like QPLEX and Weighted QMIX, should perform better than QMIX on your changed SMAC benchmark. If their true performances are taken into account, can RODE still be state of the art in a few super hard scenarios? Could you rerun other baselines on your modified SMAC and then update the true result in your paper? Although it will take you some time, we think it will benefit the whole MARL community.
Thank you very much!

Thank you very much for pointing this out!
We agree it will benefit the MARL community. We are going to update the
paper's results as soon as possible.
guestreturn @.***> 于2022年6月20日周一 15:48写道：

Thank you very much for your reply!
I believe it is not a deliberate mistake. However, we think you should
state in a prominent position in Readme.md that you have changed the SMAC
environment, which will avoid misleading followers.
The performance of RODE in the paper has absolute advantages over other
baselines, but you know, this is not true in practice. Other baselines,
like QPLEX and Weighted QMIX, should perform better than QMIX on your
changed SMAC benchmark. If their true performances are taken into account,
can RODE still be state of the art in a few super hard scenarios? Could you
rerun other baselines on your modified SMAC and then update the true result
in your paper? Although it will take you some time, we think it will
benefit the whole MARL community.
Thank you very much!
—
Reply to this email directly, view it on GitHub
https://github.com/TonghanWang/RODE/issues/11#issuecomment-1160094341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIX4MOSXSNOWWQCLMOULOVDVQAO3LANCNFSM5ZEUK3ZQ
.
You are receiving this because you commented.Message ID:
@.***>

Hi, we start to test QPLEX and Weighted-QMIX on our SC2 environments, and results will be avaliable in a few days.
In my humble opinion, there is a question that I want to discuss with you. Do you think this comparison is fair? RODE uses QMIX for the role selector and role policies for mixing. Using QPLEX or Weighted-QMIX in RODE may have different results. For now, we are sure RODE performs better than QMIX.

We are happy and excited to see that things are progressing well! Thank you for all you have done!
We know that RODE is an excellent work, and the components of its framework can be composed of any value decomposition method. However, our original intention of proposing this issue is to know the real performance comparison of different algorithms under the same SMAC. In addition, to make the comparison more fair, we believe that the neural network model size of each algorithm should be similar. As we know, the model size of RODE is twice that of QMIX but RODE performs worse than QMIX in almost all easy and hard scenarios.
We proposed this issue to point out that there is an unfair comparison in the original paper because of different SMAC, and the experimental results in your paper are quite different from the reality.

Hello there, thank you for addressing this issue, are there any updates on the new tests in the modified environment?

@TonghanWang any updates on this?


Local Development - Developer defined config values for server

Problem
The server requires many input variables to start. This prevents it from being used easily during local development. It currently fetches these from environment variables which require "real values."
Solution
Decouple the input variables from directly requiring environment variables (yet still support environment variable values being used for production). Use viper (which is MIT licensed).
Implementation

use a server.env for a developer to set values outside *.go files or having to call a script
update .gitignore
refactor [./server/config/environment.go] to load from config file with ENV overrides viper.AutomaticEnv()
update developer setup doc to include information
Allow for input variables for Azure credentials to pass through so that during local development DefaultAzureCredential will fall back to using the current security context of the Azure CLI.

Output

Server is able to start running without "real values" for local development

Picking up and starting work today on this.


(ion-virtual-scroll): routerLinkActive="active" causes virtual elements to retain active state across unrelated elements when used with child router outlet

Bug Report
Ionic version:

[x] 4.x
Current behavior:
When using ion-virtual-scroll in conjunction with routerLinkActive="active" (and a child ion-router-outlet for master-detail pattern) --- as virtual elements are re-used for new data, the active state on the element is retained (displays as active), despite the fact that the virtual element now reflects data for another entry in the list (one who's route is NOT active.)
Expected behavior:
Each virtual element's routerLinkActive state should be updated to reflect the state of that element's backing data. If routerLinkActive was set for a link/data that was displayed further up in the list, it should not be retained on an unrelated data point when scrolling down the list as that virtual element 's DOM structure gets re-used for a new data element in the list.
Steps to reproduce:
Use ion-virtual-scroll and make each item a routerLink with routerLinkActive="active".
Related code:
<ion-virtual-scroll [items]="items">
   <ion-item *virtualItem="let item" [routerLink]="['/detail',item.id]"  routerLinkActive="active">
         {{ item.name }}
   </ion-item>
</ion-virtual-scroll>

<!-- Somewhere else in the page -->
<ion-router-outlet></ion-router-outlet>

Other information:
Ionic info:
$ ionic info

Ionic:

   ionic (Ionic CLI)             : 4.8.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0
   @angular-devkit/build-angular : 0.12.3
   @angular-devkit/schematics    : 7.2.3
   @angular/cli                  : 7.2.3
   @ionic/angular-toolkit        : 1.2.3

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : not available
   Cordova Plugins       : not available

System:

   Android SDK Tools : 26.1.1 (/usr/local/share/android-sdk)
   ios-deploy        : 1.9.4
   NodeJS            : v10.6.0 (/usr/local/Cellar/node/10.6.0/bin/node)
   npm               : 6.7.0
   OS                : macOS Mojave
   Xcode             : Xcode 10.1 Build version 10B61

Relates to #16632

FYI this is still an issue in 4.7.4.


React 15.5 update: Use prop-types instead React.PropTypes

This PR will make react-fela stop throwing warnings with React 15.5 because React.PropTypes is deprecated. prop-types should be a dependency of react-fela now.
Non-related question: Do you know https://lernajs.io/? Is there some reason why you don't use it for fela?

As for the lerna question. I've seen that and actually love it, but had no time (and no benefit as everything works fine now) to switch. Although, I would love to use Lerna :P


add end to end test for dynamicfile backend mode

to run the end to end test:
make start-dev-dynamicfile
The command will do the following steps:
(1)create an image of authenticator.
(2)run a kind cluster and run the authenticator with above image in the container
(3)generate a kubeconfig to authenticate through authenticator token
(4)create an aws IAM role for testing and assumed this role
(5)run kubectl get nodes with the role

/ok-to-test

/ok-to-test

/lgtm
/approve
two issues I think should be improved for future for integration with existing CI  but as it is the script is demonstrably useful so fine with merging as-is (that is what "hack" dir is for after all I guess)


Update codety.yml for fixing the failed pipeline.

Update codety.yml for fixing the failed pipeline.

[x] I have not changed the README.md directly.

Oh, right. Apologies for missing that and thanks for the fix. 🙏


electrs: 0.9.9 -> 0.9.10

[x] Built and run on x86_64-linux.

cc @prusnak

cargoHash doesn't look right

Ah, sorry, pushed an old commit.


Shorten File Names for Go

While using kiota generated filenames we do have very long nested files that would require modification of the git flag git: longpaths: true to work. The actual file names in windows are considered to the the subfolders + path name i.e that would be “./vendor/github.com/namespace1/namespace2/subfolder/file_name_with_max_allowed_length.go”.
A fix for this issue will be to consider the namespace parameter when generating the file name so that max filename will be "namespace + filename" - 5 characters when generating the name

@rkodev can you provide some background information on the potential drawback for customers of enabling long path support please? This way we'll be able to make an educated decision here.

The only draw back for having to enable the long path support is the possible added friction for users to enable this feature for api schemas that are shipped.  This was an issue because of a request by users on why we support very long names.
There are no technical issues as a result of the long file names generated other than the convenience of automatically managing the edge case for users and not having to request them.

Thanks for providing the context. Let's leave it as is right now and see if we get additional feedback on the long paths requirement this sometimes imposes. Closing.


Can't find USDC/WETH pair

when I was fuzzing a contract I got an error:
 INFO fetching pairs for 0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48
 INFO fetching pairs for 0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48
thread 'main' panicked at src/evm/tokens/uniswap.rs:233:10:
Unexpected RPC error, consider setting env <ETH_RPC_URL>

I looked into the code and find it is due to empty response when requesting pairs
and then I manually requested the api https://pairs.infra.fuzz.land/single_pair/{network}/{token}/{weth}, which is pairs.infra.fuzz.land/single_pair/eth/0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48/0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2 in this case, I got an empty response

fixed now. our db was rekt last week and i forgot to include pool #0 when re-indexing.


Definition of global properties closes issue #24

This PR will provide the implementation of global properties along with an example of how to load parameters and define them as global properties.

This is a design, not a code issue, but I think it's worth asking if we would prefer JSON to toml..

I can take a look this afternoon
From: GuidoEspana @.>
Sent: Friday, October 4, 2024 1:44 PM
To: CDCgov/ixa @.>
Cc: Rescorla, Eric (CDC/IOD/ORR/CFA) @.>; Mention @.>
Subject: Re: [CDCgov/ixa] Definition of global properties closes issue #24 (PR #47)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

@confunguido commented on this pull request.

In examples/parameter-loading/infection_manager.rshttps://github.com/CDCgov/ixa/pull/47#discussion_r1788079798:

       context.add_plan(1.0, move |context| {

           println!("time {:?} - PersonID: {:?} - status: {:?}",

               context.get_current_time(), person,

               context.get_person_property(person, InfectionStatusType));

           context.set_person_property(

               person,

               InfectionStatusType,

               InfectionStatus::I);

       });

Can someone help here? I am having a hard time with this test. The purpose of the test is to make sure that people who are set to be infectious will eventually recover (handle_infection_status_change). However, it isn't working properly. I added this print statement here and below to check people's infection status. Weird thing is, people actually recover if I include the statement from lines 79-82, but they never do if I remove that statement. What am I doing wrong? @jasonasherhttps://github.com/jasonasher @ekr-cfahttps://github.com/ekr-cfa @k88hudson-cfahttps://github.com/k88hudson-cfa @ChiragKumar9https://github.com/ChiragKumar9
—
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/ixa/pull/47#pullrequestreview-2348751193, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BBXGWHLH5ZEUF2Z2C2UC3WLZZ3HXNAVCNFSM6AAAAABOBUYPU6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNBYG42TCMJZGM.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Invalid email addresses are showing as valid

I'm using the validatable module Some examples:
<EMAIL_ADDRESS><EMAIL_ADDRESS>And some special characters:
<EMAIL_ADDRESS>Are being allowed through.

Facing the same issue. Using devise 4.2 and emails ids such as abc@abc,com<EMAIL_ADDRESS>&<EMAIL_ADDRESS>are all returning as valid.

Just ran into this.  Looks like the new default regex only validates that the domain doesn't contain "@" or "\s".  The old regex didn't validate against space chars. :(
You can set your own regex in config/initializers/devise.rb with:
config.email_regexp = /\A[^@\s]+@([^<EMAIL_ADDRESS>The above is a hybrid of the old and new defaults (rejects commas and spaces).

Yeah, if you need some more strict validation on email you can redefine it in config. See full answer here https://github.com/plataformatec/devise/issues/3820


send data back from child route

Hi there,
I am confused about how to send data back from child route when pressing back nav button. Can you please help?
Thanks.

How can I get the rendered component instance from router?


Type Message again, and fix resulting flow issues - 2

This is a remade PR of https://github.com/keybase/client/pull/9055

@keybase/react-hackers
cc: @akalin @MarcoPolo


Fix toleration issue

Proposed changes
Fix toleration issue

Types of changes

Bugfix (non-breaking change which fixes an issue)

Linked Issues
https://github.com/rancher/rancher/issues/15585

Further comments

@loganhz can this be merged before the types pr you mentioned here

@westlywright Backend is merged. Please merge this PR. Thanks!


It looks like vendor:publish don't do its job

Hello,
I've done installation by instruction

Run the command in terminal: composer require themsaid/laravel-langman-gui
add service into config/app.php: Themsaid\LangmanGUI\LangmanServiceProvider::class
publish the configuration file & assets: php artisan vendor:publish --provider=Themsaid\\LangmanGUI\\LangmanServiceProvider

I got response Publishing complete.
But nothing has been added to project public folder and config folder. Langman home page is without style and not working properly.
I use Laravel 5.5.
What can be wrong?

I found solution here
https://github.com/themsaid/laravel-langman-gui/issues/29


Adds Ability to Create XFS-Based AMIs

Closes #65

Updates:

DiskSetup.sh
MkChrootTree.sh
MkTabs.sh
README.md

Adds: Docs/README_XFS.md

Tested:

Hand-invocation for creation of LVM-enabled AMIs
Hand-invocation for creation of simple (/ & /boot) AMIs

This is great, thanks for putting it together. Looking forward to integrate these changes in our project.

Seems silly, but should you add xfsprogs (provides xfs_admin) to the list of required progs in Docs/README_dependencies.md ? Just in case they want to use XFS but don't have the required RPM installed.

@jokajak :
XFS is the default filesystem for EL7, so, the XFS utilities are in the @core RPM-group:
$ yum groupinfo core 2> /dev/null | grep xfs
    xfsprogs


DASH options

I deselected DASH playback in Player settings and it appears DASH is still enabled... videos will not fully buffer when paused at the beginning.
Am I doing something wrong?
Also auto res doesn't work, select auto res to 720p and videos still load at 480p (yes they are available in 720p)
Newbie here, thanks!

Happens to me too! FF 36.0.4, YTC script Dev version (4.95)

I'm not seeing this behavior in Firefox 37b7. Using this video as an example, when I disable DASH, only the 360p quality option is available and the video does completely buffer when paused.

I think it's more of the progress bar issue. When I pause the video, the progress buffer bar stops moving but when I pressed play after quite some time, it suddenly jumped forward so many minutes.

Sorry should add, this problem on Chrome 41.0.2272.101 m
YTC v 2.1.7

Update to the Developer Version. It's possible this is a bug or incompatibility that has been fixed since 2.1.7.

Same problem here and I'm using the Developer Version.

YTC Script Dev Build 507, DASH disabled, video won't buffer unless the play button is pressed first.


Nested scopes with lifetime bounds

I'd like to be able to ask a Scope object for a nested scope whose lifetime is bounded by the lifetime of the outer scope. That would allow a scoped thread to spawn a more narrowly scoped inner thread that can borrow from the outer-scoped thread.
This could look like a method scope on Scope that creates a new Scope with an appropriately bounded lifetime.

It seems https://github.com/rust-lang/rust/pull/94559 implements this.

When I posted the original request, I had a specific use case that motivated it, but it's been too long and I can no longer reconstruct the specific use case. I'm going to close this.
I'm looking forward to the new standard library scopes.


set_build_number_repository cannot update path containing $(SRCROOT)

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I read the Contribution Guidelines
[ ] I read docs.fastlane.tools
[ ] I searched for existing GitHub issues

Issue Description

Complete output when running fastlane, including the stack trace and command used

[INSERT OUTPUT HERE]

Environment

[INSERT OUTPUT HERE]

@pronebird Going to close this issue since no issue information, fastlane env, or fastlane output was provided. Please open another issue with all requested information and I will be happy to look into 😊


Issue with building stream_chat_v1

I'm trying to build your sample app, stream_chat_v1, but it's giving me this error with gradle.
Snippet of error:

/C:/Users/Ming%20Jun/Documents/flutter/.pub-cache/hosted/pub.dartlang.org/drift-1.3.0/lib/src/runtime/query_builder/schema/column_impl.dart:67:3: Context: Found this candidate, but the arguments don't match.
GeneratedColumn(
^^^^^^^^^^^^^^^
/C:/Users/Ming%20Jun/Documents/flutter/.pub-cache/hosted/pub.dartlang.org/stream_chat_persistence-3.0.0/lib/src/db/moor_chat_database.g.dart:5153:7: Error: No named parameter with the name 'typeName'.
typeName: 'INTEGER', requiredDuringInsert: false);
^^^^^^^^
/C:/Users/Ming%20Jun/Documents/flutter/.pub-cache/hosted/pub.dartlang.org/drift-1.3.0/lib/src/runtime/query_builder/schema/column_impl.dart:67:3: Context: Found this candidate, but the arguments don't match.
GeneratedColumn(
^^^^^^^^^^^^^^^
2
FAILURE: Build failed with an exception.

Where:
Script 'C:\Users\Ming Jun\Documents\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 1070

What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.

Process 'command 'C:\Users\Ming Jun\Documents\flutter\bin\flutter.bat'' finished with non-zero exit value 1

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

Get more help at https://help.gradle.org

BUILD FAILED in 15s
Exception: Gradle task assembleDebug failed with exit code 1

Full error:
getstream_issue.txt
I've been trying to figure out why for the past few days but cant seem to find an answer. Please advice!

this is probably related to this issue https://github.com/GetStream/stream-chat-flutter/issues/837
I'll push a new version using the 3.4 version of stream_chat_flutter in a sec


C++ Example

Hi!
In your "readme.md" and "examples" folder, examples are provided only with python bindings.
In https://github.com/jlmelville/rcpphnsw rough equivalent of the serialization/deserialization example from this hnswlib README file is provided in 'R' language.
But nowhere C++ example of serialization/deserialization equivalent to current README example is provided. I want to use hnswlib in my c++ project. I was wondering how to do the exact functionality in c++.  It would be great if you translate examples given in python to c++.
Thank you!

@ashfaq92
Thanks for noticing! I'll make an example.

The basics of creating/re-sizing/re-loading an index, as used in RcppHNSW are below. It doesn't really do anything, but it should compile and run without causing your computer to explode, which is 90% of the battle in C++. Hopefully you can work out the rest!
#include <memory>
#include "hnswlib.h"
void example()
  auto dim = 16;
  auto num_elements = 100000;
  auto init_num_elements = num_elements / 2;
  auto M = 16;
  auto ef_construction = 200;

  using dist_t = float;
  using Distance = hnswlib::L2Space;

  // create the index
  auto space = std::unique_ptr<Distance>(new Distance(dim));
  auto appr_alg = std::unique_ptr<hnswlib::HierarchicalNSW<dist_t>>(
    new hnswlib::HierarchicalNSW<dist_t>(space.get(), init_num_elements, M,
                                         ef_construction));

  // resize it
  appr_alg->resizeIndex(num_elements);

  // or save and reload it
  std::string path_to_index = "myindex.hnsw";
  appr_alg->saveIndex(path_to_index);
  appr_alg.reset(new hnswlib::HierarchicalNSW<dist_t>(space.get(), path_to_index));
};

@jlmelville thanks for prompt response. Sorry for late reply due to "timezones :(". I have run your code and it works without error (as you said :). But it would be really nice if you update this code according to the readme file. I mean if it make HNSW graph with random numbers and finds nearest neighbor for any random query point. Then we may add this code in readme files so that future c++ beginners may embed that in their code (like me).

With the addition to above code
`auto dim = 512;
auto num_elements = 100000;
auto init_num_elements = num_elements / 2;
auto M = 16;
auto ef_construction = 200;
using dist_t = float;
using Distance = hnswlib::L2Space;

// create the index
auto space = std::unique_ptr<Distance>(new Distance(dim));
auto appr_alg = std::unique_ptr<hnswlib::HierarchicalNSW<dist_t>>(
        new hnswlib::HierarchicalNSW<dist_t>(space.get(), init_num_elements, M,
                                             ef_construction));

//    // resize it
////    appr_alg->resizeIndex(num_elements);
typedef size_t labeltype;
appr_alg->setEf(200);
auto ret=appr_alg->searchKnn(default_emb,1);//'1' is the nearest neg
std::vector<std::pair<dist_t, labeltype>> result;//result return after knn search

//    while (!ret.empty()) {
//        result.push_back(ret.top());
//        ret.pop();
//    }
int f=0;
int id_label;
float mn=100000;
if(ret.size()==0)
{
    counter_value+=1;
    appr_alg->addPoint(default_emb,counter_value);
    LOGD("This is empty ret");
} else{
    LOGD("Non empty ret");
    while (!ret.empty()) {
        result.push_back(ret.top());
        ret.pop();
    }
    for(int i =0;i<result.size();i++)
    {
        LOGD("hnswlib result:(1)distance:) %f => (2)label:) %d",result[i].first,result[i].second);
        if(result[i].first<(0.5)&&mn>result[i].first)
        {
            f=1;
            mn=result[i].first;
            id_label=result[i].second;
        }
    }
    if(f==0){
        counter_value+=1;
        id_label=counter_value;
        appr_alg->addPoint(default_emb,counter_value);
        LOGD("New entry generated");
    }else{
        LOGD("Id we get:%d",id_label);
    }
}

result.clear();`


Feature requests for colmena eval

It would be nice to be able to pass extra arguments to the file being evaluated with colmena eval. Like nix eval supports --arg{,str}. I would like to be able to do
colmena eval --argstr foo bar file.nix
and file.nix could look like
{ nodes, foo, ... }: some expression
Currently I could instead do
colmena eval -E 'args: import file.nix (args // { foo = "bar"; })'
but that's rather awkward.

Also, just like there is a --instantiate option, it would be nice to have a --build option to instantiate and build a derivation. Currently one can --instantiate, and then build the resulting drv.

Alternatively, is there any way to reproduce colmena's evaluation logic locally? This would also allow us to have flake outputs that would get evaluated as though through colmena eval. I'm imagining something like
inputs.colmena.url = ...;
outputs = {self, colmena, nixpkgs}: {
  colmena = ...;
  packages.x86_64-linux.foo = colmena.lib.eval self.colmena ( {nodes, ...}:
    nixpkgs.legacyPackages.x86_64-linux.stdenv.mkDerivation {
      ...
    };
  });


Feature request: trigger collection compaction via HTTP REST API

My Environment

ArangoDB Version:        3.8.3
Storage Engine:             RocksDB
Deployment Mode:       Leader/Follower ("Master/Slave")
Deployment Strategy:   systemd
Infrastructure:               own
Operating System:        Ubuntu 20.04
Total RAM in your machine:        256 GB
Disks in use:        SSD
Used Package:              Ubuntu .deb

Component, Query & Data
Affected feature:
HTTP API
Problem:
This is a question / feature request. Could it be possible to trigger background compaction on a single collection (not on the whole database) via HTTP REST API ?
This could be useful to periodically compact ever-growing collections, as a workaround for #14237
See also https://github.com/arangodb/arangojs/issues/630
Thanks a lot for your consideration.
Greetings,
Mathias

Hi,
I'm sorry, but the compaction is part of the RocksDB core ArangoDB is built on top of.
However, RocksDB doesn't know about the scope of collections; all it has is binary documents with an ID;
So while this had been possible with MMFiles, where a collection would map a set of files, its not possible with RocksDB anymore.
However, it seems that other people have improved their system performance by more frequently invoking the compaction, maybe that could also improve your situation?
We're still looking for a more good trigger for the compaction than wall clock, however, we didn't find any yet.

Hi, thanks for your answer.
Actually it's possible to trigger compaction for a single collection via arangosh usingdb.collection_name.compact(); so one could expect that this procedure can be triggered using HTTP API too.
Frequently invoking the compaction is exactly what we would like to do, hoping to achieve it through a cronjob-scheduled Arangojs script, but only for the collection that grows endlessly.
We would prefer to avoid writing a shell script that calls arangosh then .compact(), but if this is the only way we might consider doing this anyway.
Thanks for any advice

have a look at the documentation:
https://www.arangodb.com/docs/stable/data-modeling-databases-working-with.html#compact

I've tagged other tickets that are discussing issues around the compaction as well.

nb, arangosh only talks to arangod via the http API ; You can also explore these via the swagger interface in the UI, which will also give you a curl command.

I believe this potentially adresses the feature request : https://github.com/arangodb/arangojs/pull/756

@Zyqsempai This is relevant to what we talked about at the drivers sync. The route seems to exist but is undocumented.

Closing as the route exists, maybe still deserves to be documented.


Fix PHP 7.2 action

Fixes #348
I have had to remove a dev dependency that was causing composer to fail install.
https://github.com/sebastianbergmann/phpcpd
I tried changing version to ^4.0|^5.0 but composer still failed.
If you are using this somewhere externally you will have to find a workaround for php 7.2.

@mikebronner Should I close this? It does fix the PHP 7.2 action but I understand you may be using the removed package externally somewhere.

@dmason30 nono :) sorry, this just hasn't been a priority.

I'm opting not to move forward with PHP 7.2 testing due to incompatibility.


Bump winrm deps

We are about to update all chef/chef-dk deps to use winrm 2.0 gems. This should prevent chef external test gem conflicts and ensure users of this gem get the updated gems.

Are these cross compatible with old Chef? I would imagine so.

chef itself does not actually depend on winrm, travis pulls them in as it resolves the external gems.


m4b support?

Hi, it would be awesome if this supported m4b as an input and output format.

The tools is configurable in almost every possible way. You can add your own tools/format/presets.
So basically, you can support any format


Fix syntax on enabling cloud_sql_proxy service

"enabled" parameter for service module take value "yes" or "no"
https://docs.ansible.com/ansible/latest/modules/service_module.html

@niainaLens Can you tell me where you're seeing a syntax error? The true/false and yes/no are relatively interchangeable, other than Yamllint complaining about non-true/false truthy values. What version of ansible, python, etc are you seeing the syntax errors with?

Hi.
this isn't a syntax error, sorry
but according to ansible doc (2.8, latest), it's indicated to use choice value yes/no

@niainaLens Appreciate the input, however I'm going to go ahead and mark this as closed. As you can see, the CI build is failing as a result of this change. True/false is Boolean just as yes/no is, and linting checks favor true/false over yes/no. They are interchangeable from a functional perspective. Cheers!


Better fix for #6122 to avoid #6535

The change in #6124 introduces a regression with functions that are
implicit noexcept in a pxd file.
Here we implement a different fix for #6122 that doesn't cause the regression.
See the discussion in https://github.com/cython/cython/pull/6124#issuecomment-2282955499
Fixes: #6335

Maybe it's worth adding a test example as in https://github.com/cython/cython/issues/6335#issue-2459632294
$ cat a.pxd
cdef int f()
$ cat a.pyx
cdef int f():
    pass
$ cython -3X legacy_implicit_noexcept=True a.pyx
warning: a.pxd:1:10: Implicit noexcept declaration is deprecated. Function declaration should contain 'noexcept' keyword.
warning: a.pyx:1:10: env.entries[self.declared_name()]=Entry(<7f78f7ec0ce0>, name=f, type=int (void) noexcept).

Error compiling Cython file:
------------------------------------------------------------
...
cdef int f():
     ^
------------------------------------------------------------

a.pyx:1:5: Function signature does not match previous declaration

I really don't know how to reliably run tests locally, so I can't really test such a test. (This is a bit unfortunate, you'll see the python3-Cython package in void linux has check disabled because of this reason.)

I really don't know how to reliably run tests locally, so I can't really test such a test

In the Cython directory just checked out from git:
python3 runtests.py name_of_test

There's no need to build it or anything and it's probably easier not to.

I really don't know how to reliably run tests locally, so I can't really test such a test

In the Cython directory just checked out from git:
python3 runtests.py name_of_test

There's no need to build it or anything and it's probably easier not to.

Thanks, that works for an individual test. Is there a recommended invocation to run a reasonable subset of the testsuite that is expected to pass? Something that we can use as a check when updating the void linux package.

I added a test that shows #6535 is fixed.

3.0.x commit 48026eb40

I've reverted this for now because it was causing quite a few false positives when building Cython itself, so wasn't as specific as we hoped.
I think it'd be good to get something like this into Cython master, but I'll give up on putting it in the 3.0.x branch at this point.

I've reverted this for now because it was causing quite a few false positives when building Cython itself, so wasn't as specific as we hoped.

How so, can you be more specific?

I think it'd be good to get something like this into Cython master, but I'll give up on putting it in the 3.0.x branch at this point.

What is the status of #6535 then?
Note that #6535 is a proxy for "sagemath fails to build with cython 3.0.11".

I've reverted this for now because it was causing quite a few false positives when building Cython itself, so wasn't as specific as we hoped.

How so, can you be more specific?

It's essentially https://github.com/cython/cython/issues/6428. Run python3 setup.py build_ext -if on Cython and it generates a whole lot of warning that look wrong.

I think it'd be good to get something like this into Cython master, but I'll give up on putting it in the 3.0.x branch at this point.

What is the status of #6535 then?
Note that #6535 is a proxy for "sagemath fails to build with cython 3.0.11".

A right... it sounds like I need to revert that too. I'll look at that today.


Element Type Invalid Error rendering Custom block type

Hi, I'm wondering if this is a bug or if it's something I'm doing wrong. I'm following the instructions on the draft-js site for custom block rendering:
https://draftjs.org/docs/advanced-topics-custom-block-render-map.html#content
When I try to swap the block type to the custom block type I configure, it gives the error:
Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it’s defined in.

Here's a github I made to show the problem:
https://github.com/albertlockett/draftjs-help/
If you click just below the button, (in the editor), and the click the button to change the block type, you'll get that error in the console.
I would expect that it renders the editor to change ot my custom block type.
I experienced this in Chrome
"draft-js": "0.10.0",
"react": "15.4.2",

I think I figured out what's happening:
The docs say to do this:
const blockRenderMap = Immutable.Map({
  'MyCustomBlock': {
    // element is used during paste or html conversion to auto match your component;
    // it is also retained as part of this.props.children and not stripped out
    element: 'section',
    wrapper: MyCustomBlock,
  }
});

But the code in DraftEditorContent.react.js does this:
      if (info.wrapperTemplate) {
        const blocks = [];
        do {
          blocks.push(processedBlocks[ii].block);
          ii++;
        } while (
          ii < processedBlocks.length &&
          processedBlocks[ii].wrapperTemplate === info.wrapperTemplate
        );
        const wrapperElement = React.cloneElement(
          info.wrapperTemplate,  // this is what gets passed in as wrapper from snippet above
          {
            key: info.key + '-wrap',
            'data-offset-key': info.offsetKey,
          },
          blocks
        );

React.cloneElement is expecting an Element as the first argument, not the component
https://reactjs.org/docs/react-api.html#cloneelement
So it works if you do this:
const blockRenderMap = Immutable.Map({
  'MyCustomBlock': {
    element: 'section',
    wrapper: <MyCustomBlock />,
  }
});

So should the solution be to update the documentation, or to have DraftEditorContent create an instance of the info.wrapperTemplate Component when it tries to render the block and pass that as the argument when it creates the outputBlocks?

@albertlockett Confirmed – both the issue explanation and the workaround
Needless to say, I hope that the docs outline the API as it should be (cloning the elements feels weird)

I can also confirm that the above change fixes the problem. Please see pull request #1837


On changing date page contents are not loading .

only when we do a force reload it work

s

This issue is resolved now


Adds a core/embed block filter to transform iframes to links

Remote servers are expected to sanitize content.
In the case of iframe elements Mastodon and others simply drop them.
This PR pre-sanitizes the content, ensuring iframes display as links for remote servers.

Fixes #359
Proposed changes:

This uses the core/embed block filter to revert the iframe markup into a simple link block.

Other information:

[ ] Have you written new tests for your changes, if applicable?

Testing instructions:

Make a post inserting a youtube link (or from any of the whitelisted providers ),
The post content should display normally on site, showing the embedded iframe content
The ActivityPub representation of the post should show the original link inside a paragraph

Nice one!!


Formatted hackathon report for gdoc

After the approval I would upload the materials to the generator http://preview.biohackrxiv.org/
@joshmoore @nolden awaiting your review and OK.

@pschader

I was happy with the most recent gdoc version, so we can proceed from my side.

@joshmoore any changes necessary from your side?

Apologies @stefanches7 and thanks for the ping. In my challenge s, I haven't even had a chance to read the latest. I will ASAP!

WFM! Merging and we can always do another round if needed.


feat: update goreleaser scoop, delete deprecated params

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 74.456%

Files with Coverage Reduction
New Missed Lines
%

pkg/cmd/destroy/options.go
7
81.86%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
7054

Relevant Lines:
9474

💛  - Coveralls


how to build an exe that can run machines without codon

I built two exe by codon build -release -exe nw.py and codon build -exe nw.codon with codon. But when I run executables on another machine without codon. Neither of them can run, there will be same error.
./nw: error while loading shared libraries: libcodonrt.so: cannot open shared object file: No such file or directory
I know this is due to a lack of codon libraries at runtime. But I really want to know how to build or encapsulate an exe that can run on machines without codon.
Besides, I would like to know why there is a big difference in the running speed of these two executables. The first executable is one-fifth the size of the second, but nearly 20 times as fast.

@ChenyueYuan2003 The use of shared libraries provides several benefits, including code reusability, efficient memory utilization, and ease of updating libraries without recompiling the entire program. Codon has multiple dependencies, leading to a significant increase in the executable file size. It will definitely exceed 1GB.

Thank you for the explanation, it deepened my understanding of Codon.

@ChenyueYuan2003 The use of shared libraries provides several benefits, including code reusability, efficient memory utilization, and ease of updating libraries without recompiling the entire program. Codon has multiple dependencies, leading to a significant increase in the executable file size. It will definitely exceed 1GB.

Thank you for the explanation, it deepened my understanding of this

FYI we are looking into a fully statically-linked option that avoids needing to link to the runtime library (libcodonrt.so); no ETA but it is on our roadmap (https://docs.exaloop.io/codon/general/roadmap#infrastructure-and-tools).
Aside from that, the -release flag performs optimizations like -O3 on Clang/GCC, which is why the performance and binary size are different.

FYI we are looking into a fully statically-linked option that avoids needing to link to the runtime library (libcodonrt.so); no ETA but it is on our roadmap (https://docs.exaloop.io/codon/general/roadmap#infrastructure-and-tools).
Aside from that, the -release flag performs optimizations like -O3 on Clang/GCC, which is why the performance and binary size are different.
Thanks for the explanation. Codon has been great so far and helped me complete my assignment brilliantly, looking forward to following up with further work.

FYI we are looking into a fully statically-linked option that avoids needing to link to the runtime library (libcodonrt.so); no ETA but it is on our roadmap (https://docs.exaloop.io/codon/general/roadmap#infrastructure-and-tools).
Aside from that, the -release flag performs optimizations like -O3 on Clang/GCC, which is why the performance and binary size are different.

Thanks for the explanation. Codon has been great so far and helped me complete my assignment brilliantly, looking forward to following up with further work.


Add a reset password page

Adds a reset password page for the snippet to redirect to after confirming their password reset email

I'm having trouble using this. (When I click the reset password link, it just takes me to the login page.)
https://www.loom.com/share/d19fb48644594c648edfc652f04e8f9c

I'm having trouble using this. (When I click the reset password link, it just takes me to the login page.)
https://www.loom.com/share/d19fb48644594c648edfc652f04e8f9c

Can fix this with the same UI for invited user password setup


[QUESTION] session_id suddenly change

hi @gunthercox , I tried to embed session in my flask app, and when I create a session, I save the session_id in my database. but when I pass the session_id in generate_response() , then the function call the get() to get the session, suddenly the session changed (as well as the id) , how to solve this? thanks

@bilalbayasut which version of chatterbot are you using? Use python -m chatterbot --version in your terminal to get version?

0.5.0

There has been updates to correct a few issues with sessions since the 0.5.0 release. Feel free to check the release notes for more details: https://github.com/gunthercox/ChatterBot/releases. The latest release is 0.5.4. I'd recommend updating to the latest version to check if the problem persists.

@gunthercox how to safely update to the latest? thanks

ChatterBot can be updated to the latest version using pip. The command is:
pip install chatterbot --upgrade

can I update it by simply pull origin and merge with my existing one ? thanks @gunthercox

I don't which OS are you using, If it is Linux I would recommend virtual environments,
virtualenv venv
source venv/bin/activate
pip install --upgrade pip
pip install --upgrade chatterbot


EDS plugin filling Koha logs with needless warnings

For the most part, these are warnings about the experimental use of given/when.
Koha doesn't use given/when for this reason, and there have been patches to remove its use in the past:
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11478
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11479
I thought there was a line in Koha's coding guidelines but I haven't found it. It might be one of those things which is implicitly understood in the Koha community but isn't recorded explicitly anywhere.

I've got this patch in #68, thanks @minusdavid !

This looks to be resolved with the 19.05 release to master, but please feel free to reopen if you install the new release and still see this issue.


Get proper code coverage reported

The code coverage currently reported is incorrect. Because of the way go reports on code coverage, it isn't as easy as you'd think to get an accurate total. The report is correct, but the coverage badge needs better data.

This should be taken care of now.


Controlling minimal display time

Thank you for this great library!
There seems to be a minimal display time before please-wait splash screen closes.
In case when my page loads really fast, I have to wait for splash screen for that time and only then will it dissapear. Is there any way to control that, to make it dissapear as soon as finish() is called, even if it that was very quickly after the creation of splash screen?
Thanks

Yeah, the animation durations are controlled via CSS, and the loading screen registers listeners to wait for the animation events to fire before proceeding through it's finishing sequence.
The finish method does take arguments, though (https://github.com/Pathgather/please-wait/blob/master/src/please-wait.coffee#L167). The first argument is a boolean immediately, which when set to true, will cause the screen to disappear without any animations at all, which might be what you want.
e.g.
var loading_screen = pleaseWait({ ... }});

// some time later
loading_screen.finish(true); // finishes immediately, no animations

Thanks @NevilleS, that is it!


ZEVA - supplementary report wrong ldv sales value + error upon issuing assessment

Describe the Bug
On zeva dev, Ford's supplementary report has the wrong previous LDV sales number, and when the director tries to issue the reassessment, an error occurs.
Expected Behaviour
The above shouldn't happen
Actual Behaviour
The issue described two sections above does happen.
Implications
This is not good.
Steps To Reproduce
Steps to reproduce the behaviour:
User/Role: Director on zeva dev:

Go to Ford's supplementary report (MYR id 145, supp id 149)
See/recreate errors as described above

Tested!


Add shell in Jammy tiny stack

we hope to use jammy-tiny-stack when build a java-native-image because it can make the container image size smaller.
But I think shell is an important lib for us because we can use kubectl exec -it xx -- bash for some troubleshooting in our k8s. But I find that currently jammy-tiny-stack image doesn't include shell, is there any way to include this?
Describe the Enhancement

Possible Solution

Motivation

any update?

Hi @zhoufenqin,
I understand the value in having a shell for debugging - I've encountered this myself. But one of the goals of the tiny stack is a minimal image for the smallest possible size and security footprint. That's why there is no shell, and we are not likely to add one to this stack.
I understand it makes debugging harder, but there are built-ins (like ls, etc) which can help. You can also always rebuild the app on the base stack which does have a shell.
Given that, I will close this issue as won't-implement.


Storing binary content on s3

In addition of store files as @Lobs in database and in file system, would be great if lightadmins would allow store files on s3. This feature should need an easy way to declare aws credentials, bucket, filename and file content

@ilopezluna  Do you plan to support storing the filename or path in the relational database then?

@max-dev Honestly I think a more generic way to allow developers to provide custom input data handlers would be more useful. This means that I as a developer would like to be able to process changes for a given property of an entity on my own and provide the result of that processing back to Light-Admin. In this case this would mean that the developer implements a service method that receives the binary data of the uploaded image that stores the data in an S3 bucket. If the user requests to view the data, again a custom handler provided by the developer is executed to retrieve the image binary data from the S3 bucket. This would enable a lot more possibilities than a customized S3 annotation. :/


chore(main): release iso-websocket 0.1.5

:robot: I have created a release beep boop
0.1.5 (2023-10-31)
Miscellaneous Chores

main: release iso-base 0.1.5 (3849a49)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugomrdias/iso-repo/releases/tag/iso-websocket-v0.1.5 :sunflower:


New module: Python bindings

These commits implement python bindings for the c++ API. I've also created the metadata for a v0.13.0 release.
Check out the commit descriptions and the included doc changes for more details.

 I'll re-submit when I figure out what's going on...


nl.rubensten.texifyidea.inspections.RedundantEscapeInspection non unique shortname

Short name 'RedundantEscape' is not unique
class 'nl.rubensten.texifyidea.inspections.RedundantEscapeInspection' in PluginDescriptor[name='TeXiFy IDEA', classpath='/Users/yann/Library/Application Support/IntelliJIdea2017.3/TeXiFy-IDEA']
and
class'org.intellij.lang.regexp.inspection.RedundantEscapeInspection' in PluginDescriptor[name='IDEA CORE', classpath='/Applications/Leda.app/Contents/lib/resources.jar']
conflict
please add some prefix in inspection's short name
IJ 2017.3 Trunk
Texify 0.4

Good catch. Will do!

Will be fixed once PR #125 is merged.

I got the same error. Thanks for fixing it. Can you cut a new release soon?

@cdeil We're working towards a release shortly. See also the Milestone Page for b0.5!

Looks like you guys are very busy. Thank you so much!!!
So far I was always leaving PyCharm for editing latex, looks like now I can stay. :-)

One question: is there a place or forum to ask user questions?
At  https://github.com/Ruben-Sten/TeXiFy-IDEA#texify-idea you're sayin

Feel free to share your ideas and contributions with us. Please bear in mind that this is just a side project for us.

but you don't say where to do this (Github issues I presume?), and if you have capacity for "user question".
E.g. as a new user, the first thing I see is 1000s of warnings telling me that I should put each sentence on a new line, and that I have unresolved references (which isn't correct, I'm just using multiple tex files and it doesn't look like you understand this automatically).
So my questions at this time would be:

is there a way to refactor my tex files to have each sentence on a new line to follow your recommendation or do I have to do this manually?

is there some place where I can configure the project so that it understands that I have multiple tex files?

I'm sure that I'll have a few more questions as I'm using this plugin.
Maybe it's possible that you extend the docs in your README a bit how to use this, and / or set up some forum where one can ask user questions?

@cdeil Oh yes. I have more spare time, so the development quickens :3

is there a way to refactor my tex files to have each sentence on a new line to follow your recommendation or do I have to do this manually?

Yes, there is! This is actually default IntelliJ behaviour. When selecting the quick fix, open the sub menu and select Fix all 'Start sentences on a new line' problems in file (see also picture). This is one of my favourite features as it allows me to refactor files extremely quickly. If you don't want this behaviour, select Disable inspection from the same menu and the inspection will be disabled for that project.

is there some place where I can configure the project so that it understands that I have multiple tex files?

This happens automatically. All files that are included in some way or the other will be regarded as one 'inclusion set'. The detection is vastly enhanced in b0.5. In the future (b0.5+) you can also mark a directory as a resource directory.

Maybe it's possible that you extend the docs in your README a bit how to use this, and / or set up some forum where one can ask user questions?

We are planning on making a CONTRIBUTING.md. We do have a user forum on gitter. It is indeed a good idea to extend the README a little. However, we are uni students.. so time is not always plenty. We are also working on a website.

and that I have unresolved references (which isn't correct, I'm just using multiple tex files and it doesn't look like you understand this automatically).

This is due to a bug (see issue #165) and is fixed in b0.5.

but you don't say where to do this (Github issues I presume?), and if you have capacity for "user question".

Questions can be posted as an issue or on gitter :)
I hope this anwsers all your questions!


please add Cataclysm: Dark Days Ahead, a (very) active fork of Cataclysm

Hi, I'm the project lead for Cataclysm: Dark Days Ahead, a CC-BY-SA 3.0 fork of Cataclysm
Repository: https://github.com/CleverRaven/Cataclysm-DDA
Homepage: https://cataclysmdda.org
Forums: https://discourse.cataclysmdda.org
Language: C++ with extensive modular json data definitions
Please consider this project for inclusion in your list.

Thanks. I added an entry.

Is the status rather beta (playable but pre version "1.0") or mature (most if not all features implemented)?
Is the source code license really CC-BY-SA-3.0 as indicated in the license.txt file? Usually CC is used for artwork, less often for code.
Any other changes you like to make?

We have periodic stable releases with intervening experimental releases.  It's mature but we  continue to add features.
We inherited the CC licensing from the original authour, though it seems we can transition to GPL at some point.
Actually I misspoke, the majority of the code and data is under CC, but misc components are under GPL (font embedding), MIT and Boost licenses.
The only change I request is that the primary build system is Make, not CMake.

Change made. Thanks for the explanation.


Support GiST Index type with Postgres

Problem
I'm trying to use Prisma with geographic data in PostGIS. There's an issue to track actual PostGIS support (#1798) but this is a narrower concern - the support of GiST indexes. These were mentioned when index type support was introduced (#10080), but only BTree or Hash were implemented.
Suggested solution
Allow GiST as an option to the type parameter for an index.
Alternatives
This could be hacked in with an arbitrary migration, but that is not a clean solution.

Can you elaborate on the use cases of GiST indexes outside of PostGIS or full text search?

What about the GIN index?

Can you elaborate on the use cases of GiST indexes outside of PostGIS or full text search?

There are examples in the Postgres docs - GiST - but to summarize, besides Postgis, the you can use GiST for a different performance profile with intarray columns, making them faster to reindex but slower to query, and the GiST index for ltree values supports additional operators that the GIN index doesn't.
However, besides the other use-cases:

You can't use GIN indexes with geospatial data. They'll quickly fail and any attempts to push data in them will produce nearly-unindexed performance and warnings about the index overflowing.
There are a lot of people using PostGIS and geospatial data in Postgres.

So, from that perspective, this is not a nice-to-have for handling geospatial data, it's a necessity.

So, from that perspective, this is not a nice-to-have for handling geospatial data, it's a necessity.

This is what confuses me: We are definitely aware of that, mostly talking about GiST in context of PostGIS. But you created an issue for the usage of the index outside of the context of PostGIS work. I saw now "intarray" and "ltree" mentioned 👍

Oh! Yeah - that’s because there are use cases (like mine) in which GiST index support alone would be enough to make Prisma usable without hacks. We can already specify geometry columns, but can’t load data into them until they have the right index type.

Internal note: https://github.com/prisma/database-schema-examples/search?q="USING+Gist"

This is now available in our dev version, so you can use prisma@dev and the extendedIndexes preview feature if you want to play with it. Optimally start with a db pull on your existing database that uses these, and see if they are added correctly to the introspected schema. Afterwards you can also play with adding additional ones and then migrate (with db push or migrate dev).
Let us know how it goes please!

Are geographic data queries supported in the prisma@dev version for other databases such as PlanetScale? I'm looking for Spatial Query support for PlanetScale.
It looks like both MySQL and PlanetScale support Spatial Queries directly: https://planetscale.com/blog/how-do-database-indexes-work
However, it looks like Prisma doesn't support this per this other thread: https://github.com/prisma/prisma/issues/1798

Yes, this issue here only refers to GiST indexes.


Fixed redirection links in versioned documentation

For example:
https://docs.zendframework.com/zend-expressive/features/router/piping/
redirects to:
https://docs.zendframework.com/v2/features/router/piping/
instead of:
https://docs.zendframework.com/zend-expressive/v2/features/router/piping/
Also removed index.md as we have there index.html.
There is another issue:

https://docs.zendframework.com/zend-expressive/v1/ - works, loads index.md from v1/
https://docs.zendframework.com/zend-expressive/v2/ - shows 404, why? There is index.md in v2/.

Maybe this is not an issue, because we don't have this link anywhere, but I think we should see there something, especially the file is there.
/cc @weierophinney @froschdesign

shows 404, why? There is index.md in v2/.
Maybe this is not an issue, because we don't have this link anywhere, but I think we should see there something, especially the file is there.

This is because there's not an entry for it in the mkdocs.yml. I'll add it when merging.

Also: had to re-instate index.md in the root tree; this is required in order to build, even though we swap it with the index.html.


Create tabs component

Relates to wellcomecollection/platform#4642
There's some new UI in the search wireframes that looks like tabs. We don't currently have a frontend component suitable for this so it'll need building.
It probably makes sense for it to be abstract enough to handle future instances where we want to use tabs, rather than being focused specifically on the search interface. That said, we might need to consider at what point we'd expect the results to update when clicking on each tab as part of this work (cc @DominiqueMarshall).

@davidpmccormick I would expect the results to update when you've clicked the tab, similar to how the radio buttons on our search works now. The tabs are basically just a different visual styling to the radio buttons.


user.js is documented, user.css is not (or hardly)

The page https://help.getferdi.com/features/userfiles/ has what looks like a pretty complete explanation of how to use the user.js code injection file, but almost none about user.css .
For instance, I presume you have to define a scope to set a parameter (e.g. -webkit-font-smoothing: subpixel-antialiased;, to apply to the entire service page content), but there is no list of the available scopes.

Thanks for that help url @RJVB - i hadn't known about this.
@kytwb - that page seems to be really old, and does not reflect the current state of exposed APIs, etc. Could you please take a look to fixing that?


Superglobal array assignment

$this->postData = $_POST;

Should the above trigger this error?
The unit test file doesn't cover this - only accessing keys in the array, instead of the whole array itself.

Although I was the one saying this was most likely a false positive (as it is a simple assignment, the contents are not accessed), I do think that it might not be possible to do a better check.
If this would not trigger the error, the error would have to be triggered on accessing the contents of $this->postData, which is not easily possible to correctly detect.
So, while I do think the above statement is not directly an issue, it might be the only chance of pointing out a potential issue before it gets obscured.

Even if that's the case, I think the unit tests should include such an instance to confirm the error would be triggered.

Closing as the sniff this issue applies to has been deprecated in WPCS 1.0.0 and removed in WPCS 2.0.0.


Part of cross-section not loaded [sectionproperties implementation]

Loading a surface from a rhino file into sectionproperties resulted in not all of the cross-section being loaded.
Code snippet:
from sectionproperties.pre.geometry import Geometry, CompoundGeometry
from sectionproperties.analysis.section import Section

geom = Geometry.from_3dm(filepath="DIES01_POLYLINES.3dm")
geom.plot_geometry()

Below is a screenshot from Rhino (expected behaviour):

Below is the geometry object I get from the above code:

I get a deprecation warning in the terminal when executing the above code, so not sure if that is relevant to this issue at all:
\rhino_shapely_interop\importers.py:410: ShapelyDeprecationWarning: Iteration over multi-part geometries is deprecated and will be removed in Shapely 2.0. Use the `geoms` property to access the constituent parts of a multi-part geometry.
  [LinearRing(line) for line in line_list]

Attached is the offending Rhino file.
DIES01_POLYLINES.zip

@robbievanleeuwen thanks for this. The issue seems to be coming from here. The endpoints of the Brep edges are out (by some precision), so linemerge is not returning a list of LinearRing, but rather a list of LinearRing and a bunch of LineString. I'm working on a fix.

@robbievanleeuwen this should be fixed in master let me know if this works for you and I'll create a new release.

@normanrichardson master branch works for me :)

@robbievanleeuwen great, the updated release is complete.


Spread operator?

Will array spread operator support ever be part of this proposal?  The following of course doesn't work:
async function * generateStuff() { ... }
async function temp() {
  console.log([...generateStuff()])
  // nor does this work
  console.log([...await generateStuff()])
}

 await Promise.all([...generateStuff()]) works fine

Not part of this proposal, which is stage 3 and finalized except for responding to implementation feedback (although that is indeed causing some changes; see pull requests tab).
Another proposal could certainly contemplate something new here.

FWIW @ljharb's use of [...x] is redundant, as Promise.all already uses the iteration protocol.

Wait, is an async iterable even valid for a spread operator?

Ah, good call, thanks @littledan

Wait, is an async iterable even valid for a spread operator?

not in this proposal

Oh yeah, that too.

Huh, I hadn't thought about the fact that something could be both a sync and async iterable.  I wonder then if an await ... operator would be possible?
[
  await ...mixedIterable(), // uses mixedIterable as async iterable
  ...mixedIterable(), // uses mixedIterable as sync iterable
]

@jedwards1211 Async spread seems like a good proposal to start in order to flesh out ideas like that

@littledan @ljharb actually I can't get either await Promise.all([...generateStuff()]) with or without the spread operator to work:
async function * gen() {
  yield Promise.resolve(1)
  yield Promise.resolve(2)
  yield Promise.resolve(3)
}

async function temp() {
  console.log(await Promise.all([...gen()]))
  for await (let num of gen()) console.log(num)
  console.log(await Promise.all(gen()))
}

temp().catch(error => console.error(error.stack))

Output:
$ babel-node temp.js
[]
1
2
3
TypeError: (var)[Symbol.iterator] is not a function
    at Function.all (native)
    at _callee2$ (/Users/andy/toy-factory-tracker/temp.js:148:38)
    at tryCatch (/Users/andy/toy-factory-tracker/node_modules/regenerator-runtime/runtime.js:64:40)
    at GeneratorFunctionPrototype.invoke [as _invoke] (/Users/andy/toy-factory-tracker/node_modules/regenerator-runtime/runtime.js:299:22)
    at GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (/Users/andy/toy-factory-tracker/node_modules/regenerator-runtime/runtime.js:116:21)
    at step (/Users/andy/toy-factory-tracker/node_modules/babel-runtime/helpers/asyncToGenerator.js:17:30)
    at /Users/andy/toy-factory-tracker/node_modules/babel-runtime/helpers/asyncToGenerator.js:28:13
    at process._tickDomainCallback (internal/process/next_tick.js:135:7)
    at Function.Module.runMain (module.js:606:11)
    at Object.<anonymous> (/Users/andy/toy-factory-tracker/node_modules/babel-cli/lib/_babel-node.js:154:22)

@jedwards1211 right, per https://github.com/tc39/proposal-async-iteration/issues/103#issuecomment-306583608 this won't actually work.
I'm not actually sure why async iterators aren't also sync iterables, that yield an IterationResult for a Promise every time.

@ljharb I guess for clarity's sake:

Furthermore, we introduce a new symbol used for obtaining an async iterator from a given object, Symbol.asyncIterator. This allows arbitrary objects to advertise that they are async iterables, similar to how Symbol.iterator allows you to advertise being a normal, synchronous iterable.

@jedwards1211 ah right, thank you - that's why. not that it can't report the correct value (it could return a promise for it), but that it can't synchronously know how many IterationResults to produce, nor when it's done.

One thing we could do, in this proposal or in a follow-on, is make Promise.all and Promise.race accept async iterables as an argument. Maybe this would subsume the use cases of the potential await ... operator.

@littledan if something is both sync and async iterable, do you think it should just error out?

I think most things that are both sync and async iterable would be an instance of a sort of manual sync-to-async iterator. The intention of the language design is that they should do the same thing, right?

I think most things that are both sync and async iterable would be an instance of a sort of manual sync-to-async iterator. The intention of the language design is that they should do the same thing, right?

Of course you can only use async-from-sync iterators with for-await-of and yield*

@littledan yes, hopefully people wouldn't make mixed iterators unless both types iterated the same values.  The only time I can imagine that being useful is if someone is dealing with node-fibers madness and wishes for the sync iterator to use fibers, and the async iterator to work without it.  Still, for the purpose of thoroughness, any Promise.all etc proposal would have to be fully prescriptive about how it handles a mixed iterator

And @caitp it's kind of ridiculous but you can do sync-from-async using node-fibers.

And @caitp it's kind of ridiculous but you can do sync-from-async using node-fibers.

Just FYI, that does not have anything to do with https://github.com/tc39/proposal-async-iteration/issues/103#issuecomment-307092434. In other words, you'd have to implement your own Async-from-Sync Iterator in userspace if you wanted to use one in a context that was not for-await-of or yield* in an async generator, but would not be able to use the builtin one.

@caitp oh I see.  I thought you were implying that sync-from-async is impossible.

@littledan Has a solution to this been proposed yet? Is there further discussion somewhere?

@nayeemrmn
See https://github.com/Misiur/proposal-spread-async/blob/master/spec.md

Anyone willing to champion?

https://es.discourse.group/t/spread-async-with-some-looking-for-champion/273

Any progress on this? It's a natural assumption that the spread syntax will work with promises, and it can lead to a lot of wasted time searching for a bug elsewhere, since this behavior is naturally expected.

Why is that a natural assumption? A Promise represents one thing; spread syntax operates on containers of multiple things.


Increase the limit of attached databases

Hello
The doc says:

SQLite has a limit for attached databases: A default of 10, and a global max of 125

But I can't find a way to increase it with the library API.
Reading the SQLite doc, there is a function sqlite3_limit(db,SQLITE_LIMIT_ATTACHED,size). Is there a way to call it?
Thanks

Maybe there is a compilation flag, otherwise no. There is currently no exposed function to call sqlite3_limit.

Seems like I misunderstood the SQLite doc. Apparently there is a compilation flag SQLITE_MAX_ATTACHED (default 10, max 125). Then sqlite3_limit(db,SQLITE_LIMIT_ATTACHED,size) can be used at runtime only to lower the limit set at compile time.
So the flag should do the work. I will try that and report back.
Thanks for the help!

This on the package.json should do the trick for you.
"op-sqlite": {
  "sqliteFlags": "-DSQLITE_MAX_ATTACHED=125",
}

Closing this. If you have another issue, just comment and I will re-open this.

I confirm that it works on version 3.0.7 by adding OPSQLiteFlags="-DSQLITE_MAX_ATTACHED=125" in android/gradle.properties
5.0.5 doesn't build so I can't confirm that it works with the new way (package.json config).
Btw, is the old 3.0.7 doc still available?

I answered regarding your build issue on the other ticket. Regarding docs, no. You can take a look at old states in this repo to find the README but I moved everything to notion because it is too much work to keep everything in sync.


ws: Use WelcomeEvent to signify succesful connection

@JamyDev, thanks for your PR! By analyzing the history of the files in this pull request, we identified @connor4312 and @SimonSchick to be potential reviewers.


runtime and requirements in binder/ doesn't work

via @hydrosquall in #139:

With that said, this stops working if runtime.txt and requirements.txt are both under the binder/ folder, so I had to get rid of the binder/ folder and put both text files in the project root.
It also doesn't work if just runtime.txt is in the root and requirements.txt is in the binder/ folder, since it will fail to install the python2 kernel. This might be a bug worth investigating separately.

Kinda of related to this: if you have a requirements.txt in the top level directory, and binder/postBuild your requirements file will be ignored. Not sure if this is a feature or a bug

@choldgraf @betatim @yuvipanda I think we discussed documenting the build order and workflow (if file, then...). I can't remember if we did do that so am labeling this a docs issue for now.

the original bug should be fixed I believe, and from a documentation standpoint we mention a bit of this here now: http://repo2docker.readthedocs.io/en/latest/usage.html#preparing-your-repository
though there is no explicit "build order" section

This will be fixed in #277 I believe - we had a hard-coded path to a requirements file that was only in the Python2 buildpack, so would only be uncovered if people both had 2.7 specified in runtime.txt and used a binder/ folder

Fixed by #277.


Create Links “tiktok-44”

Automatically generated by Netlify CMS

👷 Deploy Preview for distracted-hoover-7b89cc processing.
🔨 Explore the source changes: c33d98f19719dc630fad0ae3594da7cd899afde5
🔍 Inspect the deploy log: https://app.netlify.com/sites/distracted-hoover-7b89cc/deploys/6135794cb7c4290007be573f


iOS 10.3.1 fluid disappears

Hello I'm having a problem, when the application is in the background the liquid is disappearing. If anyone can help me. This happens in the version of iOS 10.3.1

Do you code you can post showing the issue?


Discord Webhook/Bot Feature

Plugin Version:
 3.6.12
Request:
 A logging system in Discord that tracks the locations to which a user RTPs to.
How/Why:
 For servers to detect X-rayers and address hackers exploiting Rtping to their advantage, as their destinations are not logged.
Additional Context:

This has been talked about, thinking of making a webapp to import your players stats and display in on a webpage. Made a new module on the github just for this ;) Not in discord bot form, but something similar atleast


missing migrate configuration files.

https://learn.microsoft.com/en-us/azure/azure-monitor/app/migrate-from-instrumentation-keys-to-connection-strings
Article appears to be missing how to set the value when migrating individual .Net core applications and .Net Framework.
How to migrate appsettings.json?
How to migrate applicationinsights.config?

nvm, found to one line that said to look in the How to Set Connection string link


integ-cli: Implement remote FakeStorage server for build via URL tests

Implemented a FakeStorage alternative that supports spinning
up a remote container on DOCKER_TEST_HOST to serve files over
a nginx server so that tests which use URLs in Dockerfile can
build them over at the daemon side.
fakeStorage function now automatically chooses if it should
use a local httptest.Server or a remote container.
This fixes the following tests when running against a remote
daemon (Windows CI):

TestBuildCacheADD
TestBuildCopyWildcardNoFind
TestBuildCopyWildcardCache
TestBuildADDRemoteFileWithCache
TestBuildADDRemoteFileWithoutCache
TestBuildADDRemoteFileMTime
TestBuildADDLocalAndRemoteFilesWithCache
TestBuildADDLocalAndRemoteFilesWithoutCache

Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>cc: @tianon @unclejack @jfrazelle @icecrime @tiborvass

P.S. There's an issue with TestBuildCopyWildcard on Windows even though the required PR #10865 gets merged. I will work on a fix for it soon.

@ahmetalpbalkan Does it really need to be nginx? Couldn't it be a very small image with a Go based web server which serves these files?
In the past, a lot of effort was put into making the tests not need a lot of images, transfers from remote services (GitHub, the Hub, other remote servers, etc), to avoid huge dependencies and anything that would make running a run of the tests less likely to be reproducible.

@ahmetalpbalkan I agree with @unclejack , the FROM nginx is problematic.

@unclejack: Couldn't it be a very small image with a Go based web server

Base golang image is 513 MB vs nginx is 98 MB if we're concerned about being small. Pretty sure I don't want windows CI tests to download 513 MB every time. :)
On the other hand, if we want to run a static go server from command line, we need a main.go file starting the web server which we'll either embed into the test code or have a separate file.

@unclejack: [...] making the tests not need a lot of images

As far as number of the images we use in testing is concerned, we don't currently use a golang image. Therefore having a nginx image is practically the same thing as a golang image.

@tiborvass: [...] FROM nginx is problematic

Why? Do you mean there is a problem with nginx in particular?

@ahmetalpbalkan I'm not saying we need to build an image based off golang. I'm saying we need an image like that hello-world image which is just a binary built using Go.

@unclejack Problem with building a binary for some other platform is if you're on windows (1) you need to bootstrap the go compiler on the CI/dev machine for cross compiling (2) you may worry about the target arch (i.e. 386 vs amd64).
Even though we do this way, it'll add more complexity to this PR but I agree that using nginx image adds another moving part to the test and that's not really something we desire. I'm open to 'more concrete' ideas. :smile:

@ahmetalpbalkan I'm proposing that we build an image with a tiny web server that we pull from the registry and make available elsewhere (GitHub) to not rely on the Hub.

@LK4D4 @unclejack @tiborvass as we discussed, I eliminated need to nginx by adding an .ensure-webserver script which builds an offline-compiled Go web server and put in image called "webserver".
@tianon must review the script (I suck at bash scripting :facepunch:) :blush:
@jfrazelle must bootstrap go compiler on windows ci machine(s) for linux/amd64 sometime. :smile:

LGTM

wait did you update the gitignore for the generated files?

@jfrazelle hmm normally they should be cleaned up. even though they're not, they go to $DEST/webserver, which is bundles/test-integration-cli/webserver and bundles/ is already .gitignore-d.

ah nice
On Fri, Feb 20, 2015 at 5:54 PM, Ahmet Alp Balkan<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle hmm normally they should be
cleaned up. even though they're not, they go to $DEST/webserver, which is
bundles/test-integration-cli/webserver and bundles/ is already
.gitignore-d.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/10893#issuecomment-75349740.

Cross-compiling on the host for the daemon we are targeting is really fragile, but I think overall this is better than the daemon hitting ports in the running go test  instance itself.
I don't see a reason to bother maintaining two implementations though.

@tianon oh I just re-read your comment (I misread on the first time). Even though we're running on same host, building another image and launching a container etc just adds more complexity + flakiness to the test. Another reason is speed, I didn't want to slow down the main CI with building a container etc. But it appears we may sacrifice test speed in favor of not maintaining two implementations. I guess I can do what consensus thinks. Just let me know.

idk wdyt @tianon

Haha, I think we need more than just my and Ahmet's opinions here -- I'm obviously -1 to having two implementations of the same exact functionality (since inevitably, we'll only really test one or the other, or we'll get strange bugs with one that aren't in the other, etc).  I'm just worried that it's going to splinter debugging efforts at some point to have two.
It might very well be that I'm the minority in my views on this. :+1:

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "win-cli/BuildViaDockerfileURL-same-host-fix" git@github.com:ahmetalpbalkan/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

@GordonTheTurtle lol, commits are alreday signed?
In favor of moving this forward and maintaining one solution, I removed the local Go httptest.Server implementation and moved everything to use new remote file server container approach. Please review.

Sorry he's been having problems lately I can't figure it out I even wrote unit tests based off the false negative commits and they pass its freaking weird

@jfrazelle in this case it alerted when 2 signed commits existed and I pushed a third signed one. fwiw.

ok this is weird, when I remove local http server implementation, it started to fail against the daemon running at the same host.
--- FAIL: TestBuildCopyAddMultipleFiles (1.59s)
	docker_cli_build_test.go:776: failed to start file storage container. ec=1
		out=dc5673cf4c4a1f7cc3cf7d85d519ef664bc876623e7773a664e58c76327b372a
		time="2015-02-28T21:47:08Z" level="fatal" msg="Error response from daemon: Cannot start container dc5673cf4c4a1f7cc3cf7d85d519ef664bc876623e7773a664e58c76327b372a: no such file or directory"

First, I thought it's about aufs chmod issue and I removed the RUN chmod +x ./server if we're building webserver image from linux. It didn't fix the error.  It's just weird. And it works fine against remote DOCKER_TEST_HOST on Windows janky CLI.
Shall we just keep two implementations at this point? It's not really a big deal.

@tianon reverted the commit that makes all tests use containerized web server. that was causing issues above. There's not really harm in maintaining two implementations as both are fairly trivial + this is only test code. WDYT now?

Yeah, I seem to be the only one heavily concerned about multiple implementations, so OK. :+1:
It's too bad we don't have #11119, because then we could just use nginx directly instead of having to cross-compile a web server...  :smile:

@tianon nginx was the first solution I came up with. later on we decided not to stick to nginx as it's yet another moving part which can break easily when the contract (e.g. ports, paths) changes. Should we move this to code review?

Well, with #11119, it would be locked to a  specific image ID, so it wouldn't really have the potential for changing out from underneath us without us explicitly changing, but I digress.
I agree this should be at code review!

Another reason for not using nginx is we don't want to download some 90 MB over the network during the test time probably.

ping @tiborvass @jfrazelle to review again

ping @LK4D4 @cpuguy83 for review.

added some new commits to accommodate the new tests added in the meanwhile and suggestions.

@cpuguy83 @tianon moved the server code to conrib/httpserver/server.go, builds from there. much cleaner this way.

LGTM

Ping @tianon

Looks like docker-py on janky is having a superfun time too. :cry:  That should be completely unrelated to this change.

LGTM

@tianon docker-py tests look good now

@ahmetalpbalkan lost some code :D So wait before merge

@LK4D4 fixed now, I rearranged some defers. should be fine now.

LGTM

:confetti_ball: :confetti_ball: :confetti_ball:
:dancer: :dancer: :dancer:


java lang Invalid authCallback response

Given the below snippet a java.lang.Exception: Invalid authCallback response gets thrown.
  final clientOptions = ClientOptions()
    ..clientId = clientId
    ..logLevel = kDebugMode ? LogLevel.verbose : LogLevel.error
    ..autoConnect = true
    ..authCallback = (TokenParams tokenParams) async
    {
      // Custom implementation
      final resp = await remote.createAblyAuthToken(clientId);

      // Perfectly fine response
      final Map<String, dynamic> tokenMap = jsonDecode(resp.data!['token']);
      /*
       {
        keyName: '**********',
        clientId: '*****************************',
        capability: '{"public:data:*":["history","subscribe"],"verified:*":["history","subscribe"]}',
        timestamp:<PHONE_NUMBER>696,
        nonce: '7926367634923936',
        mac: '******************************************'
       }
      */

      return TokenRequest.fromMap(tokenMap);
    }
    ..authMethod = 'POST'
    ..logHandler = ({msg, exception})
    {
      print('$msg $exception');
    };

  final Realtime realtime = Realtime(options: clientOptions);

Setting autoConnect to false and manually await realtime.connect() yields similar behavior.
Looking further into the logs I see the following:
(VERBOSE): io.ably.lib.transport.ConnectionManager: setState(): setting disconnected; reason {ErrorInfo message=authCallback failed with an exception (See https://help.ably.io/error/80019) code=80019 statusCode=401 href=https://help.ably.io/error/80019}
Which doesn't make sense because authCallback returns token response and based on 2.1 here https://ably.com/documentation/core-features/authentication#token-process and Ably is not supposed to establish a direct auth connection (if it did, that'd be a 404 error).

@ben-xD I came across #156, tested the connection and sent a message, it worked despite the exception. I guess the merged PR is not published to pub.dev yet. First time taking Ably for a spin, the exception is indeed scary.  Any ETA as to when this can be published?
Thanks

Hey @1N50MN14,
It looks like you're facing a bug that was fixed by https://github.com/ably/ably-flutter/issues/156. You can expect this bug fix to be released in the next version, which should come very soon (definitely by the end of next week, but potentially by the end of this week).


New VM!

I've completely changed how the virtual machine works. It is now a layered C++ implementation instead of the C implementation. This was done for multiple reasons, with the primary reason being the use of inheritance for creating different types of VMs, and the cool side effect of setting the VM nicely for parallel execution contexts which will come soon.
ALL tests have been updated to use the new vm.
New tests were added that actually assemble some ASM and have scripted step-checks (kind of cool).
There is a new memory structure that no longer requires working around stuffing and un=packing 8-byte frames of a stack. Everything is byte-based which means a few more instructions were added.
There are now 'w' variants of push, pop, stb, and ldb, and the previous instructions changed. To accompany the functionality of moving 'words' of 8-bytes around in-addition to single bytes. This meant that all of the .asm examples had to be updated.
The external devices for the vm were also redone... sort of. Mostly just wrapped in a new ExternalIf virtual interface that can be used to easily extend the VM later on.

Travis got donked on installing CPPUTest. Its just fine


16 degree configuration?

Is it possible to obtain codes for 16 degrees configuration, like 16_AUTO_AUTO and 16_HEAT_AUTO? I've tried to read with a IR receiver but did not succeed. Maybe the binary code can be derived from the rest...

The lowest my remote goes is 17 degrees, that's why there aren't any codes below that.


Fix deploys

Now that we don't have the version in the CDK deployment, the CDK won't have any changes if we haven't updated the infrastructure. This means the app won't be redeployed to pick up the latest jar.
I think we can put it back in now though.

Probably fixed in #51


Subtitles overflow

Describe the bug
I have a video with multiple subtitles (300+) that surpass the screen
To Reproduce
add more than 25 subtitles to a video
Expected behavior
A scroll on the subtitles (optional possibility to set a max height) or subtitles groupping by language
Screenshots or Links

Desktop (please complete the following information if relevant):

OS: Windows11
Browser chrome
Version [98

Affected version
v3.0.0

you can limit it with this
.fluid_subtitles_list {
    height: 195px;
    overflow: auto;
}


xdg-shell: introduce wlr_xdg_surface_state

This is preliminary work for properly handling cached state in
xdg-shell.
cc @vyivel

Ah, I added it myself already :D
Also, the state should probably have configure serial as well.

I've updated this PR to just cherry-pick @vyivel's commit from https://github.com/swaywm/wlroots/pull/3107. Also added a small description of the change in the commit message.

Could you please update this to have wlr_xdg_surface.current as well? surface_commit_state() can be inlined.

Done!


No KeyDown variant in Event enum

Hi, I know it's possible to check if a key has been pressed using the Event enum, but is there any way to check if a key is down (without using another crate) as there is no KeyDown variant in the enum?
I'm trying to implement WASD fly controls, and using KeyPress results in a very jittery motion.

You can just track whether a key has been pressed and not released? In other words, if you at some point have gotten a key pressed event and not a key released event for that key, then it must be pressed currently. Also, the time between frames is not constant, maybe that's why it's a jittery motion.


"MARATHON_SINGLE_INSTANCE_APP" should be built into the API properly

The current implement of the MARATHON_SINGLE_INSTANCE_APP variable is hacky and is hard to use correctly:

Currently it's implemented as a label. It should be a first class citizen of the top level API, e.g. "singleInstanceApp" : true.
upgradeStrategy should be implicit if this field is enabled instead of having to define this manually.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2656. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


@stacksjs/database

Features

Migrations
Seeder
Factories
ORM (Prisma support)
Drivers for Supabase & PlanetScale

https://github.com/stacksjs/stacks/issues/235


Deprecate ClassKeywordRemoveFixer

As requested in https://gitter.im/PHP-CS-Fixer/Lobby?at=612a3c675b92082de16f62b3
Even when the fixer was introduced it was considered "harmful and encouraging bad habits".

Coverage decreased (-0.005%) to 92.312% when pulling 1ab48e2d39fb34e9431ef9196ef56ce08b6fbeac on kubawerlos:deprecate_ClassKeywordRemoveFixer into 0865c64b450590fefa43f8921488222c5361b84d on FriendsOfPHP:master.

Thank you @kubawerlos.

Nice. 👍


Add files.get command and tests

Builds on @nginnever's work, bringing the rest of the implementation to completion.
Requires https://github.com/ipfs/interface-ipfs-core/pull/28 to be merged first to get tests passing.

@noffle I got a:
 1) .files .get with a multihash:
     Uncaught AssertionError: expected 'QmPZ9gcCEpqKTo6aq61g2nXGUhM4iCL3ewB6LDXZCtioEB' to deeply equal <Buffer 12 20 12 0f 6a f6 01 d4 6e 10 b2 d2 e1 1e d7 1c 55 d2 5f 30 42 c2 25 01 e4 1d 12 46 e7 a1 e9 d3 d8 ec>

Seems that on the interface-ipfs-core tests expect two different result types, check:

https://github.com/ipfs/interface-ipfs-core/pull/28/files#r73695467
https://github.com/ipfs/interface-ipfs-core/pull/28/files#r73695554

Addressed! See https://github.com/ipfs/interface-ipfs-core/pull/28#issuecomment-237993885

Fixed the browser tests on https://github.com/ipfs/js-ipfs-api/tree/feat/ipfs.files.get :)

Closed in favour of https://github.com/ipfs/js-ipfs-api/pull/337.