Fix RCTFatalException in RCTFatal, line 133

@artsyit commented on Sun Jun 12 2016
Version: 2.5.0 (2016.06.03) | net.artsy.artsy
Stacktrace
-;RCTFatal;RCTAssert.m;133
RCTExceptionsManager;reportFatalException:stack:exceptionId:;RCTExceptionsManager.m;64
RCTModuleMethod;invokeWithBridge:module:arguments:;RCTModuleMethod.m;489
RCTBatchedBridge;_handleRequestNumber:moduleID:methodID:params:;RCTBatchedBridge.m;968
RCTBatchedBridge;handleBuffer:;RCTBatchedBridge.m;896
Reason
*** Terminating app due to uncaught exception 'RCTFatalException: Unhandled JS Exception: null is not an object (evaluating 'e._currentElement')', reason: 'Unhandled JS Exception: null is not an object (evaluating 'e._currentElemen..., stack:
unmountComponent@273:309
unmountComponent@293:1821
unmountComponent@273:345
unmountChildren@336:680
unmountChildren@335:2366
unmountComponent@334:399
unmountComponent@273:345
unmountChildren@336:680
unmountChildren@335:2366
unmountComponent@334:399
unmountComponent@273:345
unmountChildren@336:680
unmountChildren@335:2366
unmountComponent@334:399
unmountComponent@273:345
unmountChildren@336:680
unmountChildren@335:2366
unmountComponent@334:399
unmountComponent@273:345
unmountComponent@293:1821
unmountComponent@273:345
unmountComponent@293:1821
unmountComponent@273:345
unmountComponent@293:1821
unmountComponent@273:345
unmountChildren@336:680
unmountChildren@335:2366
unmountComponent@334:399
unmountComponent@273:345
unmountChildren@336:680
unmountChildren@335:2366
unmountComponent@334:399
unmountComponent@273:345
unmountComponent@293:1821
unmountComponent@273:345
unmountComponent@293:1821
unmountComponent@273:345
updateChildren@336:462
_reconcilerUpdateChildren@335:1023
_updateChildren@335:1919
updateChildren@335:1784
receiveComponent@334:1063
receiveComponent@273:503
updateChildren@336:419
_reconcilerUpdateChildren@335:1023
_updateChildren@335:1919
updateChildren@335:1784
receiveComponent@334:1063
receiveComponent@273:503
_updateRenderedComponent@293:4623
_performComponentUpdate@293:4392
updateComponent@293:3669
receiveComponent@293:2948
receiveComponent@273:503
updateChildren@336:419
_reconcilerUpdateChildren@335:1023
_updateChildren@335:1919
updateChildren@335:1784
receiveComponent@334:1063
receiveComponent@273:503
updateChildren@336:419
_reconcilerUpdateChildren@335:1023
_updateChildren@335:1919
updateChildren@335:1784
receiveComponent@334:1063
receiveComponent@273:503
updateChildren@336:419
_reconcilerUpdateChildren@335:1023
_updateChildren@335:1919
updateChildren@335:1784
receiveComponent@334:1063
receiveComponent@273:503
_updateRenderedComponent@293:4623
_performComponentUpdate@293:4392
updateComponent@293:3669
receiveComponent@293:2948
receiveComponent@273:503
_updateRenderedComponent@293:4623
_performComponentUpdate@293:4392
updateComponent@293:3669
performUpdateIfNecessary@293:3190
performUpdateIfNecessary@273:665
r@271:570
perform@276:494
perform@276:494
perform@271:1534
P@271:1706
closeAll@276:1131
perform@276:581
batchedUpdates@270:453
s@271:760
a@290:69
enqueueSetState@290:915
setState@78:317
value@14:1184
a@258:71
o@256:391
a@256:582
f@254:159
g@254:262
i@260:102
processEventQueue@254:1382
s@304:92
handleTopLevel@304:181
_receiveRootNodeIDEvent@303:613
receiveEvent@303:714
value@198:3100
@198:1319
k@198:413
value@198:1291
'
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/37029/crash_reasons/124351016

Alright so the stack trace doesn't explicitly say the Claudio Parmiggiani crash happened on the Shows tab, but that was the only way I could reproduce a crash for that particular artist. I am wondering if it's possible that the user tapped Shows and the crash happened before we could get that information.
If so, this would be solved by #186, because that artist had a couple of shows (actually fair booths) without partners.

Yes that is possible.
Is it this same crash stack trace? The other 2 crashes happen with barry-mcgee, do you reproduce it in the same way with him?

Should be fixed by #186 ; Barry Mcgee issues probably related to #184


MissionData issue with the Letter a

The MissionData file that was hex edited to test insertion uncovered a bug with the letter a, it is replace by a numbers instead of staying the letter a. It is not always the same number than the exemple shown bellow.

Looking at the original japanese uncovered the behavior of the letter a. They use a% in the original file for numbers. for exemple.

fixed


upgrade bouncycastle due to cve

ISSUE TYPE

Bug Report

https://github.com/pjfanning/cloudstack/security/dependabot/pom.xml/org.bouncycastle:bcprov-jdk15on/open

Fixed in https://github.com/apache/cloudstack/pull/5949
thanks for the report and PR @pjfanning


Error when subprocess command fails in foundry_install script

Refactor the subprocess calls in the common/foundry_install.py script so that if a call to a sub-command fails (such as magic segfaulting shown in #183) the whole script will fail.
When combined with #186, the build process will now terminate if the magic or other sub-command errors.
See example @ https://github.com/mithro/open_pdks/runs/4304473902?check_suite_focus=true and below;
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): Cell "sky130_fd_sc_hd__decap_6" is used but not defined in this file.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): cell sky130_fd_sc_hd__decap_6 was used but not defined.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): cell sky130_fd_sc_hd__fill_2 was used but not defined.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): cell sky130_fd_sc_hd__fill_4 was used but not defined.
File sky130_fd_sc_hd__decap_6 contained format error
"sky130_fd_sc_hd__decap_6" has a zero timestamp; it should be written out
    to establish a correct timestamp.
freeMagic called with NULL argument.
Traceback (most recent call last):
  File "../common/foundry_install.py", line 1648, in <module>
    cwd = destlibdir,
  File "../common/foundry_install.py", line 256, in subprocess_run
    raise SystemError(emsg)
SystemError: Command magic failed with exit code: -11
  magic -dnull -noconsole
make[3]: *** [Makefile:1048: digital-hd-a] Error 1
make[2]: *** [Makefile:856: vendor-a] Error 2
make[1]: *** [Makefile:583: all-a] Error 2
make[3]: Leaving directory '/home/runner/work/open_pdks/open_pdks/sky130'
make[2]: Leaving directory '/home/runner/work/open_pdks/open_pdks/sky130'
make[1]: Leaving directory '/home/runner/work/open_pdks/open_pdks/sky130'
make: *** [Makefile:58: build-pdk] Error 2
Error: Process completed with exit code 2.

Made this a draft pull request as there are still a few small things I would like to add to this pull request before merging, but wanted @RTimothyEdwards to start reviewing it.

I changed the error output to include the data provided on stdin (as there doesn't seem to be any other way to get that data), do you think that is a good idea?
See example at https://github.com/RTimothyEdwards/open_pdks/runs/4304630430?check_suite_focus=true
Writing all magic database files
Writing 'sky130_fd_sc_hd__decap_12'
Writing 'sky130_ef_sc_hd__fill_4'
Writing 'sky130_fd_sc_hd__decap_6'
Writing 'sky130_ef_sc_hd__fill_8'
Processing timestamp mismatches: sky130_fd_sc_hd__decap_6.
Writing 'sky130_fd_sc_hd__clkdlybuf4s15_1'
Writing 'sky130_fd_sc_hd__buf_16'
Writing 'sky130_fd_sc_hd__fill_2'
Error message output from magic script:
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): Cell "sky130_fd_sc_hd__fill_4" is used but not defined in this file.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): Cell "sky130_fd_sc_hd__fill_2" is used but not defined in this file.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): Cell "sky130_fd_sc_hd__decap_6" is used but not defined in this file.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): cell sky130_fd_sc_hd__decap_6 was used but not defined.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): cell sky130_fd_sc_hd__fill_2 was used but not defined.
Error while reading cell "sky130_ef_sc_hd__fill_12" (byte position 372): cell sky130_fd_sc_hd__fill_4 was used but not defined.
File sky130_fd_sc_hd__decap_6 contained format error
"sky130_fd_sc_hd__decap_6" has a zero timestamp; it should be written out
    to establish a correct timestamp.
freeMagic called with NULL argument.
File sky130_fd_sc_hd__fill_2 contained format error
"sky130_fd_sc_hd__fill_2" has a zero timestamp; it should be written out
    to establish a correct timestamp.
make[3]: *** [Makefile:1048: digital-hd-a] Error 1
make[2]: *** [Makefile:856: vendor-a] Error 2
make[1]: *** [Makefile:583: all-a] Error 2
make: *** [Makefile:58: build-pdk] Error 2
Traceback (most recent call last):
  File "../common/foundry_install.py", line 1659, in <module>
    cwd = destlibdir,
  File "../common/foundry_install.py", line 267, in subprocess_run
    raise SystemError("".join(emsg))
SystemError: Command magic failed with exit code: -11
  magic -dnull -noconsole
Input script was:
---------------------------------------------------------------------------
#!/usr/bin/env wish
#--------------------------------------------
# Script to generate .mag files from .gds
#--------------------------------------------
crashbackups stop
drc off
gds readonly true
gds flatten true
gds rescale false
tech unlock *
# Set the GDS input style to sky130(vendor).  This treats labels on the
# TXT purpose (5) as pins, which is unfortunately done in a lot of the
# vendor GDS files.
cif istyle sky130(vendor)
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__fill_12.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__fill_4.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_fd_sc_hd__clkdlybuf4s18_1.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_fd_sc_hd__a2111oi_0.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__decap_12.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_fd_sc_hd__buf_16.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_fd_sc_hd__clkdlybuf4s15_1.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__fill_8.gds
gds read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/gds/sky130_ef_sc_hd__fakediode_2.gds
puts stdout "Annotating cells from LEF"
lef read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/lef/sky130_ef_sc_hd__fill_12.lef
lef read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/lef/sky130_ef_sc_hd__decap_12.lef
lef read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/lef/sky130_ef_sc_hd__fakediode_2.lef
lef read /home/runner/work/open_pdks/open_pdks/sky130/sky130A/libs.ref/sky130_fd_sc_hd/lef/sky130_ef_sc_hd__fill_8.lef
puts stdout "Writing all magic database files"
writeall force
puts stdout "Done."
quit -noprompt

---------------------------------------------------------------------------

make[3]: Leaving directory '/home/runner/work/open_pdks/open_pdks/sky130'
make[2]: Leaving directory '/home/runner/work/open_pdks/open_pdks/sky130'
make[1]: Leaving directory '/home/runner/work/open_pdks/open_pdks/sky130'
Error: Process completed with exit code 2.

