Bookmark URL Misunderstanding?

Hi,
Should it be possible to connect directly to a database using the bookmark url WITHOUT having to go via the login screen?
If so, what is the format for the full URL? For example, if I connect to pgweb with URL   http://localhost:8081/#postgres://postgres:myPwd@localhost:5432/mydb?sslmode=disable then it takes me to the normal login screen but I'm expecting it just to log straight in to the database as per the bookmark details in the URL.
Am I doing something wrong here or can bookmarks only be used be on the Scheme page?

Using latest pgweb (0.9.6) you should be able to connect to the bookmark using -b or --bookmark flag. The bookmark must be saved under $HOME/.pgweb/bookmarks first.

Btw, the url you provided does not work that way. If you have a few bookmarks saved, they'll be available on the login page. There's no other way to use them in the web app, only via the command line option.


CHIP-SPV Integration

Changes enabling the use of SPIR-V backend

SPIR-V as a platform is not following the existing precedent.  And I'm not clear if the compiler generating the SPIR-V is expected to produce code using only KHR extensions.
Also, suppose one of existing platforms wanted to add SPIR-V support to existing ISA or PTX support?  Then I might want to consider SPIR-V as an --offload-arch instead of as a platform.

I have the same question as Brian. Is the 'SPIRV platform' intended to be a target-neutral platform that works for all devices supporting SPIRV, or is it intended to support SPIRV on Intel platform only? If it is the latter, then it is better called Intel platform.
This also raises a question about supporting SPIRV on other platforms, e.g. ROCm. We need to come up with a structure that allows SPIRV support for different platforms.

And I'm not clear if the compiler generating the SPIR-V is expected to produce code using only KHR extensions.

Practically the SPIR-V code emission is implementation detail in CHIP-SPV from the HIP users’ point of view. The SPIR-V code generation orchestrated by CHIP-SPV and the emitted code should be constrained by target’s (a CHIP-SPV backend, e.g. OpenCL) capabilities and the SPIR-V environment specification.

Is the 'SPIRV platform' intended to be a target-neutral platform that works for all devices supporting SPIRV, or is it intended to support SPIRV on Intel platform only?

From CHIP-SPV’s readme: “CHIP-SPV that aims to make HIP and CUDA portable to platforms which support SPIR-V as the device intermediate representation. Currently CHIP-SPV supports OpenCL and Level Zero as the low-level runtime alternatives”.
In a sense, the CHIP-SPV is target neutral (to some degree*1) but mostly the development effort has been on supporting HIP applications to run on Intel devices  - so no guarantees yet the CHIP-SPV would run SPIR-V code on other devices.
1: some capabilities are required from the target for modeling some HIP features - e.g. for OpenCL backend, including but not limited to: image support for textures, subgroup extension for __shfl functions and coarse-grained SMV (at least) for presenting device allocations as pointers on the host side code.

This also raises a question about supporting SPIRV on other platforms, e.g. ROCm.

I would imagine that each platform might have a different flavor for the SPIR-V code (e.g. ROCm would probably use SPV_AMD* extensions). Should/would HIP applications pack SPIR-V code objects for different platforms? How could we tell which platform SPIR-V code objects, embedded in HIP binaries (more specific: in clang offload bundles), are targeted at?

We need to come up with a structure that allows SPIRV support for different platforms.

Perhaps, extend the Clang offload bundle’s bundle entry ID with a (mandatory?) target platform field?
My two cents on the platform naming: could we call it “CHIPSPV” or just “CHIP”?

Yes, the intention is to support all targets with SPIR-V support, and not be Intel specific. However, some features of HIP cannot be yet expressed with khr/ext extensions only (there are only a couple in the list), but we are in progress making them such. So - meanwhile some of the features work only if the required Intel extension is supported. Note, however, that if those features are not used by the HIP kernel, it won't utilize that extension. Having Intel or AMD "flavors" of the SPIR-V target is thus not wanted nor sensible since they will be obsoleted as soon as we standardize those extensions.


Labs - all

Module: 00
Lab/Demo: 00
Task: 00
Step: 00
Description of issue
Here is the problem.
Can I ask, who is maintaining the labs?
Lab06 is out of date and has no way of working unless a trainer writes the whole thing.
Lab05 has issues with node and more issues.
How does anyone cover a good of high quality.
I shall do my best based on the effort Microsoft are applying to the course material.
Repro steps:

Thank you for the feedback. We made some updates, please let us know if these aren't working for you by filing a specific issue.


GN-3998: Instead of joining into 'block_rdfa' node - select it when deleting at the end of it

https://user-images.githubusercontent.com/769698/229424296-8a94810e-f42f-476e-b62f-21c11594f4e5.mp4

For some reason this makes every block behave like the block_rdfa I don't think there's anything wrong with your code but there's some interaction going on with the command chain I think.
fyi non-text nodes are selectable by default

Could you elaborate on this? Because I didn't see anything wrong and want to know what's wrong for the next one :)

additionally this pr doesn't deal with delete. I'm fine with splitting that into another pr though, lets get this one consistent first

the point is that the two examples: the list item and the first paragraph, are not block_rdfa nodes but they are still nodeselected

unfortunately....
minimal repro:
clear the doc, use customhtml to insert:

this is inside the rdfa node

text

backspace away the "text" and keep backspacing, now it doesn't trigger the nodeselection
but if you click on the newline right before the rdfa node, backspacing does trigger correctly

interestingly, you get p much the same behavior if you disable your selectBlockRdfaNode command altogether
I suspect you're approaching the same implementation as the selectNodeBackward command which is already a part of the backspace chain

Im actually quite surprised at that, it breaks my assumptions about "isolating"

aah further clues: the culprit actually seems to be this ad-hoc construction in the backspace chain:
  (state, dispatch, view) => {
    const isInTable = hasParentNodeOfType(state.schema.nodes.table)(
      state.selection
    );
    if (joinBackward(state, dispatch) && dispatch && view) {
      const { state } = view;
      if (!isInTable) {
        selectParentNodeOfType(state.schema.nodes.table)(state, dispatch, view);
      }
      return true;
    }
    return false;
  },

If I move the selectNodeBackward command to BEFORE this construction, it works as I had imagined
so I think it's the "joinBackward" mentioned above seems to have more power than we thought
the assumption was that it would follow the same rules as the "default" backspace behavior when it came to isolating/selectable nodes. It seems it maybe doesn't? Not sure, but I'd investigate there for sure

asking for the extra page is kind of out of scope, depending on how you look at it, so feel free to make a ticket for it and not do it in this pr
but it does have to be made non-breaking for current consuming apps


[BUG] Gemini API is not working

Describe the bug
When using macai v2.0.0-beta.1
While "Test API token & model" did work when adding Gemini APi key.
In real chats, It is just completely broken, showing blank text in both user prompt and gemini response.
BTW, I'm using Google AI studio's API key, not Vertex AI's api key.
To Reproduce
Steps to reproduce the behavior:

add Google AI studio API key
Test with basic chat
It does not work

Expected behavior
chat should work
Screenshots
https://github.com/user-attachments/assets/5a2da560-d729-4436-8fb3-ddc8039ed21a
Environment:

OS version: macOS Sequoia 15.2
Architecture: arm64

@wedlj2094 unfortunately I can't reproduce this right now, because Google API is broken for me and returns 429 (Resource has been exhausted (e.g. check quota)) for my API keys for absolutely no reason. But it definitely worked for me with gemini-1.5-flash in the past. Could you try checking with this model? Could you also try turning on/off stream responses?

I tested more, and the issue was stream response option.
If I turn off stream response, every Gemini models (including new experimental models) work.

@wedlj2094 Thanks, that was really helpful. My Gemini API started working again and I was able to reproduce, find the root cause and fix the problem.
The reason was in Gemini API response, that has changed slightly. The current fix should make macai more tolerant of small API changes from OpenAI-like APIs.

The fix is already released with the beta 2


System properties should have precedence over environment variables, fixes #675

@ppalaga do you recall why we chose to prioritise environment variables over system properties originally ?


Feature: #45 Modal.

Description
Adds a modal. See #45
Testing notes

Visit http://localhost:6006/?path=/story/molecules-modal--small and similar examples.

Example

Documentation

Notes
Took concepts from https://micromodal.vercel.app and https://patternflyelements.org/components/modal
The controls for slots in Storybook are weird and this documentation works, but is awkward. Open to suggestions!

I worked on documentation notes to describe functionality and usage.
Added events so others can detect what's happening and respond to changes.
Switched to using Tailwind classes in most cases.
Still need to add styles to different sizes of the modal (small, medium, full-screen).

I added the styles for different sizes. Going to review this later today and see if there are other things I need to add.

Going to review the W3 requirements, https://www.w3.org/TR/wai-aria-practices-1.1/examples/dialog-modal/dialog.html.

 Still need to trap the focus within the modal.
It also surfaced an issue where I wasn't able to focus our outline-button element. Not sure why.

Added an automated test that checks that the slots are filled when the modal is rendered, opens, and closes.
Also added the shouldForceAction attribute so we can force the user to take an action if needed.


Zoro.to response 502

Source information and language
Zoro.to 13.9
Steps to reproduce
Everytime I play a video it shows response 502
Expected behavior
It should work perfectly
Actual behavior
When I try to watch anime on zoro.to it after playing video for 2 to 3 minutes it always get paused and shows Response 502. I've tried everything that I can do so please solve this problem
Aniyomi version
<IP_ADDRESS>
Android version
Android 10
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.13.5.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Closing as this not reproducible on latest versions of app and extension


Uncaught Error: t.render(): A valid React element (or null) must be returned.

Config

import React  from 'react';
import MUIDataTable from 'mui-datatables';

class TeamV2 extends React.Component {

