If an URL returns status 500, CheerioCrawler logs an exception

See third comment for the correct reproduction code and bug description
Now describe the bug
If an URL returns status 500 - Internal Server Error, the CheerioCrawler logs an exception and doesn't call the handleFailedRequestFunction. Failed 500 status requests are not logged in the SDK_CRAWLER_STATISTICS file - the requestsFailed field is zero.
To Reproduce
const Apify = require('apify');
const { log } = Apify.utils

Apify.main(async () => {
    const requestList = await Apify.openRequestList('test', ['https://httpbin.org/status/500']);
    const crawler = new Apify.CheerioCrawler({
        // Let the crawler fetch URLs from our list.
        requestList,
        maxRequestsPerCrawl: 1,

        handlePageFunction: async ({ request, $}) => {
            log.debug(`Processing ${request.url}...`);
        },

        // This function is called if the page processing failed more than maxRequestRetries+1 times.
        handleFailedRequestFunction: async ({ request }) => {
                log.debug(`Request ${request.url} failed twice.`);
        },
    });

    await crawler.run()
    console.log('Crawler finished.');
});

Running this returns:
ERROR CheerioCrawler: handleRequestFunction failed, reclaiming failed request back to the list or queue {"url":"https://httpbin.org/status/500","retryCount":1}
  Error: 500 - Internal Server Error:
      at CheerioCrawler._requestFunction (/home/michal/SIP/src/scrapers/apify/node_modules/apify/build/crawlers/cheerio_crawler.js:500:19)
      at processTicksAndRejections (internal/process/task_queues.js:89:5)
      at async /home/michal/SIP/src/scrapers/apify/node_modules/apify/build/utils.js:362:26

I am reasonably sure that this exception is thrown by https://github.com/apify/apify-js/blob/cd46d5ade3cfcb4babc8d669ce611e3517efff9a/src/crawlers/cheerio_crawler.js#L588 and handled by this function https://github.com/apify/apify-js/blob/cd46d5ade3cfcb4babc8d669ce611e3517efff9a/src/crawlers/basic_crawler.js#L493 It seems that the exception is handled and passed to the logger before or while executing the return statement: https://github.com/apify/apify-js/blob/cd46d5ade3cfcb4babc8d669ce611e3517efff9a/src/crawlers/basic_crawler.js#L513 so the handleFailedRequestFunction is never executed.
Expected behavior
The exception should be caught and passed to the handleFailedRequestFunction. Statistics should include the numer of failed status 500 requests.
System information:

OS: Arch Linux
Node.js version 12.3.0
Apify SDK version 0.21.4

There are several things to explain:

By default, all Crawlers retry 3 times before considering the request failed. You can see in your error "retryCount":1 so it didn't get to the failed state yet.
You have log.debug. Debug logs are not on by default so you might have missed some messages.
Only if the request fails 3 times (or maxRequestRetries + 1 times), it is processed in the handleFailedRequestFunction and added to the statistics.
You can see the retries in the retryHistogram in the statistics log message.

Thank you for pointing out the log.debug problem. I also made a mistake copying the reproduction code in original description, the correct code is below:
const Apify = require('apify');
const { log } = Apify.utils

Apify.main(async () => {
    log.setLevel(log.LEVELS.DEBUG);
    const requestList = await Apify.openRequestList(`test`, ['https://httpbin.org/status/500']);
    const crawler = new Apify.CheerioCrawler({
        // Let the crawler fetch URLs from our list.
        requestList,
        maxRequestsPerCrawl: 10,
        maxRequestRetries: 1,

        handlePageFunction: async ({ request, $}) => {
            log.debug(`Processing ${request.url}...`);
        },

        // This function is called if the page processing failed more than maxRequestRetries+1 times.
        handleFailedRequestFunction: async ({ request }) => {
                log.debug(`Request ${request.url} failed twice.`);
        },
    });

    await crawler.run()
    console.log('Crawler finished.');
});

The output of this code is:
INFO  System info {"apifyVersion":"0.21.4","apifyClientVersion":"0.6.0","osType":"Linux","nodeVersion":"v12.3.0"}
WARN  Neither APIFY_LOCAL_STORAGE_DIR nor APIFY_TOKEN environment variable is set, defaulting to APIFY_LOCAL_STORAGE_DIR="/home/michal/SIP/src/scrapers/apify/apify_storage"
DEBUG RequestList: Loaded state from key value store using the persistStateKey.
INFO  CheerioCrawler:AutoscaledPool:Snapshotter: Setting max memory of this run to 16057 MB. Use the APIFY_MEMORY_MBYTES environment variable to override it.
DEBUG CheerioCrawler:AutoscaledPool: scaling up {"oldConcurrency":1,"newConcurrency":2,"systemStatus":{"isSystemIdle":true,"memInfo":{"isOverloaded":false,"limitRatio":0.2,"actualRatio":null},"eventLoopInfo":{"isOverloaded":false,"limitRatio":0.7,"actualRatio":null},"cpuInfo":{"isOverloaded":false,"limitRatio":0.4,"actualRatio":null},"clientInfo":{"isOverloaded":false,"limitRatio":0.3,"actualRatio":null}}}
INFO  CheerioCrawler:AutoscaledPool: state {"currentConcurrency":0,"desiredConcurrency":2,"systemStatus":{"isSystemIdle":true,"memInfo":{"isOverloaded":false,"limitRatio":0.2,"actualRatio":null},"eventLoopInfo":{"isOverloaded":false,"limitRatio":0.7,"actualRatio":null},"cpuInfo":{"isOverloaded":false,"limitRatio":0.4,"actualRatio":null},"clientInfo":{"isOverloaded":false,"limitRatio":0.3,"actualRatio":null}}}
ERROR CheerioCrawler: handleRequestFunction failed, reclaiming failed request back to the list or queue {"url":"https://httpbin.org/status/500","retryCount":1}
  Error: 500 - Internal Server Error:
      at CheerioCrawler._requestFunction (/home/michal/SIP/src/scrapers/apify/node_modules/apify/build/crawlers/cheerio_crawler.js:500:19)
      at processTicksAndRejections (internal/process/task_queues.js:89:5)
      at async /home/michal/SIP/src/scrapers/apify/node_modules/apify/build/utils.js:362:26
DEBUG Request https://httpbin.org/status/500 failed twice.
INFO  CheerioCrawler: All the requests from request list and/or request queue have been processed, the crawler will shut down.
DEBUG Statistics: Persisting state {"persistStateKey":"SDK_CRAWLER_STATISTICS_0"}
INFO  CheerioCrawler: Final request statistics: {"requestAvgFailedDurationMillis":462,"requestAvgFinishedDurationMillis":null,"requestsFinishedPerMinute":0,"requestsFailedPerMinute":59,"requestTotalDurationMillis":462,"requestsTotal":1,"crawlerRuntimeMillis":1005,"requestsFinished":0,"requestsFailed":1,"retryHistogram":[null,1]}
Crawler finished.

The reproduction code above sets maxRequestsRetries: 1, so the request is tried only twice. First request fails and is correctly reclaimed back to the queue, but the second (and last) try triggers an exception. The crawler throws an exception, which is handled by the Apify framework  logger code. This is clearly shown in the full debugger console output below:
INFO  System info {"apifyVersion":"0.21.4","apifyClientVersion":"0.6.0","osType":"Linux","nodeVersion":"v12.3.0"}
logger.js:41WARN Neither APIFY_LOCAL_STORAGE_DIR nor APIFY_TOKEN environment variable is set, defaulting to APIFY_LOCAL_STORAGE_DIR="/home/michal/SIP/src/scrapers/apify/apify_storage"
logger.js:49INFO  CheerioCrawler:AutoscaledPool:Snapshotter: Setting max memory of this run to 16057 MB. Use the APIFY_MEMORY_MBYTES environment variable to override it.
logger.js:49INFO  CheerioCrawler:AutoscaledPool: state {"currentConcurrency":0,"desiredConcurrency":2,"systemStatus":{"isSystemIdle":true,"memInfo":{"isOverloaded":false,"limitRatio":0.2,"actualRatio":null},"eventLoopInfo":{"isOverloaded":false,"limitRatio":0.7,"actualRatio":null},"cpuInfo":{"isOverloaded":false,"limitRatio":0.4,"actualRatio":null},"clientInfo":{"isOverloaded":false,"limitRatio":0.3,"actualRatio":null}}}
logger.js:37ERROR CheerioCrawler: handleRequestFunction failed, reclaiming failed request back to the list or queue {"url":"https://httpbin.org/status/500","retryCount":1} Error: 500 - Internal Server Error: at CheerioCrawler._requestFunction (/home/michal/SIP/src/scrapers/apify/node_modules/apify/build/crawlers/cheerio_crawler.js:500:19) at processTicksAndRejections (internal/process/task_queues.js:89:5) at async /home/michal/SIP/src/scrapers/apify/node_modules/apify/build/utils.js:362:26
_outputWithConsole          	logger.js:37
_log                        	logger_text.js:72
log                         	logger.js:58
internal                    	log.js:60
exception                   	log.js:96
_requestFunctionErrorHandler	basic_crawler.js:438
_runTaskFunction            	basic_crawler.js:386
processTicksAndRejections   	task_queues.js:89
Async call from async function
_runTaskFunction            	basic_crawler.js:367
_maybeRunTask               	autoscaled_pool.js:393
processTicksAndRejections   	task_queues.js:89
Async call from async function
_maybeRunTask               	autoscaled_pool.js:366
funcWrapper                 	utilities.js:176
exports.betterSetInterval   	utilities.js:183
run                         	autoscaled_pool.js:255
processTicksAndRejections   	task_queues.js:89
Async call from async function
run                         	autoscaled_pool.js:249
run                         	basic_crawler.js:266
Async call from async function
run                         	basic_crawler.js:265
run                         	cheerio_crawler.js:362
(anonymous function)        	bug.js:23
Async call from async function
(anonymous function)        	bug.js:6
run                         	actor.js:194
exports.main                	actor.js:209
(anonymous function)        	bug.js:4
Module._compile             	loader.js:774
Module._extensions..js      	loader.js:785
Module.load                 	loader.js:641
Module._load                	loader.js:556
Module.runMain              	loader.js:837
(anonymous function)        	run_main_module.js:17
logger.js:49INFO  CheerioCrawler: All the requests from request list and/or request queue have been processed, the crawler will shut down.
logger.js:49INFO  CheerioCrawler: Final request statistics: {"requestAvgFailedDurationMillis":462,"requestAvgFinishedDurationMillis":null,"requestsFinishedPerMinute":0,"requestsFailedPerMinute":59,"requestTotalDurationMillis":462,"requestsTotal":1,"crawlerRuntimeMillis":1005,"requestsFinished":0,"requestsFailed":1,"retryHistogram":[null,1]}
bug.js:24Crawler finished.

