gulp-uglify generates source maps but mapping is incorrect

I have this task to generate bundled & minified JavaScript file:
    return gulp.src(paths)
        .pipe(paths.sortByFilename ? sorter(key) : gulpUtil.noop())
        .pipe(plumber(eh('Error compiling javascript (' + key + ')')))
        .pipe(sourcemaps.init())
        .pipe(gulpBabel())
        .pipe(gulpConcat(key + '.js'))
        .pipe(gulp.dest(cp.build(gomModule) + '/js'))
        .pipe(gulpNgAnnotate({add: true, single_quotes: true}))
        .pipe(gulpUglify())
        .pipe(gulpRename({ extname: '.min.js' }))
        .pipe(sourcemaps.write("."))
        .pipe(gulp.dest(cp.build(gomModule) + '/js'));

Looks ok: .min.js gets mangled etc and sourcemaps are generated. But unfortunatelly debugging is not possible. Mapping is incorrect - stepping through the code / locals is broken. gulp-uglify seems to be the culprit because if I remove it from the the task, mapping works (with ngAnnotations in place as well as with babel transformation). Any idea that's going on? User versions are:
"gulp-sourcemaps": "^2.1.1",
"gulp-ng-annotate": "~2.0.0",
"gulp-babel": "^6.1.2",
"gulp-concat": "^2.6.0",
"gulp-uglify": "^2.0.0",
"gulp-rename": "~1.2.0"

Right now my order is something like the following and I'm also experiencing this. Running w/o uglify allows for it to map properly.
.pipe(sourcemaps.init())
    .pipe(babel({presets: ['es2015']}))
    .pipe(uglify())
    .pipe(rename(function(path) { ... }))
    .pipe(sourcemaps.write('.', { ... }))

I just rolled back to v1.5.4 and this is now working for me.

I am also getting issue with sourcemap generated. with both 2.0 and 1.5.4 also.

@pinalbhatt Posting what your chain looks like can help diagnose the problem.

I am also having this issue.
The following does not work with gulp-uglify 2.1.2, and problematic sourcemaps are produced:
return gulp
	.src( 'src/app.js' )
	.pipe( sourcemaps.init() )
	.pipe( babel( {
			presets: [ 'es2015' ],
			plugins: [ 'transform-class-properties' ],
		}))
	.pipe( uglify() )
	.pipe( sourcemaps.write( '.' ) )
	.pipe( gulp.dest( 'dist' ) )

However, as per @g5codyswartz's suggestion, it works fine with gulp-uglify 1.5.4
There is clearly some issue in later versions of gulp-uglify
"babel-plugin-transform-class-properties": "^6.23.0",
"babel-preset-es2015": "^6.24.0",
"gulp": "^3.9.1",
"gulp-babel": "^6.1.2",
"gulp-eslint": "^3.0.1",
"gulp-sourcemaps": "^2.4.1",
"gulp-uglify": "^2.1.2"

I think I am still having this problem, but there seem to be a lot of people complaining about sourcemaps without any solutions. The problem I am having is that the sourcemaps are generated, but if I add a console.error on a certain line the line number in the browser console is wrong. Which results in being sent to the wrong line in the file. I also checked the file itself and that actually perfectly matches the development file, so not sure why browsers are mapped to the wrong line.
I am using gulp-uglify in combination with gulp-concat and disabling gulp-uglify does solve the problem, but ofcourse that is no solution. Besides both should support sourcemaps just fine according to the documentation. I already tried all other suggested sollutions in other issues and this issue and nothing seems to help. I tested in both Firefox and Chrome and both had the same issue.
Is there any update or solution for this?

Spent a bunch of time looking into this, and it appears the latest<EMAIL_ADDRESS>+<EMAIL_ADDRESS>is applying the sourceMap mapping twice - if I replace applySourceMap(file, sourceMap); with instead simply file.sourceMap = sourceMap; (in minify.js), the output sourcemap appears to be correct.  Since gulp-uglify is passing in the existing mappings to uglify-js, it seems uglify-js is generating the whole composite map (not just a map from the input source).

IIRC this is supposed to be fine, but there's probably enough ambiguity in the sourcemap spec to cause problems.
I can apply that change, but need to figure out a good way to test this so there's no regression.

It's seems like it can't possibly be fine.  If map A maps lines 10:5 and 8:4 and map B does a (relative) map of 5:4 and 4:2, the output must be 10:4 and 8:2, but if that relative map is applied a second time, the output ends up being along the lines of mapping lines 10:2, 8:2 and 5:4, 2 of 3 of which are not valid.  In this case, it's applying what should be an absolute map (mapping all the way back to the original source) as a relative map.  Judging from another comment in that file, it makes me think that at one point uglify-js was not generating the absolute mapping, but a relative mapping (which is what vinyl-source-map-apply expects).  I am quite unfamiliar with the low-level stuff and the spec, so I certainly could be confused about something in here though =).

What I meant was the sourcemap tool should have enough information to know that it has the output mapping as input, not a relative mapping, and act accordingly. I haven't thought about the broken state of sourcemaps in so long it's possible I'm wrong.
I recall needing to do this to get the sourcmaps output by UglifyJS to attach to the sourcemaps of the input, particularly in cases where the original code was written in a language compiled to JS (like coffeescript). Without applying the sourcemaps you wouldn't see the original code in the browser.

I think applySourceMap always thinks it's applying a relative mapping, as it does see the old mappings, however since we're also passing the mappings into uglify-js it seems as though it's generating an absolute mapping back to the original (not just back to the source that was passed in), and then when that's passed into applySourceMap, it's applying it as a relative mapping (since from context, that's what it appears to be).
All that being said, the change I mentioned definitely fixed my case for piping gulp-babel into gulp-uglify (as is the case with all of the people posting earlier in this thread).  In a different project I'm piping browserify (with babelify as a transform) into gulp-uglify, and that project seems to be getting perfect source mapping (but now I would expect it to have the same issue), so there's something I'm missing.  I'll dig into it a bit more tomorrow or Monday and report back.  That other project is on a different version of gulp, babel, uglify, and gulp-uglify at this point, so it might take some poking =).

I'm guessing UglifyJS changed their behavior at some point, you can find older issues where that definitely wasn't the case.

Good find @Jimbly
applySourcemaps is not an idempotent operation. If encoding the sourcemap twice worked in the past, that was probably just a fortunate accident.

Ah, figured out why it was mysteriously working in my other project: When doing babel -> browserify ->gulp-uglify, the browserify step is "renaming" the file, so in that case, the applySourceMaps in gulp-uglify ends up being a no-op (source-map finds no mappings with a matching file name, so discards all of the old mappings and just copies in the new one).  If my browserify bundle was named exactly the same as one of my input files, I suspect it would be going wrong in the same way.
Conclusion: Since uglify-js is already creating a merged sourcemap (because we ask it to by specifying options.sourceMap.content), gulp-uglify should just set file.sourceMap = sourceMap instead of calling applySourceMap(file, sourceMap). Opened PR #362


Challenge http://www.freecodecamp.com/challenges/waypoint-mobile-responsive-images has an issue

Same issue like in #625
Same settings. Attached a screenshot to even better show you guy!

closing as a duplicate of #625, thanks!

I'm having trouble with this waypoint -- is it just me? Or is there a problem?

Appears to be a problem here but no solution yet.🙍🏻

On Jul 9, 2015, at 7:13 PM, Melody Stone<EMAIL_ADDRESS>wrote:
I'm having trouble with this waypoint -- is it just me? Or is there a problem?
—
Reply to this email directly or view it on GitHub.

Aww, bummer - I was on a roll.

Since @terakilobyte closed this on May 27, this should be fixed. If you, @forevermelody, need help figuring out what went wrong and suspect it might be an error with your code, @ me on Gitter or ask in our help chat room.

Apparently you had to remove the s from the https in the second kitty photo. Thanks!


Validation issue with 110 (TW)

This number is an emergency short code defined in ShortNumberMetadata.xml.
Country/region affected (e.g., "US"): TW
Example number(s) affected ("+1<PHONE_NUMBER>"): 110
The phone number range(s) to which the issue applies ("+1 555 555-XXXX"):
The type of the number(s) ("fixed-line", "mobile", "short code", etc.): short code
The cost, if applicable ("toll-free", "premium rate", "shared cost"):
link to demo

Hi @wmh,
Sorry we never responded to this. The number is considered a valid short code, see our demo (isValidShortNumber()): http://libphonenumber.appspot.com/phonenumberparser?number=110&country=TW


Auditory and visual digit span

I'm planning to contribute auditory and visual digit span tasks. These will be based on the most common method that I'm aware of: 3 trials per list length, starting with short (3 item) lists. List lengths increase by 1 item, until the participant responds incorrectly to all 3 trials of a given list length.
I'm not sure which response method to use: textbox or number pad. Maybe two different versions - one for each response type.
Feedback and suggestions welcome! I'll try to submit the pull request in the next few weeks.

I have a version of auditory span task written in psychopy. It is intended to be done with an RA who records the verbal responses. What do you mean by textbox versus number pad?
When I did it, I used a staircase up/down design to get the span. I am a big fan of adaptive difficulty and there is a paper showing that it provides a better estimate of span. Once you finish the standard version I can make a staircase version also.

Cool, thanks @steffejr! I'll still plan to submit my simple stop rule version, because I think it's useful in some cases. But I agree that adaptive difficulty is often better, and it's definitely worth having a separate staircase version.

I have a version of auditory span task written in psychopy. It is intended to be done with an RA who records the verbal responses. What do you mean by textbox versus number pad?

With an online version of digit span, I suppose it's possible to record spoken responses (for instance using one of the audio-response plugins for jsPsych, although this one is currently in development and not part of the official library yet). But in my experience, the participant's response is usually recorded some other way in online versions. The two ways I've done this are (1) using a textbox (e.g. survey-text plugin), where the participant types the in the whole digit sequence, or (2) with a visual number pad on the screen, where the participant clicks on each digit, in order. The experiment factory's digit span demo uses a number pad for responses, if you want to see what I mean.
I'd be interested to know if you have any thoughts on the best method for recording responses. I don't have a strong preference. I guess one difference is that, with the textbox, the participant can see and review/edit their whole response before submitting it.
And with both the textbox and number pad response options, it's possible to require that each response contains exactly the expected/correct number of digits. But I prefer to keep the task similar to the in-person spoken response version (at least the one that I'm familiar with), where the participant is free to report more or fewer digits than were actually in the presented sequence.

Helo @becky-gilbert
I just tried the digit span demo. The number pad works well. I think it is as close as we can get to speaking responses. When you tell an RA an answer you do not get to "check" what you said. And there is no limit to the length of the number string you are "allowed" to respond.

