Running pysplit in parallel

Dear Mellissa,
First off, great package that really facilitates my analysis ! Thank you :)
I'm currently doing an analysis for which I need to estimate the back trajectories within a given region (in contrast to a fixed point). That is, I want to run HySPLIT / pySPLIT for many different locations within that region. As this requires quite some computational time, are you aware of any smart trick to run pySPLIT /HySPLIT in parallel?
I tried to spawn multiple processes in Python using the multiprocessing module, but there seems to be an issue with HySPLIT (not so much pySPLIT I guess) that prevents multiprocessing from running several processes in parallel. Although the multi-process seems to start, I only observe one of my CPUs running at a time + after some seconds the script stops asking for user input (e.g. start location) which looks like something coming directly from HySPLIT. I admit I have no idea why these messages are coming, but it may be that HySPLIT just cannot be run in parallel.
Minimal (almost) working example
The directories of the hysplit installation/meteorological files etc. have to be modified in order to run.
# %%
import os

import numpy as np
import matplotlib.pyplot as plt
import pysplit

from multiprocessing import Pool

from tqdm import tqdm

# Define paths //
hysplit_dir = r'/home/nrieger/hysplit.v5.2.3/exec/hyts_std'
working_dir = r'/home/nrieger/hysplit.v5.2.3/working'
meteo_dir = r'/media/nrieger/extern/data/hysplit/met'
storage_dir = r'/home/nrieger/Projects/paleo/ArabianSea/paper_3mon/data/hysplit/makran/'

os.chdir(working_dir)

years = [2009]
months = [3]
hours = [15]
altitudes = [4500]
runtime = -120

# Define 9 different lon/lat pairs which should be processed in parallel
lons = np.arange(60, 71, 5)
lats = np.arange(26, 31, 2)
xx, yy = np.meshgrid(lons, lats)
locations = list(zip(xx.flatten(),yy.flatten()))

def gen_traj_at_loc(lonlat):
    lon, lat = lonlat
    basename = 'W{:d}N{:d}'.format(lon, lat)
    pysplit.generate_bulktraj(
        basename, working_dir, storage_dir, meteo_dir,
        years, months, hours, altitudes, lonlat, runtime,
        meteoyr_2digits=False, outputyr_2digits=False,
        monthslice=slice(1, 32, 2), get_reverse=False,
        get_clipped=False, hysplit=hysplit_dir
    )

processes = 4

with Pool(processes) as pool:
    processed = pool.map(gen_traj_at_loc, locations)

Hi there, just wondering if you had any luck running Hysplit/PySplit in parallel?

Hi Niclas, I encountered a similar issue when directly applying multiprocessing for the pysplit.generate_bulktraj method. I then alternatively copied the whole "hysplit" directory using the "shutil.copytree" method before paralleling the pysplit.generate_bulktraj method (remember to rename the copied direcotry and change the parameters of the pysplit.generate_bulktraj method, mainly the "working_dir" and "hysplit" parameters), and delected the copied directory using the "shutil.rmtree" method after the pysplit.generate_bulktraj method was called. It seems a bit silly, but it does work for me.


The behavior of External Stream with nullable datatype need to be discussed.

If you create a external stream with nullable data type, like below.
drop stream if exists test1;
create external stream if not exists test1(
	int8_col nullable(int8),
	uuid_col nullable(uuid),
	ipv4_col nullable(ipv4)
	) settings type = 'kafka', brokers = 'redpanda-1:9092', topic = 'test', data_format='JSONEachRow';

The user data may lack some fileds such as the IPV4 address. So there are three situations for kafka messages sent by users:

{"int8_col":1,"uuid_col":"928c88b9-f37a-41d5-bef5-9cb3663c2445"}.There is just no such field, it's ok,Proton will read Null value and can parse this piece of message.
{"int8_col":1,"uuid_col":"928c88b9-f37a-41d5-bef5-9cb3663c2446", "ipv4_col":null}. Insert Null value explicitly, it's also ok.Ptoton will read Null value successfully.
{"int8_col":1,"uuid_col":"928c88b9-f37a-41d5-bef5-9cb3663c2447", "ipv4_col":""}. Insert empty string,this piece of data will lose.Because it will handle as exception in the readIPv4TextImpl function.The handling of this situation is different. The specific behavior depends on the data type.Some data type will have value, such as {"int8_col":"","uuid_col":"928c88b9-f37a-41d5-bef5-9cb3663c2448", "ipv4_col":null},you will get 0 value, the other data type will throw exception that means you will lose this piece of data.The read method implementation of each data type is different.

Below is the output:
timeplus-OptiPlex-7000 :) select * from test1

SELECT
  *
FROM
  test1

Query id: b421e811-a39f-442a-a17f-3f5f74e5f35f

┌─int8_col─┬─uuid_col─────────────────────────────┬─ipv4_col─┐
│        1 │ 928c88b9-f37a-41d5-bef5-9cb3663c2445 │ ᴺᵁᴸᴸ     │
└──────────┴──────────────────────────────────────┴──────────┘
┌─int8_col─┬─uuid_col─────────────────────────────┬─ipv4_col─┐
│        1 │ 928c88b9-f37a-41d5-bef5-9cb3663c2446 │ ᴺᵁᴸᴸ     │
└──────────┴──────────────────────────────────────┴──────────┘
┌─int8_col─┬─uuid_col─────────────────────────────┬─ipv4_col─┐
│        0 │ 928c88b9-f37a-41d5-bef5-9cb3663c2448 │ ᴺᵁᴸᴸ     │
└──────────┴──────────────────────────────────────┴──────────┘

The main point is if the data type is Nullable and parsing fails, should a Null value be returned instead of discarding the data?

IMO, parsing failures do not equal to Null. This should be discussed case by case, maybe in some cases, we want to enhacne how some values should be parsed/converted to some specific types.


Error when saving bitmap to PDF: Software rendering doesn't support hardware bitmaps

I'm trying to save a PDF file with a bitmap captured using this library drawn into it, however, I'm facing this error.
java.lang.Throwable: Software rendering doesn't support hardware bitmaps
	at android.graphics.BaseCanvas.throwIfHwBitmapInSwMode(:1031)
	at android.graphics.BaseCanvas.throwIfCannotDraw(:146)
	at android.graphics.BaseCanvas.drawBitmap(:186)
	at android.graphics.Canvas.drawBitmap(:1583)
	at ir.masterz.monex.content.monitor.ecg.EcgViewModel.generatePdf(:280)
...

My code:
        val file = File(fileDir, "ecg.jpg")
        file.createNewFile()

        val document = PdfDocument()
        val pageInfo = PdfDocument.PageInfo
            .Builder(595, 842, 1)
            .create()
        val page = document.startPage(pageInfo)
//line 280 is below
        page.canvas.drawBitmap(chartBitmaps.first()
            .asAndroidBitmap(), 0f, 0f, null)
        document.finishPage(page)
        document.writeTo(FileOutputStream(file))
        document.close()

Checking other issues like #31 and #13 it seems I need to disable the hardware rendering but how can I do that with my current code?
There's a page.canvas.isHardwareAccelerated but it's just a getter.

It seems making a new copy from the bitmap fixes the issue.
Bitmap targetBmp = originBmp.copy(Bitmap.Config.ARGB_8888, false);

val bitmapCopy = chartBitmaps.first().asAndroidBitmap().copy(Bitmap.Config.ARGB_8888, false)

https://stackoverflow.com/a/66837703/15055605


Add a Gitter chat badge to README.md

mattkrick/cashay now has a Chat Room on Gitter
@mattkrick has just created a chat room. You can visit it here: https://gitter.im/mattkrick/cashay.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 91.772% when pulling 916755c733bed1137e18425024dc3ce9fdbd3c2c on gitter-badger:gitter-badge into f880e0fc4bf266b8a03f3a1a7eb9a02be4e1455f on mattkrick:master.


Help with FOSCAM

Hi,
csan someone please help me config PTZ for Foscam (RTM) cam?

I think it's better to ask in Hass community
https://community.home-assistant.io/t/realtime-camera-streaming-without-any-delay-webrtc/258216/1
Or Discord


Same Event Applied Twice when Generating Aggregate Snapshot

Hi,
I'm currently experiencing some unexpected behaviour related to aggregate snapshotting and I suspect it may be due to a bug.
I'm using the EventCountSnapshotTriggerDefinition to control when aggregate snapshots are created. This is working fine and snapshots are being triggered when required. However, when the AggregateSnapshotter constructs the snapshot event it publishes the most recent event twice. For example, if the event count threshold is set to 100 then the snapshotter will publish event 99 twice. In most cases this is not an issue. However, there are some events in my current implementation for which the handling is not idempotent and this is creating aggregates with an invalid state.
When I debug the snapshotting process I can see that the duplicate event is being added to the event stream when a call is made to AbstractEventBus.queuedMessages() This method returns a list of all events held by the current unit of work and its parents. In our case, the parent BatchingUnitOfWork and its child DefaultUnitOfWork both hold a reference to the same event, which leads to it being duplicated in the resultant list.
Please let me know if you require any further details regarding this issue.
Thanks,
Paul

Hi Paul,
thanks for the elaborate report. Since you mentioned the BatchingUnitOfWork and the DefaultUnitOfWork, am I correct in assuming you're using an in-process snapshotter? In other words, you did not configure an ExecutorService on the snapshotter?

Hi Allard,
thanks for the prompt response. You're correct, we haven't specified an ExecutorService for the snapshotter. We're using the SpringAggregateSnapshotterFactoryBean with its default configuration.

Hi Paul,
I'm unable to reproduce the problem on the master branch. Looking at the code, it seems that Axon already takes into account that some of the events in the "staged events" may overlap with what's returning from the Event Store, as these events may, or may not have been stored there yet.
Which version of Axon are you seeing this behavior on?

We're currently using version 4.3.

Thanks for the clarification. I managed to find the culprit and resolved it in https://github.com/AxonFramework/AxonFramework/pull/1416

Great! Thanks Allard.

Closing this issue as it has been resolved in pull request #1416.


Second version of enable/disable output icon

