babel-jest does use babel default env ("development")

Do you want to request a feature or report a bug?
I think I found a bug while converting some tests from AVA to Jest. (AVA also handle babel transformation, but jest does do the transform the same way, see below).
What is the current behavior?
I am using the first react example from the website homepage and I am getting an "Unexpected token" on the <a. This is because my babel configuration is not correctly interpreted.
Currently, it seems that babel-jest does not interpret babelrc/env.development configuration by default.
According to babelrc doc, when no BABEL_ENV / NODE_ENV are specified, "development" is used for the "env".
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal repository on GitHub that we can npm install and npm test.
Currently, I don't have any babel configuration by default. I am using webpack 2 & does not need imports to be transformed blahblah so here is my configuration
{
  "env": {
    "development": {
      "presets": [ "latest", "stage-2", "react" ]
    },
    "production": {
      "presets": [ "latest", "stage-2", "react" ]
    },
    "webpack-development": {
      "presets": [
        [ "latest",{"es2015":{"modules": false}} ], "stage-2", "react"
      ],
      "plugins": [ "react-hot-loader/babel" ]
    },
    "webpack-production": {
      "presets": [
        "react-optimize",
        [ "latest",{"es2015":{"modules": false}} ], "stage-2", "react"
      ]
    }
  }
}

As you can see, all my configs are in "env".
What is the expected behavior?
I expect jest to set BABEL_ENV to "development" if NODE_ENV==="" && BABEL_ENV === "".
If I launch jest with NODE_ENV=development it works.
Run Jest again with --debug and provide the full configuration it prints. Please mention your node and npm version and operating system.
Node v6.4.0
npm 3.10.3
ProductName:    Mac OS X
ProductVersion: 10.12
BuildVersion:   16A313a

$ jest --debug --no-cache src/modules/NavigationMenu/__tests__/index.js
jest version = 15.1.1
test framework = jasmine2
config = {
  "rootDir": "/Users/moox/Sync/Development/_EDITED_",
  "name": "-Users-moox-Sync-Development-_clients-mphasis-wyde-WynsureGroupAdmin",
  "setupFiles": [
    "/Users/moox/Sync/Development/_EDITED_/node_modules/babel-polyfill/lib/index.js"
  ],
  "testRunner": "/Users/moox/Sync/Development/_EDITED_/node_modules/jest-jasmine2/build/index.js",
  "scriptPreprocessor": "/Users/moox/Sync/Development/_EDITED_/node_modules/babel-jest/build/index.js",
  "usesBabelJest": true,
  "preprocessorIgnorePatterns": [],
  "automock": false,
  "bail": false,
  "browser": false,
  "cacheDirectory": "/var/folders/fp/xz51yg_945dc86f52ncz3jh00000gn/T/jest",
  "colors": false,
  "coveragePathIgnorePatterns": [
    "/node_modules/"
  ],
  "coverageReporters": [
    "json",
    "text",
    "lcov",
    "clover"
  ],
  "globals": {},
  "haste": {
    "providesModuleNodeModules": []
  },
  "mocksPattern": "__mocks__",
  "moduleDirectories": [
    "node_modules"
  ],
  "moduleFileExtensions": [
    "js",
    "json",
    "node"
  ],
  "moduleNameMapper": {},
  "modulePathIgnorePatterns": [],
  "noStackTrace": false,
  "notify": false,
  "preset": null,
  "resetModules": false,
  "testEnvironment": "jest-environment-jsdom",
  "testPathDirs": [
    "/Users/moox/Sync/Development/_EDITED_"
  ],
  "testPathIgnorePatterns": [
    "/node_modules/"
  ],
  "testRegex": "(/__tests__/.*|\\.(test|spec))\\.js$",
  "testURL": "about:blank",
  "timers": "real",
  "useStderr": false,
  "verbose": null,
  "watch": false,
  "cache": false,
  "watchman": true,
  "testcheckOptions": {
    "times": 100,
    "maxSize": 200
  }
}
 FAIL  src/modules/NavigationMenu/__tests__/index.js
  ● Test suite failed to run

    SyntaxError: /Users/moox/Sync/Development/_EDITED_/src/modules/NavigationMenu/__tests__/index.js: Unexpected token (6:4)
      4 | it("renders correctly", () => {
      5 |   const tree = renderer.create(
    > 6 |     { "Facebook" }
        |     ^
      7 |   ).toJSON()
      8 |   expect(tree).toMatchSnapshot()
      9 | })

      at Parser.pp.raise (node_modules/babylon/lib/parser/location.js:22:13)
      at Parser.pp.unexpected (node_modules/babylon/lib/parser/util.js:91:8)
      at Parser.pp.parseExprAtom (node_modules/babylon/lib/parser/expression.js:520:12)
      at Parser.pp.parseExprSubscripts (node_modules/babylon/lib/parser/expression.js:278:19)
      at Parser.pp.parseMaybeUnary (node_modules/babylon/lib/parser/expression.js:258:19)
      at Parser.pp.parseExprOps (node_modules/babylon/lib/parser/expression.js:188:19)
      at Parser.pp.parseMaybeConditional (node_modules/babylon/lib/parser/expression.js:165:19)
      at Parser.pp.parseMaybeAssign (node_modules/babylon/lib/parser/expression.js:128:19)
      at Parser.pp.parseExprListItem (node_modules/babylon/lib/parser/expression.js:1042:16)
      at Parser.pp.parseCallExpressionArguments (node_modules/babylon/lib/parser/expression.js:354:20)

Test Summary
 › Ran all tests matching "src/modules/NavigationMenu/__tests__/index.js".
 › 1 test suite failed, 0 tests passed (0 total in 1 test suite, run time 4.168s)

Here is the working line
NODE_ENV=development jest --debug --no-cache src/modules/NavigationMenu/__tests__/index.js
jest version = 15.1.1
test framework = jasmine2
config = {
  "rootDir": "/Users/moox/Sync/Development/_EDITED_",
  "name": "-Users-moox-Sync-Development-_clients-mphasis-wyde-WynsureGroupAdmin",
  "setupFiles": [
    "/Users/moox/Sync/Development/_EDITED_/node_modules/babel-polyfill/lib/index.js"
  ],
  "testRunner": "/Users/moox/Sync/Development/_EDITED_/node_modules/jest-jasmine2/build/index.js",
  "scriptPreprocessor": "/Users/moox/Sync/Development/_EDITED_/node_modules/babel-jest/build/index.js",
  "usesBabelJest": true,
  "preprocessorIgnorePatterns": [],
  "automock": false,
  "bail": false,
  "browser": false,
  "cacheDirectory": "/var/folders/fp/xz51yg_945dc86f52ncz3jh00000gn/T/jest",
  "colors": false,
  "coveragePathIgnorePatterns": [
    "/node_modules/"
  ],
  "coverageReporters": [
    "json",
    "text",
    "lcov",
    "clover"
  ],
  "globals": {},
  "haste": {
    "providesModuleNodeModules": []
  },
  "mocksPattern": "__mocks__",
  "moduleDirectories": [
    "node_modules"
  ],
  "moduleFileExtensions": [
    "js",
    "json",
    "node"
  ],
  "moduleNameMapper": {},
  "modulePathIgnorePatterns": [],
  "noStackTrace": false,
  "notify": false,
  "preset": null,
  "resetModules": false,
  "testEnvironment": "jest-environment-jsdom",
  "testPathDirs": [
    "/Users/moox/Sync/Development/_EDITED_"
  ],
  "testPathIgnorePatterns": [
    "/node_modules/"
  ],
  "testRegex": "(/__tests__/.*|\\.(test|spec))\\.js$",
  "testURL": "about:blank",
  "timers": "real",
  "useStderr": false,
  "verbose": null,
  "watch": false,
  "cache": false,
  "watchman": true,
  "testcheckOptions": {
    "times": 100,
    "maxSize": 200
  }
}
 PASS  src/modules/NavigationMenu/__tests__/index.js
  ✓ renders correctly (21ms)

Test Summary
 › Ran all tests matching "src/modules/NavigationMenu/__tests__/index.js".
 › 1 test passed (1 total in 1 test suite, 1 snapshot, run time 7.906s)

I finished by understanding that "test" is used. But this can be confusing when you don't know. Maybe it should just be added into the doc?

Yeah you are right, we should add documentation for this. Jest sets the NODE_ENV to test if it isn't provided and otherwise let's you use a custom override. We don't use BABEL_ENV at all. I'm gonna close as wontfix but I'm happy to accept a PR for documentation.

@cpojer I think babel-core's transform should handle reading BABEL_ENV but you need to pass along env from the user's configuration into it, which seems like it may not be happening here? https://github.com/facebook/jest/blob/ab6f4c3001a2cd917cc4df95e42dfc72f23c5a00/packages/babel-jest/src/index.js#L70

References:
http://facebook.github.io/jest/docs/getting-started.html#babel-integration
http://facebook.github.io/jest/docs/configuration.html#transform-object-string-string
https://www.npmjs.com/package/babel-core#options


Experimenting with Jest's snapshot feature

The bug I introduced described in #1192 has been annoying me. Essentially, I added a mandatory prop to a component. I didn't realize it was being used by another page, so I didn't change it. Tests were clear because that page isn't well tested.
As Kevin pointed out, perhaps Flow or Typescript would alleviate my craving of some sort of type safety that's compiler enforced. We won't be using Flow or Typescript anytime soon.
So I started to think about how we can ensure that each of the React components are tested.
I'm thinking using Jest snapshots for very basic testing of React components could go a long way. Essentially, Jest takes a snapshot the first time the tests are run. Those snapshots are committed alongside jest test js.
So I wanted to test a couple of components. Two with existing tests SectionPage and FlexibleRoster and one without SectionHeader.
Would love your thoughts @kevinrobinson and @alexsoble. Note I had to downgrade react-test-renderer to the version of react we're using. Doesn't look like that was used anywhere else.

@jhilde This seems great to explore!  👍
I'm not sure how this improves the case you ran into though, as compared to just adding smoke tests like this.  Missing props is more about just having a test at all rather than depth of coverage, so adding one-line tests seem the best immediate bang for the buck.
One other downside for this might be fatigue around snapshots changing on every change, or that we're not actually good at catching bugs by reviewing snapshot files.  That being said, I'd vote to just try it out and see how we all like it.

Also, to clarify https://github.com/studentinsights/studentinsights/issues/1192#issuecomment-339755769 I think it'd be totally reasonable to experiment with weak mode Flow and see how we all find it (similar to what you're proposing here).