The exception is triggered in _requestFunctionErrorHandler.
In my view, this is a bug, because an exception is caused by a standard Status 500 error. I'd expect that status 500 errors should be handled without throwing an exception in crawler code.

Thanks for looking into it @michal-gh
There are some inconsistencies in handling 500 errors in the SDK and we will be looking at normalizing the behavior soon.

Closing this in favor of #812


Why does EMQTT broker node have high Disk I/O usage?

OS: Linux
Erlang/OTP: OTP
EMQ: 2.2
I am observing high disk I/O usage on my emqtt node. Tried disabling the logs on node and still seeing high disk I/O usage on the node.I am using Qos0,1,2 for message transfers and mysql plugins for auth.Could you please let me know in what cases there can be high disk i/o seen?

Is is probably your mysql.
Would you check which process has the highest I/O usage? Using iotop for example.

We have caching enabled on the mysql quth queries too. FYI. will check iotop and get back.

please refer to the iotop command output.

There are two processes writing to disk.Both are of emqtt process.May i know what is this process with TID 23164 which is writing to disk?

I'm not sure what file emqttd is writing.
You could get the PID of emq (the beam process), then use lsof -p $PID to see all the files opened (only pay attention on the regular files). And then check the increasing speed of those files.
Most likely it is the log dir or the data/mnesia dir.

you could reopen this if you still have questions.


Refactor Epidoc XSLT edition structure HTML templates to split out components

To allow the Epidoc example XSLT to work both as currently (standalone) and as part of EFES, refactor the HTML edition structure templates to have the content sections (CSS/JS decalarations, inscription text, title) in named templates. EFES has its own templating system and so needs to be able to access those parts separately from the surrounding HTML.

The templates to change are:

htm-tpl-structure.xsl
htm-tpl-struct-dol.xsl
htm-tpl-struct-edak.xsl
htm-tpl-struct-inslib.xsl
htm-tpl-struct-iospe.xsl
htm-tpl-struct-spes.xsl

Done in r2662 of the Epidoc repository.


feat: add toggles to Input Panel

This PR will close #9

Closed on favor of working in one repo in #19

Closed on favor of working in one repo in #19


Mux Uploader Server Component

Let's take another shot at #712...
How to use it
import MuxUploader, { other, named, imports } from '@mux/mux-uploader-react/rsc';

