Inline images do not render

Demonstrated by https://feedyour.email/posts/kay462mibxlmfny9ir2hgtyq
From the source, looks like there's a MIME attachment reference that obviously does not translate directly to web rendering <img width="434" alt="PastedGraphic-1.png" src="cid:0C7C76F4-F224-4BB1-B3D5-2A92A1CCF277">.
Slightly cranky about how this implies that every newsletter I've ever read (via FYE, at least) exclusively links to offsite images (and the tracking implications of such).

Thinking out loud, fixing this is probably as easy as adding a mime library dependency, pulling out the attachments and then base 64 encoding them into the relevant src="data:… attributes

I mean, we could definitely add a processing step that pulls in external images and stores them ourselves and replaces the URLs! But then we are a binary asset host, so we would needs to limit possible abuse vectors etc. For whatever it's worth I do have an S3-alike account we could use for this, with a bunch of spare capacity.

ugh actually we should probably add a trailing / to the post view and support requests to relative URL's below it via MIME lookup since that would also fix this same problem for linked attachments that are not rendered in line? action item: send a test email that includes a non-image attachment and see if it breaks any different.

the code to drop attachments into data URLs was short, and now we have no assets or files to manage: https://github.com/indirect/feedyour.email/pull/512/commits/997437f3179e5ce5c02c389b2ded4e4d3a495fed

Hm, the demonstration link still doesn't seem to render correctly (in the feed nor on the web)

