added exmples to notebooks LOGIC & PLANNING.

added examples to logic.ipynb:

PropKB
tt_entail
tt_check_all
dpll_satisfiable
WalkSAT
SATplan
unify

added examples to planning.ipynb:

PDLL
Actions
Air_cargo_problem
spare_tire_problem
blocks_world

Sir, I found some examples/diagrams from the other websites good, but I didn't include them as I was skeptical if we can add content from other websites. Please let me know so that I can add them?

Nice work, but there are quite a lot of issues here, I'm afraid.

Spacing issues. You have inconsistent spacing in the text. For example, you write "precondition(positive and negative)" and "same way as before :". In the first case you need a space before the parenthesis, while in the second case you need to remove the space before the colon.

Capitalization issues. When starting a new sentence, you should be using capital letters even if before that you had a code example. It still is a sentence + you break capitalization consistency.

Wording issues. Sometimes you miss the word 'the'. For example, "It consists of following". There should be a 'the' before the word 'following'.

Awkward wording. Some other sentences are awkwardly written and are jarring to read. For example, "The PDDL (Planning Domain Definition Language) allows us to express all actions with one action schema, it consists of following four functions". This sentence is awkwardly written. The "it consists..." part does not look nice. Either it should be a separate sentence, or you should have replaced 'it' by 'which' (if by 'it' you refer to the schema). Right now it is confusing to read.

Confusing wording. This ties in with the above, but a lot of the text is hard to follow. You are using a lot of self-reference words (like 'it', 'the other', etc.) but the link to that reference is lost. For example, "Actions consists of a precondition(positive and negative) and effect(positive and negative). It consists of following". Here, Actions is a class. You write that it consists of a "precondition" and an "effect" and then you write in the next sentence that "it consists of the following". It took me a second to realize you were still talking about the class Actions, since you already told us what it consists of. At first I thought you were referring to "effects", or maybe "precondition". You could have written something like "it also consists of..." which makes it clear you are still talking about the class.

Image issues. In planning, you have added an image. While that is a great idea to help get the concepts across, the image has issues itself. The text is over the diagram, making it harder to read. Also, it would have been better if you had dropped a couple of lines before the image explaining what the reader is seeing. Granted, this can be added in the future, but still.

unknown type. This pops up from time to time in the logic Notebook, for some reason. Not sure why that happens but needs to be fixed.

Formatting/Styling. Most of the Notebooks utilize formatting to help ease the reader in. Italics, bold, etc. help a lot and look nice. A lot of the time I was not sure if what I was reading was an actual word or a function/class name. This can be easily fixed by styling function/variable/etc names as code. You can do that by wrapping the name in left-leaning quotations (the quotation usually found under the 'tilde' key -> ``). It produces more or less the following effect: code. You can use italics and bold font in similar instances too.

These are the technical issues I noticed in your PR. If you fix them, this will be fine. I want to mention another thing too though.
Here you are not explaining what is happening. Sure, you are showing how to use the code, but you are not telling the reader what is happening. I will give just one example, but that is evident in pretty much everywhere.

The following code defines the initial state for the problem

def air_cargo():
    init = [expr('At(C1, SFO)'),
            expr('At(C2, JFK)'),
            expr('At(P1, SFO)'),
            ...
            ...
            ...

This shows how to define the air cargo problem, but it does not help the reader. You should write why does it define the problem. Explain why is init initialized that way. Why use expr('At(C1, SFO)')?
The only way for the reader to understand what this means is to go read the book instead and try figure things on their own, which kind of defeats the purpose of explanatory examples. While this project is complementary to the book, the examples should have some substance of their own. Otherwise people would just take a look at the code and the tests to get a feel of how to get this to work.
In the above case, you are only showing how to define one single problem. What your example should do is show the reader how to define any problem.

Of course, this is up to @norvig, but personally I feel this should not be merged as is.

While I do agree with with @MrDupin , I think this PR can be merged as it serves as a starting point which can be improved upon. The only major problems which I have with this PR is, as Dupin metioned, some of the definitions(Air Cargo, Spare Tire) have only code but no explanation, and that it has too many unrelated commits and changes.
I recommend splitting this into two seperate PRs. One which adds examples to the notebook, and one which modifies the fol_fc_ask function. The current PR has too many commits which have too vague a description and no indication what changes they bring. It would be better if the commits represent unit changes, like 'Added example for Air Cargo' or 'Updated fol_fc_ask()'.

Just to clarify, I'm not saying the changes shouldn't be merged. They are helpful and they can definitely be merged, but not before the technical issues are dealt with (at least in my opinion).
I agree with @Chipe1 about splitting the PR. I would suggest you fix the issues first (maybe even posting them here for review) and then make the new PRs. That way the new PRs will not get bloated with patches.

@MrDupin Thank you for going through and letting me know how to improve upon. I have done some improvements and am working on more.
@Chipe1 The PR comes up with too many commits which have been already merged before and I have been trying to get rid of this. Let me know if you can guide.

@sofmonk: This is better. I only had time to skim over it, but I noticed some small issues I will quickly point out:

At the start you wrote "The PDDL class include". It is 'includes'.

In the list below that, I would suggest you make the points bold (eg. "goal_test(self):" should be bold) and write the references to functions inside these points using the code style. (eg. "initialises goal_test with kb." should be "initialises goal_test with kb.")

In the "Air-Cargo Problem" opening paragraph, you wrote "we have to define a initial" but it is 'an' instead of 'a'.

I would like to see though short introduction to the problems. For the air-cargo problem, drop a couple of lines on explaining the problem at hand.

@sofmonk One way to fix it is to diff your current branch with master, create a fresh branch which is up to date with master and apply the diff to the newly created branch.
To prevent cluttering of PRs, use one branch per PR and delete it once the PR is merged. You need not update the PR branch to the aimacode/aima-python:master unless you wish to fix any merge conflicts. There probably won't be any conflicts if the PR makes small, local changes.
Also try not to reuse commit comments within a PR as it makes changes confusing. Make it more descriptive like 'Added missing newline characters' instead of 'Update logic.ipynb'.

@MrDupin @Chipe1 Thank you for all the improvements (more to come) PR #427


scroll performance issues

Steps to reproduce
Scroll content switching leads to a decrease in program performance
Expected results
fast
Actual results
slow
Code sample
The test source code: myapp.zip
Screenshots or Video

Logs
No response
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.1, on Microsoft Windows [版本 10.0.22631.3155], locale zh-CN)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.3)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.86.2)

Hello @xyun52.
Thanks for filing this.
I tested this on the latest master and stable channels but can't seem to reproduce on the master channel.
https://github.com/flutter/flutter/assets/53122008/1b9c75ab-261e-41d7-8cb3-18ac0ff1ec49
Can you please check this video out to see if the slowness you are describing exists or if you are able to verify on the master channel as well?

Your frequency is not enough. I started noticeably declining in performance around 7 times @dam-ease

Steps to reproduce
Scroll content switching leads to a decrease in program performance
Expected results
fast
Actual results
slow
Code sample
The test source code: myapp.zip
Screenshots or Video

Logs
No response
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.1, on Microsoft Windows [版本 10.0.22631.3155], locale zh-CN)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.3)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.86.2)

Hey actuallly i have resolved performance issue

I fixed this issue but i am not able to generate pull request

@dam-ease Not only scrolling, please note that the time for adding data to each frame is also getting longer and longer

The first click

20th click

This bug affects the entire app and the pop page cannot improve, so the webpage must be refreshed

Thanks for your responses @xyun52 @Dev2409.
I see that this occurs only after multiple clicks and can reproduce on my end as well after about 10+ times.
https://github.com/flutter/flutter/assets/53122008/f6377200-c5a2-4ca4-8ee9-21cfacb15243

I fixed this issue but i am not able to generate pull request

@Dev2409 What issues are you facing with this?
stable, master flutter doctor -v

[!] Flutter (Channel stable, 3.19.1, on macOS 14.2.1 23C71 darwin-arm64, locale
    en-NG)
    • Flutter version 3.19.1 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision abb292a07e (2 days ago), 2024-02-20 14:35:05 -0800
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.86.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (3 available)
    • Damilola’s iPhone (mobile) • 00008110-001964480AE1801E • ios            •
      iOS 17.1.1 21B91
    • macOS (desktop)            • macos                     • darwin-arm64   •
      macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)               • chrome                    • web-javascript •
      Google Chrome 121.0.6167.184

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.20.0-11.0.pre.107, on macOS 14.2.1 23C71 darwin-arm64, locale en-NG)
    • Flutter version 3.20.0-11.0.pre.107 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `flutter` on your path resolves to /Users/damilolaalimi/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fcd154bd24 (2 hours ago), 2024-02-22 20:24:21 -0800
    • Engine revision 0f3ad23b84
    • Dart version 3.4.0 (build 3.4.0-167.0.dev)
    • DevTools version 2.33.0-dev.11
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.86.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (4 available)
    • Damilola’s iPhone (mobile)      • 00008110-001964480AE1801E • ios            • iOS 17.1.1 21B91
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 121.0.6167.184

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

hey @dam-ease i am attaching the screen recording so can you please check is the issue resolved?
https://github.com/flutter/flutter/assets/91386391/5ae464bb-5c21-4005-a915-3129581ab067

@dam-ease SingleChildScrollView also has issues

Can someone provide a code sample within this issue we can run to reproduce the issue?

I downloaded the .zip file, but please refrain from providing code in this way in the future.
The issue here is that the CustomScrollView is full of SliverToBoxAdapters. This is not going to be performant as the children will not be laid out lazily. Instead, you could use a ListView.builder to build rows on demand, or more directly a SliverList of SliverFixedExtentList.

This is not a scrolling performance issue caused by element height, it is a decrease in application performance caused by multiple switching of scrolling content @Piinks

I created a new sample with automatic switching @Piinks
newdemo.zip

@dam-ease Can this problem be solved


fix webpack2 warning

Hi guys. What about status for this request?

I'll take a look this weekend. Although PRs appreciated :)

This patch looks harmless. @WearyMonkey, can you update status regarding this issue