We need a second version for the already implemented (#5928) icon, see https://github.com/orgs/enso-org/discussions/5964#discussioncomment-5386969
It should look like the already existing one, but without the stroke that crosses it:

QA Acceptance Criteria
The icon above is visible on the icons demo scene.

@wdanilo: Is the cut-out in the bottom left intentional? Just double-checking because we weren't sure and I couldn't find it in the Figma design.

@wdanilo: Is the cut-out in the bottom left intentional? Just double-checking because we weren't sure and I couldn't find it in the Figma design.

good question. It is accidental, should be removed. Thanks so much for catching it!

Great, good to know. I'll get rid of it in the PR.

Thanks again for catching it!


Implement support for selectable queries

[ ] Define acceptance criteria for new search queries
[ ] Implement code to fulfill these criteria
[ ] Deploy

Deployed to stage

Switched now to prod
https://api.wellcomecollection.org/catalogue/v2/works?query=flower&_queryType=slop


Remove duplicate output of usage.

flag.Usage() default also calls PrintDefaults() so the explicit call results in two usage messages.

Codecov Report

Merging #78 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
+ Coverage   69.52%   69.55%   +0.02%
==========================================
  Files          32       32
  Lines        2911     2910       -1
==========================================
  Hits         2024     2024
+ Misses        646      645       -1
  Partials      241      241

Impacted Files
Coverage Δ

cmd/wasm-run/main.go
25% <ø> (+0.38%)
:arrow_up:

Continue to review full report at Codecov.

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


testing

[Enter feedback here]

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

ID: d6230652-e478-a07e-45cf-5d560c276c13
Version Independent ID: 888cb6ce-2c26-3d98-1327-c91805083665
Content: What is an Azure landing zone? - Cloud Adoption Framework
Content Source: docs/ready/landing-zone/index.md
Service: cloud-adoption-framework
Sub-service: ready
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#335528


Stark: Prover and Verifier over field extensions v2

Prover and Verifier over field extensions v2
Description
This is an improvement over PR #716 that avoid casting the trace to the field extension. The immediate consequence is that the prover can no longer consider all trace polynomials and all trace LDE evaluations as a single object since they live in different fields. This PR refactors the prover's and verifier's code to handle those different types.
Evaluation of constraints to build the composition polynomial still considers all columns to live in the field extension. A followup PR will improve that to fully leverage the optimized ops between different fields. This is so divided in two different PRs for making the review process easier.

Codecov Report
Attention: 26 lines in your changes are missing coverage. Please review.

Comparison is base (6063197) 95.77% compared to head (ff7559b) 95.74%.

Files
Patch %
Lines

math/src/polynomial.rs
85.14%
15 Missing :warning:

provers/stark/src/proof/stark.rs
86.15%
9 Missing :warning:

provers/stark/src/verifier.rs
98.85%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #717      +/-   ##
==========================================
- Coverage   95.77%   95.74%   -0.04%
==========================================
  Files         135      135
  Lines       30136    30293     +157
==========================================
+ Hits        28863    29003     +140
- Misses       1273     1290      +17

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


Fix padding & kerning for Design +  Fix Strange Animation issue.

Replaced List with ScrollView remove the strange animations reported in #469 & #486
Which are seen when downloading any video course that has numerous episodes.
With the change to ScrollView  .fixedSize(horizontal: false, vertical: true)  had to be added to a number of Text-Views so the Text-Views could take all the vertical Space it needs.
Removed Padding causing vertical space issue on title and date reported in #517
Removed Padding causing vertical space issue on authors information reported in #522
Added kern of -0.5 to the title "Course episodes", section titles of and title of each episode
The spacing issue that required the kerning reported in issue #523 and #520
See title before and after adding the kerning() modifiers.

see image below for before and after.

@byaruhaf just checking, are all these PRs ready for merging?

@0xTim yea all are ready for merging.


Rake variable incorrect in deploy:assets:precompile

The deploy:assets:precompile task fails because the "rake" variable is not a string but an object of class "Capistrano::Configuration::Namespaces::Namespace".
/lib/capistrano/recipes/deploy/assets.rb, line 40:
run "cd #{latest_release} && #{rake} RAILS_ENV=#{rails_env} #{asset_env} assets:precompile"
It also does not let me set the "rake" variable in deploy.rb file with:
set :rake, "rake".
Using rails 3.10, capistrano 2.8.0.

I had this issue on my server (debian) but not on my development machine (ubuntu).
Both are using rake 0.9.2


Build error on Windows with MinGw

Hi all,
I get the following build error with glob on windows with MinGw:
single_include/glob/glob.hpp:170: undefined reference to `__imp__dupenv_s'

There is a proposed fix in https://github.com/p-ranav/glob/pull/23


Make memo less prominent.

Memo can be very distracting. Some people are using it to spam users on
the network. This change hides it behind an icon.

I also changed how the loading state is represented by adding an image animation.

I actually think this could be detrimental. If someone wants to build a protocol on top of Stellar using micro transactions and memos, it would be nice to have it show up on a block explorer.

You can still view it, it's just less visible.

@bartekn actually, this was reverted a while ago and I don't know why.
https://github.com/stellar/account-viewer/commit/59e29a53e58cfaad1563b8bc91444ecfcecf8b12
Could you explain why did the code get reverted, @tomerweller?

We decided to use this https://github.com/stellar/account-viewer/pull/59 instead.


default value type covercion

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description
default values are not type coerced.
I won't argue if the expectation is correct, its a difference in behavior from v1.
Rejecting the model or coercion of the type would be better than the current behavior.
Example Code
import decimal

from pydantic import BaseModel, Field

class A(BaseModel):
    v: decimal.Decimal = Field(default="1")

a = A()
assert a.v == 1, f"{a.v=}"
# AssertionError: a.v='1'

Python, Pydantic & OS Version
pydantic version: 2.0a2
        pydantic-core version: 0.23.1 release build profile
                 install path: ~/venv/openapi3/lib/python3.10/site-packages/pydantic
               python version: 3.10.6 (main, Mar 10 2023, 10:55:28) [GCC 11.3.0]
                     platform: Linux-5.19.0-38-generic-x86_64-with-glibc2.35
     optional deps. installed: ['email-validator', 'typing-extensions']

Thanks @commonism for reporting.
You can use validate_default which is a replacement for validate_all in V1.
class A(BaseModel):
    v: decimal.Decimal = Field(default="1", validate_default=True)

I won't argue if the expectation is correct, its a difference in behavior from v1.

What is the diff?

I'm wrong, matches v1 behavior.
I butchered my unit tests to report this as failure.


Typescript paths resolved automatically

What problem does this feature solve?
Typescript has this neat feature where you can set custom paths for your directories like so:
{
	"compilerOptions": {
		"paths": {
			"@/*": [
				"src/*"
			],
			"@Components/*": [
				"modules/static/shared/components/*"
			]
		}
	},
}

This helps you to not write those long imports in your code when you have a really big up with hundreds of components. The problem is that, in the current version, Vue does not support path aliases out of the box.
This feature should be able to solve them out of the box. In a simple typescript environment, in Node.js, it is usually done with packages like "tsconfig-paths" by calling tsconfig-paths/register when running your node app.
The end user experience is that the devs won't have to spend hours setting it up due to lack of documentation (just like I do right now and no solution found).
Suggested solutions on the internet don't work:
Solution 1 (doesn't work)
// vue.config.ts
module.exports = {
	chainWebpack(config) {
		console.log('consoleProfileFinished', config)
		config.resolve.alias.delete("@")
		config.resolve
			.plugin("tsconfig-paths")
			.use(require("tsconfig-paths-webpack-plugin"))
	},
}

Solution 2
// vue.config.ts
const path = require('path')
module.exports = {
	chainWebpack: config => {
		config.resolve.alias
			.set('@', path.resolve(__dirname, 'src/main/javascript'));
	}
}

What does the proposed API look like?
There is no special API for this feature. You just use typescript paths in your tsconfig.json file

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.

hi @posva
I've created a topic here but I've added this Github issue as a feature request. It would be nice to have typescript path aliases working in the project out of the box.
What do you think?


Remove the possibility of fetching abilities from Profile

Ticket
https://netguru.atlassian.net/browse/PEOPLE-XXX
Description

what this PR is doing

Before merge checklist

[ ] CircleCI is passing
[ ] Proper labels are set

LGTM


Unable to get and run docker image

$ sudo aptitude install docker nvidia-docker nvidia-container-toolkit docker-compose docker-buildx

Added my user to kvm group et la.
$ newgrp docker
$ docker pull styler00dollar/vsgan_tensorrt:latest

ERROR:
        Can't find a suitable configuration file in this directory or any
        parent. Are you in the right directory?

        Supported filenames: docker-compose.yml, docker-compose.yaml, compose.yml, compose.yaml

$ git clone https://github.com/NotGlop/docker-drag
$ cd docker-drag
$ python docker_pull.py styler00dollar/vsgan_tensorrt:latest

Took so long and after ~10 hours of waiting with net speed around 100mb / sec on download/ upload,
so i decided to cancel it. The last file that even havent finished pulling was 5537e544ef988f4cd82c06b8d0c919448d2f76ac33509a9b5fb5979b73216c58.json

This repository contains yaml files and you did not run the command within the vsgan folder. docker-drag is not supposed to take that long, but generates a ~50gb tar file.

I have around 7 TB of free ext4 space and have tried on 200 GB zfs.
I need to clone this repo and run compose or somehow get to the pulled one ?

Clone repo and run docker-compose run --rm vsgan_tensorrt within it.

Please read more carefully, I did write # go into the vsgan folder.

And when i saw this i got though that i need to find where it got pulled.


2 instances at the same time / updating in same folder

Hi,
I am very happy with your docker solution for the Synology. it works flawless.
I want to run 2 instances, one for long trading and one for short trading. is this possible with docker? Curious how to do that.
udating to new versionis more complex because i have to make new folder and do the installation all over again. is there a way to do the update via the same folder and not having to install everything all over again?

Hello,
I didn't try to update on the same folder before, I always just install in a new folder.
If I get some free time I will try to see the situation of update on same folder and update the instructions.

In the case of running 2 instances, it should work if you have 2 different container names and folders.


Update unsafe_unlink.c

this change makes unsafe_unlink work with the latest version glibc

Can you also update the size so that it can still work with tcache-eanbled libc?

And can you please also include a link to what this patch is against?
(What new protection was introduced to make the previous unsafe_unlink not working?)

Can you also update the size so that it can still work with tcache-eanbled libc?

not sure what you mean by this

And can you please also include a link to what this patch is against?
(What new protection was introduced to make the previous unsafe_unlink not working?)

I am pretty new to heap exploitation.. so i will dig deeper and will add more details

Oh. I mean
int malloc_size = 0x80; //we want to be big enough not to use fastbins

In this line, can you change it to something big like 0x500. So that the chunk will be put in unsorted bin after free.

Thanks!

This is added during the refactor. Thanks for the input!


add some adjustment

Hi, @iampavangandhi I add some adjustment.
#3

@vnxx Kindly resolve the conflicts then test it again and provide screenshots pointing out changes you made

Hi @iampavangandhi , thanks for the response.
My updates:
I add some padding to this section

I change the background color so it's more readable

And I add some changes to the contact us form

@vnxx Thanks for the contributions but all the changes you showed via screenshots are either already done by other contributors or personal design choice (padding) also your forks is not up to date hence your PR is causing problems (removing other contributor changes) thats why I can't merge.
I suggest you to either refork or sync your fork with the master and recreate better PR. Also check https://iampavangandhi.github.io/ once again


Blocks of source content are reloaded when navigating to another page

Describe the bug
If I'm on a page that has blocks, and I click on a link to leave the page, the blocks on the source page are reloaded. This is bad for performance, especially if the blocks make requests to the backend. Requests will be made unnecessarily. Is this behavior expected? Or is this a bug?
To Reproduce
Steps to reproduce the behavior:

Create a page with a listing block with some search criteria.
Open developer tools on the network tab.
Click on a listing result link.
Note that a request is made to /++api++/@querystring-search, even if the target content has no listing block.

The linting block was just one example. All blocks of the source content are reloaded. I put console.log in other blocks and confirmed that they reload.
Expected behavior
The blocks of source content must not be reloaded.
Screenshots
If applicable, add screenshots to help explain your problem.
Software (please complete the following information):

OS: [e.g. iOS]
Browser: Chrome
Volto Version 6.4.0
Plone Version 6.0.0
Plone REST API Version 8.32.6

Sounds like a bug to me.

There's definitely something fishy going on. I couldn't replicate the bug (I see no extra @querystringsearch calls), but check the amount of extra api calls:

Each extra call is triggered by the presence of a listing block on the origin page. More listing blocks, more extra calls.

I couldn't replicate the bug (I see no extra @querystringsearch calls)

@tiberiuichim  have you defined a search criteria in your listing blocks? @querystringsearch is only called when we have a search criteria. That problem we're seeing with the listing block, which has two different data sources.
But remembering that all blocks are reloaded, not just the ones that make requests to the backend. The that make requests only make the problem easier to see.

I think #3055 is relevant for this problem.
I've tested and the flow is the following:

I click a link in the listing block
the listing block is unmounted
the listing block is recreated at the new location URL
which triggers the extra network call

@reebalazs do you have time to look at this?

Also relevant: https://github.com/plone/volto/issues/1528


Add switch to schema migration job auto-removal

Hi everyone,
for our local deployment, we use ArgoCD as the Kubernetes CI tool. When the TTL for finished schema migration jobs is set and they are removed after completion, we hit an infinite loop where Argo attempts to recreate the job (because it doesn't exist anymore), which is then removed again, and so on...
My suggestion is to add a flag in the Helm values, so that the auto-removal can be disabled. I was not sure what might be the better default value for this, so i left it enabled by now.
Cheers,
Hannes

Thanks @bauerjs1 and apologies for taking so long to get to this...your changeset seems reasonable to me.

Since its been so long since this was opened and I don't have write access to the fork, here's what I'll do:

merge this into our integration branch
add my commit on top
open a new PR to get this into main

Sorry as well, @lmsurpre, for taking so long... thank you for the integration! Good idea to just make the timeout configurable.


Option to style tag only instead of the whole line

Hi,
I saw another extension TODO Highlight v2 has options to style tag only. It also add border color and border radius options. I would prefer this because colorful comments is distracting.
Your extension plans to use tokenization to get comments. It’s a better practice than parsing the whole file with a regex like many other extensions do. 👍

Ill see what I can do, shouldn't be too hard to add in. I only just finished moving to tokenization in combination with regex today so it may be a bit before anything gets published. Thanks for the recommendation.
In regards to your comment on tokenisation, it was the only way I could actually be sure what I am about to add a decoration to is a comment, its nearly impossibly to handle all edge cases when it comes to comments especially for all the languages so I had to figure out a way to tokenise everything. The main problem with it and why most addons avoid it is, tokenisation is far more intensive to compute on documents and needs to be done sparingly, I still use regex to find potential candidates, but I use tokens to ensure it is actually a comment. The only reason I could finally add it was I created a class which acts like a boolean array that I could do bitwise operations on and each bit in it represents if there is a comment on a line or not so I can parse the document multiple times and skip all of the lines that already have comments on them. It still needs a ton of work, but I also found out I was doing a number of things multiple times per document in the previous version so overall, the extension should actually be much faster despite the switch to tokens.

Good job ! 👍

I have implemented a feature to let you only style the tag. Its a setting called "highlightTagOnly", there is also a small temp fix for border options you can use until I properly fully integrate the settings into the extension. It also includes some temp options for outline and for text decorations. For each tag decoration, you can specify the options "CSSTextDecoration", "CSSOutlineDecoration", and "CSSBorderDecoration" which will let you have a majority of control over how you would like it to look.
For TextDecoration, see this page
For Outline, see this page
For Border, see this page

I will test it this morning.
Is there a reason why the extension icon is so dull ? It always have been and wondered why.

Everything seems to work. Thanks.

One note : the border or outline is applied to the tag AND the space after it. It needs to be fixed IMO. It should be applied on the tag only.
Another request to improve the tag only feature. It should not be a global setting but a tag property to allow fine-grain customization.
Could you add a property to custom border-radius ? 😅

The problem with the space being highlighted is one would expect the colon to be highlighted as well in “todo:” so I will need to check what characters come after the tad and this can cause some problems if /when I let users set their own tag separators. It’s a weird problem.
As for the border radius, it’s weird that you cannot set that with the css border options, I’ve always hated css with a passion. I’m currently working on a way to let users specify every css option without me specifically adding support for each one.
lastly on the icon, purple is my favourite colour so when I was quickly making the icon that’s why I made it purple, I also am a fan of dark themes so I tried to get it to work nice, but it is a bit too dim for my liking too. Another reason is I have an OLED monitor so editing bright images and not moving it much can cause ghosting and reduce the life of that specific part of the screen so I try to avoid anything too bright, I have dark mode for just about everything I use.

Opened two separate issues to discuss further #7 and #8 , the main topic of this issue has been solved. Closing.


Defender tamper protection

1903 added Tamper protection feature into Windows Defender. This basically blocks all attempts to modify defender-related registry, unless you're doing it as TrustedInstaller user.
By default, the protection seems to be disabled, but nags with a warning. Scope of this issue is either:

Find how to disable the warning (preferred)

or

Enable tamper protection but find how to do the modifications in other defender-related tweaks, either via Set-MpPreference cmdlet or directly as TrustedInstaller (this theoretically should not be possible, but where there's a will, there's a way)

There is related setting under HKLM:\SOFTWARE\Microsoft\Windows Defender\Features but manually dismissing the warning doesn't seem to change anything anywhere in registry, so there may be something else in SQLite databases under C:\ProgramData\Microsoft\Windows Defender.

Perhaps the TamperProtection key in HKLM\SOFTWARE\Microsoft\Windows Defender\Features might be something that is only honored during the initial install of the OS, similar to the ShippedWithReserves key for the Reserved Storage feature?


[Feature Request] Per-app settings

Hi there! I feel like it'd be great if it also provides per-app settings, or even apple script definitions to programmatically edit the settings on the fly. Still, this is a great app! Thanks :D

I concur, side buttons work normally on some apps (without universal back), but stop working with universal back enabled.
e.g. Telegram Lite

I concur, side buttons work normally on some apps (without universal back), but stop working with universal back enabled. e.g. Telegram Lite

This is related with #57.
LinearMouse's current solution is to keep a list of apps for which the universal back and forward feature should be disabled.
However, based on my observations, it appears that quite third-party apps do not support swipe gestures but do respond to side buttons. Perhaps we should do the opposite and keep a list of apps for which the feature should be enabled?
Can anyone find examples of third-party apps that only support swipe gestures but do not respond to side buttons?

Can anyone find examples of third-party apps that only support swipe gestures but do not respond to side buttons?

It would be great to add games like dota, cs:go to ignore list, because side buttons are often used for game buttons

@aramann That makes sense! It's weird that CS doesn't have a bundle id...
Would you like to send a pull request to add com.valvesoftware.dota2 into the ignoreSet?

@aramann

hope I made it right

Looks pretty good!

com.valvesoftware.steam isn't working for cs. It runs with a unix file and doesn't have .app

You are right. I just saw the log and the bundle id was (nil).
I'll merge your pull request first.

Closing this issue since per-app settings can now be configured through configuration.


Revert "build(deps): update prometheus to v2.31.2"

This reverts commit 05238b1167bd83325f55a82b86abad10f2c88ac4.
Prometheus currently doesn't have a version v.2.31.2 available, which is causing issue when pulling images.
https://github.com/prometheus/prometheus/releases

Oddly there still is a tag but no release: https://github.com/prometheus/prometheus/releases/tag/v2.31.2.


NIFI-12777 Add support for UUID record field type in QueryRecord processor

Summary
NIFI-12777
Tracking
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-12777
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-12777

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check
[ ] JDK 21

+1 LGTM, thanks for the fix! Merging to support/nifi-1.x


Basic Node and Express: Breaking changes on Glitch

Describe your problem and how to reproduce it:
As can be seen by the numerous topics opened on the forum, over the last few days, Glitch appears to have undergone some breaking changes causing the fcc-express-bground-pkg to error out.
Specifically: https://github.com/freeCodeCamp/fcc-express-bground-pkg/issues/4
NOTE: bowser is not the issue. The issue is what is being requested by the selfCaller function. Somehow, this line is presenting this data:
selfCaller (fcc-express-bground-pkg/index.js) Line 29
r.on('data', (d) => {cb(d.toString(), req, res, r.headers)});

Where d.toString() logs this:
"Glitch Project HTML"
```

Maintenance

Waking up

          To keep Glitch fast for everyone, inactive projects go to sleep and wake up on request.

```

Resulting in this error on Line 80 : ` lowerCase = JSON.parse(lowerCase).message;`

SyntaxError: Unexpected token < in JSON at position 0
at JSON.parse ()

Add a Link to the page with the problem:
Editable Project: https://glitch.com/edit/#!/join/be8e1d09-57c3-4e1f-8b39-ad8ecee5562f
I pasted the fcc-bground package within the server.js file to debug.
Relevant Forums: https://www.freecodecamp.org/forum/t/problem-with-passing-basic-node-and-express-use-the-env-file/207721/66
https://www.freecodecamp.org/forum/t/basic-node-and-express-use-the-env-file-get-error-cannot-set-headers-after-they-are-sent-to-the-client/405090
...and many others
Additional Info

The tests and project work in both repl.it, and CodeSandbox.
There is one other issue with the fcc-bground-pkg package mentioned here, but not necessarily relevant to the bug.
Another point of interest: These lines are in server.js

var allowedOrigins = ['https://narrow-plane.gomix.me', 'https://www.freecodecamp.com'];
var origin = req.headers.origin || '*';

origin is always logged as *.

Relevant endpoint: /_api/use-env-vars

Setting a User-Agent in the header for the http.request in selfCaller seems to fix it
https://glitch.com/edit/#!/basic-node-and-express-challenges-with-user-agent
Side note: it is failing the console log challenge but I think that's just because all the code is in one file and how the log challenge gets tested. I didn't really bother to look at it.
@Sky020 @ojeytonwilliams Can you confirm this?
Not sure what user-agent we should use, or if matters? I can make a PR on https://github.com/freeCodeCamp/fcc-express-bground-pkg if this actually is the fix.

This is the starter project with a new link in the dependencies to my repo with the headers fix
https://basic-node-and-express-challenges-with-new-pkg.glitch.me
As far as I can tell, everything seems to be working.

@lasjorg , I have not been able to access Glitch projects, for the whole day. So, will see if I can confirm some other time.
I was looking into simplifying the test routes, and completely removing the reliance on fcc-express-bground-pkg.


gampeass version of yakuza 0 doesnt seem to work with mod

i use the xbox pc version and the mod doesnt seem to work with it unfortunately, hopefully you can update it so it can soon cuz this is a great qol mod.

I'm sorry but I don't have the gamepass version of the game, I can't really test it

Same for me.


Like_hashtags_from_file error

When running like_hashtags_from_file I receive this error and would appreciate any help:
2018-07-30 17:15:56,096 - INFO - Logged-in successfully as '******'!
2018-07-30 17:15:56,096 - INFO - Saved cookie!
coercing to Unicode: need string or buffer, list found
2018-07-30 17:15:56,097 - INFO - Hashtags: []
2018-07-30 17:15:56,409 - INFO - Bot stopped. Worked: 0:00:14.151062
2018-07-30 17:15:56,409 - INFO - Total requests: 3

hello, i searched on stackoverflow, it seems your error is a python one, what version of python do you have installed and what os are you running?

Hey, I have python 2.7.10 running under osx 10.13.5


Publish Microsoft.AspNetCore.Owin API

get Microsoft.AspNetCore.Owin 8.0.6 and the latest 9.0 preview added to the published API’s
@tdykstra  ~have the API ref pages instructions been updated to the latest?~
Internal work item

Associated WorkItem - 263799

Owin.IAppBuilder is documented in a compliant way, the


Add Swiftly to both the "Transit Agencies" section and the "Native apps (closed source)" section

Name: Swiftly
Link: https://goswift.ly/

Thanks @boconnell!  Can you please open a pull request to add these?
On a related note, I'm not sure Swiftly fits into the "Transit Agencies" section, but we can discuss over comments in pull request.


Optional icon generator by generateIcons option

The reason for this commit is that I use another custom generator for favicons (it generates PNGs from an SVG with a custom filename). Because of this, when esbuild-plugin-html tries to load the favicons referred URLs, it fails with the following error:
✘ [ERROR] Failed to resolve icon path: assets/icon-dark-196x196.png [plugin html]

✘ [ERROR] Failed to resolve icon path: assets/icon-dark-196x196.png [plugin html]

[build][resources][manifests]: 25.672ms
Error: Build failed with 2 errors:
error: Failed to resolve icon path: assets/icon-dark-196x196.png
error: Failed to resolve icon path: assets/icon-dark-196x196.png
    at failureErrorWithLog (node_modules/esbuild/lib/main.js:1472:15)
    at node_modules/esbuild/lib/main.js:945:25
    at node_modules/esbuild/lib/main.js:1353:9
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  errors: [Getter/Setter],
  warnings: [Getter/Setter]
}
[build]: 30.824ms
[build][resources][assets]: 38.637ms

Therefore, to ignore these errors, I added the build option generateIcons as a flag for the collectIcons.
The default value is true, so its behavior will be the same as always.
I added a test for generateIcons set to false.

Hello,
Is there anything I can do to help you accept or review this PR?
Greetings.

Hello @jlsalvador! Sorry for the late reply. If you are still interested in pursuing this PR, could you please rebase and add the same option for the collectScreens method as well?

Changes from V1 to V2:

Rebased from main.
Renamed generateIcons to generateFavicons.

Hello @jlsalvador! Sorry for the late reply. If you are still interested in pursuing this PR, could you please rebase and add the same option for the collectScreens method as well?

@edoardocavazza I think for collectScreens could be another option. For example, I might not want to generate the favicons but still need the Apple launcher icons. What do you think?

I agree!

Hi again!
Anything left to accept this PR? I thought about collectScreens would be another PR.


Provide a way to exclude certain routes/services from route decorators

For example, a user might not want to let LoggingService from logging the requests sent to HealthCheckService.
We could provide a decorator implementation which skips the next decorator when the current request matches a certain route or the execution chain contains a certain service instance/type.
To implement this feature, we might need to improve the Service.unwrap() API, e.g. add unwrap() without parameters, which always unwraps one layer.

This was fixed by #3562 and #3081


[FEA] Samples for AIR Evidence Creator + documentation

IHE AIR is a profile which defines the specific types of objects an AI application should generate, to be conformant with imaging viewers. It defines, for example, a DICOM Segmentation object should be used for a segmentation - but it has additional values required for it to be compliant (including the notion of TID1500 and a 'root result'). It would be of great value to the community to have a sample MAP that produces an AIR-compliant result, which can be used to inspire / inform developers as well as application validators (like IHE Connectathon).
In addition, there should be a section on Standards, maybe in the introduction:
https://docs.monai.io/projects/monai-deploy-app-sdk/en/latest/introduction/index.html
... that provides some detail on how MONAI MAPs can be used to satisfy AIR Evidence Creator actor and transactions.

For Comprehensive 3D SR as evidence, TID1500 (Measurement Report) is used as the root template, but with regard to DICOM Seg how is TID1500 related? Per Table 6.5.3-1, Seg IOD is separate and distinct from TID1500. What's the complaint from IHE Connectathon validation for Seg object from the example app, apart from not having Tracking Identifiers which is for longitudinal results navigation.


Updated documentation for Service Broker Tile v 1.2.0

We've pushed a minor update to the service broker today, it would be great if we could get these corresponding documentation updates committed as well.

@akirasoft We've been experiencing challenges with our docs pipelines today so we've been very busy trying to fix it and pushing manually for a lot of folks. If we don't get to this before the end of the day, we will tomorrow.

Thanks @akirasoft!


Fix typos

Found via typos --hidden --format brief

 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.

coverage: 90.61%. remained the same
when pulling 8d66ffec4938bd07526635c8dcc755213d85795b on kianmeng:fix-typos
into 76ad82303fcbf0e4ab421dbfbb39e4a4cc76aecc on jmathai:master.


Dependency issue when referencing another project in the solution NU1007

Hello, any ideas why I'm getting this warning on my Asp.Net project after referencing another project I have?

Can you show your project.json file?

The project reference where the issue occurs is on is XXX.YYY.BusinessLayer": "1.0.0-* and its dependencies
{
"userSecretsId": "aspnet5-Upm.BioForest.WebApi-20160420113805",
"version": "1.0.0-*",
"compilationOptions": {
"emitEntryPoint": true
},
"dependencies": {
"Microsoft.ApplicationInsights.AspNet": "1.0.0-rc1",
"Microsoft.AspNet.Authentication.Cookies": "1.0.0-rc1-final",
"Microsoft.AspNet.Authentication.OpenIdConnect": "1.0.0-rc1-final",
"Microsoft.AspNet.Diagnostics": "1.0.0-rc1-final",
"Microsoft.AspNet.IISPlatformHandler": "1.0.0-rc1-final",
"Microsoft.AspNet.Mvc": "6.0.0-rc1-final",
"Microsoft.AspNet.Mvc.TagHelpers": "6.0.0-rc1-final",
"Microsoft.AspNet.Server.Kestrel": "1.0.0-rc1-final",
"Microsoft.AspNet.StaticFiles": "1.0.0-rc1-final",
"Microsoft.AspNet.Tooling.Razor": "1.0.0-rc1-final",
"Microsoft.Extensions.Configuration.FileProviderExtensions" : "1.0.0-rc1-final",
"Microsoft.Extensions.Configuration.Json": "1.0.0-rc1-final",
"Microsoft.Extensions.Configuration.UserSecrets": "1.0.0-rc1-final",
"Microsoft.Extensions.Logging": "1.0.0-rc1-final",
"Microsoft.Extensions.Logging.Console": "1.0.0-rc1-final",
"Microsoft.Extensions.Logging.Debug": "1.0.0-rc1-final",
"Microsoft.VisualStudio.Web.BrowserLink.Loader": "14.0.0-rc1-final"
},
"commands": {
"web": "Microsoft.AspNet.Server.Kestrel"
},
"frameworks": {
"dnx461": {
"dependencies": {
"XXX.YYY.BusinessLayer": "1.0.0-*"
}
},
"dnxcore50": { }
},
"exclude": [
"wwwroot",
"node_modules"
],
"publishExclude": [
".user",
".vspscc"
],
"scripts": {
"prepublish": [ "npm install", "bower install", "gulp clean", "gulp min" ]
}
}

Can you share the project.json from XXX.YYY.BusinessLayer? This file looks fine to me so far...

BusinesssLayer is a .Net 4.6.1 class library, can this be the problem?

Package.config

@pranavkm can you take a look?

@anurse @pakrym @davidfowl do any of you guys have an idea on this? (Note that this was with RC1.)

Are any csproj files in the picture?

Do you have a lock file you could share?

@connde - bump

BTW I set up a quick mtg for Monday to discuss this.

Hello, sorry for the delay, We migrated to RC2 and this was fixed, apparently..


Fix NaN values during BSSRDF sampling

This is the fix related to Issue #1324 .

Looking good, thanks! You can address the minor comment I made in a later PR, it's a detail.


A link to the video in the documentation is broken

https://arend-lang.github.io/documentation/
When I try to open the video there, I get "Sorry, we couldn’t find that page".

Fixed in 70ccfc3.


Paths - get rid of join()

In most cases this means, paths are now having a backslash on Windows and not slash as our previous implementation.
Only for relative paths I left in the slash via posix.join.

Obsolete by https://github.com/Microsoft/vscode/commit/39566fa5e348996c4a573e7901149ff8487e4af8


Need an S3 bucket to be used by channel-diff script

In https://github.com/NixOS/nixos-search/pull/206 I created a script that outputs diff between 2 channels. I'd like to run this on a periodic schedile - daily - via github actions and store this json files somewhere (I thought of S3 bucket).
Finally I would consume this json files in the status.nixos.org frontend.
/Cc @edolstra

You could have it store the artifact in GitHub, e.g. using https://github.com/actions/upload-artifact.

@FRidh yes, but then I this artifact is not accessible via some public URL and can not be reused from eg. search.nixos.org or status.nixos.org.


Filter items by durability % like Ender IO conduits could.

Describe the feature you'd like to have added
I'd like to be able to transport items when they hit below 50% durability (or any other percentage of durability remaining).
This could be either a setting in one of the existing filters or a whole new filter specific to durability. Ender IO used to have a durability filter and I know Laser IO's origins stem closely from Ender IO.
My specific example being I would like to transport the occultism miner foliot out of the dimensional mineshaft when it gets below 50% durability. Then placed in the cyclic anvil to be repaired and replaced in the dimensional mineshaft when fully repaired.
As of right now, I can filter matching metadata. The miner foliot has 1600 durability, so I have durabilities 804, 803, 802, 801, 800 set to be extracted, but if the miner foliot ever drops below 800 durability before being extracted, it will be used up in the dimensional mineshaft.
Additional information
No response

The challenge with this request is durability USED to be stored as Metadata, but now that its stored as NBT its possible that mods will use a different value for their durability. It will be inconsistent and unreliable, unless I made something more complex (Like Integrated Dynamics) and I think thats beyond the scope of what I'd like to do with this mod :).
Integrated Dynamics could probably accomplish what you're after though!


Fix incorrect JetStream store dir when using $XDG_DATA_HOME #962.

Eliminated duplicate (and incorrect) function to determine the per-user data directory based on $XDG_DATA_HOME.
Kept dataParentDir() function to maintain concept, but delegated to correct implementation xdgShareHome().

Thanks a lot


[magnum] Build error

Host Environment

OS: [e.g. Windows/Linux etc...]
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
./vcpkg install xxxx
Failure logs
-Cut and paste the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

@MachinMigueldLeon Thanks for posting this issue, could you please provide detailed error log?

@MachinMigueldLeon  Is still work for this issue?

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!


Why members of CompletionItem are pointers?

What's the reasoning behind some CompletionItem members like Kind, Details, Deprecated, Preselect, etc... are defined as pointers?

Because they are optional. The nil value should be interpreted as "doesn't exist".
This should definitely be documented better...

Have you considered using an optional type?
Like:
type Option[T any] struct {
   value  T
   isSetted bool
}

func Some[T any](value T) Option[T] {
   return Option[T]{value: value, isFull: true}
}

func None[T any]() Option[T] {
   return Option[T]{isFull : false}
}

The project started before generics were available. It is definitely possible to switch, but ... for what it's worth, Go has nils. (Terrible design decision.) And nils are very often used for exactly this semantic intent. I don't love it, but it's common practice in Go (and other null-enabled languages) and I think changing this now may annoy current users of this library.

I'm ok with it!

Thank you for the discussion, it's useful. I'm not opposed to changing in the future. I will re-open this as an issue to document the use of nils, at the very least.

Thank you too.
I'm not an expert with Go, explaining me the context allowed me to understand better the design.


data exports: Handle pending and failed exports.

Discussion on czo
Prior to this change, there were reports of 500s in
production due to export.extra_data being a
Nonetype.  This was reproducible using the s3
backend in development when a row was created in
the RealmAuditLog table, but the export failed in
the DeferredWorker.  This left an entry lying
about that was never updated with an extra_data
field.
To fix this, we catch any exceptions in the
DeferredWorker, and then update extra_data to
encode the failure.  We also fix the fact that we
never updated the export UI table with pending exports.
These changes also negated the use for the somewhat
hacky clear_success_banner logic.
Testing Plan:
tools/test-backend
tools/run-dev.py

Can you rebase this?

Rebased :) looking forward to feedback.

This is great, merged, thanks @whoodes!


Copilot fails when asking about @workspace - "Maximum call stack size exceeded"

VS Code Version: 1.85.0
OS Version: MacOS Sonoma 14.2
Logs:

2023-12-13 16:04:49.219 [info] [chat fetch] engine https://api.githubcopilot.com/chat
2023-12-13 16:04:49.219 [info] [chat fetch] modelMaxTokenWindow 4096
2023-12-13 16:04:49.219 [info] [chat fetch] chat model gpt-4
2023-12-13 16:04:55.791 [info] [chat fetch] request.response: [https://api.githubcopilot.com/chat/completions], took 6571 ms
2023-12-13 16:05:01.927 [info] [streamMessages] message 0 returned. finish reason: [stop]
2023-12-13 16:05:01.930 [info] [streamChoices] request done: requestId: [53194cf0-f299-4a01-b519-a3d45c1d19df] responseId: [f37eca89-96a9-4441-814b-cad0b5d1307f] model deployment ID: [x8d3c95500d67]
2023-12-13 16:05:02.274 [info] [extension] RangeError: Maximum call stack size exceeded
    at y7 (/Users/bartlomiejsmietanka/.vscode/extensions/github.copilot-chat-0.11.0/dist/extension.js:508:2409)
    at /Users/bartlomiejsmietanka/.vscode/extensions/github.copilot-chat-0.11.0/dist/extension.js:508:4414
    at T7.getBasicChunks (/Users/bartlomiejsmietanka/.vscode/extensions/github.copilot-chat-0.11.0/dist/extension.js:508:4489)
    at /Users/bartlomiejsmietanka/.vscode/extensions/github.copilot-chat-0.11.0/dist/extension.js:511:5017

Steps to Reproduce:

Type @workspace <any prompt here> in Copilot Chat
Fails after some time with above message
There's no issue when running @workspace /tests inside some methods' context, runs as expected (writes some unit test).

If there are some additional information that I could provide let me know.

In the past, we've seen this in the semantic chunking space, so I'll defer to @andreamah for initial investigation.

I'm having same problem.

I'm unable to use the new LLM inside workspace because of this problem

Same issue on my side, any update ?

Same issue

same issue

Same issue on macbook m1 pro with osx ventura, latest vscode, and latest copilot

Does this still happen on the prerelease version of the extension?

same issue here. Solved it by switching to the prerelease version (v0.12.2023120701)

same issue here. Solved it by switching to the prerelease version (v0.12.2023120701

Does this happen on the latest prerelease?

I was able to avoid the issue on the pre-release version, thanks for the suggestion

Sounds like the fix is in the latest prerelease. Closing this unless others can repro.

I still have the same issue, do you know when the fix will be merge to release ?

The next stable release should have it.


[fixed] Refactor React.createClass, PropTypes, string refs, and fix fakeRequest

This work is based on #223 and also fixes issue with fakeRequest when value === '' and callback doesn't get called.

Fix has been implemented, which makes PR no longer valid.


Make actuator optional

Following on from discussion in #1175

https://github.com/spring-cloud/spring-cloud-stream/pull/1244


[EPIC] - Implement LocalStorage for Quiz Progress and previous high scores

What feature would you like to see?
I would like to propose a feature that will enhance user experience by utilizing localStorage to save quiz progress and their previous high scores.
Feature Overview:
Quiz Progress Saving:

Storing the user's current progress in a quiz, including the current question and answers selected.
Ensure that users can leave a quiz and return later to continue from where they left off (same session)

Implementation Details:

localStorage API:

By using the localStorage API to save and retrieve quiz data on the client side.
Implementing data validation and expiration mechanisms to manage storage efficiently.

Compatibility:

Ensures that the implementation is compatible with major browsers (Chrome, Firefox, Safari, Edge).
Handles edge cases where localStorage is unavailable or disabled.

Benefits:

Enhances user experience by allowing them to pick up where they left off.

Tasks:

[x] #1062

Define the structure for storing quiz progress and user preferences.

[x] Determine the data that needs to be stored for quiz progress (e.g., current question, selected answers, previous scores).
[ ] Define the structure for storing the progress data.
[ ] Choose appropriate keys for storing data in localStorage (e.g., quizProgress).
[ ] Decide on data types (e.g., JSON objects) for the stored information to ensure easy retrieval and manipulation.

Implement methods for saving and retrieving data using localStorage.

[ ] Write a function to save quiz progress to localStorage.
[ ] Write a function to retrieve and restore quiz progress from localStorage when a user returns.
[ ] Handle scenarios where localStorage might be unavailable (e.g., private browsing mode).
[ ] Add data validation and checks to ensure that only valid data is saved and retrieved.

Test across different browsers to ensure consistency.

[ ] Test the feature on major browsers (Chrome, Firefox, Safari, Edge) to ensure localStorage works consistently.
[ ] Verify that the quiz progress are correctly saved and retrieved in each browser (same session).
[ ] Check for any issues related to storage limits or performance.
[ ] Handle potential security concerns, such as data persistence and privacy.

Document the feature and provide instructions for developers and users.

[ ] Write documentation for the new feature, explaining how it works and how developers can integrate it.
[ ] Include code examples for saving and retrieving data using localStorage.
[ ] Provide instructions for users on how to utilize the feature.
[ ] Update any existing documentation to reflect the new feature.
[ ] Add comments in the code to clarify the purpose of key functions and structures.

Please let me know if there are any additional details or considerations that should be included. I’m looking forward to contributing to this project!
### Tasks
- [ ] https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1063
- [ ] https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1064
- [ ] https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1065
- [ ] https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1066
- [ ] https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1067
- [ ] https://github.com/freeCodeCamp/Developer_Quiz_Site/issues/1068

Saving user settings, such as selected themes, preferred programming languages, and any other customizable options.

This is outside of the scope and we are not going to be tracking all of this information.
So we are not going to be implementing this functionality.
As mentioned in the original discussion, the scope of this feature was for users to track their original progress and keep track of their previous scroes.
https://github.com/freeCodeCamp/Developer_Quiz_Site/discussions/256

Please update your issue to remove all functionality dealing with this

Provides a more personalized experience by remembering user settings and preferences.

Hey,
Okay sure,
I updated the issue just now.
Please do check and let me know

I have cleaned up the issue to remove any remaining mentions of the user preferences.
So now this is properly scoped and this will be considered the EPIC.
Moving forward, you will need to break up the work into small issues and track them in this epic.
When you create these smaller issues, you will need to create design mockups for what it will look like when a user revisits the site and wants to resume where they left off so we are on the same page for what this is going to look like.
Then you can start by creating PR's for these issues and getting them reviewed.

Moving forward, you will need to break up the work into small issues and track them in this epic. When you create these smaller issues, you will need to create design mockups for what it will look like when a user revisits the site and wants to resume where they left off so we are on the same page for what this is going to look like.

So, Do I have to use a different and well-defined task list, or does the one I updated earlier work?

If you want to copy over parts of the task list from earlier into these smaller issues, that is fine

If you want to copy over parts of the task list from earlier into these smaller issues, that is fine

Alright,
I will just start working on the same task list as I already mentioned up above.
Thank you!

I am closing all of these issues due to inactivity

I am closing all of these issues due to inactivity

I apologize,
Due to recent tests and examinations,
I was unable to attend to these issues. Can they be reassigned?

@jdwilkin4  I want to work on it , can you assign it to me please

I have assigned you the issue @Epik-Whale463
You will need to spend some time, orienting yourself with the full context of the feature and understand what the scope is for this feature.
You will also need to review the task list issues and what needs to be done there and the PR's you will need to create.

@Epik-Whale463
You will also need to comment on each of the issues related to this feature so you can be assigned to you.
Due to GitHub's access control, I can't assign you to individual issues unless you comment on the issues

@Epik-Whale463
Are you still interested in working on this issue?
If so, you will need to comment on the related issues so you can be assigned.
Otherwise, we will close this issue for now

Yes I am interested in this issue , but I'm kind of confused where to comment?

If im not mistaken, i have commented on all the issues related to the localstorage implementation

It looks like you commented on a couple of the related issues.
You can find the complete list on the issue here


Remove warnings

Thanks for the suggestion! I would love to do this but unfortunately it would break compatibility with Gradle 4.
The Gradle devs are pretty aggressive with their deprecations, it is pretty much unavoidable to have a few of them when trying to support a range of Gradle versions. For now I see making this plugin unusable with Gradle 4 is too high a price for "just" fixing a warning.


Let us work on RIOT master again

And a crude 'fix' to the lack of IPv4 support on small systems.

Fine by me

Merged


docker container is extremely outdated

Hi!
I was wondering if you guys could update https://hub.docker.com/r/scrapinghub/portia/ ?
I plan on writing out a coreOS Unit File, but am experiencing issues with such an old container for issues that I know were fixed earlier this year. :/
Mass deploying over coreOS would be pretty dreamy.

Deploys automatically pushed with each commit to master. Tagged releases will also be available from now on

Hnnnng thank you so much

Sorry it took so long


Fix the Manually Merged form

Regression bug of #19650
Close #20983
Close #21912
The "Manually Merged" form

Mark a PR as Manually Merged and close it

Wait for more time, I am testing.

Codecov Report

Merging #23015 (aec35fb) into main (4fcf3a3) will increase coverage by 47.44%.
The diff coverage is n/a.

:exclamation: Current head aec35fb differs from pull request most recent head 5582faf. Consider uploading reports for the commit 5582faf to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##           main   #23015       +/-   ##
=========================================
+ Coverage      0   47.44%   +47.44%
=========================================
  Files         0     1140     +1140
  Lines         0   150785   +150785
=========================================
+ Hits          0    71542    +71542
- Misses        0    70778    +70778
- Partials      0     8465     +8465

Impacted Files
Coverage Δ

models/issues/content_history.go
66.00% <0.00%> (ø)

models/user/badge.go
100.00% <0.00%> (ø)

routers/web/admin/emails.go
0.00% <0.00%> (ø)

routers/api/v1/repo/branch.go
50.39% <0.00%> (ø)

modules/git/repo_attribute.go
75.98% <0.00%> (ø)

modules/git/repo_language_stats_nogogit.go
56.09% <0.00%> (ø)

modules/doctor/fix16961.go
35.06% <0.00%> (ø)

routers/api/v1/user/watch.go
85.71% <0.00%> (ø)

modules/proxyprotocol/conn.go
0.00% <0.00%> (ø)

routers/api/actions/runner/runner.go
0.00% <0.00%> (ø)

... and 1130 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

LG-TM bot


Added OS requirements for Debian Jessie.

After a brief talk with @pydanny about the intention of this PR, I pull it again.
The intention of this PR is add a new OS requirements set of Debian Jessie for the script install_os_dependencies.sh , extending this way the current two: Ubuntu Trusty and Ubuntu Xenial. Since the merge of #605, a new set of OS requirements for an apt-based distribution can be added through a file called requirements-$(lsb_release -sc).
This new set of requirements are conveniently tested and, of course, I offer myself to keep this set of requirements updated.

LGTM. Don't worry about the failed checks. They are unrelated.

Grats @ddiazpinto on getting this pull request in! 👍


MaErDa Beta Tester Pack Submission to CKAN

Idk why i can't upload my MaErDa.netkan
MaErDa.txt
https://spacedock.info/mod/899/MaErDa Corp Beta Testers Pack

We already tried to add you mod to CKAN but you told us that you didnt want to.
Ref: http://forum.kerbalspaceprogram.com/index.php?/topic/154922-ckan-the-comprehensive-kerbal-archive-network-v1223-mahia/&do=findComment&comment=3072391
PR: #5534

Because i was missclicked in SpaceDock. but now i'm ready i think.

Anyway SpaceDock show my Mod has CKAN rdy anyway

It took how many time before CKAN is updated ?


Add badges, update readme API options

This adds the CI and coverage badges.
I've added the coverage call to the CI, but the token and setup may need some checking here first.

Fixes #47.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@94ef030). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #78   +/-   ##
=========================================
  Coverage          ?   67.69%
=========================================
  Files             ?        3
  Lines             ?      260
  Branches          ?        0
=========================================
  Hits              ?      176
  Misses            ?       84
  Partials          ?        0

Continue to review full report at Codecov.

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


Methods should refer to MINDS

Method ontology, speak with Tom

I close this issue because it's outdated. For the current schemas we do reference to MINDS / openMINDS schemas.


Updating Zoom setup instructions

Zoom has private cloud deployment options, not only "on-prem", adjusting help text for this.
Also, propose opening a ticket to update the help text for Zoom API URL to If you're using a self-hosted private cloud or on-premise Zoom server, enter the Zoom API URL for the Zoom server, for example https://api.yourzoom.com/v2. Leave blank if you're using Zoom's vendor-hosted SaaS service.
Similarly for Zoom URL.
Also, it looks like in Mattermost 5.0 the System Console no longer shows Beta tag for this plugin, so propose removing Beta from docs.
Also, adding upgraded screenshot of integration:

@jasonblais I merged this PR as I want to talk to someone about integrating their voice/video/screenshare solution with ours and I needed a reference, can you figure out how to review this PR to approve it retroactively, or make changes?

I'll talk to @amyblais about reviewing the PR. This is a pretty old PR from last year so the formatting might have been missed then..thanks for fixing those as well.
All plugins are still Beta as noted by the section heading (see attached below). Let me know if this is not obvious to you and we can look at making this more obvious

@it33 I'll talk to @amyblais about reviewing the PR. Looks like this is a doc from last year so I probably missed the formatting..thanks for fixing those.
All plugins are still Beta as noted by the section heading (see attached below). Let me know if this is not obvious to you and we can look at making this more obvious

@it33 Thank you for helping with editing this doc, I agree with the changes. I reviewed this PR and it looks good.
I have opened a Jira ticket to adjust the help text in System Console based on your proposal: MM-11135.
After that ticket is completed, I'll open a PR to add a new screenshot based on your proposal. I'll add to that same PR the text formatting tweaks that you proposed in this Issue.
I have a step in my doc review checklist to go through text formatting guidelines for all doc PRs.


What version of go did you initially build this with?

^^

Disregard, I got it to build and run with go 1.8.3.


High instead of Critical items selected in findings_host.rb

There is an issue in the template file "findings_host.rb".  At line 71 instead of selecting the Critical findings there is a line host.items.high_risks_unique_sorted.each. Therefore the High findings will be shown under the "Critical Findings" header.

Thanks for the report, This will be fixed in 1.7.9.


Remove root level flags used to track cell selection and focus

For #9340
To be merged after #9840
Basically I got rid of the root level state info that keeps track of the cell that's selected and focused.
This required us to keep the two in sync along with the state at the cell level.
When syncing information between multiple editors this got messy. removing this removes the issues where things could go out of sync, i.e. two ways to storing the same thing. Now we get selected information from the cells directly - single source of truth.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ds/custom_editor@666e07d). Click here to learn what that means.
The diff coverage is 7.69%.

@@                Coverage Diff                 @@
##             ds/custom_editor   #9882   +/-   ##
==================================================
  Coverage                    ?   60.6%
==================================================
  Files                       ?     555
  Lines                       ?   29737
  Branches                    ?    4482
==================================================
  Hits                        ?   18021
  Misses                      ?   10714
  Partials                    ?    1002

Impacted Files
Coverage Δ

src/datascience-ui/interactive-common/mainState.ts
53.57% <7.69%> (ø)

Continue to review full report at Codecov.

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


Fix diag related issue

Issue:

https://github.com/Microsoft/vstest/issues/452
https://github.com/Microsoft/vstest/issues/448

Fix:

Catch all exception while creating diag file and exit process peacefully.
Show diag log file to user at the beginning.

@Faizan2304,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Spaced repetition will fail if screen lock or administration password are set in advance

This hasn't been reproduced but based on the code this will occur as passwd requests 3 lines of input not 2 when they already have an amnesia password.
Proposed fix: if passwd fails ask user for their User Account password. They will probably know this as a "screen lock password" if regular user and it is their "Administration password" if they have privileges, there is a script bundled in Tails to tell me which type of user they are to use the appropriate text.
Then passwd can be recalled in an until loop as follows printf "$user_password\n$luks_passphrase\n$luks_passphrase" | passwd

To determine if the current user's account already has a passphrase set, you can check the existence of a passphrase file or directory associated with the user's account. In Linux systems, user account information is typically stored in the /etc/passwd file, and passwords are stored in the /etc/shadow file. However, you should not directly access or modify these files.
Instead, you can use the passwd command with the -S option to check the status of the user's password. Here's how you can do it in a bash script:
#!/bin/bash

# Check if the user's account has a passphrase set
check_passphrase() {
    # Get the username of the current user
    username=$(whoami)

    # Check the status of the user's password using passwd command
    passwd_status=$(passwd -S $username)

    # Extract the second field, which contains the password status
    password_status=$(echo $passwd_status | awk '{print $2}')

    # Check if the password status indicates that a password is set
    if [ "$password_status" == "P" ]; then
        echo "User $username has a password set."
    elif [ "$password_status" == "NP" ]; then
        echo "User $username does not have a password set."
    else
        echo "Unable to determine the password status for user $username."
    fi
}

# Call the function to check the passphrase status
check_passphrase

This script defines a function check_passphrase that checks the status of the current user's password using the passwd -S command. It then extracts the password status from the output and prints a message indicating whether a password is set or not. You can integrate this logic into your script to determine if the user's account already has a passphrase set or not.

Raising priority to high as this interferes with testing some of the features needed because as if installation does not complete it tries to do spaced repetition with the existing passphrase. But if you're rerunning an installation that failed it can't get past this point.


chore(package): explicitly declare js module type

See https://github.com/fastify/fastify-rate-limit/pull/332

Checklist

[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

Should we release a new version? patch ok?


Docs/influx fix

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

This fixes an issue where Influx connections were defaulting to secure, breaking existing installs. It also properly documents the various Traffic Ops configuration files (to the best of my knowledge) to make their behavior explicit and clear.
Which Traffic Control components are affected by this PR?

Documentation
Traffic Ops

What is the best way to verify this PR?
Mostly just build and read documentation. Also build an ATC environment with minimally Traffic Ops and Traffic Stats - specifically Traffic Stats's InfluxDB clusters should not be configured to use SSL -, request deliveryservice_stats (properly) and observe that it works
The following criteria are ALL met by this PR

[x] Tests are unnecessary
[x] This PR includes documentation
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4658/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4661/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4663/


Copy-paste of large body of code can't be interrupted

Repro

copy large body of a Powershell script (~100 lines).
paste into the console and watch it paint itself
try to cancel with Ctrl+C - nothing happens, the text would still slowly appear on the screen

And pasting takes significant amount of time. You can see chars appearing one by one as if there is 100ms delay between each character render.

Closing in favor of #579 which covers all of the known issues with right-click paste.


Bloodhound fails to import groups.json

Using the Bloodhound 4.0.2 I get the following error when importing groups.json:
"File created from incompatible collector"
users.json, computers.json and domains.json imports fine. Anyone knows what might be the problem?

How did you resolve this?

I've been getting this consistently with the ingestor lately for groups.json

I don't see any issue with the json though on my end.

@dsolstad ....I am having the same error while importing users.json, computers.json and domains.json , File created from incompatible collector     erro. i am using Bloodhound 4.1.0-latest version
Can you or anyone else plz tell how didi you resolve this??

@DOP7 Look at the JSON and make sure it's well formatted.

@dsolstad .. actually there is a issue with the current release of Blood hound....i uploaded the json files with bh earlier version and it was loaded successfully


[BUG]: No pause checking

There's no checking if the user is paused. When the modmail manager pauses the ticket, it pauses it only for the manager, not the user.

Thanks for reporting this issue, i'll quickly look into it

I've added the missing check. It was also missing for blocked users so I added it too.


Add a Sample to Ballerina By Example, to demonstrate enum usage

Description:
Currently, we cannot clearly identify a sample for enum usage at  Ballerina by Example.

Duplicated issue
[1] https://github.com/ballerinalang/ballerina/issues/4016


Docker machine create on Softlayer stuck in "Configuring host OS"

docker-machine create slm0 --driver=softlayer --softlayer-user=sl_user_id --softlayer-api-key=sl_api_key --softlayer-cpu=1 --softlayer-disk-size=0 --softlayer-domain=sl_domain --softlayer-hostname=m0 --softlayer-image=UBUNTU_LATEST --softlayer-hourly-billing --softlayer-memory=1024 --softlayer-region=dal09
←[34mINFO←[0m[0000] Creating SSH key...
←[34mINFO←[0m[0004] Getting Host IP
←[34mINFO←[0m[0022] Waiting for host to become available
←[34mINFO←[0m[0208] Configuring host OS
the commands waits here indefinitely. The softlayer instance gets provisioned with the settings, however there is docker provisioning does not happen. I have checked both running this from local and from another softlayer instance both yielding similar results.

@gauthamacharya - is this with a recently-compiled docker-machine, or is with the 0.1.0 release? There was a similar issue (#725) fixed a little while ago.

@hairyhenderson - I'm able to consistently reproduce this issue with the 0.1.0 release but it doesn't happen after compiling 0.2.0. 0.2.0 doesn't include the "Configuring host OS step" in its output but displays "Waiting for host setup transactions to complete". This step completes quickly and finishes creation as expected.

Confirming that the issue is closed with 0.2.0 builds ! The issues was consistent with 0.1.0 version

where to download 0.2.0 version ? or I have to build it myself ?

@wzymaster - 0.2.0 isn't quite out yet, but you can get a release candidate here: https://github.com/docker/machine/releases

thx very much and I will give it a try.

Sorry, I just download the pre-release of 0.2.0 and run docker-machine like below :
./docker-machine create --driver softlayer      --softlayer-user leow      --softlayer-api-key 0a7e316ab1004f761672ab1a890f906cb3a90ba978c5c217c2ca500fd42b820d       --softlayer-cpu 4      --softlayer-memory 8192      --softlayer-disk-size  0      --softlayer-domain imdemocloud.com      --softlayer-hostname "leo-demo1"       --softlayer-image CENTOS_LATEST_64      --softlayer-region hkg02      leo-demo1
INFO[0000] Creating SSH key...
INFO[0007] Getting Host IP
INFO[0032] Waiting for host to become available
INFO[0154] Waiting for host setup transactions to complete
Unfortunatley, it still get stuck at 'Waiting for host setup transaction to complete" step .
However, I do see this VM machine provisioned on SoftLayer and I can log in.  It looks like a clean machine without docker.
Do I run something wrong here ? @hairyhenderson
The below is the build of docker-machine which I am using :
Zhongs-MacBook-Pro:Leo_Demo wzymaster$ ./docker-machine -v
docker-machine version 0.2.0 (45f08d9)

btw, I am running docker-machine on the macBook with OS X v10.9.5

@wzymaster - can you run docker-machine with debug enabled?
Something like:
./docker-machine --debug create --driver softlayer --softlayer-user leow --softlayer-api-key xxx --softlayer-cpu 4 --softlayer-memory 8192 --softlayer-disk-size 0 --softlayer-domain imdemocloud.com --softlayer-hostname "leo-demo1" --softlayer-image CENTOS_LATEST_64 --softlayer-region hkg02 leo-demo1

@wzymaster also, you're probably going to want to change your API key, because you pasted it in your earlier comment :)

@wzymaster also - would you mind logging a new issue for what you're seeing? This is different than what @gauthamacharya originally logged, and this issue is closed...

Thanks for the reminder of my API key and just updated.


Does it work with other LLM models: Claude/anthropic

Hi there,
I found your llmgraph looks interesting, I would like to use it, just wondering if it would work with other LLM models beside current defauts one.
I am trying to test it using CLAUDE google LLM model.
Thank you

You can specify a different LLM provider, including running with a local https://github.com/jmorganca/ollama model.
You should be able to specify (via the --llm-model argument, see docs here) anything supported by LiteLLM as described here: https://docs.litellm.ai/docs/providers
You'll find Claude models under the Anthropic provider: https://docs.litellm.ai/docs/providers/anthropic
Note that the prompts to extract related entities were tested with OpenAI and may not work as well with other models.
Thanks for your interest in the project.


Only log failing the WebSocket connection when code is not 1006

Fixes #167

Again, could you paste/gist a full backtrace please? That would be helpful to evaluate your PR.
Thanks

It's fairly simple, no exception raised, just some logging:
2017-03-29 11:33:57,429 INFO websockets.protocol.fail_connection:613   Failing the WebSocket connection: 1006

I think we should downgrade some logging calls from INFO to DEBUG.
@RemiCardona I think that was one of your early concerns with this library. If you'd like to review the logging, make it more consistent and improve levels, be my guest ;-)


Combining CPL and action chunking does work! Would you like to slightly modify the codebase to an action chunk version?

Hi Joey,
I combine CPL and action chunking, and it does improve the success rate. Would you like to modify your codebase and perform one more experiment? It will save a lot of GPU memory from the original CPL method! Modifying the codebase will not take more than an hour!
I list some of my results here:
The success rate of CPL when the chunk size is 10:

The curve before 38k steps means the behavior cloning phase.  Every 40 epochs, I run 192 evaluations. The top success rate of the behavior cloning phase is 68%, and the top success rate of the CPL phase is 95% (with bias=0.125).
And this is the CPL binary classification accuracy:

I plan to test the idea in other environments. It will be great if you can verify it with the original CPL experiment setting. If we do find something, would you like to be a co-author of a short research paper?
Best,
Zhuoheng

Btw I explain the experiment details in an email to you


Release notes Nov - Dec

Working draft

@meagancojocar did you still want to land this? if not should we close?


Render state: Texture unit binding should invalidate texture binding

@bcamper found out a bug in the tracking of texture unit/id binding within tangram-js that could possibly happen in -es.
We currently don't use multiple texture unit on default shaders but this could happen on user defined uniforms textures. A fix would be, every time a new texture unit is bound the current state tracking of the bound texture should be invalidated.

FYI, fixed in JS here: https://github.com/tangrams/tangram/commit/0cee05f76de2d0c630d209c7ea1d02068f9d46e4

Closed #777 via #784


Uec simple

A version of uec powered libvirt that doesn't (directly) require nbd
There are a couple small fixes to stack.sh:

sleep between screen creation and running command
ensure devstack is owned by correct user when running as root
only apt-get update if sudo needs installed

In the future I want to make it so we launch the uec image to install dependencies and then stop it and save it as "primed" (to speed up runs afterward) - but this is good for now.
A run on my server takes 5.5 min instead of 3.5 minutes (for the build_libvirt.sh)

lgtm now - thx for the fixes!


Uber Pickup Analysis

ML-Crate Repository (Proposing new issue)
:red_circle: Project Title : Uber Pickup Analysis
:red_circle: Aim : To analyze the data of the customer rides and visualize the data to find insights that can help improve business. Data analysis and visualization is an important part of data science. They are used to gather insights from the data and with visualization you can get quick information from the data.
:red_circle: Dataset : https://www.kaggle.com/fivethirtyeight/uber-pickups-in-new-york-city
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.
Hello, ML-Crate contributors, this issue is only for the contribution purposes and allocated only to the participants of SWOC 2.0 Open Source Program.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.
A Demo Project has been created to guide you about the whole structure of presenting the Project in this repository. Here's the link of the Demo Project - https://github.com/abhisheks008/ML-Crate/tree/main/Project-Demo-Folder

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'SWOC' contributors of 'ML-Crate' project.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Participant ID :
Approach for this Project :
Are you a participant of SWOC 2.0?

[ ] YES
[ ]  No

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Full name : Deepthi M
GitHub Profile Link : https://github.com/deepthi1107
Participant ID : 294
Approach for this Project: EDA, SVM, random forest, decision tree
Are you a participant of SWOC 2.0?
YES

Issue assigned to you @deepthi1107

Full name : Venkatakrishnan R
GitHub Profile Link :https://github.com/Cody-coder017
Participant ID : 1104
Approach for this Project : Using time series models to predict the pickups
I would like to work on this issue

@Cody-coder017 first complete the issue of Jeopardy bot then only you'll be assigned to any new issues.

Issue assigned to you @Cody-coder017

Hello @Cody-coder017 , what's the update here?

At a good stage proceeding, encountered some issues with my desktop ,so just holding up the works,trying to do asap

Sure, I want to be in the waiting list, if suppose in case, due some issue, if assigned one is not able to complete
Then I'll  take it from here
Full name: Shrikrushna Bhagwat
GitHub Profile Link: https://github.com/krishna-NIT
Participant ID: 930
Approach for this Project: EDA, SVM, random forest, decision tree, time series module for predictions
Are you a participant of SWOC 2.0?
YES

@abhisheks008  there is a steady progress EDA is done working on time series and readme file
since my end semester is on i could not work at expected but im trying my best

@Cody-coder017 Don't worry, the delay is accepted, We will be waiting bro, take your time, let me know in case if you need any help, I'll be happy to learn from you too!

@krishna-NIT  sure

@Cody-coder017 what's the update here?

need 2 more days , i will complete it

Okay. @Cody-coder017


stop reading local metadata file

Problem
The local metadata file is effectively unused if remote storage is configured, because remote storage is the authority.
We do write and read it, but, we never use its contents.
In particular, during startup, we write each metadata file with the contents fetched from remote storage during preload.
Effectively, that's a VirtualFile::crashsafe_overwrite for each timeline.
Further, it turns out that we can delete a bunch of legacy code if we stop caring about the metadata file.
Refs:

noticed this in #6765
... which is part of adding tokio-epoll-uring to the write path (#6663)

Solution
This PR removes any code that reads the file.
We continue to write (and delete it) as before.
Once we've released this version and are sure we won't roll back, we will remove the code that writes it.
That'll happen in https://github.com/neondatabase/neon/pull/6769.
The price is that it's now required to run with remote storage anymore.
We agree that we want to get there eventually, so this is the first step.
see https://github.com/neondatabase/neon/issues/4099
Review
Reviewers should not just check the changes in this PR (which are code deletions, mostly).
But also they should ensure that after this change, absence of the file does not change any outcomes.
This means auditing the code base for, example,

reads of the file
stat of the file
removal of the file where NotFound isn't treated as if it is success
etc

Reviewers should not just check the changes in this PR (which are code deletions, mostly).
But also they should ensure that after this change, absence of the file does not change any outcomes.

I did some research:

I grepped for TimelineMetadata::from_bytes and outside of tests and the code removed in this PR I only found usage in one location: the pageserver ctl tool still has a command to read the metadata file from disk (see handle_metadata function), but I think this can be a followup (but also ok with having it in this PR).
Looking at the callers of PageServerConf::metadata_path, there is still an assert in calculate_logical_size for the file to exist. cleanup_remaining_timeline_fs_traces already doesn't error if the file doesn't exist. This should probably be addressed ideally in this PR, but at least in the deploy before #6769 is deployed.
Other than that I didn't find anything.

I grepped for TimelineMetadata::from_bytes and outside of tests and the code removed in this PR I only found usage in one location: the pageserver ctl tool still has a command to read the metadata file from disk (see handle_metadata function), but I think this can be a followup (but also ok with having it in this PR).

how is the metadata stored within index_part.json then deserialized?

the pageserver ctl tool still has a command to read the metadata file from disk (see handle_metadata function), but I think this can be a followup (but also ok with having it in this PR).

Good find, will put that into a follow-up.

Looking at the callers of PageServerConf::metadata_path, there is still an assert in calculate_logical_size for the file to exist.

Isn't that in a failpoint? I'd have to remove the test that uses it as as well, then.
I think it actually makes sense to keep it until #6769 as it's concerned with removal of the file / the write path.

cleanup_remaining_timeline_fs_traces already doesn't error if the file doesn't exist.
This should probably be addressed ideally in this PR, but at least in the deploy before https://github.com/neondatabase/neon/pull/6769 is deployed.

I'm confused. Assuming I'm right wrt the faillpoint in the previous quote, do you think there's more work that needs to happen or can we merge this PR as is.


How do I add a Calendar Set?

I'm using version 0.0.17 and I'm not seeing a command or an option to create a Calendar Set. I'm only seeing options for:

Daily Notes
Weekly Notes
Monthly Notes
Quarterly Notes
Yearly Notes

How do I add a new Calendar Set?

Hi, I just updated the README file for the repo to better reflect that Calendar Sets are currently only available in the beta version of Periodic Notes. More information here: https://github.com/liamcain/obsidian-periodic-notes/issues/106#issuecomment-1100474794

Thanks for taking the time to clarify this. I'll wait for this version to come out of beta, then! :)

I was wondering the same. Glad I finally found this.

It seems that this functionality is still in beta.  How does one go about using the latest beta release?

I would also like to try calendar sets. Is there a way to install the beta?

Hi, I just updated the README file for the repo to better reflect that Calendar Sets are currently only available in the beta version of Periodic Notes. More information here: #106 (comment)

I understand that, however why does the production (not beta) README file has to be updated and published before the beta final release?

Hey y'all. It's now Feb 2023.
It's been 10 months since the README was updated to state:
This README has been preemptively updated in anticipation for v1.0. Not all features described here are available in the current stable release 0.0.17.

It's been 7 months since the repo was updated at all.
What gives?

Pretty easy to install the beta if folks want beta features.

Is this still an actively developed beta for this plug-in?


Raft Metrics

This PR

Creates a global struct to maintain Raft metrics and ensures the struct is initialized lazily with a Mutex wrapper
The RaftMetrics struct uses String instead of &str to avoid any issues with lifetimes that might crop up. This will probably require regular export of the metrics to avoid memory issues.
Creates helper methods to modify the metrics
Uses helper methods when internal requests are being made to track various properties of the cluster
Adds utilities like TimedFuture and CounterGuard to help keep track of metrics

@redixhumayun Add a server endpoint to return the metrics in JSON for the UI


[RUNTIME][VULKAN] Seg fault in WorkspacePool's destructor

Problem: When VulkanThreadEntry is destructed, its member "streams_" is destructed before  "pool" (members are destructed in reverse order of their appearance in class definition).  pool's destructor calls FreeDataSpace(), which calls StreamSync(), which references "streams_", which is already destructed. This caused seg fault.
Reproduce:
Run apps/benchmark/gpu_imagenet_bench.py using Vulkan target.
Proposed Solution:
Since pool's destructor references stream_, but not vice versa, stream_ should be destructed after pool. We can wrap pool with a unique_ptr, so we can destruct it inside ~VulkanThreadEntry(), which is called before VulkanThreadEntry::stream_'s destructor. Also, we destruct pool before streams_.clear(), so we can reuse the existing stream instead of creating a new one.
More background:
This is a regression introduced by the fix for https://github.com/apache/incubator-tvm/issues/5388. In the fix, FreeDataSpace() calls StreamSync() to make sure the buffer being freed is not longer in use.

The proposal sounds good, feel free to send a PR.


Prepare 1.1.0 release

Please review this, make sure the links are correct and everything is included.
Once this is merged, the v1.1.0 tag will be created from the merge commit and the release/1.1 branch will be created from there

Codecov Report

Merging #2302 into master will increase coverage by 4.53%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2302      +/-   ##
==========================================
+ Coverage   45.54%   50.07%   +4.53%
==========================================
  Files          83       75       -8
  Lines        9185     7269    -1916
==========================================
- Hits         4183     3640     -543
+ Misses       4326     2956    -1370
+ Partials      676      673       -3

Flag
Coverage Δ

#linux
50.07% <ø> (ø)
:arrow_up:

#windows
?

Impacted Files
Coverage Δ

remotes/docker/auth.go
63.82% <0%> (-3.97%)
:arrow_down:

remotes/docker/status.go
21.42% <0%> (-3.58%)
:arrow_down:

platforms/cpuinfo.go
4.65% <0%> (-1.01%)
:arrow_down:

log/context.go
36.84% <0%> (-0.66%)
:arrow_down:

namespaces/context.go
55% <0%> (-0.56%)
:arrow_down:

errdefs/grpc.go
74.6% <0%> (-0.4%)
:arrow_down:

labels/validate.go
100% <0%> (ø)
:arrow_up:

server/config.go
0% <0%> (ø)
:arrow_up:

identifiers/validate.go
100% <0%> (ø)
:arrow_up:

images/oci/exporter.go
0% <0%> (ø)
:arrow_up:

... and 53 more

Continue to review full report at Codecov.

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

LGTM

CRI 1.0 merged, links updated, and rebased