Use 'matches these text patterns' or 'include any of these words' under 'Use mail flow rules'

Under Use mail flow rules the documentation states to create a condition as follows:

Mail flow rule condition: A message header > includes any of these words > Header name: Authentication-Results > Header value: dmarc=pass or dmarc=bestguesspass.

However the screenshot shown under action 4 shows the rule created with 'header matches' instead of 'header includes' which is the outcome when selecting a different mail flow rule condition:

Mail flow rule condition: A message header > matches these text patterns > Header name: Authentication-Results > Header value: dmarc=pass or dmarc=bestguesspass.

Please can the documentation be consistent, so that it either advises 'includes any of these words' and 'header includes' in the screenshot, or 'matches these text patterns' and 'header matches' in the screenshot, depending on what the best option to use is (possibly 'matches these text patterns' and 'header matches')?
Many thanks

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

ID: c900a9d1-897d-f7e9-6c58-a510edf56a9b
Version Independent ID: bc59ac94-233b-e558-4455-a4c7166d4023
Content: Create safe sender lists - Office 365
Content Source: microsoft-365/security/office-365-security/create-safe-sender-lists-in-office-365.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Borgquite, the screenshot was old, using the classic EAC. I updated the screenshot to match the text in the new EAC.
Any condition or exception that has 'text patterns' in the name means regular expressions (RegEx). There isn't much difference between 'includes any of these words' and 'matches these text patterns' when you use plain text letters and numbers. But even a rather benign character like a period means something in RegEx, so you would need to account for that.
Use 'includes any of these words' if you're going to use discreet text values and use 'matches these text patterns' if you're going to use a RegEx expression.
Does that help?

@chrisda Thanks for this & the clarification - appreciated :)


change the download url

Hi, I am in China and i cant link to the https://go.googlesource.com/go.
So a better address to replace it??
thanks

Yeah, due to the Google connecting issue in China. We can't use gvm at all.
Can you guys give a chance to let user change the repo url to
github one?  https://github.com/golang/go

same issue with yours, the url still is https://go.googlesource.com/go.

same problem in CHINA

我替换了下载源与版本获取源  https://github.com/mosaic101/gvm，使用方式不变

The environment variable GO_BINARY_BASE_URL was created with the following commit.
https://github.com/moovweb/gvm/commit/ed2c95bcafaa91c123ec3868570d07b8d6e8186c

Will take a look in my morning. I haven't been able to make create a new release yet so you will need to clone the repo before trying.


Add Back-end support for allowJavaScript toggle

This relates to thimble issue
https://github.com/mozilla/thimble.mozilla.org/issues/1631
To fix this issue needs to modify brackets and thimble.
This request is to modify brackets.
Here is to modify thimble
https://github.com/mozilla/thimble.mozilla.org/pull/1725
I will send thimble pull request later.

Nice work @peiying16!


Termi in the new OS

i currently have an operating system (not sure if it can run bash), we need termi to do default terminal in our os (subject to your license), but first we need:\

know if termi can run with newlib instead of linux glibc\
1st compilation
official translation
handling copyright and legal information

Because we talked on Discord and you need old shell code will be uploaded to old branch

But if you still need GUI, you may need to wait more or you can try help us


Enhance README.md for Interactivity

Description 🌟
We are looking to enhance the interactivity of the README.md file for our project. A well-crafted and informative README will help users better understand the project and encourage contributions. 📈
Objectives 🎯

Visual Enhancements:

Add diagrams or flowcharts to illustrate project concepts. 📊
Include screenshots or GIFs to demonstrate features or usage. 📸✨

Interactive Elements:

Embed links to relevant resources, tutorials, or articles. 🔗📚
Implement badges (e.g., build status, version) for immediate project insights. 🏷

User Engagement:

Encourage contributions with a clear contribution guide. 🤝💡

Resources 📁

Current README.md: View Here

Contributor Input 💬
I encourage all contributors to share their suggestions and ideas in the comments below, including images or screenshots. The most engaging and interactive proposals will be prioritized for implementation. 🗨💭
Note: Please do not modify the about section of Project Guidance; keep the original content intact. ⚠
Expected Outcome 🌈
The goal is to create a more interactive, visually appealing, and informative README.md file, ultimately leading to increased user engagement and contributions. 🚀
Deadline ⏰
Please submit your proposals by October 28th, 2024.
Additional Notes 📝
Feel free to suggest any other ideas or enhancements that you believe would benefit the README. Your input is highly valued! 💖
Connect with Me 📱
LinkedIn

can you please assign this to me @Kushal997-das

Hi @puneeth072003,
Could you please work on this? If you could send the image here, we’d love to consider adding it. Thank you!

Yep sure

can you please assign this to me

Could you please work on this? If you could send the image here, we’d love to consider adding it. Thank you!

Could you please work on this? If you could send the image here, we’d love to consider adding it. Thank you!

sure

@Kushal997-das Assign this task to me plz

@Kushal997-das Can i work on it ?


clarification on when to use transient lifetime within FastAPI

I have a FastAPI application that uses a database. I have defined the following services:
@service
def engine(app_settings: AppSettings) -> Engine:
    """Provide a database engine."""
    return create_engine(app_settings.get_db_url())

@service
def db(engine: Engine) -> Iterable[Session]:
    """Provide a database session."""
    session = Session(engine)
    try:
        yield session
    finally:
        session.close()

I'm now wondering if I need to use the transient lifetime setting on the Session service to properly close the db when the request is handled. How does this compare to the regular FastAPI dependency injection mechanism: https://fastapi.tiangolo.com/tutorial/dependencies/dependencies-with-yield/. I noticed an issue where you talk about a 'request' scoped dependency. Would that also apply here?
Thanks for your time.

Hi, yes that would apply here as well.
If the session should not be a singleton then the transient scope should be used, as it will cause a new session to be created every time it is injected. In fastapi, the cleanup will occur after the route has finished executing.
Request scopes are a feature that will be coming soon that allows scoped dependencies to share the same instance for the duration of the scope, which for fastapi will be tied to the request lifetime.
For this it would mean that if session has scoped lifetime, the container will create one session object and use that for injections in that request only.

Hi @maldoinc,
Thank you for taking the time to answer my questions. So, if I understand correctly:

I should use transient on my Session service because it needs to be a fresh session for every route request. The session will be closed by the finally block when the route request is handled.
Transient scope means always a new instance of the service, whereas the default is to return the same instance for the duration of the container lifetime.
If I mark my session dependency as transient and only request it once, would that mean that it's effectively the same as scoping it to the request lifetime within a FastAPI? And when requesting it multiple times, I would get a fresh instance everytime. And in the future, when using request scope, when I request it several times in the same request that I would get the same instance back?

Thanks for clarifying this, I just want to make sure that I'm using the container correctly and not get surprised later on.

Yes, that's all correct. Another feature coming out of the scoped lifetime work is to enable generator factories to be notified for exceptions and act accordingly.
Your session object for example can commit or rollback the changes once the router has finished depending on success
@service
def db(engine: Engine) -> Iterable[Session]:
    """Provide a database session."""
    session = Session(engine)
    try:
        yield session
        session.commit()
    except:
        session.rollback()
    finally:
        session.close()

Thanks you!


Merging client app

Merging client app to the vmd repo as it was proposed on the Monday's meeting

During the googling how to clone one remote repo to another I ran into the better way: create submodule.
What do you think?

I suggest we just copy the code and merge in one pull request directly in the Repo.

We discussed this elsewhere and came to the conclusion that maybe we want a separate repo for the client.


Documentation

Document the world

Nice work.

@orta thnx for proof reading. I'll make the updates tonight. It would be great if you could also take a peak after I've made those changes.

I'll aim to give it a real look over again today too, I only have a little bit of time yesterday

@orta Awesome, incorporating the changes ...

@orta I've incorporated all of the changes you've suggested. This is good stuff. When I see your formulation, my mind understands better what you've said then what I've tried to explain :)
Your feedback is pure bitcoin :), so what ever time you can spare on helping me with this, I really appreciate it.
:bow:

If you're willing to wait till the weekend, I'll give it another run over on our peer labs, Artsy is doing an offsite.

Yeah, sure. It's better to do this right then fast :) I have tons of things I need to improve till weekend anyway, including documentation :)

:star2:


qrcode device category (CON-318)

How is it possible to generate a qrcode that also contains the information of the device category
With apple home, if you scan the qrcode generated, the app shows me a generic accessory.

I wish he would show me the correct category, in my case a light.
I've tried other devices matter that show it

but using mfg_tool I don't see a parameter to set this setting

@stefano055415

How is it possible to generate a qrcode that also contains the information of the device category

QR code does not contain the device type/category

I wish he would show me the correct category, in my case a light.

If the product model (including the vid, pid and other details) is uploaded in the DCL, then the correct category would be shown by the Apple Home. It reaches to the DCL to check if there's an entry for that device after scanning the QR code

I've tried other devices matter that show it

Please check the vid of the device that is showing this and it should be present here

@dhrishi  ok i understand thanks


[BUG] @UserMessage does not work

retrievalAugmentor in DefaultAiService seems to not copy name from the original UserMessage

I was trying to reproduce this example:
https://github.com/langchain4j/langchain4j-examples/blob/10d96a93097e87189deff323a48971bde8d22ade/other-examples/src/main/java/OtherServiceExamples.java#L306
And I got unexpected answer from language model. According to tutorial example it should reply with:
Hello! Your name is Klaus. How can I assist you today?
But I got:
I'm sorry, but I am not able to see or know your name as I am an AI assistant and do not have access to personal information. How can I assist you today?
Code sample to reproduce
https://gist.github.com/OTR/e742c38a8c9c77ac2b83d2fc81f37d10
Logs
https://gist.github.com/OTR/a12a233308852c75f80aaa505f656925

Fixed by https://github.com/langchain4j/langchain4j/pull/740


Test proposals, with status and comments

Test that all proposals are indexed
Test that all indexed proposals has the correct status
Test that comments are indexed with the correct blockheight

Update
All that is needed left to do:

[ ] process peter's feedback on the PR, which is to change 1 test


one must not always assume https

Thanks for bringing that up @cwulfman

What's the protocol: does the person who merges the pull request also delete the branch?

@cwulfman Yes, go ahead and delete the branch when you merge a PR.  There are some exotic situations where you might want to keep the branch around, but for one-off changes like this, it's fine to delete it when you merge.


"Ask every time" option for sav dump imports

The current option works well to select which frames to use probably for most people who don't juggle various cams. For those of us that do juggle various, it requires going back to settings to change back and forth to get the right frames. An option to choose the frames during import could be really useful.

done with version 1.13.2


Feature/differentiate admin vs collaborator 2

Closed in favor of https://github.com/ExpediaGroup/insights-explorer/pull/420


