Update faq.md document

补全文档
Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

更新文档


[CODE REFORMAT] incorrect reformat for object and array children

suppose you have following code:
var arr = [{a:1},{a:2},{a:3}];

You could separate it to several lines of code. In this case you expect code like this:
var arr = [
    {
        a:1
    },
    {
        a:2
    },
    {
        a:3
    }
];

actually after reformat the result is like this:
var arr = [
{
a:1
},
{
a:2
},
{
a:3
}
];

possible fix: https://github.com/TiVo/intellij-haxe/pull/135

Since pull request #135 was merged as a fix, closing this bug.  Please re-open if the error recurs.


Connecting to cloud-sql using private-ip sometimes fails with a TLS handshake timeout

Bug Description
I am running v2.9.0/cloud-sql-proxy.linux.amd64 on a GCE n1-highmem-4 instance that is started through dataflow. The binary runs inside a u22 base image container.
As part of the container entry point, I run the following script:
CLOUD_SQL_PROXY_INSTANCES="<db-instance-name>"

/bin/cloud_sql_proxy --private-ip -u /cloudsql ${CLOUD_SQL_PROXY_INSTANCES} &

# Wait for all instances
for instance in ${CLOUD_SQL_PROXY_INSTANCES//,/ } ; do
    while ! pg_isready -h "/cloudsql/${instance}" ; do
        echo "Waiting for instance ${instance} to be online"
        sleep 1
    done
done

Occasionally, I will get the following error:

2024/04/21 01:34:43 [$DB-STRING] could not resolve instance version: failed to get instance: Refresh error: failed to get instance metadata (connection name = "$DB-STRING"): Get "https://sqladmin.googleapis.com/sql/v1beta4/projects/$PROJECT/instances/$DB/connectSettings?alt=json&prettyPrint=false": net/http: TLS handshake timeout

2024/04/21 01:34:43 The proxy has encountered a terminal error: unable to start: [$DB-STRING] Unable to mount socket: failed to get instance: Refresh error: failed to get instance metadata (connection name = "$DB-STRING"): Get "https://sqladmin.googleapis.com/sql/v1beta4/projects/$PROJECT/instances/$DB/connectSettings?alt=json&prettyPrint=false": net/http: TLS handshake timeout

And then the script gets stuck in an infinite loop, because cloud-sql-proxy quits instead of trying to connect again on the next attempt. Some of it is my fault, I should be using a process manager, but the timeout is unexpected.
The gce instance should not be throttling, and it runs in the same region as the cloud-sql instance. I do not know how to check if the other side of the auth-proxy is having issues. A lot of other instances are also connecting to is (mostly GAE), and I also see these similar issues on them occasionally.
PS: This is a new issue as a follow up for this comment I posted in a different issue: https://github.com/GoogleCloudPlatform/cloud-sql-proxy/issues/2081#issuecomment-2070406583
Steps to reproduce?
No easy reproduction steps, since this happens occasionally (~a few times a week).
Environment

OS type and version: Ubuntu 22
Cloud SQL Proxy version (./cloud-sql-proxy --version): v2.9.0
Proxy invocation command (for example, ./cloud-sql-proxy --port 5432 INSTANCE_CONNECTION_NAME): /bin/cloud_sql_proxy --private-ip -u /cloudsql $INSTANCE &

Thanks @akshetpandey.
The root problem seems to be this:
Get "https://sqladmin.googleapis.com/sql/v1beta4/projects/$PROJECT/instances/$DB/connectSettings?alt=json&prettyPrint=false": net/http: TLS handshake timeout

The SQL Admin API call isn't responding and the Proxy dies. We recently added retry support for 50x responses here: https://github.com/GoogleCloudPlatform/cloud-sql-go-connector/pull/781. I wonder if we should extend that to include more generic TLS errors.

I ended up changing my script to check if the pid is still alive and to restart it if it is not. An internal retry will definitely address the issue too.
I do want to add that something seems fishy here. Not sure if its the container, dataflow, sql admin, network routing, dns, or something else, but the error happens way too frequently.
I don't have concrete data but the failure rate I am seeing implies that sqladmin.googleapis.com has an uptime <99% in this particular situation.
Do note that this isn't the first request made in the flow. My script successfully hits the metadata server first and then this fails.

What kind of CPU usage do you have on this instance? Wondering if this is a client error.

n1-highmem-4, cpu usage at that point is pretty low.

Cpu usage is down to under 10% when we start cloud-sql-proxy. Let me know if you want more logs. I am not sure what is available, but it will be all gone in 20 days.

How many instances are you connecting to in your script?

Just the 1

I know we have seen some interesting behavior with dataflow in the past...
I'll try and see if I can get a long-running dataflow setup spun up to see if I can get a repro of the error.
Otherwise we can take a look at extending retries to TLS handshake errors as well.

Do note, I haven't run into this issue since upgrading to 2.11.3 in may.

Do note, I haven't run into this issue since upgrading to 2.11.3 in may.

In that case I will close this out for the time being. If I see anything come up in my test or if you see the issue again please re-open this and we will take another look 😄


docs(readme) DRY License

DRY License by linking to it, rather than duplicating part of it.

Nevermind, closing, going to do a different adjustment in a different PR...


Download release notes tool binary directly

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We now directly download the binary instead of using go install, which should speed-up the CI.
Which issue(s) this PR fixes:

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

None

/lgtm


Disable Clouds > Tenants view in Roles

Roles selection for Clouds > Tenants doesn't exist.

@dclarizio @h-kataria Please review.

@dclarizio Still an issue with the new UI?

@chrisarcand wow, I'm not even exactly sure that the issue is/was
@penoycentral can you look at the latest master of ManageIQ and maybe give a bit more detail on what you think is missing, if this issue is still there?  Thx, Dan

Looks like it was already fixed. Thanks.


fix: set filter badge top position based on offset height -v32 (#301)

Fixes [DHIS2-6631]
The top position of the filter badges should be based on the height of the dashboards bar, so that the dashboards bar doesn't hide the badge if its been configured to more than one row

This one is on hold because of icon alignment issues described here: https://github.com/dhis2/ui-core/issues/194


Cannot save or import a macro with GSStaticPriority

Does priority list work?  When i select it from the drop down box on the screen, hit save... come back, the box is sometimes blank and sometimes just back to sequential.  Additionally I cannot uncheck any of the use item boxes and the macro runs even when im not in combat.

This will be in for Progress 4+

Thank you.  Honestly... can't tell you how much I appreciate this add on.


[Module Proposal]: Deployment delay

Have you checked this module does not already exist in the registry?
Yes
Why is the module needed?
When deploying some Azure services together that use AzureAD for RBAC, it is common to receive errors regarding propagation time.

"Caller is not authorized to perform action on resource. If role assignments, deny assignments or role definitions were changed recently, please observe propagation time."

An example is when Azure Application Gateway is created that uses Key Vault for its SSL Certificates. On some tenants, success rate is at about 20%.
Module path
deployment-scripts/wait
Describe the module
This new module would create a sleep for a configurable number of seconds to be used in other resources DependsOn in order to delay resource creation. EG. wait.bicep

Approved. We definitely have some reservations about the pattern, but don't want to block the creation of the module if you plan to use it.


Register

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/11462
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" 2334b8ff40bc8a69d6d226381f73b06e650e64d0
git push origin v0.1.1


Platform logo looks bad on eclipse / win

From @hendrikebbers on April 5, 2018 9:16
This is how the logo looks like:

Copied from original issue: canoo/dolphin-platform#895

Same for logging...


Would D leak knowledge to G

Dear Sir:
Thanks for your elegant work. But I still have some questions about the paper.  In the flow of attack generation, I saw the mask is generated by D and then multiplied by noise from G.  However, the GAN paradigm is in the outer supervision. Would this design leak discriminator knowledge to the generator? As you know, in the competition between G and D, once either G or D know its opponent's knowledge，only two results would happen 1 the knowledge is soon abandoned 2 the competition would collapse.  I know KeepTopK to some extent is to avoid this catastrophe, but I think the visualization shows it doesn't make sense.
If I misunderstand it, please correct my understanding. Thanks.

Sorry, it seems that your question lay the keys to itself already so I don't get the point. I guess you want to improve the way to find more aggressive adversarial pixels or find some insights about the layout of themselves. For the former, although the performance of our method is good, it still has space to be modified including the issues you gave. For the latter, I hope Figure 5 in the paper might give you some help.

Thanks for your reply. I recently do some experiments on this and find how to avoid knowledge disturbance is useful. Besides, I am curious about using attribute supervision to guide attacks. This generation is similar to the text-guided image synthesis, such as [1] but still something different. Can you give me some paper references to explore this idea in Reid.
[1] Zhou X, Huang S, Li B, et al. Text guided person image synthesis[C]//Proceedings of the IEEE/CVF Conference on Computer Vision and Pattern Recognition. 2019: 3663-3672.

Tbh, it is a little beyond the duty of this repository. But I read some papers[1][2] before, which may be related to what you said and cited. Hope these help.
[1] Men, Yifang, et al. "Controllable person image synthesis with attribute-decomposed gan." Proceedings of the IEEE/CVF Conference on Computer Vision and Pattern Recognition. 2020.
[2] Yin, Zhou, et al. "Adversarial attribute-image person re-identification." arXiv preprint arXiv:1712.01493 (2017).


have Settings.add_json_panel return the panel

It would be useful to have add_json_panel return the panel, just as create_json_panel does.  Right now there's no easy way to access panels of the Settings widget (by name).

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

We are closing some of the stale issues, please create a new issue/pr if you'd like to work on this feature, or get in touch with us on #kivy-dev.


TypeError: undefined is not an object (near RNLocation.configure({..}).then(...))

I have tried rebuilding but the error still persists on iOS. The package works flawlessly on Android though.
Error screenshot:

Code:
  handleGetDirections = () => {
    var directionsData;
    RNLocation.configure({
      distanceFilter: 5.0,
      allowsBackgroundLocationUpdates: true
    })
      .then(() =>
        RNLocation.requestPermission({
          ios: 'whenInUse',
          android: {
            detail: 'fine',
            rationale: {
              title: 'Location permission',
              message: 'Your location is essential to request help',
              buttonPositive: 'OK',
              buttonNegative: 'Cancel'
            }
          }
        })
      )...

I'm also getting this same issue.
Works fine on Android but not iOS

I am getting the same error. Android Works perfect.

I am experiencing the same error in iOS.  Followed directions and used code from the repo.  Please advise.

react-native-geolocation-service is an alternative solution, which gave me no errors at all

We saw this also, I believe configure in https://github.com/timfpark/react-native-location/blob/d215c8c549330c677af9e30d6752768e6275e169/ios/RNLocation.m needs updated to return a promise.

configure does not return a Promise. Just call it once before you use any other method in the library.

@matt-oakes Awesome! I’d propose updating https://github.com/timfpark/react-native-location/blob/9a1c5bf3f683c78de8e82b2820d284e543ef9ab0/example/App.js to reflect   this then.  The reader appears accurate.

Done!


Documentation Audit

[ ] Full public API DocC coverage
[ ] Refactor all SwiftLint errors and warnings

Special DocC Pages

[ ] Explain in detail: Game Lifecycle
[ ] Explain in detail: Window Management
[ ] Explain in detail: Entity, Component, System
[ ] Explain in detail: Resource Loading, Lifecycle (Cache)
[ ] Explain in detail: Custom File Importers
[ ] Explain in detail: Audio Playback
[ ] Explain in detail: Rendering
[ ] Explain in detail: UserInput

I'm still interested in this project, but the lack of docs just makes it a nonstarter (for me at least). It's a shame, as the project seems ideal otherwise.
Personally, I want teach my son programming (by hacking on games), and teach myself various things, like 3D graphics (so I can document the knowledge and monetize the docs). So, we don't need the most mature and featureful engine (we only care about macOS and HTML5, for example). In short, we'd be willing guinea pigs.
Without committing to anything in advance, I'm generally open to helping with the project's docs, but it'd take a lot of time for me to figure out and document the project with only the demos to go on. If there was more to work from already, the whole proposition would be more promising.
Best wishes with your project, whatever happens. It's super impressive how far you've come with it.


feat(compiler): basic parser interface implemented

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#26
#25
#24
#23  👈
#22
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @delta1024 and the rest of your teammates on  Graphite

Merge activity

Jun 11, 8:42 PM EDT: @delta1024 started a stack merge that includes this pull request via Graphite.


Checklist to ship the OKD bits

We created the OKD bundles and did a basic testing but still we are not able to release the crc with OKD. We don't want to confuse user if we ship one crc binary with OCP and one with OKD. During the internal discussion @cfergeau suggested better to use preset issue https://github.com/code-ready/crc/issues/3101 and have it implemented. This issue just a checklist what all we have to do to make it happen.

[x] OKD preset https://github.com/code-ready/crc/issues/3101
[ ] Document update around OKD preset
[x] Internal CI job to create OKD bundle

https://crc.dev/blog/posts/2022-08-22-okd-on-crc/ is already published how to use crc with okd. we need to update some parts of our doc, https://crc.dev/crc/#minimum-system-requirements_gsg for example

hello, What task is pending to close this issue. I see after reading https://github.com/crc-org/crc/issues/3271#issuecomment-1265235599 documentation effort is also complete, right?

Yes, closing this one.


Build error with gcc-14

On Debian, pyregion now fails to build from source because of the standard use of gcc-14:
pyregion/_region_filter.c: In function ‘__pyx_pymod_exec__region_filter’:
pyregion/_region_filter.c:13000:3: error: returning ‘void *’ from a function with return type ‘int’ makes integer from pointer without a cast [-Wint-conversion]
13000 |   import_array();
      |   ^~~~~~~~~~~~
error: command '/usr/bin/x86_64-linux-gnu-gcc' failed with exit code 1

This was reported as Debian#1075410.
If pyregion still has some future, I would try to fix this; however the last commit here was now 2 years ago (@larrybradley was the last committer); I have some doubts. Any advice?

Should be fixed in pyregion 2.3.0 (just released)


Document the code and database deploy process

This needs formal documentation in the README or a docs/ folder (it should be version-controlled with the code, and not in a wiki, so that any version of code will always have the right docs).
Scratch notes

two main areas of work, that should be standalone:

scrapers (data only)
councilmatic (data and UI)

the API (toronto-ocd-api.herokuapp.com) sits in between each work area. So we can modify scrapers, scrape new data into the generic OCD database, then push data into the OCD API. Then, councilmatic has its own database, and has a command to pull and transform data into the format that it likes, having been pointed at the OCD API. So the OCD API and its database is like the master database with total information in a very general format (a format shared with all cities and provinces and federal "deliberative assemblies" that OCD applies to). the councilmatic database is generated from the OCD database (pulled out of db through API), and is transformed into a format more sensible for city councils. it's mostly the same, but not exactly.

the process i've been following is ideally like this:

fork opencivicdata/scrapers-ca to patcon/scrapers-ca
create git feature branch in my repo
hack on scrapers on that branch
run scrapers locally until satisfactory
run a scrape locally, then push that local database to toronto-ocd-api.herokuapp.com app

in tandem create a pull request with my feature branch to get those changes into opencivicdata/scrapers-ca

fork civictechto/tor-councilmatic to patcon/tor-councilmatic
create git feature branch in my repo
hack on tor-councilmatic on that branch
run loaddata (populates my local db with api data) and work locally until satisfactory
run loaddata locally, then push that local database to tor-councilmatic-staging.herokuapp.com

in tandem, create pull requests with any changes for all the component pieces (ie. tor-councilmatic, django-councilmatic, etc)

So you can probably see how pushing data to the API is kinda like a breakpoint, where the process is similar before and after, and people can work separately without treading on one another's toes

4 scrapers exist right now, and there are some other ideas for new ones:

[x] people
[x] committees
[x] bills, ie agenda items (scrapes vote records. very long-running, so probably wouldn't run daily)
[x] events-incremental (scrapes events 2 weeks before/after current date, creating new bills and events as encountered. can be run daily.)

[ ] add vote data. this would likely be in the decision docs, in plaintext that would need to be parsed. sometimes we'd get per-councillor counts, which would eventually be in the vote record (linked above), although with quite some delay. but in the decision docs is the only place where we get pass-fail info per-agenda-item/bill for items where a per-councillor vote is not taken. plus, it will be useful in some cases to not have to wait for the delayed vote result to show up in the vote record. (for example, today is monday, and last thursday's executive committee vote results are in vote records, but friday's city council meeting vote results are not, even though committee minutes have the tabulated vote results -- this is an extra 4 days delay, which can be important to aspiring real-time applications, which are probably the most exciting app ideas)

[ ] events (could scrape meeting schedule for full session. would need to take into account that cancelled/changed meetings don't show up here.)

by the design of the scraper tool (pupa), "objects" (person, organization, bill, event, etc) can be scraped once per scraper. So if there were two sources of rganizerions

Scraping notes
https://github.com/dougestey/toronto-city-hall-api/wiki/Thoughts-on-scraping-systematically-&-useful-endpoints
https://github.com/dougestey/toronto-city-hall-api/wiki/Scraping-Bill-Versions
https://github.com/dougestey/toronto-city-hall/wiki/Scraping-vote-record-history-from-agenda-items


Remove duplicate tests

Resolves #4232
Snippet used
import xml.etree.ElementTree as ET

def remove_duplicate_test_tags(xml_file):
    tree = ET.parse(xml_file)
    root = tree.getroot()
    test_case_names = []
    for test in root.findall('test'):
        test_case_name = test.find('testCaseName').text
        if test_case_name in test_case_names:
            root.remove(test)
        else:
            test_case_names.append(test_case_name)
    head_strings = '''<?xml version='1.0' encoding='UTF-8'?>
<!--
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#      https://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
-->
'''
    #add head strings to the xml file
    tree.write(xml_file)
    with open(xml_file, 'r+') as f:
        content = f.read()
        f.seek(0, 0)
        f.write(head_strings)
        f.write(content)

if __name__ == '__main__':
    remove_duplicate_test_tags('playlist.xml')

Signed-off-by: ArkaprabhaChakraborty<EMAIL_ADDRESS>

Check which one? :)

Wait wait wait you didn't tell me which script?????

Yes, please add the script to https://github.com/adoptium/TKG/tree/master/scripts. See the issue that Shelley opened https://github.com/adoptium/TKG/issues/392


Cordova support

Hi, it is possible to use shaka player with cordova . Idea is to bypass the missing mse in ipad/iphone browser , because cordova inject his implementation.

We have never tested with Cordova, but you can load this page to get a full report:  https://shaka-player-demo.appspot.com/support.html
Please paste the contents of that page into a reply so we know what does or doesn't work!

Closing due to inactivity.  We can reopen at any time if you need further assistance.


can not connect to sql server 2008 in mac?

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
I can not connect to SQL Server 2008 in mac .
and I can connect it in windows using same code.
I am using asp.net core 7.0 razor pages.
mac version is 11.6.5.
and I can connect to SQL Server 2008 using dbms tools.
Expected Behavior
connect well.
Steps To Reproduce
No response
Exceptions (if any)
SQL Exception:
A transport-level error has occurred when receiving results from the server. (provider: TCP Provider, error: 2 - Connection was terminated: Connection was terminated)
.NET Version
7.0.102
Anything else?
No response

Hi @ljzj2, as @ErikEJ mentioned Sql Server 2008 has reached end of support and you can see the list of supported versions here.  We would recommend that you upgrade to a newer supported version and try again.

Also, are you using Microsoft.Data.SqlClient or System.Data.SqlClient?

@ErikEJ @lcheunglci  thank you for your answer. I know SQL Server 2008 has reached end of support. I use it for cheap price some provider still use it. microsoft azure is a little expensive for me student.

@lcheunglci  I am using asp.net core 7.0 + ef core . not Microsoft.Data.SqlClient or System.Data.SqlClient.

I use Azure Data Studio in mac and it can connect to SQL Server 2008
that what I have tried.

@ljzj2 try with the developer edition of SQL server 2019 (it is free) for testing purposes and see if you are able to get connection. If you are still unable to connect, we can start investigating the issue, otherwise because that version is not supported anymore there is not much we can do.
If the issue persists with SQL server 2019 provide a simple repro application.

@lcheunglci I am using asp.net core 7.0 + ef core . not Microsoft.Data.SqlClient or System.Data.SqlClient.

Microsoft.EntityFrameworkCore.SqlServer has a dependency to M.D.SqlClient 5.0.1

same code in windows I am able to get connection so I think it is mac issue.
not sql server.
now I close this issue and I give up and I will go to use windows to develop.
and can you tell me what is difference of express edition and developer edition of SQL Server?

and can you tell me what is difference of express edition and developer edition of SQL Server?

SQL Server Developer edition lets developers build any kind of application on top of SQL Server. It includes all the functionality of Enterprise edition, but is licensed for use as a development and test system, not as a production server. SQL Server Developer is an ideal choice for people who build and test applications.

whereas

SQL Server Express edition is the entry-level, free database and is ideal for learning and building desktop and small server data-driven applications. It is the best choice for independent software vendors, developers, and hobbyists building client applications. If you need more advanced database features, SQL Server Express can be seamlessly upgraded to other higher end versions of SQL Server. SQL Server Express LocalDB is a lightweight version of Express that has all of its programmability features, runs in user mode and has a fast, zero-configuration installation and a short list of prerequisites.

you can read more on this article.


home-> My documents->My project->create project:

Add estimate to forbid to forbid to create a empty name project,and forbid to create the project which name is already exist.
I think it is a bug that created a empty name project.

Thanks, this is pushed: https://github.com/cloudera/hue/commit/abd18d832a24f0c764155ebeebf26f73b9edf5fa
For the next time, please beware to follow the same indentation as in the file. I fixed it and simplify the logic a bit:
https://github.com/cloudera/hue/commit/5f9e9bec46202102a25cb3ddf6da681ddf6a0103


handle close more gracefully

right now it crashes on a bad match and my logs have a lot of entries like this:
2015-07-13 11:54:01.305 [error] emulator Error in process <0.16957.914> on node<EMAIL_ADDRESS>with exit value: {[{reason,{badmatch,{error,closed}}},{mfa,{bullet_handler,info,3}},{stacktrace,[{bullet_handler,reply_get_mode,3,[{file,"src/bullet_handler.erl"},{line,201}]},{bullet_handler,info,3,[{file,"src/bullet_handler.erl"},{line...

I would do the change, it's matching against close here: https://github.com/extend/bullet/blob/master/src/bullet_handler.erl#L201
the thing is that I don't know what to do when I get a close, maybe add a new return value and match it here: https://github.com/extend/bullet/blob/master/src/bullet_handler.erl#L112 and then stop?

I think returning ok makes the handler to shutdown, by looking here: https://github.com/ninenines/cowboy/blob/79aa25cc03518a41351d9f5b074fed065e5713e0/src/cowboy_handler.erl#L226

Yep return ok. PR welcome. :-)

Closing this in favor of PR. Thanks!


Explicit unary constructors

Explain the implicit conversions danger, and the use of 'explicit'.
This issue was replicated from the old gitlab repo and was initially created by @chavid

Done.


Fix golint on systems with selinux enforcing

We run golangci-lint in a container with the local directory bind-mounted as a volume. However, without relabelling the directory
contents, selinux will disallow access. We can use the :Z volume option to ensure relabelling occurs.
See docs like https://docs.docker.com/storage/bind-mounts/#configure-the-selinux-label and https://docs.podman.io/en/latest/markdown/podman-run.1.html
On systems without selinux enforcing, this won't have any affect.
Without this change:
$ ./hack/verify-golint.sh
2022/03/28 11:59:12 Failed to write README file to cache dir /cache: stat /cache/README: no such file or directory

/kind bug

/hold
Let's see what approach kubernetes/test-infra#25798 settles on

/ok-to-test

Let's see what approach kubernetes/test-infra#25798 settles on

Ok, that eventually merged with the --security-opt="label=disable" approach which turns off label separation, rather than the relabelling approach I originally proposed
Updated now, and should be good to merge!
/hold cancel

I'll approve this one, and leave someone else to check and LGTM.
/approve


How to attach NLog instead of log4net

My application throwing exceptions during network loosage.
Here you write how to attach a logger into the library: https://github.com/OpenTl/OpenTl.ClientApi/wiki/How-to-attach-a-logger
But this example is only for log4net. Culd you provide a way how to implement NLog as well?

@dojo90 library supports only log4net


Define next steps for project - Martin

[x] Break down in smaller steps
[ ] Put list of smaller steps in comments and wait for someone to verify
[ ] Make tickets of the smaller steps after verification
to the verificator please close issue and move to done if you approve of the steps or leave feedback and put in progress

Time Series Database - Influx
Time Series Database - ERMs
Implementing ERM to Influx


sshd_stage config: Use a non-0 ClientAliveInterval

A user reported ssh timeouts, chaing the ClientAlivInterval to something else than 0 fixes it.
We default to nil, which I think results in 0 in the osbuild sshd_config stage? Something worth investigating at least. We could use 60/120/180.
That said other platforms have 0 default values as well, and is the standard ssh configuration. Maybe we should think about exposing this, at least onprem?

I always change my value to ClientAliveInterval 60 right away that should be good. But I always forget after modification to do the sudo systemctl restart sshd.service

This is nonzero on azure by default. cc @gicmo @thozza

Not asking to change just recommending that after deployment to update the value.  300 seconds might be a better setting or 5 minutes.

This is nonzero on azure by default. cc @gicmo @thozza

The RHEL GCE image built by Google has the ClientAliveInterval set to 420. The RHEL EC2 images built by us does not change sshd configuration, so they have the default value which comes with the package (which is 0). Also the RHEL KVM Guest image does not have any specific value set.


Pt pt translation_how to

What does this PR do?
Add translation
Why is this valuable (or not)?
Adding translation in Portuguese from Portugal
Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Update object.

Thanks for the PR. object now needs at least rust 1.42, so that will need updating here too. Also, I'll probably do another object release before releasing a new addr2line version, so just curious if there was a reason you needed this updated now?

Thanks for the PR. object now needs at least rust 1.42, so that will need updating here too. Also, I'll probably do another object release before releasing a new addr2line version, so just curious if there was a reason you needed this updated now?

addr2line reexports object, so it's not really pressing

addr2line reexports object, so it's not really pressing


fix: update referendum comments

Close: #1495

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Introduced an "Edited" tag for modified comments, enhancing user context.
Updated comment rendering logic to differentiate based on comment sources.

Enhancements

Improved handling of referendum comments and votes through new data structures and fetching methods.

Bug Fixes

Corrected conditional rendering for comment sources within the comment display logic.

Documentation

Updated interface definitions to include a new optional commentSource property for better categorization of comments.

Still some Unknown NANs


Potential backwards compatibility problem with some APIs

This issue is related to the PR #256 . That PR turns additional entity enums used in the DTOS into strings. We must discuss further whether this can cause a real compatibility problem and, if so, maybe replicate those enums in the API service.
@derekwb  I will leave this issue assigned to myself for now to discuss with @arvindn05 upon his return. We will assign to you in case this is something we must fix.

AcknowledgementStatus, object type need to be moved to common project


Listener, Worker and Reporter for L2 VRF/RR

Description
This Pr Implements offchain oracles for L2 VRF
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

Deployment

[x] Should publish npm package
[ ] Should publish Docker image

Even if there are lots of auto generated codes, pr size is quite huge. It would be nice to review if it could be separated into multiple PRs next time. Thank you 🙏

Even if there are lots of auto generated codes, PR size is quite huge. It would be nice to review if it could be separated into multiple PRs next time. Thank you 🙏

Yes, I understand, the reason I put it in a big PR is everything components are linked together, It's quite hard to actually separate it. Will try to follow the practice next time though.

@KelvinThai Also we need to solve the error in build. Otherwise it will be happening to every new PR.

@KelvinThai Also we need to solve the error in build. Otherwise it will be happening to every new PR.

Sure, some symlinks should solve the problem.

@KelvinThai Also we need to solve the error in build. Otherwise it will be happening to every new PR.

Sure, some symlinks should solve the problem.

I think we should not use symlinks for this. You can deploy new @orakl/contracts package by updating the version inside of package.json for contracts package.


Added 'smooth' tag to all trajopt trajectories.

This PR adds a smooth tag to all TrajOpt trajectories by default.

Looks good to me @psigen, let's roll with it.

Ok, thanks man.


SCSS Color Palette Design - UI/UX Color Scheme

Vote below for the CSS Framework of your choice to be implemented in this project.
To vote click the 👍 and the one with the most thumbs up over the next couple days will get the win!

Vote for Bootstrap 4

Vote for Materialize

Vote for Bulma

Vote for Skeleton

@3daddict I vote Materialize css

Materialize for the win!


New PR to test SignCLA at 2018-04-12 16:30:07.232613937 -0700 PDT m=+6.862507655

Body of PR

unsigned-fail-fork-staging: test succeeded


gns3-gui,gns3-server: 2.2.31 -> 2.2.34

Description of changes

Upgrade GNS3 GUI & Server from 2.2.31 to 2.2.34

https://github.com/GNS3/gns3-gui/releases/tag/v2.2.34
https://github.com/GNS3/gns3-gui/releases/tag/v2.2.33
https://github.com/GNS3/gns3-gui/releases/tag/v2.2.32

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

2 packages built:

gns3-gui
gns3-server


[Color Scheme - Camphell]: Name is not defined updated if user selects any other color.

Windows Terminal version
1.19.2481.0
Windows build number
23529.1000
Other Software
Test Environment:
OS: Windows 11 Version Dev (OS Build 23529.1000)
App: Windows Terminal Preview
Steps to reproduce
**Repro Steps: **
1.Open Windows Terminal.
2.Open Settings page using 'Ctr+,'
3. Navigate to the 'Color Scheme' and Select 'Camphell'.
4. Now navigate to any color and changed it according to your will.
5. Navigate over the color using Narrator and observe the issue.
**User Experience: **
Users who have cerebral palsy and in-active limbs and depend on Voice Access for their accessibility will not be able to access the options with its visible name if the accessible name defined is incorrect.
MAS Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value
Attachment :
Name is not defined updated if user selects any other color..zip
Expected Behavior
Name should get updated if user selects any other color.
Actual Behavior
Name is not defined updated if user selects any other color.

This and #15985 are the same thing - just kinda a misunderstanding of how Terminals work. You're totally free to set "Black"(and the rest of the palette) to whatever color value you want. That however doesn't change the meaning of those color palette entries to CLI applications.
This seems like by design to me honestly.

This is correct. @Priyanka-Chauhan123, "Black", "Red", "White" and the other 13 colors in the Terminal settings are terms of art. The standard language for terminal emulators defines 16 colors that have specific names. The user is allowed to redefine how those names appear, but the names may never change.

@zadjii-msft for 15985, I wonder if it would be possible to indicate "Foreground (Ochre)" or "Background (Vermillion)"?

@carlos-zamora By when it is expected to get fixed, as it is impacting product compliance

@Priyanka-Chauhan123 please see questions above directed towards you. These are not intended to be the names of the user's selected color, they are intended to be the name of the standard "color slot" the user is changing.

@DHowett Does it meant that user will not be able to change the color and it will not be modified. If this is the case, then it should not change the color visibly as well.

@Priyanka-Chauhan123 It is different. In this case, "red" is a symbolic name used as part of an API contract. It's like syntax highlighting, with tokens for "error" or "filename" or "string".
We allow the user to change to meaning of the symbolic constant ("red"), just like Visual Studio lets you change the color of "String", without changing its name. Changing its name would break the API contract.
Does that help?


About Adding Content

I want to add content in readme

I want to add an internship in readme

go ahead :)

Hi,
I want to add an internship in Readme. Can you assign this to me?

Hi, I want to add an internship in Readme. Can you assign this to me?

go ahead :)


Add functionality to check and set git config values

Using j.clients.git.pushGitRepos can error when user.name and user.email are not set.
The git client currently doesn't support setting/getting this in/from the git config.

@rkhamis should this issue be closed now it's in the development branch?


Test and document as frictionless datapackage

Using the python package:

https://github.com/frictionlessdata/datapackage-py#package

Also see pyam docu:

https://pyam-iamc.readthedocs.io/en/stable/api/io.html#the-frictionless-data-package

See #41


Viewing archived images freezes page [BUG]

Environment

Camera:  ZWO
OS: Bookworm
Allsky version: v2023.05.01_04
Pi model and amount of memory (512 MB, 1, 2, 4, or 8 GB) : Raspberry Pi4 4Gb

Bug Description
viewing archived images takes ages when clicking on a thumbnail.

Follow the instructions for Reporting Issues in the Wiki, then ATTACH a copy of the file(s) above.
Do NOT copy/paste them into this Issue.

@penguin242,
This is a known bug when there are a lot of images.
It will be fixed in the release after the next one, when we redo the WebUI.

Update: this will be fixed in the first point release to the next release.


Both machine mac-6 & mac-7 lost external connection from iPhone device.

Both machine mac-6 & mac-7 lost external connection from iPhone device.

Both external connections have been restored by rebooting both machines.


Bnv and fragments

Close #11
Close #32
Close #18
Close #7
Close #36
Imports the following:

import com.google.android.material.bottomnavigation.BottomNavigationView;
import androidx.annotation.NonNull;
import androidx.fragment.app.Fragment;
import androidx.fragment.app.FragmentManager;

Hi @GavinSlusher - are you including all gradle changes on your commits and PRs?
I merged your pull request, but got an error re: not having dependencies.
I got the error specifically on line 17 on MainActivity.java:
import com.google.android.material.bottomnavigation.BottomNavigationView;
Anyway, it was a quick fix, but just wanted to check in to be sure!

Hey Kat - I thought I was! Thanks for the heads up though, I'll double check next time


Backend takes time to start

When I run the command:
make backend-docker-start
The backend takes time to start. It first installs plone.restapi and then starts Plone. Every time I stop the container and start it again this happens. This is correct? Is there a way to start the backend faster? There used to be an api folder that had a Plone buildout, but it was removed.

@wesleybl read on from this comment https://github.com/plone/volto/pull/6110#pullrequestreview-2127049347 for alternatives. You can either use buildout or generate the latest Plone backend snapshot from the cookiecutter.

@stevepiercy @davisagli Volto is very dependent on Plone. I think it would be important to have a script to "install Plone quickly" in the Volto repository. Saying "use cookiecutter or use buildout" may be common for experienced developers but can be a hindrance for beginner developers or those who don't know Plone but want to help with Volto.
Is there any documentation on "How to install Plone for the Volto core"?
Another point is that the Volto CI does not always use the same versions as a Vanilla Plone. For example, a more actual version of pone.restapi is generally used. If these pinns were integrated, it would be easier. It could be that something doesn't work in your installation and works in the CI, or vice versa, precisely because of pinns differences.

@wesleybl Develop Volto core.

@wesleybl Develop Volto core.

@stevepiercy In this documentation, the command to start the backend is exactly the command I'm talking about in this issue:
make backend-docker-start
Another problem with this command is that the data is not persisted. If we start the backend, stop it and then start it again, the data created is no longer there.

@wesleybl I don't have an answer for you, but I put this on the agenda for next week's @plone/volto-team meeting to discuss and get back to you. Alternatively, you could post to the Community Forum to ask for an answer.

Possibly related issue: https://community.plone.org/t/troubleshooting-volto-setup-frontend-configuration-issue/19656

@plone/volto-team it appears our documentation of Develop Volto core is broken. Can someone please help out? I have not verified the issue, and I wouldn't know what to do if I did.

@wesleybl Develop Volto core.

@stevepiercy In this documentation, the command to start the backend is exactly the command I'm talking about in this issue:
make backend-docker-start
Another problem with this command is that the data is not persisted. If we start the backend, stop it and then start it again, the data created is no longer there.

@wesleybl this is true regarding the persistence of data and it's a pity. Perhaps we should modify the make command as to mount a data folder inside the volto monorepo and use it thus persisting the data.
@plone/volto-team if we tell users that this is the way to develop using docker we should make sure we persist their demoing, making it easier to come back and work on other issues.

When I run the command:
make backend-docker-start
The backend takes time to start. It first installs plone.restapi and then starts Plone. Every time I stop the container and start it again this happens. This is correct? Is there a way to start the backend faster? There used to be an api folder that had a Plone buildout, but it was removed.

@wesleybl you could test the backend changes that modelled the cookiecutter output from here
https://github.com/plone/volto/pull/6110/commits/c61a5497ba76f13b610ba60d804c9808569093c4
I would be interested in hearing how that works for you, if api was still better or if this is more flexible for you.
If that is the case then maybe we can add back the backend code that was made ready either way but was deemed as no longer needed to have the non docker way of installing the backend and as such it got deleted.

@wesleybl you could test the backend changes that modelled the cookiecutter output from here
https://github.com/plone/volto/commit/c61a5497ba76f13b610ba60d804c9808569093c4

@ichim-david I didn't understand this commit. Is the link correct? Do you want me to test this PR?

@wesleybl you could test the backend changes that modelled the cookiecutter output from here
c61a549

@ichim-david I didn't understand this commit. Is the link correct? Do you want me to test this PR?

@wesleybl you can test it https://github.com/plone/volto/pull/6168 I already know it will solve your backend-docker-start data persistency issue.
Regarding that commit that I meant to link to https://github.com/plone/volto/pull/6110/commits/aade85d02b8b63d5a514a35434aff5bbfaa609d0 which was the previous commit before the one that removed the backend folder. See the commit after my comment I left here https://github.com/plone/volto/pull/6110#issuecomment-2177843485
Here is the link to how backend folder was modelled before it was deemed as something that can be removed altogether instead of replacing the api folder
https://github.com/plone/volto/tree/aade85d02b8b63d5a514a35434aff5bbfaa609d0/backend
This is the thing that I thought might have been useful for you to test and give feedback if it was a better system rather than the api folder backend.

@ichim-david I tested the backend folder. It worked for me. I prefer her format. It will only take a while for the first installation. Starting will be faster from then on. I will be able to change a file for some testing, something that would not be possible with Docker. I can debug.
But I respect the decision to use docker. For most situations it will resolve. We have the persistence problem that you are verifying. We also have the problem of taking a long time to start. I think it was due to the installation of the "custom version" of plone.restapi. I don't see a solution for this using docker. Maybe we have our dockerfile? But I think the use of a customized version has not been used for a long time. Or is it? This delay may not be that annoying. We usually start the backend and leave it running all the time.

@ichim-david I tested the backend folder. It worked for me. I prefer her format. It will only take a while for the first installation. Starting will be faster from then on. I will be able to change a file for some testing, something that would not be possible with Docker. I can debug.
But I respect the decision to use docker. For most situations it will resolve. We have the persistence problem that you are verifying. We also have the problem of taking a long time to start. I think it was due to the installation of the "custom version" of plone.restapi. I don't see a solution for this using docker. Maybe we have our dockerfile? But I think the use of a customized version has not been used for a long time. Or is it? This delay may not be that annoying. We usually start the backend and leave it running all the time.

@wesleybl it's fast if you have a mac with a fast ssd :), it takes around 10s before starting the plone instance :). This happens due to the use of plone.restapi 9.7.0 for the teaser dynamic data fetching. Once a Plone version is released with this version that step should no longer be necessary to get rid of this bottleneck.
Maybe something can be done for the docker image, not my area of expertise so we shall see, maybe ask @ericof for some input on this issue.
Regarding the backend working I'm glad it works for you, I guess you can use this solution since it's more flexible in modifying the plone instance todo actual changes as opposed to the docker image.
Maybe once cookicutter docs and story is finished it can bring the alternative to the docker image for the Plone instance for the people that need the extra hands-down flexibility.
You could also try to generate the Plone backend using it and add any feedback https://github.com/plone/cookieplone-templates.

You could also try to generate the Plone backend using it and add any feedback https://github.com/plone/cookieplone-templates.

I have reservations about this. To see: https://github.com/plone/volto/issues/6157#issuecomment-2218363372
But it's ok :)

Let me add my 5c to this.
The Plone versions used in Volto core should be "latest", since both backend and frontend have to be "in sync" for features. eg. if main has a feature not yet in an official Plone core release (as it's the case during these days) it needs to pull in the packages that provide these features (in this case latest p.restapi). This causes the build to pip install again, and this takes some extra time. This will be "solved" when 6.0.12 will be out.
Volto core HAS to work like this, this is unavoidable. Using docker is just faster and much more convenient, and less error prone. From my point of view, you don't need anything else than the latest Plone to develop Volto features.
In the case you are developing something in the backend as well that pairs with the PR that you are working on, no matter what package it is, you should use buildout.coredev. There's no reason for not doing it while developing the Plone backend. It's not only the recommended way to develop backend Plone, but the way to go. In the past, you could go away with the api folder, yes, but it was wrong and that build was terribly outdated, and in the upcoming months was going to get obsolete.

@wesleybl I created a PR that will close this issue. See https://github.com/plone/volto/pull/6184. I would appreciate your review. Thank you!

@sneridagh
https://github.com/plone/plone-backend/pull/147
It will make the backend start faster, correct?


Logging out and logging in with another accounts keeps you on the channel you were before

Your Rocket.Chat.Android version: 1.0-beta5
Steps to reproduce:

Log in with user A
Go to a private channel
Log out
Log in with user B
You'll still see the private channel messages, even if you don't have access to it

This is related to #95
May we continue the discussion there? :)

sure..


Not working on X86

Not working on x86 based processor after last update.

Duplicate
https://github.com/pylerSM/YouTubeBackgroundPlayback/issues/107
It is indeed the same issue - x86s have received "new" version so more work will be needed.

@Abbiiddd is it fixed now?

A new version 11.05.55 has just been released fon Play for x86. I havent tested it to see if it is working with 1.8.1 or not

My appology the version,is 11.01.55


How on earth do I install this?

you may either download a binary from https://github.com/The-Eye-Team/reddit-dl/releases/latest
or if your platform is not listed, install Go (https://golang.org/) and run go get github.com/The-Eye-Team/reddit-dl. This will place the program into $GOPATH/bin/reddit-dl.


Fix Precipitation Accumulation Units

When precipAccumulation was changed from cm to mm in #122 the conversion was never changed to convert from mm to inches. This PR changes the conversion and fixes #166
~Is precipAccumulation on hourly in mm or cm? If it's cm then we need to convert it to mm otherwise the conversion on hourly will be incorrect.~ Edit: Looked at the code and it's using a different variable.

I realized last night while looking at the precipitation amounts that it was converting cm to mm by multiplying by 100 instead of 10. (I saw one day report 880mm of precipitation which is obviously incorrect).
I was doing a quick test and noticed that none of the conversions were rounded and it leads to a crazy amount of decimal places. Maybe we can do the same thing as the percentages where if rounding is enabled it rounds the values to the nearest decimal otherwise 2/4 decimal places?


feat: update chain doc logo

Update the new chain doc logo to eliminate the confusion between it and the chain homepage, explorer.

/staging

/staging

@tomtau please review and merge. Thanks


2d gradients have a second dimension of size None

When calling compute_gradients with 2d variables, the first dimension has the correct size while the second dimension has a size of None.  Is this intentional?  It makes it somewhat difficult when processing the gradients, as a dimension size of None isn't really compatible with anything else.
Tested with:
2d variables - second dimension is size None, first dimension is size of first dimension of variable
1d variables - dimension is size of dimension of variable
Tested using compute_gradients in AdamOptimizer

As I answered on StackOverflow, this is intended behavior when a dimension cannot be statically inferred. In fact, a dimension size of None is compatible with every concrete dimension size (it acts as a wildcard), and any constraints will be checked at runtime.


host: Force memory overcommitment

This patch sets /proc/sys/vm/overcommit_memory to 1, which enables virtual memory overcommit and disables heuristic based denial of memory allocations (using these heuristics is the default setting of 0).
Due to the large virtual memory allocations that Go uses, and the fact that flynn-host frequently forks, the default heuristics can cause errors like this:
fork/exec /sbin/zfs: cannot allocate memory

Since we rely on the OOM killer anyway, there's no need to arbitrarily cause memory allocations to fail and we can turn off the heuristics entirely.

I had this exact issue. didn't see this issue and thought doing flynn-host volume gc might fix things.
now it appears gc removed some volumes that was needed for some jobs to run: https://gist.github.com/hadifarnoud/139d672c3435d7a20e37f6caba78ee25
is there any way to fix this?

the more I try to fix it, the more I screw it up. I did a restore from backup but as it turns out my backup was corrupt. found an old backup but it's from my old cluster (flynnhub). at first it restored but did not work.
now it just times out at check online-hosts


Tinkers' Construct Integration of Endergy alloys

Issue Description:
Endergy alloys are not fully integrated into Tinkers' Construct.
What happens:
Endergy alloys can be used to build Tool Forges and have fluids, although the latter don't have localized names. There are no tool parts for these Alloys and no modifiers assigned to them.
What you expected to happen:
Tool parts buildable with Endergy alloys and having modifiers assigned.

Affected Versions (Do not use "latest"):

EnderIO: 5.0.39
EnderCore: 1.12.2-0.5.44
Minecraft: 1.12.2
Forge: 1.12.2-<IP_ADDRESS>96
SpongeForge? No
Optifine? Yes
Single Player

It's planned, I wouldn't call it a bug, suggestions go in http://ideas.theideawall.com/EnderIO/ if you want to suggest traits.


Updated sh_07.sh (First Pull Request)

Fixed a typo error. Fixed the error from "arguements" to "arguments".

Fixed a typo error. Fixed the error from "arguements" to "arguments".

Thanks.  I'll also update the expected dump files.
--
There is a fine line between courage and foolishness.
Unfortunately, it's not a fence.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Feat at all

https://github.com/wechaty/wechaty/issues/2394
I can continue working on wechaty pr and test afther this one accepted.

@huan ping

@huan ping


Seat.set_keyboard should accept None

Acc. to https://wayland.emersion.fr/wlroots/wlr/types/wlr_seat.h.html#func-wlr_seat_set_keyboard NULL values are accepted, at least they are not forbidden.
Proposal: Change https://github.com/flacjacket/pywlroots/blob/main/wlroots/wlr_types/seat.py#L232
    def set_keyboard(self, keyboard: Keyboard) -> None:

to:
    def set_keyboard(self, keyboard: Keyboard | None) -> None:

Great point!


Revert "Fix version command for certain git workflows"

Reverts flutter/flutter#52062
This regressed the performance of all flutter commands, including https://github.com/flutter/flutter/issues/52121 and flutter_test_performance

I don't think we should take a performance hit for everyone for a non-standard workflow

Yeah, it might be enough to just run this for flutter doctor rather than every call to --version, or maybe there's some way we can check some cache information more rapidly before actually fetching the upstream.

The tests passed in https://cirrus-ci.com/build/5163935233998848, but the checks didn't update. Landing


LV Crash when setting PNG compression to 0

Very nice work, this library makes multiple hacked libs of myself obsolete.
During testing, I stumbled upon the following unexpected behavior:
When settings the compression for PNG to 0, LV crashes hard. If 0 is an unexpected value, maybe a in range and coerce is missing.

Hi @kleinsimon, thanks for the comment, and raising the issue.
When saving png, compression level 0 is one of the supported levels (0-10 inclusive). I've just tried it with the following snippet and didn't see a crash (using LV2020 32-bit on Windows 10).

What version of LV, bitness, and OS are you seeing the crash?
Do all images you've tried crash when saving with compression level 0?
What's the image's dimension and bit depth?
Do other compression levels work for the problematic images?

If there's any code or images you can share that'd be helpful.
Edit: After writing all that, I tested saving the png with level 0 to memory rather than to disk, and am seeing error 1097 exception occurred within the external code. Are you saving the png to memory?

Ah yes, I forgot to mention that. I wrote to memory.
Setup: LV 2020 R2, 64 bit, Windows 10
I can reproduce this on another PC using you snippet. It seems independent on the actual image data.

Found and fixed the issue, thanks again for the bug report. I'll aim to package up a new VIPM build over the weekend.


Memoization

Memoize past states inside a closure to optimise undo performance with very large number of actions.
see

Hey, there is not much of a difference between them, the API is slightly different but you can switch between them without much effort. At some point they might be merged to have the exact same API.
The reason for separate projects/packages was more of a naming thing since users of angular2/ngrx might not search for redux related projects, but rather ngrx-... and vice versa
That said there is also a different implementation approach:
With undox your state looks like this:
{
  history : [ { type: 'undox/INIT' }, { type: 'INCREMENT' } ],
  index   : 1
  present : 1
}

with ngrx-undoable
{
  past    : [ { type: 'ngrx-undoable/INIT' }, { type: 'INCREMENT' } ]
  present : 1
  future  : [ ]
}

so instead of future and past in undox there is just a history and an index which points at the latest action.
However this is an implementation detail and it doesn't matter if you are using the provided selectors by both packages.
ngrx-undoable might be more approachable without using selectors.
While undox is more elegant since undo/redo an action is just changing the index.


Problem after reboot

Hello.
After restarting the server or grafana-server service, it displays error messages attached. I lose all access the web console. To get the access I do snapshot.

Hello! Seems like Grafana bug. May be better to open issue in Grafana repo?


Node.js binding

Your lib look awesome, waiting for a binding!

Moving this lib to go-streamkit ... might not be a nodejs binding for it soon. However, this project is now used for a more lightweight version that does have a nodejs implementation. Should be in this repo shortly...


DOC Add developer documentation to the website

@Charlie-XIAO I have added documentation on how to develop both the codebase and the doc locally as a contributor to Deskulpt. Could you please take a look?


Properties Page

Refer #263
This takes care of Build, BuildEvent and ReferencesPath Properties Page in CSharp Console application and References Properties page in VB console Application.
References Properties page doesn't mark the current imports yet I have commented the call till I  find a way to get the imports from Roslyn service.

Tagging @dotnet/project-system

Sorry to spam with a number of similar comments - I just noticed vslangproj performing extra validation for something I am working on and figured i'll save you the effort of listing all the ones that need work.

Thanks for the validation link @srivatsn , did not knew about them.
if we don't specify the enum options in the generalbrowseobject.xml , the properties page seems to still work out of some hardcoded value. which is not effected by the options provided in the xaml file.
On the other hand the browser object property seems to be loading the values from the .xaml but my understanding is we will hide that the options. hence for now not adding the enum options. Let me know if you think otherwise.
I have filed a separate workitem to add validations.

👍


Delete old/expired jobs from jobstore

When jobs enter a terminal state, they should have a limited lifetime after which they are removed from the jobstore.
This PR implements the necessary logic to clean up those expired jobs when the jobstore is created, as the first step in providing hooks to a general housekeeping task that will be necessary in the requester node (for other cleanup tasks, like managing jobs in invalid states etc).  This PR does not add the housekeeping.
Blocked by

[ ] Need to configure the lifetimes per job type. Will add a config struct for jobstore until new configv2 is ready

they should have a limited lifetime after which they are removed from the jobstore.

Why? Are we worried about disk space? If I was a node operator I would expect a list of all executed jobs by default to be retained indefinitely.

Why? Are we worried about disk space?

Disk space, as well as performance, iterating through jobs is something we do reasonably often. We could move jobs (and related objects) into a 'dead-job' bucket and maintain another index, although we'd still need to compact (vacuum in postgres-land) the database from time to time.

I would expect a list of all executed jobs by default to be retained indefinitely, for debugging and auditing purposes

I'd certainly expect them to be available, but I'm not sure I'd have any expectations on where the audit was held.  I'd hope it to be somewhere immutable with limited accessibility. I think this is probably why I thought an archive/delete approach might be better in that  we can wrap all of the job objects into a single document and push it somewhere so that it's retrieval wasn't dependent on the current structure of the job (and related object). This was mentioned briefly in the original ticket -> https://github.com/bacalhau-project/bacalhau/issues/2657

Disk space, as well as performance, iterating through jobs is something we do reasonably often.

It sounds to me like the access pattern is at fault here rather than the data size – it's not exactly big data. Shouldn't we be doing more intelligent querying that simply iterating all jobs? Are we using a proper query engine?

I'd hope it to be somewhere immutable with limited accessibility.

Another user-visible issue with this is that after a certain period of time (presumably quite invisible from the user) their job will no longer be available using bacalhau describe and retrieving their results will not be possible using bacalhau get.
I don't disagree with the principle of things being performant of course, but I think we should at least know what the performance impact is and explore the options for sorting that out before we choose to degrade the UX.

It sounds to me like the access pattern is at fault here rather than the data size – it's not exactly big data. Shouldn't we be doing more intelligent querying that simply iterating all jobs? Are we using a proper query engine?

We're using a kvstore, and using it inline with how you'd normally use those, alongside our own indexes for various lookups.  And yet, for cases where we might want to reconstitute the eval broker after a case, there's little alternative than iterating all of the jobs whose state is currently in-progress.
What do you mean by a proper query engine?  Like a relational one?

Another user-visible issue with this is that after a certain period of time (presumably quite invisible from the user) their job will no longer be available using bacalhau describe and retrieving their results will not be possible using bacalhau get.

Yes, that's true, and will likely cause confusion if we do nothing about it. That's definitely a place where usage data would be really useful, do users use the CLI to access outputs months after their job has completed?

I don't disagree with the principle of things being performant of course, but I think we should at least know what the performance impact is and explore the options for sorting that out before we choose to degrade the UX.

Maybe best waiting then until we have some usage data so we can work out if it is likely to become an issue or not.  Happy to pause this for now in that case.

There's been no further discussion about cleaning up old jobs, so seems safe to close this.


Fix modal padding

Questions
Answers

Description?
Padding on modal-body is missing a unit so it doesn't work and causes misalignment with modal's header and footer

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
~

Sponsor company
~

How to test?
Try adding product to the wishlist as a guest to trigger a modal.

Hello @SharakPL
Thanks for your PR. Could you please add an issue to can to see exactly the bug?

I forgot this module has to be precompiled. Trouble is I can't make the build script work. Config for this module seems broken.

New release requires different change

@SharakPL I think you can have the build script work after applying my PR changes.


How to generate refund hash for rebate?

You can set it, get it.. but how to generate?

Hi Craig,
The refund hash is created by hashing the Refund or Rebate Password (depending on the request you wish to process). So for example in PHP:
$refundHash = sha1("Refund Password");
You then add that variable to your request object.
Best,
Seán

Thanks


Support AWS Cloud Map

This new PR enables the CN-WAN Operator to work with AWS Cloud Map.

[x] Modify settings to include Cloud Map settings
[x] Implement namespace operations
[x] Implement service operations
[x] Implement endpoint operations
[x] Unit tests
[x] Update scripts to deploy/remove the operator
[x] Update documentation

Done, thank you!


az: error: unrecognized arguments: --advanced-filter error.

az feedback auto-generates most of the information requested below, as of CLI version 2.0.64

Describe the bug
az: error: unrecognized arguments: --advanced-filter data.routeTo StringIn finsei all
i am on MacOS Sierra version 10.12.6 using az cli latest version  2.0.64. trying to create event grid subscription using cli with below command.
az eventgrid event-subscription create --name $subName --endpoint-type eventhub -g $resourceGroupName --endpoint /subscriptions/$subscriptionId/resourceGroups/$subResourceGroupName/providers/Microsoft.EventHub/namespaces/$eventhubNameSpaceName/eventhubs/$eventhubName --topic-name  $eventGridName --advanced-filter data.routeTo StringIn finsei all
it throws az: error: unrecognized arguments: --advanced-filter data.routeTo StringIn finsei all error.
As per documentation it's valid argument https://docs.microsoft.com/en-us/cli/azure/ext/eventgrid/eventgrid/event-subscription?view=azure-cli-latest#ext-eventgrid-az-eventgrid-event-subscription-create
To Reproduce
Steps to reproduce the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Additional context
Add any other context about the problem here.

@amitamit281
Thanks for reporting this issue.  It is not clear if you are trying to use the CLI module or the extension and which version you are trying. The Advanced Filter option is currently available in event grid extension (and hence, the reference you mentioned is for the extension itself).
I just attempted the same command in shell.portal.com and it is working just fine for me using eventgrid extension version "0.4.2"
Check the extension version using the following command and try to update the extension to newer one if you are using an older version
:~$ az extension list
[
{
"extensionType": "whl",
"name": "eventgrid",
"version": "0.4.2"
}
]
:~$ az eventgrid event-subscription create --name  --source-resource-id "/subscriptions//resourceGroups//providers/Microsoft.EventGrid/domains/" --advanced-filter data.awe StringIn data.routeTo StringIn finsei all error. --endpoint ""

Information provided for customer. this is by design. Please feel free to contact us or reopen the issue if needed.


Fix visual overflow when overscrolling RenderShrinkWrappingViewport

Fixes https://github.com/flutter/flutter/issues/89717 for real this time. :)
https://github.com/flutter/flutter/issues/89717 is a regression that was caused by #87143 while resolving #10949
#90419 originally sought to resolve it, but the test cases in that change did not account for every case of visual overflow.
This updates hasVisualOverflow so that the clip is properly applied when overscrolling a shrinkwrapping viewport. Seen here in the golden file test I added.
Overflowing

Not Overflowing 🎉

There are a lot of tests for each of these changes, and so while I was here I cleaned then up a bit to be more organized.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Gold has detected about 4 new digest(s) on patchset 1.
View them at https://flutter-gold.skia.org/cl/github/91620


add createNode

#632

@uxitten Could you please alias createNode to h only in the test files? Also please rebase with master. 🙏

i fixed again document and run npm format command
Ok?

Need rebase?

https://github.com/hyperapp/hyperapp/pull/633#issuecomment-370019159

@uxitten Could you alias createNode to h only in the test files?

ok

what should I do now?

@uxitten Maybe rebase again, sorry! I changed the docs a bit today. 😓

what is problem in this PR?

@uxitten There is no problem. I just still haven't made up my mind about changing h→createNode. We need to collect more feedback on #632.

ok very good

Since #632 was closed should this be closed also?

Sorry, @uxitten, but this didn't make it to the cut in the end! 👋


Context sensitive dictation

This implements context sensitive dictation, which determines how to space and capitalize dictated text by examining the text around the cursor before inserting.
I suggest reading this diff in the following order:

modes/dictation_mode.talon
code/vocabulary.py
code/dictation.py
settings.talon

The word, text, and prose captures got moved from vocabulary.py to dictation.py. This is because text & prose both need format_text and prose needs auto_capitalize and dictation needs auto_capitalize. We could split this by making something (eg. auto_capitalize) a talon action, but it seems like all this text-formatting code wants to be in the same place anyways. Open to other ways to organize this.
The capitalization code all became character based instead of word based, to avoid mangling whitespace. This makes it slightly more complex.
The PR leaves context sensitive dictation off by default, since, though useful, it is flaky and invasive; in some apps the copy-to-clipboard dance is unreliable or it doesn't work. I think it merits more testing before/if we decide to turn it on by default. Since it uses a setting, we can enable it on a per-application basis too.
It is possible to have a command that does context sensitive dictation in command mode, dictate <user.prose>$: user.dictation_insert(prose). Should we add a command for this? I've been using say for it, but that is a big change in behavior from current knausj say (eg. if context-sensitivity is off, it will often make wrong guesses about capitalization & spacing; if context-sensitivity is on it does the clipboard dance a lot).

Yeah, I also hesitate to globally enable this. Are there any well-behaved applications that should opt-in by default?
I personally use say as a poor-mans dictation mode, so I'm not opposed to changing its behavior. Presumably we would respect the setting too.

I think there are really two issues. One is application support and the other is clipboard flakiness.
For application support, most text editors and web browsers will work fine, but with exceptions. Some web pages override default text editing behavior in strange ways, e.g. notion.so makes ctrl-c copy the surrounding block if nothing is selected, which screws us over; some text editors do a similar thing. Emacs doesn't work out-of-the-box because cancelling selections is different in emacs. Etc.
For flakiness, I know the clipboard is a bit flaky on Linux right now but I don't know what the state is on Mac/Windows, hopefully solid?
What I'd like to do is merge it disabled-by-default (which is the current default in the PR) and ask folks to test enabling it. Then we can selectively enable it by default in apps/contexts it works well in based on feedback. Once the draft window/text area stuff becomes more solid we can enable it there by default as well, I think that will be really nice.

Makes sense - sounds reasonable.

Oops, I haven't really encountered any issues with the clipboard stuff on Windows/Mac.

I'm thinking to merge this alongside the draft ui -- either as a submodule or directly -- once 0.1.5 lands. The combo works very well.

I just had a video chat with someone on Mac where I had them try to use this in Gmail in a browser (not sure which one) on Mac, and it looked like they experienced some clipboard issue. Didn't have time to look into it but more reason to be cautious.

@dwiel @fidgetingbits - last call for concerns before I merge this on Sunday evening


Update README.md

Updated the README.md file to include comprehensive documentation for setting up the OpenDevin Frontend project. The documentation covers installation instructions, available npm scripts, setting up the terminal, and additional resources for learning React. This commit enhances project clarity and facilitates easier onboarding for new contributors.

Hi @AbhisekOmkar , I'm seeing conflicts with the README, could you please resolve them?

Im not able to understand what is the conflict is in README

Thanks for your contribution @AbhisekOmkar , the front-end toolchain has been migrated to vite. Most of the information in the README is now obsolete (for example npm run eject is no longer valid). You can refer to https://github.com/OpenDevin/OpenDevin/pull/24.

i need to update in my local machine and then i can create new PR for it


feature(kettle): 新增kettle任务类型

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

新增kettle任务类型


pip-compile unnecessarily chooses a package version which is uncompatible with the installed Python 2 version

summary: pip-compile unnecessarily chooses a Pygments package version which runs only on Python 3 when running in a system with Python 2.7
Environment Versions

OS: Ubuntu 16.04
Python version: 2.7.12
pip version:  9.0.3
pip-tools version: 4.5.1

Steps to replicate

Create a requirements.in file with the following content:

coverage==4.4
diff-cover==0.9.8
oauthlib==2.1.0
pylint-plugin-utils==0.3
requests-oauthlib==1.1.0
six==1.11.0
splinter==0.9.0
urllib3<1.24
Markdown==2.6.11

Run: pip-compile -v --upgrade -o requirements.txt requirements.in

Expected result
requirements.txt must be created without errors
Actual result
ROUND 1
(....)
Finding the best candidates:
  found candidate coverage==4.4 (constraint was ==4.4)
  found candidate diff-cover==0.9.8 (constraint was ==0.9.8)
  found candidate markdown==2.6.11 (constraint was ==2.6.11)
  found candidate oauthlib==2.1.0 (constraint was ==2.1.0)
  found candidate pylint-plugin-utils==0.3 (constraint was ==0.3)
  found candidate requests-oauthlib==1.1.0 (constraint was ==1.1.0)
  found candidate six==1.11.0 (constraint was ==1.11.0)
  found candidate splinter==0.9.0 (constraint was ==0.9.0)
  found candidate urllib3==1.23 (constraint was <1.24)
Finding secondary dependencies:
  diff-cover==0.9.8 not in cache, need to check index
  diff-cover==0.9.8         requires jinja2-pluralize, Jinja2>=2.7.1, pygments, six>=1.6.1
  coverage==4.4 not in cache, need to check index
  coverage==4.4             requires -
  six==1.11.0               requires -
New dependencies found in this round:
  adding [u'jinja2', '>=2.7.1', '[]']
  adding [u'jinja2-pluralize', '', '[]']
  adding [u'pygments', '', '[]']
  adding [u'six', '>=1.6.1', '[]']
Removed dependencies in this round:
------------------------------------------------------------
Result of round 1: not stable

ROUND 2
Current constraints:
  coverage==4.4
  diff-cover==0.9.8
  Jinja2>=2.7.1
  jinja2-pluralize
  Markdown==2.6.11
  oauthlib==2.1.0
  pygments
  pylint-plugin-utils==0.3
  requests-oauthlib==1.1.0
  six==1.11.0,>=1.6.1
  splinter==0.9.0
  urllib3<1.24

Finding the best candidates:
  found candidate coverage==4.4 (constraint was ==4.4)
  found candidate diff-cover==0.9.8 (constraint was ==0.9.8)
  found candidate jinja2==2.11.1 (constraint was >=2.7.1)
  found candidate jinja2-pluralize==0.3.0 (constraint was <any>)
  found candidate markdown==2.6.11 (constraint was ==2.6.11)
  found candidate oauthlib==2.1.0 (constraint was ==2.1.0)
  found candidate pygments==2.6.1 (constraint was <any>)
  found candidate pylint-plugin-utils==0.3 (constraint was ==0.3)
  found candidate requests-oauthlib==1.1.0 (constraint was ==1.1.0)
  found candidate six==1.11.0 (constraint was ==1.11.0,>=1.6.1)
  found candidate splinter==0.9.0 (constraint was ==0.9.0)
  found candidate urllib3==1.23 (constraint was <1.24)

Finding secondary dependencies:
  jinja2==2.11.1 not in cache, need to check index
  jinja2==2.11.1            requires MarkupSafe>=0.23
  jinja2-pluralize==0.3.0 not in cache, need to check index
  jinja2-pluralize==0.3.0   requires inflect>=0.2.4, jinja2>=2.4
  pygments==2.6.1 not in cache, need to check index
Traceback (most recent call last):
(...)
pip.exceptions.UnsupportedPythonVersion: Pygments requires Python '>=3.5' but the running Python is 2.7.12

If any Pygments version is allowed, why does pip-compile choose one which requires Python >= 3.5 instead of choosing one which runs in Python 2.7? I'm not familiar with pip-compile internals or pip internals, so this question possibly has an obvious answer.

Hello @alanoe,
Thanks for the issue! Could not reproduce it on docker:
$ docker run --rm -it python:2.7-slim /bin/bash

root@91263d5bc366:/# pip install pip==9.0.3 pip-tools

root@91263d5bc366:/# python --version
Python 2.7.17

root@91263d5bc366:/# pip --version
pip 9.0.3 from /usr/local/lib/python2.7/site-packages (python 2.7)

root@91263d5bc366:/# pip-compile --version
pip-compile, version 4.5.1

root@91263d5bc366:/# cat requirements.in
coverage==4.4
diff-cover==0.9.8
oauthlib==2.1.0
pylint-plugin-utils==0.3
requests-oauthlib==1.1.0
six==1.11.0
splinter==0.9.0
urllib3<1.24
Markdown==2.6.11

root@91263d5bc366:/# pip-compile
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile
#
astroid==1.6.6            # via pylint
backports.functools-lru-cache==1.6.1  # via astroid, isort, pylint
certifi==2019.11.28       # via requests
chardet==3.0.4            # via requests
configparser==4.0.2       # via importlib-metadata, pylint
contextlib2==0.6.0.post1  # via importlib-metadata, zipp
coverage==4.4             # via -r requirements.in
diff-cover==0.9.8         # via -r requirements.in
enum34==1.1.10            # via astroid
futures==3.3.0            # via isort
idna==2.9                 # via requests
importlib-metadata==1.6.0  # via inflect
inflect==3.0.2            # via jinja2-pluralize
isort==4.3.21             # via pylint
jinja2-pluralize==0.3.0   # via diff-cover
jinja2==2.11.1            # via diff-cover, jinja2-pluralize
lazy-object-proxy==1.4.3  # via astroid
markdown==2.6.11          # via -r requirements.in
markupsafe==1.1.1         # via jinja2
mccabe==0.6.1             # via pylint
oauthlib==2.1.0           # via -r requirements.in, requests-oauthlib
pathlib2==2.3.5           # via importlib-metadata
pygments==2.5.2           # via diff-cover
pylint-plugin-utils==0.3  # via -r requirements.in
pylint==1.9.5             # via pylint-plugin-utils
requests-oauthlib==1.1.0  # via -r requirements.in
requests==2.23.0          # via requests-oauthlib
scandir==1.10.0           # via pathlib2
selenium==3.141.0         # via splinter
singledispatch==<IP_ADDRESS>   # via astroid, pylint
six==1.11.0               # via -r requirements.in, astroid, diff-cover, pathlib2, pylint, singledispatch
splinter==0.9.0           # via -r requirements.in
urllib3==1.23             # via -r requirements.in, requests, selenium
wrapt==1.12.1             # via astroid
zipp==1.2.0               # via importlib-metadata

Could you attach the full output from pip-compile --rebuild --verbose?

This may happen if the problematic pygments pin is already in the txt before compilation -- for example if you run pip-compile in a py3 env before doing so in a py2 env. If it's a py2 project, you can delete the txt and only run pip-compile with py2.
@atugushev -- bug or no bug?

I see. It's not a bug then IMO, since there should be a separate requirememts-py{2,3}.txt for each major python version. See also this article.

I hope the above helped. It's been some time so I will close this issue now, but please don't hesitate to reach out if you have any other issues. Thanks for the issue!


OCR doesn't work well on two-column text

See https://ambuda.org/proofing/isavasyopanishad/1/
This:

Becomes:

It works well on some two-column text. Perhaps here the columns are too close for Google OCR to tease these apart.
If better detection is infeasible, one path forward might be to ask the user to select the region in which OCR should be applied.


[v2] Increased code coverage in icon.js file

Increased the test coverage of  icon.js file in v2
Before

After

@llaske, could you please review it

Nice. Thanks


[Bug/Glitch] Missing Mob Tips from 1.20-1.21

Legacy4J Version
1.21-1.8.3
Mod Loader Version
Fabric Loader 0.16.4
Bug/Glitch Occurrence Context
There are missing tips for the following mobs:
(1.20)
Armadillos
Camels
Sniffer
(1.20.5)
Armadillos
(1.21)
Bogged
Breeze
Crash Report
No response
What happened wrong?
Tips dont show up for these mobs when they are looked at

#593


Allow PDF to be optional in actions/release-github

📝 Description of the feature
In my project PDF doc builds fail, for reasons that no one has been able to understand. It has not been considered critical because the primary documentation is HTML and this is fine. However, the lack of PDF doc causes the release-github action to fail.
It would be useful to allow PDF to be optional, to reflect that it is a secondary requirement.
💡 Steps for implementing the feature
Add a new optional input, boolean valued with default true, that specifies whether PDF is expected to be included in the release.
Adjust the logic of the steps to account for the flag.
🔗 Useful links and references
No response

Hi @iboyd-ansys, thanks for opening this issue.
The philosophy for pyansys/actions is very strict about existant artifacts and their naming. When we implemented the release-github action, we imposed the following criteria:

Always deploy wheels and sdist artifacts
Always deploy documentation-html
Always deploy documentation-pdf
Always deploy wheelhouse
Allow to deploy any additional artifacts

If a Python library can not generate this set of artifacts, it means there is some issue on their side. This forces us to go and explore such issue.
Hence, I am closing this request.
However, let's work on solving this issue on your side.


CollisionShape3D

Description
This issue tracks the progress of implementing the functionality of Godot's CollisionShape3D node type. Below you'll find all user-facing properties, methods and signals, including ones inherited from relevant node types. If you notice any missing from this list, please leave a comment down below.
Status

Not supported
Not started
Partial (not functional)
Partial (functional)
Done (as much as it can be)
Done (completely) 👈

Links
Documentation: CollisionShape3D
Inherits: Node3D
Legend

🚧 indicates a partial, incomplete or broken implementation.
🛑 indicates that it's blocked or incompatible with Jolt in some way.

Properties
Uncategorized

[x] Shape (shape)
[x] Disabled (disabled)

Properties (Node3D)
Transform

[x] Position (position)
[x] Rotation (rotation)
[x] Scale (scale)

This tracking issue has served its purpose and will now be closed. 🫡
Any remaining work has had new issues created for it.


Add method to create new shard

This will require a new LogRange to be added to the global LogRanges struct and it will require some type of call to the trillian backend.
Will we need to update keys for the new shard, or should that be its own PR?

Seems like TreeState would need to be set to FROZEN in the actual database as well for the previous active tree.

Tree is created here: https://github.com/sigstore/rekor/blob/main/pkg/api/trillian_client.go#L310

Looks like a new TrillianClient will be required for a new logID.

TreeID is retrieved here.

It looks like the actual change would need to be made in the rekor-server during initialization as mentioned in #629 , but I'm still looking for feedback on that issue. Signs are pointing to decoupling setting of the treeID from bringing up the Rekor server, but I do not yet understand how the Rekor server's flags were/are meant to be used for this.

Currently we create a new shard (tree) by passing a tlog_id of 0 here. Do we need to add a different sharding method, or will this be enough?

Closing this as Rekor creates a new shard automatically unless one is specified by the tlog_id flag.


Minified UMD detection

Following up on https://twitter.com/lewisl9029/status/1674890868661325829, here's an early version of the PR for detecting minified UMD exports in esm-cjs-lexer. :)
Unfortunately, a lot of the code in here ended up looking rather nasty. I'm still pretty new to Rust, so would love to get some tips from someone more experienced on how the code could be improved! Happy to iterate on the code until you're happy with it.
I've added a number of minimal test cases representing some of the minified UMD patterns I've encountered in the wild (some from webpack, some from rollup). I'm sure there will be more, will tackle them as they come up, feel free to ping me in issues if you see any new cases that needs to be covered going forward!
I've been testing it out in prod by publishing it to this fork: https://www.npmjs.com/package/@lewisl9029/esm-cjs-lexer
And it seems to be working well so far, albeit maybe a bit slower (though I haven't done any benchmarks yet, please do let me know if you notice any performance footguns in here as well!). Feel free to take it for a spin when you have the chance and share your findings. Cheers!

@lewisl9029 your tests works great for me and i'm going to merge this. thanks for contributing!


Timeline concept

Playing with the idea of a timeline for features, discussed with @Elchi3.
A few work in progress screenshots below.
Grid

Font-synthesis

Async clipboard

We might need to leverage a graphics library to deal with overlapping labels, and to make things generally nicer.

This is great! 🎉

We might need to leverage a graphics library to deal with overlapping labels, and to make things generally nicer.

Or use SVG? I wrote MDN's inheritance diagram in SVG years ago: https://developer.mozilla.org/en-US/docs/Web/API/SVGTextElement (source)

Future idea: also use data from browser-specs to add timeline key events from specs. Ideally, if we were also tracking dates for explainers, and intent to prototype/ship emails, that would be great.

We might need to leverage a graphics library to deal with overlapping labels, and to make things generally nicer.

Or use SVG? I wrote MDN's inheritance diagram in SVG years ago: https://developer.mozilla.org/en-US/docs/Web/API/SVGTextElement (source)

Mermaid supports timeline diagrams, I wonder if that could help?


Add package property to decide whether a package requires API status check

Currently API status check is enabled only for new data plane packages. Java team needs to enable API check for spring package also. We should add a property in package properties object that's default to true for new data plane package and use this property to decide whether to enable API status check or not. This property can be overridden if any language requires specific conditions or  if extra package requires API status check.

@praveenkuttappan to check if this is already done.

Automatic API review support is already added for Java Spring SDK.


Fix URL constructor bug for Safari

📦 Pull Request
[Provide a general summary of the pull request here.]
🗜 Versioning
(Check one!)

[x] Patch: Bug Fix?
[ ] Minor: New Feature?
[ ] Major: Breaking Change?

✅ Fixed Issues

Fixes #76
Fixes #71

⚠ Update CHANGELOG.md

[x] I have updated the Upcoming Changes section of CHANGELOG.md with context related to this Pull Request.

:rocket: PR was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


[Feature Request] Modern Web App Search Devices by Contact

Is your feature request related to a problem? Please describe.
On the old web interface, I could use the device 'search' box to lookup any information on a device and see the information easily in the sidebar. This was also customisable depending on what fields the end user had chosen to view, sort or filter by.  Old version could search by most columns, and we use 'contact' to display the driver name alongside the vehicle registration, makes looking up drivers very quick and easy. Could also quickly see a list of vehicles at any Geofence - EG a home / depot geofence.
Describe the solution you'd like
I think it would be good to 'search devices' by contact, phone, or model etc.
Would also be helpful to display the contact’s name along with the tracking device name in the main window. (Poss. customise what fields show in the side panel)
Describe alternatives you've considered
A workaround in the modern web interface, would be to include the Name and contact in the same field, eg 'REG0 123 - ADAM', instead of just 'REG0 123' and then the contact name separately , only problem, is still cannot search by other fields, eg if i want to find an identifier to update, a number etc etc.  Also makes comparing data back in other systems difficult as the device name now does not match.
Additional context
Add any other context or screenshots about the feature request here.
Current modern web interface work around example:

Old web interface for reference example:

You can already search by the identifier.
Which specific fields are you really using for the search?

Mainly the "Contact" field and to display this in the same panel along with the name.


Bitbucket cloud issue

I'm trying to run DangerSwift on Bitbucket with Bitrise but DangerSwift command always fails with the following error:
ERROR: Failed to parse JSON: keyNotFound(CodingKeys(stringValue: "user", intValue: nil), Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "danger", intValue: nil), CodingKeys(stringValue: "bitbucket_cloud", intValue: nil), CodingKeys(stringValue: "commits", intValue: nil), _JSONKey(stringValue: "Index 0", intValue: 0), CodingKeys(stringValue: "author", intValue: nil)], debugDescription: "No value associated with key CodingKeys(stringValue: \"user\", intValue: nil) (\"user\").", underlyingError: nil)) ERROR: Dangerfile eval failed at Dangerfile.swift ERROR: Could not get the results JSON file at /var/folders/6q/wgy6jtp12w5gzgm9lzcglpqw0000gn/T/danger-response.json
However, It runs locally successfully.
Dangerfile.swift only contains the following code:
// swift-tools-version:5.3
import Danger
warn("A NEW PR NOW")

I expect that in BitBucketCloud.swift, Comment struct, user variable should be optional
extension BitBucketCloud {
    public struct Comment: Decodable, Equatable {
        public struct Inline: Decodable, Equatable {
            public let from: Int?
            public let to: Int? // swiftlint:disable:this identifier_name
            public let path: String?
        }

        private enum CodingKeys: String, CodingKey {
            case deleted
            case content
            case createdOn = "created_on"
            case user
            case updatedOn = "updated_on"
            case type
            case id // swiftlint:disable:this identifier_name
            case inline
        }

        /// Content of the comment
        public let content: Content

        /// When the comment was created
        public let createdOn: Date

        /// Was the comment deleted?
        public let deleted: Bool

        public let id: Int // swiftlint:disable:this identifier_name

        public let inline: Inline?

        public let type: String

        /// When the comment was updated
        public let updatedOn: Date

        /// The user that created the comment
        public let user: User  ///  <------ HERE I THINK SHOULD BE OPTIONAL
    }
}

Thanks in advance

Note: DangerSwift is working properly with the same project on Github. But I cannot ship the repo to Github.

This makes sense to me, I suppose not always the commit email is associated to bitbucket user.

I created a PR, could you please review it?


Fix:Overlapping Components #211

The main problem was here on line 13 of d3_flow.jsx <main role="main" style={{ padding:1em, height: 100%}}>       <main role="main" style={{ padding:1em }}> Then I made the columns more responsive by having them stack on smaller screens.

This is great, thanks!


Remove six dependency (#2017)

remove six from python code

remove six from setup.py

remove six from tests

remove six from references

remove six from packaging

revert str to torch._six._string_classes

revert str to torch._six._string_classes

Merged in c9e838a


And the next SETI administered mod

Self administered netkan file as usual.
Thank you very much!
Yemo

Merging with thanks!


Add support for /filter_and_sort_candidates request

Resolves #393

@ptrv Any idea why the tests are failing for 24.3? It looks like the json encoding of the test data is running into problem. Does the encoder not accept a cons-pairs or something?

I have no idea why the tests fail on Emacs 24.3. I need to investigate.

Would you prefer that I merge this in now and create a new issue for the
failing test? It's a new feature that nobody depends on, so it won't matter
too much if it's potentially broken on 24.3.
On Fri, Jan 13, 2017 at 9:35 AM Peter Vasil<EMAIL_ADDRESS>wrote:

I have no idea why the tests fail on Emacs 24.3. I need to investigate.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/abingham/emacs-ycmd/pull/395#issuecomment-272389049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE1eDAi0npNkuvOAguI9r1g2sb8vL0wks5rRzdvgaJpZM4LiUb6
.

Let's see if the issue can be fixed first before merging.

the Emacs 24.3 version of json-encode fails to encode something like
'((a . (((a . b) (c . d))
        ((a . b) (c . d)))))

I disabled the test for Emacs versions < 24.4


Create mockups and update export sheet for the 'Current Wave' bar

Thanks to jhoijhoi for visual prototypes.

Moved to Backlog https://github.com/Praytic/youtd2/discussions/407

Closing this because won't be making current wave bar.


Bring document list styles closer to design of publishing component

A previous change to spacing of first item heading means that currently there is no padding between the bottom border of the previous item and the heading of the next.
Seperatly, the publishing component this is based on has a top border for each item.
This PR brings the spacing more in line with that component, and given that the border is now on the top of each item, adds a top border to the pagination component.
(It’d be nice to do a full review and add the document list component to the govuk-prototype-components library… and pagination too, if it doesn’t land in govuk-frontend soon).

Before
After

These styles will look odd when there's no heading before a list of posts as is currently the case on design history collection pages.
Something like this:

I could go either way. The general approach on GOV.UK seems to be a line before a heading. Wonder if there are any comparative examples we can look at?

The design history used to have a heading for document collections, like "Posts", maybe we just bring that back?