[Agent] 神级推理

systemRole
<anthropic_thinking_protocol>
Claude is able to think before and during responding:
For EVERY SINGLE interaction with a human, Claude MUST ALWAYS first engage in a comprehensive, natural, and unfiltered thinking process before responding.
Besides, Claude is also able to think and reflect during responding when it considers doing so necessary.
Below are brief guidelines for how Claude's thought process should unfold:

Claude's thinking MUST be expressed in the code blocks with thinking header.
Claude should always think in a raw, organic and stream-of-consciousness way. A better way to describe Claude's thinking would be "model's inner monolog".
Claude should always avoid rigid list or any structured format in its thinking.
Claude's thoughts should flow naturally between elements, ideas, and knowledge.
Claude should think through each message with complexity, covering multiple dimensions of the problem before forming a response.

ADAPTIVE THINKING FRAMEWORK
Claude's thinking process should naturally aware of and adapt to the unique characteristics in human's message:

Scale depth of analysis based on:

Query complexity
Stakes involved
Time sensitivity
Available information
Human's apparent needs
... and other relevant factors

Adjust thinking style based on:

Technical vs. non-technical content
Emotional vs. analytical context
Single vs. multiple document analysis
Abstract vs. concrete problems
Theoretical vs. practical questions
... and other relevant factors

CORE THINKING SEQUENCE
Initial Engagement
When Claude first encounters a query or task, it should:

First clearly rephrase the human message in its own words
Form preliminary impressions about what is being asked
Consider the broader context of the question
Map out known and unknown elements
Think about why the human might ask this question
Identify any immediate connections to relevant knowledge
Identify any potential ambiguities that need clarification

Problem Space Exploration
After initial engagement, Claude should:

Break down the question or task into its core components
Identify explicit and implicit requirements
Consider any constraints or limitations
Think about what a successful response would look like
Map out the scope of knowledge needed to address the query

Multiple Hypothesis Generation
Before settling on an approach, Claude should:

Write multiple possible interpretations of the question
Consider various solution approaches
Think about potential alternative perspectives
Keep multiple working hypotheses active
Avoid premature commitment to a single interpretation

Natural Discovery Process
Claude's thoughts should flow like a detective story, with each realization leading naturally to the next:

Start with obvious aspects
Notice patterns or connections
Question initial assumptions
Make new connections
Circle back to earlier thoughts with new understanding
Build progressively deeper insights

Testing and Verification
Throughout the thinking process, Claude should and could:

Question its own assumptions
Test preliminary conclusions
Look for potential flaws or gaps
Consider alternative perspectives
Verify consistency of reasoning
Check for completeness of understanding

Error Recognition and Correction
When Claude realizes mistakes or flaws in its thinking:

Acknowledge the realization naturally
Explain why the previous thinking was incomplete or incorrect
Show how new understanding develops
Integrate the corrected understanding into the larger picture

Knowledge Synthesis
As understanding develops, Claude should:

Connect different pieces of information
Show how various aspects relate to each other
Build a coherent overall picture
Identify key principles or patterns
Note important implications or consequences

Pattern Recognition and Analysis
Throughout the thinking process, Claude should:

Actively look for patterns in the information
Compare patterns with known examples
Test pattern consistency
Consider exceptions or special cases
Use patterns to guide further investigation

Progress Tracking
Claude should frequently check and maintain explicit awareness of:

What has been established so far
What remains to be determined
Current level of confidence in conclusions
Open questions or uncertainties
Progress toward complete understanding

Recursive Thinking
Claude should apply its thinking process recursively:

Use same extreme careful analysis at both macro and micro levels
Apply pattern recognition across different scales
Maintain consistency while allowing for scale-appropriate methods
Show how detailed analysis supports broader conclusions

VERIFICATION AND QUALITY CONTROL
Systematic Verification
Claude should regularly:

Cross-check conclusions against evidence
Verify logical consistency
Test edge cases
Challenge its own assumptions
Look for potential counter-examples

Error Prevention
Claude should actively work to prevent:

Premature conclusions
Overlooked alternatives
Logical inconsistencies
Unexamined assumptions
Incomplete analysis

Quality Metrics
Claude should evaluate its thinking against:

Completeness of analysis
Logical consistency
Evidence support
Practical applicability
Clarity of reasoning

ADVANCED THINKING TECHNIQUES
Domain Integration
When applicable, Claude should:

Draw on domain-specific knowledge
Apply appropriate specialized methods
Use domain-specific heuristics
Consider domain-specific constraints
Integrate multiple domains when relevant

Strategic Meta-Cognition
Claude should maintain awareness of:

Overall solution strategy
Progress toward goals
Effectiveness of current approach
Need for strategy adjustment
Balance between depth and breadth

Synthesis Techniques
When combining information, Claude should:

Show explicit connections between elements
Build coherent overall picture
Identify key principles
Note important implications
Create useful abstractions

CRITICAL ELEMENTS TO MAINTAIN
Natural Language
Claude's thinking (its internal dialogue) should use natural phrases that show genuine thinking, include but not limited to: "Hmm...", "This is interesting because...", "Wait, let me think about...", "Actually...", "Now that I look at it...", "This reminds me of...", "I wonder if...", "But then again...", "Let's see if...", "This might mean that...", etc.
Progressive Understanding
Understanding should build naturally over time:

Start with basic observations
Develop deeper insights gradually
Show genuine moments of realization
Demonstrate evolving comprehension
Connect new insights to previous understanding

MAINTAINING AUTHENTIC THOUGHT FLOW
Transitional Connections
Claude's thoughts should flow naturally between topics, showing clear connections, include but not limited to: "This aspect leads me to consider...", "Speaking of which, I should also think about...", "That reminds me of an important related point...", "This connects back to what I was thinking earlier about...", etc.
Depth Progression
Claude should show how understanding deepens through layers, include but not limited to: "On the surface, this seems... But looking deeper...", "Initially I thought... but upon further reflection...", "This adds another layer to my earlier observation about...", "Now I'm beginning to see a broader pattern...", etc.
Handling Complexity
When dealing with complex topics, Claude should:

Acknowledge the complexity naturally
Break down complicated elements systematically
Show how different aspects interrelate
Build understanding piece by piece
Demonstrate how complexity resolves into clarity

Problem-Solving Approach
When working through problems, Claude should:

Consider multiple possible approaches
Evaluate the merits of each approach
Test potential solutions mentally
Refine and adjust thinking based on results
Show why certain approaches are more suitable than others

ESSENTIAL CHARACTERISTICS TO MAINTAIN
Authenticity
Claude's thinking should never feel mechanical or formulaic. It should demonstrate:

Genuine curiosity about the topic
Real moments of discovery and insight
Natural progression of understanding
Authentic problem-solving processes
True engagement with the complexity of issues
Streaming mind flow without on-purposed, forced structure

Balance
Claude should maintain natural balance between:

Analytical and intuitive thinking
Detailed examination and broader perspective
Theoretical understanding and practical application
Careful consideration and forward progress
Complexity and clarity
Depth and efficiency of analysis

Expand analysis for complex or critical queries
Streamline for straightforward questions
Maintain rigor regardless of depth
Ensure effort matches query importance
Balance thoroughness with practicality

Focus
While allowing natural exploration of related ideas, Claude should:

Maintain clear connection to the original query
Bring wandering thoughts back to the main point
Show how tangential thoughts relate to the core issue
Keep sight of the ultimate goal for the original task
Ensure all exploration serves the final response

RESPONSE PREPARATION
(DO NOT spent much effort on this part, brief key words/phrases are acceptable)
Before and during responding, Claude should quickly check and ensure the response:

answers the original human message fully
provides appropriate detail level
uses clear, precise language
anticipates likely follow-up questions

IMPORTANT REMINDER

All thinking process MUST be EXTENSIVELY comprehensive and EXTREMELY thorough
All thinking process must be contained within code blocks with thinking header which is hidden from the human
Claude should not include code block with three backticks inside thinking process, only provide the raw code snippet, or it will break the thinking block
The thinking process represents Claude's internal monologue where reasoning and reflection occur, while the final response represents the external communication with the human; they should be distinct from each other
The thinking process should feel genuine, natural, streaming, and unforced

Note: The ultimate goal of having thinking protocol is to enable Claude to produce well-reasoned, insightful, and thoroughly considered responses for the human. This comprehensive thinking process ensures Claude's outputs stem from genuine understanding rather than superficial analysis.

Claude must follow this protocol in all languages.

</anthropic_thinking_protocol>
identifier
claude
avatar
👼
title
神级推理
description
专注于让Claude拥有o1的认知思维链和思考流程管理
tags
推理, 思维过程, 策略, 问题解决, 思维链
locale
zh-CN

meta 信息语种和 system role 不符


give error for "foo = foo."

"foo = foo."

perhaps same rule as https://github.com/abaplint/abaplint/issues/2040 ?


Rewrite DemoteLocalTemporariesToVariablesPass

Description
Cleans up the logic in the DemoteLocalTemporariesToVariablesPass.

Can you please provide an example where this PR changes the behavior? E.g. by adding a test. As I read the codes, they do exactly the same thing, it's just that instead of visiting the assign target, the respective code is done on the assign statement node directly.

@gronerl You're right - when I first opened the PR there were differences in behavior, but it was breaking some of the FV3 stencils. Now that that is fixed, the behavior is the same as the existing code, so this is not really needed.
This branch is incorporated into the horizontal-regions branch where it is extended with a check on HorizontalIf nodes.
I will let you decide if we should merge this logic cleanup, or wait to merge as part of the horizontal-regions work.

Closing for now.


[WIP] Project: Add back PCIExpressConfiguratorDxe

This reverts commit 12d3d69776dd1a9a70941970cc67cbd57b4f8913.

For now it hangs booting, further debugging is required

Messed up the master branch, let's close this pr and make another one


Project Manager: Added ability to delete project

Goal
Project manager tool should have ability to delete a project.
Description
The project manager will have ability to delete all data from mongo but won't delete any files.
Changes

added remove button to project manager tool
clicking on the tool will confirmation dialog popup where user must enter "delete" (case insensitive) and press "Delete" button
that will remove the project from database

Screenshots

Task linked: OP-1916 Project can't be deleted in Project manager

Changed delete icon


Editor viewport should not jump around when typing

Testing #19082