Thanks for the feedback @steffejr! That all makes sense to me. I'll submit a version that uses a number pad to get the response, and with no requirement/limit for the number of digits given in the response.

Hello Becky, I don't know if you saw this but it may be useful...
https://svanhedger.github.io/jspsych/

I had not - looks very useful! Thanks!!


add support for various DNS providers

closes #409 , #341

@MiroslavRepka , re-review the last 2 commits (https://github.com/Berops/claudie/pull/530/commits/262120334b517ebf19512e70ce8ec096c60c511c https://github.com/Berops/claudie/pull/530/commits/68aeff5b8a2d7f1a06fbc858b5fc6bab82c1e595) I had to refactor a bit, the tests passed without any issues.


Crash when trying to read devices

Very neat tool.
Replication process:

Start app
Select devices, click enter
Select device under API description and click enter

Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at SmartThingsTerminal.Scenarios.Devices.Setup()
   at SmartThingsTerminal.Program.Init(Options opts)
   at CommandLine.ParserResultExtensions.WithParsed[T](ParserResult`1 result, Action`1 action)
   at SmartThingsTerminal.Program.Main(String[] args)

Thanks @rboy1 for reporting - this can happen if you either have no devices registered (or use the wrong API key).
Will fix in next release.

I have over 300 devices and the API key is correct because I can view the other items (Locations etc).

Can you try again with the latest release: https://github.com/daltskin/SmartThingsTerminal/releases/tag/0.11.1

It's not crashing anymore but it gives the following error:

Unknown error calling API: An item with the same key has already been added.
Key: XXX iPhone

Ah ok - this is a different bug (something I'll look at over the weekend).
I suspect you have more than 1 device with the same name - short term fix is to rename one of them.
I'll close this one and create a new bug.


whether this project will affect final accuracy

So great job. Thanks for your open source, I have some problems. My config file set image size to [[2048,600],[2048,800],[2048,1000]], which applied multiscale test.  I set the
opt_shape_param=[
[
[1,3,512,512],
[1,3,1000,2048],
[1,3,2048,2048],
]
],
but the accuracy drops from 54mAP to 44mAP.  Maybe the  opt_shape_param that I set  was inappropriate.  Can you give me some advice, thanks a lot.

I have tested this repo on cascade_rcnn_r101_fpn_1x_coco, get 42.7mAP(official 43.7mAP).
Seems your problem is caused by multiscale test.

Thank you!


rds-controller makes unnecessary updates to DBSubnetGroups

Describe the bug
While looking at logs from the rds-controller version v1.1.2 I saw the following log line:
ack-rds-controller-rds-chart-658df64797-cv7lg controller 2023-04-26T08:09:54.651Z
INFO	ackrt	desired resource state has changed
{"account": "redacted", "role": "", "region": "eu-west-1", "kind": "DBSubnetGroup",
"namespace": "default", "name": "private-subnets-redacted", "is_adopted": true,
"generation": 2, "diff":
[{"Path":{"Parts":["Spec","SubnetIDs"]},"A":["subnet-xxx860164a0","subnet-xxx547ae34f","subnet-xxx27e5227d"],
"B":["subnet-xxx860164a0","subnet-xxx547ae34f","subnet-xxx27e5227d","subnet-xxx27e5227d",
"subnet-xxx860164a0","subnet-xxx547ae34f"]}]}

The subnets are just repeated. No actual update is required.
Steps to reproduce
I expect simply creating a subnet group and restarting the controller should show this log line.
Expected outcome
Prevent unnecessary updates.
Environment

Kubernetes version
Using EKS (yes/no), if so version?
AWS service targeted (S3, RDS, etc.)

I am experiencing the same with rds-v1.1.4 - there is no change over the subnetIDs, yet the controller INFO	ackrt	desired resource state has changed
Here the full logs:
// 2023-07-03T07:20:21.335Z	INFO	ackrt	desired resource state has changed
{
  "account": "<masked>",
  "role": "",
  "region": "me-south-1",
  "kind": "DBSubnetGroup",
  "namespace": "aws-rds",
  "name": "dbsubgrp-rds-mssql",
  "is_adopted": false,
  "generation": 1,
  "diff": [
    {
      "Path": {
        "Parts": [
          "Spec",
          "SubnetIDs"
        ]
      },
      "A": [
        "subnet-0de319e8bf9aad3d6",
        "subnet-07c7c491e929dc256",
        "subnet-0394c035cfcbe4ee5"
      ],
      "B": [
        "subnet-0de319e8bf9aad3d6",
        "subnet-07c7c491e929dc256",
        "subnet-0394c035cfcbe4ee5",
        "subnet-07c7c491e929dc256",
        "subnet-0394c035cfcbe4ee5",
        "subnet-0de319e8bf9aad3d6"
      ]
    }
  ]
}

// 2023-07-03T07:20:21.555Z	INFO	ackrt	updated resource
{
  "account": "<masked>",
  "role": "",
  "region": "me-south-1",
  "kind": "DBSubnetGroup",
  "namespace": "aws-rds",
  "name": "dbsubgrp-rds-mssql",
  "is_adopted": false,
  "generation": 1
}

This thing happens every time the rds controller is started for sure,
however I remember seeing this on a regular basis - maybe each day - just lost those logs now and can't confirm.

any update ?


Valid certificates are reported to expire soon

Hi, I have credhub_exporter in use. However, some time ago it started to report certificates that are still valid (until approx. August 2019), to expire within 30 days.

Hi @amuessig
That's weird, I have no clue on what is happening.

what version of credhub_exporter are you using ? (or what version of prometheus-bosh-release)

are all your certificates firing an alert ? or only some of them ? In the later case, is there anything special about those certificates ?

can you run exporter with debug log level to see if anything is going wrong ?

to tell if the problem is in the alert definition or in the exporter itself, can you check if the raw exported values match you certificate expiration date ? The exported format should give the unix timestamp of the NotAfter field of your certificate.

Here are some usefull commands:

extract Not After date from a certificate:
cat certificate.pem | openssl x509 -text -noout | grep 'Not After'

extract raw exported timestamps from prometheus:
round(max_over_time(credhub_certificate_expires_at[1h]))

Hi @psycofdj, thanks for replying.

I have prometheus v24.0.0 installed (credhub_exporter v0.1.6)
It's only three of them, they are also created by bosh. However, all are related to nats: nats_clients_director_tls, nats_clients_health_monitor_tls, nats_server_tls
I didn't run the exporter with debuglog level, yet. I will try to do so.
I manually looked on the certificates (which I even rotated two days ago. That's my result:

credhub get -n /<somepath>/nats_clients_director_tls --output-json | jq -r .value.certificate | openssl x509 -text -noout | grep 'Not'
            Not Before: Mar  6 12:56:43 2019 GMT
            Not After : Mar  5 12:56:43 2020 GMT
credhub get -n /<somepath>/nats_clients_director_tls --output-json | jq -r .value.ca | openssl x509 -text -noout | grep 'Not'
            Not Before: Mar  6 12:56:42 2019 GMT
            Not After : Mar  5 12:56:42 2020 GMT

# 2nd certificate
credhub get -n /<somepath>/nats_clients_health_monitor_tls --output-json | jq -r .value.certificate | openssl x509 -text -noout | grep 'Not'
            Not Before: Mar  6 12:56:43 2019 GMT
            Not After : Mar  5 12:56:43 2020 GMT
credhub get -n /<somepath>/nats_clients_health_monitor_tls --output-json | jq -r .value.ca | openssl x509 -text -noout | grep 'Not'
            Not Before: Mar  6 12:56:42 2019 GMT
            Not After : Mar  5 12:56:42 2020 GMT

# 3rd certificate
credhub get -n /<somepath>/nats_server_tls --output-json | jq -r .value.certificate | openssl x509 -text -noout | grep 'Not'
            Not Before: Mar  6 12:56:42 2019 GMT
            Not After : Mar  5 12:56:42 2020 GMT
credhub get -n /<somepath>/nats_server_tls --output-json | jq -r .value.ca | openssl x509 -text -noout | grep 'Not'
            Not Before: Mar  6 12:56:42 2019 GMT
            Not After : Mar  5 12:56:42 2020 GMT

The weird thing is, I have several different BOSHs with NATS running and this false positive occurs only on one of the deployments.

@amuessig
Still no clue... but not giving up :)

{deployment="credhub",environment="....",id="...",index="...",instance="...",job="credhub",name="nats_server_tls-ca",path="//nats_server_tls"}<PHONE_NUMBER>

Is this extracted from the exporter's output or from prometheus database ? If the later, could you check the raw value given by a curl on /metrics ?
Have you ever rotated these certificates before the problem occurs ? I will run some test around this.
Also, I see that these certificates (coming from bosh-deployment, I assume) are built with specials options. I'll check and see if anything is going wrong in the exporter with those.

@psycofdj
Sorry, the above export was from prometheus with the query you suggested.
Now, I directly went to the prometheus vm and did: curl localhost:9358/metrics
Result:
credhub_certificate_expires_at{deployment="credhub",environment="...",id="...",index="1",name="nats_server_tls-ca",path="/<somepath>/nats_server_tls"} 1.583413002e+09

Actually, that looks goot to me at that point..

And after "expiration" date (as it stated wrongly in the alerts), the alerts were set to state resolved.

That's very interesting. The exported value is actually correct<PHONE_NUMBER> = 5/3/2020 12:56:42).
Do you have any idea why the value in prometheus is different ? Is the exporter correclty scrapped by prometheus ?

And after "expiration" date (as it stated wrongly in the alerts), the alerts were set to state resolved.

Did not get your point, can you explain that again ? Thanks :)


exclude tests that fail inconsistently that aren't product issues

successful test run on soc9 rc3 - https://ci.nue.suse.com/job/openstack-ardana-tests/604/


Fix dashboard row style and course date view-style

https://openedx.atlassian.net/browse/MA-1599
This PR fixes 2 things:

Fix display of course date to upper case on MyCourses screen.
Fix row style of CourseDashboard screen according to mockups.

MyCourses
Dashboard

@aleffert @bguertin @1zaman plz review

:+1:


Updating translation file

There are 11 commits here, there should be 1

@MarshallOfSound
Yep, I think I messed up a bit, since I'm not that advanced with Git. Can you please advice on how to clean this up so I have first my fork updated with your changes, then update the translations file and then pull request the single commit?
I've been trying with https://help.github.com/articles/syncing-a-fork/ but either I messed up or I don't entirely understand how to update the fork. Should I re-fork and start from scratch?


Fix paren mismatch in umd wrapper

Thanks for following up to remedy this. I should have done a better job reviewing #319, but I'm glad we have this in now.


fix(canonical service): add env vars for canonical service

The primary purpose of this PR is to expose canonical service information in a way that can be leveraged by Envoy to add custom trace span tags for the service. Envoy supports a few different types of custom tag extraction, including Environment.
[ X ] Policies and Telemetry

/cherry-pick release-1.6

@douglas-reid: once the present PR merges, I will cherry-pick it on top of release-1.6 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

We can also do this in the code - we already have access to all labels via downward API. This may be preferred though for explicitness

I was trying to avoid any code changes, if possible. This seemed like the simplest approach (?), but I don't have strong feelings.

Do we need this for gateway?

Yes. I always forget those yamls.

@douglas-reid: new pull request created: #23588

In response to this:

/cherry-pick release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


No notification

I'm using a foscam FI9853EP with the plugin. I'm having troubles with notifications (Home and Eve app). I get notifications when changing the status (away, night, etc) however no notifaction when motion is detected. I do get them in the log file:
 [1/10/2017, 4:03:40 PM] [Foscam] FI9853EP Motion Detected!
[1/10/2017, 4:03:41 PM] [Foscam] Foscam-NG: Getting snapshot.
[1/10/2017, 4:03:41 PM] [Foscam] Foscam-NG: Getting snapshot.
[1/10/2017, 4:03:44 PM] [Foscam] Foscam-NG: Got snapshot.

If i go to the Home app manually I can see the snapshot.

  EventedHTTPServer [::ffff:<IP_ADDRESS>] Sending HTTP event '1.44' with data: {"characteristics":[{"aid":1,"iid":44,"value":true}]} +125ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] Sending HTTP event '1.44' with data: {"characteristics":[{"aid":1,"iid":44,"value":true}]} +22ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP request: /characteristics +25ms
  HAPServer [CC:22:3D:E3:CE:30] HAP Request: PUT /characteristics +4ms
  Accessory [Homebridge] Processing characteristic set: [{"aid":56,"iid":9,"ev":true},{"aid":56,"iid":10,"ev":true}] +9ms
  Accessory [Homebridge] Registering Characteristic "On" for events +4ms
  Accessory [Homebridge] Registering Characteristic "Brightness" for events +4ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP Response is finished +8ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP request: /characteristics +56ms
  HAPServer [CC:22:3D:E3:CE:30] HAP Request: PUT /characteristics +4ms
  Accessory [Homebridge] Processing characteristic set: [{"aid":57,"iid":9,"ev":true},{"aid":57,"iid":10,"ev":true}] +8ms
  Accessory [Homebridge] Registering Characteristic "On" for events +4ms
  Accessory [Homebridge] Registering Characteristic "Brightness" for events +4ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP Response is finished +14ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP request: /resource +50ms
  HAPServer [45:FD:69:16:9F:DC] HAP Request: POST /resource +4ms