Pulled and merged.  Magic was fixed to correct the segfault.  Both project updates have been pushed to github, so subsequent CI runs should be successful.


Support common use setAttribute(.., null)

From the bad old days I think a lot of people learned not to use removeAttribute, but instead call setAttribute with null. If you do that however it will be coerced with property types. So you get "null" instead.

We don't override setAttribute() so props will get the browser-coerced value, unfortunately. I don't think this is something we can - or really should - fix. Is there something in particular causing this issue for you?

I was not talking about overriding setAttribute. I called setAttribute(.., null) out of habit and got a coerced value in attributeChangedCallback that was "null" rather than null.
I have also noticed that when I define attributeChangedCallback and observedAttributes the props don't seem to be set on the element.

Make sure that in attributeChangedCallback you call super.attributeChangedCallback
The docs need to remind to call super when overriding a ....Callback.

I do call the super method, but the property doesn't seem to get set. And I do think the docs have that reminder.

I was not talking about overriding setAttribute. I called setAttribute(.., null) out of habit and got a coerced value in attributeChangedCallback that was "null" rather than null.

Yes, but in order to change native behaviour we'd need to override it. I don't think it's a good idea to change expectations here.

I have also noticed that when I define attributeChangedCallback and observedAttributes the props don't seem to be set on the element.

observedAttributes need to mix with the super, if you define props because when you define props that have { attribute: true } they will automatically observe the linked attribute.
Something like return { ...super.observedAttributes, myNewObservedAttributes } should work.

In the code below I have to set the this.actionTaken and this.actionStarted, as they are not set automatically by the super method. What am I doing wrong?
    get props() {
        return {
            blockId: prop.string({ attribute: true }),
            actionStarted: prop.string({ attribute: true }),
            actionTaken: prop.string({ attribute: true })
        };
    }

    static get observedAttributes() {
        return super.observedAttributes.concat('block-id','action-started','action-taken');
    }

    attributeChangedCallback(name, oldValue, newValue) {
        super.attributeChangedCallback(name, oldValue, newValue);
        switch(name) {
            case 'block-id':
                this.blockId = newValue; // is this automagically set?
                this.blockTemplate = incrementalTemplates.get(this.blockId);
                    // this[ticker] = setInterval(this.renderTick.bind(this),1300);
                break;
            case 'action-taken':
                this.actionTaken = newValue;
                this.constructor.renderer(this);
                break;
            case 'action-started':
                this.actionStarted = newValue;
                this.constructor.renderer(this);
                break;
        }
    }

Is the original issue perhaps an issue with the custom elements shim support for the attributeChangedCallback?

You need static get props() (note static).


Dissenter Extension Missing On Dissenter.com

why are you hiding? where did it go?

I am also wondering.
Its odd that the extension option on Dissenter leads to *nothing other than the browser itself.
Please give us a download option!

Apparently, they want to stop all support and use of the extension in favor of the browser. A horrible move IMO.

How? Google and Mozilla enforce extension signing, so it's not possible to install the extension from dissenter.com either.

In Chrome or Chromium you select "Developer Mode" to load the extension manually, which you can get here on GitHub. Click "Load Unpacked" to load it. On Firefox, install the Nightly or Developer version of the browser. Or, use Waterfox.

Terrible extension for months now, Torba - you ruined a good thing.

@BlohoJo Waterfox doesn't have a future. However, Pale Moon has redundancy in terms of developers and a maintained XUL platform


Update classifiers in setup file

As of https://github.com/pythad/nider/pull/230 nider supports latest versions of Python (3.6+). We need to update classifiers in setup file (https://github.com/pythad/nider/blob/master/setup.py#L56) so that PyPi shows correct versions on their index page as long as in github badge.

@pandrey2003 could you help with this?


"Analysis processing failed"

When we see "Analysis processing failed", where should we look to investigate what failed?

The ouput of the CodeQL action is saying
Processing sarif files: ["reports/test.sarif"]
Uploading results
Successfully uploaded results

Normally, there should be a failing code scanning workflow run in the Actions tab of your repository. The fact that the latest scan has a duration of 1s, implies to me that the workflow didn't really run at all (or at least it was aborted shortly after starting).
Can you locate which run this was? And add the logs?
If this repo is public, can you share the link?

Hi
This is the workflow:
https://github.com/h3xstream/sarif-samples/actions/workflows/upload-me.yml

Thanks for the information. I've forwarded this on to the code scanning team and someone should follow up.

:wave: @h3xstream We have few additional expectations from the SARIF file that are stricter than what the standard requires. These are meant to provide a reasonable UI experience. One of those is that each result should have a message. The file you uploaded does not include these values.
You can see the full explanation at: https://docs.github.com/en/code-security/secure-coding/integrating-with-code-scanning/sarif-support-for-code-scanning#result-object
To further debug this type of problems, you can use the Code Scanning API endpoint to list analyses. These errors are reported under the error property: https://docs.github.com/en/rest/reference/code-scanning#list-code-scanning-analyses-for-a-repository
Hope this helps!

So, I'm looking at

https://github.com/jsoref/examples-testing/actions/runs/3454140264
https://github.com/jsoref/examples-testing/security/code-scanning?query=is:open+pr:25 -- has an "Analysis processing failed".
https://github.com/jsoref/examples-testing/actions -- There is nothing in the /actions list directly relating to analysis processing...

Thoughts:

If at all possible, the Analysis processing failed should be a link to something. Preferably to a report showing the specific complaint(s). Alternatively, to a page explaining how to debug a failure.
I was hoping there'd be some way for me to do what I'm trying to do here, but, so far, I haven't found one...
It isn't obvious why the error message appears to be doubled.
If the errors thing should be an array, perhaps it should be reformulated as such?
if you're going to merge items into a human readable message in errors, please include whitespace between entries.
If an error can be sufficiently ambiguous, why not include the actual value you dislike instead of just the parsed portion?
I guess that results_count and rules_count aren't populated unless the sarif parser is entirely happy, ...
To make my life easier (and to enable one to pull the sarif file into other tools, e.g. VSCode), the action generates a sarif artifact, so people can retrieve it and review manually.

code-scanning/analyses/51044536
gh api /repos/jsoref/examples-testing/code-scanning/analyses/51044536:
{
  "analysis_key": ".github/workflows/spelling.yml:spelling",
  "category": "check-spelling",
  "commit_sha": "1903e85d65371f1fb587e60e92fddf822e81a55c",
  "created_at": "2022-11-13T06:26:40Z",
  "deletable": false,
  "environment": "{}",
  "error": "SARIF URI scheme \"https\" did not match the checkout URI scheme \"file\",SARIF URI scheme \"https\" did not match the checkout URI scheme \"file\",",
  "id": 51044536,
  "ref": "refs/pull/25/merge",
  "results_count": 0,
  "rules_count": 0,
  "sarif_id": "21e893e4-631c-11ed-88e3-9a1618496759",
  "tool": {
    "guid": null,
    "name": "check-spelling",
    "version": "0.0.20"
  },
  "url": "https://api.github.com/repos/jsoref/examples-testing/code-scanning/analyses/51044536",
  "warning": ""
}

The relevant sarif items
...  appear to be:
[
...
        {
          "ruleId": "unrecognized-spelling",
          "ruleIndex": 0,
          "message": {
            "text": "[haddd](#security-tab) is not a recognized word. \\(unrecognized-spelling\\)"
          },
          "locations": [
            {
              "physicalLocation": {
                "artifactLocation": {
                  "uri": "https://github.com/jsoref/examples-testing/pull/25#",
                  "uriBaseId": "%SRCROOT%"
                },
                "region": {
                  "startLine": 1,
                  "startColumn": 6,
                  "endColumn": 11
                }
              }
            }
          ]
        },
...
        {
          "ruleId": "unrecognized-spelling",
          "ruleIndex": 0,
          "message": {
            "text": "[fleeze](#security-tab) is not a recognized word. \\(unrecognized-spelling\\)"
          },
          "locations": [
            {
              "physicalLocation": {
                "artifactLocation": {
                  "uri": "https://github.com/jsoref/examples-testing/pull/25#",
                  "uriBaseId": "%SRCROOT%"
                },
                "region": {
                  "startLine": 1,
                  "startColumn": 5,
                  "endColumn": 11
                }
              }
            }
          ]
        },
...
]


Test auto attach for node.js subprocesses (aka cluster support)

Test for #40123:
Complexity: 4

[x] Windows - @aeschli
[x] OS X - @RMacfarlane
[x] Linux -  @bpasero

For node-debug we've added a mechanism that tracks all subprocesses of a debuggee and tries to automatically attach to those that are launched in debug mode. This feature is enabled by setting the launch config attribute autoAttachChildProcesses to true.
Whether a process is in debug mode is guessed by analysing the program arguments. Currently we detect the patterns --inspect, --inspect-brk, --inspect-port, --debug, --debug-brk, --debug-port (all optionally followed by a '=' and a port number).
Here is an example based on the 'cluster' node module:
test.js:
const cluster = require('cluster');
const http = require('http');
const numCPUs = require('os').cpus().length;

if (cluster.isMaster) {
	console.log(`Master ${process.pid} is running`);

	// Fork workers.
	for (let i = 0; i < numCPUs; i++) {
		cluster.fork();
	}

	cluster.on('exit', (worker, code, signal) => {
		console.log(`worker ${worker.process.pid} died`);
	});
} else {
	// Workers can share any TCP connection
	// In this case it is an HTTP server
	http.createServer((req, res) => {
		res.writeHead(200);
		res.end('hello world\n');
	}).listen(8000);

	console.log(`Worker ${process.pid} started`);
}

the launch config:
{
	"type": "node",
	"request": "launch",
	"name": "Cluster",
	"program": "${workspaceFolder}/test.js",
	"autoAttachChildProcesses": true
},

Verify that this feature works for the example from above or other node.js programs that create sub-processes in debug mode.

Hi.
I copied word for word this test plan but can't get it working.  It never attaches to the child processes.
Is there anything else I need to do?
Windows 10
VS Code ver 1.20.1 (latest)
Node ver 8.9.1
Debug console
Debugging with inspector protocol because Node.js v8.9.1 was detected.
node --inspect-brk=3971 test.js
Debugger listening on ws://<IP_ADDRESS>:3971/b5bc61df-63fc-46d7-8e53-1f99838e25d9
Master 25692 is running

Thanks.


Add a warning when saving a multichunk VPK without a "_dir" suffix

when creating the output vpks and specifying the name of the actual output vpk when naming the vpk the file cant be opend in a vpk viewer unless we name the output vpk whatever_dir.vpk
NOTE: the whaterver_dir.vpk doesnt actualy need to be named whatever_dir.vpk but it needs to have _dir at the end of the file