Some problems in the loop state

If there are some by ID binding events and in the loop state, due to the copied a, will go wrong, although you can think of some ways to modify and can fit over the problem, oh, in the loop may be passed not copy the two to achieve the next page on the last page

Sorry, just don't use IDs, use classes instead, can't see any problems with that :D


DRM 6001 error when trying to load Dash manifest on Chrome 72

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.3.4
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Are you using the demo app or your own custom app?
Custom
If custom app, can you reproduce the issue using our demo app?
Issue is only reproduceable on Chrome 72, even when using the demo app.

What browser and OS are you using?
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_16_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3625.0 Safari/537.36
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

(Can provide if absolutely necessary, but error is reproduceable using the demo app)
What configuration are you using?  What is the output of player.getConfiguration()?

Configuration settings
{
  "drm": {
    "retryParameters": {
      "maxAttempts": 2,
      "baseDelay": 1000,
      "backoffFactor": 2,
      "fuzzFactor": 0.5,
      "timeout": 30000,
      "stallTimeout": 5000,
      "connectionTimeout": 10000
    },
    "servers": {
      "com.widevine.alpha": "https://widevine-license.vudrm.tech/proxy"
    },
    "clearKeys": {},
    "advanced": {},
    "delayLicenseRequestUntilPlayed": false,
    "logLicenseExchange": false,
    "updateExpirationTime": 1,
    "preferredKeySystems": [],
    "keySystemsMapping": {},
    "parseInbandPsshEnabled": false
  },
  "manifest": {
    "retryParameters": {
      "maxAttempts": 2,
      "baseDelay": 1000,
      "backoffFactor": 2,
      "fuzzFactor": 0.5,
      "timeout": 30000,
      "stallTimeout": 5000,
      "connectionTimeout": 10000
    },
    "availabilityWindowOverride": null,
    "disableAudio": false,
    "disableVideo": false,
    "disableText": false,
    "disableThumbnails": false,
    "defaultPresentationDelay": 0,
    "segmentRelativeVttTiming": false,
    "dash": {
      "clockSyncUri": "",
      "ignoreDrmInfo": false,
      "disableXlinkProcessing": false,
      "xlinkFailGracefully": false,
      "ignoreMinBufferTime": true,
      "autoCorrectDrift": true,
      "initialSegmentLimit": 1000,
      "ignoreSuggestedPresentationDelay": false,
      "ignoreEmptyAdaptationSet": false,
      "ignoreMaxSegmentDuration": false,
      "keySystemsByURI": {
        (redacted)
      },
      "sequenceMode": false
    },
    "hls": {
      "ignoreTextStreamFailures": false,
      "ignoreImageStreamFailures": false,
      "defaultAudioCodec": "mp4a.40.2",
      "defaultVideoCodec": "avc1.42E01E",
      "ignoreManifestProgramDateTime": false,
      "mediaPlaylistFullMimeType": "video/mp2t; codecs=\"avc1.42E01E, mp4a.40.2\"",
      "useSafariBehaviorForLive": true,
      "liveSegmentsDelay": 3
    }
  },
  "streaming": {
    "retryParameters": {
      "maxAttempts": 2,
      "baseDelay": 1000,
      "backoffFactor": 2,
      "fuzzFactor": 0.5,
      "timeout": 30000,
      "stallTimeout": 5000,
      "connectionTimeout": 10000
    },
    "rebufferingGoal": 0.01,
    "bufferingGoal": 10,
    "bufferBehind": 30,
    "ignoreTextStreamFailures": false,
    "alwaysStreamText": false,
    "startAtSegmentBoundary": false,
    "gapDetectionThreshold": 0.5,
    "durationBackoff": 1,
    "forceTransmux": false,
    "safeSeekOffset": 5,
    "stallEnabled": true,
    "stallThreshold": 1,
    "stallSkip": 0,
    "useNativeHlsOnSafari": true,
    "inaccurateManifestTolerance": 0,
    "lowLatencyMode": true,
    "autoLowLatencyMode": false,
    "forceHTTPS": false,
    "preferNativeHls": false,
    "updateIntervalSeconds": 1,
    "dispatchAllEmsgBoxes": false,
    "observeQualityChanges": false,
    "maxDisabledTime": 30,
    "parsePrftBox": false
  },
  "mediaSource": {
    "sourceBufferExtraFeatures": ""
  },
  "offline": {
    "usePersistentLicense": true,
    "numberOfParallelDownloads": 5
  },
  "abr": {
    "enabled": true,
    "useNetworkInformation": true,
    "defaultBandwidthEstimate": 1000000,
    "switchInterval": 8,
    "bandwidthUpgradeTarget": 0.85,
    "bandwidthDowngradeTarget": 0.95,
    "restrictions": {
      "minWidth": 0,
      "maxWidth": null,
      "minHeight": 0,
      "maxHeight": null,
      "minPixels": 0,
      "maxPixels": null,
      "minFrameRate": 0,
      "maxFrameRate": null,
      "minBandwidth": 0,
      "maxBandwidth": null
    },
    "advanced": {
      "minTotalBytes": 128000,
      "minBytes": 16000,
      "fastHalfLife": 2,
      "slowHalfLife": 5
    },
    "restrictToElementSize": true,
    "restrictToScreenSize": false,
    "ignoreDevicePixelRatio": false
  },
  "autoShowText": 3,
  "preferredAudioLanguage": "",
  "preferredTextLanguage": "",
  "preferredVariantRole": "",
  "preferredTextRole": "",
  "preferredAudioChannelCount": 2,
  "preferredVideoCodecs": [],
  "preferredAudioCodecs": [],
  "preferForcedSubs": false,
  "preferredDecodingAttributes": [],
  "restrictions": {
    "minWidth": 0,
    "maxWidth": null,
    "minHeight": 0,
    "maxHeight": null,
    "minPixels": 0,
    "maxPixels": null,
    "minFrameRate": 0,
    "maxFrameRate": null,
    "minBandwidth": 0,
    "maxBandwidth": null
  },
  "playRangeStart": 0,
  "playRangeEnd": null,
  "cmcd": {
    "enabled": false,
    "sessionId": "",
    "contentId": "",
    "useHeaders": false
  },
  "lcevc": {
    "enabled": false,
    "dynamicPerformanceScaling": true,
    "logLevel": 0,
    "drawLogo": false
  }
}

What did you do?

player.load() with described settings.
What did you expect to happen?
Expect video to load on all browsers/user agents
What actually happened?

Video loads on most all user agents (e.g. desktop Chrome, embedded devices using Chromium) but throws a 6001 error on the affected user agent:
category: 6
code: 6001
data: []
handled: false
message: "Shaka Error DRM.REQUESTED_KEY_SYSTEM_CONFIG_UNAVAILABLE ()"
severity: 2
stack: "Error: Shaka Error DRM.REQUESTED_KEY_SYSTEM_CONFIG_UNAVAILABLE ()\n    at new shaka.util.Error (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:48495)\n/n)    at $jscomp.generator.Engine_.program_ (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:14:129361)\n/n)    at $jscomp.generator.Engine_.nextStep_ (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:10101)\n/n)    at $jscomp.generator.Engine_.next_ (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:9065)\n/n)    at $jscomp.generator.Generator_.next (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:10546)\n/n)    at http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:11119\n    at new Promise ()\n    at Object.$jscomp.asyncExecutePromiseGenerator (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:11012)\n/n)    at Object.$jscomp.asyncExecutePromiseGeneratorProgram (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:6:11308)\n/n)    at shaka.media.DrmEngine.queryMediaKeys_ (http://<IP_ADDRESS>:3000/_app/chunks/player-0ab0bfa6.js:14:129104)"

Widevine from time to time deprecates the CDMs of old browsers, if I remember correctly now it is necessary to use at least Chrome 95. For more details please contact with Widevine team. Thanks!

Widevine from time to time deprecates the CDMs of old browsers, if I remember correctly now it is necessary to use at least Chrome 95. For more details please contact with Widevine team. Thanks!

Álvaro, would this Widevine compatibility issue only apply to Shaka? I can load and play DRM videos using Widevine and dash.js on Chrome 72 without issue.


How to properly launch MainActivity from a service running in a plugin?

For my flutter app, I've developed a custom plugin which implements a service. The service needs to launch the the MainActivity of the flutter app from the plugin.
This way reloads the app even when it's already running, so I can't use it because the app loses its state:
Intent intent = getPackageManager().getLaunchIntentForPackage("com.example");
if (intent != null) startActivity(intent);

Everything's fine if the app is running in the background. Then the app doesn't reload.
I've tried this in the AndroidManifest.xml, and it doesn't help:
android:launchMode="singleTask"

Anyone?

Hi @kbrmimbyl
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself, if you disagree please write in the comments and I will reopen it
Thank you

With respect, I do disagree. Why is this issue template even present here?
"I want help writing my application
You have a question for how to achieve a particular effect, or you need help with using a particular API."


split_protein_and_ligand

Hi Wei,
I ran the example prediction code and get error when using the 'split_protein_and_ligand', i wonder if it's a network problem from jupyter lab or other problem.
Thanks!

yes. it seems a network problem. The code tries to download the sdf file but failed.
you probably have to prepare the .pdb, and .sdf file yourself.

hello, another question about the supported protein structure, does tankbind currently support the protein structure predicted by AlphaFold?

kind of. I think the result won't be very bad. but, we haven't test its performance with AlphaFold structure.


Broken documentation links

When I try and click through to the suggested doc page for the querystring.regfilter I get a 404
Link: https://www.fastly.com/documentation/reference/vcl/functions/querystring-regfilter

1:53
On the vscode marketplace page, the link to the GitHub issues page does not work either:
https://github.com/fastly/vscode-fastly-vcl/new

Thanks, it looks like that should be to https://www.fastly.com/documentation/reference/vcl/functions/query-string/querystring-regfilter/ instead


added to sprint challenge

@ielvisd

added to sprint-challenge.

added index.css

made changes to about.html and completed about.css.


Add serum id for parasol

@keone Seems like there is an issue while json parsing with following parameters:
serumV3Usdc
coingeckoId
Could you please look into it. I just added serum id , coingeckoId is also not parsing!
Yes the input fields are correctly enumerated!

yea the text could be better, it says that when it requires automerge


Problem binding physical NIC in bf_switchd.

