Fix direct links to Generated API Docs page

Support opening the API docs link in a new browser (i.e. incognito mode).
Currently, opening the CiviForm home page in incognito, unauthenticated, followed by opening the API docs page (without logging in as an admin) works, but opening the API docs page in incognito mode directly causes you to get a sign-in prompt.

It looks like /api/ routes are configured to require basic authentication.
I don't understand why visiting the home page first would get around this. That being said, we should be able to work around this by excluding the '/api/docs/' routes from the config.


Which dataset was used for hand detection/tracking?

Hello,
I was wondering what dataset was used for building the hand detection/tracking model and if it is publicly available.
Thank you!

@Pikus16  The dataset is an internal google dataset. Unfortunately due to privacy reasons it will not be publicly available see #186 #13

I would like to inquire if there would be any issues related to dataset usage permission or copyright if I incorporate palm detection and pose landmark solutions into a product or app for commercial purposes.


Wayne Homan's Tina Poetry Slam

@izayak I believe you were @wfh1972's partner today.  Did you meant to merge this PR?


Aspera doc

Improves the cmd line download section of the Aspera download doc.
cc @manics @sbesson

Changes read fine. Merging. Should this be deployed ahead of the next IDR release?

yes, why not ? Except that it is too much pressure/haste


create conda environment

Given the need for specialist libraries such as geopandas, we'll need a conda environment
Best practice would be a .yml file, installed using
conda env create --file cgrasp.yml
conda activate cgrasp

I added a rudimentary version in the latest commit


quay defaults x86_64 arch to amd64

Attempting to resolve issue: https://github.com/containers-mirror/ai-lab-recipes/actions/runs/8802778394/job/24159340863#step:3:43
Theory, stream9 tag does not have x86_64 arch target, only amd64, see stream9 tag manifests

LGTM


[v5] Does not work with RTL with nested selectors

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

Current Behavior 😯
On version 5.0.0-alpha.25, when using RTL with emotion strategy Grid component does is affected. Negative margins push the grid content outside of Card component and gets cut out.
Expected Behavior 🤔
Should remain visually as LTR mode.
Steps to Reproduce 🕹
Steps:

Open https://next.material-ui.com/components/grid/#main-content
Go to settings (top bar) and change to "Right to Left"
View examples

We depend on https://github.com/styled-components/stylis-plugin-rtl/issues/22.

Is this still broken? I could not notice anything on the prod version. Anyway, I prepared https://github.com/mui-org/material-ui/pull/25661 for updating the stylis-plugin-rtl dependency.

Update the stylis-plugin-rtl to version 2.0.2 which includes the fix https://github.com/styled-components/stylis-plugin-rtl/pull/21 that will resolve the issue

Great 🙏


Subs working with 1.5.0 -> stopped working with 1.6.0

Describe the bug
Subscriptions configured in the objects in io broker - which worked with 1.5.0 stopped working instantly with 1.6.0.
e.g.  Zigbee lights no longer react on a published "true" for the switch data point - but also all other subscriptions stopped.
To Reproduce
Steps to reproduce the behavior:

Have subscriptions working with 1.5.0
Update to 1.6.0
Try switching a light or changing something over MQTT
-> obejcts don't geht the new value - even on loglevel silly - "nothing" gets logged
downgrade to 1.5.0 -> works again

Expected behavior
Subscribe should work - afaik :-)
Screenshots & Logfiles
Log -> no messages logged
Screenshot of my subscription not working with 1.6.0

Even a:
mosquitto_pub -h <IP_ADDRESS> -t 'homeland/haushalt/iobroker/zigbee/0/90fd9ffffee94de7/state' -m 'false' -u fhem -P "(fS.(HQbX&48rvsd2y/N"    oker/zigbee/0/90fd9ffffee94de7/state' -m 'true'
mosquitto_pub -h <IP_ADDRESS> -t 'homeland/haushalt/iobroker/zigbee/0/90fd9ffffee94de7/state' -m 'false' -u fhem -P "(fS.(HQbX&48rvsd2y/N"    oker/zigbee/0/90fd9ffffee94de7/state' -m 'false'
Does not work - iobroker does not get it.
Versions:

Adapter version: 1.6.0
JS-Controller version: 4.0.15
Node version: v12.22.10
MQTT Broker: Mosquitto 2.0.14
Operating system: RasbpberryPi OS & Rancher Kubernetes Cluster on Ubuntu 20.04.4 LTS

Do I get this right: On adapter start with debug logging there is nothing being logged, not even the start of the adapter, or connecting to broker? If the adapter is not starting it most certainly won't update states.
This is how to activate debug logging for an adapter:

