Error in dependencies

After downloading the sample Workflow implementation and running it, I get an error about the dependencies.
Please see attached file with details.

Thaks @tonidorta , we detect a dependency conflict in the SDK.
We are fixing it and we'll make a new release ASAP.


mocha > 2.4.5 fails with reporter xunit

Hello,
with the new release of mocha greater than 2.4.5 (current 2.5.2) my mocha task with reporter XUnit for jenkins doesn't work any more.
It fails cause of https://github.com/mochajs/mocha/commit/d4f7905dbb9d4cc65cca02373b99e5549fa7ce2a
The Mocha XUnit Reporter https://github.com/mochajs/mocha/blob/master/lib/reporters/xunit.js was changed to get pending tests with test.isPending() instead of test.pending in the past.
For me it is not clear how I can manipulate the test object which is the parameter of
runner.on('pass', function(test) {})
Maybe someone can help me fixing this issue.

Furthermore, it's possible to run the exact same test with "grunt-mocha-test" or "grunt-simple-mocha" and have the XUnit reporter work fine, but with "grunt-mocha" the isPending function is mysteriously missing when the test is sent to the XUnit reporter.

I’m seeing the same issue with XUnit failing due to isPending(). I originally reported it with mochajs (see referenced issue above), but it appears to originate with grunt-mocha.
I have some sample code that recreates this issue.
Mocha XUnit Reporter Sample
https://github.com/matthewriley/mocha-xunit-reporter-sample

I found the root suit (for example, this) is not instanceof "mocha/lib/suite".
so isPending is not on its prototype chain. it this the root cause?

That function is not inheriting Suite because it is not a suite constructor; in fact, it's even creating the root suite: it takes an existing suite as a parameter.
(You can also add console.log calls to the XUnit reporter to see that the tests are missing the isPending function.)
Additionally, the problem does not occur with Mocha itself and not with some other integrations either, so what we really need to determine is what Grunt-Mocha is doing differently.

Sorry I didn't state it clear, I mean that the parameter itself is the root suite, not the function.

Is Grunt-Mocha creating a root suite of its own without it inheriting Mocha's Suite? As far as I can tell, Mocha is creating the root suite here: https://github.com/mochajs/mocha/blob/master/lib/mocha.js#L85 using the Suite object from here: https://github.com/mochajs/mocha/blob/master/lib/mocha.js#L41

Hi guys, is there any workarounds for this bug? I would like to use grunt-mocha on my client side and generate xUnit reports, but always got: Fatal error: test.isPending is not a function
I really appreciate any help you can provide.

@omatic-hacker Here's a patch I wrote for my project.

@cades Thanks, I will try your patch!

@omatic-hacker I fixed this problem in my fork of grunt-mocha-blanket(https://github.com/eugenet8k/grunt-mocha-blanket) which could be used instead of grunt-mocha. It's basically the same thing but with built-in blanketjs coverage support.


fix(custom-element): reflect prop default value

close #9006

I’m pretty sure if you’ll set an attribute (setProp-> reflect attr) inside of the constructor, real browser will throw an exception :(

/ecosystem-ci run

I think we may need to make a distinction between properties and attributes here.
The original issue was specifically about properties, not attributes. I think having default prop values reflected in the properties of the element make sense, but I'm not so sure about the attributes.
I've had a look through some resources about custom elements, but I can't find anything that describes what's expected for property defaults. There's a recommendation that calls to setAttribute should also set the property, and vice versa, but that isn't the scenario we're in here. A default is an implicit value, it isn't being set by the consumer of the element.
For native elements, a default value for a property doesn't lead to an attribute being set. For example, consider a <textarea>. The default value for rows is 2, which is reflected in the property, but there isn't a corresponding attribute for that default. Here's a Playground to illustrate that: example. I would expect the same from a custom element.
I suspect exposing the defaults as attributes could cause problems for SSR too. The hydration process won't be expecting those extra attributes to be present. I haven't actually verified whether this is problem in practice, as there's a lot of stuff that needs setting up to test it, but it isn't just Vue's hydration process that might be impacted, it could also impact hydration in other frameworks that are trying to use a Vue-based custom element.

Superseded by 63689ed
@skirtles-code that seems to be a separate topic from this PR - the current behavior of Vue custom elements always reflect declared props as attribute if it has a value, so this PR (and by extension 63689ed) just aligns default values with that behavior.


chore: adds tessen configuration

This PR adds the configuration for tessen and remove the individual GA tracking code so we can leverage all the goodness of Tessen and Segment. 🎸

if you're removing ga you can also remove the onRouteUpdate function from gatsby-browser.js to keep things clean

@LizBaker Ok, I see on the Docs Site we handle this a little different for OnRouteUpdate Should I take that approach on the developer site in stead of removing it all together?
https://github.com/newrelic/docs-website/blob/develop/gatsby/on-route-update.js
const onRouteUpdate = ({ location, prevLocation }) => {
  if (
    window.newrelic &&
    location.hash &&
    location.pathname !== prevLocation?.pathname
  ) {
    window.newrelic.addPageAction('hash_request', { hash: location.hash });
  }
};

@jpvajda I don't think so. I'm not sure why we're tracking that but since it doesn't have to do with tessen or GA (looks like it's the browser agent) I'd ignore it for this PR

Once I test and confirm this is working with the OSS SIte, I'll cut a release for the developer site.


If the upgrade fails mid installation, i cannot restart it to have it finish what failed before

I tried to update my flutter installation, i run the flutter upgrade tool, it executed till half because of a os error, and then it says that flutter has been upgraded regardless of the previous upgrade really finished and i have no way of "reinstall it".
Steps to Reproduce
PS D:\xx> flutter upgrade

Upgrading Flutter to 3.7.3 from 3.3.10 in C:\flutter...
Checking Dart SDK version...
Downloading Dart SDK from Flutter engine 248290d6d50a1bf1a6a246544347c03acda867a5...
Expanding downloaded archive...
Building flutter tool...
Running pub upgrade...
Upgrading engine...
Downloading Material fonts...                                        2ms
Downloading Material fonts...                                        0ms
Failed to download https://storage.googleapis.com/flutter_infra_release/flutter/fonts/3012db47f3130e62f7cc0beabff968a33cbec8d8/fonts.zip. Ensure you have network connectivity and then try again.
FileSystemException: Cannot delete file, path = 'C:\flutter\bin\cache\downloads\storage.googleapis.com\flutter_infra_release\flutter\fonts\3012db47f3130e62f7cc0beabff968a33cbec8d8\fonts.zip' (OS Error: Impossibile accedere al file. Il file è utilizzato da un altro p
, errno = 32)

PS D:\xx> flutter upgrade
Flutter is already up to date on channel stable
Flutter 3.7.3 • channel stable • git@github.com:flutter/flutter.git
Framework • revision<PHONE_NUMBER> (7 days ago) • 2023-02-08 15:46:04 -0800
Engine • revision 248290d6d5
Tools • Dart 2.19.2 • DevTools 2.20.1

Logs

Logs

[√] Flutter (Channel stable, 3.7.3, on Microsoft Windows [Versione 10.0.19045.2546], locale it-IT)
    • Flutter version 3.7.3 on channel stable at C:\flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (7 days ago), 2023-02-08 15:46:04 -0800
    • Engine revision 248290d6d5
    • Dart version 2.19.2
    • DevTools version 2.20.1

[X] Windows Version (Unable to confirm if installed Windows version is 10 or greater)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at C:\Users\xx\AppData\Local\Android\Sdk
    • Platform android-TiramisuPrivacySandbox, build-tools 33.0.1
    • ANDROID_HOME = C:\Users\xx\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Eclipse Adoptium\jdk-<IP_ADDRESS>-hotspot\bin\java
    • Java version OpenJDK Runtime Environment Temurin-17.0.3+7 (build 17.0.3+7)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Build Tools 2019 16.11.9)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools
    • Visual Studio Build Tools 2019 version 16.11.32106.194
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (version 2022.1)
    • Android Studio at C:\Users\xx\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\221.6008.13.2211.9514443
    • Flutter plugin version 72.1.2
    • Dart plugin version 221.6096
    X Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[√] IntelliJ IDEA Ultimate Edition (version 2022.3)
    • IntelliJ at C:\Users\xx\AppData\Local\JetBrains\Toolbox\apps\IDEA-U\ch-0\223.8214.52
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin version 223.8214.16

[√] VS Code (version 1.73.1)
    • VS Code at C:\Users\xx\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.54.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Versione 10.0.19045.2546]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 109.0.5414.122
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 110.0.1587.41

[√] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

@nicolasvac
Although the failure you observed was due to connectivity or network probably, I don't think there's a way in the tool to restart the upgrade.
Upon running flutter upgrade though it does say that the sdk has been upgraded and flutter doctor shows correctly indicating there may not be any issue. If so, flutter doctor would have highlighted that.

Hello @darshankawar , if i recall correctly, after the fonts the tool should be downloading other things like the compilers for the various platforms.
I dont see them downloaded in the logs, so I dont know if the tool has fully upgraded or only partially.

I see, thanks for the feedback. I'll keep this issue open for team's thoughts on expected behavior.

/cc @christopherfujino

@nicolasvac you can do flutter precache --all-platforms to have it finish what it would have done.
Maybe the action item here is to catch this crash and then tell the user they can re-try flutter precache --all-platforms. I'm not sure the automating the retry is a good idea, in case the precaching fails again, and the user is stuck in an infinite loop.

same error, stuck in the same step. pub get wont complete, so flutter is basically broken after this.
I did a downgrade and tried the upgrade again, and stuck at the same step once more.

@nicolasvac you can do flutter precache --all-platforms to have it finish what it would have done.
Maybe the action item here is to catch this crash and then tell the user they can re-try flutter precache --all-platforms. I'm not sure the automating the retry is a good idea, in case the precaching fails again, and the user is stuck in an infinite loop.

I tried this code and the problem is solved, thanks.


how to run on multi-gpu with device_map='auto'

when I use 34B llm model, single-gpu will report OOM.
so I set device_map='auto', but It seems cant use torchrun, It takes too much time to inference.
how to solve this problem?

Close the issue due to no response in weeks, plz reopen if needed

I use yi-34B LLM.
It seems visual encoder cost too much time.（32 frames per video）


Make use of the Slack signing secret in the sample bots

Slack currently complains that this is missing when initiating connections

Thanks @pmundt


Create introduction.md

is this standalone-Introduction guide going to work...?

create DataBC standalone-intro guide (will it work?)


update creating class component with es6 class in rules/prop-types

I think nowadays people don't use createReactClass as much and in official react document's example use es6 class for create class component. so I think it's more conformable for people that are newcomer in react.

The current docs contain examples of all kinds of components; so i think this PR should be closed.


The first line is not working.

When I try the code
const skyward=require('skyward-rest')
, it says this:

Do I need to do some git install or something to get this working and installed in the node modules?
Sorry, I am new to React-Native and I am still trying to figure things out.

Hey! Did you run yarn install in the root directory where package.json is located? That installs all Node.js dependencies.

Yes, I just did. That error disappeared. There is a new one now, and I dont know why it is there:

To clarify what I've done, this is my current code:

I have also run npm install skyward-rest. It cleared up the first issue. However this next issue appeared and I'm not sure what to do

To clarify what I've done, here is my code:

I ran npm install skyward-rest to get rid of the first error, but I don't know what to do about the second error.

Also, I believe that my URL is currently redirecting. What should I do to prevent that?

It still isn't working. Here is the terminal error:

There is also a pop up on const skyward=require('skyward-rest').

I have also tried npm install (I am using npm instead of yarn, so instead of doing yarn install, I am doing npm install). I have done npm install cheerio and npm install skyward-rest. This is still the error, and I can't seem to find a solution. Thanks a lot for helping me by the way, @shrey150 .

Actually, never mind, I got it working. However, a new problem has appeared. How do I get the scrapeHistory to display the course 5-digit number? For the scrapeGradebook, it is required, so I assume that it is stored somewhere, but it just isn't being displayed in the scrapeHistory function. How can I access this?


Update README.md

Thanks for maintaining this repo and we really appreciate that you include our work really quickly:)
Our research primarily focuses on the multimodal dataset distillation, so we provided a minor suggestion on the category. Please reach out if you have any question!