  render () {

    const columns = ["Name", "Title", "Location", "Age", "Salary"];

    const data = [
      ["Gabby George", "Business Analyst", "Minneapolis", 30, "$100,000"],
      ["Aiden Lloyd", "Business Consultant", "Dallas", 55, "$200,000"],
      ["Jaden Collins", "Attorney", "Santa Ana", 27, "$500,000"],
      ["Franky Rees", "Business Analyst", "St. Petersburg", 22, "$50,000"],
      ["Aaren Rose", "Business Consultant", "Toledo", 28, "$75,000"],
      ["Blake Duncan", "Business Management Analyst", "San Diego", 65, "$94,000"],
      ["Frankie Parry", "Agency Legal Counsel", "Jacksonville", 71, "$210,000"],
      ["Lane Wilson", "Commercial Specialist", "Omaha", 19, "$65,000"],
      ["Robin Duncan", "Business Analyst", "Los Angeles", 20, "$77,000"],
      ["Mel Brooks", "Business Consultant", "Oklahoma City", 37, "$135,000"],
      ["Harper White", "Attorney", "Pittsburgh", 52, "$420,000"],
      ["Kris Humphrey", "Agency Legal Counsel", "Laredo", 30, "$150,000"],
      ["Frankie Long", "Industrial Analyst", "Austin", 31, "$170,000"],
      ["Brynn Robbins", "Business Analyst", "Norfolk", 22, "$90,000"],
      ["Justice Mann", "Business Consultant", "Chicago", 24, "$133,000"],
      ["Addison Navarro", "Business Management Analyst", "New York", 50, "$295,000"],
      ["Jesse Welch", "Agency Legal Counsel", "Seattle", 28, "$200,000"],
      ["Eli Mejia", "Commercial Specialist", "Long Beach", 65, "$400,000"],
      ["Gene Leblanc", "Industrial Analyst", "Hartford", 34, "$110,000"],
      ["Danny Leon", "Computer Scientist", "Newark", 60, "$220,000"],
      ["Lane Lee", "Corporate Counselor", "Cincinnati", 52, "$180,000"],
      ["Jesse Hall", "Business Analyst", "Baltimore", 44, "$99,000"],
      ["Danni Hudson", "Agency Legal Counsel", "Tampa", 37, "$90,000"],
      ["Terry Macdonald", "Commercial Specialist", "Miami", 39, "$140,000"],
      ["Justice Mccarthy", "Attorney", "Tucson", 26, "$330,000"],
      ["Silver Carey", "Computer Scientist", "Memphis", 47, "$250,000"],
      ["Franky Miles", "Industrial Analyst", "Buffalo", 49, "$190,000"],
      ["Glen Nixon", "Corporate Counselor", "Arlington", 44, "$80,000"],
      ["Gabby Strickland", "Business Process Consultant", "Scottsdale", 26, "$45,000"],
      ["Mason Ray", "Computer Scientist", "San Francisco", 39, "$142,000"]
    ];

    const options = {
      filterType: 'dropdown',
      responsive: 'stacked'
    };

    return (
      <MUIDataTable
        title={"ACME Employee list"}
        data={data}
        columns={columns}
        options={options}
      />
    );
  }
}

export default TeamV2

Issue

util.js:546 Uncaught Error: t.render(): A valid React element (or null) must be returned. You may have returned undefined, an array or some other invalid object.
    at invariant (invariant.js:42)
    at ReactCompositeComponentWrapper._renderValidatedComponent (ReactCompositeComponent.js:828)
    at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:359)
    at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:255)
    at Object.mountComponent (ReactReconciler.js:43)
    at ReactCompositeComponentWrapper._updateRenderedComponent (ReactCompositeComponent.js:762)
    at ReactCompositeComponentWrapper._performComponentUpdate (ReactCompositeComponent.js:721)
    at ReactCompositeComponentWrapper.updateComponent (ReactCompositeComponent.js:642)
    at ReactCompositeComponentWrapper.receiveComponent (ReactCompositeComponent.js:544)
    at Object.receiveComponent (ReactReconciler.js:122)
invariant @ invariant.js:42
_renderValidatedComponent @ ReactCompositeComponent.js:828
performInitialMount @ ReactCompositeComponent.js:359
mountComponent @ ReactCompositeComponent.js:255
mountComponent @ ReactReconciler.js:43
_updateRenderedComponent @ ReactCompositeComponent.js:762
_performComponentUpdate @ ReactCompositeComponent.js:721

Your Environment

Tech
Version

mui-datatables
1.0.9

node
5.5.1

OS
Windows 10

Package dependecies
"dependencies": {
    "animated": "^0.2.2",
    "aphrodite": "^1.2.5",
    "classnames": "^2.2.5",
    "date-and-time": "^0.6.2",
    "eventsource-polyfill": "^0.9.6",
    "firebase": "^4.11.0",
    "jquery": "^3.3.1",
    "lodash": "^4.17.5",
    "material-ui": "^1.0.0-beta.37",
    "material-ui-icons": "^1.0.0-beta.36",
    "normalize.css": "^7.0.0",
    "object-assign": "^4.1.1",
    "promise": "^7.1.1",
    "prop-types": "^15.5.10",
    "react": "^15.5.4",
    "react-collapsible": "^2.0.4",
    "react-dom": "^15.5.4",
    "react-firebase-file-uploader": "^2.4.1",
    "react-magic": "^0.2.3",
    "react-number-format": "^3.1.6",
    "react-redux": "^5.0.7",
    "react-redux-firebase": "^2.0.5",
    "react-router": "^3.0.0",
    "react-tap-event-plugin": "^2.0.1",
    "react-transition-group": "^2.2.1",
    "react-router-dom": "^4.2.2",
    "google-map-react": "^0.34.0",
    "recompose": "^0.26.0",
    "redbox-react": "^1.3.6",
    "redux": "^3.7.2",
    "redux-auth-wrapper": "^1.0.0",
    "redux-form": "^6.6.1",
    "redux-form-material-ui": "^5.0.0-beta.2",
    "redux-thunk": "^2.2.0",
    "whatwg-fetch": "^2.0.3",
    "mui-datatables": "^1.0.9"
  },

Unfortunately I believe this error is due to the fact that this package requires you to run at minimum React 16. Is there any chance you can upgrade?


[Ruby] Service factory module names can conflict with model class names

When generating google-cloud-dataproc, there is a service called WorkflowTemplateService. The ruby generator creates a factory module Google::Cloud::Dataproc::V1::WorkflowTemplate which contains the factory method for creating the service client. The gapic generator's current practice is to strip the words Service and Client from the name of this module. In this case, however, there is already a proto class named WorkflowTemplate that thus conflicts, resulting in code that does not load. (Note that the proto names the service WorkflowTemplateService to thus distinguish it from the WorkflowTemplate data type, but the generator strips off Service.) The generator needs to detect this case, and refrain from stripping Service from the name if necessary to distinguish the names.
See https://github.com/googleapis/google-cloud-ruby/pull/2685

@dazuma I just CC'd you on a related internal request about this. Should we reconsider how this works (i.e. make it configurable or stop stripping vs. checking for this special case?)

Hi @jbolinger @dazuma, is there a decision made on this issue? For now, it seems gapic-generator continues to trim Services, but they are added back during packaging, and this is causing generated samples to not work. For example:

Services are added back for talent services: https://github.com/googleapis/google-cloud-ruby/blob/master/google-cloud-talent/synth.py#L80
But generated samples do not know this post-process, so the generated sample looks like this:

require "google/cloud/talent"

# [START talent_list_companies]

# List Companies
def sample_list_companies(project_id)
  # [START talent_list_companies_core]
  # Instantiate a client
  company_client = Google::Cloud::Talent::Company.new version: :v4beta1

  formatted_parent = company_client.class.project_path(project_id)

  # Iterate over all results.
  company_client.list_companies(formatted_parent).each do |element|
    puts "Company: #{element.name}"
    puts "Display name: #{element.display_name}"
    puts "External ID: #{element.external_id}"
  end

  # [END talent_list_companies_core]
end

We would want the generated samples to be able to run against generated clients, so we can't append the Service in gapic-generator for SampleGen. Any ideas?
cc/ @googleapis/samplegen

No, we have not solved this. Synth is rewriting any name collisions for the time being.
The chances of name collisions are even higher in the micro-generator but we haven't decided upon a resolution strategy for it yet. I'm not really sure what you can do short-term other than special case it for instances like these where we know synth is patching it.

Yikes. Best of luck in the micro. I hope y'all land on the side of using the full service name for sanity when working against protos. Tho the Rubyist in me hates the Service suffixes. Le sigh.
Hmm. @hzyi-google let's have the sample generator continue to trim Service suffixes and I'll use the synth scripts to add it back, as necessary. I can basically just look at each synth script for a given library and, if there is a s.replace adding Service, then I'll have it do the sample to the sample files.

SGTM

@hzyi-google @jbolinger Any updates on this? We're out of SLO.

There doesn't seem to be a plan to solve this issue in the monolith, but the micro generator will handle it.


Nodejs v14 Support added with node-sass 4.14.0

Hello. When there will be support for nodejs 14 ? At the moment, on Windows 10 x64 installation is not possible.

Do not upgrade to the new node engine before making sure it works. And you can always compile on your own.

As with previous major releases, there are several pieces that need to be done before a 4.14.0 is released with support https://github.com/sass/node-sass/issues/2632
I've pinned this issue for visibility

Reopening till the release is cut

@xzyfer the only other thing that I think we might want to land for 14 is https://github.com/sass/node-sass/pull/2769
If you want to manually fix that when you bump before cutting the release

I've been putting of rebuilding all the binaries but I guess I can bite the bullet :+1:

I've kicked off the 4.14.0 binary build, and tagged the release including #2769.

Linux, Alpine, OSX, and Windows binaries are built and uploaded. Will publish a release in the morning unless @nschonni is feeling up to it whilst Australia sleeps.

Thanks for the quick update. Works for me now with node v14. Just had to put python 2.7 on my path and all went smoothly.

Would it be possible to get that release published please? :)

Whoops knew I forgot something. Just published as v4.14.0.

Thank you! 😄

Oddly it's appearing on yarn but not npm. Not sure if that's normal - maybe just needs time to sync

@xzyfer  What about v14.16.0?

What about v14.16.0?

What do you mean? Isn't v14.16.0 working with the recent node-sass?

Yes, had an issue

<EMAIL_ADDRESS>install [path]node-sass
node scripts/install.js
Downloading binary from https://github.com/sass/node-sass/releases/download/v4.9.2/darwin-x64-83_binding.node
Cannot download “https://github.com/sass/node-sass/releases/download/v4.9.2/darwin-x64-83_binding.node”:
HTTP error 404 Not Found

Resolved somehow. Excuse for troubling


enh(notion): display block children in orphaned resources

When a database or page parent is a block that is within a page that our integration is not allowed to see, the "parentId" will be the ID of that block.
That means we cannot surface this database in the resource tree. If we use unknown as the parent, it will show up in the orphaned resources which is better.

Sorry @spolu this is WIP

sitrep?

@spolu Closing this needs more thinking


authentication problems with web interface

cups 2.2.11 on openbsd 6.5
first attempt to add a printer presented with a dialog box that would not accept either my name or root with passwords, kept just throwing up the box again, so I reset authentication to none, attempting to add a printer nothing happens, it just looks like it cant find anything but I know from using it on other systems it should ofer me to enter where the printer is, though find new printers and manage printers show no printers, attempting to view the logs via the interface wants a user name and passwd but does not present anywhere to enter it.
The only change to the conf file is to alter authentication to none.
access log just repeats
localhost - - [05/Aug/2019:15:01:16 +0100] "POST / HTTP/1.1 401 75 CUPS-Get-Devices successful-ok  many times
nothing in error_log
nothing in messages
any suggestions appreciated, thanks, I am trying to add a networked printer

Try enabling debug logging (cupsctl --debug-logging) - I suspect an issue with the PAM configuration.