@kevinrobinson Agree that a simple test can do the job. The challenge is more in creating props (providing fixtures for common things will work well).
In looking at our tests, the majority are asking, does this look right. Some count the number of columns, some search for text, etc. It seems like automated snapshots would actually do most of that testing, with less code, and more completely. Agreed, that we update a core React Component, several snapshots could change. But with auto recreation of snapshots, we can then look at the diffs to make sure the output changed in the expected way.
Mostly it was just a, I have a few minutes, what should I play with. And also excited we're on more modern tooling now. ;-)

👍 For adding smoke tests, here are some ways fixtures are used in other places in the project: local to the test, shared across a product feature and shared across the app.  If you need to make new ones that are complex, grabbing data from the console in development mode is a good way to re-use the Ruby demo data generators.
For snapshots, let's just try it!  It'll definitely be more complete in some sense, but it relies on us humans to look at those snapshots and diffs to know if the feature is working correctly. :)  In this PR here, does the roster choose the initial sort column correctly?  For me, it's harder to see that in the snapshot than in:
expect($(el).find('th:first').hasClass('sort-down')).toEqual(true);

and not much more pain to write.  For other components for things like visualizations, where TDD makes no sense and you want to verify you don't regress visual layout or styling, snapshots seem like the perfect test API.  I'm optimistic that snapshots would be helpful enough to be worth trying out!
But if we want to improve catching errors for missing props, we should just do the plain boring thing and add some smoke tests to components that don't have them :)

Moving over from email.  I'd vote to experiment and try using snapshots like you're doing here, I think I've said that in all of my comments :)

It seems like automated snapshots would actually do most of that testing, with less code, and more completely

I agree with this!  Certainly if we added snapshots for all components, those tests would be quite complete and fail anytime the product changes.  I'm not sure that this helps our users, or even us. :)
The cost that snapshots introduces is higher false-positive test failures, since snapshot tests break on any product change.  And that means test fatigue and friction for every developer on the team for every product improvement they make.  In a weird way, that approach would encourage us to make refactoring improvements that were invisible to users, but discourage us from making product improvements.  I've certainly worked on systems where tests functioned this way.  If you have other perspectives you can share that'd be helpful for me in understanding!
To re-iterate, I'd vote to experiment and try using snapshots like you're doing here, and to experiment with them on new things we build.  This seems great!  And I'd be excited about pull requests that added smoke tests to all components proactively.  But I'd vote against spending time to add snapshot tests to every component pro-actively.

I'm going to close this without merging...and perhaps do a bit more with it. This was originally meant as an experiment.


Finish implementation of trust domain alias

Describe the feature request
Approved design doc: Trust Domain Configuration
This bug is to track the remaining work for this feature.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Existing PR

https://github.com/istio/api/pull/1561, add the authn filter config.
Unblock the istio/istio deps update by removing the client-go reference of mixer. https://github.com/istio/client-go/pull/210


Missing OPENAI_API_KEY raises errors

When no OPENAI_API_KEY is set in the settings dialog chat does not work. Front end should fail more gracefully and suggest the appropriate fix OR settings should persist from .env file to front end.

Question, will end users need to provide an API key?

Thanks for the issue!

Front end should fail more gracefully and suggest the appropriate fix

We have https://github.com/refstudio/refstudio/issues/305 for more graceful error messages when the key is not set.

settings should persist from .env file to front end

If the key is set in your .env file, it should be making it into the frontend. That would be a regression if not. We will try to reproduce on our side.

Question, will end users need to provide an API key?

Yes, for the OSS version, for now. We may eventually ship with local models. For the hosted version, it will be optional.


upgrade to php 8.1

php 7.4 is eol

Implemented in 2023-02

This also includes:

[ ] Migrating the main application to the latest version of the framework
[ ] Migrating API v2 to the latest version of the framework
[ ] Migrating API v1 to the latest version of the framework

Especially the last task requires a lot of time, because this part has not been updated for a long time.

Looked at the API endpoints for v1 and v2 and did not find anything suspicious.
Everything seems to work smoothly.
Green light from my side.

@patrickakk
The website seems to work fine.
I also clicked through the API2 page. Noticed that this link https://app.swaggerhub.com/apis-docs/hashmich/DHCR-API/1.2 took a bit of time to load.
Green light from me as well, thank you for all the work!

@IvdL22  Thanks for the notification. As info: Swaggerhub is an external part, which is hosted somewhere else.

Implemented in 2023-02


CocoaPods support

Can this be put on CocoaPods?

I guess it can. I just do not feel for CocoaPods anymore, I prefer https://github.com/Carthage/Carthage. If at some point I'd spent time for packaging this, I would go for Carthage.
Do you have problems using the code or is this about dependency management?


[Feature] Add debounce props

Hi,
It would be great to add a debounce props to avoid multiple http request when we typing :).

Hi @florentsorel Nice Idea! :grin: :+1:

I was going to check the docs for this. Debounce is really handy to avoid spamming requests to the api :)


improve screen reader context and navigation of Cell outputs

Tabbing to a cell output is missing context about which cell it is for.
The suggestion from @jooyoungseo is adding markdown headers for each cell output to describe what the output is for and provide a clearer boundary around the output.
This would also allow screen readers to iterate over outputs since they can navigate directly to markdown headers.

some complications with the output webview

focusing the output will first read the previously focused output line, then read the currently focused one right after
the order of the output cells in the DOM does not match the order on the screen. I think they are just appended on when executing, but you could also rearrange the cells after executing.

the result is that browsing in the output webview seems to jump around to random cells
this sounds pretty hard to fix to actually keep them consistent, but I imagine it would be pretty confusing for the user

The cell output doesn't have context on which cell it's for, which is why that PR just has a generic 'begin cell output' label

probably easier to fix, but I haven't fully thought it through

But, because there is currently a potentially helpful initiative for a new tool from @meganrogge that would go in an entirely different direction, we will hold off on attempting further fixes for this until we know more.

Just chiming in to voice my support for this ticket, especially about cell output labeling and boundaries.
One use case that makes notebooks hard to use for me is that I can't quickly arrow down to the bottom of a particular cell output's stack trace error message, since there are no boundaries between cell outputs..

thanks for the feedback @dennischenfeng - we should have some good improvements for cell output from https://github.com/microsoft/vscode/issues/183208, in insiders now and in the next stable release next week.
I'm going to close this as a duplicate of that one, but feel free to leave feedback there if you try it out, or open a new issue.


Run unit tests on Ubuntu 24.04 in CI

Relates to #264
Summary
This adds a new entry to the matrix for unit tests in CI to run the tests on Ubuntu 24.04.
This was omitted in #264 because kcov wasn't available on Ubuntu 24.04.

Asked about it over at https://answers.launchpad.net/ubuntu/+source/kcov/+question/818841


Add capability to setup lvm thin pool and pass it to docker

Right now we setup a data lvm and metadata lvm and pass it to docker and docker in turn sets up thin pool using libdevmapper library. Now lvm tools support creation of thin pool and one can create a
thin pool using lvm and pass it to docker.
Advantage of this approach is that lvm can do all the maintenace of pool like automatic resizing and
doing metadata checks when pool is activated etc.
So this should turn out to be a better approach.

This was merged.


Koncepted #5 UX/Design - Space page improvements

https://beneficial-lillipilli-60b.notion.site/Koncepted-5-UX-Design-Space-page-improvements-bb34791bf6904c7282d749670b336eed

I have some thoughts on the current design of proposal status, vote status in the space page. @mktcode @samuveth
I'm trying to sort out the most critical elements a user should care on the proposal.
On active proposals:
-How much quorum has been reached?   The author may allocate different levels of action to the current quorum.
-The remaining time
On closed proposals:
What's the winning result?
On both:
If a user has voted on a proposal?
Based on above ideas, I would like to act a bit old-school and bring back some style we used to employ. In order to:

Highlight the Active/Closed status.   -This kind of style is more eye-catching, and also aligns with the design inside of the proposal.
By rearranging the position of proposal main status, we can remove the extra line in the closed proposal which doesn't make much sense.

Add quorum (if there is) status in active proposal status, which matters to authors. (If we were to enforce the proposal invalid on its failure of lack of quorum in future, this will be an important indicator)
Move "voted" status to the bottom of the proposal text line. It's a good place, and makes the UI neat.

Active proposal:

Closed proposal:

Hey @hjjlxm, this is great and thanks again! I wasn't happy with the proposal status visibility and I think your solution here solves this pretty elegantly.
We are currently looking for people for UI/UX would you be interested in contributing more frequently?

You're welcome @samuveth I'm glad if it helps.
I proposed out of a hypothesis which can better catch users' attention for ongoing votes. In my mind, the priority of elements should be: Proposal status ＞ Vote status  So that's why the layout. (Cuz in a space page level, proposal is more important than votes; and the vote info will be elaborated in a proposal page level)
As regards the interface, I wanted to express it in a simple but self-explanatory way, while differentiating itself from other elements in the same interface.
As an alternative for the voted icon,  to make it easy to scan with your eyes, I also chose a plan to use a color labeling, also it differentiated itself from the "core" label of the author. Wdyt?
black-font-with-green-bkg:

halfgrey font-with-green-bkg:

bold-black-uppercsed-font--with-green-bkg:

bold-green-uppercased-without-bkg:

green-without-bkg:

@hjjlxm we have now implemented it like this. What do you think?

Thanks for trying this! I've checked the implementation, and will comment on the voted design. For the "twitter like" vote bar, Idk if it's a little visually crowded cuz of too many texts in a single container, maybe I need time to get used to...
Basically, I think the design is good if the proposal title is concise, but problem comes with the long title in many spaces which we won't find in the pistachioDAO or above samples.
My "bad words" on it...

It's not very prominent amongst the proposal body and title, esp. the "Voted" text is similar to its surroundings and blends itself in, the problem is, when you want to quickly view the page, you have to summon more energy to notice the small icon.

The design may be better if it's formed as a whole unit itself. Currently the icon and the text is "separated", under some circumstances the "voted" text seems redundant which in a sense is not as neat as a single element.

If the title is long, the voted indicator is pushed to the end of the container which may cause visual crowding and make it hard to distinguish.

After I submitted my first comment, I've been thinking if there's a better design to let people better notice it with less efforts. So I tried to combine the icon with the text into one. Similar to github labeling, but an inferior touch than active/closed status by the same time still good enough to make people easily notice it.

For another solution if we may take, it's to put the indicator at the head of the title, looks like this:

To sum, I insist on my first re-position of the elements and suggest an improved look of voted indicator.


Support for WFRP4 System

This would be awesome in the wfrp4 system. specially for creating macros and similar. Can you please add support for it?

I have no experience at all with that system, so it’s difficult for me to provide support for it. I am however completely open to somebody creating a PR to add the support.


Enable DCO2

Refer to this ticket: https://github.com/BCDevOps/devops-requests/issues/1715
Remaining repositories require a PR to be able to trigger DCO2 check and enable it as mandatory.

