Fixed SpnegoWebflowConfigurer to use the evaluateClientRequest as documented

Changes include the following:

Added conditional logic when mixedModeAuthentication is false to use
startSpnegoAuthenticate flow.  Otherwise it will use
evaluateClientRequest.
Modified createEvaluateSpnegoClientAction to be equivlent to the other
Spnego actions on failure which is
CasWebflowConstants.STATE_ID_VIEW_LOGIN_FORM

Since this change fixes the documentation listed out here, this change re-enables the functionality which may cause unexpected results to those using the defaults.  Some of those properties are the following:

cas.authn.spnego.hostNameClientActionStrategy
cas.authn.spnego.hostNamePatternString
cas.authn.spnego.mixedModeAuthentication
cas.authn.spnego.ipsToCheckPattern

I have done my best to ensure as few changes are affected as possible.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Anthony Lofton seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you. Just in case, you may want to post the same patch to the 5.3.x branch as well specially if you wish to use this fix in the next 5.3 patch release.


[DO NOT MERGE] Reimplement @test_rules and @test_marginalrules

This PR reimplements the @test_rules and @test_marginalrules. These both are quite old and were hacked around, especially the with_float_conversions = true option. This PR improves the code, adds tests and the documentation.
with_float_conversions has been renamed to check_type_promotion, technically breaking, but it is not a part of the public API, but rather our test infrastructure.
The logic changed a little bit as well, thus I needed to tweak some rules to make the new code work. I'm actually quite confused why did it work before, but IMO new code is more robust. I also found one small bug! This PR also introduces a new family of promotion functions promote_paramfloattype in addition to promote_samplefloattype. Apparently in Distributions.jl the type of a sample is not necessarily aligns with the type of distribution parameters.
Some un-used functions were removed.
Sorry for long list of changes, this is mainly due to renaming of the with_float_conversions option. The main files to check are: distributions.jl, rule.jl and their tests repsectively.

@bvdmitri should we re-run CI?

The tests have been fixed and the PR is ready. Please wait for the CI and if everything is fine we can merge IMO.


susie_rss with imputed ss WIP

after the imputation, following error occurs:
rpy2.rinterface_lib.embedded.RRuntimeError: Error in susie_suff_stat(XtX = R, Xty = z, n = 2, yty = 1, scaled_prior_variance = prior_variance,  :
  The estimated prior variance is unreasonably large.
             This is usually caused by mismatch between the summary statistics and the LD matrix.
             Please check the input.

However, slalom for the input of the imputed data only show 12 outliers
	lead_pip_variant	n_total	n_r2	n_dentist_s_outlier	fraction	max_pip
0	chr17_63492371_G_A	19212	39	12	0.307692	0.03586

Needs to verify further.

Make imputation of ss optional in susieRSS
Add kringe plot to susieRSS
coloc maded but not tested.


Update readme.md

Add Pyrometer to baking tools

Thanks for the commit! Sorry for the delay in merging.


What is missing to get a release

Your readme says still under development, what help do you need to finish?

Probably a number of generated tests and some reviews. But I am currently busy with study and thus not available for a long time. I would be very grateful if someone could help to do that.

OK. I think I can invest some time in the shorter term. I will keep you posted and hopefully be able to come up with some bug reports and PRs.


Node core integration

@TimothyGu have you ever considered working towards having this project in Node core?
It seems like this would be a huge win having a global.fetch available in NodeJS.

I guess the discussion has run its course in https://github.com/nodejs/node/issues/19393. I'm happy to go with any decision made by Node.js core.


Allow redirecting 404 errors to 418 (teapot error)

Feature Description
would be great as a joke function where, instead of an 404 page, you can have a 418 page that says something along the lines of being a teapot. It could be enabled by setting a value in app.ini as true so you wouldn't get this unless you're looking for it.
Screenshots
No response

*but why meme*

but why meme

because i thought it'd be funny

bump

ok nobody is going to ever do this


Unable to delete a cluster from https://portal.armo.cloud/

Couldn't find an option to delete cluster from https://portal.armo.cloud/ .

hi @calvinroyston
Would you like to delete a specific cluster, or all your data?

Hi @BenHirschbergCa ,
The cmd "kubescape scan framework nsa" when used without --results-locally=True posts the results to Armo management portal by default . When I log in and check its there but I was unable to find any option to delete that cluster/data recorded . Is there an option to delete a cluster and recorded data ? It would be nice to have that option .

Ok, we are on it!

@calvinroyston Fixed :)

Navigate to the SaaS version, click on the settings icon on the top right of the page->

Select the cluster you want to delete and press the delete


Problem: Cannot use Git ssh:// protocol inside docker semaphoreui

Issue
We are running the latest official semaphoreui docker image with a standard configuration. Everything worked while we used a https:// connection for checking out of our git server.
Now we switched to a git repository that we can only access using the ssh: git protocol - the git part works as we can check cloning the repo to our client machines.
However, when starting a Task from within semaphoreui the following output is created:
...
5:16:27 PM Run TaskRunner with template: Print Info
5:16:27 PM Preparing: 149
5:16:27 PM Cloning Repository ssh://git@git-server:22/srv/git/infrastructure.git
5:16:27 PM Cloning into 'repository_1_5'...
5:16:27 PM Host key verification failed.
5:16:27 PM fatal: Could not read from remote repository.
...

Impact
Ansible (task execution), Other
Installation method
Docker
Database
MySQL
Browser
No response
Semaphore Version
2.10.43-e43af6b-173227814
Ansible Version
No response
Logs & errors
No response
Manual installation - system information
No response
Configuration
No response
Additional information
No response

You can add a volume to your sempahore Container with a known_hosts file.
In container path is /home/semaphore/.ssh/known_hosts


Add environment directory to jsonnet path before eval

Solves issue with not being able to find globals.libsonnet by adding the environment directory to the jsonnet path.
Signed-off-by: bryanl<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 1136

5 of 5 (100.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.007%) to 70.852%

Files with Coverage Reduction
New Missed Lines
%

pkg/util/strings/rand.go
2
84.62%

Totals

Change from base Build 1131:
-0.007%

Covered Lines:
11444

Relevant Lines:
16152

💛  - Coveralls


svelte/require-store-reactive-access

Motivation
This is coming from https://github.com/sveltejs/svelte/issues/7984.
Description
Disallow to render store itself. Need to use $ prefix or get function.
This rule is fixable.

{storeValue}

{$storeValue}

Examples

{$storeValue}

{get(storeValue)}

{storeValue}