Hello,
I got the same problem with either CUPS 2.2.10 or CUPS 2.3.0.
My setup was Debian's default plus AuthType Negotiate. Kerberos is working properly since I use it to access the CUPS server's via web using AuthType Negotiate inside  scope.
DefaultAuthType Negotiate <Location />   AuthType Default   Order allow,deny   Allow @LOCAL </Location>
But if I try to add a printer (I think it is the CUPS-Add-Modify-Printer operation), the browser just keeps loading forever.
<Limit CUPS-Add-Modify-Printer CUPS-Delete-Printer CUPS-Add-Modify-Class CUPS-Delete-Class CUPS-Set-Default>   AuthType Default   Order deny,allow </Limit>,
The log was flooded with [Client %d] entries just like the following:
D [07/Nov/2019:17:48:27 +0000] [Client 27727] Server address is "/run/cups/cups.sock".
D [07/Nov/2019:17:48:27 +0000] [Client 27727] Accepted from localhost (Domain)
D [07/Nov/2019:17:48:27 +0000] [Client 27727] Waiting for request.
D [07/Nov/2019:17:48:27 +0000] [Client 27727] POST / HTTP/1.1
D [07/Nov/2019:17:48:27 +0000] [Client 27727] Read: status=200, state=6
D [07/Nov/2019:17:48:27 +0000] [Client 27727] No authentication data provided.
D [07/Nov/2019:17:48:27 +0000] [Client 27727] cupsdSendHeader: code=401, type="text/html", auth_type=0
D [07/Nov/2019:17:48:27 +0000] [Client 27727] WWW-Authenticate: PeerCred
D [07/Nov/2019:17:48:27 +0000] [Client 27727] Closing connection.
`grep authentication error_log  -c

This is a dupe of #5596...


Navigate to Multiple pages after navigateAfterSeconds

In My Case i want to navigate to different pages based on some login response like below, like below.
_handleNavigation() {
if(UserSession.loginStatus == 0) {
print('LoginStatus -> ${UserSession.isLoggedIn}');
return LoginPage();
} else {
print('LoginStatus -> ${UserSession.isLoggedIn}');
switch (UserSession.shortName) {
case 'stay':
return AccommodationListPage();
//          break;
    case 'activity':
      return ActivityDashboard();

//          break;
    case 'transport':
      return TransportDashboard();

//          break;
    case 'tour':
      return PendingContractsPage();
  }
}

}
But am getting issues with this function: Like it "Should be String or Widget."
How can i achieve my requirement. Please help me out on this.
Thanks Alot !.

Same Error

This issue will be solved if you added the return type of Widget to the function like this
Widget _handleNavigation() {
return WhateverPage();
}

Thank you


grant_id should not be required

The current schema indicates that grant_id is required but the statuses are: planned, applied, granted and rejected.
It is unlikely that a grant_id would exist when the status is planned, applied or rejected.

The grant_id is required only if you have a funding-entry (lines 886-975), as far as I discern. Having a funding entry in itself is not required, it is possible to validate perfectly without it. A dataset is a required key, though.

This might be a matter of what an id is useful for. On one hand, it can be a PID (persistent, useful outside the place it was first defined). On the other, an id can be needed to make a repeated section uniquely referencable within the system, like a primary key in an SQL table. The latter need not be PIDs and need not be shared with anyone. Without something functioning like the latter, the only way to tell apart two datasets is the order they are defined in the json, so: dataset with index 0 vs. dataset with index 1 etc. (You could in fact use that index as an id and thus be explicit about it.)
I think the two uses has been conflated in this standard: you really need the latter type (since the standard does not define that arrays are ordered [though it may be assumed], but the first type can always be added later. This ties into #33 and #34.

I think the cardinality of grant_id should be changed to 0..1. If I recall correctly, in the "project branch" of the specification we wanted to keep the options open.


.woff2 material icons font incorrect

If my browser uses the .woff2 file for material-design-icons, then all of the icons are incorrect and the character codes are displaying the wrong icons.
eg. the keyboard-right icon is shown as a folder
If I rename the .woff2 file and use the standard .woff (using chrome/firefox), then the icons display as normal.

Please see some of the closed issues about this


Supporting two-way communication

Using the JSON response from the custom server, currently { result: "ok" } for introducing new values in the app. For instance, setting a geofencing area or changing the tracking times remotely.  Something like:
{  result:"ok",
set:{
tracking:5,
geof:{ mode":"out", center:[ 39.12, 0.28 ]}
}
}

It would be cool if Overland supported loading settings from a remote URL similar to Micropub configuration queries.

This is done now! Check out the readme for details!
https://github.com/aaronpk/Overland-iOS?tab=readme-ov-file#configuration-by-server-response


AdmissionRequestDenied: Internal error occurred: failed calling webhook "webhook.cert-manager.io": failed to call webhook

I has problem when add-on adot to AWS EKS Fargate.
Bugs description: Internal error occurred: failed calling webhook "webhook.cert-manager.io": failed to call webhook: Post "https://cert-manager-webhook.cert-manager.svc:443/mutate?timeout=10s": x509: certificate is valid for ip-172-16-17-60.ap-southeast-1.compute.internal, not cert-manager-webhook.cert-manager.svc
I was installed cert-manager:

Please help me,
Many thanks.

Did you follow this compatibility warning when installing cert manager on fargate? https://cert-manager.io/docs/installation/compatibility/. You need to change the value of webhook.securePort during installation.


tests: update hello_events tests for hw

Fix bug in fpgad. The fme.x/errors/fatal_errors was renamed to
fme.x/errors/catfatal_errors in the driver.

(cherry-pick to master from feature/plugin-api)


More refactor

Major changes in lychee-lib::filter module:

Fields in Excludes except the RegexSet is now moved to Filter.
Filter contains Option<Excludes> and Option<Includes>, which are
wrapper struct of RegexSet instead of Option<RegexSet>. As a result
the code now looks cleaner.
Factored out some filtering logics to dedicated functions.

It's possible to write tests for those functions in addition to tests
for the Filter struct.

Added docs to Filter::is_excluded and reorgnized the code.

placed derive_builder by typed_builder:

The internal interface very ugly, as admitted by the author, but we no
longer have nested Options like before.
As a result, the Client building is much easier to read.
Main benefit of typed_builder is, the arguments feeded to builder is
checked at compile time instead of run-time.

Fixed a bug in lychee::tests::usage and lychee-lib::stats::test.

Now it will clear environment variable which would otherwise cause an
issue if GITHUB_TOKEN is set.

Updated dependencies.

Well #222 was closed since I've renamed the branch to work with git-flow.

lgtm. There is one final error in https://github.com/lycheeverse/lychee/pull/225/checks?check_run_id=2354928844 that should be an easy fix.

You can build it locally, since cargo test passes.
cargo publish instead look for the package from registry, which is the one published yesterday. So it complains about not finding the function.
Again we need to publish the lychee-lib first.
P.S. And I haven't changed the version number. It could be v0.7.0.alpha since the API changes.

Works locally. Let's merge it. 😄


[Outlook] (n/a) - Minor update to trigger rebuild of doc set in OPS.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/addin-tutorial.md
:white_check_mark:Succeeded

docs/add-ins/toc.yml
:white_check_mark:Succeeded

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


More strict type for matrix multiplication

When using in DataFrame dot or @ pandas will return Series only when other is Series.
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.DataFrame.dot.html
Same can be told about Series. Result of Series @ Series is Scalar, Series @ DataFrame is Series, Series @ array is array.
https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.Series.dot.html
Reflected operands should also be included.

Hi @nagrael2,
Thanks for reporting the issue. You can see PR with the fix: https://github.com/VirtusLab/pandas-stubs/pull/132


fix auto_negotiation

https://github.com/open-traffic-generator/ixnetwork/blob/main/snappi_ixnetwork/snappi_convergence_api.py#L43
This exception on this line is misleading
The exception is raised for self._api.config_ixnetwork(config) in vport, can you check what is the best place to raise exception and catch the error


Publish to Vanity homepage shortcut and remove popup that needs to be dismissed.

Can you
A. Add a shortcut for publish to vanity homepage?
B. Avoid popup which obscures part of the window that then needs to be dismissed.
Example: if it were a small notification on the top bar, then we could ignore it if we want.

a) I didn't put a shortcut initially as it's not something that particularly happens often (i.e. there's only a few keyboard shortcuts we can actually use - so they better be used a lot).
b) You can hit esc to dismiss the noticies. It's a middle ground...


Download/Install instructions no longer match download page

The referenced downloads page has changed and the docs no longer reflect the way a user would go about installing InfluxDB2
https://portal.influxdata.com/downloads/ now uses exclusively repositories to install on Linux
Additionally links like https://dl.influxdata.com/influxdb/releases/influxdb2-2.2.0-amd64.rpm look to no longer exist and thus cannot be used for downloads.
Relevant URLs

https://docs.influxdata.com/influxdb/v2.2/install/

Additionally, we direct users to the downloads page to obtain the downloads url when we no longer list a download url, due to preferring repositories. I'm not sure how best to correct that, but I don't think we should be pointing them to that page to find a package to download anymore.

thank you, @imasoninflux @jdstrand, we'll update the filenames and verbiage to address
and cc you both and @bnpfeife for review


[CT-443] Potentially sensitive data can be written to logfiles on incremental model failure

Describe the bug
When a merge statement fails on Snowflake with a duplicate row, Snowflake will return the data from the row that failed in the format Duplicate row detected during DML action Row Values: [12345, "col_a_value", "col_b_value", "col_c_value", "etc"...]
Since dbt reports those errors back to the user via stdout and logs them in dbt.log when such an error occurs, data from the warehouse is written to disk, potentially multiple times if dbt is run from a process capturing stdout (like many orchestrators). If the incremental model processes sensitive data, this is a way for said sensitive data to find its way into logfiles, which is a pattern discouraged by OWASP
This error reporting behavior can be suppressed in Snowflake via the ERROR_ON_NONDETERMINISTIC_MERGE parameter but I don't really want to modify the behavior of the merge, just modify the logging behavior.
Logging this in dbt-snowflake instead of dbt-core because I think it's more of a quirk of Snowflake than it is a core dbt problem. dbt is just reporting the error returned from the DB, and I'm not sure if other DBs supported by dbt behave this way on a failed merge. The error message from Snowflake is helpful, but I think it's being too helpful here.
Steps To Reproduce

Create a incremental model:

-- fail_merge.sql
{{ config(
    materialized='incremental',
    unique_key='id'
) }}

with data as (
    SELECT $1 id, $2 name FROM (
        VALUES (1, 'one'), (2, 'two'), (3, 'three')
    )
)
select * from data

Run the incremental model once.

❯ dbt run -m fail_merge
Running with dbt=0.21.0
Found 231 models, 126 tests, 20 snapshots, 0 analyses, 188 macros, 0 operations, 0 seed files, 108 sources, 0 exposures

10:20:26 | Concurrency: 6 threads (target='dev_snowflake')
10:20:26 |
10:20:26 | 1 of 1 START incremental model analytics.fail_merge.................. [RUN]
10:20:28 | 1 of 1 OK created incremental model analytics.fail_merge............. [SUCCESS 1 in 1.69s]
10:20:28 |
10:20:28 | Finished running 1 incremental model in 9.08s.

Completed successfully

Modify your incremental model to return a duplicate row

-- fail_merge.sql after edit
{{ config(
    materialized='incremental',
    unique_key='id'
) }}

with data as (
    SELECT $1 id, $2 name FROM (
        VALUES (1, 'one'), (2, 'two'), (3, 'three'), (1, 'one')
    )
)
select * from data

Run your incremental model again and note the presence of the data that failed the merge in stdout and dbt.log

❯ dbt run -m fail_merge
Running with dbt=0.21.0
Found 231 models, 126 tests, 20 snapshots, 0 analyses, 188 macros, 0 operations, 0 seed files, 108 sources, 0 exposures

10:26:09 | Concurrency: 6 threads (target='dev_snowflake')
10:26:09 |
10:26:09 | 1 of 1 START incremental model analytics.fail_merge.................. [RUN]
10:26:12 | 1 of 1 ERROR creating incremental model analytics.fail_merge......... [ERROR in 3.22s]
10:26:12 |
10:26:12 | Finished running 1 incremental model in 10.86s.

Completed with 1 error and 0 warnings:

Database Error in model fail_merge (models/analytics/fail_merge.sql)
  100090 (42P18): Duplicate row detected during DML action
  Row Values: [1, "one"]
  compiled SQL at target/run/launchdarkly/models/analytics/fail_merge.sql

Done. PASS=0 WARN=0 ERROR=1 SKIP=0 TOTAL=1

❯ tail -n 8 logs/dbt.log
2022-03-31 17:26:13.020221 (MainThread): Database Error in model fail_merge (models/analytics/fail_merge.sql)
2022-03-31 17:26:13.020394 (MainThread):   100090 (42P18): Duplicate row detected during DML action
2022-03-31 17:26:13.020558 (MainThread):   Row Values: [1, "one"]
2022-03-31 17:26:13.020717 (MainThread):   compiled SQL at target/run/launchdarkly/models/analytics/fail_merge.sql
2022-03-31 17:26:13.020883 (MainThread):
Done. PASS=0 WARN=0 ERROR=1 SKIP=0 TOTAL=1
2022-03-31 17:26:13.021132 (MainThread): Sending event: {'category': 'dbt', 'action': 'invocation', 'label': 'end', 'context': [<snowplow_tracker.self_describing_json.SelfDescribingJson object at 0x11377be20>, <snowplow_tracker.self_describing_json.SelfDescribingJson object at 0x11377bfa0>, <snowplow_tracker.self_describing_json.SelfDescribingJson object at 0x11377b460>]}
2022-03-31 17:26:13.021412 (MainThread): Flushing usage events

Expected behavior
Contents of Row Values: [...] are not written to disk and/or logged to stdout. While the example above didn't include sensitive data, if the model processed sensitive data it would, and ideally that data in the warehouse should not be logged.
Screenshots and log output
See "Steps To Reproduce"
System information
The output of dbt --version:
❯ dbt --version
installed version: 0.21.0
   latest version: 1.0.0

Your version of dbt is out of date! You can find instructions for upgrading here:
https://docs.getdbt.com/docs/installation

Plugins:
  - snowflake: 0.21.0

The operating system you're using:
macOS Monterey 12.3
The output of python --version:
❯ python --version
Python 3.8.12

Additional context
Slack thread

@verhey thanks for logging this! My 5 min take is that I think we can handle this scenario by catching the presence of the error, the manipulating the message before logging it with the debugger where at least you could tell the user how many rows are duplicate? in my example below I'm not manipulating the string just giving a new string. hopefully this gives you a start?
dataders's super simple example
if "100090 (42P18): Duplicate row detected during DML action" in msg:
    logger.debug(" 100090 (42P18): Duplicate row detected during DML action!")

Not sure if you've contributed to dbt-core or dbt-snowflake before, but this guide is a great place to get started
similar error handling done previously
https://github.com/dbt-labs/dbt-snowflake/blob/184eaeb3fa65fb4090d811ea0fa6b7da3ea94574/dbt/adapters/snowflake/connections.py#L226-L227


ValidationError: All object keys must be strings

Hi
I have an endpoint with returns JSON:
{
  "results": [
    {
      "code": "string",
      "email": "string"
    }
  ]
}

I'm getting this error when I'm trying to create apistar client for this endpoint:
Code to reproduce
import apistar

schema = """
openapi: 3.0.0
info:
  title: Accounts
  version: 1.0.0
servers:
- url: https://api.com/example/1.0.0
paths:
  /users:
    get:
      tags:
      - Account
      responses:
        200:
          description: Get user details
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/users'
components:
  schemas:
    users:
      type: object
      properties:
        results:
          type: array
          items:
            type: object
            properties:
              code:
                type: string
              email:
                type: string

"""

client = apistar.Client(schema)
result = client.request('accounts')

Error
Traceback (most recent call last):
  File "/Users/piotr.py/Projects/example/.venv/lib/python3.7/site-packages/typesystem/tokenize/positional_validation.py", line 13, in validate_with_positions
    return validator.validate(token.value)
  File "/Users/piotr.py/Projects/example/.venv/lib/python3.7/site-packages/typesystem/fields.py", line 545, in validate
    raise ValidationError(messages=error_messages)
typesystem.base.ValidationError: {'paths': {'/users': {'get': {'responses': {200: 'All object keys must be strings.'}}}}}

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "example/example_bug.py", line 39, in <module>
    client = apistar.Client(schema)
  File "/Users/piotr.py/Projects/example/.venv/lib/python3.7/site-packages/apistar/client/client.py", line 22, in __init__
    self.document = apistar.validate(schema, format=format, encoding=encoding)
  File "/Users/piotr.py/Projects/example/.venv/lib/python3.7/site-packages/apistar/core.py", line 75, in validate
    value = typesystem.validate_with_positions(token=token, validator=validator)
  File "/Users/piotr.py/Projects/example/.venv/lib/python3.7/site-packages/typesystem/tokenize/positional_validation.py", line 36, in validate_with_positions
    raise ValidationError(messages=messages)
typesystem.base.ValidationError: {'paths': {'/users': {'get': {'responses': {200: 'All object keys must be strings.'}}}}}

Version
0.7.2

The openapi spec states that response codes need to be quoted in YAML (otherwise they're interpreted as ints, rather than string literals, which is what the spec expects.)

So: "200": oughta work fine.
We really will want so far nicer error messaging for this.

OAS3 YAML schema was generated by editor.swagger.io from valid swagger 2.0 JSON.

@tomchristie I replaced 200 with "200" but the client still doesn't work because of missing operationId in my schema. Consider adding a note about this.


bundle exec guard fails

Issue seems to be referenced here: https://www.gitmemory.com/issue/guard/guard/966/708039727
I am using Rails 4.2 - so next bet is to try older versions of guard, but I was hoping this would look familiar to someone:
22:14:04 - INFO - Bye bye...
/gems/gems/guard-2.17.0/lib/guard/jobs/pry_wrapper.rb:294:in `initialize': wrong number of arguments (given 3, expected 0) (ArgumentError)
	from /gems/gems/guard-2.17.0/lib/guard/jobs/pry_wrapper.rb:294:in `new'
	from /gems/gems/guard-2.17.0/lib/guard/jobs/pry_wrapper.rb:294:in `_configure_prompt'
	from /gems/gems/guard-2.17.0/lib/guard/jobs/pry_wrapper.rb:161:in `_setup'
	from /gems/gems/guard-2.17.0/lib/guard/jobs/pry_wrapper.rb:78:in `initialize'
	from /gems/gems/guard-2.17.0/lib/guard/interactor.rb:17:in `new'
	from /gems/gems/guard-2.17.0/lib/guard/interactor.rb:17:in `initialize'
	from /gems/gems/guard-2.17.0/lib/guard.rb:67:in `new'
	from /gems/gems/guard-2.17.0/lib/guard.rb:67:in `setup'
	from /gems/gems/guard-2.17.0/lib/guard/commander.rb:32:in `start'
	from /gems/gems/guard-2.17.0/lib/guard/cli/environments/valid.rb:16:in `start_guard'
	from /gems/gems/guard-2.17.0/lib/guard/cli.rb:122:in `start'
	from /gems/gems/thor-0.20.3/lib/thor/command.rb:27:in `run'
	from /gems/gems/thor-0.20.3/lib/thor/invocation.rb:126:in `invoke_command'
	from /gems/gems/thor-0.20.3/lib/thor.rb:387:in `dispatch'
	from /gems/gems/thor-0.20.3/lib/thor/base.rb:466:in `start'
	from /gems/gems/guard-2.17.0/lib/guard/aruba_adapter.rb:32:in `execute'
	from /gems/gems/guard-2.17.0/lib/guard/aruba_adapter.rb:19:in `execute!'
	from /gems/gems/guard-2.17.0/bin/_guard-core:11:in `<main>'