Hello,
Has anyone gone through the situation I'm going through to link a physical NIC to p4-dpdk-target .
I already have no idea what to do to successfully execute the binding.
The problem only occurs when using BFRT, I tested it with the DPDK examples and the card links normally.
The error that appears when trying to attach the physical card to bf_switchd is how it cannot read the ports.json file, but this only happens with physical cards.
Log syslog.
Oct 19 23:48:13 master-node dummy[942929]: EAL: Probe PCI driver: net_nfp_pf (19ee:4000) device: 0000:41:00.0 (socket 1)
Oct 19 23:48:13 master-node kernel: [34440.907257] igb_uio 0000:41:00.0: uio device registered with irq 87
MInha dúvida ao efetuar os teste descritos é:Oct 19 23:58:07 master-node dummy[957429]: 2022-10-19 23:58:07.648645 ERROR  (BF_PORT:(file=null):(func=null):0) - port_config_json_parse_ports:507: Failed to add port LINK0
Oct 19 23:59:07 master-node dummy[957429]: 2022-10-19 23:58:07.648667 ERROR  (BF_PORT:(file=null):(func=null):0) - parse_port_config_json:586: Failed to parse ports from Port Config JSON
Oct 19 23:59:07 master-node dummy[957429]: 2022-10-19 23:58:07.648705 ERROR  (BF_PORT:(file=null):(func=null):0) - port_mgr_config_import : Error in parsing the port config file
Oct 19 23:59:07 master-node dummy[957429]: 2022-10-19 23:58:07.648744 ERROR  (BF_DVM:(file=null):(func=null):0) - Device add handling failed for dev 0, sts No system resources (2), Client port_mgr
Oct 19 23:59:07 master-node dummy[957429]: 2022-10-19 23:58:07.648764 ERROR  (BF_DVM:(file=null):(func=null):0) - Device add failed for dev 0, sts No system resources (2)
Log bfshell
LINK CREATE: Dev:0000:41:00.0 Args: dev_hotplug_enabled: 0
2022-10-19 09:54:00.196125 BF_LLD ERROR - Creation of Link Port LINK0 failed
2022-10-19 09:54:00.196285 BF_PORT ERROR - port_config_json_parse_ports:507: Failed to add port LINK0
2022-10-19 09:54:00.196322 BF_PORT ERROR - parse_port_config_json:586: Failed to parse ports from Port Config JSON
2022-10-19 09:54:00.196377 BF_PORT ERROR - port_mgr_config_import : Error in parsing the port config file
2022-10-19 09:54:00.196436 BF_DVM ERROR - Device add handling failed for dev 0, sts No system resources (2), Client port_mgr
2022-10-19 09:54:00.196466 BF_DVM ERROR - Device add failed for dev 0, sts No system resources (2)
ERROR: bf_device_add failed(2) for dev_id 0
ERROR: bf_switchd_device_add failed(2) for dev_id 0
bf_switchd: initialized 0 devices
bf_switchd_switchsai_lib_init
starting SAI
bf_switchd: spawning cli server thread
bf_switchd: spawning driver shell
bf_switchd: server started - listening on port 9999
    ********************************************
    *      WARNING: Authorised Access Only     *
    ********************************************

Every help is welcome,
Thanks.

Your port config json file is mot correct...

2022-10-19 23:58:07.648667 ERROR (BF_PORT:(file=null):(func=null):0) - parse_port_config_json:586: Failed to parse ports from Port Config JSON

Your port config json file is not correct...

2022-10-19 23:58:07.648667 ERROR (BF_PORT:(file=null):(func=null):0) - parse_port_config_json:586: Failed to parse ports from Port Config JSON

And I'm not sure but I thought bfrt is the version for physical Tofino chip based ethernet switches so you can't connect your nic card to that switch in software...

The problem only occurs when using BFRT, I tested it with the DPDK examples and the card links normally.

The DPDK version is using DPDK SWX code as the dataplane switch and as such you can connect your local nic to it...
But it could be I didn't understand your setup...

Hey,
Thank you for your comment, and you are right in your speech, it ended up that I mistakenly explained my problem citing the BFRT, where the right thing is to run the switch with the DPDK SWX.
But with that, a question arises, would there be a way to check if when running bf_switchthd it is loading/finding the necessary files for it to run the DPDK SWX?
Please note that p4-dpdk-target was installed following the steps of the github repository.
Sincerely thank you.

But with that, a question arises, would there be a way to check if when running bf_switchthd it is loading/finding the necessary files for it to run the DPDK SWX?

As far as I know the code in this repo compiles to a program destined to work for DPDK SWX and as such you always know... 😏
If you want the code to support Tofino network chip family you need to compile with the Intel/Barefoot SDE (Software Development Environment) and for that you need to sign a NDA with Intel...


segmentation fault when running from a different folder on Mac

I'm using the portable version on a Mac. If I open Terminal on the realesrgan folder and run
./realesrgan-ncnn-vulkan -i /path/to/input.jpg -o /path/to/output.jpg
it works.
However if I run it from any other folder, for example
cd ..
./realesrgan-ncnn-vulkan-20220424-macos/realesrgan-ncnn-vulkan -i /path/to/input.jpg -o /path/to/output.jpg

I'm getting a segmentation error:
zsh: segmentation fault  ./realesrgan-ncnn-vulkan-20220424-macos/realesrgan-ncnn-vulkan -i /path/to/input.jpg -
Running from a different folder on Windows works fine. I was wondering if there's a way to avoid this on my side or fix on your side? Thank you.

Look the settings

Look the settings

I don't know what you mean by that. Settings of what?

I do not have the mac environment. To wait for someone else to fix it 😃

@sergey-kritskiy seems you need to specify the complete path of the models using the -m flag when using the command on files outside of its root folder.

@hay oh wow, thank you for the tip, it worked! I also had to provide full paths to input and output images

python3 inference_realesrgan.py -n RealESRGAN_x4plus -i input.png -o output.png -s 2 --fp32
i have same problem

@hay oh wow, thank you for the tip, it worked! I also had to provide full paths to input and output images

care to give an example for a run command that worked?

i have same error when i run it with any of these flags: -n -m


Avoid terminating when a precondition is not met in HTTPServerPipelineHandler

Motivation:
HTTPServerPipelineHandler has a few preconditionFailures in its state machine. This isn’t really ideal, as while these are important invariants, it is fairly easy for users to accidentally break them.
Modifications:

The first time a user hits an “illegal” state transition, we fire an error down the pipeline
The second time they hit one, we forcibly close the pipeline (and send another error)

We continue to crash (by calling assertionFailure) when in debug mode, to make it obvious to users that they have violated an invariant
However, an internal flag allows us to disable this behaviour for testing
Result:
Making an illegal state change in HTTPServerPipelineHandler will no longer crash

@swift-server-bot add to allowlist


Forwarded ports still not working on localhost

Despite https://github.com/coreos/rkt/issues/1256 and https://github.com/coreos/rkt/issues/2027 this doesn't seem to work.
I'm running CoreOS alpha 983.0.0 via https://coreos.com/os/docs/latest/booting-with-qemu.html which has rkt version 1.0.0. git tag --contains 7b1437d confirms rkt 1.0.0 is supposed to have the bugfix.
Using the hello image from https://coreos.com/rkt/docs/latest/getting-started-guide.html and running it with sudo rkt --insecure-options=image run --port=www:5555 hello-0.0.1-linux-amd64.aci results in
core@coreos_production_qemu-983-0-0 ~ $ curl -v <IP_ADDRESS>:5555
* Rebuilt URL to: <IP_ADDRESS>:5555/
*   Trying <IP_ADDRESS>...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 5555 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>:5555
> User-Agent: curl/7.47.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Wed, 16 Mar 2016 02:53:50 GMT
< Content-Length: 6
< Content-Type: text/plain; charset=utf-8
<
hello
* Connection #0 to host <IP_ADDRESS> left intact
core@coreos_production_qemu-983-0-0 ~ $ curl -v <IP_ADDRESS>:5555
* Rebuilt URL to: <IP_ADDRESS>:5555/
*   Trying <IP_ADDRESS>...
^C

