Responses are truncated to 255 characters

the string_value column for Response is a varchar(255) and mysql silently truncates values.
This means urls to datasets that are composed of a long query for example result in invalid or incomplete urls that then cause certification to fail. For example from http://www.leedsdatamill.org/api/rest/package/permanent-dwellings-by-tenure-type-in-leeds this “dataset url” which is 1954 characters long
http://opendatacommunities.org/sparql.csv?query=%23House+building+completions+by+tenure+and+time+period%2C+Leeds.%0D%0A%0D%0APREFIX+dcterms%3A+%3Chttp%3A%2F%2Fpurl.org%2Fdc%2Fterms%2F%3E%0D%0APREFIX+owl%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F2002%2F07%2Fowl%23%3E%0D%0APREFIX+qb%3A+%3Chttp%3A%2F%2Fpurl.org%2Flinked-data%2Fcube%23%3E%0D%0APREFIX+rdf%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F1999%2F02%2F22-rdf-syntax-ns%23%3E%0D%0APREFIX+rdfs%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F2000%2F01%2Frdf-schema%23%3E%0D%0APREFIX+skos%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F2004%2F02%2Fskos%2Fcore%23%3E%0D%0APREFIX+xsd%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F2001%2FXMLSchema%23%3E%0D%0APREFIX+dc%3A++%3Chttp%3A%2F%2Fpurl.org%2Fdc%2Felements%2F1.1%2F%3E%0D%0A%0D%0ASELECT+%3FgssCode+%3FisGovernedBylabel+%3Fperiodlabel+%3Ftenurelabel+%3Fobs%0D%0AWHERE%0D%0A%7B%0D%0A%3Fdata+qb%3AdataSet+%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdata%2Fhouse-building%2Fcompletions%2Ftenure%3E+%3B%0D%0A%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdef%2Fontology%2Fgeography%2FrefArea%3E%3Chttp%3A%2F%2Fstatistics.data.gov.uk%2Fid%2Fstatistical-geography%2FE08000035%3E+%3B%0D%0A%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdef%2Fontology%2Fgeography%2FrefArea%3E+%3Frefarea+%3B%0D%0A%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdef%2Fontology%2Fhouse-building%2FcompletionsObs%3E+%3Fobs+%3B%0D%0A%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdef%2Fontology%2Fhouse-building%2Fcompletions%2Ftenure%3E+%3Ftenure+.%0D%0A%3Ftenure+rdfs%3Alabel+%3Ftenurelabel.%0D%0A%3Fdata+%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdef%2Fontology%2Ftime%2FrefPeriod%3E+%3Frefperiod+.%0D%0A%3Frefperiod+rdfs%3Alabel+%3Fperiodlabel+.%0D%0A%3Frefarea+%3Chttp%3A%2F%2Fdata.ordnancesurvey.co.uk%2Fontology%2Fadmingeo%2FgssCode%3E+%3FgssCode+%3B%0D%0A%3Chttp%3A%2F%2Fopendatacommunities.org%2Fdef%2Flocal-government%2FisGovernedBy%3E+%3FisGovernedBy+.%0D%0A%3FisGovernedBy++rdfs%3Alabel+%3FisGovernedBylabel+.%0D%0A%7D%0D%0AORDER+BY+%3Fperiodlabel+%3Fobs

The only live dataset this affected was this dataset which is no longer available on http://data.gov.uk
still available on the google cache

Fixed by #1096


[Event Request] Table 83 "Item Journal Line" - OnValidateItemNoOnAfterCreateDim

Hi, in the Table 83 "Item Journal Line", in the field 3 "Item No.", in the trigger OnValidate, we would like to add our code in a specific place. To do so, we would need an event publisher, like this :

With the event integration :

Do you think it could be possible ?
Thank you for your work.

Is it ok for you to use existing event OnAfterCreateDimTableIDs for function CreateDim (parameter CurrFieldNo allows to identify the call from OnValidate trigger)?

Yes it is ok. Thank you !

Sorry, it doesn't fit my needs finally. CreateDim is called twice in the OnValidate trigger of field 3, and we want to add our code only after the first call. So the event OnAfterCreateDimTableIDs  with the CurrFieldNo parameter would not work.

@Magicrevette please create a new request for this.


Setting missing for increasing instance count for Web Services in the Render Dashboard?

Hi, cross-posting this issue from the Render forum:
https://community.render.com/t/how-to-enable-multiple-instances-support-for-a-web-service/153
I've tried following this guide for an Elixir Web Service I previously setup, but the settings page for the service in the Render dashboard has no such "Instances" setting as mentioned in the guide.
Is this a new feature actively being piloted with certain customers/plans?
FWIW the web service in question is in the Oregon region on a Standard Plus plan with a custom domain.

Ah, apparently it’s because services with disks attached do not support multiple instances, closing!


Release Action shall fail if an init fails

Describe the bug
We introduced an issue in #151 that made the dashboard import fail (see here). However, neither the PR nor the Release action caught the issue.
To Reproduce

Checkout 71dafdcaa0b358a5d8258bb5ec7c86783085f2a6
Start Faros CE; notice the failure
However, the Release workflow for that commit passed

Expected behavior
At least the Release workflow (I think the PR workflow does not build the init code) shall fail.

In this instance, I suspect Release did not fail because faros-init did not fail, because of how metabase-init.sh calls node for the import. Any init failing should fail faros-init.


Override fields in responsew, enhancement of PR #651

Describe the PR
fix bug: int,int32,int64, etc. are not replaced with integer in output docs, PR by #651
feature: support multiple fields overriding.
Relation issue
#650
#651

Codecov Report

Merging #654 into master will increase coverage by 0.04%.
The diff coverage is 94.59%.

@@            Coverage Diff             @@
##           master     #654      +/-   ##
==========================================
+ Coverage   85.03%   85.08%   +0.04%
==========================================
  Files           7        7
  Lines        1798     1803       +5
==========================================
+ Hits         1529     1534       +5
  Misses        171      171
  Partials       98       98

Impacted Files
Coverage Δ

operation.go
89.31% <94.59%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Selecting specific network interfaces for UDP and TCP

There is currently no elegant way to limit a Reticulum UDP or TCP interface to one or more specific system network interfaces (i.e. eth0, tun2, etc.).
It should be possible to specify this in the configuration file.

Implemented in 2b8b95da2b82208c2d2c813f6942e37def3f05bd


Improve readability and error reporting of taint analyzer

Purpose

Refactor taint-analyzer code and improve readability of the code.
Simplify taint-analyzer block resolution mechanism by removing the map used to store blocked nodes and using a list instead. This removes need of using multiple loops in the resolution process.
Check for invalid taint conditions of libraries and generate errors of functions are in invalid state. Previously, we will not generate an error when compiling example()  function of following code, unless it is used by an entry-point. This is changed with the current PR, since even though the function is not currently used it is already in an invalid state.

function example() {
    string taintedData = returnTaintedData();
    sensitiveFunction(taintedData);
}
function returnTaintedData() returns @tainted string {
   return "tainted value";
}
function sensitiveFunction(@sensitive string secureValue) {
}

Improve error reporting for invocations to help developers find the exact location that is passing tainted data. The error will be generated pointing to f1(args[0]); invocation instead of generating it for f2(param); invocation. Generating the error for  f2(param); invocation could have lead to confusions. If f1 has been used in multiple locations, developer will not be able to find which usage actually caused the error.

function main (string... args) {
   f1(args[0]);
}
function f1(string param) {
   f2(param);
}
function f2(@sensitive string param) {
}

:white_check_mark: Build ballerina-lang 320 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/0bb600b4aa by @ayomawdb)


nixos/ids: remove unused connman uid

The static connman uid is not referenced anywhere in NixOS.

notbit and monetdb are also unused.
smbguest is reserved but commented as unused
lxd gid is reserved but commented as unused, uid is commented out
When I "removed" monetdb I was unsure whether we should keep it in the file until we really need to reallocate.
Also the uids and gids have drifted again. If it's possible, we should have only one attrset for uid and gid in my opinion...


Ollama streaming output is garbled

Using local LLMs with Ollama produces outputs with words that are occasionally cut off or garbled. Using Ollama server outside the Chatbot-UI does not exhibit this issue.

