heroku : libX11-xcb.so.1: cannot open shared object file: No such file or directory :  "env" in launch[Options] should be marked as a dangerous config field, remind to not override.

Please refer to my 3rd post.

(node:4) UnhandledPromiseRejectionWarning: Error: Failed to launch chrome!
Mar 02 01:34:58 march2 app/worker.1: /app/node_modules/puppeteer/.local-chromium/linux-624492/chrome-linux/chrome: error while loading shared libraries: libX11-xcb.so.1: cannot open shared object file: No such file or directory

Just wasted FOUR hours debugging this, the issue is not Chromium version
The issue is because  I used " env: " in launchOptions to set a timezone, the doc says it defaults to process.env, but I override it, cause puppeteer cannnot find the path.
        const env = process.env || {};
        env.TZ = this._options.timeZone;

fixed the issue

Just wasted FOUR hours debugging this

@swang2019 sorry to hear this! :( If you think we can do a better job with documentation somewhere, please feel free to submit a PR.

Having the same "$ react-snap
Error: Failed to launch chrome!
/tmp/build_53c404e90d717f6d700d6eb26d17d426/client/node_modules/puppeteer/.local-chromium/linux-641577/chrome-linux/chrome: error while loading shared libraries: libX11-xcb.so.1: cannot open shared object file: No such file or directory"
How did you fix this? What .env variable needs to be set for chromium to run correctly?
Thanks for your help.

Where you tried to deploy Chrome ?

On Apr 19, 2019, at 5:48 AM, kelseygasser<EMAIL_ADDRESS>wrote:
Having the same "$ react-snap
Error: Failed to launch chrome!
/tmp/build_53c404e90d717f6d700d6eb26d17d426/client/node_modules/puppeteer/.local-chromium/linux-641577/chrome-linux/chrome: error while loading shared libraries: libX11-xcb.so.1: cannot open shared object file: No such file or directory"
How did you fix this? What .env variable needs to be set for chromium to run correctly?
Thanks for your help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/GoogleChrome/puppeteer/issues/4098#issuecomment-484684407, or mute the thread https://github.com/notifications/unsubscribe-auth/ALC2CQMSQX3R3LUBIKW4C6DPRDNBTANCNFSM4G3JEDWA.

I tried to deploy a react/node/express app on heroku with react-snap. I think inadvertently it added chrome. I tried adding react-snap and puppeteer config items to my package.json

Did you using “env:” when launching the browser ?

On Apr 19, 2019, at 11:11 PM, kelseygasser<EMAIL_ADDRESS>wrote:
I tried to deploy a react/node/express app on heroku with react-snap. I think inadvertently it added chrome. I tried adding react-snap and puppeteer config items to my package.json
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/GoogleChrome/puppeteer/issues/4098#issuecomment-484907271, or mute the thread https://github.com/notifications/unsubscribe-auth/ALC2CQJODHXOR5KAFKHWIBLPRHHIXANCNFSM4G3JEDWA.

I am not launching the browser. I believe react-snap is. So I am not sure where to place this "env"

OK, Do you add the puppeteer build packs into your heroku project?

On Apr 19, 2019, at 11:18 PM, kelseygasser<EMAIL_ADDRESS>wrote:
I am not launching the browser. I believe react-snap is. So I am not sure where to place this "env"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/GoogleChrome/puppeteer/issues/4098#issuecomment-484909324, or mute the thread https://github.com/notifications/unsubscribe-auth/ALC2CQL6OCBDER56NXNWNLDPRHIEXANCNFSM4G3JEDWA.

Yes, I have done that

Then sorry, I don’t know what’s your issue..

On Apr 19, 2019, at 11:23 PM, kelseygasser<EMAIL_ADDRESS>wrote:
Yes, I have done that
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/GoogleChrome/puppeteer/issues/4098#issuecomment-484910820, or mute the thread https://github.com/notifications/unsubscribe-auth/ALC2CQKKI4PXUWN3PSNLYN3PRHIXZANCNFSM4G3JEDWA.

Thanks for responding anyway!

sudo apt install -y gconf-service libasound2 libatk1.0-0 libc6 libcairo2 libcups2 libdbus-1-3 libexpat1 libfontconfig1 libgcc1 libgconf-2-4 libgdk-pixbuf2.0-0 libglib2.0-0 libgtk-3-0 libnspr4 libpango-1.0-0 libpangocairo-1.0-0 libstdc++6 libx11-6 libx11-xcb1 libxcb1 libxcomposite1 libxcursor1 libxdamage1 libxext6 libxfixes3 libxi6 libxrandr2 libxrender1 libxss1 libxtst6 ca-certificates fonts-liberation libappindicator1 libnss3 lsb-release xdg-utils

@wubigo Do you know how to install lib in CentOS? thank you.

@wubigo Do you know how to install lib in CentOS? thank you.

https://github.com/puppeteer/puppeteer/blob/master/docs/troubleshooting.md#chrome-headless-doesnt-launch-on-unix

sudo apt install -y gconf-service libasound2 libatk1.0-0 libc6 libcairo2 libcups2 libdbus-1-3 libexpat1 libfontconfig1 libgcc1 libgconf-2-4 libgdk-pixbuf2.0-0 libglib2.0-0 libgtk-3-0 libnspr4 libpango-1.0-0 libpangocairo-1.0-0 libstdc++6 libx11-6 libx11-xcb1 libxcb1 libxcomposite1 libxcursor1 libxdamage1 libxext6 libxfixes3 libxi6 libxrandr2 libxrender1 libxss1 libxtst6 ca-certificates fonts-liberation libappindicator1 libnss3 lsb-release xdg-utils

What about with the Heroku deployment issue, would this help?

sdfsfd

sudo apt install -y ...

FYI, to avoid an interactive debconf setup of tzdata (screenshot) stalling my CI server, I had to prefix the above command with (per this answer):
DEBIAN_FRONTEND=noninteractive TZ="America/New_York" sudo apt install -y ...


[Planting Flow] Make Log-In less obnoxious when starting

Currently when you want to plant a tree it forces & redirects the user to the login screen which is annoying / frustrating when you start the flow.
It does work as it eliminates the step for asking why and makes you create an account. But on a ux side, its not the best approach as it can be confusing / make a user lost when that occurs since you have to press plant again to get started. We need to eliminate that aspect when you begin the flow.
My proposal is to have a box at the bottom of the screen asking to "create an account" or "login to continue" which may not be obvious, but is a good way to start off the flow.

Started a first pass on how to make the planting sign-in less annoying as a feature.
So I am proposing to not force users to sign-in immediately. Rather than going straight to the sign-in / signup screen, users will get a notification on the screen stating to sign-in or sign-up to get started (we need to word it so its encouraging to use the app). So once the notification slides in, they can make a decision rather than the app forcing them to do something they didn't know was going to happen.
@mwpark2014 is this what you were thinking as a less obnoxious route to sign-in?
@zoobot what are your thoughts on this?

Yes, exactly!! Very clear and minimalist. Although I'm guessing we'll keep the tree layer and geolocate button around for logged out users

will close this out as this seems to be a good first pass, will iterate if needed.


Image in README.md file not displaying correctly

Description
Description
The image in the README.md file is not displaying correctly.
Steps to Reproduce

Open the README.md file in a web browser.
Observe that the image is not displayed.
Click on the suposed image, it dislays an error 404 page with the following message ; The 'krshkun/azelf' repository doesn't contain the '.github/assets/azelf.png' path in 'main'.

Expected Result
The image should be displayed correctly in the README.md file.
Actual Result
The image is not displayed.
Additional Context
I am using the following browser and operating system:

Browser: Chrome
Operating System: Windows

Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

there is no image named azelf in the assets folder

Yes, there isn't 👍 Will create one in a day or two ✅

Thank you for opening the issue @Azeezkadiri and thank you for validating @devvsakib

Thanks

I’m new to open source and this is my first contribution. I hope to be more
valuable in the future to the community 😁

You will sure be! It's noticing these small things that can improve a project that make you a great OSS contributor!


Handle default values from providers' schemas in codegen and codecs

Inside definitions of providers' schemas elements of properties sections (in definitions of types and resources) as well as  inputProperties (for resources only) default values can be specified. These should be taken into account in codegen and in codecs so that users don't have to specify them explicitly and (de-/en-)coding doesn't fail when a property with a default value is missing

I think we might already have this, but needs to make sure we cover all the cases.


cmd window auto closes

i set up config and then when  i open the cmd window it auto closes

] [ERROR] '<type 'exceptions.AttributeError'>' 'NoneType' object has no attribute 'connected'

I will consider this invalid, until you provide more information.

https://i.imgur.com/DGrpc4s.png

The disconnect is related to #24
One reconnect the captcha was required
If the captcha is required we close the connection
So you need to complete the captcha

Granted, the way this disconnects needs to be handled with a bit more grace.

Granted, the way this disconnects needs to be handled with a bit more grace.

Looks like the error is from within websocket-client, I don't really see anywhere else in tinychat.py that would cause this.

If the captcha is required (and anti-captcha api key is not present) we close the connection

Might be worth adding an additional message with something along the lines of "Please complete the captcha using this account and retry.", so the end users know this is expected behavior.
After the site key posted.

How much does this anti captcha cost? i seen something about it but im unaware on it

and how would i go about making !8ball,!Roll and all public so non mods can also use them ?

That's completely up to you to sort out.

is there a line  i can add to ban all users who post a tinychat url? i did !bs *tinychat.com dont seem to work well even tho thats what the commands file says to do

does it ban them then they come right back

no it just lets them post the link and dont even kick them

does nothing

Moving this to #37

With most current version using !bs *tinychat.com does work for me however it only kicks it doesn't ban, no setting to change this within config or bot commands, haven't looked around the code to see where this could be changed manually.

@gloriousone https://github.com/nortxort/nortbot/blob/master/config.ini#L31

@Webby7866 string bans work for me aswell, so i'm thinking this is user error more so than an issue with the bot. If you get an error (a traceback) fill out an issue request and open a new issue.

https://i.imgur.com/mqTc9jc.png here is the string file


Link to racing leaderboards project