core@coreos_production_qemu-983-0-0 ~ $ head /proc/sys/net/ipv*/conf/*/route_localnet
==> /proc/sys/net/ipv4/conf/all/route_localnet <==
0

==> /proc/sys/net/ipv4/conf/default/route_localnet <==
0

==> /proc/sys/net/ipv4/conf/eth0/route_localnet <==
0

==> /proc/sys/net/ipv4/conf/lo/route_localnet <==
0

==> /proc/sys/net/ipv4/conf/vethb8f91e3b/route_localnet <==
1
core@coreos_production_qemu-983-0-0 ~ $

Enabling route_localnet for all interfaces, or just about any other /proc/sys/net fiddling I can think of, does not help.
core@coreos_production_qemu-983-0-0 ~ $ for a in /proc/sys/net/ipv*/conf/*/route_localnet; do sudo sh -c "echo 1 >$a"; done
core@coreos_production_qemu-983-0-0 ~ $ curl -v localhost:5555
* Rebuilt URL to: localhost:5555/
*   Trying <IP_ADDRESS>...
^C

Here's the difference the iptables packet counters see, sudo iptables -L -v -n -t nat -Z:
--- good	2016-03-15 19:26:11.675948812 -0700
+++ bad	2016-03-15 19:27:01.592171331 -0700
@@ -11,7 +11,7 @@

 Chain POSTROUTING (policy ACCEPT 1 packets, 60 bytes)
  pkts bytes target     prot opt in     out     source               destination
-    0     0 RKT-PFWD-SNAT-3d33c4dd  all  --  *      *       <IP_ADDRESS>           !<IP_ADDRESS>
+    1    60 RKT-PFWD-SNAT-3d33c4dd  all  --  *      *       <IP_ADDRESS>           !<IP_ADDRESS>
     0     0 CNI-default-9f26248177c022ec  all  --  *      *       <IP_ADDRESS>/24       <IP_ADDRESS>/0

So, I don't know what's up, but #1256 does not seem fixed. I'm creating a new issue instead of reopening, in case the reason is different.

@steveeJ do you have time to take a look at this?

@tv42 I suspect this is because the port you gave on the CLI is different from the port the application is actually listening on. Could you confirm this and maybe post the ImageManifest?

@steveeJ Yes, if I don't change the port, both work:
sudo rkt --insecure-options=image run --port=www:5000 hello-0.0.1-linux-amd64.aci
core@kludgegore ~ $ curl <IP_ADDRESS>:5000
hello
core@kludgegore ~ $ curl <IP_ADDRESS>:5000
hello
core@kludgegore ~ $ rkt image cat-manifest example.com/hello:0.0.1
{"acKind":"ImageManifest","acVersion":"0.7.1+git","name":"example.com/hello","labels":[{"name":"version","value":"0.0.1"},{"name":"arch","value":"amd64"},{"name":"os","value":"linux"}],"app":{"exec":["/bin/hello"],"user":"0","group":"0","ports":[{"name":"www","protocol":"tcp","port":5000,"count":1,"socketActivated":false}]},"annotations":[{"name":"appc.io/acbuild/command-1","value":"acbuild set-name \"example.com/hello\""},{"name":"appc.io/acbuild/command-2","value":"acbuild copy \"hello\" \"/bin/hello\""},{"name":"appc.io/acbuild/command-3","value":"acbuild set-exec \"/bin/hello\""},{"name":"appc.io/acbuild/command-4","value":"acbuild port add \"www\" \"tcp\" \"5000\""},{"name":"appc.io/acbuild/command-5","value":"acbuild label add \"version\" \"0.0.1\""},{"name":"appc.io/acbuild/command-6","value":"acbuild label add \"arch\" \"amd64\""},{"name":"appc.io/acbuild/command-7","value":"acbuild label add \"os\" \"linux\""},{"name":"authors","value":"Carly Container \u003ccarly@example.com\u003e"},{"name":"appc.io/acbuild/command-8","value":"acbuild annotation add \"authors\" \"Carly Container<EMAIL_ADDRESS>


FOR NEXT loop, bug?

Hello,
I'm usin a simple FOR NEXT loop... Example:
: myLoop 5 for r@ . next ;
It's display 5 4 3 2 1 0
The problem: this loop make 6 iteration... No 5 iterations as attempted.
On otehr FORTH versions with for..next structure, tehy make only 5 iterations.
B R

have received mail from Brad


Prepare a Release ready for thinking about auto-updating

Use GitHub actions to build a MAUI app:
https://blog.taranissoftware.com/build-net-maui-apps-with-github-actions
Prepare an artefact and then a Release from the output:
https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-net

Artefacts are not deterministic therefore it is not possible to auto-update from a Github release or artefacts.
Therefore, need to investigate if there is an API that can be used to locate any update.


Any support for normal mode in future?

The gradients look absolutely stunning, I wonder if it can also be applied to normal tabs? It would be cool.

That's a good idea, and now that the latest Zen update hides the settings for each mod, I'm no longer scared about filling the mod list with color choices 😄
I will likely add this as a separate mod in the near future. I'll drop another comment here when I do so.

Thanks, just take your time.


refactor init command handler

simply logic
Signed-off-by: Gao feng<EMAIL_ADDRESS>

could you do the refactor after libhyperstart merged? if it is still needed.....

why close? i can force push this


Inconsistent index type handling

There seems to be a mismatch with the handling of column definition index types as written in the examples and the backend storing / reading them.
Example extract from com.riptano.cassandra.hector.example.SchemaManipulation:
            BasicColumnDefinition columnDefinition = new BasicColumnDefinition();
            columnDefinition.setName(stringSerializer.toByteBuffer("birthdate"));
            columnDefinition.setIndexType(ColumnIndexType.KEYS);
            columnDefinition.setValidationClass(ComparatorType.LONGTYPE.getClassName());

When storing this definition to cassandra, at some point me.prettyprint.cassandra.service.ThriftColumnDef.toThrift() will be called.
Extract from toThrift():
    ColumnDef d = new ColumnDef();
    if ( indexName != null ) {
      d.setIndex_name(indexName);
      d.setIndex_type(indexTypeToThrift(indexType));
    }

-> When creating a column definition as in the example above, the index type will not be stored, as no index name has been given.
I can't judge what's wrong: Example or toThrift()?

According to http://wiki.apache.org/cassandra/API#ColumnDef 's description of "index_name", toThrift() is right: An index type cannot be given without an index name.
-> The example could be improved by adding a columnDefinition.setIndexName(name) or dropping the setIndexType() part.

Good catch - as of early versions on 1.0.x, "index_name" must be included as well for a valid secondary index. This is not so much a bug as it is a hole in the unit tests and examples. Thanks for bringing this up.


TKCoverflowCoverView.m - Xcode Static Analyzer Warning

Hi,
TKCoverflowCoverView.m, Line 72: Is there any reason why you release and retain the img argument? I'm curious because image is just a local variable and img will be retained by the UIImageView anyway.
- (void) setImage: (UIImage *) img {

    // *** Xcode static analyzer warning here. ***
    UIImage *image = img;
    [image release];
    image = [img retain];

    // More code...
}

Thanks.

Got it with the latest commit today!


Release compatible with 1.17.1?

I only use 1.17.1 because I'm afraid if I switch versions, I would lose half or most, if not all, my mods.
Can I get a custom release for 1.17.1 compatibility? Or is it compatible with Java 16 or 17 and I can change numbers in the files from 1.18 to 1.17.1 and if so, which files and strings do I change?

Duplicate of #120


nb-psc-l7ilb: support passing in subnet

The psc internal load balancer module creates a subnet, but there are scenarios were subnets are provisioned ahead of time by the team that manages the network.
For example, our org heavily relies on shared VPCS and the networking team provisions the subnets for the dev teams, and from my understanding, only one regional managed proxy can be active in a given region for a VPC.
It would be nice to support passing in a subnet for the proxy and skip the creation of it.

Hi @derhally , thanks for the input. Certainly understand the flexibility that you're asking for.
@g-greatdevaks created the module so I'll let him chip in. From what I saw in this specific case the module doesn't even use the created subnet but the one passed in via the module param: https://github.com/apigee/terraform-modules/blob/main/modules/nb-psc-l7ilb/main.tf#L79.
In this case just commenting out the network resources on the module should do the trick for you.
I'm leaving this open to consider a clean up and implement more modular defaults.

Thanks for raising the issue.
As Daniel pointed out, the module is flexible to take subnet from the subnet resource or from the module param/variable.
I believe some usage documentation should be added to provide clarity on the same with some examples of passing subnet link(s). One example is shown here.
Will take the documentation task for this issue and update here once done.

@g-greatdevaks I'm specifically referring to the resource google_compute_subnetwork.proxy_subnet not the subnet being used with the google_compute_forwarding_rule


rename the RST_STREAM frame to RESET_STREAM frame

see https://github.com/quicwg/base-drafts/pull/1961

Codecov Report

Merging #1582 into master will not change coverage.
The diff coverage is 86.67%.

@@           Coverage Diff           @@
##           master    #1582   +/-   ##
=======================================
  Coverage   87.46%   87.46%
=======================================
  Files          95       95
  Lines        6116     6116
=======================================
  Hits         5349     5349
  Misses        547      547
  Partials      220      220

Impacted Files
Coverage Δ

stream.go
74% <0%> (ø)
:arrow_up:

internal/wire/reset_stream_frame.go
100% <100%> (ø)

receive_stream.go
91.78% <100%> (ø)
:arrow_up:

internal/wire/frame_parser.go
97.3% <100%> (ø)
:arrow_up:

session.go
73.4% <100%> (ø)
:arrow_up:

send_stream.go
89.5% <100%> (ø)
:arrow_up:

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 7087a16...022189d. Read the comment docs.


Grafana setup in docs,mesheryctl,UI

Current Behavior

I don't find any  instructions related to Grafana setup
Desired Behavior

step1: Adding Grafana setup instructions  in Guides
https://meshery.layer5.io/docs/guides
step2: Mention this guide in the metrics configuration/Grafana setting  page
step3: option in mesheryctl (grafana start) that automatically configures grafan with meshery

Resources

manual steps:
1) mesheryctl start
All  meshery containers are in meshery_default  network.
2) Run Grafana container
docker run -d -p 3000:3000 --name grafana grafana/grafana:6.5.0
3)Login to localhost:3000 and generate API key
4)Attach Grafana container to meshery_default network.
docker network connect grafana meshery_default
5)get Grafana container IP address which belongs to meshery_default subnet
docker inspect --format '{{(index .NetworkSettings.Networks "meshery_default").IPAddress}}' grafana
6) use  the IP address generated in step5 in meshery UI.
Alternatives / Additional Context

@nitishvu this is a good call out. This example shows how to bring in a separate Grafana instance and connect to it. A PR to the docs on this would be great.
In addition, a PR to include a description of the fact that the Meshery adapter for Istio includes an instance of Grafana and so too does the Meshery adapter for Linkerd.
@vijcheru we've yet to expose the MeshSync discovery that you've done. This is one of the uses of that endpoint. Will you point it out to @nitishvu?
@shivaylamba, we're in need of tying the above together - considerations of auto-discovered and manually added Grafana instances; @nveenjain, we need support for multiple instances of Grafana (perhaps, via a new environment logical object).


responsive-website

as a user, I want to have responsive website for 3 screens, laptop, monitor and phone

Working on it


Rebase xunit fork on latest upstream, and send xunit a PR for the latest relevant changes

The fork at https://github.com/aspnet/aspnet.xunit is both behind and ahead of the upstream branch.
We need to rebase the fork to the latest upstream, then send a PR with any relevant changes that we have in the fork.
@davidfowl @pakrym FYI

First part ends at aspnet/aspnet.xunit@4f8591e

Upstream change proposal in xunit/dnx.xunit#62 but it can't go forward 'til we update vnext feed.

xunit/dnx.xunit#62 is now in.


Request: Data driven tests

I'm just getting started with canopy, and I'm loving how simple it is to work with compared to CodedUI, but there's one big thing missing for me: the ability to data drive the tests.
I'd love the ability to run a test with a data set - what I'm doing now (which isn't ideal) is that all the iterations from the data set are one "test".

One of the things that I try to keep true with canopy is that 'you can do whatever you want'.
You can either iterate your different scenarios in a single test or if you want you can create a test per.
type User = { Username : string; Password : string }
let validateLogin user =
  sprintf "Validing login for %s" user.Username &&& fun _ ->
    url "http://yourpage.com"
    "#username" << user.UserName
    "#password" << user.Password
    click "Login"
    on "http://yourpage.com/home"

[
  { Username = "Bob"; Password = "12345678" }
  { Username = "Tom"; Password = "12345678" }
  { Username = "Sue"; Password = "12345678" }
  { Username = "Sally"; Password = "12345678" }
] |> List.iter validateLogin

That should create 4 tests with 'correct' descriptions.
Does this help?

Yes, it does. It gives me the reporting level I need with the dynamic data I have to work with.
Thank you.

Awesome!


Support software rendering

Right now we rely solely on GPU for rendering. This is great for performance but we won't be able to run it on machine that doesn't have GPU, like for example, a Github Actions machine! If we don't have that supported then, we can't have automatic visual testing. https://github.com/ZeroX-DG/moon/issues/57

Might consider this crate https://github.com/sinclairzx81/black

Or this: https://github.com/RazrFalcon/tiny-skia

Fixed in https://github.com/ZeroX-DG/moon/issues/65


SOAP urls parsed incorrectly

Heya,
There's an issue in the latest version of webmock.
All WSDL urls for SOAP services get appended with an WSDL=true.
I had to revert webmock to gem 'webmock', '~> 1.17.4' to get it to work again.

@NekoNova WebMock is not appending this on it's own. Perhaps one of the libs has changed. Do you see any other gems upgraded in Gemfile.lock when you upgraded WebMock to the latest version?

not really. I first placed the blame inside VCR and Faraday, but after digging deeper, I came with the Stacktraces down to Webmock.
We only use Webmock and Faraday in our RSpec, I can try checking again in the pull-request, but as far as I know we have no dependencies.

only gem I saw change on the revert was addressable

and webmock 1.18.0 also causes problems?

let me try to switch to 1.18.0, give me a second

it doesn't let me install that one.
It's either the 1.17 or the 1.21 version for some reason.
Also it seems when addressable swaps from 2.2 to 2.3 it breaks with VCR as the bug then occurs.

it's worth checking where the WSDL=true comes from. not from WebMock, I don't think from addressable either. Perhaps from your code or SOAP lib you use?

webmock 1.18.0 works sorry. this is in combination with addressable 2.3.8.
For Soap we use the latest Savon which has not been changed in a while.
This thing is.....weird to say the least

I think it may come from Faraday and that https://github.com/lostisland/faraday/issues/536 is legit.

let's continue the discusssion there then :)


Safening of partial functions

Started on a safening transformation for partial functions.
Largely based on

Frisch, Alan M., and Peter J. Stuckey. "The proper treatment of undefinedness in constraint languages." International Conference on Principles and Practice of Constraint Programming. Berlin, Heidelberg: Springer Berlin Heidelberg, 2009.

but some trickery was required for div as we do not allow domains with a hole in it.
For now, this is quite dumb: no care is taken to the context of an expression, nor whether it is top-level.
For example, arr[x] == 2 will be safened and thus introduce a bunch of extra variables/constraints.
I think it's reasonable to leave this expression "unsafe" and just let the solver to its thing with it (TODO).
BUT, in the case of division it's more tricky, as the actual API of e.g., OR-Tools does not accept 0 in the denominator's domain.
So for that one, we actually do need safening at toplevel... (hence I did it this way for now)
A solution to this is to also introduce a supported list for this transformation but it feels quite hacky...
This PR only includes safening for div and Element, but the same can be implemented for mod I believe.

@IgnaceBleukx  @Wout4
I've rewritten the function documentation to structurally describe why, what and how we do this.
Please revise just the text in the comment, see if we can get that right.
(small clean-ups/renamings/reorderings to the code will follow from getting the description right)

I wrote out the 'why' part a bit more explicitly.
Following the text, I think we should not 'safen' top-level expressions. If solvers like ortools want to be picky and not even allow a '0' in the domain for div, then within the solver interface we should detect that and call the relevant safen function there (or safen that specific constraint there knowing that the nearest boolean context is true)?
All other solvers will simply implement strict semantics and propagate the unsafe value away, correct?

I restructured the code following the explanation in comments.
There is one mismatch... in the explanation, I wrote about a 'guard'; however, in the safen functions, we don't really introduce a guard/guard conditions... its more like a flag, and we use that flag for multiple things.
e.g. in safen_range, we write ((unsafe_arg >= lb) & (unsafe_arg <= ub)).implies(is_safe), so implication so not the definition of a 'guard' more a 'condition'... is this on purpose? should we rewrite the explanation? or the code? (e.g. the is_safe == (safe_arg == unsafe_arg) part?)

The alternative would be:
    toplevel = [is_safe == ((safe_lb <= unsafe_arg) & (unsafe_arg <= safe_ub)),  # the guard
                       is_safe.implies(safe_arg == unsafe_arg),
                       # avoid additional solutions when new var is unconstrained (should always work because newly defined?)
                       (~is_safe).implies(safe_arg == safe_lb)]

which seems intuitive, but some (even toplevel) tests are failing with this.
(some tests were already failing before too)

I went through the Frisch and Stuckey paper again. Ignace implementation followed that paper's rewrite faithfully. But they didn't provide much of a constructive explanation of why this is the right rewrite.
I now rewrote the explanation at the top and the implementation with an (imho) intuitive explanation of the rewrite rule, also giving all the newly introduced variables a semantic name. It differs slightly from the rewrite of Frisch and Stuckey but should be logically equivalent.
Please proofread.
I also added an intuitive example and explanation of the side-effects of this transformation, that b <-> ~(partial==5) and `b <-> (partial!=5) mean different things. Wrt impact on us I think this exposes one thing: in the fuzztester, we are not allowed to do semantic fusion on partial functions... because it changes the semantics (e.g. if we fuze a toplevel expression with a partial expression used in a reified context, we are changing the semantics of both...)
@Wout4 do you agree? That means we either need to make sure our 'seeds' are totalized first; or we need to check in semantic fusion that the selected numerical exprs are not partial...
Also, I think the implementation for toplevel partials was incomplete, nbc was not added to toplevel. However,  I believe we should NOT do the rewrite for toplevel exprs, as for that case its the strict semantics anyway (unless all solvers don't accept 0 in div/mod like ortools does? seems to be more its design choice?)

good point on the fuzzing, I think we can do a check to only fuse safe expressions

Took me a while to get the more elaborate code of safen_hole. In my version the Boolean variables were not created explicitly but logically it seems equivalent and probably avoids generation of extra variables downstream.
(Side thought:  when we do CSE, we should also test whether the negated expr already exists in the store!)
Toplevelness
I modified the behaviour of the transformation to act differently when toplevel. Essentially, we don't safen whenever the neirest Boolean context is toplevel, except when asked for it explicitly in the function args.
This was needed for both OR-Tools (due to the API) and Z3 (due to underspecification of div)
https://microsoft.github.io/z3guide/docs/theories/Arithmetic/#division
so -1 / 0 == 1 would have been allowed...
Fuzz testing
Regarding the semantic fusion in our fuzz-tester, we certainly cannot fuse unsafe expressions...
Assume we have two constraints b == (x / 0 == 3) and max(p,q,r) == 10 and we fuse x / 0 and pto a new variablezwithz = (x / 0) + p`. Then the resulting constraints will become:
b == ((z - p) / 0)  == 3 # fine
max(z - (x / 0), q,r) == 10) # NOT FINE

So the model will always be UNSAT... So here we are changing the Boolean context of the unsafe expression...

@IgnaceBleukx What do you think of the last commit, seems a bit silly but I like to support the divide by 0 case.

Yes, indeed great to also support that case. I updated the comments in the code a little bit and made an intvar instead of a boolvar.
Actually, the TODO you added already happens automatically when the _nbc gets merged to the neirest Boolean expression. As we do expr = some_boolexpr & BoolVal(False), this already becomes Boolval(False) :)
Also added an explicit test for the only 0-division case.
Good to merge for me.

Aha perfect! will merge


GULD ERC20 token

Icon for the GULD ERC20 token.
website: https://guld.io
contract address: 0x9847345de8b614c956146bbea549336d9c8d26b6
etherscan: https://etherscan.io/token/0x9847345de8b614c956146bbea549336d9c8d26b6

Hi @isysd  please provide additional details per the latest submission guidelines, and re-submit when ready.

[ ] Must have a 'NEUTRAL' reputation or 'OK' reputation on Etherscan.

Thank you for the link and information @estebanmino. We already submitted some information to etherscan, but did not see this detail on their reputation system. We'll update our submission to them, and get back to you when they have approved.

@ferdreyfus @riomyers note the requirements of both etherscan (link) and metamask.


[Regression] - V2 - CLA Not Signed - Admin names are not displayed in Contact Company Admin acknowledgement

Observed that admin names are not displayed in Contact Company Admin acknowledgement. I tried for Blackberry company and that company has admin

Verified that the issue is fixed


Why is This happening?

What's the problem you are having while you are using C-REAL?
Have you checked the errors in the console?
Yes
this is what I get :
Enter token. Note: Whatever you entered here will not be displayed.

Enter your discord tag or user ID. It is recommended to use discord user ID because same unicode names are hard for the code to check.

811297773****7507

Tips:
The default command_prefix is: .
Your currect command_prefix is: .
Use .config to config the settings and more info about how to config.
Traceback (most recent call last):
File "C:\Users%user%\Desktop\NUKIN\c-realV2.py", line 238, in
checkToken()
File "C:\Users%user%\Desktop\NUKIN\c-realV2.py", line 218, in checkToken
if not 'id' in requests.get(url='https://discord.com/api/v8/users/@me', timeout=timeout, headers=headers).json():
File "C:\Users%user%\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\requests\models.py", line 900, in json
return complexjson.loads(self.text, **kwargs)
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1264.0_x64__qbz5n2kfra8p0\lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1264.0_x64__qbz5n2kfra8p0\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1264.0_x64__qbz5n2kfra8p0\lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

i can't run it with this error. bot is still offline

I was able to recreate the error you had it looked like in cmd you can't copy and paste stuff. The programming environment that I was using didn't have the problem. I guess I'll remove the "hiding feature for now"

Thanks


[stress testing] Improve UX and sub-command usage of stress deploy scripts

See https://github.com/Azure/azure-sdk-for-net/pull/28320#discussion_r855675487
We should support more fine grained operations in dev usage of the stress scripts. Right now the script is written with the approach that its goal is to build and deploy a stress test to a cluster. However, it would be more useful to take an approach that focuses on automating various operations, like deploy, build, buildImage, debug. For example, we have some custom configuration around building of dockerfiles with external directories. In order to find the correct docker command you have to run the deploy script and copy the command from the log output. It would be better to just provide a sub-command or function like buildImage for this case.
Operations I can think of that we do today, some via the script and some manual:

build (building the stress code, perhaps we should provide a common build script hook for this)
buildImage (build the docker image)
pushImage (push the docker image)
run (run the docker image, this may be trickier or unnecessary since we haven't prioritized local flows given the ease of using k8s)
debug (ssh into the container and/or attach a debugger to a container process? This is probably its own feature/issue)
logs (tail the logs from an active stress test)
show (show all active instances of this test)
login
template (helm template, validate stress config)
deploy (normal operation of the script, login/build/push/helm install)
deployAzure (deploy the bicep template, so that people can run locally against a static set of resources)
uninstall/remove (remove test from cluster)
dashboard (open dashboard links, etc.)
docs (open docs)
useLocalConfig/useRemoteConfig (switch helm repo under the hood for local addons dev)
init (this could use the stress generator, or just create a boilerplate gitignore/dockerfile/templates/chart.yaml/etc.)

This should be considered super long term stretch goal, IMO. If anything the script can just print out the proper command lines for some of these things (ie, dry-run).

A full overhaul of the text UI is a stretch, but I think there are some operations that would be helpful to add today, like image build only, use local addon, and show test.
In my experience, people rarely look at or discover the command line output or would know which ones can be run standalone.


Views, Functions, and Triggers

Are there plans to add Views, Functions, and Triggers to pgdiff?

I don't have plans for Views, Functions, and Triggers mostly because I don't have a need to compare those.  If I had SQL that could be used to generate one or all of these I'd be willing to add some comparisons.

This is on branch 0.9.1 which will be merged soon.


Add table of contents generation function to ReadmeTest

A new method named testGenerateTOC() has been added to the ReadmeTest.php file. This function generates a Table of Contents for the README.md file by parsing section headers. The purpose is to improve usability and navigation within the README file.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.762%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2779

Relevant Lines:
2872

💛  - Coveralls


(FACT-3448) Add Amazon to base fact utility

This commit adds Amazon as a condition to the os_processors_and_kernel_expected_facts method.

I haven't not been able to test this yet as there isn't a passing puppet-agent build that includes Amazon Linux.
Marking this as a draft until there is a passing build I can test against.


alerts for aggregated API metrics

defines alerts for aggregated API as requested in https://bugzilla.redhat.com/show_bug.cgi?id=1772564

will something clean up the new alerts on downgrade? I am asking because the alert I am going to add will likely fire as the previous versions don't have https://github.com/openshift/origin/pull/24496.

requires https://github.com/openshift/origin/pull/24496

will something clean up the new alerts on downgrade? I am asking because the alert I am going to add will likely fire as the previous versions don't have openshift/origin#24496.

backport it.

backport it

how far back? I think up to 4.2

/retest

/retest

/retest

/retest

moved the alerts kubernetes-mixin repo  https://github.com/kubernetes-monitoring/kubernetes-mixin/pull/355

closing as it is replaced by https://github.com/openshift/cluster-monitoring-operator/pull/691
/close


issue #482: reorder options in auto help text

this PR allows to redefine order using a Comparison function.
I choose to create a new ComparableOption struct from the Specification class to not break existing visibility rules (Specification is internal).
To redefine order you then have to provide a Comparison object as a static property of HelpText class before calling AutoBuild :
HelpText.OptionComparison = HelpText.RequiredThenAlphaComparison;
var helpMessage = HelpText.AutoBuild(parseResult,...)

Maybe not the perfect solution but it works well.
PR comes with the order that @tehkyle proposed.
When no comparison is provided before calling AutoBuild then it defaults to the current behavior (declaration order)
Unit test is also provided
Maybe a better and more elegant solution would be to provide a fluent API to configure HelpText AutoBuild.
Something like
HelpText.With(parseResult)
                .OnError((err) => { })
                .OnExample(example) => { })
                .WithOrder((ComparableOption attr1, ComparableOption attr2) => 1)
                .Build();

@maintaners what do you think ? do you prefer I rework it with fluent API or can we stay with it ?

I 've finally moved to a fluent API design as I found static properties are hell.
I kept the existing AutoBuild method not to break existing code.
var message = HelpText.CreateWith(parseResult)
                .WithComparison(HelpText.RequiredThenAlphaComparison)
                .OnError(error => {
                    throw new InvalidOperationException($"help text build failed. {error.ToString()}");
                })
                .OnExample(ex =>
                    {
                        return null;
                    })
                .Build();

Thanks  Olivier Duhart  for your effort.

It's preferred that every feature has its own branch/PR so every PR pass its own test locally and CI server then merged.
Can you move the  Fluent Interface feature to a separate branch/PR and let this PR for Reorder options only and pass CI checks.

@moh-hassan , CI fails on a sensless unit test. So wouldnt it be easier to just remove this unit test and let CI succeed ?

The fail at line 186
It seems that you test the ordering using the new syntax Fluent Interface (FI).
Try to  do your unit test of ordering options using the standard AutoBuild method and Isolate FI.
That is why it's necessary to  move the Fluent Interface feature to other PR with its own unit tests.

Good that you discovered the fail reason.
Test can cover: no-short-name/no-long-name/no-short-and-no-long-name/both-short-and-long-name.

you can benefit from NameText : Gets a formatted text with unified name information.

@moh-hassan concerning unit tests, as order is defined using a custom Comparison function, I simply test with 3 cases :

order by required then alpha longName as asked by @tehkyle in issue.
order by required then alpha shortName : this one for testing comparison overriding
no comparison : this one to that behavior defaults to previous behavior (ordered by declaration order)

More tests are not that usefull I guess

@Olivier
HelpText fluent API shouldn't be part of this PR and I suggest moving it to other PR.

I 've reverted all fluent API code. I 'll come back with the fluent API enhancement later if PR accepted and merged in develop

It looks fine.

You add a new parameter with default null to AutoBuild method. That is working.
To minimize the parameters passed to  AutoBuild method, What about if it's set   as a public property like other setting in HelpText :
public Comparison<ComparableOption> Comparison {get;set;}

Provide test case with some properties as value options one required and the other Value properties not required.

For documentation, Can you provide a wiki page describing how to to use the Order option feature with examples.

@moh-hassan
Regarding your remark on the Comparison value as an instance attribute : I agree there is to much parameters.
The autobuild method build the  HelpText instance so we have to give it the optional comparison object if we want it to be used.
At the end of the method HelpText object is built AND helpmessage computed.
I don't see any simple method to remove parameters still keeping all the possibilities.
that was one of the purpose of my fluent API proposal : make it clearer to build an HelpText and compute message.

All help configuration is supposed to be done using error Action, but this configuration is effective only (due to a bug in HelpText) when the parser fire parsing errors and it's not effective when using "--help".
This bug is resolved in the PR  #467  (it's not merged yet) and now all configuration can be done using error Action
I merged your PR with PR #467 locally and did POC to use an instance variable instead of a parameter and it's working fine.
     var   message = HelpText.AutoBuild(parseResult, error =>
        {
            error.OptionComparison = HelpText.RequiredThenAlphaComparison;
	return error;
        },
        ex =>ex );

I can merge PR #467  to develop.
What are your suggestions?

@moh-hassan I would be easier if you merge #467 to develop, then I will rebase my branch with develop and do the fix according to #467 fix.
It look more appealing than the way I've done it : too many parameters I agree.
Let me know if you agree and do the merge.
thanks
Olivier

It's merged.
Now test case can be:
     [Fact]
    public void AutoBuild_with_ordering2()
    {
        var parser = Parser.Default;
        var parseResult = parser.ParseArguments<Options_HelpText_Ordering_Verb1, Options_HelpText_Ordering_Verb2>(
                new[] { "verb1", "--help" }) //call help
            .WithNotParsed(errors => { ; })
            .WithParsed(args => {; });

       //AutoBuild using instance variable without passing comparison  parameter
        string message = HelpText.AutoBuild(parseResult,
            error =>
			{
				 error.OptionComparison = HelpText.RequiredThenAlphaComparison;
				return error;
			},
            ex => ex);
        string helpMessage = message.ToString();
		var helps = helpMessage.ToNotEmptyLines().Skip(2).ToList<string>();
        List<string> expected = new List<string>()
        {
            "  -a, --alpha      Required.",
            "  -b, --alpha2     Required.",
            "  -c, --bravo",
            "  -d, --charlie",
            "-e, --echo",
            "-f, --foxtrot",
            "--help           Display this help screen.",
            "--version        Display version information.",
            "value pos. 0"
        };
        Assert.Equal(expected.Count, helps.Count);
        int i = 0;
        foreach (var expect in expected)
        {
            Assert.Equal(expect.Trim(), helps[i].Trim());
            i++;
        }
    }

I have trouble with  onerror action. I sync my fork with upstream , I see your commandline#439 merge but it seems it does not call onerror when parse is ok.
I look at the AutoBuild code and indeed if parse is ok it does not call onerror. did something go wrong when merging ? could you check ?

indeed if parse is ok it does not call onerror

That is correct.
onerror is called when parser fire parsing error or call '--help'
You need to modify the test cases and call help as given below ( my example show the use case of help and it is passed)
             var parseResult = parser.ParseArguments<Options_HelpText_Ordering_Verb1,    Options_HelpText_Ordering_Verb2>(
            new[] { "verb1", "--help" }) //call help
        .WithNotParsed(errors => { ; })
        .WithParsed(args => {; });

Note: This is the official way to test help: either by passing args '--help' or parser errors. It's not tested on parser success.
Also, you can use ToNotEmptyLines() method instead of splitting args using \r \n.

ok I didnot see the --help in test case. I correct it right now

@moh-hassan last build seems to have failed on upload failure. I can not restart and it does not seem to relate to my PR. Could you check it and maybe restart the build ?

I think the problem is temporary and it's related to CI  appveyor.
The parameter OptionComparison  in AutoBuild isn't removed here. Can you update the commit fix  and remove the parameter.

@moh-hassan  : code seems ok, maybe you are looking at a stale version.

I looked at your last commit fix (cff9fd1) for file changed  here  and here.
Can you point me where can I look for the last update?

+1
Thanks Olivier Duhart for fixing.

@moh-hassan thank you for merging . Glad to have help a bit this project.


Register v0.2.0

@JuliaRegistrator register

Error while trying to register: "Tag with name v0.1.0 already exists and points to a different commit"

Registered in https://github.com/JuliaRegistries/General/pull/72144


Java: Enable plan validation in the SequentialPlanner

We've been using the experimental java branch with the SequentialPlanner. Sometimes, the LLM will return a plan that uses some example skills we don't implement in our project.
For example, the plan might look like
Steps:
  - CustomSkill.MyCustomSemanticFunction ...
  - Emailer.EmailTo ...

where we have implemented CustomSkill.MyCustomSemanticFunction but not Emailer.EmailTo
It happens infrequently (we have the temperature set to 0 and see it <5% of the time in our tests), so I tried to implement logic in our project to inspect the resulting Plan to check if any of the steps weren't available to the kernel, but the steps on the plan are not accessible.
I assume this will be a common problem; I could see a few different ways to approach it:

Make the steps accessible (even a read-only list or copied list) so that the consumer can do the validation
Add some validation method on the Plan itself for a user to check if it's executable and retry creation if it is not
Perform some validate plan and/or retry creation as part of the plan creation to reduce the likelihood of consumers getting back a plan that can't be executed.

@johnoliver - we have this feature in .net, can you take a look and see if we have this in Java to stop making up plugins that are not there. - @alliscode  can provide more details here also.

We will take a look at implementing this our side, @alliscode is this the correct code to be looking at:
https://github.com/microsoft/semantic-kernel/blob/main/dotnet/src/Planners/Planners.Core/Sequential/SequentialPlanParser.cs#L171

Is this something you'd be open to contributions on if it becomes a high priority for us before it becomes a high priority for you?

@lnavarette Yes we are absolutely open to contributions, feel free to message in the discord if you want to discuss it, I believe the validation model c# side is at: https://github.com/microsoft/semantic-kernel/blob/main/dotnet/src/Planners/Planners.Core/Sequential/SequentialPlanParser.cs#L171 so can replicate that, and if there is any other validation you think we can do let us know.

Closing as we are not planning on implementing planners right now. Instead we will be implementing tool calling for the V1 release line.


Unable to authenticate Bot through Teams

Github issues for C# /JS / Java/ Python should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Sample information

Download code BotBuilder-Samples / samples / csharp_dotnetcore / 18.bot-authentication /
Configure AAD connector at Azure Bot site
Enable Teams Channel
Try to sign in at Teams

Describe the bug
Few days ago it works fine and I can get the 6 digits authentication code and type into Teams to finish the login. However since last week. when sign in process open https://token.botframework.com/api/oauth/signin?signin=xxxxxxxxx to generate the code. the page ask browser to close the page itself. Chrome and the new Edge dev preview version will directly close it without displaying the code. for old version of Edge you can select not to close page to get the auth-code.
The issue is only happening with Teams. in Bot emulator you can still sign in as it does ask for auth-code.
To Reproduce
Steps to reproduce the behavior:

Trigger the login process from Teams.
Click on login button of the card
Waiting the sign in page to be opened at your default browser (better to make edge as your default)
The page will be closed asthmatically.

Expected behavior
you should be a 6 digits code
Screenshots
N/A
Additional context
N/A
[bug]

Were you using the middleware in order to get it to work initially? The middleware would look something like:
// hook up onSend pipeline
turnContext.OnSendActivities(async (ctx, activities, nextSend) =>
{
    foreach (var activity in activities)
    {
        if (activity.ChannelId != "msteams") continue;
        if (activity.Attachments == null) continue;
        if (!activity.Attachments.Any()) continue;
        if (activity.Attachments[0].ContentType != "application/vnd.microsoft.card.signin") continue;
        if (!(activity.Attachments[0].Content is SigninCard card)) continue;
        if (!(card.Buttons is CardAction[] buttons)) continue;
        if (!buttons.Any()) continue;

        // Modify button type to openUrl as signIn is not working in teams
        buttons[0].Type = ActionTypes.OpenUrl;
    }

    // run full pipeline
    return await nextSend().ConfigureAwait(false);
});

There was an update recently that made it so you no longer need the middleware. Instead, follow these steps:

Download the newest sample
Create your Teams Bot in the App Studio Manifest Editor
Under Domains and Permissions, ensure that token.botframework.com has been added as a valid domain.

Optionally, enable Web app Single Sign-On with your appId and https://token.botframework.com/.auth/web/redirect

Click Install and start talking to your bot

If you've done substantial work to your bot and don't want to use a fresh sample, I believe you can just add this to the top of your OnTurnAsync():
if (turnContext?.Activity?.Type == ActivityTypes.Invoke && turnContext.Activity.ChannelId == "msteams")
    await Dialog.Run(turnContext, ConversationState.CreateProperty<DialogState>(nameof(DialogState)), cancellationToken);
else
    await base.OnTurnAsync(turnContext, cancellationToken);

If that doesn't work, you can try using this:
protected override async Task OnUnrecognizedActivityTypeAsync(ITurnContext turnContext, CancellationToken cancellationToken)
{
    if (turnContext?.Activity.Type == ActivityTypes.Invoke)
    {
        await turnContext.SendActivityAsync(
        new Activity()
        {
            Type = ActivityTypesEx.InvokeResponse,
            Value = null
        });
        await Dialog.Run(turnContext, ConversationState.CreateProperty<DialogState>(nameof(DialogState)), cancellationToken);
    }
}

The middleware made it so the cards in Teams use Action.OpenUrl (which no longer works) instead of Action.Signin (which is what every other channel uses).

verified and working. (In my case that is)

Thanks mdrichardson for your reply, The code that I use are implemented through IBot interface so I will have to change my code to use those methods you advised. will download latest Samples then see how I can leverage it, will update back next week.

hi mdrichardson, I have changed my code by following latest samples, it works fine with Bot Emulator, I can sign in successfully and get my token. however when I deploy it to Teams it doesn't work. the Login card is displayed but there is no popup when I click the login button. Can you advise if I missed any code or configuration? Thanks

The only time that I experience it is if I miss this step:

Under Domains and Permissions, ensure that token.botframework.com has been added as a valid domain.

If you're not using App Studio to add your bot to teams, you need to add:
"validDomains": [
    "token.botframework.com"
  ]

to your manifest.json

Michael, I just enabled Teams channel from my Azure Bot so I have no idea where to find the manifest.json and add above validdomains. Can you advise me where I can find it? Thanks

The easiest way is to install App studio in teams.
Go to the manifest editor
Add new app

Add your existing bot ( you will need your ID)

Fill out all other required info. It does not matter what you insert. As long as your bot id is ok.
Save.
Test and distribute. (testing ofc)

Thanks Michael, after follow above steps I can login from Teams now :) Thank you very much for your help. Just one question left, can other users use https://teams.microsoft.com/l/chat/0/0?users=xxxx-xxx to access this bot, or they need to follow up same steps?

You have a few options:

Have your users install the bot from your manifest. You can export it from App Studio
Install the bot for group and channel scopes
Publish your bot so anybody can access/download it

Not sure if I can reply this close case, just want to provide an update, I tested with other user but it doesn't work. As I don't want to Publish my bot so I have built a private authentication page to finish the authentication instead of use Both Framework. just for reference.

I am not using app studio. Is there any way to add valid domain from C# and publish it to azure. So that I access it from teams

@SudhirVeerabhahu 	You have to use either App Studio or a manifest.json. For the manifest, just make sure you have:
"validDomains": [
     "token.botframework.com"
  ]


Update my local dev environment with branch master

Hi ! all
I have a silly question to ask ^^
I've installed since last week and yesterday a new project using quickadmin by Installing the package via composer require laraveldaily/quickadmin
And add watching option to be notified of all conversation.
Then I've seen thats there was updates since 3 days from Fixed issue #15
I'd like to know what is the best way to retrieve the last modification from the master and update my laravel project without having to report each modification.
May be clone from https://github.com/LaravelDaily/quickadmin.git or download zip and simply past quickadmin source and then copy past in vendor/laraveldaily/quickadmin ?
will I need to execute again php artisan quickadmin:install ?
Commits on Mar 1, 2016

@LaravelDaily
Merge pull request #25 from twoSeats/twoSeats-patch-1  …
LaravelDaily committed 18 hours ago
cf8a6aa

@LaravelDaily
Merge pull request #26 from justutiz/master  …
LaravelDaily committed 20 hours ago
376e654

@justutiz
Fixed issue #18
justutiz committed 20 hours ago
e7f8129

Commits on Feb 28, 2016

@twoSeats
Update 2015_10_10_000000_create_menus_table  …
twoSeats committed 3 days ago

Hi @DigitalCoder - thanks for the question. I've just released a new stable version with latest fixes, so you can do composer require easily.

Thx !


[SPARK-24029][core] Set SO_REUSEADDR on listen sockets.

This allows sockets to be bound even if there are sockets
from a previous application that are still pending closure. It
avoids bind issues when, for example, re-starting the SHS.
Don't enable the option on Windows though. The following page
explains some odd behavior that this option can have there:
https://msdn.microsoft.com/en-us/library/windows/desktop/ms740621(v=vs.85).aspx
I intentionally ignored server sockets that always bind to
ephemeral ports, since those don't benefit from this option.

Test build #89602 has started for PR 21110 at commit 7c84e1d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2506/
Test PASSed.

Test build #89602 has finished for PR 21110 at commit 7c84e1d.

This patch fails from timeout after a configured wait of `300m`.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89602/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

From a very quick look, the test failure looks global.

Test build #89611 has started for PR 21110 at commit 7c84e1d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2514/
Test PASSed.

Test build #89611 has finished for PR 21110 at commit 7c84e1d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89611/
Test FAILed.

retest this please

Test build #89667 has started for PR 21110 at commit 7c84e1d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2554/
Test PASSed.

Test build #89667 has finished for PR 21110 at commit 7c84e1d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89667/
Test PASSed.

Merged to master.


Complicated file structure

Modifying the complicated, messy file structure to an organised file structure which will help in better understanding and tracking related files/ folders.

you can assign that task to me under gsso-ext and Hacktoberfest label..

hey @SaanyaGarg5. Good to see that you are interested to take up this issue, but i have already been assigned with this issue and currently working on it.

Hey @07sumit1002. While simplifying the file structure, i had to go through the entire project and change the required file/ image paths in the code. So can you add level2 tag for this? Also can you add hactoberfest and gssoc labels/ tags as well for this issue?
I have created a pull request for this issue - https://github.com/07sumit1002/CabRental/pull/241
Thank you!


net.ltgt.errorprone 0.0.8 not compatible with dagger 2.8

Not sure if it is bug in errorprone or android/errorprone gradle plugin but first reporting here. Combination mentioned in title causes NoSuchMethodError: com.google.common.base.Preconditions.checkState because errorprone configuration still uses guava 19.0.
More details: https://github.com/google/dagger/issues/511#issuecomment-262435064

Note: the net.ltgt.errorprone gradle plugin issue tracker is at https://github.com/tbroyer/gradle-errorprone-plugin; yet I don't think this is a bug with the plugin specifically, but rather with ErrorProne itself; possibly due to this code in JavaC (called via this code), therefore leaking ErrorProne's own class-loader to the annotation processors' class-loader.
Should ErrorProne set the procloader option when delegating to JavaC?

Error Prone should already be installing a processor classloader that avoids leaking the internal version of guava: https://github.com/google/error-prone/blob/390d90ea894e8270f1c9ddc09df20a01feb1647a/check_api/src/main/java/com/google/errorprone/BaseErrorProneCompiler.java#L176
Either that's broken, or there's something else going on here.

Here is example of project which reproduces this issue: https://github.com/koral--/GithubClient
To reproduce run eg. ./gradlew assembleDebug.

@cushon The MaskedClassLoader still delegates to its parent. In the case of a Gradle build, the URLClassLoader returned from MaskedClassLoader#preRegisterFileManager internals delegates to its parent (the MaskedClassLoader), which delegates to its parent (the SelfFirstClassLoader from the net.ltgt.errorprone gradle plugin: https://github.com/tbroyer/gradle-errorprone-plugin/blob/v0.0.8/src/main/java/net/ltgt/gradle/errorprone/ErrorProneCompiler.java#L76) which corresponds to the errorprone Gradle configuration (a single dependency on com.google.errorprone:error_prone_core, which depends on com.google.errorprone:javac that brings the JavaFileManager used to initialized the MaskedClassLoader parent class-loader, which will actually be the same class-loader as the one MaskedClassLoader and the rest of Error Prone are loaded from, so the MaskedClassLoader is effectively bypassed) and thus contains Guava 19.
Should the gradle plugin strategy be changed? or should MaskedClassLoader override loadClass et al. instead? (only exposing java., javax., com.google.errorprone. and org.checkerframework.dataflow., possibly sun. and com.sun. too)

BTW, I can reproduce with Maven too
test-1.0-SNAPSHOT-project.tar.gz

Here's the repro-case with Maven, Gradle, and Bazel builds: https://github.com/tbroyer/error-prone-issue-482
Bazel builds OK, Maven and Gradle exhibit the problem.

FWIW, @koral-- a workaround is to put Guava 20.0 in the Error Prone classpath.
For Gradle:
// Override Guava to workaround issue google/error-prone#482
configurations.errorprone.resolutionStrategy.force 'com.google.guava:guava:20.0'
// Alternatively:
/*
dependencies {
  // This is the default value if not set:
  errorprone 'com.google.errorprone:error_prone_core:latest.release'
  // Override Guava to workaround issue google/error-prone#482
  errorprone 'com.google.guava:guava:20.0'
}
*/

