Update containerd

Fixes https://github.com/containerd/cri/issues/1126.
This PR:

Updated containerd to include the event fix for shim v2 https://github.com/containerd/containerd/pull/3207
Add cri as required plugin.

If the required plugin is not there, would containerd refuse to start up?

@yujuhong Yeah. Or if the required plugin is not successfully initialized.

/lgtm


Question: log child machine transitions

I'm new to this library so sorry if I am asking something obvious here.
This is what I want to do:

I want to attach a simple log function to my state transitions. So that I can follow what is happens.
I know there is a way to do this nicely when you interpret and launch your state machine this way:

const promiseService = interpret(promiseMachine).onTransition(state =>
  console.log(state.value)
);

But I have two complications:

Using react hooks:

const [current, send] = useMachine(parentMachine);

I can just log current using  console.log(current); and I will see the state transitions. Is this correct?

How do I attach onTransition to child machines using invoke?
Here I have a parent machine that is invoked using react hooks. The parent machine in turn invokes child machines. How can I attach a state transition logger in the best way so that I can follow the full flow (both parent machine transitions and child machine transitions)?

configuring: {
      invoke: {
        id: 'childMachine1',
        src: childMachine,
        onDone: 'render'
      }
    },

I explicitly do not want to attach "console log" actions to each state. I just want to follow the state transitions in one place globally for a hierarchical set tree of machines.

For now: learning and logging yes. I am build a ui which I think will end up with a handful of machines.

one root machine
one machine that handles initial setup (child of root)
one machine that handles connectivity (connected, retry, error, disconnect). I guess this is parallel somehow
one machine that handles rendering of the ui after inital setup is done (so this will be running as a child to root, in parallel with the connectivity machine I think).

So, in order to follow everything that is happening, I need some tool (console.log) to keep my sanity.
I might be going about this completely wrong though...so any hints/suggestions are welcome.


WIP: Fixing multiple small things

Changed index.js so it has all the logic needed before creating a project
Renamed the "greeter" to "hello-near" across all contracts
Added logic to show messages before and after creating a project
Fixed multiple tests
Fixed package.json generation

TO FINISH:

Fix create-near-app tests
Check what is wrong with near-sdk-js

@gagdiez i took your commit to my other branch


Error "Could not resolve"

