Custom subdomain handle suddenly stopped validating

Describe the bug
Two days ago, I signed in to Bluesky on web and found my profile was showing an "Invalid handle." I use a subdomain at charles.littlegreenfootballs.com - it has been working for several months before this, and I have not changed anything in the DNS setup or the .well-known/atproto-did file.
When I use "Change handle" in the app's settings, it actually verifies the subdomain and handle; but my posts all continue to show "invalid handle."
I went through and double-checked everything about the setup and there's nothing wrong as far as I can tell. Here's that atproto-did file. It returns the file as text/plain with status 200.
Screenshots

Details

Platform: macOS and iOS
Platform version: latest versions of both
App version: 1.86.0

Did more testing and the TXT record method works for me, but the subdomain still shows "invalid handle."

Some people told me they could still see my subdomain handle. Could this be a federation issue?

@devinivy

To cross one other possibility off the list, I changed the subdomain record from A to CNAME - still no subdomain handle.

Is there some other information I can provide to help you folks look into this? I was planning to offer subdomain handles to trusted members of my site, but I can't do that while this bug is still alive.

I don't see anything on our end that would be preventing the handle from being used/validated.
Any chance you could go through the handle-change process one more time to charles.littlegreenfootballs.com so I can take a look at what state our identity database ends up in? Hopefully that should give me enough info to figure out what the failure mode is here and get us back on track.

OK, I'll do that now. Do you want me to leave it at "invalid handle" then, or can I change it back to the TLD?

Leave it at invalid handle for now, I'll be quick.

OK, it's invalid. This is what I see:

When I get one of my posts with the API, the handle shows as "handle.invalid".

Yep, I see where we're detecting it as invalid and am trying to figure out how the bit that does that is failing rn.

Could it be the length?

May want to check commits for Friday night/Saturday morning because that's when it suddenly began failing.

It's not length related or anything, I think it's got something to do with the prod network, digging into it.
The same code works fine running in other environments and sees your identity as valid etc.

So for some reason it looks like we can't curl https://charles.littlegreenfootballs.com/.well-known/atproto-did successfully from within PoP2.
I'm wondering what could possibly be going on there. We use a proxy to get out to the internet from that machine but it's not having any problems talking to other hosts on the internet.
Here's a curl:
* Establish HTTP proxy tunnel to charles.littlegreenfootballs.com:443
> CONNECT charles.littlegreenfootballs.com:443 HTTP/1.1
> Host: charles.littlegreenfootballs.com:443
> User-Agent: curl/7.81.0
> Proxy-Connection: Keep-Alive
>
< HTTP/1.1 200 OK
< Date: Tue, 25 Jun 2024 20:15:52 GMT
< Proxy-Connection: keep-alive
< Server: ATS/9.1.1
<
* Proxy replied 200 to CONNECT request
* CONNECT phase completed!
* ALPN, offering h2
* ALPN, offering http/1.1
*  CAfile: /etc/ssl/certs/ca-certificates.crt
*  CApath: /etc/ssl/certs
* TLSv1.0 (OUT), TLS header, Certificate Status (22):
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.0 (OUT), TLS header, Unknown (21):
* TLSv1.3 (OUT), TLS alert, decode error (562):
* error:0A000126:SSL routines::unexpected eof while reading
* Closing connection 0
curl: (35) error:0A000126:SSL routines::unexpected eof while reading

Attempting to curl from our router machines (the ones that act as internet entry/egress-points) shows the same issue in PoP2.
I can't even curl or ping littlegreenfootballs.com from our PoP2 datacenter.
Is there any chance you or your hosting provider is blocking Bluesky's IPs in our east-coast DC?
iirc we're not using our BGP prefix IPs in that datacenter yet.

That's odd - it was working fine for months, something must have changed. What IP should I check?

Try <IP_ADDRESS>, that's likely what we're trying to connect from in PoP2.

Not finding that IP in any of the usual places.

It's routed via Cogent on the east coast, any chance there's some kind of hosting provider filtering going on? Are you hosting on AWS or Digital-Ocean or somewhere else? I can try to spin up something and see if the provider is at fault.

The provider is Hosting Matters - OS is Centos 8. Web server is LiteSpeed/6.0.12.

The TXT record method still works, by the way. Wouldn't that also fail if there was an IP block somewhere?

Okay, it might be worth submitting a ticket with their helpdesk to see if there's some kind of firewall rule blocking connections.
Re-TXT record, the TXT record stuff is validated via DNS, we use CloudFlare and Google as DNS providers so they'll proxy requests to your DNS server on our behalf.

OK, I'll put in a ticket asking them to check <IP_ADDRESS>

Thanks, and sorry this is such a pain. The internet and routing is built on relationships and reputation and we're still pretty new :)

OK, ticket's filed, I'll update here when they answer. They're usually pretty quick.

And thank you for your assistance!

Possibly relevant?
SSL Library Error: error:0A000126:SSL routines::unexpected eof while reading #22690

Host says the IP is not being blocked, and they flushed all other blocks.

OpenSSL on my server is v 1.1.1k, are you on the newer version?

I think this might actually be an issue with cogent routing.
I'm unable to even use Cogent's looking-glass to ping your server from their Washington DC router.
From San Jose tho it works fine:

I think I might need to ring up Cogent.

My subdomain is working again, thanks. I guess Cogent straightened out whatever was clogging the pipes.

Closing this issue as solved.


Add tests for partner_settings partials

Resolves #4309
Description
The partner_settings partial was previously inadvertently deleted from the Partner Profile Edit view because there are no tests asserting for its presence. This PR adds a test for the presence of the partial when a Partner or Bank edits the Partner Profile and the presence of the show partner_settings partial when a Partner views its profile. To test that the correct request types are displaying in the partial, I assert for the presence of an enabled request type and assert for the exclusion of a disabled request type.
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
The tests all pass as expected. I tested each expectation in isolation by making each fail with the following changes:

render_template fails when I remove the code that renders the partial in the show/edit profile views.
expect(response.body).to include(...) fails when I remove the corresponding conditional and HTML from the partial for a request type that is enabled
expect(response.body).not_to include(...) fails when I remove the conditional to ensure the HTML appears in the partial for a request type that is disabled

Thank you!


Security template

Task/Issue URL: https://app.asana.com/0/488551667048375/1164365196861356/f
Tech Design URL:
CC:
Description:
This PR is related to the changes introduced by https://github.com/duckduckgo/Android/pull/760 to improve our Issues sections templates.
Github provides under Securitysection a place to publish our security policy.
This PR creates a section there and it will provide a direct access Issues -> New issue section as shown in the image below:

Steps to test this PR:
1.
1.

Internal references:
Software Engineering Expectations
Technical Design Template

Closing this, after talking with @bwaresiak, we need further discussions about how to communicate what versions we are supporting (based on our minsdk). I will bring this topic to our next mobile meeting.


pythonPackages.*: switch many packages to disabledTestPaths

Motivation for this change
Replaces #115303
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

There are many good changes here @SuperSandro2000 , and some merge conflicts now. Perhaps the amount of packages changed can be reduced to such an amount that the PR won't require using staging as the base branch.


Add examples/documentation for the test framework.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @linsun Here is some examples & documentation for the test framework.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
8c44d9f0e5c0c08ebc9186bfe19e594be0a81336
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
360c519ee0828a4e436f4d41fbecea47d0aa38de
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
360c519ee0828a4e436f4d41fbecea47d0aa38de
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
a0658132366d0485a3cffb13a8ba65722493a99c
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
360c519ee0828a4e436f4d41fbecea47d0aa38de
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
a0658132366d0485a3cffb13a8ba65722493a99c
link
/test istio-unit-tests

prow/istio-presubmit.sh
a0658132366d0485a3cffb13a8ba65722493a99c
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
360c519ee0828a4e436f4d41fbecea47d0aa38de
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
a0658132366d0485a3cffb13a8ba65722493a99c
link
/test istio-unit-tests

prow/istio-presubmit.sh
a0658132366d0485a3cffb13a8ba65722493a99c
link
/test istio-presubmit

prow/istio-integ-local-tests.sh
a0658132366d0485a3cffb13a8ba65722493a99c
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I think the main question here is, how would you verify the behavior? It
looks like simply looking at xDS is not enough, you want to verify the
actual behavior. In essence, it sounds like you want to write a e2e-like
tests.
Some of the Pilot tests should be working this way already.
On Tue, Apr 2, 2019 at 3:16 PM John Howard<EMAIL_ADDRESS>wrote:

@howardjohn commented on this pull request.
One thing I have had trouble with when trying to use the framework is to
translate real config/setup to a test.
A concrete example:
I made a change that impacts adds a wildcard match as a fallback for each
route. This change is behind a Pilot feature flag. It behaves differently
for each different settings of outboundTrafficPolicy in mesh config. I want
to send real requests and observe that I see expected results.
Is this something that is possible? A real example of something like this
would be really helpful. The closest I have seen is the sidecar api test,
but this seems to just intercept Pilots xDS response and analyze it, which
doesn't fully fulfill this. I am happy to be the real example as well
In tests/integration/examples/example_test.go
https://github.com/istio/istio/pull/13000#discussion_r271514974:

+// TestMain needs to be implemented and tests should be executed through framework.TestSuite
+func TestMain(m *testing.M) {

// Start your call with framework.NewSuite, which creates a new framework.Suite instance that you can configure
// before starting tests.
framework.

NewSuite("galley_conversion", m).

// Labels that apply to the whole suite can be specified here.

Label(label.Presubmit).

// You can restrict the execution of the whole suite to a particular environment. This restricts execution

// to the native environment.

RequireEnvironment(environment.Native).

// You can specify multiple setup functions that will be run as part of suite setup. setupFn will alwaus be called.

typo: always
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/13000#pullrequestreview-221926683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQR8I70_tnMX0gpa7NJp4ZLxPi7UTL3Fks5vc9bCgaJpZM4cYzPL
.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
eec5680eaaef2c314ba563be41461c1825c50819
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
9bab2f96aac58e6afd14d68dd92a202620b20271
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
9bab2f96aac58e6afd14d68dd92a202620b20271
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
9bab2f96aac58e6afd14d68dd92a202620b20271
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
9bab2f96aac58e6afd14d68dd92a202620b20271
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Normal Buffer

What?
  Changing the way get normal

Why?
   Godot 4 now have normal buffer which can access the same with depth

How?
   create smapler2D for normal texture
   then get access the uv we want

After Checking i don't think it works

Damn, I had no idea they added a normal buffer. Definitely seems promising, I'll explore it a bit and see if I can get it working. It would definitely improve the shader. Thank you!

Update: got it working, the shader is now much better. The normal buffer simply has to be remapped with texture(NORMAL_TEXTURE, SCREEN_UV).rgb * 2.0 - 1.0;


If a file name ends with html.md, server could not load the URL

🐛 Bug Report
If a file name ends with html.md, the server could not load the URL. It is throwing an error
Cannot read property '1' of null at /docusaurus/lib/server/server.js:251:24
Have you read the Contributing Guidelines on issues?
Yes.
To Reproduce

Create a file with html.md in the documentation
Run the docusaurus server
Open html.md with the docusaurus server
Observe the error
Cannot read property '1' of null at /docusaurus/lib/server/server.js:251:24

Expected behavior

It should open the link without errors
Actual Behavior

html.md link is not opening, throwing the above error.
Reproducible Demo
It is easily reproducible who is running docusaurus server. No need for DEMO.

@endiliey Can I work on this?

Yes go ahead.

Actually I can't even reproduce this issue. Can you give more detailed repro instructions?


Update alerting_rules.md

Docs only change. Fix ALERTS example to use regex.

This is not a promql expression, therefore I don't think this should be fixed.

Interesting. Why's that?
Someone could be reading the docs, see this expression, copy and paste it into their expression executor, run it, and not seeing anything.
Copy:
ALERTS{alertname="<alert name>", alertstate="pending|firing", <additional alert labels>}

Paste and Adjust to:
ALERTS{alertstate="pending|firing"}

No results, head-scratch. Or simply believe they don't have any firing or pending alerts.

Mmh indeed it does not seem harmful.

We have discussed this in the bug scrub and decided that in both case this is confusing, we should improve that part of the documentation to make it more meta-syntactic.
Something like alertstate="<pending or firing>".
Are you willing to update this pull request in that direction? Thanks!

Sure thing, that works for me. Updated.

Thanks!


Unable to take mobile screenshot

Perhaps I am missing something obvious, but I am trying to figure out how to emulate a mobile device/view and take a screenshot, and I am stuck.
My code is here:
https://gist.github.com/ccampbell/5fb8cee476b41474d23d6a12778e6f7f
I have tried everything:

Setting the user agent using SetUserAgentOverride (that option seems to have been removed)
Setting the user agent as a command line flag
Setting the viewport width, height, scale
Setting the width and height and scale in emulation.SetDeviceMetricsOverride
Setting the screen orientation to portrait
Setting touch emulation

You can see in the setViewportAndScale method.
The screenshot comes out like this:

So the size and scale seem okay, but the page content is not correctly being resized to fit the viewport
Any help would be greatly appreciated. Thanks!

@ccampbell were you able to get this working? If not, perhaps look into how other libraries like Puppeteer accomplish this. If they can do it, it's very likely we can too.

No, unfortunately I was not able to. I ended up using puppeteer.

For future reference, this can very likely be accomplished with https://chromedevtools.github.io/devtools-protocol/tot/Emulation#method-setDeviceMetricsOverride, which allows emulating a mobile device.

 They're practically the same, so I'm closing this issue.


accepts_nested_attributes_for should enable autosave

i updated from 2.3.4 to 2.4.1, and now update_attributes is no longer updating nested records.  not sure if this provides all the information you need, but if you need additional, let me know.
class PlaceType
  include Mongoid::Document
  has_one :survey
  accepts_nested_attributes_for :survey
end

class Survey
  include Mongoid::Document
  belongs_to :place_type
  has_many :questions, :class_name => "SurveyQuestion"
  accepts_nested_attributes_for :questions, :reject_if => :all_blank
end

class SurveyQuestion
  include Mongoid::Document
  belongs_to :survey
  field :text
  field :question_type, :default => "text"
  attr_accessible :text, :question_type
end

From 2.3.4...
> PlaceType.find('4f164050e1607ca575000036').update_attributes({"survey_attributes"=>{"questions_attributes"=>{"1326861546806"=>{"text"=>"foobar", "question_type"=>"text"}}}})
=> true
> PlaceType.find('4f164050e1607ca575000036').survey.questions
=> [#<SurveyQuestion _id: 4f164eece1607caac0000001, _type: nil, survey_id: BSON::ObjectId('4f164050e1607ca575000037'), text: "foobar", question_type: "text", question_type_data: nil>]

From 2.4.1...
> PlaceType.find('4f164050e1607ca575000036').update_attributes({"survey_attributes"=>{"questions_attributes"=>{"1326861546806"=>{"text"=>"foobar", "question_type"=>"text"}}}})
=> true
> PlaceType.find('4f164050e1607ca575000036').survey.questions
=> []

Issue caused by: 2bbb85f659c8209f393a83fe0d93862160acab44

Current workaround is to set :autosave => true on the child relations:
class PlaceType
  include Mongoid::Document
  has_one :survey, :autosave => true
  accepts_nested_attributes_for :survey
end

class Survey
  include Mongoid::Document
  belongs_to :place_type
  has_many :questions, :class_name => "SurveyQuestion", :autosave => true
  accepts_nested_attributes_for :questions, :reject_if => :all_blank
end

ActiveRecord turns this on by default when setting accepts_nested_attributes_for, Mongoid should do this as well.

After starting to implement this I am seeing some other side effects of having the autosave functionality change here, so going to flag this for 3.0 since we have an easy workaround for now.

The autosave workaround doesn't work for embeds_many and I still can't get nested attributes working

@pspeter3 Can you be more specific on what your issue is?


pkg/tlsutil: impl none of the CA and App TLS assets are found

Implement and test the case where none of the CA and application TLS assets are found. In this case, GenerateCert should simply generate both and put them into the k8s cluster.

LGTM


fix: import/no-extraneous-deps in .ts config files

Several config files such as webpack.js are not listed as {js,jsx}.
Fixes #121

Thanks @matt-tingen !
Unfortunately this solution removes the js and jsx from the regex, when they should remain.
Have fixed in this PR: #190
Thank you again for this! And sorry for the delay 😬


Create a readme for Filters

How to create a Readme for Filters ?

Copy each component code and paste it in Tailwind Playground
Take a screenshot of that. (Make sure to have a proper aspect ratio. 16:9 will be good) and add it to the readme in the table format
Make sure to enclose the screenshot under anchor tag so that one can just choose the filter and after clicking that image will be redirected to the code of that component.
NOTE:- Make sure each component screenshot is of equal size so that it fits in the layout.

For Reference you can see Button Readme

Please assign me @Rajdip019


Git Ignore **

Description
I have a problem when I have a special ignore in Git like /images/**/
This should ignore all subfolders in "images" but not the files within, in Git it works perfectly but in Atom it shows the images folder empty without the files in the tree view panel, when the files should appear except the sub-folders.
Versions
Version 1.24.1

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/tree-view/issues/100. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


8/23/2024 Update Failing to complete on RG353M (v1 screen)

When attempting to OTA update to the recent 8/23/2024 build from the prior 7/21/2024 build, the update does not complete and produces the following error messages.
In the update script's terminal text:

The update couldn't complete because the package did not download correctly. Please retry the update again.

In the Update utility afterward:

There was an error with atempting this update. Did you make sure to enable your wifi and connect to a wifi network? If so, enable remote services in options and try to update again.

Wifi was enabled in all attempts and confirmed connected in the Wifi utility.

Tried enabling remote services as instructed in the utility error message and retrying, no change.

Tried connecting my access point's 5Ghz and 2.4Ghz networks, as well an alternative access point, all with the same results.

Behavior occured with both my RG353M (v1 screen) and my wife's RG353P (v1 screen) (I know, 353P is not officially supported, but historically works with the 353M builds, with OS+games cards between the two being completely interchangeable).

I then performed a fresh-flash of ArkOS to my OS card using the base-build from the Wiki (1/27/2024), and immediately upon boot, re-connected to wifi, and ran the Update tool again. Based on the lengthy amount and duration of scrolling terminal text, a large number of updates seem to have been installed successfully to the freshly flashed ArkOS, but at the end of the process, the same terminal text error flashed before showing the same Update utility error quoted above.
Upon reboot, the version number displayed at boot time is now shown as 8/23/2024... But running the Update function again to ensure no further updates returns the same errors quoted above.
UPDATE:
I was able to manually download the three 8/23/2024 update files (2 zip, 1 z01) from the commit history here: https://github.com/christianhaitian/arkos/commit/6bc222c0fb433962b71f29f7023d32d878a43b97#diff-3119b6d20dd10a09047d6f5f6d073eb634914219b7e8098050d84bb60203f4ef , placed them in the dev/shm folder in device storage, and re-ran the Update function. Updates finished installing without error, and checking again after, shows no new updates available, if that helps. Seems like they just didn't download via the Update script in the usual way.

Sorry about that.  It's been resolved now.


How to get token level embedding?

Using the pooling strategy i can get the first or the last token, how do I get a token embedding for all the tokens in a sentence?

Set pooling to None.


refactor(ci): remove publish.yml that's no longer used

When I was heading out to make a new release seeing both CI & Release and Publish to NPM had me pause for a minute to figure out which one to use:

And it appears that it's safe to delete publish.yml since it's setup to only run on pushes to a studio-v3 branch:
https://github.com/sanity-io/sanity-plugin-internationalized-array/blob/ad9ae6153821cd6f791f0eae23026dfb47ede8c5/.github/workflows/publish.yml#L4
A branch that has since been deleted 😌

Yeah that was an old publish script I used to use. Not as 🪄 as semantic-release.


Enforce cluster-wide a security baseline for seccomp profile installation

What would you like to be added:
In a cluster where different tenants can install seccomp profiles in different namespaces, it is paramount to enforce cluster-wide
a security baseline for seccomp profile installation. This will prevent that the tenants will introduce system calls which are not allowed by the cluster operators in order to avoid container escapes and such.
It should be possible to install only a subset of the baseline. For example, a seccomp profile should only be successfully installed when it contains a subset of the system calls available in the baseline profile (aka cluster default seccomp profile).
Why is this needed:
This will allow a cluster administrator to enforce which system calls are allowed to be enabled in the cluster via the operator.  This will prevent from introducing dangerous system calls which might lead to container escapes.
We want to prevent that someone extends the seccomp profile installed in a namespace beyond the cluster default profile(https://kubernetes.io/blog/2021/08/25/seccomp-default/#seccompdefault-to-the-rescue).
User story covered

In this thread is a design discussion: https://kubernetes.slack.com/archives/C013FQNB0A2/p1650358471997719


More statistical results/summaries in CSV exports

I recall mean standard error coming up a couple of times. I don't remember what CPCe provides in its exports off the top of my head - that's definitely worth checking. Other than that, we could ask NOAA for a list of things.
Worth pointing out that issue #373 was done recently, so it's good to check whether everyone knows about it.

Worth noting here that the summary numbers can just be unconditionally added to the export file; if someone doesn't need them, they're not taking up much space anyway. So, this task might not even involve adding any options to the UI.


Paging breaks when last item has an ampersand (&) in the name

Expected behavior:
Clicking on the next arrow should go to the next page of results, even if the last item contains an ampersand
Actual behavior:
If the last item contains an ampersand the next page never displays and there is an error message in the background
Steps to reproduce:
Create 26 inventory items and ensure that the 25th one has an ampersand in the name.  Click on the next page button
Screenshots (if applicable):

OS and Browser:
All

Turns out this issue was fixed in version 0.9.2.  Our Zambia hospital was on version 0.9.1 when they encountered this issue.  Upgrading them to 0.9.2 fixed the issue.


Terminology: WebID Profile

The current draft specification includes, in the Terminology section, a definition of a WebID Profile:

WebID Profile or Profile Document
A WebID Profile is an RDF document which uniquely describes the Agent denoted by the WebID in relation to that WebID. The server must provide a text/turtle [turtle] representation of the requested profile. This document may be available in other RDF serialization formats, such as RDFa [RDFA-CORE], or [RDF-SYNTAX-GRAMMAR] if so requested through content negotiation.

Even the terminology uses two names for this entity: "WebID Profile" or "Profile Document", and throughout the draft specification, this entity is referred to, variously, as:

WebID Profile Document
WebID Profile
Profile Document (and Profile document)

If a single phrase were used to denote this entity, there would be several advantages: first, the text of the draft specification would be more consistent; second, other documents (e.g., external specifications) could be much more precise when referring to or refining the notion of a WebID Profile Document.
I have no particular preference about the phrase that is chosen to describe this entity, just that it be consistent.

@csarven @bblfish —
https://github.com/w3c/WebID/commit/48f571868c7cb5d5612e48a3eae6342ce85b0983 and https://github.com/w3c/WebID/commit/c3e40307a1f74e8b9aa376f58d8c0e2c5fe43fc7 should have been put through the normal PR process, even if it were a second PR doing the same thing as #4 has failed to (entirely) do.
Having not done so left lines 537, 539, and 705 referring to the document, and line 706 referring to the WebID document, where all should instead refer to the WebID Profile Document.
https://github.com/w3c/WebID/pull/13 fixes those 4 lines (plus one instance of the document which is better as this document)

I'm not aware of any "normal PR process" guaranteeing work that's sound, complete, let alone unanimity.
PR 8 made significant improvements based on the need in this issue. What additional ceremony, discussions, reviews, or "process" is expected for a PR open for 6 months at this point on the 11th hour? The suggestions you're making now could've been done much earlier - not to imply that they're not appreciated. If something - which is fairly subjective in some cases - wasn't part of an original PR, I don't see that as an issue or something to be pedantic about or a blocker. We can move on by removing friction as reasonably as possible and then follow up with new updates.
Our goal was to be consistent with terminology and clarify context. That does not entail that every "the document" needs to be replaced with "the WebID Profile Document". This is very editorial. To take an example, in https://github.com/w3c/WebID/commit/c3e40307a1f74e8b9aa376f58d8c0e2c5fe43fc7#diff-e71370fe4d6679f0e463b9635a1270db4399459e10231e8e9c471976b4641b5eR537 , you're recommending "denotes the document describing" changed to "denotes the WebID Profile Document describing", i.e.:

The WebID Profile Document URI - "http://www.w3.org/People/Berners-Lee/card" (without the #i hash tag) - denotes the document describing the person (or more generally any agent) who is the referent of the WebID URI.

to

The WebID Profile Document URI - "http://www.w3.org/People/Berners-Lee/card" (without the #i hash tag) - denotes the WebID Profile Document describing the person (or more generally any agent) who is the referent of the WebID URI.

I don't actually agree that change is necessary. The context of "the document" is sufficiently clear and readable (to me). Had that suggestion was made in the PR, I wouldn't have necessarily raised an objection against it either. If it was so pressing, it could've been raised any time within the past 6 months.

I agree that we also need to take into account the readability of the text, and too much repetition will make the text unreadable. This is for the same reason that Turtle is much more readable to a human than NTriples is. :-)

(@bblfish — tangent... As I have understood it, the avoidance of repetition in Turtle, through the use of ; and ,, was more to favor human authors than human readers. Of course, the inconsistent handling of the empty ;. vs ,;. vs ,. sequences that are found in some human-authored Turtle — and even in some software-authored Turtle, as that software itself was human-authored — has helped neither authors not readers, and may be addressed in a contemplated next revision of Turtle.)
My suggested additional WebID Profile Document changes are aimed in part at increasing comprehensibility for readers with English as a second (or third, etc.) language, for whom the context of document may not be sufficiently clear.


Couch iOS library is FIPS compliant

Hi,
We are using Couch CBLite iOS for development.
Is this library FIPS compliant?
As per apple documentation, from iOS 7 onwards they are FIPS compliant.
https://support.apple.com/en-in/HT202739
Can you please confirm this?
Thanks,
Anil Kumar

@snej and @pasin Can you answer for this question?

This is an iOS question and there is already a ticket (https://github.com/couchbase/couchbase-lite-ios/issues/1418) open in iOS. So I'm closing this ticket.

By default, encryption is done by SQLCipher, a fork of SQLite with encryption support. SQLCipher claims FIPS compliance for their paid enterprise edition but apparently not the open source code. According to this mailing list thread the FIPS compliance has to do with how the crypto primitives are implemented; on Apple operating systems we call into the system CommonCrypto library which as you've pointed out is FIPS compliant.
If you choose ForestDB storage instead of SQLite, the encryption is done by our own ForestDB library. This library itself has not been FIPS certified, although, as with SQLCipher, the actual crypto is done using Apple's complliant AES256 implementation.
FYI, we request that developers ask questions on the Couchbase web forum or the Couchbase Mobile mailing list. We use Github issues only for bug reports and feature suggestions. Thanks.


Determining what is slow when resizing is needed

There a re a lot of things happening when one source is resized. It is clear that the performance suffers from a lot of resizing, but not so clear why.

resizing all linear operators (gradients, PSFs, etc) for the source
catching the exception
reinitializing bSDMM

My suspicion is it's the first item. If the third item is the bottleneck, it's possible doable to rewrite bSDMM to allow for a transparent resizing on the fly. This would also render the exception obsolete.

became largely obsolete with dbdbb3667ba69e7dc1d1a320aa4ad4bdd4884cae


optim sha512 :rocket: :rocket: :rocket:

Pull Request type

Please check the type of change your PR introduces:

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

What is the current behavior?
< [PASS] alexandria_math::tests::sha512_test::test_size_zero (gas: ~5698)
> [PASS] alexandria_math::tests::sha512_test::test_size_zero (gas: ~632)
<         steps: 735949
>         steps: 85153
<         memory holes: 73172
>         memory holes: 4849
<         builtins: ("range_check_builtin": 142428, "bitwise_builtin": 2025)
>         builtins: ("range_check_builtin": 15797, "bitwise_builtin": 2702)

< [PASS] alexandria_math::tests::sha512_test::test_sha512_size_one (gas: ~5697)
> [PASS] alexandria_math::tests::sha512_test::test_sha512_size_one (gas: ~633)
<         steps: 735926
>         steps: 85256
<         memory holes: 73172
>         memory holes: 4759
<         builtins: ("range_check_builtin": 142425, "bitwise_builtin": 2025)
>         builtins: ("range_check_builtin": 15812, "bitwise_builtin": 2720)

< [PASS] alexandria_math::tests::sha512_test::test_sha512_lorem_ipsum (gas: ~16591)
> [PASS] alexandria_math::tests::sha512_test::test_sha512_lorem_ipsum (gas: ~1823)
<         steps: 2149930
>         steps: 242168
<         memory holes: 217632
>         memory holes: 13335
<         builtins: ("range_check_builtin": 414769, "bitwise_builtin": 5929)
>         builtins: ("range_check_builtin": 45564, "bitwise_builtin": 8096)

Does this introduce a breaking change?

[ ] Yes
[x] No

Other information

Added comments in code to describe optimisations,
Mostly all them bit shifts and rot functions now use precomputed powers of two instead of trying to compute it on every steps. That gives us 9x performance gain.

Actually there are more optimisations possible. But I’ll look at that another day.


dbg! in production

https://github.com/aevyrie/bevy_framepace/blob/main/src/lib.rs#L229 prints every frame, can it be removed?

Debug messages can be filtered out in your binary crate, yes. Noting here in case anyone stumbles across this.


fix(curriculum): prevent hardcoding pokemon

Checklist:

[X] I have read and followed the contribution guidelines.
[X] I have read and followed the how to open a pull request guide.
[X] My pull request targets the main branch of freeCodeCamp.
[X] I have tested these changes either locally on my machine, or GitPod.

Ref #54063

Two tests were added. One randomly generates an invalid Pokemon name and tests for a Pokemon not found response. The other test randomly generates a valid Pokemon id and compares it to the types received from our Pokemon API and sees if they match. If I had just used a few regular names, we would have to continually add new options to our generic tests. It also makes test writing a lot smoother.

I am living a couple of wording suggestions, I never completed this project so I don't have a project to test these new tests with.

Except there is a solution attached at the bottom of the file you can test against.


Delay x hours before switching to next race

Is your feature request related to a problem? Please describe.
I rarely watch the races live and the card jumps to the next race immediately after the race has finished.
Have you tried if it is already a feature? Have you checked the wiki?
Yep and couldn't see a delay option
Describe the solution you'd like
Delay switching to the next race as soon as its finished by the specified number of hours

I'm gonna make it configurable

To be sure. Which card are you talking about?

Hi, I'm using the "next_race" card, thanks

You should be able to do it with the latest version


Sample project does not work & missing file

The sample project throws a runtime exception because it can't find the roslyn compiler. After fixing this by updating nuget packages, the mdf data file cannot be found. Can't even get past the first step of running the the project.

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

ID: 72d855c6-51fd-a1da-5c44-8b44658c0211
Version Independent ID: 15c87b07-e89e-2b0c-6337-3940a5259101
Content: Build ASP.NET app with SQL Database - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-dotnet-sqldatabase.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

Managed to fix this: You need to create an App_Data folder in the solution folder. The MDF file is created in this folder, but the folder has to exist for the MDF to be created.

@ashley-pillay, Glad to know that you have sorted out the issue. Thanks for the update. We will close this out, but if you feel you need more information please just let us know.


RDS data gibberish

Hello
I am trying to decode the RDS, but the data are gibberish: I can "distinguish" the station name, but most of the time, it looks randomized. (I am running the code on a SAMD21)
I first thought my board was damaged, but I can have it work well with a Raspberry (station name/ text and time) using this code
https://github.com/franckinux/python-rd5807m
It would really cool to update the serial monitor to show the  RDS data
thank you a lot for all your work!

This is what I get
(the correct station should be CLASS.21)
c cR21
ÆÜcRS.cR
CLAScR21
OBAScRcR
CLAScT21
CLcRS.cR
as  S.21
ÕFAS
asc cR21
asÆÜS.21
CL¿ cR21
CLAS‡ cR
CLcRcR@
CLcRS.21
CLcSS.21
CLcSS.21
AScScS
CLcSp 21
CLAS  21
ascS
CLcSS.21
asc ¿ 21

I have dedicated few time to this project due to my academic activities.
I will check and fix it  as soon as possible.
Thanks.

Could you try it again?


Add resources limits and requests for each chart.

For development purpose, the resources  are disabled by default and can be enabled under the development flag in the values.yalm files.

This change is

This is a great overall cleanup effort @Ciello89.  Really nice work. You have caught a ton of items that needed to be corrected.
There are a few issues to address.

Thanks all for the reviews, I'm on it now!

Review status: 0 of 33 files reviewed at latest revision, 22 unresolved discussions.

glance/templates/api.yaml, line 15 at r1 (raw file):
Previously, Ciello89 (Marcellin Tchassem) wrote…
I tried to keep consistency with the other charts here, but this could be done in another cleanup effort

Unless I'm missing something, changing this to dep_check would make this consistent with the most recent charts being added.  I'm not against changing this to dep_check here.

glance/templates/api.yaml, line 44 at r1 (raw file):
Previously, Ciello89 (Marcellin Tchassem) wrote…
Done.
@intlabs that would be good idea. We could do it as enhancement!

I'm not against that idea Pete.  If it's going to be used throughout each chart, it makes sense to break it into a common template.

Comments from Reviewable

i agree with @wilkers-steve, @Ciello89...the more that we can clean up with the reviews we get from the community, the less work we have to keep track of in the future. ;)

Hey @Ciello89 - While merging, we invalidated a few files in your request.  Can you update to latest so we can merge.

Hey @alanmeadows - I resolved all the conflicts. It should be good to go now.

Hey @alanmeadows, I think all the conflicts have been resolved now. It should be good to go.


Suggestion Model and CRUD

Add this feature and also QuarterMarks crud

Story Points: 3
See the discussion in Backend S2 W8
Powered by Parabol


is there a way to find out where the configuration file of youtube-dl is ?

Is there a way to find out where the configuration file of youtube-dl is ? The only info. I could find is -
 --config-location PATH
              Location of the configuration file; either the path to the config or its containing directory.

which means you could give it a path but not a way to query it.

Configurations are stored at default locations. --config-location is used to provide user configuration under custom location. Read FAQ.


Support for CSP subscriptions

Discussed in https://github.com/helderpinto/AzureOptimizationEngine/discussions/69

Originally posted by spike28742 March 22, 2023
Hi
When I try to deploy I am getting an error that the subscription is not supported.  I managed to deploy the solution about a month ago.  Are there any limitations why the solution will not work on a CSP based subscription?
Also noticed that the cost recommendations never worked, are there any specific requirements for these to work?
Thanks for your support and keep up the great work on this tool.

Thank you for raising this issue, @spike28742. It was recently addressed in the conversation around pull request #67. You'll find there the answers to your questions. Based on the feedback, the AOE recently got a new deployment switch (AllowUnsupportedSubscriptions) that allows customers using CSP and other unsupported subscriptions to still use the tool at their own risk. See the FAQ for more details.

Note": I will add a note to my blog article to reference the FAQ.


Dirt Road: Compile simple JSon exchange REST service

We need to be able to handle a web service that sends and receives an object.  Example:
[Route("api/[controller]")]
interface IFortuneController
{
    [HttpGet("")]
    Task<IEnumerable<Fortune>> ListFortunes();

    [HttpPost("")]
    Task<Fortune> CreateFortune(Fortune fortune);

    [HttpGet("/{id}")]
    Task<Fortune> Get(string id);

    [HttpPut("/{id}")]
    Task<Fortune> Update(string id, Fortune item);
}

Eventually, we need to be smart about handling URIs, etc.

This has been done, albeit the discovery part is not working yet.


Anti-aliasing in v0.98 needs explanation

From the 0.98 changelog:
Weathermap will use anti-aliasing if your GD supports it (php-bundled GD doesn't) (thanks shd)
I can see the changes to Weathermap.class to add transparent backgrounds to everything, which supposedly turns on GD's TTF anti-aliasing.  I also see the boolean login added in commit 73450f711405a23aecb8768c94eee841749a5cc6.
What I can't make any sense of is the "php-bundled GD doesn't" comment.  It seems to directly contradict the PHP changelog for 7.2.0:
7.2.0 | imageantialias() is now generally available. Formerly it was only available if PHP was compiled with the bundled version of the GD library.
In general, anyone running a vaguely-modern version of PHP will get anti-aliasing, and was probably getting it to begin with.
Recommend either removing the confusing parenthetical caveat from the changelog, or expanding on it a bit so non-PHP-developers can understand whether they're getting anti-aliasing or not.

There's plenty of PHP 7.1 and lower in the world still! I just checked CentOS 7, and it will currently install PHP 5.4 if you ask yum for PHP (+ years of redhat backported security patches, but still php 5.4).
The relationship between php, GD and various distros is not as simple as you would like. I haven't checked recently if Debian still use the actual gd package rather than PHP's own bundled gd. But if that's the case, then there is still a distinction, because the functionality is (AFAIK) still different.
I'll check into the current situation on a variety of OS. I'm far more concerned with answering "I enabled antialiasing and I'm not getting it" over and over, than removing a few words from the manual. Agreed that expanding it might be useful though.
It's not TTF antialiasing, it's line-drawing anti-aliasing, incidentally. freetype draws with antialiasing anyway.
Personally, I think it looks ugly, anyway :-)

Yep, Debian still use libgd, not bundled: https://packages.debian.org/stretch/php7.0-gd
This is what I'm talking about: "To enable GD-support configure PHP --with-gd[=DIR] , where DIR is the GD base install directory. To use the recommended bundled version of the GD library, use the configure option --with-gd ."
https://www.php.net/manual/en/image.installation.php

Yep... my point was that since the assertion in the changes file is the logical opposite of what the PHP docs say, it leads to confusion.  It certainly let to confusion by me - I'm using an older PHP (7.0), with the built-in GD, and antialiasing is working for me.  I went down a rabbit hole this morning because of the caveat in the changelog, without thinking of zooming in enough to tell if it was antialiased or not...
BTW, I agree about ugly - small, rotated text is nearly illegible one of our maps [https://observium.merlin.mb.ca/weathermap/infrastructure.html].

Again, imageantialias() is nothing to do with text, it affects line drawing - so links and nodes.
Freetype does its own antialiasing when rendering fonts. I'm pretty sure it always does it. It's also not great, especially at angles, at inter-character and inter-line spacing. That's been a problem for quite some time (if it still does it, since 2007 - I haven't checked in a while), and why I stopped trying to add cool text features. ("cool" like multi-line text). It partly depends on the font you use, but not completely. That's when I decided SVG would be a better place to spend some time, if I ever got it :-)
I will have a look at improving the manual. I think probably by expanding what check.php says about it, and referring to that, since check.php can actually look at your php installation, even if you don't know how it got there.


Usage examples

Do you have any examples on how to use this package with a letter document class like dinbrief or scrlttr2? I would be mostly interested in how to properly align the stamp in scrlttr2 to fit into a DIN lang envelope.

After unsuccessfully messing around with scrlttr2s addrfield=backgroundimage option, I realized that the most straightforward solution seems to be to prepend the stamp to the recipient parameter:
\documentclass[version=last]{scrlttr2}
\usepackage[ngerman]{babel}
\usepackage{postage}
\begin{document}
\setkomavar{fromname}{Peter Musterfrau}
\setkomavar{fromaddress}{Hinter dem Tal 2\\
                         54321 Musterheim}
\begin{letter}{%
    {\centering\includestamp[compact]{sample-stamps}\par}
    Petra Mustermann\\
    Vor dem Berg 1\\
    12345 Musterhausen%
  }
\opening{Liebe Vereinsvorsitzende,}
\end{letter}
\end{document}

The result looks very similar to what you get if you add sender/recipient information in the Deutsche Post web app and select "Ausdruck 2-spaltig (DIN A4)" as output format.
How about adding this as an example to the package? Or do you know of a better way to include the stamp in scrlttr2?

After unsuccessfully messing around with scrlttr2s addrfield=backgroundimage option, I realized that the most straightforward solution seems to be to prepend the stamp to the recipient parameter:
\documentclass[version=last]{scrlttr2}
\usepackage[ngerman]{babel}
\usepackage{postage}
\begin{document}
\setkomavar{fromname}{Peter Musterfrau}
\setkomavar{fromaddress}{Hinter dem Tal 2\\
                         54321 Musterheim}
\begin{letter}{%
    {\centering\includestamp[compact]{sample-stamps}\par}
    Petra Mustermann\\
    Vor dem Berg 1\\
    12345 Musterhausen%
  }
\opening{Liebe Vereinsvorsitzende,}
\end{letter}
\end{document}

The result looks very similar to what you get if you add sender/recipient information in the Deutsche Post web app and select "Ausdruck 2-spaltig (DIN A4)" as output format.
How about adding this as an example to the package? Or do you know of a better way to include the stamp in scrlttr2?

Any interest in adding such an example?


Consider additional DataType variant

https://github.com/rigetti/qcs-sdk-c/blob/f7632fc6426ee23f9c99b8fe4fb66015029ac85a/src/lib.rs#L259-L262
There is now another conversion into ExecutionData, like this one, that operates on Vec<Vec<i16>>. Instead of introducing a new DataType variant now (Short ?) , we are re-using Byte, which could truncate the result. Not an issue now, but it's worth tracking!

I see no reason not to add Short to get closer to parity with the Rust SDK. I think the only reason I didn't do it initially was I didn't know of a Quil program I could use to test it. Do you have one in mind?

@dbanty no program per se, but this was in response to a bug that was caught in the existing system. the quickfix is all we need to keep things moving smoothly, so this issue is just backlog. I don't think there is anything that would block us from adding Short though.


Full screen mode playback controls

Can you add playback controls in full screen mode?

Like Car Mode just in full screen? What Controls would you want?

I apologize, more details were needed. When I am watching a video in full screen mode and touch the screen to pause or rewind, etc. It jumps back to portrait mode then i have to switch back when i am done.

How about binding the AreTransportControlsEnabled state to the full screen state? If in full screen is true then AreTransportControlsEnabled as well.
        void ShowFullscreenVideo(bool state)         {             mediaPlayerElement.IsFullWindow = state;             mediaPlayerElement.AreTransportControlsEnabled = state;

Makes sense to me


Add support for Commit Search endpoint

Added support for this new preview endpoint which lets users
search for commits based on various conditions.
GitHub announcement -
https://developer.github.com/changes/2017-01-05-commit-search-api/
Docs - https://developer.github.com/v3/search/#search-commits
Fixes #508.
\cc @shurcooL @gmlewis

@shurcooL can you please review again? 🙂

Thanks @shurcooL, I will make the changes as suggested.

@shurcooL I have made changes as you suggested.

@shurcooL can you please this again?

Merging.

Integrated in https://github.com/google/go-github/commit/3b59f32154c06d3e31093f10e58700aeaf039405

Thanks @gmlewis


Add DeepCopy capabilities for DynamicObj

Update: renamed this issue so we can focus on the combine rework in another one: #43
Both should do the same (or there needs to be a _.combine instance method), and i think they should be adapted to not mutate any of the 2 objects but rather return a new object.

Additionally, CopyDynamicPropertiesTo should not fail if overwrite is set to false. The expected behavior is that on overWrite = false, new properties are added to the target, but existing ones are kept as-is.

This has been a gigantic rabbit hole, and the solution is not very satisfiying, but we at least have some Deep copy capabilities in https://github.com/CSBiology/DynamicObj/pull/42 . It is i think impossible to return a completely copied result though, as we do not have enough knowledge about boxed dynamic props to deep copy any type/class (and there might be ones that cannot be deep copied anyways). I have settled on a "good enough" approach for now.

So typed objects without a "Copy" method will result in a "DynamicObject" with the same fields?

So typed objects without a "Copy" method will result in a "DynamicObject" with the same fields?

Yes but Note that this is a draft PR. I think there is a point to be made that static properties should be copied as well. That way, we can "recover" the derived class if needed.
I think there is no other way for this method. This is the 'try maximum nested deep copy' method. I think as long as this is well documented everything is fine.

Oh, so currently only the dynamic properties are copied? Shouldn't it be a simple addition to copy the static fields too? E.g. decided by an optional boolean parameter?
Yes, there is no secure way to deep copy classes, even via Reflection. You can never be sure how the constructor is meant to be used.

Note that this will not be closed via #42, combine still needs work afterwards

Closed via #42


[Add a Feature] Add disabled support

What feature

Yes, there is a disabled in the code but it does nothing rn

Could you tell me what disabled means?

Button parameter

It should work plz give me the code

await ctx.send("Are you sure?", components=[[Button(label="Yes", custom_id="yes", style=ButtonStyle.green, disabled=True), Button(label="No", custom_id="no", style=ButtonStyle.red)]])

Are you using the newest version of this library?

Yes, I downloaded it like, 3 days ago


Add ability to check if user is authenticated or not

There should be a way to this this with the headers being sent from the browser ($request->getUser(), $request->getPassword()).
How should this be accessed?

Dependency injection
Helper method
Facade

This should be completely optional, so users should still be able to just include the middleware and be on their way. This can be done in a minor release, so it'd become 1.1.0.
The middleware can be edited to make use of this new way to determine if the user is logged in.

Fixed in f065ec3b7cb69efd8ac5c7e5716d2a25f16e4337


feat: add option to disable magic links

Adds an option to disable magic links, as they are more prone to email sending abuse than other email authentication methods.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 57.862%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/api/magic_link.go
1
3
33.33%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.005%

Covered Lines:
9126

Relevant Lines:
15772

💛  - Coveralls

Generate link is used to generate a link that is sent separately from the Auth server, so that's fine. This only blocks magic links generated from the endpoint.

Yup, that makes sense - let's document that the toggle only blocks magic links generated from the endpoint then.


Fastlane >= 2.76.1 with slack-notifier 2.3.2 conflict

Hey,
seems the latest (as of writing this) Fastlane version 2.76.1 needs slack-notifier 2.3.2 but xcov 1.3.x still depends on slack-notifier ~> 1.3.
Could you please update that dependency so slack notifications work again? Thanks!

I'm sorry, we use Fastlane version 2.83 with Xcov version 1.3.2 and slack-notifier plugin 2.3.2.
Since the update to fastlane version 2.83, our Xcov Slack Notification ignores the "slack_username" and "slack_channel" properties.
Do you have any ideas, how to fix this problem?


Event if I setTakeCount to 10 in the TableQuery, I get a result set of more than 200 Records.

Hi @anandcpadiya , can you take a look at testTableQueryWithTake(...) in https://github.com/Azure/azure-storage-android/blob/master/microsoft-azure-storage-test/src/com/microsoft/azure/storage/table/TableQueryTests.java
If you are doing the same logic, can you provide repro code and what device you are testing on?

Worked perfectly. Thanks a lot.
I did not find any example regarding it. So putting a question was the only options for me.
It would be better if someone from your side put a perfect example of it inside https://github.com/Azure/azure-storage-android/blob/master/microsoft-azure-storage-samples/src/com/microsoft/azure/storage/samples/table/TableGettingStartedTask.java.

Great! We have an open item to improve our samples #16 and are actively working on a new approach toward samples.

Closing as we already have an item tracking improving samples


Cargo build error ... libc imported redundantly

Hi .. I just checked out a copy but immediately ran into a compilation error:

jlbirch@jlb6740:~/github/nix$ cargo build
Updating crates.io index
Updating git repository https://github.com/rust-lang/libc
Compiling libc v0.2.54 (https://github.com/rust-lang/libc#6bc42d40)
Compiling nix v0.13.0 (/home/jlbirch/github/nix)
Compiling void v1.0.2
Compiling cfg-if v0.1.9
Compiling bitflags v1.0.4
error: the item libc is imported redundantly
--> src/mount.rs:64:9
|
2  | use libc;
|     ---- the item libc is already imported here
...
64 |     use libc;
|         ^^^^
|
note: lint level defined here
--> src/lib.rs:13:9
|
13 | #![deny(unused)]
|         ^^^^^^
= note: #[deny(unused_imports)] implied by #[deny(unused)]
error: aborting due to previous error
error: Could not compile nix.
To learn more, run the command again with --verbose.

I don't see where this use at either line 2 or 64 was added recently so I assume this used to be valid syntax? Any pointers on compiling?

I haven't seen that before. What os and toolchain are you using?

Hi .. I am using:

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=18.04
DISTRIB_CODENAME=bionic
DISTRIB_DESCRIPTION="Ubuntu 18.04 LTS"

My toolchain is:

Default host: x86_64-unknown-linux-gnu
installed toolchains
stable-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu (default)
installed targets for active toolchain
wasm32-unknown-unknown
wasm32-unknown-wasi
x86_64-unknown-linux-gnu
active toolchain
nightly-x86_64-unknown-linux-gnu (default)
rustc 1.36.0-nightly (08bfe1612 2019-05-02)

I guess this is new behavior in the latest nightly compiler.  I'll fix it.

Hi .. yes, I think it is new. I see the same error reported recently for other projects:
https://github.com/paritytech/parity-ethereum/issues/10594
https://github.com/rust-lang/rust/issues/60484#issuecomment-489072191
Though I don't see a ton of reports nor the rust change where this would have broken this current code. Was looking for an attribute or compiler flag to ignore this but couldn't find one. For now I just commented out line 64. Thanks for the response.

Fixed by PR #1054


High resolution output with affordable memory

Great work!
I have two questions:

Resolution: if I want to render scenes with higher resolution (e.g. the solo video with 1080p), what should i do?
Memory: if i render high resolution, the cost of memory is too larger, are there some approach to save memory? (e.g. reduce cache feature?)

Very thanks!

You'd need to train the model with higher resolution. In our experiments, 540p takes around 40GB VRAM to train.
The consumption mainly comes from the convolutional U-Net and the fact that we render / supervise on the entire image. So it may be possible to reduce VRAM requirements by patch-based training, which would require some engineering effort.

You'd need to train the model with higher resolution. In our experiments, 540p takes around 40GB VRAM to train.
The consumption mainly comes from the convolutional U-Net and the fact that we render / supervise on the entire image. So it may be possible to reduce VRAM requirements by patch-based training, which would require some engineering effort.

Thanks for your reply!
Based on your reply, I have two more detailed questions:

"The consumption mainly comes from the convolutional U-Net": So the GPU costs mainly comes from the dynamic 2D foreground layers instead the nerf model for 3d background?  In my case, i mainly care about the background, could i reduce the costs of the foreground layer to work around it ?
"patch-based training": is there any code/repository i can refer to ?

You can try jointly training at a low resolution to obtain masks that include the objects + shadows. Then, upsample the mask to a high resolution and optimize only the NeRF model with high resolution images.
I don't have one in mind, the general idea is using not entire images when creating a training batch, but do a random crop. If you mainly care about the background, (1) may be an easier path.

You can try jointly training at a low resolution to obtain masks that include the objects + shadows. Then, upsample the mask to a high resolution and optimize only the NeRF model with high resolution images.
I don't have one in mind, the general idea is using not entire images when creating a training batch, but do a random crop. If you mainly care about the background, (1) may be an easier path.

Great idea! I will have a try.


add Milvus as Vector Store

I'm on it, it should be out next week (15/10/2023)

Oops, i forgot to close that one. The MR : https://github.com/theodo-group/LLPhant/pull/33


Add script engine metrics

Rationale
Measure usage of "alternate" R script engines (docker, rserve)
Related Pull Requests

Changes

Curious about opinion on using array vs map {"1":{}, "2":{}}


v2.0.0

2.0!
First real breaking change! tracking progress here. Most discussion should probably occur on the individual issue pages.
core package (eslint-module-utils)
Pull out src/core into eslint-module-utils, to be published separately for other plugins to leverage.
(copied from #234, closes #233)

[ ] bikeshed a better name? (if you're interested in this, open an issue and reference this plzzz)
[ ] document (README, etc)
[ ] publish as v1
[ ] add published version to plugin package dependencies

Coverage decreased (-1.7%) to 95.882% when pulling 3b2147822f2b0b729465c99cc30d11e5959f0173 on v2 into b2184f083a16c1e1a7f078def628865ac9a9dca9 on master.

Coverage increased (+0.1%) to 97.832% when pulling 6e3b530cdab64f2ae6bd82d99b160a5394420a1a on v2 into 0d98253ea3bc1c5998223cab7b1a095ba943b210 on master.

Coverage increased (+0.1%) to 97.838% when pulling d5af34e0f71e6694299616b06a85ee541d4c2217 on v2 into 0d98253ea3bc1c5998223cab7b1a095ba943b210 on master.

We should probably drop support for Node v4 (if it's not already done) and update the rules to the latest format (http://eslint.org/blog/2016/07/eslint-new-rule-format). Willing to do the work, but that should not appear on master before we're close to releasing.

I'm up for the new format, but it wasn't obvious to me that it's urgently necessary, since the older format is still supported and updating the rules should not be a noticeable change for users, right?
Also: isn't ESLint still supporting Node v4?

Sorry, meant dropping support for Node < v4.
The older format is still supported, and the new format is supported by the latest 2.x versions. I consider it a breaking change, even if I haven't encountered any problems with it. Most plugins I've seen that have made the jump released a major version for it, though they almost all also started using ES2015 features supported by Node v4.

Let's knock it out, then. Feel free to push updated rules to this branch.
And yeah, this branch is v4+. 😎

Coverage increased (+0.1%) to 97.933% when pulling ef1745adae82f300c1eb3009473616b2975bfac5 on v2 into 2b471c72f30c06a33ff8fd67b8c3ffc061bdda92 on master.

@jfmengels anything else you can think of that we should address before publishing this?
I'm thinking of publishing as v2.0.0-beta.0?

Coverage increased (+0.09%) to 97.922% when pulling 20079c643be9494886a7add1a5c572dddd9e1396 on v2 into 2b471c72f30c06a33ff8fd67b8c3ffc061bdda92 on master.

Ah, good job for the rule format.
Let's publish a beta v2 (don't forget --tag beta when publishing).
I think we could take a look at updating the recommended config, so that we can at least give a taste of the useful rules that don't need config. I'm thinking no-unresolved, no-extraneous-dependencies, etc...
But we can do that after the first beta version.

@jfmengels I'll try to merge this over the weekend or early next week.

Cool! :)

:tada:


Feature: Manage attributes (climate entities)

Hello, is it possible to manage also state_attr like built-in history card do with climate sensors (adding target temperature, current temperature and state on/off)?
Thank you

No, not directly. Attributes are not supported at this time, mostly due to performance reasons. You can however use template sensors to create entities out of your attributes and graph those instead.
For reference, it is better to ask questions like these on the HA forum thread here. Then the community can also answer. The github is more about bugs and FRs. Thanks !


Need a big example

Hi. Сan you show any complete project on the cockpit? Same like best practice.
I want to look:
how to organize customizable menu with hierarchy?
how to use global (for my app and cockpit admin panel) auth / ACL for separate anonimus, admin and other users? I want to add same controls on pages for admin.
where is right way to use cache, session?
that is block in $app object (can't find setter)?
how manage regions visibility by dynamic conditions?
how to add js callback on form success / error event?
how enable debug and view and trace errors?
how add a handler to collection item CRUD events?
and many other noob questions )
Now i try to find out all in cockpit / lime / mobgolite docs, but complete working example is match better for me.
Offtopic:

Have you plans to add multiple option for all fields type like in collection link?
 Mongodb had no problems this? How about queries with many conditions on many fields collection? Sorry, can't test it by myself now.

I want to understand how to build user friendly app not only for me, but for client too.
Thanks, and sorry for my english again.

+1


Partial sphere calculator added

Related Issue #1509
I have added a calculator for finding different parameters of partial sphere.

@sairish2001 Resolved merge conflicts


ga4 tracking for filter panel and section open/close

NOT READY FOR REVIEW

@alexbowen it's possible that some existing functionality using the data-ga4-expandable attribute might help you here, see: https://github.com/alphagov/govuk_publishing_components/blob/main/docs/analytics-ga4/trackers/ga4-event-tracker.md#advanced-use

@alexbowen it's possible that some existing functionality using the data-ga4-expandable attribute might help you here, see: https://github.com/alphagov/govuk_publishing_components/blob/main/docs/analytics-ga4/trackers/ga4-event-tracker.md#advanced-use

Yeah i was aware of this. Will have a look again at this now i am more familiar with ga4 stuff. was have trouble getting data-ga4-expandable to work before but am about to revisit


Jekyll::Tags::IncludeTagError on Bundler?

Hi there,
Would you please have a look at jekyll/jekyll/issues/3389?
Jekyll::Tags::IncludeTagError on Bundler might be caused by this plugin.
This error occurs in _includes/head_tag.html, included in _layouts/default.html, but it doesn't in one which is not included in it and inserted in _layout/default.html directly. Maybe a problem with directories?
Thanks!

Until 3.0.0 is officially supported, I probably won't have time to investigate into this too deeply. Will keep the issue upon until then.

I'm having a similar issue -- not sure what it means, exactly (Ruby isn't my forte).
Here's the error (works fine without the plugin):
/path-to-project/_plugins/jekyll-last-modified-at-master/lib/jekyll-last-modified-at.rb:10:in `<module:LastModifiedAt>': cannot load such file -- jekyll-last-modified-at/generator (LoadError)
	from /path-to-project/_plugins/jekyll-last-modified-at-master/lib/jekyll-last-modified-at.rb:3:in `<module:Jekyll>'
	from /path-to-project/_plugins/jekyll-last-modified-at-master/lib/jekyll-last-modified-at.rb:2:in `<top (required)>'
	from /usr/local/Cellar/ruby/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/Cellar/ruby/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:75:in `block (2 levels) in require_plugin_files'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:74:in `each'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:74:in `block in require_plugin_files'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:73:in `each'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:73:in `require_plugin_files'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:18:in `conscientious_require'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/site.rb:97:in `setup'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/site.rb:49:in `initialize'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/commands/build.rb:30:in `new'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/commands/build.rb:30:in `process'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/lib/jekyll/commands/build.rb:18:in `block (2 levels) in init_with_program'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `call'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `block in execute'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `each'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/command.rb:220:in `execute'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary/program.rb:42:in `go'
	from /usr/local/lib/ruby/gems/2.2.0/gems/mercenary-0.3.5/lib/mercenary.rb:19:in `program'
	from /usr/local/lib/ruby/gems/2.2.0/gems/jekyll-3.0.1/bin/jekyll:17:in `<top (required)>'
	from /usr/local/bin/jekyll:23:in `load'
	from /usr/local/bin/jekyll:23:in `<main>'


powerdns: fix 32-bit builds against glibc

Things done

Built on platform(s)

[ ] x86_64-linux
[x] i686-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This is for staging-next because (surprisingly) the support for 64-bit time_t on 32-bit platforms was only added in glibc 2.34 which still isn't in nixpkgs master.

The announcement says

full support requires a minimum kernel version of 5.1

but that's hopefully satisfied in most of the (32-bit) cases.

What will happen if I run this on a pre 5.1 kernel?

I have no idea beyond that announcement.  They say "full support", so I'd hope that with 5.0 you might not run into any problems even.


Add MySQL regression test

same as genSpringTest properties but the MySQL db connection

Add
set comment out or set false if you do not have a MySQL db set to test with
enabled=true
to properties file to make skipping easy when no MySQL DB is available.

checked in. See Sheets2DBTest.testWithgenSpringMySQLTest()


CompositeDrawable's default child comparer breaks if drawable under another parent is given

I wasted about an hour on this today (while trying to fix https://github.com/ppy/osu/pull/14939). Essentially what I did there is, I moved hitCircleOverlay a level down to be in another container (OverlayLayer), but then forgot to update a ChangeInternalChildDepth() call  to OverlayLayer.ChangeChildDepth().
You would expect that I should get this exception:
https://github.com/ppy/osu-framework/blob/5ed9c8ecdaae5102da96f9ce5f99d1ac37c2dc05/osu.Framework/Graphics/Containers/CompositeDrawable.cs#L591-L593
But I didn't, and the thing crashed deep in absolute transform sequences instead. About an hour of debugging later it turned out that IndexOfInternal() was returning 0 for hitCircleOverlay when called from LegacyMainCirclePiece,  even though it was not even a direct child of it anymore at that point.
This will happen if the CompositeDrawable's comparer doesn't check that both drawables are actually its child, which the default implementation does not, as it just reads the drawables' direct properties:
https://github.com/ppy/osu-framework/blob/5ed9c8ecdaae5102da96f9ce5f99d1ac37c2dc05/osu.Framework/Graphics/Containers/CompositeDrawable.cs#L365-L380
This also breaks things like ContainsInternal():
https://github.com/ppy/osu-framework/blob/5ed9c8ecdaae5102da96f9ce5f99d1ac37c2dc05/osu.Framework/Graphics/Containers/CompositeDrawable.cs#L442-L445
The obvious fix (to me) would be to swap out that broken check to a drawable.Parent != this check, but I'm not super sure if that's always valid...? (especially around things like proxying), which is why I'm opening this in the first place.

but I'm not super sure if that's always valid...? (especially around things like proxying)

It should be valid. The parent check sounds good to me.


Inconsistent target attribute in links/buttons

Description
Links for github and discord are always being opened in a new window, but there are two cases in which they are opened in the same window. That is for: "GitHub ⭐" button in the main navigation bar, and "discord" link in the footer.
Steps to Reproduce

1.Go to https://trypear.ai/
2. Click on the "GitHub ⭐" button in the main navigation menu
3. Click on the "Discord" link in the footer
Expected Behavior
The links should be opened in a new tab
Actual Behavior
It is opening in current window
Additional Information

Sprint Board Tag

I'll take care of this


typescript compilation errors "Cannot find name 'Promise'"

error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/ng1/directives/viewDirective.d.ts
(8,17): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/ng1/statebuilders/views.d.ts
(40,66): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/state/stateService.d.ts
(74,39): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/transition/transitionHook.d.ts
(15,1919): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/transition/transitionHook.d.ts
(21,40): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/transition/transitionHook.d.ts
(28,87): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/transition/interface.d.ts
(209,66): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/view/interface.d.ts
(48,13): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/resolve/resolveContext.d.ts
(76,57): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/resolve/resolvable.d.ts
(25,14): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/resolve/resolvable.d.ts
(66,62): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/state/interface.d.ts
(12,44): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/transition/transition.d.ts
(37,14): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/transition/transition.d.ts
(232,12): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/common/common.d.ts
(333,58): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/common/common.d.ts
(333,75): error TS2304: Cannot find name 'Promise'<project_root>/node_modules/angular-ui-router/commonjs/common/common.d.ts

package version 1.0.0-beta.1

Same for me with Webpack. Not happening if I use SystemJS or Bower

@ivanvoznyakovsky Fixed it by installing es6-shim typings

hm, I'll give it a try. I used es6-promise but that did not help.
thnx @micaelmbagira

try typings install --global dt~es6-shim
Let me know if it works or not.

ok, es6-promise works perfect. I just did it wrong. I have globally installed typings v.1 but in progect we still use 0.3 so format is different and it was not taken into consideration when installed w/ typings install. adding it as ambientDependencies entry did the trick.
thnx guys.

Can you close it ?

oh, sure :)

For what it's worth, bluebird promises also work:
> npm install tsd -g
> tsd install bluebird

and then in your script, add
/// <reference path="./typings/bluebird/bluebird.d.ts" />

I recommend using typescript 2.0+ and adding to your tsconfig.json:
{
  "compilerOptions": {
    "lib": ["es2015", "dom"]
  }
}

This tells typescript to use the .d.ts files from /node_modules/typescript/lib/lib.<library>.d.ts

Or just include in your package.json @types/es6-promise (In typescript 2.x)
javascript npm i --save @types/es6-promise

Small note: remember to add a reference to your index.d.ts in your index.ts (otherwise this issue can drive you crazy!)
/// <reference path="../typings/index.d.ts" />

I simply did -

typings install --global dt~es6-shim

then I added - "lib": ["es2015", "dom"] to compilerOptions . Now it builds fine without those errors.

node_modules/@types/selenium-webdriver/remote.d.ts(139,29): error TS2304: Cannot find name 'Map'.
i fix bt remove this in Map<string, string> | in  setEnvironment(env: Map<string, string> | {[name: string]: string} | null):

This worked for me

npm install typings --global

typings install --global dt~es6-shim

then I added - "lib": ["es2015", "dom"] to compilerOptions . Now it builds fine without those errors.

Also, make sure your using a recent version of typescript-> npm install -g typescript (-g denotes global install)

This worked for me

npm install typings --global
typings install --global dt~es6-shim
then I added - "lib": ["es2015", "dom"] to compilerOptions . Now it builds fine without those errors.

Only that halped me. Thanks!


'text' has not been registered,this can happen if*Metro(the local dev server)is run from the wrong folder......

大神，你好，我是自己新建的一个项目，然后通过demo上的文件进行打基础包和业务包，打包通过命令行打包和工具打包都试过
我想问下，基础包打包的时候是否需要设置入口文件打包？
我设置入口文件打包就不会下面的问题

自己打包就没成功，大神的demo可以实现跳转。
"react": "16.9.0",
"react-native": "0.61.5",
"react-native-dqgbui": "^1.0.11",
"react-native-event-bus": "^1.0.0",
"react-native-fs": "^2.16.6",
"react-native-gesture-handler": "^1.6.1",
"react-native-largelist": "^1.2.12",
"react-native-modal-datetime-picker": "^8.7.1",
"react-native-reanimated": "^1.9.0",
"react-native-safe-area-context": "^3.0.5",
"react-native-screens": "^2.8.0",
"react-native-secharts": "^1.7.0",
"react-native-splash-screen": "^3.2.0",
"react-native-storage": "^1.0.1",
"react-native-svg": "^12.1.0",
"react-native-webview": "^10.3.2",
"realm": "^6.0.5",

解决了，是命令行打业务包的问题


Override palette in config

Is your feature request related to a problem? Please describe.
The heuristics for defining the base palette do not work for my color schema.
Describe the solution you'd like
Being able to set the base palette as a config option
Describe alternatives you've considered
Hard-code my palette in hl.lua
local function make_palette()
  local bg     = Color.from_hex(get_bg("Normal") or "#151515")
  local fg     = Color.from_hex("#D0D0D0")
  local red    = Color.from_hex("#FF0086")
  local orange = Color.from_hex("#FD8900")
  local yellow = Color.from_hex("#ABA800")
  local green  = Color.from_hex("#00C918")
  local cyan   = Color.from_hex("#1FAAAA")
  local blue   = Color.from_hex("#3777E6")
  local purple = Color.from_hex("#AD00A1")

  local config = require("neogit.config")

  local bg_factor = vim.o.bg == "dark" and 1 or -1
...

Reasonable enough, sure.

Totally on board with this, somehow the get_fg("Error") in my setup is black instead of red! so Neogit looks weird like this

customizable palette or a configuration to use the default Neogit color palette would be a good idea.

Sorry, forgot to document it 😅
You can provide a table to config.highlight to overload any derived colors.
https://github.com/NeogitOrg/neogit/commit/daad0e1b7df8da2260455339f9615c640b19c27d


feature: implied odds calculator

A natural counterpart to probability inference would be odds inference, i.e., output a set of prices for a given set of probabilities (that sum to 1) for a target overround/margin, where the sum of the inverse output prices would = 1 + target overround/margin.
def test_calculate_implied_odds() -> None:
    odds = [2.6, 2.4, 4.3]
    margin = sum([1 / o for o in odds]) - 1
    implied_probabilities = shin.calculate_implied_probabilities(odds)
    res = shin.calculate_implied_odds(implied_probabilities, margin=margin)
    assert pytest.approx(res) == odds

Would you be interested in including something like this in the library?
R's implied package does this, and I've put together a very raw python implementation based from that which you can see diff'd against the typing branch here: https://github.com/peterschutt/shin/pull/1/files.

I think it's a useful addition. A couple of comments:

I would call the parameter overround rather than margin as I think the term "margin" is less well defined
I'd rather do the optimisation in rust

I think it's a useful addition. A couple of comments:
1. I would call the parameter overround rather than margin as I think the term "margin" is less well defined

2. I'd rather do the optimisation in rust

Yep, all that makes sense to me.
I'm not particularly comfortable in rust yet, but I'll have a crack at it for the experience. Would you like to keep a python implementation available too?

Note that for reasons I cannot currently recall, in one of my other packages I simply ported scipy.optimize._optimize._minimize_scalar_bounded by hand: https://github.com/mberk/kelly/blob/26164e45b102e7decead3f67df10fb119ee5bc61/src/lib.rs#L5-L146 (if I had to say, I think there might have been issues finding a Rust optimiser that was truly bounded)

I think easiest would be to copy that code straight in here then, would that suit? Where are you heading with the # TODO comments in that func?

Copying it in would be fine - certainly I don't personally currently have the capacity to pick an off the shelf alternative or confirm that the port is required if none of them provide exactly the required functionality
I haven't made any progress with those TODOs but I don't think they'll create any problems in this case

In the notes for the public function that wraps that scipy minimization func (ref):

Notes
-----
Finds a local minimizer of the scalar function `func` in the
interval x1 < xopt < x2 using Brent's method. (See `brent`
for auto-bracketing.)

... and the argmin crate has a Brent implementation, so I've gone ahead and used that.
I'll be spending more time on it as my rust is pretty raw, but feel free to comment on it at any point if you have the time (https://github.com/peterschutt/shin/pull/1/files) - but also no pressure.


[WIP]Metrics: KVStore usage metrics.

There are few cases when Kvstore is used (Ipcache, identity, and node)
this PR creates some metrics about the use of the KVStore usage and the
event types.
The reason to do in the watcher is because it's easy to count the types,
instead of monkey patching all the methods to increase the numbers.
The metrics add are the following:

kvstore_usage_total

Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

This change is

test-me-please

test-me-please

test-me-please


Fix model release bug.

There is a bug where population is not added to the model release upon creation. This change fixes that bug by 1) including the population field when a new model release is created, 2) populating the externalDataProviderId(population data provider) and externalPopulationId fields when converting from external to internal model release, and 3) populating the population field when converting from internal to external model release.

This change is


"Now Playing" Lyrics doesn't work.

🧐 Have you read the FAQ and viewed similar issues?

[X] I have read the FAQ
[X] I have checked older issues, open and closed
[X] I did a clean install of SpotX as described in the FAQ

Сountry of your account
Spain
What is your Spotify Plan?
Free
Do you have Spicetify installed?
Yes
ℹ Computer information
- Spotify: Spotify for macOS (Apple Silicon)<IP_ADDRESS>5.g920a7ada
- Windows:
- PowerShell:

📝 Description

The "Now Playing Button" just doesn't appear.

...
📸 Screenshots

you don't read the FAQ
lyrics should be in the right sidebar
also this is a repo for windows


[WIP] Allow binding to additional service instances.

This is incomplete but ready for feedback. One thing I haven't figured out is how to cram the updated policy templates into config-sample.json, but maybe that means the idea of keeping go-templates that generate json inside a json file wasn't a good idea in the first place 🤔

About templating: the problem is that the templates (which generate json) are themselves stored in a json file. So for example, at https://github.com/cloudfoundry-community/s3-broker/blob/master/config-sample.json#L80, we can't say
...
"Resource": {{resources ""}}
...

Because it wouldn't be valid json. If we used yaml for the config, we could say
iam_policy: |
  {
    ...
    "Resource": {{resources ""}
    ...
  }

because that's a valid inline string. Which is why I was thinking of changing the config to yaml, or allowing json and yaml for backwards compatibility. WDYT?

JSON is YAML so no harm in making the switch?

2k+ files changed.


add documentation TODO entries

It would be helpful when using some documentation generators. At least, there would be a complete list of functions/methods
BTW, I would suggest that merging the Wiki and the /docs is good to new users.

Changes Unknown when pulling 6596187d098f097c5b3e87836f59f13c6514e352 on hczhcz:master into ** on Chatie:master**.

Yes, I agree with you. The document should be generated from the source code, which will be fully synced with the latest version. We had some discussion about it, see #73 #252.
And I also agree with you about merging the Wiki API page to /docs. Because the Wiki page is created before we decide to use jsdoc to document Wechaty. Now we have the plan to do this, so I believe as soon as we merged the API docs to the jsdoc, we should redirect the Wiki page to /docs, which is good for new users.
I appreciate your PR for moving this forward: add a @todo is a good start. More contributions are very welcome. Thanks!


docs: change yaml syntax for pantheon integration

The Issue

https://github.com/ddev/ddev/issues/6599

the yaml example for setting up pantheon integration did not have the correct amount of spacing and this change corrects that
How This PR Solves The Issue
Updated the yaml syntax. While editing the yaml I also converted the markdown syntax for lists so it will be easier to maintain moving forward. Lastly, the letter sub bullets do not format as expected. I took the initiative to reformat as I felt like 1a and 1b should be there own numbered items.
Manual Testing Instructions
Automated Testing Overview

Release/Deployment Notes

whomp whomp. Look at that there are test to confirm the numbered lists are specified 1,2,3 ... 😬 .
Let me know if you'd like me to update those and what file they are declared.


Basic browsing for and view for App Engine

This contains the skeleton for browsing and views App Engine services/versions/instances.

Thanks! PTAL.

Small thing, but LGTM


added license and notice

This is a proposal for the license.
I am not sure about the copyright holder.

APL seems to be a decent choice.
It's a bit tricky though - in principle need consent from everyone who contributed to AMUSE code to implement license...
I guess up to now the "implied" license was "completely free" / MIT style.
One thing to consider: some community codes in AMUSE are GPL codes. I'm not completely sure APL and GPL are compatible licenses to the point that the interfaces to those codes can be APL. They may need to be under a different license. This could be very tricky stuff, and probably isn't relevant in practice, but still may need to be considered...

good points; one of the objective of this pull is to get input from everybody involved!
about the GPL: I think indeed we can include the GPL codes, but the interface on the "native" (so fortran/C/ whatever) side needs to be in the same licence (or compatible) since these build the executable which link to the (GPL) code; this is no problem;

Yes, I think that's right. We're communicating through MPI with those codes, so essentially it's a different thing.
But we'll need to be explicit about the licenses of those interfaces.

Let me just state here that I'm fine with using the Apache license v2.0 where it comes to my contributions.

Maybe now (before the book is released) is a good time to add the license?

This is probably something we should discuss at the upcoming AMUSE meeting...

Sorry, but I don’t see the subject.  What is the issue here?
Stephen L. W. McMillan
Department Head and Professor of Physics, Drexel University
+1<PHONE_NUMBER>/2723, FAX +1<PHONE_NUMBER><EMAIL_ADDRESS>
On Mar 9, 2018, at 8:59 PM, Steven Rieder<EMAIL_ADDRESS>wrote:
This is probably something we should discuss at the upcoming AMUSE meeting...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub https://github.com/amusecode/amuse/pull/144#issuecomment-371993385, or mute the thread https://github.com/notifications/unsubscribe-auth/AB5Q7t59Qwex_llXCrG3Vc-Mt-6r62Phks5tczOAgaJpZM4N8bE5.

Defining a license for AMUSE. Inti suggested we use the Apache Public License. It would be good to have a decision on this.

Hi Steven,
Sounds like something to discuss in April..
Cheers,
Simon
On 03/10/2018 03:14 AM, Steven Rieder wrote:

Defining a license for AMUSE. Inti suggested we use the Apache Public
License. It would be good to have a decision on this.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/amusecode/amuse/pull/144#issuecomment-371994454,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AETyzP2CSsW4dAKGYO_s4dXpKrj04BcKks5tczccgaJpZM4N8bE5.

Yes, we will use MIT or BSD license! this will work with the codes too
as these can have their own license
Arjen
On 13-04-18 16:20, ipelupessy wrote:

did we reach a conclusion on this one?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/amusecode/amuse/pull/144#issuecomment-381150340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAoHjh0fib908KTwCrjVX5xWgBr0gBm5ks5toLQagaJpZM4N8bE5.

@arjenve: shall we merge this or do you want to change the licence (Apache v2, almost the same as MIT)

Please make sure to note where code may fall under another license (i.e. included community codes)...

@rieder : that is in the NOTICE - can you check if you agree?

Looks fine to me, though maybe it would be good to add an exception clause in case there is no explicit license for a community code ('all rights reserved' or something?).

@arjenve I am inclined to merge now...


Add release please

Package / App
Other
Description
Add Release Please GitHub Action.

[ ] Add release-type variable which can be node or expo.
[ ] Add docs for writing commit messages (Conventional Commits).

Alternatives
No response
Other info
No response

@dansvend can this be closed as not planned ?

@HaakonSvane Yes, sir 🫡


Converting circular structure to JSON

[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ X] support request

Hi,
I'm using formly inside a "Matdialog", and I found something strange.
When I open the dialog for the second time the mandatory fields are already "touched",
and if i try to stringify the form fields (the second time) I get this error
ERROR
Error: Converting circular structure to JSON
I can not understand where I'm wrong.
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/angular-3rd8sk-ytzwep

passed before it gets initialized by formly-form component:

cloning the fields before it gets initialized by formly-form component would solve the issue:
  constructor(@Inject(MAT_DIALOG_DATA) public data: any) {
    this.model = data.item;
    this.fields = <FormlyFieldConfig[]>JSON.parse(JSON.stringify(data.fields));
  }

Well, I'm looking for a fix since I think this can be solved from our side.

try with 5.0.0-rc.4 which solve the issue, you don't have to use JSON.stringify either.

Thank you very much!

this fixed was reverted in 5.0.0-rc.5 in favor of https://github.com/ngx-formly/ngx-formly/issues/1429#issuecomment-465186967