Sync

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@mariusvniekerk not sure if this is important for ppc64le and aarch64 but I'm merging this now to help with qt, we can fix those archs later.


No such file or directory "quill.snow.css.map"

Hello,
Error: ENOENT: no such file or directory, open '../node_modules/react-quill-new/dist/quill.snow.css.map'
When building or running on prod, I guess there will be a problem here. Is there a solution?

@NonthapatKim This should not be a problem - it will only affect the developer tools.
If you want to remove the warning, import quill.snow.css from ../node_modules/react-quill-new/node_modules/quill/dist/quill.snow.css instead

Thank !


This page is (cannot think of a non-rude word but basically useless

So I am following the MS Lesson for creating snippets
I create the snippet as they outlined.
Then I attempt to import.  Go to the folder with the file and select it and I get file(S) I only select one file - are not valid.
At this point I have no idea and all I did was use the code provided in the "Lesson".
Does anyone even test these lessons???

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

ID: 77c7c36c-84d9-9237-3d78-689e76d080f9
Version Independent ID: 6584735a-e8db-429d-5427-687fc709e6f8
Content: Walkthrough: Create a code snippet in Visual Studio - Visual Studio (Windows)
Content Source: docs/ide/walkthrough-creating-a-code-snippet.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@sqlguy6212 -- Lloyd, thank you for your feedback.  Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

Please let me know if you are than able to complete this lesson.


Add canterbury.ac.nz domains to allowed hosts in local settings

This will allow developers working within the UC network to access django instances running on other developer's machines, via the .canterbury.ac.nz.

A test failed, but was due to Travis network error. Have restarted it.
Will review tomorrow when on site.


OSX glslangValidator

from the atom lint setup page:
" there is now official MacOS release has made consistent testing across the three main platforms hard. Currently glslangValidator master is used for the Travis build which targets MacOS. If you're developing on a Mac and want to run the unit tests, see .travis.yml for an example of how to get this."
I cant seem to find out what this means, been searching around, do you have a direct link?  is there anything currently available for glslangValidator on OSX?  I cant seem to find it.

Oops that's a typo. It should read "no official MacOS release". I double checked and there are still only binaries for Windows and Linux here: https://cvs.khronos.org/svn/repos/ogl/trunk/ecosystem/public/sdk/tools/glslang/Install/
I updated the Readme to remove the bits about MacOS testing and Travis. @Arcanemagus removed the MacOS CI build as it just caused problems because of the lack of an official MacOS glslangValidator release.

That and builds were taking ~2 hours because Travis CI had starved their open source OS X infrastructure of build machines 😛.

There now seems to be builds for osx available https://github.com/KhronosGroup/glslang/releases

anything on brew?

@nkint http://formulae.brew.sh/formula/glslang has 6.2.2596 which is the latest release number in the link @submersibletoaster provided.
That entire section of the README can go.

FYI I released glslang-validator-prebuilt, an npm package to install latest built binary to node_modules`.
https://www.npmjs.com/package/glslang-validator-prebuilt


Add simple status reporter, connecting to wifi and sending an email with latest containers' metadata

Replaced by https://github.com/WitnessAngel/witness-angel-nvr/issues/3


Make firestackModule.js working for v3

this._firestack.database.ref(...refName) (line 39) should be this._firestack.database().ref(...refName)

Thanks for the patch.


IP-Access List Not Working With <IP_ADDRESS>/24

Checklist

[x] Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
[x] Are you sure you're not using someone else's docker image?
[x] If having problems with Lets Encrypt, have you made absolutely sure your site is accessible from outside of your network?

Describe the bug
I am trying to use <IP_ADDRESS>/24 in IP Access List because my local network is configured with that IP and I want to allow some proxy sites only from my local network
But nginx throwing some error
2020/12/07 12:51:21 [warn] 193#193: low address bits of <IP_ADDRESS>/24 are meaningless in /data/nginx/proxy_host/4.conf:47

To Reproduce

Create an Access List With the IP <IP_ADDRESS>/24

Expected behavior
IP Allow list should work
Screenshots

Operating System
Ubuntu

your subnet is <IP_ADDRESS>/24 and not <IP_ADDRESS>/24 if yout configure your subnet right in the access list it will work.


[Event Change Request] codeunit 415 "Release Purchase Document" - OnAfterReleasePurchaseDoc

🕮 Describe the request
Could you please add the parameter SkipWhseRequestOperations  to the event OnAfterReleasePurchaseDoc?
local procedure "Code"() LinesWereModified: Boolean
var
    PurchLine: Record "Purchase Line";
    PrepaymentMgt: Codeunit "Prepayment Mgt.";
    NotOnlyDropShipment: Boolean;
    PostingDate: Date;
    PrintPostedDocuments: Boolean;
    IsHandled: Boolean;
begin
    with PurchaseHeader do begin
        [...]

        if PrepaymentMgt.TestPurchasePrepayment(PurchaseHeader) and ("Document Type" = "Document Type"::Order) then begin
            Status := Status::"Pending Prepayment";
            Modify(true);
            //+Change
            //-OnAfterReleasePurchaseDoc(PurchaseHeader, PreviewMode, LinesWereModified);
            OnAfterReleasePurchaseDoc(PurchaseHeader, PreviewMode, LinesWereModified, SkipWhseRequestOperations);
            //-Change
            exit;
        end;
        Status := Status::Released;

        OnCodeOnBeforeCalcAndUpdateVATOnLines(PurchaseHeader);
        LinesWereModified := LinesWereModified or CalcAndUpdateVATOnLines(PurchaseHeader, PurchLine);

        OnCodeOnBeforeModifyHeader(PurchaseHeader, PurchLine, PreviewMode, LinesWereModified);

        Modify(true);

        if NotOnlyDropShipment then
            if "Document Type" in ["Document Type"::Order, "Document Type"::"Return Order"] then
                if not SkipWhseRequestOperations then
                    WhsePurchRelease.Release(PurchaseHeader);

        //+Change
        OnAfterReleasePurchaseDoc(PurchaseHeader, PreviewMode, LinesWereModified);
        OnAfterReleasePurchaseDoc(PurchaseHeader, PreviewMode, LinesWereModified, SkipWhseRequestOperations);
        //-Change
    end;
end;

[IntegrationEvent(false, false)]
//+Change
//-local procedure OnAfterReleasePurchaseDoc(var PurchaseHeader: Record "Purchase Header"; PreviewMode: Boolean; var LinesWereModified: Boolean)
local procedure OnAfterReleasePurchaseDoc(var PurchaseHeader: Record "Purchase Header"; PreviewMode: Boolean; var LinesWereModified: Boolean; SkipWhseRequestOperations: Boolean)
//-Change
begin
end;

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


[BUG] Ollama answer in the chat cannot be parsed

Describe the bug
I am using Ollama latest and tinyllama.
with a very simple chatflow in Flowise:

and this is the result in the test chat:

But when I enter in cmd with Ollama and ask the same question everything is super fine:

To Reproduce
Steps to reproduce the behavior:
Setup the flow and try to chat
Expected behavior
To have as minimum the same recipe for scrambled eggs :) Same answer as cmd one.
Screenshots
Applied above
Flow
Applied above
Setup

Docker setup
Flowise Version 1.6.0
OS: Linux Leap 15.4
Browser Chrome
Ollama version 0.1.29

Additional context
Add any other context about the problem here.

Click additional parameters and remove the system message and prompt etc. only put the user input and response nothing else.
Does it fix it?

running from command line vs using conversational agent is different. Conversational agent has built-in prompt that ask the LLM to give data in JSON format, and its tailored to ChatGPT. For other models, try using LLMChain

Thank you, now it is ok!


Unzipping tar.gz flattens files to have no more than 1 directory

I have a nested tar.gz file. I can extract these files using cygwin/tar no problem.
In SharpZipLib I'm doing the following:
{
	// fileGitBlog is a FileInfo object pointing to tar.gz file
	using (Stream sourceStream = new GZipInputStream(fileGitBlob.OpenRead()))
	{
		using (TarArchive tarArchive = TarArchive.CreateInputTarArchive(sourceStream))
		{
			tarArchive.ExtractContents(@"c:\temp\somewhere\");
		}
	}
}

Expected behavior
I expect to see the same results as cygwin, which is a directory structure populated with files
    SomeDir1\
    SomeDir1\SomeDir1a\
    SomeDir1\SomeDir1a\someFiles.txt
    SomeDir2\
    SomeDir2\SomeDir2a\someFiles.txt

Actual behavior
    SomeDir1\
    SomeDir1a\someFiles.txt
    SomeDir2\
    SomeDir2a\someFiles.txt

Version of SharpZipLib
SharpZipLib 0.86.0
Obtained from (place an x between the brackets for all that apply)

Package installed using:

NuGet

This seems odd. Nested directories should work. Could you provide a sample file?

Closing this as unable to reproduce. Could be caused by #38/#338.


Multipart uploads cause an infinite loop if passed sliced data

Describe the bug
Data is a self-slicing type, so you can provide someData[0..<size] to an UploadPart. However, if you do that, StreamableHttpBody gets into a state where it loops forever trying to read 0 bytes.
Expected Behavior
I would expect self-sliced data to be supported, since that prevents the need for spurious copies during uploading.
Current Behavior
I tracked it down to this code in StreamableHttpBody:
logger.debug("read \(toRead) bytes from data")
if toRead == 0 && data.endIndex == position {
    return nil
}

In this case, data.endIndex refers to the end index of the full data object, not just the slice. So this check never succeeds, and we continue reading 0 bytes indefinitely.
Reproduction Steps
This is the code for an actor that uploads a file in chunks by slicing data.
public actor FileUploadService {
    public static let shared = FileUploadService()
    private let s3: S3Client

    public init() {
        let config = try! S3Client.S3ClientConfiguration(region: "us-east-1")
        self.s3 = S3Client(config: config)
    }

    public func uploadFile(at url: URL) async throws {
        let id = UUID()
        let input = CreateMultipartUploadInput(acl: .publicRead, bucket: "some-bucket", key: "\(id.uuidString.lowercased())/\(url.lastPathComponent)")
        let upload = try await s3.createMultipartUpload(input: input)
        let data = try Data(contentsOf: url, options: .alwaysMapped)

        var completedParts = [S3ClientTypes.CompletedPart]()

        // Iterate over 1MB chunks
        let chunks = data.chunks(ofSize: 1_000_000)
        for (offset, part) in chunks.enumerated() {
            let partNumber = offset + 1
            Logger.upload.log("Uploading part \(partNumber) of \(chunks.count)...")
            do {
                let partInput = UploadPartInput(body: .data(part), bucket: upload.bucket, contentLength: part.count, key: upload.key, partNumber: partNumber, uploadId: upload.uploadId)
                let partResult = try await s3.uploadPart(input: partInput)
                let completedPart = S3ClientTypes.CompletedPart(eTag: partResult.eTag, partNumber: partNumber)
                completedParts.append(completedPart)
                Logger.upload.log("Finished uploading part \(partNumber) of \(chunks.count)...")
            } catch {
                Logger.upload.error("upload part failed: \(error)")
                let abort = AbortMultipartUploadInput(bucket: upload.bucket, key: upload.key, uploadId: upload.uploadId)
                let _ = try await s3.abortMultipartUpload(input: abort)
                break
            }
        }
        let parts = S3ClientTypes.CompletedMultipartUpload(parts: completedParts)
        let completeInput = CompleteMultipartUploadInput(bucket: upload.bucket, key: upload.key, multipartUpload: parts, uploadId: upload.uploadId)
        let completedUpload = try await s3.completeMultipartUpload(input: completeInput)
        print(completedUpload.location!)
    }
}

// MARK: Helper for chunking data into slices

/// Iterates over fixed-sized consecutive subsequences of a given
/// collection. The final chunk may be smaller than the provided chunk size.
///
/// - Example:
///
/// ```
/// (0..<100).chunks(ofSize: 17)
///
/// // 0..<17
/// // 17..<34
/// // 34..<51
/// // 51..<68
/// // 68..<85
/// // 85..<100
/// ```
///
/// If the collection being chunked is empty, the chunks collection will
/// also be empty.
public struct Chunks<Data: Collection>: RandomAccessCollection {
    public typealias Element = Data.SubSequence

    var chunkSize: Int
    var collection: Data

    public init(_ collection: Data, chunkSize: Int) {
        precondition(chunkSize > 0)
        self.chunkSize = chunkSize
        self.collection = collection
    }

    public subscript(index: Int) -> Element {
        let subrangeStart = collection.index(
            collection.startIndex,
            offsetBy: index * chunkSize,
            limitedBy: collection.endIndex
        )!
        let subrangeEnd =
            collection.index(
                subrangeStart,
                offsetBy: chunkSize,
                limitedBy: collection.endIndex
            ) ?? collection.endIndex
        return collection[subrangeStart..<subrangeEnd]
    }

    public var startIndex: Int { 0 }
    public var endIndex: Int {
        let ratio = Double(collection.count) / Double(chunkSize)
        return Int(ratio.rounded(.up))
    }
    public func index(after i: Int) -> Int { i + 1 }
}

extension Collection {
    public func chunks(ofSize chunkSize: Int) -> Chunks<Self> {
        Chunks(self, chunkSize: chunkSize)
    }
}

You can trigger the bug by uploading anything over 1MB.
Possible Solution
I think by checking data.count instead of data.endIndex, this will be fully resolved.
Additional Information/Context
No response
AWS SWIFT SDK version used
1.0.40
Compiler and Version used
Xcode 16.1 (16B40), Apple Swift version 6.0.2 (swiftlang-<IP_ADDRESS>.2 clang-16<IP_ADDRESS>), swift-tools-version: 6.0
Operating System and version
macOS 15.0 (24A8332)

@harlanhaskins Thank you for the report, and especially for the detailed reproduction instructions.  I didn't get the exact failure as you - I got a fatalError for trying to create an invalid range instead of a hung transfer - but there is definitely a problem when the startIndex isn't zero.
We'll get a fix together soon and update here once it ships.

@harlanhaskins A fix for this issue shipped in AWS SDK for Swift 1.0.43 today.
Please update, retest, and if your issue is resolved, please close this ticket.

Confirmed; I removed my workaround and this now behaves as expected with 1.0.43. Thank you!


Is this project not maintained anymore?

have the same question?

Please see https://github.com/GoogleChrome/workbox/issues/3149#issuecomment-1954664700.

Closing in favor of https://github.com/GoogleChrome/workbox/issues/3149#issuecomment-2019983986.


Renamed test cases and changed vsphere test cases to use existing libraries

@bmdepesa @luthermonson Can you review the changes?

LGTM


Initialize cluster via connection URL

Hi, and thanks for the great library!
Wanted to mention that it would be nice to be able to initialize the ClickhouseCluster class via a URL like<EMAIL_ADDRESS>The clickhouse_driver library that you use actually supports it via a Client.from_url static method.
Unfortunately, clickhouse_driver does not yet expose the "parse url" API, and I need to parse the URL manually to pass it to ClickhouseCluster.

Hey,
Thank you for the idea.
I made the requested changes, https://github.com/zifter/clickhouse-migrations/pull/25
You can find the necessary changes in the latest release 0.7.0

#26 is merged
@selevit can you check the latest release 0.7.1?

Big thanks for the quick fix!
It worked for me. Just a minor – IMO it looks a bit weird that it's required to pass the db name explicitly as None, like this.
cluster.migrate(
    db_name=None,  # use default db
    migration_path=MIGRATIONS_DIR,
    ...
)

@selevit yes, looks weird, but I don't want to make breaking changes in API


Too many open files

After running for some time, extension stops refreshing result, with error in log like this:
(gnome-shell:8393): Gjs-WARNING **: JS ERROR: GLib.Error g-unix-error-quark: Too many open files
Restarting Gnome helps for a while, until that error shows again.
I assume it is due ulimit -n limitation and non closed file descriptors, as I see from /proc/8393/fd.

What command did you run and for how long? I've had this running for some time and not seen it crash like this, but I've made some changes there will get merged into stable once I get scrolling working. Feel free to clone unstable if you want to test.
I'll close this if you can confirm the problem is fixed.

First, I tried simple cat command, then just echo something for the test, result was the same. I set 1 second refresh though, for faster result.
I`ll try your unstable branch later tomorrow, I must go now...

Unstable branch doesn't refresh at all for me, it executes command one time. Also I don't need scrolling, I hope you'll make it as option ;)

would you please add the option to make the command a one-off? either with a checkbox, or, perhaps functionally better, display "once" in the refresh interval (internally, that would be zero)

would you please add the option to make the command a one-off? either with a checkbox, or, perhaps functionally better, display "once" in the refresh interval (internally, that would be zero)
never mind, on this system, it doesn't work at all, it just presents the configuration screen once clicked up, changing to a custom command is fine, but it never gets executed.
uname -a:
Linux Shodan <IP_ADDRESS>-1-MANJARO #1 SMP PREEMPT Wed Mar 11 20:18:29 UTC 2015 x86_64 GNU/Linux
gnome-shell --version:
GNOME Shell 3.14.3

Same happens to me. Looks like this is related to GLib.spawn_async_with_pipes
https://github.com/elken/gnome-shell-extensions-command-output/blob/stable/extension.js#L48
see
https://gist.github.com/buzztaiki/1487781#gistcomment-1214635


Update mtd.md to include the fast track benefits

@JohanFreelancer9 : made the changes. pls review.


Genecode/Ensembl name space and release version

From Brian Aevermann:
Also we should be tracking the annotations spaces used within the onotology. is that part of the imported ontology make file?
Gencode and Ensembl mostly merged a few years back, but gencode maintains a larger annotation system of hypothetical and putative genes. The most important thing is to maintain a record somewhere in the ontology the actual annotation system used during the data analysis steps as this will be an important distinction when interpreting the results in the future.

Also we should be tracking the annotations spaces used within the onotology. is that part of the imported ontology make file?

I think our import file automatically updates right?

The most important thing is to maintain a record somewhere in the ontology the actual annotation system used during the data analysis steps as this will be an important distinction when interpreting the results in the future.

Not sure how to handle this, but we should have a record of this

@jeremymiller  - can we get details of ENsembl version for the mouse reference we're using?

version metadata should live in taxonomy nodes from #191


Area within historic district

I will start working on this using geopandas to do aggregation

Description

Using data from open data compute the area of each PUMA, Borough, and City that is in a historic district.
Split historic district boundaries by PUMA and Borough boundaries the sum area within geographic area
Calculate percent by dividing the area in a historic district by the total area of the geography

Will import the data from data library as a .csv as this is easiest to load into geopandas

I adopted bash code from the bash/config.sh in the db-cpdb repo to load in .env files. My implementation lives in /ingest/data_library and differs in two ways:

The cpdb code allows for multiple .env files to be loaded as it uses a .env and version.env. EDDT only has one .env currently so it's config.sh can only read one.
The cpdb code has this sed 's/#.*//g' pipe that as far as I can tell is there to clean out comments. My .env files don't have comments and I don't know why they would so I left this out. If I'm misunderstanding the role of this sed command or there is good reason to allow .env files to have comments then this pipe can be added again

Not sure where to get PUMA boundaries, found download at https://www.census.gov/cgi-bin/geo/shapefiles/index.php?year=2010&layergroup=Public+Use+Microdata+Areas and saving to /resources folder but there may be a better way

Getting this error message when summing areas
UserWarning: Geometry is in a geographic CRS. Results from 'area' are likely incorrect. Use 'GeoSeries.to_crs()' to re-project geometries to a projected CRS before this operation.
Not sure how to interpret, going forward on analysis and hopefully CRS issues are easy to correct afterwards

Pushed code in jupyter notebook that iterates through PUMAs and calculates fraction of total area within historic district in each. When I checked numbers it seemed that the PUMA in south west manhattan (3801) had too low a fraction. Just realizing now that issue may be clipping PUMAs to shoreline, when I plotted PUMA dataset the boundaries weren't clipped to shoreline. Next step is find PUMA boundaries that are clipped to shoreline. Then incorporate this code into .py files

I'm working on cleaning up the data dictionary and noticed something surprising in the current definition for this indicator (copied below). It is saying that we included historic districts that were denied designation or overturned. Is that true? If it is, why did we do that?

Area within a historic district. All designated historic districts or areas under consideration for historic district designation (i.e. calendared) by the New York City Landmarks Preservation Commission (LPC), including items that may have been denied designation or overturned.


Login on Facebook with another account does not work

You can login on Facebook with an account and then Logout.
But if you want to Login again with another account, you only have 2 options:

Login with the old account
Cancel
(Cancel redircts to the homepage and cancels the login)

So there is no way to login with another account.
Apparently, it's an issue with the cookies/cache from m.facebook.com website, so if you clear the cookies and cache in Chrome, it will work, but it is not a solution.
I saw on other forums, that the login must include "FORCE_DIALOG_AUTH" parameter.
Thanks for your help

Sorry, I was wrong. Now it works.


Thank you very much for your FTFlash program, it helped me a lot

Thank you very much for your FTFlash program, it helped me a lot! 😄
I wish you good and stable health.

Hi @alex-kerner-ext-orion, I'm so happy you enjoyed this app! I made it for a one-off project last week and I'm surprised to see that someone else is using it already! The internet is an awesome place 🚀
I see the issues you opened on https://github.com/swharden/FTFlash/issues and I'll address those shortly 👍


First crack at updating the readme

Type: documentation
Goals
Include up front example
Simplify
Get users to things quicker
Move complicated details to wiki
No functional change
may require some updates to wiki
Release Notes

Please bike shed, I don't want to go to far with this re-design without some feedback.

For the FIR example, I think we should do several things:

Make this as terse as possible. Clash uses an FIR filter on their website and they keep it nice and compact.
I agree with @ducky64 in that the filter should be parameterized by coefficients and not by a window function.
The length parameter should be implicit based on the number of coefficients
We should not use an anonymous/structurally typed bundle for the IO
(Maybe) The FIR filter should be parameterized in terms of data type and not locked to UInt
We should include examples of using the filter, e.g., for different windows/coefficients
We should show the output of this in Diagrammer as opposed to with some custom SVG

I'm still golfing this down, but the following would be a start. (Not tested, though!)
class FIRFilterIO(length: Int, bitwidth: Int) extends Bundle {
  val in = Input(UInt(bitwidth.W))
  val out = Output(UInt((bitwidth * 2 + length - 1).W))
}

class FIRFilter(bitwidth: Int, coefficients: Seq[Int]) extends Module {
  val io = IO(new FIRFilterIO(coefficients.size, bitwidth))

  val z = Seq.fill(coefficients.size - 1){ Reg(UInt(bitwidth.W)) }

  z.foldLeft(io.in){ case (r, l) => l := r; l }

  io.out := coefficients
    .zip(io.in +: z)
    .map{ case (b, x) => b.U * x }
    .reduce(_ +& _)
}

@ducky64 @seldridge
How about
class FIR(bitWidth: Int, coeffs: Seq[Int]) extends MultiIOModule {
  val (in, out) = (IO(Input(UInt(bitWidth.W))), IO(Output(UInt(bitWidth.W))))

  out := coeffs
          .zip(in +: coeffs.tail.map { _ => Reg(UInt(bitWidth.W)) })
          .map { case (c, r) => c.U * r }
          .reduce(_ +% _)
}

mildly tested

@ducky64 @seldridge Ok trying again. This down to a single assignment. I had to make the coefficients UInts to avoid conversion. Adding a generator is tricky here without bringing in DspTools to manage the type management. Unless I'm missing something making signature something like
FIR[T <: Data](gen: T, coeff: Seq[T]) is hard to make the code work
class FIR(bitWidth: Int, coeffs: Seq[UInt]) extends MultiIOModule {
  val (in, out) = (IO(Input(UInt(bitWidth.W))), IO(Output(UInt(bitWidth.W))))

  out := coeffs
          .zip(in +: coeffs.tail.scan(in) { case (a: Data, _) => RegNext(a) })
          .map { case (c, r) => c * r }
          .reduce(_ +% _)
}

One question on the FIR example - what's our intended audience? Software functional programmers looking to learn some hardware, or digital designers looking to use/adopt Chisel? Depending on the audience the example could be attractive or repulsive. Maybe consider two examples for different audiences?

I'd argue that we should try to make the code should be understandable by the average software engineer (let's say, Java, C, C++ familiarity, experience with functional programming at the level of Python) and intuitively readable by the average hardware engineer who has some concept of what map and reduce does (ideally, this wouldn't be required, but I don't think we can make a powerful example without those fundamental constructs; compromise would be for the comments to patch across knowledge gaps).
I think code golf should be an explicit anti-goal, and breaking things up into multiple assign statements with comments as necessary can vastly increase readability. When the competition is the dumpster fire that is Perl generating (System)Verilog, I think achieving less-lines-of-code is already trivial, so there's no need to compromise on readability.
I also think we should try to avoid uncommon (in terms of the average audience - think Java, C, C++, Python, Verilog) syntax where possible in favor of more explicit constructs. Case statements should probably be out, and _, +:, and +% are debatable since symbol soup can be confusing. Deeply nested parentheses, like in the zip line, might also be questionable.

This is great feedback... I think there are competing Chisel features that we need to balance including:

Chisel is the same as Verilog (with restrictions that prevent user error)
Chisel is more "powerful" than Verilog/VHDL
Chisel provides a hardware compiler framework

The current example shows (2) only.
(1) is related to @edwardcwang's comment. Perhaps two examples, one that is (non-parameterized) Verilog translation and one that pushes the language. (1) is also tricky as there are advanced Chisel features (withClockAndReset, async reset, etc.) that provide Verilog parity, but likely are out of scope of an example for (1).
(2) could be shown by re-implementing (1) as parameterized with functional programming features.
(3) is not shown, but could be, e.g., BoringUtils, Verilog attribute injection, flatten/inlining. These, however, may require showing output Verilog/FIRRTL.
@ducky64: point taken; the golfing may be unnecessary here. This is more motivated by how Haskell is commonly presented as being "more powerful" than other languages, e.g., you can tightly represent this complex imperative construct with a series of functions. For reference, they use this example on their landing page. This is more an aspirational example of "Woah, I'd like to learn how to do that." One data point...
primes = filterPrime [2..]
  where filterPrime (p:xs) =
          p : filterPrime [x | x <- xs, x `mod` p /= 0]

One more idea to consider, pushing towards showing (1) and (2):
class FIRIO[A <: Data with Num[A]](dataType: A) extends Bundle {
  val in = Input(dataType.cloneType)
  val out = Output(dataType.cloneType)
}

/** A 3-coefficient FIR filterl  */
class FIRRectangular3(bitWidth: Int) extends Module {
  val io = IO(new FIRIO(UInt(bitWidth.W)))

  val b0 = 1.U
  val b1 = 1.U
  val b2 = 1.U

  val z0 = RegNext(io.in)
  val z1 = RegNext(z0)

  val p0 = io.in * b0
  val p1 = z0 * b1
  val p2 = z1 * b2

  io.out := p0 + p1 + p2
}

/** A generator of FIR filters of arbitrary coefficients and underlying data type (UInt, SInt, FixedPoint) */
class FIRGenerator[A <: Data with Num[A]](dataType: A, coeffs: Seq[A]) extends Module {
  val io = IO(new FIRIO(dataType))

  def dotProduct(a: Seq[A], b: Seq[A]): A =
    a.zip(b)                           // (a0, a1, a2) zip (b0, b1, b2)    => ( (a0, b0),  (a1, b1),  (a2, b2)  )
      .map{ case (aa, bb) => aa * bb } // ( (a0, b0), (a1, b1), (a2, b2) ) => ( (a0 * b0), (a1 * b1), (a2 * b2) )
      .reduce(_ + _)                   // (x, y, z)                        => x + y + z

  /** Construct registers that will hold time-delayed versions of the input */
  val z: Seq[A] = Seq.fill(coeffs.size - 1){ Reg(dataType.cloneType) }

  /** Construct a shift register from the input and time delay registers */
  z.foldLeft(io.in){ case (right, left) => left := right; left }

  /** Compute a dot product of the inputs/time delays with the coefficients */
  io.out := dotProduct(io.in +: z, coeffs)
}

class GeneratedFIRRectangular3 extends FIRGenerator(UInt(8.W), Seq(1.U, 1.U, 1.U))

class FIRFilterSpec extends ChiselFlatSpec {

  "foo"  should "foo" in {
    println(generateFirrtl(new FIRRectangular3(8)))
  }

  "bar" should "bar" in {
    println(generateFirrtl(new GeneratedFIRRectangular3))
  }

}

All this is great stuff for consideration but I think the immediate decision should be whether what I have done is better than what is currently there. I would argue that it is. The current page rambles on about ugly installation stuff that is not relevant. The new version shows some code then gets you to the bootcamp and template fast. I believe solving this was what I was tasked with doing. If someone wants to inject some other piece of code that's fine with me, but this PR should be acted on soon or dropped for further debate and consideration. I vote for going simple and soon to be followed up by more later.

@chick I think there's still some work to be done:

I think we need to have a good idea of who the target audience is for the overview. Is it PL academics? Is it software engineers looking to play wtih FPGAs? Or is it hardware engineers who wouldn't have a formal PL background? Because as written right now, a lot of terms are thrown around which might be correct but not meaningful to all members of these audiences. Example: hardware construction language, domain-specific language (totally an implementation detail), strong type checking.
Also, what's the message? The overview currently seems to be a mishmash of various facts about Chisel, but I'd imagine what a potential user needs answered is, "why should I use this", and more implicitly, "how does this compare against what I'm using right now".
Basically, the overview is probably going to be the first thing people read (if they find this repo first, as opposed to the website - though we might consider unifying the blubs), so you want to make a good impression there. You really only get one first impression.
I think it's worth discussing this in realtime at the next dev meeting.
One benchmark for a good intro: find a few 151 students, show them the overview text, and ask them how they interpreted it, and whether they would try Chisel. And keep in mind that this is even a bit of a low bar for intro text: 151 students would have taken the 61a/b/c series and have a reasonable programming background.
https://www.chisel-lang.org/ gives a cert error. Because the cert is issued to research.eecs.berkeley.edu...
Lots of copy-editing, the little things that make the documentation look more professional and clean instead of an afterthought / completely slapdash. Unless you intend this to be a rough draft, in which case, sure. Example: inappropriate use of headers to make text larger, which puts a header underline and appears as a section that doesn't yet have content. Caps in middle of sentence ("visit The wiki installation page"), broken sentences ("When you are ready to build and test your own circuits. Chisel3 uses"). You know, the kind of things that would say to users "hey, we really care about having users and are doing our best to support them", rather than "we spent the minimum amount of time putting together this minimum-viable-readme".
I'd also argue the front-page example is pretty important: it's the one place where you're showing rather than telling. And one would hope what is shown reinforces what is told.

@seldridge I think in designing the example, there's kind of an overarching question of whether we want to be more like Python (powerful but still mass-adoption friendly) or Haskell (super efficient for power users, but steep learning curve). It appears much more programmers use Python than Haskell, though we can only conjecture at the causality (why).

Consider putting yourself in the shoes of someone in your target audience and showing them the code snippet.

For the Haskell-style folks, the response might reasonably be "I'll spend some quality time reading through and and trying to comprehend this example because it's so concise."
For the Python-style folks, the response might reasonably be "wtf is this ****, it makes no sense at all. Wow, this language seems like it could have quite the nasty learning curve."

I'd also recommend showing this to hardware engineers who haven't used Chisel before and asking for feedback.
Remember that there is always a third option: you can place the easily understandable example in the readme, but link to more advanced examples. The message would be kind of "this example shows that Chisel is miles ahead of the dumpster fire that is Perl generating Verilog, but if you're into playing code golf rather than practical software engineering we've [got you covered too]"


Update readme to talk about sizing modern refinery (stress relief, etc)

Refinery readme doesn't adequately explain stress relief or sizing refinery; needs a bit of focus to get it right.

This is some good goalie work for me this week. Picking it up.

was this closed by #955 or just updated?

Ah, forgot the note! Thanks.


Add halt! to stop Clerk

It would be great to be able to stop clerk via the repl using a single command. This PR achieves that by

Extracting the webserver stop code out of nextjournal.clerk.webserver/start! into the stop! function
Extracting the beholder stop code out of nextjournal.clerk/serve! into the stop-watcher! function
Adding a new function nextjournal.clerk/halt! that calls both functions from 1 and 2

Hey @SneakyPeet, sorry it took a while to get around to this but it's now merged after some minor changes. Thanks for your PR!


Updated Journal week 2

:+1: Staying alive! Good structure, and I agree with your insight on DOM's.

:+1: Yay we survived! Good rough draft. Keep up the good work!


Bring the options menu back to life. Should match its pre-merge glory…

…... except for the alignment part.
Relates to #24.

thank~


Add Stylecop ruleset

Added basic Stylecop, using same rules as https://github.com/Azure/azure-webjobs-sdk
Still open (for a new PR):

setting up nuget package publishing
signing assembly

Fixes #22

We'll add nuget package publishing as part of #25


pymssql showing in logs when pyodbc is specified

config
  - name: tap-mssql--db
    inherit_from: tap-mssql
    variant: buzzcutnorman
    pip_url: git+https://github.com/BuzzCutNorman/tap-mssql.git
    config:
      dialect: mssql
      driver_type: pyodbc
      database: db1
      user: myuser
      port: 1433
      sqlalchemy_url_query:
        driver: ODBC Driver 17 for SQL Server
        trustServerCertificate: yes
        multiSubnetFailover: yes
      sqlalchemy_eng_params:
        fast_executemany: 'true'

inherit from
  - name: tap-mssql
    variant: buzzcutnorman
    pip_url: git+https://github.com/BuzzCutNorman/tap-mssql.git
    config:
      dialect: mssql
      driver_type: pyodbc
      port: 1433
      sqlalchemy_url_query:
        Driver: ODBC Driver 17 for SQL Server
      sqlalchemy_eng_params:
        fast_executemany: 'true'

Error message
2023-10-20T17:57:05.771235Z [info     ] time=2023-10-20 13:57:05 name=target_snowflake level=INFO message=Table db."table1" exists cmd_type=elb consumer=True name=target-snowflake producer=False stdio=stderr string_id=target-snowflake
2023-10-20T17:57:07.507385Z [info     ] time=2023-10-20 13:57:07 name=target_snowflake level=INFO message=Table db."table1" exists cmd_type=elb consumer=True name=target-snowflake producer=False stdio=stderr string_id=target-snowflake

asides from the specifics of the error -its weird the error has pymssql in it not pyodbc.

Looking at the error message it is referencing a tap-mssql--db.  Would you be able to pass along that tap's config as well.

should be that first one in the config above

I was able to replicate the issue.  It seems once I remove the pip_url and variant items from the tap-mssql--db config inheritance started to work properly again.  If those two config options are present and say driver_type is not in the config since it has a default the default is used which would be pymssql.  Please try removing pip_url and variant from tap-mssql--db's config and let me know if things working as expected.


example not working

after building:
./target/asciidoc-splitter-1.2.1-SNAPSHOT-bin/bin/asciidoc-splitter.sh  -s ./examples/sample/input/ -o ./examples/sample/output/
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.headius.backport9.modules.Modules (file:/home/paulwright/repos/asciidoc-splitter/target/asciidoc-splitter-1.2.1-SNAPSHOT-bin/lib/backport9-1.3.jar) to method sun.nio.ch.NativeThread.signal(long)
WARNING: Please consider reporting this to the maintainers of com.headius.backport9.modules.Modules
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
14:38:16,098 SEVERE [] (main) ERROR: null: java.nio.file.NoSuchFileException: /home/paulwright/repos/asciidoc-splitter/examples/sample/input/titles-enterprise
14:38:16,124 WARNING [] (main) Found 1 issues. 1 Errors.

Perhaps this example should be removed if it doesn't conform in some way to expected input

The examples really need to go away, they're more of test cases. It needs to be cleaned up.


Update dependent package pinning to newer versions.

[X] Have you provided a meaningful PR description?
Bumps the minimum version requirements for dpcpp, dpctl, numba, dpnp in meta.yaml and all environment files.

👍


pylint unable to verify fastapi import

Using:
FastAPI 0.63.0
Python 3.7.7
OS: Mint 19
I need to prevent pylint from reporting E0401 error. Below are the steps taken to reproduce the problem in an isolated virtual environment (command + output is shown).
derek@pc:~/tmp$ mkvirtualenv slow
created virtual environment CPython3.7.7.final.0-64 in 182ms
  creator CPython3Posix(dest=/home/derek/.envs/slow, clear=False, global=False)
  seeder FromAppData(download=False, pip=latest, setuptools=latest, wheel=latest, via=copy, app_data_dir=/home/derek/.local/share/virtualenv/seed-app-data/v1.0.1)
  activators BashActivator,CShellActivator,FishActivator,PowerShellActivator,PythonActivator,XonshActivator
virtualenvwrapper.user_scripts creating /home/derek/.envs/slow/bin/predeactivate
virtualenvwrapper.user_scripts creating /home/derek/.envs/slow/bin/postdeactivate
virtualenvwrapper.user_scripts creating /home/derek/.envs/slow/bin/preactivate
virtualenvwrapper.user_scripts creating /home/derek/.envs/slow/bin/postactivate
virtualenvwrapper.user_scripts creating /home/derek/.envs/slow/bin/get_env_details
(slow) derek@pc:~/tmp$ pip freeze
(slow) derek@pc:~/tmp$ pip install fastapi
Collecting fastapi
  Using cached fastapi-0.63.0-py3-none-any.whl (50 kB)
Collecting starlette==0.13.6
  Using cached starlette-0.13.6-py3-none-any.whl (59 kB)
Collecting pydantic<2.0.0,>=1.0.0
  Using cached pydantic-1.8.1-cp37-cp37m-manylinux2014_x86_64.whl (10.1 MB)
Collecting typing-extensions>=<IP_ADDRESS>
  Downloading typing_extensions-<IP_ADDRESS>-py3-none-any.whl (26 kB)
Installing collected packages: starlette, typing-extensions, pydantic, fastapi
Successfully installed fastapi-0.63.0 pydantic-1.8.1 starlette-0.13.6 typing-extensions-<IP_ADDRESS>
(slow) derek@pc:~/tmp$ nano test.py

The test.py file simply has one line:
import fastapi

Now run pylint on the file:
(slow) derek@pc:~/tmp$ pylint test.py
************* Module test
test.py:1:0: C0114: Missing module docstring (missing-module-docstring)
test.py:1:0: E0401: Unable to import 'fastapi' (import-error)
test.py:1:0: W0611: Unused import fastapi (unused-import)

If I try to install and import another library, I do not have this error:
(slow) derek@pc:~/tmp$ pip install requests
Collecting requests
  Using cached requests-2.25.1-py2.py3-none-any.whl (61 kB)
Collecting idna<3,>=2.5
  Using cached idna-2.10-py2.py3-none-any.whl (58 kB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.4-py2.py3-none-any.whl (153 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2020.12.5-py2.py3-none-any.whl (147 kB)
Collecting chardet<5,>=3.0.2
  Using cached chardet-4.0.0-py2.py3-none-any.whl (178 kB)
Installing collected packages: idna, urllib3, certifi, chardet, requests
Successfully installed certifi-2020.12.5 chardet-4.0.0 idna-2.10 requests-2.25.1 urllib3-1.26.4

Add the import requests to the test file.
(slow) derek@pc:~/tmp$ pylint test.py
************* Module test
test.py:1:0: C0114: Missing module docstring (missing-module-docstring)
test.py:1:0: E0401: Unable to import 'fastapi' (import-error)
test.py:1:0: W0611: Unused import fastapi (unused-import)
test.py:2:0: W0611: Unused import requests (unused-import)

indeed!
/Users/spacex/PycharmProjects/fastatpi-start/main.py:1: [E0401(import-error), ] Unable to import 'fastapi'

Thanks for the help here everyone! 👏  🙇
Thanks for reporting back and closing the issue @gamesbook  👍

For completeness, if Spyder uses its own version of Pylint instead of the one you install in the local environment for your project, then it would actually be Spyder's fault/error. That would probably be a report/request there.

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


WIP: adding Golang

Hi @paul-111!  Thanks for opening your first pull request, and welcome to the Veracruz project!   🎉
What are your plans with this pull request?  Do you intend to add some Go language translations of our existing examples, in sdk/rust-examples?  That would be very helpful, and a valuable contribution to the project, if so...
However, before any pull request can be merged, two issues need to be resolved:
Developer Certificate of Origin
As this is your first commit to the Veracruz repository can you please certify that you have authorisation to contribute code using the Developer Certificate of Origin?  This is a requirement of the Veracruz organization, which we agreed with the Linux Foundation when we were adopted.  To do this, it's enough to include the following text in your PR message, at the top:
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Signed commits
Can I ask that you sign all of your commits using GPG?  We adopted this rule a while ago as a security/integrity mechanism and code cannot be merged into the main Veracruz repository without this signing.  If you don't know how to do this, then follow these instructions:

Generate a new GPG key: gpg --gen-key and follow the instructions asking for your name, email, etc.
List the GPG keys available: gpg --list-secret-keys --keyid-format=long.  Make a note of the ID of the key you’ve just generated (it’s a long string of capital letters and numbers),
Export the public key component of the generated key to stdout : gpg --armor --export $KEYID where $KEYID is the long string of letters and numbers, above.
Go to your Github profile’s settings page, go to the Keys section, add a new GPG key and copy-paste the public key printed out above.
Tell Git to use this key when signing commits: git config --global user.signingkey $KEYID , again where $KEYID is the long string of letters and numbers, above.
  Note that Github signs their own merge commits made to a repository when a pull request is merged, but the command above may not show the signed commits correctly (they’ll be highlighted in red) as you need to import Github’s public key to verify the signature.  To do this, import Github’s key by: curl https://github.com/web-flow.gpg | gpg --import Showing the signatures in Git log should now work, and Git should recognise that Github’s merge commits were signed by Github (they should now be blue).

Once you've setup GPG signing, you usually have to export GPG_TTY=$(tty) in your terminal (or you'll get an error about a missing tty) prior to commiting/rebasing.  Once that is done, edit your PR to sign all commits and push again.

Hii @dominic-mulligan-arm i am working as an outreachy applicant under mentor Shale Xiong where he gave me task regarding "Instructions for compiling Golang into WASM and running it using Wasmtime ", the fact that i am working under outreachy program do i still need this Developer certificate of origin and Signed commits issues to solve ? Thank you

Hi @paul-111, OK great, thanks for that information---I know @ShaleXIONG has been very busy recently co-ordinating with different prospective candidates from Outreachy.  I think what would make your PR especially valuable is not just information on how to compile Go programs for Wasm, but also to try to translate some of our existing examples to Go, in addition to the instructions, as I mentioned.  Can you try and translate, e.g., one or two of the Rust examples into Go, and add them to this PR?
(Further, just echoing @dreemkiller): yes as a prospective Outreachy intern you need to include the Developer Certificate of Origin in your commit message, as it means we then know that the work is yours, and you have permission to submit it to Veracruz.  Also, every commit that goes into our repository must be signed by its owner: Github will simply not allow us to merge your PR without that signing.  This is a security/integrity mechanism and unfortunately cannot be over-ridden, even if it is a bit of a pain for new users 😀

Hi @paul-111 If you have questions on the merge request, we can have a video chat and you can share your screen so we sort out the problem together. You can directly find/ping me in the slack or email me.
I also remember you had the problem of clone the repo; however it looks like you sort it out.

@paul-111 is this PR being worked on, still?

@dominic-mulligan-arm  hii yes, this PR needs some modification and i am working on it and i might be able to update it by 2/3 days

Great, thanks @paul-111.  In the meantime, I will convert it into a draft until it is ready for review.

@dominic-mulligan-arm  hii regarding this gpg key , would you plz tell me is there any specific directory under which i have to create it or just simply under veracruz?

@dominic-mulligan-arm hii regarding this gpg key , would you plz tell me is there any specific directory under which i have to create it or just simply under veracruz?

You can do it from the project root directory (/veracruz) to be safe. But it should also work from any other directory (I tried it both ways). The GPG key is not specific to a single repository.
<EMAIL_ADDRESS>
You can follow the steps on this page if it helps!


Add highlight to active techdocs navigation item and parents

This change adds highlighting to navigation items in tech docs. It highlights
the whole nav tree with the same color and an underline.
I figured this a patch as we already have some highlighting, but this "fixes"
it with color-blind assistance, ie. underline, and highlights the full tree.
Fixes #5596.
Signed-off-by: Crevil<EMAIL_ADDRESS>Light mode

Dark mode

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Starting to review!

Great idea. I'll see what I can do 👍 the underline is in place to make for a better experience for color blind and a like. Having just the color reduces the value IMO. WDYT?

Great idea. I'll see what I can do 👍 the underline is in place to make for a better experience for color blind and a like. Having just the color reduces the value IMO. WDYT?

Looks good, thanks for trying to improve the scroll too!

Hmm. Based on this, we should already have the scrolling functionality in place 🤔
https://github.com/squidfunk/mkdocs-material/issues/2155#issuecomment-1030816004

@Crevil looks like it's only available to insiders, not released to non sponsors because the funding goal hasn't been reached. :shrug:

Sure, let's merge, thanks again 🙌🏻

I have created #12872 which adds navigation scroll as well 🙏


No module named 'dataset.deepmind'

Hi, similar to previous :
Traceback (most recent call last):
  File "eval_math_gpt.py", line 37, in <module>
    from dataset.deepmind import DeepMindMathDataset
ModuleNotFoundError: No module named 'dataset.deepmind'

On commenting out the same next error :
Traceback (most recent call last):
  File "eval_math_gpt.py", line 360, in <module>
    parser.add_argument('--arch', default='gpt2', choices=transformers.GPT2_PRETRAINED_MODEL_ARCHIVE_LIST)
AttributeError: module 'transformers' has no attribute 'GPT2_PRETRAINED_MODEL_ARCHIVE_LIST'

We have updated the script again. Hopefully it's workable.

still getting the second error. which version of the lib do you have ?

Seems to work for transformers 4.3.2 on my end.


MISC should be using question instead of Question to display the correct message.

MISC should be using question instead of Question to display the correct message.

Please review If it makes sense @zainhoda :)

Thank you!


Fix typo

python -> Python

Learn Build status updates of commit 91d7f3c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/sentinel/connect-log-forwarder.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


Add coinmarketcap.com as source for teeracle

Snapshot of discussion in https://github.com/integritee-network/worker/pull/594 to give some more information about the code rework.


running fetchcord as .service is failing to get terminal info

Debug info
----out.py----

----DE/WM----
deid: N/A
wmid: N/A
wmline item 0: ['WM: N/A']

----TERMINAL----

fontline: ['Font: N/A']
termid: N/A
termline item 0: Terminal: N/A
themeline: Theme: N/A

----HOST INFO----

hostline: Host: N/A
batteryline: Battery: N/A
resline: Resolution: N/A

----GPU INFO----

gpuinfo: GPU: N/A
gpuvendor: N/A

----CPU INFO----

cpuvendor: N/A
cpumodel: N/A
cpuinfo: CPU: N/A
cpuline item 0: ['CPU: N/A']
memline: Memory: N/A

----OS INFO----

sysosline: OS: N/A
sysosid: N/A
diskline: Disk: N/A
packagesline item 0: Packages: N/A
Unknown GPU, contact us on github to resolve this.
Unknown DE/WM, contact us on github to resolve this.
Unknown shell, contact us on github to resolve this.
Unknown Host, contact us on github to resolve this.(Keyerror)
Unsupported Terminal. contact us on github to resolve this.(Keyerror)
Unsupported Distro, contact us on the GitHub page to resolve this.(keyerror)

----testing.py----
----DE/WM----

deid: N/A
wmid: N/A

----TERMINAL/SHELL----

termid: N/A
shellid: unknown

----HOST INFO----

hostid: o

----GPU INFO----

gpuvendor: N/A

----CPU INFO----

cpumodel: N/A

basically fetchcord as a .service is trying to get the terminal its running on (systemd)
it cannot display it so you put this handy thing that fixes that, it doesnt.
setting custom time 15 seconds
Custom terminal font not set because a terminal font already exists, ['Terminal Font: Iosevka 15.0 ']
Traceback (most recent call last):
  File "/home/wael/.local/bin/fetchcord", line 5, in <module>
    from fetch_cord.run_rpc import main
  File "/home/wael/.local/lib/python3.8/site-packages/fetch_cord/run_rpc.py", line 11, in <module>
    from fetch_cord.testing import gpuid, cpuappid, appid, desktopid, termappid, hostappid, shellid, moboid
  File "/home/wael/.local/lib/python3.8/site-packages/fetch_cord/testing.py", line 119, in <module>
    termline[0] = "Terminal: %s" % args.terminal
TypeError: 'str' object does not support item assignment

this will repeat several times.

Hello, testing.py won't exists anymore,  but i'll need to check that on 2.7

since this was 21 DAYS ago i will now test it if it still works

Yeah, sorry for the delay, i'm currently the only person active at the time on the project, i made a new architecture on branch 2.7 and now i'm trying to close the issues :)

does it work yet?

I will close the issue now, reopen if it still occurs


Fix RETURNs opcodes

Pull request type
Please check the type of change your PR introduces:

[x] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?
The return_data of a given context is assigned a new segment with the RETURN opcode.
The RETURNDATASIZE and RETURNDATACOPY use the calling context data instead of the child context one.
Fix: #327
What is the new behavior?
The return_data segment is filled but no new segment is created.
The RETURNDATASIZE and RETURNDATACOPY use the child_context data.
Other information
It's sort of a hack that I have used to be able to store the ret_offset in memory because we cannot assign a segment in memory.

Resources-tests fails because this PR adds one field to the execution context


Perfectionism regarding Handling of Number Types

Basically, only one QNumberPath is used for handling all different number types.
Currently known Flaws

[ ] only Double and Decimal should allow for round, ceiling and floor functions

Challenges
Changing from one QNumber type to several opens a can of worms:

understanding of casting rules becomes relevant

adding Edm.Byte and Edm.Int16 => Should be Edm.Int16?
what if we add value directly => Edm.Byte add 16.1
differences: v2 vs v4?

testability is limited

RAP & CAP don't support arithmetic operators

V4 only: casting would be relevant

reopen, if this becomes more relevant.


Fix ListContainers request on empty ListContainersDetail

Previous code was generating an invalid include="" parameter when provided ListContainersDetail was empty (zero value).
Sample code to reproduce the issue (using Azurite latest (3.13.1) dockerhub image - https://hub.docker.com/_/microsoft-azure-storage-azurite):
resp, err := serviceURL.ListContainersSegment(
		context.Background(),
		azblob.Marker{},
		azblob.ListContainersSegmentOptions{}) // or even ListContainersSegmentOptions{Detail:azblob.ListContainersDetail{Metadata: false}}
if err != nil {
  log.Fatalf("Couldn't list containers: %s", err) // gives 400 bad request, GET http://<IP_ADDRESS>:10000/devstoreaccount1?comp=list&include=&timeout=61
}

Previous code was building the equivalent of a []ListContainersIncludeType{""} (a slice of 1 string element with empty string as value), which is playing badly with the way func (client serviceClient) listContainersSegmentPreparer(....) is building that 'include' parameter:
// see zz_generated_service.go around lines 417++
func (client serviceClient) listContainersSegmentPreparer(...., include []ListContainersIncludeType, ...) {
[...]
  if include != nil && len(include) > 0 {
          // include query param is added as soon as we have a non-nil, non-empty slice.
          // in above case, it effectively do: params.Set("include",""), leading to invalid request for Azure/Azurite
	  params.Set("include", joinConst(include, ","))
  }
[...]
}

/run test


Columns under multi-column header do not display data.

When using the igGrid with Angular 2 and the grid has multi-column headers any column that is under a multi-column header will not display data.
Angular2_Service.zip

This is a bug related to renderNewRow API in the igGrid(it wasn't able to recognize if there are multi column headers) and it is working fine with the latest dev code.

@igMPeterson @dkamburov I will mark this as fixed as soon as 16.2 is released.

I am having the same issue, but I am using code in the Angular2_API branch. Does the Angular2_API branch have this fix.


Fix CreateSessionResponse parsing

The endpoint for creating video sessions returns a response wrapped in an array. This PR fixes the parsing failing due to expecting a non-wrapped JSON response instead.

Codecov Report

:exclamation: No coverage uploaded for pull request base (8.x-beta@2d14d22). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             8.x-beta     #420   +/-   ##
===========================================
  Coverage            ?   91.69%
  Complexity          ?     1962
===========================================
  Files               ?      334
  Lines               ?     5274
  Branches            ?      230
===========================================
  Hits                ?     4836
  Misses              ?      345
  Partials            ?       93


feat: add a default internal schema

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Fixes #2964
This patch introduces a new built-in default schema as namespace of tables created by greptime applications. We have more information discussed in #2964 . In most cases, I think our user won't need to know about this schema.
Checklist

[x]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

@waynexia If I understand correctly it should be done here: https://github.com/GreptimeTeam/greptimedb/pull/2974/files#diff-44b4862dfce3f3bb50a4845cd5bc075169fa40e0a678ff23ed9816c59c62ac06R308

@waynexia If I understand correctly it should be done here: https://github.com/GreptimeTeam/greptimedb/pull/2974/files#diff-44b4862dfce3f3bb50a4845cd5bc075169fa40e0a678ff23ed9816c59c62ac06R308

@WenyXu @fengjiachun Please take a look

Do we need to register this schema on metasrv? Others looks good to me

Yes, it is. BTW, there is some future work that registering schema should be executed only on the metasrv leader.


Oheger bosch/licenses/#3274 rename license classes

Renamed two classes to complete the rework of license classifications. Please refer to the commit messages for further details.

Merging this despite the unrelated failure of GitRepoFunTest on Azure / Windows.


Remove dwp_check blocked

As part of this ticket we are removing
the no longer used features.
This PR removes the stand-alone dwp_check feature.
It is currently blocked because a data-retention policy is needed.

:+1: Looks good, I'm looking forward to merging this :)


Disable a cop for a specific directory

Expected behavior
I would like to be able to disable a cop for a specific directory.  This is an issue arose from working with minitest and bumping into Metrics/BlockLength violations.  I want that cop disabled for test/ but enabled for everything else.
Actual behavior
Currently a cop is either on or off for all files and folders.  I can exclude the test/ directory entirely but that disables all cops and I just want to disable one cop.
Steps to reproduce the problem

Create a test directory that contains a file with a Metrics/BlockLength violation and another violation.
Create a lib directory that contains a file with a block that violates the maximum number of lines.
I want to exclusively see the non-Metrics/BlockLength violation in test/ and the Metrics/BlockLength violation in lib.

RuboCop version
Include the output of rubocop -V. Here's an example:
[0] » rubocop -V
0.45.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.1 x86_64-linux)

This is already something that is supported by RuboCop.
Add this to your project's .rubocop.yml file.
Metrics/BlockLength:
  Enabled: true
  Exclude:
    - test/**/*

@rrosenblum excellent! thank you.

Just to chime in, shouldn't adding a new .rubocop.yml file in the directory of interest (here, test/), with
Metrics/BlockLength:
  Enabled: false

have the same effect? We're currently debating which usage is preferable: a Single Point of Truth should mean "no surprises", but at the cost of a (potentially) unwieldy process of flipping through Exclude: directives to see which if any apply to the file/directory that's confounding you at the moment.
Thoughts on that?

@jdickey we have a 'Single Point of Truth' and it works well.  A nice side effect is that it standardizes some directory structures.
I would prefer looking at one file over searching the file tree and mentally determining the inheritance.

@jdickey, I agree with @lanej. I think there is good value in maintaining most/all of the configuration in a single top level .rubocop.yml. Both ways are supported because everyone will have their own opinions on the topic.
One scenario that I can think of which would require multiple .rubocop.yml files is if you wanted to have different configuration values for a cop in a directory. For example, rather than disabling Metrics/BlockLength for the test directory, the test directory could have a .rubocop.yml file that changes the Max value of Metrics/BlockLength. As far as I know, changing the value of EnforcedStyle can only be done with separate configuration files.

@rrosenblum Agreed, which is why I gave a 👍  to @lanej. My own personal feeling aligns with what I understand of yours:

enable/disable cops per directory in a master .rubocop.yml;
set project-wide defaults there;
use per-directory .rubocop.yml files for experimentation; when you find what works, then
keep "approved" per-directory settings in the top-level .rubocop.yml

We had one project a while back that would rewrite subdirectory-level .rubocop.yml files based on tags in the most recent commit of each of the files in that directory. We're much better now, thanks.

We generally use a mixture of styles: Our project standards all sit within PROJECT_ROOT/.rubocop.yml, which by default just inherits from our central house style gem.
However, we have additional RSpec-specific settings in PROJECT_ROOT/spec/.rubocop.yml, because we have much more relaxed rules about method length, line length, etc. and it helps to have all those differences in just the one place. The spec/ rubocop file inherits from the project file, and from the house style gem's RSpec defaults.
At the moment, we also have a separate settings file for Rails apps' db folder, but I'm in the process of phasing that out.


simple the call chain for cluster related commands

Simple the call chain for cluster related commands for JedisCluster
Motivation:
The long call chain will be written to execute all cluster commands in ClusterCommands.
For example. to retrieve cluster info:
JedisCluster.getClusterNodes().values().iterator().next().getResource().clusterInfo()
What's more, to avoid hot calls to single fixed node:
Collection<JedisPool> jedisPools = JedisCluster.getClusterNodes().values();
if(jedisPools.isEmpty()) {
     throw new JedisClusterException("not any known node in cluster");
      }
      Collections.shuffle(jedisPools);
     return jedisPools.iterator().next().getResource().clusterInfo();
}

Modifications:
provide new method: getAnyClusterNode() with nodes shuffled.
Result:
it will be simple to execute all cluster related commands:
JedisCluster.getAnyClusterNode().getResource().clusterInfo()

compile error


Add mentions

Add mentions for: csharp-mvc, csharp, java, php, python and ruby projects

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: aleksanderfedorov:x: metoouYou have signed the CLA already but the status is still pending? Let us recheck it.


As a user, I want the card image list paginated so that I don't have to browse through all the cards at once.

Notes for testing
Card list should have next and previous links for navigating to other chunks of results; you should be able to navigate and see the next set of results.
There should also be a drop down with alpha-labels showing the sections of results, and you should be able to navigate between sections that way also.

YES, I can use the NEXT and PREVIOUS buttons at the top of the page to skip to the next chunk of cards. So if I'm on Catel thru Colens and want to get to the next bunch, I press "next" and see displayed Collins thru Dent. AND I see that's what I'm going to get thanks to the "DISPLAYING" button.
YES, if I want to move more quickly through the cards, I can click the drop down DISPLAYING button and see the alpha-labels, and I can move through those. So if I click on the displaying "Parr - Potte" I will get to a page that starts with "Parrain" and ends with "Pottecher"

QUESTION: for the alpha-labels, is there a character limit on what displays? Right now they seem mixed. I think a 3 character limit would look better.
For example, there is "A - Bos" (why A? Why not "Abr"?)
"Kn - Leve"  (Why Kn and not "Kni"
"Pott - Rey," (note that there's a comma included here, which I think looks odd)
"Win - E" (this shouldn't end with "E")

Also, the last three cards go PAST "Z" which should be the last cards listed I would think, if we're sticking with alphabetical.

Tablet:

YES, I can use the NEXT and PREVIOUS buttons at the top of the page to skip to the next chunk of cards.
YES I can click the drop down DISPLAYING button and see the alpha-labels, and I can move through those.

Mobile:

YES, I can click through to NEXT and PREVIOUS buttons, but they're not listed here as next or previous. Just arrows. I imagine this is because the lack of room on the phone display.

YES, I can use the DISPLAYING drop down menu to go through chunks of names

@clmahoney just a note on the sorting - the reason that there are different character lengths in the options is because we automatically add only enough characters that are needed to distinguish between pages. so, for example:
Abreu
Ably
Abbott
...
Hemingway
Harris
Ingrid
...

to distinguish between the top three, we need at least three characters (Abr, Abl, Abb) since the first two are the same. to distinguish between Hemingway and Harris we need only two (He, Ha) and between those two and Ingrid only one (H, I). there's no limit, and the sort will automatically use the minimum number of characters required to properly distinguish between the names at a page boundary.
hope this makes sense.

@thatbudakguy thanks for explaining! Makes a lot of sense


using albert type model

hi
trying to run experiments with Albert for all sampling methods. The check model head function in samplying.py is rejecting the model as it fails to find the lm or sc attributes (mlmhead and sequence classifier). I can change this function but is there a reason albert (albert-base-v2) model shouldnt work
thanks
Ron

already asked closing


Feature/test cases

Summary

Added Test cases to validate the tests mentioned in https://github.com/moest-np/center-randomize/issues/4 .

एक विद्यालयको परिक्षार्थी संख्या हेरी सकभर १००, २०० भन्दा बढी परीक्षार्थी एकै केन्द्रमा नपर्ने गरी बाँढ्न पर्ने
आफ्नै विद्यालयमा केन्द्र पार्न नहुने
दुई विद्यालयका परीक्षार्थीको केन्द्र एक अर्कामा पर्न नहुने, अर्थात् कुनै विद्यालयका परीक्षार्थीको केन्द्र परेको विद्यालयका परीक्षार्थीहरूको केन्द्र अघिल्लो विद्यालयमा पार्न नहुने ।
एकै स्वामित्व / व्यवस्थापनको भनी पहिचान भएका केन्द्रमा पार्न नहुने
विगतमा कुनै विद्यालयको कुनै केन्द्रमा पार्दा समस्या देखिएकोमा केन्द्र नदोहोऱ्याउन नहुने
प्रत्येक पटक केन्द्र तोक्ने प्रोग्राम चलाउदा फरक फरक नतिजा आउने गरी ऱ्यान्डमाइज भएको हुनु पर्ने

Pull request type

Please try to limit your pull request to one type, and submit multiple pull requests if needed.

Please tick the type of change your PR introduces:

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behaviour?
Currently there is no Tests to validate/ accept that the feature submission is passing the acceptance criteria.

Please describe the behaviour that needs to be modified or linked to a relevant issue.
This Pull request is adding a feature in utils to parse a process csv file and declared testcases to tackle the problem.
Either pytest or simple python test_cases.py needs to be run to ensure that the new submissions are passing the acceptance criteria

Issue Number: #4

Please describe the changes or updates being made by this pull request.

Added custom TSV parsers class in utils folder to retrieve the informations from the input and results folder
Added unit tests to validate the results are as expected
tests need to be run multiple times as the results are non deterministic to get the best desired outcome of this program

Checklist
Please review and complete the following checklist before submitting your pull request:

[X] I have tested the changes locally and they work as intended.
[X] I have provided a detailed description of the changes made.
[X] I have reviewed the formatting and ensured it follows the project's style guidelines.
[X] I have assigned the appropriate labels to the pull request.
[X] I have added necessary documentation or updated existing documentation for the changes made.
[X] I have addressed code review feedback and resolved any conflicts with the main branch.

Other information

Include screenshots of the component before and after the change.
As per the image the test is failing if any of the test cases outlined in the issue is failing

@sapradhan Could you please kindly review my new updates ??

looks good to me, much needed testing. is there a way to repeat test say 10-20 times, regenerate the output as well?

Yes absolutely, in my fork I have integrated GitHub actions to run this test 10 times and produce output. Link Here:- https://github.com/samurato/center-randomize/actions
I am not sure if actions will run in the main repo if it will I will modify the actions and push it to the main repo.

Probably will create a new issue to implement this later if this PR is merged

a bit skeptical about test_undesired_cscode_scode_pair it fails with a version of prefs.tsv.
dont think the way get_scode_cscode_id disregards scode / cscode is the correct. at least problematic history is not bidirectional
skipping that test but merging as a lot of PRs are blocked because of lack of tests

a bit skeptical about test_undesired_cscode_scode_pair it fails with a version of prefs.tsv.
dont think the way get_scode_cscode_id disregards scode / cscode is the correct. at least problematic history is not bidirectional
skipping that test but merging as a lot of PRs are blocked because of lack of tests

Understood. I will work on a more optimised code to tackle those test cases.

a bit skeptical about test_undesired_cscode_scode_pair it fails with a version of prefs.tsv.
dont think the way get_scode_cscode_id disregards scode / cscode is the correct. at least problematic history is not bidirectional
skipping that test but merging as a lot of PRs are blocked because of lack of tests

Understood. I will work on a more optimised code to tackle those test cases.
@sapradhan @sumanashrestha

In my previous commits I had added individual test cases to very for each less preferred allocation. For which reason was assumed as a unique type
But I was made aware that the  reason column is an open text field. Hence I  generalised the logic for any combinations of less preferred to be flagged as fail.  Can we assume the reason column to be a classifier or have a seperate column as a reason type ?

In that way we can customise tests for each reason.

the test should simply check for each row in output file if there is a preference score it should be > PREF_CUTOFF


.NET Framework Support

In the current 6.0.0 beta support for net451 was dropped in favour of netstandard20 which we understood to provide support for .NET 4.6.1 and up, however recently there is increasing consensus that .NET Standard 2.0 support for .NET Framework below 4.7.2 is problematic.
It might be a good idea to in this case explicitly multi-target 4.6.1, or alternatively make it clear that support below 4.7.2 will be non-ideal and possibly problematic.
Thoughts?

My opinion would be to go with adding a net461 target back.

Mine too, it was dropped because it felt a bit gross straddling 2 major versions of the logging abstractions package.
See here:
https://twitter.com/stuartblang/status/1034427988224421888

We have to support .NET framework  4.6.x at present for a large number of codebases, so adding it back it might be necessary.

We are back to this:
https://github.com/justeat/JustSaying/pull/361#issuecomment-399457168
😐

I guess fixing this is a must for 6.0.0 now?

I think so, yes. We need a second target, be it 4.6.2 or 4.5.1

I thing based on it being a major bump since 5 and 4.5.1 being out of support (?) now, we should go with net462.


FR - Duplicate Tweets "New Session" & "Joined Session"

When a user starts a new session and twitter is configured, server will tweet " kicked off a new session" and " joined the session", effectively creating duplicate tweets and spamming the timeline.
Suggest suppressing " joined the session" tweet in the event of a new session.

Old issue cleanup: Partially resolved by #363; not planning further action.


feat: add git_submodule_clone

Closes #609

Not sure if this is a faulty implementation or not, but calling submodule.open() actually failed after I called this clone implementation. No error message, just () was displayed inside the stacktrace.

Thanks for taking the time to help me out @ehuss! :)


adjust preprocessor directives

The first directive used by non-Unity platforms such as Xamarin, it shouldn't be removed.


Sqauare  zoom animation added

Title and Issue number

I have added the project .Can it be merged now?
Close #4974

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md

@MohitGupta121 ,@TusharKesarwani Please review my PR.


compile() on react-native-i18n package is not found; build always failing when package is used

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.

Here is the diff that solved my problem:
diff --git a/node_modules/react-native-i18n/android/build.gradle b/node_modules/react-native-i18n/android/build.gradle
index 2614c62..746b873 100644
--- a/node_modules/react-native-i18n/android/build.gradle
+++ b/node_modules/react-native-i18n/android/build.gradle
@@ -22,5 +22,5 @@ android {
 }

 dependencies {
-  compile "com.facebook.react:react-native:+" // From node_modules
+  implementation "com.facebook.react:react-native:+" // From node_modules
 }

This issue body was partially generated by patch-package.

It worked for me :)
Thanks @racquelroque04

This will help https://www.youtube.com/watch?v=1JhdbLl-Kdg


refactor fluentd-cloudwatch

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/assign @unguiculus

/ok-to-test

/ok-to-test

/ok-to-test

@unguiculus , the reason of failure is that the docker image of fluentd in testing machine is not update to date.
I0514 12:28:35.914]   9m		7m		5	kubelet, gke-jenkins-default-pool-51cafeba-czr7	spec.containers{fluentd-9762-1-fluentd-cloudwatch}	Normal		Pulled			Container image "fluent/fluentd-kubernetes-daemonset:v0.12.33-cloudwatch" already present on machine

Its SHA is sha256:0a6763c174ac9456ae3b71ae4485ff5f9ab7ecd5a1542e71248a72c54666f02c, while the latest docker image with the same tag is 9c467e632224d5060218362d5efc585477870272ebb803fe2341d3261d65c58d. I run the test in my local machine, the test can be passed.  So what's your idea? Cleaning the docker images in test machine or Just changing pull policy to Always?

It looks like it is due to configmap mounts being readonly from Kubernetes 1.9.4. What's you cluster version?

My cluster version is 1.9.3.

/lgtm


Changing the endpoint name

PR into Azure/mlops-v2
Checklist
I have:

[ ] read and followed the contributing guidelines

Changes

fixes #

already fixed in another PR


Brain Tumor Classification by ANN

💥 Proposal
to make a model that can classify the tumor into its respective category based on image
the dataset i am gonna use is:- https://www.kaggle.com/datasets/sartajbhuvaji/brain-tumor-classification-mri/code
@khusheekapoor
I am GSSOC'23 contributor
i would like to do this project
so kindly assign this to me

to make a model that can classify the tumor into its respective category based on image
the dataset i am gonna use is:- https://www.kaggle.com/datasets/sartajbhuvaji/brain-tumor-classification-mri/code
@khusheekapoor
I am GSSOC'23 contributor
i would like to do this project
so kindly assign this to me

@lcs2022026  - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.


Refactor the file layout

Right now CSV.pm (maybe it should be pm6 by now, or even rakumod) includes everything. And I mean everything. RangeSet, IO::String (?)

this should probably get a module all by itself

CVS::Row... Lots of things. It's not a big deal, but it might make stuff easier to document, handle and, well refactor.

I like it in the same file, but this specific class could maybe do with its own file


How to setup?

I'm not able to set it up right... I'm not seeing the game editor, only the grid.
I've followed the instructions on the wiki page. The only thing I didn't quite understand was "Add the prefab test tiles or your own files to the Tiles array". What do you mean by that?
Also, on play, I notice an error:
NullReferenceException: Object reference not set to an instance of an object
LevelEditor.SetupPrefabsButtons () (at Assets/2DTileMapLevelEditor/Scripts/LevelEditor.cs:384)
LevelEditor.SetupUI () (at Assets/2DTileMapLevelEditor/Scripts/LevelEditor.cs:224)
LevelEditor.Start () (at Assets/2DTileMapLevelEditor/Scripts/LevelEditor.cs:183)

Any chance you could help me with this issue?

The idea of the level editor is that the user can drag and drop their own tiles into the grid.
These tiles should be prefabs created by the game developer (in this case probably you ;)).
There is a Tiles array in the LevelEditor prefab, which could be empty on start:

But that should be filled with tiles (prefabs) to build the level (e.g. the test tiles):

This should resolve the error, if not let me know.

Thanks for your reply! While it does work right now, I'm having trouble aligning it properly. I can't seem to find a good aspect. The select button is either invisible half visible. Any recommendations? Simply choosing 16:9 (example iPhone 5 Wide 16:9) doesn't make the editor visible.

Have you considered changing the width of the elements using the LevelEditor object?

I do not know the dimensions of your prefabs, but that might fix the problem :)

I'm using the included test prefabs to test it. Selecting a prefab and clicking the grid to place things works now, but I see no toolbar on the left...

Any errors in the log?
Are the demo scenes working correctly?
I use the editor myself for multiple project and have not yet encountered this 😅

Let me know if the error is resolved please, then I will close this issue :)

Tried it with a new project: it works!
Still can't figure out how it works though... I'm able to build a level, save it, etc., but the moment I "stop" the game, the level disappears. Or is this normal behavior? Is there a way to show the level in the scene?

That is indeed the normal behaviour. The levels should be saved as files which can then be used by your game in combination with the level editor. As described in #4:
I use the Level Editor methods LoadLevelUsingPath or LoadLevelFromStringLayers to load the level and then ToggleLevelEditor(false) to disable the Level Editor. You could also remove the whole component of course (destroy(game object) after loading.
Hope that helps, else let me know and I'll try to make it work in another way.


Show aliases as part of the url builder

This is a first pass at showing the aliases on the url builder page, it could do with more ui/ux thought going into it.

Does alias over here also mean dependencies?
Like will requestAnimationFrame have Date.now?

@azizhk alias does not include dependencies for polyfills, alias is this json file --><EMAIL_ADDRESS>Did you want it to show dependencies of polyfills as well?

Did you want it to show dependencies of polyfills as well?

Well not my end use case per se.
My use case is that if I check default then Array.prototype.filter should automatically get checked. Like checked by default, which cannot be unchecked. Which on hover can say "Included as you have selected default bundle.
I know maybe too much trouble to implement 😅 . Can ignore.

@azizhk That is a good feature request, could you open a new issue for it please? :-)


Edit install instructions to reflect that sudo is not required

@rschlussel do you have any other comments?


CMakeList Warnings - TOT Master

------------------------------- ROCMChecks WARNING --------------------------
Options and properties should be set on a cmake target where possible. The
variable 'CMAKE_CXX_COMPILER' may be set by the cmake toolchain, either by
calling 'cmake -DCMAKE_CXX_COMPILER="/opt/rocm/bin/hipcc"'
or set in a toolchain file and added with
'cmake -DCMAKE_TOOLCHAIN_FILE='. ROCMChecks now calling:
CMake Warning at /home/kiriti/develop/docker/build/extern/rocm-cmake/share/rocm/cmake/ROCMChecks.cmake:46 (message):
'CMAKE_CXX_COMPILER' is set at
/home/kiriti/develop/docker/MIVisionX/amd_openvx/openvx/hipvx/CMakeLists.txt:<line#>
shown below:
Call Stack (most recent call first):
amd_openvx/openvx/hipvx/CMakeLists.txt:9999 (rocm_check_toolchain_var)
amd_openvx/openvx/hipvx/CMakeLists.txt:36 (set)
-----------------------------------------------------------------------------

PR #833 fixes this issue. @paveltc can you verify this?

-- Found HIP: /opt/rocm/hip version: 5.1.20531-cacfa990
-- AMD OpenVX -- OpenVX built with HIP Support
-- AMD OpenVX -- OpenVX module added
-- hip::amdhip64 is SHARED_LIBRARY
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29.1")
-- Checking for module 'libavcodec'
--   Found libavcodec, version 58.54.100
-- Checking for module 'libavformat'
--   Found libavformat, version 58.29.100
-- Checking for module 'libavutil'
--   Found libavutil, version 56.31.100
-- Using FFMPEG -- Libraries:/usr/lib/x86_64-linux-gnu/libavcodec.so;/usr/lib/x86_64-linux-gnu/libavformat.so;/usr/lib/x86_64-linux-gnu/libavutil.so;/usr/lib/x86_64-linux-gnu/libswscale.so Includes:/usr/include/x86_64-linux-gnu
-- amd_media -- Building with HIP
-- AMD OpenVX Media Extension -- amd_media module added
-- Found Threads: TRUE
-- hip::amdhip64 is SHARED_LIBRARY
-- hip::amdhip64 is SHARED_LIBRARY
-- Found Protobuf: /usr/local/lib/libprotobuf.so;-lpthread (found version "3.12.0")
-- Inference Generator -- utilities/inference_generator module added
-- amd_nn -- Building with HIP
-- AMD OpenVX Neural Network Extension -- amd_nn module added with HIP backend
-- Found AMDRPP: /opt/rocm/rpp/lib/libamd_rpp.so
-- amd_rpp -- Building with HIP
-- AMD Radeon Performance Primitives -- amd_rpp module added with the HIP backend support
-- OpenCV Found -- Version-4.5.X Supported
-- AMD OpenVX OpenCV Extension -- amd_opencv module added
-- hip::amdhip64 is SHARED_LIBRARY
-- hip::amdhip64 is SHARED_LIBRARY
-- vx_amd_migraphx -- Building with HIP backend
-- AMD OpenVX MIGraphX Extension -- amd_migraphx module added with the HIP backend
-- Found HIP: /opt/rocm/hip version: 5.1.20531-cacfa990
-- runvx built with HIP Support
-- Found HIP: /opt/rocm/hip version: 5.1.20531-cacfa990
-- Utilities -- runvx and mv_deploy modules added
-- Found HIP for rocAL: 5.1.20531-cacfa990/opt/rocm/hip
-- ROCm Augmentation Library -- rocAL module added
-- rocAL library is going to be built with HIP support
-- rocAL built with ENABLE_SIMD
-- rocAL - CMAKE_CXX_FLAGS: -O3 -O3 -fopenmp -msse4.2 -mavx2 -Wall  -fPIC -pg -pthread -std=gnu++14 -Wno-deprecated-declarations
-- rocAL library is going to be built with video decode functionality
-- Configuring done
-- Generating done
-- Build files have been written to: /home/kiriti/develop/cmakelist/build


Redesign donations page

redesign has been sitting around in the figma for a while now so thought i'd get to it


Add a CompositeTile source

I want to composite a tile image (served by OSM, for example) with a floating-point data tile, so that I can access both at the same time via the band operator of WebGL style expressions.
This requires multiple textures to be available within a shader at the same time. However, it seems that ol/DataTile and ol/webgl/TileTexture currently don't support uploading multiple data types at once.
This PR adds support for this functionality.
Additionally, it introduces a new virtual utility source, ol/source/CompositeTile.
new WebGLTileLayer({
  source: new CompositeTile({
    sources: [new OSM(), new SomeOtherSource()],
  }),
}),

This allows tile sources to be combined as if they were a single source, making the data equally accessible via the band operator.

It looks like you intend to accept reproj tiles in the composite instead of reprojecting composite sources?  In that case should the composite projection be the view projection and the tilegrid be sources[0].getTileGridForProjection(viewProjection)?  Note there may be conflicts with changes proposed in #15860 and #15988.

Hi @mike-000, thank you for your comment!
Ah that's right, intended to accept a reprojected tile as an image.
Since the CompositeTileSource just hooks and proxies on getTile function, is it possible to set projection and tileGrid to simply null maybe?
Oh, and I'll take a look at these PRs.
Thanks.

Passing the projection from the renderer's getTile request onto the component sources is correct.  Projection and tilegrid should be set null in the composite source setup.  The renderer will call the composite source's getTileGridForProjection method so an override may be needed for that to return the result from a component source (and as in GeoTIFF if two component sources have incompatible grids throw an error?).

Indeed, thank you!
So, how about this way?
It collects and redistributes projection and tileGrid mappings.

Be aware that currently if the requested projection is equivalent to source.getProjection() getTileGridForProjection() returns source.getTileGrid() (if set) regardless of what was set for the projection using setTileGridForProjection().
#15988 would change that for a DataTile source if a transformMatrix is set - and a [1, 0, 0, 1, 0, 0] matrix (or some other override) could be used to "reproject" to a new grid with no rotating or flipping (following on from #15988 I would like to see a similar change for other raster tile sources as there is currently no way to use TileDebug for rotated grids).

Yeah, I see. Thank you.
I'll keep an eye on that.

layer.getData() now returns both the extended data and the color data of the pixel, even for canvas layers.
https://deploy-preview-16090--ol-site.netlify.app/en/latest/examples/composite-tile-source-pmtiles-elevation.html
Is this somewhat useful...? 🤔

In the example it looks like the OSM is not interpolated.  Is it possible to handle the different interpolate settings of the original sources?


Drop Tensorflow Addons support (TF>2.14)

Hi @faustomorales,
first of all, thanks for this awesome package and your work!
Bug: Tensorflow Addons (TFA) dropped maintenance with Keras update 2->3.
Solution: All necessary functions of TFA are available in Keras 3.
Affected positions: Activation layer GELU.
Pull Request: Remove TFA import in layers.py and use Keras 3 GELU.
Side effect:
Tensorflow versions utilizing < Keras 3 (all versions bellow 2.15?) are not supported anymore.
Positive effect:
All up-to-date Tensorflow versions >=2.15 are functional now, again.
Fixes issues #44 and #43
Cheers,
Dominik

Sidenote:
Another interesting implementation maybe would be to move the TFA import into the else section prior to the commit, so it only imports TFA if it is not available in the current Keras version.
Don't know if this works with the Python interpreter but maybe this will allow compatibility also with older Tensorflow/Keras versions.


Syslog appender intermittently losing messages from a Dropwizard command

I have two different DropWizard components, currently using DW 2.0.16. One component is a fairly typical HTTP service. The other is just a CLI application, implementing a handful of DW commands. Both are configured to use the syslog appender to send certain messages to an Rsyslog service (running in CentOS 8.2)
The messages generated from the HTTP service are all arriving at the Rsyslog service as expected, and getting handled correctly. But the messages generated from the CLI commands are intermittently not arriving. Sometimes about half of them arrive and get logged, sometimes less than that.
My guess is that DropWizard's use of an async appender is causing the log messages to be queued for a worker thread to eventually pick up, but that thread is failing to pick up all the messages because the command completes and exits quickly. But that is just a theory that is consistent with the limited data I have.
Looking for ideas to find out of this is a DropWizard bug, or a misconfiguration on my part. And most importantly, I am eager to find a workaround.

Additional note - In order to prove to myself that this was not a case of UDP packets getting lost, I used tcpdump to monitor the UDP traffic on my node. As I suspected, the cases where syslog messages were missing showed no UDP traffic hitting the network layer.
So this really looks to be something happening on the Java side, either in Dropwizard or the underlying Logback implementation.

As a workaround, I have found that adding this to my command code before exiting seems to fix things up:
LoggingUtil.getLoggerContext().stop();

If it's something related to #3677, we probably should reset all appenders before exiting. Therefore we could extend ConfiguredBundle.cleanup() to call the reset() method of the LoggingFactory.

Actually DefaultLoggingFactory.reset() seems to do too much. Therefore manually stopping the logger context and appenders should be faster on command cleanup.

As a workaround, I have found that adding this to my command code before exiting seems to fix things up:
LoggingUtil.getLoggerContext().stop();

After revisiting this, I think this should be done manually. Users are permitted to use logging throughout the whole application, i.e. they can override Application#onFatalError and use a logger in it. Therefore we can't generally stop the LoggerContext at any point in the framework.
However, we could improve the handling on asynchronous cleanup in a ConfiguredCommand and explicitly flush all appenders, if a configuration is provided. See #5094

@TimothyMDean Could you please verify, if the problem persists? As we have another appender flush on application shutdown, this may be enough to get your logs. Then you don't have to disable logging completely.

@zUniQueX Unfortunately I can no longer verify this. I have changed employers since reporting the issue, and I know longer have access to the source code.

@TimothyMDean Thanks for the fast reply. I'll then close this for now.


fails to render after fresh install

Hey
We're using flameTimeWarpML (0.4.4 build 15) in a shared env. We run the installer from the shared folder, and all installs well but...
It pip's the latest release of Numpy (2.0) and 1.0 is required. Lucky for me, I have a few systems running, and working, so copied the /home/flame/flameTimewarpML/miniconda3 folder over to the non-working version, and voila... we render.
Also, I find the manual install confusing... preferably we do it manually in a shared env of course. If source and destination is specified more explicit (or explained) it might be easier....
OS: Rocky Linux 8.7
DKU: DKU_18.2.2
Flame: 2023.3.2, 2024.2.2
Thanks!

Use .0.4.5, its way easier.  You just drop the package in a central defined hooks folder.  No need to initialize all that stuff.

I've started using v0.4.5 dev 003 in production at work and though I stumble upon some little issues here and there to be fixed it generally gives me a better result, so I would say yes pleas try it and it likely to be an update soon in order to fix some minor issues


[Question] Error: unsupported osType centos specified

Is Centos7 not supported?

Nice! Thank you.


Added a test for the rake task dealing with the personal namespace migration

Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

LGTM, however the coverage needs to be 100%.

@monstermunchkin now it's passing. It's a bit of a pity, but testing a rake task means that we suddenly have to provide tests for all rake tasks, which is something that we don't want to do. Therefore, I just wrapped the tested code in :nocov: blocks. Moreover, I've also moved this code in a special rake file where it makes more sense.


useSuspenseCondition

This is for long-running tasks that don't resolve from a simple fetch. Endpoint is subscribed (meaning they can use polling or other methods like websockets); then on updates - they check a conditional (function set in endpoint) and if does not pass, throws a promise for next polling frequency.
This will be used for workflow result stream. By using suspense in react 18 the entire flow is very simple - useTransition can simply be used on the workflow button to grab next results.
const WFR = endpoint.extend({ condition: (response) => response.status === 'done' })

const workflowResults = useSuspenseCondition(WFR, { id })

const [isPending, startTransition] = useTransition();
const handleRun = useCallback(

Closing in favor of discussions for new ideas: https://github.com/data-client/rest-hooks/discussions/2400