commands  jpterm --server  not working

Os: Windows 10
Python: Python 3.11
Terminal: Powershell (Windows terminal)
to reproduce the error:
pip install "jupyterlab>=4"
pip install jupyter-collaboration
jupyter lab --port=8000 --no-browser
then in another terminal:
jpterm --server http://<IP_ADDRESS>:8000/?token=972cbd440db4b35581b25f90c0a88e3a1095534e18251ca8
or
jpterm --server http://<IP_ADDRESS>:8000
display  :

then in another terminal:
jpterm --server http://<IP_ADDRESS>:8000/?token=972cbd440db4b35581b25f90c0a88e3a1095534e18251ca8

I see that this is the same token as on the README, but you need to replace it with whatever JupyterLab gave you. The command:
jupyter lab --port=8000 --no-browser

should print something like this in your terminal:
    To access the server, open this file in a browser:
        file:///home/david/.local/share/jupyter/runtime/jpserver-631663-open.html
    Or copy and paste one of these URLs:
        http://localhost:8000/lab?token=dec1420d65fc7a82f9dbdaa175be430eda5602083c82390a
        http://<IP_ADDRESS>:8000/lab?token=dec1420d65fc7a82f9dbdaa175be430eda5602083c82390a

Now you need to launch jpterm with:
jpterm --server http://<IP_ADDRESS>:8000/?token=dec1420d65fc7a82f9dbdaa175be430eda5602083c82390a

Note that it's the same URL without the lab.

I confirm it was the problem. thanks.


Suggestion: Add mod localization capability

Hello! I'm a user who often uses your mode. Many users also use this mode in Korea, but the mode is in English, so it's a little inconvenient to use.
Do you have any thoughts on adding a localization function?
Of course, I know that there are difficulties in producing it, but if you make it, I think I will use it more comfortably.
And if you add a localization function, I will help you translate Korean.
Grammar can be weird because I'm a little weak in English conversation. Please understand this.
Thank you for updating the mods frequently and thank you for reading this.

If you mean for UI elements in-game, the only non-localized part is the "Show/Hide Mod Stats" button on the inspect windows, mentioned in #8
If you mean the settings in the F12 menu, unfortunately those strings get used in the cfg file, which means changing them would lose all your settings. The mouseover descriptions could be localized, though.
Which do you mean (or both?)

Oh I think the explanation was a bit lacking I'm sorry. It was the F12 part you mentioned.
I'm a beginner in coding and I don't know as well as you, but I got your source code and added a localization function to F12 to code it.
Added one more language tab. By default, English is displayed, and if you choose another language and restart the game, it's changed.
When I tested it, I didn't find any problems except that the Language menu doesn't translate when I change to Korean. However, there can be bugs. As I said above, this point can be because I'm a beginner in coding. Sorry if there is.
I'll leave it as an attached file, so if you're interested, please get it and apply it, and if you think it's okay, I'd appreciate it if you could add it at the next update.
UIFixes-2.3.4 Translate.zip

While I appreciate the hard work, I don't think that keeping translations of the options menu is sustainable. It prevents me from being able to make changes, and will be constant upkeep.


fix: SSO Callback redirect fix for unavailable state cookie. Fixes #11268

Fixes #11268
Motivation
SSO Callback from Microsoft Azure AD returns with HTTP 400 error because of missing state cookie header. Code is trying to check if Cookie header with name state exists -> Set the cookie header with name state -> Return with HTTP 400 error. Reload of page after HTTP 400 error redirect user to argo-workflow UI with successful authentication (Because during reload Cookie header with name state is present)
Modifications
Modified function HandleCallback of file server/auth/sso/sso.go. Logic is changed to look for cookie header with name state. If cookie header not found then set the cookie header.
Bottom of the function is changed as well to verify if there was an error during initial cookie header fetch and set redirect only if there was no error along with pre-existing logic.
Verification
I have tested by setting up argo-workflows in AKS cluster and performed SSO Login using Microsoft Azrue AD. I no longer get HTTP 400 error and whole login and redirect journey is working fine. In the end user is redirected to the argo-workflow UI with correct authentication.

Trying to understand the existing code, I believe we first call HandleRedirect() to redirect the user to the SSO endpoint, and we pass a query parameter called redirect_uri which looks like this (and has state as a query parameter in it):

And then there's an assumption that the SSO endpoint is going to call back to us using that redirect_uri. And then in HandleCallback() we are expecting to receive a query parameter called "state". And for some reason you are not receiving it?

From the code, it does look like this "state" cookie is merely being used to elegantly redirect the user back to the original page requested, as opposed to being used for any actual authorization, so I think you're right that it shouldn't return a 400 error just because the cookie isn't present.

So, do you think if changes looks ok ?.
I am sure that it is not the problem with browser as there are other
components who are doIng SSO authentication with OIDC provider including
ArgoCD. And whole flow os working fine.
On Thu, 29 June 2023, 00:02 Julie Vogelman, @.***>
wrote:

From the code, it does look like this "state" cookie is merely being used
to elegantly redirect the user back to the original page requested, as
opposed to being used for any actual authorization, so I think you're right
that it shouldn't return a 400 error just because the cookie isn't present.
—
Reply to this email directly, view it on GitHub
https://github.com/argoproj/argo-workflows/pull/11270#issuecomment-1611892637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKVUE2LARLYVLFDAWYWBYDXNR2CJANCNFSM6AAAAAAZVDZTDE
.
You are receiving this because you authored the thread.Message ID:
@.***>

So, do you think if changes looks ok ?.

Please see, I did add a comment in-line in the code.

After a close look I found that, I am testing argo workflow behind reverse proxy and doing URL rewrite, because default callback url is intercepted by oauth2-proxy at reverse-proxy. Because of URL Rewrite I am loosing the state cookie that was created while initiating the SSO login and hence during callback handling, cookie isn't available and seeing 400 error.
We may not need this PR. Also, fix I introduced isn't correct. I will see if I can reject this PR and close it.


Inconsistent syntax?