could just force add _dir to the end of the filename but then again you can just tell people to actualy just specify _dir if they are trying to create a vpk using -o

This is a required VPK feature, directory VPKs (VPKs that reference numbered archives) will always end in _dir
I will add a warning to the console

I pushed a new release with the fixes to the issues you've been reporting (thanks for that btw! much appreciated)


Update readme.md

Fix PrintFile example
Trying to run the example as-is in LINQPad threw exceptions. Generally the passed in values were not read.

Add '=' suffix to file argument
Add ':' suffix to strip argument
Add comments on why this is needed

Crickey.
Thanks you for finding this.

Thanks for the lib, this is great stuff.


fix(config): don't use module condition (fixes #10430)

Description
node does not use module condition. But Vite was using that when bundling (replacing specifier with absolute path) config.
This PR uses esbuild to resolve specifier instead of Vite's internal resolver.
I came up with three solutions for this.

Add noModuleCondition option to vite's internal resolver and use that for config bundling (similar to #10442)

In Vite 4, I think we could clean up these options but I think it's better to avoid introducing a new option for now.

Use esbuild to resolve specifiers

As we don't need fine-grained option and just need to imitate node's resolver, I think using esbuild's one is more performant. Also we could avoid a new option. This PR is implementing in this way.

Use import.meta.resolve / import-meta-resolve (ponyfill)

import-meta-resolve is only 63.3kB and adding this won't affect the package size much. We could perfectly imitate node's resolver by using this. But I think we could just use esbuild's resolver as it's reliable enough and could be more performant.

superseds #10442
fixes #10430
refs #10254 #10347 #10420
Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

/ecosystem-ci run

Could we add a test to ensure we don't revert https://github.com/vitejs/vite/issues/10430 in the future?

It seems Vitest is failing because esbuild uses paths in tsconfig.json. I didn't find a way to disable this. Maybe we have to do it in a different way.
Sure. I'll add tests when we decided the approach. 👍
@bluwy What way did you have in mind?

You might hate me for this, but tsconfig: 'package.json' works for Vitest for me 🙈 We could point to Vite's package.json instead to disable tsconfig altogether since I don't think it's relevant for nodejs bundling (?).
I really like the esbuild solution, but otherwise we may have to use import-meta-resolve.

That's neat. But unfortunately I guess that will break scripts using useDefineForClassFields.

Hmm yeah maybe an alternative for now is to go with no1. The initial idea I had was to introduce a strictNodeResolve: boolean option, so that allows us to expand more in the future. We can start with removing the module condition, and possibly the production/development condition?
If it happens to be a lot of work, import-meta-resolve would be the easier way out.

superseded by #10528


Parser does not support ~ (relational inverse)

p = {3|->5, 3|->9, 6|->3, 9|->2} &  q =p~
in REPL gives
Parse error: Unexpected input '~' in line 1 column 168.
Additional information: mismatched input '~' expecting {, EQUIVALENCE, IMPLIES, '&', 'or'}
Exception in thread "JavaFX Application Thread" Parse error: Unexpected input '~' in line 1 column 168.run

'~' is not yet supported by the Z3 translator

should be supported now


Transition inside forms crashing in Safari 9.1.1

Vue.js version
2.0.0-beta.6
Reproduction Link
http://jsbin.com/miyoru/edit?css,js,output
Steps to reproduce
In the jsbin above, click in the Toggle button.
What is Expected?
An animation expanding the input field.
What is actually happening?
The animation starts and then the application crashes.
PS: if you remove the <form> tag, the animation works accordingly.

This is not a Vue bug. Safari seems to treat the <button> inside the form as the submit button by default, so it submits the form and reloads the page.

yes, if you add v-on:submit.prevent to the form it stays visible

 without type="button" considered as the submit button by default in any browser


When using ACROSTIC, report accurate metrics still

The -V report should reflect the exact entropy (assuming perfect RNG) of the generated password, taking all parameters into account.
Currently, when combining the -a and -V options, the printed metrics do not take the acrostic into account, i.e. they use the default numwords and disregard limiting the first letter of each word.

Looking at the code, I see some moderate refactoring is due, in order to fix this issue. Before I begin to work on a PR, I would like to share my plans here.
I am thinking to always use the acrostic and its worddict in generate_xkcdpassword() (instead of numwords and wordlist), and if no acrostic was given, use acrostic = "_" * numwords and worddict = { '_': wordlist }. All this should happen before the call to verbose_reports(), so the calculations there can take the acrostic into account.
What do you think?

Work in progress can be seen here. I would like to make the interface with _to_acrostic_and_worddict less fragile, sprinkle some more comments around the code and improve the commit messages before submitting the PR.
Let me know if you would like me to submit as-is, because it might be a while before I have time to work on it again.

Thanks for your work on this - I like your solution to the problem and I can see a few places the wider code base could be refactored to streamline this approach.
Feel free to open and update the PR as suits you. I'll make some time later this week to merge it in some form!

Fixed in latest release, xkcdpass-1.11.0


Add warning to logging page about non-integer sample intervals

Resolves #106
See rendered view:

Pull #99 is waiting for the v1.3 release in AMDC-Firmware, but since this applies to previous versions of the v1 codebase, I think this pull can be merged right away.


AttributeError: Could not find IP or hierarchy configure in overlay

Hi, using the ultra.bit present in the repo for Ultrasound Classification throws this error:

Same error is thrown for subsequent blocks also:

Am I missing anything! Is it because I did not do anything with the Digilent vivado IP library?
PS: My goal is to classify some other dataset using your CNN i.e. use the same structure( layers and overlay). So, need your guidance
Thanks

Sorry, I don't know what the problem is exactly. Maybe you could check the file fpga_drive.py, or check if the pynq package is installed correctly.

Hi @shobhitt10 @ZhaoqxCN
I am also facing the same issue. i still don't understand why it appears. did you solve this issue?

anyboby can answer the question? I meet the same question,too.

I also encountered the same problem. Has anyone successfully solved it? Can you answer it


Upon encountering validation error, provide index of error

One of our users uploaded a 200 row manifest with many columns, and the validation report shows the following:

It would be helpful if it provided the index, or at least the row, of the error, so that the user can find the value quickly.

Yeah, the reason we removed row index was because we tried to avoid the confusion between google sheet row index and datatable row index. But I agree with you, if filled manifest has a lot of rows and just a couple of wrong values , it will be hard for users to locate the invalid values.
So I think we will make following changes:

[ ] add the row index back into error table and I believe the index will match the google sheet index
[ ] add the row number back into preview table, but index here will start with 2 instead of 1 to avoid confusion.
[ ] add search function in each column, so user can search invalid values in preview table as well.

@allaway and @milen-sage, do you think this will be helpful?

Yes, agreed @rrchai,  given the row index now match Google sheet's indexing, displaying this info would be helpful for the user.

add search function at the top of each column, so user can search invalid values each column in preview table as well.

This^ would be useful but if it takes more than a couple of hours to implement let's prioritize w.r.t. other work on the DC app.

Thanks, both! That makes a lot of sense.
@milen-sage  - thankfully, simple search is very easy to implement in reactable or DT: https://shiny.rstudio.com/gallery/nhl-play-by-play.html


tag a release

Greetings!
As requested on the WantedPorts page, I would like to make a port for this project.  Unfortunately, no version is tagged in this repository and I do not really want to package a random possibly unstable project state.
Please tag a release if possible so I can base the port on that release and version number.

I just created 0.0.1
Thank you for maintaining the port.

This is as stable as recent. If you hit issues in porting, please let us know. One issue: the .conf file. That really should be in ‘/usr/local/etc/mkjail’ and a port might be the push we need to change the code.  Unless you can patch it in the port.

If I understood the logic in src/bin/mkjail correctly, the configuration file is fetched from the same prefix as the script, so if the script is in /usr/local, the correct configuration file is used already.
Would you mind writing a pkg-descr file for the port?  I'm not super familiar with this port's features, so if you write the description, it'll probably be a lot better than if I attempted to do it.

I have added information to the README.md, how's that?

Thanks for the quick response!
Port submitted in PR #256871.  Someone please edit the WantedPorts page to take out this one.


0.4 - Improve Extension object support

NOTE: This is just to show that some progress is being made on this module, however I have a habit of taking months and months of time off from using working on this, so please if you're waiting for this PR or me covering it off is going to save you time, please communicate that here

These were condensed into a single commit and pushed in.


[kernel] remove LagrangianDS::_workMatrix

All is in the title.
Remove and transfer it if necessary to the DS Graph.

Lagrangian::massLU() should also be suppressed.

Done at least for LagrangianDS::massLU()

@xhub : should I start this work, if you have a work with subsequent work that goes in that direction ?

I haven't work on the LagrangianDS::_workMatrix. I worked on the memory vectors. I will try this week to polish that and fix the bug with SchatzmannPaoliOSI and push it on master

Ok, i will let working the Professional ;_0


Resolve syncing conflicts from FromPrivateLiveToMaster to master

Pull request opened by Docs to resolve syncing conflicts from FromPrivateLiveToMaster to master.

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Validate against duplicate sample-info-file columns

Add an additional validation step to the beginning of SPP that would compare the columns in the sample-sheet against the columns in the sample-info file and return an error if a duplicate exists. It would be best for this function to live in metapool so that it could be used in notebooks as well. We could also add this validation to a validation notebook as well; either or both would work.
This is to prevent the issue of an SPP run failing because the owner of a project uploaded a sample-info sheet with a column name that KL has already reserved for its own use e.g. 'sample_plate', 'tube_id'.

As Antonio mentioned, we will also need to update these columns in Qiita:
https://github.com/qiita-spots/qiita/blob/master/qiita_db/metadata_template/sample_template.py#L61
Could you issue PR in qiita updating/adding the columns not allowed in the sample/prep files? I don't think we need new tests, just the lists. Thanks.


Decals appear double from the distance

I noticed since 3.9 when you zoom in on decals from the distance they appear double like that:
Closeup:
http://duion.com/files/temp/screenshot_TG_BuildingSite_day006-00000.png
Zoomed from far:
http://duion.com/files/temp/screenshot_TG_BuildingSite_day006-00001.png

Ok a bit of false alarm, it only appears on that certain spot, since there is a pavement mesh and below that there is the terrain and the decal gets applied to both, the pavement and the terrain underneath, since the pavement is not very thick. Now when you stand elevated and further away the decal below on the terrain shines through.
So the solution would be to make sure the decal only gets applied to the first surface it hits.


[Tests] Add tests for the replacement of SESSION_USER with SESISON_CONTEXT('APPLICATIONUSER')

Currently, we don't seem to have tests asserting if SESISON_CONTEXT('APPLICATIONUSER') works on HANA databases. We have some xsjs files in the integration-tests/ui module but they don't seem to be run.

This issue is closed as the project is no longer maintained