Defaults
If no endpoint is specified, will use @mux/mux-node to create a direct upload with the following configuration:
{
  cors_origin: '*',
  new_asset_settings: {
    playback_policy: ['public'],
    encoding_tier: 'baseline',
}

How to configure
Accepts all the same params as Mux Uploader, as well as
interface MuxUploaderServerProps extends MuxUploaderProps {
  clientOptions?: ClientOptions; // allows you to configure the @mux/mux-node client
  uploadCreateParams?: Partial<Mux.Video.Uploads.UploadCreateParams>; // allows you to configure the direct upload
  uploadRequestOptions?: Partial<Mux.RequestOptions>; // allows you to configure the direct upload request
}

uploadCreateParams will be deep-merged with the default options described above
How it gets bundled
"build:rsc:cjs": "esbuild src/rsc.tsx --target=es2019 --platform=node --bundle --sourcemap --metafile=./dist/cjs-rsc.json --format=cjs --loader:.css=text --outdir=dist --out-extension:.js=.cjs.js --external:react --external:prop-types --external:server-only --external:./index",
"build:rsc:esm": "esbuild src/rsc.tsx --target=es2019 --platform=node --bundle --sourcemap --metafile=./dist/esm-rsc.json --format=esm --loader:.css=text --outdir=dist --out-extension:.js=.mjs --external:react --external:prop-types --external:server-only --external:./index",

lemme point out some interesting flags, here:

--platform=node: in order to avoid bundling dependencies like crypto, esbuild requires this of us. Luckily, Server Components run on the server, so, this is correct
--external:server-only: the contents of this package is literally just throw new Error('This is a server component and can't be imported in a client component') (paraphrased). If we were to bundle it, our bundle would immediately throw an error. I suspect that, in practice, server-component-compliant bundlers will know to ignore this import. Even so, should server-only  be marked as a peer dependency? 🤔
--external:./index: In order for server component bundlers to leave the server stuff on the server and the client stuff on the client, we have to ship the server bundle and the client bundle separately. By marking index as external, we're telling esbuild "don't include the client bundle in the server bundle". Instead, consumer bundlers will see this in dist/rsc.mjs and resolve dist/index.mjs (or dist/rsc.cjs.js and dist/index.cjs.js respectively).

TODO items:

[ ] get some feedback
[ ] consider this as a way to export the module
[ ] documentation
[ ] two questions for those of you who understand bundlers better than I do...

[ ] am I correct about what I did with --external:server-only and --external:./index? This seems to work in our test Next app, so that's a good sign...
[ ] why is @mux/mux-uploader-react/rsc giving me a red squiggle in the nextjs example? why can't typescript find it? or is that just something in my environment?

Other passing thoughts
I don't re-export the types, so right now if you want those, you'll have to import type { ... } from '@mux/mux-uploader-react'. TypeScript should pick up on that; I'm not worried

What could it look like for this component (or set of them) to be able to return things like an asset ID / playback ID?

Running into a huge blocker with this. Consider the following:
import MuxUploader from '@mux/mux-uploader-react/rsc';

function MuxUploaderPage() {
  return (
    <MuxUploader onSuccess={console.log} />
  );
}

export default MuxUploaderPage;

However, you can't pass client functionality (in our example, console.log) to server components. Props have to be serializable.
Gotta think of another interface if we want this to happen...

What could it look like for this component (or set of them) to be able to return things like an asset ID / playback ID?

@mmcc btw just to answer this. Server Actions would make this really slick. Under the hood, we could implement something like this:
// packages/mux-uploader-react
import 'server-only'
import MuxUploaderClient from './index'

const MuxUploaderServer = (props) => {
  const upload = await mux.video.uploads.create(...);
  return (
    <MuxUploaderClient
      endpoint={upload.url}
      onSuccess={
        "use server"
        const assetId = await getAssetIdForUpload(upload.id)
        props.onAssetId(assetId)
      }
    />
)
}

I've actually got this implemented in another branch.
Just gotta think through... my previous comment...


[5.8] Quick Navigation default enablement

Explanation: Introduce quick navigation as a default feature in docc-render by defaulting getSetting for the enableQuickNavigation function to true, still working as a customization point through a theme settings file (https://forums.swift.org/t/quick-navigation-enablement-request/62523)
Scope: New feature for all docs
Issue: N/A
Risk:
Testing: Render any archive without a theme file and assert that Quick Navigation is enabled
Reviewer: @franklinsch
Original PR: https://github.com/apple/swift-docc-render/pull/518

@swift-ci please test

@swift-ci test


feat: wallet icon on connect button

Signed-off-by: Chris Gianelloni<EMAIL_ADDRESS>

merge conflict. Will add this details tho. Thank you


multinode: Run RefStack Tempest tests by default

A full tempest test run takes a long time, and may not always be
necessary. Let's run RefStack by default, and make it easy to switch to
the full test suite.

Related change: https://github.com/stackhpc/stackhpc-kayobe-config/pull/617


Extension seems broken

Screenshots

Desktop (please complete the following information):

Browser: chromium
Browser version: 116.0.5817.0 (Official Build) snap (64-bit)
Extension version: 0.3.0

Hi @aluyama,
Thank you for reporting the issue! This is probably due to styles that were located at CDN. I released a new version of the extension with updated styles that are built-in now so the issue should be gone. Please update to the latest version of the extension.

Thanks, now no problem.

Again, UI is broken :(
but not more as ago.

Desktop (please complete the following information):
Browser: chromium
Browser version: 116.0.5845.4 (Official Build) snap (64-bit)
Extension version: 1.0.0
Additional details:
I didn't notice any UI issues in other extensions. (ex. dark reader, marvellous suspender)

Hi @aluyama,
Got it, seems you cannot access Google Fonts (https://fonts.googleapis.com/icon?family=Material+Icons) due to some setting in browser/network or limitations in your location. I will make them to be built-in in the next release.
Thanks


When a ChoiceCellEditor is selected nothing happens

I selected "yes" on a ChoiceCellEditor new components to be displayed is expected but that didn't happen. 3.3

When selecting "yes" the new components appear now... Need to check this out more

The new components appear now I'm closing this issue.


Non recursive consolidator

Following the discussion here:
https://github.com/trapexit/mergerfs/issues/709
I want to move files around, but not the folders, to match this new policy, I just want to move my files around, not the subfolders.
mergerfs.consolidate could work, but the rsync command is recursive so that my folders will get moved around too, because rsync will recurse on folders with the "-a" which implies recursion, but I could simply replace it with: "-lptgoD" (the same as -a without the recurse flag) which then won't recurse.
This will likely work (correct me if I'm wrong), and  could be useful for other people. If I make the changes:
Should I give it a new name/tool? Or
Should I change the description of the consolidator so that it reflects this since now not all the data will be a on a single drive?

rsync isn't doing any recursion. The tool is. The easiest way to get it to act on one directory is to add a flag to disable the os.walk by setting dirs = [].

Got it. rsync is used to copy files, not folders, so the recursive option in rsync is meaningless.

Doesn't mean your request is invalid. If you want it to only work on one directory rather than walking each sub directory I can add that.

It does what I want it to do.
I didn't fully understand the code, so I had added a dry-run flag to look at what rsync was doing.
The description:

Consolidate files in a single mergerfs directory onto a single drive.
combined with the recursive flag made me think that all of the files and subfolders would be merged.
Rather, the actual behaviour is each subfolder will contain all of it's files on a single drive.


[BUG] LMSYS (claude-v1) disconnect each 10 min

how prevent that?

Could you follow the bug report template to provide more useful info? Such as errors in console. Thanks.

How can view that logs using windows installer?
El mar, 27 jun 2023, 7:02 p. m., Sun Zhigang @.***>
escribió:

Could you follow the bug report template to provide more useful info? Such
as errors in console. Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/sunner/ChatALL/issues/301#issuecomment-1610382772,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADIWFBBGHF4YC7MPANTORTXNNX7RANCNFSM6AAAAAAZVVKSXA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@johnfelipe Thank you for the clear demo. Let me think about how to refresh the session

not only need to refresh and/or retry answer, is need to all steps in this video:
https://github.com/sunner/ChatALL/assets/428820/a4ce89d8-8651-4d7e-b799-233795815ed4

refresh
say hello or something
Delete answer
retry question and is OK

Before the 3rd step, refresh it again (refresh 2 times), you can get the answer.
Yes, this need to be improved. I'll give it a try.


Remove redundant arguments for StatItem#isAllowable

Currently, when we check if is allowable in StatItem, we have two unnecessary arguments which are not used in the method. So we can remove them.

Codecov Report

Merging #1377 into master will decrease coverage by 0.18%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1377      +/-   ##
==========================================
- Coverage   32.57%   32.39%   -0.19%
==========================================
  Files         691      691
  Lines       34537    34537
  Branches     6812     6812
==========================================
- Hits        11252    11188      -64
- Misses      21357    21417      +60
- Partials     1928     1932       +4

Impacted Files
Coverage Δ

...ava/com/alibaba/dubbo/rpc/filter/tps/StatItem.java
79.16% <ø> (ø)
:arrow_up:

...libaba/dubbo/rpc/filter/tps/DefaultTPSLimiter.java
66.66% <100%> (ø)
:arrow_up:

...ba/dubbo/remoting/transport/netty/NettyClient.java
72.58% <0%> (-11.3%)
:arrow_down:

...mon/serialize/support/dubbo/GenericDataOutput.java
66.31% <0%> (-9.13%)
:arrow_down:

...libaba/com/caucho/hessian/util/IdentityIntMap.java
25% <0%> (-5.96%)
:arrow_down:

...om/alibaba/dubbo/rpc/filter/ActiveLimitFilter.java
83.33% <0%> (-5.56%)
:arrow_down:

...mmon/serialize/support/dubbo/GenericDataInput.java
58.68% <0%> (-5.02%)
:arrow_down:

...a/dubbo/remoting/transport/netty/NettyChannel.java
61.25% <0%> (-5%)
:arrow_down:

...ba/dubbo/remoting/transport/netty/NettyServer.java
67.85% <0%> (-3.58%)
:arrow_down:

...main/java/com/alibaba/dubbo/rpc/RpcInvocation.java
15.11% <0%> (-2.33%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Update Conversation Flows

Modify the conversation flows in our chatbot to accommodate LLama's response format and input requirements.

For this issue, we are planning on adding a flow control mechanism to decide whether to proceed with report generation or consult LLaMA-2 for an answer.
We have to incorporate logic to handle the "start report" command specifically, while other inputs are processed through LLaMA-2.

Managed to end conversation flow

Managed to ask a question during the conversation flow
Modified the msgrcvd function:

Added a new function to call the llama model when a question is asked:

Modified the code to send the details of the report at the end:

Modified existing process_answer function tu handle the summary generation:

Created a function to prepare the summary:

modified funciton to create a pdf:

pdf generated:

Great work, passes tests and achieves purpose.


Add [[Category:Uploaded with OpenRefine]] to each file uploaded to Commons with OpenRefine

We already have a nice OpenRefine tag on Wikimedia Commons. However, it would also be great to add a tracking category to each file that is uploaded to Wikimedia Commons with OpenRefine. With such a tracking category, we can more easily run metrics to measure usage / uptake of files in Wikipedia and other Wikimedia projects, and their page views.
I have created one for this purpose: Category:Uploaded with OpenRefine which basically imitates the one that the Pattypan tool uses.

This now works, so I will take the freedom to close this issue (or is there anything that still needs fixing or improving?).

The category name should ideally be configurable in the wikibase manifest, so my intention was to keep this open until then, but after all maybe this is not needed.


Checks are not run on PR involving forked repositories

When a PR is done from a forked repository, the checks are not done. We are using pull_request as target, instead of the newest pull_requests_target. Read differences here
This should fix, for example, #12 PR.

In fact it's even easier, we are triggering on push event instead of pull_request.


Set default account description modal + transaction list element border bug

Purpose
Shows a modal first time a default account is set, describing the action at what it implies. Also fixes a small style bug on transaction list elements introduced in #60
Changes

Show prompt modal when setting default account first time
Hide top border for first transaction list element in list group.

Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.

Closes #91

The transaction list element border bug is now present on the top, in the infinite list.

The transaction list element border bug is now present on the top, in the infinite list.

😢
Should be fixed now!


Add comments to the index.html file

Please describe about the issue , like which comments you wants? If possible I want to do it.

@Sowham-3098  as you must have seen the web page is quite big if you could possibly comment down different sections ...like where the navbar is beginning or ending etc it would be a great help

Yeah I am happy to do that

Yeah you can assign me

@RIKI-05 Please go ahead

Please assign me this issue.


When to use keras.layers.core.Layer?

Hi,
Sorry if this is a simple question as I am new to Keras. I have seen a couple of examples where people were using the function Layer in keras.layers.core. But, I could not find a documentation about it. Can someone explain me what is Layer used for? And is it different from keras.layers.core.Dense?

The Layer class is just an abstract interface used to implement layers. It's not a functional NN layer on its own. You don't need to use it unless you're trying to write your own layers, in which case see https://keras.io/layers/writing-your-own-keras-layers/ .


Should not be able to create existing sandbox/container.

Add integration test for duplicated sandbox/container creation.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

@mikebrow Thanks for reviewing!


[hi] Cloud computing

Please describe why this term needs to be added
I want to localize cloud computing term to hindi localization.

/assign

/assign

Don't we have the self assign system, @jihoon-seo

@abh139 it seems that we don't have a self assign system.
BTW, it seems that you are already assigned to another issue: #771.
We can assign you one term at a time, so if you want to work on this, please unassign yourself from the issue #771,
or please resolve #771 and then you might be able to be assigned on this.

@Garima-Negi @anubha-v-ardhan Can you assign me for this issue?

@Garima-Negi @anubha-v-ardhan Can you assign me for this issue?

Hi Bishal, I believe you already have an issue assigned to you. We only assign 1 issue to a person at a time

@Garima-Negi Actually I have already made a PR for that assigned issue.
Can You please review it so that I can work on another issue?

Sure whenever the reviewers have time they will review.

May I work on this? @jihoon-seo

We can only assign this to one person.
@octonawish-akcodes, since you opened this issue and showed interest (https://github.com/cncf/glossary/issues/1005#issuecomment-1170880428), Do you wish to work on this? else we can assign the issue to next person on first come first serve basis

@anubha-v-ardhan I would like to work on that after merging [hi] version_control.md
Could you pls assign me this issue?

@anubha-v-ardhan Please assign this to someone else, I am tackling some issues in my local configuration in rebasing so I am fixing that, please assign this to someone else.

Can I work on this issue?

@devsachinsharma Are you still interested in working on this issue?

@anubha-v-ardhan  Yes, I'm interested. But can you please review the remaining part of this issue https://github.com/cncf/glossary/pull/1082 . Thank you.

/reassign @devsachinsharma

@shivam-sharma7 please comment on this issue so that I can assign it to you.
your PR has been reviewed recently

Hey @Garima-Negi can you assign me?

@shivam-sharma7 done!

Resolved by #1250


fix: add react and react-dom to resolve.dedupe at config

Adds "react" and "react-dom" to resolve.dedupe in the config hook. @vitejs/plugin-react does the same at https://github.com/vitejs/vite-plugin-react/blob/main/packages/plugin-react/src/index.ts#L289-L291.
For some context, the missing dedupe of these React packages caused issues in my own React meta-framework (@lazarv/react-server) when adding support for @vitejs/plugin-react-swc. Using Vite Environment API it was not able to resolve "react/jsx-dev-runtime". I usually work on the framework in a pnpm workspace and the framework also provides it's own strict experimental React version.

My plan is to do the opposite and remove the dedupe option from the Babel plugin: https://github.com/vitejs/vite-plugin-react/issues/280#issue-2092681383
For me users should always have exactly one version of React installed. Why is this not the case for you?
How can you be sure that the dedupe option will keep the one you want?

My issue is that Vite uses dedupe to select the root at https://github.com/vitejs/vite/blob/14e14726d11ac55c37cc2d15aaf935be18ef925b/packages/vite/src/node/plugins/resolve.ts#L726-L737
if (dedupe?.includes(pkgId)) {
  basedir = root
} else if (
  importer &&
  path.isAbsolute(importer) &&
  // css processing appends `*` for importer
  (importer[importer.length - 1] === '*' || fs.existsSync(cleanUrl(importer)))
) {
  basedir = path.dirname(importer)
} else {
  basedir = root
}

With a root at /Users/lazarv/Projects/react-server/packages/react-server and having an importer at /Users/lazarv/Projects/react-server/examples/todo/src/index.tsx, when it's trying to resolve react/jsx-dev-runtime and if I don't have react in dedupe then the basedir will be incorrect as it's using the importer's directory, while I only have a resolvable React in the root.
I had a lot of resolution issues because React is not installed in the consumer project, so I do a lot of resolution workarounds to achieve what's necessary for the framework to work like a CLI tool too on any given React component file.
Sorry @ArnaudBarre I missed the existing PR at @vitejs/plugin-react removing resolve.dedupe. My current workaround is to add both react and react-dom to the Vite config in my framework. As everything worked as expected with the Babel version of the plugin, I suspected that resolve.dedupe is just missing here.


Resolve ambiguities in favor of common geographic characters

A relatively small number of Simplified Chinese characters (182) are ambiguous, each one potentially corresponding to two different Traditional Chinese characters (or in a few cases, three of four characters). There are also five cases where a Traditional Chinese character can convert to either of two Simplified Chinese characters depending on the pronunciation, and a handful of characters that appear in both variants but with a different meaning.
Most of these ambiguities seldom appear in toponyms or POI names, but a few are very common in toponyms. For example, 宁 is either the Simplified form of 寧 or the Traditional form of 㝉, but 寧/宁 is much more common than 宁/㝉 in toponyms. Fortunately, ICU avoids simplifying 宁 to 㝉, but there may be other cases to watch out for.
To resolve these ambiguities, either get a native Chinese speaker to weigh in on the more likely choice in each case, or conduct some analysis of fully tagged features in OSM. For example, we can say for sure that 寧/宁 is more likely than 宁/㝉 because there are 28 places containing 寧 in name:zh-Hant and 宁 in name:zh-Hans but none at all containing 宁 in name:zh-Hant and 㝉 in name:zh-Hans. Similarly, 面/面 is more common than 面/麵 by 1,177 to 0. A similar analysis can be conducted using geographical items in Wikidata.

Is this still an issue now that I'm using opencc4j?

Can’t say for sure without going through the exercise above. The list on Wikipedia is a list of inherent ambiguities in the writing system. A converter has to be opinionated, since it doesn’t prompt you for the right answer each time it encounters an ambiguity, but we’d just want to see if it comes up with a reasonable answer for each case. It’s a bit lower-priority than getting the thing to work though.


Host Regexp

Currently globbing for *.*.localdomain or inserting (.*).localdomain does not allow to reach the host localhome.localhost.localdomain.
It could be nice to define Hosts with Regular Expressions, in so multiple levels of subdomains are supported.

@almereyda  Thank you for your suggestion!
Taxy supports regex patterns in virtual hosts starting with v0.3.34.


No new connections when running client on different PC's

I can't see any new connections when running the RAT client on different PC's. Once I was done building it, I took the client and put it onto another computer in my home and when I ran it, there wasn't any connection on my other computer which was running the rat program itself. I get connection refused on port 4782 even though I portforwarded and I allowed it through my firewall, with the program itself too. I dont know if I need to use my local ip address or my ipv4 address for no-ip. Also, does the hostname/IP in the connections tab need to use the no-ip address or the ip address? thank you.

It's definitely your firewall what's blocking Quasar. Please open the port 4782 TCP in Windows Firewall and your router to. Configure your router to forward port 4782 to your local computer to the same port.

Ok also, in my ipv6 firewall settings in my router, should I click enable incoming traffic at the bottom?
https://gyazo.com/9c3d5ea20773debe9a72308223cb8de6

I also get this in my routers firewall settings https://gyazo.com/25d9b619694ea19af130f4881c63034e

I just tried everything and it didn't work for me. Still no new connections when I try to add the no-ip to the host in the builder. https://gyazo.com/726341ecc6d7fa10e82f22af6bd532af also says connection refused again and it says everywhere the port is still closed. https://gyazo.com/1eb5b5ebf5b404ccd7970d5cc17b268a

it just started saying the port is successfully going but I still also get the request timed out in CMD https://gyazo.com/8ee9bf477ab8d2b61453a7fef8abe96e

Not exactly sure what I'm doing wrong. https://gyazo.com/6699d9cf80ee5c01b7df99d399525ee4 here's a screenshot of everything. Port is forwarded, port is allowed through firewall and it is also showing that it's being used when I'm listening. But I still get the request timed out.

Try it out to see if it's working, it doesn't matter if your IP is reachable via ICMP (ping). It should work now.

Alright it works, I had to get a friend to test it out by using his own network/IP and it worked.

Thank you.

Am I able to use control PC's on my own network though? Like if I infected another machine on my wifi will I be able to control it from my main machine with the program on it?

Of course you can use Quasar only in your local area network. Internet connection is not needed for it to work. For example if your server is running on the LAN IP <IP_ADDRESS> then just build a client which connects to the server IP and then install it on every computer in your LAN. Your dynamic dns is not needed for it to work on LAN.

Alright thank you for your help, Appreciate it. I've gotten my problems answered.

No problem! Thanks for using Quasar. :)


Add Microsoft XKCD cartoon to the "Workplace" chpater

Is this the cartoon being referred to?
It looks like it was originally created by "Manu Cornet", is under a creative commons license, and can be reused if appropriate credit is provided. I'm unsure if cropping the image to include only the MS org chart is appropriate as it would remove the artist signature - may need to contact the artist.

Yup this is the one...I think it would be appropriate to crop it and add attribution (saying it was adapted) or to use the whole thing as you think fits the space better.


Srs617

This fix is to make sure we don't get any error when trying to fetch the bannerType. Currently the error was coming due to two reasons:

The default state for userType was set to "External" thus the two useEffects using the userType were picking two different values intermittently.
The dependency array for the useEffects was not using the loggedInuser and userType which is necessary to maintain consistent values throughout.

Key Changes:

Dependency Array in First useEffect: Added loggedInUser to the dependency array to ensure it re-runs when loggedInUser changes.
Dependency Array in Second useEffect: Added userType to the dependency array to ensure it re-runs when userType changes.

@nupurdixit13 , could we please use conventional commit pattern for PR title
https://www.conventionalcommits.org/en/v1.0.0/#summary

@nupurdixit13 , could we please use conventional commit pattern for PR title https://www.conventionalcommits.org/en/v1.0.0/#summary

@mishraomp : I remember this being discussed in the past but wasn't sure if we have already started implementing it. Thanks for reminding! Let me know if the new message adheres to the standards.

@nupurdixit13 , could we please use conventional commit pattern for PR title https://www.conventionalcommits.org/en/v1.0.0/#summary

@mishraomp : I remember this being discussed in the past but wasn't sure if we have already started implementing it. Thanks for reminding! Let me know if the new message adheres to the standards.

thanks @nupurdixit13 , yeah, this works great


How to configure with Azure Automation Runbook?

I am trying to use Azure Automation Runbook to monitor all gateways under my account, but I can't overcome the Login-OnPremisesDataGateway since it has popup window. If I run as Azure, will show "the Internet Explorer engine is not available", if I run as Hybrid worker, will show "Showing a modal dialog box or form when the application is not
running in UserInteractive mode is not a valid operation". Is there any way I can apply above in Azure Automation? Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2a971ee3-1d8f-50bf-0a23-20bf592779bd
Version Independent ID: b69b3560-08fd-6c9e-4bb3-4dfa3838d223
Content: High availability clusters for On-premises data gateway - Power BI
Content Source: powerbi-docs/service-gateway-high-availability-clusters.md
Service: powerbi
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Thanks for your comment, @ShifengWu .
In addition to a documentation inquiry, this looks like a possible support request as well. You might find a solution in the Power BI Community, found at https://community.powerbi.com. If not, you can get support by going to https://support.powerbi.com.
We’ve forwarded your feedback to the writer, who should respond to this thread shortly. Thanks!

Adding the author.

Hi @mgblythe, thanks for follow this, is there have any update? Can we add PromptBehavior same as AuthenticationContext.AcquireToken so that we can avoid the pop up window? Thank you so much.

@ShifengWu -- sorry, but I'm unable to find an answer for this issue. Please try support in the Azure portal.


_generate_random_poses of RegNeRF is wrong

At the 1122 line of internal/datasets.py
    # Generate random poses.
    random_poses = []
    cam2world = poses_avg(poses)
    up = poses[:, :3, 1].mean(0)
    for _ in range(n_poses):
      t = radii * np.concatenate([2 * np.random.rand(3) - 1., [1,]])
      position = cam2world @ t
      lookat = cam2world @ [0, 0, -focal, 1.]
      z_axis = position - lookat
    random_poses.append(viewmatrix(z_axis, up, position))
    self.random_poses = np.stack(random_poses, axis=0)

Its obvious that  random_poses.append(viewmatrix(z_axis, up, position)) should in the for-loop, but it lies outside, which leads to only 1 random pose is genertated.

AFAICT, based on my testing, both cases have the same outcome.

AFAICT, based on my testing, both cases have the same outcome.

The metric may be the same. However, if random_poses.append(viewmatrix(z_axis, up, position)) is in the for-loop. the random_poses will be a list contains n_poses of view-matrix, but in this case, it is a list which only contains one view-matrix.

True. A quick question. Can you replicate the results in the paper?

Haven't tried yet, but I will do some experiments in the next few days. Is there any problem?

I can reproduce the results via the pre-trained models, but I cannot train custom data.

I can reproduce the results via the pre-trained models, but I cannot train custom data. Most likely, this is related to https://github.com/google-research/google-research/issues/1096

I can reproduce the results via the pre-trained models, but I cannot train custom data. Most likely, this is related to #1096

I'm sorry that I can't help you. I just ran this code a few times to understand some data processing flow. However, issues with custom datasets may not be related to #1096 since it just states that there is no NLL Loss in the code.

No problem, I figured it out. :)

so anyone knows how to calculate RealNVP Loss? @amughrabi @SJoJoK

so anyone knows how to calculate RealNVP Loss? @amughrabi @SJoJoK

sorry, idk.


Suddenly stopped working, help

Traceback (most recent call last):
File "...\stable-diffusion-webui\modules\call_queue.py", line 57, in f
res = list(func(*args, **kwargs))
File "...\stable-diffusion-webui\modules\call_queue.py", line 36, in f
res = func(*args, **kwargs)
File "...\stable-diffusion-webui\modules\txt2img.py", line 107, in txt2img
processed = modules.scripts.scripts_txt2img.run(p, *p.script_args)
File "...\stable-diffusion-webui\modules\scripts.py", line 766, in run
processed = script.run(p, *script_args)
File "...\stable-diffusion-webui\extensions\Kohaku-NAI\scripts\kohaku-nai.py", line 186, in run
raise Exception("Failed to generate image: " + str(failed_img_data))
Exception: Failed to generate image: Generation failed
Extension suddenly stopped working yesterday, tried -
Delete and reinstall extension
Downloaded another a1111 with fresh extension install
Did not work. NAI side generation is normal, regular txt2img in a1111 functions normally too.

https://github.com/KohakuBlueleaf/Kohaku-NAI/pull/10
update it.

Thanks. All green now.


Cannot see print statements in container log

Question

I cannot see my print() statements in the container log for my operator.  Here is an example:
def check_list_size(selectors):
    if len(selectors) > 5:
        print("Exceeded maximum selectors. You may only have up to 5 selectors in a Fargate profile.")
    else:

I've also tried using the python logging library.  That didn't work either.  All I see are the event/logs from the operator framework.
Checklist

[ x] I have read the documentation and searched there for the problem
[ x] I have searched in the GitHub Issues for similar questions

Keywords

logs print logging

I am also having an issue with try except statements.  It doesn't appear that kopf is honoring them.  It is also failing to detect the region.  I created another operator with an earlier version of kopf and didn't have this issues.


Remove Mnemonic in Chrome Extension in storeMnemonicAndSeed

Investigate if this should be done: I think we might want to remove the mnemonic from background.js in storeMnemonicAndSeed

Done in 0.2.5.


[MGDOBR:639]: Enum to comply with naming convention

MGDOBR-639
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] The layers in the kustomize folder have been updated accordingly.
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

@Aakanksha0311 What is the status of this? Can it be updated/merged or closed?

@Aakanksha0311 What is the status of this? Can it be updated/merged or closed?

I think it can be closed. Thanks for pinging.


[Rn] 2684 borderbox design fix

Description
fixed design of borderbox in exportwallet
About #2684

@sirpy everything looks good. snapshots are failing, if you're not planning to add more changes we could update snapshots and merge

please update snapshots and merge


Calendar creation

Description
In this branch I created the view for the calendar where a user(eventually an admin but at the moment anyone) can go on and view each day and how many people are allowed to be absent and how many people have requested it off. The user can modify how many people are allowed to be off and also see and indicator if that amount has been surpassed in the amount of requests.
Tests
Added selenium tests to test all the functionality of the javascripts to make sure they are operating as expected
UI Changes

Issues

Still need to make it so only admins can see the page
Will probably be bugs when we implement other features with it so watch out for that

Additional Notes
Add any other information about the PR here.

Two merge conflicts with forms/models and forms/tests


Composite Function in Google Cloud Service

Hi, I'm trying to perform a composite operation on an image that I am sending through a stream in GCS. I've already done some prior research on this, I'm aware that the composite function doesn't support streams as an input via issue #2200.
I'm considering, as a solution, converting my file input to a raw image buffer that can be read by the composite function before then turning it into a read stream that I can then send through the render output, performing any extra image transformations that I'd like to then. Any ideas on how I would go about doing that?
// Transformer sets the file attributes based on prior queries
  const transformer = sharp()
    .resize(resize)
    .modulate({
    brightness: brightnessVal
    })
    .toFormat(ext);

  // Loads the pictures from bucket
  const gcs = new Storage();
  let fileOne = gcs.bucket('bucketName').file(imageOneParameters.fileName);
  let fileTwo = gcs.bucket('bucketName').file(imageTwoParameters.fileName);

  //v DO THE COMPOSITE HERE? v
  // fileOne + fileTwo = compositeFile ???

  // Create a stream
  let stream = compositeFile.createReadStream()
  stream.on('error', function(err) {
    console.error(err);
    res.sendStatus(err.code).end(err);
  });

  // Add headers
  res.setHeader("content-type", "image/"+ext);
  res.setHeader("cache-control", "public, max-age=31536000, immutable");

  // Render
  stream.pipe(transformer).pipe(res);
});

@Oculus049 Did you see https://sharp.pixelplumbing.com/api-composite ? Were you able to make any progress with this?

Closing due to inactivity but please feel free to reopen with more details if further help is required.


rekey feature doesn't work with multitenancy

A rekey feature was added a few months ago and works for standalone agents. However, in multitenancy mode rekey-ing the base wallet causes the subwallets to get a AEAD decryption error error. I'm not sure of the relation between the base and subwallet keys that would be causing this.
I think supporting rekey for multitenancy should be supported and this problem fixed. It can essentially work as a key-rotate feature which is a security feature.
Also, it will help any old multitenant deployments (<0.0.12) that created their base wallets with a blank key to upgrade.

This is full stack trace for the error.
acapy-error-log.txt

I think I've found the cause of the problem: this error occurs when single_wallet_askar_manager opens a subwallet using it's profile config. As I already described, I didn't set the wallet.key parameter when creating the subwallet, so the key of the subwallet happens to be an empty string.
In acapy 1.0.1, when I use the rekey feature to change the base wallet's wallet key from DEFAULT_KEY, which is an empty string, to , and then call a subwallet/tenant API (e.g. GET /schemas), the profile config passes the key parameter to  instead of the empty string. (This is the key of the base wallet, not the key of the subwallet)
This is not the key that was used to create the subwallet and as a result askar throws an AEAD decryption error.

And the reason why the subwallet profile is that, in the get_wallet_profile() function, single_wallet_askar_manager  extands context.settings by setting sub_wallet_settings without wallet.key of the subwallet. see below:
https://github.com/openwallet-foundation/acapy/blob/06d1cf8f9d36fec2d2054ce6d81eba0f962867eb/aries_cloudagent/multitenant/single_wallet_askar_manager.py#L40-L90

I think this issue should be split into a couple of different issues.
Firstly, single_wallet_askar_manager.get_wallet_profile() should set the wallet.key of the subwallet.
Second, either the PUT /multitenancy/rekey API should be added to allow replacing the key of a subwallet that is already set incorrectly, or the PUT /multitenancy/wallet API should be modified to allow replacing the wallet.key.
Thirdly, POST /multitenancy/wallet should either generate a random wallet key from acapy if the subwallet is created in managed mode, or raise an error indicating that wallet.key is missing, as in unmanaged mode.


Digital independence (from Google & others)

Phone
Email (& calendar?)
Photos
Cloud storage / backups
Browser

"If you don't pay for a service, you're the product"

Related: #11

Solutions:

Perkeep
OwnCloud
Open source email?
Open source calendar?
etc.

Awesome list of alternatives for different use cases https://www.reddit.com/r/degoogle/comments/g1yu01/google_alternatives_huge_list_restore_your_privacy/

Google killed off phone recorder apps: https://nllapps.com/apps/acr/google-denies-phone-number-accesss.htm
Also:

Unfortunately Google Play Store policy prevents us from providing direct apk downloads from our website.

WTF

Google/Android removed yet another great use case
https://www.androidpolice.com/2019/08/12/sms-backup-is-now-broken-due-to-gmails-api-changes-but-theres-a-workaround/


Initial support for Virtual Desktop Workspace.

@abhilash-keloth For this effort, are you focused on adding support for just the azurerm_virtual_desktop_workspace type or are you considering a more encompassing module that creates a host pool, app group, VMs, etc. (similar to all the resources created by the Marketplace app - "Create Windows Virtual Desktop - Provision a host pool")?
I started stubbing out a module to provision a WVD host pool myself and am happy to contribute.

@abhilash-keloth For this effort, are you focused on adding support for just the azurerm_virtual_desktop_workspace type or are you considering a more encompassing module that creates a host pool, app group, VMs, etc. (similar to all the resources created by the Marketplace app - "Create Windows Virtual Desktop - Provision a host pool")?
I started stubbing out a module to provision a WVD host pool myself and am happy to contribute.

@jleonelion : Planning for all the examples including host pools and app group. I already have few in my branch akeloth-vdi. I am trying to add VMs to the host pool and it's bit tricky with extensions. Please share if you have anything in place for the same.

@jleonelion : Planning for all the examples including host pools and app group. I already have few in my branch akeloth-vdi. I am trying to add VMs to the host pool and it's bit tricky with extensions. Please share if you have anything in place for the same.

@abhilash-keloth Thanks for pointing out the branch.  I took a look at your additions. I did a little cleanup, but overall it like you've added these resource types to the module.  If I understand the design pattern here, next step would be to create vm_extensions types for Microsoft.Powershell.DSC and Microsoft.Compute.JsonADDomainExtension.  Then we put it all together in a landingzone example (which would go in a different GitHub project).  Is this your understanding?

@abhilash-keloth Thanks for pointing out the branch.  I took a look at your additions. I did a little cleanup, but overall it like you've added these resource types to the module.  If I understand the design pattern here, next step would be to create vm_extensions types for Microsoft.Powershell.DSC and Microsoft.Compute.JsonADDomainExtension.  Then we put it all together in a landingzone example (which would go in a different GitHub project).  Is this your understanding?

@abhilash-keloth Thanks for pointing out the branch. I took a look at your additions. I did a little cleanup, but overall it like you've added these resource types to the module. If I understand the design pattern here, next step would be to create vm_extensions types for Microsoft.Powershell.DSC and Microsoft.Compute.JsonADDomainExtension. Then we put it all together in a landingzone example (which would go in a different GitHub project). Is this your understanding?

@jleonelion : yes, both extensions for Domainjoin and session host are completed in my environment. It's pushed to my branch as well. We are in the midst of redesigning or finalizing the way to implement extensions.


Loading searchindexurl doesn't work with actual BaseURLs that have folders like https://abc.aiprm.com/AIPRM-XYZ/

Great job with the theme, thanks!
We needed to remove the artifical prefixing of the language code to make this work in search.html
to even use the default _relearn_searchindex.js from the assets folder of the theme
BaseURL is https://abc.aiprm.com/AIPRM-XYZ/
Site is multilingual
However the static files are only deployed 1 time, to https://abc.aiprm.com/AIPRM-XYZ/_relearn_searchindex.js
The special case that someone deploys the same static files to language-prefixed folders cannot be default.
Also it was constructed wrong, as it ended up in https://abc.aiprm.com/en/_relearn_searchindex.js
which doesn't even sit in the same home folder/vserver on the http
and the same for all other languages.
In a 2nd line of code we had the relLangURL, which again implies the special case being default - assets in language folders and enforces the current language prefixed
here's the diff and changed file attached, not sure how to commit/add directly to repo
diff --git a/layouts/partials/search.html b/layouts/partials/search.html
index 3c5acda17c..9b1c143f48 100644
--- a/layouts/partials/search.html
+++ b/layouts/partials/search.html
@@ -11,14 +11,14 @@
{{- warnf "UNSUPPORTED usage of 'searchpage' output format found, remove it from your config; see https://mcshelby.github.io/hugo-theme-relearn/introduction/releasenotes/7/#7-0-0" }}
{{- end }}
{{- if not .Site.Params.disableSearchIndex }}

     {{- $url := path.Join .Site.LanguagePrefix (trim (or .Site.Params.searchIndexURL "searchindex.js") "/") }}

     {{- $url := path.Join (trim (or .Site.Params.searchIndexURL "searchindex.js") "/") }}
     {{- if .IsHome }}
       {{- $templateres := resources.Get "_relearn_searchindex.js" }}
       {{- $resultres := $templateres | resources.ExecuteAsTemplate $url .Site.Home }}
       {{- /* the following code causes Hugo to generate our file - although it is in comments */}}<!-- {{ $resultres.RelPermalink }} -->
     {{- end }}
   <script>

     window.index_js_url={{ (printf "%s%s" $url $assetBusting) | relLangURL }};

     window.index_js_url={{ (printf "%s%s" $url $assetBusting) | relURL }};
   </script>
     {{- if not .Site.Params.disableSearchPage }}
       {{- with .Site.GetPage "_relearn_searchpage" }}

search.html.zip
If there's a use case where people have separate static asset folders, then that would be a separate mode to implement imho,
but the default theme should work out-of-the-box imho.
Hope this helps.

Ok found the PR feature :)

Thanks for the detailed bug report. Just for completeness, can you please post your hugo.toml. This helps in recreating a failing test before starting to fix and break for other configurations.

I can not replicate this with my configuration. Please provide your hugo.toml or even better, a link to your repo.
Nevertheless, I have a vague idea why this may go wrong with your setup.

Thanks, just saw your q for feedback and fix, didn't get to this sooner.


Added maintenance notice and links to the new JEA

This change is

Hi @rpsqrd, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Ryan Puffer). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Use one-dimensional Color arrays for custom effects

To improve performance and maintainability, we should see if the two-dimensional arrays in the custom (grid) effects can be replaced with one-dimensional arrays instead.

Implemented in #153


Add OSX, better way to present custom web controller, fix one bug

1 / Fix OAuth1 if callback url returned by service has no query (only fragment)
2 / Add compatibility with OSX, add demo project inspired from iOS one (many code to factorise)
3/ Normally a controller is not responsible to choose its parent controller as you do with the code
UIApplication.sharedApplication().keyWindow?.rootViewController?.presentViewController(
                        self.webViewController!, animated: true, completion: nil)

A parent controller, not only the root one, could do it (moreover this root controller doesn't exist in OSX)
Maybe the web view controller could be in a storyboard, a segue must be performed to show it (with maybe a custom animation)
Maybe a web view is already present in the application, and no web view must be shown, only and url update of the current one
That's why

I rename the protocol WebViewProtocol to OAuthSwiftURLHandlerType, move to its own file,
Replace in OAuth objects the property webViewController by an authorize_url_handler: OAuthSwiftURLHandlerType
Make a default implementation to open URL externally has you do if there is no webViewController assigned to the OAuth object

In consequence, no more UIKit(or AppKit for OSX) reference to the core framework
Users of framework can make their own code to show the web view which handle the url
OAuthWebViewController can always be extended with default behaviour (this object could be excluded from core framework and be part of a submodule in podspec)

This would be nice, i really need to make the web view under my navigation controller

in the mean time, someone knows how to do it manually right now ? im kinda of a newbi..

Hi @phimage
Thank you!
https://github.com/dongri/OAuthSwift/commit/43a05b2c7c59b01a12e0cfe0a688cc64e6cfd931

you are welcome
sorry for the "key" instead of "oauth_token" (just testing one website api...)


fix: fix segments display crashing (#2770)

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
This fixes issues with segments retrieval crashing when a trait value type unsupported by a segment operator is stored.
The new Flagsmith engine version performs necessary runtime checks to avoid this.
How did you test this code?
Relevant test cases have been added to Flagsmith engine's test suite.

Codecov Report
Patch and project coverage have no change.

Comparison is base (fce2e3a) 95.53% compared to head (cabec54) 95.53%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2789   +/-   ##
=======================================
  Coverage   95.53%   95.53%
=======================================
  Files         994      994
  Lines       28072    28072
=======================================
  Hits        26818    26818
  Misses       1254     1254

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


nuclei: 2.7.8 -> 2.7.9

Description of changes
https://github.com/projectdiscovery/nuclei/releases/tag/v2.7.9

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 200170 run on x86_64-linux 1

1 package built:

nuclei


better interpolate inner/outer scope resolve, more tests

Coverage increased (+0.5%) to 68.033% when pulling 5b26be40bf667023097003868b15f7b3724ba7e8 on interpolate-up into c6036b54f5e6dde5682c13649f23673a72cda520 on master.


Add whitepaper reviews

Adds whitepaper reviews from the monero-downloads repository and updates links. The documents still exist in monero-downloads to avoid redirects.

Covered by #1109 already!


⚠ Bofrost.be has degraded performance

In 335cff8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1914 ms

Resolved: Bofrost.be performance has improved in 6f88500.


option to start fetch when listeners are attached

lets say i am downloading file A, then i close the service, the file will have a status of DOWNLOADING.
then i start the download service, fetch will continue downloading the file but in this case i some callbacks will not be called since i attach the listener after Fetch.Impl.getInstance(fetchConfiguration);
there should be an option where fetch will autostart once a listener has been attached so that we capture all information from the callbacks

@duncanokeyo looking into this

@duncanokeyo  just released a new version of Fetch. 2.2.0-RC7 with loads of fixes. Please let me know if it fixes this issue for you. Note there are breaking changes in this release. Please see release notes first.

fetch 2.2.0-RC8 has been released with fixes

works as expected..


Events keep on propagating to the DOM event with autoPreventDefault=true

First/Main Issue
As far as I guessed, autoPreventDefault=true should prevent events caught by the PIXI Canvas to propagate further on default, but it doesn't work
For example, I click an in-game NPC, an HTML dialog opens, the click hits that new HTML dialog too, pretty bizarre

Second/Curious Issue - Not directly related
Second issue that I just noticed, when you put a small canvas, on top of a full page canvas, both canvases capture all mouse move events, even though the move is outside the smaller canvas
I wanted only the small canvas to capture the move event, stop propagation, so I can detect which canvas the mouse is on, doesn't work, fails miserably
Second issue just made me realise I still have no idea how PIXI interaction works
I can patch the first issue manually, just wanted to report my experience, I've checked the docs, but as far as I see, they don't cover the interactivity logic, I've re-inspected the sources, got no leads from there either, and created this issue later on

As far as I guessed, autoPreventDefault=true should prevent events caught by the PIXI Canvas to propagate further on default, but it doesn't work

event.preventDefault() prevents the default action a browser would take when that event occurs. It does not stop the propagation of the event. That is what event.stopPropagation() is for.


Can we leverage any Data Source for Glue Tables behind the scenes?

Is it possible that we can leverage any of the sources defined by Glue?
https://docs.aws.amazon.com/glue/latest/dg/populate-data-catalog.html
I.e. can we make an RDS DB Table available in the catalog through Glue?
If not, what needs to happen to tap into this Glue abstraction in Data.All?

Hi @pascalwhoop, at the moment data must be stored in S3. A data.all Dataset consists of both S3 bucket + Glue database. We use S3 because is the center-piece of AWS data lakes connecting multiple data sources:

For the specific case of RDS, you could implement something like the step 3 of this blog: https://aws.amazon.com/blogs/big-data/integrating-aws-lake-formation-with-amazon-rds-for-sql-server/. The other steps would be handled by data.all creation or import of datasets.
But, we are open to direct storage in other data sources. it is definitely something that raises interest. So, to the question: "what needs to happen to tap into this Glue abstraction in Data.All?"
Assumptions:

data is already stored in RDS DB in one of the environment AWS accounts
WE DO NOT HANDLE TABLE SHARING

User experience:
As a data.all user with access to the RDS environment account and the RDS DB

I click on import dataset and specify the storage type as RDS and I give my RDS database as input
When the data.all object has been created I can click on crawl dataset and sync tables. They are now available in the catalog

Implementation:

Modification of "Import dataset form" UI  to add new input parameters (type of storage)
Modification of input of API and API import dataset code
Modification of RDS metadata database and the table for datasets
Modification of dataset stack (in cdkproxy)
Modification of crawler to allow RDS: https://docs.aws.amazon.com/glue/latest/dg/crawler-data-stores.html

Keep in mind that those are drafted steps and could of course, change. Maybe you can define the assumptions and user experience and we can refine the implementation steps.

Hi @dlpzx thx for paying attention to this.
I think your user experience is pretty spot-on.
| As a use case owner with an RDS hosted in my account, I want to expose some of my tables as data products to other teams easily
Particularly the experience would be

create new dataset in the catalog
select my environment
select my database
create dataset
wait for objects to be spun up
ability to select schema / tables to expose

This is a separate topic but I think it may also be a good UX if one gets a "waiting screen" within the same journey that lets me wait for the data.all objects instead of breaking the journey into 1) create dataset 2) add data to it.
Assumptions wise, I am not deep enough in the tech to understand what the RDS needs to be configured like for the glue crawler to be able to access the DB. I suppose there is some form of credentials handling that needs to happen?


Update Zerl version and client build system

This should make things Just WorkTM for the most part. It also eliminates some kludges and generally streamlines things.
How to test: run just client inside the flake. Maybe run the server as well and connect to it if you're feeling particularly paranoid.

I would like to have a list of things in the PR description that I should test locally to verify all the changes. Is just client enough within Nix or is something else needed?

That should be enough, but you can also run the backend and try connecting to it if you're feeling paranoid. (I updated the PR description as well.)


Created KeyboardManipulator tests for SetInterval, ResetInterval, and ResetIntervals

In order to allow for the KeyboardManipulator tests to access intervals, an enumerable key was created KeyPressIntervals in KeyboardManipulator.cs and KeyboardIntervalInterceptor.cs, allowing the KeyboardManipulator class access to the key press intervals from KeyboardIntervalInterceptor.cs. All of the 3 new tests have passed, with all 5 previous tests still passing as well

Hey @AdielRoyt !
I'm glad you are interested in this project :)
Great idea to create a collection with intervals! I think this collection could be converted into a dictionary, so that in addition to the keys, users could also get the interval set for them, maybe some users would find this very useful!
Also, we can improve the tests, and add a check for SetInterval(key, TimeSpan.Zero) - when using this method, if an interval has been applied to a key it will be removed, this call is analogous to ResetInterval(key)
I also propose to put the interval tests in a separate class, which could be called KeyboardManipulatorIntervalTests
I accept this PR, but if you have a desire to improve and add what I described above, you can open a new PR and make those changes. Thanks for your contribution! :)

