Bug: TypeError: Class constructor TextNode cannot be invoked without 'new'

I get this error after compiling my wrapper code so that I can use the same editor on multiple projects easily.
So i followed this example: https://lexical.dev/docs/concepts/serialization#handling-extended-html-styling
which is what seems to be causing the issue.
https://github.com/The-Code-Monkey/TechStack/blob/main/packages/lexical/src/index.tsx#L54
Lexical version:
Steps To Reproduce

import @techstack/lexical into a repo
after adding it you get this error

Link to code example:
https://codesandbox.io/p/sandbox/nice-microservice-yq586s?file=%2Fsrc%2Findex.js
The current behavior
There is an error
The expected behavior
Either something is wrong with the docs or the Lib needs updating as it should replace the element

I suspect this is your problem: https://stackoverflow.com/questions/51860043/javascript-es6-typeerror-class-constructor-client-cannot-be-invoked-without-ne
Have you tried targeting a later version of javascript in your transpilation?

My tsconfig is targeting esnext not es6

Also it builds fine it's just when I use it in another package

My tsconfig is targeting esnext not es6

Ah, I see - I missed the base config you extended.

Also it builds fine it's just when I use it in another package

I'd expect the transpilation to work fine, anyway. You'd see a runtime error (which is what you're seeing).
Do you think you could provide a stack trace? I can't repro this in the playground with the same setup (another thing pointing to an environment/build issue).

@acywatson I've posted a minimal reproduction using code sandbox issue happens there for me, anything that is currently on the GitHub link I sent is that package version that's in the code sandbox

Another thing to note I have tested it locally using storybook and all is fine. All I can guess is that my rollup build command is breaking something but I wouldn't have thought so.

Hmm - one quick test could be to search the built files for "TextNode.call" and see if that's happening anywhere.

Hmm ok I'll try that tomorrow.

@acywatson there is a textnode.call in the cjs built file but only within the extendedTextNode constructor where you would expect it to be.
I cannot find the same thing in the esm version.

@acywatson there is a textnode.call in the cjs built file but only within the extendedTextNode constructor where you would expect it to be.
I cannot find the same thing in the esm version.

I might be getting confused - should there be a TextNode.call? I think the whole source of the error is that TextNode is an ES6 class and therefore must be instantiated with new, as opposed to using Function.prototype.call.

Switching to tsc fixed this
On Fri, 1 Mar 2024, 19:32 Acy Watson, @.***> wrote:

@acywatson https://github.com/acywatson there is a textnode.call in the
cjs built file but only within the extendedTextNode constructor where you
would expect it to be.
I cannot find the same thing in the esm version.
I might be getting confused - should there be a TextNode.call? I think the
whole source of the error is that TextNode is an ES6 class and therefore
must be instantiated with new, as opposed to using Function.prototype.call.
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/lexical/issues/5647#issuecomment-1973795686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHOFWX57OYD23Q34L3U4T4LYWDJT3AVCNFSM6AAAAABDVACW3OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZTG44TKNRYGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


[Reactor] ThreadLocals are not propagated to reactor-netty threads

Since reactor-netty does not use schedulers from Reactor, Sleuth's executor service decorator does not handle the tasks submitted to / originated from reactor-netty.
The usage of onEachOperator hides it, because it wraps onNext and, when an item comes from a thread where ThreadLocals were not set, it sets them.

reproducer:
https://github.com/jumal/sleuth-webflux-no-trace-context-issue
works with M1, fails with M2+

This won't be a problem with the latests snapshots and the next ilford release. Here (https://github.com/jumal/sleuth-webflux-no-trace-context-issue/pull/1) you can see how we can leverage the WebFluxSleuthOperators to retrieve the tracing context and pass it to the BaggageField API to retrieve the baggage.

Done via https://github.com/spring-cloud/spring-cloud-sleuth/commit/1c33332320e0516e9dcf4fa3b06f642bb24ac2cf


Set up continuous integration for site deployment

Develop branch builds and deploys https://dev.yearn.finance
Master branch builds and deploys https://yearn.finance

commit b0b5aeb3201f4a70b0e2c89f6d945a04b80565a3
Author: Arber Xhindoli<EMAIL_ADDRESS>Date:   Sat Oct 17 19:19:42 2020 -0400

    feat: create production deployment action (#148)

commit<PHONE_NUMBER>3e4209d5d84b57718a132fb3ec9130 (HEAD -> develop, origin/develop)
Author: x48<EMAIL_ADDRESS>Date:   Sat Oct 24 22:21:05 2020 -0500

    ci: update branches and CloudFront distribution ID in CI deployment scripts (#149)


Game broken

When attempting to access this at https://githubschool.github.io/github-games-Rebvos/, I am getting a 404. This could be caused by a couple things:

GitHub pages needs to be enabled on master. You can fix this in the repository settings.
the index.html file is incorrectly named inde.html. We will fix this together in class.

Can you please fix the first bullet @Rebvos?

page has been enabled.

The game is still broken re-opening to investigate

fixed


Unable to split ruby hashes containing strings with more than one space

SplitJoin has trouble splitting hashes that have values that contain more than one space.
a_hash = { a_key: "a value" } # succeeds
a_hash = { "a key" => "a value" } # succeeds
a_hash = { a_key: "a longer value" } # fails
a_hash = { "a longer key" => "a value" } # fails
a_hash = { :yet_another_key => "yet another-value" } # succeeds
a_hash = { :yet_another_key => "yet another value" } # fails

Good catch. The plugin was detecting method calls within strings, it seems. I've pushed a potential fix to master. Could you try it out and let me know if it seems to work?

Hi Andrew - I'm afraid it doesn't appear to fix it.  I've cloned your repo to my local machine and loaded it using Vundle:
Plugin 'file:///Users/daveok/.vim/bundle/splitjoin.vim'
The HEAD for the repo is pointing at:
30d7eae Don't detect function calls within strings
The plugin itself works as before, but it still suffers the issue on the 'fails' lines of the test code.

Hm, that's odd. That's the correct hash, and it definitely works on my machine, while it wasn't working before that commit.
Is it at all possible that there's two instances of the plugin somehow? Could you try putting a finish at the top line of the plugin/splitjoin.vim file, to check that splitjoin stops working, so it's definitely the plugin in ~/.vim/bundle/splitjoin.vim that's active?
If this is definitely not fixing the issue for you, I guess I'll have to make a debug branch and add some extra output to figure out what's different for your configuration compared to mine.

The problem was I needed to close all my mvim instances - splitjoin.com checks to see if it's already been loaded, and if it has, it finishes.
And now that I have it loading properly, I can see that yes, it does indeed fix it!  Thanks a mill Andrew!

Good, glad it's been resolved :)


Can't get cube source record size.

Dear All!
I deploy kylin on Apache Hadoop, when i build the job ,  it failed on the last step. here is the some log message:
[QuartzScheduler_Worker-4]:[2014-11-24 19:24:47,280][INFO][com.kylinolap.job.flow.JobFlowListener.jobWasExecuted(JobFlowListener.java:93)] - cube_job_group.test_kylin_cube_without_slr_empty.30bbd69c-33ae-4983-aaf7-f46ab42b88dd.14 status: FINISHED
[QuartzScheduler_Worker-4]:[2014-11-24 19:24:47,280][DEBUG][com.kylinolap.cube.CubeManager.loadCubeInstance(CubeManager.java:604)] - Loading CubeInstance kylin_metadata_qa(key='/cube/test_kylin_cube_without_slr_empty.json')@kylin_metadata_qa@hbase:<IP_ADDRESS>:2181:/hbase9827
[QuartzScheduler_Worker-4]:[2014-11-24 19:24:47,295][DEBUG][com.kylinolap.common.persistence.ResourceStore.putResource(ResourceStore.java:166)] - Saving resource /job/30bbd69c-33ae-4983-aaf7-f46ab42b88dd (Store kylin_metadata_qa@hbase:<IP_ADDRESS>:2181:/hbase9827)
[QuartzScheduler_Worker-4]:[2014-11-24 19:24:47,300][INFO][com.kylinolap.job.flow.JobFlowListener.updateCubeSegmentInfoOnSucceed(JobFlowListener.java:238)] - Updating cube segment FULL_BUILD for cube test_kylin_cube_without_slr_empty
[QuartzScheduler_Worker-4]:[2014-11-24 19:24:47,301][ERROR][com.kylinolap.job.flow.JobFlowListener.jobWasExecuted(JobFlowListener.java:117)] - Can't get cube source record size.
java.lang.RuntimeException: Can't get cube source record size.
        at com.kylinolap.job.flow.JobFlowListener.updateCubeSegmentInfoOnSucceed(JobFlowListener.java:274)
        at com.kylinolap.job.flow.JobFlowListener.jobWasExecuted(JobFlowListener.java:99)
        at org.quartz.core.QuartzScheduler.notifyJobListenersWasExecuted(QuartzScheduler.java:1985)
        at org.quartz.core.JobRunShell.notifyJobListenersComplete(JobRunShell.java:340)
        at org.quartz.core.JobRunShell.run(JobRunShell.java:224)
        at org.quartz.simpl.SimpleThreadPool$WorkerThread.run(SimpleThreadPool.java:573)

here is some version info:
Kylin 0.6.2
hadoop 2.4.1
hbase 0.98.4-hadoop2
zookeeper 3.4.5
hive 0.13.0
I find some config info on hbase
get 'kylin_metadata_qa_job', '/job/1eaf05f8-e9f5-4dd7-9748-ab9b3098a512'
{
    "uuid": "1eaf05f8-e9f5-4dd7-9748-ab9b3098a512",
    "name": "test_kylin_cube_without_slr_empty - FULL_BUILD - BUILD - PST 2014-11-24 23:00:15",
    "type": "BUILD",
    "duration": 461,
    "steps": [
        {
            "name": "Create Intermediate Flat Hive Table",
            "info": null,
            "interruptCmd": null,
            "sequence_id": 0,
            "exec_cmd": "hive -e "DROPTABLEIFEXISTSkylin_intermediate_test_kylin_cube_without_slr_desc_FULL_BUILD_1eaf05f8_e9f5_4dd7_9748_ab9b3098a512;nCREATEEXTERNALTABLEIFNOTEXISTSkylin_intermediate_test_kylin_cube_without_slr_desc_FULL_BUILD_1eaf05f8_e9f5_4dd7_9748_ab9b3098a512n(nCAL_DTdaten,
            LEAF_CATEG_IDintn,
            LSTG_SITE_IDintn,
            META_CATEG_NAMEstringn,
            CATEG_LVL2_NAMEstringn,
            CATEG_LVL3_NAMEstringn,
            LSTG_FORMAT_NAMEstringn,
            SLR_SEGMENT_CDsmallintn,
            PRICEdecimal(38,
            16)n,
            SELLER_IDbigintn)
...}
]

in source code: com.kylinolap.job.flow.JobFlowListener.updateCubeSegmentInfoOnSucceed(JobFlowListener.java:272)
               JobStep createFlatTableStep = jobInstance.findStep(JobConstants.STEP_NAME_CREATE_FLAT_HIVE_TABLE);
                if (null != createFlatTableStep) {
                    String sourceRecordsSize = createFlatTableStep.getInfo(JobInstance.SOURCE_RECORDS_SIZE);
                    if (sourceRecordsSize == null || sourceRecordsSize.equals("")) {
                        throw new RuntimeException("Can't get cube source record size.");
                    }
                    sourceSize = Long.parseLong(sourceRecordsSize);
                } else {
                    log.info("No step with name '" + JobConstants.STEP_NAME_CREATE_FLAT_HIVE_TABLE + "' is found");
                }

the "info" is null, cased throw a RuntimeException,  some one have idea with it??

I had this problem, and it turned out it was because i had configured hive to use tez, not mr.

Hi,
I'am trying to build my first cube using Apache Kylin, everything goes fine until last step where I'm getting error:
java.lang.IllegalStateException: Can't get cube source record count.
at com.google.common.base.Preconditions.checkState(Preconditions.java:149)
at org.apache.kylin.job.cube.UpdateCubeInfoAfterBuildStep.doWork(UpdateCubeInfoAfterBuildStep.java:104)
at org.apache.kylin.job.execution.AbstractExecutable.execute(AbstractExecutable.java:107)
at org.apache.kylin.job.execution.DefaultChainedExecutable.doWork(DefaultChainedExecutable.java:50)
at org.apache.kylin.job.execution.AbstractExecutable.execute(AbstractExecutable.java:107)
at org.apache.kylin.job.impl.threadpool.DefaultScheduler$JobRunner.run(DefaultScheduler.java:132)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)

According to this issue above error occurs because Kylin attempt to find this pattern in hive's output: "HDFS Read: (\d+) HDFS Write: (\d+) SUCCESS".
Correct output from cube building tutorial: github.com/KylinOLAP/Kylin/wiki/Kylin-Cube-Build-and-Job-Monitoring-Tutorial:
http://i.imgur.com/71tEeIy.png
My output in Kylin looks corrupted:
http://i.imgur.com/GCbrN03.png
However when I check Hive log it looks ok:
2015-05-27 08:40:13,419 INFO  [main]: ql.Driver (Driver.java:execute(1285)) - Starting command:
INSERT OVERWRITE TABLE kylin_intermediate_Kubek_19700101000000_2922789940817071255_f23ac1b1_10fe_4112_ac9e_b4e6baf07654 SELECT
FACT_TABLE.DATE
,FACT_TABLE.MONEY_ADVERTISER
,FACT_TABLE.MONEY_PUBLISHER
FROM DEFAULT.ADVSTATS as FACT_TABLE
...
2015-05-27 08:45:05,132 INFO  [main]: ql.Driver (SessionState.java:printInfo(824)) - MapReduce Jobs Launched:
2015-05-27 08:45:05,148 INFO  [main]: ql.Driver (SessionState.java:printInfo(824)) - Stage-Stage-1: Map: 1   Cumulative CPU: 17.32 sec   HDFS Read: 44644035 HDFS Write: 2347008 SUCCESS
2015-05-27 08:45:05,153 INFO  [main]: ql.Driver (SessionState.java:printInfo(824)) - Total MapReduce CPU Time Spent: 17 seconds 320 msec
2015-05-27 08:45:05,167 INFO  [main]: ql.Driver (SessionState.java:printInfo(824)) - OK

I'm using Hortonworks Sandbox 2.2:
hadoop-2.6.0
hbase-0.98.12
hive-0.14.0
zookeeper-3.4.6

Can someone tell me why my kylin logs preview looks like that and first of all is that could be a reason of last step error?

The fastest way to get response from our developers is to send email to our mail list<EMAIL_ADDRESS>and remember to subscribe our mail list via<EMAIL_ADDRESS>

That is possible, as hadoop may fail to get counter sometimes even if the job succeed; We observed this problem in our deployment, and to bypass it I made a commit on 0.6 branch in last week (see https://github.com/apache/incubator-kylin/commit/a4692dba681bc2f136e02c64565639eb0080fcc9);
Please discard your current job, and submit it again, then see whether this time it can finish all steps; If not, it may relate with the specific hadoop output format in your env;
If need, I can make the same change on v0.7 branch;

We have moved to Apache git and use Apache JIRA to track the issues; This repository has been abandoned; Please subscribe our mailing list for any public discussion (as hongbin mentioned above); see you there;


Account History E2E Testing

What would you like to be added:
Currently, the test within Account History is very limited. It only checks the soundness of the RPC does not test it end to end with different setups. We need complete/full e2e testing on account history to prevent regression.

Duplicate of #292 ?

/duplicate #292


Add support to non-standard logout parameters

What does this pull request change?
Add new config extraLogoutParameters. Use the extra logout parameters in the logout request.
Why is this pull request needed?
To support non-standard logout parameters.
Issues related to this change
https://github.com/soofstad/react-oauth2-pkce/issues/51

Thank you for this pull request. I think it will be very usefull.
However, not sure this needs to be solved by "extraLogoutParameters". Variants of this "logout_uri/redirect_uri" is almost standard, and so can be added to all requests. The value is already a part of the config.
I don't think there is any harm in making a request with a few more parameters than what the provider expects.
So I suggest this PR update the logout function to instead of using "extraLogoutParameters", always also send redirect_uriand logout_uri with the value config.logoutRedirect ?? config.redirectUri.
AWS Congnito is a commonly used auth provider, so it would be nice if it was supported "out-of-the-box". Withouth users of the library has to figure out what the correct "extraLogoutParameters" needs to be. :slightly_smiling_face:

Hi @soofstad , thank you for your reply. I see your point. It is a good idea to pass extra parameters without asking the users to configure them.
Only one more concern. As Cognito document mentions, Amazon Cognito requires either a logout_uri or a redirect_uri parameter in your request to the /logout endpoint.. Those two parameters are for different purposes.

logout_uri: Redirect your user to a custom sign-out page with a logout_uri parameter. Use logout_uri only with a client_id parameter. You can also use the logout_uri parameter to redirect your user to the sign-in page for another app client.
redirect_uri: Redirect your user to your sign-in page to authenticate with a redirect_uri parameter. Add client_id, scope, state, and response_type parameters that you want to pass to your /login endpoint. When you want to redirect your user to your /login endpoint to reauthenticate and pass tokens to your app, add a redirect_uri parameter.

I am not sure what would happen if we provide both two parameters to Cognito. If we choose one, I think logout_uri is a better choice.
What do you think?
Albert

logout_uri seems to be Amazon Cognitos alias for post_logout_redirect_uri as defined in the OIDC spec.
redirect_uri seems to be a Cognito specific parameter.
I guess there will be a use for these "extraLogoutParameters" no matter how much we add to the logout request by default.
Let's try and get this PR merged, and then create a separate PR for changing default logout params.
Have you been able to test this PR properly against Cognito? I have no account to test with there.
I will do some testing on my own, add some tests, and approve this PR as soon as I can.

logout_uri seems to be Amazon Cognitos alias for post_logout_redirect_uri as defined in the OIDC spec. redirect_uri seems to be a Cognito specific parameter.
I guess there will be a use for these "extraLogoutParameters" no matter how much we add to the logout request by default.
Let's try and get this PR merged, and then create a separate PR for changing default logout params.
Have you been able to test this PR properly against Cognito? I have no account to test with there. I will do some testing on my own, add some tests, and approve this PR as soon as I can.

Hi @soofstad , sorry, I tried but failed to test the PR against Cognito. I configured the project ms-example-app to point to my Cognito account. However, I don't know how to update the dependencies of ms-example-app to make it refer to my GitHub fork or local react-oauth2-code-pkce package.
I tried and updated the dependencies to git@github.com:albertwangnz/react-oauth2-pkce.git. However, when I executed yarn install, I can only pick up versions from the main project. That means it still pointed to the main project not my fork.
I also tried to yarn build and yarn pack my fork locally, but I was confused about how to do that. Since the package.json in root folder has name package-development, so yarn pack gave me package-development-v0.1.0.tgz. The src/package.json looks good. But yarn build gave me error Command "build" not found. since there is no build script in the src/package.json.
Then I stuck there.

You can use a locally checked out package in your actually application like this.....

Remove the line that refers to the "official" react-oauth2-pkce package version in the package.json of your application
Run npm install --save ../react-oauth2-pkce/src

For the 2nd bit there replace the path with where ever you have checked out the project

You can use a locally checked out package in your actually application like this.....

Remove the line that refers to the "official" react-oauth2-pkce package version in the package.json of your application
Run npm install --save ../react-oauth2-pkce/src

For the 2nd bit there replace the path with where ever you have checked out the project

Hi @anthony-vito  I tried but got this error. Also tried the suggested solution but was no lucky.

I tested that it sends the extra parameters okey, and that nothing else broke.
For future reference, how exactly did this 3-step guide not work for you? https://github.com/soofstad/react-oauth2-pkce#develop
I'm thinking if the develop documentation needs to be updated.

Hi @soofstad  thank you for approving the PR. Could you kindly help with adding a new tag/version 1.9.2?
Cheers.

I tested that it sends the extra parameters okey, and that nothing else broke.
For future reference, how exactly did this 3-step guide not work for you? https://github.com/soofstad/react-oauth2-pkce#develop
I'm thinking if the develop documentation needs to be updated.

This works. Thanks a lot! :-）


databus2 example  client exception nul

when I  tail -f logs/databus2-client-person.out
2014-03-27 15:48:00,652 +364546 [-RelayPuller] (INFO) {person_Person_null} picked a relay:[server=DatabusServerCoordinates [_name=localhost, _address=localhost/<IP_ADDRESS>:11115, _state=ONLINE], subs=[[ps=[uri=databus:physical-source:ANY;role=ANY;rk=], pp=*:*, ls=[name=com.linkedin.events.example.person.Person]]]]
2014-03-27 15:48:00,654 +364548 [-RelayPuller] (INFO) {person_Person_null} Relay Puller switching to request sources
2014-03-27 15:48:00,682 +364576 [-RelayPuller] (ERROR) {person_Person_null} Source not found on server: com.linkedin.events.example.person.Person
2014-03-27 15:48:00,682 +364576 [-RelayPuller] (INFO) {person_Person_null} picking a relay; retries left:2147483647, Backoff Timer :BackoffTimer [_config=BackoffTimerStaticConfig [_initSleep=1, _maxSleep=60000, _sleepIncFactor=1.1, _sleepIncDelta=1, _maxRetryNum=-1], _name=-RelayPuller.errorRetries, _retrySleepMs=32894, _retriesNum=89, _retryStartTs=1395906116783], Are we retrying because of SCNNotFoundException : false

I'm also having the same issue. I think there is miss match to the source name. I tried changing the name on the relay, but I got avro vs schema missmatch error. I think it is easier to change it on the client side, but I could not find any option to change this. Looks like the client by default takes the package name for the source.
from the relay log:
2015-05-29 19:12:04,807 +14601 [binlog-parser-1] (INFO) {OpenReplicator_person} BEGIN sql: BEGIN
2015-05-29 19:12:04,808 +14602 [binlog-parser-1] (INFO) {OpenReplicator_person} startXtionQueryEvent[header=BinlogEventV4HeaderImpl[timestamp=1432951924000,eventType=2,serverId=101,eventLength=71,nextPosition=5068,flags=8,timestampOfReceipt=1432951924807],threadId=37,elapsedTime=0,databaseNameLength=7,errorCode=0,statusVariablesLength=26,statusVariables=[QFlags2Code[flags=0], QSQLModeCode[sqlMode=0], QCatalogNzCode[catalogName=std], QCharsetCode[characterSetClient=33,collationConnection=33,collationServer=8]],databaseName=or_test,sql=BEGIN]
2015-05-29 19:12:04,809 +14603 [binlog-parser-1] (INFO) {person} src:com.linkedin.events.example.or_test.Person(40) #src:1 #evt:1 scn:21474841606 ms:-1 sizeInBytes:33 msEvent:0 msTimeElapsed:0 msQueryExec:-1 prodRate:0.00 consRate:-33.00
2015-05-29 19:12:04,809 +14603 [binlog-parser-1] (INFO) {person} src:person(0) #src:1 #evt:1 scn:21474841606 ms:-1 sizeInBytes:33 msEvent:0 msTimeElapsed:0 msQueryExec:-1 prodRate:0.00 consRate:-33.00

and this is from the client log:
2015-05-29 19:13:15,167 +63090000 [-RelayPuller] (INFO) {person_Person_null} Relay Puller switching to request sources
2015-05-29 19:13:15,189 +63090022 [-RelayPuller] (ERROR) {person_Person_null} Source not found on server: com.linkedin.events.example.person.Person
2015-05-29 19:13:15,190 +63090023 [-RelayPuller] (INFO) {person_Person_null} picking a relay; retries left:2147483647, Backoff Timer :BackoffTimer [_config=BackoffTimerStaticConfig [_initSleep=1, _maxSleep=60000, _sleepIncFactor=1.1, _sleepIncDelta=1, _maxRetryNum=-1], _name=-RelayPuller.errorRetries, _retrySleepMs=20422, _retriesNum=84, _retryStartTs=1432890426405], Are we retrying because of SCNNotFoundException : false
2015-05-29 19:13:37,656 +63112489 [-RelayPuller] (INFO) {person_Person_null} picked a relay:[server=DatabusServerCoordinates [_name=localhost, _address=localhost/<IP_ADDRESS>:11115, _state=ONLINE], subs=[[ps=[uri=databus:physical-source:ANY;role=ANY;rk=], pp=*:*, ls=[name=com.linkedin.events.example.person.Person]]]]

and the curl request:
curl -s http://localhost:11115/sources
[{"name":"com.linkedin.events.example.or_test.Person","id":40}]

Finally I was able to fix the issue. Like I mentioned in previous post, source name miss-match was causing the issue.
The client code has static source name:
static final String PERSON_SOURCE = "com.linkedin.events.example.person.Person";
I'm not there yet to recompile the client code, so I changed the source name at the relay:

changed the name value in conf/sources-or-person.json
and changed the namespace value in schemas_registry/com.linkedin.events.example.or_test.Person.1.avsc

:~/devel/github/databus/build/databus2-example-client-pkg/distributions/logs$ tail -5 client.log
2015-05-29 11:45:18,843 +1157 [callback-1] (INFO) {PersonConsumer} firstName: boris, lastName: asdsdasd, birthDate:<PHONE_NUMBER>00, deleted: false
2015-05-29 11:45:18,853 +1167 [callback-1] (INFO) {PersonConsumer} firstName: boris, lastName: asdsdasasdasdd, birthDate:<PHONE_NUMBER>00, deleted: false
2015-05-29 11:45:18,903 +1217 [callback-1] (INFO) {PersonConsumer} firstName: boris, lastName: asdsrrrdasdd, birthDate:<PHONE_NUMBER>00, deleted: false
2015-05-29 11:45:18,928 +1242 [callback-1] (INFO) {PersonConsumer} firstName: boris, lastName: asdsrrrdfsfsfdasdd, birthDate:<PHONE_NUMBER>00, deleted: false
2015-05-29 11:46:10,164 +52478 [callback-1] (INFO) {PersonConsumer} firstName: chavdar, lastName: asdsrrrdfsfsfdasdd, birthDate:<PHONE_NUMBER>00, deleted: false


ci: add Azure Static Web Apps workflow file Azure Static Web Apps CI/CD #1

Describe the bug
i am unable to deploy my static website using vs code .
##[debug]Evaluating condition for step: 'Build And Deploy' ##[debug]Evaluating: success() ##[debug]Evaluating success: ##[debug]=> true ##[debug]Result: true ##[debug]Starting: Build And Deploy ##[debug]Loading inputs ##[debug]Evaluating: secrets.AZURE_STATIC_WEB_APPS_API_TOKEN_THANKFUL_COAST_0[2](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:2)7D2D910 ##[debug]Evaluating Index: ##[debug]..Evaluating secrets: ##[debug]..=> Object ##[debug]..Evaluating String: ##[debug]..=> 'AZURE_STATIC_WEB_APPS_API_TOKEN_THANKFUL_COAST_027D2D910' ##[debug]=> '***' ##[debug]Result: '***' ##[debug]Evaluating: secrets.GITHUB_TOKEN ##[debug]Evaluating Index: ##[debug]..Evaluating secrets: ##[debug]..=> Object ##[debug]..Evaluating String: ##[debug]..=> 'GITHUB_TOKEN' ##[debug]=> '***' ##[debug]Result: '***' ##[debug]Loading env Run Azure/static-web-apps-deploy@v1 /usr/bin/docker run --name eecb5efa07d1741fe885[3](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:3)3962555c9595_ac[4](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:4)[5](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:5)ec --label 72882e --workdir /github/workspace --rm -e INPUT_AZURE_STATIC_WEB_APPS_API_TOKEN -e INPUT_REPO_TOKEN -e INPUT_ACTION -e INPUT_APP_LOCATION -e INPUT_API_LOCATION -e INPUT_OUTPUT_LOCATION -e INPUT_API_BUILD_COMMAND -e INPUT_APP_ARTIFACT_LOCATION -e INPUT_APP_BUILD_COMMAND -e INPUT_ROUTES_LOCATION -e INPUT_SKIP_APP_BUILD -e INPUT_CONFIG_FILE_LOCATION -e INPUT_SKIP_API_BUILD -e INPUT_PRODUCTION_BRANCH -e INPUT_DEPLOYMENT_ENVIRONMENT -e INPUT_IS_STATIC_EXPORT -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_RETENTION_DAYS -e GITHUB_RUN_ATTEMPT -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_SERVER_URL -e GITHUB_API_URL -e GITHUB_GRAPHQL_URL -e GITHUB_REF_NAME -e GITHUB_REF_PROTECTED -e GITHUB_REF_TYPE -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e GITHUB_ACTION_REPOSITORY -e GITHUB_ACTION_REF -e GITHUB_PATH -e GITHUB_ENV -e GITHUB_STEP_SUMMARY -e RUNNER_DEBUG -e RUNNER_OS -e RUNNER_ARCH -e RUNNER_NAME -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/portfolio-website/portfolio-website":"/github/workspace" 72882e:ecb5efa07d1741fe885339[6](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:6)2555c9595 DeploymentId: 0[7](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:7)5aa[8](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:8)e[9](https://github.com/TheBaljitSingh/portfolio-website/runs/7046376699?check_suite_focus=true#step:4:9)-45e3-44ed-8acf-fbdb37c29a7e App Directory Location: '/' was found. No Api directory specified. Azure Functions will not be created. Looking for event info The content server has rejected the request with: BadRequest Reason: No matching Static Web App was found or the api key was invalid. For further information, please visit the Azure Static Web Apps documentation at https://docs.microsoft.com/en-us/azure/static-web-apps/ If you believe this behavior is unexpected, please raise a GitHub issue at https://github.com/azure/static-web-apps/issues/ Exiting ##[debug]Docker Action run completed with exit code 1 ##[debug]Finishing: Build And Deploy

A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Device info (if applicable):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Check out the details here https://nitro.unjs.io/deploy/providers/azure.html#azure-static-web-apps
Two of the main issues I missed the first time I tried were solved by having a closer look at those instructions.
You need to adjust the yml file in your .github/workflows directory to have
          app_location: "/" # App source code path
          api_location: ".output/server" # Api source code path - optional
          output_location: ".output/public" # Built app content directory - optional

And also
mkdir -p .output/server
touch .output/server/.gitkeep
git add -f .output/server/.gitkeep

Hi @TheBaljitSingh we were unable to find a Static Web App with your default hostname (thankful-coast-027d2d910). Is it possible that it somehow got deleted? This could explain why your deployment is failing.

Im experiencing this issue too, what's the solution please?


Add detail to error response

Coverage remained the same at 100.0% when pulling c9c9f72c994092a5a7b1fc4194d7c09b28fb2625 on handle-error-details into f38d6fe271ac744765130964e5de96640f4f2fc1 on master.


Failed to import caffe on SD card image

Hi, I downloaded the provided SD card image from the Google Drive link and attempted to run both the Lenet and CIFAR_10 Jupyter Notebook examples. It fails on the first step "Import Caffe" with the following error:
ImportError                               Traceback (most recent call last)
 in ()
2 caffe_root = '/home/xilinx/caffe/'  # this file should be run from {caffe_root}/examples (otherwise change this line)
3 sys.path.insert(0, caffe_root + 'python')
----> 4 import caffe
/home/xilinx/caffe/python/caffe/init.py in ()
----> 1 from .pycaffe import Net, SGDSolver, NesterovSolver, AdaGradSolver, RMSPropSolver, AdaDeltaSolver, AdamSolver
2 from ._caffe import set_mode_cpu, set_mode_gpu, set_device, Layer, get_solver, layer_type_list, set_random_seed
3 from ._caffe import version
4 from .proto.caffe_pb2 import TRAIN, TEST
5 from .classifier import Classifier
/home/xilinx/caffe/python/caffe/pycaffe.py in ()
11 import numpy as np
12
---> 13 from ._caffe import Net, SGDSolver, NesterovSolver, AdaGradSolver,
14         RMSPropSolver, AdaDeltaSolver, AdamSolver
15 import caffe.io
ImportError: libprotobuf.so.10: cannot open shared object file: No such file or directory

Sorry for the late reply. The project doesn't support python 3 because at the time of developing the project python 3 doesn't support protobuf and caffe properly. Please switch to a python 2 kernel in the Jupyter notebook.


Require KWargs for struct initialization

What was wrong?
Example:
struct House:
    vacant: bool
    price: u256

Currently, House can be instantiated as House(true, 1000000).
This means:

Property assignment can easily get mixed up leading to bugs
struct initialization is visually very close to regular function invocations

How was it fixed?
With this change it is required to be instantiated like House(vacant=true, price=1000000)
While working on this it got me thinking if this is something that should be handled on the parser stage instead but I don't think that would be straight forward because technically there is nothing the syntax is identical with method invocation syntax.
We could move this into the parser stage if we used a distinct syntax such as House(vacant: true, price: 100). But I'm personally ok to keep it as with the check being handled by the analyzer.

Something I noticed while working on #189 is that initialization doesn't work much of the time, since the values are in an unpredictable order upon retrieval.
The following will fail to compile for me:
struct House:
    price: u256
    size: u256
    rooms: u8
    vacant: bool

contract Foo:
    pub def create_house():
        self.my_house = House(1, 2, u8(5), false)

as the attribute types just so happen to be retrieved in the the order u256, u8, u256, bool.
I'll fix this in my PR.

fixed. I just added another attribute for key order to the struct struct


loggy_rsyslog_ng_tls error using default configuration

I got the following error trying to use the loggly_rsyslog_ng LWRP. Does the provider class need to be manually required?
NameError
---------
No resource, method, or local variable named `loggy_rsyslog_ng_tls' for `Chef::Provider::LogglyRsyslogNg ""'

Cookbook Trace:
---------------
/var/lib/aws/opsworks/cache.stage2/cookbooks/loggly_rsyslog_ng/providers/default.rb:38:in `block in class_from_file'

Hello Alex,
Could you please you show me part of the code you use for calling this LWRP (except for sensitive data of course)?
I want to reproduce the error, so I can fix it.
I think it happens because I called loggly_rsyslog_ng_tls provider without arguments inside loggly_rsyslog_ng provider :-|

Just for now try to use cookbook like:
loggly_rsyslog_ng_tls 'rsyslog.loggly.crt'

loggly_rsyslog_ng '/etc/rsyslog.d/22-loggly.conf' do
  install_tls_certs false
  loggly_token      'my_very_secure_token'
  log_files         [{
      :filename => '/var/log/syslog',
      :statefile => 'syslog',
      :facility => 'local6'
                    }]
  tags              [ node.chef_environment ]
end

This is the full code of the recipe I've written to use it:
shared_path = "/srv/www/APP/shared/log"
env = node[:deploy][:APP][:rails_env]

loggly_rsyslog_ng '/etc/rsyslog.d/21-rails.conf' do
  loggly_token node[:loggly][:token]
  tags [env, 'APP']
  log_files([
    {filename: "#{shared_path}/#{env}.log", tag: "rails"},
    {filename: "#{shared_path}/unicorn.stderr.log", tag: "unicorn"},
    {filename: "#{shared_path}/unicorn.stdout.log", tag: "unicorn"}
  ])
end

loggly_rsyslog_ng '/etc/rsyslog.d/21-sidekiq.conf' do
  loggly_token node[:loggly][:token]
  tags [env, 'APP']
  log_files([
    {filename: "#{shared_path}/sidekiq.log", tag: "sidekiq"}
  ])
end

loggly_rsyslog_ng '/etc/rsyslog.d/21-nginx.conf' do
  loggly_token node[:loggly][:token]
  tags [env, 'APP']
  log_dirs([
    {directory: "/var/log/nginx", tag: "nginx"}
  ])
end

And this is the full error stacktrace from chef:
NameError
---------
No resource, method, or local variable named `loggy_rsyslog_ng_tls' for `Chef::Provider::LogglyRsyslogNg ""'

Cookbook Trace:
---------------
/var/lib/aws/opsworks/cache.stage2/cookbooks/loggly_rsyslog_ng/providers/default.rb:38:in `block in class_from_file'

Resource Declaration:
---------------------
# In /var/lib/aws/opsworks/cache.stage2/cookbooks/buzzstarter/recipes/logging.rb

4: loggly_rsyslog_ng '/etc/rsyslog.d/21-rails.conf' do
5:   loggly_token node[:loggly][:token]
6:   tags [env, 'APP']
7:   log_files([
8:     {filename: "#{shared_path}/#{env}.log", tag: "rails"},
9:     {filename: "#{shared_path}/unicorn.stderr.log", tag: "unicorn"},
10:     {filename: "#{shared_path}/unicorn.stdout.log", tag: "unicorn"}
11:   ])
12: end
13:

@jakshi ping

Hello Alex,
Do you have the same error with last version of cookbook?
Kindest Regards.
Konstantin.

Yep, same error.

Alex Sharp

On Jul 15, 2015, at 1:15 AM, Konstantin<EMAIL_ADDRESS>wrote:
Hello Alex,
Do you have the same error with last version of cookbook?
Kindest Regards.
Konstantin.
—
Reply to this email directly or view it on GitHub.

Same issue here:
# project/config/chef/site-cookbooks/my-project/recipes/loggly.rb

include_recipe "loggly_rsyslog_ng"

loggly_rsyslog_ng '/etc/rsyslog.d/22-loggly.conf' do
  loggly_token      '##removed##'
  tags              [ node.chef_environment ]
end

When provisioning I get this error:
==> staging: [2015-07-21T23:18:06-07:00] ERROR: No resource or method named `loggly_rsyslog_ng' for `Chef::Recipe "loggly"'

# Abbreviated Cheffile.lock

    loggly_rsyslog_ng (4.2.0)
      apt (~> 2.0)
      rsyslog (~> 1.13.0)

Yea, the default recipe is broken, I think because it defers to the SSL version, which doesn't require the default file. I think requiring the file will fix it.

Alex Sharp

On Jul 21, 2015, at 11:20 PM, Noah Spitzer-Williams<EMAIL_ADDRESS>wrote:
Same issue here:
project/config/chef/site-cookbooks/my-project/recipes/loggly.rb
include_recipe "loggly_rsyslog_ng"
loggly_rsyslog_ng '/etc/rsyslog.d/22-loggly.conf' do
loggly_token      '##removed##'
tags              [ node.chef_environment ]
end
When provisioning I get this error:
==> staging: [2015-07-21T23:18:06-07:00] ERROR: No resource or method named loggly_rsyslog_ng' for Chef::Recipe "loggly"'
Abbreviated Cheffile.lock
loggly_rsyslog_ng (4.2.0)
  apt (~> 2.0)
  rsyslog (~> 1.13.0)

—
Reply to this email directly or view it on GitHub.

Hello @ajsharp
I added simple test setup for the cookbook, Reproduce the error and fixed it.
Please try new cookbook version.

Hello @ajsharp
As I saw no error in my tests and there're no error reports from you I suppose that it's fixed.
I'm closing the issue.
Kindest Regards.
Konstantin.


Bug: foundry reports successful creation of a repo when using --template to nonexisting url

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
No response
Operating System
macOS (Intel)
Describe the bug
running this command just report a failure, but instead reports it as sucessful
$ forge init --template github.com/sambacha/zen-template
Initializing /Users/sbacha/foundry-boilerplate from https://github.com/github.com/sambacha/zen-template...
Cloning into '/Users/sbacha/foundry-boilerplate'...
remote: Not Found
fatal: repository 'https://github.com/github.com/sambacha/zen-template/' not found
    Initialized forge project.

cc @0xYYY looks like we're missing another git check somewhere

cc @0xYYY looks like we're missing another git check somewhere
💯 thx!


Need a better method of resource management.

 Need a new form of memory management that allows this behavior without complicating code.
ref: commit 930ea8d42cf3f87193bfdbb2cdb98b5ca572e270

Closed until relevant again.


Added support for iOS, watchOS and tvOS throughout source code

Didn't realise that there were conditionals throughout the source code when I said the PR would be trivial. So here is the rest of the work. Thx!

For me to take your PR, I need you to fill out and return the CLA...  For this PR, it'll be easier for me to just make the changes.  I'm doing that now and should have the release ready shortly...

no worries. thx much!

On 21 Sep 2016, at 22:26, Bill Abt<EMAIL_ADDRESS>wrote:
For me to take your PR, I need you to fill out and return the CLA... For this PR, it'll be easier for me to just make the changes. I'm doing that now and should have the release ready shortly...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/IBM-Swift/BlueCryptor/pull/4#issuecomment-248732397, or mute the thread https://github.com/notifications/unsubscribe-auth/AQPPLSlSH9v7wS-oZACSrRCosc-CpI6kks5qsZLdgaJpZM4KDOhs.

It's released...  Sorry I missed that the first time.  The test suite is now updated as well.


Overwriting of SCSS / LESS / CSS Hierarchy

Hey Guys,
Firstly Great package & helped me save abit of time in setup phase.
A few questions :
Is this still supported? -  I see a few question go unanswered.
Styling Hierarchy / overwriting
Currently, Im stuck in the situation where id like my styles to overwrite the default AdminLTE / Bootstrap themes styles but its seeming quite difficult (having to use !important).
Is there a way (without editing default in node_modules / bower_comps to use your overrides. Might be a noobish question but any direction would be appreciated
This is my webpack.mix.js file  (i've commented out & removed so fo the copied files and re-coiled npm-dev but still nothing :
** **************** Code start ****************** **
`mix.js('resources/assets/js/app.js', 'public/js')
.js('resources/assets/js/app-landing.js', 'public/js/app-landing.js')
.less('node_modules/bootstrap-less/bootstrap/bootstrap.less', 'public/css/bootstrap.css')
.less('resources/assets/less/adminlte-app.less','public/css/adminlte-app.css')
.less('node_modules/toastr/toastr.less','public/css/toastr.css')
.sass('resources/assets/sass/app.scss', 'public/css')
.combine([
'public/css/app.css',
'node_modules/admin-lte/dist/css/skins/light-blue.css',
'public/css/adminlte-app.css',
'node_modules/icheck/skins/square/blue.css',
'public/css/toastr.css'
], 'public/css/all.css')
.combine([
'public/css/bootstrap.css',
'resources/assets/css/main.css'
], 'public/css/all-landing.css');
//APP RESOURCES
//.copy('resources/assets/img/.','public/img')
//VENDOR RESOURCES
//.copy('node_modules/font-awesome/fonts/.','public/fonts/')
//.copy('node_modules/ionicons/dist/fonts/.','public/fonts/')
//.copy('node_modules/admin-lte/bootstrap/fonts/.','public/fonts/bootstrap')
//.copy('node_modules/admin-lte/dist/css/skins/.','public/css/skins')
//.copy('node_modules/admin-lte/dist/img','public/img')
//.copy('node_modules/admin-lte/plugins','public/plugins')
//.copy('node_modules/icheck/skins/square/blue.png','public/css')
//.copy('node_modules/icheck/skins/square/blue@2x.png','public/css');
if (mix.config.inProduction) {
mix.version();
}
mix.browserSync('xxxx');`
** **************** Code end ****************** **
Will def continuing using AdminLTE -laravel going forward
Bests

Hi! I'm using this package a while in more projects. Today I have installed it for a new project and unfortunately got the same issue.

Any feedback on this one: ?

My solution was that I have installed the previous adminlte node packages. 2.3.4.

ahh thanks, Would give it a try but I've made quite abit of progress since then. Dont want to revert the entire install. Going to what i can find will post solution here.

Is this still supported?
Yes as much as I can. I welcome any help anyone can provide for example resolving issues or proposing PR...

@neomobil 2.3.4? As far as I know 2.3.11 worked before but now this versions has problems also
I'm working to know what happens to adminlte node package because I amb facing problems with new version 2.4 and 2.3.11

@acacha In terms of overwriting your LESS & SASS Styles in webpack.mix.js. Whats the recommended hierarchy or how would i add your node module imports to my webpack.mix.js.

@acacha sorry, i didn't remember correctly. It was 2.3.11. This package works

In the new version are path problems, i think because of the missing bootstrap directory

@neomobil new version 5.0.4 is released today trying to patch lot of problems with adminlte 2.4.2 version. See last changes

Thank you. I will test it.

@lee-mustard Umm I'm not sure if I understand you: node_modules/admin-lte package is not mine is from Almasaaed https://github.com/almasaeed2010/AdminLTE

@neomobil be nice ;-) the new release version contains ugly patches I have not find better solution until AdminLTE solves some issues I report in their Github repo. Feel free to propose any changes or PR!

@acacha Thanks. Good idea.


Update getRequest to use a useCallback

Yes, this may be the solution. Because linter wanted to use inline function inside useCallback, but it was not possible outside of the useGetRequest. LGTM :+1:


CookieFileStorage not working and restarting my server

My code:

Console:
CONSOLE COOKIES CookieFileStorage { storage: { idx: {} } }
and after consoling cookies server restarting

#597


bugfix

bugfixes:

samples diversely is clicked with no clusters in inputClusters
samples diversely is clicked when numSamples exceeds number of available clusters
samples diversely is clicked under insufficient/invalid conditions

lines changed in src/components/SamplingSection.tsx: 39-41, 58, 71(close if statement)

LGTM, thanks!


Reset switch

Add a reset switch to reset at whatever point the button is pushed at.

Can i bdemjndfs

CAn i be assigned this

Yes. The time limit is 1 hour.

Can I get assigned to this ?

The issue has been solved.


Allow to build complex queries by bypass SqlStatement instance as parameter

Seems that it is not possible right now to do something like this:
import {SQL} from "sql-template-strings"

function prepareSet(values) { /* implementation skipped */}

pg.query(SQL`UPDATE ${SQL("table")} SET ${SQL(prepareSet({key: value})} WHERE id = ${id}`)

I mean that if parameter is already instance of SQLStatement, it could be inserted to query as-is (with possible merging nested parameters).
What do you think about it?

This functionality is provided with .append(). I think passing template strings into template strings is confusing tbh.

As for me, this one:
pg.query(SQL`UPDATE ${SQL("table")} SET ${SQL(prepareSet({key: value})} WHERE id = ${id}`)

is less confusing than
pg.query(SQL`UPDATE `.append("table SET").append(prepareSet({key: value})).append(SQL` WHERE id = ${id}`)

btw, you can choose the way as you want, I suggest just to allow passing SQLStatement as template variable.

I guess why not, as long as it has test cases covering all the cases where both statements have values and not

Btw, SQL("table") doesn't actually work. The first parameter to a template string tag is an array.

My pull request would allow the following:
// example (no idea what prepareSet would do)
const prepareSet = obj => SQL`preparedName = ${obj.key}`;

// auto back-tick escaping of single/double quoted parts + auto merge of values
SQL`UPDATE '${`table_${name}`}' SET ${prepareSet({key: value})} WHERE id = ${id}`

Hope that works better for this case too 👋


Introduce feature flag enable-step-actions

This PR introduces the feature flag enable-step-actions. The follow up PRs will use this feature flag to gate the use of StepActions in Tasks.
This is the second item on the implementation Issue https://github.com/tektoncd/pipeline/issues/7259

Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
[WIP] Introduces a new feature flag "enable-step-actions: true/false" to gate the use of step actions.

/kind feature

/assign


'add' event does not fire when watching a UNIX domain socket on macOS

Describe the bug
The add event does not get triggered when watching a single UNIX socket. It works for regular files and also works when globbing, so I suspect this is a bug.
Versions:

Chokidar version 3.0.2
Node version v12.4.0
OS version: macOS 10.14.5

To Reproduce
Make sure not to use globbing, and just open up a regular UNIX domain socket.
var chokidar = require('chokidar');
var net = require('net');
// One-liner for files and directories starting with 'test'
chokidar.watch('test.sock', {}).on('all', (event, path) => {
  console.log(event, path);
});
const server = net.createServer();
server.listen('test.sock');
// Does not output anything, while we'd expect it to output a single 'add' event

Expected behavior
For the previous piece of code to just work. That is, for it to register the newly created socket.
Additional context
I'm creating a small auto-configuring process manager, and I'd like to use this trick to only bind to the socket when it is actually there.

FWIW, I discovered this same bug in a Rust library that uses FSNotify under the hood. Assuming this library uses FSNotify as well, I think there is a bug in FSNotify itself that does not detect UNIX domain sockets. I replicated the issue using the raw C API and opened a bug with Apple


Add test case for bug 'fixed' by dropping of Swift 3 compatibility

This was https://bugs.swift.org/browse/SR-3806.

@swift-ci Please smoke test


Different results in python 3

Thanks for sharing the code!
I ran the pre-trained model in python 3 to do predictions for my input images. The compatibility score I got in python 3 is different from those in python 2 using the same checkpoint file (the one you shared). Wondering if you know why this might be the case?

Sorry I realize I loaded the images differently. The results are consistent


Add Stepper Process to the Tag Generator

Consulted with Nandana about breaking the Tag Generator down into steps for easier navigation and user convenience.
Implementing a stepper will also help reduce ambiguity about button labels received from user testing.
Chali will create 2 variations for team to review.

Stepper process can be viewed here. Will review with UX team for review.

@chalimar to update this issue with requirements for adding "step 1" etc... to existing tag generator. Redesign will be a different issue


Use controller.Impl tracker and cleanups

:broom: Update or clean up current behavior

/kind cleanup
Pre-review Checklist

[ ] At least 80% unit test coverage
[ ] E2E tests for any new behavior
[ ] Docs PR for any user-facing impact
[ ] Spec PR for any new API feature
[ ] Conformance test for any change to the spec

Release Note

Docs

/retest


Download position in queue

Is it possible to associate a position for a new download and change the position in the queue of an existing download? The aria2.addUri method has a position argument and there is a aria2.changePosition method as well.

yes that's a good idea. i wanted to change the order in the queue but could not :(


webhook testing

OpenShift Hello World
test
echo payload
testing issue comment triggers
null change
set properties in stage
Webhook testing

test

test

test

test

test

test

test

test

test

test it boi!

test

test


Configure Azure Pipelines

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

Accidentally tried to pull to wrong place  :)


Do we need didRender focus handling for bulma-modal-close?

This is here as a placeholder issue.
Lets determine if we want to have any type of focus handling for the close button on modal render. I removed this due to inconsistencies, as it's currently not working with Ember 2.6.1 but is working with Ember 2.10.0
  //TODO Not working
  /**
    *
    */
  didRender() {
    this.$().focus();
  }

@alexdiliberto What I've experienced by other addons is that focus handling, and a11y in general, is left to the consumer to implement.
I'm not against supporting this though. Perhaps we do this by default, and provide a way for consumers to opt-out.


jp bash parser fails silently if config.json is not correct

Need to add a test to deploy.sh to check config.json for errors.
cc: @tmbgreaves

Addressed in PR #22, commit https://github.com/alan-turing-institute/binderhub-deploy/pull/22/commits/4d686c6e8340f4bd342a1e92551d61d0863cb444
I've added some checks for the minimum set of variables to be correctly read in from the configuration file.

I'm in a big quandary over whether to add in the linter or not. In favour, it would be good to actually check the config file and tell a user where any errors are. On the other hand, it puts the linter dependency on us plus javascript, and we already have checks in place to make sure that the config we need has been read in correctly.

If our tests fail reliably and produce useful error messages, then I don't see a need to add in an extra dependency.

Let's leave it out then :-) It's been generating useful errors for me when I've made genuine mistakes and messed up my config...


support vm events available to systrace

For example,  API_TIMELINE_DURATION in dart_api_impl.cc
DART_EXPORT Dart_Handle Dart_Invoke(Dart_Handle target,
                                    Dart_Handle name,
                                    int number_of_arguments,
                                    Dart_Handle* arguments) {
  DARTSCOPE(Thread::Current());
  API_TIMELINE_DURATION(T);
  ...
}

(https://github.com/flutter/engine/pull/14323/files) this PR  make API stream events available to systrace. but only can record TimeBeginEndScope, not including TimelineDurationScope.
with this PR, Dart_Invoke event can support in systrace

so I choose to replace all uses of TimelineDurationScope with TimeBeginEndScope, and remove  unnecessary code to prevent future use of TimelineDurationScope method.

https://dart-review.googlesource.com/c/sdk/+/128200

relation to https://github.com/flutter/flutter/issues/47771


Some libraries, even tho the configuration is set to remove them, are…

… still present in the dependency tree

Describe your change:

Checklist - Check boxes in the PR UI, not by editing the text
Pull Request

[ ] PR description is meaningful.
[ ] There are no more than a few commits to merge.
[ ] There are no more than a few lines of code to review.

Code style

[ ] Maximum line length does not require scrolling in Github
[ ] FMT and ScalaStyle and ScalaFix are good

Testing

[ ] Unit tests for the added/modified code
[ ] Overall unit test coverage good
[ ] Unit tests are passing

Dependencies

[ ] The plugin is added to itself
[ ] The latest version is added to itself
[ ] New dependencies do not have vulnerabilities

Security

[ ] There is no known vulnerability being added to this project
[ ] Everything on this list is checked

Coverage increased (+0.2%) to 73.415% when pulling 2aab882f62c9d5ec5cfd6d14060a9e6b1c4df011 on attempt_to_fix_removing_dependencies into f594371322c39f0d8e8109c57036b2bd109d6c3f on master.


ESC ] 4 ;  ; rgb :  /  /  ESC ... in WindowsTerminal?

That works in a console.  Below I'm changing color 0, which is the background color.
Before:

After:

Should it work in WindowsTerminal?  It doesn't.

You are redefining color 0, which is one of the colors in the 16-color palette (usually assigned to black.)
The vintage console only supports having indexed colors for the foreground or the background.
All other terminals, WT included, support 18 paletted colors: the 16 from ANSI/VT, plus a foreground and background which are independent.
You'll want to use OSC 11 (if I remember properly) to set the background, not OSC 4 to overwrite black.

My only reference is https://docs.microsoft.com/en-us/windows/console/console-virtual-terminal-sequences which doesn't mention OSC 11.
Is there a better reference, one that describes OSC 11 in detail?

Hmmm!  If I have the terminators right [0x7 (BEL) or 0x9C (ST?)], I find that
OSC]10;16;... changes the color of the background
OSC]10;17;... changes the color of the foreground
Are they the last two colors in the 18-color palette you spoke of?
And OSC]11;;... changes the color of the cursor ( doesn't seem to matter)
I wish I knew how it's supposed to work.

While the sequences from my previous post work as stated, I guess I didn't try hard enough.  These also work and are as I have found documeted.
ESC]10;rgb:r/g/bST ... foreground color
ESC]11;rgb:r/g/bST ... background color
ESC]12;rgb:r/g/bST ... cursor color