Terraformer fails when running validate command

How to categorize this issue?
/kind bug
What happened:
I have set up a validate command as shown in the example in the documentation.
When starting the container with a validate command I get  the message   Success! The configuration is valid. followed by an error:  Error: Insufficient features blocks  on <empty> line 0:  (source code not available)
The second error seems to come from running the plan command. Any changes I make to the configmap seem to have no impact on the message (i tried introducing some syntax errors but the error remains the same)
When running the apply command I get the same error directly on the apply command. I believe rbac is properly set up (i followed the example) because when i take my serviceaccount away, I get an error about permissions.
The logs:
{"level":"info","ts":"2023-08-28T09:56:08.552Z","msg":"terraform process finished successfully","step":"executeTerraform","command":"init"}
{"level":"info","ts":"2023-08-28T09:56:08.557Z","msg":"executing terraform","step":"executeTerraform","command":"validate","args":"-chdir=/tf validate -no-color"}
Success! The configuration is valid.

{"level":"info","ts":"2023-08-28T09:56:09.719Z","msg":"terraform process finished successfully","step":"executeTerraform","command":"validate"}
{"level":"info","ts":"2023-08-28T09:56:09.719Z","msg":"executing terraform","step":"executeTerraform","command":"plan","args":"-chdir=/tf plan -no-color -var-file=/tfvars/terraform.tfvars -parallelism=4 -detailed-exitcode -state=/tfstate/terraform.tfstate"}

Error: Insufficient features blocks

  on <empty> line 0:
  (source code not available)

At least 1 "features" blocks are required.
{"level":"error","ts":"2023-08-28T09:56:11.366Z","msg":"terraform process finished with error","step":"executeTerraform","command":"plan","error":"exit status 1","stacktrace":"github.com/gardener/terraformer/pkg/terraformer.(*Terraformer).executeTerraform\n\t/go/src/github.com/gardener/terraformer/pkg/terraformer/terraformer.go:299\ngithub.com/gardener/terraformer/pkg/terraformer.(*Terraformer).execute\n\t/go/src/github.com/gardener/terraformer/pkg/terraformer/terraformer.go:210\ngithub.com/gardener/terraformer/pkg/terraformer.(*Terraformer).Run\n\t/go/src/github.com/gardener/terraformer/pkg/terraformer/terraformer.go:117\ngithub.com/gardener/terraformer/cmd/terraformer/app.addSubcommand.func1\n\t/go/src/github.com/gardener/terraformer/cmd/terraformer/app/terraformer.go:110\ngithub.com/spf13/cobra.(*Command).execute\n\t/go/src/github.com/gardener/terraformer/vendor/github.com/spf13/cobra/command.go:916\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\t/go/src/github.com/gardener/terraformer/vendor/github.com/spf13/cobra/command.go:1044\ngithub.com/spf13/cobra.(*Command).Execute\n\t/go/src/github.com/gardener/terraformer/vendor/github.com/spf13/cobra/command.go:968\nmain.main\n\t/go/src/github.com/gardener/terraformer/cmd/terraformer/main.go:25\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250"}
{"level":"info","ts":"2023-08-28T09:56:11.378Z","msg":"triggering final state update before exiting","step":"finalStateUpdate","timeout":"1h0m0s"}
{"level":"info","ts":"2023-08-28T09:56:11.387Z","msg":"successfully stored terraform state","step":"finalStateUpdate"}
{"level":"error","ts":"2023-08-28T09:56:11.387Z","msg":"error running terraformer","error":"error executing terraform plan: terraform command failed with exit code 1: exit status 1","stacktrace":"main.main\n\t/go/src/github.com/gardener/terraformer/cmd/terraformer/main.go:27\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:250"}

What you expected to happen:

The pod should run without errors
Changes in the config map should reflect in the error
The error should clearly reflect what is wrong

How to reproduce it (as minimally and precisely as possible):

Run a pod with terraformer v2.23.0 with a validate command as such

apiVersion: v1
kind: Pod
metadata:
  name: hello-world-terraformer
  namespace: terraformer-poc
spec:
  activeDeadlineSeconds: 1800
  containers:
  - name: terraform
    image: ***/terraformer:v2.23.0
    imagePullPolicy: IfNotPresent
    command:
      - /terraformer
      - validate
      - --zap-log-level=debug
      - --configuration-configmap-name=dev-hello-world.tf-config
      - --state-configmap-name=dev-hello-world.tf-state
      - --variables-secret-name=dev-hello-world.tf-vars
    resources:
      requests:
        cpu: 100m
    terminationMessagePath: /terraform-termination-log
  restartPolicy: Never
  serviceAccountName: terraformer
  terminationGracePeriodSeconds: 600

Create configmaps as such:

apiVersion: v1
kind: ConfigMap
metadata:
  name: dev-hello-world.tf-config
  namespace: terraformer-poc
data:
  providers.tf: |
    terraform {
      required_version = ">=0.12"
      required_providers {
        azurerm = {
          source  = "hashicorp/azurerm"
          version = "~>2.0"
        }
      }
    }
    provider "azurerm" {
      features {}
    }
  main.tf: |
    resource "azurerm_resource_group" "rg" {
      location = "westeurope"
      name     = "terraformer-poc"
    }
  variables.tf: ""
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: dev-hello-world.tf-state
  namespace: terraformer-poc
data:
  terraform.tfstate: ""
---
apiVersion: v1
kind: Secret
metadata:
  name: dev-hello-world.tf-vars
  namespace: terraformer-poc
type: Opaque
stringData:
  terraform.tfvars: ""

Anything else we need to know?:
I'm authenticating to Azure with env vars containing an SPN according to the terraform docs. I have also tried directly placing the credentials in the provider block but its the same result. I'm guessing the problem could be with authentication, in which case the error should more clearly reflect the reason. (but ofcourse im not sure about that)
Environment:

Gardener version (if relevant): NA
Terraformer version: 2.23.0
Kubernetes version (use kubectl version): 1.27.3
Cloud provider or hardware configuration: Azure
Others:

I found the issue: it was not related to authentication. I merged the file providers.tf into main.tf and i removed the "terraform" block from it.


Simplify the page_params generation logic #3853

Moved each item in the realm data section down into fetch_initial_state_data so it entirely pulls from register_ret, and then into page_params_core_fields.  These were moved under the realm section of and renamed with the realm_ prefix and involved similar frontend page_param changes.  Please review if this is OK.
Moved the user_profile data section down into fetch_initial_state_data so it entirely pulls from register_ret, and then moved into page_params_core_fields. This did not involve any renaming of fields so no frontend changes were made.
Moved subbed_info, people_list, etc. to use the names matching what is returned by the Zulip API, and then moved into page_params_core_fields.  Renamed the JS on the frontend (and thus also page_params)

Tested on my local dev server.
Fixes #3853

Automated message from Dropbox CLA bot
@fionabunny, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Thanks for working on this @fionabunny!
A few quick things:

http://zulip.readthedocs.io/en/latest/version-control.html might be helpful reading for our commit message style.
We rarely comment out code. You can just delete it!
Feel free to fix up and push just the first few commits of this branch, so that we can iterate through them quickly.

You can leave a comment on this thread when you're done, which will notify us to take another look. Thanks!

@fionabunny the rest of your changes also looked really good, except that this branch revealed a number of minor but tricky bugs in our register_ret testing system in test_events.py (see zulip.readthedocs.io/en/latest/events-system.html for details on that)
I ended up merging the rest of this, fixing those tricky events system issues as I went (since they all seemed like the kind of thing that would be hard for someone new to the project to resolve).
You can read the merged versions in git log -p, but essentially all I ended up doing is making the necessary changes to test_events.py and events.py for some of the more complicated features.
Huge huge thanks for doing this!
As part of fixing the tricky events system issues, I moved all the desktop notifications changes to the new system as well.  I suspect we're very close to being able to do the final step of removing page_params_core_fields and replacing it with register_ret.keys() and thus closing #3853 entirely :).

Oh, one more note: we'll want to update the documentation in a few places in docs/ (the new feature tutorial, at the very least) when we finally remove page_params.core_fields.

Automated message from Dropbox CLA bot
@fionabunny, thanks for signing the CLA!


Adding Binaries in Images

Hi,
I have a sample java based hello world spring boot Gradle project. I want to add other tools in the containers like wget, curl, and git.
I am using paketo build packs to build images. I have a requirement to include binaries of other tools in the image.
For Ex: wget,curl,git in /usr/local/bin.
I have gone through a lot of workarounds:
Using custom buildpacks
Creating a new image with the docker file
I want to avoid any overhead or maintenance work and I want CNB to handle any deps.
Hence looking for a better approach, How can I achieve this using kpack and how to add mixins in kpack stack? Any help or suggestion would help a lot.

@tsusadivyago
Does the full paketo stack fit your use case? It contains wget, curl, and git.
Besides the curated stacks from paketo, your best bet is a custom buildpack or to create your own stack

yes, paketo stack fits 99% requirement.
but we need terraform binary in the container.
what we are doing now is (2 approaches):

Keeping the terraform binary in source code with other files this binary is also copied to the meta.inf... folder in the workspace
we are creating an image and later using the docker file to create another image with terraform installed.

Is creating a custom pack or stack the only option I am left with?
I want to know how other devs are handling this use case, how are you getting your binaries in the container?

Concretely, to add a binary to an existing stack I would do the following:

clone the stack repo in question (e.g. jammy-full-stack)
parse the stack.toml file with a TOML parser
add the required packages (and PPAs if necessary) to the package list
write the stack.toml file back
run jam create-stack with the location of the updated stack.toml.

If you want to install something that isn't available via a package, then I would suggest you instead modify the Dockerfiles for build and run to download it via wget.

Hey, Thanks for the reply. I will try these out.

please re-open the issue if you have questions!


ModalOverlay of Guided Tour disappears after canceling skip

Describe the Bug
No response
Steps to Reproduce the Problem

open the map editor
cancel the tour via the X
cancel the modal popup

Expected Result
Previous tour step is shown with modal overlay, i.e. the background darkens and is not clickable.
Actual Result
Previous tour step is show, but without modal overlay, i.e. the background is normal and clickable
PermaplanT version
738-allow-continue-tour
Operating System
Windows
Browser
Microsoft Edge
Versions of other relevant software/extensions
No response
Additional info/logs
No response

I can reproduce with Firefox. The problem is a bit worse than I thought: When I click somewhere, it counts like a confirmation. At least this should be fixed for the release tomorrow but better is to fix this fully.

Will be fixed with #753.


Optimize kashminer Performance by Expanding RxVMPool

Expanded RxVMPool to enhance mining efficiency.
Implemented code-level restrictions for mainnet mining.
Changed KAS symbols to KSH for consistency.

Hello, I want to meet with the kash administrator, I want to help the project and create a community, can you send me your discord account for this, let's work together.