Probably a late reply, but I'm with a rails 4.2/ruby 2.4 project, using guard-rspec (4.7.3) and guard (2.18.1) with no problems.


Redcarpet::Render::Base.block_quote reads nil as its parameter

I'm trying to only render a blockquote of a markdown in HTML.
Here's my subclass:
  class ArticleCardHTMLRender < Redcarpet::Render::Base
    def postprocess(full_document)
      """

        #{full_document}

      """.html_safe
    end

    def block_quote(quote)
      puts "quote"
      puts quote
      puts "quote"
      """

        #{quote}
        placeholder

      """
    end
  end

When it renders, quote is not printed to the console.
When I replace Redcarpet::Render::Base with Redcarpet::Render::HTML, I get the expected output and see the following output in the console:
quote

Example quote.

quote

The issue doesn't happen with header(text, header_level).
I use the latest version: 3.5.0.

Hello,
Sorry for the late answer but actually the problem lies in the fact that Redcarpet::Render::Base doesn't implement any method by default ; nil is returned from every callback so you have to provide an implementation for each of them.
Thus, to fix your problem, you might have to do the following:
class ArticleCardHTMLRender < Redcarpet::Render::Base
  def paragraph(text)
    %(
#{text}
)
  end

  # ...
end

Or extends from Redcarpet::Render::HTML as you mentioned (it's generally easier to go this way).


Update to a single repo structure

This PR gets rid of the submodule junk and adds some required tooling for doing everything in a single repo. I totally mangled the 6.x versions with my experimentation (whoops), so after this ships, I will update to 7.0.0 which will be the new version going forward. This also updates the rules that we changed our minds on from the styleguide PR: prefer-const switches from 0 to 1, and init-declarations switches from 1 to 0.
cc/ @bouk sorry for the huge mess :P

Could you add a script/setup to run the required set up commands?
On Friday, 5 February 2016, Chris Sauvé<EMAIL_ADDRESS>wrote:

This PR gets rid of the submodule junk and adds some required tooling for
doing everything in a single repo. I totally mangled the 6.x versions with
my experimentation (whoops), so after this ships, I will update to 7.0.0
which will be the new version going forward.

--
Bouke

some small things but lgtm


Make storage of static data consistent

Description
Currently, the vast majority of static data stored in lance is in JSON format, but a few files are YAML or plain text. I propose that this all be turned into one format (JSON, and while I'm there make the JSON formatting consistent across all JSON files) so all static data is stored consistently.
Reasoning
This is a QoL improvement to make the bot more consistent, akin to the spring cleanup but without the huge amount of effort reviewing that took, instead this would be 2 commits:

Turning the non-JSON files to JSON and implementing the code changes to reflect that
Formatting each JSON file under bot/resources using Prettier

Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

Why JSON instead of YAML?

I'd argue we would be better off switching everything to Yaml or Toml as it is much more readable :P but I'm not even sure that is worth doing

In our resources folder we have 3 yaml/toml files (https://github.com/search?q=repo%3Apython-discord%2Fsir-lancebot+extension%3Ayaml+extension%3Atoml+path%3Abot%2Fresources) and 43 json files (https://github.com/search?q=repo%3Apython-discord%2Fsir-lancebot+extension%3Ajson+path%3Abot%2Fresources), so a change to all JSON would be much easier.
I prefer JSON for this sort of use case, as it's more similar to Python's syntax so it's more clear how it will translate into a Python object once json.loaded, and i'd say people are probably more likely to be familiar with JSON than yaml/toml.
That said, my opinion on whether this should actually be done isn't too strong either way, if somebody wants to do it I'd be fine with that though.


Configure the CronJob to use the worker image from Quay.io

Instead of referencing the image from the internal image stream,
directly reference the worker image in Quay.io (or any other registry
it might be stored in). Removing this indirection should ensure the
image to be updated each time the CronJob runs.
Signed-off-by: Hunor Csomortáni<EMAIL_ADDRESS>

Removing this indirection should ensure the image to be updated each time the CronJob runs.

For a (cron) job, the image stream indirection has not much sense, indeed. On the other hand, it should work with it as well if the image stream is configured (which it should be) with auto_import_images: true.

Yes, indeed, but given the incompatibilities of the CronJob API versions between OpenShift 3.X and 4.X, which break the update mechanism when using image streams in 4.X, and given that we use 3.X for testing but 4.X in production, this seemed to be the best way to keep the image up to date in future jobs, while also making the tests pass.


Hubot connecting and responding, but doesn't show as logged in

So at some point our robot stopped showing as logged in.  It responds to messages and works fine in pretty much every respect, but the OS X HipChat client shows it as offline, whereas the web client shows it as online and available.
Im wondering if this is some kind of HipChat client issue or maybe the Hubot module needs to be updated to account for some change in the API?

See https://github.com/hipchat/hubot-hipchat/issues/220 for details.


Selector issue crashes process instead of returning error

Hi, I'm using this very useful package to inline styles of websites I'm scraping with node, but I keep encountering this error every now and then:
/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/lib/inline.js:302
  if (selector.length === 0) {
               ^

TypeError: Cannot read property 'length' of undefined
    at getPseudoElementType (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/lib/inline.js:302:16)
    at handleRule (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/lib/inline.js:82:29)
    at Array.forEach (<anonymous>)
    at inlineDocument (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/lib/inline.js:34:9)
    at juiceDocument (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/lib/inline.js:332:3)
    at module.exports (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/lib/cheerio.js:61:22)
    at onInline (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/juice/index.js:75:7)
    at /Users/jonas/Projects/personal/web-element-grabber-full/node_modules/web-resource-inliner/src/html.js:267:9
    at /Users/jonas/Projects/personal/web-element-grabber-full/node_modules/async/dist/async.js:2955:19
    at wrapper (/Users/jonas/Projects/personal/web-element-grabber-full/node_modules/async/dist/async.js:268:20)

In principle that's not a problem, I assume the website being scraped has some malformed html, and I expect this to happen, however since this error is not caught and the process quits instead of the callback from juiceResources returning an error I can't properly handle the error.
I've tried additionally wrapping the call to juiceResources in a try/catch, but it did not catch the error.
Is there a way to have this error not quit the node process?

I believe #349 fixed this


Web Support

Tracking for fzyzcjy/flutter_convenient_test#283

Removed the goldens code from convenient test dev because it doesn't compile on Web. This is because it genuinely isn't supported by the underlying flutter test. Eventually this should be split out into conditional compilation that is platform dependent.

Due to flutter/flutter#111136 not yet being in stable development has to move to the master branch for now.
Further thoughts - the current supported approach for integration tests in flutter web uses chromedriver / whatever other driver depending on browser -> potentially would need to package that within the tool.
Even after doing this there seems to be internal errors in flutter that stop this from even loading a basic app. Until/if flutter gets a way to run integration tests on web using flutter run/test rather than flutter drive I don't think this will move anywhere.

Refer to flutter case #111136 , any alternative way to continue the web development by httpcllient
thanks


Part 1 step 2 of Hands on mentions missing File Name Prefix template requirement

It seems the File Name Prefix requirement in the Basic Execution Template was removed as mentioned here:
https://github.com/LabVIEW-DCAF/DCAF-Project-Templates/issues/32
However the Hands On documentation instructs to add a prefix (TCRL) which cannot be done in the newest template. See Part 1 step 2 of Hands on.

We will update this when we are doing the movements and changes mentioned in issues 76 to 78


Added participant list if you are not attending. Added status to show…

…ings

Solves #92 and #75


Extend ThreadVariablesContext To Support Stacks

Given that nested contexts are supported in a range of different logging components would it be possible to extend the ThreadVariablesContext member on ILog to include support for the underlying stack operations so that NDC-style contexts can be supported through Common.Logging?
These new members would then map to equivalent members on ThreadContext.Stacks["NDC"] for log4net, NestedDiagnosticsContext for NLog, LogContext for Serilog, and so on, to provide common NDC functionality across logger components.

Interesting suggestion; do you have any thoughts about what this support might look like?

Initital thoughts were an additional or derived interface with the stack operations on that are utilised by NDCs.  The type used for ThreadVariablesContext would then implement this and map those operations onto the corresponding members of the actual loggers.
I'll fork and try it out first and issue a PR if all well and good, if that's ok?

Yes, that would be great; let me know if you run into any issues with which we can assist!

Pull Request created: #129

Additional Pull Request created: #131

PR https://github.com/net-commons/common-logging/pull/131 merged; thanks for the contribution!


optimization

https://developers.google.com/speed/pagespeed/insights/?url=chlambase.org&tab=desktop

Integrated minification pipeline with grunt. Run "grunt minify" to update the code in /build.
Things left to do to increase performance further:

concatenate the files into a single js file (to reduce the number of get calls)
minify html (might not be necessary with g zip compression)
Figure out bug with dependency injection and uglify plugin to  allow mangling variable names (tried ng annotate...)

Fixed problem with variable mangling (wrong order of commands..). Now, variable names will be mangled to further compress the files in chlambase for even faster speed! Yay

Fixed file concatenation. Problem was that angular requires a specific ordering (which I had not specified).

Integrated HTML minifier in pipeline. Now, everything is minified and compressed as small as possible. While its certainly possible to make more improvements in the future, for now it is at a very good point and will mark as resolved


Remove custom apt source

Matches .travis.yml used in other repos

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


Ruby 2.6.3 ❤

Latest and Greatest Ruby!

Coverage decreased (-0.007%) to 97.917% when pulling bd0e27cf283d113dd58ca673aae1eaf31089c007 on tonyta:ruby-2.6.3 into 945a0838da747370d03d49580ef14ffb25aa29b9 on sfbrigade:master.

Thanks for help to keep the project up-to-date!


Add events filter

Use /v2/events?only=EVENT1,EVENT2,.. to filter events.

maybe instead of only we call the filter event_name?

maybe instead of only we call the filter event_name? in case we want to add more filters later

done!

@ouziel-slama we also need a line in the release notes!


chore(main): release volcano-data-deconvolution 1.0.4

:robot: I have created a release beep boop
1.0.4 (2024-06-03)
Miscellaneous

zenodo: try specifying affiliation string as zenodo show it (d8bd9ae)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/engeir/volcano-data-deconvolution/releases/tag/volcano-data-deconvolution-v1.0.4 :sunflower:


Do not call strlen() on null

Fixes #656.

Just a suggestion to solve this issue less invasive (better maintainability) on the original code  https://github.com/justinrainbow/json-schema/blob/6d04026ebc25e8ea0594a345eaa387f2ccb3b3e5/src/JsonSchema/Constraints/Constraint.php#L49
array_filter((array)$i, 'strlen')

and no other change (maybe a space after the cast operator depending on code-style).
and while touching this area, for the calling function https://github.com/justinrainbow/json-schema/blob/f4f0c3457811006f861c466a10c6c24f3e92f9ae/src/JsonSchema/Constraints/UndefinedConstraint.php#L40
$path = $this->incrementPath($path, $i);

as $path param is null-able. but that could be on a different commit, perhaps earlier already in master, and must not go into this PR.
the build is choking on the php-cs-fixer no_superfluous_phpdoc_tags rule, I recommend to remove it from configuration as it is incompatible with the code-style.
just my 2 cents.

There's duplicate https://github.com/justinrainbow/json-schema/pull/666


cmd/cue: trim bug: conjunctions and definitions

Originally opened by @vikstrous2 in https://github.com/cuelang/cue/issues/1062
What version of CUE are you using (cue version)?

$ cue version
cue version v0.4.0 linux/amd64

Does this issue reproduce with the latest release?
yes, also tested with master
What did you do?
#def2: {
	in2: string
}
#def1: {
	in1: string
	out: #def2 & {
		in2: in1
	}
}
a: #def1 & {
    in1: "test"
}