So! The documentation for these is right under the "Operating System Commands" section in the inimitable xterm control sequences guide. We have a backlog item booked to update our docs with all our supported sequences and the versions they came out in.
Your finding is pretty interesting! This set of OSC can be "chained" -- if you start with 10, and specify multiple colors (For your sequence, E]10;16;XYZ, 16 here was interpreted as the first color, and XYZ the second!), you will set resources 10, 11, 12, ...

"`e]10;rgb:ff/00/00;rgb:00/ff/00;rgb:00/00/ff`e\"
       ^ FOREGROUND ^ BACKGROUND ^ CURSOR

The 10 indicates that we wanted to start with the foreground.
You could do the same thing with three sequences, E]10;, E]11; and E]12;.

(We do not support all the resources specified in OSC Ps ; Pt ST)

And finally, last note before I give you space to reply (!) (:smile:)
The venerated Console also supports this "separate" palette, though it does make some applications act strange.
It's in the "Terminal" tab in the old settings sheet, marked down as "experimental" terminal settings, with a link to a fairly wordy blog post explaining them:

Thanks for all the info, especially about "chaining" the sequences.
I have never played with that tab in the settings.  Do "separate foreground" and "separate background" act as two "extra" colors (perhaps in the 18-color palette you mentioned earlier)?

