RelationManager: use pivot's primaryKey if available

Package
filament/filament
Package Version
v2.12.1
Laravel Version
v9.11.0
Livewire Version
v2.3
PHP Version
PHP 8.1.5
Bug description
If I create LocationUser with two same location_id and user_id:

pivot_id
location_id
user_id

11
2
7

12
2
7

Filament will fail to distinguish between them, assign them user_id for x-model, so they both will be selected for bulk operations. Would it be possible to configure the column used by tables to distinguish records?
My best guess would be to check whether getKey() on pivot returns null, which would be usual for pivot table, and then apply the current logic. If it returns something, use it as key.
Table config as below. All tables have models.
User:
- id

Location:
- id
- date_from
- date_to
- is_active

LocationUser:
- pivot_id, PK
- location_id
- user_id
- pivot_date_from
- pivot_date_to
- pivot_is_active

Steps to reproduce

Create tables as in description
Treat pivot_id as PK for LocationUser
Avoid inbuilt attach() / detach() methods
Create two records, bridging the same records

Relevant log output
No response

Related to #1804

Please test #2377, the docs for this feature are in the PR.
The primary key on the pivot table needs to be id, unless you specify a custom key name on a custom pivot model. Also ensure that id (or your pivot key name) is listed in the withPivot() array of the relationship AND its inverse relationship.

It appears to be working, though I failed to use my custom PK, so I removed prefixes from intermediate table. They won't be needed directly anyway if this pulls off, but I'll keep trying.
Working:

Attaching / detaching / editing records.
Table selector and actions correctly use PK
Saving pivot data through $record->pivot directly in custom actions

Issues:

For all of this to work, I have to use Model, not Pivot. Maybe it's just my messed config.
Pivot's own id key masks record's id, so creating url links for them is impossible. Collection::pluck() is a little confused because of masked id allowing duplicates
There is a pivot available on $record now, but I can't access it in columns and inputs, using dot notation,
though it works in custom actions, when injecting the collection.
You are also using pivot_ prefixed properties, and removing them before passing record to user. My pivot_is_active became pivot_pivot_is_active, and still qualified for purging from record, so I was a little confused when I didn't see my keys.

Would it be possible to ditch merging resource with relation and make pivot available with dot notation?
Alternatively, we could have pivot_ prefixed attributes, and smash then into their relation when saving.

@danharrin I'm confused about this PR:
class Posts extends Model
{
    public function users(): MorphToMany
    {
        return $this->morphToMany(User::class, 'userable')
            ->using(Userable::class)
            ->withPivot(['user_id', 'options'])
            ->withTimestamps();
    }
}

class Userable extends MorphPivot
{
    /**
     * @var string
     */
    protected $primaryKey = 'user_id';

    protected $casts = [
        'options' => AsArrayObject::class,
    ];
}

What I'm I missing here?
Thanks!


Enabling 'Outline' and navigating to another file causes error

Describe the bug
00:03:51 msg_show Error executing lua callback: ...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:318: Invalid window id: -1
stack traceback:
	[C]: in function 'nvim_win_get_height'
	...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:318: in function 'create_preview_win'
	...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:398: in function <...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:383>

Steps to reproduce

open two files e.g. two java files
enabled 'Outline' when inside one of the files
navigate to the other file

Expected behavior
I expect it to NOT throw error when navigating to another file while 'Outline' is enabled.
Neovim version (nvim -v)
latest
lspsaga commit
latest
Terminal name/version
iterm2

@glepnir still an issue.

14:18:04 msg_show Error executing lua callback: ...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:318: Invalid window id: -1
stack traceback:
	[C]: in function 'nvim_win_get_height'
	...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:318: in function 'create_preview_win'
	...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:399: in function <...re/nvim/lazy/lspsaga.nvim/lua/lspsaga/symbol/outline.lua:384>

I can't reproduce after patch.

@glepnir ok try this instead:

have two files open
enable outline inside one of the files
remove focus from outline. focus your cursor in your code instead
navigate to the other file
focus outline now

update and try thanks


Payload size limit?

Is there a size limit for the payload passed to the New-PSOneQRCode function?
I'm trying trying to pass a payload which is around 4002 characters, and when running the function I get the following error messages:
Exception calling "CreateQrCode" with "2" argument(s): "Sequence contains no elements"
At line:59 char:5

$data = $generator.CreateQrCode($payload, 'Q')

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : InvalidOperationException

Exception calling "GetGraphic" with "1" argument(s): "Object reference not set to an instance of an object."
At line:61 char:5

$byteArray = $code.GetGraphic($Width)

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : NullReferenceException

Exception calling "WriteAllBytes" with "2" argument(s): "Value cannot be null.
Parameter name: bytes"
At line:62 char:5

[System.IO.File]::WriteAllBytes($outPath, $byteArray)

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : ArgumentNullException

Only the first exception matters. The others are follow-up errors. Can you verify that $payload actually has content? The exception seems to indicate that it is empty.

I took the function and modified it to pass in a device serial number and hash:
$payload = @"
Serial number: $($csv.'Device Serial Number')
Device hash: $($csv.'Hardware Hash')
"@
The output of which would be:
Serial number = 1223434
Device hash = T0HQAgEAHAAAAAoAFQNhSgAACgAkA2FKYFK5JwQCCQ..........(etc.)
If I remove the second line of the payload it works, which made me think it was a character limit of the second line.

I justvtested, and yes there seems to be a payload limit, based on the text you are wrapping. Which makes sense because there is a physical limit of how much information a QR code with a given size can represent.

In my test, the limit kicked in at around 1600 chars, depending on the actusl text a bit.

Having a read up on the standards and it appears its possible, but maybe the QRCoder binary doesn't support it. I'll keep digging! Many thanks for your help!


Update Kube-router to v2.1.0 to fix ICMPv2 Neighbour Solicitation packets drops

Environmental Info:
K3s Version:

Node(s) CPU architecture, OS, and Version:

Cluster Configuration:

This version update should fix the issue discussed in https://github.com/k3s-io/k3s/discussions/9807
Describe the bug:

Steps To Reproduce:

https://github.com/k3s-io/k3s/discussions/9807#discussioncomment-8970056
Expected behavior:

Actual behavior:

Additional context / logs:

Validated on master using commit d973fad | version 1.29
Cluster nodes:
ip-[redacted].us-east-2.compute.internal   Ready   control-plane,etcd,master   31s   v1.29.4-rc1+k3s1   [redacted]   [redacted]   Ubuntu 22.04.1 LTS   5.15.0-1019-aws   containerd://1.7.15-k3s1

journalctl -u k3s.service | grep "Starting network policy controller version"
Apr 17 17:57:34 ip-[redacted] k3s[1992]: time="2024-04-17T17:57:34Z" level=info msg="Starting network policy controller version v2.1.0, built on 2024-04-16T23:18:20Z, go1.21.9"
Apr 17 18:01:08 ip-[redacted] k3s[8767]: time="2024-04-17T18:01:08Z" level=info msg="Starting network policy controller version v2.1.0, built on 2024-04-16T23:18:20Z, go1.21.9"


[pyspotify 2.0.0b3 + libspotify 12.1.51] Created playlist shows up as "Loading..." in Spotify client

I am writing a Python script to automatically generate Spotify playlists from the subreddit /r/Music. Adding tracks to an existing playlist works, but when I create a playlist it will show up as "Loading..." in the Spotify client and will never actually load.
I have tried re-starting the client, logging out the current session in Python but it does not seem to work. The playlist remains as "Loading..." and does not seem to be initialised, although it does appear in the playlist container in Python.
import spotify
import threading

def connection_state_listener(session):
    if session.connection.state is spotify.ConnectionState.LOGGED_IN:
        logged_in_event.set()

def logout(session):
    logged_out_event = threading.Event()

    def logged_out_listener(session):
        logged_out_event.set()

    session.on(spotify.SessionEvent.LOGGED_OUT, logged_out_listener)
    session.logout()

    if not logged_out_event.wait(10):
        raise RuntimeError('Logout timed out')

def create_playlist(session, name):
    container = session.playlist_container
    container.load()
    playlist = session.playlist_container.add_new_playlist(name)
    while playlist.has_pending_changes:
        session.process_events()
    print "Playlist succesfully created:", name

logged_in_event = threading.Event()

# user credentials
username = 'bart.eijk'
password = 'secret'

# create config (locate your Spotify appkey file here!)
config = spotify.Config()
config.user_agent = 'rMusicToSpotifyPlaylist'
config.load_application_key_file(filename='/Users/account/spotify_appkey.key')

# create session and login
session = spotify.Session(config)
loop = spotify.EventLoop(session)
loop.start()
session.on(
    spotify.SessionEvent.CONNECTION_STATE_UPDATED,
    connection_state_listener)

session.login(username,password)
logged_in_event.wait()

if session.connection.state is spotify.ConnectionState.LOGGED_IN:
    print "User logged in:", session.user.display_name
else:
    print "Something went wrong! Current state is: ", session.connection.state

#create playlist and log out
create_playlist(session,"test playlist2")
logout(session)

Great that you got it working! :-)


[BUG]在Windows客户端上使用的问题

客户端是小秘的3.0.20 (Emby for Windows_3.0.20_v3.0(Carnival).exe)
弹幕加载有问题，具体表现为
1.弹幕在加载成功后有概率消失
2.手动点击播放下一集弹幕不会自动关联，也不会显示，手动关闭字幕再打开字幕出现字幕，但是字幕是上一集的。


Add modifiers for windicss

Hello, thank you for opening this PR! I really appreciate that.
I will try to add some tests for modifiers for Windi and Tailwind Css.

Hi, I've been playing with quasar-app-extension-ssg and it seems great! However some of the classes wasn't being pulled through from criticalcss due to the special chars.

Does it sounds good to you ?

Looks perfect to me.