We do not and cannot ever render the demonstration link correctly, sorry. :( We have not been saving the full raw encoded email body with attachment bodies, only the HTML and plaintext parts. If you send a new email with attachments to a feed, they should get rendered inline now, though.


Can't Install

Couldn't complete installation. Requested files and 2 screenshots attached.
Good luck!
inspector.log
init.log

same issue as #185


Problem with configure SSL websocket connection

Hi, I've a problem with ssl websocket connection, I already read this:

https://github.com/GeniusesOfSymfony/WebSocketBundle/blob/master/Resources/docs/Ssl.md
https://github.com/GeniusesOfSymfony/WebSocketBundle/issues/64
https://github.com/ratchetphp/Ratchet/issues/161

But, I can't understand how figure it out! :S
I had a http web socket connection and it worked fine, but now, I've to work with https, i already have configured the ssl certificates in my domain, but i can't do work the wss connection.
my configuration is:
config.yml
gos_web_socket:
    pushers:
        zmq:
            default: true
            host: ip_server
            port: 55555
            persistent: true
            protocol: tcp
    server:
        port: 62108
        host: ip_server
        router:
            resources:
                -<EMAIL_ADDRESS>
I installed stunnel, with this configuration:
/etc/stunnel/myconfig.conf
cert = /etc/ssl/project_folder/certificate.crt
key = /etc/ssl/project_folder/certificate.key

chroot = /var/run/stunnel4/
pid = /var/run/stunnel4/websocket.pid   // I think this is wrong, but I don't know what to put here.
#userId
setuid = nobody

#group id
#groupid = nobody

[websockets]
accept = 8443
connect = 62108

The web page dose this:
var websocket = WS.connect("wss://ip_server:8443");

In the console i can see this:
WebSocket connection to 'wss://ip_server:8443/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED

Can someone tell what I doing wrong ? :(
(I'm using apache server)

Nobody?
@ProPheT777
@matthieuy

From my side nope sorry, your configuration looks good to me

I had a same  problem  and  I changed the  configuration, I post it when I can (problaly this weekend)
Send from my  phone
Matthieu

Have you some log or something in order to figure out? That can be very
helpful :)
On Fri, 8 Apr 2016 12:53 pm Matthieu<EMAIL_ADDRESS>wrote:

I had a same problem and I changed the configuration, I post it when I can
(problaly this weekend)
Send from my phone
Matthieu
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/GeniusesOfSymfony/WebSocketBundle/issues/135#issuecomment-207374980

I can't find out the log file! :(
Do you know where is it?

for nginx depends of your conf, check inside /var/log/nginx_err.log (or something like that, for the name of log file)

I use something like that in config for log file :
monolog:
    handlers:
        websocket:
            type: stream
            path:  "%kernel.logs_dir%/%kernel.environment%_socket.log"
            level: debug
            channels: ['websocket']
        main:
            type: stream
            path: "%kernel.logs_dir%/%kernel.environment%.log"
            level: debug
            channels: [!event, !websocket]

Do you use an IP or the hostname/domain in your configuration file ?

I've bad news guys! :( for times problems and dead lines, my client decided to use http instead https! :(
In this moment I can't dedicate more time for this feature, but I don't will give up with this!
sorry for the problems! :(

ProPheT777 I'm using a server with nginx try to establish the connection from the client with wss but will not let me. As would the connection should I use stunnel as in the documentation.


500 Error on DataAPI Example

First, thanks for a great set of scripts.  You've made this much, easier.
I have been trying to use the DataAPI example in the Readme.   When I copy and paste your instructions (inserting my own keys) I get the following output:

from hathitrust_api import DataAPI
data_api = DataAPI(your_oauth_key, your_oauth_secret)
ocrtext = data_api.getpageocr('dul1.ark:/13960/t00z82c1q', 14)

HTTPError                                 Traceback (most recent call last)
 in ()
----> 1 ocrtext = data_api.getpageocr('dul1.ark:/13960/t00z82c1q', 14)
/Users/jbauer/Projects/oauth_create/hathitrust-api/hathitrust_api/data_api.pyc in getpageocr(self, doc_id, sequence)
146
147         """
--> 148         return self._makerequest('pageocr', doc_id, sequence=sequence)
149
150
/Users/jbauer/Projects/oauth_create/hathitrust-api/hathitrust_api/data_api.pyc in _makerequest(self, resource, doc_id, sequence, v, json, callback)
79
80         r = self.rsession.get(url, params=params)
---> 81         r.raise_for_status()
82
83         return r.content
/Users/jbauer/envs/oauth_create/lib/python2.7/site-packages/requests/models.pyc in raise_for_status(self)
771
772         if http_error_msg:
--> 773             raise HTTPError(http_error_msg, response=self)
774
775     def close(self):
HTTPError: 500 Server Error: Internal Server Error
I've tried inputting my keys as plain strings and unicode, and I also got new keys incase I had copied them incorrectly.  I also installed requests-oauthlib from source rather than using the PYPI version.
Any suggestions are most welcome!

@sckott I haven't had a chance to dig into it. From a quick look at the version 2 docs, it seems like the authentication should work with what is here. The data endpoints, however, need to be rewritten to match the new url scheme.

@toddgaron just made some changes which should get the data API wrapper back into working order.


plugin not working?

Do not see any go language server feature described in here(https://github.com/golang/tools/tree/master/gopls). Is the plugin really working?
version info:
lapce version: 0.2.4
lapce go version: v2022.11.10
go version: go1.19.3 darwin/arm64
gopls version: v0.10.1

I'm not sure where crab21 was seeing that, but in my case, to get the plugin to work I had to explicitly point to my gopls installation under the plugin's settings.

I get this when installing/reloading the plugin:

lapce version: 0.2.5
lapce go version: v2022.11.10
go version: go1.19.3 windows/amd64
gopls version: v0.10.1

It's most likely due to missing GOROOT/GOBIN


Update README.md

Cleans up the indenting for the file tree.
Fixes a few broken links.

Whoops.. Seems I sent this PR just a few minutes after you'd made some changes to the file already. My bad. '^.^
Anyway, the changes are very small (main reason I sent the PR at all is the broken links, which is just as few lines), and the PR itself, even if not mergeable, serves to convey the information, so no need to recreate or anything, I think; Can just implement the changes if wanted whenever you next update the README.
Will leave it open for now in case it doesn't get seen due to being closed and you can just close it when you see it.

Ah cheers @Maikuolan ! There's definitely issues like dummy text in the menu.
Will take a look at your suggestions shortly as I plan to add more info for running the server and how the game rules folder works


Add Celery configuration

Would be nice if the cookiecutter ask the user if Celery is needed in the project

hello, i think i could take this issue

Hello @jairoufps, go for it!
Remember that you would need to change:

Docker compose file.
Production docker compose.
Integration with Django to check scheduled tasks.
If you want more background on how to implement this feel free to send me a PM on discord or just add a comment in this issue.

Have a nice day,
Adrián.


Failing to start after tab issues / attempting to install hyper-sync-settings

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 7
Hyper.app version: 1.0.0 & 1.1.0
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/DeeZeeThreezy/1f0b9216537c7e9be35dd06aefc7d615
Relevant information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app: Idk

Issue
After following the instructions copied below from hyper-sync-settings, I attempted to close and restart Hyper.
The Windows taskbar icon for Terminal highlights for a bit, then disappears. When I open up Task Manager, I see the amount of Hyper.exe processes grow as I click to start Hyper.
Even after 5-10mins, it never opens up.
Something interesting, though, is that 4 Hyper.exe processes start up with just 1 attempt to start Hyper from scratch. I wonder if this has to do with tab startup, as I had unsuccessfully tried to open up a few tabs while it was executing a long task at the start of the day. The restart I had tried above was the first restart I had done since this tab situation.
I feel like either...

Something went wrong with the hyper-sync-settings install
Or the failed tabs aren't quite alive enough for Hyper to start

These are the logs I get from SquirrelSetup.log...
ï»¿2017-01-23 13:14:14> Program: Starting Squirrel Updater: --createShortcut=Hyper.exe
2017-01-23 13:14:14> ApplyReleasesImpl: About to create shortcuts for Hyper.exe, rootAppDir C:\Users\domipe\AppData\Local\hyper
2017-01-23 13:14:14> ApplyReleasesImpl: Creating shortcut for Hyper.exe => C:\Users\domipe\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Zeit, Inc.\Hyper.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: About to save shortcut: C:\Users\domipe\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Zeit, Inc.\Hyper.lnk (target C:\Users\domipe\AppData\Local\hyper\Update.exe, workingDir C:\Users\domipe\AppData\Local\hyper\app-1.1.0, args --processStart "Hyper.exe")
2017-01-23 13:14:15> ApplyReleasesImpl: Creating shortcut for Hyper.exe => C:\Users\domipe\Desktop\Hyper.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: About to save shortcut: C:\Users\domipe\Desktop\Hyper.lnk (target C:\Users\domipe\AppData\Local\hyper\Update.exe, workingDir C:\Users\domipe\AppData\Local\hyper\app-1.1.0, args --processStart "Hyper.exe")
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: Atom.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: ConEmu (x64).lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: HipChat.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: JetBrains DataGrip 2016.3.1.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: Microsoft Outlook 2010.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: SQL Server 2014 Management Studio.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: trace32.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: Visual Studio 2015.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: Visual Studio Code.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: Willis Towers Watson - Chrome.lnk
2017-01-23 13:14:15> ApplyReleasesImpl: Examining Pin: Windows Explorer.lnk
2017-01-23 13:15:32> Program: Starting Squirrel Updater: --processStart Hyper.exe
2017-01-23 13:15:32> Program: Want to launch 'C:\Users\domipe\AppData\Local\hyper\app-1.1.0\Hyper.exe'
2017-01-23 13:15:32> Program: About to launch: 'C:\Users\domipe\AppData\Local\hyper\app-1.1.0\Hyper.exe':
2017-01-23 14:06:02> Program: Starting Squirrel Updater: --processStart Hyper.exe
2017-01-23 14:06:02> Program: Want to launch 'C:\Users\domipe\AppData\Local\hyper\app-1.1.0\Hyper.exe'
2017-01-23 14:06:02> Program: About to launch: 'C:\Users\domipe\AppData\Local\hyper\app-1.1.0\Hyper.exe':

If somebody can tell me where any other logs are, I'll happily post it.

hyper-sync-settings instructions

Create a new personal access token which has the gist scope. Save your token to personalAccessToken inside ~/.hyper_plugins/.hyper-sync-settings.json. Alternatively, set the HYPER_SYNC_SETTINGS_PERSONAL_ACCESS_TOKEN environmental variable using this token.

Create a new gist and save it. Save your gist id (last part of the url after the username) to gistId inside ~/.hyper_plugins/.hyper-sync-settings.json. Alternatively, set the HYPER_SYNC_SETTINGS_GIST_ID environmental variable using this id.

Restart Hyper.

Looks to be due to  .hyper.js having hpm-cli in it. Makes sense not to have it in there in hindsight.
plugins: ['hpm-cli']

So, removing it allowed me to open and use Hyper again.
However, I feel this is still a problem for Hyper. It should fail gracefully. Let me know it didn't like the plugin. At the very least, allow me to use Hyper without the plugin.

Hey there, please try again with the new Hyper v2 release and open a new issue if the problem stills exists. Thank you!


Disengage lateral control gradually over 1 second when brake is pressed on a curved road to allow for the driver to grab the handle

Scenario

Driver is paying attention and is ready to grab the handle and press the brake in case of an emergency.
Car is driving on a freeway on a road that's curved to the right with OP engaged.
Car in front of of the driver's car suddenly brakes hard.
Driver press the brake and grabs the handle. OP disengages both long and lat controls.

Behavior with room for improvements
The driver will instinctively press the brake and grab the handle at the same time. However, pressing the brake and grabbing the handle doesn't happen at the same exact time.

If the driver grabs the handle first and then presses the brake a split second later, the car's lateral trajectory is unchanged (since the handle is grabbed while turned) while the car decelerates.
If the driver presses the brake and then grabs the handle a split second later, the car's lateral trajectory is changed because OP released the lateral control while driving on a curved road.

Two problems

The car will swing to the left for a split second and then the driver will correct it back to right. This creates a swerve which makes the emergency situation more dangerous.
The handle will start to turn CCW (because the car wants to go straight) which makes it more difficult to grab. A stationary handle is easier to grab than a spinning handle.

Suggestion
When brake is pressed while going on a curved road,

Disengage long control immediately.
Disengage lat control if the handle is grabbed.
Release lat control slowly over 1 second, as opposed to immediately disengaging.
If the handle is grabbed before the end of 1 second, immediately disengage as soon as the handle is grabbed.

Rationale

This gradual release gives the driver 1 extra second to grab the handle before the lateral control is completely disengaged.
The handle will be turning slower due to the gradual release, making it easier to grab.
There are no situations where it is safer for OP to disengage lateral control immediately while braking on a curved road.
If the driver wishes to turn the handle to avoid running into the car ahead, OP would disengage as soon as the handle is grabbed and thus the driver's action is not impeded. Remember that this is after the brake is pressed - long control is already disengaged and the handle grab disengages only the 1-second delay.
This is a very short action in a specific situation, and thus drivers wouldn't even notice that there was a change.
Lateral control is fully disengaged after 1 second anyway. This is not a version of Always-On-Lateral feature.

Alternatives

The gradual release can be engaged only when the brake is fully pressed (to signal emergency).
It can also be engaged above a certain speed beyond a certain angle.
This behavior can be optional.

We'd be open to checking out a PR that implements this, but I'm going to close since it's not on our immediate roadmap.

FYI, this can be implemented for many Hyundai vehicles without any panda safety changes:
https://github.com/commaai/openpilot/pull/20511/files

@gregjhogan Nice, it's the same idea. But it seems like Comma is (was?) against idea. I think it's a good idea, but all I can do is to bring it up again.


[docs] P-tuning for sequence classification

Moves the p-tuning for sequence classification notebook into the docs.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Problem when tracing back orginal genomes

Hi, maybe I missed something but I followed the guide for indexing GTDB and searched the index against my genes of interest. However, when I tried to find the taxnonomy information of the hit genomes, apparently I could not find them through grepping the assembly_summary.text. For example, in the output I got there are some ID such as CAKQHN010000007.1 and CASJNN010000412.1 which were not found in the file, or even when I searched on the GTDB website.
Could you be able to help me with the issue?
Thanks a lot.

CAKQHN010000007.1 should be the sequence ID, you need to use the column (genome ID) before it to search :)

https://bioinf.shenwei.me/LexicMap/tutorials/search/#output-format
1.  query,    Query sequence ID.
2.  qlen,     Query sequence length.
3.  hits,     Number of subject genomes.
4.  sgenome,  Subject genome ID.
5.  sseqid,   Subject sequence ID.


meme: 2017-09-10 -> 2020-05-28

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Can you mention in the commit that you changed it to buildGoModule?


Hard-coded "test" view in package index.js is a problem

PREFACE: Thanks many times over for creating this NPM package!
I had a bit of head-scratcher setting up my first Node.js app that uses this package.  The first test I ran was trying to access my Node.js app from a browser.  No matter what URL I tried I got the following error dump:
Failed to lookup view "test" in views directory "views"
Error: Failed to lookup view "test" in views directory "views"
at Function.render node_modules/express/lib/application.js:580:17)
at ServerResponse.render node_modules/express/lib/response.js:971:7)
at node_modules/alexa-app/index.js:841:15
at Layer.handle [as handle_request] node_modules/express/lib/router/layer.js:95:5)
at next node_modules/express/lib/router/route.js:137:13)
at Route.dispatch node_modules/express/lib/router/route.js:112:3)
at Layer.handle [as handle_request] node_modules/express/lib/router/layer.js:95:5)
at node_modules/express/lib/router/index.js:281:22
at Function.process_params node_modules/express/lib/router/index.js:335:12)
at next node_modules/express/lib/router/index.js:275:10)
at expressInit node_modules/express/lib/middleware/init.js:40:5)
at Layer.handle [as handle_request] node_modules/express/lib/router/layer.js:95:5)
at trim_prefix node_modules/express/lib/router/index.js:317:13)
at node_modules/express/lib/router/index.js:284:7
at Function.process_params node_modules/express/lib/router/index.js:335:12)
at next node_modules/express/lib/router/index.js:275:10)

This was confusing because it only happened when I initialized the alexa-app object and nowhere in my code is a path to a route or view named "test".  This happens due to the code below found in the package index.js module, since this code fires when it's not a Alexa request and only when the debug option is used (i.e. - no schema or utterance is defined).
if (options.debug) {
  target.get(endpoint, function(req, res) {
    var schemaName = req.query['schemaType'] || 'intent';
    var schema = self.schemas[schemaName] || function() {};
    if (typeof req.query['schema'] != "undefined") {
      res.set('Content-Type', 'text/plain').send(schema());
    } else if (typeof req.query['utterances'] != "undefined") {
      res.set('Content-Type', 'text/plain').send(self.utterances());
    } else {
      res.render("test", {
        "app": self,
        "schema": schema(),
        "utterances": self.utterances()
      });
    }
  });
}

I realized after the fact that this is what you mean in the sample code comments about being able to respond to POST requests:
// now POST calls to /test in express will be handled by the app.request() function
But I think this needs a little more explanation since a typical Node.JS developer will be used to the usual set up of a route and a corresponding view in app.js.  Perhaps this text would be better?:
/**
With the above Alexa app options you will automatically be provided with a route that responds to POST requests to the "/test" document.  However, if you use a typical view engine like Jade/Pug for example, you will need to set up a view named "test" to render the request or you will get an error from Express complaining about a missing view for "/test".  For example, "test.jade" if you're view engine is Jade.
*/
Also, it would be helpful to give some people suggestions as what to put in the "view".
I think a better approach would be to change the Alexa app constructor to something like the following:
var alexa_app = require("alexa-app");
var alexaApp = new alexa_app.app(appName, testRouteName);

And explain to people that testRouteName is an optional parameter with the default name "test".  Then modify the code in the package index.js. file to use that route name when setting up the route and to call render() with that route name; with the default value for testRouteName being "test" if testRouteName is undefined or NULL.  This will give experienced Node.JS developers a clearer picture of what is going on.
Also, this saves the developer from having to rename any existing view named "test" if they already have one.

This makes sense, try to PR it?


Overhaul

WIP will be rebased do not merge
@fractalcat @oswynb please feel free to comment/review incrementally
borel & borel-core overhaul. Changes:

get rid of MetricGroup and ResourceMeasure: these are ways to group metrics, but the information has been added to the Metric data type, only to be retroactively recovered when processing a Borel request. I'll use a run-time config to allow the user to define supported OpenStack resources and their groupings.
more to come

Looks good. Like the config changes particularly. Can we make the config format support comments?

this example config I whipped up here might or might not be needed, depending on how ceilometer-common turned out.
Regardless I'd like to move any and all hardcoded values into this config. It uses configurator, so there's already comments: # it looks like this


Missing deployment information

The page text provides a link to further information: “ For more deployment samples, see the Web SDK repository Readme.”
Unfortunately due to repo reorg it no longer contains anything useful. Where is this additional content now?

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

ID: 684d1f55-3d8b-9ba6-ee65-e8f84c2b4244
Version Independent ID: 2db330b4-e830-715f-0a15-e406f85e1d3b
Content: Visual Studio publish profiles (.pubxml) for ASP.NET Core app deployment
Content Source: aspnetcore/host-and-deploy/visual-studio-publish-profiles.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@AngelosP can you recommend a new link?

@kenleese Here's a permalink to the README file: https://github.com/dotnet/websdk/blob/0e8c90a4a71c98a0eae4311fa53176843fdf84bf/README.md. @vijayrkn is working on moving this info to the root of the dotnet/websdk repo.

Thank you.
Respectfully, perhaps it would be an improvement to move this useful content into the docs.

I think that's a great idea. It would prevent this issue from occurring again in the future. The content would also get localized in the docs. I'll work with @vijayrkn to make this happen.


Backport #1059 to the 1.20 branch

Description
Backports #1059 to the 1.20 branch via a cherry pick.
Checklist
The following items should be checked before the pull request can be merged.

[ ] Code follows existing style.
[ ] Unit tests added (if applicable).

+1

+1


pbrbasic example incomplete?

There's some code in for rendering a skybox, but no actual draw calls for it.

Yes, that's a left-over. The skybox is only used in the image-based lighting versions of the pbr examples. Will remove the obsolete code, thx for noticing.


How To Automate PAT Generation

We have a Azure Vm Scale Set that adds the vm to the deployment group as it is initiated. We would the like the build process too avoid a UI interface currently the only known way to generate a PAT is via the Webapp. Are there other options?

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

ID: 2bfea574-db69-a5fd-37fe-64ecfdac502a
Version Independent ID: 4037d475-f194-2ac9-ffd6-70530fc3140a
Content: Authenticate access with personal access tokens in Azure DevOps
Content Source: docs/organizations/accounts/use-personal-access-tokens-to-authenticate.md
Product: devops
GitHub Login: @chcomley
Microsoft Alias: chcomley

@JarekToro I'll see if I can find out, looking at the source code for Git Credential Manager for Windows, there seems to be an example of that here:
https://github.com/Microsoft/Git-Credential-Manager-for-Windows/blob/d8b3b320c253cea8594fcc3f3f57d76b0abc1d79/AzureDevOps.Authentication/Src/Authentication.cs#L421
Would that work for you?

Thank you it looks promising.


Vulnerability CVE-2022-37434 from use of madler-zlib 1.2.12

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
https://nvd.nist.gov/vuln/detail/CVE-2022-37434
📝 Steps to reproduce
pip install pytwin
Scan the resulting venv.  The following vulnerability is found from use of madler-zlib 1.2.12
https://nvd.nist.gov/vuln/detail/CVE-2022-37434
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
N/A
🐍 Which Python version are you using?
3.10
📦 Installed packages
numpy==1.26.1
pandas==2.1.1
python-dateutil==2.8.2
pytwin==0.5.0
pytz==2023.3.post1
pywin32==306
six==1.16.0
tzdata==2023.3

Hi @landon-kanner thanks for leveraging this vulnerability issue. Could you please recommend a way to fix it or someone at Ansys that could help us doing so?
(@chrpetre FYI)

Hi @lboucin.  Please work with @MaxJPRey to remove zlib 1.2.12 from your codebase or document a Compensating Control.  I would recommend switching to zlib <IP_ADDRESS>


Veeam- Czechia and USA are not recognized as countries

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/src/veeam/index.html
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in browser
Click on 'Run scraper' button
Check jobs with location in Czechia and USA

EXPECTED RESULTS:
Czechia and USA should be recognized as countries
ACTUAL RESULTS:
'Czechia is not a country' and 'USA is not a country' is displayed for jobs with location in Czechia and USA

Fixed


gRPC currently insane at head of v1.0.x

At current HEAD of v1.0.x (10c57a03ef939668dc4e0a4ad17e095), I observe the following:

Clean git pull --recurse-submodules upstream v1.0.x; git submodule update --init; git clean -xdf to get a nice pristine environment.
make test_python passes.
./tools/run_tests/run_tests.py --language=python --newline_on_success passes.
git status shows no changed files.
./tools/run_tests/run_tests.py --language=sanity --newline_on_success fails.
git status shows changes to src/boringssl/err_data.c and third_party/protobuf.
./tools/run_tests/run_tests.py --language=python fails.

Five reactions:

It is surprising to me that any invocation of run_tests.py has the capacity to alter content under source control; I would expect that git status after any run_tests.py invocation would display exactly what it displayed before that invocation.
This problem seems technically independent of Jenkins but practically compounds the pain we're feeling from Jenkins.
This is slowing my getting @thunderboltsid's changes merged; I am going to proceed with my own best judgement and merge what I am confident is safe.
I think this problem is happening exactly as described on master as well but I have not independently verified that suspicion.
This is release blocking and release blocks scare me.

So, I strongly suggest doing --use_docker when running these tests.

Closing for bankruptcy.


Failed to build when webpack alias for asset is an array of paths instead of a single path

See my full repro here: https://github.com/Rush/webdiscus-repro and run npm run reproduce.
Or to reproduce by hand, setup the resolve alias as a list of paths, instead of a single path
resolve: {
  alias: {
    assets: [ assetPath1, assetPath2 ]

and then require the asset from pug
Module not found: Error: Can't resolve '/code/webdiscus-pug-node-example/assets,/code/webdiscus-pug-node-example/assets2/fonts/ibm-plex-sans-v7-latin-regular.woff2' in '/code/webdiscus-pug-node-example/server/views'

Requiring same asset from JS/TS works fine via:
require('assets/fonts/ibm-plex-sans-v7-latin-regular.woff2');

Thanks for adding it to the "in development" milestone. :) I hope I will not have to wait for long as I'm slightly blocked, but not too worry too much :)

hallo @Rush
thank you for very important issue. I have enhanced the resolving of alias for array.
You can see possible usage cases in the fixture and resolve.alias for compile method.

⚠ Limitation for compile method and alias as an array
Usage of a variable in request don't work if alias is an array:
resolve: {
  alias: {
     Images: ['/path_1/', '/path_2/'],
  }
}

- var basename = 'logo';
img(src=require('Images/' + basename + '-c1.jpeg')) <-- does not work

img(src=require('Images/logo.jpeg')) <-- works

This is limitation of webpack compiler self, webpack don't recognize an asset request contained a variable. But the webpack resolver resolve this assets correct. This limitation does not affect the render and html methods.
New version is 1.7.1

Awesome! THe workaround for not using variables in the require expression is simply to prepare utility data structures. Such as:
- const statusIcons = { draft: require('images/status/draft.png'), reject: require('images/status/reject.png'), review: require('images/status/review.png'), approve: require('images/status/review.png') };

Fixed, works great now. Thank you!


[SPARK-42880][DOCS] Update running-on-yarn.md to log4j2 syntax

What changes were proposed in this pull request?
Update log4j1 syntax to log4j2, and use ${sys:spark.yarn.app.container.log.dir} to relocate log path.
see https://issues.apache.org/jira/browse/SPARK-42880
Why are the changes needed?
Since Spark3.3 has changed log4j1 to log4j2, some documents should also be updated.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Just doc.

cc @viirya FYI

Yarn NM injects spark.yarn.app.container.log.dir as a system property, so we use ${sys:xxx} to refer it during logging initialization.
https://logging.apache.org/log4j/2.x/manual/lookups.html#system-properties-lookup

Merged to master


fix: fix typo, moustache -> mustache

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jingpiao Chen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you, @ppiao. Could you please sign and accept the Contributor License Agreement, as this is mandatory?


RHMAP-2596 Update fh-sdk to 1.3.3

@wei-lee @edewit I dropped in the dll files from here
https://github.com/feedhenry/fh-dotnet-sdk/tree/master/Dist/lib/wp80

:+1: thanks that is super


trigger a save action

How can I trigger a save action from JS? For example I have loaded some bootstrapData to formbuilder but it should save data after that.

did you figure it out? I have disabled the auto save but want to trigger save manually in specific places.


[Event Request] Codeunit 12 "Gen. Jnl.-Post Line" OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry

Dear ALAppExtensions team,
On behalf of 4PS I would like to request integration event 'OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry' to be changed in procedure 'ApplyCustLedgEntry' in codeunit 12 "Gen. Jnl.-Post Line".
    procedure ApplyCustLedgEntry(var NewCVLedgEntryBuf: Record "CV Ledger Entry Buffer"; var DtldCVLedgEntryBuf: Record "Detailed CV Ledg. Entry Buffer"; GenJnlLine: Record "Gen. Journal Line"; Cust: Record Customer)
    var
        OldCustLedgEntry: Record "Cust. Ledger Entry";
        OldCVLedgEntryBuf: Record "CV Ledger Entry Buffer";
        NewCustLedgEntry: Record "Cust. Ledger Entry";
        NewCVLedgEntryBuf2: Record "CV Ledger Entry Buffer";
        TempOldCustLedgEntry: Record "Cust. Ledger Entry" temporary;
        Completed: Boolean;
        AppliedAmount: Decimal;
        NewRemainingAmtBeforeAppln: Decimal;
        ApplyingDate: Date;
        PmtTolAmtToBeApplied: Decimal;
        AllApplied: Boolean;
        IsAmountToApplyCheckHandled: Boolean;
        ShouldUpdateCalcInterest: Boolean;
    begin
        OnBeforeApplyCustLedgEntry(NewCVLedgEntryBuf, DtldCVLedgEntryBuf, GenJnlLine, Cust, IsAmountToApplyCheckHandled);
        if not IsAmountToApplyCheckHandled then
            if NewCVLedgEntryBuf."Amount to Apply" = 0 then
                exit;

        AllApplied := true;
        if (GenJnlLine."Applies-to Doc. No." = '') and (GenJnlLine."Applies-to ID" = '') and
           not
           ((Cust."Application Method" = Cust."Application Method"::"Apply to Oldest") and
            GenJnlLine."Allow Application")
        then
            exit;

        PmtTolAmtToBeApplied := 0;
        NewRemainingAmtBeforeAppln := NewCVLedgEntryBuf."Remaining Amount";
        NewCVLedgEntryBuf2 := NewCVLedgEntryBuf;

        ApplyingDate := GenJnlLine."Posting Date";

        OnApplyCustLedgEntryOnBeforePrepareTempCustLedgEntry(GenJnlLine, NewCVLedgEntryBuf, DtldCVLedgEntryBuf, NextEntryNo);
        if not PrepareTempCustLedgEntry(GenJnlLine, NewCVLedgEntryBuf, TempOldCustLedgEntry, Cust, ApplyingDate) then
            exit;

        GenJnlLine."Posting Date" := ApplyingDate;
        // Apply the new entry (Payment) to the old entries (Invoices) one at a time
        repeat
            TempOldCustLedgEntry.CalcFields(
              Amount, "Amount (LCY)", "Remaining Amount", "Remaining Amt. (LCY)",
              "Original Amount", "Original Amt. (LCY)");
            TempOldCustLedgEntry.CopyFilter(Positive, OldCVLedgEntryBuf.Positive);
            //OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry(GenJnlLine, OldCVLedgEntryBuf, TempOldCustLedgEntry); //old
            OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry(GenJnlLine, OldCVLedgEntryBuf, TempOldCustLedgEntry, NewCVLedgEntryBuf); //new
            OldCVLedgEntryBuf.CopyFromCustLedgEntry(TempOldCustLedgEntry);

    [IntegrationEvent(false, false)]
    //local procedure OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry(var GenJournalLine: Record "Gen. Journal Line"; var OldCVLedgerEntryBuffer: Record "CV Ledger Entry Buffer"; var TempOldCustLedgEntry: Record "Cust. Ledger Entry") //old
    local procedure OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry(var GenJournalLine: Record "Gen. Journal Line"; var OldCVLedgerEntryBuffer: Record "CV Ledger Entry Buffer"; var TempOldCustLedgEntry: Record "Cust. Ledger Entry"; NewCVLedgEntryBuf: Record "CV Ledger Entry Buffer") //new
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 51064.


chore(main): release xcm-analyser 1.3.4

:robot: I have created a release beep boop
1.3.4 (2024-09-30)
Bug Fixes

Fix ignored ESlint errors 🔧 (495fc06)

Miscellaneous Chores

Perform monthly maintenance check 🛠 (e85ed15)

Continuous Integration

Setup Codecov bundle analysis 📦 (c3e0e53)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

xcm-analyser-v1.3.4
:sunflower:


use clustertasks in pac/tekton dogfood, and add -f as alias to tkn pac

Add -f as alias to file-name
Use ClusterTasks instead of fetching remotely

Codecov Report

Merging #925 (a0747ec) into main (af4732a) will increase coverage by 0.33%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #925      +/-   ##
==========================================
+ Coverage   63.92%   64.25%   +0.33%
==========================================
  Files          79       79
  Lines        5150     5136      -14
==========================================
+ Hits         3292     3300       +8
+ Misses       1546     1525      -21
+ Partials      312      311       -1

Impacted Files
Coverage Δ

pkg/cmd/tknpac/generate/generate.go
43.57% <0.00%> (ø)

pkg/params/clients/clients.go
10.34% <0.00%> (-0.50%)
:arrow_down:

pkg/git/git.go
74.35% <0.00%> (+10.07%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

/retest


Adding activity on collection

I guess hence we prior showing important events like BUYs, we should probably in future show the total amount of events like make offers/bids collection wisely to determine demand on collection trending valuation.
I guess it might worth to show LIST, GIFTs accounting for activity to determine if washtrading is occuring, one more pseudometric to go

@yangwao I see this related to basilisk, is this basilisk specific requirement? seems to me like more general issue

seems to me like more general issue

yes, offers and bids are basilisk specific.

total amount of events like make offers/bids collection wisely to determine demand on collection trending valuation.


CL: Partially overlapping / gap price range test/logic

Closes: #XXX
What is the purpose of the change
Slight adjust to logic to allow for gap positions
Adds gap position pool test and concurrent position pool test

Going to merge this now, can address any further comments in future PRs (wanting to work on swaps now without large branch divergence)


CLOUDP-131035: Add e2e test for "atlas backup exports buckets" commands

Proposed changes
Adds e2e test for "atlas backup exports buckets" command.
Jira ticket: CLOUDP-131035
Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
[x] I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
[x] I have updated test/README.md (if an e2e test has been added)
[x] I have run make fmt and formatted my code

I think we should group all backup tests in one place, not only bucket tests

@Razeer123 I've run the test and looks like it's red

@fmenezes You're right, it's annoying because it worked well. I will fix it, check it properly and will let you know.

@fmenezes You're right, it's annoying because it worked well. I will fix it, check it properly and will let you know.

@Razeer123 just make sure in your last commit you trigger the test so it appears in GitHub checks


Add an initial implementation of an iOS driver with failing tests

cc @kpgalligan

Merging so I can rebase kevins code on this and work forward from there. Feel free to comment on anything and I'll follow up


Stop using delete range+manual compaction in RocksDB based LogDB

This feature in RocksDB is the suggested method to delete a selected range of KVs. Sadly, with recently reported/fixed bugs in RocksDB, I just have the feeling that it is not stable enough to be used in production.
Stop using delete range+manual compaction in RocksDB based LogDB, revert back to the regular delete by key style approach. It might be slower but far better than a risky one slightly faster.

fixed in v2.1.4.


This setting can negatively impact Exchange if not deployed correctly

Perhaps a link to https://support.microsoft.com/en-us/help/4055652/access-checks-fail-because-of-authz-access-denied-error-in-windows-ser
and a note about Exchange for this policy?

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

ID: ef8b6967-0e45-bcdd-6d70-54759d9f7ec8
Version Independent ID: 6651ebc1-04f0-421f-891b-6957ca37da7c
Content: Network access - Restrict clients allowed to make remote calls to SAM
Content Source: windows/security/threat-protection/security-policy-settings/network-access-restrict-clients-allowed-to-make-remote-sam-calls.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@vboyev-MSFT  - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@vboyev-MSFT - From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/c9d9547f0db84b8cf133497f75a9fd2bd7a58ab6. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!


IE8 error SCRIPT5007: Unable to get property 'replace' of undefined or null reference

How to reproduce

Leaflet version I'm using: Leaflet 1.1.0
Browser (with version) I'm using: IE8
OS/Platform (with version) I'm using: window 10

What behaviour I'm expecting and which behaviour I'm seeing
IE8 error SCRIPT5007: Unable to get property 'replace' of undefined or null reference
Minimal example reproducing the issue
var overlayLayers = {

}
L.control.layers(baseLayers, overlayLayers).addTo(map);

fix
leaflet-src.js, line 174:
function template(str, data) {
	if(!str) return;
        ....
}

Hi!
From you very short example code, it looks like you have identified the layers control as the source of this error, but looking at the code for the layers control, it does not call the template function you refer to.
I suspect the problem is really somewhere else, but it's not possible to investigate without a proper, fully running example that reproduces the issue, so I need to ask you to expand the sample code. Ideally, put it on http://playground-leaflet.rhcloud.com/ or a similar service.

Closing for now, happy to re-open when we get more information on the issue!

You didn't write where baseLayers comes from and what's inside there. You probably didn't initialise them correctly.
However... the solution is to provide an actually defined string as argument to tileLayer() (or other way you initialise layers).
Otherwise the template function will try to use it and replace vars on undefined, throwing that error.


<EMAIL_ADDRESS>build failed on the crossplane-runtime v1.16

What happened?

I've updated the crossplane-runtime v1.16 deps for the function-kcl, but it builds failed in CI. See the PR https://github.com/crossplane-contrib/function-kcl/pull/105 and the CI pipeline https://github.com/crossplane-contrib/function-kcl/actions/runs/9106600367/job/25034062200?pr=105
How can we reproduce it?

In the function-kcl repo https://github.com/crossplane-contrib/function-kcl and bump the crossplane-runtime to v1.16, then build the code.
What environment did it happen in?
Crossplane version: v1.16
CI environment: https://github.com/crossplane-contrib/function-kcl/blob/main/.github/workflows/ci.yaml

Same problem here (trying to update crossplane-runtime to use https://github.com/erstcloud/function-app-staging/issues/33:
# github.com/crossplane/function-sdk-go/resource/composed
/home/poe/go/pkg/mod/github.com/crossplane/function-sdk-go@v0.2.0/resource/composed/composed.go:125:27: cannot use &Unstructured{} (value of type *Unstructured) as "github.com/crossplane/crossplane-runtime/pkg/resource".Composed value in variable declaration: *Unstructured does not implement "github.com/crossplane/crossplane-runtime/pkg/resource".Composed (missing method GetObservedGeneration)
# github.com/crossplane/function-sdk-go/resource/composite
/home/poe/go/pkg/mod/github.com/crossplane/function-sdk-go@v0.2.0/resource/composite/composite.go:49:28: cannot use &Unstructured{} (value of type *Unstructured) as "github.com/crossplane/crossplane-runtime/pkg/resource".Composite value in variable declaration: *Unstructured does not implement "github.com/crossplane/crossplane-runtime/pkg/resource".Composite (missing method GetObservedGeneration)


(BUG)The "Delete account" button [TypeError: is not a function ]

Precondition:
The context menu is opened
Steps to reproduce:

Click the "Delete account" button
2)Pay attention to the lack of pop-up with the deleting information and error message in the console

Actual result: The "are you sure?" pop-up isn't displayed on the screen after clicking the "Delete account" button. The [TypeError: is not a function ] error message is displayed in the console
Expected result: The "are you sure?" pop-up  should be displayed on the screen after clicking the "Delete account" button
Video:https://www.screencast.com/t/rTqHxRh2
Checked on DEV V0.24.1-5
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome  83.0.4103.116

@AnastasiiaOdnoshevna unable to reproduce

This issue appears after the changes for #1906. And I already fixed it earlier and the fix is already in master - so it cant be reproduced.

The issue isn't reproduced
User is able to delete the account
Checked on DEV V0.25.0
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome  83.0.4103.116

The "Delete account" button [TypeError: is not a function ] is displayed
<EMAIL_ADDRESS>  Password: 123Qwe123Qwe)
Video: https://www.screencast.com/t/ewgjwRESoG
Checked on DEV V0.25.3
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome  83.0.4103.116

@AnastasiiaOdnoshevna we've lost the changes related to #1906 Then we've re-pushed them manually. So that becuase the bug was appeared again. Now we'll re-push the fix too

@AnastasiiaOdnoshevna fix was restored in the master branch. Now it's deploying, you could re-check in a 10-15 min

fixed User is able to delete the account
Video: https://www.screencast.com/t/7xfYftr7kQV
Checked on DEV V0.25.3
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome 83.0.4103.116


Humidity reading too low

Hello,
I am using a custom board with SHT40-B and STC3X-C + Arduino UNO.
The Humidity sensor is giving me too low value of humidity. In the room, it should be ~50% (we have the evaluation board from Sensirion) meanwhile the sensor is giving only ~15%.
It used to work for several days and for some reasons, the humidity reading stopped working today.
I have also tried to exchange the sensor but still, it gives out the same result.
On the other hand, the temperature values from the SHT41 are correct.
The sensor's Serial Number is: 0xEFD5F11
Please help.
Best,
Thai.

I just exchanged the sensor to SHT45-AD1B. With the same board, same code, same Arduino, the sensor spits out correct value again.
This is really strange, any help here would be really appreciated.
Thanks in advance.

Hi,
A possible issue with low humidity readings, is that the SHT40 sensor got contaminated by the environment. A possible solution is to run the heater pulse for a couple seconds. You have to ensure that the temperature does not go over 120°C.
Please have a look at our applications notes about creep mitigation for more information:
https://sensirion.com/media/documents/A88858C9/629626D4/Application_Note_Creep_Mitigation_SHT4x.pdf

I close this issue since it is not related to the driver code. Please contact Sensirion support directly if you need further assistance.


Release to track track/2.3

Context
Part of https://github.com/canonical/kfp-operators/issues/579 and follow up from https://github.com/canonical/kfp-operators/issues/586
Once we have the tracks and the charms have both manifests and rocks updated, we'll need to push to the new tracks
What needs to get done

Publish all KFP charms to 2.3/stable

Definition of Done

The charms are published to 2.3/stable and ready to be used by the bundle

Duplicate of https://github.com/canonical/kfp-operators/issues/596


kubectl log spits out "Unexpected path for command running" when only a pod name is specified

On a cluster running 0.13.2, I ran kubectl get pods, picked out the name of a pod that only contains a single container, let's call it foo, and then ran kubectl log foo. It prints nothing other than the message "Unexpected path for command running". If I add on the container name (kubectl log foo container), then it prints nothing, presumably because the pod has no logs.
The docs for kubectl log say that the container name is optional if the pod only has one container, so I wouldn't expect this response.

+1, we're having to work around this on guides

@jlowdermilk


Stormblessed: Watch Giveaway

Stormblessed requests the watch of the watch_keyword Giveaway. See the MS search here and the Stack Exchange search here.

Giveaway has been seen in 51 true positives, 4 false positives, and 0 NAAs.

Overall SE search is currently down. However, searching just on SO shows 867 results. Even without expanding that to all of SE, that's will likely result in too many FP. Try to find additional text that makes it more unique to spam.


Cannot specify separate form field name and file name in multipart file upload

First, thanks for this library, it generally does a great job and we use it to good effect in many places and really like the api and the overall attention to detail in its operation.
If I am reading this correctly:
https://github.com/dakrone/clj-http/blob/72f531f00d099f9676c60e2891c12d412b2a3a53/src/clj_http/multipart.clj#L36
When the FileBody object is constructed, its filename is either set to the form field name or is not provided (in which case it defaults to copying the filename of the provided file).
Ideally, I'd like to specify the form field name and the filename separately. In fact, I need to do to appease the server I'm posting to.
Thanks in advance for your time and consideration.

Looking at the code, you should be able to use :part-name for the part name, and :name for the filename to be used for the actual FileBody, does that work for you?

Ah! Neat. I had missed :part-name. That definitely gets me going; thank you!


Be able to closurize constructors

It would be nice to be able to closurize constructors. I find myself writing a de-serialization for DateTime that needs to be of the form
if (isUtc) {
  return new Date.utc(<long parameter list...>);
} else {
  return new Date(<identical long parameter list...>);
}

It's not that big a problem, but it would be nice to be able to write the parameter list once. There are some obvious questions. How would you refer to the default constructor? Could you provide type arguments? Is it an issue that invoking a closure might implicitly be doing a "new"?

I guess https://github.com/gbracha/metaclasses/blob/master/proposal.md is supposed to lead to another solution (as reference for people finding this issue first).

I guess https://github.com/gbracha/metaclasses/blob/master/proposal.md is supposed to lead to another solution (as reference for people finding this issue first).

I think this can be closed.
void main() {
  var constructor = new SomeClass#someConstructor;
  var x = constructor();

  print(dateConstructor(isUtc: true)(2000,1,1));
  print(dateConstructor(isUtc: false)(2000,1,1));
}

Function dateConstructor({bool isUtc}) {
  if(isUtc) {
    return new DateTime#utc;
  } else {
    return new DateTime#;
  }
}

class SomeClass {
  SomeClass.someConstructor() {
    print('constructing...');
  }
}

runs fine, just the analyzer still shows warnings for #.
output:

constructing...
2000-01-01 00:00:00.000Z
2000-01-01 00:00:00.000

  var constructor = new SomeClass#someConstructor;
  var x = constructor();

This doesn't work anymore. At least not in the dartpad and on my project.
Any explanation ?

@rrousselGit that feature was removed a short time after it was added. They'll try to find a better approach sometimes after Dart 2.0

Now that new is optional it changes the way the syntax feels.
Since new is still a keyword we're already not allowed to define a named constructed as new, so ClassName.new tearing off the default constructor shouldn't be breaking, and makes named constructor tearoffs look like static method tearoffs.

so ClassName.new tearing off the default constructor shouldn't be breaking, and makes named constructor tearoffs look like static method tearoffs.

Following that logic, default is probably a better option than new

I have a plan! It's still mostly in my head, but it exists.
It requires allowing the id<typeargs> syntax in more places. When we do that, a lot of things become possible:

Denoting a constructor: List<int>.generate
Denoting a specialized generic function: T id<T>(T x)=> x; var intId = id<int>;
Denoting a specialized generic type: Type listOfInt = List<int>;

I hope we can use the context type to figure out whether something like List<int> or Symbol denotes the type or the default constructor. If the target type is a function type or Function, it's the constructor, otherwise it's the Type. That avoids having to add more syntax that isn't actually related to the constructor.

As a user of the language, when I need to get a constructor as a function, I would very much prefer to put it bluntly:
List<int>.generate as Function;
Map<String, String>.new as Function; // ".new" or ".default" - doesn't matter
//etc.

This kind of expression should be special-cased by a compiler, but it's as intuitive as it gets IMO.

Just want to throw in my $.02:
Is there any particular reason why Date itself can't be used for the default ctor function reference? AFAIK there aren't any situations where there'd be ambiguity between referencing the Date type and Date's default ctor, and it'd mimic the call syntax much more closely, without needing an odd special syntax for it:
if (isUtc) {
  d = Date(...);
} else {
  d = Date.utc(...);
}

// becomes
final ctor = isUtc ? Date : Date.utc;
d = ctor(...);

I'm not at all familiar with how the Dart compiler works internally, but from what I know of compilers in general it's definitely possible to say, "Hey, that typename is being used as an expression, so it has to mean the default constructor".
Am I missing an edge case where there is ambiguity?

It's unlikely that final ctor = isUtc ? Date : Date.utc; can be made to work, mainly because the type system cannot guess what you mean.
The Date expression is evaluated without any type hint, then the result of that is combined with the Date.utc constructor tear-off result, which is likely a function type.
If Date has type Type, then even if it's a callable object, the type of ctor will still be Object.
In Dart 2, class instances are not functions. To make Date useful as both a Type object and function-typed  tear-off of the constructor, the same object would have to both be a function and implement Type. That's very confusing to the type system. We might be able to make Date a Type object that is also callable (but I'm not sure what the run-time type of that would be,)
I think it's more promising to let the meaning of Date depend on the context type. Then you would have to write:
Function ctor = isUtc ? Date : Date.utc;`

to give the type hint needed to distinguish Date-as-a-Type from Date-as-a-constructor.

@lrhn Whoops, I forgot that the type is, itself, an object (i.e. final foo = Date is valid, and gives you the RTTI for Date). Call it a C++ hangover.

Over the past year working with Flutter. I've found several times I would like to refer to a constructor function. What was the problem with the old Someclass#someconstructor reference? Is this issue actually scheduled to be worked on?

This is a nice trick, but it solves only the trivial variant of problem. In the title of the issue, we have
if (isUtc) {
  return new Date.utc(<long parameter list...>);
} else {
  return new Date(<identical long parameter list...>);
}

Not sure which case is more common though. Maybe constructors with identical long parameter lists are rare? If so, spending time on would have very low ROI.

That's one of the reasons why I think spread operator for classes would be lovely.
() => Foo() is fine, but if it has tons of arguments, then it's not anymore.

I assume tear-offs for constructors and getters/setters are still planned.
Closures are only a workaround until it's done.

I think we should also consider the utility in functions like map as another case. For example, let's say you make a Flutter widget FancyText which renders text... fancily. You have an array of strings to put into a Column so they all render in a nice, er, column. Right now you'd have to write:
Column(
  children: textArray.map((s) => FancyText(s))
)

With this, you could write
Column(
  children: textArray.map(FancyText.new)
)

That's obviously a trivial example, but it is a (very nearly) real one that I recently had to use, and in my experience with other languages -- typically more functional ones, though not exclusively -- it's very common. Letting ctors be called like any other function taking arguments and returning a FancyText would be useful.

In similar situations we've moved an SDK issue to the language repository, but in this case https://github.com/dart-lang/language/issues/216 contains a more recent version of the same proposal, so I'll close this one. Please go there for additional discussion!


WIP: Hook to rewrite terms by tactic

This PR proposes a new feature, process_with_tactic t p, that rewrites term p using tactic t.
This feature is very similar to the with_tactic t p hook, and its implementation follows the same structure.
The main difference is that instead of creating a new goal for p, it runs tactic t on p, before replacing it in its original context.
A few questions:

How to ensure that the resulting goal from applying t actually corresponds to a transformed version of the initial p?
How to provide better error reporting when there are more than one goal remaining?

Follow-up question: All goals generated during the traversal of a term are ensured to be proof-irrelevant: https://github.com/FStarLang/FStar/blob/master/src/tactics/FStar.Tactics.Hooks.fs#L270.
Instead of this new process_with feature, could we perform the proof-irrelevance check in by_tactic_interp: https://github.com/FStarLang/FStar/blob/master/src/tactics/FStar.Tactics.Hooks.fs#L82, and return the conjunction of the goals as the simplified term instead of True with disjoint goals? I think this would also simplify the codebase; passing and returning gs would not be needed anymore.

About this:

How to ensure that the resulting goal from applying t actually corresponds to a transformed version of the initial p?

The general style used in the tactic framework for this is to generate a new uvar ?u and present to the tactic the goal p = ?u.
And then after the tactic runs you can replace p in its original context with ?u, since we know that it has to be provably equal to it.

The general style used in the tactic framework for this is to generate a new uvar ?u and present to the tactic the goal p = ?u.
And then after the tactic runs you can replace p in its original context with ?u, since we know that it has to be provably equal to it.

I've seen that in other parts of the tactic framework, but I'm not sure whether with_tactic follows this methodology; it seems that it instead extracts an assertion (with a syntactic marker), and replaces it by true, proving the assertion itself on the side.
Regarding my follow-up question, I implemented a variant of with_tactic that follows the same methodology, but replaces the assertion by the conjunction of the goals left after running the tactic instead of replacing it by true: https://github.com/FStarLang/FStar/commit/d2f461da0f3088a95912d9eef0af67d7a1bac7c4 .

That is indeed the case for with_tactic: Splitting and replacing with true.
I thought what you wanted was a mechanism to ensure that whatever the tactic does to p you want to replace it in place with a term that is equivalent to p. Presenting p = ?u to the tactic and replacing with ?u is a mechanism to do that. But, maybe I misunderstood what you wanted.

This is subsumed by PR #2321, closing


Update Big Number Thumbnail

Update big number thumbnail in selection panel

@kristw @williaster

Codecov Report

Merging #5782 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5782   +/-   ##
=======================================
  Coverage   63.76%   63.76%
=======================================
  Files         364      364
  Lines       23099    23099
  Branches     2572     2572
=======================================
  Hits        14729    14729
  Misses       8355     8355
  Partials       15       15

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 9f2b502...35eda2e. Read the comment docs.

@kristw I'll handle that in another PR lowkey having issues with my dev environment at the moment


fix: Bump catapult on Concourse CI to consume cf-o upgrade

Description

This consumes catapult's changes to upgrade cf-operator too on kubecf-upgrade
target. Which means this may fix the upgrade job.
Motivation and Context

To fix the upgrade job.
How Has This Been Tested?

Deployed several times, and upgraded successfully from 2.4.0 to current master with this catapult commit. I've
With current master I can't reproduce, but in the past, after upgrading cf-operator, the cc-deployment-updater of the scheduler pod gets stuck not ready. On the deployment-updater I get DatabaseDisconnectError (…) Mysql2::Error: WSREP has not yet prepared node for application use.
Nevertheless, this change needs to go in.
Compared catapult's history between the previously pinned hash and this one, no additional changes needed to CI.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code has security implications.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Superseded by https://github.com/cloudfoundry-incubator/kubecf/pull/1340.


bug: restore data by mistake in tidb backend mode

Bug Report
Please answer these questions before submitting your issue. Thanks!

What did you do? If possible, provide a recipe for reproducing the error.
loader data into tidb using lightning

What did you expect to see?

What did you see instead?

Versions of the cluster

TiDB-Lightning version (run tidb-lightning -V):
(paste TiDB-Lightning version here)

TiKV-Importer version (run tikv-importer -V)
(paste TiKV-Importer version here)

TiKV version (run tikv-server -V):
(paste TiKV version here)

TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
(paste TiDB cluster version here)

Other interesting information (system version, hardware config, etc):

Operation logs

Please upload tidb-lightning.log for TiDB-Lightning if possible
Please upload tikv-importer.log from TiKV-Importer if possible
Other interesting logs

Configuration of the cluster and the task

tidb-lightning.toml for TiDB-Lightning if possible
tikv-importer.toml for TiKV-Importer if possible
inventory.ini if deployed by Ansible

Screenshot/exported-PDF of Grafana dashboard or metrics' graph in Prometheus for TiDB-Lightning if possible

log: tidb-lightning.log
data dump files: test_lightning2.tar.gz
config: lightning.toml.txt


Angular 2+

Do you have the type script code for DetectRTC.

Couldn't find typings for DetectRTC. Instead I use following code:
declare const DetectRTC: any; in my custom declarations file.
It stops TypeScript's moaning about unknown types.
Still, the typings would be great to have.


Add FilterChipPicker<>

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
The user needs to select zero or many items in a list.
Problem: no widget allows the user to select multiple items in a list.
Solution: display multiple FilterChip widgets from left to right, top to bottom, and bind the results to a list.
Proposal
Here is a naive sketch implementation. It works, but it will need more work. For example, a simple toString() won't be enough. It will probably need an item builder. How to handle the state is another question. Yet another question is whether it should allow vertical orientation instead of horizontal.

import 'package:flutter/material.dart';

class FilterChipPicker<T> extends StatefulWidget {
  const FilterChipPicker({
    required this.items,
    required this.selectedItems,
    super.key,
    this.itemWidth = 120,
  });

  final List<T> items;
  final List<T> selectedItems;
  final double itemWidth;

  @override
  State<FilterChipPicker<T>> createState() => _FilterChipPickerState<T>();
}

class _FilterChipPickerState<T> extends State<FilterChipPicker<T>> {
  @override
  Widget build(BuildContext context) => Wrap(
        spacing: 8,
        runSpacing: 8,
        children: widget.items
            .map(
              (item) => SizedBox(
                width: widget.itemWidth,
                child: FilterChip(
                  elevation: .5,
                  label: Text(item.toString().split('.').last),
                  selected: widget.selectedItems.contains(item),
                  onSelected: (selected) {
                    setState(() {
                      if (selected) {
                        widget.selectedItems.add(item);
                      } else {
                        widget.selectedItems.remove(item);
                      }
                    });
                  },
                ),
              ),
            )
            .toList(),
      );
}

Good point. Will look into that.

Hey there 👋,
Can I take up this issue?

I'm planning on submitting a PR for this but I'd prefer to hear from the Flutter team that this would be something useful

I'm planning on submitting a PR for this but I'd prefer to hear from the Flutter team that this would be something useful

(Just repeating the discussion from Discord here for completeness:)
I think this is probably best done as a package, rather than in the core framework.

Hi @huycozy
Here is a PR, and a live sample
See PR: #126308
See the live sample here

I'm having trouble figuring out the issue with the Linux analyzer. Does anyone know what is going wrong here?
https://github.com/flutter/flutter/pull/126308/checks?check_run_id=13333338346

Hi @Hixie . Could you please point me to the Discord discussion?

Per the discussion on Discord: ChipSet is a nice little utility however  it's probably best shared as a package or even an API doc example.

Hi @HansMuller. I had very little to do with that conversation. I think someone else submitted a similar widget after I added the original GitHub issue and then started the conversation on Discord.
For future reference, what is the defining difference between a widget that goes into the core library and a widget that should be an external package?
I would have answered that widgets should follow the MD website and in this case, the MD 3 website says this:

Multiple chips should appear together in a group, whereas there should be no more than 3 buttons in a single arrangement.

So, it seems like this addition to the core is a step toward implementing the MD functionality, as the MD website specifies. Although, there are changes I could make to the final implementation if necessary.

For future reference, what is the defining difference between a widget that goes into the core library and a widget that should be an external package?

There's isn't a hard and fast rule about this, it's a judgement call that has to balance the cost of adding and sustaining more core API, vs exposing new utilities in a package or as an API doc example. In this case an example would be the best way to share ChipSet.


Fixing typo in doc

There was (I think) a little number issue in the doc or I don't understand why you're starting both times with 1.?

Ah, yes you're right that's a typo👍
Could you make the change in DOC.md? luasnip.txt is automatically generated from it

Strange, the DOC.md looks fine. It's numbered as expected.

What do you think should I do @L3MON4D3 ?

I'm not at my PC currently, could you test https://github.com/kdheepak/panvimdoc

At my PC now, panvimdoc has similar issues :/
pandoc --metadata=project:LuaSnip --lua-filter scripts/skip-blocks.lua --lua-filter scripts/include-files.lua -t scripts/panvimdoc.lua DOC.md -o doc.txt generates
doc.txt, which doesn't have the correct indices either.

Try adding an empty line after Its parameters are and before the 1.

pandoc --metadata=project:LuaSnip --lua-filter scripts/skip-blocks.lua --lua-filter scripts/include-files.lua -t scripts/panvimdoc.lua DOC.md -o doc.txt

I can't use this command yet due to this error message:
pandoc: /usr/lib/libc.so.6: version `GLIBC_2.34' not found (required by pandoc)

I'll have to wait until this is fixed.

You'll have to p -Syu pandoc, I got the same error :P


pysc2 rendering crash on ubuntu 18.04

Hello here,
When I  started to work on pysc2 game environment, I have encountered the following the issue.
$ python3 -m pysc2.bin.play --map Simple64
pygame 1.9.6
Hello from the pygame community. https://www.pygame.org/contribute.html
I0730 12:07:32.165569<PHONE_NUMBER>80224 sc_process.py:135] Launching SC2: /home/hello/StarCraftII/Versions/Base59877/SC2_x64 -listen <IP_ADDRESS> -port 17730 -dataDir /home/hello/StarCraftII/ -tempDir /tmp/sc-pwcmbv3s/ -eglpath libEGL.so
I0730 12:07:32.169494<PHONE_NUMBER>80224 remote_controller.py:167] Connecting to: ws://<IP_ADDRESS>:17730/sc2api, attempt: 0, running: True
Version: B59877 (SC2.4.0)
Build: Nov 27 2017 21:52:46
Command Line: '"/home/hello/StarCraftII/Versions/Base59877/SC2_x64" -listen <IP_ADDRESS> -port 17730 -dataDir /home/hello/StarCraftII/ -tempDir /tmp/sc-pwcmbv3s/ -eglpath libEGL.so'
Starting up...
Startup Phase 1 complete
I0730 12:07:33.173148<PHONE_NUMBER>80224 remote_controller.py:167] Connecting to: ws://<IP_ADDRESS>:17730/sc2api, attempt: 1, running: True
Startup Phase 2 complete
Attempting to initialize EGL from file libEGL.so ...
Successfully loaded EGL library!
Successfully initialized display on device idx: 0, EGL version: 1.5
Running CGLSimpleDevice::HALInit...
Calling glGetString: 0x7f5d323e8fe0
Version: 4.6.0 NVIDIA 440.100
Vendor: NVIDIA Corporation
Renderer: GeForce RTX 2080 Ti/PCIe/SSE2
OpenGL initialized!
Listening on: <IP_ADDRESS>:17730 (17730)
Startup Phase 3 complete. Ready for commands.
I0730 12:07:34.175502<PHONE_NUMBER>80224 remote_controller.py:167] Connecting to: ws://<IP_ADDRESS>:17730/sc2api, attempt: 2, running: True
Requesting to join a single player game
Configuring interface options
Configure: raw interface enabled
Configure: feature layer interface enabled
Configure: score interface enabled
Configure: render interface enabled
Entering load game phase.
Launching next game.
Next launch phase started: 2
Next launch phase started: 3
Next launch phase started: 4
Next launch phase started: 5
Next launch phase started: 6
Next launch phase started: 7
Next launch phase started: 8
Game has started.
Updated EGL surface size to 384, 192
Sending ResponseJoinGame
Exception in thread Renderer:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/renderer_human.py", line 1706, in render_thread
self.render_obs(obs)
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/renderer_human.py", line 69, in _with_lock
return func(*args, **kwargs)
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/stopwatch.py", line 212, in _stopwatch
return func(*args, **kwargs)
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/renderer_human.py", line 1724, in render_obs
surf.draw(surf)
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/renderer_human.py", line 658, in
lambda surf: self.draw_raw_layer(surf, from_obs, name, color))
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/stopwatch.py", line 212, in _stopwatch
return func(*args, **kwargs)
File "/home/hello/.local/lib/python3.6/site-packages/pysc2/lib/renderer_human.py", line 1668, in draw_raw_layer
surf.blit_np_array(color[layer])
IndexError: index 255 is out of bounds for axis 0 with size 2
I tried to  find a solution, but nothing works till now. Any help or fix would be much appreciated.
Regards,

I get the same error with you when I use the StarCraftII V4.6.2.
Then I redownload the SC2 V4.10 (Latest version), the error can be solved.
The error maybe caused by a version mismatch.
I think pysc2(3.0.0) and SC2 V4.10 can work.


App that makes windows transparent

Description of the new feature / enhancement
Similar to the "Always On Top" app in behavior, it would make the targeted window transparent.
The user could also define the transparency level from 0%-100%
Scenario when this would be used?
As an example, I work with one monitor only and do test automation against web apps.
Most of the time while debugging the tests and the app, I have to resize my IDE to be able to see the app while also being able to see the code, and stepping over it.
Despite resizing the window to be much smaller, it still makes sometimes me lose some important details, because the IDE is in front of the app.
I believe it could be useful in many other cases where we need to overlap information visually, changing music, etc...
Supporting information
No response

Yep I want a powertoys tool that makes windows transparent. (perhaps also the taskbar, like the "translucentTB" app).

duplicate https://github.com/microsoft/PowerToys/issues/1084

/dup https://github.com/microsoft/PowerToys/issues/1084


Add [da] translation to vim.desktop

Adding danish translaton even though i would prefer the strings added to the po files but i translate them here for now

Codecov Report

Merging #3089 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3089      +/-   ##
==========================================
+ Coverage   76.04%   76.05%   +0.01%
==========================================
  Files          92       92
  Lines      135746   135747       +1
==========================================
+ Hits       103223   103242      +19
+ Misses      32523    32505      -18

Impacted Files
Coverage Δ

src/gui_beval.c
62.39% <0%> (-0.86%)
:arrow_down:

src/if_xcmdsrv.c
83.99% <0%> (-0.18%)
:arrow_down:

src/window.c
82.17% <0%> (-0.04%)
:arrow_down:

src/if_py_both.h
76.6% <0%> (ø)
:arrow_up:

src/channel.c
83.13% <0%> (+0.07%)
:arrow_up:

src/gui_gtk_x11.c
48.13% <0%> (+0.14%)
:arrow_up:

src/os_unix.c
57.6% <0%> (+0.18%)
:arrow_up:

src/gui.c
49.76% <0%> (+0.2%)
:arrow_up:

src/term.c
60% <0%> (+0.2%)
:arrow_up:

src/message.c
74.82% <0%> (+0.24%)
: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 675e8d6...7a7c1be. Read the comment docs.

It seems to have been added.


New term: post-treatment Lyme disease syndrome

Closes #8047

The parent is inferred. It is the same as the term 'post-COVID-19 disorder' MONDO:0100320

The parent of 'post-COVID-19 disorder' MONDO:0100320 is MONDO:0021674 'post-viral disorder', is that the intended parent term? I thought Lyme disease was caused by a bacteria.
I still don't see the new term with a parent term when inferred.

@twhetzel
The pattern used for this term is the "post-infectious-disease" pattern.
This pattern was used for 'post-COVID-19 disorder' MONDO:0100320 which was autoclassified with the reasoner to MONDO:0021674 'post-viral disorder'.
Similarly, by using this pattern, 'post-treatment Lyme disease syndrome' is autoclassified (after the reasoner has run) as a 'post-bacterial disorder', as shown below. (

The reasoner has to run in order to see this inferred parent)

Yes, I understand the reasoner has to run to see the parent and the screenshot I posted was after running the reasoner. I checked again and after stopping and re-running the reasoner the inferred parent now shows up.


Interactive init

Allow to specify the migrations dir and other configurations.
Also should create the migration dir if it does not exist.

was implemented in #468.


Was working until Sept 30th, now I get an error when I try to generate images

Describe your environment

GPU: M1 Pro
VRAM: 32g
CPU arch: arm
OS: macOS 12.6
Python: Anaconda
Branch: main
Commit: commit 8e76bc2b5de7cb4b69f758518be38278f6d1099c (HEAD -> main, origin/main, origin/HEAD) # best I could find, there was no 'Merge'

Describe the bug
Following mac install guide, everything works (though I needed to use sudo often). It was working fine until yesterday, and It started to crash when generation was requested. Now even with several re-installs and rebuilding the environment I get this:

Request to generate with prompt: a rubber duck
Setting Sampler to ddim
/Users/evanadnams/opt/InvokeAI/ldm/modules/embedding_manager.py:153: UserWarning: The operator 'aten::nonzero' is not currently supported on the MPS backend and will fall back to run on the CPU. This may have performance implications. (Triggered internally at  /Users/runner/work/_temp/anaconda/conda-bld/pytorch_1659484612588/work/aten/src/ATen/mps/MPSFallback.mm:11.)
placeholder_idx = torch.where(
Traceback (most recent call last):
File "/Users/evanadnams/opt/InvokeAI/ldm/generate.py", line 303, in prompt2image
uc, c = get_uc_and_c(
File "/Users/evanadnams/opt/InvokeAI/ldm/dream/conditioning.py", line 16, in get_uc_and_c
uc = model.get_learned_conditioning([''])
File "/Users/evanadnams/opt/InvokeAI/ldm/models/diffusion/ddpm.py", line 828, in get_learned_conditioning
c = self.cond_stage_model.encode(
File "/Users/evanadnams/opt/InvokeAI/ldm/modules/encoders/modules.py", line 455, in encode
return self(text, **kwargs)
File "/opt/anaconda3/envs/ldm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/Users/evanadnams/opt/InvokeAI/ldm/modules/encoders/modules.py", line 450, in forward
z = self.transformer(input_ids=tokens, **kwargs)
File "/opt/anaconda3/envs/ldm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/Users/evanadnams/opt/InvokeAI/ldm/modules/encoders/modules.py", line 420, in transformer_forward
return self.text_model(
File "/opt/anaconda3/envs/ldm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/Users/evanadnams/opt/InvokeAI/ldm/modules/encoders/modules.py", line 373, in text_encoder_forward
hidden_states = self.embeddings(
File "/opt/anaconda3/envs/ldm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/Users/evanadnams/opt/InvokeAI/ldm/modules/encoders/modules.py", line 276, in embedding_forward
inputs_embeds = embedding_manager(input_ids, inputs_embeds)
File "/opt/anaconda3/envs/ldm/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/Users/evanadnams/opt/InvokeAI/ldm/modules/embedding_manager.py", line 156, in forward
embedded_text[placeholder_idx] = placeholder_embedding
NotImplementedError: The operator 'aten::index_put_impl' is not current implemented for the MPS device. If you want this op to be added in priority during the prototype phase of this feature, please comment on https://github.com/pytorch/pytorch/issues/77764. As a temporary fix, you can set the environment variable PYTORCH_ENABLE_MPS_FALLBACK=1 to use the CPU as a fallback for this op. WARNING: this will be slower than running natively on MPS.

Note:
To Reproduce
Generate an image

Hi @eadnams22 , based on the error, the environment variable PYTORCH_ENABLE_MPS_FALLBACK=1 is not set in your ldm environment.
This variable tells the system to use your CPU for some operations which are not yet supported on M1 Macs. However, it should be automatically set when you activate the environment e.g. conda activate ldm.
Can you please try:

activate the environment e.g. conda activate ldm
run this to explicitly set the variable export PYTORCH_ENABLE_MPS_FALLBACK=1
run the script e.g. python scripts/dream.py
try to generate an image normally

What do you get after that?
PS: You shouldn't need to use sudo at any point - that's an indication that something went sideways at some point during setup.

It is activated. First thing I tried, as per the warnings.
(ldm) evanadnams@eta-m1bp-2021 InvokeAI % echo $PYTORCH_ENABLE_MPS_FALLBACK
1
(ldm) evanadnams@eta-m1bp-2021 InvokeAI %

I've also uninstalled and removed everything including git and brew, and rebuild from scratch, same issue.
Thankfully my windows install works perfectly, even been adding new opjects successfully, but would be nice to have working on my mac as well.

Thanks for trying all that.
The only thing else I can think to try is to make a new user account and try there. If sudo is needed there, there must be something wonky deeper in your macOS installation, and I'd guess that's pointing to the same root cause.
Sorry I can't be more helpful - maybe somebody else will have other ideas.

Thanks for trying all that.
The only thing else I can think to try is to make a new user account and try there. If sudo is needed there, there must be something wonky deeper in your macOS installation, and I'd guess that's pointing to the same root cause.
Sorry I can't be more helpful - maybe somebody else will have other ideas.

Thanks! Yeah, was gonna try a new user next. Any advice on which folders to put things into?

I've got everything organized in ~/Documents, if that's what you mean.
Also, while conda worked fine for me, mamba does things a bit quicker and didn't need an environment variable to prevent environment creation from stalling while installing pip dependencies. I installed it with brew install --cask mambaforge.

Have you tried rm -rf /opt/anaconda3 and then reinitializing anaconda?

Any luck?

Have you tried rm -rf /opt/anaconda3 and then reinitializing anaconda?

I have tried that, for now I'm using it from my windows machine, will try again with the mac when I have some more time.

Hi @eadnams22 , did you ever get this figured out?

If you get back to this an still have issues on the latest release, please let us know. Closing as no activity in a while.


Fix Example HttpStaticFileServerHandler: raf is not closed closes #8604

Motivation:
Fixes the issue as mentioned below
Recently I use this HttpStaticFileServerHandler as file server, but I can't download mp4 file correctly. Then I found raf is not closed.
There is a issue mention about it: #2235, but it seems don't fix it.
I asked about it at stackoverflow: https://stackoverflow.com/questions/53500128/netty-httpstaticfileserverhandler-problem/53511572#53511572
Modifications:
made raf as final
added raf.close() call inside try/catch block
Result:
After this change the RandomAccessFile handler will be closed
Motivation:
Explain here the context, and why you're making that change.
What is the problem you're trying to solve.
Modification:
Describe the modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

@samaitra is this over SSL or not ?

@normanmaurer This is not over SSL. The issue is fixed in HttpStaticFileServerHandler

@samaitra I really dont understand why this should fix anything... Do you have a reproducer that I can run that fails before the fix and works after it ?

Without a reproducer I am not feeling comfortable pulling this in :/

@normanmaurer Thank you for your feedback. yes, it is not consistently reproducible. I will await further instructions from @yao-qingdong for steps to reproduce.
If you want me to close the PR I can close.


Add Relation class

removed backticks from links
updated links to other sections

Coverage increased (+0.4%) to 95.537% when pulling 7e6cf417b4b3d1fd24f6dcb412d285431a995b47 on heisian:new-docs into 25d704561d2111836ab8853e00829504071c4adc on Vincit:new-docs.

Thanks for this, but I'm making large changes to the documentation and it's easier for me to just do it than to explain the changes to other people.

oki doki. thanks for your ongoing work!


new field type from plugin

is there any way to create a new filed type for forms (backend) plugin through a plugin?
Where I need to start?
Thanks!
Looking forward, Balint.

Yup, basically form fields are just Twig templates.  All you need to do is create a plugin of your own that provides your custom fields as Twig template files, and use the Twig template paths event to register the location of these.
For example the Form plugin does exactly this, you can see the fields defined here: https://github.com/getgrav/grav-plugin-form/tree/develop/templates/forms/fields
And then this path is registered by the plugin via the onTwigTemplatePaths() event:  https://github.com/getgrav/grav-plugin-form/blob/develop/form.php#L76-L79
Now the paths under the templates/ folder is important.. it needs to be forms/fields/yourfield.html.twig
Also your plugin will need to to subscribe to this onTwigTemplatePaths() event, like the form plugin does: https://github.com/getgrav/grav-plugin-form/blob/develop/form.php#L32-L40

Thank you @rhukster , your explanation should be put in the documentation: https://learn.getgrav.org/forms/blueprints/advanced-features. I spent hours to follow the the docs and checked the core code, finally I found this issue. It was so easy!

Feel free to add it to the docs as a PR :) I just don't have time to do it all...

I will try. It is just that English is not my mother language :).


Adding SourceGraph to the Family

SourceGraph will help keep codeowners-generator active!
This PR adds their logo our Readme.
thanks SourceGraph!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
238

Relevant Lines:
238

💛  - Coveralls


Ensure clusterName can be used from global.kubernetes.clusterName

Proposed changes
Fixes https://github.com/splunk/splunk-connect-for-kubernetes/issues/184
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I have read the CLA
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

@gmichels Can you rebase your PR to latest develop and resolve the conflicts. Thanks.

I accidentally made another unrelated commit on the branch a while ago so I closed this PR an opened a new clean one in #207.


Remove demo.gif from master branch

The gif is 3MB large, and will be loaded anytime the package will be added with yarn/bower etc.
We don't need this gif as users or developers, so this asset could be hosted with github pages or using the Github CDN Issue trick

Yeah but still, if I just add this package it will be downloaded. If every project had some gif files, my vendor folders in every project would be tremendous.


⚠ api.piste.gouv.fr healthcheck has degraded performance

In ce70c4e, api.piste.gouv.fr healthcheck (https://api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 6241 ms

Resolved: api.piste.gouv.fr healthcheck performance has improved in fb65333.


Merge upstream updates

This merges updates to the original (Gabriele's) repo since March, including persistence and the ability to continue playing after 2048.

My fork is way out of date and has conflicting commits. Trying to resolve the conflicts broke stuff, so I'm just going to leave it as is for now.


Update index.md

updates

Docs Build status updates of commit d31c0df:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Community/index.md
:warning:Warning
View
Details

Community/index.md

Line 2: [Warning] Attribute redirect_url is reserved for use by Docs. Remove it from your file metadata.
Line 2: [Warning] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2: [Warning] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 3: [Warning] Attribute redirect_document_id is reserved for use by Docs. Remove it from your file metadata.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


The 'register_table' methods should allow you to pass a string representing a table that's already in the database

At the moment, all of the register table methods assume the user is passing in a pandas dataframe/list of dicts etc.
But there are cases when you want to tell Splink that an already-existing table represents some Splink table such as the labels table, a tf table or whatever.
That is, the user might want to tell splink that a table called "term_frequencies_first_name" represents the Splink entity: "__splink__df_tf_first_name"
def register_table
def register_table_input_nodes_concat_with_tf
def register_table_predict
etc.

Think this is resolved


get_files interface has changed

Hi,
Just a quick note to let you know that the docs are out of date in the section on get_files. I couldn't figure out how to get this to work, at first. I had to use .url instead of .publicPath to get my file path.
Thanks!

Thanks @johncronan .
@rvlb could you please check this?


There is an issue here.

There is an issue here.

In landscape view on iPhone x, the image is pushed a few pixels to the right.
If you rotate the screen after zooming in, the position will not come out properly.

@eastswift Sorry for late reply. I've been busy with my company project but now I am back. Anyway, thank you for your report, I just checked and noticed the same issue, I will fix it and release an update  asap.

@eastswift I've fixed those issues with new version 1.2.2. Can you please check and make sure that they are ok now.

It has not been fixed yet. If you turn the image over while zoomed in, the next image will not be displayed. ^^;

@eastswift Yes you are right. I will try to fix it as soon as possible.

@eastswift @ducnv2910 Sorry for late answer. I believe the issue is now completly fixed. Please check latest pod version (3.0.1).


BUG:Sidebar not opening on click of Hamburger Menu Icon

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Description: The hamburger menu icon (three lines) is unresponsive when clicked. The sidebar does not open, preventing users from accessing the sidebar content.
Steps to Reproduce:
Navigate to the website.
Click on the hamburger menu icon.
Expected Result:
The sidebar should open upon clicking the hamburger menu icon.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@arghadipmanna101, Please assign this issue to me!
I fix the sidebar not opening when click of hamburger Menu Icon.


Integration with Jenkins CI pipeline

ok to test

ok to test

ok to test

ok to test

retest this please

ok to test

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please


Indicate environment in emails

We want to let the user know which environment sent the email they receive, particularly in the case of the "short url requested" emails, so people know if they can ignore it because it's not a production one.
For: https://trello.com/c/SAcBmzf3/85-include-indication-of-which-environment-sent-the-email-in-short-url-manager
See: https://github.com/alphagov/govuk-puppet/pull/5338 which adds the env var we need.

I'm slightly surprised that this data isn't available already, without the changes in Puppet, as the environment name appears at the top if I go to the short url manager in integration?

Good point @cbaines - I don't know how or where that info is set, I was just copying what signon does for its emails.  I'll see if there's an existing way to get this info.

Good point @cbaines - I don't know how or where that info is set, I was just copying what signon does for its emails. I'll see if there's an existing way to get this info.

I think your solution looks really good, so I'm fine going ahead with this now that we know how it works for the admin template (thanks for investigating by the way).

For those following along, we had the follow up conversation in slack.  The UI gets the "production" or "integration" labels from GovukAdminTemplate.environment_label (see: https://github.com/alphagov/govuk_admin_template/blob/7f036644b78d054ca94eed93b7936181380a918c/lib/govuk_admin_template.rb) which has its value set by an initializer written out as part of the deployment process by our capistrano recipes (see: https://github.com/alphagov/govuk-app-deployment/blob/c588a59f42f39317e49361f9de831b96cfe8fdc4/recipes/govuk_admin_template.rb).  We decided it was a bit brittle to rely on this value as it's specifically for the admin template, which the emails aren't.
On the puppet PR we also started a conversation about making a DEPLOY_ENV environment variable available to all apps, rather than needing to set one for each app that needs it (signon and short_url_manager currently).
For now we're leaving this as is, but hopefully we can change it soon.

Flagging as DNM for now as we need the puppet PR to go out first

Removed DNM as puppet change has been rolled out.


Machine mac-45 went dead from monitoring network connection website.

Machine mac-45 went dead from monitoring network connection website.

mac-45 is up and running fine. However, salt-call command wants to upgrade m2crypto from v0.36 to v0.38 that reason machine marked as dead from monitoring website.