Yep, they are exactly that! They are accessible in VT using SGR 39 and 49 ("default foreground", "default background"). There's also cursor color, which is settable with OSC 12 but does not map to a SGR.

Yep, they are exactly that! They are accessible in VT using SGR 39 and 49 ("default foreground", "default background").

Are they what ESC[0m gives?

Indeed they are. For console, they replace the old "default" colors in any place except the API (which doesn't report with a data type large enough to express more colors :()

Aha!  That's nice.  So I can have "default" colors that are not a combination of two colors from the console's 16-color palette.

Aha! That's nice. So I can have "default" colors that are not a combination of two colors from the console's 16-color palette.

In a console, I can set "DefaultForefround" and "DefaultBackground" in the Properties dialog.  Can I set them on the fly with a control sequence?  I tried SGR 39 and SGR 49 (with 2;r;g;b, as in 38/48); that didn't work.  [If that question is answered above, I couldn't find it.]

I tried SGR 39 and SGR 49 (with 2;r;g;b, as in 38/48);

I'm sorry what did you try? You don't need to use the 2;r;g;b bits with 39/49, they'll suffice on their own:
printf "Default text \e[31;42m red on green \e[39m default on green \e[31;49m red on default \n"

I'm sorry.  I misinterpreted the purpose of SGR 39/49.
What I really want to do is SET (on the fly) the console's default foreground/background and/or WT's defaultForeground/defaultBackground.
Can either or both of those be done?
It's part of a (mostly academic) project to let the user issue a command to change the console's (or WT's) color scheme.  Some of the schemes don't work correctly if those fore/back colors are not set.
And (off-topic) ... Are the Creators Update console colors still the default in Windows 10?  If so, maybe a "Creators Update" scheme in WT would be appropriate.  I added such a scheme myself, works fine.  I'll post it if it will be of any value.

Oh, yea I believe OSC 10 and OSC 11 should set the values of those colors. The properties dialog might not update accordingly - that's not really actively maintained, so I'm not sure about that. IIRC OSC 10&11 are fairly new so you'll need the OpenConsole.exe from the Terminal or Windows 11 to see support for those.

And (off-topic) ... Are the Creators Update console colors still the default in Windows 10? If so, maybe a "Creators Update" scheme in WT would be appropriate. I added such a scheme myself, works fine. I'll post it if it will be of any value.

Yep, those are still the defaults - the scheme we affectionately call "Campbell"

Hmmm!  I talked about OSC 10/11 with @DHowett in another thread and for some reason ruled them out.  I'll try again.  They're not documented for the console.  Should they work in a console?
If my REG backup of HKCU\Console is correct, the Creators Update colors are not the same as Campbell.  Did they get some tweaking?

Everything that works in Terminal will eventually work in Console -- the only hold-up is how long it takes our code changes to get out to public Windows versions.
There is a chance that the Creators' Update (I can't remember when that was) predates Campbell. In that case, the values are the same as the "Vintage" theme.

I have these.  I'm pretty sure the translations are correct.  The order is Campbell, Vintage, CreatorsUpdate.
/*          0         1         2         3         4         5         6         7         8         9        10        11        12        13        14        15            16        17       */
/*        black      red      green    yellow     blue     purple     cyan      white    briBla     bRed     bGreen    bYello    bBlue     bPurpl     bCyan    bWhite         fore      back      */
	{0x0C0C0C, 0x1F0FC5, 0x0EA113, 0x009CC1, 0xDA3700, 0x981788, 0xDD963A, 0xCCCCCC, 0x767676, 0x5648E7, 0x0CC616, 0xA5F1F9, 0xFF783B, 0x9E00B4, 0xD6D661, 0xF2F2F2,     0xCCCCCC, 0x0C0C0C},
	{0x000000, 0x000080, 0x008000, 0x008080, 0x800000, 0x800080, 0x808000, 0xC0C0C0, 0x808080, 0x0000FF, 0x00FF00, 0x00FFFF, 0xFF0000, 0xFF00FF, 0xFFFF00, 0xFFFFFF,     0xC0C0C0, 0x000000},
	{0x0C0C0C, 0x0037DA, 0x13A10E, 0x3A96DD, 0xC50F1F, 0x881798, 0xC19C00, 0xCCCCCC, 0x767676, 0x3B78FF, 0x16C60C, 0x61D6D6, 0xE74856, 0xB4009E, 0xF9F1A5, 0xF2F2F2,     0xCCCCCC, 0x0C0C0C}

Campbell and Vintage are ultimately from settings.json; CreatorsUpdate from these:
"ColorTable00"=dword:000c0c0c
"ColorTable01"=dword:00da3700
"ColorTable02"=dword:000ea113
"ColorTable03"=dword:00dd963a
"ColorTable04"=dword:001f0fc5
"ColorTable05"=dword:00981788
"ColorTable06"=dword:00009cc1
"ColorTable07"=dword:00cccccc
"ColorTable08"=dword:00767676
"ColorTable09"=dword:00ff783b
"ColorTable10"=dword:000cc616
"ColorTable11"=dword:00d6d661
"ColorTable12"=dword:005648e7
"ColorTable13"=dword:009e00b4
"ColorTable14"=dword:00a5f1f9
"ColorTable15"=dword:00f2f2f2

I extracted this from the default win32 settings embedded in 10.0.15063
ColorTable00: 000000
ColorTable01: 800000
ColorTable02: 008000
ColorTable03: 808000
ColorTable04: 000080
ColorTable05: 800080
ColorTable06: 008080
ColorTable07: C0C0C0
ColorTable08: 808080
ColorTable09: FF0000
ColorTable10: 00FF00
ColorTable11: FFFF00
ColorTable12: 0000FF
ColorTable13: FF00FF
ColorTable14: 00FFFF
ColorTable15: FFFFFF

I extracted this from the config initialization code in the console in 10.0.15063 (you can see the current version here)
    _ColorTable[0] = RGB(0x0000, 0x0000, 0x0000);
    _ColorTable[1] = RGB(0x0000, 0x0000, 0x0080);
    _ColorTable[2] = RGB(0x0000, 0x0080, 0x0000);
    _ColorTable[3] = RGB(0x0000, 0x0080, 0x0080);
    _ColorTable[4] = RGB(0x0080, 0x0000, 0x0000);
    _ColorTable[5] = RGB(0x0080, 0x0000, 0x0080);
    _ColorTable[6] = RGB(0x0080, 0x0080, 0x0000);
    _ColorTable[7] = RGB(0x00C0, 0x00C0, 0x00C0);
    _ColorTable[8] = RGB(0x0080, 0x0080, 0x0080);
    _ColorTable[9] = RGB(0x0000, 0x0000, 0x00FF);
    _ColorTable[10] = RGB(0x0000, 0x00FF, 0x0000);
    _ColorTable[11] = RGB(0x0000, 0x00FF, 0x00FF);
    _ColorTable[12] = RGB(0x00FF, 0x0000, 0x0000);
    _ColorTable[13] = RGB(0x00FF, 0x0000, 0x00FF);
    _ColorTable[14] = RGB(0x00FF, 0x00FF, 0x0000);
    _ColorTable[15] = RGB(0x00FF, 0x00FF, 0x00FF);

As far as I can tell, that's "Vintage", which is pretty curious since I am sure we changed the color scheme at some earlier point. However, I can't find any mention of Campbell's black (ColorTable00) in the windows source tree as of 10.0.15063.

We announced Campbell on Aug 2, 2017, and mentioned a Windows build numbered 10.0.16257.
The scheme you extracted from the Creators' Update (10.0.15063) is certainly not the stock one!

I don't know how far back Windows build 10.0.15063 goes.  Those are certainly Vintage.
If Campbell is from 2017, I'm pretty sure it pre-dates CreatorsUpdate.  In 2017 I was still on Windows 7.  :-)
My REG backup is from right after CreatorsUpdate (which came to me late); I didn't like the new colors and changed them back to Vintage.  It's dated 2019-07-01.  I also found the same thing by Googling creators update colortable00.  In the same place there was

Windows 10 build 16257 got the new scheme.

Do you have an up-to-date (including CreatorsUpdate) Windows 10?  What does HKCU\Console say?

FWIW, Windows would try to port your reg entries on an upgrade, and we wouldn't blow them away to install the new defaults. So if you upgraded from 7 to 10 to CU, then you probably had Vintage migrated the whole way through.

Do you have an up-to-date (including CreatorsUpdate) Windows 10? What does HKCU\Console say?

Sorry, I'm not sure what you mean. These are distinct versions of Windows, and being up-to-date while still being on the Creators' update will put you somewhere around 10.0.15063.9999. Moving off the Creators' update to a newer versions of Windows¹ will mean you're no longer on the Creators' Update.
The newest version of Windows 10 is 20H1 (10.0.19041.1) plus all of its servicing updates (which bring the build number to 10.0.19043.xxx). I definitely have one of those, and it definitely has Campbell as a color scheme.
A stock install of the Creators' Update, as released in October 2017, doesn't use Campbell. The door for getting feature updates into that version closed some time in June 2017, and Campbell was authored in early July 2017. :smile:
I did find that result you're talking about by searching, and it seems like that blog post is explaining how to backport Campbell to an older version of Windows. Totally fine and good, just not evidence that Campbell existed for that version!

I'm getting lost.  Those REG entries came from my registry in July 2019.  Consulting records, that's when I got this computer, so they came installed.  So maybe I shouldn't be talking about Creators Update.  I also found those entries them here. I think I got a (pretty much) new OS in the Jan2021 20H2 update.  That's as far back as my update history goes.  Maybe this thread is going nowhere!

Ah, I think I figured it out. If you installed in 2019, you've got Campbell. For sure.
What we're seeing here, though, is two translation issues!
First: the color ordering for console colors, as well as legacy VGA attributes (on which console colors are based) swaps red and blue (and derivatives like teal and cyan).

In this example, I have lined up red and blue.
	  ... 1         ... 4
          ... RED       ... BLUE
CAMPBELL {... 0x1F0FC5, ... 0xDA3700, },
                 \ /           \ /
                 / \           / \
REGISTRY {... 0xC50F1F, ... 0x0037DA, }

Thanks!  I know all about the red/blue swap and just plain failed to see that that's what was happening ... and that I didn't take that into consideration in making my C array initializer ... or, more likely, that I swapped them twice.  Doh!
I'm sorry I wasted so much of your time on this.

I'm sorry I wasted so much of your time on this.

You've probably guessed by now, but I love answering questions like this 🙂

And thanks again to @zadjii-msft.  My color scheme sampler is working well both in a console and in WT.


Search issue with hasVirtualScroll

When hasVirtualScroll is true and onSearch event is set, search doesn't work.

Fixed in 2.8.0 release.


Improve the animation sequence after sending funds

Currently, it stuck in the action sheet, when you can watch an animation of the spinner. It should collapse immediately after you this is sent. So
a. You enter numbers and recipient
b. You confirm in the action sheet
c. You authenticate
d. Then, the slow animation (spinner and tick/checkmark) is only to indicate that you successfully sent an order. It does not mean that the transaction succeeded. It is only to tell you, that the action you started is on its way. Kind of like feedback from the button, that it was clicked.
e. We see the screen with an illustration
f. We come back to the screen where we started the process

Please take a look:
https://invis.io/QEZL2T2KD4Z
I sent you the invitation to Invision, please accept.

That mockup is really useful!

@colourfreak please check the latest version and close if complete.

@colourfreak please check the latest version and close if complete.

@colourfreak please check the latest version and close if complete.

@colourfreak please check the latest version and close if complete.

a. Please update the illustration https://zpl.io/V4gOmAM
b. Can we change "Show Transaction Details" to "OK, great!" as a user, I would like to just continue "show transaction details" is not necessarily what I want.  So it will end up in the same screen, but the name on the button is more user-friendly.


Add logo to auth layout

These changes were merged in from your other PR so I'm going to close this :D


[Logan] Enable diffing of video, math, widgets, iframes, imgs, svgs.

I'm a little nervous about this, as it looks like it will wrap these elements with ins/del tags like everything else. My concern is that we break any css selectors using direct child > math. Hopefully the selectors aren't that specific but I'd be lying if I said I haven't written those style selectors a ton of times.

Would it be worth porting a few of these tests over to see how it fares when a few operations like this are performed in tandem? https://git.inkling.com/web/html-comparer/pull/10/files#diff-e66e39757af5f0947a27bd75aa07be3eR1


[stable/spring-cloud-data-flow] different MySQL improvements

What this PR does / why we need it:

remove hardcoded image name: use the values from the MySQL subchart
remove hardcoded database name: use the value that yet exists
update the version of the MySQL subchart

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] title of the PR contains starts with chart name e.g. [stable/chart]

