Revision # on page revisions

On revision page better to display the revision number ( 79) and not the ordinal number of the record ( 6) like an attachment screen. For example, i can see revision ( 6) but if i click Preview i can see revision ( 79). Better if i can show this number ( 79) instead ( 6).
A similar situation with all revision numbering.
Working with revisions is always a scrupulous process. The correct display of the revision number will always give a transparent understanding of the deletion or restoration process.

Thanks for the suggestion @dean-mrack,
The current revision number was actually a later addition to BookStack, the idea being to give a more friendly page-relative revision count number instead of database ID. The database ID remains in URLs for backwards compatibility & ensured stability thanks to the uniqueness.
We could expose the original revision ID in the list but I think this will likely just cause more confusion, when shown alongside the existing page-relative number. The described benefit "The correct display of the revision number will always give a transparent understanding of the deletion or restoration process." is not really a strong enough reason to overcome the added confusion in implementing this change, especially in what is a relatively low-traffic feature.

First, I really like the bookstack. Graceful, aesthetically pleasing, fast and MarkDown) Thanks a lot.
let me add. Wiki is good for that, that it allows you to make changes without regard to possible loss of content, which gives the author or authors confidence in the safety of his work. For me, this is an important argument in choosing a system. I am ready to put up with the fact that the page version numbering does not start with 1, but I would not like to be mistaken because I did not remember correctly which revision number on Page Revision and on Detals. For example on pix. Test page version 3 is actually 122 on detail page. Some pages may have hundreds of revisions.
The best way for me show the absolute revision number. Or show the sequence number of the record and the revision number, like in githab )


Add support for excluding globs and not only file names

Follows #82
Both strings and globs (inside the array!!) are working now

Awesome, thanks!!

published as 2.4.0


Deduplicating whitelabel release note

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


The first session file is saved to Home without asking the users.

Vapor seems to create a session file in the user's home directory.
A more appropriate approach would be asking the user where to save it.
(Think about firing up Word and Word creates a document in the home...)

fixed by pr #151


Should we have a unitals namespace?

