Fix: build wkhtmltopdf with patched Qt4

Motivation for this change
At some point the build for wkhtmltopdf was switched to upstream Qt5. This is not intended to happen according to the
documentation that wkhtmltopdf itself has published:
When using the version currently available in NixOS we got a lot of complaints that wkhtmltopdf isn't working as expected any longer, specifically with weirdness around X integration, bad renderings, etc. I'm not sure why this was changed to Qt5.
However, I pulled in the original version of the package and updated it to the latest wkhtmltopdf. This works on my 19.03 based system but for some reason I get into a a weird situation on master where this fails to compile: qt4 itself builds fine, but wkhtmltopdf fails with:
In file included from ../lib/multipageloader_p.hh:24,
                 from ../lib/multipageloader.cc:21:
../lib/multipageloader.hh:30:10: fatal error: QWebPage: No such file or directory
   30 | #include <QWebPage>
      |          ^~~~~~~~~~
compilation terminated.
make[1]: *** [Makefile:434: multipageloader.o] Error 1
make[1]: *** Waiting for unfinished jobs....
In file included from ../lib/converter.cc:22:
../lib/converter_p.hh:27:10: fatal error: QWebSettings: No such file or directory
   27 | #include <QWebSettings>
      |          ^~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [Makefile:448: converter.o] Error 1
In file included from /nix/store/ycrslxg21nzdiq9whvd0xwpv1whni595-qt-mod-4.8.7-7480f44/include/QtCore/QString:1,
                 from ../lib/logging.hh:25,
                 from ../lib/logging.cc:21:

I can see that QtWebkit is not included in the Qt4 output, and the build output seems to configure but not build it. I'm a bit confused by that and would like to:
a) discuss whether we would agree in general whether moving back to the patched qt4 version will be accepted
b) get some help figure out why this works on 19.03 but not master.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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
[ ] Fits CONTRIBUTING.md.

Mentioning @orivej @lovek323 @phreedom @sander as potential partners to discuss this.

I'm OK with keeping it working with Qt4 if possible.
wkQt on master probably does not include webkit due to #92261 and missing dependencies. You can either add them to the overridden buildInputs or replace wkQt = qt4.overrideAttrs with wkQt = (qt4.override { buildWebkit = true; }).overrideAttrs, but you will have to fix the build of the old gstreamer first.

I have added a commit that fixes the build on master. It was failing due to gcc9 update. gstreamer turned out to be unnecessary.

Ok, sorry, I was on vacation. Nevertheless: I've removed the trailing whitespace accident and hope that checks will be fine now. Anything else needed before we can merge this?

Pinging @jb55 to get his blessing on this. I just reviewed this again and this is the same patch we've been running at FCIO for a while now. :)

don't need my blessing, I haven't been homesteading this for a long time now. happy to pass maintainership

@jb55 i really just need someone to push the button. I don't have commit rights, yet. :) maybe my colleague @dpausp wants those? not sure I can be trusted with commit rights as I do things to irregularly at the moment. ;)

Great, so now that this has been around for a while and I wanted to push it on for merging, we got reports that this build causes incorrect renderings. I have a version around that has updated build settings that work better.
Does anyone have an idea for how to validate rendering quality through tests? In particular the current build seems to break with some CSS issues and using the upstream build options from the official build automation works. I'll update this PR later today.

@ctheune Thanks for bringing up the issue!
I was wondering how to "build wkhtmltopdf with patched Qt4"?
Is it to build from source https://github.com/wkhtmltopdf/wkhtmltopdf? If yes, what steps shall I follow? (I don't see makefile, so I guess I can't run make.)
Shall I install it into my home directory?
Shall I also build dependency qt from source and install it to my home directory?

Hey @grassit I'm not sure what you're asking. Your question seems not really related to this change here and is quite open-ended. If you're asking about general build issues with wkhtml then I'd suggest contacting their mailing list. Unfortunately I'm too busy to write down an overview of the build process. The download page has a few bits about it, though: https://wkhtmltopdf.org/downloads.html

@ctheune

Great, so now that this has been around for a while and I wanted to push it on for merging, we got reports that this build causes incorrect renderings. I have a version around that has updated build settings that work better.

Have you / Do you need to update the PR with your changes?

Does anyone have an idea for how to validate rendering quality through tests? In particular, the current build seems to break with some CSS issues, and using the upstream build options from the official build automation works. I'll update this PR later today.

I'm not sure but if you can reproduce the issue with a very small pdf, perhaps you can do binary validation on the resulting pdf?
I can help get this through the finish line as I'll be needing it soon as well.

BTW I tested your PR with the current master, it built fine.

I'm going to pass this over to my colleague @dpausp as I'm out of the loop by now ... :)

qt4 has been dropped https://github.com/NixOS/nixpkgs/pull/174634


2022.1.2 Change in CFBundleIdentifier means no (current/old) devices are recognised.

https://github.com/FlyingDiver/Indigo-GeoFence/blob/bbc27dab7b96705b1239d994c672605cbc2d42c0/GeoFence.indigoPlugin/Contents/Info.plist#L12
Need to add the dot back in… eg. indigo.plugin

https://github.com/FlyingDiver/Indigo-GeoFence/releases/tag/2022.1.3


Detect broken connections / make timeout configurable

Hi! 👋
I was trying to use the deluge integration of Home Assistant which uses this egg. I noticed that the deluge device in Home Assistant stayed unreachable after the deluge server was restarted. This happened because a broken connection was not detected and thus never reset.

Detect closed connections in _receive_response
When the connection to the deluge server was lost (the server was
restarted or shut down), _receive_response was stuck in a infinite
loop before. Since recv never returns any data the function never broke
out of the while loop. Raising a ConnectionLostException ensures the
retry logic can work as expected.

Make the socket timeout configurable

looks great.


Custom estimators

Completed the Custom Estimators Notebook.

this has two notebooks numbered 6

this has two notebooks numbered 6

Are you sure? My local has just the custom estimator one? Which other one are you seeing as 6? Can you try deleting your local branch and pulling it down again?


Help

https://github.com/devcontainers/cli/blob/main/.devcontainer/devcontainer.json

Looks like this issue is created on accident, closing!


Feature Request: Ability to tag Clue scrolls

The collection log community currently is developing a meta based around tagging clue steps based on their unique item ID and sorting them into categories. If we were able to leverage this plugin and it's export feature, that would make it extremely handy in terms of sharing the list of "meta" tags.
Unsure if the restriction of gear items is a hard limitation here

I now see it relies on the current item equipped, so maybe a fork would be more appropriate :)


Implement automatic updates

Implement the automatic updates of the meta information of the libraries

Propose a draft how you want to implement this.

Is implemented in #19. The git-scrabber searches for information of libraries that have a valid url specified in the markdown file. Afterwards, the information from the report is added to the libraries. If an information (e.g. the used block ciphers of a library) is specified in the markdown, this data is prefered over the collected data.


fix: accessible button

This PR makes the button elements more accessible. It

changes the type of buttons from div to button
removes/abstracts a styling that was depending on the div type of the buttons

Fixes #123

@dnsos It is not enough to change on button. We have several buttons on the components. Will you take this on? I added the role attribute to that specific one in #122 so we don't have to go into changing styling of buttons for now

Okay, I will then search for all buttons in the components and add a button role to them.

Okay, I will then search for all buttons in the components and add a button role to them.

@dnsos awesome! Take a look at https://github.com/technologiestiftung/tsb-trees-frontend/blob/master/src/components/ButtonRound/button-round.test.js for an example of a small test. It would be awesome to add some basic tests while you at it.


Crash Caused Other Mod Configs to Delete

Hello, I just spoke with bolyai from Wynntils because my Item UI had vanished along with the whole Wynntils config. I can send you the full conversation over Discord if need be.
He suggested I send this report your way.
I have a low performance system so I didn't think much of the crash when it happened. The game froze and Minecraft closed.
crash-2024-04-29_01.09.19-client.txt
Kind regards.
This feels like I'm writing an email... I've never done this before

really..?
i taught it was fixed in 1.20.2-1.1.1.fix.1 but wasn't.
I'll work for fixing it.
i'm so sorry for crashing and it caused deleting config.


python311Packages.todoist-api-python: 2.0.2 -> 2.1.1

Diff: https://github.com/Doist/todoist-api-python/compare/refs/tags/v2.0.2...v2.1.1
Changelog: https://github.com/Doist/todoist-api-python/blob/v2.1.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 248347 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.todoist
python311Packages.todoist-api-python
python311Packages.todoist-api-python.dist


Added xml serialization support

Hello there,
i've added some support for xml serialization of the MathML objects by implementing System.Xml.Serialization.IXmlSerializable as we do need this for our application. It would be cool if you could implement this change, so we can keep using your library from NuGet.
A simple test is provided in Tests.Program.cs, but maybe it does need another few tests.
Also I've used the Visual Studio auto formatting for all edited classes to make the code look better.
Thanks in advance!

Thank you very much for making this project better!

You're welcome. Also thank you for this implementation.
Could you also publish this at nuget.org? Thank you!

Sure, I've already published this lib to nuget.org, And I'll update it soon.
Regards!


Something went wrong! file an issue at https://git.io/JJmN9?

when pushing a new file in my repository that language indicator in my profile readme saying this error
Something went wrong! file an issue at https://git.io/JJmN9 Maximum retries exceeded Please add an env variable called PAT_1 with your github token in vercel

after refreshing it was gone(couple of time after)


TLS Socket servername needs to be set to host when KafkaClient.createBroker is called

Question
Whenever I tried accessing my SSL configured Kafka cluster externally, I always ran into a NotLeaderForPartition. It looks like even though the correct host for the leader broker was found, the servername on the TLS socket kafka-node tries to establish with the leader always remained the same one I originally set it to in the sslOptions.
Here are the options I passed into my KafkaClient:
const kafkaClientOptions = {
	kafkaHost: '<myKafkaHost>',
	ssl: true,
	sslOptions: {
		ca: fs.readFileSync('...'),
		cert: fs.readFileSync('...'),
		key: fs.readFileSync('...'),
                servername: '<myKafkaHost>',
	},
};

I was able to fix this by adding a single line of code to kafkaClient.js
...
if (self.ssl) {
    self.sslOptions.servername = host; // this is the line I added
    socket = tls.connect(port, host, self.sslOptions);
  }
...

I'd like to make this change to the library myself as obviously updating a file in your node_modules is not a permanent solution.
Does anybody see an issue with me making this change?
Environment