Welcome back to the community :) and thanks for the PR.
The leaderboard project is really neat, but we want to avoid linking to external projects from our README, so we've decided to reject this change. We appreciate your understanding!
Closing this PR.

That's understandable 🚀


Astronaut_runner

PR Description 📜
Created a new game called Astroaut runner.
Fixes #3331

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.

[x] I have performed a self-review of my own code or work.

[x] I have commented my code, particularly in hard-to-understand areas.

[x] My changes generates no new warnings.

[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE

[x] I have added screenshot for website preview in assets/images

[x] I have added entries for my game in main README.md

[x] I have added README.md in my folder

[x] I have specified the respective issue number for which I have requested the new game.

Thank you soo much for contributing to our repository 💗

@tanishkaa08 resolve the merge conflicts !

Thank you  @tanishkaa08 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

please review it.

and if possible tell the issue and also how to solve it.

Thank you  @tanishkaa08 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Support logs with multiple lines

In our case, we have logs that either by our code or third parties, the reality is that they can generate multi-line logs.
For example, for a log to output a stacktrace:
2024-01-01 12:00:00,000 [1] DBG [TestClass] - A text
other text
other text2
2024-01-01 12:00:00,000 [1] DBG [TestClass] - A text 2
other text3
other text4

The problem comes that although my regex works in regex101.com, I see that the implementation makes it read the file line by line and execute the regex line by line instead of as a whole:
https://github.com/Analogy-LogViewer/Analogy.LogViewer.RegexParser/blob/d0b36774f5fb4b5e144bd55a63599421cf7d728d/Analogy.LogViewer.RegexParser/AnalogyRegexParser.cs#L271
This makes my regex not work obviously because it expects the input of the regex to be the content of the whole file and not line by line.
Would it be possible to support this different behavior when executing the regex? I know that the feedback is not the same because by not reading line by line you can't report progress (messagesHandler.ReportFileReadProgress()). But we are talking about that not all log files size is 100MB or more.
If so I could lend a hand with a PR if needed.
Thank you.

Each data provider factory can have multiple different parser so this can be an addition to the existing one:
https://github.com/Analogy-LogViewer/Analogy.LogViewer.RegexParser/blob/d0b36774f5fb4b5e144bd55a63599421cf7d728d/Analogy.LogViewer.RegexParser/IAnalogy/RegexDataProvidersFactory.cs#L14
we can can create new provider, something like RegexFullFileOfflineDataProvider. You can implement it in a pr if you wish or I will try to get to it next week.


Add GH action workflow for integration tests

Link to JIRA ticket if there is one: GQE-201
New Features
Breaking Changes
Bug Fixes
Improvements
Dependency updates
Deployment changes

@haraprasadj @krishnaa05 @mfshao This new integration_tests check had been added to this repo's required checks, which prevented other PRs from being merged. New checks should only be added to the list of required checks after they are added to master branch, I removed it and we can add it again when this PR is merged

@haraprasadj @krishnaa05 @mfshao This new integration_tests check had been added to this repo's required checks, which prevented other PRs from being merged. New checks should only be added to the list of required checks after they are added to master branch, I removed it and we can add it again when this PR is merged

Sorry, I forgot to revert it, thought this was ready to merge but got blocked by random test failures


Checks if labels are not empty

fixes #4 ?

I incorporated the fix into the revised version of getDataFrames.


Unable to play video on 3G or LTE service

Awesome work on plugin.
We are using this plugin for our app, but on WIFI its working fine, while 3G or LTE network without any error it silently close itself.
Could you please let us, what could be wrong here, if you can guide me I can look myself and can send you pull request.

why closed? the same question, and i use the latest version, but still have this issue

@maxroom can you open a new ticket? this one has no details and we have a template for information we want to get from you


Add safehelpline.mil

Found this: http://www.safehelpline.mil

Thanks for finding this and submitting it! In the future, new domains that aren't found by the scan need to get added to mil.csv, since dotmil-domains.csv is auto-generated by the scan. I'll add something to try to make this clear to the readme.

Can do!


Chat - Multiline link structure changes if user adds emoji when editing

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app or go to staging.new.expensify.com
Navigate to any conversation
Send a message with a multiline hyperlink in the following format:
multilinex
Text
Click/Tap on the Edit icon to activate the edit message box
Edit the message and add any emoji inside message but do not edit the link

Expected Result:
Edited message still displays the hyperlink with the multiline format
Actual Result:
Multiline link structure changes if user adds emoji when editing
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
https://expensify.testrail.io/index.php?/tests/view/3467098
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://platform.applause.com/services/links/v1/external/dd338a69358f7772aacd3f4ed98a0cef9e9b8baf591b83be3a1e51a57fec5c89
https://platform.applause.com/services/links/v1/external/78260a7b242fea9b7dc94c34a678643252deedf770a6bd504159c0d08e5eeaf1
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Multiline link structure changes if user adds emoji when editing
What is the root cause of that problem?
We're validating if the anchor text contains only emojis in the 'link' with rule in here
What changes do you think we should make in order to solve the problem?

We need to check if the anchor text g1 is multiline:
      if (g1.includes('/n') && g1.match(CONST.REG_EXP.EMOJIS) || !g1.trim()) {
          return match;
      }

What alternative solutions did you explore? (Optional)
NA
Result

As confirmed from this comment https://github.com/Expensify/App/issues/18386#issuecomment-1541043535, it's intentional not to render multiline link if the link text contains emoji.

Agreed, this is not a bug


fix: remove :hover to -hover

:tada: This PR is included in version 1.0.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error table download failures

In GitLab by @wtbarnes on Dec 9, 2022, 16:05
Since switching over to the alternate SSW mirror in !167, I've noticed that the downloads for the V3 error table files are failing. I don't understand why as the URL resolves just fine: https://sohoftp.nascom.nasa.gov/solarsoft/sdo/aia/response/aia_V3_error_table.txt
However, when trying to download an error table using aiapy.calibrate.util.get_error_table, I get the following exception,
WARNING: SunpyUserWarning: [Errno 2] No such file or directory: '/Users/wtbarnes/sunpy/data_manager/aiapy.aia_V3_error_table.txt' [sunpy.data.data_manager.cache]

---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Input In [2], in <cell line: 1>()
----> 1 error_table = aiapy.calibrate.util.get_error_table()

File ~/mambaforge/envs/mocksipipeline/lib/python3.9/site-packages/aiapy/calibrate/util.py:216, in get_error_table(error_table)
    214 def get_error_table(error_table=None):
    215     if error_table is None:
--> 216         error_table = fetch_error_table()
    217     if isinstance(error_table, (str, pathlib.Path)):
    218         table = astropy.io.ascii.read(error_table)

File ~/mambaforge/envs/mocksipipeline/lib/python3.9/site-packages/sunpy/data/data_manager/manager.py:85, in DataManager.require.<locals>.decorator.<locals>.wrapper(*args, **kwargs)
     80 if self._cache_has_file(urls):
     81     # If we can't find a file matching sha_hash, but the url is already
     82     # in the database
     83     raise ValueError(f"{urls} has already been downloaded, but no file "
     84                      f"matching the hash {sha_hash} can be found.")
---> 85 file_path = self._cache.download(urls, self._namespace)
     86 file_hash = hash_file(file_path)
     87 if file_hash != sha_hash:
     88     # the hash of the file downloaded does not match provided hash
     89     # this means the file has changed on the server.
     90     # the function should be updated to use the new
     91     # hash. Raise an error to notify.

File ~/mambaforge/envs/mocksipipeline/lib/python3.9/site-packages/sunpy/data/data_manager/cache.py:84, in Cache.download(self, urls, namespace, redownload)
     81     else:
     82         return Path(details['file_path'])
---> 84 file_path, file_hash, url = self._download_and_hash(urls, namespace)
     86 self._storage.store({
     87     'file_hash': file_hash,
     88     'file_path': str(file_path),
     89     'url': url,
     90     'time': datetime.now().isoformat(),
     91 })
     92 return file_path

File ~/mambaforge/envs/mocksipipeline/lib/python3.9/site-packages/sunpy/data/data_manager/cache.py:164, in Cache._download_and_hash(self, urls, namespace)
    162         errors.append(f"{e}")
    163 else:
--> 164     raise RuntimeError(errors)

RuntimeError: ["[Errno 2] No such file or directory: '/Users/wtbarnes/sunpy/data_manager/aiapy.aia_V3_error_table.txt'"]

The download fails and then the data manager cannot find the file (the confusing part of how this error message is handled is probably more of an upstream sunpy issue).
The error table test is also failing locally for me:
__________________________________________________________________________ test_error_table[None] ___________________________________________________________________________

self = <sunpy.data.data_manager.cache.Cache object at 0x110dfd300>, urls = ['https://sohoftp.nascom.nasa.gov/solarsoft/sdo/aia/response/aia_V3_error_table.txt']
namespace = 'aiapy.'

    def _download_and_hash(self, urls, namespace=''):
        """
        Downloads the file and returns the path, hash and url it used to download.

        Parameters
        ----------
        urls : `list`
            List of urls.

        Parameters
        ----------
        path : `str`
            The path of the file to be hashed.

        Returns
        -------
        `str`
            SHA-256 hash of the file.

        Parameters
        ----------
        msg : str
            Warning message.
        stacklevel : int
            This is interpreted relative to the call to this function,
            e.g. ``stacklevel=1`` (the default) sets the stack level in the
            code that calls this function.
        """
>       warnings.warn(msg, SunpyUserWarning, stacklevel + 1)
E       sunpy.util.exceptions.SunpyUserWarning: [Errno 2] No such file or directory: '/Users/wtbarnes/sunpy/data_manager/aiapy.aia_V3_error_table.txt'

../../../mambaforge/envs/aiapy-dev/lib/python3.10/site-packages/sunpy/util/exceptions.py:89: SunpyUserWarning

which seems to be the same failure that is causing the conda-forge build failures: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=619803&view=logs&j=656edd35-690f-5c53-9ba3-09c10d0bea97&t=e5c8ab1d-8ff9-5cae-b332-e15ae582ed2d&l=631

In GitLab by @wtbarnes on Jan 10, 2023, 11:47
The reason this is failing is because the file on this SSW mirror has Content-encoding: gzip which cannot be decoded when downloading using parfive. This error is not propagated through when using the download manager, but can be seen when using parfive directly,
import parfive
dl = parfive.Downloader()
dl.enqueue_file('https://sohoftp.nascom.nasa.gov/solarsoft/sdo/aia/response/aia_V3_error_table.txt', path='.')
foo = dl.download()
print(foo.errors)

yields
1/0 files failed to download. Please check `.errors` for details
Files Downloaded:   0%|                                                                                | 0/1 [00:00<?, ?file/s]
[<parfive.results.Error object at 0x1072eb540>
https://sohoftp.nascom.nasa.gov/solarsoft/sdo/aia/response/aia_V3_error_table.txt,
400, message='Can not decode content-encoding: gzip']

It is not clear whether this is a problem with this file or a bug in parfive, but this was not an issue with the HESPERIA SSW mirror we were previously using.
See also this upstream issue on parfive: https://github.com/Cadair/parfive/issues/121

We have patched this and opened another issue to undo the fix when parfive is fixed.


[BEAM-433] Remove Dataflow references in ExampleOptions

not ready for review yet
It depends on other PRs to get in first.

obsolete


Tooltips are often unhelpful because of size or placement

I find that the tooltips are often too big or rendered in the wrong direction, causing the most important information (the sizes) to be unavailable. Some examples:
Hovering over an entry at the top of the screen causes the tooltip to be rendered upwards for some reason:

Long lists of files are not truncated, causing the size information to be unavailable:

vite-bundle-visualizer is a wrapper of  rollup-plugin-visualizer, maybe you should issue at its repo, and it's better to help fix the problem if you provide a sample project.

Sure, I'll file a bug over there.


Malloc bug with fetchAll and ssh.

Hi,
I have a bug with fetchAll action.
I make this simple code for testing this bugs:
'use strict';
var path = require('path');
var Git = require('nodegit');

var fetchOpts = {
  callbacks: {
    certificateCheck: function() { return 1; },
    credentials: function(url, userName) {
      return Git.Cred.sshKeyNew('git', path.resolve('/Users/holblin/.ssh/id_rsa.pub'), path.resolve('/Users/holblin/.ssh/id_rsa'), '');
    }
  }
};

Promise.resolve()
.then(function() {
  var all =[];
  for (var i = 0; i < 2; i++) {
    all.push(updateIt('repo_'+i));
  }
  return Promise.all(all);
})

function updateIt(i) {
  var repository, localHeadCommit;
  var dir = 'tests/'+i;

  return Git.Repository.open(dir)
    .then((repo) => repository = repo) // save the reference for futur use
    .then(() => console.log('['+i+'] openRepo'))
    .then(() => repository.fetchAll(fetchOpts)) // fetch all data from all remotes
    .then(() => console.log('['+i+'] fetchAll'))
}

For use it, make this commands first:
npm -i path nodegit
mkdir tests
cd tests
git clone<EMAIL_ADDRESS>repo_0
git clone<EMAIL_ADDRESS>repo_1
cd ..

I have a Mac os, X yosemite version 10.10.5. I use nvm with node v5.5.0.
My uname -a is Darwin zolesio 14.5.0 Darwin Kernel Version 14.5.0: Tue Sep  1 21:23:09 PDT 2015; root:xnu-2782.50.1~1/RELEASE_X86_64 x86_64.
The nodegit version is<EMAIL_ADDRESS>Sometimes, it finished with:
[repo_0] openRepo
[repo_1] openRepo
node(5166,0x1026cd000) malloc: *** error for object 0x102303d00: pointer being realloc'd was not allocated
*** set a breakpoint in malloc_error_break to debug
Abort trap: 6

Sometimes with:
[repo_0] openRepo
[repo_1] openRepo
node(5058,0x105087000) malloc: *** error for object 0x10170e960: double free
*** set a breakpoint in malloc_error_break to debug
Abort trap: 6

Sometimes with:
[repo_0] openRepo
[repo_1] openRepo
Bus error: 10

Sometimes with getting stuck and use 200% of my CPU.

You'll need to create a new fetchOpts for each call. Internally in C++ those opts will be freed after the call is completed which will result in a segfault if they are used in other parts of the app.

After try I still have similar issues.
My new code is:
'use strict';
var path = require('path');
var Git = require('nodegit');

function fetchOpts(){
  return {
    callbacks: {
      certificateCheck: function() { return 1; },
      credentials: function(url, userName) {
        return Git.Cred.sshKeyNew('git', path.resolve('/Users/holblin/.ssh/id_rsa.pub'), path.resolve('/Users/holblin/.ssh/id_rsa'), '');
      }
    }
  };
}

Promise.resolve()
.then(function() {
  var all =[];
  for (var i = 0; i < 2; i++) {
    all.push(updateIt('repo_'+i));
  }
  return Promise.all(all);
})

function updateIt(i) {
  var repository, localHeadCommit;
  var dir = 'tests/'+i;

  return Git.Repository.open(dir)
    .then((repo) => repository = repo) // save the reference for futur use
    .then(() => console.log('['+i+'] openRepo'))
    .then(() => repository.fetchAll(fetchOpts())) // fetch all data from all remotes
    .then(() => console.log('['+i+'] fetchAll'))
}

Still have the same results.

After discution with you dirrectly on gitter I use the .enableThreadSafety(); feature.
This reduce a lot the errors but I still have errors (like 1/10 with my experimental testing).
var Git = require('nodegit');
Git.enableThreadSafety();

/cc @srajko

Oh boy.  I experimented with this a bit and it looks like our locking strategy might be insufficient - the bulk of what we were going for was making sure that no two libgit2 calls run concurrently on the same repository.  But this example demonstrates that concurrent fetches on separate repositories are also susceptible to crashing.
As a test, I modified the thread safety code to completely prevent concurrent access to libgit2, and that does seem to prevent the crashes.  So we may need to rethink our thread safety strategy.
Thanks for submitting this @holblin!

@holblin can you try to test this with 0.11.3?

@johnhaley81 Yes it works !
I also remove Git.enableThreadSafety(); from my code and it's works too !
Thanks to the team for big reactivity and take care of my case ;-)