/assign @jlegrone

@sebastien-prudhomme please update your PR with latest master and fix the conflict

@sebastien-prudhomme please update your PR with latest master and fix the conflict

Done

/ok-to-test

/lgtm


Harware: Waterproof connectors

Waterproof connectors
2 x two-pole will do. The solenoid is connected internally

Make it simler and cheaper. Just buy a connector box and use some screw connectors.
The box can also be the way to make the electricity cable get through in an okay way.


Improvement: add the possibility to use any image from url or from path

Hi,
I'm currently trying to build userland (to build raspimjpeg) on the arm-runner-action, but cmake has a bug in the latest available version in Raspberry Pi OS, so I must download cmake from source and build it before building using cmake to build userland... and it takes around 3h 40min. So being able to have a cached image would be a great addition...
I do not have the knowledge to add this feature, but I suppose that the url or the path to the image could be passed using the base_image input.
Regards,
Vincent
P.S.: you can close this issue if it is not relevant.

Makes sense. Alternatively, you can always install binaries such as a dpkg that would be saved elsewhere.

PR #13 adds support for custom urls as base_image.

Fixed in #13 / #20


Which license?

Which license is the social-buttons-server released under?

LICENSE file is in repo, changed to MIT.

@thomasdavis: oh, I'm very sorry! I was in the gh-pages branch, where there is no license. Thanks though!