This is as much as I was able to simplify the bug. There might be a bit more it could be simplified though. Inlining #def1 or #def2 seems to "fix" it.
What did you expect to see?
probably no change
What did you see instead?
  {
      #def2: {
        in2: string
      }
      #def1: {
        in1: string
        out: {
                in2: string
        }
      }
      a: {
          in1: "test"
          out: {
-             in2: "test"
+             in2: string
          }
      }
  }
Aborting trim, output differs after trimming. This is a bug! Use -i to force trim.
You can file a bug here: https://github.com/cuelang/cue/issues/new?assignees=&labels=NeedsInvestigation&template=bug_report.md&title=

Thanks for raising this @vikstrous2.
Confirmed with ecb17c94:
exec cue trim x.cue

-- x.cue --
#def2: {
	in2: string
}
#def1: {
	in1: string
	out: #def2 & {
		in2: in1
	}
}
a: #def1 & {
	in1: "test"
}

gives:
> exec cue trim x.cue
[stdout]
  {
      #def2: {
        in2: string
      }
      #def1: {
        in1: string
        out: {
                in2: string
        }
      }
      a: {
          in1: "test"
          out: {
-             in2: "test"
+             in2: string
          }
      }
  }
Aborting trim, output differs after trimming. This is a bug! Use -i to force trim.
You can file a bug here: https://cuelang.org/issues/new?assignees=&labels=NeedsInvestigation&template=bug_report.md&title=
[exit status 1]
FAIL: /tmp/testscript915350260/repro.txt/script.txt:1: unexpected command failure

