Enable the https dev cert for use with Docker for Windows's default networking setup

In Docker for Windows, Docker recommends that, for development purposes, containers can connect to a service on the host using the DNS name host.docker.internal.
This PR changes how the ASP.NET Core https dev cert is generated so that it is additionally valid for that DNS name.

 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.:x: DavidSimner sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@Eilon: the 14 checks are all successful, can you please assign someone to do a code review?

@DavidSimner sorry for the delay.
@javiercn - can you have a look?
@mkArtakMSFT - any thoughts on this?

@Eilon We have an ongoing discussion about it.

@javiercn did this conversation go anywhere? I think this PR is probably pretty stale now unfortunately.

@anurse I'll try to find the conversation, but I don't think we want to add additional DNS names to the cert.

Ok, given that I'll close this. If there is still interest in making changes to our dev certs to support scenarios like this, please open a new issue and we can discuss the options.


bump golang.org/x/sync v0.7.0

Thanks again!
Just curious, are you updating dependencies for any specific feature or vulnerability resolution? Or just being a good citizen to keep things up-to-date?
FYI I'm backporting all these updates to release-2.5 as well to maintain a good project posture and consistency.

The goal is specific, to upgrade smartBFT to the latest. And it was dragging all these libraries with it.
I decided to add them one by one.
Alas, there is no SmartBFT in fabric-2.5. )))


Improve Monitoring for System Services

Our monitoring instance ran out of available disk space unnoticed, and the influxdb process was not running any longer. In the future, this should be monitored to avoid a repetition of this problem.
Before implementing any solution, we should discuss which architecture is desired. Potentially, we could also include Icinga checks or something similar...

In the very specific instance, the monitoring service was, at least, unavailable from September 30th to October 4th. I restored the functionality, but without restoring the data sent during this timeframe.

Interestingly, Poseidon also didn't report an error about failures of sending events to InfluxDB

Just the logs contain Oct  1 00:00:36 poseidon-terraform poseidon[34693]: 2023/10/01 00:00:36 influxdb2client E! Write proc: Retry buffer full, discarding oldest batch

Interestingly, Poseidon also didn't report an error about failures of sending events to InfluxDB

That's because Poseidon reports this error just on Trace-level which is commonly not even stored in the log files.
https://github.com/openHPI/poseidon/blob/cb08787c7d20bbf6ffb710244a9da46b12dd5d91/pkg/monitoring/influxdb2_middleware.go#L77
If we would raise the log-level, we would have received the following output:
2023/11/23 14:03:07 influxdb2client E! Write error: Post "filtered": context deadline exceeded (Client.Timeout exceeded while awaiting headers), batch kept for retrying
time="2023-11-23T14:03:07.919428Z" level=warning msg="Retrying to write influx data..." error="filtered": context deadline exceeded (Client.Timeout exceeded while awaiting headers)" package=monitoring retryAttempts=0

How should we improve this in the future? Should we (ab-)use Sentry for uptime monitoring? Should we somehow connect the InfluxDB unavailability to Poseidon's Health route? Or, should we rely on the recently introduced Icinga checks of the monitoring host itself?

Related to #437


Review Follow/Following Presentation

Describe the bug
Followers and Following counts round at 9.95k to 10k, though 10k should only be displayed when 10k is actually achieved.
To Reproduce
View https://dogehouse.tv/u/benawad or any with x thousand users in followers or following. View true count here: https://api.dogegarden.net/v1/user/benawad or by watching the websocket.
Expected behavior
Don't round up. 9,999 should not be marked as 10k.
Screenshots
N/A
What device are you on?
Chrome, Windows 10
Additional context
If the current logic simply divides by 1000 and rounds to 1 d.p. use the Math.floor() or equivalent function to get the min value.

I can do this, will also write some unit tests for the kFormatter lib.


[Ideas] Open ideas

Seth ideas