As a middle aged hobby coder just starting out with Go I appreciate the simple and straight forward syntax (I’m middle management by day .. and I get the tingling feeling there’s a joke in there somewhere😄). I also enjoy to read the (so far) consistent reasoning of the maintainers for choices made.
Do you have any idea why this (https://github.com/akutz/go-generics-the-hard-way/blob/490784b7d7c082ea84ddeadefcb02c9b03ff7a5a/03-getting-started/03-constraints.md) logical OR in the generic constraints wasn’t two pipes too? Or is the subtle difference that this generic constraint statement doesn’t actually “result” in a Boolean value or something? Something like why variable assignment and function parameters (signature) have the type the other way around.
I love stuff like this, partly, I’m sure, because if it makes logical sense it sticks too. Not really an issue with this repo of course (which was timely, complete and well thought out!).

 I think it is because they refer to these a type sets, and maybe there is some logical leap to the bitmask syntax?

I'm not on you or the creators' level, I do simple adhoc stuff and have to get familiar with generics anyway, but I hope this doesn't point in the direction that generics is bolted on from community pressure because they come from other languages with generics. I mean, there's got to be a reason why Go only had the ingenious and rare Interfaces for the longest time. I respected that and the simplicity (like me!) although I understand from this repo that it doesn't completely cover the generics use case. To really get a close look at generics you can't do better than this repo though, I'll pass it along to my fellow simpletons.

I didn't quite catch that on the front page! Love it! Yay! I may not necessarily be getting crazy slow after all 😄 Conscious enough to appreciate masterful pedagogy though, don't worry.
Thanks for the back and forth!

If nothing else I think it bears work to add a call out on the appropriate page that mentions the | is acting in a way some people consider to be a logical OR, aka ||.

Ah, I found it. The Wikipedia article on Union type has many examples from other languages, ex. PHP, Python, Typescript, where the single, vertical bar | is used the exact same way as it is Go generics and union type sets.

So it may resemble generics in other languages more than its own logical statements elsewhere..

I added this note https://github.com/akutz/go-generics-the-hard-way/commit/62f3e381fa96e4684495f406cf741afc2a27b61f and closed this issue.


Most users said they did have a (physical) photo, but never got a chance to use it.

This user journey confused users, as they were asked about a photo that seemed relevant, yet never given an opportunity to use it.
Some who were told to do to a photo store would abandon the process at that point.
Some tried to click back and find an alternate route.

I've ammended PR #334 so that the word 'digital' is included on the "You need a photo page" in the first sentence and in the question:

closed as a duplicate of #301


Document the create_csv_table function's sensitivity to column order

Detail
Document the create_csv_table function's sensitivity to column order
Relates

https://github.com/aws/aws-sdk-pandas/issues/1918

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: c610fa3c28bdfcb8cd7b284e606693ae1699eb7a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Signup and Login  Authentication not working due to inappropriate POST request and URL

The signup and login functionality on the website is currently not working as expected. After investigation, it has been identified that the issue lies with the inappropriate POST request and URL being used for these processes.
Steps to Reproduce:

Visit the website.
Attempt to sign up or log in using valid credentials.
Observe that the signup/login process fails to complete successfully.

Expected Behavior:
The signup and login processes should successfully create a new user account or authenticate an existing user, respectively.
Actual Behavior:
The signup and login processes are failing, preventing users from creating new accounts or logging in.

Additional Information:
Upon further analysis, it was found that the POST request being sent to the server is not correctly formatted or contains incorrect parameters. Additionally, the URL being used for the signup and login requests seems to be inappropriate or incorrect.
Proposed Solution:
To resolve this issue, I will handle  code responsible for handling the signup and login functionality. Ensure that the POST request is properly formatted with the required parameters and that the URL being used is appropriate for these operations. Once the necessary changes are made, retest the signup and login processes to ensure they are functioning as expected.
please assign me this issue ,I will fix it asap

Please @codervivek5  assign it to me I will create a PR today

@codervivek5  please review my pull request and assign appropriate level,I have contributed to my level best .

ok I will check it


isTrigger prop not implemented

Problem
The documentation says an isTrigger prop can be passed to the body to make it work as a collision detection trigger (similarly to how cannon-es, use-cannon work following this PR.
However, p2-es does not implement an isTrigger prop anywhere in the codebase.
You can instead pass the sensor boolean when instantiating a shape (codebase ref, but the current implementation off use-p2 doesn't allow to pass extra props to shapes.
Solution?
Add ability to pass extra props to shapes when creating a body or implement the isTrigger prop.
Also here's the p2.js sensor demo

Will take a look, meanwhile you can set collisionResponse false like in the example https://github.com/pmndrs/use-p2/blob/main/examples/src/demos/PhysicalCharacterController/Heart.tsx#L55

You can use it now as expected. Set an isTrigger prop to the body, it will set the sensor prop on each shape of the body. See examples/src/demos/Trigger.tsx


Cleaned up some warnings from the static analyzer

Fixes #121

@gonzalonunez, almost there...
Please fix inconsistency in return types between header and implementation files (that's what fails travis)

Should've fixed it all. Waiting to make sure Travis doesn't fail.

Seems to be missing something. Checking over it now.

Looks like it's related to refresh methods signature.

Ow, right... Those methods are declared also in PFObjectPrivate.h - that's what is missing.

Ah, great catch. I'll go ahead and change those now. Thanks for the tip, would've taken me all night! Lol

@gonzalonunez, almost there, a single change around fetch method in PFUser and we are good to go.

Fixed the issue in PFUser.m and squashed the commits 🙌

Looks perfect! Merging now!


docs: fix hyperlink

The LINE official tutorial link at README.md isn't valid anymore.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
930

Relevant Lines:
996

💛  - Coveralls


add eTagValidation for uploadFileMultipart

Computes and validates the computed eTag matches the eTag from AWS S3.
Made the validator an option with a default of None so we don't break existing consumers of the function and also allows to opt in rather than forced to get the new behavior.
We have tested this and verified with S3 and have used it successfully.
related to: https://github.com/laserdisc-io/fs2-aws/issues/1108

@semenodm I don't really have any more items I want to address.  Let me know if I need to make any additional changes or if you have questions on anything.

@BusyByte i found the optimization for the digest, take a look at latest commits.
also I was thinking about weather we need to do unconditional digest calculation if no digest calculator was provided.

doing this

@semenodm  the changes in b70615a LGTM

@BusyByte thank you for the contribution, this was nice one.

@semenodm just some notes for future major releases I think it'd be good to return from uploadFileMultipart

Whether md5 validation happened and successful
How many total bytes were uploaded to S3
What the max partNumber/partId was.

We send events and record metrics with this information and exposing it on the return from uploadFileMultipart.
This however would be a breaking change so I am not proposing it until the next major version.


No module named experiment and test_data

from expan.experiment import Experiment raises an ImportError: No module named experiment.
I have to change it to from expan.core.experiment import Experiment and it works.
Also from tests.test_data import generate_random_data doesn't work, I have to use
from tests.tests_data import generate_random_data but this raises an another error cannot import name generate_random_data.

right, we need to investigate the first behavior further, but at least for the moment the second issue can be solved by
from tests.tests_core.test_data import generate_random_data
I agree that the folder names might not be very intuitive, but I'm open to suggestions;-)


Rule suddenly scanning excluded directory

Before upgrading to v3, I had a rule that would scan my system for pesky .DS_Store files and delete them - excluding ~/Library. Since upgrading and with no change to the rule, ~/Library is being scanned as well which takes a lot of time.

OS: macOS 14.2.1
Output of organize --version: v3.0.0

Here is the relevant rule config:
rules:
  # Delete .DS_Store files
  - name: 'Delete .DS_Store files'
    locations:
      - path: '~/'
        exclude_dirs: ['Library', 'Dropbox', '.Trash', 'Pictures']
        system_exclude_dirs:
          ['.git', 'node_modules', '.docker', '.local', '.archive']
    subfolders: true
    filters:
      - name: .DS_Store
    actions:
      - delete

Thank you for reporting this issue! This should be fixed in f9f18726fa087be37f70b6cdf8fdc257efe2bf19. I'll release the new version as soon as possible.

The fix is released!


Events Stacked with Absolute Positioning

I have the module up and running and "modes" such as "Daily" or any vertical list work fine. In "Monthly" or "Weekly" however all events on a given day are stacked on top of one another via the absolute positioning of timeline elements. Image of titles stacked is attached.
I haven't touched the core css or the custom css. Any reason this would occur? I also checked for any conflicts with other modules and restarted all systems.
I'm using the basic config here:
    {
      module: "MMM-CalendarExt2",
      config: {
        calendars: [
          {
            url: URL,
            name: "Main Calendar",
            maxItems: 99,
            scanInterval: 10006030,
            beforeDays: 0,
            afterDays: 7,
            maxIterations: 999
          }
        ],
        views: [
          {
            mode: "week",
            position: "bottom_bar",
            slotCount: 1,
            timeFormat: "h:mm A",
            slotMaxHeight: "auto"
          }
        ],
        scenes: [
          {
            name: "DEFAULT"
          }
        ]
      }
    },

thank you that fixed it.


Java library: set default value for commandType as "Text"

I tried to use Text as default but when I ran a Java function without the commandType specified, it threw an error:
Microsoft.Azure.WebJobs.Extensions.Sql: The type of the SQL attribute for an input binding must be either CommandType.Text for a direct SQL query, or CommandType.StoredProcedure for a stored procedure.
I'll leave it empty for now but can certainly revisit this.
Originally posted by @lucyzhang929 in https://github.com/Azure/azure-functions-sql-extension/pull/379#discussion_r986189755

I removed the commandType from some of our Java input binding samples and the functions worked as expected (tests also passed on ad hoc build pipeline). Not sure if there was a change in our extension that fixed this or a change in how the Java worker passes information to the sql extension.
Closing this issue since the error no longer appears.


GPT-BigCode Model Architecture

This PR is intending to add the GPT-BigCode Model Architecture. It will contain the following additions/changes:

Added GPTBigCode implemented with FMS modules
Added tests for GPTBigCode

I am not sure if I understand the issue of position ids, but in in gpt_bigcode.py, we do not need to consider all possible HF models... but just focus on popular big code models and how they work.


add additional mcca example

Hi, thanks for the amazing toolbox! I would like to add an additional example using the MCCA implementation in python-meegkit since I was initially having trouble understanding how the implementation is used to extract an underlying signal from concatenated data matrices. Maybe this will help others when applying more complex signals to the MCCA function.
The example is example #1 taken from de Cheveigné et al., 2018. I believe that the code is correct and reproduces the results shown in Figure 4. Note: this is my first pull request, so I apologize in advance if anything isn't properly implemented.

Hi, I'm happy to see that my pr has passed all but one of the tests. With regards to the test that failed, I'm not sure I can fix it on my end. Here is the error:
remote: Permission to nbara/python-meegkit.git denied to github-actions[bot].
fatal: unable to access 'https://github.com/nbara/python-meegkit/': The requested URL returned error: 403

Best,
John Kyle

Hey John, thanks for this! I will have a look asap. The doc build job should be an easy fix

Hi @johnkylecooper, the example looks great.
FWIW, you don't need to generate both the example_*.ipynb example AND the example_*.py files.
Just do you code in the .py file, and run make build-examples and the notebooks will be automatically generated from the .py files (including headers and sections).
Again, thanks for the contribution. I don't use the code in meegkit that much these days for my own work but I'm happy to keep maintaining it if it helps others 🙂
PS: the doc build fails it comes from a fork. It's a CI bug, nothing to do with your example.

https://nbara.github.io/python-meegkit/auto_examples/example_mcca_2.html#sphx-glr-auto-examples-example-mcca-2-py  looking good ! ;)

https://nbara.github.io/python-meegkit/auto_examples/example_mcca_2.html#sphx-glr-auto-examples-example-mcca-2-py looking good ! ;)

Thanks! :)

Hi @johnkylecooper, the example looks great.
FWIW, you don't need to generate both the example_*.ipynb example AND the example_*.py files.
Just do you code in the .py file, and run make build-examples and the notebooks will be automatically generated from the .py files (including headers and sections).
Again, thanks for the contribution. I don't use the code in meegkit that much these days for my own work but I'm happy to keep maintaining it if it helps others 🙂
PS: the doc build fails it comes from a fork. It's a CI bug, nothing to do with your example.

Also, thanks for the tips! :)
Yes, I mainly use meegkit for the preprocessing tools (for eeg recordings) and now mcca. It's very handy! I have a lot of respect for you guys who maintain open-source projects! I hope to be able to contribute more to projects in the future!
Going to delete my branch and delete the fork now.
Best,
John Kyle


Not creating thumbnail

I use the Upload plugin from https://github.com/josegonzalez/cakephp-upload
The problem is, while the main image is uploaded fine, it is not creating thumbnails.
I have enabled imagick and it detected when I do phpinfo()
Here is my model code
public $actsAs = array(
    'Upload.Upload' => array(
        'image' => array(
            'thumbnailSizes' => array(
                'featured' => '720x400',
                'xsmall' => '98x98',
                'small' => '152x110',
                'medium' => '400x222',
                'large' => '225x145',
                'medium_home' => '232x128',
                'xlarge' => '720x632',
                'editorial' => '199x300',
                'medium_editorial' => '180x249',
                'small_editorial' => '152x211',
                'xsmall_editorial' => '98x136'
            ),
            'path' => '{ROOT}webroot{DS}uploads{DS}{model}{DS}{field}{DS}'
        )
    )
);

Any idea what should I change?

Any feedback? Did it work? Can I close this issue now @williamn ?

Yes you can. I have no time to retry this issue. Thx.

Thanks!


Potential diffusion metric: average/max deviation from nominal b-vector

Some head-motion metrics from fMRI can be directly applied (e.g., framewise displacement), but in the case of dMRI I think it would be interesting to have a metric reporting the differences between the b-vectors before b-matrix rotation (with head-motion realignment matrices) and the original (nominal) b-vectors.
The rationale behind this metric is that some large head motions do not affect the b-vector (translations), while FD and other head motion metrics are more volume-to-volume absolute measurements.
Another +1 for this is the ease of computation (once you've estimated head motion, of course).
cc/ @arokem

Okay, I just realized this is duplicate of #1216. I'll move the conversation there.


⚠ Bofrost.si has degraded performance

In 8424623, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 1642 ms

Resolved: Bofrost.si performance has improved in 369f30c after 6 minutes.


safe executable ERB in dynamic segments in API (#937) (#938)

Addresses: https://github.com/restarone/violet_rails/issues/934
Allow ERB in API Actions

https://user-images.githubusercontent.com/35935196/179740864-bb5a7b31-0daf-4ce2-959c-55ff673f10a1.mov
Co-authored-by: Pralish Kayastha<EMAIL_ADDRESS>

Staging Confidence Check :chart_with_upwards_trend:

API general regression pass :heavy_check_mark:
Performance & monitoring :hourglass:


Seriously

Prime is becoming a cobalt dev, rust was just too easy

import Control.Concurrent (threadDelay)
import Control.Monad (when)
import Data.List (permutations)
import Network.AWS.DynamoDB
import Network.AWS.Data.Text (toText)
import System.Environment (getEnv)
import Database.MySQL.Simple

waitUntilReady :: IO ()
waitUntilReady = do
  ready <- checkIfWordIsReady
  when (not ready) $ do
    threadDelay 1000000
    waitUntilReady

checkIfWordIsReady :: IO Bool
checkIfWordIsReady = do
  let allWords = permutations "seriously"
  host <- getEnv "DB_HOST"
  port <- read <$> getEnv "DB_PORT"
  user <- getEnv "DB_USER"
  password <- getEnv "DB_PASSWORD"
  database <- getEnv "DB_NAME"
  conn <- connect defaultConnectInfo
    { connectHost = host
    , connectPort = port
    , connectUser = user
    , connectPassword = password
    , connectDatabase = database
    }
  result <- query conn "SELECT COUNT(*) FROM words WHERE word IN ?" (Only allWords)
  let count = fromOnly (head (head result))
  close conn
  return (count > 0)

saveToDynamoDB :: IO ()
saveToDynamoDB = do
  let item = toText "message" .= toText "YOU ARE A CHAD"
  env <- newEnv Discover
  tableName <- getEnv "TABLE_NAME"
  let req = putItem (toText tableName) & piItem .~ item
  void $ runResourceT $ runAWS env $ send req

main :: IO ()
main = do
  waitUntilReady
  saveToDynamoDB

<!DOCTYPE html>

  <meta charset="UTF-8">
  <title>Chadness</title>

  <div x-data="{
    message: 'Loading...',
    link: 'https://www.youtube.com/@ThePrimeagen',
    loadMessage: function() {
      var dynamodb = new AWS.DynamoDB({apiVersion: '2012-08-10'});
      var params = {
        TableName: 'chadness',
        Key: {
          'id': {S: 'message'},
        },
      };
      dynamodb.getItem(params, function(err, data) {
        if (err) {
          console.log(err);
          this.message = 'YOU ARE A JavaScript DEV';
          this.link = 'https://www.youtube.com/@t3dotgg';
        } else {
          console.log(data);
          if (data.Item && data.Item.message && data.Item.message.S) {
            this.message = data.Item.message.S;
          } else {
            this.message = 'YOU ARE A JavaScript DEV';
            this.link = 'https://www.youtube.com/@t3dotgg';
          }
        }
      }.bind(this));
    },
  }" x-init="loadMessage()">

      Go to channel

^ this was generated by ChatGPT

CHADStack
nvim.lua >= 5mb
rewrote arch linux kernel in rust.
else gtfo go back to PythonScript or whatever inferior languages simpletons use these days

Such a chad move leaving the repo empty.

I contemplate the notion that at times, the vacuity and stillness we encounter embody the essence of our being, reaching beyond mere human existence. This void, as manifested by the empty repository, prompts me to ruminate on its significance.

 What the heck is this?

the repo is not empty
the repo is not up-to-date
LIVE COBOL
LOVE COBOL


Add a "Pause" button for long-running processes

Add a "Pause" button while uploading files is in progress. With the next finished file the upload process should stop and a "Continue" button should re-start the uploa with the next file.
Reason: When uploading a really large batch of file (> 500) it could be usefiil to pause the upload for some time.

I have transferred this issue from the CommonsExtension repository to the general OpenRefine one, as I think this fits better there (pausing other kinds of data uploads may also be a request).

I have changed the title of this issue to make it more general: this is now about making it possible to pause any long-running process, such as:

reconciliation
add column by fetching urls
data extension
wikibase edit upload

This is something I intend to work on as part of the reproducibility project, as it touches on the structure of the process queue.

Hmm, any possibility for Clustering pausing across large bin sets? Maybe/maybe not? or only some algorithms? or is Clustering not a true long-running process, I forget.

Clustering is indeed not the same sort of process (because it is not tied to an operation), so I would keep this outside the scope of this issue. We have a separate issue about this process: https://github.com/OpenRefine/OpenRefine/issues/241

Closed by https://github.com/OpenRefine/OpenRefine/commit/06595aeca093c26db8d233b04ceeeb907f834753


Allow pinning setup-trivy by hash

I work with repos where all GitHub Actions are pinned by commit hash, and passlisted in the repo settings. As of https://github.com/aquasecurity/trivy-action/pull/406, it is not possible to do this anymore, since it internally fetches aquasecurity/setup-trivy by tag, and there does not seem to be any way to override that. Even if skip-setup-trivy: true is set, it seems GitHub will require passlisting the tag version.

Even if skip-setup-trivy: true is set, it seems GitHub will require passlisting the tag version.

Could you expand more on this, where does this passlisting happen? Is it a GitHub requirement?

This could also be fixed by using a commit hash reference in action.yml for setup-trivy. Users who do not restrict their referenced Actions will not notice the difference.


Distinguish pending invoices in weekly email with Red color on dates field

Change the color to red for pending invoices dates in the weekly "List of unpaid invoices" email.
A similar color coding is in the place portal. Please make the similar for email.

Update: All the columns in the email is red now. We need only the expected date to be in Red color for invoices, due day of which all has passed today date.


#200 update readme.md

The 200th commit
Include Instagram link in Readme

Also add the prettier badge

Happy coding. :grinning:


Which CLIP model do you use?

I notice that you provide image encoder on your own space, is it different from the models released by openai?

@haofanwang hi, for IP-Adapter of SD 1.5, we use  OpenCLIP-ViT-H-14. But for IP-Adapter of SD xl, we use OpenCLIP-ViT-bigG-14.

It seems that the model in your
own space is different from the one mentioned above. At least, the file size is different, which raises concerns about its integrity or compatibility.


strict flake8-type-checking

recent flake8-type-checking encourages cast() using the name of the class rather than the actual class, presumably with the aim that you might find more things you don't need to import at runtime

However, it seems it doesn't notice that some imports can be moved into a TYPE_CHECKING block (e.g. VCSDependency in test_factory.py).

Seems we have to take care of that ourselves: snok/flake8-type-checking#127


pyGeo Geometry Deformation given DVGeometry FFD

Purpose
This PR adds a function within DVGeometry to update a pyGeo geometry object with a given FFD deformation. The purpose of this function is to allow users to generate an IGES (or Tecplot) file of their updated geometry to be used after the optimization (for example when using CAD). An example usage of this function is included in ./examples/deform_geometry/ and associated documentation is added in the pyGeo documentation under Tutorials.
To allow for a tutorials / examples section in the docs, I reorganized the documentation structure to have separate categories: installation, tutorials, and API.
Type of change
What types of change is it?
Select the appropriate type(s) that describe this PR

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[X] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing
This function is separate from current tests and is difficult to test as it's output is an IGES or Tecplot file.
Checklist
Put an x in the boxes that apply.

[ ] I have run flake8 and black to make sure the code adheres to PEP-8 and is consistently formatted
[ ] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation

Not sure why the docs build is not triggering, I can just manually merge this if another maintainer is able to review the PR.

@bbrelje do you need more changes in this PR? @bernardopacini do you want me to review this?

If you have time, that would be great @anilyil.

I am seeing some strange behavior in DAFoam when I try to use this. I do not think it is related, but just to be sure let's hold off on merging this until I have that working properly.

It looks like this PR adds a couple of large files
examples/deform_geometry/geo/wing.dat (650,000+ lines)
examples/deform_geometry/geo/wing.igs (98,000+ lines)
Should we download those like we do the mesh files for adflow?

It looks like this PR adds a couple of large files
examples/deform_geometry/geo/wing.dat (650,000+ lines)
examples/deform_geometry/geo/wing.igs (98,000+ lines)
Should we download those like we do the mesh files for adflow?

Actually that's a good point. However, once it's committed it's pretty difficult to purge since if you just remove the files they're still in the history (and will get downloaded when you run git fetch). If we really want to remove those you may have to do some rebasing/force pushes.

The issue was with the DAFoam case, so this is ready after all.
Would you like me to try to remove the large files from the history?

Do you know how you would go about doing it? I'm not 100% sure if rebasing + force push would work, or if you need to make a new branch and cherrypick.

Doesn't squash merging just do one commit with the files that are remaining in the last version of the PR? So I think we can just remove the files and once we squash merge, it will only add the changed files in the git history of mdolab/pygeo. The files will stay in the history of @bernardopacini's fork though

Doesn't squash merging just do one commit with the files that are remaining in the last version of the PR? So I think we can just remove the files and once we squash merge, it will only add the changed files in the git history of mdolab/pygeo. The files will stay in the history of @bernardopacini's fork though

I may be wrong, but I believe the individual commits are still transferred also, it will just appear as a single commit. For example, individual commits like this are still accessible after it's squashed and merged.

oh I see, so they would take up space in the mdolab repo right?

okay that makes sense, then I will also need to do that for the overset tutorial. Thanks for the explanation @nwu63!

Yeah unfortunately purging large files from a repository is not easy. I tried to do that for ADflow once and nearly screwed everything up.

That makes sense, I can go through that process once the PR is approved. I'll address the other comments first.
Another option might be using the git filter-branch command, but I have not used it before so I am not sure how exactly it will work.

Yeah actually if you wanna try that it sounds like a better solution. We can talk offline to work through that after you address the rest of the comments.

Yeah unfortunately they would. If you want to be sure that they are removed, I would recommend the following once the rest of the code is approved:
* close this PR

* make a new branch from `master`

* set the state of this branch to be the same as this PR branch (you may have to have another directory and copy/paste, I don't know how to check out the state of a branch into the working space)

* add everything except for the files we don't want to commit

* make a PR new PR and link to this PR so we preserve the discussion

We lose the commit history and the discussions are going to be in another PR, but this will work. Alternatively, rebasing/force pushing might work too but I'm not 100% sure.

@bernardopacini you could also use the same approach... but replicate the steps commit by commit, so to not lose the history:

Make a clean copy of pyGeo and set a new branch
On the current branch (and folder), roll back to the first commit of this PR
Copy the files from your active pyGeo folder to the new pyGeo folder, and recommit with the same message
Back to the current branch, move one commit ahead, then copy to the other folder and recommit
Continue this until the last commit here, this time avoiding to add unwanted files.

This way we don't use the history. Does it make sense @nwu63 ? Also happy to help offline.

I don't think this is entirely necessary since this is a smaller development, but you can do it if you want I guess.

Sure, I guess this is really up to the developer, depending on how much the single steps are relevant for future maintenance

This PR is approved and I moved the large files out of the current state (downloadable via the get-input-files.sh script instead). I will now try to remove the large files from the history before we merge this.

After some testing with @bernardopacini, we believe that because this PR comes from a fork, squash + merge will not add the large files to the mdolab repo. Since @anilyil has approved already I'm going to merge this.


Add fixture to manage the environment variables for e2e tests

https://github.com/ansible/ansible-rulebook/pull/349 should be updated accordingly, @ttuffin


No more "IP-Address" Input in TCP Out Node

What are the steps to reproduce?
Im not sure what is the exact way to reproduce this issue. But it comes without warning when you switch from "Connect to" to "Listen on" or "Answer to TCP".
I know that the only Setting is "Connect to". But in Case of testing many Setups to communicate with my robot i found this issue.
Copy the Node from another Node Red brings back again the Input Line temporary.
Of course i set "deploy" after any change.
What happens?
Using TCP Out Node to connect with my Kawasaki Robot. Temporary it´s not possible to change/edit the IP Adress. The Input Window (normally to insert or Edit IP Adress) is not show. (See Picture).
What do you expect to happen?
I think thats a bug in the node itself.
Please tell us about your environment:

[ ] Node-RED version: 1.0.3
[ ] Node.js version: 12.13.1
[ ] npm version: 6.12.1
[ ] Platform/OS: Raspbian Stretch and Buster
[ ] Browser: Up to Date Firefox and/or Chrome

In the instance where the IP Address line is not shown, can you expand the info table so we can see the full set of node properties:

As shown below.
Node Red is the client - The Robot the Server

Can you check the browser JavaScript console for any error messages when you open the dialog?

Is this LOG (attached) ok for you.

When not, please give me instructions to make a LOG you want.
Microsoft Edge also none IP Adress Input Window. For a few moments checked.

most odd. Will investigate.

I can't see why this would happen unless somehow node.js async calls get out of sync (so to speak)... anyway the existing decision code is a bit shonky so I have made it explicit so only one possible path per option. Hopefully this fixes it as I can't recreate here.

The Problem is still present.
When i install a new image with raspbian stretch on a Kunbus Revolution Pi.
Install fresh Node Red ...
No Adress Input Field is show.
I have to "export" the TCP Out Node, from another Pi where the Node works fine.
On this Pi, i can change the IP Adress and export the Node to the other Pi(s).

Can you access your flow file directly, copy out the TCP node configuration that is causing the issue and share here?


letter-spacing not taken into account

Hi, thanks for creating an maintaining this repo.
I am running into an issue where the truncating is not working as expected. If i specify there to be 2 lines to be shown, it will show something like
lorem ipsum
lorem ipsum
lorem...

I did some debugging as I found it worked elsewhere, and found out removing letter-spacing makes it work correctly. Looking through the source, it indeed appears there is no mention of letter-spacing in the css properties.
Is there a way in which the letter spacing can be taken into account when calculating the width, either automatically or by specifying a custom prop?
Here is a WebpackBin of the problem https://www.webpackbin.com/bins/-KueNc5FP6xDfCRLM4zp

Hi, thanks for reporting this issue.
Unfortunately, the canvas context.font API (which this component relies on) does not support a letter spacing / word spacing argument. So there's no way to easily support that feature.
See https://github.com/One-com/react-truncate/issues/30#issuecomment-275107694 for similar context.

Chrome now supports letter-spacing in v56 see CreateJS/EaselJS#872. Could be viable to provide a flag to expose the functionality or in 56=< choose to include line-height.


Feature proposal: clear action creator

Are you submitting a bug report or a feature request?

Feature request
A common pattern I'm running into for dependent fields (e.g. field B's value depends on field A having a value) is that I want to clear field B when field A changes.
So I do something like:
// Assume field B, C, and D are conditional and A having a value
handleFieldAChange(event, value) {
  const { change } = this.props;

  if (!value) { // if field a was emptied
    change('b', null);
    change('c', null);
    change('d', null);
    untouch('b', 'c', 'd');
  }
}

change doesn't support passing in multiple fields currently. In my case, I know that the value for each changed field will be null, so an action creator clear() (or some better name) with the same signature as untouch() would be very useful:
handleFieldAChange(event, value) {
  const { clear } = this.props;

  if (!value) {
    clear('b', 'c', 'd');
  }
// Behavior: args.forEach(arg => { blur(arg); change(arg, null); })

Alternative / additional suggestion: Change the argument order of change to (form:string, value:any, ...fields:string), to bulk change values. The downside is that similar action creators will need to be changed to be consistent with the signature.
Is there a better existing way that I'm missing? If not, would there be interest in this functionality? If so I can take a crack at it over the weekend. Thanks!

Seems sensible to me, I'd say go for it!

Note as well the linked issue.. maybe there are other cases for having bulk setting of formValues.

Hm, I just saw the issue, so maybe a changeMany or something would be more general-purpose. Thanks! (By the way, I'm not sure what the best way to proceed on this is -- I've filed another issue #3295)

setFormValues is another name suggestion ;).

Consider also #2224

@danielrob So I'm running into an issue in the implementation of a bulk change action, and was hoping one of the maintainers could chime in... (cc @erikras)
The issue: the function signature for actions that change multiple fields is ...fields, taking rest arguments instead of an array for convenience/ nicer syntax, I guess. This means that the list of fields must be the last arguments passed.
However, internally the change action also has a functionality for custom touch and persistentSubmitErrors. The signature of this function is (form: string, field: string, value: any, touch: boolean, persistentSubmitErrors: boolean).
Any sort of bulk change action can't have touch and persistentSubmitErrors as the last argument while also accepting spread field names vs. an array. If these two arguments were moved to the front, however, the exported action creator would have two arguments the user shouldn't care about, as createReduxForm should be responsible for passing these arguments based on configuration.
Not supplying these arguments at all would make the bulk change action behave subtly different from the change action. Bug reports galore 😱
The bulk change action could simply dispatch an array of change actions, which replicates how users are currently using change() to set multiple fields to a given value. But IMO the reducer should ideally handle all values at once for better performance (unless the form values object has some immutable-based optimization for deep-setting values? I've seen the set and get functions but haven't dived into their implementation details).
Thanks!

@erikras Do you have any ETA on when this feature will be released in a stable version?

Fix published in v7.2.0.

It looks like clearFields doesn't clear the store value or errors when destroyOnUnmount:false. Is this expected?


Access to openMenu() function from parent component

Hello,
I'm trying to open the menu from a link external to the slide but it looks like i don't access to the method.
Here's an example (maybe it's not the correct way, i'm beginner with VueJS)
<template>
  <Slide :burgerIcon="false" ref="slideMenu">
    <!-- links -->
  </Slide>
  <OtherComponentExample></OtherComponentExample>
  <span v-on:click="openSlideMenu">
    Text on click...
  </span>
</template>

<script>
  import { Slide } from 'vue-burger-menu'
  import OtherComponentExample from 'my-component'

  export default {
    name: "Example",
    components: {
      OtherComponentExample,
      Slide
    },
    data() {
      return {
        // some data...
      }
    },
    methods: {
      openSlideMenu: function () {
        console.log('Test open menu');
        this.$refs.slideMenu.openMenu();
      }
    }
  }
</script>

I want to open the menu with my method openSlideMenu().
Thank you

@HamHamFonFon the right way to do it, is to use a store, and then you should have a variable in state, which define if the slider is opened or no, and then you can access it from anywhere on the app, that's the right way to do it in vuejs

Hmm. This doesn't seem to work for me. Can we get an elaboration on the trick in the docs, or codepen? Thanks!

this worked for me:
     navSidebarOpen(e) {
      e.preventDefault();
      e.stopPropagation();
      this.$refs.slideMenu.$children[0].openMenu();
    }


Add .gitignore and remove impacted files

It should be confirmed that the Vercel build is doing the npm install.

It should be confirmed that the Vercel build is doing the npm install.

Vercel build is still working (based on preview link).


docs(getting-started): sort the external dependencies

This is preparatory to some more substantial work to add macOS installation hints in future commits.

I signed it.


(MD) Resend Data for Park Pediatrics up to 9/10

Maryland is unable to located any data from Park Pediatrics before 9/10. Please resend data prior to that data.
SR Message Counts:
0 results for 9/3,
2 for 9/4,
3 for 9/5,
11 for 9/7,
9 for 9/8,
2 for 9/9 and
1 for 9/10

@mlighdc @Adrian-Brewster The data was resend today and has been delivered to MD.


Figure out how to extract the barcodes in Aksel's code

We need the barcodes from the digitization images to be retained in species-web. It looks like the species-ocr scripts are already gathering these barcodes so I just need to figure out how to retrieve them and get them uploaded to species-web.

I've set up a test environment on my local machine with the species-ocr repo and have successfully retrieved barcodes from the output.json file. I think we just need to edit the 6_uploado_to_web.py file to include the id from the specimen table during upload. My proposed change is here:

    "specimen": [
        {
            "barcode": specimen["id"],
            "guid": specimen["guid"],
            "digitiser": specimen["metadata"]["digitiser"],
            "date_asset_taken": specimen["metadata"]["date_asset_taken"],
            "image_file": specimen["image_file"],
            "checksum": specimen["checksum"],
        }
        for specimen in group["specimen"]
    ],

Maybe I can figure out how to test this on my computer as well, without having to go through the rest of the scripts. I should ask Khaled/Bhupjit.

I tested the updated code by saving it as a json file and it worked correctly. Next I need to figure out how to add barcode to the sql db so we can start adding these.

Khaled helped me figure out how to update species-web. Only the folderupload.js and schema.prisma files will need to be updated in that repo.

Updates to production can be followed here: #542


Evil remains active in calc-mode

What did you expect to happen?
Evil to be fully disabled in calc-mode.
What actually happened?
Evil mode remains (partially) active. Particularly the hjkl keys, interfering with, for instance help.
Additional details:
None, really. However, a default keybind for calc and/or quick-calc might be useful (eg. SPC t c), but that would be a separate issue.
Steps to reproduce:

Open calc mode (M-x calc or calc-mode).
Press (for instance) the h key.

System information:
https://pastebin.com/34GUJJAE

We use evil-collection, which provides evil integration for calc. Your options:

Study evil-collection-calc to learn how to use it.
Disable evil-collection-calc: (add-to-list '+evil-collection-disabled-list 'calc)
Or forcibly revert to emacs state in calc-mode: (set-evil-initial-state! 'calc-mode 'emacs)

Since this isn't a bug and our issue tracker isn't for how-to or config help questions I will close this.

;; "h" 'calc-help-prefix ; TODO: Rebind?

This explains it. Thank you!


Any way to connect some uptime-kuma installations to one server with uptime-luma

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Hi,
Very good service to check the aviability of a lot of services!!! Amazing !!!!!
I've started to do a mini-server to check a lot of thing of my house and from internet.
But the quiestion is, there is any way that I can install a small uptime-kuma and configure for check some services, and send the info (online/offline, time of ping...) to the mini-server with uptime-kuma.
Is for check devices that are not in my network with ping and no need to use the TCP port check with port forwarding.
Very thanks.
Nico
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.21.1
💻 Operating System and Arch
Docker with Synology
🌐 Browser
Safari 16.2
🐋 Docker Version
20.10.3-1308
🟩 NodeJS Version
12.22.12-2031 // 10.20.1-1012 // 16.17.1-1010

See #84


Update CDN SRI Hash for 2.1.0

Addresses #2203

The main README needs to also be updated: https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/lib/msal-browser#compiled

The main README needs to also be updated: https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/lib/msal-browser#compiled

done

Coverage increased (+0.1%) to 83.347% when pulling 2a5b5ebc8a9c63d013cb6ca5d8891ed318125635 on fix-hash-2.1.0 into db612e3ddfd7130cb3a975328094c6b7dfc051ac on dev.


Spelling/graphic error of GUI on cfund page

I have noticed this too after the 4.6.0 release, but haven't had any time to make any changes

Few more quirks:
1.) There is some scaling issue with Windows OS. Currently my laptop is set at recommended 125% scaling and NAV Core's Fonts are not scaling properly.
2.) We have "1ndia and 0ndia" instead of "India".
3.) The lines underlined in "red" does not look good and can be replaced with "dots" when there is less space or can use smaller fonts.

I think this was fixed in 4.6.0? @aguycalled

I think this is No longer applicable as of the new #530 PR


Notifications and other minor things

This is currently WIP but mail notifications should work now. I plan to add HTTP as well. I extracted the make invoice dependency because it's not possible to get r_hash (that we need to subscribe to settlements) from the encoded invoice without admin macaroon and that's a dangerous requirement. Make invoice doesn't return r_hash even though the underlying REST API provides it so I made appropriate changes and removed the dead code that's not relevant to lnd (we don't support anything else anyway). This also greatly reduced the amount of dependencies to just gjson.
I also fixed macaroon loading so it's not loaded from disk every time the invoice is created. Comments were previously completely ignored even when allowed in config. Now they're properly used in notifications. Added optional private channels hints (for nodes with no or small public channels), TLS certificate support (for REST API connection). Refactored repeating error reporting code.
So it's a draft for now, feel free to test and review.

I highly appreciate your contributions and agree with them on a high level
from what you've described. I like the mail support. We might want to
introduce some threshold for the earned sat amount to be notified otherwise
a lot of emails might get sent out(my node sees lots of micro payments
~10sats)  Currently away I will review/merge when I'm back on my desk in a
couple of days.
In the meantime I was working on  integrating lnd invoice events into
https://github.com/hieblmi/go-lnd-router-events which returns preimage and
r_hash and which I planned to use in go-host-lnaddr.
( Commit
https://github.com/hieblmi/go-lnd-router-events/commit/8ef4c98e7072bd0a7b1b57fc835dec2d85abf8c2
)
But if you've done the lifting here already we can build on that. Or
extract your event notification into its own module later.
Cheers
On Sat, Oct 16, 2021, 09:47 rkfg @.***> wrote:

This is currently WIP but mail notifications should work now. I plan to
add HTTP as well. I extracted the make invoice dependency because it's not
possible to get r_hash (that we need to subscribe to settlements) from
the encoded invoice without admin macaroon and that's a dangerous
requirement. Make invoice doesn't return r_hash even though the underlying
REST API provides it so I made appropriate changes and removed the dead
code that's not relevant to lnd (we don't support anything else anyway).
This also greatly reduced the amount of dependencies to just gjson.
I also fixed macaroon loading so it's not loaded from disk every time the
invoice is created. I want to add the TLS certificate path to the config so
that we don't just ignore TLS errors silently. So it's a draft for now,
feel free to test and review.
You can view, comment on, or merge this pull request online at:
https://github.com/hieblmi/go-host-lnaddr/pull/6
Commit Summary

Notifications WIP
https://github.com/hieblmi/go-host-lnaddr/pull/6/commits/de3d830418f28ea8ee4be17f7c49774f3e3d7cd5

File Changes

M config.json
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-587cb980af76fdc7e52369fd0b9d926dff266976b6f8ac631e358fecc49ff8cf
(38)
M go.mod
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6
(5)
M go.sum
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-3295df7234525439d778f1b282d146a4f1ff6b415248aaac074e8042d9f42d63
(274)
A http_notif.go
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-3a6fa9a565d06d949687ceee1a439291b34b124456ed4c8d14145911a84ab30c
(17)
A mail_notif.go
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-cbd1ff949841dda174e1c8861b69ee7ac98b4cbcf40aa1d0f411d041098241e7
(38)
M main.go
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-2873f79a86c0d8b3335cd7731b0ecf7dd4301eb19a82ef7a1cba7589b5252261
(42)
A makeinvoice.go
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-54bc8f0fbd05053ff64dce723e98797388770e9d3daa7f55b63b08ce993792a9
(125)
A notification.go
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-b74ceddf45c49b67d2c9fc0b9122668103bf1c153d560c552a0503524d517b22
(36)
A settlement_handler.go
https://github.com/hieblmi/go-host-lnaddr/pull/6/files#diff-8bf9cf6edcc17b029509b72cf87d90fec150c9bf7ea04cd7d66cd99a2c5b26a7
(81)

Patch Links:

https://github.com/hieblmi/go-host-lnaddr/pull/6.patch
https://github.com/hieblmi/go-host-lnaddr/pull/6.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hieblmi/go-host-lnaddr/pull/6, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOEXPSRPXXR4YKEP33N2OLUHGGBLANCNFSM5GDW5NGQ
.


Updates PIN code create and enter screens

Summary of Changes
New PIN create and enter screens using the react-native-confirmation-code-field package.

Introduces PIN input component that replaces the text input fields in PIN create and enter screens
Adds validation logic to ensure only digits are used for PIN
Updated wording and alert messages for PIN screens

Related Issues
N/A
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[ ] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[ ] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[ ] Updated documentation as needed for changed code and new or modified features;
[ ] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

@PatStLouis Looks like @amanji added place holder values to the French translation file. Able to translate?

@marcodrompre Tagging you in case you have time for the translation.

Here's our suggestions


Update PipelinedSimpleModulusEncryptor.cs

fix contentOfFirstBlockLength bug

Thanks, I’ll look at it soon.

Thanks for bringing this bug to my attention, I can't believe I didn't notice it.
I added a unit test to identify the issue and verify the solution. I found that 'size' has already the correct value, so nothing has to be added or subtracted at it.


only search for nongolang licenses in root

Issue #, if available:
Description of changes:
The gather-nongolang-licenses was introduced for helm charts, and then later used for the linux build.  The intention is to make sure our produced artifacts, including charts, have the proper upstream license included.  What we were doing makes sense, however, we end up including all the licenses in the vendor directory(s).  This we def do not to include all these.  The problem here is the chart ends up getting put into a secret and the recent update to cert manager puts it over the size limit.
This changes the find to only look in the root directory, which should be all we need for the helm repos. I checked each build to ensure we were getting a license file for each helm chart.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold

/lgtm

/approve

/cherry-pick release-0.20

/cherry-pick release-0.19


update to prisma 2.9.0

The 2.8.1 prism has a problem that they solved in 2.9.0
/usr/bin/node[329084]: ../src/node_http_parser_impl.h:529:static void node::{anonymous}::Parser::Initialize(const v8::FunctionCallbackInfov8::Value&): Assertion `args[3]->IsInt32()' failed.
update your package so that you can use it

resolved, I downgraded the node to 12.18.4


Add lock around task cgroup

Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

Merging #1795 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1795      +/-   ##
==========================================
- Coverage   49.38%   49.33%   -0.06%
==========================================
  Files          77       77
  Lines        7692     7692
==========================================
- Hits         3799     3795       -4
- Misses       3248     3252       +4
  Partials      645      645

Flag
Coverage Δ

#linux
49.4% <ø> (-0.06%)
:arrow_down:

#windows
44.89% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

metadata/content.go
31.05% <0%> (-0.95%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM


ESM build of<EMAIL_ADDRESS>fails when target=denonext

What happened?
Steps to Reproduce
I'm building a Deno service that will emit OTLP metrics to Honeycomb. Importing the OTLPMetricExporter class from the ESM module like so:
import { OTLPMetricExporter } from "https://esm.sh/<EMAIL_ADDRESS>
fails with the error:
Uncaught TypeError: Class extends value undefined is not a constructor or null
  at https://esm.sh/v130/@opentelemetry/exporter-metrics-otlp-proto@0.41.1/denonext/exporter-metrics-otlp-proto.mjs:8:1424

Expected Result
I expected I could import the ESM module and use it the same as I can using an npm specifier like so:
import { OTLPMetricExporter } from "npm<EMAIL_ADDRESS>
NOTE: I want to deploy this service to Deno Deploy, which does not currently support npm specifiers.
Actual Result
See the error above.
Additional Details
I pulled the failing module, formatted it for legibility, and found the problem lay within the @opentelemetry/otlp-proto-exporter-base module:
   77   var L = u("@opentelemetry/otlp-proto-exporter-base");
   78     i = u("@opentelemetry/core"),
   79     x = u("@opentelemetry/exporter-metrics-otlp-http"),
   80     v = u("@opentelemetry/otlp-exporter-base"),
   81     y = u("@opentelemetry/otlp-transformer"),
   82     q = f(),
   83     g = "v1/metrics",
   84     D = `http://localhost:4318/${g}`,
   85     m = { "User-Agent": `OTel-OTLP-Exporter-JavaScript/${q.VERSION}` },
   86     d = class extends L.OTLPProtoExporterNodeBase {

Line 77 requires the @opentelemetry/otlp-proto-exporter-base module and later line 86 attempts to extend the OTLPProtoExporterNodeBase property of L. For some reason, the ESM build does not expose a OTLPProtoExporterNodeBase property on the L object:
The following code:
import * as ExporterBase from "https://esm.sh/<EMAIL_ADDRESS>console.log(ExporterBase);

outputs:
[Module: null prototype] {
  OTLPProtoExporterBrowserBase: [Function: m],
  ServiceClientType: {
    "0": "SPANS",
    "1": "METRICS",
    "2": "LOGS",
    SPANS: 0,
    METRICS: 1,
    LOGS: 2
  }
}

However, the following code using the npm specifier:
import ExporterBase from "npm:@opentelemetry/otlp-proto-exporter-base";
console.log(ExporterBase);

outputs:
{
  OTLPProtoExporterNodeBase: [Getter],
  send: [Getter],
  OTLPProtoExporterBrowserBase: [Getter],
  ServiceClientType: [Getter],
  getExportRequestProto: [Getter]
}

Note the existence of the OTLPProtoExporterNodeBase property.
I understand the Deno is a browser-first runtime with backwards support for Node libraries, but I'm not clear on why the ESM version of this module would exclude the OTLPProtoExporterNodeBase property. I'll try and do more research here, but figured I'd open this issue in the mean time.
OpenTelemetry Setup Code
import { metrics } from "https://esm.sh/<EMAIL_ADDRESS>import { Resource } from "https://esm.sh/<EMAIL_ADDRESS>import {
  AggregationTemporality,
  MeterProvider,
  PeriodicExportingMetricReader,
} from "https://esm.sh/<EMAIL_ADDRESS>// Had to fallback to an npm specifier.
import { OTLPMetricExporter } from "npm<EMAIL_ADDRESS>
const honeycombTeam = Deno.env.get("HONEYCOMB_TEAM");
const honeycombDataset = Deno.env.get("HONEYCOMB_DATASET");

// Initialize our OTLP rig and return a Meter that we can use to register
// instruments.
const initOTLP = () => {
  // Register a service.name so this isn't set as 'unknown_service'.
  const resource = Resource.default().merge(
    new Resource({
      "service.name": "trackjobs",
    }),
  );

  const metricExporter = new OTLPMetricExporter({
    url: "https://api.honeycomb.io/v1/metrics",
    headers: {
      "x-honeycomb-team": honeycombTeam,
      "x-honeycomb-dataset": honeycombDataset, // Where is this reflected?
    },
    temporalityPreference: AggregationTemporality.DELTA,
  });

  const metricReader = new PeriodicExportingMetricReader({
    exporter: metricExporter,
    exportIntervalMillis: 60000,
  });

  const meterProvider = new MeterProvider({ resource: resource });
  meterProvider.addMetricReader(metricReader);
  metrics.setGlobalMeterProvider(meterProvider);

  const meter = metrics.getMeter("webapp");
  return meter;
};

const meter = initOTLP();

package.json
N/A. I'm using Deno.

Relevant log output
No response

Deno and workerd are not specifically supported so I may not be able to help you, but if there is a simple change that can be made to enable this I'm happy to do it. The initial issue may be already fixed as we had published a change related to our tslib dependency which broke importing in ESM mode, but that has been fixed in the latest release. Please give it a try and let me know.

I'll give that a try later on
The workaround for now has been importing directly from the esm build
import { getExportRequestProto } from '@opentelemetry/otlp-proto-exporter-base/build/esm/platform/util';

Hmm looks like this might be because we're not building ESM for @opentelemetry/exporter-metrics-otlp-proto. Opened #4097 to tackle this.

I'll remove the bug label for now, as Deno is a runtime we do not support at the moment.
I'll label it as enhancement instead so that we can track if #4097 or #4098 improves the situation.


Add python

Adding python 2 to the Travis build to see if it resolves the current build issue with libv8.

Thanks for trying this. I didn't know how to do it, and I was afraid that I
would break something.
Thinking out loud:

Is this installing Python3 or Python2?
Does libv8 need Python2?
Is order important, i.e., should it be sudo apt-get install python
exiftool  instead of sudo apt-get install exiftool python
Could we replace therubyracer? Maybe with miniracer?
https://github.com/rubyjs/mini_racer. Are there issues installing
miniracer on the VM?

On Sat, Dec 12, 2020 at 7:04 PM codeclimate[bot]<EMAIL_ADDRESS>wrote:

Code Climate has analyzed commit 982410f
https://github.com/MushroomObserver/mushroom-observer/commit/982410fff369571cded9c82210ce7c30d5bd5e6c
and detected 0 issues on this pull request.
View more on Code Climate
https://codeclimate.com/repos/58cfc16d1035280287001d93/pull/723.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/723#issuecomment-743940150,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFEULWVCF6GGCMQH333SUQVM7ANCNFSM4UZAEDOQ
.

I will try python2.
(I'll create a new branch. It may take ~1 hr because I'm having some issue with my laptop.)

Add Pivotal ticket. https://www.pivotaltracker.com/story/show/176132861

Same/similar issue installing python2 on Travis. See https://travis-ci.org/github/MushroomObserver/mushroom-observer/builds/749441904#L50659
Next I will try substituting mini_racer (note the underscore) for therubyracer.

The original PR did install python2 and got a lot further during the libv8 install process, but there was still a segfault.

mini_racer seems like a better approach.


Add multi asset deposit and withdraw to Balancer strategy

Added optional multi-asset deposit, withdraw and checkBalance to strategies
Added isMultiAssets  to the Strategy struct used by the Vault's strategies mapping. This config tells the vault whether a strategy supports multi assets.
Removed _deprecated from the Strategy struct so only one slot is ready from storage, not two. This is safe as the storage location of the mapped strategies does not overlap.
Changed Balancer deposit and withdraw from using a fixed amount of BPT to a fixed amount of pool assets
Withdraw of Balancer Pool Tokens from Aura adjusted for BPT already held in the strategy contract.
Balancer strategy implements checkBalance that uses the pool's virtual price (getRate) to value the Balancer Pool Tokens (BPT) back to ETH

This is not in scope of the Balancer strategy so will close


Buffer corruption.

Hello,
This started a couple of releases back I think. From time to time the buffer atom-typescript uses to compile the code gets mangled and it starts displaying weird errors and worst of all, the generated code (if you save & it automatically builds) is all broken.
The only solution I could find was to close the file and reopen it, which then gets rid of the errors and a subsequent save, generates a correct output file.
Is this affecting other users as well or is it just me? It's a very serious issue, but I failed to find another issue on the topic.
Thanks.

This just started happening to me after I updated Atom to 1.19.4.

Frequent occurrence for me, too.

This is likely also related to the issue introduced in TextBuffer in 1.19.4 (more info in #1333).

Pretty sure this can be closed. Resolution: update to Atom 1.19.5.

I am experiencing corruption with Atom 1.20.1 as soon as I have multiple buffers on the same file (right click and choose Split Right).

I recently experienced corruption too, but I couldn't replicate.


question - is it possible to get the progress of the parser

I'm looking for a callback hook to know how much progress (i.e. bytes out of total) the parser has completed. Is this possible ?

it's a nice idea for improvement (feel free to PR).
meanwhile, if you use the parse_fd() function, it should be easy to 'tell' the current offset from any callback using lseek(fd, 0, SEEK_CUR);


Support a debug terminal like JS Debug

The JavaScript debugger has a JavaScript Debug Terminal which you can open in the integrated terminal panel. From there, you can run arbitrary commands and if they execute your TS/JS files you can set breakpoints without ever having to set up a launch config.
This is useful for when the execution of your files is wrapped by other commands. e.g. you might a separate bash script to run a series of python files in different ways. Instead of creating a launch config for each file, you can just execute the bash script in the debug terminal. Additionally, you may want to change the arguments that you pass to the script multiple times while debugging. Leveraging the native terminal history is nice here. Lastly, you might just not want a launch config. They seem somewhat permanent, and you might just want this while you are writing a script.

upvote

#321 had 5 upvotes - consider this when ranking for importance


Fix "n" key bugs.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @showell!


Set Dashboard - Part 1

Hi,
set dashboard controller, views and assets properly. for example: dashboard should be open if I open http://localhost:port/myshopping/dashboard link.
You need to create below functions / pages.
Function Name:   index
Function File:       dashboard.php (controller)
URL: http://localhost:port/myshopping/dashboard
View File   applications / views / dashboard / home
Function Name:   login
Function File:       dashboard.php (controller)
URL: http://localhost:port/myshopping/dashboard/login
View File   applications / views / dashboard / login
Function Name:   logout
Function File:       dashboard.php (controller)
URL: http://localhost:port/myshopping/dashboard/logout
View File   applications / views / dashboard / logout
Function Name:   settings
Function File:       dashboard.php (controller)
URL: http://localhost:port/myshopping/dashboard/settings
View File   applications / views / dashboard / settings
Function Name:   add_product
Function File:       dashboard.php (controller)
URL: http://localhost:port/myshopping/dashboard/add_product
View File   applications / views / dashboard / add_product
Function Name:   view_all_products
Function File:       dashboard.php (controller)
URL: http://localhost:port/myshopping/dashboard/view_all_products
View File   applications / views / dashboard / view_all_products
Regards,

some pages are not in our project(currently) .. but i've created all required pages..!!

re-open

Done sir !


Smart-home Power Monitor

The device is not integrated
Device normal info

tuya
PG-01

product link

Can you provide your device info and device specifications?

This is the product.  In tuya IoT Platform these data are reported

There are updates?


Url encode spaces

When the scope is https://ads.microsoft.com/ads.manage offline_access some IDE's don't allow for clicking the whole url.

Nice catch!
I wonder if it would also work to use rawurlencode to get any other characters?

You could but then it would convert all characters to their entity equivalent.
So https:// would turn to https%3A%2F%2F. Not super friendly to look at.

True. I'll get this into the next release!
Feel free to join our Discord channel


Support accessing type arguments from smoke

I'm trying to use smoke for some simple JSON parsing. It all works fine; except that when a type I'm deserialising into has a List<x>, it seems that smoke does not provide any ability to get at the generic type.
Currently I'm working around this (it's just a prototype) by hard-coding a list of the types I know I'm deserialising lists of:
var s = smoke.nameToSymbol(k);
var d = smoke.getDeclaration(t, s);
if (d.type.toString() == "List<ClientGridColumn>")
  smoke.write(obj, s, v.map((sv) => Serializable.fromJson(ClientGridColumn, sv)).toList());
This obviously isn't a scalable solution. It seems like this is the sort of thing that should be possible with smoke; so I hope it might be considered!

This issue has been moved to dart-lang/smoke#15.


[cri/config] : fix range iterator issue in ValidatePluginConfig

Go uses the same address variable while iterating in a range, so use a copy when using its address.
Sample code to highlight the issue: https://play.golang.org/p/YSqm1Gu3O0i

Please sign your commit to pass CI validation

@estesp The error appears to be: FAIL - commit subject exceeds 90 characters . Is there a character limit on the subject ?

yes, commit messages should start with a short "subject" line, then a blank line and then any more informative/descriptive text and then sign-off. In your case because that string of text just continues into a full sentence it breaks the 90-char limit check. The title of this PR would be a better commit subject and the current text should be the longer detail. Example:
This is my commit title

This is why I made this commit. Here are some more details on
what is solves.

Signed-off-by: ....

Thanks @estesp . Changes made.

LGTM


Adds support for custom transport protection algorithms

Motivation:
In some cases AES GCM might not be supported by a server/client, and
clients might require custom algorithms to be implemented in their
projects.
Modifications:

Makes NIOSSHTransportProtection, NIOSSHSessionKeys and ExpectedKeySizes public
Makes transport algorithms list configurable
Adds sequence of incoming and outgoing messages to parser and
serializer
Fixes connection state machine state update if message can not be read yet
Tests

Agreed, but at least it should be referenced there, thank you!


Added tests for AddIntegerType and AddFloatType

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Superseded by this PR


Feature: Extra Examples (outside a simple the viewer)?

I'm currently working with masse, torques, etc, seeing this series. And was hoping to take the examples directory a bit further (Although I'm not sure how much of this is practical). By creating sort of a tutorial/example with a simple pendulum to obtain units of torque, tuning values of kp, kd, ki etc...
A graphing of PID error like the profiler/sensor section of the simulate viewer which generates live graphs.
Perhaps a wiki with these:

[x] 4-bar-mechanism example, for creation of constraints
4-bar-example
[x] Simplified mujoco_viewer.py in extended-example-tutorial commits: 97783ab 5839301 aab2d48
[ ] Add Sensor Graphs
[ ] PID tuning and example in wiki

Existing graphing:

Add Sensor Graphs

I understand that you want to add more features in the current viewer, but please note that there is an ongoing effort in the official mujoco github to create python bindings for simulate.cc: https://github.com/deepmind/mujoco/pull/201
Once that is merged, there would most probably be no need for mujoco-python-viewer :)

Added graph rendering for viewing, although UI elements for python are not supported in the mujoco yet #23


Listgroup

📑 Description
List group re-written.

New API
Extended examples
Introduction of ListItem although hidden in majority of cases

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed
[x] My pull request is based on the latest commit/version

ℹ Additional Information

Thank you for the PR.


Update to v1.23.11

Signed-off-by: matttrach<EMAIL_ADDRESS>

Codecov Report
Base: 9.77% // Head: 9.77% // No change to project coverage :thumbsup:

Coverage data is based on head (41e89ca) compared to base (48fa8a6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##           release-1.23   #6144   +/-   ##
============================================
  Coverage          9.77%   9.77%
============================================
  Files               136     136
  Lines             10003   10003
============================================
  Hits                978     978
  Misses             8826    8826
  Partials            199     199

Flag
Coverage Δ

unittests
9.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Drone CI: https://drone-pr.k3s.io/k3s-io/k3s/4719
(failed)

Done CI: https://drone-pr.k3s.io/k3s-io/k3s/4720/1/4
failed - flakey test? restarting

Drone CI: https://drone-pr.k3s.io/k3s-io/k3s/4723/1/2
failed - "no space on device"

sorry, Drone passed: https://drone-pr.k3s.io/k3s-io/k3s/4726


APN - Unable to push to connected devices from server

//config.push.json
{
  "apn": {
    "passphrase": "AWESOMEPASS",
    "key": "pushKEY.pem",
    "cert": "pushCERT.pem",
    "gateway": "gateway.sandbox.push.apple.com"
  },
  "production": false
}

//CLIENT init & alert w/ token
Push.init({});
  Push.addListener('token', function(token) {
    if (token.apn) {
      alert('Got ios token ' + token.apn);
    }
  });

//TEMPLATE EVENT
Template.layout.events({
    "click .push-link": function () {
      Push.send({from: 'push',title: 'Hello',text: 'World',badge: 99,query: {}});
    }
  });

//SERVER init & Debug flag
if (Meteor.isServer) {
  Meteor.startup(function () {
    Push.debug=true;
    console.log(Push.send({
      from: 'Test',
      title: 'Small message',
      text: 'Hello world',
      query: {} // Query the appCollection
    }));

    Push.allow({
        send: function(userId, notification) {
            return true; // Allow all users to send
        }
    });

  });
}

//CONSOLE OUTPUT
20150102-15:15:49.264(-6)? (STDERR) WARNING: Push APN is in development mode
I20150102-15:15:49.339(-6)? QFdPio68DGKawvCJ9
=> Meteor server restarted
I20150102-15:16:04.294(-6)? Push: Send message "Small message" via query {}
I20150102-15:16:04.296(-6)? Push: Sent message "Small message" to 0 ios apps 0 android apps
I20150102-15:16:34.304(-6)? Push: Send message "Hello" via query {}
I20150102-15:16:34.306(-6)? Push: Sent message "Hello" to 0 ios apps 0 android apps

//METEOR SHELL - Push.appCollection.find()
{ _mongo:
   { _connectCallbacks: [ [Function] ],
     _observeMultiplexers:
      { '{"ordered":false,"collectionName":"_raix_push_notifications","selector":{},"options":{"transform":null}}': [Object],
        '{"ordered":false,"collectionName":"_raix_push_app_tokens","selector":{},"options":{"transform":null}}': [Object],
        '{"ordered":false,"collectionName":"meteor_accounts_loginServiceConfiguration","selector":{},"options":{"transform":null,"fields":{"secret":0}}}': [Object],
        '{"ordered":false,"collectionName":"photos","selector":{},"options":{"transform":null}}': [Object],
        '{"ordered":false,"collectionName":"users","selector":{},"options":{"transform":null,"fields":{"profile":1,"username":1}}}': [Object] },
     _onFailoverHook: { nextCallbackId: 10, callbacks: [Object] },
     _docFetcher:
      { _mongoConnection: [Circular],
        _callbacksForCacheKey: {} },
     _oplogHandle:
      { _oplogUrl: 'mongodb://<IP_ADDRESS>:3001/local',
        _dbName: 'meteor',
        _oplogLastEntryConnection: [Object],
        _oplogTailConnection: [Object],
        _stopped: false,
        _tailHandle: [Object],
        _readyFuture: [Object],
        _crossbar: [Object],
        _lastProcessedTS: [Object],
        _baseOplogSelector: [Object],
        _catchingUpFutures: [] },
     db:
      { domain: null,
        _events: {},
        _maxListeners: 10,
        databaseName: 'meteor',
        serverConfig: [Object],
        options: [Object],
        _applicationClosed: false,
        slaveOk: false,
        bufferMaxEntries: -1,
        native_parser: false,
        bsonLib: [Object],
        bson: [Object],
        bson_deserializer: [Object],
        bson_serializer: [Object],
        _state: 'connected',
        pkFactory: [Object],
        forceServerObjectId: false,
        safe: false,
        notReplied: {},
        isInitializing: true,
        openCalled: true,
        commands: [],
        logger: [Object],
        tag:<PHONE_NUMBER>176,
        eventHandlers: [Object],
        serializeFunctions: false,
        raw: false,
        recordQueryStats: false,
        retryMiliSeconds: 1000,
        numberOfRetries: 60,
        readPreference: [Object] },
     _primary: '<IP_ADDRESS>:3001' },
  _cursorDescription:
   { collectionName: '_raix_push_app_tokens',
     selector: {},
     options: { transform: null } },
  _synchronousCursor: null }

On starting the app the iOS device alerts token, I can push to it via APN tester using the dev cert for this app ID.
How else can I debug this?

I'm having similar issues on my end as well.  @dennisharrison are you using an APNs Developmen iOS certificate?

Lets focus on the Push.appCollection.find().fetch() to see if any tokens have been registered on the server.
Push: Sent message "Hello" to 0 ios apps 0 android apps - this shows that the send command found no apps in Push.appCollection

@raix Push.appCollection.find().fetch() is empty.  I've not taken insecure off of this project yet, so I figured it would register every device connecting?  Another message I got when trying to send a message from the server is:
W20150103-08:51:19.024(-6)? (STDERR) Error: Meteor.userId can only be invoked in method calls. Use this.userId in publish functions.
W20150103-08:51:19.024(-6)? (STDERR)     at Object.Meteor.userId (packages/accounts-base/accounts_server.js:19:1)
W20150103-08:51:19.024(-6)? (STDERR)     at EventEmitter.Push.send (packages/raix:push/lib/common/notifications.js:37:1)

So I tried changing this method to use this. instead of Meteor. - but I'm not sure that's helpful other than squelching the error?
Push.send = function(options) {
  // If on the client we set the user id - on the server we need an option
  // set or we default to "<SERVER>" as the creator of the notification
  // If current user not set see if we can set it to the logged in user
  // this will only run on the client if Meteor.userId is available
  var currentUser = Meteor.userId && Meteor.userId() ||
          Meteor.isServer && (options.createdBy || '<SERVER>') || null;

@idmontie Yes, using a development specific certificate - reading through the longer issue about APN help here I see that one of the troubleshooting steps is to use a production certificate instead.  I'm not sure I'm to that problem yet though?   Shouldn't we see the collection updated either way since the device is grabbing a token anyway?

@dennisharrison could you try the 2.4.1 version,

OK, so that Push.send stuff on server is proper now it looks like:
I20150103-11:23:15.193(-6)? Push: Send message "Small message" via query {}
I20150103-11:23:15.199(-6)? Push: Sent message "Small message" to 0 ios apps 0 android apps
I20150103-11:23:15.201(-6)? Push, GUIDE: The "Push.appCollection" is empty - No clients have registred on the server yet...

Still though when looking at Push.appCollection.find().fetch(), it's an empty array.
Digging around earlier I did uncomment:
// XXX: should we do a test of the connection? It would be nice to know
        // That the server/certificates/network are correct configured

        // apnConnection.connect().then(function() {
        //     console.info('CHECK: Push APN connection OK');
        // }, function(err) {
        //     console.warn('CHECK: Push APN connection FAILURE');
        // });
        // Note: the above code spoils the connection - investigate how to
        // shutdown/close it.

The return was connection OK.  I've since recommended it, plus I just overwrote everything with your latest version from github.
As a sanity check, should I also look into using Push.sendAPN() to the device to make sure it works?  What is the syntax for APN, just a string?  Do you have an example of sendAPN I can dig up and modify to test that it's not a certificate issue?
Also, where should I look to try and debug why the client isn't trying to add itself to the collection?

@dennisharrison The sendAPN and sendGCM is really internal api / advanced / deeper integration api - Its documented in ADVANCED.md - I would recommend that you get the client up and running (sending tokens to the server into Push.appCollection)
It should be enough to set credentials in config.push.json and use the Push.send command - this is the basics.
The idea is to depend on the appCollection for sending out notifications - its automatically maintained when services invalidates tokes etc. Its possible to add metadata to the appCollection data, still WIP but this will allow almost every thinkable usecase, still keeping the api isomorphic and simple (Push.setMetadata is implemented - but still in flux)

@dennisharrison I did add the debug info Push, GUIDE: The "Push.appCollection" is empty - No clients have registred on the server yet... You gotta get the clients working before playing around on the server-side.

Add the quick token test:
client.js:
Push.addListener('token', function(token) {
  alert('Token: ' + JSON.stringify(token));
});

Btw. dont call Push.init({}); ever :) Ive deprecated it in favor of the config.push.json file (Its replaced by Push.Configure but its again only for advanced use - it will throw errors if used with the config.push.json)

I've taken out Push.init({}) - it will forever be banished :)
I get the alert with the proper apn token.  What does the options array need to look like to send a simple test message with Push.sendAPN()?
I'm using:
Push.sendAPN("46459ee3026471cd990dd567b9}103aa01d0fe93560c92ace26b3dd6c1511e2da",{from:"test",title:"test messaage",text:"hello test",createdAt:new Date(),createdBy:"test"});

And I've got some logs in the sendAPN function to make sure that it's organized in the way I think it should be based on the other methods:
//userToken
46459ee3026471cd990dd567b9}103aa01d0fe93560c92ace26b3dd6c1511e2da