For Maven:
<build>
  <plugins>
    <plugin>
      <groupId>org.apache.maven.plugins</groupId>
      <artifactId>maven-compiler-plugin</artifactId>
      <version>3.6.0</version>
      <configuration>
        <compilerId>javac-with-errorprone</compilerId>
        <forceJavacCompilerUse>true</forceJavacCompilerUse>
        <!-- maven-compiler-plugin defaults to targeting Java 5, but our javac
             only supports >=6 -->
        <source>7</source>
        <target>7</target>
      </configuration>
      <dependencies>
        <dependency>
          <groupId>org.codehaus.plexus</groupId>
          <artifactId>plexus-compiler-javac-errorprone</artifactId>
          <version>2.8.1</version>
        </dependency>
        <!-- override plexus-compiler-javac-errorprone's dependency on
             Error Prone with the latest version -->
        <dependency>
          <groupId>com.google.errorprone</groupId>
          <artifactId>error_prone_core</artifactId>
          <version>2.0.14</version>
        </dependency>
        <!-- Override Guava to workaround issue google/error-prone#482 -->
        <dependency>
          <groupId>com.google.guava</groupId>
          <artifactId>guava</artifactId>
          <version>20.0</version>
        </dependency>
      </dependencies>
    </plugin>
  </plugins>
