Support spill over when writing sorted data in hive buckets

Most applications can easily generate partitioned data files and upload them directly to s3. Presto can directly query these files. However, for performance reasons, it is often highly desirable to bucket the data. Further, if the bucketed data is sorted, the performance will be even better.
For example, let us say that we have a source table data with schema (CREATE TABLE data (value string )) and we want to create an optimized version of the table (data_bucket) which has bucketing enabled (CREATE TABLE data_bucket  (value string) WITH (external_location='s3n://bucket-name/',format = 'orc',bucketed_by=ARRAY['value'],sorted_by=ARRAY['value'],bucket_count=1024) )
Now if we try to run the query (insert into data_bucket (select * from data)) the query will likely fail if the cluster doesn't have enough memory. The error will resemble like this
Query exceeded per-node total memory limit of 8GB [Allocated: 7.91GB, Delta: 107.28MB, Top Consumers: {TableWriterOperator=7.83GB, ExchangeOperator=36.46MB, LazyOutputBuffer=32.13MB}]

This happens because Presto is sorting the data in memory before writing it to the target table.
It would be great if the spill over functionality is supported for hive table writing

Writing sorted bucketed tables already uses spilling. The data is written to sort temporary files on the target file system (related #2301 to allow using local disk). The sort buffer size is controlled by hive.writer-sort-buffer-size which defaults to 64MB. These files are then merged together, possibly in multiple passes.
The total distributed memory requirement will thus be 64MB * 1024 (bucket count) for sorting. You'll also need memory for the 1024 ORC file writers, which is primarily controlled by hive.orc.writer.stripe-max-size which also defaults to 64MB. Thus, in total, you would need around 128GB, which is less than the available memory on a single, typical 256GB machine.
Rather than reducing the buffer sizes, which have drastics effects on performance, I recommend adding a few more machines or using instance types with more memory. An 8GB per-node query limit seems low for a write-heavy workload. Remember that you're optimizing for throughput. If you use small buffers and have to spill everything, the jobs will run for much longer -- it's cheaper to use larger machines for less time.
I'm happy to discuss this further on Slack: https://prestosql.io/slack.html

awesome, thanks @electrum . I will play with the configuration options.


Proper user deletion

This pull request implements the feature that lectures/media can remain in the database when their editor/teacher deletes her/his MaMpf account. The account gets destroyed, but these materials (and only these) will be transfered to a new user who gets an archived flag.

 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.

Good point, I added the sanity check (users have to enter their password).


🚧 Dev ➾ Document Instructions for Using Beacon Wallet on a Sandbox

@### 🔗 Epic|Feature Link
https://ecadlabs.productboard.com/feature-board/5219572-taqueria-features/features/14699515
🚥 Definition of Done

[x] 🎯 Planning and Design
[ ] 🏎 Dev Implementation
[ ] 🏟 Peer / User Reviewed
[ ] 🏆 Validated and Accepted

✅ To Do
No response
🔭 Scope
🏎 In Scope:

[x] Writing the series of steps required to connect Beacon to a local Flextesa instance for use in dApp development
[ ] Code snippets or a concrete example of how to make this work

🙅♂ Out of Scope:

[ ] Planning for any wallet integration features
[ ] ANY development work on the Taqueria repo
[ ]

🧰 Work Description
Several team members have been unable to get Beacon wallet working with Flextesa in a dApp. In order to understand the friction points in the process and unblock team members, we need a clear and concise understanding of how to make Beacon work with Flextesa
If there is a blocker or issue that prevents this from being possible, clearly identify the issue and any possible solutions (of any complexity)
⚖ Acceptance Criteria
✨ Features:

[ ] None!

📚 Documentation:

[ ] Step by step instructions for configuring and connecting to Beacon in a Flextesa environment
[ ]

🎢 Test Plan
No response
🎆 Release Notes
No response
🐘 Sizing
0.5
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I've created some modifications in #1296 to ensure that users can interact with local sandboxes without CORS issues. No proxying services like Caddy are required, nor do we require any specifc interaction with matrix nodes.

Writing the series of steps required to connect Beacon to a local Flextesa instance for use in dApp development

Here are the summary of required steps necessary to connect your dapp to a local sandbox:

Start the sandbox
In your dapp:

// Import your taqueria config.json file so that we can dynamically
// determine the the sandbox label & RPC url
import taqConfig from "taq/config.json"

// Use Taquito and the Beacon SDK to add the sandbox network to your wallet
import { TezosToolkit } from "@taquito/taquito";
import { BeaconWallet } from "@taquito/beacon-wallet"

const Tezos = new TezosToolkit(taqConfig.sandbox.local.rpcUrl);

const wallet = new BeaconWallet({
  name: "Beacon/Sandbox Test"
});

Tezos.setWalletProvider(wallet)

wallet.requestPermissions({
      network: {
        type: 'custom' as NetworkType,
        name: taqConfig.sandbox.local.label,
        rpcUrl: taqConfig.sandbox.local.rpcUrl
      }

Use taq transfer to transfer funds from Bob to your wallet address.

You can retrieve your public key hash using
const current = await wallet.client.getActiveAccount()
console.log(current)

Code snippets or a concrete example of how to make this work

I created a demo dapp to showcase how you can connect to a sandbox:
https://github.com/ecadlabs/taqueria-beacon-sandbox

Further Insight
From a developer workflow standpoint, I think we're going about things the wrong way with regards to how we generate sandbox test accounts.
Some points to note:

Most developers will write dApps for users that will interact with their dApp using a Beacon Wallet implementation, such as Temple.
It should probably be anti-pattern to use the InMemorySigner, whether in development or production.  It might be ok for very rapid prototyping, but behind that, developers should really have tools that make it dead simple to get started with a Beacon Wallet implementation

Taking the above notes into consideration, Taqueria should seize the opportunity.
Understanding a dApp developer's workflow
With the above notes in mind, we should be encouraging developers to install a Beacon Wallet implementation, such as Temple and AirGap. Given how prominent software wallets are in the cryptoverse, I think there's a high likelihood that developers will already have a wallet installed and that they're familiar with. As such, "getting a wallet" should be their first step.
Now that a developer has a wallet, they can create accounts from within the wallet. For instance, the developer might have an account that represents their own identity, and then create some test accounts, such as Bob, Alice, and Bill.
Their next step should be to start a sandbox. This is where I think we need to do things differently. When you run taq start sandbox local, instead of generating accounts right away, we should ask to connect to their Beacon Wallet. Once connected, we can call the getAccounts() method using the Beacon SDK to present them a list of accounts.
We could then display something like this:
Found accounts: Mike, Bob, Alice, and Bill

Select an option below:
1) Fund all accounts with the same amount of Tz
2) Fund each account with a specific amount of Tz

Selection (1 or 2, default: 1): [  ]

If a developer selects 1 then they're asked for an amount. Otherwise they're prompted to enter an amount for each account.
The data collected above becomes the default template for any sandbox.

Continuing on the thought above...
Once a default funding template has been created for a sandbox, that replaces our means of declarative accounts. So the next time you start a sandbox, such as taq start sandbox localghost, it would use the template to instantiate the sandbox-specific accounts.


when i use react-native link react-native-device-info  show error

Summary

Version
1.5

Affected OS
android

OS Version
all

react
16.8.3

react-native
0.58.6

Current behavior

error:
Package name not found in XXX\android\app\src\debug\AndroidManifest.xml

But i have the file

Expected behavior
normal link but not

Please include all the information on the issue template

@mikehardy  I've made a few changes. Have a look

Hmm, might be that we have a problem linking on windows (you did not put the OS you develop on, but I am guessing based on direction of slashes in the path in th error). But no one else has reported so that makes me think this is something peculiar to your environment
Can you clone the repo and run the example? In general for any module I want to integrate I do that if they have an example, to make sure it runs and to see what sort of output to expect. You should try that to make sure.
Other than that, you might follow the manual linking steps to see what is missing - I am guessing it is in android/settings.gradle, but without seeing the contents of android/*.gradle and your full AndroidManifest.xml I'm unsure.
Examining the contents of those same files in the example vs the files in your project may get you to the answer

Ok, I'll try it first

oh！ deeply sorry， when i change implementation "com.facebook.react:react-native:+" , it  is  run ok ， i think it's not your problem， so sorry！


Head does not inject tags on server

I am using a custom Document, and it appears Head is not injecting the tags when rendering SS. If I disable js in the browser, I still can see the Head component render, but nothing appears in my document <head>

It's not exactly clear what you mean. Try to submit us a sample repo where we could reproduce it.

I was mistakenly using a <head> element instead of the <Head> component from next/document in my custom Document, which it seems was causing this error. All better now!


RFC: Replace 6.00.1x, PY4E, and CS50P for 6.100L Introduction to CS and Programming Using Python

Problem:

OSSU lists courses X, Y and Z that cover the same topics when fewer courses could suffice.
OSSU recommends course X to teach a topic, but there exists a higher quality course that covers the same material.

Duration:
1 month
Background:
6.00.1x Introduction to Computer Science and Programming Using Python is the recommended course from OSSU. Recently, the course had a few problems such as not being able to do the finger exercises properly and, recently, the autograder for the problem sets does not work.
MIT recently released 6.100L Introduction to CS and Programming Using Python on OCW. It includes:

Lecture videos and their corresponding lecture codes and notes.
Finger exercises for each lecture and their solutions.
Problem sets with separate test files.
Recitations for additional exercises with solutions.

Advantages:

Covers the same material as 6.00.1x, but gentler and longer.
Because the course is gentler and longer, students may not need CS50P or PY4E as a prerequisite.
The course is also very recent, so it also covers a more recent version of Python compared to 6.00.1x and PY4E.

Disadvantages:

6.100L might be a bit less organized than 6.00.1x, but the Calendar tab gives you the breakdown on what lectures the assignment covers and when the assignments are due.

Proposal:

Replace 6.00.1x, PY4E, and CS50P for 6.100L.

Alternatives:

Replace only 6.00.1x for 6.100L and leave both PY4E and CS50P as supplements for additional Python practice.

This looks like a plausible RFC. I am interested to hear how learners taking this course compare its pace to CS50P and PY4E.
From the 6.100L course syllabus:

This class is a full-semester version of 6.100A (formerly 6.0001 Introduction to Computer Science and Programming in Python). The material covered is the same, but the pace is slowed down. Our goal with this course is to give students who have never programmed the time to practice the concepts.

@etherealcomity We've been in an evaluation state of the intro to programming courses for quite awhile. The course page welcomes students to take one of the two candidate courses and to provide feedback in the RFC. I would welcome a PR that updates that course page to offer this as an alternative. May need to do some wordsmithing to explain clearly the different options.

I just watched the first few lectures of this course, and it seems very good to me. Good enough that we won't need Py4e/CS50P. Of course, that might be just because I have already done too many intro-CS courses, so we will need to wait from someone who is a beginner to actually find the truth.

6.100L might be a bit less organized than 6.00.1x, but the Calendar tab gives you the breakdown on what lectures the assignment covers and when the assignments are due.

We had the OCW version of 6.0001 earlier, and I don't think there were much confusion, and 6.100L is much more organized than that. It has the materials sorted out by lectures.
@waciumawanjohi Actually, I had a different but similar idea. Let's just remove CS50P, Py4E and 6.00.1x, add this course, and create a course page explaining that if people find it too difficult, they can do Py4E or CS50P or the Georgia Tech Intro Python course from the extra courses list to get an even gentler introduction, and comment on this RFC stating that they faced problem. If they can do it without any difficulty, then also comment on this RFC. Based on the feedback, we can then decide whether to remove the mention of the extra courses. Even just keeping it like that (without asking to comment on this RFC) seems reasonable to me.

Let's just remove CS50P, Py4E and 6.00.1x, add this course, and create a course page explaining that if people find it too difficult, they can do Py4E or CS50P or the Georgia Tech Intro Python course from the extra courses list to get an even gentler introduction, and comment on this RFC stating that they faced problem. If they can do it without any difficulty, then also comment on this RFC.

I would be comfortable making that switch in a month when the RFC period expires, assuming no great objections to this RFC.

Well I don't object to the RFC but I personally believe that archived version of MITx 6.00.1x too should still at least be available as a supplement. It's finger exercises are combinations of short programs, conceptual short questions and mcqs which are better aimed at building important concepts. Yeah a minor difference but at least I would consider it important. As I have worked across both simultaneously.

I'm just commencing and happy to try 6.100L out


Test for open files doesn't work when using log2 for bucket imput

hello,  when you start chia_plot with -u 10, then the test at https://github.com/madMAx43v3r/chia-plotter/blob/290b8a75b32a736b083990bcec24922cffa91966/src/chia_plot.cpp#L270   will test if 30 files can be opened instead of 1024 + 20.
Suggestion,  include sys/resource.h and replace that test block with setting it to size (open files) needed?
        {
                const int num_open_files = pow(2, log_num_buckets) + 20;
                std::cout << "num_open_files: " << num_open_files << std::endl;
                struct rlimit open_files_limit = {num_open_files, num_open_files};
                if (-1 == setrlimit(RLIMIT_NOFILE, &open_files_limit)) {
                        std::cout << "setrlimit failed, tried setting open files at: " << open_files_limit << std::endl;
                }
        }

chiapos is already doing something similar.

fixed and done, good catch


Fixes https://github.com/guerrerotook/securitas-direct-new-api/issues/175

I did a little bit of additional cleanup, but the important change is in check_code()

I made some changes to config_flow.py to allow arming and disarming without a code (the code was looking for an inexistent key and crashing).
I also did some minor organization of imports and constants for default values.
I hope you find this useful. Thanks for your work creating this integration.

Thanks for the PR @cantupaz , can you resolve your conflicts please?

I'll send a new PR in a little bit


Adding queues for communication between services

Research which queue service can be used. Candidates:

https://github.com/vsouza/docker-SQS-local
RabbitMQ (https://medium.com/swlh/async-messaging-with-kotlin-and-rabbitmq-d69df1937b25)

RabbitMQ will be used as it's a more standard solution than SQS-local


Constant data is emitted not in compliance with C89 standard

C89 permits only 509 characters in a logical source line:

w2c2 does not appear to limit line length. I am currently looking at emitted code that has a 173 kB long source line, which is definitely excessive.

Though there might be limits in the standards, is this an actual problem? Did you run into particular problems with a certain (old) compiler?

Did you run into particular problems with a certain (old) compiler?

I actually ran into an issue with code editors, which generally do not enjoy having a 175 kB long line. Even less can struggle with inputs like this if they're big enough!
That said, this is one of the most common compatibility issues with C89.

Though there might be limits in the standards, is this an actual problem?

In my view, if you claim adherence with C89, you need to actually follow the restrictions in the standard, or clearly document your deviation. Otherwise it is not a useful claim.


Add better Tag validation

Right now, it's possible to create a tag that looks like a tag that already exists, but it gets save with leading/trailing blank space, or a different case… and ends up with a different slug.
  It introduces issues on the UI/UX as well.  When I go to create an article it assumes I already know what tags are in there so if we introduce a base validation for uniqueness with some sanitize mechanism, the user/staff still has to wait for AR to bark back a validation error.
What if we did something using selectize/selec2 where you can do a first_or_create type pattern but with JS so you type stuff and the form fires ajax to the backend to live search existing tags, then allows you to create somehow (maybe a tertiary modal in line)?
I'm going to spike this a bit today but any feedback would be appreciated.

I think that there is opportunity to do some better UI around this, but I think that that's a separate/later concern.
I think first we should clean up the way Tags are save in the back end to avoid these dupes. Then sometime in the future, we can think through and implement a different UI.
As a design principle, we try to do everything we can in Rails/backend, instead of Javascript/frontend.

Ok this works, picking this back up as I've been overseas.

Shouldn't we validate with the slug since the sluged version of the tag name is lowervase, timed, and space are replace with dash, so we'll avoid a lot of missed case, tho ?

To cleanup the tags, we could write a one shot task that would merge de duplicate tags association in a single tag and remove the duplicate?

Ok, so I thought I'd write a script, but after some investigation and debugging it'd be overkill!
All articles tagged with the weird named tags are also tagged with the right tag, so we can simply delete all the Tagging mapping article with the weirdly named Tag and then delete those tags and everything should be fine.
This should do the job:
Tagging.where(tag_id: [2,3,4,5,6,7,8]).delete_all
Tag.where(id: [2,3,4,5,6,7,8]).delete_all

If you wanna know how I get those ids, here's what I did on my machine with the latest data dump:
    grouped_tags = {}
    Tag.all.map { |t| grouped_tags[t.slug]=Array.new }
    # group tags by slug
    Tag.all.map { |t| grouped_tags[t.slug].push(t.id) }

    duplicated_tags = grouped_tags.filter { |k,v| v.length > 1 }

duplicated_tags should give you 4 slugs (anarchism, veganism, inside-front, hardcore) with different ids associated. The last id (the greater number) of each slug is the right one. So I delete all the Tagging and Tag of the other ids.
Once this is done it should be alright since the Slug concern validate the uniqueness of a slug before creating a new one.
If you want, I could check to get access and validate that it's the same data in produciton before doing the operation, but it should be.

@just1602 Thank you. Sorry for the delay. I did the things in your comment above. 👍🏻


Fails to build class when enum string doesn't start with a letter

const themes = {
  a: '-theme-a'
};

Component.propTypes = {
  theme: PropTypes.oneOf(Object.values(themes))
};

Class generation from the above code will break when generating enum name for the string -theme-a. Error:
Cannot read property 'toUpperCase' of undefined

Fixed in 0.1.0


Is there any way to upload a pic file into flatnotes?

When i drag a pic into flatnotes in chrome. it's alway insert a large of garbled characters( is that base64 code of pic ?  i don't know)?.
I wordering is there any way to upload a pic file, and return back a url, maybe like  "./image/xxx.jpg", and then i can insert into markdown editor.

is that base64 code of pic ?

That's correct.
I appreciate it's not ideal and I would like to improve it at some point.  It's the 2nd most popular suggestion now and the one that affects my own personal use the most so I'm hoping to get onto it soon.


Add Phase of Moon app

Displays an image and a text description of the current phase of the moon. Accurate in both hemispheres.

I have read the CLA Document and I hereby sign the CLA

recheck

You can run the following to fix the linter issues locally, then you can commit the changes:
make format

I love this kind of app...
One thing, you don't need to project the time into any particular time zone to get unix out of it as UNIX time is based on the number of seconds from 1 Jan 1970 00:00:00 UTC, which is the same regardless of timezone.

I love this kind of app...
One thing, you don't need to project the time into any particular time zone to get unix out of it as UNIX time is based on the number of seconds from 1 Jan 1970 00:00:00 UTC, which is the same regardless of timezone.

Thanks - v new at this, so I'll amend here and that will come up in a future push

Does anyone have any suggestions as to why the build is failing? I'm outside my expertise here.

@alan-fleming There's a blank line at then end of the star file.  Delete that line.

@henryso I'm absolutely certain the previous build failed because the linter complained there wasn't a blank line at the end!
Oh well... learning is fun...

I think when I ran make lint on my system before committing it adds the line. I noticed last night when I ran make lint it adds it, so it may not like two empty lines?


Update README.md, change parameter names on string generator

Resolves Issue #7

One suggestion is that in one of the examples you can use .colour() instead of .color() to show that they can be used interchangeably.

@shadowtime2000 Ah yeah good call, knew I was forgetting something

Alright, readme has been updated, noticed I also accidentally removed the colour function, so I added that back in as well


Lookahead

Both ECMA-262 and python support negative and positive lookahead.

@alexanderkjeldaas: Sorry I've been dormant on this for so long.  Thanks for the PR.  Merging.


ap-3393: Add print button

What
Link to story

Added css to show/hide open guidance content when printing
Added pagebreaks for printing
Resized images for printing

Describe what you did and why.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Aside from my comments, this is ok.


Suggestion to improve efficiency.

You use hash to create id, I think that you can improve the efficiency of the rolebase system by creating an id serially 0..N and use a map node to id for the optimizations. Using a serial id can help improve the memory efficiency because you can put all the memory in vectors instead of maps.

The code has been refactored and optimized significantly since this, including as how described in the comment above. I'm going to go ahead and close since so many optimizations have applied over the last several months. We can open new issues if other opportunities emerge.


Remove dependency on behat/mink-browserkit-driver

I don't want to use Browserkit, I want to use Laravel Dusk to test browser interaction.  It would be great if this package did not rely on Mink at all.

ah nice idea. I also started https://github.com/alnutile/pickle to try and get the best of Gherkin and Dusk


Implement the Builder API

Desired behavior

Propose a Builder API to build a RoadGeometry. It should be used to generate RoadGeometries at build time via programatic descriptions or used by parsers of specific map formats, such as lanelets or OSM.
Victory condition
Define and implement the builder API that allows to build a RoadGeometry.

One comment that could affect Builder API and its implementation.
The process of the programmatically creating the road geometry can be approached via different ways:

Building lanes as they are added to the RoadGeometry. Which would allows to have intermediate states of the road geometry as it is built.

Adding all the instructions for the creation of the road geometry and then running the process of building it.
Like:

...
...
RoadGeometryDiagram rg_diagram;
Lane lane_a{id, line_string_a, line_string_b};
rg_diagram.AddLane(lane_a)
...
...
auto rg = RoadGeometryBuilder(rg_diagram);

I found the latter a better approach as the process of building would be easier and straight forward as the Lanes don't have to change their information about connections when new lanes are added because the diagram will have that information already. Also, parallelizing the building could be done to minimize the time of building.


Enhancement: Replace ethers.js and bignumber.js in Core and Common

Description
This Pr aims to replace:

Ethers.js with Viem
Bignumber.js with the browser native BigInt

Upon review of this PR I plan to update this branch with a new versioning type that will publish outside of our alpha flow in the effort to maintain separation.
This will allow testing of these packages in different build environments by our team. After testing we can also possibly, have some external customers test the new packages for feedback.
Requirements:

node version >=16.15.1

Benefits: ~258kb reduction + using the latest community driven , open source software (Viem)
Bundlephobia package analysis:

Current bundle analysis
Bundle analysis with removal of ethers/bignumber.js and addition of Viem

Currently build with the internal svelte demo/webpack 5
PLEASE NOTE- Checklist must be complete prior to review.
Checklist

[x] Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
[x] Check the box that allows repo maintainers to update this PR
[x] Test locally to make sure this feature/fix works
[x] Run yarn check-all to confirm there are not any associated errors
[x] Confirm this PR passes Circle CI checks
[x] Add or update relevant information in the documentation

Docs Checklist

[x] Include a screenshot of any changes (see docs README on running locally)
[x] Add/update the appropriate package README (if applicable)
[x] Add/update the related module in the docs demo (if applicable)
[x] Add/update the related package in the docs/package.json file (if applicable)

Hey! Was just curious why this PR was closed? Any issues?


ORC-58: Move code for reading rows from Reader to RowReader

This PR divides the current Reader class into Reader class and RowReader class
Reader class reads the footer and metadata from an ORC file.
RowReader class reads the rows from the file.

@omalley, please let me know your feedback. Thanks!

I might rename "Reader" something like "FileTailReader" but, in general, the refactoring makes sense. The schema can be read and columns metadata known before column selection is decided.

The ReaderOptions needs to split in half too:

ReaderOptions

setTailLocation
setErrorStream
setSerializedFooter

RowReaderOptions

include
range
throwOnHive11DecimalOverflow
forcedScaleOnHive11Decimal

Included review comments.
This patch is huge because I split the monolithic Reader.cc into various files

Reader.hh/Reader.cc: ReaderImpl and RowReaderImpl
Statistics.hh/Statistics.cc: ColumnStatisticsImpl, StatisticsImpl, Type*StatisticsImpl
StripeStream.hh/StripeStream.cc: StripeStreamsImpl, StreamInformationImpl, StripeInformationImpl
Options.hh: ReaderOptions, RowReaderOptions

@omalley any feedback ?

This is looking good. To compile on MacOS, I needed to replace some of the std::move with REDUNDANT_MOVE. See omalley:pr/41

Replaced with REDUNDANT_MOVE. valgrind does not show any memory leaks due to this patch.

I think we should have FileContents be a shared ptr so that if the user deallocates the Reader, the RowReader won't reach into deallocated memory.

I updated the patch to use a shared_ptr for FileContents.

Deepak,
We got a notice of a new problem from Coverity. Can you look at it?
** CID 173749:  Uninitialized members  (UNINIT_CTOR)
/c++/src/Reader.cc: 259 in orc::RowReaderImpl::RowReaderImpl(std::shared_ptrorc::FileContents, const orc::RowReaderOptions &)()

*** CID 173749:  Uninitialized members  (UNINIT_CTOR)
/c++/src/Reader.cc: 259 in orc::RowReaderImpl::RowReaderImpl(std::shared_ptrorc::FileContents, const orc::RowReaderOptions &)()
253         } else {
254           previousRow = firstRowOfStripe[firstStripe]-1;
255         }
256
257         ColumnSelector column_selector(contents.get());
258         column_selector.updateSelected(selectedColumns, options);

CID 173749:  Uninitialized members  (UNINIT_CTOR)
Non-static class member "rowsInCurrentStripe" is not initialized in this constructor nor in any functions that it calls.

259       }
260
261       const RowReaderOptions& RowReaderImpl::getRowReaderOptions() const {
262         return options;
263       }
264

@omalley
https://issues.apache.org/jira/browse/ORC-127
https://github.com/apache/orc/pull/77
Thanks.


meetup: use calendar instead of core

There has been previous work to remove the dependency on Core in favor of Base.
meetup was the last exercise depending on Core. Should this be merged we could remove Core and Core_kernel from the dependencies completely by migrating the test-generator to Base, too.

Very nice!
For the README update, this will probably break configlet, but let's fix that at a later time when we get configlet hooks into CI. The preferred way is to keep extra exercise-specific README information in a .meta/hints.md file that configlet copies into the README.


Version number 01  fails to load.

Description
Version number 01  seems to not load. Changing to 1 works.
Triton Information
Container 20.03
To Reproduce
Should be trivial to reproduce.
Describe the models (framework, inputs, outputs), ideally include the model configuration file (if using an ensemble include the model configuration file for that as well).
Expected behavior
Load  model with version number starting 0 as first digit.

We intentionally ignore version directory with leading zeros and log warning appropriately if that occurs. The reason is to avoid ambiguity on loading versions (i.e. load "01" or "1"?)

This may sound frivolous - what is the ambiguity between the numbers 01 and 1 , unless you are treating them as strings rather then numbers ?

Triton will treat them as numbers. With leading zeros, it is possible to have two version directories "01" and "1", and each directory has different model file. Triton sees both directories as version 1 and in such case it is not clear that which directory is the "correct" one to be loaded as version 1. By ignoring directories with leading zeros, we enforce that "1" will be loaded, and user should ensure the model file in "1" is the proper file for version 1.

Thanks  for the clarification - appreciate.


exclude tests (etc) from install via .gitattributes

add .gitattributes file with export-ignore to exclude tests (etc) from package zip downloaded by composer, a la https://blog.madewithlove.be/post/gitattributes/

seems pretty small and safe to merge, but it will make files that were previously available to consumers no longer available. i have a looming v3 planned. might as well wait until then.

Need update list of ignored after last changes

closing due to inactivity


fix typo in built-in modules section

This part of the lesson is about keyword lists.

oops! Good catch :)


TS: add fetch() polyfill

As discussed https://github.com/benallfree/pockethost/discussions/76#discussioncomment-4047842, some users need a fetch() function to make api calls.

moved to roadmap


Blurred text on 4k (3840*2160) screen in the miniquad demo

Hello,
I noticed that in the miniquad web demo https://not-fl3.github.io/miniquad-samples/emigui.html the text is blurred.
I tested it on both Firefox and Brave (chromium based) and in both cases the text is rendered in the same way.
Here is a screenshot with Firefox on the left and Brave on the right : https://imgur.com/LIdPU6d
The text become clear when I zoom out (at 67% zoom on brave the text is clear).
But the Egui feature demo https://emilk.github.io/emigui/index.html is rendering sharp text : https://imgur.com/pPRb14W
I noticed that imgur heavily rezises/compresses the screenshots I linked, so the blurriness of the text is not so clear but I can send them by email if you want (the text really is blurry).
I am on Linux 64bit (Manjaro) with a RTX 2080.
Thank you for your awesome work, I wish you the best of luck with this project :+1:

Blurry text is likely an issue with HDPI support. Looking at https://github.com/not-fl3/emigui-miniquad/blob/master/examples/test_window.rs I can see that raw_input.screen_size is set, but raw_input.pixels_per_point is not. This is most likely the culprit. I suggest opening a ticket or PR on https://github.com/not-fl3/emigui-miniquad

@emilk  I know this was closed a while ago, but I ended up implementing my own version in miniquad, and I was able to get a much crisper to version by rendering egui to it's own render target at double the resolution of the screen. I definitely would suggest to backend implementer to do render the UI to it's own render target anyways, but just  a heads up

@emilk  I know this was closed a while ago, but I ended up implementing my own version in miniquad, and I was able to get a much crisper to version by rendering egui to it's own render target at double the resolution of the screen. I definitely would suggest to backend implementer to do render the UI to it's own render target anyways, but just  a heads up

@JustinRyanH Sounds like your miniquad integration is better (and more up-to-date) than the one at https://github.com/not-fl3/emigui-miniquad - would you mind publishing it to crates.io so others can also make use of it?

@JustinRyanH Sounds like your miniquad integration is better (and more up-to-date) than the one at https://github.com/not-fl3/emigui-miniquad - would you mind publishing it to crates.io so others can also make use of it?

@emilk I'll do one better. I'll talk to not-fl3 and see if I can get a PR to fix his.

@emilk I'll do one better. I'll talk to not-fl3 and see if I can get a PR to fix his.


approve users link not just from event page

it should be possible to go to approve users page from any page when you are admin. Currently you need to be in an event page

This is already the case

Reopening since that the remaining location will be addressed in #324


add a mute button to notification (resolve #54)

Implemented issue #54 by sending custom intent and handling it in ForegroudService, without the need of extra system permission.
The linter recommended using setAction() to send intent from notification. Considering that when the reminder is triggered, to mute can be an urgently needed operation, I suppose tapping the notification is more intuitive than action buttons.

Added a mute button at main activity, in case the user wants to mute manually

Set iconSpaceReserved=false for all preferences in Settings, for a better appearance

It may take a bit of time until I review this in detail. Overall looks good!

Set iconSpaceReserved=false for all preferences in Settings, for a better appearance

Can you move that part to a new PR?

Insightful suggestions! All done now.

Thanks!
For the future: Please make sure you don't work on your main as that name conflicts locally with the original main branch.


Typescript

https://github.com/bluelovers/recoil-persist/blob/master/dist/index.d.ts

I have added the typescript definitions in 1.1.0 version.
@bluelovers thank you 👍

I have added the typescript definitions in 1.1.0 version.
@bluelovers thank you 👍

paths seen here is typed as a string, rather than a string array, and both parameters are not marked as optional.
The fork above seems to have done this correctly. Can I raise a PR against this issue to fix?

paths seen here is typed as a string, rather than a string array, and both parameters are not marked as optional.
The fork above seems to have done this correctly. Can I raise a PR against this issue to fix?


App Migration Assistant Installer not launching

I have downloaded the App Service Migration Assistant installer. When I launch the installer, a "Please wait..." status message pops up quickly then goes away. After this, no other windows open showing any installation of this program. I have tried this several times. Is this a known issue? Running on Windows 10.

Even in a successful install, there unfortunately isn't great feedback that installation is complete. If everything worked as it should, the main indicator would be a shortcut on your desktop labeled AppServiceMigrationAssistant and the bits would be installed at %localappdata%\Programs\azure-appService-migrationAssistant
If the above didn't happen then there might be an indicator of why in Event Viewer (click Windows key and search for that name) in the recent Windows Logs > Application events.
Please let me know if the bits were installed or if you see anything in the Event Viewer logs about it after an unsuccessful install.

Thanks for the quick reply! You're correct....it added a shortcut to my desktop after sometime. Closing issue.


Patching and upgrading while the exchange mitigations are in place

I am going to upgrade and patch my Exchange 2016 tomorrow.  However is it going to be a problem with the mitigations still installed? Or should I rollback the mitigations first then patch and install?   There is no documentation that I was able to find on this scenario.
The first time I attempted to run upgrade from CU11 to CU19 it crashed and all the services were disabled. And all the server component were set to inactive.  I just don't want to run into any problems again.

Hey @susanTCI
You should be able to do a CU in place upgrade or SU (security update) in Exchange with services disabled and server component state set to inactive.
This is a state my Exchange servers are in prior to an in place upgrade as there are several pre configuration steps I do to insure connectivity for all protocols are blocked to the server. For contextual (not instructional) purposes,the pre configuration steps I do prior to an upgrade are:

fail over databases and block database activation.
stop transport.
set the server component state of server wide offline to inactive.

With that said, since you had a bad experience and want to avoid this, your best option is to apply only the backend cookie mitigation prior to the CU upgrades across all servers.
The backend cookie mitigation will block the unauthenticated malicious access to your servers while you patch. You will need to re-apply this mitigation after the CU upgrade completes as we found the CU upgrade removes all the web.config settings that are in place for rewrite module. You do not need to re-install the IIS module just re-apply the config.
With MSI install:
ExchangeMitigations.ps1 -ApplyBackendCookieMitigation -Website "Default Web Site" -FullPathToMSI "full path to msi"
Without MSI install:
ExchangeMitigations.ps1 -ApplyBackendCookieMitigation -Website "Default Web Site"
Once you have completed the SU you can rollback the backend cookie mitigation using as follows:
ExchangeMitigations.ps1 -RollbackBackendCookieMitigation -Website "Default Web Site"
Your end to end process will look like this:

run the backend cookie mitigation across all servers.

after each CU is complete, re-apply the backend cookie mitigation.

after each SU is complete rollback the backend cookie mitigation.

For more information on what each mitigation does please visit: https://msrc-blog.microsoft.com/2021/03/05/microsoft-exchange-server-vulnerabilities-mitigations-march-2021/
For more information patching please visit: https://techcommunity.microsoft.com/t5/exchange-team-blog/released-march-2021-exchange-server-security-updates/ba-p/2175901
For more information on the exchange vulnerabilities: https://aka.ms/exchangevulns
If you have any questions feel free to ask.
Thanks!

Thanks for your help.  Is that the only mitigation that I have to but back in.

Hey @susanTCI  the ApplyBackendCookieMitigation  will mitigate only CVE-2021-26855, which is the main vulnerability that allows for unauthenticated access to your Exchange server. We believe ApplyBackendCookieMitigation is sufficient enough on it's own to temporarily mitigate your Exchange server so you can be protected while you patch.
The other mitigations for ECP, OAB, and UM are defense in depth.

I have a question regarding the URL Rewrite mitigation applied to an Exchange Server 2019 with latest CU and SU.
I have been running the Test-ProxyLogon script regularly, along with the CompareExchangeHashes script to spot any activity since I patched my server.  I've tried looking for any comments regarding the CompareExchangeHashes, but have not seen any.
When I run the CompareExchangeHashes script, it does two things that I find odd and false-positive like.
The following is how I'm running it:
.\CompareExchangeHashes3.29.21.ps1 -CompressSuspiciousAndMissingFilesToCSV $true
NOTE: I rename the script to the date I downloaded it to track any changes, as it's not all that clear to me if this script is being changed regularly for whatever reason by the MS team.
The results are output to to the screen as well as the csv and zip file as expected.
Of note though is that it flags the web.config in the results.csv and copies it to the exchange_version_suspicious folder.
this web.config file only has the URL rewrite modification added to it - so it is like it is flagging the URL rewrite config as the suspicious code.  Why is it doing that?
Once I've applied the latest CU and SU, do I roll back the URL Rewrite mitigation?  Is that mitigation (workaround) replaced by some kind of code fix from the SU install?
Also, the Test-ProxyLogon script - is it supposed to it save the last 30 days of logs to a CVE Log collection folder? I'd expect it to only write a copy of the logs that it found suspicion in.
Thanks,
Mark

Hi @mark0910
We no longer support the compare file hashes script and it's since been depreciated/archived.
Recommended guidance today is as follows

Use EOMT to mitigate and scan your unpatched servers for malware. If malware is found there is remediation guidance at https://aka.ms/exchangevulns

2)Once mitigation/scanning and remediation (if necessary) is complete, if your running Exchange 2016/2019 your going to update to the latest CU that was released in March. If your running Exchange 2013, you will need to apply the SU.
If you applied the SU to Exchange 2016/2019 previously your servers are patched, however, we highly recommend everyone get on the latest CU ASAP so if and when future security updates are applied it's a much easier process.

After the CU and SU are applied, you can roll back the EOMT mitigation.

Once you have the SU or CU applied to your servers there is no need to reapply the mitigation. In fact EOMT will skip the mitigation all together if it detects your on the latest CU or have applied the SU.
You are welcome to run MSERT via EOMT or on your own as you like to insure no malicious activity has persisted after the patch.


[gen-apidocs] fix Go lint issues, improve error handling, replace navdata generator with regular JSON encoder

This PR fixes a whole lot of general linting issues, adds extra error handling, fixes some typos and replaces the navdata generator with a more generic, recursive implementation.

/lgtm
/approve
Kicking this in.
We can iterate later if things not working as expected.

I'm looking through the scroll-apiref.js to learn more about how the navdata is used.


Visual C++ compatibility fixes

Enables building libgdiplus using Visual C++:
A couple of minor fixes which allow building libgdiplus natively on Visual C++ (Windows)]

use tmpfile instead of mkstemp to get generate a temporary file
Define _USE_MATH_DEFINES so M_PI is defined
Don't #include Unix-only headers
Don't redefine max and min
Fix macro expansion

Running the System.Drawing and System.Windows.Forms unit tests on Linux as part of https://travis-ci.org/CoreCompat/libgdiplus/jobs/226520941

@akoeplinger System.Drawing + System.Windows.Forms tests are green for this change as well. The pointer-related changes caused a regression, so I'll handle that in a separate PR.

@akoeplinger This should be better :). I can't include windows.h in font.c because libgdiplus redefines a lot of Windows-related functions, so the best option appeared to be to define a mkstemp compat shim in a separate file.


Add the https and wss feature.

Add the https and wss feature.
Modify the wts.sh file to create the certificate folder.

Request a new one.


Modify each of the services to cover the operations of the associated API

The financial service currently supports the application features of displaying, deleting, and updating an Asset resource. The create functionality is not in place yet. Add the supporting operation for creating an Asset and also a Debt resource. Copy the same functions
into each of the services completing the modules for the resources we have currently.

This was completed with feature/issue-1 and merged to main. Done.


make opening times a combo of a page and opening hours

References #2490
Who is this for?
🚁
What is it doing for them?
Make opening hours a combo-like page, mixing a query of opening hours and the opening hours page.
I don't like it, but it works, my pragmatism is stretching.
This allows the page to be included in content lists and search results.
Another way would be to make this a specific, singleton type in prismic, and map all the information in that type - then we could just query it.
This might be useful if we wanted to have a special promo for this.

I've also added a perf page to our datastudio doc.


[CRITICAL FEATURE REQUEST] Disable game boot animation

Is your feature request related to a problem? Please describe.
Current behaviour: when launching any app from cross launcher, an "Android" boot animation will play.
Describe the solution you'd like
Add a toggle in the system settings to disable game boot animations.
I consider this a critical feature request because the animations make the launching/loading of apps slower for no good reason, which is especially annoying when switching between apps frequently. Also the stock animation doesn't look that good (Imo)
Describe alternatives you've considered
Attempted to customise the gameboot animation following the documentation here: https://github.com/EmiyaSyahriel/CrossLauncher/blob/master/CUSTOM.MD
Outcomes:

The id.psw.vshlauncher/files/dev_flash doesn't exist by default
When I tried creating the relevant directories and placing my own custom resource at id.psw.vshlauncher/files/dev_flash/vsh/resource/COLDBOOT.MP3, the launcher still uses the stock Android animation
Note: I tried using both .mp3 and .mp4 formats for the resource, neither worked

Additional context
The animation i'm referring to: screenshot

I've added back the option to skip gameboot and launch apps directly in release Alpha 20221205_A0139, can be changed via Settings > System Settings > Enable Gameboot. This release should also fix the customization file loading regarding Cold boot and Game boot. In case it solves this issue for you, you may close it or I will close it myself if you don't respond before year end.
By the way, the gameboot & coldboot were in form of .png image being faded and scaled by code with .mp3 file played as it's BGM. There is no .mp4 support planned as they were too hard to implement while keeping things lightweight.


Istio 1.4.6 airgap testing

Airgap testing for: https://github.com/rancher/rancher/issues/25695
On 2.4 and 2.3.6

@rancher-max This could be tested with v2.3.6-rc5.

Validated on Rancher v2.3.6-rc5 in both fresh install and upgrade from v2.3.5
All below bullets were done for both fresh install and upgrade scenarios. Only showing one screenshot in most cases as the data is the same on both.

1.4.600 is the only available version (in upgrade scenario it is the only available version to upgrade to)

All images are correct and deployed successfully when all relevant options are enabled

sidecars deploy properly when auto-injection is enabled in namespace
Traffic Metrics and Traffic Graph pages show correct data

Links all work and display correct data
Saving changes via tools page and apps page are both successful and changes are deployed successfully
Previous bug regarding unable to add custom sidecar image is fixed here (note the ImagePullBackOff error is because I did not actually have the image available in my Airgap setup, but this scenario passes if the image is being used as specified, which it is):


benchtop: fix docs and remove warnings

#207  👈
master

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

Join @gabriele-0201 and the rest of your teammates on  Graphite


New FlaUI nuget release request

Hi,
I do not know if this is the correct place to ask but, I wonder when there is a new release available? We would like to update our FlaUI nuget package to include the changes, where some methods in the FlaUITestBase are able to be overriden thanks to the changes made by johan-eriksson.
Thanks in advance!
Best regards,
Alex

I finally released a new version and deprecated quite some stuff so it is easily possible to work and extend FlaUI with Visual Studio 2022.


Fix network fee calculation

Andreas pointed out that if you do a payment of 1 with a fee of 0.1 %, instead of returning a fee of 0.001 the lib returns 0.00100101. Also it turns out that the raw value is also wrong, so it does not seem to be the conversion of raw to value.
We most likely do a calculation without using bignumber somewhere.

It's not a bug, it's a
FEATURE
https://github.com/trustlines-network/project/wiki/Capacity-Imbalance-Fee-Specification: fee = (imbalance_generated - 1) / (d - 1) + 1
He sends 1 with 8 decimals and an imbalance fee of 0.1% so fee = (100000000-1)/(1000-1) + 1 = 100101


Make bootstrap.sh safe from bad connection

This allows to run the bootstrap.sh directly and "safely" without storing it on disk.

@michaelshobbs ?

my only concern is variable scoping but it's probably ok. @omeid can you test this with different $DOKKU_TAG and $DOKKU_REPO env vars?

Works just fine for me.
$ curl https://raw.githubusercontent.com/omeid/dokku/patch-1/bootstrap.sh | DOKKU_TAG=v0.2.0 DOKKU_REPO="https://github.com/omeid/dokku.git" bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2719  100  2719    0     0   5058      0 --:--:-- --:--:-- --:--:--  5053
Preparing to install v0.2.0 from https://github.com/omeid/dokku.git...
Ign http://archive.ubuntu.com trusty InRelease
Ign http://archive.ubuntu.com trusty-updates InRelease
Ign http://archive.ubuntu.com trusty-security InRelease
25% [Working]^C
$
`

:+1:

It would be nice if install.dokku.io pointed to https://raw.githubusercontent.com/progrium/dokku/patch-1/bootstrap.sh

Who owns dokku.io...?

$ whois dokku.io
[Querying whois.nic.io]
[whois.nic.io]

Domain : dokku.io
Status : Live
Expiry : 2016-01-04

NS 1   : dns1.registrar-servers.com
NS 2   : dns2.registrar-servers.com
NS 3   : dns3.registrar-servers.com
NS 4   : dns4.registrar-servers.com
NS 5   : dns5.registrar-servers.com

Owner  : Ernst Rijsdijk
Owner  : Noordgouw 28
Owner  : Ursem
Owner  : Noord-Holland
Owner  : NL
`

Yup I did the same thing. No idea who that is.

No idea either.

I've tried reaching out to that guy, no response. If anyone wants to do that for us, that would be pretty pro.

@josegonzalez It is important that the domain ownership is passed on to the owner/maintainer of dokku though. Having two trust points for this sort of "run remote code on the fly" is kinda less than optimal.

@omeid we never refer to dokku.io anywhere, and it's not like we're telling people to install code from a domain other than a github-controlled domain.


GEOMESA-1560 Spark SQL Write

Fixes GeoCQEngine write. Adds docs and tests
Signed-off-by: Atallah Hezbor<EMAIL_ADDRESS>

the travis build failed - i think overrideFid in FeatureUtils.copyToWriter was renamed and merged into master just now, causing the error

@elahrvivaz ah that explains it. I was totally stumped.

Moved to


Not able to Inject in swift Project

Injection fails in my project at the below line
while ( field->flags == 0x2 ) {
if ( field->optional ) {
field = field->optional;
*optr++ = '?';
*optr = '\000';
}
--More Logs
TtC11TransitFare19FirstViewController is implemented in both /Users/ajasingh/Library/Developer/CoreSimulator/Devices/1459A52A-E4D5-4302-BF1F-7034EF5119B6/data/Containers/Bundle/Application/879D997B-5AE3-432E-B795-45BFC232FF56/TransitFare.app/TransitFare and /Users/ajasingh/Development/SwiftTransportApp/TransitFare/iOSInjectionProject/build/Debug-iphonesimulator/InjectionBundle2.bundle/InjectionBundle. One of the two will be used. Which one is undefined.
1 injections performed so far.
2016-03-16 17:39:59.466 TransitFare[15349:3729938] Ignore any warning, Swizzled TransitFare.FirstViewController 0x12178d130 -> 0x10e0a43a0
(lldb)

Hi, your project must have an instance variable the “-injected” code does not cope with. It’s a bit of a hack really. I’m afraid you’ll have to find an alternative to using -injected such as using “INJECTION_BUNDLE_NOTIFICATION”. Injection itself will still work it is the “sweep” of all the objects in your application for -injected that is failing.

If you want to “TeamView” to try to sort this out let me know.

Xcode 7.2.1

If you follow the stack up you can see the class and instance varaible that is causing the problem. Can you tell me it’s type?


Fix #62350

Fix for the issue #62350.
When we run a file with a path that has a space in it instead of running "c:\Users\matb\folder\sp ace.html" it will now execute & 'c:\Users\matb\folder\sp ace.html'. This way powershell will be able to execute the file as it is pretended.

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

@ff7 can you please recreate the PR, it looks like something went terribly wrong and there are multiple unexpected commits and conflicts in this PR.

My bad @alexandrudima, I was using an older version of vscode. I believe now that this pull request doesn't have conflicts. Here is the link : #65321.


Remove Deploy Script

Coverage remained the same at 89.474% when pulling e5ec520300f37aba6f0da094bc0332b65e01a6dd on remove-deploy-script into ae2290f24c13446c932c098c798eec47441df04f on master.

Coverage remained the same at 89.474% when pulling e5ec520300f37aba6f0da094bc0332b65e01a6dd on remove-deploy-script into ae2290f24c13446c932c098c798eec47441df04f on master.


[BEAM-9919] Added an External Transform API to Go SDK

It is now possible to execute an External (a.k.a, Cross-language) Transform within pipelines authored using the Go SDK.
It also includes a Word count example run in LOOPBACK mode using the test python expansion service and executed by Python's portable runner.

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] 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.

R: @lostluck

CC: @chamikaramj @robertwb @youngoli

I've addressed the initial comments with the exception of ones suggesting organizational changes. In general this discussion will dictate where the final (and cleaner) API will reside and how the user will make use of it. Since I won't be the stakeholder for this feature after a few weeks, I thought it'd be worthwhile bringing in more opinions – @lostluck @robertwb @youngoli
The challenges and possible solutions are as follows:

Where should all xlang code be?
There are multiple moving parts related to the execution of cross-language transforms which are now condensed into one TryCrossLanguage function in external.go

Allowing various combinations of inputs/outputs (n:n, 1:0, 0:1)
Correctly adding ExternalTransform to Graph representing the unexpanded cross-language transform within the pipeline
Building the ExpansionRequest
Querying the expansion service for the corresponding ExpansionResponse
Querying the expansion service for Artifact Resolution and Staging
Handling the expanded components of the ExternalTransform

Decomposing these functions correctly within package(s) dedicated to ExternalTransform whether as beam/.../xlangx or beam/.../xlang or both will be ideal. It helps in segregation of responsibilities so that each function can be developed to maturity independent of each other. I can then make incremental progress on different fronts without compromising overall functionality that this PR has already achieved.
Futhermore, external.go can then use the package(s) to expose generic APIs such as beam.CrossLanguage (for n:n inputs/outputs), beam.CrossLanguageAsSource (for 0:1 inputs/outputs) and beam.CrossLanguageAsSink (for 1:0 inputs/outputs).
In the future, when the existing API is to be deprecated, these functions could then be renamed to their beam.ExternalTransform... counterparts. The exisiting API could be renamed to beam.LegacyExternalTransform and implemented using the API that exists as the outcome of this PR (since it is already backwards compatible for the most part).
@lostluck already pointed to a possible segregation by removing pipeline handling code within universal.go into graphx/xlang.go to be used in graphx/translate.go. I could explore similar options in alignment to the results of this discussion.

How is ExternalTransform associated to pipeline construction?

Adding to Graph and/or MultiEdge
Currently, graph.Opcode for external transforms already exists (with value as "External") and is used in conjunction with graph.Payload by the existing external transform API. Adding ExternalTransform to Graph will pull in proto and grpc dependencies within beam/core/graph regardless of whether the pipeline contains any cross-language transform or not. Adding ExternalTransform to MultiEdge will need additional logic to distinguish between an external transform using the proposed v/s current API.
One scanario in which  I believe this method could be useful is if correct pipeline construction required explicit handling of ExternalTransform when each MultiEdge is being added during the graphx.Marshal call. As of now I don't feel that is the case.

Augmenting Pipeline and Scope
As is implemented in this PR, ExpandedTransforms map[string]ExternalTransform field is added to Pipeline. The key is the MultiEdge's ID that represents an external transform within Graph. Using this, during pipeline construction, the Go proto representing this transform is swapped with the expanded ExternalTransform. All of the components associated with the expanded ExternalTransform are also added to the pipeline proto. This method does not conflict with any pipeline representation/construction code for strictly Go pipelines and may be more suitable for the future.
An existing problem worth calling out is the necessity to pass in the Pipeline reference to beam.CrossLanguage to add the new ExternalTransform in the ExpandedTransforms map. A possible solution to this would be:

Adding func (p *Pipeline) RegisterExternalTransform(k string, e *ExternalTransform) function to Pipeline
Adding AddMapEntry func(string, *ExternalTransform) field to Scope
Changing func (p *Pipeline) Root() Scope to  return Scope{scope: p.real.Root(), real: p.real, AddMapEntry: p.RegisterExternalTransform}

beam.CrossLanguage should then be able to use s.AddMapEntry(key, externalTransform) instead of the current implementation.

@lostluck My updates are ready to review!

Thanks a ton @lostluck!


go.mod: Bump controller-tools fork version to v0.8.0-2 to allow XValidation kubebuilder markers

Bumps controller-tools fork to the newly released version v0.8.0-2 which adds support for kubebuilder XValidation markers (https://github.com/cilium/controller-tools/pull/5). This allows using Common Expression Language (CEL) to validate custom resource values in cilium CRDs.
Example marker in CRD types:
  // +kubebuilder:validation:XValidation:rule="self.keepAliveTimeSeconds <= self.holdTimeSeconds",message="keepAliveTimeSeconds should be smaller than or equal to holdTimeSeconds"

Generated API:
   x-kubernetes-validations:
   - message: keepAliveTimeSeconds should be smaller than or equal to
       holdTimeSeconds
     rule: self.keepAliveTimeSeconds <= self.holdTimeSeconds

/test

@rastislavs Do plan on maintaining this fork mid/long term? If yes, we should think about getting rid of the replace directive and require our fork.

@rolinh

@rastislavs Do plan on maintaining this fork mid/long term? If yes, we should think about getting rid of the replace directive and require our fork.

I think we would first need to bump the fork to a more recent upstream version, which is a bit tricky ATM because of fork-specific changes (it is described in the issue https://github.com/cilium/cilium/issues/24657). For now I really just wanted to add the XValidation support as a minimal change.
cc @christarazi

I think we've been using the fork for a long time and currently no plan to move off of it given we depend on extra features in the fork. So we might as well just depend on the fork in go.mod rather than have a replace directive.


adding NisOS package installation of neomutt.

List of maintainers was taken from the NixOS/nixpkgs entry of neomutt and The Installation commands were taken from the nixos manual.
However, there also does a different way of installing packages exist and i'm unsure which I should choose.

@cstrahan @hiberno @erikryb what's your opinion on this issue?

Looks good to me, except for the part that I do not actually use NixOS right now. I haven't had the time to remove myself as maintainer. I am about to do it now.

Looks good to me too. I prefer to install packages declaratively using configuration.nix, as you wrote.

I think I'll tackle this issue this weekend. Thanks for your feedback @cstrahan @hiberno @erikryb @vrthra

My commits  c9496f9 199fded and 561df31 were merged into a92d8a3357e187d5a94caa764f7d41a38b19af4d - which is also the reason why this pull request has now conflicts.

This commit addresses the feedback in this Pull Request, so I'll close this PR now.


cabal install succeeds, but hwk does not run

With GHC 9.0 (and cabal-install 3.x):
$ cabal install hwk
...
$ echo 1 2 | hwk reverse
ERROR: Won't compile:
<no location info>: error:
    Could not load module ‘Data.List.Extra’
    It is a member of the hidden package ‘extra-1.7.9’.
    You can run ‘:set -package extra’ to expose it.
    (Note: this unloads all the modules in the current scope.)
    It is a member of the hidden package ‘extra-1.7.9’.
    You can run ‘:set -package extra’ to expose it.
    (Note: this unloads all the modules in the current scope.)
    Use -v (or `:set -v` in ghci) to see a list of the files searched for.

Does installation rely on specific cabal settings?  Maybe then they should be given in the instructions...

See a similar problem at https://github.com/gelisam/hawk/issues/215.
I can fix the problem with
$ cabal install --lib extra

Sorry for not seeing this/responding earlier.
I think the "workaround" you mentioned (cabal install --lib ...) is the only viable one at the moment.
Though I am open to other suggestions of how to handle this better if it's possible.

Is there some upstream awareness/documentation/migration guide on such issues.  I have the feeling that GHC-9 and v2-cabal have created similar problems in other projects.  At least I come across similar errors quite often these days.

Right, it is a good question - I don't know TBH, not aware of anything particular.
I even became aware of cabal install --lib rather late...
So far I only found https://cabal.readthedocs.io/en/3.6/cabal-commands.html?highlight="install --lib"#adding-libraries-to-ghc-package-environments.


Add docs for running a new EN using AWS AMI

Proposed changes

This PR adds documentation for running a new EN using AWS AMI

Types of changes
Please put an x in the boxes related to your change.

[ ] Minor Issues and Typos
[x] Major Content Contribution
[ ] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to reach out. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING GUIDELINES
[x] I have read the CLA and signed by comment I have read the CLA Document and I hereby sign the CLA in first time contribute
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

I have read the CLA Document and I hereby sign the CLA

Please make sure that the new 'aws-ami.md' entry is added to sidebars.js as well.
For example, with my suggested changes, the sidebars.js is shown as below:
const commonSidebar = [
  {
    type: 'html',
    value: '<span class="sidebar-divider" />',
  },
  {
    type: 'category',
    label: 'Node Quick Reference',
    link: { type: 'doc', id: 'misc/operation/operation' },
    items: [
      'misc/operation/configuration',
      'misc/operation/node-log',
      'misc/operation/log-management',
      'misc/operation/kaia-command',
      'misc/operation/troubleshooting',
      'misc/operation/chaindata-snapshot',
      'misc/operation/aws-ami',
      'misc/operation/node-pruning',
      'misc/operation/upstream-en',
    ],
  }, ...

@scott-klaytn  Thanks, added to sidebar as you suggested.


Usage of the reconstruct_from_prior Function

Hi,
First of all, thank you for the amazing code and all your hard work on this project!
I have a question regarding the reconstruct_from_prior function. This function is not documented, and although I have read through the code, I am not confident in my understanding.
Is my understanding correct that this function reads an existing reconstruction and recalculates the points?
In what scenarios would recalculating the points be necessary?
My primary goal is to add new images to an existing reconstruction as mentioned in the this PR178
If there is an existing method for this, I would greatly appreciate your guidance.
Thank you for your time and assistance!

For adding more images to an existing reconstruction, this function: https://github.com/mapillary/OpenSfM/blob/1106198c79a0417fe59df68293c59ba7688533c0/opensfm/reconstruction.py#L1386,
and this file: https://github.com/mapillary/OpenSfM/blob/main/opensfm/actions/extend_reconstruction.py
may be of interest to you.


ReferenceError: TouchEvent is not defined

event instanceof TouchEvent
TouchEvents are not defined by a few browsers which are not Chrome, such as FireFox.
https://github.com/pixijs/pixi.js/blob/923a8eced8e43f71d48d7916305b48588bccf69b/src/interaction/InteractionManager.js#L1723

Considering the documented param for this method, might I suggest the following?
!(event instanceof MouseEvent && event instanceof PointerEvent)

What is different about your setup that window.ontouchstart exists, but TouchEvent does not? For me, Firefox 57 has neither of these defined, so this.supportsTouchEvents would be set to false when the InteractionManager is created.

FireFox 57.0.1 (64-bit), MacOS High Sierra (10.13.1)
It looks like on of my extensions was defining a function on the window.ontouchstart namespace. My apologies for not being more thorough in my investigation.


Error adding domains?

Traceback (most recent call last):
File "sublert.py", line 165, in lookup
conn = psycopg2.connect("dbname={0} user={1} host={2}".format(DB_NAME, DB_USER, DB_HOST))
File "/usr/local/lib/python3.6/dist-packages/psycopg2/init.py", line 126, in connect
conn = _connect(dsn, connection_factory=connection_factory, **kwasync)
psycopg2.OperationalError: could not connect to server: Connection timed out
Is the server running on host "crt.sh" (<IP_ADDRESS>) and accepting
TCP/IP connections on port 5432?
could not connect to server: Network is unreachable
Is the server running on host "crt.sh" (2a0e:ac00:c7:d430::5bc7:d430) and accepting
TCP/IP connections on port 5432?
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 141, in _new_conn
(self.host, self.port), self.timeout, **extra_kw)
File "/usr/lib/python3/dist-packages/urllib3/util/connection.py", line 83, in create_connection
raise err
File "/usr/lib/python3/dist-packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
OSError: [Errno 101] Network is unreachable
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 601, in urlopen
chunked=chunked)
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 346, in _make_request
self._validate_conn(conn)
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 852, in _validate_conn
conn.connect()
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 284, in connect
conn = self._new_conn()
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 150, in _new_conn
self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.VerifiedHTTPSConnection object at 0x7f90f73e89b0>: Failed to establish a new connection: [Errno 101] Network is unreachable
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/requests/adapters.py", line 440, in send
timeout=timeout
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 639, in urlopen
_stacktrace=sys.exc_info()[2])
File "/usr/lib/python3/dist-packages/urllib3/util/retry.py", line 398, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='crt.sh', port=443): Max retries exceeded with url: /?q=%25.datagran.io&output=json (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f90f73e89b0>: Failed to establish a new connection: [Errno 101] Network is unreachable',))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "sublert.py", line 448, in
multithreading(parse_args().threads)
File "sublert.py", line 416, in multithreading
adding_new_domain(domain_to_monitor)
File "sublert.py", line 224, in adding_new_domain
response = cert_database().lookup(domain_to_monitor)
File "sublert.py", line 184, in lookup
req = requests.get(url, headers={'User-Agent': user_agent}, timeout=30, verify=False) #times out after 30 seconds waiting (Mainly for large datasets)
File "/usr/lib/python3/dist-packages/requests/api.py", line 72, in get
return request('get', url, params=params, **kwargs)
File "/usr/lib/python3/dist-packages/requests/api.py", line 58, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 520, in request
resp = self.send(prep, **send_kwargs)
File "/usr/lib/python3/dist-packages/requests/sessions.py", line 630, in send
r = adapter.send(request, **kwargs)
File "/usr/lib/python3/dist-packages/requests/adapters.py", line 508, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='crt.sh', port=443): Max retries exceeded with url: /?q=%25.datagran.io&output=json (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f90f73e89b0>: Failed to establish a new connection: [Errno 101] Network is unreachable',))

Hi,
I tried to reproduce the issue with the same domain you're trying to monitor but it worked just fine. From the stack trace you provided, I guess there was an issue with your internet connection so the tool was unable to connect to the database.
Regards.


Change dependency version requirements to be "here to next major"

Version requirements are now specified as, for example,
pandas = "^1.4.1" where the caret means "from this version
up to the next major version number increment."

Hi James, you were worried about testing versionality, but I don't think I know how to do what you want. Could you expand a bit?


Feature Request "Rest Button"

A button that restores expended spell slots

What do you mean? Not sure I understand

Currently, expended spell slots must be restored individually by clicking each spells restore icon. I would like a button that restores spells and focus pools all at once.

I think you're in the wrong issue tracker, this is for Foundry, a smart contract development toolchain :smile: No icons or spells here!


Local Volume Provisioner: Create versioned yaml files (Issue #462)

Addresses issues raised in #462

Create versioned yaml files.
Fail if a configmap file is not found for the bootstrap process. Do not attempt it automatically create a default one.
Setup compatibility matrix.

cc @msau42

/assign

Note that the example files are currently broken, external-storage/local-volume/bootstrapper/deployment/kubernetes/bootstrapper.yaml has latest for bootstrap and v1.0.0 for the provisioner, and example-config.yaml is appropriate for latest.    If this PR isn't going to land ASAP, it'd be nice if the provisioner in the example was changed to :latest.   Thanks!

/lgtm
Thanks for cleaning this up!

@wongma7 - can you PTAL and merge?


Run support/update.js

Github emoji have been updated.

Thanks for info.
Content of light version is wrong, i'll update package myself in the nearest time.

Yeah I just realized the CI test failed, I guess you should update light to Unicode 6.0

No, that seems to be due lodash api change between 3->4. light version should not be changed at all.

Strange..I did not update any dependencies. I think you should consider specifying versions in your package.json or use shrinkwrap

That's because all was done when lodash was 3.x, and now it's 4.x.
I'll fix it soon.

I've commited fixed scripts, but don't like result. Light version was changed:
+  "pout": "😡",
-  "star": "⭐",
-  "zap": "⚡",
-  "umbrella": "☔",
-  "hourglass": "⌛",
-  "watch": "⌚",
-  "mahjong": "🀄",
-  "coffee": "☕",
-  "anchor": "⚓",
-  "wheelchair": "♿",
-  "aries": "♈",
-  "taurus": "♉",
-  "gemini": "♊",
-  "cancer": "♋",
-  "leo": "♌",
-  "virgo": "♍",
-  "libra": "♎",
-  "scorpius": "♏",
-  "sagittarius": "♐",
-  "capricorn": "♑",
-  "aquarius": "♒",
-  "pisces": "♓",
-  "exclamation": "❗",
-  "heavy_exclamation_mark": "❗",
-  "black_medium_small_square": "◾",
-  "white_medium_small_square": "◽",
-  "black_circle": "⚫",
-  "white_circle": "⚪",
-  "white_large_square": "⬜",
-  "black_large_square": "⬛"

Probably, guys from github changed glyph codes. Need to recheck content of https://github.com/markdown-it/markdown-it-emoji/blob/master/support/visible.txt

@puzrin Let me know if you need help, I'm currently using my own fork for my project.

I've scheduled "light" chars list recheck to the start of nearest week, if not urgent. Need to manually check changed chars - are those available in some OS-es or not. Main criteria of light list - chars should exist on the most of devices.

More fixes in master. Light data is now ok.
I think we will finish tomorrow.

Closing - everything fixed in 1.3.0


Rainloop under active development?

Is this project still under active development? It's been a year since the last updates were made.

It seems dead, or semi-dead. However, and if interested, there is an active fork called Snappymail. Don't get me wrong, I like Rainloop, I just worry about some issues that have as far as I can tell remain unfixed.
https://github.com/the-djmaze/snappymail

Well, you sir, is what we call a scholar and a gentlemen!
Thanks for the redirect! Already switched over my deployment!

Whatever is your choice, be careful with Rainloop right now:
https://thehackernews.com/2022/04/unpatched-bug-in-rainloop-webmail-could.html?m=1

Yeah, that company literally opened an issue here earlier and no one answered. I used Rainloop on my Nextcloud install yet since the issue is still open and the security flaw is still at play, I had to remove it from my install. Shame.
As per your own article, SnappyMail does not suffer the same security issues.

I have done a comment here:

https://github.com/RainLoop/rainloop-webmail/pull/2187#issuecomment-1233475036

Please use SnappyMail from @the-djmaze, we can thanks for this work!

https://snappymail.eu/
https://github.com/the-djmaze/snappymail/

Please note that SnappyMail supports SCRAM-SHA-* for connection, very good security:

https://github.com/the-djmaze/snappymail/issues/182
https://github.com/the-djmaze/snappymail/blob/master/snappymail/v/0.0.0/app/libraries/RainLoop/Model/Account.php

Linked to:

https://github.com/RainLoop/rainloop-webmail/issues/2185
https://github.com/RainLoop/rainloop-webmail/pull/2183
https://github.com/RainLoop/rainloop-webmail/issues/2180
https://github.com/RainLoop/rainloop-webmail/issues/2162
https://github.com/RainLoop/rainloop-webmail/issues/2142
https://github.com/RainLoop/rainloop-webmail/issues/2134
https://github.com/RainLoop/rainloop-webmail/issues/1831


Issue:257 Fix Bulk lookup FIB for IPV4

Issue: https://github.com/CloudNativeDataPlane/cndp/issues/257
Signed-off-by: Vikas Jain<EMAIL_ADDRESS>Signed-off-by: Vikas<EMAIL_ADDRESS>

Issue: https://github.com/CloudNativeDataPlane/cndp/issues/257
Signed-off-by: Vikas Jain<EMAIL_ADDRESS>

retest

To help fix the DCO error, click on the Details link next to the DCO action to get some help with adding the Signed-Off-by line.


fix/copyright year dynamic in footer

fixes: #15
Made the footer to show the latest copyright year
screenshot :

@PrathameshSonje This already fixed by @rhnas110 Please create new issue if you want to do contribute


Stop generating explicit interface member implementations

Closes #1384
Saves a whole lot of generated code (and a bit of binary size), and makes stuff just easier to read too.

Superseded by #1553.


Support Multiple Vesting Schedules in Vesting Pallet

Right now we have a pretty strict limitation that only one vesting schedule is allowed per user.
This means if you already have a vesting schedule (say from participating early in Polkadot), that you cannot receive any more manifests with a vesting schedule.
You might imagine that the treasury would want to support vesting payouts at some point, and a single user would want to be the target of multiple vesting payouts.
The solution here should be pretty simple. We should allow a user to have a vector in storage with max length MaxVestingSchedules (configurable) to represent the vesting schedules for the user.
A lock is placed on the account with the maximum balance across all vesting schedules. When a user updates their vesting, we go through the schedules and correctly update the lock, removing any completed schedules.
Since there is still some limit to number of vesting schedules, we should add an additional extrinsic which allows a user to "combine" vesting schedules.
This would combine the balance of two vesting schedules, and create one vesting schedule whose lock time is the greatest of the two.
This means that a user can always extend their vesting schedule to consolidate vesting balances, but also free up space to collect more vesting schedules.

FYI our vesting pallet implementation already support this https://github.com/open-web3-stack/open-runtime-module-library/blob/master/vesting/src/lib.rs

So only root can change a user's vesting schedule?
This implementation does solve more than 1 vesting schedule, but it would be quite easy to lock out a user from getting more vesting schedules by sending them 20 vested transfers.
Seems on the right track though.

update_vesting_schedules is to modify existing vesting schedule. e.g. extend it or reduce it.
Our implementation only put one lock. I don't think it is possible to consolidate multiple vesting schedule instances.

but it would be quite easy to lock out a user from getting more vesting schedules by sending them 20 vested transfers

Good point... Maybe I should not allow anyone to call vested_transfer... At least at Acala we don't have this requirement.

It is not about creating one lock, my suggestion would be the same, but limiting the size of the Vec. Basically, my idea is that you let the user combine two schedules into one by accepting the worst of the two unlocking schedules.

I agree with multiple schedules


Can you access the game server console without rcon?

Is it possible to access the game server from outside of the container, without using rcon?
Something with docker exec perhaps?

No, the only way to interact with a running CS2 server is via rcon


fix: main.ts does not use dot-env

For some reason it always thinks CDK_DEFAULT_REGION = us-west-2
Though .env is and .aws/credentials are us-east-1
(and it picks up the new ACCOUNT_ID correctly)

Bizarrely, even if I hard code the stack to create everything in us-east-1, 'npx projen deploy' still builds the stack ITSELF in us-west-2.
Restarting CLI to see if that helps...

Weird
current credentials could not be used to assume 'arn:aws:iam::712023778557:role/cdk-hnb659fds-deploy-role-712023778557-us-east-1', but are for the right account. Proceeding anyway.
current credentials could not be used to assume 'arn:aws:iam::712023778557:role/cdk-hnb659fds-deploy-role-712023778557-us-east-1', but are for the right account. Proceeding anyway.
current credentials could not be used to assume 'arn:aws:iam::712023778557:role/cdk-hnb659fds-deploy-role-712023778557-us-east-1', but are for the right account. Proceeding anyway.

Did not have bootstrap. Hmm.
 ⏳  Bootstrapping environment aws://712023778557/us-east-1...
Trusted accounts for deployment: (none)
Trusted accounts for lookup: (none)
Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize.


chore: prefer use of global TracerProvider/MeterProvider

Remove setting of TracerProvider in samples where it is actually not needed as the global TracerProvider should be preferred on default.
I found it also inconsistent to set a TracerProvider but no MeterProvider.

I added one more commit to align sequence of provider.register() and registerInstrumenations().

Are you up to make the same in contrib, so we have unified examples ?

Sure, will do once this landed.


React alternative

Hi!
Great work with these icons they look really good.
I wanted to add a humble suggestion for those who need a component and don't want to use react. I recently created a custom element to work with various icon libraries
https://github.com/lekoala/last-icon
Just added the support for your icon set :-)
Keep up the good work!

Glad you like them, if you want to contribute to the library feel free to add a similar .js template for iconoir here to support the project!


Make sure repo_dir is an empty directory or doesn't exist before 'dump-repo' (#20205)

Backport  #20205

bot


Great work！

Hello, I have carefully read your article, and it's simply fantastic.  This will be a great work.  Thank you for your effort.  By the way, are you from Europe?

Hi, Thank you. I'm from india


Investigate possibility of improving multi-confirmation accept/deny

Steam app includes such option now, if there is special undocumented Web call that makes it possible to accept/deny many confirmations in one call, it'd be much better than current single-call approach.
Wishlist.

There is https://steamcommunity.com/mobileconf/multiajaxop.
Also, Geel upgraded SteamAuth to support the new endpoint.

Thanks for heads up @aleksasavic3, this is now done and works nicely at least for me, I'll release pre-release later, I like this method much better than old one.
Sadly there is still no way to get multi-confirmation details, but Valve at least doesn't restrict this option to be synchronous, so ASF can launch tasks in parallel, as opposed to confirming, which was forced synchronous before.

@JustArchi Glad to help! :smiley:


Cache repositories

the idea is to cache locally the plugins and groups

I am tempted to give https://github.com/hashicorp/go-getter a try for this one.

Why not! ( if the package is not too big )


"Force Quit…" shortcut doesn't work

Command-Option-Escape doesn't open the "Force Quit" dialog when Karabiner Elements is installed. Verified on Sierra and El Capitan.

+1 Was having this issue this morning and was going to file an bug…turns out this is related to #89.

I've fixed this issue in v0.90.47.
Please upgrade to the latest version from "Misc > Check for updates" or
https://pqrs.org/latest/karabiner-elements-latest.dmg

I'm still having issues with the the Alt key (for example, using Alt-drag to make a rectangular selection or Alt-click moving the cursor in iTerm).


color map [future request]

plasma_color.zip run binary at $1000
can we have something like color map (or so called attribute) for this  type of effects?  necessity of using Text Color every sngle time is kind of slow ;)

in plasma char and this type of effects I can build single row before put chars on screen, but I can do that with colors.

something like DrawString would be nice but instead chars it should take color values.


LInux Hardened Kernel Updates for 2024-12-25

Closes https://github.com/NixOS/nixpkgs/issues/365700
Happy holidays <3
Previous bump: https://github.com/NixOS/nixpkgs/pull/365214
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Built and running 6.12.6-hardened1

Have you checked if scx works?

Only normal usage, didn't check any particular feature.


Page frozen and will not refresh

Challenge http://freecodecamp.com/challenges/waypoint-bring-your-javascript-slot-machine-to-life has an issue. The page is frozen and I can't refresh it. I'm using Chrome on a Mac so I tried the hard refresh (CMD + Shift + R) and nothing happens. I tried logging out and logging back in, using different browsers, etc. and the page is still frozen every time I open it up.

Dear camper, your issue is related to: https://github.com/FreeCodeCamp/FreeCodeCamp/issues/2264
Please see Help I've Found a Bug.

Hey @BillKroger thanks for posting this issue!
The main thread is actually #1960.  Chances are your unfinished solution (which is crashing your browser) is actually saved to you account.  I'm not sure there is any way to delete the saved solution yet, but you can probably just move on to the next challenge for now.

Try hitting the Reset button on the page on the left

Try opening 'freecodecamp.com' open your console and go to resources tab(chrome). There click on the localStorate link in the nav bar on the right. Delete all the entries on the right side. Then navigate to the challenge. This removes all your local stored challenges.

@benschenker Could you relay this information to others that are running into this issue?

@BerkeleyTrue I put it in a wiki page! https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Clear-Your-Browser's-Local-Storage  Right now it only has your chrome instructions though.

FF is similar except you need to do it in the command line one by one which is a pain.


Only fallback to non-TLS connections if explicitly requested

Currently the api.Dial function has a fallback to non-TLS if connecting with TLS fails. This function is used to connect to the Discovery Server, the Monitor Servers and Routing Services that don't announce their certificate. Although it prints a warning, this fallback is obviously not very secure, and should be disabled unless explicitly enabled by a user.

Done with api.AllowInsecureFallback


wsl terminal getting stuck if receive special char "▒"

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.17134.950]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)
WSL terminal getting stuck after receiving special char like "▒"  :

start  Debian app ( installed from Windows Store)
connect to Juniper firewall:

$ ssh fw
--- JUNOS 15.1X49-D170.4 built 2017-11-04 19:54:00 UTC

connect to secondary node:

{primary:node0}
fw> request routing-engine login node 1

What's wrong / what should be happening instead:
If I am using putty for the same operation I see such output:

{primary:node1}
fw> request routing-engine login node 0
▒
--- JUNOS 15.1X49-D170.4 built 2017-11-04 19:54:00 UTC
{secondary:node0}

Strace of the failing command:
https://gist.github.com/EpiqSty/e5c7865c495ff6d81bf07f0c6776ce46

[Version 10.0.17134.950]

There is not enough information here to repro. Try 1903 aka 19H1 aka 18362 aka May 2019 Update as a baseline. If it doesn't help please post specific CLI steps starting from clean install of your distro.

1903
19H1
18362
May 2019 Update

1903
19H1
18362
May 2019 Update

@therealkenc - thanks i'll try May 2019 Update and let you know


Eliminate warnings for default use.

Previously pex would warn for default usage scenarios under both
Python 3.11 and Python 3.12. For Python 3.12, the --pip-version
defaulted to 23.2 but a warning about vendored Pip 20.3.4 was issued
anyway; this is now fixed. For both Python 3.11 and Python 3.12, the
vendored version of packaging used the deprecated sre_constants
stdlib module; this is now fixed with packaging 23.1 being vendored
and selected for Python >= 3.7.
Fixes #2186

Hrm, so packaging 22.0+ drop support for LegacyVersion (an internal type really), but that means support for arbitrary equality === in version specifiers is also dropped. As such, this is a backwards incompatible change which Pex doesn't do. I'll back out this portion of the change for now while I think about a path forward for the packaging warnings.


create-turbo not working

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
N/A
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Windows
Which canary version will you have in your reproduction?
(empty)
Describe the Bug
npx create-turbo is not working. Running npx create-turbo using the latest and canary versions prompts with "Where would you like to create your Turborepo?" then fails while downloading with error ">>> no "package.json" found at"
Expected Behavior
npx create-turbo@latest -e with-vite -m npm needs to create a new local repo using the "with-vite" template.
To Reproduce
Run npx create-turbo@latest.
Additional context
This was working earlier today, and suddenly stopped working,
https://github.com/user-attachments/assets/6a135a56-6efe-472e-82bc-4e2bf5427aca

Same issue occurs to me.
System: Mac OS.
Package manager: both npm and pnpm.
I tried:

turbo daemon restart
turbo daemon clean
clearing npx cache
moving away all other repos that use Turborepo next to my dir


Fix project loading in VisualFSharp and FSharp solutions

Some projects were misnamed that were causing the VisualFSharp and FSharp solutions to not be able to load everything successfully.

Just an extra note, the only thing that changed was the cases of two project names.  I assume that builds just fine on Windows but was failing on Ubuntu.

What's the build error?

My guess is mono crashing (as usual).
On Sat, Sep 15, 2018, at 11:31, Nat Elkins wrote:

What's the build error?

— You are receiving this because you commented. Reply to this email
directly, view it on GitHub[1], or mute the thread[2].
Links:

https://github.com/Microsoft/visualfsharp/pull/5651#issuecomment-421612729
https://github.com/notifications/unsubscribe-auth/AGBEzjMKFdlmWAeRjVJTcSIwzb6pEM3cks5ubUdugaJpZM4WqbwM

It does appear to have to do with my changes actually.  From the log I see
2018-09-15T16:40:19.2233235Z MONO_ENV_OPTIONS= msbuild /p:Configuration=release /p:TargetDotnetProfile=net40 tests/FSharp.Core.UnitTests/FSharp.Core.Unittests.fsproj

I'll give this a quick once over to check for similar issues.

Oops, hadn't saved the file.

@NatElkins yeah, happens to me all of the time.

Encore un fois: https://github.com/Microsoft/visualfsharp/blob/master/tests/FSharp.Core.UnitTests/FSharp.Core.Unittests.fsproj#L122
https://github.com/Microsoft/visualfsharp/blob/master/tests/FSharp.Core.UnitTests/FSharp.Core.Unittests.fsproj#L123

@NatElkins thanks for sticking with this :-)


url_params on a page track view appear not to be sent to Matomo

Filing this issue also on behalf of @ericof because I chatted with him about this.
We've been trying to run some campaigns on the plone.org website, where we link from a twitter/mastodon post to an article on plone.org with url parameters. A link for example would be like this:
https://plone.org/news-and-events/news/2023/plone-6-0-6-released?mtm_campaign=releases&mtm_medium=social
The idea is that matomo picks up the url parameters and counts one hit to the releases campaign for the social media medium. However: Matomo never records this 'hit' .  We suspect that volto-matomo in the AppExtra component that gets configured only pushes the href but strips of any url params here:
https://github.com/eea/volto-matomo/blob/4a62439b4a1aaf6287b4ce0475ab259916edcdee/src/MatomoAppExtra.jsx#L12-L25

@avoinea, @nileshgulia1  this is quite important to the Plone Foundation :-) Could you prioritize it, please?

Alin is on holiday. I will look into this.

@nileshgulia1 Thank you! It would help us a lot if you are able to fix this

Should have been fixed in PR #35 last year already. Closing this issue


TypeError: Argument 'bb' has incorrect type (expected numpy.ndarray, got list)

I'm getting this while trying to train the matterport mask rcnn model on my own data created with this tool. Anybody know what this is caused by? Another person opened this issue here: https://github.com/cocodataset/cocoapi/issues/139 but closed it without saying how to fix.

If you weren't using the master branch, try updating to 0.1.5, and rerunning it on your dataset. I think that may fix your issue.
https://github.com/waspinator/pycococreator/releases/

Okay thanks, going to try 0.1.5 now

Looks like that error is gone, thanks!! However, I'm now getting a new error, but this one thankfully doesn't crash training. I'm not sure if it's because of the way the data was generated, config settings, or pycocotools api to be honest, but I'm going to create a new issue for it.


Checkout rp config always

Which issue this PR addresses:
Fixes (https://dev.azure.com/msazure/AzureRedHatOpenShift/_workitems/edit/11049871/)
What this PR does / why we need it:
This PR allows checking out of the RP Config to happen to for all ev2 generator pipelines. It Removes the one off checkout in the mirroring / deployment generations. It also removes the not needed dev pipelines.
Test plan for issue:
I tested that this PR works by running the generator pipelines based of this branch.
Is there any documentation that needs to be updated for this PR?
Not that I'm aware of... tech debt cleanup/ increasing the number of pipelines that pull from the RP Config.
Verifying with team to see if there is documentation on where the generators pull their configs.

/azp run ci


Update Device Orientation Notes

The Portrait first devices are being counter rotated by applications. We need to inform driver developers to take this into account.

@gerritsMSFT, thanks for the PR!