//myDevice
{ token: <Buffer 46 45 9e e3 02 64 71 cd 99 0d d5 67 b9 10 3a a0 1d 0f e9 35 60 c9 2a ce 26 b3 dd 6c 15 11 e2 da> }

//note
{
	encoding: 'utf8',
	payload: { messageFrom: 'test' },
	expiry:<PHONE_NUMBER>,
	priority: 10,
	retryLimit: -1,
	device: undefined,
	alert: 'hello test',
	badge: undefined,
	sound: undefined,
	newsstandAvailable: undefined,
	contentAvailable: undefined,
	mdm: undefined,
	compiled: false,
	truncateAtWordEnd: false,
	urlArgs: undefined,
	category: undefined
}

However, the message does not show on the device (I don't know what the expectation should be here).
I will leave the sendAPN alone for now and concentrate on getting the client in the collection.
Still the Push.appCollection.find().fetch() is an empty array for me.

@dennisharrison the 2.5.0 got some bug fixes - I havent had the time to properly test this on android - but now it works (and added much better payload, its now consistent on both platforms and preserves types via ejson)
I'm not sure why the token isnt sent to the server - when the mobile app is started the server should log something like Push: Got push token from app:

Updating to 2.5.0 - can you point me in the direction of the code that's supposed to update the server from the client?  Which method should do it?  I'm going to be poking around either way, but would appreciate any hints :)