(Question) How to set up desiredCapabilities for our browser?

What the title says.
I basically got let browserCap = OpenQA.Selenium.Chrome.<what here?> (where the browser doesn't need to be Chrome, of course), but when I put that last dot, IntelliSense does not give me DesiredCapabilities.
If I do let bC = OpenQA.Selenium.Remote... the IntelliSense only gives me ReadOnlyDesiredCapabilities (and besides, I'm running tests on the local machine, not remote).
Specifically, how do I add the capabilities javascriptEnabled and acceptSslCerts?
Thanks in advance.

I have never done this, but I think I found how.  I dont have a F# project handy, so here is the answer in C#.
            var options = new ChromeOptions();
            options.AddAdditionalCapability(CapabilityType.AcceptSslCertificates, true, true);
            options.AddAdditionalCapability(CapabilityType.IsJavaScriptEnabled, true, true);

Then you can start chromeWithOptions like I linked to in your other ticket.

Alright.
I'll verify it in the next day or so and give an answer.
Thanks!
(Keeping open until I verify).

This are a simple case of adding ChromeOptions the usual way. Easy-peasy.
Thanks.


iOS deployment target override

On the .ios folder you can change the deployment target to a higher version of iOS by editing the Podile and Runner.xcworkspace. However, each time you run flutter pub get these changes will disappear and you will once again be set to iOS 8.
Is there a way to change this behavior? For most apps it doesn't make sense supporting such an old version.
Before running flutter pub get:

After running flutter pub get:

You should not be using the module project unless you are doing add2app - the .ios folder is only created to bootstrap a minimal application for local development. If you want to edit source files, the best approach would be to create a regular project instead.


Tests: Bluetooth: Mesh: Add BLOB Server persistent storage BSim tests

A bit of RFC: in commit https://github.com/zephyrproject-rtos/zephyr/pull/56653/commits/b45405aedfe9e2e70e3dbc39bb298dff3d6ca6e0 I'm abandoning transfer that entered BT_MESH_BLOB_XFER_PHASE_WAITING_FOR_START phase, but XFER_START message wasn't handled yet. srv->state.xfer.size == 0 is used to verify if XFER_START was handled, as it should be set to some value if it did. Maybe we can just enter here BT_MESH_BLOB_XFER_PHASE_WAITING_FOR_START and erase nothing, but I'm not sure. So alternative is:
	if (srv->phase == BT_MESH_BLOB_XFER_PHASE_WAITING_FOR_START) {
		LOG_DBG("Transfer from 0x%04x (%llu) hasn't started yet", srv->state.cli,
			srv->state.xfer.id);
		return 0;
	} else {
		phase_set(srv, BT_MESH_BLOB_XFER_PHASE_SUSPENDED);
	}

These tests are passing locally but failing in CI, I'm converting to draft until I resolve this

Hi @KKopyscinski, I really do not like the idea to add blob tests to the persistence test suite if they do not check anything related to the settings subsystem.
Please create two test lists with and without persistence (how we've already done for provisioning tests. see sample).

Hi @KKopyscinski, I really do not like the idea to add blob tests to the persistence test suite if they do not check anything related to the settings subsystem. Please create two test lists with and without persistence (how we've already done for provisioning tests. see sample).

When I resolve issue with passing this on CI (I think I got race somewhere) I'll split these from regular BLOB Cli tests

@PavelVPV @alxelax this is ready for review now. New tests are run with pst overlay and old ones run with default.

Seems like something broke in twister, but doesn't look like my code

LGTM, when CI passes, I'll approve it.

Seems like something broke in twister, but doesn't look like my code

Will be fixed in https://github.com/zephyrproject-rtos/zephyr/pull/56813. Please rebase once it is merged.

Seems like something broke in twister, but doesn't look like my code

Will be fixed in #56813. Please rebase once it is merged.

Rebased, but seems like something more is still broken

@alxelax @PavelVPV Seems like CI is passing


Ignore 'Value' as a manf# entry

In my version of KiCAD custom fields are by default assigned the string 'Value' whenever the edit window is opened for that part. For other parts this stays blank and kicost aggregation works fine. Is it possible to consider the string 'Value' as equivalent to empty for aggregation purposes.

Which version of KiCad are you using?

I use nightly builds. However just an option to ignore a specific non-empty
string would be good.
On Aug 26, 2017 22:56, "Hildo Guillardi Júnior"<EMAIL_ADDRESS>wrote:

Which version of KiCad are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xesscorp/KiCost/issues/66#issuecomment-325127340, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAd_hJAMzrXdJaH73CwAGjqZBV6hPdnIks5scBzwgaJpZM4NSzbl
.

May you attach the KiCad schematic (or xml file) example if this issue? To check.


Update to process tweet.js file

Now twitter doesn't give you a tweets.csv file, instead of it, they give you a tweet.js file with a few changes. I have updated the script to work with the new format.

Looks good. Let's get it merged.


Add user-transpiled circuits tutorial

Summary
Add short tutorial showing users how to submit user-transpiled circuits via sampler primitive.

Should the first histogram have any labels (axes)?

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 increased (+0.1%) to 69.207%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
2052

Relevant Lines:
2965

💛  - Coveralls

Seems the histograms aren't rendering correctly through the github image viewer, I think because my format is svg, i'll just switch it to png as it's easier.


lnpeer+brontide: reduce memory footprint using read/write pools for message encode/decode

This PR is a follow up to #2419 and #2385, introducing an lnpeer.WritePool and brontide.ReadPool.  Requests to read/write messages are submitted to the respective pool, which maintains a set of workers that handle the encoding/decoding. This builds on the prior PRs, since each pool is backed by a lnpeer.WriteBufferPool and brontide.ReadBufferPool, ensuring that buffer used in each are efficiently reused before being returned to the runtime.
With #2419 and #2385, the total memory that will be allocated is linear in the number of peers/connections maintained by the daemon. With the read and write pools, we modify that to be at most linear in the number of specified workers. More importantly, these changes completely decouple read and write buffer allocations from the peer/connection lifecycle, allowing LND to tolerate flapping peers with minimal overhead.
Nodes that have a large number of peers will see the most drastic benefit. In testing, I was able to connect (w/o gossip queries) to nearly 900 unique nodes, all while keeping LNDs total memory under 15 MB. This configuration could have easily connected to more nodes, though that was all that reachable via the bootstrapper.
This same test used between 90-100MB on master, and continues to grow as more connections are established. In contrast, the memory used with read/write pools remains constant even as more peers are added.
Note that of the 15MB, only about 1MB is actually from the read/write pools, the remaining overhead is created by other subsystems operating in LND, and is reflected in both measurements. Coupled with more efficient recycling of unused buffers, I'd expect the reduction in memory related to read/write buffers to be one or two orders of magnitude for most nodes.
Depends on:

[x] #2385
[x] #2419

All the dependent PRs have been merged now, so this needs a rebase!

@Roasbeef pool.Read and pool.Write now reside in the pool package, lmk how you like the new structure. some lingering godocs left to finish in the new unit tests, but o/w should be ready to another review, ptal

@Roasbeef i've added some commits that implement a PoC split ReadNextHeader/ReadNextBody, while still retaining the unified ReadMessage method for non-adversarial cases. I must say, I much prefer this approach as it pushes all timeout handling into the peer and keeps the brontide package much simpler :)

After experiencing https://github.com/lightningnetwork/lnd/issues/2726 I have changed writeMessageTimeout to 25 on my node and it appears to have helped.
@cfromknecht was there a reason for change from 50 to 10?


WIP-Worker threading (Don't Merge)

Ok, so, you violated the notion of having a pull request implement one thing, and mashed a bunch together.


Update PendingEveryNDaysRepeat.php

The endsAfter method is indirectly updating the $this->start_at value due to object reference instead of creating its own CarbonInterface object.

Hi.
This is not true. There is toImmutable() called on start_at so any modifications return new object. This PR is redundant.


separate button colors into different classes

Because colored buttons are all subclasses of the .button class, they can't be applied separately when imported using @import (reference). The button.less file should have a top-level .button style and .blue style, etc.

So move them below the .button { } class and remove the &?
For example the first class below .button{ } would be
.blue {     background-color: hsl(var(--blue-hue), 100%, var(--color-lightness));     color:            white;   }

That's what I was thinking, yeah, but I discovered over the weekend that I was wrong about not being able to apply them. You can do it with this:
button {
  .button;
  .button.blue;
}

(This might accomplish the same thing - not sure:)
button {
  .button.blue;
}

I think either of those is actually fine. The alternative, if we implement this issue, would be this:
button {
  .button;
  .blue;
}

In this case, there's a slightly higher risk of accidentally applying the .blue class to things that aren't buttons. So I'm thinking we should leave this as is and close this issue.


go-nats-streaming : This download does NOT match the one reported by the checksum server.

Recently followed your bolg
I like it so much
I read your blog and followed the steps
cd publisher
go get github.com/nats-io/go-nats-streaming

The following problems have occurred：
sun@terminal publisher % go get github.com/nats-io/go-nats-streaming

go: downloading github.com/nats-io/go-nats-streaming v0.4.4
go: downloading github.com/nats-io/nkeys v0.1.2
github.com/nats-io/go-nats-streaming imports
	github.com/nats-io/go-nats imports
	github.com/nats-io/nkeys<EMAIL_ADDRESS>verifying module: checksum mismatch
	downloaded: h1:6JJydNSjt/lqhfM+ngyaLk9Wa/d3UW/YyEqj8OrDV4w=
	sum.golang.org: h1:kWUa9fUdAOW426qaAD1V4+suF4On2eTr2n/FE/XSGpg=

SECURITY ERROR
This download does NOT match the one reported by the checksum server.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

I tried the following methods to solve it
go clean -modcache
rm go.sum
go mod tidy

It's no use
So is there any way
thank you

Hey @SunA0! It seems this is because watermill-nats uses github.com/nats-io/nats-streaming-server, which is now deprecated.
I think the correct steps would be:
go get github.com/ThreeDotsLabs/watermill
go get github.com/ThreeDotsLabs/watermill-nats
go get github.com/nats-io/stan.go

stan is the new package and these should be enough to run the examples. I'll review it and update the post, but you can try it anyway. :)

I adopted the above suggestions and executed the following additional commands
go get github.com/nats-io/nkeys

Finally succeeded
thanks :)


feat: add KongServiceFacade RBACs, release charts

What this PR does / why we need it:
Adds KongServiceFacade RBAC rules and releases both charts (changed the ingress's Chart.yaml to refer to kong by file:// instead of https:// which allows bumping both charts in a single commit).
Which issue this PR fixes
Part of https://github.com/Kong/kubernetes-ingress-controller/issues/5152.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] PR is based off the current tip of the main branch.
[x] Changes are documented under the "Unreleased" header in CHANGELOG.md
[x] New or modified sections of values.yaml are documented in the README.md
[x] Commits follow the Kong commit message guidelines

@pmalek I changed the kong/kong repository from https:// to file:// deliberately to allow bumping both charts in a single commit. Is this something that we'd like to avoid? As I understand, we always want to make them be in sync - that enables us to make it happen without an additional PR to bump kong/ingress dependencies separately.


Questionnaires added to flat menu

Closes #1363

Previously questionnaires added to flat menu were not showing on web page.
This problem is fixed by setting the default value of show_in_menu to True for all questionnaires.

write management command to update show in menu for existing pages


Python 3.11 quotechar must be 1-character string error

Changed quotechar from "" to "'" in config.py and datafile.py
This should fix the "quotechar" must be a 1-character string error as based on issue discussing it in Python 3.11+ as it changes the quotechar to a 1-character string, as requested. This shouldn't cause other issues as based on the answer to https://stackoverflow.com/questions/16833209/typeerror-quotechar-must-be-an-1-character-string
"quoting controls what will be quoted when (defaulting to QUOTE_MINIMAL, i.e., only when quotes are absolutely necessary). If you turn off quoting entirely (QUOTE_NONE), the value of quotechar is of course meaningless.",
the quoting is set to QUOTE_NONE so the quotechar is ignored when its used in the datafile.py, except for the whatever check is raising this error.
This fixes the issue on my machine with python 3.11.1. I did not test it under any other circumstances. I'm new at this so, if I missed something please let me know.

@Rozinig  thanks for your PR.  Your bug fix has been improved by @Fried-Squid in the PR #13 that I just merged. Thanks to both of you, the next release of tatoebatools should support python 3.8 to python 3.11.


Undefined behavior with -emit-spirv-directly when using custom intersection shaders

I've been evaluating slang's new direct spirv backend on mixed entry point setups, and so far it's going well. I did hit this one bug though with custom intersection programs that I think we should address.
If I write the following code:
[shader("intersection")]
void AABBIntersection(uniform AABBGeomData record) {
  ReportHit(0.1f, /*hitKind*/ 0, /*attribute*/ float3(1.f, 1.f, 1.f));
}

[shader("closesthit")]
void AABBClosestHit(uniform AABBGeomData record, inout Payload payload, in float3 color) {
  payload.color = color;
}

Then I see this correct image:

But if i reverse the order of these entry points:
[shader("closesthit")]
void AABBClosestHit(uniform AABBGeomData record, inout Payload payload, in float3 color) {
  payload.color = color;
}

[shader("intersection")]
void AABBIntersection(uniform AABBGeomData record) {
  ReportHit(0.1f, /*hitKind*/ 0, /*attribute*/ float3(1.f, 1.f, 1.f));
}

Then the hit attributes received by the closest hit program appear to be corrupt. Here I just get a black cube:

But in other cases, I get total undefined behavior:
Intersect, then closest hit:

Versus closest hit, then intersect

deviceCode.zip
For reference, I've zipped the slang file that I'm using, as well as the correct and buggy spirv generated by slang

Also sent Yong a DM over slack, but for completeness, I have a very simple minimal reproducer here: https://gitlab-master.nvidia.com/nmorrical/intersect-bug-repro
Should be able to use this to recreate the bug on different systems.

@pmistryNV was able to fix the bug. Behavior should be correct with a future driver release.


configure is called with undefined

when i try to do the most simple aurelia setup i can manage:
jspm init
jspm install aurelia-bootstrapper

then add index.html, with a body like this:
<body aurelia-app>
    <script src="jspm_packages/system.js"></script>
    <script src="config.js"></script>
    <script>
      System.import('aurelia-bootstrapper');
    </script>
  </body>

i also have app.js with copy-pasta from the docs:
import {LogManager} from 'aurelia-framework';
import {ConsoleAppender} from 'aurelia-logging-console';

LogManager.addAppender(new ConsoleAppender());
LogManager.setLevel(LogManager.logLevel.debug);

export function configure(aurelia) {
  // the variable 'aurelia' is undefined here
  aurelia.use
    .defaultBindingLanguage()
    .defaultResources()
    .history()
    .router()
    .eventAggregator()
  aurelia.start().then(a => a.setRoot('app', document.body));
}

the app.html is just a pair of template tags
however, the thing crashes with TypeError: Cannot read property 'use' of undefined, which is annotated above. for some reason the configure function in app.js is being called with undefined ... !?

try renaming app.js to main.js.  The bootstrapper is looking for main.js, not app.js.

GET http://localhost:8001/app.js 404 (File not found)
es6.promise.js:139 Unhandled promise rejection Error loading "app" at http://localhost:8001/app.js
File not found: http://localhost:8001/app.js

I don't think that's the case.

Trust me, I tried it and it works :)  Once you have main.js in place, then you need app.js which usually has the router, and app.html with "Hello World"
Then fire up and it should run.  If it doesn't, I will be a skeleton on GitHub, but I am going out for the evening at the moment.

Ok, I missed that I need both. Perhaps it was the instruction to rename that threw me.
So the bootstrapper wants main.js where we do configuration of the Aurelia instance, which by default expects app.js as the "entry point"?
Have a nice evening :dancer:


feat: improve user experience when preview fails

:tada: This PR is included in version 4.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Nullable BigInteger field causes NPE

The v3.16.0 code at org.simpleflatmapper.jdbc.impl.getter.BigIntegerResultSetGetter does not support NULL values for a field of type java.math.BigInteger. I think its behavior should be amended like in most other non-primitive type getters:
From
	public BigInteger get(final ResultSet target) throws Exception {
		return new BigInteger(target.getString(column));
	}

To
	public BigInteger get(final ResultSet target) throws Exception {
                final String value = target.getString(column);
                return target.wasNull() ? null : new BigInteger(value);
       }

thanks will have a look should be able to push out new version with the fix tonigh

	public BigInteger get(final ResultSet target) throws Exception {
		String val = target.getString(column);
		if (val == null) return null;
		return new BigInteger(val);
	}

should do as String is null. the reason it goes through string is there no way to get a big integer from the resultset directly. I wonder if passing by a BigDecimal might be better?

jooq seems to be going through big decimal

just pushed 3.16.2 to maven central should be there in 20-30 minutes.

I switched to 3.16.2 and the fix works well.
Thank you!
From: Arnaud Roger<EMAIL_ADDRESS>Sent: Wednesday, March 28, 2018 11:06 PM
To: arnaudroger/SimpleFlatMapper<EMAIL_ADDRESS>Cc: Zohar Shay<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [arnaudroger/SimpleFlatMapper] Nullable BigInteger field
causes NPE (#498)
just pushed 3.16.2 to maven central should be there in 20-30 minutes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arnaudroger/SimpleFlatMapper/issues/498#issuecomment-377018453,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AO5C78zb_NCHQC7YlTj4sBREaQdo6449ks5ti-04gaJpZM4S-XeU
.[image: Image removed by sender.]


KafkaEmbedded without @Rule/after() can cause build failures with System.exit

Following the example of using @EmbeddedKafka with KafkaEmbedded in the documentation worked fine in many situations, but found that when executing on build servers as part of a gradle build my tests would succeed, but the overall build would fail with "Process 'Gradle Test Executor 1' finished with non-zero exit value 1".
This was due to improper shutdown of Zookeeper or Kafka originating from the embedded Kafka.    I wasn't able to trace down exactly where it was happening, but I determined that if properly shutdown KafkaEmbedded.after() this would not occur.    And this would would also get called if the JUnit Rule were used, which wasn't mentioned in the example from the docs and should look like this.
@Autowired  @Rule  public KafkaEmbedded kafkaEmbedded;
I don't know if anything more should be done about it, but I would propose to update the example in the documentation, it was rather difficult to isolate the cause.

The @DirtiesContext should help you.
The @EmbeddedKafka add KafkaEmbedded bean into the applicationContext, but without @DirtiesContext that applicationContext is cached after the test class has finished and, therefore KafkaEmbedded.after() isn't called in time.
Right, the @Rule does that independently of the Spring Test Framework, that's why we don't see issue there.
Your @Autowired @Rule isn't good because it is a mix of concerns. In this case you eliminate the result of the @EmbeddedKafka and just get a @Rule in the end.
Now quoting the Reference Manual:

Starting with version 2.0, if you are using Spring's test application context caching, you can also declare a KafkaEmbedded bean, so a single broker can be used across multiple test classes.

So, you may consider do not use @EmbeddedKafka, but just @Rule for your use-case.

Ok, I guess I didn't realize this was the purpose '@DirtiesContext' in the example.    The '@ClassRule' usage shown later would probably accomplish what I need, but then what is the reason you would use the '@EmbeddedKafka' annotation instead?

One more time:

so a single broker can be used across multiple test classes.

Another sample is: you can use @Rule/@ClassRule for any tests, but sometime when we talk about Spring configurations we might need EmbeddedKafka as a bean to leverage dependency injection.
But right, mostly it is for Spring Test Framework caching capabilities.

Yeah I read that, but I guess it's really not clear how it enables that so I saw some opportunity to help improve the docs/example.   @DirtiesContext will close out the context and then the other Test will get a fresh KafkaEmbedded instance.      But without @DirtiesContext, it may not properly shut things down, so there must be a different way to structure the tests to enable both (sharing and proper shutdown).    But I'm fine to let this go as the @ClassRule does what I need.

public @interface DirtiesContext
Test annotation which indicates that the ApplicationContext associated with a test is dirty and should therefore be closed and removed from the context cache.

If someone stumps into this problem, I've found a workaround:
https://github.com/gradle/gradle/issues/11195#issuecomment-619501232

Thanks @pwamej - if'd you'd like to issue a pull request, please do.
Otherwise, I'll address this tomorrow.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_b57df916-1317-4a3b-a772-eab66834193b/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_b57df916-1317-4a3b-a772-eab66834193b.git

Docs Build status updates of commit d87bdcd:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-02-10-05-08-32 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-02-10-05-08-32

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


gtk3: 3.24.23 -> 3.24.24

Motivation for this change
Remove an old patch. Includes support for upstream primary selection
protocol, fixing primary selection for sway on the master branch.
Things done

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

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

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

This also fixes copying into clipboard links via context menu and from address bar, that went broken with Firefox 83.0 on sway. https://bugzilla.mozilla.org/show_bug.cgi?id=1631061#c31


License?

I can't find any indication of what licensing this is provided under.  I do hope I've not simply missed it somehow.  Without license, from my understanding, I am to assume I do not have permission to use it.  Can someone point me in the right direction, or clarify the license of this project?

I just haven't added a license. :smile: I'll do that later today. I'll put it under a MIT license so the intention is free of course.
I publish my code with the understanding that if it lacks license it's completely free.
Cheers

Having looked at Githubs disclaimer for license less projects your understanding was correct actually.
I've added a license for the code now any way.


Fixes QsCompiler build scripts in macos, linux

Hi all
A couple of minor changes I had to put it to make the project build on mac os x.
I realize the change as it is probably needs to be changed so it works for both win and posix type os. Happy to make those changes if you guys think is worth it

 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.:x: eginez sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@eginez Sorry for the Mac compatibility issue & thanks for addressing it!
It actually looks like windows is fine with teh '/' slashes in the project files, so I'd suggest to not make a destinction by platform and just go with '/' instead of '' everywhere in those two project files.

Sure thing I'll collapse it all  to forward slash


serve static files with nginx in prep for github pages replacement

Interesting choice.
Personally, I just serve static assets from S3 for my blogs and websites (currently behind CloudFlare for HTTPS). But then I'm also using Hugo instead. https://github.com/nathany/hugo-deploy

For sure. In most scenarios I would put this on S3 but I'm trying to eat a little bit of dogfood here, too. The end result will be fronted with CloudFront so the difference should ideally be negligible.

Yah, dogfooding makes sense.


[Windows] Fix windows is_path_invalid, and apply it to directory creation.

Fixes https://github.com/godotengine/godot/issues/88120 and applies validity checks to the directory creation.

Does this also fix #38198 ? Or can it be fixed alongside this with further changes?

Does this also fix https://github.com/godotengine/godot/issues/38198 ?

No, it is about platform independent String method, I'm not sure if it should account for Windows specific reserved names.

Makes sense. It can be worth reviewing the issues closed as duplicates of #38198, I think some might actually be solved by this.

Thanks!

Cherry-picked for 4.2.2.


[HELP] How to specify API version for k3s cluster?

How can I launch a k3s cluster with an earlier version of the API? Right now it's pretty easy to launch one on 1.14, but if I want to deploy for example a 1.11 cluster, what's the right way to do it?
It looks like the --version flag is for the k3s image, not necessarily for the API version, and I see that the --server-arg flag is also available for passing flags to the k3s server, but I can't find any documentation on how to specify the API version.
Is this possible, or am I missing something?

Hi there, thanks for opening this issue.
Probably this is more of a question for https://github.com/rancher/k3s, since all the Kubernetes part is coming from there. I think you cannot deploy a different version of Kubernetes, than the one provided by the k3s image you choose (maybe @ibuildthecloud or @erikwilson can tell you more on this). What you can definitely do though is
a) building and running your own k3s image
b) setting apiVersions for specific objects by passing api-server arguments (see https://kubernetes.io/docs/reference/command-line-tools-reference/kube-apiserver/#kube-apiserver) to k3s' --kube-apiserver-arg (see https://github.com/rancher/k3s#customizing-components) which you then pass to k3d's --server-arg

Ah that sounds great, thank you so much for the answer @iwilltry42!
I'll post the question on the k3s repo, but if you don't mind I'll leave this issue open for a few days in case anybody can/want to add any comments.
Thanks!

Linking your issue from k3s here: https://github.com/rancher/k3s/issues/648
Answer from @erikwilson:

At the moment the version of k8s is compiled into k3s, where we are vendoring in a modified version of k8s in vendor.conf. If you wanted to use an earlier version of k8s it would require a custom compilation, but some of the features of k3s may depend on certain flags being support in k8s. Additionally it is not possible to swap out an upstream version of k8s, it would need to be our modified k3s version.

Anything left here @patoarvizu or can we close this issue?

Ah sorry I missed your first message @iwilltry42. Yes, it can be closed.

In case someone else is looking for this, use the --image flag:
k3d cluster create local \
   --agents 3 \
   --servers 3 \
   --image "rancher/k3s:v1.20.4-k3s1" \
   -p "8082:80@loadbalancer"

See available image tags on Docker Hub.
Source: https://k3d.io/v5.5.1/usage/advanced/cuda/


chore: qd: konflux-qe-admins perms for pods

we want to be able to restart pods for QD and ci-helper-app on staging cluster in cases when for example secrets get updated in vault

/retest-required


Return activities with then option to exclude archived activities

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (8809d9d) 85.30% compared to head (e7a6276) 85.31%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #382   +/-   ##
=========================================
  Coverage     85.30%   85.31%
- Complexity     1909     1911    +2
=========================================
  Files           262      262
  Lines          7317     7318    +1
  Branches        380      380
=========================================
+ Hits           6242     6243    +1
  Misses          878      878
  Partials        197      197

Impacted Files
Coverage Δ

...ps/hmppsactivitiesmanagementapi/entity/Activity.kt
93.79% <100.00%> (+0.04%)
:arrow_up:

...tivitiesmanagementapi/resource/PrisonController.kt
87.50% <100.00%> (ø)

...activitiesmanagementapi/service/ActivityService.kt
83.15% <100.00%> (ø)

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


Scan binary files

Description:
This is a rough implementation that fixes #1541.
General feedback on approach and potential optimizations are welcome.
TODO:

[ ] Check mimetype?
[ ] Handle bare repositories
[ ] Decide whether it's better to copy files to tmp -> read mimetype -> read (scan), versus copy directly to a buffer.
[x] Add handling for non-git
[x] Add a limit for file size
[ ] Measure performance
[ ] Add test cases

Anything else?
Checklist:

[ ] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[ ] Have you lint your code locally prior to submission?

Testing is failing due to an abundance of "file not found" errors. This appears to be caused by #1333.
6:04PM ERR failed to scan binary file error="error getting file in tree: file not found" commit=bc26fcd27702806e96991764482078b3f0905f44 path=scripts/.scala/project_940fb43dce-2f45572b79/classes/main/inc$minus73.tasty


Prevent flickering due to scrolling when terminal is resized and cursor is off-screen

I've opened an issue about this before, but I still don't understand how to solve this problem (if it's solvable at all). It seems to me that, if a resize happens and the cursor gets pushed off-screen, then that will always trigger a scroll, which the application can detect and scroll the viewport back up and redraw the screen. The problem is this will introduce flickering as the application states before and after the resize will be separated by a third intermediary state in which the viewport was scrolled down. So far I see no way to have smooth, non-layout-breaking resizing if resizing can cause the viewport to scroll. So I have a couple questions:

Is there a way to prevent this flickering from happening?
If it is impossible to prevent auto scrolling when cursor is pushed off-screen, would always having the cursor at the next to last line and maybe polling frequently enough mitigate this issue?
Couldn't an application just opt-out of this behavior. Maybe there could be a console mode that you could set to always have the cursor within the screen if the window is resized that would work only when an application switches to the alternate screen buffer.

Could you link the previous discussion? I'm having a hard time finding it in the backlog.
IMO, if you're creating a commandline application that's drawing to the entire viewport (like vim, emacs, etc), then the right way to do this is with the alternate screen buffer. This will create a second virtual buffer that you can use to draw to the entire screen, which doesn't have any sort of scrollback to deal with.
See The Alternate Screen Buffer for more details

Previous issue was #4181. By the way, I am using the alternate screen buffer and I still get this problem.

Wait lemme just clarify something, are you using the alt buffer in conhost or in the Windows Terminal? The Terminal unfortunately doesn't support the alt buffer quite yet (#381), so that'll definitely cause problems, but I'd think that the alt buffer in conhost would be fine...

Yes, this also happens in conhost.

I'll test it in conhost and get back to you.

Okay, I can confirm the same problem occurs while using the alt buffer in conhost.

Thanks for the confirmation :smile:


Add j2 mapping for  TokenPersistence configurations

Description
Currently, there is no J2 mapping for the following configuration in the identity.xml file
<!-- Configs related to OAuth2 token persistence -->
        <TokenPersistence>
            <Enable>true</Enable>
            <PoolSize>{{oauth.token_generation.pool_size}}</PoolSize>
            <RetryCount>{{oauth.token_generation.retry_count_on_persistence_failures}}</RetryCount>
        </TokenPersistence>

Therefore, It is required to add a j2 mapping for  TokenPersistence configurations.
Steps to Reproduce
N/A
Affected Component
APIM
Version
3.2.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Documentation has been improved https://github.com/wso2/docs-apim/pull/7977


[WIP] add MaskRCNN

~Merge after #778~
TODOs

[ ] Investigate if it is possible to remove cv2 dependency https://github.com/chainer/chainercv/blob/f40211809b45f8221f87104399c4e24ef633a07f/chainercv/links/model/mask_rcnn/mask_head.py#L113

Wierd segfault during training...
 *** Process received signal ***........]  1.42%
 Signal: Segmentation fault (11)........] 17.32%
 Signal code: Address not mapped (1)
 Failing at address: 0x67ee6000s, 21:36:26.539275.
 [ 0] /lib/x86_64-linux-gnu/libpthread.so.0(+0x11390)[0x7fbb433f5390]
 [ 1] /usr/local/lib/python3.6/site-packages/cv2.cpython-36m-x86_64-linux-gnu.so(+0x7971a0)[0x7fb9d82c31a0]
 [ 2] /usr/local/lib/python3.6/site-packages/cv2.cpython-36m-x86_64-linux-gnu.so(+0x61ce58)[0x7fb9d8148e58]
 [ 3] /usr/local/lib/python3.6/site-packages/cv2.cpython-36m-x86_64-linux-gnu.so(+0x7cf482)[0x7fb9d82fb482]
 [ 4] /usr/local/lib/python3.6/site-packages/cv2.cpython-36m-x86_64-linux-gnu.so(+0x7d0879)[0x7fb9d82fc879]
 [ 5] /usr/local/lib/python3.6/site-packages/cv2.cpython-36m-x86_64-linux-gnu.so(+0x2ef784)[0x7fb9d7e1b784]
 [ 6] /usr/local/lib/libpython3.6m.so.1.0(_PyCFunction_FastCallDict+0x13c)[0x7fbb43733d6c]
 [ 7] /usr/local/lib/libpython3.6m.so.1.0(_PyCFunction_FastCallKeywords+0x4d)[0x7fbb43733bfd]
 [ 8] /usr/local/lib/libpython3.6m.so.1.0(+0x18b9fb)[0x7fbb4378c9fb]
 [ 9] /usr/local/lib/libpython3.6m.so.1.0(_PyEval_EvalFrameDefault+0x1363)[0x7fbb43785a83]
 [10] /usr/local/lib/libpython3.6m.so.1.0(+0x18be0a)[0x7fbb4378ce0a]
 [11] /usr/local/lib/libpython3.6m.so.1.0(+0x18badb)[0x7fbb4378cadb]
 [12] /usr/local/lib/libpython3.6m.so.1.0(_PyEval_EvalFrameDefault+0x34f)[0x7fbb43784a6f]
 [13] /usr/local/lib/libpython3.6m.so.1.0(+0x18c0f3)[0x7fbb4378d0f3]
 [14] /usr/local/lib/libpython3.6m.so.1.0(+0x18badb)[0x7fbb4378cadb]
 [15] /usr/local/lib/libpython3.6m.so.1.0(_PyEval_EvalFrameDefault+0x1363)[0x7fbb43785a83]
 [16] /usr/local/lib/libpython3.6m.so.1.0(+0x18be0a)[0x7fbb4378ce0a]
 [17] /usr/local/lib/libpython3.6m.so.1.0(+0x18badb)[0x7fbb4378cadb]
 [18] /usr/local/lib/libpython3.6m.so.1.0(_PyEval_EvalFrameDefault+0x34f)[0x7fbb43784a6f]
 [19] /usr/local/lib/libpython3.6m.so.1.0(_PyFunction_FastCallDict+0x41b)[0x7fbb4378eacb]
 [20] /usr/local/lib/libpython3.6m.so.1.0(_PyObject_FastCallDict+0x10e)[0x7fbb436fa86e]
 [21] /usr/local/lib/libpython3.6m.so.1.0(_PyObject_Call_Prepend+0x61)[0x7fbb436fb1f1]
 [22] /usr/local/lib/libpython3.6m.so.1.0(PyObject_Call+0x47)[0x7fbb436fac17]
 [23] /usr/local/lib/libpython3.6m.so.1.0(+0x147506)[0x7fbb43748506]
 [24] /usr/local/lib/libpython3.6m.so.1.0(PyObject_Call+0x47)[0x7fbb436fac17]
 [25] /usr/local/lib/libpython3.6m.so.1.0(_PyEval_EvalFrameDefault+0x1c69)[0x7fbb43786389]
 [26] /usr/local/lib/libpython3.6m.so.1.0(_PyFunction_FastCallDict+0x41b)[0x7fbb4378eacb]
 [27] /usr/local/lib/libpython3.6m.so.1.0(_PyObject_FastCallDict+0x10e)[0x7fbb436fa86e]
 [28] /usr/local/lib/libpython3.6m.so.1.0(_PyObject_Call_Prepend+0x61)[0x7fbb436fb1f1]
 [29] /usr/local/lib/libpython3.6m.so.1.0(PyObject_Call+0x47)[0x7fbb436fac17]
 *** End of error message ***

$ mpiexec -n 8 python3 train_multi.py --batchsize 8
--------------------------------------------------------------------------
WARNING: There are more than one active ports on host 'coco1-worker-0', but the
default subnet GID prefix was detected on more than one of these
ports.  If these ports are connected to different physical IB
networks, this configuration will fail in Open MPI.  This version of
Open MPI requires that every physically separate IB subnet that is
used between connected MPI processes must have different subnet ID
values.

Please see this FAQ entry for more details:

  http://www.open-mpi.org/faq/?category=openfabrics#ofa-default-subnet-gid

NOTE: You can turn off this warning by setting the MCA parameter
      btl_openib_warn_default_gid_prefix to 0.
--------------------------------------------------------------------------
[coco1-worker-0:115997] 7 more processes have sent help message help-mpi-btl-openib.txt / default subnet prefix
[coco1-worker-0:115997] Set MCA parameter "orte_base_help_aggregate" to 0 to see all help / error messages
0 (1, 3, 1344, 576)
6 (1, 3, 800, 1216)
4 (1, 3, 1088, 800)
7 (1, 3, 800, 1216)
15 (1, 3, 800, 1088)
19 (1, 3, 1088, 800)
19 (1, 3, 704, 1344)
41 (1, 3, 800, 1088)
[coco1-worker-0:116005] Failed to cuMemcpy GPU memory, rc=-1
--------------------------------------------------------------------------
The call to cuMemcpyAsync failed. This is a unrecoverable error and will
cause the program to abort.
  cuMemcpyAsync(0x7f5f2f22ac00, 0x7f5df4dfb000, 8192) returned value 1
Check the cuda.h file for what the return value means.
--------------------------------------------------------------------------
Exception in main training loop: MPI_ERR_TRUNCATE: message truncated
Traceback (most recent call last):
  File "/home/yuyu2172/chainer/chainer/training/trainer.py", line 316, in run
    update()
  File "/home/yuyu2172/chainer/chainer/training/updaters/standard_updater.py", line 170, in update
    self.update_core()
  File "/home/yuyu2172/chainer/chainer/training/updaters/standard_updater.py", line 182, in update_core
    optimizer.update(loss_func, *in_arrays)
  File "/home/yuyu2172/chainer/chainermn/optimizers.py", line 28, in update
    self.communicator.bcast_data(target)
  File "/home/yuyu2172/chainer/chainermn/communicators/mpi_communicator_base.py", line 615, in bcast_data
    self.mpi_comm.Bcast(buf)
  File "mpi4py/MPI/Comm.pyx", line 579, in mpi4py.MPI.Comm.Bcast
Will finalize trainer extensions and updater before reraising the exception.
Traceback (most recent call last):
  File "train_multi.py", line 249, in <module>
    main()
  File "train_multi.py", line 245, in main
    trainer.run()
  File "/home/yuyu2172/chainer/chainer/training/trainer.py", line 349, in run
    six.reraise(*exc_info)
  File "/usr/local/lib/python3.6/site-packages/six.py", line 693, in reraise
    raise value
  File "/home/yuyu2172/chainer/chainer/training/trainer.py", line 316, in run
    update()
  File "/home/yuyu2172/chainer/chainer/training/updaters/standard_updater.py", line 170, in update
    self.update_core()
  File "/home/yuyu2172/chainer/chainer/training/updaters/standard_updater.py", line 182, in update_core
    optimizer.update(loss_func, *in_arrays)
  File "/home/yuyu2172/chainer/chainermn/optimizers.py", line 28, in update
    self.communicator.bcast_data(target)
  File "/home/yuyu2172/chainer/chainermn/communicators/mpi_communicator_base.py", line 615, in bcast_data
    self.mpi_comm.Bcast(buf)
  File "mpi4py/MPI/Comm.pyx", line 579, in mpi4py.MPI.Comm.Bcast
mpi4py.MPI.Exception: MPI_ERR_TRUNCATE: message truncated

Seemingly working with the current master
Chainer: https://github.com/chainer/chainer/commit/afe903389d822583a5355e9d46e6766d048ebeb5
CuPy: https://github.com/cupy/cupy/commit/155228fdf6bb148a4c5537dfe08fbfef5329e416

For speeding up training process
What I did

Use MultiprocessIterator
Do resize in different process created by MultiprocessIterator
Stop CPU/GPU copy of masks

What I did not do

Use polygon representation of masks. Although this speeds up training as done by Detectron, I wanted the training script to work with instance segmentation datasets that do not have polygon representation of masks.

The performance of the trained model at this moment (should be 33.9mmAP)
mmAP (all): 0.31496558
mmAP (large): 0.46482706
mmAP (medium): 0.33771247
mmAP (small): 0.14785959

Finally reproduced. It was critical to implementmask_to_segm correctly. 2d44d66
mmAP (all): 0.34442434
mmAP (large): 0.50526273
mmAP (medium): 0.36985907
mmAP (small): 0.16083415

Performance after deleting `+1 to bbox https://github.com/yuyu2172/chainercv/commit/6513e2480e98ce3773ec5566c9923be51f75bb23
mmAP (all): 0.33432382
mmAP (large): 0.5024262
mmAP (medium): 0.36352885
mmAP (small): 0.14599578

After https://github.com/chainer/chainercv/pull/781/commits/7e1e3ecee5930c527c7f1c89cdd4719d826095e5 and https://github.com/chainer/chainercv/pull/798
mmAP (all): 0.33900946
mmAP (large): 0.5053451
mmAP (medium): 0.3659876
mmAP (small): 0.1571092

mmAP (all): 0.35994896
mmAP (large): 0.5378797
mmAP (medium): 0.39182574
mmAP (small): 0.16467538

Uploaded Weight scores
ResNet50  Mask
mmAP (all): 0.34175715
mmAP (large): 0.50817233
mmAP (medium): 0.36931357
mmAP (small): 0.15625913

ResNet50 Bbox
mmAP (all): 0.37964782
mmAP (large): 0.4980176
mmAP (medium): 0.4137187
mmAP (small): 0.22214551

ResNet101 Mask
mmAP (all): 0.35996467
mmAP (large): 0.53794646
mmAP (medium): 0.39182332
mmAP (small): 0.16468713

ResNet101 Bbox
mmAP (all): 0.40394193
mmAP (large): 0.5254983
mmAP (medium): 0.4450884
mmAP (small): 0.23563214

pfnCI, test this please

pfnCI, test this please

@Hakuyume
Please review

TODO:
mask_loss -> mask_head_loss
bbox_loss -> bbox_head_loss

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please

Define min_size and max_size in train_multi.py.
use only cv2 backend in mask_utils.py

Change the behavior of resize. Make it break when chainer.config.cv_resize_backend == 'cv2' and cv2 is not installed.
Force cv2 backend in mask_utils.py

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please


in-source build leads to missing headers

when using out-of source build the headers are installed correctly, but not in-source
see https://github.com/davisking/dlib/blob/master/dlib/CMakeLists.txt#L891

Expected Behavior

Headers getting installed
Current Behavior

Headers are not installed
Steps to Reproduce

$ curl -L https://github.com/davisking/dlib/archive/v19.19.tar.gz | tar xz && cd dlib-19.19 \
 && cmake -DBUILD_SHARED_LIBS=ON -DCMAKE_BUILD_TYPE=Release . && make install
$ cat install_manifest.txt
/usr/local/lib64/libdlib.so.19.19.0
/usr/local/lib64/libdlib.so
/usr/local/include/dlib/config.h
/usr/local/include/dlib/revision.h
/usr/local/lib64/cmake/dlib/dlib.cmake
/usr/local/lib64/cmake/dlib/dlib-release.cmake
/usr/local/lib64/cmake/dlib/dlibConfig.cmake
/usr/local/lib64/cmake/dlib/dlibConfigVersion.cmake
/usr/local/lib64/pkgconfig/dlib-1.pc

Version:
Where did you get dlib:
Platform:
Compiler:
version is 19.19, Linux

You can't run cmake and tell it to generate all the build files in that folder, it stomps all over a bunch of other dlib files.  Do mkdir build; cd build; cmake .. instead.

Yes, that's what I said: out-of-source builds are fine.
If it's borked, then the cmakelists should not allow it in the first place instead of silently messing up the install.

Sure.  It's disabled now on master.

thanks


change mapObj category to object

I don't if this was intentional, but mapObj is in the List category. I think it belongs in the Object category. If there is strong opposition, then feel free to close the request.
I apologize if I didn't build some auxiliary files. make told me 'dist/ramda.js' is up to date.

Definitely not intentional.  Thanks for the fix.
We no longer commit dist/ramda.js, so you should be all set.


Basic Services related to TAC is not working

The following basic services are responding with 500 internal error ,

/v1/redirect-topology-change-information
/v1/inquire-oam-request-approvals
/v1/redirect-oam-request-information
/v1/redirect-service-request-information

In the controller/BasicServices.js , the call to nf-core-model-ap-bs/basicServices/BasicServicesService dont require the attribute originator.
For example , for redirectOamRequestInformation , the service call shall be as below ,
await BasicServices.redirectOamRequestInformation(body, user, xCorrelator, traceIndicator, customerJourney, req.url)
Similarly all the 4 services needs to be modified and tested.

Issue is fixed in the version 1.1.0.a_impl. All the four mentioned APIs are working as expected in this package.
hence closing this issue.


fix: add prop for newTab to serialize CMSLink

May be missing something, but pretty sure the prop for newTab is missing from the serializer for CMSLink.
Hope this helps ;-)

Thanks @58bits! 🙌 🙌 🙌

Welp looks like @JarrodMFlesch actually took care of this some time ago, not sure how I missed that but https://github.com/payloadcms/website/commit/889554f938287aa23c6ec56ab85b917ca869c276 resolves the two. Thanks


Add convert from AnyMValue and AnyMSeq methods to each Monad comprehender / adapter

This will make converting from an abstracted AnyM type to a concrete type potentially much simpler.
We will probably need to create a Seq equivalent of ValueComprehender to account for the differing return types when converting from AnyMSeq to (for example) a List versus an Optional.

Replaced with type safe adapters.


reading .hdf5 file in sopa?

Hi @quentinblampey, Can I ask you if the case I have a .hdf5 file of the merscope AnnData (it has been segmented by cellpose but not annotated, and UMAPs are there), do you think it can be read by sopa, and run Tangram directly for cell type annotation? I tried to dun Tangram separately but had some trouble with the process. Thank you very much!

Hi @KunHHE, can you try to directly use spatialdata_io as below? It should be able to read the Vizgen segmentation. Inside this spatialdata object, you'll have an AnnData object that you can use to run Tangram from sopa.
import spatialdata_io

sdata = spatialdata_io.merscope("/path/to/data")

NB: the new SpatialData version is still not released, so I still can't release sopa==2.0.0


sddm kde5 plasma theme

I wanted to know if it is possible (and HOW? if it is) to have kde5 plasma theme for sddm without installing the whole kde5.

Linux noisecrew 4.9.17 #1-NixOS SMP Wed Mar 22 11:44:07 UTC 2017 x86_64 GNU/Linux

nix-env (Nix) 1.11.8

@ttuegel

As per nixos/modules/services/x11/desktop-managers/plasma5.nix, setting
{
  services.xserver.displayManager.sddm.package = pkgs.sddmPlasma5;
}

in your /etc/nixos/configuration.nix will do it. The Plasma 5 theme has many dependencies, so do not be alarmed if this requires you to download many new packages.


Countdown

Echo countdown to chat.  Also prevent "start" until countdown has completed

Countdown is echoed to chat.  Need to put in start prevention w/ toggle


Change in v3 - clearing parameters from URL clears them from the state

If you're at /wherever?animal=ant and you navigate to /wherever?, in v3 you are given {} as the URL state - IE the value is completely removed from the state.
In v2, you'd still get a value in the state. It's happening just because of how setState no longer accepts Partial<T> anymore. In v2, React would just treat it as a partial state update so you'd get back the previous values.
I had started putting in a fix but realised it's not actually clear what the ideal behaviour is here. I think I'd actually expect to get nothing back from the state, ie the new behaviour is correct, but I can see arguments both ways.
I'm guessing this wasn't an intentional change - thoughts?

And hopefully this is the last breaking change... it's

This was an intentional change, I consider the old behvaior to be a bug, you had no way of clearing the state!

I have been fighting this problem recently. Thank you for changing it, but to @CharlesRea s point, would it be possible to add a flag for the old way vs the new way?

Any chance to get this ported to the v2 line? You changed over to using hooks exclusively in v3, and I'm not at a point where I can upgrade React in my project.

Sure thing @Cosmo439, I would be up for accepting a pull-request if you fancy having a go ad adding a flag for the v2 branch.


merge 1.0 into dev

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, ./.github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

This brings all of the new packages and tooling changes from the 1.0 branch into the dev branch.
Change Log

Merge 1.0 into dev branch and resolve conflicts

Does this affect the toolchain?

YES
NO
Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-YYYY-XXXX

Test Methodology

Pipeline build id: xxxx

Suggestion: I'm a bit concerned about how much tooling has deviated between 1.0 and dev. I'd like to see everyone with changes here sign off on this PR. I think the only reasonable way to do this is to have everyone review their own commits (ie explicitly add everyone as a required reviewer).

Suggestion: I'm a bit concerned about how much tooling has deviated between 1.0 and dev. I'd like to see everyone with changes here sign off on this PR. I think the only reasonable way to do this is to have everyone review their own commits (ie explicitly add everyone as a required reviewer).

Good idea; I've added individual reviewers to the PR

Signing off on the things I've touched,

Signing off on the things I've touched.

Signing off on my commits


Cannot inject spy function which accepts arguments

The following snippet causes my dependency injection to blow up:
mocks: {
  someModule: {
    someService: {
      method: sinon.spy(function(arg){ return arg; })
    }
  }
}

It causes the following cascading awfulness:
Error: [$injector:modulerr] Failed to instantiate module function ($provide, $injector) due to:
        TypeError: 'null' is not an object (evaluating 'argDecl[1]')
            at annotate (C:/dev/static-file-server/app/libs/angular/angular.js:3494)
            at C:/dev/static-file-server/app/libs/angular-mocks/angular-mocks.js:2219
            at partiallInjectMethod (C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:2542)
            at C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:2567
            at forEach (C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:466)
            at partiallyInjectObject (C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:2568)
            at C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:2602
            at forEach (C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:466)
            at C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:2607
            at invoke (C:/dev/static-file-server/app/libs/angular/angular.js:4219)
            at C:/dev/static-file-server/app/libs/angular/angular.js:4136
            at forEach (C:/dev/static-file-server/app/libs/angular/angular.js:326)
            at loadModules (C:/dev/static-file-server/app/libs/angular/angular.js:4157)
            at createInjector (C:/dev/static-file-server/app/libs/angular/angular.js:4041)
            at workFn (C:/dev/static-file-server/app/libs/angular-mocks/angular-mocks.js:2426)
            at C:/dev/static-file-server/app/libs/angular-mocks/angular-mocks.js:2409
            at injectDeps (C:/dev/static-file-server/node_modules/ng-describe/dist/ng-describe.js:2753)
            at C:/dev/static-file-server/node_modules/karma-jasmine/lib/boot.js:124
            at C:/dev/static-file-server/node_modules/karma-jasmine/lib/adapter.js:171
            at http://localhost:9876/karma.js:190
            at http://localhost:9876/context.html:234

Edited to add that this is necessary because I need to spy on a service call made by another service during initialization. With controllers, I can simply wait until after I've set up the spy to instantiate the controller. But with services, there's no api option to manually instantiate the service. And if I try and set up the spy in the beforeEach block within the 'tests' function, it's too late for the spy to capture calls.
e.g:
angular.module('myModule', ['otherModule'])
.factory('someService', function(otherService){
   var initData = otherService.thing(); // this is the call I want to spy on
  return { theData: initData };
});

Unfortunately, the service gets created before the beforeEach block in the tests function, thus the conundrum.  I'm relatively certain that I could skip injecting the service under test and use the injector manually, but that's the sort of thing I want to avoid with ng-describe :)

👍


Docs

I big fan of Pug and want to use it in React, but have no docs for this project :(

Thanks for reminding me about this.  See pugjs/pug-www#24


Update Dependencies: StyleLint 16.0

StyleLint 16 released recently.  Appears to be some breaking changes affecting gulp-sylelint.
Bunch of these:
The CommonJS Node.js API is deprecated. See https://stylelint.io/migration-guide/to-16'
and a failure:
TypeError in plugin "gulp-stylelint"
Message:
    formatter is not a function

Yes, there are a lot. Instrctions here: Migrating to 16.0.0. Simply updating peerDependencies won't cut it. I have very little time to investigate, so any help appreciated.

I took a stab at it in PR #11.  Unit tests are at least passing...

Hello, just wondering what the new timeline for a release of gulp-stylelint with latest stylelint is to be released ?

Would also love to see this, seems like every gulp-stylelint fork gets abandoned...

This is not abandoned. We're been using 15 in the projects. On Friday our team finally has the checkup for future upgrades and this is one of them. Will let you guys know after that. Sorry for tjhe delays.

Good to hear, I don't mean to be pushy about what is a free library, I appreciate the update!

My notes from today.


missing PromptTextNode

PromptTextNode not found

I am also having this issue. I can't feed in any "normal" string nodes from other modules, and I can't tell where this PromptTextNode is supposed to be coming from.

Ah. As per a commend on a PR, the missing node is here: https://github.com/AIFSH/UtilNodes-ComfyUI
Installing it, it does seem to work.

Hi , I forked it and fixed.

for my, 'pip install git+https://github.com/huggingface/transformers -U'

Hi , I forked it and fixed. https://github.com/benjiyaya/OmniGen-ComfyUI/

Thanks for that - loading yours now - can i ask if it also addresses the fact that it reloads the model every generation? For me using the app version in the same cloud GPU it runs in 30 seconds and 130-180 on comfyui - the only difference seems to be dropping the model out of vram somehow…

The CR Text node from the Comfyroll pack works as well when connected directly, but not when sending its signal via anything everywhere nodes

Hi , I forked it and fixed. https://github.com/benjiyaya/OmniGen-ComfyUI/

@benjiyaya i used your repo, still getting this

I just made a PR that no longer requires the prompt_text node https://github.com/AIFSH/OmniGen-ComfyUI/pull/22


2.0.0-beta.1 migration documentation LineCartesianLayer.rememberPoint not found

Area
Maybe it should be rememberPoint -> LineCartesianLayer.point?
I also see it is no longer remembered?
Error
Unresolved reference: LineCartesianLayer.rememberPoint
Additional information
No response

Hello! Thanks for the report. The replacement API is, indeed, LineCartesianLayer.point. We’ve corrected the release notes and will get back to you as soon as we can to describe the reason why the new function doesn’t use remember.

Generally, remember should be used in these cases:

when the same instance of a class must be retained across recompositions for functional reasons (consider, for example, VicoScrollState)
for heavy calculations that needn’t be performed on each recomposition

Previously, some of the compose module’s factory functions used remember when neither of these conditions was met. This did the following:

It created redundancy.
It needlessly made some factory functions composable, increasing the number of cases where two factory functions were required for the same API—one for composable contexts, and one for ExtraStore lambdas and the like.
It led to inconsistency with those factory functions where remember was unneeded and thus, as expected, not used.

Vico 2.0.0 Beta 1 addresses this excessive use of remember, among other things, hence the dropping of remember for LineCartesianLayer.Point. It’s clear that condition 1 isn’t met. As for condition 2, LineCartesianLayer.Point is only a light data carrier—instantiating it doesn’t constitute a heavy operation. (It does require a small amount of processing, of course, but so does retrieving a value saved by remember.)
In strong skipping mode, the use of remember for objects of unstable types facilitates composable skipping. Some of the factory functions from which remember was removed in Vico 2.0.0 Beta 1 return such objects. There’s no regression here, however, as Vico uses strong skipping mode only since the same version. (It’s on by default in Kotlin 2.0.20 and later.) Regardless, skipping will be made possible in all these cases—and many more—but by means of marking the APIs in question as stable, which is more appropriate here. We’re working on this right now.