Additional comments
Svelte can define your own store. So we need to support it at some point. (But I think it's difficult. Is there any way to provide it to TypeScript users?)
https://svelte.dev/docs#component-format-script-4-prefix-stores-with-$-to-access-their-values-store-contract

Thank you for suggesting the rule!
That rule sounds good to me!
I still don't know how to type check if it's a store or not, but I think we can get TypeScript type information from a node like this:
const tools = getTypeScriptTools(context)
if (!tools) {
   return {}
}
const { service, ts } = tools
const checker = service.program.getTypeChecker()
// ...
const tsNode = services.esTreeNodeToTSNodeMap.get(node);
const type = checker.getTypeAtLocation(tsNode);

The following code is a deprecated rule the other day, but it may be helpful.
https://github.com/ota-meshi/eslint-plugin-svelte/blob/7be12eb2d1752902ab495245ccfe7ba7c8516cc8/src/rules/%40typescript-eslint/no-unnecessary-condition.ts#L160-L166

typescript-eslint's no-floating-promises rule seems to identify Promise by checking the then method.
I think we might be able to identify the store instance by checking the subscribe method in the same way.
https://github.com/typescript-eslint/typescript-eslint/blob/54141946ac2a25799bb622d47d9a3fd1fb316ef9/packages/eslint-plugin/src/rules/no-floating-promises.ts#L184-L206

We should also check references from extractStoreReferences in case users don't use typescript.

I'll try to work on implementing this rule.


Setup typescript and apply typescript-related fixes

Mostly JSDoc updates for compatibility with typescript

Overall looks solid. Will be a big win!
A nice-to-have would be giving some of the Object types more concrete types, either with literals { a: number, b: string}, or interfaces.
Will let someone more familiar with the code give the final ok, unless you'd like me to take a deeper pass.


CSV export of has_many association fails with searchable set to an array

Describe the bug
As above, when a has_many association is included as a field in the list view, and searchable is set to an array (not true) export to CSV fails with:
ActiveRecord::StatementInvalid in RailsAdmin::MainController#export
PG::UndefinedTable: ERROR: missing FROM-clause entry for table

This happens only when the list has been filtered by the has_many association, and we are "Exporting found" records. It does not happen if we do not filter by the association and export all records.
Here is the log output:
Processing by RailsAdmin::MainController#export as HTML
  Parameters: {"authenticity_token"=>"fake", "send_data"=>"true", "all"=>"true", "schema"=>{"only"=>["id", "created_at", "updated_at", "pinned"], "include"=>{"project"=>{"only"=>["id", "full_name", "preferred_name", "pronoun", "slug", "date_of_birth", "date_of_passing", "living", "prefix", "obituary", "welcome_message", "privacy_mode", "role", "created_at", "updated_at", "status", "webcrawler_status", "notification_status", "subdomain", "legacy_id", "embedding_enabled", "plan", "obituary_url"], "methods"=>["cover_image"]}, "group"=>{"only"=>["id", "name", "title", "slug", "description", "group_website_url", "cta_url", "cta_text", "primary_color", "secondary_color", "button_color", "reflections_enabled", "privacy_mode", "created_at", "updated_at", "subgroup_name", "contributions_enabled", "member_name", "stay_connected_enabled", "reporting_enabled", "memory_card_color", "banner_columns", "memories_enabled", "experimental_stay_connected_jotform_id"], "methods"=>["logo", "project_image", "banner_image", "banner_image_mobile", "themes"]}, "group_categories"=>{"only"=>["id", "name", "created_at", "updated_at", "priority", "enabled", "subcategory_limit", "featured", "featured_color", "link", "private_name"], "methods"=>["featured_icon"]}, "group_project_attributes"=>{"only"=>["id", "key", "value", "notes", "created_at", "updated_at"]}}}, "csv_options"=>{"encoding_to"=>"", "generator"=>{"col_sep"=>","}}, "return_to"=>"http://localhost:3000/admin/group_project?f%5Bgroup_project_attributes%5D%5B80623%5D%5Bv%5D=asdf&query=", "csv"=>"", "f"=>{"group_project_attributes"=>{"80623"=>{"v"=>"asdf"}}}, "model_name"=>"group_project"}

ERROR:  missing FROM-clause entry for table "group_project_attributes" at character 2396

STATEMENT:  SELECT "group_projects"."id" AS t0_r0, "group_projects"."project_id" AS t0_r1, "group_projects"."group_id" AS t0_r2, "group_projects"."created_at" AS t0_r3, "group_projects"."updated_at" AS t0_r4, "group_projects"."pinned" AS t0_r5, "projects"."id" AS t1_r0, "projects"."account_id" AS t1_r1, "projects"."creator_id" AS t1_r2, "projects"."full_name" AS t1_r3, "projects"."preferred_name" AS t1_r4, "projects"."pronoun" AS t1_r5, "projects"."slug" AS t1_r6, "projects"."cover_image_data" AS t1_r7, "projects"."date_of_birth" AS t1_r8, "projects"."date_of_passing" AS t1_r9, "projects"."living" AS t1_r10, "projects"."prefix" AS t1_r11, "projects"."obituary" AS t1_r12, "projects"."welcome_message" AS t1_r13, "projects"."privacy_mode" AS t1_r14, "projects"."role" AS t1_r15, "projects"."created_at" AS t1_r16, "projects"."updated_at" AS t1_r17, "projects"."status" AS t1_r18, "projects"."webcrawler_status" AS t1_r19, "projects"."notification_status" AS t1_r20, "projects"."subdomain" AS t1_r21, "projects"."legacy_id" AS t1_r22, "projects"."embedding_enabled" AS t1_r23, "projects"."plan" AS t1_r24, "projects"."obituary_url" AS t1_r25, "groups"."id" AS t2_r0, "groups"."title" AS t2_r1, "groups"."slug" AS t2_r2, "groups"."description" AS t2_r3, "groups"."group_website_url" AS t2_r4, "groups"."logo_data" AS t2_r5, "groups"."cta_url" AS t2_r6, "groups"."cta_text" AS t2_r7, "groups"."primary_color" AS t2_r8, "groups"."secondary_color" AS t2_r9, "groups"."button_color" AS t2_r10, "groups"."reflections_enabled" AS t2_r11, "groups"."privacy_mode" AS t2_r12, "groups"."created_at" AS t2_r13, "groups"."updated_at" AS t2_r14, "groups"."subgroup_name" AS t2_r15, "groups"."contributions_enabled" AS t2_r16, "groups"."member_name" AS t2_r17, "groups"."name" AS t2_r18, "groups"."themes" AS t2_r19, "groups"."project_image_data" AS t2_r20, "groups"."stay_connected_enabled" AS t2_r21, "groups"."banner_image_data" AS t2_r22, "groups"."banner_image_mobile_data" AS t2_r23, "groups"."reporting_enabled" AS t2_r24, "groups"."memory_card_color" AS t2_r25, "groups"."banner_columns" AS t2_r26, "groups"."memories_enabled" AS t2_r27, "groups"."experimental_stay_connected_jotform_id" AS t2_r28 FROM "group_projects" LEFT OUTER JOIN "projects" ON "projects"."status" != $1 AND "projects"."id" = "group_projects"."project_id" LEFT OUTER JOIN "groups" ON "groups"."id" = "group_projects"."group_id" WHERE ((group_project_attributes.key ILIKE '%asdf%') OR (group_project_attributes.value ILIKE '%asdf%') OR (group_project_attributes.notes ILIKE '%asdf%')) ORDER BY "group_projects"."id" asc

Reproduction steps
Here is the relevant model config. Again, if I change it to searchable true, there is no problem.
class GroupProject < ApplicationRecord
  belongs_to :group, inverse_of: :group_projects
  belongs_to :project, inverse_of: :group_projects

  has_and_belongs_to_many :group_categories
  has_many :group_project_attributes, dependent: :destroy, inverse_of: :group_project

  default_scope { order(pinned: :desc, created_at: :desc) }

  accepts_nested_attributes_for :group_categories, :project

  rails_admin do
    object_label_method :rails_admin_label

    list do
      include_all_fields

      field :group_project_attributes do
        label "Attributes"
        pretty_value do
          value.map do |attribute|
            # Generate the URL to the GroupProjectAttributes record
            attribute_link = bindings[:view].show_path(model_name: "group_project_attribute", id: attribute.id)

            # Escape the key, value, and notes to prevent XSS
            safe_key = ERB::Util.html_escape(attribute.key)
            safe_value = ERB::Util.html_escape(attribute.value)
            safe_notes = attribute.notes ? "(#{ERB::Util.html_escape(attribute.notes)})" : ""

            # Create the link for each attribute's key and display the value and notes
            bindings[:view].link_to("#{safe_key}: #{safe_value} #{safe_notes}", attribute_link).html_safe
          end.join(" <br> ").html_safe
        end
        filterable true
        queryable true
        searchable [:key, :value, :notes]
      end
    end
  end

  def rails_admin_label
    "#{project.full_name} <GroupProject \##{id}>"
  end
end

class GroupProjectAttribute < ApplicationRecord
  belongs_to :group_project, inverse_of: :group_project_attributes
end

Expected behavior
There should be a SQL JOIN happening.
Additional context

rails version: <IP_ADDRESS>
rails_admin version: 3.2.0
rails_admin npm package version: 3.2.0

We are seeing the same thing! In our case, searchable is set to a jsonb query:
searchable "groupings.document ->> 'name'"


Update to 2017.1.3

Should be merged after #29 so that there are tags for 2017.1.2 and 2017.1.3.

Thanks for merging these @sjoerdmulder! 👍


[Snyk] Upgrade @types/gh-pages from 3.2.2 to 3.2.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @types/gh-pages from 3.2.2 to 3.2.3.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a month ago, on 2023-10-18.

Release notes

Package name: @types/gh-pages

3.2.3 - 2023-10-18

3.2.2 - 2023-09-20

    from @types/gh-pages GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (4e135ba) 74.74% compared to head (3ca4d93) 74.74%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #51   +/-   ##
=======================================
  Coverage   74.74%   74.74%
=======================================
  Files           8        8
  Lines        1964     1964
  Branches      204      204
=======================================
  Hits         1468     1468
  Misses        489      489
  Partials        7        7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Depend on Textual >= 0.41

memray tree already uses features that require this version (TextArea was introduced in Textual 0.38, and TextArea.text gained a setter in Textual 0.41).
Correct our package metadata to accurately reflect our supported versions of the Textual dependency.
See #518

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (853df63) 92.39% compared to head (7781a75) 92.32%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #522      +/-   ##
==========================================
- Coverage   92.39%   92.32%   -0.08%
==========================================
  Files          91       91
  Lines       11270    11270
  Branches     1571     1571
==========================================
- Hits        10413    10405       -8
- Misses        852      860       +8
  Partials        5        5

Flag
Coverage Δ

cpp
85.66% <ø> (-0.22%)
:arrow_down:

python_and_cython
95.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Change old paypal links to new Nisei patreon links

Also reindents the two touched files.

 P1: Change donation link to NRDB


Validate and save upon field change

Validate and save upon field change as opposed to extension load.
This was disrupting my editorial team's workflow.
After updating the schema, once the extension was in view then the entry status would change from publish to pending changes.
This proposed file change will only change the entry status upon a field being changed rather than the extension being loaded.

@philmirez you should be all set. Since you probably installed this using srcdoc you'll want to update that source inside the Contentful web app.


Add methods to remove reaction by user id

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
Adds methods removeReaction(String userId) and removeReaction(long userId) so bots can remove reactions without fetching a User object.

Why not removeReaction(User.fromId(id))?

Huh, looks like User.fromId(id) is a relatively new addition that works for this situation. Closing...


New functions for MathProgBase

Prepares Gurobi to be used in MathProgBase with the functions:
copy,  delconstrs!, delvars!, chgcoeffs
Adds tests

Hard to review before seeing the corresponding PR to MathProgBase

Actually I have no idea about what to do with callbacks, since I have not
been dealing with them, maybe just check if callbacks exist and tell the
user that they were not copied?
I see the copy part mostly for LP, since MIPs tend to be slow to solve and
thus rebuilding from scratch would not be a big bottleneck, but its hard to
tell...
I am finishing the MathProgBase PR, but I getting used to some few
differences in GLPK
On Fri, Dec 23, 2016 at 6:04 PM, Miles Lubin<EMAIL_ADDRESS>wrote:

@mlubin commented on this pull request.
In src/GurobiSolverInterface.jl
https://github.com/JuliaOpt/Gurobi.jl/pull/78#pullrequestreview-14335506
:

@@ -35,11 +35,11 @@ function copy(m::GurobiMathProgModel)
copy(m.obj),
copy(m.lb),
copy(m.ub),

                          deepcopy(lazycb),

                          deepcopy(cutcb),

                          deepcopy(heuristiccb),

                          deepcopy(infocb),

                          deepcopy(options))

                          deepcopy(m.lazycb),

It's a bit unclear if it's safe to copy callbacks. How would this work if
the callbacks hold a reference to the old model, for example?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaOpt/Gurobi.jl/pull/78#pullrequestreview-14335506,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANOj8f7eKPfNlBXDt5cwItWhQqfoW6F1ks5rLClCgaJpZM4LUhaM
.

Can we move with this one?

Yes, but there's a merge conflict thanks to https://github.com/JuliaOpt/Gurobi.jl/pull/89


Resize card height

Description
Are there any plans to add a setting where you can adjust card height and icon size?
This is similar to https://github.com/ajnart/homarr/issues/216, but that was for card width only. The ability to control card height (and/or icon size) would help reduce the amount of vertical scrolling users need to do
Priority
Low (Nice-to-have)