37 CFR 1 - notice amendment generates unexpected instruction label

Dev environment: current Master [ 89535d0 ] + fr-notices PRs 7 & 8.
Running eregs --debug fill_with_rules 37 1 results in a failure when processing 68 FR 48286 amendment 19 at regparser/notice/amendments/appendix.py:16.  This failure was previously encountered by @meriouma and a workaround implemented here.
Here's the code which fails when trying to access an invalid index in label_parts.
def content_for_appendix(instruction_xml):
    label_parts, amdpar = label_amdpar_from(instruction_xml)
    if len(label_parts) > 0 and 'Appendix' in label_parts[1]:

ipdb gives the following context:

instruction_xml:  <POST label="2[title]"/>
len(label_parts): 1

etree.tostring(amdpar):
'<AMDPAR>19. Add a new center heading and §§2.200 and 2.201 to read as follows:<EREGS_INSTRUCTIONS final_context="2-?-201"><POST label="2[title]"/><POST label="2-?-200"/><POST label="2-?-201"/></EREGS_INSTRUCTIONS></AMDPAR>\n

I am not familiar enough with the FR Notice parsing code to recognize what this might be, any ideas?

Hey @gregoryfoster, I just submitted #370 to fix the explosion here -- @meriouma was definitely right.
A quick note for context: even though you're parsing part 1, we use a single structure to represent notices (which may include amendments to multiple parts.) This is leading to a seemingly unrelated amendment messing up your parse -- sorry that that's not obvious!
Second, regarding the specific amendment: it's adding a "heading" in front of two sections, which sounds similar to a subpart or a subject group, but doesn't match any concept we have; we have headings within sections, but not between them. In any event, something to keep an eye on if you're planning to parse part 2 later.

Thanks for your quick fix, @cmc333333.  There's an identical problem for interpretations which I've fixed in PR #371, though please see my comment about my accidental inclusion of your commit from #370.
Unfortunately, I'm continuing to have trouble with this particular amendment for the reason you anticipated in your comment: headings between sections.  Aside, this notice (68 FR 48286) repeats that same instruction pattern in amendment 20.
@meriouma identified this error and coded a workaround here.  However, I've begun to feel like we're kicking the can down the road as the last three errors in 37 CFR 1 are caused by this particular notice amendment pattern.
After applying PRs #370 & #371, the parser stops on create_xml_changes(amended_labels, section, notice_changes) in regparser/notice/amendments/fetch.py:121
ipdb> print amended_labels
[(POST, ['2']), (POST, ['2', '200']), (POST, ['2', '201'])]
ipdb> print section
None
ipdb> print notice_changes
<regparser.notice.changes.NoticeChanges object at 0x10a74add0>

section being None is a problem, and that traces up to fetch_amendments(notice_xml) in regparser/notice/amendments/fetch.py:76, where the cache presents the following content object for the amendment:
ipdb> print content
Content(struct=None, amends=[(POST, ['2']), (POST, ['2', '200']), (POST, ['2', '201'])])

My interpretation is the parser isn't sure where to place this first instruction (the title between sections).  What's your read, and where do you think we should go from here?

Hey @gregoryfoster, sorry for the delay!
As a quick reproduction recipe, we can run
eregs clear
eregs preprocess_notice 03-20489
eregs write_to somewhere