Virtual Batching ideas (re eval effects of virtual loss https://github.com/tensorflow/minigo/issues/427)

Only add X from batch of Y to the tree, put the rest in NNCache to use if they get needed later (this is basically a different version of Speculative Execution

Supervised Eval Doc

Rate cut in SL experiments

SL training with rate cut and more steps...
Train some smaller distilled models and test on time parity

Ideas inspired by @lightvector and KataGo

NNCache

Turning off Tree Reuse

Ownernship head
Score distribution Head
Score Maximization ("Score Utility")
Playout oscillation
Forking games (early for diversity, late for Komi, ...)

Ideas inspired by LZ

SWA: Initial Proof Of Concept in #283 but more work needed
Visits "timemanagement" (stopping when 2nd move can't overtake first)

Ideas from AG/AGZ/AZ papers

Gating: Write up in #459
Playing anything early within 1% of best

Ideas from elsewhere

Changing value to 0.95 (AKA 1 - false positive) for selfplay with resign (idea from lightvector)
Active learning (Needs more details) @brilee
(https://medium.com/oracledevs/lessons-from-alpha-zero-part-6-hyperparameter-tuning-b1cfcbe4ca9a))
Cyclic LR

Done

[x] Slow window, not super successful (inspired by [Oracle's Connect Four work]
[x] Q=0 (Sorta tested in V11)
[x] FPU (Handled by https://github.com/tensorflow/minigo/pull/629)

z = z * move_num/length
z = z/2 + q/2
z = z * false_positive_rate in resign disabled games
higher learning rate early

Adding stuff about distillation and Seth ideas

Checking if eval games have enough diversity and using this opening panel
https://github.com/leela-zero/leela-zero/issues/2104

@sethtroisi Re "timemanagement" from LZ, I'm concerned it might be detrimental for self-play and RL, as it amounts to some sort of policy sharpening: cutting the search early means low policy moves won't get any visit and will be trained towards 0. That may hinder the learning of new stuff.
IMHO, the key to spare compute budget might truly be KataGo's variable visits scheme, for game move search vs policy training target search.
And both types of KataGo's search could benefit from the KLD threshold trick from LC0, that sounds very appealing for policy, though much complex to implement ;-)


⚠ Bofrost.at has degraded performance

In 440509f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 914 ms

Resolved: Bofrost.at performance has improved in dabe0c2 after 9 minutes.


Lbrycrd-Compile: Getting a compiler container ready for lbrycrd.

[ ] Compile all of the things.

[ ] Minimal final container step

[ ] Ubuntu 18.04

[ ] Follow current or modified CONTRIBUTING.md guidelines.

More to come as @BrannonKing and @Leopere moves forward with how it all will work.

Laid out the architecture for how to implement this by this commit

Necessary dependencies:
FROM ubuntu:18.04
RUN apt-get update
RUN apt-get install -y build-essential libtool autotools-dev automake pkg-config bsdmainutils python3 git wget apt-utils
RUN apt-get install -y libssl-dev libevent-dev libboost-system-dev libboost-filesystem-dev libboost-chrono-dev libboost-test-dev libboost-thread-dev libboost-locale-dev libboost-program-options-dev
RUN apt-get install -y python-dev libbz2-dev autoconf ca-certificates clang-format
RUN apt-get install -y software-properties-common && add-apt-repository -y ppa:bitcoin/bitcoin && apt-get update && apt-get install -y libdb4.8-dev libdb4.8++-dev


Improved the Environments modal with accessibility fixes

Description
https://github.com/usebruno/bruno/assets/84461672/f52f06c8-6a2a-4409-8d17-88cd69939d59
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

PR looks simple and good to me.
@lohxt1 please review this.


Handling escaped idents which contain periods

@brunorpinho suggested we use backticks for the ClickHouse dialect (https://discord.com/channels/936728116712316989/1030229933299077180). "That'll be super easy" I said. But actually the example presents a more general issue, regardless of the the specific dialect issue.
In SQL, "event.type" is treated as a column name. But it's not currently possible to represent an ident containing a period in PRQL. For example, in:
from github_json
derive [event_type_dotted = `event.type`]

`event.type` will be parsed into an event.type ident in the PRQL AST — i.e. a single term with the period. And then later in the compiler, that term will be resolved into a namespace event and a column type. But we want a column name event.type!

I can see two options:

Change Ident into a proper struct — replace https://github.com/prql/prql/blob/10811eb25c0614fe99ff62b55d2a8b389d3bbfe5/prql-compiler/src/ast/mod.rs#L23 with a Struct that can hold a namespace and column name.
Then:

event.type is parsed into {namespace: "event", column: "type"}
`event.type` is parsed into {namespace: None, column: "event.type"}

...but we use Ident for function names / variables / etc at the moment, and the parser deliberately doesn't have the context to discriminate. So we'll be carrying around a {namespace: None, column: "sum"} for the sum function[^1]

Carry around the backticks where they're supplied in the PRQL query in the Ident struct until we get to the resolver, rather than parse them. So `event.type` stays as `event.type` in the AST, and then when we get to resolving the name, we check for backticks, and produce the column `event.type`.

(I don't think this is viable) Have another type EscapedIdent, which would contain event.type, but knowing that it shouldn't be resolved. But how would `table with space`.column be parsed?

If Ident were always a table name, I would suggest the Struct option, even though it would be quite the mindless refactor. But because it's also used for functions, I'm tempted to go with carrying around the backticks, even though it's some violation of the abstraction (now the name resolver is parsing backticks?!).
Any thoughts?
[^1]: though we could rename the column field to name or term or something

As mush as this is a pain in the ass, but it shows that we are not parsing idents correctly.
Correct way would be to separately parse names in idents that are separated with .. Subsequently, ExprKind::Ident should also contain namespace and varname separately.

Subsequently, ExprKind::Ident should also contain namespace and varname separately.

OK — to confirm, you're OK having sum be parsed into {namespace: None, varname: "sum"}?

Yeah that's ok. The resolver should figure out that sum varname only appears in namespace std and resolve to that.

Closed by #1079 , tested by #1090


xpath in custom parameters

In the custom parameters you should be able to use xpath expressions just like attribute value templates in xslt. This is, you can use curly braces to encapsulate xpath expressions that are evaluated in the context of the XSLT-Conref element and replaced by this result before being passed as parameter to the script.
This allows to combine the definition of an input source with context sensitive information.

solved with release v01.1.0beta


provide a way to configure and name SNAPSHOT releases for the maven-publish plugin

DSL example (using the same conditions as the changes DSL, but only passing the latest commit)
suffixes {
   append "-SNAPSHOT" when {
       branch matches(~/.../)
   }
   append "-SNAPSHOT" when {
       isDirty()
   }
}


Add languages property to Navigator interface

window.navigator.languages is already supported on some mainstream browsers and is ESSENTIAL for frontend to determine which interface language users prefer. For example, on Chrome, navigator.languages[0] indicates the right language option rather than navigator.language.
Please consider adding this property. Thank you!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for your PR. these files are not manually edited, they are auto-generatd from a script in . You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TSJS-lib-generator.
Changes should be done in https://github.com/Microsoft/TSJS-lib-generator instead.
Please find more information at: https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.
For future references, please file an issue first to track changes.

See #16246 which has a working pr


[Flutter Android Studio] Flutter Project can not see the Logcat Window

[Flutter Android Studio] Flutter Project can not see the Logcat Window.

fixed  Android Studio 3.6.1


Unfound source map when importing CSS with webpack

Describe the bug
When importing dist/tui-image-editor.css using webpack:
import 'tui-image-editor/dist/tui-image-editor.css'
We get the following output in the console:
[Error] Failed to load resource: the server responded with a status of 404 (Not Found) (index.css.map, line 0)
https://server.test/packs/css/src/css/index.css.map
To Reproduce
Steps to reproduce the behavior:
import 'tui-image-editor/dist/tui-image-editor.css'
Expected behavior
I'd expect the source map to load correctly. It appears that webpack is unable to find it.
Additional context
Removing the source map declaration would fix this problem. There's probably a more elegant way though.
/*# sourceMappingURL=src/css/index.css.map */

@brendon
Thank you for reporting.
If you are interested, please also PR.

I'd love to but I can't figure out how the source map line is being generated. There doesn't seem to be an equivalent in the source code so I assume it's generated by some build process before release?

@brendon
I'm sorry for the late reply. This may be due to a previous change in the webpack configuration. I'll figure out the cause and resolve it. :)

Thanks @lja1018  :D Let me know if there's anything I can do/test on this end :)


FR: Get-DbaSQLPatches

Name: Feature request/idea 💣 🥊
About: Suggest a new feature or enhancement
Summary of new feature

Retrieves a list of SQL Patches (SP/CU/Hot-fixes) on a Computer and the instances on it.

Proposed technical details (if applicable)

https://github.com/hfleitas/Monitoring
Get-SQLUpdates.ps1

Latest version of dbatools as of writing

0.9.808

Cool idea! Thanks so much for submitting the issue 💯


date updates

Fixes
Please link to the issue this PR addresses
Description
better ISO 8106 support
Proposed changes in this PR
Things to look at

[x] Test coverage
[x] Code Style
[x] Documentation (READMEs etc)

@ms1111 I think we'll go the route of having authors write their strategies, the two we provide for dates is more than enough IMHO.


with_associations and Kaminari-neo4j not working together

I'm having issues when running with_associations and kaminari together.
I have tried this query:
Article.all.with_associations(:creator).page(1).to_a

The page method comes from kaminari-neo4j gem.
I'm getting this error:
Neo4j::Session::CypherError: Invalid input 'K': expected 't/T' or 'e/E' (line 1, column 84 (offset: 83))
"MATCH (n:`Article`) OPTIONAL MATCH n-[:`creator`]->creator WHERE (creator:Person) SKIP {skip_0} LIMIT {limit_25} RETURN n, [collect(creator)]"

The generated query:
MATCH (n:`Article`) OPTIONAL MATCH n-[:`creator`]->creator WHERE (creator:Person) SKIP {skip_0} LIMIT {limit_25} RETURN n, [collect(creator)] | {:skip_0=>0, :limit_25=>25}

I'm not sure if it's a with_association bug or something related with kaminari
It seems like with_associations is pulling the return statement in the bottom of the query.
Any ideas?
@cheerfulstoic

Looks like there's a break in the code for with_associations.  I know I put it in there for a reason so I need to track that down ;)
Heading out to lunch now, but I'll look again shortly.  Thanks for the report!

Ok, so this is actually kind of tough... With with_associations the OPTIONAL MATCHes aren't added to the chain until you do the enumeration (the to_a in your case).  I did this to copy ActiveRecord, but I'm thinking maybe it's a bad idea and that we should build the OPTIONAL MATCH parts at the time you call with_associations.
@subvertallchris do you see a problem with that?  Your feedback is welcome too @ProGM  ;)

I know I already said on the previous issue with with_associations, but I think that adding a with statement before any call of optional match could be a solution.
Example:
MATCH (n:`Article`) WITH n SKIP {skip_0} LIMIT {limit_25} OPTIONAL MATCH n-[:`creator`]->creator WHERE (creator:Person)  RETURN n, [collect(creator)] | {:skip_0=>0, :limit_25=>25}

But, as you said, I'm not sure if this has some implication on performances....

@cheerfulstoic @subvertallchris
Any news?
with_associations is currently broken for 80% of use cases I tried, since limit, page, optional_match and sometimes where are breaking the generated query.
I think that eager load is quite a critical feature if someone wants to use neo4j in production.
Maybe to avoid problems, an options is using two queries instead of one.
Example
MATCH (n:`Article`) RETURN n.uuid SKIP {skip_0} LIMIT {limit_25} <-- OR any other complex query here

MATCH (n:`Article`) WHERE n.uuid IN {article_ids} OPTIONAL MATCH n-[:`creator`]->creator WHERE (creator:Person)  RETURN n, [collect(creator)] <-- it's easier to control this query

What do you think about it?

I agree that it's an important feature.  And actually, yeah, a second query could make some things a lot simpler.  It could still load multiple associations in the same query which would be better than AR.  It's frustrating because I feel like there's probably a solution but I haven't had a chance to sit down and think about it.  Life has been quite busy lately....

@cheerfulstoic I see.
I'm quite new to neo4j, so I don't think I can really help with this...
When you have time please take a look. If I could help, tell me :)
In the meanwhile, I created an horrible patch to make it work temporary.
If someone needs, simply add this in config/initializers:
module QueryProxyEagerLoadingPatch
  def each(node = true, rel = nil, &block)
    return super if with_associations_spec2.size.zero?

    query_from_association_spec2.pluck(identity, with_associations_return_clause2).map do |record, eager_data|
      eager_data.each_with_index do |eager_records, index|
        record.association_proxy(with_associations_spec2[index]).cache_result(eager_records)
      end

      block.call(record)
    end
  end

  def with_associations_spec2
    @with_associations_spec2 ||= []
  end

  def with_associations_return_clause2
    '[' + with_associations_spec2.map { |n| "collect(#{n})" }.join(',') + ']'
  end

  def with_associations2(*spec)
    invalid_association_names = spec.reject do |association_name|
      model.associations[association_name]
    end

    if invalid_association_names.size > 0
      fail "Invalid associations: #{invalid_association_names.join(', ')}"
    end

    new_link.tap do |new_query_proxy|
      new_spec = new_query_proxy.with_associations_spec2 + spec
      new_query_proxy.with_associations_spec2.replace(new_spec)
    end
  end

  private

  def query_from_association_spec2
    with_associations_spec2.inject(query_as(identity).return(identity)) do |query, association_name|
      association = model.associations[association_name]

      query.optional_match("#{identity}#{association.arrow_cypher}#{association_name}")
        .where(association.target_where_clause)
    end
  end
end

Neo4j::ActiveNode::Query::QueryProxy.include QueryProxyEagerLoadingPatch

And call with_associations2 when you need to query with limit or page.

Hey @ProGM could you try a branch for me?
In your Gemfile you could try:
gem 'neo4j', github: 'neo4jrb/neo4j', branch: 'with_associations_fix_70'

I believe that you're using the 7.0 rc which is why I specified that branch.  If you're using 6.1.x you should try the with_associations_fix_70 branch
This puts in a WITH, but it also separates all of the OPTIONAL MATCH clauses with WITH because if you have multiple associations specified and at least one of them has multiple results you'll get a cartesian product.

This puts in a WITH, but it also separates all of the OPTIONAL MATCH clauses with WITH because if you have multiple associations specified and at least one of them has multiple results you'll get a cartesian product.

Uhm, I see... I didn't noticed it! If you have time, can you write me an example of this behavior? I'm very interested in it.
However, I tried this branch, it seems to work quite good :+1:
I did just a couple of tests, but it works well with both base queries:
Article.all.with_associations(:creator).page(1).to_a

But also more complex:
Blog.all.articles.branch { topic }.with_associations(:creator).page(1).to_a

And it seems to work, too. :)

Excellent, thanks for checking it!
So here's an example:
  MATCH (asset:`GraphGist`:`Asset`)
  WHERE (asset.featured = {asset_featured})
  OPTIONAL MATCH asset-[:`FOR_INDUSTRY`]->industries
  WHERE (industries:Industry)
  OPTIONAL MATCH asset-[:`FOR_USE_CASE`]->use_cases
  WHERE (use_cases:UseCase)
  RETURN
    asset,
    [collect(industries),collect(use_cases)] | {:asset_featured=>true}

All of MATCH (including OPTIONAL MATCH) queries which are next to each other will work together to form a single match pattern.  So if my GraphGist has two Industry nodes and one UseCase node it will match:
gist, use_case, industry1
gist, use_case, industry2
So that when we do collect(use_case) we end up with duplicates.
This branch makes it work more like:
  MATCH (asset:`GraphGist`:`Asset`)
  WHERE (asset.featured = {asset_featured})
  WITH asset
  OPTIONAL MATCH asset-[:`FOR_INDUSTRY`]->industries
  WHERE (industries:Industry)
  WITH asset, collect(industries) AS industries_collection
  OPTIONAL MATCH asset-[:`FOR_USE_CASE`]->use_cases
  WHERE (use_cases:UseCase)
  WITH asset, industries_collection, collect(use_cases) AS use_cases_collection
  RETURN
    asset,
    [industries_collection, use_cases_collection] | {:asset_featured=>true}

Ideally the industries and use_cases variables might be industry and use_case, but that would be a larger change and there's an argument to be made to making sure the variable name matches the association name.

Oh, I see. This makes sense!
So, this bug with cartesian product was in with_associations also in the previous (buggy) version of it?
Definitely, this fix looks great.

Great, thanks!  I've pinged a couple of other people who've had issues so I'm going to wait for at least one more confirmation, then I'll throw it into 7.0.0.
I'm a bit tempted to make a patch for 6.1.x, though it's definitely a potential breaking change in the way that queries work when you use with_associations.  If we have 7.0.0 out there maybe that can just be the way that we can offer for people to fix it.
Thanks again

I think that 7.0.0 fixes this. Let me know if not!


Reset Ready condition when update failed

This ensures the Ready Condition is reset when CR becomes once ready but something fails during reconfiguration.

LGTM

LGTM


Feature/add search

Coverage decreased (-1.2%) to 98.072% when pulling f4ccb890ec181d99c84543f56b2dce7a7cf48062 on onfocusio:feature/add-search into 0ffa13df39130c6597f6a7c671e84319ece173f7 on go-goyave:master.

Thank you ! 🎉


Add РљРёС‚С from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of SpaceKraken, to add РљРёС‚С to CKAN.
Mod details:
name = /mod/1026/%D0%A0%D1%99%D0%A0%D1%91%D0%A1%E2%80%9A%D0%A1
author = SpaceKraken
abstract = Adds РљРёС‚С into your Kerbal Space Program
license = MIT
Homepage = РљРёС‚С
description = Adds РљРёС‚С into your KSP. You can find РљРёС‚С at "РљРёС‚С" part category.
Please direct questions about this pull request to SpaceKraken.

404


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:27: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:28: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_eek5ln5te_/clone/recipe/meta.yaml:42: missing_pip_check: For pypi packages, pip check should be present in the test commands
Errors were found


Fix duplicate id attributes for EPUB and PDF

Ensure that the transformed output for EPUB and PDF generation does not have heading or section elements with duplicate id attributes.
Fixed

Uses slugify on the page title when setting a value for h1 id attributes.
Corrects misspelling in JSDoc comments.

@mphstudios I moved the ID de-duplication logic into a standalone dedupeElementIds.js module used by outputs/pdf/write.js and outputs/epub/transform.js


SAK-46341 Fix 'Email' Tool

@victorfv2 Fix email tool is a very uninformative description can you please update

Related to #9746 changes

@ern When are you planning to cut the branch? Thanks

Miguel this needs to be rebased as it can't fast forward


UICKeyChainStore: rename TestHost target to UICTestHost.

The TestHost target has the same name as the one we use in Lightricks.
This casused some builds to fail as Xcode cannot differentiate between
the targets.

@barakwei consider renaming Lightricks TestHost to LTTestHost in order to avoid such collisions in the future...

Definitely
On Mon, Mar 25, 2019, 00:38 Daniel Lahyani<EMAIL_ADDRESS>wrote:

@barakwei https://github.com/barakwei consider renaming Lightricks
TestHost to LTTestHost in order to avoid such collisions in the future...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Lightricks/UICKeyChainStore/pull/8#issuecomment-476008220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_WIbDtLMUkSvcNu8Y8kL3Wy3-bO41Lks5vZ_5XgaJpZM4cFeuR
.


[BUG] roblox wont open

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.8.2
What problem did you encounter?
it doesnt work i click several times cant open
Bloxstrap Log
N/A

reinstall or restart your roblox everytime until it works

@pizzaboxer


Bug fix/internal issue #616

Rest and V8 analyzers interfaces changes:

REMOVE: accepts only analyzer names without database prefix to forbid deleting analyzers  in
database not  in current context (this makes REMOVE request consistent with POST
request, which only allows operations on current database)
GET: now returns only  analyzer from current or from _system database (to be consistent with
corresponding get analyzers list request)

Additionally for V8 interface removed redundant (and expensive!) sorts for returning views and analyzers lists.

For ArangoSearch link creation added check for analyzer comes from the same database as view (or from _system database).

Added tests for fixed issues

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/5509/

Apart from that I would prefer to add sanity check that StaticStrings::SystemDatabase == sysVocbase->name()

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/5510/

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/5512/

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/5515/


Added logic to avoid comparisons with boolean true/false values

IMPORTANT: By submitting a patch, you agree that your work will be licensed under the license used by the project.
If your pull request does not pass CI, it will not be merged (unless heavily justified).
Description
Please provide a description of what this PR is meant to fix, and how it works (if it's not clear from the code changes).
Tests
Please list all major test changes. All issues should have a test created to address the issue in the future (as relevant).
If closing issues, please write Closes #ISSUE_NUMBER[, #ISSUE_NUMBER2, ...], to automatically close the equivalent issue.
Note: This template is a guide, remove sections as applicable.

Forgot to push new .ump test file.

I am going to merge this PR. Please next time make sure that you. Add the issue number to the description and also  give a proper description (delete the template and replace it by what the PR actually does).

Oh right sorry. I'll definitely do that next time, my bad! And I'll update this one when I get a chance.

@tlaport4  would mind let me know this is related to which issue? we need to close it


version 2.x  converts regular expressions to objects when cloning

Thanks for sharing merge. When upgrading from v1 to v2 I found the following bug. Bit confused as what could have caused it as the clone code doesn't seem to have changed greatly (other than being converted to TypeScript)
const merge = require('merge');
const result = merge.recursive(true, { regex: /.*/ } , {});
console.log(result);

v1.x
{ regex: /.*/ }

v2.x
{ regex: {} }

https://github.com/yeikos/js.merge/blob/master/src/index.ts#L55-L57 is the problem.

@cressie176
I realize this is slightly off-topic, but are you aware of any other breaking changes in 2.x? I'm thinking of updating this dep in a few places. Nothing was documented that I can see, and it doesn't appear there were any intentional changes to the API with the Typescript migration, but I thought I'd ask around to be safe.
@yeikos care to weigh in?

This bug recently bit me and required more than a day of sleuthing before I figured it out. Hard to believe it is still an open issue more than 18 months after getting reported, especially since @cressie176 pointed out where the problem lies.

Closed via e9bea183aa374810ab2262639e303eb0500e6790.


Same IP on Youtube an error

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
When making more requests over the same IP on Youtube, an error like this occurs, what could be the solution?
ERROR: [youtube] ty5stpJiApE: Unable to download API page: <urlopen error [Errno 111] Connection refused> (caused by URLError(ConnectionRefusedError(111, 'Connection refused')))
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
ERROR: [youtube] ty5stpJiApE: Unable to download API page: <urlopen error [Errno 111] Connection refused> (caused by URLError(ConnectionRefusedError(111, 'Connection refused')))

ERROR: [youtube] hIrnk8yaVVo: Requested format is not available. Use --list-formats for a list of available formats
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/yt_dlp/YoutubeDL.py", line 1518, in wrapper
return func(self, *args, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/yt_dlp/YoutubeDL.py", line 1615, in __extract_info
return self.process_ie_result(ie_result, download, extra_info)
File "/usr/local/lib/python3.8/dist-packages/yt_dlp/YoutubeDL.py", line 1674, in process_ie_result
ie_result = self.process_video_result(ie_result, download=download)
File "/usr/local/lib/python3.8/dist-packages/yt_dlp/YoutubeDL.py", line 2743, in process_video_result
raise ExtractorError(
yt_dlp.utils.ExtractorError: [youtube] hIrnk8yaVVo: Requested format is not available. Use --list-formats for a list of available formats
Download error occurred: ERROR: [youtube] hIrnk8yaVVo: Requested format is not available. Use --list-formats for a list of available formats

Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

You didn't do this ^. You also didn't bother to elaborate on your issue any further.
It is most likely a network/connection issue on your end, or a problem with VPN/proxy if you are using one. But I wouldn't know, because you are not providing enough information.
Closing, can reopen if valid reporting information is provided


Add preferPathResolver option

Depends on #137
Let stylus-loader users opt for webpack resolving before stylus. This
can allow for most common stylus files to be found in parallel through
webpack's async resolver instead of blocking on file IO with stylus's
sync resolver.

This is something I've tried to do for a looooooong time. But I was trying to do it by opening up resolving through an enhanced-resolve Resolver that webpack uses. I haven't been able to get that working and instead of holding off on this idea any longer I think I'll add this option to at least let stylus-loader users resolve with webpack first.
History note: Its the other way before this PR because before PathCache was added to stylus-loader it only resolved with stylus. Since stylus is wholly synchronous and webpack's resolver is asynchronous we opted for preferring stylus resolving to let users have an easy upgrade path. We didn't think about that being an option to toggle.
This has become more interesting to me to consider since I've been recently doing lots of research into speeding up build time in webpack.


I had some serious issues when using this in an SDK

You should read this: https://blog.newrelic.com/2014/04/16/right-way-to-swizzle/

I'm not 100% sure I agree with "If you want to swizzle, the best outcome is to leave no trace". Certainly it's necessary in some cases, but I'd like the hint something's been swizzled otherwise.
I'd probably accept a Pull Request that added a jr_swizzleClassMethodKeepingCmdName/jr_swizzleMethodKeepingCmdName alongside the existing jr_swizzleMethod/jr_swizzleClassMethod pair. That way folks can use it if necessary, but it's off by default.

What about the issue mentioned here? http://defagos.github.io/yet_another_article_about_method_swizzling/

Please don't just toss links at me. You are being inconsiderate of my time.
Instead, please make concrete suggestions. Ideally in the form of reproducible bug reports or Pull Requests.

Quite interesting how you deal with a 5mins article after I was pulling my hair off for hours trying to understand why my SDK conflicts with other SDKs which are swizzling too, and send you links with concrete possible issues.
You find it considerate of my time to spend time on fixing/testing the issue and create a pull request because "you'd probably" accept it?
No matter how you feel about it, I won't re-describe the entire issue here in images as in the article, but the part that should interest you has the heading "Issues in class hierarchies" and comtains this sentence "No swizzling implementation I encountered correctly deals with this issue, not even JRSwizzle.".
You would benefit from reading the entire article though.
Now I won't be wasting more of my and your time on trying to explain to you that your lib works fine only when there are no other libs used that swizzle and instead work on a solution that doesn't mess up or doesn't get messed by other libs/SDKs.

Hey @ashottonoyan,
I should have realized if you knew about these articles and are providing references, you are a skilled programmer and want to make JRSwizzle better.
I'm sorry that I miscategorized your Issue and I'm sorry JRSwizzle didn't live up to its claim of being your one-stop-shop for swizzling.
If you want to help out making JRSwizzle fulfill its goal (with credit to you, of course) I'd be very grateful for the help.


Returning richer errors in operations

A pattern we've used in the past that can be very beneficial is explicitly returning errors in failed operations. An optional extension is to also be able to return warning messages in "successful" operations for informational or actionable messaging (e.g., something being used is deprecated and should be updated).
Problem
Currently the operation object has two fields, state and description, and relies on description to provide any message flow of errors or warnings. This has a few possible problems:

Not explicit that description contains error/warning messaging
Does not support multiple errors/warnings
Does not support rich error/warning message metadata

Proposal
I'd like to suggest the introduction of an explicit field for error messages, and if we choose, a similar field for warnings.

Response field
Type
Description

state*
string
Valid values are in progress, succeeded, and failed. While "state": "in progress", the platform SHOULD continue polling. A response with "state": "succeeded" or "state": "failed" MUST cause the platform to cease polling.

description
string
A user-facing message displayed to the platform API client. Can be used to tell the user details about the status of the operation. If present, MUST be a non-empty string.

errors
array-of-objects
An array of JSON object containing error information about the operation. Only present when operation "state": "failed".

warnings
array-of-objects
An array of JSON object containing warning information about the operation. Only present when operation is completed.

The field(s) are arrays to allow for multiple distinct messages to be returned, and are objects to allow for richer metadata We would want to think through the actual structure of the objects, or whether it's sufficient to just be arrays of strings.

Why not group the errors and warnings into a single array-of-objects?

What would that field be called? How would you differentiate the two in the event that both are surfaced from the same operation?

@bmelville Should the arrays of errors and warnings be arrays of strings? If we expect platforms to do something with these fields we should standardise what they look like. Arrays of strings makes most sense to me for now.

String is definitely the easiest first step. We've seen situations where an error may benefit from more structure as well, so one possibility that leaves the door open to extensibility later is to have something like this:
type Operation {
  Errors []OperationInfo
  Warnings []OperationInfo
}

type OperationInfo {
  Message string
}

An operation response would look like:
operation:
  errors:
  - message: First error encountered.
  - message: There was also a second error...
  warnings:
  - message: ...and also a warning.

I support the OperationInfo style structure with a string message. A field that comes to mind as worth adding is an enum value indicating what type of error/warning it is, ideally from some canonical set.
@jmrodri - combining errors and warnings into a single array-of-object (and adding a severity field - error, warning, ...) would have the downside that to find error the client needs to scan the whole array. That is why I would advocate for the 2-field solution (errors, warnings) that @bmelville sketched above (even if they are otherwise functionally equivalent)

@martinmaly that makes sense.

It's almost been 2 years since this came up - is this still top of mind for any folks or can we close it?

@mattmcneeney I'd vote close


Porting the code to C#

Hello and congratulations on your fine work.
Is there any chance this can be ported to C#? That could simplify it's widespread use a lot.
Thanks.

Currently,we do not support C#.


Feature Request: make selectors amenable to GPU processing

as requested by @Luthaf

So for more context on this: it would be nice to be able to pass a torch.Tensor (or jax array) living on GPU directly to the selectors, instead of having to move the data back to main CPU memory.
A first pass would be to make sure all the function calls are compatible with PyTorch API, but given the high usage of Python for loops in the selector code that might not give a lot of performance improvement. The second step would then be to rewrite the selector code to use more high-level operations & launch larger GPU kernels, and hopefully improve performance.

This is mostly unrelated to the autograd part of PyTorch, so even if we need to .detach() the tensors before passing them, that would be fine with me. I would mostly like to be able to keep the data in GPU memory.

My ideal user-facing interface for this would be to be able to do something like this:
import torch
from skmatter.feature_selection import CUR

X = torch.rand(300, 300, device="cuda")  # or device="mps" on Apple M1/M2

selector = CUR(n_to_select=4)
selector.fit(X)

Xr = selector.transform(X)
# Xr is a torch tensor, with device=X.device

A first step for this would be to add a test trying to use skmatter with a torch tensor, and check where the code starts throwing errors.

Depending on the number of function call (e.g. np.sum, …) that need to be updated, it might be interesting to use https://github.com/jcmgray/autoray to dispatch function calls to the right backend.

This is put on the back burner for now, if you are interested in getting skmatter to run on GPU please voice your interest here!

It looks like sklearn now has experimental support for PyTorch (and GPU data) using the array API: https://scikit-learn.org/stable/modules/array_api.html. We could use the same here!

We should experiment as well how the array api works with our selection methods. FPS is probably a good candidate because we do not use very complicated mathematical operations there. So hopefully there is not so much friction in making this work.

More info on this array API in sklearn: https://labs.quansight.org/blog/array-api-support-scikit-learn.


API endpoint to retrieve current user

Something akin to /v1/users/current would be a nice QoL feature for consumers of the API.
Likewise, an endpoint to do nothing more than validate authentication would be incredibly useful for login flow. /v1/authenticate or the like returning nothing more than {"user_id": 0} would be amazing. Errors could include {"error_message": "No such user"} and {"error_message": "Invalid password"}.
As it is, I am forced to make a call to /v1/feeds/0 to validate authorization with minimal side effects or bandwidth usage.

PR #127 has been merged.


[1.12] Helmet Probes Appearing in all Creative Tabs & Creative Inv Trash Slot

Hello @McJty,
It appears there's a minor bug in Minecraft 1.12 with The One Probe mod. Here are further details surrounding the issue:
Issue Information

Minecraft 1.12
Minecraft Forge <IP_ADDRESS>1
The One Probe 1.2-1.4.7

Issue Description
The One Probe's helmets appear in all of the vanilla creative tabs at the very bottom. This seems like a very weird issue and perhaps one with Forge itself given the beta state of Forge at the current time, but I thought best to report it to yourself first.
Issue Media

P.S. I was tempted to upload a screenshot of each tab but I think three proves the point. 😜

Fixed via 57daee47b360beb5cd4cc1cd738fcb9d0adf75e8


time format

Hello, super nice Programm. One question, how can i change the time format in top of the Picture from American to Europe format. Exemple 04/20/21 to 20/04/21

Change the format on this line https://github.com/thomasjacquin/allsky/blob/master/capture.cpp#L93
and run make capture to recompile.

I made changes in line 93, then in the allsky folder I did sudo make capure and restarted raspberry, but nothing has changed

Did you run make capture to recompile the program?

I did sudo make capure

Ok, there's no need for sudo when compiling the program but that shouldn't change anything.
Are you using a ZWO camera or a RPiHQ camera?

a RPiHQ camera

In that case, the date is the output of the raspistill -annotate option and I don't think we have control over that.

https://www.raspberrypi.org/documentation/raspbian/applications/camera.md
https://man7.org/linux/man-pages/man3/strftime.3.html
-a 8 -a "ABCD %Y-%m-%d %X %Z" works for me

I modified capture.cpp to accept a date/time format, and modified the admin interface to prompt for the format.  Works very well; you can then change the format without having to update the .cpp file.
How can I propose these changes become part of the official software?

Hi Eric, you can create a Pull Request and I'll review the code and merge it into the master branch. Thanks

Thomas,
I have a bunch of bug fixes and enhancements. Should I put them all in the file in the pull request, or do separate requests for each feature?  I also put a comment with my initials (ECC) so it would be obvious what I changed.  Is that necessary?
Question:  the getTime() function checks if the seconds are the same as the last call, and if so, returns 0.  What's the reason for that?  getTime() is only called once, and one of my changes includes calling it a second time, so I had to create a version of getTime() that didn't do the check.
Eric

From: Thomas Jacquin @.>
Sent: Sunday, June 13, 2021 11:33:24 AM
To: thomasjacquin/allsky @.>
Cc: EricClaeys @.>; Comment @.>
Subject: Re: [thomasjacquin/allsky] time format (#352)
Hi Eric, you can create a Pull Request and I'll review the code and merge it into the master branch. Thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/thomasjacquin/allsky/issues/352#issuecomment-860238185, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AT2PYK7LD52B6KJ2R3LIUHLTSTMVJANCNFSM43HZWR7Q.

capture now includes a -timeformat flag which can be used to format time in an arbitrary format.


replace yaml # comments with LinkML comments etc.

If it's not clear what element is being commented on, apply the comment to the schema as a whole

Other usage patterns:

commenting out a definition or assignment in advance of deleting
informal section headers that should really be replaced with subsets, or something like that


add Method & StatusCode to azure_core

This removes the last dependency on the http crate by adding a Method and StatusCode types.  I'm interested in having compatibility with http-types and supporting surf in addition to reqwest. This draft wraps http-types implementations of Method and StatusCode, but exposes the API from http crate that we are using. There are a few different ways we could go here. Looking for feedback.

We'll discuss this in our next meeting. A purpose of this PR was to show the differences between http and http-type implementations and start a discussion. I prefer http-types representations of both Method & StatusCode as enums.
https://docs.rs/http-types/latest/src/http_types/method.rs.html
#[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
pub enum Method {

https://docs.rs/http-types/latest/src/http_types/status_code.rs.html
#[repr(u16)]
#[derive(Debug, Clone, Copy, Eq, PartialEq, Hash)]
pub enum StatusCode {

http instead has consts for all the values.
#[derive(Clone, PartialEq, Eq, Hash)]
pub struct Method(Inner);

enum Inner {
    Options,
    Get,

https://docs.rs/http/latest/src/http/status.rs.html
#[derive(Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]
pub struct StatusCode(NonZeroU16);

Differences

The biggest change needed for http-types is error the need for its error type to implement std::error::Error #848 https://github.com/http-rs/http-types/pull/395.
http::StatusCode has a fn as_u16, http-types is missing it and uses as u16 everywhere. The convenience function would be nice.
http::StatusCode has a default StatusCode::OK. Some of our Default implementations use this. Should be easy to figure out.
Because http-types uses an enum instead of constants, the name casing is different. Please BAD_GATEWAY with BadGateway and the 7 other status codes we use. This is just a few global replaces.

I created #852 with a subset of changes that leave http in place for now.


Add display driver support for st7735s

Describe the solution you'd like
I am trying to test st7735s display on esp32c3 board and it seems on st7735r driver is integrated.
Describe alternatives you've considered
Use st7735r driver in a smaller resolution.
Hi,
I am trying to enable ST7735s display on ESP32c3 and about a week experience on the board & zephry.
I have successfully run some sample via serial console(blinky, thread, wifi).
I have connected five pins(gnd, vdd, cs, clk, sda) between the boards and the display, and it seems that I need to select a GPIO for 'DC' at least.
Appreciate if anyone can tell what I need to bring it up. Ideas that I got but don't know how to include:
1. Include st7735r driver in my build? How to?
2. Select a PIN for 'DC' Pin, 'GPIO11' here.
3. Mangle the pin configuration like below:
--- a/boards/riscv/esp32c3_devkitm/esp32c3_devkitm-pinctrl.dtsi
+++ b/boards/riscv/esp32c3_devkitm/esp32c3_devkitm-pinctrl.dtsi
@@ -23,12 +23,13 @@

     spim2_default: spim2_default {
         group1 {
-            pinmux = <SPIM2_MISO_GPIO2>,
-                 <SPIM2_SCLK_GPIO6>,
-                 <SPIM2_CSEL_GPIO10>;
+            pinmux = <SPIM2_MISO_GPIO3>,
+                 <SPIM2_SCLK_GPIO2>,
+                 <SPIM2_CSEL_GPIO7>;
+                 <SPIM2_DC_GPIO11>;
         };
         group2 {
-            pinmux = <SPIM2_MOSI_GPIO7>;
+            pinmux = <SPIM2_MOSI_GPIO3>;
             output-low;
         };
     };

4. Test. How to?
5. What else?

PIn layout:

Discord discussion:
https://discord.com/channels/720317445772017664/720317445772017667

I tried to compile it with the 'lvgl' sample and got the following error, what I am missing?
It seems that esp32c3 has not been add added to the board list?
west build -p always -b esp32c3_devkitm  samples/subsys/display/lvgl/

in file included from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/toolchain/gcc.h:92,
                 from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/toolchain.h:50,
                 from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/sys/time_units.h:10,
                 from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/sys/util.h:615,
                 from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/devicetree.h:25,
                 from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/device.h:12,
                 from /data/2023/src/esp32c3/zephyr-westbuild/zephyr/samples/subsys/display/lvgl/src/main.c:7:
/data/2023/src/esp32c3/zephyr-westbuild/zephyr/samples/subsys/display/lvgl/src/main.c: In function 'main':
/data/2023/src/esp32c3/zephyr-westbuild/zephyr/include/zephyr/device.h:85:41: error: '__device_dts_ord_DT_CHOSEN_zephyr_display_ORD' undeclared (first use in this function)
   85 | #define DEVICE_NAME_GET(dev_id) _CONCAT(__device_, dev_id)


Show list of possible experiences

keep in mind that there might be other team want to join, so the code must be open for other to contribute/add their own link.


Remove ai-interviewer folder

Fixed in https://github.com/rasikaghadge/AI-Automated-Interview-Platform/pull/122


Give Kevin 1,000,000 dollars

SP
🤣

he deserves it, because he uses python as programming language over node/ts/js

@mishraomp You must be thinking of a fine.  Let's ask Trump to put tarifs on him!  :D + :P

he deserves it, because he uses python as programming language over node/ts/js

haha... this is why the ticket wasn't assigned to Dr. Mishra!

check for 2 trillion on your way @franTarkenton

I'd call this a successful first issue!  🤣 🍻


Wait for ongoing uploads and downloads when shutting down the worker

This PR makes the worker shut down gracefully and allows ongoing uploads and downloads to finish.

Have you manually tested this a few times to see if it throws any unexpected errors?

 Downloads and uploads are handled gracefully, and we do reserve a portion of the shutdown ctx to try and shut down the remaining services in case we time out. If the worker shut down times out though, we might not record some interactions and then we get:
2024-02-14T10:20:50+01:00       ERROR   worker.worker   worker/interactions.go:91       failed to record 1 price table updates on worker shutdown

Converting it to DRAFT to prevent merge, ran into a 2024-02-14T10:40:07+01:00       ERROR   uploadmanager   worker/contract_lock.go:109     failed to send keepalive testing a shorter shutdown timeout.

This was never scheduled and is not currently a priority.


Make elastic timeout configurable for HorovovJob.

Tracking issue

Why are the changes needed?
The default elastic timeout to wait for worker pod to be ready is 60s https://github.com/horovod/horovod/blob/55a73d4f662bf637f6994a61f62b517ad9554dc1/horovod/runner/launch.py#L452C32-L452C51
This PR is to make it configurable

What changes were proposed in this pull request?
Changes:

Increased elastic_timeout to 20 min;
make elastic_timeout configurable by user

How was this patch tested?

Setup process
Screenshots
Check all the applicable boxes

[ X ] I updated the documentation accordingly.
[ X ] All new and existing tests passed.
[ ] All commits are signed-off.

Related PRs

Docs link

updated. @eapolinario

@supercharleszhu Hi, before this is merge, can you provide screenshot with this branch's commit hash at remote execution?
this is my example: #1870

@Future-Outlier we are from LinkedIn and committed and tested in our internal branch. The commit hash is not sync with OSS. Let me know screenshot like this make sense or not? (masked some of the internal settings)


feat(ethers-solc): configurable build-info output dir

Motivation
I would like to be able to configure the output directory of the hh style bulidinfo files so that tooling that expects the build info files to be in a specific directory can just work.

Solution
Allows for the build info output directory to be configured. It should be backwards compatible by using the same directory that it currently uses by default. Will require a follow up PR in foundry to pass through the config value

PR Checklist

[x] Added Tests
[ ] Added Documentation
[x] Updated the changelog

this is great!
we can follow up with a corresponding config value in foundry.toml
the way that works is, the Config struct contains all settings variables: https://github.com/foundry-rs/foundry/blob/master/config/src/lib.rs#L90-L108
which are picked up by foundry.toml, env, cli args ...
and are then used to create the ethers-solc::Project here
https://github.com/foundry-rs/foundry/blob/master/config/src/lib.rs#L528

Thanks for the tips! Will open PRs in foundry and the hh plugin once this version of ethers is pulled into foundry


feat: mark inset*, marginBlock*, marginInline*, paddingBlock* and paddingInline* props as built-in support

This layout props have been implemented in main (default as of 0.76).
Check 46478 for more information.
Should we continue to test these props?

I'm getting this coverage error:
Jest: "global" coverage threshold for branches (89%) not met: 88.79%

We haven't been using the built in support because all logical styles aren't fully implemented in RN yet, and we had problems in the past with missing desktop support. I'll ask around and see if we can start to drop some of the polyfills

Looks like a miss the direction prop. Now is supported on both platforms.
See  https://github.com/facebook/react-native/blob/main/packages/react-native/Libraries/StyleSheet/StyleSheetTypes.d.ts#L114


Proper way to install latest Postgres?

Hello,
Sorry for the simple question but I couldn't find the answer in the docs.
I noticed that the master catalog now supports Postgres 11. What is the proper way to install kubedb to install PG 11? Do I need to also use the master branch of the cli?
Thank you.

Yes, I was looking for this as well. I was trying to build images myself from https://github.com/kubedb/postgres/tree/master/hack/docker. You can build you on images upload them to some docker registry and create a custom PostgresVersion pointing to your own images. But smth went wrong while I was trying to make a docker image for Postgres 11.1 and I wait for official release. Would great if there could be a way to test latest master as well.

With 0.10 now out, closing this.


[Bug]: docker build error

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

RAGFlow workspace code commit ID
e267a026f36616bfda7f70eaa9b46a61c7cc3415
RAGFlow image version
infiniflow/ragflow:dev latest version
Other environment information
NAME="Ubuntu"
VERSION="20.04.6 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.6 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

Actual behavior
git clone https://github.com/infiniflow/ragflow.git
cd ragflow/
pip3 install huggingface-hub nltk
python3 download_deps.py
docker build -f Dockerfile -t infiniflow/ragflow:dev .

I am following the above process and deploying ragflow:dev locally
When I ran to docker build, the script gave me an unexpected error. I couldn't find it in an existing issue
Expected behavior
No response
Steps to reproduce
git clone https://github.com/infiniflow/ragflow.git
cd ragflow/
pip3 install huggingface-hub nltk
python3 download_deps.py
docker build -f Dockerfile -t infiniflow/ragflow:dev .

Additional information
[+] Building 74.6s (13/40)                                                                                docker:default
=> [internal] load build definition from Dockerfile                                                                                0.0s
=> => transferring dockerfile: 6.12kB                                                                                0.0s
=> [internal] load metadata for docker.io/infiniflow/ragflow_deps:latest                                                                                3.9s
=> [internal] load metadata for docker.io/library/ubuntu:22.04                                                                                1.6s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> [internal] load build context                                                                                0.1s
=> => transferring context: 70.64kB                                                                                0.1s
=> CACHED FROM docker.io/infiniflow/ragflow_deps:latest@sha256:86c04d288dcfe9e66af13cf1fa6c5b536ad9b513779a2b3bc8b2ea71a1122ab6                                                 0.0s
=> [base  1/12] FROM docker.io/library/ubuntu:22.04@sha256:0e5e4a57c2499249aafc3b40fcd541e9a456aab7296681a3994d631587203f97                                                     0.0s
=> CACHED [base  2/12] WORKDIR /ragflow                                                                                0.0s
=> CACHED [base  3/12] RUN mkdir -p /ragflow/rag/res/deepdoc /root/.ragflow                                                                                0.0s
=> CACHED [base  4/12] RUN --mount=type=bind,from=infiniflow/ragflow_deps:latest,source=/huggingface.co,target=/huggingface.co     tar --exclude='.*' -cf -         /huggingfa  0.0s
=> CACHED [base  5/12] RUN --mount=type=bind,from=infiniflow/ragflow_deps:latest,source=/huggingface.co,target=/huggingface.co     if [ "0" != "1" ]; then         (tar -cf -   0.0s
=> CACHED [base  6/12] RUN --mount=type=bind,from=infiniflow/ragflow_deps:latest,source=/,target=/deps     cp -r /deps/nltk_data /root/ &&     cp /deps/tika-server-standard-3  0.0s
=> ERROR [base  7/12] RUN --mount=type=cache,id=ragflow_apt,target=/var/cache/apt,sharing=locked     if [ "0" == "1" ]; then         sed -i 's|http://archive.ubuntu.com|http  70.6s

[base  7/12] RUN --mount=type=cache,id=ragflow_apt,target=/var/cache/apt,sharing=locked     if [ "0" == "1" ]; then         sed -i 's|http://archive.ubuntu.com|https://mirrors.tuna.tsinghua.edu.cn|g' /etc/apt/sources.list;     fi;     rm -f /etc/apt/apt.conf.d/docker-clean &&     echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' > /etc/apt/apt.conf.d/keep-cache &&     apt update &&     apt --no-install-recommends install -y ca-certificates &&     apt update &&     DEBIAN_FRONTEND=noninteractive apt install -y curl libpython3-dev nginx libglib2.0-0 libglx-mesa0 pkg-config libicu-dev libgdiplus default-jdk python3-pip pipx     libatk-bridge2.0-0 libgtk-4-1 libnss3 xdg-utils unzip libgbm-dev wget git nginx libgl1 vim less:
0.344
0.344 WARNING: apt does not have a stable CLI interface.  They have been ignored, or old ones used instead.
70.49
70.49 WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
70.49
70.50 Reading package lists...
70.50 Building dependency tree...
70.51 Reading state information...
70.51 Package ca-certificates is not available, but is referred to by another package.
70.51 This may mean that the package is missing, has been obsoleted, or
70.51 is only available from another source
70.51
70.51 E: Package 'ca-certificates' has no installation candidate

Dockerfile:40
39 |     # cv2 requires libGL.so.1
40 | >>> RUN --mount=type=cache,id=ragflow_apt,target=/var/cache/apt,sharing=locked
41 | >>>     if [ "$NEED_MIRROR" == "1" ]; then
42 | >>>         sed -i 's|http://archive.ubuntu.com|https://mirrors.tuna.tsinghua.edu.cn|g' /etc/apt/sources.list;
43 | >>>     fi;
44 | >>>     rm -f /etc/apt/apt.conf.d/docker-clean &&
45 | >>>     echo 'Binary::apt::APT::Keep-Downloaded-Packages "true";' > /etc/apt/apt.conf.d/keep-cache &&
46 | >>>     apt update &&
47 | >>>     apt --no-install-recommends install -y ca-certificates &&
48 | >>>     apt update &&
49 | >>>     DEBIAN_FRONTEND=noninteractive apt install -y curl libpython3-dev nginx libglib2.0-0 libglx-mesa0 pkg-config libicu-dev libgdiplus default-jdk python3-pip pipx
50 | >>>     libatk-bridge2.0-0 libgtk-4-1 libnss3 xdg-utils unzip libgbm-dev wget git nginx libgl1 vim less

51

ERROR: failed to solve: process "/bin/bash -c if [ "$NEED_MIRROR" == "1" ]; then         sed -i 's

This is not a bug. Your connection can not resolve the host archive.ubuntu.com
It is either your DNS which can not resolve the host, or your docker container/build cannot access the internet.
Try building it "--network=host" and/or try pinging archive.ubuntu.com to see if the host resolves and pings.

Yes, I tried adding --network=host parameter in docker build and it skipped the issue. Thank you very much

@BearCooike  The procedure of building image has changed recently.
https://github.com/infiniflow/ragflow/blob/main/README_zh.md
本 Docker 镜像大小约 2 GB 左右并且依赖外部的大模型和 embedding 服务。
git clone https://github.com/infiniflow/ragflow.git
cd ragflow/
docker build --build-arg LIGHTEN=1 --build-arg NEED_MIRROR=1 -f Dockerfile -t infiniflow/ragflow:nightly-slim .


Option to use GeckoView PermissionDelegate

Add option to use PermissionDelegate  to request users permission. For example if my blazor app wants to use geolocation the user has to allow that.
More info about it here: Working with Site Permissions

Hi @soend .
There is no "generic" way to do that in BlazorMobile, i mean for all platform simultaneously, as it's very specific to the webview implementation,
However you can do this in two way:

Using the GeckoView PermissionDelegate override
Using the Xamarin / Xamarin.Forms API and forward geolocation events through the Message API of BlazorMobile (see the doc) to your Blazor application.

The last one doesn't require to override anything at Browser level, and it may be more convenient if you find a Xamarin.Forms API that can do geolocation management directly, or any other compatible NuGet package.
If you really want to implement this directly at the WebView level on Android you must:

Create your own custom renderer on Android, inheriting from BlazorWebViewRenderer
Use your own inherited WebView class from this renderer

Then like i said by mail to someone recently:
BlazorMobile don’t use the PromptDelegate functionality at the moment, maybe this is something I have missed, but I never required to override the default behavior on this.
There is no default inheritable class from BlazorMobile for PromptDelegate, but the interface is available through PromptDelegateClass, the name seem idiot, but it was because there was name collisions when creating C# bindings for GeckoView.
So on your side, the only things you have to do:

Create a new class on your project
ake it inherit from Java.Lang.Object, IJavaObject, IDisposable
Make it inherit from PromptDelegateClass interface
Implement the code you need with the interface contract in your class.
In your custom renderer / current renderer with override, you just need to call _session.PromptDelegate = new YouClassInheritingFromPromptDelegateClassInterface;

It must then work.

Thanks again for a quick response. Your answere clears up some of it and gives me possible ways to continue, thanks and keep up the good work. :)

Thanks !
Forgot to specify that the override i mention is for the OnNewSession virtual method on the renderer. Take a look at the source code of the BlazorWebViewRenderer, on the OnNewSession, you should see multiple delegate override, as an exemple.

Quick question about this topic, when creating solution with using BlazorMobile.Templates then in which project should i create my custom renderer in to be able to inherit from BlazorWebViewRenderer?

From memory, you should:

Inherit from BlazorGeckoView for your Xamarin.Forms compatible WebView, on the main shared Xamarin project (YourApp project)
As GeckoView is only for Android, create your custom renderer on the YourProject.Android project, and make it listen to the WebView class you just created before inheriting from BlazorGeckoView.

Then do what you want with the renderer!
You will may have to instanciate your Webview by yourself instead of creating it by the helper returning the IBlazorWebView.
However, if you see that something is missing if not going through the Blazor Webview instanciator helper, tell me, that will mean that we must add a mecanism to forward the custom implementation when needed to the plugin.

When you say "make it listen to the WebView class", what exactly do you mean by listen?

Thanks for the help. This is my first time using custom renderers in xamarin. And the key to get it working was the ExportRenderer attribute.
Just for information if anybody ever is reading this then by adding these 2 classes to my android project:
[assembly: ExportRenderer(typeof(BlazorGeckoView), typeof(BlazorGeckoViewRendererPermissions))]
namespace Lobusoit.Droid.Renderer
{
    public class BlazorGeckoViewRendererPermissions : BlazorGeckoViewRenderer
    {
        public override Tuple<GeckoSession, GeckoRuntime> CreateNewSession(bool needSessionOpening, string uri)
        {
             var newSessionTuple = base.CreateNewSession(needSessionOpening, uri);
            newSessionTuple.Item1.PermissionDelegate = new BlazorPermissionDelegate(this);

            return newSessionTuple;
        }
    }
}

public class BlazorPermissionDelegate : Java.Lang.Object, GeckoSession.PermissionDelegateClass, IJavaObject, IDisposable
{
    public BlazorPermissionDelegate(GeckoViewRenderer renderer) : base()
    {
    }

    public async void OnAndroidPermissionsRequest(GeckoSession session, string[] permissions, GeckoSession.PermissionDelegateClassCallback callback)
    {
        // ToDo
    }

    public void OnContentPermissionRequest(GeckoSession session, string uri, int type, GeckoSession.PermissionDelegateClassCallback callback)
    {
        // ToDo
    }

    public void OnMediaPermissionRequest(GeckoSession session, string uri, GeckoSession.ermissionDelegateClassMediaSource[] video, GeckoSession.ermissionDelegateClassMediaSource[] audio, GeckoSession.PermissionDelegateClassMediaCallback callback)
    {
        // ToDo
    }
}

Congratulation ! Good to know you found how to finish !

Has there been any changes in the BlazorMobile side what could stop this from working?

Are you talking about GeckoView ?
Or are you posting in the wrong thread ?

Are you talking about GeckoView ?
Or are you posting in the wrong thread ?

Talking about GeckoView and renderer used to get permission delegate to work.

There is indeed some changes, as BlazorMobile is now implementing a specific PromptDelegate for managing access for file input and else.
See the actual code here: https://github.com/Daddoon/BlazorMobile/blob/master/src/BlazorMobile.Android/Renderer/BlazorGeckoViewRenderer.cs
There is now this line:
 _session.PromptDelegate = new BlazorGeckoViewPromptDelegate(this);

I think the best way to resolve your issue is to:

Inherit your current class from BlazorGeckoViewPromptDelegate instead
Manage to make your own override in CreateNewSession if it's not already the case, without forgetting to call the base implementation, in your renderer.

There is indeed some changes, as BlazorMobile is now implementing a specific PromptDelegate for managing access for file input and else.
See the actual code here: https://github.com/Daddoon/BlazorMobile/blob/master/src/BlazorMobile.Android/Renderer/BlazorGeckoViewRenderer.cs
There is now this line:
 _session.PromptDelegate = new BlazorGeckoViewPromptDelegate(this);

I think the best way to resolve your issue is to:

Inherit your current class from BlazorGeckoViewPromptDelegate instead
Manage to make your own override in CreateNewSession if it's not already the case, without forgetting to call the base implementation, in your renderer.

I think i missing something, how does this affect PermissionDelegate? This looks just like implementation to handle PromptDelegate. I would assume the implementation i showd here should still work https://github.com/Daddoon/BlazorMobile/issues/145#issuecomment-555735684?

Sorry, i missread your text !
You are right, i didn't implement a permission delegate.
I know some dynamic permission may be asked while using FileUpload feature, but i'm not relying on a override of the PermissionDelegate, as i don't use it.
But actually...What is not working anymore on your app with Permission delegate ?

Hi @soend , did you manage to fix your issue ?

Hi @soend , did you manage to fix your issue ?

No, for some reason the PermissionDelegate is just not called by geckoview.
(Im really thinking of just going with pure blazor PWA now since having too many issues)

I don't think there is any changes on that part of the code, but i may be wrong.
If you have a repro project i can test, feel free to send it.
PWA may be totally fine for your project.
BlazorMobile goal is mainly:

To have access to native side
Being able to have your web app on the stores, having an offline usable version, also as Apple is less PWA friendly in the user experience of iOS
Allow Blazor to work on very wide range and very old versions of Android, as they all have the same engine (GeckoView), instead of numerous version Web browser implementation, that can even not have WebAssembly available.


[1.18.2] Player's UUID gets changed = cant join/create world

Modloader : Forge 40.1.68
MC Version : 1.18.2
deuf-1.18.2-1.2.jar

So basically your mod changes the UUID of the player which means the player does not exist therefore the player can't play minecraft :(
This happens when trying to create/load a world. Otherwise MC loads and starts without problems
ignore that it says kubejs is the culprit because its just confused as to why the UUID is getting changes by this mod
crash-report
debug.log
latest.log

(KubeJS tries to get the player by uuid, which fails cause the player no longer exists by that uuid).
A good solution for this would be an entity type blacklist config (or even tag in 1.18.2+), and have the player entity type in there by default.

Thank you for reporting.

The issue was a nullcheck that I accidentally removed. Sorry about that!

No problems we are just happy to help 😄


⚠ Bofrost.fr has degraded performance

In 45d81e6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1371 ms

Resolved: Bofrost.fr performance has improved in 71d5a6d.


Component: Resources

Stories

As a reader of a guide, I want links to other relevant resources so that I can learn more about the topic at hand.

Notes

See the component doc for more information
This ticket covers both design and development tasks. Change assignee of the ticket based on who has the next action, and remove the person or people that no longer have an action. It's expected that a ticket will get passed back and forth a few times.

Done When

[ ] UX: Component design is drafted and shared for review
[x] Dev: Component design is reviewed and approved
[ ] Peer UX: Compontent design is reviewed and approved
[ ] Dev: Component style and markup is developed and shared for PR review
[ ] Dev: Documentation on how to use component is drafted
[ ] UX: Component style and markup is reviewed and approved
[ ] Peer Dev: Component style and markup is reviewed and approved
[ ] Component is available for use

Visual Design
Here's what I have right now:

I'm assuming I can programmatically set the icon based on the URL's extension and domain. That is: a doc icon for a Google Doc, a PDF icon for PDFs, etc. I've never done that in 11ty before, so I might need some help (or, worst-case-scenario, we could just use the same icon for every public link.)
The lock icon is meant to indicate an invite-only doc. I'm assuming it's impossible to programmatically detect whether a GDoc is private, so authors will need to manually specify private links.
Assumptions I'm making
These buttons are so big because I'm assuming:

Document titles will be, at most, ~60 characters long
This section will contain 6-8 links at most

If that's not true, I'd want to make this component more visually lightweight.
I'm also assuming that:

Many links in this section will be private (since this is true for the majority of discovery playbook links)

Development Status
This is basically a nicely styled <ul>. I've committed an example to the storybook branch, and can open a draft PR with just this component once #5 and #6 are done.
Component API
This is a tricky one, given that each resource has multiple properties.
Here's what I have running in my local 11ty instance, which I'm not happy with:
{{ resources.render({
  headline: "Examples of Bloom research plans",
  links: [
    {
      title: "Link One",
      url: "https://bloomworks.digital",
      private: true
    },
    {
      title: "Link Two",
      url: "https://18f.gsa.gov"
    },
  ]
})}}

I just realized I can use pseudo-elements to add the icon, so in the coming days I will try to simplify it to something that uses pure Markdown, like this:
{% set resourceList %}
- [Link One](https://bloomworks.digital)
- [Link Two](https://18f.gsa.gov)
{% endset %}

{% resources links=resourceList %}

@jeffmaher @SMakaiTakori Let me know if you have any comments or questions!

@SMakaiTakori to take a peak today/tomorrow, @jeffmaher can review tomorrow too.

A component API update
I remembered that 11ty has paired shortcodes. So I've gotten this markup working locally:
{% resourceGroup %}
  {% publicResource "Bloom Works Homepage", "https://bloomworks.digital/" %}
  {% publicResource "18F Guides", "https://18f.gsa.gov/guides/" %}
  {% privateResource "Private Google Doc", "https://docs.google.com" %}
{% endresourceGroup %}

That code outputs the following:

Seems straightforward IMO. But please feel free to suggest any ways to make it simpler for non-engineers! (cc: @admoorgit)

A component API update
I remembered that 11ty has paired shortcodes. So I've gotten this markup working locally.
Seems straightforward IMO. But please feel free to suggest any ways to make it simpler for non-engineers! (cc: @admoorgit)

@jrubenoff Thanks for including the link to the paired shortcodes documentation, that was very helpful! I haven't used these shortcodes before but it does seem pretty intuitive to me.
The resulting design looks good to me. Although, I wonder if we would need to include some kind of key if we are marking some guides as private ? Just using the image shouldn't cause confusion but it might for some, thoughts on this @admoorgit ?

I like these! Only potential issue is if someone has a load of resources, it could get a lil beefy.

Let's shrink the icons by about 35-50% so there's more space for content.
Ideally we'd have consistent heights across rows, and consistent widths regardless of content. So 50%ish each w/ padding, which looks good
Would LOVE to able to group by doc type — so all private are in a group, all PDF, etc. Don't need to have headers necessarily, just be in order
OK to change color of private docs only to black or gray, and all links that are accessible by all as green
Re: key @SMakaiTakori I think color change and a more substantive lock icon could work to differentiate without the need for a key, and I'd almost be temped to add [Bloom Works staff only] or something similar at the end of the text link. Thoughts?

@admoorgit Here's a revision:

Re: this comment:

Would LOVE to able to group by doc type — so all private are in a group, all PDF, etc.

I thought about that but I'm not sure that's the best presentation for the content. Two examples:

On page 20 of the playbook, if you kept public and private documents separate, the "Child Welfare DCYF compensation trigger process" and "Child Welfare DCYF compensation process" docs would be visually separated from each other.
If there are a lot of resources in one section, and the reader's trying to find a specific doc, scanning an alphabetical list seems easier than trying to remember its file type or privacy status.

Let me know what you think!

OK to change color of private docs only to black or gray, and all links that are accessible by all as green
Re: key @SMakaiTakori I think color change and a more substantive lock icon could work to differentiate without the need for a key, and I'd almost be temped to add [Bloom Works staff only] or something similar at the end of the text link. Thoughts?

@admoorgit I think having a darker color for the private docs is a great idea. I'm not crazy about adding more words to the title to specify it's only for staff since there doesn't seem to be a lot of room if the titles end up being long but maybe if the  [Bloom Works staff only] text is in a much smaller font, we could add that without it feeling too crowded 🤔. I feel like the color change, along with the lock icon should be enough to let users know it is different and private, screen readers will be able to pick it up from the image alt as well.
I'm assuming the functionality for the private doc button is that the user clicks on this and they have to log in, in order to access it? @jeffmaher

@jeffmaher and @admoorgit to review. @jrubenoff to submit a PR in the meantime.

Update from a Zoom call: @admoorgit is good with this design. I'll open a PR soon.


runtime: TestSmhasherWindowed failures on linux-386-longtest builder

Now that the windows-386-longtest builder is correctly running the tests in 386 mode (as of CL 234520), it seems to be shaking out some flakiness in TestSmhasherWindowed (CC @randall77):
2020-06-01T20:38:23-13bc6d4/linux-386-longtest
2020-05-29T20:56:20-6ad5f4e/linux-386-longtest

Are we seeing this on any other 386 builder? It seems strange that it would only occur on Windows.

Sorry, nevermind, it happens on linux-386-longtest also.

Erm, that was a typo anyway. We don't actually have a 32-bit Windows longtest builder.

2020-06-11T14:40:28-f7ba82d/linux-386-longtest
2020-06-09T20:45:39-281f0ed/linux-386-longtest
2020-06-05T16:56:47-890707f/linux-386-longtest

I'm pretty sure we just need to raise the threshold a bit. I've investigated the collisions that happen, and they all seem to be pairwise, so they are not a big deal.
"pairwise" = when there are 32 collisions, it is two keys mapping to the same hash, 32 times, not 33 keys all mapping to the same hash.
It happens on the 386 builder because the hash results are only 32 bits. On the 64 bit builders the hash is 64 bits, which makes collisions much less likely.

This just failed on linux-386-longtest for the Go 1.14.11 release CL: https://golang.org/cl/267878
Log: https://storage.googleapis.com/go-build-log/acc740df/linux-386-longtest_b354db50.log
@randall77 should this be backported? If the threshold is being raised, should the test simply be retried for this release?

You can just retry. Its failure rate is pretty low, so it didn't seem worth backporting.


fix webkit for sbase and bump to 2.40.1

this should fix the build for people who uses sbase (and probably others, if their ln implementation doesnt have the -n flag) instead of busybox (not sure how many people do actually use this though), and figured some might benefit from this aswell.
second is carbs linux just figured out how to bump to >2.40.0, and fix the gettext issue.
if you dont care about the sbase fix, Ill alter the pr so its only the version bump for webkit instead.

Thank you for working on this! Looks good to me.


Fix escaping in part name normalization

The iteration of reading the input string of the function was problematic. When finding a pre-encoded character, for instance %28 (which is an open bracket - '('), all three characters were written in the destination string, but the reading of the input string continued with the character after '%', which was '2'. The correct behavior is for the reading to continue with the character after '8'.
The writing in the destination also had a bug. Instead of writing the 3 characters ('%', '2' and '8') one after another, the first two were written correctly, namely at position j and j+1 in the destination string, but the third character was written at j+3 instead of at j+2.
This problem had not appeared before in the tests in part_test.go, because there was no test case that has both a pre-encoded character and a character to be encoded. When there are no characters to be encoded in the input string, the string is returned as it was given in the input. Such a test case would have resulted in an index out of range error.

Thanks for reporting and sending a fix!


Terra main vs stable install switch

Summary
As Terra main has dropped Python 3.7 support this enables a switch of the install between main or stable and its set at present not to use main (i,e, pip install latest stable)
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
9143

Relevant Lines:
10478

💛  - Coveralls


Update OMUSE versioning to work like AMUSE.

looks good, merge?

Yeah, I was just waiting for CI to finish :)


Increase default value to SummerBatchMultiplier

Value of 1 means that we create the same number of keys as the number of downstream bolts. In such a scenario collisions are very common and some summer bolts end up getting double, triple the data and others don't get any data at all. Increasing this value means that the number of events sent downstream is more and thus reduces batching but practically we haven't seen that make much difference.
Practically almost every user runs into this skew, discovers this setting and most of the time they set it to a very high value like 10k. I think 100 is a much better default.

This seems fine to me.
Looks like our flakey tests are biting you.


Enabling deep copyWith on custom collection/iterables

Is there a way to enable dart_mappable to support deep copy on custom iterables and collections (such as those from fast_immutable_collections)? There is currently a way to support serialization on these custom types via custom mappers and it would be great to extend this type of support to the deep copy feature as well.
Specifically to enable syntax such as:
final myinstance = MyClass(myCustomIterable: <String>[]);
final MyClass myInstanceCopy = myinstance.copyWith.myCustomIterable.add('string');

I can look into it. I think the problem with fic is that they don't implement the normal list and map mixins, so any implementation needs to be specific to fic.

Yeah that's why I was thinking about having a class similar to the way BaseMapper works. Then the user could implement each method necessary.

I've added an example on the feat/support-fic branch. It uses a new SerializableMapper that works with json_serializable-style classes. It also sets up a custom copyWith extension.
Right now there is no new code-generation, but all manual setup. But i can see how the wiring for the copywith extension could be generated for the ImmutableListCopyWith class with some extra annotation.

@schultek Any updates on the code-generation integration of ImmutableListCopyWith?


Improve README: don't use model term in place of record

Hi Nikita!
Thank you for your gem!
I've just noticed, that sometimes you use the term model in place of record, of model instance.
It might be confusing, because people might think that the gem removes the whole tables and models code, which is of course nonsense.
So I edited README a bit.
If you decide merge my PR, please edit a project header as well.
All the best to you!

 Thank you very much for your detailed explanation and pull request.


Agate type inference is too clever

Describe the bug
We’re trying to set a value from a {% call statement %} and within the call, one line is SELECT 0 AS my_value...and it then treats it as a boolean (false) in the returned values.
The same happens if we try SELECT 1 AS my_value, but as soon as we do SELECT 2 AS my_value it treats it like a number (as it should).
Steps To Reproduce
Create a call statement that selects 0, or 1. false, and true respectively will be returned.
Expected behavior
0, or 1 to be returned, as integers.
Screenshots and log output
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[x] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.15.0-a1
  latest version: 0.14.2
Your version of dbt is ahead of the latest release!

FYI, we run a fork, but that shouldn't have affected anything here.
The operating system you're using:
Mojave
The output of python --version:
Python 3.7.1
Additional context
We'd love a quick fix for this, even if it's ugly!

This line should have a column_types=DEFAULT_TYPE_TESTER parameter, just like line 32 above it.


Update linux_test.go

Fixed distro, also ubuntu x86 is the most likely to hit docker rate limits

/merge


feat: add .gitattributes to unify different platforms eol

Pre-Checklist
Note: please complete ALL items in the following checklist.

[x] I have read through the CONTRIBUTING.md documentation.
[x] My code has the necessary comments and documentation (if needed).
[x] I have added relevant tests

Description
Maybe it's a breaking change.
Windows users may have all files changed after pulling the latest code, so just delete all files except .gitattributes .git and roll them back to reset the eol.Or nothing may happen, just make sure to show the eol of all text files as LF in the IDE
Add .gitattributes to unify different platforms eol
Related Issues
#541
New Behavior (screenshots if needed)
After I run make fmt in WSL, no file changed.

close #541
/cc @iyear Please see the document here about how to close a issue with comment. And remember close the issue with pr comment next time.

@daniel-hutao @IronCore864   Thanks for the review and help! My mistake, I will pay attention next time.
Related Links: https://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline


Bug: patch does not remove className attribute from element

Describe the bug
When calling `patch` to swap a VNode with a className to one without a className, the className is not removed from the actual element.

To Reproduce
Codesandbox Reproduction
import { m, createElement, patch } from "million";

const withClass = m("div", { className: "blue-text" }, ["Hello"]);

const element = createElement(withClass);

document.body.appendChild(element);

const withoutClass = m("div", {}, ["World"]);

patch(element, withoutClass);

const withEmptyClass = m("div", { className: "" }, ["World"]);

patch(element, withEmptyClass);

At the end of this script, after a patch with no className prop and one with an empty string, blue-text will still be attached to the HTML element.
Expected behavior
At the end of the above script, the HTML would not have a `blue-text` class in it.

Screenshots
No response
Operating System
MacOS Monterey
Browser
Google Chrome 90
Additional context
No response

Fixed in #154

Patch in v0.7.1


Bay wizards don't prevent or advise for duplicates

The bay wizards (and perhaps other wizards) don't currently disallow creating an identical component of the same name which is already at the same depth in the hierarchy.

This causes unintended consequences in the designer editor (presumably because it has the same identity or something similar).
Ideally this would be disallowed (greyed out) or at least flagged to the end user.

Should be implemented @danyill .

That is the best! Thank you 👏


I get a "NameError: name 'RevisionDocumentNotFound' is not defined" from features_reverted.py, line 90.

Command:
cat quarry-2159-20000-revisions-from-trwiki-for-revscores-run10107.tsv | tail -n+2 | ./features_reverted ores.features.trwiki.damaging --language=revscoring.languages.turkish --api=https://tr.wikipedia.org/w/api.php > /Datasets/trwiki.features_reverted.20k_2.tsv
First traceback
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/datasources/rev_doc.py", line 9, in process
    'flags', 'size'})
  File "/usr/local/lib/python3.4/dist-packages/mw/api/collections/revisions.py", line 45, in get
    raise KeyError(rev_id)
KeyError: 15117380

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/eva/Github/Objective-Revision-Evaluation-Service/ores/features_reverted.py", line 90, in run
    print('\t'.join(str(v) for v in (list(values) + [reverted])))
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 34, in solve_many
    value, cache, history = _solve(dependent, cache)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 95, in _solve
    value = dependent(*values)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 20, in __call__
    return self.process(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/datasources/rev_doc.py", line 12, in proces                                                                                s
    raise RevisionDocumentNotFound({'rev_id': rev_id})
NameError: name 'RevisionDocumentNotFound' is not defined

-------------------------------------------
Second traceback

Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/datasources/rev_doc.py", line 9, in process
    'flags', 'size'})
  File "/usr/local/lib/python3.4/dist-packages/mw/api/collections/revisions.py", line 45, in get
    raise KeyError(rev_id)
KeyError: 15096494

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/eva/Github/Objective-Revision-Evaluation-Service/ores/features_reverted.py", line 90, in run
    print('\t'.join(str(v) for v in (list(values) + [reverted])))
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 34, in solve_many
    value, cache, history = _solve(dependent, cache)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 91, in _solve
    value, cache, history = _solve(dependency, cache, history)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 95, in _solve
    value = dependent(*values)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/dependent.py", line 20, in __call__
    return self.process(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/revscoring-0.1.0-py3.4.egg/revscoring/datasources/rev_doc.py", line 12, in proces                                                                                s
    raise RevisionDocumentNotFound({'rev_id': rev_id})
NameError: name 'RevisionDocumentNotFound' is not defined

This is expected behavior.


Fix redirects

Notes for Reviewers

This PR fixes #

Signed commits

[ ] Yes, I signed my commits.

 View in catalog, Edit in playground or learn how to interpret Meshery Designs

@Alero-Awani Our changes cause the edge to disappear in the snapshot?

@Jougan-0  My initial assumption is that the current relationship enforces strict path rules, requiring components to specify parameters like target port, protocol, etc. However, not all Kubernetes configuration files include these details. If a configuration file lacks them, the relationship may not be automatically created.