Describe the bug
Im getting this error when I build he project in the bolt command line:
Could not resolve "./OrderStatusBadge.vue" from "home/project/src/components/dashboard/orders/mobile/OrderCard.vue?vue&type=script&setup=true&lang.ts"
file: /home/project/home/project/src/components/dashboard/orders/mobile/OrderCard.vue?vue&type=script&setup=true&lang.ts
    at getRollupError (file:///home/project/node_modules/rollup/dist/es/shared/parseAst.js:558:41)
    at Module.error (file:///home/project/node_modules/rollup/dist/es/shared/parseAst.js:554:42)
    at ModuleLoader.handleInvalidResolvedId (file:///home/project/node_modules/rollup/dist/es/shared/node-entry.js:20108:37)
    at eval (file:///home/project/node_modules/rollup/dist/es/shared/node-entry.js:20068:26)

There is a bug because that file exist... I opened in Stackblitz and it work very well, just run:
npm run build
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-fhi6vz
Validations

[X] Please make your project public or accessible by URL. This will allow anyone trying to help you to easily reproduce the issue and provide assistance.

Steps to reproduce

Go to the bolt terminal
Run the command "npm run build"

Expected behavior
It must build successfully.
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 86842847,
  "usedJSHeapSize": 81590603,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = joansafr
Chat ID = 5291f610eca0

Additional context
No response

Hi,
Your project is not public, I can't open it.


Cannot build argument constructor when deriving Command

Hi, I believe this could be a bug but also could be a mistake of my behalf.
namespace App\Domain\Model;

data Isbn = Isbn { string $isbn } deriving (ToString) where
    Isbn:
        | strlen($isbn) !== 10 | strlen($isbn) !== 13 => 'Invalid Isbn length'
    ;

data Title = Title {string $title} deriving (ToString);
data Book = Book { Isbn $isbn, Title $title };

namespace App\Application;

data BorrowBook = BorrowBook { \App\Domain\Model\Book $book } deriving (Command);

Throws the following error Exception: Cannot build argument constructor for App\Domain\Model\Book%

Your example is invalid. Unfortunately fpp is not smart enough to tell you exactly what you did wrong, but here is a working example:
namespace App\Domain\Model {
    data Isbn = Isbn { string $isbn } deriving (ToString, FromString) where
        Isbn:
            | strlen($isbn) !== 10 || strlen($isbn) !== 13 => 'Invalid Isbn length'
        ;

    data Title = Title {string $title} deriving (ToString, FromString);

    data Book = Book { Isbn $isbn, Title $title } deriving (ToArray, FromArray);
}

namespace App\Application {
    data BorrowBook = BorrowBook { \App\Domain\Model\Book $book } deriving (Command);
}

Thanks for the working example @prolic. Is there any plans to get fpp smart enough to tell a user when they make a mistake like the above?


Cleaner API for utilizing all GPUs if available

Description
Can we have a cleaner way of utilizing GPUs?
Current scenario
According to this [1]
from mxnet import npx
npx.set_np()
num_gpus = npx.num_gpus()
ctx = [mx.gpu(i) for i in range(num_gpus)] if num_gpus > 0 else [mx.cpu()]

Equivalent in PyTorch would be
device = torch.device("cuda" if torch.cuda.is_available() else "cpu")

Proposal
To make this easier for first time user of MXNet or (PyTorch user migrating to MXNet)
Can we have 2 changes

Additional API
mx.gpu().is_available()
or something on similar lines

Benefit - Adding this API would save user 3 lines of code and using npx.num_gpus() >1 is a round-about way of saying cuda.is_available()
Created a discussion on discuss forum for the same [2]

Additional boolean parameter
mx.gpu(all_gpus=True)

Benefit - Adding all_gpus parameter is convenient than [mx.gpu(i) for i in range(num_gpus)]
References

https://gluon-cv.mxnet.io/build/examples_classification/transfer_learning_minc.html
https://discuss.mxnet.io/t/cuda-is-available-in-mxnet/5096

I think it would be better to introduce a separate function called mxnet.all_gpus(): List[mxnet.Context], instead of adding a parameter to mxnet.gpu. This way, the return type of mxnet.gpu will remain mxnet.Context, instead of becoming Union[mxnet.Context, List[mxnet.Context]].

@ChaiBapchya @nickguletskii There's one (function)[https://github.com/d2l-ai/d2l-en/blob/d9eb3df062a3abf6bb0af21a022a03edd79badb9/d2l/d2l.py#L300-L303] in d2l-en that could be a possible reference.


Redo command restrictions

Make them work as modules instead, meaning removing per-channel restrictions to full on guild-wide.
This will probably make people mad.

Discord is helping with this; consider removing command restrictions in v5


When I use 6* Tesal T4 (15GB) GPU  resources to run sqlcoder2 model and enter a question like  what is the total number of the table, the model query is very slow and takes about 10 minutes. Is it because my gpu resources are not good?Should I use A100(80GB) gpu?)

Hi there, it seems like the model is using the CPU instead of GPU. We've generally seen inference speeds of ~8-10 seconds on 4xTesla T4 GPUs. Would recommend taking a look at your CUDA configurations.

Hi there, it seems like the model is using the CPU instead of GPU. We've generally seen inference speeds of ~8-10 seconds on 4xTesla T4 GPUs. Would recommend taking a look at your CUDA configurations.

As shown in the picture, it shows that the each  GPU resources are already in use, and a total of more than 50 GB of memory is spent. Is it still that the GPU resources are not used?Should I use model=model.cuda()  to  let all model  run in the gpu?

Not quite sure what the issue might be. It could be an issue with how your GPUs are connected, but no idea what else could be causing this.

Not quite sure what the issue might be. It could be an issue with how your GPUs are connected, but no idea what else could be causing this.

Do you mean that the multi-card capability of the GPU is not used? Generally, if I want to use multiple gpu cards to run the model for inference, what modifications need to be made to the inference.py file?


emacs in non-gui mode (-nw): how to accept copilot suggestion?

Apologies if this is off-topic or if the answer ends up being idiosyncratic, but pressing "tab", which works to accept copilot suggestions in my GUI emacs window, does not work when using Emacs with -nw option. Instead, it makes the suggestion go away.
Would this happen to be something to do with the plugin??

To determine which key you pressed and the command it is bound to, use M-x describe-key.

Oh, thank you very much.
It says : indent-for-tab-command &optional ARG
So maybe it's possible for my settings to have different bindings for gui and non-gui emacs?
This is what's in my init.el, which makes things work in the GUI!
(defun my/copilot-tab ()
  (interactive)
  (or (copilot-accept-completion)
      (indent-for-tab-command)))

(with-eval-after-load 'copilot
  (define-key copilot-mode-map (kbd "<tab>") #'my/copilot-tab))

What is the key you pressed? I guess that it is TAB rather than <tab>.

Thank you for the hint!
So :

TAB = Ctrl-i
 is more of a virtual concept in emacs rather than a physical key. They can be mapped separately sometimes.
(Typically?)  is mapped to Ctrl-i in the GUI but not in terminal; see https://github.com/syl20bnr/spacemacs/issues/4024#issuecomment-161122099
Changing <tab> to TAB in my init.el solves the problem
I suggest it might make sense and be harmless to assign both keys in the default installation?

Other references:
https://stackoverflow.com/questions/26195884/tab-v-tab-in-emacs


[bug] cmake install does not work properly on Windows

The install command creates a $(pwd) directory on windows:
cmake --install i
-- Install configuration: "Release"
-- Installing: C:/Users/danimtb/repos/flappy-frog/build/$(pwd)/install/bin/mygame.exe

Moreover, it does not copy the assets to the installation folder so the games fails to execute at the very begining

This was probably caused due to wrong instructions for windows on the readme file:
cmake .. -DCMAKE_MODULE_PATH=$(pwd) -DCMAKE_PREFIX_PATH=$(pwd) -DCMAKE_TOOLCHAIN_FILE=conan_toolchain.cmake -DCMAKE_INSTALL_PREFIX:PATH=$(pwd)/install


Do not allow simple n2k provider with no serial port

Current way of picking the serial port leads easily to simple providers with no serial ports, that show as a bogus error. API should not allow it and UI should be improved.

API improvement in #915


Order Received page issue

The order received page shows Coinbase data even when payment through stripe . see here in this test order i made

This can be fixed by modifying the function that actually generates the code to run a check.
Function is located on line 123 in coinbase-commerce.php
It is called cb_order_meta_general.
It can be modified as follow:
`/**

Add order Coinbase meta after General and before Billing

@see: https://rudrastyh.com/woocommerce/customize-order-details.html

@param WC_Order $order WC order instance
*/
function cb_order_meta_general( $order ){ ?>
get_meta( '_coinbase_charge_id' ); ?>

Coinbase Commerce Data

Coinbase Commerce Reference # get_meta( '_coinbase_charge_id' ) ); ?>

}`

Pasting that in my functions.php file creates a 500 error. Are you sure about the code? Hope to see as an option in the plugin soon!

Hey @topraf , you should not paste it in your funcitons.php. Instead, you can modify as mentioned in my comment above:
"Function is located on line 123 in coinbase-commerce.php"
I am using it in a production environment with no issues.

Ah yeah ok; thanks! I'll keep the issue open so it can be implemented in the next version of the plugin

@topraf, this is fixed in the latest code, please stay tuned for the newest release.


Vulcanization is not working

I have used lazy loading Polymer.Base.importHref().After that vulcanization is not working.Without doing vulcanization it was taking more time to load landing page.

Are you using polymer-cli? If so, you should specify all elements being imported lazily as fragments in your polymer.json file, see this guide.

I have not use polymer-cli.But i have specified all elements in polymer.json.

Which versions of Polymer and vulcanize do you use?

Polymer 1.6.1

I can suppose a few reasons, maybe I could help if you create a repro in a separate github repo (as small as it would be enough to reproduce).
Also, consider using polymer-cli (or polymer-build if you prefer to write gulpfile yourself) instead of just running vulcanize from the command line.
BTW submitting issues to corresponding github repos is a good idea, because Tools team will look into them quickly.

This issue was moved to Polymer/polymer-bundler#456


Bibtex-cite plugin jumping to top of page

Bug Report
Description
bibtex citations now link to the top of the page when clicking in the article; however, copying the link and going to that address separately will bring you to the appropriate citation
Steps to Reproduce
Steps to reproduce the behavior:

Go to https://www.algorithm-archive.org/contents/computus/computus.html
Click on any citation
past link into browser to find correct citation

Notes
Potential fix for honkit (bibtex-cite plugin): https://stackoverflow.com/questions/3252730/how-to-prevent-a-click-on-a-link-from-jumping-to-top-of-page
Environment

OS: Arch
Browser: Firefox, QuteBrowsery
Browser Version: [88,v2.2.1]

For Algorithm Archive Developers

[ ] The bug can be reproduced
[ ] The bug can be fixed (if not, please explain why not in a comment below)
[ ] There is a timeline to fix the bug
[ ] The bug has been fixed (Please link the PR)

Resolved in #815


Auto-Complete Suggestions only after a Custom Defined Number of Characters

There was a Feature Request of this, previously, that is now closed. I am Re-Submitting the Request.
It is a waste of time to receive Suggestions after only typing 1 Character when the nature of your Code has you typing 1 Character Functions. Having to constantly press Escape to Close the Suggestion Window. And because I see the value of having Suggestions of Variables, Functions, etc. I am not interested in setting the Suggestion Delay at a higher value and then have to wait for the Suggestions to Popup, when half of the time it would be faster to perform a Script-Wide Search to find what I'm looking for. Same Result = No Time Gained.
Also when considering that a Program like Notepad++ has this setting implemented, it is reasonable to assume that this is not something very complicated to implement in Visual Studio Code.
Previous issue:
https://github.com/microsoft/vscode/issues/113305
Regards

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Updated README - corrected Perimeter Church Mission Trip Listing

Chris, the README had a bad url for Perimeter Church Mission Trip Listing. I updated the link to go to the custom widget setup they are using.

Good catch - the original link was from before they launched.


Allow plain connection

Currently the library tries to negotiate SSL/TLS automatically - but on some systems certs are not installed nor is SSL/TLS desired.

NVM, this can be solved with the InsecureSkipVerify flag


Add capa both contract version for consistency

What this PR does / why we need it:
This shouldn't really be needed, but it's consistent with capa repo and might catch edge cases.
https://github.com/kubernetes-sigs/cluster-api-provider-aws/blob/8da95554b65f675c11fc3ac02d3e5503407e0090/config/crd/kustomization.yaml#L4
https://github.com/kubernetes-sigs/cluster-api/blob/f784f1e9e93bec81cc9fc60e82549289ca79c6c0/util/conversion/conversion.go#L64
https://github.com/kubernetes-sigs/cluster-api/blob/f784f1e9e93bec81cc9fc60e82549289ca79c6c0/util/conversion/conversion.go#L96
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/hold

/test verify


Added missing inline error for 'source' field

Adds missing inline error for 'source' field.

Demo https://2508-84182601-gh.circle-artifacts.com/0/artifacts/build/index.html
Fixes #653

Looks good :+1:


Fix: MuxBearerClosed socket closed when reading data

Closes #1632.
client.abort() causes following error at cardano node cardano.node.LocalErrorPolicy:Error:182 IP LocalAddress "/socket@58" ErrorPolicyUnhandledApplicationException (MuxError MuxBearerClosed "<socket: 58> closed when reading data, waiting on next header True"). If remove this call, it works fine.
When making this edit, I was guided by:

Sebastian's comment in #1632 - "client should send MsgRelease and MsgDone after the query"
Ouroboros network spec, section 3.12 Local State Query mini-protocol, Figure 3.8: State machine of the Local State Query mini-protocol.
Examples from Pallas library, in examples used same type of Client - NodeClient, containing multiplexer inside. And there is no abort call at the end of example. Although in this example we see only client.send_release() and no send_done as how is it done in Mithril and as Sebastian advises.

@falcucci what is the purpose of the client.abort() in the PallasChainObserver?
Is it safe to remove it or do we need to replace with a different command to gracefully close the connection to the Cardano node?

It remotely shotdown the current task so it might happen instantaneously and interrupt a parallel task that is running or even somehow breakes the message sent by the server if there is something idle to be sent like a block, pending response or even a ping pong message.
I think it's reasonable to replace the approach to close the plexer client (if it's really needed at our use case) to avoid hanged connections only
Maybe we could guarantee we abort the client when and if the mithril instance is killed, so it would be enough.
If this isnt a risk (hanged connections) this pr looks good to me.

I'll made some experiments tomorrow regarding graceful shutdown of muxer and demuxer. I have a suspicion that by calling abort we interrupting tick method in the middle of message sending/receiving, which can cause socket closed error on server side.

@scarmuega Please see aboves comment on the tick insight.

@abailly-iohk

This error is raised here in ouroboros-network and is caused by the fact the MuxBearer tries to read from the underlying Socket which returns an empty ByteString.
Tracing the call to the network library lead me to the low-level recv function whose documentation says:
-- | Receive data from the socket.  The socket must be in a connected
-- state.  This function may return fewer bytes than specified.  If
-- the message is longer than the specified length, it may be
-- discarded depending on the type of socket.  This function may block
-- until a message arrives.
--
-- For TCP sockets, a zero length return value means the peer has
-- closed its half side of the connection.

I followed the call tree a bit further down and ultimately it ends in C-land with the c_recv function which actually returns 0 if the socket is closed by the peer. There's even a test for that!
So it seems what happens is that the client closes the connection, which leads to this exception being raised in the Mux code which terminates the Mux and all the mini-protocols.

@scamuerga @falcucci Can you point me to the client code?

@abailly-iohk here you can check the client source code from pallas side.


Freezes while loading model.

In c# the line model=Model("[model path"])  freezes until the model is loaded.
Can I suggest having a LoadModelAsync function to load the model in the background so I can display a loading screen? (I am using Unity and c#)
Never mind, I have managed to do it by running this in a Task. It seems to work.
The same with:
generator.ComputeLogits();         generator.GenerateNextToken();
I also run those in a Task and it seems to work OK. But I'm not sure if this is the correct way to do things.

Yep, that's the correct way. As we use onnxruntime to load models, it has no asynchronous way to load them, so you did the best thing which is to put it in a background thread.


Remove all 'null' usages in code

Takes advertises itself as a library without any null. That's not true, I see at least 26 null usages in Java code.
I'd expect removing null from as many places as possible and replacing them with alternatives.
If some external API actually demands null and there's no workaround, we need to admit it in README, and say about few exceptions related to bad external APIs.
I'm fine to solve that issue with PDD approach.

@mkordas I am aware of the task, give me some time to find a developer...

@mkordas I set milestone here to 1.0, let me know if it is wrong

@mkordas thanks for this report, I added 15 mins to your account, in transaction 56ac901105fa01580100020c

@happy-neko the task is yours,please go ahead

We should make clear what exactly we consider null-free design project-wise. From my point of view:

Every public API method we expose should not return null. Alternatives are throwing exception or returning nullable container if return value is optional (Optional in guava / java 8, in takes we have similar Opt<T>).
Do not accept null values as method parameters values in out API (up to debate). Alternative approach is to mark parameters with @NotNull  / @Nullable annotations for self-documentation and static analyser purposes.

Additional consideration:

We could not stop using 'null' internally because this is how language is designed and core Java library and almost every external library uses it extensively. Workarounds here are pointless.

I have reviewed takes code (except unit tests because they do not expose public API) and found no null usages that violates these principles. So it is pretty safe to say that "null-free" is true.
@mkordas what do you think?

@Happy-Neko please see http://www.yegor256.com/2014/05/13/why-null-is-bad.html - it states that null is bad  in general, not only in public API

@mkordas The problem is we can do nothing if foreign API method can return null or accepts it as parameter.

@Happy-Neko we can use Opt or Optional in such cases

we can use Opt or Optional in such cases

@mkordas We still need to test for Optional value presence every time we made null check before.
This is hiding the problem under extra layer of code complexity imho.
@yegor256 What do you think?

@Happy-Neko I disagree with this:

We could not stop using null internally because this is how language is designed and core Java library and almost every external library uses it extensively. Workarounds here are pointless.

We can avoid using null everywhere. Let's try to do it. It's a matter of principle :)

@Happy-Neko I share the same view as @yegor256 - let's try at least.
Moreover, can you add name tag to your comment https://github.com/yegor256/takes/issues/555#issuecomment-180811432?

@yegor256 @mkordas Ok.

@davvd We need more time here.

@mkordas First step done.
I have implemented Opt.Nullable container to store possible-null value returned by library methods (PR #610).
Next puzzles:

Wrap all library methods that may return null in Opt.Nullable #608
null value should not be placed in Opt.Single container #609

@davvd We need more time here.

@happy-neko sure, thanks for telling me, take your time

@0crat in

@0crat in (here)

@yegor256 Job gh:yegor256/takes#555 is in scope.

Job gh:yegor256/takes#555 assigned to @caarlos0, please go ahead (policy).

@caarlos0 this job was assigned to you 8 days ago.
It will be taken away from you after 10 days from start (this is our policy).

Oops! Job gh:yegor256/takes#555 is not assigned to anyone.

Oops! Job gh:yegor256/takes#555 is not assigned to anyone.

The job #555 assigned to @neonailol/z. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

@mkordas there is no public api that can return null in project. Opt.Nullable was rejected in favor of Opt.Empty/Opt.Single and it's already widely used in library, i believe this task can be closed.

@yegor256 almost 10 days passed, and no response from the ticket reporter, i think this task can be closed

@mkordas @neonailol closing.

Order was successfully finished: +30 points just awarded to @neonailol/z, total is +555

The job #555 is now out of scope


Simple Dockerfile mode: provided Dockerfile is always skipped

FROM fakebaseimage
ENV JAVA_APP_DIR /deployment123
COPY maven /deployment123/
EXPOSE 8080 8778 9779
CMD echo "Hello World"

I provided this Dockerfile on the project root directory.. It is always skipped be it  Zero Config mode.
or XML config mode.
This is where the SImpleDockerFIle mode is checked for. https://github.com/fabric8io/docker-maven-plugin/blob/master/src/main/java/io/fabric8/maven/docker/AbstractDockerMojo.java#L346 .
FYI, it works fine in DMP, so the issue is on the FMP side.
Documentation : https://maven.fabric8.io/#simple-dockerfile-build

Will be fixed in eclipse/jkube#197


Copilot can't see any code if too much is highlighted

There seems to be a hidden line limit for the amount of highlighted code that can be sent with a chat request. Anything beyond that and Copilot acts like nothing is highlighted (despite the fact that the chat UI indicates the file and all the highlighted lines were actually sent):

This leads to a lot of unnecessary confusion and frustration. There either needs to be clear feedback when the number of highlighted lines is too high, or preferably the API needs to be improved so it can handle more code at once.
The latter would be a much better option - Claude 3 Opus can handle huge amounts of code, and as a result can have much more meaningful interactions when dealing with large sections of a codebase. I'm finding Claude to be far more capable than Copilot in this respect. Copilot needs to catch up or it'll be left behind.

We've increased the token window quite a bit and made better efforts here to make Copilot work with large documents. Give it a try and let us know what you think 👍


no $HOME environment variable

Breaks situations where ansible depends on it. Mostly when it expands to write to temporary files that usually go into $HOME/.ansible/tmp which gets 'expanded' to /.ansible/tmp that doesn't exist and is owned by root

This is fixed now, no?

Not in systemd...


Create Terraform vars for test/dev cluster, CP4D, etc for release branch 4.6

Fixes: #217
Signed-off-by: Aishwarya Kamat<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: aishwaryabk
To complete the pull request process, please assign bpradipt after the PR has been reviewed.
You can assign the PR to them by writing /assign @bpradipt in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @aishwaryabk. Thanks for your PR.
I'm waiting for a ocp-power-automation member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: aishwaryabk, yussufsh
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [yussufsh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: aishwaryabk, bpradipt, yussufsh
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [bpradipt,yussufsh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Image not mapping volumes

Hello,
I'm deploying the image on AWS via Elastic Beanstalk, here is my Docker.aws.json
{
  "AWSEBDockerrunVersion": "1",
  "Ports": [
    {
      "ContainerPort": "8080"
    }
  ],
  "Volumes": [
    {
      "HostDirectory": "/var/jenkins_home/",
      "ContainerDirectory": "/var/jenkins_home/",
      "readOnly": false
    }
  ]
}

The Dockerfile itself contains only the image and exposes 8080. Just wanted to ask if anybody else had problems mapping the volumes? This is the only image I'm having trouble with - neither /var/jenkins_home is created on EC2, nor volume is being mapped even if I created the directory before that. I'd like to provide you with more information, but I don't see any errors in the logs such as permission denied.
I understand this issue might not be directly related to the image, but having problem only with this one in a pretty big setup makes me think that there's something I'm missing.
Thanks.

questions should go to mailing list


Hass.IO to IFTTT Messages failing to send

Home Assistant release with the issue:
0.94.3

Operating environment (HassOS/Generic):
hass.io running on Ubuntu 16.04

Supervisor logs:
Error communicating with IFTTT
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/urllib3/contrib/pyopenssl.py", line 472, in wrap_socket
    cnx.do_handshake()
  File "/usr/local/lib/python3.7/site-packages/OpenSSL/SSL.py", line 1915, in do_handshake
    self._raise_ssl_error(self._ssl, result)
  File "/usr/local/lib/python3.7/site-packages/OpenSSL/SSL.py", line 1647, in _raise_ssl_error
    _raise_current_error()
  File "/usr/local/lib/python3.7/site-packages/OpenSSL/_util.py", line 54, in exception_from_error_queue
    raise exception_type(errors)
OpenSSL.SSL.Error: [('rsa routines', 'RSA_padding_check_PKCS1_type_1', 'invalid padding'), ('rsa routines', 'rsa_ossl_public_decrypt', 'padding check failed'), ('asn1 encoding routines', 'ASN1_item_verify', 'EVP lib'), ('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 467, in send
    low_conn.endheaders()
  File "/usr/local/lib/python3.7/http/client.py", line 1224, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/local/lib/python3.7/http/client.py", line 1016, in _send_output
    self.send(msg)
  File "/usr/local/lib/python3.7/http/client.py", line 956, in send
    self.connect()
  File "/usr/local/lib/python3.7/site-packages/urllib3/connection.py", line 370, in connect
    ssl_context=context)
  File "/usr/local/lib/python3.7/site-packages/urllib3/util/ssl_.py", line 355, in ssl_wrap_socket
    return context.wrap_socket(sock, server_hostname=server_hostname)
  File "/usr/local/lib/python3.7/site-packages/urllib3/contrib/pyopenssl.py", line 478, in wrap_socket
    raise ssl.SSLError('bad handshake: %r' % e)
ssl.SSLError: ("bad handshake: Error([('rsa routines', 'RSA_padding_check_PKCS1_type_1', 'invalid padding'), ('rsa routines', 'rsa_ossl_public_decrypt', 'padding check failed'), ('asn1 encoding routines', 'ASN1_item_verify', 'EVP lib'), ('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])",)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/ifttt/__init__.py", line 69, in trigger_service
    res = pyfttt.send_event(key, event, value1, value2, value3)
  File "/usr/local/lib/python3.7/site-packages/pyfttt/sending.py", line 12, in send_event
    return requests.post(url, data=payload)
  File "/usr/local/lib/python3.7/site-packages/requests/api.py", line 116, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/api.py", line 60, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 646, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 498, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ("bad handshake: Error([('rsa routines', 'RSA_padding_check_PKCS1_type_1', 'invalid padding'), ('rsa routines', 'rsa_ossl_public_decrypt', 'padding check failed'), ('asn1 encoding routines', 'ASN1_item_verify', 'EVP lib'), ('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])",)

Description of problem:
I am noticing every time I send an ifttt.trigger I am seeing an error in the logs, and the message is failing to send.

A little more info. If I manually send the event and spam it like 15 times it will eventually go through.


Accessibility - VoiceOver does not read deleted character on macOS

VSCode Version: 1.25.1
OS Version: macOS High Siera 10.13.6

Steps to Reproduce:

Turn voiceover on (command + F5)
Launch the VSCode
Create a file
Get the code editor in focus by pressing command + shift + E
Interact with the text by pressing option + ctrl + down arrow
Turn quick navigation off by pressing left and right arrows at the same time (you will hear "quick nav off")
Now, type some code or any kind of content and than try deleting character by character using backspace

Expected result: Voiceover should read the deleted character unless that feature is turned off in voiceover settings.
Actual result: Voiceover reads nothing, it basically stays quiet.

Does this issue occur when all extensions are disabled?: Yes/No
Yes

This works with latest electrno-4-0. And works nicely on the web
Thus closing
@adincebic if you can still reproduce the issue with latest vscode insiders let us know and I will reopen https://code.visualstudio.com/insiders/


Default to llvm-lib when using clang-cl in msvc environment.

The problem is that the vendor librarian can't handle object modules compiled with clang-cl -flto. llvm-lib on the other hand can handle any object modules, which makes it a preferred option.
At @ChrisDenton's request.

Thanks! I'd like to take a little bit of time to run some tests locally but otherwise this looks good.


Accessing poem request data in async-graphql context

Curious how you would pass request data (for example, Bearer Token from headers) into an async-graphql endpoint?

The following is an example of how to get the token from the header and pass it to async-graphql.
struct Token(String);

#[handler]
async fn graphql_handler(
    schema: Data<&StarWarsSchema>,
    req: Json<Request>,
    headers: &HeaderMap,
) -> Json<Response> {
    let token = headers
        .get("token")
        .and_then(|value| value.to_str().ok())
        .map(|value| Token(value.to_string()));
    Json(schema.execute(req.0.data(token)).await)
}

Perfect, thanks @sunli829 🔥


Chat message send icon was not refreshed

Overview
The chat message send icon was not updated after the result was copied from the search.
Issue Statement:
When the chat user select a message from the search result, the message will be copied to the message window awaiting the user to click to send the message. The icon for the send message was not change to send icon but it shows attach icon instead.

 Assistify.Chat Issue #62: Chat message send icon was not refreshed

Fixed on Assistify-side - PR towards RC is a new issue


Django: Getting inline Jinja tags inserted into a text field need to parse

When an Admin Dashboard user enters Jinja tags inside an HTML text field, a web address location such as ,  Django needs to redirect the web visitor to the right address location.
More details about what I am trying to accomplish and for a general description from an experienced Django developer in the community can be found here:  https://forum.djangoproject.com/t/dynamic-hyperlinks-in-textfield-redirecting-routing-to-correct-location-remotely-and-locally/10073

The solution to this one does not require any code changes or a PR. Writing static URLs inside TextFields works this way for example:

You are welcome to have a look at our annotated Essay List with summaries for all the content on our website.

The above will open the essay list summary page locally and remotely. The above can be used instead of this:

You are welcome to have a look at our annotated Essay List with summaries for all the content on our website.

Thanks Abu!


Missing 3AM from the timepicker

Here is my setup:
$('#timeStart').datetimepicker({
format: 'h A'
});

Go here: http://jsfiddle.net/Eonasdan/0Ltv25o8/
Open the first picker.
Go to time picker.
Switch to AM.
Click on the hours.

it's there for me. I need more details, please. Browser, OS, time zone, locale

Chrome - latest, Win8, (UTC+02:00) Helsinki, Kyiv, Riga, Sofia, Tallinn, Vilnius, locale - bg

can you inspect element where the 3 is supposed to be and see what it does.
You'll need to turn debug:true on so that it doesn't close on you

the td is not present at all. on each row there are 4 td, on the first one there are only 3

have you tried the dev branch?


chore(main): release 1.2.3

:robot: I have created a release beep boop
1.2.3 (2023-10-16)
Bug Fixes

no sonar bot message (4291806)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/psioz-org/go-util/releases/tag/v1.2.3 :sunflower:


How to phrase force push / overwrite?

I tried to find out how to force push using phrase v2 cli in the documentation:

https://phrase.com/cli/
https://help.phrase.com/help/phrase-in-your-terminal
https://developers.phrase.com/api

but I don't find a way.

Hi @WaKeMaTTa,
what exactly do you mean by force?
Our push and pull commands are not protected in any way so pulling will override and pushing as well.
You can specify a parameter for pushing to either update translations (default) or not:
push:

    sources:

    - file: ./locales/en.json

       params:

        update_translations: true

https://help.phrase.com/help/configuration

Thanks. It's what I was asking for.
But I was expecting an option on the command, something like phrase push --update-translations.

It would be nice if update-translations had an option to only update keys with translations that have not already been registered on this key before this way you an work asynchronously both in file/code and in phrase and not mess up each-others keys so much.


Refactor tests to use pytest conventions

@wesm, this PR is an example of how pytest is typically used.  I'd like to get your feedback on whether this is something we could do going forward in ibis.
I've ported enough code to the pytest way of things to make the current expression, sqlite and postgres test suites pass (i.e., I haven't run against a real Impala instance)
Pros:

Less repetition with fixtures
Parameterized fixtures run each case of a series of repetitive tests as a single test, which makes the failing cases easier to track down
Easier to specify the difference between failure due to lack of implemenation (xfail) vs expected failures (pytest.raises)

Cons:

A bit of a perf penalty for the extra abstraction 3.34s for the master to 4.00s with this PR
A bit magical to have fixtures specified as function arguments if unfamiliar with pytest
Can be difficult to track down origin of failures during test discovery phase (this was my personal experience and there may be a better way since the last time I looked into this)

Impressive @cpcloud! I've only recently learned to use py.test fixtures, and I love them. The apparent magic is worth being aware of though.

@tdhopper thanks! Balancing the magic I think is particularly important for onboarding new devs. Py test is less straightforward to grok at first glance.

This is great, though I worry I may have given you carpal tunnel with this refactoring. Let me verify the patch with a full test run (including on Impala, wish this were easy for anyone to do) and merge

I think it's a good idea to move all the tests to use pytest conventions, mostly a function of my laziness that I didn't do it when I started working on the project.


mk-docker-opts.sh: replace with busybox shell compatible version

Tested with lots of different options and it works just fine.
It avoids us needing to ship bash in the container.

Closes #417

@tomdee
Would be awesome to have a test that uses a predefined env file and tests against expected output options. Can I motivate you to add such a test?

I really don't think it's worth having tests for this. It's bad enough having this ugly piece of functionality, but adding a test and a way to run the test is only going to make it harder to remove. A better long term approach would be to wrap this functionality into flanneld itself, but it's not a good use of by time to do that at the moment.

@tomdee I haven't touched this script (using or writing) at all. But I think that as long as it's hosted in this repository and users rely on this to use flannel with Docker it's worth to ensure it's functionality. E.g. it's used in CoreOS directly https://github.com/coreos/coreos-overlay/blob/master/app-admin/flannel/files/flanneld-rkt.service#L48, so we definitely want this to work as long as it's there.

@steveeJ I've added a test script. PTAL.

@tomdee

I've added a test script. PTAL.

Fantastic, thanks! Shall we run it on travis?

OK, updates pushed @steveeJ PTAL.

OK, indentation fixed. @steveeJ PTAL

@tomdee I think this one is almost good to go. The last steps before merging IMO are:

enable the mk-docker-opts-test in travis
split the mk-docker-opts rewrite / test into a separate commit

It is already being tested on travis. I'm not sure what you mean by splitting it into a separate commit?

@tomdee somehow I was tricked by my own thoughts, I actually meant the exact opposite: squashing the commits.

OK, squashed. I'm going to merge this when it goes green.

LGTM


Can't continue to pipe output of exec

Hiya! I am trying to use gulp-exec to run a PHP compilation that continues to stream to a minified then a file.  I can't seem to get this to work though.  I am sure there is some simple answer that I am just missing.  I see the output gets put on stdout with the .reporter function but I'm not sure how to get that back into a stream.
/* ******************************************************************
 * Basic PHP Compile Function
 * *****************************************************************/
function phpBuild(phpFilename, htmlFilename) {
  return () => {
    gulp.src(phpFilename)
      .pipe(exec('php <%= file.path %>'))
      .pipe(htmlMin({ collapseWhitespace: true }))
      .pipe(exec.reporter())
      .pipe(rename(path.basename(htmlFilename)))
      .pipe(gulp.dest(path.dirname(htmlFilename)));
  } // return
} // phpBuild

The other way to go could be this:
const exec = require('child_process').exec;
const through2 = require('through2');
function phpBuild(phpFilename, htmlFilename) {
return gulp.src(phpFilename)
.pipe(through2.obj(function (file, enc, cb) {

  exec('php '+phpFilename, function (err, stdout, stderr) {

    // FRAGILE: assume no error

    file.contents = new Buffer(stdout);

    cb();

  });

}))

.pipe(htmlMin({ collapseWhitespace: true }))

.pipe(rename(path.basename(htmlFilename)))

.pipe(gulp.dest(path.dirname(htmlFilename)));

} // phpBuild
Or look at opt.pipeStdout from https://github.com/robrich/gulp-exec/blob/master/index.js#L36

Awesome. That seems more in line with being the "gulp" way. Thanks for the help.

Where we ended up with this:
/* ******************************************************************
 * Basic PHP Compile Function
 * *****************************************************************/
function phpBuild(phpFilename, htmlFilename) {
  return () => {
    gulp.src(phpFilename)
      .pipe(plumber())
      .pipe(through2.obj((file, enc, cb) => {
        exec('php ' + phpFilename, (err, stdout, stderr) => {
          if (err) { gutil.log(err); }
          file.contents = new Buffer(stdout);
          cb(err, file);
        }); // exec
      })) // php pipe
      .pipe(htmlMin({ collapseWhitespace: true }))
      .pipe(rename(path.basename(htmlFilename)))
      .pipe(gulp.dest(path.dirname(htmlFilename)));
  } // return
} // phpBuild


Backport/1592 platform test logs

What this PR does / why we need it:
rel-1592 needs to also publish platform test logs. These are backports from main to enable this.

The nightly test run https://github.com/gardenlinux/gardenlinux/actions/runs/11478962127/job/31945764877 now includes the upload of the chroot and platform test logs.


KAFKA-3422: Add overloading functions without serdes in Streams DSL

Also include:

remove streams specific configs before passing to producer and consumer to avoid warning message;
add ConsumerRecord timestamp extractor and set as the default extractor.

Ping @ijuma @miguno @gwenshap for reviews. I suggest this patch to be merged in trunk before 0.10.0 branch cut.

@guozhangwang: I am reviewing, but I think moving forward we should begin to have atomic PRs -- this PR conflates three distinct functionality changes (1. overloading functions w.r.t serialization, 2. changes to Streams config setup, 3. timestamp extractor).

@miguno Yes I used to have atomic PRs, but due to the fact that 1) we are having a hard deadline for 0.10.0 branch cut and 2) review round-trip takes longer due to time zone difference, I'm merging multiple ones into one to shorten the period. This is just a one-time thing.

Thanks for the reviews @miguno @ewencp , merged to trunk.


The size of the iOS long image is different from the actual size

Steps to reproduce
A long graph with a width of 1563 and a height of 20802 is obtained with a width of 1563 and a height of 16384.
Expected results
Same size as the picture
Actual results
Does not match the size of the picture
Code sample
Code sample
    Image image = Image.file(orgFile!);
    ImageStream stream = image.image.resolve(ImageConfiguration());
    stream.addListener(new ImageStreamListener((ImageInfo info, bool synchronousCall) {
      int height = info.image.height;
      int width = info.image.width;
      debugPrint('imageInfo width=$width, height=$height, synchronousCall=$synchronousCall');
    }));

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Flutter (Channel stable, 3.22.2, on macOS 15.0.1 24A348 darwin-x64, locale zh-Hans-CN)
    • Flutter version 3.22.2 on channel stable at /Users/xxx/FlutterSDK/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 761747bfc5 (5 months ago), 2024-06-05 22:15:13 +0200
    • Engine revision edd8546116
    • Dart version 3.4.3
    • DevTools version 2.34.3
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

Hi @wahkim
For us to further investigate this issue, please provide a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code.
Also, please share the device info (OS, model name) you are testing on. There was a similar issue that occurred on iOS 13-15 at https://github.com/flutter/flutter/issues/91685 that you can check to see if it's the same case (that issue has been responded at https://github.com/flutter/flutter/issues/91685#issuecomment-951184905)

Code examples are provided above
device:
iphone 8plus, system version 14.6
iphone 13 , system version 18.0
It should be the same problem as #91685, but I don't see a solution

20802 is larger than that devices maximum texture size, so flutter is unable to create a texture bigger than that.

20802 is larger than that devices maximum texture size, so flutter is unable to create a texture bigger than that.

I see that the height of the picture taken on Android is also 16384, but the width is 1231, why on iOS only changed the height and width did not change, why not do the same for Android?

iphone 8plus, system version 14.6
iphone 13 , system version 18.0

Per https://developer.apple.com/metal/Metal-Feature-Set-Tables.pdf:

iPhone 8 Plus: Chip A11 Bionic > Apple4 GPU family, Maximum 2D texture width and height: 16,384 px
iPhone 13: Chip A15 Bionic > Apple8 GPU family, Maximum 2D texture width and height: 16,384 px

I see that the height of the picture taken on Android is also 16384, but the width is 1231, why on iOS only changed the height and width did not change, why not do the same for Android?

Per https://github.com/flutter/flutter/issues/91685 conclusion, it's working as intended so I am going to close the issue as a duplicate. If you disagree, please write in the comments and I will reopen it.

iphone 8plus, system version 14.6
iphone 13 , system version 18.0

Per https://developer.apple.com/metal/Metal-Feature-Set-Tables.pdf:

iPhone 8 Plus: Chip A11 Bionic > Apple4 GPU family, Maximum 2D texture width and height: 16,384 px
iPhone 13: Chip A15 Bionic > Apple8 GPU family, Maximum 2D texture width and height: 16,384 px

I see that the height of the picture taken on Android is also 16384, but the width is 1231, why on iOS only changed the height and width did not change, why not do the same for Android?

Per #91685 conclusion, it's working as intended so I am going to close the issue as a duplicate. If you disagree, please write in the comments and I will reopen it.

Can you do the same thing with Android?

No unfortunately, please check https://github.com/flutter/flutter/issues/91685#issuecomment-951184905 again: "...This probably deserves a more detailed API design."


Corrects typo to make the word "messages" plural.

Since this documentation will be around for decades, let's make it as typo-free as possible.
There is ambiguity in this sentence whether only a single message can be processed or if the singular form of "message" is a typo that should be "messages".

wiseleo wrote:

Since this documentation will be around for decades, let's make it as typo-free as possible.
There is ambiguity in this sentence whether only a single message can
be processed or if the singular form of "message" is a typo that
should be "messages".

Thanks.  I already had this one though.
--
Facepalm reply #9: "Speed up, you can drive 80 here" "Why, the cars behind us
are also driving 60"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Default elm-format binary path is not suitable for Windows

The default appears to be /usr/local/bin/elm-format, which is okay for mac/linux.  But on Windows, I think the default should probably just be elm-format so that it will be found if it's on the PATH.
Also, the error message when this happens is misleading: it says "elm-format exited with code 1", but actually elm-format was not found at all.

I have a solution for this, but I need to find a Windows box to test it.
@emandres what do you put in your binary path for elm-format?

I have a ~\bin folder in my home directory that is on my PATH. That's where I put the elm-format.exe file.

I use vagrant init opentable/win-2012r2-standard-amd64-nocm && vagrant up for windows testing

Same problem. I use win7

For anyone with this question, and my future self: I fixed this by inputting the exact path to the elm-format.exe. In my case it is: C:/Program Files (x86)/Elm Platform/0.18/bin/elm-format.exe
Works perfect after doing this!

Thanks @Eeveline! ... I gave the platform install a try and this worked (as opposed to npm) for others I had to get the elm-format from https://github.com/avh4/elm-format/releases/ and copy it into the platform bin.

@eveline 's comment helped me here ( thank you 😄  ) - but if that doesn't work make sure you got all this steps right.
valid for windows people still struggling with this:
👉 step1: apm install elm-format
👉 step2: then make sure you have Elm Platform installed from here.
👉 step3: make sure you unzipped the elm-format executable from here  - and place it inside the same folder as the Elm Platform installation - you can see it in the path below.
👉 step4: include C:/Program Files (x86)/Elm Platform/0.18/bin/elm-format.exe path in the box of elm-format package - inside the Atom UI - Settings tab as you can see below.

Great summary. Feel free to add to the Readme :)
On 19 May 2017 17:34, "Aion-github"<EMAIL_ADDRESS>wrote:

@Eveline https://github.com/eveline 's comment helped me here ( thank
you 😄 ) - but if that doesn't work make sure you got all this steps
right.
valid for windows people still struggling with this:
👉 step1: apm install elm-format
👉 step2: then make sure you have Elm Platform installed from here
https://github.com/elm-lang/elm-platform.
👉 step3: make sure you unzipped the elm-format executable from here
https://github.com/avh4/elm-format/releases/tag/0.7.0-exp - and place
it inside the same folder as the Elm Platform installation - you can
see it in the path below.
👉 step4: include C:/Program Files (x86)/Elm Platform/0.18/bin/elm-format.
exe path in the box of elm-format package - inside the Atom UI - Settings
tab as you can see below.
[image: atom error elm-format exited with code null - how to fix this
error in windows 10]
https://cloud.githubusercontent.com/assets/20288140/26254054/f634c298-3cbd-11e7-919e-b76436bde606.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/triforkse/atom-elm-format/issues/7#issuecomment-302736044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAaTGOr-dVRLbNUAKrrNG4THBJ0AJIOlks5r7badgaJpZM4Ie84f
.


feat: Implementing Structured Output Support in AI Role-Playing Module

Issue related:
#416
Example

The task:

Problem: Prove that for all positive integers `n` and `k` (where `k \\leq n`), the following identity holds:
`\\sum_{i=0}^{k} (-1)^i \\binom{n}{i} \\binom{n-i}{k-i} = (-1)^k`

The expected output structure defined by the user:

Lemma <NUM_1>:
    <PROOF>
Lemma <NUM_2>:
    <PROOF>
And so on...
Proof by Lemmas:
    <PROOF>

The output generated by  the LLM:

Lemma 1:
    The task involves selecting subsets from a set and applying the inclusion-exclusion principle to avoid overcounting. The inclusion-exclusion principle is applied by considering the `(-1)^i` factor as accounting for the overcounting of subsets when subsets that include specific `i` elements are excluded. Further clarification on the inclusion-exclusion principle can be provided upon request.

Lemma 2:
    The proof requires demonstrating that the alternating sum simplifies to `(-1)^k`. The alternating sum represents the application of the inclusion-exclusion principle to count subsets of size `k` correctly by adding and subtracting counts to avoid overcounting. Additional examples or explanations of the alternating sum's role in combinatorial proofs can be provided if needed.

Lemma 3:
    The proof strategy involves showing that a fixed subset `F` is counted exactly once. The proof ensures that for each subset of size `i` that is a proper subset of `F`, it will be counted and then canceled out in the subsequent term, leaving the fixed subset `F` counted exactly once with the sign `(-1)^k`. Further explanation on the cancellation of subsets in the sum can be provided upon request.

Lemma 4:
    The argument requires formalization to show that all subsets except the fixed subset `F` cancel out in the sum. The process involves examining the contribution of each subset of size `i` to the sum and showing that all subsets except the fixed subset `F` cancel out due to the alternating signs. Details on the formalization process can be expanded if further clarification is needed.

Proof by Lemmas:
    The proof concludes by validating the identity for all positive integers `n` and `k`. The final outcome of the proof is that the left-hand side of the equation is equal to `(-1)^k` for all positive integers `n` and `k` where `k \leq n`. If there are questions about the implications of the proof's outcome, further information can be provided.

@ZhangT-tech Hi, well, can you review the PR, which would be merged into the feature-multi-agent-prod? Thanks.


[Test] Card Vault Presenter Tests & MVP

[Test] Card Vault Presenter Tests & MVP
Migración de CardVaultActivity al nuevo MVP para testear el Presenter.
Closes #326
Coverage: Card Vault Presenter
Instructions: 82%
Branches: 79%

Codecov Report

Merging #529 into v3 will increase coverage by 1.25%.
The diff coverage is 31.59%.

@@           Coverage Diff            @@
##              v3    #529      +/-   ##
========================================
+ Coverage   7.74%      9%   +1.25%
========================================
  Files        195     196       +1
  Lines      15281   15598     +317
  Branches    1371    1440      +69
========================================
+ Hits        1184    1404     +220
- Misses     13958   14022      +64
- Partials     139     172      +33

Impacted Files
Coverage Δ

...ain/java/com/mercadopago/PaymentVaultActivity.java
0% <ø> (ø)
:arrow_up:

...m/mercadopago/providers/CardVaultProviderImpl.java
0% <0%> (ø)

...c/main/java/com/mercadopago/CardVaultActivity.java
0% <0%> (ø)
:arrow_up:

...com/mercadopago/presenters/CardVaultPresenter.java
72.76% <84.34%> (+72.76%)
:arrow_up:

...rc/main/java/com/mercadopago/CheckoutActivity.java
8.99% <0%> (-0.41%)
:arrow_down:

...in/java/com/mercadopago/model/MerchantPayment.java
0% <0%> (ø)
:arrow_up:

...c/main/java/com/mercadopago/model/PaymentData.java
0% <0%> (ø)
:arrow_up:

...src/main/java/com/mercadopago/model/PayerCost.java
50% <0%> (+2.38%)
:arrow_up:

sdk/src/main/java/com/mercadopago/model/Token.java
7.14% <0%> (+7.14%)
:arrow_up:

... and 8 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 94a3ed1...312ff0a. Read the comment docs.


joblib trainable seg: fixed script regression and integrated "Save Classifier" into existing buttons.

Makes it possible to run the classifier the app "learned" on an image locally with the included script use_ml_image_segmentation_classifier.py.
Makes it so you don't need the "Save Classifier" button: pressing the "Download classifier" or "Download classified image" button will then fill the respective dcc.Store with the classifier or classified image and then make a blob and make this available for download. Otherwise dash.no_update is passed to the dcc.Stores (like before).

Thanks for cleaning up my workarounds :-). I'll merge this PR into my branch and check on playground that everything works fine.


StartTrajectory service should support initial pose

The initial pose feature is the recommended way to use localization, in particular on larger maps.
Currently, initial_pose is only available through the start_trajectory executable.
More details are the in PR where initial_pose was introduced.
ROS service StartTrajectory.srv should take initial pose as an argument (lua string should not be necessary).
This is a missing feature.
At the same time, the format of the initial pose argument could be reworked, it is unnecessarily complex to set the pose in 2D.
Ideally, the format should be so easy that start_trajectory_main.cc can be deleted.

When you say the format of the pose argument is overly complex, do you mean only for calling the executable rather than the .srv (which is loaded with parameters already)?
Would the best approach to adding initial_pose to StartTrajectory.srv be with a geometry_msgs/Pose initial_pose field? i.e. Okay to depend on the standard ros messages?

My 2 cents since we use this service in production since more than a year:
Initial pose would be easy to add but is not really the bottleneck IMO.
The ugliest part of this service is the binary string of the trajectory options proto, which is impossible to build by client code that should only know the message contract from cartographer_ros_msgs. The start_trajectory_main.cc executable makes this convenient because it hides exactly these implementation details away from the user/client with a proper interface that has only 3 simple options.
My ideal message definition would follow that interface:
geometry_msgs/Pose initial_pose
string configuration_directory
string configuration_basename
---
cartographer_ros_msgs/StatusResponse status
int32 trajectory_id

and all the configuration parsing, proto etc details are handled by the node. Then I'd be happy to see start_trajectory_main.cc die, but before: nah

My issue description was misleading.
The real issue is what @MichaelGrupp said, building the trajectory options proto is not user-friendly.
The need for start_trajectory_main.cc proves that.

I can do a PR for the refactored service as I wrote above, should be not too hard and would simplify things a lot.

Or, because this would be breaking existing API, a short RFC summarizing of what is written here.

I like your proposed solution @MichaelGrupp, that would suit our use case (lost localisation, need new trajectory with the same options as we started up with, just a new initial pose) well. In the meantime I'll just call out to the start_trajectory executable.

I suggest we continue the discussion here: https://github.com/googlecartographer/rfcs/pull/40

@gaschler @MichaelGrupp @neomanic
Firstly, thanks for you guys great work !
As @gaschler mentioned, detailes about the initial_pose is in the PR where initial_pose was introduced.
Although I've successfully set the initial translation for the pure_localization process, I still find it's very hard for the robot to localize itself in large symmetric environment without the rotation information(mainly the Yaw angle).
And I'm still not clear about what determines the orientation of the /map frame, as shown in issue /1218, so it's quite hard for me to find the correct rotation information.
So I'm wondering how can I get the rotation information (mainly the Yaw angle) to more accurately initialize the robot?
Any hint or comment is appreciated.
Thanks a lot !!

Implemented on master.


[FEEDSTOCK REQUEST] Optimized Python to be added to OpenCE

OpenCE Team,
I have worked the last few months with Calvin Sze on creating an optimized Python for the ppc64le platform.
It would be great if the OpenCE project could incorporate an optimized Python for the ppc64le platform.
Thanks,
Jesse

Hi @harmojd ,
we've started on this task (we've already created a https://github.com/open-ce/python-feedstock with teh necessary build commands). Will keep you posted...

Alexander,
This is great news!  Thank you for your help!
Thanks,
Jesse
On Mon, Feb 6, 2023 at 8:29 AM Alexander Lang @.***>
wrote:

Hi @harmojd
https://urldefense.com/v3/__https://github.com/harmojd__;!!IoaeSlivZg!0mR-5wXL9W_CUsv6gl4GcH8vsptK1KMIRWpTaH8HlM0DgkLudxFUHFihDWh6QW6_jmUqJhJT7cXAWthnZlZl_tHafevu$
,
we've started on this task (we've already created a
https://urldefense.com/v3/__https://github.com/open-ce/python-feedstock__;!!IoaeSlivZg!0AiDAHoLIq4QOmjoIIuazaRIyqhLmMAkJv1E41TCKm3pn4itk8yLcnuDmtDbcIlhsyPE4s32QEn6mbLcF_Rn4rKJr1hieL8Qn33myZdRT6EIvbLrHSHilBHIhX2SYSZl3tGVpJ8$
https://urldefense.com/v3/__https://github.com/open-ce/python-feedstock__;!!IoaeSlivZg!0mR-5wXL9W_CUsv6gl4GcH8vsptK1KMIRWpTaH8HlM0DgkLudxFUHFihDWh6QW6_jmUqJhJT7cXAWthnZlZl_mA0yXQC$
with teh necessary build commands). Will keep you posted...
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/open-ce/open-ce/issues/754*issuecomment-1419172699__;Iw!!IoaeSlivZg!0mR-5wXL9W_CUsv6gl4GcH8vsptK1KMIRWpTaH8HlM0DgkLudxFUHFihDWh6QW6_jmUqJhJT7cXAWthnZlZl_ueYm4VL$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AHHKVFTEHSYQT7QPNCWJJZTWWEDDLANCNFSM6AAAAAAUB6FNQQ__;!!IoaeSlivZg!0mR-5wXL9W_CUsv6gl4GcH8vsptK1KMIRWpTaH8HlM0DgkLudxFUHFihDWh6QW6_jmUqJhJT7cXAWthnZlZl_nrUTAp2$
.
You are receiving this because you were mentioned.Message ID:
@.***>

Done!-) https://github.com/open-ce/open-ce-python/releases/tag/open-ce-python-v1.0.0


Refactoring: Rename envViz... and agenda...

We updated the user-facing to say cseMachine... and control..., but the internal program names are still envViz... and agenda.... These need to be renamed to be consistent with the new naming.
Refer to https://github.com/source-academy/frontend/pull/2678#pullrequestreview-1630378523.

Partially resolved by #2744, but we still need to rename envViz… to cseMachine


Protobuf issues?

When trying to compile the solution I keep getting issues after compiling with Protobuf 3.0.2
namespace "google::protobuf::internal" has no member "MergeFromFail"	creator	E:\git\trance\src\common\trance.pb.cc	1039
What are the versions needed to compile this project?

Did some digging. First time using anything like this, but i'm learning.
3.0.0-beta1


..

.

@habbocorona1 instagram will not let you unfollow more people. It has a cooldown of 5-8 mins. You cant try it doing it on the web manually.


Design Showcase

Taking the opportunity to re-think the showcase.

[ ] The landing page should have filtering by tool, similar to the tools directory.

TODO

[ ] Add meta-data to single pages.
[ ] Add previous/next to single pages.
[ ] Consider pagination/scrolling for list page.

IMAGES

[ ] incorporate imgix
[ ] make sure we have a way of working locally when the image isn't on imgix
[ ] Optimize "picture" element on list/single pages.
[ ] Add lazyloading for images on list/single pages.
[ ] use a placeholder.

forgot to reference this commit [wip] Add showcase single page aa815c5637436289e44c7e1d5a7fb25cb8e8906b

Hey, @DirtyF Just checking. This is the code you used to make imgix not load locally, correct?
{% if site.image_url == 'http://localhost:3000' %}
    {% assign image = page.image_path | prepend: site.image_url %}
  {% else %}
    {% assign image = page.image_path | prepend: site.image_url | remove: 'uploads/' | append: '?w=900' %}
  {% endif %}

And I guess, most importantly, I want to make sure that was working to your satisfaction before I replicate that behavior on the new site.

Whatever you think is best to be able to develop locally, as long as we have all content

But what we have now (on the Jekyll site) works for you locally? I just want to make sure it's correct before I port the behavior.

Solution to above issue with localhost:
{{ if in (print $.Site.BaseURL) "localhost" }}
   ...
{{ else }}
   ...
{{ end }}

https://github.com/thenewdynamic-org/thenewdynamic.org/blob/hugo/layouts/partials/_components/image-picture-element.html#L3


feat: web content pipeline for rag

https://www.reddit.com/r/LocalLLaMA/comments/192dz3r/q_is_it_possible_to_give_ollama_access_to_a_local/

Integrating a feature in Ollama WebUI that allows users to provide a URL and have it automatically processed through RAG would be an exciting addition. However, determining how to trigger this feature raises some questions. For instance, we wouldn't want every URL mentioned in the chat to automatically trigger the RAG process since that could lead to unnecessary processing and potentially unwanted results.
One solution could be adding a button that explicitly triggers the RAG process for a given URL. However, it would be ideal if we could make this feature as automatic as possible while maintaining control over when it's triggered. One way to achieve this could be by implementing some sort of semantic routing model that recognizes specific commands or keywords in the chat input and automatically triggers the RAG process for a provided URL.
For example, if a user types "Read this article" followed by a URL, Ollama WebUI could automatically recognize the command and trigger the RAG process without requiring any additional steps. This approach would maintain the clean interface we currently have.
Here's some further information on Semantic Routing:
YouTube
Jupyter notebook
Repo
OpenAI must be implementing a similar concept in ChatGPT Plus to figure out whether to generate images or code or what have you, as it wouldn't be feasible or cost-effective to require two API calls for every message, one with GPT-3.5 to determine what to use and another to perform the action with GPT-4/DALLE-3/CodeInterpreter. Therefore, implementing a semantic routing model in Ollama WebUI that recognizes specific commands and triggers the RAG process accordingly would likely provide significant benefits for users while maintaining an intuitive and uncluttered interface.

how about adding an endpoint you can publish data to? something like
curl -X post <myUrl.com>/rag --data 'my.csv'
then option to add chromaDB?
Maybe also support for external db?

Additional information about web content pipeline, I think we can integrate search api for RAG. Like serpapi(free plan is 100 times search/month), Wikipedia api or other.

how about adding an endpoint you can publish data to? something like curl -X post <myUrl.com>/rag --data 'my.csv' then option to add chromaDB? Maybe also support for external db?

The RAG endpoint to scrape web pages was added in #333 with /web.

Any update to this yet?
Thanks.

You can now add website content to rag pipeline directly using '#' command followed by the website url, let me know if you guys encounter any issues!
As for the API integration support, let's continue our discussion here: #586, Thanks!

You can now add website content to rag pipeline directly using '#' command followed by the website url, let me know if you guys encounter any issues!
As for the API integration support, let's continue our discussion here: #586, Thanks!

Thanks mate. Been waiting for this. Will try the latest update.

Working great for me 💯  Congrats @tjbck for landing this absolutely huge feature!


Handle the AUTH_FAILED after a PULL_REQUEST

This commit clean-up the OCaml client to terminate correctly if we got such error.

thanks


[RFC] Nixpkgs manual structure: visibility of package usage notes

Describe the bug
Currently package usage information such as OpenGL use on non-NixOS is inside «Builders» section.
Additional context
Some parts of Nixpkgs manual are intended for package users, and some for package developers. Right now the parts for package users are inside the section «Builders» even though they provide no information about any kind of builders.
Previously I tried to separate package-specific development notes and package-specific use notes in #60682 and move the use notes higher (as they are more likely to be useful to the first-time manual reader) but apparently the result was not self-evident enough as shown by #71434
Still, I believe that putting subsections on locales or OpenGL or Nginx ETags inside Builders → Packages does not match the top-level header, hurts discoverability and makes the manual less useful.
As I have previously failed to find a good structure for all that, I ask what are the better alternatives.

Currently package usage information such as OpenGL use on non-NixOS is inside «Builders» section.

I wrote some documentation recently and noticed this. Steam and weechat should definitely be moved elsewhere.

The part Builders describes builder functions as well as packages obtained using some of those functions. I suppose the chapter Packages could become a part Packages.
I merged all of it together with as reason basically a comment you made:

On the other hand, I think the package notes are a mess right now, as there is no separation between notes of type «this is how you update X.org» and «this is how you use Steam»

The part Builders describes builder functions as well as packages obtained using some of those functions. I suppose the chapter Packages could become a part Packages.

That definitely sounds like an improvement.

I merged all of it together with as reason basically a comment you made:

On the other hand, I think the package notes are a mess right now, as there is no separation between notes of type «this is how you update X.org» and «this is how you use Steam»

I tried to create some separation at least for clear-cut cases with muddy cases getting assigned wherever they felt slightly less mis-fitting. I think we eventually need to have this separation here, so that we can tell the users that one chapter is useful for them even if they are not yet ready to edit non-plain-data Nix expressions.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2022-10-22-documentation-team-meeting-notes-12-nixcon-edition/22689/1


Unknown type "JSON" error when using graphql-type-json

Describe the bug
I'm using graphql-type-json  to add a JSON scalar type. When running the code generator, I'm getting the following error: Error: Unknown type "JSON".
To Reproduce
Created the following repo containing an example of the issue: https://github.com/kgajera/graphql-code-generator-json-scalar-bug
Expected behavior
The JSON scalar type is recognized.

@kgajera Thank you for reporting it. I managed to reproduce it in graphql-toolkit. Working on a fix.

Fixed in 1.6.0.


How do I use ImGuiListClipper for drawing a flow container of tiles

I am trying to simulate a scroll area and only draw the visible tiles...
                    var size  = ImGui.GetContentRegionAvail();
                    var index = 0;
                    var count = Tiles.Length;

                    var clipper = new ImGuiListClipperPtr();
                    clipper.Begin((int)(size.X * size.Y), TileSizeY);

                    for (var i=0; i<Tiles.Length; i++)
                    {
                        if (ImGui.GetItemRectMax().X < size.X) ImGui.SameLine();

                        var image = Image(Tiles[index].GetTexture(), new(1,0), new(0,1));
                        Raylib.BeginTextureMode(image);
                        Raylib.DrawRectangleLines(0,0, TileSizeX, TileSizeY, Color.WHITE);
                        Raylib.EndTextureMode();
                        index++;
                    }
                    clipper.End();

How exactly does the .net version of the listclipper work>?
How would I simulate a fake scroll area without looping 60000 times?
How do I get the min and max rows and columns of the visible scroll area?

ListClipperPtr crashes the program and there is no ImGuiList class to pass to it's constructor

https://github.com/ocornut/imgui/blob/master/imgui_demo.cpp#L5658

yes it crashes.
Also please provide C# code when people ask for examples ListClipper doesn't have a Begin, Step or End method only ListClipperPtr does so the c++ api is not a good refrence

Confirming that ListClipperPtr crashes for me as well :(

Nevermind! I figured out that using this works without crashing
ImGuiListClipperPtr listClipper = new ImGuiListClipperPtr(ImGuiNative.ImGuiListClipper_ImGuiListClipper());

I have also pushed a workaround on dear imgui side:
https://github.com/ocornut/imgui/commit/e95d66faa8e6c0255572632f34fad97a32f680cd


Ability to mark all test case as skip and vice-versa

Issue Type: Feature Request
Hi Team,
First of all, thanks for this wonderfull VS code environment. I love it's feature and working with it.
I am writing to place a request by which we could be able to skip all the test case in one go.
Use case: In webdriverio, we can define many test case in one describe block. User should be able to skip/unskip all test case by any short-cut key. I know that we can write describe.skip but i have a need to run one test case only and others should not run i.e. if we have 100 test in one describe block, i have a need to run 1 test only and other 99 should be skipped. So to achieve this, we have to skip those 99 test manually that is a cumbersome task.
Advantage: while fixing automation script, if there is only one test case that need to be run/debug, then we got to put skip keyword manually to all other test case. Imaging doing this where we have 100 test case in one describe block.
Other possibilities: There could be nested describe blocks. But again we are concerned here for test case that starts with 'it' block. So no matter where 'it' block appears it would be marked as skil/unskip if fall under selected describe block.
describe("Bug", () => {
    it("", () => {
    });
    it("second", () => {
    });
    it("", () => {
    });
    it("", () => {
    });
    it("", () => {
    });
    it("", () => {
    });
    it("", () => {
    });
});

VS Code version: Code 1.50.1 (d2e414d9e4239a252d1ab117bd7067f125afd80a, 2020-10-13T15:06:15.712Z)
OS version: Windows_NT x64 10.0.18363

In mocha it is possible to use .only() to run only a single test from a test suite.
Did you try this?

Since the requested functionality depends heavily on the used testing framework, I do not see us implementing this feature in VS Code generically.

@weinand Thanks for highlighting. I placed this request keeping in mind that VS code is able to format my code by shift+ctrl+f irrespective of which language/framework is used as obviously each language/framework has different way of writing a function or class etc. so i thought it is possible to identify a test irrespective of used framework.

VS Code delegates all language specific functionality (e.g. formatting) to (mostly third-party) extensions.
Your feature request would fall into that category too.
Please check out this extension: https://marketplace.visualstudio.com/items?itemName=hbenl.vscode-test-explorer
Maybe it supports already what you are asking for.


gco and gcO create conflicts inside Neovim as both have gc prefix

WhichKey: checking conflicting keymaps ~

WARNING conflicting keymap exists for mode "n", lhs: "gc"
rhs: `

The above message is displayed at the very bottom, when :checkhealth is run in nvim.
As every user will face this conflict due to the "gc" prefix in both of them.

that's just a warning. all good

How can I turn off this warning can you help out me

Why? It's a warning. It does literally nothing.


Update sogouinput to <IP_ADDRESS>43

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating only the sha256:

[x] I verified this change is legitimate How do I do that? and am providing confirmation below:

The homepage and url should remain http://pinyin.sogou.com/mac/

ok, fine! Actually current vision is temporary, since we can not access the official download url.
and why I try to change that homepage url since it offense some rules here.
ok drop this pr. @commitay tks


St Stephens School Rome Request

The school has an iLab that includes development as a part of the curriculum. A Python class is offered for first- and second-year students. The school site that showcases the Ilab is: https://sssrome.it/flagship-programs and you can confirm that this is the institution's official email because the accrediting agencies show that this is the website of the school. you can check that on: https://councilofinternationalschools.secure.force.com/Map/exmapanythingSchoolInfo?Id=0010Y00000bHE4LQAW

Thank you for your quick note there and for updating me on the waiting process, I appreciate the info note :)

@Alpino320 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Error handling: reject extracts that are too large

Extract should say "whoa that’s too big" at some useful part of the process. @sleepylemur, is there a practical upper limit on extract sizes? Can we have two limits, one that requires someone to press an "I understand this will take a while" button and another that we simply refuse to fulfill?

It's very dependant on the area of the world in question and more specifically how much OSM data there is within the region of the extract. I feel that we should just come up with some upper limit and enforce it on the front end.  If you look at the metro extract cities repo, I believe there is a test in there that validates the bbox... one might be for too large.

https://github.com/mapzen/metroextractor-cities/blob/master/spec/bbox_size_spec.rb

Thanks Grant, very helpful. Looks like the maximum size is 75° x 75°?

No, that's the region threshold which doesn't apply for odes. The city threshold is 5x5.

That’s much more reasonable, thank you. We’ll use it.


Fixed small spelling issue in issue template.

What issue does this PR address?
Just a small spelling issue.
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit follows our guidelines
[x] Unit Tests for the changes have been added (for bug fixes / features)

Other information:
No unit test needed I would say 😅

Thanks!


Export seed as a human-transcribable QR code

This is an evolving preview with sorta-okay UI integration but it IS functioning!

Go through any seed entry process (manual, generate, dice) and at the end it'll display your seed as a simplified 25x25 QR code. Transcribe that (for testing purposes, snap a pic; real world: paper and pen!). You can also navigate to display your 12-word seed later and it'll display the QR after showing the words.
Restart the SeedSigner fresh, go to import a seed. You'll see a new option to scan from QR code. Voila! Your seed is back!
The human-transcribable QR code is kept to a minimum of data by using the seed words' indices in the embit.bip39.WORDLIST. Each word's index is expressed as a four-digit zero-padded number:
0023 1038 0472 0342 etc

That's all concatenated into a single 48-digit string and passed into the QR code generator.
I need to dig more into the QR code generator to see if there'd be any difference with feeding it an array of ints (16 bits/word vs 4*(??bits/digit)/word) or if that's even possible.

Great feature!!

Refactored display_seed_phrase to consolidate the three versions for 12-word, 24-word page 1, 24-word page 2.
Now shows full seed phrase after 12th/24th word is generated.
Refined logic with show_qr_option to display seed QR when generating a new seed (12th/24th word, dice) or when viewing a saved seed. But skips seed QR display during xpub generation and signing PSBT flow.
Dynamically changes menu item from "Save Seed" to "View/Save Seed" to "View Seed" depending on how many slots are used.
Conceals passphrase on display_seed_phrase.
Dice UI defaults to #5 to reduce joystick moves necessary on each roll.


[1.10.2] Capacitor Bank Issue

What happens:
Capacitor Bank In/Out gets set to 0 after a server restart (Chunks are not loaded)
This is happening every time the server restarts.
This is not an issue when only EnderIO is installed.
So must be a mod conflict in my modpack that is causing some issues. Also had some issues with the Refined Storage computer not reacting to redstone. Could be some chunk loading issue or something.
(Not a mod dev, so ive got no clue really :p)
Steps to reproduce:

Place Capacitor Bank
Restart Server
Check Bank, Input/Output is now at 0
...

Modpack (My own custom)
http://pastebin.com/7QwpstLv

Affected Versions (Do not use "latest"):

EnderIO-1.10.2-3.1.170
EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge-1.10.2-<IP_ADDRESS>5

Your most recent log file where the issue was present:
Not sure exacly what you guys need, so posted the last files and did a server restart before posting them.
https://dl.dropboxusercontent.com/u/128571833/fml-server-1.log
https://dl.dropboxusercontent.com/u/128571833/fml-server-2.log
https://dl.dropboxusercontent.com/u/128571833/fml-server-3.log
https://dl.dropboxusercontent.com/u/128571833/fml-server-latest.log

see #3763

Just to be sure, it's only the IO limits that reset and always both of them together? So the side config and the RF storage doesn't get reset?

Both IO always reset together. Ive not checked to see if sides get reset or not since ive not needed to setup the sides as im using it right now. RF Storage is kept between restarts. Gonna try out and see if the sides reset between restarts.
Sure, give me a build and ill run some tests

Thx, I'll make one tomorrow.

@HenryLoenwind Have you made this build yet? (Just so I havent been missing anything like downloading it from a special page that I dont know about xD)

Sorry, got side-tracked by Real Life. I'll start right now, expect a version in about 2 hours.

Ok, build is running: http://ci.tterrag.com/job/EnderIO-1.10/172/
There's a chance that this already fixes the problem. If not, set the config value debugTraceCapLimitsExtremelyDetailed (server and client) and see what the logfile says. (Note: Going from "-1" (meaning "use defaults") to the actual default value is normal.)

No problem, Ill run some tests tomorrow (its 23:00 here) and ive been away the whole day
Ill post any finds tomorrow

First tests seems good. 3 Restarts and going far away to unload the chunks and the Cap banks retained their I/O settings on all tests.
Ill play around with it some more but it seems to be fixed, Nice!

Ok, what I changed was: When the GUI is opened, all buttons and text fields are initialized with the values from the TileEntity. Whenever a button changes, it triggers a network packet to the server with all the GUI data. In combination, opening the GUI sent 4 packets...which usually contained just the existing data. I removed those 4 packets.

Logged in today and noticed 1 cap bank had its I/O reset to 0. The other are just fine.
Enabled the debug but cant seem to replicate the issue now. Will keep the debug enabled for awhile and see if I can catch when the issue happens

Thx, for doing that. I'm quite sure I eliminated all instances of resetting the value with the GUI, but the resetting at other times, e.g. server start, is still unsolved.

Have not seen any of the Cap banks revert back to 0 I/O. So it feels like its fixed. Or that I doesn't want to show itself while the Debugging is enabled =D
Thanks for all the help Henry :)

no, no, I have to thank you. Without people who can reproduce errors and who are willing to run debug versions, we'd never be able to fix all bugs.


How to put Progressbar

How to put progressbar when audio load from uri

Same as for local file except total length could be determined wrong (-1 for example, I think default Android MediaPlayer causes this behaviour).
See demo app for more info:
Java code
XML layout

how to put in this code
public class AudioPlayerActivity extends AppCompatActivity  {
String Content_id,audio_url;

AudioView audioView;

private ProgressDialog pDialog;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_audio_player);

    audio_url = getIntent().getStringExtra("audio_url");

    // Progress dialog
    pDialog = new ProgressDialog(this);
    pDialog.setCancelable(false);

    audioView=(AudioView)findViewById(R.id.audioview);

//        pDialog.setMessage("Please Wait...");
//        showDialog();
    try {
        audioView.setDataSource(audio_url);
        audioView.start();
    } catch (IOException e) {
        e.printStackTrace();
    }

//        hideDialog();
}
@SuppressLint("MissingSuperCall")
@Override
protected void onStop() {
    super.onPause();
    audioView.pause();
}

@Override
protected void onDestroy() {
    super.onDestroy();
    audioView.pause();
}

private void showDialog() {
    if (!pDialog.isShowing())
        pDialog.show();
}

private void hideDialog() {
    if (pDialog.isShowing())
        pDialog.dismiss();
}

}

I do not see your ProgressBar. Where do you want to put it and what should it do? Default AudioView has SeekBar showing current track percentage completion and allowing to seek over the track.

Show progressbar when audio url load

If you download audio to local storage then it's not related question.
If you want to show progress while AudioView is preparing (it does not matter from url or local) then you could implement AudioViewListener for AudioView and override onPrepared() method. Then set listener through AudioView's setOnAudioViewListener().

Closed due to inactivity


Help for booking slots for 45+

can we use same code to book slots for 45+?

it can be used for any age group


ncon should take a BaseBackend object

Since we are now supporting passing BaseBackend to Node constructors, we should also
allow it in ncon

Yes. Should be easy to fix right? I think we just need to update the typing

that's right, I think it should be woring already

has anyone taken the lead on this?


MAIN B-18642 add origin gbloc to move screen front end

B-18642
PR to INT:
https://github.com/transcom/mymove/pull/12335
https://github.com/transcom/mymove/pull/12355
https://github.com/transcom/mymove/pull/12462
Summary
This is the frontend work to implement origin GBLOC in the move headers for office users.
USMC moves are unique and therefore the moves as Marine Corps as BOS will display / USMC in addition to origin GBLOC. e.g. KKFA / USMC
How to test
Non-USMC:

Create any move or use any existing moves you have.
Access either as SC, TOO, TIO, or QAE (complete counseling for the move if you want to access as TOO/TIO).
Go to the move of your choice and look at the move information header and you should see Origin GBLOC matching the origin GLOBC of the move (See image below).

USMC:

Create a move as a Marine Corps BOS or use HHGMoveAsUSMCNeedsSC testharness.
Access either as SC, TOO, TIO, or QAE with USMC GBLOC (complete counseling for the move if you want to access as TOO/TIO).
Go to the move of your choice and look at the move information header and you should see Origin GBLOC matching the origin GLOBC of the move with  / USMC added (See image below).

NOTE: SC vs TOO/TIO/QAE may show different origin GBLOC and this is intentional according to this confluence page.
Screenshots

verify rpt ran.
there were diffs.
B18642-12400.verify_rpt.txt

verify rpt:
B18642-12400.verify_rpt.txt
there were diffs.

verify rpt: B18642-12400.verify_rpt.txt
there were diffs.

Verified the report and is expected and correct.


Add ruby linter

Adding govuk-lint to ensure styleguide is adhered to as part of CI Build

Going to go again on a clean branch


Why is " a unsafe character?

Hi,
Regarding the following source snippets:
https://github.com/vsch/flexmark-java/blob/c0313d67e0146292a10d04eb8944faff991579e9/flexmark-util-sequence/src/main/java/com/vladsch/flexmark/util/sequence/Escaping.java#L43-L45
https://github.com/vsch/flexmark-java/blob/c0313d67e0146292a10d04eb8944faff991579e9/flexmark-util-sequence/src/main/java/com/vladsch/flexmark/util/sequence/Escaping.java#L64-L78
Why is the " character is considered unsafe in the following context?
My usecase: I am processing Markdown, which is then processed using velocity. I rely on the fact that " is outputted as " and not as ".
I am parsing the following line of markdown (which is a function call in velocity)
$object.myfun("myString")

Expected:
$object.myfun("myString")

Actual:
$object.myfun("myString")

Sources that this is valid:

https://pagedart.com/blog/single-quote-in-html/
https://stackoverflow.com/questions/2210430/are-single-quotes-valid-in-html-xhtml (Strictest answer says that " is not allowed in attribute, but says nothing about plain text)

Workaround:
I am currently setting the XML_SPECIAL_RE field with my own replacer, which ignores the "case.
Proposed solution:
https://github.com/vsch/flexmark-java/blob/c0313d67e0146292a10d04eb8944faff991579e9/flexmark-util-html/src/main/java/com/vladsch/flexmark/util/html/HtmlAppendableBase.java#L128
 appendable.append(s);

Thanks for your time!

My apologies for taking a year to address this. You are right. This was probably inherited from original code.
I will merge your PR for next release.

Hi :)
It's okay. Hm....
Seeing it now there is a downside to this solution as well... Links for example . If the String "bla" contained a ", it would break the generated HTML.
So might be better to have a better workaround than that (I can look into this if you think that the solution above is not good enough)

It's okay :)
However I cannot run the following test
com.vladsch.flexmark.integration.test.SpecIntegrationTest

java.lang.AssertionError: Resource path: 'D:\com\vladsch\flexmark\test\util\spec.txt' not found.

We can also leave that for now, I have a workaround for my project :) And I guess I am off spec TBH


Use file(GENERATE) to create a CPackConfig file

The current way that the CPack module works is to generate a CPack configuration file after all calls to install() within a project. With file(GENERATE) this approach becomes superfluous and we can setup a way to generate the entire file based on generator expressions. This also means we'll have to make a dummy CPack.cmake file so we can override its existence so we don't have subprojects break our build (as well as break the subproject)

Closing, as this project has burned me out, and I cannot continue working on it in its current state. CMake has changed massively since this project was started. Most of it is no longer necessary. A new repository will be launched in September.


feature/require.context

https://github.com/IndexXuan/vue-cli-plugin-vite/commit/35c37be613c84761764b1307d14fcefdda100bd1


Typescript

Introduce TypeScript according to CRA's guideline
install types for project packages
install and configure TS-Plugins for ESLint
refactor components and tests into TypeScript

ANNOTATION:

There is still a TypeScript error in the Cypress test, I couldn't figure out.

Please check the configuration again, some typescript errors are not shown in the console.. Also there are still some unconverted js files, serviceWorker.js setupTests.js.


Some glosses have 'None' as value of one of the Phonology fields.

The problematic glosses are:

CASSIS, 2134, field: relOriMov
DOORGEVEN-C, 705, field: oriCh
ST-MICHIELSGESTEL, 273, field: movCh
PERSOON-A.PL, 658, field: virtObj (this is actually a char field, so not necessarily a problem)
wesseltest3, 3117, field: handedness (this is a test gloss of course)

So far none of these caused errors, but with some of the new code by @susanodd they did.
@ocrasborn how shall we solve this/who will solve this?

To clarify, on the phonology fields with a choice, "no choice" is 0 (zero), and "never filled in" is a null value. But somehow the value 'None' appears in the database records mentioned above.
This leads to a type error on the query that searches for Minimal Pairs / Homonyms. The query looks for "empty" phonology fields and chokes on 'None' in the where comparison (the query crashes... because...)
Either the field being compared is not a string (but 'None' is a string, so it's not an empty value) or not an integer (the index of a pulldown choice is a number,where 0 is "no choice") (here, if None is a string, not one of the indices, it can't be compared to numbers. But if the query is written as relOriMov = None, the database looks for a field named None, which is a syntax error.)

I have no clue where it came from! It's never been one of the possible values we created for most of the fields. And in fact, when I edit the signs now, the pull-down menu automatically changes to '-', which is what it should be. I've thus edited and saved all of them. For the char field virtObj, I've simply removed the content altogether, it wasn't necessary to have it there, but of course it could arise again in the future because users are free to type in anything.

Does this solve this issue, @vanlummelhuizen ?

This issue does not seem relevant anymore.


Fix remove-comments

I had a few spare minutes this morning so I thought I'd have a go at a couple of bundler issues.
First is the issue with multiline comments that contain a ].
I also added a test. I'm not sure if that's the kind of test you're after or if you want more. Let me know.

I have rebased, re-added the call to removeComments to the bundler and rebuilt.
(I don't have merge permissions)

Merging, let's hope this doesn't break things like my other attempts at this.