why not empty the cmd line when clicking  debug

Type: Bug
I typed 'rm -rf ' but didn't execute, but I forgot, then I clicked 'debug', the debug command was automatically typed and execute with the previous 'rm -rf', and my file were all deleted.
VS Code version: Code 1.74.3 (Universal) (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T17:07:18.579Z)
OS version: Darwin x64 20.6.0
Modes:
Sandboxed: No
Remote OS version: Linux x64 5.13.0-28-generic

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.75.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Sorry for the trouble. Tracking this in https://github.com/microsoft/vscode/issues/106743


Customize underlying tsserver setup()

Hey there, thanks for your work on this plugin.
I would like to turn off single_file_support (or change root_dir), since it causes many conflicts when working with Deno, how am I supposed to do that?
https://github.com/pmizio/typescript-tools.nvim/blob/3b05146d96b7c3d86e5f540d9ac871823e318d5a/lua/typescript-tools/init.lua#L15-L44

You can just put them into plugin setup function all code you pointed is only default configuration.
require("typescript-tools").setup {
  -- here put lspconfig props
  settings = {
    -- here is place of any option introduced by this plugin
  }
}

Thank you!
I completely missed the first part of the README/Configuring, I noticed that on_attach was shown in the example, but didn't make the connection that the rest would work.