See to what extent this is/isn't related to https://github.com/cue-lang/cue/issues/1298

Hi All,
Thanks for cuelang. Its a great piece of software. I have come up against this. My example is below - it will error with a non concrete value for repo
#task: { 	// Add a repo to helm 	addRepo: { 		repo: { 			name: string 			url:  string 		} 		cmd: "helm repo add \(repo.name) \(repo.url)" 	} } output: { 	repo: { 		name: "dummy" 		url:  "dummy.com" 	} 	mergedTask: #task.addRepo & {repo: repo} }
It seems that, the definition of 'repo' in the context where it is conjuncted (second to last line) is superceeded by the definition of 'repo' in the #task.addRepo block.
A work around is to quote the key when conjuncting.
#task: { 	// Add a repo to helm 	addRepo: { 		repo: { 			name: string 			url:  string 		} 		cmd: "helm repo add \(repo.name) \(repo.url)" 	} } output: { 	repo: { 		name: "dummy" 		url:  "dummy.com" 	} 	mergedTask: #task.addRepo & {"repo": repo} }
Which will return output as expected

Possibly better title - "Conjuncting schema, with a struct nested under a key, results in non concrete schema values shadowing concrete values"

Hi All,
Thanks for cuelang. Its a great piece of software. I have come up against this. My example (minimized for clarity) is below - it will error with a non concrete value for repo
#task: {
	// Add a repo to helm
	addRepo: {
		repo: {
			name: string
			url:  string
		}
		cmd: "helm repo add \(repo.name) \(repo.url)"
	}
}
output: {
	repo: {
		name: "dummy"
		url:  "dummy.com"
	}
	mergedTask: #task.addRepo & {repo: repo}
}

It seems that, the definition of 'repo' in the context where it is conjuncted (second to last line) is shadowed by the definition of 'repo' in the #task.addRepo block (where this shadowing, unwanted definition is now in nearest enclosing block)
A work around is to quote the key when conjuncting.
#task: {
	// Add a repo to helm
	addRepo: {
		repo: {
			name: string
			url:  string
		}
		cmd: "helm repo add \(repo.name) \(repo.url)"
	}
}
output: {
	repo: {
		name: "dummy"
		url:  "dummy.com"
	}
	mergedTask: #task.addRepo & {"repo": repo}
}

Which will return concrete output as expected

A further comment. I am assuming this is the same (possibly) bug - or if not my misunderstanding. However i am not running fix/trim. I am running cue eval or cue export


[fix] [broker] In Key_Shared mode: remove unnecessary mechanisms of message skip to avoid unnecessary consumption stuck

Motivation

https://github.com/apache/pulsar/pull/7105 provide a mechanism to avoid a stuck consumer affecting the consumption of other consumers:

if all consumers can not accept more messages, stop delivering messages to the client.
if one consumer can not accept more messages, just read new messages and deliver them to other consumers.

https://github.com/apache/pulsar/pull/7553 provide a mechanism to fix the issue of lost order of consumption: If the consumer cannot accept any more messages, skip the consumer for the next round of message delivery because there may be messages with the same key in the replay queue.
https://github.com/apache/pulsar/pull/10762 provide a mechanism to fix the issue of lost order of consumption: If there have any messages with the same key in the replay queue, do not deliver the new messages to this consumer.

https://github.com/apache/pulsar/pull/10762 and https://github.com/apache/pulsar/pull/7553 do the same thing and https://github.com/apache/pulsar/pull/10762 is better than https://github.com/apache/pulsar/pull/7553 , so https://github.com/apache/pulsar/pull/7553 is unnecessary.
Modifications
remove the mechanism provided by https://github.com/apache/pulsar/pull/7553 to avoid unnecessary consumption stuck.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

x

@equanz

By the way, if you could, please check the new comments https://github.com/apache/pulsar/pull/20179#issuecomment-1543445903 ?

Sorry, I've been so busy lately. I left a comment to you https://github.com/apache/pulsar/pull/20179#discussion_r1198087967

@poorbarcode just curious to know if the logic that was removed in this PR was causing problems. Is this PR an optimization or a bug fix?

@lhotari

Is this PR an optimization or a bug fix?

I think it should be called a fix.

just curious to know if the logic that was removed in this PR was causing problems.

If there have two consumers [c1, c2] on the same subscription, and there have some messages in the Replay Queue, and the c1 was stuck by the client and its permits came to 0, the consumer c2 will not receive any messages in the Replay Queue, it caused by the nextStuckConsumers is not empty(c1), then Dispatcher will try to read new messages, but there are no new messages now.

@codelipenghui @massakam do you have a chance to do a post-merge review since you have worked a lot on Key_Shared?

@lhotari It looks good to me https://github.com/apache/pulsar/pull/10762 covered the fix that https://github.com/apache/pulsar/pull/7553 do and this PR doesn't revert the tests introduced by https://github.com/apache/pulsar/pull/7553.

@lhotari It looks good to me #10762 covered the fix that #7553 do and this PR doesn't revert the tests introduced by #7553.

@codelipenghui @poorbarcode the part I'm wondering is the lack of tests for reproducing the problem that this PR intends to fix. Is there a way to add a test for this PR? /cc @Technoboy-

@lhotari

the part I'm wondering is the lack of tests for reproducing the problem that this PR intends to fix. Is there a way to add a test for this PR?

I will do this


Fixed documentation install path

Signed-off-by: Nick Avramoussis<EMAIL_ADDRESS>

I realize this is such a minor change, I just rolled this into my documentation PR - #944

@danrbailey Thanks! I would prefer though that, where possible, we merge PRs that don't require any changes i.e. this->master->your branch.

@danrbailey Thanks! I would prefer though that, where possible, we merge PRs that don't require any changes i.e. this->master->your branch.

Yes, I agree. I would prefer that we use pendingchanges instead of modifying the change notes directly. In this case, I brought the few lines across myself to avoid having to resolve any conflicts.


[Snyk] Upgrade: org.apache.commons:commons-lang3, org.webjars:webjars-locator-core

Snyk has created this PR to upgrade multiple dependencies.
👯♂ The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

org.apache.commons:commons-lang3from 3.15.0 to 3.16.0 | 1 version ahead of your current version | 2 months agoon 2024-08-01
org.webjars:webjars-locator-corefrom 0.58 to 0.59 | 1 version ahead of your current version | 3 months agoon 2024-06-08

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Good catch.  Looks like we need a PR to correct the PR.
On Fri, Oct 4, 2024 at 8:19 AM Frank Duckart @.***>
wrote:

@.**** commented on this pull request.
In pom.xml
https://urldefense.com/v3/__https://github.com/uhawaii-system-its-ti-iam/uh-groupings-ui/pull/1028*discussion_r1788123879__;Iw!!PvDODwlR4mBZyAb0!X-nqnSaKjImCzbLvJFUMOqK-p38s-rL7xp-UcBKArhpCcAByqoh0TEdmn4hVExa5qud41CE7k3he0vdqvJ8-rBQewg$
:

@@ -229,7 +229,7 @@
3.0-beta-02

                   <source>classpath/META-INF/resources/webjars/angularjs/${angularjs-version}/angular.min.js">

                   <source>classpath/META-INF/resources/webjars/angularjs/${angularjs-version}/angular.min.js">

I realize this PR is closed, but it's interesting that this entity
substitution for the greater-than symbol didn't break the pom. Hmm.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/uhawaii-system-its-ti-iam/uh-groupings-ui/pull/1028*pullrequestreview-2348835878__;Iw!!PvDODwlR4mBZyAb0!X-nqnSaKjImCzbLvJFUMOqK-p38s-rL7xp-UcBKArhpCcAByqoh0TEdmn4hVExa5qud41CE7k3he0vdqvJ8TnjKBlg$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ACX2CMEYI5WDB2JLG3EMNB3ZZ3L4PAVCNFSM6AAAAABOUNS6TCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNBYHAZTKOBXHA__;!!PvDODwlR4mBZyAb0!X-nqnSaKjImCzbLvJFUMOqK-p38s-rL7xp-UcBKArhpCcAByqoh0TEdmn4hVExa5qud41CE7k3he0vdqvJ_4l1oIiA$
.
You are receiving this because you are subscribed to this thread.Message
ID: <uhawaii-system-its-ti-iam/uh-groupings-ui/pull/1028/review/2348835878
@github.com>

The PR is closed but it was not merged. Here's PR to remove the sign from POM: https://github.com/uhawaii-system-its-ti-iam/uh-groupings-ui/pull/1038/files


Support parsing messages with missing space after colon

This makes the parser more robust to some malformed legacy messages, such as in #28.
This builds on top of #28.