The switch to DCO-2 is complete for this repo.
All epos in the list has been updated other than as noted here.

Assigning to @amanji to trigger the DCO-2 check in aries-oca-explorer.

FYI — I don’t have permissions on that repo to make the change. I can see the Settings, but only the “General - Access / Code and Automation” sections with minimal controls.  No way to invite other admins.  Looks like you have to do it @amanji .

Resolved by: https://github.com/bcgov/aries-oca-explorer/pull/31

All done


adding in props for "emptyTreeComponentProps" and "emptyTreeComponent"; making test suite more modular

This PR includes the following:

adds in props for emptyTreeComponentProps and emptyTreeComponent.
The emptyTreeComponent allows you to specify a component to render when tree is empty, or no nodes in the tree match the filter. For example, if no entries match the filter, you can have the component like this render:

Breaks the tests from a single index.js file into more modular files based on specific functionality being tested. This will make managing the new tests/adding new tests much easier

Adds a new filter prop that allows you to specify the filter criteria. Previously, the filter would automatically filter with the node name === to the search term input. I refactored it such that filter is a function that takes the searchTerm and the node as input. If the function returns true, the node passes the filter and is displayed, if the node returns false, it is not shown.

Various README cleanups

@tiansijie made the update

LGTM!


Notification widget volume bar sets wrong value

from google play user
media volume max is 15

set media volume to 10 from app
press the last section in notification widget
observe

expected volume is 15
actual volume is 14
reproducible on google pixel

fixed in https://github.com/punksta/volume_control_android/commit/c851c8db3514a6b922a0be9242edf082a4a41403

:+1:
Thank you.


Typo in docs for CPU limit

Just got bitten by this. Here is a PR for fixing a typo in the docs. The variable name in pkg/backends/jmeter/config.go  is named JMETER_MASTER_CPU_LIMITS, not JMETER_MASTER_CPU_LIMIT.

Codecov Report

Merging #296 (0ddbcda) into master (235c889) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##           master     #296   +/-   ##
=======================================
  Coverage   56.04%   56.04%
=======================================
  Files          37       37
  Lines        4116     4116
=======================================
  Hits         2307     2307
  Misses       1686     1686
  Partials      123      123

Thank you for the contribution @thiagoarrais :+1:


Added cmd_remove (removes a song from the queue based on index)

Remove command allows a user to remove a song from the "to-be-played" queue based on the song's index. Plan to add "remove song by name" feature in the future.

https://github.com/CArchetype/MusicBot/commit/ddb842794fcc20003ac24b380bed1b8ef418553c

I figured this functionality had already been done by someone. Just curious, why hasn't that remove command made it to the release version of MusicBot?


I tried to install the latest version of bloodhound/azurehound but I always ended up with the wrong version of the package installed. I figured I could perform a version query manually to figure out to what tag specifically latest did resolve:
$ go list  -m github.com/bloodhoundad/azurehound@latest

What did you see happen?
The command above returned the following output:
github.com/bloodhoundad/azurehound v1.2.4

What did you expect to see?
The repository has many tags higher than 1.2.4, the latest one being 2.1.8:
$ git ls-remote --tags origin
71f631448e47c50d3e41e898ae03a73597c0bf03	refs/tags/rolling
c31d34892044c782defcd07751ae2c324879d5c5	refs/tags/v1.1.1
27f60cc277db2884e4f64fbbd1a4c4cc37b35350	refs/tags/v1.1.2
820f7948d2b15fffe5c97ea129f6edfe18af4bbc	refs/tags/v1.1.3
dcbf0fc6a88ec0af9e7d91f4f33e84cbe6a6faba	refs/tags/v1.2.0
a6ff55b9561f881fd4fbc74770267cf4876f6b42	refs/tags/v1.2.1
33bb4f0dab469493c8927f164777d614a3675acb	refs/tags/v1.2.2
a942107766d6ad2b2f9954ac9087418245c0e754	refs/tags/v1.2.3
e6f62d1c0ddd4c11bb9531ffcce69073327b599b	refs/tags/v1.2.4
b3a75f370381ed9dc3c2930e206cdd112d3e7622	refs/tags/v1.2.4^{}
60e31176f8b1bb0fc80c44bede30710b4ebed994	refs/tags/v2.0.0
2e39b1ca6d4913a570b9da30b141c917921163f8	refs/tags/v2.0.1
2379ab55f4a5c12ed2cae977ab33a2d59f5a0192	refs/tags/v2.0.2
18e629ecf4cf63f4fea815d4307bb8659e381a9d	refs/tags/v2.0.3
74ccf0fbfeba132f1f10ec2ae3fd535f99305bc2	refs/tags/v2.0.4
8eda432a103d7205937bb1c146a0579aab8106cc	refs/tags/v2.0.5
d2ff1b66bcd343e615255f37ca08fcd018b3b6a4	refs/tags/v2.1.0
c4637f89fba73dbb5d725a695bcdda88efbbe701	refs/tags/v2.1.1
a4a6de574091e4d17fbbaa7479400846a2bf1a3c	refs/tags/v2.1.2
1dd97b0e20c42ebf029c99d290472a6c4fc4d867	refs/tags/v2.1.3
2ffd8e9c749880c602464ac90add8511a934aab1	refs/tags/v2.1.4
06a6d182e2e061df871ebbbd2865f132bbeb633a	refs/tags/v2.1.5
faeb09f8cb6d397a6f97a8f4c61da28441f30c35	refs/tags/v2.1.6
937a90a18b43e32d5e70e4bc9c6009e9ecd78e80	refs/tags/v2.1.7
6889fa8e819e6f39930ebb58bd00e29506e20822	refs/tags/v2.1.8

I did noticed the pair of braces around 1.2.4 which might have something to do with that but according to https://go.dev/ref/mod#version-queries, latest is supposed to selected the highest release, which should be 2.1.8, if I didn't miss anything obvious.

Closing as working as intended.
1.2.4 is the highest v1 version.
Go requires major version suffixes for v2+.
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions

Try go list -m github.com/bloodhoundad/azurehound/v2@latest.

@seankhliao @timothy-king  thank you for the quick feedback. From my pair of shoes it was a bug, that’s why I opened the issues. TIL


NIST 2.0

Is there a plan to update the tool to NIST 2.0?

Also looking forward to get NIST 2.0. Tried to use the custom module (CSF 2.0), it works, and I can export to CSETW file. However, I am not able to import back. Tried different custom modules and it was not possible.

Bump

OK we are looking for 2.0 as well.

Also looking forward to get NIST 2.0. Tried to use the custom module (CSF 2.0), it works, and I can export to CSETW file. However, I am not able to import back. Tried different custom modules and it was not possible.

Where did you get the custom module?

The custom module assessment failing at uploading is being worked on and intended to be release in the next version

Also looking forward to get NIST 2.0. Tried to use the custom module (CSF 2.0), it works, and I can export to CSETW file. However, I am not able to import back. Tried different custom modules and it was not possible.

Where did you get the custom module?

You can create it under the Tools dropdown

Also looking forward to get NIST 2.0. Tried to use the custom module (CSF 2.0), it works, and I can export to CSETW file. However, I am not able to import back. Tried different custom modules and it was not possible.

Where did you get the custom module?

Go to "Import Modules" and you will find the "Cybersecurity Framework 2.0" module. Just change the "shortname" property because it needs to be unique.

Also looking forward to get NIST 2.0. Tried to use the custom module (CSF 2.0), it works, and I can export to CSETW file. However, I am not able to import back. Tried different custom modules and it was not possible.

Where did you get the custom module?

Go to "Import Modules" and you will find the "Cybersecurity Framework 2.0" module. Just change the "shortname" property because it needs to be unique.

Excellent that works!  Thanks

I would love to have an update to include NIST CSF 2.0 too!

ID.RA-10 is missing in the module.

We have CSF 2.0 completed and should be releasing a new CSET version in the next few weeks.


Missing b2vec2 from b2_math.h

I'm trying to build for C# binding.
For some reason b2vec2 is not generating.  When I manually add b2_math.h to box2d.i I get a list of errors:
b2_math.h(56) : Warning 503: Can't wrap 'operator -' unless renamed to a valid identifier.
b2_math.h(62) : Warning 503: Can't wrap 'operator ()' unless renamed to a valid identifier.
b2_math.h(68) : Warning 503: Can't wrap 'operator ()' unless renamed to a valid identifier.
b2_math.h(74) : Warning 503: Can't wrap 'operator +=' unless renamed to a valid identifier.
b2_math.h(80) : Warning 503: Can't wrap 'operator -=' unless renamed to a valid identifier.
b2_math.h(86) : Warning 503: Can't wrap 'operator *=' unless renamed to a valid identifier.
b2_math.h(147) : Warning 503: Can't wrap 'operator -' unless renamed to a valid identifier.
b2_math.h(153) : Warning 503: Can't wrap 'operator +=' unless renamed to a valid identifier.
b2_math.h(159) : Warning 503: Can't wrap 'operator -=' unless renamed to a valid identifier.
b2_math.h(165) : Warning 503: Can't wrap 'operator *=' unless renamed to a valid identifier.
b2_math.h(438) : Warning 503: Can't wrap 'operator +' unless renamed to a valid identifier.
b2_math.h(444) : Warning 503: Can't wrap 'operator -' unless renamed to a valid identifier.
b2_math.h(449) : Warning 503: Can't wrap 'operator *' unless renamed to a valid identifier.
b2_math.h(454) : Warning 503: Can't wrap 'operator ==' unless renamed to a valid identifier.
b2_math.h(459) : Warning 503: Can't wrap 'operator !=' unless renamed to a valid identifier.
b2_math.h(476) : Warning 503: Can't wrap 'operator *' unless renamed to a valid identifier.
b2_math.h(482) : Warning 503: Can't wrap 'operator +' unless renamed to a valid identifier.
b2_math.h(488) : Warning 503: Can't wrap 'operator -' unless renamed to a valid identifier.
b2_math.h(505) : Warning 503: Can't wrap 'operator +' unless renamed to a valid identifier.

Thank you for reporting the issue. Sadly I dont have spare time for working on the issue right away ... However here are some steps I am thinking of, for solving the problem:

The box2d.i is simply a copy-paste from the box2d.h. However there are indirect #includes pointing to some basic utilities unlisted in the file, and as a result SWIG does not generate bindings on these basic utilities, which makes the bindings actually unusable.

For quick test-outs, a quick way to force generate bindings for indirect #includes is to use -includeall option when calling SWIG. However doing so will parse and generate bindings for indirectly #included std headers as well, so this method is not always favorable...
Eventually we have to add the indirectly #included files that includes public structures or defines we wish to generate bindings of into the list in the box2d.i; I quickly skimmed through the folder and made a list of files we might need to add (cant guarantee to be 100% accurate)#include "b2_common.h"     // version numbers, contant defines, etc.
#include "b2_types.h"   // public base type aliases
#include "b2_math.h"    // various math structures