Hello, I want to meet with the kash administrator, I want to help the project and create a community, can you send me your discord account for this, let's work together.
@BlacEagle
Thank you for your interest and willingness to support the project! Everyone is welcome to join our community. You can easily connect with us and others interested in the project through our Discord server. Please use this link to join: https://discord.gg/yx46racf. Looking forward to collaborating with you!


Add supporting data-response

I think https://github.com/yiisoft/data-response can help for fast start with yii-rest if it will be provided "out of the box"

Also https://github.com/yiisoft/yii-rest/blob/master/src/ResponseFactoryInterface.php can be removed

The package should be added to composer.json, require section.


clarify the main purpose of the github installation method

@msouff : I have increased your level of access to the repo. Let me know if that helps and lets you merge.
Secondly, for the PR, I feel like you don't have the latest Master within the branch you are creating and maybe that's why you are getting this conflict.
Can you try the following commands:
git checkout master
git pull master
git checkout githubinstallcarification
git rebase master
#At this point you should start seeing a merge conflict locally
#complete the rebase and commit your changes after resolving the conflicts
git push -f


[Bug] [Core] [Spark-Starter] Multiple Sinks blocked and never started in STREAMING mode.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Multiple Sinks blocked and never started in STREAMING mode. Spark Engine just start one sink.
In my case, just one console print record.
SeaTunnel Version
2.3.1
SeaTunnel Config
env {
  job.mode = STREAMING
  job.name = "SeaTunnel"
  spark.master = "local[*]"
}

source {
  FakeSource {
    row.num = 1
    schema = {
      fields {
        name = "string"
      }
    }
    result_table_name = "fake0"
  }

  FakeSource {
      row.num = 1
      schema = {
        fields {
          age = "int"
        }
      }
      result_table_name = "fake1"
    }
}

sink {
  Console {
    source_table_name = "fake0"
  }

  Console {
      source_table_name = "fake1"
    }
}

Running Command
./bin/start-seatunnel-spark-3-connector-v2.sh --master local --deploy-mode client --config ./config/streaming.conf

Error Exception
23/04/21 16:33:06 INFO SharedState: Setting hive.metastore.warehouse.dir ('null') to the value of spark.sql.warehouse.dir.
23/04/21 16:33:06 INFO SharedState: Warehouse path is 'file:/home/cheney/expr/seatunnel/apache-seatunnel-incubating-2.3.1/spark-warehouse'.
23/04/21 16:33:09 INFO V2ScanRelationPushDown:
Output: name#0

23/04/21 16:33:10 INFO CodeGenerator: Code generated in 147.67237 ms
23/04/21 16:33:10 INFO AppendDataExec: Start processing data source write support: org.apache.seatunnel.translation.spark.sink.SeaTunnelBatchWrite@34f2d3a6. The input RDD has 1 partitions.
23/04/21 16:33:10 INFO SparkContext: Starting job: save at SinkExecuteProcessor.java:125
23/04/21 16:33:10 INFO DAGScheduler: Got job 0 (save at SinkExecuteProcessor.java:125) with 1 output partitions
23/04/21 16:33:10 INFO DAGScheduler: Final stage: ResultStage 0 (save at SinkExecuteProcessor.java:125)
23/04/21 16:33:10 INFO DAGScheduler: Parents of final stage: List()
23/04/21 16:33:10 INFO DAGScheduler: Missing parents: List()
23/04/21 16:33:10 INFO DAGScheduler: Submitting ResultStage 0 (MapPartitionsRDD[2] at save at SinkExecuteProcessor.java:125), which has no missing parents
23/04/21 16:33:10 INFO MemoryStore: Block broadcast_0 stored as values in memory (estimated size 11.5 KiB, free 434.4 MiB)
23/04/21 16:33:10 INFO MemoryStore: Block broadcast_0_piece0 stored as bytes in memory (estimated size 5.6 KiB, free 434.4 MiB)
23/04/21 16:33:10 INFO BlockManagerInfo: Added broadcast_0_piece0 in memory on <IP_ADDRESS>:42561 (size: 5.6 KiB, free: 434.4 MiB)
23/04/21 16:33:10 INFO SparkContext: Created broadcast 0 from broadcast at DAGScheduler.scala:1513
23/04/21 16:33:10 INFO DAGScheduler: Submitting 1 missing tasks from ResultStage 0 (MapPartitionsRDD[2] at save at SinkExecuteProcessor.java:125) (first 15 tasks are for partitions Vector(0))
23/04/21 16:33:10 INFO TaskSchedulerImpl: Adding task set 0.0 with 1 tasks resource profile 0
23/04/21 16:33:10 INFO TaskSetManager: Starting task 0.0 in stage 0.0 (TID 0) (<IP_ADDRESS>, executor driver, partition 0, PROCESS_LOCAL, 4585 bytes) taskResourceAssignments Map()
23/04/21 16:33:10 INFO Executor: Running task 0.0 in stage 0.0 (TID 0)
23/04/21 16:33:10 INFO ConsoleSinkWriter: output rowType: name<STRING>
23/04/21 16:33:10 INFO FakeSourceSplitEnumerator: Starting to calculate splits.
23/04/21 16:33:10 INFO FakeSourceReader: wait split!
23/04/21 16:33:10 INFO FakeSourceSplitEnumerator: Assigned [FakeSourceSplit(splitId=0, rowNum=1)] to 1 readers.
23/04/21 16:33:10 INFO FakeSourceSplitEnumerator: Calculated splits successfully, the size of splits is 1.
23/04/21 16:33:10 INFO FakeSourceSplitEnumerator: Assigning splits to readers 0 [FakeSourceSplit(splitId=0, rowNum=1)]
23/04/21 16:33:11 INFO FakeSourceReader: 1 rows of data have been generated in split(0). Generation time:<PHONE_NUMBER>894
23/04/21 16:33:11 INFO ConsoleSinkWriter: subtaskIndex=0  rowIndex=1:  SeaTunnelRow#tableId= SeaTunnelRow#kind=INSERT : QPFYX

----Block Here-------

Flink or Spark Version
spark 3.3.0
Java or Scala Version
OpenJDK 11
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

#4128 maybe can resolve this problem, you can continue work on it. Looking forward your contribution!

#4128 maybe can resolve this problem, but also there are some problems, you can continue work on it. Looking forward your contribution!

OK


Add "End Day" button

I'd like to be able to mark that my day is complete

This should give me a day summary

Maybe I can see days, not just time log


Add additional error checking to check extra769

Add additional error checking to check extra769
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@toniblyx I've raised a new PR for Prowler

cool, thanks! I'm working on a new function to check AWS CLI version, let me finish it and we can probably add more context to that message.

@toniblyx I thought about doing something like that, I could see gating each check individually on the minimum version needed to run it. I could also see doing a minimum version to run prowler and letting people know if their awscli is backlevel or not.

I'm going to create an issue to track that AWS CLI version discussion and share what I got so far.


Require phpunit/phpunit

Remove phpunit/phpunit from required composer.
Use require-dev for that.

create a pr for it


New validation rule: InvalidCategory

Adds a new validation rule that checks the values in a categorical column are part of its category set
Question

Should the validation be case-sensitive or the opposite?
On the roadmap for validating lab IDs, it is currently a varchar dataType. Should we:

Make it a categorical part
Populate the catSetID column with for example labCat
Allow the user to create their own custom dictionary where they add the allowable labID categories

Future Possible Changes

Rule IDs should be named to be the "invalid form". For example, MissingMandatoryColumns rather than NoMissingMandatoryColumns
The meta field for each column in a table in the cerberus schema should be a list with the following shape for each dictionary,{
    "ruleId": string,
    "meta": List<Dict> which contains the rows in the ODM dictionary used to populate this part of the schema
}

This should make the code to populate the validationRuleFields column in the error report easier

Looks good.


Changing colors through variables in less files

Hello @JoseRFelix .
I use your https://github.com/JoseRFelix/antd-dark-mode/tree/master as the basis of my own. The theme switcher works great. The question is how can I change colors for themes. I am trying to change the values of variables in less files. For example in dark-theme.less:
@ primary-color: red
But this has no effect. The only way that works is to change the colors directly in the css dark-theme.css of the file that are in public folder. This is not very convenient (
Tell me how to more correctly and quickly change the base colors of themes. Thank you so much.

Sorry. I solved the issue. Everything is in the article) Change in the file and collect through gulp. It's work.
https://dev.to/joserfelix/dynamic-theme-switching-in-ant-design-how-to-change-between-light-and-dark-themes-5b8p
Section Compiling Less Files With Gulp
Thanks a lot for the library.


Would it be possible to support non s6-overlay images?

Hi 👋🏼
It would be great if your images could support a non s6-overlay container as well, maybe something like homeassistant/home-assistant:2022.6.1-nonroot ? From what I can gather, there isn't any need to have s6-overlay since there is only one process running in this container anyways: python3 -m homeassistant --config /config
Having s6-overlay is a security issue because the container is running as root, even if you use PUID/PGID to drop root permissions right before the application launches this could also be view as a security threat. Having a good security posture and following container best practices is important IMO.
PID   USER     TIME  COMMAND
    1 root      0:00 /package/admin/s6/command/s6-svscan -d4 -- /run/service
   15 root      0:00 s6-supervise s6-linux-init-shutdownd
   17 root      0:00 /package/admin/s6-linux-init/command/s6-linux-init-shutdownd -c /run/s6/basedir -g 3000 -C -B
   22 root      0:00 s6-supervise s6rc-oneshot-runner
   23 root      0:00 s6-supervise s6rc-fdholder
   30 root      0:00 /package/admin/s6/command/s6-ipcserverd -1 -- /package/admin/s6/command/s6-ipcserver-access -v0 -E -l0 -i data/
   59 root      0:00 s6-supervise home-assistant
   60 root      1h54 python3 -m homeassistant --config /config
   96 root      0:00 /bin/sh
  102 root      0:00 ps

What benefits do you all find in using s6, maybe there is something I am not aware of?

You can use any image you want, our base images use S6, if you like something different, that is up to you 👍
../Frenck

Many things related to discovery are other examples (e.g., DHCP discovery).
You are free to use/run Home Assistant however you like, but we (as a project) have no interest in supporting was is suggested. This has been extensively discussed with the whole core team. The decision has been made and documented in our ADR records.


optional fields are required in createIssue()

optional milestoneId is required in createIssue()
I need to create an issue without a milestoneId.
http://docs.gitlab.com/ce/api/issues.html#new-issue
http://timolshansky.com/java-gitlab-api/ (createIssue)

updating issue for all optional fields. need a method to create an issue with just the required fields, projectId and title and submit optional fields if set.
maybe something like:
createIssue(int projectId, GitlabIssue gitlabIssue)


Fine tune and add final data for A1H