I've pulled this PR and did a quick test with one of my applications, it seems to work as intended.
The problem with missing space after colon ( #28 and #29 ) is no longer an issue.

Thanks for the confirmation @bonan, let's get this in :shipit:

awesome. thanks.


update index.js

there is no need of an extra module (http) to create server , it comes with app instance

thx


Fix STI-related bug in validates_timeliness exposed in Rails 3.2

This pull request addresses a previously reported issue: https://github.com/adzap/validates_timeliness/issues/88
The cause is quite complicated, but I'll try to keep my explanation brief.
The issue is exposed by changes in ActiveRecord::AttributeMethods#define_attribute_methods between rails 3.1 and 3.2.  This method now calls itself on an AR subclass's superclass.  The effect of this is that ValidatesTimeliness::AttributeMethods#define_attribute_methods is called on all levels of an STI hierarchy.  Hence, define_timeliness_methods is called on all levels of an STI hierarchy.  And herein lies the problem:
module ValidatesTimeliness
  module AttributeMethods
    extend ActiveSupport::Concern

    included do
      class_attribute :timeliness_validated_attributes
      self.timeliness_validated_attributes = []
    end
    ...

end

As you can see above, timeliness_validated_attributes is defined as a class_attribute, which means that any subclasses will "inherit" a copy of it, including the values added to it by the superclass.  For instance,
class Author < ActiveRecord::Base
  validates_date :published_on
  validates_date :famous_on
end

class BadAuthor < Author
  validates_date :died_on
end

Loading development environment (Rails 3.2.14)
1.9.3-p448 :001 > Author.timeliness_validated_attributes
 => [:published_on, :famous_on]
1.9.3-p448 :002 > BadAuthor.timeliness_validated_attributes
 => [:published_on, :famous_on, :died_on]

Validates_timeliness generates "#{attr_name}=" methods for each of the attributes in a class's timeliness_validated_attributes. The methods are defined in an anonymous module which is inserted into the corresponding class's ancestor chain.
In the context of our STI example, this effectively means that the same "published_on=" method is defined twice in two separate modules in BadAuthor's heirarchy.  Via super, these calls are chained together with the result of the first being the argument to the next.  When you combine this with :use_plugin_parser set to true, it messes up the @timeliness_cache and, hence, the published_on_before_type_cast, and, hence, the validation of an invalid date...
The solution is to use a class level instance variable instead of a class_attribute. This way, each variable with a validates_date declaration (or validates_time, etc.) is only tracked and acted upon once via the class it is declared in.
Here is the change:
module ValidatesTimeliness
  module AttributeMethods
    extend ActiveSupport::Concern

    included do
      class << self
        attr_accessor :timeliness_validated_attributes
      end
      self.timeliness_validated_attributes = []
    end
    ...

end

And now things look right:
Loading development environment (Rails 3.2.14)
1.9.3-p448 :001 > Author.timeliness_validated_attributes
 => [:published_on, :famous_on]
1.9.3-p448 :002 > BadAuthor.timeliness_validated_attributes
 => [:died_on]

Phew!  Long explanation for such a small change :)

Getting a spec for this would be great. I have released v4 of the gem for Rails 4.x. So things are slightly different, though the definition of the timeliness_validated_attributes is the same.

I was able to reproduce the issue on Rails 5.2 with validates_timeliness 5.0.0.alpha2. What I found is that BadAuthor still outputs all fields as part of its timeliness_validates_attributes, however, the bug making it such that it doesn't property validate the parent class's attributes (#88) no longer appears to be an issue.
2.3.3 :004 > BadAuthor.timeliness_validated_attributes
 => [:published_on, :famous_on, :died_on]

I wonder if this PR is no longer needed.

@bboe great thanks for this. BadAuthor should inherit all of the parent's validated attributes and its validations. So I don't think that is part of the bug. The bug from reading is not that the child has the parent's validation, that is desired, but that it gets messed up when validating.
So if you have tested it,  I have happy to close this.


Add Docker container

We currently have a devcontainer to support development.
To allow for easy testing by users, we should supply a docker image.

That should partially address #78


take advantage of josso service to know whether logged in

this is a service you can query to tell whether you're logged in...but you need a josso ID. once you've acquired a session, you can use this service to send in the session object and see whether it's still active

catalog/jossoHelper/sessionInfo?includeJossoSessionID=true&josso=xxxxxxxxx
(but includeJossoSessionID is optional, and josso=xxxx may also be optional since that will be sent with the session info anyway)

you can also keep a session alive by hitting this lightweight status (or the josso helper) every 10 minutes: catalog/status?format=json. it returns a status number. 0 is good, 1 is warning, 2 is bad

https://www.sciencebase.gov/catalog/jossoHelper/sessionInfo?includeJossoSessionId=true

hitting this counts as a session refresh: https://www.sciencebase.gov/catalog/status?format=json

Doesn't #81 close this ?

I think the Josso service would have been a way using an additional server round-trip to verify status. Let's see how our session refresh counter works in the wild. Could always add the round trip later if we still have issues.


⚠ ClamAV has degraded performance

In 472ec70, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 197 ms

Resolved: ClamAV performance has improved in 4267d5a after 14 minutes.


Searching with ctrl+F for collapsed parameter or text is hard

Is your feature request related to a problem? Please describe.
Sorry for creating many issues, they are by no means urgent, just nice to have.
When searching with ctrl+F for e.g. parameters that are collapsed in the doc (for example problem_type in https://huggingface.co/docs/transformers/main_classes/configuration), browsers (Firefox, Chromium) redirect to a place that is not relevant. I think it redirects to the place where the parameter would be if not collapsed (after clicking on "Expand 11 parameters", it is the right place).
This makes it hard to search collapsed parameters with ctrl+F in the doc.
Describe the solution you'd like
Not sure, probably redirect to the place where we have "Expand 11 parameters"? Not sure how doable that is.

Thanks for reporting! cc @mishig25 as this is really related to documentation. For documentation issues, we also have https://github.com/huggingface/doc-builder FYI, I'm transferring this issue


this.captcha.getResponse() Fail to obtain Token

I try to get the token, but I do not, which is the best way?
This is mi code:
Component HTML
<re-captcha (captchaResponse)="handleCorrectCaptcha($event)" site_key="key" size="invisible"></re-captcha>
component:
` export class LoginComponent implements OnInit {
private auth: any;
@ViewChild(ReCaptchaComponent) captcha: ReCaptchaComponent;
constructor(private authService : AuthService,  private au :AngularFireAuth, private router : Router, private component: ReCaptchaComponent)
{
this.auth = firebase.auth();
}
ngOnInit() {
}
loginUser(value: any)
{
    let token = this.handleCorrectCaptcha(event);
    let promise = this.authService.login(value.email, value.password, token);

}
recoverPass(mailAddres: any){
    this.auth.sendPasswordResetEmail(mailAddres.email).then( response => {
        alert('An email has been sent to your account to retrieve your password');
        console.log('Se envio un correo a su cuenta.',response);
    }).catch( error => {
       alert('');
       console.log('error', error);
    });

}
handleCorrectCaptcha(event)
{
console.log(event);
//this.captcha = event.getResponse(); console.log(event.getResponse());
} `
Thanks for helpme

The token is there, you are using it incorrectly.  token should be a class variable and populated in handleCorrectCapctha().


Arg for depth cache freq added

Argument added to start_depth_socket to change the update frequency according to the official doc.
This resolved #487. There's also #435, but that was forked from the feature/asyncio branch and seems to be stale.

This is excellent, thanks! @vakker should the default in the case that no param is given be 1000? A default of 1000 would maintain the same behavior as the current version.

@OpenCoderX good point, fixed

@sammchardy Is there anything we can help with that would expedite the next release?

please resolve conflict, and i will merge

please resolve conflict, and i will merge

But the interval parameter is already there, so I'm not sure what's the point in resolving this conflict and merging this PR at this point.

But the interval parameter is already there, so I'm not sure what's the point in resolving this conflict and merging this PR at this point.

thank you @vakker another pr was merged. i think the assert was a nice touch!

thank you @vakker another pr was merged. i think the assert was a nice touch!


E-commerce - get for product variant is not working

When trying to get variant I get error:
405 Client Error: Method Not Allowed for url: https://us16.api.mailchimp.com/3.0/ecommerce/stores/store/products/1/variants/1
I've checked and this is caused by sending POST instead of GET:
return self._mc_client._post(
      url=self._build_path(store_id, 'products', product_id, 'variants', variant_id),
      **kwargs
)

Is that expected behaviour?

Hello @szewczykmira that is an error. Looking at that endpoint, we also have an error with the all() endpoint for the same reason. I have patched this on the master branch, and it should be on PyPi in a bit.
@charlesthk, Please put 2.0.17 up on PyPi, thanks

Hi @stephenross, thanks, the release is on PyPi !

@stephenross @charlesthk Thanks for quick reaction!


Implement MarbleTesting.ColdObservable.

Also: Start refactoring how subscriptions and scheduling are implemented.

Coverage increased (+3.7%) to 87.879% when pulling 48171db53ec6dde4a4311b407d2635f5f674bf98 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+3.7%) to 87.879% when pulling 48171db53ec6dde4a4311b407d2635f5f674bf98 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+3.7%) to 87.879% when pulling 48171db53ec6dde4a4311b407d2635f5f674bf98 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+3.7%) to 87.879% when pulling 48171db53ec6dde4a4311b407d2635f5f674bf98 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.3%) to 88.485% when pulling 9c8cd4a40eb62fdb37917a9ba704d06014636ec8 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.3%) to 88.485% when pulling bc646da9e7fa23ec1e855c69c14348fba1a9d55e on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.3%) to 88.485% when pulling 158c89f2d6028538271878bc9382766adff7ccb3 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.2%) to 88.415% when pulling 158c89f2d6028538271878bc9382766adff7ccb3 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.2%) to 88.415% when pulling 158c89f2d6028538271878bc9382766adff7ccb3 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.3%) to 88.485% when pulling 945b2c9f19fee99663403a214dd9066685fe6eb4 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.3%) to 88.485% when pulling 945b2c9f19fee99663403a214dd9066685fe6eb4 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.

Coverage increased (+4.3%) to 88.485% when pulling 945b2c9f19fee99663403a214dd9066685fe6eb4 on cold-observable into efd446b057655656d2966f7862bef1184e6ac112 on master.


Updates to Registry Schema Classes

Description
Some issues were noted last week with the registry YAML schema classes. This PR addresses those issues and adds a couple extra unit tests to cover them:

The container_registry field only accepted user-created ACR accounts (aka a single string input). It also needs to accept a system-created-acr account. At the moment, the only valid value for a system-created ACR account user-assigned value is 'premium'.
Both the existing system created storage account and the new system created acr account objects should have a dump only value for their resource id.

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


CyberSecurity vulnerabilities

This plugin works great, but sadly it's a no-go for us due to security concerns.
Black Duck Binary Analysis reports critical and high severity vulnerabilities against nghttp2 1.3.6 (CVE-2015-8659 and CVE-2020-11080). Need to upgrade to 1.6.0 or later to address these. Latest version is 1.59.0.
Also OpenSSL 1.1.1 is out-of-support. Should be upgraded to 3.0 or 3.2.

The latest NScurl release is built with nghttp2/1.58.0
I'm not sure why Black Duck is reporting nghttp2 version 1.3.6, but it's obviously inaccurate.
As for openssl 1.1.1 I'm aware of its EOL and it's on my todo list to upgrade
The main challenge for me is to build a binary that's backward compatible with ancient Windows builds (talking about NT 4, Win 2000...)
I've had a few unsuccessful attempts to switch to 3.x but I'm not there yet.

Thanks for the quick response.
In all likelihood BDBA reports the wrong version. I'll manually update the version number in BDBA which should get rid of the reported vulnerabilities. The outdated version of openssl I can live with for now.
Thank you again for creating and maintaining this great NSIS plugin.

FYI: openssl was upgraded ro version 3.3.0 in nscurl/1.2024.4.30
Enjoy!


[Bug]: ARK Survival Ascended - Large Backups