#include "b2_block_allocator.h"    // class B2_API b2BlockAllocator
#include "b2_stack_allocator.h"  // class B2_API b2StackAllocator

#include "b2_shape.h"   // class B2_API b2Shape, etc.
#include "b2_distance.h"  // struct B2_API b2DistanceProxy and various structures
#include "b2_collision.h"     // struct B2_API b2Manifold, struct B2_API b2WorldManifold, etc.
#include "b2_joint.h"    // class B2_API b2Joint

#include "b2_time_of_impact.h"   // struct B2_API b2TOIInput, struct B2_API b2TOIOutput, etc.
#include "b2_contact_manager.h"   // class B2_API b2ContactManager

#include "b2_rope.h"    // class B2_API b2Rope, etc.

The generic guide for binding overload operators can be seen from this section from the SWIG 4.0 doc; when it comes to C#, this example from the test suite of the SWIG source shows that extra glueing code is needed for the bindings to work.

In practice we might need to rename all operators beforehand (like this), and prepare glueing code for each target language... If there are too many to glue, we might have no choice but only glue most frequently used ones and document them as aliases to the renamed "normal" operators.

If you have any progress on your C# build, success or failure or plain discussion, dont hesitate to share it here and I will see if there is anything I can help with. A pull request would be even nicer ^-^

I'm on a windows environment, is there an alternative to make for the build?  I've just been running swig -c++ -csharp box2d.i manually.

I got it working.
  Then copying the box2d.dll and all *.cs files to my test project root—with a copy command to include the box2d.dll in my bin/Debug/net6.0 folder. I was able to get this dotnet core 6 program running:
// Program.cs

namespace Program
{
    class Program
    {
        static void Main(string[] args)
        {
            b2World world = new b2World(new b2Vec2(1,1));

            var bDef = new b2BodyDef();
            bDef.type = b2BodyType.b2_dynamicBody;
            var body = world.CreateBody(bDef);
            bDef.position.Set(0, 0);
            var bShape = new b2PolygonShape();
            bShape.SetAsBox(10, 10);

            var fDef = new b2FixtureDef();
            fDef.shape = bShape;
            fDef.density = 1.0f;
            fDef.friction = 0.3f;

            var fixture = body.CreateFixture(fDef);

            const float timeStep = 1.0f / 60.0f;
            const int velocityIterations = 6;
            const int positionIterations = 2;

            for (int i = 0; i < 1000; i++)
            {
                world.Step(timeStep, velocityIterations, positionIterations);
                var y = body.GetPosition().y;
                Console.WriteLine(y);
            }

            Console.WriteLine("Hello World...");

            return;
        }
    }
}

And the last few lines of the output:
...
135.98746
136.26245
136.53773
136.8133
137.08913
137.36523
137.64162
137.91829
138.19524
138.47246
138.74995
139.02773
Hello World...
PS C:\Projects\box2dc#2>

At this moment, I'm not sure how far I want to take this.  While performance isn't a major concern, I just don't know enough about SWIG and how it plays with the garbage collector.
Anyway, this is my CMakeLists.txt file if you're interested:
project(SWIGBOX2DWRAPPER)

cmake_minimum_required(VERSION 3.24)

# Add main.cpp file of project root directory as source file
set(SOURCE_FILES box2d_wrap.cxx)

include_directories(${CMAKE_SOURCE_DIR}/box2d/include/box2d)

# Add executable target with source files listed in SOURCE_FILES variable
add_library(swigbox2d_wrapper SHARED ${SOURCE_FILES})
target_link_libraries(swigbox2d_wrapper ${CMAKE_SOURCE_DIR}/box2d/build/bin/Debug/box2d.lib)

(it creates swigbox2d_wrapper.dll that I renamed to box2d.dll manually)


Validate if user is already in database while creating it

Firebase auth validates duplicated emails but we can improve it while recording on database


Czech translation

Hello @bitwalker,
I found some mistakes in czech translation. If you want, I prepare PR.

I just merged a PR that applied some fixes, but if you don't mind giving it a second look, that'd be awesome :)

PR https://github.com/bitwalker/timex/pull/633


Subject: Request for Explanation of Styled Evaluation Scores in Table 1

Hi,
Thank you for your great work! I have a question regarding the Styled Evaluation column in Table 1 (e.g., 89.47 for IV-S in Ours (One-DM)).
Could you please explain the procedure used to calculate the scores in this column? Specifically, I would like to understand:
What steps are followed to compute these scores?
What metrics or methods are used in the calculation?
Are there any specific experimental settings or evaluation criteria applied to compute these values?

Hello~Thanks for your attention！We use FID to measure the style performance of each method under four scenarios (i.e., IV-S, IV-U, OOV-S, OOV-U). More details are provided in Styled Handwritten Text Generation of Section 4.2

Thank you


functional bug

just got an error when we were trying to toggle between drugs:
⛑
Failed component prop type: Invalid component prop figure key props supplied to Graph. Bad object: { "props": { "children": "No data selected" }, "type": "Div", "namespace": "dash_html_components" } Valid keys: [ "data", "layout", "frames" ]
4:48:22 PM
Failed component prop type: Invalid component prop figure key props supplied to Graph.
Bad object: {
"props": {
"children": "No data selected"
},
"type": "Div",
"namespace": "dash_html_components"
}
Valid keys: [
"data",
"layout",
"frames"
]

WIP

returns 'no data'


Bump faraday version to latest

Bumping required faraday version to v1.0.0 from v0.7.6. This requires dropping support for Ruby 2.2 - but fixes Ruby 2.7. For details see:
https://github.com/lostisland/faraday/blob/master/CHANGELOG.md#v10

Any updates on a release?

We will do a release next week, after merging: #952

@HazAT Can you confirm the release day?
I got problems with the old faraday version.

Released 3.0.0 just now.


Type generation skipped for 'Microsoft.Storage/storageAccounts/tableServices/tables@2019-06-01'

https://github.com/Azure/bicep-types-az/blob/b609d7dbb043470bc9f16af5743f0454dd74124f/generated/storage/log.out#L12
Reported under https://github.com/Azure/bicep/issues/784#issuecomment-773181312

This looks to have been corrected before #454. Types for Microsoft.Storage/storageAccounts/tableServices/tables are being generated, including in the API versions that raised the warning quoted above (2019-06-01 and 2020-08-01-preview).


Use proper relationship to present cluster information

When a vm is not running there is no relationship between a vm and a host and as a result cluster information is not presented in vm details. In Rhev this relationship is only available when a vm is running on specific host.
This PR fixes:
https://bugzilla.redhat.com/1373826

@miq-bot assign @blomquisg

Looks good to me.

@dclarizio I'm worried this would break other providers.  Do you have any insight there?

@jameswnl is gonna run this against vmware to make sure this lookup doesn't break against another infra provider.

LGTM and the test on vmware is fine too.

@blomquisg do you think it is good enough check to merge this PR?


Support for Turbolinks

Another gem: react-rails implements react-ujs:
https://github.com/reactjs/react-rails/tree/master/lib/assets/javascripts
maybe we could steal it to provide the same experience for turbolinks?

This is definitely something we need to support. Do you think We should support Turbolinks < 5 ?


Index Autoconfiguration, overcoming filedescriptor limitations

This change implements auto-configuration of index settings, including configuring the maximum number of segments which is directly connected to the number of file descriptors. If not externally given it tries to increase the limit (usually from 1024 to 1M). In case of running low on file descriptors index operations are automatically throttled so that concurrent compaction can catch up and reduce open descriptors.
Also changes the queue implementation - after benchmarking - to use condition variables instead of a spin-lock if the queue runs full or empty. This improves indexing performance and also fixes a problem in the spin-lock implementation that could cause cpu hogging.
fixes #60

Coverage decreased (-0.1%) to 93.448% when pulling 8e3fd936b309eed5545af0c85b5d03c135bbb99e on hendrikmuhs:filedescriptor-limits into 4e84ab4c6fdf54833c33a7cdb034b2a662a731eb on KeyviDev:master.


Add prefer-template rule

PR checklist

[X] Addresses an existing issue: #772
[X] New feature, bugfix, or enhancement

[X] Includes tests

[X] Documentation update

Overview of change:
Added the prefer-template rule, which warns on string literal concatenation.
CHANGELOG.md entry:
[new-rule] prefer-template

Test failures fixed by #2234.

IMO it's always better to use a template (except for simple cases like x + "!").
const str = "a" +
	someVeryLongFunction(someVeryLongArgument()) +
	"b";

should be:
const x = someVeryLongFunction(someVeryLongArgument());
const str = `a${x}b`;

If you like complex nested expressions, you can still do this:
const str = `a${
	someVeryLongFunction(someVeryLongArgument())
}b`;

You're absolutely right for the cases you mentioned.
I meant this:
const str = "I'm a really really really long text " +
            "that needs to be split into several lines " +
            "to keep it readable. "+
            "But I shall not contain any line breaks. " +
            "How would you write this as a template string?";

Actually, it already would never warn for concatenation of string literal + string literal. Added a warning to use a template literal if the strings did contain newlines.

@andy-hanson thanks!

@ajafff not sure which is cleaner, but in your example you can always do
const str = `I'm a really really really long text\
that needs to be split into several lines\
to keep it readable.\
But I shall not contain any line breaks.\
How would you write this as a template string?"`;


Switch input method so slowly

When I switch input method by "ctrl+space", It should take one second or more.

Switch input method?  Changing Overtype?  You doing something like, "forgive the crudeness":

using System;
using System.Windows.Forms;
using ScintillaNET;

namespace WindowsFormsApp2
{
    public partial class Form1 : Form
    {
        public Form1() => InitializeComponent();

        private void Form1_Load(object sender, EventArgs e)
        {
            sc.ClearCmdKey(Keys.Insert); //Clear the default binding

            sc.AssignCmdKey(Keys.Control | Keys.Space, Command.EditToggleOvertype);

            testOvertype.ShortcutKeyDisplayString = "Ctrl + Space";
        }

        private void sc_UpdateUI(object sender, ScintillaNET.UpdateUIEventArgs e) => label1.Text = ((sc.Overtype) ? "OVR" : "INS" );

        private void testOvertype_Click(object sender, EventArgs e) => SendKeys.Send("^ ");
    }
}

It's instant for me and I get the side-effect of the scintilla control inserting a space every time.  That can be worked around.
If this wasn't what you're talking about, PLEASE let me know.

@tobeypeters OP's issue is about IME.
https://en.wikipedia.org/wiki/Input_method