You'll see the error in the last step; when we added the notice-and-comment feature, we began including the list of changes in the output JSON for each notice. Unfortunately, this increased the chance that misparses of an unrelated regulation part might cause yours to explode. I think the most immediate path forward is to wrap that step in a try-catch (with a warning); this field isn't used unless we're displaying a notice during the N&C period.
Accounting for the specific error's a bit more tricky as that header is causing two issues. When we try to piece together what changes amendments are making, we effectively take two passes; the header causes an issue with each.
In the first phase (a preprocessing plugin), the AMDParser looks at the amendment instructions (AMDPARs) and tries to derive a more machine-readable list of changes. It then inserts its analysis into the XML metadata (so that we aren't modifying the actual amendment text). This process is trying to derive meaning from natural language, so it's rather error prone. In this case, it tokenized amendment 20 (of 03-20489, "Add a new center heading and §§2.206 through 2.209 to read as follows:") into

create
heading (without a known location)
2.206
2.209

Combined with contextual information those tokens lead to instructions to create a new heading for all of part 2 and create new sections 2.206 and 2.209. This is clearly not what we wanted -- not only should the heading not apply to the whole of 37 CFR 2, we're not accounting for the "through", which would imply the creation of 2.207 and 2.209 as well. So, we'd want to change the AMDParser's logic (and potentially grammar) to account for a situation like this. I'll note here that the preprocessing step will skip any amendments which already have EREGS_INSTRUCTIONS -- it's therefore possible that one could manually add the XML to get the right effect.
The second phase involves finding XML for specific instructions (i.e. the "contents" of the change rather than the instruction). The error you're seeing indicates that the parser's not finding the appropriate XML for the 2, 2.206, nor 2.209 instructions. As noted above, the first is understandable (and should probably be fixed in the AMDParsing stage) but the latter two are likely a bug with the logic that determines which XML section to grab. It looks like this logic wouldn't find the sections due to that extra header. That should be a pretty small change, I'd think.
Hope all the context helps! Let us know if we can help.


Cycle Gan typo in Excercise Notebook

The get_data_loader signature in the CycleGAN_Exercise notebook has image_dir='summer2winter-yosemite'.  It should be image_dir='summer2winter_yosemite' (underscore not dash) to match the solution and the actual directory from the zipfile.

This issue has been resolved.


Allow agent to use gRPC roundrobin loadbalancing when talking to collector

Requirement - what kind of business use case are you trying to solve?

Currently users are only allowed to specify static list of collector hosts/ports for agents to talk to. We should provide a mechanism to have agent talk to a gRPC roundrobin load balancing backed by external dns service
Problem - what in Jaeger blocks you from solving the requirement?

None
Proposal - what do you suggest to solve the problem or improve the existing situation?

We add the option for user to specify external naming.Resolver for gRPC.
Any open questions to address

gRPC roundrobin load balancing backed by external dns service

We have it already: https://www.jaegertracing.io/docs/next-release/deployment/#discovery-system-integration . We even make use of that with the Jaeger Operator: https://github.com/jaegertracing/jaeger-operator/issues/332

please update title and description, they are not accurate. What you're really trying to do is support custom discovery and load balancing configuration.


Implement deleteObject(persistenceId: String, revision: Long) properly

As discussed in https://github.com/apache/incubator-pekko-persistence-r2dbc/pull/8#discussion_r1158930062, the current implementation of deleteObject(persistenceId: String, revision: Long) just defers to deleteObject(persistenceId: String) (which means it ignores the revision).
The implementation should instead be improved so it only deletes correct revision.

@mdedetrich do you want this in 1.0.0?

@mdedetrich do you want this in 1.0.0?
Should go in 1.1.0

#92 merged


added typescript support

I needed this in typescript so i thought i would share. Tested with examples in readme.

Well, this is not getting merged, but maybe someone will use it.

https://github.com/cupglassDEV/random/tree/main
Hey, i made this typescript package that solves this problem. Its currently published on JSR, so you can add the package to Node without transpilling

The api is a little bit diffrent, because i want it to be more strict than before


Remove lowercasing of repo names

The lowercasing of repo names is entirely unnecessary, and only serves
to obfuscate the repo list in the ui and to complicate things for anyone
trying to use the rest api. By keeping the repo names intact, there is no
longer a need to first call the repos endpoint in order to lookup an
actual name, if the name is the only piece of information that is needed.

Thanks. I'm pretty sure the lower-casing is a relic. I can't even remember what problem it originally addressed.


valueGetter and select cellEditor conflict

when I implement to a column field and cellEditor all is good. but when Its cellEditor and valueGetter 'all hell breaks loose! the 'cellValueChanged' event wont fire.
Im writing in angular 2 if this an important matter regarding this issue.

you need either field or newValueHandler set on colDef for editing to work
maybe you should be using newValueHandler instead of listening on the event


Suggestion: Incorporate paper impact into visualization

It could be useful to have nodes corresponding to higher impact papers visualized bigger visually.

@pavel-jahoda-sewts
Now high-impact papers can be found. Please click on the link below to check the latest version.
https://yuukicammy--conference-projector-wrapper.modal.run/en
Thanks!


aiohttp 0.17 release plan

I believe we have big enough list in our CHANGES.txt for publishing new release.
What open issues do you want implement before releasing?
Let's make a list if needed.

Let's do that by the end of next week, I'd like to work on routing subsystem
Отправлено из Outlook
On Fri, Jul 3, 2015 at 6:15 AM -0700, "Andrew Svetlov"<EMAIL_ADDRESS>wrote:
I believe we have big enough list in our CHANGES.txt for publishing new release.
What open issues do you want implement before releasing?
Let's make a list if needed.
—
Reply to this email directly or view it on GitHub.

@fafhrd91 ok, let me know when you will be ready.

As I see new routing system and signals are not ready yet but we have a lot of pending changes waiting for new release.
I suggest making cut-off and release implemented bugfixes/features right now.

ok

On Aug 4, 2015, at 11:09 AM, Andrew Svetlov<EMAIL_ADDRESS>wrote:
As I see new routing system and signals are not ready yet but we have a lot of pending changes waiting for new release.
I suggest making cut-off and release implemented bugfixes/features right now.
—
Reply to this email directly or view it on GitHub https://github.com/KeepSafe/aiohttp/issues/431#issuecomment-127696422.

Done


[AutoPR containerservice/resource-manager] Revert node public ip support from 2019-04-01 model.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5841

(message created by the CI based on PR content)

Codecov Report

Merging #5050 into restapi_auto_containerservice/resource-manager will increase coverage by <.01%.
The diff coverage is 75%.

@@                                Coverage Diff                                 @@
##           restapi_auto_containerservice/resource-manager    #5050      +/-   ##
==================================================================================
+ Coverage                                           49.13%   49.13%   +<.01%
==================================================================================
  Files                                                6486     6486
  Lines                                              182098   182096       -2
==================================================================================
  Hits                                                89478    89478
+ Misses                                              92620    92618       -2

Impacted Files
Coverage Δ

...tainerservice/v2019_04_30/models/agent_pool_py3.py
26.31% <100%> (+1.31%)
:arrow_up:

...naged_cluster_agent_pool_profile_properties_py3.py
26.31% <100%> (+1.31%)
:arrow_up:

...0/models/managed_cluster_agent_pool_profile_py3.py
71.42% <50%> (ø)
:arrow_up:

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 ca5a780...77aacd7. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5009


Add Networking! ACK! book by Julia Evans

What does this PR do?
Add resource (book)
For resources
Description
This is a zine about COMPUTER NETWORKING!
Why is this valuable (or not)?
The book provides knowledge regarding different networking concepts in a fun and engaging manner.
How do we know it's really free?
There is a free download link to the book on the author's website and the author mentions it as a freebie.
For book lists, is it a book? For course lists, is it a course? etc.
It's a book.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Julia is amazing.


Use conda-forge for Dask in Travis (not special branch download)

DO NOT MERGE UNTIL..
https://github.com/dask/dask/pull/2654 is merged.
Because we were temporarily using a "jcrist/masked-array" branch of dask
-- which is now merged.
See https://github.com/SciTools/iris/pull/2747#issuecomment-327003711

STATUS UPDATE:

PR was merged, but no version yet defined at that time
latest code has modded this to reference latest-master
that is still not the desired endpoint !

Later... there is now a tagged version 0.15.3, but unfortunately it introduces some new problems for us #2769

STATUS UPDATE:
Latest Dask version (0.15.4) should now be suitable
-- let's do this ...


ThreadSanitizer: data race on socket close/receive_from.

fix https://github.com/ros2/rmw_fastrtps/issues/391,
Signed-off-by: Tomoya.Fujita<EMAIL_ADDRESS>

As you can see here all tests are timing out when closing.
I am almost sure that this race is not harmful, so I would prefer a patch that avoids thread sanitizer to complain.

@MiguelCompany
I am taking this back, since I believe,

this is false positive.
seems to be not a problem in fastrtps.
even if there is data race, fastrtps is not the right place to fix but boost.


“Unable to find project.json or *.csproj” while creating a new Interface or class in VS Code(1.32.3)

I am getting this issue "Unable to find project.json or *.csproj" while creating a new class or interface in my Angular project in vs code. It was working fine, but accidentally I am getting this issue. Due to this issue, I am unable to create new files in my project. I can't find any solution from previous ones, so don't do it duplicate. please help me.

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Error Unable to find project.json or *.csproj (#68356)

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Error Unable to find project.json or *.csproj (#68356)

nope, the previous one also did not resolve.

Please file the issue against the C# extension. Thanks!


waitForConnectedPeer() does not wait

This is a bug report
Problem
await waku.waitForConnectedPeer() does not actually wait for connected peers
In the following code snippet and log, you can see that getPeers returns an empty set after calling await waku.waitForConnectedPeer(). After manually waiting 2 seconds, the second call to getPeers is not empty.
  waku = await Waku.create({ bootstrap: true });
  log('await waku.waitForConnectedPeer()');
  await waku.waitForConnectedPeer();
  log('waku.relay.getPeers():', waku.relay.getPeers());
  log('sleep 2s');
  await sleep(2000);
  log('waku.relay.getPeers():', waku.relay.getPeers());

log:
await waku.waitForConnectedPeer()
waku.relay.getPeers(): Set(0) {} <---- unexpected
sleep 2s
waku.relay.getPeers(): Set(1) { '16Uiu2HAmL5okWopX7NqZWBUKVqW8iUxCEmd5GMHLVPwCgzYzQv3e' } <---- expected

Proposed Solutions
await waku.waitForConnectedPeer() should wait until a peer is actually connected
Notes

js-waku version: 0.14.2 AND 0.15.0
node version: 16.6.2
ubuntu 20.04 under wsl2

Investigation done.
The behavior is actually expected. Waku.waitForConnectedPeer() returns the peers that are connected and identified by the libp2p node.
WakuRelay.getPeers returns the peers that are part of the gossipsub sub mesh. ie, peers to whom relay messages are sent and peers that would forward relay messages.
Learn more about mesh in the gossipsub specs: https://github.com/libp2p/specs/blob/master/pubsub/gossipsub/gossipsub-v1.0.md#gossipsub-the-gossiping-mesh-router
If getPeers does not return peers just after waitForConnectedPeers it is because some processing needs to be done and control messages need to be exchanged for peers to be in each other's mesh.
Moving forward I can see several solutions that can be implemented separately or together:

Rename getPeers to getPeersInMesh to make it clearer that the peers returned are not just any connected waku relay peer.
Provide a waitForPeersInMesh method on WakuRelay that would simply wait for peer to be part of the gossipsub mesh

@dao, let me know if implementing the 2 solutions would help or if you would like another API to fit your needs.

from the examples and docs I've seen, awaiting waitForConnectedPeers signals that everything is ready to go - ie, that messages can be relayed . If "everything is ready to go" means that peers are in mesh, then I would lean towards the contents of waitForConnectedPeers being replaced rather than adding a new function waitForPeersInMesh since the existing examples devs find (eg waku uber) are unlikely to be updated  from waitForConnectedPeers to waitForPeersInMesh when the latter is actually intended.

Sounds good. Will action.

You can use this workaround in the meantime, after or instead of waitForConnectedPeers.
      await new Promise((resolve) =>
        waku.libp2p.pubsub.once('pubsub:subscription-change', () =>
          resolve(null)
        )

After further investigation and the implementation of waitForRemotePeer, I'd like to clarify that there is no way to guarantee that a relay node is ready on both sides.
Indeed, for Alice to start sending messages to Bob via relay, the following needs to happen:

Alice connects to Bob
Alice & Bob exchange identify messages, both agree to open a stream with relay protocol.
Alice sends a subscribe to Bob for Waku Relay default pubsub topic
Bob processes the subscribe by adding Alice to the peers for the given pubsub topic and sends a subscribe too.
Alice needs to maintain her gossipsub mesh (heartbeat), if there are not enough peers (less than 6) in the mesh for the given pubsub topic then she'll add a peer from the topic to it (Bob), send a CTRL GRAFT message to Bob. Now Alice will forward messages to Bob for the given topic.
Bob does the same.

For Alice's point of view, it is possible to know if steps 1 to 5 have happened and that is what waitForRemotePeer does.
However, Alice cannot know (as far as I understand) whether step 6 happened. So while Alice may start sending relay messages from 5, Bob may not send relay messages to Alice just yet.
So in a test environment, I see no choice but waiting a bit: https://github.com/status-im/js-waku/blob/b257885204ecfc5c8357824ed76fa6d0728740a1/src/lib/waku_message/index.node.spec.ts#L49
In a production environment, other steps should happen (such as retrieving messages using Waku Store) before an app is "ready". so the race condition is not as likely.
Note that work is in progress to make the WakuRelay.send method provide more feedback and actually return the recipients to whom a message was sent: https://github.com/libp2p/js-libp2p-interfaces/pull/119


Problem with quoted Strings ending with N

If I try to create an Migration File for oracle the the following:
insert into table (COLUMN) values ('TESTWITH_ENDING_N');
insert into table (COLUMN) values ('some other quoted string');

The script fails because flyway does not terminate the first statement correctly. I will also provide a pull request to actually fix the problem.

duplicate of #971


Explorer tweak before new design v0.9

We will work with @exezbcz about some small and impactful changes to explorer before we come back to redesign the whole explorer after the collection view.
Primary, we will be putting filters on the side like any other marketplace has, which we will be pretty great about.
Plus, we can update buttons to upcoming styles too.
The thing is, we probably won't have time to circle back sooner than 6 weeks tbh
We mostly likely keep all functionality we have
If you have suggestions for explorer, now is your time.
Proposed changes

put filters on the side
update pagination button to neobrutalism style
change the grid of items default based on min-width per card, i.e. @ 1024px will be 4, at @ 1920px be 6-8
anything else you are wishing for xmas
rename Gallery -> Items
add blockchain switch

Current state

A while ago (#2947) I had started to work on something like this.

If you have suggestions for explorer, now is your time.

I would like to:

be able to collapse side filters
move sorting option in the top right
change grid card size
filter by media type (#1162)

Note:

should we keep pagination?  I don't find so much use since we have infinite scroll

A while ago (#2947) I had started to work on something like this.

If you have suggestions for explorer, now is your time.

I would like to:

be able to collapse side filters
move sorting option in the top right
change grid card size
filter by media type (Filter for file type #1162)

Note:

should we keep pagination?  I don't find so much use since we have infinite scroll

Nice! good ideas.
For version 0.9, I believe we will keep things basic before returning to it once the collection detail and other tasks are completed.
Thus we won't be adding many new features to the makeover. It would be only layout change and bit of a reorganizational change.
Things that are going to be in the makeover:

collapsible burger menu
layout change of the filters
items in grid

I dont think we will add filter by media to this version.
ad pagination - @yangwao said its a feature
Thanks!

should we keep pagination?  I don't find so much use since we have infinite scroll
ad pagination - @yangwao said its a feature

Yes, we can drop pagination (looking R & OS), doesn't make sense now and eats some space.
yet pimp infinite scroll to randomly not bounce on scroll up and test addressing properly. I recall we have bug when you go on item and return back it jumps you on different view port of page

Closing and continue to #4474


FALCON-2173 BacklogEmitterService should emit 0 for process which don…

…ot have backlog

Have done the dev testing for restart of the server and onAdd method.

+1. Will merge shortly.


Delete in edit pane unintuitive

When deleting a link in the edit pane that has multiple keywords you can unintentionally delete multiple keywords if you don't know what you are doing.
If you hit delete on this one, all keywords shown would get deleted.

While it's not the best solution (though probably something that should exist anyways), I have staged some javascript to confirm that the user wants to delete the links before actually deleting them.  @moronbros if you have any feedback on this change let me know.

ship it!


Updated the solution for Oddplaced_evenplaced_sum.cpp

Please see the NOTE:- It is clearly mentioned that you have to attach ss of code and output

This issue is closed


Toggle for maximum projectile definition clamping in the editor

Is your feature request related to a problem? Please describe.
In GB Studio 3.0, the editor would compile scenes with more than 5 projectile definitions. In GB Studio 3.1 however, the editor will remove any projectile definitions over the 5 definition limit. This means even when adjusting the engine to support more than 5 projectile definitions (MAX_PROJECTILE_DEFS in projectiles.h), it is no longer possible to include more than 5 projectile definitions in a scene even if it does not crash the game.
Describe the solution you'd like
Please add a toggle that reverts to the old editor behaviour.
Describe alternatives you've considered
I can use VM_PROJECTILE_LOAD_TYPE for some cases, but it is not the same as the previous functionality. Another alternative could be the editor projectile clamp respecting MAX_PROJECTILE_DEFS in projectiles.h, if possible.

This clamp code should probably just be removed from the IDE. It duplicates the code in the engine for seemingly no benefit

https://github.com/chrismaltby/gb-studio/commit/7ff62f2752f0616fe31a314d074540f4f0d0874c
Brought up again recently in the discord, i mentioned this when the commit was going through, but agree, the projectile defs limit should only be a warning, and not enforced on compile, so users can tinker with the engine and pushing the defs limit, as its a trade off in ram.
Removing this line should fix it, while keeping the warning.
https://github.com/chrismaltby/gb-studio/blob/7ff62f2752f0616fe31a314d074540f4f0d0874c/src/lib/compiler/compileData.js#L956


[SPARK-24499][SQL][DOC][Followup] Fix some broken links

What changes were proposed in this pull request?
Fix some broken links in the new document. I have clicked through all the links. Hopefully i haven't missed any :-)
How was this patch tested?
Built using jekyll and verified the links.

cc @dongjoon-hyun @gatorsmile @xuanyuanking

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4104/
Test PASSed.

Merged build finished. Test PASSed.

Test build #97567 has started for PR 22772 at commit 2048c39.

I think we better do some proof readings before doing multiple minor followups.

@HyukjinKwon Sure..

It's okay. the doc fix was huge and there should likely be some mistakes. I will read it closely too this weekends.

LGTM
Let us merge this PR first. If we find more, feel free to open a new follow-up PR

Thank you @gatorsmile @HyukjinKwon @xuanyuanking


DynamicalSystems extension

This is a very naive extension to add compatibility for DynamicalSystem types. This would make using GAIO.jl an easy addition to a julia dynamical system analysis pipeline, and allow one to use the integrators from DifferentialEquaitons.jl. This implementation is highly inefficient. Unfortunately there is a ton of overhead within DifferentialEquations, so I don't know if it's even possible to make an efficient implementation. I'll keep trying some things.

Ok so I've reduced the performance difference down to ~2x with comparable memory use. The benchmark file is added to the repo for reproducibility; here are the results:
Lorenz system: mapping a box covering a nonzero equillibrium

(top: hardcoded RK4, middle: using DynamicalSystems, bottom: Using DifferentialEquations)
Henon system: computing the relative attractor

(top: Simply using the map, bottom: using DynamicalSystems)
I'll add some info about using DynamicalSystems into the docs, then this PR will be ready.

Ok, awesome.  This seems to be ready, so can I merge it?

Alright, it is now ready to merge. I just wanted to test the PR on a GPU first, tests indeed pass.


[Service Bus] Remove requiresPreprocessing, compatibilityLevel

We have learnt from the service team that requiresPreprocessing and compatibilityLevel are not properties they expect to be set on rules. This PR cleans them up

/azp run js - servicebus - tests

/azp run js - servicebus - tests

Good to merge if the tests succeed

/azp run js - servicebus - tests


Change CUI and SUI minting to base64 approach

CUIs generated will be base64 of the CodeID that is the initial creator if they do not already exist in UMLS. SUIs will be of the string itself the same way with same considerations.

Completed


Feature Request: Add option to disable Bluetooth Settings popup with ESTDeviceManager

Basic information
Estimote iOS SDK version: 4.27.0
iOS device(s) affected: All models (tested on iPhone 5s and iPhone X
iOS version(s) affected: iOS 11.2.6 and iOS 11.3
Additional information
Please read the checklist and place x sign where apropriate for your use case.

[x] My beacons have Estimote Monitoring enabled (check it in the Estimote Cloud, or via app).
[x] My iOS device/devices supports BLE (iOS must be 5.0 or higher).
[x] My app has NSBluetoothPeripheralUsageDescription in its Info.plist file.
[x] My app has Location Permissions granted.
[ ] My iOS device/devices have Bluetooth enabled.
[x] My app has NSLocationWhenInUseUsageDescription and NSLocationAlwaysAndWhenInUseUsageDescription keys in its Info.plist file.
[x] My app has NSLocationAlwaysUsageDescription or NSLocationWhenInUseUsageDescription in its Info.plist file.
[ ] If my app is supporting iOS 10 and earlier, and has NSLocationAlwaysUsageDescription key in its Info.plist file.
[x] If my app has to run in the background, it has UIBackgroundModes key with value bluetooth-central in its Info.plist file.

Description
I'm creating an iOS app that uses Estimote beacons. It also includes beacon telemetry.
When I launch my app with Bluetooth Off in Settings App, I get a popup on screen to go to settings to turn on bluetooth.
I only get this popup when I instantiate the ESTDeviceManager as follows. If I don't include the below lines, I don't get prompted to turn on bluetooth, but then my Telemetry won't work.
self.deviceManager = [ESTDeviceManager new];
self.deviceManager.delegate = self;

Is there a way to prevent being prompted to go to settings? I have tried the following to see if it would override what's in the EstimoteSDK:
    CBCentralManager* bluetoothManager =[[CBCentralManager alloc]
                    initWithDelegate:self
                    queue:dispatch_get_main_queue()
                    options:@{CBCentralManagerOptionShowPowerAlertKey: @(NO)}];

Expected behavior:
Have a way to disable the bluetooth settings popup. Get the popup if we set a flag to true
Actual behavior:
Popup to go to settings to turn on bluetooth appears

I have tried rolling back to Version 4.26.3 and the bluetooth dialog no longer appears.
Was there a flag set inside the old version of ESTDeviceManager that has now been changed in the newest version?

I checked and no, there was no such change. Maybe it's something else you've also done in your code?


ngrok: 2.3.40 -> 3.0.2

Description of changes
ngrok v3 was released this morning. Changelog from v2 is here:
https://ngrok.com/docs/ngrok-agent/changelog
And a detailed upgrade guide is here:
https://ngrok.com/docs/guides/upgrade-v2-v3
Notably, arguments must now be double hyphens: --token. Previously,
single hyphens were accepted but those invocations will now error. The config
file format has also changed and the binary will warn with how to fix it.
Things done

Built on platform(s)

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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[x] (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

[ ] Fits CONTRIBUTING.md.

Tested execution of the amd64 binary and ngrok http 8080. I also work at ngrok and am part of this release.

@bobvanderlinden let me know if there's anything I can do to make this easier to review. Thanks!

the aur package is at .3
also
https://ngrok.com/docs/guides/upgrade-v2-v3

Thanks for the review! Do you mean we should rename ngrok-2 to ngrok-3 (since this upgrade is intended as a replacement)? Or did you intend to create a new ngrok-3 package?
I have reservations about the second approach if that's what you are getting at. Wanted to clarify before I made the changes.

Awesome, renamed ngrok-2 to ngrok-3 and removed the ngrok-2 attribute in all-packages.nix. I think this is what you were going for?

And I broke something. I'll dig into it :eyes:.

Had to generate docs. CI passes now.

@bobvanderlinden I think I addressed your concern. Let me now what you think. Thank you!

Moved documentation to 21.11 from 21.05. Everything else should be similar.

Friendly bump. Hope to get this one out before 22.11 everyone :crossed_fingers:! Thank you!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/523


Can't build project for iPhone simulator

i' m facing issue while building ios build
app taking too much time load and not complete the build
already added all integration documents and files in pod and info.plist but still facing issue
the same working fine in android

@JiyadAhammad Hello. Can you attach the error log? Thank you.

Sort the Problem by reducing the number of dependency added in pod file
its solve my issue


FUSE protocol violation for server mounted at /usr/local : cache incoerent ! To prevent data corruption disable the data cache by mounting with -o direct_io

Hello.
I'm working hard to test nomadbsd and to customize it as I want. I'm preparing a list of functions to improve and I will propose them to you soon. For example,at the moment,sometime I get the error that you see on the pictures below that it needs to be fixed,otherwise nomadbsd freezes and nothing works anymore. The error is "FUSE protocol violation for server mounted at /usr/local : cache incoerent ! To prevent data corruption disable the data cache by mounting with -o direct_io or as directed,otherwise by your FUSE documentation"

who knows,if it has been fixed ?


Menus: Create context menu component (single column) to render menu based on JSON document as specified

As a Platform.Bible extension developer, I want to be able to add menus to my webviews and contribute to extensible points in platform-provided menus.

This particular task is about having menu components that take a "final" menu JSON document for a single column, context menu and display the menu items appropriately. The menu JSON document definition is in the menu contribution design document.
It is important for the component to align with the schema from the design document.
Note that the items in a menu should be considered fixed while the menu is open/drawn on the screen.  Once the menu is closed/hidden, its content should be reloaded before rendering again because extensions might have been enabled/disabled or something else might have changed that would modify the list of available menu items.

Note: The GridMenu component allows for multiple columns (and has the class "papi-multi-column-menu"). If we use that component, even though there is a single column, it will display a "column name" at the top, which I assume is not what we want. I'm thinking we probably want to factor out the part of MenuColumn that maps items to MenuItems to use for our context menu and then us that component internally to implement MenuColumn.


Can't unbind hammer from element (jquery) still not working

This is still problem as http://jsfiddle.net/LSrgh/65/ shows. Any suggestions?

@jdinard is this still true?

I've updated you jsBin example from above to work.
var onkeys="touch tap release";
var ham = Hammer(el,{prevent_default: true});

var fun = function(ev) {
    console.log(ev.type);
};

ham.on(onkeys, fun);
ham.off(onkeys, fun);

Working jsbin for how to not leak memory: http://jsfiddle.net/LSrgh/141/


drivers/sensor: lsm6dso: Add drdy_pulsed property in DT

Add drdy_pulsed property in Device Tree in order to select how data ready irq should behave (either pulsed or latched mode). Moreover change/fix the API called to set drdy irq mode. (fix #51944)

@teburd Do you have any concerno with this PR?


just a console.log

https://github.com/readium/r2-shared-js/blob/66313d619b75a47b8476e95f31492cb976c4759e/src/_utils/zipHasEntry.ts#L20

Fixed via https://github.com/readium/r2-shared-js/commit/280e30802df7dac0f982d7990df032eadb2e84f9


odd rendering of the "full block" starting in recent updates

This could be one of many things, but a recent update to this package has made it difficult to use certain fonts, particularly (but not exclusively) those with ligature support. I like to use the minimap-titles package to quickly make ASCII art using FIGlet. The problem is that the "U+2588 : FULL BLOCK {solid}" character no longer seem to be rendering appropriately.
Here's what Fira Code does now.

Here's Source Code Pro (looks good!)

And here's a bonus issue occurring in Inconsolata (I'm not sure if this was happening before though).

I've tried basic end-user things like fiddling with different themes for UI/syntax, and poking around the config file, but I'm not sure what altered the rendering of some of these characters in the first place.

Hmm. Unless by "recent" you mean v3.4.0, which was released almost a year ago, I don't have any ideas what might have changed.
FWIW, FiraCode works fine on my end (Atom v1.43.0, fonts v3.10.0):

I have this in my stylesheet:
atom-text-editor {
  font-feature-settings: "zero", "ss01", "ss03", "ss06";
  font-variant-ligatures: normal;
}

Also of note, Inconsolata seems to "just be like that", at least in the programming fonts test drive app it behaves in a similar fashion:

Inconsolata-g is fine though.
For here's the text I used for testing:
█     █    █     ████   ███   ████
██   ██   █ █   █    █   █   █    █
█ █ █ █  █   █  █        █   █
█  █  █  █████  █  ███   █   █
█     █  █   █  █    █   █   █    █
█     █  █   █   ████   ███   ████

It may have been the 3.4.0 update, actually. "Time flies when you're having fonts." Thanks for taking a look! I'll open a new one if I have a breakthrough.


Restrict changes done for MWPW-136057 to commerce modal

Restrict modal-open css class to commerce modal for CC

Resolves: MWPW-136656
Test URLs:
Milo

Before:  https://main--milo--adobecom.hlx.page/drafts/ruchika/discover1?martech=off
After: https://modal--milo--ruchika4.hlx.page/drafts/ruchika/discover1?martech=off

DC

Before:  https://main--dc--adobecom.hlx.page/acrobat/online/convert-pdf?martech=off
After: https://main--dc--adobecom.hlx.page/acrobat/online/convert-pdf?milolibs=modal--milo--ruchika4&martech=off

CC

Before: https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?martech=off#after-effects
After: https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?martech=off&milolibs=modal--milo--ruchika4#after-effects

Tested with the link:
https://main--dc--adobecom.hlx.page/acrobat/online/convert-pdf?milolibs=modal--milo--ruchika4

Adding the slack conversation link about the issue and it's validation done by DC team
https://adobe-mwp.slack.com/archives/C03HAU5SU2E/p1695085781970119
cc @TsayAdobe @JFernandezAdobe

Verified the flow in chrome,edge,firefox and brave browser for the CC url  :
https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?martech=off&milolibs=modal--milo--ruchika4#after-effects

In chrome :
https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?martech=off&milolibs=modal--milo--ruchika4#animate

In edge :
https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?martech=off&milolibs=modal--milo--ruchika4#animate

In firefox :

In brave :

https://modal--milo--ruchika4.hlx.page/drafts/ruchika/discover1?martech=off#animate

The fix is fine and tested and has a production issue tied to it, so I'm fine if you go ahead and merge this (given another approval) - but I'm not sure the fix lives in the correct place.
Might be worth to merge this to fix the issue and then see if the code can be moved

As discussed over slack and here, we'll be addressing the location of the code for commerce modal in a later PR and will be merging this PR for now to address the production issue.


highlight active route

makes it much easier to see which routes are active right now, and also being matched:

Hey @brumm! Thanks for the PR. You need to target this branch/file for it to be included in the demo: https://github.com/nathanhammond/ember-route-alias/blob/master/tests/dummy/app/styles/app.css
Looking forward to a second PR! :smile:

Haha! Thanks, I created a new one for the right file ;)


Adds update deploy permissions for metrics-server-nanny

Signed-off-by: JoshVanL<EMAIL_ADDRESS>What this PR does / why we need it:
Adds update permissions to metrics-server-nanny so that it is able to change the metrics-server deployment.
Adds deployment update permissions to metrics-server-nanny container

/assign @simonswine

/test puppet-tarmak-acceptance-centos v1.14

/test puppet-tarmak-acceptance-centos v1.14

/lgtm

@JoshVanL can you prepare a cherry-pick?


[WIP] migrate unit-test to use Konflux's Trusted-artifact pattern

Konflux's Trusted Artifact pattern - https://konflux-ci.dev/docs/advanced-how-tos/using-trusted-artifacts/#migrate-to-trusted-artifacts

/test openshift-builds-operator-on-pull-request

/test openshift-builds-operator-on-pull-request

/test openshift-builds-operator-on-pull-request

/test openshift-builds-operator-on-pull-request


Use shared CI template

What does this PR do?
Use a shared template for some CI jobs.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #119 (8549820) into develop (17f7503) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #119   +/-   ##
==========================================
  Coverage      87.38%   87.38%
  Complexity       169      169
==========================================
  Files             21       21
  Lines            523      523
  Branches          65       65
==========================================
  Hits             457      457
  Misses            42       42
  Partials          24       24

Impacted Files
Coverage Δ

...com/datadog/gradle/plugin/DdAndroidGradlePlugin.kt
75.00% <ø> (ø)

...in/com/datadog/gradle/plugin/DdCheckSdkDepsTask.kt
84.09% <ø> (ø)

...in/kotlin/com/datadog/gradle/plugin/DdExtension.kt
85.71% <ø> (ø)

...m/datadog/gradle/plugin/DdMappingFileUploadTask.kt
88.97% <ø> (ø)

...in/com/datadog/gradle/plugin/RepositoryDetector.kt
100.00% <ø> (ø)

...og/gradle/plugin/internal/GitRepositoryDetector.kt
97.06% <ø> (ø)

...m/datadog/gradle/plugin/internal/OkHttpUploader.kt
86.30% <ø> (ø)

...n/com/datadog/gradle/plugin/internal/ProjectExt.kt
100.00% <ø> (ø)


Powered Spawner not producing mobs

Issue Description:
The powered spawner receives energy, "works" until the flame reaches 100% and resets back to 0% without producing a mob.
What happens:
The powered spawner receives energy, "works" until the flame reaches 100% and resets back to 0% without producing a mob.
What you expected to happen:
A mob to spawn
Steps to reproduce:

Place Powered Spawner with the desired mob type inside
Power it
Receive no mob spawn
...

Affected Versions (Do not use "latest"):

EnderIO: 1.10.2-<IP_ADDRESS>_beta
EnderCore: 1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: forge-1.10.2-<IP_ADDRESS>5

Your most recent log file where the issue was present:
Logs show nothing when this happens
See video: https://gfycat.com/BlandTemptingAfricanharrierhawk

Still happens in EnderCore-1.20.2-<IP_ADDRESS>-beta and EnderIO-1.10.2-<IP_ADDRESS>_beta as of today. The first couple of times the spawn works. After a few "waves" of mobs, nothing spawns.

Question, Are you using an entity limiter on your server? It sounds like if you are you have reached the max entities and it is stopping the spawn.

No, nothing like that. And night/day doesn't matter btw. It happens randomly after a couple of "waves". (in case you're thinking of mob limit)

Like I said before: The logfile shouldn't matter in this case. There is no output to the console. It just doesn't work. No errors.

The logfile does matter. We've been getting plenty of reports over the last months that different blocks just stop working and we cannot see any reason for it. There certainly isn't any code in our blocks that decides to do something or just fake it. So there must be an outside influence. However, with nobody caring enough to even add a mod list, all we can do is twiddle our thumbs and facepalm hard.
Next build will add notification bubbles to the spawner; like those of the farm. Maybe that'll help you figure out what#s happening.

We have not had any reports on our server (Running DW20 1.10 so no idea of version off the top of my head) But we have not been having any issues. IF any come up, I will make sure to get you all the information. :)

I think I found the problem, it seems to be the dark utilities tether when using an enderman spawner. I ran into it while playing the DW20 1.10 pack, so I did some testing in creative. The wither spawner works, and the enderman only fail after a few waves of spawning into the tether. If you restart without the tether it works until you place the tether down again, then fails a few waves later.


About Flash firmware process

Flash firmware works by sending unlock instructions to the device, then setting up a standard HTTP server, and then sending the server address to the device.The device will go to the HTTP server and request data.
Of course, I use TCP to set up the server in my tool.

When the device receives unlock instructions, it will report to EWeLink server.
EWeLink server will know that you agree with our statement.
The device needs to connect to the network during this process, so a router with Internet access is needed to unlock the device.

Flash firmware works by sending unlock instructions to the device, then setting up a standard HTTP server, and then sending the server address to the device.The device will go to the HTTP server and request data.
Of course, I use TCP to set up the server in my tool.

@ZZLinvec - can you please provide more details when you post instructions.  I can't understand what you mean.
What do you mean by "setting up a standard HTTP server"?
What do you mean by "sending the server address to the device"?
What do you mean by "Of course, I use TCP to set up the server in my tool"?
Can you please provide detailed instructions how to update firmware in DIY mode?

@ZZLinvec

EWeLink server will know that you agree with our statement.

This statement (loosing warranty) is illegal in Europe.
You still need to give FULL WARRANTY  for the hardware part of your product.
Dump question: When soldering the pins and flashing my firmware I don't agree any statement. For itead everything is fine then?

@ZZLinvec ?

@alexbk66 At present, it is explained according to my code, and I suggest you to look at my code

@alexbk66 The Flash firmware code is at the start of the 599 lines of https://github.com/itead/Sonoff_Devices_DIY_Tools/blob/master/code/Dialog_text.py.

I've got no idea what's going on there. That's the first time I see when device manufacturer to the sample code, not documentation...

Ha ha ha ha ha ha, document at https://github.com/itead/Sonoff_Devices_DIY_Tools/blob/master/other/SONOFF DIY MODE Protocol Doc v1.4.md

Man, I keep asking - how do I update firmware to latest version?
For example Shelly for devices Execute<Update>("ota?update=1"); That's all.
So, how do I update firmware on Sonoff devices?
I'm asking because I wrote Sonoff plugin for HomeSeer - so I must implement it for my users!
Otherwise I will recommend all my users to choose Shelly instead, I have a plugin for Shelly as well.

All you need to do is set up a standard HTTP server for the device to fetch data, and the server holds the firmware for the device.Then, send the unlock instruction to the device. After the device is successfully unlocked, send the server information according to the OTA instruction.The device will then automatically get the address of the service.After downloading the firmware, the device will restart automatically and the device will start running your program.

This feature is for users to swipe third-party firmware.But it depends on the device

So there's no way to update Sonoff firmware?


core::fmt::write! causes unexpected behavior

#![no_std]
#![no_main]

use core::fmt::Write;
use core::panic::PanicInfo;
use arduino_hal::delay_ms;
use ufmt::uWrite;

#[arduino_hal::entry]
fn main() -> ! {
    let dp = arduino_hal::Peripherals::take().unwrap();
    let pins = arduino_hal::pins!(dp);

    /*
     * For examples (and inspiration), head to
     *
     *     https://github.com/Rahix/avr-hal/tree/main/examples
     *
     * NOTE: Not all examples were ported to all boards!  There is a good chance though, that code
     * for a different board can be adapted for yours.  The Arduino Uno currently has the most
     * examples available.
     */

    delay_ms(1000);
    panic!("UwU");

    loop {
    }
}

//modified from uno-panic.rs
#[panic_handler]
fn panic(info: &PanicInfo) -> ! {
    avr_hal_generic::avr_device::interrupt::disable();
    let dp = unsafe { arduino_hal::Peripherals::steal() };
    let pins = arduino_hal::pins!(dp);
    let mut serial = arduino_hal::default_serial!(dp, pins, 57600);
    serial.write_str("this should print properly at least but doesn't");
    delay_ms(1000);
    let mut writer = Writer {
        u_write: serial
    };
    writeln!(&mut writer, "{}", info).ok();
    loop {

    }
}

struct Writer<T> {
    u_write: T
}

impl<T: uWrite> Write for Writer<T> {
    fn write_str(&mut self, s: &str) -> core::fmt::Result {
        self.u_write.write_str(s).map_err(|x| match x {
            _ => core::fmt::Error
        })
    }

    fn write_char(&mut self, c: char) -> core::fmt::Result {
        self.u_write.write_char(c).map_err(|x| match x {
            _ => core::fmt::Error
        })
    }
}

this is my code
what i expected:

this screenshot is actually from qemu
(the bug is not reproducible on qemu)
what i got:

it somehow repeats
i tried to modify the code so that it uses heapless
//modified from uno-panic.rs
#[panic_handler]
fn panic(info: &PanicInfo) -> ! {
    avr_hal_generic::avr_device::interrupt::disable();
    let dp = unsafe { arduino_hal::Peripherals::steal() };
    let pins = arduino_hal::pins!(dp);
    let mut serial = arduino_hal::default_serial!(dp, pins, 57600);
    serial.write_str("this should print properly at least but doesn't");
    delay_ms(1000);
    let mut buffer = heapless::String::<256>::new();
    writeln!(&mut buffer, "{}", info).ok();
    serial.write_str(buffer.as_str()).ok();
    loop {

    }
}

different output but still

i later tried to make buffer static but didn't worked
i won't post the result tho
static mut BUFFER: heapless::String<256> = String::new();

//modified from uno-panic.rs
#[panic_handler]
fn panic(info: &PanicInfo) -> ! {
    avr_hal_generic::avr_device::interrupt::disable();
    let dp = unsafe { arduino_hal::Peripherals::steal() };
    let pins = arduino_hal::pins!(dp);
    let mut serial = arduino_hal::default_serial!(dp, pins, 57600);
    serial.write_str("this should print properly at least but doesn't");
    delay_ms(1000);
    unsafe {
        writeln!(&mut BUFFER, "{}", info).ok();
        serial.write_str(BUFFER.as_str()).ok();
    }
    loop {

    }
}

edit: i found that this bug only happens on debug builds

The problem is that core::fmt uses a lot of memory for its formatting.  For any non-trivial cases, it usually surpasses the capabilities of an AVR chip so essentially you just cannot use it at all.  That's why all the examples make use of the ufmt crate instead.
However, there is one place where core::fmt sneaks in and cannot be replaced so easily: panic message formatting.  The only real solution at the moment is to make sure that all panics do not provide a formatted message and instead simply panic with a static message / no message.  For example, you need to replace all Result::unwrap() calls with something like
trait ResultExt {
    type T;
    fn unwrap_avr(self) -> Self::T;
}

impl<T, E> UnwrapExt for Result<T, E> {
    type T = T;

    fn unwrap_avr(self) -> Self::T {
        match self {
            Ok(t) => t,
            Err(_) => panic!(),
        }
    }
}

(Or alternatively follow your suggestion from https://github.com/Rahix/avr-hal/discussions/463 but that means you don't get any custom panic behavior at all)

@Rahix i thought it might be caused by insufficient sram but i didn't thought further because compiling with release profile fixed it
(maybe core::fmt heavily rely on compiler optimizations to reduce ram usage)
about immediate abort, iirc just avoiding print of error message didn't strip out all of the error messages

Relying on optimizations with core::fmt is very hit or miss.  Compiling with LTO manages to eliminate a lot of overhead a surprising number of times, but it does not always work.  And when it doesn't, there is little you can do, unfortunately.

iirc just avoiding print of error message didn't strip out all of the error messages

The things is that you need to ensure all panicking code paths do not lead to a panic with a message.  This is not an easy feat of course.  The ResultExt shown above just takes care of the worst offender - Result::unwrap() which usually pulls in the full core::fmt::Debug implementations of the error types.  But to get rid of all error message strings, you'll of course need to do a lot more.  A list that certainly isn't exhaustive:

Option::unwrap()
Any uses of ops::Index/ops::IndexMut where the bounds check isn't optimized away - replace with fallible .get()/.get_mut calls and then work around the following Option::unwrap() as before.
If overflow checks are enabled, you get panicking code paths from a lot of maths operations.  Either disable overflow checks or use the fallible methods on integer types instead (e.g. u32::checked_add())


Reuse components and retrieve workflow definitions from data index instead of openapi.json in sonataflow-deployment-webapp

@fantonangeli pleasen assign me this issue.

Is this somehow related to

https://github.com/apache/incubator-kie-issues/issues/1275

?

@tiagobento it's also related to that, but it's the continuous of this

https://github.com/apache/incubator-kie-tools/pull/2302

As described here: https://github.com/apache/incubator-kie-tools/pull/2302#issuecomment-2135417701


Topic has outdated screenshot that mentions Visual Studio Team Services

This topic has an outdated screenshot that also references Visual Studio Team Services:

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

ID: 98def25b-48a6-8a17-e321-21b8f55173b5
Version Independent ID: 4d55763c-0dd1-4ae3-1d02-89e21ea3d69e
Content: Use the Azure portal to create a pipeline - Azure Pipelines
Content Source: docs/pipelines/get-started-azure-devops-project.md
Product: devops
GitHub Login: @mlearned
Microsoft Alias: mlearned

these are being updated now, and we should be able to close this soon.

4 months later~


Restivus with todos example app

I am trying to add restivus to the todos example app.
However, it looks as thought restivus is calling iron-router (which I understand is no longer a dependency of restivus.
The following error message is produced:
Error: Route constructor first parameter must be a Router
    at Object.Utils.assert (packages/iron_router/packages/iron_router.js:30:1)
    at new Route (packages/iron_router/packages/iron_router.js:320:1)
    at Restivus.Restivus.addRoute (packages/nimble_restivus/packages/nimble_restivus.js:562:1)
    at Restivus.Restivus._initAuth (packages/nimble_restivus/packages/nimble_restivus.js:922:1)
    at new Restivus (packages/nimble_restivus/packages/nimble_restivus.js:538:1)
    at lib/collections.js:19:1
    at lib/collections.js:23:1
    at C:\Users\aholyoake\WeatherStation\.meteor\local\build\programs\server\boot.js:242:10
    at Function._.each._.forEach (C:\Users\aholyoake\AppData\Local\.meteor\packages\meteor-tool\1.1.10\mt-os.windows.x86_32\
nderscore\underscore.js:79:11)
    at Array.forEach (native)

The code I have used is
Todos = new Mongo.Collection('todos');
if(Meteor.isServer)
{
  Api = new Restivus({useDefaultAuth:true, prettyJson:true});
 Api.addCollection(Todos);
}

(tried in lib/collections.js and server/publish.js separately)
As you can see this is on Windows, which may be a complication!
I'm running the latest meteor and have removed the pinned-version iron-router from the packages file.
Restivus was installed with meteor add nimble:restivus
Any ideas?

Here is the output of meteor list
accounts-password     1.1.4  Password support for accounts
blaze-html-templates  1.0.1  Compile HTML templates into reactive UI with Meteor Blaze
check                 1.1.0  Check whether a value matches a pattern
insecure              1.0.4  (For prototyping only) Allow all database writes from the client
iron:router           0.9.4* Routing specifically designed for Meteor
jquery                1.11.4  Manipulate the DOM using CSS selectors
less                  2.5.1  Leaner CSS language
meteor-base           1.0.1  Packages that every Meteor app needs
mobile-experience     1.0.1  Packages for a great mobile user experience
mongo                 1.1.3  Adaptor for using MongoDB and Minimongo over DDP
nimble:restivus       0.8.4  Create authenticated REST APIs in Meteor 0.9+ via HTTP/HTTPS. Setup CRUD endpoints for Collections.
session               1.1.1  Session variable
standard-minifiers    1.0.2  Standard minifiers used with Meteor apps by default.
tracker               1.0.9  Dependency tracker to allow reactive callbacks

My best guess is that Restivus' Route object is being overwritten by Iron Router. I believe this is the same issue as described in #170. I was waiting on a PR from the user that reported it because I don't really have the time to work on Restivus nowadays. I'll make an exception since this is a really annoying bug (and I already have it fixed locally as you'll read in the other thread I linked).
I'm gonna push up a fix onto a new branch here, but I'll need you to test it locally, if you don't mind, to see if that actually resolves your problem. I don't want to have to publish a new version just to find out this didn't fix your issue. Gimme a sec.

Alright. I just submitted PR #182 with a potential fix. Can you test it out locally for me? It's on the route-namespace branch.

Cheers for the speedy response - I'm getting the same error as the travis CI build when installed locally
error: File not found: lib/package-scope-variables.js

Whoops. Left an old untracked file in there. Sorry about that. I had the file locally so everything ran okay for me. This is why we use CI and check with others :) Let me fix that up and push again. One sec.

Alright. Try it again. Just grab the latest on that branch.

It's gone one step along the line it seems - I've got something very close to the original error message.
However, it's bombing out during the addCollection method, rather than during the call to new Restivus
Just to clarify what I've done - I have created a packages directory in my project, downloaded the zip for the route-namespace branch (as I have a corporate firewall to deal with), run meteor remove nimble:restivus and then immediately afterwards meteor add nimble:restivus.
Here's the current message
C:\Users\aholyoake\AppData\Local\.meteor\packages\meteor-tool\1.1.10\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\fibers\future.js:245
						throw(ex);
						      ^
Error: Route constructor first parameter must be a Router
    at Object.Utils.assert (packages/iron_router/packages/iron_router.js:30:1)
    at new Route (packages/iron_router/packages/iron_router.js:320:1)
    at Restivus.Restivus.addRoute (packages/nimble_restivus/packages/nimble_restivus.js:562:1)
    at Restivus.Restivus.addCollection (packages/nimble_restivus/packages/nimble_restivus.js:616:1)
    at lib/collections.js:9:1
    at lib/collections.js:22:1
    at C:\Users\aholyoake\WeatherStation\.meteor\local\build\programs\server\boot.js:242:10
    at Array.forEach (native)
    at Function._.each._.forEach (C:\Users\aholyoake\AppData\Local\.meteor\packages\meteor-tool\1.1.10\mt-os.windows.x86_32\dev_bundle\server-lib\node_modules\underscore\underscore.js:79:11)
    at C:\Users\aholyoake\WeatherStation\.meteor\local\build\programs\server\boot.js:137:5
Exited with code: 8
Your application is crashing. Waiting for file change.

Hi Kahmali,
I've done a meteor reset and that cleared my cache out. I'm getting data out of the endpoints now, so it looks good to me!
Cheers for the help!

Phew! I was about to give up on you :P Iron Router is such a headache (just search "iron router" in the Restivus issues and you'll see what I mean). I strongly recommend that you check out some tutorials that use Flow Router. I think most folks would agree that it's the best router for Meteor. We use it extensively in NimbleNotes and it's never been an obstacle to development. I wish I could say the same for Iron Router.
I'll go ahead and bump the version and publish that. Thanks for reporting the issue and testing that out for me! Greatly appreciated! I'll let you know once it's published.

I just published v0.8.5 with this fix. Thanks so much for your help!


Replace beta with preview in callout

Closes https://github.com/newrelic/newrelic-quickstarts/issues/414

@nr-kkenney I thought about it 😂 felt like we'll be dumping the whole component soon anyway 🤷♀

@LizBaker I think keeping it as a betabanner is more reflective of what it does, so the name is cool. Perhaps we can re-use this in the future for other beta callouts. so I'll merge this in now
If this component only on the devsite? or is it part of the global theme? I could see it being useful on the docs site eventually. cc @roadlittledawn


DDL table structure query syntax is wrong...

System information:

Windows 10
DBeaver version 7.0.5
no Additional extensions

Connection specification:

Postgresql 9.4
Postgresql-42.2.5.jar

I am using Parent child concept in my project(Inheritance)... i checked the child table DDL statement it is showing parent table name two time.
for reference please check the image .

hi @ramrasi ,
unfortunately, cannot reproduce it. Is it happens for all of the tables?

@uslss
Thanks for your reply...
i saw in all master-slave tables. Please refer following images.


Iterate over all row to get specific indexes

You have a tensor of indexes A of shape (M, 1).
a tensor containing your values of shape (M, N).
you want to iterate over all rows to get the element at the specified column and get a tensor of shape (M, 1)
@agibsonccc proposed this but it doesn't work actually.
Nd4j.zeros(10, 20).get(NDArrayIndex.all(),new SpecifiedIndex(Nd4j.zeros(10).data().asInt()));

You can do this. It works but maybe later we can try to think of a more 'elegant' solution.
All I am doing is flattening the array and indexing out linearly.
m is of shape (M,1)
mn is of shape (M,N)
If you want to grab from columns instead of rows, switch the size(0/1) to size(1/0)
        INDArray these_indexes = Nd4j.linspace(0,m.size(0)-1,m.size(0)).mul(mn.size(1)).add(m.transpose());
        INDArray result = Nd4j.toFlattened(mn).get(new SpecifiedIndex(these_indexes.data().asInt())).transpose();


Long term maintainability

Hi,
In terms of updating old components generated via CLI, would this have to be done with git by forking / cloning this repo and merging upstream changes here or?
Is there any recommendation for the best way to do this?
Ideal outcome: being able merge the latest changes from shadcn/ui to acme/ui somehow
Thanks :)

@wondering2151 I'm looking into a solution for this. Either via documentation or the CLI.
Open to suggestions and ideas from the community.

I think would be cool to give each component its own version. It could be stored in a comment in the file, or in components.json. When a change is made to a component upstream, its version will be bumped, and a command in CLI could check for new versions. I feel like this approach would be better than just diffing the code, because if you have custom changes in the components, it will cause troubles.

I ran into a similar issue and ended up chaining diff and add commands for convenience. #941
Initially, I was maintaining an array of installed components (enhancing Add) and iterating through it, but using the diff command provided me with more detailed information.

I feel like this approach would be better than just diffing the code, because if you have custom changes in the components, it will cause troubles.

Agree with @MaximMaximS here. I ran into some issues with the experimental diff command due to stylistic differences. So diffing is kind of noisy for me and not too useful (it's saying all the components have updates but it's all just stylistic differences).

@shadcn sorry for ping, but what do you think about this?

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Old client will generate recipe parsing errors on server

The recipe structure has changed, and so if you use an old client prior to the recent change, you will get an error on the server side:
2021-08-04 20:38:55 [INFO] Turbinia version: 20210611.dev20+gdd6b969
2021-08-04 20:38:57 [INFO] Disabling non-allowlisted jobs configured to be disabled in the config file: volatilityjob, binaryextractorjob, bulkextractorjob, photorecjob
2021-08-04 20:38:57 [INFO] Starting Prometheus endpoint.
2021-08-04 20:38:57 [INFO] Running Turbinia Server.
2021-08-04 20:38:57 [INFO] Starting Task Manager run loop
2021-08-04 20:49:57 [INFO] Processing PubSub message<PHONE_NUMBER>225310
2021-08-04 20:49:57 [WARNING] No globals recipe specified, all recipes should include a globals entry, the default values will be used
2021-08-04 20:49:57 [ERROR] Recipe item jobs_allowlist has no "task" key. All recipe items must have a "task" key indicating the TurbiniaTask to which it relates.
2021-08-04 20:49:57 [INFO] Received evidence [GoogleCloudDisk:turbinia-test-persistent-disk:None] from PubSub message.
2021-08-04 20:49:57 [INFO] Adding new evidence: GoogleCloudDisk:turbinia-test-persistent-disk:None
2021-08-04 20:49:57 [INFO] Writing new task AbortTask into Datastore
2021-08-04 20:49:57 [INFO] Recipe Invalid

Not sure if need/want to fix this or just require that the clients be updated.  Adding this bug in case others run into this and have opinions.

Closing this since the new release has been out for a while now.


Near sputnik dao.md

I suggest to complete your translation before create a PR, And if you want to show others you have devoted into some topic, you can create a branch.

Oh yes, I see what you mean. thanks！

翻译质量评价： 较差


evaluate should allow null for resolver and result

This should fix https://github.com/Microsoft/TypeScript/issues/11981
I've also changed the return type of XPathExpression.evaluate to XPathResult based on https://developer.mozilla.org/en-US/docs/Web/API/XPathExpression

Hi @goenning, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

👍 Thanks @goenning !


Full screen overlap issue on multi-display setup

VSCode Version: 1.30.2
OS Version: Windows_NT x64 10.0.16299

Steps to Reproduce:

Have 2 monitors (1920x1200 twice)
Open Visual Studio Code (v1.30.2, all extensions are disabled)
Go to the full-screen mode (F11)
Return back by pressing F11 again
Maximize the window
Make the window active

Expected result
Visual Studio Code window maximizes and fits one monitor resolution
Actual result
Visual Studio Code window maximizes and overlaps second display
(see attached photo)

Confirmed still active.
VSCode Version: 1.37.1
OS Version: Windows_NT x64 10.0.18362
Steps to Reproduce:
Have 2 monitors (1920x1200 twice)
Open Visual Studio Code
Maximise window
Note that VSCode window is fully filling and contained in one monitor
Go to the full-screen mode (F11)
Return back by pressing F11 again
Expected result
VSCode window should be fully filling and contained in one monitor
Actual result
VSCode window spills into 2nd monitor to the right (also spills to a monitor to the left and above)

@deepak1556 let's check this out in exploration builds

@sbatten this is fixed with latest exploration.

@deepak1556 assigning to you as you track our electron upgrades


Delete hangs if a non existent node name is provided in delete modal

Step to repro, master-head, commit

From Explorer, select a node to delete
Enter a  non-existent node name in the delete modal.
Note that UI shows that it couldn't find a match but the delete button goes to "deleting mode" and get stuck in "deleting" for forever, and doesn't come out from deleting. The only way to come out if the users were specifically selected the "cancel" button

Verified on 2.5-head, 6c03627

From Explorer, select a node to delete
Enter a non-existent node name in the delete modal.
With the fix, now the delete doesn't hang and shows "error" in delete buttons
Users are now forced to cancel that operation to come from the error condition

Verified on 2.5-head, 6c03627

From Explorer, select a node to delete
Enter a non-existent node name in the delete modal.
With the fix, now the delete doesn't hang and shows "error" in delete buttons
Users are now forced to cancel that operation to come from the error condition


nedb no longer maintained

nedb is no longer maintained, consider new data storage package.
priority: low
will continue current state for now but will privately test new dependencies to replace nedb.

Merged with #12


Thaumcraft Research Parenting Crash

When trying to add Prereqs to my custom Research, the client crashes when opening the edited Thaumonomicon tab.
Everything except the parenting works perfectly when removing the last two "addPrereq"-lines.
The Research appears at the given tab and coordinates in the Thaumonomicon and it is also round and spikey (as some other researches in the BASICS tab).
(EDIT: I should mention this happens in SMP. I tried in Single Player, parenting worked. I changed something in the script. It breaks down again. I am tracing back my steps to find out what I did wrong.)

But the strange thing is that there appears a parsing error when just commenting them out.
MT Code:
// ### Knowledge Fragment Doubling ###//

val bala    = <Thaumcraft:ItemShard:6>;
val knowl   = <Thaumcraft:ItemResource:9>;
val knowl2  = <Quadrum:fragmentInfused>;
val knowl3  = <Quadrum:fragmentVibrate>;
val end      = <minecraft:ender_pearl>;

mods.thaumcraft.Aspects.set(knowl2, "cognitio 8, alienis 4, desidia 1");

mods.thaumcraft.Research.addResearch("FRAGMENTDOUBLE", "BASICS", "sensus 1, vitreus 1, alienis 1, desidia 1, cognitio 1, terra 1", 5, -3, 12, knowl3);
game.setLocalization("tc.research_name.FRAGMENTDOUBLE", "Knowledge Expansion");
game.setLocalization("tc.research_text.FRAGMENTDOUBLE", "[GF] Why not both?");
mods.thaumcraft.Research.setConcealed("FRAGMENTDOUBLE", true);
mods.thaumcraft.Research.setSpikey("FRAGMENTDOUBLE", true);
mods.thaumcraft.Research.setRound("FRAGMENTDOUBLE", true);

mods.thaumcraft.Crucible.addRecipe("FRAGMENTDOUBLE", knowl2, knowl, "tenebrae 10, vacuos 10, desidia 10");
mods.thaumcraft.Infusion.addRecipe("FRAGMENTDOUBLE", knowl2, [end,end,end,end,end], "alienis 20, motus 20, potentia 20", knowl3, 5);
mods.thaumcraft.Arcane.addShaped("FRAGMENTDOUBLE", knowl*2, "aqua 20, terra 20, perditio 20, aer 20, ordo 20, ignis 20", [[bala, bala, bala],[bala, knowl3, bala],[bala, bala, bala]]);

mods.thaumcraft.Research.addPage("FRAGMENTDOUBLE","gf.research_page.FRAGMENTDOUBLE");
game.setLocalization("gf.research_page.FRAGMENTDOUBLE", "You got very lazy about finding these Knowledge Fragments lately. But you found a workaround...<LINE>You discovered that simple copying doesn't work out pretty well since you need new parts of knowledge to complete your all new research. First you connect a Fragment to another one in the void. After making them vibrate in synch, you must apply a heavy magical pulling force between them while at the same time maintaining a stabilization of your half of the connection to extract the second Fragment from the vast nothingness.");

mods.thaumcraft.Research.addCruciblePage("FRAGMENTDOUBLE", knowl2);
mods.thaumcraft.Research.addInfusionPage("FRAGMENTDOUBLE", knowl3);
mods.thaumcraft.Research.addArcanePage("FRAGMENTDOUBLE", knowl);
mods.thaumcraft.Warp.addToResearch("FRAGMENTDOUBLE", 1);

mods.thaumcraft.Research.addPrereq("FRAGMENTDOUBLE","INFUSION",true);
mods.thaumcraft.Research.addPrereq("FRAGMENTDOUBLE","KNOWFRAG",false);

Client Crash Report

That texture is sexy! You made them?!

I did :)

By the way.. as you are into Modtweaker Thaumcraft Research Modification... Hidden and Lost modifiers got added! That means you can link a research to an item/block/entity or make it lost for it to be accessed by knowledge fragment researches :)

UPDATE: I have still no idea what's the problem here...

Could you track down which one of the preRequ is causing it?

Any. As far as I tried, every single line containing a preReq causes a crash when taking a look on the corresponding thaumonomicon page... It only works without every single one of these lines.

I just figured out that the Thaumonomicon is handled client-side. Since my scripts are server-sided the client doesn't know how to draw the prereq lines and crashes. Will tell more when I find a workaround.


Firebase crashlytics

I followed the basic steps to integrate the firebase crashlytics,&  i uploaded the run scripts steps.
here are my run scripts
GOOGLE_APP_ID = MyApp ID
"${PODS_ROOT}"/FirebaseCrash/upload-sym-util.bash "${PROJECT_DIR}/My File Key path"
i Uploaded the Debug Info Format as DWARF with dsym file in Built settings & i added the Google info plist too in my project
& i  Uploaded the missing required dSYMs file in firebase dashboard too, even though its asking again and again missing required dSYMs

Kindly provide the suggestion to solve the issue

i have a same problem

If you're using bitcode, Apple will occasionally rebuild your app from bitcode in the App Store. You'll have to upload new dSYMs each time this happens, which you can download from the Apple developer center.

Yes i downloaded the dSYMs file from the Apple developer account, still i have the same issue

I think this is what you're trying to say, but to make sure, have you also uploaded those new dSYMs to Crashlytics?


Links to internal confluence architecture information

In 'Interpret MaC outputs', search for "Most of the labels are outlined in Designing Service Alerting”. There are couple of sections that have placeholder NOTE text. This information is currently in internal documentation, it will need to be decided where this info is kept and how it's best presented in the external docs.

Pending work on SEGAS Public Docs-as-Code site.


UI freezes after capturing photo on iOS

The UI gets stuck after capturing a photo on iOS 14.7.1 (not sure about other versions) and no errors are reported, but if I launch the app again from Xcode when I already have it installed it starts working. However if I uninstall it then install it again the same issue occurs.
I paused the debugger while it was frozen and the debugger gave me the following location in the code. It seems to be related to the image context or the drawInRect function:
UIGraphicsBeginImageContext(CGSizeMake(enhancedImage.extent.size.height, enhancedImage.extent.size.width));
[[UIImage imageWithCIImage:enhancedImage scale:1.0 orientation:UIImageOrientationRight] drawInRect:CGRectMake(0,0, enhancedImage.extent.size.height, enhancedImage.extent.size.width)];

I'd be happy to help but I've never done any native iOS development so it'll take a while for me so i'd be very grateful if you could look into this.

Hello, after iOS 14 release, I didn't have the chance to do any testing regarding this package because I don't have any access to a mac device although I'm sure there is couple of bugs that need fixing.
Did you handle camera permissions before building the scanner widget ? Because once the scanner widget gets rendered for the first time, if there isn't any camera permissions it doesn't work even after the user accepts the camera permission, it needs disposing then a new rebuild or in your case: launch the app again from Xcode.
So you should render a temporary widget until the user accepts or denies the permission.

I see. Yes, I made sure that I did handle them. This is a better rewrite of the original react native package: https://github.com/HarvestProfit/react-native-rectangle-scanner
The maintainer claims that the original package has old iOS code with lots of memory leaks.
I'll probably look into creating a Flutter wrapper for it when I have time.

Hello @eliasteeny is there any update regarding this topic?
My application is pretty dependant on this and am having trouble for the last month regarding my iOS app.

Hello @amirhasanovic, unfortunately there isn’t any new update. I still don’t have any access to a macintosh machine to test and fix this bug.
Can you run your app through Xcode and provide the exception and stacktrace being thrown so I can try to provide a hotfix for the time being.

@eliasteeny thanks for your response. I have sent you an email via the address that I found on your website. Hope to hear more from you.

Hi, Did you ever find a work around for this on iOS?

I suspect this bug is related to CIContext, code will work fine if no work of CIImage/CIFilter is involved (i.e. not rendering the final output), I fixed this by moving the image rectifying code into dart using dart_image library, slow as hell but at least working.


Fix error "findDOMNode is deprecated..."

See https://github.com/reactjs/react-transition-group/issues/668

Hi @abmenzel, I have fixed the issue and created a PR for that. Let me know if any changes are required. Thanks.

Thank you for your contribution @amrendranath nice work!


Feature/auth update

Combined #931 and #940

LGTM!

TODO:

Some refactoring
Increase coverage
Unify the way how to require_api_url etc in all of the commands not changed in this PR
Add optional timeout for any_key_to_continue method
Add more environment_variable: definitions to options in all commands
..to be continued

Found couple of minor issues. After removing migration run from spec helper we are ready to hit the green button


Fix compatibility issue with rack 2.3

Rack gem changed inplementation of Rack::Builder.load_file method in version 2.3 . It used to return array where first item was the rack app object.
in new implementation the method returns directly the rack app. Here is a change: rack/rack@b95cb72#diff-79892c6315a7f9c02489803e95ce828a8a32759b9f8f196aa98e6826eb562e55L107
to keep compatibility with older versions of rack we wrap the return value to Array()

HI @sickill ,
is there any objection about this PR? Can we merge?