This is what an adapter start should look like:
022-02-22 22:06:40.513  - debug: mqtt-client.0 (10962) Redis Objects: Use Redis connection: <IP_ADDRESS>:9001
2022-02-22 22:06:40.554  - debug: mqtt-client.0 (10962) Objects client ready ... initialize now
2022-02-22 22:06:40.557  - debug: mqtt-client.0 (10962) Objects create System PubSub Client
2022-02-22 22:06:40.559  - debug: mqtt-client.0 (10962) Objects create User PubSub Client
2022-02-22 22:06:40.560  - debug: mqtt-client.0 (10962) Objects client initialize lua scripts
2022-02-22 22:06:40.586  - debug: mqtt-client.0 (10962) Objects connected to redis: <IP_ADDRESS>:9001
2022-02-22 22:06:40.596  - debug: mqtt-client.0 (10962) objectDB connected
2022-02-22 22:06:40.599  - debug: mqtt-client.0 (10962) Redis States: Use Redis connection: <IP_ADDRESS>:6379
2022-02-22 22:06:40.609  - debug: mqtt-client.0 (10962) States create System PubSub Client
2022-02-22 22:06:40.611  - debug: mqtt-client.0 (10962) States create User PubSub Client
2022-02-22 22:06:40.621  - debug: mqtt-client.0 (10962) States connected to redis: <IP_ADDRESS>:6379
2022-02-22 22:06:40.622  - debug: mqtt-client.0 (10962) statesDB connected
2022-02-22 22:06:40.908  - debug: mqtt-client.0 (10962) Plugin sentry Do not initialize Plugin (enabled=false)
2022-02-22 22:06:40.916  - debug: mqtt-client.0 (10962) Plugin sentry destroyed because not initialized correctly
2022-02-22 22:06:40.924  - info:  mqtt-client.0 (10962) starting. Version 1.6.0 in  /opt/iobroker/node_modules/iobroker.mqtt-client, node: v14.19.0,  js-controller: 3.3.22
2022-02-22 22:06:41.011  - debug: mqtt-client.0 (10962) enabled syncing of 0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.TV (publish/subscribe:true/false)
2022-02-22 22:06:41.012  - debug:  mqtt-client.0 (10962) enabled syncing of  0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.power  (publish/subscribe:true/false)
2022-02-22 22:06:41.012  - debug:  mqtt-client.0 (10962) enabled syncing of  0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.powerAVR  (publish/subscribe:true/false)
2022-02-22 22:06:41.013  - debug:  mqtt-client.0 (10962) enabled syncing of  0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.subwoofer  (publish/subscribe:true/false)
2022-02-22 22:06:41.013  - debug: mqtt-client.0 (10962) enabled syncing of mqtt-client.0.pi4home.clients.iobroker.online (publish/subscribe:false/true)
2022-02-22 22:06:41.014  - debug: mqtt-client.0 (10962) enabled syncing of mqtt-client.0.pi4home.clients.tester.online (publish/subscribe:false/true)
2022-02-22 22:06:41.014  - debug: mqtt-client.0 (10962) enabled syncing of mqtt-client.0.pi4home.clients.tester (publish/subscribe:false/true)
2022-02-22 22:06:41.015  - debug:  mqtt-client.0 (10962) complete Custom:  {"0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.TV":{"enabled":true,"publish":true,"pubChangesOnly":true,"pubAsObject":false,"qos":0,"retain":true,"subscribe":false,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":false,"topic":"0_userdata/0/virtualDevice/Entertainment/Wohnzimmer/TV","type":"boolean"},"0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.power":{"enabled":true,"publish":true,"pubChangesOnly":true,"pubAsObject":false,"qos":0,"retain":true,"subscribe":false,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":false,"topic":"0   userdata/0/virtualDevice/Entertainment/Wohnzimmer/power","type":"number"},"0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.powerAVR":{"enabled":true,"publish":true,"pubChangesOnly":true,"pubAsObject":false,"qos":0,"retain":true,"subscribe":false,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":false,"topic":"0   userdata/0/virtualDevice/Entertainment/Wohnzimmer/powerAVR","type":"number"},"0_userdata.0.virtualDevice.Entertainment.Wohnzimmer.subwoofer":{"enabled":true,"publish":true,"pubChangesOnly":true,"pubAsObject":false,"qos":0,"retain":true,"subscribe":false,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":false,"topic":"0   userdata/0/virtualDevice/Entertainment/Wohnzimmer/subwoofer","type":"boolean"},"mqtt-client.0.pi4home.clients.iobroker.online":{"enabled":true,"topic":"pi4home/clients/iobroker/online","publish":false,"pubChangesOnly":false,"pubAsObject":false,"qos":0,"retain":false,"subscribe":true,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":true,"type":"mixed"},"mqtt-client.0.pi4home.clients.tester.online":{"enabled":true,"topic":"pi4home/clients/tester/online","publish":false,"pubChangesOnly":false,"pubAsObject":false,"qos":0,"retain":false,"subscribe":true,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":true,"type":"mixed"},"mqtt-client.0.pi4home.clients.tester":{"enabled":true,"topic":"pi4home/clients/tester","publish":false,"pubChangesOnly":false,"pubAsObject":false,"qos":0,"retain":false,"subscribe":true,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":true,"type":"mixed"}}
2022-02-22 22:06:41.015  - debug: mqtt-client.0 (10962) found 1 additional topic to subscribe to
2022-02-22 22:06:41.016  - info:  mqtt-client.0 (10962) Try to connect to  mqtt://<IP_ADDRESS>:1883?clientId=pi4home, protocol version 4 with lwt  "clients/iobroker/online"
2022-02-22 22:06:41.110  - info: mqtt-client.0 (10962) connected to broker
2022-02-22 22:06:41.115  - debug: mqtt-client.0 (10962) trying to subscribe to 3 topics
2022-02-22 22:06:41.117  - debug: mqtt-client.0 (10962) trying to subscribe to 1 topics
2022-02-22 22:06:41.128  - debug: mqtt-client.0 (10962) successfully subscribed to 3 topics
2022-02-22 22:06:41.129  - debug:  mqtt-client.0 (10962) subscribed to:  {"pi4home/clients/iobroker/online":0,"pi4home/clients/tester/online":0,"pi4home/clients/tester":0}
2022-02-22 22:06:41.131  - debug:  mqtt-client.0 (10962) received message false for id  mqtt-client.0.pi4home.clients.iobroker.online=>{"enabled":true,"topic":"pi4home/clients/iobroker/online","publish":false,"pubChangesOnly":false,"pubAsObject":false,"qos":0,"retain":false,"subscribe":true,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":true,"type":"mixed"}
2022-02-22 22:06:41.132  - debug:  mqtt-client.0 (10962) received message true for id  mqtt-client.0.pi4home.clients.tester.online=>{"enabled":true,"topic":"pi4home/clients/tester/online","publish":false,"pubChangesOnly":false,"pubAsObject":false,"qos":0,"retain":false,"subscribe":true,"subChangesOnly":false,"subAsObject":false,"subQos":0,"setAck":true,"type":"mixed"}
2022-02-22 22:06:41.133  - debug: mqtt-client.0 (10962) successfully subscribed to 1 topics
2022-02-22 22:06:41.134  - debug: mqtt-client.0 (10962) subscribed to additional topics: {"pi4home/clients/#":0}

;-) Sorry - no startup and co is logged - but in the moment where I publish on MQTT "nothing"  is logged - no error no action belonging to my publish.

Please post a screenshot of you instance settings, especially prefixes for subscribtions and publishes aswell as a log of the startup, at least everything related to connecting to your broker and one of the not working states, e.g. 'enabled syncing of ....'.
Remember to remove any private information as passwords or broker URLs.

Sorry for the delay - you've already fixed it.
Nice work!
Thank you!


disabling rook mgr module in CephCluster

Sometimes, during installation with ODF, the rook mgr module in ceph cluster is being enabled. This pr ensures that it is disabled during installation.
For example:
sh-4.4$ ceph mgr module ls | grep rook
rook                  on
With this change:
sh-4.4$ ceph mgr module ls | grep rook
rook                  -

/cherry-pick release-4.17


JIT leads to overflow error for large integers in X64

import jax.numpy as jnp
import numpy as np
from jax import config, jit
config.update('jax_enable_x64', True)

big = np.iinfo(np.int64).max

print(np.array(big).dtype)  # int64
print(np.array(big + 1).dtype)  # uint64

print(jnp.array(big).dtype)  # int64
print(jnp.array(big + 1).dtype)  # uint64

print(jit(jnp.array)(big).dtype)  # int64
print(jit(jnp.array)(big + 1).dtype)  # OverflowError: Python int too large to convert to C long

Seems like the bug is that
print(jnp.array(big + 1).dtype)  # uint64

should error as well?

I think I prefer the error to silently overflowing, which is what NumPy does. (And if you do want NumPy's behavior, you can first cast to a NumPy array.)

Just to clarify, when you say "silently overflowing" are you referrring to the behavior of switching to uint64 if the value is too large for int64, and swittching to dtype('O') if the value is too large for uint64?

Yeah I guess "overflowing" isn't a correct description, on further thought.
Maybe we should follow NumPy here. We could certainly have the same uint64 behavior for jit of integers outside the int64 range. We can't use dtype('O'), of course, so we'd have to error in that case.

OK, that sounds like a reasonable approach.
The reason I was going down this route is related to #5952: I need to figure out what to do in X32 mode with Python integers between int32.max and uint32.max. If we choose to promote to uint64 here, we should probably choose to promote to uint32 there.


add github-ci

Hello!
I add github-actions and dependabot for github-actions. I hope it is useful.
Thanks.

@zaher Could you, please, revert my PR?

Ok but after checking it, I want to understand it, thank you in anyway :)
Maybe i suspend it for a while until we can get new release


CSS not displayed properly with latest version

I am using the latest version (2.1.4 on npm) which allows me to add customized headers while sending the post request. However the CSS doesn't seemed to display correctly. 2.1.3 and prior versions worked properly.

Hi @DEDZTBH
Thanks for the report, I've just pushed up v2.1.5 to npm so hopefully this resolves the issue, if not let me know.
Cheers
Rowan

Unfortunately the new version still looked the same way as 2.1.4 did...
The 'remove' looked like a link instead of a button like previous versions, and the done icon (I'm using font awesome) looked tiny and not centered so I wonder if these css failed.
Or as I noticed, they looked different from older versions because of change of style?

Grrr, alright try again with 2.1.6, Im pretty sure it's sorted this time sorry.

It worked! Thanks!


Removal of GetDecision and DeliberateVote methods

Issue 1: Infrastructure team is going to expect a particular type returned for each vote e.g. map[uuid.UUID]bool for voting to accept agent. Currently our functions return interfaces which will have to be asserted with the type infrastructure is expecting.
Solution: For each voting function let's just return the type that infrastructure is expecting and rather than an interface type Removes need for asserting the type later in the code.
Issue 2: Go is statically typed, so our GetDecision() and DeliberateVote() methods can only return a certain type. But, each vote will require us to return a different type e.g. map[uuid.UUID]bool or map[uuid.UUID]float64. My understanding is that both of these methods simply call the correct vote function.
Solution: Let's just remove the GetDecision() and DeliberateVote() methods and place the voting function directly in the baseBiker method when we override it e.g. in DecideJoining() we calle VoteToAcceptWrapper
Issue 3: For voteInputs, each vote type may have a different type of Candidate e.g. list of agent UUIDs or list of governance types reperesented as ints.
Solution: Make a CandidateType struct consisting of all of the possible candidate types. Then, only input the relevant candidate type fromm the CandidateType struct into our voting function e.g. VoteToAcceptWrapper will only be provided with Candidates.AgentCandidate which is a list of agent UUIDs.
Happy to have a meeting to discuss and explain further.

Hey Basheq, cool, your idea sounds better as its more consistent with the other frameworks such as NavigationDecisionFramework. Just so I'm clear, would the code then look something like below? Where would the IdVoteHandler come into it? Or is the VoteOnJoinHandler the IdVoteHandler in this case? Please correct me if I'm wrong anywhere! :)
type VoteOnJoinHandler struct {
   IDecisionFramework[IdVoteInputs, IdVoteMap]
  *inputs IdVoteInputs
}

func (*vojh VoteOnJoinHandler) GetDecision(IdVoteInputs) IdVoteMap {
   // call our current VoteToAcceptWrapper function
   return vote
}

func NewVoteOnJoinHandler() *VoteOnJoinHandler {
   return &VoteOnJoinHandler{
      inputs: &IdVoteInputs{},
   }
}

Yeah that looks exactly like what I had in mind!


Adding debugger for languages?

Hi,
How hard/compilcated do you think it is to add a debugger option like this. ?
I went through the roadmap but didn't see any plans of adding debugger and was thinking if maybe I can help with it, but I am unsure on how one would go about it?
Any ideas?

I was trying to understand how this website does it but i noticed that no http/websockets calls that go when i run/debug the code. Does this mean its just compiling every language to javascript and just executing it on browser?

See https://github.com/radian-software/riju/issues/168, just FYI.
That said, let me see what I can do about answering your questions.

How hard do I think it would be? Probably a big project, but doable.
Is it on the roadmap? Yeah, I didn't have any plans for it, and I have other things on my mind, but it doesn't seem out of line for the project, so if it were implemented (and I get things unbroken enough to actually deploy updates) then I wouldn't object to merging it in.
How to go about it? I'm not sure. I'd have to do a survey of different languages' debuggers to see what the typical behavior is. You'd want to start by coming up with a set of abstractions that describes a debugger for an arbitrary language, and then figure out how those abstractions could be configured to invoke different language debuggers specifically. Then you'd wire up buttons on the frontend to trigger the relevant configurations on the backend, like how LSP is set up currently.
How does onlinegdb work? Looks like it's got a websocket to me, you might have just missed it.

I'm not 100% sure how the debugger integration works, it's possible that the buttons are literally just sending words like continue and next on stdin to the pty, although that doesn't quite explain all the behavior I see. pdb might have some kind of control port that can be used to issue commands out of band.

Ya they use websockets, my bad. Most likely I opened my network tab after the websocket connection was established, so missed it.
Thanks for the input. Will try to make one for python just to get an idea of how things happen.

PS: Awesome project!

Yeah, if you want to have a good time, rather than starting from the mess I have made in Riju, you might want to actually take https://github.com/radian-software/python-in-a-box as a starting point.

(Same underlying technology but way simpler because it does not attempt to do everything.)

Yeah, if you want to have a good time, rather than starting from the mess I have made in Riju, you might want to actually take https://github.com/radian-software/python-in-a-box as a starting point.

This is really easy to understand. Thanks!

Hi,
This is what i could come up with:
https://github.com/RahulBadenkal/python-in-a-box-debugger
Reference loom: https://www.loom.com/share/0f0d65d0087b46babfc8f1e1c03515ae
Minor change:

Using built in child process package of node instead of node-pty. As needed to separate out stdin and stdout content.

Idea:

Same as what you suggested last time, using python3's pdb module and piping the output from the terminal to browser.

Trying to replicate pycharm's debug functionality, but faced a few issues right off the bat:

The program in pdb auto restarts after ending, i would prefer it to terminate
Not sure how to get the current line number. The info is there in the terminal, but the only way i can think of extracting is some sort of regex/string processing but then since the data from the terminal comes in batches and not all together not sure how to apply regex on a continuos stream?


Setup a basic CI/CD system

Let's use the basic version of travis ci to get unit tests and simple integration test using local docker builder.

/assign rpasricha

/assign rpasricha

/priority p0


Environment option is "production"

If environment option is configured and is "production" then use the default hosts. Part of http://docs.ably.io/client-lib-development-guide/features/#RSC11.

It's not "use the default hosts" but "don't prepend the environment (ie. production-) to the hosts", custom or not. environment="sandbox",restHost="fake.ably.io" should send requests to sandbox-fake.ably.io, while environment="production",restHost="fake.ably.io" should send them to fake.ably.io.


Record redirects and add wp2static_list_redirects filter

Closes #566.
This adds two columns to the crawl cache table: status and redirect_to. To record this information, the table is written to even if it is disabled for caching purposes.
Redirect targets are detected, but are not added to the crawl queue. This should happen, but it's probably better to handle that later on.
This adds a wp2static_list_redirects filter that takes and returns an array of redirects. The array is indexed by URL. Each redirect has the form ['url'=> '...', 'redirect_to' => '...']. Deployment addons can use this to get a list of redirects to create. Other addons can add redirects from other sources (.htaccess, redirection plugins, etc.)
Example of the wp2static_list_redirects array format:
['/favicon.ico' =>
  ['url' => '/favicon.ico',
   'redirect_to' => '/wp-admin/images/w-logo-blue.png'],

 '/page/1/' =>
  ['url' => '/page/1/,
   'redirect_to' => '/']
]

Bloody beautiful, @john-shaffer  - elegant as always. Thanks so much for your time on this!


Customizing individual columns that are not in additional rows

It's quite common IMO to have selection or edit/delete controls as first column in a table. While there's a great way to customize columns in "additional" rows using Tr->Td..., I haven't found a way to insert  controls into columns that would trigger React events.

Something like:
var data = [{a: 1, b: 2}, {a: 3, b: 4}];

React.renderComponent(

        {data.map(function(row) {
            return (

                    {row.a}
                    {row.b}

                        Delete

            )
        })}

,
    document.getElementById('my-table')
);

should work. Remember that this is all still React, and anything you can do with React you can do with Reactable
Feel free to reopen the issue if you run into any other issues :smile:

@glittershark Do you still feel this is a valid solution? It seems like your response is trying to do what this pull request is trying to enable: https://github.com/glittershark/reactable/pull/196


document generation wanted

@ouonline  Is there any plan that ppl.cv also generate doxygen documentations like ppl.nn does?

use doxygen docs/Doxyfile to generate docs.


Exclude markdownIt from vendor

What problem does this feature solve?
Currently, I have a huge bundle size because prism.js is included in my vendor build. I want to avoid the Mb cost...
What does the proposed changes look like?
Essentially, the markdownit component needs to be available to the asyncData instance directly. I was thinking I could just use require in asnycdata—not sure if this will actually work—but if it were an option in the module that would be a lot easier. Some ways to do that are discussed in the nextjs blog post. https://arunoda.me/blog/ssr-and-server-only-modules
I tried the second option but couldn't get it working... maybe someone had success with another one of the options?

This feature request is available on Nuxt.js community (#c125)

Is there still no simple solution out there to render markdown on build? It seems silly to include this package when using static site generation.


Add identify env to cicd

Move env var files
Remove old infra files
Add first draft network terraform

PR to dev, will merge from there


view endorsements

The app can endorse (pin) accounts but doesn't show them on profiles.

doesn't seem to be returned by API


adding gen sprite, adjusting components to use old icon names

 (2) React:ions Implement Sprite Renderer


support absolute alias + fix extension appending

Currently aliasing to absolute paths is broken. Also, when the import path already includes the .js extension, it is still incorrectly appended.
This commit fixes both.

Coverage increased (+0.4%) to 97.727% when pulling d1b4f43a9ac929ced37402e374ce96dfd285544f on yyx990803:master into 4e6c1f32e5b050f901f43a4558604d777a769a13 on frostney:master.

Thank you so much for this. 👍

This just landed with 1.2.0.
I was a bit on the fence if it should be a new major (breaking) version number. Since I didn't intend for absolute paths before, I felt being bumping just the minor version indicating a new feature is a better fit.


Disable view menu items when window is hidden

When you minimize or close the main window, the window-related menu items become disabled.

Self-merging. Review still welcome :)

lgtm :+1:


Add success notification on submitted verification request

On successful submission of the verification request, we display a success notification on top of the page (detail view of the request). Here's what it looks like:

Please feel free to give suggestions on either styling or the copy.

hi @tortila  I'm holding off on reviewing this until Hannah has had a chance to respond about the preferred text and styling of the flash message. Now we have the email motifications merged, it might make sense for the copy to mentino that  as well now.

Hi @tortila As mentioned in today's call, please go ahead and merge this request, don't let me be a blocker. I can do the styling in a separate branch, that's absolutely no problem.

hi @tortila - this looks ok to me, but there is a merge conflict. Can you please resolve the conflict and merge in?

Thanks for the green light @mrchrisadams. This is now rebased against master branch and ready to merge


Don't Stringify JsonMetaData. Already handled by steem-js

Fix #311
Ref https://github.com/steemit/steem-js/blob/6ce1a43afa34a5c82509c3f15cf3cbb8642c0f3a/src/broadcast/index.js#L91

Can you add the app?

Added app


Introduce a Reset button

Button should ideally let you go back to the default settings while keeping the image in place.

Assigning to @kodbilenadam

Closed by #12


Compilling Error using RSA and SHA256 combination to verify

Hi guys, I'm trying to test the library combination of SHA256 and RSA to verify signatures made in this combination for a project.
When building the project, I get this wild error that I can't understand where it comes from or if I'm doing something wrong, since the documentation doesn't seem to explain this method very well.

error[E0277]: the trait bound CoreWrapper<CtVariableCoreWrapper<Sha256VarCore, sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<UTerm, B1>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>>>: digest::Update is not satisfied
--> src/container/file_format.rs:84:29
|
84  | ...                   PaddingScheme::new_pss::<Sha256, _>(OsRng),
|                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait digest::Update is not implemented for CoreWrapper<CtVariableCoreWrapper<Sha256VarCore, sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<UTerm, B1>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>>>
|
= note: required because of the requirements on the impl of digest::digest::Digest for CoreWrapper<CtVariableCoreWrapper<Sha256VarCore, sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<sha2::digest::generic_array::typenum::UInt<UTerm, B1>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>, sha2::digest::consts::B0>>>
note: required by a bound in PaddingScheme::new_pss
--> /home/neirth/.cargo/registry/src/github.com-1ecc6299db9ec823/rsa-0.5.0/src/padding.rs:143:33
|
143 |     pub fn new_pss<T: 'static + Digest + DynDigest, S: 'static + RngCore>(rng: S) -> Self {
|                                 ^^^^^^ required by this bound in PaddingScheme::new_pss

The code that I have written to replicate this comes to be something similar to this, which is inspired by the unit test of the method.
file.seek(SeekFrom::Start(4)).unwrap();
file.read_exact(&mut signature_buff).unwrap();

let mut hasher = Sha256::new();

file.seek(SeekFrom::Start(4 + 128)).unwrap();
copy(&mut file, &mut hasher).unwrap();

match RsaPublicKey::read_public_key_pem_file(public_key) {
    Ok(rsa_public_key) => {
        return rsa_public_key.verify(
            PaddingScheme::new_pss::<Sha256, _>(OsRng),
            &hasher.finalize(),
            &signature_buff
        ).is_ok();
    }
    Err(_) => panic!("[Error]: Failed to process the file {}", public_key)
}

I may be missing something, but I can't find any type of reference to save me from the step.

What versions of rsa and sha2 are you using?

@newpavlov, now, I'm using these versions:
sha2 = "0.10.1"
rsa = "0.5.0"

You were right, it was totally a problem of conflicting dependencies. Thanks so much for the help. :D


All white  ! Can’t see the quoted posts and many things !

@JustifydWarrior any thoughts?

Here's what it looks like on desktop. Looks this way no matter which theme is selected.

Posted that here. Thought it was a seperate glitch. Will close my issue.
https://github.com/bitclout/frontend/issues/301

Posted that here. Thought it was a seperate glitch. Will close my issue.
https://github.com/bitclout/frontend/issues/301

Apologies, missed that. Yeah I assume these are both related to the theming change.

Probably a few things overlooked by @JustifydWarrior
I synced the upstream and will look at it.
Stinks today was a super busy day for me with my kids so I'm on and off (Sunday)
Hopefully a few others can look too or Jus can get a moment.

.modal-content {
...
background-color: var(--secalt); <// Looks like this line  failing on invalid variable:

if you remove it, it defaults to white.
it seems to be missing from the themes, where before there was a default.
I believe the honest answer is that we need to add these to the themes, even if they refer to other vars

actually no, that doesn't work... something else must be setting secalt to transparent

okay... so it looks like the modal is outside the scope of the theme container, it needs access to the variables. they are all undefined for the modals. yes that's on me and my change for not testing the modals better.
I should have a fix soon. but i'm honestly, wondering about the transparent defaults and where they come from.

The only issue is the modal containers are added to the DOM outside the  tag. if you drag the modal inside, it works as expected.
This is an small change but perhaps significant, and seems like the proper way to do it. But I have no clue on how to do it.
I'll struggle around with it, but perhaps this is enough for someone faster to fix it.

ok so it looks like the directive has been placed into all the modals already, but they're all missing the [ngClass]="theme"

#307 should fix this

#307 should fix this

This makes issues in themes.
It changed Themes Colors, and Modal still show gray background and black text color. You have to fix that again.
https://github.com/bitclout/frontend/issues/305

Is it working now normally? Looks okay on my end? Can we close this?


Save All should save existing files even when an unsaved file is currently open

Feature request

VSCode Version: 1.10.2
OS Version: Windows 10 Home 1607

Steps to Reproduce:

Open an existing file
Make a change to said file
Create a new file (CTRL+N)
Make a change to the newly created file
Save all (CTRL+K S by default)

Expected behavior: the changes to the previously existing file are saved, and a 'Save As' dialog opens asking where to save the newly created file.
Actual behavior: as above, but the changes to the previously existing file are not saved.

@NiekPas I cannot reproduce the problem with 1.10.2 nor the latest insiders build on Windows 10. Is there anything specific about your setup or the files? Does the console in the dev tools (Help > Toggle Developer Tools) show any errors? (Back to you @bpasero)

@NiekPas ping

@bpasero Oddly, I cannot reproduce the bug any longer either, neither on VS Code 1.10.2 nor on 1.11.1. Very strange.


Map: Higher-Order bleed-through

There are certain places (two shown below) where the higher-order entity bleeds through the empty spaces of the lower order. In the case of Nevada, it is state data bleeding through the county gaps; in the case of Russia, it is country data bleeding through federal subject gaps.

Splitting the files to iso1 / iso2 / fips features.geojson is the solution to solve this. We'd be able to turn on and off individual layers on the map.

Actually https://github.com/covidatlas/coronadatascraper/pull/432 is a cleaner solution. For example in the US, we probably want state level data by default, not country one, so it's better not to use files for that.
Still, it'll be quite important to not load br_muni for example by default.

This is absolutely intended. Unless we have COMPLETE data for a given region, we render the higher order and the lower order data. This is determined by the aggregate property. For Russia, it's possible that property is set incorrectly, for Nevada, it's functioning as designed.

Could you clarify what you mean by complete data? Does this include (unassigned) counties?

I guess my point is that I'm not sure that this is working as expected (considering that Utah was incomplete but not showing aggregate) and that I'm not sure that I agree with the expectation -- that we should display higher-order on a lower-order map. The problem is it juxtaposes data we don't have a location for onto portions of the state that have not confirmed cases, essentially showing cases where there aren't any confirmed. I would recommend removing this feature during the #432 merge, so that the county toggle shows only confirmed counties / regions and the state map shows the aggregate.

This is definitely worth exploring more and elaborating on. Will keep this in mind as we re-work the map.

Closing, I believe this issue is stale / doesn't apply to Li / covidatlas.com.


BUGFIX: Prevent Drum Fill Flag from being applied in non-drum gamemodes

Prevents the Drum Fill Starpower Flag from being applied in non-drum gamemodes.

Fixed in https://github.com/FireFox2000000/Moonscraper-Chart-Editor/commit/c5c08a28403259971e43b191efccdf3e962fb02e


Failure with _ types

I am trying to run wit-bindgen over the cache.wit file from https://radu-matei.com/blog/intro-wasm-components/.
It does work fine if I install the32e63116d469d8046727fae3c1333a7d35d0c5d3 revision (as mentioned in the blog post), but not if I install the current main branch producing an error that it found an _ when a type was expected. Since the 32e63116d469d8046727fae3c1333a7d35d0c5d3 revision is already merged into main, I wonder if this is a regression.
> cargo install wit-bindgen-cli --git https://github.com/bytecodealliance/wit-bindgen --rev 32e63116d469d8046727fae3c1333a7d35d0c5d3

> wit-bindgen wasmtime --export cache.wit
Generating "bindings.rs"

> cargo install wit-bindgen-cli --git https://github.com/bytecodealliance/wit-bindgen

> wit-bindgen wasmtime --export cache.wit
Error: expected a type, found keyword `_`
     --> cache.wit:14:43
      |
   14 | delete: function(key: string) -> expected<_, error>
      |                                           ^

Here is a minimized wit file to reproduce the issue:
delete: function() -> expected<_, _>

And the full cache.wit file for reference:
// Type for cache errors.
enum error {
    runtime-error,
    not-found-error,
}
// Payload for cache values.
type payload = list<u8>
// Set the payload for the given key.
set: function(key: string, value: payload, ttl: option<u32>) -> expected<_, error>
// Get the payload stored in the cache for the given key.
get: function(key: string) -> expected<payload, error>
// Delete the cache entry for the given key.
delete: function(key: string) -> expected<_, error>

_ was replaced with the explicit unit type.

Thanks for the clarification. I have made a PR (https://github.com/bytecodealliance/wit-bindgen/pull/226) to reflect this in the format documentation.

_ is still referenced in the Types section of the grammar documentation:
expected ::= 'expected' '<' expected-ty ',' expected-ty '>'
expected-ty ::= '_'
              | ty


Add sync_images() method to spmd_block class

This patch adds a new method for spmd_block class called sync_images() to synchronize only a subset of images as opposed to sync_all(). The chosen approach is to manage concurrentlly a map of barriers that is expanded each time a new combination of image ids is given via a sync_images() call. Local and distributed versions have been implemented and unit tests have been changed to test those features.

@atrantan Would you be able to create some documentation for this, please?

@hkaiser Done in c60a41d4ba8963a533cae951c6281f5b7bf468b1


fix: Fix issue where title image is stretched

Fixed the stretching of the title image by adding an objectFit="contain"

Resolve conflicts and use "Toyota Presents" logo


Canonical naming of variants of Platform releases

In the release notes for 2021.09, there is no clear terminology to distinguish different variants of the Platform release, that is, the different collections with Coq and packages that comprise the release. This means that one can no longer unambiguously claim that a certain Coq project repo commit or tag is *Coq Platform 20XX.YY compatible", since the project may work with one combination of Coq version and package collection but not another in the same Platform release.
For example, I might have a project compatible with Coq 8.13.2 and the 2021.02 collection of packages that is not compatible with 8.13.2 and the 2021.09 collection. To make matters worse, my project might not even be compatible with the Coq Platform 2021.02 release, since it relies on some bugfix to the 2021.02 packages that only appeared in the 2021.09 release.
To enable making verifiable compatibility claims, I think there should be some official terminology to pinpoint a specific opam switch generated by a Coq Platform release. This would also be needed for testing compatibility using, say, a Docker image.
For example, we could say explicitly that a Platform release consists of several variants:

2021.09.0+8.13+recommended (for 8.13.2 with latest package collection)
2021.09.0+8.13+compatibility (for 8.13.2 with old package collection)
2021.09.0+8.14+experimental (for 8.14.0 with beta package collection)

I reworked the use of terms like "version", "variant", "release" and so on in main.

Coq version
Coq Platform release
Package pick

The latter one could also name variant, but I think package pick is clearer.
I will do another PR to add the suggested labels latest-stable and beta, but it gets tight in the one liner descriptions of the the package picks.

This solves the issue as far as I am concerned, but I repost here for reference the tentative identifiers of each opam switch that were discussed on Zulip (<platform-release>-<coq-version>-<package-pick>):
Current release:

2021.09.0-8.13-2021.09 / latest-stable
2021.09.0-8.13-2021.02
2021.09.0-8.14-2021.09 / latest-beta

Then once release 2021.11.0 is out we get:

2021.11.0-8.14-2021.11 / latest-stable
2021.11.0-8.13-2021.09
2021.11.0-8.13-2021.02

Please note that in main the opam switches are named as above (with a __coq_platform prefix), but the separator is not - but ~, so it is e.g. 2021.11.0~8.14~2021.11. Is far as I remember this has some advantages in compatibility with various version ordering mechanisms in use, but I can't say I remember the exact details.

I think ~ vs - only matters for actual version names, not for names of switches (which are not ordered).

Yes, ~ should not matter for switch names. Hence, to make Érik's life easier when generating Docker images, we may want to consider renaming the switches using only -, according to format __coq_platform_<platform-release>-<coq-version>-<package-pick>, for example:

__coq_platform-2021.09.0-8.13-2021.09
``__coq_platform-2021.09.0-8.13-2021.02`
``__coq_platform-2021.09.0-8.14-2021.09`

Originally it was like that and afair someone convinced me to use ~ instead of -, but I can change it back.


Ds/hide metrics

Hide existing metrics page in preparation for testnet launch.

Summary by CodeRabbit

New Features

Introduced a MetricsPage component for displaying various metrics related to connected chains, DApps, and packet statistics with date range functionality.
Added environment variables for local setup to improve application configuration.

Documentation

Updated the README with instructions for running the application locally and setting environment variables.

Refactor

Updated import statements and code formatting in several API files for consistency.
Simplified the MetricsPage component into a Home function component that redirects to /packets.

Style

Improved code readability and consistency across multiple files with formatting adjustments.

Chores

Adjusted navigation links in the navbar, enhancing the user interface by removing duplicate and irrelevant links.

Seems like a lot of changes for what can be done with one line change...


Control-service: KubernetesService class is very bloated and needs to be cleaned up

What is the feature request? What problem does it solve?
the KubernetesService class is an abstract class which is extended by two classes, the ControlKubernetesService and the DataJobsKubernetesService.
The DataJobsKubernetesService is responsible for deploying and reporting status updates data jobs.
All of the methods that are specific to the DataJobsKubernetesService actually exist in the KubernetesService.
This is very confusing, the fact that most of these methods live in the abstract class gives the impression that these would be used by both extending classes but this is not the case.
It also leads to extra confusion when creating an instance of ControlKubernetesService and the property supportsK8sV1CronJobs needs to be set even though it will have absolutely no impact.
Suggested solution
Move all code specific to the DataJobsControlService into the DataJobsControlService.
Leaving the class KubernetesService to be able to focus on generic abstract functionality.
In general I also think we should avoid inheritance where possible in favour of composition https://en.wikipedia.org/wiki/Composition_over_inheritance#:~:text=Composition over inheritance (or%20composite,than%20inheritance%20from%20a%20base.

I think the Control Servie as collection of services - at least that was the initial idea- the public API tends to reflect that (https://iaclqhm5xk.execute-api.us-west-1.amazonaws.com/data-jobs/swagger-ui/index.html) :

Data Jobs Service
Properties Service
Execution Service
Deployment Service
Sources Service

They live in single monolith for development and resource efficiency reasons. But they should be independent. So I'd have liked to have separate Kubernetes Service for each of those services .
There is also other issue discussing refactoring of KubernetesService - https://github.com/vmware/versatile-data-kit/issues/586

@tozka is that to say that you don't want to move forward with this PR https://github.com/vmware/versatile-data-kit/pull/1601 ?
I think the best path forward is iterative changes like in that PR.

@murphp15
I did not see #1601 when I wrote that.
I am a very firm believer in doing things iteratively and in small increments (and I've tried to explain why here). I was only sharing how I see the end state which might be in the far future.

Triaged. We will keep it and we are working on this iteratively.


Adds "Complete ProfileInstanceList" and colouring

Fixes #157, Fixes #156

Adds the new block and adds green colouring to required blocks.


Migrating Engine targets from Goma to Reclient

This tracks the status of Goma to Reclient migration.
Context: Goma will be deprecated EOY, and we need to switch over to Reclient.
Action items:

[x] Setup rbe projects/instances/workers/servers via terraform - Done by @yusuf-goog
[x] Create rbe recipes modules to support reclient context replacing goma context, https://flutter-review.googlesource.com/c/recipes/+/49541, https://flutter-review.googlesource.com/c/infra/+/50860
[x] Update gn related configs to point to the new wrapper instead of gomacc, https://github.com/flutter/engine/pull/45271, https://github.com/flutter/buildroot/pull/757, https://github.com/flutter/buildroot/pull/758
[ ] Migrate Engine targets

https://github.com/flutter/engine/pull/46272 to restore targets that affect framework, waiting for 3.16 branch.

Hi folks, could you please share an update on the current progress of migrating from Goma to RBE? How much is still left to do?

Flutter team is still working on the migration, but we have a couple of blockers from our side. https://github.com/flutter/flutter/issues/136570 is one example.
Also we need to collect some baseline data before moving forward. Still targeting in Q4, but we cannot guarantee.

Thank you, that's very helpful!

Hi, what is the current status of this issue? Did you stop using goma?

We're still using goma in the build that @keyonghan mentions, and migrating from goma to reclient for local developer workflows is blocked on a small amount of documentation and tooling work. I suspect that there are still a few weeks of calendar time remaining before we can start deleting goma support from the tree.
I would also really like to move our clang-tidy usage to reclient, but that is blocked on b/326591374 and any issues that it is potentially masking.

Hi, what is the current progress of goma deprecation in flutter? Can we assume you finished migration to reclient and shutted down your own goma instances?

@zanderso ping?


virtualization_virtual-machines_create

Hello,
i try to use the api of netbox to create a new virtual machine with this request
{
"name":"Server_1",
"site":{"id":1,"slug":"site1","name":"site1"},
"cluster":{"id":1,"name":"demo"},
"role":{"slug":"Applicative Server","id":1,"name":"Applicative Server"},
"vcpus":4,
"memory":4160,
"disk":15,
"tenant":{"slug":"example","id":1,"name":"Example"}
}
the reply :
{
"cluster": [
"Primary key must be an integer"
],
"role": [
"Primary key must be an integer"
],
"tenant": [
"Primary key must be an integer"
]
}
did you have a good exemple for me ?

ok i try with the id and it's work :
{
"name":"server_1",
"site":{"id":1,"slug":"site1","name":"site1"},
"cluster":1,
"vcpus":4,
"role":1,
"memory":4160,
"disk":15,
"tenant":1
}

Thank you for your interest in NetBox. GitHub issues are intended for reporting reproducible bugs and requesting features, and must be submitted using the template provided here. For general discussion, questions, or assistance with installation issues, please post to our mailing list.


Fix issue #2346

Chris/Cody: I haven't run these through the build farm yet (will try that tomorrow), but please take a look anyway to ensure you're okay with the general direction.
One potential improvement that occurs to me as I type this just now: the validation cache code ultimately wants a UUID for its cache ID. external_revision_generator.py currently takes either a SHA1 Git commit ID or the contents of an arbitrary revision file (converted to a SHA1 hash if necessary), and writes the hash to the header file (which is converted back to a UUID at runtime by the layers). Perhaps the script should just convert its input to a UUID immediately, and write that to the header file (so validation cache runtime code can ingest it directly). What do you think -- useful simplification, or overthinking?

Thanks, Cort.  I like the change, but it will be Monday before I get a chance to test it.

Okay, I added a commit that allows the android-generate.* scripts to continue working as is, allowing downstream toolchains to work without modification.  I'll merge this to master soon.

cherry-picked to master


environment naming

Change from --dev and --prod by --local and --containers.

Haha I think this is the 2nd duplicate of #55 , the first one being #57 . Is it possible that one of those was meant to be renaming the commands? For example: invenio-cli services ... or something like we discussed in Chicago?


🐛 BUG: statement loo long when importing to D1

Which Cloudflare product(s) does this pertain to?
D1
What version(s) of the tool(s) are you using?
wrangler 3.17.1
What version of Node are you using?
19.9.0
What operating system are you using?
Linux (Ubuntu)
Describe the Bug
I'm trying to migrate a D1 DB using the legacy backend to the new backend but I get an error from the Cloudflare API: statement too long. The DB has a single table with only ~150 rows but some columns contain a lot of text, so I imagine that one or more rows must hit the API's statement length limit.
Is this expected? Is there another way to import larger rows into D1?
Please provide a link to a minimal reproduction
Not applicable
Please provide any relevant error logs
npx wrangler d1 execute <db> --file <dump>.sql
🌀 Mapping SQL input into an array of statements
🌀 Parsing 148 statements
🌀 Executing on remote database <db> (uuid):
🌀 To execute on your local development database, pass the --local flag to 'wrangler d1 execute'

✘ [ERROR] A request to the Cloudflare API (/accounts/<id>/d1/database/<uuid>/query) failed.

  statement too long [code: 7500]

Looks the same as #2367 (closed without resolution, "new backend fixes the issue")

Found my own workaround.
Since adding this data worked via Workers in the first place (there doesn't seem to be any statement length limit there), I built one for batch importing the data (8 MB). Here it is, in case anyone runs into the same issue:

View source
// Note: I'm using Cloudflare Pages so this is a Pages Function, deployed as a Worker. Might need to be adapted to your use case

export async function onRequest(context) {
  // 1. get json data. I got this from my sqlite3 dump: https://stackoverflow.com/a/67186486
  const response = await fetch("<your-domain>/dump.json");
  const json = await response.json();

  // 2. prep statement
  const stmt = context.env.<DB>.prepare(
    "INSERT INTO <table> (id, created_at, title, <etc>) VALUES (?1, ?2, ?3, <etc>)"
  );

  // 3. prep batch statement
  const stmts = json.map((a) =>
    stmt.bind(
      a.id,
      a.created_at,
      a.title,
      // <etc>
    )
  );

  // 4. exec batch statement
  const result = await context.env.<DB>.batch(stmts);

  // 5. log all returned data with a dummy status code
  return new Response(JSON.stringify(result), { status: 201 });
}

All rows were successfully written, so my issue is solved. I'll keep it open for a bit anyways, in case the D1 team wants to look at why the regular import mechanism fails with long statements.

Thank you! It's helped

Found my own workaround.
Since adding this data worked via Workers in the first place (there doesn't seem to be any statement length limit there), I built one for batch importing the data (8 MB). Here it is, in case anyone runs into the same issue:
View source
All rows were successfully written, so my issue is solved. I'll keep it open for a bit anyways, in case the D1 team wants to look at why the regular import mechanism fails with long statements.


fixed - duplicates of metas in notification call

Description

If we have a decision with multiple items we should have only one insert in the personalization.decisions array
{
  "decisions": [
    {
      "id": "TNT:113866:1"
    },
    {
      "id": "TNT:115166:0"
    },
    {
      "id": "TNT:113712:1"
    }
  ]
}

Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[x] I have made any necessary test changes and all tests pass.
[x] I have run the Sandbox successfully.

@Aaronius could you please take a look again, I have made some other changes. We need to skip some results that contain errors from metas. We will discuss this later if we should send this data to konductor.


Variable explorer (RuntimeError:)

From<EMAIL_ADDRESS>on 2014-09-20T01:54:15Z

Create file
t = [[[1, 1], [1, 2], [1, 3], [1, 3]], [0, 2], [0, 3], [0, 4], [0, 6], [0, 7], [0, 8]]
t = 1
Run Debug file
Run current line (first)
Switch Variable explorer and edit 't' (image)
Press the 'OK' button in the sequence (red numbers)
Mistake the second press.

What is the expected output? 1. Traceback (most recent call last):
File "C:\Python27\lib\site-packages\spyderlib\widgets\dicteditor.py", line 459, in <lambda>
lambda eid=id(editor): self.editor_accepted(eid))
File "C:\Python27\lib\site-packages\spyderlib\widgets\dicteditor.py", line 467, in editor_accepted
index = data['model'].get_index_from_key(data['key'])
File "C:\Python27\lib\site-packages\spyderlib\widgets\dicteditor.py", line 219, in get_index_from_key
return self.createIndex(self.keys.index(key), 0)
RuntimeError: wrapped C/C++ object of type DictModel has been deleted 2. If only First run:
Empty Variable explorer
Second run: ok.

If only First run:
Is not possible to continue debugging(press button 'Run current line').
To continue need switch to Ipython console.
Second run: continue debugging ok. What version of the product are you using? On what operating system? Windows7, Python 2.7.8 (default, Jun 30 2014, 16:08:48) [MSC v.1500 64 bit (AMD64)]

Spyder 2.4.0dev
hg heads
changeset:   4091:acf756421eab
changeset:   4089:cf2ed4c20e5d
Attachment: 123.jpg
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1977

From ccordoba12 on 2015-02-08T11:27:02Z
Summary: Variable explorer (RuntimeError:)  <span class="oldvalue"> (was: Variable explorer(RuntimeError:)) </span>
Labels: Cat-VariableExplorer MS-v2.4


Problem with blob URI and Service Worker

This issue is a Question
Related issue(s) and/or pull request(s)
See also #433*
Hi! I'm using exploded .epub files, loaded from the backend. Works like a charm. But when I put all these files in cache with Service Worker, the IMG and CSS files fail to load because they are being loaded with blob.
My question is: There is some way to use the absolute path and retrieve data from cache storage directly?
I'm almost finishing the cache feature: list all files, create a cache storage entry named by epub ID on backend, use put(request, response) method to simulate a request from service worker origin. All HTML is loaded normally, but IMG and CSS won't.
Anyone can help?
Thanks in advance!

To recap: when the Readium cloud/web reader opens exploded / unzipped EPUBs, only the HTML documents are loaded as "blobs" into the host iframes (with their respective origin set via the "base URL"), all other resources (CSS, images, etc.) are loaded from their respective HTTP URLs, as normal. However, if the EPUB contains obfuscated fonts, then this triggers a special code path which requires all publication resources to be loaded via "blobs" (just like when opening packed / zipped EPUBs).
So, I fail to understand how a Service Worker caching strategy would change this pattern, as the SW "proxy" should transparently handle fetch requests, even if the HTTP response payload is ; behind the scenes ; constructed from a resource stored in a local cache. I am aware of issues related to audio/video "streaming" (HTTP partial requests / byte ranges), but this doesn't seem to be the case here, right?
Thank you for clarifying :)


asyncBlurFields doesnt do anything for complex fields probably

Hey, i'm started using this repo and have to say its really awesome to use so far, thank you!
Maybe I'm missing something but I have a check for payment information in a form.
I'm using complex fields like:
'Payment.Iban',
'Payment.Bic',
...
The payment information should be validated async by the server, so i tried to use Asynchronous Blur Validation from the examples. It works fine when i use just a simple field like 'Iban' without nested properties. So just 'Iban' would work.
How to use nested fields like 'Payment.Iban'? It will not work for me.
Here is what i'm putting in asyncBlurFields
asyncBlurFields: ['Payment.Iban']
I did'nt get any
redux-form/START_ASYNC_VALIDATION or
redux-form/STOP_ASYNC_VALIDATION
events.
Any idea what i'm doing wrong?

May it helps, when i'm clicking submit the asyncValidate is called but not on blur.

In v6, it's working: https://jsfiddle.net/gustavohenke/7jqx8whb/

@gustavohenke What's the status of maintenance/bugfixes on the v5 branch? v6 is a massive migration and I really don't like how this project has pushed a massive rewrite while v5 still had bugs.
Especially since v6 forces you to upgrade to redux-hot-loader beta... which forces you to upgrade to react-router v4.
It seems like getting this small fix will require rewriting my entire project. Why is the only production release of this one which forces you to use beta versions of other tools? This is v5, not v0.5...

@gustavohenke we also have a old project on v5 we cant migrate to v6.
What we do at the moment is a lot of workarounds to handle the issues.
The workaround for that issue is to do a ajax request and dispatch stopSubmit if there are validation errors.

https://github.com/erikras/redux-form/pull/2645#issuecomment-284908300
And @morgante, I was not the one who fixed this issue, so I don't know what commit did it.


View compilation

@tdykstra  please review

Fixes #2307

@danroth27 Should I add a link to view compilation in "additional resources" in the publishing docs?

Or in performance?

Or both?

@tdykstra  added project.json


🆕 Software Suggestion | I have written extremely private disposable email bot.

Basic Information
Name: TrashEmail
Category: opensource disposable email bot.
URL: https://github.com/TrashEmail/TrashEmail
Description
Open source and fully private disposable email telegram bot.
Why I am making the suggestion
I want to get this reviewed so that I can proudly put this on your site and grow my community as well.

My connection with the software

I have developed and hosted this tool for community.

[X ] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

I don't think that this is a good fit for the community. The site doesn't encourage the use of telegram, so adding a bot for it seems a little odd...

I agree to that but the telegram bot solution that I have hosted for community, is extremely privacy focused unlike the other disposable mail services.

You've missed the point. I understand that you designed this bot for privacy however the very fact that it relies on Telegram makes it less private. Plus, private disposable emails already exist without having to rely on Telegram.

Out of scope.


acquire shouldn't require a specific machine

Looking at the docs and talking to kiko, MAAS is designed to just acquire.
POST /api/1.0/nodes/ op=acquire

Acquire an available node for deployment.

Constraints parameters can be used to acquire a node that possesses certain characteristics. All the constraints are optional and when multiple constraints are provided, they are combined using ‘AND’ semantics.

I'll slate this for 1.1.0.

https://github.com/jjasghar/knife-maas/pull/13


remove call to get_guc_variables in pltsql_revert_guc

Description
DRAFT
Issues Resolved
[BABEL-4881]
Sign Off
Signed-off-by: Tanzeel Khan<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 72.509%

Files with Coverage Reduction
New Missed Lines
%

contrib/babelfishpg_tds/src/backend/tds/tdscomm.c
2
73.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.006%

Covered Lines:
41112

Relevant Lines:
56699

💛  - Coveralls


Now that I think about it, I guess I could have added this to the "Typos" PR. Whoops.


Import from Figma?

(Don't want to ask for features out of the blue but was wondering if someone is working on it already)
Is someone working on some Figma import to Presentator? I saw someone did an Adobe XD plugin ( https://github.com/spazola/presentator-plugin ) and was wondering about Figma too

I don't think anyone has started working on this yet, but I recommend to wait until v2 is released (stable version should be released in the next 1-2 months) since the API will have some changes.

@ganigeorgiev is this a possibility now?

@elioqoshi The new API is publicly available (https://presentator.io/docs/introduction), but I haven't started working on the Figma plugin yet.

Similar to the Adobe XD issue, I couldn't find an easy and reliable way to import a Figma file directly in Presentator and I guess a plugin will be required in this case.
I'll try to implement a Figma plugin like the Adobe XD one for the next release (mid January).
In the meantime, if anyone think that the export flow from the XD plugin could be improved, please comment so I could take it in consideration during development.

Pre-release version of the plugin (waiting review from the Figma team) could be found here - https://github.com/presentator/presentator-figma.
If someone wants to test it, please follow the manual installation instructions.
Note: The plugin should work with both the free service (aka. https://app.presentator.io) and a self hosted setup, but because Figma plugins run inside a browser environment, CORS policies apply.


Gemming IF Hammer will reset it's level

Playing FTB Genesis with Apotheosis 6.4.1.
I was able to add some affixes to a level 4 Infinity Hammer by upgrading with some godforged pearls and it gave me 2 gem slots. When I tried to slot a light weapon gem, the hammer reset its level to 0 and lost all the affixes.  I tried the same process 4 times with the same results

https://github.com/Shadows-of-Fire/Apotheosis/issues/1090


Jfrog npm install bug with self-signed certificate

Describe the bug
I tried to do a jfrog npm install with self-signed certificate but I have an SSL error x509: certificate signed by unknown authority from a gitlab pipeline.
Current behavior
I initialize the NPM_CONFIG_CAFILE and copy the certificates under the .jfrog/security/certs/.
Then when I execute the command ./jfrog npm install.
I so have the behavior: [Warn] (Attempt 1) - Failure occurred while sending GET request to https://url/: Get "https://url/": x509: certificate signed by unknown authority
Reproduction steps
No response
Expected behavior
I would expect the initialization of npm variable and the copy of certificate in jfrog folder enough to be working.
Current bypass
I specify the certificates in the variable Golang: SSL_CERT_FILE for jfrog CLI.
It currently works but I would like to know if there is another more standard way ?
JFrog CLI version
2.14.0
Operating system type and version
linux/arm/v7
JFrog Artifactory version
No response
JFrog Xray version
No response

Hi,
No one has encountered the problem?

Good day everyone,
No news on this problem?

I did encounter this, too. At the moment we're stuck on version 2.52.9 in order to avoid the panic.

Hi everyone,
Still no news on this bug?


update

Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

just wanted to update my fork :(


Corrupt csv file wriiten on Windows 10

Hi
Saving as csv on Windows 10, the line separators are written as cr, cr, lf.  Ingesting into Open Office calc gives blank lines between the data.
Cheers
Colin

Thanks, for the report! Should be fixed by someone sometime ;-)


Feature Request: Support for EnumMember(Value = )

Request
Would it be possible to add support for the EnumMember attribute?
[EnumExtensions]
enum UserTypeTest
{
    [Display(Name = "مرد")]
    [EnumMember(Value = "men_value")]
    Men,

    [Display(Name = "زن")]
    [EnumMember(Value = "women_value")]
    Women,

    //[Display(Name = "نامشخص")]
    None
}

internal class Program
{
    static void Main(string[] args)
    {
        Console.WriteLine("Hello, World!");

        var canParse = UserTypeTestExtensions.TryParse("men_value", out var parsed);

        if (canParse)
        {
            Console.WriteLine($"Parsed men_value as {parsed}");
        }
        else
        {
            Console.WriteLine("Unable to parse using EnumMember attribute");
        }

        // ideally something like this that would return string values: men_value, women_value
        var allValues = UserTypeTestExtensions.GetValues();

        // ideally something like this for fetching the actual Value that is set: men_value or women_value based on actual enum
        // var value = UserTypeTest.Men.ToValueFast();
    }
}

Use Case
For our use case, we have DTO classes that are mapped from JSON and validated
{
   "npc_gender": "men_value",
}

class RequestDto
{
   [JsonPropertyName("npc_gender")]
   public string? NpcGender {get; init;}
}

Validation happens, then it's mapped if valid:
class Request
{
   public required UserTypeTest UserType {get; init;}
}

We use currently have manual mapping to internal types, but ideally custom enum handling in things like mapperly or other cases of using an auto mapper someone could make use of these methods going from RequestDto to Request
VERY Bad implementation
I won't pretend to even have suggestions on how this would be implemented properly or efficiently, but this is a rough version of what I use now:
internal class Program
{

    static void Main(string[] args)
    {
        var emValue = UserTypeTest.Women.ToValueStringFast();
        var canParse = EnumExtensions.TryParse<UserTypeTest>(emValue, out var back);

        Console.WriteLine($"CAN PARSE: {canParse} PARSED: {emValue}; BACK: {back}");

        var canParse2 = EnumExtensions.TryParse<UserTypeTest>("None", out var back2);

        Console.WriteLine($"CAN PARSE: {canParse2} PARSED: BACK2: {back2}");

    }
}

// bad implementations
static class EnumExtensions
{
    public static string ToValueStringFast(this Enum enumVal)
    {
        var type = enumVal.GetType();
        var memInfo = type.GetMember(enumVal.ToString());
        var attributes = memInfo[0].GetCustomAttributes(typeof(EnumMemberAttribute), inherit: false);

        if (attributes.Length == 0)
        {
            throw new Exception($"EnumMember attribute not found for {enumVal?.ToString()} of {enumVal?.GetType()}");
        }

        if (((EnumMemberAttribute)attributes[0]).Value == null)
        {
            throw new Exception($"EnumMember value for {enumVal?.ToString()} of {enumVal?.GetType()} was null. Cannot get as string.");
        }

        return ((EnumMemberAttribute)attributes[0]).Value!;
    }

    public static bool TryParse<T>(string? value, out T? val)
        where T : Enum
    {
        var type = typeof(T);

        foreach (var field in type.GetFields())
        {
            EnumMemberAttribute? attribute = Attribute.GetCustomAttribute(field, typeof(EnumMemberAttribute)) as EnumMemberAttribute;
            if (attribute != null)
            {
                if (attribute.Value == value)
                {
                    var emValue = field.GetValue(null);
                    if (emValue == null)
                    {
                        throw new Exception($"Enum member for {value} as {typeof(T)} was null");
                    }

                    val = (T)emValue;
                    return true;
                }
            }
        }

        val = default;
        return false;
    }
}

So this behaviour is sort of already supported, but it uses the [Display] or [Description] attributes instead. Unfortunately, I think that's actually going to be a problem for your example, where you don't want [Display] to be the serialized value (presumably)🤔 We could add support for [EnumMember] in the same way, but I don't think that will help you in the end given you want both attributes. I'm not sure how to support both without making breaking changes unfortunately...

You're right, I would like to avoid using Display so that for other scenarios with html where both may be in use:
<option value="EnumMemberValue">DisplayValue</option>
I can rip out the relevant pieces from here around Display and just add local versions of more specific methods for my use case. I just figured it wouldn't hurt to ask first.
Thanks for the quick response.


Parsec

henlo i am the mr parsec and i am here to sue you /s
In all actuality, good luck m8 😄

Thank you so much! Borgo told me about what you told him, now I'll be fully into this project hoping that one day it will be good enough to be noticed.

hoping that one day it will be good enough to be noticed

You mean like uh... Now? I work for Parsec 😅

Yes I know, I mean noticed by more people so that someone actually starts using it and spread the word


Rename project from trollop to optimist

will need to:

[ ] rename github project
[ ] change awesome ruby
[ ] put in a redirect in rubygems

I may have missed some
Fixes #75

/cc @jrafanie @Fryguy

Do we want to deprecate the old class?
Something like this:  https://github.com/thoughtbot/factory_bot/blob/master/lib/factory_girl.rb
In this way, the constant can stay around for a release or so?

I support this. Having a gem referencing the old-timey word for slut is really not great.

@kbrock The renaming mostly. I think it's fine to maintain backward compat to avoid putting people in messy situations.

@jrafanie @Fryguy ok - added warning - you all ready to do this?
I'll need one of you to go into settings and rename this github project from trollop over to optimist.

@jrafanie @Fryguy ok - added warning - you all ready to do this?

@kbrock what warning?

I'll need one of you to go into settings and rename this github project from trollop over to optimist.

@Fryguy will have to do this, I don't have permission to do it.


translations PR 20180802 (Adds Romanian)

Ideally this PR should be review by community members for each language.
Note: In the event that you see an english translation instead of the desired language is because this PR was submitted before the translation was made on the Mattermost Translation Server and probably will be submitted for the next PR.

@enahum can you fix the issue? components/admin_console/SchemaAdminSettings › should match snapshot with settings and plugin

@cpanato should be ok now


contains, isDefined, isEmpty, ... for Option

Hi,
I'm integrating fp-ts into a project, I noticed some member functions I would expect to be present in the Some / None classes were missing, like contains, isDefined, isEmpty (and possibly others).
I realize it's possible to get this functionality using other functions, like e.g. isNone, isSome, etc.
Is this a design choice (ie using global functions in which you pass Option instances instead of member functions on Some / None)?
I'm quite willing to create a pull request for these methods if you want me to.
wkr,

Is this a design choice

Yes, namely the members of a type-class instance, for example { map, of, ap, chain } for Monad. In addition I've just added the methods I use the most (in particular fold and getOrElse), feel free to add some useful member function though.
Another option, in case you judge a method just a nice-to-have, is augmenting the base module in your own code
import * as option from 'fp-ts/lib/Option'

declare module 'fp-ts/lib/Option' {
  interface None<A> {
    someMethod(a: A): boolean
  }
  interface Some<A> {
    someMethod(a: A): boolean
  }
}

option.None.prototype.someMethod = function<A>(a: A) {
  ...
}

option.Some.prototype.someMethod = function<A>(a: A) {
  ...
}

Thanks,
understood.  I just created a pull request for this #184, added methods and unit tests for :

contains
isEmpty
isDefined
nonEmpty
forEach
exists
orEmpty


emqttd 0.10.0 Feature Reuqest

Discuss features of 0.10.0 release in this issue.

Would anyone agree with "shared subscription" support?
with this feature, we can bring up EMQTTD with some workload distribution
which was formerly impossible with most MQTT implementations.
Previously mentioned by this post:
http://knolleary.net/2012/04/11/queuing-with-mqtt/
Some of MQTT implementations which already supports "shared subscription":
IBM MessageSight
https://www.ibm.com/developerworks/community/blogs/c565c720-fe84-4f63-873f-607d87787327/entry/easily_balance_messaging_workload_with_new_ibm_messagesight_v1_1
JoramMQ
http://www.scalagent.com/en/joram-33/technology-36/mqtt-protocol

Enhance HTTP admin feature (eg: I'm a nodejs user, not familar with erlang, enhance HTTP feature can make emqttd easier extending with other language)。
1, HTTP publish message by 'username', not 'clientId'. Because one user can have multiple clientId, this makes application server sending message easier.
2, HTTP query user online status and active status(offline but not expired)
3, HTTP interface support keep-alive, application server can keep long HTTP connection with emqttd, improve performance.

add more wiki about plugins.
1, how to write a plugins, how to hook emqtt event
2, how to integrate plugin with emqttd(the pre-bulid one from your website)

add auto subscribe and auto  publish fucntion.
for example.
when connented

auto subscribe /client/$c     here $c is clientid
auto publish some message(for example sessionid )  to  /client/$c
3、client will get sessionid for bussiness.

hi, lsxredrain
I'm using hooks to provide such funcionality, and it looks pretty
straightforward:
emqttd_broker:hook('client.connected',
{?MODULE, client_connected},
{?MODULE, client_connected, []}).
client_connected(?CONNACK_ACCEPT, #mqtt_client{client_id = ClientId,
client_pid = ClientPid}) ->
subscribe_to_topics(ClientPid,
my_find_topics_by_client_id_function(ClientId));
glad to know if there is a more convenient way to achieve auto-subscribe
functionality.
--
Santiago
On Wed, Jul 15, 2015 at 3:45 AM, lsxredrain<EMAIL_ADDRESS>wrote:

add auto subscribe and auto publish fucntion.
for example.
when connented

auto subscribe /client/$c here $c is clientid
auto publish some message(for example sessionid ) to /client/$c
3、client will get sessionid for bussiness.

—
Reply to this email directly or view it on GitHub
https://github.com/emqtt/emqttd/issues/191#issuecomment-121508884.

@toh2526, I agree that no queue in MQTT is painful for workload distribution. "shared subscription" or "queue" will be implemented in 0.10

Wondering if this has been implemented. Adding workload distribution can enable RPC like features for devices. If already implemented, any info on how to switch it on could greatly help.

Hi @KrishnaPG, The 1.x release supports "$queue/topic" style workload distribution. The enterprise edition of the broker will support local, shared subscriptions in R2 release.

Thank you @emqplus
You have mentioned 'enterprise edition' - what is it? I was under the impression that this is Apache License. Is this project going to divide into commercial line with divided functionality?


Not able to down load Electrum-BTCP-P_1.0.0-win.7z Wallet -  (Failed Forbidden)

Please help me with he Wallet Download as I have tried multiple times but the download fails (Failed Forbidden) always. FYI, I have the Electrum ZCL wallet installed on the same machine already. Thanks for looking into this.

Same problem. ZClassic works fine, this one doesn't... BTCP has a lot of problems.. :(


Sporadic driver error during K8s upgrade: listen tcp <IP_ADDRESS>:0: bind: address already in use

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Very similar issue to https://github.com/rancher/rancher/issues/19742
On two occasions now I've encountered separate customer issues where during a K8s upgrade, through the Rancher UI, the rolling upgrade will stall with this error:
Error starting driver: failed retrieving port for driver: listen tcp <IP_ADDRESS>:0: bind: address already in
Getting stuck here: https://github.com/rancher/kontainer-engine/blob/65780a8839e471d3842580d5d9075ae9a3b70a02/service/service.go#L400
This issue causes the cluster controller inside Rancher to try to bind to a port that is already in use.
Some customers report that eventually it self heals and upgrade completes. The quick fix for this is to cycle the Rancher pods.
It's sporadic in that it doesn't appear to happen with each downstream cluster. The commonality I've seen with customers is this:

Custom cluster
Ubuntu 18.04
K8s version < 1.118.x

Result:
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.4.5
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): cloud and VMware
Kubernetes version (use kubectl version):

v1.17.6 --> v1.18.6

Docker version (use docker version):

(paste the output here)

gz#11024
gz#12126

Workaround:
Kubernetes 1.15 and above
kubectl -n cattle-system rollout restart deployment rancher

Kubernetes 1.14 and below
kubectl -n cattle-system set env deployment --env="LAST_MANUAL_RESTART=$(date +%s)" rancher

Verification:
kubectl -n cattle-system get pods -l app=rancher    <<== Wait for all pods to be recycled

Rancher v2.4.5 HA local cluster with RKE k8s v1.18.3
Downstream cluster:

Custom with 3 EC2 instances (t3.medium) all roles (etcd/worker/control plane)
Ubuntu 18.04
k8s v1.17.13-rancher1-2

Upgrade from v1.17.13-rancher1-2 to 1.18.10-rancher1-2 I did a couple of kubernetes upgrades with this same version combination without noticing anything in the rancher server logs.
I will try with v1.16.x and v1.15.x to v1.18.10 which are available in this Rancher version to see if I can reproduce.

Rancher v2.4.5 HA local cluster with RKE k8s v1.18.3 (11/10/2020)
Downstream clusters:

Custom with 3 EC2 instances (t3.medium) all roles (etcd/worker/control plane)

Ubuntu 18.04

k8s v1.15.12-rancher2-3

Custom with 3 EC2 instances (t3.medium) all roles (etcd/worker/control plane)

Ubuntu 18.04

k8s v1.16.15-rancher1-3

Upgrading these clusters to 1.18.10-rancher1-2 I didn't notice anything in the rancher server logs. Upgrade was successful.

Note: this won't reproduce on a as is system.
You need to somehow fill up all the ports to hit this error.

@deniseschannon I will be trying a few times. I tried with a 20 nodes cluster upgrading to latest 1.18 as described here.
3 etcd, 3 control plane, 14 workers.

Rancher version v2.4.5 (11/11/2020)
Single node Rancher server.
Downstream Custom cluster k8s v1.17.13-rancher1-2. 2 control plane, 3 etcd, 50 workers.
Upgrade to k8s v1.18.10-rancher1-2 I didn't notice the error in the Rancher server logs.
I will try now on an HA v2.4.5 Rancher server.

I tried in a 1 Control Plane, 3 etcd, 5 workers but now running a script to use all the control plane node’s ephemeral network ports and doing an upgrade at the same time. I still see the upgrade getting applied without issues.
We might need to dig further to isolate the root cause.

@izaac It looks like this is caused by golang bug 16691
Also, we found that rancher-machine uses the same bind to port 0 which causes this issue.
Example: We found a Rancher pod with 57k open ports due to node pools scaling up and down.

I was able to reproduce the issue in Rancher v2.4.5 (t3a.xlarge)
The easiest way to reproduce this is to bring up a Rancher single install.
Bring up a 1.17 or lower k8s version cluster or clusters.
Go into the Rancher docker container and exhaust the ephemeral ports by opening LISTEN connections.
Perform a k8s upgrade on the cluster (or on one of the clusters) to 1.18.

In this case the upgrade self recovered.

I was able to reproduce the issue in Rancher v2.4.5 (t3a.xlarge)
The easiest way to reproduce this is to bring up a Rancher single install.
Bring up a 1.17 or lower k8s version cluster or clusters.
Go into the Rancher docker container and exhaust the ephemeral ports by opening LISTEN connections.
Perform a k8s upgrade on the cluster (or on one of the clusters) to 1.18.

In this case the upgrade self recovered.

Based on this and some stress testing I did, I can conclude this problem is not how we use Go to assign a port, instead it's something taking all the connections the system can handle until ephemeral port starvation. When we bind to port 0, Go's net.Listen function used here will find an available port for us, but in the case no ports are available, it will throw the listen tcp <IP_ADDRESS>:0: bind: address already in use error which to me is the correct behavior since it could take a really long time before a port is available.
Taking all that into consideration, we can shift our focus onto how the connections are being handled.
@mattmattox We need data from the setups and netstat output (hopefully showing a high amount of connections) to investigate. Any chances you could get that for us?
@izaac Once we get some data from Matt, can we reproduce the scenario to confirm there is a problem with rancher-machine/upgrading, that results in a high amount of connections and no recovery?

We diagnosed an environment that was encountering errors like this to having the telemetry client running and holding "ESTABLISHED" connections against the Rancher API server.
netstat -an showed 50,000+ connections open between localhost:<RandomPort> and localhost:443 and a -plant showed the telemetry client as the one holding the connections.
I think there may be some type of issue w/ the underlying api serving objects to the telemetry client that is causing the hang... Turning off the telemetry client (turning telemtry opt to out from in) fixed the hanging established connections.

Confirmed @Oats87 diagnose.
Telemetry client is not releasing nor reusing connections to Rancher API on every report() execution, every 6h by default.  The number of connections is growing up until they are exhausted, generating listen tcp <IP_ADDRESS>:0: bind: address already in use errors..
As workaround, telemetry client may be killed at Rancher docker/pod to release connections. Rancher should restart telemetry client automatically.
Submitted PR https://github.com/rancher/telemetry/pull/47 to address the issue
To test it:

access to rancher docker/pod
force telemetry report() execution: curl -X POST http://localhost:8114/v1-telemetry/report
check the number of telemetry connections: netstat -natup |grep telemetry | wc -l
force telemetry report() execution: curl -X POST http://localhost:8114/v1-telemetry/report
check the number of telemetry connections: netstat -natup |grep telemetry | wc -l .

Telemetry connections should grow up on current version, but shouldn't grow up on patched version

Rancher version v2.5.5
Docker install
4 single node clusters
Make sure metrics is checked during initial Rancher configuration. (Allow collection of anonymous statistics)
Install net-tools in the docker container.
Reproduced using the steps provided here https://github.com/rancher/rancher/issues/28360#issuecomment-771134043
Rancher version v2.5-head
Rancher version master-head
With same setup above I still see the connections count going up.
I see the telemetry fix merged but do we need to update something in the Rancher repo too?

@izaac , are you using the patched version of telemetry client??

@rawmind0 Have you updated Rancher to use the patched telemetry client?

Updated telemetry version on Rancher 2.5 and master

PR's merged at 2.5 and master

Rancher version master-head (02/03/2021) 54b3a755b
I was able to check this with the steps provided https://github.com/rancher/rancher/issues/28360#issuecomment-771134043
I see the number of connections is kept steadily low even after multiple request to the report endpoint.
I also validated the /v1-telemetry endpoint was properly updated after adding clusters.
Still waiting for a build of 2.5-head to validate that branch.

Rancher version v2.5-head-head (02/03/2021) c6cdf815a
I was able to check this with the steps provided https://github.com/rancher/rancher/issues/28360#issuecomment-771134043
I see the number of connections is kept steadily low even after multiple request to the report endpoint.
I also validated the /v1-telemetry endpoint was properly updated after adding clusters.

TELEMETRY_VERSION=v0.5.14

for others: found this happening in 2.4.5, rancher-machine versions rancher-machine version v0.15.0-rancher43, build f675ca21.  TELEMETRY_VERSION=v0.5.12
My symptoms were that my rancher-launched clusters were no longer able to provision new nodes on AWS, but it was still able to terminate nodes on AWS.
Enabled debugging via instructions here: https://rancher.com/docs/rancher/v2.x/en/troubleshooting/logging/
Connected to each of the nodes remotely and logged errors from the instance via docker logs --since 20m [cid] | grep "create cmd" to find the offending machine. This helped to find the actual rancher-machine create command being run. Copying the command used and adding a debug flag (rancher-machine --debug create -d amazonec2 ... ) revealed the errors.
experienced errors mentioned
() DBG | Error loading RPC server: listen tcp <IP_ADDRESS>:0: bind: address already in use
and
dial tcp: missing address
When I attempted to run the command on another machine, there was another Error: No machine name specified. Manually adding a name worked, unclear why it wasn't included in the original command via the debug output.
There was some memory pressure on the nodes, but don't think it was enough to corrupt anything unless something happened in the past few days.  Ultimately, restarting the offending node seemed to help. I did need to delete the nodes that were failing to provision once the node had completed restart.
I hope that helps someone else!


Get interpreter for workspace and all

Found while testing that we don't get all of the interpreters.
I can see the environment showing up in Pythons Select Interpreter, but not in our kernel picker.
The difference was with the way get the list of interpreters, we provide a Uri.
I added an additional call to get interpreters without a Uri & that magically returned all interpreters.

Codecov Report

Merging #9440 (e3fa7e9) into main (7b5cd3c) will decrease coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff          @@
##            main   #9440   +/-   ##
=====================================
- Coverage     74%     74%   -1%
=====================================
  Files        219     219
  Lines      10939   10940    +1
  Branches    1571    1572    +1
=====================================
- Hits        8121    8115    -6
- Misses      2190    2192    +2
- Partials     628     633    +5

Impacted Files
Coverage Δ

src/client/api/pythonApi.ts
72% <100%> (+<1%)
:arrow_up:

src/client/common/cancellation.ts
72% <0%> (-6%)
:arrow_down:

...c/client/datascience/variablesView/variableView.ts
87% <0%> (-3%)
:arrow_down:

...lient/datascience/variablesView/notebookWatcher.ts
93% <0%> (-2%)
:arrow_down:

src/client/datascience/telemetry/telemetry.ts
85% <0%> (-1%)
:arrow_down:


Avoid overwrite fields in merged CR

Fix issue https://github.ibm.com/IBMPrivateCloud/roadmap/issues/61393
When we are merging all CS CRs in the tenant scope, we extract the profile from each individual CS CR into a list.
For example

common-service CR with starter size
cp4i-cs CR with large size
cp4ba-cs CR with medium size

Then we merge different profiles together to get the LARGEST value(it respect to large size), and save it into a summarized profile summaryCR.
But I always initialized this summarized profile variable summaryCR as long as either one of resources OR spec field is empty.
if summaryCR == nil || summaryCR.(map[string]interface{})["spec"] == nil || summaryCR.(map[string]interface{} ["resources"] == nil {
	summaryCR = map[string]interface{}{
		"name":      operator.(map[string]interface{})["name"].(string),
		"spec":      map[string]interface{}{},
		"resources": []interface{}{},
	}
}

Many of our profiles only contain one field. So summaryCR for each operator is always be initialized
### Only contain spec
- name: ibm-im-operator
    spec:
      authentication:
        authService:
          resources:
            limits:
              cpu: 2000m

### Only contain resources
- name: keycloak-operator
  resources:
    - apiVersion: k8s.keycloak.org/v2alpha1
      data:
        spec:
          db:
            host: keycloak-edb-cluster-rw
      force: true
      kind: Keycloak
      name: cs-keycloak

To fix this, I change it to initialize the field individually, to avoid overwrite.
How to test
Re-produce the issue

Install Bedrock v4.3 in the most simple topology
Check the default CS CR named common-service with starter profile
Create second CS CR with large profile
:white_check_mark: Verify that large profile is applied to OperandConfig

> oc -n test-size get operandconfig common-service -o jsonpath='{.spec.services[?(@.name=="ibm-im-mongodb-operator")].spec.mongoDB.resources.limits}{"\n"}{.spec.services[?(@.name=="ibm-im-operator")].spec.authentication.replicas}{"\n"}{.spec.services[?(@.name=="edb-keycloak")].resources[?(@.name=="keycloak-edb-cluster")].data.spec.instances}{"\n"}'

{"cpu":"3000m","memory":"3Gi"}
3
3

Create third CS CR with medium profile
:x: OperandConfig is overwritten by medium profile which is not expected

> oc -n test-size get operandconfig common-service -o jsonpath='{.spec.services[?(@.name=="ibm-im-mongodb-operator")].spec.mongoDB.resources.limits}{"\n"}{.spec.services[?(@.name=="ibm-im-operator")].spec.authentication.replicas}{"\n"}{.spec.services[?(@.name=="edb-keycloak")].resources[?(@.name=="keycloak-edb-cluster")].data.spec.instances}{"\n"}'

{"cpu":"2000m","memory":"2Gi"}
2
2

Apply the fixed code

Update CS operator CSV

> oc project <your-namespace>
> oc patch csv ibm-common-service-operator.v4.3.0 --type="json" -p '[{"op": "replace", "path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/imagePullPolicy", "value":"Always"}]'
> oc patch csv ibm-common-service-operator.v4.3.0 --type="json" -p '[{"op": "replace", "path":"/spec/install/spec/deployments/0/spec/template/spec/containers/0/image", "value":"docker-na-public.artifactory.swg-devops.com/hyc-cloud-private-scratch-docker-local/ibmcom/common-service-operator-amd64:dev"}]'

:white_check_mark: Verify that large profile is applied to OperandConfig

{"cpu":"3000m","memory":"3Gi"}
3
3

Create fourth CS CR with medium profile
:white_check_mark: Verify that large profile is kept to OperandConfig

{"cpu":"3000m","memory":"3Gi"}
3
3

Using this image, I ran the scenarios @Daniel-Fan described successfully. I also tried a couple scenarios that I had run into while investigating root cause successfully so I think this pr is good to merge. The scenario that mimics what aiops originally saw was where the CS CR was present before the cs operator which I mimicked in the following way:

scaled down cs operator
deleted default CS CR
deleted operandconfig
created dummy aiops CS CR using large size
replaced cs operator image in csv
scaled cs operator back up
watched for reconcile to complete
verified operandconfig was using large size after cs operator intialization

Great work Daniel!


Error Displaying Cover Series

To obtain the cover of a manga series, we take the first page of volume 1.
However, if the series has no volumes but only chapters, an error occurs, as there is no page for volume 1 (because it doesn't exist).
Solution
Take the first page of the first chapter or take a default image for the cover.

default image cover


[RFE] enhance prepare:Install

Please support:
prepare:
  how: install
  packages: |
       package1 package2
       package3 package4

If I have a really long list of packages, it gets highly unmaintainable... but the solution would be to split it to multiple how: install, correct?

This is probably fixed.


XD-3493: Update SI, SBoot, and AMQP dependencies

DO NOT MERGE yet, still working on an issue related to command line options with boot 1.3

@ericbottard The latest commit works a bit better but I hit a nasty issue related to the spring-hateos version. xd-container will not start with spring-hateos-0.17.0 - it requires 0.19.0 (the declared dependency in boot 3.1.M5). However I had to downgrade to 0.17.0 to get around https://github.com/olivergierke/spring-restbucks/issues/37 (see note in dependencies.properties for more info). Other than that, hsqldb-server wont start as it is looking for XD_JMX_ENABLED. I haven't figured out where that is referenced @ilayaperumalg . Property settings for xd-singlenode and xd-admin seem to work as expected with this commit but will require more testing.  But the spring-hateos thing will be a challenge.

Merged as 496f0303d3fa7a482ffc7c30f6132a7cef9a5ce9


Umm.. This is actually made with the old version of npm!

I don't wanna sound like a Karen but I would highly recommend using the latest version of Npm!

Yes because the bot was made like 1 year ago


Question: Permission vs Scope

Hello,
It's clear that using scopes as authorization rules is bad idea and that's what I what my question will be about. In your video from NDC Oslo you are presenting that

Let's have Authorization Server
It hosts applications and permissions information
Resource server received token issued by Identity Server
Resource server additionally contact Authorization Server

get permissions for current user
store them to current Users claims

Good approach, but I miss few points and I would like to ask you for details.
On IdentityServer, you're modelling claims and scopes. Scope define group of functionalities on top of resources client can do - higher abstraction of claims and preventing client to access some apiResource at all.
If you want to model permissions, you should link them to this scopes. If I understand it correctly, it's hierarchy between scopes and permissions.
Another question is, permissions belongs to resource server, what is in identity server model - ApiResource. You should be able to link them somehow with that API resource, what means your two components - Identity Provider and Authorization Provider just use one extended database.
Thanks for your hints.
Btw, don't you have code from this presentation available please?
BR
Ondrej

scopes, as per the OAUTh2 RFC, are permissions the user has granted the client. permissions in our NDC talk were about user permissions. those are two different sets of permissions.


Fix icons fixed sized

change manual sized flex to none in order to dynamically adjust size with theme constants

@ecarreras a alpha ja es veu correctament ;)

Sí, correcte! ja es veu bé! 💪🏻

:tada: This PR is included in version 2.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore: bump deps and commit Cargo.lock

Current dependencies on/for this PR:

main

PR #68

PR #69   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 28, 3:43 AM: Graphite rebased this pull request as part of a merge.


Issue reported to Prasad by his friend

"On the password it actually kept making my upper case letters automatically lower case on the first box
But the confirmation box it didn’t automatically do that. So they were a mismatch"
Cc: @Praasd

Verified this issue on app v 2.0.69(429)


Add partitionKey and transactionPartitionKey bindings

Fixes https://github.com/Azure/azure-sdk-for-net/issues/31691

/azp run net - servicebus - tests

API change check
API changes are not detected in this pull request.

/azp run net - servicebus - tests