This issue is about IME.
SciTE is a SCIntilla based Text Editor. It's just work fine. But Text Editor based on ScintillaNET, even the ScintillaNET demo, switching input method by "ctrl+space" will take one second or more. it is annoyance.

Confused.  I have the winform and WPF project and neither have anything about ImeMode and ctrl-space isn't a built-in scintilla shortcut.  Please point me in the right direction.
If I manually set ImeMode to something, it seems instant.  I want to help.

Maybe you only use English instead of using the input method in other languages.

I'd like to figure this out.  Cause, I'm not making an editor.  But ... the system I'm writing has an editor built-in.  I'm trying to provide absolutely everything I can.  If anybody uses my system, they will probably bring those files into Visual Studio or vice-versa anyway.  If the editor portion gets good enough, maybe I can copy it out and make a separate app from it.


HDXDSYS-810 Make changes to IDMC GIDD datasets as per Maria's email

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 95.918%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
141

Relevant Lines:
147

💛  - Coveralls


realpath fail on MacOS

Description
When realpath is used on a non existing path, it output a stat error:
# (from function `assert_output' in file test/test_helper/bats-assert/src/assert_output.bash, line 194,
#  in test file test/realpath.bats, line 84)
#   `assert_output ""' failed
#
# -- output differs --
# expected :
# actual   : stat: : stat: No such file or directory

When realpath is used on a relative path, it output either the relative path or a semi absolute path:
 # (from function `assert_output' in file test/test_helper/bats-assert/src/assert_output.bash, line 194,
#  in test file test/realpath.bats, line 36)
#   `assert_output "${EXISTING_FILE}"' failed
#
# -- output differs --
# expected : /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/tmp.0wXa8mMl
# actual   : /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/./tmp.0wXa8mMl
# --
#
not ok 163 get realpath from filename
# (from function `assert_output' in file test/test_helper/bats-assert/src/assert_output.bash, line 194,
#  in test file test/realpath.bats, line 45)
#   `assert_output "${EXISTING_FILE}"' failed
#
# -- output differs --
# expected : /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/tmp.jZyr783Y
# actual   : tmp.jZyr783Y
# --
#

Solved by using code from bats-core, and fixing tests to take account of MacOS /privacy/var.


Default image pull policy to Always

Since the default tag is "latest" and features arrive fairly frequently, it's more intuitive to have the latest image be more aggressive about checking for updates in the registry. The download itself is skipped if the latest image is already on the node, so it's a minor network penalty at pod creation time.

Would have helped https://github.com/itzg/docker-minecraft-server/issues/3179


feat: add biome as formatter

add biome to format json and javascript things.
biome is a replacement for prettier with better performance.
I'm not using npx to run command since npx noticeably slow down the startup of biome.
biome refuse to format file with extension javascript, thus  I'm using js instead of javascript in the test.
Estimated time consumption for different commands:
time npx prettier --stdin-filepath demo.json
Executed in  457.11 millis    fish           external
usr time  432.23 millis    0.00 micros  432.23 millis
sys time  199.27 millis  703.00 micros  198.57 millis
time npx @biomejs/biome format --write demo.json
Executed in    1.15 secs      fish           external
usr time  635.26 millis    0.40 millis  634.87 millis
sys time  340.36 millis    1.12 millis  339.24 millis
time prettier --stdin-filepath demo.json
Executed in  145.69 millis    fish           external
usr time  120.74 millis  363.00 micros  120.38 millis
sys time   89.82 millis  113.00 micros   89.71 millis
time biome format --write demo.json
Executed in   57.95 millis    fish           external
usr time   30.06 millis  415.00 micros   29.64 millis
sys time   48.54 millis  763.00 micros   47.78 millis

lgtm!


[Bug]: The Community Embedding step in the official documentation is not performed

Do you need to file an issue?

[ ] I have searched the existing issues and this bug is not already filed.
[ ] My model is hosted on OpenAI or Azure. If not, please look at the "model providers" issue and don't file a new one here.
[ ] I believe this is a legitimate bug, not just a question. If this is a question, please use the Discussions area.