Description
Are there any plans to add a setting where you can adjust card height and icon size?
This is similar to https://github.com/ajnart/homarr/issues/216, but that was for card width only. The ability to control card height (and/or icon size) would help reduce the amount of vertical scrolling users need to do
Priority
Low (Nice-to-have)

Hey ! This is a little bit hard because it would mean the icon can be misaligned. Misaligned icons would work but i would need to change the basic format of the app from a grid to a masonry.
Do you have any recommendations on the size you'd want the apps to get ? (like 80% of current value, …)

maybe offering a range between 60-100% of current value would be useful
im imagining this as a global setting that would apply to all app cards simultaneously (not setting the height differently for individual apps), so if they're all vertically resized to the same %, would there still be misalignment? (I'm not familiar with the UI library you're using, so not sure how it controls things like icon sizing/positioning)

This really would be a great addition for those of us with a ton of apps and to help fit more in on devices with limited screen size (phones, tablets, etc).  Anything that could be done to allow a higher density of apps on the screen would get a big 👍 from me.

Hi, I agree with other users, that would be great !
Et merci pour ton boulot c'est au top @ajnart 🥇


Constraint version bumps of importlib metadata

Description
Constraint allowed version bumps of importlib-metadata because importlib-metadata>=4.9.0 requires python>=3.7
Self Check:

[ ] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[x] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

I will add an item to: https://github.com/inmanta/infra-tickets/issues/121

Processing this pull request

Merged into branches master in 7da89febeca931c2adac939d299fe0d32984c405


Using @group causes 404 on load

Describe the bug
When I specify @group on components, the components are successfully split in the left nav, but the initial page that loads on vuese serve --open becomes:
Server running at http://<IP_ADDRESS>:5000/#undefined

Even if I am able to get a valid path opened, if I click on the "Components" header to return me to the index (e.g., http://<IP_ADDRESS>:5000/#) I still get the following in the browser in the right pane of the page:
404
The requested path could not be found
Note that if I click on any of the component items themselves that the correct markup loads and renders. But that initial load, or if the Components header is clicked, results in a 404.
To Reproduce
Steps to reproduce the behavior:

In your Vue component descriptor, add @group Controls to any component. I was able to reproduce this with any control, whether it was first alphabetically or not. I also tried adding a group to all controls with the same result.
Generate the docs with vuese gen
Open the docs with vuese serve --open

Note that steps 2 and 3 were performed using npm with those scripts defined in the package.json.
Expected behavior
Either first component is opened in the browser, with any clicks on the Components header also redirecting to that component.
Or an index page can be defined in some way to be used as the Overview for the docs.
Screenshots

Environmental information:
System:
OS: macOS 11.4
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Binaries:
Node: 14.4.0 - ~/.nvm/versions/node/v14.4.0/bin/node
npm: 6.14.5 - ~/.nvm/versions/node/v14.4.0/bin/npm
Browsers:
Chrome: 91.0.4472.114
Firefox: 88.0.1
Safari: 14.1.1

I did find that adding one line to the reducer in getFirstPath in the index.js of the @vuese/cli file fixed the issue for me (though it's not clear to me why the reducer fails otherwise):
function getFirstPath(config) {
    const entryPath = path.resolve(`${config.outDir}/index.html`);
    const reg = /(?:[\s\S]+)sidebar\:([\s\S]+)\}\)/;
    const entrySourceStr = fs.readFileSync(entryPath, 'utf-8') || '';
    try {
        const regRes = entrySourceStr.match(reg)[1];
        const routesConfig = new Function(`return ${regRes}`)();
        const firstRoute = routesConfig.reduce((p, v, i) => {
            if (i === 0) {
                const { links } = v;
                return links[0] && links[0].link
            }
            return p;
        }, '');
        // Read the first preview configuration injected when opening the browser.
        return `#${firstRoute}`;
    }
    catch (e) {
        // If there's an error, follow the previous logic.
        return '';
    }
}

The only change here is that I return p when the index is not 0.

+1 Same issue


ci: add Maven setting file when computing project version

Issue
https://github.com/gravitee-io/issues/issues/XXXXX
Description
A small description of what you did in that PR.
Additional context

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here


Add Magento 2 extension

Information

Name:
Link:
Description:

Checklist:

[ ] I have read the contribution guidelines thoroughly.
[ ] I ensure my submission follows each and every point of it.

Hey @peterjaap, thank you so much for your contribution! Let's merge that :)


[web] Adds link to the Clarity Plunker Templage`

Signed-off-by: Matt Hippely<EMAIL_ADDRESS>

@scott - I didn't realize I was also committing changes to the node_module package.json files. Do I need to resubmit without them?


[bluebird_v3.x.x] Promisable type option to Promise/input param on some methods

Fixes #1082
Changes mirror the current API at http://bluebirdjs.com/docs/api-reference.html
I'm no pro with this lib, so I'll reach out to the issue's OP, and have him check that my changes are accurate.

Updated specs to match the changes, this should now be good to merge.


Very Helpful - request for more

This is very helpful and a nice, concise discussion. Please create similar documentation for all of the Defender products

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

ID: 6e32e2ed-5ae3-d1c6-6a5b-25b225647dbf
Version Independent ID: 3f5d0ae5-a9b0-f755-c2f6-c38c75e3e485
Content: Step-by-step threat protection stack in Microsoft Defender for Office 365 - Office 365
Content Source: microsoft-365/security/office-365-security/protection-stack-microsoft-defender-for-office365.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

Good idea. But not directly actionable as an item to track in a GitHub issue.

@DeanGross Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


README should point to new post-re-organisation JSON locations

README currently has:
Each problem can have a file <problem>.json
Rendered, this just shows up as "Each problem can have a file .json", but that's a separate issue.
The more important thing is that this should get changed to match the location after #351 moved to per-exercise directories.

Good catch, fixing that now.


ERROR  `/sw.js' not found

When running jekyll serve I get the following error everytime I load a page in the browser
[2018-07-06 09:42:43] INFO  WEBrick::HTTPServer#start: pid=7146 port=4000
  Server running... press ctrl-c to stop.