You are absolutely welcome :) I'll take your feedback into consideration.


Getting started outdated version

I wasted around 2 hours wondering why it's not working when I did everything like in this tutorial:
https://socket.io/get-started/chat/
I downloaded code from github. Copied everything from index.js - not working, everything from index.html - not working. Then I look to package.json - damn, that's some old one. Finally when I copied package.json and reinstalled it started working.
But I can't get it going with newest version because... tutorial is just shi**y, sorry

Same here...
Thanks for the package.json idea

I realized something- the source courde on github included outdated client version of socket.io, that's why it wasn't working for newest package.json
But it still misleading anyway

Closed by https://github.com/socketio/chat-example/commit/c16c98540d43c4ad5033aadf7cc2df468f1b9652

Closed by https://github.com/socketio/chat-example/commit/c16c98540d43c4ad5033aadf7cc2df468f1b9652


Long timeout

I need check my service is ok or not。 sometimes it will breakdown.
how to reproduce the issue
 try:
    r = requests.get("http://xxx.com/",timeout=1)
    print(r.status_code)
except requests.exceptions.Timeout:
	print 'timeout'
except requests.exceptions.RequestException as e:
    print(e)

it always let me wait for 30 seconds
I hope it only waits one second, if server doesn't response or the link is unreachable, throw exception
below are exception message