Describe the bug
The Community Embedding step in the official documentation(https://microsoft.github.io/graphrag/posts/index/1-default_dataflow/) is not performed,I do not see the result of vectozation of the community embedding in create_final_community_reports.parquet file

Steps to reproduce
No response
Expected Behavior
No response
GraphRAG Config Used
# Paste your config here

Logs and screenshots
No response
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

All text fields flow through an optional embedding step, but only one is turned on by default, entity.description. We use the entity description embedding for local search, but there are no other places we use the other embeddings right now, so we turn them off the save time and expense. We have parameters in the config to turn on any of the embedding fields if you need them for other processes though - please see GRAPHRAG_EMBEDDING_TARGET and GRAPHRAG_EMBEDDING_SKIP if you need to enable others

All text fields flow through an optional embedding step, but only one is turned on by default, entity.description. We use the entity description embedding for local search, but there are no other places we use the other embeddings right now, so we turn them off the save time and expense. We have parameters in the config to turn on any of the embedding fields if you need them for other processes though - please see GRAPHRAG_EMBEDDING_TARGET and GRAPHRAG_EMBEDDING_SKIP if you need to enable others

@natoverse, Thank you very much for your answer. Based on the above, I have a new question about how to configure to enable the Community Embedding. I do not find this configuration in the setting.yaml file

This is the full list of fields that GraphRAG can embed:

text_unit.text
document.raw_content
entity.name
entity.description
relationship.description
community.title
community.summary
community.full_content

If you want to turn on selected fields, there is a two step process. First, set GRAPHRAG_EMBEDDING_TARGET=all. This will turn on every optional field. However, if you only want to add community.summary along with the default entity.description, you would skip all of the others: GRAPHRAG_EMBEDDING_SKIP=text_unit.text,document.raw_content,entity.name,relationship.description,community.title,community.full_content
The env vars map to settings.yml hierarchically, e.g.:
embeddings:
    target: "all"
    skip: [text_unit.text,...]


Provide alternative options on going to court page

https://trello.com/c/hJ2mmMc0/394-alternative-options-to-court-page

Released in https://github.com/ministryofjustice/pflr-cait/releases/tag/v1.0.5


Improve readme with information that are not implicit

Added troubleshooting for libvirt-devel pkg that is a requirement.
Seperate some doc from readme, referenced;
and finally, added a small reference about the contributors on author section
Read this here
A good readme is always encouraging new contributors.

Coverage decreased (-8.4%) to 49.397% when pulling 3329d7b3dfd0a3dc1f50618325cee6ec09d56e2c on MalloZup:impr-layout into 4907902bc70d82681d6f116f4095d7b84af503ca on dmacvicar:master.

Coverage decreased (-8.4%) to 49.397% when pulling 3329d7b3dfd0a3dc1f50618325cee6ec09d56e2c on MalloZup:impr-layout into 4907902bc70d82681d6f116f4095d7b84af503ca on dmacvicar:master.


add real weather feature

For example, when it rains in Asia/Jakarta, it automatically rains in-game too.

Sorry, wont be doing this. The intention here is to be as much like vanilla as possible. You can fork and do it though.


Automated installation not working.

After clicking on build now button. Nothing happening.
Older non dark gd index working fine. Can u please check in this.

use the manual method. builder isn't working


Change mirror to kernel.org for cygwin

When looking at the team-cymru.org site it shows a sunset message.
We could use something like rsync://rsync.osuosl.org/cygwin/ instead
if that is preferred.

The file we are currently mirroring from the old repo appear to match the files in the new suggested repo (down to mtime). Merging.


fix fhirType value for DateTimeType in QuestionnaireResponseValidator

fix fhirType value for DateTimeType

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1009
Description
Clear and concise code change description.
fix fhirType value for DateTimeType in QuestionnaireReponseValidator which was causing the crash
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: Bug fix
Screenshots (if applicable)

Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read How to Contribute
[x] I have read the Developer's guide
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[ ] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

can you add a test case what would fail with the old code but pass with the new code?

I'm not sure what exactly we want to test here. Any dateTimeType answer expectd would have caused the crash in the old code because DateTimeType.fhirType() returns dateTime and not datetime. Do you want me to add tests to ensure that having a DateTimeType response item does not cause a crash?.

awaiting review


Soft deletes for messages

Hi,
Great package!
As I could notice Threads and Participants use soft deletes.
However, Message model doesn't use soft deletes.
Is there any particular reason you don't use it in on Message model?
It would be great to include soft deletes into Message model as if somebody needs to delete it completely, he can use forceDelete on model.

@a-komarev Thank you. Yeah, I know that. This was just notice added as all other model have soft deletes and only messages doesn't have.

I think it's not done because of optimization reasons. We need @cmgmyr to join the discussion.

@zlorkovic hey, and thanks for using the package!
Honestly, I can't remember if I had an initial reason for this, or it was a simple oversight. I see no reason to not include soft deletes for messages though. This might work well with some new changes in the future too.
I have this pretty much done right now, so I'll push the code shortly and tag a new version in a few days.


Offset behaves oddly for negative values

When using:
{placement: 'bottom', offset: '10'}
the tooltip is correctly placed 10px below the default position. However when using:
{placement: 'bottom', offset: '-10'}
the tooltip is placed approximately 20px above and 10px to the left of the default position - the expected position is 10px directly above the default position.
The tooltip also changes placement to top.
Using 2.0.0-rc.25

You need to disable the flip modifier which is switching the position when the popper overlaps with the target:
popperOptions: {
  modifiers: {
    flip: {
      enabled: false,
    },
  },
}

See this issue


fix(crit): read from stdin in CLI app

The crit CLI app now supports reading input from stdin. Input can also
be sent through a pipe or I/O redirection.
Signed-off-by: Prajwal S N<EMAIL_ADDRESS>

Codecov Report

Merging #77 (0b59e91) into master (25c584f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #77   +/-   ##
=======================================
  Coverage   62.04%   62.04%
=======================================
  Files           1        1
  Lines         137      137
=======================================
  Hits           85       85
  Misses         37       37
  Partials       15       15

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 25c584f...0b59e91. Read the comment docs.


fix: GOFF Authorize connecting in a secure way to the websocket

This PR

allow wss connection to the WebSocket endpoint.

Related Issues
Fixes #624

I'm looking into the FOSSA issue.

The licensing issue has been resolved. You can either ignore the failed check or force a recheck.

The licensing issue has been resolved. You can either ignore the failed check or force a recheck.
Thanks look good now !


TableName on dynamic generated strucs

Your Question
Hello! I want to know if it is possible to create a table with relations from a struct created by reflection
I run into the problem that I can't assign a name to it since it doesn't have a Type.Name

The document you expected this should be explained

Expected answer

example:
struct1Fields := []reflect.StructField{
	{
		Name: "field1",
		Type: reflect.TypeOf("string"),
	},
}
struct1 := reflect.StructOf(struct1Fields)
struct2Fields := []reflect.StructField{
	{
		Name: "struct1",
		Type: struct1,
	},
}
struct2 := reflect.StructOf(struct2Fields)
DB.AutoMigrate(reflect.New(struct1), reflect.New(struct2))

my problem is when i try to create that "tables" GORM can't get a properly name to it in MySQL so tries to run a malformed query (CREATE TABLE `` (....)
we tried some ways to implement "Tabler" interface with anonymous fields but isn't possible because when schema tries to parse tablename creates a new *struct value and our anonymous field lose Tabler interface implementation
schema.go:123
modelValue := reflect.New(modelType)
tableName := namer.TableName(modelType.Name())
if tabler, ok := modelValue.Interface().(Tabler); ok {
	tableName = tabler.TableName()
}
if en, ok := namer.(embeddedNamer); ok {
	tableName = en.Table
}
if specialTableName != "" && specialTableName != tableName {
	tableName = specialTableName
}

any idea about how can we solve it? is possible to implement a new Namer strategy with modelType as parameter?
like this:
tableName := namer.TableName(modelType.Name())
tableName = namer.TableNameFromType(modelType)

refer to https://gorm.cn/docs/conventions.html#TableName

refer to https://gorm.io/docs/conventions.html#TableName


[Cosmos] Remove unused classes that were added from shaded libraries

Remove classes that are not used in:
guava25 package
guava27 package
apachecommons package

@srnagar - a lot of the types are referenced indirectly (via inheritances, composition, utility) by the few types that the cosmos directly use. I removed many classes that were not referenced at all, but yes many methods in shaded types can be removed but need to ensure that won't cause any break.

Adding missing cosmos and client label so this will show up in the Cosmos' team radar now.

Went over the classes in these packages, and currently all of them are being used. Will circle back on this once we are at cusp of developing next major version of Java SDK for Cosmos DB.


How to create Session with api key?

I need to create Session with API Key.
iam_api_key = 'MY KEY'
iam_serice_endpoint = 'https://s3-api.us-geo.objectstorage.softlayer.net'
iam_service_id = 'ServiceId-123'

token_manager = DefaultTokenManager(api_key_id=iam_api_key)
session = ibm_boto3.session.Session(token_manager=token_manager, ibm_service_instance_id=iam_service_id)

It doesn't work, getting
File "/usr/local/lib/python3.6/site-packages/ibm_botocore/auth.py", line 711,
in get_signature  return self.sign_string(string_to_sign)
File "/usr/local/lib/python3.6/site-packages/ibm_botocore/auth.py", line 621, in sign_string
new_hmac = hmac.new(self.credentials.secret_key.encode('utf-8'),
AttributeError: 'NoneType' object has no attribute 'encode'

I do manage to create Session with access key and secret key. This one for example, works fine:
session = ibm_boto3.session.Session(aws_access_key_id=access_key,
aws_secret_access_key=secret_key)

@paul-carron can you please check this? thanks

Currently looking into it. Will update shortly.

@paul-carron thanks. I also tried without TokenManager and provide api key directly to ibm_boto3.session.Session(...)
Got the same exception


Allow configurable prefix size and wire up to jbrowse text-index

Xref https://github.com/GMOD/jbrowse-components/discussions/2803


Fix ci

The problem was travis not installed for near organization, installed now

@nearmax Seems near-sdk-rs refer to near/borsh, but borsh is still under nearprotocol now.

Should I move borsh to near now? Based on moves i did recently original
nearprotocol one would be redirect by github, so it's probably not breaking
any code using nearprotocol/borsh
On Sun, Mar 22, 2020 at 4:58 PM Maksym Zavershynskyi <
<EMAIL_ADDRESS>wrote:

@nearmax approved this pull request.
Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/near/near-sdk-rs/pull/113#pullrequestreview-379071310,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADFFFCG7TEQQELQFYNK2DUDRI2Q2HANCNFSM4LQRP67Q
.

Should I move borsh to near now? Based on moves i did recently original
nearprotocol one would be redirect by github, so it's probably not breaking
any code using nearprotocol/borsh

Yes


python311Packages.slack-sdk: 3.26.2 -> 3.27.0

Diff: https://github.com/slackapi/python-slack-sdk/compare/refs/tags/v3.26.2...v3.27.0
Changelog: https://github.com/slackapi/python-slack-sdk/releases/tag/v3.27.0
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

6 packages built:

prowler
prowler.dist
python311Packages.slack-bolt
python311Packages.slack-bolt.dist
python311Packages.slack-sdk
python311Packages.slack-sdk.dist

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

6 packages built:

prowler
prowler.dist
python311Packages.slack-bolt
python311Packages.slack-bolt.dist
python311Packages.slack-sdk
python311Packages.slack-sdk.dist

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

6 packages built:

prowler
prowler.dist
python311Packages.slack-bolt
python311Packages.slack-bolt.dist
python311Packages.slack-sdk
python311Packages.slack-sdk.dist


Error: An object could not be cloned

Describe the bug
I can't connect to server because it says

(both SSH & password)
Versions
Ray version (you can see this in "About Ray"): 1.18.1, just updated after a long time without use on this machine
You can use composer show to get the version numbers of:

spatie/ray package version:
spatie/laravel-ray package version (if applicable):

PHP version: 8.0.12
Laravel version: 8.69.0
Expected behavior
Connection to host
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS 12.0.1

Additional context
I just updated to a new version of macOS so it can be something with this

Thanks for reporting.
We've received similar reports, and will investigate and fix this very soon.

Is there a temporary solution, like downgrade or something?

Is there a temporary solution, like downgrade or something?

Let me respond to myself, maybe it helps you: Yes, version you download from the download page (at least for Apple Sillicon) is actually v1.17.1 and it works flawlessly.

I have the same error - I am trying to download an older version for Mac (intel) but unable to do so, the download page only seems to allow me to download the very latest version - can anyone help?

My bad, releasing a fix now!


[Fix #12087] Fix false positives for Style/ArgumentsForwarding

Additionally fixes:

#12089
#12096
#12100

In Ruby < 3.2 this cop was overzealous and not considering various additional args/kwargs and other reasons to not allow forward-all (...), and was additionally assuming that forward-all meant all arguments were replaced, which was not always the case.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

//cc @koic @bbatsov - please let me know what you think. I think there's a few points of simplification in the implementation that I've tried to highlight, and of course the bug fixes

@owst This looks good to me. Can you squash your commits into one?

@owst This looks good to me. Can you squash your commits into one?

Thanks @koic - done!

Thanks!


After installing meg still it asking for installing again and again

Specifications

Operating system and version:Kali 2018

Issue details

this fixed the issue
download a binary from the the releases page (e.g. if you're on 64 bit linux it's meg-linux-amd64-0.2.2.tgz, if you're on 32 bit linux it's meg-linux-386-0.2.2.tgz).
Extract the tarball:
▶ tar xvzf meg-linux-amd64-0.2.2.tgz
meg
And move it into your $PATH, e.g. in /usr/bin/:
▶ sudo mv meg /usr/bin/meg


feat: Increase code coverage transaction

Greetings again, team. I have created the following pull request aiming to enhance the code coverage of the transaction tests.
In this case it is not 100%, but other few lines remaining.

coverage: 96.599% (+1.5%) from 95.102% when pulling 865b2181925b66f9235da27144d441d729c94abe on rodolfopietro97:increase-code-coverage-transaction into 0da202cbdc8000c4ffbbcb7f872df8a0ba390689 on vechain:master.


DEVELOP-1206 - required_paths is now a configurable variable

What problems does this PR solve?
This PR removes the hardcoded requirement that all archiving from biotanks must validate the presence of an Unaligned folder. This enables us to archive Olink folders, which do not have this subfolder. Instead, the service now accepts a required_dirs parameter.
This PR will be paired with another PR to snpseq_packs that will send these variables.
How has the changes been tested?
Automatic tests plus local testing using Postman client.
Reasons for careful code review
If any of the boxes below are checked, extra careful code review should be inititated.

[ ] This PR contains code that could remove data

@withrocks can you please take another look?

Looks cool! I think it would be good to get an extra review from @matrulda  because of the remove logic.

Yeah, the check for an Unaligned dir was hardcoded to run on biotanks only, so we would not have caught it on staging. Sigh.


feat: wrap constructor callback with block concurrency while

try to implement #1 proposed by Glen

Merge as is since it is a backward compatible PR.


February Calendar Displaying Wrong January Previous Day

February calendar display wrong January previous day, if it exists.
It display 28 or 29 as January last day, should be 31.
I also confirm it happens in all of your calendar object on your demo page https://chmln.github.io/flatpickr

Found it.
https://github.com/chmln/flatpickr/blob/gh-pages/dist/flatpickr.js#L450
If you pass 0, I wanna get January days in month, the condition will fall to self.currentYear as month. Since 0 || 'anything' will be 'anything'.


Grape 1.4

Besides upgraded Grape, it changes the response body for 204 HTTP status, now it is an empty string instead of an empty hash.
Rack introduced lazy body initialization (https://github.com/rack/rack/commit/8c62821f4a464858a6b6ca3c3966ec308d2bb53e)
which only works with strings. However, when 204 HTTP status is returned, Grape doesn't stringify the response, thus, a hash is returned as a response body. Rack::MockResponse fails with
TypeError (no implicit conversion of Hash into String)

It could be a bug in the JsonAPI resources gem, why does it return an empty hash for 204 HTTP status?!. Definitely, it isn't a bug in Grape, nor Rack. The logic for formatting the response body in Grape hasn't changed for 5 years.
The main motivation for this upgrade is a fix for memory leak https://github.com/ruby-grape/grape/pull/2084.

Looks good to me. Thanks, Dima!


Task - Edit type specimen - type type selection

When I select type type (holotype, paratype etc) the white area next to the type name is click sensitive. I do changes by mistake when clicking on white area. Restrict the hyperlink to the name only

white area next to the type type are still select the type (holotype or lectotype)


Bug of selecting targets when computing logits contribution

In this piece of code:
https://github.com/TomFrederik/unseal/blob/main/unseal/hooks/common_hooks.py#L259-L276
`            temp = attn_weights[...,None] * value[:,:,None]
if len(c_proj.shape) == 2:
c_proj = einops.rearrange(c_proj, '(head_dim num_heads) out_dim -> head_dim num_heads out_dim', num_heads=attn_output.shape[1])
c_proj = einops.rearrange(c_proj, 'h n o -> n h o')
temp = temp[0,:,:-1] # could this be done earlier?
new_temp = []
for head in tqdm(range(temp.shape[0])):
new_temp.append([])
for i in range(math.ceil(temp.shape[1] / batch_size)):
out = temp[head, i*batch_size:(i+1)*batch_size] @ c_proj[head]
out = out @ vocab_embedding # compute logits
new_temp[-1].append(out)
            # center logits
            new_temp[-1] = torch.cat(new_temp[-1])
            new_temp[-1] -= torch.mean(new_temp[-1], dim=-1, keepdim=True)
            # select targets
            new_temp[-1] = new_temp[-1][...,torch.arange(len(target_ids)), target_ids]#.to('cpu')`

I found the last line of select targets problematic. new_temp[-1] should be of size (dst_positions, src_positions, vocab_size).  We need to compute the contribution of information movement src -> dst to the logits of target_ids at dst positions.  So this line should be:
new_temp[-1] = new_temp[-1][torch.arange(len(target_ids)), :, target_ids]
Right?

Da Xiao

I also wrote a cleaner version of this piece of code using einsum:
temp = torch.einsum('bnij,bnjd->nijd', attn_weights, value) # b=1
temp = temp[:, :-1]
new_temp = torch.einsum('nijd,nde->nije', temp, c_proj)
new_temp = new_temp @ vocab_embedding  # nije,ev->nijv
new_temp -= torch.mean(new_temp, dim=-1, keepdim=True)
new_temp = new_temp[:, torch.arange(len(target_ids)), :, target_ids] # our
new_temp = new_temp[:, :, torch.arange(len(target_ids)), target_ids] # your
new_temp = rearrange(new_temp, 'i n j -> n i j')

Agreed for the first one, disagree on the second one because of batching issues (see emails I send to you).
Thanks a lot for catching this one! :))


Trying to set uiLayoutContainer's size via controller varaible

I am trying to set the uiLayoutContainer's size attribute from a controller's variable like thus:
$scope.defaultWidth = "200px";

But it doesn't work. What am I doing wrong? Is it supposed to work this way?

Should be:

You are completely right. My lack of Angular knowledge. Thank you :-)
However, when I resize the window after having pulled the split bar, the layout jumps back to the initial sizes. It happens in your plnkr too. How can I avoid that?

@nitech no worries.  Looking into #100.


Detect ICU with pkg-config when possible.

Better detection of ICU in bootstrap.sh with the help of pkg-config.
Checking for the header under /usr/include/unicode is not very robust because on some Linux distros the header is not dirctly in include. E.g. on older Ubuntu 14.04 it is in /usr/include/x86_64-linux-gnu/unicode.
Asking pkg-config is a better way to check if the library is installed.

@mjcaisse please merge - thanks.

command is a builtin command for bash but I don't think it works for the standard /bin/sh interpreter. I'm not sure if which is more universal for checking the existence of pkg-config but platforms that don't default /bin/sh to bash are going to fail with the script if this is merged.
Thoughts?

Command is more portable than which https://stackoverflow.com/questions/592620/how-to-check-if-a-program-exists-from-a-bash-script
It is part of POSIX http://pubs.opengroup.org/onlinepubs/9699919799/utilities/command.html

Thank you @dimztimz for the feedback.

Reminder that this commit has not been cherry-picked into master yet.


Bug - RegexNotFoundError: Unable to extract user

This has started happening 7 days ago when downloading from DailyMotion
Youtube_dl version
~/bin/youtube-dl --version
2019.06.08
Command used:
./youtube_dl https://www.dailymotion.com/haa-2-2/videos --verbose -i --match-title 'home|away' --reject-title 'preview|peek' --dateafter now-7days --download-archive download.log -o ~/dl/
Error:
`[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'https://www.dailymotion.com/haa-2-2/videos', u'--verbose', u'-i', u'--match-title', u'home|away', u'--reject-title', u'preview|peek', u'--dateafter', u'now-10days', u'--download-archive', u'/home/xuxo/dl/videos/Home-Away/.downloaded', u'-o', u'Home&Away-%(upload_date)s-%(id)s.%(ext)s']
[debug] Encodings: locale ISO-8859-1, fs ISO-8859-1, out ISO-8859-1, pref ISO-8859-1
[debug] youtube-dl version 2019.06.08
[debug] Python version 2.7.14 (CPython) - Linux-4.4.0-109-generic-x86_64-with-debian-stretch-sid
[debug] exe versions: avconv 3.4.1-1, avprobe 3.4.1-1, ffmpeg 3.4.1-1, ffprobe 3.4.1-1, rtmpdump 2.4
[debug] Proxy map: {}
[dailymotion:user] haa-2-2: Downloading webpage
ERROR: Unable to extract user; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/home/xuxo/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
ie_result = ie.extract(url)
File "/home/xuxo/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/home/xuxo/bin/youtube-dl/youtube_dl/extractor/dailymotion.py", line 486, in _real_extract
webpage, 'user'))
File "/home/xuxo/bin/youtube-dl/youtube_dl/extractor/common.py", line 1014, in _html_search_regex
res = self._search_regex(pattern, string, name, default, fatal, flags, group)
File "/home/xuxo/bin/youtube-dl/youtube_dl/extractor/common.py", line 1005, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract user; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
`

full_user isn't finding any matching regex in the webpage HTML. The regex might have to change. I assume it's trying to get the title of the webpage?
https://github.com/ytdl-org/youtube-dl/blob/695720ebe81166b3ee249eb3916e3c7819ef57a8/youtube_dl/extractor/dailymotion.py#L484-L486
The playlist HTML scraper doesn't seem to find any video URLs on the user's videos page. Maybe dailymotion's HTML has changed?
https://github.com/ytdl-org/youtube-dl/blob/695720ebe81166b3ee249eb3916e3c7819ef57a8/youtube_dl/extractor/dailymotion.py#L469
^This for loop isn't finding any videos.

Then the dailymotion plugin must be broken then.
As I said. The command above was working properly, and finding/downloading the videos matching the regex a month ago. Sadly it is no longer the case

Even though thats rather obvious, but it still didn't change.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--ffmpeg-location', u'D:/ffmpeg/bin/', u'--verbose', u'https://www.dailymotion.com/GenialDaneben']
[debug] Encodings: locale cp1252, fs mbcs, out None, pref cp1252
[debug] youtube-dl version 2019.08.02
[debug] Python version 2.7.14 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg 4.0.2, ffprobe 4.0.2
[debug] Proxy map: {}
[dailymotion:user] GenialDaneben: Downloading webpage
ERROR: Unable to extract user; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "c:\python27\lib\site-packages\youtube_dl\YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "c:\python27\lib\site-packages\youtube_dl\extractor\common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "c:\python27\lib\site-packages\youtube_dl\extractor\dailymotion.py", line 492, in _real_extract
    webpage, 'user'))
  File "c:\python27\lib\site-packages\youtube_dl\extractor\common.py", line 1014, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "c:\python27\lib\site-packages\youtube_dl\extractor\common.py", line 1005, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract user; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.


