Extension issue:  Unable to load python kernel in interactive mode in viscose

Issue Type: Bug
Extension Name: python
Extension Version: 2020.9.111407
OS Version: Darwin x64 19.6.0
VSCode version: 1.49.2

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Hey @lmscalar, would you mind providing repro steps for this issue? Unfortunately the info above isn't sufficient for us to diagnose the problem you encountered. Thanks in advance!


Reword motivation for deal.raises()

I found the documentation here hard to follow, so I rephrased it based on what I think we're trying to communicate. Let me know what you think!

Looks good, thank you :)


Users/kavipriya/webdrivertask

Adding webdriver task to acquire required webdriver

@ShreyasRmsft Resolved PR comments

Prachi got back saying we wanted to attempt all the downloads before failing the task

Also parallellize the fetching of each webdriver

@kaadhina Can we merge or close this?


feat: add serve alias

propose to add a npx quartz serve for npx quartz build --serve
Signed-off-by: Aaron<EMAIL_ADDRESS>

Ideally, quartz might want to decouple serve from build (since I assume build can also be build for production and local testing as such), but I want to engage this idea first before doing anything more complicated.

I'd prefer to not have aliases like this, wondering what the main use case is? Do you just want to preview without constantly rebuilding the site?

yep, I think I want a preview-only mode. Thinking more about this, we probably only saved a few milli, which is not significant since the build is already pretty fast. I will close it then.


docs: automate components overview build

Description

Added script to get the frontmatter information from '/components/' path.
Automated the building of /components/ overview.

Pull Request Checklist

[x] Ask the contributors if a similar effort is already in process or has been solved.
[x] Review the contribution guidelines.
[x] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[x] Ensure all gulp scripts successfully compile.
[ ] Update, remove, or extend all affected documentation.
[x] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

@braddialpad any comment on the technical side here? or is it good to go?


add Colors type definitions

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

I was hoping this would go through pretty fast so that I could continue to add types in the future and start using them without continuously patching/unpatching things as we develop our side, is there still something blocking this?


Incompatibility with Rack 2.1.0 and higher

Hi!
I use Grape in my application and after updating Rack to 2.1.1, I get an error
NoMethodError - undefined method '[]' for nil:NilClass without any backtrace. During debugging I found that it was raising in Grape::Router#cascade?
def cascade?(response)
  response && response[1][Grape::Http::Headers::X_CASCADE] == 'pass'
end