Thanks for your contribution, I have approved and merged your request.
It's great to see the emergence of multimodal dataset distillation. Nice work!


Fix: claim button and menu diviver theme colors

Description

This PR reports  bugs of the the beta testing
The Claim button and loading spinner on the dark theme do not match the appearance of other buttons and are hard to read.
The menu divider should be lighter on both light and dark themes.

Implementation

Updated theme colors for better readability and consistency.

Screenshots
Claim dark issue

menu divider too dark

menu fix

claim button text on dark theme is white now, didn't have tips to show it but is fixed now! :D

JM! @maciejka @MSghais This one is ready for review!


Anne Happy! not working.

Only shows episode 1 and there is no text when I press play.

Anne Happy is one of the remaining shows I need to process.  This anime is non-trivial because I don't have vods for episodes 6 onward, and the mid-episode breaks don't have a consistent timing.

Anne Happy! is in the latest update.


”Înfrățirea”, ”Piscu Lung”, ”Poienile” are listed in the drop-down menu

Description:
When user writes ”Înfrățirea”, ”Piscu Lung”, ”Poienile” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Înfrățirea"

Expected results
The location appears in the drop-down menu as "Sat Înfrățirea, DOLJ (Bulzești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Piscu Lung"

Expected results
The location appears in the drop-down menu as "Sat Piscu Lung, DOLJ (Bulzești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Poienile"

Expected results
The location appears in the drop-down menu as "Sat Poienile, DOLJ (Bulzești)".


unhandled exception

maybe something went wrong at that time with animepahe.
should be fixed by itself after sometime.


fly should warn/tell the user that they can use -x when it will transfer large files