https://github.com/raix/push/blob/master/lib/client/client.js#L112
https://github.com/raix/push/blob/master/lib/server/server.js#L13
Im turning in for now

@raix Thank you, I'll keep plugging away and see where I can get by tomorrow morning!

@dennisharrison I have a todo to write up some help on dealing with certificates... sorry have not got there. Here's a short version that may help you debug this.. my guess is certificates are probably not configured correctly.
The general approach is get this working in development config, then once working swap to production. As per https://github.com/raix/push/blob/master/docs/IOS.md
For now just do it for development (so a subset of above) and you'll be using the config file with the development/sandbox settings. It seems like you have that covered, so I won't detail.

in Member Center create App ID with explicit option:

And Push on:

Edit that App ID and create a Development SSL Certificate.
Go and create the PEM files as per instructions https://github.com/raix/push/blob/master/docs/IOS.md

Create a provsioning profile for iOS app Development

Use the AppID from (1) above

run Meteor with your phone connected via cable. meteor run ios-device --mobile-server http://<local ip of server>

You should see them app open on your phone etc etc
OK, so as per above it seems you have the Raix-Push stuff working and you see the token in the logs. Copy it.
Go grab one of the free (or paid) APN test tools - you can download from the OSX App Store.
From here you should be to send a hardcoded Payload  . I use:

You choose your certificates.
So, outside of the Meteor code you should be able to send your App a hardcoded notification. I found this helpful as it removed any issues in my code and validated the certificates were working.
If this does not work, I am guessing something is messed up with either the SSL certificates or the provisioning profile.

@raix I've got an alert right before:
https://github.com/raix/push/blob/master/lib/client/client.js#L91
And I see it on the phone, but I've got a console log in:
https://github.com/raix/push/blob/master/lib/server/server.js#L14
And I don't see anything in the server console.  Any thoughts on how to figure out what's happening here?
@adamgins It's really good to see you in this issue, I read all the details of yours over and over so far :)
I can push directly to the device using APN Tester.  Thank you for the specifics on certificates and I think we should collect all the information we have and help Raix on the documentation front, I will have no issues doing that as soon as I can get this working!  Feel like I'm close at this point.

Stoked!!! re: documentation - Go go for it.
can you just add some extra console.log to see where it's going? .. assuming Push.debug has been set for the server.

@adamgins I don't know where else to add the console log for this method!  I've added it in the very first line of the method, and I guess that means it's not being called?  How do you debug Metor.call not running the server method?
I have console logs after:
https://github.com/raix/push/blob/master/lib/client/client.js#L91
and after:
https://github.com/raix/push/blob/master/lib/server/server.js#L14
Push.debug is set - certainly.
I can send messages directly from the meteor shell by using Push.sendAPN() since setting the provisioning profile like you suggested! (!!!)
So, all I need to do now is sort out how to get this token inserted into the db - so close, oh so close, I can tell.