</build>

For Ant, I suppose one can add Guava before error_prone_ant in the <compilerclasspath> (no idea when simply dropping the error_prone_ant.jar in $ANT_HOME/lib or ~/.ant/lib, are JARs listed in alphabetical order?)

@tbroyer it does not seem to work: https://jitpack.io/com/github/koral--/GithubClient/37a6383f6f/build.log

Ouch! error_prone_core actually bundles its dependencies (but also declares them as runtime dependencies), including Guava 19.0, so the Guava 20.0 has to be forced to appear before Error Prone in the classpath; and because Gradle's dependencies{} doesn't give any guarantee on the ordering, and net.ltgt.errorprone uses Gradle's Configuration type rather than FileCollection (where all it actually needs is an Iterable<File>), I think you were right that there's no workaround for Gradle builds.
I've updated my comment above for Maven and Ant.

I was forgetting that the MaskedClassLoader approach works for Bazel because javac (but not Error Prone) is on the bootclasspath, so JavacFileManager.class.getClassLoader() returns the system classloader. Also that the maven plugin does its own hacky classloading.
So I agree this is broken, but I'm not sure what the best fix is yet.
And we may as well start using guava 20.

May I suggest starting by removing the maven-shade-plugin from error_prone_core (keep it only in error_prone_ant) so that workarounds are easier should that happen again with Guava 21; and if course updating to Guava 20?
Then we can look at fixing the classloading issue.