@adzbuck,
If you want to try if your classes are processed, I just published the latest version of quasar-app-extension-ssg with updated beastcss.
Don't forget that you can customize the beastcss options to whitelist some selectors.


binary-search-tree: Onboard to Catch2.

This moves the only tricky test suite over to Catch, specifically the iterator test in test_sort().
FYI - @elyashiv
Related to #233, dependent on #244.

I'm adding a commit to move this to only be unlocked at the end of the core exercises, as it fairly obviously is the trickiest exercise we have. I'd rather not have people do 2 core exercises and then encounter this.


[List] Secondary list action rendering issue when list is virtualized (react-window)

Following the docs example to create a virtualized list with react-window and adding a list secondary action seems to cause some CSS issues. Example below.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

The secondary list actions are misaligned.
Expected Behavior 🤔

The secondary action should display correctly in line with each list row.
Steps to Reproduce 🕹

Steps:

Follow the docs guide on creating a virtualized list
Follow the docs guide on adding a secondary action to a list item

Codesandbox demo: https://codesandbox.io/s/material-demo-bb1k9
Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
^4.4.3

React
latest

Browser
all

TypeScript
no

etc.

Solved by wrapping the  in a div. Should we (happy to PR) document this somewhere? The docs example for using a secondary list item action currently do not wrap it in a div.

You can avoid the excess div by passing additional props to ListItem:
<ListItem ContainerComponent="div" ContainerProps={{style}} key={index}>
https://github.com/mui-org/material-ui/issues/16026#issuecomment-498669261


[FN] Lock stream in NetworkPeerConnection

Prevents us trying to access a stream once a connection has been closed.
https://stratisplatformuk.visualstudio.com/StratisBitcoinFullNode/_workitems/edit/3811

@fassadlr @dangershony anyone know anything about AsyncLock? I think I must have used it wrong... Tests are timing out

Looks like we want to do it a different way


Parametrizing application name for more flexibility

resolves https://github.com/dbt-labs/dbt-snowflake/issues/148

Problem : As per the current implementation, the application name is hard coded to "dbt". But as per some use cases, it might require some custom application name.

Solution : Making the application name parametrized. Whatever value the user will pass as application in profiles.yml that will be considered as application name, if user doesn't pass any value then it will be hard coded to "dbt".

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

Hi Everyone, is there any update on this PR? @dbeatty10 , @mikealfare please let me know if this is acknowledged.

Closing per https://github.com/dbt-labs/dbt-snowflake/issues/148#issuecomment-2092936654.

Closing per https://github.com/dbt-labs/dbt-snowflake/issues/148#issuecomment-2092936654.


impl TryFrom for Instant and Duration

With this change, the minimum required version of easytime with --no-default-features goes up to Rust 1.34 (the minimum required version of the default feature has not changed).

bors r+


Unused dependency check

cargo-udeps is a handy tool to check that all declared dependencies are actually used in a crate, would be a useful check to add.

think we can close this older issue? as we don't have a path towards it.
we do actively use @bnjbvr 's cargo-machete in our larger projects, and cargo-udeps is indeed also an option.
would be great with a fully automated flow that has no false positives or manual fix ups though, but that would probably be another standalone tool or future Cargo feature.

Yah, forgot this was here.


Postman Collection Script Manger

Summary: Provide an open-source solution for aggregating and managing scripts across all of a team's Postman collections. Using Git or the Postman API to pull many collections, organize the scripts, and allow for easier indexing, managing, editing, and publishing of pre-request and test scripts that exist as part of the Postman collection format. Providing a simple, open-source way for developers to manage scripts across many different APis.
Skill: Familiarity with Postman, collections, and Node.js scripting.

Not ready to be a 2023 GSoC project. Postman Collections are a public repo, but would probably not qualify at them moment for participation.

open source projects wishing to apply need to have a solid community; the goal of GSoC is to bring new contributors into established and welcoming communities. While you don’t have to have 50+ community members, the project also can’t have as few as three people.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target-new/pull/8 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target-new.git

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

archvieSourceFolder/TOC.yml
:white_check_mark:Succeeded

archvieSourceFolder/content/someContent.md
: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.


Enable OME logs - parameter not found