I was able to fix this by adding a line to the code to disable the streaming.
const response = await fetchChatResponse(     process.env.NEXT_PUBLIC_OLLAMA_URL + "/api/chat",     {       model: "mistral",//chatSettings.model,       messages: formattedMessages,       stream: false,       options: {         temperature: payload.chatSettings.temperature       }     },     false,     newAbortController,     setIsGenerating,     setChatMessages   )
Here is the output without streaming. It would be nice to be able to stream though.

Interesting... I suspect there's an error with how we're parsing the stream. Going to add this to the top of my bugs list cause this is a nasty one for sure.

@mckaywrigley do you have any updates on this?, it is still happening for me

Hello, I am having the same issue here:
System: Ubuntu 22.04, latest version of ollama and latest pull of chatbot-ui as of 4 pm EDT
Running prompt in ollama cli I get a different response than in the chatbot-ui.
Attached are text files for comparison of python code output.
Hopefully this is a simple fix!
chatbot-ui-output.txt
ollama-cli-output.txt


Brainstorming how to add videos

Couldn't you add videos by using open cv to capture the videos, getting the frames, syncing the frames to a clock, and then converting each frame to a surface object(by way of using an image buffer?)

(This also means that you could add gifs with a little bit of coding)

Couldn't you add videos by using open cv to capture the videos, getting the frames, syncing the frames to a clock, and then converting each frame to a surface object(by way of using an image buffer?)

As kivy supports video playback by default
It is really not that hard to put a video as a wallpaper
As for gif I believe that can also be done easily
I think I can get the video as wallpaper done today itself


Add support for SDXL LoRA models in diffusers format.

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[x] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Have you updated all relevant documentation?

[ ] Yes
[ ] No

Description
This PR introduces 2 related changes:

This PR adds support for SDXL LoRA models in the diffusers format. There are 2 popular SDXL LoRA weight formats: Stability AI format and diffusers format (produced by diffusers, kohya_ss, and soon invoke-training). Prior to this change, InvokeAI only supported the Stability AI format, and trying to load a LoRA model in diffusers format would raise an exception.
This PR fixes a bug in the current version of InvokeAI that causes all text encoder LoRA weights to be ignored. In other words, prior to this change InvokeAI was only loading UNet LoRA weights for SDXL models.

Related Tickets & Documents
Relevant Discord conversation where someone encountered this issue: https://discord.com/channels/1020123559063990373/1133929466763161650/1139504370627661896
QA Instructions, Screenshots, Recordings
I tested the following models:
Voxel XL

https://civitai.com/models/118536
Trigger word: "voxel style"
Format: Stability AI
This LoRA was already 'working' before this PR. But the performance seems to be improved after fixing the text encoder bug.
Before change:

After change (Same seed and configs):

SDXL William Eggleston Style

https://civitai.com/models/119667
Trigger words: "by william eggleston"
Format: diffusers
Before this change: raised Exception
After change:

Angelina Jolie SDXL

https://civitai.com/models/125123/angelina-jolie-sdxl
Trigger words: "ohwx woman"
Format: diffusers
Before this change: raised Exception
After change:

Embroidery Style - SDXL

https://civitai.com/models/122308
Trigger words: None?
Format: diffusers
Before this change: raised Exception
After change: The LoRA runs, but does not match the results on CivitAI. I'm still trying to figure out why.

Added/updated tests?

[ ] Yes
[x] No : From what I can see, there is 0 test coverage of the LoRA loading code right now. Do we have any plans to change this?

[optional] Are there any post deployment tasks we need to perform?

If anyone has ideas as to why the Embroidery Style LoRA isn't working as expected, I'd love to hear them. I'm still investigating this, but not sure if it's worth holding up this PR over it.


Add ability to specify arbitrary function types in input files

Summary

This PR is a feature
It does the following:

Extends the currently restricted set of possible functions in the input file

The initial Inlet functions PR (#385) hardcoded the set of allowed functions.  To extend this, this PR generates the set of permissible function signatures at compile time and uses that to instantiate the FunctionWrapper type.
It also renames the inlet::FunctionType enum to inlet::FunctionTag (which enumerates the allowed types for function arguments/return values).  It replaces the inlet::FunctionType name with a set of type aliases, so FunctionTag::Vec3D is the enum used to specify the signature, and FunctionType::Vec3D is the vector type (currently primal::Vector<double, 3>) that is actually passed to/returned from the function.
Examples are added demonstrating how functions in a Lua input file might be used to define mfem::Coefficients.

Side question -- are there any inlet examples that define a callback function within a lua deck and call that function from within the deck? If not, should there be?

I added a lambda verifier to the mfem coefficient example that actually calls the function and checks its result.  We could also project the coefficients onto a GridFunction (thus calling the Lua functions) but my thinking was that demonstrating the use of a verifier on a function is more relevant to an Inlet example.

Side question -- are there any inlet examples that define a callback function within a lua deck and call that function from within the deck? If not, should there be?

I added a lambda verifier to the mfem coefficient example that actually calls the function and checks its result.  We could also project the coefficients onto a GridFunction (thus calling the Lua functions) but my thinking was that demonstrating the use of a verifier on a function is more relevant to an Inlet example.


Generalize indexable ruleset

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:
Now that custom rules loading implementations (and related, custom rulesets) can be swapped into falco in a customizable way, there is some functionality in evttype_index_ruleset that could be used by other rulesets, specifically the part that segregates filters by ruleset and enables/disables filters based on name substring + tags. This PR makes a base class for that. See the commit comments for more details.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NO

NONE

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mstemm
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 [mstemm]

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

9685bac Squash w Actually add indexable_ruleset to build
e0be57e Squash w/ Actually add indexable_ruleset to build

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

/milestone 0.39.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, mstemm
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 [jasondellaluce,mstemm]

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

LGTM label has been added.  Git tree hash: 2b16ef9e657a42a7641a0972210b87db15b5ec83

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, mstemm
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 [FedeDP,jasondellaluce,mstemm]

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

/unhold


[Feature request]Allow access to AccessibilityFeatures from flutter

Until the "ios shader jank" issue is resolved i was wondering if allowing access to reduceMotion and disableAnimations would help in reducing the amount of perceived jank on ios. I know it won't fix the issue but some apps don't need any animations but are relying on third party components that use animations. A global trigger for animations would help.
Another quick fix would be a temporary option to globally override all animations durations to zero.
https://api.flutter.dev/flutter/dart-ui/AccessibilityFeatures-class.html

These fields reflect what the phone's user has chosen and cannot be programmatically enabled by Flutter


DNS not working after docker daemon restart (live-restore)

We're using calico with docker (libnetwork-plugin) and every hw node has a local dnsmasq to proxy and cache DNS requests. So our docker configuration points the dns of every container to the host machine. So far everything works perfectly fine.
After restarting the docker daemon using the live-restore feature it seems that at least the iptables rules in the container nat table gets messed up, so dns requests stops working.
This is the table when a sysbox container is freshly started or restarted
# iptables -L -t nat
Chain PREROUTING (policy ACCEPT)
target     prot opt source               destination
DOCKER_OUTPUT  all  --  anywhere             <IP_ADDRESS>
Chain INPUT (policy ACCEPT)
target     prot opt source               destination
Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination
DOCKER_OUTPUT  all  --  anywhere             <IP_ADDRESS>
Chain POSTROUTING (policy ACCEPT)
target     prot opt source               destination
DOCKER_POSTROUTING  all  --  anywhere             <IP_ADDRESS>
Chain DOCKER_OUTPUT (2 references)
target     prot opt source               destination
DNAT       tcp  --  anywhere             <IP_ADDRESS>          tcp dpt:domain to:<IP_ADDRESS>:46183
DNAT       udp  --  anywhere             <IP_ADDRESS>          udp dpt:domain to:<IP_ADDRESS>:51249
Chain DOCKER_POSTROUTING (1 references)
target     prot opt source               destination
SNAT       tcp  --  <IP_ADDRESS>           anywhere             tcp spt:46183 to:<IP_ADDRESS>:53
SNAT       udp  --  <IP_ADDRESS>           anywhere             udp spt:51249 to:<IP_ADDRESS>:53

After Daemon restart additional rules appear, which are the default when not using sysbox.
iptables -L -t nat
Chain PREROUTING (policy ACCEPT)
target     prot opt source               destination
DOCKER_OUTPUT  all  --  anywhere             <IP_ADDRESS>
Chain INPUT (policy ACCEPT)
target     prot opt source               destination
Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination
DOCKER_OUTPUT  all  --  anywhere             <IP_ADDRESS>
DOCKER_OUTPUT  all  --  anywhere             <IP_ADDRESS>
Chain POSTROUTING (policy ACCEPT)
target     prot opt source               destination
DOCKER_POSTROUTING  all  --  anywhere             <IP_ADDRESS>
DOCKER_POSTROUTING  all  --  anywhere             <IP_ADDRESS>
Chain DOCKER_OUTPUT (3 references)
target     prot opt source               destination
DNAT       tcp  --  anywhere             <IP_ADDRESS>           tcp dpt:domain to:<IP_ADDRESS>:39639
DNAT       udp  --  anywhere             <IP_ADDRESS>           udp dpt:domain to:<IP_ADDRESS>:43355
DNAT       tcp  --  anywhere             <IP_ADDRESS>          tcp dpt:domain to:<IP_ADDRESS>:46183
DNAT       udp  --  anywhere             <IP_ADDRESS>          udp dpt:domain to:<IP_ADDRESS>:51249

Since sysbox rewrites resolv.conf to use <IP_ADDRESS> as dns and not <IP_ADDRESS> dns requests breaks.
If you need any more information I am happy to provide them.
Thanks!

Thanks for reporting this one @andreasnanko!
Will copy/paste below some of the notes exchanged in our slack discussion ...
--
The idea of having Sysbox modifying the iptables is to allow the inner containers to also reach the DNS server that seats in the host (which is something that you need for Calico's case). Without Sysbox doing this re-adjustment, the inner-containers (those inside the sysbox container), would see a generic DNS resolver (<IP_ADDRESS> or similar), so they wouldn't ever hit the host DNS.
My idea with this experiment (disabling dns-aliasing) was to verify that things work fine when this 'readjustment' isn't made. But even if it does work, that wouldn't fix the DNS-reachability issue for your inner containers.

Problem is reproduced in regular docker environments interconnected through Calico CNI. I suspect though, that Calico has not much to do with this issue and that problem is simply a consequence of Docker operating in 'live-restore' mode.
As mentioned above, Sysbox has specialized logic to alter the forwarding state of containers that rely on the host DNS-server for their operation. This is typically the case of containers that have been placed within custom docker-networks during initialization (through the docker --net attribute).
In Calico's setup, we also want all DNS traffic generated within the sysbox container to make use of the host DNS-server, so this logic is applicable too. Now, we need to identify a way that allow us to interact with Docker to prevent the forwarding state (previously injected by Sysbox) to be wiped out upon dockerd being restarted.

Hi @andreasnanko , thanks for reporting the issue.
We do a lot of testing for Sysbox, but Docker "live-restore" is one we had missed, thus we did not catch this. Apologies for that.
I was able  to reproduce the problem you reported. As you and Rodny noted, Sysbox does some iptable magic to modify the DNS resolve inside the Sysbox container, with the purpose of allowing inner Docker containers (launched inside the Sysbox container) to resolve DNS correctly too.
The problem you found is that when the Docker daemon at host level is configured with "live-restore", restarting that Docker daemon causes it to inject the original rules back into the iptables in the container's network namespace, thus colliding with the iptables rules injected by Sysbox and breaking DNS resolution.
This is going to be a tricky one for us to resolve, because there is notification from Docker to Sysbox that the container's iptables have been modified when Docker restarts. We need to think if there is some way that Sysbox can detect this, though it's not trivial.
Rodny had suggested using disabling the behavior by which Sysbox replaces the iptable rules for the container's DNS. You can try this (by setting the alias-dns=false in the sysbox-mgr systemd service unit), but the likely result is that containers created inside the outer container will have their DNS set to <IP_ADDRESS>. Not sure if this would serve as a work-around or not for your case.
If I may ask, how important is this fix for your company's use-case? This will help us prioritize among the other issues we are working on. Thanks!

@andreasnanko, just wanted to let you know that I ran a quick test to verify the proper operation of sysbox-mgr's alias-dns=false setting, and everything worked well for me, meaning that the knob was properly honored. I made a silly mistake last time I checked, sorry for any confusion this may have caused.
Please give it a try and let us know how it goes. As we said before, this may fix the forwarding issue after docker restart, but it may not be enough for your inner containers.
Now, let me ask you the following for my own understanding. What's the advantage that you are seeing in having docker's live-restore feature if you could simply rely on docker-swarm to manage your services? To me it looks like docker-swarm is a more matured and complete option. Please let me know if I'm missing something here as live-restore is a relatively new feature for me.

@rodnymolina seems to work. we use live-restore for docker upgrades or config changes without having to restart the containers if not needed.

Hi @andreasnanko, good to hear you were able to work-around it by setting alias-dns=false in the sysbox-mgr's command line.
Let's keep this issue open until we have a proper fix: ideally sysbox should detect the docker live-restore event and restore the dns aliasing on the containers.
Let us know if there is anything else we can help with in the meantime.


Scraping Tutorial

Before asking
Check through Issues and closed ones too if your question was already answered.
Question
I'm interested in learning and scraping the providers, so I can help in both personally and for the app.
Please let me know on how the scraping works in Python and as it looks easy to understand than other languages that I have previously attached on providers.
If possible, please provider me an guide or tutorial on scraping a site with the existing provider.
eg. what tools to use, how to identify certain things, etc.

Tools that i use are:
VSCode, HTTP Toolkit
Things you should learn how to use:
https://beautiful-soup-4.readthedocs.io/en/latest/
https://www.python-httpx.org/
https://docs.python.org/3/tutorial/index.html
Well i am pretty bad at explaining Things, i just looked how @Poseidon444 Scraped Sites and copyed things that i need: Search, Results and so on.
I chose a Provider and intercepted every HTTP Connection with HTTP ToolKit.
I looked how the Sites get the Links from the providers some like gogoanime have it in the data-video element some get it with ajax.
Just keep trying trying and trying.
(May contain broken English)

I'd recommend techwithtim videos to learn scraping, I'd also suggest you learn some html structuring so that you know what to scrape.


db error

Describe the bug
A clear aDETAIL: Detail redacted as it may contain sensitive data. Specify 'Include Error Detail' in the connection string to include this information.)
[2023-05-07 13:37:04.3309] [W] [ShareRecorder] Retry 3 in 00:00:08 due to Npgsql: PostgresException (23514: no partition of relation "shares" found for row
DETAIL: Detail redacted as it may contain sensitive data. Specify 'Include Error Detail' in the connection string to include this information.)
I installed Miningcore foloowing all instruction to a T and yet console keeps spitting out this error.
Im not very good in postgresql so could someone help me fix this issue please.
thanks!

You have to create a shares partition per pool you're launching.
CREATE TABLE shares_mypool1 PARTITION OF shares FOR VALUES IN ('mypool1');

(as described in the README)

So to be clear I have to add the shares partion per coin pool and not the cluster name?
Ive googled and searched github but every answer is read the readme paragraph..
Thanks for the help much apreciated.

If you launch a pool called btc1 you have to create a partition shares_btc1. If you launch a pool called eth1, you have to create shares_eth1 and so on.

awesome! thank you! no more errors


Persona Bar Security Analyzer showing FAIL for Telerik and Default.aspx

Steps to reproduce
Log-in a SuperUser and go to Persona Bar > Security > Security Analyzer
Current result
Two FAILs at the bottom - one for Default page other for Telerik
Expected result
Those two should be PASS. There should be only one FAIL for the biography field.
Affected version

[x] 9.2
[x] 9.2.1

This has been addressed in #2151 (Telerik failure) and dnnsoftware/Dnn.AdminExperience.Extensions#589 (Default.aspx fail)
As these PRs were merged into development, I also rebased onto release branches


Don’t use unsafeFlags when building SwiftSyntax

The group info file was only being used when building SwiftSyntax as part of the toolchain. But nobody should be using SwiftSyntax from the toolchain, so it wasn’t providing any value.
Exclusivity checking is no longer a performance problem, so there’s no reason to disable it anymore.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows


Fix queue length excluding gpu kernels without parent event

What does this PR do?
Fixes issue where queue lengths were negative
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)

[x] N/A

[x] Did you write any new necessary tests?

[ ] N/A

[ ] Did you make sure to update the docs?

[x] N/A

[ ] Did you update the changelog?

[ ] N/A ??

Tests
Checked on a trace where negative values show up-
These are now fixed -

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.01 :tada:

Comparison is base (743c79b) 90.24% compared to head (a11b55f) 90.25%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   90.24%   90.25%   +0.01%
==========================================
  Files          27       27
  Lines        2039     2042       +3
==========================================
+ Hits         1840     1843       +3
  Misses        199      199

Impacted Files
Coverage Δ

tests/test_trace_analysis.py
100.00% <ø> (ø)

hta/analyzers/trace_counters.py
96.55% <100.00%> (+0.12%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@briancoutinho can we also add the trace I provided and add test cases for it. The traces we had for our existing test cases didn't cover that case.

Here's the trace file that we can use to add test cases against negative queue length values.
negative_queue_length_values.json.gz


Python wrapping : Compute image statistics

CircleCI was down and couldn't report build status to GitHub. Everything should be fixed now.
Build is successful on circleCI and Cdash.

Thanks for reviewing Deepak.

You can use tubeWrapCallMacro for this instead of explicitly defining it.
tubeWrapCallMacro doesn't allow argument passing. We would need to add another macro for this. Do you think it would worth implementing it?

Ahh !!! Did not pay attention to the argument. Yes a new macro might be useful. It looks like a general use case. Maybe call it tubeWrapCallWithArgumentsMacro?

This describes how to define variadic macros (macros with variable number of arguments). I am not sure if it is supported by all compilers. If not, we should define such member functions explicitly as you have done.


Upgrade Protostar to 0.12

Protostar 0.12 will be released tomorrow (30.05.)!
From now on, Protostar will use exclusively Scarb to manage your dependencies.
Consequences:

no cairo_project.toml
no linked-libraries

Tests should pass when the 0.12 version is released

@Pet3ris I just bumped it to non-existing (at that time) Protostar version, so the CI did not install Protostar. Will update the version when we release it today (currently there is 0.12.0 as pre-release).
When it comes to Scarb <> Protostar interaction, they are separate tools and each of them needs to be installed separately.
From 0.12.0 Protostar uses Scarb as a dependency manager, therefore, it is required to have Scarb added to PATH to use Protostar.

@Pet3ris 0.12.0 is released already!


x/net/http2: suspicious uint32→int conversion in http2 frame implementation

Go version
analyzed net library from commit: f12db26b1c9293fa3eb95c936e548d2c1fba4ba9
What operating system and processor architecture are you using (go env)?
Analyzed source code with linter - so the env is irrelevant.

What did you do?
I ran custom govanish linter against golang/net GitHub repo and it found that following statements were removed from compiled binary:
2024/01/04 22:56:02 it seems like your code vanished from compiled binary: func=[maxHeaderStringLen], file=[/home/sivukhin/code/go-net/http2/frame.go], lines=[1519-1519], snippet:
	return 0

func (fr *Framer) maxHeaderStringLen() int {                             // 1512
	v := fr.maxHeaderListSize()                                      // 1513
	if uint32(int(v)) == v {                                         // 1514
		return int(v)                                            // 1515
	}                                                                // 1516
	// They had a crazy big number for MaxHeaderBytes anyway,        // 1517
	// so give them unlimited header lengths:                        // 1518
	return 0                                                         // 1519 <- this return is not found in attributed assembly
}                                                                        // 1520

This is ok because my machine is 64 bit, but when I inspected this code I found it weird because conversion of types uint32 -> int32 -> uint32 on 32bit architectures should results in noop because there is no loss of precision.
I implemented explicit conversion funcs with int32 / int64 types to prove this statement:
func ConvertInt32(v uint32) int32 {
	if uint32(int32(v)) != v {
		return 0
	}
	return int32(v)
}

func ConvertInt64(v uint32) int64 {
	if uint32(int64(v)) != v {
		return 0
	}
	return int64(v)
}

func TestConvert(t *testing.T) {
	v := uint32(math.MaxUint32)
	t.Log(v)
	t.Log(ConvertInt32(v))
	t.Log(ConvertInt64(v))
}
/*
=== RUN   TestName
    ast_test.go:90:<PHONE_NUMBER>
    ast_test.go:91: -1
    ast_test.go:92:<PHONE_NUMBER>
--- PASS: TestName (0.00s)
PASS
*/

Godbolt link with compiled functions against x86 gc (tip) target: https://godbolt.org/z/688bf4jEs
I suspect that this part of the code will produce negative MaxStringLen header. Not sure if this will results in any bad consequences but I think it's better to fix this issue and implement more straightforward check.
What did you expect to see?
Conversion from uint32 to int should always produce non-negative numbers
What did you see instead?
Conversion from uint32 to int can produce negative numbers

(attn @neild)

@sivukhin, want to send a fix?
(https://go.dev/doc/contribute)

Yes, sure, will prepare fix tomorrow.

@bcmills, done: https://go-review.googlesource.com/c/net/+/554235
Also, what do you think about adding helpers in stdlib which will simplify integer type conversions to native int/uint?
I saw similar issues in other repositories (for example here: https://github.com/sajari/docconv/blob/master/snappy/decode.go#L33). It will be great to have more explicit and less error-prone way to perform such conversion through API like func TryConvertUint32ToInt(value uint32) (int, bool) { ... }


Worked on every issue regards to payment plan

I worked on the:

Cancel a payment plan enhancement Issue
Update a payment plan enhancement Issue
Get payment plan enhancement Issue
Get payment plans enhancement good first issue
Create payment plan enhancement issue

Kindly Review

and also Thumb Up👍 for publishing the first version😁

You are welcome 🙂🙂. I will be looking forward into the rest of the issue


Default Save Location?

Hi,
I left this tool running on a large Site, I came back on the command line window had closed, nothing open in Excel.
Is there a default save location?
thanks

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

ID: 0a9d928d-fab5-99af-105f-86cb0520dcfa
Version Independent ID: ded494e8-1cc3-04bb-fec9-1c44e7231d06
Content: The SharePoint modernization scanner
Content Source: docs/transform/modernize-scanner.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Hi @princeJasbo,
When the tool runs in creates a folder in the folder where the .exe was and all reports are created in that folder. The tool will not automatically open the created reports.

Hi @jansenbe ,
We ran the tool and the tool is struck at Launching Wizard screen. Could you please lets us know if it is a bug?

Sometimes in indeed takes a while for the wizard to show...I'm experiencing similar things and haven't pinpointed the root cause. I'm suspecting Anti-virus soft to cause the delay in loading as on machines with no specific AV software (like servers) it loads without delay. Can try and wait longer or possibly create an exclusion in your AV soft for this .exe and see if that resolves the issue?

No folder was created in the folder were the .exe was, cannot find the repot... annoying


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #43.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #43

Apparently since 3.1.0 there is an additional plugin called actuator, that we did not install, we took the occasion to fix this now.


git in WSL 2 does not pick up changes made in Visual Studio

Environment
Windows build number: Microsoft Windows [Version 10.0.18363.900]
Your Distribution version: Ubuntu 18.04
Whether the issue is on WSL 2 and/or WSL 1: Linux version 4.4.0-18362-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #836-Microsoft

Steps to reproduce

Using Zsh as default shell in Ubuntu 18.04, but I can reproduce it in Bash as well.
Open a Ubuntu 18.04 app, navigate to the git repo on local filesystem.
Open the Visual Studio solution in Visual Studio 2017.
Make changes to tracked .cs file, Save in Visual Studio 2017.
go back to Ubuntu 18.04 app, do a gst or git status, there is no dirty files.
(close, and open a new Ubuntu 18.04 app, navigate to the repo again, do a git status will show the changes)

Expected behavior
git status on Ubuntu 18.04 app shows the changes made in Visual Studio 2017.

Actual behavior
git status on Ubuntu 18.04 app does not show the changes, but a git status after close and reopening the Ubuntu 18.04 app does.

my teammate have suggested that Visual Studio 2017 maybe doing something with the git locks, so this may not be a WSL issue.

You might be able to get some feedback/suggestions in a Visual Studio forum.


[cartservice] Update to .NET8

Changes
Update cartservice to .NET8 which is new LTS release.
Replaces #1269
Fixes #1262
Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[x] CHANGELOG.md updated to document new feature additions
[ ] Appropriate documentation updates in the docs N/A
[ ] Appropriate Helm chart updates in the helm-charts N/A

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

@austinlparker, there is a problem while restoring package. (probably the same as on main). Still do not know how to proceed.

Also tested locally and it seems to be working. Fingers crossed!


Add the transform methods as inherent methods on geometric types

This adds transform_point, transform_vector,
inverse_transform_point and inverse_transform_vector as inherent
methods on the applicable geometric transformation structures, such
that they can be used without the need to import the Transformation
and ProjectiveTransformation traits from alga.

Updated; sorry for the delay, I didn't get an email notification.

@tpdickso No worries. Thank you for all this !


[NSError] Use AnyHashable for userInfo consistently

Hm, there are certainly places where we intentionally wanted to restrict user info keys to String. cc @phausler


Bug: Directive Object attributes   generates an error

I have been struggling debugging this.
Here's a fiddle reproducing the bug.
Basically, with the following controller code:
    $scope.myArray = [3, 4, 5, 6, 7];
    $scope.object = {test: $scope.myArray};

The following template:
  <mydirective attributes="myObject"></mydirective>

Should be equivalent to:
  <mydirective attributes="{test: myArray}"></mydirective>

The second example works, but generates an error in the console.
In the fiddle, it is:
angularjs.org/angular-1.0.1.js:5526 Error: 10 $digest() iterations reached.
but in my code it was RangeError: Maximum call stack size exceeded on the equals method.
It seems like some useless evaluation is done in that case. Don't know exactly why.

Any reason you are using version 1.0.1? I got your fiddle to work by using version 1.3.15 and adding the line myApp.controller('MyCtrl', MyCtrl);

No reason indeed... I must have forked an old fiddle.
It means I haven't reproduced the bug because in my app, I'm using 1.3.13. And I tried with it also, and it also works.
I guess I should close the issue as there are very few chances now I can reproduce the bug.


Bug 2046521: Allow for disabling TLS verification for fetching ISOs.

Bug 2046521: Allow disabling TLS verification
Cherry-picked from e61d9e7.
Original PR: https://github.com/openshift/assisted-image-service/pull/53
Description

How was this code tested?

Manual, see original PR
Assignees

/cc @carbonin
/cc @
Links

https://github.com/openshift/assisted-image-service/pull/53

Checklist

[x] Title and description added to both, commit and PR
[x] Relevant issues have been associated
[x] Reviewers have been listed
[x] This change does not require a documentation update (docstring, docs, README, etc)
[ ] Does this change include unit tests (note that code changes require unit tests)

/bugzilla refresh

/bugzilla refresh

/assign @eranco74

FYI, The bot is unhappy about the field in the "Advanced fields" in Bugzilla not pointing at 2.4.1. If this is ready to merge that is the only thing that is missing before refreshing.

Oh that's where it is! Thank you @lranjbar!
/bugzilla refresh

/bugzilla refresh


Rewriting to /public is absolutely not the right way to change the document root

After doing some reading, I'm seeing Azure actually suggesting to people running Laravel (and any others that don't host out of the root project structure) applications to use mod_rewrite to rewrite requests to point to "/public".
This is an absolutely ridiculous suggestion and is effectively some of the worst advice you could give.  Forcing people to have "/public" in their URL structure because you guys didn't take the time to research the bare-minimum configuration options for your offering is not in any way acceptable.
I suggest you urgently add support for the ability to configure the document root.  Not only is the suggestion to use mod_rewrite (.htaccess) a cop out, but it actually violates security best practices by hosting your project root.  Yes, I understand the rewrite is there, but there are countless issues with this methodology, due to the nature of .htaccess.
The public directory in Laravel functioning this way is by design and not something to be subverted.

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

ID: a7d301ba-4ace-99a8-deba-4ed7f3413de0
Version Independent ID: 62100f8c-36be-3817-74da-ba72e75685f7
Content: Build a PHP and MySQL web app in Azure App Service on Linux
Content Source: articles/app-service/containers/tutorial-php-mysql-app.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

Hi @atrauzzi , thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Just to correlate: https://twitter.com/Omega_/status/1062920980753010688

@atrauzzi I've communicated your feedback with the product team. Unfortunately currently this is the only route other than building your own custom image. Please feel free to bring your feedback to https://feedback.azure.com. At this point the doc needs remain as is. #please-close

Custom image as in, container?

@atrauzzi That's correct. Running a custom container.

Alright.  I think you guys need to put some priority on exposing this as an option.  Running containers (especially as a kludge) consumes more resources than regular PaaS on Web Apps.  If I'm required to be running ~3 applications in a plan just to extract a competitive value, those resources are a bit precious.
I guess alternately, you guys could up the amount of RAM on offer at each tier (without increasing the price!!).

Just wanted to chip in here, as this issue is what came up in a google search.
I have to agree that the .htaccess suggestion that is currently (still) in the [docs is definitely bad advice, and it doesn't even work with the PHP 8.0 image which runs Ngnix rather than Apache.
Allowing an env var to set the document root would be hugely helpful. Or even better, an env var for a custom additional config file to be included would allow even more flexibility.
That said, there actually is a way to do this right now. I've ended up using the following solution:

Copy the web server config file into the home folder.
Make whatever changes you need to this file.
In the configuration for the app, set the Startup Script to copy your file back to the proper location.

Apache: cp /home/apache2.conf /etc/apache2/apache2.conf
Nginx: cp /home/default /etc/nginx/sites-available/default && service nginx restart
(The restart is necessary as nginx has already been started at the time the startup script runs)

Azure doesn't care, especially about PHP.

Just wanted to chip in here, as this issue is what came up in a google search.
I have to agree that the .htaccess suggestion that is currently (still) in the docs is definitely bad advice, and it doesn't even work with the PHP 8.0 image which runs Ngnix rather than Apache.
Allowing an env var to set the document root would be hugely helpful. Or even better, an env var for a custom additional web server config file to be included would allow even more flexibility.
That said, there actually is a way to do this right now. I've ended up using the following solution:

Copy the web server config file into the home folder.

Make whatever changes you need to this file.

In the configuration for the app, set the Startup Script to copy your file back to the proper location.

Apache: cp /home/apache2.conf /etc/apache2/apache2.conf
Nginx: cp /home/default /etc/nginx/sites-available/default && service nginx restart
(The restart is necessary as nginx has already been started at the time the startup script runs)

Then a better solution could be to have Startup Script with the following command just to replace document root directly in the config:
sed -i 's//home/site/wwwroot//home/site/wwwroot/public/g' /etc/nginx/sites-available/default && service nginx reload


Remove the release scripts and docs from knative's era

Changes
We are now dogfooding pipeline for doing the releases (see ./tekton
for more information). Those scripts are now deprecated, so let's
remove them.
/assign @bobcatfish
/hold
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ :no_good_man: ] Includes tests (if functionality changed/added)
[ :no_good_man: ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide
for more details.

woot let's do it!
We should also remove anything we're vendoring (if we can) that's specific to releases :D
/lgtm
/meow space

/hold cancel


Add .NET 7.0 SDK feature to devcontainer configuration

Motivation and Context
Resolves #1421 by specifying .NET 7.0 SDK as feature for dev container to install.

I tried to validate this in a codespaces against this branch/PR and I'm seeing this:

It does look like the change you made has come across:

Any suggestions on what I should try to validate the fix?

@craigomatic - If you had it open in a Codespace previously, you will need to rebuild the container and reopen.


This plugin's read me suggests a depricated plugin.

you should suggest this -> https://pub.dev/packages/connectivity_plus
not this -> https://pub.dev/packages/connectivity

It has been fixed. ;)

But the README still remains suggesting connectivity plugin which is discontinued.


python3Packages.torch-bin: 1.13.1 -> 2.0.0

Description of changes
Pytorch 2.0 is released.

Blog
Release note

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] 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.05 Release Notes (or backporting 22.11 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.

Out of curiosity, I would like to consult a few questions. It seems that torch.compile in Pytorch 2.0 requires triton. Currently triton is not in Nixpkgs yet.

Does torch.compile work with this PR?
If so, how does it achieve that? My guess is that the wheel binary will have compiled triton shared libraries in it, but what about the triton python package that wraps it?

Thanks a lot!

@breakds IIRC PyTorch distributes their own builds of Triton from the branch OpenAI maintains for (with?) them: https://github.com/openai/triton/tree/torch-inductor-stable.
I'll try it out later to see if it works.
To be clear, I personally don't think a broken torch.compile should prevent this from getting merged: from what I remember from trying to package Triton a few months ago, it needs MLIR and a few other things we don't have, as well as some manual patches.

@breakds IIRC PyTorch distributes their own builds of Triton from the branch OpenAI maintains for (with?) them: https://github.com/openai/triton/tree/torch-inductor-stable.
I'll try it out later to see if it works.
To be clear, I personally don't think a broken torch.compile should prevent this from getting merged: from what I remember from trying to package Triton a few months ago, it needs MLIR and a few other things we don't have, as well as some manual patches.

Thanks a lot for the explanation, especially the information about the torch-inductor-stable branch. That makes sense to me.
Agree that a broken torch.compile shouldn't be a blocker, thought it is an important feature of torch and would be nice to have. I tried to package triton and hit the problem that llvm.mlir not in Nixpkgs yet. I assume #163878 is an ongoing effort to address that.

@junjihashimoto I was unable to build the derivation because sympy wasn't added to its inputs. I tried to package the nightlies a month or two back and IIRC we can drop a few inputs and we need to add a few new ones (including sympy).
$ nix build --impure -L github:NixOS/nixpkgs/refs/pull/221652/head#python3Packages.torch-bin
python3.10-torch> Sourcing python-remove-tests-dir-hook
python3.10-torch> Sourcing python-catch-conflicts-hook.sh
python3.10-torch> Sourcing python-remove-bin-bytecode-hook.sh
python3.10-torch> Sourcing wheel setup hook
python3.10-torch> Using wheelUnpackPhase
python3.10-torch> Sourcing pip-install-hook
python3.10-torch> Using pipInstallPhase
python3.10-torch> Sourcing python-imports-check-hook.sh
python3.10-torch> Using pythonImportsCheckPhase
python3.10-torch> Sourcing python-namespaces-hook
python3.10-torch> Sourcing python-catch-conflicts-hook.sh
python3.10-torch> unpacking sources
python3.10-torch> Executing wheelUnpackPhase
python3.10-torch> Finished executing wheelUnpackPhase
python3.10-torch> patching sources
python3.10-torch> configuring
python3.10-torch> no configure script, doing nothing
python3.10-torch> building
python3.10-torch> no Makefile or custom buildPhase, doing nothing
python3.10-torch> installing
python3.10-torch> Executing pipInstallPhase
python3.10-torch> /build/dist /build
python3.10-torch> Processing ./torch-2.0.0-cp310-cp310-linux_x86_64.whl
python3.10-torch> Requirement already satisfied: typing-extensions in /nix/store/bndw06wps3i7xpqdk6ryq6wiqg11ggy8-python3.10-typing-extensions-4.5.0/lib/python3.10/site-packages (from torch==2.0.0) (4.5.0)
python3.10-torch> ERROR: Could not find a version that satisfies the requirement sympy (from torch) (from versions: none)
python3.10-torch> ERROR: No matching distribution found for sympy
python3.10-torch>
error: builder for '/nix/store/wrbw0hl6p10cc0x230j8z7f655yal0x4-python3.10-torch-2.0.0.drv' failed with exit code 1

Ah shoot, so while Triton may be bundled with their distribution when installing using pip, because Nixpkgs' python builder prevents that, we need to make sure we install it.
@junjihashimoto you'll probably have to add a triton-bin package: https://download.pytorch.org/whl/triton/.
After that's added you'll need to update the torch-bin derivation with it.
This is what I was able to get required additional dependencies prior to running into Triton being missing:
# Fix for torch-bin
(final: prev: {
  python3Packages = prev.python3Packages.overrideScope (pfinal: pprev: {
    torch-bin = pprev.torch-bin.overrideAttrs (oldAttrs: {
      propagatedBuildInputs = oldAttrs.propagatedBuildInputs ++ (with pprev; [
        pprev.sympy
        pprev.jinja2
        pprev.networkx
        pprev.filelock
        pprev.triton # :(
      ]);
    });
  });
})

@junjihashimoto those commits helped!
Trying to use torch.compile, I can see that I'm getting permissions errors because it's not executable.
Try adding
chmod +x $out/lib/<whatever python version>/site-packages/triton/third_party/cuda/bin/ptxas

to whatever phase feels appropriate in the triton-bin derivation.
After I did that, I was able to use torch.compile! 🎉

@breakds @ConnorBaker
Thank you for your help!
I tested this sample. It works.
https://gist.github.com/junjihashimoto/76f73f118289a5c33f4e311b66a7677a

@katanallama
In my case, libtriton.so is linked to /nix/store/wvm2hvqdbbsp1f11463mrw8nyv678ipm-gcc-12.2.0-lib/lib/libstdc++.so.6 .

I created the environment as follows.
nix-shell -p python3 'let pkgs=import ./default.nix {}; in pkgs.python3.withPackages (p: [p.torchvision-bin p.torch-bin])'

Why is it linked to a different file?

The link of libz.so.1 should be fixed.

ptxas links to /lib64/ld-linux-x86-64.so.2, so I'll fix it.

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

2 packages failed to build:

python310Packages.torchvision-bin
python310Packages.torchvision-bin.dist

28 packages built:

python310Packages.lit
python310Packages.lit.dist
python310Packages.openai-triton
python310Packages.openai-triton-bin
python310Packages.openai-triton-bin.dist
python310Packages.openai-triton.dist
python310Packages.torchWithCuda
python310Packages.torchWithCuda.dev
python310Packages.torchWithCuda.dist
python310Packages.torchWithCuda.lib
python310Packages.torchWithRocm
python310Packages.torchWithRocm.dev
python310Packages.torchWithRocm.dist
python310Packages.torchWithRocm.lib
python310Packages.torchaudio-bin
python310Packages.torchaudio-bin.dist
python310Packages.trainer
python310Packages.trainer.dist
python311Packages.lit
python311Packages.lit.dist
python311Packages.openai-triton
python311Packages.openai-triton-bin
python311Packages.openai-triton-bin.dist
python311Packages.openai-triton.dist
python311Packages.torchaudio-bin
python311Packages.torchaudio-bin.dist
tts
tts.dist

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

36 packages built:

python310Packages.lit
python310Packages.lit.dist
python310Packages.openai-triton
python310Packages.openai-triton-bin
python310Packages.openai-triton-bin.dist
python310Packages.openai-triton.dist
python310Packages.torch-bin
python310Packages.torch-bin.dist
python310Packages.torchWithCuda
python310Packages.torchWithCuda.dev
python310Packages.torchWithCuda.dist
python310Packages.torchWithCuda.lib
python310Packages.torchWithRocm
python310Packages.torchWithRocm.dev
python310Packages.torchWithRocm.dist
python310Packages.torchWithRocm.lib
python310Packages.torchaudio-bin
python310Packages.torchaudio-bin.dist
python310Packages.torchvision-bin
python310Packages.torchvision-bin.dist
python310Packages.trainer
python310Packages.trainer.dist
python311Packages.lit
python311Packages.lit.dist
python311Packages.openai-triton
python311Packages.openai-triton-bin
python311Packages.openai-triton-bin.dist
python311Packages.openai-triton.dist
python311Packages.torch-bin
python311Packages.torch-bin.dist
python311Packages.torchaudio-bin
python311Packages.torchaudio-bin.dist
python311Packages.torchvision-bin
python311Packages.torchvision-bin.dist
tts
tts.dist


late note

What is your question?
I don't know why, but my notes are slow! No matter how I set them! please tell me the reason__

press ctrl in the freeplay or story mode menu

I'm cured! thank you!

@yuutocat close this


Crop and Lock zoom

Description of the new feature / enhancement
It would be great if the crop and lock function also had a zoom or resize feature. So crop and lock the section of the window you desire, then grab a handle and resize the window to what you want, enlarging (magnifying) the contents of the window.
Scenario when this would be used?
There are many programs whose font size or general look is too small for modern screens. Though there are some magnifying applications, they are more like lenses where everything is magnified in a certain area or the whole screen is magnified. Some also let you change font size, but that doesn't work in many applications. It would be great to be able to lock the area you want cropped and just zoom that (by grabbing the window edge/corner and dragging).
See example attached.

Supporting information
No response

I believe this is currently possible? Can you explain more if it isn't for you, or how resizing the cropped window doesn't fit your expectation?
/needinfo

Cropping in thumbnail mode does exactly what I'm thinking, but you can't interact with the cropped window (click buttons, move sliders, etc). It would be nice if the same scaling behavior could be used with the reparent mode where interacting with the window is possible.


#810: fixed reading of test resources with system-dependent line break sequence

Looks like it ran into a conflict?

Was fixed by commit ed8758411f0dc5dc0506922e5a7ef4f2cbc5db7d in between (s. #808)


Setting auto mode => weird autotake-off

As @alexcherpi and I have faced,
when we set the mode to auto from the Ground Control Station, it is actually taking off with a command of 0 velocity
the problem is that due to velocity estimation inaccuracy it is taking off and moving already to compensate this velocity estimation error, even though auto take-off was not set yet !!

Fixed by #331


Establish a new environment to discontinue the use of Docker.

Establish a new environment to discontinue the use of Docker, and create two new evaluation script: one for a single model and one for an ensemble model.
There are many copied and pasted files that haven't actually been modified.

我覺得看起來沒什麼問題，絕大部分的檔案都沒變化但不知道為什麼被註記成是有修改過的。
此外aicup_ts_ensemble那個資料夾有需要上傳嗎?

忘記加進去gitignore，我把它們刪掉了


Summary: Bumps version from 0.8.3 -> 0.8.4

Tracker
https://www.pivotaltracker.com/story/show/
Details
Review Checklists
Author

[ ] Squashed & Rebased
[ ] Commit message(s) conform to template (Summary, Details, Tracker)
[ ] No browser warnings/errors
[ ] Ran bin/qa

Reviewer

[ ] Readable
[ ] Tests adequately cover implementation
[ ] Backwards Compatible (Data, UI)
[ ] No junk (Comments, Typos, Test Focus, Debug Statements)
[ ] Components are divided well

👍 lgtm


Issue while switching language

When ever i change the language it crashes the application. I can't tell weather it's MDX issue or next-international

I've created the repo with basic fresh project. Please check
https://github.com/iStorry/mdx-next-international-test

mdxeditor author here, I traced down the problem in the editor to this piece in my code (lives in useEffect):
    document.body.appendChild(popupContainer)
    editorRootElementRef.current = popupContainer
    setEditorRootElementRef(editorRootElementRef)
    return () => {
      document.body.removeChild(popupContainer)
    }

Apparently, by the time the editor is unmounted, the popup container is no longer a direct child of the body element. I find this weird.
This is the fix: https://github.com/mdx-editor/editor/commit/5de52ad2c12eebdfe39f3ddb0f3ae13b857d7243. However, this might cause issues with other libraries who manage popups by moving them to the body.

@petyosi Thank you for the quick response.


Nexus 6 evaluated as rooted

I face a problem appearing similar to https://github.com/scottyab/rootbeer/issues/13 for a regular Nexus 6 on a non-rooted (and non-tampered-with) Android 6.0.1 system. Specifically, checkForRWPaths() returns true when hitting these variable assignments:

line: "none /dev/cpuset cgroup rw,relatime,cpuset,noprefix,release_agent=/sbin/cpuset_release_agent 0 0"
pathToCheck: "/sbin"

I think instead of checking whether the entire line contains pathToCheck, it should only look for pathToCheck within the mount point line.split(" ")[1]

should be fixed in #18


Issue Reporter doesn't show me any errors when submitting doesn't work

Does this issue occur when all extensions are disabled?: Yes
I'm banned/blocked from doing anything in a Github organization, so when I was trying to report an issue to one of that org's repos, pressing the "Create on GitHub" button didn't do anything. I think it doesn't work because I'm banned/blocked, so I expect the Issue Reporter to show me an error.

VS Code Version: 1.85.1 (user setup)
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:

Get banned/blocked from a repo/org
Try to submit an issue for an extension with the Issue Reporter
Now, when you try to submit, it won't work and it won't show you any errors.

Version: 1.85.1 (user setup)
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:49:37.021Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

related https://github.com/microsoft/vscode/issues/199864 https://github.com/microsoft/vscode/issues/188296 https://github.com/microsoft/vscode/issues/175278


Refactor/delegate emailer to authservice

I am curious: I was under the impression that we would be removing email-based login in favour of OIDC.  If so, then why are we improving this code when we could be working towards removing it?

Regarding @arielshaqed's comment, we synced f2f, writing here to document: While we may want to phase out email-based login, this wasn't yet decided upon.


Dismissing a landscape presented window doesn't return outgoingWindow orientation.

If I present a window with a root view controller which forces landscape orientation:
- (UIInterfaceOrientationMask)supportedInterfaceOrientations
{
    return UIInterfaceOrientationMaskLandscapeRight;
}
- (UIInterfaceOrientation)preferredInterfaceOrientationForPresentation
{
    return UIInterfaceOrientationLandscapeRight;
}
- (BOOL)shouldAutorotate
{
    return YES;
}

Upon dismissing the window:
[TTWindowManager.sharedInstance dismissTopWindowWithCompletion:NULL]

The outgoingWindow orientation remains landscape & I can't return the orientation back to portrait.

I'm trying to reproduce the issue with the demo project.
This is pretty much what I'm experiencing:
http://d.pr/f/1bRM7

Removing the backgroundAnimationWindow seems to improve this issue.


Video preview images overlapping in profile view

See https://gfycat.com/GeneralWillingGrebe.

Good find! Caught that right before I pushed 1.4 :P

If you haven't submitted to the App store yet and have time, another issue I noticed was if you have an album with a gif with sound, the sound continues to play after you close the album. I would make a proper issue submission but my MacBook Pro just crashed and I'm trying to figure out what happened...

@djjcast  I have already pushed it, but I will look into it for the 1.4 bug fix update! I feel you on that MBP crashing, I have had to reinstall Mac OS twice already due to kernel issues....


Error loading module luv

What i'm doing wrong? I have built luv as a module and i got 'luv.dll', then to try it out i put it in the same folder as a lua script:
local luv = require('luv')  but when i run the script i get this error: Error loading module 'luv' from file '.luv.dll', the specified procedure couldn't be found.

Upon further investigation i discovered the following. If i doluajit test.lua or lua test.lua it gives me that error. If i use wxlua, which i built using luajit everything is fine...

I was running the script with git bash, i then switched to a plain cmd window(i'm on windows 7) and got a more clear error: the procedure entry point GetSystemTimePreciseAsFileTime could not be located in Kernel32.dll. I think i know, it's because they dropped support for windows 7. Infact using a less recent version it works.

Yep, the minimum Windows version became Windows 8 in Libuv 1.41.0: https://github.com/luvit/luv/releases/tag/1.41.0-0


Added target attribute and compiler detection for intrinsics

This is a first stab at issue #336 to add attribute target to the functions that are using processor specific instructions. I have added HAVE_ATTRIBUTE_TARGET_[FEATURE] and others that set ZNG_TARGET_[FEATURE]. I do the compiler check in cmake. Maybe somebody else can do the configure script.
I did find some SSSE3 instruction in the code (_mm_shuffle_epi8), but I don't see any cmake or configure detection for SSSE3.
I could really find an attribute target for NEON adler instructions.

Some consistency formatting things I didn't do but would have liked to do:

Rename HAVE_HIDDEN to HAVE_ATTRIBUTE_HIDDEN
Rename HAVE_INTERNAL to HAVE_ATTRIBUTE_INTERNAL
Rename X86_SSE4_2_CRC_INTRIN to X86_SSE42_CRC_INTRIN and SSE4_CRC to SSE42_CRC
Rename ZLIB_INTERNAL to ZNG_INTERNAL

Obviously all build errors need to be resolved, before anyone can approve this change...

I can't seem to figure out why the 4 instances on Travis are failing due to ssse3 attribute. Maybe somebody else can take a look.

@nmoinvaz I found old gcc regression report about ssse3 not working in function target attributes. Has been broken since gcc 5.0.

Do you have the link to the report? The strange thing is the clang instance is failing too.

https://gcc.gnu.org/ml/gcc-patches/2013-04/msg00740.html

It looks like it requires -DWITH_NATIVE_INSTRUCTIONS=ON to compile correctly. Not sure what I should do about that. What should the default behavior be? Always default to -msse3 flag when detecting __GNU__ or always turn on WITH_NATIVE_INSTRUCTIONS when detecting __GNU__.

Obviously we need to check if the gcc is broken and avoid using the function target attribute.

Looks like I got it. It wasn't doing the attribute detection properly because the instructions were being optimized away by the compiler I guess. Now it can do a mixed mode, where it does target attributes if it detects it will work correctly, otherwise it will use compiler flags.
On compilers that have problems with target attributes, -DWITH_NATIVE_INSTRUCTIONS=ON should get everything working with target attributes.
Side note WITH_NATIVE_INSTRUCTIONS is good, but I see the need for also having something like 'WITH_OPTIM_INSTRUCTIONSwhich would use-march=core2which are minimum compiler features for the optimizations. Setting-march=core2(64-bit) or-march=prescott(32-bit) also fixes the issue and gets everything working with target attributes. HavingWITH_NATIVE_INSTRUCTIONS` might use other instructions that wouldn't work on the minimum processor.

Looks like the issue I am having is due to the use of multiple __attribute__((__target__(""))) on the functions. In two functions in crc_folding.c it doesn't like the use of ZNG_TARGET_SSE2 with ZNG_TARGET_SSSE3 and ZNG_TARGET_PCLMULQDQ. It will only work if it is defined like in one attribute: __attribute__((__target__("sse2,ssse3,pclmul"))).
Not sure how to handle that cleanly?
#if defined(HAVE_ATTRIBUTE_SSE2) && defined(HAVE_ATTRIBUTE_SSSE3) && defined(HAVE_ATTRIBUTE_PCLMULQDQ)
#  define ZNG_TARGET_PARTIAL_FOLD __attribute__((__target__("sse2,ssse3,pclmul")))
#endif

Travis CI is successful. Not sure what is wrong with buildkite since I don’t have access to it.

Remember that the sse/avx attributes are supersets of eachother.
SSE2 implies SSE, SSE3 implies SSE2 and SSE, etc. So you only need to set the highest one.

@Dead2 That is mostly true, but in some cases specifying fallback target means alternative version of intrinsic can be used. For example a lot of AVX intrinsics have SSE equivalent that use less registers with same effect.

@mtl1979 Yes, but isn't this to enable us to compile code that requires a certain minimum instruction set, like we have always done? Such as the CRC32 instruction or PCLMULQDQ etc. So we should specify their minimum required instruction set to match the functable tests.
We don't have any functionality for instruction set fallbacks today, so that should not be part of this PR.
GCC also supports making multiple versions of a function, compiled for different instruction sets, and such optimizations would need in-depth research to justify the added complexity by improving speed significantly.
march serves a similar but different purpose, allowing the compiler to use more optimal instructions for tasks that can be solved with more simple instructions, and the distro march setting must be respected here (allowing the user to set a minimal march is also OK, but must not by default override a march set for example by rpm macros.)
PS: I am writing this from a mountain cabin with severely limited internet connectivity, so I have not reviewed any changes, only read comments notifications by email.

@Dead2 As far as I understand the function target attribute works in conjunction with "march" specified on command line... When "march" is set to lower than highest specified in target attribute, it chooses the instruction set that intersect both choices.
If there is only one target specified, it will fail with error during compilation. Main purpose of function "target" attributes is to allow compiling inlined code that will use run-time check to choose the right function or group of functions.
It works little similar with "mtune" on command line, but doesn't actually generate code, it only verifies already generated code that was wrapped in "target" pragma block.
If you ask gcc to generate ifunc-style wrapper for multiple targets using single function with attribute "target_clones", it will of course generate new code if the function is used at least once.

@Dead2, let me know which way you would like me to go with this PR after you've had a chance to look at the code. Enjoy the cabin. :-)

I have noticed something interesting. The code in crc_folding.c uses sse2,ssse3,sse4.1,pclmulqdq while the code in insert_string_sse.c and deflate_quick.c uses sse4.2. So I need to account for the possibility sse4.1 versus sse4.2. What I think is also interesting, is that there is no path that uses sse4.2 crc functions from crc32.c.

I have found a separate issue in CMakeLists.txt where if using MSVC it does not use crc_folding.c because the checks for intrinsics are not being done.

I see the reason why no sse4.2 path in crc functions from crc32.c - it is because intel crc32 intrinics are CRC32-C not CRC32-B. I suppose this is okay for deflate_quick and quick_insert_string where it might just be comparing two hashes.

To avoid cluttering the function definitions more than needed, I suggest having the target attribute on the line before the function definition.
Also, I am wondering whether it might be easier and more easily understood and maintained to use target pragmas instead. https://gcc.gnu.org/onlinedocs/gcc-4.7.2/gcc/Function-Specific-Option-Pragmas.html#Function-Specific-Option-Pragmas
Those are what I usually use in other projects.
The nice thing about the pragmas is that they don't need any checks in the configure/cmake scripts, and no need for a big chunk of define macros to make the target attribute defines needed either.
The pragmas will also be used by compilers that try to be gcc compliant, and will just be ignored by the rest.
The pragmas can also work on a whole file (making them drop-in the same as the current -msseX parameters), and having the pragma in the file header (after includes) would likely make it pretty easily understood.
The only downside I see to the pragmas is that they do not limit themselves to one function, requiring a pragma reset after each function if you want that, but since we already have whole-file settings, that is not really a big downside for us.
Is there any reason not to use the gcc pragmas to make life easier for ourselves?
I understand that this does not help with MSVC and such, but that is not solved here anyways.

@Dead2 I don't think clang supports GCC target param? But I think clang supports attribute target.

The code inspector application keeps failing. I used to have code-inspector on my minizip repo but too many false positives. Instead I use LGTM. https://lgtm.com/

CodeInspector and a few of the others are just something I am testing out currently, I am aware that there is problems with some of them. I'll add LGTM, thanks for the suggestion.
I'll remove some of them again later, once I know what is worth keeping.

Ok I have made the latest changes. Feel free to review now.

I just checked, and PCLMULQDQ first appeared in a Westmere cpu, and it does not appear in any cpu that does not support SSE4.2. Therefore there is likely no reason to disallow the compiler from utilizing instruction sets up to that if it deems it to be beneficial to performance, since those functions cannot be utilized on SSE4.2-less cpus anyway.
Also, I notice that you have changed what functions gets the flags, I'd prefer that to be a different PR so it can be more easily reviewed and checked for specific performance regressions.
For example currently all functions in deflate_quick gets the full sse4.2 instruction set, and that is fine because we never use deflate_quick on older cpus anyways.
With your current changes, only the functions utilizing sse4.2 intrinsics gets those instruction sets enabled, limiting the compiler to only use older instruction sets for the other functions. This also limits what functions the compiler can inline, since it cannot inline a sse4.2 or ssse3 function into a sse2 function for example.
So, as an initial step, please set the same instruction sets as before for all functions in a file that previously had a file-wide instruction set specified. Changing the instruction sets used per function should be a separate discussion and PR. There might well be benefits to be had there though.
This is also not compiling on my local machine currently, cmake elects to use -msse3 etc instead of the attributes, and the compiler does not recognize the attribute defines and complains about syntax error. Will investigate further later.

I am not sure how this is happening, but the cmake script is somehow only finding support for pclmulqdq as a target attribute, sseX is not detected correctly. This is using GCC9.1 on Fedora 30.
[ 12%] Building C object CMakeFiles/zlib.dir/functable.c.o /usr/bin/cc -DHAVE_ATTRIBUTE_PCLMUL -DHAVE_BUILTIN_CTZL -DHAVE_HIDDEN -DHAVE_INTERNAL -DUNALIGNED_OK -DX86_CPUID -DX86_PCLMULQDQ_CRC -DX86_QUICK_STRATEGY -DX86_SSE2 -DX86_SSE4_2_CRC_HASH -DX86_SSE4_2_CRC_INTRIN -DZLIB_DLL -D_LARGEFILE64_SOURCE=1 -D__USE_LARGEFILE64 -I/home/hansr/github/zlib/zlib-ng  -msse4 -O2 -DNDEBUG -fPIC   -std=c99 -o CMakeFiles/zlib.dir/functable.c.o   -c /home/hansr/github/zlib/zlib-ng/functable.c
Also, the defines in zutil.h needs to have a fallback define that defines the name but without content (like the ZLIB_INTERNAL for example). (This would also have hidden the current failure though)
Also, a suggestion after having attempted to read through the cmake code to find out how this fails; How about we first check for target attribute support, and then check for either -msse3 OR msse3 as a target attribute? Currently we first check for -msse3 and if that is supported we try to upgrade to a target attribute. I think this could make the code simpler and also could help keep the instruction-set-support detection grouped instead of split up like now.
The compile test could potentially even be done using a single code example instead of two separate ones, by using a define to enable the target attribute, like we do in the real compile. I think that should make slightly shorter code total, but would require the test source code to reside in a string that gets shared by the two tests. That could allow us to group all the test source code in a separate area of the script, or even into a separate cmake file. (I think that would be acceptable for this PR, but a future PR might explore restructuring the arch-dependent checks into separate cmake files, as the cmake script is now getting very big and can be hard to grok due to the high level of inter-dependencies going on).

Btw, related to earlier comments, clang supports its own target attribute pragmas
https://clang.llvm.org/docs/AttributeReference.html#id267
See bottom answer for an example:
https://stackoverflow.com/questions/46165752/does-clang-have-something-like-pragma-gcc-target

So, as an initial step, please set the same instruction sets as before for all functions in a file that previously had a file-wide instruction set specified.

I'm not sure what you are referring to. If a function used a particular instruction set, I set it to have that particular attribute target. Was there a particular function that I did not set correctly? This is the way I had it before too I believe.

I am not sure how this is happening, but the cmake script is somehow only finding support for pclmulqdq as a target attribute, sseX is not detected correctly.

In order to debug the issue with cmake you can run rm CMakeCache.txt & cmake . --debug-output and see CMakeFiles\CMakeError.log to see the error.
I think maybe your idea for the pragma has some merit, but then the code would have many pragmas for the different compilers.
Perhaps it would be better for somebody else to take over this who has more experience with target attributes.

@nmoinvaz Thanks for the cmake tip, I didn't know about the --debug-output parameter. I had wiped the cache and the CMakeFiles folder and looked in the log files to try to find the error, but it looked to me like there was no error, that it somehow just skipped the tests.
I will try to make a pragma-based PR soon-ish, then we can see whether that works out fine or not. I might need some help with the cmake file though, we will see. I feel like it has grown to a size where it needs to use include files. (like many other projects use with separate SRCDIR/cmake/*.cmake files where each file handles separate features (For example ssl, jemalloc, dtrace, ctest. I don't know what the best layout would be for us, but I am thinking one for each arch, one for ctest, one for fuzzing, one (or more) for compiler tests should be a good start.)


Upgrade com.ortto:androidsdk to 1.5.1

Hello @scrummitch @Draal 👋
Thank you for your work providing this Flutter SDK!
I've noticed you fixed some deeplink issues in ortto-push-android-sdk release 1.5.1
As we are integrating the flutter SDK into our app, I'm experiencing issues regarding linking to the app (app crash when pressing the notification on android). I think that's because we are still in 1.5.0 behind the flutter SDK.
Would that make sense to upgrade the android sdk dependency from 1.5.0 to 1.5.1 in the flutter sdk and publish a new version? Seems like it would solve the issue on my side, it'd be greatly appreciated.
https://github.com/autopilot3/ortto-flutter-sdk/blob/c1940392fa46a8b49fb2e401e5fc4152b0fe957a/ortto_flutter_sdk_android/android/build.gradle#L50
Many thanks

Hi @rlemasquerier you're absolutely spot on here. The latest version of the flutter SDK includes updates to our Android SDK (v1.5.1) . This latest update fixes an issue where deep links without a in-app notification were sometimes causing a crash.

@scrummitch Great, thank you 👍 !


[Umbrella issue] modernize build system

[x] switch to version catalog
[x] https://github.com/apollographql/apollo-kotlin/issues/3531
[x] #4335
[x] enable build cache
[ ] #4416

Closing in favor of https://github.com/apollographql/apollo-kotlin/issues/4416
Note we can monitor our build system at ge.apollographql.com


Cookbook documents its attributes

Seek out attribute access, and insist they are mentioned in the README.
Several gross things here make this hard to implement - ideas welcome:

We'll never detect all attribute accesses, as it is easy to do a set-by-hash, etc.
I'm glossing over what is an "attribute name"
Some people document attributes as node[foo], some as default[foo], some with symbols
Really, unless there is a set format, parsing the README to find an attribute mention may be hopeless.
I personally like to mention when I am setting attributes to be used by a dependent cookbook, but others may want to hide that as an implementation detail.

Could be implemented as a 0..1 metric, with 1 representing 100% documentation.

What does document mean in this case? Is it sufficient to say that it's listed in the README? That's effectively what this check does.

I'd rather say something about ruby source files, like "for every file in attributes/*.rb, each attribute defined there has a comment explaining it." I think trying to parse README.md looking for attribute references in text is going to be a huge bikeshed and won't really benefit us.

Really, unless there is a set format, parsing the README to find an attribute mention may be hopeless.

:+1: :100:


Add Discv4::LookupRandom

Describe the feature
add a lookup_random similar to what geth has: https://github.com/ethereum/go-ethereum/blob/4e1e37323d87967708b58546f040990db7c2820c/p2p/discover/v4_udp.go#L276
probably need one for Discv5 too eventually - https://github.com/ethereum/go-ethereum/blob/4e1e37323d87967708b58546f040990db7c2820c/p2p/discover/v5_udp.go#L300
Additional context
needed for reth crawler similar to geth's https://github.com/ethereum/go-ethereum/blob/master/cmd/devp2p/discv4cmd.go#L225

@mattsse as discussed offline, im up to take this on


[stable/datadog] Fix DataDog location for useConfigMap in docs

This corrects the location for useConfigMap to be agents.useConfigMap (plural agents) instead of agent.useConfigMap.

/assign @clamoriniere


[FLINK-9900][tests] Include more information on timeout in Zookeeper HA ITCase

What is the purpose of the change
With this PR we include more debugging information in the error message if the ZookeeperHighAvailabilityITCase times out.
Brief change log

remove Timeout annotation to a) deduplicate timeout logics and b) allow addition of new logic
add utility method creating thread dump
on failure to retrieve job information or if the job isn't in FINISHED state, print more debug info
stop polling state when reaching any terminal state, to prevent loops if the job reaches a FAILED state (for whatever reason)

+1

hi @zentol , checkstyle error :
src/test/java/org/apache/flink/test/checkpointing/ZooKeeperHighAvailabilityITCase.java:[312] (regexp) RegexpSingleline: Trailing whitespace


Redmine upload files forbidden.

Hello Guys,
I have next env:
modsecurity-crs (3.0.0-3~bpo8+1) (debian  jessie-backports)
libapache2-mod-security2 (2.9.1-2~bpo8+1) (debian  jessie-backports)
protected app: redmine (http://www.redmine.org/)

When I'm trying to upload file(jpg or pdf or mp3 or any) I have error and next modsec_audit.log
Message: Warning. Match of "rx ^%{tx.allowed_request_content_type}$" against "TX:0" required. [file "/usr/share/modsecurity-crs/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "995"] [id "920420"] [rev "2"] [msg "Request content type is not allowed by policy"] [data "application/octet-stream"] [severity "CRITICAL"] [ver "OWASP_CRS/3.0.0"] [maturity "9"] [accuracy "9"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "OWASP_CRS/POLICY/ENCODING_NOT_ALLOWED"] [tag "WASCTC/WASC-20"] [tag "OWASP_TOP_10/A1"] [tag "OWASP_AppSensor/EE2"] [tag "PCI/12.1"]
Message: Access denied with code 403 (phase 2). Operator GE matched 5 at TX:anomaly_score. [file "/usr/share/modsecurity-crs/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "57"] [id "949110"] [msg "Inbound Anomaly Score Exceeded (Total Score: 5)"] [severity "CRITICAL"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"]
Message: Warning. Operator GE matched 5 at TX:inbound_anomaly_score. [file "/usr/share/modsecurity-crs/rules/RESPONSE-980-CORRELATION.conf"] [line "73"] [id "980130"] [msg "Inbound Anomaly Score Exceeded (Total Inbound Score: 5 - SQLI=0,XSS=0,RFI=0,LFI=0,RCE=0,PHPI=0,HTTP=0,SESS=0): Request content type is not allowed by policy"] [tag "event-correlation"]

--57d53c62-B--
POST /uploads.js?attachment_id=1&filename=11-21-56%2010.05.2017.jpg HTTP/1.0
Host: redmine.site.ru
X-Real-IP: xxx
X-Forwarded-For: xxx
X-Forwarded-Proto: https
Connection: close
Content-Length: 11824
Accept: application/js
Origin: https://redmine.site.ru
X-CSRF-Token: c9dyNKvPCcX3pZ7OisJ7CEmYdljhAq+K2RNoq7lIC1I=
User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.82 Safari/537.36
X-Requested-With: XMLHttpRequest
Content-Type: application/octet-stream
DNT: 1
Referer: https://redmine.site.ru/issues/41
Accept-Encoding: gzip, deflate, br
Accept-Language: ru-RU,ru;q=0.8,en-US;q=0.6,en;q=0.4
Cookie: redmine_dtkeeper_ru=18943202e6f77d4a39e5eccfbaf75659; _redmine_session=BAh7DEkiDHVzZXJfaWQGOgZFRmkNSSIKY3RpbWUGOwBGbCsHNK83WUkiCmF0aW1lBjsARmwrB3RGQllJIg9zZXNzaW9uX2lkBjsAVEkiJTQ3YzAxOWExMzQ5MTk1ZDE1ZGM5MDgyOTNlOTFjMjJiBjsAVEkiEF9jc3JmX3Rva2VuBjsARkkiMWM5ZHlOS3ZQQ2NYM3BaN09pc0o3Q0VtWWRsamhBcStLMlJOb3E3bElDMUk9BjsARkkiCnF1ZXJ5BjsARnsJOg9wcm9qZWN0X2lkaQk6DGZpbHRlcnN7B0kiDnN0YXR1c19pZAY7AFR7BzoNb3BlcmF0b3JJIgZvBjsAVDoLdmFsdWVzWwZJIgAGOwBUSSIPdHJhY2tlcl9pZAY7AFR7BzsISSIGPQY7AFQ7CVsGSSIGNgY7AFQ6DWdyb3VwX2J5MDoRY29sdW1uX25hbWVzMEkiFmlzc3Vlc19pbmRleF9zb3J0BjsARkkiDGlkOmRlc2MGOwBU--9b4651a35d4b50536fec8cdf2ff3bf6d3d659a5e

This is strange because the content type application/octet-stream is allowed in the default REQUEST-901-INITIALIZATION.conf file. Can you check the rule in your crs-setup.conf (https://github.com/SpiderLabs/owasp-modsecurity-crs/blob/v3.0/master/crs-setup.conf.example#L336) and REQUEST-901-INITIALIZATION.conf#L158 and make sure application/octet-stream is included in the list? If the crs-setup.conf rule is commented, it means you just need to ensure the content type is available in the initialization file.

Hi @oleg9301  and @void-in, I am going to close this issue as it is related to OWASP CRS. @oleg9301 if you are still on trouble you may want to open the issue here: https://github.com/SpiderLabs/owasp-modsecurity-crs


[AutoDiff] Add SR-12656 negative test.

Add negative test for SR-12656: differentiation transform crash for opaque result types.
This issue is lower-priority, since afaik no one is using differentiation with opaque result types.

@swift-ci Please smoke test

Merging since this NFC patch adds only a negative test.
I added reviewers to inform you about the SR-12656 crasher.


fs_readdir/fs_closedir race condition leads to use-after-free

Version: >= 1.28.0
Platform: Linux, should also affect Windows but haven't tested

If uv_fs_closedir is called from within a uv_fs_readdir callback (as is done in test-fs-readdir.c), it is not guaranteed that the uv_fs_closedir will happen after the uv_fs_readdir req has been cleaned up. If the close does happen before the uv__fs_readdir_cleanup, then there is a use-after-free that happens in uv__fs_readdir_cleanup, as it accesses dir->dirents (but dir would have already been freed in uv__fs_closedir).
I was able to trigger this by running UV_USE_VALGRIND=1 ./uv_run_tests fs_readdir_empty_dir multiple times (took like 10 runs for me to trigger it once). Here's the output, with some debug prints to show the order of readdir/closedir:
not ok 1 - fs_readdir_empty_dir
# exit code 125
# Output from process `fs_readdir_empty_dir`:
# uv__fs_readdir_cleanup (dir: 0x5c681c0)
# uv__fs_closedir (dir: 0x5c681c0)
# uv__fs_closedir (dir: 0x5c70840)
# uv__fs_readdir_cleanup (dir: 0x5c70840)
# ==16925== Invalid read of size 8
# ==16925==    at 0x4E49BB7: uv__fs_readdir_cleanup (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E52B65: uv_fs_req_cleanup (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x42C765: empty_readdir_cb (in /home/ryan/Programming/libuv/out/cmake/uv_run_tests)
# ==16925==    by 0x4E4FEA5: uv__fs_done (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E44E27: uv__work_done (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E4A899: uv__async_io (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E62CD0: uv__io_poll (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E4B3C7: uv_run (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x42CCB4: run_test_fs_readdir_empty_dir (in /home/ryan/Programming/libuv/out/cmake/uv_run_tests)
# ==16925==    by 0x40A77A: run_test_part (in /home/ryan/Programming/libuv/out/cmake/uv_run_tests)
# ==16925==    by 0x40919E: main (in /home/ryan/Programming/libuv/out/cmake/uv_run_tests)
# ==16925==  Address 0x5c70840 is 0 bytes inside a block of size 56 free'd
# ==16925==    at 0x4C2EDEB: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
# ==16925==    by 0x4E45C2E: uv(float, long double,...)(...) (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E4E21B: uv__fs_closedir (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E4FC4A: uv__fs_work (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E445CA: worker (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x527C6B9: start_thread (pthread_create.c:333)
# ==16925==  Block was alloc'd at
# ==16925==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
# ==16925==    by 0x4E45BFF: uv__malloc (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E4DFF2: uv__fs_opendir (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E4FC1C: uv__fs_work (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x4E445CA: worker (in /home/ryan/Programming/libuv/out/cmake/libuv.so.1.0.0)
# ==16925==    by 0x527C6B9: start_thread (pthread_create.c:333)
# ==16925==

So, uv_fs_readdir() is documented as not being thread safe. Looking at the code, it seems like a problem would occur if uv__fs_readdir_cleanup() is called before uv__fs_closedir() (not the other way around) because uv__fs_closedir() doesn't check for req->ptr being NULL, while uv__fs_readdir_cleanup() does. Can you add an if (req->ptr == NULL) return; to the beginning of uv__fs_closedir() and see if that fixes the crash?


trouble migrating from 0.6.8 -> 0.7.0

So, I removed all Reflection references from my code and all Reflection derives from structures interfacing with clickhouse...
However I have one generic writer function that looks something like
pub async fn go<A: Clone + std::fmt::Debug + Send, R: Serialize, F: Fn(A) -> R>(&self,client: Client,map: &F) -> Result<()> {
      let mut inserter = client.inserter(...);
      inserter.write(&map(...)).await?;
      ...
}

now I'm getting this error which I find unintuitive
error[E0277]: the trait bound R: clickhouse::row::Primitive is not satisfied (at the &map function call)
(as clickhouse::row::Primitive trait is private)
What would be the best way to resolve?...
p.s. before R: Serialize used to be R: Reflection + Serialize

replacing Reflection with clickhouse::Row and Reflection derive with Row derive fixed the problem
should'a looked more careful at what changed...


Painful to use with Dgraph

Is your feature request related to a problem? Please describe.
Dgraph is kinda nice, in that it takes a schema and augments it with built-in resolvers that generally just work.  However,
using Dgraph with genqlient is painful.
The specific problems arise with not omitting empty values.  While this is painful for ints, and mostly painful for strings, it is possible to work around this problem with schema types by insisting on pointers.
However, a filter is defined like this:
input ImageFilter {
  id: StringHashFilter
  name: StringExactFilter
  digest: StringExactFilter
  has: [ImageHasFilter]
  and: [ImageFilter]
  or: [ImageFilter]
  not: ImageFilter
}

input StringHashFilter {
  eq: String
  in: [String]
}

input StringExactFilter {
  eq: String
  in: [String]
  le: String
  lt: String
  ge: String
  gt: String
  between: StringRange
}

input StringRange {
  min: String!
  max: String!
}

enum ImageHasFilter {
  id
  name
  digest
  scans
  deployments
}

This query:
query queryImage($filter: ImageFilter!) {
    queryImage(filter: $filter) {
        id
        name
        digest
    }
}

produces a request like this:
{
  "query": "\nquery queryImage ($filter: ImageFilter!) {\n\tqueryImage(filter: $filter) {\n\t\tid\n\t\tname\n\t\tdigest\n\t}\n}\n",
  "variables": {
    "filter": {
      "name": {
        "eq": "moby/buildkit",
        "in": null,
        "le": "",
        "lt": "",
        "ge": "",
        "gt": ""
      },
      "digest": {
        "eq": "sha256:890dcae054e3039f6c6b76bf0da80a130fa6e6bb1f3624063ef0210ac2c57b06",
        "in": null,
        "le": "",
        "lt": "",
        "ge": "",
        "gt": ""
      },
      "has": null
    }
  },
  "operationName": "queryImage"
}

This will effectively break the query as it is asking for a silly request which is not self-consistent: asking for something that is eq to a string and also le, ge, lt, and gt the empty string makes no sense.
Describe the solution you'd like
I really don't know what solution I'd like, other than to be able to (perhaps by-type) handle strings as pointers.
That, or a way to indicate what I'd like to have rendered somehow...
Describe alternatives you've considered
I can write my query to take the name and digest, but this creates a lot of "queryByNameAndDigest()" type almost-identical queries.
Additional context
This is just one example of where rendering too much JSON gets me into trouble with Dgraph...

Hmm, this seems similar to some of the issues that have come up with Hasura. Maybe take a look at #272 and (a) see if any of the existing config options or workarounds discussed there work for you and (b) comment on what overlaps or differs? Unfortunately I haven't had time to look back at the latest discussion there but I think there are some good ideas.