Orchestrator partial + storage and grain model enhancements

Anything under Node.Contracts and Orchestrator.Client is at the very beginning so no use reviewing, it'll change.
LiteDbStorageProvider conversion to bson
Accounts grain business and persistence layers

Take note that my latest changes include modifications to most dependencies to have the latest stable nuget versions and to have the same versions on all projects


No Icons, can't save changes to properties.

PWSH RC2
How

Uninstall RC1
Install RC2

What
Application installs without icons.
Cannot save changes to properties:
Unable to modify the shortcut
C:\ProgramData\Microsoft\Windows\Start Menu\Powershell 7-preview (x64).lnk
Check to make sure it has not been deleted or renamed.

Sounds like a duplicate of #11680
Workaround: https://github.com/PowerShell/PowerShell/issues/11680#issuecomment-578541388

Alright, thanks, I'll take a look. Closing.


Skip pending request if new content is set

Starcounter/starcounter-include/issues/25

@warpech What about now?


Add in fighting guaranteed Winked/Romantic Arrow monsters?

Once you have gotten past the end of the expected window for wink/arrowed monsters, they are guaranteed. Can be tracked with the counter  1504:Romantic Monster window end loc=* type=wander:rparen.gif (i think).  Obvs not required, would be nice to have, can probably add a PR for myself, just adding this as a reminder to do so.