Max retries exceeded with url: / (Caused by NewConnectionError('<requests.packages.urllib3.connection.HTTPConnection object at 0x3d344d0>: Failed to establish a new connection: [Errno 8] nodename nor servname provided, or not known',))

The requests timeout parameter is per connection, and Requests will make as many connection attempts as there are domain names. However, Requests cannot control the timeout of the DNS lookup, which is the problem here. Unfortunately, Python doesn't allow us any way around this problem: there is just no cross-platform way to do a DNS lookup with a timeout using the things installed by default.
Sorry!


blog-example crashing

When trying to view the example it's crashing.
I'm sure this info isn't important but:
OS: Windows 8.1
Browser: Chrome 43.0.2357.130 m

It's probably cause of this package vsivsi:file-collection, which also makes it crash when I deploy to modulus.

There is a known issue for Modulus deploys, but I didn't know it broke on meteor.com as well. It shouldn't.

@rgoomar Rishi, you deployed that site. I can't get to the logs, or attempt to restart it. Can you take a look?

The file-collection package shouldn't cause any problems when deployed to meteor.com. If you have evidence that it does, please file an issue with a reproducing repo and I'll look into it.
@hellogerard  Has reported issues deploying an app using file-collection with demeteorizer, but last I heard that was an issue with demeteorizer not respecting that different atmosphere packages may require different, incompatible npm dependencies. If so, that's definitely a demeteorizer bug.  I've deployed file-collection apps using Meteor-up (mup) without difficulty.

