request for pixelsnail pretrained .pt files

Hi @vvvm23,
I saw your code today, it is novel and inspiring an I benefited a lot from it.
I'm sorry to bother you, but could you upload the .pt file you learned with pixel snail?
Since I want to test the decoder alone, so I followed your very kind instructions.
but it takes a very long time because my GPU performance is not enough. :sweat_smile:
Thanks for your time!

You don't necessarily need the pixelCNN pretrained to test the decoder, however you will need it to produce the best results(otherwise they will be blurry), because in my opinion which might not be right, the main target to insert pixelCNN is to add more variety to the generated results. The biggest difference between VQ-VAE and vanilla VAE is that the pixelCNN used here is designed to fit a prior  distribution of the codebook which is low-dimention representation of the input, whereas the vanilla VAE simply presume the prior is Gaussian, both these priors are used to control the generated results.  However, training the autoregressive model like pixelCNN takes enormous  computation cost, thus the VQ-VAE combine with codebook and pixelCNN together to make this whole thing feasible, and did improve the image quality a lot.
So, assumptions here, if your task is GENERATION which takes variety and image quality as the targets, then you will need the pretrained pixelCNN, but if your task is something like RECONSTRUCTION or DENOISING, etc. then you can just directly inference but you have to designate the inputs.


fix(theme): Improve editor tooltip legibility in Light Steel Blue theme

PR Details
Improve editor tooltip legibility in Light Steel Blue theme
- Updated the Background property to use a dynamic resource for the background color.
- Updated the BorderBrush property to use a dynamic resource for the border color.
Description
The Light Steel Blue theme had issues with legibility in editor tooltip, where it used black font on an almost-black background. This change addresses this problem by updating the theme style.
Related Issue
Fixes: #1191
Motivation and Context
Improve Editor UI UX

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Looks good, thanks.


Introduction to Javascript

Introduction to JavaScript
What is JavaScript?

Explored the concept of JavaScript

I gone through the introduction of JavaScript

I gone through the  JavaScript basics


As a Partner I want to ensure the portal is visible ..so that I ensure it is used by those who need it so it becomes an asset and selling point for scheme participation

original id#91
This is more in line with Objectives. Needs much more detail on how the portal will be visible (e.g. SEO, promotion on ICES site, promotion on partner sites...

We think this might be part of the reason to accommodate the dataset in ICES and linking it with a group of experts

@LynnHeeley this is more of an objective/mission statement, suggest to reject as user story

@LynnHeeley this is more of an objective/mission statement, suggest to reject as user story

Steering Group Jan2020 agreed:
Objective, not user story.


logs-logs-logs: Some details potentially missing in hints.md and introduction.md

I am not sure whether these are by design (i.e. we want the user to search the Go docs) or just an oversight, but after going through the exercise, I feel that:

in hint #2 (hints.md), it should be mentioned that a rune must be first converted before it is concatenated with a string
in the very last exemple of introduction.md, there should be an import "unicode/utf8" (because utf8.RuneCountInString() is used)

Thanks for bringing this to our attention!

Indeed, the hints should mention how to convert a rune to a string, since the solution will very likely use it. Looking at the whole file, the hints for task 3 could probably also get more love and be me more complete.
I agree. I can see how someone new to Go can be confused by seeing the utf8 being used and seemingly coming from nowhere.


patch: Unblue Getting Started generic screenshots

After a quick feedback from the team, I feel like generic device type screenshots for the Getting Started guides might be more helpful than having blurred screenshots.
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!

@balena-ci rebase


Is there any way to call static function of game ?

Hello, guys.
First of all, thank you all who has contributed to this great mod.
Anyway, while I was implementing a mod in lua, I got a question.
What I tried to do is:
staticTweakDBInterface = Game.TweakDBInterface
staticItemIDInstance = Game.ItemID

if staticTweakDBInterface ~= nil and staticItemIDInstance ~= nil then
    local weaponRecord = staticTweakDBInterface:GetWeaponItemRecord(staticItemIDInstance:GetTDBID(weapon:GetItemID()))

    print(tostring(weaponRecord:ItemType():Type()))
    ...
end

Every time I call static function of static instance, I got an error below:
attempt to index a sol.Scripting::InternalIndex::<lambda_1> value (global 'staticTweakDBInterface')
stack traceback:

So I'm wondering that even though accessing to static instance was fine, why I can't call static function of it.
If there is possible way to do so, please let me know.
Thank you.

Courtesy of @psiberx (is on our Discord server in cet-tips channel, should be added to wiki sometime I hope too)

Static function calls

Static native function:
Game.StatsEffectsEnumToTDBID(1)

Static scripted function:
Game['GetMountedVehicle;GameObject'](player)

Static scripted function with no args:
Game['Pi;']()

Static scripted class method:
Game['StatusEffectHelper::ApplyStatusEffect;GameObjectTweakDBIDFloat'](player, effectID, 0)

Also, this for some generic syntax (courtesy of @WopsS )

native - call like Game.func_name(parameters)
scripted - call like Game'[class_name::]func_name;parameter_types'

@WSSDude420
Oh my god.
Thank you so much for replying.
It would be so helpful for many people.
And thank you for making lua sandboxing.

Is in Readme, click on Discord image ;)

And generic syntax - i Fd up a bit, we are discussing it now :D
But those samples should be valid at least.

@WSSDude420
Found ! Thank you again for your help !

@enzo522 dunno how you are called on Discord now lol, wanted to ping you there (if you have joined in the meantime that is, kind of assumed you did... maybe you have different name there :P )
I updated the sample part for now with some better info, generic syntax hopefully will follow soon.
I will try to add this to wiki in the evening cause it should definitely be mentioned there too.
https://github.com/yamashi/CyberEngineTweaks/issues/471#issuecomment-775828087

@WSSDude420
I've just joined CP77 Modding Community !
Actually, the user name is "TheBs65422" without quotes which is same with my Nexus Mods account.
I've also just signed up Discord becaus of Cyber Engine Tweaks Discord though ;)
I'm surprised that there are so many people and so many informations.

You find more info and more people to ask about these on Discord currently unfortunately :/

Yeah, you are right.
I hope many people are able to edit Wiki page.
I think it would be better that some influencing people like you get authority to edit and manage Wiki page.
Then many people will find some fun with making mods. ;))

@WSSDude420
Although you helped me a lot, I still couldn't find the way to call some static functions ...
The example code you said seems to work:
[2021-02-09 21:19:02 UTC+09:00] [info] >  Game['StatusEffectHelper::ApplyStatusEffect;GameObjectTweakDBIDFloat'](Game.GetPlayer(), effectID, 0)
[2021-02-09 21:19:02 UTC+09:00] [error] Error: Function 'StatusEffectHelper::ApplyStatusEffect;GameObjectTweakDBIDFloat' parameter 1 must be TweakDBID.

If I pass second parameter properly, the function call would success I guess.
But here is my problem:
local Trainer = {
    isInfiniteAmmo = false
}

function Trainer.ToggleInfiniteAmmo()
    Trainer.isInfiniteAmmo = not Trainer.isInfiniteAmmo
end

function Trainer.OnUpdate()
    if Trainer.isInfiniteAmmo and Game ~= nil then
        local player = Game.GetPlayer()

        if player ~= nil then
            local activeWeapon = Trainer.GetActiveWeapon(player)

            if activeWeapon ~= nil then
                local sharedData = activeWeapon:GetSharedData()
                local blackboardDefs = Game.GetAllBlackboardDefs()

                if sharedData ~= nil and blackboardDefs ~= nil then
                    local magazineAmmoCount = sharedData:GetUint(blackboardDefs.Weapon.MagazineAmmoCount)
                    local magazineAmmoCapacity = sharedData:GetUint(blackboardDefs.Weapon.MagazineAmmoCapacity)

                    if magazineAmmoCount ~= nil and magazineAmmoCapacity ~= nil and magazineAmmoCount < magazineAmmoCapacity then
                        Trainer.RefillAmmo()
                    end
                end
            end
        end
    end
end

function Trainer.GetActiveWeapon(target)
    if target == nil then
        return nil
    else
        local transactionSystem = Game.GetTransactionSystem()

        if transactionSystem == nil then
            return nil
        end

        local weapon = transactionSystem:GetItemInSlot(target, TweakDBID.new("AttachmentSlots.WeaponRight"))

        if weapon == nil then
            weapon = transactionSystem:GetItemInSlot(target, TweakDBID.new("AttachmentSlots.WeaponLeft"))
        end

        return weapon
    end
end