use getCounters not get(), but yeah, very good idea


ESLint: Class method uses this

A rule class-methods-use-this is buggy regarding React Components (as noticed by @ronaldruzicka ), and also here.
Possible solutions:

Ignore the rule completely (what is a benefit of it?)
Use exceptMethods option for this rule, as mentioned here, to make it ignore React methods until they are properly parsed.

Ad 1. it tells you that class method is pointless when you don’t need this and that you can use plain function instead.
Ad 2. that would help in case of componentDidMount etc. but once you need custom renderSomething() methods, it’s useless.
I like the hint that you shouldn’t overuse class methods when not needed. But writing render helpers outside of React classes seems weird to me as well.
So I don’t really know at the moment.
cc @janpanschab

Support for exceptMethod has been added
Let’s try it and close the issue.


Figure out how we can re-enable healthchecks on the kube-apiserver

We disable healthchecks on the api-server because paired with bootstrapping and checkpointing, they can make the pivot process somewhat flaky due to the apiserver using an flock (and "running", but not actually serving requests).
For example during bootstrap:

bootstrap api-server starts (acquires lock)
self-hosted api-server starts (waits on lock)
kubelet tries to health-check self-hosted api-server, but fails (not listening yet)
kubelet keeps restarting apiserver over and over
eventually bootstrap api-server stops (and gives up lock)
As long as the kubelet doesn't die in this window, it will eventually restart the self-hosted apiserver (but this could be a long window due to backoff).