large git repos take a long time (and have other issues - see https://github.com/concourse/fly/issues/31) so it would be helpful to remind the user that they can use -x to avoid this.
This could also take the form of a tip: TIP: use fly execute -x to avoid copying large git repos

Moved to concourse/concourse#1082

Moved to concourse/concourse#1084


Only enable DDL forwarding if web_access role exists

Problem
Summary of changes
Checklist before requesting a review

[X] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

@vadim2404 approved, but PR in current state certainly breaks some flows, i.e. users won't be able to create extensions anymore.
Also we should document and add release notes, that users won't see other roles tables / rows by default since this is released.

users won't be able to create extensions anymore.
Users will still be able to perform these grants themselves right?

@vadim2404 approved, but PR in current state certainly breaks some flows, i.e. users won't be able to create extensions anymore.
Also we should document and add release notes, that users won't see other roles tables / rows by default since this is released.

My idea was to test it on staging first. I don’t want to break the existing habits too much, btw

FWIW, here is the model we used for current design
https://gist.github.com/lubennikovaav/ae1946e7321c37653e070b1b34fbbb16
It uses superuser to GRANT access to all object to the new user.
We may test, If we can share all needed privileges using only db-owner role.

So in order to retain the old behavior, I've created a role called neon_superuser. Roles and Databases automatically created through the UI will have all privileges on all databases, while roles created manually will be more similar to vanilla postgres and require manual grants. @kelvich and I discussed and decided this was probably the best course of action: advanced users can setup roles how they want, and simple users can stick with sane defaults.

This relies on the fact that the code path inside of compute tools that creates roles can only be executed if the role was created with UI.

Here's a little table describing the 3 possible states of roles and databases, and how they will be upgraded to the neon_superuser scheme. It turns out to be pretty simple - essentially create_neon_superuser handles migrating all projects to the new scheme.

web_access
Use KMS, but no neon_superuser (current state of affairs)
new project

Roles
All roles (including web_access) will be granted neon_superuser. They will have neon_superuser, pg_read_all_data, and pg_write_all_data. The pg_read_all_data and pg_write_all_data will be redundant with neon_superuser, but it's probably fine to not revoke them explicitly.
Same as previous
All new roles created through UI will be granted neon_superuser

Databases
All privileges on all databases granted to neon_superuser on creation of neon_superuser
Same as previous
When database is created, all privileges on it will be granted to neon_superuser


feat: add parsing support for JSON items

Add FromStr and parse associated functions to json-abi items and params.
Supersedes #310
Closes #291

Small thing about this is that all parse functions use the same parser, so they will accept and ignore stuff like "indexed" and state mutability instead of returning an error, which I don't think is that bad


Set AllReady.ScenarioTest culture to US to match web app.

Fix #2033

Brilliant, thanks @jimparsley !


Create callbacks for Solver

Create a base class for callback e.g Callback and rewrite learning rate halving and early stopping using it. Rewrite Solver accordingly.
This would give the user more freedom to define it's own callbacks.

Solved by #19


Consolidated plugin path

Currently, the API exposes user_plugin_path and bundled_plugin_path. On a typical OSX installation, these point to ~/Library/Application Support/Binary Ninja/plugins and /Applications/Binary Ninja.app/Contents/MacOS/plugins respectively. Plugins that were installed by the user will end up in the user_plugin_path directory, whereas plugins that were installed by the plugin manager will end up in the bundled_plugin_path directory. This causes a problem because plugins have no consistent way of knowing in which of these directories they were installed. If a plugin needs to access files relative to itself, the only currently reliable method is to calculate relative paths from the __file__ attribute Python provides.
It would be useful if the API exposed a plugin path that would always correspond to the path under which the current plugin had been installed, so that authors would not need to fall back on __file__

Thats a good suggestion,  I'm going to change the issue name to be reflective of the API that should exist. BNGetCurrentPluginPath()

Because we have to support both C, Python, and other plugins, this is not something we can safely implement in the core. In short -- your proposed workaround is actually the only right solution.


Add BIG-IP version and deployment matrix to nightly tests #375

@pjbreaux @dflanigan
Issues
Fixes #375
Problem
The nightly test only included one combination of BIG-IP:deployment (11.6.1 overcloud). We need to test all the combinations we support on a nightly basis.
Analysis

Create separate targets for each of the cells in our supported matrix
Run all tests even if the previous ones fail
Ignore failures on functest_all tests so we run all tests
Removing -j option for make because we do not want these to run in parallel due to resource constraints.
Moved clone of devtest repo to a script that only runs once to avoid checking out twice is same location each time we execute a new target.
Fixed TLC file dir path
Added a bit longer traceback for debugging failures
Make unique TLC session names to workaround session dirs not deleting because of NFS changes for nova storage
Removed some commented out code

Okay, I'm thumbs up on this.


emmet.extensions not working

Issue Type: Bug
The "emmet.extensionsPath" related settings do not work after the update.
Before the update, I was using it well.
It's the same thing with the reinstallation and the samples.
VS Code version: Code 1.48.0 (db40434f562994116e5b21c24015a2e40b2504e6, 2020-08-13T07:50:42.600Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 3192)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.90GB (9.21GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Yes, that is right. After the update, custom snippets stopped picking up. It became inconvenient to work. This is a very useful tool.

Not working here also:
VS Code version: 1.48.2
OS version: macOS Catalina

When do you plan to fix this bug?

There have been four updates since the error appeared

There have been four updates since the error appeared

ok, in the latest build the problem is solved?

There have been four updates since the error appeared

ok, in the latest build the problem is solved?

No! This is the problem. Already four updates in a row, such a necessary tool is ignored :(

Reopening because it auto-closed.
@TD540 can you confirm whether the fix works tomorrow in the next insider's build for macOS? Thanks in advance

Confirmed it works on latest Windows insiders. Will re-close and add this issue to current milestone.

/closedwith 2dbd0ae6a49e64c32611c4206c0917f7144d4051

\closedWith 2dbd0ae

Thank you to whoever fixed this!

For verifiers:

Write an Emmet snippets file or use the example as per https://code.visualstudio.com/docs/editor/emmet
Change the emmet.extensionsPath setting to point to that file
Create a new HTML or CSS file and type one of the snippets. Ensure that the snippet is suggested in the Intellisense.


New build scripts

The parameter type declaration without variable is not supported for Foxy. One option is to remove the galactic fixes https://github.com/tiiuae/odometry2/commit/0ab1daae016c79263060172b9c951f50973d9dd9.
The type decleration <T> from
https://github.com/tiiuae/odometry2/blob/dc53bb8231e69794c7dd8b2b2e07e7e3116d38dd/src/odometry2.cpp#L371
Removing type declaration <T> will result in deprecation warnings in the Galactic build. However, the build will succeed in both distributions.

Yes, It is better to leave galactic change out from this now.


Feature Request: Image in docker hub

I would love to have an image in docker hub so that I don't have to build SimLN myself. I personally use both ARM and x86 architectures and I think these two are good as a starting point for images.
I have twice (Torq + Scaling Lightning) made multi-architecture docker builds that upload to docker hub on a tag and on a github release so please feel free to take a look at the scaling repo to see how I tackled it.
I'm also happy to contribute or talk through how it works in the projects I've done it for; Torq and Scaling do it two different ways.

It's been a while, but I've managed to do this before too for multi-arch. Happy to give it a look/chat if necessary to tackle this.

@sr-gi would you mind me picking this up?

@maxwedwards not at all. Feel free


MAID-2398 chore/Adapt to changes in safe-node-app API

r? @krishnaIndia
(maidsafe_highfive has picked a reviewer for you, use r? to override)

email app is trying to auth via test on osx. webhosting works fine.


Add Workplace recipe

Q
A

Bug fix?
No

New feature?
Yes

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

@antonmedv thanks for the merge. Can you create a new tag with this recipe please?

Yes. Sure.


Add RunTestsInContext Command

This command tries to replicate the CTRL+R, T experience in full VS. I'm planning on sending a followup PR hopefully this weekend to add DebugInContext, and then a PR to the vscode extension to enable the commands. While Omnisharp does have an existing TestCommandService that does some of this, it doesn't quite replicate the experience: it will only run the test method that it's invoked in, not potentially the entire test class or whatever other context it's in. Additionally, it's not unified with the test running framework the test of the V2 test commands use, so I opted to introduce an entirely new command, RunTestsInContext, which will walk up the syntax tree for a given location to find the nearest test method or class, if one exists, and either run the method itself, or run the entire class.
Commit 1 moves Omnisharp forward to the 3.1 LTS SDK, from 3.0.
Commit 2 does some refactoring of the existing test commands to remove .Result usage and plumb async all the way through.
Commit 3 Implements RunTestsInContext.

I've now added DebugTestsInContextGetStartInfo. I don't see any existing tests for the debugging commands: am I just missing something? If not, how should I go about testing this command?

@JoeRobich I've implemented the vscode side here: https://github.com/OmniSharp/omnisharp-vscode/pull/3772


Optional Fields Feature

In most cases submitted values will be received onSubmit and typically posted to an API endpoint for a backend for which there may be optional fields. Since initialValues is a required prop and rightfully so there may be some values set to, for example, and empty string and the user never interacts with them because they are optional.
It would be nice if you add a feature to scan through a validation schema or allow a developer to specify optional fields in initialValues via ? just as is done in many other query languages and schema definitions. Formik would then remove those optional fields from the values dictionary so as to save the developer the trouble of first checking if the fields weren't changed during the use of the form and then transforming those fields and removing them from values.

Just a thought: initialValues is required, but you don't have to set an initial value for any prop. It's up to you if there is a default or not (if not they default to undefined).


service: Reduce backend ID allocation space

Currently, we store service backend ID as uint16 in the BPF maps, which is going to be changed to uint32 in upcoming releases. Meanwhile, the backend ID allocator allocates backend IDs as uint32 and then casts the allocated ID to uint16.
To avoid having the same backend IDs for two different backends in the BPF maps until the proper fix will land (#8041), make sure that the allocator won't allocate an ID > max(uint16).

This change is

test-me-please

Coverage decreased (-0.02%) to 41.909% when pulling 1f4df4b316eb970df89133a7e9f06f0add634303 on pr/brb/fix-backend-alloc-u32 into c347b6cc8664432e29eaebd8942daa8dd0b27af1 on master.


getting different result from  model.save_weights and  model.model.save

I have been using  model.save_weights so far and everything works well except that it appears that I cannot resume training from saved weights. My understanding of the documentation is that I need to use model.save to save everything, do a load_model from the file saved and do model.fit again.
So I try model.save and do a load_model but it returns different results  from those produced by save weights and load_weights.
Below is my setting.
Keras 2.0.8
Python 3.5.2
Theano 0.9.0
Can anyone give me an explanation for this and tell me how to properly use them?
Thanks.
Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[ ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

You can absolutely resume training just by using save_weights and load_weight. If it produces different results, it's a bug in the keras codebase. Can you give a script which shows evidence of the results not being the same? As mentioned in the guidelines, a script is useful because otherwise it's hard to reproduce your issue. Thank you!


MM-14977 Remove metadata backwards compatibility

Summary

Remove API calls for reactions, files and opengraphdata
Remove checks for metadata as they should exist if post exists
Add metadata empty obj for creating posts and comment
Remove waitForHeight callback in size_aware_component

Ticket Link
MM-14977
Related Pull Requests
https://github.com/mattermost/mattermost-redux/pull/811

@sudheerDev Just awaiting the conflicts and test failures.

@mkraft Yup. I will be on it mostly before the start of next week. Just does not feel as important as the other TODO's in my list so this took a backseat.

@mkraft Will let you know once i turn the dev review label on it.

@sudheerDev Do you mind just re-adding me when it's ready?

@migbot @mkraft PR is up for review


Add PHP to sampling tests

Run new sampling system tests for PHP as well.

There are test failures. I change the status to draft

Hi @bwoebi  , do you still plan to work on this?


chore(ci): codegen check

Add checks for codegen
Checkout before setup go so go.mod is present; remove redundant config, go.mod in root is the default

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

Project coverage is 55.49%. Comparing base (dbcc449) to head (43eb1c5).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
+ Coverage   55.32%   55.49%   +0.17%
==========================================
  Files          10       10
  Lines        1146     1146
==========================================
+ Hits          634      636       +2
+ Misses        437      433       -4
- Partials       75       77       +2

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


"Don't care" values in enum constants

Example of an enum with xs in one of the values:
module top(input logic clk, output logic[3:0] o);
   typedef enum logic [3:0] {
        REF      = 4'b0001,
        DESELECT = 4'b1xxx
   } dfi_cmd_e;

   assign o = DESELECT;
endmodule

Corresponding UHDM. I know that the vpiDecompiles are correct, but the value for DESELECT is not. I'm not sure how it should look as per the standard, but this probably isn't it.
  |vpiTypedef:
  \_enum_typespec: (dfi_cmd_e), line:5:6, endln:5:15
    |vpiName:dfi_cmd_e
    |vpiBaseTypespec:
    ...
    |vpiEnumConst:
    \_enum_const: (REF), line:3:9, endln:3:12
      |vpiName:REF
      |vpiDecompile:4'b0001
      |UINT:1
      |vpiSize:4
    |vpiEnumConst:
    \_enum_const: (DESELECT), line:4:9, endln:4:17
      |vpiName:DESELECT
      |vpiDecompile:4'b1xxx
      |UINT:1
      |vpiSize:4

Fixed by #2333


Fix Inexor's HTML5 user interfaces.

@aschaeffer and I came to the conclusion that development of the user interface has priority since it is blocking other projects including the entity system. From my point of view it is not working at all. We should fix this soon.
Technical problems
The following problems need to be fixed

[ ] It's not clear how to start CEF in Inexor ?
[ ] "Cannot GET /api/v1/interfaces/hud/" will be displayed when I click:

This should be displayed in Inexor to show CEF is working but its not visible.

[ ] Sometimes it is neccesary to toggle fullscreen to display the CEF layer.
[ ] Crosshair disappears once you enable CEF layer with /showapplayer 1.
[ ] When making a rightclick in Inexor while CEF is enabled, a browser context menu appears:

To reproduce this bug, type /new_ui 1 and /showapplayer 1 .

[ ] Sometimes CEF's mouse is visible sometimes it isn't.
[ ] You must not be in main menu when attempting to start CEF! If you do nothing will happen:

Having trouble displaying CEF content has nothing to do with implementing specific UIs. Please keep it small and simple (KISS).

Do we need/have individual issues for the different UIs then? I think it is necessary to split between making the UI work with the applayers and creating a candidate - I agree.

Yes.
But please be careful do not create duplicates. We already have several UI related issues.

Ok Lets split this ticket up.
@Croydon but no ticket about these specific issues.

Hud: #312 & #98

Ah ok I will not make a new ticket then.

Please do, I just pointed out that there is already one for implementing a HUD.


Update spark-operator getting-started

Fixed Service Account (RBAC) path.
Ref: https://github.com/kubeflow/spark-operator/pull/2191

/approve

Hi, @ChenYi015
I know you're busy, but could you approve?

@ha2hi Sorry for the late reply. Thanks for updating the docs.
/lgtm


cannot create storage buckets when clusterName has a period in the name

HKube micro-service
mostly storage-manager
Describe the bug
If CLUSTER_NAME env has a period (.) in the name, s3 bucket creation fails in azure

@yehiyam solved?

no, but currently we are not working in Azure. Didn't test it yet in AWS


6292 Streaming Services Emissions-Apportioned Endpoints - Add Fields …

Ticket
https://github.com/US-EPA-CAMD/easey-ui/issues/6292
Changes
Added fields ADD_DATE and USERID to "/emissions/apportioned/annual

@ergjustin, we are getting unit_id as unitId here just like all other snake_case values. If we talk about the other unitid, it's not present in the database table so we are not getting that in the response.


Add set_cursor_close_on_commit  connection parameter

The CURSOR_CLOSE_ON_COMMIT is not available in Microsoft SQL Parallel Data Warehouse, so it fails to connect when it uses this parameter.
It was added as a configuration property.

Coverage decreased (-0.1%) to 72.85% when pulling 53634d39687d0e2248c0e4b9de03fd860d31670a on modelabcl:master into 36f228275c2bfde2baa45fe70d29d1eb1d44379b on livehelpnow:master.

Hey!.
None of the connection properties are being tested, so it's expected to decrease coverage,
Do you have any ideas on how to test it, or would be approved as is ?

Hey!.
None of the connection properties are being tested, so it's expected to decrease coverage,
Do you have any ideas on how to test it, or would be approved as is ?

I'm sorry, I forgot this.
It's actually OK to leave it as nil as default, because sqlserver will use CURSOR_CLOSE_ON_COMMIT OFF by default, and PDW fails only if you try to set this option, so there is no need to explain it on the Readme.


Localization support

For some components, we'll need to provide a solution to set some labels in multi-language. Some examples:

Month/day names in a calendar or date-picker component.
Labels of some internal fields. For example reveal button of password input or "Go" button in pagination (currently we use attributes for that but it's not practical since you need to set them every time.
Error messages for inputs. Currently we use native browser error messages and there is no easy way to set them globally

For that kind of very common use cases it's unavoidable to provide a localization solution.
In my opinion requirements are like:

We should respect HTML's "lang" attribute to set a language.
It should be easy to set translations globally
Translations should accept variables or maybe functions
There should be a solution for pluralization as well

Maybe we can consider using a small library like shoelace's localize that is proven works smoothly and is lightweight enough.
Any opinions are very welcome.

Nice job @AykutSarac 🚀
Only thing is, there is an issue about releases. You already released v3.0.0 but this feature released as v.3.0.0-beta.2. The reason is main branch should be merged to next branch after stable version is done. Otherwise v3.0.0 tag is not present in next branch. I think #784 was for that but maybe it's forgotten. Please have a look.


Sound playback shuts down speaker

Describe the bug
Sound playback using aplay shuts down sound playback
To Reproduce
Use the command
aplay /tmp/tts_output.mp3

Screenshots
If applicable, add screenshots to help explain your problem.

Fixed by attaching microphone directly to the NUC


missing multiverse IDs: bug?

Hi
I understand a ICard won't have a  multiverse ID when the card is not on Magic the gathering's gatherer website.
But I found many cases (I think) of cards that exist but still have no multiverse ID.
For example the tenth edition (10E) card set. All cards in this set should be on gatherer, right?
Please look at the screenshot and see how many cards in this set have no multiverse ID.

Look at card "angel of mercy". There exist 2 cards: one with number 2 and one with number 2 "star". I think the 2 cards are the same but have just another art. (see many more examples in the screenshot).
I want to download each different art for each card using Scryfall using the multiverse ID.
All cards with a "star" in the number seem not to have a multiverse ID. So I can't download those cards.
Is this a bug in the API, the SDK wrapper or does gatherer NOT include cards with the same name but other art?
I'm sure I saw the same cards with other art in my magic collection (in many different sets) so this is a bit of a mystery :-)
ps: the screenshot is raw data I got by using the SDK and put in a GUI (nothing changed or edited by myself)
Thanks for the help!

I believe that the numbers with the star are foil versions of the card, which I would assume Gatherer does not keep track of, hence the lack of the multiverseId and imageUrl attributes.

if you call the api directly with
https://api.magicthegathering.io/v1/cards?name=angel of mercy
you can see that a few don't have multiverseid.

Hi
Thanks for the explanation!
However I found more cases for which I don't know an explanation. This time in the 8ED (3 cases have no multiverseID, all other cards have).
Some cards seem to have card number ending with "s" and they don't have a multiverseID.
Does anybody know what those cards are exactly?

Lisa

sorry, no idea. The lib just takes what the API gives it


Support Spring Proxy classes marked with @Component for activities

Is your feature request related to a problem? Please describe.
Right now when i try to registerActivitiesImplementations with a Spring Proxy bean, it failed with error
Caused by: java.lang.IllegalArgumentException: Class doesn't implement any non empty interface annotated with @ActivityInterface:
xxx.xxx.xxx.ActivityImpl$$EnhancerBySpringCGLIB$$40723738
	at io.temporal.common.metadata.POJOActivityImplMetadata.<init>(POJOActivityImplMetadata.java:109) ~[temporal-sdk-0.11.0+1-0-7.jar:0.11.0+1-0-7]
	at io.temporal.common.metadata.POJOActivityImplMetadata.newInstance(POJOActivityImplMetadata.java:53) ~[temporal-sdk-0.11.0+1-0-7.jar:0.11.0+1-0-7]
	at io.temporal.internal.sync.POJOActivityTaskHandler.registerActivityImplementation(POJOActivityTaskHandler.java:110) ~[temporal-sdk-0.11.0+1-0-7.jar:0.11.0+1-0-7]
	at io.temporal.internal.sync.POJOActivityTaskHandler.registerActivityImplementations(POJOActivityTaskHandler.java:179) ~[temporal-sdk-0.11.0+1-0-7.jar:0.11.0+1-0-7]
	at io.temporal.internal.sync.SyncActivityWorker.registerActivityImplementations(SyncActivityWorker.java:55) ~[temporal-sdk-0.11.0+1-0-7.jar:0.11.0+1-0-7]
	at io.temporal.worker.Worker.registerActivitiesImplementations(Worker.java:326) ~[temporal-sdk-0.11.0+1-0-7.jar:0.11.0+1-0-7]

Looking through the code, the ActivityInterface was not recognized as an interface in Proxy class.
Describe the solution you'd like
We want to make this activity class recognizable.
Describe alternatives you've considered
N/A.
Additional context
How to attach a screenshot? lol

Hi Gina,
Any chance you can provide an example of your wiring or at least how do you specifically make the injection?
If you inject (or request from the Spring container) specific Activity interfaces - Spring proxies should implement the right interfaces.
My guess is that your wiring is probably organized in a way that all activities are implementing some marker interface and you ask Spring to give a collection of all instances implementing this marker interface. This will not work because Spring will create proxies that implement the marker interface only, but not actual Activity interfaces.
Let me know if you can share more information here.


Travis should only sign and build APK on master branch

Description (required)
Fixes #1056
What changes did you make and why?
Travis now checks we're on the master branch before pushing a release to Google Play. Otherwise, things like this will create an alpha build for Google Play:
https://travis-ci.org/commons-app/apps-android-commons/builds/468986786
(Worked on a non-master branch of commons-app, which resulted in Travis running a continuous-integration/travis-ci/push build - see #2148)
Tests performed (required)
Not really able to test, so need someone to look at carefully.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #2162   +/-   ##
======================================
  Coverage    5.69%   5.69%
======================================
  Files         233     233
  Lines       11593   11593
  Branches     1076    1076
======================================
  Hits          660     660
  Misses      10878   10878
  Partials       55      55

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 1d80cba...ffe9b2d. Read the comment docs.


Temporal data?

Hi,
Thanks for making your repo open source and thanks for providing this API. I think it might be interesting to add in which day each case was reported, this might be useful for building plots of cases over time, or timeline maps (such as https://public.tableau.com/views/coivd-nzmap/Dashboard1?:embed=y&:embed_code_version=3&:loadOrderID=1&:display_count=y&:origin=viz_share_link). While MOH don't have a "date reported" column on their site, we can correlate the case numbers with the days/case numbers on https://github.com/CSSEGISandData/COVID-19/blob/master/csse_covid_19_data/csse_covid_19_time_series/time_series_covid19_confirmed_global.csv - e.g.
Case 1 was reported on 2/28/20
Cases 2-3 were reported on 3/4/20
Case 4 was reported on 3/6/20
and so on. These dates could be added as properties to the geojson, or as a column to the csv.

@neon-ninja very good timing - I got another request today, from a researcher needing a date column. I've taken a first stab at this in a Google Sheet that I have set up to pull the CSV API periodically, see the right-most columns in the "DATA" tab:
https://docs.google.com/spreadsheets/d/1NJiDg2z7Y79RUkQ-a648fFD_EmziztYLLkQW_FOrLGo/edit#gid=576672848
I had already manually added the more recent dates, but needed a source for the cases reported last week, which I found in the second link you posted - so thank you!
I'm not sure yet whether it is worth adding a persistent data store to the API service and joining records scraped from MOH with that automatically. Someone else is working on a standard data format (an SQL schema) that (I hope) MOH will adopt, in which case they might add a date column, and perhaps also make the whole scraping/normalisation thing unnecessary.
Please let me know what you think.

Nice. It's up to you where you want to draw the line - otherwise before you know it you'll be adding things like census stats for each affected location. It's fairly trivial to do the join at the application level.

I think you might have an issue with the M/D/Y parsing though - some cases are classified as April - 2020-04-01 for example

@neon-ninja good catch. I think it is right now, however it looks like this effort may overtake mine with much better data quality: https://github.com/nzherald/nz-covid19-data
I think I read a comment in one of their repo's issues that they're planning to add reported date.

@neon-ninja good catch. I think it is right now, however it looks like this effort may overtake mine with much better data quality: https://github.com/nzherald/nz-covid19-data
I think I read a comment in one of their repo's issues that they're planning to add reported date.

well, but no JSON there? (I bet really annoying when the format of the source has been changed)

@neon-ninja note that MOH has added date columns, and the API now exposes these.


up-board: balena-image: don't install grub modules

The bootloader has all necessary modules built into the EFI image. Free
up some space by removing unnecessary GRUB modules.
This frees up ~3.5M.
Change-type: patch
Signed-off-by: Joseph Kogut<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@klutchell This build has been running all night, is there a way I can check on the status?

@resin-jenkins add to whitelist

Status was still Expected — Waiting for status to be reported  suggesting the queue was full or it wasn't whitelisted to run.


Adding Italics Version to Fast-Font

Hello,
I would like to apply the fast-font feature to my preferred e-reader font. I managed to get it to work for the regular font but I do not know how to get it to work with the italics. Can you please help?
I have uploaded the original fonts here.

I did a bit more work trying to apply the fast-reading. I think I have it about 90% right. But I'm curious how to get it to work better with text that is already bold. Currently, if the base text is meant to be bold, then the entire line retains the same boldness. Does that mean I need to make a set of glyphs that have a different boldness as well? I've uploaded what I have done to the same drive.

Hi,
I just looked at your results, which are really cool! Congratulations!
Regarding your question:
The boldness of already existing bold-formatted text is actually a problem.
I also thought about that for a long time, but there is unfortunately no 100% perfect solution.
The only way is, as you already suggested, to add an extra set of glyphs with an even stonger boldness.
There exist several fonts, that have a bold and a extra-bold subset of glyphs.
I am pretty sure, your desired typewriter font does not directly offer such an extra-bold subset.
That means you would have to edit the glyphs in a tool like FontLab and make them bolder. Unfortuantely that probably also changes the character spacing. That's why it is not perfect. It would be much more streamlined if you use a font that is already prepared for bold and extra-boldness.
Afterwards you need to also adjust the feature-code:

add a new category for the extra bold glyphs. For instance @AZE = [a.ebold-z.ebold A.ebold-Z.ebold];
add a new rulest of calt, that checks if a glyph is bold and make it extra bold. You would have to first do the extra-boldness part and afterwards the "normal" boldness. That means you probably have to duplicate the whole code, what's also not that nice from the programmers perspective.

But just give it a try! 😉


chore: bump grpc to v1.67.0

Manually fix:  https://github.com/envoyproxy/gateway/pull/4305

can you also update dependabot.yml with following:
      golang.org:
        patterns:
          - "golang.org/*"
          - "google.golang.org/*"

It looks like the GRPC upgrade broke the e2e and conformance tests.
Please hold this PR.


Have an option to control UI/font size for image_viewer or gltf_viewer

Is your feature request related to a problem? Please describe.
Currently the filament UI doesn't provide options to scale or magnify UI/font.
Describe the solution you'd like
I'd like to have control over the size of UI/font for a certain set of sample apps.

These are just sample/test apps, our goal is not to support them like real apps.

In ImGuiHelper.cpp, we can pass a different value via io.Fonts->AddFontFromFileTTF(fontPath.c_str(), 16.0f); to change the font.
To do this dynamically, we may want to call ClearFonts() before readd the font again, which might also have to call ClearTexData() and ClearInputData() from ImFontAtlas.


General support of custom formats for exporting (starting from CSV)

For data, which represented as columns (like dlt) would be great to have a possibility to apply custom exporter. For example, export to CSV. The current version supports exporting only to the original format. It means DLT can be exported into DLT only. But as soon as render already reformats data to columns, it also can be exported to formats like CSV.
Feature:

export data represented as columns to CSV
export should be available for: all rows; selected rows; all rows in search.

Technical suggestion:

@marcmo probably it make sense to do this feature on plugin base system. It can be on nodejs side of rust side. Needs to be discussed.

I could make sense to represent the content of a row as a key-value-mapping (keys=columns).
that way we could create a generic exporting and filtering mechanism


composer install eror

I downloaded the SDK, uncompress it and issued a composer install but I get the following error:
[Composer\Json\JsonValidationException]
"./composer.json" does not match the expected JSON schema:

type : Does not match the regex pattern ^[a-z0-9-]+$

It seems that your composer.json file is not formatted according to latest Composer version (1.10.9).

The error is relative to the composer.json:
"type":"Library"
should be
"type":"library"
lower case 'L' (library)
Is anybody checking the code works with new technologies (Composer, PHP)?


Enhance FindCommand for v1.3

Add 4 predicates (TagsContainKeywordsPredicate, StudentIdContainsKeywordsPredicate, StudentNameContainsKeywordsPredicate, and ProjectNameContainsKeywordsPredicate) to support finding keywords in the 4 different fields.
Add 4 subclasses of FindCommand to support the above.
Modify Developers' Guide and Users' Guide to update developers and users on the newly improved FindCommand.

Important Note on using Find
In previous iterations, there was only one find keyword. In this update, there are four sub-keywords instead to support the change:

find-t: find by Student's Tags
find-id: find by StudentId
find-n: find by StudentName
find-proj: find by Project Name

More details can be found in the UG.
This issue resolves #110.

To-do: Check that the Test files are correctly updated. I copied and pasted most of them, so will check these in a future PR.

I will fix the CI bugs first. Please do not merge.

Codecov Report
Base: 64.49% // Head: 65.24% // Increases project coverage by +0.75% :tada:

Coverage data is based on head (7db7a90) compared to base (f53c1dd).
Patch coverage: 90.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #125      +/-   ##
============================================
+ Coverage     64.49%   65.24%   +0.75%
- Complexity      484      521      +37
============================================
  Files            91      101      +10
  Lines          1659     1715      +56
  Branches        183      188       +5
============================================
+ Hits           1070     1119      +49
- Misses          537      539       +2
- Partials         52       57       +5

Impacted Files
Coverage Δ

...java/jeryl/fyp/logic/commands/HelpFindCommand.java
0.00% <ø> (ø)

...java/jeryl/fyp/logic/parser/HelpCommandParser.java
33.33% <ø> (ø)

.../java/jeryl/fyp/logic/parser/FypManagerParser.java
78.26% <50.00%> (-6.74%)
:arrow_down:

...el/student/StudentIdContainsKeywordsPredicate.java
85.71% <85.71%> (ø)

...yp/model/student/TagsContainKeywordsPredicate.java
85.71% <85.71%> (ø)

...l/fyp/logic/parser/FindStudentIdCommandParser.java
90.00% <90.00%> (ø)

...fyp/logic/parser/FindStudentNameCommandParser.java
90.00% <90.00%> (ø)

.../jeryl/fyp/logic/parser/FindTagsCommandParser.java
90.00% <90.00%> (ø)

...ain/java/jeryl/fyp/logic/commands/FindCommand.java
100.00% <100.00%> (ø)

...ryl/fyp/logic/commands/FindProjectNameCommand.java
100.00% <100.00%> (ø)

... and 9 more

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

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


Return exit code

Fixes a bug with the test wrapping logic. We need to return both the result of m.Run and any errors. This is so in the tests we can call os.Exit and signal to Circle that the job should fail.

I prefer it this way because it matches how we do the setup in tests that don't have the logic extracted.


Failed assertion  in gestures/converter.dart ('!state.down': is not true.)

Steps to Reproduce
I just had this happening twice when going back from a demo page to the home page in the gallery app (TalkBack was on).
I was running unmodified Flutter TOT (synced to cbfed1960f80964ba89f0093a49e1a0d44ca95d9)
Logs
E/flutter ( 3509): [ERROR:topaz/lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
E/flutter ( 3509): 'package:flutter/src/gestures/converter.dart': Failed assertion: line 131: '!state.down': is not true.
E/flutter ( 3509): #0      _AssertionError._doThrowNew (dart:core-patch/dart:core/errors_patch.dart:37)
E/flutter ( 3509): #1      _AssertionError._throwNew (dart:core-patch/dart:core/errors_patch.dart:33)
E/flutter ( 3509): #2      PointerEventConverter.expand (package:flutter/src/gestures/converter.dart:131)
E/flutter ( 3509): #3      _SyncIterator.moveNext (dart:core-patch/dart:core/core_patch.dart:149)
E/flutter ( 3509): #4      ListQueue.addAll (dart:collection/queue.dart:635)
E/flutter ( 3509): #5      BindingBase&SchedulerBinding&GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:46)
E/flutter ( 3509): #6      _invoke1 (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:115)
E/flutter ( 3509): #7      _dispatchPointerDataPacket (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:73)
E/flutter ( 3509): [ERROR:topaz/lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
E/flutter ( 3509): 'package:flutter/src/gestures/converter.dart': Failed assertion: line 131: '!state.down': is not true.
E/flutter ( 3509): #0      _AssertionError._doThrowNew (dart:core-patch/dart:core/errors_patch.dart:37)
E/flutter ( 3509): #1      _AssertionError._throwNew (dart:core-patch/dart:core/errors_patch.dart:33)
E/flutter ( 3509): #2      PointerEventConverter.expand (package:flutter/src/gestures/converter.dart:131)
E/flutter ( 3509): #3      _SyncIterator.moveNext (dart:core-patch/dart:core/core_patch.dart:149)
E/flutter ( 3509): #4      ListQueue.addAll (dart:collection/queue.dart:635)
E/flutter ( 3509): #5      BindingBase&SchedulerBinding&GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:46)
E/flutter ( 3509): #6      _invoke1 (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:115)
E/flutter ( 3509): #7      _dispatchPointerDataPacket (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:73)

Run flutter analyze and attach any output of that command also.
Analyzing 1277 files...
No analyzer warnings! (ran in 37.8s; 13 public members lack documentation)

Flutter Doctor
$ flutter doctor
[✓] Flutter (on Linux, locale en_US.UTF-8, channel master)
• Flutter at /usr/local/google/home/amirha/git/flutter
• Framework revision cbfed19 (2 hours ago), 2017-10-25 07:18:09 -0700
• Engine revision ec19da1c80
• Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.2)
• Android SDK at /usr/local/google/home/amirha/Android/Sdk
• Platform android-26, build-tools 26.0.2
• Java binary at: /usr/local/google/home/amirha/progs/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] Android Studio (version 2.3)
• Android Studio at /opt/android-studio-with-blaze-2.3
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] Android Studio (version 2.3)
• Android Studio at /usr/local/google/home/amirha/progs/android-studio
• Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2)
• Flutter plugin version 18.0
• Dart plugin version 172.4155.35

[✓] Connected devices
• Pixel • FA6890301419 • android-arm • Android 8.1.0 (API 27)

Is this issue still relevant?

Hard to say without further analysis, though if know one hit this for a year I guess we can close it and re-open or open a new issue the first time we see it again.


[Snyk] Security upgrade ssri from 7.1.0 to 8.0.1

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Regular Expression Denial of Service (ReDoS) SNYK-JS-SSRI-1085630
Yes
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: ssri
    The new version differs by 9 commits.

3eec7a3 chore(release): 8.0.1
2083289 chore: update package-lock and dev dependencies
76e2233 fix: simplify regex for strict mode, add tests
41b764f chore(release): 8.0.0
4062735 fix: harden SRI parsing against ../ funny business
a6811cb fix: throw null when sri is empty or bad
1727a7c chore: consistent project setup
4a963e5 fix: IntegrityStream responds to mutating opts object mid-stream
0e78fd7 feat: remove figgy-pudding

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Changelog for major version change. Doesn't seem likely to impact us.


Make it easier to specify the output-capture mode for pytest with python-test

It can be annoying for python-test to always capture stdout but log DEBUG.

https://github.com/vkottler/vmklib#python
PYTEST_EXTRA_ARGS does this fine


Fix Multi-Delete Capcodes on MYSQL

Description
In some circumstances on MySQL, using the multidelete can throw an error
"Cannot delete or update a parent row: a foreign key constraint fails (`pagermon-test`.`messages`, CONSTRAINT `messages_alias_id_foreign` FOREIGN KEY (`alias_id`) REFERENCES `capcodes` (`id`))"

This recreates with column with an SET NULL on delete key.
This requires dropping the alias_id column first, which can take quite some time on a large database.
It will also require an alias refresh once the process is complete.
Fixes # No issue created
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Tested Migrations on copy of my production database.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have updated changelog.md with changes made

@pagermon/collaborators i'm not aware of a better way of doing this than dropping the column and recreating, it doesn't seem to be alterable.
Let me know if you know a better way 👍

I ran this against my database with 2.5mil messages and 2000 aliases, it was not a fun time ... i can't imagine it would be any fun running that on sqlite either tho 😁


Can't use SleekDB without Composer

I get "can't find IoHelper" each time I try to use SleekDb without Composer. Managed to fix it by replacing line 19 in Store.php to
foreach (glob(__DIR__ . '/Classes/*.php') as $traits) {
Also had to add these lines to Query.php:
require_once __DIR__.'/Classes/CacheHandler.php';
require_once __DIR__.'/Classes/DocumentFinder.php';
require_once __DIR__.'/Classes/DocumentUpdater.php';

Thank you for reporting the bug.
Fixed with version 2.7.2.
PS: When using SleekDB without composer just use the following line:
require_once __DIR__ . "<PATH_TO_SLEEKDB>/Store.php";

Then you can use every class.

require_once __DIR__ . "<PATH_TO_SLEEKDB>/Store.php";

use SleekDB\Store;
use SleekDB\Query;

// ...


Make string labels selectable

It's currently impossible to select text form a label.

as mentioned it conflicts with the drag select operation

I'd solve this by allowing the drag selection either

only on the selection column (i.e., a classical cross-set like in Bertifier https://aviz.fr/bertifier_app/Matrix.html?csv=1 - here there are multiple cross-filters for different options all only with a fairly narrow "region")
Also allowing it also on the rank column
Though I think just on the selection column would be the most consistent.

You could also check the direction of the drag operation:

vertical = select rows
horizontal in a single row/cell = select text
horizontal over multiple rows = select text


[main] Upgrade to latest dependencies

release 1.15 -dsimansk
/cc knative-extensions/serving-progressive-rollout-approvers
/assign knative-extensions/serving-progressive-rollout-approvers
Produced by: knative-extensions/knobots/actions/update-deps

/lgtm
/approve


Removed flib dependancy and install pexpect using easy_install_package

This allows cpe_macos_server to function without requiring flib on the machine.

Apparently chef is deprecated easy_install and pip is not included by default in OS X, so I'd rather axe this unless someone has a good idea that is a quick (and good) fix.


Backport #9102 to the 1.49 release branch

#9129 but to the branch that matters, this time with the version bump.
Does this look correct?

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.49.0. Please double check that you specified the right target!

Thanks! I don't think it's that worth it to get CI working at this point, so I'm going to hit the merge button on this one. I'll do the publish afterwards as well.


Syncing from upstream zalando/patroni (master)

bt_gitbot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 15 (100.0%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.009%) to 99.837%

Files with Coverage Reduction
New Missed Lines
%

patroni/daemon.py
1
98.75%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
11004

Relevant Lines:
11022

💛  - Coveralls


How to remove DEFAULT_UA?

Hi,
Could you tell me how to remove the DEFAULT_UA? First I thought there would be a simple --no-useragent setting or even --user-agent flag where I could quickly set an alias but apparently not then I tried making a default ~/.config/httpie/config.json file
{
"headers": {
"User-Agent":"",
}
but that didn't work either

The User-Agent is just a header and can be set as such. To remove User-Agent entirely
http --offline pie.dev/headers User-Agent:''

or to send a blank one (which I'm not sure is what you want)
http --offline pie.dev/headers User-Agent:' '

n.b. that's not actually blank, I believe it will send a blank space as the UA.
See the documentation on setting headers here

Yes, just specify the header without any value.
$ http --offline pie.dev/headers User-Agent:

GET /headers HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Host: pie.dev

Docs: https://httpie.io/docs/cli/empty-headers-and-header-un-setting


Convert Sans Kannada to pure-Glyphs source (Read notes!)

Please note, this is different from the Monotype build in a few ways:

Some kerning rules are not being applied. This is because in the MTI file, the kerning feature has a mark filtering set to skip over certain marks and kern bases. Unfortunately there is no way to add a mark filtering set to kerns generated by ufo2ft kernWriter. As far as I can tell the differences are quite minor, and the gain in having the kerning rules in the Glyphs sources rather than manually in a per-master feature block far outweighs the small differences.
On the other hand, mark positioning wasn't being applied to Vedic marks before, and now it is, so that's a win.
No UI build, but we are planning to handle the UI-ification via a separate script.

Please test carefully!

there is no way to add a mark filtering set to kerns generated by ufo2ft kernWriter

we can add support for that, if there's need for it. Please file a feature request when you get the chance, thank you


False positive: component can only have one instance-level

The following Svelte component:

  hello!

makes the plugin crash with:
1.svelte[error] 1.svelte: ParseError: A component can only have one instance-level

  hello!

hmm, that is odd given that the stack trace suggests its coming from svelte itself

Do you need the reproduction repo? Yes, I saw that the error was coming from the compiler itself, but perhaps it's not the original file content that is passed to it (or passed with a wrong set of options).

I suspect its this line https://github.com/UnwrittenFun/prettier-plugin-svelte/blob/668fb44587fc8cc8f112f135400211d2483261d7/src/index.ts#L31 picking out all scripts regardless of where they are


update caddy to 2.6.4 alpine

This update has been broken for some time on Kubernetes/OpenShift.  It looks like increased security contexts are the problem.

Thanks for the PR!
Any successful deployments (not always required) will be available below.
API available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@DerekRoberts looks like the cypress e2e saved us, the UI is not working, need to look deeper tomorrow


Refactor backend structure

Right now my little backend is a mess. To make it better I want to extract its parts into projects:
Core
Defines interfaces (or classes/records) for DTO entities.
Defines interfaces for Database services.
Defines interfaces for FileSystem services, as well as interfaces (classes/records) for results.
Database
Exposes one or many services that provide DTO entites.
Provides an extension method to register the DB with the IServiceCollection.
Provides an extension method to register it's own services with the IServiceCollection.
References Core
FileSystem
Exposes services that can read data from the file system.
Provides extension methods form services registration.
References Core and Database (Database is required to save data)
Utils
A collection of extension methods, mappers etc
Api
Consumes the services from other projects. Probably has an amount of own services that are used to transform data to/from other layers' models, run validations etc.

I've extracted code into separate projects and refactored it a little bit.
There are many TODOs I have for refactoring, but this particular task is done.


Document and refactor the plotting scripts dir

This is a minor touch-up to store the plotting scripts in the relevant order (and added a small doc snippet on what are the scripts about). I'll merge this once the tests pass. Thanks!


Tom Beucler, Abstract/Slides upload

Uploaded abstract and slides, changed fulldetails.md to add hyperlinks for the Thursday talk

Thanks!


ACCUMULO-3238 Table.ID Namespace.ID Refactor

Replaced all occurrences of tableId and namespaceId Strings
that aren't a part of the user facing API with type safe
Table.ID and Namespace.ID objects.
Table.ID and Namespace.ID both extend the abstract class AbstractId
Created new methods in Tables and Namespaces for internal
use of these type safe objects.

NOTES:
Utils.getNextTableId was used for both table ID and namespace so I used generics and reflection.
WriteExportFiles.exportTable() got dinged by FindBugs for not closing the OutputStreamWriter
Some questions/concerns:
RenameTable & Utils passes around a String tableID and long tid - is this “tid” the actual numeric tableID?
TableDiskUsage has internal and external IDs?  I didn't touch this class.
SequentialWorkAssigner & ReplicationTarget - peerName and tableID are both used as keys in the same map but peerName seems like a tableName in tests
TabletStateChangeIterator.onlineTables - set of Strings parsed from “tables” iterator options… looks like table names but calls: onlineTables.contains(tls.extent.getTableId())

I believe I have addressed all concerns.  @joshelser if you aren't opposed to the Table and Namespace wrapper classes, I will merge this PR.


Docker container should log to stdout

Currently the Docker container is configured to log to files /var/tmp/burrow.  The burrow.out logfile appears to be hardcoded:
https://github.com/linkedin/Burrow/blob/053606df50e12e41b1f0fc2de498ce05fa8d5c97/main.go#L63-L64
It's generally preferred for apps in Docker containers to log to stdout, to allow their logs to be viewed with 'docker logs'.  This also allows for frameworks to collect logs and redirect them to log aggregation servers.
Please remove the hardcoded assumption that a logfile will always be used and allow SeeLog to log to the console (which is configurable with the  setting).

OOPS!  I opened this under the completely wrong project!  Please disregard!


The cpu is working in the gpu version

I'm running GPU version 2.0.0, and have installed the NVIDIA drivers, but the CPU is busy!

I'm having the same problem. Have you figured out how to fix it?

I'm having the same problem too.


[Bug] date_picker doesnt load value when wrapper class does not include form-group

Originally submitted by Jens Chr. Andersen via Stack Overflow question date_picker field doesnt show value from database in backpacker 4.1 on laravel 8
Bug report
What I did
Used a date_picker field and set the wrapper->class option to a value that does not include form-group

            $this->crud->addField([
                'name' => 'date',
                'label' => 'Date',
                'type' => 'date_picker',
                'default' => date('Y-m-d'),
                'wrapper'   => [
                    'class' => 'col-md-6'
                ],
            ]);

What I expected to happen
I expected the field to be pre-filled on the edit page when an entity has a stored value for the corresponding column
What happened
The field remained empty and the following error appeared in the console
bundle.js?v=4.1.22@2c87086173c0c1b746d6f0fd49783f6eefa787e9:2 Uncaught TypeError: Cannot read property 'length' of undefined
    at bpFieldInitDatePickerElement (edit:654)
    at HTMLInputElement.<anonymous> (edit:1336)
    at Function.each (bundle.js?v=4.1.22@2c87086173c0c1b746d6f0fd49783f6eefa787e9:2)
    at x.fn.init.each (bundle.js?v=4.1.22@2c87086173c0c1b746d6f0fd49783f6eefa787e9:2)
    at initializeFieldsWithJavascript (edit:1331)
    at HTMLDocument.<anonymous> (edit:1347)
    at c (bundle.js?v=4.1.22@2c87086173c0c1b746d6f0fd49783f6eefa787e9:2)
    at u (bundle.js?v=4.1.22@2c87086173c0c1b746d6f0fd49783f6eefa787e9:2)

Reproducing the error

clone https://github.com/DoDSoftware/demo/tree/date-picker-error
install the Backpack demo as normal from the cloned repo branch
navigate to /admin/article/1031/edit
note that the "Date" field is unfilled
open the console
observe the noted js error

What I've already tried to fix it
Updated js in date_picker.blade.php to not rely on the wrapper having the class form-group in order to find the hidden field containing the incoming column value.
Submitting PR with proposed solution.
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
### PHP VERSION:
PHP 7.4.5 (cli) (built: Apr 19 2020 07:36:30) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
    with Zend OPcache v7.4.5, Copyright (c), by Zend Technologies
    with Xdebug v2.9.3, Copyright (c) 2002-2020, by Derick Rethans

### LARAVEL VERSION:
v8.1.0@d769a497a20e4a374ff7454dde7eee65145e3659

### BACKPACK VERSION:
4.1.22@2c87086173c0c1b746d6f0fd49783f6eefa787e9

Hello @DoDSoftware
Thanks for such a clear issue report. Easy to know what the problem is and how it could be solved (thanks for the PR tho) !
I am going to close the issue and review the PR #3309 so we can push the fix asap, probably monday the next.
Wish you the best,
Pedro


Add nested associations

This PR makes it possible to add associations to any child. The source
and destination can then be specified relatively. This increases the
locality of information and eases refactoring of bigger systems.

Fixes #13

Thanks Sam! I'd been meaning to get around to this.
So much more ergonomic! 👍


Upgrade to Websocket seems to ignore 'typed_per_filter_config' of route when using default HTTP filter chain

Config:
                        - match:
                            path: "/mqtt"
                          route:
                            upgrade_configs:
                              upgrade_type: "websocket"
                            cluster: service_mqtt
                          decorator:
                            operation: "mqtt"
                          typed_per_filter_config:
                            internal_oauth2_oidc:
                              "@type": type.googleapis.com/envoy.config.route.v3.FilterConfig
                              disabled: true

Logs:
[2024-02-01 07:12:11.904][34][debug][oauth2] [source/extensions/filters/http/oauth2/filter.cc:441] can not skip oauth flow
[2024-02-01 07:12:11.904][34][debug][oauth2] [source/extensions/filters/http/oauth2/filter.cc:373] path /mqtt does not match with redirect matcher. redirecting to OAuth server.

Version: Docker v1.29-latest

@stevenzzzz might have some insight (based on having added route.proto's typed_per_filter_config)
@alyssawilk probably has some knowledge about the websocket part (based on having had something to do with it 5 years ago!)
One thing to check since we can't see it in the provided config snippet is does the oauth filter you're trying to disable have name= internal_oauth2_oidc, or for simplicity, is that same typed_per_filter_config respected if you don't have a websocket upgrade in there?

IIUC, as long as your http-request goes through the filter-chain, right now typed-filter config is always honored no matter what your http method is.
Per the latched config, are you using the right "filter name" in your typed-filter-config?
I think the oauth2 filter name is "envoy.filters.http.oauth2" per my code search.

With 'https://' requests the typed-filter config will respected, with 'wss://' not:
http:

wss:

One of the not disabled filter:

@stevenzzzz Any idea why wss request catched by disabled filterchain and https not?

When Envoy XDS loads various layers' per_filter_configs_ (either route or VHost... ), there is no selection of what connection type it is.
when a filter going through the layers of per-filter config, I don't think there should be any filtering as well.  I cant see mostSpecificPerFilterConfig called by oauth2/filter.cc. Are you adding the typed_per_filter_config to your wrapper filter impl (internal_oauth2_oidc)?
I am not sure how your impl is done, nor can I tell from the config snippets shared here, so can't give you meaningful suggestion there.
But I'd probably check the customized wrapper filter impl, as far as I can tell, the original oauth2 filter impl doesnt read per-filter config, unless the wrapper impl feeds that into nested filter's passThroughMatchers()

/reopen


Jobs maxing at 10,000

Malspider appears to stop launching new jobs once the number of the completed ones reaches 10,000 (as seen here). It still consumes a large amount of RAM. Restarting the server seems to temporary solve it, as the "completed jobs" counter resets to zero.

Hey Alex,
New jobs are still launched, but scrapyd (according to the default
settings) will only keep 10k finished jobs in the launcher. This means once
you reach 10k, the oldest finished jobs will be removed from the launcher
to keep the # at <= 10k. You can change this setting in the
malspider/scrapyd.conf file.  Look for "finished_to_keep" and change it to
whatever you like. scrapyd ships with the default value of 100, but I
changed it to 10k for Malspider. You'll also see a "jobs_to_keep" setting.
This relates to how many log files to store.
Ideally we should have a counter in the database to track the # of jobs
completed, or get rid of the "finished jobs" status altogether. I'm leaning
towards getting rid of it since there are plenty of spidering stats and
info about outstanding and running jobs. Do you agree with that?
-James
On Wed, Nov 30, 2016 at 3:34 AM, Alex Shatberashvili <
<EMAIL_ADDRESS>wrote:

Malspider appears to stop launching new jobs once the number of the
completed ones reaches 10,000 (as seen here
http://i.imgur.com/itENqeu.png). It still consumes a large amount of
RAM. Restarting the server seems to temporary solve it, as the "completed
jobs" counter resets to zero.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ciscocsirt/malspider/issues/14, or mute the thread
https://github.com/notifications/unsubscribe-auth/AR0QEOV8t6aiGhxq7awm7em257U5oCxPks5rDTUngaJpZM4K_7kH
.


pkg/maps: add BPF ipcache for egress policy enforcement

Plumb the contents of the ip-identity cache to a BPF map for lookup in the datapath.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>Related-to: #1488
Plumb the contents of the ip-identity cache to a BPF map for lookup in the datapath.

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

had to bump the map size in the datapath to match the go analog of it, which explains the recent test failures.
I've marked the map as non-persistent in the mean time because we might want to change things in the near term and not break upgrade. @tgraf is this reasonable?

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

Conntrack test failed, which is unrelated to my changes. Rebased to fix merge conflict with BPF SHA, and started tests again.

test-me-please

I kept getting hit by merge conflicts with the bpf.sha. None of the CI failures have been related to this change AFAICT, and have been related to other problems.

test-me-please


intercept xhr requests made at application startup

Is there a good way to have requests that are made at startup get connected to the trace?
The last request here is made to the backend that served up the initial trace span but it is not being made with the traceparent header present.

Doing a little digging, it appears that XHR requests made at startup aren't connected to a server-provided trace because isTracingZone isn't true on the  Zone, as far as I can tell.

Thanks for reaching out about this. I have been meaning to put a disclaimer on the repo, but this specific repo is not being actively maintained at this point, but instead work has focused on OpenTelemetry JS, which has similar functionality, see https://github.com/open-telemetry/opentelemetry-js and https://github.com/open-telemetry/opentelemetry-js-contrib/tree/master/plugins/web.
I would recommend giving those a try and if the problem is with them as well, then file an issue with that repo - the maintainers are more active and the community is stronger there.
See https://opensource.googleblog.com/2019/05/opentelemetry-merger-of-opencensus-and.html for more about the broader merge of OpenCensus into OpenTelemetry.


test_svars_single_precision is failing

https://github.com/ansys/pyfluent/actions/runs/5886606148/job/15964724767#step:22:1560

@mkundu1 there are a number of nightly failures - are they covered by issues?

@seanpearsonuk Re-opened #1814 and linked PR.


refactor!: Rename name to label and use value for label content

I think the new property name and the new use of value on nodes with Role::Label makes sense because:

For non-label nodes, their label is now determined by either a direct string in the label property, or the labelled_by relation. The word choice is more consistent.
On a label itself, it doesn't make sense to talk about the "label of a label", but "value of a label" does make sense.
For labels that have full text support via TextRun nodes, we can now automatically get the label's value from its descendant text runs, rather than having to set the value directly; this follows automatically from the existing support for constructing the value from text in accesskit_consumer::Node::value.

I'm on board with these changes, but I expect the transition to the value property to be painful. Maybe you should split this PR so it has its dedicated entry in the changelog, until we properly document everything?
Also, you're effectively creating a new kind of nodes which are labeled by their value, and I wonder if you should explicitly name them by adding a new method in the consumer crate. I currently have no idea how we'd name this class of nodes, but I don't really like having this check on roles in multiple places.

I don't think I understand the benefit of splitting the PR.

I don't like the extra churn that splitting the PR would cause in the code itself, just for one extra line in the change log.

Thanks for adding documentation. It doesn't mention that it also applies to Role::Image though. This actually make me question the purpose of this: if I hear "the value of an image", I won't think "text alternative". I'd say that the label property is more appropriate for this role. What was your reasoning for including the image role in this change?

Maybe this is to allow a rich text alternative?

You're right about Role::Image. I fixed that, and also addressed your other suggestion.


Autoplay false not working

When it is loading a video, and  autoplay(false) is setted, nevermind at the end of the loading the video runs anyway.
Do you have some solution?

Change the YouTubeExecCommand.LoadVideo to YouTubeExecCommand.CueVideo in order to obtain the result you are looking for.


build: org scoped, version and description in package.json

Build Request
I remember we had to scope the name of our dapp with the org name in package.json. Probably worth doing it in the portal
{
  "name": "@dfinity/portal",

At the same time would be cool to provide a version number, description etc.

Addressing this in https://github.com/dfinity/portal/pull/2633


Minor text changes

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Collapse the hero text into a single paragraph
Add a CTA to the hero (maybe a Console link?)

Collapse the hero text into a single paragraph
Add a CTA to the hero (maybe a Console link?)


Support host component from another module

Error I'm getting:
	Error: Type EditorTestHostComponent is part of the declarations of 2 modules: AtlasDevToolsModule and DynamicTestModule! Please consider moving EditorTestHostComponent to a higher module that imports AtlasDevToolsModule and DynamicTestModule. You can also create a new NgModule that exports and includes EditorTestHostComponent then import that NgModule in AtlasDevToolsModule and DynamicTestModule.

Is it possible to support my host component coming from another module?

Can you add the full code, please? I can't help you with only this.

Short version:
  let host: SpectatorWithHost<HelloWorldComponent,EditorTestHostComponent>;
  const createHost = createHostComponentFactory({
    component: HelloWorldComponent,
    imports: [AtlasDevToolsModule, RouterModule.forRoot([])],
    providers: [{provide: APP_BASE_HREF, useValue: '/'}],
    host: EditorTestHostComponent
  });
  it('works', () => {
    const host = createHost(view);
  });

I can't post the full code because it's not open source, but the key part is that EditorTestHostComponent is part of AtlasDevToolsModule, which seems to be the issue that Angular is complaining about.
I don't know what the right workaround/solution is, because it seems like your code is assuming that if there's a host component, that host component isn't part of any other module and needs to be included in the dynamic host module's declarations:.
https://github.com/NetanelBasal/spectator/blob/master/src/lib/src/config.ts#L60

You can create a new component that extends this one.

That is a reasonable workaround. Thanks for the help!

You're welcome.


Update fwlink for unsupported workspace setting

The fwlink https://go.microsoft.com/fwlink/?linkid=839878 needs to be update and point to the security explanation https://github.com/Microsoft/vscode-docs/blob/vnext/release-notes/v1_9.md#settings-and-security

// @dbaeumer fyi

fwlink updated to point to https://code.visualstudio.com/docs/customization/userandworkspace#_settings-and-security


npm install on macOS throws error

When running npm install for this repo on macOS, the first run gives the following output:
><EMAIL_ADDRESS>install /Users/ads/src/kompendium/node_modules/jest-haste-map/node_modules/fsevents
> node install.js

internal/modules/cjs/loader.js:883
  throw err;
  ^

Error: Cannot find module 'nan'
Require stack:
- /Users/ads/src/kompendium/node_modules/jest-haste-map/node_modules/fsevents/[eval]
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:880:15)
    at Function.Module._load (internal/modules/cjs/loader.js:725:27)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at [eval]:1:1
    at Script.runInThisContext (vm.js:132:18)
    at Object.runInThisContext (vm.js:309:38)
    at internal/process/execution.js:77:19
    at [eval]-wrapper:6:22
    at evalScript (internal/process/execution.js:76:60) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/Users/ads/src/kompendium/node_modules/jest-haste-map/node_modules/fsevents/[eval]'
  ]
}
gyp: Call to 'node -e "require('nan')"' returned exit status 1 while in binding.gyp. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:351:16)
gyp ERR! stack     at ChildProcess.emit (events.js:315:20)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)
gyp ERR! System Darwin 20.2.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/ads/src/kompendium/node_modules/jest-haste-map/node_modules/fsevents
gyp ERR! node -v v14.15.1
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok

Despite this, the run does finish. After this first run, no files are changed. Running npm install again, the install passes with no errors, but updates the package-lock.json file.

:tada: This issue has been resolved in version 0.7.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 0.7.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Enhancement] PolicyException moves to beta

Description
PEs move to beta from alpha.
Slack discussion
No response

PolicyException is enabled by default https://github.com/kyverno/kyverno/pull/8545.


feat: [Picker] add autoOpen allow open panel auto when the widget is built

In some scenarios, we hope that time related forms will automatically pop up a panel when loaded to save the need for clicking again.
This PR affects DatePicker, TimePicker, and Picker
Pre-launch Checklist

[x] I have updated CHANGELOG.md with my changes
[x] I have run "dart format ." on the project
[x] I have added/updated relevant documentation

Thank you for your reply.
I have some questions that I would like to discuss with you.
Does 'using keys' mean the following:
GlobalKey key = GlobalKey();

DatePicker(
  key: key,
  selected: v,
);

(key.currentState as dynamic).open();

If so, this calling method can indeed achieve the desired effect, but it seems more cumbersome. More encapsulation is needed to achieve parameter level logic control.
Moreover, _DatePickerState is private and can only be cast through "as dynamic" when we call the open method externally, it seems a bit strange.
I am not sure if my understanding is in line with popular thinking.

autoOpen can lead to unexpected behavior by the user. If the widget is ever entirely rebuilt, the panel would open as well. I prefer to leave this kind of control to the developer.

Moreover, _DatePickerState is private and can only be cast through "as dynamic" when we call the open method externally, it seems a bit strange.

Ideally, we'd make the necessary changes to make the picker states public.

autoOpen can lead to unexpected behavior by the user. If the widget is ever entirely rebuilt, the panel would open as well. I prefer to leave this kind of control to the developer.

You' re rignth. I overlooked this point.
From this point of view, it would indeed be better for users to decide for themselves.


chore: update kubebuilder toolchain and test k8sversion

Fixes #N/A
Description
This updates testing versions to K8s v1.28, and removes a stale check for kubebuilder installs when arm64 wasn't supported.
How was this change tested?
make toolchain && make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 80.029%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/disruption/drift.go
2
79.1%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
8347

Relevant Lines:
10430

💛  - Coveralls


FR: Hierarchy Notes doesn't warn when outline includes comments

Is your feature request related to a problem? Please describe.
When I write MOC in Obsidian, I usually want to write some comments before the link.
But if I use Hierarchy Notes, it will warn me. For example:

I think the warning should be removed.
Describe the solution you'd like
I know this warning is from "Specifying Different Up Fields: You can tell Breadcrumbs to override the default up field name, by putting the field name you want before the note".
But I think it can be changed. For example, breadcrumb will only recognize specific grammar before the link.

I tried. The comments does not work in front of the link but works after the link, can you simply put the comment after the link?

I tried. The comments does not work in front of the link but works after the link, can you simply put the comment after the link?

😲Oh, I see.
So this is just a feature request😂

Yeah, as @HananoshikaYomaru says, a comment before the link won't work (BC tries to use it as a field). But a comment after is fine. Does that work for you? I think it might have to be a breaking change to support only a specific syntax infront of the link

Yes, it works.
So if my feature request needs break change, I think it's ok to change my behavior. 😂


Clean up docs style

Hi @yelouafi! Thanks for the amazing work on this library.
This PR contains a big clean-up of the docs. Specifically, it addresses:
Spelling/grammar

Fix various typos and grammatical mistakes
Use slightly more formal language where appropriate
Ensure consistent punctuation

Consistency

Switch to single-line paragraphs (currently, this is all over the place: some lines are capped at 80 chars, some at 100, and some not at all, so I switched to the most widely-used style in the JS community)
Ensure that doc titles are consistent with their README links
Ensure consistent title capitalization
Ensure two-line spacing between paragraphs and code

Example code

Ensure consistent use of while () and if ()
Fix a few syntax errors
Ensure consistent comment spacing
Use sh templates for console commands

Thanks so much for this!
I'll update the gitbook site asap


Doesn't allow PATCH method

Is there a reason not to allow PATCH? Semantically, it's useful in REST.

It appears to be simple to add it. My fork has the changes, if a pull request is welcome.

A pull request would definitely be welcome.