Is this a duplicate with https://app.zenhub.com/workspaces/experience-607d9d5e68b95200150fec37/issues/cdcgov/prime-reportstream/4585?


[Data Mapper] Cannot adjust/change a mapped attribute from the UI

Description:
Once you map a given source attribute to a target attribute, add some transformation and realize that you've made the mapping to a wrong field, you can change that from the UI.
You can ofcourse go to the source view and do it. It'll be great if user could make this change from the UI

@wx-yz Currently we are not supporting changing the links by manipulating them them directly. However you can still achieve this with the expression editor like below.


Usability test improvements

Pull request to merge code changes for improvements according to the usability test

Pipeline successful -> merge approved


Update translation

Add translation for notification panel
Update zh-cn.json

Heyo, thanks for your interest in Collapse! While I understand that submitting a PR may be easier for you, per our contributing guidelines (found here), we use Crowdin for our localization needs. Should you have any contributions you'd like to make, we recommend making them there, as we no longer merge PRs containing localization fixes since Collapse 1.69.13-pre.
I know you put a lot of effort into translating this and apologize for having your time wasted. This process is necessary so that we can streamline our development pipeline and reduce the stress we have while developing Collapse for our community.


Third-party deprecation details

Docs Build status updates of commit 89a2f39:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/skills/third-party-deprecation.md
:white_check_mark:Succeeded
View

docs/skills/toc.md
:white_check_mark:Succeeded
View

docs/toc.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


make proto seems to be introducing non-vendored dependencies

Currently, make proto seems to add an import that's not vendored.

What's the import?

Something like google/api.  Can you reproduce?

Fixed


tests: Use HTTP 1.1 instead of HTTP 1.0

Envoy does not support HTTP 1.0, so use HTTP 1.1 instead.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

Build 1 of this PR got stuck most likely due to recent issues with Jenkins which I believe I have resolved. If this build doesn't time out after 2 hours,  I'll manually abort it and clean it up.


Introducing generic Any.to type conversion method

Pull Request Description
Provides Any.to conversion method. Such a method allows us finish #4821 - e.g. use Text.from instead of Text.to_text where possible. This method is a per-requisite of further work on

#7320

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] All code follows the
Java,
All code has been tested:

[x] Unit tests have been written where possible.

8a56db5 demonstrates the fix proposed in #4879 really works and Any.to conversion can accept additional optional arguments.


System of project

Can I ask if the source code is running on Linux or Windows?

Everything was run on Ubuntu

Thank you so much! But I have a big question now. I have installed the Java,Spark,Scala,Scala-Sbt on the Ubuntu of one machine. But I don't know how to use the project in the next step. Could you give me some suggestions? Thanks!

Try to run these tasks: https://github.com/aluxian/Tweeather#running


Open folder in CDK via terminal (Similar to code .)

It'd be nice to have the ability to quickly type cdk . in a terminal to quickly open up the CDK at that directory.

Actually, it'd be even nicer, to have a miix-cli thingy to build and upload a control bundle from the command line, and to use the CDK for local testing without uploading anything at all for those cases where sharing in-progress work with an online service isn't appropriate or even prohibited.
This could be an NPM package, too... hint hint 😗


STDIN not working.

PHPMD version: 2.10.0
PHP Version: 8.0.3
Installation type: phar file
Operating System / Distribution & Version: macOS 11.4

Current Behavior
According to issue #542 STDIN should be fixed upstream. Issue #185 is quite old, is that still an issue? I was hoping for STDIN functionality on this release, so I could implement it easily in am IDE plugin.
Expected Behavior
Was hoping to be able to use STDIN for development of a plugin.
Steps To Reproduce:
Run phpmd - json cleancode,codesize,controversial,design,naming,unusedcode does not lint anything.
Checks before submitting

[X] Be sure that there isn't already an issue about this. See: Issues list
[X] Be sure that there isn't already a pull request about this. See: Pull requests
[X] I have added every step to reproduce the bug.
[X] If possible I added relevant code examples.
[X] This issue is about 1 bug and nothing more.
[X] The issue has a descriptive title. For example:  "JSON rendering failed on Windows for filenames with space".

Hello, 👋
STDIN implies you first output an other command as shown on https://github.com/phpmd/phpmd/issues/542#issuecomment-570605989:
I use cat with this simple file test.php (but it would work with any command so does not need any file):
<?php

class Foo
{
    public function bar()
    {
        $a = 9;
    }
}

And I run:
cat test.php | phpmd - json naming

And get:
{
    "version": "@package_version@",
    "package": "phpmd",
    "timestamp": "2021-04-28T07:53:22+00:00",
    "files": [
        {
            "file": "php:\/\/stdin",
            "violations": [
                {
                    "beginLine": 7,
                    "endLine": 7,
                    "package": null,
                    "function": null,
                    "class": null,
                    "method": null,
                    "description": "Avoid variables with short names like $a. Configured minimum length is 3.",
                    "rule": "ShortVariable",
                    "ruleSet": "Naming Rules",
                    "externalInfoUrl": "https:\/\/phpmd.org\/rules\/naming.html#shortvariable",
                    "priority": 3
                }
            ]
        }
    ]
}

So I can see STDIN is working as expected.
Please provide full steps to reproduce the issue if you still have troubles and I'll re-open.
Thanks.

@kylekatarnls I'm getting some very confusing results with this, too.
I copied your test.php file from above, and ran it with this phpmd.xml file
<?xml version="1.0"?>
<ruleset name="Project Code Standards"
         xmlns="http://pmd.sf.net/ruleset/1.0.0"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://pmd.sf.net/ruleset/1.0.0
                     http://pmd.sf.net/ruleset_xml_schema.xsd"
         xsi:noNamespaceSchemaLocation="
                     http://pmd.sf.net/ruleset_xml_schema.xsd">
    <description>
        Rules to apply (or not apply) for said project
    </description>

    <rule ref="rulesets/cleancode.xml">
    </rule>

    <rule ref="rulesets/codesize.xml">
    </rule>

    <rule ref="rulesets/controversial.xml">
    </rule>

    <rule ref="rulesets/design.xml">
    </rule>

    <rule ref="rulesets/naming.xml">
    </rule>

    <rule ref="rulesets/unusedcode.xml">
    </rule>
</ruleset>

When I ran phpmd normally (i.e. not trying to use STDIN), I got these results:
./vendor/bin/phpmd test.php text phpmd.xml
/home/geraint/workspace/sandbox/phpmd-test/test.php:7  ShortVariable        Avoid variables with short names like $a. Configured minimum length is 3.
/home/geraint/workspace/sandbox/phpmd-test/test.php:7  UnusedLocalVariable  Avoid unused local variables such as '$a'.

However, when I ran phpmd using STDIN, I got some very strange results indeed:
cat test.php | ./vendor/bin/phpmd - text phpmd.xml
php://stdin:50   LongVariable    Avoid excessively long variable names like $redactedVariableName1. Keep variable name length under 20.
php://stdin:57   LongVariable    Avoid excessively long variable names like $redactedVariableName2. Keep variable name length under 20.
php://stdin:156  ShortVariable   Avoid variables with short names like $e. Configured minimum length is 3.
php://stdin:169  LongVariable    Avoid excessively long variable names like $redactedVariableName3. Keep variable name length under 20.
php://stdin:182  ElseExpression  The method redactedMethodName1 uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
php://stdin:224  ElseExpression  The method redactedMethodName2 uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
php://stdin:483  ElseExpression  The method redactedMethodName3 uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

(Note: I have edited the variable and method names in the output)
This is strange, because that input is from a much bigger file that is in an entirely separate project.
I had been running phpmd against that other file, in my efforts to get phpmd to work with null-ls.nvim - it seems to be weirdly cached.
I got the same results, even when I did not pipe anything into phpmd:
./vendor/bin/phpmd - text phpmd.xml
php://stdin:50   LongVariable    Avoid excessively long variable names like $redactedVariableName1. Keep variable name length under 20.
php://stdin:57   LongVariable    Avoid excessively long variable names like $redactedVariableName2. Keep variable name length under 20.
php://stdin:156  ShortVariable   Avoid variables with short names like $e. Configured minimum length is 3.
php://stdin:169  LongVariable    Avoid excessively long variable names like $redactedVariableName3. Keep variable name length under 20.
php://stdin:182  ElseExpression  The method redactedMethodName1 uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
php://stdin:224  ElseExpression  The method redactedMethodName2 uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.
php://stdin:483  ElseExpression  The method redactedMethodName3 uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

After some head scratching, I narrowed the problem down to a folder called ~/.pdepend.
If I rm -rf it, things start behaving sanely.
To reproduce the problem, take your test.php file, and run the following:
cat test.php | ./vendor/bin/phpmd - text phpmd.xml
php://stdin:7  ShortVariable        Avoid variables with short names like $a. Configured minimum length is 3.
php://stdin:7  UnusedLocalVariable  Avoid unused local variables such as '$a'.

Now, edit the file, and change $a to $b, and run phpmd again:
cat test.php | ./vendor/bin/phpmd - text phpmd.xml
php://stdin:7  ShortVariable        Avoid variables with short names like $a. Configured minimum length is 3.
php://stdin:7  UnusedLocalVariable  Avoid unused local variables such as '$a'.

Note that phpmd is still reporting $a as the problem, even though that variable no longer exists.
Now, delete the .pdepend folder, and run phpmd again:
rm -rf ~/.pdepend
cat test.php | ./vendor/bin/phpmd - text phpmd.xml
php://stdin:7  ShortVariable        Avoid variables with short names like $b. Configured minimum length is 3.
php://stdin:7  UnusedLocalVariable  Avoid unused local variables such as '$b'.

phpmd is now correctly reporting a problem with $b and not $a.
This problem doesn't occur if I use a filename, rather than STDIN.
Unless this is somehow , it seems like STDIN is not working as expected, and this bug should still be open.
FWIW, I've seen this problem this using:

PHP 7.4.27, 8.0.16, and 8.1.1
PHPMD 2.11.1
Running on Windows 10 / WSL2 / Ubuntu 20.04


For specific commands, they should not be translated into Chinese

English per google translate:
For specific commands, they should not be translated into Chinese
For example: dotnet ef database is deleted, obviously dotnet ef database drop should be kept
Original:
对于具体的命令，不应该翻译为中文
例如： dotnet ef 数据库删除 ，很明显应该保留 dotnet ef database drop

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: 4d1bdeea-0ebc-04ff-5b58-1020ad9cd72a
Version Independent ID: 847fe92a-2940-afbf-b2ee-e68f8a8060be
Content: EF Core 工具参考 ( .NET CLI) -EF Core
Content Source: entity-framework/core/cli/dotnet.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: avickers

@ajcvickers I'm going through the localized repro's and transferring here as appropriate. Let me know if that's OK.

Related https://github.com/dotnet/EntityFramework.Docs/issues/3068