Or during checkpointing:

api-server is down
Start a checkpointed copy of apiserver (and acquire lock)
eventually self-hosted api-server comes back up (waits on lock)
kubelet tries health-check on self-hosted apiserver and fails (because it's not listening yet)
kubelet keeps restarting apiserver container over and over
The problem here is that the checkpointer must use the /runningcontainers endpoint to determine if there is a local apiserver running (https://github.com/kubernetes-incubator/bootkube/blob/0a9b736d7e99a9156a2e6d7ce084213f9a19032a/cmd/checkpoint/main.go#L34-L36), because the kubelet's /pod endpoint could have stale data. However, the /runningcontainers endpoint is only looking at the container runtime state - which means that it might think that no apiserver is running (if it looks while kubelet is restarting the apiserver - there is no container). It then thinks it needs to restart the checkpointed copy of the api-server, then starts process over again. This leads to it sometimes eventually settling down based on timing, but it is quite flaky.

We could revisit how we determine local running state. Or maybe revist the lock process (upstream something like the kubelet lock, where you can specify "exit-on-lock-contention" so we can coordinate control between two components). Or we might continue to work around it maybe by doing something like using an exec healthcheck that is aware of the lock state (via @chancez).

Some of kubelet's flock stuff is in util/flock, so it can be re-used. The --exit-on-lock-contention code which monitors the lock file is here but can probably be shifted over. I'm wondering if a readiness probe can be set up using something like inotify to pass the moment the self-hosted apiserver acquires the lock. Then a HTTP liveness probe will ensure the pod can serve traffic, with an initialDelaySeconds that equals or slightly exceeds the flock --timeout. What do you think?

The readiness probe against the lock is an interesting idea. IIRC, however, the liveness would be checked prior to readiness (and would still cause the pod to die). Could be wrong on that though.
Also @diegs and I were talking about this yesterday, and we may not need the lock at all. This is mostly to protect against flapping during checkpointing. From another thread:

We have to use two api endpoints on the kubelet (/pods and /runningpods) to safely make checkpointing decisions. The /pods endpoint gives us the internal representation of what the kubelet thinks is the local state. These are full pod-specs/status. However, this data is only updated after it is successfully been persisted to the apiserver. So the process is "inspect local state to populate a list of pods --> update apiserver --> update internal state --> can now be queried by /pods". If an apiserver cannot be contacted -- this data becomes stale (represents state the last time it could successfully contact an apiserver).

The "/runningpods" endpoint, however, is the local state as populated by the container runtime. This information is up to date regardless of whether an apiserver can be contacted. It also is only a small subset of the podspec/status.

Because the /pods endpoint could be stale when an apiserver is absent -- we can't make "should we start/stop this checkpointed pod" decisions based solely on that information. For example, we could check /pods and see that an apiserver pod is in a running state, when it really is not.

Instead we need to rely on the "/runningpods" endpoint which is essentially a direct mapping to docker ps. The problem here with port conflicts is that when the checkpointer checks /runningpods -- the apiserver pods might be fighting over the port and in a failure loop. So it will go into "should start checkoint" then "should stop checkpoint" over and over again because it may see the pod happened to try to run, or no pod running purely depending on that moment in time.

The flock allows us to see, for example, that when /runningpods is called the "replacement" apiserver pod is in fact running (even if its just sitting on the flock) -- so the checkpointer can stop the checkpointed copy -- then lets the real apiserver take over.

However, all we really care about is checking the "intention" of running a pod, which we could also enforce by having a container that does nothing but sleep forever. This way when we query /runningpods we will see the apiserver pod (even if it's in a failure loop), because one container of the pod (sleep forever) is still running.
(some testing on this is being done in: https://github.com/kubernetes-incubator/bootkube/pull/525/files#diff-1a342de84d1bbb83d110cdca78afa143R153)

IIRC, however, the liveness would be checked prior to readiness (and would still cause the pod to die)

Not if you set the initialDelaySeconds higher or equal to the flock timeout. After 30s, the apiserver will either be running (and the liveness probe will begin) or dead because it hasn't acquired the lock (in which case no liveness probe ever runs). That's my assumption though.

However, all we really care about is checking the "intention" of running a pod, which we could also enforce by having a container that does nothing but sleep forever. This way when we query /runningpods we will see the apiserver pod (even if it's in a failure loop), because one container of the pod (sleep forever) is still running.

I get how this might help from a checkpointing perspective, but what about the scenario outlined in https://github.com/kubernetes/kubernetes/issues/45786 where there are race conditions. You'd still need some way of:

Ensuring that the self-hosted pod starts up after the bootstrap one in a synchronised way
Ensuring that the self-hosted pod is actually listening when it should be (including anticipated delays when we know it might not be)

Would the sleeping sidecar address these as well or are you just thinking about it from a checkpointing POV?

all we really care about is checking the "intention" of running a pod, which we could also enforce by having a container that does nothing but sleep forever. This way when we query /runningpods we will see the apiserver pod (even if it's in a failure loop), because one container of the pod (sleep forever) is still running.

As a side-note, last I looked into this we don't actually need a sidecar or a flock for the checkpointing -- the /runningpods endpoint will still list the pod because the pause pod is still there (even if the actual apiserver container has died). This is how we can determine "intention".
It's been a while since I looked to make sure our code respects that -- but it should just be looking for the pod by name (and it would be in the /runningpods output with just the 'pause' container

Looks like etcd health has been added to the apiserver healthcheck: https://github.com/kubernetes/kubernetes/pull/49412 -- that makes re-enabling this even more useful.
There have been reports that when replacing an etcd node the apiserver doesn't actually try and re-establish a connection. I haven't validated this - but this type of check would mitigate that (would restart the apiserver if healthcheck failed).

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

This is something we need to get into render so that its enabled by default. Hopefully we should target 1.17

/area render

/remove-lifecycle stale

/remove-lifecycle stale

I use readinessProbe for kube-apiserver Pod in Flexkube, however, I don't use pod-checkpointer at the moment. However, I use SO_REUSEPORT for kube-apiserver Pod, which will be added in 1.19, so I can have multiple instances running at a time (the readiness probe check however, is satisfied if any instance is running, as it runs on host network, which is not great).
I plan to use pod-checkpointer to avoid keeping static controlplane running. However, it seems to me, that for automatic recovery, kube-controller-manager and kube-scheduler pods should also be checkpointed, but they should have a different checkpointing mechanism than kube-apiserver. For those pods, leader election should be checked instead.
I've also experienced some flakyness with pod-checkpointer while working on Lokomotive, so perhaps pod-checkpointer needs some improvements to resolve this.

@invidian We see this often with pod-checkpointer. I think we should absolutely look into this, soon. Would you be up for a meeting on this?

@andrewrynhard meeting sounds good, though I don't have much knowledge about the pod-checkpointer. Let's coordinate on Slack :+1:

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

This project is being archived, for more information see this issue:
https://github.com/kubernetes/org/issues/2824
/close


Consider switching from mdBook to Bookdown

I hoped there will be more activity in mdBook, but it still seems immature:

https://github.com/rust-lang/mdBook/issues/815
https://github.com/rust-lang/mdBook/issues/88

We should consider to ditch it in favor of Bookdown which has been used for making professionally-looking books for quite a while. This switch will allow us to have both online and PDF/ePub versions out of the box.

There is even better alternative, which is more language-agnostic and more Python-friendly, Quarto:

Quarto® is an open-source scientific and technical publishing system built on Pandoc. Quarto documents are authored using markdown, an easy to write plain text format.
In addition to the core capabilities of Pandoc, Quarto includes:

Embedding code and output from Python, R, and JavaScript via integration with Jupyter, Knitr, and Observable.
A variety of extensions to Pandoc markdown useful for technical writing including cross-references, sub-figures, layout panels, hoverable citations and footnotes, callouts, and more.
A project system for rendering groups of documents at once, sharing options across documents, and producing aggregate output like websites and books.
Authoring using a wide variety of editors and notebooks including JupyterLab, RStudio, and VS Code.
A visual markdown editor that provides a productive writing interface for composing long-form documents.

See all examples at their gallery.

Closing this since we completed the switch to Quarto.


Add Redeemers to functionality to build unbalanced transaction

We need the Haskell server to convert PlutusData wrapped Redeemers to indexed redeemers. Once this has been implemented, we need to tweak the constraint processor to incorporate this.

Blocked on #156 for now

Just writing down a few notes on how we can do this so I don't forget:

Convert the redeemers to the Types.Transaction style and set the execution units in each one to zero (we have every other piece of information we need)
Attach the redeemers, datums, etc... to the transaction. At this stage the missing ex units in the redeemers is OK (according to Ogmios docs)
Convert the transaction (it can still be unbalanced at this stage) to hex-encoded string and call Ogmios protocol for evaluating the tx without submitting it

Created a separate issue for Ogmios https://github.com/Plutonomicon/cardano-browser-tx/issues/174 as we are hardcoding the ex units for now.

Closing for the new issue of using Ogmios


about the [single_line_after_imports] rule, probably not a bug, just a little strange

For configuration or updating questions please read the README and UPGRADE documentation,
or visit: https://gitter.im/PHP-CS-Fixer
When reporting an issue (bug) please provide the following information:
The PHP version you are using ($ php -v):
=> 7.3.8
PHP CS Fixer version you are using ($ php-cs-fixer -V):
=> 3.0.0-DEV
The command you use to run PHP CS Fixer:
=> php-cs-fixer fix
The configuration file you are using, if any:
<?php
return PhpCsFixer\Config::create()
    ->setRules([
        'single_line_after_imports' => true
    ]);

If applicable, please provide minimum samples of PHP code (as plain text, not screenshots):

before running PHP CS Fixer (no changes):

<?php

use namespace1;
use namespace2;
// use namespace3;
echo 'test';

with unexpected changes applied when running PHP CS Fixer:

<?php

use namespace1;
use namespace2;

// use namespace3;
echo 'test';

with the changes you expected instead:

<?php

use namespace1;
use namespace2;
// use namespace3;

echo 'test';

According to PSR-2 rules, there MUST be one blank line after the use block.
But if the last use statement in the use block is commented out, the last statement will be put after a new blank line;
I think this isn't a bug, just point out that.

I don't think we can do anything about this: it would be hard to reliably detect that the comment is temporarily disabled code.
Even if we were able to, I think such code should be removed completely so PHP CS Fixer should never even see it. I'd prefer a rule dedicated to that rather than updating all existing rules to handle this comment as if it was actual code.

closing as answered, feel free to discuss some more here though :)