Node version: v8.12.0
Kafka-node version: 4.1.3
Kafka version: TBD

I face the exact same issue. If not automatically adding the servername, at least allow a callback to change sslOptions per host being connected to


Please fix slider thumb which is getting cropped

This doesn't looks good and i can't fix it using ref so opening the issue

Can't fix it either, were you able to find a fix for it?

@ShivamJoker , @AntoLepejian  Sorry I can't fix it. There is too much work now. When I get free I'll fix it

A temporal solution might be thru CSS. This worked for me:
.container > svg: {     overflow: 'visible'   }


[FEAT] make use of ramdisk for logging

Objective
At the moment, on the devkit, logs are saved in .local/config/neon. I have noted that the image is also loading a ramdisk of 20Mb which is enough for storing logs. Why not make use of it?
Initial Implementation Requirements

you could simply create a symlink from .local/config/neon to /ramdisk
you could modify all init scripts to output logs to /ramdisk

Other Considerations
If we do not use ramdisk for the logs, then it should be better remove it, in our little machines, 20Mb could be put at a better use than sit there

The logs should be saved at ~/.local/state/neon, but in any case those should utilize the ramdisk. There might be some additional consideration for handling the log archives (currently handled as subdirectories in the log directory)


Bad Download when video title contains colon : on Windows

On Windows, via the CLI, I tried downloading the video titled TwoSet Violin & SSO: Mendelssohn Violin Concerto as follows:
ytdownload https://www.youtube.com/watch?v=zlRRdGnzA00
this started and completed the download successfully:
[YTDownload] Downloading 100.000%
╔══════════════════════════════════════════════════╤═══════════════════╤═════════╗
║ YTDownload                                       │ Status            │ Percent ║
╟──────────────────────────────────────────────────┼───────────────────┼─────────╢
║ TwoSet Violin & SSO: Mendelssohn Violin Concerto │ Download Finished │ 100%    ║
╚══════════════════════════════════════════════════╧═══════════════════╧═════════╝

Downloaded in: 24.000s
Downloaded to ./videos/TwoSet Violin & SSO: Mendelssohn Violin Concerto.mp4

But when I looked in the videos directory, there was a 0 byte file in there titled TwoSet Violin & SSO
So I'm guessing there's an issue when the video title contains colons, which are not allowed (besides for drive letters) on Windows.

Thanks for the report, fixed with<EMAIL_ADDRESS><3


In Edge, 'Read Aloud' doesn't work with extension enabled

Description
When trying to use the 'Read Aloud' Text to Speech feature in Edge, if the DDG Privacy Essentials extension is enabled, no audio can be heard.
Steps to Reproduce