OAuth2 Login Redirects back with no permissions

Hi,
I'm attempting to implement the OAuth2 settings for IdentityManager. I'm using my own idsvr3 host.
Running as local machine works like a charm, and I added a user with the default admin role (IdentityManagerAdministrator).
When I click the login link, it redirects to idsvr3 correctly. I login, and it then redirects me back to idmgr. However, it only gives me the login link, none of the admin links.
If I login as a user without the admin role, it behaves in the same way.
I think my configuration may be at fault.
      var factory = new Thinktecture.IdentityManager.Host.MembershipRebootIdentityManagerFactory("IdentityManagerMembershipReboot");

            app.UseIdentityManager(new IdentityManagerConfiguration()
            {

                IdentityManagerFactory = factory.Create,
                 AdminRoleName= "IdentityManagerAdministrator",
                SecurityMode =   (IdentityManager.SecurityMode) Enum.Parse(typeof(IdentityManager.SecurityMode), Properties.Settings.Default.SecurityMode),

                      OAuth2Configuration = new OAuth2Configuration
                    {
                        AuthorizationUrl = Properties.Settings.Default.AuthorizationUrl ,
                        Issuer = Properties.Settings.Default.IssuerUri,
                        Audience = "https://idsrv3.com/resources",
                        ClientId = "implicitclient",
                        SigningCert = Cert.Load(),
                        Scope = "idmgr",

                        //PersistToken = true,
                        //AutomaticallyRenewToken = true
                    }

            });

The above config settings are:
AuthorizationUrl = https://localhost:44333/core/connect/authorize<
SecurityMode = OAuth2
IssuerUri=Same as the one specified in Idsvr3.
It seems as though the token coming back from idsvr3 can't be read by idmgr.
The eveng log has the below message. So something is getting returned, but I don't know if it can be interpreted.
Any ideas?
[Thinktecture.IdentityServer.Core.Results.AuthorizeRedirectResult]: 25/11/2014 12:18:43 AM -- Redirecting to: http://localhost/IdentityManager/#/callback#access_token=eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsIng1dCI6ImEzck1VZ01Gdjl0UGNsTGE2eUYzekFrZnF1RSIsImtpZCI6ImEzck1VZ01Gdjl0UGNsTGE2eUYzekFrZnF1RSJ9.eyJjbGllbnRfaWQiOiJpbXBsaWNpdGNsaWVudCIsInNjb3BlIjoiaWRtZ3IiLCJzdWIiOiIzMDZmODc2Zi0wMDEwLTQxYWEtYTEyYy03MzA4YTk0YjM2MDMiLCJhbXIiOiJwYXNzd29yZCIsImF1dGhfdGltZSI6MTQxNjg3NDcyMywiaWRwIjoiaWRzcnYiLCJuYW1lIjoiZXdlcmV3ciIsInJvbGUiOiJJZGVudGl0eU1hbmFnZXJBZG1pbmlzdHJhdG9yIiwiaXNzIjoiaHR0cHM6Ly9zZW1hZ3JvdXAuY29tLmF1IiwiYXVkIjoiaHR0cHM6Ly9zZW1hZ3JvdXAuY29tLmF1L3Jlc291cmNlcyIsImV4cCI6MTQxNjg3NTA4MywibmJmIjoxNDE2ODc0NzIzfQ.fTYIsdV9reIkLrOEJM0zsCV0L-gq6dkvFFrzMM0CYizjStG5JWjuqKivnSASn0CW8Jkcdb_vFWypK-VSzHcSuZ09AqYPzBBzeRk1xHSikK_JfYu75ESGcf0QTbgtsuFFs7KLnkaMjyj65JsVNiUNKeF8AQ9vFDxCyeHfp15aw2QBLC44fKz0KbEcK9C9u3d_VTVgr6Y5b7Ps0ocqLBcZYite-s7ah5F3ppQa2lugRmGwG4bjZJtv8Rt0eoet7iKqL5rjbTd_tjeg8QULBpt9Y50I9Ef33hNheDv6eik254EM_dIF3ltWsStQaxcdAqP0LxSWGdmGkEDSN0P7JJQnGA&token_type=Bearer&expires_in=360&scope=idmgr&state=8951963276200745

I have more details on the behaviour of this issue.
When I login the first time with a valid user, it shows the "Logout" link correctly before it shows the other links (eg. New User etc). Once the other links display, the "Logout" link is replaced with "Login"
If I hit Login again, sometimes it will change to logout and I will have access. Otherwise it remains as login. On different environments, I can hit login up to 10 times before it will be reflected. Not sure if this is an IDM issue or not.

Is there any update?
I'm stuck in "SecurityMode = SecurityMode.OAuth2" too.. :(

Hi @hungvq , I never got this working. My requirements for our usermanagement include quite a bit of custom functionality, so I had to write my own thing on top of Membership Reboot. I then secured this site using Identity Server.

Thank for your infor @parkinsona.
So, I have to build my own one.
BTW, https://contacts.google.com is first good idea :)
(IM can not change the UI).

You shouldn't need to build your own. I had to because of our custom requirements.

Just so everyone's clear -- I'm fully aware of the current complexity in setting up IdMgr for external authentication. I wish it were easier, but given the security concerns we really need to use tokens. I'm working this week (in fact) on trying to make it easier.

This issue should be moot now, given the rework I did today. The security model can be configured to use the host's authentication (OIDC, local logins, ADFS, whatever). I'll close for now and once you shift to the new model let me know if new issues arise.


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In a934585, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 10440 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in 9e95184 after 1 hour, 32 minutes.


Cant price AMBO on polygon

Seeing issues pricing AMBO (0x9B8B6A1298d34B3c4bBDDce8a7fF0149121592C1) on polygon.
There is a quickswap pool (AMBO/USDC native) - https://dexscreener.com/polygon/0x6e8e0683e3e98302e2d44c44e191e4c47a72590e
but getting a 0 price back even after adding native as a custom connector

Thank you for highlighting the issue.
The problem lies in the QuickSwap factory used in SpotPriceAggregator. This factory is a contract that is a fork of UniswapV2, and indeed, the pool with AMBO is not present in this factory.
This pool is in a different factory, which likely belongs to a newer version of the QuickSwap. Therefore, a new oracle needs to be added for QuickSwap. I have created a task for this.
If you have the time and willingness, you are welcome to contribute to this update.

This pool quickswap v3, where it looks like the current oracle is quickswap v2.
Might be as simple as deploying a uniswap v3 like oracle with the correct params. I'll can take a stab at finding these params

Quickswap v3 actually forks Algebra Integral contracts, docs on the difference between uniswap v3.
2 important differences:

pool address derivation logic is different
only 1 pool per pair

Thinking I need to create a new AlgebraLikeOracle.sol, which will look similar to UniswapV3LikeOracle.sol, but with an updated _getPool function and getRate wont need to loop through fees[].

Thank you for your research! This saved me time!
This is my cry of the soul - why they changed slot0() to globalState() in pools 😭
I will do this in the coming days as soon as I decide the best approach – whether to add a separate oracle with duplicate code or to make changes to the UniswapV3LikeOracle.

Great! Thanks for building and maintaining this repo, these contracts are amazing at getting accurate reference prices

It's already fixed. Thank you for the improvement suggestion!


Using DevTools after injecting a script

I have a use case where I am using chrome-remote-interface with chrome-launcher to inject a script into a page (using Runtime.evaluate). That works great!
However, I would like to be able to debug the injected script if needed, preferably via Chrome's built-in DevTools. I've tried using the Debugger API, which will allow me to properly "break" on a debugger statement, but upon opening the DevTools, it appears the chrome-remote-interface connection is broken.
Based on https://github.com/cyrus-and/chrome-remote-interface/issues/213#issuecomment-317695406, it sounds like you can't use both DevTools and chrome-remote-interface simultaneously. Is this accurate? And, if so, is there a recommend approach to debugging injected scripts?
Thanks in advance! This library is awesome 💯

Component
Version

Operating system
macOS 10.12.3

Node.js
8.2.1

Chrome/Chromium/...
59.0.3071.115

chrome-remote-interface
0.24.2

Hey, thanks! :)
Correct, Chrome doesn't allow multiple connections to a single tab (yet? they're working on this apparently...), from the main page:

We currently do not support multiple clients connected to the protocol simultaneously. This includes opening DevTools while another client is connected. [...] After disconnection, some apps have chosen to pause their state and offer a reconnect button.

There are a couple of workarounds to this:

support reconnection on your side, something that after disconnection tries periodically to fetch the target info (CDP.List) for a given id and attempt a new connection if the webSocketDebuggerUrl field is present (when already inspected such field is omitted);

use chrome-remote-multiplex by @johnspackman as suggested in #186. I'm not sure whether there could be problem with certain domains though, for example the DOM domain:

It is important that client receives DOM events only for the nodes that are known to the client. Backend keeps track of the nodes that were sent to the client and never sends the same node twice. It is client's responsibility to collect information about the nodes that were sent to the client.

Thanks for the detailed response. Looks like chrome-remote-multiplex might be the approach I want for now!

https://github.com/johnspackman/chrome-remote-multiplex is built to only forward those domain events which are relevant and provide an emulation rather than just a verbatim copy ... however this is currently limited to the Runtime domain.  I'll add an issue for this and support for DOM event filtering shortly


Add sleeps in spec before and after signing in quiz participant

Hopefully this will reduce flakiness. A screenshot showed the login screen one time when this spec flaked.

Closing this for a better branch name and a better approach.


Deprecate masternodeaddr

i.e. remove dash.conf-started remote masternode support

@crowning- masternode.conf is deployed on control wallet only usually, so I see no reason to remove masternodeprivkey.

@schinzelh my v0.12.1.x Masternode on testnet (dashd-only) still has the example masternode.conf deployed. Will this change in the future?

Empty masternode.conf is always generated if there is none but I would avoid using it for hot masternode. Basically in any setup:
"hot" wallet/masternode == dash.conf + masternode=1 + masternodeprivkey=<asdasdadad>
"cold" wallet/control node == masternode.conf

Yep, that's my default setup for hosting.
Hosted node has dash.conf parameters while I send masternode.conf to the customer. Node40 does the same.


feat: add gas fee when getCallParameters return gas value undefined