[2018-07-06 09:43:03] DEBUG accept: <IP_ADDRESS>:45172
[2018-07-06 09:43:03] DEBUG Jekyll::Commands::Serve::Servlet is invoked.
[2018-07-06 09:43:03] DEBUG Jekyll::Commands::Serve::Servlet is invoked.
[2018-07-06 09:43:03] DEBUG Jekyll::Commands::Serve::Servlet is invoked.
[2018-07-06 09:43:04] DEBUG Jekyll::Commands::Serve::Servlet is invoked.
[2018-07-06 09:43:04] ERROR `/sw.js' not found.
[2018-07-06 09:43:04] DEBUG close: <IP_ADDRESS>:45172

Any idea where that comes from? Is it even related to minima?

minima does not declare any service worker at the moment. This is probably because you have another local project that has declared a service worker. You can unregister the service worker in the browser dev tools or ignore this error.

Alright, I choose to ignore it then :D
Thank you


WIP: Auto-fixer for pyproject.toml

add missing pyproject.toml
add or update [build-system] map
detect additional build requirements by parsing "setup.py"
remove unwanted build requirements such as cmake

Fixes: #260
Fixes: #274

Replaced by #407


Include mac_meth and mac_lib in the FIPS provider

Seems like these were missed somewhere along the MAC movements.

Merged, thanks.


[skip CI] try fix trigger for unlabeled PR pings

@pmeier I got pinged incorrectly. I assume that's what you try to fix?

@datumbox Yes. Could you link the PR where you were pinged? I can't see anything on #4011.

It's the 4011 but I got it on the notifications panel.

This probably happens for failed workflows that follow a push event on master. If this is done, the behavior should also go away.

It's complaining because:
Run DATA=$(python .github/foo.py ea6be1240e5d83888b8d7fb08062496fa0c0c711)
python: can't open file '/home/runner/work/vision/vision/.github/foo.py': [Errno 2] No such file or directory
Error: Process completed with exit code 2.

I think this got fixed earlier in a different PR.


[capture-promotion] Update capture-promotion for basic semantic-sil usage.

[capture-promotion] Update capture-promotion for basic semantic-sil usage.
In this commit I update capture-promotion for very basic usage and verify this
by adding a version of one of the tests with ownership (and thus the verifier)
enabled.
I am going to move over the rest of the tests in subsequent commits with changes
necessary for capture promotion to work with them.
rdar://29870610

Ah... this looks better.

@swift-ci smoke test and merge

mmm... and looks like when I ran my last test locally I typed in capture_propagation as well, so this is going to fail.

I wish those passes had names that were further apart.

I did this in a different PR.


Events are incorrectly placed when their duration is less than step

Do you want to request a feature or report a bug?
Bug
What's the current behavior?
Events are incorrectly placed when their duration is less than the step property. In the image below you can see that the event starts at 16.00, but it's placed at the 15-16 slot (rather than on the 16-17 slot). You can see the behavior in this CodeSandbox as well.

This was bug introduced by https://github.com/intljusticemission/react-big-calendar/pull/1204. I can see that @TrySound tried to fix it first by reverting the behavior added by the PR above with https://github.com/intljusticemission/react-big-calendar/commit/17b6cfdf7c71e6b9117e9d552231ab17f8ff081c but then added it back (with some adjustments) with https://github.com/intljusticemission/react-big-calendar/commit/02c289245a403f40e1c98095c87b58d83a9c32a0.

I am experiencing the same problem. I wrote about it in this PR #1462 .

For me this was fixed with https://github.com/jquense/react-big-calendar/pull/1462.


Progress Circle Widget

Implement a progress circle widget which can be used for various timer/stopwatch type applications.
Example of the progress moving from 1.0 to 0.0 with some updating text in the center.

// start time declared above.
let total_millis = 60 * 1000;
ui.label("60 second timer");
let now: DateTime<Utc> = Utc::now();
let elapsed = now - start_time.unwrap_or(now);

let remaining_millis = total_millis - elapsed.num_milliseconds();
let rem_percent = 1.0 - (elapsed.num_milliseconds() as f64 / total_millis as f64);
let stroke = Stroke::new(6.0, ui.visuals().selection.bg_fill);
let progress_circle = ProgressCircle::new(rem_percent as f32)
    .text(format!("{:.2}", (remaining_millis as f64  / 1000.0)))
    .clockwise(false)
    .segments(3600)
    .stroke(stroke);
ui.add(progress_circle);

Nice work @bmbenson,
However your new widget is not accessible to assistive technologies and unfortunately there's a missing piece in the puzzle. See #4139. Once it's merged, you can have a look at what changes I made to the progress bar, as it's basically the same thing. I am not completely sure whether you'd want to expose a decreasing value if clockwise is set to false though. Having something going from 100% to 0% might be confusing to most people who aren't able to see that there's a circle involved.

Thank you @DataTriny -
The clockwise option controls the draw direction and is independent of the completion percent, so I believe the response.widget_info block will look very similar to your ProgressBar addition.

I appreciate your PR, but I don't think this belong in egui. There are a hundred different ways a user may want to show progress, and the goal of egui is not to be a huge toolbox of every single thing a user could want. We already have a progress bar which should be enough for most people, and if they want something different they can just paint it themselves, or use a 3rd party crate.

Makes sense - thank you for the response(and egui)!


ExpressionDependency does not work

$updated_at = rand(1, 10);
        echo $updated_at . PHP_EOL;
        $dependency = new \yii\caching\ExpressionDependency(
            [
                //'expression' => 'date("Y-m-d H:i:s")', // normal work
                'expression' => '$this->params', // use 'params' not work
                'params' => ['updated_at' => $updated_at]
            ]
        );
        $cacheKey = 'goods_view';
        $date = Yii::$app->cache->getOrSet($cacheKey, function () {

            return [rand(20, 100)];
        }, 3600, $dependency);

        print_r($date)  ;
        exit();

Also, when changing the expression, you must use './yii cache/flush-all' to clear the cache to take effect.
How to use both expression and params parameters correctly ？
Additional info

Q
A

Yii version
<IP_ADDRESS>?

PHP version
5.5

Operating system
mac os

As per the current implementation, params and expression gets cached and that's the reason for the above issue. As a work-around, you can create a function in a model or an object which returns the dependency data and call it in the expression.
Eg:-
$dependency = new \yii\caching\ExpressionDependency(
        [
            'expression' => '$this->params[\'model\']->function()',
            'params' => ['model' => $model]
        ]
    );

Params should not, I think.

@samdark According to my undestanding, only the dependency data should be cached and not the whole dependency object because, one should be able to validate the cache with custom expressions, sql and params when getting the data from cache. So, as a simple solution, what I suggest is, the Cache::get() method should be able to accept a dependency object too like this:
public mixed get (mixed $key, yii\caching\Dependency $dependency = null).
If this $dependency is not null, the actual cached dependency data should be set to this $dependency and yii\caching\Dependency::isChanged() should be executed on this object instead of the cached dependency object.
If this second argument is not null, isChanged() will be executed on the cached dependency and will work as normal.
So, the get() method will look something like this:
public function get($key, $dependency = null)
    {
        $key = $this->buildKey($key);
        $value = $this->getValue($key);
        if ($value === false || $this->serializer === false) {
            return $value;
        } elseif ($this->serializer === null) {
            $value = unserialize($value);
        } else {
            $value = call_user_func($this->serializer[1], $value);
        }
        if (is_array($value) && $value[1] instanceof Dependency) {
        	if($dependency instanceof Dependency){ // if the second argument is given, set the cached dependency's data to this dependency and use this for checking if the dependency data is changed.
        		$dependency->data = $value[1]->data;
        	}else{ // else, proceed with the cached dependency
        		$dependency = $value[1];
        	}
        	if(!$dependency->isChanged($this))
            	        return $value[0];
        }
        return false;
    }

The Cache::getOrSet() method too shall be changed like this so that it uses the given dependency object for passing to the get() method(this will also fix what is mentioned in this issue by @seakk):
public function getOrSet($key, $callable, $duration = null, $dependency = null){
        if (($value = $this->get($key, $dependency)) !== false) { //passing the given dependency object so that dependency evaluation will be done with this object.
            return $value;
        }
        $value = call_user_func($callable, $this);
        if (!$this->set($key, $value, $duration, $dependency)) {
            Yii::warning('Failed to set cache value for key ' . json_encode($key), __METHOD__);
        }
        return $value;
    }

The advantage here is that, one can use custom dependency objects when getting data from cache which allows him to validate the cache each time with custom expressions and params for ExpressionDependency, custom sql and sql-params for DbDependency and the same for all the child dependencies extending from yii\caching\Dependency. In short, only the Dependency::data will taken from the cache and all other properties will be of the given dependency object(if it's set in the second parameter)
Can you please share your thoughts on this so that I can send a pull request?

I don't like the idea of having get() that modifies data.

Isn't it setting dependency data?

The cached dependency's data is set as the new passed dependency's data(if it's passed as 2nd argument) so that, dependency evaluation is performed on this new dependency with old data(cached dependency's data). The cached dependency and its data remains unchanged.

Well, that's what I mean. The proposal for get() isn't only getting data but modifying (copying) dependency. The method has a side effect thus becomes non-idempotent.

Agreeing with you. But, according to the current implementation, ExpressionDependency::params is of no use as one cannot change it's value(for which it is actually intended) after the cache is saved. How about moving params to \yii\caching\Cache as dependencyParams so that one can use it in the expression as $cache->dependencyParams?


(PUP-10473) Verify puppetserver registers its authconfigloader

Verify puppetserver registered its authconfigloader class, which indicates it
will authorize REST requests. If it fails to do so, either due to
misconfiguration or bug, then return HTTP Unauthorized (401) for all v3 REST
endpoints so we fail securely.
Note when legacy routes was enabled (in 6.x) indirected routes performed the
authorization check within its indirection2uri method. But here we do the
check before the route is called, since we just need to know the request method
and path to format the error message.

CLA signed by all contributors.

nit, I think it's uri2indirection.

jenkins please test this with servertests


clip path (and others) being lost

when running a new svg through, it is not displaying correctly.
here is the output: https://dev-web-streetsupport.azurewebsites.net/find-help/accommodation/ on the first icon (read our advice guide), you can see the clip path around the speech bubble has been lost, and the line and window has not been clipped. also, there should be a white frame around the window.
viewing it via gulp-svg-sprite's generated preview page, it appears to be ok:

however, when i delete the embedded svg markup from the source via dev tools, it displays incorrectly:

any ideas, or should i try and build the image in a different way?

something to do with the clip path - i re-drew using intersections

It sounds like this might be related to https://github.com/jkphl/svg-sprite/issues/233. So, you could solve your issue by altering the source SVG?

no worries, i re-drew it and fixed it. cheers!


example of querying balance in command line

Can we have in the readme an example for querying the balance of an account please?
Also the explorer doesn't support auto-detecting user's FT Tokens (like etherscan) right?

near call $ID ft_balance_of '{"account_id": "'bob.$ID'"}' --accountId $ID  ... I think


Example get files changed in pr

Command: cargo run --example git_pr_files_changed  --features="git" $ADO_REPO_NAME $ADO_PR_ID
Example Output:

@johnbatty , please review

I made a few cleanups/formatting changes (tip: use "cargo format").  Thanks!


Check user role

Hello
I don't know where can I get help about this, so I write here.
How can I check if the xyz user has ADMIN rank/role?

At the very least, you can try checking permissions for a channel, provided you have a user.

It also kinda depends on how you want to achieve this, do you want to check by role name? by role ID? by permission?

Ok, I know now. I used the examples. :) I found today this DSharpPlus and it is very good. :) Better than Discord.net :D Before I used discord.net but there is many missing function which I need. So thank you @NaamloosDT .  :D


ORA-12545: Connect failed because target host or object does not exist

I'm trying to use go-oci8 on my project. I use OSX. I already prepare the oracle instantclient and everything, and I can finally use it using sqlplus.
This is my tnsnames.ora
210=(DESCRIPTION=
        (ADDRESS_LIST=
            (ADDRESS=
                (PROTOCOL=TCP)
                (HOST=<IP_ADDRESS>)
                (PORT=1521)
            )
        )
        (CONNECT_DATA=
            (SID=ORCL)
            (SERVER=DEDICATED)
            (SERVICE_NAME=orcl.my.local)
        )
    )

This is the env.
export ORACLE_HOME=/Applications/oracle/instantclient_11_2
export PKG_CONFIG_PATH=$ORACLE_HOME/pkg
export GO_OCI8_CONNECT_STRING="(DESCRIPTION=(ADDRESS_LIST=(ADDRESS=(PROTOCOL=TCP)(HOST=<IP_ADDRESS>)(PORT=1521)))(CONNECT_DATA=(SID=ORCL)(SERVER=DEDICATED)(SERVICE_NAME=orcl.my.local)))"
export ORACLE_SID="ORCL"
export DYLD_LIBRARY_PATH=$ORACLE_HOME:$DYLD_LIBRARY_PATH
export PATH=$PATH:$ORACLE_HOME

It's working if I use sqlplus, sqlplus scott/tiger@210.

The problem is I cannot connect to the oracle server using go-oci8 example.
I have put oci8.pc file inside $PKG_CONFIG_PATH and below is the content of the file.
orainc=/Applications/oracle/instantclient_11_2/sdk/include
oralib=/Applications/oracle/instantclient_11_2

Name: oci8
Description: Oracle Instant Client
Version: 12.1
Cflags: -I${orainc}
Libs: -L${oralib} -lclntsh

Everytime I run it, this error always showing up.
ORA-12545: Connect failed because target host or object does not exist

Could you help me solving this problem? Or am I missing something. Thank you in advance.

Could you please show me your oracle.go?

oracle.go is sample of the project. here is the link https://github.com/mattn/go-oci8/blob/master/_example/oracle.go https://github.com/mattn/go-oci8/blob/master/_example/oracle.go.

On Aug 4, 2015, at 2:01 PM, mattn<EMAIL_ADDRESS>wrote:
Could you please show me your oracle.go?
—
Reply to this email directly or view it on GitHub https://github.com/mattn/go-oci8/issues/65#issuecomment-127502386.

Try with
export GO_OCI8_CONNECT_STRING="youruser/yourpass@210"

It's working now. Thanks!


Model Eval loss is NaN, if I set input masking method to be casual[BUG]

Bug description
when I set the input masking to casual, it causes eval loss to be NaN, But if I tried with MLM, the pipeline is able compute loss for eval.
Steps/Code to reproduce bug

Try setting casual or clm in input masking using torch module.
inputs= tr.TabularSequenceFeatures.from_schema(
schema,
max_sequence_length=sequence_length,
d_output=d_model,
aggregation="concat",
masking="causal",
)

Environment details

Transformers4Rec version: 0.1.7
Platform: Kaggle Jupyter Notebook
Python version:3.7
Huggingface Transformers version:4.17.0
PyTorch version (GPU?):1.9.1

@john-marinton thanks for your interest in the repo. Are you testing any of the examples in the repo? if yes, which notebook? Are you using FP16 optimization? can you share a bit more info?


Revisit migration page in documentation

Link: https://laravel-tenancy.com/docs/hyn/5.x/migrations

[x] verify all migration related commands are documented
[x] give one example (and explain) of how to run commands against all tenants, default behavior
[x] give one example (and explain) of how to run commands against specific tenants

Added three todos in your start post.

@luceos what other things we can do with the migrations.md file here?


Make storeAccessToken public

Instagram's recommended way of authenticating users is via server-side. Through server-side auth, a code is sent to the server and the server then returns an access token. By making this method public, we can save the access token that we receive in return.
Checklist

[x] I've tested my changes.
[x] I've updated the documentation, if necessary.
[x] I have read the contibution guidelines.

Proposed Changes

Make saveAccessToken public to allow server-side authentication.

Seems right to me, thanks @josecalles 👍


Add a plugin to create course plans

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
We need a plugin to easily build a course plan
Describe the solution you'd like
A course plan looks like the screenshot below.
It could also be a more structured index with nested subtitles.
The plugin should allow adding titles by level.
Describe alternatives you've considered
Should we allow markdown?
Discovery, Documentation, Adoption, Migration Strategy

Since we are using CKeditor for many other content, i recommend to avoid using any structured text formatting like ReST or Markdown. It perturbs users and you will have to bring them a lot of help to using it, and finally it will results in problems.
By the way, i don't think you need formatting here, course plan will have to be correctly styled (according to an ergonomic design) and that's all. Having links in course plan will results to nasty user tricks, even bold or italic would be disturbing.

A plugin for course plans was added in PR https://github.com/openfun/richie/pull/947.


Remove synchronized methods/blocks from the codebase

Revise synchronized blocks/methods in critical code path to avoid virtual threads pinning

any plan to merge this mr? seems intresting for virual thread

coverage: 87.157% (+0.01%) from 87.147%
when pulling 1e9eb69cdf71519fe584fc5c3cf46f08a2b16b4f on m1ngyuan:patch-1
into 8ee1ebf1d0f29f5bc417563cf25559eea6dba0d0 on mybatis:master.

I have updated the PR.
Please see each commit for the details and let me know if you noticed any problem.
Questions/clarifications are also welcome.

Thank you for your update, LGTM @harawata.

Merged.
If someone finds a problem, please let us know!
Thank you again for your contribution, @m1ngyuan !


Join event is not handled if the username is not lowercase

If the username is not lowercase the Join event is not detected which breaks joining channels after the bot is initialized and stops the emission of join events. There should be either an error when the username is not lowercase, automatically changing the username to lowercase or just detect the username when it is lowercase.

+1!
I pulled my hair not understanding why the bot did not connect..
Can confirm I had capitalized name "Myname" and .join event never fired.


How to kill the thread pc-broker-poll

Hey there!
I am having an issue with closing the consumer, as the pc-broker-poll continues to work. Is there any way to kill this thread? Thanks
"pc-broker-poll" #32 prio=5 os_prio=31 cpu=379.01ms elapsed=65.90s tid=0x00007fad53183000 nid=0xbe03 waiting on condition  [0x00000003109a5000]
   java.lang.Thread.State: WAITING (parking)
	at<EMAIL_ADDRESS>Method)
	- parking to wait for  <0x00000007139af7d0> (a java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject)
	at java.util.concurrent.locks.LockSupport.park(java.base@11.0.11/LockSupport.java:194)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(java.base@11.0.11/AbstractQueuedSynchronizer.java:2081)
	at java.util.concurrent.LinkedBlockingQueue.take(java.base@11.0.11/LinkedBlockingQueue.java:433)
	at java.util.concurrent.ThreadPoolExecutor.getTask(java.base@11.0.11/ThreadPoolExecutor.java:1054)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(java.base@11.0.11/ThreadPoolExecutor.java:1114)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(java.base@11.0.11/ThreadPoolExecutor.java:628)
	at java.lang.Thread.run(java.base@11.0.11/Thread.java:829)

Hi, this is a known bug and is being addressed by the following two PRs:
https://github.com/confluentinc/parallel-consumer/pull/593
https://github.com/confluentinc/parallel-consumer/pull/591
We are planning to release the updated version this week.

@johnbyrnejb I see... thx!


Update README.md

We think your audience will appreciate it. This product might be worth mentioning in your blogs.

Is there any possibility to add a tutorial link in your blogs.


Added functionality to show only 1 notification at a time and hide no…

…tification option

Hey, thanks for the PR.
What are your thoughts behind this PR, you haven't put any description of it.
There are some stuff that doesn't quite make sense. First off, you create an array of CRNotification views that essentially never hold more than one element, since you can removeAll every time a notification is created. Here it would make sense to just keep one optional notification instead of a list.
The method isNotificationVisible can just return the first condition as a result instead, which is a little cleaner. You check for an empty array which should be done through isEmpty for better readability. But what might be better would be to use currentNotification?.superview != nil provided that the implementation relies on a single notification instead of a list.
If however, a list is used, this could maybe provide a queue system where the next notification in line would be displayed, when the other is dismissed. Or maybe decide when presenting the notification, whether or not the given notification should be allowed displayed or not based on whether a notification is actually present at the given time. In this way, a promotion system is implemented and one notification can actually be more important than others. If for some reason you'd go the extra mile, a notification or CRNotificationType could actually have a priority value that decides whether or not the notification is allowed to be presented over the one that is already on screen.

Hello Casper Riboe
This is my first ever PR in any Github project so please don’t mind and ignore it, I was not expecting quick response on it, but you are great and thanks for responding. I liked CRNotification very much that’s why I decided to modify it as per my requirement, but I know this change is not required for all developers.
But I think I can improve it as per your suggestions and make it flexible. I can add option where user can mention to choose between displaying one notification at a time or allowing multiple notifications. But hiding this notification without tapping was missing I think this feature you should implement in it, this is just my suggestion.
Thanks
Best Regards
Irfan Anwar
On 11-Apr-2018, at 4:43 PM, Casper Riboe <notifications<EMAIL_ADDRESS>wrote:
Hey, thanks for the PR.
What are your thoughts behind this PR, you haven't put any description of it.
There are some stuff that doesn't quite make sense. First off, you create an array of CRNotification views that essentially never hold more than one element, since you can removeAll every time a notification is created. Here it would make sense to just keep one optional notification instead of a list.
The method isNotificationVisible can just return the first condition as a result instead, which is a little cleaner. You check for an empty array which should be done through isEmpty for better readability. But what might be better would be to use currentNotification?.superview != nil provided that the implementation relies on a single notification instead of a list.
If however, a list is used, this could maybe provide a queue system where the next notification in line would be displayed, when the other is dismissed. Or maybe decide when presenting the notification, whether or not the given notification should be allowed displayed or not based on whether a notification is actually present at the given time. In this way, a promotion system is implemented and one notification can actually be more important than others. If for some reason you'd go the extra mile, a notification or CRNotificationType could actually have a priority value that decides whether or not the notification is allowed to be presented over the one that is already on screen.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/dkcas11/CRNotifications/pull/12#issuecomment-380422793, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQCP-8-k4Eti5peO2N9DCq4o4v6G2Mi6ks5tnexngaJpZM4TPlkp.

Hey, no worries.
Try and customize it to allow users to decide whether one should be allowed at a time or not :-)

This is now implemented (not from this PR).


Tooltip: improved the readability.

The code at the point where popperVM is generated should be separated by an easy-to-understand function etc.
Because easy-to-understand function names replace comments and maintainability is improved.

The function is not reused. Is it necessary to do this?

Thanks for your PR!
In my opinion, it is unnecessary to create an intermediate variable for creating popperVM.
Plus:

generatePopperVM function has no need to be reused  in other contexts.
beforeCreate hook is simple enough to understand.

So I think the current code doesn't break the readability.

I appreciate your review!
That's right.
I close this pull request!


404 Not Found broadcast

Stucked on this issue for a few hours, I keep getting the error Illuminate\Broadcasting\BroadcastException: Pusher error: Not Found, and I can't identify what's not found
I currently run php8 with apache.
below are the configs I use.
For the broadcasting config
`
pusher' => [
'driver' => 'pusher',
'key' => env('PUSHER_APP_KEY'),
'secret' => env('PUSHER_APP_SECRET'),
'app_id' => env('PUSHER_APP_ID'),
'options' => [
'cluster' => env('PUSHER_APP_CLUSTER'),
'useTLS' => false,
'encrypted' => true,
'host' => env('PUSHER_HOST', '<IP_ADDRESS>'),
'port' => env('PUSHER_PORT', 6001),
'scheme' => env('PUSHER_SCHEME', 'http'),
],
`
and for the websocket config,
`
'apps' => [
[
'id' => env('PUSHER_APP_ID'),
'name' => env('APP_NAME'),
'key' => env('PUSHER_APP_KEY'),
'secret' => env('PUSHER_APP_SECRET'),
'capacity' => null,
'enable_client_messages' => false,
'enable_statistics' => true,
],
],
`
the Vue config is
`
window.Echo = new Echo({
broadcaster: 'pusher',
key: process.env.MIX_PUSHER_APP_KEY,
wsHost: window.location.hostname,
wsPort: 6001,
forceTLS: false,
disableStats: true,
});
`

????

@chistel how you fixed this issue

Yea, had to do some configuration on the server

@chistel  could you please post it here so it helps me to solve it? Thanks @chistel


Add line break in expression/argument lists with more than 4 arguments

How to achieve this formatting? I.e. same as parameters of a procedure/function
CREATE OR REPLACE PACKAGE BODY tools AS
    -----------------------------------------
    -- Private functions and procedures START
    -----------------------------------------
    TYPE arr IS
        VARRAY(20) OF VARCHAR2(20) NOT NULL;

    orderable_key_prefixes arr := arr(
        'aaaaaaaaaaaa',
        'bbbbbbbbbbbb',
        'cccccccccccc',
        'dddddddddddd',
        'eeeeeeeeeeee',
        'ffffffffffff',
        'gggggggggggg',
        'hhhhhhhhhhhh',
        'iiiiiiiiiiii',
        'jjjjjjjjjjjj',
        'kkkkkkkkkkkk',
        'llllllllllll',
        'mmmmmmmmmmmm'
    );

END;
/

When I format I get this:
CREATE OR REPLACE PACKAGE BODY tools AS
    -----------------------------------------
    -- Private functions and procedures START
    -----------------------------------------
    TYPE arr IS
        VARRAY(20) OF VARCHAR2(20) NOT NULL;
    orderable_key_prefixes arr := arr('aaaaaaaaaaaa', 'bbbbbbbbbbbb', 'cccccccccccc', 'dddddddddddd', 'eeeeeeeeeeee', 'ffffffffffff', 'gggggggggggg', 'hhhhhhhhhhhh', 'iiiiiiiiiiii', 'jjjjjjjjjjjj', 'kkkkkkkkkkkk',
    'llllllllllll', 'mmmmmmmmmmmm');
END;
/

By enabling these three lines in the Arbori program.
| .breaksProcArgs & [node) arg & [node^) paren_expr_list & [node+1) arg_list     -- salvisberg: disabled, breaks for assoc_args only
| .breaksProcArgs & [node) arg_list & [node^) paren_expr_list & [node-1) arg     -- salvisberg: disabled, breaks for assoc_args only
| .breaksProcArgs & [node) ',' & [node^) arg_list & [node+1) arg                 -- salvisberg: disabled, breaks for assoc_args only

This will change the formatting of function parameters as well. I've disabled these lines, because I thought that if you want to have line breaks, then you should use the named notation as suggested by G-7110. So I like to have
xyz := to_char(x, 'DD.MM.YYYY');

On a single line. I do not like this result:
xyz  := to_char(
    x,
    'DD.MM.YYYY'
);

You see, fixing one problem is usually easy. But taking the all cases into account makes it more difficult. ;-)
I hope this answers your question.

I understand, you fix your sink and realize your chimney fell down :)
Anyway, I noticed in Toad for Oracle, they have that neat feature where you can configure wrapping of the parameters based on their count. So let's say I can accept having 4 params on 1 line (in proc call or in my case in array definition) but I would like to break them on separate lines if there are 5 or more or the line length exceeds the threshold.
Would that be a (viable) option?

Yes, this is doable.

I wonder if number of arguments is the best possible condition to switch formatting form in-line to multi-line.
Wouldn't it be better to use margin settings or expression(argument list) length?

@jgebal yes, you are right. But calculating the correct positions for the token during the formatting is not simple. You would need to do that at the very end. For correct results, iteratively. It's more than adding just a new line. For example It impacts the comma handling and indentation. I decided to go with the number of arguments, because it is much simpler.

formatting code  is amazingly complex especially with SQL and it's rich vocabulary and versatility of optional items.
I truly admire what you're doing.

Thanks @jgebal. I do not think it is that complicated. The problem is that there are a lot of unit tests missing. I've started to create some for each change. But I have basically no idea what the side effects of a change are. So I try to format some of my code and I see a lot of cases that are not handled they way I would like it. Then I check if it was the case before and if yes, then I keep it that way. So it's you and others such as @krokyk that drive the change by creating issues. I'm still thinking about how to proceed to get a more complete set of unit tests. All options lead to a lot of work...
Regarding this case. I'm not happy with counting arguments. I found a case where 6 short parameters are passed (creating an object type) that would easily fit on line, but are splitted now. So maybe an imperfect counting mechanism, such as counting the content of an argument and considering let's say 70% of the configured line size as a limit would be better.

(1) Make number of arguments/expressions configurable

Configuration is the "Advanced Formatting" section. That's it. There is one similar parameter. It is named "Line Breaks: Commas per line in procedure" (1 comma means two arguments). We are currently not using this parameter in a consistent way.
Another option is to create an own formatter extension with dedicated formatting preferences. This makes sense when Arbori is not enough and some post-processing is necessary. However, this would require an additional context menu entry. Such as "Beautify" or "Custom Format" or similar. Making this compatible with SQLcl will become more difficult. So this is really the last resort and we'd like to avoid it. So the settings can be configured with what is available in "Advanced Formatting". Everything else will be hard-coded. We choose the value we like most ;-)

(2) Second - if the margin is exceeded

As explained previously, calculating the exact number of characters used by all tokens on the line can happen after formatting only. However, an estimation is possible and in combination with a safety margin (e.g. 70%) this is doable and will lead most probably to better results than the current implementation (3).
Arbori gets the full parse-tree and has access to all the args and its length. I know how to query them. I'd consider the length plus 1 or 2 chars per arg for parenthesis, comma and space between args. So getting the total chars of all args is doable. The challenge is to define the max. number of chars allowed before splitting args. For that we would need the tokens before the argument list. At that time not all formatting steps are completed, so we cannot reliably calculate the line size. We have to estimate it and define a safety margin. A percentage somehow related with the line size (it matters if it is 40 vs. 70 vs. 200 vs. 400). This needs some more thinking. But whatever we choose will not be perfect. There will be cases where the post-processor of SQLDev will identify a line which is longer than the max and it will add a line break plus an indentation of the previous line before a token. And of course there will be other cases where everything would fit into a line but we are splitting it. The latter is better IMO, because we control the formatting result.

(3) If more than X arguments, split arguments 1 per line

I would not combine (2) and (3). It's either (2) or (3). Doing (2) right should lead to a better result. Probably only corner cases could produce a better result.

I was not suggesting combining 2 & 3, it's more like if-else if.
if margin_exceeded {
    //split each arg into its own line
} else if num_args > 4 {
    //split each arg into its own line
} else {
    //do nothing
}

Well, I was suggesting something like this:
if margin_exceeds {
   // split, each arg on own line
} else {
   // do nothing
}

I really thing the result will be better and the solution will be simpler.


Support for group ironman collection log

It would be nice to have a shared GIM collection log that would show group unlocks next to individual unlocks in the collection log UI.
Is there an API at https://collectionlog.net where another plugin could fetch user collection log data?

I am also looking into this. Found docs but have not looked to deep yet.
https://docs.collectionlog.net/


Add release flow

Chore
Describe the chore
Once a github release is made, it should trigger a github action that builds, test, and deploys the java package to the appropriate service.
Additional context
Currently there is no decision on wether this should just be GitHub Packages or something like OSSRS from sonatype.

Release flow has been added to GHA.
The settled solution, is I provided a PAT with package:write access, which expires after 180 days. 31/08/2023
It would be better if someone from the team could provide a PAT to write to packages, since they are more stable.


Update About.js

Test to see if pipeline is triggered to deploy SWA and comment on the PR

SWA was not deployed and comment not posted - as expected
Closing test PR


Clean up RDMA code

Replace all friends with proper interfaces
Split endpoint into parts used by different threads to simplify reasoning about multi-threaded code

@drbasic @budevg


python311Packages.griffe: 0.42.2 -> 0.44.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.griffe is: Signatures for entire Python programs
meta.homepage for python311Packages.griffe is: https://github.com/mkdocstrings/griffe
meta.changelog for python311Packages.griffe is: https://github.com/mkdocstrings/griffe/blob/0.44.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.44.0 with grep in /nix/store/yw0j6920632bwzk2fihga8vpfa2n1p8m-python3.11-griffe-0.44.0
found 0.44.0 in filename of file in /nix/store/yw0j6920632bwzk2fihga8vpfa2n1p8m-python3.11-griffe-0.44.0

Rebuild report (if merged into master) (click to expand)

9 total rebuild path(s)

9 package rebuild(s)

First fifty rebuilds by attrpath

griffe
netbox
netbox_3_6
netbox_3_7
python311Packages.griffe
python311Packages.mkdocstrings-python
python312Packages.griffe
python312Packages.mkdocstrings-python

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yw0j6920632bwzk2fihga8vpfa2n1p8m-python3.11-griffe-0.44.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.griffe https://github.com/r-ryantm/nixpkgs/archive/50994a6468cb5d96e10d4bdf021653d68531ef53.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/50994a6468cb5d96e10d4bdf021653d68531ef53#python311Packages.griffe

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yw0j6920632bwzk2fihga8vpfa2n1p8m-python3.11-griffe-0.44.0
ls -la /nix/store/yw0j6920632bwzk2fihga8vpfa2n1p8m-python3.11-griffe-0.44.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages failed to build:

netbox (netbox_3_7)
netbox_3_6

8 packages built:

griffe (python311Packages.griffe)
griffe.dist (python311Packages.griffe.dist)
python311Packages.mkdocstrings-python
python311Packages.mkdocstrings-python.dist
python312Packages.griffe
python312Packages.griffe.dist
python312Packages.mkdocstrings-python
python312Packages.mkdocstrings-python.dist

Maintainer pings
cc @fabaff for testing.

#305353


merge from arlac77/template-esm-only,arlac77/template-github

package.json

chore(package): add ^12.3.0 remove ^12.2.0 (devDependencies.documentation)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FIX: cpac_templates.tar.gz unpacking incorrect directory

Fixes #28
Introduced this bug in my last commit, cpac_templates.tar.gz was unpacking in the root directory, should unpack within /cpac_resources to match with default_pipeline.yaml
2 steps forward, one step back... but this fixes it.

sorry that i missed this, including now

let me know if there is anything I can do on my end to make the merge easier!


Explain how to concat-fields / extract more than one field as key/value in a variable

I have a json structure like this
{
 "items": [
  { "id": 5, "name": "Vienna" },
  { "id": 6, "name": "Salzburg"}

 ]
 ..
 }

I try to create a variable "countyId" which allows the user to select the county "Vienna" or "Salzburg" and use the id (5 or 6) in follow up queries.
I'm trying since more than three hours various ways of specifying my field, but I'm unable to come up with a way to get them used together, or concat them into a string so I can split them up in a regex.
These are a few of the patterns I have tried, but to no avail:
$.items[*][id,name] // this gives me 5, Vienna, 6, Salzburg
$.items[*].id // this gives me just the id, which is not user friendly
$.items[*].name // this gives me the name, which would be user friendly, but i cannot use it to resolve other parameters
$.concat($.items[*].name,$.items[*].id) // does not work, although I hoped it would ( https://stackoverflow.com/questions/45941225/how-to-concatenate-json-elements-with-jsonpath-jayway )
$.items[*] | { .name, .id } | @concat
$.items[*].name@string() + 'aaa'

so I need a way to either use two fields of an object for text + value or concat two fields into a string, so I can separate them using regex.
Please advice.

This sounds like a duplicate of https://github.com/marcusolsson/grafana-json-datasource/issues/79? Could you check it out to see if that issue would fit your use case?

I've just published v1.1.0 to the marketplace, which includes experimental support for this. Would you mind checking it out and letting me know whether it addresses your use case?

Closing due to inactivity.


NEW issue

Thank you.
This code from https://github.com/vishwas-r/WP-CanvasJS/wiki/Chart-Types#5-stacked-bar-chart
not work properly.
8. Candlestick Chart:
[canvasjschart options="{title:{text: 'CanvasJS Candlestick Chart'}, data: new Array({type: 'candlestick', dataPoints: new Array({ x: 10, y: [20, 30, 10, 15] },{ x: 20, y: [30, 40, 20, 35] },{ x: 30, y: [35, 45, 25, 40] },{ x: 40, y: [45, 50, 30, 40] },{ x: 50, y: [40, 60, 35, 55] })})}"]

Updated example


feat(profiles): Profile Package implementation

Introduces profiles as a standalone library.
begin migrating sfpowerkit profiles
resolve typescript signature errors
remove ProgressBar references
update invalid import paths to use path aliases
include rc files
removed powerkit commands from sfpowerscripts-cli (temporarily)
updated sfpowerscripts-cli package lock
Checklist
All items have to be completed before a PR is merged

[x] Adhere to Contribution Guidelines
[x] Updates to Decision Records considered?
[x] Updates to documentation at DX@Scale Guide considered?
[ ] Tested changes?
[ ] Unit Tests new and existing passing locally?

@azlam-abdulsalam after discussion with you:

snapshot tests
salesforce libs all bumped to latest
Typescript version matching
integration to sfpowerscripts.core


[beta] Consider changing title for "JSON APIs and Ajax: Trigger Click Events with JavaScript"

Challenge trigger-click-events-with-javascript has an issue.
As this challenge is about adding an onclick handler for a DOM element, I think the title of this challenge should be changed to "Handle Click Events with JavaScript with the onclick Property".
If this challenge were to cover the topic of triggering click events, it should cover someDOMElement.click() instead.
The challenge description could also further explain that the onclick property only is one way to handle click events in JS.

Cat Photo Finder

  The message will go here

  <button id="getMessage">
    Get Message
  </button>

@Greenheart can i propose the below changes to the description? I rephrased the portion on event handlers to hint that the "onclick" event handler is one of many. For the deletions, my intention is to reduce the information that the camper does not need to complete this particular challenge. This will help to prevent confusing the camper and allowing him to focus on learning one concept per challenge, as per the style guide. Any suggestions to help me improve the description is welcomed 😄
Challenge Description:
This section covers how to get data from APIs. APIs - or Application Programming Interfaces - are tools that computers use to communicate with one another. <-- mentioned in introduction. Also to prevent lengthy descriptions.
You'll learn how to update HTML with the data we get from these APIs using a technology called AJAX. <-- shift to the Get JSON with the Javascript XMLHttpRequest Method challenge, where the camper constructs his first AJAX request.
You want your code to execute only once your page has finished loading. For that purpose, you can attach a JavaScript event to the document called DOMContentLoaded. Here's the code that does this:

document.addEventListener('DOMContentLoaded',function() {

});

You can implement event handlers that goes inside of the DOMContentLoaded function. You can implement the onclick event handler, which triggers when the user clicks on the element, by adding the following code:

document.getElementById('getMessage').onclick=function(){};

@hay-dee I'm so sorry for the delayed response, I've been pretty busy (starting a new job).
I like your suggestions, thanks for taking the time! Feel free to open a pull request, or talk to others about it in the chat rooms :blush:

@Greenheart Wow congratulations on your new job! No problems with the delay, i have submitted a PR to address the issue raised. You can review the PR when you have the time 😄


Its Slow on Mac M1

why is it slow on Mac M1 takes several min to gen 1 token

Not a lot I can do on my end regarding this, the model is quite large and CPU inference is known to be slower than GPU in general. You can try adjusting the threads to see what works best for your system.


Fix examples in documentation

A few things like expect(1).toEqual(1) won't compile because the compiler won't be able to determine the type.

fixed by #133


Re-add encode() method

The encode() method in 2.x was a handy way to encode images according to configuration (not hard-coded). Now it must be:
switch( $type ) {
    case 'image/jpeg': $img = $image->jpeg(); break;
    case 'image/gif': $img = $image->gif(); break;
    case 'image/png': $img = $image->png(); break;
    case 'image/webp': $img = $image->webp(); break;
}

It would be a good idea to add the method in 3.x again but use different methods to set the quality/number of colors depending for the output format.

Yes, I agree.

Hi,
I agree too and I think it misses the type guessing of an Image like before:
        $image = $this->imageManager->make($input);
        $extension = match ($image->mime) {
            'image/jpeg' => 'jpg',
            'image/webp' => 'webp',
            'image/gif' => 'gif',
            'image/png' => 'png',
            default => throw new RuntimeException('Invalid mime type.'),
        };
        $data = $image->encode($extension);

Thanks,
Mickaël

I agree too and I think it misses the type guessing of an Image like before:

Not only the media type but ideally also the type and quality of compression, the number of colors and perhaps other things.

But quality and number of colors should be set by their own methods if possible because they depend on the output format, e.g.:
$image->quality( 75 );
// or
$image->maxcolors( 255 );
$encoded = $image->encode( 'image/jpeg' );
$encoded->save('path/to/file');

The encode() method can also accept mime types and file extension names:
public function encode( $mime ) : EncodedImage
{
        $mime = match ($mime) {
            'image/jpeg' => 'jpg',
            'image/webp' => 'webp',
            'image/gif' => 'gif',
            'image/png' => 'png',
            default => $mime,
        };
        return $this->driver->encode($this, $mime);
}

The encode() method was reintroduced in version 3.1. See documentaion.

@olivervogel As long as there's no method available which maps (mime) types to the class names, the new encode() method has no value.

@olivervogel As long as there's no method available which maps (mime) types to the class names, the new encode() method has no value.

Explain this in more detail.

In an application, the output format is either configured or selected by the user. In both cases, the format is always a string and to be most portable, it's most often the mime type that is used. Using an intervention class name would tie the application exclusively to the library and changing the image processing library later would be a bigger issue.
Thus, we have the (mime) type as input and the intervention class name as required parameter. If the library doesn't provide a method to do the mapping, the application code still needs to implement:
switch( $type ) {
    case 'image/jpeg': $img = $image->encode(JpegEncoder::class); break;
    case 'image/gif': $img = $image->encode(GifEncoder::class); break;
    case 'image/png': $img = $image->encode(PngEncoder::class); break;
    case 'image/webp': $img = $image->encode(WebpEncoder::class); break;
}

So there's no value to use encode() over:
switch( $type ) {
    case 'image/jpeg': $img = $image->jpeg(); break;
    case 'image/gif': $img = $image->gif(); break;
    case 'image/png': $img = $image->png(); break;
    case 'image/webp': $img = $image->webp(); break;
}

As of version 3.2 you can use encodeByMediaType() in your example code.


Bit-counting API is inconsistent with that of primitive types

There are several differences between Uint/Limb and Rust's primitive integer types:

Bit size of the integer: Limb::BIT_SIZE, but u*::BITS
Type of the above constant: Limb::BIT_SIZE is usize, but u*::BITS is u32
Return types of zero/bit counting methods: Limb/Uint::bits() is usize, but u*::leading_zeros()/trailing_zeros() is u32

Unless there was some specific reason these choices were made, perhaps it makes sense to mimic the existing standard lib API.

Agreed, would suggest aligning with the primitives in #153


added reference to mssql exporter

Reference to a new exporter for Microsoft SQL Server.

that's a fair recommendation! I will rework the logic to be on demand only.

I've released a new version of the exporter which only fetches data upon scrape. Thanks @brian-brazil for your suggestion.

Thanks!


Do not build benchmark tests when building container

It also tries to find gtest when tests are enabled, which fails on
CADES, for example.

It also tries to find gtest when tests are enabled, which fails on
CADES, for example.

I don't understand the issue.  We do pull the google test.  Also how is the failure related to the Dockerfile?

I don't understand the issue. We do pull the google test. Also how is the failure related to the Dockerfile?

This is unrelated to docker container per se. However, BENCHMARK_ENABLE_TESTING is ON by default, and it builds several tests when building benchmark. We don't need them, so might as well disable.
The failure was on CADES, which does not have gtest. The only reason I mentioned that is that is what lead to this discovery.

Actually, why do we need google test clone? We should probably remove that too, that's unnecessary.

retest this please


Calibration error

I am getting the following error when I run on-chip calibration.
"invalid value in rs2_run_on_chip_calibration........"
Are there any other possibilities to do calibration of Intel Realsense D435i by using python codes?

librealsense
2.49.0 RELEASE

OS
Windows

Name
Intel RealSense D435I

Serial Number
038322070705

Firmware Version
<IP_ADDRESS>

Advanced Mode
YES

Camera Locked
YES

Usb Type Descriptor
2.1

Product Line
D400

Asic Serial Number
039223050798

Firmware Update Id
039223050798

Please provide a description of the problem

Hi @Nishanthiruthayaraj   The RealSense SDK has a Python example program for on-chip calibration called depth_auto_calibration_example.py at the link below.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/depth_auto_calibration_example.py
If you have been working in C++ up until now, https://github.com/IntelRealSense/librealsense/issues/7087 provides an On-Chip calibration script shared by a RealSense user to compare your own code to.
On-chip calibration can additionally be performed in the RealSense Viewer tool by going to the 'More' option near the top of the Viewer's options side-panel and selecting On-Chip Calibration from its drop-down menu.

Note that you have to turn off all streams before run on chip - calibration, otherwise it informs error.

Note that you have to turn off all streams before run on chip - calibration, otherwise it informs er

Hi @Nishanthiruthayaraj The RealSense SDK has a Python example program for on-chip calibration called depth_auto_calibration_example.py at the link below.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/depth_auto_calibration_example.py
If you have been working in C++ up until now, #7087 provides an On-Chip calibration script shared by a RealSense user to compare your own code to.
On-chip calibration can additionally be performed in the RealSense Viewer tool by going to the 'More' option near the top of the Viewer's options side-panel and selecting On-Chip Calibration from its drop-down menu.

Thank you very much for the on-time reply. Error resolved

Note that you have to turn off all streams before run on chip - calibration, otherwise it informs error.

Thanks for the hints. My problem is already resolved. Will try your idea if I get the same error.

Excellent news, @Nishanthiruthayaraj that you achieved a solution - thanks for the update!   :)

Case closed due to solution achieved and no further comments received.


FreeBSD - Freenas jail - install-gui.sh throws Error: Platform: "freebsd" is not supported

Hi,
I followed several times the tuto here
I'm sure i did well all steps but when i run sh install-gui.sh, i have these errors :
"(venv) [root@Chia ~/chia-blockchain]# sh install-gui.sh
Updating FreeBSD repository catalogue...
FreeBSD repository is up to date.
All repositories are up to date.
Checking integrity... done (0 conflicting)
The most recent versions of packages are already installed
Running git submodule update --init --recursive.
Running git submodule update.

<EMAIL_ADDRESS>postinstall /root/chia-blockchain/chia-blockchain-gui/node_modules/cypress
node index.js --exec install

/root/chia-blockchain/chia-blockchain-gui/node_modules/cypress/lib/tasks/state.js:53
throw new Error(Platform: "${platform}" is not supported.);
^
Error: Platform: "freebsd" is not supported.
at getPlatFormBinaryFolder (/root/chia-blockchain/chia-blockchain-gui/node_modules/cypress/lib/tasks/state.js:53:13)
at Object.getBinaryDir (/root/chia-blockchain/chia-blockchain-gui/node_modules/cypress/lib/tasks/state.js:81:44)
at Object.start (/root/chia-blockchain/chia-blockchain-gui/node_modules/cypress/lib/tasks/install.js:263:27)
at Object. (/root/chia-blockchain/chia-blockchain-gui/node_modules/cypress/index.js:16:36)
at Module._compile (node:internal/modules/cjs/loader:1092:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1121:10)
at Module.load (node:internal/modules/cjs/loader:972:32)
at Function.Module._load (node:internal/modules/cjs/loader:813:14)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12)
at node:internal/main/run_main_module:17:47
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><0.10.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but noneis installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none isinstalled. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/webpack-dev-server/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"freebsd","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"freebsd","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: node index.js --exec install
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above."
Thanks in advance for the help and sorry if there are some mistakes, English are not my native language  :)

Fixed via https://github.com/Chia-Network/chia-blockchain-gui/pull/306


Allocate reserved command codes.

Unlike unknown commands, these do not trigger a fatal error. Instead, they simply return failures. In the future, these codes can be repurposed if needed. The code values were chosen at random.

Closing in favor of https://github.com/chipsalliance/caliptra-sw/pull/680


Some questions about z_cfg

Hi, have you tried setting the pred_z_flag and gt_z_flag within z_cfg to True to train and go for evaluation of accuracy? I tried pred_z_flag and gt_z_flag within z_cfg set to True but reported an error, now I'm trying to modify the associated loss for the added z values. If you haven't done the above attempts, can you give me some advice? Really thank you very much!!!

Thanks for your interest, we haven't tried to predict the z values. To support 3D prediction, the computing of geometric representations should be modified. In our method, computing the sine value of the angle between two displacement vectors is required, which is different between 2D and 3D cases.

Thank you so much for the advice you provided, I really appreciate it. Your insights are very valuable to me and I will seriously consider and try to apply the suggestions as much as possible. Thank you for your generosity in sharing!


Samples

Currently there is only a single sample page with a not-very-helpful grid demo.  This should be exploded into a sample site broken into categories.  The sample site should exist in this repo (probably under samples?) and should be published via gh-pages and linkable from the docs.
It would be cool to have a "live" sample site like jsfiddle.  It's all client side so it shouldn't be to hard.
Samples should cover at the very least the topics in the readme and can probably just be copied and pasted in there.

In order to fit with the documentation framework, samples should be contained in the ./tutorial dir or be inline with the jsdoc comment using the @example tag.
I've forked the https://github.com/TonyGermaneri/docstrap project.  Live docs will be added there giving live tutorials instead of the static tutorials.
Docs must be committed in the gh-pages branch, not the master branch.  Keep docs out of the master branch.

Demo
https://tonygermaneri.github.io/canvas-datagrid/tutorials/demo.html
Docs
https://tonygermaneri.github.io/canvas-datagrid/docs/canvasDataGrid.html
Tutorial
https://tonygermaneri.github.io/canvas-datagrid/docs/tutorial-sample.html#Create a new grid


Remove redundant length(x) calls

This is a small speed optimization.  In many functions, length(x) is called twice.  The change is simply to assign the integer variable to length(x) before using it in the compare within the assert().

Submit a PR?

Ready for review of PR.

Closed by #14


Add an informational homepage.

There is a need to develop an informational homepage for the portal. Some thoughts on what should be added include the following:

[ ] System information.
[ ] Links to external site stats pages.
[ ] News and updates area.
[ ] Stats to be decided on later. (Only if some sort of persistent data storage is used...)

There is a choice to be made when it comes to completing the above goal. both carry pros and cons which is why I am posting this here before deciding on which way to go myself. Anyone interested in this project is welcome to reply with their thoughts on this topic.

Keep things very light weight and generate this data using "maintenance" bash scripts.
Use a scripting language such as PHP and add a web based administration area.

PROS
Bash Script

Very little overhead using light weight bash script generated .html files.

Easy to use web interface with no need to log directly into the feeder.
Possibility to expand the portal's capabilities even more so in the future by retaining queriable data.

CONS
Bash Script

Harder for people newer to *NIX to comprehend.
Need to log directly into the system in order to make updates.
Very basic minimal persistent data.

System overhead.

Decided to use PHP+XML. There is no real homepage I am leaving the choice of what to display first up to the users. Although some of the ideas posted here will be used at a later date.


Chrome Notification Center URL goes to the homepage

Windows 10 Pro - 1903 - 18362.720
Chrome Version 84.0.4147.125 (Official Build) (64-bit)
The behavior with PushJs seems to vary. I've noticed 2 variances.
An example of how this is set up is :
assuming link = www.example.com/asd/dsa
var config = {
            body: text,
            icon: "/img/notification.png",
            link: link,
            tag: text
        };

        if (link) {
            config.onClick = function () {
                window.open(link);
                this.close();
            };
        }

A notification pop-up on the right side of the screen. If I click on this pop-up, it opens to the correct/full URL.

1b) Issue is here If I leave it alone for 4 seconds, it calls onclose() by itself and it closes this pop up, BUT it adds itself to the Windows Notification Center. Clicking the notification here goes to the homepage of the URL, in this case, www.example.com, and drops everything after the /. How can I fix this ?

If I trigger the same message multiple times, it goes directly to the Windows Notification Center. Clicking on this notification now goes to the correct/full url.

A temporary workaround seems to be setting requireInteraction = true, so the pop up doesn't hide itself, but closing the tab that generated the notification, then clicking on the notification would open a new tab and goes to the homepage instead of the full url.

@jasontestinsight @Nickersoft
I am experiencing the exact same issue. Did you ever find a way to resolve it?


XH.fetchService.fetch() throws on 307 Temporary Redirect

If a fetch request returns a 307 Temporary Redirect, an exception is thrown from the following line:
https://github.com/exhi/hoist-react/blob/develop/svc/FetchService.js#L149
I think since we are setting redirect: 'follow' in our fetch options, we should respect that and not throw on the 3XX Redirect statuses. Response.ok only returns true for 200-299. Currently working around this in a client app by using the fetch api directly.

@TomTirapani   -- @cnrudd  has looked in to this carefully and is pretty sure that the redirects are followed fine, and the problem is with the JSON parsing of an HTML result.
Can you confirm with him and close this?

Having looked more closely at the issue, it seems the problem I was seeing is due to the API not returning a Location header with the 307 redirect. Instead, it responded with the expected content but a 307 status.
With a Location header, fetch will perform a follow up request at the correct resource, so the final response will return a 200 and therefore Response.ok. This is the correct behaviour and we fully support it:

However, without a Location header, the final response will be a 307 and it will fail the Response.ok check.
So I think this was a case of a misconfigured API. I will work with the API team and ask them if they can update the API response.
Apologies for the misdirection :(