Go to any website you want to have read aloud
Make sure the DDG Privacy Essentials extension is enabled
Click the 'Read Aloud' button on the right hand side of the address bar (it's an 'A' with curves around the top right corner)
The Read Aloud bar will appear across the top of the page, but the moving highlight will not be there, and the audio speech cannot be heard.

Expected behavior:
Audible speech from the Read Aloud function.
Actual behavior:
No audible speech from the Read Aloud function.
Versions

Extension: 2<IP_ADDRESS>
Browser: Microsoft Edge Version 111.0.1661.62 (Official build) (64-bit)
OS: Windows 11 22H2 22621.1413

Additional Information

Issue is 100% repeatable
Not sure if this used to work on previous versions
I was unable to find the extension debugging console in Edge
I will add screenshots or video if requested
I have verified that with the DDG extension disabled (but all other extensions I use still enabled) Read Aloud works
Read Aloud is a very useful accessibility feature for folks with disabilities

Thanks for this report. I was unable to reproduce the issue in the Edge browser and the latest extension version. Are you still experiencing the problem?

For some months now, I have been having problems with Edge's Read Aloud whenever DDG Privacy Essentials is activated. Turn it off, and Read Aloud works fine.
My experience duplicates seffyroff's experience. Same description. Same steps.
Privacy Essentials Version 2023.5.9
MS Edge Version 113.0.1774.35 (Official build) (64-bit)


Indexer timeout when server is under pressure

I'm testing the merge & demuxing.
Two interesting things happened:

first the actor MergeUploader was killed without apparent reason, it seems concurrent to the opening of segments to demux but this may be just a coincidence.
when the server is under heavy load, the indexer has a timeout, it only happens when I'm making huge merge.
secondly, the demuxing continue to work for 30 min as it is protected during the tantivy demux.

I think we need to raise the actor's heartbeat, 1 second seems a little bit too short.
@fulmicoton @guilload: let's choose between 2, 3, 4, or 5 seconds... have you an opinion on that?

Is there some gain to killing actors that fast? External services can be flaky, e.g. due to high load. An HTTP response can easily take several seconds. Killing and restarting can put additional load on these services by doing work multiple times.

@PSeitz For the moment the goal is to detect bugs and identify where we block without knowing it. In production we can increase this limit.

I think right now for the indexer we have a guard that did not exist before?
The tantivy term sort was the culprit. It can take >1s if we have a lot of terms.

Do you still experience timeouts? If yes in which actors?

We can consider printing warnings/errors instead after a threshold, in order to have something closer to production settings running.

Did not see the guard in the indexer!  Will test it again

Let's close for now and reopen it when it happens again.


Unable to install from scratch. Service depends on non-existent service.

Hi,
Currently having a problem pulling the skeleton, as per the installation instructions.
Run: composer create-project sylius/plugin-skeleton .
Output: https://pastebin.com/fpcmEgfb
Everything appears to install with no errors and such. I then quickly added a .env for database string and created schema and run fixtures.
When I visit the site or run behat tests I get the following:
The service "security.authorization_checker" has a dependency on a non-existent service "security.authentication_manager". Did you mean one of these: "security.authorization_checker", "security.authentication_utils", "security.authentication.manager", "security.authentication.guard_handler"?
Environment:

PHP 7.4.26 (cli) (built: Nov 30 2021 08:21:51) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.26, Copyright (c), by Zend Technologies
Composer version 2.1.14 2021-11-30 10:51:43

Is this potentially because i am using PHP7.4 and not the 7.3 specified in the composer.json?
(Just a note, 7.3 drops out of support at midnight tonight)

Hello, I "fixed" this issue by downgrading symfony/security-bundle to 5.3.*. Sylius is not 100% compatible with Symfony 5.4


[New Feature]: Add outputs.tf for INT, PST, and OPS  environments

Checked for duplicates
Yes - I've already checked
Alternatives considered
No - I haven't considered
Related problems
No response
Describe the feature request
We need to add an outputs.tf file for INT, PST and OPS environments.

it is a set of output variables that terraform can create for us. Other frameworks like AWS SAM and AWS CDK have a similar concept. These variables can be referenced in other stacks, but we mainly use them in terraform as an additional mechanism to extract variables (that variables.tf doesn't do in certain situations).
They also serve as additional documentation. Teams use this set of variables to output things like queue names or URLs and things of the sort that terraform knows about at the point of deployment, that would otherwise be cumbersome to get. On other projects, these are used to output things like Lambda names, API Gateway URLs, SQS queue names/URLs, and other outputs of a stack (like a name of all the resources created).
For example, we have terraform code that outputs the MOZART_IP to a text file called mozart_ip.sh for use by other scripts. This could be replaced by shell command like $(terraform output mozart_ip). So it can help in automating certain tasks.


Keep multiple search result

Like notepad++, tabular results or similar will be great. Sometimes i need to search twice the same string or regex, in big project this is a problem.
Expected

Search "A"
The results will show as a tab in the search bar
Search "B"
The results will show as a tab in the search bar
Clic on "A" results will show the first save result

https://github.com/microsoft/vscode/issues/16488


Add a node "GroundingDinoDetect"

A node "GroundingDinoDetect" has been added, which solely calls the GroundingDino model for detection. It outputs the corresponding image with detection boxes and masks, as well as annotation information in LabelMe format.

Wow this is so usefull - I have switched to this fork


Timescale?

I don't know exactly what the error concerns, but it occurs when using filters in music

I'll have another release later that addresses this, but just for the record, what OS is this running on?

I'll have another release later that addresses this, but just for the record, what OS is this running on?

Linux aarch64

I mean more specifically, like Ubuntu 18 or Raspbian 64 bit.

I mean more specifically, like Ubuntu 18 or Raspbian 64 bit.

Canonical Ubuntu 20.04

Can you please try timescale transforms on https://github.com/Cog-Creators/Lavalink-Jars/releases/tag/3.4.0_1347 and reply here whether it works or not? Alternatively you can find me in the JDA server if you wish to DM me. My discord tag is aikaterna and you will see recent messages from me in the lavalink and lavaplayer channels.

Can you please try timescale transforms on https://github.com/Cog-Creators/Lavalink-Jars/releases/tag/3.4.0_1347 and reply here whether it works or not? Alternatively you can find me in the JDA server if you wish to DM me. My discord tag is aikaterna and you will see recent messages from me in the lavalink and lavaplayer channels.

Yes its working

Thanks, I appreciate the help on this. If you have other issues in the future don't hesitate to open an issue with your logs.

Thanks, I appreciate the help on this. If you have other issues in the future don't hesitate to open an issue with your logs.

Sure thanks for the quick reply. ;)


New GPTs: Research Tools

Please add Daily Research Digest (https://chat.openai.com/g/g-Nmhk1adrD-daily-research-digest), Daily AI Research Digest (https://chat.openai.com/g/g-Z4yeZHkyy-daily-ai-research-digest), and Scientific Research Digest (https://chat.openai.com/g/g-XrX7bd1HU-scientific-research-digest), which are tools for finding and summarizing the latest scientific research.
PR: https://github.com/Anil-matcha/Awesome-GPT-Store/pull/163
Please let me know if I should just submit 1 GPT per issue going forward / if you want me to split this up into 3 issues. Thx

Merged


Select selectedOptions not working

Describe the bug
Select component's selectedOptions attribute not working
To Reproduce
Steps to reproduce the behavior:

Create Select component & create SelectOptions
Set selectedOptions

Example
<BlSelect selectedOptions={[{ text: "a", value: "a", selected: true }]}>
    <BlSelectOption value="a">aaaa</BlSelectOption>
    <BlSelectOption value="b">bbbb</BlSelectOption>
</BlSelect>

Versions
Baklava v2.0.0-beta.39
React 18.2.0

Hello Aykut,
selectedOptions attribute is not defined in the bl-select component, it is only a getter. You can set selected attribute to each bl-select-option to show them in selected state. But I think such a use (set all of them via parent) should also be considered.

Hello Aykut,
selectedOptions attribute is not defined in the bl-select component, it is only a getter. You can set selected attribute to each bl-select-option to show them in selected state. But I think such a use (set all of them via parent) should also be considered.

Hi, thank you for clarification. I think managing selected options from parent would be so handy.


Embeds array for send message async

Since discord has added the support for it in the api I thought why not add it :)

See #129

@quinchs Mine would not break preexisting peoples old SendMessageAsync that uses a single embed.

@d4n3436 thoughs?

It would be better to move the  single embed parameter to the end to keep consistency with other methods that also added embed array params (like #72)
Honestly I would just replace the single embed param with the embed array. It will be a breaking change but a better one, since the single embed param will have to be removed eventually.

@d4n3436 This would allow for people not breaking their code if they wish to use the single one then they can it will simply add the single one to the array if the array is null.

@d4n3436 This would allow for people not breaking their code if they wish to use the single one then they can it will simply add the single one to the array if the array is null.

Adding an extra parameter anywhere but at the end will be a breaking change anyway.
A better solution may be:

Keep the embed param in the same place it is right now.
Add the embed array param at the end.

This solution won't break anyone's code and will allow people to use the embed array param if they want to.

True ill move them to the end.

Wait that was my idea 😐
nvm

It is done @d4n3436 sorry I had no idea you had a pr, I would just have added the latter stuff to yours if I had known.


Fix an issue with inline_local_panic_functions

The micro-pass which inlines the panic functions did not filter those functions from the declaration groups.

We could have also run reorder_decl after that pass instead of before


Kernel sometimes dies with notebook executor

For example https://github.com/Qiskit/documentation/actions/runs/12015838807/job/33494742907
task: <Task finished name='Task-35' coro=<execute_notebook() done, defined at /home/runner/work/documentation/documentation/scripts/nb-tester/qiskit_docs_notebook_tester/__init__.py:253> exception=DeadKernelError('Kernel died')>
Traceback (most recent call last):
  File "/home/runner/work/documentation/documentation/scripts/nb-tester/qiskit_docs_notebook_tester/__init__.py", line 268, in execute_notebook
    nb = await _execute_notebook(path, config, working_directory.name)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/work/documentation/documentation/scripts/nb-tester/qiskit_docs_notebook_tester/__init__.py", line 346, in _execute_notebook
    await notebook_client.async_execute()
  File "/home/runner/work/documentation/documentation/.tox/py311/lib/python3.11/site-packages/nbclient/client.py", line 709, in async_execute
    await self.async_execute_cell(

We can't tell which notebook it is from the logs as the error doesn't contain any defining information and the notebooks all run asynchronously. We can whittle it down if jobs fail while only running a subset of notebooks.

I managed to reproduce a similar problem locally when I added more notebooks to the script.
zmq.error.ZMQError: Too many open files

I fixed this locally by increasing my ulimit to 6000 (ulimit -n 6000). Hopefully we can set this in our action too.

@frankharkins let's keep this open until it's been a few weeks of not seeing it to confirm #2464 did fix the issue.


x' should be rendered as x\prime, not x^\prime

Ugh, this behavior is font-dependent. x' should translate into x^\prime, but some fonts define the \prime glyph to be a big symbol, and some define it as already in the superscript position.
Asana Math uses the "big prime" glyph, so it looks ok:

But Libertinus Math uses a pre-superscripted prime glyph, so it gets shifted up twice:

5a35335 checks the size of the font's prime glyph to adjust between \prime and ^\prime. Also for similar glyphs such as \dprime.

Really fixed by e3a42c0 which uses the font's ssty GSUB feature to pick an appropriate prime glyph.


Is it possible to visualize a group of vertices with one color?

e.g. For one label, I'd like to group them several categories, and visualize them using different color per category. Is it possible to do that/

Currently color grouping works per unique vertex label, but not per vertex property. You could contribute this feature to allow color grouping per vertex property instead and I will be happy to review it.


warnings carrying over from base

gs <- gsplot() %>%
  lines(1:10, 1:10, axes=FALSE) %>%
  points(2:7, 7:2)
gs

plot.window(xlim=c(1,10), ylim=c(1,10))
lines(1:10, 1:10, axes=FALSE)

gs <- gsplot() %>%
  lines(1:10, 1:10, axes=FALSE) %>%
  points(2:7, 7:2)
gs

One thing...R in general is really bad about forgetting warnings. I'd put this as a low priority.


Backport Eclipse changes to release-15.09

See commits.
I'm looking for feedback, whether we'd like to make such changes on the stable branch or not. I'm thinking 'yes' (obviously).
CC @rycee, @muradu, @DamienCassou

I definitely think this is a good idea. Since Eclipse and the plugins don't get built by Hydra we are vulnerable to URL breakage. This is not a big problem for Eclipse itself and the core plugins such as JDT but I did have that problem with TestNG, for example and the 5b290be commit in this PR includes a fix that I hope will provide more URL stability.
The Scala plugin is another example where the file is updated in place so the hash may change without warning; in fact I just checked and the Scala hash in master differs from a fresh download :-( I'll make a PR for this new version.
So basically I happily support this PR because it is limited in scope (only affects Eclipse and Eclipse plugins) and will fix some issues that currently exist in stable Nixpkgs due to broken URLs.

Added the Scala plugin bump in #12638, I suggest adding that commit to this PR.

+1

Cool. Added scala plugin bump to PR. Merged.


Setting required to input element within TextInput

When setting required to TextInput it is not set to the input element - only on the aria attribute:
https://github.com/primer/react/blob/254e289b845c5167febd304be082fe8493aa87ab/packages/react/src/TextInput/TextInput.tsx#L149
Could you set it so that built-in form validation can be used fully? For now it is awkward to be able to set a pattern or other constrains but that the component built-in validity is always true if no value is set.

Hi!
This is intentional change! (https://github.com/primer/react/pull/4023)
aria-required communicates the same message to screen readers as required, but avoids browser's default validation messages. Example of browser validation message:

If you'd like to use validation messages that match Primer's design, please use <FormControl.Validation>. example in storybook ↗ | Source code for example ↗ | Primer docs ↗
Hope that answers your question. If not, feel free to comment on this issue and I'll open it again!


Add presets to custom curve (auto FK script)

Add a few preset curves in the custom curve for the Auto FK script:

Cube
Pin
Joint shape

Also add an option to include scale in the control creation

Added Cube as a custom shape : https://github.com/TrevisanGMW/gt-tools/commit/53e4154d2f28a3a2dc485312a7f87a940d7c4be6


GH-35866: [Go] Provide a copy in arrow.NestedType.Fields() implementations

Rationale for this change
Data types should be left immutable after they've been constructed.
What changes are included in this PR?
Provide a copy of fields in the following implementations:

*arrow.StructType.Fields()
*arrow.unionType.Fields()

Are these changes tested?
arrow.TestFieldsImmutability was added to ensure the behavior.
Are there any user-facing changes?
Now the fields for structs & unions will be immutable.

Closes: #35866

LGTM just want to have the documentation indicate that it makes a copy so that callers can choose to instead use the Field(i) methods if they want to avoid the copies

Just FYI, Field(i) provides a copy of the arrow.Field (note: the Type field will point to the same instance, but the pointer itself is copied)

@candiduslynx true, (a mistake in my wording) i meant if they didn't want to make a copy of the entire slice of fields. Copying an individual field is extremely cheap, but under certain circumstances it could be desireable to avoid copying the entire slice of fields.
Either way most times that a slice is returned from a method in go, the average usage assumes that it isn't making a copy of all the elements so it's good to ensure the godoc is clear.


ES5 support

hiya
i am using version 8.6.12 and i'm getting a crash with the IE11. i can see import and () => used in the transpiled/compiled code, which obviously isnt ES5 friendly. is this change by design or has it gone unnoticed?
looking at 9.0.5, the tsconfig seems unchanged, so i'm assuming it will crash in the same way.. 🧐
cheers

ok, i just been told IE is not supported. maybe a supported browsers in read me would be good? closing now.

This sounds odd as I only export a UMD module (not using import) -
https://github.com/DominicTobias/react-image-crop/blob/master/webpack.config.js#L13
However yes you’re right IE is no longer supported, definitely should
mention in the Readme ;)
On Sat, 11 Dec 2021 at 10:38, dnsmob @.***> wrote:

Closed #459 https://github.com/DominicTobias/react-image-crop/issues/459
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DominicTobias/react-image-crop/issues/459#event-5751484339,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFZT6RNEF4NXOWMH4OWNGLUQMS2ZANCNFSM5J2Z3I3Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Safari iOS/macOS - Game audio continues playing after leaving page

Version

Phaser Version: 3.24.1 and earlier

Operating system: macOS 10.15.7 and iOS 14

Browser: Confirmed on Safari 14.0, seen on older versions too.

Description

Bug only occurs on iOS/macOS Safari.
Bug occurs only when an iframe on the page redirects to the same domain.
Bug only occurs when Web Audio is enabled.

When I click on an iframe on the same page as the game, that tells the browser to navigate to another page on the domain, any game sound that is currently playing will continue playing on another webpage.
Example Test Code

https://github.com/AdamXA/phaser3-webaudio-bug
git clone git@github.com:AdamXA/phaser3-webaudio-bug.git npm install npm start
Forked from Phaser 3 boilerplate.
In the example, once it is running, hear the audio playing, then click the white "Click here" text in the top left of the canvas to be redirected.
Additional Information
This is one weird bug.

You're navigating away from the page in which Phaser lives? If so, I can't see how we could possibly fix this, sounds like (yet another) Safari bug to me.

Thanks I'll try forward it onto them then 👍

https://bugs.webkit.org/show_bug.cgi?id=218078

should be fixed in Safari Tech Preview 117

That's unusually quick of them to address :)

Closing this off as they've thankfully fixed it.


error: "description": "{"response":null,"error":"Unauthorized missing scopes","error_description":""}",

"description": "{"response":null,"error":"Unauthorized missing scopes","error_description":""}", (edited)
We are sending scopes in our auth request so not sure what’s going on here
Thank you.

We see that sometimes as well, usually indicates that the end user didn't approve of the requested scopes needed for the issued request.

That's right. We've made docs changes to reflect missing scopes: https://developer.tesla.com/docs/fleet-api#faq & https://developer.tesla.com/docs/fleet-api#authorization-scopes


[BDD] Rename a wishlist

Implement frontend to support this part of the API and behavior tests.
Scenario: TBD
Given our []
When []
Then []

Hey team! Please add your planning poker estimate with ZenHub @Fivetoes @jiuliwei @XinyuC97 @yx2512 @ZX-WANG


WIP: Add drf spectacular support

This PR adds support for DRF spectacular (see issue #61 ).
I didn't test this locally, and haven't seen any tests written for drf-yasg so I didn't write tests for this code. As such this PR is currently WIP, and I will try to find time to complete it in the near future. If someone else would like to test it or add tests that will be awesome.
Changes:

Added support for drf spectacular following the example of drf-yasg
Resolved issues reported by pytest (wrong version in assert + deprecated import statement)

This looks very clean. Once tests are in place, I'll be happy to merge this as is.
Before we do however, we need to add some demo code in the demo project, specifically implementing DRF-spectacular, and to build tests around that.

cheers - i'll try to find the time this weekend to finish this.

I've just pushed through version 2.0.0, with a revised setup for loader classes. I actually think it will make the implementation of a DRF-spectacular loader class even cleaner than before.
I've also upgraded the demo project to Django 3.1, so unfortunately this might cause some conflicts.
An example of the revised drf-yasg loader class can be found here. Let me know if you need any help.

Ok, I updated the PR and added some tests along the lines you had in place 😄 . So as far as im concerned this one is finished!

Great! I'll review it today 👍

I'm aiming to merge #74 this week and update the docs, for which I could also add some info about this implementation.
If it's OK with you, I'll wait to push this to pypi until that's done, and it will be released as a part of v2.1.0.

sure thing, sounds good. Cheers :)


Update travis.ci config to launch a mongodb database

Removed travis.ci due to lack of webdriverjs support.


Which packages to use for solc v0.4?

This is more of a support Q but figured you may be able to help out.
https://forum.openzeppelin.com/t/how-to-use-solidity-v4-with-openzeppelin-upgrades/3727

As I replied on the forum, for Solidity 0.4 you can install<EMAIL_ADDRESS>which doesn't have a peer dependency on zos-lib.


LFCs on Raw KEGG Map

Hi @noriakis,
Is it possible to use this package to visualize LFCs for target genes on raw KEGG Maps? For example, this image:

I already have a list of reaction numbers and corresponding LFCs for the genes I'm interested in, I just don't know how to write a code that can achieve this

Hi @Ramirj, I have made highlight_entities accept named numeric vector in devel and main branch.
So you can do something like:
vecs <- c(-1, 1) |> setNames(c("CDKN2A","CDC45"))
highlight_entities("hsa04110", vecs, legend_name="interesting")

Make sure you specify which column to search for by name argument. In you case I think it should be reaction.

Hi @noriakis,
Thanks for the help, I read through the documentation and was able to figure out a way to visualize LFCs on raw KEGG maps. I just had to add the overlay_raw_map function to my original code. Here's the code I made in case anyone else has a similar issue:
Make pathway diagram with LFCs for genes present in lab data
Create a map of pathway for Oryza Sativa + map EC number labels
library(ggkegg)
library(BiocFileCache)
Fetch and cache RN to EC map
url <- paste0("https://rest.kegg.jp/link/reaction/ec")
bfc <- BiocFileCache()
path <- bfcrpath(bfc, url)
convert <- data.frame(data.table::fread(path, header = FALSE, sep = "\t"))
rntoec <- convert$V1 |>
strsplit(":") |>
vapply("[", 2, FUN.VALUE = "a") |>
setNames(convert$V2)
Map the EC number to gene nodes
g <- pathway("dosa00010") |>             #change depending on pathway you want to map
mutate(ec = rntoec[reaction])
Define a vector of node names to highlight
nodes_to_highlight <- reaction
Add LFCs
Create a vector containing log fold changes
LFC_vector <- LFCs
Add log fold changes to the graph
g <- g %>%
mutate(LFC = ifelse(reaction %in% nodes_to_highlight, LFC_vector, NA))
Add color gradient to visualize LFCs
gg <- g |>
filter(type %in% c("compound", "gene")) |>
ggraph(layout = "manual", x = x, y = y) +
geom_node_rect(aes(fill = LFC), size = 5) +
scale_fill_viridis_c(option = "viridis", direction = 1, begin = 0.60, end = 1) + # Customize color scale
overlay_raw_map("dosa00010")+
theme_void()
gg
Save diagram as png file
ggsave("whatever.png", gg, width = 10, height = 6, units = "in", dpi = 300)


Fix mnist download for single GPU

When dist is not initialized, the context manager should yield instead of return

@ravi-mosaicml is there a use case where you have multiple ranks but dist is not initialized?

@ravi-mosaicml is there a use case where you have multiple ranks but dist is not initialized?

Yes, if using this function before dist.initialize() is called -- it's an easy mistake to make, especially if a user wants to write some dist code before using this function.
I incorporated this fix in https://github.com/mosaicml/composer/pull/1218/files#diff-cb8a6849a875ed68d93fc6beca6a6f24f7d0e520b87509433a4b94ce4bf09866, so going to close this PR, since it would now be 0-line diff.


TypeError: Cannot read property '_bound' of undefined

Here's the trace:
TypeError: Cannot read property '_bound' of undefined
at HTMLButtonElement.obj.removeEventListener (http://localhost:9001/bower_components/zone/zone.js:296:19)
at http://localhost:9001/bower_components/angular/angular.js:2274:46
at http://localhost:9001/bower_components/angular/angular.js:2429:9
at Array.forEach (native)
at forEach (http://localhost:9001/bower_components/angular/angular.js:320:11)
at jqLiteOff (http://localhost:9001/bower_components/angular/angular.js:2427:5)
at Object.JQLite.(anonymous function) (http://localhost:9001/bower_components/angular/angular.js:3099:17)
at Object.fn (http://localhost:9001/app/directives/helper.js:133:29)
at Scope.$digest (http://localhost:9001/bower_components/angular/angular.js:12533:29)
at Scope.$apply (http://localhost:9001/bower_components/angular/angular.js:12798:24)

@irthos I'm closing this issue as we can not reproduce it. Feel free to re-open if you can add more information.

Sorry to forget about this. We were not able to repro this again either. Thanks.


Explaination needed.

It would be great if you could explain a bit about your project and files.

Sorry for the late reply, I'll add an explanation soon!


razzle build with TypeScript plugin outputs compiler errors

I have a working razzle project with the TypeScript plugin, however when I build the production bundle, although the bundle is built, I also get a bunch of compiler errors. E.g.

/Users/harry/Code/razzle-ts/src/App.tsx
ERROR in /Users/harry/Code/razzle-ts/src/App.tsx(2,19):
TS7016: Could not find a declaration file for module 'react-router-dom/Route'. '/Users/harry/Code/razzle-ts/node_modules/react-router-dom/Route.js' implicitly has an 'any' type.
Try npm install @types/react-router-dom if it exists or add a new declaration (.d.ts) file containing declare module 'react-router-dom';

Declarations are installed and the project works fine when running yarn start

It seems I actually had some compile errors and they weren't being emitted by the server during dev.


Raycasts hitting from inside: return collision point in global coordinates

Fixes https://github.com/godotengine/godot/issues/59335 for both 2D and 3D.

Thanks!


[WIP] Collect correct arguments when setproctitle is called

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area driver-kmod
/area driver-bpf
/area driver-modern-bpf
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
Some days ago we faced this strange thing in the PR regarding e2e tests (https://github.com/falcosecurity/libs/pull/967#discussion_r1133483599).
The question was, why the modern probe is taking something different respect than the other 2 drivers? the answer is pretty funny.
nginx under the hood calls the setproctitle method, what this method does is quite strange it moves the actual args of the process from mm->arg_start to mm->arg_env, so into the environment variables space, and overwrite the content of mm->arg_start with the "process title". In our case the modern probe prints:
nginx: master process nginx -g daemon off;

this is because it tries to read a string until the first \0 is faced, in this case since args memory and env memory are contiguous we are reading both the process title (nginx: master process) and the exe+args (nginx -g daemon off). The other 2 drivers try to read only the args memory and for this reason, we read just the process title nginx: master process instead of real exe and args.
This patch tries to solve this issue in all three drivers but we have some concerns:

in the old probe this part of the code is probably the most fragile, so I had to rewrite it :( it is still too complex for some kernels like 4.14 but I can simplify it a little bit! Btw this is always the same topic we have here https://github.com/falcosecurity/libs/issues/940, changes like these that touch fillers like proc_startupdate are very dangerous from the stability point of view. This is why we were evaluating a partial refactor...
The actual patch takes inspiration from there https://elixir.bootlin.com/linux/latest/source/mm/util.c#L965 (this is a kernel helper that tries to address the issue of setproctitle function). What this function does is check if there is a null terminator at the end of the args memory, if no, it considers this area modified by setproctitle and checks for the real args into the env memory. BTW it could happen that for some reason args memory misses the final terminator, this would mean that we read the env memory even if  setproctitle was not called and so we read junk...take a look at the forkX_father_setproctitle test in drivers_tests

Before proceeding in fixing the verifier losing some days (:laughing:) we would be sure that this is what we want :) WDYT @LucaGuerra @Molter73 @incertum ?
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Wow extraordinary research work :heart: , need to look more into it, hence only posting some initial thoughts:

How common is setproctitle usage? Any ideas?
Yes, we should invest into the improvement because exe, exepath, proc name and args and env and the like are all super important fields for many Falco rules
Initial concern is around what you mentioned that we could "read the env memory even if setproctitle was not called" ... curious if we could find some smarter checks

How common is setproctitle usage? Any ideas?

Not sure about that but as we saw from e2e tests at least nginx use it :/ in that case, we were collecting the proc title nginx: master process

Yes, we should invest into the improvement because exe, exepath, proc name and args and env and the like are all super important fields for many Falco rules

agree with that :+1:

Initial concern is around what you mentioned that we could "read the env memory even if setproctitle was not called" ... curious if we could find some smarter checks

we will try to find something else :mag:

Maybe in Dec or early next year we could revive this discussion?

Just checking in here: How do we all feel wrt prioritization? Since this PR has been created a while ago.

No super priority at the moment IMO

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten


Upgrade Dependencies

I'm curious, what is your intention with this? Seems like killercup hasn't ever published this and I'm assuming this still has a ways to go and would become trivial after clap-rs/clap#2914 (otherwise, its a pain to get everything right, see clap_mangen for how far we still have to go)

nothing serious. i am coding a small school project in rust and wondered if there is a way to generate a simple readme.md file from my command line arguments.
once i stumbled upon this repo i found out it does not work because of outdated dependencies.
since i have fixed it for my own usage i thought it will be nice to push it here.


No Window.FlowDirection support for desktop applications

WinUI XAML does not support FlowDirection or equivalent attribute on the Window element. Additionally, the Window class does not contain a FlowDirection property for programmatic access.
This impacts developers migrating from WPF XAML or WinRT XAML and overall global readiness.
Potential workaround:
// ...
var native_window = managed_window.As<IWindowNative>();
var extended_style = GetWindowLong(native_window.WindowHandle, GWL_EXSTYLE);
SetWindowLong(native_window.WindowHandle, GWL_EXSTYLE, extended_style | WS_EX_LAYOUTRTL);
managed_window.Activate();
// ...

Steps to reproduce the bug

File > New > WinUI 3 blank desktop packaged application
Attempt to add FlowDirection=FlowDirection.RightToLeft attribute to root Window element in MainWindow.xaml
Observe error

Expected behavior
Easy way to enable RTL support via XAML and programmatically.
Version Info
NuGet package version: Microsoft.WinUI 3.0.0-preview4.210210.4
Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (21313)
Yes

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

@codendone FYI

@Austin-Lamb FYI

Marking this as a feature proposal and throwing it into the freezer is very disappointing.
This is missing i18n functionality, something Microsoft would once consider a blocker to shipping. Everyone that wants to support RTL will now have to have to manually plumb in the necessary interfaces and P/Invoke signatures (C#/Win32 does not support WinUI) to support something supported in WinForms over a decade ago. WinUI Preview 3 didn't even properly support WS_EX_LAYOUTRTL, so I can only imagine what bugs lie in wait.

@riverar, you are right this is basic funtionality that should be there, but it requires a new APIs, and new APIs is a feature request.

Hi, is there any workaround for this that can be used from C#?

@marb2000 FYI

any news on this item?

Any news?


Feature/#78 geojson export

Moved all code to one class.
Added Circle and Ellipse export.
Added single point MilStdSymbol export.
Fixed matching braces.  Removed redundant brackets.
Fixed stroke color class.

To test, build branch.  Start test basic and from the test menu, draw some shapes.  For example, circle, polygon, line.
Then from the menu, select Export Feature to GeoJSON.  The export string is shown in the log.

What's the issue number in source tree
-----Original Message-----
From: rsupnekar<EMAIL_ADDRESS>Sent: Monday, March 20, 2017 12:13 PM
To: missioncommand/emp3-android<EMAIL_ADDRESS>Cc: Khan, Mubina I CIV USARMY SEC (US<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [Non-DoD Source] Re: [missioncommand/emp3-android] Feature/#78 geojson export (#83)
All active links contained in this email were disabled. Please verify the identity of the sender, and confirm the authenticity of all links contained within the message prior to copying and pasting the address to a Web browser.

Assigned #83 < Caution-https://github.com/missioncommand/emp3-android/pull/83 >  to @mubinakhan < Caution-https://github.com/mubinakhan > .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub < Caution-https://github.com/missioncommand/emp3-android/pull/83#event-1006996684 > , or mute the thread < Caution-https://github.com/notifications/unsubscribe-auth/AYTgg5p1mYtDKN7GkqhwrVarbfDreViZks5rnqWUgaJpZM4Mg9AS > . Caution-https://github.com/notifications/beacon/AYTgg1H4b9aot7PmbL8d3ZIyWZLhIWBPks5rnqWUgaJpZM4Mg9AS.gif

Reintegrate


[std.regex] Bit-NFA kickstart engine

This adds an alternative "kickstart" engine to locate a prefix of a given pattern.
It is far more general the the original ShiftOr while a being few times slower,
still it's many times faster then any full-blow engine.
UPDATE: it's even ~x2 times faster then grep! Now that's some news :)

Win32 is blocked by https://issues.dlang.org/show_bug.cgi?id=15989

Now with immutable regexes!

Looks like you have a typo in the issue number. 9381 isn't about regex.

Looks like you have a typo in the issue number. 9381 isn't about regex.

Right my bad issue is 9391.

All green!

@DmitryOlshansky by the new flow you can merge your work now, optionally with follow-up work after the review.

@andralex Will try to resolve the issues first then merge.

Auto-merge toggled on

Seems like this caused a regression w/ building Higgs.
Issue 16626 – [Reg 2.073] recent dmd nightly runs out of memory building Higgs.

Seems like this caused a regression w/ building Higgs.
Issue 16626 – [Reg 2.073] recent dmd nightly runs out of memory building Higgs.

The new version is more demanding on memory during CTFE so yeah, most likely it now fails for some patterns. Dunno what to do here would really love to see the new engine sometime soon.

Seems to me we should disable that engine in CTFE for now.
Let's continue the discussion in Bugzilla https://issues.dlang.org/show_bug.cgi?id=16626#c2.

Seems to have caused another regression.
Issue 17066 – [REG2.073a] std.regex captures got immutable

@MartinNowak will look into it, though sounds highly suspicious.

@MartinNowak  https://github.com/dlang/phobos/pull/5042


How to remove resource from state manually

There is an OriginAccessControl for AWS CloudFront that was accidentally deleted on AWS Console.
Because this was part of an old Nextjs app that was removed from sst.config.ts, when I run sst deploy it tries to remove it but fails with error 404 since it can't find on AWS:

operation error CloudFront: GetOriginAccessControl, https response error StatusCode: 404, RequestID: 510077df-37af-433c-93d7-4ba8a7ccead9, NoSuchOriginAccessControl: The specified origin access control does not exist.

I dig into the prod.json in the pulumi files and found its URN. Then I tried sst remove --target <urn> but it also fails with the exact same 404 error.
sst refresh also doesn't work, it still keeps the resource in the state as if it existed on AWS (though it doesn't).
How can I let SST know this resource no longer exists?

Aside from removing directly from the state file, @thdxr is there another way?

👋 I have the same issue, isn't there any CLI command to achieve that ? I have the error on a Next.js project where the bucket that stores my asset files have been deleted manually.
I guess it would be something similar to terraform state rm ?


chore: code owners should be in one line

Codecov Report

Merging #260 (df82a3d) into main (6a6b192) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #260   +/-   ##
=======================================
  Coverage   59.09%   59.09%
=======================================
  Files          42       42
  Lines        2596     2596
=======================================
  Hits         1534     1534
  Misses       1062     1062

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 6a6b192...df82a3d. Read the comment docs.

:tada: This PR is included in version 5.5.3 :tada:
The release is available on:

v5.5.3
GitHub release

Your semantic-release bot :package::rocket:


Add "export" option to pm

Add export functionality to pm so that content can be shared.
$ dtf pm export

This will generate the XML for the "manifest.xml", and zip all content up.

Added: https://github.com/jakev/dtf/commit/2c45e6ba111e0d2a011723041fc74e26710ee5a8


Diff support for color.ui = always

When color.ui is always, the diffs are broken :
git config --global color.ui always

git config --global color.ui auto

Nice, thank you :+1:


Requirements have not been satisfied properly.

http://imgur.com/a/5lJG9
here is a image if issue which i am getting please help me to solve this out.
Thank You in advance.

ERROR: Requirements have not been satisfied properly. Please look at the README file for configuration instructions.

If you still cannot resolve this error, please submit an issue here:
	https://github.com/k4m4/kickthemout/issues

Following these steps to solve it
https://github.com/k4m4/kickthemout/issues/126#issuecomment-320682721

Hi there. Have you followed the installation process in the README file? Thanks.

Hi there. Have you followed the installation process in the README file? Thanks.

Sorry, I followed it but I've faced the error with scapy package, I can't install it via brew, so I had to manually install it.

Good to hear that. @hieuhuynh93
@biggersmilesj If you are still experiencing problems, be sure to re-open the issue. Thanks!


Cross-compiling lxd through go build instead of go install

When cross-compiling with go, go build must be used instead of go install: https://dave.cheney.net/2015/08/22/cross-compilation-with-go-1-5.
Adding a CROSS_COMPILE variable to the Makefile to change the default "go install" to "go build" would be nice.
What do you think of this proposal? Perhaps, this feature should not only be linked to cross-compilation?

LXD can't be cross compiled due to it using cgo on most platforms.

LXD can be cross-compiled, we just need to enable cgo an give it a cross-compiler by setting CC_FOR_TARGET. Indeed, I was planning to add it to buildroot. runc and docker have been added recently to buildroot and they both used cgo like LXD. I think it would be nice to also add a buildroot package for LXD. I can made a patch for buildroot only but it would be better to have it upstream.

Ok. Well, send a pull request and we can take it from there.
I'd probably prefer you make two new targets, "cross" and "client-cross" or something like that, rather than have the existing targets do completely different things based on an env variable (since go install will install in $GOPATH/bin whereas go build will dump the binary in the current working directory).

Closing this issue as I don't really think that anyone who's in the business of cross-building will have much of a problem typing "go build -v github.com/lxc/lxd/lxc" instead of "make client-build" and there's also a problem with adding those two Makefile targets. "lxc" and "lxd" are both directories in the git tree. So calling "go build" from within the Makefile's location won't work as the resulting paths already exist.

Can you please add the  lxd makefile for buildroot environment


Pipenv locking taking forever

$ pipenv install
Pipfile.lock (b9c44e) out of date, updating to (ec4b23)...
Locking [dev-packages] dependencies...

Then it gets stuck for quite a while.

I'll take a look at pinning some of the package versions.


Extract ABE class

The abe class should be moved to a different repository and made available through composer. This provides better dependency management and removes the need to update the gist.

I have decided to integrate the class further by writing classes for each row-type. This means that the ABE gist


gbk Chinese filename Garbled err... in Nodejs16

is there anyway to set decode/encode 'UTF-8/GBK'?

{
name: 'test1/- +-+-.txt',
flags: { encrypted: false, solid: false, directory: false },
packSize: 2044,
unpSize: 5256,
crc:<PHONE_NUMBER>,
time: '2022-07-24T04:19:12.000',
unpVer: '5.0',
method: 'Normal',
comment: ''
}

Sorry it cannot be supported.
The OEM charset support is done by Windows API but node-unrar.js is based on the linux version of unrardll which OEM charset is not supported.
抱歉无法支持GBK/GB2312编码。
Unrar的本地（OEM）编码支持是由Windows API提供的，node-unrar.js是基于Linux版本的unrardll编译的，无法提供对本地（OEM）字符集的支持。


🐞[bug]: Navbar after scroll

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
when we scroll down on the screen the navbar kind of gets mixed , i would like to improve the navbar and add some svgs to it
Expected behavior
I think the navbar should be distinct and should not get mixed with the rest of the screen and it should be eye-catching
Add ScreenShots
No response
What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@KajalDeore04 go ahead.


No '…' shows next to 'Break Lease & Change Access Tier' for ADLS Gen2 blobs in Chinese

Storage Explorer Version: 1.11.1
Build: 20191203.1
Branch: master
Language: Chinese(zh-CN) / Chinese(zh-TW)
Platform/OS: Windows 10/ Linux Ubuntu 18.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select '中文 (简体)' -> Restart Storage Explorer.
Expand an ADLS Gen2 account -> Blob Containers.
Create a new blob container -> Upload a blob.
Check the localized actions 'Break Lease & Change Access Tier' on context menu.

Expect Experience:
Shows '...' next to 'Break Lease & Change Access Tier'.
Actual Experience:
No '…' shows next to 'Break Lease & Change Access Tier'.

More Info:
ENU screenshot:

      <Val><![CDATA[Change Access Tier...]]></Val>
      <Tgt Cat="Text" Stat="Loc" Orig="New">
        <Val><![CDATA[更改访问权限层...]]></Val>

https://devdiv.visualstudio.com/DevDiv/_git/StorageExplorer?path=%2Floc%2Floc%2Fzh-Hans%2Fsrc%2Fcomponents%2Fse-adls2-extension%2Fresources%2FFileSystemEditor.json.lcl&version=GBmaster
https://devdiv.visualstudio.com/DevDiv/_git/StorageExplorer?path=%2Floc%2Floc%2Fzh-Hans%2Fsrc%2Fcomponents%2Fse-blob-extension%2Fresources%2FBlobEditor.json.lcl&version=GBmaster
      <Val><![CDATA[Break Lease...]]></Val>
      <Tgt Cat="Text" Stat="Loc" Orig="New">
        <Val><![CDATA[中断租约...]]></Val>

https://devdiv.visualstudio.com/DevDiv/_git/StorageExplorer?path=%2Floc%2Floc%2Fzh-Hans%2Fsrc%2Fcomponents%2Fse-blob-extension%2Fresources%2FBlobEditor.json.lcl&version=GBmaster
https://devdiv.visualstudio.com/DevDiv/_git/StorageExplorer?path=%2Floc%2Floc%2Fzh-Hans%2Fsrc%2Fcomponents%2Fse-adls2-extension%2Fresources%2FFileSystemEditor.json.lcl&version=GBmaster

This issue has been fixed on build 20191205.1. So we close it.


[FEATURE] support graceful shutdown for grpc server

Code of Conduct

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

Search before asking

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

Describe the feature
Refer to: https://github.com/hyperium/tonic/pull/169
Motivation
No response
Describe the solution
No response
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Hi @zuston I am the fresher regarding rust, can I try to this task if no one is interested?

@wenlongbrother assigned to you.

Hi @jerqi here is pr#1292, could you please take a look? Thanks


graphical and format updates for workflow steps on home page

Stop gap formatting updates to the "Typical Workflow .." on the docs home page
I know Design will give us the new graphical version of this soon, but in the meantime I thought it would be nice to spruce it up. Here is what I've done:

Added new formatting and graphical circled images (embedded CSS for now)

Simplified, shortened up some of the text

Updated one or two of the links (e.g., "scale a swarm" now goes directly to a topic in the tutorial on how to scale a service).

Let me know if you think this is going in the right direction. You will need to look at it on the Netlify preview to see what it actually looks like.
@johndmulhausen @Raphaelbouchard @mstanleyjones @thaJeztah
Signed-off-by: Victoria Bialas<EMAIL_ADDRESS>

LGTM, merging, please send more suggestions and we can iterate. @Raphaelbouchard  I think we should use the text/content updates here for the new Design version of this when it's ready. We will also need to create pages for the links in your new model.


DateInRegion to Date object in version 4.0.2

Could you please advise how do we get Date object from DateInRegion.
Basically in previous version, I could see we had Date overload.
Date(year: 2020, month: 12, day: 31)
But I could not see any property or method to get Date object from SwiftDate DateInRegion in version SwiftDate 4.0.2.
Please advise.

Hi @dhamukrish see #267


When importing, if the second column is not the user name then the UI doesn't show the assignments correctly.

Because comments are now allowed to be imported without a corresponding assignment, if the second column is not the user's name, then it gets picked up as the user name.
There needs to be better validation around the first and second columns. See attached.
gradebook_export-1292d9e5-492b-44d9-abf2-79aa1242c2a1.csv.zip

Check if second column looks like one of the other fields we check for and just move on if there is no name.

Actually, an assignment without points would look like the student name, so it may need to be based on validation of the contents of the column.


Upgrade to telemetry 1.2

Hello! I just watched the elixirconf talk, very cool stuff. I'm doing a small PoC using pacer to evaluate it, and found that when trying to add it to an existing code base I got a version conflict for telemetry.

What does your PR aim to do?
Upgrade to the latest version of the telemetry library

The reason/why for the changes
Avoid conflicts with other libraries that depend on newer versions of telemetry

Validation and verification instructions (how can we verify that your changes are working as expected; if possible please provide one or two code samples that demonstrate the behavior)
Unit tests. I guess could also hook it up to a dashboard and take a look that it still works.

Additional commentary if necessary -- tradeoffs, background context, etc.
N/A

Sounds good, I’ll do that! On 18 Oct 2023, at 20:13, Zack Kayser @.***> wrote:
@zkayser commented on this pull request.
Hey @vorce 👋
Thank you for submitting a PR!
We do absolutely want to support Telemetry 1.0+. However, we are going to need to support a couple of versions to maintain compatibility for users who are on projects where they can't yet upgrade Telemetry for whatever reason.
What do you think about changing the version requirement to use an or expression. I'm thinking the dependency requirements we're going to want to support are going to look pretty much exactly like what Absinthe does for Telemetry https://github.com/absinthe-graphql/absinthe/blob/2ba17d8487b71be86be977ed2219a3b774ca7f45/mix.exs#L77C7-L77C7.
Would you mind making that change in this PR?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@vorce We just published version 0.1.1 with these changes, if you are pulling it from hex.pm it should be available now.


Travis CI is broken

It looks like Travis CI is currently broken, and I can't add this repo, I think only @deanmalmgren can. @deanmalmgren can you go to https://travis-ci.com/ and add this repository? travis-ci.org is apparently now defunct.

I just got textract set up on travis-ci.com. All I can say is WOW did
Travis CI bungle the transition from .org to .com. @traverseda @jpweytjens
you should have access to the Travis CI build now. The build is failing,
but at least Travis CI is working again.
https://travis-ci.com/github/deanmalmgren/textract
Can you take it from here @traverseda?
On Wed, Jul 28, 2021 at 3:31 PM traverseda @.***> wrote:

It looks like Travis CI is currently broken, and I can't add this repo, I
think only @deanmalmgren https://github.com/deanmalmgren can.
@deanmalmgren https://github.com/deanmalmgren can you go to
https://travis-ci.com/ and add this repository? travis-ci.org is
apparently now defunct.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/deanmalmgren/textract/issues/387, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB6NODO5NZK6P4YFTUCPXDT2BSJ7ANCNFSM5BFCUPNA
.

Yep, I think that covers all the steps I couldn't do myself. Thanks!


Sergey, I have wrote to you email, I would like to develop workaround limiting https://www.youtube.com/get_video_info?video_id={videoId}. call. Please stop closing this and communicate. youtube-dl needs refactoring to be able to use with youtube

Current concept of youtube download is completely useless for archiving youtube channel, which is funny, when using name youtube-dl :)
Current version 2020.01.01 downloads webpage a than sequentially downloads video info page and than by dates or name of the video file set by -o selecting, if video should be downloaded or not. This concept leads to too many request error, it is only matter of time, when your ip start to be blocked by google.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', 'bestvideo+bestaudio', '--verbose', '--restrict-filenames', '--playlist-reverse', '-i', '--no-mtime', '-o', '%(uploader)s [%(channel_id)s]/%(upload_date)s - %(title)s [%(id)s].%(ext)s', '-ci', 'https://www.youtube.com/watch?v=VoaY8YRberk']
[debug] Encodings: locale cp1250, fs utf-8, out utf-8, pref cp1250
[debug] youtube-dl version 2020.01.01
[debug] Python version 3.7.0b4 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: ffmpeg git-2019-10-28-68f623d, ffprobe git-2019-10-28-68f623d
[debug] Proxy map: {}
[youtube] VoaY8YRberk: Downloading webpage
[youtube] VoaY8YRberk: Downloading video info webpage
WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests
Using browser (chrome, firefox) same machine same time everything is OK, youtube plays video and show all information.
youtube-dl needs to refactor and use for grabbing video information JSON google API (in which case is possible to pay for increasing quota and free qouta is in ten of thousands videos information querry per day) or have to much more properly simulate real person usage and grab necessary new video info from
https://www.youtube.com/watch?v={youtube video ID}
and list of channel new videos ID grab from
https://www.youtube.com/channel/{youtube channel ID}/videos
Current version (usage of everyday checking of list of channels for new videos and archive them) is broken for youtube, or functioning only few days with luck (i am checking last 5 videos in about 40 channels once per a day and it took about 2-3 weeks to be blocked).
youtube-dl -v -f bestvideo+bestaudio --restrict-filenames --playlist-reverse --playlist-end 5 -i --no-mtime --dateafter now-7days -o "%(uploader)s [%(channel_id)s]/%(upload_date)s - %(title)s [%(id)s].%(ext)s" -ci --batch-file=download.txt

Yep can confirm youtube-dl in the present state is broken for archiving channels. You will get 429s.
I recommend suspending any archival scripts until it is fixed / a workaround is developed, as you might just get your IP blocked fro a very long time.

Same here, there is problem with playing with youtube-dl.
When i'm using Firefox - it is going well - video is playing, when i'm trying to play with youtube-dl i'm getting
root@music:/home/user# youtube-dl https://www.youtube.com/watch?v=Y2u95GYuFUI [youtube] Y2u95GYuFUI: Downloading webpage [youtube] Y2u95GYuFUI: Downloading video info webpage WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests


Remove br-tag and format

Removing extra br tag and run automatic formatter.

 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.Pyhajarvi Maaret 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.


Remove "security" as a type and add a "cve" attribute

"security" is a poor choice to have as a type as it is almost always also a bug that needs to be fixed. However, for bug fixes that are security issues it would be useful to include the "bug id" (i.e. - the CVE number) for the issue. This would make it easier for users to locate the specific changes that corrected a specific issue.

Fixed in 48a66f9bfdbccf83861849a170838e87d05d1d74. As discussed in the Slack channel, we agreed to use issues for CVEs, e.g., <issue id="CVE-2021-44228" url="https://www.cve.org/CVERecord?id=CVE-2021-44228">.


Fix Kinetics dataset docstring

The docstring is using the term "clip" instead of "video".
In the context of the video datasets / io / samplers:

a video is a video file
a clip is a part of a video file

cc @pmeier

It's a pre-existing failure: https://hud.pytorch.org/pr/pytorch/vision/7990
I saw it earlier last week, but couldn't match it to a change in torch core. I was hoping it would just go away but we might need look into it more seriously. In any case, it's not related to this PR.


Measure coverage of test cases

Cf. https://github.com/Componolit/RecordFlux/pull/1181#discussion_r972926563

To measure the coverage over integration and unit tests the tests that are marked compilation or verification will be moved to separate directories. The resulting directory structure will look as follows:
tests/
 - unit/
 - integration/
 - compilation/
   - test_unit.py
   - test_integration.py
 - verification/
   - test_unit.py
   - test_integration.py

To be consistent to the existing files, the filenames should be of the form foo_test.py.


error when creating CRUD struncture

I tried to create CRUD structure using enso:cli, but always I'm getting this error
Please check the attached G-DRIVE link
The menu's route does not match the configured permissions
video
Thanks in advance

The documentation is not up to date yet I think, but it is no longer necessary to add the permission group, just the path as such. In your case you should just write index  in the menu path.

And for future consultations on the same topic, I think it would be appropriate for you to do so on the package repository, for organizational reasons :)

@gandesc please review the docs for cli

@itarafath please use the issue template from now on.

Thanks, @heyner29 @aocneanu
Administration routes do not working except the new one, after CRUD structure created

Yes, that was also discussed here in this thread

@heyner29
Could you please describe this little more.
This is confusing me
Thanks


About the linear space spanned by the input

Thanks for your great work!
https://github.com/liangyanshuo/InfLoRA/blob/2c774547d48c40fe5bb8c4a393f7b370e1664148/models/vit_inflora.py#L237
I have a question regarding this part of the code. In previous papers, the linear space spanned by the input was calculated using randomly sampled inputs. However, in this implementation, the linear space is calculated by averaging the product of the input image tokens matrix with its transpose.
Could you please clarify if this method is an approximation of the input linear space? Additionally, are there any references or proofs?
Thank you!

I am sorry for the late reply.
In a typical linear layer of a Transformer, each input is a token of some input sequence. In other words, each token is an individual input. Therefore,  "linear space spanned by the input token" would be more precise than "linear space spanned by the input". In fact, you can examine the code of GPM for CNNs (https://github.com/sahagobinda/GPM), where the "linear space spanned by the input" should be called "linear space spanned by the input patch" since each input is a patch from some image when you transform a convolution layer to a linear layer.
I hope this clarifies your question.


Don't allocate all the memory up front when receiving incoming sends

I was thinking the same thing. I don't do all of it though, the biggest cost will still be serialization


Use Object.keys for empty object check - improves performance

When using moment together with ChartJS, I found out that there is performance issue with isObjectEmpty  when rendering large datasets. This function takes around 30% of total time required to render the chart.
When using Object.keys, it only takes ~2% of total time using Chrome 57.

I've added hasOwnProperty check and also changed from Object.keys to Object.getOwnPropertyNames which I think is more correct in this case.
I've also created jsperf test (https://jsperf.com/momentjs-isobjectempty-test/1) and apparently for..in loop is slow only in Chrome. Firefox has almost the same speeds for all three methods.

I just ran the jsperf test on my iPad, and the old for-in loop seems to be reliably faster in mobile Safari:

So chrome wins a lot, firefox wins a bit, iPad loses a bit? Chrome is the most popular browser so it makes sense to merge this.

Merged in d0db6976231663d5a8aca19a47cdd7bcaabeb630


How to check logs?

Hi, I have been running some issues that some web requests made by python scripts under the webhook not working. I tried to access the logs, but the journalctl is not giving much info except 500 Internal Server Error. For the record the python scripts run fine standalone.
Is there a log file specific for webhook? I couldn't find it described anywhere in this repo. I will much appreciate any help.

Hi @CarlosEvo running in non-interactive mode I;ve been using
journalctl -u webhook
And while it is light on the logs it should give some info, have you tried hitting the webhook directly from a web browser to see what happens?

Webhook will log to the stdout/stderr if you use the -verbose flag when starting it up. So make sure to redirect that output to a file of your choice.
If the webhook is running as a service, make sure that it is started in the verbose mode (with the -verbose flag) and check out the journal logs.

@hassanbabaie I did used jornalctl -u webhook to debug. Unfortunately my hook requires POST data, so I can't really debug it with a browser (Zapier doesn't help either). It is more of a python script problem. Nonetheless @adnanh helped me a lot with the -verbose option.
Thank you both, really appreciate it.

Glad to hear that :-) Cheers!


Bug in [doctest] mimesis.utils.pull

There is a bug in [doctest] mimesis.utils.pull:
Expected:
    'Mon.'
Got:
    'Fri.'

I'm going to deal with it

@Valerievich Can you please add more information about how to reproduce this bug?

Sure! Just run pytest --doctest-modules and you'll see DocTestFailure

Can't start fixing this. My PR hasn't approved yet.

@Valerievich merged! You can start fixing this.

@lk-geimfari Since we've merged PR we could close this related issue.


coordinates inside displacement function

Hi,
I would like to generalize displacement(x, y, z) such that ux and uy are read from a list istead of being
a function of (x,y,z), but I realized that
x,y,z and ux and uy have a dimension larger than the number of atoms.
I would expect that the modifier acts on the coordinates of the unstrained model,
which have coordinates model.system.x with len(model.system.x) = number of atoms.
I would be glad if you could clarify what x,y,z are and why its values and lengths are different from the unmodified model, in order to understand how to read
ux, uy from an external list.
In the following the scipt where the issue is exemplified,
the aim is to substitute ux with ux_ext inside the function.
Thank you for any advice!
modifier-gen.txt

The onsite_modifier-function will compute the changed positions first, and only then take the appropriate shape. The larger number of sites comes from all the other sites that are first computed, but lie outside the polygon. The function is also called twice, for the A and B sublattice of graphene. The original, distorted and all the sites before the cutout are shown in following figure:

We are working on a better implementation of the model class such that this could be used to read out the updated positions from a .xyz-file. This work is still at the first stages, but this will help with the problem you are encountering. Right now, it is harder to tackle this problem. Right now, I just suggest you to print out the values and the sublattice that they correspond to, then, you can change the values manually for that system.
`


committed into hacktoberfest

please the check it and merge it with your code.

Doesn't seem relevant.


Rollup fails with Unexpected token error while parsing json. (node_modules/libphonenumber-js/metadata.min.json)

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ x] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior

While running  , json file throws unexpected token.
Expected behavior

There shouldn't be any parse error for json file.
Minimal reproduction of the problem with instructions

Add ng2-validation, which has libphonenumber-js as dependency.  Use validation api in the code.  Then try to run rollup.
What is the motivation / use case for changing the behavior?

Somehow json parsing is not working.
Please tell us about your environment:

MacOS High Sierra, VSC, NPM

Angular Seed Version: aaaaf75

5.0.1

Node: node --version = v7.1.0

For support requests you can use gitter or take a look at project's the wiki page in order to get more complete idea on how the it works.


add warning message if there are unsaved element changes

Fix #154
This PR enhances the tree. If the user adds/deletes/edits the properties of an element and clicks on another element, deselects an element or clicks on a geodata object a warning popup is displayed. The user has three options:

Discard changes and switch to the other element or whatever the user wanted to do
Save changes and then complete the users action
Cancel

I'm still looking for a solution for the problem that a user can click on a popup and the popup opens in the background independent of the users choice (1., 2. or 3.). But you can test the current state and give some feedback @eScienceCenter/spacialists

Adding new Objects like contexts and artifacts through right-click is still possible. This should also trigger the warning popup.

I added the check for unsaved changes to the right-click/context menu and the 'create new top-context' button as well. Could you please test again, @dirkseidensticker ?


⚠ Fox Sports has degraded performance

In 35f66c5, Fox Sports (https://foxsports.com) experienced degraded performance:

HTTP code: 200
Response time: 2031 ms

Resolved: Fox Sports performance has improved in f4dc8c7.


Update front page with version info

As a result of the already fixed bug I filed on Thursday I just became aware that the homepage is lagging.
On the http://tile38.com/ page there are some easy to copy examples on how to install tile38. Until now I was under the impression that the version listed in these examples, 1.7.5, was the latest as there was no other info on the front page (there is the link to the github release page but no version info)
Please, either keep these copy&paste links up to date or put a generic version string e.g tile38-<version.number.string>.tar.gz so it is clear that these are just examples.

Thanks for pointing this out. I wasn't aware that the homepage was lagging. The page now has the corrected version.


[T5A1][W09-A2]Lee Pei Xuan

@pyxyyy Some comments added. Please close the PR after reading comments.


.NET Core 2.0 support

I saw Nuget page:
https://www.nuget.org/packages/AngleSharp/
It seems that version 0.9.9 is the latest version, and the way I understood it only supports .NET Core 1.0.
Do you have a plan to support .NET Core 2.0 also? Because our entire platform is in .NET Core 2.0, and based on an almost huge R&D we found this library to be awesome and we want to use it.

.NET Core 2.0 is a superset of 1.0 and thus this should work automatically if I am not mistaken. What problem are you running into?


Update README.md

I made adjustments to README file reflecting some new changes in the new script.
I also updated main.py with new features and functions so SCiMS uses Bayesian statistics to infer host sex.
New unit tests.
New test folder containing required input files for running an example.
Removed some of the files no longer needed in the new scripts (not sure if this is a good idea, but I'm learning :D), please give feedback!

For some reason, my output files has "np.float64 is a lot of places"
sample-id	actual_sex	SCiMS sample ID	Total reads mapped	Total reads mapped X/Z	Total reads mapped Y/W	SCiMS predicted sex	Posterior probability of being male	Posterior probability of being female	Status	Rx	Rx 95% CI	Ry	Ry 95% CI
S28M1000000	male	S28M1000000	1563190	38420	5464	male	0.9999280321267194	7.196787328056509e-05	Success	1.0239	(np.float64(1.048), np.float64(1.0))	0.1245	(np.float64(0.121), np.float64(0.128))
S56F150	female	S56F150	238	12	0	female	0.0	1.0	Success	0.9785	(np.float64(1.081), np.float64(0.876))	0.0	(np.float64(0.0), np.float64(0.0))
S79F300	female	S79F300	466	20	0	female	0.0	1.0	Success	1.0197	(np.float64(1.073), np.float64(0.966))	0.0	(np.float64(0.0), np.float64(0.0))
S90M250	male	S90M250	379	11	0	female	0.0	1.0	Success	0.9755	(np.float64(1.079), np.float64(0.872))	0.0	(np.float64(0.0), np.float64(0.0))

Thanks for checking, Kobie! I've updated the main script. Could you try again and see if the np.float64 are gone?

Thanks for checking, Kobie! I've updated the main script. Could you try again and see if the np.float64 are gone?

Sure thing! It looks like that fixed it on my end:
sample-id	actual_sex	SCiMS sample ID	Total reads mapped	Total reads mapped X/Z	Total reads mapped Y/W	SCiMS predicted sex	Posterior probability of being male	Posterior probability of being female	Status	Rx	Rx 95% CI	Ry	Ry 95% CI
S28M1000000	male	S28M1000000	1563190	38420	5464	male	0.9999	0.0001	Success	1.0239	(1.048, 1.0)	0.1245	(0.121, 0.128)
S56F150	female	S56F150	238	12	0	female	0.0	1.0	Success	0.9785	(1.081, 0.876)	0.0	(0.0, 0.0)
S79F300	female	S79F300	466	20	0	female	0.0	1.0	Success	1.0197	(1.073, 0.966)	0.0	(0.0, 0.0)
S90M250	male	S90M250	379	11	0	female	0.0	1.0	Success	0.9755	(1.079, 0.872)	0.0	(0.0, 0.0)

I noticed that, if there is weird stuff in the metadata file like for example if two columns accidentally got merged:
sample-id	actual_sex
S28M1000000	male
S56F150	female
S79F300	female
S90M250male
We still get an output:
sample-id	actual_sex	SCiMS sample ID	Total reads mapped	Total reads mapped X/Z	Total reads mapped Y/W	SCiMS predicted sex	Posterior probability of being male	Posterior probability of being female	Status	Rx	Rx 95% CI	Ry	Ry 95% CI
S28M1000000	male	S28M1000000	1563190	38420	5464	male	0.9999	0.0001	Success	1.0239	(1.048, 1.0)	0.1245	(0.121, 0.128)
S56F150	female	S56F150	238	12	0	female	0.0	1.0	Success	0.9785	(1.081, 0.876)	0.0	(0.0, 0.0)
S79F300	female	S79F300	466	20	0	female	0.0	1.0	Success	1.0197	(1.073, 0.966)	0.0	(0.0, 0.0)
S90M250male		S90M250male	379	11	0	female	0.0	1.0	Success	0.9755	(1.079, 0.872)	0.0	(0.0, 0.0)
Should there be a check to make sure there are no strange things happening with the metadata file?

I noticed that, if there is weird stuff in the metadata file like for example if two columns accidentally got merged: sample-id actual_sex S28M1000000 male S56F150 female S79F300 female S90M250male
We still get an output: sample-id actual_sex SCiMS sample ID Total reads mapped Total reads mapped X/Z Total reads mapped Y/W SCiMS predicted sex Posterior probability of being male Posterior probability of being female Status Rx Rx 95% CI Ry Ry 95% CI S28M1000000 male S28M1000000 1563190 38420 5464 male 0.9999 0.0001 Success 1.0239 (1.048, 1.0) 0.1245 (0.121, 0.128) S56F150 female S56F150 238 12 0 female 0.0 1.0 Success 0.9785 (1.081, 0.876) 0.0 (0.0, 0.0) S79F300 female S79F300 466 20 0 female 0.0 1.0 Success 1.0197 (1.073, 0.966) 0.0 (0.0, 0.0) S90M250male S90M250male 379 11 0 female 0.0 1.0 Success 0.9755 (1.079, 0.872) 0.0 (0.0, 0.0)
Should there be a check to make sure there are no strange things happening with the metadata file?

That's a good point! I updated it so that if the prefix from idxstats doesn't match with the respective ID in metadata, it will raise an error. So we need to make users are aware that their idxstats prefix should be the same as their sample id in metadata table.


neovimUtils.grammarToPlugin: improve error message on invalid grammarPlugins

added error message for when treesitter overlay authors mess up
addresses https://github.com/NixOS/nixpkgs/issues/356274
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Can you squash the commits, too, please.

No problem. Confusing error messages are a common complaint so it seemed like a good addition
Honestly thank PerchunPak because the fix to the grammars is a big relief. I no longer need to sloppily hotfix them in nixCats anymore just to make sure some of them dont throw random errors
Least I could do was fix the neorg overlay and add the message XD

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 357062

x86_64-linux

aarch64-linux

x86_64-darwin

aarch64-darwin

Also theyre talking about reworking how host providers will work in the future for nvim? I dont know much about it, is that something we are going to have to think about? Currently we add those by just setting the host variables like this in neovim/utils.nix and then we also do this thing where we generate some sort of manifest thing? Will these still be the way to do it? Id be interested to know because I replicate that logic in nixCats as well and its the only thing I dont fully understand. I know what its for and why its there sorta but also not enough to explain it to someone. So really if anyone has any knowledge to drop on that let me know here or on discord lol

CC @teto
(btw we have a matrix room for this: https://matrix.to/#/#nixpkgs-vim:nixos.org)

I stole my own username and cant figure out how to link that account to this one XD


[BUG] Moc checks for Powershell Remoting are incorrect in assumptions about single-node availability

Describe the bug
When you run Initialize-AksHciNode or Set-AksHciConfig, the Moc cmdlet has to do some stuff to determine if a node has Powershell Remoting enabled:
The first step is to check and enable Powershell Remoting (Common.psm1)...
function Enable-Remoting
{
    <#
    .DESCRIPTION
        Enables powershell remoting on the local machine.
    #>

    Write-Status $($GenericLocMessage.comm_ps_remote) -moduleName $global:MocModule

    Enable-PSRemoting -Force -Confirm:$false
    winrm quickconfig -q -force
}

And then after it needs to test it (Moc.psm1)...
function Test-Remoting
{
    <#
    .DESCRIPTION
        Validate that powershell remoting to a node is working.

    .PARAMETER nodeName
        The node to remote to.
    #>

    param (
        [String]$nodeName
    )

    try
    {
        Invoke-Command -ComputerName $nodeName -ScriptBlock { return $null } -ErrorAction Stop
        return $true
    } catch {}

    return $false
}

The problem is this logic assumes that on a single-node system, you can Invoke-Command against yourself without it throwing an error and thus you receive a false-negative. This specifically happens when CredSSP hardening is configured and you are not specifying an FQDN to yourself.
As an example, here are screenshots proving that the "raw" commands to enable remoting work fine on the single-node:

Invoke-Command from a remote server works fine:

But Invoke-Command with shortname to itself fails:

And Invoke-Command with FQDN to itself succeeds:

And to no surprise, specifying shortname and a fresh set of creds also works perfectly fine:

Why this happens makes perfect sense if you run Get-WSManCredSSP. Only the FQDN of the "local machine" is specified in WSMan for delegating creds:

Other than hacking up the Powershell module to use FQDN, I am unsure how to proceed here.

Well, after smashing my head against a keyboard for awhile, it appears that this user's Hyper-V host was misconfigured to use incorrect DNS servers that couldn't resolve shortnames, but obviously if the FQDN was specified, it conditionally-forwarded and resolved correctly.
I'm still not 100% convinced that my issue is self-inflicted, but you know what they say:

Let me tweak a few more things and I'll confirm if this should be closed or not. Stay tuned!

Negative, def not a DNS problem on my side. Brand new server with correct DNS, the same error occurs:

I just edited Moc, Kva, and AksHci modules and replaced $env:ComputerName with something that generates the full FQDN and it appears to work!

There appears to be a few spots in KVA, MOC, and AKSHCI cmdlets that also use hostname instead of $env:COMPUTERNAME. Again, by replacing that with something that generates an FQDN, everything has been happy in my world.

@madhanrm - was this something looked at in newer PowerShell modules?


AUTHENTICATIONFAILED

I got the following error
Can you help me ?
got an extra server challenge: eyJzdGF0dXMiOiI0MDAiLCJzY2hlbWVzIjoiQmVhcmVyIiwic2NvcGUiOiJodHRwczovL21haWwuZ29vZ2xlLmNvbS8ifQ==

got failure response: NO [AUTHENTICATIONFAILED] Invalid credentials (Failure)

I change my settings too.(Enable imap and enable security)
I try use OAUTH login and password are correct
Desktop / Server (please complete the following information):

PHP: [e.g. 7.4]

Hi @EvgenyGulyaev ,
thanks for your report.
Please checkout https://www.php-imap.com/examples/oauth
The challenge you've received is base64 encoded and decodes to:
{"status":"400","schemes":"Bearer","scope":"https://mail.google.com/"}

It looks like you've forgot to add https://mail.google.com/ to the scope during your token generation process.
Best regards,

@Webklex  I got this message after the following code
  $this->client = Client::account('gmail');
        $this->client->connect();

I'm not sure But I don't have  "App passwords"

Ok, I added application password,
But I got the same error

Oh..... It is a magic  =)
I randomly added this => 'authentication' => 'oauth', in my config and I got error about  auth
I deleted this row. I solved this problem, thank you for your advice


fix prod release builds

https://github.com/replicatedhq/embedded-cluster/actions/runs/7907096603/job/21583366051
this make command doesn't exist anymore, instead there's the full embedded-cluster-linux-amd64 command

This only makes changes to the release process, not to anything covered under tests, so I'm going to merge before tests complete