Unitals tend to be example specific, so they are not easily pulled out.
Not sure if we should have "Unitals" namespaces in the same way as concepts and quantities, because there would most likely be very few of them, (at least, prior to parameterization as mentioned in #91).
It feels like this will become more clear after the cleanup of examples is finished.

This issue is not very clear to me.  It probably needs to be expanded with an example or two.  I'm sure that when you're staring at a particular Unitals.hs file, trying to pull stuff out, it is obvious that there is design to be done.  But what you wrote isn't enough.

Since the most recent redesign, this is no longer an issue. Unitals can (and should) be pulled out into the Quantities namespace, since they are just quantities (with units and symbols as opposed to "maybe" having them).


can you create a version using tailwindcss?

Hi is there a posibility that you would have a working example of tailwind ui with alpine js?

I could and I will once I find some time.

But looking at the Tailwund UI license, it does not permit "creating examples" (it seems): https://www.notion.so/Tailwind-UI-License-644418bb34ad4fa29aac9b82e956a867
But we should have a proper Tailwind + Alpine template repo.

would love to see a tailwind + alpine modules working together...
Hope you can keep me posted thanks


Fix docs generator search constants id

This PR fixes a regression introduced in https://github.com/crystal-lang/crystal/pull/10875.
Constants don't have a html_id property, instead they have an id one.
Currently all constant anchors in search, redirect to #undefined (e.g. ARGV => /toplevel.html#undefined).

Thanks @GeopJr for your first contribution 🎉


heap-buffer-overflow in jpgicc.c

In source code utils/jpgicc/jpgicc.c, function HandleEXIF.
As below code shows:
 for (ptr = cinfo ->marker_list; ptr; ptr = ptr ->next) {

        if ((ptr ->marker == JPEG_APP0+1) && ptr ->data_length > 6) {
            JOCTET FAR* data = ptr -> data;

            if (memcmp(data, "Exif\0\0", 6) == 0) {

                data += 6; // Skip EXIF marker

                // 8 byte TIFF header
                // first two determine byte order
                pos = 0;
                if (read16(data, pos, 0) == INTEL_BYTE_ORDER) {
                    swapBytes = 1;
                }
            ...

pos may be larger than data's size, which results a heap buffer overflow.
command line:  ./utils/jpgicc/.libs/jpgicc bug2  /dev/null
In AddressSanitizer:

The crafted file is in the zip:
bug2.zip

Fixed per 8564c9b4fbf07d23062bbbbd87d5eb10dd92073f Thanks for reporting


How can i get the message for topic "camera_info"?

Hi,recently，i am using the bluerov2.This document  is very helpful to me.I want to run the orbslam2 using the camera of the bluerov2. And,when i used the camera,i can not found the message for ROS topic "camera_info",so that the camera calibration work can not be done.Can you give me some advice to help me solve this problem? Thank you very much!

I added the function to publish camera_info, and you need to put the calibration config to ~/.ros/camera_info/camera.yaml. Let me know if this works.

@jinkunw it works well,Thank you!


Made main file more efficient, added some comments, neatened up code, changed some code to follow python conventions, and more!

Title says it all.
The script should run faster, and will no longer crash on some edge cases.

Never used python for an actual project before but I'll be sure to follow these conventions in the future


Add context.http.url for HTTP client spans

See https://github.com/elastic/kibana/issues/20145

Pull Request


🐛 Bug Report: oneOf operator inside template giving error "circular structure to JSON"

📜 Description
while filling one of the scaffolding template we are getting error "Converting circular structure to JSON --> starting at object with constructor 'Object' | property 'oneOfSchema' -> object with constructor 'Object' | property 'oneOf' -> object with constructor 'Array' --- index 1 closes the circle". This was not coming with backstage v1.19, it all started when we switched to v1.20.3
👍 Expected behavior
it should work in v1.20.3
👎 Actual Behavior with Screenshots

when you start filling a template which has oneOf operator it gives you error of circular dependency. looks like this has been fixed for other operators but not for onOf
👟 Reproduction steps

Create a dummy template
add oneOf operator inside it

title: Framework details
    required:
      - frameworkVersion
    properties:
      frameworkVersion:
        title: Framework Version
        type: string
        oneOf:
          - title: "7.0"
            const: net7.0
          - title: "6.0"
            const: net6.0
          - title: "5.0"
            const: net5.0
        default: net7.0
        description: Select .Net framework version

        ui:options:
          rows: 5

fill this template, after selecting the framework version from dropdown, once next is clicked you will get circular dependency error

📃 Provide the context for the Bug.
we are not able to release our new templates
🖥 Your Environment
its not env specific, its coming in local as well
OS -Mac, 13
browser - chrome
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Hello
When I reproduced this issue I could see it has a circular json structure during the JSON parse. https://github.com/backstage/backstage/blob/8b13af849a801129abd1d22406ff546fe9849fce/plugins/scaffolder-react/src/next/lib/schema.ts#L126

Also figured out we can remove the circular json as mentioned here https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Errors/Cyclic_object_value
I will implement it on the extractSchemaFromStep function to support all schema operators, will test and raise a pull request.

thanks @deepan10 , till the time you are fixing it do you have any workaround for the same

@Nitinvonage ,
We can use the anyOf operator for this to fulfill the same purpose or enum as mentioned in backstage docs  https://backstage.io/docs/features/software-templates/input-examples#array-with-custom-titles

Closing as a duplicate of #20256


[spark] Support requirements.txt in model tar file

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report
Patch coverage: 72.64% and project coverage change: +1.62 :tada:

Comparison is base (bb5073f) 72.08% compared to head (054b2cd) 73.71%.

:exclamation: Current head 054b2cd differs from pull request most recent head 0432fc5. Consider uploading reports for the commit 0432fc5 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2528      +/-   ##
============================================
+ Coverage     72.08%   73.71%   +1.62%
- Complexity     5126     6937    +1811
============================================
  Files           473      684     +211
  Lines         21970    30280    +8310
  Branches       2351     3138     +787
============================================
+ Hits          15838    22320    +6482
- Misses         4925     6434    +1509
- Partials       1207     1526     +319

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
67.10% <0.00%> (+7.89%)
:arrow_up:

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

api/src/main/java/ai/djl/modality/nlp/Encoder.java
66.66% <ø> (ø)

.../main/java/ai/djl/modality/nlp/EncoderDecoder.java
64.00% <ø> (ø)

... and 228 more

... and 352 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Misleading examples

The examples given here are wrong; it took me a lot of time and efforts to realize it. In both examples, "C:\Windows\winsxs" is given as the source. But unfortunately, this implies that one can extract capabilities from one's own running copy of Windows into an offline image. What adds credit to this belief is the fact that in Windows Vista and later, Windows aggressively claims C: for the system partition.

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

ID: 70a11a3f-a432-fd7e-d05e-8b80e891b3af
Version Independent ID: 256de206-99fd-d0d4-4f1a-3321432d36f8
Content: Add-WindowsCapability (dism)
Content Source: docset/windows/dism/add-windowscapability.md
Product: w10
GitHub Login: @brianlic-msft
Microsoft Alias: brianlic

@officedocsbot assign @yogkumgit

Hi @skycommand , thank you for your feedback.
I think we should remove "-Source "C:\Windows\winsxs"" from these examples, this paremeters is not requiered.
Are you agree? Thanks.

@dariomws306 Hi. Theoretically, replacing "C:\Windows\winsxs" with "\REMOTE-PC\Windows-share\winsxs" should do the trick.
In practice, however, I cannot answer your question with certainty. If I could, I'd have filed a pull request instead. You see, even before 1809, when Microsoft still published FODv2 discs, I could not use this command with a "-Source" parameter. In a last-ditch effort, I replaced it with "Add-Package", which worked!
I was hoping that by filing this feedback item, someone who knows about the problem or one of Microsoft developers weigh in.

I'm afraid I cannot say I agree. I did a few tests.
This command works:
Add-WindowsCapability -Online -Name "Msix.PackagingTool.Driver~~~~<IP_ADDRESS>"

But this do not:
Add-WindowsCapability -Path "E:\Queue\mp\" -Name "Msix.PackagingTool.Driver~~~~<IP_ADDRESS>"

So, no. Removing "-Source" does not solve the problem.
Also, I tried adding -Source switch. It looks like in all cases, DISM was looking for Msix-PackagingTool-Driver-Package~31bf3856ad364e35~amd64~~.cab at the path I specified as the source.

Second example works for me:
Add-WindowsCapability -Name "Language.TextToSpeech~~~fr-FR~<IP_ADDRESS>" -Online -LimitAccess -Source "C:\Windows\winsxs"
I cannot test the first example because I don't have an OS image in my disk to add this capability. But It will use the same files in "C:\Windows\winsxs", so it should work.
Have you test in another OS/Machine?

Hi. Thanks for writing back.

Have you test in another OS/Machine?

Dozens. My job requires that I work on many different machines. And anyway, I cannot reproduce it on a pristine VM.

Second example works for me: Add-WindowsCapability -Name "Language.TextToSpeech~~~fr-FR~<IP_ADDRESS>" -Online -LimitAccess -Source "C:\Windows\winsxs"

May I please know what does "works for me" exactly means here? Do you mean that when you run it, it completes without generating an error message? That, I am afraid, is not enough; this command tells the servicing stack to add the French language from your running OS to your running OS. In other words, nothing! I am sure you agree that a command that does nothing meaningful isn't a good example of using PowerShell to help admin accomplish their business-critical goals.
Perhaps it is better that I contact Microsoft Premium Support about this. Nevertheless, the writers of Doc pages are responsible for writing examples that are easily reproducible and useful at the same time. Don't you agree?

Hi @skycommand.
Yes I think that if you are trying to use in complex scenarios the cmdlet and it doesn't work the best place to go is support.
Here we are available to help people to fix docs mistakes and improve them. And you are in the right direction about that examples are usually simple examples with easy and useful purpose.
Thank you again for your feedback.

@dariomws306 Good news. Microsoft Support agent referred me to the old documentation pages for the old dism.exe. They are much clearer. I'll probably submit a PR to fix this documentation page based on that. I think it is safe that I close this topic and take it off your plate.


navigator.registerProtocolHandler()

navigator.registerProtocolHandler()
http://www.w3.org/TR/2011/WD-html5-20110525/timers.html#custom-handlers

This issue was imported from Google Moderator
Moderator votes: +3
Issue added by Manuel Bieh on 2014-03-17
To vote this issue up or down, simply include +1 or -1 in your comment.

Already added at http://caniuse.com/#feat=registerprotocolhandler


defineVitestConfig from @nuxt/test-utils is not a drop-in replacement for defineConfig from vite (No test suite found in file)

Environment

Operating System: Darwin
Node Version:     v18.20.1
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      srcDir, buildDir, modules, ssr, nitro, serverHandlers, css, components, imports, build, builder, typescript, vue, hooks
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
Unfortunately I could not reproduce this behaviour in a fresh repo.
Describe the bug
Using defineVitestConfig from @nuxt/test-utils gives me some errors on random(?) test files located in a specific folder (./src/plugins/ in this case). It seems that vitest doesn't find those tests inside this directory (No test suite found in file /omniweb/gsd/src/plugins/frontend-commons.spec.ts). I tried to place a simple test.spec.ts inside with a basic test but it doesn't pick that up either.
However if I move those test files to another directory at the same level (plugins2) it runs those test. If I rename the folder to plugins it again does not run those test files...
I really can't imagine what's going on here. Are there some forbidden path patterns for some reason?
Additional context
If I use defineConfig from vite these tests will run. But I need to use the Nuxt import since it helps with those nuxt import aliases like #app etc.
Logs
No response

I am also getting this issue, but it's related to latest vitest versions, it was not present before


This fixes #78170

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

In the editor find, it will always move to the first match. If you select the second match and toggle any of the buttons, it will move to the first one.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ATheCoder sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I believe right now it should work the same way as the editor find works.


IPFS

Description
Project proposal
Why
Used by many projects, but they don't have token economics.
Khaos, the subproject of the Dev Protocol, also makes heavy use of IPFS.

Code of Conduct
By submitting this pull request, I confirm I've read and complied with the CoC 🖖

// Voting action starts a new vote every time... a bit confusing...

// Voting action starts a new vote every time... a bit confusing...

// I've removed the extra configurations, so voting comments are consolidated into one.

// I've removed the extra configurations, so voting comments are consolidated into one.


[Candidate 4v4 Dominion ZH v1] One Supply Dock can be moved behind

Observed behaviour
One Supply Dock can be moved behind. Not the others.
Expected behavior
Mirrored Supply Docks offer equal space for unit mobility.
Screenshots

fixed


Symbol.toString shall return the name

This issue was originally filed by @tomyeh

Otherwise, it is hard to debug.

I don't think there's any intention of making this change, at least at this point in time. Closing for now.


Pls support threshold alarm

📑 I have found these related issues/pull requests
It is an awesome tool! Could you please add threshold range alarm feature in case to notify some jitters(for ping type).
Thanks~
Btw, page will auto scroll to top when switch period options, I think it's better not move.

The scroll position issue for the PingChart period dropdown should be fixed in #4264.

The scroll position issue for the PingChart period dropdown should be fixed in #4264.

it's weird, I'm sure that I've deploied the lastest version..

https://github.com/louislam/uptime-kuma/pull/4264 is neither merged nor in the 1.x series.

This issue is really unclear what it wants.
How should this work exactly (what would you like to monitor?) what do you mean with threshold for jitter?
@hadwinfu
please fill out the whole form instead of just a single part with unrelated content.
There is a reason why we ask the other parts..


Avoid spawning drive()

Hi,
Is there a way to avoid spawning the drive function each time an LDAP query needs to be done? I tried manually calling turn(LoopMode::SingleOp) after sending a request but the request gets stuck.
I understand the reasons you explained in https://github.com/inejge/ldap3/issues/53 but, since this is a client library, I believe that the async version should work just like the sync one and wait until the result is available. Spawning a future would make sense if there are events being pushed from the server which I don't believe is the case in LDAP.
Thanks!

Is there a way to avoid spawning the drive function each time an LDAP query needs to be done?

Spawning the drive! task is done once per connection, not once per query. It's done in order to implement the multiplexed nature of an LDAP connection.

I believe that the async version should work just like the sync one and wait until the result is available. Spawning a future would make sense if there are events being pushed from the server which I don't believe is the case in LDAP.

If you look at sync code, you'll see that it uses the async elements internally, including the spawning of the drive! task once the connection is established.
As for server-initiated messages, they exist: RFC 4511, Section 4.4, describes unsolicited notifications which can be sent independently of any client-initiated exchange.

Thank you for the reply.
I totally understand and I agree that spawning drive is the right design choice in order to handle multiplexed responses.  However, I believe that spawning a task, even if it is per connection, may introduce an overhead for simpler tasks such as conducting a single query.
Given the significant variation in the scale and complexity of tasks for which this library might be used, I would like to propose an enhancement to make spawning the drive task optional when multiplexed messages or server-initiated messages are expected. I believe it would be a valuable addition for developers who are using this library for applications where performance is critical.
I understand that this might involve a considerable amount of work, but I believe that this enhancement is worthwhile as it would greatly increase the library's flexibility and adaptability to different use case scenarios.
I'd be more than happy to hear your thoughts on this and would be willing to assist in any way that I can, if this is something you'd be interested in pursuing.
Thanks again for the library, and I'm looking forward to hearing your thoughts on this suggestion.

If we're discussing adaptability, let's first agree on some lower limits to complexity: this library is decidedly std-only, therefore a process will have at least a couple of MB for code and heap space. With that in mind:

I would like to propose an enhancement to make spawning the drive task optional when multiplexed messages or server-initiated messages are expected.

[Not expected, surely?] If you forgo multiplexing and keep Tokio as the async executor, the savings are meagre: a few KB because of simpler I/O code (the rest of decoding machinery must still exist), a few more KB because you don't need the connection-wide message-id to channel hashmap.
I suppose that using a custom bare-bones executor would produce a more significant simplification (but not if you want to keep TLS!) at the cost of rearchitecting the library to be more executor-agnostic. I'm not prepared to go there, not until it's clearer what kind of abstractions some future iteration of async Rust is going to provide.
With those constraints, I don't think that trying to simplify I/O interaction is worthwhile.

[Not expected, surely?] If you forgo multiplexing and keep Tokio as the async executor, the savings are meagre: a few KB because of simpler I/O code (the rest of decoding machinery must still exist), a few more KB because you don't need the connection-wide message-id to channel hashmap.

Even without benchmarking the code I am quite certain that writing and reading from a socket will always be faster than splitting the socket, spawning a task and exchanging messages through channels. I am not talking about memory usage, this is about performance.
If you look at other async libraries on crates.io it is extremely rare that a client library requires spawning a task for something trivial such as performing a query. For example, IMAP is also a multiplexed protocol with server initiated messages but async libraries are not forcing users to spawn a task when they to do something simple like reading a single email.

Even without benchmarking the code I am quite certain that writing and reading from a socket will always be faster than splitting the socket, spawning a task and exchanging messages through channels. I am not talking about memory usage, this is about performance.

I expect that network round-trip latency would dominate in the response time, especially for simple searches where the server has everything indexed and RAM-resident. Benchmarks would be the preferred way to establish a performance baseline. E.g., user/kernel CPU usage when a gigabit connection is saturated, or the request load is kept steady at 100k RPS, for the expected operation mix. Then you can substitute a simplified I/O driver and retest for the same target(s). Alternatively, performance targets stemming from design requirements could be used.
I don't think I would seriously consider reworking the I/O architecture without seeing convincing evidence of performance improvement. Even that mustn't come at the expense of seriously affecting backwards compatibility and API ergonomics.

I expect that network round-trip latency would dominate in the response time, especially for simple searches where the server has everything indexed and RAM-resident.

That is true, but what I mean is that all that extra work the library is doing to read a response is taking resources from other processes running in the program. For a small amount of requests this won't matter of course but if, like in my use case, you need to handle thousands of requests per second this adds up and ends up impacting performance. Recently I found a Rust library that was cloning String's all over the place rather than passing &str references, and of course the performance difference is negligible but in a high-throughput environment this does have an impact and degrades performance. If one keeps adding this small inefficiencies all over the place then everything slows down.

I don't think I would seriously consider reworking the I/O architecture without seeing convincing evidence of performance improvement. Even that mustn't come at the expense of seriously affecting backwards compatibility and API ergonomics.

No worries, I'll just fork the library and simplify the API for my use case.


[M] Team Learning for Rate Limiting and Generating API Keys

This ticket is to set aside self-directed time for the team to learn about rate limiting and generating API Keys.
(#122 and #200); reviewing AWS documentation - https://docs.aws.amazon.com/apigateway/latest/developerguide/api-gateway-api-key-source.html
Reach out to Mark if stuck on what to review.
Acceptance Criteria:

understanding of the documentation around API gateway and rate limiting;
comfortable implementing ticket #122 and #200
Outstanding questions are answered

Approach for tracking this ticket - will assign to everyone and then if you've completed the learning unassign yourself and once no one is assigned then we can move to done :)


creates "kf-infra-gitops-sa" IAMService Account

Relates to https://github.com/kubeflow/community-infra/pull/21
I am uncertain whether "roles/editor" is the correct permission for the sa. I am also not sure how Cloud Config Connector has been set up and whether it will automatically apply "iam-service-accounts.yaml" or not. I am assuming it simply applies all YAMLs within within "prod/namespaces/kf-infra-gitops/*" .
Relates to kubeflow/community#274

/assign @jlewi

Rebase seems to have gone wrong, two commits from previous PR show up here as well. Can fix it later on.

The service account doesn't need editor permissions. The service account will be used to administer the Kubeflow calendar. So at this time it shouldn't need any GCP permissions.
nit: I would recommend creating a service account just for administering the calendar and thus the name should be more like "kf-calendar-admin". This way we can grant privileges to that GSA that are scoped to just what it needs to administer the calendar.
The ACM configuration is here:
https://github.com/kubeflow/community-infra/blob/master/management/instance/configsync/config-management.yaml
To learn more about how ACM does GitOps you can read the docs.

/lgtm
/approve


Support for drop files?

I have a bunch of JS that handles drag/dropping files form the desktop into the browser, uploading them to a server, and displaying them in the browser.
I can't find any docs on how you would test this using Poltergeist. Would I use drag_drop functionality with some uploaded file element? Would I fake a JQuery.Event object and trigger the "drop" event manually with execute_script?
Is any of this supported and working in Poltergeist?

Closing as this is apparently working now.


feat(tutorial): complete step 5 carbon-tutorial-fergbrowne.eu-gb.mybluemix.net

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Deployed App:
carbon-tutorial-fergbrowne.eu-gb.mybluemix.net

Apologies, forgot to include it
carbon-tutorial-fergbrowne.eu-gb.mybluemix.net


[Enum Request] Table 7000003 - Field 24 "Status"

Hello,
we need to be able to extend the options of field 24 - "Status" in table 7000003 - "Posted Cartera Doc.".
Could you please change it into an enum?
(The same field is in table 7000004 - "Closed Cartera Doc.")
Thanks

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


Adding cypress + example test

Changes Proposed

Check List

[x] The title of my pull request is a short description of the changes
[ ] This PR relates to some issue:
[x] I have documented the changes made (if applicable)
[x] I have covered the changes with unit tests

Added cypress, and some first component test

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (88ea89a) 29.85% compared to head (1fa8425) 29.85%.
Report is 12 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #476   +/-   ##
=======================================
  Coverage   29.85%   29.85%
=======================================
  Files          10       10
  Lines        1323     1323
=======================================
  Hits          395      395
  Misses        887      887
  Partials       41       41

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add spanish translation

Fixes #216

Since we're not able to maintain translations going forward, we are not accepting translation PRs at this time.


Detect if js file when customizing

H! Thank you for this amazing tool :)
I was able to programatically create data using a js file, eg $ json-server index.js but when creating the server in custom scripts the js file is not evaluated. The workaround is simple enough by wrapping the file with require.
var router = jsonServer.router(require('db.js'));

One assumes this should be done automatically and probably in a better way like when using the CLI.

Thanks, exactly what I was looking for!

Thanks very very much!!!!


Publish the package to PyPI

Hi man,
thanks for your useful package. Could you submit django-acme-challenge to PyPi? This way we could automate our python requirements with "pip freeze".

Good call, @flavoi .
It's up on PyPI now and the ReadMe reflects the new install process.


Feature: Add default icon/marker image

Like GoogleMaps, a user should be able to set a custom marker image or icon, right now it uses a built in marker icon

This is fixed by PR #9 from @royyeah


Any plans integrating directly with dbt api

any plans integrating directly with dbt api ? using python instead sub process to integrate
something like
import dbt.main as dbt
args = ["run"]
args.extend(["--project-dir", 'project_dir'])
args.extend(["--profiles-dir", 'profiles_dir'])
args.extend(["--profile", 'profile_name'])
args.append('--log-cache-events')
res, success = dbt.handle_and_check(args)

related https://github.com/dbt-labs/dbt-core/issues/4593

TL;DR: dbt is a cli
That said, and based on the Docs for dbt v1.1

The primary interface into dbt-core is on the command line. It is designed to be invoked with commands, arguments, and flags. Starting in v1, this interface is contracted, with backwards compatibility guaranteed.
It is possible to import and invoke dbt as a Python module. This API is still not contracted or documented, and it is liable to change in future versions of dbt-core without warning. Please use caution when upgrading across versions of dbt if you choose to run dbt in this manner!

This means that they're starting to support a python API, probably towards having a better integration with python data frameworks like pandas, spark and such. However this is still not even close


Feat pb coverage

Experimenting with coverage reports here

Closing for now as this isn't moving to a current priority.


Upgrade Flyway to v9

DSpace is currently using Flyway 8.4.4, but the latest version is 9.22.2. Officially, Flyway 8 only supports PostgreSQL up to version 14, while Flyway 9 includes support for PostgreSQL 15.
Since DSpace lists 15 as supported PostgreSQL version it should IMHO also use the respectable Flyway version.
Flyway 8 seems to run without problems with PSQL 15 though. Flyway 9 is a fairly straightforward upgrade, the only relevant API change looks to be the removal of flywayConfiguration.ignoreIgnoredMigrations(true); in favor of flywayConfiguration.ignoreMigrationPatterns(String ...)
Perhaps it would be sensible to wait until Flyway releases a version which is compatible with PSQL 16 before the upgrade

I agree that, in general, we should be keeping Flyway up-to-date.  Moving this to the 8.0 board in search of a volunteer, though priority is still lower at this time.
This may be related to #9107 as that update to PostgreSQL 16 appears to be returning a warning from Flyway that it may be unsupported. See this comment: https://github.com/DSpace/DSpace/issues/9107#issuecomment-1756705855
I agree we may want to wait until Flyway has official support for PostgreSQL 16 and then update to that version.   Currently Flyway only reports support for PostgreSQL 15 here: https://documentation.red-gate.com/fd/postgresql-184127604.html

I just took another look at the Flyway API change. I *think flywayConfiguration.ignoreIgnoredMigrations(true); would need to be replaced with flywayConfiguration.ignoreMigrationPatterns("*:ignored"), but there isn't really much documentation for that. The deprecation message for ignoreIgnoredMigrations just says Use ignoreMigrationPatterns instead without detailing how, so the only documentation I can go off is https://documentation.red-gate.com/fd/ignore-migration-patterns-184127507.html

This will be fixed by #9321 which upgrades us to Flyway 10.  So, we'll skip over Flyway v9 and go straight to v10.

@tdonohue is this something that could be backported to DSpace 7? Since that lists Postgres 15 as a supported version, yet this is a warning when running on 7.6.2 (possibly earlier).
Thanks!

@misilot : The upgrade to Flyway v10 cannot be backported, as it was highly dependent on the migration to Jakarta EE which occurred in DSpace 8.0 (see #9321).  It's impossible to backport Jakarta EE support to 7.x as it's a significant refactor.
The warning you are seeing in 7.6.2 can be safely ignored (as it also occurred before 7.6.2 if you used Postgres 15), see this comment https://github.com/DSpace/DSpace/issues/9107#issuecomment-1757834891   As noted there, Flyway is only used when you run ./dspace database migrate.  We've never seen any reported issues with running that using Postgres 15 + DSpace 7.6.x
I have not investigated whether it's possible to upgrade 7.6.x to Flyway v9, as I believe Flyway (finally) added full Postgres 15 support to later versions of Flyway 9 (it did not exist in early versions of Flyway 9).  I'm not against that upgrade, but it'd require someone to look into the feasibility.  In my opinion though, it's low priority as we've found Flyway 8 works fine with Postgres 15 even with the warning.


Enable holoviews testing py3.10

Seems fixed on master

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 91.145%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6536

Relevant Lines:
7171

💛  - Coveralls


Add Linxdot RK3566 hardware defs

Issue

Link:
Summary:

How

Screenshots

References

Checklist

[ ] Tests added
[ ] Cleaned up commit history (rebase!)
[ ] Documentation added
[ ] Thought about variable and method names

Relates to https://github.com/NebraLtd/helium-linxdot-rk3566/issues/3

Relates to https://github.com/NebraLtd/hm-diag/pull/682


Draft a phenotypic data browsing, filtering, and selection GUI

Concept
A browsable, filterable, searchable GUI for selecting fields of interest from any data dictionary (or possibly all data dictionaries).
Inputs
The GUI would take as input this repository. It would read from all the available data dictionaries within <STUDY>/phenotype/ subfolders, as output by dictionary.py scripts.
Drafts
Please insert draft concepts/ideas into this Google Slides deck here:
https://docs.google.com/presentation/d/1OWgNupUz2FyKCEzoAjjDtgunaiWHmjT1ArG1asgndig/edit?usp=sharing
Implementation
I recommend using PySimpleGUI as I know it works well and is simple and stable and built in Python. Concepts can of course be drawn and pictured, or drafted, or even just a messy draft on a napkin.
Outputs
As output, you should receive a small data dictionary subset of the selected fields of interest. You should also be able to "auto-save" progress as you click so if the window closes you don't lose a bunch of reading and selection work.
The dream
In the far-flung future, this would also integrate with BIDS TSVs to provide a sneak preview to the user what the distribution of data points are. Like if it were a Yes/No question, then how many Yes and how many No are in the dataset?

I forgot to add a nice-looking example of something like this from New Relic: https://docs.newrelic.com/attribute-dictionary/

Here is a draft for the GUI

@RobertoFelipeSG's Drafted hackathon GUI added to the repo with commit 8a3fc4fb9b1e0ed151fc50fc8201cb768d0ed503.


Some suggested hardware requirements

Maybe you can get away with 16GB of RAM. So far I've tried 8GB and that's clearly not enough. 32GB is used in the demo video.

Thanks for PR. I have merged your commit at #180.


Restricting cache clients in liqo-route component

Description
Liqo-Route component is made up by three operators: route operator, overlay operator and symmetricRouting operator.
Overlay and symmetricRouting operator used to watch for changes for all pods in the cluster, and reconciling only the needed pods. Now the overlay operator watches for events only the "liqo-route" pods in the namespace where the operator is installed. In the other hand the symmetricRouting operator watches the pods that are running on nodes different from the node where the operator is running and ignores the pods schedule on a virtual node. For this reason all the pods offloaded to a remote cluster presents the label liqo.io/shadowPod=true.
Fixes #(issue)
Liqo-Route is now faster while configuring the routes configuration on the nodes.
How Has This Been Tested?
Existing unit and e2e tests covers the changes made in this PR.

/rebase

/rebase

/merge


Move model_state inside JIT in training loop

Is axon unusable atm? The credit_card_fraud example doesn't work because of this (at least using EXLA).
Is there a way to work around it while this issue persists?

Axon has not been fully updated to Nx+EXLA v0.2 yet, which is why you may run into bugs. If you rollback to previous EXLA versions on Git (the ones before EXLA.Backend were introduced), then it should work.

Guess i'll wait then. Might as well have my new code with EXLA 0.2 already. Thanks!

Now I'm getting this error in the evaluator Loop in recur_predict_fun. I cannot get any Loop evaluator to work with EXLA.
** (RuntimeError) cannot invoke Nx function because it relies on two incompatible tensor implementations:
Nx.Defn.Expr and EXLA.Backend. You may need to call Nx.backend_transfer/1 (or Nx.backend_copy/1)
on one or both of them to transfer them to a common implementation
    (nx 0.3.0-dev) lib/nx/shared.ex:447: Nx.Shared.pick_struct/2
    (nx 0.3.0-dev) lib/nx.ex:3238: Nx.element_wise_bin_op/4
    (axon 0.1.0-dev) lib/axon/compiler.ex:379: anonymous fn/15 in Axon.Compiler.recur_predict_fun/3
    (axon 0.1.0-dev) lib/axon/compiler.ex:192: Axon.Compiler.call_predict_cache/6
    (axon 0.1.0-dev) lib/axon/compiler.ex:336: anonymous fn/8 in Axon.Compiler.recur_predict_fun/3
    (elixir 1.13.4) lib/enum.ex:1715: Enum."-map_reduce/3-lists^mapfoldl/2-0-"/3
    (axon 0.1.0-dev) lib/axon/compiler.ex:334: anonymous fn/15 in Axon.Compiler.recur_predict_fun/3
    (axon 0.1.0-dev) lib/axon/compiler.ex:192: Axon.Compiler.call_predict_cache/6

@alisinabh can you provide a way to reproduce the issue? :)

Also please make sure to use latest Axon too!

@josevalim You can try examples/vision/cifar10.exs example. The the evaluation loop is not working because of this error.
Also the accuracy metric seems wrong to me but I'm not sure.
I'm also using the latest Axon from the main branch.

In the last commit (06a5526266dbfa7165168433e58633809fc04bf6) there is a backend_transfer call added to one of recur_predict_fun.
Maybe we need to add this call to all of them?


Updates the CDN paths in the storybook.

Fixes #1205
Checklist

[x] I have made corresponding changes to the documentation.
[ ] ~I have updated the types definition of modified exports.~
[ ] ~I have verified the functionality in some of the neeto web-apps.~
[x] I have added the necessary label (patch/minor/major - If package publish
is required).

Reviewers

@AbhayVAshokan let's review and merge this after tomorrow's compliance release since this deletes the files from the repo.


Correct lingering FATAL_FAILURE sites

This should have been done as part of 5444bed32f1405ebb53b0c37d3b.
It's a defect in our test infrastructure that this bug in a test wasn't detected as part of the original change.

LGTM.

@jtattermusch: non-unit test failures must be unrelated because this pull request changes only a single Python unit test. Please merge?

I cannot merge without interop test because the merge button is grey if they are not green.


Updated the unit test name for scripts for better concistency

Updated the unit test name for scripts for better concistency
As with building and linting, we're only testing the JS that will be delivered to the browser, not that which is used for the build workflow.

Isn't this the same as #952 ?

@sebworks so it is. Closing


fix(45192): Implement interface only working after specifying import

Fixes #45192

Thanks for the contribution!


IOS Issue , SocketException .

hello all. ,
i am using the ntp function , to get the real time ,
await NTP.now();
and i got this exception on ios device  ,
SocketException: OS Error: Connection reset by peer, errno = 54, address = dns.google.com, port = 53945
any idea what is wrong !!

hi @Peet-A
have you tried to use the previous version 1.0.7?

hi @Peet-A
have you tried to use the previous version 1.0.7?

thanks @oonid  , i did as you said and it's working now , the only thing i did is to remove the little caret syntax (^) :
 ntp:^1.0.7   to  ntp: 1.0.7

so I think the problem still related to the new DoH feature (on the latest version).
will wait for the response from @knezzz
going to remove all the DoH, or mix the implementation with InternetAddress.lookup

Yeah that is most likely due to DoH. I think best thing would be to remove DoH and revert back to InternetAddress.lookup, since DoH didn't really solve any problems for the lib (still not working on the web), it only created more issues.

This package gives me the correct time when I am using Wi-Fi, but when i am using cellular data, it gives me the error of SocketException: OS Error: Connection timed out, errno = 110, address = dns.google.com, port = 44198. Why is that?

@shabeenabarde Not sure, could you paste the whole log here?

@shabeenabarde Not sure, could you paste the whole log here?

Weird.. today I am not getting the error. I don't know if it's because I deleted the package from the folder and downgraded to version 1.0.7. If the error persists again, I shall let you know.

This should be fixed with 2.0.0. Closing this, if this is still happening feel free to reopen


Styles for a hrefs are missing from the latest update

Fixed in https://github.com/ummahusla/PotatoCSS/commit/5262fa9be696fb100208df865ede8c0cf7fb92fb


python37Packages.mautrix: 0.8.5 -> 0.8.6

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.mautrix is: "A Python 3 asyncio Matrix framework."
meta.homepage for python37Packages.mautrix is: "https://github.com/tulir/mautrix-python"
meta.changelog for python37Packages.mautrix is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.8.6 with grep in /nix/store/i3p6wifx29dlnczj42mryagzyc6a4yhj-python3.7-mautrix-0.8.6

directory tree listing: https://gist.github.com/b04b6cb8ce9daf084cd9bf3a5392ba92

du listing: https://gist.github.com/c4f9e621bde95048516d7bf64641d9da

Rebuild report (if merged into master) (click to expand)

27 total rebuild path(s)

7 package rebuild(s)

7 x86_64-linux rebuild(s)
7 i686-linux rebuild(s)
6 x86_64-darwin rebuild(s)
7 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mautrix-telegram
python37Packages.mautrix
python37Packages.mautrix-appservice
python38Packages.mautrix
python38Packages.mautrix-appservice
python39Packages.mautrix
python39Packages.mautrix-appservice

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i3p6wifx29dlnczj42mryagzyc6a4yhj-python3.7-mautrix-0.8.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.mautrix https://github.com/r-ryantm/nixpkgs/archive/3f4112d5cf71d07220fbc1ea123dcb5394aad56b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i3p6wifx29dlnczj42mryagzyc6a4yhj-python3.7-mautrix-0.8.6
ls -la /nix/store/i3p6wifx29dlnczj42mryagzyc6a4yhj-python3.7-mautrix-0.8.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

mautrix-telegram
python37Packages.mautrix
python38Packages.mautrix
python39Packages.mautrix

Maintainer pings
cc @nyanloutre @ma27 for testing.

Result of nixpkgs-review pr 108353 1

4 packages built:

mautrix-telegram
python37Packages.mautrix (python37Packages.mautrix-appservice)
python38Packages.mautrix (python38Packages.mautrix-appservice)
python39Packages.mautrix (python39Packages.mautrix-appservice)

Result of nixpkgs-review pr 108353 1

4 packages built:

mautrix-telegram
python37Packages.mautrix (python37Packages.mautrix-appservice)
python38Packages.mautrix (python38Packages.mautrix-appservice)
python39Packages.mautrix (python39Packages.mautrix-appservice)

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108353 run on x86_64-darwin 1

3 packages built:

python37Packages.mautrix (python37Packages.mautrix-appservice)
python38Packages.mautrix (python38Packages.mautrix-appservice)
python39Packages.mautrix (python39Packages.mautrix-appservice)

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108353 run on x86_64-darwin 1

3 packages built:

python37Packages.mautrix (python37Packages.mautrix-appservice)
python38Packages.mautrix (python38Packages.mautrix-appservice)
python39Packages.mautrix (python39Packages.mautrix-appservice)

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108353 run on x86_64-linux 1

7 packages failed to build and are new build failures:

mautrix-telegram: log https://hastebin.com/raw/seyonawulo
python37Packages.mautrix: log https://hastebin.com/raw/efikuxowax
(python37Packages.mautrix-appservice): log https://hastebin.com/raw/efikuxowax
python38Packages.mautrix: log https://hastebin.com/raw/idufeyehuc
(python38Packages.mautrix-appservice): log https://hastebin.com/raw/idufeyehuc
python39Packages.mautrix: log https://hastebin.com/raw/wamasujeku
(python39Packages.mautrix-appservice): log https://hastebin.com/raw/wamasujeku

re-checked manually and everything builds.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108353 run on x86_64-linux 1

7 packages failed to build and are new build failures:

mautrix-telegram: log https://hastebin.com/raw/seyonawulo
python37Packages.mautrix: log https://hastebin.com/raw/efikuxowax
(python37Packages.mautrix-appservice): log https://hastebin.com/raw/efikuxowax
python38Packages.mautrix: log https://hastebin.com/raw/idufeyehuc
(python38Packages.mautrix-appservice): log https://hastebin.com/raw/idufeyehuc
python39Packages.mautrix: log https://hastebin.com/raw/wamasujeku
(python39Packages.mautrix-appservice): log https://hastebin.com/raw/wamasujeku

re-checked manually and everything builds.


"Loading episode" page is always a little taller than the browser window

...and therefore always displays a vertical scrollbar, and if you press up/down arrow (which you may do as you've just pressed down 48 times to get to that page), the whole page scrolls, which is visually jarring.

Fixed in 8b53f38985a530c0081bb7fa7a9631d9d93c67ef


Fix the usage for secret create

- What I did
Fix the usage for secret create and keep the consistency between code and document
document is
Usage:  docker secret create [OPTIONS] SECRET
code is
Use:   "create [OPTIONS] SECRET [SECRET...]"
- How I did it
- How to verify it
- Description for the changelog
Remove "[SECRET...]" from the Use for secret create in code.
Signed-off-by: yuexiao-wang<EMAIL_ADDRESS>

@thaJeztah @ehazlett PTAL

As discussed in https://github.com/docker/docker/pull/28922#discussion_r89968396, and https://github.com/docker/docker/pull/28760#discussion_r90194954


Replace -highly experimental-  with -experimental- in CC doc warning

Replace highly experimental with experimental in configuration-cache documentation warning admonition at https://docs.gradle.org/nightly/userguide/configuration_cache.html#config_cache:intro

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


ClassMethods hydrator not nydrating fields from database

I have a field in my database called hash_tags, but when the hydrator gets loaded from ClassMethods, it always skips adding this field to the returned attributes array.
It looks like it's filtering out the word has. I've had to take a hackish away around this in order to get it to work.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html

Three years later. Weee.


[ Question ] TLS client certificate verification

Hi,
I just have a question related to client connection. What is with_tls supposed to do at client builder? As far as I read at other issues is for verify the server certificate (see #1271), but I need to pass it before opening the connection. If I'm developing a desktop application, how can I get the certificate?
I never worked with TLS seriously and there's a few things that I don't understand still.
Regards

with_tls allows you to customize the TLS part of the QUIC connection. Minimally, clients need to call this function and provide a way to verify the server's certificate as a part of the TLS handshake.

If I'm developing a desktop application, how can I get the certificate?

That depends. Who are you trying to connect to?

Oh, but how can I get the server certificate? I'm not sure but as far as I know certificate is given by the server at TLS handshake, am I wrong? Sorry, as I said before, I never worked with TLS at this level.

That depends. Who are you trying to connect to?

Actually just a QUIC test server made with this library. I don't have intention to use other kind of services, just a QUIC server. That's why I don't understand how to give the certificate to the client.

I've already read the client-server example, but I still don't understand how can client get previously the certificate. I don't want to use test certs in production for obvious reasons.
I think I have not explained well, so I'll say what I'm trying to do. I want to have a QUIC server with his cert and key (the development server has a self-signed certificate). Then, I want a client where the final user can choose what server wants to connect without having previously the certificate..
I understand the purpose of with_tls function on client, but there's something to avoid that check? I don't want to develop a service just to share client certificate to client if TLS handshake already gives it.

Typically a client doesn't store the exact certificate the server sends over; it should store the certificate that signed the server's certificate chain. However since in your case your server is using a self-signed certificate, there's really nothing the client can use for verification except the exact certificate the server sends over.
I would caution you against turning off certificate verification when using s2n-quic, especially in a production use-case. There's really no point in setting up a secure encrypted channel if you don't authenticate who you're speaking to.

Oh, I understand, unless the certificate is self-signed, the certificate that client should receive is one from some CA?

I would caution you against turning off certificate verification when using s2n-quic, especially in a production use-case.

Yep, I'll keep it in mind. But how can I let s2n_quic know that I don't want to verify the certificate? I was reading the documentation and the only thing I found was this but... I don't know how to use it.

There's really no point in setting up a secure encrypted channel if you don't authenticate who you're speaking to.

I know, I'm still researching about TLS and I found a few things about authenticity, but at this point I just want to have a server where I can send "hello" and receive "world".

Oh, I understand, unless the certificate is self-signed, the certificate that client should receive is one from some CA?
Yes, the client needs something that it can use to trust the server certificate, like a root certificate.

There is no way of turning off certificate verification in s2n-quic. It's not a functionality that we expose to users since it is insecure.

Oh, well, it makes sense. I'll try another library or make my own QUIC implementation. Anyway, thanks for your help and explanations.


Implementation of OpenAPI "default" response

“Default” means this response is used for all HTTP codes that are not covered individually for this operation.

https://swagger.io/docs/specification/describing-responses/#default
It was already allowed in \cebe\openapi\spec\Responses::__construct, but ignored by openapi-psr7-validator

Good catch! I'll review this asap.
Looks like I've missed this default response keyword.


[Bug]: KIDS SEC AND ACCESSORIES UNDER PRODUCTS ARE NOT SHOWN WHEN YOU ARE IN FEATURES OR ABOUT SECTIONS

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When you are in about/features page and click on products it is not showing the kids and accessories in the dropdown
I'd like to contribute and work on it @Tejashri-Taral
Add Screenshots / Video Clip

What browsers are you seeing the problem on?
No response
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

@Tejashri-Taral can you check this issue?


[5.4][Proposal] Enable global route key name

I have been building quite a few projects over the past few months where all data from routes was retrieved based on UUIDs or other identifiers besides the numeric ID in the database so I was forced to use a BaseModel just to set the route key name for all models, otherwise the BaseModel was useful because there was no shared logic between the Eloquent Models.
<?php

namespace App;

use Illuminate\Database\Eloquent\Model as Eloquent;

class BaseModel extends Eloquent
{
    /**
     * Get the route key for the model.
     *
     * @return string
     */
    public function getRouteKeyName()
    {
        return 'uuid';
    }
}

With this PR it would be possible to just set a value called route_key_name in the config/database.php which would then be used instead of the primary key of a model. If the value isn't set it would just instead use the primary key like it is already doing now.
config/database.php
/*
|--------------------------------------------------------------------------
| Route Key
|--------------------------------------------------------------------------
|
| The "Route Key" is used to retrieve Eloquent Models from the database
| that have been defined in routes like /{user}.
|
| By default the Model would be retrieved based on the primary key which
| which usually is "id". When building an API you might only expose the
| UUID or some other secondary/public identifier of records so that all
| records would need to be retrieved by the "uuid" column instead of "id".
|
*/

'route_key_name' => 'uuid',

Prefer just using the base model approach rather than adding a config option. I try to avoid adding them wherever possible.

@taylorotwell This would remove the need for any config options and is quite simple.
<?php

namespace Illuminate\Database\Eloquent;

abstract class Model
{
    protected static $routeKeyName = 'id';

    public static function globalRouteKeyName($value)
    {
        static::$routeKeyName = $value;
    }

    public function getRouteKeyName()
    {
        return static::$routeKeyName ?? $this->getKeyName();
    }
}

<?php

namespace App\Providers;

class AppServiceProvider extends ServiceProvider
{
    public function boot()
    {
        \Eloquent::globalRouteKeyName('uuid');
    }
}


Threads.SynCa and SynCaFun no longer needed (after GPU PR merge)

SynCaSend and SynCaRecv are now using NeuronFun. Could do some further experimentation to see if it might somehow be beneficial to use fewer threads via SynCa param but I doubt that would be the case..
Also is there any benefit to using this strategy on CPU too: https://github.com/emer/axon/issues/171 ?

@siboehm are you planning to take another pass through threading, including updating DWt to go per-neuron or per-synapse? If so, this can be done in that context..

@siboehm are you planning to take another pass through threading, including updating DWt to go per-neuron or per-synapse? If so, this can be done in that context..

I want to get data parallel finished first, but after that I'll take a look at this again, won't be a very complicated change.

everything is neuron parallel and using the same NThreads param -- much simpler.


Could I change the max_seq_length?

I need to input a length greater than the maximum sequence length.Could I need change the max_seq_length?

Hey yes, you can increase it up to 2048.
I'm not sure if the performance will be better than just cutting your inputs to be shorter.

Hey yes, you can increase it up to 2048. I'm not sure if the performance will be better than just cutting your inputs to be shorter.

Thank for your replay. Could I increase it up to 4096? Does max_seq_length refer to the longest sequence length of tokens? If it is in Chinese, what is the approximate length limit?

4096 will work for the sgpt-bloom models I think as they use Alibi embeddings so you could theoretically use any sequence length you want.
Yes it does
It is the same for all languages


Exclude root node from the cert

The root node is not a real node, so it's measurements should not be in the certs.

Not doing anymore, since there are now two ways the root node could have measurements - either it's auto-inited with a measurement or deriveContext, recursive=True is called on it. And its that big of a deal if we include some dummy measurement in the cert.


jitpack down

The maven repository at https://jitpack.io appears to be down. Does anyone have a working alternative?

Down again!

jitpack.io is down again, perhaps for the holiday break since nothing seems to be happening. What's an alternative so we can break with jitpack?


Removed question marks on note

@riverar,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


getSelectedValues throws an error when using a select-multiple

🐛 Bug report
Current Behavior
getSelectedValues throws an error because options is a HTMLOptionsCollection and not an array.  getSelectedValues tries to do a .filter on the options, but since it isn't an array, that call fails.
Expected behavior
I would expect the function to complete without error.  Thereby making select multiple lists work properly.
Reproducible example
This can actually be demonstrated by running the Checkbox example form from the examples folder in Github
Suggested solution(s)
If you convert the options object to an array, it works properly.  The following code works, but there may be a better way:
return Array.apply(null, options).filter(function (el) {
    return el.selected;
  }).map(function (el) {
    return el.value;
  });

Additional context
None
Your environment

Software
Version(s)

Formik
2.0.1-rc.13

React
16.9.0

TypeScript
Not using

Browser
All

npm/Yarn
6.11.3

Operating System
OSX - Mojave

With the recent edition, the getSelectedValues accepts an array a param and should now cause any problem. I see you are using formik rc which is not supposed to be a stable release, can you try with the latest formik release (probably 2.2.0)


Its a bit harder to reproduce but happens sometimes.

I think it's all about tweaking the shapes and raycasts, isn't it ?

It happens while you are running towards enemy


Trouble updating pagespeed to support SSL

I have a Bitnami  Ubuntu server currently running apache and modpagespeed but need to upgrade. I tried installing the latest mod-pagespeed-stable_current_amd64.deb but got an error apache isn't installed and was missing a  library. Upgraded to Ubuntu 14.04.6 and that resolved the library issue but recognizing apache still caused me a problem. dpkg doesn't see apache. It's installed in /opt/binami/apache.
So, I tried following https://www.modpagespeed.com/doc/build_mod_pagespeed_from_source instructions, but it won't compile. It complains of a missing file. Any chance you might have that file? I searched your source and didn't see it. APR_Pools.h
This is the error.
pagespeed/system/apr_mem_cache.cc:25:34: fatal error: apr_pools.h: No such file or directory
#include "apr_pools.h"  // NOLINT
^
compilation terminated.
make: *** [out/Release/obj.target/instaweb_system/pagespeed/system/apr_mem_cache.o] Error 1

You can probably do:
ar x mod-pagespeed-stable_current_amd64.deb
it will give you a data.tar.gz, you can extract the apache module from there and put it in the right place.

Thanks for the response. I was able to extract the binaries from a beta deb package and put those in place of the existing ones.


memory error in running solution 5 (no issues with previous sols)

    for batch_id, data in enumerate(batch_gen):
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/site-packages/torch/utils/data/dataloader.py", line 417, in __iter__
    return DataLoaderIter(self)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/site-packages/torch/utils/data/dataloader.py", line 234, in __init__
    w.start()
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/process.py", line 105, in start
    self._popen = self._Popen(self)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/context.py", line 212, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/context.py", line 274, in _Popen
    return Popen(process_obj)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/popen_spawn_posix.py", line 33, in __init__
    super().__init__(process_obj)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/popen_fork.py", line 20, in __init__
    self._launch(process_obj)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/popen_spawn_posix.py", line 48, in _launch
    reduction.dump(process_obj, fp)
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/multiprocessing/reduction.py", line 59, in dump
    ForkingPickler(file, protocol).dump(obj)
MemoryError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/site-packages/deepsense/neptune/job_wrapper.py", line 111, in <module>
    execute()
  File "/home/andrew/anaconda3/envs/neptune/lib/python3.5/site-packages/deepsense/neptune/job_wrapper.py", line 107, in execute
    raise ExperimentExecutionException("Exception during experiment execution", ex)
deepsense.neptune.exceptions.ExperimentExecutionException: ('Exception during experiment execution', MemoryError())
Calculated experiment snapshot size: 0 Bytes
Process exited with return code 1.

Hey @yipcma
This is known issue and I'm working on it. I will get back to you with solution.


Integration tests

In this PR we add integration tests for the validation code generation.
We introduce two modules, one of which acts as an extension to the standard validation library, and the other declared the validation rules and launches ProtoData with all the validation components.
We also add the support for (min), (max), and (range) options.
Also, starting with this PR, we publish artifacts to the Maven repository associated with this GitHub repository. Previously, we attempted publishing into ProtoData's Maven repository, but failed due to the lack of permissions in the standard GH credentials. This hints at the intention of GH Packages designers to separate artifacts repo-wise. We will try that for now and see if it's handy enough.

@armiol, PTAL.

@armiol, PTAL again.


Handle errors returned from package init and start

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #17950
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #17888   +/-   ##
=======================================
  Coverage   15.17%   15.17%
=======================================
  Files          46       46
  Lines        1206     1206
  Branches      188      188
=======================================
  Hits          183      183
  Misses       1009     1009
  Partials       14       14

Continue to review full report at Codecov.

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


Extension causes high cpu load

Issue Type: Performance
Extension Name: vsliveshare
Extension Version: 0.3.1013
OS Version: Darwin x64 18.2.0
VSCode version: 1.30.0

:warning: Make sure to attach this file from your home-directory: /Users/victorguyard/ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Make sure to attach this file from your home-directory: /Users/victorguyard/ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt

@FlipFloop Thanks for the issue report, but did you forget to attach the file?

Attaching hdennison-ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt

Also reported on Linux at #1457.

What do you mean to attach it

hdennison-ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt (from Dec 21st) is showing lots of module loads. This should have been fixed when we start using web pack for Live Share extension.

Same for #1486 (https://github.com/MicrosoftDocs/live-share/files/2737204/ms-vsliveshare.vsliveshare-unresponsive.cpuprofile.txt)

We believe this issue is fixed by a combination of 2 changes:

We switched to webpack to avoid excessive loading of many small modules.
We resolved an issue related to the use of the source-map-support module that made some errors result in an infinite loop processing the stack.

The first fix is already available in the latest version of the extension in the marketplace. The second fix will be available in the update to be published tomorrow.


Disks missing in 0.8.0

RancherOS Version: v0.8.0
Where are you running RancherOS? baremetal
I have missing disks since the upgrade, they're on a controller card which have the silicon image SIL3112 chipset, which means I probably need the sata_sil module to get that working.
Is there anything I can do on my own?

ah, its build as a kernel module - so you should be able to ros service enable kernel-extras and either use ros service up kernel-extras and then modprobe to get the module, or reboot.
sorry - I'll need to investigate to see what changed there.

aah, maybe kernel-extras has just been disabled during the upgrade. I'll
check that in a bit and get back with an update.
On Wed, Feb 15, 2017, 7:21 PM Sven Dowideit<EMAIL_ADDRESS>wrote:

ah, its build as a kernel module - so you should be able to ros service
enable kernel-extras and either use ros service up kernel-extras and then
modprobe to get the module, or reboot.
sorry - I'll need to investigate to see what changed there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rancher/os/issues/1609#issuecomment-280092906, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABKLts8yFTxv4y7N9-40idaBZCxj8k0gks5rc0FTgaJpZM4MBxub
.

oooo. it might have changed, yes.

I'm having trouble running sudo ros s enable/up kernel-extras.
In version v0.7.1 I had kernel-extras enabled and I was using debian-console in which I installed zfs through the backports and I had setup user-docker to use zfs as a backend storage.
Now when I've upgraded to v0.8.0, kernel-extras was still enabled but I guess I'd need to do sudo ros s up kernel-extras to rebuild modules for the current kernel.
But as my user-docker daemon can't start because of my zfs pool not being available I can't get the kernel module that I need to bring my zfs pool up.

Should the kernel-extras really be relying on the user-docker daemon and not on the system-docker daemon?

I tried cloning the os-kernel repo, I thought I could build and run the kernel-extras image on system-docker instead.
But I couldn't get through the build script because It couldn't find the docker daemon, I even tried adding DAPPER_DOCKER_SOCKET and point it to the system-docker.sock file. still no luck.

I solved this by removing the rancher.docker.storage_driver which I had set to zfs which allowed user-docker to be ran on /var/lib/docker without my zfs mountpoint.
So I got kernel-extras enabled and started to get the module I was looking for and I got the zfs service to run and find my pool properly.
I'd say this issue is resolved.


feat: support CanvasRenderingContext2D: createPattern() method

@XGHeaven  I have already optimize the code style and fix  a bug that create pattern dosen'ntwork when the first canvan element has been drawn .


All tests in numba_dppy/tests/dppl should be updated

DPPLTestCase should be replaced with unittest.TestCase.
from numba_dppy.testing import DPPLTestCase should be removed
from numba_dppy.testing import unittest should be replaced by import unittest
All from __future__  lines should be removed
numba_dppy/testing.py should be checked if it is needed anymore.

numba_dppy/testing.py contains used functions (i.e. expectedFailureIf() decorator). So each element of this module could be considered - is used or not.

@vlad-perevezentsev any update on this?


Guard against unsupported window mode on startup

Closes #6262

I'd have hoped this GameHost code would guard against unsupported window modes, but this runs before the window is created so it has no effect.
https://github.com/ppy/osu-framework/blob/79d8e449cfb88a32b125ae3530ae53aee96b3f80/osu.Framework/Platform/GameHost.cs#L1229-L1236

@peppy this pull (or https://github.com/ppy/osu-framework/pull/6264, if it gets merged, I guess) must make it into a framework package before we do a game release.


Feedback for: /fundamentals/performance/critical-rendering-path/render-blocking-css.html

The bullet list at the end of the page points out:

The second declaration is also render blocking: “screen” is the default type and if you don’t specify any type, it’s implicitly set to “screen”. Hence, the first and second declarations are actually equivalent.

The default is "all" since HTML 5 and, if I remember correctly, it was the de facto standard previously as well, despite the HTML 4 spec saying it's "screen". Modern browsers seem to use "all" as the default regardless of doctype, anyway.

PR has been merged, so I assume it's OK to close this.


Check and raise if job limit option will be ignored

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1563867
"Limit" option will be ignored by Ansible Tower if job template has no "prompt limit on launch" option checked.
With this change error will be raised for that case.

@gmcculloug @bzwei please review

@gmcculloug @bzwei @bdunne PR updated

@gmcculloug  @bzwei  @bdunne  PR updated

I had a similar thought to https://github.com/ansible/ansible_tower_client_ruby/pull/119#discussion_r210573230, which I think is still not resolved.  The method name implies "ignoring" the options, but then an ArgumentError is raised.  Instead of the "ignore" term, perhaps "validate" or "verify" ...for the constant maybe INVALID_OPTIONS?

@Fryguy @gmcculloug @bdunne @bzwei PR was updated 3 days ago. Please review.

@Fryguy: Bronagh @bronaghs explained that IGNORABLE_OPTIONS sounds ambiguous.
We debate that, agreed on the semantical name, and I changed it to  REQUIRED_ATTRIBUTES_FOR_OPTIONS.

@miq-bot assign @bdunne

@bdunne PR updated

@bdunne PR updated

@AlexanderZagaynov I released v0.18.0 including this change

Thank you @bdunne !


Bump bin-build dependency version

Currently any package which uses advpng-bin will get audit failures because bin-build 2.2.0 has a vulnerable dependency. v3.0.0 should avoid that.

Done in https://github.com/imagemin/advpng-bin/commit/77833032242332a0ee8e895c8f4666b0e13721b5


Alphabetical sorting of drop-down lists in sidebar

Is your feature request related to a problem? Please describe.
It's quite frustrating when changing "Environments" (endpoints effectively), and the list isn't sorted the way you expect. It seems to be sorted by internal ID (based on the order you've added the endpoints), rather than alphabetical as you'd expect such a list to be sorted.
Describe the solution you'd like
Sort the "Chance Environment" drop-down lists alphabetically.
Describe alternatives you've considered
Adding the endpoints alphabetically, making sure to remove and re-add any endpoints which are out of order. Could get really hard if you have a lot of endpoints.
Additional context
The Endpoints management page does this (as well as being able to sort by URL or Group, so I know it's possible.

@G-regL We've added this in the latest version of Portainer (1.17.1)  via https://github.com/portainer/portainer/pull/1902


Add ability to merge stock items

Must be of the same part
Must be of the same supplier part?

@SchrodingersGat Isn't it what the "Group by Part" stock global settings is intended for?

No, "merging" here means taking two separate stock item entries and making them into a single entry.
For example, you have a packet of 7 red pencils, and a packet of 2 red pencils. Merge them into a single packet of 9 red pencils.
However there are so many potential edge cases to handle here, so this one has been waiting for a while.

Ok makes sense 👍

@SchrodingersGat
Just wondering, couldn't you just add up what's left in those stock items (the once you'd like to merge) and then create a new one which combines all the others and delete the ones you sourced from?

@afkiwers yep, that could be done. The question is what do you do with the "history" of the stock items..
I think that the simplest option is that if the user wants to merge stock items, the history is gone and they just have to accept that. There could be a warning message to this effect.

@SchrodingersGat When you say history, couldn't you just add an automatic note to the new item which contains information about what stock was merged together (location etc) and when using simple formatted text?


OnBlur event remove syncError from field

What is the current behavior?
1.have a form with a field that fails validation - it mention in syncError list
2.rerender form, in my case I change one column view to two column

as reconciliation happen onBlur event triggered for all fields

onBlur event remove syncErrors and form become valid

What is the expected behavior?
1.have a form with a field that fails validation - it mentions in syncError list
2.rerender form, in my case I change one column view to two column
3.form stay invalid
Sandbox Link
sorry I haven't done it yet
What's your environment?
tested on chrome
reproducible only in redux form 7.4.2, in 7.3.0 all good

Please find some time to add the sandbox please, it's all a bit unclear what you are reporting.


[Backport 2.x] Fix getAlerts API for standard Alerting monitors (#870)

Issue #, if available:
#842
Description of changes:
Backport #870
CheckList:
[X] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #875 (000e243) into 2.x (2ac70d2) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                2.x     #875      +/-   ##
============================================
- Coverage     76.48%   76.46%   -0.03%
  Complexity      116      116
============================================
  Files           125      125
  Lines          7059     7061       +2
  Branches       1033     1034       +1
============================================
  Hits           5399     5399
- Misses         1134     1140       +6
+ Partials        526      522       -4

Impacted Files
Coverage Δ

...rch/alerting/transport/TransportGetAlertsAction.kt
76.92% <100.00%> (+0.45%)
:arrow_up:

... and 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Map.deep_merge skips structs

Now doesn't deep_merge on maps since they already have atom keys and they don't support Enumerable

Hello, @kipcole9! This is your first Pull Request that will be reviewed by Ebert, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.


feat! content addressable transaction pool

Closes: #884

Wondering how much code here was copied from the existing implementations

Wondering how much code here was copied from the existing implementations

v2 started as a complete copy of v1 so there's a significant amount although the protocol does make some substantial changes to information flow. For example, you no longer have a go routine for every peer you connect with.

Since the cat mempool code seems inspired by the priority mempool I wonder if we need to address any of the performance issues Sergio alluded to in https://github.com/informalsystems/audit-celestia/issues/36

We've been running the priority mempool in 100 node networks without any noticeable problems to performance. If there are any known problems then let me know and we can work them out. I'm also indifferent if we want to base CAT off the FIFO mempool. My understanding was that priority based block production and eviction was an important design requirement.

I have split this out into 4PRs: #941, #942, #943 & #944
I have kept this in draft mode as a point of reference so people can view the complete change

I have split this out into 4PRs: #941, #942, #943 & #944. All current suggestions have been incorporated here and in those PRs
I have kept this in draft mode as a point of reference so people can view the complete change

I just saw this comment, will move my suggestions to the relevant PRs.

Closing this as all the constituent parts have been completed


System tests are failing

The system tests are currently failing with the error:
 1) Resource
       lifecycle
         "before all" hook:
     Error: Service accounts cannot create projects without a parent.
      at new ApiError (node_modules/@google-cloud/common/src/util.ts:179:5)
      at Util.parseHttpRespBody (node_modules/@google-cloud/common/src/util.ts:353:32)
      at Util.handleResp (node_modules/@google-cloud/common/src/util.ts:291:22)
      at /Users/beckwith/Code/nodejs-resource/node_modules/@google-cloud/common/src/util.ts:634:14
      at onResponse (node_modules/retry-request/index.js:200:7)
      at /Users/beckwith/Code/nodejs-resource/node_modules/teeny-request/src/index.ts:104:36
      at process._tickCallback (internal/process/next_tick.js:68:7)

@stephenplusplus you likely have the most context here.  Can you please take a look?

This "parent" concept is new to me. I went to dig in, and it looks like the API has passed us by a bit-- we're at v1, and there's a v2 now. Evidently, there's a new concept, "Organizations" (among others), which we don't have any methods/classes written for. An organization or a "folder" is what is expected as a parent, but I'm not sure a Service Account can have one. And it doesn't look like there are API endpoints to create them-- they are driven by UI actions when logged in as a Google Suite or a Cloud Identity account holder: https://cloud.google.com/resource-manager/docs/creating-managing-organization
Any ideas?

@stephenplusplus It would be nice to get this lib updated to make it functional again :) Right now we can't ship an update.

@JustinBeckwith I believe this will have to be set up from a long-door-651 project owner.

Thanks for that info @lukwam. To confirm, from the point of view of our integration tests, are these the steps to get started?:

We need to sign up for a Cloud Identity account (or, maybe there is one somewhere else already?)
We then use the Admin Console to create a service account that has the Project Creator permission

Yes, if you have a domain name, you can sign up for Cloud Identify for free. This will create a Organization for your domain and then you can create Cloud Identity users in that domain, just like in a G Suite domain, but without Gmail/Calendar/Docs/Hangouts/etc. If any of those users create a project, it will be IN the org and there is no way for one of those users to create a project that isn't in your org. And if you create a service account, you could grant that service account the Project Creator role in that Organization and then it can create projects with the Org (or one of it's folders) as the parent.

@dferber90 Did you found a solution to this problem ? I am having it as well.

Greetings folks!  If you're still having issues, please make sure to open a new bug :)

@asafyish I think I ended up using the REST API. I'm not sure as it's been a year and that project has been abandoned since.


Improvement on KODI connection needed

tmm_logs_2018-07-07.zip
Version: 3.0-SNAPSHOT
Build: 2018-07-03 00:22
OS: Linux 4.15.0-22-generic
JDK: 1.8.0_171 amd64 Oracle Corporation
What is the actual behaviour?

Settings - External Devices - Data of KODI machine - Connect: nothing happens,
no feedback, so I don't know if the connection is established.
Tools - KODI (Unknown): what does 'Unknown' mean? The name of the KODI machine?
Does not work.
Tools - KODI (Unknown) - Application - Quit: Does not work.
Tools - KODI (Unknown) - Application - Mute / UnMute: leads to a NullPointerException,
see attachment.
Tools - KODI (Unknown) - Application - Volume different levels: Does not work.
Tools - KODI (Unknown) - System - all submenus: Do not work.
Tools - KODI (Unknown) - Video Library - Scan for new content - Scan ALL Datasources:
Last level not activated
Tools - KODI (Unknown) - Audio Library - Scan for new content - Scan ALL Datasources:
Last level not activated
This entry is very interesting, see #300!
None of the entries above is translated to German.

What is the expected behaviour?
Make the KODI connection transparent (with feedback) and working.
Steps to reproduce:
See description above.
Additional
Have you attached the logfile from the day it happened? Yes

Well, yes, there needs much work to get this going :|
First we need to have an idea, what we want to solve with this connection... and then we need time...
Current methods are just there to play a bit...
"Unknown" comes from that fact, that TMM is still not able to connect to your Kodi instance.
I still see "Error calling Kodi: Verbindungsaufbau abgelehnt (Connection refused)" in your logs.
Which Kodi version do you have?
Are your ports correct?
...?

ok, seems not to work with Kodi 18 Leia at all :|

fixed some issues, at least my latest Kodi 18 Leia works...

The KODI version I'm running is 17.6. By default, the HTTP port is set to 8080. Changed that to 9090, but that didn't help with the newest tMM build (2018-07-09 12:33). Tried the RPC interface with the KODI JSON-RPC Browser (Link), and that works. There is one strange thing: I have to login twice to get the connection.
Concerning ideas about the RPC connection, I would like to support you. Can we have a chat or so, because I think the issues is not the right platform to exchange ideas, it should be more interactive.

In addition: I'm running Kodi 17.6 on a Raspberry Pi, based on OSMC 2018-06-07.

http should be 80/8080 per default, and TCP defaults to 9090
(cannot even be changed inside Kodi, only in webinterface)
Well, we have a (unannounced) public Discord channel you might want to join:
https://discord.gg/gb6V9TZ
But bear with us if we don't respond immediately - we have all day jobs & family, and barely time beside...

we've migrated v3 to GitLab (https://gitlab.com/tinyMediaManager/tinyMediaManager) along with all issues. The issues for v2 at GitHub will be closed - we will respond to the corresponding issue in GitLab in the future
br
Manuel


Electrum 1.04 disconnected

Wallet is disco.  Upgraded from 1.02.  Wanted to get coins back on Bittrex, but no go?

Click the 'red dot' for disconnected, and try the other server.


Cant choose spawnlocation

Spigot 1.12, quests 2.9.2
When i try to creat an event with mobs spawning i cant use the spawning location.
Nothing happends when right clicking and typing add.
No error logs.

I assume you're speaking of the Kill Mobs stage objective
http://imgur.com/a/PV4BG
Was just able to right-click to select location in my test environment. It's possible another plugin could be blocking the event, in which case you'll want to seek help from that plugin's author or the Spigot forums.

Closing due to lack of response from OP.


Customize Login/Subscribe Now button color

I see in the "aggregate_auth.py" file there are hardcoded values for the button HEX codes. Unless there is something obvious I am missing, this seems to be an issue when modifying st-paywall to work in an app.
Thoughts on included a parameter for the global colouring across all buttons?

Would like to second this request.. even just having a few different color options would be a huge improvement!


docs: mention required rustup install for find and verify command in …

…install instructions
fixes #467

bors r+


keypadButtonBackgroundDrawable

Hello
Your lib is great
But keypadButtonBackgroundDrawable does not apply correctly
Even when I set
app:keypadButtonBackgroundDrawable="?attr/selectableItemBackgroundBorderless"
it's apply only to zero button

Sorry for getting too late on this. Let me investigate this issue and get back to you on this.

Yeah, this would be great if this was fixed soon. Im putting off using the library until this is fixed.
Thank you.

Can't reproduce this issue.

Hi, did you find any solution about this?


Hook up -emit-interface-path to a simple AST printer

Next step after #18090 for module stability / textual interfaces. Still mostly just a dummy at this point, but we can start writing tests.

@swift-ci Please test

@CodaFi, the second commit is the one relevant to you, since you were looking at filesystem stuff.

Build failed
Swift Test OS X Platform
Git Sha - caafaeee0ecf2d4d9e950e7746c035d6ff4670f5

@swift-ci Please test macOS

PS: Always a pleasure reviewing your code. Your care inspires me to look for perfection.

Rebased, feedback incorporated.
@swift-ci Please smoke test

@swift-ci Please smoke test


feat(obstacle_avoidance_planner): fix reference points' boundary width

Description

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kosuke55 @takayuki5168
Are there any reason why this isn't merged? Also, please make sure that you always write your PR is about under "Description" section.


Make MAX limit for retrieved rows an env variable

Is your feature request related to a problem? Please describe.
Currently the  MAX limit is hardcoded
Describe the solution you'd like
I would like it to be a ENV variable i can change.

We should also update the spec accordingly to state that the max can be configured and advise against setting it too low as a low max would make more likely the below issues detailed in The Mother of all Queries section of the wiki:

Hidden partial results
Limit not honoured

Hi @c0c0n3 , Please assign this issue to me if no one is working on it. Thank you

Hi @daminichopra, assigned to you. thanks for offering :-)

closed by #330


Crashing upon initiating a random encounter

Hi,
I often crash when having a random encounter. In this recent crash, I first had a random encounter. I then lost one of my character (the rogue, game.party[1]), which made me reload my save game. I then tried once more to find a random encounter, which crashed the game.
The first random encounter had 2 sea hags (one was alive at the end, when my rogue died), 2 water snake and 2 birds (Kapocinth I guess, one was alive at the end, when my rogue died). I crashed before being able to see the 2nd random encounter. No crash dump was generated. By looking at the log, I find strange to see so many different kind of enemy destroyed (also there is trolls, and they usually only hang with themselves)
I'm also unsure what to think of the shitload of errors ("[core] [error] Unable to register texture", and the "[core] [error] Unable to load MDF file").
TemplePlus.log.txt

Thanks, the log was very helpful. (I see that you found it eventually?)
The texture errors are unrelated and harmless, they just indicate there's no texture for certain parts of the map or townmap (the ones that are just black).

That's very possible (regarding resting on the map). I've never crashed after this fix while initiating a random encounter; thanks (I'd believe it was my main major crash when playing this game).

Yeah, the cause of the crash was the "encampment" script setup. That's why it was "seldom", because it happens about 1 out of 16 times or something like that.


Azure Pipelines clock issues

In build 55800 the clock keeps jumping back and forth about 16 hours:
2020-06-18T16:46:46.1993748Z
2020-06-19T09:28:05.5534482Z
https://dev.azure.com/powershell/2972bb5c-f20c-4a60-8bd9-00ffe9987edc/_apis/build/builds/55800/logs/35

@TravisEz13 @adityapatwardhan Do you know why the timestamp is acting like this?

The lines that are being logged in one time zone are on the service and the other are on the agent.

The link to report an issue is here:  https://developercommunity.visualstudio.com/spaces/22/tfs.html?type=problem