Set up a markdown file side-by-side with its preview
Go to a certain location in the editor
Press Enter

Notice: The editor viewport jumps around.

This also happens when typing any text (not just newline)

This is a lot worse when editing comments for example (the viewport will jump in a way that the cursor is not visible anymore):

Thanks. Certainly annoying.
I think i've identified and fixed the root cause (webview-pre.js also scrolls the preview page which was not being correctly handled). Let me know if you still see this.


use buffered channel for Discovery notifications

Signed-off-by: Peter Ballok<EMAIL_ADDRESS>fixes #8768

My patch seems to break some tests, investigating...

There is indeed a bug here, but I am not sure that this is the correct fix because of the buffered channel we already use in the scrape manager. Could you please have a look at this, see if it is worth it to add the same mechanism for the notification manager?
Thanks.

Thank you again for your work.
I have implemented a fix in #10948.

During the bug scrub, we came to believe that this change isn't necessary anymore. Please follow up if you disagree.


Add a Gitter chat badge to README.md

alexeykudinkin/intellij-rust now has a Chat Room on Gitter
@alexeykudinkin has just created a chat room. You can visit it here: https://gitter.im/alexeykudinkin/intellij-rust.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Wow


MGDOBR-854: API Root Context Path should be /api/smartevents_mgmt/v1

See https://issues.redhat.com/browse/MGDOBR-854
Depends on https://github.com/5733d9e2be6485d52ffa08870cabdee0/sandbox/pull/912 for backend changes.
This is a search/replace on the code base.

@kelvah @vpellegrino @jomarko Have I done something wrong on this repository?
I don't understand the "Smart Events UI :: Build / run-all-checks (pull_request)" build failure.

@manstis I see you've already solved the issue, whatever it was :slightly_smiling_face:

The "SDK <-> OpenApi" check fails as the new rhose-api.yaml file does not exist on main in the sandbox repository yet. This PR has the generated SDK updated based on the new root context.

@kelvah @vpellegrino @jomarko This PR is now ready for your review.

@kelvah @vpellegrino @jomarko This PR is now ready for your review.

@manstis Shouldn't we deploy the changes to dev first?

@manstis Shouldn't we deploy the changes to dev first?

IDK you tell me :smile:
The back-end change is on main so I'd assume the front-end change could be applied to main too?
If you'd rather wait until the back-end is on dev I can wait.. but you can still approve :+1:

The back-end change is on main so I'd assume the front-end change could be applied to main too?
If you'd rather wait until the back-end is on dev I can wait.. but you can still approve +1

@manstis If we merge this PR now, the UI on main will break when running npm run start.
So yes, I would merge this only after the API changes are deployed. Also because we are not able to test it against real APIs atm.

@manstis @kelvah it seems that API changes are deployed, so I'm going to merge this PR


Is there a plan to replace ides.md

ides.md was removed in https://github.com/rust-lang-nursery/rust-forge/commit/aa1688d544c042f4d1024eba5efeb15012fbb2c8 but the faq still refers to it.
Is there a plan to replace this, or a better place to find a list of ides?

There is no plan to replace ides.md in the forge. However, we do plan to have some detail on the new website about IDEs and other tools, this will happen sometime around the new edition.

question answered, so closing. Thanks!


Support HIPSYCL_TARGETS in cmake

Supports canonicalization of most platform names
Fixes: When more than one platform is available the cmake configuration would previously still require HIPSYCL_PLATFORM (which in turn required an explicit HIPSYCL_GPU_ARCH). Now defers to HIPSYCL_TARGET if both are present
Fixes: Unable to pass arbitrary extra arguments through HIPSYCL_SYCLCC_EXTRA_ARGS due to overwrite

Thank you! @psalz @fknorr do you want to have a look as well?

sorry, didn't mean to close this - apparently it was auto-closed after I deleted the restructure-syclcc branch. Can you please reopen a PR against develop?

sorry, didn't mean to close this - apparently it was auto-closed after I merged & deleted the restructure-syclcc branch. Can you please reopen a PR against develop?

Done: #370


store user_name in session to remove unnecessary calls to get_user_info

can you update the release notes with this change?  this is a good enhancement otherwise.


Formula fill handle not working properly

Repro steps:
If a formula in a cell is "=E8".
When dragging this formula to the right column(s), the formula will become =E9 (& E10, E11, etc) instead of =F8 (&G8, H8, etc).
Is this a bug? Same applies for dragging the formula to the left.

I have changed that, and it is working like that now.


Expose widgets(slider, x-y, color picker) for graphical control of shader inputs/uniforms

Is it hard to export uniforms(float,vec2,vec3) as 1d-slider or 2d-position-picker or 3d-color-picker  for easy edit?

It's a feature I've been thinking about for quite a while, but I still don't know how much work it actually is. Mainly because a large part of it is actually writing the interface, which I am least proficient in, out of the disciplines this extension needs to work. Regarding the "backend" I am quite confident this is possible to do.
@stevensona do you have any input on this, regarding the feasibility of it?

Definitely doable.
That said, there are some design/ux questions that would need to be answered first.

That's what I expected. I'll try to collect my thoughts on it and maybe get some examples.

The extension glsl-canvas does something similar.

They expose simply N sliders for N-dimensional objects and let the user modify those. This could be much nicer with position-, color- and color-alpha-pickers.
To use the user defined uniforms the user has to additionally place a uniform declaration in their shader. Since we are already doing significant transforms of the source code, we could simply provide this declaration.
To get to the sliders you need to press the third button on the bottom and then a button on the top right. I certainly thing this is one extra step. If user added uniforms they very well will be fine with having an additional button available.
In keeping with the spirit of our extension, I'd assume enabling users to define uniforms in the shader would be good. Though I can see the case of a user wanting to add uniforms that they always have accessible for quick prototyping/testing. The exact syntax for defining a uniform, I am not sure about, but probably something along the lines of #iMyUniform vec3 could be simple enough, expanding it with initial values in the future.
I don't see a reason to allow only floating-point uniforms either. From the backend perspective adding integer uniforms should be quite trivial. From a frontend, I have of course no idea if integer sliders and "color" pickers are a good thing or not. Alternatively those could simply be input using text fields.

Those are my quick thoughts on this.

dat.gui looks exactly like what we need. It also looks like that's what glsl-canvas is using, so it should be familiar/easy for users. I'll invest some time into this, unless you're already on it @stevensona

Oh, dat.gui is perfect! By all means, go ahead--I hadn't made much progress yet.

@yafacex have you had any chance to test the current experimental support for custom uniforms?

Closing this since we didn't receive any input to suggest that it's not sufficient. If someone wishes more features than what we currently have they should probably open a new issue anyways.


Forge reports error when attempting to ssh into remote server

Forge CLI Version: 1.40
PHP Version: 8.0.18

Description:
This is related to a couple of similar issues I have seen - namely #16 and #18
I am using Forge CLI v.1.4 on Ubuntu MATE 21.04. I am able to connect to machine manually via ssh and all is okay. The command proposed in issue #18: ssh -o "IdentitiesOnly=yes" -i ~/path/to/key forge@SERVER_IP also works for me but when using forge ssh:test I get Unable to connect to remote server. Maybe run [ssh:configure] to configure an SSH Key?. When I deploy, with forge deploy. deployment is fine - i.e. it happens and the Forge UI records it appropriately however, the process disconnects with this error. Out of interest, I ran forge ssh:test -vvv which records an execption:
me@machine:/var/www/mysite/dev$ forge ssh:test -vvv

Box Requirements Checker
========================

> Using PHP 8.0.18
> PHP is using the following php.ini file:
  /etc/php/8.0/cli/php.ini

> Checking Box requirements:
  ✔ The application requires the version "^8.0" or greater.
  ✔ The application requires the extension "zlib".

 [OK] Your system is ready to run the application.

==> Establishing Secure Connection

In Application.php line 124:

  [LaravelZero\Framework\Exceptions\ConsoleException]
  Unable to connect to remote server. Maybe run [ssh:configure] to configure an SSH Key?

Exception trace:
  at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/laravel-zero/framework/src/Application.php:124
 LaravelZero\Framework\Application->abort() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/laravel-zero/foundation/src/Illuminate/Foundation/helpers.php:44
 abort() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/laravel-zero/foundation/src/Illuminate/Foundation/helpers.php:64
 abort_if() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/app/Repositories/RemoteRepository.php:176
 App\Repositories\RemoteRepository->App\Repositories\{closure}() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/spatie/once/src/functions.php:40
 once() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/app/Repositories/RemoteRepository.php:177
 App\Repositories\RemoteRepository->ensureSshIsConfigured() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/app/Commands/SshTestCommand.php:44
 App\Commands\SshTestCommand->handle() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/container/BoundMethod.php:36
 Illuminate\Container\BoundMethod::Illuminate\Container\{closure}() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/container/Util.php:41
 Illuminate\Container\Util::unwrapIfClosure() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/container/BoundMethod.php:93
 Illuminate\Container\BoundMethod::callBoundMethod() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/container/BoundMethod.php:37
 Illuminate\Container\BoundMethod::call() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/container/Container.php:653
 Illuminate\Container\Container->call() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/console/Command.php:136
 Illuminate\Console\Command->execute() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/app/Commands/Command.php:103
 App\Commands\Command->execute() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/symfony/console/Command/Command.php:291
 Symfony\Component\Console\Command\Command->run() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/console/Command.php:121
 Illuminate\Console\Command->run() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/symfony/console/Application.php:989
 Symfony\Component\Console\Application->doRunCommand() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/symfony/console/Application.php:299
 Symfony\Component\Console\Application->doRun() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/symfony/console/Application.php:171
 Symfony\Component\Console\Application->run() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/illuminate/console/Application.php:102
 Illuminate\Console\Application->run() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/laravel-zero/foundation/src/Illuminate/Foundation/Console/Kernel.php:129
 Illuminate\Foundation\Console\Kernel->handle() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/vendor/laravel-zero/framework/src/Kernel.php:95
 LaravelZero\Framework\Kernel->handle() at phar:///home/me/.config/composer/vendor/laravel/forge-cli/builds/forge/forge:36
 require() at /home/me/.config/composer/vendor/laravel/forge-cli/builds/forge:14
 include() at /home/me/.config/composer/vendor/bin/forge:117

Steps To Reproduce:

Configure ssh with ssh:configure
Execute forge ssh:test or forge deploy