@Neobii  The issue using file-collection on Modulus is a bug in how demeteorizer handles npm dependencies.  https://github.com/onmodulus/demeteorizer/issues/125

Here are the logs from the blog example:
[Fri Jul 03 2015 14:22:22 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 14:22:27 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 14:22:27 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 14:22:37 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:22:43 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:22:44 GMT+0000 (UTC)] INFO HIT /favicon.ico <IP_ADDRESS>
[Fri Jul 03 2015 14:22:50 GMT+0000 (UTC)] INFO HIT /favicon.ico <IP_ADDRESS>
[Fri Jul 03 2015 14:22:54 GMT+0000 (UTC)] NOTICE Starting application on port 13309
[Fri Jul 03 2015 14:22:54 GMT+0000 (UTC)] INFO STATUS waiting -> starting
[Fri Jul 03 2015 14:22:54 GMT+0000 (UTC)] INFO STATUS starting -> running
[Fri Jul 03 2015 14:22:55 GMT+0000 (UTC)] WARNING
/meteor/dev_bundles/0.4.18/lib/node_modules/fibers/future.js:245
throw(ex);
^
[Fri Jul 03 2015 14:22:55 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 14:22:56 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 14:22:56 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 14:23:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:23:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:24:07 GMT+0000 (UTC)] INFO STATUS waiting -> starting
[Fri Jul 03 2015 14:24:07 GMT+0000 (UTC)] NOTICE Starting application on port 13309
[Fri Jul 03 2015 14:24:07 GMT+0000 (UTC)] INFO STATUS starting -> running
[Fri Jul 03 2015 14:24:18 GMT+0000 (UTC)] WARNING /meteor/dev_bundles/0.4.18/lib/node_modules/fibers/future.js:245
throw(ex);
^
[Fri Jul 03 2015 14:24:18 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 14:24:18 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 14:24:18 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 14:24:47 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:24:53 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:24:53 GMT+0000 (UTC)] INFO HIT /favicon.ico <IP_ADDRESS>
[Fri Jul 03 2015 14:24:59 GMT+0000 (UTC)] INFO HIT /favicon.ico <IP_ADDRESS>
[Fri Jul 03 2015 14:26:52 GMT+0000 (UTC)] NOTICE Starting application on port 13309
[Fri Jul 03 2015 14:26:52 GMT+0000 (UTC)] INFO STATUS waiting -> starting
[Fri Jul 03 2015 14:26:52 GMT+0000 (UTC)] INFO STATUS starting -> running
[Fri Jul 03 2015 14:26:54 GMT+0000 (UTC)] WARNING /meteor/dev_bundles/0.4.18/lib/node_modules/fibers/future.js:245
throw(ex);
^
[Fri Jul 03 2015 14:26:54 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 14:26:59 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 14:26:59 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 14:28:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:28:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:32:34 GMT+0000 (UTC)] NOTICE Starting application on port 13309
[Fri Jul 03 2015 14:32:34 GMT+0000 (UTC)] INFO STATUS waiting -> starting
[Fri Jul 03 2015 14:32:35 GMT+0000 (UTC)] INFO STATUS starting -> running
[Fri Jul 03 2015 14:32:36 GMT+0000 (UTC)] WARNING                         throw(ex);
^
[Fri Jul 03 2015 14:32:36 GMT+0000 (UTC)] WARNING
/meteor/dev_bundles/0.4.18/lib/node_modules/fibers/future.js:245
[Fri Jul 03 2015 14:32:36 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 14:32:36 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 14:32:36 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 14:33:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:33:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:38:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:38:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:43:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:43:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:44:26 GMT+0000 (UTC)] NOTICE Starting application on port 13309
[Fri Jul 03 2015 14:44:26 GMT+0000 (UTC)] INFO STATUS waiting -> starting
[Fri Jul 03 2015 14:44:26 GMT+0000 (UTC)] INFO STATUS starting -> running
[Fri Jul 03 2015 14:44:27 GMT+0000 (UTC)] WARNING      ^
[Fri Jul 03 2015 14:44:27 GMT+0000 (UTC)] WARNING                         throw(ex);
[Fri Jul 03 2015 14:44:27 GMT+0000 (UTC)] WARNING /meteor/dev_bundles/0.4.18/lib/node_modules/fibers/future.js:245
[Fri Jul 03 2015 14:44:28 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 14:44:28 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 14:44:28 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 14:48:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:48:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:53:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:53:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:58:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 14:58:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:03:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:03:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:06:40 GMT+0000 (UTC)] NOTICE Starting application on port 13309
[Fri Jul 03 2015 15:06:40 GMT+0000 (UTC)] INFO STATUS waiting -> starting
[Fri Jul 03 2015 15:06:40 GMT+0000 (UTC)] INFO STATUS starting -> running
[Fri Jul 03 2015 15:06:46 GMT+0000 (UTC)] WARNING /meteor/dev_bundles/0.4.18/lib/node_modules/fibers/future.js:245
throw(ex);
^
[Fri Jul 03 2015 15:06:46 GMT+0000 (UTC)] WARNING TypeError: Cannot call method 'indexOf' of undefined
at __coffeescriptShare (packages/ryw:blog/server/boot.coffee:55:13)
at /meteor/containers/945f1b1c-5cda-b9b7-4e2a-9719485b268b/bundle/programs/server/boot.js:229:5
[Fri Jul 03 2015 15:06:51 GMT+0000 (UTC)] ERROR Application crashed with code: 8
[Fri Jul 03 2015 15:06:51 GMT+0000 (UTC)] INFO STATUS running -> waiting
[Fri Jul 03 2015 15:08:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:08:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:13:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:13:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:18:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:18:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:23:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:23:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:28:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:28:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:33:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:33:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:38:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:38:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:43:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:43:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:48:02 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:48:08 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:49:31 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:49:37 GMT+0000 (UTC)] INFO HIT / <IP_ADDRESS>
[Fri Jul 03 2015 15:49:37 GMT+0000 (UTC)] INFO HIT /favicon.ico <IP_ADDRESS>
[Fri Jul 03 2015 15:49:43 GMT+0000 (UTC)] INFO HIT /favicon.ico <IP_ADDRESS>