[1/10/2017, 4:20:46 PM] [Foscam] Foscam-NG: Getting snapshot.

Are you able to get motion notification randomly or you never get it at all? Since your log has FI9853EP Motion Detected!, homebridge already notify HomeKit that motion is detected. HomeKit is responsible on generating the push notification. We can do nothing about that. Based on my experience, the motion notification is hit or miss when I am in the same network (LAN) and it gets worse if I am not in the same network (WAN).

@adekka , which version of node.js are you using?


Better formating for file-based reuslts pages

monospace font for reform file text
white space respected in text from file
More descriptive/appropriate text on file-based reform input page

Resolves #358 In particular, the text for the file is displayed in a monospace font with whitespace respected:

cc @martinholmer @MattHJensen

@talumbau,  Thanks so much for the improvements in TaxBrain pull request #365.

@talumbau,  I tried using the taxbrain/file/ page, but it didn't seem like the changes in TaxBrain #365 were active yet.  When do you expect #365 changes to be active on the website?

Hmm... I'm not sure what you mean. When I go to www.ospc.org/taxbrain/file, I see the following:

text changes on the input page based on suggestions in #358, including the link the REFORMS.md file on Github
the desired text on the results page before the JSON parameters are displayed
monospace font and white space respected on display of file text.

For example, this reform:
http://www.ospc.org/taxbrain/6433/
So can you describe in what way the processing on www.ospc.org/taxbrain/file does not reflect what is described in #365 ?