Building and running elvish on MSYS2 + Windows 10 mangles paths

This is going to be a problem for anyone trying to use elvish under Windows 10 + MSYS2:
$ ./elvish.exe -c 'put $args' x /x/y
▶ [x X:/y]

This came to my attention because of a golang-nuts mailing list discussion thread that I commented on: https://groups.google.com/forum/#!topic/golang-nuts/Z2LY7mKIzO0. See also
http://www.mingw.org/wiki/Posix_path_conversion
https://stackoverflow.com/questions/7250130/how-to-stop-mingw-and-msys-from-mangling-path-names-given-at-the-command-line
https://github.com/msys2/MSYS2-packages/issues/84
I confirmed that using the env var workaround "fixes" this problem:
$ ./elvish.exe -c 'put $args' x /x/y
▶ [x X:/y]
$ MSYS2_ARG_CONV_EXCL="*" ./elvish.exe -c 'put $args' x /x/y
▶ [x /x/y]

Obviously that isn't a practical solution. If the Go community doesn't come up with a solution generally applicable to all Go programs it might be necessary for Elvish to implement its own solution.

Note that using the MSYS2 mktemp program produces output that assumes elvish is linked against the MSYS2 DLLs. For example, /tmp/tmp.uV8LueyETl which, for elvish, is really C:/msys64/tmp/tmp.uV8LueyETl. I noticed this because @zzamboni's "alias" module uses mktemp which results in this startup error:
Exception: failed to open file /tmp/tmp.uV8LueyETl: open /tmp/tmp.uV8LueyETl: The s
ystem cannot find the path specified.
Traceback:
  C:\Users\krade\.elvish\lib/github.com/zzamboni/elvish-modules/alias.elv, line 18:
      echo 'aliases['$name'] = $'$name'~' > $-tmpfile
  C:\Users\krade\.elvish\lib/github.com/zzamboni/elvish-modules/alias.elv, line 92:
          -load-alias $name $file
  C:\Users\krade\.elvish\lib/github.com/zzamboni/elvish-modules/alias.elv, line 97:
    init
  C:\Users\krade\.elvish\rc.elv, line 34:
    use github.com/zzamboni/elvish-modules/alias

Note that I really don't care if elvish works on Windows. I'm only reporting these issues because there seems to be an implicit goal to support Windows as well as UNIX like platforms.


provide zzio606 module for driver and document

Signed-off-by: wusheng.tws<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

感谢提交 Pull Requests！
Thanks for submitting a pull request.

Security Check : 239aaaac3b2eff58557be1003cda21f94b832be2

auto build success,number:49

autotest success,number:1086


Could not load CSS

Reproduction
I have a Remix project that I just updated to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I am using Tailwind CSS with PostCSS and loading it directly in my app/root.tsx file:
import './styles/tailwind.css'
Before the update, everything worked fine. However, after the update, the page initially loads with no CSS applied, and then after a few milliseconds, it displays the page with the correct CSS. In the terminal, the following message appears:
Could not load /Users/cleytoncarvalho/Projects/test/app/styles/tailwind.css
Here’s my package.json file:
"dependencies": {
"@remix-run/node": "^2.15.1",
"@remix-run/react": "^2.15.1",
"@remix-run/serve": "^2.15.1",
"isbot": "^5.1.17",
"react": "^19.0.0",
"react-dom": "^19.0.0",
"tailwindcss-animate": "^1.0.7"
},
"devDependencies": {
"@remix-run/dev": "^2.15.1",
"@remix-run/route-config": "^2.15.1",
"@remix-run/routes-option-adapter": "^2.15.1",
"@tailwindcss/typography": "^0.5.15",
"@types/react": "^19.0.1",
"@types/react-dom": "^19.0.2",
"autoprefixer": "^10.4.20",
"postcss": "^8.4.49",
"remix-flat-routes": "^0.6.5",
"tailwindcss": "^3.4.16",
"typescript": "^5.7.2",
"vite": "^6.0.3",
"vite-tsconfig-paths": "^5.1.4"
}
Does anyone know how to fix this issue?
System Info
System:
    OS: macOS 15.1.1
    CPU: (8) arm64 Apple M1
    Memory: 66.34 MB / 8.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.15.0 - ~/.asdf/installs/nodejs/20.15.0/bin/node
    npm: 10.7.0 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: 9.10.0 - ~/Library/pnpm/pnpm
    bun: 1.0.29 - ~/.asdf/shims/bun
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 131.0.6778.140
    Safari: 18.1.1