Seems to be this line that is causing the issue.

It should be working now


Add newrelic to requirements

Coverage remained the same at 97.411% when pulling 31ba0cb320d9355d5bcc02df04652552f20cafab on staging into 313026a7fef2e6a889b6832486ffe068b265cb7f on master.


The Qt example seems affected by race conditions

Hi,
I honestly just glanced the code, but the Qt example seems to invoking UI methods from the socket callbacks, which (I guess) are executed from an ASIO thread, not the main Qt one.
https://github.com/socketio/socket.io-client-cpp/blob/9663ca3c64a79c5bbd9f403fddd8a82e70f6f589/examples/QT/SioChatDemo/mainwindow.cpp#L41-43 should use Qt::QueuedConnection, otherwise the connected slots will be executed in line in the current thread.

Thanks for reporting, I'm just start to use QT for this demo, what you guess about the thread is true, and the purpose I used Signals/Slots is to invoke the UI operations in UI thread. It seems I choose a wrong way?   I'll try to verify.

Hi, @lultimouomo I've test by set breaking point at OnNewMessage() (which is triggered by signal from asio thread) and OnMessageReturn()(which is triggered by return key, absolutely in UI thread)
I found OnNewMessage() executes in the same thread with OnMessageReturn(),
which means the signals/slots mechanism WILL post the invocation to the UI thread.