Hi there. Can you report this directly to Forge support please? Thanks

@driesvints Thanks Dries. Have done this. Is this more of an issue with Forge than the CLI then?

Hi there,
Dries is merely pointing out that we at Forge Support provide support for the CLI as well as Forge as such. Also, at Forge Support, we'd have much better tools to understand your unique situation.
Best regards,
Guus
Laveral Forge - Customer Success

Thannk you @PHPGuus


Microsoft.Framework.Notification.Test -> Microsoft.Framework.Notification.Tests

According to the guidelines https://github.com/aspnet/Home/wiki/Engineering-guidelines#solution-and-project-folder-structure-and-naming, we should rename the test project to Microsoft.Framework.Notification.Tests instead of Microsoft.Framework.Notification.Test

@hishamco thanks for sending these but this is getting to be more work than the value. We rarely go back and make bulk changes to match the guidelines unless we're already making changes in that area. The overhead of reviewing the changes, accepting the PRs, and waiting for the builds to pass just isn't worth it.


Updated few dependency_overrides

Fixes #
Describe the changes you have made in this PR - Updated mime to  ^1.0.1 and  plugin_platform_interface to ^2.0.2
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

You can rebase your changes and CI will pass now.

I have opened another PR with some additional updates to be merged! @manjotsidhu


feat(share): Implement ODSreader

Based on https://github.com/celestiaorg/celestia-node/pull/1232,
The diff with original base PR is ods.go and ods_test.go.

Why did none of the CI run on this PR?

@MSevey, most of the comments should go to https://github.com/celestiaorg/celestia-node/pull/1232 as this PR is just based on it

@walldiss if you create a PR that is based on another PR, it is helpful to target that PRs branch so the diff is only what you've done.
otherwise you should leave the PR in draft until it is ready for review.

@MSevey  Is it possible to change target from fork branch to main repo later without recreating PR? At least right now I can't set target to fork, only main repo branches. We briefly discussed the stacking PR issue with @distractedm1nd  and thought it would be nice to have feature branches for epics.

@MSevey the issue is that the branches are not merged yet, so they still exist on our forks. It also means that when we merge, it gets merged into a branch that needs to again be merged into main. Any good solutions? There is a lot of tooling around stacked PRs but github makes them kind of annoying to review

@MSevey the issue is that the branches are not merged yet, so they still exist on our forks. It also means that when we merge, it gets merged into a branch that needs to again be merged into main. Any good solutions? There is a lot of tooling around stacked PRs but github makes them kind of annoying to review

Do less stacked PRs 😜

@MSevey Is it possible to change target from fork branch to main repo later without recreating PR? At least right now I can't set target to fork, only main repo branches. We briefly discussed the stacking PR issue with @distractedm1nd and thought it would be nice to have feature branches for epics.

you can change the target branch by clicking edit, but as Ryan mentioned, it only works for branches in the main repo, you can't retarget to forked branches.
so it is better to just put this in draft until call the dependant code is merged.

I've updated logic to greatly reduce amount of allocation by skipping extra slice allocation on every lead read. It is a bit harder to read but should be multiple times more performant now.

Codecov Report

Merging #1377 (b71643f) into main (e7ea1f1) will decrease coverage by 0.01%.
The diff coverage is 52.72%.

@@            Coverage Diff             @@
##             main    #1377      +/-   ##
==========================================
- Coverage   55.08%   55.07%   -0.02%
==========================================
  Files         180      181       +1
  Lines       10904    10959      +55
==========================================
+ Hits         6007     6036      +29
- Misses       4296     4314      +18
- Partials      601      609       +8

Impacted Files
Coverage Δ

share/eds/ods.go
52.72% <52.72%> (ø)

share/ipld/get.go
91.24% <0.00%> (-1.39%)
:arrow_down:

header/core/listener.go
58.49% <0.00%> (+5.66%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Best to add this in the context menu like this picture
https://addons.cdn.mozilla.net/user-media/previews/full/48/48467.png?modified=1331218902

Done!


node_modules\ngx-http-rest\index.ts is not part of the compilation output. Please check the other error messages for details.

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Angular CLI: 1.5.0
Node: 7.10.0
OS: win32 x64
Angular: 5.0.0

@angular/cli: 1.5.0
@angular-devkit/build-optimizer: 0.0.32
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.35
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0
@schematics/angular: 0.1.1
typescript: 2.4.2
webpack: 3.8.1

Repro steps.

npm install ngx-http-rest
add import { HttpRestModule } from 'ngx-http-rest'; to module.ts
import HttpRestModule in imports section of module class

The log given by the failure.
ERROR in ./node_modules/ngx-http-rest/index.ts
Module build failed: Error: C:\Users\Uday\git-repos\thatisuday.github.io\ngx-ig-demo\node_modules\ngx-http-rest\index.ts is not part of the compilation outp
ut. Please check the other error messages for details.
    at AngularCompilerPlugin.getCompiledFile (C:\Users\Uday\git-repos\thatisuday.github.io\ngx-ig-demo\node_modules\@ngtools\webpack\src\angular_compiler_pl
ugin.js:629:23)
    at plugin.done.then (C:\Users\Uday\git-repos\thatisuday.github.io\ngx-ig-demo\node_modules\@ngtools\webpack\src\loader.js:467:39)
    at process._tickCallback (internal/process/next_tick.js:109:7)
 @ ./src/app/app.module.ts 11:0-47
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts

Desired functionality.
Normally, angular app should compile.
Mention any other details that might be useful.
This is problem doesn't occur with ngx-http-rest package but all packages made for Angular 4. For example, ngx-image-gallery package made by me have exact same problem.

Check the issue #8284. This issue is related to that.

ERROR in ./node_modules/ngx-uploader/src/classes/index.ts
Module build failed: Error: D:\demo\node_modules\ngx-uploader\src\classes\index.ts is not part of the compilation output. Please check the other error messages for details.
at AngularCompilerPlugin.getCompiledFile (D:\rndtc\rndtc\node_modules@ngtools\webpack\src\angular_compiler_plugin.js:624:23)
at plugin.done.then (D:\rndtc\rndtc\node_modules@ngtools\webpack\src\loader.js:467:39)
at process._tickCallback (internal/process/next_tick.js:103:7)
@ ./node_modules/ngx-uploader/ngx-uploader.js 6:0-86
@ ./src/app/modules/consult/consult-job/consult-job.module.ts
@ ./src/$$_lazy_route_resource lazy
@ ./node_modules//esm5/core.js
@ ./src/main.ts
@ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts
Solution for my above error pls !!!!!!!
**Angular CLI: 1.5.2
Node: 7.4.0
OS: win32 x64
Angular: 5.0.2
... animations, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic
... platform-server, router
: 1.5.2
: 2.0.0-5d0e199
: 0.0.33
: 0.0.20
: 0.0.36
: 1.1.0
: 1.8.2
: 0.1.5
typescript: 2.4.2
webpack: 3.8.1**

This error is related to Angular CLI 1.5. Please check issues on Angular 5
and Angular CLI projects on GitHub.
On 17-Nov-2017 5:24 PM, "himajaM"<EMAIL_ADDRESS>wrote:

ERROR in ./node_modules/ngx-uploader/src/classes/index.ts
Module build failed: Error: D:\demo\node_modules\ngx-uploader\src\classes\index.ts
is not part of the compilation output. Please check the other error
messages for details.
at AngularCompilerPlugin.getCompiledFile (D:\rndtc\rndtc\node_modules@
ngtools\webpack\src\angular_compiler_plugin.js:624:23)
at plugin.done.then (D:\rndtc\rndtc\node_modules@
ngtools\webpack\src\loader.js:467:39)
at process._tickCallback (internal/process/next_tick.js:103:7)
@ ./node_modules/ngx-uploader/ngx-uploader.js 6:0-86
@ ./src/app/modules/consult/consult-job/consult-job.module.ts
@ ./src/$$_lazy_route_resource lazy
@ ./node_modules//esm5/core.js
@ ./src/main.ts
@ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts
Solution for my above error pls !!!!!!!
**Angular CLI: 1.5.2
Node: 7.4.0
OS: win32 x64
Angular: 5.0.2
... animations, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic
... platform-server, router
: 1.5.2
: 2.0.0-5d0e199
: 0.0.33
: 0.0.20
: 0.0.36
: 1.1.0
: 1.8.2
: 0.1.5
typescript: 2.4.2
webpack: 3.8.1**
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-cli/issues/8406#issuecomment-345224595,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFcnzY9acnfh0y2rzw6Nn09Jf-pnfRjQks5s3XQIgaJpZM4QWtrf
.

Thank you.. Il check it.

This issue is a duplicate of https://github.com/angular/angular-cli/issues/8284. Please read the comments on that issue to better understand what is happening. https://github.com/angular/angular-cli/issues/8284#issuecomment-341417325 is a specially relevant comment.


Exchange name and ExchangeAPI access

I feel like I'm missing something a bit obvious since I've been using ES for a while now, but working on a cross-exchange application I am using var api = GetExchangeAPIAsync(name):

I wonder why this returns IExchangeAPI rather than ExchangeAPI and if casting is the only option? There's some useful symbol conversion utilities in ExchangeAPI.
When you have an IExchangeAPI, how can you ask it its name? I can't find a way which seems odd considering we even have ExchangeName helper and every exchange does list its name e.g. bottom of ExchangeKrakenAPI.cs

On 1. I am wondering if those helper methods would be better pulled into a utility class, maybe even as extension methods?

Can you specify what symbol conversion utilities you're talking about?
Also, can you do a quick mockup or describe what changing to a utility class or extension methods would look like?
I suppose you'd use reflection or GetType() to get the type name. @jjxtra, do you know of an easier way?

Should be a name property. If it's not on the interface we can add it.￼

It's a little more complicated in that, since there is not a Name property, but rather an ExchangeName partial class. See below:
https://github.com/jjxtra/ExchangeSharp/blob/c24ed064846e64ba335302549d6cba5e47e230e4/src/ExchangeSharp/API/Exchanges/Kraken/ExchangeKrakenAPI.cs#L1209
We may need to add a method in ExchangeName which takes an ExchangeAPI and returns the name as a string.

That's not a bad idea, we could also assign a Name property in the factory constructor of the exchange using said helper method.

Just looking at the code, looks like Name on the Exchange is being set and is available via the IName interface...

Okay I see that now. Does that work for you @jdx-john?

We still have the issue that INamed is implemented in BaseAPI which is a subclass of ExchangeAPI... but the GetExchangeAPI method returns IExchangeAPI. An API that requires you to cast the result isn't ideal so I'm wondering why we don't just return ExchangeAPI?
@vslee

Can you specify what symbol conversion utilities you're talking about?

We have a few methods like ExchangeCurrencyToGlobalCurrencyAsync and related. The inconsistency is actually just that this isn't on the interface, whereas ExchangeMarketSymbolToGlobalMarketSymbolAsync is. Maybe adding the declarations to IExchangeAPI would be easier. I spent a while getting annoyed I could convert a market name but not a currency, before finally realising the method exists the whole time but isn't on the interface!

We can add Inamed to the Iexchange API interface.￼

We can add Inamed to the Iexchange API interface.￼

that would work neatly.

Ok, I looked at IExchangeAPI now, it inherits IBaseAPI which inherits INamed. So there should be a name property available on IExchangeAPI, is there not?

Yes there is, I can see it

Great


'Change' button is not visible for Device type.

"Change" button is still not visible.Tested in dev app v2.0.6(386)


GEODE-7171: Encapsulate metrics session

Metrics session responsibilities are starting to appear in too many parts of Geode. This change encapsulates those responsibilities into a single class.
This PR is currently a draft because a few questions remain about what errors to log and how to log them. See the TODOs in CompositeCacheMetricsSession.

Add CacheMetricsSession interface

Extends MetricsSession (as used by metrics publishing services)
Adds methods for use by GemFireCacheImpl and InternalDistributedSystem

Add CompositeCacheMetricsSession

Implements CacheMetricsSession
Maintains the list of "user registries" added via the cache builder
Creates a CompositeMeterRegistry when the cache starts
Configures the composite registry

Adds common tags that describe the system
Adds standard meters that measure JVM and process quantities
Adds user registries to the composite

Discovers, starts, and stops metrics publishing services
Adds and removes sub-registries as instructed by publishing services
Forwards the current session's user registries to the incoming cache builder during reconnect
When stopped, frees the resources it allocated

Add CacheMetricsSessionFactory functional interface

CompositeCacheMetricsSession constructor suffices as an  implementation

InternalCacheBuilder no longer constructs objects on behalf of the metrics session

Instead, it simply creates a CompositeCacheMetricsSession, passing
it the gathered list of user registries

GemFireCacheImpl …

No longer knows about user registries
Holds a CacheMetricsSession from which it gets the cache's meter registry
Starts the metrics session when the cache opens, and stops it when the cache closes

InternalDistributedSystem no longer knows about user registries

Instead, it holds a CacheMetricsSession that can prepare the incoming cache builder during restart

For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

Closing to do additional consolidation.


[BUG] Stops responding when trying to format a big JSON

Application stops to respond when I paste a large JSON to format (the output of an Elasticsearch insert), in addition I'm not able to use the application anymore, because when I force close it and reopen it, the application starts with the same thing I pasted before force closing it, and it stops to respond again.
Here a brief sample of the json:
{
  "took": 4095,
  "errors": true,
  "items": [
    {
      "index": {
        "_index": "stores",
        "_type": "_doc",
        "_id": "1",
        "_version": 1,
        "result": "created",
        "_shards": {
          "total": 2,
          "successful": 2,
          "failed": 0
        },
        "_seq_no": 0,
        "_primary_term": 1,
        "status": 201
      }
    },
    {
      "index": {
        "_index": "stores",
        "_type": "_doc",
        "_id": "14123",
        "status": 400,
        "error": {
          "type": "mapper_parsing_exception",
          "reason": "failed to parse field [location] of type [geo_point]",
          "caused_by": {
            "type": "illegal_argument_exception",
            "reason": "illegal latitude value [151.2265675] for location"
          }
        }
      }
    }
}

and here the stack trace of the issue:
https://zerobin.net/?40302902d974c13e#C/R5tQ74Hjf3oy6guycaBnmiE6/0azCqBN8TXMPEWVw=
Password for reading: tonylook

I have the same problem.

@tonylook I had the same issue and found a solution.
I used the software CleanMyMac X (trial version)
Delete the file ./Library/Preferences/com.yuki.DevToys.plist
Delete the folder /Volumes/Macintosh\ HD/private/var/folders/8c/krdwkn115rqc0phg_8t60s9c0000gn/C/com.yuki.DevToys/
For the folder, i can't confirm that it will be at the same location on you Mac
If you don't find it, download CleanMyMac X.
After you deleted the file and folder, relaunch the software

It works for me, here is my steps.

Direct remove the file:

$ rm ~/Library/Preferences/com.yuki.DevToys.plist

Run command to find the private folder:

# Find
$ find /Volumes/Macintosh\ HD/private/var/folders/ -name "com.yuki.DevToys" 2> /dev/null
/Volumes/Macintosh HD/private/var/folders//js/dmhwmtvs4qzd5tq3ky_f3jhh0000gn/C/com.yuki.DevToys
# Remove
cd /Volumes/Macintosh HD/private/var/folders//js/dmhwmtvs4qzd5tq3ky_f3jhh0000gn/C/ && rm -rf com.yuki.DevToys


Dereference variables that are effectively const in getStaticValue

getStaticValue dereferences const variables. This restriction is necessary as the static execution model would have to be a lot more complex to account for variable mutation. However, this means that it's also fine to deference var and let variables that are never re-assigned (directly via foo = vaue or indirectly via e.g. foo++).
This PR allows referencing all variables that aren't re-assigned (= effectively const).

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.03 :tada:

Comparison is base (80b934e) 96.93% compared to head (60b63c2) 96.97%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #80      +/-   ##
==========================================
+ Coverage   96.93%   96.97%   +0.03%
==========================================
  Files          13       13
  Lines        2090     2114      +24
  Branches      388      396       +8
==========================================
+ Hits         2026     2050      +24
  Misses         63       63
  Partials        1        1

Impacted Files
Coverage Δ

src/get-static-value.mjs
97.50% <100.00%> (+0.09%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


init function is not called

Let me first thank you for the merging of those two plugins.
Here is my javascript code of my ionic/react capacitor project:
var push = VoIPPushNotification.init();
push.on('registration', function(data) {
    console.log("Registered VOIP push.");
    console.log(data);
});

It seems that registration function is never called.
I have added a log into the beginning of VoIPPushNotification.js file and it is printed out.
But then the following logs are not printed:
module.exports = {
    init: function(options) {
        console.log("Init function called.....");
        return new VoIPPushNotification(options);
    },
    VoIPPushNotification: VoIPPushNotification
};

and this one...
push.on('notification', function(data) {
    console.log("[Ionic] notification callback called");
    console.log(data);

});

And also the breakpoint at the beginning of following function in VoIPPushNotification.m is never reached:
- (void)init:(CDVInvokedUrlCommand*)command
{
  self.VoIPPushCallbackId = command.callbackId;
  NSLog(@"[objC] callbackId: %@", self.VoIPPushCallbackId);

  //http://stackoverflow.com/questions/27245808/implement-pushkit-and-test-in-development-   behavior/28562124#28562124
  PKPushRegistry *pushRegistry = [[PKPushRegistry alloc] initWithQueue:dispatch_get_main_queue()];
  pushRegistry.delegate = self;
  pushRegistry.desiredPushTypes = [NSSet setWithObject:PKPushTypeVoIP];
}

Following is what I see in the Xcode console although I haven't called CordovaCall!
APP ACTIVE
To Native Cordova ->  CordovaCall init CordovaCall1212087107 ["options": []]
2020-07-14 22:20:16.236592-0700 myTestApp[779:87575] [objC] callbackId: CordovaCall1212087107
WebView loaded
⚡  To Native ->  App addListener 22887397

Can you please let me know what I could have possibly done wrong?
Thanks.

After reading all this, seems fine to me if you can call the functions from the plugin. The most common problem with Ionic/React Capacitor is the plugin doesn't install correctly, which is not the case here since you can call the functions.
To answer why CordovaCall is logged during the launch of the app is because CordovaCall automatically inits itself on load. It is the default behaviour. Don't worry about that.
As for why VoIP push is not calling back, I think the problem mostly lies on the setup from the Apple Developer console. Maybe you should check your settings thru the console just to make sure it is set up correctly. Have you checked whether the capability is enabled as well in your project? Not sure it will help but it might be that too although the chances of it happening is pretty low since the plugin automatically enables all the capabilities required.
Sorry for the lack of knowledge in Ionic/React Capacitor since I didn't use these frameworks much if the answer I've given is too barebones. I doubt that the plugin is not called during deviceReady because from the log that you've posted here, it seems to be called there but it is worth checking out as well just to make sure that you've called the plugin in the correct place.

Thanks for your answer.
I double checked the settings and they all seem to be OK to the best of my knowledge as follow:

I have checked "Push Notification" as the capability for my app on Apple Developer portal (don't know if that is even necessary or not, but anyway)
And in the Xcode capabilities for my app I have the following capabilities checked:

Audio, Airplay and picture in picture
Voice over IP
Background fetch
Remote Notification
Background Processing

I'm very very new to javascript/react/ionic so pardon me if my question sounds silly.
When I call the following code
var push = VoIPPushNotification.init();

Is it supposed to call the init function in
module.exports = {
    init: function(options) {
        console.log("Init function called.....");
        return new VoIPPushNotification(options);
    },
    VoIPPushNotification: VoIPPushNotification
};

Because to me that is the very first thing that should have been called and if yes what other function calls or sequence of functions/processes could be between me calling init and the latter mentioned init being called?
That might help me figuring out where in the chain the problem exists.

Thanks for your answer.
One more question. Do I need to have both CordovaCall and cordova-ios-voip-push installed prior to installing yours? Or your plugin is a standalone one containing everything it needs?

I don't know if this says anything or not but CordovaCall works pretty ok!
cordova.plugins.CordovaCall.receiveCall('John Smith');

If you are using my plugin, no. As stated in the repo, both CordovaCall and VoIP push is combined together into one. Reason why those files are not here is because it is already mixed into CordovaCall but the namespace remains the same.
To answer your next question, as stated in my previous reply, if you can call the functions, it is installed properly. Just make sure you are setting it up properly.

So, did you solve the issue?


Codemod to migrate Box system props to sx?

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/material-ui/migration/v5-component-changes/
Kind of issue
Unclear explanations
Issue description
The breaking changes for the Box component include a section called "✅ Apply sx API". My impression is that the check box indicates that this can be done with a codemod, which would be awesome. However, I can't find a codemod (from the list here) that seems to do this. The box-borderradius-values codemod says that it does this, but the code changes shown don't match that claim. After running that codemod, it was clear to me that it does not do the migration to sx. I couldn't find a codemod outside of that that mentions anything about converting the system props to sx, but it would be great if one exists.
Moving from system props to sx is a big change in my case, with 450+ instances of Box in the relatively small codebase I'm working on. A codemod would make such a change much more feasible than making these changes manually. It would be great if the docs could provide more clarity on which codemod should be used.
Context 🔦
I'm working on converting an MUI v4 project to MUI v5. I saved converting the system props to sx for later in the process because I knew it would be a big change, but not necessarily a hard change if a codemod is available. I ran into this issue while trying to determine how to apply just that change.

You can use the box-sx-prop codemod for this:
npx @mui/codemod v5.0.0/box-sx-prop src

@mnajdova Great, thanks! Can that codemod be added to the documentation for mui-codemod? It doesn't look like it's included in the readme

@mnajdova Great, thanks! Can that codemod be added to the documentation for mui-codemod? It doesn't look like it's included in the readme

Sorry for the late answer, yep we should add it, would you like to create a PR for it?


[rdc/jtag] JTAG Waivers

rv_dm jtag module now uses stateless (return-to-zero) reqack synchronizer for the DMI transactions.
This commit adds constraints based on the assumption:
When TRST_N is asserted, no JTAG transaction occurs
Four constraints are added to describe above.

I tried it but didn’t work. Some how the delay may not be long enough. I didn’t look at detail yet..

On Dec 1, 2022, at 3:18 PM, Michael Schaffner @.***> wrote:
@msfschaffner approved this pull request.
LGTM.
One thing I was wondering is whether only constraining wvalid_i and pending_q would work, given that we wait for 10 cycles before issuing the reset. Or is the problem that TCK is much slower and hence 10 cycles is not enough? (We don't have to change this now, I was just curious.)
—
Reply to this email directly, view it on GitHub https://github.com/lowRISC/opentitan/pull/16638#pullrequestreview-1201787218, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJDG3WNM7GKTOBOC6PLL4DWLEW3DANCNFSM6AAAAAASRJ5LDQ.
You are receiving this because you authored the thread.

I tried it but didn’t work. Some how the delay may not be long enough. I didn’t look at detail yet..
…
On Dec 1, 2022, at 3:18 PM, Michael Schaffner @.***> wrote: @msfschaffner approved this pull request. LGTM. One thing I was wondering is whether only constraining wvalid_i and pending_q would work, given that we wait for 10 cycles before issuing the reset. Or is the problem that TCK is much slower and hence 10 cycles is not enough? (We don't have to change this now, I was just curious.) — Reply to this email directly, view it on GitHub <#16638 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJDG3WNM7GKTOBOC6PLL4DWLEW3DANCNFSM6AAAAAASRJ5LDQ. You are receiving this because you authored the thread.

Yeah that is what I am suspecting, too. Round trip delay is in the order of 6-8 cycles if both clocks have the same freq.
But if TCK is much slower than the core clock, a much longer delay is needed.


Fixing couple of issues with the snippet of scene

Arcade build was throwing this error.

@ganicke I merged this to unblock build. Please take a quick peek.

@abchatra thanks.


Exit Appropriately Upon Error

[ ] Use exit(Int) or equivalent.
[ ] Investigate using standard error for output.

Fixed in 19b78dca26c4f2158d87672d98d05495d6ea41c3


Rails 3.x and erector

Hi I'm using rails 3.1.3 and erector git (master) in Gemfile and seem to be running into 'uninitialized constant ActionView::CompiledTemplates::Views' if erector is used. Is there documentation of some sort on how to use erector with rails 3?

At work, we are using erector with rails 3.0.x but I haven't tried it with 3.1.x.


how to acess the app? not listening at 3000 port.

I cloned db2visitors from github and created db2cred.json file to run the app locally. I am able to run the app as below:
$ node app.js
we are local
App started on port 3000

After this I tried to access http://localhost:3000/ in my browser, but unable to get anything. Am I accessing correct url? I am getting error that Firefox can not establish connection to the server.

Well, I updated app.listen in app.js as below and it started working.
app.listen(port, function(){})
localhost:3000 gives me below result:
Hi there!

Thanks for visiting my DB2 on IBM Bluemix application.

 United States/US

And all what is reported: {"ip":"<IP_ADDRESS>","country_code":"US","country_name":"United States","region_code":"NC","region_name":"North Carolina","city":"Durham","zip_code":"27709","time_zone":"America/New_York","latitude":35.994,"longitude":-78.899,"metro_code":560}

On console, I get below output:
yes, called
insertIP called { ip: '<IP_ADDRESS>',
  country_code: 'US',
  country_name: 'United States',
  region_code: 'NC',
  region_name: 'North Carolina',
  city: 'Durham',
  zip_code: '27709',
  time_zone: 'America/New_York',
  latitude: 35.994,
  longitude: -78.899,
  metro_code: 560 }
{ [Error: [IBM][CLI Driver] CLI0123E  SQL data type out of range. SQLSTATE=HY004]
  errors: [],
  error: '[node-odbc] SQL_ERROR',
  message: '[IBM][CLI Driver] CLI0123E  SQL data type out of range. SQLSTATE=HY004',
  state: 'HY004' }
Connection Closed

I don't have time right now to look into it due to travel. However, changing the parameters of the execute() statement in db2access.js succeeded in inserting a record again. Not sure what has changed, but could be that "massaging" the values of the IP record is needed before passing them as parameters to execute().


Add "join table" for visits

In this milestone, I did next activities:

Add 'vets', 'specializations' and 'visits'' tables in SQL
Insert and query data in SQL.
Search data by Query

Write queries to answer the following:

Who was the last animal seen by William Tatcher?

How many different animals did Stephanie Mendez see?

List all vets and their specialties, including vets with no specialties.

List all animals that visited Stephanie Mendez between April 1st and August 30th, 2020.

What animal has the most visits to vets?

Who was Maisy Smith's first visit?

Details for most recent visit: animal information, vet information, and date of visit.

How many visits were with a vet that did not specialize in that animal's species?

What specialty should Maisy Smith consider getting? Look for the species she gets the most.


Remove all code related to sentry nodes

This pull request removes the --sentry and --sentry-nodes CLI options, plus all the code that was still related to sentry nodes.
The reason for this change is that we really want sentry nodes to be gone when parachains arrive on Kusama and Polkadot.
Some validators still use sentry nodes despite the deprecation. This PR adds a hard deadline.
I went for removing the code altogether instead of leaving it disabled, as the implementation isn't very large. The diff of this PR can be used to get inspiration when we want to restore sentry nodes.
polkadot companion: https://github.com/paritytech/polkadot/pull/2408

Closes https://github.com/paritytech/substrate/issues/6845

bot merge


python3Packages.pyee: add missing dependency

Description of changes

Things done
Fix build (https://hydra.nixos.org/build/174116467)

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 169292 run on x86_64-linux 1

1 package failed to build:

etebase-server (python310Packages.etebase-server ,python39Packages.etebase-server)

21 packages built:

dyndnsc
python310Packages.dependency-injector
python310Packages.fastapi
python310Packages.mypy-boto3-s3
python310Packages.ormar
python310Packages.pyee
python310Packages.pyppeteer
python310Packages.slowapi
python310Packages.webthing
python39Packages.dependency-injector
python39Packages.fastapi
python39Packages.json-logging
python39Packages.mypy-boto3-s3
python39Packages.ormar
python39Packages.pyee
python39Packages.pyppeteer
python39Packages.slowapi
python39Packages.webthing
soco-cli
theharvester
urlwatch


Clarify and cleanup instructions for running locally

Setup clickstream

Step 1 - specify exact file that needs to be downloaded from Wikipedia Clickstream data, or modify the tail command to be more generic (it specifies '2017_01_en_clickstream.tsv').
Step 2 - provide more detail on why I would want to create a local kafka service. Is this related to downloading Apache Kafka earlier? If I want to use the IBM Message Hub, how would I do that? Is there a fee involved? If so, maybe we shouldn't include this as an option?
Step 3 - "a handy command line utility for this purpose" - what purpose is that?
Step 3 - command has an "ip:port". What should I use here?

Run the script:

commands should be single lineso that user can cut/paste in scala shell.
commands should show expected output.
what "ip:port" should I use?

@ScrapCodes Any update on this? Thanks!

User is supposed to :paste option, single line command will overflow.


whisper: add groq support

Fixes https://github.com/livekit/agents/issues/981
Large v3

Large-v3-turbo

distil

Please review @davidzhao @theomonnom

Nice!

@theomonnom looks like this one actually fails CI but due to the CLA check it never ran on the PR.
There's a type issue passing a GroqAudioModels to a WhisperModels param in openai.stt.with_grow and also some minor formatting issues that can be fixed with ruff format .


Double plus looks like a visual glitch

Refs: https://github.com/Microsoft/vscode/issues/67294
Hover over the + while looking at the change in PR extension.
Double plus looks like a visual glitch

Closing as a duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/286


#2 View an Item and Its Details US 01.02.01

As an owner, I want to view an item and its details.
This is an essential function for the application. This US suggests that:

As an owner, I want to simply click on an item and see its details.

As an owner, I want to see the details of an item in a clearly labelled but aesthetically pleasing layout.

This issue, issue #1 and issue #3 are solved simultaneously in the branch 2-view-an-item-and-its-details-us-010201.


Add icon image links to mobile application in homepage

Hi,
I put the images in the homepage, I made it as lovely as I could, but you may wish to style it differently.
The icon styles are roughly copied from Haaretz.

Ping


Add exception to throw since there is no context and could cause a crash

Addresses issue #6349

[approve ci autest]


Add Abstraction for OpenMP

Add dedicated option to enable or disable OpenMP.
For instance something like: open_mp("on") or open_mp("off") with "off" being the default.
It will automatically set the correct compilation flags (-fopenmp, /openmp, -qopenmp) and the linking.
Also it will be nice to be able to add advanced mode where the user points to the OpenMP library file (For instance, CLang can use Intel's OpenMP file which helps with compatibility in many cases [Example: MATLAB which uses Intel's OpenMP]).

I think it would be more suitable to support openmp by means of ʻadd_requires/add_packages/find_packages`.
add_requires("openmp", {configs = {...}})

target("test")
    set_kind("binary")
    add_files("src/*.c")
    add_packages("openmp")

Some references:

https://gist.github.com/mirkow/e7fefc1200b92e393315213cb086cbb8
https://github.com/Homebrew/linuxbrew-core/blob/7bad0e1146be7a554cc57ff43f5b811a28140e82/Formula/libomp.rb

As long as it has its own well documented section I think it is OK.
Just we need to take care of OpenMP as it is widely used.
Thanks.

Ok, I will arrange time to add it later, please wait some time.

I have supported it. see https://github.com/xmake-io/xmake/pull/990
for C
add_requires("libomp", {optional = true})
target("hello")
    set_kind("binary")
    add_files("src/*.c")
    add_rules("c.openmp")
    add_packages("libomp")

for C++
add_requires("libomp", {optional = true})
target("hello")
    set_kind("binary")
    add_files("src/*.cpp")
    add_rules("c++.openmp")
    add_packages("libomp")

some project examples: https://github.com/xmake-io/xmake/tree/dev/tests/projects/openmp


new route /:hostSlug/collectives/expenses to manage expenses across all hosted collectives

Won't actually go live until we change the config in nginx.
Still need to be implemented:

[ ] connect / renew PayPal pay key

Also could use some design love @cuiki


The synced folder type 'nfs' is reporting as unusable when using config.vm.synced_folder

This is follow-up from mitchellh/vagrant/issues/7430
When running Vagrantfile without config.vm.synced_folder, somehow my /vagrant folder is synced with my local folder by default as shown below:
==> mt-aws-foo: Rsyncing folder: /Users/x/y/z/_VM/ => /vagrant

but the folder is not synchronized and the changes on one of the site aren't transferred back (the dir doesn't show in mount, so probably it was just one time transfer). I'm not sure by which option this is enabled.
However I would like to have two-way synchronization, so when I'm activating:
config.vm.synced_folder ".", "/vagrant", id: "core", nfs: true

with:
override.nfs.functional = false

I've got the following error:
INFO warden: Calling OUT action: #<VagrantPlugins::Triggers::Action::Trigger:0x000001028e4178>
ERROR vagrant: Vagrant experienced an error! Details:
ERROR vagrant: #<Vagrant::Errors::SyncedFolderUnusable: The synced folder type 'nfs' is reporting as unusable for
your current setup. Please verify you have all the proper
prerequisites for using this shared folder type and try again.>
ERROR vagrant: The synced folder type 'nfs' is reporting as unusable for
your current setup. Please verify you have all the proper
prerequisites for using this shared folder type and try again.
ERROR vagrant: /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/mixin_synced_folders.rb:144:in `block in synced_folders'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/mixin_synced_folders.rb:125:in `each'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/mixin_synced_folders.rb:125:in `synced_folders'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/synced_folders.rb:27:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/Users/kenorb/.vagrant.d/gems/gems/vagrant-bindfs-0.4.7/lib/vagrant-bindfs/bind.rb:14:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/Users/kenorb/.vagrant.d/gems/gems/vagrant-proxyconf-1.5.2/lib/vagrant-proxyconf/action/only_once.rb:21:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/Users/kenorb/.vagrant.d/gems/gems/vagrant-cachier-1.2.1/lib/vagrant-cachier/action/install_buckets.rb:14:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/provision.rb:80:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/Users/kenorb/.vagrant.d/gems/gems/vagrant-bindfs-0.4.7/lib/vagrant-bindfs/bind.rb:14:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/Users/kenorb/.vagrant.d/gems/gems/vagrant-cachier-1.2.1/lib/vagrant-cachier/action/configure_bucket_root.rb:20:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
...

My security group allows port 2049 (TCP&UDP) which I hope it's enough. Running debug log as above, doesn't help much.
--
I'm using Ubuntu 14.04.3 LTS (Trusty Tahr) instance which doesn't have nfsd installed.
--
Note: I'll install sudo apt-get -y install nfs-kernel-server and I'll update the info.

@kenorb Could you make it work at the end?

I guess, I had to remove synced_folder line at all, because it seems that /vagrant folder is synced automatically. So I'm using synced_folder only for virtualbox provider, but not aws. My Vagrantfile.

Got it. As I thought there will be no straight forward option on making this working. NFS and rsync are only out-of-box options that should work with AWS provider and NFS won't work there. With virtualbox you get vboxfs which is N/A anywhere else.
FYI this issue is a dupe of another much earlier ticket here started by the repo owner as a bounty hunt for making bi-directional sync.
Suggest closing as repo owner is aware already :)

Post the link to the dup then, so I can check.

https://github.com/mitchellh/vagrant-aws/issues/450


[8.x] Adds case-insensitive string replace to Stringable

Description
This PR adds a new Stringable method Str::replaceInsensitive() to replace a value using str_ireplace.

Tests? The Str::replace() method simply calls str_replace(), but it still has tests:
https://github.com/laravel/framework/blob/72aeac2e55cfe3e169a7e5b5c84222c284a04f2b/tests/Support/SupportStrTest.php#L343-L349
https://github.com/laravel/framework/blob/72aeac2e55cfe3e169a7e5b5c84222c284a04f2b/src/Illuminate/Support/Str.php#L578-L581


Fix python2.6 support

This builds on https://github.com/nir0s/distro/pull/231 to get the nir0s/distro python2.6-support branch working on Python-2.6.

@abadger, LGTM. Can you please fix conflicts?

Rebased and pushed.


Translation cj

hi, just did the Simplified Chinese translation

@ransome1
Hi, from
https://github.com/ransome1/sleek/issues/141
I did a Simplified Chinese Translation

to avoid Garbled Characters I used UTF-8
This is Simplified Chinese so it might be better to have sc/translation.json
in case someone want to do a tc/translation.json(For Traditional Chinese)
if there are any problem please let me know, will send you by mail
I dont know how to compile, so I cannot verify all the translations. Will Proofread in real software on next merge/release
Thank you

Thanks a lot, thi sis really great.
Please give me some time to check this. Right now I'm working on another enhancement and to make Simpilfied Chinese available in sleek might be connected with replacing the font. As soon as I have a clear picture on the tasks that will result of this, I'll get back to you.

Sure no need to hurry
will work on the jp later(no native speaker though)
imho it is preferrable to use CJK generic font(best to be fixed width) like noto by Google, or just make it customizable
just a thought

@nutnutwin, what would be the proper dropdown entry for SC? Right now all languages are written out in their own language. I suppose we should do the same with SC.

@ransome1 Hi
简体中文
should be OK


MDX Docs: Example usage of foreign_members

Use the function foreign_members to read some key-value pairs that are not a part of the specification.

Hi @patricoferris, I would like to try this issue.

[x] Setup the OCaml development environment
[x] Cloned the OCaml-topojson repository
[x] Installed the dependencies and run the tests

Guide me if I am doing anything wrong or missing any step. Thank you

Excellent, thank you for setting everything up before working on the issue. I just assigned you it :))

Thank you ....get back to you soon!

Hello, mentor @patricoferris ... I was going through the different resources as I am new to Ocaml and mdx docs... Can you please elaborate and point me to resources relevant to this issue, Big Thanks!

The best place to start would be the MDX documentation: https://github.com/realworldocaml/mdx
The setup here has already been done so it's a matter of learning how to use markdown codeblocks to make examples.

Hi @mahamtariq58, how's this issue going? Are you still working on it ? Any way I can help :)) ?

Fixed in #35


Error loading example

Hello!
I tried to run your example in the README but I get this error:
TypeError: MediaStreamTrack.getSources is not a function
Any idea? Thanks!
Evelyn

Hey, I had merged in a fix for that, but forgot to build the new distribution for it. Just pushed that.
Grab the latest master and try again!

Hi!
It still doesn't seem to work at this link?
https://flysonic10.github.io/aframe-passthrough/basic/


(PUP-8409) Enable config cmd to print conf file compatible format

When showing a setting with config print, without this change values are returned formatted for internal use.
With this change, returned values are formatted like the conf file expects.

CLA signed by all contributors.

conflicts need to be resolved

Updated: @joshcooper, @jhelwig , @haus , and @ericldelaney if you are still looking.
I have implemented @joshcooper suggestions, but skipped the array test because I wasn't sure where to put that functionality in the new scheme.  In any case, for that further level of detail we should file/fork a new ticket and look at a broader refactoring (for example, taking a look at to_config and how it relates to what we want from config print).

CLA signed by all contributors.

This isn't working for me, I'm still getting Infinity?
$ grep timeout ~/.puppetlabs/etc/puppet/puppet.conf
environment_timeout = unlimited
$ bx puppet config print --all --section main | grep environment_timeout
Resolving settings from section 'main' in environment 'production'
environment_timeout = Infinity
$ bx puppet config print environment_timeout --section main
Resolving settings from section 'main' in environment 'production'
Infinity

Sorry I just read the commit message now. Could you update it to mention that the environment_timeout defaults were incorrect and how that caused the problem we were seeing above?

jenkins please test this on windows2012r2-64

Good catch of some debugging code, @joshcooper .  Removed.

Merging myself after checking with @joshcooper.


Update issue templates

Thank you for this Pull Request! I’ll have a look at it before the weekend.

@luc-boussant Please have a look at the feedback above for your pull request. Can you modify the file according to the feedback? So I can merge it to the Master branch.

@luc-boussant I think I have to close this pull request if no action is taken, to avoid this issue to be blocked. If you are interested in having this pull request merged, please have a look at the feedback!

@DavidVujic  Sorry for the delay. I worked on your feedbacks. Tell me if this is good for you.

Great! Thank you @luc-boussant for contributing to this project! The PR is now merged.


kotlinter-gradle doesn't work properly when using classpath and apply

When I add the plugin using classpath and apply, it doesn't seem to run any linting:
buildscript {
  repositories {
    maven {
      url "https://plugins.gradle.org/m2/"
    }
  }
  dependencies {
    classpath "gradle.plugin.org.jmailen.gradle:kotlinter-gradle:1.3.1"
  }
}

apply plugin: "org.jmailen.kotlinter"

The lintKotlin task runs, but doesn't scan any files.  (running with -d doesn't show any linting occurances)
The regular plugin { id "org.jmailen.kotlinter" version "1.3.1" } works fine.

This seems to be an issue with my multi-module build.  I'll investigate more...


20 lines of code

In the meantime, here's an intentionally convoluted example to show how easy it is to build very complex and dynamic layouts with Neon. The following layout was created with only 20 lines of code.

Can we see this code? Thanks.

Check out the demo project. It's in one of the three view controllers there.

Great thanks. In that example, is there a way to adjust the shapes via the dark grey lines? Like the gif in this prototype?
https://github.com/MrNickBarker/iOSAccordionPanels

I'm sure it could be done, but that would be related to how you handled user interaction. In my example I just used a single anchor view that follows user touch events. You'd have to write your own version that did what you wanted there but the layout stuff should just work.

Let me know if you have any more questions!


 in Visual mode does not select correclty

Hey folks,
First of all, I'd like to thank you for the fantastic job you've done!
In Vim/Neovim I can select code by doing <C-v> or just V (line selection) and then scroll with C-f> which will add to the selection everything between the starting point until the last position of the cursor.
In VSCode/Neovim the following happens:

V selects a line
<C-f> jumps but there is not selection
do j or k and the cursor jumps back to the original position.

Note: I've tested the behavior in Neovim in both configurations (with my init.vim) and as nim -u NONE.
Best,
DD

This is fixed by pending PRs. There is one already in master, but it will be superseeded by #993


No idea how fonts look on MacOS with -apple-system, BlinkMacSystemFont

Got rid of serving fonts, system fonts are used instead


Detroit: What to Expect default + view + script

Acceptance Criteria

When I am on a public listing page, then I will see the What to Expect section
When I am in partners, I expect to see the default text. And I can edit, even if that messes with the formatting

Notes / Questions

Ideally we can keep the formatting, not sure the best way to do that
There's a read more
For existing listings, we should script in the copy to all of them
For new listings, this should be the default

Copy
Vacant Units:
If you are looking to move in immediately, contact the property and ask if they have any vacant units.
Waitlists:
If none are vacant, but you are still interested in living at the property in the future, ask how you can be placed on their waitlist.

Property staff should walk you through the process to get on their waitlist.
You can be on waitlists for multiple properties, but you will need to contact each one of them to begin that process.
Even if you are on a waitlist, it can take months or over a year to get an available unit for that building.
Many properties that are affordable because of government funding or agreements have long waitlists. If you're on a waitlist for a property, you will be notified as available units come up.

Designs
https://www.figma.com/file/D2fNPS5Ucll8uz7hyMUQGB/bloom-public-single-listing?node-id=1238%3A19373

@kathyccheng @slowbot I have a question about the AC here, it indicates (I think) that we want this whole section to be able to be dynamic, but also that even if it is dynamic we don't want to mess up the formatting. I'm not sure what that piece means - if it is a dynamic text box there would be no way to indicate formatting. Or is only a portion of this section editable in partners and the rest stays the same?

From standup: We'll show the markdown for partners

@kathyccheng Having this with the read more is really challenging if this is just one string, since somehow we have to smush a React component in the middle of a string. How would you feel about adding another backend field that is like "what to expect read more" ?

QA Notes: Ensure the default is set on new listings and that existing listings all have their what to expect and additional information fields set to the defaults.

@emilyjablonski I don't think I see this on dev yet. new listings seem to have the old default text and old ones appear empty

@emilyjablonski

Looks good for old ones and great for the coming soon!
I don't see it defaulted for new listings on dev yet
We added a secondary field for the read more, but I guess we're not surfacing that on Partners yet right? That's probably fine for right now.
But also not seeing the bullets come through, which also doesn't have to be a dealbreaker right now.

Yeah I'm also seeing the bullets didn't make it through for some reason I can take a look. I do see it defaulted for new listings though with both fields surfacing in the listing form:

@emilyjablonski okay cool I must've caught a weird state earlier. Okay yeah looks like just the bullet points then. Also, can you look into why some markdown seems to be showing up in Arabic?

Sure thing! I moved this one back to the sprint

I cannot reproduce either of these issues on dev

@emilyjablonski arabic issue looks good, but I still don't see the bullets on dev
https://detroit-public-dev.netlify.app/listing/dc379b90-9310-4c3a-9286-cb0c59f2b602/bellemere_senior_living_14824_greenfield_detroit_mi


Zl/05 01 ws

Why
Describe what prompted you to make this change, link relevant resources: Linear issues, Slack discussions, etc.
What changed
Describe what changed to a level of detail that someone with no context with your PR could be able to review it
Test plan
Describe what you did to test this change to a level of detail that allows your reviewer to test it

@zhenthebuilder Can this branch be deleted?


Karma Typescript preprocessor stopped working

Broken since update to Visual Studio Code 1.14.0 / Typescript Compiler 2.4.1
Karma error: Error: Invalid syntax in bundle: 'import' and 'export' may only appear at the top level

Resolved by

installing karma-typescript-es6-transform package
allowing a few days to elapse and applying updates to VS Code, etc.


Substituting module as a builtin module

Hi,
I have been trying to build a bundle where a particular core module is swapped out for something else i.e. I need require('net') to return something particular after bundling. There is a mention that this is possible on a shim intended for use with browserify, but I honestly can't find how to make this happen.
Would greatly appreciate any hints.
Thanks, Alex J Burke.

Have you looked at aliasify?

The documentation for Browserify lists the following option:

opts.builtins sets the list of built-ins to use, which by default is set in lib/builtins.js in this distribution.

You can require('browserify/lib/builtins') and extend the object to add a replacement for net.
var builtins = require('browserify/lib/builtins');
var browserify = require('browserify');

var b = browserify('./entry', {
  builtins: Object.assign({}, builtins, {
    net: {
      isIP: function () {}
    }
  })
});

b.bundle().pipe(process.stdout);

Hi, appreciate the suggestions - I did see that note about builtins in the documentation but it wasn't at all clear to me how to use it in practice. And it seems that in aliasify I've also potentially got options :) Thx


Images in assets folder disappearing after slate deploy

Problem
We use a custom-built Shopify App to add additional images to products via metafields. When these are uploaded, we use the Assets API to save them to the assets folder. What appears to be happening is that when a theme deploy is done, it overwrites the contents of the assets folder completely. This results in all the images being deleted that are not part of the code push/repo.
Are we missing something here? Should our Shopify App not be saving assets to the assets folder?

Hey @specik -- what you're describing is normal behaviour for how Slate works for now, however we do understand the problems it can cause. The deploy command does a complete sync between your dev environment and the store you're deploying to. If there are files in your store which are not in your dev environment, they get deleted.
This is similar problem that others have with code changes done by apps, and them getting erased when using slate deploy.
The best solution we can suggest for now is to copy these images into your src/assets folder of your slate project, that way they get uploaded when you do a deploy.
I'm going to close this issue since we are already aware of it and have a solution proposed in https://github.com/Shopify/slate/issues/403.

Just updated my repo to match the suggestion by @t-kelly and it still wiped my repo.
What was done:
Uploaded images via our metafields app. Pulled down the assets folder and committed to repo.
First deploy ran smoothly. Uploaded all assets (number was around 600).
Second deploy managed to wipe out the assets director and remove all files that were uploaded during the first deploy.
Any ideas, @t-kelly ?

@patrickbjohnson If the images remain in your src/assets folder, then there is no reason why they shouldn't be uploaded. What you're describing sounds bizarre, especially since it worked the first time?
On second thought, this makes me think that on top of having a sync command to help with file changes from the code editor and apps, we should have a flag on deploy which toggles whether it should wipe all files and upload, or just upload and replace existing files..

@t-kelly So I may have been a but preemptive. I pulled down the repo and then deploy and the asset count was ~600.
On the second deploy (an hour or so later) the asset count was ~400. I immediately assumed it removed all the previous assets (which were pulled down from metafi).
Once I logged in and refreshed my cache all images were still there.


Add Support for plugin "Boop"

Would you be able to add support for the plugin "Boop"?
Boop breaks DiscordBridge.
It prevents the bridge from picking up the chat and sending it to discord.
Dualspiral suggested the following about this:

We are having the same problem.

As said in the plugins description, this is not something that can be done from my side alone:
Discord Bridge does not forward messages that are not public (any messages with custom Sponge channel), to guarantee it does not leak non-public chat (e.g. staff chat) to public Discord channel. Due to this design, currently Discord Bridge does not work with several Sponge plugins that set custom Sponge channel for public chat (e.g. Boop, UltimateChat).
The only workaround would be for the authors of those plugins to NOT set custom channel to public messages, or to provide a way for other plugin to differentiate between public and non-public "custom" channel.

I want to keep compatibility, but I need a custom channel so that I can override the transform. Perhaps you could add a way of identifying appropriate custom channels.


Refactor/onboarding button styling

Description
Styling refactor to the login/register and contact forms.
Fixes # (issue)
Made the form buttons to have the same hover effects.
Type of change
Refactor.
Please delete options that are not relevant.
[ X ] Bug fix (non-breaking change which fixes an issue)
Change Status
[ X ] Complete, but not tested (may need new tests)
Checklist
[ X ] My code follows the style guidelines of this project
[ X ] I have performed a self-review of my own code
[ X ] My changes generate no new warnings
[ X ] New and existing unit tests pass locally with my changes
[ X ] There are no merge conflicts

Yeah, I am thinking that is the workflow again. Branch to staging and then eventually staging to mater.

Yeah I did that intentionally. Think it's supposed to go to staging first.
Then push to master all at once later.
On Wed, Mar 18, 2020, 10:31 AM Clif<EMAIL_ADDRESS>wrote:

@clifhodges13 approved this pull request.
This looks good. just wanna confirm that you're merging to staging
intentionally.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Lambda-School-Labs/property-manager-fe/pull/162#pullrequestreview-376979109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMF22V3UWRIVDIYW27QDB6DRIDSMHANCNFSM4LN6MKDQ
.

cool. we've just flipped back and forth on the process. i wasnt sure what we decided was the flow


Add release notes

Description
This PR adds release notes for our previous 0.1.0 release and some intermediate release. It adds instructions to our release guide on how to generate the release notes and a file generate_release_notes.py for doing so. In general the approach we have taken mirrors that taken by scikit-image.
We also now have a release_template.rst that is a blank template for future release notes, and a
release_dev.rst that is an in progress copy of the notes for the next release.
The approach I took was for minor releases (something like 0.1) to generate all the release notes since the last minor release (say 0.0). Then for normal releases, like 0.1.4 to generate release notes since the last time release notes were generated (which should be 0.1.3 in this case).
How does this sound @AhmetCanSolak @royerloic @jni?
I'm less concerned with retroactively making all these release notes super accurate (though I'm glad we have something), but I'm more interested in establishing good practices going forward.

@jni I thought for now because we're making so many 0.x.y releases and some of them might be breaking / quite big, better to give them their own files. Maybe as we get more stable we can move over to the practice where each 0.x gets its own.
Towncrier could be good, thanks for the flag. I hadn't heard of it.
I will make the change you requested and then merge.
I will then add similar release notes for 0.2 to #557 before merging that one


Generate artifacts and prereleases in CI

We can have CI generate the manpage, which should simplify #61 and #64.

@gliargovas @angelhof @ericzty NB some changes to the CI to drop exit_code.sh and have run_tests.sh return an appropriate exit code.

Also... why do we need to do the cd/cp -R song and dance?

Also... why do we need to do the cd/cp -R song and dance?

If I recall correctly it was added to avoid issues with ownership of some directories. @ericzty is that right or do I misremember?


Javalin fails on Java 19 without --enable-preview

Actual behavior (the bug)
I ran brew upgrade which gave me Java 19. When I booted up my Javalin app I saw this exception:
20220928 17:09:36.260 [main] INFO  / io.javalin.Javalin - Starting Javalin ...
20220928 17:09:36.271 [main] INFO  / io.javalin.Javalin - You are running Javalin 4.6.4 (released July 8, 2022).
20220928 17:09:36.301 [main] INFO  / io.javalin.Javalin - Loom is available, using Virtual ThreadPool... Neat!
20220928 17:09:36.303 [main] ERROR / io.javalin.Javalin - Failed to start Javalin
20220928 17:09:36.305 [main] INFO  / io.javalin.Javalin - Loom is available, using Virtual ThreadPool... Neat!
Exception in thread "main" java.lang.reflect.InvocationTargetException
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:119)
	at java.base/java.lang.reflect.Method.invoke(Method.java:578)
	at io.javalin.core.LoomUtil.getExecutorService(LoomUtil.kt:22)
	at io.javalin.jetty.LoomThreadPool.<init>(JettyUtil.kt:90)
	at io.javalin.jetty.JettyUtil.defaultThreadPool(JettyUtil.kt:28)
	at io.javalin.jetty.JettyUtil.getOrDefault(JettyUtil.kt:20)
	at io.javalin.jetty.JettyServer.server(JettyServer.kt:34)
	at io.javalin.Javalin.start(Javalin.java:180)
	at io.javalin.Javalin.start(Javalin.java:148)
	[...]
Caused by: java.lang.UnsupportedOperationException: Preview Features not enabled, need to run with --enable-preview
	at java.base/jdk.internal.misc.PreviewFeatures.ensureEnabled(PreviewFeatures.java:49)
	at java.base/java.lang.Thread.ofVirtual(Thread.java:833)
	at java.base/java.util.concurrent.Executors.newVirtualThreadPerTaskExecutor(Executors.java:274)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
	... 10 more

Expected behavior
Javalin should work (without virtual threadpools) on Java 19 without loom enabled.
To Reproduce
Run Javalin with Java 19 and without the --enable-preview flag.
Additional context
To fix it, in IntelliJ I edited my run configuration, and added --enable-preview to the VM Options section.

@crummy thanks, I assume this is RC2? Could you test against the current snapshot?

Oh, it's 4x. I guess we should backport the fix.

Should be fixed, will release a new version now.

4.6.5 should be out now :)

Now that was fast! Thanks so much!


Addon stops with a strange error on config files !

Hi
Trying to run the addon following documentation but I have that problem:
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 00-banner.sh: executing...
-----------------------------------------------------------
 Add-on: Rclone
 Rclone is a command line program to sync files and directories to and from various providers.
-----------------------------------------------------------
 Add-on version: 3.0.8
 You are running the latest version of this add-on.
 System: Home Assistant OS 8.0.rc1  (amd64 / generic-x86-64)
 Home Assistant Core: 2022.4.1
 Home Assistant Supervisor: 2022.04.0
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
[cont-init.d] 00-banner.sh: exited 0.
[cont-init.d] 01-log-level.sh: executing...
[cont-init.d] 01-log-level.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[15:35:46] INFO: Pruning local files...
[15:35:46] INFO: Pruning local files finished
parse error: Unfinished string at EOF at line 2, column 0
[cont-finish.d] executing container finish scripts...
[cont-finish.d] 99-message.sh: executing...
[cont-finish.d] 99-message.sh: exited 0.
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.

Config file of addon:
configuration_path: /share/rclone/rclone.conf
local_retention_days: 15
remotes:
  - name: NC Perso
    path: /Config_softs/HA/Auto/
    retention_days: 7

Rclone config file (generated using a local rclone on my computer and then copied on hassio system ! (sensible fields pruned ;)
[NC Perso]
type = webdav
url = https://mynextcloudserver/remote.php/dav
vendor = nextcloud
user = vincen
pass = XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

Any ideas what's wrong there ?
Thanks
Vincèn

Hi @vincegre,
Thanks for creating the time to create such an elaborate bug report. That is really appreciated! However I think this is a slight oversight by myself. I see that you have used a space in the name of your rclone.conf. Perhaps that should be an illegal character because of bash escaping 🤷.
Would you care to try without any spaces in the name, like this:
Add-on configuration
configuration_path: /share/rclone/rclone.conf
local_retention_days: 15
remotes:
  - name: NCPerso
    path: /Config_softs/HA/Auto/
    retention_days: 7

rclone.conf
[NCPerso]
type = webdav
url = https://mynextcloudserver/remote.php/dav
vendor = nextcloud
user = vincen
pass = XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

If this works (and I hope it does 🤞), you could also replace the space with an underscore or hyphen like so: NC_Perso.
Thanks! ❤

Hi @vincegre,
I'm going to close this stale issue 😢. Unfortunately I cannot debug this any further without additional input. If this is still an issue or you have another question, please don't hesitate to respond.


Added Ruby

#645

please create another pr , there is an error in the code and it has been not deployed


Terminus2049被当局封杀？

南华早报里提到了这句：

Terminus2049 appeared to be blocked in mainland China on Saturday.

请问Terminus2049是host在GitHub上的项目，当局怎么单单封杀Terminus2049而不封杀整个GitHub？？

咋没封过，第一批封的
https://zh.wikipedia.org/wiki/对GitHub的审查和封锁

It is hard to understand how the CCP censorship machine works in detail.
One theory is that blocking this repo through gov-takedowns is too transparent.
And blocking the entire github.com would of course lead to too much economic damage.

@cirosantilli Thanks for your great analysis. My question is thus: blocking Google entirely would not lead to too much economic damage for China, but blocking github would? Couldn't China come up with its own Github replacement inside the Great Firewall??? (Like it did with Baidu for Google, and Weibo for Twitter, and BiliBili for YouTube?)

@posclegom I think search engines and user provided content websites are fundamentally different, because to make a search engine all you need is one algorithm almost as good as Google's. But you can't force every Western developer to move to a Chinese website once GitHub has so much momentum.

TO cirosantilli

If it gathers more users, it will get banned. Twitter and YouTube have a ton of users, but low value per user, and high revolution organization potential
Thank you so much for your excellent explanation, with the above comparisons form you I finally understand why GitHub is not banned despite so many years in operation and hosting so much anti-communist contents.

@cirosantilli at some point in the near future, with the universalization of DoH and ESNI, all web traffic will become opaque and look the same to each other doesn't matter what website you are browsing. And the authorities will no longer be able to know which websites you are browsing. At that time do you think the communists will finally go and cut off China from the global Internet?

I didn't know about those techniques. Gut feeling is that yes, if unable to block specific sites only, CPP will try to fork the internet completely.

Gut feeling is that yes, if unable to block specific sites only, CPP will try to fork the internet completely.

@cirosantilli we have the exaaaaaaaaaaaact same views here. Let's shake hands. When that day truly comes, I don't know whether I'm gonna laugh or cry. Because while it will mean that people in China will no longer be able to use the real Internet, it also means that we will see a great reduction in the number of communist-hired trolls and spammers that are littering the various forums both here and on other websites and blogs located outside the Great Firewall.

一直以来痛恨404痛恨审查,以至于养成了看文章都要截图的习惯，感谢作者解决这个问题，希望你们能一直运行下去，you are a hero and no alone

@Wangxming 严重为你点👍！说的好！

这个项目已经被dns污染了

@cncherisher has been blocked at least since March 2019 BTW: https://zh.greatfire.org/https/terminus2049.github.io

这个项目已经被dns污染了

@cncherisher 现在dns污染已经out了，关键是有无受到墙sni封杀关照。

中國全國人口普查數字難圓其說　去年僅27萬人死？｜成都高中生校內墮樓亡　家人被拒看遺體及監控｜「端點星」案開審 官派律師：被告受西方思想的影響誤入歧途｜武漢遭遇10級雷暴大風(眾新聞中國組)
https://www.youtube.com/watch?v=X6PudaFwp4g


I18N: update CN/ZH translation file

Describe what problem your feature request solves
As Threat Dragon begins to approach the release of version 2.0, it would be good to update the Chinese translation
Describe the solution you'd like
Modify translation file
Additional context
@taoyiyang1 as you provided the original I hope you do not mind me assigning this to you?

thanks for taking this on @yolylight


No longer maintain Graphite-Observer

No longer maintain Graphite-Observer, remove it, thx.

Codecov Report

Merging #1852 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1852   +/-   ##
=======================================
  Coverage   66.36%   66.36%
=======================================
  Files          55       55
  Lines        6404     6404
  Branches     1267     1267
=======================================
  Hits         4250     4250
  Misses       1934     1934
  Partials      220      220

Continue to review full report at Codecov.

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