@talumbau,  Look at this page (which doesn't have the spacing right):
http://www.ospc.org/taxbrain/6434/

Ah, so the monospace font and respecting whitespace does not occur on error reporting. I will create an issue to fix this.


linux-kernel: add instructions about Xenomai4

Add instructions about Xenomai4
Readiness

[ ] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[ ] Run spelling and grammar check, preferably with linter.
[ ] Avoid changing any header associated with a link/reference.
[ ] Step through instructions (or ask someone to do so).
[ ] Review for wordiness
[ ] Match tone and style of page/section.
[ ] Run make linkcheck.
[ ] View HTML in a browser to check rendering.
[ ] Use semantic newlines.
[ ] follow best practices for commits.

[ ] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[ ] End message with sign off/DCO line (-s, --signoff).
[ ] Sign commit with your gpg key (-S, --gpg-sign).
[ ] Squash commits if needed.

[ ] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for 534ceea are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2201/docs/artifacts/html/index.html

Docs for 534ceea are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2201/docs/artifacts/html/index.html

Docs for 3ee93d85001e903ebaa686eab55b2b897a473aff are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2216/docs/artifacts/html/index.html

suggestions applied

Docs for 1493deb4cf0f4c666723c34d1486bcbff79391cc are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2217/docs/artifacts/html/index.html


fcitx5-skk

Tests:

Dict can be loaded (see Debug output)
type aiueo to get あいうえお
type q followed by aiueo to get アイウエオ
type Nihongo SPC TAB to get 日本語
type Hajimemashite SPC Yoroshiku SPC o Negai SPC shimasu to get 初めまして宜しくお願します
type Koushou SPC SPC to get a candidate window

test user word:

type Kenryuu SPC, preedit='🔻けんりゅう【】'
type Inui SPC, preedit='🔻けんりゅう【乾】'
type Ryuu SPC and select 隆, preedit='🔻けんりゅう【乾隆】'
type RET to commit
type Kenryuu SPC to get 乾隆 directly

Also I'm not able to build it locally.
CMake Error in src/CMakeLists.txt:
  Imported target "PkgConfig::GObject2" includes non-existent path

    "/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include/ffi"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

  * The path was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and references files it does not
  provide.

Reproduced. I'll try to solve it soon.

  Imported target "PkgConfig::GObject2" includes non-existent path

    "/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk/usr/include/ffi"

You may need to reinstall pkg-config. It seems to hardcode MacOS SDK path.


Enable CLI access

Add CLI access to main.py which parameterizes input directory.

Closed by #6 .


Remove kiam from default apps

What this PR does / why we need it
Now we use irsa.  Do we still need to deploy kiam? I guess not.
Checklist

[X] Update changelog in CHANGELOG.md.

This can't be merged until we fix external-dns-app to work with IRSA.


Update private-dns-getstarted-cli.md

\ is incorrect, ` is required for new line. Example of attempt to run code with \
At line:3 char:5

--zone-type Private \

~

Missing expression after unary operator '--'.
At line:3 char:5

--zone-type Private \

~~~~~~~~~

Unexpected token 'zone-type' in expression or statement.
At line:4 char:5

#please-close


in windows : composer install: it is missing from your system. Install or enable PHP's pcntl extension.

as documented in the readme. file 👍 in windows 10
after clone when run composer install this error happened:
 composer require laravel/horizon
Using version ^5.7 for laravel/horizon
./composer.json has been updated
Running composer update laravel/horizon
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - laravel/horizon[v5.7.0, ..., 5.x-dev] require ext-pcntl * -> it is missing from your system. Install or enable PHP's pcntl extension.
    - Root composer.json requires laravel/horizon ^5.7 -> satisfiable by laravel/horizon[v5.7.0, ..., 5.x-dev].

To enable extensions, verify that they are enabled in your .ini files:
    - C:\xampp\php\php.ini
You can also run `php --ini` inside the terminal to see which files are used by PHP in CLI mode.

Description:
after clone when run composer install this error happened:
To Reproduce
Steps to reproduce the behavior:

clone
go to cloned repo folder
cp .env.example .env
run composer i
see the error

Expected behavior
composer install run successfully then Horizon install carefully  on windows

for now, I remove horizon and run composer I to find a better solution
composer remove laravel/horizon
composer i

You need to install the pcntl extension. I have no idea how to do that on Windows though.
I cannot recommend windows as development environment, you should try on WSL2 👍


Implement deduction guides

@Naios

It would be nice if function_base supported deduction guides similar to std::function so that lambdas can have sensible type erased deductions.
https://en.cppreference.com/w/cpp/utility/functional/function/deduction_guides

Commit Hash
2d3a878ef19dd5d2fb188898513610fac0a48621
Expected Behavior
This should compile
fu2::unique_function([](int &i) { return i + 5; })

The idea is awesome, I would highly appreciate a pullrequest for this feature.


Use read lock when iterate tablet_map in TabletManager::start_trash_sweep

For #3299
In our cluster, there are many stream load jobs.
We found that these jobs slow down periodically.
By add logs, we found that TabletManager::start_trash_sweep will take write lock for a long time.
In this PR, we split start_trash_sweep in two phases:
First, iterate the tablet map with read lock.
Second, remove the empty tablet in map with write lock.

@liutang123 Please add a issue for this PR.

By add logs, we found that TabletManager::start_trash_sweep will take write lock for a long time.

How long? And after this PR applied, how long will it be?

By add logs, we found that TabletManager::start_trash_sweep will take write lock for a long time.

How long? And after this PR applied, how long will it be?

We set tablet_map_shard_size to 16, this function will holds each map lock about more than 1 second.

@chaoyli Hi, do you have time to see this PR again.
This PR has been online in our cluster for two days.


Added array to snackbar qml

🎉 New feature
Summary
if we start getting too many notifications at once to show in the snackbar, this logic will  display them consecutively.
ESC will close the popup, otherwise with the mouse click some of the popup will close inmediately if you are clicking several times to configure something.
Build on top of https://github.com/ignitionrobotics/ign-gui/pull/369
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[x] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@chapulina

Added a Dismiss button
Aligned text
used bold to highlight some relevant information such as: topics or paths

Added a Dismiss button

Thanks! I was thinking of making that optional though. Most implementations have buttons as optional, see

<EMAIL_ADDRESS>https://material.angular.io/components/snack-bar/overview

Usually the button text is configurable and there's a way to register a callback when the button is clicked. I don't think we need all these features just yet, but it would be good to:

structure the API in a way that it can be extended later
the button should be optional for now, so messages that aren't very important have a timeout and can be dismissed by the usual means (like subscription to topics), but messages that are important have a button that must be clicked to dismiss (like a failure to save a file).
make the button flat like in the implementations above (from the spec "Don’t use a filled or elevated button in a snackbar, as it draws too much attention.")

Aligned text

I think it would be better to follow the spec and align the text to the left like it was already.

@ahcorde , I have some suggestions on #375, let me know what you think

Argh, Bionic is broken with
   qrc:/qml/IgnSnackBar.qml:48 Cannot assign to non-existent property "horizontalPadding"

That's something I added on #375. Let me find an alternative

@osrf-jenkins run tests please

Bionic is happy now, the webhooks have been acting up all day. Merging.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-04-13-fortress-edifice/1367/1


Script thinks I'm using Windows?

In the Morning!  Thank you for the script.
Here is what I'm doing.
I am using LXC Debian 11 container on Proxmox.

instructions worked for installing git, git clone and chmod.
Ran the dependency checker and was able to see I had to install a few dependencies. The check dependencies check then passed.
When running 2 - failed with a wine32 error. installed wine.

The problems:
When running the installed it prompts for version. I entered 7.5.
It hangs for ~3-5 minutes, then starts throwing errors.
Errors are below. It looks to think I'm using Windoze?
Waiting for Wine to initialize...0009:err:module:__wine_process_init L"C:\windows\system32\dude-install-7.5.exe" not found
Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)
Failed creating new xdo instance
Waiting for Wine to initialize...Error: Can't open display: (null)

Hi, this integration does not support containers.
The error indicates that it does not detect a screen or display, for this reason it does not open the installer.

Feel a bit silly with the question as it is using WINE ...
I will try this on a Debian 11 VM.
Thanks in advance for the script.


Autocomplete breaks when multiple longer contexts match

I have multiple contexts from AWS clusters, which by default have the entire cluster ARN as their context name (e.g. arn:aws:eks:us-east-2:112233445566:cluster/my-cluster-in-aws).
If I try to autocomplete with only one matching context, then everything works fine, however, if there are multiple matching contexts (e.g. hitting TAB after typing arn:aws), it will autocomplete until the first diverging character and no list of options will come up on a second press. Typing any further characters that match one of the options does not allow autocompleting the context name either.
Example:
Let us assume there is two contexts and we try to autocomplete.

$ kubectx
arn:aws:eks:us-east-2:112233445566:cluster/my-cluster-in-aws
arn:aws:eks:us-west-2:112233445566:cluster/my-cluster-in-aws-two

$ kubectx arn

Now I hit TAB to autocomplete.
$ kubectx arn:aws:eks:us-

Another TAB hit does nothing at this point. The two options are not shown.
I type an 'e' to only match the first context.
$ kubectx arn:aws:eks:us-e

Pressing TAB now does nothing anymore, the first context is not autocompleted.
I am aware I can rename the contexts. If I rename to use shorter names, then everything works fine. I have also tested the presence of the slash in the context name. Removing that has no effect. If there is only a single match of such a length, the autocomplete works without issues.

are you using bash or zsh
are you using the Go implementation or the bash script implementation of kubectx? (vi $(which kubectx) can help find).


StackOverflow badge shows 'invalid parameters' instead of reputation count

Are you experiencing an issue with...
shields.io
🐞 Description
StackOverflow badge shows 'invalid parameters' instead of reputation count

🔗 Link to the badge
No response
💡 Possible Solution
No response

https://github.com/badges/shields/pull/8539

Yeah I'm going to close this one as a dupe of https://github.com/badges/shields/issues/5415


Hardhat | Cannot read property 'deployProxy' of undefined

I am using the latest @openzeppelin/hardhat-upgrades and running the sample script for Hardhat Upgrades, but I am getting this error:
UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'deployProxy' of undefined
This is the packaje.json:
{
  "name": "hardhat-project",
  "devDependencies": {
    "@nomiclabs/hardhat-ethers": "^2.0.0",
    "@nomiclabs/hardhat-waffle": "^2.0.0",
    "chai": "^4.2.0",
    "ethereum-waffle": "^3.2.0",
    "ethers": "^5.0.19",
    "hardhat": "^2.0.3"
  },
  "dependencies": {
    "@chainlink/contracts": "0.0.11",
    "@openzeppelin/contracts-ethereum-package": "^3.0.0",
    "@openzeppelin/hardhat-upgrades": "^1.3.0"
  }
}

Please share your hardhat.config.js.
Do you have require("@openzeppelin/hardhat-upgrades") in it?

Yes, @frangio , that was a silly issue, can you please add that in the readme.

Yes, good idea. https://github.com/OpenZeppelin/openzeppelin-upgrades/commit/c489338b9d1446fea93ee1dd3bd0cb0af212f8b2


WIP: ruby 3.1

docker-compose up --force-recreate --build rspec-3.1 gives:
rspec-3.1_1  |   57) K8s::Transport#self.config for a typical kubeadm admin.conf overriding other options uses the overriden server
rspec-3.1_1  |       Failure/Error: new(YAML.safe_load(File.read(File.expand_path(path)), [Time, DateTime, Date], [], true))
rspec-3.1_1  |
rspec-3.1_1  |       ArgumentError:
rspec-3.1_1  |         wrong number of arguments (given 4, expected 1)
rspec-3.1_1  |       # ./lib/k8s/config.rb:114:in `load_file'
rspec-3.1_1  |       # ./spec/k8s/transport_spec.rb:83:in `block (5 levels) in <top (required)>'
rspec-3.1_1  |       # ./spec/k8s/transport_spec.rb:89:in `block (5 levels) in <top (required)>'

etc

The necessary changes for YAML.safe_load and YAML.parse_stream signatures look straightforward enough. If yaml-safe_load_stream should be separately maintained and updated it would be nice to port it from kontena org as well and publish under existing gem name, though

@dwillett the original guys @kke etc already have done it, please see https://rubygems.org/gems/yaml-safe_load_stream2
there are some legal (?) issues why kontena repos can not be touched.
so can you pleawe check if that works and lets get this done?

Ah I see. I did update to that new gem as part of the 3.0 compatibility already. It would need the update for 3.1 still, though. I can submit a PR to the forked repo (this one I assume) if someone is available to merge and release a new version

yes, I merge and release once it's there

Hi Team,
what's the blocker on having the gem support for 3.1 ? There's something I could do to help this merge?

the above dependecy wont get merged so somebody needs to publish that gem with another name. everything else is done.

To clarify (for my own later understanding), k8s-ruby has a dep on yaml-safe_load_stream2 , which has a broken signature causing a failing test; a PR to fix it exists, (https://github.com/tsutsu/yaml-safe_load_stream2/pull/1#issuecomment-1427905288) but will never be merged (not sure why), so @matti says the way forward is to fork yaml-safe_load_stream2 (again) and publish it under a gem name, and update the k8s-ruby dep accordingly.

fork or just "inline" the gem

We believe this was covered by #41 and #42.

very good


cover recursive divide. Really needed?

Nowhere is recursive divide invoked. It does not look a sophisticated piece of work that is hard to generate only when needed.  Do we need it at all?

it seems like its being used by
class DivideAllFieldsBy(StreamInstanceOperator):
    divisor: float = 1.0
    strict: bool = False
    recursive: bool = True

    def process(
        self, instance: Dict[str, Any], stream_name: Optional[str] = None
    ) -> Dict[str, Any]:
        return recursive_divide(instance, self.divisor, strict=self.strict)

which has many issues, like it does not use the same schema as other field to field classes. and can be applied to containers and not only integers and floats. this can create ambiguity and lack of control.

Thanks, @elronbandel , I missed this use..

As far as I know DivideAllFieldsBy is not used.
As you mentioned, it does not follow the field operator.
In my view this code should be deleted.
The other alternative is to rewrite it as a FieldOperator, but we can delay it until is actually needed.

Then it should be in a different PR.
CC: @dafnapension


Corrected overlapping of back-button in some activities

Fixes: #1055
By correcting the z-index of the back-button is fixed. As shown:

It fix the issue but the back button no longer works in Tangram.

It fix the issue but the back button no longer works in Tangram.

Thanks @llaske, Yes you are right. Only half of it is working because the z-index of tangram-name-content i.e. the title is higher than the back-button.
I fixed it now. Could you please review it once again!

Nice. Thanks.


[Improve][Connector-V2] Enhance  FakeSource connector

Search before asking

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

Description
Thoughts on enhance FakeSource:

[ ] SeaTunnelSchema support parse row type from config file #2765
[ ] Support define row type
[ ] Supports define the total number of rows of data
[ ] Supports direct definition of data values(row)
[ ] Support value's range to avoid exceed in some connectors.
[ ] Support random generate null value for validating connector's stability.

Anyone willing to help?
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Now SeaTunnelSchema can not parse row type from config file, so we should upgrade first. I can do this work.

Please assign it to me, I am interested in FakeDatasource and console

I think fake source need support define the key fields and generate unique value for key fields. It's very useful to test exactly-once sink which implement exactly-once by Idempotent write data.

我认为假源需要支持定义关键字段并为关键字段生成唯一值。测试通过幂等写入数据exactly-once实现的接收器非常有用。exactly-once

good idea, I will do it

Support value's range to avoid exceed  in some connectors.
Support random generate null value for validating connector's stability.

I will do it but lately

I suggest FakeSource support parallel, I want to use FakeSource and LocalFileSink test the data consistency while some engine node shutdown(For SeaTunnel Engine integration testing). The FakeSource task will run in more than one cluster node only if it support parallel.


Allow shout-irc to be proxied behind a /path/ url

contains three small changes to use relative urls instead of absolute rooted at /

favicons in index.html
badge and pop sound in shout.js
the url for the socket.io endpoint

LGTM :+1:

Might be petty, but you may want to change your commit message from shout-irc to lounge to avoid any confusion.

:+1:

:+1:

Merged in #20 with reworded commit message (but you still are credited in the contributors and you'll be part of the changelog :p).
Thanks a lot @gdamjan, great feature!!


Upgrade to Pants 2.8.0.dev4.

It's not ideal to use a dev version in an example repo, but
this allows us to streamline this repo by removing the
global venv and instead demonstrate running manage.py commands
via ./pants run - even runserver, with pants triggering
restarts!

cc @g-cassie

The other small win I had in 2.8 was to make a pex_binary for django-admin in the root BUILD file like this:

Nice. May merge that separately though.

Should this also use an alias so it's less verbose? I imagine almost every org would want to set that up.

Sorry, what is the "this" here?

This PR, an alias for ./pants run path/to:manage_py_bin -- so that all the user rules is ./pants manage-py --help

There will be multiple manage.py in a repo, so that kind of alias doesn't really make sense.


Add DRY principle

The Remove duplicate code section is a DRY principle.

Thanks for the PR. Could you use more attomical commits with smaller steps, so I can see what exactly has changed?
Now it looks like 100 lines was removed and 100 was added.

I just change the title of the section.
Move the section down the document.
And add link to Wikipedia.

I add fix #45 in my PR, so that there is no conflict.

@peter-gribanov If that's the case, let's merge it then. Thank you for info and the code update


Remove energy efficient agent and related tests, documentation

Signed-off-by: Alejandro Vilches<EMAIL_ADDRESS>
Relates to #2301
Fixes #2310

NOTE: created as draft for now because I'm still looking into the appropriate changes for some the integration tests and tutorials.


[WIP] Add chainerx.clip

Add chainer.clip
TODO: Add tests

Seems to be duplicate...:
https://github.com/chainer/chainer/pull/7285
Scalar handling and default argument of min/max is kind a difficult...

Oh sorry. @take-cheeze could this be closed then?

@aksub99 OK. Feel free to comment to https://github.com/chainer/chainer/pull/7285
Thanks for the contribution!


Adds support for control streams, bidir/unidir streams, and changing of streams based on group, object, etc.

Adds control stream and supports unidirectional, bidirectional, and creating new streams based on track, group, object, etc.

Resolves #98
Resolves #104
Resolves #106
Resolves #107
Resolves #108
Resolves #115

Resolves #94


Fix 500 issue with 'GET /tenants' API call

When a server has a wrong formatted tag, current implemantation
would return 500 error for GET /tenants call.
In that case, we should not fail and return the list of valid
tenants.
For the wrong formatted tags, this pr adds the log so that one can
look up the logs when cleaning up the tags.

Codecov Report

Merging #4061 into master will increase coverage by 9.69%.
The diff coverage is 46.15%.

@@             Coverage Diff              @@
##             master    #4061      +/-   ##
============================================
+ Coverage     56.96%   66.65%   +9.69%
  Complexity       20       20
============================================
  Files          1033     1033
  Lines         51383    51393      +10
  Branches       7195     7195
============================================
+ Hits          29270    34257    +4987
+ Misses        19948    14773    -5175
- Partials       2165     2363     +198

Impacted Files
Coverage Δ
Complexity Δ

...a/org/apache/pinot/common/config/TagNameUtils.java
69.23% <0%> (+5.12%)
0 <0> (ø)
:arrow_down:

...ntroller/helix/core/PinotHelixResourceManager.java
60.85% <50%> (+6.77%)
0 <0> (ø)
:arrow_down:

...pache/pinot/core/util/SortedRangeIntersection.java
83.82% <0%> (-7.36%)
0% <0%> (ø)

...regation/function/customobject/QuantileDigest.java
55.53% <0%> (-2.22%)
0% <0%> (ø)

...va/org/apache/pinot/common/data/TimeFieldSpec.java
92.59% <0%> (-1.24%)
0% <0%> (ø)

...ator/transform/function/BaseTransformFunction.java
29.95% <0%> (+0.42%)
0% <0%> (ø)
:arrow_down:

...g/apache/pinot/common/utils/helix/HelixHelper.java
56.25% <0%> (+0.56%)
0% <0%> (ø)
:arrow_down:

...a/org/apache/pinot/common/utils/ServiceStatus.java
77.21% <0%> (+0.63%)
0% <0%> (ø)
:arrow_down:

...ment/creator/impl/SegmentColumnarIndexCreator.java
87.45% <0%> (+0.76%)
0% <0%> (ø)
:arrow_down:

...r/transform/function/ValueInTransformFunction.java
39.2% <0%> (+0.8%)
0% <0%> (ø)
:arrow_down:

... and 329 more

Continue to review full report at Codecov.

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

Do we already have checks in the path where the config is being set to not allow invalid configs? (I understand someone can directly update znode with an invalid value and we will still need checks on the get path)

Would be also good to add a test.

@sunithabeeram There's a POST /tenants API that tries to assign tags for untagged servers/brokers using a configured number of instances (In this case, it's guaranteed to have a valid tag). However, we have been heavily depending on helix API or a script for adding/removing tags directly because we usually pick specific instances and associate them to a tenant.
One improvement would be adding an api to associate a specific server/broker to a tenant and we perform validation for that.
By the way, I added the unit test for this.


Add missing cluster codes to bitmaps in ethernet-network-diagnostics-cluster.xml and wifi-network-diagnostics-cluster.xml

Problem
EthernetNetworkDiagnosticsFeature and WiFiNetworkDiagnosticsFeature are missing cluster codes. This PR just adds those with ./scripts/tools/zap_regen_all.py.

Looks like a duplicate of https://github.com/project-chip/connectedhomeip/pull/24673 ?


[ F# 4.2] better warnings on mutable struct

As described in https://github.com/Microsoft/visualfsharp/issues/3923 in certain cases, the F# compiler assumes that .NET struct types are shallow-immutable when the user forgets to declare a struct value mutable using let mutable x = ....

This assumption is made because there is no way to analyze whether a .NET struct type is mutable, and
performing a defensive copy in situations like ``

This applies to method calls like x.SetFoo(3) but not to property setters x.Foo <- 3. In the latter case the operation is assumed mutating and an error is given at compile time.

This can lead to differences in optimized and non-optimized code when let x = .... is used for a .NET struct value and a subsequent mutating operation is performed.

The following adjustment is proposed for F# 4.2:

Give an optional warning (off by default) whenever a .NET struct type has been assumed to be immutable. However this is very noisy, as it triggers in very basic all property getter accessors are assumes to be immutable

Add a heuristic to give an improved diagnostic in some cases. Specifically if a method call has void/unit return type and starts with name "set" then give a warning (on by default) when used on a .NET struct value that has not been declared mutable.

Part (1) is fine, but part (2) of the change may be contentious: it is only a heuristic, but any name-based heuristic is unfortunate.
I'm slating this for F# 4.2 rather than pushing in the rule as a bug fix.

Thanks Don - that’s a huge help when the optimiser is doing seemingly strange things that make complete sense when explained!

@KevinRansom @brettfo  The build-from-source is failing when an error message is added to FSComp.txt:
error FS0039: The field, constructor or member 'tastLikelyMutationOfConstant' is not defined. ...
error FS0039: The field, constructor or member 'tcStructTypeAssumedImmutable ...

Does the build-from-source rely on a checked-in version of a generated file?

@KevinRansom @brettfo The build-from-source is failing when an error message is added to FSComp.txt

What's the process for automatically updating this file please?  I tried build buildfromsource but it didn't get updated.

CI seems broken - this is shown as "all checks passed" but no tests have been run

CI is down for all dotnet repos (us included) until at least 12/1, 3PM PST (~6 hours after I posted this.)

Brett says:

All affected generated files under the src\buildfromsource directory need to be updated.
For example, if you change a value in src\fsharp\FSComp.txt, you’ll need to make the corresponding change in src\buildfromsource\FSharp.Compiler.Private\FSComp.fs and FSComp.resx.
The reason for this is that the version of the compiler that’s used to build the src\buildfromsource directory doesn’t yet have my changes to auto-generate the *.fs and *.resx files.  I’ll double-check on the versions, but I think once the source-build work upgrades to dotnet CLI 2.2 (maybe 2.1?) then the appropriate targets will be present so that we can delete the generated files.

It looks like we can just
copy /y src\fsharp\FSharp.Compiler.Private\obj\coreclr\* src\buildfromsource\FSharp.Compiler.Private

etc.

Retarget at in master https://github.com/Microsoft/visualfsharp/pull/4576

Would it be possible to have an attribute to control the defensive copy behaviour with an attribute? For example: [<AssumeImmutable>] and [<AssumeReadOnly>] (both because they have a different semantics). Heuristics is still remains a heuristics.


add new GC benchmark

tests producer-consumer pattern

both ends allocate lots of short lived garbage

trigger many (~200) collections but heap remains very small (5MB)

LGTM and uses some functionality not yet covered by the other tests (message passing).


Class 'SSilence\ImapClient\SubtypeBody' not found in /home1/u8q7m3r8/public_html/require/phpimap/IncomingMessage.php:361

Hi!
Every time I try to open a message, I got this message.
I put every file from the ImapClient to my folder, but it said, SubtypeBody not found. What can I do?
Fatal error:  Uncaught Error: Class 'SSilence\ImapClient\SubtypeBody' not found in /home1/u8q7m3r8/public_html/require/phpimap/IncomingMessage.php:361 Stack trace: #0 /home1/u8q7m3r8/public_html/require/phpimap/IncomingMessage.php(178): SSilence\ImapClient\IncomingMessage->getBody() #1 /home1/u8q7m3r8/public_html/require/phpimap/IncomingMessage.php(159): SSilence\ImapClient\IncomingMessage->init() #2 /home1/u8q7m3r8/public_html/require/phpimap/ImapClient.php(640): SSilence\ImapClient\IncomingMessage->__construct(Resource id #26, 12, 'decode') #3 /home1/u8q7m3r8/public_html/cm/inc/foglalasok/foglalas/loadUzenetvaltas.php(33): SSilence\ImapClient\ImapClient->getMessage(12) #4 {main}   thrown in /home1/u8q7m3r8/public_html/require/phpimap/IncomingMessage.php on line 361

Are you able to install via composer we highly recomend this

Thanks for your very fast reply! Okey, then i try it!

Let me know! No problem

I try it with composer, but it is still hard for me! :)
But I found a solution: If I require all files at first, (not only that some what I find in this doc: http://ssilence.github.io/php-imap-client/gettingstarted.html) it's working.
  require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/ImapClientException.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/ImapConnect.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/ImapClient.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/Section.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/Helper.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/IncomingMessage.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/IncomingMessageAttachment.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/TypeAttachments.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/TypeBody.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/SubtypeBody.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/OutgoingMessage.php";   require_once $_SERVER['DOCUMENT_ROOT']."/require/phpimap/AdapterForOutgoingMessage.php";

When using composer are you using the auto loader?

Yes!
And sorry for askint it there, but can I search with this library? For e.g. only list emails from a specific email?

Just got a pc so sorry for not replying. The readme contains a link to the docs :D

I get a Fatal error: Uncaught Error: Class 'Imap' not found , even with the suggested workaround


Add script for dependency updates

add a script to semi-automatically update dependencies:

major updates flagged for user intervention,
minor updates automatically merged onto pyproject.toml,
patch updates automatically merged onto poetry.lock.

e.g.
bash scripts/poetry-update.sh -n

if anybody has thoughts on this, or suggestions, feel free to reach out!
IMHO this strikes a good balance between staying somewhat-up-to-date with our deps, and keeping a widened but reasonable dep list.

I usually create a PR to update the deps, so it should be apparent if stuff does not work -- in any case, the minor version locking is just a convention, the actual installed version is governed by the lockfile, and that does track patch versions, too.
I don't push all the patch versions every time, but your poetry is independent of that, eg you can run poetry update whenever you want.
(worst case is you'll have some git diffs in your local lockfile.)


Selectively build TFLite for iOS failure

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
2.12.0
Custom Code
No
OS Platform and Distribution
MacOS Ventura 13.1
Mobile device
N/A
Python version
3.9.16
Bazel version
5.3
GCC/Compiler version
14.0.0
CUDA/cuDNN version
N/A
GPU model and memory
N/A
Current Behaviour?
This was a subsequence bug from another issue(https://github.com/tensorflow/tensorflow/issues/59853#issuecomment-1481642687) and @yishuangP asked me to file a separate issue for Tensorflow Tools.

I'm trying to build a tf framework from certain tflite model to reduce framework size. However I keep seeing below error. The command I'm using is `bash tensorflow/lite/ios/build_frameworks.sh --input_models=tflite_model_vocab_changes_run_18026681_full_dataset.tflite --target_archs=x86_64,armv7,arm64`

ERROR: /Users/ghe/projects/tensorflow/tensorflow/BUILD:1128:21: declared output 'tensorflow/libtensorflow_framework.2.dylib' was not created by genrule. This is probably because the genrule actually didn't create this output, or because the output was a directory and the genrule was run remotely (note that only the contents of declared file outputs are copied from genrules run remotely)
ERROR: /Users/ghe/projects/tensorflow/tensorflow/BUILD:1128:21: Executing genrule //tensorflow:libtensorflow_framework.2.dylib_sym [for host] failed: not all outputs were created or valid
realpath: illegal option -- -
usage: realpath [-q] [path ...]
Target //tensorflow/lite/ios/tmp:TensorFlowLiteSelectTfOps_framework failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 1046.736s, Critical Path: 138.53s
INFO: 3339 processes: 25 internal, 3314 local.
FAILED: Build did NOT complete successfully

### Standalone code to reproduce the issue

```shell
There is no code involved. To reproduce:

1. git fetch a new project
2. Copy the toy model I provided into its root directory
3. Run `bash tensorflow/lite/ios/build_frameworks.sh --input_models=tflite_model_vocab_changes_run_18026681_full_dataset.tflite --target_archs=x86_64,armv7,arm64` command

It will compile for a couple minutes, and fail finally.

Relevant log output
$ bash tensorflow/lite/ios/build_frameworks.sh --input_models=tflite_model_vocab_changes_run_18026681_full_dataset.tflite --target_archs=x86_64,armv7,arm64
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=314
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  'build' options: --define framework_shared_object=true --define tsl_protobuf_header_only=true --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true --experimental_cc_shared_library --experimental_link_static_libraries_once=false --incompatible_enforce_config_setting_visibility
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.tf_configure.bazelrc:
  'build' options: --action_env PYTHON_BIN_PATH=/Applications/Xcode.app/Contents/Developer/usr/bin/python3 --action_env PYTHON_LIB_PATH= /Library/Python/3.9/site-packages --python_path=/Applications/Xcode.app/Contents/Developer/usr/bin/python3
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  'build' options: --deleted_packages=tensorflow/compiler/mlir/tfrt,tensorflow/compiler/mlir/tfrt/benchmarks,tensorflow/compiler/mlir/tfrt/jit/python_binding,tensorflow/compiler/mlir/tfrt/jit/transforms,tensorflow/compiler/mlir/tfrt/python_tests,tensorflow/compiler/mlir/tfrt/tests,tensorflow/compiler/mlir/tfrt/tests/ir,tensorflow/compiler/mlir/tfrt/tests/analysis,tensorflow/compiler/mlir/tfrt/tests/jit,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_tfrt,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_jitrt,tensorflow/compiler/mlir/tfrt/tests/tf_to_corert,tensorflow/compiler/mlir/tfrt/tests/tf_to_tfrt_data,tensorflow/compiler/mlir/tfrt/tests/saved_model,tensorflow/compiler/mlir/tfrt/transforms/lhlo_gpu_to_tfrt_gpu,tensorflow/core/runtime_fallback,tensorflow/core/runtime_fallback/conversion,tensorflow/core/runtime_fallback/kernel,tensorflow/core/runtime_fallback/opdefs,tensorflow/core/runtime_fallback/runtime,tensorflow/core/runtime_fallback/util,tensorflow/core/tfrt/eager,tensorflow/core/tfrt/eager/backends/cpu,tensorflow/core/tfrt/eager/backends/gpu,tensorflow/core/tfrt/eager/core_runtime,tensorflow/core/tfrt/eager/cpp_tests/core_runtime,tensorflow/core/tfrt/gpu,tensorflow/core/tfrt/run_handler_thread_pool,tensorflow/core/tfrt/runtime,tensorflow/core/tfrt/saved_model,tensorflow/core/tfrt/graph_executor,tensorflow/core/tfrt/saved_model/tests,tensorflow/core/tfrt/tpu,tensorflow/core/tfrt/utils
INFO: Found applicable config definition build:short_logs in file /Users/ghe/projects/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /Users/ghe/projects/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:ios in file /Users/ghe/projects/tensorflow/.bazelrc: --apple_platform_type=ios --apple_bitcode=embedded --copt=-fembed-bitcode --copt=-Wno-c++11-narrowing --noenable_platform_specific_config --copt=-w --cxxopt=-std=c++17 --host_cxxopt=-std=c++17 --define=with_xla_support=false
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/llvm/llvm-project/archive/55dd04f6bcf797b4ff20e74158377bcc912b9870.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
INFO: Analyzed target //tensorflow/lite/ios/tmp:TensorFlowLiteC_framework (144 packages loaded, 7602 targets configured).
INFO: Found 1 target...
Target //tensorflow/lite/ios/tmp:TensorFlowLiteC_framework up-to-date:
  bazel-out/applebin_ios-ios_x86_64-opt-ST-74bbdaf492e7/bin/tensorflow/lite/ios/tmp/TensorFlowLiteC_framework.zip
INFO: Elapsed time: 38.614s, Critical Path: 5.29s
INFO: 13 processes: 2 internal, 11 local.
INFO: Build completed successfully, 13 total actions
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=314
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  'build' options: --define framework_shared_object=true --define tsl_protobuf_header_only=true --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true --experimental_cc_shared_library --experimental_link_static_libraries_once=false --incompatible_enforce_config_setting_visibility
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.tf_configure.bazelrc:
  'build' options: --action_env PYTHON_BIN_PATH=/Applications/Xcode.app/Contents/Developer/usr/bin/python3 --action_env PYTHON_LIB_PATH= /Library/Python/3.9/site-packages --python_path=/Applications/Xcode.app/Contents/Developer/usr/bin/python3
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  'build' options: --deleted_packages=tensorflow/compiler/mlir/tfrt,tensorflow/compiler/mlir/tfrt/benchmarks,tensorflow/compiler/mlir/tfrt/jit/python_binding,tensorflow/compiler/mlir/tfrt/jit/transforms,tensorflow/compiler/mlir/tfrt/python_tests,tensorflow/compiler/mlir/tfrt/tests,tensorflow/compiler/mlir/tfrt/tests/ir,tensorflow/compiler/mlir/tfrt/tests/analysis,tensorflow/compiler/mlir/tfrt/tests/jit,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_tfrt,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_jitrt,tensorflow/compiler/mlir/tfrt/tests/tf_to_corert,tensorflow/compiler/mlir/tfrt/tests/tf_to_tfrt_data,tensorflow/compiler/mlir/tfrt/tests/saved_model,tensorflow/compiler/mlir/tfrt/transforms/lhlo_gpu_to_tfrt_gpu,tensorflow/core/runtime_fallback,tensorflow/core/runtime_fallback/conversion,tensorflow/core/runtime_fallback/kernel,tensorflow/core/runtime_fallback/opdefs,tensorflow/core/runtime_fallback/runtime,tensorflow/core/runtime_fallback/util,tensorflow/core/tfrt/eager,tensorflow/core/tfrt/eager/backends/cpu,tensorflow/core/tfrt/eager/backends/gpu,tensorflow/core/tfrt/eager/core_runtime,tensorflow/core/tfrt/eager/cpp_tests/core_runtime,tensorflow/core/tfrt/gpu,tensorflow/core/tfrt/run_handler_thread_pool,tensorflow/core/tfrt/runtime,tensorflow/core/tfrt/saved_model,tensorflow/core/tfrt/graph_executor,tensorflow/core/tfrt/saved_model/tests,tensorflow/core/tfrt/tpu,tensorflow/core/tfrt/utils
INFO: Found applicable config definition build:short_logs in file /Users/ghe/projects/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /Users/ghe/projects/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:monolithic in file /Users/ghe/projects/tensorflow/.bazelrc: --define framework_shared_object=false --define tsl_protobuf_header_only=false --experimental_link_static_libraries_once=false
INFO: Found applicable config definition build:macos in file /Users/ghe/projects/tensorflow/.bazelrc: --apple_platform_type=macos --copt=-DGRPC_BAZEL_BUILD --copt=-w --define=PREFIX=/usr --define=LIBDIR=$(PREFIX)/lib --define=INCLUDEDIR=$(PREFIX)/include --define=PROTOBUF_INCLUDE_PATH=$(PREFIX)/include --cxxopt=-std=c++17 --host_cxxopt=-std=c++17
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/llvm/llvm-project/archive/55dd04f6bcf797b4ff20e74158377bcc912b9870.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
INFO: Build options --apple_bitcode, --apple_platform_type, --copt, and 1 more have changed, discarding analysis cache.
INFO: Analyzed target //tensorflow/lite/tools:list_flex_ops_no_kernel_main (7 packages loaded, 1231 targets configured).
INFO: Found 1 target...
Target //tensorflow/lite/tools:list_flex_ops_no_kernel_main up-to-date:
  bazel-bin/tensorflow/lite/tools/list_flex_ops_no_kernel_main
INFO: Elapsed time: 0.292s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
~/projects/tensorflow/tensorflow/lite/ios/tmp ~/projects/tensorflow
~/projects/tensorflow
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=314
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  'build' options: --define framework_shared_object=true --define tsl_protobuf_header_only=true --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true --experimental_cc_shared_library --experimental_link_static_libraries_once=false --incompatible_enforce_config_setting_visibility
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.tf_configure.bazelrc:
  'build' options: --action_env PYTHON_BIN_PATH=/Applications/Xcode.app/Contents/Developer/usr/bin/python3 --action_env PYTHON_LIB_PATH= /Library/Python/3.9/site-packages --python_path=/Applications/Xcode.app/Contents/Developer/usr/bin/python3
INFO: Reading rc options for 'build' from /Users/ghe/projects/tensorflow/.bazelrc:
  'build' options: --deleted_packages=tensorflow/compiler/mlir/tfrt,tensorflow/compiler/mlir/tfrt/benchmarks,tensorflow/compiler/mlir/tfrt/jit/python_binding,tensorflow/compiler/mlir/tfrt/jit/transforms,tensorflow/compiler/mlir/tfrt/python_tests,tensorflow/compiler/mlir/tfrt/tests,tensorflow/compiler/mlir/tfrt/tests/ir,tensorflow/compiler/mlir/tfrt/tests/analysis,tensorflow/compiler/mlir/tfrt/tests/jit,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_tfrt,tensorflow/compiler/mlir/tfrt/tests/lhlo_to_jitrt,tensorflow/compiler/mlir/tfrt/tests/tf_to_corert,tensorflow/compiler/mlir/tfrt/tests/tf_to_tfrt_data,tensorflow/compiler/mlir/tfrt/tests/saved_model,tensorflow/compiler/mlir/tfrt/transforms/lhlo_gpu_to_tfrt_gpu,tensorflow/core/runtime_fallback,tensorflow/core/runtime_fallback/conversion,tensorflow/core/runtime_fallback/kernel,tensorflow/core/runtime_fallback/opdefs,tensorflow/core/runtime_fallback/runtime,tensorflow/core/runtime_fallback/util,tensorflow/core/tfrt/eager,tensorflow/core/tfrt/eager/backends/cpu,tensorflow/core/tfrt/eager/backends/gpu,tensorflow/core/tfrt/eager/core_runtime,tensorflow/core/tfrt/eager/cpp_tests/core_runtime,tensorflow/core/tfrt/gpu,tensorflow/core/tfrt/run_handler_thread_pool,tensorflow/core/tfrt/runtime,tensorflow/core/tfrt/saved_model,tensorflow/core/tfrt/graph_executor,tensorflow/core/tfrt/saved_model/tests,tensorflow/core/tfrt/tpu,tensorflow/core/tfrt/utils
INFO: Found applicable config definition build:short_logs in file /Users/ghe/projects/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /Users/ghe/projects/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition build:ios in file /Users/ghe/projects/tensorflow/.bazelrc: --apple_platform_type=ios --apple_bitcode=embedded --copt=-fembed-bitcode --copt=-Wno-c++11-narrowing --noenable_platform_specific_config --copt=-w --cxxopt=-std=c++17 --host_cxxopt=-std=c++17 --define=with_xla_support=false
WARNING: Download from https://storage.googleapis.com/mirror.tensorflow.org/github.com/llvm/llvm-project/archive/55dd04f6bcf797b4ff20e74158377bcc912b9870.tar.gz failed: class java.io.FileNotFoundException GET returned 404 Not Found
INFO: Build options --apple_bitcode, --apple_platform_type, --copt, and 2 more have changed, discarding analysis cache.
INFO: Analyzed target //tensorflow/lite/ios/tmp:TensorFlowLiteSelectTfOps_framework (435 packages loaded, 36544 targets configured).
INFO: Found 1 target...
ERROR: /Users/ghe/projects/tensorflow/tensorflow/BUILD:1128:21: declared output 'tensorflow/libtensorflow_framework.2.dylib' was not created by genrule. This is probably because the genrule actually didn't create this output, or because the output was a directory and the genrule was run remotely (note that only the contents of declared file outputs are copied from genrules run remotely)
ERROR: /Users/ghe/projects/tensorflow/tensorflow/BUILD:1128:21: Executing genrule //tensorflow:libtensorflow_framework.2.dylib_sym [for host] failed: not all outputs were created or valid
realpath: illegal option -- -
usage: realpath [-q] [path ...]
Target //tensorflow/lite/ios/tmp:TensorFlowLiteSelectTfOps_framework failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 1046.736s, Critical Path: 138.53s
INFO: 3339 processes: 25 internal, 3314 local.
FAILED: Build did NOT complete successfully

This issue originally reported by @heguanyu has been moved to this dedicated repository for LiteRT to enhance issue tracking and prioritization. To ensure continuity, we have created this new issue on your behalf.
We appreciate your understanding and look forward to your continued involvement.

Original Issue: https://github.com/tensorflow/tensorflow/issues/60088


Refactor workflow to deploy ChannelScriptPackage

"dev ip <package_file_path | dir_path>", install package from package_file_path, or compile and install the package from dir_path, according to the parameter is a csp file or a dir.
add sgcompiler method compile_package_with_output(), ChannelScriptPackage will be dumped into output file.

çargo fmt check is failed. @guangyuz


Update slack-status extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Addresses this bug: #12995


Update the minimum docker image version for sparse bert

Description
None of these docker images works for sparse.

You can choose the tensorflow version and python version you want. Here, we list some possible images:

- `nvcr.io/nvidia/tensorflow:19.07-py2` contains the TensorFlow 1.14 and python 2.7.
- `nvcr.io/nvidia/tensorflow:20.12-tf1-py3` contains the TensorFlow 1.15 and python 3.8.
- `nvcr.io/nvidia/pytorch:20.03-py3` contains the PyTorch 1.5.0 and python 3.6
- `nvcr.io/nvidia/pytorch:20.07-py3` contains the PyTorch 1.6.0 and python 3.6
- `nvcr.io/nvidia/pytorch:20.12-py3` contains the PyTorch 1.8.0 and python 3.8

cuSparseLt requires CUDA11.2 above. The minimum docker image version should be 21.02

Reproduced Steps
check the requirements

Hi, Iuoyu. Some older docker images only support basic features like FP16.
If you need to use advanced feature like sparse, newer docker image is required. You can refer support-matrix to check the CUDA version of different docker images.

The docker image of document is updated to 22.04.


Improve Test fixtures

Proposed change
We should adopt TestContainers to improve the test reliability in particular to spin up and down the servers.
Use case
Test Containers make it easy for us to manage instances of Nats to be used on unit tests and avoid manual management of processes. All that the test agent (CI machine, developer machine, etc) needs to have is the ability to run containers (i.e. Docker).
Contribution
I'll look on what it takes and write details later on here.

we also test against different server versions one being the main branch to help catch potential issues on server development. main branch builds won't be on docker hub I don't think.

That shouldn't be a problem. We could build, as part of the CI, a local container image using main. Then run test containers from that local image. Not a problem at all. We can smuggle this into the actions. The other versions we pull from Docker hub as usual.

personal opinion: I've only seen containers used on the aspire project, I have almost zero experience, but if running the server can be 'abstracted' in some fashion.. maybe? maybe some kind of a switch/setting?

This will be trivial. All you would need is to have docker desktop installed on your machine (Mac/Windows/Linux) and it will be fine. The CI agent on GitHub Actions is actually a container which also support "Docker in Docker" meaning that we wouldn't have any issues running there as well.

we can maybe start by writing a few redundant tests using testcontainers? like a POC?

Yes, we can do it all separated in a different fixture and pick up the tests that are more likely to fail than others. Once it is safe, we just swap the fixtures on the rest.

that sounds quite good @galvesribeiro! My last whinge if I may, sorry 😬 I think it's also good to be able to run the server on Windows and test againsts that.

We can still use containers. Windows Containers is a thing, so we can still use the same API as long as we run the Windows tests on Windows agents. I never had to run Windows containers on GitHub Actions even tho I know there are Windows agents. I guess we will find out soon 💃🏻


Cirth-RobertC

I made a cirth runes thing, since I was on a tolkien kick when I got the email for this. I've been coding since about 5th grade, mostly in c++, python, and javascript, with some more obscure (and annoying) langs sprinkled in. I will add an update to this that makes it use proper kerning with bt.bounds and things, but I figured I would submit what I had and work on the rest later

Very fun project! Thanks for sharing.
Congrats! You've earned a Blot you can order it here if you are a teen.


More pylint 2 fixes

Description:
More pylint 2 fixes.
Related issue (if applicable): See #15487 for some background info.
Checklist:

[ ] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Nice!


Branches Aren't Just For Birds problem

when i "git push origin add-CodeStrider35" and i verify my login and password, it says: remote: permission to jlord/patchwork.git denied to CodeStrider35. Fatal: unable to access 'https://github.com/jlord/patchwork.git/': The requested URL returned error: 403
my file "add-CodeStrider35.txt" is saved in the contributors folder.
I tried to go back and follow the steps again on http://jlord.us/git-it/challenges/branches_arent_just_for_birds.html and now its saying my branch's directory is clean...nothing to add or commit but my file is in the contributors folder...I'm so lost.

Fatal: unable to access 'https://github.com/jlord/patchwork.git/': The requested URL returned error: 403

This would come from you trying to push to my repository rather than your fork of it. Check your remotes:
$ git remotes -v

You should have one that points to your fork, which should be: https://github.com/CodeStrider35/patchwork.git. In Git-it, the remote named origin should point to your branch while the one named upstream should point to mine, and is just for pulling (not pushing, since you don't have access to change the code in another person's repository unless you're added as a collaborator).
You can rename origin to upstream by running:
$ git remote rename origin upstream

Then you can add a remote to point to your fork:
$ git remote add origin https://github.com/CodeStrider35/patchwork.git

You'll be able to push to origin now :+1:  Hope that helps!

I'll try this when I get in - just wanted to say thank you on your answer. I appreciate how  you laid out the issue and provided the solution!

@jlord - still having problems - I make the file in patchwork/contributors as add-CodeStrider35. the file has my name as specified 'CodeStrider35'. The check in process is what is not working for me. next I put git status and it says 'On branch add-CodeStrider35, nothing to commit, working directory clean. Then I put git add add-CodeStrider35.txt and it says 'fatal: pathspec 'add-CodeStrider35.txt did not match any files. Then I put git add contributors/add-CodeStrider35.txt. nothing happens, it just goes to the next C:\patchwork>
Thinking that I finished that step, I go to the next and type: git commit -m "my first branch" and it says, On branch CodeStrider35, nothing to commit, working directory clean. I believe something went wrong here but as I'm typing directions verbatim, I go to the next and type git push origin CodeStrider35 and it gives me the same error message I typed in my first.

I tried your directions to and it just says fatal: remote origin already exists...

@jlord - problem fixed!!! TY

Oh, good! How did you fix it? It may help others down the line who have the same issue.

programmer friend helped me - I'll ask for the steps again and I'll post here asap.

I ran into this same problem. Your answer was great in that it helped me to understand what the issue was so thanks for that, but this http://stackoverflow.com/questions/3011402/leaving-github-how-to-change-the-origin-of-a-git-repo helped me to figure out how to fix it. I found the second answer to be very simple to implement.

this helped me too.


Q: Is possible add cornerRadius to main view controller?

Good morning!
First, thanks for this awesome lib!
Someone can help me?
Is possible add cornerRadius to main view controller when I click or swipe?
Thanks

any solution about this is issue ?

I try two ways:
The first is to modify the transition a DrawerTransition like DrawerFloatTransition for example and on the method endTransition, you can apply the corner radius on the content:
open override func endTransition(content: DrawerContent, side: DrawerSide) {
        super.endTransition(content: content, side: side)

        content.contentView.layer.cornerRadius = 33
}

The second way, without change the code, is to add a view with a container. On the view, you apply the corner radius.
It's the first time i use this library and i need to make it with a shadow:


Update .gitignore, add .nvmrc:

ignore all potential .env files
ignore .yarn (if project is installed locally and yarn is configured accordingly)
add .nvmrc: only relevant for NVM users

Looks, good! I will revert the nvmrc locally since you can use either node 16/18 if you have them locally and it will be fine. :)


如何安装，有没有中文文档

Summary:
Task covering patch release work.
Dev Complete: RELEASE_DATE (Typically ~1 week prior to upstream release date)
List of required releases:
To release as soon as able for QA:

VERSION

To release once have approval from QA:

VERSION (Never release on a Friday unless specified otherwise)

Prep work:

[ ] PM: Dev and QA team to be notified of the incoming releases - add event to team calendar
[ ] PM: Dev and QA team to be notified of the date we will mark the latest release as stable - add event to team calendar [ONLY APPLICABLE FOR LATEST MINOR RELEASE]
[ ] QA: Review changes and understand testing efforts
[ ] Release Captain: Prepare release notes in our private release-notes repo (submit PR for changes taking care to carefully check links and the components, once merged, create the release in GitHub and mark as a draft and check the pre-release box, fill in title, set target release branch, leave tag version blank for now until we are ready to release)
[ ] QA: Validate and close out all issues in the release milestone.

Vendor and release work:

[ ] Release Captain: Vendor in the new patch version and release rancher/kubernetes
[ ] Release Captain: Tag and release any necessary RCs for QA to test K3s and KDM on the Rancher side
[ ] Release Captain: Tag and release when have QA approval

Post-Release work:

[ ] Release Captain: Once release is fully complete (CI is all green and all release artifacts exist), edit the release, uncheck "Pre-release", and save.
[ ] Release Captain: Prepare PRs as needed to update KDM in the appropriate dev branches.
[ ] PM: Close the milestone in GitHub.

请阅读此处的文档：
https://docs.k3s.io/zh/


Allow serialization of selected trajectory IDs.

The optional parameter trajectory_ids allows to serialize only
a subset of the data. This is for example useful for re-mapping
on top of an existing map and serializing only the new map.

Follow-up for cartographer_ros: https://github.com/googlecartographer/cartographer_ros/pull/749

@cschuet This also affects the grpc interface, I just added the optional param there without functionality yet. What's your opinion?

CLA bump

As discussed in the open house session, this is more a simple prototype to illustrate the behavior. Some interface details like the function params need to be discussed.
If a more thorough discussion regarding this feature is needed (e.g. more selection features than just trajectory IDs), I'm also fine with opening an RFC. @gaschler

Closing in favor of WIP on new trimmers.


Allow default port to be specified via command line flag

It may be useful to allow the default port to be specified from a command line flag.

Whoops, same as #16.


Media manager section of admin panel shows errors in browser console but everything seems work well

Version information
-Laravel: v5.7.17
-Voyager:v1.1
-PHP: 7.2
-Database: MySQL v5.7
Description
Fresh instal of laravel and voyager. When I go to:
http://voyager/admin/media
In the console of browser an error accurs:
GET http://voyager/storage/users/ 403 (Forbidden)
also when as I navigate through folder here I get one more:
GET http://voyager/storage/users/December2018/ 403 (Forbidden)
The picture wich I chosen as avatar is located here (Media Library > users > December2018) and displaying well and actions (delete, rename) with it are seem to working well. Why do I get this errors in browser console?
Also I noticed that this picture(my admin avatar) is duplicating (appear in two places/paths) in:
voyager\storage\app\public\users\December2018
and also in:
voyager\public\storage\users\December2018
Is it ok? Also same actions are apply to both of them.
Steps to reproduce the behavior:

install laravel via composer
install voyager via composer
config an .env in laravel, so I change next parameters:

APP_URL=http://voyager
DB_DATABASE=voyager
DB_USERNAME=root
DB_PASSWORD=
My server is accessable on http://voyager I use Open server.

past "php artisan voyager:install" to cmd and press enter
make an admin account via cmd  "php artisan voyager:admin<EMAIL_ADDRESS>--create"
I tried to config voyager\config\filesystems.php

'public' => [
'driver' => 'local',
'root' => storage_path('app/public'),
// 'url' => env('APP_URL').'/storage',
'url' => '/storage',
'visibility' => 'public',
],
or
'public' => [
'driver' => 'local',
'root' => storage_path('app/public'),
'url' => env('APP_URL').'/storage',
//'url' => '/storage',
'visibility' => 'public',
],
both ways produce same result for me.

So here I go to http://voyager/admin/media and this is were an error accurs in my browser console

GET http://voyager/storage/users/ 403 (Forbidden)
Expected behavior
If an image displays well and actions are working why should this errors accur? I wont any errors
Additional context
As I wrote in description section, I am not sure if the same picture should apper on two diffrent paths.

The image in 2 paths is perfectly normal, as you've already shown from your filesystems.php snippet. Please refer to the laravel documentation for details on how that all works, but essentially path/to/voyager/public/storage is a symlink to path/to/voyager/storage/app/public.
As for why you get the 403 on http://voyager/admin/users, that's almost certainly because your webserver is configured to refuse directory listings. However the root problem is why it's trying to load that in the first place.

Thanks! But can you please give me any tips where should I look to fix this issue with my webserver config?

emptynick's solution is working. fixed the errors.


No Bridge found

So i followed the Setup Instructions, but it allways says no Bridge found even tried dmx-hue setup -i with the IP adress of the hue Bridge behind it

I'm having the same issue. I have an older Hue Hub (Same model number :<PHONE_NUMBER>A)  which works but a newer one purchased in the last month that comes up with this error. No bridge found


Make the readme.md more informative

Sir bro update the readme.md to show in detail all tweaks Ktweak applies, why it applies and what we can gain from changing the parameter. For example:
/proc/sys/kernel/perf_cpu_time_max_percent: 25 -> 5
This is the maximum CPU time long perf event processing can take as a percentage. If this percentage is exceeded (meaning perf event processing used too much CPU time), the polling rate is throttled. This is reduced from 25% to 5%. We can afford inaccuracies with perf events in exchange for more time that a foreground task can use.
This will make Ktweak different from other optimizers which just says "CPU/GPU optimized and ram improved".

Also in your XDA post you said "In order to remain genuine, I have commited to explaining each and every kernel tweak that KTweak applies. Grab your coffee, this could take a while."
But the details in the XDA post are very outdated.


fix(fw): allow Bytecode(Opcode)

🗒 Description
Without this, I cannot fill eof tests with:
$ fill  --fork Osaka tests/osaka/eip7692_eof_v1/
...
___________________________________________________________ test_invalid_opcodes_after_stop[fork_Osaka-eof_test-terminating_opcode_STOP-opcode_OPCODE_0C] ___________________________________________________________
[gw3] linux -- Python 3.10.12 /home/user/sources/ethereum/execution-spec-tests/.venv/bin/python3
tests/osaka/eip7692_eof_v1/eip3540_eof_v1/test_all_opcodes_in_container.py:141: in test_invalid_opcodes_after_stop
    terminating_code = Bytecode(terminating_opcode)
src/ethereum_test_vm/bytecode.py:92: in __new__
    raise TypeError("Bytecode constructor '__new__' didn't return an instance!")
E   TypeError: Bytecode constructor '__new__' didn't return an instance!

I think #932 introduced it.
I'm not sure if this is the correct way to Python this. Alternative I have is to do Bytecode() + some_opcode instead of Bytecode(some_opcode) in the said test
🔗 Related Issues
na
✅ Checklist

[x] All: Set appropriate labels for the changes.
[ ] All: Considered squashing commits to improve commit history.

I'm not sure if this is the correct way to Python this. Alternative I have is to do Bytecode() + some_opcode instead of Bytecode(some_opcode) in the said test

Definitely Bytecode(some_opcode) is preferred and we should apply your fix in this PR.
The alternative seems to fix the error because + is overloaded in Bytecode which results in Bytecode() + some_opcode returning the expected type:
https://github.com/ipsilon/execution-spec-tests/blob/bf37249b38548ae672f822ee45c18866fc1975f6/src/ethereum_test_tools/vm/opcode.py#L155-L158
I still don't understand why Bytecode(some_opcode) doesn't work in this context - I didn't look into it further though, hope that's ok! Happy to look if you'd like more clarification.


Base UI display with API URL .

Base UI display with API URL.
Cypress Config

My Code

URL displaying on the browser.

Hey! Thanks for the report! Just releasing a fix 👍

Thanks @filiphric , it's fixed in the updated version.2.9.1


services/horizon/ingest: OffersProcessor fuzzing function

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
Adds a fuzzing function to OffersProcessor. I used this one because ingestion code is actually in a processor (see https://github.com/stellar/go/issues/3196`).
Known limitations

A have a generator function for corpus but will push it to a separate repo (https://github.com/stellar/go/issues/3197).

Generator source:
package main

import (
	"bufio"
	"fmt"
	"os"

	"github.com/stellar/go/clients/horizonclient"
	"github.com/stellar/go/xdr"
)

func main() {
	client := horizonclient.DefaultPublicNetClient

	opRequest := horizonclient.OperationRequest{Order: horizonclient.OrderDesc, Limit: 200}
	ops, err := client.Operations(opRequest)
	if err != nil {
		panic(err)
	}

	records := ops.Embedded.Records
	txHashes := map[string]bool{}

	for _, op := range records {
		t := op.GetType()

		if t != "manage_sell_offer" && t != "manage_buy_offer" {
			continue
		}

		txHashes[op.GetTransactionHash()] = true
	}

	dir, err := os.Getwd()
	if err != nil {
		panic(err)
	}
	fmt.Println(dir)

	for hash := range txHashes {
		fmt.Println(hash)

		tx, err := client.TransactionDetail(hash)
		if err != nil {
			panic(err)
		}

		var meta xdr.TransactionMeta
		err = xdr.SafeUnmarshalBase64(tx.ResultMetaXdr, &meta)
		if err != nil {
			panic(err)
		}

		for i, opChanges := range meta.V2.Operations {
			f, err := os.Create("./corpus/" + fmt.Sprintf("%s-%d", hash, i))
			if err != nil {
				panic(err)
			}

			w := bufio.NewWriter(f)
			_, err = xdr.Marshal(w, opChanges.Changes)
			if err != nil {
				panic(err)
			}
			w.Flush()

			f.Close()
		}
	}
}

@bartekn do you still want to pursue this?

@2opremio probably not as a part of this PR but a fuzzer for OffersProcessor should be part of the fuzzing suite @tamirms was working on.

Then, is it OK if we close the PR but keep the branch around?

Yes, I think so.