(https://docs.microsoft.com/en-us/microsoft-365/compliance/ome-message-access-logs?view=o365-worldwide#enabling-message-access-audit-logs-in-powershell)
Section for enabling OME logging references command that does not work as ecpected.
Set-IrmConfiguration -EnablePortalTrackingLogs $true
Set-IRMConfiguration: A parameter cannot be found that matches parameter name 'EnablePortalTrackingLogs'.

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

ID: d6e8b911-7fb5-372d-a4a9-36be28e0f076
Version Independent ID: d6e8b911-7fb5-372d-a4a9-36be28e0f076
Content: Encrypted message portal activity log - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/ome-message-access-logs.md
Service: o365-seccomp
GitHub Login: @v-tophillips
Microsoft Alias: v-tophillips

@ConfigConfused Thank you for your feedback. I was able to execute the cmdlet on the exchange online v2 module. I have made changes to reflect that via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9479
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @ConfigConfused Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Is it possible this commandlet got dropped from exo v3 ? I'm getting
Set-IrmConfiguration -EnablePortalTrackingLogs $true
Set-IRMConfiguration: A parameter cannot be found that matches parameter name 'EnablePortalTrackingLogs'.

Is it possible this commandlet got dropped from exo v3 ?

That's what it looks like to me.  I opened https://github.com/MicrosoftDocs/office-docs-powershell/issues/11057 so hopefully they'll get the right team to fix this.

They closed the new issue that I opened.  Apparently, this flag requires E5 licensing, so if you don't have the most expensive plan it won't work.


Custom Repository Features for the best software!

I could try and test appget and penzo it's incredible! I am a system administrator and I worked with different software for managing the deployment, your idea is simple and functional (I also tried similar ones like scoop or chocolatey) I think your idea is smarter
I noticed that with the automatic package creation command the reppository folder is generated, in the appget setup, but I noticed that the yaml file created cannot be used anyway
it would be possible to implement the possibility of having an external repository, so that even in the corporate world you can use your software for fast deployments
according to me really is the only feature that is missing, for the rest it is TOP !!!

Hello,
great prise for this perfect software! But I have to put in a word for option of having own repo - I use a lot software which don't have available setup file on web.
It would be great if I can have setup files on my own web server (apache) or using samba (\server\apps).
Creating configurating file would be just be saved local and the file I could upload to my own repo.
Thanks a lot!


[Review][CARBONDATA-1381] Add test cases for missing scenarios in Partition feature

1.Remove unused methods and files
2.Added tests to cover uncovered lines and scenarios in Partition feature

retest this please

retest this please

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/228/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/236/

retest this please

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/255/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/258/

retest this please

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/261/

retest this please

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/264/

ok to test

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/289/

ok to test

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3419/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/290/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3420/

ok to test

ok to test

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3423/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/293/

ok to test

ok to test

ok to test

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3424/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/294/

ok to test

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3425/

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/296/

ok to test

ok to test

ok to test

ok to test

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/298/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3426/

Please refer ApacheSDVTests/300

SDV Build Failed with Spark 2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/342/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/541/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/25/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1206/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/575/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1139/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1730/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3079/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4323/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/4001/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5211/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7146/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/5922/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/73/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8822/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/577/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2025/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2202/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10277/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/98/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/97/

Build Failed  with Spark 2.4.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.4/94/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3180/


Setting "Identifier" color in Color Theme Editor for VS2017 breaks SemanticColorizer

Had some trouble getting the SemanticColorizer to work in my custom VS2017 theme. I tracked it down to the "Identifier" color being set within the Color Theme Editor. If "Identifier" is set to Automatic then everything is OK, but setting a specific color prevents any other colors from being set using SemanticColorizer.
Not sure if it's something that can be worked around, or is worth working around as I'm not entirely sure what the point of the "Identifier" color is, but I thought it was worth bringing up in case anyone else runs in to this problem.

Thank you @r-tanner-f for figuring this out. I gave up on this already. VS2017 behaves very strangely with syntax highlighting. Plugins register new types for the color and font dialog. But I have not found any description of what overwrites what. All semantic types are a specialization to identifier.
I will do some more experiments on my side. Maybe we can give a better how to use this plugin description.

The new 1.3.0 release also fixes some odd issues with the ordering.
I hope this fixes this issue as well.


manifests location for a plain+v0 bundle

According to the API

For an OCI image the manifests must be located at the root-level, i.e. /manifests directory for the provisioner to be able to locate the plain+v0 bundle.
For a Git repo there is a field directory to specify the location.
For a ConfigMap there is a field pathto specify the location.

What is the rational for the differences? It seems that the UX may improve by having the location set in a consistent way across the three options, i.e:

using the same name for the field,  path seems the more natural choice
supporting the specification of the location for OCI images

For an OCI image, the idea was that the image should only ever contain the bundle. We didn't want to encourage people to pack a whole bunch of extraneous stuff in an image because we have to ship that image around and unpack it to extract the bundle.
For a git repo, yes, directory lets you specify a subdirectory of the git root to extract. It looks like the code clones the repo and then returns a bundle FS rooted at the specified directory. Not sure if there's a way to avoid needing to pull the entire commit if we know we don't need a specific subdirectory. The idea here is that someone could point rukpak at a directory: bundle in a standard SDK project.
The path in configmap is a path to a specific directory within the bundle. In the configmaps case, all of the specified paths are rooted at the bundle root.
I think these are the correct choices, but at a minimum it seems like we should beef up the docs to explain things more. FWIW, I created #645 because we don't have any docs right now for the configmaps source.

My point was why to call it directory for git and path for ConfigMaps?

Maybe the name can converge, but they do have different meanings, so converging them might actually increase confusion.

git directory is the directory within the repo root that should be treated as the bundle root.
configmaps path is the directory within the bundle root where the configmap's data should be mounted as files.

This is indeed confusing.
What benefits do you expect from mounting multiple ConfigMaps? It seems to go in the opposite direction of immutability.
Why does the mount point need to be exposed to users? Isn't it implementation internals of the plain provisioner? Especially if you don't want the bundle file structure to be configurable.
In the pod spec it is named mountPath.
ReadOnly bool `json:"readOnly,omitempty" protobuf:"varint,2,opt,name=readOnly"`
// Path within the container at which the volume should be mounted.  Must
// not contain ':'.
MountPath string `json:"mountPath" protobuf:"bytes,3,opt,name=mountPath"`

Nevertheless a directory is not a path. Does it mean that for git you would accept /bundle but not /config/bundle?

still relevant

What benefits do you expect from mounting multiple ConfigMaps?

To overcome the etcd size limit of a single configmap.

Why does the mount point need to be exposed to users?

Different bundle formats have different requirements for where files need to exist relative to the bundle root. The ConfigMaps source is not specific to the plain provisioner.

This design was inspired by the openshift build API: https://github.com/openshift/api/blob/8891815aa476232109dccf6c11b8611d209445d9/build/v1/types.go#L461-L463

It seems to go in the opposite direction of immutability.

Right now, for better or worse, there are validating webhooks in rukpak that ensure referenced config maps are immutable and cannot be deleted

Different bundle formats have different requirements for where files need to exist relative to the bundle root. The ConfigMaps source is not specific to the plain provisioner.

Sorry I am not following why it needs to be exposed to users. I understand that for builds it matters as the Dockerfile, batch or whatever is used for the build and provided by the user needs to access the data.
For RukPak my understanding is that it is only internal code to the RukPak provisioners that accesses the content of the configmaps and applies it to Kubernetes. If different bundle formats have different requirements for where files need to exist relative to the bundle root can't the specific provisioners take care of it?

I would like to see us have both sensible defaults for where we expect files to live inside a source, as well as allowing the user to change things if the defaults don't work for them.
I imagine we'll want to have a knob to configure the path in a source to consider its root; this is the "from".
We'll also want to have a knob to configure the path in the local staging area where the source's files go; this is the "to".
This is what we're working toward in #703

@ncdc #703 is surely moving us in the right direction. I am happy to close this and continue the discussion there.


Events cause readouts

Mouse(Move and click), and keyboard Arrow key events are causing event code(what i assume them to be) readouts. No errors in stack or nothing.. Seems to only be happening in react-blessed. Tried reproducing in neo-blessed and blessed but they seem to be working properly.
My start script is:
nodemon --exec babel-node --presets env,stage-2,react --plugins transform-decorators-legacy,transform-class-properties --watch projects --watch interface interface/index.js
So purhpas this could be a babel issue? Any thoughts?
Edit: I forgot to add that no events are working/firing when this is happening as well. Not even screen events.

Hum. That's quite strange indeed. Can you provide a link to your code or a minimal reproducible version of it?

Maybe you are in mosh or screen env?


Call loadMissing() in a isolated model replica

Laravel Version: 8.46.0
PHP Version: 8.0.7
Database Driver & Version:

Description:
I have a question maybe a little silly. I'm trying to replicate and store a model instance in a variable to call the loadMissing method. The idea is to call the loadMissing method without being affected by the original model.
Steps To Reproduce:
I have the following method in my User model:
class User extends Authenticatable
{
   ...

   public function myCustomMethod()
   {
        $cloned = $this->replicate()
            ->loadMissing('roles.permissions')
            ->getAttribute('roles')
            ->flatMap(function ($role) {
                return $role->permissions
            });

        return $cloned;
   }

The problem is that in this way the original model instance is affected, and it loads in the original instance all the relations that I am loading only in this method.
My question is: is there a way to load and isolate relationships in a replica of the model instance without affecting the original model?
I have also tried cloning the instance and trying to use the loadMissing method on the cloned instance, but the original instance is also affected. For example:
class User extends Authenticatable
{
   ...

   public function myCustomMethod()
   {
        $original = $this;
        $cloned = clone $original;

        $cloned->loadMissing('roles.permissions');

        dump($original);
        dump($cloned);

        die();
   }

But in this way, I get the following results. Notice that both the original and the cloned instance are loading the relationships when I'm just trying to load the relationships in the cloned instance:
$original

$cloned

I was wondering if this is normal behavior. If it is a normal behavior, which I suppose it is not, I was wondering if there is any way to use the loadMissing method on a copy, replica or clone of an instance without modifying the original instance. Thank you very much in advance.

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


add Squarespace case study to index page

@kbarnard10 here is the PR for the squarespace case study and addition to the index page, thank you!

/lgtm
/approve

@zacharysarah Case study files also go in CSS/ and Images/ so I cannot approve.


added generator for say, fixes #530

I have the generator working, and I think everything is good to go. Please let me know if there are any changes I need to make.

The build looks like it's faling because it doesn't like how the SayTest class is defined in exercises/say/say_test.rb. However, the WordyTest class is defined the same way in exercises/wordy/wordy_test.rb. Thoughts?

I made the changes and generated another test. It looks like the expected values are being created as strings this time and the method names don't contain '-' or ',' anymore.
Let me know if you would see any more changes that need to be made, and I can also squash the commits down if you would like.

Looks like I finally got the build green! I definitely put Travis CI though it's paces for this PR.

@definemethod Go ahead and squash the commits, and do a force push on your branch to update this PR, and then a final review can be done... looking good!

@tommyschaefer; I made the changes you suggested in 0ccebce. Thanks for your help and feedback on this PR.
It looks like there are other tests in the xruby repo that don't have generators. Would it be alright if I created them and put up some PR's?

@ajwann Awesome! Thank you so much for all of your work on this!
In regards to other generators, absolutely! That would be fantastic! :+1:
@Insti @kotp I'm going to go ahead and merge this since @Insti and I approved this a while ago and it hasn't received any other feedback! Thank you both for helping me review this PR!


Page crashes when using a long series of space characters

Some combinations of plain string and long series of space characters causes too much recursion and crashes the page:
a

a

Tested here: https://probablyup.com/markdown-to-jsx/

Happening to us too. Initially thought it was because of image rendering, but firefox throws an exception about recursion depth so I believe its the same issue

Confirmed, ran into this with a block of SQL today that came from an IDE where whitespace was set to 4 spaces. The offending block of text:
WITH SalesByPrice AS
  (SELECT product_subcategory,
          ROUND(sale_price,2) AS sale_price,
          Avg(unit_qty_sold) unit_qty_sold,
          lag(sale_price, 1) OVER (PARTITION BY product_subcategory
                                   ORDER BY sale_price) Prevsale_price,
                                  lag(Avg(unit_qty_sold), 1) OVER (PARTITION BY product_subcategory
                                                                   ORDER BY sale_price) Prevunit_qty_sold

Tested here: https://probablyup.com/markdown-to-jsx/

also on 7.4.0

I tracked this issue (or maybe a similar one?) down to
const CODE_BLOCK_R = /^(?: {4}[^\n]+\n*)+(?:\n *)+\n?/

https://github.com/quantizor/markdown-to-jsx/blob/acd970dc7be8f2390ed916fe1b1cf0874a8534f6/index.tsx#L186C1-L187C1
being checked in
https://github.com/quantizor/markdown-to-jsx/blob/acd970dc7be8f2390ed916fe1b1cf0874a8534f6/index.tsx#L937
Tester for catastrophic backtracing: https://regex101.com/r/85LH2r/1
For me, a suitable testcase in index.compiler.spec.tsx is
it('handles \\r\\n without crashing', () => {
  render(
    compiler(
      'Test \r\n                                                                                \r\n Test'
    )
  )

  expect(root.innerHTML).toContain(`Test`)
})


fix a typo in README.md

This change is to fix #1491 , a typo in README.md.

LGTM

Coverage increased (+0.005%) to 83.456% when pulling 9275acbfa609fc8790a7657a977c46276d85dac2 on neozwu:fix-readme-typo into fa0720c65576ed68419772b04b463176bc8061d7 on GoogleCloudPlatform:master.


Added logic to delete BuildTools_Full.exe

You can't simply add Remove-Item to delete the BuildTools_Full.exe because it will get executed before the exe finishes.  To workaround this, logic was added to wait on the exe before it is deleted.  Also removed extraneous whitespace.
@rakeshsinghranchi, @vivmishra, @dleeapho

This addresses https://github.com/Microsoft/dotnet-framework-docker-samples/pull/3/files/341b68656deb9a44de1dbf2e867579c955e446d3#diff-e6049d7c254383d93f762d69b880e4da

@dleeapho - could you please take a look at this change?


Segmentation fault with cyclic imports

This bug was initially reported in https://github.com/bellard/quickjs/issues/329, but it seams that there it didn't get any attention, so I'm reporting it here too.
To reproduce create two files a.mjs and b.mjs  and run qjs a.mjs.
a.mjs:
import { fb } from "./b.mjs";
export { fb };

b.mjs:
import * as $a from "a.mjs";
export function fb() {};

There is no crash if export { fb }; is removed from a.mjs or the import in b.mjs is changed to import {} from "a.mjs";.
AddressSanitizer:DEADLYSIGNAL
=================================================================
==277778==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000000 (pc 0x0000004fbc1c bp 0x7ffcc9070200 sp 0x7ffcc90700c0 T0)
==277778==The signal is caused by a WRITE memory access.
==277778==Hint: address points to the zero page.
    #0 0x4fbc1c in js_build_module_ns /quickjs/quickjs.c:28041
    #1 0x4fc144 in JS_GetModuleNamespace /quickjs/quickjs.c:28076
    #2 0x58bf20 in js_inner_module_linking /quickjs/quickjs.c:28333
    #3 0x58b7e3 in js_inner_module_linking /quickjs/quickjs.c:28268
    #4 0x5a175d in js_link_module /quickjs/quickjs.c:28441
    #5 0x5a175d in JS_EvalFunctionInternal /quickjs/quickjs.c:34409
    #6 0x5a2487 in JS_EvalFunction /quickjs/quickjs.c:34425
    #7 0x40bec9 in eval_buf /quickjs/qjs.c:67
    #8 0x40bff5 in eval_file /quickjs/qjs.c:103
    #9 0x40b050 in main /quickjs/qjs.c:516
    #10 0x7ff77b15a10d in __libc_start_call_main (/nix/store/k7zgvzp2r31zkg9xqgjim7mbknryv6bs-glibc-2.39-52/lib/libc.so.6+0x2a10d) (BuildId: bc8ec5f3ac2561de8f08b232685038c7167bf4b7)
    #11 0x7ff77b15a1c8 in __libc_start_main_alias_1 (/nix/store/k7zgvzp2r31zkg9xqgjim7mbknryv6bs-glibc-2.39-52/lib/libc.so.6+0x2a1c8) (BuildId: bc8ec5f3ac2561de8f08b232685038c7167bf4b7)
    #12 0x40b584 in _start (/quickjs/qjs+0x40b584)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /quickjs/quickjs.c:28041 in js_build_module_ns
==277778==ABORTING

Fix in #568.


Image Visualizer

Currently, we need to convert images into HTML to display them with HTML Visualizer.
We need an Image Visualizer that can display images directly without any conversion.
The Image Visualizer should take the binary of an image as the input.

@shengquan-ni will check on this

#2111 solved this.


[Bug]: Icon for password on input widget overlaps with widget boundary

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Icon for password on input widget overlaps with widget boundary
Steps To Reproduce

Drag and drop an input widget on canvas
Change input type to password and zoom into the eye icon that appears on the input box

Public Sample App
No response
Environment
Deploy Preview
Severity
Low (Inconsistent UI/UX)
Issue video log
No response
Version
cloud

@ichik What we should do about this?

@KelvinOm could we reduce inner button size to fit?


Issue #1012. Added offsetAngle option for radar charts.

Solution for https://github.com/chartjs/Chart.js/issues/1012
Seems like with just a bit of refactoring to the polar chart's controller, this same thing could be applied to that chart as well.
I did not add any new tests for this, I think I will need some guidance on how best to test this.

I'm not sure of the best way to test this. One option is to create the scale object then fake out the sizing done by the chart and test that angles returned from getDistanceFromCenterForValue are correct.
Another option is to create an entire radar chart (no need to fake out the sizing) and then test the scale functions correctly.
I quickly looked at the existing tests and it looks like we have one for the getDistanceFromCenterForValue. https://github.com/chartjs/Chart.js/blob/master/test/scale.radialLinear.tests.js#L390 Since that didn't break, that's a good sign.
You could also try gulp coverage then look at the generated html report to see which lines aren't covered. That's also a good starting point for writing a test.
CC: @tannerlinsley @simonbrunel @derekperkins @zachpanz88

+1 to merge now that tests are written

Agreed


Unrecoverable error: rangecheck in .putdeviceprops

I am having the following problem:
$ ./pdfScale.sh -s 0.9 infile.pdf outfile.pdf
Unrecoverable error: rangecheck in .putdeviceprops
$ ./pdfScale.sh -v -s 0.9 infile.pdf outfile.pdf
pdfScale.sh v1.0.11 - Verbose execution
Checking dependencies
Scale factor: 0.9
Input file: infile.pdf
Output file: outfile.pdf
Width: 0 postscript-points
Height: 0 postscript-points
Translation X: 0
Translation Y: 0
Unrecoverable error: rangecheck in .putdeviceprops
I have no clue what's causing this or why. The input is a DIN-A4 PDF-File with 105 pages produced by pdfnup.

humm.. can you annex that PDF so I can test it?
Also, what system are you running this?
Height: 0 postscript-points
Translation X: 0
Translation Y: 0```
These should not be Zero, something is going wrong there...

I have an older Linux Mint (17.1) with a kernel 3.13 on an x86_64bit pc.
The cat/grep command returns no output.
I can't publish the original PDF here but I will try to find something that I can publish where I have the problem.

Ok, both files have been created with pdfnup, for the attached file I just removed all pages except for two before running pdfnup.
Output this time:
$ pdftk file.pdf cat 3-4 output /tmp/input.pdf
$ pdfnup /tmp/input.pdf
          ----
  pdfjam: This is pdfjam version 2.08.
  pdfjam: Reading any site-wide or user-specific defaults...
          (none found)
  pdfjam: Effective call for this run of pdfjam:
          /usr/bin/pdfjam --suffix nup --nup '2x1' --landscape -- /tmp/input.pdf -
  pdfjam: Calling pdflatex...
  pdfjam: Finished.  Output was to '/home/user/input-nup.pdf'.
$ ./pdfScale.sh -v -s 0.9 input-nup.pdf out.pdf
pdfScale.sh v1.0.11 - Verbose execution
Checking dependencies
  Scale factor: 0.9
    Input file: input-nup.pdf
   Output file: out.pdf
         Width: 0 postscript-points
        Height: 0 postscript-points
 Translation X: 0
 Translation Y: 0
Unrecoverable error: rangecheck in .putdeviceprops

input-nup.pdf
out.pdf
input.pdf

Ok, thanks for the info and for the files.
If the cat + grep command does not work, nothing else will work, since we won't have a valid paper size to base the resizing on.
The file input.pdf works fine here (it finds de MediaBox definition, and also resizes):
$ cat input.pdf | grep -a '/MediaBox'
/MediaBox [0 0 595.32 841.92]
/MediaBox [0 0 595.32 841.92]

$ pdfScale-master/pdfScale.sh -v -s 0.9 input.pdf
pdfScale.sh v1.0.11 - Verbose execution
Checking dependencies
  Scale factor: 0.9
    Input file: input.pdf
   Output file: input.SCALED.pdf
         Width: 595 postscript-points
        Height: 842 postscript-points
 Translation X: 33.055225
 Translation Y: 46.777310

File: input.SCALED.pdf
The file input-nup.pdf though has no MediaBox definition. So this one fails when greping for the page size. I have found a /BBox [0 0 595.32 841.92] on that file. That is an A4 size in postscript points. Honestly, I am not sure why you don't have a Mediabox on your pdf, but I am just now creating a version that will try to use /BBox if /Mediabox is not available and then abort execution if none of them is available.
I will post it soon.
Cheers!
Gus

Please try to use the new version on your problematic files and let me know.
pdfScale v1.1.3

Okok calm down I just got up and read this. I will try that now.

Just get the latest master or tag and try ;)
I went a long way talking too much.
I try not to do that, but well... I fail... LOL

Looks good to me at first glance.
However in case you want it to be very exact: Try scaling the output of pdfnup with -s 0.5. It's a little bit off center.
It seems to me as if it's always a little bit off center but too little to notice unless using a way to small parameter.
Doesn't bother me, just wanted to mention that.
I was just referring to the final edit where you said "Let me know so we can close this please."

Oh and with the -i parameter it works just fine. No offset there.

Now I want to know: Should I always use -i or just when it's offcenter without -i? And if I should always use it why make it a parameter and not include it or put the parameter the other way around (so it's enabled by default)?

Humm... the really weird thing is that the sizes rendered with and without imagemagick are the same. I mean, try posting the verbose output both with and without the -i parameter here.
I prefer not to use -i by default because it creates a new dependency.

Ok, not quite... the values are inverted in your file when not using identify:
$ ./pdfScale.sh -v -s 0.5 ../input-nup.pdf
pdfScale.sh v1.2.10 - Verbose execution
Checking for ghostscript and bcmath
  Scale factor: 0.5
    Input file: ../input-nup.pdf
   Output file: ../input-nup.SCALED.pdf
         Width: 595 postscript-points
        Height: 842 postscript-points
 Translation X: 297.500000
 Translation Y: 421.000000

$ ./pdfScale.sh -i -v -s 0.5 ../input-nup.pdf
pdfScale.sh v1.2.10 - Verbose execution
Checking for ghostscript and bcmath
Checking for imagemagick's identify
  Scale factor: 0.5
    Input file: ../input-nup.pdf
   Output file: ../input-nup.SCALED.pdf
         Width: 842 postscript-points
        Height: 595 postscript-points
 Translation X: 421.000000
 Translation Y: 297.500000

I wonder if I can just check which one is bigger and assign it to the width.

The BBox is really set like that, not sure now.
/BBox [0 0 595.32 841.92]

I couldn't find a way to determine if it is landscape either.
Can you please create a n-up file that is not landscape so I can check its BBox declaration?

Aright dude...
v1.3.1 is out. You made me think that I should be able to get the page size using ghostscript. I then found a solution but it involved an external PostScript file. I then embedded that file into my script and it is now working fine.
People can still use the older cat + grep method with the -c parameter.
This version should not require the -i or -c parameters to work correctly though. If anything, you will still have problems when using the -c option. I myself have use cases for -c, so I kept it.

Please let me know if the new version works well for you.
Cheers!

Ok, but I don't have the time right now (big test tomorrow), so I will try that in >36h. Sorry for the delay.

No worries

Please do not use version 1.3.x that are before 1.3.5.
The input file name was hard-coded there.
Fixed now.
Cheers!

ok... I was studying Postscript and trying to make something myself but hat is not going to well. Also, I am not sure about the license of the embedded script so I removed it.
What I did now, is to create an adaptive method for page sizing. It will try cat+grep, mdls, pdfinfo, identify, then fail. This way should be fine on most systems and people can always force a method with the -m parameter.
Your n-up file for example, failed for me on cat+grep but then went fine on mdls and pdfinfo (automatically). Basically, if there is no /MediaBox declaration on the PDF, the cat+grep method will fail, on which case you will need to have one of the other external options for the script to run. Having a single one of them available is enough.
You can get the new version here v1.4.5.


GAN

Hi, I want to ask what change I need to make to apply Diffusion-GAN to the other GAN repo ?

@jkla139 your URL is broken. (text is correct, href is to issues/url)
https://github.com/Zhendong-Wang/Diffusion-GAN#build-your-diffusion-gan


Swift 3

Update to Swift 3 syntax and Xcode 8 settings.

Thanks, the updates were just shipped in v0.4.0


Add mjml-python to ports, and update missing implementation for MRML.

Fixes #2815
MRML now supports mj-include, and so does mjml-python.

👍


Off by one on HoverInformation

When hovering the first character of a field the hover-information isn't working correctly

Hi, did this regress in https://github.com/0no-co/GraphQLSP/pull/209? It is an issue for me in 1.4.1


Add faucet for miden-node

The Miden node generates 2 accounts by default at genesis:

a Faucet account
a regular account

We need to make the faucet available and usable on node spin-up, this would enable request of funds and testing of tx's from users of the network.
In this PR I propose to add a new crate called faucet to the node workspace and use actix-web to create a web-server that will serve html, css and js static files.
The faucet receives requests of funds through an endpoint where users on the frontend pass their AccountId. The server responds with the note generated by the tx. The end user is then able to import this newly received note into their client and consume it locally.
closes: #215

Awesome! One question: would it make sense to add this feature into the miden-node crate so that the faucet could be started together with the node (maybe using a command line arg).
One benefit of this is fewer components for users to manage. The downside is potential extra complexity in the node. I'm not sure how these balance each other.


too many open files

After some time of the server running we get the following error when attempting to load the webpage:
Error 500 /opt/traccar/web/release.html (too many open files)

There are two things you should consider doing:

Increase the open files limit for Traccar service
Add resetDelay parameter to the config file for the protocol(s) that you use

Can you point me where to do this? and is there a recommended number to use? Ie. should it be based on how many cars are listed or reporting gps at any given time or how many users are logging into the web interface?
basically same questions on resetDelay. What should a normal range be based on?

Thank you.

Both. Each connection counts as an open file. It's an operating system setting.

The value is in seconds. I would recommend to pick a value slightly bigger than your device reporting interval.

Hello tananaev,
I am using traccar 3.5 release with around 71 devices. It's been months I am monitoring traccar behavior trying to find the solution to this problem. Our devices feed interval are set to 10 sec, I tried configuring resetDelay slightly higher this interval to 15 seconds. But with this configuration, "too many files open" error occurs within couple of days. Then I tried with 2 mins, this problem occurs in a week. And also I tried with 0 sec, its take couple of weeks to re-occur this problem.
I cannot understand why is this happening. Please help me out.
Thanks

@mani-rai, what you are describing doesn't make any sense. The lower timeout you set, the longer it should be till you get to the limit (ideally never). I suspect you have some problem with config.

@tananaev, I am using devices of tk103 protocol. My additional configuration was just
<entry key='tk103.resetDelay'>120</entry>
And my user soft limit is 4000. Let me check with operating system limit once.

You have it set up to 2 minutes. If your devices report every 10 seconds, it doesn't make sense to put such a long period.
Also, you should be using "timeout" parameter instead of "resetDelay". Both mean the same thing, but the second one will be eventually removed.
You should check file limit for the Traccar process.


Fix hoisting of functions in blocks

We prepended to the first block we saw after the FunctionDeclaration
which is not correct. We should prepend the function to the current
block.
Fixes #1627

PTAL

LGTM


Implement negative Bitmap height

Checklist

[x] Check that you've added passing tests and documentation
[x] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc) if your changes affect the public API
[x] Run rustfmt on the project
[x] Run just build (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
This is still a WIP MR, but I'm opening it in this state as an opportunity for early feedback.
So far, I've:

identified the issue
Written a failing test

I intend to:

change the type of height to i32, from u32.
If the height is negative, then mutate the representation of the order of rows in memory (maybe need to just not change the order, as it is default for BMPs to be "upside down")

Fixes #8

@jamwaffles Great timing :wink:

@rfuest should we add #[non_exhaustive] to Header?

I'm not sure if there will be more changes to the Header struct without also breaking other parts of the API, but I guess it couldn't hurt to add the attribute.

Great timing :wink:

That is uncanny, especially when our review comments were nearly identical :joy:

I'm not sure if there will be more changes to the Header struct without also breaking other parts of the API, but I guess it couldn't hurt to add the attribute.

The only issue I can see is someone wanting to create a BMP from some raw parts, but it's easy enough to add that as a from_raw_parts method later instead of constructing a Header manually.

The only issue I can see is someone wanting to create a BMP from some raw parts, but it's easy enough to add that as a from_raw_parts method later instead of constructing a Header manually.

OK. But at the moment Header is pretty much useless on its own, because no struct that uses a Header can be constructed without parsing a BMP file.

Almost there...

Great! Just let us know when it is ready to review.

I'm not sure why the changelog has a conflict...  that's odd.

I'm not sure why the changelog has a conflict... that's odd.

No idea, but I've resolved the conflict now.

Ok, so if everything looks good, I'll collapse the commits.

Ok, so if everything looks good, I'll collapse the commits.

We are using "Squash and merge" to merge PRs, so there is no need to collapse the commits.

Of course!

This is released in 0.3.0-beta.2.


Calculate AchievementProgress

The progress of the Achievement should be calculated and set by the progress of all AchievementConditions needed for the Achievement.
The progress should be shown in the AchievementControl.

[x] The progress of the Achievement should be calculated and set by the progress of all AchievementConditions needed for the Achievement.


Added normalization of documents to support multlline completions

In order to support completions of docs when there are multiple lines, documents are normalized and the cursor position is recomputed to account for the normalization.
https://github.com/user-attachments/assets/01ca3a0a-2944-434e-b559-caa7cd964dd9
It is not shown in the video, but the way this is implemented allows for an arbitrary amount of newlines without breaking completions. For example,
SELECT
*
FROM
<table>

WH

would still return a completion.

Thanks for the PR, I will review later today 🚀
P.S. Just in case you thought it was a typo, us British folk typically use s instead of z 😆

Thanks for the PR, I will review later today 🚀
P.S. Just in case you thought it was a typo, us British folk typically use s instead of z 😆

Yikes! Tell you I'm uncultured without telling you I'm uncultured 😳


Added note about client side property casing.

This just tripped up someone in the workshop.
/cc @Particular/microservices-workshop-maintainers

Nice! :-)


id_token support (OpenID)

Is there a particular reason for this library not to return the id_token when present aside the access and refresh token?

I was also looking for the ID token today and found it was missing.

Me too, so I've raised a PR ^

Hi there!
Sorry for the delay here, but as I wrote in #29 (unfortunately without linking back to this issue), I didn't originally intended for this module to be an OpenID Connect client.
Since then, I came to the conclusion that it would probably be best to open up the OAuth2 Grant classes to allow other classes to inherit from them and override or extend their functionality (as would be required for proper OIDC support).
I've also started working on a more proper OIDC client that uses this mechanism to add spec-conformant handling of ID tokens, among other extensions that OIDC makes to the OAuth2 spec.
If you're interested in that, check out my latest comment in #29. Any feedback would be much appreciated!


Fixing default chevron icon appearance in accordion component

Task: task-1017791
Link: preview
There is an issue with the default chevron icon appearance in the accordion component (check the screenshot below). This PR fixes it
Testing

Visit accordion page
Inspect component, in dev tool disable "display: flex" rule for .accordion summary. You should not see the default chevron showing up

Here is the same but in main branch:

Additional information

Contributor checklist

[x] Did you update the description of this pull request with a review link and test steps?
[x] Did you submit a changeset? Changesets are required for all code changes.
[ ] Does your pull request implement complex UI logic (js/ts)? Consider adding an integration test to test your user flow.
[ ] Does your pull request add a new page to the documentation site? Add your new page for automated accessibility testing in /integration/tests/accessibility.
[ ] Does your pull request change any transforms? Did you test they work on right to left?

@ollips  The preview link in the description isn't working for me.

@ollips The preview link in the description isn't working for me.

Oh, that's right, you need to run it locally. I'll update test steps in a sec to reflect that


OQB Rep 2 Cover node sending patchy data to gateway

Node has gaps in data on the chart view. Seems to not be sending back logged data to the gateway

#168


Get Coriolis matrix

Is there a way to get the coriolis matrix using this approach? I realize that we can get the "C(q, qdot) * qdot" term using the inverse dynamics function, but is there a way to isolate just the C(q, qdot) matrix?

It is not implemented yet.

Thanks for the update!


Update configure-the-operational-data-feed-for-power-bi-with-biztalk-…

…server.md
Send my feedback with the changes and mainly it necessary to change "http://localhost/OperationalDataService" with "http://localhost/BizTalkOperationalDataService" because the first is incorrect

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/configure-the-operational-data-feed-for-power-bi-with-biztalk-server.md
:white_check_mark:Succeeded
View

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.


UI Issue

When a user creates a super long name the point buttons spread out instead of staying flex-start

THIS BEHAVIOUR IS INTENDED GO AWAY.

kidding, you should be able to change the styling in the main_page.tmpl.html

Added a PR for this https://github.com/joelfooxj/go_pointing_poker/pull/3


(#9) Configure to deploy to maven

For #9 :

Configure rultor file

@rultor merge

@rultor merge

@rultor merge

@baudoliver7 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@baudoliver7 Oops, I failed. You can see the full log here (spent 15s)
+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 4494
+ echo '1.69.1 839260fab'
1.69.1 839260fab
+ date
Thu Jul 22 23:48:15 CEST 2021
+ uptime
 23:48:15 up 298 days,  7:54,  0 users,  load average: 4.46, 4.60, 4.64
+ cat
.rultor.yml in yegor256/home doesn't allow Minlessika/takes-utils to use its assets (there are 111 friends), see http://doc.rultor.com/reference.html#assets
+ exit -1
'cid' file is absent, container wasn't started correctly

@rultor merge

@baudoliver7 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@baudoliver7 Oops, I failed. You can see the full log here (spent 15s)
+ set -e
+ set -o pipefail
++ dirname ./run.sh
+ cd .
+ echo 5060
+ echo '1.69.1 839260fab'
1.69.1 839260fab
+ date
Thu Jul 22 23:51:14 CEST 2021
+ uptime
 23:51:14 up 298 days,  7:57,  0 users,  load average: 4.46, 4.58, 4.63
+ cat
.rultor.yml in yegor256/home doesn't allow Minlessika/takes-utils to use its assets (there are 111 friends), see http://doc.rultor.com/reference.html#assets
+ exit -1
'cid' file is absent, container wasn't started correctly


Kayobe container image build may fail if collections or roles need updating

Steps to reproduce

checkout kayobe-config repo
add some roles or collections to etc/kayobe/ansible/requirements.yml
run kayobe control host bootstrap
modify role or collection version requirements in requirements.yml
build kayobe container image

Expected
Image builds successfully
Actual
Image fails to build. The command that fails is kayobe control host bootstrap. It fails because the roles or collections in etc/kayobe/ansible/{roles,collections} are of an incompatible version.
Workaround
Remove the roles and collections before building the image.
I also used this change to the Dockerfile which worked, but would flatten any non-galaxy roles/collections:
diff --git a/docker/kayobe/Dockerfile b/docker/kayobe/Dockerfile
index 786a541..e0c83fc 100644
--- a/docker/kayobe/Dockerfile
+++ b/docker/kayobe/Dockerfile
@@ -73,6 +73,7 @@ RUN --mount=type=ssh,uid=1000 --mount=type=tmpfs,target=/tmp/src --mount=type=bi
   sudo chown -Rf stack:stack /tmp/src && \
   # Strip the secrets so that we don't need to pass in a vault-password
   grep -lR "\$ANSIBLE_VAULT" /tmp/src | xargs rm -f && \
+  (rm -rf /tmp/src/etc/kayobe/ansible/roles/* /tmp/src/etc/kayobe/ansible/collections/* || true) && \
   bash /tmp/src/.automation/utils/kayobe-automation-install && \
   (rm -f /stack/.ssh/{id_rsa,id_rsa.pub} || true) && \
   (mkdir /stack/.ansible || true) && \

Interesting issue. I did look at the differences between kayobe control host bootstrap and kayobe control host upgrade and the differences seem to be:

upgrade uses ansible-galaxy force install
bootstrap generates the openrc files

Could we potentially add a build ARG to switch the behaviour to use upgrade instead of bootstrap? It is a little concerning that we'd make destructive modifications to a users git checkout by default (if we switched to upgrade by default). It can also be quite useful to not force delete the roles if you are testing some changes. What do you reckon?


Fix generating proxy templates in helm-chart

helm-generate.sh failed to generate proxy template because of incorrect path. This PR fixes this.
The logging is missing /.
For example, this line supposed to be
Generating csi templates into /Users/shawnsun/desktop/alluxio/integration/kubernetes/helm-chart/alluxio/csi/, but it looks like
Generating csi templates into /Users/shawnsun/desktop/alluxio/integration/kubernetes/helm-chart/alluxiocsi/

alluxio-bot, merge this please

@ssz1997 Do you think some doc change is needed to mention now we have proxy enabled in k8s?


[POC][F] Build EntitySelectorDisclosure

@jen-castiron I think the combo typehead + entity selector works! Let me know if you see performance or UX issues.
Can preview here.

@1aurend I'm having trouble using the typeahead - if I type too fast, it doesn't register all the key strokes. I'll see if I can find the root of the problem, because I really want to get this in!

@1aurend I'm having trouble using the typeahead - if I type too fast, it doesn't register all the key strokes. I'll see if I can find the root of the problem, because I really want to get this in!

We did something on Scarecrow that improved the way state was handled, and I remember thinking it should get ported back over here. But, of course, I don't remember what now...


Update wallet-adapter examples readme

@randall-Mysten Your comment was not taken into the #4538 as the PR was auto-merged before I could pull the changes. I ported those same changes here!

Awesome, thanks for the follow up.


Fix literal rounding in codegen functions

Fixes the failing test case in #1815. Added a test that would otherwise fail without the change.

Is it possible to add a to_string unit test as well?
TEST_CASE(literal_to_string_float_precision)
{
    migraphx::literal x{1 / 0.00787402f};
    EXPECT(x.to_string() == "126.99993142003703");
}

This can probably go in test/literal_test.cpp.

migraphx::literal x{1 / 0.00787402f};
    EXPECT(x.to_string() == "126.99993142003703");

Seems like this test fails because the string value doesn't match directly. I think I will rewrite it to prevent rounding.


MethodDecorator gets TS2322/TS2315

TypeScript Version:  2.4.2
Code
export function route(method: string, path: string): MethodDecorator {
    return (target: any, name: string, descriptor: TypedPropertyDescriptor<Function>) => {
        routeManager.regeisterRoute({
            constructor: target.constructor,
            function: descriptor.value,
            method,
            path
        });
    }
}

export function route(method: string, path: string): MethodDecorator<Function> {
    return (target: any, name: string, descriptor: TypedPropertyDescriptor<Function>) => {
        routeManager.regeisterRoute({
            constructor: target.constructor,
            function: descriptor.value,
            method,
            path
        });
    }
}

Expected behavior:
No error
Actual behavior:
error TS2322: Type '(target: any, name: string, descriptor: TypedPropertyDescriptor<Function>) => void' is not assignable to type 'MethodDecorate
or'.
  Types of parameters 'descriptor' and 'descriptor' are incompatible.
    Type 'TypedPropertyDescriptor<T>' is not assignable to type 'TypedPropertyDescriptor<Function>'.
      Type 'T' is not assignable to type 'Function'.

error TS2315: Type 'MethodDecorator' is not generic.

For the first one, it is caused by stricter generic checks (#16368).
It seems you have to define a CustomMethodDecorator first, since there is no generic on MethodDecorator, and its declaration requires a non-constraint generic.
Or just enable the --noStrictGenericChecks option.

For the second one, there is indeed no generic on MethodDecorator, the error message seems a little weird.
(lib.d.ts)
declare type MethodDecorator = <T>(target: Object, propertyKey: string | symbol, descriptor: TypedPropertyDescriptor<T>) => TypedPropertyDescriptor<T> | void;

CustomMethodDecorator or --noStrictGenericChecks will works, but I think it should infer T as Function in case 1, or make MethodDecorator generic in case 2.

MethodDecorator is defined as <T>(...something: SomeType[]): SomeOutput<T>;
So that it'll accept all kinds of T without any constraint, but in your case 1 it only accept Function, it is indeed not compatible from stricter perspective.
NOTE: With --noStrictGenericChecks enabled, all generics will be considered any before comparing.

make MethodDecorator generic in case 2.

I'm not familiar with that MethodDecorator type, it'd be better to raise another issue for that things.

There is no requirements that decorators return one of the predefined decorator types in lib.d.ts.
I would strongly advise against enabling --noStrictGenericChecks and instead using the following (which is actually has a richer type but passes the checker thanks to inference.
declare const routeManager: {
  regeisterRoute: (config: {
    constructor?: Function,
    function?: Function,
    method: string,
    path: string
  }) => void;
};

export function route(method: string, path: string) {
  return <T extends object, K extends keyof T, R extends Promise<any>>(target: T, name: keyof T, descriptor: TypedPropertyDescriptor<() => R>) => {
    routeManager.regeisterRoute({
      constructor: target.constructor,
      function: descriptor.value,
      method,
      path
    });
  };
}

class A {
  @route('GET', 'api/values') m() {
    return Promise.resolve(['a', 'b']);
  }
}

Not only will this pass, but the decorator will validate that the method actually returns a Promise for a resource (just an example).
Trying to exactly match the predefined decorator types would preclude this very useful typechecking behavior.

Well, it works. Just like what @ikatyang says to define a CustomMethodDecorator.

@Norgerman Ah, I missed that comment. Sorry for being redundant.

In case somebody needs an exact code, his is how I solved this issue in my project:
interface MethodDecoratorCustom<Instance extends object = object> {
    (
        target: Instance | Constructor<Instance>,
        key: PropertyKey,
        descriptor: TypedPropertyDescriptor<Method<Instance | Constructor<Instance>>>,
    ): void | TypedPropertyDescriptor<Method<Instance | Constructor<Instance>>>;
}

(See playground for definitions of Method and Constructor, as well as for a usage example.)


GNN-based Meta-Learning for Sparse Portfolio Optimization

Hello,
Let me start by saying that I am a fan of your work here. I have recently open-sourced by GNN-based meta-learning method for optimization. I have applied it to the sparse index-tracking problem from real-world (after an initial benchmarking on Schwefel function), and it seems to outperform Fast CMA-ES significantly both in terms of producing robust solutions on the blind test set and also in terms of time (total duration and iterations) and space complexity. I include the link to my repository here, in case you would consider adding the method or the benchmarking problem to your repository. Note: GNN, which learns how to generate populations of solutions at each iteration, is trained using gradients of the loss function, as opposed to black-box algorithms here.

Sincerely, K

Hi @kayuksel! I saw your post on Reddit I think, very cool stuff :) we'd always love to integrate new techniques into EvoTorch, so if you have time, maybe you'd consider reformatting your code to work in the EvoTorch searchalgorithm API and open a pull request? Otherwise I'll see if I can get to doing that myself when I find time.

Thank you very much, Timothy. Would it help you if I provided a highly simplified version (~75 lines) as below? FYI, I also created a game-theoretic or adversarial version of the generative model for global optimization that utilizes a positive surprise mechanism obtained through a surrogate model (critic) trained simultaneously with Adaptive Gradient Clipping. It is quite competitive against the best optimizers in Nevergrad in large-scale non-convex optimization problems, especially with noisy rewards. Note: The only bottleneck seems to be the random seed sensitivity, for which GradInit (arxiv.org/abs/2102.08098) seems to be a solution.
import torch
import torch.nn as nn

def schwefel(x):
    x = x * 500
    return 418.9829 * x.shape[1] - (x * x.abs().sqrt().sin()).sum(dim=1)

def init_weights(model):
    for m in model.modules():
        if isinstance(m, nn.BatchNorm1d):
            m.weight.data.fill_(1)
        elif isinstance(m, nn.Linear):
            nn.init.xavier_uniform_(m.weight, gain = 5/3)
        if hasattr(m, 'bias') and m.bias is not None: m.bias.data.zero_()

class LSTMModule(nn.Module):
    def __init__(self, input_size = 1, hidden_size = 1, num_layers = 2):
        super(LSTMModule, self).__init__()
        self.rnn = nn.LSTM(input_size, hidden_size, num_layers, batch_first=True)
        self.h = torch.zeros(num_layers, 1, hidden_size, requires_grad=True).cuda()
        self.c = torch.zeros(num_layers, 1, hidden_size, requires_grad=True).cuda()
    def forward(self, x):
        self.rnn.flatten_parameters()
        out, (h_end, c_end) = self.rnn(x, (self.h, self.c))
        self.h.data = h_end.data
        self.c.data = c_end.data
        return out[:,-1, :].flatten()

class Extractor(nn.Module):
    def __init__(self, latent_dim, ks = 5):
        super(Extractor, self).__init__()
        self.conv = nn.Conv1d(args.noise, latent_dim,
            bias = False, kernel_size = ks, padding = (ks // 2) + 1)
        self.conv.weight.data.normal_(0, 0.01)
        self.activation = nn.Sequential(nn.BatchNorm1d(
            latent_dim, track_running_stats = False), nn.Mish())
        self.gap = nn.AvgPool1d(kernel_size = args.batch, padding = 1)
        self.rnn = LSTMModule(hidden_size = latent_dim)
    def forward(self, x):
        y = x.unsqueeze(0).permute(0, 2, 1)
        y = self.rnn(self.gap(self.activation(self.conv(y))))
        return torch.cat([x, y.repeat(args.batch, 1)], dim = 1)

class Generator(nn.Module):
    def __init__(self, noise_dim = 0):
        super(Generator, self).__init__()
        def block(in_feat, out_feat):
            return [nn.Linear(in_feat, out_feat), nn.Tanh()]
        self.model = nn.Sequential(
            *block(noise_dim+args.cnndim, 480), *block(480, 1103), nn.Linear(1103, args.funcd))
        init_weights(self)
        self.extract = Extractor(args.cnndim)
        self.std_weight = nn.Parameter(torch.zeros(args.funcd).cuda())
    def forward(self, x):
        mu = self.model(self.extract(x))
        return mu + (self.std_weight * torch.randn_like(mu))

actor = Generator(args.noise).cuda()
opt = torch.optim.AdamW(filter(lambda p: p.requires_grad, actor.parameters()), lr=1e-3)
best_reward = None
for epoch in range(args.iter):
    torch.cuda.empty_cache()
    opt.zero_grad()
    z = torch.randn((args.batch, args.noise)).cuda().requires_grad_()
    rewards =  schwefel(actor(z).tanh())
    min_index = rewards.argmin()
    if best_reward is None: best_reward = rewards[min_index]
    actor_loss = rewards.mean()
    actor_loss.backward()
    nn.utils.clip_grad_norm_(actor.parameters(), 1.0)
    opt.step()
    with torch.no_grad():
        if rewards[min_index] > best_reward: continue
        best_reward = rewards[min_index]
        print('epoch: %i loss: %f' % (epoch, best_reward.item()))

It is also good and applicable for combinatorial optimization, simply by sampling with the sigmoid of logits from the generator.

Hello again,
I've made a quick comparison on 30-dim Schwefel (as the hardest math function) against Nevergrad here.
In my experiments, it easily scales to 100K+ dimensions in most other optimization benchmark functions.
https://github.com/kayuksel/genmeta-vs-nevergrad
Sincerely, K

fyi, I have also applied it to MovieLens 1M matrix factorization problem (with 500K parameters), the codes are in the repository.


Trains cannot track special argument values

When an argument is of dictionary type as follow:  {'a > 5': 0.4},
Trains raises an exception and stops documenting the following arguments in 'hyperparameters' tab

Bug verified, the trains-server  does not allow white-space in the hyper-parameters.
It will be fixed in the next trains-server release.
Until then, a workaround would be removing the spaces:
 {'a>5': 0.4},

Hey Martin,
Thank you for your fast response!
The example I gave you wasn’t quite similar to my use case. Actually, I have a dictionary argument which contains both whitespaces and ‘.’ Character.
This argument looks like:
{‘a > 0.5’: ‘0.4’}
Will you have a fix for that constraint (of special characters ‘.’) as well in the next release?
Best regards,
Eyal

On 19 Dec 2019, at 9:59, Martin.B<EMAIL_ADDRESS>wrote:
Bug verified, the trains-server does not allow white-space in the hyper-parameters.
It will be fixed in the next trains-server release.
Until then, a workaround would be removing the spaces:
{'a>5': 0.4},
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/allegroai/trains/issues/69?email_source=notifications&email_token=AOCM745XVYUAM33JOZ74QXLQZMSXHA5CNFSM4J4VVK3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHIYNKQ#issuecomment-567379626, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOCM746RKPLZYFNAJOEC3KTQZMSXHANCNFSM4J4VVK3A.

Yes @eyalziv, currently  " ", "." and "$"  (space, dot, dollar_sign) are not supported in a hyper-parameter key. All three restrictions will be fixed in the next trains-server release :)

Hi @eyalziv ,
Trains and Trains-Server v0.13.0 were just released, with no restriction on hyper-parameter keys.


Jiach/use x ms date for hmac

It looks like there is something wrong with the AAD recordings a lot of them were just emptied


cachix does not build (broken dependency) blocking nixpkgs-unstable

Describe the bug
https://hydra.nixos.org/build/134154387
To Reproduce
Steps to reproduce the behavior:

...
...
...

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

fixed by https://github.com/NixOS/nixpkgs/pull/108362

fixed by https://github.com/NixOS/nixpkgs/pull/108362


Request invoked too often

Just tried the updated version and I'm getting.
06-13 01:52 telethon.telegram_bare_client INFO     Invoking ['GetHistoryRequest(peer=InputPeerChannel(channel_id=1111111, access_hash=-11111111), offset_id=0, offset_date=None, add_offset=0, limit=50, max_id=0, min_id=31, hash=0)'] failed 1 times, connecting again and retrying 06-13 01:52 telethon.telegram_bare_client INFO     Reconnection aborted: already connected 06-13 01:52 telethon.telegram_bare_client WARNING  Request invoked too often, wait 23s
Have changed, channel_id and Access_hash.
Seems to only do it on this updated one.

Maybe you just tried many consecutive times? Or trying have multiple telebagger instances working at same time?

Hmm yeah could be multiple instances. Is there a way to run multiple
instances without being invoked?
On Wed, 13 Jun. 2018, 8:08 am tomillo<EMAIL_ADDRESS>wrote:

Maybe you just tried many consecutive times? Or trying have multiple
telebagger instances working at same time?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/davidmckenzie/telebagger/issues/3#issuecomment-396772854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOjtC3vBUHdAvtLwZ9lvyxm6_2ZWDzP2ks5t8FgWgaJpZM4UlU29
.

Would there be a way to have multiple channelids and webhooks in the one bot?
So config file have
"Channels": {       "Channelid1":"webhook1",       "Channelid2":"webhook2",       "Channelid3":"webhook3",       "Channelid4":"webhook4",       "Channelid5":"webhook5"
Something like that?

I would also like that function to be developed by @davidmckenzie  🤓


Question: How do you run stylelint?

I see that that stylelint is a devDependency here, but I cannot figure out how it get's run... What am I missing?
Thanks for the sweet starter pack! 🥂

You should be good with this: https://github.com/styled-components/stylelint-processor-styled-components
I would say style linting is not in the scope of this starter / boilerplate. One should remove the dependency to stylelint and drop the file .stylelintrc. I guess this is a leftover from the starter this starter was forked from. In order to avoid this kind of question in the future, all mention of stylelint should be purged from the source. How you lint your styles is not important to show people how to build a documentation static webpage with gatsby.

IMO Issue can be closed as wontfix.

@ericwindmill I got the answer but I think this is still unresolved because as @Levino said:

One should remove the dependency to stylelint and drop the file .stylelintrc.

Or a new issue should be created...

@Levino Even better.. created #20


Wrap useSearchParams in Suspense

Next 14 and later require useSearchParams to have a Suspense boundary. It's not a problem when running yarn dev, but to make a release with yarn run build, you get this error:
   Generating static pages (0/4)  [    ] ⨯ useSearchParams() should be wrapped in a suspense boundary at page "/". Read more: https://nextjs.org/docs/messages/missing-suspense-with-csr-bailout
    at a (/Users/bcotton/guacsec/guac-visualizer/.next/server/chunks/288.js:1:10576)
    at h (/Users/bcotton/guacsec/guac-visualizer/.next/server/chunks/288.js:1:22225)
    at /Users/bcotton/guacsec/guac-visualizer/.next/server/app/page.js:247:8556
    at lx (/Users/bcotton/guacsec/guac-visualizer/.next/server/app/page.js:247:9491)
    at nM (/Users/bcotton/guacsec/guac-visualizer/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:47419)
    at nL (/Users/bcotton/guacsec/guac-visualizer/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:64674)
    at nI (/Users/bcotton/guacsec/guac-visualizer/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:46806)
    at nM (/Users/bcotton/guacsec/guac-visualizer/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:47570)
    at nM (/Users/bcotton/guacsec/guac-visualizer/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:61663)
    at nL (/Users/bcotton/guacsec/guac-visualizer/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:64674)

Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error

 ✓ Generating static pages (4/4)

> Export encountered errors on following paths:
	/page: /
error Command failed with exit code 1.

In Next 14, you can disable this check, which I'll do for now because it seems to work. But Next 15 and higher will not support disabling it, so it's something we'll want to fix eventually.
It appears the only place that useSearchParams is called is in hooks/useGraphData.ts (in the eponymous function). That function only gets called in app/page.tsx. I tried guessing at a few places to put <Suspense> tags, but none of them eliminated the error.

I’m interested in taking on these issues. Could you please assign them to me? I would be happy to work on them.


OutboundTrafficPolicy to REGISTRY_ONLY doesn't appear to work in 1.10.3 version

Bug Description
Hi,
In Istio 1.10.3, setting the OutboundTrafficPolicy to REGISTRY_ONLY doesn't appear to take effect as it is not blocking external traffic. I can still do curl requests to external sites within the cluster succesfully. However, the configmap istio-1-10-3 is updated in istio-system with this configuration. Please note that, we followed the istio operator installation method.
Please let me know if there are any other configurations that we should consider updating to block all external traffic out of service registry scope.
Version
istioctl version
client version: 1.10.3
control plane version: 1.10.3
data plane version: 1.10.3 (9 proxies)
-------------------------------
kubectl version
Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.0", GitCommit:"af46c47ce925f4c4ad5cc8d1fca46c7b77d13b38", GitTreeState:"clean", BuildDate:"2020-12-08T17:59:43Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.13", GitCommit:"2444b3347a2c45eb965b182fb836e1f51dc61b70", GitTreeState:"clean", BuildDate:"2021-11-18T00:11:26Z", GoVersion:"go1.15.15", Compiler:"gc", Platform:"linux/amd64"}

Additional Information
bug-report.tar.gz

bug-report.tar.gz

Is the workload you are curling from injected with 1-10-3 revision?

Is the workload you are curling from injected with 1-10-3 revision?

@bianpengyuan Yes, the workload is injected with 1.10.3 envoy proxy. Still it is connecting to external domains.

I am mainly wondering if it is injected with 1-10-3 revision. since that is the configmap name suggests istio-1-10-3. It should be reflected in the pod spec.

I am mainly wondering if it is injected with 1-10-3 revision. since that is the configmap name suggests istio-1-10-3. It should be reflected in the pod spec.

@bianpengyuan  I have attached the pod description of the workload from where the curl was tried. It has 1.10.3 sidecar container. Please let me know if you notice any discrepancy.

Ok that looks good to me. Can you collect envoy access log for the curl request? https://istio.io/v1.10/docs/tasks/observability/logs/access-log/

Also have you tried restarting istiod pod? Does that fix the issue?

Also have you tried restarting istiod pod? Does that fix the issue?

Restarting istiod pods didn't make any difference. Interestingly I am not able to see access logs for my curl requests in the pod. Access log is enabled in the mesh config and the ingress gateway logs are printing access logs in the same cluster. I am checking this further.

Hello @bianpengyuan @praseed44 , I'm experiencing the same issue as is described here, but in my case, until istio version 1.11.7 the external traffic is blocked, when Sidecar CR setting outboundTrafficPolicy with REGISTRY_ONLY is present in the namespace.
My scenario is the following:
I have two workloads, server and a client, the server serves simple http service and client requests the service via istio-ingressgateway by using the following command:
(<IP_ADDRESS> is IP of istio-ingressgateway service)
$ curl -sD - -o /dev/null -Hhost:vmi-server.example.com http://<IP_ADDRESS>
HTTP/1.1 200 OK

The client has istio-proxy sidecar and the server doesn't have istio-proxy sidecar.
DestionationRule:
apiVersion: networking.istio.io/v1beta1
kind: DestinationRule
metadata:
  name: vmi-server-dr
spec:
  host: vmi-server
  subsets:
  - labels:
      version: v1
    name: v1

apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: vmi-server-vs
spec:
  gateways:
  - vmi-server-gw
  hosts:
  - vmi-server.example.com
  http:
  - match:
    - uri:
        prefix: /
    route:
    - destination:
        host: vmi-server
        port:
          number: 4200
        subset: v1

apiVersion: networking.istio.io/v1beta1
kind: Gateway
  name: vmi-server-gw
spec:
  selector:
    istio: ingressgateway
  servers:
  - hosts:
    - vmi-server.example.com
    port:
      name: http
      number: 80
      protocol: HTTP

apiVersion: networking.istio.io/v1beta1
kind: Sidecar
metadata:
  name: registry-only-sidecar
spec:
  outboundTrafficPolicy:
    mode: REGISTRY_ONLY

Service for the server:
apiVersion: v1
kind: Service
metadata:
  name: vmi-server
spec:
  internalTrafficPolicy: Cluster
  ipFamilies:
  - IPv4
  ipFamilyPolicy: SingleStack
  ports:
  - port: 4200
    protocol: TCP
    targetPort: 4200
  selector:
    app: vmi-server-app
  sessionAffinity: None
  type: ClusterIP
status:
  loadBalancer: {}

Service for the client:
apiVersion: v1
kind: Service
metadata:
  name: istio-vmi-app-service
spec:
  internalTrafficPolicy: Cluster
  ipFamilies:
  - IPv4
  ipFamilyPolicy: SingleStack
  ports:
  - port: 1500
    protocol: TCP
    targetPort: 1500
  selector:
    app: istio-vmi-app
  sessionAffinity: None
  type: ClusterIP
status:
  loadBalancer: {}

Istio-proxy logs on the client that runs the request, I see records showing allow_any
[2022-03-08T11:36:32.752Z] "GET / HTTP/1.1" 200 - via_upstream - "-" 0 12 1 1 "-" "curl/7.42.1" "05e02332-320d-96bb-b439-c8ed18a3b73b" "vmi-server.example.com" "<IP_ADDRESS>:80" PassthroughCluster <IP_ADDRESS>:54674 <IP_ADDRESS>:80 <IP_ADDRESS>:53628 - allow_any
[2022-03-08T11:36:33.040Z] "GET / HTTP/1.1" 200 - via_upstream - "-" 0 12 1 1 "-" "curl/7.42.1" "249c0f30-b0c6-90ae-b534-4e1f20e0d9b4" "vmi-server.example.com" "<IP_ADDRESS>:80" PassthroughCluster <IP_ADDRESS>:54674 <IP_ADDRESS>:80 <IP_ADDRESS>:53630 - allow_any

Which IIUC should not be the case with the Sidecar setting OutboundTrafficPolicy to REGISTRY_ONLY and no explicit ServiceEntry allowing the host.
Thanks in advance for looking over the case, let me know what else I could provide

@howardjohn Hello, can I ask for a visibility on this?


Allow the user to pass arbitrary scala compiler options

With the DSE java-driver, we need to pass -Ybreak-cycles to the scala compiler.  Instead of hardcoding that option in ZincCompiler, this PR introduces a new command line parameter: -co or --compilerOptions that allows arbitrary options to be passed.
A new environment variable has been added in gatling.sh: ZINC_OPTIONS.  If it is set, then its content is used as the value for -co.  I added it because I think compiler arguments should not be passed in $@ and end up in the real Gatling invocation.
Q: in CommandLineConstants.scala, two long parameters are in kebab-case, while the third one (related to the compiler) is in camelCase.  I did not know which one to choose.  Should I favour consistency with other compiler parameters (camel) or with the majority of parameters (kebab)?
I tested this code locally as follows:
$ ./bin/gatling.sh
GATLING_HOME is set to /tmp/gatling
13:18:00.810 [WARN ] i.g.c.ZincCompiler$ - Pruning sources from previous analysis, due to incompatible CompileSetup.
13:18:02.753 [ERROR] i.g.c.ZincCompiler$ - /tmp/gatling/user-files/simulations/ExampleDseSimulation.scala:11: illegal cyclic reference involving class Cluster
13:18:02.754 [ERROR] i.g.c.ZincCompiler$ -   private val cluster = DseCluster.builder().addContactPoint("<IP_ADDRESS>").build()
13:18:02.754 [ERROR] i.g.c.ZincCompiler$ -                                    ^
13:18:02.773 [ERROR] i.g.c.ZincCompiler$ - /tmp/gatling/user-files/simulations/ExampleDseSimulation.scala:13: illegal cyclic reference involving class Cluster
13:18:02.773 [ERROR] i.g.c.ZincCompiler$ -   private val statement: PreparedStatement = session.prepare("SELECT now() FROM system.local;")
13:18:02.773 [ERROR] i.g.c.ZincCompiler$ -                                                      ^
13:18:05.911 [ERROR] i.g.c.ZincCompiler$ - two errors found
13:18:05.913 [ERROR] i.g.c.ZincCompiler$ - Compilation crashed
[...]

$ ZINC_OPTIONS="-Ybreak-cycles" ./bin/gatling.sh
GATLING_HOME is set to /tmp/gatling
13:29:53.734 [WARN ] i.g.c.ZincCompiler$ - Breaking cycle in base class computation of class Cluster in com.datastax.driver.core (List(class Cluster, trait Closeable, trait AutoCloseable, class Object, class Any))
[...]
Choose a simulation number:
[...]

Actually, could you please create a PR against master. This is a new feature, and we'll also have to port it into our maven plugin.
Also, could you please add some documentation, typically in configuration.rst.

Ok let me try and rewrite that on top of master.  One question thought, as of commit f91728962, master does not compile on my machine.
[error] (gatling-compiler / Compile / compileIncremental) Compilation failed```

My understanding is that you are working on the compiler as well.  So it is not safe for me to go back to a previous commit and work from there.  That would most likely introduce merge conflicts.  Could you let me know when you are done?

master does not compile on my machine

Really? Works for me and Travis too.

Replaced by #3498