@dennisharrison does the mobile app have connection to the meteor server? sometimes you have to set the --mobile-server flag in order for the mobile app to connect to the server.
// Add the connection helper in the template {{connection}}
Template.hello.helpers({
  connection: function() {
    return Meteor.status().status;
  }
});

@raix this could be the biggest 'duh' moment for me if you're right :)
I'm going to check for this right now, but I've been building the app / running the server using the --mobile-server :port command.  Is it --mobile-server or --server?  Update this shortly.

@dennisharrison Super - I think it would benefit the community with a step by step walkthrough - If you need a place for it we could create a markdown or add it in the wiki. (gist are ok for pure code - but some screen dumps would help too)

@raix Yes, create the page you would like on the wiki and give me access or tell me where you want it and I'll add some screenshots too.

I've added you guys as collaborators
You're welcome to edit / create pages etc. in the wiki - I've added a custom sidebar for better HOC - We could move the stuff in /docs folder into wiki too at some point.
(If you want to edit the code please create a dev/test branch keeping the master clean :) or clone and make pr's)

Got it, working on the development APN writeup right now, stepping through another app to make sure I've got all the stuff right, then I'll do the production writeup.

Steps I should have followed are here:  https://github.com/raix/push/wiki/iOS-Micro-Walkthrough
@raix Please feel free to close the issue - Thank you!