function Trainer.RefillAmmo()
    Game.AddToInventory("Ammo.HandgunAmmo", 1000)
    Game.AddToInventory("Ammo.ShotgunAmmo", 1000)
    Game.AddToInventory("Ammo.RifleAmmo", 1000)
    Game.AddToInventory("Ammo.SniperRifleAmmo", 1000)
    Game.AddToInventory("Ammo.Special", 1000
end

The code above is the feature of Infinite Ammo I've made.
Simply automatically refill ammo when player is shooting.
But I think it would be better that "refill ammo of weapon which I shoot".

So I tried to use the static function of WeaponObject:
public final static func GetAmmoType(weapon: wref<WeaponObject>) -> ItemID {
    let weaponID: ItemID;
    let weaponRecord: ref<WeaponItem_Record>;
    let ammoID: ItemID;
    weaponID = weapon.GetItemID();
    weaponRecord = (TweakDBInterface.GetItemRecord(ItemID.GetTDBID(weaponID)) as WeaponItem_Record);
    ammoID = ItemID.CreateQuery(weaponRecord.Ammo().GetID());
    return ammoID;
}

The link is here: https://codeberg.org/adamsmasher/cyberpunk/src/branch/master/cyberpunk/items/weapon.swift#L294
As you mentioned, I wrote the code below in my mod:
function Trainer.OnUpdate()
    ...
        Trainer.RefillAmmo(activeWeapon)
    ...
end

function Trainer.RefillAmmo(weapon)
    if weapon ~= nil then
        local ammoType = Game['WeaponObject::GetAmmoType;ItemID'](weapon:GetItemID()
    end
end

But still got an error:
[2021-02-09 21:04:20 UTC+09:00] [error] stack index -1, expected userdata, received nil: value is not a userdata
stack traceback:
	[C]: in field 'WeaponObject::GetAmmoType;WeaponObject'
	...ugins\cyber_engine_tweaks\mods\SimpleTrainer\Trainer.lua:515: in field 'RefillAmmo'
	...ugins\cyber_engine_tweaks\mods\SimpleTrainer\Trainer.lua:345: in field 'OnUpdate'
	...\plugins\cyber_engine_tweaks\mods\SimpleTrainer\init.lua:33: in function <...\plugins\cyber_engine_tweaks\mods\SimpleTrainer\init.lua:32>

If I did something wrong, please let me know :D

There is apparently one more dump...
Seems like there are aliases for some things and you have to use those.
https://cdn.discordapp.com/attachments/727800260486758410/794019913425092628/types.zip
Basically, classType in that generic syntax format have to be from dump above...
While paramTypes seem to follow adamsmasher repo I linked before.
Yeah, is kind of complex for "simple thing" :D :/
This seems to work after I inserted it into a bit modified code you posted
Game['gameweaponObject::GetAmmoType;WeaponObject'](weapon)

This also works (for the other overload, though looking at your code, I would prolly stick to first one)
Game['gameweaponObject::GetAmmoType;ItemID'](weapon:GetItemID())


path/filepath: Abs can give incorrect results on unix systems

filepath.Abs can give incorrect results on unix systems.  The reason for this is it calls filepath.Clean() which does not take into account symbolic links when handling ...  For example in the sample program below it is incorrect to transform dir2/dir1/../afile to dir2/afile as dir1 is a symbolic link.

What version of Go are you using (go version)?

go version go1.6.2 linux/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/kevina/gocode2"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?

Use filepath.Abs in combination with symbolic links and ..
Here is a sample program at https://play.golang.org/p/ZAhkresUaD, but note it won't run on the playground.  To reproduce the results it needs to be run on a Unix system.
The programs creates the following files/dirs/symlink
afile
dir1/
dir2/
dir2/dir1 -> ../dir1

And than uses the path dir2/dir1/../afile to read afile.  This correctly resolves when the path it relative or the absolute path is created manually, but fails when filepath.Abs() is used.
The path /somedir/dir2/dir1/../afile should resolve to /somedir/afile; however filepath.Abs() transforms that to the incorrect /somedir/dir2/afile.

What did you expect to see?

The sample program should run without error.

What did you see instead?

open /tmp/abstest062312103/dir2/afile: no such file or directory

This is working as intended, or at least as documented:
Abs returns an absolute representation of path. If the path is not absolute
it will be joined with the current working directory to turn it into an
absolute path. The absolute path name for a given file is not guaranteed to
be unique. Abs calls Clean on the result.

And Clean clearly states that:
Clean returns the shortest path name equivalent to path by purely lexical
processing.

The effect is that unusual symlinks can cause Abs to return non-working paths.
That's unfortunate, but it's unavoidable as documented.
Symlinks like these cause strange behavior in other programs too. See the transcripts at the beginning of https://9p.io/sys/doc/lexnames.html, for example.

I was afraid you were going to say that but I needed to be sure.
I have created my on version of Clean that does not remove ".." at https://github.com/kevina/safepath.


[mesheryctl] Error in all the mesheryctl commands specific to Apple Silicon M1 chip

See mesheryctl Command Tracker for current status of commands.
Current Behavior

When executing any mesheryctl command there is an error ERRO[0000] failure getting variant            error="getCPUInfo for OS darwin: not implemented" at the first line but after that the command performs its desired behaviour. This error is only showing on Apple Silicon (M1 chip).
Desired Behavior

There shouldn't be any error in any of the mesheryctl commands.
Screenshots/Logs

Environment

Host OS: Mac (Apple Silicon M1 chip)
Meshery Server Version: v0.5.38
Meshery Client Version: stable- v0.5.38
Platform: Docker

Implementation

Acceptance Tests

Mockups

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
Layer5 Community Slack

@sayantan1413 will you be working on this (self-assigning) or is this posted with the intention of others picking it up (to be labeled with "help wanted")?

No @leecalcote I want other contributor to pick up this issue and work on it.

@sayantan1413 Where you able to investigate what the root cause of the issue is?

@sayantan1413 Were you able to find which of our dependencies is causing this issue? If it is a small fix we can try to fix this.

@sayantan1413 Could you also confirm this is not an issue anymore?


typoの修正

after_delivery -> after_deliver
原著: https://guides.rubyonrails.org/action_mailer_basics.html#action-mailer-callbacks

You could use an after_deliver to record the delivery of the message. It also allows observer/interceptor-like behaviors, but with access to the full mailer context.

確かに! deliver 系のメソッドは動詞ですね...!! 修正 PR ありがとうございます! 🙏 ✨


Caption button in title bar Not clickable when ExtendContentintoTitlebar is given true

Describe the bug
The caption buttons are not clickable when the window is maximized. It works when the window is opened , it stops working after being maximized .
Steps to reproduce the bug
1.When activating a window set ExtendContentintoTitlebar  property to true then activate  the window
2. After the window is opened maximize the window the caption buttons are not clickable
Expected behavior
The caption button to work  even after maximizing the window
Screenshots
No response
NuGet package version
No response
Packaging type
Packaged (MSIX)
Windows version
Windows 10 version 21H2 (19044, November 2021 Update)
IDE
Visual Studio 2022
Additional context
Nuget version 1.01

Hello @vishal2447,
You need to check if title bar customization is supported before calling AppWindowTitleBar.ExtendsContentIntoTitleBar. The reason for that is because titlebar customization with WinAppSDK windowing APIs only works on Win11+. See more here.

Will these APIs be supported for Windows 10 in future?

m_window = new MainWindow();
m_window.Title = "Main Window";
m_window.ExtendsContentIntoTitleBar = true;
I m using this property for extending the content and not using the AppWindowTitleBar.ExtendsContentIntoTitleBar , Is this property also supported only in WIndows 11?

No, WinUI's Window.ExtendsContentIntoTitleBar works on both win10 and win11 and is a good alternative until we are able to bring AppWindowTitleBar API support further downlevel.

Yes ExtendsContentIntoTitleBar works on Windows 10. It was working correctly on WinAppSDK 1.0.0. After WinAppSDK 1.0.1 update Minimize, Maximize, Close buttons clicks but not working on Windows 10.

this should be fixed in both Window custom titlebar as well as AppWindow custom titlebar apis. Both are using the same implementation downlevel.


set.status isn't working

I have this endpoint definition that sets set.status to 400 and logs that it does so.
But the actual response status I see in Chrome DevTools is always 200.
Am I doing something wrong?
app.post('/account', ({body, set}: any) => {
  const {email, password} = body;
  const goodEmail = validEmail(email);
  const goodPassword = validPassword(password);
  const good = goodEmail && goodPassword;
  set.status = good ? 200 : 400;
  console.log('/account: set.status =', set.status);
  return <span>{good ? 'A new account was created.' : ''}</span>;
});

In my case the request is coming from the browser using htmx. Htmx takes that HTML response and inserts it into the DOM. It doesn't seem though that htmx has any impact on this because I see in the DevTools Network tab that a POST request is sent and the response status is 200 even though I log on the server that it is 400.

I just tried changing my endpoint to return a string instead of HTML.
That fixes the problem with the response status code.
So something about returning HTML using JSX seems to break the functionality of set.status.

@mvolkmann it looks like a bug on elysia-html package
I found a similar issue on their repository you can check there
https://github.com/elysiajs/elysia-html/issues/56

Oh I am experiencing the same issue. Is there a workaround?

Unable to reproduce in the latest Elysia version, closing as complete.
If the problem still persists, feel free to reopen the issue. Thanks.


Delay cache download until app finishes loading

Is there a way to install the app and use the cache if it exists when the user returns to the site but, at the same time, when the user accesses the site for the first time I want to download the cache only after the app bootstraps. Or at least when the app stops doing a lot of requests.
In my case, the install is configured on the app entry and I can see it starts downloading the resources before the app finishes bootstrapping. According to webpagetest it's actually delaying the bootstrap.

I was expecting the install to happen only on page load but when I checked the code I noticed this is not the case.
Closing the issue


Blind and Self rolls not working when making rolls with Mess on

I've discovered that by using Mess the blind and self roles don't work as intended when using certain functions, specifically skill/ability checks and saving throws made by clicking the corresponding sections in the character sheet. It's as if they were made as public rolls, everyone can see them.
I made sure only Mess was on when using the blind/self roles feature to check if it was the mod responsible, and when no mods and even all the other mods were installed blind and self rolls worked properly. Using features/items/spells seemed to work fine, it appears to only affect the checks made in the attributes tab, being as mentioned before skill/ability checks and saving throws.
Browser for player's used, Chrome. Foundry Version, 0.6.5 5th Edition Version, 0.94 Mess Version used, 0.11.2

This won't be fixed, only update for mess that will be coming (regarding rolling issues) is removing them.
My replacement is now submitted and waits for approval. Meanwhile you can download the module and look at its features from its repository:
https://github.com/Moerill/fvtt-mars-5e
Manifest: https://raw.githubusercontent.com/Moerill/fvtt-mars-5e/master/module.json
(I'm currently going through all rolling related issues, pasting this as information and closing them)


Fix nPick lines calculation

Well ... well ... I accidentally merged too many commits in another PR, this one has nothing left to merge. Closing.


Correct replacements for deprecated methods

Changelog lists wrong methods as replacements for deprecated methods.

Thanks for the quick merge! This is still incorrect on the release page. Maybe also fix it there? Confused me a lot. :)

Yep I'll correct them ASAP ... it will update on the documentation website on the next commit of the gh-pages branhc


add missing #include <net/ip6.hpp> into ncp_spi.cpp

LGTM 👍

Current coverage is 65.71% (diff: 100%)

Merging #543 into master will decrease coverage by 0.01%

@@             master       #543   diff @@
==========================================
  Files            84         84
  Lines         11843      11843
  Methods        1594       1594
  Messages          0          0
  Branches       1395       1395
==========================================
- Hits           7785       7783     -2
  Misses         3641       3641
- Partials        417        419     +2

Powered by Codecov. Last update 563685e...acdf6cc

👍


Support for phpspec 4.0

Hello again, another major phpspec release is just around the corner ;)

Hi @pamil !
Thank you very much for this pull request :heart_eyes:
We can follow the same approach we had for phpspec 3.x release:

I created a 2.0 branch from the master to maintain this extension 2.0.x supporting phpspec 3
The master is now the 3.0-dev branch to prepare the future 3.0.0 release supporting phpsec 4 (cf https://github.com/akeneo/PhpSpecSkipExampleExtension/pull/17)

You can rebase your PR on master, there is still a 7.0 issue, then we'll be able to merge and release a 3.0.0-alpha to test it with projects (i have a good use case for it, as Akeneo PIM master is now using php 7.1 :wink: )
Cheers.

Rebased and fixed compatibility with PHP 7.0 🎉

Hi @pamil !
Sorry, i missed the notification, thanks a bunch for this PR :tada:
I merge and prepare the release :dancer:


Enable 2sxc to change views based on standardized concepts in URL (like a details view in an app)

completed


How to get current model indent?

monaco-editor version: 0.17.1
Browser: Chrome
OS:  Window 10
Steps or JS usage snippet reproducing the issue:
I used to use getOneIndent line 2268. But now it's removed. What API instead it?

Does this help https://microsoft.github.io/monaco-editor/api/classes/monaco.editor.textmodelresolvedoptions.html? You can get these by calling editor.getModel().getOptions()

@spahnke Thank you. I'll implement it by myself.
	public getOneIndent(): string {
		this._assertNotDisposed();
		let tabSize = this._options.tabSize;
		let insertSpaces = this._options.insertSpaces;

		if (insertSpaces) {
			let result = '';
			for (let i = 0; i < tabSize; i++) {
				result += ' ';
			}
			return result;
		} else {
			return '\t';
		}
	}

https://github.com/microsoft/vscode/blob/24f62626b222e9a8313213fb64b10d741a326288/src/vs/editor/common/model/textModel.ts#L663


[SPM] swift-syntax conflicts

Everything is visible on the screenshot. Hope for a quick fix 🤞🏻 Regards

hey, if you are brining in Sourcery as framework dependency and using SwiftSyntax directly as well, it's on you to match the framework versions?
I'm not sure what you expect us to do here, we don't even know what your setup is

I see the apple org changed for SwiftSyntax, it's no longer https://github.com/apple/swift-syntax We can update that, you'll still need to match up versions if you are brining in Sourcery as framework dep I think

If you update this, it will be enough for me. Thank's


no start line error 'need info'🙌

<EMAIL_ADDRESS>added 17 packages from 12 contributors in 8.776s

4 packages are looking for funding
run npm fund for details
C:\Users\TOSHIBA>npm fund
C:\Users\TOSHIBA
+-- https://opencollective.com/core-js
| --<EMAIL_ADDRESS>+-- https://github.com/sponsors/ljharb | --<EMAIL_ADDRESS>+-- https://github.com/sponsors/isaacs
| --<EMAIL_ADDRESS>+-- https://github.com/sponsors/feross | --<EMAIL_ADDRESS>+-- https://www.patreon.com/feross
| --<EMAIL_ADDRESS>-- https://feross.org/support
`--<EMAIL_ADDRESS>

C:\Users\TOSHIBA>npm i jshint
npm WARN registry Unexpected warning for https://registry.npmjs.org/: Miscellaneous Warning ERR_OSSL_PEM_NO_START_LINE: error:0909006C:PEM routines:get_name:no start line
npm WARN registry Using stale data from https://registry.npmjs.org/ due to a request error during revalidation.
npm WARN saveError ENOENT: no such file or directory, open 'C:\Users\TOSHIBA\package.json'
npm WARN enoent ENOENT: no such file or directory, open 'C:\Users\TOSHIBA\package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN TOSHIBA No description
npm WARN TOSHIBA No repository field.
npm WARN TOSHIBA No README data
npm WARN TOSHIBA No license field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})


#22 my idea

I modified the OrderedSet (restrict operation) and add class OrderedSetAdapter
(Every element in OrderedSet consist of both their index and value, so OrderedSet can only interact with OrderedSet. And I write a OrderedSetAdapter base on OrderedSet to interact with Python set, and always return Python set)
and fix a comment mistake;
add other Python set method as much as possible;
change propery items, ,map to _items, _map

Tested and Passed on Python27 and Python35, by the way.

Because sometimes I want a data struct:
a Python set, but can record the order of its element ,so I found this OrderedSet .
The OrderedSet lacks some methods,however,it can't behave exactly like a Python set .
I can't replace the Python set with this class.
set1 = {1, 2}
set2 = {2, 3}
#some code...
set3 = set1.difference(set2) #{1}
#############################
set1 = {1, 2}
set2 = {2, 3}

set1 = OrderedSet(set1) #Old OrderedSet
set2 = OrderedSet(set2)
#some code...
set3 = set1.difference(set2) #raise AttributeError: 'OrderedSet' object has no attribute 'difference'

As the resaon of above,I add the other methods which Python set has but OrderedSet doesn't.
Since ordered set is different from ordinary set after all,I use the OrderedSetAdapter (subclass of OrderedSet) to interact with Python set and itself.
set1 = OrderedSetAdapter([1, 2, 3])
set2 = OrderedSetAdapter([3, 2, 1])

set1 & set2 #{1, 2, 3}, OrderedSetAdapter|set op OrderedSet|set => set
#######################################################################
set1 = OrderedSet([1, 2, 3]) #New OrderedSet
set2 = OrderedSet([3, 2, 1])
set3 = OrderedSet([1, 3, 2])

set1 & set2 #OrderedSet([]) OrderedSet op OrderedSet => OrderedSet
set1 & set3 #OrderedSet([1])

Using OrderedSet to interact with itself.Both of two ordered set data struct behaved more exactly like Python set.EXTRA COMPLEXITY IN CODE FOR MORE COMPATIBILITY AND EASE IN USE.

There already is a class you can pass an OrderedSet into to perform set operations on it that disregard the order. That class is set. OrderedSets are iterable, so they can be passed as the argument to set.
I don't see the use case of OrderedSetAdapter, and by breaking compatibility, this PR would interfere with many existing uses of OrderedSet.
Again, please feel free to fork OrderedSet or create your own version if you want it to do things differently.


Menu - Ability to define content attachment

Feature Request
Description
Like the popup, it would be nice to choose where the menu is attached, at least left or right. Since we aren't using the arrow, it would be nice if all the text could align (as closely as possible) on the left.
Additional Context / Screenshots
current:

desired:

I've discussed this and potential work arounds with @neilpfeiffer

We're open to this. Need to think through how we want to open this API.


[v19.x] Backport 19.3 candidates, v19.3.0 release notes and version bump

Issue being fixed or feature implemented
backporting backport-candidate-19.x to v19.x
What was done?
How Has This Been Tested?
in progress...
Breaking Changes
Should be none but there were a couple of merge conflicts, pls test
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Reindexed with --assumevalid=0, no issues.
Gitian hashes
175c82c11a8db623fbdba5056966ef44e412a3dee08982dc250e12ba80ce8f40  dashcore-1c81bddf1c02-aarch64-linux-gnu-debug.tar.gz
f4eeca697f8aa76297a8e5396c435bb714da178322f80ec82cca7454eff52cb7  dashcore-1c81bddf1c02-aarch64-linux-gnu.tar.gz
625027cf0599a5893608d1ffa3c0a656919c405f41f9c12cc822bf643fe65554  dashcore-1c81bddf1c02-osx64-debug.tar.gz
5dc48867554b7054c86aa1eea06ef2929822b742f2092806a281c8289bdfa907  dashcore-1c81bddf1c02-osx64.tar.gz
84d94208e833f3ee705063ed31993805181b5077cfecff00381cd5160232ef24  dashcore-1c81bddf1c02-osx-unsigned.dmg
611c67a8bdcc784884ed88d4fae1e48158c7628c42b3393e89785f208e15e5be  dashcore-1c81bddf1c02-riscv64-linux-gnu-debug.tar.gz
4e87cca67b16e51fa243a83b11353f27e15d14e89a1c4962e3db67cb811b0708  dashcore-1c81bddf1c02-riscv64-linux-gnu.tar.gz
461b70b91879e95baf6d4e89652dff33bd7cd1530c4c291b970dadc289017184  dashcore-1c81bddf1c02.tar.gz
0f6745d3c4f1cf3f8e4e9dacb3507e060b95b68b1bd05845e7736ab6907cc35f  dashcore-1c81bddf1c02-win64-debug.zip
b85c176bb50f816b750d9e2c0efb7fd89196702079c06220cf61188cc12e514f  dashcore-1c81bddf1c02-win64-setup-unsigned.exe
e5a08b39ac802e300b162e1a3a11dc7c31e4ce797cea1071e43c815aa5fb88d0  dashcore-1c81bddf1c02-win64.zip
c2a3154996c4d059b0443e534025f08d6d3679cb02450f929a097766bc022998  dashcore-1c81bddf1c02-x86_64-linux-gnu-debug.tar.gz
b2d8a3d25bb959478351fb504a867d86680830a069501e91d788f479e68c8d30  dashcore-1c81bddf1c02-x86_64-linux-gnu.tar.gz


JSX Syntax

Have you considered supporting JSX syntax? I was thinking of making a babel plugin, but I'm curious if you've either gone down that road already or decided against it.

I gave a quick go at implementing JSX using the @babel/plugin-transform-react-jsx plugin and a bit of webpack configuration. Ideally this would all be done in a babel, but for now you can check it out at https://github.com/mattjennings/replay-with-jsx

Thanks for bringing this up, the plugin looks awesome!
My initial plan was to support JSX and I did some prototyping around it. However I found the greatest benefit JSX brings is more readable nesting - and since Replay doesn’t make use of the nesting, the extra build step (e.g. your Babel plugin) didn’t make it worth using.
I think if you prefer JSX it’s great you have options like your plugin, but I kind of prefer the existing way to just use functions and avoid the build step.

That makes sense! I did notice that all of the JSX I wrote was at a single level, so it really does end up being syntactic sugar. I'm happy to maintain the plugin, though!


feat(workflows): clean up dispatching syntax

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1079  👈
#1078
main

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

Join @MasterPtato and the rest of your teammates on  Graphite


Add custom argument representation handling

Add the ability for types to control how they are sent when used as arguments to redis commands.
This is useful for types which don't want to use their Stringer representation when used as an argument to a redis command.
Examples of this is a time.Duration which the user wants to store the number of seconds or a time.Time being stored as int instead of a string.

:+1: looks sane to me


Can't set refresh_interval to default value (null)

NEST/Elasticsearch.Net version:
6.3.0
Elasticsearch version:
6.4.0
Description of the problem including expected versus actual behavior:
refresh_interval setting cannot be changed to default (null) because null values are not sent.
I also don't see an option to just send null's for a single request.
Expected: refresh_interval set to null
Actual: refresh_interval doesn't update to null
Steps to reproduce:

Create Index using CreateIndexAsync
Update Index settings to set refresh_interval to -1

UpdateIndexSettingsDescriptor GetUpdateDescriptor(Time refreshInterval)
{
   return new UpdateIndexSettingsDescriptor()
      .Index("my-index")
      .IndexSettings(s => s
         .RefreshInterval(refreshInterval)
      );
}

await _client.UpdateIndexSettingsAsync(GetUpdateDescriptor(-1));

Reset refresh_interval to default by providing null

await _client.UpdateIndexSettingsAsync(GetUpdateDescriptor(null));

Retrieve settings of Index, refresh_interval is still -1, GET my-index/_settings returns:

{
  "my-index": {
    "settings": {
      "index": {
        "refresh_interval": "-1"
      }
    }
  }
}

Provide ConnectionSettings (if relevant):
new ConnectionSettings(node).DefaultTypeName("_doc")
Provide DebugInformation (if relevant):
N/A

This is indeed something we do not support now. We'll look at adding support for this in 6.4.0.

I've opened #3465 to address.


Cass County MN

I'm wondering if this will work even though the split tag is deprecated. If not, I would love to use this example to learn how a regex would apply in this case.

Yes the split tag is deprecated but will still work as expected.  Looks like you have a bad json file in your PR. Do you mind removing -  sources/us-mn-cass.json

Whoops, killed that errant one.


Updated features.astro with hover effects on icons

Updated features.astro file with hover effects on the icons.
These icons will now scale their size, when hovered over.
Thus, solving issue #13
Please Accept This Contribution for GSSoC'23.

@AnshSinghSonkhia Hey could you please provide screenshot of changes


Use cases for Respect\Cli

Hey guys! Let's sketch up some use cases to get a better idea of what we're going to code. Yesterday I was drunk thinking on these main scenarios:

Automation: Users of Respect components would like to automate tasks to use on cron jobs or even manually. Using the same PHP codebase and the same INI configuration files without having to code coupled Cli logic would enable them to do so effortlessly.

Daemonization: Bitches like daemons. We could use PEAR_System_Daemon or ressurrect Respect\Daemon (resting in peace at https://github.com/alganet/Daemon_deprecated).

Debugging: instead of spreading out var_dump()s all around the place an user could interact with the application components by starting an interactive session inside the code dependencies. Maybe we could even save/restore debug sessions freezing out components.

Please, send all your love to this issue.

Pinging @augustohp and @henriquemoody to check out this.


DO NOT MERGE - Verifying Windows daemon CI

Signed-off-by: John Howard<EMAIL_ADDRESS>Verifying Windows CI picks up a compile error in the Windows daemon. Do not merge this PR obviously!

@jfrazelle @tianon

CI fixed :)


create a ground truth options file

working towards #1379
the objective of this and forthcoming PRs is to move all options out of environments, for the following reasons:

philosophically, options live in the agent, not in the environment*
practically, the current setup is starting to cause problems in our pybullet environments.**

*for this reason, we may also want to remove env.predicates in the future too, but this is lower priority because it's not currently bothering me 🙂
**our pybullet environments currently create two robots: one that is meant to be "in the environment" and one that is meant to be "in simulation" (in the agent). the latter is used for kinematics checks during planning. but now that we're working on adding more sophisticated motion planning, we would need not just two robots, but two of everything. adding two of everything would be really gross. so what we will do instead is force the agent to construct its own pybullet simulation inside of the ground truth options. and it will be up to the agent to sync up pybullet to the current state. if all goes well, we may even be able to remove state setting from the pybullet environment.
I'm opening this PR to get feedback before going forward. once we converge, there will be a million changes to make in tests/ to get rid of all of the env.options. at the moment, I've just removed most of the env.options in predicators/.

Interesting - I understand and like the rationale for this proposal! It occurs to me that we currently have a long and somewhat-complicated ground_truth_nsrts.py file floating around as well. Is the idea that our new ground_truth_options.py file will basically look like this (that's what it seems to me from this PR)? Maybe we can think about combining these two or splitting them up into environment-specific files or something just to make things easier to grok?

@NishanthJKumar thanks for reviewing! yeah, the idea was that this would mirror ground_truth_nsrts.py.

Maybe we can think about combining these two or splitting them up into environment-specific files or something just to make things easier to grok?

that's a good idea. how about the following structure?
ground_truth_models/
    __init__.py    # implements get_gt_nsrts(), get_gt_options, and shared utilities
    blocks/
        nsrts.py
        options.py
    cover/
        nsrts.py
        options.py
    ...

This is great! I think this new organization would make things much cleaner!

Makes a lot of sense!
One question: why have the options be in a separate file but not have the samplers be in a separate file, especially when the samplers can be similarly complex? I wonder, especially as we do more sampler learning as part of the "learn options/samplers without demos project", if it would help to also have a standalone file for the samplers, e.g.
cover/
        nsrts.py
        samplers.py
        options.py

Just putting it out there.

this PR now does the following:

creates ground_truth_models/ as discussed above
creates get_gt_options
moves the NSRTs and options for the basic cover environments
fixes all the imports and tests to remove env.options

future PRs will move the NSRTs and options over for the other environments and eventually the options property in BaseEnv will be removed completely


Update landing page

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit a4c435a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/index.yml
:warning:Warning
View (qsharp-preview)
Details

articles/index.yml

Line 0: [Warning] Invalid file link:(~/overview/index.md).
Line 0: [Warning] Invalid file link:(~/intro-to-katas.md).
Line 0: [Warning] Invalid file link:(~/contribute/index.md).

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 70f5b70:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/index.yml
:warning:Warning
View (qsharp-preview)
Details

articles/index.yml

Line 0: [Warning] Invalid file link:(~/overview/index.md).
Line 0: [Warning] Invalid file link:(~/intro-to-katas.md).

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 7a975a7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/index.yml
:warning:Warning
View (qsharp-preview)
Details

articles/index.yml

Line 0: [Warning] Invalid file link:(~/overview/index.md).
Line 0: [Warning] Invalid file link:(~/intro-to-katas.md).

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 1bf4fb8:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

articles/index.yml
:x:Error

Details

articles/index.yml

[Error]

Unable to load file 'index.yml' via processor 'Landing': Schema validation failed. Please validate the file and make sure it conforms to schema 'Landing' (https://static.docs.com/ui/latest/schemas/Landing.schema.json):
Required properties are missing from object: url. Path 'landingContent[4].linkLists[1].links[0]'.
Required properties are missing from object: text. Path 'landingContent[4].linkLists[1].links[1]'.

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 1e63a2a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/index.yml
:warning:Warning
View (qsharp-preview)
Details

articles/index.yml

Line 0: [Warning] Invalid file link:(~/overview/index.md).
Line 0: [Warning] Invalid file link:(~/intro-to-katas.md).

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit ee35e0a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/index.yml
:warning:Warning
View (qsharp-preview)
Details

articles/index.yml

Line 0: [Warning] Invalid file link:(~/overview/index.md).
Line 0: [Warning] Invalid file link:(~/intro-to-katas.md).
Line 0: [Warning] Invalid file link:(~/aka.ms/try-quantum-katas).

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 8ddc129:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.yml
:white_check_mark:Succeeded
View (qsharp-preview)

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/index.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit 56eae09:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.yml
:white_check_mark:Succeeded
View (qsharp-preview)

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

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
AcrolinxScore
LowestCategoryScore
Issues
Scorecard
Processed

articles/TOC.yml
100
100
0
link
:white_check_mark:

articles/index.yml
100
100
0
link
:white_check_mark:

articles/machines/index.md
87
71
21
link
:white_check_mark:

articles/machines/toc.yml
100
100
0
link
:white_check_mark:

More info about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues

Docs Build status updates of commit bea50a3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/TOC.yml
:white_check_mark:Succeeded
View (qsharp-preview)

articles/index.yml
:white_check_mark:Succeeded
View (qsharp-preview)

articles/machines/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

articles/machines/toc.yml
:white_check_mark:Succeeded

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


Allow for duplicate requests when no x-github-delivery header is specified

@JasonEtco here's a proposal for implementing a fix for #76.

Seems like it may be a little more complicated as the x-github-delivery header value is also used as the reactjs key for the ListItem:

Yep - this is because GitHub sends a unique id for every payload. I'd say we fall back to a timestamp in that case, which Smee includes on all payloads:
 const id = item['x-github-delivery']  || item.timestamp

Seems like it may be a little more complicated as the x-github-delivery header value is also used as the reactjs key for the ListItem:

Yep - this is because GitHub sends a unique id for every payload. I'd say we fall back to a timestamp in that case, which Smee includes on all payloads:
 const id = item['x-github-delivery']  || item.timestamp

@JasonEtco I added your suggestion, but the tests are still having the issue, I assume the timestamp property just needs to be added to the test, but I'm on my mobile at the moment, will try have a look at it properly when I have a gap.

It's a pleasure @JasonEtco, thanks for the refinements & merging it in.


Wording of limitation causing confusion with customers

The following statement in this doc is causing some confusion readers:
"By design, a Managed Instance needs a minimum of 16 IP addresses in a subnet and may use up to 256 IP addresses. As a result, you can use subnet masks /28 to /24 when defining your subnet IP ranges."
More than once I have spoken with individuals that have confused this to mean the largest subnet in which MI could be provisioned is a /24. Please clarify if possible.

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

ID: 2b9fa4c5-9ea3-7494-3b5a-fee6bccc9e0e
Version Independent ID: 4493877b-f253-3451-a034-2a485b72bc22
Content: Azure SQL Database Managed Instance determine VNet/subnet size
Content Source: articles/sql-database/sql-database-managed-instance-determine-size-vnet-subnet.md
Service: sql-database
GitHub Login: @srdan-bozovic-msft
Microsoft Alias: srbozovi

@jiroseng Thank you for the detailed inquiry. We are investigating this and will update this thread with additional details.

@jiroseng In looking through the information, a network mask bit of /28 (14 hosts per network) will work for a single general purpose or business-critical deployment. A mask bit of /27 (30 hosts per network) is ideal for a multiple Managed Instance deployment within the same VNet. Mask bit settings of /26 (62 hosts) and /24 (254 hosts) will further allow scaling out the VNet to support additional Managed Instances.

@CarlRabeler This is something that needs to be reviewed to ensure clarity.

update pending https://github.com/MicrosoftDocs/azure-docs-pr/pull/67502

PR merged @Mike-Ubezzi-MSFT please close

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


go: provide a means to change argv[0]

In Docker for Mac, this is used so that all the processes belong to the com.docker.* family.

Could you have a look, @djs55?  TIA!

Looks good to me!

Can't install it myself.


Use version variable for download URLs

Simplifies updating a formula to only be resetting hashes. May prevent nitty errors in the future

Looks like brew test-bot is somehow not realizing this is a tap anymore and is trying to install the homebrew core spin project: https://github.com/fermyon/homebrew-tap/actions/runs/6854772721/job/18638412540#step:8:90
Previously didn't happen: https://github.com/fermyon/homebrew-tap/actions/runs/6828962688/job/18574160348#step:8:36

Looks like the upstream issue is already merged.  Do we need to wait for a release -- or just retrigger the action?


slack rate limits

https://api.slack.com/docs/rate-limits#events_api
I've been hit by slack rate limits (and not even doing that many calls.. they've become much more stringent with that recently). When that happens we get back http 429, which slack-web forwards to the caller, and there is a HTTP header: Retry-After: <number of seconds>.
I think that maybe slack-web should optionally handle that for the user. I can maybe try to make a PR, when I get some time, but we'd need to agree as to how to do implement that.
Do we add a new option to all existing calls RateLimitHandling = RateLimitFail | RateLimitWaitAndResend? Do we do it automatically out of the box, without any way to disable it (in which case there is no API change)? Or do we consider slack-web is the wrong layer in the stack to deal with that, and we change nothing compared to the current situation?

Your pull request would be for option 1, right?

I wanted your opinion before starting anything (if I'll find time to do it at all). So all options are on the table.

Yeah, option 1 sounds good. Is that the best for your use case?


params error

 <router-link  :to="{ name:'customersDetail',params: { id: item.id }}" v-for="item in customers">...</router-link>
the Console shows:

vue.js?3de6:515[Vue warn]: Error when rendering component :

vue.js?3de6:2902Error: [vue-router] missing param for named route "customersDetail": Expected "id" to be defined(…)

but it works well,just show the errors.Is there something wrong?

Hi, thanks for filling this issue. Please follow the Issue Reporting Guidelines and provide a live reproduction on jsfiddle, codepen etc. Thanks!


1.1.1 traces in DEBUG mode

In version 1.1.1 lot of traces are in DEBUG mode, how can I configure the log in INFO/ERROR mode ? any idea ?

Looking at the ui container, there is a LOG_LEVEL env variable, may I change that env variable to info ?
docker exec -ti harbor-ui /bin/bash
root [ /harbor ]#
root [ /harbor ]#
root [ /harbor ]#
root [ /harbor ]#
root [ /harbor ]# env
HOSTNAME=e441hhhc2bb80
UI_SECRET=........
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
PWD=/harbor
JOBSERVICE_SECRET=.........
CONFIG_PATH=/etc/ui/app.conf
SHLVL=1
HOME=/root
LOG_LEVEL=debug
GODEBUG=netdns=cgo
_=/usr/bin/env

You can change the log level in common/config/ui/env file and restart Harbor after that.

Maybe we could add that to admin server also?  @ywk253100

Hi, The following container's have the LOG_LEVEL env variable:
harbor-jobservice
harbor-ui
harbor-adminserver


Fix Bug Preventing Reupload

Compare a re-uploaded solution only with the user's latest uploded solution to that same exercise.

Please fix the lint errors, @itamargran


Can this support lutron caseta dimmer switches too?

I really want to be able to associate additional actions with my lutron caseta dimmer switches. In particular, I want to be able to make "lights off" on a dimmer switch turn off other lights too (but turning it off with HomeKit shouldn't do this). This plugin README only indicates support for pico remotes. Can this be extended to trigger actions with the dimmers themselves?

I would go ahead and just check bus logging to see if events for this are sent, but I don't have a Pro bridge yet (no point in buying it until I know I can do what I want).

Unfortunately the data on the bus is the same whether you press the switch or the lights are controlled by HomeKit.

So the bus doesn't say "dimmer switch button pressed", instead it says "light at N%"? That's disappointing.


feat: upgrade to yarn v4.4.1

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore(master): release 0.5.7

:robot: I have created a release beep boop
0.5.7 (2024-08-08)
📚 Documentation

add deno CLI installation section (dfd421c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

0.5.7
:sunflower:


Tests for Normalizers

Add tests for normalizers (? @typesupply)

Yeah, these need tests. I’ll take it on.

Done with #231


feat(client): exports map

This PR enables the new DX that we want for our driverAdapters preview for edge functions. In order to learn more about this, please read this document. Additionally, this document turns the previous document and other related research (successes and failures) into a summary and a plan, so you may be interested into reading it.
Exports maps
In a nutshell, we want to get rid of @prisma/client/edge and engineType=wasm. In order to achieve this, we use an exports map. The exports map is responsible for loading the correct client for the correct runtime.
Custom outputs
However, it is important to note that export maps don't work with custom outputs, because they only work from within node_modules. This is where things are more complicated: we want to make it work there as well so that our DX is equal in all directions.
Warning channels
In order to make this work, a user using custom outputs will be prompted to add their local package folder as a dependency to their own package.json. We have 3 channels to let them know about that:

pnpm prisma generate will emit the first piece of information with instructions and docs
If they missed that, then they will get a deprecation warning on their imported client (via types)
If they missed that, then they will get a warning with the same information as above (via runtime)

Once they have setup their local prisma client as a dependency, the exports map works and all the warnings disappear. A warning can happen if a user has either not imported their local client as a dependency, or they have misconfigured their tsconfig.json's moduleResolution. In either of these cases the exports maps either don't work or don't work optimally and the warning will be displayed through one of the channels.
Warning emission
In a few words, the mechanism that supports turning these warnings on and off is based on the exports map itself. First of all, such warnings can only appear on custom output clients, they should never appear otherwise.
When an exports map isn't working, and a user is importing their client like eg. ../prisma/client, under the hood ../prisma/client/index is actually loaded. We rely on this behavior to:

Include a runtime warning in index
Include a type-level warning in index.d.ts
(We always output instructions for custom outputs on generate)

Warning suppression
So index becomes a "warning index" for custom outputs, to deliver further instructions. When instructions are applied, the exports map then works and now ../prisma/client was replaced with a db import (since we ask users to make it a proper dependency now). So now when a user imports db, it is gated through the exports map and does not load index anymore, it loads default -- the warning-less counterpart of index.
Under the hood, the way this works is that default is the actual generated client and types, and index is just a thin wrapper on top of it to emit warnings as needed. For the types index overrides the PrismaClient type to mark it as deprecated and re-exports all the rest. For the runtime, index re-exports everything from default as well and has an additional console.warn.
Better imports
This then supports our users so that they don't have to think about importing @prisma/client vs @prisma/client/edge vs @prisma/client/wasm. That also means that we could simplify our instructions on generate, so the part about @prisma/client/edge is gone there too (the Accelerate hint remains though).
Exports mappings
Another thing that the exports map enables for us is the ability to remove engineType=wasm. This is no longer a requirement because on runtimes like workerd, the exports map will point to @prisma/client/wasm under the hood, which is a client capable of loading the wasm engine in edge runtimes. Let's recap our runtimes and entrypoints:

@prisma/client/default: Our Node.js client that works with engineType
@prisma/client/edge: Now a client that is solely meant to run with Accelerate
@prisma/client/wasm: Similar to edge, but can run both Accelerate and Wasm

Of course, all these imports happen under the hood and are managed by the exports map. In a nutshell, these are the mappings that the exports map helps us achieve can be summarized like the following:

nodejs -> @prisma/client/default (Accelerate, Library, Binary)
worker -> @prisma/client/wasm (Accelerate, Wasm)

@prisma/client/edge could thus in theory show a deprecation warning too in the future, and we have a task for it for GA.
Removal of engineType=wasm
Because the exports map is able to do the right decisions for us, it means we no longer need engineType=wasm. We will now always have a wasm client generated, and a small tradeoff of that means that the query-engine.wasm will always be included into the generated folder, unless prisma generate --no-engine is being used, which will restrict to only being able to use Accelerate in edge runtimes.
Another consequence of this is that Node.js will always use the library (or binary) engine, and after this PR will no longer be able to load Wasm files. This is a conscious choice, as we've measured that driverAdapters are slightly more efficient via N-API, we will only load the Wasm engine if we need to (ie. on edge runtimes). This is not set in stone and we can change it in the future, of course.
New client runtime
As you might have guessed, as there is a new @prisma/client/wasm endpoint there is also a new client runtime: client/runtime/wasm. The wasm client runtime shares much of its configuration with the edge one and are nearly similar in the way they are built.
The key differences between edge and wasm after the changes from this PR is that edge is now exclusively used for Accelerate and thus only contains the DataProxyEngine. On the other hand wasm includes both the LibraryEngine and DataProxyEngine (that follows the usual pattern that we always include Accelerate in any of our runtime builds).
Additionally, as mentioned earlier, library will not include any Wasm loading abilities anymore.
Changes to our test setup
Because we made that the Wasm engine cannot be loaded in Node.js anymore and because engineType=wasm went way, it required us to adapt our tests and functional tests to accommodate these changes. The main change in our test setup is the introduction of --client-runtime which under the hood will force loading one of the compiled client runtimes (found in packages/client/runtime). They can be edge, wasm, or node, and of course these namings are meant as a parallel to our exports map.
Being able to run --client-runtime wasm has enabled us to test our wasm client more thoroughly as it is the same runtime file that runs on Cloudflare Workers. Previously, we ran our tests against the node runtime even when testing Wasm via driverAdapters (ie. the un-polyfilled version of the runtime). Now only the Wasm loading remains Node.js based, as we run our tests in jest, not workerd.
Other changes
Generation cleanup
In order to implement this, I had to touch many places in our generation code. I felt like it was becoming critical to cleanup in order to be able to maneuver in this space now and in the future, as the code is getting more confusing and less structured over time and as we add new things on top of legacy. I took the opportunity to deep dive and improve things along the way.
Generation performance
As I added more things, it degraded the performance by 40% (according to CodSpeed). I noticed that we were cloning the dmmf with klona multiple times in order to generate the clients. So I turned our DMMF into a Readonly type and got rid of klona. Additionally, I made the properties of DMMFHelper lazy, so they only compute when we need them. This resulted in a 30% generation speed increase (according to CodSpeed).
Fix of a bug in WasmLoader
I found that there was a race condition in our Wasm loader, which could probably happen in production. I found this because it was exacerbated somehow by the changes in our functional test setup.
Cleanup of "dummy file" logic
We have some code that creates dummy files before a client is generated. I needed to update this area and noticed quite some duplication there, which could make changes prone to error. So a small bonus refactor along the way.
Better snapshots for edge/wasm clients
We've had a long standing issue where we had to skip some tests and snapshots because edge/wasm clients cannot read from disk, however in order to show pretty errors with line numbers we also read the source files from the path retrieved by the stacktrace. Now a $fs global utility is injected by jest to allow edge/wasm clients to do just that, so we don't need to skip tests.
/integration

I think we should also adjust a generated name in .prisma/client/package.json. Currently it is invalid and some package managers complain about it when you try to add custom path as a dependency
Also worth double-checking: in case of multiple generated clients, how would package managers react to the packages with identical names?

Yes, I agree. We discussed that here. After looking into it, it seems the package needs to be both private and have a better name. From my first exploration, it seems the name does not actually matter for package managers, though I think we should make it unique for other tooling.

Just so it won't get lost. pnpm actually complains about invalid name of generated package if it is added as dependency.
To reproduce:

generate client at custom path
add it as dependency as our new warning suggests
remove generated client and re-run pnpm i
generate client at same location and run pnpm i again

For what it's worth, this PR is not a minor compatible change as demonstrated in https://github.com/newrelic/node-newrelic/actions/runs/7712498218/job/21020203083?pr=1983#step:6:16711

Thanks for the comment @jsumners-nr, we're looking into it.
We have a regression reported here, don't know yet if this is related: https://github.com/prisma/prisma/issues/22870

sooo much simpler now 🙏 🎉
(Next.js, using both Edge early access + normal Node engine, locally + deployed to Vercel)


ShareViaFacebook call callback before share in facebook app

My code is
			           $cordovaSocialSharing.shareViaFacebook(null, canvas.toDataURL(), null) 					   .then(function(result){ 						   CoinService.winCoins($scope, "add_coin"); 					   });
The callback is called before user share in facebook ap and the callback always pass 'true' in result param, so user receive coins without  share anything.
They have any way to fix it?

Duplicates:
#829
#733
#722

Fine!

@TalissonBento follow #841, I found a way to let the callback be resolved after the user share on android too
About always getting 'true' as a result I tend to think that there is no solution about that, see post scriptum of  #841 too


Linting with flake8

Last PR I introduced flake8 but some exceptions had to be added in order to successfully pass the pipeline. Now I'm removing:

F401: module imported but unused
F541: no need to use f-strings if there's no placeholder

thanks!


Suggestion: comparison with existing programming language detection tools

It seems that some have claimed that their project of predicting programming languages has a higher accuracy than HL.js, is it possible to take a look and see what makes it so special?

https://github.com/andreasjansson/language-detection.el#model-performance
https://github.com/github/linguist
https://github.com/blakeembrey/node-language-detect
https://github.com/ts95/lang-detector
https://github.com/Prev/shaman

some have claimed

Obviously "some" doing some "claiming" is a pretty worthless standard. :-)
Closing as a duplicate of https://github.com/highlightjs/highlight.js/issues/1213.  Feel free to drop your comments over on that thread though.  Thanks!
Surely worth taking a look into if anyone ever finds the time.  Auto-detection doesn't seem to be a super high priority though - though eventually I plan to swing back around to it.


FileNotFoundError: No file './data/heli.png' found in working directory

Traceback (most recent call last):
File "/home/user/Downloads/PYTHONTROOPER-main/paratrooper.py", line 712, in
SimpleGraphics.Run(screen_width, screen_height, window_width, window_height, Create, Update, Draw, OnKeyPress, OnKeyPressed, OnKeyRelease, OnMouseMove, OnExit, 'Consolas', 10)
File "/home/user/Downloads/PYTHONTROOPER-main/SimpleGraphics.py", line 64, in Run
init()
File "/home/user/Downloads/PYTHONTROOPER-main/paratrooper.py", line 533, in Create
heli_image = SimpleGraphics.LoadImage("./data/heli.png")
File "/home/user/Downloads/PYTHONTROOPER-main/SimpleGraphics.py", line 168, in LoadImage
return Image(file)
File "/home/user/Downloads/PYTHONTROOPER-main/SimpleGraphics.py", line 33, in init
self.image_data = pygame.image.load(file)
FileNotFoundError: No file './data/heli.png' found in working directory '/home/user/Downloads/PYTHONTROOPER-main'.

Hey uh, sorry for replying this late. I forgot to upload the data files. It should work now.


WS chat is set as unread for Over Limit Approver when an expense/report is submitted to another approver

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: N/A
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Issue reported by<EMAIL_ADDRESS>Slack conversation: https://expensify.slack.com/archives/C05LX9D6E07/p1730281713256759
Action Performed:

Create Control workspace, invite Submitter, Approver A, Approver B
Set approval workflow in OldDot as Advanced Approval + Submitter > Approver A + Approver B as $10 overLimitForwardsTo
As Submitter, create expense for + $10 in the WS chat

Expected Result:
The WS chat should remain read for Approver B since the expense isn't ready for their attention.
Actual Result:
The WS chat is unread for Approver B even though they have nothing to do.
Workaround:
Nope!
Platforms:
All
Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Replicated and issue seems unique

Can this be external @Beamanator ?

Ehhhhh i think this should be internal, because externally we'd just do a bandaid fix most likely

I'm working on related things in https://github.com/Expensify/App/issues/51689 so i can probably take this today

FYI I'm seeing that this isn't specific to overLimitForwardsTo, I'm seeing even forwardsTo is seeing the WS chat unread when it should be, am i crazy?

You might not be crazy! Jason reported for overLimit and I repro'd that. If this extends to forwardsTo let's fix there too.

Giggidy - I'll look into this now!
I would go either HIGH or MEDIUM priority for this one, what do y'all think?

Ok sorry but even one more point: I am pretty sure the overLimitForwardsTo got the Unread workspace when the expense was created / added to the workspace chat - even before the submitter submitted it!

I'd say medium, it's confusing, not debilitating.

Ah, you're on delay submission - so you're saying the Draft expense also set as unread? I don't think that should happen for anyone. Wanna clean that up now too?

Ok so a bit more clarity:

Yes on delayed submission 👍
I thought the problem was about SUBMITting the report, but it's actually about SUBMITting the expense (which is a bit confusing since that's not actually submitting the report, it's just creating an expense on a new report, but it's not submitted yet)

I think i was previously confused by this issue title 😅  so submitting THE EXPENSE is what causes the unread 👍 , not submitting the report to any approver 👍

Also note: It even looks like the submitsTo sees the workspace chat as unread when the submitter just creates an expense on a draft report, BEFORE submitting it

This also seems wrong since they have nothing to do, but I can flag this more generally. I can confirm we've discussed not making the WS chat unread for further approvers when they don't have anything to review.

@MitchExpensify once #435128 is done, let's confirm this works.

Aah shucks, ok then I'll 🏃  for now 👍
Does this also wipe out the "and unread" part of the requirement of this issue?

Expected Result:
Approver B should see that the WS Chat is GBR and unread.

Yeah, I think it should, but we still need GBR to show.

Yep yep 👍  i just commented in the other issue, GBR fix should hopefully be live

Not overdue, waiting on https://github.com/Expensify/Expensify/issues/435128 to hit prod to test


Prometheus and Grafana retention and scrapping

Added option for scrapping
Added option for retention

Look what you make do @mickmcgrath13


HTTP ToolKit doesn't work after patcher

Mac OS Sequoia 15.1

can you share debug logs using node . start

Nothing there

Nothing there

uh maybe try repatching again? node . restore && node . patch

Same thing on linux after updating to 1.17.2-1

Same errors and here are the log:
--- Launching HTTP Toolkit desktop v1.19.0 ---
Config checked in 27 ms
Certificates setup in 11 ms
Standalone server started in 10 ms
Server started in 40 ms
Total startup took 88 ms
Running /Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home/bin/java:
Exception in thread "main" java.lang.NoClassDefFoundError: com/sun/tools/attach/AgentLoadException
at java.lang.Class.getDeclaredMethods0(Native Method)
at java.lang.Class.privateGetDeclaredMethods(Class.java:2701)
at java.lang.Class.privateGetMethodRecursive(Class.java:3048)
at java.lang.Class.getMethod0(Class.java:3018)
at java.lang.Class.getMethod(Class.java:1784)
at sun.launcher.LauncherHelper.validateMainClass(LauncherHelper.java:690)
at sun.launcher.LauncherHelper.checkAndLoadMain(LauncherHelper.java:672)
Caused by: java.lang.ClassNotFoundException: com.sun.tools.attach.AgentLoadException
at java.net.URLClassLoader.findClass(URLClassLoader.java:387)
at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:359)
at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
... 7 more
=> Java attach APIs are not available

bump
Running java:
Exception in thread "main" java.lang.NoClassDefFoundError: com/sun/tools/attach/AgentLoadException
at java.lang.Class.getDeclaredMethods0(Native Method)
at java.lang.Class.privateGetDeclaredMethods(Unknown Source)
at java.lang.Class.privateGetMethodRecursive(Unknown Source)
at java.lang.Class.getMethod0(Unknown Source)
at java.lang.Class.getMethod(Unknown Source)
at sun.launcher.LauncherHelper.validateMainClass(Unknown Source)
at sun.launcher.LauncherHelper.checkAndLoadMain(Unknown Source)
Caused by: java.lang.ClassNotFoundException: com.sun.tools.attach.AgentLoadException
at java.net.URLClassLoader.findClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
at sun.misc.Launcher$AppClassLoader.loadClass(Unknown Source)
at java.lang.ClassLoader.loadClass(Unknown Source)
... 7 more
=> Java attach APIs are not available

I cannot see the response for some requests (it just spins indefinitely). Mock editing isn't working and I still have the initialization dialog when starting. It looks like I'm getting ratelimited but I also got 428 status codes. Maybe did they add authentication or TLS fingerprinting?
logs.txt

Me too

No internet connection and file is not cached

PLIIIIIIISSSSSSS HEEEEEEEELP 🙏🙏🙏🙏🙏

I could help and see what requests the program can't get a response to, but that would require HTTPToolkit 😂

Someone reported this github link in HTTP Toolkit


dev mode webpack

Problem
Our current start script runs like this:
npm run build-client-watch & npm run start-server

Webpack (build-client-watch) runs as a background job (that's what the single ampersand does). This creates a couple problems:
1: The background job can turn into a zombie process causing confusion while debugging
2: It is possible to reload a browser tab before the webpack process finishes. express will happily serve the stale bundle.js
Proposal
Use webpack-dev-middleware. Webpack ships a middleware that can be added with app.use.
Webpack dev middleware bundles the javascirpt just-in-time when it is requested. It's impossible(probably) to get a stale bundle. And there will be only one process for boilermaker projects.
Problems created
dev/deployed configuration
This will create a little more disconnect between the dev and deployed environments. It makes sense to build a bundle.js on disk for deployed environments, so we'll need to write our webpack configuration such that it can be used both by the middleware and by webpack command line. (this is not difficult)
inconsistent with other projects
We follow this pattern in some other projects, it would be best to find them and update them to follow this pattern as well. (senior enrichment, some later jr phase projects)

Oh and also there's the concurrently package, which color-codes the concurrent tasks by namespace (and does a whole bunch more besides). Might be worth considering.

Popping into this discussion to offer a comment of questionable value: on all my current projects, I've found that just using Parcel is totally the way to go for building a frontend. It doesn't have a quick way to connect to an express server, but it would be straightforward to write a parcel plugin that:

Looks for ${bundler.options.rootDir}/server.js
Requires it
Calls it with bundler.server

(Parcel plugins are just functions which are called with the bundler.)
Bonus points if it also provides a command that will do this, only without the bundler and with an express server serving on $PORT and statically serving dist (which parcel bundles to).
Arguably, suffering through the slings and arrows of outrageous webpack is an important rite of passage, but my current life is better this way for sure.


See if remove empty intervals still necessary in CRSP sensitivity workflow

There is a task that might now be redundant.

Not necessary, and PR #919 will close this issue.


Implement Rotatable plan

Spec: https://material-motion.gitbooks.io/material-motion-starmap/content/specifications/motion_family/direct_manipulation.html

💻 I'm working on a diff at http://codereview.cc/D1966

🎊 My diff is ready for review at http://codereview.cc/D1966


fix: Set default MOTD if unset

Fixes #1261
-v "$MOTD" is used to check if a environment variable is unset, and returns 0 (FALSE) if the value is set.
Thank you to @Stealthii for identifying this issue.

Sorry for committing my own suggestion. Wanted to get this merged and published ASAP.


Add testExistsRelationshipWhereClauseParsing() unit test.

Code is already in place, all you need to do is add the test. :)

#22

Hi! implementing this test, I realized that does not exists an implementation for caching a query for Relationship Absence. It would be when we try to do something like:
$authors = collect([(new Author)->doesntHave('books')->first()]);
I tried to execute it, and it returns an error when trying to create a cache key:
ErrorException: Undefined index: column
/home/vagrant/Code/laravel-model-caching/src/CachedBuilder.php:106
/home/vagrant/Code/laravel-model-caching/vendor/laravel/framework/src/Illuminate/Support/Collection.php:1197
/home/vagrant/Code/laravel-model-caching/src/CachedBuilder.php:107
/home/vagrant/Code/laravel-model-caching/src/CachedBuilder.php:28
/home/vagrant/Code/laravel-model-caching/src/CachedBuilder.php:203
Can I open an issue for this?
Thanks

@brunosantos thanks for reporting, yes, please create an issue! :)


Updgrading poetry CI job version

Introduction
Fixing bug on Windows CI tests.
Changes
Updating version of the github workflow used to install poetry

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


Output running haml-lint

Versions:

ruby 2.1.1
overcommit 0.27.0
haml-lint 0.13.0

Console output:
git commit -m 'asdf'
Running pre-commit hooks
Analyzing with haml-lint...................................[HamlLint] FAILED
Hook raised unexpected error
Unexpected output: unable to determine line number or type of error/warning for message 'undefined method `type' for false:FalseClass'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook/pre_commit/base.rb:30:in `block in extract_messages'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook/pre_commit/base.rb:28:in `map'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook/pre_commit/base.rb:28:in `extract_messages'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook/pre_commit/haml_lint.rb:14:in `run'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook/base.rb:45:in `block in run_and_transform'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/utils.rb:211:in `with_environment'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook/base.rb:45:in `run_and_transform'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:107:in `block in run_hook'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/interrupt_handler.rb:63:in `disable_until_finished_or_interrupted'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:106:in `run_hook'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:59:in `block in run_hooks'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:58:in `each'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:58:in `run_hooks'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:39:in `block in run'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/interrupt_handler.rb:84:in `isolate_from_interrupts'
/Users/userName/.rvm/rubies/ruby-2.1.1/lib/ruby/gems/2.1.0/gems/overcommit-0.27.0/lib/overcommit/hook_runner.rb:24:in `run'
.git/hooks/pre-commit:69:in `<main>'
Analyzing with RuboCop......................................[RuboCop] OK
Running JsTranslations...............................[JsTranslations] OK
✗ One or more pre-commit hooks failed

Hey @yosvel-quintero,
Please update to the latest version of haml-lint and overcommit, and let us know if you still see this issue.


We need += and -= operators for UPDATE

When updating some link with * cardinality it is often useful to only add or remove targets rather than redefine the entire target set. To this end we can add += and -= operators in place of :=. They would only appear in SET clause of UPDATE statements.

@vpetrovykh, the implementation for this has landed, we need to update the docs and add this to the tutorial/cheatsheet etc.


Added new node.js util functions

Added definitions to the node util module.

isBoolean
isNull
isString
isNullOrUndefined
isNumber
isString
isSymbol
isUndefined
isObject
isFunction
isPrimitive
isBuffer

Thanks! Merged as c061557


Added Annulus Calculator

Fixes Issue🛠

Closes #  #1576
Description👨💻
Added Annulus Calculator
Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[x] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Damini2004, You should add your calculator in alphabetical order in main index.html file of the project.

@Damini2004 UI needs to be improved little bit. Also, you have specified that you will provide diagram?
https://github.com/Rakesh9100/CalcDiverse/issues/1576#issuecomment-2227200662,

@Nihal4777  Changes done
Ohh i thought she was asking to give diagram of that particular formula, for diagram implemetation i need some time
KInldy review and merge it.

@Nihal4777 Added.

@Damini2004 On small screens, there is overflow.

Yes it will overflow because the radius is increased then it not possible to visible the bigger circle. so we can provide diagram of inner circle rather than outer circle. Are you ok with it.

Where is the proper diagram, just white circle is visible. Also, that is hiding the result, pls fix @Damini2004

Are you working @Damini2004

Will close the PR if changes are not made today @Damini2004


Improvements to number and int const eval

Uses specialized operators for Number Ord ops rather than going the the Ord dictionary indirection. This means const eval for Number matches the runtime behavior in the presence of NaN, which is undefined for the purposes of Number Ord.
Adds overflow checks to Int operators, and bails on const eval in the case that overflow is detected.
cc @anttih

Thanks! I added NaN to the snapshot.