@tbroyer Thanks for the pointer.

May I suggest starting by removing the maven-shade-plugin from error_prone_core (keep it only in error_prone_ant)

+1. It is very confusing that maven-shade-plugin is used in artifact with name "core". Moreover, other dependencies are not included in core artifact, like dataflow. It's also unclear to me, why JSR305 also included in core error prone artifact? Is it because of auto value dependency? I understand that auto value simplifies the life of developers. OTOH it poisons your class path and in retrospective, it seems too be a  bad idea and counter productive to use it in low level libraries, that would need to provide end exhibit their dependencies.
FWIW given this class path collision in Bazel JUnit test integration library with auto value version used in Gerrit Code Review, auto value usage in Bazel JUnit test integration was abandoned: https://github.com/bazelbuild/bazel/issues/2044.
Bottom line: error prone core artifact shouldn't provide anything except error prone own classes. Shaded artifact should be called error_prone_all_in_one or error_prone_ant or some such.

FWIW, AutoValue has, by design, zero runtime dependency, so it shouldn't “poison your class path”, and the bugs in both ErrorProne and Bazel's TestRunner are that AutoValue is bundled in the uberjars.

I believe this issue can be closed.


ZFlowExecutor - RetryUntil implementation.

@ashprakasan
[info]   Compilation completed in 11.737s.
[error] /home/runner/work/zio-flow/zio-flow/zio-flow/shared/src/main/scala/zio/flow/ZFlowExecutor.scala:67:52: value getOrElse is not a member of Either[zio.flow.Remote[A],A]
[error] /home/runner/work/zio-flow/zio-flow/zio-flow/shared/src/main/scala/zio/flow/ZFlowExecutor.scala:67:52: value getOrElse is not a member of Either[zio.flow.Remote[A],A]
[error]     def eval[A](r: Remote[A]): UIO[A] = UIO(r.eval.getOrElse(throw new IllegalStateException("Bad state.")))
[error]                                                    ^
[error]     def eval[A](r: Remote[A]): UIO[A] = UIO(r.eval.getOrElse(throw new IllegalStateException("Bad state.")))