Used Package Manager
pnpm
Expected Behavior
CSS should load properly.
Actual Behavior
CSS does not load properly.

i have the same issue, but i'm still on "react": "^18.2.0".
I'm currently debugging,...


Test 'database is locked'

This is not intended to be merged for now since it takes a long time to run (10s to 100s seconds on a 32 core Xeon E5-2620 @ 2.10GHz).

Coverage remained the same at 58.169% when pulling 72c4216e779bc1fd78da77e2b12ce6ce4b2ee04e on typeless:database-is-locked into 3198c7714d5f8be941dd06c30208e623aa5abf90 on mattn:master.


Add CoDa.jl extension

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Version 2 roadmap

Branch: next-ver-2

[x] normalize generated contract names (BREAKING)
[x] abi parser should normalize empty names to undefineds (BREAKING)
[x] change names of target packages since NPM keeps on flagging them as spam 😢

https://github.com/ethereum-ts/TypeChain/pull/219

https://github.com/ethereum-ts/TypeChain/releases/tag/v2.0.0


Eclipse adjustments list

Open Resource (control x) includes .class files
https://stackoverflow.com/questions/2632419
This seems to be happening when I build a web app in a bin directory.  Click on project properties, then Resource/ Resource Filters / Add Filter / Exclude all / Folders / folder named 'bin'

This is pertinent:

Exclude the following for a complete cleanup of Open Resource dialog
Files *.class
Folder bin
Folder build
Folder target


When you click on "optimal step size"

Maybe should set the slider to the optimal value? It's a little hard to find it by sliding around.

Fixed


How to import the dyld_shared_cache with analyzeHeadless in the file system mode.

I want to make some automatic analysis on a dylib in dyld_shared_cache, so I hope to import the dyld_shared_cache in file system mode and load the target nested file. It is easy in Ghidra GUI, but when I tried to use analyzeHeadless's -import option, it costs a long time and seems to import the whole binary. How should I solve this issue? Thanks!

Current the headless analyzer doesn't use / understand 'file system mode'.
You could manually handle importing using GFilesystem stuff yourself in a headless pre-script, but its a bit involved.

Thanks for the reply!
Could you tell me which part of the source code can I refer to for importing dyld_shared_cache with GFileSystem.

Thanks for the reply!
I will close this issue and try it.


Setup connection to Trusted-Intermediary using OAuth2

This PR setup RESTTransport to connect to Trusted-Intermediary staging server using OAuth2 (two-legged) authentication.
Test Steps:

Checkout the PR
./gradlew clean package
docker-compose up
./prime multiple-settings set -i ./settings/staging/0149-etor.yml
Setup local vault with
Path for this secret: FLEXION--ETOR-SERVICE-RECEIVER
{
"@type": "UserApiKey",
"apiKey": "Flexion Private Key from KeyBase",
"user": "flexion"
}
Download the attached test file: etor-patient-bundle-test-json.txt
Copy the content of the json file to Postman Body
Set header as follow:
client: flexion.etor-service-sender
Content-Type: application/fhir+ndjson
x-functions-key: 09xifjf....444s
9.) POST: http://<IP_ADDRESS>:7071/api/reports

10.) Check history API:  GET: http://<IP_ADDRESS>:7071/api/waters/report/b953c2b0-80ce-4ea9-a25b-067a7e074974/history

Test file:
etor-patient-bundle-test-json.txt
Changes

Added Signed JWT generation from Private Key.
Added authType: two-legged to transport setting.
Added fakePrivateKey for unit test to cover the new code.

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[x] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue
https://app.zenhub.com/workspaces/engagement-6166edbd409257001e09f1b2/issues/gh/cdcgov/prime-reportstream/9675

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

@oslynn, is this code supposed to know the token we return expires after 5 minutes and the code will re-call our token endpoint to get a new one?  Or is that code above/outside this area?


Add an option for branch name

The new default in recent git versions is "main" for the main branch, but we still want to be able to support "master" from the past 20 years of git repos.

Implemented in e43289468cc3f6f68a7c85e65402d249eb897c4c


Dashboard auth

Configure Dashboards to use SSO

[x] Kubernetes dashboard (requires auth proxy)
[x] Grafana
[x] Harbor
[x] Minio
[x] Kiali
[ ] Make sure they all work for ready-to-go mode. This will require reconfiguring the auth provider endpoints.

DONE


[cloud_firestore] Query.orderBy(FieldPath.documentId) throws exception at v0.14.0

Describe the bug
Query.orderBy(FieldPath.documentId) stated throwing exception at v0.14.0.
It works well until v0.13.7+1.
To Reproduce
Steps to reproduce the behavior:

Run this.

FirebaseFirestore.instance
    .collection('foo')
    .orderBy(FieldPath.documentId)
    .get();

Error log

E/flutter (12130): [ERROR:flutter/lib/ui/ui_dart_state.cc(171)] Unhandled Exception: type 'FieldPathType' is not a subtype of type 'FieldPath'
E/flutter (12130): #0      Query.orderBy (package:cloud_firestore/src/query.dart:238:15)
E/flutter (12130): #1      _DropdownButton.build.<anonymous closure> (package:example/pages/paging_page/paging_page.dart:119:14)
E/flutter (12130): #2      PopupMenuButtonState.showButtonMenu.<anonymous closure> (package:flutter/src/material/popup_menu.dart:1120:28)
E/flutter (12130): #3      _rootRunUnary (dart:async/zone.dart:1198:47)
E/flutter (12130): #4      _CustomZone.runUnary (dart:async/zone.dart:1100:19)
E/flutter (12130): #5      _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
E/flutter (12130): #6      Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
E/flutter (12130): #7      Future._propagateToListeners (dart:async/future_impl.dart:725:32)
E/flutter (12130): #8      Future._completeWithValue (dart:async/future_impl.dart:529:5)
E/flutter (12130): #9      Future._asyncCompleteWithValue.<anonymous closure> (dart:async/future_impl.dart:567:7)
E/flutter (12130): #10     _rootRun (dart:async/zone.dart:1190:13)
E/flutter (12130): #11     _CustomZone.run (dart:async/zone.dart:1093:19)
E/flutter (12130): #12     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
E/flutter (12130): #13     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:1037:23)
E/flutter (12130): #14     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
E/flutter (12130): #15     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)

Expected behavior
There are no errors.
Additional context
There's a problem here.
field_path.dart
  static FieldPathType get documentId {
    return FieldPathType.documentId;
  }

FieldPath.documentId is FieldPathType, but it is tried to be assigned to FieldPath.
query.dart
  Query orderBy(dynamic field, {bool descending = false}) {
    assert(field != null && descending != null);
    _assertValidFieldType(field);
    assert(!_hasStartCursor(),
        "Invalid query. You must not call startAt(), startAtDocument(), startAfter() or startAfterDocument() before calling orderBy()");
    assert(!_hasEndCursor(),
        "Invalid query. You must not call endAt(), endAtDocument(), endBefore() or endBeforeDocument() before calling orderBy()");

    final List<List<dynamic>> orders =
        List<List<dynamic>>.from(parameters['orderBy']);

    assert(orders.where((List<dynamic> item) => field == item[0]).isEmpty,
        "OrderBy field '$field' already exists in this query");

   // Type mismatch here (FieldPath vs FieldPathType)
    FieldPath fieldPath = field is String ? FieldPath.fromString(field) : field;

Flutter doctor
Run flutter doctor and paste the output below:
❯ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, 1.21.0-9.0.pre, on Mac OS X 10.15.6 19G73, locale en-JP)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] IntelliJ IDEA Community Edition (version 2019.1.3)
[✓] VS Code (version 1.47.3)
[✓] Connected device (6 available)
    ! Error: monoPhone is not connected. Xcode will continue when monoPhone is connected. (code -13)
    ! Error: monoyagon is not connected. Xcode will continue when monoyagon is connected. (code -13)
    ! Error: monoPhone7Plus has recently restarted. Xcode will continue when monoPhone7Plus is unlocked. (code -14)

• No issues found!

@Ehesp
A FieldPath.documentId related error was fixed partically at #3200, but .orderBy(FieldPath.documentId) is still broken.

Release with a fix is now published. Latest versions can be seen here.
Thanks again for the bug report :)


Update ReadMe for Custom Task

This readme is out of date: https://github.com/epfml/disco/blob/develop/information/TASK.md as noted by Dennis Yurkevich

I'll go further and say we need to review/update all docs again, especially the DEV/discojs/discojs-node README's. They should go in greater details on how to interact with the lib in general, not only the training part (e.g. how to get training stats, how to validate/test a model, etc.).

yes we can start from what we have, but indeed could be nice to give some example-based guides on how to build stuff on top of disco - always keeping the audience in mind from their needs, that is to build sth impressive quickly and easily. our own code structure is less important in general, just a means for the first thing. anyway, we have some good base and can improve step by step

yes we can start from what we have, but indeed could be nice to give some example-based guides on how to build stuff on top of disco - always keeping the audience in mind from their needs, that is to build sth impressive quickly and easily. our own code structure is less important in general, just a means for the first thing. anyway, we have some good base and can improve step by step

I completely agree. I thought of making Markdown guides containing code samples of a few lines along explanations — what's expected by the code & what it outputs — similar to what a Jupyter notebook might look like (e.g. in the style of this existing guide)


is it possible to run the sprocket with transparent background?

Hi,
I want to run the sprocket on a Set tob box, So one of the rquirements is to have the webpage displayed but with a transparent background,i.e the video is visible in the area not covered by HTML content.
is this possible with sprocket? do I need to modify some code?
regards

Hi! Of course, it should be possible with the RenderWidgetHostView::SetBackgroundColor(...) API function. You can pass a transparent value to this and after that you can see what is behind your web view. (The color of the application window. I don't exactly know what are the system requirements of transparent windows - my current Linux desktop does not support it.)
I made a little example: https://gist.github.com/davidsz/d75947fa56540b6b38e6
It also sets a TRANSLUCENT_WINDOW flag for the app window, maybe it will work for you.
Note: Most of the web pages have background color, so I think this "feature" is not too useful by itself. (In fact only the minimal HTML documents are transparent.) WebContents::InsertCSS(...) could help you to remove predefined backgrounds from the pages.
Happy experimenting, best regards!