Description
I connecting to Metamask using walletconnect-legacy-provider, then using the writeContract function, the function calling the transaction failed. As a result of figuring out the cause, the prepareWriteConfig function was returning gasPrice as undefined. I decided that this was an issue of WalletConect, and got a hint from pr(https://github.com/wagmi-dev/wagmi/pull/2344) below and added logic to find a new gas fee using publicClient when the gas fee is undefined.
If it is not fixed in this way, I would appreciate it if you could tell me how to solve the problem that gasPrice returns undefined in prepareWriteContract when using walletconnect legacy.
Then have a nice day. Fighting wagmi team~
Additional Information

[X] I read the contributing docs (if this is your first contribution)

Your ENS/address: 0x05C844b78d17E3b4be53eEBb72ec5Bcd32f85ad6

Thanks for the PR! This was left out intentionally as Wallets will estimate the gas needed for a transaction to execute themselves. I tried to reproduce your issue with MetaMask + WalletConnect v1, however, I was unable to reproduce it and was successfully able to write to a contract. Do you mind providing a minimal reproducible example via https://new.wagmi.sh so I can look into it further?
If you really need to retrieve a gas estimate, you can consider using viem's estimateContractGas. We will consider adding this as a Hook into wagmi soon!


react-tools -> babel <3

Swap out the use of react-tools for babel. The main reason for doing this is the way babel polyfills at a library level rather than needing it globally, meaning you won't need anyone to change anything about your environment to use fixed-data-table.
This solves the common Object.assign doesn't exist issue.

@JedWatson Im using your babel-plugin-object-assign project here :) Unfortunately i couldn't use the project as a dependency as it was giving me an error about using two different instances of Babel, i think your project might need to use peerDeps to solve this but not sure. Anyway thanks for creating this!

Love this! Waiting for it!

Any progress on merging this? We're some ES6/babel users eagerly waiting to use it.

@ifrit :) I have merged it in, we are aiming to do the release next week.

Excellent news! thanks @pieterv!

@pieterv glad to hear it! sorry I missed the first notification - let me know if you're still having issues, I've been working to try and resolve the two-babels problem (and a couple of others) recently.

@pieterv any update on doing a release with this :)?

Yeah, that would be great. I tried pulling it from github directly and running the build scripts but couldn't get it to render properly.

Same, I need this!

Hey @cesarandreu, @ifrit, @zackify sorry for the delay but we have now done the v0.2.0 release, with this code in it! Check it out, and please let us know if something doesnt work for you :)

Awesome, thanks @pieterv! Seems to be working.


How does Lightgbm predicts for new categories on the test set

Hi,
I'm working on a problem in which I have new categories for some features on my test set, that are not in my training set.
When I'm trying to predict on those features, I'm still getting a result, and not an error as I would expect.
So my question is what does the prediction function do in such cases?
How does it return a prediction?
I'm adding a python code example, to make my question more clear.
import numpy as np
import pandas as pd
import lightgbm as lgb
np.random.seed(1)
X1 = np.random.choice(["aaaaaa","bbb","ccc","ddd"],size=15)
X2 = np.random.choice(["ahhhm","avocado","eggplant"],size=15)
X1[14] = "other"
X2[14] = "other"
X = pd.DataFrame({'X1': X1, 'X2': X2}).apply(lambda col: col.astype('category'))
X_train = X.head(14)
Y_train = np.random.choice([0,1],size=14)
X_test = X.tail(1)
lgb_dat_train = lgb.Dataset(X_train,label = Y_train,categorical_feature = X_train.columns.tolist())
params = {
'objective': 'binary',
'metric': 'binary_logloss',
'learning_rate': 0.01,
'min_data_per_group' : 1,
'num_iterations' :1,
'num_leaves' :4,
'min_data_in_leaf' :1
}
gbm = lgb.train(params=params,train_set=lgb_dat_train)
gbm.predict(X_test)
output: array([0.71213804])
Thanks a lot,
Matan

LightGBM only uses the seen categories for prediction.
For example, the trained split point may is  if is A or is B then left child, then for an unseen category, it will be in the right child.


SAM-2705 Export assessments to markup text

Recent conflicts are resolved

Branch conflicts... can someone rebase?

Conflicts resolved

Modified the code with the suggestions

I've created constants to get permissions in SamigoConstants class. They should be used in others classes instead of being hard coded. For instance, AssessmentBean class.

I think you just missed one opportunity to use StringUtils, and my one other comment about the braces. Other than that, this is looking good. Thanks @annuelo for addressing these so quickly. Also, thank you for introducing those constants. Now that they're in place, they can be used in other places as you mentioned (for future PRs).

Done!

Thanks @annuelo! LGTM


OnRep_PlayerState CreateHUD can fail if Player state has replicated on GSHeroCharacter.cpp before the player controller

Only noticed this when I was testing my code on dedicated server that I had set up with player state with the GAS components on it as the example, but guess it applies to yours as well.
When you call PC->CreateHUD(); line 855 GSHeroCharacter.cpp we know we have a valid Player state as it's called from OnRep_PlayerState and we also check the cast.
However On dedicated server I think there is a timing issue and in more real world situation I was getting a lot of occasions where the check for Player state in void AGSPlayerController::CreateHUD()  (line 35) GSPlayerCOntroller.cpp was nullptr even tho it was replicated in the base character so the hud would create but not populate with the initial replicated values.
As the calling function was always guaranteed to have player state as it was called OnRep_PlayerState I updated the code to just pass a reference to that player state over to the player controller and not rely on the player controller also replicating at the same time and having it available to set the initial HUD values.
so just updated to
void CreateHud(class AGSPlayerState* PS);
and always passing it then it works every time.
Only small tweak but it was really obscure and seemingly random so thought I'd share just in case it was worth updating and it was small update.
I also tweaked the HUD class so you explicitly call start listening for attributes from C++ via blueprint event rather than having it start on HUD construct so I could really make sure initial replication had happened first before listening for more changes.
thanks again for the resource, learning so much from the code base.

Add a parameter is good, but by the way, why not use PlayerController->GetPawn() and then use pawn->GetPlayerState()? Anyway pawn is retrieved here

Passing a parameter sounds like a good approach to this race condition. Feel free to make a PR with that change if you have the spare cycles.


Add google analytics scripts to head.hbs

https://rust-lang.github.io/mdBook/format/configuration/renderers.html?highlight=analytics#html-renderer-options

Thank you for setting this up, @geoknee!
To give context on this edit, there is a current data push to set up more formalised tracking on both the developer documentation and the stack specifications with the goal of understanding developer and collaborator activity. The starting point is to work on creating a baseline to better understand how users find these resources and how they typically use them. Additionally, these tools also allow the team to better track  errors and remain on top of any issues with the sites during usage.
All this data is being collected with the key point of better informing improvements and updates to address documentation gaps or pain points. This data will also used for the team to understand the rating of our pages in terms of accessibility and usability, particularly for mobile sites. This is all part of my larger technical content strategy being led by , I'm trying to get this set up for all of my like the main touch points.


Link to other open science initiatives and consider overlap

https://openlifesci.org/openseeds/

https://github.com/openmodelingfoundation/openmodelingfoundation.github.io


pad_sequences bug in Batch

hi,dear
want to use the pad_sequences in batch ,but got the bug down
>>> inputs=tf.keras.Input(shape=(None,),dtype=tf.int32)
>>> inputs
<tf.Tensor 'input_1:0' shape=(None, None) dtype=int32>
>>> tf.keras.preprocessing.sequence.pad_sequences(inputs)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/../python3.8/site-packages/tensorflow/python/keras/preprocessing/sequence.py", line 156, in pad_sequences
    return sequence.pad_sequences(
  File "/../python3.8/site-packages/keras_preprocessing/sequence.py", line 57, in pad_sequences
    num_samples = len(sequences)
  File "/../python3.8/site-packages/tensorflow/python/framework/ops.py", line 752, in __len__
    raise TypeError("len is not well defined for symbolic Tensors. ({}) "
TypeError: len is not well defined for symbolic Tensors. (input_1:0) Please call `x.shape` rather than `len(x)` for shape information.

could pls help me ?
thx

@ucasiggcas
In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

class Model(tf.keras.Model):
    def __init__(self, maxlen=10):
        super(Model,self).__init__()
        self.maxlen=maxlen
    def call(self,inputs):
        seqs_pad=tf.keras.preprocessing.sequence.pad_sequences(inputs)
        return seqs_pad

def gen():
    seqs=[np.random.randint(0,1000,size=10).tolist() for k in range(5,15)]
    yield seqs
#batch_size=10,dim=10,
dataset=tf.data.Dataset.from_generator(gen,output_types=(tf.int32),
                                             output_shapes=((10,10)))
model=Model()
@tf.function
def train_step(inputs):
    with tf.GradientTape() as tape:
        seqs_pad=model(inputs)
        print(seqs_pad)

for inputs in dataset:
    train_step(inputs)

bug
TypeError: in user code:

    pad_sequences_issue_.py:23 train_step  *
        seqs_pad=model(inputs)
    pad_sequences_issue_.py:10 call  *
        seqs_pad=tf.keras.preprocessing.sequence.pad_sequences(inputs)
    /../python3.8/site-packages/tensorflow/python/keras/preprocessing/sequence.py:156 pad_sequences  **
        return sequence.pad_sequences(
    /../python3.8/site-packages/keras_preprocessing/sequence.py:57 pad_sequences
        num_samples = len(sequences)
    /../python3.8/site-packages/tensorflow/python/framework/ops.py:851 __len__
        raise TypeError("len is not well defined for symbolic Tensors. ({}) "

    TypeError: len is not well defined for symbolic Tensors. (inputs:0) Please call `x.shape` rather than `len(x)` for shape information.

@ucasiggcas I tried to run your code and faced this issue while reproducing. In order to reproduce the issue reported here, could you please provide the complete code and the dataset , tensorflow version you are using. Please refer to the link and let us know if it helps? Thanks!

No, have tried
could not be useful to my code

@ucasiggcas Thanks for the update! Could you please have a look at the gist and let us know if it is the way to reproduce the issue reported here ? Please let me know if I am missing something while reproducing. Thanks!

hi
you forget import np
should need the code
import numpy as np
then will reproduce the error

@jvishnuvardhan Was able to reproduce the issue on colab using TF v2.6 and tf-nightly ,please find the gist here  for reference .Thank you!

yes
it's the problem
thx

Hi there, pad_sequences is only for NumPy data, not for TF tensors, hence the error. Prefer using tf.pad https://www.tensorflow.org/api_docs/python/tf/pad


Require email confirmation setting not ignored when domain restriction is in place

BookStack Version: 0.18.4
PHP Version: 7.1
MySQL Version: MariaDB 10.2

This setting does not do what it says.

If domain restriction is used then email confirmation will be required and the below value will be ignored.

However, I believe that the current behavior is correct and should be kept.  The text should be corrected/removed.  Additionally, the signup process should not prompt the user that email conformation is pending.
Expected Behavior
User log in is blocked, pending email confirmation
Current Behavior
User is able to log in.
Steps to Reproduce
Domain restriction is enabled.
Require email confirmation is disabled
User signs up, gets prompted to confirm email.
User attempts to log in

Thank you for raising this issue.
I think its important that email confirmation is forced, At least for now, otherwise the domain restriction is essentially useless and it ensures a security-first approach.
The only place I can think where not-forcing email would be ideal is when only using social authentication options which is not yet a possible feature.