and the response looks like
[#<Rack::Response:0x00007fc561a338c8 @status=200, @header={"Content-Type"=>"application/json"}, @writer=#<Method: Rack::Response(Rack::Response::Helpers)#append>, @block=nil, @length=0, @body=["{\"foo\":\"bar\"}"], @buffered=false>, nil, nil]

I made some investigation and found that Rack team removed #to_ary from Rack::Response in version 2.1.0 and this change made the result of Grape::MIddleware::Formatter#build_formatted_response not implicitly convertable to Array. This is why I had response like a mentioned above instead of response like
[response_status, response_headers, response_body]

To fix this behavior just #finish an instance of Rack::Response in Formatter middleware:
change
Rack::Response.new(bodymap, status, headers)

at https://github.com/ruby-grape/grape/blob/master/lib/grape/middleware/formatter.rb#L49 to
Rack::Response.new(bodymap, status, headers).finish

@ignatbagaev I have the same issue. I'm unable to upgrade to 1.3.0 because of the dry-types dependency. How did you solve this?


Feat: Added notification listener handler functions to support latest changes

Summary

Added decisionNotificationHandler and addTrackNotificationHandler in Optimizely client
Added some unit tests just to verify all conditions of adding DecisionHandler and trackNoitificationHandler

@mnoman09 looks like there are failing unit tests. Can you please fix those?

@mnoman09 looks like there are failing unit tests. Can you please fix those?

Actually we have to wait for the release of java sdk decision notification changes. Then we can update the java sdk version in android sdk and all of it will work. So I think we should not merge it now.

Java SDK 3.1.0 was released yesterday.

I've upgraded to Java 3.1.0 and I have a passing custom Travis run of the FSC tests here: https://travis-ci.com/optimizely/fullstack-sdk-compatibility-suite/builds/110942171


Clarify what does "Insert my local MAC address" do?

In the Interface setup view, there is a button Insert my local MAC address, like so:

I think it would be worth clarifying on the GUI page, what does that button exactly do, and what happens when one leaves the box blank?
In particular, does "my local" in that sentence mean the address of the connected user's MAC address who is configuring the web UI? Or does Insert my local MAC address mean Insert this adapter's default MAC address? If so, I feel that the second sentence would be more appropriate to use, since the reference "my local" is vague to me. I understand this is probably crystal clear to you, but I was really confused.
Also, I think it would make sense to add a small text paragraph there saying what will happen if the box is left blank, and show what the adapter's default MAC address actually is: e.g. "If this field is left empty, the adapter's default MAC address 03:1F:53:01:33:36 will be used".
Are these assumptions correct?

Hi @juj,
It wasn't / isn't clear, I agree... After a review, the code "means" the following: the visible remote MAC address that is being used to connect to the OPNsense box via your current GUI session, noting that it will be the nearest router hop by the way how the link layer works.
That is what is normally necessary if you put the OPNsense in front of your home router to be able to talk to the modem in front of it, telling the modem that the "correct" device is connecting.
Your assumption is correct and the text could be changed slightly, we don't know if the adapter is hard or software so its MAC may be spoofed / generated and should not be displayed.
As to how "my local address" should be reworded: what do you think now?
Cheers,
Franco

Thanks for the clarification. Reading through your explanation, I admit I might be at a slightly too beginner level on networking concepts to grab the explained use case to be able to recommend a good wording.
What happened in my scenario is that I had a Realtek network interface on my OPNsense router box connecting to my WAN side DSL box, and I had connected my local PC directly to the OPNsense box LAN port to configure the box. Initially I was having problems with the OPNsense box's NIC to obtain a DHCP address for the WAN side. At the time I was not aware of the trouble with the Realtek NIC card drivers not working well with FreeBSD, and I was browsing around the OPNsense settings pages to find what might be the problem, testing various settings. At some point I thought to try clicking the "Insert my local MAC address" button thinking that it would "autofill" the hardware MAC address of the adapter into the MAC address field, with a "well, does not harm to explicitly add this in so the field will not be left looking empty" thought on it. Some time after that, I think the Realtek NIC card intermittently was able to get a DHCP lease, so I thought I was all in green and did not think much about it.
Until a bit later, when I revised my network setup topology to my intended configuration, where instead of connecting my PC directly to the OPNsense box, I would add a 24-port network switch in between the PC and the OPNsense box, to connect more devices to my LAN. After that, everything got really crazy, as it turns out that the OPNsense box and my PC were all of a sudden operating with colliding MAC addresses, and the router in between the PC and OPNsense was throwing packets all over, causing huge latencies and about 90% of packet loss, with random 10% of packets reaching their intended destination eventually. This led to a couple of days of hair scratching and debugging, and thinking that this brand new 24-port switch might be even physically broken, since connecting the PC directly to the OPNsense box had worked.
Only by a random chance I was able to figure out that there was a MAC collision, when I was looking at the switch's configuration page, which was fluctuating the same registered MAC address between two different ports, which was a "hold on, what's going on there" moment. After that, clearing the MAC address fields of the OPNsense box resolved the problem.
I am not sure what a good wording would be, but whatever it ends up being, it would be good to be clear to ensure it's something very explicit to not have a chance to be confused to being a sorts of a "autofill the defaults" action, like my mistake was.

Thanks for the explanations. I've reworded / reordered the help text a bit, although there isn't much obvious to change. I've subtly added "leave empty if unsure", "(use with care)" and a hint that this is normally not required, and if only for "WAN interfaces".

Cheers,
Franco


I think there is a typo, either in the article or in the code

https://github.com/acmi-lab/PU_learning/blob/a9174bda92c7411906056c789011cfa41749ee5f/estimator.py#L66
Line 66 appears to be the UCB portion from Algorithm 1 in the arXiv paper ("method 2"). But the paper "Mixture Proportion Estimation and PU Learning: A Modern Approach" says sqrt(log(4 / delta) / 2 / n), whereas the code says sqrt(log(1 / delta) / 2 / n).
So I think there is a typo somewhere, either in the code or in the paper.

Updated, thank you for catching that.


Should the -c --clipboard option be the default?

It surprised me that the tool didn't take stdin when I piped in a curl to a web page.
I'd say, if there's some way to detect it, if the tool is getting something from STDIN, then STDIN should be used.
If the user isn't piping anything and but they have something on their clipboard, then sure, the clipboard could be the fallback in that scenario.
But yeah, just popping in to say that not accepting STDIN was a bit of a shock.

Yeah. It should. Maybe I Dufke some time in the next days. PRs welcome

Accepts STDIN now v0.4.1


Added find nCr #194

Since this is unoptimized, you can rename the file and add unoptimized in beginning then I will merge it.

@amitsaini4556  Updated the fileName

@amritansh22  Any updates on this ?

Please star this repository to show your support, in the meantime I am reviewing your code.


Bridge: only has stellar chain?

There's no point of showing a combobox if it's only stellar?
maybe we should convert to tabs? or something else?

Verified, Devnet 9280a08.


[BUG]: select - Expected 0 arguments, but got 1.

What version of drizzle-orm are you using?
0.27.2
What version of drizzle-kit are you using?
No response
Describe the Bug
When trying to select and specify attributes from a table I get the error mentioned above.
example:
db.select([model.id]).from(model)

or
db.select({id: model.id}).from(model)

This in the docs:
const result = await db.select({
    id: users.id,
    lowerName: sql`lower(${users.name})`,
  }).from(users);

Expected behavior
I can add an array or object of expected attributes as the documentation says.
Environment & setup
running locally in docker using MySQL 8.
node 16.20.0
tsc-watch 6.0.4
typescript 5.1.6

db.select({id: model.id}).from(model)

this example will work well in the latest drizzle-orm version


Updates to Readme

Typos referring to 'election workloads' -> should be "finance workloads"
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hello, and thanks for your contribution an interest in Landing Zone Accelerator! The financial/tax best practices README has been updated in our latest release and no longer has incorrect references to the elections industry. I will be closing this PR out. Thanks again!


Group and Proxy Repositories

Request details
While migrating from Nexus 3 to Reposilite I really missed the "Group" and - to a lesser extent - "Proxy" repositories. Although I can emulate those repository types (groups by proxying multiple repositories via localhost, proxies by having only read and a single proxied repository) it would be nice to have those special repository type in reposilite. It would make the UI much cleaner and show the intended purpose of the repository much better.

After some testing this morning I believe that having dedicated repository types could also improve performance significantly by avoiding unnecessary lookups.

proxying multiple repositories via localhost

You don't have to link local repos through url, you can simply use repository name in link field:

it would be nice to have those special repository type in reposilite. It would make the UI much cleaner and show the intended purpose of the repository much better

I agree, but current approach is the most flexible one, because you can also mix local repos with mirrored ones if needed. It's also easier to maintain & manage + atm it'd be a breaking change if we'd like to modify this structure.

After some testing this morning I believe that having dedicated repository types could also improve performance significantly by avoiding unnecessary lookups.

If caching is enabled, this lookup is still needed.


[IBCPDE-847] Set an upper-limit to log file size

problem
There is currently no upper-limit to how large a log file can get, which can cause download and upload bottlenecks when communicating with Synapse, and also be troublesome for organizers/participants who need to troubleshoot with the log file but are having to weed through unnecessary log information.
solution
Set an upper-limit to how large a log file can get. I chose to set an upper limit based on the number of lines in the log file (max_lines) rather than the file size itself, because it's easier to visualize how long an error traceback can get in line number vs. byte size. An error traceback can vary in length, but usually contains the most relevant traceback information in the last ~20 lines (the current default), but this value can be experimented with using the max_lines argument.
testing & preview
Tower-dev run for a submission with a standard output spanning 30 lines.
The resulting Docker execution log only captures the last 20 lines (lines 11-30), as requested by the default max_lines value.

There is currently no upper-limit to how large a log file can get, which can cause download and upload bottlenecks when communicating with Synapse,

Is this really a problem that we need to solve for? What is the size of this log file that this becomes a bottleneck to this process?

and also be troublesome for organizers/participants who need to troubleshoot with the log file but are having to weed through unnecessary log information.

The other problem in this case is that if the log messages that indicate the problem are above the truncated messages - then we don't have full context on why something failed. I have had issues during startup of services that didn't cause other errors or force exceptions to be thrown until thousands of log messages later.
Overall - This seems like we are solving for a non-existent problem from my POV

There is currently no upper-limit to how large a log file can get, which can cause download and upload bottlenecks when communicating with Synapse,

Is this really a problem that we need to solve for? What is the size of this log file that this becomes a bottleneck to this process?

and also be troublesome for organizers/participants who need to troubleshoot with the log file but are having to weed through unnecessary log information.

The other problem in this case is that if the log messages that indicate the problem are above the truncated messages - then we don't have full context on why something failed. I have had issues during startup of services that didn't cause other errors or force exceptions to be thrown until thousands of log messages later.
Overall - This seems like we are solving for a non-existent problem from my POV

One of the initial issues for limiting logs is actually to reduce the risk that participants return the entire prediction files back in their log file. That said, there's no such thing as 0 risk as long as we are returning logs to participants

There is currently no upper-limit to how large a log file can get, which can cause download and upload bottlenecks when communicating with Synapse,

Is this really a problem that we need to solve for? What is the size of this log file that this becomes a bottleneck to this process?

and also be troublesome for organizers/participants who need to troubleshoot with the log file but are having to weed through unnecessary log information.

The other problem in this case is that if the log messages that indicate the problem are above the truncated messages - then we don't have full context on why something failed. I have had issues during startup of services that didn't cause other errors or force exceptions to be thrown until thousands of log messages later.
Overall - This seems like we are solving for a non-existent problem from my POV

One of the initial issues for limiting logs is actually to reduce the risk that participants return the entire prediction files back in their log file. That said, there's no such thing as 0 risk as long as we are returning logs to participants

Makes sense - If that is the case then the code as written is going to fail @jaymedina - You are reading an entire file into memory, but could very well be above the maximum memory available where the code is being execute. You should look to only read portions of the file into memory (It'll require re-working your function logic)

@BryanFauble it wouldn't fail, it would just take a long time to execute. But I see this point - the original logic involved seeking to the end of the file and reading from the bottom up to a certain byte size, which I can recycle. The current limit is 5KB as @thomasyu888 mentioned, but I can set this value as an arg to make this easier to tweak.

Screenshots 2 & 3 imply that the log for the 3rd submission is 10 bytes larger than the 2nd submission, when I think it should be 30 (so 3rd submission's log should be 48.86), so I'm not sure what's going on with that.

But I downloaded the files locally and they'll be hovering around 50Kb - I think the one with non-ascii characters is larger because of the symbols etc that require more bytes (it's because I slice the non-encoded log_text string), so if there are non-ascii chars in a log file, it'll be slightly larger than 50Kb but negligible (unless all the characters are non-ascii)

Screenshots 2 & 3 imply that the log for the 3rd submission is 10 bytes larger than the 2nd submission, when I think it should be 30 (so 3rd submission's log should be 48.86), so I'm not sure what's going on with the way the Synapse backend computes file sizes. I'd be curious to know if anyone has an idea on why the file size is different on Synapse.

@jaymedina Consider the differences in the representation of data:
1 KB = 0.976562 KiB = 8 Kb
One of the ways to be consistent though that the content of the file won't cause compression algorithms to handle them completely differently would be to use non-repeating, sudo random bytes.


chore(data-picker): fix non alpha-numeric formatted year

General summary

Test OK ✅ (local storybook)


include wagtail urls to django sitemap

I'm using wagtail app in my project. Is it possible to join  django sitemaps (https://docs.djangoproject.com/en/1.11/ref/contrib/sitemaps/) with wagtail sitemaps? I know that there are "wagtail.contrib.wagtailsitemaps", but it only show my wagtail app urls.

Hi @vyckiuz - please use the Wagtail tag on Stack Overflow for support queries.


Fix number of replicas in log entry

Signed-off-by: Karel Suta<EMAIL_ADDRESS>Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

A tiny fix related to test logging, so no JIRA created.


Cancancan adapter is registered as cancan

Currently, cancancan cannot be set in config.authorize_with because it's registered with name cancan - same as cancan
I am unsure why it's like this but it seems not to be intended.

Build failure seems to be unrelated to this..


Some questions

I created this issue to find out how to deal with implementation problems:

[ ] HumbleRL doesn't provide information about allowed actions, how should it be handle? I have two ideas:

use debug info to return allowed actions from step method (requires changes in humblerl)
create game specific utils (good for short term, bad in general)

Look at classes in 'humblerl.utils.wrappers'. They provide state space and action space informations through 'humblerl.Environment' interface.
Pozdrawiam,
Piotr Januszewski

Wiadomość napisana przez Mateusz Jabłoński<EMAIL_ADDRESS>w dniu 20.05.2018, o godz. 12:41:
Assigned #1 to @piojanu.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

You need to implement 'humblerl.Environment' interface in Game class of AlphaZero.
Pozdrawiam,
Piotr Januszewski

Wiadomość napisana przez Piotr Januszewski<EMAIL_ADDRESS>w dniu 20.05.2018, o godz. 15:01:
Look at classes in 'humblerl.utils.wrappers'. They provide state space and action space informations through 'humblerl.Environment' interface.
Pozdrawiam,
Piotr Januszewski

Wiadomość napisana przez Mateusz Jabłoński<EMAIL_ADDRESS>w dniu 20.05.2018, o godz. 12:41:
Assigned #1 to @piojanu.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.


Liquid syntax error: Unknown tag

Hi,
I got your plugin to work using a trivial generated site (jekyll new mysite).  So I know it works!
But then I tried to get it to work with an existing Jekyll site and though I have triple checked that I did all the setup steps correctly, I am still getting this when I try and build the site:
/home/dtenenba/dev/wiki/vendor/bundle/ruby/2.7.0/gems/liquid-4.0.0/lib/liquid/document.rb:23:in `unknown_tag': Liquid syntax error (line 5): Unknown tag 'glossary' (Liquid::SyntaxError)

The line it is complaining about just has something like:  bla bla {% glossary consideration %} bla bla and there is an entry for consideration in my _data/glossary.yml file.
I have added the gem to my Gemfile and run bundle install. I've added the plugin to my _config.yml. CSS is all set up. Not sure why I am still seeing this. Do you have any ideas of how I could go about troubleshooting this?
One thing that might matter, this site has a remote theme.
If you want to see the site, it's in this branch.
Thanks....

Hey,
I'll take a proper look tomorrow. Just one thing to check first that might be the problem. I think this gem should be listed in the jekyll_plugins group in the Gemfile. I recall this is a group that has special meaning in Jekyll, as these are plugins loaded at compile time.
Check this minimal repo set up:
https://github.com/erikw/jekyll-4.x.x-test/blob/main/Gemfile#L19

Hey, that worked!
The site built and the glossary item rendered after doing this. Thanks!
) $ git diff
diff --git a/Gemfile b/Gemfile
index 464a6fc..3677c0a 100644
--- a/Gemfile
+++ b/Gemfile
@@ -1,9 +1,12 @@
 source 'http://rubygems.org'

+group :jekyll_plugins do
+  gem 'jekyll-glossary_tooltip'
+end
+
 group :development, :test do
 #   gem 'jekyll', '~> 3.8.3'
   gem "github-pages", group: :jekyll_plugins
-  gem 'jekyll-glossary_tooltip'
   gem 'jekyll-redirect-from', '~> 0.14.0'
   gem 'jekyll-sitemap', '~> 1.2.0'
   gem "nokogiri", ">= 1.10.4"

Oh great! =)


Deployment guide feedback

Section heading: Terraform?
Documentation issue description: Are there any plans to package this deployment in Terraform instead of CloudFormation?

Nothing in the works right now.

@CossyCosmas FYI


multiple choice proposals v1

Adds the cw-proposal-multiple contract with single-choice voting (first-past-the-post). Ranked choice to come
is_passed and is_rejected need the closest review: https://github.com/DA0-DA0/dao-contracts/blob/8552506f9d60489e7e8c3917fddca22d1105236a/contracts/cw-proposal-multiple/src/proposal.rs
Voting strategy design choices:

get rid of the percentage option entirely for multiple choice proposals. (it is difficult to reason about a fixed percentage threshold when proposals can have a variable number of options)
There will always be a quorum for voter turnout (since we are removing the percentage option, it makes sense to have a min bar for turnout after which the highest voted choice will be selected)
There will be two voting strategies, either single-choice (first-past-the-post, ie the option with the highest number of votes gets selected), or ranked-choice (single-transferable vote, ie multiple choices can be ranked, and votes will cascade to lower preferences until an option receives the majority of votes)
We will include the abstain / "None of the Above" option — like in single choice proposals, it will contribute towards quorum and will also allow proposals with only bad options to be voted against.
The voting strategy for a proposal module will be configurable by config, just like how they are for thresholds currently

improvements to be made after review with zeke:

multiplechoiceoption —> checked multiple choice option (or CheckedProposalOption)

wrapper type that adds the none option
will add index and vote count
can move validate_choice s there
add a constraint to the number of choices, eg 10 to prevent gas issues

calculate vote result: consider finding top choices in one pass

test for the overflow case in is_choice_unbeatable

later: remove option for non members execute?

2 raw dao issues:

after rebasing copy the update_status and passed checks into proposal_single

// run a query to query voting modules for validating that this is a dao

Codecov Report

Merging #279 (7b68bb2) into main (e531c76) will decrease coverage by 3.00%.
The diff coverage is 72.14%.

@@            Coverage Diff             @@
##             main     #279      +/-   ##
==========================================
- Coverage   96.92%   93.92%   -3.01%
==========================================
  Files          37       43       +6
  Lines        8695     9698    +1003
==========================================
+ Hits         8428     9109     +681
- Misses        267      589     +322

Impacted Files
Coverage Δ

contracts/cw-proposal-single/src/staking_tests.rs
100.00% <ø> (ø)

contracts/cw-proposal-single/src/utils.rs
100.00% <ø> (ø)

contracts/cw-proposal-multiple/src/contract.rs
36.08% <36.08%> (ø)

packages/testing/src/helpers.rs
57.62% <57.62%> (ø)

contracts/cw-proposal-multiple/src/state.rs
80.00% <80.00%> (ø)

contracts/cw-proposal-multiple/src/proposal.rs
82.43% <82.43%> (ø)

contracts/cw-proposal-multiple/src/tests.rs
85.35% <85.35%> (ø)

packages/voting/src/threshold.rs
84.00% <85.71%> (+0.66%)
:arrow_up:

packages/testing/src/tests.rs
87.86% <95.83%> (+0.07%)
:arrow_up:

contracts/cw-proposal-single/src/tests.rs
98.59% <97.87%> (ø)

... and 5 more

Continue to review full report at Codecov.

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

re: slight code coverage decrease, I've ensured that the all the critical code paths are test-covered and I think codecov is a little wonky with its calculations as the coverage still decreased even after excluding some test files from the calculation. Think it's good to merge.


[Incubator/kafka] Upgrade kafka-exporter

Upgrade kafka-exporter to 1.2.0 for better performance and bugfix

/ok-to-test

@benjigoldberg Does it look good to you ?

Thanks for the contribution @danielqsj!
/lgtm


how to link units

option 1:
link the triple of (employer, union, affected_city)
option 2:

link employers
link unions
link employer_id, union_id, affected_city

I think we ultimately want this more normalized structure so let's try option 2.

we could get a list of the locals from odpr and use that as a gazetteer

we have an approach we are building.


Change all serialisable enums across Crux to be "internally tagged"

Now we introduced the ability to serialise as JSON (in #194), we want to make some ease of life improvements to JSON serialisation formats.
From our experience, handling JSON serialisation and deserialisation in TS, Kotlin and Swift is much easier when enums are internally tagged.
For example it avoids having to look ahead in enums with different variant types in order to determine which variant to use.
Note: this will not affect bincode serialisation, but is a breaking change for JSON.

[ ] Crux macros
[ ] Every serialisable enum across all crates

for completeness, this is the error from serde_generate
  Error: type tracing failed: Not supported: deserialize_any:

  An unsupported callback was called during (de)serialization tracing. In practice, this happens when an
  unsupported Serde attribute is used. Attributes specific to self-describing formats (JSON, YAML, TOML)
  are generally not supported. This includes: `#[serde(flatten)]`, `#[serde(tag = "type")]`,
  `#[serde(tag = "t", content = "c")]`, and `#[serde(untagged)]`.

  To fix this, avoid unsupported Serde attributes or use custom (de)serialize implementations with different
  behaviors depending on the Serde callback `(De)Serializer::is_human_readable()`.


FormatException on specific date and time (very strange)

Describe the bug
Getting the following exception on this specific date range only i.e. 23-03-12 02:00:00 to 23-03-12 02:59:59 with the following message
FormatException: Error parsing 23-03-12 02:00:00, invalid hour value: 2 in en_US with time zone offset -4:00:00.000000. Expected value between 3 and 3. Date parsed as 0023-03-12 03:00:00.000..
To Reproduce
The timezone should be either UTC -4, -5, -6, -7 and running a local test as following will reproduce it. Please note that this works fine for all the time windows except 02:00:00 till 02:59:59
void main(List<String> arguments) {
    String date = "23-03-12 02:00:00";
    DateTime? dateTime = DateFormat('y-MM-dd HH:mm:ss').parseStrict(date).toLocal();
}

Expected
The library should work as expected on the above mentioned time range.

This can be reproduced with
DateFormat('yy').parseStrict("23");

as the verify method checks if the year is valid, where 23 is not in the valid range of 2023 to 2023.
This should be fixed by adapting the validation code to check for the _hasAmbiguousCentury variable.

The issue I am facing is due to the Daylight saving adjustment in the mentioned timezones, which occurs on 12 March every year, hence the verify method fails because there is no consideration of Daylight saving.

I am not sure I understand - the problem you described in the first post is reproducible without any specific reference to a time, just using the year. Could you elaborate?

The issue I am facing is specific to the time (particularly hour) and not the year. If you try to parse this date 23-03-12 02:00:00 with this format y-MM-dd HH:mm:ss then the time returned is DayLight saving adjusted, i.e. 23-03-12 03:00:00 which then goes through the series of verify checks as I am using the DateFormat.parseStrict method. It fails to verify the Hour field and throws the FormatException with the following Stack trace;
DateBuilder._verify (date_builder.dart:159)
DateBuilder.verify (date_builder.dart:131)
DateFormat._parse (date_format.dart:400)
DateFormat.parseStrict (date_format.dart:384)
Data.getDate (data.dart:875)

Hope the above stack trace helps.

The format y-MM-dd HH:mm:ss would expect the date to be 2023-03-12 02:00:00, which then works. Does this solve your problem?

Try it on the 26th of March at 2:00am.
This failure depends on the underlying OS/browser's daylight savings time
transition code. This is different between North America/Europe/elsewhere.
But also, it's not that there's no accommodation of daylight savings time.
It's failing precisely because it does take it into account. There are
DateTimes that are not valid because they don't exist. If you try to create
one, the underlying DateTime code will adjust it to be something different.
It fails validation rather than silently changing the value.
On Mon, Mar 27, 2023 at 8:58 AM Moritz @.***> wrote:

The format y-MM-dd HH:mm:ss would expect the date to be 2023-03-12
02:00:00, which then works. Does this solve your problem?
—
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/intl/issues/570#issuecomment-1485066867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2QU6DMFBTBCUICUZG7S53W6GFGDANCNFSM6AAAAAAWAYVR3U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Proposal: Support for addition of tests using frameworks like Unittest, Pytest etc.

Warning: this project appears to have blatantly ripped off the work of researchers over a year on a new multi-modal model, Atlas-1, and is attempting to scam open source devs into doing that work https://github.com/OthersideAI/self-operating-computer/issues/67

I'm definitely interested in a way to automate testing. If someone makes a PR that'd be great!

We did get a PR that added evaluate.py so I'll close this now!


Add Snyk + Node.js template

Needs checking and editing.
Anyone is free to take this over and complete it.

Righto, I've taken this one over and switched the plugin out for using the docker image direct.
Unfortunately, It seems the seek plugin is using a no longer supported docker image.


Saleor 1670 visual improvements for dashboard homepage

I want to merge this change because it fixes hover overflow on dashboard homepage and adds padding on arrow icons.

PR intended to be tested with API branch:
Screenshots
before:

after:

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Translated strings are extracted.
[ ] Number of API calls is optimized.
[ ] The changes are tested.
[ ] Data-test are added for new elements.
[ ] Type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-1670 Visual improvements for dashboard homepage

Just to be clear - it should fix all the lists with highlighted elements and with rounded corners (like the one in homepage).


dynamic qanda section

Hello all.
I'm having troubles with a  qanda section on an interview-like post, I'd like to either remove the automatic enumeration, replace it with bullets, or manually set the number before the question, I have a text structure similar to this:
[qanda]
*question 1*::
  answer 1

Text related to Q1 but outside the scope of Q1.

 [qanda]
resets to Q1!
*question 2*::
answer 2

Is there any way to do this?
Many thanks in advance

This is an AsciiDoc question. You need to use a list continuation to prevent the list from breaking. See https://asciidoctor.org/docs/user-manual/#list-continuation
Here's how that would look:
[qanda]
question 1::
answer 1
+
Text related to Q1 but outside the scope of Q1.

question 2::
answer 2

Please reopen if you need more information.


Bring back IBulkResponse for easier mocking

Describe the feature:
Since 7.0.0, IBulkResponse interface was removed and Bulk methods now return BulkResponse. This causes mocking of all Bulk operations in consuming applications a little more difficult.
I think it was beneficial to be able to mock the response and test the application behavior and usage of Items, Errors, ItemsWithErrors objects.
I have a work around, but it seems more elegant to operate on interfaces for the sake of consumers.

Hi @mszczepaniakebsco thanks for raising this.
We've added interfaces for responses at a time when Elasticsearch had say 30 API's, since then Elasticsearch has grown to nearly 230+ API's  the cognitive and literal byte size of these interfaces were weighing on us during development.
We've kept the interfaces for SearchResponse<> and are committed to keep these around since
we feel 80% of the people that want to mock the client want to do so for that API.
I am a strong believer though that you shouldn't mock what you don't own and should instead mock/stub/fake your layer that calls Elasticsearch.
In the cases where you call the client directly from your controller and have no layer of indirection in between (which is also totally okay of course) we feel that injecting an IElasticClient with an InMemoryConnection is a viable alternative. We are working on releasing a package to aid with creating an in memory client that behaves a certain way see #3908

Hi @Mpdreamz , I totally understand the burden of maintaining 230+ API with interfaces, base classes etc.
I disagree with your statement that I shouldn't mock what I don't own. It's fair to say: I shouldn't test the code that I don't own, and IMHO mocking is meant exactly for that. I have my layer that calls ElasticSearch and part of that code needs to be tested. I need to test how my code reacts to various characteristics of ElasticSearch response objects, for example: what my code supposed to do if BulkResponse.IsValid is false. I need to write code that does what is expected and I want to test it.
Right now, I am mocking the response by parsing properly crafted JSON object using RequestResponseSerializer, but that seems unnecessarily cumbersome.
With all that said, another idea that would help the consumers test the responses is to mark all/most/critical properties as "virtual". This way the Response object can be mocked without much hassle on the consumer end and also should not add any additional weight on your end.
Thoughts?

I have a similar problem, but in my case I'm using Cat, Indices and  Snapshot endpoints so I'm having troubles now with mocking both endpoints and responses.
I would like to be able to set up responses for the calls and simulate when server returned an error.
Could you also look into setting up responses not via deserializing json-string but actually via object initialization. Right now some of the models have internal setters - for example IndexStats and all it's members has internal setters and you need to actually find IElasticsearchSerializer in order to properly get objects and maybe I'm blind but it took me a while reading/decompiling to figure out how to do it.
I also found testing section in documentation for nest and elasticsearch.client but it contained only references to Java but it would be really nice to have it if you will go with introducing testing library.

@Mpdreamz what about making those stuff "virtual" as suggested from @mszczepaniakebsco

@Mpdreamz what about making those stuff "virtual" as suggested from @mszczepaniakebsco

I will agree as it will allow us to use mock libraries


Update README with getting started information

The README should include the following information:

A link to the Durable Functions documentation
Code snippets for .NET Isolated
Notes about source generators
Information about how this repo relates to the existing GitHub repos (azure/durabletask and azure/azure-functions-durable-extension)
Standard Microsoft repo README stuff

We can probably avoid talking about the sidecar for now to avoid confusing people.

@cgillum I think most of this is covered by the README now. Anything missing or can we close this issue?


Infinite loop when using the SeparateGroups option

I tried to use the new SeparateGroups feature but was stuck in an infinite loop when I called the "separateGroups" function. I noticed that by starting the iterator at 0 instead of 1 the problem was solved.

This solution did not work

@claudels Thanks. Could you please provide an example of the issue that it solves?

This CodePen shows the issue. If you remove the parentheses in the RegExp, you will be stuck in an infinite loop.

@claudels Thanks for the CodePen. Haven't investigated why but I can't reproduce using this CodePen:
https://codepen.io/anon/pen/RqWRGY?editors=1010
What's the circumstance that causes it not to work?

This is because you imported version 8.11.1 of mark.js but the SeparateGroups option is not in this release but only present on the master branch. In my CodePen, I used the master version with: https://cdn.rawgit.com/julmot/mark.js/master/dist/jquery.mark.js

@Mottie Could you please have a look at this PR as the initial implementation (not released yet) is coming from you?

I ended up cleaning up the code and adding tests. It should be good to merge.

Thanks @Mottie @claudels! 🎊

Thank you 👍 , you are very responsive. Do you have any idea when version 9.0.0 will be released?

It doesn't solve the infinite loop problem.

text abc text abc

to filter first match
let count = 0;
$('p').markRegExp(/(\w+)\s+(abc)/gi, {
    separateGroups: true,
    'filter' : function(elem, match) {
        result ++count > 2 ? false : true;
    }
});

or if text node doesn't contains match
$('p').markRegExp(/(\d+)?\s+(x)?/gi, {
    separateGroups: true
});

or stupid regex /()\s+()/g.
The cause of the problem is resetting RegExp lastIndex in the wrapMatches method without making sure that at least something is wrapped.
If it doesn't, the same text node is returned. The next iteration matches it again ...


🐞 - ExpressionChangedAfterItHasBeenCheckedError occurs when navigating via the dropdown menu in TuiTabsWithMore

Playground Link
https://stackblitz.com/edit/angular-t8ebht?file=src%2Fapp%2Fapp.component.html
Description
When pages are switched using tabs, everything works fine. However, when switching via the dropdown menu in TuiTabsWithMore, NG0100: ExpressionChangedAfterItHasBeenCheckedError error occurs.
The error is present if a is used in the markup. However, if button is used instead, everything works correctly.
The error occurs within this template (https://github.com/taiga-family/taiga-ui/blob/main/projects/kit/components/tabs/tabs-with-more.template.html#L63C17-L63C42), where the shouldShow method initially returns false and then true.
Angular version
12.2.17
Taiga UI version
3.73.0
Which browsers have you used?

[X] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[X] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[ ] Android

Not reproducible in v4 using the same code


Reminders can be deleted by anyone

Currently any and all reminders can be deleted by any member. This needs to be fixed ASAP so that members can only delete their own reminders.

Working on it


📌 FROM Containous TO Traefik 🚀 ✨

Routing "traefik-helm-chart" from Containous to Traefik ... more information on that later! :speak_no_evil: :detective:
:heart: :heart: :heart:

Dear everyone,
Traefik turns 5 years old today :birthday: !

We’re celebrating this by:

giving Traefik’s home a new name: Traefik Labs
by releasing a new product: Traefik Pilot
and by sharing our vision for the future of cloud-native networking

Happy birthday Traefik!
https://twitter.com/traefik/status/1308747253738360833


Simplify donation request handling

@mykiwi I fixed your comment.

@tgalopin any review and approval on your side please?


Ensure we keep short file names

While system may support long file name (over 250 chars), it seems
ansible doesn't really like that.
Also, it may look really weird having a hook name so long it takes
multiple lines to display.
In order to make it more readable, and avoid any file name length issue,
we're now truncating the hook name to 20 chars. This will lead to better
UX, and may avoid some weird issues.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

/lgtm

/approve


[Hackathon] Implement make session key for user verification

It need session key for user verification. And It's stored to client's storage like cookie.


Fix: add missing python wrappers for MolDraw2DQt

This is a bit tricky since PyQt is wrapped using sip and we need boost::python wrappers.
Fortunately sip provides a function which allows us to get the raw pointer behind one of its objects and we can pass that directly to C++.
There's another bit of fun around lifetime management of the objects - trying to GC the QPainter object before we've removed the MolDraw2DQt object results in a seg fault.
This handles both problems by providing a python convenience function which creates a MolDraw2DQt object from a QPainter and sets up the lifetime management properly.

@ZontaNicola: would appreciate your feedback on this one.
There's not a lot there, but if you have suggestions about ways to handle the PyQt<->Boost thing more cleanly I'd welcome them.

FWIW: the test failures here are almost certainly because I have the linux CI build setup to use the system Qt and PyQt5 from conda. That's not a recipe for success, so I'll need to do some more work on the linux-ci configuration. This shouldn't impact the review of the PR though

I'm far from being an expert in Qt, PyQt, or python boost, but even though the unwrapping/casting method doesn't look very nice to me, it seems to be what other people are doing (at least from what I have googled around to see if other people had similar problems). The object's "lifetime management" workaround looks a bit hackish, but I guess it also comes from the same issue: PyQt and boost::python don't seem to be designed to interact between themselves.
Regarding the tests, I have downloaded this patch and tried the tests locally, and found some problems: when I import rdkit.Chem.Draw, MolDraw2DFromQPainter isn't defined, and so tests testBasics and testMemory2 fail with errors. MolDraw2DFromQPainter_ (note the final underscore) is defined and working, though.
Apparently, I'm having some problems to import sip, which explains why the function isn't defined. If I change init.py to use from PyQt5 import sip instead of import sip, then everything works fine. I think it might make sense to update this code to use sip from PyQt5, so that we are guaranteed to use the same version that was used to generate the Qt wrappers. Maybe this could also explain the segv Greg observed ?

My suggestion to use sip from PyQt5, instead of the separate package.

Ok, I made those changes, but I don't think that's actually going to help.
For conda installs of PyQt5 there is no PyQt5.sip module. Installing the pyqt conda package gets you a standalone sip package (this is what I've been using).
(rdkit_build) root@3ce870abb14b:/src/rdkit/build# conda install PyQt
Collecting package metadata (current_repodata.json): done
Solving environment: done

## Package Plan ##

  environment location: /opt/conda/envs/rdkit_build

  added / updated specs:
    - pyqt

The following NEW packages will be INSTALLED:

  pyqt               pkgs/main/linux-64::pyqt-5.9.2-py38h05f1152_4
  sip                pkgs/main/linux-64::sip-4.19.13-py38he6710b0_0

Proceed ([y]/n)?

Preparing transaction: done
Verifying transaction: done
Executing transaction: done
(rdkit_build) root@3ce870abb14b:/src/rdkit/build# ipython
Python 3.8.5 (default, Sep  4 2020, 07:30:14)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.19.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: from PyQt5 import sip
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-d9a10bac1b11> in <module>
----> 1 from PyQt5 import sip

ImportError: cannot import name 'sip' from 'PyQt5' (/opt/conda/envs/rdkit_build/lib/python3.8/site-packages/PyQt5/__init__.py)

For conda installs of PyQt5 there is no PyQt5.sip module. Installing the pyqt conda package gets you a standalone sip package (this is what I've been using).

Oh, I see. I'm using "vanilla" Python, and it seems there's separate packages for PyQt5, PyQt5-sip and sip (I have more or less the same on 2 different linux machines and a Mac, all of them with "vanilla" Python):

Package
version
PyPI page

PyQt5
5.12.2
https://pypi.org/project/PyQt5/

PyQt5-sip
4.19.17
https://pypi.org/project/PyQt5-sip/

sip
5.5.0
https://pypi.org/project/sip/

PyQt5-sip and sip versions are quite different, and my PyQt5-sip version seems to be close to conda's sip. I'd say that while PyPI's sip is the last available release, PyPI's PyQt5-sip and conda's sip are actually the specific sip version that was used to build PyQt5.
That's a bit messy, but I think we could use something like:
try:
    from PyQt5.Qt import *
except ImportError:
    # If we can't find Qt, there's nothing we can do
    sip = None
else:
    try:
        # Prefer the PyQt5-bundled sip
        from PyQt5 import sip
    except ImportError:
        # No bundled sip, try the standalone package
        try:
            import sip
        except ImportError:
            # No sip at all
            sip = None


Missing The 'linux/input.h' - Windows

I'm trying to run the newest v2.0-rewrite on Windows and apparently evdev requires linux/input.h
      The 'linux/input.h' and 'linux/input-event-codes.h' include files
      are missing. You will have to install the kernel header files in
      order to continue:

That is a bug that I'll fix today, you can fix it by removing the 'app/plugins/EvdevControler' folder.

This should now be solved in the newest update I did 👍

@Tumppi066 Thanks, but now the First Time Panel won't start:
[0.031] [src\logger.py]  - Logger initialized!
[0.125] [src\settings.py]        - ('Theme',)
[0.344] [main.py]     - ("No module named 'evdev'",)
pygame 2.5.0 (SDL 2.28.0, Python 3.9.2)
Hello from the pygame community. https://www.pygame.org/contribute.html
[0.531] [src\settings.py]        - ('LSTRDrawLanes',)
Exception in Tkinter callback
Traceback (most recent call last):
  File "C:\Program Files\Python39\lib\tkinter\__init__.py", line 1892, in __call__
    return self.func(*args)
  File "D:\Downloads\Euro-Truck-Simulator-2-Lane-Assist-2.0-rewrite\src\mainUI.py", line 59, in <lambda>
    helpers.MakeButton(pluginFrame, "First Time Setup", lambda: switchSelectedPlugin("plugins.FirstTimeSetup.main"), 2, 0, width=20, style="Accent.TButton")
  File "D:\Downloads\Euro-Truck-Simulator-2-Lane-Assist-2.0-rewrite\src\mainUI.py", line 127, in switchSelectedPlugin
    ui = plugin.UI(pluginFrame)
  File "D:\Downloads\Euro-Truck-Simulator-2-Lane-Assist-2.0-rewrite\plugins\FirstTimeSetup\main.py", line 47, in __init__
    self.page0()
  File "D:\Downloads\Euro-Truck-Simulator-2-Lane-Assist-2.0-rewrite\plugins\FirstTimeSetup\main.py", line 71, in page0
    self.logo = self.logo.resize((width, height), Image.ANTIALIAS)
AttributeError: module 'PIL.Image' has no attribute 'ANTIALIAS'

Have you tried restarting the app?
If so then do this.
Installed with installer:

Go into ~/venv/Scripts
Open a command prompt (click the file explorer address bar -> cmd.exe)
Type .\pip.exe uninstall pillow
Type .\pip.exe install pillow
Try again

Installed from source:

Open a command prompt (click the file explorer address bar -> cmd.exe)
Type .\pip.exe uninstall pillow
Type .\pip.exe install pillow

@Tumppi066 Same thing

@rknetwork
Ok that's interesting pillow really should be working.

@Tumppi066 Got it fixed:
Search and replace this:
Image.ANTIALIAS
with this:
resample=Image.LANCZOS

Yup I for some reason your pillow doesn't include the default ANTIALIAS ._.
In any case good that you got it fixed, will add that to the FAQ on discord!

@Tumppi066 Also, I just wiped the whole thing and decided to start from scratch using installer, here is what was missing in vdev:
D:\Downloads\Euro-Truck-Simulator-2-Lane-Assist-anaconda-updater\venv\Scripts
pip install sv_ttk psutil pillow pygame cv2 opencv-python
pip install dxcam onnx playsound mss pygments vgamepad wheel
pip install onnxruntime keyboard chlorophyll

This is extremely weird since the installer should just get all of them from the requirements.txt file in the app folder.
I guess I should probably check my installer again xD

This is extremely weird since the installer should just get all of them from the requirements.txt file in the app folder.
I guess I should probably check my installer again xD

Yes, but when 1 package fails (evdev) - rest are ignored  😎

Ah seems that I didn't push the changes yet xD
I added a check for different systems so that it now doesn't do that xD

Anyways, I wasn't able to get it to work properly for weeks now, I tried both ETS2 and ATS, same result... Can you running it on a normal 1080p monitor with 1920x1080 resolution?

What part exactly are you having issues with? And yes it should definitely work on a 1080p monitor :D

The lanes detecting correctly
Setting up the app side of the input.
Setting up the game side of the input.
Inaccurate / "Bouncy" / Aggressive steering

What part exactly are you having issues with? And yes it should definitely work on a 1080p monitor :D

The lanes detecting correctly - NO
Setting up the app side of the input. - YES
Setting up the game side of the input. - YES
Inaccurate / "Bouncy" / Aggressive steering - YES

The capturing camera is following the steering, when too close to the lane markings - it stops detecting lanes or detects wrong, this is when chaos begins

Can you send me an image of what the app sees then?
Preferably in a place where it doesn't detect them well and I'll check how it looks.
I'm using a different technique than UFLD called LSTR. It is much more prone to errors if the camera is not set just right. The flipside being it can run without a GPU. I could try and check v2 and see how well it runs and them might implement that as an option 👍

@Tumppi066 I will wait for the new version, I'm giving up on this one.
None of the lane recognition projects look promising at this point.
FYI: I found that CoordinateX,Y are being provided by the telemetry. Maybe it's worth comparing them to the GPS route and adjust steering accordingly. =)

You can actually check how it should look and work here (though I had the sensitivity etc... wrong so it's bouncing)
https://github.com/Tumppi066/Euro-Truck-Simulator-2-Lane-Assist/issues/25#issuecomment-1612848281

Unfortunately I can't actually get the route from the game (well I get the start and end, but no the route itself) xD
I could do that otherwise 👍

You can actually check how it should look and work here (though I had the sensitivity etc... wrong so it's bouncing) #25 (comment)

I have tried every possible scenario, you know - I'm a maniac...

Unfortunately I can't actually get the route from the game (well I get the start and end, but no the route itself) xD
I could do that otherwise 👍

The game does autosaves right, it should include the route. I'm still researching this

@rknetwork
That's actually genius. Want to come to a discord call tomorrow so I can get your lane assist fixed? I suddenly have an urge to get you onboard with developing some kind of route plugin xD. I already have access to the ETS2 API because of a plugin another person made

@Tumppi066 Let's hold off on this one for now... I lost my patience =))) I think I should write something in Rust hitting the memory directly...

No worries!
Tell me if you find something, I'm focusing on the tutorials for v2 xD

Been a while so I will close this now regardless :)


Muliselect drag and drop not working on Firefox browser

Muliselect drag and drop not working on Firefox browser even for the demo website.

Thank you for logging this! We will address this issue as soon as possible.

Hello @kanwalpreet-singh! It appears this is actually a browser compatibility issue with the HTML5 draggable attribute. Firefox does not support this attribute without the JS hack documented here: http://stackoverflow.com/questions/3977596/how-to-make-divs-in-html5-draggable-for-firefox
Try adding the script from the top answer to see if this fixes your issue. We have decided not to include this on the plugin itself as we want to avoid collisions with other scripts developers may already have for their application.


Want to add Share buttons so that user can easily share your website at different social media

Want to add Share buttons so that user can easily share your website at different social media
User Friendly Feature

!assign

/assign

Any suggestions for new features? @maintainers

Go ahead @grraghav120! Do share the screenshot when you create PR so that I can suggest changes if necessary.
About new features, whenever I get an idea I create an open-for-all issue so you can look for that :)
For now, just work on the share button :) Don't forget to star the repository!


Update release finalization to push postrelease branch

Should get autocreated based on the promoted tag, then we can consume it in our postrelease workflow

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @kennknowles


Read from stdin when file path is '-'

I've found it convenient to pipe logs directly into Gameboy Doctor to avoid having to make/maintain log files. A neat side effect is that the emulator will exit once Gameboy Doctor exits (on mismatch error or reaching the end of the log file).
To read from stdin just specify "-" as the path like this:
emulator | ./gameboy-doctor - cpu_instrs 1

Example usage:

Unfortunately the way I did this changes the indentation of the main loop so it might be nice to find a way to preserve the with statement to avoid that.

Looks like there's a conflict with your other PR which I just merged, mind rebasing quickly? ty

No worries, thanks for making the tool :)


test kit traces

Looks interesting. Perhaps we can move SpanNode to the testkit too, if we can get rid of SpanData from Java's OpenTelemetry.

Hi @iRevive didn't expect someone to be looking over at this already.
I've been trying to test the metrics and traces. Using the few clues I had here, I was able to infer that MetricsSdk gave me what I needed to test metrics created by my Http telemetry middleware. There wasn't a corresponding Sdk-like utility for traces but I found something in the TracerSuite that looks like it could be extracted.
For now I've already implemented something inspired by that method so I can successfully test traces, albeit having to drill down to java classes. But I plan to make the contribution formally so that it includes mappings from java types as you mentioned, e.g. SpanData to something in scala.

Closing because tried to do some magic. I wanted to amend my commit email to not be work email. Ended up fudging it up. Luckily not a lot of stuff to manually copy over.

Will keep my draft PR private to my fork for now.
https://github.com/chr12c/otel4s/pull/1


Warning  FailedScheduling   (x7 over 26s)  default-scheduler  0/5 nodes are available: 5 Insufficient cpu

Hello,
I have setup a k8s cluster with the following:
$ kops validate cluster
Using cluster from kubectl context: es.k8s.local

Validating cluster es.k8s.local

INSTANCE GROUPS
NAME                    ROLE    MACHINETYPE     MIN     MAX     SUBNETS
bastions                Bastion t2.micro        1       1       utility-eu-west-1a
data                    Node    t2.medium       2       2       eu-west-1a
master-eu-west-1a       Master  m3.medium       1       1       eu-west-1a
nodes                   Node    t2.medium       2       2       eu-west-1a

NODE STATUS
NAME                                            ROLE    READY
ip-172-20-39-51.eu-west-1.compute.internal      node    True
ip-172-20-40-39.eu-west-1.compute.internal      node    True
ip-172-20-43-105.eu-west-1.compute.internal     node    True
ip-172-20-44-5.eu-west-1.compute.internal       node    True
ip-172-20-62-224.eu-west-1.compute.internal     master  True

Your cluster es.k8s.local is ready

so I have one master, 2 nodes and 2 data nodes
when I try to add the es-client, this fails, with the following error:
$ kubectl describe po es-client-7b59bd6d64-vmvv5
Name:           es-client-7b59bd6d64-vmvv5
Namespace:      default
Node:           <none>
Labels:         component=elasticsearch
                pod-template-hash=3615682820
                role=client
Annotations:    kubernetes.io/limit-ranger=LimitRanger plugin set: cpu request for init container init-sysctl
Status:         Pending
IP:
Controlled By:  ReplicaSet/es-client-7b59bd6d64
Init Containers:
  init-sysctl:
    Image:  busybox:1.27.2
    Port:   <none>
    Command:
      sysctl
      -w
      vm.max_map_count=262144
    Requests:
      cpu:        100m
    Environment:  <none>
    Mounts:
      /var/run/secrets/kubernetes.io/serviceaccount from default-token-nb2zw (ro)
Containers:
  es-client:
    Image:  quay.io/pires/docker-elasticsearch-kubernetes:6.2.2_1
    Ports:  9200/TCP, 9300/TCP
    Limits:
      cpu:  2
    Requests:
      cpu:      2
    Liveness:   tcp-socket :transport delay=0s timeout=1s period=10s #success=1 #failure=3
    Readiness:  http-get http://:http/_cluster/health delay=20s timeout=5s period=10s #success=1 #failure=3
    Environment:
      NAMESPACE:     default (v1:metadata.namespace)
      NODE_NAME:     es-client-7b59bd6d64-vmvv5 (v1:metadata.name)
      CLUSTER_NAME:  es
      NODE_MASTER:   false
      NODE_DATA:     false
      HTTP_ENABLE:   true
      ES_JAVA_OPTS:  -Xms256m -Xmx256m
      NETWORK_HOST:  _site_,_lo_
      PROCESSORS:    2 (limits.cpu)
    Mounts:
      /data from storage (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from default-token-nb2zw (ro)
Conditions:
  Type           Status
  PodScheduled   False
Volumes:
  storage:
    Type:    EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
  default-token-nb2zw:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  default-token-nb2zw
    Optional:    false
QoS Class:       Burstable
Node-Selectors:  <none>
Tolerations:     node.kubernetes.io/not-ready:NoExecute for 300s
                 node.kubernetes.io/unreachable:NoExecute for 300s

Events:
  Type     Reason            Age                      From               Message
  ----     ------            ----                     ----               -------
  Warning  FailedScheduling  <invalid> (x13 over 2m)  default-scheduler  0/5 nodes are available: 5 Insufficient cpu.

what am i missing?

0/5 nodes are available: 5 Insufficient cpu. you don't have enough CPU available in your Kubernetes nodes.

@pires what ec2 node size would you recommend for a test setup?

Honestly, I don't have an answer for that :(

Hi @nkhine, I think you should start with the t2.large for the nodes and scale it out depending of your consumption.

You can exec cat /proc/cpuinfo to check cpu.
May be your machine's Insufficient cpu, Because :


Bump to polkadot-v0.9.16

Not required anymore (bumped to version v0.9.24 in the mean time: https://github.com/pendulum-chain/substrate-stellar-sdk/pull/8)

I reopened the branch because it is required by spacewalk. The dependency in Spacewalk of this branch should be removed and this PR be closed without merging.


pull request for e2e test at 01/26/2017 20:56:10

:white_check_mark: Validation status: passed
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.


init.pp: use the postfix default for mydestination

It is less surprising if the puppet module's default configuration matches a standard default postfix configuration.

I understand the idea, but I'm not really keen on changing defaults. You can easily change that value in hiera with postfix::mydestination in your common data file.

Why would the module defaults be different from standard Postfix defaults though? That’s a bit weird.

In this particular case, not having localhost in $mydestination breaks delivery of most system-generated mail going to root@localhost.

Obligatory comment to keep this alive.  stale-bot is gross.

I missed this in release 3.0.0 #331 :(

A breaking release was merged to remove Puppet 6. Let's not miss the opportunity again and merge this!


Fix Travis CI build failure

From the logs:

The command "xcodebuild -scheme FadeButton -project FadeButton.xcodeproj -sdk iphonesimulator -destination 'platform=iOS Simulator,name=iPhone 7,OS=10.0'" exited with 65.

Research what's going on and fix it.

Fixed in #20


Add leave field

This PR adds functionality to accept a leave argument (l/LEAVE).
Functionality has been added for add, edit.
Testing has been added for this new leave argument.
User guide has been updated with usage instructions for leave.
This PR closes issues:
#40 , #41.

Not all of checkstyle have passed.


coinstring: use createHash directly

Meh. I'm not sure about this. What's the payload on including full crypto? Is it really a big deal if using something like Google Closure compiler (remove unused JS)?

The payload is some 300k in a typical browserify build.
On 5 May 2015 01:39, "JP Richardson"<EMAIL_ADDRESS>wrote:

Meh. I'm not sure about this. What's the payload on including full crypto?
Is it really a big deal if using something like Google Closure compiler
(remove unused JS)?
—
Reply to this email directly or view it on GitHub
https://github.com/cryptocoinjs/coinstring/pull/5#issuecomment-98757380.

For what its worth,  in my builds,  the closure compiler was not able to remove the unused JS associated with this.

Okay, so if a person is going to be concerned about payload size, why not just go all of the way and use sha256 and ripemd160 algorithms directly since createHash pulls in unnecessary hashes?

@jprichardson because they don't exist as separate packages?   Maybe that is something we should fix upstream.
I think  sha256 = require('sha256') would be great,  where the sha256 module pulls in the common code that exists in createHash rather than the other way round.
But,  that requires a bit more work,  for a much smaller pay off compared to what we can get right now with this change.
No doubt, that is more ideal though.

because they don't exist as separate packages?

Umm....

https://www.npmjs.com/package/sha256
https://www.npmjs.com/package/ripemd160

They've existed for 1.5 years :p

@jprichardson but not using the same streamable API.   We can definitely change that though.

@jprichardson good to merge

Rebased.  Sorry about the mess around @jprichardson.

@jprichardson the tests are passing, coveralls is dying for some unknown reason though.

Alright dude, published.


Streaming echo server is very slow

Hi,
I am testing the echo.rs from examples and it seems that when I post a 60KB file into /echo it takes more than 1 second. The same request to /echo/reversed takes 5 milliseconds.
> time curl -i -v "http://<IP_ADDRESS>:3000/echo" -XPOST --data-binary @test.html 0.000u 0.006s 0:01.00 0.0%      0+0k 0+1io 0pf+0w
With buffering:
> time curl -i -v "http://<IP_ADDRESS>:3000/echo/reversed" -XPOST --data-binary @test.html 0.000u 0.005s 0:00.00 0.0%       0+0k 0+1io 0pf+0w
I'm using:
rustc 1.44.0
hyper = "0.13"
Is there any reason for this?
Thanks!

It seems to be related to Expect: 100-continue client header curl sets and waits for HTTP/1.1 100 Continue before it starts sending the file contents.
Is hyper dealing with 100 continue?
Thanks!

It is supposed to send 100 Continue responses. It does wait until the user tries to poll the request body. Since the echo server sends a response immediately, before polling the body, the 100 Continue cannot be sent afterwards. Though, curl could consider that the body is accepted if the response is a 200.

Yes, that's because in the reversed case, it has to poll the request body before it can return a Response. In the streaming examples, it returns a Response before polling the body.
The relevant spec is in RFC 7231 Section 5.1.1.
Relevant:

A server that responds with a final status code before reading the
entire message body SHOULD indicate in that response whether it
intends to close the connection or continue reading and discarding
the request message.

Which hyper does by not sending Connection: close. curl could use that to realize it doesn't need to keep waiting 1 second, since once it receives the 200 OK, it is now impossible to receive a 100 Continue.

Makes sense, thanks!
So we can consider this as a bug in curl client and not something to do with hyper.

Possibly. curl could chose to change its behavior. But the echo server can also be changed slightly, to poll for a chunk of the body first, before responding. This would send a 100 Continue. 🤷


reduces stripped JRE to what zipkin needs.

Before, we added extra modules that are only used by our demo images. This turns out a fool's errand as recent Elasticsearch needs even more than before (e.g. kerberos stuff). Instead, we will switch the image to only pick what's required in the zipkin builds. Ancillary images can use the full dist.

thanks for the look!


Fixes mismatched anonymous module bug with require.js

Fixes #15

To use, here is the config line you need for require.js:
require.config({
    paths: {
        store2: 'bower_components/store2/dist/store2'
    }
});

It would probably be best if you add a test for this also, to ensure it does not get reverted.

Because of this (and this.define passed in the closure) it is now broken with ember-cli and ember-cli-node-assets... I would provide a patch, but I don't understand the change... How is it supposed to work? Now it complains for not having a named store2 amd module, which makes sens.

I don't know, and i don't have time to figure this out. James King, can you chime in here? I'm so sick of these varying module systems.

@tchak this patch should name the module as 'store2' (the same as the JS file). I don't know how ember works, maybe it's expecting a different name?

@Jamesking56 I should not have brought ember in here, it has nothing to do with the problem. ember-cli can work with either a standard requirejs named export or an anonymous one. Providing named exports is rather unusual in my experience. But if you want it to be a named export as store2 I think the correct expression would be define('store2', [], function () {}). Your patch is to make it define(['store2'], function () {}). I am not an expert on requirejs but this means to me that it depends on store2 which is undefined. I am not sure what is going on here and how it can work for you.

@nbubna I understand your pain. Sorry for the truble and thanks for your work.

Well, I've tested this patch and it seems to work perfectly fine in my Marionette / Backbone SPA.
What happens when you try my fork in Ember?

In fact, I'll edit my fork now and try your way. There's some articles online that suggest the way you're describing as being the more accepted and 'correct' way.

Also @nbubna the existing AMD test should test for this.

Any progress on this @Jamesking56 ? Should we change to the way @tchak recommends?
And "should test" as in "should already" or "should be made to"?

I've made the change my in fork (in master) for @tchak to test, shall I create a new PR?
And what I mean is the AMD test should already cover testing this (it already exists in your repo)

Yeah, go ahead and make the PR. I hope it works for @tchak , but if not, at least it appears to be the more accepted/correct/whatever way of doing it.

Alright, i just went ahead and made the change. The AMD test passes, and i hope it helps. :)

Oh, release version is 2.5.0


[feat] Allow flexible node allocation to select nodes in any state

Fixes #1032

@teojgo Can you please fix the title of the PR?

This PR is mixed with commits from other PRs. Can you fix it?

Yes I made a mistake when pushing because it have forked from another branch.

Hello @teojgo, Thank you for updating!

In the file reframe/core/schedulers/__init__.py:

Line 403:80: E501 line too long (86 > 79 characters)

Do see the ReFrame Coding Style Guide

Codecov Report

Merging #1374 into master will increase coverage by 0.08%.
The diff coverage is 98.36%.

@@            Coverage Diff             @@
##           master    #1374      +/-   ##
==========================================
+ Coverage   91.66%   91.75%   +0.08%
==========================================
  Files          83       83
  Lines       12673    12709      +36
==========================================
+ Hits        11617    11661      +44
+ Misses       1056     1048       -8

Impacted Files
Coverage Δ

unittests/test_schedulers.py
93.78% <97.36%> (+1.96%)
:arrow_up:

reframe/core/schedulers/__init__.py
98.83% <100.00%> (ø)

reframe/core/schedulers/local.py
100.00% <100.00%> (ø)

reframe/core/schedulers/slurm.py
57.54% <100.00%> (+1.72%)
:arrow_up:

reframe/frontend/cli.py
78.15% <100.00%> (+0.17%)
:arrow_up:

reframe/core/meta.py
96.77% <0.00%> (-3.23%)
:arrow_down:

Continue to review full report at Codecov.

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

I can still see stale changes from the constraint PR. Plus there is another error that exists also in master most probably. If you pass --flex-alloc-nodes without an argument, it will complain, whereas it should just assume idle.

But isn't the point of having it to pass an argument. There is no default if the argument is used. Should I put one?

@teojgo I think we say that the default is idle and here we imply that passing an argument is optional, which is not the case as it is now.

Yeah not using the option at all implies idle

But if you use the option an argument is required

Ok, now I get it. But we should still change the documentation of the command line option:
--flex-alloc-nodes=POLICY
blah blah blah

If this option is not specified, the default allocation policy for flexible tests is 'idle'.


MSAL-React: Error when logging out user

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.20.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.1.2
Description
The full application is wrapped with the MsalAuthenticationTemplate-component, as the application is not available for not-authenticated users.
When trying to sign out (using the ClientApplication.logoutRedirect() function)  we are experiencing some unexpected behaviors, even though the logout process does indeed work.
Right before getting redirected to the logout page, we run into the following error:
BrowserAuthError: no_account_error: No account object provided to acquireTokenSilent and no active account has been set. Please call setActiveAccount or provide an account on the request.
We assumed that providing an AccountInfo object inside the EndSessionRequest (in the logoutRedirect-function) would fix the problem, but the issue continued to appear.
We then added an acquireTokenSilent to the sign-out process after finding a solution to a similar issue (link) ... but the same error still presented itself.
In the end, we also tried setting an active account, right before logging out, but the error still accrued.
Are we missing something obvious here?
MSAL Configuration
{
    auth: {
        clientId: "***",
        authority: "https://login.microsoftonline.com/***",
        redirectUri: "/", // index page
    },
    cache: {
        cacheLocation: 'sessionStorage',
        storeAuthStateInCookie: false,
    },
};

Relevant Code Snippets
index.tsx:

const renderApp = () => {
    return ReactDOM.render(
        <MsalProvider instance={publicClientApplication}>
            <App />
        </MsalProvider>,
        document.getElementById('root'),
    );
};

All application is wrapped in this component:
const AuthenticationHandler: FunctionComponent = ({ children }) => {
    const { accounts } = useMsal();
    const authRequest: RedirectRequest = { ...loginRequest, account: accounts[0] };
     return (
        <MsalAuthenticationTemplate
            interactionType={InteractionType.Redirect}
            errorComponent={ErrorComponent}
            loadingComponent={LoadingComponent}
            authenticationRequest={authRequest}
        >
            {children}
        </MsalAuthenticationTemplate>
    );
};

// signOut function

export const signOut = async (msalInstance: PublicClientApplication, account: AccountInfo) => {
    const token = await acquireTokenSilent(msalInstance, account);
    const activeAccount = msalInstance.getActiveAccount();
    if (!activeAccount) msalInstance.setActiveAccount(account);
    const logoutRequest: EndSessionRequest = { idTokenHint: token.idToken, account };
    msalInstance.logoutRedirect(logoutRequest).catch((e) => {
        console.error(e);
    });
};

const acquireTokenSilent = async (msalInstance: PublicClientApplication, account: AccountInfo) => {
    const request: SilentRequest = {
        ...loginRequest,
        account,
    };

    const authResult = await msalInstance.acquireTokenSilent(request);
    return authResult;
};

Identity Provider
Azure AD / MSA
Source
External (Customer)

@AndreaEsposit This is happening because acquireTokenSilent is being called somewhere in your app after the logout has cleared the cached accounts. Most likely this is caused by one of your components re-rendering when the logout happens. Make sure all your calls to acquireTokenSilent check that 1. at least one user is signed in and 2. the inProgress variable is 'None'.
You do not need to call acquireTokenSilent or setActiveAccount before the logout.

That makes sense, thank you. I will add those checks and close the issue if this solves our problem (I'm confident this will fix the problem, Might take some hours for me to test this though)

It was exactly as @tnorling mentioned, some other components were trying to call acquireTokenSilent with clients. length = 0.
By adding a simple check, we avoid the error, I am however curious to know if there Is a way to check the inProgress variable via the PublicClientApplication/IPublicClientApplication objects.


Add unit tests for CPrivateSend::IsCollateralAmount

This adds a test for CPrivateSend::IsCollateralAmount
Most of the other PrivateSend features are difficult to do since they are wallet based, but I am looking into that.
Also,  the indentation of test_dash.cpp has been adjusted since I had to adjust the file to get the test to work. (This is in line with how bitcoin does formatting fixes, but I guess I could revert that if need be)

Can we add 16 milestone?


Unable to upload images through API request

Bug description
I´m trying to automate the upload of evidences in each finding using the V2 api. The request path is POST
/findings/{id}/files/. The problem is that I´m unable to upload images due to the encoding type of the form. I have tried both via web (/api/v2/doc/) and via cURL as you can see in the requests below.
└─$ curl -X POST "https://example.com/api/v2/findings/464/files/" -H  "accept: application/json" -H  "Content-Type: multipart/form-data" -H  "Authorization: Token TOKEN" -F "title"="XSS de prueba" -F<EMAIL_ADDRESS>{"file":["The submitted data was not a file. Check the encoding type on the form."]}

└─$ curl -X POST "https://example.com/api/v2/findings/464/files/" -H  "accept: application/json" -H  "Content-Type: multipart/form-data" -H  "Authorization: Token TOKEN" -F "title=XSS de prueba" -F<EMAIL_ADDRESS>{"file":["The submitted data was not a file. Check the encoding type on the form."]}

└─$ curl -X POST "https://example.com/api/v2/findings/464/files/" -H  "Content-Type: multipart/form-data" -H  "Authorization: Token TOKEN" -F "title=XSS" -F<EMAIL_ADDRESS>{"file":["The submitted data was not a file. Check the encoding type on the form."]}

Several files have been tested and the error is the same always.
I have searched for this error and I think is a problem related with Django but, as I was unable to find where this request is handled in the server, I couldn´t move forward.
Steps to reproduce

Send a POST request as is shown in /api/v2/doc/

Expected behavior
I expect to upload the file (in this case images) to the finding I pass as a parameter in the URL
Deployment method (select with an X)

[ ] Kubernetes
[X] Docker
[ ] setup.bash / legacy-setup.bash

Environment information

Operating System: Linux
DefectDojo Commit Message: [2021-02-05 22:45:34 +0100] f567f53a: Merge pull request #3806 from madchap/fix-diagram-master [ (HEAD -> master, origin/master, origin/HEAD)]

Screenshots (optional)
Here is an image with the error when the request is done through a browser

@Maffooch Did you manage to find a way to submit the files during your PR testing?

Hi, have you been able to find the reason of this issue?

#5509

merged for release 2.5.0


Fix installing linux upstream kernel with -CustomKernel

Tested on Ubuntu 16.04, Ubuntu 18.04 and CentOS 7.5

@lizzha what is the time of such a build on Ubuntu and Rhel?
usually, the kernel build can take a very long time if not properly optimized.
please check the docs here and make sure that the kernel build is optimized as much as possible:
https://github.com/LIS/lis-pipeline/blob/master/docs/Configure-RhelKernelBuilder.md
https://github.com/LIS/lis-pipeline/blob/master/docs/Configure-UbuntuKernelBuilder.md

@ader1990 on a D4_v3 VM, with make -j4, on Ubuntu the total kernel build and install time was about 35 minutes, while on CentOS 7.5 the time was about 55 minutes.
I tried to change the make thread to 8, which is 2 times of the core count, the build time doesn't get reduced, even a bit longer.
So I updated the script to set the make thread number same as the core number, and also use ccache as the docs suggest.

The logic for other branches are not changed, so I tested the following -CustomKernel value for sanity check

a local rpm package
a public URL points to a deb package

@ader1990 on a D4_v3 VM, with make -j4, on Ubuntu the total kernel build and install time was about 35 minutes, while on CentOS 7.5 the time was about 55 minutes.
I tried to change the make thread to 8, which is 2 times of the core count, the build time doesn't get reduced, even a bit longer.
So I updated the script to set the make thread number same as the core number, and also use ccache as the docs suggest.

the ccache won't help on the first build, unfortunately. the only thing that helps is the core count (but only if you have an excess of 16 or 32 cores). what helps a lot is the parallel compression (in the docs).
you can use a different vm size (minimum of 16 cores and premium storage) plus the pigz hack -- I think that can get you to 15mins-20mins.

I have tested with multiple deb kernel files at once on hyperv and it works as expected.
@adriansuhov please make sure this change does not affect the azure kernel functionality.
@lizzha let me know when the ccache is removed (it does not bring any value on cold build) and when you consider it as final patch (you can use the no merge label until this is ready).

@ader1990 thanks.
I would keep the ccache, as it speeds up recompilation, which will be needed for developers.  I'm doing some test with parallel compression, will let you know when the PR is ready for merge.

@ader1990 I tested the parallel compression, but it doesn't show improvements in built time. So I would keep the patch as-is.
Could you please approve and merge?

@ader1990 I tested the parallel compression, but it doesn't show improvements in built time. So I would keep the patch as-is.
Could you please approve and merge?

sure, we can revisit this build kernel thing when we do the general refactor, so that we do not need to keep two separate code bases of kernel build -- one in lisav2 and one in lis-pipeline.

@ader1990 I tested the parallel compression, but it doesn't show improvements in built time. So I would keep the patch as-is.
Could you please approve and merge?

I am waiting @adriansuhov sign off, to make sure the azure kernel pipelines are in order after merge, will merge asap after his sign off, thanks.


Rework hash schemas

This branch depends on https://github.com/dry-rb/dry-types/pull/211
This is a promising start, the end goal is discarding hash schemas. I removed most of the code that relies on inheritance, the next step is removing inheritance altogether. I plan to provide backward compatibility for hash schemas, however, the main point is to provide enough flexibility for building custom-behavior schemas without introducing a ton of barely compatible options. At the moment, I think we'll have to separate weak/symbolized schemas from other types.

@solnic this is ready to go to master, I replaced all the schemas we have with a schema builder for backward compatibility (and besides that, it was easier than rewriting from scratch). The PR doesn't break anything beyond https://github.com/dry-rb/dry-types/pull/211 which is a reasonable breaking change for default types. My plan is to provide a new flexible yet concise and simple API for building schemas. I'd also deprecate the old schemas as well


Simple interaction model

Is it possible to define a model somehow that will be equivalent to:
k = GPflow.kernels.RBF(200)
k2 = k + (k*k)
m = GPflow.gpr.GPR(Y, X, kern=k2)

I would like to avoid specifying three separate kernels, e.g., k2 = k_1 + (k_2 * k_3), because then I will have three different length scales. I am under the impression from #322 that this cannot be done yet.

how about something like this?
class InteractKern(GPflow.kern.Kern):
    def __init__(self, k):
        super().__init__()
        self.k = k

    def K(self, X, X2=None):
        return self.k(X, X2) + self.k(X, X2) * self.k(X, X2)

    def Kdiag(self, X):
        return self.k.Kdiag(X) + self.k.Kdiag(X) * self.k.Kdiag(X)

k = GPflow.kernels.RBF(1)
k2 = InteractKern(k)

This code is not tested, and is off the top of my head. I think you could generalize to any simple function, instead of just k + k*k quite easily, like this:

class InteractKern(GPflow.kern.Kern):
    def __init__(self, k, f):
        super().__init__()
        self.k, self.f = k, f

    def K(self, X, X2=None):
        return self.f(self.k(X, X2))

    def Kdiag(self, X):
        return self.f(self.k.Kdiag(X))

def f(K):
    return K + K * K

k = GPflow.kernels.RBF(1)
k2 = InteractKern(k, f)

Of course this is only valid a limited set of functions f: addition and multiplication are fine.


Produce per-state report on connection types

Data will now be updated every Tuesday. Future work to update stats.html to be able to use it.


Change .jsnb format to resemble .ipynb more.

Here's an example .ipynb -file
The .ipynb -files have a good structure for diffing etc by having multiple lines inside the JSON. The source for an code block is an array of lines as strings.
This is example .jsnb -file: [{"kind":1,"language":"markdown","value":"**JavaScript support**"},{"kind":2,"language":"javascript","value":"1 + 2 + 3"},{"kind":1,"language":"markdown","value":"**Function definition support**"},{"kind":2,"language":"javascript","value":"function add(x, y) {\r\n  return x + y;\r\n}"},{"kind":2,"language":"javascript","value":"add(1000, 337)"}].
.jsnb file doesn't have any linebreaks. Additionally source is a single string with line breaks done with \r\n. This means even a single character change in code or markdown will result in really unreadable diff.
Changing the format will obviously risk breaking existing notebooks, so this is better done sooner than later.

It's a good point.
But diffing notebooks using their text formats also feels wrong. VS Code has native notebook diffing support, which can be used despite any underlying data format:

I'll close it for now.

Ok. I put priority in normal git's diff and diff of tools like github, which are very difficult to read without line breaks, but priorities differ.


Small clean up and added some tests

Removed rethrowing exceptions (fixes https://github.com/fpanaccia/Wkhtmltopdf.NetCore/issues/29)
Very minor refactorings in some places.
Added some tests

@fpanaccia  can you please fix  AppVeyor build  and tests to it?

Hi!, i had planning to delete AppVeyor and use github actions...i just did the upload...i will add the testing in the weekend

@fpanaccia  Thank you!

@fpanaccia  can you somehow setup actions to be run only on main repo, not the forks?

it seems that there is a workaround....but i think i gonna remove the action and use azure devops


[WIP] Feat/mint

@moon-strider could you please add the description! remember the external minecraft-interface repo which is an integral part of this PR


Issues with time, mass, and tselecopts

Mass was being passed to rungtd1d but mass wasn't an input for rungtd1d. Was running into a problem where tselecopts wasn't defined so it wouldn't work. Not sure what that does though. For the altitude 1-D case had to select the first element of dtime or assert isinstance(t,(datetime,str)) wouldn't work

The issue with the time actually had to do with the way that msis_demo.py works so I changed it back

This was a collection of careless mistakes. I made the parameters MASS and TSELECOPS globals because I don't foresee a need to change them. They allow turning on/off various computations inside MSIS.


Update vite.mdx

added step to update index.css so that the new Shadcn CLI tool with correctly parse the tailwind configuration. This will prevent an error from being thrown.

This will closes issue #4677

@dmcleish91 it looks like the new version of Vite (afaik), uses src by default. We added a fix in https://github.com/shadcn-ui/ui/pull/4741. Does this work? Please reopen if not.


Installation inside a Docker container fails

FROM ubuntu:16.04
RUN apt-get install -y python3 python3-pip > /dev/null
RUN pip3 install localimport

Results in
  Downloading localimport-1.5.2.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-jfrxurzp/localimport/setup.py", line 20, in <module>
        long_description = restify(),
      File "/tmp/pip-build-jfrxurzp/localimport/setup.py", line 14, in restify
        return fp.read()
      File "/usr/lib/python3.5/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 28: ordinal not in range(128)

Using
ENV LANG C.UTF-8

in the Dockerfile makes it work, but we should probably just open the README file with a proper encoding.


Conflicting information in FAQ

The FAQ mentions both:

Indium is an addon mod for Sodium

…and also:

Indigo disables itself when Sodium is installed.

Which one is it? The rest of the FAQ sounds like I should use both Sodium AND Indigo, but this is confusing.

Indigo is not Indium - though Indium is adapted from Indigo. Technically when you use Sodium and Indium with Fabric API there are 2 implementations of the Fabric Rendering API installed - both Indium and Indigo (included in Fabric API) - but Indigo is disabled when Sodium is installed, as it is not compatible; Indium is an alternative implementation that works with Sodium.

Oh, that is confusing naming! Could there maybe be a little hint at that in the FAQ, please?


Quotes are removed in data uri during concatenation

Hi,
I've just encontered a bug during concatenation of files containing inlined SVG
div {
    background-image: url("data:image/svg+xml;utf8,<svg xmlns='http://www.w3.org/2000/svg' version='1.1' width='32' height='24' viewBox='0 0 32 24'><polygon points='0,0 32,0 16,24' style='fill: rgb%2851, 51, 51%29'></polygon></svg>");
}

produces
div {
    background-image: url("data:image/svg+xml;utf8,<svg xmlns=http://www.w3.org/2000/svg version=1.1 width=32 height=24 viewBox=0 0 32 24><polygon points=0,0 32,0 16,24 style=fill: rgb%2851, 51, 51%29></polygon></svg>");
}

You'll notice that all single quotes are removed after concatenation, which results in bad SVG. Same problem appears with double quotes.
This problem disappears when I use the option rebaseUrls : false. Is this the correct behavior ?
When using base64 encoded resource, there is no problem.
I'm using gulp-concat-css 2.2.0

So what are any other options?
Is this going to be fixed?

Ran into the same issue. Problem is within the rework-plugin-url package: https://github.com/reworkcss/rework-plugin-url/issues/7

I lost several hours today trying to find what happened with my svg background images.
This should have been solved since many years.

Still not fixed which is why I need to switch to gulp-concat then :/


Checks that quantized Tensor's elements have type tuple.

Fix for the issue #5568

Can one of the admins verify this patch?

I signed it!

@tensorflow-jenkins test this please


Refine Project Charter

A project charter was created to set expectations for the upcoming project. This ticket documents some work to update the charter.
Description:

[ ] Add 'Press' to key stakeholders
[ ] Split out stakeholders into those who create economic value & impact and those who need to understand what that impact is (industry vs public)
[ ] Update project outcomes to include analysis of problem and leverage points in system

Hey @LindsayMacfarlane, here are the changes I'm proposing to the charter. Any feedback on the above?

I have slight reservations about adding press as stakeholders. There's probably no issue in identifying, but we will have to discuss if and how discovery can include them.


Merge core.bus and core.mqtt components

Merging these two components would simplify the interface used for publishing and receiving MQTT messages and remove the need of importing two core modules to do one operation.

Planned in version 0.3.1.


Remove any warning/alert messages from the outputs

Sometimes the cli outputs extra information which can break deployment pipelines if not treated correctly.

When I run realm-cli secrets list --app feature-mule-20792-qewgs -f json Im expecting that the output will only contain a single object in which I can find my secrets. But as you can see in the image above, the output of this command printed two extra objects, which, if not ignored, can break deployment pipelines. And since these objects appear only once, there is no reason for treating them in our scripts. In the image below you can see that in a second run those extra objects disappeared

My suggestion is to always have an expected output for each command, that also never changes in a major version. And whenever you change the schema of the outputs, you should release another major version of this cli. If you release a minor with a change in the schema, and developers use dependabot and koadiak to automatically update the version of the cli in their build images whenever a new minor is released - like we do - this would break their builds.
If you really want to keep these alert messages, add them as an extra key called messages inside the expected json, instead of printing as a separate object.

Currently, our json output is a very naive interpretation of the CLI's standard output... it simply structures the output that would otherwise print visually for a regular/manual user.
The messaging that shows here has been included in the CLI since our "beta.5" days and has remained consistent since.  And as far as "expected output" goes, what I would recommend keying in on is the level: "info" message output, as that will consistently be a single object for the output of this particular command.
What I can bring back to our product team though is a suggestion for some --concise/--quiet/--verbose flag to help manage what output gets delivered (it's been a regular topic of discussion for us).  The problem we've always run into is that the preferred default behavior generally differs based on the type of user you user: would prefer to opt in to --verbose in a CI/CD environment yet we want regular users to have to opt out of extra information when they're simply using the tool manually from the terminal


Update xpp-data-primitive.md

Fix typos and format text

:white_check_mark: Organization membership check passed
Requester is not a member of the MicrosoftDocs organization.