Looks great @dennisharrison Thanks for the iOS writeup!

Agreed - awesome job!!!


docs(log-messages): log messags hints for rosbag replay simulation users

Description
This PR is one of group of PRs that aim to fix Autoware logging system to achieve the goal of reducing excessive error and warning logs on Autoware launch.
Part of:

https://github.com/autowarefoundation/autoware.universe/issues/5539

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The Reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@xmfcx
I am facing a problem with autofix commits.

When the !!! warning is in the same indentation with numbering, it fails to see the order and hence expects point # 2 to be # 1.

When the !!! warning is in the same indentation with the text above it, it deletes the space under !!! warning and makes the warning message as same indentation as !!! warning .


dit inference model

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


tour: Outputting ASCII 0xC causes output on web page to be different than output on computer

Context: https://tour.golang.org/methods/12
On the Exercise: rot13Reader
I did it incorrectly at first which caused the output to contain an ASCII 0xC (NP form feed, new page) which causes the output to be cleared so instead of what you would see on a local computer:
b len 32768
count 21 err <nil>

b len 32768
count 0 err EOF

You only see:
b len 32768
count 0 err EOF

Program exited.

It made  me think I was doing something wrong in reading the string. Here's a full program that will reproduce the problem:
import (
	"fmt"
	"io"
	"os"
	"strings"
)

type rot13Reader struct {
	r io.Reader
}

func (rot *rot13Reader) Read(b []byte) (count int, err error) {
    fmt.Println("b len", len(b))
    count, err = rot.r.Read(b)
    fmt.Println("count", count, "err", err)
    for i := 0; i < count; i+=1 {
        b[i] = 12 // NP form feed, new page
    }
    return count, err
}

func main() {
	s := strings.NewReader("Lbh penpxrq gur pbqr!")
	r := rot13Reader{s}
	io.Copy(os.Stdout, &r)
}

CC @campoy

I don't see how to fix this other than adding a warning regarding special ASCII characters.
I don't want to change the current behavior as some cool programs depend on it, such as this awesome ASCII train: http://play.golang.org/p/NOycgN2i6b

It probably wouldn't have helped. If other stuff relies on this behavior then I guess it doesn't make sense to change it.

Thanks for reporting!


Display help output on stdout #8980

This is a small adjustment of the work done by @rhatdan and @tiborvass. It
suppresses redundant error messages and handles a case where docker is run with
a bad option without a command.
Fixes #8980 and closes #7887

@jfrazelle: Rebase done. For the last time I hope...
Thanks for taking care of it.

We have only been working on this one for 9 months now...

great LGTM, ping @tiborvass :)

LGTM
If we discover other inconsistencies, we can fix them in a separate PR. I agree this took way longer than all of us wanted. Thanks everyone for your patience.