You're right, I thought the connection type (i.e. whether to execute the slot in line or post an event on the loop) was chosen at connection time, according to the thread affinity of the sender and the receiver.
http://doc.qt.io/qt-5/qt.html#ConnectionType-enum states instead that it's chosen when the signal is actually emitted, so the code is OK.
I would consider indicating Qt::QueuedConnection explicitely anyway, since it makes the code intention more clear.

Hi, @lultimouomo I think indicating Qt::QueuedConnection explicitly is not the good choice, AutoConnection is better than QueuedConnection in this case because If you did signal AddListItem in UI thread somewhere, inline execution is preferred,  so you can indicating Qt::AutoConnection explicitly instead.


deviantart error

Ripme version: 1.7.90

Java version: 1.8.0_221

Operating system: win7 Microsoft Windows [Version 6.1.7601]

Exact URL you were trying to rip when the problem occurred: https://www.deviantart.com/enginetrap/gallery/

Please include any additional information about how to reproduce the problem:
When I try to rip a deviantart it shows "deviantart album detected" but when rip started it says "Error: Read timed out"

Expected Behavior
Downloading ...
...

For me, RM is the best ripper YES. It was. Will be. Now there is no, and there are no alternatives either.


fix a scoping issue caused by RxSwift 3.0

Sorry about this @sunshinejr! Updating to 3.0.0 moved Reactive from RxCocoa to RxSwift, so it broke this.
I should have checked to make sure RxPermission compiled with the new RxSwift before submitting a PR 😅

No problem, I also didn't have time to check :< Hope it is now working, huh! Thanks 👊


make sure audience in jwt access token contains client id

While trying to get OIDC login to Rancher working with CAS, their OIDC library wanted the access token in JWT format and it wanted the audience(s) in the token to contain the client id, but CAS was returning something else. Per the specification CAS could return a list of audiences but if it is only returning one, it needs to be the client id.
https://openid.net/specs/openid-connect-core-1_0.html#rfc.section.2
aud:
REQUIRED. Audience(s) that this ID Token is intended for. It MUST contain the OAuth 2.0 client_id of the Relying Party as an audience value. It MAY also contain identifiers for other audiences. In the general case, the aud value is an array of case sensitive strings. In the common special case when there is one audience, the aud value MAY be a single case sensitive string.
This was oidc library code:
https://github.com/coreos/go-oidc/blob/v3/oidc/verify.go#L273

 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.


Add allcontributors setup

[x] Closes #371

I saw this at the SciPy conference and I liked the concept. I tried my best to recognize contributions, but probably missed some. If I did, please let me know. Also, if you would prefer not to be recognized on the README, please let me know.
Tag:
@jswhit @micahcochran @jorisvandenbossche @cjmayo @cffk @zippy1981 @jdkloe @georgeouzou @djhoese @mitkin @dopplershift @artttt @ocefpaf @heitorPB @sebastic @nickeubank @mdunphy @matthew-brett @jdemaeyer @bmwiedemann @ReallyNiceGuy @ChrisBarker-NOAA @kbevers @rouault @cgohlke @chrrrisw

So does the bot automatically update the README for this service?

So does the bot automatically update the README for this service?

From what I can tell from the docs, it will create a PR when you ping it.


add back fanv1 service

the Home app doesn't let you customize the icon for a FanV2 (i.e. to show a ceiling fan).
I'm not sure if I'm ready to merge this yet. I've got it plumbed through to OpenHAB as well https://github.com/ccutrer/openhab-addons/commit/114fe0625d226098504a89243e763049abc53c42. When I tested, the accessory works as expected, EXCEPT that on iOS 16, the new Home refuses to show the fan anywhere except for if you go to the hub and list its accessories. Or if it's on it does show on the top status line of a room. This feels like an iOS bug to me. So I'm going to let this hang out here until the next iOS update comes, and test again.

@yfre Okay... I want to merge this now. Apparently iOS doesn't like when a basic fan isn't 0-100%, but is fine with a FanV2 being 0-4 speeds.  Restoring it to be on a Dimmer item it's working fine in iOS.

@yfre Okay... I want to merge this now. Apparently iOS doesn't like when a basic fan isn't 0-100%, but is fine with a FanV2 being 0-4 speeds. Restoring it to be on a Dimmer item it's working fine in iOS.

interesting.
yes, the PR looks for me. will merge it now.


BUG! TypeError: font.finalize is not a function

I don't know why, but this morning, I encounter a strange bug with pdfkit .end() method.
When I call doc.end();, My app crash and give me this stacktrace:

TypeError: font.finalize is not a function
at PDFDocument.end (C:\Users\James\Desktop\c4t\server\node_modules\pdfkit\js\document.js:202:14)
at app.get (C:\Users\James\Desktop\c4t\server\routes\pdf.js:8:13)
at Layer.handle [as handle_request] (C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\layer.js:95:5)
at next (C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\route.js:137:13)
at Route.dispatch (C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\route.js:112:3)
at Layer.handle [as handle_request] (C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\layer.js:95:5)
at C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\index.js:281:22
at Function.process_params (C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\index.js:335:12)
at next (C:\Users\James\Desktop\c4t\server\node_modules\express\lib\router\index.js:275:10)
at module.exports (C:\Users\James\Desktop\c4t\server\tools\i18n.js:16:5)

I try to encapsulate method to check where it's can crash.
app.get("/test", function(req, res) {         var doc = new PDFDocument();         doc.end();  });
This crash too. I think the problem come from pdfkit in node_modules. I try to reinstall it without success.
Thanks!
James

I figure out that was a global method I have created that crash pdfkit. I modify my global method and everything was ok. My bad.

I figure out that was a global method I have created that crash pdfkit. I modify my global method and everything was ok. My bad.

Can you elaborate? I have a similar error.

Check in your app.js. It's a long time ago, but I remember that a specific var in your app.js is already used by pdfkit. Just rename all your globals var in your app.js and you will see your app will works.


Custom prompt ordering

There should be an option for custom prompt ordering, like:
SPACESHIP_PROMPT_ORDER=(
  user
  host
  dir
  git
  hg
  # so on
)

A useful feature for those, who wants to prompt character at the beginning, time before prompt character and so on.
Probable implementation:
spaceship_prompt() {
  # ...

  for segment in $SPACESHIP_PROMPT_ORDER; do
    spaceship_$segment
  done

  # ...
}

Working on it. PR soon.

I hope custom prompt will have some rules about ordering, right ?
Because most of the prompts are related with through the prefixes like on, via, at, in . So, Is it a good idea to change those rules?

@sfarzy not sure what you're talking about

Most of the prompt sections are relatively ordered. Like,
root at ~/Dotfiles on master via ⬢v4.6.1

What happens if the user wants  ordering like ( git dir user ). Leading to a prompt like,
on master at ~/Dotfiles root

This seems bit odd. Some constraints to prevent this might help.

@sfarzy yeah, also thought about it.
I don't think that constraints are good for us. I believe users need to have freedom of choice which sections they want to use. We shouldn't force them to see user only at the beginning of the prompt, for example.
We have a PREFIXes options for disabling/enabling prefixes. We could also add SUFIXes, if this would be needed. It's easy to set the function as a value of this options, so users can define their own logic of showing prefixes (I think we need to add a note about it to make users aware of that feature).

Also, could be applied to git prompt:
indicators+="$(spaceship_git_uncomitted)"
indicators+="$(spaceship_git_unstaged)"
indicators+="$(spaceship_git_untracked)"
indicators+="$(spaceship_git_stashed)"
indicators+="$(spaceship_git_unpushed_unpulled)"

@sfarzy what do you think about merging user and host segments into single one called context, for example? That may solve a problem with confusing ordering.

That seems reasonable.  host segment already uses the spaceship_user function.

Okay then. Gonna merge these parts.

Closed by #78