Not sure what else I can use instead of UIO(r.eval.getOrElse(throw new IllegalStateException("Bad state."))).
UIO(r.eval.right) => Works with scala 2.11 but fails in scala 2.12 and 2.13 as right projections on Either are deprecated.

@ashprakasan You can match on the Either, for example.


[Feature]: Don't show empty TrashCard

Requirements

[X] I did a search to see if there is a similar issue or if a pull request is open.

Is your feature request related to a problem?
No
Describe the solution you'd like
If there is no Trash entry in the calendar for the given time period, the card is still using blank space on the dashboard (without showing any TrashCards). It would be better to have an option to don't show the card, if it is empty. For example, the official entity filter card does have the option: "show_empty: false"
Describe alternatives you've considered
A conditional card would also be possible, but then I would have to work with a template-helper. If I set the TrashCard to show the next 3 days, I would need to create a template-boolean-helper which is getting true if the next event starts within 3 days. But I don't know how this template would look like.
Additional context
No response

I will hide it if nothing is available
During editing the view we need to have a placeholder, otherwise you are not able to modify some settings till a event appears

If you're done with editing it will look like that


Fixed the cpuid example to work with some vcpu_factory interface chan…

…ges.
Signed-off-by: Brendan Kerrigan<EMAIL_ADDRESS>

Works on windows 10 x64. Should be merged.

Note that this will break again once we merge the JSON PR as the module file will have to be updated. We also need to get the other example up to date as well.


ssl support

presently it's not supporting ssl, it is good to have it.

Anyway to get ssl with tls support using psk and psk-identity connection support?


Let the user decide the label type of pie charts

The user can choose the label type from "Category Name", "Value", and "Percentage". The user can also decide whether the labels should be outside or inside the pie.

Coverage decreased (-0.03%) to 80.981% when pulling 8c38413b320dc6839675456a03d7d1d22a3cc267 on x4base:pie_label_type into f43e5f18d58c73b10aea3d7ca603e916bed5ed9e on airbnb:master.

Nice!
On Tue, 26 Jul 2016 06:19 Coveralls<EMAIL_ADDRESS>wrote:

[image: Coverage Status] https://coveralls.io/builds/7161094
Coverage decreased (-0.03%) to 80.981% when pulling 8c38413
https://github.com/airbnb/caravel/commit/8c38413b320dc6839675456a03d7d1d22a3cc267
on x4base:pie_label_type into f43e5f1
https://github.com/airbnb/caravel/commit/f43e5f18d58c73b10aea3d7ca603e916bed5ed9e
on airbnb:master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/airbnb/caravel/pull/819#issuecomment-235165444, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEdFuEhNzG4wwg_C8exQzrG-K5MDNsfcks5qZZjYgaJpZM4JUyrF
.

--

Alan Cruickshank
E<EMAIL_ADDRESS>T: +44<PHONE_NUMBER>

lgtm!


Library not loaded: '/Users/runner/hostedtoolcache/Python/3.10.6/x64/lib/libpython3.10.dylib'

Thanks for your reply @kayhayen
For me, error only happens if I run the app built from Github Actions (macos-latest) in my local machine. I downloaded this build using actions/upload-artifact@v3.
I've tried some configuration and was able to include libpython3.10.dylib in the resulting bundle. However, I get the same error when running the built app.
You can find my builds here:

PR: https://github.com/roniemartinez/nuitka-build/pull/1
Built artifacts/installer: https://github.com/roniemartinez/nuitka-build/actions/runs/3031630183

Error (same as before):
/Applications/myapp.app/Contents/MacOS/myapp ; exit;
dyld[60478]: Library not loaded: '/Users/runner/hostedtoolcache/Python/3.10.6/x64/lib/libpython3.10.dylib'
  Referenced from: '/Applications/myapp.app/Contents/MacOS/myapp'
  Reason: tried: '/Users/runner/hostedtoolcache/Python/3.10.6/x64/lib/libpython3.10.dylib' (no such file), '/usr/local/lib/libpython3.10.dylib' (no such file), '/usr/lib/libpython3.10.dylib' (no such file)
[1]    60478 abort      /Applications/myapp.app/Contents/MacOS/myapp

Bundle contents:
$ ls /Applications/myapp.app/Contents/MacOS | grep dylib
libcrypto.1.1.dylib
libgdbm.6.dylib
libgdbm_compat.4.dylib
libintl.8.dylib
liblzma.5.dylib
libpyside6.abi3.6.3.dylib
libpython3.10.dylib
libshiboken6.abi3.6.3.dylib
libssl.1.1.dylib

It looks like it is still linked to system's libpython3.10.dylib instead of the included file.
Originally posted by @roniemartinez in https://github.com/Nuitka/Nuitka/issues/1771#issuecomment-1242945559

As an aside, how portable are github action Python installations anyway? They might be compiled towards only macos-latest, meaning the binaries are not backportable.

Says Nuitka:

Nuitka:INFO: Created binary that runs on macOS 10.15 (x86_64) or higher.

Not too bad, CPython does 10.9, not sure about Anaconda.

So, this has been fixed in a hotfix release, turns out CPython was broken too after my dependency scanner rewrite. The main Python DLL was not updated.