Panel Version
1.0.2
Wings Version
1.11.8
Service
games-steamcmd/ark_survival_ascended
Modified
No, I did not modify the egg
Expected Behavior
A default ".pteroignore" that only saves the configuration and game files.
This is my ignore pattern for the official servers:
*
!ShooterGame/Saved/Config/WindowsServer
!ShooterGame/Saved/SavedArks/*_WP/*.arkprofile
!ShooterGame/Saved/SavedArks/*_WP/*.arktribe
!ShooterGame/Saved/SavedArks/*_WP/*_WP.ark

Actual Behavior
Backups are huge as they include the entire runtime.
Steps To Reproduce
Install the ASA egg
Install logs
https://pteropaste.com/CommandOutputsPasswordsSoNo

a panel version of 1.0.2?
And this is not an egg issue. As backups are handled by the panel and wings not the egg

Hi @QuintenQVD0. the backups are indeed handled by the panel, but the eggs can provide default files like ".pteroignore". The server can't know what to backup or ignore in every egg, that would be impossible.

At the very least I think is should at least be:
*
!ShooterGame/Saved


Cannot get to the COUCHDB repo on RHEL 7-8 or Rocky 8

Hello,
I cannot update COUCHDB as I cannot get to the repo and I have tried the url from many different networks.
http://apache.bintray.com/couchdb-rpm/el7/x86_64/repodata/repomd.xml: [Errno 14] HTTP Error 502 - Bad Gateway
Any information would be greatly appreciated.
Thanks

apache.bintray.com is decommissioned (all of bintray is I think).
see https://docs.couchdb.org/en/3.2.2-docs/install/unix.html#enabling-the-apache-couchdb-package-repository


How can I automatically run the subsequent tasks of my workflow

Hi,
Apologies for my novice question but I just need some guidance, I'm conducting a proof of concept for the Conductor Framework so that if applicable we can use it on our project.
Currently, I created two workflows - one for sequential flow and the other is for parallel flow.
Sequential:

Parallel:

What I want to achieve is that, when I call/run the service_a for Sequential Flow, all the other task in the flow will run ->In Progress -> Completed. Same with Parallel flow, once I run the 3 task - service a, c & e all other task within that workflow should start to run, not just scheduled. Is that possible? Thanks.

yes, as of now it's of type - SIMPLE, I do have running worker which calls/poll the service_a, I also understand that I can use the poll api to call service_a. What I would like to know /understand is if there's a way for me to only call/poll service_a and all other task within the workflow will poll/start once the first task is completed.
Also, I'm not that sure if I'm understanding the concept itself or what will be the approach to take for me to integrate the Conductor with the existing application, need some guidance. Thanks.

Hi @JMBalares11 ,
If I understand it correctly. You will have to have running workers for the all the tasks. The workflow server will schedule the tasks according to the workflow definition. For instance, if it is sequential, it will only start once the last task is completed. Also, for parallel tasks, workers can poll them in parallel.
 You can try out the HTTPTask which runs on the server.

Ohh, so in every task in my workflow, I need to have a corresponding worker? What I'm visualizing right now is that, you can poll the task by either creating a worker - Client component, or using the poll api, then once you poll a task and set it to completed, the next task will automatically start, and so on with the other task - SCHEDULED -> IN PROGRESS(poll by worker or api) -> COMPLETED.
I do have a bunch of microservices to orchestrate and it's a bit shady for me on what approach to do. I have tried the HTTP task to call a simple service, say for instance after polling the service_a, it will call the simple service, return a success and set the task status to COMPLETED. The next task will be automatically set to "SCHEDULED" and need another trigger to start the event/task on the workflow. What I want is that, all my tasks will have an HTTP task (1 task, 1 microservice call), then once first task is completed, the other task will automatically set to IN PROGRESS.

The worker also intern calls the poll api. We will need different workers at least in the Java client. It has a separate thread pool for each worker. In the TaskCoordinator you can choose to create multiple workers in the same process.
Also,  the next task will start automatically. For instance  :

Once the HTTP task completes it automatically triggers the WAIT task and moves to IN_PROGRESS.

Closing this issue since it has already been answered. Thanks @mudit3774 for the response.


fasta: does not report runtime error when final field initialize twice (initializer and constructor)

Language/Classes/Constructors/Generative_Constructors/execution_of_an_initializer_t02.dart
Language/Classes/Constructors/Generative_Constructors/initializing_formals_execution_t02.dart
Language/Variables/final_t01/01
Language/Variables/final_t02/01

/cc #28263

Still the issue with fasta.

I have a local fix for this.

Fixed in 470127ae46ca783f7d625010c028073464cdb179.


Cannot create MosaicGlobalRestriction with referenceMosaicId

1. Expected behavior and actual behavior.
I'm having problems when creating a mosaic global restriction with a referenceMosaicId, getting the error Failure_RestrictionMosaic_Previous_Value_Must_Be_Zero (see 2. Steps to reproduce the problem).
The issue might not be in the SDK, but in the catbuffer serialization or server. Please feel free to escalate the issue to another repo if someone finds the root of the problem. Even though, we could start by adding an e2e test.
2. Steps to reproduce the problem.
Assuming  key = randUInt64Value

Create a new restrictable mosaic M1 with account A
Announce with A a MosaicGlobalRestriction TX1:

Property
Value

mosaicId
M1

restrictionKey
key

previousRestrictionType
None

previousRestrictionValue
0

newRestrictionType
EQ

newRestrictionValue
1

TX1 gets confirmed
Create a new restrictable M2 mosaic M2  with account B
Announce with B a MosaicGlobalRestriction TX2

Property
Value

mosaicId
M2

referenceMosaicId
M1

restrictionKey
key

previousRestrictionType
None

previousRestrictionValue
0

newRestrictionType
EQ

newRestrictionValue
2

TX2 does not get confirmed, raising the error 0x80510003 - Failure_RestrictionMosaic_Previous_Value_Must_Be_Zero

You can also find here the code used (steps 1-3).

Specifications like the version of the project, operating system, or hardware.

<EMAIL_ADDRESS>c2 <IP_ADDRESS>

Tested got the same error here.
Tried announced 2 global restriction transactions here

ReferenceMosaic (M1) -- successfully announced

      MosaicId: M1
      Key: UInt64.fromUint(60641),
      PrevValue: UInt64.fromUint(0),
      PrevType: MosaicRestrictionType.NONE,
      NewValue: UInt64.fromUint(0),
      NewType: MosaicRestrictionType.GE,

Another Mosaic (M2) -- Error Failure_RestrictionMosaic_Previous_Value_Must_Be_Zero

      MosaicId: M2
      Key: UInt64.fromUint(60641),
      PrevValue: UInt64.fromUint(0),
      PrevType: MosaicRestrictionType.NONE,
      NewValue: UInt64.fromUint(1),
      NewType: MosaicRestrictionType.GE,
      ReferenceMosaicId: M1

@dgarcia360 @Jaguar0625

under investigation by core


Flash with links show as a escaped string when redirect

In the following example some action calls the before_filter_that_redirects  this assigns a flash message that has a link and redirects to show. Then show assigns other flash message with another link (to google in this example). When they are shown the :error message shows as Message while the :notice message shows as Google with a working link
def show
  flash[:notice] = "#{view_context.link_to "Google", "www.google.com"}".html_safe
end

def before_filter_that_redirects
  flash[:error] = "#{view_context.link_to "Message", "www.facebook.com"}".html_safe
  redirect_to show_path
end

Interesting.  I just looked at the source, and it appears that it should only be escaping if the string is not marked html safe.  So I am puzzled by this.  If you want to open the gem up and find the line number where the bad escaping happens, it would help me a lot, otherwise I'll get to it when I can.  This gem isn't very actively maintained anymore, due to lack of time on my part (and Pivotal handed it off to me a long time ago).


Data retrieval in form of JSON instead of YAML

YAML made some problems already, when it once came with keys in form of a string, and later in ruby symbols. JSON will solve this issue while being smaller to transfer from api.github.com.

done in redesign branch


[New Port Request] OpenXLSX

Library name: OpenXLSX
Library description: C++ library for reading, writing, creating and modifying Microsoft Excel® files, with the .xlsx format
Source repository URL: https://github.com/troldal/OpenXLSX
There is another issue concerning this library, but that was closed a few months ago, so I'd like to ask sir @gritukan whether he has an ambition strong enough to handle this port.

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Update dependencies

Most importantly, allows byron-proxy to be built with cabal.

This PR is no longer needed! Closing.


Refactor timeframes, assets, and flows to use DuckDB for input data

Pull request details
Describe the changes made in this pull request
To use DuckDB instead of tablee
List of related issues or pull requests
Closes #710
Collaboration confirmation
As a contributor I confirm

[x] I read and followed the instructions in README.dev.md
[x] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[x] Tests are passing
[x] Lint is passing

@suvayu, thanks for the review and suggestion 😄 I think we need to re-evaluate all these changes made during this week, given the results from the benchmarks in the latest PRs. @abelsiqueira has the PR #720 to compare performance. Let's review the big picture in the next meeting and decide how to continue.


JSX Alignment Error Issue

Bug Report

TSLint version: 5.2.0
TypeScript version: 2.2.2
Running TSLint via: VSCode

TypeScript code being linted
render() {
        const suggestionsBottomOffset = 32 + (this.props.rows * 24) + 'px';

        return (
	    <div
                style={{
                    height: suggestionsBottomOffset
                }}
                className='textarea--wrapper'
            >
                {/* other code here but makes no difference to the error */}
	    </div>
	);
}

with tslint.json configuration:
{
    "defaultSeverity": "error",
    "extends": [
        "tslint:recommended",
        "tslint-react"
    ],
    "jsRules": {},
    "rules": {
        "indent": [false],
        "quotemark": [true, "single"],
        "object-literal-sort-keys": false,
        "ordered-imports": [
            false
        ],
        "interface-name": [
            false
        ],
        "member-access": [
            false
        ],
        "trailing-comma": [
            true,
            {
                "multiline": "never",
                "singleline": "never"
            }
        ],
        "arrow-parens": false,
        "jsx-wrap-multiline": false,
        "jsx-no-lambda": false,
        "jsx-no-multiline-js": false,
        "eofline": false,
        "jsx-self-close": false,
        "jsx-boolean-value": false,
        "no-unused-expression": false
    },
    "rulesDirectory": []
}

Actual behavior
This produces an error on the rule "jsx-alignment" - "Tag closing must be on its own line and aligned with the opening of tag."

This error seems to occur only when the element in question is the outermost element, i.e. it does not appear in this situation:

Expected behavior
No error.

huh, that's strange. I've seen a lot of JSX and haven't run into this bug. I don't have time to look into this at the moment, but open to investigation / PRs if this is a problem

@alsiola can you upload file with error?
When I copy provided code with error  (form Edge or Opera) into VS Code and enable rendering of whitespace characters - I see Tab characters in several lines and one of them - opening tag.

Probably there goes error and ident rule from TSLint package might help here.
If issue is still reproducible with consistent whitespaces for identation - I'll take a look at it.