[Support] Using default json, Homebridge has an error of "state_cmd" missing and restarts

Describe Your Problem:
I am having an issue setting up a dummy accessory. I want the accessory to do nothing, but the provided config.json causes issues for me. I understand that you need a state_cmd for the plugin to work, but I have no idea what this should be for a default setup. It is not included in the config

Link to Logs:

Paste of Logs:

Cmd4 Config:

Screenshots:

Environment:

Node.js Version:
NPM Version:
Homebridge Version:
homebridge-cmd4 Version:
Operating System: Raspbian / Ubuntu / Debian / Windows / macOS / Docker / other
Process Supervisor: Systemd / init.d / pm2 / launchctl / Docker / hb-service / other / none

Hi,
I am not quite sure what you mean by a dummy accessory, however you do not
need any state_cmd if you set the accessories Cmd4_Mode to "Demo". In this
mode, all the characteristics will be used first from the config.json and
then whatever they are in memory, which is known as "Cached".
I hope this helps.
John Talbot
On Mon, May 17, 2021 at 1:00 AM Kyle Worrall @.***>
wrote:

Assigned #110 https://github.com/ztalbot2000/homebridge-cmd4/issues/110
to @ztalbot2000 https://github.com/ztalbot2000.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/110#event-4752090952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX4BYLJHEMDI7ZX7TXLTOCPHHANCNFSM447VOZTA
.

This worked thank you so much

Actually it half worked. The accessory doesn’t turn off, only on. It gets stuck when turning off

Hi,
Can you send me your config.json?
Thanks,
John
On Mon, May 17, 2021 at 4:52 AM Kyle Worrall @.***>
wrote:

Actually it half worked. The accessory doesn’t turn off, only on. It gets
stuck when turning off
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/110#issuecomment-842146480,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCXYTWJ6SEWSWLCVKQMDTODKOHANCNFSM447VOZTA
.

{
"platform": "Cmd4",
"name": "Cmd4",
"allowTLV8": false,
"outputConstants": false,
"restartRecover": true,
"Cmd4_Mode": "Demo",
"accessories": [
{
"Type": "AirPurifier",
"DisplayName": "My_AirPurifier",
"Active": "ACTIVE",
"CurrentAirPurifierState": "INACTIVE",
"TargetAirPurifierState": "MANUAL",
"LockPhysicalControls": "CONTROL_LOCK_DISABLED",
"Name": "Air Purifier",
"SwingMode": "SWING_DISABLED",
"RotationSpeed": 0,
"Manufacturer": "Hosome",
"Model": "1010",
"SerialNumber": "A659B",
"StateChangeResponseTime": 1
}
]
}
This is the config part for the cmd4 plugin

Hi Kyle,
Thanks for the file.  I'll test it shortly.
Ttyl,
John
On Mon, May 17, 2021 at 7:29 AM Kyle Worrall @.***>
wrote:

{
"platform": "Cmd4",
"name": "Cmd4",
"allowTLV8": false,
"outputConstants": false,
"restartRecover": true,
"Cmd4_Mode": "Demo",
"accessories": [
{
"Type": "AirPurifier",
"DisplayName": "My_AirPurifier",
"Active": "ACTIVE",
"CurrentAirPurifierState": "INACTIVE",
"TargetAirPurifierState": "MANUAL",
"LockPhysicalControls": "CONTROL_LOCK_DISABLED",
"Name": "Air Purifier",
"SwingMode": "SWING_DISABLED",
"RotationSpeed": 0,
"Manufacturer": "Hosome",
"Model": "1010",
"SerialNumber": "A659B",
"StateChangeResponseTime": 1
}
]
}
This is the config part for the cmd4 plugin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/110#issuecomment-842248187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX3L4OSD6SYR42WLKETTOD43FANCNFSM447VOZTA
.

Hi Kyle,
I set up your config.json, which looks okay.  I turned on Debug mode so
that I can see what Cmd4 is doing ( homebridge -D ). I then removed all my
Home accessories and just added yours.
The behavior is not 100% what I expected, but there are so many accessories
and characteristics, I cannot possibly try them all from the Home App.
What I did find was that I could tap on the device, The icon would turn
white and go on, but the icon would have a spinning wheel. The debug logs
did show the device going on.
Knowing a little about how the Home App behaves, I went out of the app and
back in.  The spinning wheel disappeared and the icon was still white
(On).  The same could be said for turning it "off".
When the device was on, I could press on its white icon and change all the
parameters.  The debug logs confirmed this and these states were affected
immediately ( no spinning icon).
I have a feeling I know why the behavior is not quite what we would expect
and will look into it, but this may be more to do with Demo mode and the
"Active" characteristic requiring confirmation from a device that the
state_cmd would have.
If you really want to play with a state_cmd and non Demo mode, there are
examples in the distribution; Specifically State.js or AnyDevice.js.
I'll leave this open for now as I look further into the spinning icon.
Ttyl,
John
On Mon, May 17, 2021 at 9:04 AM John Talbot @.***> wrote:

Hi Kyle,
Thanks for the file.  I'll test it shortly.
Ttyl,
John
On Mon, May 17, 2021 at 7:29 AM Kyle Worrall @.***>
wrote:

{
"platform": "Cmd4",
"name": "Cmd4",
"allowTLV8": false,
"outputConstants": false,
"restartRecover": true,
"Cmd4_Mode": "Demo",
"accessories": [
{
"Type": "AirPurifier",
"DisplayName": "My_AirPurifier",
"Active": "ACTIVE",
"CurrentAirPurifierState": "INACTIVE",
"TargetAirPurifierState": "MANUAL",
"LockPhysicalControls": "CONTROL_LOCK_DISABLED",
"Name": "Air Purifier",
"SwingMode": "SWING_DISABLED",
"RotationSpeed": 0,
"Manufacturer": "Hosome",
"Model": "1010",
"SerialNumber": "A659B",
"StateChangeResponseTime": 1
}
]
}
This is the config part for the cmd4 plugin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/110#issuecomment-842248187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX3L4OSD6SYR42WLKETTOD43FANCNFSM447VOZTA
.

Hello again Kyle,
I did spend more time investigating this accessory type.  I'm right in
that the AirPurifier is one of the odd Accessories that Demo mode may not
behave as expected.  In order for the accessory to change state from Active
to Inactive, another set of characteristics come into play and that being
CurrentAirPurifierState and TargetAirPurifierState. It's further confused
by the latter states not being 1:1.
I'll try Cmd4 in non demo mode with a state_cmd that can control the
characteristic differences to make sure it does behave as it is supposed to.
Ttyl,
John
On Mon, May 17, 2021 at 9:59 AM John Talbot @.***> wrote:

Hi Kyle,
I set up your config.json, which looks okay.  I turned on Debug mode so
that I can see what Cmd4 is doing ( homebridge -D ). I then removed all my
Home accessories and just added yours.
The behavior is not 100% what I expected, but there are so many
accessories and characteristics, I cannot possibly try them all from the
Home App.
What I did find was that I could tap on the device, The icon would turn
white and go on, but the icon would have a spinning wheel. The debug logs
did show the device going on.
Knowing a little about how the Home App behaves, I went out of the app and
back in.  The spinning wheel disappeared and the icon was still white
(On).  The same could be said for turning it "off".
When the device was on, I could press on its white icon and change all the
parameters.  The debug logs confirmed this and these states were affected
immediately ( no spinning icon).
I have a feeling I know why the behavior is not quite what we would expect
and will look into it, but this may be more to do with Demo mode and the
"Active" characteristic requiring confirmation from a device that the
state_cmd would have.
If you really want to play with a state_cmd and non Demo mode, there are
examples in the distribution; Specifically State.js or AnyDevice.js.
I'll leave this open for now as I look further into the spinning icon.
Ttyl,
John
On Mon, May 17, 2021 at 9:04 AM John Talbot @.***> wrote:

Hi Kyle,
Thanks for the file.  I'll test it shortly.
Ttyl,
John
On Mon, May 17, 2021 at 7:29 AM Kyle Worrall @.***>
wrote:

{
"platform": "Cmd4",
"name": "Cmd4",
"allowTLV8": false,
"outputConstants": false,
"restartRecover": true,
"Cmd4_Mode": "Demo",
"accessories": [
{
"Type": "AirPurifier",
"DisplayName": "My_AirPurifier",
"Active": "ACTIVE",
"CurrentAirPurifierState": "INACTIVE",
"TargetAirPurifierState": "MANUAL",
"LockPhysicalControls": "CONTROL_LOCK_DISABLED",
"Name": "Air Purifier",
"SwingMode": "SWING_DISABLED",
"RotationSpeed": 0,
"Manufacturer": "Hosome",
"Model": "1010",
"SerialNumber": "A659B",
"StateChangeResponseTime": 1
}
]
}
This is the config part for the cmd4 plugin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/110#issuecomment-842248187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX3L4OSD6SYR42WLKETTOD43FANCNFSM447VOZTA
.

Hi Kyle
I modified the State.js in the Cmd4 distribution, calling it Purifier. I
altered the config.json to use the state_cmd of the AirPurifier.  The path
to the AirPurifier script of .homebridge/Cmd4Scripts/AirPurifier is
important as to maintain dummy state information between calls it creates
and writes to .homebridge/Cmd4Scripts/Cmd4States
The device does go on and you can change the rotation speed. when turning
off, the icon spinner happens until the polked characteristic of
CurrentAirPurifierState is queried from the AirPurifier script, which it
updates during the Active/Inactive cycles.
So all this does resolve the issue.  I'll put the AirPurifier script in the
Examples path for others to look at in the future.
I'm more interested though in Cmd4 handling multiple related
characteristics and therefore will keep this open for a while.
On Mon, May 17, 2021 at 9:55 PM John Talbot @.***> wrote:

Hello again Kyle,
I did spend more time investigating this accessory type.  I'm right in
that the AirPurifier is one of the odd Accessories that Demo mode may not
behave as expected.  In order for the accessory to change state from Active
to Inactive, another set of characteristics come into play and that being
CurrentAirPurifierState and TargetAirPurifierState. It's further confused
by the latter states not being 1:1.
I'll try Cmd4 in non demo mode with a state_cmd that can control the
characteristic differences to make sure it does behave as it is supposed to.
Ttyl,
John
On Mon, May 17, 2021 at 9:59 AM John Talbot @.***> wrote:

Hi Kyle,
I set up your config.json, which looks okay.  I turned on Debug mode so
that I can see what Cmd4 is doing ( homebridge -D ). I then removed all my
Home accessories and just added yours.
The behavior is not 100% what I expected, but there are so many
accessories and characteristics, I cannot possibly try them all from the
Home App.
What I did find was that I could tap on the device, The icon would turn
white and go on, but the icon would have a spinning wheel. The debug logs
did show the device going on.
Knowing a little about how the Home App behaves, I went out of the app
and back in.  The spinning wheel disappeared and the icon was still white
(On).  The same could be said for turning it "off".
When the device was on, I could press on its white icon and change all
the parameters.  The debug logs confirmed this and these states were
affected immediately ( no spinning icon).
I have a feeling I know why the behavior is not quite what we would
expect and will look into it, but this may be more to do with Demo mode and
the "Active" characteristic requiring confirmation from a device that the
state_cmd would have.
If you really want to play with a state_cmd and non Demo mode, there are
examples in the distribution; Specifically State.js or AnyDevice.js.
I'll leave this open for now as I look further into the spinning icon.
Ttyl,
John
On Mon, May 17, 2021 at 9:04 AM John Talbot @.***>
wrote:

Hi Kyle,
Thanks for the file.  I'll test it shortly.
Ttyl,
John
On Mon, May 17, 2021 at 7:29 AM Kyle Worrall @.***>
wrote:

{
"platform": "Cmd4",
"name": "Cmd4",
"allowTLV8": false,
"outputConstants": false,
"restartRecover": true,
"Cmd4_Mode": "Demo",
"accessories": [
{
"Type": "AirPurifier",
"DisplayName": "My_AirPurifier",
"Active": "ACTIVE",
"CurrentAirPurifierState": "INACTIVE",
"TargetAirPurifierState": "MANUAL",
"LockPhysicalControls": "CONTROL_LOCK_DISABLED",
"Name": "Air Purifier",
"SwingMode": "SWING_DISABLED",
"RotationSpeed": 0,
"Manufacturer": "Hosome",
"Model": "1010",
"SerialNumber": "A659B",
"StateChangeResponseTime": 1
}
]
}
This is the config part for the cmd4 plugin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ztalbot2000/homebridge-cmd4/issues/110#issuecomment-842248187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSBCX3L4OSD6SYR42WLKETTOD43FANCNFSM447VOZTA
.

Okay. Thank you so much for the help! I really appreciate it

Closed as per previous comments

Can anyone help me to create a dummy Air Purifier?
Just install the plugin and set up the config inside the plugin. Everything works fine but i can't find a way get ì "Off". It stuck with "Turning off.." and spinning wheel.
Config is the same as the docs provided only without "state_cmd" and "polling".
When i tried to use them, it throw error that directory not found.
Can anyone provide me a complete working config for this accessory?


[FEATURE]Support OpenSearch timestamp in Observability

Is your feature request related to a problem?
Cannot run PPL query that has an OpenSearch timestamp type in it's index mapping in Observability.
What solution would you like?
Support OpenSearch timestamp in Observability
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

Thanks for raising the issue!
Could you post one example for helping understand the use cases?

Adding more info here to clarify the problem.
If a timestamp for an index has a format set with non epoch_millis value(s), queries contain this timestamp as selected time range would fail with 503.
For example, this is the index mappings that would lead to having the above issue:
 "nyc_taxi" : {
  "mappings" : {
   "properties" : {
    "anomaly_type" : {
     "type" : "keyword"
    },
    "timestamp" : {
     "type" : "date",
     "format" : "yyyy-MM-dd HH:mm:ss" // has a format
    },
    "value" : {
     "type" : "float"
    }
   }
  }
 }
}

However removing the format or adding ||epoch_millis to the format in the index mappings seem to avoid having this issue.
// Both worked

"timestamp" : {
   "type" : "date"
 }

or

"timestamp" : {
   "type" : "date",
    "format" : "yyyy-MM-dd HH:mm:ss||epoch_millis"
 }


Add XeVM block store/load ops definition

I've split the single cache control record into multiple of the same class to add some flexibility for changes to cache control policies and their number. It makes the ops fields a bit more explicit but verification (and other) logic will now be isolated for each cache level.
Depends on #387.

This is the one below xegpu, at the same level as llvm.


New vws keys 26

Fixes #30
Fixes #29

Coverage remained the same at 100.0% when pulling fc53bc38c84f51fae48b5b309e49a87591ec89bd on new-vws-keys-26 into c4c429ad3871a69e2111667be73c62f7d49f054d on master.


iframe not centered

Describe the bug
iframe not centered but left aligned in a 100% width white window in 2.0 version. Used to work fine in older versions
Are you able to reproduce the bug in the demo site
Yes
To Reproduce
Steps to reproduce the behavior:

Go to https://mcstudios.github.io/glightbox/ section IFRAMES AND INLINE ELEMENTS
Click on the middle image (grass) that opens the google maps iframe
It's left aligned and with lots of whitespace on the right.

Expected behavior
Should be centered
Screenshots

Desktop:

OS: MacOS Mojave
In Chrome and Safari

This issue has been fixed in the latest version 2.0.1, please download again and try it.


feat: rename “项目组” as monorepo

PR Details

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

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

Checklist

[ ] My code follows the code style of this project.
[ ] I have updated changeset
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

需要在 https://github.com/modern-js-dev/clabot-config 里加一下 username 哈


Update Terraform terraform-aws-modules/ec2-instance/aws to 4.1.2

This PR contains the following updates:

Package
Type
Update
Change

terraform-aws-modules/ec2-instance/aws (source)
module
patch
4.1.1 -> 4.1.2

Release Notes

terraform-aws-modules/terraform-aws-ec2-instance
v4.1.2
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

@balena-ci I self-certify!


Enhance Readme with debug instructions

This could be brief and just state the Google Chrome Inspector can be used
I have written a post in the past that is related:  https://developers.redhat.com/articles/2021/12/08/nodejs-serverless-functions-red-hat-openshift-part-3-debugging-cluster

closed via #197


Trying to get a user profile using public  URL - LinkedInError: 400 Client Error: Bad Request

I'm not managing to get the results from a profile using it's URL.
Basically I'm trying to use that request:
application.get_profile(member_url='http://www.linkedin.com/profile/view?id=238343623&authType=name&authToken=LMsx&trk=apia3334503s3406633*')
I also tried to use only:
application.get_profile(member_url='http://www.linkedin.com/profile/view?id=238343623')
Didn't work either, same error
Everything is working, I get a response using no parameters or the 'user id' as parameter, but I need to use the 'user public URL' to my searches.
Best Regards
Michel

How can view linkedIn users profile by id (huwY24VU2K) this ID i have got from invitation plugin


mini.surround still depends on timeoutlen

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.surround
Description
With default mappings, even if |s| is diabled with
vim.keymap.set({ 'n', 'x' }, 's', '<Nop>')
as described in the documentation , the combination "sa" needs to be pressed within timeoutlen.
However, for example combination "dd" does not.
Neovim version
v0.10.1
Steps to reproduce
In a text file containing text, press s.
Wait longer than timeoutlen.
Press a.
Expected behavior
after pressing iw) the word under cursor will be surrounded by ()
Actual behavior
iw) are inserted inside of the word. Because "a" acted like a normal nvim command.

This is how mappings work in Neovim: if mapping contains more than one key press, each of them should be pressed no longer than 'timeoutlen' milliseconds after the previous. The documentation says that it will prevent occasional usage of built-in s action (which will delete character under cursor and start Insert mode) and it does exactly that.

However, for example combination "dd" does not.

dd is not a mapping created by user but keys for built-in action, which can behave differently.


Error in async mode and strange benchmark test

Hi!
For some reason I write my own require.resolve() implementation and use yours and some other for benchmarking.
So, today I saw async mode in node-resolve and add it to test suite. And its sometime works, and sometime not.
See yourself - add https://github.com/Meettya/node-resolve/commit/04d85e5769434804ffd9bd29072be3c509230032
npm install node-bench -g
npm install lodash
and run test from root dir (may uncomment assert in async to get error)
node-bench ./bench/async-resolve_vs_other.js
and even more - if node-bench right - in my tests sync mode faster than async - its strange.

@Meettya @zxqfox I'm not sure what's actionable here - I'm going to close this for now, but happy to reopen if needed.


If users name themselves after a static path their profile is inaccessible

For example, I don't recommend naming yourself "markets". I mentioned this on Discord and I'm putting it here for posterity because we should really fix this and not forget it.
As discussed, I favor the fix "put user profiles under a separate URL path."

Collecting my arguments for keeping usernames in the top-level namespace below. In which case this bug would be addressed by disallowing any static routes in the username validation function.
Things the username validation function should do:

Disallowing anything that matches an existing route
No 1- or 2-character usernames (Beeminder used to allow these and kinda regretted it)
Must not match a monster regex with things you think to reserve (happy to share what Beeminder uses)

And 3 isn't critical. Between single characters, things that are already static URLs, and things that are still available despite users filling the top-level namespace as fast as they can, you're not actually backing yourself into any kind of corner by not putting users in a folder, so to speak. It works for Facebook and Twitter and others.
Aesthetically, I think it's really nice to have manifold.markets/[name] URLs. If I want to point people to my stuff I like being able to say like facebook.com/dreev or bmndr.co/d or manifold.markets/dreev.
Also, when I link to manifold.markets/dreev/will-elon-buy-twitter or whatever, it's nice to be able to lop off the "/will-elon-buy-twitter" to see my other markets.
Aren't markets, not users, the most fundamental thing on Manifold?
Yeah, it's similar to goals being the fundamental thing on Beeminder. In that case it makes sense to prefix the goal with whose goal it is. So my pushups goal is bmndr.co/d/push. I could see an argument for the creator being less fundamental on Manifold so it's slightly nicer to have manifold.markets/will-elon-musk-buy-twitter than manifold.markets/joeschmoe/will-elon-musk-buy-twitter.
Is it a big pain to restrict usernames so there aren't collisions?
I don't think so. Mostly you can just let usernames take precedence. And you want restrictions on usernames anyway so whether they're top-level doesn't change much. Like, "home" is a slightly dumber username when it's manifold.markets/home vs manifold.markets/users/home but kinda dumb regardless. And it's not like users will ever fill up the namespace. If someday you want like manifold.markets/trades and some user took "trades" it won't be hard to find something that's not taken.
Final thoughts
I'm extra weird but ideally I'd make users hand-pick the slug. Like you pick the title/question -- "Will Elon buy Twitter?" -- and then the UI says "now pick a one or two word slug for a nice url like "manifold.markets/dreev/twittermusk".
(You might actually need something like that for betting via Discord bots.)

We'd need to support old urls anyways to prevent link rot in markets we or others have already shared. Like:
/[x]/slug -> redirect to /mkt/slug
/[x] -> check if x is a valid user -> redirect to /user/x

resolved in #1766 by MayMeta 🙏


Added mcmaukissa

General:

[ ] The pull request title is descriptive. (ex. Added Lunar Network, not Updated metadata.json)
[ ] The pull request does not contain any unrelated commits. (ex. commits from previous pull requests)

Mapping Additions or Updates:

[ ] Folder with appropriate name has been created / updated (must not include any non-alpha characters or whitespace).
[ ] No changes were made to the file's formatting (4 spaces per tab).
[ ] There are no syntax errors.
[ ] There is no pre-existing mapping matching my id (check if there is an existing folder).
[ ] My field values match your requirements.
You can view our patterns here: metadata.schema.json, or take a look below and complete the field checklist:

[ ] id: a lowercase string, which should match the folder name (ex. myserver)
[ ] name: a string (ex. MyServerPvP)
[ ] description: hook between 16 and 80 characters (ex. Home of Competitive Minecraft PvP)
[ ] addresses: an array with lowercase strings (ex. ["my.server", "your-server.com"])

You do not need to specify subdomains, Lunar Client services automatically detect them.

[ ] primaryAddress: the primary address that people connect to with (please include the subdomain if required) (ex. mc.my.server)
[ ] minecraftVersions: an array with Minecraft versions as strings _(ex. ["1.18._", "1.19.2"] - Must be versions or subversions supported by Lunar Client)*
[ ] primaryMinecraftVersion: a Minecraft version as a string (ex. 1.19.2 - Must be a subversion supported by Lunar Client)
[ ] primaryColor: a hexademical color code that primarily distinguishes the server (ex. #00FFFF)
[ ] secondaryColor: a hexademical color code that accompanies the primaryColor of the server (ex. #FF0000)
[ ] primaryRegion: the primary region where your server operates in (ex. NA)
[ ] regions: a list of regions where you have servers located that service your players (ex. ["NA", "EU", "AS"])
[ ] gameTypes: a list of games that describe the content on your server, must be a max of 3 listed (ez. ["PVP", "UHC", "HCF"])
[ ] crossplay: whether the server has support for Bedrock Edition players (through proxies such as GeyserMC)
[ ] (optional) presentationVideo: YouTube video ID (in slug) to server trailer / introduction (ex. 7EV4cPuJvXE)
[ ] (optional) website: url of server website, must include URL schema (http:// or https://) (ex. https://www.your-server.com)
[ ] (optional) store: url of server store, must include URL schema (http:// or https://) (ex. https://store.your-server.com)

Socials

[ ] (optional) twitter: username of twitter account without the @ (ex. MyServer)
[ ] (optional) discord: invite link to discord (ex. https://discord.com/invite/4ceEJuH)
[ ] (optional) youtube: slug or username of youtube channel (ex. MyServer)
[ ] (optional) instagram: username of instagram account (ex. MyServer)
[ ] (optional) twitch: username of twitch account (ex. MyServer)
[ ] (optional) telegram: slug of telegram group (ex. MyServer)
[ ] (optional) reddit: slug of subdreddit wtih 'r/' (ex. MyServer)
[ ] (optional) tiktok: username of tiktok account (ex. MyServer)
[ ] (optional) facebook: slug of facebook page (ex. MyServer)

Compliance

[ ] (optional) privacyPolicy: url to your Privacy Policy (ex. https://www.lunar.gg/privacy)
[ ] (optional) termsOfService: url to your Terms of Service (ex. https://www.lunar.gg/terms)
[ ] (optional) rules: url to your Rules (ex. https://www.lunar.gg/rules)

Media:
Logo

[ ] My logo is a png file.
[ ] I have uploaded my logo to my server folder (ex. lunarnetwork) and named it logo.png.
[ ] My logo has a transparent background and is square (1:1 aspect ratio).
[ ] My logo is 512 pixels in width and height.
[ ] My logo is my own property and complies with relevant copyright/privacy laws.

Background

[ ] My background is a png file.
[ ] I have uploaded my background to my server folder (ex. lunarnetwork) and named it background.png.
[ ] My background has an aspect ratio of 16:9.
[ ] My background is atleast 1920 pixels in width and 1080 pixels in height (can be bigger, but must remain the same aspect ratio).
[ ] My background contains relevant content pertaining to the server, is my own property, and complies with relevant copyright/privacy laws.
[ ] My background doesn't contain any markings, text, or logos (unless part of a build).

I'm not adding this, can you get a better logo and background


Its not working, i cant install it

I tried installing on python but the git clone or the download txt file didnt work so i went on git bash and i cloned it but then after that i dont know what to do it dosent say
I am on windows 11 btw

Alright i got it working but now they all say error

I have everything fixed nevermind


JMESPath query not working..!!

Hi,
Trying to understand why this is not working..!!
az devops security group list --query "[?displayName=='PROPS'].descriptor" --org $org --project $prj
Using MS VSCode, with the JMESPath for VSCode EXTENSION.  I create a json with the information of 2 groups, then run the same query [?displayName=='PROPS'].descriptor and I get the data I need.

But.. when I use the query in AZ, does not get me anything..

Any help would be appreciated.
Thanks
Norman.

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

ID: 5348dfb5-d361-4857-1459-6d7f3cbf5d6c
Version Independent ID: 50319cc5-540d-6689-e5e7-748281d7d9ee
Content: az devops security group
Content Source: latest/docs-ref-autogen/ext/azure-devops/devops/security/group.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

This is because the security group array is in propery graphGroups:
{
  "continuationToken": null,
  "graphGroups": [
    {
      "description": "The default project team.",
      "descriptor": "",
      "displayName": "public Team",
      // ...
    },
    // ...
  ]
}

Thus the query needs to be
az devops security group list --organization https://dev.azure.com/{}/ --project {} \
--query "graphGroups[?displayName=='{}'].descriptor" -o tsv

vssgp.Uy0x...

Hi Jiasli,
Thank you very much, it worked.  I know this is pretty basic, but I am having so much difficulty trying to wrap my head around this JSEMPath, have not been able to find real examples on the web.
Either way, thank you very much.

Hi,
Sorry to keep bugging with this... but tried to follow your example... but in another list, and still keep getting nothing..  this time is with devops users.

I captured the object, to see what is the property name is... in this case show "items"  so I tried and keep getting nothing...
What am I doing wrong?

ran the same exersice, in MS Code, and the query worked, so just needed to add the property name "items"

This http://jmespath.org/ has tutorials and examples on pretty much all different things. There might be some things you will be able to figure out just playing with it as you did above.
Closing this issue because it seems like you were able to get it working.
Please file a new issue if you face any issues.


Adding Petersburg and Constantinople fork blocks to Rinkeby

The table below shows evidence of the fork numbers.

block
date
name

3,660,663
jan-2019
Constantinople

4,321,234
may-2019
Petersburg

Note 1: EIP 1716 mistakenly says Petersburg would activate in block 9,999,999 (in two years from now). I sent a PR aiming to correct it to the block number 4,321,234, introduced by Geth 1.8.24.
Note 2: Istanbul block number is taken care by PR #68.

EIP 1716 fix got merged: https://github.com/ethereum/EIPs/pull/2352#issuecomment-549598925

Thanks!


filter seeing inconsistent argument format

I have a filter method on a RequestStrategy. Sometimes the first argument it receives is formatted like:
{
  "operations": {
    "op": "addRecord",
    "record": {
      "id": "c5439563-2d48-46bf-8bbf-5e39616e96b6",
      "type": "event",
      "attributes": {...}
    }
  },
  "id": "b583d854-907a-4aaa-833f-f21fcbf13853"
}

And sometimes it's formatted like:
{
  "operations": [
    {
      "op": "addRecord",
      "record": {
        "id": "c6665021-9f81-491f-8768-9f5aa3694d57",
        "type": "user",
        "attributes": {...}
      }
    }
  ],
  "options": {},
  "id": "90e8bab4-16b3-49b6-8c41-68a4e9042a66"
}

Notice that in the first one operations is a POJO and in the second it's an array.
Is it intentional that these differ? If it's a bug, I can make a reproduction.

Ah, sorry for the confusion and breaking change. This is intentional and is explained in the WIP v0.17 docs here. There are subtle breaking changes in the Query and Transform interfaces to bring them into alignment and allow for clearer and more consistent return values (singular vs. arrayed).
In your filter I'd recommend interacting with the transform's operations as an array using this utility:
import { toArray } from '@orbit/utils';

const operations = toArray(transform.operations);

The same approach is recommended for evaluating query.expressions.

Thanks, the singular vs plural makes sense.
I still have a related ambiguity on query.expressions that is not just singular vs plural. I'm trying to filter queries by type, and I had to check both type and record.type:
return toArray(query.expressions).every(
  (expression) =>
    expression.type !== 'patient' && expression.record?.type !== 'patient'
);

A record-specific query expression could implement any of these interfaces.
To be fully inclusive of all possibilities you should be aware that expression.records could contain an array of identities. But this check would only be necessary if your application finds records in this way (i.e. q.findRecords([...identities])).
If your purpose is to block certain queries from being forwarded to a source and inspection of the query expressions feels too granular, you might consider using an option that tags your queries in a particular way. For example:
// at the call site
await source.query((q) => q.findRecords('patient'), { remote: 'patientServer' });

// in your strategy
filter(query) {
  return query?.options?.remote === 'patientServer';
}

Closing, but please comment if anything is unresolved.


[BUG] reduce scatter cannot be overlap when using zero

Describe the bug
reduce scatter cannot be overlap when using zero
To Reproduce
DeepSpeed Configs:
json = {
    "train_batch_size": 64,
    "train_micro_batch_size_per_gpu": 1,
    "steps_per_print": 1,
    "zero_optimization": {
        "stage": 3,
        "overlap_comm": true
    },
    "gradient_clipping": 1.0,
    "prescale_gradients": false,
    "fp16": {
        "enabled": false,
        "loss_scale": 0,
        "loss_scale_window": 500,
        "hysteresis": 2,
        "min_loss_scale": 1,
        "initial_scale_power": 11
    },
    "bf16": {
        "enabled": true
    },
    "wall_clock_breakdown": false
}

Expected behavior
reduce scatter should be overlap with computation
ds_report output
[2024-02-27 20:14:22,024] [INFO] [real_accelerator.py:191:get_accelerator] Setting ds_accelerator to cuda (auto detect)
--------------------------------------------------
DeepSpeed C++/CUDA extension op report
--------------------------------------------------
NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
 [WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
 [WARNING]  async_io: please install the libaio-devel package with yum
 [WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
async_io ............... [NO] ....... [NO]
fused_adam ............. [NO] ....... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_lion ............... [NO] ....... [OKAY]
 [WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
evoformer_attn ......... [NO] ....... [NO]
fused_lamb ............. [NO] ....... [OKAY]
fused_lion ............. [NO] ....... [OKAY]
inference_core_ops ..... [NO] ....... [OKAY]
cutlass_ops ............ [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
ragged_device_ops ...... [NO] ....... [OKAY]
ragged_ops ............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
 [WARNING]  using untested triton version (2.0.0), only 1.0.0 is known to be compatible
sparse_attn ............ [NO] ....... [NO]
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
--------------------------------------------------
No CUDA runtime is found, using CUDA_HOME='cuda-11.7'
DeepSpeed general environment info:
torch install path ............... ['python3.10/site-packages/torch']
torch version .................... 1.13.1+cu117
deepspeed install path ........... ['deepspeed']
deepspeed info ................... 0.13.3+39f6ee59, 39f6ee59, master
torch cuda version ............... 11.7
torch hip version ................ None
nvcc version ..................... 11.7
deepspeed wheel compiled w. ...... torch 1.13, cuda 11.7

Screenshots

System info (please complete the following information):

GPU count and types [4 machines with x8 A100s each]
Interconnects [4 machines connected with 4IB]

I think you tagged the wrong person - I am a Deepspeed contributor.


missing libs in aWind

I am having problems in aWind... the output says that some of the .h files are missing, Log.h, LinkedList.h, AHelper.h, etc. is there a fix for this Andrei?

sorry, i miss read the directions


App doesn't open when clicking on action buttons in iOS

i send notifications using the following data but app does not open when i click the Answer action button in iOS. Action button click event not working plz check it.

{
    "to": "",
    "mutable_content": true,
    "content_available": true,
    "data": {
        "content": {
            "id": 4878,
            "privacy": "Public",
            "channelKey": "channel_alert",
            "title": "***Appointment Alert***",
            "body": "test",
            "showWhen": true,
            "autoCancel": true,
            "payload": {
                "data": "{\"id\":0,\"user_name\":\"Hardik Dhola\"}"
            }
        },
        "actionButtons": [
            {
                "key": "decline",
                "label": "Decline",
                "autoCancel": true,
                "buttonType": "KeepOnTop"
            },
            {
                "key": "answer",
                "label": "Answer",
                "autoCancel": true,
                "buttonType": "Default"
            }
        ]
    }
}

Try to do the same thing with a local notification and check if this strange behavior persists.

I have the similar problem.
Notification is created fine and buttons are displayed but when clicked on ACCEPT, the app does not come to the foreground.
Future<void> _firebaseMessagingBackgroundHandler(RemoteMessage message) async {
  // If you're going to use other Firebase services in the background, such as Firestore,
  // make sure you call `initializeApp` before using other Firebase services.
  // await Firebase.initializeApp();

  print("Handling a background message: ${message.messageId}");
  AwesomeNotifications().createNotificationFromJsonData(message.data);
}

my payload is
{
	"to": "",
	"mutable_content": true,
	"content_available": true,
	"priority": "high",
	"data": {
		"content": {
			"id": 102,
			"channelKey": "mychannel",
			"title": "Message Title",
			"body": "Notification body.",
			"showWhen": true,
			"autoCancel": true,
			"summary": "A test is available.",
			"defaultPrivacy": "Public",
			"notificationLayout": "Inbox",
			"payload": {
				"request_id": "slkjfkldsf",
				"request_type": "my_request",
				"display_text": "Would you like to accept?"
			}
		},
		"actionButtons": [{
				"key": "ACCEPT",
				"label": "ACCEPT",
				"autoCancel": true
			},
			{
				"key": "REJECT",
				"label": "REJECT",
				"autoCancel": true,
				"buttonType": "KeepOnTop"
			}
		]
	}
}

Does this behavior persist if you create this notification locally?
What platform are you using to do these tests?

I got the problem! The default action button behaviour for iOS is to not call the app, but there is a way to change this behaviour with some special options.
I gonna activate it in next release. Thank you so much guys to show me this.

That is very cool. I will try your patch this weekend and revert back with my results.
Awesome stuff man.


Don't useCallback unless necessary

https://github.com/React-Vancouver/website/blob/49ffc4bfe0e8ede26b40f0c31c237be10b900ab3/src/components/constructs/Layout/partials/Nav/Nav.js#L30-L31
it's recommended by the react team to not useCallback unless you see perf issues. I'd recommend just creating the function without the useCallback hook.

Hello @AkimaLunar,
I believe I can get this one fixed. It is just to remove the useCallback function and the code would be like:
const openMobileNav = () => setIsMobileNavOpen(true);
As some others projects they are requesting to add the issue in a different file, I believe it is better to confirm if it is okay change the code in the file (website/src/components/constructs/Layout/partials/Nav/Nav.js) and add it in a different branch?
If this is okay, I will do a PR ASAP.
Thanks


[api-minor] Combine the textContent/textContentStream parameters

Rather than handling these parameters separately, which is a left-over from back when streaming of textContent was originally added, we can simply pass either data directly to the TextLayer and let it handle things accordingly.
Also, improves a few JSDoc comments and typedef-imports.

/botio test

/botio test


Introduce override_default_config(config) API in lieu of setup(config) for server-specific configuration

It might be nice to introduce configure():
require"lspconfig".sumneko_lua.configure {
  on_attach = on_attach,
  ...
}

That can be used instead of setup().
This way, the less bulky iteration setup style remains the same:
local servers = {
  "sumneko_lua",
  "pyright",
}

for _, lsp in ipairs(servers) do
  nvim_lsp[lsp].setup { on_attach = on_attach }
end

...and one can easily modify the iteration loop:
for _, lsp in ipairs(servers) do
  lspconfig[lsp].setup(coq.lsp_ensure_capabilities {
    on_attach = on_attach,
  })
end

...without having to manually specify:
require"lspconfig".sumneko_lua.setup(
  coq.lsp_ensure_capabilities {
    on_attach = on_attach,
    ...
  }
)

...for each customized language server.

A related issue is that there does not seem to be a way to easily access and override the default_configs from within the API for each language server.

are you asking for a way to set "default" or global values? that could be useful, but should not use a similar-yet-different function name (configure vs setup). maybe it could be a setup() function that lives on lspconfig module directly.

Yes. Perhaps another name is override_default_config.

We already have this
https://github.com/neovim/nvim-lspconfig/blob/master/ADVANCED_README.md#example-override-default-config-for-all-servers

That applies the same configuration to all servers. This is intended for server-specific configuration, just like server_name.setup().
The problem with using setup() directly for server-specific configuration is that one can't use the for loop style shown above. The intended goal of this feature request is to decouple configuration from any other things that setup() does.

I still don't see the utility, the setup{} call is effectively this, with the only side effect being adding the filetype watcher which can be disabled with autostart  = false

I suppose one could maintain their own config table instead, so the proposed API isn't as useful as I imagined initially:
local servers = {
  "sumneko_lua",
}

local configs = {
  sumneko_lua = { ... },
}

for _, lsp in ipairs(servers) do
  local config = configs[lsp] or {}
  config.on_attach = on_attach
  nvim_lsp[lsp].setup(config)
end

Perhaps something like this could be added to the ADVANCED_README.md, but other than that, the issue may be closed.

Closing as resolved


No operator matches the given name and argument types. You might need to add explicit type casts.

This error occured when seeding the database. My guess is that type mismatching is occuring but explicit type cast didn't solve the issue. A help would be much appriciated. Thanks @paulpierre
🌱 Seeding database ...
sqlalchemy.exc.ProgrammingError: (psycopg2.errors.UndefinedFunction) operator does not exist: entity_status = integer
LINE 3: ...splay_name = 'project-pepetamine.md' AND document.status = 2
^
HINT:  No operator matches the given name and argument types. You might need to add explicit type casts.
[SQL: SELECT document.id, document.uuid, document.organization_id, document.project_id, document.display_name, document.url, document.data, document.hash, document.version, document.status, document.created_at, document.updated_at
FROM document
WHERE %(param_1)s = document.project_id AND document.display_name = %(display_name_1)s AND document.status = %(status_1)s]
[parameters: {'param_1': 1, 'display_name_1': 'project-pepetamine.md', 'status_1': 2}]

comment line 815 in helper.py

did you manage to solve the problem?

yes,i have solved
Vào Th 3, 26 thg 12, 2023 vào lúc 20:42 vassilevev <
@.***> đã viết:

did you manage to solve the problem?
—
Reply to this email directly, view it on GitHub
https://github.com/paulpierre/RasaGPT/issues/62#issuecomment-1869555877,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJ54NKZETOQT6YD2L5QRN3YLLH6DAVCNFSM6AAAAABAF4VXJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGU2TKOBXG4
.
You are receiving this because you commented.Message ID:
@.***>

I solved this issue:
step1: comment this line
step2:  use openai==0.28.1 in here
if the issue still persists, let me know.

@usUpal its work!

Does it integrate with whatsapp?
Vào Thứ Tư, 27 tháng 12, 2023, vassilevev @.***> đã
viết:

@usUpal https://github.com/usUpal its work!
—
Reply to this email directly, view it on GitHub
https://github.com/paulpierre/RasaGPT/issues/62#issuecomment-1870179559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZJ54NON6BJVWSKTTZEBVWLYLP3ONAVCNFSM6AAAAABAF4VXJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGE3TSNJVHE
.
You are receiving this because you commented.Message ID:
@.***>

Does it integrate with whatsapp? Vào Thứ Tư, 27 tháng 12, 2023, vassilevev @.> đã viết:
…
@usUpal https://github.com/usUpal its work! — Reply to this email directly, view it on GitHub <#62 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZJ54NON6BJVWSKTTZEBVWLYLP3ONAVCNFSM6AAAAABAF4VXJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZQGE3TSNJVHE . You are receiving this because you commented.Message ID: @.>

I didn't try it. if you try, let me know.

@usUpal its work!

ok then, I am closing this issue.


[Bug] duplicate rules breaks kompile error messages

module TEST
  imports INT
  claim 1 => 0
  claim 1 => 0
  claim 1 => 0
endmodule
[Error] Compiler: Claims are not allowed in the definition.
	Source(/home/radu/work/test/./test.k)
	Location(4,9,4,15)
[Error] Compiler: Claims are not allowed in the definition.
	Source(/home/radu/work/test/./test.k)
	Location(5,9,5,15)

Only 2 error messages (instead of 3). Second kompile only one:
[Error] Compiler: Claims are not allowed in the definition.
	Source(/home/radu/work/test/./test.k)
	Location(4,9,4,15)

It got very confusing when I was trying to add tests for rules with different attributes.
Should we disallow them?
How about rules with the same contents but different attributes?
How about a rule and a claim which has the same contents?
Also. Error messages don't get updated with the new location if I add new lines before the rule.

This looks like it's a consequence of a bug that doesn't correctly update the location information of rules after they've been retrieved from the cache. As a result, two of the three error messages are identical and thus get automatically deduped. I think if we fix this issue we should see more sensible behavior afterwards.

We won't focus on fixing the rule de-dupe, but the error message de-dupe. This can be done by correctly updating location information of rules after retrieving from cache.


Authenticating Google AppEngine service accounts

Hi,
I am looking to use oauth2_proxy to secure services that are accessed from our App engine application. Each application is provided a service account that is used for any oauth calls that are made from the application. There are also API's provided to get the service account name and a token for a specified scope.
Here is a python script that if run on the appengine application would pass the assert call.
import json
from google.appengine.api import app_identity, urlfetch

access_token, _ = app_identity.get_access_token(['https://www.googleapis.com/auth/userinfo.email'])
json_data = json.loads(
    urlfetch.fetch(
        'https://www.googleapis.com/oauth2/v1/userinfo?alt=json&access_token={}'.format(access_token)
    ).content
)

assert app_identity.get_service_account_name() == json_data['email']

I am wondering if there is a way today for us to pass the access token and the service account name and  oauth2_proxy will validate with Google that the supplied service account is in the authenticated-emails-file?
Thanks!

Hi. Did you ever solve this? I am trying to do almost the same thing. Would be really nice if ouath2_proxy could handle this.


put demos in their own repository?

The majority of code in this repo is now demos. Should demos get their own repo? Perhaps we can keep 1 master demo here that demonstrates all core features?

In my opinion it is nice to have all demos listed in this current repository, otherwise one must put in more effort to find different solutions from other "areas".

Regarding the /demos/ folder I will

Add a readme and screenshot to every demo
Automate the build process so binary versions of demos are always available

Reorganization is happening here
https://github.com/swharden/ScottPlot/tree/cleaner-demos


pep8: Fix rules

Automated message from Dropbox CLA bot
@nikolaystanishev, it looks like you've already signed the Dropbox CLA. Thanks!

@umairwaheed how can I fix the conflicts. What is the content for the comments?

I fix the conflicts and my commit message

OK I merged abc2ff4a06e3732c756e71aadc8a310724a270c2, which has all the changes from your commit that I thought were handled well, with a few small tweaks, most importantly fixing these regressions:
diff --git a/zerver/tests/test_subs.py b/zerver/tests/test_subs.py
index 1f5c553..d6d4c8c 100644
--- a/zerver/tests/test_subs.py
+++ b/zerver/tests/test_subs.py
@@ -1523,7 +1523,7 @@ class SubscriptionAPITest(ZulipTestCase):
                 streams=[stream1, stream2, private]
             )

-        peer_events = [e for e in events\
+        peer_events = [e for e in events
                        if e['event'].get('op') == 'peer_remove']

         notifications = set()
diff --git a/zerver/views/streams.py b/zerver/views/streams.py
index 17c2b33..a4eaabe 100644
--- a/zerver/views/streams.py
+++ b/zerver/views/streams.py
@@ -295,7 +295,7 @@ def filter_stream_authorization(user_profile, streams):
         if stream.invite_only:
             unauthorized_streams.append(stream)

-    authorized_streams = [stream for stream in streams if\
+    authorized_streams = [stream for stream in streams if
                           stream.id not in set(stream.id for stream in unauthorized_streams)]
     return authorized_streams, unauthorized_streams

Huge thanks for your work on this @nikolaystanishev!  I think there's 2 approaches for how you proceed; either do a git rebase or just use the linter to redo the changes that I didn't merge.  I suspect the latter may actually be the better strategy, since the lines I didn't merge are the ones that need to be changed anyway :).

@nikolaystanishev I just wanted to check in on this PR; are you planning to address the remaining violations using the better line-wrapping approach I suggested above?  Or should we close this?

I merged the rest of this via #2669.  Thanks again @nikolaystanishev for all your work on this!


[Pawn Game] Better map interaction system to support items and blueprints

This can probably be made fairly easily by having a single mouse input catcher that will find everything the player can interact with at the click location.

Somewhat refactored the right click system to move away from the physics system and to use a single mouse input catcher (pawn_game_world) for map interactions. From here it is trivial to add support for items and blueprints to the right click thing


Incorrect floor for stacked cards

without any filters

with buy now filter:

https://canary.kodadot.xyz/ahp/collection/11

blocked by

#7482

Williams be fixed along with the upcoming pr to revive stacked cards


debug: [Do Not Merge] testing failure in PR#25

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Closing as build is now passing in #25


Add Noor to Contributors list

@pionoor, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Configurable header forwarding for fragment requests

The forwarded headers are currently hard-coded in https://github.com/zalando/tailor/blob/master/lib/filter-headers.js#L16.
This could be configurable via Tailor(options) with sane defaults. My particular use case is that tailor runs in a private, trusted network where headers are used for authentication.
I'd be happy to contribute it myself if considered useful.

This would be a great contribution. Thanks in advance.

@jmaicher Thanks for opening the issue.

This could be configurable via Tailor(options) with sane defaults. My particular use case is that tailor runs in a private, trusted network where headers are used for authentication.

The only problem with adding the headers in the options list is that

It will end in a huge list if in future fragments starts understanding CSP and other security related headers.
You cant conditionally add/remove header based on GET/POST or any other logic.

The other option which I can think of is requestFragment starts accepting the filterHeader function as part of a argument and you could tune the header function as per your needs.
I am open to discussion. Let me know your thoughts.
/cc @lmineiro

@jmaicher thoughts?

What about building the filtered request before calling requestFragment?
This would keep that function even cleaner, leaving to whatever happens between receiving the edge request and calling the fragment the required transformation/header filtering.
Composition and testing also seems simpler, but I admit my lack of knowledge of this particular "ecosystem"

@vigneshshanmugam  I like the general idea of a filterHeader function, but I'm not sure yet what arguments it should get?! Where do I get the request method from?
@grassator :+1: on keeping the interface clean. Regarding the first argument check, is your intention to avoid breaking changes in the public API? If not, I'd rather go with just an argument object to keep the code cleaner.
@lmineiro We also need to consider that requestFragment is already configurable from the outside, giving users full control over how requests are constructed and dispatched. Of course, we could think about splitting this concern into a factory and dispatch function, but I'm not sure it's worth it.

@jmaicher Please have a look at the PR #110


Use correct name for src folder

The Arduino Library Specification requires that the source folder be named src, but it was named scr.
Reference:
https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5:-Library-specification#source-code

if you please , would you check the liberary again.


Page should define MSAPassThrough

I came to the page to figure out what this property does and is used for.  The current description didn't help much.  What is MSA? Why do Legacy Applications need it?
More information to help the developer make an informed decision would be super useful.

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

ID: af97da54-d81f-8cc1-8a1a-ba10ed5dc41b
Version Independent ID: c82c4c28-89a1-950f-2b58-b1795045f54c
Content: WindowsBrokerOptions.MsaPassthrough Property (Microsoft.Identity.Client) - Azure for .NET Developers
Content Source: xml/Microsoft.Identity.Client/WindowsBrokerOptions.xml
Service: active-directory
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Managed Identity:0.5516926,Azure.Identity:0.21526892,Docs:0.049555216'

This issue has been transferred to the repository where the Microsoft.Identity.Client is maintained as #3670.


PHP8 Support

Hello,
I am currently checking which of the libraries I use support php8. Therefore I also checked this and tried what happens when I update the dependencies and run with php8.
The dispatcher and its production dependencies itself seems to run without changes needed. With a very small change, all tests are successful (I will create a PR for this), but only if phpunit and laminas is upgraded. php-cs-fixer is currently not supported (see https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/4702)
That means, to make sure dev dependencies are compatible, they would have to be updated. But for example phpunit 9 only supports 7.3+, which would also raise the minimum version in this library. Is this something for a "version 2" to support php8+? Or upgrade the dev dependencies to test under php8 and keep the minimum version 7.0 anyways?
What do you think about this?

Hi, thanks for telling me.
I just released a new version (v1.0.1) with support for PHP 8.


update pypy version in CI workflow

This should fix the CI (hopefully)

Looks like twisted in trunk requires Python 3.8 minimal. So either we disable CI to not run for twisted trunk when using 3.7 OR we upgrade the project to 3.8

ahh, hi Omer;) and thanks for chiming in. as you say, we have 2 options. I'd go with "Python 3.8 minimal." or even CPython 3.9 minimal and whatever PyPy minimal this corresponds to. I have good experiences with 3.9 and it makes sense IMO rgd a minimal Python version (including distros I guess even though I am usually running from vanilla sources ..). also, I would definitely want to track Twisted trunk since it is such an essential pieces, also because of txaio downstream projects like Crossbar.io which require tx rather than aio ..

hey Tobias, I tried to update a few deps, its still failing. Will find a proper fix later tonight and update this PR

first, yes, thanks!
then, oh well:( there seems to be yet another CI issue. I don't have time to look into ..
third, fwiw, I am running this python these days regularily:
(.venv) oberstet@intel-nuci7:~/scm/typedefint/rfminer$ python -V
Python 3.12.1

thus, we might add CPython 3.12 in testing
finally, I recently stumbled across
https://devclass.com/2024/01/10/just-in-time-compiler-likely-to-arrive-in-python-3-13/
so PyPy is going to get some competition? ;) I have not yet done any testing, or even performance testing / comparison with PyPy
and then: PyPy not only has JIT, but incremental GC, which is essential for real-world performance eg in Crossbar.io. so I am not holding my breath;)

cool, thanks a lot!!
one more note:
we should also update https://github.com/crossbario/txaio/blob/50a111a95b19168127681429bebd3827e1e2f6e5/setup.py#L105
fwiw, if required, I don't have a problem bumping min versions of deps https://github.com/crossbario/txaio/blob/master/requirements.txt .. if required ..

One of the test failures is caused by a regression in the upstream code. I have notified the upstream https://github.com/twisted/twisted/pull/12040#issuecomment-1888181534

Hi Omer, thank you so much! I mean, both for figuring it out, and for stepping up within Twisted, moving things forward! this is awesome. even though it is a Twisted regression, and the fallout is hitting us within Autobahn etc, this even more so kinda supports what I said with keeping Twisted trunk in the CI here: we would not even know what's coming down the road and will affect us later if we wouldn't include tx trunk, and we would not be able to provide early feedback to Twisted upstream, and that obviously wouldn't be helpful for anyone, neither "us" nor "them" as in Twisted, and it wouldn't be true OSS good citizenship. Anyways, I'm sure you tick the same;) Again, thank you for contributing and helping!

@oberstet looks like CI is all good now

In a follow up PR, we may want to update Python 3.11 to 3.12 instead as that's the latest stable now

fyi, reconfigured required CI for

In a follow up PR, we may want to update Python 3.11 to 3.12 instead as that's the latest stable now

yep, +1! I think this would make sense, we have CPython 3.9 + 3.12, and PyPy 3.9 and 3.10, and tx trunk and releases. this would give us "look back" and "look forward" visibility into any issues ..

another note, I just wanted to disable (but not delete) the "deploy" action, and that worked previously in GH actions (I could disable individual actions via web UI before), but it doesn't seem to work/be available anymore -- or I am too dumb to find it;)


Bundle import paths are invalid when packaged using pkg

Describe the bug
I'm not entirely sure but this line might be the culprit on file generate-user-component-entry.ts. Basically, this path generated by the bundler leads nowhere so using absolute path packaged might be the solution here.

tsconfig.json
{
  "compilerOptions": {
  	"target": "ESNext",
  	"module": "Node16",
  	"jsx": "preserve",
  	"lib": ["dom", "ESNext"],
  	"allowJs": true,
  	"outDir": "build",
  	"rootDir": "src",
  	"strict": true,
  	"noImplicitAny": true,
  	"esModuleInterop": true,
  	"resolveJsonModule": true,
  	"skipLibCheck": true,
  	"forceConsistentCasingInFileNames": true,
  	"sourceMap": true,
  	"inlineSources": true
  }
}

Error output
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:198
      base = Object.assign(new Error(base.message), base);
                           ^

Error: 'default' is not exported by ../../../../../../../../../snapshot/webapp/server/build/admin/containers/bulkUser.jsx, imported by .adminjs/.entry.js
  at error (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:198:30)
  at Module.error (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:12560:16)
  at Module.traceVariable (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:12919:29)
  at ModuleScope.findVariable (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:11571:39)
  at Identifier.bind (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:7570:40)
  at AssignmentExpression.bind (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:5400:23)
  at ExpressionStatement.bind (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:5400:23)
  at Program.bind (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:5396:73)
  at Module.bindReferences (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:12556:18)
  at Graph.sortModules (/snapshot/webapp/server/node_modules/rollup/dist/shared/rollup.js:23155:20) {
code: 'MISSING_EXPORT',
url: 'https://rollupjs.org/guide/en/#error-name-is-not-exported-by-module',
id: '/Users/deniz/Documents/PROJECTS/WEB PROJECTS/bus-fs/webapp/server/dist/.adminjs/.entry.js',
pos: 36,
loc: {
  column: 7,
  file: '/Users/deniz/Documents/PROJECTS/WEB PROJECTS/bus-fs/webapp/server/dist/.adminjs/.entry.js',
  line: 2
},
frame: '1: AdminJS.UserComponents = {}\n' +
  "2: import Component1 from '../../../../../../../../../../snapshot/webapp/server/build/admin/containers/bulkUser'\n" +
  '          ^\n' +
  '3: AdminJS.UserComponents.Component1 = Component1\n' +
  "4: import Component2 from '../../../../../../../../../../snapshot/webapp/server/build/admin/containers/bulkUser'",
watchFiles: [
  '/Users/deniz/Documents/PROJECTS/WEB PROJECTS/bus-fs/webapp/server/dist/.adminjs/.entry.js',
  '/snapshot/webapp/server/build/admin/containers/fileUpload.jsx',
  '/snapshot/webapp/server/build/admin/containers/documentation.jsx',
  '/snapshot/webapp/server/build/admin/containers/bulkUser.jsx',
  '\x00rollupPluginBabelHelpers.js'
]
}

Installed libraries and their versions

"@adminjs/express": "^5.0.0"
"@adminjs/sequelize": "^3.0.0"
"adminjs": "^6.1.4"
"pkg": "^5.8.0"

To Reproduce
I expect this issue to be reproducible by just packaging any adminjs project. You just have to add compiled [tj]sx files as assets. If it doesn't do that I can try to create a minimal repository.
Expected behavior
It should start as normal when packaged. I'm using pkg to create optimized and minimal version of my server before deploying it to AWS so backward compatibility etc. is not important.

Can you share a repository where I can reproduce this and some information about your OS etc? I've just tried reproducing in a project I'm working on privately and it passes although I'm getting a warning:
prebuild-install WARN install No prebuilt binaries found (target=v16.16.0 runtime=node arch=arm64 libc= platform=darwin)

Hello, after investigating the problem I've found the root cause. Error log was already pointing out but the problem was my tsx files were being compiled like this:
function Board() { ... }
exports.default = Board

When I switched to jsx and used the original export like this:
export default Board() { ... }

The error was gone. So this is either a typescript related configuration problem or something about rollup configuration.


Fix issues with AQE and DPP enabled on Spark 3.2 [databricks]

Fixes #3653.
This fixes a number of issues with GpuBroadcastToCpuExec.  In Spark 3.2+, there's an assert to verify that the child of an AdaptiveSparkPlanExec is a BroadcastQueryStageExec if it tries to compute the broadcast.  The transition plans would place GpuBroadcastToCpuExec immediately under AdaptiveSparkPlanExec which triggered the assert.
Unfortunately creating a new BroadcastQueryStageExec instance requires shimming, since the signature of BroadcastQueryStageExec has changed across Spark versions.  Newer versions require a canonicalized plan parameter, so I needed to add a shim method to create a new instance given an old one and a new child to place underneath.
Besides fixing that assert, there was also a mishandling of the broadcast data in GpuBroadcastToCpuExec.  The code assumed that ColumnarBatch.rowIterator would return a unique row per iteration, but it returns the same row object instance for each iteration and mutates that object's internal state to produce the proper row when the row's data is requested.  The code was producing an array of InternalRow without manifesting the data from the row, which caused each entry to be the same ColumnarBatchRow instance, pointing at the last row of data.  In addition the UnsafeRow instances were being reused in a similar manner.  The fix is to not force the internal rows to manifest in an array until they are being converted to an unsafe row, and when converting we need to make a copy of each unsafe row.
One last issue, which ended up being the majority of the change in this PR, is that BroadcastQueryStageExec expects a BroadcastExchangeLike child, so to fix the original assert, GpuBroadcastToCpuExec needs to be a BroadcastExchangeLike.  Unfortunately BroadcastExchangeLike was forcing shims due to Databricks changing the signature of that class.  Rather than shim yet another class that needs BroadcastExchangeLike, I ended up creating a shim v2 version of BroadcastExchangeLike called ShimBroadcastExchangeLike that encapsulates the differences between Apache Spark and Databricks with this class.  This not only allows us to avoid shimming GpuBroadcastToCpuExec but also allows us to unshim GpuBroadcastExchangeExec which was only being shimmed due to this trait signature difference.

build

Looks like a databricks301 build issue:

[2021-09-28T20:35:06.698Z] [ERROR] /home/ubuntu/spark-rapids/sql-plugin/src/main/301db/scala/com/nvidia/spark/rapids/shims/v2/Spark30XShims.scala:69: not enough arguments for method apply: (id: Int, plan: org.apache.spark.sql.execution.SparkPlan, _canonicalized: org.apache.spark.sql.execution.SparkPlan)org.apache.spark.sql.execution.adaptive.BroadcastQueryStageExec in object BroadcastQueryStageExec.

[2021-09-28T20:35:06.698Z] Unspecified value parameter _canonicalized.

[2021-09-28T20:35:06.698Z] [ERROR]       newPlan: SparkPlan): BroadcastQueryStageExec = BroadcastQueryStageExec(old.id, newPlan)

[2021-09-28T20:35:06.698Z] [ERROR]

build

build

There are some double-close issues in some cases, converting to draft while I investigate.

Found the double-buffer issue, had to do with multiple columns appearing in the same batch all referencing the same buffer.

build

I think I've fixed the problems with HostColumnVector deserialization.  Queries seem to be computing proper results, but I'm seeing the CPU and GPU execute queries with very different shapes (e.g.: CPU never runs a stage with more than 35 partitions, yet GPU will run many partitions with the full 200 partitions).  Investigating.

The occasional lack of AQE shuffle coalescing on GPU queries is unrelated to this change.  Filed #3713 to track it separately.

build

Note I am running with this change in spark2a, and I haven't found the NPE I was seeing before.


idam integration

If you're not doing any of the security stuff on the front end anymore then you can probably delete the dam-guard file along with its spec file to fix the broken build.

Codecov Report

Merging #17 into master will decrease coverage by 1.63%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
- Coverage   88.34%   86.71%   -1.64%
==========================================
  Files          17       16       -1
  Lines         326      286      -40
  Branches       16       13       -3
==========================================
- Hits          288      248      -40
  Misses         35       35
  Partials        3        3

Impacted Files
Coverage Δ

src/app/dm-listview/dm-listview.component.ts
76.92% <ø> (-0.44%)
:arrow_down:

src/app/app.module.ts
100% <ø> (ø)
:arrow_up:

src/app/dm-upload/dm-upload.component.ts
100% <100%> (ø)
:arrow_up:

src/app/dm/document-store.service.ts
86.66% <66.66%> (-0.84%)
:arrow_down:

src/app/auth/jwt.service.ts
83.33% <0%> (-16.67%)
:arrow_down:

Continue to review full report at Codecov.

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


[Bug]: Error Hydration Mismatch when using TabsRoot on Nuxt

Environment
Developement/Production OS: MacOS
Node version: 20.10.0
Package manager<EMAIL_ADDRESS>Radix Vue version: 1.3.1
Vue version: 3.4.13
Nuxt version: 3.9.1
Nuxt mode: universal
Nuxt target: server
CSS framework<EMAIL_ADDRESS>Client OS: Windows 10 19043.1110
Browser: Chrome 90.0.4430.212

Link to minimal reproduction
https://stackblitz.com/edit/nuxt-starter-8ryhek?file=app.vue
Steps to reproduce
You can check on reproduction link that i provide
Describe the bug
When use prop v-model or defaultValue at TabsRoot component i got error Hydration Mismatch
Expected behavior
No response
Context & Screenshots (if applicable)

Hydration issue will be keep track here 😁

Is it related to this discussion @zernonia ? Auto-Generated IDs + SSR (Hydration Mismatch issue)

AHHH my bad.. I copied the link and not paste it lol! Yes that's the discussion 😁


[ENH] Update hippunfold and micapipe apps info

Follow up to the issue #560 . Updated the information of the hippunfold and micapipe apps.

did some clean up of the yaml


Log this at beginning not at the end

https://github.com/PhilipMay/mltb/blob/6464b5bdc14d207a91e76d9e15759e2c712bdc32/mltb/omlflow.py#L231

Does not work. Most values are only known at the end and not at the beginning.
Closing...

reopen again - should work for some values
logging at the beginning is good because otherwise exceptions might skip logging


how to add custom message type

Hi, thanks for your great work on the offline cyber record. I want to add some custom message type, how can I make it work?

what do you mean by custom message type?
read
the message metadata are in the record file itself, if you want to read the message, cyber_record has already done this part of the work.
write
If you want to write some custom messages to create a record file, you can follow these steps

create a proto file for the custom message
compile the proto file to python protoc -I=$SRC_DIR --python_out=$DST_DIR $SRC_DIR/your_message.proto
Fill data into messages in python and use the api provided by cyber_record to write messages

thanks for your quick reply, after some thought, I also choose your proposed method to write custom message to cyber_record.


Best practices for security advisories doc should use positive statements

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/enterprise-cloud@latest/code-security/repository-security-advisories/about-coordinated-disclosure-of-security-vulnerabilities#best-practices-for-vulnerability-reporters
What part(s) of the article would you like to see updated?
Best practices should be updated to use positive statements rather than negative. Typically people are expecting a list of what to do, rather than what not to do.
Additional information
No response

We're going to keep the language, so I'm going to close this. See PR comments for details:

https://github.com/github/docs/pull/19826#issuecomment-1216665424
https://github.com/github/docs/pull/19826#issuecomment-1216677431


Fix incorrectly skipping setLastResponse when using CommandUtil.send

I encountered a bug with sending an embed through message.util.send, and then later trying to edit it through message.util.edit. The edit call would error internally because lastResponse was null even after the send completed. This is due to discord.js returning an empty array for embed.files, rather than null. The empty array cascaded to the local hadFiles and was interpreted as truthy on line 90, skipping the setLastResponse call.
if (hadFiles || (this.lastResponse && this.lastResponse.attachments.size)) return sent;
I haven't tested anything about this bug in version 8, as CommandUtil is very different there.

Can you see if this is still needed?


Port fork_ami to a Jenkins multistep job

Signed-off-by: Michail Kargakis<EMAIL_ADDRESS>

@stevekuznetsov @jhadvig

@stevekuznetsov got any idea why some steps have been skipped in https://ci.openshift.redhat.com/jenkins/view/All/job/ami_build_origin_int_rhel_fork/1/consoleFull? For example, there is no INSTALL GOLANG step...

Step failed:
/tmp/hudson3703443209542702395.sh: line 26: TRELLO_URL: unbound variable

Yeah saw that but I was wondering why we report it back as a success..

@kargakis yeah very unclear ... would have to check return code of the script, as ssh should just mirror that.

Wait, that's running on the Jenkins host, so it should just be the return code of the shell itself.

@stevekuznetsov this should be ready for a review. I've removed the extended tests since they don't run as part of the original fork ami.

yeah, removed it in the interim
On Thu, May 11, 2017 at 5:02 PM, Steve Kuznetsov<EMAIL_ADDRESS>wrote:

@stevekuznetsov commented on this pull request.
In sjb/config/test_cases/ami_build_origin_int_rhel_fork.yml
https://github.com/openshift/aos-cd-jobs/pull/220#discussion_r116014612:

 elif [[ "${TARGET_ORIGIN_BRANCH}" == "master" ]]; then

   oct prepare docker --version=1.12.6 --repo=oso-rhui-rhel-server-* -u https://mirror.openshift.com/enterprise/rhel/dockerextra/x86_64/os/

 else

   echo "[FATAL] No Docker version is known for \`${TARGET_ORIGIN_BRANCH}\`.

         Contact Steve Kuznetsov<EMAIL_ADDRESS>for assistance."

   exit 1

 fi

type: "host_script"

title: "prepare repositories"
script: |-

 oct prepare repositories

 # Replace origin repo with the specified fork

 git clone git@github.com:"${ORIGIN_FORK_ID}"/"${ORIGIN_REPO}"

 oct sync local origin --src "${ORIGIN_REPO}" --dest /data/src/github.com/openshift/origin --branch "${ORIGIN_BRANCH}" --merge-into "${TARGET_ORIGIN_BRANCH}"

the --dest looks unnecessary here
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/aos-cd-jobs/pull/220#discussion_r116014612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADuFf0_IaIxAmDFqeBn94ruLePTLv3viks5r4yLzgaJpZM4NI5Rj
.

@stevekuznetsov is this a straight port of fork_ami? In particular, for the fork AMIs still pruned according to their Name tag, will they still be named correctly?

No, we'll probably need to update the pruning logic -- but it should neatly fit into the tag-based approach, I think @kargakis is adding a new tag to show it's a fork.

We got our first fork AMI yesterday, this should be ready for a final pass @stevekuznetsov

@stevekuznetsov all comments addressed, new build running in https://ci.openshift.redhat.com/jenkins/view/All/job/ami_build_origin_int_rhel_fork/36/


--block option to block until jobs have finished

Grid engine has the -sync y option that makes qsub block until the submitted job finishes (or errors out). This can be useful when writing dynamic pipelines with stages that can't be submitted all at once with dependencies (because which later stages run depend on the outputs from earlier stages).
It appears PBS does not have this feature (and unsure about other platforms), but we could make our own by polling qstat.

The PBS hack way of blocking (e.g. for job id 12345):
qsub -W depend=after:12345 -I -x true

Discussion re: PBS https://github.com/adaptivecomputing/torque/issues/268

After bringing it up on the mailing list, and being pointed to that thread, it looks like there's been movement and it will be implemented.
Pity clusters usually run old versions for a long time.

Okay, let's use my hack above for now. It should work generally, I think.

srun can be used for slurm.

Need this feature to fix up all the ants scripts so they play nicer with Niagara

Job submission for blocking needs to be done via a map or other parallel object, otherwise you get sequential blocking during submission.

PBS hack version requires #156 implemented

At some point PBS added -Wblock=true. It is not clear which versions of PBS (Torque, PBSPro, OpenPBS) actually support this, but it should be added
https://lists.gnu.org/archive/html/help-make/2017-10/msg00004.html

Fixed by #213


FkRecords.java: /DEV message key is not suppo...

The puzzle 432363033 originating from #303 has to be resolved:
https://github.com/eo-cqrs/eo-kafka/blob/b5678e759eb76e884e1e9e9b8545150f1c1582f1/src/main/java/io/github/eocqrs/kafka/fake/FkRecords.java#L80-L80
"/DEV message key is not supported".
The puzzle was created by h1alexbel at 2023-06-28 17:49:09 +0300.
Estimation is 45 minutes.
If you have any technical questions, don't ask me, I won't be able to help. Open new issues instead.

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2024-02-15T01:19:31.298907.
Estimation here is 45 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@l3r8yJ Don't forget to close this ticket before the deadline (2024-02-15T01:19:31). You are past the first half of the allowed period.

@l3r8yJ Looks like you've missed the task deadline (2024-02-15T01:19:31). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2024-02-25T01:29:57.523874.
Estimation here is 45 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@h1alexbel Don't forget to close this ticket before the deadline (2024-02-25T01:29:58). You are past the first half of the allowed period.

@h1alexbel Looks like you've missed the task deadline (2024-02-25T01:29:58). You are now resigned from this task.
Please stop working on it, you will not be paid. I will assign it to someone else soon.

@h1alexbel I couldn't find any assignee for this task. This is either because there are no contributors with role DEV available or because the project does not have enough funds.
Please, make sure there is at least one available contributor with the required role and the project can afford to pay them.


cm is not a funciton

seems cm is a ES Module, It can not be run as a function.

I use codemirror@5, it works

I have the same bug alert.

I use codemirror@5, it works
codemirror@5 can not set cursor successfully


Revert "Text inline widgets, TextSpan rework (#33946)"

This reverts commit 14414f350ac040f9626f1aa0b3071f42cc9136a6.
Pending changes to allow smooth plugins upgrading

This is on hold as the fix forwards has landed (https://github.com/flutter/flutter/commit/160a5680aedc4eb6b7bfc9131752c7ada6b2e8e0).

Latest luci mac builder is green after "Infra Failure" error. Flutter-build is green.


Update source-distribution.xml

modify ignored item
Fixes #ISSUSE_ID.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 812

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 68.529%

Totals

Change from base Build 540:
0.006%

Covered Lines:
11495

Relevant Lines:
16774

💛  - Coveralls


Can you give an example about the imported data format on Conversational AI/Slot Filling and Intent Classification?

I try to import data to this module but always failed. I have tried the following format:

JSON

[
	{
		"author": "Human",
		"text": "Hi, Robot!"
	 Tell me what you want."
	}
]

it raised [73c4adce-99f5-4742-848f-ec7fba836235] Validation error (Failed to parse input file upload/13/8028ed8f-bot_conversation.json: Unexpected character found when decoding array value (2)).

CSV

Human,Robot
"Hi, Robot!","Nice to meet you, man! Tell me what you want."

it raised [Validation error] Error at item 0: "humanMachineDialogue" key is expected in task data [assume: item["data"] = task root with values] :: {'data': {'Human': 'Hi, Robot!', 'Robot': 'Nice to meet you, man! Tell me what you want.'}, 'file_upload_id': 50}

Can u plz give an example of .json file rather than rendered style to help me with this? THX a lot!

You can get example of input using the network panel of your browser:

You should use this task structure:
{
    "humanMachineDialogue": [
        {
            "author": "Human",
            "text": "Hi, Robot!"
         Tell me what you want."
        },
        {
            "author": "Human",
            "text": "Order me a pizza from Golden Boy at Green Street "
        },
        {
            "author": "Robot",
            "text": "Done. When do you want to get the order?"
        },
        {
            "author": "Human",
            "text": "At 3am in the morning, please"
        }
    ]
}

@makseq How to annotate each utterance's label?


Add glow button

Fixes Issue🛠
Closes #64
Description👨💻
Added a beautiful button with rainbow glow effect

[X] New feature (non-breaking change which adds functionality)

Checklist✅

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added demonstration in the form of GIF/video file
[X] I am an Open Source Contributor

Screenshots/GIF📷

Good one, Merged!! Congrats on getting your PR merged to this repo under SWOC S4 🎉🥳 @rishicds

Nice feature @rishicds

thank you @GhoshAditi !


Setting timeout has no effect

iControl attempts to set session timeouts here, however requests Session has no timeout attribute, instead it must be passed as a kwarg to get()(request()) as seen here

I see the same behavior. The value needs to be sent as an argument in the operation per http://docs.python-requests.org/en/master/user/advanced/

It appears that this line: https://github.com/F5Networks/f5-icontrol-rest-python/blob/1.0/icontrol/session.py#L415 has been deprecated.

archiving this repo, this request will not be implemented.


Implement a TCP, binary-based interconnection protocol

As a developer, I need to be able to use a protobuffer generator that uses simple TCP (UDP?) connections and protobuff for arguments in order to have efficient communication between nodes.
Definition of Done:

[ ] The generator has a way to listen to a port and do "routing"
[ ] "Routing" is the act of directing a message to one of the interface's method according to the first byte (or some other data, open to discussion)
[ ] Once the route has been chosen, the arguments are deserialized according to the method chosen
[ ] The act of routing does not allocate
[ ] Routing is nothrow
[ ] There is a client class that can be instantiated with an API as template parameter and an address / port as parameter
[ ] The client class has support for timeouts
[ ] The client class can perform asynchronously (either by itself or through runTask)
[ ] Both client and server code are critical for performance, hence the code is performant.
[ ] There is support to reject inbound peers (agora #202 )

Since I wrote LocalRest, which is very similar, I'll assign this one to myself.

After #2520 is there something that is left in this issue ? #2526 is an optimization (which we want), but I think all the features are here @omerfirmak ?

Yeah, all I can think of are optimizations. Closing this.


Angular Forms and Form Validation

Use Angular support for HTML forms
Establish two-way data binding between form fields and JavaScript variables and object properties.
Use Angular's support for form validation
Use JavaScript to perform form validation in Angular.

Done


Use deterministic permissions and times

Use deterministic directory timestamps
Use deterministic permissions

Also make some test tweaks.

I guess it's okay, since tests pass, but it's a lot of changes clumped together, so I feel it needs quite some testing.

Yes, needs a bit of time before it hits production.


Update zplug.zsh

resolve macos m1 :
(eval):1: ./zplug.zsh: bad interpreter: /usr/bin/zsh: no such file or directory

What about using /bin/zsh?

I mean /bin/zsh, not /usr/bin/zsh.

Yes, /bin/zsh yes. Why not unify with zsh-yarn-autocompletions ?

Oh, I should update it in zsh-yarn-autocompletions as well.

zsh-yarn-autocompletions was updated several days ago. Would you like to update here as well?


Can't delete objects using deleteObjects: getting the next error from seaweedfs log: panic: runtime error: index out of range

No matter whats is the amount of objects we try to delete we get the same error of connect ECONNREFUSED, and the log is: goroutine 611 [running]: github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).DeleteMultipleObjectsHandler.func2(0xc0009cfc80, 0x27) 	/go/src/github.com/chrislusf/seaweedfs/weed/s3api/s3api_object_handlers.go:189 +0x26c github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).streamRemove.func1.1(0x559de676ed20, 0xc000873590, 0xc0005290e0, 0xc0008f7e90) 	/go/src/github.com/chrislusf/seaweedfs/weed/s3api/filer_util.go:165 +0x42 created by github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).streamRemove.func1 	/go/src/github.com/chrislusf/seaweedfs/weed/s3api/filer_util.go:153 +0x273 I0320 16:00:24     1 config.go:27] Reading security.toml from  I0320 16:00:24     1 config.go:30] Reading : Config File "security" Not Found in "[/ /root/.seaweedfs /etc/seaweedfs]" I0320 16:00:24     1 tls.go:51] load cert/key error: open : no such file or directory I0320 16:00:24     1 s3.go:138] S3 read filer buckets dir: /buckets I0320 16:00:24     1 s3.go:145] connected to filer seaweedfs-filer-localnode.tls.ai:8888 grpc address seaweedfs-filer-localnode.tls.ai:18888 I0320 16:00:24     1 s3.go:178] Start Seaweed S3 API Server 30GB 1.61 at http port 8333 panic: runtime error: index out of range [116] with length 116

Could you add more logs. Seems there are other panic errors down below.

For ECONNREFUSED, this shows it is failing to connect to filer gRpc port.
Usually filer port is 8888, the gRpc port is 8888+10000=18888.

thanks for the response, here is the rest of the log:
`anic: runtime error: index out of range [116] with length 116
goroutine 611 [running]:
github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).DeleteMultipleObjectsHandler.func2(0xc0009cfc80, 0x27)
/go/src/github.com/chrislusf/seaweedfs/weed/s3api/s3api_object_handlers.go:189 +0x26c
github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).streamRemove.func1.1(0x559de676ed20, 0xc000873590, 0xc0005290e0, 0xc0008f7e90)
/go/src/github.com/chrislusf/seaweedfs/weed/s3api/filer_util.go:165 +0x42
created by github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).streamRemove.func1
/go/src/github.com/chrislusf/seaweedfs/weed/s3api/filer_util.go:153 +0x273
I0320 16:00:24     1 config.go:27] Reading security.toml from
I0320 16:00:24     1 config.go:30] Reading : Config File "security" Not Found in "[/ /root/.seaweedfs /etc/seaweedfs]"
I0320 16:00:24     1 tls.go:51] load cert/key error: open : no such file or directory
I0320 16:00:24     1 s3.go:138] S3 read filer buckets dir: /buckets
I0320 16:00:24     1 s3.go:145] connected to filer seaweedfs-filer-localnode.tls.ai:8888 grpc address seaweedfs-filer-localnode.tls.ai:18888
I0320 16:00:24     1 s3.go:178] Start Seaweed S3 API Server 30GB 1.61 at http port 8333
panic: runtime error: index out of range [116] with length 116
goroutine 1300 [running]:
github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).DeleteMultipleObjectsHandler.func2(0xc00083fbc0, 0x27)
/go/src/github.com/chrislusf/seaweedfs/weed/s3api/s3api_object_handlers.go:189 +0x26c
github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).streamRemove.func1.1(0x5648929d2d20, 0xc0005db9c0, 0xc0009417a0, 0xc00084ca20)
/go/src/github.com/chrislusf/seaweedfs/weed/s3api/filer_util.go:165 +0x42
created by github.com/chrislusf/seaweedfs/weed/s3api.(*S3ApiServer).streamRemove.func1
/go/src/github.com/chrislusf/seaweedfs/weed/s3api/filer_util.go:153 +0x273
`

Great! This is a bug and I will try to fix it.

Added fix. If you do not build from git tip, you may need to wait for the next release.

@chrislusf thank you! and yes we need a new release for this fix.


remove Dockerfile or use it with workflows?

A Dockerfile is not needed for GH workflows, dependencies are easy to install to GH runner in the workflow yaml.
Is there any need to maintain and retain the Dockerfile and build the custom image?

Probably should modify workflows to use the custom image / Dockerfile

removing docker-compose, but using the dockerfile


misc/cgo/testplugin: test failing on macOS 12 (arm64) on the 1.17 release branch

On the 1.17 release branch (but not the 1.16 branch nor tip), the macOS 12 (arm64) builder has been failing since it was brought up.
Failures look like:
--- FAIL: TestIssue25756pie (1.20s)
    plugin_test.go:274: ./issue25756pie.exe: exit status 2
        panic: plugin.Open("life"): plugin was built with a different version of package sync/atomic

        goroutine 1 [running]:
        panic({0x10476ed20, 0x14000012270})
        	/tmp/buildlet/go/src/runtime/panic.go:1147 +0x41c fp=0x14000105ea0 sp=0x14000105dc0 pc=0x10469c74c
[...]

From https://build.golang.org/log/8d6c23e736ff876b9ec8c34f4dd3bb6ed497c56e. See https://build.golang.org/?branch=release-branch.go1.17.
This may be a problem with Go, with the builder (it still has a known issue set, #49149), or macOS 12.
CC @golang/release, @mknyszek (in case this is related to #49923).

Maybe this is a duplicate of #46239.

This is indeed the same as #46239. That test has a skip that applies to the macOS 11 builder, which explains why it only fails on the macOS 12 one, due to the different name:
if os.Getenv("GO_BUILDER_NAME") == "darwin-arm64-11_0-toothrot" {
	t.Skip("broken on darwin/arm64 builder in sharded mode; see issue 46239")
}

Closing in favor of #46239 which has more information.


[feat]:  ready to migrate

Tag Name
pfe-timestamp
RHDS Only

[ ] The component will be RHDS only, delete the PFE component once published

Additional Info
PFE Timestamp has already been updated in PF 1:1. We now need an <rh-timestamp> component for RHDS. No design tweaks necessary.

@kylebuch8, I believe @zhawkins had an idea for timezone implementation. Can you sync up with him as you tackle this component?

Intl.DateTimeFormat() does a lot of the heavy lifting for us and comes with the browser.

@markcaron confused by the bullet point in the description that says RHDS only...we're still keeping pf-timestamp, right? I do see PF4 has it as well, although it's a beta feature. If we're keeping in both libraries, should I explore using controllers instead of creating a base class/extending and all that jazz @bennypowers? If there's anything else I need to tackle as part of the feature, let me know.

yeah if you can abstract out the computation parts to a controller that'd let us ship both without extending

Timestamp controller went out with pfe-core 2.4.0

@hellogreg Do we need a separate docs issue or should we add the docs work to this issue?

In the previous releases we had separate issues for things like docs, design, and dev. while there are advantages to that, it could also be the case that separating issues out by role led to some confusion and ultimately cramming for the release. Could we try a more holistic approach to tracking that treats docs, design, accessibility and dev as parts of a whole rather than isolated, independent tasks?

@coreyvickery I agree with @bennypowers. Let's try keeping the docs issue here.

@eyevana Ready for you.
XD: https://xd.adobe.com/view/a749fc5b-c52f-4a97-beae-30e655c89bb6-f21a/
Content doc: https://docs.google.com/document/d/1o0XUMKqez5bt4u7dtGlLIfL-m9IXuMyYUislYqgHSX4/edit
Images: https://www.dropbox.com/sh/a8esxfydh50rhx6/AAAZzIxlqtvHGfuCQsREkv3Ua?dl=0

@coreyvickery Can you provide the image and summary for the elements page, please?

@eyevana Done, check your alerts.


Added support for danish,norwegian and swedish

Wow great! Thank you for the contribution! :+1:

Oops, I didn't see the test files are missing. Can you add them please?


Panic when running SHOW DIAGNOSTICS with text/csv

Bug report
System info: InfluxDB 1.5 and likely older
Steps to reproduce:
$ curl -G 'http://localhost:8086/query' -H 'Accept: text/csv' --data-urlencode 'q=SHOW DIAGNOSTICS'

Expected behavior: It likely shouldn't panic.
Actual behavior: It does panic.
Additional info: This is likely because SHOW DIAGNOSTICS is weird. It is the only command that changes which columns exist in the row within the same statement, which is pretty weird. Most queries use the same columns for the entire statement, but this command doesn't.

This is a recoverable panic so it doesn't kill the server.


Templatetag: "render_model_add" should allow you to wrap any arbitrary content

Currently, {% render_model_add %} is sorta limited to presenting an icon, which, to be fair you can replace and re-position with CSS. I think it would be tremendously more useful if it could be used to wrap any arbitrary content. This would provide the developer a lot more options in creating frontend interfaces that are more in-keeping with the style and format of the site they are building.

Fixed by #3875